Sync: Keybindings conflict shown as a single large change

Issue Type: Bug
Testing #85941
I disabled keybindings sync, changed a keybinding on both machines (the other is Mac) and then reenabled keybindings sync.
The conflict is shown as a single large change over the entire file:

Comparing the changes shows that there is just a small diff:

VS Code version: Code - Insiders 1.41.0-insider (97855786a014be2440751b038b373c3726e11fe8, 2019-12-03T05:40:39.806Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (4 x 2195)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
8.00GB (2.97GB free)

Process Argv

Screen Reader
yes

VM
44%

Extensions (29)

Extension
Author (truncated)
Version

unique-lines
bib
1.0.0

ssh
chr
0.0.4

ghlink
dt
1.2.0

remotehub
eam
0.5.0

tsl-problem-matcher
eam
0.1.0

EditorConfig
Edi
0.14.2

tslint
eg2
1.0.44

git-project-manager
fel
1.7.1

vscode-pull-request-github
Git
0.13.0

rest-client
hum
0.23.0

solidity
Jua
0.0.64

theme-karyfoundation-themes
kar
18.2.1

graphql-for-vscode
kum
1.15.3

vscode-docker
ms-
0.9.0

remote-containers
ms-
0.87.0

remote-ssh-edit-nightly
ms-
2019.12.2400

remote-ssh-nightly
ms-
2019.12.2400

remote-wsl
ms-
0.41.1

azure-account
ms-
0.8.7

azurecli
ms-
0.4.6

github-issues-prs
ms-
0.9.2

debugger-for-chrome
msj
4.12.2

vetur
oct
0.22.6

vscode-versionlens
pfl
0.24.0

vscode-yaml
red
0.6.1

vscode-workspacecontains-canary
rob
0.0.2

vscode-open-in-github
sys
1.13.0

ayu
tea
0.18.0

vscodeintellicode
Vis
1.2.0

(1 theme extensions excluded)

Yes, this is expected to show the complete changes as we do not 3-way diff to show  conflicts. Open for ideas here.

The 2-way diff that the diff editor does would be sufficient. Based on the diff shown in the second screenshot, you could create these finer grained merge conflict regions.

Ok, I can show remote (incoming) on the left and current on the right and allow user to update the current and apply changes.


Created YAML file to build glTF-SDK on Azure DevOps

Build\AzureDevOps\azure-pipelines.yml contains the YML description on building the Windows, MacOS, and iOS builds of the glTF-SDK.
Does not yet build Android since Azure DevOps doesn't yet support ninja and glTF-SDK hasn't been tested with the gradle generator.
Also fixed some minor issues with the cmake build scripts.

The build definition is here https://gltfsdk.visualstudio.com/build/_build?definitionId=1


NPE in entity comparison in parse

I don't have many details is this is from an outside user, but there seems to be a situation where an NPE can occur when comparing entities
The report: http://crashes.to/s/b20eb2c81a0
The reactiveX errors are from my not catching the error, and everything starts with me parsing Facebook's search page.

May be here?
Not sure if
final String name = reader.consumeTo('=');

may return null

I have a feeling this in an Android issue I've seen reported on other projects where on first install of an app, on some devices, Android won't find resources in a jar if they're loaded statically. Entities is loading the entity mapping file statically on startup.

Are you referring to here?
private static final HashMap<String, String> multipoints = new HashMap<>(); // name -> multiple character references

Would you consider adopting a singleton pattern if that may help with compatibility?

Bump? I have another NPE here and I'm assuming it's a similar issue

Looks like a different stack. Can you include the stacks here so we don't have to keep hitting that other site and can find in search etc please.
Entities have changed implementation so I'd recommend bumping to 1.11.1 if the initial stack is because of the entity loading issue. f3035cafbbd8830de75f641920f8e5dbf0070f7c
For the different stack (Attributes.getIgnoreCase:67) - that's been reimplemented since whatever version your stack is. The map impl was refactored. Suggest revving to the current release.

It is indeed a different stack. Just the NPE lead me to believe the issue is similar. This was on 1.10.3, so I will close this and see if it will reappear.
Stack for search:
Non-fatal Exception: java.lang.NullPointerException: Attempt to read from field 'java.util.LinkedHashMap$LinkedHashMapEntry java.util.LinkedHashMap$LinkedHashMapEntry.after' on a null object reference
       at java.util.LinkedHashMap$LinkedHashIterator.(LinkedHashMap.java)
       at java.util.LinkedHashMap$LinkedHashIterator.(LinkedHashMap.java)
       at java.util.LinkedHashMap$KeyIterator.(LinkedHashMap.java)
       at java.util.LinkedHashMap$KeyIterator.(LinkedHashMap.java)
       at java.util.LinkedHashMap.newKeyIterator(LinkedHashMap.java:452)
       at java.util.HashMap$KeySet.iterator(HashMap.java:1188)
       at org.jsoup.nodes.Attributes.getIgnoreCase(Attributes.java:67)
       at org.jsoup.nodes.Element.id(Element.java:133)
       at org.jsoup.select.Evaluator$Id.matches(Evaluator.java:91)
       at org.jsoup.select.Collector$Accumulator.head(Collector.java:42)
       at org.jsoup.select.NodeTraversor.traverse(NodeTraversor.java:31)
       at org.jsoup.select.Collector.collect(Collector.java:24)
       at org.jsoup.nodes.Element.getElementById(Element.java:708)
       at com.pitchedapps.frost.web.SearchWebView$setupWebview$1.apply(SearchWebView.kt:61)
       at com.pitchedapps.frost.web.SearchWebView$setupWebview$1.apply(SearchWebView.kt:28)
       at io.reactivex.internal.operators.observable.ObservableMap$MapObserver.onNext(ObservableMap.java:59)
       at io.reactivex.internal.operators.observable.ObservableSubscribeOn$SubscribeOnObserver.onNext(ObservableSubscribeOn.java:58)
       at io.reactivex.observers.SerializedObserver.onNext(SerializedObserver.java:113)
       at io.reactivex.internal.operators.observable.ObservableDebounceTimed$DebounceTimedObserver.emit(ObservableDebounceTimed.java:140)
       at io.reactivex.internal.operators.observable.ObservableDebounceTimed$DebounceEmitter.run(ObservableDebounceTimed.java:165)
       at io.reactivex.internal.schedulers.ScheduledRunnable.run(ScheduledRunnable.java:61)
       at io.reactivex.internal.schedulers.ScheduledRunnable.call(ScheduledRunnable.java:52)
       at java.util.concurrent.FutureTask.run(FutureTask.java:237)
       at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:272)
       at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
       at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
       at java.lang.Thread.run(Thread.java:761)

Please open an issue if you get a recurrence on the current release.


update: README.md: linking to archive.org

#614

the right link now is https://cis.temple.edu/~pwang/GTI-book/

but the content is different, the old page has/has more information


Custom Window Size Option

Great work with this, just what i was looking for!  Thank you.
Noticed the feature to choose a custom window size is available in your other popup add-on...Is it possible to add it to this one?  Most videos i watch on YouTube are a bit narrower than the default size.
Also curious if it would be possible to have an option to automatically send an 'ctrl+space bar" command after the pop up windows opens to set it on top using the common auto hot key script?

Popup Video v0.0.6 support custom window size.
About auto hot key script, I don't know how to do that.
Thanks for your feedback.


Fix make coverage

Adds a workaround for https://github.com/golang/go/issues/27333 to make make coverage work.

Works on my machine!
/lgtm

/lgtm
/approve


rails3 with cache_cache classes = false causes problems with many 2 many associations

I have isolated a problem with rails3 and many 2 many associations in this sample rails3 app: git://github.com/mbj/bugapp
It is NOT reproducible under a non rails3 cache_classes = false environment.
The app is featuring two models:
class Article
  include DataMapper::Resource
  property :id, Serial
  has n, :persons, :through => Resource
end

class Person
  include DataMapper::Resource
  property :id, Serial
  has n, :articles, :through => Resource
end

One Controller:
class ApplicationController < ActionController::Base
  protect_from_forgery

  # routed from GET /a, creates some records needed for demonstration
  # you have to hit it to get this demo working

  def create_person_and_articles
    person = Person.create
    person.articles.create
    person.articles.create
    head :ok
  end

  # routet from GET /b, is basically a noop, (Articles are linked already), but
  # causes problems when classes got reloaded! (so hit at least twice after a fresh start)

  def assing_articles_another_time
    person = Person.first
    person.attributes = { :articles => Article.all }
    person.save
    head :ok
  end
end

Hitting /b after /a results in the following exception where sqlite complains over the already existing join record. DataMapper tries to create the join records, where it should not.
mbj@seon ~/devel/bugapp $ rake db:automigrate
(in /home/mbj/devel/bugapp)
[datamapper] Finished auto_migrate! for :default repository '/home/mbj/devel/bugapp/db/development.sqlite3'
mbj@seon ~/devel/bugapp $ ./script/rails s -p 4003
=> Booting WEBrick
=> Rails 3.0.0.rc application starting in development on http://<IP_ADDRESS>:4003
=> Call with -d to detach
=> Ctrl-C to shutdown server
[datamapper] Setting up the "development" environment:
[datamapper] Setting up :default repository: '/home/mbj/devel/bugapp/db/development.sqlite3' on sqlite3
[2010-08-10 23:00:02] INFO  WEBrick 1.3.1
[2010-08-10 23:00:02] INFO  ruby 1.9.2 (2010-07-11) [x86_64-linux]
[2010-08-10 23:00:02] INFO  WEBrick::HTTPServer#start: pid=2919 port=4003

Started GET "/a" for <IP_ADDRESS> at 2010-08-10 23:00:14 +0200
  Processing by ApplicationController#create_person_and_articles as HTML
  SQL (38.603ms)  INSERT INTO "people" DEFAULT VALUES
  SQL (29.642ms)  INSERT INTO "articles" DEFAULT VALUES
  SQL (0.124ms)  SELECT "article_id", "person_id" FROM "article_people" WHERE ("person_id" = 1 AND "article_id" = 1) ORDER BY "article_id", "person_id" LIMIT 1
  SQL (6.635ms)  INSERT INTO "article_people" ("article_id", "person_id") VALUES (1, 1)
  SQL (11.762ms)  INSERT INTO "articles" DEFAULT VALUES
  SQL (0.101ms)  SELECT "article_id", "person_id" FROM "article_people" WHERE ("person_id" = 1 AND "article_id" = 2) ORDER BY "article_id", "person_id" LIMIT 1
  SQL (7.455ms)  INSERT INTO "article_people" ("article_id", "person_id") VALUES (2, 1)
Completed 200 OK in 150ms

Started GET "/b" for <IP_ADDRESS> at 2010-08-10 23:00:17 +0200
  Processing by ApplicationController#assing_articles_another_time as HTML
  SQL (0.079ms)  SELECT "id" FROM "people" ORDER BY "id" LIMIT 1
  SQL (0.137ms)  SELECT "articles"."id" FROM "articles" INNER JOIN "article_people" ON "articles"."id" = "article_people"."article_id" INNER JOIN "people" ON "article_people"."person_id" = "people"."id" WHERE "article_people"."person_id" = 1 GROUP BY "articles"."id" ORDER BY "articles"."id"
  SQL (0.034ms)  SELECT "id" FROM "articles" ORDER BY "id"
columns article_id, person_id are not unique (code: 19, sql state: , query: INSERT INTO "article_people" ("article_id", "person_id") VALUES (1, 1), uri: sqlite3:///home/mbj/devel/bugapp/db/development.sqlite3)
Completed   in 13ms

DataObjects::IntegrityError (columns article_id, person_id are not unique):
  app/controllers/application_controller.rb:14:in `assing_articles_another_time'

Setting cache_classes = true, eliminates the problem. Running in rails console also. I'd expect the bug somewhere in the model reload code.
Note about running my example bugapp, you need to use dm-rails from git://github.com/datamapper/dm-rails, see Gemfile. dm-rails-1.0.0 from rubyforge is not yet compatible with rails3.
If you feel this bugreport would fit better in another bugtracker (like rails one under rails.lighthouseapp.com) please let me know I will post this issue there.
ruby -v
ruby 1.9.2dev (2010-07-11 revision 28618) [x86_64-linux] (ruby-1.9.2-rc2)

Created by mbj - 2010-08-10 21:49:08 UTC
Original Lighthouse ticket: http://datamapper.lighthouseapp.com/projects/20609/tickets/1385

Does adding DataMapper.finalize to the class reloading code in Rails fix this?  I've poked around in that code before, but don't really remember where it is now.

FWIW, I reckon Rails could cheat the cache_classes thing by forking before each request rather than trying to reload into the same process.  I doesn't reload anything that happens in the initializers anyway.

@d11wtq I'm not sure if this is still a problem. Can somebody confirm that?
ps. the amount of bug reports that are specific to has n :through => Resource makes me think this should be removed from DM and people should be using explicit join models :P


Need an option to disable "add to" on blueprint conig

Need an option to disable "add to" on blueprint conig

Closing this for now. Feel free to send a PR if you feel confident about it because this is not a feature we can add right now. Thank you for your contribution guys, we appreciate.


The Cursor should autofocus in search field in smartblock insertion window

When Triggered using the keyboard shortcut, the smartblock window pops up, with the list of available smart blocks.
Pressing up and down arrow keys allows you to select smartblock using the keyboard.
But however, if you have to search, you must first click on the search field using the mouse.
If the search field could auto-pull focus when it appears, the keyboard workflow will be seamless and faster.

Fixed in next version


[go_router] Support double slash after schema for deep linking

I encountered some limitations where other SDK's require deepLinking schema to be as "myApp://..." with double slashes. But go_router works only with "myApp:/..." which I cannot use in other SDK's.
Use case
Do not ignore  paths with double slashes after schema.
So users won't be confused with it when acidentally adding double slashes.
Proposal
"myApp://..." and "myApp:/..." should work the same.

Hi @KestasVenslauskas, I believe this should already be supported. Is there a particular issue you're running into when using double slashes?

@danagbemava-nc So if I run adb shell am start -d myApp:/screenA on android app it opens the screenA route. Even if the app is force stopped... But if I run adb shell am start -d myApp://screenA nothing happens if the app is running already. And if the app is force closed then it only opens the app with initial route...

Hi @KestasVenslauskas, please provide a complete minimal reproducible code sample so that we can investigate this.
Thank you

@danagbemava-nc Here you go https://github.com/KestasVenslauskas/go-router-paths-issue

Hi @KestasVenslauskas, thanks for the repro. This is indeed a bug and I think it is a duplicate of https://github.com/flutter/flutter/issues/100624. Can you take a look at https://github.com/flutter/flutter/issues/100624 to see if my suspicion is correct?


convert event bus service to use async-service APIs

What was wrong?
The services we use to manage the event bus were still using the legacy asyncio service.
How was it fixed?
Updated it to use the new async-service APIs.
Cute Animal Picture

No clue but in running this branch I pretty consistently am ending up with a bunch of timeouts and hard failures from the various endpoints failing to connect to each other... so need to figure that out before this moves forward


[Follow up] Zigbee & ZWave devices Help

Thank you Nicolas. I have made progress by making devices' "On/Off" switches appear in the Home app, though not what they are named in the Harmony app. For example, what I have labeled "Floor Lamp", is listed as "zwave4" and "Smart Things Outlet 1" is "zigbee0x21B7" That's fine.  However, I can't seem to get the dimmers to show up for the plugs which have them.  I do see the "brightness" function update in HomeBridge when I change it through the Harmony App.  I'm not sure what I'm overlooking.
Here's what I think is the relevant log for my example zwave4
[3/15/2021, 10:21:20 AM] [Harmony Hub] (Harmony Hub)INFO - getting home controls ...
[3/15/2021, 10:21:21 AM] [Harmony Hub] (Harmony Hub)INFO - got home controls : {"cmd":"harmony.automation?getstate","code":200,"id":"0.22116405160595698","msg":"OK","data":{"zigbee0x2EAE":{"status":0,"on":false,"batteryLevel":-1},"zwave2":{"status":2,"on":false,"statusMsg":"nocon"},"zigbee0x21B7":{"status":2,"batteryLevel":-1,"statusMsg":"nocon","on":false},"zwave4":{"status":0,"brightness":255,"on":true},"zwave3":{"status":2,"on":false,"statusMsg":"nocon"}}}
[3/15/2021, 2:54:49 PM] [Harmony Hub] INFO - onMessage : Refreshing Home Automation Switch {"zwave4":{"status":0,"brightness":255,"on":false}}
[3/15/2021, 2:54:49 PM] [Harmony Hub] (Harmony Hub)INFO - refreshHomeSwitch - Refreshing home switch zwave4 to false
[3/15/2021, 2:54:52 PM] [Harmony Hub] INFO - onMessage : Refreshing Home Automation Switch {"zwave4":{"status":0,"brightness":255,"on":true}}
[3/15/2021, 2:54:52 PM] [Harmony Hub] (Harmony Hub)INFO - refreshHomeSwitch - Refreshing home switch zwave4 to true
[3/15/2021, 2:55:05 PM] [Harmony Hub] INFO - onMessage : Refreshing Home Automation Switch {"zwave4":{"status":0,"brightness":106,"on":true}}
[3/15/2021, 2:55:05 PM] [Harmony Hub] (Harmony Hub)INFO - refreshHomeSwitch - Refreshing home switch zwave4 to true
[3/15/2021, 2:55:16 PM] [Harmony Hub] INFO - onMessage : Refreshing Home Automation Switch {"zwave4":{"status":0,"brightness":164,"on":true}}
[3/15/2021, 2:55:16 PM] [Harmony Hub] (Harmony Hub)INFO - refreshHomeSwitch - Refreshing home switch zwave4 to true
[3/15/2021, 2:55:20 PM] [Harmony Hub] INFO - onMessage :
Refreshing Home Automation Switch {"zwave4":{"status":0,"brightness":164,"on":false}}
HomeBridge shows the devices, their capabilities, and their state
First I tried:
{
"name": "Harmony Hub",
"hubIP": "192.168...",
"hubName": "Harmony Hub",
"publishGeneralMuteSwitch": true,
"publishGeneralVolumeSlider": true,
"linkVolumeControlToTV": true,
"homeControlsToPublishAsAccessoriesSwitch": [
"zwave3",
"zwave2",
"zwave4;on",
"zwave4;brightness"
],
    "platform": "HarmonyHubWebSocket"
},

That made the switch in the home app disappear
{
"name": "Harmony Hub",
"hubIP": "192.168....,
"hubName": "Harmony Hub",
"publishGeneralMuteSwitch": true,
"publishGeneralVolumeSlider": true,
"linkVolumeControlToTV": true,
"homeControlsToPublishAsAccessoriesSwitch": [
"zwave3",
"zwave2",
"zwave4"
],
"devicesToPublishAsAccessoriesSwitch": [
"zwave4;on",
"zwave4;brightness"
],

"platform": "HarmonyHubWebSocket"

},
That made the switch come back, but still no dimmer. Also, I made sure the plug is categorized as a light so the Home app will allow for a dimmer, which I think is they way it's supposed to be set up.
What should I be looking to do next?

Hi, unfortunately I don't support sliders for commands, don't even know if it is possible to detect that the light is a dimmer ... (don't have such device so hard to test !)


Add support for -Dbuild.version_qualifier

Signed-off-by: Amit Singh<EMAIL_ADDRESS>Description
-Dbuild.version_qualifier
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/1632.
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #468 (a28baff) into main (fa00555) will increase coverage by 0.18%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #468      +/-   ##
============================================
+ Coverage     78.18%   78.37%   +0.18%
- Complexity     4162     4177      +15
============================================
  Files           296      296
  Lines         17659    17659
  Branches       1879     1879
============================================
+ Hits          13807    13840      +33
+ Misses         2958     2932      -26
+ Partials        894      887       -7

Flag
Coverage Δ

plugin
78.37% <ø> (+0.18%)
:arrow_up:

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

Impacted Files
Coverage Δ

...ain/java/org/opensearch/ad/model/ModelProfile.java
70.90% <0.00%> (-1.82%)
:arrow_down:

...rch/ad/transport/AnomalyResultTransportAction.java
80.13% <0.00%> (-0.69%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
72.36% <0.00%> (+0.18%)
:arrow_up:

.../main/java/org/opensearch/ad/ml/CheckpointDao.java
70.19% <0.00%> (+0.64%)
:arrow_up:

...ain/java/org/opensearch/ad/task/ADTaskManager.java
77.35% <0.00%> (+0.67%)
:arrow_up:

...ava/org/opensearch/ad/task/ADHCBatchTaskCache.java
92.59% <0.00%> (+2.46%)
:arrow_up:

...java/org/opensearch/ad/task/ADBatchTaskRunner.java
85.71% <0.00%> (+3.49%)
:arrow_up:


[1.x] Add description for services within the config-orchestration.stub file.

What does this PR do? :robot:

Adds comment to the services key of the config-orchestration.stub file describing what it is meant for.

Does this relate to any issue? :link:
Closes #23

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

Comparison is base (e32454a) 90.51% compared to head (0fe4072) 90.51%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #29   +/-   ##
=========================================
  Coverage     90.51%   90.51%
  Complexity      172      172
=========================================
  Files            22       22
  Lines           559      559
=========================================
  Hits            506      506
  Misses           53       53

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


Move HttpFaultInjector to azure-sdk-tools

Move https://github.com/mikeharder/HttpFaultInjector to the azure-sdk-tools repo.

Fixed in https://github.com/Azure/azure-sdk-tools/pull/1320

Fixed in https://github.com/Azure/azure-sdk-tools/pull/1320


Could you show me an example of CronWorkflow together with DAG?

Hello. I would like to setup a cronworkflow together with DAG conditions.
Could you tell me if it is supported?
Unfortunately you don't have such examples, and I wasn't able to create a working one.

You can convert any Workflow into a CronWorkflow, including those that use DAGs. Take a look at the instructions in: https://github.com/argoproj/argo/blob/master/docs/cron-workflows.md


Kill the native process when user abort the pipeline

https://github.com/SanojPunchihewa/f5n/blob/e1655432caa87538b6a29f1124cc87589f77f9d4/app/src/main/java/com/mobilegenomics/f5n/activity/ConfirmationActivity.java#L325

as for now this cannot be done. the go to method is to have interrupts in the original c program.


monaco-languageclient/lib/vscode-compatibility is nowhere to be found.

could not resolve the vscode-compatibility

What documentation are you following? You need no alias.
The first one doesn't exist for months, the second one brakes everything by redirectong any subimport in vscode to the main export

Check tour monaco editor version

The lib wasn't updated yet to the 0.35

Can you please explain how for others who may have the same issue ?

Btw, we have these sections in the README:
https://github.com/TypeFox/monaco-languageclient#monaco-editor-compatibility-table
https://github.com/TypeFox/monaco-languageclient#troubleshooting
Didn't yarn install warn you about the incorrect peerDependency monaco-vscode-api demands?


Using value classes and data class as 'opaque' types in the example, also added kover plugin and some tests

Hi, and thx for making this nice validation lib out of 'parse don't validate' & arrow !
Maybe the build.gradle.kts need some tweaking by your multiplatform expertise ?
If ok, I can also add even more coverage for tribune-core in a separate PR later.
For me, using opaque types together with validation makes a lot of sense, but it may be subjective.
In Kotlin, that requires using 'internal' modifier together with modules.. but having a 'domain objects' module is nice anyway I think.
Sorry that this didn't land in nice separate commits.

Great job :)

And more PRs welcome. I'm planning on expanding out the docs a lot.


55 improve pr template

Close #55

:tada: This PR is included in version 2.2.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore: fix codecov

Signed-off-by: Filip Burlacu<EMAIL_ADDRESS>

If we can merge this I can rebase https://github.com/trustbloc/hub-auth/pull/57 and codecov will give me a useful analysis


revisit annotation-xml support

One of the more curious items from the backlog:

embed annotations in a more HTML5 way (OpenAnnotation, ARIA, data-* etc)

I think I added this before I realized that the OAWG does not specify a serialization. Probably falls into the "spec limitation" area.

This was partially fixed in v2.6 (MathML is now added as data attribute in the MathML if the AssitiveMML extension is present). I still like the idea and we might come back to this for the revamp of the menu.

There's also the context of the scholarly html W3C community group, there's schema.org microdata and RDFa.

Can you say more about what annotations you have in mind?  Do you mean things like the original TeX code that is currently added in a  element in the MathML?

I was actually thinking that this should be closed as too vague / general. At the very least it should be a meta issue.
Anyway, iirc, this started out as a thought about the new OpenAnnotation spec and about the fact that annotation-xml does not fit together with other standards (like ARIA or data-*) and suffers from only having a chance to work on the web if browsers cared about MathML in the first place. (Of course that last point is now somewhat wrong since Chrome accepted code to add this weird behavior of rendering the first annotation-xml instead of MathML -- sigh.)
Anyway, then I think I starte thinking about exposing things like sources in the data tags to make it easier to build tools on top of MathJax output without needing to understand its APIs. But there's also ARIA mentioned which now connects to the semantic enrichment project etc.
Hence my thought I should close this as not really an issue but a wide ranging topic of discussion.

Oh, and there's also a schema.org thread requesting some format for exposing mathematics. And this connects to things like the sorely outdated Units note from the MathWG and the siunitx&physics package etc.

https://github.com/schemaorg/schemaorg/issues/236

I start thinking about exposing things like sources in the data tags to make it easier to build tools on top of MathJax output without needing to understand its APIs.

This is one of the reasons I was thinking about pushing everything into the DOM for 3.0 (via elements that would be hidden from view, but that would contain the data in various forms needed by MathJax).

This is one of the reasons I was thinking about pushing everything into the DOM for 3.0 (via elements that would be hidden from view, but that would contain the data in various forms needed by MathJax).

Right. I just have a different point of view on the "via elements" part, I think. For me, the data should to stay on the critical elements, i.e., the elements realizing the visual rendering. Something worth discussing with the TC once we're further along, I'm sure.


How to use SecretStore for automatic pwd PSSession to linux?

For a long time, I didn't know the usefulness of SecretStore.
Can it be used to realize automatic password PSSession to linux?
（use ssh key-file i known）
How to do it?
New-PSSession -HostName <IP_ADDRESS> -UserName root ? ? ?

SSH does not take a credential argument as far as I am aware.  So SecretManagement/SecretStore is not useful in this scenario since there is no way to pass in a credential or token secret to SSH from SecretStore.  But using SSH with password is not very secure anyway, since it involves passing the password secret to the target computer.
SSH with key based authentication is better.  SecretStore does not currently work with SSH key authentication (like a client side ssh key agent), but this would be interesting to look into as an enhancement.


digest: Add header above plain text view of digest.

This PR adds a header above the plain text view
of the digest to be more clear about what the user is
seeing.
Fixes #21165
Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@Lotiun please update the PR to us the wording given in this comment: https://github.com/zulip/zulip/issues/21165#issuecomment-1048072545
Thanks!

Thanks!
Let's try unbolding that heading, and leaving more vertical space above it. Could you please post one or more screenshots with those changes?

I applied those tweaks and merged this, thanks @Lotiun!


Cache helper functions

A few bridges use the cache interface directly like the Instagram Bridge and Ello Bridge, and maybe soon the Soundcloud Bridge. It's however rather verbose, and all these three examples are doing almost exactly the same thing.
So my proposal is introducing two new helper functions: loadCache(key) and saveCache(key, value):

They could both use the current bridge's class name as cache scope, like in the previously mentioned bridges.
Passing null as the value to saveCache() could purge that cache entry, so loadData() returning null would mean that nothing has been cached.
An (optional?) duration argument for loadCache() could cause the function to return null if the cache entry is older than specified, similar to the duration argument for getSimpleHTMLDOMCached().

I added a PR for this, please let me know if this works for you. It seems to work for ElloBridge at least:
	private function getAPIKey() {
		$key = $this->loadCacheValue('key');

		if($key == null) {
			$keyInfo = getContents(self::URI . 'api/webapp-token') or
				returnServerError('Unable to get token.');
			$key = json_decode($keyInfo)->token->access_token;
			$this->saveCacheValue('key', $key);
		}

		return $key;

	}

That looks great to me! Thank you so much!

This is still relevant.


sq autotuning/algorithm enhancement

Type of Change
enhancement and bugfix

Does this PR target v2.2?


[Bug]: Text on org members bubble overflows when there are more than a 100 members in an organization

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When there are more than a 100 members listed under + of an organization, the text overflows the bubble boundary.
Steps To Reproduce

Go to Appsmith Internal Apps organization and observe the +105 spill over from its boundary

Public Sample App
https://app.appsmith.com/applications#appsmith-internal-apps
Version
Cloud

This is fixed.


Update internal cypress image to use Debian Bookworm

The Cypress app team would like to use Debian bookworm as its default base for internal images over Bullseye. There is currently a hurdle with this as it requires cypress to update playwright-webkit from 1.24.2 to at least 1.47.2.

@AtofStryker

The Cypress app team would like to use Debian bookworm as its default base for internal images over Bullseye.

Does that require a change in this repo? I don't see bullseye being used here anymore, but maybe I have overlooked something?

There is currently a hurdle with this as it requires cypress to update playwright-webkit from 1.24.2 to at least 1.47.2.

https://github.com/cypress-io/cypress/issues/29954 is related and<EMAIL_ADDRESS>is the minimum version required for Debian 12 (bookworm) support as mentioned in this issue

https://github.com/cypress-io/cypress/issues/29973 describes a problem which first occurs with<EMAIL_ADDRESS>and continues through to the current<EMAIL_ADDRESS>I progressed this issue as far as I could. I haven't seen any involvement from the Cypress.io team so far.

Also WebKit is not part of the factory process at this time.

@MikeMcC399 Yes, we want to add a base-internal image with bookworm.

@jennifer-shehane

Yes, we want to add a base-internal image with bookworm.

That is what I don't understand, because such cypress/base-internal images already exist, although they use an older version of Debian bookworm (12.1) not the latest (12.7).
$ docker run --rm --entrypoint cat cypress/base-internal:18.17.1-bookworm /etc/debian_version
12.1

I don't know where this image came from though, as I can't find it under https://github.com/cypress-io/cypress-docker-images/tree/master/base-internal/releases/node-18, nor in any PR.
Similarly cypress/base-internal:18.17.1-yarn-berry is also based on Debian 12.1 bookworm.
Maybe this is a request for a new cypress/base-internal image with a higher version of Node.js if it is part of the Cypress 14 work?

@MikeMcC399 Yah this is part of 14 work. It's intention is to build the binary on a distro that would support glibc 2.28, which is the same glibc version Node.js is using in LTS.

@jennifer-shehane

Yah this is part of 14 work. It's intention is to build the binary on a distro that would support glibc 2.28, which is the same glibc version Node.js is using in LTS.

For Cypress 14 it would probably mean producing an image similar to the existing cypress/base-internal:18.17.1-bookworm fitting to whatever Electron version you go with. For example cypress/base-internal:20.17.0-bookworm for Electron v32.1.2.
Each of the following operating system versions meets the requirement of glibc >= 2.28 on Linux systems. It's not constrained to Debian 12 (bookworm), although of course this choice would fit:

OS version
GLIBC version

Ubuntu 20.04.6 LTS
ldd (Ubuntu GLIBC 2.31-0ubuntu9.16) 2.31

Ubuntu 22.04.5 LTS
ldd (Ubuntu GLIBC 2.35-0ubuntu3.8) 2.35

Ubuntu 24.04.1 LTS
ldd (Ubuntu GLIBC 2.39-0ubuntu8.3) 2.39

Debian 11.11
ldd (Debian GLIBC 2.31-13+deb11u11) 2.31

Debian 12.7
ldd (Debian GLIBC 2.36-9+deb12u8) 2.36

@mschile

Has this issue been resolved through https://github.com/cypress-io/cypress-docker-images/pull/1226 from @ryanthemanuel ?

@mschile

I was puzzled by https://github.com/cypress-io/cypress/issues/30382. Was that deliberate or is it a typo?

Debian buster (version 10 - oldoldstable) reached end-of-life on June 30, 2024 already. See Debian LTS for overview. Debian bookworm (version 12 - stable) on the other hand is supported until June 2028.

Closing as completed with https://github.com/cypress-io/cypress-docker-images/pull/1226.
I'm going to open a new issue in the cypress repo to update the internal images being used.


Fix stdlib and Puppet version requirements in metadata.json

Official stdlib docs state that stdlib 2.x is not compatible with Puppet 3.x.
We require Puppet versions 3.4.0 or greater due to #121

what's up with the test failures?
 1) nodejs when run on Debian release 7.0 with legacy_debian_symlinks set to true the file resource /usr/bin/node should be in the catalog
     Failure/Error: should contain_file('/usr/bin/node')
     Puppet::Error:
       undefined method `ref' for nil:NilClass on node testing-worker-linux-f25aadab-1-20168-linux-8-61003744
     # /home/travis/.rvm/gems/ruby-1.9.3-p551/gems/puppet-3.4.3/lib/puppet/resource/catalog.rb:527:in `block in to_catalog'

The test were failing for some reason only on 3.4.0 - I have no idea why. Since the travis.yml has now changed on master, I have rebased and tests on the new Travis CI matrix are now all green.

Before this was moved across, tests were okay. This should probably be merged in before the next release is cut. Any objections?

Can you make this into a new PR so that Travis can greenlight it once more? Call me a paranoid nitpick :)

@ffrank why does it need a new PR? I see greens from travis and here: http://planck.nibalizer.com:5000/modules/puppet-community/puppet-nodejs

Rebase apparently did the trick? I'm happy either way :+1:


Not possible to make an excluded folder visible again

There is no way to make an excluded folder visible again.
v.1.0.6.

In the settings under "Excluded Paths" you can exclude and un-exclude any path on your phone.

Only unchecked phone root folder (and SD-card root, if availbale) listed there.
But manually excluded folder are not listed. So it is not possible to get them back.
Tested on four devices: Android 4.4.2, 6.0, 7.1, 8.0 and in Android Studio emulator.

This is like a file explorer, you can click on the phone root folder to view the subfolders of the phone root folder. And somewhere (what ever path you excluded) there you should find your excluded folder.


Added Support for angular 1.4.1

Updated old npm libraries to new ones, fixed unit tests
added support for fullcalendar 2.3 and angular 1.4
Something in angular 1.3.0.rc-4 has broke the calendar(view will not update visible unless any view or date is clicked). For the calendar to work use to timeout to render the calendar.

There has been some changes merged in before this one which may have created a conflict. @kamthamc try to pull the latest.

@ballerabdude Reverted Formatting and fixed merge conflicts

Now it looks far more better to read!! Thanks @kamthamc :)

+1 for me. I maybe don't understand part with changing internal methods like destroy, or init. But if test works then I'm ok with that :+1: .

Angular has scope.$destroy method. So, just renamed to avoid confusion. Used same name convention for init.
Thanks and Regards
Kamatham Chaitanya Kumar

On Jul 9, 2015, at 12:10 AM, Kamil Gałuszka<EMAIL_ADDRESS>wrote:
+1 for me. I maybe don't understand part with changing internal methods like destroy, or init. But if test works then I'm ok with that  .
—
Reply to this email directly or view it on GitHub.

if you try running grunt we get an angular not defined error and when I add angular to the global list it still fails. Can you take a look please?

Did you do do a bower install. The tests and demo are working which is isn't possible if angular is not defined
Thanks and Regards
Kamatham Chaitanya Kumar

On Jul 9, 2015, at 5:46 AM, Abdul Hagi<EMAIL_ADDRESS>wrote:
if you try running grunt we get an angular not defined error and when I add angular to the global list it still fails. Can you take a look please?
—
Reply to this email directly or view it on GitHub.


Is there any plan to support kotlin Multiplatform?

Is there any plan to support kotlin Multiplatform?

you may use jetbrains compose: https://github.com/JetBrains/compose-jb

No plans for this repo right now


CDEs with Image Data

Hey Patrick!
I want to utilize neural CDEs for the purpose of time series with images at different time points, as opposed to structured data.
I have seen that on the torchdiffeq repo, they use a convolutional layer in their "ODEFunc" in order to use image data. I was wondering if a similar approach could be taken with neural CDEs. I'm just not sure how to prepare the data for doing such with the interpolation and cubic spline. I was wondering if you had any suggestions for how I could go about doing this.
In addition (unrelated to the main question), I was wondering whether interpolation with the hermite cubic spline could be performed during model training, as opposed to beforehand during data preprocessing.
I would greatly appreciate any help with these questions.
Regards,
Aashish

In other words, run a neural CDE over a video.
In principle this is definitely possible. Just interpolate your data as normal; then just use a convolutional vector field as normal. (I don't think anyone has tried it in a serious way -- or if they have they haven't published their results -- so I don't know how effective this is. Try it and find out? Perhaps one might wish to use some kind of optical-flow-based interpolation?)
Of course you can use interpolation during model training; it's mathematically equivalent. For most applications it's slower though, as you're just repeatedly re-interpolating the same training data.


Update ceph_rgw_docker_extra_env to add bind ip

This patch adds passing the RGW_CIVETWEB_IP to the docker
container. This IP defaults to the value of radosgw_civetweb_bind_ip.
radosgw_civetweb_bind_ip default to ipv4.default
Without this value, the RGW containter will bind to <IP_ADDRESS>

luminous-ansible2.3-docker_cluster successfully passed, it's all that matters.


DA-142 Notification service should use OSGP client-certificate. Same …

OC-305 has changed the certificate for notification services for other domains. This pull request does the same changes only for distributionautomation.

All is well.


Open in OS' default browser

I can already choose what browser I can use by default in the operating system. While I see how it can be convenient for some people to choose a default for developing different from the OS default, I think it would be nevertheless better to just default to the system-wide default, instead of Firefox just because.
This is how I set my user preferences to "open in default browser, whichever it is at any time" in OS X, in case you want to incorporate it in the default preferences. I do not know the necessary command lines to do that in other OS.
{
    "supportedBrowsers": {
        "default": [
            {
                "osname": "posix",
                "platform": "darwin",
                "command": "open"
            }
        ]
    },

    "selectedBrowser": "default"
}

To open in your system default browser for OS X, the current View In Browser > Settings - User should contain:
{
  "posix": {
    "darwin": {
      "default": "open"
    }
  },
  "browser": "default"
}

I'm trying ST again, not by choice, and I'm trying to make it work for me. It's once again frustrating, as I'm a TextMate user, and ST just seems to do things in an odd way.
Could someone explain why this won't work (to set Safari as default) in View in Browser — User?
[         // tried with and without parenthesis  	{"browser": "safari"}  ]
In trevor's post he does the below, could someone explain why?
{    "posix": {      "darwin": {       "default": "open"     }   },   "browser": "default" }

To format code on GitHub: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet#code
macOS is POSIX compliant and runs on Darwin. In the snippet I provided, I set the option for default on a POSIX machine running Darwin to use the command open, the Darwin command for opening files on macOS. When you pass a URI to the open command, it opens the URI in your default browser.
The syntax can be seen in the View In Browser.sublime-settings file.
So the snippet I provided simply defines a command for default which allows you to set "browser": "default".
However, it also wipes out the command definitions for all the other browsers. So if you don't need an option for default, then your Sublime Text > Preferences > Package Settings > View In Browser > Settings - User should contain only:
{
  "browser": "safari"
}

If you want support for a default as well as all the other browsers, then your Sublime Text > Preferences > Package Settings > View In Browser > Settings - User should contain:
{
  "posix": {
    "darwin": {
      "default": "open",
      "firefox": "open -a \"/Applications/Firefox.app\"",
      "safari": "open -a \"/Applications/Safari.app\"",
      "chrome": "open -a \"/Applications/Google Chrome.app\""
    }
  },
  "browser": "default"
}

This will allow you to open with your default browser using control+option+v, but also open with a specific browser using control+option+s for Safari, control+option+f for Firefox, and control+option+c for Chrome.

OK, I wasn't clear. Sorry. Let me try again.
Markdown; yea, I know Markdown. I just missed the single tick in the "insert code" selector. I was quite tired, when I wrote the post. Thanks for the link.
Mac OS, POSIX, UNiX Darwin; yea, got that.
My question was directed toward why ST seemed to acting differently with "view in browser" than it does with other plugins. At least the few that I've used, and modified.
What I was expecting to work
SUBLIME-KEYMAP has this format:

[
    {"keys": ["modifier + key"], "command": "do _this"}
]

Modifying that setting would would be this:

[
   // yes I know negative commands don't exist, illustration only
    {"keys": ["modifier + key"], "command": "don't_do_that"}
]

I know that works, because I've done it. And that is what I was expecting to work in the USER file.
What I tried that failed
I expected to put the following in the USER file …

Code snippet 1

{
  "browser": "safari"
}

and override the ONE setting you built in (below)

Code Snippet 2

{
   […]
	"browser": "firefox"
}

That didn't work (duh). It sounds like your saying that when I put the first code snippet in, it COMPLETELY wipes out the default settings. Is that right?

That didn't work (duh). It sounds like your saying that when I put the first code snippet in, it COMPLETELY wipes out the default settings. Is that right?

Yep :) That is, if you put
{
  "posix": {
    "darwin": {
    }
  }
}

That wipes out all the browser settings for macOS.
But if you just put:
{
  "browser": "safari"
}

Then you should be all good. That will only override the browser key and not the macOS browser settings.
Also, this should work in your User key bindings configuration file:
[
  { "keys": [ "ctrl+alt+v" ], "command": "view_in_browser", "args": { "browser": "safari" } }
]

But the default control+option+s should open Safari by default.

Alright, thanks for the thorough response. I must have done something wrong when setting

{
    "browser": "safari"
}

I'll try it again tonight.


Issue with ise export tool in vscode

Describe the Bug with repro steps

We have a consumption logic with When A Message Is Received In A Queue action (managed connector) and the return type of this is ServiceBusMessage.

When we export this using export tool to standard logic app the connector been converted to built-in and then the response type of the action is an Array. which is fine.

But the next steps we have considering the output would be servicebus message and the tool is not able to recognize it and it just take it as a 1 message and using it wrong it next action expressions.

What type of Logic App Is this happening in?
Standard (VSCode)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response

@vinodkumarnallavade can you share the original logic apps definition, or a screenshot of the logic app?

Closing this issue as lack of information from user. Will reopen if needed


[Question] Meta Data In Custom Recipes

So I just wanted to ask how do you set the meta data for the blocks and catalyst in custom recipes?

Catalyst meta data can be set via "catalyst-meta",
Input types use "meta" as key (see the energy cell example),
but I'd recommend using the /compactmachines3 recipe copy-shape command to get the json for the types and shape.

Ok, thanks for clearing that up. :)


Error when adding an app to app group

Exception: SQL [select [rel].[id], [rel].[name], [rel].[description], [rel].[asset_code], [rel].[created_at], [rel].[updated_at], [rel].[organisational_unit_id], [rel].[kind], [rel].[lifecycle_phase], [rel].[parent_asset_code], [rel].[overall_rating], [rel].[provenance], [rel].[business_criticality], [rel].[is_removed], [rel].[entity_lifecycle_status] from [application] [rel] join [application] [self] on ([rel].[asset_code] = [self].[asset_code] or ([rel].[parent_asset_code] = [self].[parent_asset_code] and [self].[parent_asset_code] <> ?) or [rel].[parent_asset_code] = [self].[asset_code] or ([rel].[asset_code] = [self].[parent_asset_code] and [self].[parent_asset_code] <> ?)) where ([self].[id] = ? and [application].[entity_lifecycle_status] = ?)]; The multi-part identifier "application.entity_lifecycle_status" could not be bound. / org.jooq.exception.DataAccessException

Cannot recreate - can this be closed ?

Yes, this was a one liner fixed as part of #703

thanks


[DESIGN] SfProductCardHorizontal on mobile / listing view

Describe how this feature will help us improve
Actually we don't have any specified view for SfProductCardHorizontal (listing) on mobile, which is really essential to have a proper mobile view also.

View and new components added to library.

thank you @mbaran-dvnt


Foobar2000: Renaming playlists folder failed

Bug Report
Package Name: Foobar2000
Current Behaviour
Rename conflict occurs when playlists and playlists-v1.4 folder exits at the same time

Expected Behaviour
Delete playlists folder without renaming

Additional context/output

Possible Solution

System details
Windows version: 10
OS architecture: 64bit
PowerShell version: 5.1.19041.1320
Additional software: [(optional) e.g. ConEmu, Git]
Scoop Configuration

    "lastupdate":  "2022-01-05T18:31:43.4638200+08:00",
    "SCOOP_REPO":  "https://github.com/lukesampson/scoop",
    "SCOOP_BRANCH":  "master",
    "aria2-enabled":  true,
    "aria2-max-connection-per-server":  16,
    "aria2-split":  16,
    "aria2-min-split-size":  "1M"

Please make a PR!


Advice to integrate a theme developped with the quasar framework?

Hi,
We are willing to develop our frontend application with the quasar framework . Do you have any advice for us to be successful? What are the necessary conditions for a vue-storefront theme to work?
Cheers

Also, would it be possible to use your pure core library (i.e. the calls to vue-storefront-api / magento) in a frontend we would develop with quasar? I mean we might not want to build our frontend app with the webpack config you provided, it depends how we can integrate quasar in there. We are currently trying to evaluate all that, so any advice is good for us to take from you guys.

Hi! It can be pretty hard as nobody yet tried to do so. You can very easily get the idea how vs-api works reading our docs: https://github.com/DivanteLtd/vue-storefront-integration-boilerplate/blob/master/1. Expose the API endpoints required by VS/Required API specification.md
there is a module @vue-storefront/store which is just a vuex implementation of the vsapi calls; theorethicalyl You could use it in You quasar project however with the next release 1.5 - things got complicated as we;re decoupling this module to modules: https://github.com/DivanteLtd/vue-storefront/blob/master/doc/api-modules/about-modules.md

Hello, I'm having the same question. I would like to integrate vue-store-front with a Quasar project on the front part.
The backend part is normally already managed as we use Symfony API-Platform with Sylius, which is already possible to integrate with.
Thanks for further informations about this and sorry to have re-enable this issue.
Cheers

Hello, I'm having the same question. I would like to integrate vue-store-front with a Quasar project on the front part.
The backend part is normally already managed as we use Symfony API-Platform with Sylius, which is already possible to integrate with.
Thanks for further informations about this and sorry to have re-enable this issue.
Cheers


anuko-time-tracker: init at <IP_ADDRESS>06

Description of changes
Anuko Time Tracker is a simple, easy to use, open source time tracking system written in PHP.
Things done

Packaging steps

[x] Package Anuko Time Tracker
[x] Create NixOS module for a service
[x] Add release note changes

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes (no longer required)

[x] Fits CONTRIBUTING.md.

This is my first contribution to nixpkgs. Let me know if something doesn't check out and what I can do to fix it.

I intend to create a NixOS module to run this webapp as a service. That seems a bit more daunting and will take some more time to learn.

Hi @Janik-Haag , I've added a NixOS module for the Anuko Time Tracker service, a test for the module and release notes. Would you mind having another look at this PR? Many thanks!
As I expected the service took way longer to implement than the package, but I learned a lot from what was already there in nixos/modules/services/web-apps. If you can, please check if I haven't followed any known anti-patterns again.
Now I am curious on how I can use this service in my NixOS server's system configuration before this PR is merged. And even so, my servers system configuration use the latest stable release (22.11), but this PR is based on master.
I use flakes by the way.
So I figure I could add another input to my flake, pointing to my fork of nixpkgs, something like:
inputs.my-nixpkgs.url = "github:michaelshmitty/nixpkgs/anuko-timetracker";

But then I am not sure how I can use the service in my nixos configuration. Would I just do:
my-nixpkgs.services.anuko-time-tracker.enable = true;

or something like that? If that even works, how will it handle the fact that my entire server setup uses the nixpkgs input from the 22.11 branch, but this would then reference the master branch? I will experiment a bit by myself, but I'm a bit confused about how that is resolved, if it is even possible. If you or anyone else has any experience with that, thanks!

Thanks to suggestions from @robbevp I managed to create an overlay that brings the anuko-time-tracker from my nixpkgs fork into my flake, as well as import the anuko-time-tracker NixOS module for the service from my nixpkgs fork into the configuration of one of my hosts. At the end I struggled quite a bit with the fact that it would not honor the allowUnfree config setting, but I managed to get that working too:
First I added my nixpkgs fork on the anuko-time-tracker branch as an input to my servers configuration flake.nix:
{
  ...
  inputs.my-nixpkgs.url = "github:michaelshmitty/nixpkgs/anuko-timetracker";
  ...
}

Then I had to re-import my-nixpkgs to be able to set the config to allowUnfree = true; Otherwise it would not evaluate the anuko-time-tracker package even though my host configuration has nixpkgs.config.allowUnfree = true; set. This tripped me up for quite a while. I managed to fix it with some help from SO like this:
outputs = { self, nixpkgs, my-nixpkgs, ... }:
  let
    ...
    my-nixpgs-with-anuko = import my-nixpkgs {
        system = "x86_64-linux";
        config.allowUnfree = true;
    };
  in
  {
    nixosConfigurations = {
      some-hostname = nixpkgs.lib.nixosSystem {
          system = "x86_64-linux";
          modules = [
            # Bring in the package:
            ({ pkgs, ... }: {
              nixpkgs.overlays = [
                (self: super: {
                  anuko-time-tracker = my-nixpgs-with-anuko.anuko-time-tracker;
                })
              ];
            })
            # Bring in the Nixos module for the service:
            "${my-nixpkgs}/nixos/modules/services/web-apps/anuko-time-tracker.nix"
            ./machines/some-hostname
          ];
        };
    };
    ...
  }

Then in ./machines/some-hostname/default.nix I was finally able to do:
services.anuko-time-tracker.enable = true;

If anyone has suggestions on how this can be improved, I'm all ears. If not, I hope it can help someone else struggling with this.

@Janik-Haag sure, I'll revisit this asap. I recall some of the suggestions didn't work because of the design choices made by the upstream developer. I'll do what I can.

Thank you, take your time a few days more ore less won't matter at that point. If you cant resolve it the suggested way please comment why and mark it as resolved.
That's a great contribution for your first one, feel free to make more :heart:
You can also ping me in the future for reviews and merges and also if you have any questions please ask them.

@Janik-Haag thanks for pinging me on this. I've rebased, updated the package to the latest upstream version and addressed the suggestions by @SuperSandro2000 where I could. I've tested this on some of my servers and this seems to work well. I don't think there's much more that can be done but I'll be happy to maintain this and fix issues when they are opened. If this goes well, I'd be happy to look into maintaining more packages.

One small last nitpick, can you split the module in its own commit called nixos/anuko-time-tracker: init and the package init in anuko-time-tracker: init at <IP_ADDRESS>76? After that once CI ran through successfully I'm happy to merge this. Thank you for your contribution, btw this looks great for your first module :heart:
Happy to review your future PRs :D

One small last nitpick, can you split the module in its own commit called nixos/anuko-time-tracker: init and the package init in anuko-time-tracker: init at <IP_ADDRESS>76?

Sure, I'll do that. Thanks for your time!

CI run was successful, and @michaelshmitty made the suggested changes and I couldn't find anything to improve anymore.


WIP: Add Building Outside Convox doc

Release Playbook

[ ] Rebase against master
[ ] Code review
[ ] Merge into master
[ ] Verify changes at http://site-staging.convox.com
[ ] Promote release on site-production in the convox/production Rack

 document how to use apps made of pre-existing images

Closing without merge.


@onu-ui/nuxt not published to npm

Description
I wanted to try out the onu ui nuxt module, but it looks like it is not published.
I tried to install the package (example from docs below), searched the npm registry, but it is not there.
pnpm i @onu-ui/nuxt -D

Am I missing something?

Sorry for that, i'm not complete it, and i not recommend use it. (WIP)


Question: How to setup networking with doppio in browser?

I have read through this wiki page and through src/natives/java_net.ts, but I am not fully certain how this works.
The image on the wiki page is not loading, which may have held some clues.
If I understand right, whenever a socket connection to host:port is requested from Java land, that gets converted to a ws://host:port request. Hence, if I need to connect to xyz.com:80 via HttpURLConnection, I will need to first run a websocket server (websockify) on xyz.com:80.
Alternatively, I could run websockify on my own host and configure the http proxy in Java Land to point to this, so that all http requests get routed to my host, and then proxied to the final destination.
Is the above right?
Thanks!

If I understand right, whenever a socket connection to host:port is requested from Java land, that gets converted to a ws://host:port request. Hence, if I need to connect to xyz.com:80 via HttpURLConnection, I will need to first run a websocket server (websockify) on xyz.com:80.

Yup, you got it right.

Alternatively, I could run websockify on my own host and configure the http proxy in Java Land to point to this, so that all http requests get routed to my host, and then proxied to the final destination.

We haven't implemented it that way, but yes, this is possible.
The current networking code needs to be tested/cleaned up/updated to use the latest websockify. I admit that I have let it languish for the past couple of years, mainly because I never set up automated testing with websockify. Once we have automated tests that fail, it should not be too difficult to fix the implementation.

Thanks for the detailed response, as always! :+1:

Here are actual instructions from our 2014 demo. That's how it is supposed to work.


Allow changing NickJS options after the initial constructor

Please enable changing NickJS options after the initial constructor. Specifically, I would like to modify:

loadImages flag
blacklist/whitelist
(and the print* flags would be nice, too)

Motivation: I would like to use a special configuration just when logging in

Login page requires loadImages=true and no blacklists
but other pages are OK with loadImages=false and blacklists (and load faster without images & JS files)

Hi Leftium, sorry for the delay.
This is not officially supported but you can do nick.options.loadImages = false and similar. This will only work on new tabs, so close your tab with tab.close() and open a new one. It should use the new options. Don't worry, cookies are kept between tabs.
Can you confirm this works? Thanks


[Bug]: Show Context in Cards

Describe the bug
My Flashcard

My Cram section shows the Author in context, which is not good for reversed card

The context is already showing me that Robert Greene is the Author
To Reproduce
Ive described the bug above. Please check
Expected behavior
The context should not give
Screenshots
No response
OS of your device
Win10
Obsidian version
v1.5.3
Plugin version
v1.12.4
Installer version
No response
Additional context
No response
Config file
No response

Hi @SyedIrtiza555 context can be turned off within the plugin settings

But the issue is that I need to see the context but, it reveals the author
heading but not the first heading. It should behave like it is doing with
the first heading
On Tue, Jul 23, 2024 at 7:48 AM Ronny Zulaikha @.***>
wrote:

Hi @SyedIrtiza555 https://github.com/SyedIrtiza555 context can be
turned off within the plugin settings
image.png (view on web)
https://github.com/user-attachments/assets/ec6b3ded-f9a3-4104-b128-e4c6f1c45425
—
Reply to this email directly, view it on GitHub
https://github.com/st3v3nmw/obsidian-spaced-repetition/issues/1026#issuecomment-2244148288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNDYKV5E6UNFYRI5F5SJY3ZNXABBAVCNFSM6AAAAABLHXGP4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBUGE2DQMRYHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I haven't understood what you are after.
Please write exactly what you are after for each of the cards
Card 1
Context:
Question:
Answer:
Card 2
Context:
Question:
Answer:

The cram session is showing:
Book Name>Author>
Robert Greene
But, I want it like this,
Book Name>.....
Robert Greene
Although, the cram session is behaving as expected in the case of Summary i.e:
Book Name>.....
this this and this
But, in case of author it is showing me the frontside of the card, which I dont want to see

I think you are using the following text within the markdown file:
#flashcard

# Summary
?
this this and this

# Author
??
Robert Greene

Q1. Is that correct?

So the problem is that you are seeing the following:

And you don't want to see the > Author displayed as part of the context.
Q2. Is that correct?

In typical usage the context is not used as part of the flashcard text. I.e. typical usage is separating out the context information from the flashcard text (the blank line between the # Context that doesn't give away the answer and Author is important:
# Context that doesn't give away the answer

Author
??
Robert Greene

The ?? signifies a "bidirectional" flashcard, meaning that there are two cards generated.
However, the context displayed is the same for both cards. Therefore you can't put # Author as part of the flashcard text.
Hope that helps

Now you understood my problem.... I'll try '?'. Thankyou.
On Thu, Aug 1, 2024, 6:25 AM Ronny Zulaikha @.***>
wrote:

I think you are using the following text within the markdown file:
#flashcard
Summary
?
this this and this
Author
??
Robert Greene
Q1. Is that correct?
So the problem is that you are seeing the following:
image.png (view on web)
https://github.com/user-attachments/assets/9f6ff840-36a3-4649-8977-0421dcf89802
And you don't want to see the > Author displayed as part of the context.
Q2. Is that correct?
In typical usage the context is not used as part of the flashcard text.
I.e. typical usage is separating out the context information from the
flashcard text (the blank line between the # Context that doesn't give
away the answer and Author is important:
Context that doesn't give away the answer
Author
??
Robert Greene
The ?? signifies a "bidirectional" flashcard, meaning that there are two
cards generated.
However, the context displayed is the same for both cards. Therefore you
can't put # Author as part of the flashcard text.
Hope that helps
—
Reply to this email directly, view it on GitHub
https://github.com/st3v3nmw/obsidian-spaced-repetition/issues/1026#issuecomment-2261763193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMNDYKRGJUJHLTTGOOT4Q33ZPGFA5AVCNFSM6AAAAABLHXGP4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRRG43DGMJZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Fix build errors

There is also a bug in https://github.com/KhronosGroup/SPIRV-Tools/blob/4c239bd81bf79337e4a0cfe1c9da9caf1b25108a/test/fuzz/transformation_move_instruction_down_test.cpp#L610 if I recall correctly.

It seems tests are failing because of the assertion in the fact manager. https://github.com/KhronosGroup/SPIRV-Tools/blob/51b826140a63b74c93cfe8074ef643dec026adb4/source/fuzz/transformation_flatten_conditional_branch.cpp#L581-L582 should be moved after the call to InvalidateAnalyses.
Alternatively, you could comment this out as well as disable the corresponding tests and I will resolve this issue when I refactor the fact manager as in #3698.

Thanks - I am working on a fix.  Seems like these PRs landed in an unfortunate order :)


linkis-gateway有共享session问题

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
gateway部署多个节点导致enginemanager创建engine失败，失败原因是enginemanager recevieAndReply 方法内request.user为空， 抛异常了
Describe the solution you'd like
A clear and concise description of what you want to happen.
把user_id放在cookie或者引入redis，mysql
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Yes, now Gateway does not do session sharing, if you are interested, you can do it together


Enable style check for cudamapper

Would just require the CMakeLists.txt in cudamapper folder to have one more line as specified here https://gitlab-master.nvidia.com/genomics/GenomeWorks/blob/master/cudapoa/CMakeLists.txt#L67

Added with #180


fixed issue6

[x] Fixed issue #6
[x] Created env file.

[x]  Fixed issue #6
[x]  Created env file.

Run this command on your branch
git commit --amend -m "Connected to Mongo ATLAS"
git push origin <branch-name> -f

[x]  Fixed issue #6
[x]  Created env file.

Run this command on your branch
git commit --amend -m "Connected to Mongo ATLAS"
git push origin <branch-name> -f

Great job 👍

Great job 👍

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code, design

@all-contributors please add @karunkarthik-git for code, design

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for code

@all-contributors please add @karunkarthik-git for design

@all-contributors please add @karunkarthik-git for design


Refactor json codecs from parse-json-processor to common

Description
Refactor json codecs from parse-json-processor to data-prepper-plugins/common for reuse.
Issues Resolved
Resolves #[1082]
Check List

[ ] New functionality includes testing.
[ ] New functionality has a documentation issue. Please link to it in this PR.

[ ] New functionality has javadoc added

[X] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This refactoring for codecs is not required.


Please add a license

Hello! Could please consider licensing your code under a permissive license?

Thanks for your reminding, we have added a license.

Thank you very much!


Add the uri to the error message of the xaml loader

Add the uri to the error message, that way nested load errors can be diagnosed much quicker.

Done.


Make stasis bed power toggleable

About the PR
Fixes #18167
Why / Balance
Was requested in #18167 by Emisse
Media

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

Changelog
:cl:

tweak: Stasis bed can be toggled on or off!

But you'd have to move patients around anyway, since stasis bed doesn't provide medical bed stats

this is still a valid point, people will move them anyway for the passive healing
it takes the same amount of time to click and click drag someone as to alt click it then alt click after theyre healed

Having ready to go stasis is too valueable in medical, even if it means that you drop patients on floor IC.


Disable close dialog by back button option

I know there is an option to barrierDismissible, but it only prevents tap on the screen, if the back button got tapped, the dialog is closed. Is there any option that makes back button disabled when the dialog show up? Btw this plugin is really awesome :)

Thank you @dennbagas, surely I would look into your request.

i have same idea

I proposed a pull request for this issue, please review it

sorry @dennbagas actually I was out of station, I will do it asap

I have fixed the issue in latest update pub v1.2.0, library will be updated soon, thank you!


lama cleaner

It can't find lama cleaner, it tells me in read the readme to install manually. Any ideas on what could be occurring.

Go here and install via pip or there's a Windows 5-minute installer you can purchase for $5.
I had issues and errors with the pip method so finally used the purchased installer and it worked.
https://lama-cleaner-docs.vercel.app/

Lama Cleaner can be installed by pip.
For Windows:
 .\venv\Scripts\activate
pip install lama-cleaner
deactivate


Parameter Count seems failed in destructuring object parameters

Describe the bug
Parameter Count seems failed in destructuring object parameters
To Reproduce
Steps to reproduce the behavior:

What is the exact code you are analyzing?

function dummyFuncOne({ param1, param2, param3, param4, param5 }) {
  return 'One'
}

const dummyFuncTwo = ({ param1, param2, param3, param4 }) => {
  return 'Two'
}

What is the output you are getting?

-  rrd ~ parameter count
   src\views\Test\parameter-count.js
   👉 Max number of function parameters should be 3. See: https://vue-mess-detector.webmania.cc/rules/rrd/parameter-count.html
   function dummyFuncOne has 5 parameters 🚨

   src\views\Test\parameter-count.js
   👉 Max number of function parameters should be 3. See: https://vue-mess-detector.webmania.cc/rules/rrd/parameter-count.html
   function dummyFuncTwo has 4 parameters 🚨

Expected behavior
Correctly count the destructured object parameters.
Screenshots

Used version number of vue-mess-detector: 0.57.0
Used version number of node & yarn<EMAIL_ADDRESS><EMAIL_ADDRESS>

@David-Pena it is like that by design, isn't it?
From the code maintainability viewpoint they have 4 and 5 parameters.
I understand the issue, but I want to be sure how we think about this and clearly state that in the docs.

it is like that by design, isn't it?

I don't think so, right now I don't remember completely but I think its just misssing the correct logic to identify destructured params

From the code maintainability viewpoint they have 4 and 5 parameters.

I think destructured props shouldnt be limited. In this case its only one param so it shouldnt be reported, what do you think? @rrd108

When the param is an object - I think still there is a reasonable limit of non optional properties
but for now I am ok handling it as one

When the param is an object - I think still there is a reasonable limit of non optional properties

Lets create a separate issue for that change to the rule

Do you have any progress in here? @rrd108
@zymbth would you like to send a PR?

@David-Pena Sorry for the late reply, just got back from vacation. I've been a bit busy lately, try to contribute in the future.

This is solved in https://github.com/rrd108/vue-mess-detector/commit/bbded51c4de87bb8e0f24c90e334b01b570e5b43


refactor(api): import ts-reset in entrypoint

It simplifies the config and slightly improves startup times in
development.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

It's no-longer used. The only thing it was doing was sourcing all files and this PR removes the need.

@naomi-lgbt it's ts-node that's the problem here. If you use tsc it works just fine (it compiles everything), but ts-node tries to save startup time by compiling lazily. Which is great... until you need something that's not directly required.


[Feature Request]

🚀 Feature Request: Implement Odd-Even Sort
Is your feature request related to a problem? Please describe.
I’m frustrated when sorting algorithms like bubble sort or insertion sort are used in our application because they can be inefficient for larger datasets. The need for an efficient, parallelizable sorting algorithm is evident, especially when dealing with large arrays. An odd-even sort could provide a simple yet effective solution.
Describe the solution you'd like
I would like to implement the Odd-Even Sort algorithm in our project. This algorithm works by repeatedly comparing adjacent elements in the array. The sorting is done in two phases:
Odd Phase: Compare elements at odd and even indices, swapping them if they are out of order.
Even Phase: Compare elements at even and odd indices, swapping them if they are out of order.
This process is repeated until no swaps are needed, indicating that the array is sorted.
Would you like to work on this feature?

[x] Yes
[ ] No

sir, could you please increase the level for this issue?

sir, could you please increase the level for this issue?


added sparse.map_values

Second part of #39507
adds a sparse.map_values operation for sparse Tensors in analogy to ragged.map_flat_values.
I've copied the general structure of the code from the ragged analog, in particular the function is currently decorated with @dispatch.add_dispatch_support, of which I don't know the purpose.
When multiple sparse Tensors are supplied as part of a map_values invocation, the current implementation checks that all indices and shapes match. This test might be a bit more expensive than the corresponding one for ragged tensors, since there are as many indices as there are values, so it might make sense to add an additional parameter to disable checks.
I have opted against this, because

since the function takes arbitrary kwargs, this might introduce incompatibilities
It breaks the interface analogy with the ragged version
I expect that people who really need this last bit of performance, and are able to prove that in their code these checks always success, can write their custom code if this is really necessary.

@ngc92  Can you please check @edloper's comments and keep us posted. Thanks!

Sorry for the git spam, I screwed something up when I merged the tf changes back into my branch.
@edloper All your recommendations, except for the name of the function, should be in the code now
I do think the name is good as it is though, because it keeps the consistency with ragged tensors. And I would expect potential users of this method to have at least a very broad understanding of what a sparse tensor is.
It might make sense to highlight the
Note in particular that even though `tf.add(0, 5) != 0`, implicit zeros
  will remain unchanged. However, if the sparse tensor contains any explict
  zeros, these will be affected by the mapping!

in the documentation, at least I would expect me as a first user of the method to not necessarily think of that, even though it makes perfect sense in hindsight.

Can you check sanity build please? Looks like a pylint issue

@ngc92 can you please check build failures.

Pylint still fails
FAIL: Found 9 non-whitelisted pylint errors:
tensorflow/python/ops/sparse_ops_test.py:193: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

tensorflow/python/ops/sparse_ops_test.py:194: [C0301(line-too-long), ] Line too long (83/80)

tensorflow/python/ops/sparse_ops_test.py:194: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

tensorflow/python/ops/sparse_ops_test.py:195: [C0301(line-too-long), ] Line too long (90/80)

tensorflow/python/ops/sparse_ops_test.py:195: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

tensorflow/python/ops/sparse_ops_test.py:205: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

tensorflow/python/ops/sparse_ops_test.py:206: [W0311(bad-indentation), ] Bad indentation. Found 8 spaces, expected 6

tensorflow/python/ops/sparse_ops.py:2774: [C0301(line-too-long), ] Line too long (82/80)

tensorflow/python/ops/sparse_ops.py:2812: [C0330(bad-continuation), ] Wrong continued indentation (add 30 spaces).

Failed example:
    tf.sparse.to_dense(tf.sparse.map_values(tf.add, s, 5)).numpy()
Expected:
    [[6 7 0]
     [0 9 0]
     [6 0 0]]
Got:
    array([[6, 7, 0],
           [0, 9, 0],
           [6, 0, 0]], dtype=int32)

The docstests are failing. Let's remove numpy() and consider the output as tensor. You can use ... to hide parts of the output which are not relevant.

@ngc92 Can you please check @mihaimaruseac's comments and keep us posted. Thanks!

@ngc92  Any update on this PR? Please. Thanks!

update on this PR? Please. Thanks!

I'll try fixing it in the merge commit.


Inline docs: audit code blocks to get rid of extraneous spaces

This block has one too many spaces. The entire src should be checked for this.

Closing in favor of the central API doc tracking issue #447


Redesign: Update size+position panel

Feature Description
See Figma

[ ] Remove "Set background"
[ ] Update inputs
[ ] Update toggles
[ ] Replace tooltips

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@embarks, I discovered some minor visual issues with both the panels and the numeric inputs in #6591, so you might want to hold off with this one until that one is merged - it just makes sure we can align and style things as they're supposed to. Or copy the edits in to get going and handle it in the merge. 🤷

This will also collide a bit with the Page background panel which uses the flip toggles -- I'll be replacing the general flip controls in the same place most likely.

Verified in QA


2020-11-17 GUARD-811 Server-Unavailable-Errors

Summary
Customer's Magento server randomly throws different HTTP errors for different HTTP endpoints during SV's full inventory sync:
System.Net.WebException: The request was aborted: The request was canceled.
System.Net.WebException: The remote server returned an error: (503) Server Unavailable
System.Net.WebException: The remote server returned an error: (502) Bad Gateway.
After the fix, access library will retry HTTP request if faces these errors.

Made combined PR: https://github.com/skuvault/magentoAccess/pull/26/commits


[VirtualBox] emulators/virtualbox-ose-additions 6.1.18

⚠

VirtualBox 5.2.x is no longer supported!

– re: https://www.virtualbox.org/wiki/Downloads, support for 5.2.⋯ ended in July 2020.

Live boot of helloSystem should, please, include version 6.1.18 (or greater) of the guest additions.
https://www.virtualbox.org/wiki/Changelog-6.1#v18
https://www.freshports.org/emulators/virtualbox-ose-additions/#history
https://svnweb.freebsd.org/ports?view=revision&revision=567950
https://github.com/helloSystem/ISO/blob/134f232aa0aefcf41b5742e7fc3a9c53d861bcc4/settings/packages.common-13#L25
https://github.com/helloSystem/ISO/blob/d8764a2eef5bf7e5f410610cac1b6df05ef267b1/settings/packages.common-12#L24
If latest will remain out of scope for helloSystem:

@crees please, might you add emulators/virtualbox-ose-additions – 6.1.18 – to the list of things that you occasionally build with helloSystem in mind?

Thanks

We are using whatever version is in last quarter's packages. We cannot use quarterly because it is a moving target that can break at any time.

Thanks, this was mentioned nineteens days ago.

Please retry with the latest experimental build; VirtualBox Guest Additions are now preinstalled, activated, and working :100:


Can't filter symbol list with "g" as first letter

It pulls up the goto dialog, and that's not very cool.
It probably happens with other ui hotkeys too.

This can be fixed by doing #2414

probably related to #1533

Probably will be fixed by #2217

Fixed as of the sidebar refactor 2.4.2921


[TACHYON-296] Deploy a specific commit of Tachyon via Vagrant

This PR adds the new feature to deploy/vagrant via an added tachyon_version.yml API.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2834/
Test PASSed.

Thanks! Could you please also add some document on how to use this new feature?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2835/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2836/
Test PASSed.

@calvinjia comments resolved.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2847/
Test PASSed.

Removed mvn package from EnvSetup.rb since ufs/*/init.sh have taken care of it. In EnvSetup.rb, we only check out the commit.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2851/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2852/
Test PASSed.

+1 LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2856/
Test PASSed.

I found that each vagrant command will execute Vagrantfile.... I assumed only vagrant up will load this, the git clone logic is included in Vagrantfile, so every time you type vagrant xxx, git clone will be run. I'll come up with a new PR.


Improve support for classes, labels and checkboxes

PR coming soon

Awesome! Thanks for your contribution. Feel free to comment if you have any questions or need some feedback. @czosel Will take a look at it.

I'll close this since we have basic support for checkboxes now.


Remove an owner from CODEOWNERS that is no longer part of Azure

I'm going to override check-enforcer because there's no way CODEOWNERS changes would affect a perf pipeline that shouldn't be running to begin with.

/check-enforcer override


swatch color vs square color

Hello, the swatch color and the picker color doesn't match. See screenshot.
The problem is that inside the square the color is brighter then what is shown in the swatch.
Is there a way to fix this?

This is because lightness is not shown in that color picker. I have attempted a fix in v3.0.0 which should make the color closer to the swatch.


Scrolling down is not as smooth as scrolling up

I'm using ConnectBot v1.9.2-50-g94c1907 on a Samsung Galaxy Note 4 with Android KK v4.4.4 and when scrolling long outputs (such as ls -Rl output) I noticed that the text scrolls smoothly when going down (i.e. when you keep your finger pressed, the text goes fast just by moving the finger a few pixels away).
Unfortunately the same is not true - or at least it's not the same - when scrolling up (i.e. if you keep the finger pressed you have to travel a longer distance to make the text scroll and it does not scroll continuosly but "in jerks").

This is especially noticeable on long history texts: you go quickly up, but not so quickly down.
Here's a video recorded while scrolling.

I have also noticed this issue, and assumed that the fast scrolling upwards was a bug - certainly it is too fast for me, scrolling up several pages when I just intend to scroll a few lines. And yes, scrolling downwards is very slow.
For me, ideally scrolling should match other apps, e.g. web browsers, where the text moves in sync with the finger, but if the finger flings, it should keep scrolling with some momentum.
It would sometimes also be useful (maybe as an option when defining the scrollback buffer size) to have a usable scrollbar.


Enable access to x-ms-client-request-id

As per the docs:

Provides a client-generated, opaque value with a 1 KB character limit that is recorded in the analytics logs when storage analytics logging is enabled. Using this header is highly recommended for correlating client-side activities with requests received by the server.

Client request IDs are already generated by the CLI. What is the scenario in which you want to manually control them?

Pretty much for the reason the docs state:

correlating client-side activities

Let's say I've got a backup script.  In that scenario I would like to be able to correlate messages generated by the CLI with a given execution of the backup script.
Also, for example, looking at ps output on a Linux box, it might be a nice thing to be able to see the process correlation.

cc/ @johanste

The title reads like you want to "read" the generated client request ID rather than injecting your own ID. Am I right?

Apologies for the badly worded title.
I was thinking more the opposite, i.e.az foobar --request-id ij2mv03vm2 or whatever.

The telemetry logic is tightly correlated to the ID and we need to guarantee this ID to be generated randomly between calls.  Should we open up the generating logic and a flaw is introduced in the customization of the generator the CLI behaviour will be disrupted.
Please also be noted:

If two subsequent write requests (two PUTs, POSTs, or DELETEs) have the same id, the Network Controller assumes that last request is a retry and returns the same result it returned for the previous request.
Source

As in CLI, we're the client to consume this ID. The consumer of the CLI should in term adopt their own logic to coorelate their command execution and loging.
May I understand your broader scenario? There is probably a different solution to it.

lack of response , closing


Get epoch time of upload

When using .getChannelVideos would it be possible to also return a numerical time of upload (for example, Epoch) as it tends to sometimes get older videos, which can break something getting the latest video (for example, im working on a discord bot to auto post uploads, that compares the video to the last video, if it has a different ID it will update, but if a video gets privated, or it returns an older video it will post that video aswell)
Sorry if this doesnt make sense, basically just return a numerical upload time

same question here, is this possible?

Hi unfortunately we wouldn't be able to implement this, as the channel videos page only contains relative timestamps for the uploads, e.g. 4 hours ago, 2 days ago or 6 weeks ago. To get the exact timestamp you'll need to either use YouTube's official Data API (requires an account and has ratelimiting) or use something like ytdl-core to scrape the watch page for that video.


UK African and Winter uniforms plus derandomisation

Splits UK gear definitions
Adds UK African uniforms
Adds UK Winter uniforms
Derandomises UK loadouts
Already split US weapons into their own file since the UK loadout makes use of some of them
Fixes Japanese composition to call correct Medic case name

Closes #17
Closes #7

Err, I pre-merged the changes from the main branch into my own to fix merge conflict in advance but for some reason it's now listing all those changes as new. So the stuff from Feld's and Phil's PRs are now showing up here as well, please ignore those...


Full controll over parsing response data

When users use a custom vendor provider, they have minimal control over parsing the response, which can only be touched on by implementing parse_response_data.
In avante, response data will be processed by parse_and_call at first, assuming that all of the streaming chunks have the standard structure that can be matched sequentially with the keywords event and data. While things are not always true. In AWS bedrock, the streaming response is constructed in such format:
event{"bytes":"eyJ0eXBlIjoibWVzc2FnZV9zdGFydCIsIm1lc3NhZ2UiOnsiaWQiOiJtc2dfYmRya18wMU1CVmVzdXAzMWdFNTdHZWpZZzNjb3QiLCJ0eXBlIjoibWVzc2FnZSIsInJvbGUiOiJhc3Npc3RhbnQiLCJtb2RlbCI6ImNsYXVkZS0zLTUtc29ubmV0LTIwMjQwNjIwIiwiY29udGVudCI6W10sInN0b3BfcmVhc29uIjpudWxsLCJzdG9wX3NlcXVlbmNlIjpudWxsLCJ1c2FnZSI6eyJpbnB1dF90b2tlbnMiOjI0NzgsIm91dHB1dF90b2tlbnMiOjJ9fX0=","p":"abcdefghijklm"}

As you can see, the real response is encoded as base64 and placed in bytes, which is stopped by:
https://github.com/yetone/avante.nvim/blob/2d87dff33bd4cd47d19056562ac9c9ee06d76cb8/lua/avante/llm.lua#L553
Can we have a full control?
cc @aarnphm

oh wow they don't follow sse

The problem with this is there is no way to set the current_event_state correctly, if users can have custom on_callback like this. I'm not sure how to approach this.

We still need parse_and_call. Perhaps we could have a data_match function that allows users to customize their data matching.

or we can just expose the whole parse_and_call for users to modify the events themselves.

See wiki and #114


[cli_config] Pub badges

Now that the first version is published, add more badges.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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>:
0.0%

Covered Lines:
242

Relevant Lines:
242

💛  - Coveralls

Thanks! 🙏


leaflet canvas example

This is completely insane, but it shouldn't be too difficult to render these client-side, draw the results to canvas per tile, and add them to a Leaflet canvas layer.

this should be its own thing


go/packages: overlays cause loading of unneeded dependencies

Forked from https://github.com/golang/go/issues/32457.

The similar issue reproduces in golangci-lint: after updating x/tools from 685fecacd0a0 to 755ce86c7629 go/packages started loading dependencies when not needed.
As I understand the regression was introduced in this overlays commit.
And this code adds all dependencies into needPkgsSet and after this to needPkgs, which are used to run go list on them:
	// Do another pass now that new packages have been created to determine the
	// set of missing packages.
	for _, pkg := range response.Packages {
		for _, imp := range pkg.Imports {
			pkgPath := toPkgPath(imp.ID)
			if _, ok := havePkgs[pkgPath]; !ok {
				needPkgsSet[pkgPath] = true
			}
		}
	}

/cc @matloob

Unfortunately, it's not always possible to avoid loading extra dependencies, and go/packages can't tell if they're unneeded unless the transitive set of dependencies is not changed at all.
I'll send a CL to avoid loading extra dependencies if the transitive set of dependencies is not changed. But I don't think this problem can be solved satisfactorily until go list starts handling overlays itself...


[Automated] Update eventing-kafka-broker-eventing nightly

/assign @knative/delivery-wg-leads
/cc @knative/delivery-wg-leads
Produced via:
for x in eventing-crds.yaml eventing-core.yaml; do
  curl https://storage.googleapis.com/knative-nightly/eventing/latest/$x > ${GITHUB_WORKSPACE}/./third_party/eventing-latest/$x
done

Details:

/retest-required

/lgtm
On Fri 4. Feb 2022 at 12:05, Knative Prow Robot @.***>
wrote:

New changes are detected. LGTM label has been removed.
—
Reply to this email directly, view it on GitHub
https://github.com/knative-sandbox/eventing-kafka-broker/pull/1859#issuecomment-1029886002,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABGPTSVFGJMUF6N3Z25ETTUZOXG3ANCNFSM5NOTFZHQ
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Sent from Gmail Mobile

/test pull-knative-sandbox-eventing-kafka-broker-channel-reconciler-tests-ssl

/retest

/retest-required


util: use slices package for contains

use slices package to check whether the slice contains the element or not.

@mergifyio queue


feat: add inline documentation on all namespaced functions

For some of them, I also added example snippets.
Generally speaking, I think we should aim towards adding inline documentation on all public APIs—this really, really improves the developer experience when we don't already know about something.

I'm good with this, really nice!

@aidan-casey did you plan on figuring out that PHP CS Fixer rule? Just wanna make sure this can get merged soon :)

I'm just going to merge this, we can finetune with new PRs later