text:
Support for KafkaProxy for @KafkaListener

I think it would be very helpful to have support for swapping out the "backend" of @KafkaListener to support connecting via the Kafka RESTProxy instead of the normal kafka-client.
I realize you may not get 100% support, but I see MessageListenerContainer already has some UnsupportedOperationException defaulted methods, and even just very simple consuming/producing would be a nice addition.

Sounds like a separate implementation and not a mix of concerns for the @KafkaListener.
May you, please, share as much info on the matter as possible?
Maybe some PoC would be really great.
I see that RESTProxy is a part of Confluent Platform: https://docs.confluent.io/current/kafka-rest/docs/index.html. We don't have any integration with that one though...

The reason it would be a mix of concerns, IMO, is that @KafkaListener could/would/should be like PreparedStatement, or JdbcTemplate, where my code doesn't care if I'm pointing at the RESTProxy or using a KafkaClient.
As it stands right now, if I develop a number of @KafkaListeners and then somebody in security says "oh, we just decided to restrict kafka--please go through the services gateway", I have a ton of work to do. If I could just include spring-kafka-rest (separate, but related), configure a RestTemplate, and bob's your uncle.
Since they are ultimately exposing the same thing via two different mechanisms, it seems like it shouldn't be that painful to do... just implement some new factories and impl classes. I'm not that familiar with the way it's been abstracted though, so it could be much worse than in #IvoryTower.

I'd be glad to--I was just trying to get some guidance. This sounds like a good transatlantic task :) I'll try and get something next week.
Should I just do it in spring-kafka/src? or should I add (at least for the sake of reviewing) a spring-kafka-proxy/src?

I don't think a new subproject is necessary; it will share a bunch of stuff from s-k, including the abstract container. Perhaps a new package o.s.k.rest.listener. With, perhaps a rest-based KafkaTemplate in o.s.k.rest.producer or similar.
This will probably have to be a 2.2 feature since we'll need changes in ConcurrentMessageListenerContainer to support a collection of abstract containers instead of
List<KafkaMessageListenerContainer<K, V>> containers;

We'll probably fork off a 2.1.x branch after we release 2.1.5 next week and master can become the 2.2 dev branch.

sounds like a plan.

Any news regarding this topic? i think many will be interested in this feature :)

We're still open to contributions, but it's unlikely to make it into 2.3; the release candidate is due in a couple of weeks.

Any news regarding this issue ? we are interested as well to use Kafka proxy with Kafka-Spring.
Thanks.

There are no updates; the project itself does not have plans to implement it; but we will consider a community contribution.

authors: VictorGetz artembilan garyrussell nnordrum swisair1

repo: spring-projects/spring-kafka


text:
GH-466 - Allow using ApplicationModuleTest outside of module package

Closes GH-466

That's polished and merged.

source: gharchive/pull-request

authors: lukasdo odrotbohm

repo: spring-projects/spring-modulith

license_type: permissive

license_source: github-api


text:
Support kv v2 through spring-data repositories

My previous implementation used kv v1 and something similar to the following standard repository setup:
@Secret(value = "example/location")
class Credentials {
    @Id String id;
    String data1;
    String data2;
}

interface CredentialsRepository extends CrudRepository<Credentials, String> {
}

It would be great if spring-vault also supported something similar to the following for kv v2 for upgrading (or some sort of different versioned repository interface)
@Secret(value = "example/location")
class Credentials {
    @Id String id;
    String data1;
    String data2;
}

interface CredentialsRepository extends CrudRepository<Versioned<Credentials>, String> {
}

This would prevent the need for manually implementing this sort of interface using VaultVersionedKeyValueOperations

Do you want to give it a try by submitting a pull request? We also could make use of @Version to map the CAS parameter/secret version.

I've done some further investigation and there's probably two parts to this:

Allow spring-data repositories to work with un-versioned @Secret using the kv2 backend - the fix for this seems to be rather than using KeyValueTemplate use VaultKeyValueTemplate1/VaultKeyValueTemplate2. VaultTemplate appears to have some logic for this already but the repositories don't seem to use this
The above update to support versioned secrets when using the kv2 backend, which is more complex

I'm struggling to get to grips with the project more compared to other spring projects I'm familiar with due to different patterns in the use of autoconfiguration and beans.

Repository support was added before we had VaultKeyValueOperations and therefore, we never adopted the code to VaultKeyValueOperations. Given that direct VaultOperations comes with its own advantages (e.g. full flexibility) I'm not sure whether we require migrating repository support (namely VaultKeyValueAdapter) to VaultKeyValueOperations at all.
The distinction, whether to use KV1 or KV2 should be derived from the keyspace that maps to the context path. We have already a KeyValueDelegate that distinguishes between both backend versions to retrieve a secret. It would make sense to reuse or extend that class as needed.
Happy to guide you through Spring Vault if you like.

source: gharchive/issue

repo: spring-projects/spring-vault


text:
Please make either SecurityConfiguration#DEFAULT public or the default constructor public

Version: [2.6.1]
Issue
In 2.5.1, the springfox.documentation.swagger.web.SecurityConfiguration#DEFAULT field is public, as is shown here: https://github.com/springfox/springfox/blob/2.5.0/springfox-swagger-common/src/main/java/springfox/documentation/swagger/web/SecurityConfiguration.java#L26. However this was changed in 2.6.x. I'd like to know the reason why it has been changed. And if possible, please make it public again to accommodate the following use case.
Use case
Programmatically configure Swagger's API key, example code in 2.5.0:
@EnableSwagger2
@Configuration
class SwaggerConfiguration {
    private ApiKey apiKey = null;

    @PostConstruct
    void init() {
        // some logic to assign apiKey according to some configuration
    }

    @Bean
    springfox.documentation.swagger.web.SecurityConfiguration security() {
        if (this.apiKey == null) {
            return springfox.documentation.swagger.web.SecurityConfiguration.DEFAULT;
        } else {
            return new springfox.documentation.swagger.web.SecurityConfiguration(null, null, null, null,
                    "Your access token", ApiKeyVehicle.HEADER, apiKey.getKeyname(), null);
        }
    }

Now 2.6.x in order to use the default security config, one needs to do the following, which is very cumbersome.
@EnableSwagger2
@Configuration
class SwaggerConfiguration {
    // This line is really cumbersome, and will potentially cause issues if the default parameters don't match with Swagger's.
    private static final springfox.documentation.swagger.web.SecurityConfiguration DEFAULT_SWAGGER_SECURITY_CONFIG =
        new springfox.documentation.swagger.web.SecurityConfiguration(null, null, null, null, null, ApiKeyVehicle.HEADER, "api_key", ",");

    private ApiKey apiKey = null;

    @PostConstruct
    void init() {
        // some logic to assign apiKey according to some configuration
    }

    @Bean
    springfox.documentation.swagger.web.SecurityConfiguration security() {
        if (this.apiKey == null) {
            return springfox.documentation.swagger.web.SecurityConfiguration.DEFAULT;
        } else {
            return new springfox.documentation.swagger.web.SecurityConfiguration(null, null, null, null,
                    "Your access token", ApiKeyVehicle.HEADER, apiKey.getKeyname(), null);
        }
    }

Ideal Solution
Please make either of the following public:

springfox.documentation.swagger.web.SecurityConfiguration#DEFAULT
SecurityConfiguration's default constructor

Or please advise other ways or workarounds.
Thanks!

@cooniur Thanks for the suggestions. We don't want to expose DEFAULT because it is really used for when there is no configurations specified. And the default constructor is not exposed because if someone is overriding the configuration they should be deliberate about it. You could define that constant in your code or create a factory method with only the overridden values.

Hi @dilipkrish, I totally understand the reason why you want to keep the default constructor private.
Maybe at least could you make the DEFAULT public? Therefore, users wouldn't have to maintain their own default security config if they just need to use whatever the default one is out of the box. And moreover, users wouldn't need to worry about keeping it up to date with the constructor if the constructor would change in the future.
It seems there is no obvious harm by doing so, is it?

@cooniur, maybe you could use Conditional annotation instead of the field check. It depends on the logic in your init() method.
As I can see (I checked it for 2.6.1), you could just return null value instead of DEFAULT in your Bean annotated method.

Closing this issue for now. We can reopen if you feel strongly after the 2.7.0 release

Hi @ris58h , thanks for the advice of using the @Conditional. I actually took this approach, too.
I'm ok not to use the DEFAULT security configuration for now. So closing this ticket is fine with me, too.
Thanks everyone for the help! :)

Can you please add builder to the class?
I also suggest to use ```@Singular `` list fields. https://projectlombok.org/api/lombok/Singular.html
Thanks!

@bademux thats a great idea! However we would still like to validate the built configuration, so may be a hand written builder. Would you like to help?

@dilipkrish We can use annotations like @NotNull with lombok.
Or if more control is needed we can use factory method annotated with @Builder

authors: bademux cooniur dilipkrish ris58h

repo: springfox/springfox


text:
Suggestion: OperationNameGenerator should be able to consider Controllers/WARs as well

Namely because otherwise commonly used names like list come up as duplicates even though they are unique in their own Controller/WAR.

It will add a number at the end to distinguish the names right?

@dilipkrish Yes, exactly. Any reason not to have this as a plugin similar to ParameterNameReader?

authors: dilipkrish mvdobrinin


text:
当前编辑人对列表增加行、删除行，其余共同编辑人员的边框线消失，刷新页面后边框线会再次出现

我这边会尝试进行复现去排查问题，请耐心等待

已修复

repo: springreport/springreport


text:
ssi-contexts v0.1.3

Changes since ssi-contexts v0.1.2:

Add Aleo signature inline context (https://github.com/spruceid/ssi/pull/360)
Add Universal Wallet 2020 context (https://github.com/spruceid/ssi/pull/383)
Updated JsonWebSignature 2020 context (re: https://github.com/w3c-ccg/lds-jws2020/pull/92)
Update Credentials examples context (https://github.com/spruceid/ssi/pull/375)
Update traceability context (https://github.com/spruceid/ssi/pull/375, https://github.com/w3c-ccg/traceability-vocab/compare/e91ec2963074dd67a10234b5f842bca1ebbe2c65...a28b978aca3c9b1928acf1ff0bca0ca675185635 / https://github.com/w3c-ccg/traceability-vocab/compare/6890cb319b0f2a6557978f2193ae7ad478a9c536...b615de29e354cc1823c85411ac43f0c55e87ca55)
Update schema.org context (https://github.com/spruceid/ssi/pull/375)
Add shell script to fetch context files (https://github.com/spruceid/ssi/pull/376)

Published:

https://github.com/rust-lang/crates.io-index/commit/9662b28f60ffa46aae10f20ba9865513b79cae77
https://crates.io/crates/ssi-contexts/0.1.3
https://github.com/spruceid/ssi/releases/tag/ssi-contexts/v0.1.3

repo: spruceid/ssi


text:
didkit build error

while I was building credible, following the steps in README, I had some problem running
 make -C lib ../target/test/android.stamp

error:
// snip..
"-Wl,-Bdynamic" "-ldl" "-llog" "-lunwind" "-ldl" "-lm" "-lc" "-Wl,--eh-frame-hdr" "-Wl,-znoexecstack" "-L" "/home/uni01/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/i686-linux-android/lib" "-o" "/home/uni01/spruceid/didkit/target/i686-linux-android/release/deps/libdidkit.so" "-Wl,--gc-sections" "-shared" "-Wl,-zrelro,-znow" "-Wl,-O1" "-nodefaultlibs"
  = note: ld: error: unable to find library -lunwind

however I have libunwind installed on my system
libunwind.so
libunwind.so.8
libunwind.so.8.0.1
libunwind-x86_64.a
libunwind-x86_64.so
libunwind-x86_64.so.8
libunwind-x86_64.so.8.0.1

and about LD_LIBRARY_PATH:
/usr/lib/x86_64-linux-gnu

so what is going on and what i have missed?

Use forked repository https://github.com/alan-turing-institute/trustchain-mobile.git
and follow the instructions from install_trustchain_mobile.md file to setup the project.
This is updated latest repo.
Android and iOS running successfully without any issues.

authors: cgair shabaad

repo: spruceid/wallet


text:
🛑 Terminalserver is down

In 0677609, Terminalserver ($URL_TS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Terminalserver is back up in 16c1ae2.

repo: spschweiz/status

license: MIT


text:
s3_backend breaking change in minor version

Hi!
We are currently running 3.0.5 in our codebase and we have the following function in our tests:
def change_moto_s3_object_last_modified(bucket, key, value: Union[datetime, timedelta, int]):
     obj = moto.s3.s3_backend.buckets[bucket].keys[key]

We recently did a update of dependencies in the project which used 3.1.13.
Doing so broke the build:
AttributeError: module 'moto.s3' has no attribute 's3_backend.
The way I see this is that we are importing something that is not having any undeescores, therefore we aren't importing anything internal from the lib.
I suspect the culprit could be https://github.com/spulec/moto/commit/79a2a9d4232ffd7e4ff7279b9516d1722a32dc74.
Are we importing internals of moto into our the tests and we should not use the library like that? I'm not sure of the desired outcome of this issue should be. Feel free to close if you see fit.
Thank you for an amazing library.

Hi @Lilja, this is indeed considered part of the internal API. Only the decorators and non-beta API's under moto.moto_api are guaranteed to not have breaking changes.
The aim of Moto is only to mock the boto3 clients/resources, as having an API to mock everything would be too much work to create/maintain. Overriding the internal API's to customize (for example) the last modified date can be useful, and we do let people know it's an option, but always with the caveat that things may change.
I agree that it would've been more clear if everything was prefixed with _ - I'll think of a way to implement this properly.
In the meantime, you should be able to access the same functionality using s3_backends["global"]. The s3_backend was just a shortcut, and removed to simplify things.
Let me know if you have any other questions.

thank you @bblommers ❤

authors: Lilja bblommers

repo: spulec/moto

license_source: bigquery


text:
CloudWatch - Implement list_metrics pagination

Fixes #2010
Also fixes the issue that 'test_cloudwatch_boto3.py' never ran in tests.
This is because it was marked as executable (https://stackoverflow.com/a/1579787) - I've chowned it as appropriate and fixed a time-format issue

Coverage increased (+0.3%) to 94.017% when pulling 36a5d80443466317611732f546d500595fe7faa3 on bblommers:feature/#2010 into 502957f1f9560cb6dff75271e6812498f4ff7bba on spulec:master.

authors: bblommers coveralls


text:
Support SES get_identity_verification_attributes

Adds the get_identity_verification_attributes API
I used the scripts to generate the code, the took the XML from a debug output of awscli.
I'm not too sure if this is all correct, however the test passes.
I wasn't sure if there was a way of setting a status other than Success, although I feel that might mean we would need to explicitly record/track other status separately from the core boto3 API?

Codecov Report

Merging #5078 (9d22b5f) into master (8ad1212) will decrease coverage by 0.00%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##           master    #5078      +/-   ##
==========================================
- Coverage   95.84%   95.84%   -0.01%
==========================================
  Files         667      667
  Lines       66453    66468      +15
==========================================
+ Hits        63692    63706      +14
- Misses       2761     2762       +1

Flag
Coverage Δ

servertests
36.62% <20.00%> (-0.01%)
:arrow_down:

unittests
95.78% <93.33%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

moto/ses/models.py
96.64% <87.50%> (-0.21%)
:arrow_down:

moto/ses/responses.py
100.00% <100.00%> (ø)

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 8ad1212...9d22b5f. Read the comment docs.

I wasn't sure if there was a way of setting a status other than Success, although I feel that might mean we would need to explicitly record/track other status separately from the core boto3 API?

Well, this looks like a good start - if people need the ability to track other statuses we can always refine it.
Thanks for the PR @Amwam!

authors: Amwam bblommers codecov-commenter


text:
PR: Install bundled fonts for user on Windows and fix Linux CI

Fix CI for Linux
Add way to copy and register font files to prevent error when using bundled fonts on Windows with prevent untrusted fonts setting on.

Fixes #167

Note: Not totally sure to see how I could test/check this is working as expected

Note: Not totally sure to see how I could test/check this is working as expected

I think the simplest thing to do is to check thar our fonts are properly registered and installed in the system. I found this answer in StackOverflow that could be useful for that:
https://stackoverflow.com/a/68796070/438386

repo: spyder-ide/qtawesome

license: mit


text:
Stops after I turn on extended display.

Description
What steps will reproduce the problem?

Spyder gets an error whenever I turn on my extended display.
Traceback
  File "/Users/kyeongnam/opt/anaconda3/lib/python3.7/site-packages/spyder/app/mainwindow.py", line 1393, in handle_new_screen
    self.show_dpi_change_message)
TypeError: 'method' object is not connected

Versions

Spyder version: 4.1.1
Python version: 3.7.4
Qt version: 5.9.6
PyQt5 version: 5.9.2
Operating System: Darwin 19.3.0

Dependencies

# Mandatory:
applaunchservices >=0.1.7      :  0.2.1 (OK)
atomicwrites >=1.2.0           :  1.3.0 (OK)
chardet >=2.0.0                :  3.0.4 (OK)
cloudpickle >=0.5.0            :  1.3.0 (OK)
diff_match_patch >=20181111    :  20181111 (OK)
intervaltree                   :  None (OK)
IPython >=4.0                  :  7.13.0 (OK)
jedi =0.15.2                   :  0.15.2 (OK)
nbconvert >=4.0                :  5.6.1 (OK)
numpydoc >=0.6.0               :  0.9.2 (OK)
parso =0.5.2                   :  0.5.2 (OK)
pexpect >=4.4.0                :  4.8.0 (OK)
pickleshare >=0.4              :  0.7.5 (OK)
psutil >=5.3                   :  5.7.0 (OK)
pygments >=2.0                 :  2.6.1 (OK)
pylint >=0.25                  :  2.4.4 (OK)
pyls >=0.31.9;<0.32.0          :  0.31.9 (OK)
qdarkstyle >=2.8               :  2.8 (OK)
qtawesome >=0.5.7              :  0.7.0 (OK)
qtconsole >=4.6.0              :  4.7.1 (OK)
qtpy >=1.5.0                   :  1.9.0 (OK)
rtree >=0.8.3                  :  0.9.3 (OK)
sphinx >=0.6.6                 :  2.4.4 (OK)
spyder_kernels >=1.9.0;<1.10.0 :  1.9.0 (OK)
watchdog                       :  None (OK)
zmq >=17                       :  18.1.1 (OK)

# Optional:
cython >=0.21                  :  None (OK)
matplotlib >=2.0.0             :  None (OK)
numpy >=1.7                    :  None (OK)
pandas >=0.13.1                :  None (OK)
scipy >=0.17.0                 :  None (OK)
sympy >=0.7.3                  :  None (OK)

Duplicate of #11903

It'll be fixed in our next version (4.1.2), to be released next week.

repo: spyder-ide/spyder


text:
Missing variables in explorer (2 chars or longer/capital letters only)

Issue Report Checklist

[x] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Kind of a weird bug: If my variable names only consist of capital letters and have a length of 2 or more charachters they do not get displayed in the variable explorer. When calling the variable inside the console they are available and my scripts also run fine which means the compiler has the missing variables loaded. When the variable name only consists of a single capital letter then the variable explorer displays it without a problem.
What steps reproduce the problem?

Declare a new variable with only capital letters inside console or some script

What is the expected output? What do you see instead?
Should see the variable inside the explorer but it's not shown. Every other variable gets displayed.
Paste Traceback/Error Below (if applicable)
No Traceback
Versions

Spyder version: 5.2.1
Python version: 3.9.2
Qt version: 2.0.0
PyQt version: 5.12.3
Operating System name/version: Debian 11 - Bullseye

Dependencies

# Mandatory:
atomicwrites >=1.2.0          :  1.4.0 (OK)
chardet >=2.0.0               :  4.0.0 (OK)
cloudpickle >=0.5.0           :  2.0.0 (OK)
cookiecutter >=1.6.0          :  1.7.3 (OK)
diff_match_patch >=20181111   :  20200713 (OK)
intervaltree >=3.0.2          :  3.1.0 (OK)
IPython >=7.6.0               :  7.31.0 (OK)
jedi >=0.17.2;<0.19.0         :  0.18.1 (OK)
jellyfish >=0.7               :  0.9.0 (OK)
jsonschema >=3.2.0            :  4.4.0 (OK)
keyring >=17.0.0              :  23.5.0 (OK)
nbconvert >=4.0               :  6.4.0 (OK)
numpydoc >=0.6.0              :  1.1.0 (OK)
parso >=0.7.0;<0.9.0          :  0.8.3 (OK)
pexpect >=4.4.0               :  4.8.0 (OK)
pickleshare >=0.4             :  0.7.5 (OK)
psutil >=5.3                  :  5.9.0 (OK)
pygments >=2.0                :  2.11.2 (OK)
pylint >=2.5.0                :  2.12.2 (OK)
pyls_spyder >=0.4.0           :  0.4.0 (OK)
pylsp >=1.3.2;<1.4.0          :  1.3.3 (OK)
pylsp_black >=1.0.0           :  1.0.1 (OK)
qdarkstyle =3.0.2             :  3.0.2 (OK)
qstylizer >=0.1.10            :  0.2.1 (OK)
qtawesome >=1.0.2             :  1.1.1 (OK)
qtconsole >=5.2.1;<5.3.0      :  5.2.2 (OK)
qtpy >=1.5.0                  :  2.0.0 (OK)
rtree >=0.9.7                 :  0.9.7 (OK)
setuptools >=49.6.0           :  60.5.0 (OK)
sphinx >=0.6.6                :  4.4.0 (OK)
spyder_kernels >=2.2.0;<2.3.0 :  2.2.1 (OK)
textdistance >=4.2.0          :  4.2.2 (OK)
three_merge >=0.1.1           :  0.1.1 (OK)
watchdog >=0.10.3             :  2.1.6 (OK)
xdg >=0.26                    :  0.27 (OK)
zmq >=17                      :  22.3.0 (OK)

# Optional:
cython >=0.21                 :  None (NOK)
matplotlib >=2.0.0            :  None (NOK)
numpy >=1.7                   :  None (NOK)
pandas >=1.1.1                :  None (NOK)
scipy >=0.17.0                :  None (NOK)
sympy >=0.7.3                 :  None (NOK)

Hi @timptner thank you for the feedback! Could you check the Variable Explorer options to see if you have checked the Excluded all-uppercase variables or Excluded capitalized variables?:

Let us know if unchecking that helps!

Yes, sorry. Didn't know about that setting.

No problem, glad is working as expected for you now @timptner !

authors: dalthviz timptner


text:
Tried using this solution.  Further got error ignored explicit argument 'orge'

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Closing because this is empty and doesn't seem like an Spyder error. Instead, if it's an error that shows up when running your code, we recommend you to ask for help in
https://stackoverflow.com


text:
Feature Request: Sort tabs alphabetically

Description of your problem
I see that it's been a long way to get the tabs in Spyder to become moveable: https://github.com/spyder-ide/spyder/issues/564
However, it would be really useful if Spyder would either (a) provide a standard setting to sort them alphabetically or (b) at least save the position according to the users preferences.
When a project consists of 10 or more .py-files, at some point searching for the correct tabs becomes really unhandy...
Versions and main components

Spyder Version: 3.2.4
Python Version: 2.7.14 64bits
Qt Version: 5.6.2
PyQt Version: 5.6
Operating system: Windows 7 x64 Professional

That's why we added a file switcher to the Editor, which can be activated with Ctrl+P by default. Is that not enough?

My intention is just to contribute ideas to an easier and more efficient coding environment. When my currently activated tab is in the middle of, say, 10 open tabs and I'm looking for a module which I cannot see directly in the tab row, I have to search for it, since I don't know whether it is located to the left or to the right from my current tab position.
Unfortunately, pressing Ctrl+P is not really helping me, because also there one needs to scroll (since not all modules fit in) and they are not sorted as well...
My current workaround is to manually (!) sort the modules in the .spyproject/workspace.ini file and I was just wondering if this could be achieved somehow automatically directly in Spyder?

@fgotzens, it's not a bad idea, but we don't have time to implement it. You're welcome to send us a pull request to do it, though.

@fgotzens @luciansmith If you could take a look at https://github.com/spyder-ide/spyder/issues/7993 and give your feedback please, that would be much appreciated. Otherwise, this issue may get closed as resolved after PR #8015 is merged.

Thanks for the heads-up!  It does indeed look like a good solution to me.

This was fixed by PR #9109.

authors: ccordoba12 fgotzens jnsebgosselin luciansmith


text:
i don't know whats wrong with my code. some one help!!!

Problem Description
I don't know whats wrong with the code line...       (var(input), name (""))
can someone please fix it.
What is the expected output? What do you see instead?
the expected out come is a guessing game
this is the entire code...
5# -- coding: utf-8 --
"""
Created on Tue Feb 27 19:42:25 2018
@author: hcarr
"""
def main():
import random
import time
(var(input), name (""))
number = random.randint(1, 10)
tries = 1
print('hi what is your name?')
(var(input) name(""))
print ('hi ' name ' i am thinking of a number between 1 & 20')
guess = int(input("have a guess  "))
if guess < number:
print("Your guess is too low.")
if guess > number:
print("Your guess is too high.")
while guess != number:
tries += 1
guess = int(input("try again  "))
if guess < number:
print("Your guess is too low.")
if guess > number:
print("Your guess is too high.")
if guess == number:
print("well done ",name," the number was", number, "and it only took", tries, "tries")
time.sleep(5)
question = input("do you want to play again [Y/N] ")
if question == "y":
main()
else :
if  (datetime.now(<0400)
print("thanks for playing ",name," enjoy the rest of your day.")
else: (datetime.now(>1900)
print("thanks for playing",name," enjoy the rest of your night.")
exit
main()

Sorry, this is not a general Python programming forum but a place to report problems or bugs present in Spyder.
You can consider asking your question here:
https://stackoverflow.com/


text:
Fix quotes and traceback lines in runcell

Pull Request Checklist

[x] Read and followed this repo's Contributing Guidelines
[x] Based this PR on the latest version of the correct branch (master or 3.x)
[x] Followed PEP 8 code style
[x] Ensured this pull request hasn't eliminated unrelated blank lines/spaces,
modified the spyder/defaults directory, or added new icons/assets
[x] Wrote at least one-line docstrings following PEP 257for any new functions
[x] Added at least one unit test covering the changes, if at all possible
[x] Described the changes and the motivation for them below
[x] Noted what issue(s) this pull request resolves, creating one if needed
[x] Included a screenshot, if this PR makes any visible changes to the UI

Developer Certificate of Origin Affirmation
By submitting this Pull Request or typing my name below, I affirm the
Developer Certificate of Origin
with respect to both the content of the contribution itself and this post,
and understand I am releasing it under Spyder's MIT (Expat) license.

Description of Changes

Added a new line to the end of the code to separated the triple quotes of run cell and removed the rstrip() calls.
Also removed the code that added a new line after every key word block. This code seems to indicate that it is necessary to have the code run in the console but this dosen't seem to be needed in the ipython console. This code was changing the traceback line numbers.
Issue(s) Resolved

Fixes #
#8244 and #8242

Hello @bcolsen! Thanks for submitting the PR.

There are no PEP8 issues in the file spyder/app/tests/script.py !

There are no PEP8 issues in the file spyder/plugins/editor/widgets/base.py !

There are no PEP8 issues in the file spyder/plugins/editor/widgets/editor.py !

test_shortcutssummary_texts is the test failure. I tested on my set up and it seems flaky. I had to turn flaky up to 5 or it would still fail sometimes.

I tested on my set up and it seems flaky. I had to turn flaky up to 5 or it would still fail sometimes.

Please add to it the @flaky(max_runs=10) decorator, so it gets run 10 times by pytest.

test_shortcutssummary_texts is the test failure. I tested on my set up and it seems flaky. I had to turn flaky up to 5 or it would still fail sometimes.

Yeah, this test needs to be reworked a little. It should not be too difficult. I'll propose something.

Ok no, it is simpler than that and I think you should do it in this PR @bcolsen . Simply change Escape by Esc here and it should be OK :
https://github.com/spyder-ide/spyder/blob/master/spyder/widgets/tests/test_shortcutssummary.py#L63

Thanks @jnsebgosselin. I'll look at this tonight.

@ccordoba12 This is ready for review. @jnsebgosselin 's fix worked


text:
PR: Include QDarkStyle as an explicitly installed dep for the dev builds in the Contributing Guide

Pull Request Checklist

[x] Read and followed this repo's Contributing Guidelines
[x] Based this PR on the latest version of the correct branch (master or 3.x)
[ ] Followed PEP 8 code style
[ ] Ensured this pull request hasn't eliminated unrelated blank lines/spaces,
modified the spyder/defaults directory, or added new icons/assets
[ ] Wrote at least one-line docstrings following PEP 257for any new functions
[ ] Added at least one unit test covering the changes, if at all possible
[x] Described the changes and the motivation for them below
[ ] Noted what issue(s) this pull request resolves, creating one if needed
[ ] Included a screenshot, if this PR makes any visible changes to the UI

Description of Changes
I was following the CONTRIBUTING file to contribute to Spyder and after doing all the installation using the condas commands, I still ended up with dependencies problems. Specifically, I had trouble with the QDarkStyle package. After I run the command pip install -r requirements/requirements.txt it worked. So, I added a line saying that if the user had trouble with dependencies he/she should run this command, to fix it.
Issue(s) Resolved
No issues on this
Fixes #

Specifically, I had trouble with the QDarkStyle package.

You just needed to install QDarkStyle (pip install qdarkstyle), presumably.

pip install -r requirements/requirements.txt

The problem is this will install/update all the dependencies using pip, which is a recipe for problems especially if the user runs it in existing environment that may have older versions than the pip ones, especially   base.
The basic issue here is the dependencies for the development Spyder are different from those for the latest beta; right now that is the case since we have PyLS, QDarkStyle, etc. as deps for Spyder dev but not beta 1, which was released a long time ago. Instead of this change, we should instead just add the necessary dep(s) to the line that already tells the user to explicitly install PyLS. Of course, this won't be an issue anymore in a week or two when Beta 2 is released and we'll have to update this anyway, so with only a few reports of this I'm not sure its worth doing but if you want to go ahead.

I understand the problem of adding the requirements. Sorry! I just added it on the dependencies. Hope it is right :)

@leportella For a neater commit history, since the change is so trivial and the first commit is entirely redundant, can you make the change in one commit (squash them, rebase -i then edit or fixup, git reset HEAD~2 then commit again, etc)? And while your at it, if you don't mind starting the commit with a capital letter per convention. Thanks!

Closing in favor or PR #8299.

authors: CAM-Gerlach ccordoba12 leportella


text:
sqlite/json: support -> and ->> operators

What do you want to change?

Beginning with SQLite version 3.38.0 (2022-02-22), the -> and ->> operators are available for extracting subcomponents of JSON.[^1]

[^1]: 4.6. The -> and ->> operators
https://www.sqlite.org/json1.html#jptr
seems the latest sqlc v1.23 release does not support it:
https://play.sqlc.dev/p/25c82bb68f7a141175f4e51fd58047e1dc1466f61e02d059f5dcd6edf06a04d3
What database engines need to be changed?
SQLite
What programming language backends need to be changed?
Go

valid repro: https://play.sqlc.dev/p/73b7b8eb6ad22a3cceeb18db49bd99f377a1a707229c2a3486b58e5cc580311d

authors: orisano runsisi

repo: sqlc-dev/sqlc


text:
[Bug] - Import-DbaCsv does not close the session after finishing execution

Environmental information

#### PowerShell version:

Name                           Value
----                           -----
PSVersion                      7.0.0-rc.3
PSEdition                      Core
GitCommitId                    7.0.0-rc.3
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0ÔÇª}
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1
WSManStackVersion              3.0

#### dbatools Module version:

Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.78\dbatools.psd1
Version : 1.0.78

SQL Server:

Microsoft SQL Server 2019 (RTM-GDR) (KB4517790) - 15.0.2070.41 (X64) 	Oct 28 2019 19:56:59 	Copyright (C) 2019 Microsoft Corporation	Developer Edition (64-bit) on Linux (Ubuntu 16.04.6 LTS) <X64>

Aditional info: running inside a container
Report

After running Import-DbaCsv, if I run sp_who it still shows an active connection (although the status is sleeping).
Went to check again after ~10minutes and the connection is no longer there (but I'm assuming that it was SQL who killed it).
Host used

[X] powershell.exe
[ ] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[ ] Still buggy in powershell.exe

Errors Received

Steps to Reproduce

Import some csvs to the database, ie:
 Import-DbaCsv -Path $logFiles -Delimiter '|' -SqlInstance "Localhost" -SqlCredential $credentials -Database logHolder -Table LogsTable

Expected Behavior

The connection should be closed after finishing the execution of the command.
Actual Behavior

The connection was still open after the command execution, preventing me to, for example, delete the database without killing the session first.

This wouldn't be a bug, by my view, it is by design because the module depends on connection pooling be default.
You can use connection command to explicitly open and close the connection.

Connection pooling is indeed a default SMO behaviour. The workaround here would be to use a non-pooled connection, by either providing a connection string explicitly requesting such connection, or using -NonPooledConnection switch in Connect-DbaInstance. I'm closing this one unless there are any objections.

authors: DanielSSilva nvarscar wsmelton

repo: sqlcollaborative/dbatools


text:
Sqlwatch prerelease

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

The purpose of this PR is to enable Install-DbaSqlWatch to install GitHub's prereleases. Currently, I release beta versions of SQLWATCH as source code for people to manually install. This is not ideal and being able to deploy beta versions with dbatools would be beneficial for myself and the end users.
Approach

GitHub prereleases are identified in the API with a true/false prerelease tag which can be seen here:
https://api.github.com/repos/marcingminski/sqlwatch/releases
"prerelease": false,
"prerelease": true,

This functionality was achieved by adding the following logic to the existing function:
[switch]$PreRelease
...
Where-Object { $_.prerelease -eq $PreRelease }

If switch is set to true, it will filter API's output to show prereleases only. If no switch is given it will default to $false and thus return releases.
Commands to test

I have marked an old SQLWATCH 1.3.20 release as pre-release so there is no risk of current version of Install-DbaSqlWatch downloading this test pre-release before this PR is approved.
https://github.com/marcingminski/sqlwatch/releases
Deploy stable release as usual (no changes):
Install-DbaSqlWatch -SqlInstance 192.168.1.121 -SqlCredential $cred -Database SQLWATCH -verbose

Expected output, should return release 1.3.3:
VERBOSE: Performing the operation "Downloading latest release from GitHub" on target "VM-MBPWIN10-1".
VERBOSE: [22:19:13][Install-DbaSqlWatch] Checking GitHub for the latest release.
VERBOSE: GET https://api.github.com/repos/marcingminski/sqlwatch/releases with 0-byte payload
VERBOSE: received 12981-byte response of content type application/json; charset=utf-8
VERBOSE: [22:19:14][Install-DbaSqlWatch] Latest release is available at https://github.com/marcingminski/sqlwatch/releases/download/1.3.3/SQLWATCH_1.3.3.zip

Deploy pre-release (new functionality):
Install-DbaSqlWatch -SqlInstance 192.168.1.121 -SqlCredential $cred -Database SQLWATCH -verbose -PreRelease

Expected output, should return release 1.3.20:
VERBOSE: Performing the operation "Downloading latest pre-release from GitHub" on target "VM-MBPWIN10-1".
VERBOSE: [22:23:46][Install-DbaSqlWatch] Checking GitHub for the latest pre-release.
VERBOSE: GET https://api.github.com/repos/marcingminski/sqlwatch/releases with 0-byte payload
VERBOSE: received 12981-byte response of content type application/json; charset=utf-8
VERBOSE: [22:23:46][Install-DbaSqlWatch] Latest pre-release is available at https://github.com/marcingminski/sqlwatch/releases/download/SQLWATCH-v1.3.20/SQLWATCH_1.3.20.zip

Screenshots

Learning

niiiice!

authors: marcingminski potatoqualitee


text:
Count throws null reference with no rows when upgrading to 2.3.8

I'm having an issue upgrading from release 2.3.7 to 2.3.8 in using Count when no rows are returned, as it now seems to throw a null reference exception.
As a sample, consider this scenario: I often use the execution package of SqlKata (with PgSql) to:

dynamically build some query without the select clause.
clone it and add a .Count<int>(new[]{"fieldname"}) to count the results.
if results are present, complete the query with .Select, .OrderBy, and .Skip/.Limit, and run it to fetch rows.

This pattern is essentially for paging, where the same query must be executed once (without sorting and paging) to get the total, retrieving just the rows count; and then a second time to get the specified page. It worked fine, but something seems now broken in 2.3.8, because when I execute Count for a query which returns no rows, I get a null exception, e.g.:
// this throws when no rows are returned
int total = query.Clone().Count<int>(new[] { "document_id" });

System.NullReferenceException
  HResult=0x80004003
  Message=Object reference not set to an instance of an object.
  Source=SqlKata.Execution
  StackTrace:
   at SqlKata.Execution.QueryExtensions.CreateQueryFactory(XQuery xQuery)
   at SqlKata.Execution.QueryExtensions.CreateQueryFactory(Query query)
   at SqlKata.Execution.QueryExtensions.Count[T](Query query, String[] columns, IDbTransaction transaction, Nullable`1 timeout)

Thank you, here is my relevant code. First, this is a trivial utility method to get a query factory ready to use (that's a base class for different repositories):
protected QueryFactory GetQueryFactory()
{
    var qf = new QueryFactory(GetConnection(), SqlCompiler);
    qf.Connection.Open();
    return qf;
}

The following method is used to dynamically apply filters from a filters object (filter) to a query (query):
private static void ApplyTextUnitFilter(TextUnitFilter filter, Query query)
{
    // analyzed
    if (filter.Analyzed != null)
    {
        // analyzed, eventually with specific outcome(s)
        if (filter.Analyzed.Value)
        {
            query.WhereNotNull("au.response_type");

            // filter by outcome if requested
            List<AnalysisResponseType> whiteTypes = new();
            List<AnalysisResponseType> blackTypes = new();

            if (filter.Failed == true)
                whiteTypes.Add(AnalysisResponseType.Failure);
            else if (filter.Failed == false)
                blackTypes.Add(AnalysisResponseType.Failure);

            if (filter.Incomplete == true)
                whiteTypes.Add(AnalysisResponseType.Incomplete);
            else if (filter.Incomplete == false)
                blackTypes.Add(AnalysisResponseType.Incomplete);

            if (filter.Complete == true)
                whiteTypes.Add(AnalysisResponseType.Complete);
            else if (filter.Complete == false)
                blackTypes.Add(AnalysisResponseType.Complete);

            if (whiteTypes.Count > 0)
                query.WhereIn("au.response_type", whiteTypes);
            if (blackTypes.Count > 0)
                query.WhereNotIn("au.response_type", blackTypes);
        }
        // not analyzed, no outcome
        else query.WhereNull("au.response_type");
    }

    // edited
    if (filter.Edited != null)
    {
        if (filter.Edited == true)
            query.WhereNotNull("tu.last_modified");
        else
            query.WhereNull("tu.last_modified");
    }

    // min
    if (filter.MinUnitNumber > 0)
        query.Where("tu.number", ">=", filter.MinUnitNumber);

    // max
    if (filter.MaxUnitNumber > 0)
        query.Where("tu.number", "<=", filter.MaxUnitNumber);
}

This is the method failing with the new version: it gets a number of "text units" from a document using the specified filter. To this end, it first builds a query; then, it applies filters to it, using the above method; then it gets the total count, and later fetches the rows required for the requested page:
public DataPage<TextUnitInfo> GetTextUnits(int documentId,
    TextUnitFilter filter)
{
    if (filter == null) throw new ArgumentNullException(nameof(filter));

    using QueryFactory qf = GetQueryFactory();

    var query = qf.Query("text_unit AS tu")
                  .LeftJoin("text_unit_attribute AS tua",
                    j => j.On("tu.document_id", "tua.document_id")
                          .On("tu.number", "tua.unit_number"))
                  .LeftJoin("analyzed_unit AS au",
                    j => j.On("tu.document_id", "au.document_id")
                          .On("tu.number", "au.unit_number"))
                  .Where("tu.document_id", documentId)
                  .Where("tua.name", InputTextAttribute.NR_ATTRIBUTE);
    ApplyTextUnitFilter(filter, query);

    // get total
    int total = query.Clone().Count<int>(new[] { "tu.document_id" });
    if (total == 0)
    {
        return new DataPage<TextUnitInfo>(
            filter.PageNumber, filter.PageSize, 0,
            Array.Empty<TextUnitInfo>());
    }

    // complete query and get page items
    query.Select("tu.document_id", "tu.number", "tu.text",
        "tu.source_text", "tu.last_modified", "tua.value AS label",
        "au.response_type", "au.message")
        .OrderBy("tu.number")
        .Skip(filter.GetSkipCount()).Limit(filter.PageSize);

    List<TextUnitInfo> items = new(filter.PageSize);
    foreach (var d in query.Get())
    {
        items.Add(new TextUnitInfo
        {
            DocumentId = d.document_id,
            Number = d.number,
            Text = d.text,
            SourceText = d.source_text,
            LastModified = d.last_modified,
            Label = d.label,
            ResponseType = (AnalysisResponseType?)d.response_type,
            Message = d.message
        });
    }
    return new DataPage<TextUnitInfo>(filter.PageNumber, filter.PageSize,
        total, items);
}

I've released a new release, 2.3.9 can you upgrade and check if the issue persist?

Thank you, this seems to work, or at least all my unit tests now go green.

Ok great, I will close the issue then, thanks for reporting!

authors: Myrmex ahmad-moussawi

repo: sqlkata/querybuilder


text:
ViewState is not implicitly saved in onSaveInstanceState(), and there is no hook to Flow to trigger a state save

In onSaveInstanceState(), a Flow Dispatch is not triggered, and as a result, the views' state is not preserved into the State objects, but onResume() triggers a REPLACE direction that would destroy the previous view along with its view state.
I personally added a package-internal class that manually places the view state into the InternalLifecycleIntegration's KeyManager, but it is not a canonical solution.
public class ForceBundler {
    public static void saveToBundle(Activity activity, View... activeViews) {
        InternalLifecycleIntegration internalLifeCycleIntegration = InternalLifecycleIntegration.find(activity);
        if(internalLifeCycleIntegration != null) {
            KeyManager keyManager = internalLifeCycleIntegration.keyManager;
            for(View view : activeViews) {
                State state = keyManager.getState(Flow.getKey(view));
                state.save(view);
                if(view instanceof Bundleable) {
                    state.setBundle(((Bundleable) view).toBundle());
                }
            }
        }
    }
}

Which I call in onSaveInstanceState()
@Override
protected void onSaveInstanceState(Bundle outState) {
    ForceBundler.saveToBundle(this, root.getChildAt(0));
    super.onSaveInstanceState(outState);
}

Personally I would expect a call to the changeKey() method in onSaveInstanceState() as that triggers state persistence, and this to be managed automatically through the KeyChanger.

Here's where BasicDispatcher does it in the sample: https://github.com/square/flow/blob/master/flow-sample-basic/src/main/java/flow/sample/basic/BasicDispatcher.java#L46
This behavior should probably just be added to KeyDispatcher

@edenman that's wonderful, but it doesn't run on device rotation.

@Zhuinden I just tested out rotation, seems to call save/restore on the screen view just fine...did you remember to set ids on all of your layout files?

@edenman ...seriously, I seem to have forgotten to add an android:id="@+id/custom_view" to my custom viewgroups. With that added, view state is persisted properly. Now I kinda feel like an idiot.
However, that doesn't change the fact that in order to save your state to a bundle instead of a BaseSavedState using View.onSaveInstanceState(), you'll have to access the State directly from the KeyManager.
Well at least now the custom view can handle it instead of the container in onSaveInstanceState(), and I don't have to worry about hacking the view state in myself, so that's neat.

I'm a little confused. @Zhuinden can you describe in high-level terms what you expect to happen that isn't happening?

Are you trying to force a state save?  If so, why?

@loganj the KeyDispatcher provides the State objects for the given context from the KeyManager and allows persistence to the State's Bundle parameter, however, the custom view itself has no means of persisting to the State's Bundle on its own. I personally did that here.
So there isn't really a simple way of saving to the State's Bundle directly for the given Context in onSaveInstanceState() from the custom view.

I see. The Bundle wasn't really intended for use by the view, but by the dispatcher. The use case motivating it was dispatchers who wanted to save non-view state alongside the view state.
Note that State#save uses View#saveHierarchyState-- your ViewGroup's children should already be persisted so long as they have IDs, there's no need to try to save them explicitly as your example does.

@loganj even then, I'm not sure what kind of state you can truly persist that runs on traversals, but not on configuration change. Essentially, I still don't see how to store the current key state to the state without actively forcing it in onSaveInstanceState(), as traversals don't run for onPause(). The previous keys had the chance to store their state to the State, but not the current key.

It sounds like the root problem here is that Flow doesn't allow saving state on lifecycle events. That's true. We've been handling lifecycle events outside of Flow, but it probably makes sense for Flow to handle that case for you. Tracked as #181.

Continuing in #181

authors: Zhuinden edenman loganj

repo: square/flow


text:
Feature: offer a way to determine if a response was sent gzipped

I'd like to understand if a response was gzipped so that I can verify my servers are behaving correctly in tests, but BridgeInterceptor eats the relevant header information. Instead of just stripping headers, it'd be ideal if there were a way to recover 'suppressed headers' or simply determine original encoding of the underlying source, while still getting automatic decoding of the response.

Try this:
Response response = ...
boolean gzip = response.networkResponse().header("Content-Encoding", "").contains("gzip")

Perfect, thanks.

authors: markelliot swankjesse

repo: square/okhttp


text:
UnsupportedOperationException from JDK8 252 onwards

Testing a potential fix for https://github.com/square/okhttp/issues/5970
Will need to be backported to 3.14.x and 3.12.x

Raised by @DanielThomas we need to change activation logic here also.  Otherwise we won't work on 8-252

@swankjesse Can you also take a look please?

authors: monkey-mas yschimke


text:
Error when inserting lots of data.

I previously had an insert (on db creation) adding 505 rows of data. One column had dummy data (an empty string) and I changed it add ~100 characters per row. After that building the APK started failing with the following error:
...
    ACONST_NULL
    ICONST_1
    ACONST_NULL
   L219
    LINENUMBER 1808 L219
    INVOKESTATIC kotlin/text/StringsKt.trimMargin$default (Ljava/lang/String;Ljava/lang/String;ILjava/lang/Object;)Ljava/lang/String;
    ICONST_0
    ACONST_NULL
    BIPUSH 8
    ACONST_NULL
   L220
    LINENUMBER 1799 L220
    INVOKESTATIC com/squareup/sqldelight/db/SqlDriver$DefaultImpls.execute$default (Lcom/squareup/sqldelight/db/SqlDriver;Ljava/lang/Integer;Ljava/lang/String;ILkotlin/jvm/functions/Function1;ILjava/lang/Object;)V
   L221
    LINENUMBER 1809 L221
    RETURN
   L222
    LOCALVARIABLE this Lghplayer2/android/north101/gloomplayer2/Database$Schema; L0 L222 0
    LOCALVARIABLE driver Lcom/squareup/sqldelight/db/SqlDriver; L0 L222 1
    MAXSTACK = 7
    MAXLOCALS = 2

Cause: java.lang.IllegalArgumentException
Element is unknownThe root cause was thrown at: ByteVector.java:213
	at org.jetbrains.kotlin.codegen.TransformationMethodVisitor.visitEnd(TransformationMethodVisitor.java:88)
	at org.jetbrains.org.objectweb.asm.MethodVisitor.visitEnd(MethodVisitor.java:878)
	at org.jetbrains.kotlin.codegen.FunctionCodegen.endVisit(FunctionCodegen.java:992)
	... 62 more
Caused by: java.lang.IllegalArgumentException
	at org.jetbrains.org.objectweb.asm.ByteVector.putUTF8(ByteVector.java:213)
	at org.jetbrains.org.objectweb.asm.ClassWriter.newUTF8(ClassWriter.java:1141)
	at org.jetbrains.org.objectweb.asm.ClassWriter.newStringishItem(ClassWriter.java:1163)
	at org.jetbrains.org.objectweb.asm.ClassWriter.newConstItem(ClassWriter.java:1091)
	at org.jetbrains.org.objectweb.asm.MethodWriter.visitLdcInsn(MethodWriter.java:1165)
	at org.jetbrains.org.objectweb.asm.MethodVisitor.visitLdcInsn(MethodVisitor.java:595)
	at org.jetbrains.org.objectweb.asm.tree.LdcInsnNode.accept(LdcInsnNode.java:71)
	at org.jetbrains.org.objectweb.asm.tree.InsnList.accept(InsnList.java:162)
	at org.jetbrains.org.objectweb.asm.tree.MethodNode.accept(MethodNode.java:817)
	at org.jetbrains.kotlin.codegen.TransformationMethodVisitor.visitEnd(TransformationMethodVisitor.java:68)
	... 64 more

I was able to fix it by breaking it up into multiple inserts.

compilation failure or runtime failure

Compilation failure. Happens during :app:compileDebugKotlin
(:app:generateDebugSqlDelightInterface succeeds).

can you paste here the code that caused the compilation failure? .sq and .kt

ClassAbilityTest.sq.txt
Rename to .sq
I've duplicated the class and removed all references to other classes and types and it still fails during compilation with the same error. So dropping that into an existing project should hopefully also cause the error.
Breaking it up into multiple inserts succeeds (see attached):
ClassAbilityTest.sq.txt

I'm guessing we blow the limit for number of bytecodes in a single method.

Yea, that was my guess as well. The error could be confusing though, I was able to easily figure it cause it was the last thing I changed.

is there a limit on how big a string can be - cause the size of the generated create method wouldnt change but in the second .sq file there would be multiple different strings.

Ah yes. Based on the stacktrace coming out of ASM the problem is around creating a new string constant.

thats annoying. we might just be out of luck here unless theres a way to break that string requirement.

Looks like its length is represented by a two-byte, unsigned value so that'd give us 65k bytes. We could chunk it and use buildString if that length is exceeded.

:+1: sounds good

authors: AlecStrong JakeWharton North101

repo: square/sqldelight


text:
implement StoreFrom function to transmute stores

Fixes #118

Build is broken though

authors: Serabe erizocosmico

repo: src-d/go-kallax


text:
Partitions: Change Indexable interfaces to make use of partitions

Base IndexLookup interface will change to be able to get IndexValueIters per Partition.

type IndexLookup interface {
	Values(Partition) (IndexValueIter, error)
}

type IndexableTable interface {
    Table
    WithIndexLookup(IndexLookup) Table
    IndexKeyValues(Context, []string, Partition) IndexKeyValueIter
}

This is partly done in #329
Only missing thing is making index drivers work with partitions

Since now we have partitions, we should store the partitions in different pilosa frames. Right now, we still send an iterator with everything to the index to create it, so we don't know which rows belong to each partition.
I think we should change the Save method of the IndexDriver interface to accept a IndexableTable instead of an IndexKeyValueIter. That way, the driver can iterate partitions and get the IndexKeyValueIters for each partition and save them in different frames.
WDYT @ajnavarro?

Question - shall we add partitions to the driver index (and/or mapping) to quickly get everything  just from particular partition?

@erizocosmico  yep, totally needed.

@ajnavarro just updated with a second possibility, which one you think it's better?

@kuba-- yep

I think for now we can get by with not storing info about partitions in the index or mapping. Gitbase can get the partitions from the repository pool and the mem table has them hardcoded, so it's fine for now.

authors: ajnavarro erizocosmico kuba--

repo: src-d/go-mysql-server


text:
Enable to set DCO through the config file

Signed-off-by: Waren Long waren@sourced.tech
Fix https://github.com/src-d/modelforge/issues/103

Tests are passing, The default behavior is now with DCO according to modelforgecfg.py

repo: src-d/modelforge


text:
Document that cabal.project is used only for packages

I appreciate haskell-flake's support for cabal.project. I think it might be ignoring source-repository-package: https://cabal.readthedocs.io/en/stable/cabal-project.html#specifying-packages-from-remote-version-control-locations
haskell-multi-nix with example:
https://github.com/peterbecich/haskell-multi-nix/tree/source-repository-package-example
https://github.com/peterbecich/haskell-multi-nix/blob/source-repository-package-example/cabal.project
This should attempt to pull haskell/text bogus commit 123
cabal.project
packages:
    ./foo
    ./bar

source-repository-package
  type: git
  location: https://github.com/haskell/text.git
  tag: 123
  -- tag: 73620de89d43ee50de2d15b7bc0843bf6d6e9b9a

cabal build all fails as it should. However, nix build succeeds; I think it is getting text from NixPkgs as usual.
Cabal pulls the real commit 73620de correctly. nix build doesn't indicate if it is pulling this commit or not.

I know the text dependency could be specified in flake.nix: https://community.flake.parts/haskell-flake/dependency/#path
Thank you

haskell-flake only parses packages from cabal.project, nothing else.

Yea, this is not supported, but we should document it clearly. You want to specify those source dependencies are flake inputs: https://community.flake.parts/haskell-flake/dependency#path

Thanks, https://community.flake.parts/haskell-flake/dependency#path works great.
If the source-repository-package is kept in cabal.project, it appears to cause an error in haskell-flake at present. example:
https://github.com/peterbecich/halogen-chess/tree/source-repository-example
My goal is to have the project buildable by either plain Cabal, or Nix with haskell-flake.
Can haskell-flake's cabal.project parser be modified to ignore the source-repository-package?

haskell-flake itself ignores everything but packages -- so it does ignore source-repository-package. Maybe cabal (as invoked from Nix) is the only one that is forcefully using it?
In your first project, your only local package lives at project root, hence /cabal.project gets included in sdist of your only package. You can have haskell-flake ignore cabal.project entirely by manually specifying the path:
haskellProjects.default = {
  packages.halogen-chess.source = builtins.path { path = ./.; filter = path: _: baseNameOf path != "cabal.project";;
}

(Perhaps we should have a packages.*.filter option to specify these patterns using https://www.tweag.io/blog/2023-11-28-file-sets/ ?)

You're correct, thanks! Moving halogen-chess.cabal into a subdirectory fixes it: https://github.com/peterbecich/halogen-chess/tree/fix-cabal-project-in-nix/haskell by keeping cabal.project out of the sdist.
I can't get the second solution working; both of these cause infinite recursion.
packages = {
  halogen-chess.source = builtins.path {
    path = ./.;
    # filter = path: _: baseNameOf path != "cabal.project";
  };
};

packages = {
  halogen-chess.source = builtins.path {
    path = ./.;
    filter = path: _: baseNameOf path != "cabal.project";
  };
};

I'm going to try to reproduce this with the fundamental make-package-set functions
https://github.com/NixOS/nixpkgs/blob/e35d582350f5b92c769161532a663fb7fcce74f7/pkgs/development/haskell-modules/make-package-set.nix#L552-L597 , and then file an issue in nixpkgs

It needs to be self instead of ./.; but builtins.filter still doesn't work due to the way local packages are detected. This requires some debugging (and I have other things of priority to take care), so in the meanwhile perhaps you want to move cabal.project out of the way somehow? Eg. git mv cabal.project cabal.project.template (and ask users to move it back when using non-nix setup).
But yea, leveraging the --offline flag, while having it work correctly, would be one proper fix here. Another solution is to implement the filter option I mentioned above. This seems like a trivial solution that's not hacky.

Thanks for your time, @srid

authors: peterbecich shivaraj-bh srid

repo: srid/haskell-flake


text:
ZipOutputStream does not consider setOverrideExistingFilesInZip for existing files

When use net.lingala.zip4j.io.outputstream.ZipOutputStream to write file into a zip archive, setOverrideExistingFilesInZip (of net.lingala.zip4j.model.ZipParameters) seems not works and not consider file existence.
In the follow example you'll found a.txt twice in the zip.
	public static void main(String[] args) throws Exception {

		File fileA = new File("/path/to/fileA");
		File fileB = new File("/path/to/fileB");

		try (ZipOutputStream zipOutputStream = new ZipOutputStream(new FileOutputStream("/path/to/file.zip"))) {
			ZipParameters zipParameters = new ZipParameters();
			zipParameters.setOverrideExistingFilesInZip(true);

			zipParameters.setEntrySize(fileA.length());
			zipParameters.setFileNameInZip("a.txt");
			zipOutputStream.putNextEntry(zipParameters);

			try (InputStream inputStream = new FileInputStream(fileA)) {
				IOUtils.copy(inputStream, zipOutputStream);
			}
			zipOutputStream.closeEntry();

			zipParameters.setEntrySize(fileB.length());
			zipParameters.setFileNameInZip("a.txt");
			zipOutputStream.putNextEntry(zipParameters);

			try (InputStream inputStream = new FileInputStream(fileB)) {
				IOUtils.copy(inputStream, zipOutputStream);
			}
			zipOutputStream.closeEntry();
		}
}

setOverrideExistingFilesInZip works only with ZipFile api and not with ZipOutputStream. When using ZipOutputStream it will not be possible to modify a zip file (i.e., to remove a file from zip for example). I will add it to the documentation for setOverrideExistingFilesInZip.

Thanks for reply and for documentation update!


text:
Errors attempting to run schema

My attempt to generate srleecode schematic items always gives an errror
Executing task: ng generate @srleecode/domain:appGroupingFolder --name=xom --no-interactive --dry-run <
Cannot find module '@srleecode/domain/shared/utils'
Require stack:

/home/stclair/__/@workspace/pim/node_modules/.pnpm/@srleecode+domain@12.7.2-alpha-1/node_modules/@srleecode/domain/generators/grouping-folder/create-app/src/lib/shared/initialise-workspace.js
/home/stclair/__/@workspace/pim/node_modules/.pnpm/@srleecode+domain@12.7.2-alpha-1/node_modules/@srleecode/domain/generators/grouping-folder/create-app/src/generator.js
/home/stclair/__/@workspace/pim/node_modules/.pnpm/@nrwl+tao@12.7.2/node_modules/@nrwl/tao/src/shared/workspace.js
/home/stclair/__/@workspace/pim/node_modules/.pnpm/@nrwl+tao@12.7.2/node_modules/@nrwl/tao/src/commands/generate.js
/home/stclair/__/@workspace/pim/node_modules/.pnpm/@nrwl+tao@12.7.2/node_modules/@nrwl/tao/index.js
/home/stclair/__/@workspace/pim/node_modules/.pnpm/@nrwl+cli@12.7.2/node_modules/@nrwl/cli/lib/init-local.js
/home/stclair//@workspace/pim/node_modules/.pnpm/@nrwl+cli@12.7.2/node_modules/@nrwl/cli/bin/nx.js
The terminal process "/usr/bin/bash '-l', '-c', '/home/stclair//@workspace/pim/node_modules/.bin/ng generate @srleecode/domain:appGroupingFolder --name=xom --no-interactive --dry-run', 'ng generate @srleecode/domain:appGroupingFolder --name=xom --no-interactive --dry-run'" failed to launch (exit code: 1).

Terminal will be reused by tasks, press any key to close it.
Any reason as to why this is occurring?

Can you use 12.6.2?
I am going through a rewrite of the packages, so any of the published versions that end with alpha are not stable and are likely to have issues. I should be able to stabilize it soon.

Closing as the latest alpha should not have this issue and 12.6.2 can be used until the code in alpha is ready to be released

authors: srleecode st-clair-clarke

repo: srleecode/domain


text:
Publish 1.3.0

Because 1.2.1 and 1.2.2 are not recommended, can we have a 1.3.0 which includes fixes committed since then?

@rarkins 1.3.0 is now published. please give it a go.

authors: graingert rarkins

repo: sroze/ngInfiniteScroll


text:
🛑 Home Assistant is down

In 028ff30, Home Assistant (https://ha.srueg.ch/manifest.json) was down:

HTTP code: 502
Response time: 483 ms

Resolved: Home Assistant is back up in 5ed9866 after 16 minutes.

repo: srueg/upptime


text:
Entitas DSL (Domain Specific Language)

The current code generator
The code generator is very flexible and can be customized heavily to fit all your need.
It currently works like this:

ICodeGeneratorDataProvider converts a specific source format into ComponentInfo[]
ICodeGenerator converts ComponentInfo[] into CodeGenFile[]
ICodeGenFilePostProcessor writes files to disc.

Again:

DataProvider(source) returns ComponentInfo[] (source to intermediate format)
Generators(ComponentInfo[]) => CodeGenFile[] (intermediate format to source code)
PostProcessor (source code to disk)

Example:
You write a new PositionComponent (source)

TypeReflectionProvider(sourceCode) uses reflection to create ComponentInfo[] (intermediate format)
ComponentExtensionsGenerator (and others) create CodeGenFile[] based on ComponentInfo[] (result)

source => intermediate format => result
As we all know, the TypeReflectionProvider uses reflection based on the last compiled assembly. This means, if there are compile errors the assembly is not updated and generating again won't include any changes made to the source code until it can compile again.
That's why we're working on a new code generator DataProvider based on Roslyn to be able to create the intermediate format even when the solution does not compile.
Once we have it, we can simply swap the current ICodeGeneratorDataProvider which is the TypeReflectionProvider with the RoslynDataProvider and we're done. Everything works because the source analysis is decoupled from the actual code generation.
That means we can write any kind of ICodeGeneratorDataProvider that converts any kind of source into the intermediate format and swap it. E.g. a JsonDataProvider that can parse a json where all components are defined.
Suggestion
We could write a custom DSL which defines all components in a single file and we can create a new DSLDataProvider to convert this source into ComponentInfo[]. This file is not affected if there a compile errors or not. That means, we can regenerate anytime we want.
This would be an alternative to the RoslynDataProvider. Currently, Roslyn doesn't fully work on all platforms, that's why I'm looking to find alternatives.
Let me know what you think.

If you have doubts or arguments against it, let me know.

Looks good, but when Roslyn will work in those platforms? The DSL solution looks like a temp workaround until it works (and in those platforms where Roslyn works is better to use it because you don't need to write that DSL)
In any case, the concept is cool !
PS: Question, i'm currently on windows, how would be for me the swap to Roslyn ? I need to install something ? Or will be bundled with entitas releases?

Afaik it's up to the Microsoft/Xamarin/Mono guys to implement missing features for all platforms. I cannot really tell when / if.
It's planned that any new code generator will be bundled with Entitas, so nothing to do on your part.

I'm not sure if you are aware of this yet, but all of the uFrame ECS system was open sourced recently: link. uFrame was also using a code generator that did not rely on reflection. Maybe something can be lifted from there?

They use CodeDom. I did not look at CodeDom yet, but now would be a perfect time to do so :)

Nice, I wish I could give you any pointers but I never had time to dive into that part of uFrame..

That was already a good hint to finally fiddle with CodeDom :) Thanks

Will close because of #77

authors: JuDelCo carlos-avila sschmid

repo: sschmid/Entitas-CSharp


text:
Quiz App

I wanted to add a quiz application.

Sure !!
Assigned to u. Complete it in 10 Days and follow contributing.md  Guide

repo: ssitvit/Code-Canvas


text:
Fix code coverage action

Fixing up code coverage was done as a seperate PR because it turns out codecoverage was in the original repo, it has just been broken since March of 2020!

Moved to more modern calling of code coverage (use the pytest plug-in)
Use github action to push code coverage report
Minor config fix: making sure install works correctly under WSL

@gordonwatts Looks good now, so I'll go ahead and merge. The Rucio adapter has little coverage but we can add unit tests in follow-on PRs as needed.

And thanks for catching my error there!

authors: AndrewEckart gordonwatts


text:
url to add

https://arxiv.org/abs/2312.02464

when ~~~~~~~~~~~~~

when ~~~~~~~~~~~~~

NOW

wonderful~

authors: kuaiqushangzixiba robmarkcole sstary

repo: sstary/SSRS


text:
Added Key LEDs

The code to turn on and off backlit LEDs was added.  The LEDs should be connected to pin 13 and they turn on when a connection has been made.  This has not been tested yet but should work.

I had to refuse this PR. Two reasons, first:
a connection LED inside a closed box make no sense,
second:
the connection is therefore indicated by the LCD display

authors: Moving-Light sstaub

repo: sstaub/eOS


text:
[FEAT] Option to hide seperator character in read mode

I love this plugin, but I'm missing a visual component. Say we have this line of text in obsidian:
Spaced repetition
?
Spaced repetition is an evidence-based learning technique that is usually performed with flashcards.
I'd like to see this when I turn on read mode
Spaced repetition
Spaced repetition is an evidence-based learning technique that is usually performed with flashcards.
or
Spaced repetition is an evidence-based learning technique that is usually performed with flashcards.
Some people maybe don't care for it so I thought it could just be a setting that you can turn on and off or a format type thing you set yourself in the settings.

This is actually quite hard to do as we'd have to write a CodeMirror extension (if I understand correctly).

No problem! I just changed the character "?" to a visually more pleasing one:

Thanks for the response!


text:
Algorithm to compute the derivatives of the COM velocity

I put the algo in a new center-of-mass-derivative.hpp/xx file, with new unittest file, following the policy used for other *-derivatives.hpp/xx files.
I put the binding in expose-kinematic-derivatives because creating a new file at this point seemed overkilled, but I can modify that choice if need be.
The algorithm is computed in the local frame, which makes it compact and elegant. I validated it for the simple humanoid in the unittest, but also for talos arm and legs in python.

By the way, you can have access to these derivatives through the centroidal dynamics derivatives that already exist and bind.

About computing COM first: the current algo seems quite minimal using vcom. And it is reasonable to expect that the user would have computed the vcom quantity before computing its derivative.
About centroidal algo: Agreed, but I wanted COM only. BTW, the centroidal algo seems an order of magnitude more complex. I did not check timings yet.

Additional question: I have a unittest in Python. What should I do with it?

Pushed. Thanks for the review.

@nmansard, we have a place for Python unit tests in unittest/python.
These tests are working but far from complete, and tend to be very simple, their main purpose being checking that the binding mechanism itself is working properly.
If you want to add your unit test, I would put it there, it will certainly not do any harm. All unit tests in that folder are implemented by employing the unittest framework, so I would just make sure your own test is structured in the same way.
Also, all unit tests can be run together using the script unittest/python/tests.py. I am not sure this is super useful (there are many ways to tell Linux to run all files in a folder), but if you want this feature, you can just add the proper import in that file

@nmansard Seems good to me. Maybe, if you have time to add the Python test it would be nice.

Anyway, @nmansard thank you for your contribution.

I added the python tests. You can merge.
BTW: I really don't like the unittest in Python :(

Ok. Sounds good. I'm just waiting for Travis.

authors: gabrielebndn jcarpent nmansard

repo: stack-of-tasks/pinocchio


text:
Chat history is gone

Describe the bug
just clicked on the pricing and after i reviewed it and closed it, all the history with bolt chase was gone
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-r5nhql
Steps to reproduce
no specific steps
Expected behavior
keeping the chat
Screen Recording / Screenshot
No response
Platform
Browser name = Safari
Full version = 17.6
Major version = 17
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.6 Safari/605.1.15
performance.memory = undefined
Username = colruedu
Chat ID = fb7b1e0e052e

Additional context
No response

@colruedu workaround in #39, complete fix coming very soon!


text:
database: revisit project database indexes

The initial implementation coming from https://github.com/stacklok/mediator/pull/539 chose UUIDs as indexes. This might or might not have been the right choice. Let's get back to this and research what is the appropriate choice to use so we don't crash and burn in the future. @evankanderson left some insights on the PR.

this has been addressed.

repo: stacklok/minder


text:
Adds commit lint to the CI

closes #1310
This is my first time doing any CI work, so even though I've tested it, and it is annoyingly good at detecting mistakes on commit messages, I'd suggest taking a close look before merging it.
It'd also be good to add the same check on the local build; that way, contributors can check errors before submitting a PR 😄
To test it, just send a couple of PRs to my fork

Thanks for the PR!
do you know what config does the action use by default? For some reason I'm not able to configure commitlint locally - I wanted to pipe a couple of commit messages through the linter so see what would get flagged.

@jhrozek, thanks for taking a look. I've tested several cases, and it looks like the default follows Minder's guidelines and should allow setting up semantic release.
I am not 100% sure but I think that this is the default config.
To test it, please open as many PRs as you want on my fork, and you'll see the errors: There are some example PRs open.
If there is any rule that needs to change, we can create a config for it or base it on the other config examples in the commitlint repo.

Ignore my last comment, I don't know why I thought that Minders guidelines were asking for semantic release commit messages, when they don't 🤦
I'll update this PR with a config that follows Chris Beans recommendations tomorrow 🙏

@jhrozek great idea! I've just added the rule to ignore WIP messages.
Thanks for taking the time to review this one.

authors: andresgalante jhrozek


text:
Dual write rule instances to new and old tables

[In draft while I do some testing]
Relates to #3485
Any time a profile is created or updated, write the rule instances to both the new and old rule instance tables. Mark the entries in the old tables as migrated so that we skip over them when we run the migration process.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.544% (+0.1%) from 52.398%
when pulling 99f2cc9ad22fb3d81eed601cd10cc5c09c1f4b13 on rule-instance-table
into 19a62a27520c9a55117614d66e7b71fd5e5452ee on main.

authors: coveralls dmjb


text:
Add certifier for *.internal-registry.ci.rox.systems

Arguably, this is not the right location for this CronJob. But this is Hackathon.

Arguably, this is not the right location for this CronJob. But this is Hackathon.

@gavin-stackrox  Will you put it in the right location after the hackathon?

authors: gavin-stackrox tommartensen

repo: stackrox/infra


text:
ROX-12516: Use Cloudflare IP instead of Google

Description
We were using Google's IP that was stable and responding with 404.
Now it's no longer responding and cause test failures.
This PR swiches to well known Cloudflare DNS IP 1.1.1.1
Testing Performed
CI

Current dependencies on/for this PR:

master

PR #3067   👈

This comment was auto-generated by Graphite.

/retest

/retest

@lvalerom I think we should merge it and if it does not make the situation any better then merge your PR to skip this for investigation.

Postgres test is failing but not the test we fixed. Let's merge

repo: stackrox/stackrox


text:
ROX-16212: Remove panic from failed charts

Description
Removes the panic when creating generating Secured Cluster manifests with SHA. Instead this should surface the error to the caller.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

/retest

authors: fredrb msugakov


text:
Add stacker and signer how to guides

Placeholder ultimately

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@netrome Since the SDK is not finalized, what is the policy on merging this PR? Should we keep it in a branch, or merge and then change master if something changes?

I'd prefer the latter. Let's have the end state we're aiming for documented and update the docs on master as things change.

authors: CLAassistant jferrant netrome

repo: stacks-network/sbtc-docs


text:
Interaction.Behaviors Issue

I'm new to studio visual code so please forgive me for this probably newbie error. Heres what happened:
c:\Users[USERNAME]\Downloads\OpenUtau-master\OpenUtau\UI\MainWindow.xaml(11,6): Error: The tag 'Interaction.Behaviors' does not exist in XML namespace 'clr-namespace:System.Windows.Interactivity;assembly=System.Windows.Interactivity'. Line 11 Position 6.
c:\Users[USERNAME]\Downloads\OpenUtau-master\OpenUtau\UI\MidiWindow.xaml(11,6): Error: The tag 'Interaction.Behaviors' does not exist in XML namespace 'clr-namespace:System.Windows.Interactivity;assembly=System.Windows.Interactivity'. Line 11 Position 6.

Solved by figuring out the dependency error.

repo: stakira/OpenUtau


text:
🛑 STAKR.space site is down

In 371c85b, STAKR.space site (https://stakr.space) was down:

HTTP code: 0
Response time: 0 ms

Resolved: STAKR.space site is back up in a6ed588 after 10 minutes.

repo: stakrspace/upptime


text:
🛑 Authentication Server is down

In f73201f, Authentication Server (https://auth.its-telekom.eu/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Authentication Server is back up in 3c561e8.

repo: stamateas/upptime


text:
⚠ GitLab Server has degraded performance

In d9198d3, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6282 ms

Resolved: GitLab Server performance has improved in 7b1d933.


text:
🛑 ReportPortal Server is down

In b6088ef, ReportPortal Server (https://reporting.its-telekom.eu/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: ReportPortal Server is back up in 9b6e8e6.


text:
Removing unused header files

Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:
Removes two unused header files.
Intended Effect:
Clean up the math library.
Side Effects:
This may affect open pull requests, but should be a quick merge.
Documentation:
None.
Reviewer Suggestions:
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

@rtrangucci, want to take a look. Should take < 1 min.

yes, will take a look tonight

@syclik looks good to me

authors: rtrangucci syclik

repo: stan-dev/math


text:
laplace_sample service console output has too many newlines

Summary:
The console output of the laplace_sample method contains hundreds of blank lines.
Description:
This arises from two things:

messages sent to the logger include newlines which aren't needed as the callbacks writer add these, e.g.: https://github.com/stan-dev/stan/blob/e043da75516985fa8e5ed32c826e19dc9f9a0730/src/stan/services/optimize/laplace_sample.hpp#L64
the logger is called on empty stringstreams

To fix the former, remove newlines.
To fix the latter, check the stringstream before calling the logger.
 model.write_array(rng, unc_draw, draw_vec, include_tp, include_gq, &write_array_msgs);
    if (refresh > 0) {
      if (write_array_msgs.peek() != std::char_traits<char>::eof())
        logger.info(write_array_msgs);

Reproducible Steps:
Use CmdStan branch https://github.com/stan-dev/cmdstan/tree/feature/1123-laplace-approx to invoke the laplace_sample method,  model and input files are in the src/test/test-models/ directory.
>  src/test/test-models/multi_normal_model laplace mode=src/test/test-models/multi_normal_mode.csv

Current Output:
the beginning of the console ouput from call to laplace_sample - refresh is default 100, so lots of whitespace between iteration messages.
Calculating Hessian

Calculating inverse of Cholesky factor

Generating draws

iteration:
0

Expected Output:
Calculating Hessian
Calculating inverse of Cholesky factor
Generating draws
iteration: 0
iteration: 100
iteration: 200
iteration: 300
iteration: 400
iteration: 500
iteration: 600
iteration: 700
iteration: 800
iteration: 900

Additional Information:
Provide any additional information here.
Current Version:
v2.30.0

Thanks for figuring this out.  This is why I was asking if there was a newline appended to all of the callbacks.  I had thought the answer was "no", but it's apparently "yes"!

There is a longstanding issue to make the logger ignore blank lines: #2577. Partial fix was started in #3098

The issue arose because I assumed there wasn't a newline inserted at the end of each message.  If we are going to add newlines, then we probably do not want to ignore blank lines, as they correspond to newlines in the output.

In that case we can probably close those two

authors: WardBrian bob-carpenter mitzimorris

repo: stan-dev/stan


text:
remove dup def for _log funs, fixing #1768

Summary:
Remove duplicate definition of propto=false version of _log user-defined functions when there is a forward declaration.
Intended Effect:
Allow recursive density definitions.
How to Verify:
There is a new model test that passes now and didn't pass before.
Side Effects:
None.
Documentation:
Behavior now matches existing doc.
Reviewer Suggestions:
Anyone.

Thanks!

authors: bob-carpenter syclik


text:
Fixing the integration variable

In the definition of the Cumulative Distribution Functions the integration variable was incorrectly defined as $\theta$ instead of $y%.

I'll just fix this with the next manual update and thank you for the correction in the acknowledgements.  Thanks!

Oops, forgot the reference:
https://github.com/stan-dev/stan/issues/2336#issuecomment-331960060

authors: bob-carpenter mapio


text:
Issue/3151 laplace logging newlines

Submission Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Remove spurious whitespace from messages sent to the logger.   Added unit test to verify template parameter jacobian.
Intended Effect
Compact console output; messages are 1 per line.
How to Verify
Unit tests.
Side Effects
N/A
Documentation
N/A
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

While reviewing https://github.com/stan-dev/cmdstan/pull/1134 I noticed the newlines were still logging and realized this PR hadn't been merged yet. @bob-carpenter do you have a chance to finish reviewing the latest changes?

ready to merge when tests pass.

@WardBrian - why are ubuntu checks failing w/ complaint about g++?

Better to ask @serban-nicusor-toptal I think. Github did recently change the meaning of ubuntu-latest, but not recently enough that I think this would break it? https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/

I think it actually is related to the ubuntu-latest changes, not sure clang-6 is available on 22.04 any longer. We can ignore it for this PR I think, I'll ask Nic to take a look

authors: WardBrian mitzimorris


text:
Tenant Migrate-Refresh Force

When running tenants:migrate-fresh I'm finding that it's hanging on the migrating step and not going any further. Is it possible to force this as you would in normal laravel?

If you add --force => true here, does it help?
https://github.com/stancl/tenancy/blob/95ee295da1527e280dea94d50054de90c754f7c0/src/Commands/MigrateFresh.php#L47-L49
You can find this file in vendor/stancl/tenancy/src/Commands/.

Thanks @stancl, i'll give that a try.

@jeverington Did it help? If yes, I will add the flag to that call in the package.

@jeverington I will be releasing 2.3.0 soon. Did it help and should I add it to package source code?

@stancl my initial problem was fixed due to a problem with a migration that was being swallowed. I feel that adding force by default would fix more problems than it would cause.
Sorry for the slow reply, I wasn't paying attention to my notifications.

fix more problems than it would cause

Sounds right, also you never use migrate fresh in production, so --force probably makes sense.

I'll keep this open to see it in the milestone though.

Resolved in https://github.com/stancl/tenancy/commit/d0bd8f2ad88d154fe448f5ed93c18ab8749e272f

authors: jeverington stancl

repo: stancl/tenancy


text:
Remove already produced ebook, add Dunsany novel

Removed Verne, Journey to Center of the Earth because it has been produced and released. Added Dunsany's The King of Elfland's Daughter, just added to PG in January 2020, first published in 1924. It is under 70,000 words.

Thanks!

authors: acabal jupdike

repo: standardebooks/web


text:
Put the burden of compatibility on devices and not the books

This rewrites the messaging around our downloads to make it clearer what we offer. I think it was clear from the recent MobileRead thread that there’s some misunderstanding with what we offer, and hopefully by moving the compatibility statements from our books to the ereaders we could clear some of them up.

These changes are OK in general, except that I don't want to remove the compatible label, because we want to encourage almost everyone to download that file. As I stated in that thread, "compatible" must have some reasonable definition, and just because it doesn't meet the definition of some guys on a forum using abandonware from 2012, does not make it incompatible. We would not expect a compatible Word document prepared using MS Word 2020 to work on Word 1.0 for MS-DOS from 1987, would we?
Maybe the more accurate phrasing would be "compatible epub 3" file. (Because at the core of it, their beef is that we are not shipping epub 2 files.) But, that invites confusion from less technical people as to, "does my ereader support epub 3?" The answer is of course almost always yes, except for those few guys on Mobileread. Saying "all compatible devices" invites the same question: Is my device compatible? And the answer is the same as above, so that can also be confusing.
And, as I also stated in that thread, this issue has literally never come up in 6+ years of millions of ebook downloads. It's not a real problem.

I’m totally happy to not take this change if you think it’s solving a non-existent problem 🙂
One thing I would say is that we should probably scrub references to epub2. There’s the one in this change of course, but we’ve got a few comments throughout the toolchain as well.

If you want to tweak this commit to remove the "compatible" changes, then go for it,  I think the rest is good. You can also tweak any epub2 comments in the toolchain. Long ago we did release epub2 files but that hasn't been the case for a long time now.

Sorry, wasn’t 100% on what you meant. I’ve put “Compaptible” back before epub in the title, and added one more compatible into how-to-use-our-ebooks. Think this this OK? Happy to make more changes.

Sorry, wasn’t 100% on what you meant. I’ve put “Compaptible” back before epub in the title, and added one more compatible into how-to-use-our-ebooks. Think this this OK? Happy to make more changes.

Thanks!

Thanks!

authors: acabal robinwhittleton


text:
Duplicated alt texts

Replacing two copy-pasted alt texts and adding semantic apostrophes.

Thanks!

Thanks!

authors: AutomCoding acabal


text:
BUG - Bot allows same user to be added multiple times.

Summary
It is possible to successfully add User X twice to a standup configuration.. their answers are duplicated in the report.
Steps To Reproduce
Self explanatory.
Expected behavior
The bot should ignore requests to add the same user if already a member of the report.
Observed Behavior
User is listed multiple times in the /standupviewconfig
Members: userx, userx, usery, usery
Environment (if applicable):
Can be provided if necessary but isn't likely environment specific.

Hi @skyscooby! I'm able to reproduce the bug locally.
Would you be willing to help fix the issue?

authors: harshilsharma skyscooby

repo: standup-raven/standup-raven


text:
some questions about learnable parameters in Distill-VQ

Hi, shitao, much thanks for your nice paper Distill-VQ and the well-organized open-resourced codebase.
I have some questions about learnable parameters in Distll-VQ after reading your paper:

In section 3.2, you mentioned The above optimization problem involves two groups of learnable parameters: the IVFPQ related parameters **O** and **C**, and the query encoder. However, the hard assignment in Eq.2 and Eq.3 is not differentiable. How to update IVFPQ related parameters O and C during training?

Looking forward to your reply.

Thanks for your interest in our work!
Firstly,  we use the nearest center and codewords to calculate the student score in the optimization object, so the O and C will be updated based on the loss.
Second, we use the straight-through estimator in PQ to make the hard assignment differentiable.  You can refer to the code.

authors: RuixiangZhao staoxiao

repo: staoxiao/LibVQ


text:
Support any combination of embedding provider, model, and vector dimensions in the sam command for tables

From #1726
For tables, users can insert or update and cause multiple embedding provider calls to run.

will pickup in the new year

repo: stargate/data-api


text:
In support

What this PR does: add in operation for jsonapi
Which issue(s) this PR fixes:
Issue 291
PR 348
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

The current test fail is because of that querybuilder quarkus-common change has not been merged.

repo: stargate/jsonapi


text:
question about "No 'bbr_deployment' key specified in the endpoint json"

I tried to backup my cf deployment using BOSH BBR Deployment Plugin and met below error logs.
bbr plugin starting up...
bbr_deployment   No 'bbr_deployment' key specified in the endpoint json
It seems that bosh director endpoint must return "cf" using json response.
But I think the normal bosh director does not return it.
What should I do to fix this error?

The endpoint json it's referring to is the configuration for the backup target.  The bbr_deployment plugin needs to know which deployment you want BBR to backup, which you would give via the Web UI when you configured the job.

How can I set 'bbr_deployment' to bbr-director plugin?
There is no field about it in Web UI.

Seems like a bug in the BBR plugin.  @ramonskie care to take a look?

i will take a look it at today.

the deployment field was not there since the rewrite

@ramonskie I installed shield using shield-boshrelease v8.0.8.
And I do not use shield-addon-bbr-boshrelease. I don't know the existence of that before. Should I use it to backup bosh director and bosh deployment(cf)?
And I'm trying to backup both the bosh director and a bosh deployment(cf).

okay well the bbr_plugin needs https://github.com/shieldproject/shield-addon-bbr-boshrelease
alongside the agent you use from shield.
otherwise it can't find the bbr cli.

authors: eujungkim jhunt ramonskie

repo: starkandwayne/shield


text:
storage changes

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #829

PR #830

PR #831

PR #832

PR #833

PR #834

PR #837

PR #845

repo: starkware-libs/papyrus


text:
refactor(starknet_api): use tx_hash constructor macro

Move tx_hash! macro constructor to core.rs in SN API and use macro throughout codebase.

This change is

authors: noam-starkware reviewable-StarkWare

repo: starkware-libs/sequencer


text:
Shift mask points.

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#654  👈
#644 : 1 other dependent PR (#646 )
#642
#639
#638
#637
#636
#651
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite

repo: starkware-libs/stwo


text:
Pull Request Template

I think there should be a PR request template to help standardise.

i'd like to create one here !!

Sure, go ahead

Should i also create the .md file for this & add the template there ?

Yep in .github folder you have to create a pull_request_template.md file

yes! created

i've taken example from the EddieHub Support Repo (with some changes) if thats okay, otherwise I can change the whole text !!

authors: AllanRegush starlightknown verma-kunal

repo: starlightknown/Cyberon


text:
cannot connect to RDP, wrapper fully supported, error : The graphics display components in the remote session failed to start up

installed and replaced the conf file (wrapper fully supported)
https://github.com/stascorp/rdpwrap/issues/1883

disabled WDDM
https://github.com/stascorp/rdpwrap/issues/858#issuecomment-523192537

when connecting get the error : The graphics display components in the remote session failed to start up

Not an RDP Wrapper issue.
Solutions in https://serverfault.com/questions/1090540/windows-server-2022-rdp-connection-fails-with-error-0x3-extended-error-0x11

You can ask this in Telegram chat or in Discord by sebaxakerhtc

authors: jay-sync sashaqwert

repo: stascorp/rdpwrap


text:
Find and it works https://github.com/stascorp/rdpwrap/issues/2549

          Find and it works https://github.com/stascorp/rdpwrap/issues/2549

[10.0.22621.3447]
funciona
muchas gracias
por su tiempo...

Your issue is spam-like, because not contains new information.

https://github.com/stascorp/rdpwrap/issues/2549

authors: erasmocanodiaz sashaqwert