MAINT - Add package constraints to conda recipe

Describe the bug
Add at least this constraint to the conda package: pydantic < 2.0.
For repro, see: https://github.com/conda-incubator/conda-store/issues/584. I don't know if the lower bound needs to be specified.
There are other constraints in conda-store/conda-store-server/environment.yaml, but I don't have a repro for those. So I suggest to just fix this for pydantic for now.
Expected behavior
pydantic 1.* is installed when you install conda-store-server.
How to Reproduce the problem?
See above.
Output
No response
Versions and dependencies used.
No response
Anything else?
No response

Important: make sure the repro mentioned above no longer repros when constraints are added.

This can be addressed with a patch via conda-forge/conda-forge-repodata-patches-feedstock.

Already contained in the release issue, closing as addressed


Dockerizing django app

Dockerizing a django app with sqllite using as a database

Please dont upload PYC files.


Backend exited with code 1

Did  docker-compose build --no-cache backend, and afterwards docker-compose up and received following error:

And

Tried to find more information about this "exec user process caused..." problem and here says everyone it's about endings..

Yes, git changes the line endings automatically. But are you user you are using latest development? There this should not happen anymore.

What does your Dockerfile for generator2 looks like?

Your comment in #88 indicates that this is working now? @VlaD-T

broken (agian) with commit a9f60bf912aa869b4503e9887f9dae0c5638f3f3
fixed with commit 208d42920164428fb093a781d5f79c20016a09d8

Finally testet with Windows 10, all tests passed


[WIP] Add metrics

Awesome! I'm going to check this out now.

I know this is a WIP, but couple of things:

"Create a metric point" should be something like "Create metric". The metric itself is what the current form does, whereas the point is a measurement of that metric. So you'd add a point to the metric, make sense?
Can we show the metric description on the Metrics page?

Sure for both things! I think the "Create a metric point" lang string, was already there before this PR, thanks for spotting it :)

Needs rebasing.

How far off are we on this? :)

I just rebased, pushing. The only thing missing is to set the graph data.

So just the metric points themselves now?

@joecohens can you update the original issue with a task list for what's left?

Moved to https://github.com/cachethq/Cachet/pull/449.


Best practice to configure rails for multiple Redis Connections

Overview
We use redislabs to host our redis clusters.
For each dataset we have different redis connections, in our case 4.[ ABC, DEF, GHI, JKL] where ABC might be:
redis://:REDACTED@pub-redis-18963.us-west-mz.1.ec2.REDACETEDtiadata.com:18963

UseCase
I'd like my Audience model to use the connection settings for ABC.
I'd like a few separate classes to use connection DEF.
The main app uses base Redis.current for all its basic caching.
Ideally i could have something like:
Redis.abc
Redis.def
etc...
Question
How do you suggest a rails app set it self up to have multiple connections to different redis connections?

In this case setting up a constant hash is probably the simplest way to do it:
REDIS_CONN = {
  abc: "whatever",
  def: "whatever"
}

Then
class Audience
  self.redis = REDIS_CONN[:abc]
end

class Other
  self.redis = REDIS_CONN[:def]
end

In real-world use, I find the YAML-based Settingslogic to be very helpful.  You can put these Redis conns in a YAML:
# redis_conn.yml
development:
   abc:
   def:
staging:
   abc:
   def:
production:
   abc:
   def:

Then:
class RedisConn < Settingslogic
  source "#{Rails.root}/config/redis_conn.yml"
  namespace Rails.env
end

And finally:
class Audience
  self.redis = RedisConn.abc
end

class Other
  self.redis = RedisConn.def
end

Cheers,
Nate

@nateware thanks a bunch.
Question 1
Just to confirm, in your example here:
class Audience
  self.redis = RedisConn.abc
end

"self.redis" is just pointing to the config, essentially a host string.
With the addition of the 'include Redis::Objects" in a class, does RedisObjects look for that attribute "redis"  and then auto create a connection pool?
Question 2
Assuming connection address of

abc: redis.myapp.com:6325/0
def: redis.myapp.com:6325/2

How would you initialize those two connections so that they can be used in a standalone manner through an app? Assuming the connections use a connection pool.
I'm looking for the best practices. Right now I have something that feels kind of janky in my /initializers/redis.rb file.


CORS preflight OPTIONS support for /sql (http fetch) endpoint

Problem
HTTP fetch can't be used from browsers because proxy doesn't support CORS 'preflight' OPTIONS requests.
Summary of changes
Added a simple OPTIONS endpoint for /sql.
Checklist before requesting a review

[X] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Rust codestyle check is unhappy:
r
         })
     } else if request.uri().path() == "/sql" && request.method() == Method::OPTIONS {
-      Response::builder()
-        .header("Allow", "OPTIONS, POST")
-        .header("Access-Control-Allow-Origin", "*")
-        .header("Access-Control-Allow-Headers", "Neon-Connection-String, Neon-Raw-Text-Output, Neon-Array-Mode, Neon-Pool-Opt-In")
-        .header("Access-Control-Max-Age", "86400" /* 24 hours */)
-        .status(StatusCode::OK)  // [20](https://github.com/neondatabase/neon/actions/runs/5536089619/jobs/10103284592?pr=4706#step:8:21)4 is also valid, but see: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS#status_code
-        .body(Body::empty())
-        .map_err(|e| ApiError::BadRequest(e.into()))
-
+        Response::builder()
+            .header("Allow", "OPTIONS, POST")
+            .header("Access-Control-Allow-Origin", "*")
+            .header(
+                "Access-Control-Allow-Headers",
+                "Neon-Connection-String, Neon-Raw-Text-Output, Neon-Array-Mode, Neon-Pool-Opt-In",
+            )
+            .header("Access-Control-Max-Age", "86400" /* [24](https://github.com/neondatabase/neon/actions/runs/5536089619/jobs/10103284592?pr=4706#step:8:25) hours */)
+            .status(StatusCode::OK) // 204 is also valid, but see: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS#status_code
+            .body(Body::empty())
+            .map_err(|e| ApiError::BadRequest(e.into()))
     } else {
         json_response(StatusCode::BAD_REQUEST, "query is not supported")
     }

WebSockets don't need CORS (which my experience with the driver confirms): https://stackoverflow.com/questions/23674199/why-is-there-no-same-origin-policy-for-websockets-why-can-i-connect-to-ws-loc

Ah, looks like I need to change my tab settings! I'll update the PR ...

Ah, looks like I need to change my tab settings! I'll update the PR ...

A bit more than just tabs, you can run cargo fmt --all to get this fixed.
@@ -225,9 +225,12 @@ async fn ws_handler(
         Response::builder()
             .header("Allow", "OPTIONS, POST")
             .header("Access-Control-Allow-Origin", "*")
-            .header("Access-Control-Allow-Headers", "Neon-Connection-String, Neon-Raw-Text-Output, Neon-Array-Mode, Neon-Pool-Opt-In")
+            .header(
+                "Access-Control-Allow-Headers",
+                "Neon-Connection-String, Neon-Raw-Text-Output, Neon-Array-Mode, Neon-Pool-Opt-In",
+            )
             .header("Access-Control-Max-Age", "86400" /* 24 hours */)
-            .status(StatusCode::OK)  // 204 is also valid, but see: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS#status_code
+            .status(StatusCode::OK) // 204 is also valid, but see: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/OPTIONS#status_code
             .body(Body::empty())
             .map_err(|e| ApiError::BadRequest(e.into()))
     } else {

Ah — thanks @bayandin.

Hmm. cargo fmt --all gives me:
error: no such command: `fmt`

And I don't seem to have either rustfmt or rustup installed. Could my problem be that I installed Rust with homebrew instead of rustup? Have I perhaps missed something else obvious? (I have approximately zero experience with Rust).

And I don't seem to have either rustfmt or rustup installed. Could my problem be that I installed Rust with homebrew instead of rustup? Have I perhaps missed something else obvious? (I have approximately zero experience with Rust).

Yeah, rustup is needed to manage the entire suite of rust tooling and versions, including cargo, rustc, rustfmt etc

Thanks guys. Having removed the homebrew installation and installed with rustup, this all just worked.


Add Killing Me Slow by Player1, Sarah de Warren

Song Name
Killing Me Slow
Artist Name
Player1, Sarah de Warren
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=FBlTMrwGvN0
Direct File Link
No response
Song ID
595361
Start Offset [ms]
0
Extra
Created from Auto Nong.

accept
I mean it is good, but it syncs a bit, but not too much but the sync is still good

.reject
has nothing to do with the song it replaces. and the original song is used a lot


get_models seems broken in 1.8

Steps to reproduce

Install dbt-core 1.8.0b2 with a compatible adapter
Create a new analysis and paste {{ log('repro: ' ~ codegen.get_models(directory='somedir')) }}
Run dbt run -s your_analysis

Expected results
A log with your models
Actual results
Compilation error 'dict object' has no attribute 'nodes'
System information
packages:
  - package: dbt-labs/codegen
    version: 0.12.1

Which database are you using dbt with?

[x] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
Core:
  - installed: 1.8.0-b2
  - latest:    1.7.11   - Ahead of latest version!

Plugins:
  - postgres:    1.8.0b1 - Ahead of latest version!
  - timescaledb: 1.8.0b2 - Up to date!

The operating system you're using: macOS
The output of python --version: Python 3.11.8

hey @sdebruyn were you wrapping that in an if execute tag? wondering if it's just this -- there aren't nodes in the graph in the parsing stage that would typically apply to an analysis file.

Could the example in the readme (https://github.com/dbt-labs/dbt-codegen?tab=readme-ov-file#generate_model_yaml-source) that uses codegen.get_models be updated to include 'if execute'?  As the documentation is written now, it results in the error described in this issue.


Fix ATTEMPT_EXPOSE_NON_PUBLIC_SYMBOL validation for Objects, Unions and other types.

Description:
See Referred pull.
https://github.com/ballerina-platform/ballerina-lang/pull/18341/files#diff-4025beda7fd769ab390b6d789e6cec16R1241
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Spec allows exposing non-public types. Hence we should reconsider this adding this as a warning.

Related issues
https://github.com/ballerina-platform/ballerina-lang/issues/30324
https://github.com/ballerina-platform/ballerina-lang/issues/23049

Should we prioritize this issue given that users would have to make constructs public to get rid of the error right now?
In addition to the compiler changes, we will also have to consider aspects such as doc generation.


chore: Disable Ruby head in CI temporarily

The failure in main is preventing a release of v17 of the gem, so until we reach consensus about what to do with head, I think it's reasonable to exclude head for CI.
ref: https://github.com/bytecodealliance/wasmtime-rb/issues/286

The release workflow is already ignoring head, so probably something else is going on.

I can confirm that gems are building successfully, but actions/download-artifact apparently can't find the generated files.
Error: Unable to download artifact(s): Artifact not found for name: cross-gem

I'll close this for now, we can re-open it as part of https://github.com/bytecodealliance/wasmtime-rb/issues/286.


Feature external strimzi listener

Sorry, it was a miss click. I'll create an another PR, after I'm really done.

No problem. thanks for the suggestion - I can't review today but will aim to tomorrow.

Please ignore this for now. I’ll let you know.
Taylan
Atruvia AG | www.atruvia.de
AG Frankfurt a. M. HRB 102381 | Sitz der Gesellschaft: Frankfurt a. M. | USt-IdNr. DE 143582320
Vorstand: Martin Beyer (Vorstandssprecher), Ulrich Coenen (Vorstandssprecher), Daniela Bücker, Ralf Teufel
Vorsitzender des Aufsichtsrats: Jürgen Brinkmann
Von: Nigel Jones @.>
Datum: Mittwoch, 25. Januar 2023 um 15:37
An: odpi/egeria-charts @.>
Cc: Tunc Taylan Turunc @.>, State change @.>
Betreff: Re: [odpi/egeria-charts] Feature external strimzi listener (PR #229)
Sorry, it was a miss click. I'll create an another PR, after I'm really done.
No problem. thanks for the suggestion - I can't review today but will aim to tomorrow.
—
Reply to this email directly, view it on GitHubhttps://github.com/odpi/egeria-charts/pull/229#issuecomment-1403720661, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AY4KPZVZUU77XQCRMKDK323WUE3CJANCNFSM6AAAAAAUGLJWPQ.
You are receiving this because you modified the open/close state.Message ID: @.***>


Nonfunctional Requirements

Functional
The program should:

Accept as input a YouTube URL.
Validate the input to ensure validity.
Extract the video ID from the given URL.
Fetch the transcript for the given URL.
Format the fetched transcript into a readable format.
Additionally accept as input a filename.
Save the formatted transcript to the given filename.

Nonfunctional

Performance

Reasonable time to fetch and save transcripts.

Reliability

Gracefully handle errors.

Security

Secure handling of user input.
Not violating terms of service.

Portability

Platform-independent.

Maintainability

Code should be well-structured, commented to allow for updates/fixes as required.

Scalability

The program should be able to handle larger requests (longer videos) without significant performance degredation.

https://github.com/bit-bangin/GetYouTubeTranscript/blob/5fb5175ebdfb30e14dda696ff5f330a67b21da53/Planning/ProjectPlanning(GYTT).ipynb


path option can't be used in multi-level way

# path/to/gem1/Gemfile
gem 'gem2', path: 'path/to/gem2'

# path/to/gem2/Gemfile
gem 'gem3', path: 'path/to/gem3'

# path/to/gem3/Gemfile
gem `fuubar`

cd /path/to/gem3/ && bundle install
> Bundle complete!

cd /path/to/gem2/ && bundle install
> Bundle complete!

cd /path/to/gem1/ && bundle install
> Could not find gem 'gem3', which is required by gem 'gem2', in any of the sources.

I understand this might be not preferable feature at all. Could you please give a piece of advice how this can be handled (without adding gem3 dependency directly to gem1?

Can you please provide directions for reproducing the issue? It's hard to tell what's going on without seeing the gemspecs in question. Thanks!

@segiddins , here is repository with these three gems and steps to reproduce: https://github.com/vbyno/gems_path_example

Thanks!

From the example, it looks like you'll need to add ../gem3 as a path source? Otherwise, bundler has no idea that there's a gem in that directory

@segiddins , do you suggest adding ../gem3 path to gem1 Gemfile? Yes, it will fix it. But I would like not to add this dependency explicitly to gem1 if it already exists in gem2.

it's not adding a dependency to gem1, it's just telling the Gemfile where to find  gem3 which it seems you want to pull in via a path source

Hm... But isn't it strange gem1 doesn't know where to find gem3? I mean, it knows where to find gem2. And gem2 knows where to find gem3. Why can't gem1 get this knowledge from gem2?

Nope, gem2 knows no such thing. When you put a Gemfile in a directory, you are not telling the gem anything at all, you are only telling Bunder that if it runs in that directory it should follow the directions in the Gemfile. If another gem uses gem2, it will not have any ideas about where to get gem3, because gems cannot contain instructions about where to get gems from.

@indirect , thanks for a clarification. I assume Issue can be closed. See no way this can be handled without pointing at gem3 in gem1's Gemfile or excluding gem3 dependency from gem2. Probably this is impossible without executing gem2's Gemfile which would be silly.


[Accessibility]A11y_AzureDataStudio_AllJobs_Keyboard:"All jobs" arrow is not accesible through keyboard and the focus is not logical .

Bug Description:
Check out Keros! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
Environment Details:
Application name: Azure Data Studio (Agent, Create External Data, Notebooks)
App Version: 1.8.0-insider (user setup)
VS Code 1.33.1
Electron: 3.1.8
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.18362
Additional Details:
Tool Name:
MAS Reference:MAS2.4.3,2.1.1
Repro Steps:
1.Install the app Azure Data Studio.
2.Naviagte to the SQL agent tab->Jobs->Click on any the jobs in the table.
3.observed the key board focus on arrow near "all jobs" heading.
Actual:
Key board focus is not logical as it directly jump on run button as soon as we enter in all jobs page and also that arrow is not accessible through keyboard.

Expected:
"all jobs" arrow should be accessible through keyboard and the first focus should be on that arrow as soon as we enter in that page.
User Impact:
User with motor disabilities will find difficult to navigate the page.
Recommendation:
Please refer the following snippet link for fixes:
https://microsoft.sharepoint.com/teams/msenable/mas/Pages/browse-fixes.aspx
MAS Reference:<IP_ADDRESS>.1.1
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={0de7fbe1-ad7e-48e5-bcbb-8d986691e2b9}
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
Attachment for Reference:
5744_AccessibilityA11y_AzureDataStudio_AllJobs_Keyboard.pptx
Does this issue occur when all extensions are disabled?: Yes

Verified in the below environment:
Application name: Azure Data Studio
Version: 1.9.0-insider (user setup)
Electron: 3.1.8
Node.js: 10.2.0
V8:6.6.346.32
OS : OS: Windows_NT x64 10.0.18362
Tool Name: Keyboard
Bug has not been fixed it is still repro hence reopening the bug.
Attachment for reference:-
5744_Reopened_A11y_AzureDataStudio_Alljobs_Screenreader.pptx


Slate for powercord no longer works

Describe the bug
Slate does not apply to powercord (I activate / toggle the theme and it makes no changes at all to discord even after restart. I have tested with other themes and they do work (The Fluent Theme for powercord also does not work))
To Reproduce
Install Slate to Powercord as per the standard procedure for adding themes (i.e. by git cloning to the powercord themes dir or adding through the powercord context menu)
Screenshots

Information (please complete the following information)
Discord channel: Canary
OS: Windows
Mod: Powercord
Discord language: English

It is also worth noting I had someone else from the Powercord discord community try installing the theme and they also noted that it does not apply / work for them, even on a restart

Thanks Gibbu


Outstanding CEQ changes to sidepanel

See Indicators for UI and Export - New TAB for highlighted changes

these are in yellow in the Indicators for UI and Export sheet


下载不下来

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

These issues are getting ridiculous and completely destroys meaningful notifications on this repository. You have my sympathies @FrenchBen for having to put up with all the garbage. No offence to you OP.
The title translates to "Download is down" according to google translate btw.

@emilkje thanks for the kind words - I've been thinking about implementing an auto-close feature, because of this.
@datafu Please make sure that you're on a stable connection and use a solid DNS such as Google DNS.


Idea: Appear as a custom Avatar Instead of a pill

It would be cool if I could appear as an avatar, either custom or a more better default to represent myself in the test environment, instead of showing as a large white pill.

Looks like the future is already here with the release of Clientsim. Please close my ticket.


Single file selector

Hi,
Thank you for the really nice plugin. I am sure if there is something missing in the documentation but I cannot find the option to allow user to select one file only. If I set maxNumberFiles to 1, it does allow selecting only one file, but the problem is the file picker is still in multi file selection mode. So the user cannot just tap a file to select it, they have to 'select' the file kind of like how they do when selecting multiple files.
Is this a limitation of Android/iOS or do you think something is missing here?

As far I remember iOS has this problem. It's related with native library. Sorry, I won't be able to help here.


tried to generate test and received error "Cannot read property 'match' of undefined"

(node:12436) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'match' of undefined
at forEach.param (C:\Users\Jason\AppData\Roaming\npm\node_modules\ngentest\src\get-directive-data.js:86:26)
at Array.forEach ()
at getDirectiveData (C:\Users\Jason\AppData\Roaming\npm\node_modules\ngentest\src\get-directive-data.js:53:43)
at parseTypescript.then.tsParsed (C:\Users\Jason\AppData\Roaming\npm\node_modules\ngentest\index.js:44:17)
at process._tickCallback (internal/process/next_tick.js:68:7)
at Function.Module.runMain (internal/modules/cjs/loader.js:834:11)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)
(node:12436) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by
rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:12436) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

can you release the fix via npm, have the same issue ?

In my case the error was caused by:
    constructor(
        @Inject(APP_CONFIG) public config: IAppConfig,
        protected ref: ChangeDetectorRef,
        protected notificationService: NotificationService,
        protected selectableService: SelectableService,
        **protected searchObject: BehaviorSubject<SearchModel>,**
        protected eventService: EventService,
        **protected el?: ElementRef,
        protected collectionLayout?: BehaviorSubject<LayoutTypeEnum>**
    ) {

special characters inside the constructor

 and the ? before the variable


v1.1.1

Documentation and test updates, with a minor change in R package repository used for robust distances.

Codecov Report
Base: 85.67% // Head: 88.16% // Increases project coverage by +2.48% :tada:

Coverage data is based on head (a866b59) compared to base (9debe24).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   85.67%   88.16%   +2.48%
==========================================
  Files           9        8       -1
  Lines         405      414       +9
==========================================
+ Hits          347      365      +18
+ Misses         58       49       -9

Impacted Files
Coverage Δ

src/rmpv.jl
97.31% <ø> (ø)

src/internal.jl
100.00% <0.00%> (ø)

src/BioProfiling.jl

src/transform.jl
97.36% <0.00%> (+0.14%)
:arrow_up:

src/struct.jl
91.46% <0.00%> (+0.32%)
:arrow_up:

src/diagnostic.jl
55.42% <0.00%> (+13.64%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Restyle the toast into an alert

Previously, success and error messages were sliding in from the top right and auto dismissing. The weren't quite prominent enough and sometimes you didn't finish reading the error before it disappeared.
Now I've restyled all of these to be alerts which come down from the top center of the screen and persist with the option of X'ing them out to hide.
Success example:

Error example (and smaller viewport):

I don't know what's up with the system tests. I tried to fix them in the other PR by changing the chrome-driver version to the latest, but that didn't seem to fix it. Looks like the failures in this PR are different than from there though so I'm not sure what's up. I tried to open a PR with just a change to the readme to see if it was my forgot-password branch, but it still failed in the same way. https://github.com/AllYourBot/hostedgpt/pull/480 passed system tests though, but I'm not sure why the changes there would have caused a pass. I think maybe the tests are just acting inconsistently, but I've rerun the tests multiple times on my branches and it hasn't passed yet haha. I'll keep digging, but wanted to update you about what's going on.

good fix though. I like that more than the readme section. Readmes can get way too long and hard to navigate so definintely prefer this kind of thing when possible


Feat: 新增卡片组件

🤔 这个 PR 的性质是？

新特性提交 新增卡片组件

📝 更新日志

feat(Card): 新增卡片组件

[ ] 本条 PR 不需要纳入 Changelog

☑ 请求合并前的自查清单
⚠ 请自检并全部勾选全部选项。⚠

[x] 文档已补充或无须补充
[x] 代码演示已提供或无须提供
[x] TypeScript 定义已补充或无须补充
[x] Changelog 已提供或无须提供

Codecov Report

Merging #739 (345e291) into develop (a0db0df) will increase coverage by 4.01%.
The diff coverage is 80.58%.

:exclamation: Current head 345e291 differs from pull request most recent head 6f26e47. Consider uploading reports for the commit 6f26e47 to get more accurate results

@@             Coverage Diff             @@
##           develop     #739      +/-   ##
===========================================
+ Coverage    57.56%   61.57%   +4.01%
===========================================
  Files          346      348       +2
  Lines        12828    12898      +70
  Branches      2405     2430      +25
===========================================
+ Hits          7384     7942     +558
+ Misses        5200     4708     -492
- Partials       244      248       +4

Impacted Files
Coverage Δ

src/date-picker/date-picker.tsx
46.57% <0.00%> (+0.72%)
:arrow_up:

src/dialog/props.ts
70.00% <0.00%> (-30.00%)
:arrow_down:

src/form/const.ts
100.00% <ø> (ø)

src/slider/props.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

src/time-picker/time-picker.tsx
35.86% <0.00%> (+3.20%)
:arrow_up:

src/time-picker/time-range-picker.tsx
40.13% <0.00%> (+27.54%)
:arrow_up:

src/utils/withInstall.ts
87.50% <33.33%> (-12.50%)
:arrow_down:

src/card/props.ts
71.42% <50.00%> (+71.42%)
:arrow_up:

src/form/form-item.tsx
70.47% <75.00%> (+27.41%)
:arrow_up:

src/card/card.tsx
92.30% <92.30%> (ø)

... and 63 more

Continue to review full report at Codecov.

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


Update getRawTransaction endpoint

Pull Request Test Coverage Report for Build 227

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 66.906%

Totals

Change from base Build 222:
0.0%

Covered Lines:
774

Relevant Lines:
1125

💛  - Coveralls

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


Switch cookie serializer to json

Brakeman is reporting this as a fix, otherwise Whitehall's builds aren't passing.

You'll likely need to use brakeman -I to ignore the failure here.

 Fix Smokey errors with youtube-nocookie


How to support the SQL?

This is a best sandbox server. I already used these application.
and I want to know how to enable the SQL support?

The Jobe installer builds a system with sqlite3. The CodeRunner sql question type uses this via Python3.  If you're using Jobe outside of CodeRunner you'll have to write your own glue code in Python (or other language of your choice) to set up an sqlite3 database file and run sql commands on it.
Here's the template for the CodeRunner question type as a text file: sqltemplate.txt. If you have further questions related to CodeRunner, please ask them on the CodeRunner question authors forum .
If you're looking for sql support beyond the level of sqlite3, I'm afraid you're out of luck.

Thanks for your template


Use ccdVisit table skyNoise to compute DM image depth

Since it's in DN, I guess we want to do
D_sky = zeroPoint - 2.5*np.log10(5*skyNoise)

What kinds of numbers does this give? Let's plot D_sky against D_sigma where the latter is our simple estimate based on the median flux uncertainty across all DIAsources in the visit.

Conversation in DarkEnergyScienceCollaboration/Twinkles#407 is related to this issue


Error while generating apidocs

Executing the task
mvn javadoc:javadoc
produced this error:
Exit code: 1 - spark-2.3/src/main/java/spark/Request.java:267: error: semicolon missing
     * Example: query parameter 'id' from the following request URI: /hello?id=foo&id=bar
The problem is with the ampersand character (&), it should be replaced by &

Fixed.


Converting spec/dummy test app to use rails/webpacker for config

Moving from a custom config to rails/webpacker config
See docs https://github.com/rails/webpacker/blob/master/docs/webpack.md.

This change is

See #1247


Viki Ross mobile newsletter input

Cc @exezbcz wdyt
Originally posted by @vikiival in https://github.com/kodadot/nft-gallery/issues/5281#issuecomment-1471668924
so we will put the subscribe button to the second row but it will be still one container (top and bottom part connected

@roiLeo approves


Adding 3D Art Extreme Token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Hi!! If you need somebody to save you time and merge it for you, i'm your guy!!
https://github.com/SolanaTokenMerger

I can help refer this https://github.com/zakimzf/profile/pull/1
PS: if you don't need my help.. please ignore this!


Qt/ui: Add replay capabilities

[ ] Add widgets needed for replay in ui
[ ] Add a small program replay_ui to replay route in a simple UI
[ ] Integrate into qt/ui

simple replay ui:

I was actually thinking we would use the normal onroad UI for this. Would that simplify things?

I was actually thinking we would use the normal onroad UI for this. Would that simplify things?

I need time to think about this. I am worried that reusing onroad ui will introduce new complexity.

We also have some design already for an on-device replay. We will get back to you in the new year about our plans with this!


changeAction is behind when changing 'select' fields

changeAction is currently only called on 'click' and 'keyup' which does not fit the select field. When the 'select' field is clicked the action is called and the drop down is activated, but when the new option is selected the action is not called.
Could the 'change' action be used to trigger the changeAction function, or is there a reason only 'click' and 'keyup' are supported? (This is all in dynamic-form.js under _buildPostRender()).
I can submit a quick pr for this change too.

Hi @maprules1000 sounds reasonable.  Would love a PR ❤


feat(Interaction): add grab/ungrab helper methods to interactors

The InteractorFacade now has a Grab and Ungrab helper method that
allows an Interactable to be passed to a specific Interactor to
initiate a grab (or alternatively ungrab).
This makes it possible to initiate a grab/ungrab from either the
Interactor or the Interactable.

Removed the whole invoke thing now. Provides way of always going through the Interactor to do the grab/ungrab
toggle grab type on an interactable is handled by the interactor code.
It's possible to pass custom collision data to the Interactor grab if a precision grab is necessary but it has to be manually provided (if the objects aren't currently touching). If they are touching then the precision grab should work as it has a collision point already.
Only supports single grabbing/ungrabbing of objects.


Migration creates numeric column, not string

Nice library, thanks :)
I can see Money Ecto.Type dumps as a string. However, my Postgres migration is creating a numeric column.
Here's my schema:
defmodule ThingFinder.Item do
  use Ecto.Schema
  import Ecto.Changeset

  schema "items" do
    field :title, :string
    field :permalink, :string
    field :photo, :string
    field :price, Monetized.Money
  end
end

Ignore me, user error :)


fluentbit integrate OpenSearch plugin in fluentbit-operator

Signed-off-by: chengdehao<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Does this PR introduced a user-facing change?

0.13.0

Additional documentation, usage docs, etc.:

Reference https://github.com/fluent/fluent-operator/pull/298

The code branch of fluentbit operator is fluentbit-operator, not release-0.13.
We can switch to fluentbit-operator and then git rebase release-0.13 to sync up with that branch and then submit this PR to fluentbit-operator.
After that, cherry-pick the pr to release-0.13

closed via https://github.com/fluent/fluent-operator/pull/307


Update doc based on "non-fix" bug : ONECRM 966456

bug: https://dynamicscrm.visualstudio.com/OneCRM/_workitems/edit/966456/

@PedroCadavezF - Closing this to re-create it in the private repo.


.setgame acting strange

before updating to nadeko 2.8.2, .setgame would allow me to set nadeko's playing status to anything even if it wasnt actually a game. after updating, it has begun giving me a message saying "Command Errored after ............ Message: .setgame Nothing, Error: Value is not a PlayingType" anytime i call the setgame command with a non-game set of characters

and the nadeko bot commands page still says that is viable as a command, so my guess is either the commands page is not updated to the new version or setgame is being buggy

Kwoth updated Nadeko to make use of discord's API for playing types. Use Playing, Watching, or ListeningTo before your status text now.

Oh! Okay that makes sense. Thats actually pretty cool. Thanks for the help
On Tue, Dec 12, 2017 at 01:40 tukimoop<EMAIL_ADDRESS>wrote:

Kwoth updated Nadeko to make use of discord's API for playing types. Use
Playing, Watching, or ListeningTo before your status text now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kwoth/NadekoBot/issues/1920#issuecomment-350997681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfR1lX9Z8VdpvIWilmlEXQDqFvXFo3F8ks5s_koFgaJpZM4Q-pg8
.

--
-Dyv

.h .setgame


Help wanted: use   in newly created pages.

Following the tutorial from : https://akveo.github.io/ng2-admin/articles/013-create-new-page/
I created the 'new' Component,
I tried to use  popular-app in the template for the newly created page, but failed.

profile page h1 title

    <popular-app></popular-app>     <========== a directive

How I can reference this tag  , which is declared in app/src/pages/dashboard/popularApp
in the newly created component : app/src/pages/new/new.component.ts
I believe I'm missing some import at somewhere, but I don't know where it is.

your directive must exports in a module under declarations?

@Newan  thank you for your reply.  I tried following still no avail.
I exported PopularApp in   app/src/pages/dashboard/dashboard.module.ts
import { NgModule }      from '@angular/core';
import { CommonModule }  from '@angular/common';
@NgModule({
  imports: [  ],
  declarations: [ ],
  providers: [   ],
  exports:[    <===== newly added
    PopularApp   <==== export
  ],
})
export class DashboardModule {}

and I import DashboardModule into  app/src/pages/new/new.module.ts
@NgModule({
  imports: [
    CommonModule,
    FormsModule,
    routing,
    DashboardModule,  <====== imported
  ],
  declarations: [
    NewComponent,
  ]
})
export class NewModule {}

The funny thing is:    brings in entire dashboard demo page.
I thought I might have done something funny here.

if I add <popular-app> </popular-app> to the app/src/pages/dashboard/todo/todo.html, it can be correctly recognized though.
The only difference is todo and popular-app are declared under same module 'DashboardModule'

instead import DashboardModule, I only declare the popular-app, and it seems recognized it.
import { PopularApp } from '../dashboard/popularApp';

@NgModule({
  imports: [
    CommonModule,
    FormsModule,
    routing,
  ],
  declarations: [
    NewComponent,
    PopularApp   <===== declare it here.
  ]
})
export class NewModule {}

tried two components, popular- app and pie-chart ( as this one has data service), all using the same way to import it.

nice template to work with.


Don't consume Eirini extensions from the bosh-release

Is your feature request related to a problem? Please describe.
To include changes from Eirini extensions, we need to bump the eirini-bosh-release
Describe the solution you'd like
With KubeCF, there is no need to consume eirini-bosh-release jobs, but they can be native components. This would help integrating the extensions with #465
Describe alternatives you've considered
Keep using the eirini bosh release
Additional context
See also https://github.com/cloudfoundry-incubator/kubecf/issues/577

depends on #477

It doesn't necessarily, we can still consume eirini as-is from the bosh release, and avoiding to consume the extensions job


[WIP] [tests] add a test case for resolving services with IPv4/IPv6 addresses

Depends-On: openthread/ot-br-posix#1090

Closing this PR as of now since the issue is not prioritized.


[1.13] Incomplete information in the warning message on using tf.contrib.predictor

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version: 1.13.1
Doc Link: https://www.tensorflow.org/api_docs/python/tf/contrib/predictor/from_saved_model https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/saved_model/load https://www.tensorflow.org/alpha/guide/keras/saving_and_serializing

Describe the documentation issue
On using tf.contrib.predictor.from_saved_model to get predictions from a saved_model, the warning message says that there will be a new function to load saved models in TensorFlow 2.0, and nothing else.
Specifically, consider the following snippet of code (from https://colab.research.google.com/github/suyash/transformer/blob/master/imdb_sentiment_demo.ipynb)
p = predictor.from_saved_model(
    export_dir,
    input_names={"input_text": "input_text:0"},
    output_names={
        "prediction": "dense/Softmax:0",
        "attention_0": "attention/attention_weights:0",
        "attention_1": "attention_1/attention_weights:0",
    },
)
outputs = p({"input_text": inputs})
print(outputs["prediction"])
print(outputs["attention_0"])
print(outputs["attention_1"])

This prints the following warning message
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/contrib/predictor/saved_model_predictor.py:153: load (from tensorflow.python.saved_model.loader_impl) is deprecated and will be removed in a future version.
Instructions for updating:
This function will only be available through the v1 compatibility library as tf.compat.v1.saved_model.loader.load or tf.compat.v1.saved_model.load. There will be a new function for importing SavedModels in Tensorflow 2.0.

I have trying to figure out extracting multiple outputs from the same saved model. The current guide for saved models at alpha (https://www.tensorflow.org/alpha/guide/keras/saving_and_serializing) does not provide information on loading, while the equivalent for stable (https://www.tensorflow.org/guide/saved_model) does.
I am trying to use the new tf.saved_model.load function. I have prepared a demo for my use-case at https://colab.research.google.com/gist/suyash/2c7e5b77ea4d94d5c3b8c3e178d06878. Specifically, I create a model with dense layers, convert it to an estimator, trained it while defining an Exporter for evaluation checkpoints.
Now, when I load using the new function, I am able to obtain values for the dense_2 tensor, however in an equivalent scenario, how do I get the values of the dense_1 tensor in the current approach. As I show in my predictor example, the output of the predictor will have 3 keys, "prediction", "attention_0" and "attention_1". How do I get the output of the "dense_1" tensor in my 2.0 example?
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?

@allenlavoie Can you PTAL? Thanks!

So basically you want to fetch an intermediate Tensor? The loaded signatures do have a .prune method which takes input and output Tensors from the function graph and gives you a new function (signature_function.graph will get to the Tensor objects).
The documentation for prune() should be improved; it needs a docstring and should probably take Tensor names instead of just Tensors. And we should add a pointer to the 2.x guide to the contrib predictor depreciation warning.

@allenlavoie I am not seeing anything about a prune method on the documentation page for tf.saved_model.load, where can I find that. Is there also some place I can find equivalent 2.x code for fetching intermediate values from saved_models.

@suyash I think this was resolved already. I see the prune method mentioned almost in the end of this page.
I am closing this issue. Please feel free to reopen if I am mistaken. Thanks!


Can't use the lastest merged version of the mod.

I've been wanting to use the lastest merged ver of this mod but I can't get it to work.

I was going to check the changes but one of thr game patches broke chess so I waited for another one. For now you can go to the unknown styles menu and search for the new styles. Changes will be persistent so it should work until the update.


Handle some errors

I ran a test which ended up crashing with nil dereference due to unhandled error. So fix some of many places we cheerfully ignore errors.

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?
Courtesy of your friendly test nag.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: qfel
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @qfel. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren, qfel
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qfel: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, ozevren, qfel
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-istioctl-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-istioctl-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-istioctl-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-istioctl-k8s-presubmit-tests-master

prow/release-test.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test release-test-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-istioctl-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-istioctl-k8s-presubmit-tests-master

prow/release-test.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test release-test-master

prow/e2e_pilotv2_auth_sds.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test istio_auth_sds_e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@qfel: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

/retest

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-istioctl-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-istioctl-k8s-presubmit-tests-master

prow/release-test.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test release-test-master

prow/e2e_pilotv2_auth_sds.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test istio_auth_sds_e2e-master

prow/integ-framework-local-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-framework-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/integ-istioctl-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-istioctl-k8s-presubmit-tests-master

prow/release-test.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test release-test-master

prow/e2e_pilotv2_auth_sds.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test istio_auth_sds_e2e-master

prow/integ-framework-local-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-framework-local-presubmit-tests-master

prow/integ-mixer-local-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qfel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-k8s-presubmit-tests-master

prow/release-test.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test release-test-master

prow/e2e_pilotv2_auth_sds.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test istio_auth_sds_e2e-master

prow/integ-framework-local-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-framework-local-presubmit-tests-master

prow/integ-mixer-local-presubmit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test integ-mixer-local-presubmit-tests-master

prow/istio-unit-tests.sh
ad862daf8a393e445a723d774318d8e605fe6c69
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all


Update resolvconf-apt.yml

Thanks! Should be addressed by #15 but please re-open if I closed this in error!


Get replication lag information from pg_stat_replication

Convert write_lag, flush_lag and replay_lag from interval to float before expose the information.

Coverage increased (+0.2%) to 64.897% when pulling 6f74aa3fe314aae0ee3655f3da4d3ec94badb078 on azlev:master into e2df41f43effdfbb4544b3efb70350380ad895fe on wrouesnel:master.

Changing built in collectors to depend on a Postgres database function isn't a call for the exporter to make - this is the type of thing which should be supplied via a queries.yml file.


Unable to get hacks to install. remote_installer.sh confirms firmware bin was pushed, but no change from cam

So I have a few v2's in my house, about half have SD cards, the others do not. So far tested on 2 cams with no SD cards installed (one cannot due to, I assume, a failed SD card slot or connecting hardware. The rest of the cam is fully functional). After doing some trial and error I have come as far as having the remote_installer script work and confirming the cam has pulled the firmware from the local server but the camera doesn't reboot, no audible announcements, and the live feed from the Wyze app continues to run.
Currently operating from an Ubuntu WSL install on my Windows 10 v2004 machine. All dependencies properly installed, python 3.8.2.
I started with the most recent FW version, <IP_ADDRESS> for the v2's but even downgraded to the noted last confirmed working version on the README page of <IP_ADDRESS> just to rule things out. Both do the same. Even tried rebooting the cams amidst all this. Here's a snippet below of the script on one cam I have attempted to hack.

Device type:      Camera (WYZEC1-JZ)
Device name:      Kitchen
Firmware version: <IP_ADDRESS>
IP Address:       <IP_ADDRESS>

Pushing firmware to this device? [y/N]:y
INFO:root:Serving firmware file './firmware.bin' as 'http://<IP_ADDRESS>:11808/firmware.bin', md5=68e279ca2d3208bbb55111ac6d38c432
<IP_ADDRESS> - - [24/Oct/2020 15:31:51] "GET /firmware.bin HTTP/1.1" 200 -
INFO:root:Checking device, mac=XXXXXXXXXX

Any thoughts on where I should look next or additional troubleshooting steps I can try? Ultimately I'd like to remove the SD cards altogether and just have everything fed over NFS and leave it at that.

Never mind. Closing this out. PEBKAC error. I'm so used to cloning repos directly especially if no binaries are involved. Grabbed the zipped release linked on the front page and utilized it and it is fine. Apparently there's enough different between that and the full repo clone to cause issues despite it looking like it completed everything ok.


Allow usage by tsserver 2.2.1 #7

Issue #7
This PR modifies the index.ts so it complains both with the PluginModuleFactory and
PluginModule interfaces. This allows for the tsserver 2.2.1 to load the plugin and keeps retro compatibility with previous loaders of this library.
This PR also modifies the project structure in a way that can be interpreted as opinionated. This are the modifications:

Added test folder.
Added intermediary folder for compilation .build (gitignored)
Removed lib folder from git. (gitignored)

The reason for the changes is that the tests also need to be compiled and would pollute the release files.
Also, I believe this release files / artifacts shouldn't be part of the source code. These should be generated right before release. This avoids conflicts in merges etc on files no developer really cares about. The release of this files should be done probably using an npm module.
I also modified the package.json file:

Moved non strong dependencies to peerDependencies.
Added missing and test dependencies (tape / types)
Added test script (runs the test with npm test)
Added build script (builds the artifacts / lib folder with npm run build)

@Istar-Eldritch thanks for your PR. I tell me why you have removed compiled js files? I have tried to follow the same idea than TypeScript have a lib folder which contains compiled js files.
As you have removed js files, when js files are compiled? When plugin will be published with npm publish?
Why do you use tape (I didn't know) and not mocha, chai like TypeScript?

For compiled js files, it seems angular language service doesn't host js files https://github.com/angular/angular/tree/master/packages/language-service
I'm OK with that. It's just the .build folder which seems strange (I have never seen that). I think we should activate test too with travis.

I'm glad you found other repos that do not keep the deployment files. Having the compiled files in the repo sounds to me like committing the java artifacts or classes. It doesn't really add value and it may in fact create problems with merges and conflicts, appart from deviating the attention curing CR from the changes that really matter in the project.
Regarding the reasons behind I started with tape and not mocha, this blog post summarizes it quite well.

Too much configuration: Choose an assertion library, chose a reporting library, chose a task runner (Grunt, Gulp, etc…) Then figure out how to translate the documentation examples to the reporting library / task runner you chose. All of this is too much cognitive load. Vs: Choose Tape. Done.
Globals: Mocha, Jasmine, and several other alternatives pollute the global environment with functions like describe, it, etc… Some assertion libraries extend built-in prototypes. Aside from removing the self-documenting nature of simple module exports, those decisions could potentially conflict with the code you’re trying to test. Vs: Tape’s simple module export.
Shared State: Functions like beforeEach and afterEach actively encourage you to do something you definitely should not do: Share state between tests. Vs. Tape: No such functions for global state sharing. Instead, call setup and teardown routines from individual tests, and contain all state to local test variables.

At @repositive we switched from mocha to tape 6 months ago for all our backend services, and we won't look back. The reason we switched was the shared state of the tests, at that moment I was doing some crazy test with streams, and the simplicity of the structured vs stateful tests helped a lot.
As I told you, I'm used to do stuff with mocha, so if you see benefits on using it I can change the PR.

Thanks a lot @Istar-Eldritch for your great claraification. I'm not a big expert with JS tests framworks, so I trust you.
I think next step is to execute test with travis.


Cross-compile for Scala 2.11

Spark 2.0 is using 2.11 by default, making hbase-rdd compatible.
This would really help us with Spark 2.0 migration. Thanks.
@fralken

Good idea, thanks!

Any chance for a new release any time soon? :)

Yes, it's released. Sorry I didn't mention it explicitly.


[LOG] Reduce verbose exception log by catch exceptions

Proposed changes
In our product enviroment, we use LVS to dispatch requests to FEs,
however, LVS will send probes to check whether FE is alive, and will
close the connecttion immediately. It will cause much verbose log,
this patch aim to reduce these log by catch related exceptions.
Types of changes
What types of changes does your code introduce to Doris?
Put an x in the boxes that apply

[] Bugfix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation Update (if none of the other choices apply)
[x] Code refactor (Modify the code structure, format the code, etc...)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[] I have create an issue on (Fix #ISSUE), and have described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes
[] I have added tests that prove my fix is effective or that my feature works
[] If this change need a document change, I have updated the document
[] Any dependent changes have been merged

Hi @acelyc111 ,ut failed at MysqlProtoTest.testNegotiateSendFail

@morningman Fixed

Hi @acelyc111 ,ut failed at MysqlProtoTest.testNegotiateSendFail

@morningman Fixed


get error NotFoundHttpException in RouteCollection.php line 179

Hi
my root not "/public" and only "/"
when I install laravel file-manager I get this error:
NotFoundHttpException in RouteCollection.php line 179
see below image:

please help me :(

screenshot of my consol:

I find out that conflict with language middleware. how to slove it?
`<?php
// app/Http/Middleware/Language.php
namespace App\Http\Middleware;
use Closure;
use Illuminate\Http\Request;
class Language
{
public function handle(Request $request, Closure $next)
{
// Check if the first segment matches a language code
if (!array_key_exists($request->segment(1), config('translatable.locales'))) {
// Store segments in array
$segments = $request->segments();
// Set the default language code as the first segment
$segments = array_prepend($segments, config('app.fallback_locale'));
// Redirect to the correct url
return redirect()->to(implode('/', $segments));
}
    return $next($request);
}

}
?>`

what file did you edit? where is it located? Thanks!

hi,
I git same error how do you fixit


isort

sort import blocks

[!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

#196  👈
#195
#194
#193
#192
#191
#190
#189
#188
#187
master

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

Join @chrisaddy and the rest of your teammates on  Graphite

Merge activity

May 19, 10:11 PM EDT: @chrisaddy started a stack merge that includes this pull request via Graphite.


Use correct KES construction

This PR updates ouroboros-network to use the newest KES constructions from cardano-base, as well as fixing some bugs pertaining to how we signed with and evolved our KES keys.
Each commit can be reviewed independently.

bors merge

bors retry


Devices not caching? Re-detected every restart

Hi, Love the plugin! I have my tuya devices working locally!
I however am having an issue where my devices are not cached and re-detected/removed and re-added upon every restart of homebridge.
This leads to the devices disappearing and reappearing in homekit, breaking any automations and resetting the device customizations such as room selected.
To Reproduce
Setup device automations using tuya detected devices.
Restart homebridge.
Devices are removed and re-added to homekit, breaking automations as the devices are detected as a new.
Expected behavior
Devices are cached and not re-detected upon homebridge restart/Not removed from homekit until detected within plugin.
Logs
[9/12/2021, 11:24:40 AM] [homebridge-tuya.TuyaLan] Discovered Porch Lights (xxxID) identified as SimpleDimmer (3.3)
[Tuya] Changing ping gap for Porch Lights to 9s
[9/12/2021, 11:24:40 AM] The plugin 'homebridge-tuya-lan' registered a new accessory for the platform 'TuyaLan'. The platform couldn't be found though!
[Tuya DEBUG] reconnect called for Porch Lights
[9/12/2021, 11:24:40 AM] [homebridge-tuya.TuyaLan] Connected to Porch Lights
[Tuya] Sending first query to Porch Lights (3.3)
[Tuya] Heard back from Porch Lights with command 10
[9/12/2021, 11:24:40 AM] [homebridge-tuya.TuyaLan] Ready to handle Porch Lights (SimpleDimmer:3.3) with signature {"1":false,"2":57,"3":10,"4":"halogen"}
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Discovered Bay Window (xxxID) identified as SimpleDimmer (3.3)
[Tuya] Changing ping gap for Bay Window to 9s
[9/12/2021, 11:24:43 AM] The plugin 'homebridge-tuya-lan' registered a new accessory for the platform 'TuyaLan'. The platform couldn't be found though!
[Tuya DEBUG] reconnect called for Bay Window
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Connected to Bay Window
[Tuya] Sending first query to Bay Window (3.3)
[Tuya] Heard back from Bay Window with command 10
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Ready to handle Bay Window (SimpleDimmer:3.3) with signature {"1":false,"2":25,"3":10,"4":"LED"}
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Discovered Ceiling Fan (xxxID) identified as Fanlight (3.3)
[Tuya] Changing ping gap for Ceiling Fan to 9s
[9/12/2021, 11:24:43 AM] The plugin 'homebridge-tuya-lan' registered a new accessory for the platform 'TuyaLan'. The platform couldn't be found though!
[Tuya DEBUG] reconnect called for Ceiling Fan
[Tuya] Discovery ended.
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Connected to Ceiling Fan
[Tuya] Sending first query to Ceiling Fan (3.3)
[Tuya] Heard back from Ceiling Fan with command 10
[9/12/2021, 11:24:43 AM] [homebridge-tuya.TuyaLan] Ready to handle Ceiling Fan (Fanlight:3.3) with signature {"1":true,"3":"2","9":true,"10":100,"101":0,"103":0,"105":10,"106":"mode1"}
[Tuya DEBUG] decrementing this._connectionAttempts, currently 2
[Tuya DEBUG] decrementing this._connectionAttempts, currently 1
[Tuya DEBUG] decrementing this._connectionAttempts, currently 2
[Tuya DEBUG] decrementing this._connectionAttempts, currently 1
[Tuya DEBUG] decrementing this._connectionAttempts, currently 2
[Tuya DEBUG] decrementing this._connectionAttempts, currently 1
Homebridge Config
{
"platform": "homebridge-tuya.TuyaLan",
"devices": [
{
"name": "Porch Lights",
"type": "SimpleDimmer",
"manufacturer": "Light",
"model": "DS01C",
"id": "xxx",
"key": "xxx"
},
{
"name": "Bay Window",
"type": "SimpleDimmer",
"manufacturer": "Light",
"model": "DS01C",
"id": "xxx",
"key": "xxx"
},
{
"name": "Ceiling Fan",
"type": "Fanlight",
"manufacturer": "Light",
"model": "DS03",
"id": "xxx",
"key": "xxx",
"dpActive": 1,
"dpRotationSpeed": 3,
"dpLightOn": 9,
"dpBrightness": 10
}
]
},
Environment (please complete the following information):
Latest Onzu homebridge docker.
https://github.com/oznu/docker-homebridge

I believe this was due to having homebridge-tuya-lan and homebridge-tuya installed at the same time. Causing various issues.


DISTX-734:Add enableloadbalancer to stack response to create respective cli response

This change is required for adding the enable load balancer for the response in the datahub cli command created. The thunderhead PR for this change is https://github.infra.cloudera.com/thunderhead/thunderhead/pull/7465

@satyabollineni Don't you need to add logic in StackToStackV4ResponseConverter to set the new field in the response object?

@hreeve-cloudera Thanks for the response here. I have simulated the same logic thats used for enableloadbalancer in the stackV4request in the response too of StackToStackV4ResponseConverter, Please let me know If there is any further suggestion on this.

@satyabollineni Don't you need to add logic in StackToStackV4ResponseConverter to set the new field in the response object?

@hreeve-cloudera Thanks for the response here. I have simulated the same logic thats used for enableloadbalancer in the stackV4request in the response too of StackToStackV4ResponseConverter, Please let me know If there is any further suggestion on this. But this is actually leading to compatability error. So I think It's better that we avoid this logic in response as the intent of this change was only to add the enable-load-balancer field to the response.ftl files in thunderhead. Please suggest me otherwise.

Adding a new field to an API response message shouldn't be causing a compatibility error. Have you rebased your code recently? If not you should and push the rebased branch. If you have rebased and the swagger-compatability-test is failing, I'd suggest going to jenkins and retrying it manually. That test can sometimes be inconsistent.


unique and relation together on datamodel.prisma

type Product {
id: ID! @id
component: Component @relation(link: INLINE) @unique
}
I deploy successfully but actually it doesn't unique.
Edit
I am using Prisma 1.3.4 and using mongoDB connector.
I am sorry about that i misunderstand the problem. Actually it's work for @relation(link: INLINE) @unique together.
But I encountered the problem is the graphql schema of mutation, The update product mutation doesn't allow me to use unique componentId to filter the product. For example,
mutation {
  updateProduct(
    where: { id: "productId" }
    data: {
      component: {
        update: {
          where: {
             componentId: "I want tp put componentId here, but schema doesn't allow it. It's appear when i changed to "@unique" without "@relation" "
          }
          data: {
            component: { connect: { id: "componentId" }
          }
        }
      }
    }
  ) {
    id
  }
}

try using @unique
like this:
type Product {   id: ID! @id @unique }

It doesn't same...
for example,
It doesn't help me delete if i change to @relation(link: cascade).

@codingfunwoody
Can you please tell me which connector you are using and what is your Prisma version?
I am using 1.34.3 with postgres connect and constraint is properly created in the database.

@pantharshit00
I am using Prisma 1.3.4 and using mongoDB connector.
I am sorry about that i misunderstand the problem. Actually it's work for @relation(link: INLINE) @unique together.
But I encountered the problem is the graphql schema of mutation, The update product mutation doesn't allow me to use unique componentId to filter the product. For example,
mutation {
updateProduct(
where: { id: "productId" }
data: {
component: {
update: {
where: {
componentId: "**I want put componentId here, but schema doesn't allow it. It's appear when i changed to "@unique" without "@relation" **"
}
data: {
component: { connect: { id: "componentId" } }
}
}
}
) {
id
}

This doesn't sound very intuitive to me as you are trying to update a product using the mutation.
I think you shouldn't be able to filter here. The component available in the update input is the one connected the product. The best here is to separate out the mutations.


feat: Add protos for service-level comparison testing

Add submodule of api-common-protos
Add echo proto (original version and updated version for testing) to test service-level, file-level changes.
Add protobuf protos for echo.

I would suggest also adding https://github.com/protocolbuffers/protobuf as a submodule, to bring in the standard protobuf protos - e.g. any.proto.
And perhaps don't bring in echo.proto to use for testing, rather use protos that are as small as possible to test each specific change separately?

Sounds good, pulled in api-common-protos and protobuf as submodules.


Password with special characters doesn't work either escaped or unescaped

Passwords with special characters (e.g. test?password) break connection string parsing.
Escaping the password (with urllib.parse.quote_plus()) fixes parsing, but the value is not unescaped so login fails.

If you use pysqlsync.factory.get_parameters(url), a ConnectionParameters object is constructed from the output of urlparse, which handles special characters in every component of the URL. In fact, integration tests have already had <YourStrong@Passw0rd> as the password, and they pass. I have now updated the password to <?YourStrong@Passw0rd> and they still pass for all supported databases PostgreSQL, MySQL, Oracle and Microsoft SQL Server.

In order to improve robustness of parsing username and password components of connection string URLs, unquote is now being applied to the output of urlparse to reverse percent encoding. I chose unquote over unquote_plus because RFC3986 appears to permit plus (and many other special characters) verbatim in username and password. New unit tests have been added with complex use cases. Code has been pushed to the branch dev. Thanks for reporting the issue!


Update README.md

Thanks for creating a pull request!

Hello there, thanks for creating a new Pull Request!


displaying full names on hover for long names only

Proposed Changes

Fixes #7787
Displaying asset names for long names only
asset.name > 18 seems logical considering w-48 remains constant, but if we change w-48 to some other value, we can increase 18 to something else, but don't think w-48 will change, even if it does, we can adjust 18 to some sensible mid-number like 15-16

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

@SwanandBhuskute Can you add links to example assets where the feature can be seen in action?

@SwanandBhuskute Can you add links to example assets where the feature can be seen in action? Also attach Before and After Screenshots

https://care.ohc.network/assets
the feature can be seen into action on this route
currently this is how it works on localhost
Before hovering:

After hovering

optional working video
https://github.com/user-attachments/assets/491afe27-de5e-42ba-8840-e5a3331311ee

Member

so I should do changes in CAREUI.css file?

No why? Just use those classes like how it's used in other places.

so I should do changes in CAREUI.css file?

No why? Just use those classes like how it's used in other places to show tooltips.

Using tooltip and tooltip-top classes
the hover doesn't position that good

so using tailwind only, to achieve same effect as earlier without extra style attribute inside

@SwanandBhuskute you can use it in conjuction with translate-* classes to control where it's positioned like how it's done in most other usages of the tooltip classes.

@SwanandBhuskute you can use it in conjuction with translate-* classes to control where it's positioned like how it's done in most other usages of the tooltip classes.

@rithviknishad i have done as you said, kindly review

LGTM


'Ibid.' not capitalized at the beginning of a sentence when invoked within footnote

In note-based citation styles with 'ibid', citations that do print as 'ibid.' and that are being invoked from within a footnote apparently do not get capitalized.
Example document:
\cite{Lipinska2014}
\cite{Lipinska2014}
\Cite{Lipinska2014}
\footnote{\cite{Lipinska2014}. \cite{Lipinska2014}.}
\footnote{\Cite{Lipinska2014}. \Cite{Lipinska2014}.}
\footcite{Lipinska2014}
\Footcite{Lipinska2014}

and example bib:
@book{Lipinska2014,
  location = {Leiden and Boston},
  title = {Moving Sculptures},
  subtitle = {Southern Netherlandish Alabasters from the 16th to 17th Centuries in Central and Northern Europe},
  isbn = {978-90-04-27093-0},
  pagetotal = {386},
  timestamp = {2016-08-29T13:43:40Z},
  language = {english},
  number = {2},
  series = {Studies in Netherlandish Art and Cultural History},
  author = {Lipi{\'n}ska, Aleksandra},
  date = {2014}
}

With pandoc TestDoc_ebdKlein.tex --bibliography=Test.bib --csl=chicago-note-bibliography-with-ibid.csl -o Test.docx, I get

Most importantly to me, irrespective of the capitalization of the cite command itself, none of the citations invoked from within an explicit \footnote{} are capitalized. (See footnotes 4, 5.)
Called from outside an explicit footnote, irrespective of the capitalization of the cite command \cite/\autocite, 'Ibid.' will be capitalized.
Only for the \footcite command, capitalization appears to make a difference (footnotes 6, 7).

The discrepancy between \footcite and \Footcite surprised me, but it turned out to be a bug in pandoc.  After the change I just made, they should both produce the uncapitalized version.
In general, pandoc's translation of biblatex citation commands to native pandoc citations is not going to be 100% accurate, because these are quite different systems.  There's no flag or parameter in a pandoc citation that says "capitalize the first letter."  So we do the best we can and translate \Footcite and \footcite the same way.
You get capitalization for regular cite commands that are being transformed into footnotes because it's a footnote style.  This is because of a general transformation we run when converting citations to notes.
It isn't run on things that are already notes.
In principle we could have our function look also for existing notes that happen to start with a citation, and apply the capitalization there, too.

Thank your for taking the time to consider this. In general - to the best of my knowledge - biblatex's citation commands work independent of whether an 'in-text' or a note citation style is being loaded. Conversely, where and how a citation appears is governed by the appropriate choice of citation commands.
So when reading a LaTeX document that was written with the output of note-based citations in mind, one will probably hardly ever encounter the standard \cite command in the text. In other words, when you write

You get capitalization for regular cite commands that are being transformed into footnotes because it's a footnote style.

, this rule should have rather little empirical relevance, since a LaTeX document wouldn't have been written accordingly to begin with, or am I wrong? Instead, in theses cases one will mostly see either \footcite or \footnote{ [...] \cite{...} [...]}.

In principle we could have our function look also for existing notes that happen to start with a citation, and apply the capitalization there, too. I believe that would help with your case.

It would, yes, in those cases where a footnote starts with a citation first, which then may or may not be followed by additional text. At least in the \footcite use case, I guess for now I might try to add something along the lines of \renewcommand{\footcite}{\cite} in the source tex document to achieve the same.
But in those cases when a footnote first sees some text - ended by a punctuation mark that warrants capitalization afterwards -, which is then followed by a citation, this would still produce uncapitalized 'ibid.'s, am I wrong?
Could one test, in addition, whether a citation is preceded by a dot punctuation, indicating the end of a sentence? Or is that impossible, which you mean when you write

There's no flag or parameter in a pandoc citation that says "capitalize the first letter."

?
P.S.: Sorry for putting so much emphasis on the capitalization of the LaTeX citation commands themselves in the original post. I think I misinterpreted the function of said capitalization.

, this rule should have rather little empirical relevance, since a LaTeX document wouldn't have been written accordingly to begin with, or am I wrong? Instead, in theses cases one will mostly see either \footcite or \footnote{ [...] \cite{...} [...]}.

Yes, but this is one of the differences between citeproc and biblatex.  Biblatex allows you to produce a mix of footnote and parenthesized citations in the same document; with citeproc, it's either a footnote style (in which all citations are footnotes) or a parenthesized or numeric style.  So we do the best we can.


Do we support compression for query results?

Sometimes I need to return a very large selection query response.

No, we dont support compression for large results. Definitely a good feature to support. When you say large - does it mean 10k, 100k or 1m?

Will these queries have group by or just select + filter. I am thinking if its better for servers to send compressed response to broker and broker simply concatenate them and sends it across.

In this case, the queries are select + filtering.

Does that mean broker will act as pass through i.e  it just concatenates the responses from the servers.

Yes, broker will merge data from all the servers then send back, but this may eat too much memory for a single query.

We should support cursor for this use case. We can still compress responses
but cursors will keep brokers simple and we don't pay all the processing
cost upfront.
On Thu, Aug 25, 2016 at 5:37 PM, Xiang Fu<EMAIL_ADDRESS>wrote:

Yes, broker will merge data from all the servers then send back, but this
may eat too much memory for a single query.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/linkedin/pinot/issues/399#issuecomment-242588342, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAmRnLtBkXn10IWreGuL59VeXQzymtAks5qjjUwgaJpZM4JlC3A
.

@antumbde can you elaborate on the solution. I get the cursor part from client but how would it work between broker and server.

For simple select + filter, we don't need scatter-gather. We can let only
one server process at a time and return compressed response that are passed
through broker. This works for simple select, filter, project queries.
That doesn't work for group-by. But, reading large datasets with group-by
is lower probability use case.
@ Xiang Fu : Is this for presto plugin ? Is the limitation mentioned above
a problem for your use case ?
-Adwait
On Thu, Aug 25, 2016 at 5:44 PM, Kishore Gopalakrishna <
<EMAIL_ADDRESS>wrote:

@antumbde https://github.com/antumbde can you elaborate on the
solution. I get the cursor part from client but how would it work between
broker and server.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/linkedin/pinot/issues/399#issuecomment-242589313, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAmRk4LXl8W95Kp-Ru5TLQTuwqoriJgks5qjjb1gaJpZM4JlC3A
.

Yes, for presto mostly and I need to get this cursor feature in java client implementation.
My feeling is it's fine to do so as broker could choose when to push down the request to a server.
Also I'm thinking is it possible to write intermediate query response to disk on broker and mmap them for merger or stream out.


Res.time() in rest assured is giving slightly higher response time as compared to rest client. Is there any way to get accurate response time with rest assured?

Could you push your sample code to public repository?


Expected character: ) in Regex + Boolean operator expression (Invalid filter)

This is my JSON
{   "a": {     "b": "cdefg"   } }
This is my JayWay JsonPath expression
[?((($.a.b =~ /cd.*/) && ($.a.b == 'cdefg')))]
I'm getting an error Expected character: ) when running here
Any reason why ?
BTW, if i'm changing sides of operands, like so [?((($.a.b == 'cdefg') && ($.a.b =~ /cd.*/)))] it matches OK

Hii, I am curious on your issue but I think I did not get it right.
I tested the project according to your issue as below:
@Test     // https://github.com/jayway/JsonPath/issues/692     public void issue692() {         final String jsonDemo = "{ \"a\": { \"b\": \"cdefg\" } }";         final ReadContext context = JsonPath.parse(jsonDemo);         final Filter validFilter = Filter.parse("[?((($.a.b == 'cdefg') && ($.a.b =~ /cd.*/)))]");         final Filter invalidFilter = Filter.parse("[?((($.a.b =~ /cd.*/) && ($.a.b == 'cdefg')))]");         assertThat(validFilter.apply(createPredicateContext(context))).isEqualTo(invalidFilter.apply(createPredicateContext(context)));     }
And the outcome seems alright. Would you kindly let me know if I did not get your problem?

I've tested that on the evaluator at https://jsonpath.herokuapp.com/
[?((($.a.b == 'cdefg') && ($.a.b =~ /cd.*/)))] is a valid filter and return filter results
[?((($.a.b =~ /cd.*/) && ($.a.b == 'cdefg')))] isn't a valid filter and return with error Expected character: )

@Chaoyi-1230 Any updates on my last comments?

Hii moshevi, sorry for the delay.
I have verified the problem you pointed out, and I am trying to fix it now.
Anyone else is interested in this issue is welcome to take part in :)

@Chaoyi-1230 Any progress on fixing this?

I see there's a fix for that on [(https://github.com/json-path/JsonPath/pull/661/commits/e68e9f2f9b04d741b85797335ac78f1a8c7317c2)]
Any idea when that will be released?

Fixed in release 2.6.0


fix: add metrics token support

Metrics guide should follow the same pattern to tell users to copy token url from UI and CLI.
Alternatively we can give them curl to obtain that value (if somehow they did not copied it before)
Source: https://issues.redhat.com/browse/MGDSTRM-8460

Amazing collaboration. This was low effort PR from my side. Now it is very well ready to be merged. Thank you so much for making the change.
Once that is merged all token urls will be consistently provided by UI and CLI. Some example outputs still have it but that is not affecting users in any way

@wtrocki ack
@AedinC Since the related Jira belongs to you, can you have a look and approve if you agree? Just to ensure I'm not stepping on your toes. Once you're good, go ahead with the merge. Thanks.

LGTM


⚠ Samsung has degraded performance

In 8dc5527, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2519 ms

Resolved: Samsung performance has improved in bbdbb8d.


docs: add Japanese README

I created Japanese translated README.

Thank you for your contribution! The document looks great.


[Improve][Connector-V2] Refactor hive source & sink connector

Purpose of this pull request
According to #2555, this pr has optimized the hive sink connector on top of it.

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

@CalvinKirs  PTAL


pandoc fail to render bibliographic note

As discussed on the list I no longer get notes in my bibliography (such as bibtex.csl).
This regression was introduced sometime after pandoc=<IP_ADDRESS> & pandoc-citeproc=0.7.2 .
% Test
% Author

# Introduction

Here is a test citation [@WatsonCrick1953]

# References

---
references:
- type: article-journal
  id: WatsonCrick1953
  author:
  - family: Watson
    given: J. D.
  - family: Crick
    given: F. H. C.
  issued:
    date-parts:
    - - 1953
      - 4
      - 25
  title: 'Molecular structure of nucleic acids: a structure for deoxyribose
    nucleic acid'
  title-short: Molecular structure of nucleic acids
  container-title: Nature
  volume: 171
  issue: 4356
  page: 737-738
  DOI: 10.1038/171737a0
  URL: http://www.nature.com/nature/journal/v171/n4356/abs/171737a0.html
  language: en-GB
  note: this is a note
...

The regression was introduced by the fix for #94.


Fails to load on port 80 and HMR won't disable when you change port.

Environment
Windows
Node 16.16.0
Reproduction
NUXT_NO_SSR=true NUXT_HOST='<IP_ADDRESS>' PORT=80 npm run dev
Describe the bug
I was expecting to have HMR off with port 80 to showcase an application to others remotely.
HMR kept refreshing and I had to search for many ways to attempt to stop it. It won't turn off. Interestingly, when you try to use port 80 while trying to remove ssr, the application never loads for my domain. Quite fustrating.
Additional context
No response
Logs
No response

is this how to disable hmr?
  vite: {
    server: {
      hmr: false
    }
  },

I still get the error:
ℹ Listening on: http://<IP_ADDRESS>:3000/
ERROR connect EINVAL <IP_ADDRESS>:80 - Local (<IP_ADDRESS>:0)
at internalConnect (node:net:1041:16)
at defaultTriggerAsyncIdScope (node:internal/async_hooks:464:18)
at GetAddrInfoReqWrap.emitLookup [as callback] (node:net:1187:9)
at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:111:8)

I'm running on OpenShift 4.x and there is only port 3000 to be configured on this deployment:
❯ oc -n ${PROJECT} describe dc/${APP_NAME}-baserow-web-frontend

Name:           take2-baserow-web-frontend
Namespace:      acf456-test

...
Pod Template:
...
    Port:       3000/TCP
    Host Port:  0/TCP
    Args:
      nuxt
      --config-file
      /baserow/web-frontend/config/nuxt.config.prod.js


[Bug Report]: Repetitions do not work for table columns

Environment
Carbone Version: latest
Node Version: 13
Browsers: chrome latest
Desktop OS: ubuntu 18.04
Expected Behaviour
The repetitions to work for table columns the same way they work for rows.
Actual Behaviour
Only the two first columns of the table are rendered if there are more in the data

So I've been doing a bit of investigation here in the hope that I could either solve it or at least figure out a workaround. For now the best workaround I've come up with is splitting my array into 2 element slices and having multiple (i, ,i+1) paired columns in the table based on the number of entries in the array. This limits it to only supporting arrays of length N to 2N where N is the number of pairs you have, and does involve some data wrangling on the input values to split them across the groups as needed.
The cause of the issue is actually the structure of the odt - it correctly inserts the rows in the table with the right number of table cells but the table columns aren't being adjusted to add the extra columns.

For anyone else looking to workaround this, if you save the template in docx format it can generate more than 2 columns. It's just an issue with odt files.


go.sum is not synced with your go.mod

When trying to use the latest version with go modules, I encountered a problem with your go modules:
github.com/4kills/go-libdeflate<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/4kills/libdeflate
                but was required as: github.com/4kills/go-libdeflate

Try using go mod tidy to resolve it.

This should fix it #9


[enhancement]: Allow for Java 17 runtime version on AzureSpringCloudV0

Task name
AzureSpringCloudV0
Describe your feature request here
Currently this task only allows for picking Java 8 or 11, even though Java 17 is also supported on Azure Spring Apps as a runtime version. Allow for Java 17 as a runtime version.

I'd even extend this request to Java 21.
Azure Spring Apps supports Java 21 now, but the AzureSpringCloudV0 task throws the following error:
"Invalid arguments: properties.source.runtimeVersion with value Java_21 is not supported for type Jar"
As for Java 17: I think its not documented, but if you just try with RuntimeVersion: Java_17 it should work.

@vermegi Thanks for reporting this! We're working on making java runtime 17 and 21 available for AzureSpringCloudV0 task.


cornmeal is actually obtainable now

About the PR
You can actually grind corn into cornmeal now.

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

:cl:

fix: You can actually grind corn into cornmeal now.

well shit

i think that test fail is an error because i have no other explanation
its exactly the same formatting that tomatoes have at line 584

wait what? i swear when i added cornmeal i made it obtainable...

yeah, does this not work anymore?

grinding corn currently gives nutriment and vitamin
by adding the one line YML change i added, you should actually get the cornmeal

weird

Test it and then it should be good to merge.

 I'm malding.


Godot connecting to internet without asking me (Even if the firewall is turned on)

macOs Sierra
Issue description:
Godot connecting internet without asking me (Even if the firewall is turned on)
there is no Connect/Disconnect options in godot (Offline Editor)
Steps to reproduce:
1.Godot blocked in Firewall
2.Godot is active in Network Activity Monitor(macos sierra)
Link to minimal example project:

I noticed this too, I wonder what's going on..
On Dec 3, 2017 7:11 PM, "Razah"<EMAIL_ADDRESS>wrote:

macOs Sierra
Issue description:
Godot connecting internet without asking me (Even if the firewall is
turned on)
there is no Connect/Disconnect options in godot (Offline Editor)
Steps to reproduce:
1.Godot blocked in Firewall
2.Godot is active in Network Activity Monitor(macos sierra)
Link to minimal example project:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/13936, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2zVvBpu8DL9PAPde_S75EsubNmFSks5s8xyIgaJpZM4Qz6oJ
.

Is this happening when idling or when previewing the game? Because the debugging happens over network protocol, so it might register network activity.
Also: you didn't mention the Godot version.

godot version -> http://godot3builds.digitecnology.com/builds/2017-12-02.13:00:36.5ddf288c7/godot-v3.0-osx.fat.zip
This is happening when AssetLib activity, maybe also debug i dont know. Godot bypassing sierra firewall. I want to use editor offline,(security) i am a paranoid.

AssetLib is stored externally, so when you use the assetlib you connect to the internet.

but we made it so unless you open the assetlib, it wont connect
On Mon, Dec 4, 2017 at 5:58 AM, Zireael07<EMAIL_ADDRESS>wrote:

AssetLib is stored externally, so when you use the assetlib you connect to
the internet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/13936#issuecomment-348899064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2wPtoWuqhxxnlN3_dp9_XWC_vtgUks5s87Q-gaJpZM4Qz6oJ
.

"Zireael07 : AssetLib is stored externally, so when you use the assetlib you connect to the internet."
In this case Assetlib becomes a trojan! godot is a virus.

Well feel free not to use the AssetLib.

@icnikerazah We took special care with the assetlib, if you dont use it it won't connect.

How can I be sure that Gdot can not copy harmful files to my computer? My firewall is ineffective.

@icnikerazah you have the full source code and there are thousands of users watching, I would think you are pretty safe. I mean, this is not a conspiracy to steal your files, we are making a game engine here.

i think godot should respect peoples's OS Firewall. This is not a professional approach. All Other Apps respect user preferences.

@icnikerazah what do you mean by "respect firewall"? I don't think macOS build in firewall is capable of blocking outgoing connections at all (at least there is no UI for it).

You are right, i was wrong. i am not a network expert. outgoing connections are safe.

@icnikerazah Godot uses incoming connection for debugging. Debugger is listening on port 6007 to accept connections form running projects.

Same applies to Windows default firewall as well.
By default debugger is listening on all interfaces, while this is not a security issue by itself, it is probably redundant, and firewall warnings may be frustrating to users.
This can be "fixed" by only listening to remote_host specified in editor settings (default to localhost, and shouldn't trigger firewall), but I'm not sure if this won't cause any problems with remote debugging.
May be remote_host should be changed to check list with all interfaces, with only localhost selected by default.
https://github.com/godotengine/godot/blob/master/editor/script_editor_debugger.cpp#L1156
String current = EditorSettings::get_singleton()->has_setting("network/debug/remote_host") ? EditorSettings::get_singleton()->get("network/debug/remote_host") : "*";
int remote_port = (int)EditorSettings::get_singleton()->get("network/debug/remote_port");
if (server->listen(remote_port, IP_Address(current)) != OK) {
	EditorNode::get_log()->add_message(String("Error listening on port ") + itos(remote_port), true);
	return;
}

May be remote_host should be changed to check list with all interfaces, with only localhost selected by default.

By default the editor listen on <IP_ADDRESS> i.e. localhost. You can change that setting in the Editor Settings (where you have the list of all interface), but the default is <IP_ADDRESS>

@Faless remote_host editor setting is passed to running project, editor always listen on all interfaces.
https://github.com/godotengine/godot/blob/master/editor/script_editor_debugger.cpp#L1156 listen(uint16_t p_port, const IP_Address &p_bind_address = IP_Address("*")) is called without address specified.

@bruvzg, you are right, my bad! I must have confused it with the client setting.
So I guess we could use the same option used for by the client to connect when run from the editor (which defaults to <IP_ADDRESS> and guarantee that the debugger client and server will always talk the same language).
https://github.com/godotengine/godot/blob/master/editor/editor_run.cpp#L44
And maybe add an additional boolean settings Listen on all interfaces.
Thoughts?


Savegame

Adds the ability to save the game

I have updated the pull request with the proposed changes @MortenTobiasNielsen