Fixed the problem of unable to stop ble-scan from the second time on …

Fixed the problem of unable to stop Bluetooth scanning from the second time on Linux platform

Could you update the summary to describe the issue and solution? existing summary of Fixed the problem of unable to stop Bluetooth scanning from the second time on Linux platform does not actually explain what the problem was and what the approach of fixing is.
From what I can see, the fix is to reset isStopping back to false in the Stop method ... why is this correct and why is this correct to do before the delegate scan complete callback is called? Should the stop be a RAII that sets it to true at start and false at end instead? Why is it called isStopping instead of stopScanExecuting for example to be precise about its intended usage?

The fix may be correct - since it seems once you say StopScan you can never reset mIsStopping to false again.
However I am unsure about intended design - maybe this was supposed to create a new scan for every new scan and never re-use a scanner (even though that is odd...). Maybe instead Init should ensure that previous resources are released and should also initialize mIsStopping to false.

It looks like BleManagerImpl::InitiateScan will do Init and StartScan on mDeviceScanner.
To me this seems like Init should make sure that stopped status is cleared (and also ensure it cancels any current scan).

It looks like BleManagerImpl::InitiateScan will do Init and StartScan on mDeviceScanner.
To me this seems like Init should make sure that stopped status is cleared (and also ensure it cancels any current scan). Or more likely make ShutDown set mIsStopping to false rather than the existing change in Stop.

Yes, mIsStopping can indeed be cleared during Init or Shutdown, but from my point of view, the role of mIsStopping is more to prevent reentrancy while waiting for PlatformMgrImpl().GLibMatterContextInvokeSync(MainLoopStopScan, this), so I think the value should be cleared immediately after the function ends.

It looks like BleManagerImpl::InitiateScan will do Init and StartScan on mDeviceScanner.
To me this seems like Init should make sure that stopped status is cleared (and also ensure it cancels any current scan). Or more likely make ShutDown set mIsStopping to false rather than the existing change in Stop.

Yes, mIsStopping can indeed be cleared during Init or Shutdown, but from my point of view, the role of mIsStopping is more to prevent reentrancy while waiting for PlatformMgrImpl().GLibMatterContextInvokeSync(MainLoopStopScan, this), so I think the value should be cleared immediately after the function ends.

This explanation should be as part of the comment when this is set to false. This also feels a bit like a code-smell for such coupling, however I have no immediate ideas on how to lay things out to not feel as messy.

It looks like BleManagerImpl::InitiateScan will do Init and StartScan on mDeviceScanner.
To me this seems like Init should make sure that stopped status is cleared (and also ensure it cancels any current scan). Or more likely make ShutDown set mIsStopping to false rather than the existing change in Stop.

Yes, mIsStopping can indeed be cleared during Init or Shutdown, but from my point of view, the role of mIsStopping is more to prevent reentrancy while waiting for PlatformMgrImpl().GLibMatterContextInvokeSync(MainLoopStopScan, this), so I think the value should be cleared immediately after the function ends.

This explanation should be as part of the comment when this is set to false. This also feels a bit like a code-smell for such coupling, however I have no immediate ideas on how to lay things out to not feel as messy.

Actually looking at the code, the MainLoopStopScan is called to isStopping false, so having this cleared in ShutDown does sound like it would work and prevent reentrancy problems. Name is still odd ("currently stopping vs fully stopped").
Maybe we should have a state enum variable instead with states like Uninitialized, Initialized, Stopping, Stopped or something like that.


[fabric-1.16.x] Unable to place any block unless its placed on the top side of a block

Unable to place any blocks on a full block on the sides.
 Sorry about that.


config: add control plane config identifier

Description: Adds support for specifying control plane identifier in Discovery Response and displays it in /configdump( we initially discussed to display in /configdump but later thought it would be as text stat since text stat is taking time I am currently displaying it in `/configdump' and once the text stat is available, will publish it as text stat also).
Risk Level: Low
Testing: Added automated tests
Docs Changes: Added
Release Notes: Updated
Fixes #5770

@htuch  Sorry, I realized the original PR https://github.com/envoyproxy/envoy/pull/6317 is closed because I accidentally removed the branch it seems. I only realized today when I started working on this again - I will continue to work on this next few days and will let you know once it is ready for
review.
Sorry you have to look at the old PR for the context (as Git is not allowing me reopen the PR)

@htuch There are primarily two comments left in the previous PR. Here is the status of both of them

Make ConfigTracker intelligent and let it manage config to avoid static stuff in grpc_mux_impl. Here is the comment history https://github.com/envoyproxy/envoy/pull/6317#discussion_r273558939. I have implemented this and is ready for review. With this change, all static stuff has been removed.
Make association with cluster/listener so that if two clusters/listeners point to same gRPC cluster and they can potentially talk to two different management servers - current implementation overwrites the last connection for the gRPC service because it is tracking at the gRPC service level. Here is the comment history https://github.com/envoyproxy/envoy/pull/6317#discussion_r273564138.
I looked at it and it seems we have wire-in the the concept of subscription_parent or some thing which tells who is creating this subscription like cluster/listener and add it in the proto. I was wondering since if we solve the #2943 this is not needed , Do you know if there are any plans to fix it in near future? or should we bring in the concept of subscription parent (or whatever we decide) and move forward?

@ramaraochavali for #2943, we need to have this rolled forward (CC @dmitri-d). Otherwise, what we have in this PR is not going to be a reliable solution. It's OK to rely on this when it does merge though I think, but it's not the most robust thing (e.g. what if we decide to multiplex N:2 streams in the future, rather than N:1). It would be reasonable to merge what is in this PR once #2943 arrives as a first step with a TODO in case this happens.

@htuch when you say

I'm happy to have a simple version for now in terms of implementation that only deals with what we have today, but the API should be expressive enough to deal with anticipated future use cases.

Are you saying we should bring in some simple concept in API to handle this case for now?

@ramaraochavali I'm saying I'd like the API to be expressive enough to deal with a scenario in which we decide one day to create two streams for an EDS management server config source instead of one.

@htuch Fixed the index issue. Regarding

API to be expressive enough to deal with a scenario in which we decide one day to create two streams for an EDS management server config source instead of one.

What do you think about adding some thing similar to stream_identifier to the ConfigSourceControlPlaneInfo? So if we decide to create n streams for a grpc_service we can treat them as n config sources instead of one.

@ramaraochavali +1 to stream identifier, you just need to have a stable way to maintain this as different streams come and go.

I will work on this in few days.

@htuch I have been busy with other stuff so not able to address review comments. Can you please re-open this PR - I am not able to?

@ramaraochavali done.

/nostalebot

@htuch Thank you. Can you also add no stalebot label - looks like you are refactoring the whole subscription factory stuff and might take more time fore me to merge that stuff.

@ramaraochavali I think https://github.com/envoyproxy/envoy/pull/7200 will land in the next day or so. I'll set this label, please let me know if this should be closed out at any point.


Offline mode

I currently want to create offline mode where by users can ad songs or music to their library immediately they add to their library they can play those songs in offline mode.
How can I achieve that.

you can set cache in application like this ：
   //set cache
        String destFileDir = Environment.getExternalStorageDirectory().getAbsolutePath() + "/ExoCacheDir";
        ExoDownload.getInstance().setOpenCache(true); //open cache
        ExoDownload.getInstance().setShowNotificationWhenDownload(false);
        ExoDownload.getInstance().setCacheDestFileDir(destFileDir); //set cache dir

and the songs will be downloaded to a local folder

Please use the latest version, the master branch is no longer maintained

okay but is it going to only download and cache the the songs I played or it is going to download the songs user have added to play in the library

What features do you need to process the song cache?

I have music streaming platform which has 4 tabs, Discover, trending, new music and videos,
I am adding an add button whereby user can click and add songs to their Library, immediately they click on that, they can play the songs in the Library offline

No problem, as long as you set the cache, it will be downloaded automatically when playing music, and you will not need the network when you play it again next time.

okay, So I should set it at the library Activity and also How can I make users clear or remove songs they don't want again

Hi please Can you help me with this, I have created add to my library where individual selected songs are stored and kept but my boss wants me to add a download button to it where by when the selected song is added and the user goes to the library and clicks on a download button the song should download and stored for offline play, please how can I trigger the download button to work with your library to able to download it


Upload a downsampled minial dataset retaining the fMRIPrep layout for testing

Currently such dataset are outdated.
Just need to do that at some point

There's a dataset on my local machine and CC for now


Test with gtk3.22.30

Part of https://github.com/gtk-rs/examples/pull/173
cc @GuillaumeGomez

CI passed

Thanks!


Please add support for Prometheus remote storage

An example could be found at: https://github.com/prometheus/prometheus/tree/master/documentation/examples/remote_storage/example_write_adapter

Thanks for the RFE. I'll talk with the team and see where this gets prioritized. Thanks again!

I know you've found it, but for completeness https://github.com/signalfx/metricproxy/pull/76

Closing. Thanks for the suggestion.


add Anyang University of korea

add Anyang University of korea

@dongyeop00 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


How to port forward in GKE?

Hi,
We have performed the helm install and can see the following services in GKE.

So now which services to port forward and how can we achieve that? Is there documentation we can follow for GKE?
Regards,
Rias

The helm chart is not maintained by prometheus-operator maintainers.
But they do mention accessing grafana in their quick start(which doesn't use helm).
https://prometheus-operator.dev/docs/prologue/quick-start/#access-grafana


Fetching wrong update (experimental instead of stable)

While trying to get from 0.13.x to 0.13.19 (now stable), factorio-updater wants to get 0.14.0. Example:
python update_factorio.py --for-version 0.13.15 --package core-linux_headless64 --dry-run
Dry run: would have fetched update from 0.13.15 to 0.13.16.
Dry run: would have fetched update from 0.13.16 to 0.13.17.
Dry run: would have fetched update from 0.13.17 to 0.13.18.
Dry run: would have fetched update from 0.13.18 to 0.14.0.

The experimental update is working as expected:
python update_factorio.py --for-version 0.13.15 --package core-linux_headless64 --experimental --dry-run
Dry run: would have fetched update from 0.13.15 to 0.13.16.
Dry run: would have fetched update from 0.13.16 to 0.13.17.
Dry run: would have fetched update from 0.13.17 to 0.13.18.
Dry run: would have fetched update from 0.13.18 to 0.14.0.
Dry run: would have fetched update from 0.14.0 to 0.14.1.

This is the get-available-version i receive:
  {
      "from": "0.13.17",
      "to": "0.13.18"
    },
    {
      "from": "0.13.18",
      "to": "0.13.19"
    },
    {
      "from": "0.13.18",
      "to": "0.14.0"
    },
    {
      "from": "0.13.2",
      "to": "0.13.3"
    },
    {
      "from": "0.13.3",
      "to": "0.13.4"
    },
    {
      "from": "0.13.4",
      "to": "0.13.5"
    },
    {
      "from": "0.13.5",
      "to": "0.13.6"
    },
    {
      "from": "0.13.6",
      "to": "0.13.7"
    },
    {
      "from": "0.13.7",
      "to": "0.13.8"
    },
    {
      "from": "0.13.8",
      "to": "0.13.9"
    },
    {
      "from": "0.13.9",
      "to": "0.13.10"
    },
    {
      "from": "0.14.0",
      "to": "0.14.1"
    },
    {
      "stable": "0.13.19"
    }

I think i got the problem: The updates are stored in a map (current version, next version). The update (0.13.18, 0.13.19) is later getting overwritten by (0.13.18, 0.14.0), right? So using the current_version as a unique identifier isn't working.

I have proposed pull request #12 as a fix for this problem.

The update (0.13.18, 0.13.19) is later getting overwritten by (0.13.18, 0.14.0)

How unpleasant. And, of course, if they'd been presented in the opposite order we'd never have met this bug.

Now they have released 0.13.20 as experimental. So there is 0.13.19 as latest stable, 0.13.18 as latest point to upgrade to 0.14.0 and experimental releases in 0.13 and 0.14. Seems like the updater does support all of this, but it's weird that there is no way to get to 0.14.x from Factorio >= 0.13.19.

I understand .20 went into the stable track very soon after your comment, but it's really not a setup they'd thought about before, is it?


Restarting container causes fluentd.conf to duplicate config

From @jschneiderhan on October 26, 2016 18:51
Logs of a deis fluentd container during it's first run:
ubuntu@ip-172-20-12-148:~$ sudo docker logs cf1ae5af2bc1
2016-10-24 18:45:58 +0000 [info]: reading config file path="/opt/fluentd/conf/fluentd.conf"
2016-10-24 18:45:58 +0000 [info]: starting fluentd-0.14.4
2016-10-24 18:45:58 +0000 [info]: spawn command to main: /usr/bin/ruby2.3 -Eascii-8bit:ascii-8bit /usr/local/bin/fluentd -c /opt/fluentd/conf/fluentd.conf --under-supervisor
2016-10-24 18:45:58 +0000 [info]: reading config file path="/opt/fluentd/conf/fluentd.conf"
2016-10-24 18:45:58 +0000 [info]: starting fluentd-0.14.4 without supervision
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-mixin-config-placeholders' version '0.4.0'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-mixin-plaintextformatter' version '0.2.6'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-mixin-rewrite-tag-name' version '0.1.0'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-plugin-deis_output' version '0.1.0'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-plugin-elasticsearch' version '1.6.0'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-plugin-kubernetes_metadata_filter' version '0.25.3'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-plugin-remote_syslog' version '0.3.2'
2016-10-24 18:45:58 +0000 [info]: gem 'fluent-plugin-sumologic-mattk42' version '0.0.4'
2016-10-24 18:45:58 +0000 [info]: gem 'fluentd' version '0.14.4'
2016-10-24 18:45:58 +0000 [info]: gem 'fluentd' version '0.14.0'
2016-10-24 18:45:58 +0000 [info]: adding filter pattern="kubernetes.**" type="kubernetes_metadata"
2016-10-24 18:45:59 +0000 [info]: adding match pattern="**" type="copy"
2016-10-24 18:45:59 +0000 [info]: adding source type="tail"
2016-10-24 18:45:59 +0000 [info]: using configuration file: <ROOT>
  <source>
    @type tail
    path "/var/log/containers/*.log"
    pos_file "/var/log/containers.log.pos"
    tag "kubernetes.*"
    format json
    read_from_head false
  </source>
  <filter kubernetes.**>
    @type kubernetes_metadata
    kubernetes_url "https://<IP_ADDRESS>:443"
    bearer_token_file "/var/run/secrets/kubernetes.io/serviceaccount/token"
    verify_ssl false
  </filter>
  <match **>
    @type copy
    <store>
      @type "deis"
    </store>
  </match>
</ROOT>
[... and so on ...]

If I restart the container, the config is duplicated:
ubuntu@ip-172-20-12-148:~$ sudo docker restart cf1ae5af2bc1
cf1ae5af2bc1
ubuntu@ip-172-20-12-148:~$ sudo docker logs cf1ae5af2bc1
[... previous logs ...]
2016-10-26 18:46:46 +0000 [info]: reading config file path="/opt/fluentd/conf/fluentd.conf"
2016-10-26 18:46:46 +0000 [info]: starting fluentd-0.14.4 without supervision
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-mixin-config-placeholders' version '0.4.0'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-mixin-plaintextformatter' version '0.2.6'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-mixin-rewrite-tag-name' version '0.1.0'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-plugin-deis_output' version '0.1.0'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-plugin-elasticsearch' version '1.6.0'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-plugin-kubernetes_metadata_filter' version '0.25.3'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-plugin-remote_syslog' version '0.3.2'
2016-10-26 18:46:46 +0000 [info]: gem 'fluent-plugin-sumologic-mattk42' version '0.0.4'
2016-10-26 18:46:46 +0000 [info]: gem 'fluentd' version '0.14.4'
2016-10-26 18:46:46 +0000 [info]: gem 'fluentd' version '0.14.0'
2016-10-26 18:46:46 +0000 [info]: adding filter pattern="kubernetes.**" type="kubernetes_metadata"
2016-10-26 18:46:47 +0000 [info]: adding match pattern="**" type="copy"
2016-10-26 18:46:47 +0000 [info]: adding filter pattern="kubernetes.**" type="kubernetes_metadata"
2016-10-26 18:46:47 +0000 [info]: adding match pattern="**" type="copy"
2016-10-26 18:46:47 +0000 [info]: adding filter pattern="kubernetes.**" type="kubernetes_metadata"
2016-10-26 18:46:47 +0000 [info]: adding match pattern="**" type="copy"
2016-10-26 18:46:47 +0000 [info]: adding source type="tail"
2016-10-26 18:46:47 +0000 [info]: adding source type="tail"
2016-10-26 18:46:47 +0000 [info]: adding source type="tail"
2016-10-26 18:46:47 +0000 [info]: using configuration file: <ROOT>
  <source>
    @type tail
    path "/var/log/containers/*.log"
    pos_file "/var/log/containers.log.pos"
    tag "kubernetes.*"
    format json
    read_from_head false
  </source>
  <filter kubernetes.**>
    @type kubernetes_metadata
    kubernetes_url "https://<IP_ADDRESS>:443"
    bearer_token_file "/var/run/secrets/kubernetes.io/serviceaccount/token"
    verify_ssl false
  </filter>
  <match **>
    @type copy
    <store>
      @type "deis"
    </store>
  </match>
  <source>
    @type tail
    path "/var/log/containers/*.log"
    pos_file "/var/log/containers.log.pos"
    tag "kubernetes.*"
    format json
    read_from_head false
  </source>
  <filter kubernetes.**>
    @type kubernetes_metadata
    kubernetes_url "https://<IP_ADDRESS>:443"
    bearer_token_file "/var/run/secrets/kubernetes.io/serviceaccount/token"
    verify_ssl false
  </filter>
  <match **>
    @type copy
    <store>
      @type "deis"
    </store>
  </match>
  <source>
    @type tail
    path "/var/log/containers/*.log"
    pos_file "/var/log/containers.log.pos"
    tag "kubernetes.*"
    format json
    read_from_head false
  </source>
  <filter kubernetes.**>
    @type kubernetes_metadata
    kubernetes_url "https://<IP_ADDRESS>:443"
    bearer_token_file "/var/run/secrets/kubernetes.io/serviceaccount/token"
    verify_ssl false
  </filter>
  <match **>
    @type copy
    <store>
      @type "deis"
    </store>
  </match>
</ROOT>
[... and so on ...]

I'm not sure what's up yet. Looking into it now. Apologies in advance if this is already known or intentional behavior.
Copied from original issue: deis/fluentd#68

From @jschneiderhan on October 26, 2016 19:10
Oh - looks like the boot script is just appending the custom configs to the file without wiping any previous ones. Perhaps it would make sense to wrap all the custom configs between a custom comment string and wipe it at every boot before appending the new configs?

From @jchauncey on October 26, 2016 19:18
if the pod died that should have been a new config. hwo did the container restart without the pod dying?

From @jchauncey on October 26, 2016 19:32
ah k that makes sense. Its definitely a valid usecase just something I hadnt considered. We should probably make sure that we are building the config file correctly.

From @jschneiderhan on October 26, 2016 19:35
Oh good. I was worried I was being a PITA.
I notice that fluentd supports an @include directive. Perhaps we include the custom config and rewrite the file each time? I'm happy to give it a shot if you think it's a decent idea.

From @jschneiderhan on October 26, 2016 19:28
I was testing out a config change quickly and restarted the container manually. If you're never worried about the container restarting, then this isn't an issue.

From @jschneiderhan on October 27, 2016 12:17
I guess what I'm suggesting is putting all the dynamic configuration in a different file that we can rebuild each time instead of appending to the fluentd.conf file, then having a single include for the dynamic configuration at the end of fluentd.conf. So basically the fluentd.conf becomes static, and instead of appending to fluentd.conf we append to fluentd.custom.conf. All the logic for what gets written stays the same, things just get appended to the dynamic file, and it's rebuilt each time boot is run, so it should end up in the right state no matter how many times boot is run.

From @jchauncey on October 26, 2016 21:8
well the includes would need to be conditional which means you just move them from 1 spot to another. Might not be too bad but i wonder if it makes it confusing on what needs to happen for a paritcular config file to be included in the main one. Make sense?

From @jschneiderhan on October 27, 2016 14:34
@jchauncey Sure thing! Like you mentioned, if a pod dies it would recreate the container, so this is probably a pretty rare edge case, so not that important. Thanks for entertaining the idea. When/if you settle on an implementation direction, let me know. I'd be happy to help out and create a PR.

From @jchauncey on October 27, 2016 14:23
Alright let me think about this for a day. I have some ideas but I have a ton of prs in flight so want to make sure I get those merged before I start on this.


Update GUIController.java

I have a question, why do you do it like this?  What difference does it make?

It lowers complexity


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

Atom Version: 1.10.2
Electron Version: 0.37.8
System: Mac OS X 10.11.6
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
at emitMany (events.js:121:13)
at EventEmitter.emit (events.js:195:7)
at EventEmitter. (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
at emitTwo (events.js:100:13)
at EventEmitter.emit (events.js:185:7)
At /Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (/Applications/Atom.app/Contents/Resources/electron.asar/browser/objects-registry.js:44:28)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/rpc-server.js:309:31)
    at emitMany (events.js:121:13)
    at EventEmitter.emit (events.js:195:7)
    at EventEmitter.<anonymous> (/Applications/Atom.app/Contents/Resources/electron.asar/browser/api/web-contents.js:141:25)
    at emitTwo (events.js:100:13)
    at EventEmitter.emit (events.js:185:7)
    at metaToValue (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:178:13)
    at undefined.remoteMemberFunction (/Applications/Atom.app/Contents/Resources/electron.asar/renderer/api/remote.js:112:18)
    at AtomEnvironment.module.exports.AtomEnvironment.getWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:636:29)
    at AtomEnvironment.module.exports.AtomEnvironment.storeWindowDimensions (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:670:36)
    at WindowEventHandler.module.exports.WindowEventHandler.handleWindowBlur (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:182:35)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61

Commands
  7x -8:32.3.0 golang:godef (atom-text-editor.editor.is-focused)
     -0:21.4.0 core:copy (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "projectHome": "/Users/augmn/Desktop/AUGMNPROJ/devops/docker-build/repos/fork"
  },
  "editor": {
    "invisibles": {},
    "tabLength": 4
  }
}

Installed Packages
# User
autocomplete-go, v1.1.0 (active)
builder-go, v1.0.0 (active)
go-config, v1.1.4 (active)
go-get, v1.0.3 (active)
go-imports, v0.1.1 (inactive)
go-plus, v4.1.0 (active)
gofmt, v1.1.6 (active)
gometalinter-linter, v1.1.0 (active)
gorename, v1.0.2 (active)
linter, v1.11.4 (active)
navigator-godef, v1.0.3 (active)
tester-go, v1.0.4 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.52.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.44.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.5.0 (active)
one-light-ui, v1.5.0 (inactive)
one-dark-syntax, v1.3.0 (active)
one-light-syntax, v1.3.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (inactive)
about, v1.5.3 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.2 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.1 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.39.0 (active)
find-and-replace, v0.201.0 (inactive)
fuzzy-finder, v1.3.0 (active)
git-diff, v1.1.0 (active)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.2 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.0 (active)
metrics, v0.53.1 (active)
notifications, v0.65.0 (active)
open-on-github, v1.2.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.242.2 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.4.1 (active)
styleguide, v0.47.0 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.100.2 (active)
timecop, v0.33.2 (active)
tree-view, v0.208.2 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.33.0 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.2 (active)
language-csharp, v0.12.1 (active)
language-css, v0.37.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.42.1 (active)
language-html, v0.45.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.23.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.2 (active)
language-less, v0.29.5 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.2 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.69.0 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.56.0 (active)
language-shellscript, v0.22.4 (active)
language-source, v0.9.0 (active)
language-sql, v0.23.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.9 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

This is a duplicate of https://github.com/atom/atom/issues/11692. Feel free to subscribe there for updates.


Add moss stones under the sign boards

To protect from unintended destruction (especially digging from underground).

Done!


Feat wise global layernorm

Feature-wise global layer normalization as it is used in TCN++ in FUSS and firstly proposed in Universal Sound Separation.
Normalization is applied over all frames for each channel.

Thanks for the PR, that looks good !
Maybe we can add a comment in the docstring that it comes from TCN+ in the USS paper?


PageInformation improvements

Minor things, just adding a title tag and making sure that the PageInformations editable state doesn't go weirdly narrow

 Item names are jannkkky


[cs-libguarded] Add new port

Fixes #37511.

[ ] Changes comply with the maintainer guide.
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[ ] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[ ] The generated "usage text" is accurate. See adding-usage for context.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

When checking repology, I noticed that there is already a port for this package: libguarded. I think that the cs prefix is needed, but then the other port must be turned into an empty proxy for this port.

Once this PR is accepted, i will submit another PR to fix this.

Once this PR is accepted, i will submit another PR to fix this.

No, you must do it in this PR. Vcpkg nightly CI will install both ports, and then they will probably clash. This cannot be accepted.
And I think we should seriously consider the cslibguarded name pattern proposal. It would align docs, cmake package and port. But somebody else will have to decide. (Disclaimer: I'm a community member.)


Add font-family fallback and optimize font loading

Optimized font loading by adding font-display: swap. Additionally, moved font-family values into CSS variables.

Thank you for this, may I ask you how you selected the fonts in "font-body"? Also, would failing to load Literata fallback on the second, and so on?

@tomfran ah, I made an error of using the font stack I was testing out on an example site instead of using something to fallback for Literata. Will update the PR.

Also, would failing to load Literata fallback on the second, and so on?

Yes, it will fallback to the first font available in the stack and based on the usage of font-display: swap, will keep it on that fallback font until the main font is loaded and ready to be swapped.

I just realized that the menu was using monospace instead of Monaspace, I added a small font size to bring back some consistency in the size, as I saw my personal website changed quite a lot.
I liked better the old monospace font there😅

@tomfran ah, I thought that was a typo (pun intended). btw monospace renders differently across browsers and OS. Better to be specific even if you wanted a different monospace font for menu.

Yeah it's better to use Monaspace everywhere, at least this was the intention.


[AUTOPATCHER-CORE] Upgrade mariadb to 10.6.20 none

[AUTOPATCHER-CORE] Upgrade mariadb to 10.6.20 none
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=669292&view=results
buddy build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=669297&view=results

fix build break. Buddy build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=669658&view=results


Generated  models and request builders

This pull request was automatically created by Azure Pipelines. Important Check for unexpected deletions or changes in this PR.

@rkodev this is impacted by the file changes, would you mind having a look please?

@baywet There is a WIP PR here https://github.com/microsoft/kiota/pull/3376

superseded by #375


Extend Custom user interface colors for sidebar and page explorer

Is your proposal related to a problem?
I'm able to easily adjust some colors of the admin page. I used the documentation to customize the interface and setted the css to
:root {
    --color-primary: red;
    --color-primary-darker: red;
    --color-primary-dark: red;
    --color-primary-lighter: red;
    --color-primary-light: red;
    --color-input-focus: red;
    --color-input-focus-border: red;
}

However, I find it somehow inconsistent that the sidebar and page explorers are not able to be adjusted.
Describe the solution you'd like
I would like to modify the purple colors just like the possible settings above.
Describe alternatives you've considered
I can hack into the css:
.sidebar__inner{
    background-color: black;
}

.hover\:w-bg-primary-200:hover{
    background-color: black;
}

.w-bg-primary{
	background-color: black;
}

however I think this could be bypassed by the really nice cusumization functionality described above.
Additional context
With the two css combined I get the following result:

As you see, I would need to hack into lots of other css variables to replace all purple with e.g. black.
With a good editor it doesn't seem as much of an issue to replace identical color values with css variables. But maybe I'm wrong. If you point me to the right files I could then open a PR with adjustable colors for sidebar and page explorer (and branding_welcome)

Hi @lukasalexanderweber - which version are you running? Support for colour customisation has been greatly extended in the upcoming Wagtail 4.0 - https://github.com/wagtail/wagtail/pull/8824

ahh nice I see
I'm running on 3.0.1., but now can't wait till 4.0 is released 🚀
thanks for the info @gasman!


#110: Top menu allows PageData

Please do not merge this. When using a page in the menu and then viewing from edit mode the the edit mode URLs get cached in the menu. Suggest not caching the menu when in edit mode to avoid this. Will record a video later to demonstrate.

Please do not merge this. When using a page in the menu and then viewing from edit mode the the edit mode URLs get cached in the menu. Suggest not caching the menu when in edit mode to avoid this. Will record a video later to demonstrate.

@davidknipe that issue is resolved in branch 121 (HeaderController and _Layout), I didn't fixed in this branch to prevent conflict code

I just tried this again with the referenced updated files and the issue still persists. To recreate:
Reset the application
Go straight to edit mode
Drag a page in to the top menu
Publish
Log out
Notice that the page being linked is now the internal Episerver URL
To resolve only execute this line if not in edit mode:
https://github.com/episerver/Foundation/blob/3c2414a71d9d7c41f24e8afff83df1077fe74108/src/Foundation.Cms/ViewModels/Header/HeaderViewModelFactory.cs#L79

I just tried this again with the referenced updated files and the issue still persists. To recreate:
Reset the application
Go straight to edit mode
Drag a page in to the top menu
Publish
Log out
Notice that the page being linked is now the internal Episerver URL
To resolve only execute this line if not in edit mode:
https://github.com/episerver/Foundation/blob/3c2414a71d9d7c41f24e8afff83df1077fe74108/src/Foundation.Cms/ViewModels/Header/HeaderViewModelFactory.cs#L79

yes, I have updated in branch 121


Ui enhancements

1.)  Replaced "Oauth" field in the UI to now be the integration type of the capability.
2.)  Added the ability to preview the response from the url before creating a new capability.
3.) Updated the db seed to include an example of each integration type.

Other than that it looks good to me.


[Suggestion] Frontier back to greatness

Hello!
I used to play the server about a year ago, I played in Stratos. I remember Battle Frontier with many spawners for EVs and it was great! You didn't have to beg around people to help you find certain spawners and it was so fun! I don't know why it was removed, but I think it'll help the server by helping newcomers find the spawners easily which are as of now very hard to find (can tell as a new Terra player).
Thanks for reading!

Though we already have a lot of EV spawns (Nyr is a good example), we could easily help with the signalization by making floating text around those spawn points. All spawns in Nyr should allow you to train up EVs as needed. Alternatively, we might need to double on spawners. Thoughts, @Vohras2 ?

We already have plenty of spawners that give +2 evs for pokemon. With the current surplus of spawners we have, I don't think this really needs to happen.


tokendumper fails when it comes to cf captcha

Although this issue doesn't belong to extension, I can't find a good place to report my issue.
[>] Read https://steamdb.info/tokendumper/ before using this

[!] Update check failed: Response status code does not indicate success: 403 (Forbidden).
[!] Your submission may fail if you continue with the dumping.

(omitted)

Submitting tokens to SteamDB...
<!DOCTYPE html>
<!--[if lt IE 7]> <html class="no-js ie6 oldie" lang="en-US"> <![endif]-->
<!--[if IE 7]>    <html class="no-js ie7 oldie" lang="en-US"> <![endif]-->
<!--[if IE 8]>    <html class="no-js ie8 oldie" lang="en-US"> <![endif]-->
<!--[if gt IE 8]><!--> <html class="no-js" lang="en-US"> <!--<![endif]-->
<head>
<title>Attention Required! | Cloudflare</title>

(omitted)

</body>
</html>

Whoops: Response status code does not indicate success: 403 (Forbidden).

Exiting...
Press any key to exit...

Yes. Works for me now.


Draft PPP

Codecov Report

Merging #247 (e36421d) into master (fdfe579) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #247   +/-   ##
=========================================
  Coverage     76.24%   76.24%
  Complexity      892      892
=========================================
  Files           130      130
  Lines          2572     2572
  Branches        303      303
=========================================
  Hits           1961     1961
  Misses          512      512
  Partials         99       99

Continue to review full report at Codecov.

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


chore(juniper): release 0.1.0

Coverage remained the same at 62.099% when pulling 7b1d29e50149d1e63036d4bcb603cd1824403a97 on juniper-0.1.0 into adb9411a5801dfa92a6b65e223283d5febe699b7 on master.


⭐ 13 - Recipe Modifier (Hacktoberfest Demo)

🤩 Thank you for participating in the CopilotKit Hacktoberfest event.
Follow This Link to Get Started!🎉
NOTE: CopilotKit will allow multi-assign issues
Please show your support and give a ⭐ to the CopilotKit repository.
Demo:
Recipe Modifier
Description
Create a tool that allows users to input a recipe and get suggestions for substitutions or alterations
Technology Requirements
Using CopilotKit

Required: Your project must utilize CopilotKit as a core component.
Installation: Include CopilotKit by following the installation guide at CopilotKit Quickstart.
Documentation: Familiarize yourself with CopilotKit's features through the Official Documentation.

Styling with Shadcn-UI

Required: All UI components in your project should be styled using Shadcn-UI.
Installation: Install Shadcn-UI documentation

Can you please assign me this issue?

Hello, Can you assign me  this one??!!

hi @NathanTarbert  can u assign me ?

@quest-bot loot #625


Unexpected trailing comma

Would you consider turning off the warning for "Unexpected trailing comma" ??
I save time and mental load when reordering lists by not having to track that last comma.  I'm not aware of any issues that could be caused by allowing this extra comma.
module.exports = {
  languages: [
    'chinese_simplified',
    'chinese_traditional',
    'japanese',
    'english',
    'italian',
    'french',
    'spanish', // <= This is good
  ], // <= This is good
}

Hi @jcalfee! Thanks for the thoughtful feedback!
This rule change has been discussed previously, and here was the decision we came to at that time:

If there's two ways to do something and there's no difference in correctness or chance of introducing a bug, then standard tends to 'just pick something'. So, relaxing the rule to allow either style isn't really a viable option.
Given that it's gotta be either 'disallow' or 'enforce', I think disallow is the least controversial option. Enforcing trailing commas would confuse a lot of beginners to programming or JS, IMO.

With the above in mind I'm going to close this issue now, but please feel free to continue the conversation!

I agree that with only disallow and enforce, it should be disallow.  But I'd rather have it be optional.  I’m not opinionated enough to say we should have them, but I am opinionated enough to say we should not force people to have/not have them.
Also reordering a list is a common activity and not having a trailing comma would increase chances of introducing a bug as well as increasing the commit diff noise.

I personally would love to have trailing comma enforced, but I don't think that making this optional is the way to go.

[...] but I am opinionated enough to say we should not force people to have/not have them.

The whole point of standard is to be opinionated and to pick one way. Otherwise, that same argument could be made about semicolons, space vs. tabs, and basically every other rule. I think it's important that standard enforces one of them.


Add link from GraphQL types to Strawberry types

Description
I'm toying with strawberry internals, and I often need to find the links between GraphQLTypes and their strawberry definitions.
For now I have been using Schema.get_type_by_name(graphql_type.name) and similar funcions, but it feels a bit clumsy.
Instead, this PR takes advantage of GraphQLNamedTypes (and others) "extensions" to store links back to the originating strawberry type.
Types of Changes

[ ] Core
[ ] Bugfix
[x] New feature
[ ] Enhancement/optimization
[ ] Documentation

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to Paulo Costa for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

Merged latest strawberry changes :)

@paulo-raca thanks for this PR! Me and @BryceBeagle spent a bit of time refactoring the tests and we also updated the release notes to make it easier to understand what this release adds.

Thank you for looking into those!


Switch some tests to use unique pointers instead of raw pointers.

This gets rid of some error paths in the tests that could leak memory, thus making clang static analysis happy.
Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

CI:

Linux
Linux-aarch64
Windows


FastFlow performance

Hello
I am really interested in anomaly detection models. Therefore I am following your work with the one of https://github.com/AlessioGalluccio/FastFlow and https://github.com/mjack3/FastFlow-1 ;  since Fastflow appears to be one of the most promising frameworks. Actually, after training your implementation on my medical images, without much success, I have tried to reproduce your results. For example, on the carpet, I am still very far from your results with a ROC-AUC of "only" 81%. This score is reached after a few epochs. The loss function is extremely low. Do you observe a similar behaviour? Did you change some hyper-parameters?
Thank you very much in advance :-)
Emilie

Well, I could also observe that the performance fluctuates in most cases. However I couldn't 81% AUROC on the carpet. Can you share some performance graphs?

Hello @cytotoxicity8
Thank you for your very quick answer. Below you can find the evolution of the loss and the AUROC for the carpet, the wood and the hazelnut.  As you can see while the loss curves seem to converge there are no improvements on the AUROC curves.

HaloAE_WACV (13).pdf

![LossHazelnut](https://user-images.githubusercontent.com/33092597/182841353-ee1febc1-2ecf-

4567-8b70-20c081d8f4a7.png)

Hi @emathian, are you using another DeiT pre-trained weight? And a modified DeiT module?
I see some modifications here. https://github.com/emathian/LNEN_FastFow_gathierry/blob/main/fastflow.py
I'm not sure what's the root cause right now. But maybe you can run the code without modification, and then modify your code incrementally.
The results I list in README are all experiments based on the code in this repo. However, I only tested once so there could be some variance. But it shouldn't be as much as what you mentioned.

Dear @gathierry,
Thanks for your comment :) .
I can't use timm on my server because their proxy forbids to download things. I will check this problem with them. Nevertheless, I use deit.deit_base_distilled_patch16_384 and I used the weights published here https://github.com/facebookresearch/deit, so I think it is equivalent.
In parallel, I'm studying the effect of feature extractors on performance.
I think ViT based feature extractors may not be the best because of the columnar architecture that does not capture local and global information. For my medical images, I train another hidden auto-encoder (ViT based ^^') to reconstruct the images, I used the encoded images as FastFlow inputs. This step is important given the significant differences between lung tumors and Imagenet.
Furthermore I observed that FastFlow seems to be biased by the hospital, it manages to discriminate the center from where the patient was treated, which is not relevant information in my case. I wonder if the model fails to generalize with respect to large color variations. I simulated MvTec objects with many colors, keeping only one extra shade for the test set, in order to simulate the introduction of a new hospital.Curiously, the distribution of FastFlow scores is not biased by the type of color increase. Perhaps the effect I'm seeing is from other sources of variation, I don't know how well FastFlow is able to normalize over a highly variable training set.
Thank you very much ^^

In that case it should be the same. I see you added DDP, are you using more than 1 GPU for training MVTec? If you only use 1 GPU then I think everything should be the same. Might need more debug if you really want to reproduce the result.
I'm not sure whether Fastflow is suitable for medical image but I think there's quite big difference between MVTec and medical images. For example, the shape of transistors are all similar but even a healthy lung of different people can look very different (feel free to correct me if this is not a good example). So IMHO it is possible that models working on MVTec may not achieve comparable performance on your data.

I am parallelizing on my dataset which is much larger than MVtec that is why I added DDP. Let's debug 😄! I will finish by finding the solution.

How to get the above  performance graphs and where to modify the code？Thanks

How to get the above performance graphs and where to modify the code？Thanks

Try to use tensorboard, it is easy. (Insert some codes in main.py)


NullPointerException on ExtensionCallMarklogic for PUT and DELETE

When doing a ExtensionCallMarklogic on a Rest Extension service (which does nothing) with no payload for PUT and DELETE => java.lang.NullPointerException
On ML side, the status is 200 (OK).

There's a fix for this in the incubator project, but I didn't see any tests for it. So adding tests as part of this ticket.

There's a fix for this in the incubator project, but I didn't see any tests for it. So adding tests as part of this ticket.

Note that I was not able to reproduce this for a DELETE. This may be due to the connector now using 5.3.2. of the Java Client, which requires a non-null input for a PUT, but not for a DELETE.

Note that I was not able to reproduce this for a DELETE. This may be due to the connector now using 5.3.2. of the Java Client, which requires a non-null input for a PUT, but not for a DELETE.


sAjaxSource vs ajax

When i use sAjaxSource ,All the data is showing on all pages..
But when i replace it with ajax..Everything is working fine..

"bProcessing": true,
"bServerSide": true,
"sAjaxSource": "{{ route("admin.ajax_employees") }}",
"aaSorting": [[ 1, "asc" ]],

Showing 1 to 5 of 22 entries  but it is actually showing all 22 records

  >"bProcessing": true,
  >  "bServerSide": true,
  > "ajax": "{{ route("admin.ajax_employees") }}",
  >"aaSorting": [[ 1, "asc" ]],
Showing 1 to 5 of 22 entries  but it is actually showing all 5 records
Please provide some fix for this too..

@ajayfroiden, sAjaxSource is the old way of using dataTables and I already drop support for the legacy / DT1.9.x version. Note: sAjaxSource works/behaves differently against ajax in terms of the data being sent to the server.


[pingora-proxy] Invalid frame header when upstream websocket connection

Describe the bug
The upstream connection is created, send some data, then encounter Invalid frame header, and the connection closed.

Pingora info
Please include the following information about your environment:
Pingora version: 0.2.0
Rust version: 1.79.0
Operating system version: Debian11

Do you have a minimal reproduce that you can share? Or maybe some tcpdump will help.
Also to clarify:

The upstream connection is created, send some data, then encounter Invalid frame header, and the connection closed.

The upstream (server) sent some date, then it encountered the invalid frame header and then it actively closed the connection due to the invalid frame?

No, it's not easy to re-produce, I'll try to get the tcpdump next time when it's possible.
I think it's the chrome's behavior to close the connection when encountered invalid frame.


[BAEL-2118] - Add a section in String Split Article

@pmaric I have mistakenly added all the code in master directly for both BAEL-2118 and BAEL-2077.


Assignment two

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
ERD diagram and sql doc
What did you learn from the changes you have made?
Coalesce, Update, Delete, Insert, Rank, RowNumber
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Lots of questions required some in depth thinking, many tries adjusting the formatting of the SQL. When to use a CTE, Temp Table, Subquery.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Most of my challenges I solved by slowly reading over the question to understand exactly what it's asking
How were these changes tested?
Changes were tested on the FarmersMarket.DB file
A reference to a related issue in your repository (if applicable)
Checklist
I can confirm that my changes are working as intended

Disconsider the last message. I found the 02_activities/assignments/Assignment2.drawio.png in in your latter commit.


Update page_iterators.py to remove unnecessary print

appologies, i have commited this already, exactly as you put it into your PR, i'll close this PR then


Guid::uuid4() doesn't generate a Guid

echo get_class(Ramsey\Uuid\Guid\Guid::uuid4());
will return
Ramsey\Uuid\Rfc4122\UuidV4.
I didn't expect this behavior at all.

Also consider this:
$f = new Ramsey\Uuid\UuidFactory(new Ramsey\Uuid\FeatureSet(true));

var_dump($f->uuid4());

will return:
PHP Fatal error:  Uncaught Ramsey\Uuid\Exception\InvalidArgumentException: The byte string received does not contain a valid version in vendor/ramsey/uuid/src/Guid/Fields.php:83

Are these two different issues, or do you consider them related?
As for the 1st issue, please see #301. The static methods are not part of the interface and should not be used from the subclasses. This is a bug. I’m sorry for the confusion.

As I am trying to create Guids in both cases, those issues are related to
me, maybe from a technical perspective they aren't for you.
I can create separate issues if you want to.
On Mon, Apr 6, 2020, 21:32 Ben Ramsey<EMAIL_ADDRESS>wrote:

Are these two different issues, or do you consider them related?
As for the 1st issue, please see #301
https://github.com/ramsey/uuid/issues/301. The static methods are not
part of the interface and should not be used from the subclasses. This is a
bug. I’m sorry for the confusion.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ramsey/uuid/issues/306#issuecomment-610122903, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADAJPANVJB7E46ZH6GI6H3RLJ7EBANCNFSM4MCWVZBA
.

The first issue is an oversight on my part. You should not have access to Guid::uuid4(). This is a bug. Only the Ramsey\Uuid\Uuid class should have the static methods. I will roll out a release soon that disables these static methods on child classes (they are only child classes for the sake of BC and easy transition from version 3 to 4; they will no longer be child classes in version 5).
I’ll look into the second issue. At a glance, this looks like it should work, so I think you’ve uncovered another bug here.
Thanks!

$f = new Ramsey\Uuid\UuidFactory(new Ramsey\Uuid\FeatureSet(true));
var_dump($f->uuid4());

This is solved in master branch already, not working on 4.0.1 though.

The master branch is the same as 4.0.1 right now. The only difference is white space. How can this work in master and not in 4.0.1?

Can't tell, that's what I tried. Don't trust, verify.
On Mon, Apr 6, 2020, 22:54 Ben Ramsey<EMAIL_ADDRESS>wrote:

The master branch is the same as 4.0.1 right now. The only difference is
white space. How can this work in master and not in 4.0.1?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ramsey/uuid/issues/306#issuecomment-610144507, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADAJPDMOSGITDAMQD7OAQ3RLKIXNANCNFSM4MCWVZBA
.

@ramsey It's not fixed. The bug just doesn't always appear, it's random. I posted more details in the separate issue: https://github.com/ramsey/uuid/issues/314#issuecomment-734339547


STEP-4: initialization of microservices (adding build.gradle files)

Create and add to the project architecture diagrams.
Acceptance cretaria:

"build.gradle" for compensation-device-service created.
"build.gradle" for full-information-service created.
"build.gradle" for power-transformers-service created.
"build.gradle" for high-volt-cables-service created.
"build.gradle" for lighting-information-service created.
"build.gradle" for protective-equipment-service created.
"build.gradle" for start-information-service created.
"build.gradle" for config-service created.
"build.gradle" for gateway created.
"build.gradle" for registry created.
"build.gradle" for "power-supply-design project" created.

Implements in https://github.com/EllsaG/power-supply-design/pull/35 pull request.


Add S2N_RESULT to a couple functions.

Description of changes:
On going ops work to move to the S2N_RESULT pattern.
Testing:
This is a refactor change. Testing includes local test runs and the standard CI.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2371 (81a62a4) into main (efbb48b) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2371      +/-   ##
==========================================
- Coverage   81.84%   81.82%   -0.03%
==========================================
  Files         268      268
  Lines       18300    18299       -1
==========================================
- Hits        14978    14973       -5
- Misses       3322     3326       +4


Investigate increasing the verbosity of logged events

Consider also providing a constructed command for the user to test themselves.

Done prints whenever playbook is run 81cc647e113ab77e525a8106d39918a0eb311df9


feat(aws-s3-sns): created new construct

Issue #, if available:
N/A
Description of changes:
README for the new aws-s3-sns construct
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 51db29c057eff20b2dbc8b3f2de8c91709ca6d64
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: ee52701558a874237904bf9ac95686e7d24d12d7
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

The aws-s3-sns construct has 100% line coverage for unit tests, and respective integration tests. I've deployed each integration test case and validated the events are actually published to the destination SNS Topic.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: adf993df65031482b3ee2d5a560badd474085a8a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 70e899424fc3cc63554ff341be77f34e018226bb
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: fa4eb4b50ef8056088445735d6db4a3ba570f34c
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: d330ada1656b754e057c191a8ada8ece5b715f41
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 7048e3f4cd4d07ab79359edc10503a4e03c51695
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 1f6d9bc26563a0894868fdd266c094dc3b206f7b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Waybar per device

Need to find a way to configure waybar on a per device basis as I use NixOS on both a laptop and desktop and do not need for example battery information on my desktop

Fixed in commit


PORTALS-3307 - getting started component

Create a getting started component for the EL Portal
jira ticket: https://sagebionetworks.jira.com/browse/PORTALS-3307
I got this as close as I could to the figma mock up but to my eye it's still not quite right.

hm something about merging main into this branch broke the site running locally. Looking into that now 👀

hm something about merging main into this branch broke the site running locally. Looking into that now 👀

solved by rebuilding repo


Adjustments in the design of homepage

My comments/suggestions (in Polish, sorry ;):

The design:

Magento's logo - https://cdn.worldvectorlogo.com/logos/magento-2.svg.

@filrak could you please pick this issue? :)

@karolbzik is it ready to be closed?
ps. Logo need to be clickable

@patrykpiston will check it.


Mqtt errors

Hi, sometime I get this exceptions:
What can be the reason of them?

MqttError('Operation timed out')
MqttCodeError('Could not publish message')

"Operation timed out" can occur for different reasons:

Your computer failed to send the message in time
The server did not respond in time
"Could not publish message" can occur due to the timeout.

It's difficult to pinpoint the culprit from the raw exceptions alone. It can be anything from a temporary network failure to a faulty network driver. Maybe you can try with a localhost server and see if that helps. :+1:

"Operation timed out" can occur for different reasons:

Your computer failed to send the message in time
The server did not respond in time
"Could not publish message" can occur due to the timeout.

It's difficult to pinpoint the culprit from the raw exceptions alone. It can be anything from a temporary network failure to a faulty network driver. Maybe you can try with a localhost server and see if that helps. :+1:

You can also try to increase the timeout parameter and see if that changes things. The default timeout is 10 seconds.

You can also try to increase the timeout parameter and see if that changes things. The default timeout is 10 seconds.

The broker and device not at the same region so maybe there are issue with timeout so I will try this:

You can also try to increase the timeout parameter and see if that changes things. The default timeout is 10 seconds.

Maybe timeout should be different for etch  QOS?

Did this two error messages can be for same operation? of publish?

for example i try to publish "a" to topic "1" with qos1
Then for some reason timeout occur and both messages will received at Exception catch?

try:
  await self.clients.get(broker).publish(topic, msg, qos=1)
except MqttError as identifier:
  _logging.error("Mqtt publishing Failed", extra={
      "data": {"exception": identifier, "message": msg, "topic": topic}})

"Operation timed out" can occur for different reasons:

Your computer failed to send the message in time
The server did not respond in time
"Could not publish message" can occur due to the timeout.

It's difficult to pinpoint the culprit from the raw exceptions alone. It can be anything from a temporary network failure to a faulty network driver. Maybe you can try with a localhost server and see if that helps. 👍

Thanks

The broker and device not at the same region so maybe there are issue with timeout so I will try this:

You can also try to increase the timeout parameter and see if that changes things. The default timeout is 10 seconds.

Maybe timeout should be different for etch  QOS?

Did this two error messages can be for same operation? of publish?

for example i try to publish "a" to topic "1" with qos1
Then for some reason timeout occur and both messages will received at Exception catch?

try:
  await self.clients.get(broker).publish(topic, msg, qos=1)
except MqttError as identifier:
  _logging.error("Mqtt publishing Failed", extra={
      "data": {"exception": identifier, "message": msg, "topic": topic}})

"Operation timed out" can occur for different reasons:

Your computer failed to send the message in time
The server did not respond in time
"Could not publish message" can occur due to the timeout.

It's difficult to pinpoint the culprit from the raw exceptions alone. It can be anything from a temporary network failure to a faulty network driver. Maybe you can try with a localhost server and see if that helps. 👍

Thanks

Maybe timeout should be different for etch QOS?

That would make sense. Good thing is that timeout is already a parameter so you can do this as a user already. :+1:
As long as you have problems with timeouts in the system, you might also see other strange errors. :) Fix the timeouts first and foremost. Then, if you still get other errors, we can have a look at those. :+1:

Maybe timeout should be different for etch QOS?

That would make sense. Good thing is that timeout is already a parameter so you can do this as a user already. :+1:
As long as you have problems with timeouts in the system, you might also see other strange errors. :) Fix the timeouts first and foremost. Then, if you still get other errors, we can have a look at those. :+1:

Hey there, I'm guessing that we can close this as there has been no more discussion. If there's anything left unclear, please reopen!


native: channel management screens

Fixes TLON-1716.
Fixes TLON-1712
Fixes TLON-1708

FYI, drag and drop is still a bit janky on android, but functionally works. Could follow up later to clean that up.

Heard on the section-draggging usability.
Here's an alternative approach: https://www.figma.com/design/mUIcOGeELxWswuusYUnEiW/2024-Q2-Tlon.app?node-id=4001-64564&t=Mlu2XTVUcydU1IET-4

@patosullivan Does this seem amenable?

@jamesacklin I updated to use the arrows

Raised TLON-2265 and TLON-2266 in experiential testing. Code otherwise looks good.

Fixes TLON-2266


Ignore watch task, or tasks beginning with 'watch:'

Fix for #51
Related to #72

Thanks for the review @sindresorhus. Changes have been made.

Thanks :)


Error: SocketException type unknown instead of connectionError

Package
dio
Version
^5.3.1
Output of flutter doctor -v
[✓] Flutter (Channel stable, 3.10.6, on macOS 13.4.1 22F770820d darwin-arm64, locale en-EG)
    • Flutter version 3.10.6 on channel stable at /Users/AbdelazizMahdy/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f468f3366c (3 weeks ago), 2023-07-12 15:19:05 -0700
    • Engine revision cdbeda788a
    • Dart version 3.0.6
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/AbdelazizMahdy/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.68.0

[✓] Connected device (3 available)
    • iPhone 14 Pro (mobile) • 7E7D7194-D699-40A5-9281-8E5D860CA41F • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 13.4.1 22F770820d darwin-arm64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Dart Version
3.0.6
Steps to Reproduce

launch a request while internet is down

Expected Result
a dio exception with type DioExceptionType.connectionError
Actual Result
Exception: DioException [unknown]: null
Error: SocketException: Failed host lookup: '.*****.' (OS Error: nodename nor servname provided, or not known, errno = 8)

Hey, thanks for rasing this issue. @zezo357 would you be interested in contributing this as a fix?

Hey, thanks for rasing this issue. @zezo357 would you be interested in contributing this as a fix?

I would like that, I will try when I get some free time.

In the meanwhile, any workaround to detect the error is a connection issue?
Also, may be related, but the same happens with the timeout errors, returns unknown instead of the correct exception.

@joaopegoraro You can always check if it is a SocketException, not sure in web. Please provide a reproducible sample otherwise, thanks!


Fix several test subcommands.

Fix #267.

Forgot the test class stuff, will fix tomorrow.

The test class stuff has been fixed too, making this PR ready for review.


Check for excessive CPU usage

Original report by me.

The tool constantly consumes 3-4% CPU when used with spammy device.
Should check whether this is unavoidable or some bug is lurking inside the tool.

The number of threads used to process the logs was cut in half recently. Don't think I want to track this further.


Add extra space between Identity feature and the actual feature name

Description
This is based on @imatiach-msft's comment https://github.com/microsoft/responsible-ai-toolbox/pull/1637/files/6dfdf14a889a4264b28d60529ff7a7151ea77cce#r945832160. So adding the extra space in counterfactual scatter plot.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1641 (76bfac8) into main (0a4c5dc) will decrease coverage by 13.43%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1641       +/-   ##
===========================================
- Coverage   89.01%   75.58%   -13.44%
===========================================
  Files         105       10       -95
  Lines        5489      258     -5231
===========================================
- Hits         4886      195     -4691
+ Misses        603       63      -540

Flag
Coverage Δ

unittests
75.58% <ø> (-13.44%)
:arrow_down:

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

Impacted Files
Coverage Δ

erroranalysis/erroranalysis/_internal/__init__.py

...ranalysis/erroranalysis/analyzer/error_analyzer.py

raiutils/raiutils/models/model_utils.py

...ponsibleai/responsibleai/_tools/shared/__init__.py

raiwidgets/raiwidgets/__version__.py

erroranalysis/erroranalysis/analyzer/__init__.py

raiwidgets/raiwidgets/cohort.py

.../databalanceanalysis/aggregate_balance_measures.py

raiutils/raiutils/version.py

...sponsibleai/responsibleai/managers/base_manager.py

... and 85 more

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


Bug: select using DuckDB - Invalid Input Error: No magic bytes found at end of file

Reported by user:
Following the quickstart tutorial, merged by the "denmark" branch into "main".
Using the UI's DuckDB SELECT * FROM READ_PARQUET('lakefs://quickstart/main/lakes.parquet');
Returns the following error:
Error: Invalid Input Error: No magic bytes found at end of file 'lakefs://quickstart/main/lakes.parquet'

@nopcoder did we manage to fix this?  Please close if we did, unassign and we'll triage if not.
THANKS!

Not fixed - removed myself and added ui tag

I was not able to reproduce this issue. I followed the quickstart tutorial and was able to query the main branch right after merging from the denmark-lakes branch.
In particular, there was no issues with caching: DuckDB caching is disabled by default, and browser cache is not a problem because the ETag of the file changes after the merge so the cache is not used.
I'm closing this issue now, feel free to re-open if more information is available. CC @nopcoder.


IC50 convention

In the 3PL model you have
Use of a three parameter model here sets the min at zero.
def formula(X, A, B, C):
    return A / (1 + (X / C)**B)

the 4PL model is f(x) = D - (A - D)/(1 + X/C)**B).
A is typically defined as the minimum response, which is being simplified to zero for biological relevance. We should use consistent naming.

D is the min here, unless this is wrong.
https://www.aatbio.com/tools/ic50-calculator

We'll just change the variables to more intuitive labels rather than A/B/C

Variable naming has been updated.


Error with vue3

I'm trying to use this great editor with vue3 in my project, using the simplest example from here.
It come out with error:
vue-json-editor.vue:17 Uncaught SyntaxError: The requested module '/node_modules/vue-json-editor/assets/jsoneditor.js' does not provide an export named 'default'
Can I use this editor in vue3? Then how?

声明xxx.s.ts文件
declare module 'vue-json-editor'

there is a fork here :
https://github.com/joaomede/vue3-json-editor


[AutoPR network/resource-manager/compute/resource-manager] Publicipprefixaugust

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3618

(message created by the CI based on PR content)


Wondering about piping stdin to program under winpty control

I'm experimenting with running Windows console software under both winpty and proxywinconsole.
Of interest is whether I can pipe input into a powershell script running under winpty.
Example:
echo "$PASSWORD" | winpty -Xallow-non-pty powershell.exe "C:\script.ps1"
where script.ps1 reads the password on standard input.
Reason is that it's not safe to send passwords into the powershell script through command line parameters, as they can be inspected.  Was hoping to have the script read from stdin.
As of right now, trying to redirect standard input into a program under winpty control seems to exit immediately.  proxywincosnole seems capable of doing this, except that it echoes the input, making the password show up on the screen, which is undesirable.
Understanding that it's not supported, what challenges would exist in attempting to implement this?

Maybe winpty is hitting EOF on stdin and immediately shutting down?  Maybe it'd need to keep running instead.
It might be as simple as removing the inputHandler.isComplete() check from https://github.com/rprichard/winpty/blob/d02e42d815823b9e87999700695e94b9b4718e26/src/unix-adapter/main.cc#L612:
        // Check for an I/O handler shutting down (possibly indicating that the
        // child process has exited).
        if (inputHandler.isComplete() || outputHandler.isComplete() ||
                (errorHandler != NULL && errorHandler->isComplete())) {
            break;
        }

Any update on this?
:terminal command on Vim use winpty. And currently we have issue about echo-back. :terminal command can write standard input to the command like below.
:'<,'>terminal wc

But winpty doesn't support writing stanadard input for the command. So CUI app show echo-back. Do you have plan to use input from pipe?
https://github.com/rprichard/winpty/blob/master/src/agent/Agent.cc#L376-L382

Other use cases:

peco doesn't work via winpty.
E.g.: Running ls | winpty peco on mintty doesn't work.
Vim can open stdin by specifying "-" as a filename.
E.g.: Running dir /b | vim.exe - on Command Prompt can edit a filelist of the current directory.
However, running ls | winpty vim.exe - on mintty doesn't work.
(Assuming this vim.exe is Win32 version, not Cygwin or MSYS2 version. Of course, if Cygwin/MSYS2 version of vim is used, ls | vim - should work. This is just an example.)

I think there are more other use cases for piping stdin.

@rprichard How about this?
diff --git a/src/agent/Agent.cc b/src/agent/Agent.cc
index efc3e7f..b0ebca6 100644
--- a/src/agent/Agent.cc
+++ b/src/agent/Agent.cc
@@ -149,7 +149,8 @@ Agent::Agent(LPCWSTR controlPipeName,
              int initialRows) :
     m_useConerr((agentFlags & WINPTY_FLAG_CONERR) != 0),
     m_plainMode((agentFlags & WINPTY_FLAG_PLAIN_OUTPUT) != 0),
-    m_mouseMode(mouseMode)
+    m_mouseMode(mouseMode),
+    m_useStdin((agentFlags & WINPTY_FLAG_STDIN) != 0)
 {
     trace("Agent::Agent entered");

@@ -380,6 +381,21 @@ void Agent::handleStartProcessPacket(ReadBuffer &packet)
         sui.hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE);
         sui.hStdError = m_errorBuffer->conout();
     }
+    HANDLE readtmp, readpipe;
+    if (CreatePipe(&readtmp, &m_stdin, NULL, 0)) {
+        if (DuplicateHandle(
+            GetCurrentProcess(), readtmp,
+            GetCurrentProcess(), &readpipe,
+            0, TRUE, DUPLICATE_SAME_ACCESS)
+        ) {
+            inheritHandles = TRUE;
+            sui.dwFlags |= STARTF_USESTDHANDLES;
+            sui.hStdInput = readpipe;
+            sui.hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE);
+            sui.hStdError = GetStdHandle(STD_ERROR_HANDLE);
+        }
+        CloseHandle(readtmp);
+    }

     const BOOL success =
         CreateProcessW(programArg, cmdlineArg, nullptr, nullptr,
@@ -429,7 +445,15 @@ void Agent::handleSetSizePacket(ReadBuffer &packet)
 void Agent::pollConinPipe()
 {
     const std::string newData = m_coninPipe->readAllToString();
-    if (hasDebugFlag("input_separated_bytes")) {
+    if (m_stdin != nullptr) {
+        if (newData.size() > 0) {
+            DWORD written = 0;
+            WriteFile(m_stdin, newData.data(), newData.size(), &written, NULL);
+        }
+        if (m_coninPipe->isClosed()) {
+            CloseHandle(m_stdin);
+        }
+    } else if (hasDebugFlag("input_separated_bytes")) {
         // This debug flag is intended to help with testing incomplete escape
         // sequences and multibyte UTF-8 encodings.  (I wonder if the normal
         // code path ought to advance a state machine one byte at a time.)
diff --git a/src/agent/Agent.h b/src/agent/Agent.h
index 5775e08..26940cc 100644
--- a/src/agent/Agent.h
+++ b/src/agent/Agent.h
@@ -76,6 +76,8 @@ private:
     const bool m_useConerr;
     const bool m_plainMode;
     const int m_mouseMode;
+    const bool m_useStdin;
+    HANDLE m_stdin = nullptr;
     Win32Console m_console;
     std::unique_ptr<Scraper> m_primaryScraper;
     std::unique_ptr<Scraper> m_errorScraper;
diff --git a/src/include/winpty_constants.h b/src/include/winpty_constants.h
index 11e34cf..316b3ed 100755
--- a/src/include/winpty_constants.h
+++ b/src/include/winpty_constants.h
@@ -76,11 +76,14 @@
  * See https://github.com/rprichard/winpty/issues/58. */
 #define WINPTY_FLAG_ALLOW_CURPROC_DESKTOP_CREATION 0x8ull

+#define WINPTY_FLAG_STDIN              0x10ull
+
 #define WINPTY_FLAG_MASK (0ull \
     | WINPTY_FLAG_CONERR \
     | WINPTY_FLAG_PLAIN_OUTPUT \
     | WINPTY_FLAG_COLOR_ESCAPES \
     | WINPTY_FLAG_ALLOW_CURPROC_DESKTOP_CREATION \
+    | WINPTY_FLAG_STDIN \
 )

 /* QuickEdit mode is initially disabled, and the agent does not send mouse

Maybe, this is better to have winpty_close_stdin().

When pipe is closed, m_coninPipe->isClosed() should return true, So probably, winpty_close_stdin is not required. (AFAICS)

Works fine for me. :)

This patch is not enough to use stdin. Probably stdin-pipe should be provided separated from conin-pipe.

@rprichard do you have interesting about stdin support? If you do, I'll send pull-request based on patch above.

Sorry, I'm probably not going to have time to work on this soon. I don't know when I'll be able to.

@rprichard Do you have plan to move repository into org and assign contributors for winpty? AFAIK, winpty is used in some projects. Visual Studio Code, vim, etc.

No, I don't have a plan like that.

Okay. Can I help something to go forard for this?


Success result is returned even light-oauth2 returns an error message

In OauthHelper::getTokenResult, it blindly returns success if there is anything returned from the light-oauth2 token service. Sometimes, the returned body might be an error which needs to be checked before constructing the result.

 Success result is returned even light-oauth2 returns an error message

 Success result is returned even light-oauth2 returns an error message


Please add a type

npm install @types/express-winston
404 Not Found: @types/express-winston

I know it's a rather old issue, but this has been solved now. See: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/express-winston


Determining runner when test files have near identical syntax?

Is there an established way to determine the runner when test files have near identical syntax? Was considering creating a PR to add support for Kahlan and Peridot.
These both use the extension of .spec.php and have very similar syntax. Only thing I can think of is check for the presence of the binary or grep composer.json to see if it's in there. Thoughts?

Yeah, the same thing is bugging me as well, there is no established way yet. The JavaScript and Python runners suffer from the same problem, though for JavaScript you can at least check in node_modules/ and package.json.
If composer.json is where you declare dependencies, then looking in it definitely sounds like a good idea. Though I guess not every PHP project uses a composer.json, right?

You'd be hard pressed to find a recent PHP project that doesn't use composer

I am not such an expert in Viml. How can I check if a string appears in composer.json and store the result as a boolean or count?

Take a look at s:rails_version(), where I check Gemfile.lock (like a Ruby alternative for composer.json) for Rails and extract the version. Basically the method is readfile() which returns an array of lines, and then you can iterate over those lines and use matchstr(), which returns either the matched string or 0 (since Vim has a different regex behaviour by default, I prefer to add \v at the beginning which makes it a regular POSIX regex).

I decided the best and simplest solution for my PR #178 was just to check for the presence of the runner executable.


x/build/cmd/gerritbot: editing a title or description should update the Gerrit change

In this change the title needs to be adjusted, but it won't get imported unless the PR branch's HEAD SHA changes. We should detect differences in the title and import if need be.

This is also affecting PR https://github.com/golang/tools/pull/26#issuecomment-368262382.


Usage is always null when done streaming chat

Bug Report
Overview
Usage is always null when streaming, even if the stream is complete.
To Reproduce
Attempt a stream against the chat models via StreamCompletionEnumerableAsync.
Expected behavior
At the end, I would expect the last result to contain Usage, but it doesn't (at least on my PC).
Screenshots

Additional context
The non-streaming way does return non-null Usage.

Sorry I misread your issue.
I will see what I can do to check to make sure the usage reflects properly in the event data, and if it does, set that in the final message result.

Thanks for looking into this, I appreciate the helpful notes!


Documention/usage of withTRPCNext

// Copied from https://trpc-swr.vercel.app/server-side/setup#using-nextjs
import { withTRPCSWR } from "@trpc-swr/next"
const MyApp = ({ Component, pageProps }) => {
  return <Component {...pageProps} />;
};

export default withTRPCSWR({})(MyApp);

The import is renamed to withTRPCNext. The withTRPCNext function expects 2 arguments, and both of them cannot be empty apparently.
The following line throws the error
export default withTRPCNext({}, {})(MyApp);

// error: Argument of type '{}' is not assignable to parameter of type 'CreateTRPCClientOptions<AnyRouter>'.

I tried passing the trpc config like below. But it still expects more the second argument to be fullfilled
export default withTRPCNext(
   {
      transformer: SuperJSON,
      // transformer: superjson, // optional - adds superjson serialization
      links: [
         httpBatchLink({
            url: `${process.env.NEXT_PUBLIC_API_HOST}`,
            fetch(url, options) {
               return fetch(url, { ...options, credentials: 'include' })
            },
         }),
      ],
   },
   {}, // Error
)(MyApp)

// Type '{}' is missing the following properties from type '{ createClient: CreateClient<AnyRouter>; useContext: () => TRPCContextType<AnyRouter>; Provider: TRPCProvider<AnyRouter>; SWRConfig: FC<...>; }': createClient, useContext, Provider, SWRConfigts

I'm not quite sure what to pass here. A hint would helpful. Heres my package versions.
"typescript": "~5.2.2",
"@trpc/client": "^10.43.1",
"@trpc/server": "^10.43.1",
"@trpc-swr/next": "^2.0.1",
"swr": "^2.2.4",

Hi, apologies for the late answer! This is a documentation error. withTRPCNext is internal code which is exported for legacy reasons. I will be revisioning the documentation this week to split everything into plain react, Next App dir and Next pages dir. Glad you caught this, thanks for the issue!
Here is the correct usage:
// lib/trpc.ts
import type { AppRouter } from "@/server/_app";
import { createTRPCSWRNext } from "@trpc-swr/next";
import { httpBatchLink } from "@trpc/client";

export const api = createTRPCSWRNext<AppRouter>({
	links: [
		httpBatchLink({
			url: "/api/trpc",
		}),
	],
})

// _app.tsx
import { api } from "@/lib/trpc"
export default api.withTRPC(MyApp)

Lets leave this issue open until documentation is improved.

Thank you for following up with this. I have one more question
Can we create server side helpers without access to the router and context?
According to the docs i need to create a ssr helper like this, which expects my context and router.
export const createSSR = () => {
    return createProxySSGHelpers({
        ctx: {}, // my context expects req,res,db etc
        router: trpcRoot,
    })
}

trpc has this docs for external routers.
But thats for react-query. Is it possible to achieve this with trpc-swr?

Its not possible with the SSG helpers unfortunately. SSG helpers are a wrapped version of router.createCaller.
You can get away with not passing context though, but that depends on what you actually need in your context. What's your use case for not passing in router, req res etc?

Yah and importing too. I have a fastify setup. Its a lil annoying to re-structure the router out of the backend module.
Importing router directly causes nextjs to process the backend which is set up and bundled somewhat differently. So I'm immediately greeted by some errors.
Having them separate has been really intuitive for me. Also useful for deploying them separetely. For example the nextapp to the edge and the backends to actual servers.
Thank you for clarifying thou. I'll figure somth out

I also keep mine separate, but very separate so I can drag the tRPC router into any framework. You need modify the tRPC context and create two context adapter functions, one for next and one for fastify. Then you need one unified context which is the context type you'll use for the router itself, both of the functions should produce the same unified context in the end.
I definitely see the use case for this though. What you're asking for is the tRPC client to run on the server over http instead of createCaller and have the added SWR helpers for hydration etc.
Its definitely doable!

My bad lol misclick.
Thank you for the sugestion and yah this feature would indeed be helpful for users.
My suggestion would be to keep it in the roadmap, if this gets enough attention one day, this would be a worthwhile implemention.

#54 Is in canary right now with some changes for Next 14 AppDir.
Going to improve docs and refactor some more pages dir related code this week.


Docs: Added JS Docs Examples for typedDatadocs

Checklist:

[x] Performed a self-review of the code
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Solves #1090

Kindly review @ivpavici

kindly review @ivpavici

@dsperac refactor and close this one


Add support for Garden

🎉 New feature
Summary

Update CMakeLists.txt and package.xml files to reference Garden dependencies.
Update README and RELEASING to document Garden support
Add workflow job for Fortress + Humble

Test it

Checklist

[ ] Signed all commits for DCO
[ ] Added tests (not applicable)
[ ] Added example and/or tutorial (not applicable)
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I presume the Github workflow won't support Garden until binary releases exist; correct me if I'm wrong.


Console Spam due to cobble Generator

Using the All The Mods 1 pack using version 2.2d --1.10.2 , a player placed a cobblestone Generator over a hopper causing a ton of console spam containing the following.
[ic2-poolthread-6/WARN] [IC2.EnergyNet]: Tile nc.tile.passive.Passives$TileCobblestoneGenerator@2dab83df (BlockPos{x=-3851, y=67, z=-6571}) removal without registration
PS since the IC2 api seems involved, I have industrialcraft-2-2.6.210 -ex110 installed also if that matters.
I tried to remove cobble using an EnderIO item conduit which also creates the spam. I am unsure how to get the cobble out without spamming console.

Secondarily, can you at least make a config option that this cobble gen may require RF to operate?

Hmmm... that's a strange one, but I'll see if I can find the cause. As for adding your suggestion, sure - shouldn't be a problem.

How much power (RF/t) do you think the cobble gen should use if it needs it?

There doesn't seem to be a way to fix this - Mekanism appears to have a similar issue. The problem is that the machine has to be removed and added back into the IC2 energy-net every time the block updates, as otherwise cables connecting to it disconnect permanently.

Added the RF gen config option :)


time_format configure

Describe the bug
I have prolem with time zone log fluend!
when i run any service and fluentd service.
fulend was able to get the service's log but the log time is incorrect
time displayed on log is path + time + ".log" (testlog.19700101_0.log)

To Reproduce
version: '3.7'
networks:
microservicesnet:
services:
fluentd:
container_name: fluentd
user: root
image: fluent/fluentd:v1.13.0-1.1
restart: always
ports:
- "24224:24224"
- "24224:24224/udp"
command:
/usr/bin/fluentd -c
/fluentd/etc/fluent.conf -v
volumes:
-  /var/lib/docker/containers:/fluentd/log/containers
- ./fluent.conf:/fluentd/etc/fluent.conf:ro
- ./logs:/output/
networks:
- microservicesnet
eureka
eureka:
build:
context: ../fluentd-docker-logging-master/pe4j-eureka-server
dockerfile: Dockerfile
environment:
TZ: Asia/Tokyo
image: eureka
ports:
- "8761:8761"
container_name: eureka
logging:
  options:

    tag: "docker.{{.Name}}"

restart: on-failure
depends_on:
  - fluentd
networks:
  - microservicesnet

Expected behavior
Your Environment
- Fluentd version:1.13.0

Your Configuration
 <source>
  @type tail
  format json
  read_from_head true
  tag docker

  path /fluentd/log/containers/*/*-json.log
  pos_file /tmp/container-logs.pos
</source>

<match docker>
  @type file
  path /output/testlog
  utc
  # time_slice_format %Y%m%d
  # time_slice_wait 10m
  # time_format %Y%m%dT%H%M%S%z
  # compress gzip
</match>

Your Error Log
1970-01-01T00:33:41Z	docker	{"log":"2021-12-09 08:45:03 +0000 [info]: #0 fluentd worker is now running worker=0\n","stream":"stdout"}
1970-01-01T00:33:41Z	docker	{"log":"\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":"  .   ____          _            __ _ \n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":" /\\ / ' __ _ () __  __ _ \ \ \ \\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":"( ( )\_ | '_ | '| | ' \/ ` | \ \ \ \\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":" \\/  )| |)| | | | | || (| |  ) ) ) )\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":"  '  || .__|| ||| |\, | / / / /\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":" =========||==============|/=///_/\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":" :: Spring Boot ::       (v2.3.12.RELEASE)\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
1970-01-01T00:33:41Z	docker	{"log":"\n","stream":"stdout","attrs":{"tag":"docker.eureka"}}
Additional context
No response

after  configure fluent.conf file time is correct but not automatically generate log file

  @type tail
  format json
  read_from_head true
  tag **

    @type "json"
    time_format "%Y-%m-%dT%H:%M:%S.%NZ"
    time_type string

path /fluentd/log/containers//-json.log
pos_file /tmp/container-logs.pos
<match **>
@type file
path /output/testlog
utc
time_slice_format %Y%m%d
time_slice_wait 10m
time_format %Y%m%dT%H%M%S%z
compress gzip

This seems to be a problem with your configuration file. Did you manage to solve this?

This seems to be a problem with your configuration file. Did you manage to solve this?
log file was generated after a day. 👍


Changed the link for Stack Overflow mlkit

added the right tag name: firebase-mlkit

Pull request 500!


Cast Android TV Receiver - Correct implementation with Media3

What is the current best approach to implement the Cast Android TV Receiver SDK with Media3?

I did not find any useful information on this to date.
The Media3 Demo project directory contains only Cast Sender example.
ExoPlayer project related references Cast docs, CastAndroidTvReceiver example, Marc Medium post are all outdated, deprecated and they no longer exist in Media3 (ie MediaSessionConnector).
Based on findings from CastPlayer source code, it always uses remoteMediaClient.queueLoad method, which requires a specific approach on Receiver side.

I have a very basic implementation done at the moment, but I don't know exactly when and what to broadcast with the MediaStatus differently from the official example mentioned above.
For example (int) currentTimeline.getPeriod(mediaItemIndex, period).uid in CastPlayer seekTo method returns (after disconnection and after new connection) wrong value (exceeding the queue size). But this works fine with the Default Web Receiver App, so from this I conclude that I am missing something to properly integrate on the Receiver side.
Is there an official demo planned, like is there the one for Cast Sender?

Like you state, the CastPlayer uses the RemoteMediaClient to prepare the player with a queue and send playback commands. These messages are then sent by the Cast SDK to the receiver SDK on the cast device.
In the manifest of the cast demo app is a cast options provider registered. It uses DefaultCastoptionsProvider provided by the library that declares the app id to be started.
You can do your own receiver and change that ID to your app ID.
I don't know much about the receiver side API to be honest. The web receiver SDK gives you a <cast-media-player> element and API components like a context, a PlayerManager and friends. Your app basically displays that HTML-element and the playback commands sent by CastPlayer through RemoteMediaClient are received and applied by the SDK on the receiver end.  The rest around that is your HTML/JS app.
From what I know, CastPlayer works with the default receiver, a styled receiver or such a custom receiver that uses the Cast web SDK.
For more information on how to implement such a receiver we can't help you here in much more detail I'm afraid. The links above should be a starting point.

First thank you for your attention Marc,
I know you get a lot of questions every day and I'm sorry, I probably didn't describe my question well enough. I'm working on integration Cast Android TV Receiver App (not the Web Receiver App):

And as I mentioned above, I have already completed the basic integration and it partially works, but I have already spent a lot of time on some edge cases and I still cannot solve some of them.
That's why I asked if you are planning any official demo / integration support, like you have for the Cast Sender part (the CastPlayer)?
Btw: The link to your medium post with the old MediaSessionConnector I mentioned because it is listed as a reference link in the code lab for ATV app integration at the very bottom of the page 4 🙂

Yeah, sorry I'm not familiar with this. That's an app published by Cast folks. We weren't involved.
I can just point to this page that references a code lab and a sample app, but you are probably better informed about these resources than I am.
We don't have plans to provide a Android TV receiver app any time soon.
Sorry for not having a better answer.
@pflammertsma or @AquilesCanta  may have some thoughts.

@marcbaechinger  @AquilesCanta @pflammertsma
hello guys, any update regarding this?
@TomasValenta  were you able to figure it out on your own?


Is there a bad way of using the loadable lazy loading thing?

can i just use it with every component and container?
or is there some thought to it?
purely a question and I couldn't find good answers on the internet
because I'm seeing weird behaviours and I'm suspicious of it

I never had any troubles with it. Container or component is a logical abstraction. For React they are all the same. Its not special to this boilerplate. I basically wrapped react features with a function.

https://github.com/facebook/react/issues/16493 i had this bug and thought it was something wrong with the component, Sorry!