Update the end date when the start date is more recent

If i set the start date of a task to 8th december and the end date to 8th december as well, and then change the start date to 10th december, the start date will not be updated (well in that case it will just take the end date, which is the same) because it will me more recent than the end date. Wouldn't it be more convenient that both the start/end date are set to 10th december instead ?

you're right, that's probably the better way to handle that. Since should be an easy fix I'll add it to 1.1.8

Done


Proposal: Add HxTrigger

First and most important: Thank you for building the awesome plugin. I think the combination Spring + JTE +. htmx has huge potential!
I was wondering, what you would think about a @HxTrigger annotation, like htmx-spring-boot has included:
https://github.com/wimdeblauwe/htmx-spring-boot#response-headers
This is quite nice to have.

What do you want to do with HxTrigger? In my new project I am using htmx-spring-boot in combination with Spring ViewComponent and it pairs nicely.

Ah, I didn't know that you can pair them. I will give this a try. That would make the need for HxTriggers obsolet. I wanted to trigger a refresh for certain parts, when it's not directly related to the triggering change. A user adds a todo to the list, the list refreshes (inside a different component)
Do you already know when 0.7. will be ready?

0.7 is mostly done but I will change the template context still a bit as I want to only pass in the record to the template and not just the properties.
But you can use 0.7-SNAPSHOT already as the tests are already passing. You can look at the example folder for a usage of the new record syntax.

Yes, I am already on that branch and made my first test app. I like the
kte and the new record / template style.  I had the same thought about
passing just the record (data class) into the template.
Tonight I am going to try to add htmx-spring-boot.
On Thu, 14 Sept 2023 at 23:49, Thomas Schühly @.***>
wrote:

0.7 is mostly done but I will change the template context still a bit as I
want to only pass in the record to the template and not just the properties.
But you can use 0.7-SNAPSHOT already as the tests are already passing. You
can look at the example folder for a usage of the new record syntax.
—
Reply to this email directly, view it on GitHub
https://github.com/tschuehly/spring-view-component/issues/11#issuecomment-1720198977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANP2PEKDK6VRNEVPDILBPDX2N3W3ANCNFSM6AAAAAA4YS7V7A
.
You are receiving this because you authored the thread.Message ID:
@.***>

I just added the dependency (implementation("io.github.wimdeblauwe:htmx-spring-boot:3.0.0")) and get the following error:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'viewActionConfiguration' defined in URL [jar:file:/Users/felixeckhardt/.m2/repository/de/tschuehly/spring-view-component-core/0.7.0-SNAPSHOT/spring-view-component-core-0.7.0-SNAPSHOT.jar!/de/tschuehly/spring/viewcomponent/core/action/ViewActionConfiguration.class]: Unsatisfied dependency expressed through constructor parameter 1: Error creating bean with name 'requestMappingHandlerMapping' defined in class path resource [org/springframework/web/servlet/config/annotation/DelegatingWebMvcConfiguration.class]: Failed to instantiate [org.springframework.web.servlet.mvc.method.annotation.RequestMappingHandlerMapping]: Factory method 'requestMappingHandlerMapping' threw exception with message: No qualifying bean of type 'org.springframework.web.servlet.ViewResolver' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {@org.springframework.beans.factory.annotation.Qualifier("viewResolver")}

Maybe you have an idea.

Can you post an example repo?
I could not reproduce the error locally in the kte example: https://github.com/tschuehly/spring-view-component/tree/v0.7.0-SNAPSHOT/examples/kte-example

Here is my playground. It's a little e-mail address playground app. If you have any tips on how to improve it, please let me know:
https://github.com/don41382/htmxjetdemo
I compared your and my project. After removing @EnableWebMvc the exception on startup was gone. Thanks for the help.
I also added a few questions:

is it possible to use a @GetMapping inside a @ViewComponent - this would help to use e.g. @PathVariable.
is it possible to return an empty view content? Sometimes I don't need any context

I have to take a look at supporting @PathVariable with ViewActions.
I removed the EmptyContext as the template resolving is based the enclosing class of the data class/record

Hey there,
I removed the ViewActions from Spring ViewComponent in v0.8.0, as there is a better approach and the Annotation Processor added a lot of complexity.
The goal is to create a functional approach for HTMX Endpoints described here in an extension at a later date: https://github.com/wimdeblauwe/htmx-spring-boot/issues/104
btw, you can add @ViewComponent and @Controller on the same class.


ci: Add integration test for EKS

Ported from https://github.com/paradedb/helm-charts/pull/116

wrong repo, oops


Allow base64 string as dynamic import path / worker path

Description of the problem
Add the ability to use a base64 string in file paths for workers / dynamic imports, that loads that base64 string as code, instead of a file.
Should also be available for alt.evalModule
Desired solution for the problem
let result = alt.evalModule("data:text/javascript;base64,J2EnIDwgJ2In"); // "'a' < 'b'" as base64

Check if string starts with data:text/javascript;base64,, then the text after that is the actual base64 string.
Alternatives you considered
No response
Additional context
This is particularly useful for the usage with bundlers.
Version
No response
Scope
client

Type assertions


Opening notifications in "Read" section still marks them as read and removes them from the list.

Summary:
On tapping a notification in read section of notifications, the app behaves as if in unread section and marks the notifications as read, removing them from the list.
Steps to reproduce:

Go to read the section in the notification.
Tap on an item to view it.
Get back to see notification removed from read section.

What did you expect the app to do, and what did you see instead?
Notification should not be removed.
Device and Android version:
Nokia 6.1+ android 10 stock.
Commons app version:
Latest master.
Would you like to work on the issue?
Yes.

I can reproduce that on latest master indeed, good find!

Since you already have an issue assigned, please ask when you want to work on this one and we will assign you, thanks :-)

I can work on this one now.
I will start as soon as assigned ; )


Dynamically add markers onCameraMove event

Hello,
I have a request to dynamically load Markers from database based on CameraPosition. I am using the onCameraMove event of GoogleMap class.
So far, I have tried the StreamBuilder approach, but no success. I also tried the setState method, but no success at all.
When I use the StreamBuilder, I need to always create a new instance of GoogleMap, and this consumes a lot of resources (CPU usage is up to 90%). The problem is that when I add new markers into the Set, the GoogleMap object does not refresh automatically, so I need to always create a new instance of GoogleMap.
The SetState approach runs into the same problem. So, could you please provide me an example that help me to achieve my requirement?
Regards,
Rafael

@rafaelsimonassi
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

@rafaelsimonassi
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Pyodbc not finding driver

Environment

Python: 3.8.3
pyodbc:  4.0.30
OS: Windows 10
DB: ms Access
driver: {Microsoft Access Driver (*.mdb, *.accdb)}

Issue
Though the Microsoft Access Driver is installed, pyodbc can't seem to find it.  I'm not setup to debug in C, so I can't really track down the driver discovery function in pyodbc.
simple code:
import pyodbc

driver = '{Microsoft Access Database (*.mdb, *.accdb)}'
path = r'C:\Users\jmarshall\source\Bandz_Dev\Apps\ms tech support database.accdb'

conn = pyodbc.connect(driver=driver, dbq=path)
conn.close()

exception
Exception has occurred: InterfaceError       (note: full exception trace is shown but execution is paused at: _run_module_as_main)
('IM002', '[IM002] [Microsoft][ODBC Driver Manager] Data source name not found and no default driver specified (0) (SQLDriverConnect)')
  File "C:\Users\jmarshall\source\Bandz_Dev\test.py", line 6, in <module>
    conn = pyodbc.connect(driver=driver, dbq=path)
  File "C:\Users\jmarshall\AppData\Local\Programs\Python\Python38-32\Lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\jmarshall\AppData\Local\Programs\Python\Python38-32\Lib\runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "C:\Users\jmarshall\AppData\Local\Programs\Python\Python38-32\Lib\runpy.py", line 265, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "C:\Users\jmarshall\AppData\Local\Programs\Python\Python38-32\Lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\jmarshall\AppData\Local\Programs\Python\Python38-32\Lib\runpy.py", line 194, in _run_module_as_main (Current frame)
    return _run_code(code, main_globals, None,

proof of both the 32bit and 64bit drivers.

How do we run this down?

Resolved here. Please close.

This was a mistake on my part, can find solution in the SO link.


move to protonmail openpgp fork

Fixes #286
Signed-off-by: Bob Callaway<EMAIL_ADDRESS>

Looks like a dependency is there still from go crypto GPG to github.com/cavaliercoder/go-rpm


Arguments missing for Octopus Deploy tool

Octopus Deploy provides the tool Octo.exe, which is integrated into Cake under the Cake.Common.Tools.OctopusDeploy namespace. One feature that the tool provides is to deploy the latest release from a selected channel (using options --channel "Beta" --version latest).
The thing is that not every option is currently available through Cake, such as the channel option, so one has to resort to invoking Octo.exe directly. It would be nice if this option, and maybe others, could be added to OctopusDeployReleaseDeploymentSettings.

@islopma

so one has to resort to invoking Octo.exe directly.

Instead of resorting to calling Octo.exe directly you could always use ArgumentCustomization on OctopusDeployReleaseDeploymentSettings as a workaround.
That said. Are you in a position to submit a PR for adding Channel and Version/ReleaseNumber to OctopusDeployReleaseDeploymentSettings?

That's a nice workaround. I will test it and if it works I will be submitting a PR with the Channel option.

Fixed by #1590


[WIP] Implement Delete for OCI Layout

Attempt to resolve #454
Based on the implementation of #470, but with RWMutex.
DeletableStore is basically copied from oci.Store with an added lock.

Codecov Report

Merging #582 (b74100d) into main (0582b77) will decrease coverage by 0.72%.
The diff coverage is 51.26%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #582      +/-   ##
==========================================
- Coverage   74.21%   73.49%   -0.72%
==========================================
  Files          51       52       +1
  Lines        4832     5026     +194
==========================================
+ Hits         3586     3694     +108
- Misses        914      980      +66
- Partials      332      352      +20

Files Changed
Coverage Δ

internal/resolver/memory.go
90.00% <0.00%> (-10.00%)
:arrow_down:

content/oci/deletableOci.go
49.44% <49.44%> (ø)

content/oci/storage.go
63.52% <80.00%> (+1.88%)
:arrow_up:

... and 1 file with indirect coverage changes

Created graph.MemoryWithDelete, which is basically graph.Memory with a lock. I think there are lock problems with IndexAll, I need help/discussion on that.


Remove AbstractTask usage

Failing on Gradle 7.2 with:
Cannot create task ':downloadInParallel' of type 'AbstractTask' as this type is not supported for task registration.

after https://github.com/gradle/gradle/pull/12830

Hey, thanks for the fix.

Hey, thanks for the fix.

Thanks for quick release!


Add validation to services dropdown of educators

Now that we've implemented #307, educators can enter any string they like for the providing educator name when recording a new service.
Educator names come in from the Student Information System as "LastName, FirstName" and are stored that way in our database.
Would be nice to validate that that outside service provider names are entered the same way. This will make the data easier to work with if we need to surface or query it down the line.
This would include client-side validation that provided_by_educator_name includes the string ", ". It would also add some helpful hints to the user if they've entered a name that's doesn't match the format.

I'll see if I can do something for this!


[SPARK-19353][CORE] Generalize PipedRDD to use I/O formats

This commit allows to use arbitrary input and output formats when
streaming data to and from the piped process. The API uses
java.io.Data{Input,Output} for I/O, therefore all method operating
on multibyte primitives assume big-endian byte order.
The change is fully backward-compatible in terms of both public API
and behaviour. Additionally, existing line-based format is available
via TextInputWriter/TextOutputReader.

JFYI the failing scalastyle checks are unrelated to the PR.

Decided to keep this outside of Spark.


create sample fixtures of NDNP data

Descriptive Summary
In order to write tests for our NDNP batch ingest code, we will need fixtures to run the tests against. We should create a small mini batch of data in ./spec/fixtures/files/ndnp that includes enough data to successfully test the creation of titles, issues, pages, and attached derivatives.
Background
Batch ingest of NDNP-spec-compliant digitized materials is an important use case voiced by a number of stakeholders that we need to support.
This ticket will be complete when
There is a batch of data in  ./spec/fixtures/files/ndnp that includes files and directory structures matching LoC's data/batches format to successfully test the creation of titles, issues, pages, and attached derivatives.

We have decided to accomplish this by creating a separate gem to manage the fixtures. This decision was based on the fact that the fixtures may end up comprising >200MB, which isn't something that most people are going to want to add to their app.
By adding a separate fixtures gem as a :test and/or :dev dependency only, it will allow us to access the fixtures without forcing adopters to add all the files to their local installation.
newspaper_works_fixtures gem has been created.
This ticket will be complete when (updated)
NewspaperWorksFixtures gem has been added to NewspaperWorks as a development dependency in newspaper_works.gemspec...
spec.add_development_dependency 'newspaper_works_fixtures', git: 'https://github.com/marriott-library/newspaper_works_fixtures'

... and we have verified that the fixture paths are accessible, and the changes have passed in CI.

Closed via #62.


Improve diagnostics for failed Standard Library Tests

Currently, when standard library test fails it is mentioned in a summary but with no specific information.
This leads to output like:
[SQLite File] Schemas and Databases: [3/3, 7ms]
    - should be able to get current database and list databases [1ms]
    - should be able to get current schema and list schemas [1ms]
    - does not allow changing schema or database [4ms]
[FAILED] [SQLite File] Tables and Table Types: [3/4, 211ms]
    - should be able to list table types [39ms]
    - should be able to list tables [121ms]
    - [FAILED] should be able to filter tables by name [40ms]
        Reason: 2 did not equal 1 (at C:\runner\_work\enso\enso\test\Table_Tests\src\Database\SQLite_Spec.enso:139:13-76).
    - should be able to filter tables by type [9ms]
[SQLite File] Error Handling: [2/2, 54ms]
    - should wrap errors [18ms]
    - is capable of handling weird tables [36ms]

(…almost 3k lines of logs here…)

Value_Type: [2/2, 11ms]
    - should have a nice display text representation [3ms]
    - should use correct in-memory logic to reconcile pairs of types for operations like union/iif [8ms]
4312 tests succeeded.
1 tests failed.
121 tests skipped.
25 groups skipped.

This is difficult to read and understand — at the end of failing command there is just information that 1 test failed, but no information about which test it was. The only way to find out is to scroll through the endless stream of logs and look for the failure. (Well, one could search for [failed] but this is not obvious.)
I suggest that:

the summary should contain information about which test failed;
this information should contain some string that can be used to search for the failure details in the logs.

I think the idea is indeed worth considering.
For now, we have an environment variable REPORT_ONLY_FAILED which if set, makes only failed tests printed in the report.
The downside of that is that we can't track progress of which tests are being executed on the CI - the process would go silent for a long time, and if all tests succeed not print anything. IMO that is not a good default for CI.
Also please note that the generated HTML test report based on the JUnit.xml files (example) does contain a summary of failed tests at the bottom - I usually use that and it's been working quite well for me.
Not trying to discourage improvements to the test lib, I think they could be valuable.

Test stability much better these days!

It feels like this was actually handled by #9986


Kobalte styling is not working

https://github.com/zirbest/unocss-preset-primitives/blob/bef9fd1b130c7c4855acb19b5792236b2e783762/src/index.ts#L37-L39
I am newbie to Unocss, but only if I replace the ~=' with a simple - makes it work with Kobalte Componets.
 selector: (s: any) =>
              match[1] === "-not"
                ? `${s}[${selector}]:not([${selector}-${match[2]}])`
                : `${s}[${selector}-${match[2]}]`,

thnx @andrewdoro
fixed in v0.0.1-beta.5


Add rubocop-rails and rubocop-rspec

What's changed basically?
Just added rubocop-rails and rubocop-rspec
I use rubocop-rspec first time in my life, so I want to talk about its configuration with you.
Let's talk about .rubocop.yml according to rubocop-rspec gem. I added comments to every point of rubocop-rspec configuration.
# One exception tests sounds good, but in practise it becomes a huge problem. Test examples run too many times and the whole test process becomes slow.
# Moreover, remember about `and` function that we can use. `expect(page).to have_css(css_1).and have_css(css_2)`
RSpec/MultipleExpectations:
  Max: 3

# Let's see at this file https://github.com/Purple-Magic/tramway/blob/main/spec/helpers/navbar_helper_spec.rb
# 3 nested groups is not enough, cause we have levels `className` -> `functionName` -> `successOrFailure` -> `certain_case`. It should be 4 at least
RSpec/NestedGroups:
  Max: 4

# According to expectations count. Let's imagine that we need 3 expectations in one test and also we need one line to make `service.call` or stuff like that. 3 * 3 + 1 = 10
RSpec/ExampleLength:
  Max: 10

# I love this kind of expression `expect(form_class).to receive(:new).with(object).and_return(form_object)`, because it's short
RSpec/StubbedMock:
  Enabled: false

I don't see much harm in this. A lesser value of RSpec/ExampleLength and RSpec/NestedGroups could force you to make more helpers and e.g. example groups, which could increase the readability of the code.
But I'm not a big fan of too small tests either, can't comment on that with certainty.

I don't see much harm in this. A lesser value of RSpec/ExampleLength and RSpec/NestedGroups could force you to make more helpers and e.g. example groups, which could increase the readability of the code. But I'm not a big fan of too small tests either, can't comment on that with certainty.

@haukot what about helpers and that kind of stuff, there are other cops that make us use them. So, example is used only for the purpose I described in the comment in the config file.

The forcing of described_class is great, instance_double and class_double instead of double are good too. Not sure about subject replacing.
Other changes I don't see as important

@haukot I thought about subject replacing, looks like subject without a name should be used in cases of using it_expected methods only. Otherwise, the subject should have a name.


Blank space above integrated terminal when using decimal value for window.zoomLevel

Issue Type: Bug
Steps to reproduce:

Install the Ubuntu font family (download link is at the bottom of the page).
Start vanilla session of VS Code Insiders.
Open Command Palette, enter Preferences: Open Settings (JSON), paste the following settings, and save:

{
    "editor.fontFamily": "Ubuntu Mono",
    "telemetry.enableTelemetry": false,
    "telemetry.enableCrashReporter": false,
    "terminal.integrated.fontSize": 16,
    "window.zoomLevel": 0.4
}

Open Command Palette and enter View: Toggle Terminal.
Drag the Terminal tab to the Activity Bar and click on it.

Actual result: There is blank space above the terminal contents (you can see the effect better when filling the terminal by repeatedly pressing Enter and examining the size of the scrollbar; there is blank space above the scrollbar slider even when scrolled all the way to the top).
Expected result: There is little to no space above the terminal contents (the height of the blank space should not exceed the height of one terminal line).
Additional information:

The blank space vanishes when using 0.3 or 0.5 for window.zoomLevel.
The developer tools indicate that the terminal-split-pane div has the correct height, but the height of the xterm-screen div is too small.
Due to firewall restrictions, I'm currently not able to provide screenshots.

VS Code version: Code - Insiders 1.60.0-insider (f0913d7718bbc4a805cd3a051335b4df60ca5d77, 2021-08-17T05:15:18.727Z)
OS version: Windows_NT x64 10.0.18363
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.59GB (15.51GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Here are some screenshots, taken on a different computer with regular VS Code, not Insiders. In the first one, the bug occurs; in the second one, it doesn't due to a different window.zoomLevel. I'm not able to always trigger it, it also seems to depend on the window size and the previous settings. If you can't reproduce it right away, try a different window.zoomLevel and/or resize the window.

Thanks for all of the info

@meganrogge Did you perform step 1 of the repro steps correctly? The terminal in your GIF doesn't use Ubuntu Mono as configured, but some other font.
(The editor should be in the same font, but it's Times New Roman or something similar, which strongly suggests that Ubuntu Mono has not been installed correctly.)

Happened to me on linux. Changing console font size fixed the issue. I believe its because of the font family being used and availables sizes and miss calculation on terminal cols and rows.

Then changing font size:

I've got this peculiar behavior with the integrated terminal on macOS with extensions disabled and default settings. In my screencast, you can see how adjusting the "window.zoomLevel" setting impacts the positioning of the first line in the terminal either raising or lowering it. I guess this occurs when triggering a re-render at specific zoom levels.
Interestingly, I've pinpointed the minimum positive number that triggers this behavior in my setup. It appears that any slight increase in the zoom level, enough to exceed a particular threshold in 64-bit floating point representation, can cause this issue. This smallest triggering value in my tests is 0.000000326920437485165902734277260967277189962487682350911200046539306640625 which is<PHONE_NUMBER>010101111100000111001101001010000111000000100000010010, so any number at least one bit greater starting with ...011 causes the padding shift.
Additionally, adjusting the font size also introduces a noticeable top padding either smaller or bigger, though I found the changes due to invisible zoom level adjustments more striking.
https://github.com/microsoft/vscode/assets/41355752/0826f734-5d16-42fc-ad05-5b4ecf20b6d7

Yep, pretty much the same thing I'm experiencing. My "workaround" was to set a zoom level and font size compatible with my fixed window size and live with it. But that is the same thing as IM experiencing for sure.

I got the same problem when I set window.zoomLevel to 1, or terminal.integrated.lineHeight to 1.2. And finally I get it right when I set zoomLevel to 1.05 and terminal.integrated.lineHeight to default 1.

Yep, its a mix of setting zoomlevel and fontsize and praying.


Revert "fix(ci): disable windows build (#843)"

This reverts commit 5708df9d3c3a2315cd8cd8b00362746c3e44dca6.

Motivation

Solution

We disabled this because unfortunately a CI test kept failing and our automated release process kept going OOM...


[PE-D][Tester E] Standardize error message produced by addavail command, DD/MM/yyyy formatting

To replicate:
addavail 1 a/
In the System when addavail 1 a/, error shown as "Availabilities must be in the format of dd/MM/yyyy" but in UserGuide it state DD/MM/YYYY

Labels: type.DocumentationBug severity.VeryLow
original: zoebelle-pang/ped#5

actually, I feel this is quite invalid since the help message in our app and UG do not need to be the same


[Question] Inverse of Get/Select?

I find it amazingly useful to just call Get or Select instead of having to recursively loop through my struct fields and assign to them myself.
But it's equally annoying to loop through them to build an INSERT or UPDATE.
Is it feasible to create functions that does this in sqlx? If you think so, please give me advice about how you think it ought to work and I'll give it a shot.
Another useful thing along the same lines would be something that created a CREATE TABLE statement for a struct, possibly with CREATE INDEX statements where needed. Would that be feasible and valuable?

Implicit field names for insert statements sounds nice, but creating tables from structs sounds like an ORM feature.
Also creating such function(that creates CREATE TABLE statements) requires some way of setting all the field properties that user wants per field or just setting some defaults and expect users to modify the statement, or expect them to modify table fields in another query(ALTER TABLE or st).
It all seems like a lot of trouble to automate something that people usually need to customize for themselves in my opinion.

So you might be interested in the insert statement feature?
Do you have any particular thoughts on that or should I just prepare a PR
and send you?
On Wed, Feb 28, 2024, 06:51 TLW @.***> wrote:

Implicit field names for insert statements sounds nice, but creating
tables from structs sounds like an ORM feature.
Also creating such function(that creates CREATE TABLE statements) requires
some way of setting all the field properties that user wants per field or
just setting some defaults and expect users to modify the statement, or
expect them to modify table fields in another query(ALTER TABLE or st).
It all seems like a lot of trouble to automate something that people
usually need to customize for themselves in my opinion.
—
Reply to this email directly, view it on GitHub
https://github.com/jmoiron/sqlx/issues/912#issuecomment-1968282313, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB25MG7BAQPLD66NLWMFLYV3AVNAVCNFSM6AAAAABDCQS5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGI4DEMZRGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

So you might be interested in the insert statement feature? Do you have any particular thoughts on that or should I just prepare a PR and send you?
…
On Wed, Feb 28, 2024, 06:51 TLW @.> wrote: Implicit field names for insert statements sounds nice, but creating tables from structs sounds like an ORM feature. Also creating such function(that creates CREATE TABLE statements) requires some way of setting all the field properties that user wants per field or just setting some defaults and expect users to modify the statement, or expect them to modify table fields in another query(ALTER TABLE or st). It all seems like a lot of trouble to automate something that people usually need to customize for themselves in my opinion. — Reply to this email directly, view it on GitHub <#912 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAB25MG7BAQPLD66NLWMFLYV3AVNAVCNFSM6AAAAABDCQS5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGI4DEMZRGM . You are receiving this because you authored the thread.Message ID: @.>

not a maintainer... just wanted to weigh in 🙃

Fair enough :)
On Wed, Feb 28, 2024, 12:23 TLW @.***> wrote:

So you might be interested in the insert statement feature? Do you have
any particular thoughts on that or should I just prepare a PR and send you?
… <#m_-117507800482917100_>
On Wed, Feb 28, 2024, 06:51 TLW @.> wrote: Implicit field names for
insert statements sounds nice, but creating tables from structs sounds like
an ORM feature. Also creating such function(that creates CREATE TABLE
statements) requires some way of setting all the field properties that user
wants per field or just setting some defaults and expect users to modify
the statement, or expect them to modify table fields in another query(ALTER
TABLE or st). It all seems like a lot of trouble to automate something that
people usually need to customize for themselves in my opinion. — Reply to
this email directly, view it on GitHub <#912 (comment)
https://github.com/jmoiron/sqlx/issues/912#issuecomment-1968282313>, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB25MG7BAQPLD66NLWMFLYV3AVNAVCNFSM6AAAAABDCQS5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGI4DEMZRGM
https://github.com/notifications/unsubscribe-auth/AAAB25MG7BAQPLD66NLWMFLYV3AVNAVCNFSM6AAAAABDCQS5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYGI4DEMZRGM
. You are receiving this because you authored the thread.Message ID: @.>
not a maintainer... just wanted to weigh in 🙃
—
Reply to this email directly, view it on GitHub
https://github.com/jmoiron/sqlx/issues/912#issuecomment-1968775058, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAB25PYBGFROU5TZBCDTX3YV4HSTAVCNFSM6AAAAABDCQS5BSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRYG43TKMBVHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


sharding-jdbc support mariaDB

Is there a plan to support mariaDB for Sharding-jdbc?

duplicate with #2701 2705


[Bug] Unable to attach or mount volumes: unmounted volumes=[configmap-ocp-injected-certs]

Describe the bug
grafana-deployment pod is stuck in Init stage.
Version
Grafana version - v3.10.4
Openshift version - 4.7
To Reproduce
Steps to reproduce the behavior:

kubectl create namespace grafana
kubectl create -f deploy/crds
kubectl create -f deploy/roles -n grafana
kubectl create -f deploy/cluster_roles
kubectl create -f deploy/operator.yaml -n grafana
kubectl create -f deploy/examples/persistentvolume/Grafana-OpenShift.yaml -n grafana

Here the issue is the deployment gets stuck a at init state with detailed error as shown in the screenshot attached.

@rakeshk121 is there any reason why you can't try v4? I assume this is your first time using the operator and we are trying to focus on v4 as much as possible.

@NissesSenap thanks for the response.  I have tried the v4 but it did not work for me.
# I cloned the master (which will have v4)
# I have tried to deploy Grafana by  referring to documentation -> https://github.com/grafana-operator/grafana-operator/blob/master/documentation/deploy_grafana.md
# The following is the error im facing
[kubenode@master grafana-operator]$ kubectl apply -k deploy/manifests/
error: rawResources failed to read Resources: Load from path latest failed: 'latest' must be a file (got d='/home/kubenode/rakesh/grafana/grafana-operator/deploy/manifests/latest')

Is the issue due to incompatibility with openshift- 4.7.0 ? Please help.

If you are on OCP you can use OLM so you should just be able to use the UI if you want to install the operator.
I updated my master branch today and was able to deploy the operator using kubectl apply -k deploy/manifests/.
What version of kubectl are you using? Personally I'm using:
k version --client
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.4", GitCommit:"b695d79d4f967c403a96986f1750a35eb75e75f1", GitTreeState:"archive", BuildDate:"2021-11-18T21:20:32Z", GoVersion:"go1.17.3", Compiler:"gc", Platform:"linux/amd64"

You can also use the kustomize cli as mentioned in the docs.
But since you are on OCP I would use OLM.

@NissesSenap Im using openshift - 4.7 which has k8s version  as follows:
[kubenode@master ~]$ kubectl version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.1-5-g76a04fc", GitCommit:"95881af", GitTreeState:"clean", BuildDate:"2021-04-26T16:57:42Z", GoVersion:"go1.15.7", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20+", GitVersion:"v1.20.0-1073+df9c8387b2dc23-dirty", GitCommit:"df9c8387b2dc2308da01706ff89982c6b7ad9c48", GitTreeState:"dirty", BuildDate:"2021-05-15T22:42:25Z", GoVersion:"go1.15.7", Compiler:"gc", Platform:"linux/amd64"}

Does k8s 1.20 support the grafana operator 4.1 ?
I have as well tried to deploy the Grafana operator from the UI of openshift.
Version of grafana operator - v4.1 .
But the Grafana operator pod fails with 'CreateContainerError' as attached in the screenshot below.

Yes the grafana-operator is supported in kubernetes 1.20.
Well this is really interesting, i think you found a very special bug.
https://bugzilla.redhat.com/show_bug.cgi?id=1934177
A well written explication can be found here:
https://danielmangum.com/posts/runc-chdir-to-cwd/
I would advice you too update your cluster to a newer OCP version where this is fixed. This is probably not the only container where you might hit this issue.
We could try to implement a solution at our side as well but can't give a estimate when that would happen.


GIFs not displaying for Android devices

Whenever I open the gallery of Android devices, this plugin ignores all GIF files, so I don't see any option for select GIF.

@baronha Please give any solution. How can i resolve it this problem ?

@baronha Please give any solution. How can i resolve it this problem ?
I will check the native library on the Android side to see if this can be resolved.

@baronha Please give any solution. How can i resolve it this problem ?

I will check the native library on the Android side to see if this can be resolved.

Hi, can help to solve this issue?

Check new version: 2.0
Docs: https://baronha.github.io/react-native-multiple-image-picker/


Bug: (new URL(...)).searchParams.get(...) is null when calling maptilersdk.geocoding.byId(...)

Everything is in the title.
This is because when calling byId() function, the apiKey is not set in the URL that is passed to the callFetch function. But the callFetch function requires the api key to be set already.

Hey @toniopelo , thanks for reporting this! I'll try to fix that quick!

@jonathanlurie Hi! I've already published a PR linked to this issue, you can see it mentioned just above your comment. It should fix the problem. You might want to do it another way but it's a very minimal PR you could merge without too much effort. :)


Remediating RBAC Roles on Assignments

Overview/Summary

Remediating default policy/initiative assignments using Owner role to be least privilege where possible. Updated assignments:

DINE-ActivityLogPolicyAssignment.json (Management Group: Intermediate Root)
DINE-AksPolicyPolicyAssignment.json (added additional required role)
DINE-ResourceDiagnosticsPolicyAssignment.json (Management Group: Intermediate Root)
DINE-SQLEncryptionPolicyAssignment.json (Management Group: Landing Zone)
DINE-VMBackupPolicyAssignment.json (Management Group: Landing Zone)
DINE-VMMonitoringPolicyAssignment.json (Management Group: Intermediate Root)
DINE-VMSSMonitoringPolicyAssignment.json (Management Group: Intermediate Root)

Addresses user story: AB#30459
This PR fixes/adds/changes/removes

Updates default assignments listed above

Breaking Changes

N/A

Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
Testing URLs
Azure Public

As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Ensured contribution guidance is followed.
[X] Updated relevant and associated documentation.
[X] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

GTM


Duplicate comment when converting back from document

The yaml will contain duplicate comment when converting the parsed document by "toString"
Example:
test1:
  foo: #123
    bar: 1

After parseDocuments and doc.toString()
test1:
  foo: #123
    #123
    bar: 1

This should be fixed by 169a07e (which incorrectly refers to issue #17, oops).
The source of this bug was pretty deep: The context of an AST node contains an undocumented reference to its parent node, which is used at a couple of stages internally. For a block collection, its context was previously shared with its first item as it was functionally equivalent from the AST parser's point of view. However, during the object parse stage this ended up meaning that a map within a map with a comment before the first item had that comment parsed not only for the collection object, but also for the first item, due to a context.parent reference that was incorrect for the first item.
As a fix, the first-item and collection contexts are now separate objects, as they should be.


Doc dbg 2

Merging this by hand to try and see if we can understand why we are getting a failure.  The CI checks don't really have any impact on this change.


Design

We need to list basic features for MVP and sketch app design.

MVP milestone https://github.com/KSIUJ/mordor/milestone/1
Closing this issue as the app is already in development.


feat: Expo SDK 48, react-native 0.71.3

Please verify the following:

[ ] yarn test jest tests pass with new tests, if relevant
[ ] README.md has been updated with your changes, if relevant

Describe your PR
Upgrades Ignite to Expo SDK 48 and React Native 0.71.3

[x] Vanilla iOS
[x] Vanilla Android
[x] Expo iOS
[x] Expo Android
[ ] Expo Web

Currently experiencing issues with Expo Web, related Expo issue here: https://github.com/expo/expo/issues/21276
yarn expo:web

WARNING in<EMAIL_ADDRESS>Module Warning (from ./node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>

@frankcalise  in the actual version of detox 19.12.1 , in the package json , we see the version of jest in the peer dependencies , I think upgrading detox is the solution to use jest 29

Small typo int the issue title - says 72.3 not 71.3

@frankcalise Expo fixed the Can't resolve '@react-native/assets-registry/registry' webpack warning from expo-asset in https://github.com/expo/expo/pull/21559.
yarn upgrade expo
npx expo install --fix

@frankcalise Expo fixed the Can't resolve '@react-native/assets-registry/registry' webpack warning from expo-asset in expo/expo#21559.
yarn upgrade expo
npx expo install --fix

Thanks will take a look!!

@frankcalise Expo fixed the Can't resolve '@react-native/assets-registry/registry' webpack warning from expo-asset in expo/expo#21559.
yarn upgrade expo
npx expo install --fix

Have you run into this error lately?

TypeError: (0 , _nonSecure.nanoid) is not a function

I did encounter that but can't remember which library it was, can you share the stacktrace or console log? There should be a hint of who's using nanoid. I think it was an issue with how it was imported

@frankcalise Ok I remembered now, it was an issue with react-navigation. If you update it should not throw that error.
For reference I think it was fixed here

This is now live for use under npx<EMAIL_ADDRESS>new or npx ignite-cli@beta new 🥳

:tada: This PR is included in version 8.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CLICKHOUSE-2720: some progress on website

Can one of the admins verify this patch?

file:///home/vludv/ClickHouse/docs/build/html/_static/logo.svg net::ERR_FILE_NOT_FOUND


Build_all.sh fails

Using Ubuntu 18.04, it downloaded a lot of things then crashes with:
...
usr/share/man/man1/python3.1 -> python3.8.1
usr/share/man/man1/python3.8.1

sent 326.07M bytes  received 170.24K bytes  217.49M bytes/sec
total size is 325.30M  speedup is 1.00
Installing for i386-pc platform.
grub-install: error: hostdisk//dev/loop0 appears to contain a squash4 filesystem which isn't known to reserve space for DOS-style boot.  Installing GRUB there could result in FILESYSTEM DESTRUCTION if valuable data is overwritten by grub-setup (--skip-fs-probe disables this check, use at your own risk).
umount: /mnt: not mounted.
loop deleted : /dev/loop25

It failed because there is already something mounted for /dev/loop0. If you look at the last line, kpartx actually created a loop device for the disk image at /dev/loop25, but the script assumes /dev/loop0.
I recommend against trying to build your own image for now, we are going to be switching to buildroot for the build system and then it will work much better on people's systems that aren't mine.

Ok! Makes sense!


Unexpected behavior when combining different overloads of IQueryProvider<>.Where()

This works as expected:
MyDatabase.Query<MyPoco>()
    .Where(p => p.MyProperty == myValue)
    .Where("EXISTS (SELECT * FROM myTable)")
    .ToList();

But when I reverse the order of the two where clauses like this,
MyDatabase.Query<MyPoco>()
    .Where("EXISTS (SELECT * FROM myTable)")
    .Where(p => p.MyProperty == myValue)
    .ToList();

the condition from the string overload ("EXISTS ...") is completely missing from the generated SQL. This is definitely not what I would expect.
This is with NPoco v3.9.0, but we also discovered this behavior with older versions.

yeah looks like its a bug.
Will try and have a look over the weekend, otherwise it will be monday.

Ahhh screw it. Fixed.

That was quick! :) Thank you!


[IDE] Database - Queries work only after switching database

Describe the bug
Executing queries in the database perspective works only after switching between dynamic/custom dabatase with HANA datasource.

What version of the XSK are you using?
0.10.0-SNAPSHOT

To Reproduce
Steps to reproduce the behavior:

Run XSK connected to HANA Cloud
Make sure you have a schema and a table (ie MYSCHEMA.MYTABLE)
Open Database perspective and write SELECT * FROM MYSCHEMA.MYTABLE
See error table not found
Switch database from custom to dynamic and back to custom
Execute query again and see that this time it's working file

Expected behavior
Query works fine on a freshly opened database perspective.
If I have to configure something before being able to execute queries this needs to be clearly indicated and the query panel needs to be disabled

Progress to be tracked in:

https://github.com/eclipse/dirigible/issues/1213

@dpanayotov please transfer this issue to: https://github.com/eclipse/dirigible/issues

This issue is closed as the project is no longer maintained


The same blocks are assigned to the same container

Reported by @jdoliner:
The way Pachyderm works right now, in a map job, the same blocks are always assigned to the same container, due to the fact that we simply use the hash of a block's name to determine which container gets it.
Now imagine if you have a repo where 90% of the files have the same content (i.e. they consist of the same blocks).  Now if you run a map job on this repo, 90% of the data will be sent to one container, no matter how many parallel containers you are running.
So the solution here is probably to hash filename + blockref when determining which container gets a particular block.

@derekchiang this is fixed now right since BlockInShard takes a filename?

Yes.


Autofocus Edit Column Title

Feature Request
When editing a column title, the current text should be highlighted by default with the cursor active so a user can just start typing the changes as desired.
Story
As a facilitator of a retrosepective
I want the current text to be highlighted by default when I click the pencil
So that I can easily edit the column names without getting frustrated
Acceptance Criteria
Given the facilitator is setting up a retro board
When they click the pencil on a column name
Then the text should be focused and highlighted by default

The changes related to this issue have been migrated to PROD.


Focus the editor when clicking the Draft window

When you click somewhere on the gutter (left or right) of a document, it should place the cursor in the nearest block
When you click on the bottom/empty part of a short document, it should focus the last block

This is still an issue with the new editor. Maybe @iskaktoltay can look into this

This is fixed now!


Merge and Update Remove and Delete patterns

Removed the Remove pattern. 😆
Combined it into the delete pattern.
Made some updates to both.
Replaced a few images to keep it inline with changes that happened in the reusable component development.
Moved a few bits and pieces into the anatomy section as they're optional/options to choose from.

@alisonjoseph it looks to me like the description is in the frontmatter at the moment but not 100% sure (I'm a bit new to contributing 😂)

Hey again @alisonjoseph , finally figured out how to update the nav.
Should all be good now.

Hi @connor-leech & @alisonjoseph I can't see this update on the carbon site, do you know when it will be up (or is there something we need to do?) Thanks

@vikkipaterson the updates get auto-deployed to the website ~15-20 min after a PR is merged, so the updates should be up there once this gets merged in.

@vikkipaterson @alisonjoseph The anchor links weren't consistent with the page headers which resulted in them breaking. I have some suggestions that are pending before I approve/merge

Hi @connor-leech I've re-phrased the "remove" subtitles to be in the same format as the deletion ones and fixed the anchor links - hopefully that'll work!


Add author to RecordsFilter

We already index the author property for RecordsWrite but we currently don't expose it as a filter.

@LiranCohen Shouldn't we add tests that use this new filter property?

@frankhinek yup! Missed that.


sqlx does not compile with pc-windows-gnu and mingw-64

Hello, I have tried to compile an empty project with sqlx version 0.4.1 as a dependency, using rust stable 1.47.0 x86_64-pc-windows-gnu alongside mingw-w64 (8.1.0, seh), and windows detects a crash in the compiler when sqlx is being compiled (slqx-macros and sqlx-core pass without issues). It is worth noting that the crash occurs only if any of the databases is selected as a feature (i.e., just selecting runtime-actix-native-tls compiles fine, but when I add either sqlite or postgres, it crashes). I would like to investigate more the issue but Windows just sends me the "A problem caused the program rustc.exe to stop working correctly" message, and in the terminal it just says "could not exit successfully", with error code 255. Just in case, I use a self-compiled sqlite3 library, from version 3.33.0 (amalgamation code) which compiles fine with libsqlite3-sys.

As a quick update, compiling sqlx with either rust stable 1.48.0, or with a nightly build, does not crash but fails to compile. The error is related to mingw's linker, ld.exe.
 = note: C:/mingw-w64/x86_64-8.1.0-posix-seh-rt_v6-rev0/mingw64/x86_64-w64-mingw32/bin/ld.exe: Error: export ordinal too large: 67748

So now it seems a problem with Mingw

Duplicate of #615


Add KubeCon 2021 banner, April 30 - May 7 2021

Signed-off-by: Celeste Horgan<EMAIL_ADDRESS>Adds a banner for KubeCon 2021 to help with registration. This also adds the /static/announcements directory, so that any images that go with announcements can be kept somewhere tidy :)
cc: @kubernetes/steering-committee for approval.

/lgtm

@celestehorgan . The banner looks great!
Aside, the banner sub-text on the "Partners" and "CaseStudies" pages is styled differently.

Page preview:
https://deploy-preview-27844--kubernetes-io-master-staging.netlify.app/blog/
/approve

Whoops, forgot this content requires special approval.

@kbhawkey – @liggitt's already signed off, so I think you're good to merge!

/approve
👍


Signature newlines Windows vs Unix

There is an issue when attempting to sign a file on windows due to newlines. Windows uses \r\n and Unix uses \n. When cloning a repository it seems like git converts newlines to the Windows format automatically depending on your git configuration, which means signature created on Windows may not match the actual file pushed to your git repository.
cc @donny-dont

https://help.github.com/articles/dealing-with-line-endings/ for reference

I am also facing this problem, but I'm unsure what the solution should be. I'm thinking the easiest way to deal with it would be to try to deduce the autocrlf option, but that seems obtrusive. Perhaps a more robust, and less obtrusive, way to deal with it would be to always convert crlf sequences to lf sequences for purposes of signing, thus making this a non-issue - but that might break existing signatures, if users use a non-default autocrlf option. Thoughts?

I thinking stripping \r with bytes.Replace is probably fine. This is what Go itself does to handle \r\n and \n discrepancies.

Would it also make sense to trim \n if it's the last character? Some text editors forcibly add it as a result of the different semantic ideas people have about line termination (you know, whether the last line requires a terminator.)
Either way, if you want, I can make a pull request for this.

I think dropping \r should be fine

signatures gone in upcoming release. closing! :)


add font_size zero to doc

I was going to add the ability to suppress text, but this package already has the functionality.
The special case of font_size 0 should be mentioned in the docs.

Right, thanks!


L96 GitHub working patch to rxjs 6

What are you adding/fixing?

Attempting to make it compatible with rxjs version 6, mostly using rxjs-compat.
Have you added tests for your changes?

Not yet. Unfortunately didn't have enough time.
Will this need documentation changes?

No, although the changes are on how rxjs imports are structured and the new ways of using its components (e.g. combineLatest, Observable, Subject, map, etc). There is plenty of docs describing the changes. (https://auth0.com/blog/whats-new-in-rxjs-6/), but that doesn't directly concern this package
Does this introduce a breaking change?

So far none apparent. I am currently using it in a project almost in production.
Other information
I ran the example with yarn example and tested it, had no errors on the console. Had to update the code to support the braking changes of rxjs 6, other than a few warnings (as you can see in the screenshots) I have not come across any problem. I am also testing as much as I can on the project in which I am using it on, hope it helps.

would be great to have support for RxJS 6. Currently can't use it in Angular 7. Anyone know of a workaround ? thanks

If you want: https://www.npmjs.com/package/@hackages/ngxerrors


feat: XSB authentication cleanup

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 8 committers have signed the CLA.:white_check_mark: PreethiMaai:white_check_mark: bpvstaty366:white_check_mark: Abhishk123:white_check_mark: Nandana-NNR:white_check_mark: anjalimukundan:white_check_mark: Roebot926:x: jbigel:x: semantic-release-botjbigel seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: revamp cards design on contributors page 🪄✨

Close #1307
Description

Revamped the card design on contributors page, removed unwanted gradients during hover.
Cards were revamped as per the OS-CODE Theme color, so our site will have a consistent color patterns around the web which improved the overall UI & UX.

Screenshots

B E F O R E
A F T E R

Checklist:

[x] I have linked the PR to the correct issue.
[x] I have read the Contribution Guidelines
[x] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 the cards has been revamped, please merge this pr update the code on main branch.
Thank you!


Implement unit tests for Emailer & EmailMessage

These were previously ignored. Some refactoring was needed to inject dependencies.
Builds on https://github.com/azkaban/azkaban/pull/1663 (1st commit is the same) – for the actual scope of this PR, see the 2nd commit only: https://github.com/azkaban/azkaban/pull/1664/commits/972e43e2747cd2e0f95b4a4c34ea3a7d7c3beec3

thanks! If i understand your change description correctly, https://github.com/azkaban/azkaban/pull/1663 is a subset of this PR? so we will only need to review  https://github.com/azkaban/azkaban/pull/1663?

I think it’s much easier to grasp if #1663 is dealt with first, separately. Usually at least @HappyRay has preferred splitting changes to individual pull requests to make it easier to follow.

Thanks @chengren311 for the review!
@juhoautio I am sorry for the delay in reviewing your changes. I have been assigned some high priority work that I may not be available to do reviews for a while.

Replaced by https://github.com/azkaban/azkaban/pull/1691


Accuracy for Object detection

🚀 Feature
Accuracy for Object detection
Motivation
I am doing a lot object detection projects and I was thinking it would be a bit easier if there was Accuracy because I am more used to be using accuracy
Pitch
it is to have a Accuracy for Object detection
Alternatives
I couldn't find any
Additional context
None
With best regards,
Ranuga

Hi @Programmer-RD-AI,
Could you please explain how this accuracy for object detection would work differently from the accuracy metric that is already part of torchmetrics?

Hi, I think this is a good idea. It's great that mAP exists now since it's the most common metric for object detection.
I think it would be valuable to also include precision, recall, f1, true positive, false positive, and false negative (true negative doesn't make sense to me in a detection setting). Here's why

The suggested metrics are more common and easier to understand. This is great for people with no previous experience in object detection.
Some models don't output bounding boxes with confidence scores. For example in text detection it's common to use segmentation models that don't do that.
Many models filter their predictions with a confidence threshold. Remaining boxes are often treated "equally" in that they produce a signal to the system, no matter if the confidence is 0.99 or 0.7. These systems would better be evaluated via mentioned metrics.

The calculations needed for this would be very similar to the mAP detection metric.


Sound doesn't play on iPad

The audio doesn't play on my iPad.
Model: iPad Air (2021)

have you tried tapping once on the screen
it doesn't play on some devices due to overload of videos and audios at once.

have you tried tapping once on the screen
it doesn't play on some devices due to overload of videos and audios at once.

Will try.

Nope, still doesn't work.

Nope, still doesn't work.

did you try with the latest update??

Nope, still doesn't work.

did you try with the latest update??

I'll try later.

Works now.

Works now.

yeah it was cuz of overload
I reduced video size to 50 from 200


Support multiple drivers

This package, which is confusingly named rabbit, only supports the Redis driver. It would be much more useful if other drivers were also supported (like, for instance RabbitMQ).

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


What is the meaning for the benchmark output tokens_per_sec and generation_tokens_per_second?

I run benchmark like this:
mpirun -n 2 --allow-run-as-root python benchmark.py \
    -m llama_13b \
    --mode plugin \
    --batch_size "1;8;16" \
    --input_output_len "710,190" \
    --max_input_len 750 --max_output_len 200

I got this:
[BENCHMARK] model_name llama_13b world_size 2 num_heads 40 num_kv_heads 40 num_layers 40 hidden_size 5120 vocab_size 32000 precision float16 batch_size 1 input_length 710 output_length 190 gpu_peak_mem(gb) 0.0 build_time(s) 116.39 tokens_per_sec 43.09 percentile95(ms) 5120.208 percentile99(ms) 5120.208 latency(ms) 4409.816 compute_cap sm80 quantization QuantMode.0 generation_time(ms) 3751.546 total_generated_tokens 189.0 generation_tokens_per_second 50.379

I see there are two token per sec numbers, which is correct? and what is the meaning for each of them?
I can't find any documentation mentioning that.

tokens_per_sec means the throughput of end to end inference. It is computed by generated_tokens / total_latency. In your case, it is computed by 190 / 4409.816 * 1000 = 43.09.
generation_tokens_per_second only consider the generation. It means the thorughput during generation and computed by generated_tokens / generation_time. In your case, it is computed by  189 / 3751.546 * 1000 = 50.379.

tokens_per_sec means the throughput of end to end inference. It is computed by generated_tokens / total_latency. In your case, it is computed by 190 / 4409.816 * 1000 = 43.09.
generation_tokens_per_second only consider the generation. It means the thorughput during generation and computed by generated_tokens / generation_time. In your case, it is computed by 189 / 3751.546 * 1000 = 50.379.

Got it, the lower number includes the prefill stage.

what about  gpu_peak_mem mean ? It is 0 in my case.

Could you take a try on latest main branch?


Add appropriate metadata for linking and sharing Together

Intro
This issue has been opened specifically to give 100Devs members a chance to get involved. The issues presented will have more instruction and be easier to solve. If you are already contributing to this project or your skills are well-ahead of this issue, consider letting someone else take this on!
Summary

👆 There is currently... no metadata that describes the app when sharing the link. You will need to edit Together/client/public/index.html to add a short description and image that represents the app.

💡 Seek feedback from the #Together thread (100Devs Discord)

The Task
Creating your branch

Fork the Together repository.
Navigate to the development branch using git checkout development.
Create a new branch using git checkout -b your-branch-name-here.
If you have questions, more instructions about this can be found in our contributing document.

Setting up a local server

Execute npm install
Execute npm run dev-concurrent in a vscode terminal.
If you get errors, you may have to do one of the following:
Delete your node_modules folder and then run npm i
Make sure you have a .env file with the necessary secrets

What you need to do

Navigate to Together/client/public/index.html
Edit the meta description tag with enticing text.
Take a screenshot or source part of the image from the design discussion #17 (something that best represents the app)
Place the screenshot file somewhere in the public folder and link to it from index.html

Example code for linking image with meta properties:
<meta property="og:image" content="url"/>
<meta property="og:image:secure_url" content="url"/>

When you are done
-Use git add . and then git commit -m 'add a descriptive commit message here' to commit your changes to your local branch.
-Push your commit(s) using git push.
-Open your fork in GitHub and you will see a message saying you can create a pull request. Make sure your pull request is pulling into the 'development' branch instead of 'main'.
Other Info
Who to contact
If you have any questions, you can either send a message in the Together channel in 100Devs Discord.
Relevant file locations
Together/client/public/index.html
Potential Problems
You will need to do some research on your own to determine current best practices.
If you're interested in working on this feature, please comment so you can be assigned

I'd like to take this up.

Hey there @uaravindshenoy just checking in on this! Thank you.

Have been caught up in some personal stuff for a while, I'll try and get on this over the weekend if that's OK.

Have been caught up in some personal stuff for a while, I'll try and get on this over the weekend if that's OK.

Yep that's ok! Please let us know if you expect any more delays.

Hey Caleb, I would like to take up this issue. If it's ok.

Hey Caleb, I would like to work on this issue.

Hey Caleb, I would like to work on this issue.

assigned. @intelagense can you ensure that the issue is correct and @Letsbuilditnow doesn't re-do any of the work already completed?

Here is another resource as well that I bookmarked when creating the issue:
https://web.dev/learn/html/metadata/

Hey there @itisMilan just checking in on how things are going!

Hey @Caleb-Cohen, I have added the meta tags but when I run the test I am getting some error related to prettier.
11:4   error  Delete `␍`  prettier/prettier
  12:65  error  Delete `␍`  prettier/prettier
  13:36  error  Delete `␍`  prettier/prettier  14:25  error  Delete `␍`  prettier/prettier  15:16  error  Delete `␍`  prettier/prettier
  16:3   error  Delete `␍`  prettier/prettier
  17:1   error  Delete `␍`  prettier/prettier
  18:28  error  Delete `␍`  prettier/prettier

✖ 5327 problems (5327 errors, 0 warnings)
  5327 errors and 0 warnings potentially fixable with the `--fix` option.

I thought git automatically fixes these for you?  And why are there five thousand of them 👀

I thought git automatically fixes these for you? And why are there five thousand of them 👀

Prettier is really starting to irk me, why is it so hard to get it to be consistent between developer environments?

Hey @Caleb-Cohen, I have added the meta tags but when I run the test, I am getting some error related to prettier.
11:4   error  Delete `␍`  prettier/prettier
  12:65  error  Delete `␍`  prettier/prettier
  13:36  error  Delete `␍`  prettier/prettier  14:25  error  Delete `␍`  prettier/prettier  15:16  error  Delete `␍`  prettier/prettier
  16:3   error  Delete `␍`  prettier/prettier
  17:1   error  Delete `␍`  prettier/prettier
  18:28  error  Delete `␍`  prettier/prettier

✖ 5327 problems (5327 errors, 0 warnings)
  5327 errors and 0 warnings potentially fixable with the `--fix` option.

👆 @timmyichen @RascalTwo I know this is a CRLF vs LF problem. Do y'all have a best practice answer for how to correct this or avoid it entirely in the future?

I think this issue is related to this recent PR . https://github.com/Together-100Devs/Together/pull/369

@itisMilan
can you attempt changing CRLF to LF?

@itisMilan
can you attempt changing CRLF to LF?

@intelagense suggested it and I have done that but still getting the same error.

@itisMilan

can you attempt changing CRLF to LF?

@intelagense suggested it and I have done that but still getting the same error.

Brutal. I noticed you have to save the file after changing the setting. Are you sure you saved the setting?

Yes, I have saved it. So this error comes when I run "npm run lint" but, when I run "npm run test-frontend" it's showing "all specs passed!"

Updated the endOfLine to auto in .prettierrc file, now the errors are gone.


new docs structure with automated API ref

@Naraenda Please review/help

@LisaDelaney
Can you add to .gitignore file at editors section:
Editors
.venv
.vscode

@saadrahim @LisaDelaney
Created a pull request about the PR findings:
https://github.com/LisaDelaney/hipSPARSELt/pull/1/files

@saadrahim @LisaDelaney
https://github.com/LisaDelaney/hipSPARSELt/pull/2

@vin-huang Can you please review this?


pachd is slow after upgrading to 1.13.2

What happened?:
Interaction with pachd is very slow. For example pachctl update pipeline can take over 10 seconds to return.
When typing pachctl list job the command hangs and after a few seconds I see the following output:
~
~
~
... (this fills the whole screen)
~
~
~
~
:...skipping...

EDIT: upon inspection, I am seeing that pachd is crashing and getting OOM killed. The pod is becoming unhealthy for some reason.
What you expected to happen?:
pachd works in a timely manner.
How to reproduce it (as minimally and precisely as possible)?:
run pachctl list job or pachctl update pipeline with the latest version in an environment similar to mine.
Anything else we need to know?:
This started after upgrading to 1.13.2
Environment?:

Kubernetes version (use kubectl version): 1.20.2
Pachyderm CLI and pachd server version (use pachctl version): 1.13.2 for both
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s): azure AKS 1.20.5

Updating on this: i've updated my pachd spec to 4cpu and 16GB and my etcd volume to use premium storage. Will test if this solves this and if it does I will close the issue.

@RaananHadar Thanks for filing the issue. Can you also provide info about what was the previous version pachd? Also the resources for pachd in previous version?

Thanks for the quick response @nitinjainsj . I used version 1.12.5 with the default resources requests, limits and storage classes for etcd (I did use premium class object storage). I did not notice any slowdowns with these settings. I can update that deploying with the increased resources allocation really helped. Pachd no longer hangs for over 10 seconds.. but I do see some slow downs of a few seconds when running pachctl list job.

Agent Dale Georg linked Freshdesk ticket 243 for this issue.

Agent Dale Georg linked Freshdesk ticket 243 for this issue.

Agent Dale Georg linked Freshdesk ticket 285 for this issue.

Agent Dale Georg unlinked Freshdesk ticket 243 from this issue.

closing this as this should behave differently in 2.x


Added Responsive Registration and Login Page as a part of Hacktoberfest 2021 (#25)

#25 issue done - for Hacktoberfest 2021
Added Responsive Login Page, Registration Page with HTML, CSS, and a bit of Javascript

Can you share the ss of the login page ?


Corrected Azure Cosmos DB DocumentDB API->SQL API

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/spring-framework/configure-spring-boot-starter-java-app-with-cosmos-db.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@rloutlaw - Can you review & merge? Do these get staged?

@mimig1 can you reopen this vs. a merge into the master (and not live) branch? I can't merge direct into live-that's our  ref pub team only.

@rloutlaw - Will do.

@rloutlaw - Created https://github.com/Azure/azure-docs-sdk-java/pull/406 to merge to master instead


BC17.1 ON PREM to 365 Sales Version 1710 (<IP_ADDRESS>145) integration many questions

After many combinations of settings and reading what we think is all the relevant documentation, we are almost giving up.
At least to me is not clear, but maybe someone has the answer. The CRM and BC are in different Azures, as implemented by different companies.
-Is it even possible to make this integration using NavUserPassword authentication? Do we have to make any extensions?
-Do you have to install the apps manually in the CRM from the BC kit?
-Do we have to configure anything special in the Administration console of BC for this to work?
-Could it work with a self signed certificate for the web client?
-Can you still use Office 365 authentication type in CDS page in Business Central or is it deprecated?
-How exactly to register the app in Azure of the CRM? Do you still choose WEB?
-How should the redirect URL look like? Must it be this type "https://MyServer:443/BC160/OAuthLanding.htm" in both systems?
-Did anyone succeed the integration between these versions?
Thank you,

Updates: we did not give up and have answers:
-Is it even possible to make this integration using NavUserPassword authentication? Do we have to make any extensions?
Yes, it worked without extensions.
-Do you have to install the apps manually in the CRM from the BC kit?
No, there was something installed because of previous tests, we uninstalled it and then ran Redeploy Integration solution. This installed the solutions in Sales again.
-Do we have to configure anything special in the Administration console of BC for this to work?
We checked Enable SSL but to this point, I am not sure it had an effect.
-Could it work with a self signed certificate for the web client?
Has to be a proper one
-Can you still use Office 365 authentication type in CDS page in Business Central or is it deprecated?
Yes, it worked with 365 authentication
-How exactly to register the app in Azure of the CRM? Do you still choose WEB?
Has to be Web, only when assigning permissions you have to choose Business Central
-How should the redirect URL look like? Must it be this type "https://MyServer:443/BC160/OAuthLanding.htm" in both systems?
we used the url in our browser, that was starting with HTTPS, and at the end of it added OAuthLanding.htm
-Did anyone succeed the integration between these versions?

Thanks for your tenacity in these areas. There has been considerable churn in these areas during the last year. For example, authentication requirements have changed, CDS is now Dataverse, and the process for registering an app in Azure portal is different.
We've been doing our best to keep up, and have recently published some updated content that I think aligns with your questions. Please have a look at Connecting to On-Premises Versions of Business Central 2019 release wave 1 and Microsoft Dynamics NAV 2018.


security/acme-client: logfile is scambled in the GUI

@fichtner: The logfile looks distorted in the GUI right now. Do you have some advise how I might fix this? :)

squid has a similar issue. the regex is fixed for BSD, not per daemon.

@fraenki It may be the hardcore solution for your log but you can look at the log viewer of the net/quagga plugin. It gets the log from a configd call as a JSON structure and and passes it through.

@fraenki you need core commit 11f359c and plugins commit 24ff623. Core commit won't be in the 17.7.1 release, so you can try this. I added the plugin fix to 17.7.1 anyway, it is harmless. "6" was guessed from your screenshot. May be off by one. ;)

I can test https://github.com/opnsense/core/commit/11f359c on top of 17.7.1 if you like me to...

@Woi all help is welcome:
opnsense-patch 11f359c
... but I think you know this :)
Cheers,
Franco

Looks nice now :+1:

Thank you, Marco. We'll land this in 17.7.2 then :)

Reporting #243, I stumbled about this bad looking line breaks in the date column, if a long log message exists:

This can't be fixed. It writes a proper entry per line. It's the header dump that makes this awkward. 😊

This can't be fixed. It writes a proper entry per line. It's the header dump that makes this awkward.

We'll change acme.sh's default log level in the next release (https://github.com/opnsense/plugins/pull/254). This will finally fix this issue :)

@fraenki nice, thanks!

Still broken on 17.7.2...

it's not broken, literal newlines in log messages from the particular service are, er, silly...

Whatever, it looks broken ;)

ps: did you reboot? the services don't auto-restart

Ah, no, I haven't. Will try that tonight.

@StrikerTwo The fix changes the log setting of acme.sh, that's why it will only fix new log entries. Maybe clear the log and try again.

Ah, I see. I have rebooted the box and generated some new entries in the LE log and it looks better now. Thanks!


[WIP] Always re-copy Framework/podspec for iOS module project

Description
When deleting the ephemeral directory for an iOS module project, recopy a debug Framework and podspec into the ephemeral directory so that the next run of cocoapods does not fail.
fixes https://github.com/flutter/flutter/issues/42956

@jmagman  How flaky was the test previously? Should I be able to tell that this is "fixed" fixed in 2/5/10 runs?

@jmagman How flaky was the test previously? Should I be able to tell that this is "fixed" fixed in 2/5/10 runs?

If I recall, unscientifically... 3-4 times? 😅


auto-pr: fix setup-homebrew, env usage

Hmm, this didn't do the trick:
==> certbot: marking as revision 2
/usr/local/Homebrew/Library/Homebrew/utils/ast.rb:150:in `replace_stanza': Could not find 'revision' stanza! (RuntimeError)
	from generate-prs.rb:106:in `block in <main>'
	from generate-prs.rb:59:in `each'
	from generate-prs.rb:59:in `<main>'
Error: Process completed with exit code 1.

https://github.com/Homebrew/brew-pip-audit/actions/runs/5489503701/jobs/10003741749


ValueError: Found array with 0 sample(s) (shape=(0, 2)) while a minimum of 1 is required by NearestNeighbors.

Traceback (most recent call last):
File "/content/ER-NeRF/data_utils/process.py", line 388, in
extract_background(base_dir, ori_imgs_dir)
File "/content/ER-NeRF/data_utils/process.py", line 86, in extract_background
nbrs = NearestNeighbors(n_neighbors=1, algorithm='kd_tree').fit(fg_xys)
File "/usr/local/lib/python3.10/dist-packages/sklearn/neighbors/_unsupervised.py", line 176, in fit
return self._fit(X)
File "/usr/local/lib/python3.10/dist-packages/sklearn/neighbors/_base.py", line 491, in _fit
X = self._validate_data(X, accept_sparse="csr", order="C")
File "/usr/local/lib/python3.10/dist-packages/sklearn/base.py", line 565, in _validate_data
X = check_array(X, input_name="X", **check_params)
File "/usr/local/lib/python3.10/dist-packages/sklearn/utils/validation.py", line 931, in check_array
raise ValueError(
ValueError: Found array with 0 sample(s) (shape=(0, 2)) while a minimum of 1 is required by NearestNeighbors.

Had frames with no person in the video


Backport "HBASE-24200 Upgrade to Yetus 0.12.0" to branch-1.4

Signed-off-by: Sean Busbey<EMAIL_ADDRESS>Signed-off-by: Duo Zhang<EMAIL_ADDRESS>

Pre commit check failed?

Indeed. Testing if PR works for branch-1.4, #2603 .

Yeah, looks like it's broken on 1.4. Filed HBASE-25231.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-1.4 Compile Tests _

+0 :ok:
mvndep
2m 21s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  9s
Maven dependency ordering for patch

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

4m 14s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2601/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2601

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 4227864d8471 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-agent/workspace/Base-PreCommit-GitHub-PR_PR-2601/out/precommit/personality/provided.sh

git revision
branch-1.4 / e34c0e3

Max. process+thread count
41 (vs. ulimit of 10000)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2601/4/console

versions
git=1.9.1 maven=3.0.5

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Upgrade kustomization files to Kustomize v5

What this PR does / why we need it:
Migration from kustomize v4 to kustomize v5
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #2292
Checklist:

[ ] Docs included if any changes are user facing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
77

Relevant Lines:
77

💛  - Coveralls

Thanks!
/lgtm


Improve "Complete the organization profile" banner

At present, the owner of a new Zulip organization is immediately shown a banner at the top of the app that says:

Complete the organization profile to brand and explain the purpose of this Zulip organization.

As discussed on CZO, we should:

[ ] Change the banner text to say the following, which better explains what the profile is for:

Complete your organization's profile, which is displayed on your organization's registration and login pages.

[ ] Delay showing the banner until 15 days after the organization is created.

@zulipbot claim


API Authorization in javascript from outside a cluster

Hi, is there any code in javascript on API Authorization from outside a cluster?
Thank youl

While this repository only contains the go code, I'm not aware of any Javascript implementations. It should be fairly trivial to reimplement, you can look at the pkg/token/token.go source for how to generate a token. Another example would be the Python implementation in the AWS CLI


[NDMII-2865] Add initial async rdnsquerier functionality.

What does this PR do?
Adds initial rdnsquerier functionality as a foundation for providing reverse DNS enrichment of private IP addresses to NDM NetFlow events, and a noop rdnsquerier implementation that is used by default.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
No specific QA is needed for these changes because the component is disabled by default.  QA has already been done to confirm that it is disabled by default and that the changes cause no difference in the agent or NDM NetFlow functionality.  These changes can be tested/QAed with subsequent related changes once full reverse DNS enrichment and related configuration is supported.

/merge


Which env should be first when there's two?

Proposed change
When running with an older Python (just 2.7 prior to #1239, <= 3.6 and more in the future), there are two environments:

$NB_PYTHON_PREFIX - the env where the notebook server runs
$KERNEL_PYTHON_PREFIX - the env where the kernel runs

These two variables are both always defined, so absolute paths with env variables can always be used to disambiguate. In most cases, these two are the same, but in the too-old-python cases, they will differ. The question is: which should be first, and when?
There's ambiguity on the command-line, because both envs are on $PATH. Right now, we are in a bit of a peculiar situation, because $NB is first on path so that e.g. jupyter-lab will launch in the server env, but we set CONDA_DEFAULT_ENV so that conda install installs in the kernel env -it is assumed that runtime installation will want to go in the kernel environment. However, this means tha conda install and pip install will install to different environments, and python script.py, ipython, etc. will launch in the server env, not the kernel one. (aside: %pip install from a notebook will work, because that reliably resolves to the kernel's env, regardless of $PATH).
I think in most cases, when the two differ the least surprise will occur if $KERNEL_PYTHON_PREFIX comes first. This is easy enough to switch, but we need to be careful to ensure that the launch command finds the NB prefix first, even if the rest of the time the KERNEL prefix should come first. Anything that relied on the server env being first may also break. This should only come up, though, for commands that are present in both prefixes - python, pip, jupyter, etc. It's not a pure win, though - common postBuild commands operate on the server env, like jupyter labextension install. Some of these could break if we switch.
Whether we change it or not, I think we should surface these environment variables, and encourage folks to use them when running commands to be explicit about which env they are running in, especially in postBuild.
Alternative options

Keep the status quo - server env comes first, kernel comes second, special-case conda install
Only remove conda special-case, so conda and pip are both pointed to the same environment (the server env)

Who would use this feature?
Users who execute jupyter or python or pip commands in the terminal, either at runtime in the container (e.g. in a terminal), or in steps like postBuild.
One of the more common postBuild steps is something like jupyter labextension install. This relies on the current $NB prefix being first. It will actually work most of the time in both cases, as long as jupyterlab is not also in the kernel env (jupyter [subcommand] works across envs if both are on your path - it just picks the first it finds).
How much effort will adding it take?
It may be tricky to get the details right in keeping our default launch command working, if we switch the kernel to be in the front, since there are a lot of cases:

no command specified
user specified, e.g. jupyter-lab (which should be in the front here?)
BinderHub, which currently specifies jupyter-notebook

Who can do this work?
Folks with path/shell environment and possibly conda env activation experience

I think we have issues from a while ago about people being surprised that if they run python in the terminal (of jupyterlab) that they don't find the dependencies that they thought they installed. Which adds some data points to the "the NB env should be first".


Does consume timeout produce a message that is null or an exception?

Couldnt find this clearly explained in the API documentation so decided to ask here. Does a call to consume timing out produce a message that is null always or throw an exception? Also if the answer is that it produces a message that is null does any other situation produce a null message? In essence can a null message be automatically assumed to have happened because of a timeout?

[ 1.2.2] Confluent.Kafka nuget version.

yes, if you get null, that is always because no message is available to be consumed before the specified timeout.
note also that exceptions thrown by Consume should generally just be logged, they aren't terminal.

Thank you for the answer!


pubsub message count alert triggering

https://pantheon.corp.google.com/monitoring/alerting/incidents/0.mhlvq8sxehac?project=flutter-dashboard&pageState=("interval":("i":("s":"2022-05-05T15:37:05.000Z","e":"2022-05-05T16:29:26.979Z")))

@CaseyHillers can you please take a look

I rolled back Cocoon to 701db1e. go/engine-build isn't getting it's results updated, and it's likely caused by https://github.com/flutter/cocoon/pull/1816.
I can revisit the PR and try to fix the bug in the logic. I didn't realize the hotfix was still being relied on.

FYI, alert has recovered: https://pantheon.corp.google.com/monitoring/alerting/incidents/0.mhlvq8sxehac?project=flutter-dashboard&pageState=("interval":("d":"P42D","i":("s":"2022-05-05T15:37:05.000Z","e":"2022-05-05T16:59:04.000Z")))

De-duping with https://github.com/flutter/flutter/issues/102998 for the fix. I think we want to explicitly fail on builds that don't have a commit_key


🛑 Arma 3 Server is down

Server has been down since 05/30/2023
Dashboard gives 504 Error when attempting to access the Arma Server.

Resolved: Arma 3 Server is back up in 14abe2f.


If we close the Notification[Toastr] on one tab, can we get notified on another tab and the toastr gets dismissed there as well.

If we close the Notification[Toastr] on one tab, can we get notified on another tab and the toastr gets dismissed there as well. ?

No, you have to write your logic to do that in the toastr.options.onclick call back.
You can use cookies or session storage or db to handle the state of the notifications.


Fixed a typo and an extra word

OPS Build status updates of commit 2577032:
:clock10: Preparing: average preparing time is 1 min(s) 6 sec(s)

OPS Build status updates of commit 2577032:
:clock10: Incremental building: average incremental building time is 47 sec(s)

OPS Build status updates of commit 2577032:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/locale/addresses.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix CRD shortName

Codecov Report

Merging #320 (a7ad788) into master (d90b78a) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #320      +/-   ##
==========================================
- Coverage   37.41%   37.39%   -0.03%
==========================================
  Files          21       21
  Lines        8943     8943
==========================================
- Hits         3346     3344       -2
- Misses       5249     5251       +2
  Partials      348      348

Impacted Files
Coverage Δ

KubeArmor/monitor/systemMonitor.go
32.50% <0.00%> (-0.39%)
:arrow_down:

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 d90b78a...a7ad788. Read the comment docs.


[Draft PR]: Upgrade to Spark 3.

resolves #

Description

Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change to the "dbt-spark next" section.

I wanted to update the folks involved in the PR regarding what's the goal and the current status. I've seen questions come up here and I thought it's best to address this and bring folks to the same page so there's some clarity.
2 main goals:

Move completely over to Spark 3. This means the testing and documentation. The current dbt-spark master is pointing to 3.0 Spark but the testing infra hasn't successfully moved over.
godatadriven/spark has been serving as the base image for the containers. Ideally, we should be in parity with Open source Spark: https://hub.docker.com/r/apache/spark/tags rather than other images. Open source spark is already on version 3.2.1 while the godatadriven/spark image is still on 3.1.1 (not even 3.1.3).

Current status:

To solve the 1. goal - the only current blocker as it stands is the moving of the integration-spark-thrift

Attempts at this include testing godatadriven/spark: versions 3.0, 3.1 and latest. All of them don't seem to succeed.
Test failures by versions of images: 3.0, 3.1
At this point my theory is some permissions issue for the user, since show queries seem to work fine. The docker-compose command works and the image comes up but the thrift test is the blocker here.
I looked at the godatadriven/spark images from Spark 2 to 3 and the only large changes look like
Java 8 -> 11, Spark 2.4.5 -> 3.1.1, Hadoop 2.7 -> 3.2.0
These are big changes but doubt they would impact thrift particularly.

With regards to solving for 2. I hit a few roadblocks there:

With the regular Spark image, there's a failure in the integration-spark-thrift coming from the image itself. I think this is a bug from the entrypoint.sh in Spark.
With the pyspark image: I see failures like above and the integration-spark-session as well. Looks like a permissions issue.

Other possible issues:

The fishtownanalytics/test-container is also pretty old - but looking at it's Dockerfile, it might not be the root cause.

Is there possibly any help I can provide with this? I noticed the godatadriven image was at Spark 3.0 which personally, I don't think there's a good reason to use Spark 3.0.x series now.
Do we know what comes from the godatadriven spark image that we aren't getting elsewhere? Is it just convenience (e.g. to have a Spark image) or is it more than that?

Thanks for the help and context along the way, and also the review, @jtcohen6.

Pressing merge! 🤞


sci-hub.tw not available

Hey, i can't use your otherwise great extension because sci-hub.tw is not reachable atm.
It would be nice if there was an option to manually set the url of the sci-hub domain to get rid of this issue.

Agreed, as it is this add-on is not usable.

Opened a Pull-request

Hi, i'd like to ask if there is any progress in changing the domain?

Nope, seems project is abandoned, no-one has accepted the request.
Seems like sci-hub.tw is working however.

you can download the code(zip), unzip it and change the domain in ‘background.js’, then enable developer mode and load this extension in chrome

See #10 which allows end-user option to change sci-hub domain and the instructions to manually install as a comment in that PR.


Type definitions for TypeScript

Thank you for a great library!
I'm interested in using redux-loop in TypeScript and I would be able to write type definitions that match this documentation: https://github.com/redux-loop/redux-loop/blob/3.0/docs/ApiDocs.md
Would you be interested in accepting a pull request with type definitions?

These are already implemented in #110 and will be coming soon. Thanks for offering to work on them though!

It looks like a TS rewrite and definition file generator was added at some point and then removed later before 3.0 went out. Is any help needed for this feature?

hi. yes help would be very much appreciated. I've been meaning to learn about typescript more since I have no experience with it but haven't had time yet. We went through a few API changes and the one that was in typescript is not what ended up released so any help in getting typings or converting to typescript would be very much appreciated. Please see #130 for more detail.


WFLY-4972 Ignore intermittently failing EjbAppDsRemoveTestCase

Windows Build 2160 is now running using a merge of e8353f3bebdf8d640c466651d311aae564da265a

Linux with security manager Build 1981 is now running using a merge of e8353f3bebdf8d640c466651d311aae564da265a

Linux Build 7108 is now running using a merge of e8353f3bebdf8d640c466651d311aae564da265a

Linux with security manager Build 1981 outcome was SUCCESS using a merge of e8353f3bebdf8d640c466651d311aae564da265a
Summary: Tests passed: 877, ignored: 284 Build time: 0:07:09

Linux Build 7108 outcome was SUCCESS using a merge of e8353f3bebdf8d640c466651d311aae564da265a
Summary: Tests passed: 2955, ignored: 355 Build time: 0:46:50

Windows Build 2160 outcome was FAILURE using a merge of e8353f3bebdf8d640c466651d311aae564da265a
Summary: Tests failed: 2, passed: 2953, ignored: 355 Build time: 0:51:28
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.xsite.XSiteSimpleTestCase.testPutRelayedToBackups: <no details avaliable>

org.jboss.as.test.manualmode.ejb.client.cluster.EJBClientClusterConfigurationTestCase.testServerToServerClusterFormation: <no details avaliable>


Upstream eio changes

There are many changes in eio:

support runtime events
provide separate names, logs and automatic locations
tracing of cancellation contexts and fiber movement

Ideally, we have to keep in mind that the runtime events interface that meio understands could be useful for other async libraries. So one way ahead is to extract a set of runtime event types that both eio and other libraries can use to provide their events.

One hard thing is automatic caller location. This is currently a hack in my fork of eio.

We could probably do with some benchmark information (at some point) to understand the impact of running the tracing calls in an intense Eio program too?

Because it's using get_callstack it definitely has some runtime overhead. We could also have a global switch for that.
Ideally if the compiler generates the location info at the call site then there's less runtime overhead.


Added device presets for LLDB that build the static device stdlib.

Added device presets for LLDB that build the static device stdlib.

@swift-ci please smoke test


Clarify that @oct in  changes between B and C

The main documentation for <note> says that pname and oct conform to Acoustical Society of America representation, but didn't say that oct changes between B and C, not G-A. The tutorial at https://music-encoding.org/tutorials/101-quickstart  also links to this Wikipedia page, so I added it here for more detail and added a sentence for clarification of oct.
I'm not sure how to build these docs to test them - I copied the <ref> syntax from another source file, I think it's correct.

thx @alastair


HomeKit component only works in single instance

So I have 3 Raspberry Pi's running Home Assistant on the same network.  Originally I used Home Assistant for Homebridge in a single instance running on one of the 3 Pi's to bridge all my accessories (from all 3 Pi's) over to HomeKit, which was working fine.
When the native HomeKit component came out earlier this year I wanted to deprecate Homebridge and use the HomeKit component to bridge my accessories over to HomeKit.  I converted my first Pi, removing it from the Homebridge and adding the native HomeKit component and that's working just fine.
However, I'm having no luck trying to do the same on the second Pi.  When I add the HomeKit component on the 2nd Pi I see the following:

Home Assistant Bridge on the first Pi (along with all accessories behind it) stops working, shows in HomeKit as unreachable.
Home Assistant UI on the second Pi shows a setup tile for HomeKit with the pairing code, but when trying to add the accessory to HomeKit it sits and spins for a few minutes and eventually fails.

When I remove the HomeKit component from the second Pi, the first Pi eventually recovers after a couple minutes and Home Assistant Bridge starts working again without further intervention.
This looks to me like theres some kind of conflict that prevents multiple instances of the HomeKit component from running on the same local network.  I don't believe this is logical or by design, so I would consider it a bug.
Any ideas how to further troubleshoot or fix this?

Have you tried assigning a different port to the second instance using the port parameter for the homekit component?

Yep change one of the ports of any instance and you'll be fine.

Yes, I've tried
homekit:
port: 51839
with the same results as before.
Since each Pi has its own IP address, there shouldn't be a problem with using the same port, but in any case, it doesn't seem to make any difference.

@JAYMAN-ATX I'm having the same issue.  I have tried to change the port and manually set the IP addresses as well.  Any ideas on further action?

Also I don't know if this helps but I have been seeing this in my logs:
2018-07-13 13:45:28 WARNING (SyncWorker_7) [homeassistant.components.homekit] HomeKit is not ready. Either it is already running or has been stopped.
This is from the instance where the HomeKit is started second and will not show up for pairing.

@frenck any ideas on this one? I'm running HassOS in two virtual machines with their own MAC addresses 😃

@erfind Have you tried them individually? So that only one VM is running.

Hey! I did, unfortunately it didn’t help resolve the issue. 😕 If one had already been paired then the other wouldn’t show up.
Sent from my iPhone

On Jul 17, 2018, at 16:25, cdce8p<EMAIL_ADDRESS>wrote:
@erfind Have you tried them individually? So that only one VM is running.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@erfind I finally was able to debug and reproduce this issue and I think I've found the reason it's not working. The Home app doesn't like that both Home Assistant HomeKit instances have the same display_name and serial_number. I'll open a bugfix PR soon.