FreeSurfer brain mask issue in ds000148

ANTs worked well

But Freesurfer applied the mas with an offset

sub-28

Seems like t1_merge did not perform very well. The ANTs mask also includes part of the eyeball so I would say it is not perfect either.

Look at the raw data first

This is a duplicate #841.


what's wrong with my demo.py ?

self.device = dai.Device(self.create_pipeline())
RuntimeError: No available devices

You get this error message when the device is not connected or connection is not ok. Check your connection.

I have one webcam connected to my computer and it can work well with python open cv ,so it is strange to see that error

Have you followed the install requirements described there : https://docs.luxonis.com/en/latest/pages/tutorials/first_steps/#install-requirements (in particular if you are on linux, the commands in the Warning paragraph) ?

Do I have to use OAK-1 or OAK-D ? I now just use ordinary webcam for testing.

Sure, you have to :-)
Like it is said in the first line of the README:
Running Google Mediapipe single body pose tracking models on DepthAI hardware (OAK-1, OAK-D, ...).
I have another repo (with an older version of the models) that should run with your webcam: https://github.com/geaxgx/openvino_blazepose

thank you so much， but can you tell me the relationship between Mediapipe and blazepose ?and where I can buy the OAK-1 ?

Closed.


java lang  Math.cos()  can't be parsed to 'Ast'

Version: 2.2.0
Module: quill-jdbc
Database: quill-jdbc
Perhaps I’m missing something. I really appreciate any help.
Expected behavior
Just work
Actual behavior
cmd18.sc:1: Tree 'java#10.lang#64.Math#618.cos#43992(t#86508.p#23314)' can't be parsed to 'Ast'
val mapped = quote {
                   ^
Compilation Failed

Steps to reproduce the behavior
Using Ammonite scala shell:
import $ivy.`io.getquill::quill-jdbc:2.2.0`
import io.getquill._
val ctx = new SqlMirrorContext(MirrorSqlDialect, Literal)
import ctx._
def myRandom() : Double = {
  var r = new scala.util.Random()
  2 * 3.1416 * (r.nextInt(2000)/2001.0)
}
case class R0Tuple(x: Int, p: Double = myRandom(), q: Double = myRandom()) {
}
case class R1Tuple(x: Int, p: Double, q: Double, c:Double = Double.NaN, s:Double = Double.NaN)
val mapped = quote {
    query[R0Tuple].map(t => R1Tuple(t.x, t.p, t.q, java.lang.Math.cos(t.p)))
  }

Workaround
I don't get any workaround for this problem.
@getquill/maintainers

@joao-parana Hello, everything which is inside a quote will be translated into target language (e.g. SQL) hence it need to be known at compile time by quill parser or extended with infix see http://getquill.io/#extending-quill-infix.
for example:
val cos = quote {
  (i: Int) => infix"cos($i)".as[Int]
}
ctx.run(query[Entity].map(p => cos(p.i)))
//SELECT cos(p.i) FROM entity p

Please use our gitter channel on general purpose questions.

Tank you @mentegy


In python module tests paths to the data should be determined relative to file

Now they are just the paths relative to the current directory which makes a one who wanna run the tests to go to the main repo directory first and run the tests from it. It's better to
import os
currentDir=os.path.dirname(__file__)
mainDir=os.path.join(currentDir, "..", "..")

and then compute data paths relative to the main dir

???

The testcase structure is quite common in python and I think we will keep it as it is. We are sorry for the massive close the issues. The main goal is to minimize the amount of issues so that actually the developer can take action on them. Please feel free to open new issues when you think is useful.
The discussions are now welcomed at https://discuss.xgboost.ai/

Bulk closing issues is a sign of disrespect to the people who have spent own time to write an issue to improve the project a bit.

Letting a long list of issues rot without any response is also a sign of disrespect. Yes, bulk closing was not quite ideal, but we maintainers feel that having a short, organized list of issues that are actionable would help everyone coordinate efforts.

bulk closing issues even with code and proposals on improvement

We will re-open issues containing bug reports. As for feature requests, we plan to maintain a single list of feature requests (#3439) so that potential contributors could easily find them.


Feature Request: Add Support for Proactive Remediation scripts

https://graph.microsoft.com/beta/deviceManagement/deviceHealthScripts

@ugurkocde maybe something I can help you out with? It will almost be the same function as the platform scripts with a few little differences here en there.
If you could give me access to the 2.6 with the platform script I could continue on your platform script function.

Finished and shipped with 2.6.0. Thanks for the Feature Request :)


Feature request: make polling interval  configurable in get_lock

I'm referring to:
https://github.com/bcit-ci/CodeIgniter/blob/9b8f2b7a8405acd1b8ad5956ada3d84472b1e8ae/system/libraries/Session/drivers/Session_redis_driver.php#L441
https://github.com/bcit-ci/CodeIgniter/blob/9b8f2b7a8405acd1b8ad5956ada3d84472b1e8ae/system/libraries/Session/drivers/Session_redis_driver.php#L448
(and memcached too)
As stated in https://github.com/bcit-ci/CodeIgniter/pull/5708#issuecomment-473145166 the optimal sleep time value may vary depending on the context (for instance, in my context, it's a very low value).
So, I kindly ask you to make the sleep timeout configurable like the other session parameters. At the moment, I'm extending the driver in order to override _get_lock, but it's not an optimal solution (the method stays the same except for the timeout...).
I'm available to submit a PR, if you need.
Thanks in advance

no update :(

Would be a great feature for our Azure Redis setup.


[google_sign_in] Add iOS unit tests

Add google_sign_in iOS unit tests to validate we don't break (much) in preparation for https://github.com/flutter/flutter/issues/86436, which included SDK removals and signature changes without deprecations. Native code plugin coverage increased from 41.4% to 78.4%.
Add lightweight generics to the plugin.

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The description says "Turn on code coverage", but I'm not seeing project changes other than deployment target.

Ah sorry I removed that in an intermediate commit since it's not being checked anywhere, so not worth the overhead in CI.  Fixed the description.


Improve rayx-core test coverage

Slit is now tested, and we finally have tests that require non-sequential tracing.


Tag Preview only showing app icons instead of images of windows

I'm on NixOS with awesome-4.3.  I installed the bling repo as instructed (cloning into .config/awesome/bling).  When I installed it I found that it threw an error when I tried to load the awesome config.  I "fixed" this by commenting out the module = require(... .. ".module"), line in bling's init.lua.  I reasoned that this shouldn't affect the tag_preview functionality but maybe I'm wrong?
After commenting out the require .module line I enabled the tag_preview exactly as instructed in the docs.  And it seems to be working but the preview shows only the window layout with program icons rather than the content of the actual window?  Is there some reason why it might be failing to get the image?

I think the issue of the first paragraph is #218
The issue of the second paragraph, I don't know


Fix multi-value context key DynamoDB example

Issue #, if available:
N/A
Description of changes:
The DynamoDB multi-value context key examples omit the fact that GetItem, PutItem, UpdateItem (and likely other operations) always include the primary key attributes in the dynamodb:Attributes context key. Therefore:

The ForAllValues example provides a false sense of security since it does not protect for the case when the user implicitly requests all attributes by not specifying a ProjectionExpression.
The ForAnyValue example provides a condition that will never evalute to false since it includes the partition key in the value list.

This commit fixes the previously mentioned problems in the examples. Additionally, the ForAnyValue example is updated to use the UpdateItem operation instead of PutItem since it has more semantic meaning.
Other minor changes like the order of appearance of Warning and Important messages and the reference to the DynamoDB documentation are updated to more logical places.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

👋

Thank you for taking the time to submit this pull request. This topic will be reviewed for accuracy and updated accordingly.
Once I have more information I'll provide an update here.

Closing this issue or pull request in advance of archiving this repo. For more information about the decision to archive this repo (and others in the 'awsdocs' org), see the announcement on the AWS News Blog.


New Functionality - useDateFieldInfo()

I have implemented a new functionality in the project that introduces a feature similar to the existing useStringFieldInfo() function. The new function is called useDateFieldInfo(). It provides essential information and utilities for handling date fields within the project.

https://github.com/iway1/react-ts-form/pull/115

merged and released


[Sign Up page] Incorrectly displayed labels on Sign Up form

Environment: Google Chrome Version 119.0.6045.201 (Official Build) (64-bit)
Preconditions
Page https://zero-waste-staging.onrender.com/en is opened.
Steps to reproduce

Go to 'Sign Up' page.

Actual result
'Email..' label is described as mandatory field and it is marked with an asterisk.
Postconditions
'Email', 'Password', 'Re-password', 'First Name', 'Last Name' labels should be displayed as mandatory fields with asterisks.
'Email..' label should be displayed correctly as 'Email'.
User story and test case links
"User story https://github.com/ita-social-projects/ZeroWaste/issues/626
TC #


Adding SVG subelements via Turbo Streams leads to them not being rendered

Overview
I'd like to edit SVG elements with turbo streams. One problem is that when I try to add an element via a turbo_stream, it isn't rendered. For instance, appending a <circle> inside of an SVG.
Code example
Showing the code for a rails app, but the idea would be the same with another framework:
html page:

  <svg>
    <g>
      <g is="turbo-frame" id="circles">
        <circle r="15" fill="#004d40" transform="translate(50, 50)"></circle>
      </g>
    </g>
  </svg>

  <%= button_to "Action", action_path, method: :post %>

Turbo stream event:
<%= turbo_stream.append "circles" do %>
  <circle r="15" fill="#004d40" transform="translate(50, 100)"></circle>
<% end %>

After the event fires, we see it in the browser's inspect, and it's in the right spot. But it's not visible:

We can "fix" it by wrapping the <circle> in its own <svg>, but then there's a bunch of unneeded SVGs.
My uninformed hypothesis is that it has to do with element namespaces? And that SVGs may require the namespace when adding elements?

After checking the namespaceURI of each element, I do think it might be the problem. For the existing SVG circle, it's 'http://www.w3.org/2000/svg' and for the circle that the turbo stream adds it's 'http://www.w3.org/1999/xhtml'

The append action uses document.append under the hood which by default will use the namespace http://www.w3.org/1999/xhtml and has very strict rules about it, you can get around this by using a custom turbo stream action that implements document.insertAdjacentHTML on 'beforeend' to get the same append behavior but more flexibility with namespaces.
This is an example of a custom action to handle this that I've tested
import { StreamActions } from "@hotwired/turbo"

StreamActions.appendSvgCircle = function() {
  this.removeDuplicateTargetChildren()
  this.targetElements.forEach((targetElement) => {
    let div = document.createElement('div');
    div.appendChild(this.templateContent.cloneNode(true));
    targetElement.insertAdjacentHTML('beforeend', div.innerHTML);
  })
}


[Dependency Scanner] Ensure the Clang dependency scanner working directory matches the invocation.

The Swift compiler does not have a concept of a working directory. It is instead handled by the Swift driver by resolving relative paths according to the driver's working directory argument. On the other hand, Clang does have a concept working directory which may be specified on this Clang invocation with '-working-directory'. If so, it is crucial that we use this directory as an argument to the Clang scanner API. Otherwiswe, we risk having a mismatch between the working directory specified on the scanner's Clang invocation and the one use from the scanner API entry-points, which leads to downstream inconsistencies and errors.

@swift-ci test

Can you add a test for this?

Without this change, https://github.com/apple/swift/pull/60807 fails all driver tests that perform dependency scanning (and explicit builds). This is because the driver runs tests in a different directory than where the compiler is located. So merging it will ensure we don't regress the correct behavior introduced by this change.
Otherwise it is a tricky thing to test without introducing the above change.

@swift-ci test


read incremental.md spec if instructions=md

This is an ugly quick hack but works for me, I created a file incremental.md with the spec of what I want it to do, type 'md' in the terminal and it reads the file
Ideally I want to have a folder with several files so I can add library documentation.
But an advanced system would use embeddings on the docs and library repos to create a more compact context.
Instead of reinventing the wheel its best to use llama-index and/or langchain to create the retrieval agent and smol only takes care of implementing changes in the repo.

Yea at some point , probably should rewrite this with langchain.js
Im just not familiar with that yet at the moment


[Bug Report] TypeError: slotActivator.querySelector is not a function

Describe the bug
Error in v-on handler: "TypeError: slotActivator.querySelector is not a function"

If you want to get the element of the input(or activator) slot, you should use this.$slots.input[0].elm, not this.$slots.input[0]
https://github.com/ly525/v-md-date-range-picker/blob/2ca9e997e1109c5f53411bd08fd4e964793aba5e/src/components/Picker.vue#L241
To Reproduce
Steps to reproduce the behavior:
<v-md-date-range-picker>
  <input type="text" slot="input" />
</v-md-date-range-picker>

Version Info (please complete the following information):

v-md-date-range-picker 2.5.0

:tada: This issue has been resolved in version 2.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Automated release notes

Both as a non-tech-savvy user as well as a developer, I'd be great to have release notes that are easily readable and nicely formatted. To improve the DX, the release notes as well as the associated GitHub releases should be generated automatically.
Specifications

Use GitHub Actions workflows
Create release notes/changelog based on Conventional Commits v1

Acceptance criteria

[x] A release can be triggered manually and by pushing a tag to the repository
[x] PR titles must follow Conventional Commits

[x] Easy to understand error messages if wrongly formatted title is present

The GitHub Action is currently also getting executed when pushing to a PR. This is caused by using the synchronize event, leading to the unwanted runs.


Add download/install instructions to README

npm install instascan and var Instascan = require('instascan');
Use instascan.min.js from release zip
Bower?

Fixed with<PHONE_NUMBER>0e6385a5b92116a59ae8860f0c668d.


gdx-pay (iOS)  transaction.getTransactionData() == null

Hello! I test in-app purchase service in my application (for ios) . Tell me please what's wrong: I made a purchase, it was successfully. But validation at the server not passed because "transaction.getTransactionData() == null".

I am not sure if that's actually implemented yet for the iOS backend? Maybe some of the iOS-backend developers are able to comment?

I looked at the code and the Base64 decoder seems to be the wrong one. I recommend to use java.util.Base64

Base64 has different implementations (i.e. slightly different characters):
https://en.wikipedia.org/wiki/Base64#Implementations_and_history
Just make sure to use the corresponding decoder/encoder!


Event delays when polling on underlying xcb_connection's FD

I'm having a strange issue that I can't seem to pin down.  When I use a simple event loop:
loop {
    let event = conn.wait_for_event()?;
    handle_xcb_event(&conn, event)?;
}

... then everything seems to work fine.  But if I try to poll (either using nix or mio) the FD returned by conn.as_raw_fd(), I get strange delays processing events.
I want to poll the FD because I have other FDs that I need to monitor as a part of my event loop.   Even if I keep it simple and only poll the XCB FD, I still see these weird delays, where events will get seen and processed by my application sometimes 10 or more seconds after they occur (i.e., a property change on a window I'm interested in).  Moving my mouse around over the windows I create in my app, or pressing keys on the keyboard, speeds up event delivery/handling.
For reference, things using nix look like this:
loop {
    let mut fds = Vec::new();
    fds.push(PollFd::new(conn.as_raw_fd(), PollFlags::POLLIN));

    let nready = poll(fds.as_mut_slice(), -1)?;
    if nready > 0 {
        for fd in fds {
            if fd.revents().filter(|pf| pf.contains(PollFlags::POLLIN)).is_some() {
                let event = conn.wait_for_event()?;
                handle_xcb_event(&conn, event)?;
            }
        }
    }
}

Any ideas?  Not sure if this is related to the XCB binding, or Rust itself, or just something I'm doing wrong...

Did some more experimenting, and this is indeed my fault.  It looks like libcxb will buffer data pulled from the FD, so there are two issues with my code:

When poll() says there is data available on the FD, I need to repeatedly call conn.poll_for_event() in a loop to ensure that I get all the events that have already been read from the FD.
I should unconditionally call conn.poll_for_event() at the start of my event loop (before calling poll()), because a previous conn.wait_for_reply() or something during my event handling code may have read and queued more events from the FD.

Sorry for the noise!


Add GlobalAveragePool3D Layer

This PR adds GlobalAveragePool3D layer (#82).
TODO:

[x] Add layer
[x] Export and Import Function (ModelLoader and ModelSaver)
[x] Add Documentation
[x] Add Unit Test

This PR also adds a unit test for GlobalAveragePool2D which was missing.

Also, you could merge with master to give me the ability to run TC build

@therealansh
"Copy to clipboard
e: /mnt/agent/work/c1e3495e9fda7093/api/src/main/kotlin/org/jetbrains/kotlinx/dl/api/core/layer/pooling/GlobalAvgPool3D.kt: (44, 14): Var-property public open val weights: Map<String, Array<>> defined in org.jetbrains.kotlinx.dl.api.core.layer.pooling.GlobalAvgPool3D cannot be overridden by val-property public abstract var weights: Map<String, Array<>> defined in org.jetbrains.kotlinx.dl.api.core.layer.Layer"
I've updated the weights property, please visit it in different layers like Merge layers to copy simple set method


CAMEL-17564: Added authSource param to MongoDB connection endpoint

The authSource parameter allows you to connect to a MongoDB database as a user whose credentials are stored in a database other than the default "admin". So if you have a user called test-user in registered in the test database, you can use the following endpoint:
"mongodb:mongo?hosts=localhost:27017&username=test-user&password=testpwd&authSource=test&..."

[X] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes. See CAMEL-17564
[X] Each commit in the pull request should have a meaningful subject line and body.
[X] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[X] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[X] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/main/CONTRIBUTING.md

I've talked to @djcoleman, I'll add the instance of later on. Merging.


Cannot load scripts or css files refered in html template when using webpack. 404 errors.

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.0.0-beta.61
Electron version
16.0.5
Operating system
Windows 11 (21H2)
Last known working Electron Forge version
NA
Expected behavior
I am trying to integrate the three.js editor into electron, and package it as a local app using electron-forge + webpack. There were some difficulties along the way, and webpack packaging did not work as expected.

Additional information
No response

Unfortunately, due to the limited amount of time of the maintainers and the limited amount of information present in this issue, we cannot help debug your app at this time. Instead, you should consider using one of the community forums, as there are more people there available to answer questions.


Dark theme issue in mobile screens

@arpanaditya After changing to dark theme, when clicked on the navbar icon, the theme gets toggled to light theme even before getting inside the navbar and selecting dark theme.

@iampsr8 Thanks for bringing this to notice. I can't seem to find what's causing the problem. Could you please help?

@arpanaditya  From what I seem to find you have implemented the set-theme state in the mobile component itself, where the useState is initialized with the light theme, so when we click on the mobile nav icon the mobile component opens and the theme gets initialized with light theme, analogous to that the same thing will happen in the web version as well i.e. when you reload the page in web the theme gets initialized with light theme. Instead of creating the state in the individual component you can create in the App.js file. Here I am linking some resources to help you

Dark theme using react
Dark mode react
Hope this helps👍


Let chatroom scroll to a specific chat

Contributes to https://github.com/Hylozoic/HyloReactNative/issues/663
This should be SAFE to merge, it only adds new functionality, no changes to routing etc

Yeah, I was getting it in my head that urls are case-insensitive but that
doesn't apply to query strings
On Sat, 14 Oct 2023 at 10:19, Tibet Sprague @.***>
wrote:

@.**** approved this pull request.
In src/routes/ChatRoom/ChatRoom.connector.js
https://github.com/Hylozoic/hylo-evo/pull/1555#discussion_r1359494001:

const search = getQuerystringParam('search', state, props)

const postIdToStartAt = querystringParams?.postid

Oh one more little thing, let’s call this postId
—
Reply to this email directly, view it on GitHub
https://github.com/Hylozoic/hylo-evo/pull/1555#pullrequestreview-1678244090,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHT4AQF7JTNRRWRZRZ5ZYLX7LCQJAVCNFSM6AAAAAA54PJO6SVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNZYGI2DIMBZGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Error for loading pretrained backbone of swin

when loading the pretrained backbone of swin downloading from https://github.com/SwinTransformer/storage/releases/download/v1.0.0/swin_tiny_patch4_window7_224.pth, there is no 'backbone' in the state_dict's key. Therefore, the state_dict from https://github.com/open-mmlab/mmdetection/blob/master/mmdet/models/backbones/swin.py will be an empty dict.
 state_dict = OrderedDict()             for k, v in _state_dict.items():                 if k.startswith('backbone.'):                     state_dict[k[9:]] = v

convert_weights=True adds 'backbone.' prefix.
https://github.com/open-mmlab/mmdetection/blob/ff9bc39913cb3ff5dde79d3933add7dc2561bab7/mmdet/models/utils/ckpt_convert.py#L135

In the configs/swin/retinanet_swin-t-p4-w7_fpn_1x_coco.py, add convert_weights=True to the dict backbone in the dict model.


Organization can set a logo

Org admins can set a logo for the organization on the settings screen. Logo shows up on the organization login screen

see #24 for a place to put the logo


Prepend relative volumes with absolute path

Closes #17.

Codecov Report
Attention: Patch coverage is 67.85714% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 73.31%. Comparing base (d20f7ac) to head (0240768).

Files
Patch %
Lines

compose.go
70.37%
7 Missing and 1 partial :warning:

main.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   73.32%   73.31%   -0.02%
==========================================
  Files           6        6
  Lines         851      873      +22
==========================================
+ Hits          624      640      +16
- Misses        190      197       +7
+ Partials       37       36       -1

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


MVA courses for understanding docs

I suggest that you should create some MVA courses which have experts talking about docs.

⚠ Idea migrated from UserVoice
Created By: Saalam Zia
Created On: 2017/03/11 09:30:33 +0000
Votes at Migration: 1
Supporters at Migration: 1

We are generally very open about docs - I will bootstrap a conversation with the team about documentation talks on existing video channels.
cc: @thedanfernandez @mairaw @BillWagner

Do you have any specifics about what kind of content about docs was requested?

@BillWagner content that shows how to contribute to docs, intricacies of the platform, etc.

Closing this issue for now. Please refer to our contributor guide to learn more about how to contribute to docs.microsoft.com.

There’s also the streaming that @csharpfritz and I did on Twitch talking about docs:
https://youtu.be/rZj-h9PQsi8


Add main and boot to package

Add the main.py and boot.py files to the setup.py to become part of a distribution

Duplicate of #19


Rename defaultSignal to zero?

Just another thought I'm having while tutorial-writing. I think we originally added defaultSignal to handle absurd cases where we don't care what's returned. But in some places (for instance, loops or delays with the default initial state) we now consider it okay to rely on the fact that the defaultSignal is the "zeroed out" version of the signal. I think a rename might make this clearer.

Perhaps we should split the use cases into two definitions - 'absurdSignal' and 'zeroSignal', perhaps that would help if we wanted to prove that 'absurdSignal' is never actually present in a design or similar.
We could also probably hide the defaults in a lot of places by making a specialised/wrapper for 'resize_default' that works with vectors of signals, and just calls resize_default default_signal.

Perhaps we should split the use cases into two definitions - 'absurdSignal' and 'zeroSignal', perhaps that would help if we wanted to prove that 'absurdSignal' is never actually present in a design or similar.

I think that if you use a defaultSignal for a certain case, and you prove that the circuit behaves correctly, then whether the case is not supposed to happen or whether your spec is satisfied by zero in that case is sort of an internal detail to the proof. Do you have specific cases in mind where it would be beneficial to expose the difference?

We could also probably hide the defaults in a lot of places by making a specialised/wrapper for 'resize_default' that works with vectors of signals, and just calls resize_default default_signal.

Already done! Vec.resize_default uses defaultSignal as the default:
https://github.com/project-oak/silveroak/blob/e83fe025e5186c59ea6e31256f4c2f8f86cc5505/cava/Cava/Lib/Vec.v#L144-L147

The concept of a default signal is an important one for the generated SystemVerilog because for each type we generate there is a default value it has in the SystemVerilog semantics and it is very important our definition of default value matches what SystemVerilog does (which at the bit-level corresponds to "all zeros").

I think that if you use a defaultSignal for a certain case, and you prove that the circuit behaves correctly, then whether the case is not supposed to happen or whether your spec is satisfied by zero in that case is sort of an internal detail to the proof. Do you have specific cases in mind where it would be beneficial to expose the difference?

I don't have a concrete use case and so we shouldn't worry too much, but it may be useful to prove non-functional properties - if an implementation unintentionally depends on an 'absurdSignal' it could result in additional unintended resource usage.


refactor(substrait): refactor ReadRel consumer

Rationale for this change
some minor refactoring after #12800 as promised.
Are these changes tested?
existing tests
Are there any user-facing changes?
no

🚀


[c++] Add ASAN run of CI to weekly CI job

Update bld/script.sh to skip downloading TileDB prebuilt for all non-Release configurations.
Add new weekly CI job that runs the CI for the ASAN configuration.
Update developer docs for building with sanitizers.

[sc-34306]

@jp-dark is there a way to manually trigger a run of this PR before merging it? (From https://github.com/single-cell-data/TileDB-SOMA/actions I think not ...)

@jp-dark is there a way to manually trigger a run of this PR before merging it? (From https://github.com/single-cell-data/TileDB-SOMA/actions I think not ...)

I updated it to run in the PR if it is modified. However, it is almost certainly going to fail.


power calculation for charger without meter

First I want to say - this is a great project - I really like it and have to say this is a great thing to reduce our CO2 footprint - thank you!
When the charging approaches the end of the cycle the car uses less current than the set current.
EVCC in my view calulates still with the set current, which results in wrong total charging power.
Likely this only applies to chargers that do not have a build in meter (I have an Elli Wallbox Connect).
Please see attached picture where the charging is close to end end of the charge cycle, with PV power of 6.9 kW.
The charge current in my best understanding is set to 7.4kW based on the export.
The home battery charging is at 3.7kW, and the typical daily home idle power is 0.3kW (here 0W as calculated).
As the output is now a 11.1kW we have a source - too good to be true.
For a charger without a meter I would think and consider.
Battery charging is correct as measured and delivered by the inverter (I have a SH10RT).
If the calculated "out" > "in" there is something weird happening, which likely is that the car consumes less than the allowed set current.
My idea would be, introducing an static idle current that would be set in evcc.yaml home_idle=200, representing a safe min of 200W home consumption.
As "out" = "in"
CO = Consumption
GE = Grid export from meter
BC = Battery charger from meter
HI = home_idle
CH = Charger
CHS = Charger Setpoint = progrmmed max charge power (current)
1st pass
CO = "in" - GE - BC - CHS
2nd pass - if CO<HI then {CH = "in" - GE - BC - HI; CO = HI}
In this way chargers without meters would get a safety check that prevents the issue "out" > "in" and that would increase the charge current intergral significantly.
Since some time in the charge log there is no odometer anymore (always 0km for an e-up using the VW template).

It is highly recommended to always use a chargemeter as many internal calculations also depend on it. Without real metering, this is just a kind of emergency mode.
The loading counters can be retrofitted internally in many boxes.
However, evcc also supports external meters. Here you are completely free in the choice of meter type and can use inexpensive models such as the SDM72DMv2.


Update content on

URL
https://difund.herokuapp.com/projects/digital-infrastrcuture-incubator
Content to be modified

We'd love it if this page would link to the Incubator site (https://incubator.codeforscience.org)
Can we change the title description from [Supporting projects to implement research-informed best practices at the time of need on governance, sustainability, and inclusion] to: [Supporting open source project leaders implementing best practices in sustainability, governance, and community health]
Can we change the subtitle from [Code for Science & Society is building an expert-advised cohort program for open digital infrastructure projects who wish to implement research-based recommendations relating to three common challenges faced by digital infrastructure projects: governance, sustainability, and community health.] to [Code for Science & Society's cohort program is designed to amplify the impact of research-based recommendations by providing tailored support for open source project leaders as they iterate solutions around a single sociotechnical challenge.]
Danielle Robinson is ED of Code for Science & Society, the other team members listed (Ruben and Talan) are not ours
We'd love to add Rayya El Zein (Program Manager, Digital Infrastructure Incubator) as another team member for this project

thank you!

Done!


At the moment, when the board shutsdown, the board appears in the resin dashboard as "Post provisioning". Isn't this the required behavior?

Seems not to be an issue anymore.


PlayerWarp

The ReadMe has a typo for
/api/v1/players/main/actions/warp/{location}: Warps the player to the specified location
which actually is
/api/v1/players/main/actions/warpPlayer/{location}: Warps the player to the specified location
But then again, it always returns a Bad Request, no matter what location I try (e.g. BusStop or Blacksmith). Teleporting there with console commands works fine.
Response status code does not indicate success: 400 (Bad Request)

Okay nevermind, just realized it a closed list:
public enum WarpLocation
{
Random = 0,
Farm,
Beach,
Mountain,
Desert,
Island,
FrontDoor
}


Console Buffer Overwrite

After 5 rows of output, the output buffer will overwrite the 5th row over and over again.

I'm unable to replicate on my end:


[CLI] URGENT! The Network Bytes Transfer Leak

Description
In the latest version of wandb=0.10.22, the Network Bytes transferred and received appears to increase drastically compared to a previous version wandb=0.9.5 that I had been using. See this report for an example.

I have started upgrading wandb to the latest version a couple of days ago and start running experiments on AWS as usual. As shown in the figure above, the network transfer for the latest wandb version is several magnitudes higher (about 100x more bytes transferred) than that of the older version. This issue unfortunately is URGENT and has a devastating effect on running on cloud providers that charge for data transfer such as AWS. As shown in the figure below, the EC2 costs have increased 10-15x due to the DataTransfer-Out-Bytes, costing thousands of dollars more on the same workload (about 28 concurrent experiments overnight).

How to reproduce

Download the [source code] (https://wandb.ai/costa-huang/cleanRL/runs/1vq7p5se/code?workspace=user-costa-huang)
Install the requirements.txt
And just run with python sdqn_atari.py
Then install wandb==0.9.5, and run the same script by python sdqn_atari.py
On the report page, select those two experiments and compare the network transfer

Environment

OS: Linux
Environment: my own machine and on AWS
Python Version: 3.8.8 (see the report)

Looks like this is a problem with syncing tensorboard. Setting wandb.tensorboard.patch(save=False) seems to reduce the data transfer significantly.
    import wandb
    wandb.init(
        project=args.wandb_project_name, entity=args.wandb_entity,
        # sync_tensorboard=True,
        config=vars(args), name=experiment_name,
        monitor_gym=True, save_code=True
    )
    wandb.tensorboard.patch(save=False)

Looks like this is not a problem with tensorboard specifically, but has to do with how files are synced in general. Had I recorded video through the gym interface (monitor_gym=True), the data transfer also explodes. Hmmm

Went through the debug-internal.log and debug.log and found a couple more things. It seems the problem is with the PolicyLive. Every run with the following debugging output seems to eventually blow up the data transfer.
saving file agent.pt with policy live

Given the same workload, this is the network usage of wandb==0.10.22

This is the network usage of the wandb==0.9.5:

My suspicion is that whenever a file is saved with that PolicyLive, it seems the client will try to upload it immediately once it was uploaded, resulting in the 15MB/s data sent. This means even if only the tensorboard files are saved, as they grow larger, the data transfer will explode. So is the case with any sort of file saving. In the earlier version of 0.9.5, however, it seems regardless of how many files there were to be saved, the uploading speed was capped at 200 KB/s data sent.

Hey @vwxyzjn sorry to hear you're seeing such a large difference between these versions, I want to help get to the bottom of this ASAP.
The logic for PolicyLive should be the same in 0.9.x and 0.10.x.   PolicyLive which is what we use for tensorboard event files or any files that match calls to wandb.save will upload the file atmost once every 15 seconds if the file changes.
There were changes to how we sync stdout / stderr between 0.9.x and 0.10.x.  Can you try running a job with wandb.init(..., settings=wandb.Settings(console="off")) and see if that changes anything?  Are you able to provide a sample script and the methodology you used to capture these stats?

Thanks for the prompt reply. I really appreciate your help. Below is a minimal script that I came up with the reproduce the data transfer issue:
import torch
import torch.nn as nn
import torch.optim as optim
import torch.nn.functional as F
from torch.distributions.categorical import Categorical
from torch.utils.tensorboard import SummaryWriter
import wandb
import time
import random
import os

experiment_name = "test"+str(int(time.time()))

run = wandb.init(project="cleanRL", entity=None,
                 sync_tensorboard=True, name=experiment_name,
                 monitor_gym=True,
                 save_code=True)

class Agent(nn.Module):
    def __init__(self, mapsize=16*16):
        super(Agent, self).__init__()
        self.mapsize = mapsize
        self.network = nn.Sequential(
            nn.Conv2d(27, 16, kernel_size=3, stride=2),
            nn.ReLU(),
            nn.Conv2d(16, 32, kernel_size=2),
            nn.ReLU(),
            nn.Flatten(),
            nn.Linear(32*6*6, 256),
            nn.ReLU(),)
        self.actor = nn.Linear(256, self.mapsize*78)
        self.critic = nn.Linear(256, 1)

    def forward(self, x):
        return self.network(x.permute((0, 3, 1, 2))) # "bhwc" -> "bchw"

device = torch.device('cpu')
agent = Agent().to(device)
optimizer = optim.Adam(agent.parameters())
writer = SummaryWriter(f"/tmp/{experiment_name}")
for i in range(20000):
    time.sleep(0.1)
    loss = agent.forward(torch.rand(4, 16,16, 27)).mean()
    optimizer.zero_grad()
    loss.backward()
    writer.add_scalar(f"test", loss.item(), i)
    print(i, loss.item())
    nn.utils.clip_grad_norm_(agent.parameters(), 0.5)
    optimizer.step()
    if not os.path.exists(f"models/{experiment_name}"):
        os.makedirs(f"models/{experiment_name}")
        torch.save(agent.state_dict(), f"models/{experiment_name}/agent.pt")
        wandb.save(f"models/{experiment_name}/agent.pt")
    else:
        torch.save(agent.state_dict(), f"models/{experiment_name}/agent.pt")

As shown in the following screenshot, the

And adding settings=wandb.Settings(console="off")) also does not help. See

In comparison, wandb==0.9.5 does not have this issue.

Thanks @vwxyzjn For now, let's change the your call to wandb.save and make it wandb.save(f"models/{experiment_name}/agent.pt", policy="end") and see if that fixes things.  We're still digging into the root cause of the difference between the versions.

Additionally its possible that this problem is accellerated when there are multiple runs in progress (these are all on different nodes)

@Bam4d I @vwxyzjn reports are related to upload traffic.  Your report seems connected to download traffic.  Can you share code that's causing the increase in downstream traffic?

@vanpelt I can confirm on my end wandb.save(f"models/{experiment_name}/agent.pt", policy="end") fixes the data transfer temporarily.

The logic for PolicyLive should be the same in 0.9.x and 0.10.x. PolicyLive which is what we use for tensorboard event files or any files that match calls to wandb.save will upload the file atmost once every 15 seconds if the file changes.

Btw I tested it and it seems in 0.9.5 the 21 MB model file are uploaded every 80 seconds or so, while in the 0.10.22 the model file is uploaded every second.

Thanks so much for looking into it @vanpelt. I don't know if it is related, but it seems some 0.10.22 runs are missing requirements.txt like https://wandb.ai/costa-huang/cleanRL/runs/27d439rk/files, whereas the 0.9.5 runs almost always store the requirements.txt.

Thanks @vwxyzjn I'm really glad it's working now, we're going to prioritize a fix to ensure we don't save files in "live" mode so frequently.
I have runs with the latest version of the cli that contain requirements.txt, I'm not sure what's going on with your runs.  Can you send the debug-internal.log from one of the runs that isn't saving requirements.txt?

Thank you!
https://gist.github.com/vwxyzjn/ea9519a94c4469ad252fc1064aa94b2b, with corresponding run https://wandb.ai/costa-huang/cleanRL/runs/3c9rycrc/files?workspace=user-costa-huang. In comparison, the requirements.txt is present for https://wandb.ai/costa-huang/cleanRL/runs/3kymam3z/files. In both cases, I crash the process, which seems to make a difference.
I think what happens is if the process crashed, then the requirements.txt is not saved. You can reproduce this by simply running the script in a terminal and then just force kill it. The real use case is when I run experiments using preemptible instances, they usually crash as well, so resulting in the requirements.txt not saved.
I think in the earlier version of 0.9.5, the requirements.txt is saved right in the beginning instead of the end like the latest version.

Hi @vanpelt, I have tried to use wandb==0.10.24 that seems to have fixed the frequent uploading issue. Thank you for the fix and I am closing the issue for now.

@vanpelt
I am facing a similar issue. Within 2-3 minutes of the run, 'sent' and 'received' shows 500MB and 2GB respectively. And my wandb setup is increadibly simple. I am using only these wandb functions.
wandb.init(project='proj_name', name='exp_name', entity="username", dir='<exp_log_directory>')
wandb.config.update(<argparse_config>)

wandb.log({key: value}, step=itr_num)   # storing loss periodically

Total stdout log file doesn't exceed 2MB. I am not using wandb for any kind of model saving or anything. Any help to resolve this issue will be greatly appreciated. (I have just started using wandb and it is great. But being a newbie, I don't know how to debug things yet.)

Hey @chaitanya100100, which wandb version are you using? Could you send the debug logs for this run?

@chaitanya100100 our network traffic graphs monitor network usage on the entire machine you're running on.  Is it possible there was a different user or process on that machine that was consuming bandwidth?

Hello @armanhar @vanpelt ,
I am using wandb version 0.12.10 .
I am running that experiment on a cluster machine. So yes, it might be possible that other processes are using that bandwidth.
Let me try to test this on an isolated machine. I will post my findings here soon.

@chaitanya100100 Hi there, I wanted to follow up on this request. Please let us know if we can be of further assistance or if your issue has been resolved.


Should n_dest be calculated using n() or ndistinct()?

If using n() -- denominator used to calculate weights (1 / n_dest) might be inflated by duplicate rows. This could be resolved by explicitly running dplyr::distinct() on the table first, but this raises issues with potentially losing values -- i.e. more structured duplication checking should be done BEFORE calculating weights, or checking values are appropriately duplicated across data map rows in one-to-many case.
It probably best to avoid invisibly running distinct() on the data, as it might be trickier/less obvious for a user to recreate the weights externally using an "unwrapped" dplyr pipeline. Using n_distinct() is somewhat more acceptable if the user is made aware of this...?
Perhaps it would be best to:

Include an explicit MESSAGE on how n_dest / weight are calculated as a kind of tidylog style self generating documentation -- possibly allowing for some custom function to be passed in?
WARN individuals if duplicate rows are present, SUGGEST tidying duplicates and re-creating data map

Some code that internally runs distinct() -- probably don't do this.
https://github.com/cynthiahqy/conformr/blob/cf769c091e7fee2c5c64cdbe02188f6ccb7e77b9/R/code-dict-tools.R#L66-L85

This is related to the validation condition for duplicate links inside the validate_xmap() generic function.


extract_headers Unit Tests

Basic unit tests for extract_headers method in runnable.

spec

That is not my understanding. The requestHeader can give any field/value for valid HTTP request headers - not just 'Accept' or 'Content-Type'. However, if the requestHeader element contains explicit 'Accept' and/or 'Content-Type' fields, the values for those fields are used in place of the individual 'Accept' and 'Content-Type' elements explicitly listed in the operation. I think this ability to overwrite likely has to do with the need to look for placeholders (for replace_variables) within the requestHeader values.

From the spec, use of the 'requestHeader' element is for explicitly set "Accept" or "Content-type". So does that mean use of requestHeader would end up with either "Accept" or "Content-type", but not both?

That is not my understanding. The requestHeader can give any field/value for valid HTTP request headers - not just 'Accept' or 'Content-Type'. However, if the requestHeader element contains explicit 'Accept' and/or 'Content-Type' fields, the values for those fields are used in place of the individual 'Accept' and 'Content-Type' elements explicitly listed in the operation. I think this ability to overwrite likely has to do with the need to look for placeholders (for replace_variables) within the requestHeader values.

Got it! Just approved the request.


Is it possible to forbid accessing missing keys?

Hi there,
I'm using addict.Dict instances to store some predefined configuration. The configuration is stored as JSON and loaded into Dict as follows.
import json
from addict import Dict
with open(filename) as f:
    config = Dict(json.load(f))

It works great. Then I use this configuration to set up some properties of my objects.
class Object:
    def __init__(self, config: Dict):
        self.x = config.section_a.x
        self.y = config.section_b.sub_section.y
        self.z = config.section_c.value
        ...  # etc.

The problem with this approach is that if I made a typo in the attribute name, I'll not get an attribute error but an empty dictionary and my class's internal logic fails later on when encounters dictionary type instead of expected value.
self.z = config.section_c.vallue   # typo that returns {}
...
z_squared = self.z**2  # fails because of the wrong type

I understand that this behavior is expected and required to enable the creation of dynamic nested dictionaries. However, in my case, I use Dict as a "frozen" configuration object that is not expected to be modified during the program's lifetime and would like it to fail early in case of typos or missing entries. With the current behavior, the program doesn't fail on an object's creation but raises a (non-obvious) error later on.
I wonder is it possible to add a frozen attribute to the Dict initializer? Like the following snippet shows.
from addict import Dict
config = Dict(config, frozen=True)

# or

import addict
config = addict.freeze(addict.Dict(config))

And if a "frozen" object is asked a missing key, the error is raised.
config.section.missing_key  # raises KeyError

Would be glad to hear your thoughts!

Hi @devforfu,
Good suggestion. I'd be open to adding this, sure! Would you like to take a stab at the implementation?

Hi @mewwts,
Thank you for the response! Yes, looks good! Probably it is worth also to return a Dict instance? Like c = Dict(config).freeze()?
Ok, sure, not a problem. I'll try to cope up with something and open a PR.

Yes, sure, but then in the example
a = Dict(config)
b = a.freeze()

I would expect b to be a separate instance to a, right?

Moved discussion to #123


Replace ng-app by angular.bootstrap

This way the module is bootstraped next to where it is defined

@dmitriz Thanks!


Sending a DM to closed account results in "Auth GetRestrictedAuthTokenByEmail error"

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Created new DM with<EMAIL_ADDRESS>in New Expensify
<EMAIL_ADDRESS>has an Expensify Account created in a closed state by Concierge on 2023-06-26.
Pasted content into newDot,  edited a few things (including custom subject header) and sent

Expected Result:
Message should send successfully .
Actual Result:
Received this error message
"Unexpected error while posting the comment, please try again later
Auth GetRestrictedAuthTokenByEmail returned an error"
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.33-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @StefStavri
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687775413798129

View all open jobs on GitHub

Hmm, I was able to send a message to closed account and didn't get an error.

Bumped the Slack thread for more eyes.

Ongoing convo: https://expensify.slack.com/archives/C04QEB4MJEQ/p1688146166939669?thread_ts=1687561229.524519&cid=C04QEB4MJEQ

Is this still not fixed? 😓

Bumped this thread - https://expensify.slack.com/archives/C049HHMV9SM/p1688573563663839?thread_ts=1687775413.798129&cid=C049HHMV9SM

@Beamanator any update here or can we close?

Heyey! In Slack I mentioned I couldn't reproduce :D Sounds like you tried too and couldn't reproduce, right? Let's close! :D

Same issue here. Trying to send a DM to a closed account for this onboarding SDR agent.
Is this specific to team.expensify.com emails perhaps @Beamanator ?

Just commenting to remove the overdue since this was just reopened today. I'll leave for @Beamanator to take a look.

Bump @Beamanator - is this bug still happening?

I don't think this is ongoing - @StefStavri could you ask in Slack if you're still seeing this?


UnkownPeer exception thrown before Devices have a chance to talk to eachother

This is more of a convenience request.
If we have two processes:
Replier:
from robocluster import Device
dev = Device('replier', 'test')
@dev.on_request('topic')
def give_thing():
  return 'Hi'
dev.start()
dev.wait()

Requester:
from robocluster improt Device
dev = Device('requester', 'test')
@dev.every('1s')
async def get_thing():
   data = await dev.request('replier', 'topic')
   print(data)
dev.start()
dev.wait()

Running this as is results in an UnknownPeer getting raised.
Catching the exception allows things to run properly.
It would be slick if it could somehow wait for a round of heartbeats (or however this works now) before throwing the exception, to give devices on the network a chance to anounce themselves.

Fixed by #51


detections are not associated into the track

I trained your models on my custom dataset and then ran the track.py with your track.yaml.
The problem is that the detections are good in each frame, but these detections belonging to the same track are not associated, leading to a huge number of tracks.
Could you give me some suggestions on how to set your track parameters in track.yaml?
Appreciate.

Debugging on a custom dataset is beyond the support of this repository. But you should check out the classification scores and thresholds in track.yaml.


docs typos

Thank you for your patch. However, class is an invalid variable name as it is a reserved word.


hotfix. View data with only 1slice + MWF demo + improve ergonomy

Various fix for reported bugs on the recent pull request

@gab-berest if you could just pull this branch (Dev), launch qMRLab and approve if working. Thanks!


[4.0.0] cordova e2e tests

test e2e expectations in terms of cordova project state

create
build
run
emulate
platform

update
add

plugin

add/remove

Closing. This milestone is not being pursued.


Root composer.json requires bref/laravel-bridge ^0.2.3 -> satisfiable by bref/laravel-bridge[0.2.3].

I am using laravel 5.8 and getting this error while composer require .
Using version ^0.5.32 for bref/bref
Using version ^0.2.3 for bref/laravel-bridge
./composer.json has been updated
Running composer update bref/bref bref/laravel-bridge
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - illuminate/queue[v6.0.0, ..., 6.x-dev] require illuminate/console ^6.0 -> found illuminate/console[v6.0.0, ..., 6.x-dev] but it conflicts with another require.
    - illuminate/queue[v7.0.0, ..., 7.x-dev] require illuminate/console ^7.0 -> found illuminate/console[v7.0.0, ..., 7.x-dev] but it conflicts with another require.
    - illuminate/queue[v8.0.0, ..., 8.x-dev] require illuminate/console ^8.0 -> found illuminate/console[v8.0.0, ..., 8.x-dev] but it conflicts with another require.
    - bref/laravel-bridge 0.2.3 requires illuminate/queue ^6.0|^7.0|^8.0 -> satisfiable by illuminate/queue[v6.0.0, ..., 6.x-dev, v7.0.0, ..., 7.x-dev, v8.0.0, ..., 8.x-dev].
    - Root composer.json requires bref/laravel-bridge ^0.2.3 -> satisfiable by bref/laravel-bridge[0.2.3].

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

How to resolve this?

What version of Laravel are you using?

@mnapoli veris

What version of Laravel are you using?

version is 5.8

I think that's the problem, this package requires Laravel 6 to 8: https://github.com/brefphp/laravel-bridge/blob/master/composer.json#L22-L23

yes...thanks..i upgraded to 6 and it worked. @every1 who aracing this issue UPGRADE TO 6 OR ABOVE.


Add underscore prefix to internal submodules

Issue and/or context: #872

You're welcome @bkmartinjr !!


Consider other project name, as git-shell means something else

@Unknwon as I mentioned on Gitter, IMHO git-shell is unlucky name. While I understand your intention that you wanted use pure git shell command but there are few caveats:

This project is not a shell, it is a library (module, API, wrapper), see definition of Shell

There is already git-shell part of official Git project and it is something else (a real but restricted login shell in fact) see https://git-scm.com/docs/git-shell

Therefore consider renaming the project to something like go-git-module, go-git-wrapper or some other name which reflects what is the purpose of this project. This will help community to properly lookup your project and help with extending it.

I think give better README description is good enough for people to clarify what is does.
Thoughts?

Yet this project may be very useful for some others, given it has right name and right description, this is what GitHub searched first.
Giving explanation in README will clarify that to anyone coming there by mistake, looking for shell, but this won't help much finding go module or go wrapper for git.

OK... how about gogs-shell.

Don't get me wrong, because this project is great, but why you want call shell something that isn't a shell according to common definition https://en.wikipedia.org/wiki/Shell_(computing)?
If you don't like git-wrapper, maybe git-cmd then?

I want to call it git-module but based on your argument it is also too vague ..

Maybe it is vague, but at least it is what the name says :)

Cool! So... a lot rename on Gogs main repo..

Luckily GitHub redirects the old name to the new name well, including git clones. https://help.github.com/articles/renaming-a-repository/

But Go import path does not redirect :joy:


Update setupNode to node v20

Change-type: patch

Moving to https://github.com/product-os/flowzone/pull/1111


Create defensive copies of data being backed up

Currently the kiota serialization code is returning
shallow copies of the data in the serialization
writer. This could lead to errors if there's reuse
of the writer prior to the returned data being
completely consumed
This PR works around the issue in the backup
path by returning defensive copies of the data in
the serialization writer. It also adds a set of
tests for Exchange objects to try to catch buffer
reuse errors
This does not address any outstanding issues that
may be in the graph SDK package for posting items
Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[ ] :sunflower: Feature
[x] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Test Plan

[ ] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

Looks good to me!

gonna close this as there's a better temporary fix in and also an upstream PR to fix it in the library causing the issue


sshd listens on the wrong IPv6 address on LAN interface

I have IPv6 on my LAN interface enabled and configured to track the WAN interface. However, sshd on opnsense does not seem to be listening to the LAN IPv6 address. A glances at  /usr/local/etc/inc/plugins.inc.d/openssh.inc confirms this:
# This file was automatically generated by /usr/local/etc/inc/plugins.inc.d/openssh.inc
Port 22
Protocol 2
Compression yes
ClientAliveInterval 30
UseDNS no
X11Forwarding no
PubkeyAuthentication yes
Subsystem sftp internal-sftp
AllowGroups wheel admins
PermitRootLogin yes
ChallengeResponseAuthentication no
PasswordAuthentication no
HostKey /conf/sshd/ssh_host_rsa_key
HostKey /conf/sshd/ssh_host_ecdsa_key
HostKey /conf/sshd/ssh_host_ed25519_key
ListenAddress <IP_ADDRESS>
ListenAddress fe80::1:1

opnsense is telling sshd to listen on fe80::1:1 but that doesn't make sense (you typically don't have layer 7 services listening on a link-local address afaik) and sshd refuses to do it anyway:
Oct  4 21:53:55 firewall sshd[37432]: error: Bind to port 22 on fe80::1:1 failed: Can't assign requested address.

The correct behavior is to listen to the IPv6 LAN address, if there is one, and definitely not any link-local address.
I scanned openssh.inc and might be able to look into this, but I'm not sure yet if I'll have the time/ability to fix it.
Finally, I notice that the opnsense web interface listens on all IPv4 and IPv6 IPs:
tcp6       0      0 *.80                   *.*                    LISTEN
tcp4       0      0 *.80                   *.*                    LISTEN
tcp6       0      0 *.443                  *.*                    LISTEN
tcp4       0      0 *.443                  *.*                    LISTEN

Any reason sshd can't do the same (and controll access via firewall) instead of going through all the trouble to find interfaces to listen to?

This is probably not 18.7.4 ?

On 5. Oct 2018, at 04:01, Charles Ulrich<EMAIL_ADDRESS>wrote:
I have IPv6 on my LAN interface enabled and configured to track the WAN interface. However, sshd on opnsense does not seem to be listening to the LAN IPv6 address. A glances at /usr/local/etc/inc/plugins.inc.d/openssh.inc confirms this:
This file was automatically generated by /usr/local/etc/inc/plugins.inc.d/openssh.inc
Port 22
Protocol 2
Compression yes
ClientAliveInterval 30
UseDNS no
X11Forwarding no
PubkeyAuthentication yes
Subsystem sftp internal-sftp
AllowGroups wheel admins
PermitRootLogin yes
ChallengeResponseAuthentication no
PasswordAuthentication no
HostKey /conf/sshd/ssh_host_rsa_key
HostKey /conf/sshd/ssh_host_ecdsa_key
HostKey /conf/sshd/ssh_host_ed25519_key
ListenAddress <IP_ADDRESS>
ListenAddress fe80::1:1
opnsense is telling sshd to listen on fe80::1:1 but that doesn't make sense (you typically don't have layer 7 services listening on a link-local address afaik) and sshd refuses to do it anyway:
Oct  4 21:53:55 firewall sshd[37432]: error: Bind to port 22 on fe80::1:1 failed: Can't assign requested address.
The correct behavior is to listen to the IPv6 LAN address, if there is one, and definitely not any link-local address.
I scanned openssh.inc and might be able to look into this, but I'm not sure yet if I'll have the time/ability to fix it.
Finally, I notice that the opnsense web interface listens on all IPv4 and IPv6 IPs:
tcp6       0      0 *.80                   .                    LISTEN
tcp4       0      0 *.80                   .                    LISTEN
tcp6       0      0 *.443                  .                    LISTEN
tcp4       0      0 *.443                  .                    LISTEN
Any reason sshd can't do the same (and controll access via firewall) instead of going through all the trouble to find interfaces to listen to?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

PS: you’ve limited SSH access to LAN via gui, the default is listen to all like you said you’d expect. 😊

This is probably not 18.7.4 ?

Err, sorry, this is only 18.7. I should have mentioned that. I see commits in the changelog that might already address this and will upgrade to 18.7.4 to see if that fixes it.

PS: you’ve limited SSH access to LAN via gui, the default is listen to all like you said you’d expect.

You are correct, my mistake.

@cu no worries, thanks for the follow-up! :)


Add tests for AWS transforms

Use moto to mock the services.

Done: https://github.com/Rizhiy/pycs/commit/c59f11cf87ae2a3cb4df9792e2512b63e710629b

moto doesn't support appconfig, decided to use real configuration instead.
Done: https://github.com/Rizhiy/pycs/commit/c59f11cf87ae2a3cb4df9792e2512b63e710629b


Proper handling of Windows service management

Problem
Scpahandre only runs on windows as a bare executable. To use it as a daemon it needs to integrate with windows service management (sc.exe).
Solution
Integrate windows service events management in scaphandre, when compiled for and used on windows.
This crate may help : https://docs.rs/windows-service/latest/windows_service/

Start works fine.

[ ] graceful stop
[ ] graceful restart


Including docs directory.

This change is

Reviewed 152 of 152 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Added possibility to use CDATA sections in XML files

Thank you very much for your contribution, looks good to me, let's wait for @bartoszmajsak review as well and we merge it.

Any chance for getting this in?

Tomorrow morning I ll take a look
El 25 may. 2017 7:49 p. m., "Tobias Stadler"<EMAIL_ADDRESS>escribió:

Any chance for getting this in?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shrinkwrap/descriptors/pull/108#issuecomment-304076142,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcmYZDgKDKZ3SUH9OIl-Xkcnh1iw7Ilks5r9b8ogaJpZM4Lt50e
.

Thanks for your contributions. Would you share some use cases when this is useful? We could use it then in the blog post announcement.

In my case the CDATA-section prevents my IDEs code formatting from screwing up the yams configuration

Thank you for your contribution @tobiasstadler!


[Event Request] codeunit 444 "Purchase-Post Prepayments" - OnBeforeUpdateCrMemoDocNos

Hello, can you please add a new Event OnBeforeUpdateCrMemoDocNos in codeunit 444 "Purchase-Post Prepayments" ?
    local procedure UpdateCrMemoDocNos(var PurchHeader: Record "Purchase Header"; var ModifyHeader: Boolean)
    var
        PurchasesPayablesSetup: Record "Purchases & Payables Setup";
        ErrorContextElement: Codeunit "Error Context Element";
    begin
        OnBeforeUpdateCrMemoDocNos(PurchHeader);
        if GLSetup."Journal Templ. Name Mandatory" then begin
            PurchasesPayablesSetup.Get();
            PurchasesPayablesSetup.TestField("P. Prep. Cr.Memo Template Name");
            GenJournalTemplate.Get(PurchasesPayablesSetup."P. Prep. Cr.Memo Template Name");
            GenJournalTemplate.TestField("Posting No. Series");
            PurchHeader."Prepmt. Cr. Memo No." :=
                NoSeriesMgt.GetNextNo(GenJournalTemplate."Posting No. Series", PurchHeader."Posting Date", true);
            ModifyHeader := true;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeUpdateCrMemoDocNos(var PurchaseHeader: Record "Purchase Header")
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10907.


dynatrace_entity returns old APPLICATION_METHOD that is not subscribed

Describe the bug
We have a module that queries Key User Actions in order to perform further configuration (Apdex and SLOs).
On Web Applications that have had KUAs removed and added back in the past, we observe that the following code:
data "dynatrace_entity" "kua_with_apdex" {
  count           = length(local.kua_with_apdex)
  entity_selector = "type(\"APPLICATION_METHOD\"),entityName(\"${local.kua_with_apdex[count.index].nom}\"),fromRelationships.isApplicationMethodOf(entityId(\"${dynatrace_web_application.application.id}\"))"
}

Will return invalid (old) KUAs. Terraform Apply then returns an error when attempting to define the Apdex for this Method:
Error: User action is not subscribed: APPLICATION_METHOD-098D3F4F2C21FAE6
Expected behavior
Only configuration elements that are still valide should be returned by dynatrace_entity and dynatrace_entities data sources.

The current, valid KUA is like so:
{
  "entityId": "APPLICATION_METHOD-9D331E070782C327",
  "type": "APPLICATION_METHOD",
  "displayName": "loading of page /clic-webapp/clic",
  "firstSeenTms":<PHONE_NUMBER>737,
  "lastSeenTms":<PHONE_NUMBER>687,
  "properties": {
    "detectedName": "loading of page /clic-webapp/clic"
  },
  "tags": [],
  "icon": {
    "primaryIconType": "web"
  },
  "fromRelationships": {
    "isApplicationMethodOf": [
      {
        "id": "APPLICATION-B17D43675FCB4BD2",
        "type": "APPLICATION"
      }
    ],
    "isApplicationMethodOfGroup": [
      {
        "id": "APPLICATION_METHOD_GROUP-D3F738EEAEABC6AD",
        "type": "APPLICATION_METHOD_GROUP"
      }
    ]
  }
}

Querying .settings/objects with schema "builtin:rum.web.key-performance-metric-load-actions" and scope "APPLICATION-B17D43675FCB4BD2" only returns one Method, and not two:
{
  "items": [
    {
      "objectId": "vu9U3hXa3q0AAAABADNidWlsdGluOnJ1bS53ZWIua2V5LXBlcmZvcm1hbmNlLW1ldHJpYy1sb2FkLWFjdGlvbnMAC0FQUExJQ0FUSU9OABBCMTdENDM2NzVGQ0I0QkQyACRhMjhmNDhlMC00MTUyLTMwYmYtYmU0NC1iNzg5N2UwZDlkYmG-71TeFdrerQ",
      "scope": "APPLICATION-B17D43675FCB4BD2",
      "value": {
        "kpm": "VISUALLY_COMPLETE",
        "thresholds": {
          "toleratedThresholdSeconds": 3,
          "frustratingThresholdSeconds": 12
        },
        "fallbackThresholds": {
          "toleratedFallbackThresholdSeconds": 3,
          "frustratingFallbackThresholdSeconds": 12
        }
      }
    }
  ],
  "totalCount": 1,
  "pageSize": 100
}

We reproduced this issue with version 1.42.0.

Hello @mattBaumBeneva,
I retraced the steps you've described. The only thing where my results differ from yours is, when querying for the already unsubscribed key user action via entities API. The Web Application was still listed within the fromRelationships. I can only assume, that relationships between entities are subject of "aging out over time".
But in any case - the reason why the KUA is showing up as the result of the data source is, that the provider specifies now-3y in order to not miss out on any entities. That behavior clashes with the fact, that Dynatrace doesn't simply delete KUAs that aren't assigned to a Web Application (metrics may have been produced for them already).
The only way to deal with this issue on short notice is to offer the attributes from and to for the data sources dynatrace_entity and dynatrace_entities. That at least gives you some sort of control over what is still valid and what not. The upcoming release will contain the additional attributes.
But on our todo list is also a better solution. Currently Key User Actions are getting configured WITHIN the resource for Web Applications - although configuration happens via a separate endpoint. My plan is to deprecate that block within dynatrace_web_application and instead introduce a resource dynatrace_key_user_action. ... and the next logical step would be to introduce a data source dynatrace_key_user_action that doesn't need to use the Monitored Entities API, but uses the same REST endpoint than the resource with the same name.

OK. Thanks!


error: add another case for isLimitExceeded

Some registrars (e.g. ".lu") will return a Maximum query rate reached message, which is currently not covered.

Hello @mfycheng
Thanks for your PR.


When installing Microsoft c/c++ and gnu global at the same time, gnu global does not work

ubuntu 16.04
gnu global 6.6.4
vscode 1.47.2
When installing Microsoft c/c++ and gnu global at the same time, opening a large project and using the jump function and find references, vscode seems to give priority to the analysis results of c/c++, so gnu global will not work.But I want to use some of the functions of the two plug-ins, so I am sad

ubuntu 16.04
gnu global 6.6.4
vscode 1.47.2

I meet same issue, when default C/C++ plugin add,  it don't work. Looks like some conclict.
How to solve it?

i don't solve it so far


[Feature Request]:  Enhanced scroll back button with timeline on it for smoother functioning.

Is there an existing feature request for this?

[X] I have searched the existing feature requests

Describe the feature you would like to request
Enhanced scroll back button with timeline on it for smoother functioning.

What problem does this feature solve?
Scroll back button
Alternative solutions
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[ ] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository
[X] I would like to help implement this feature

Hyy @shriyadindi, I'm assigning this issue to you.


feat(chart): add dots for LineChart #168

Close #168

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


port toranj test007-traffic-router-sleepy.py

Codecov Report

Merging #66 into master will decrease coverage by 1.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   41.72%   40.24%   -1.49%
==========================================
  Files          37       37
  Lines        3851     3735     -116
==========================================
- Hits         1607     1503     -104
+ Misses       2244     2232      -12

Impacted Files
Coverage Δ

silk/node/wpan_node.py
55.14% <0.00%> (-10.58%)
:arrow_down:

silk/utils/jsonfile.py
30.55% <0.00%> (-9.93%)
:arrow_down:

silk/tools/wpan_table_parser.py
33.88% <0.00%> (-6.86%)
:arrow_down:

silk/node/base_node.py
46.21% <0.00%> (-6.46%)
:arrow_down:

silk/utils/signal.py
66.19% <0.00%> (-4.18%)
:arrow_down:

silk/utils/process.py
47.82% <0.00%> (-1.12%)
:arrow_down:

silk/tools/otns_manager.py
79.87% <0.00%> (-1.06%)
:arrow_down:

silk/tools/watchable.py
38.94% <0.00%> (-0.64%)
:arrow_down:

silk/node/fifteen_four_dev_board.py
29.19% <0.00%> (-0.21%)
:arrow_down:

silk/tests/silk_replay.py
87.34% <0.00%> (-0.08%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


View helpers

A pattern I've been doing a lot is making helpers to easily render RemoteData values. At the core it's an interface that invoke rendering methods based on the current state. This is often wrapped in a ViewAnimator that can easily switch between initial, loading, success, and error views.
Making this an optional module can be helpful.

I think at this point it makes sense to be Compose first.


Static generation of swagger.json

Hi, I'd like to get the swagger.json statically generated, so it can be analyzed without the server running. Is this documented or supported in some way? Sorry if I have missed it.
Should I just generate it by a custom script that calls the endpoint and writes it to a file?

@naftis that is they way. Starting a hapi server is super lightweight and should be pretty quick.


az webapp config appsettings list should support the optional parameter --ids

Related command
az webapp config appsettings delete
az webapp config appsettings set

Is your feature request related to a problem? Please describe.
The delete and set commands both support the optional Parameters --ids. When I try it with list I get this error:
ERROR: the following arguments are required: --resource-group/-g, --name/-n

Describe the solution you'd like
I would like to be able to run a command like this:
az webapp config appsettings list --ids "/subscriptions/****/resourceGroups/foobar-rg-01/providers/Microsoft.Web/sites/foobarweb"

instead of only being able to do this:
az webapp config appsettings list --name foobarweb --resource-group foobar-rg-01

Describe alternatives you've considered

Additional context
Similar feature request: https://github.com/Azure/azure-cli/issues/22527

route to CXP team

I support this feature request, as it would bring consistancy to the command structure


Update hack folder with update_deps files

updating hack folder with update_deps files.

/assign @julz

You probably need to run clean licenses script as well.

You probably need to run clean licenses script as well.

Is that a script that already exists?  If not, what are you thinking it should do?

They do in others (probably in upstream hack as well).

They do in others (probably in upstream hack as well).

It looks like update_deps calls update_licenses:
https://github.com/knative/test-infra/blob/master/scripts/library.sh#L569
Were you looking for something else?

yeah, that should remove them... but they are present in your pr?

@vagababov
Looking at what update_licenses does...
It makes a call to this go-licenses tool:
    run_go_tool github.com/google/go-licenses go-licenses save "${dir}" --save_path="${dst}" --force || \     { echo "--- FAIL: go-licenses failed to update licenses"; return 1; }
iiuc, the go-licenses tool will loop through the required libs & create subdirectories for each of the required license files:
https://github.com/google/go-licenses/blob/master/save.go#L89
What projects have the licenses files removed, so I can take a peek?
/serving looks like it has all the licenses in these sub directories as well:
https://github.com/knative/serving/tree/master/third_party/VENDOR-LICENSE
Same for a couple of the other knative-sandbox projects:
https://github.com/knative-sandbox/net-contour/tree/master/third_party/VENDOR-LICENSE

cc @chizhg @n3wscott
I am not sure :)

Til about this repo!

/hold just if you want to check something

/hold should unhold?

/unhold


Backport x/sys upgrade to 0.16 release to support go 1.17

Backport just x/sys upgrade from c99d6e1c9587d683c8a262925255d8609a9843d8 to v0.16-release branch so that next 1.16.* release will support go 1.17.
More context:

blocking https://github.com/Homebrew/homebrew-core/pull/83413
Go issue https://github.com/golang/go/issues/46763
elvish issue https://github.com/elves/elvish/issues/1389
workaround https://github.com/macports/macports-ports/pull/11983
workaround https://github.com/Homebrew/homebrew-core/pull/83995

After merging it please:

make / tag a new release (eg v0.16.2 on v0.16-release branch)

Simpler alternative to #1388
Fixes #1389
Altrenatively you can make a new release branch (eg v0.17-release) from the master branch somewhere after https://github.com/elves/elvish/commit/c99d6e1c9587d683c8a262925255d8609a9843d8 and tag a new release (eg v0.17.0) there, but that might include other, unrelated changes.

Thanks, @xiaq !

Hmm, this broke the building of the website because the x/sys dependency was not in website/go.mod.
I'll fix that and bump to 0.16.3...

Tnx! I was testing using Makefile and all tests passed as in CI. With initial PR #1388 on master i updated both go.mod files 😇


4.2 - Use PHP 7.3 for analysis on Travis

This PR is a follow-up of #2813.
We use PHP 7.3 to do the analysis.

Coverage remained the same at 100.0% when pulling e070914cbe8e1a703766dd66e4b7c99eff43ff26 on adriansuter:patch-travis-analysis into 6a7dbbad585061365a777d9f2d0a16cc9d4b6907 on slimphp:4.x.


Add recursion support to /1.0/images/aliases #473

Same pattern used as the other recursion commits.
I also added a little refactoring:

extracted test for "recursion" query parameter to its own function ( isRecursionRequest() ) in daemon.go
doAliasGet() returns shared.ImageAlias{} instead of shared.Jmap{....}
aliasGet() returns SmartError instead of NotFound or InternalError

Closes #473
Signed-off-by: Matt Morrison<EMAIL_ADDRESS>

Server change looks good. Can you also make the change to the client code to use recursion=1?

Sure - which client command needs to be updated?
lxc image list already uses /images?recursion=1 to get both images and aliases recursively.

I can at least think of "lxc image alias list"

Currently lxc image alias list is a simple list of the aliases, like:
$ lxc image alias list
oracle_6_5
centos
debian
ubuntu

I've updated the client to pull the aliases with recursion=1, and now output a table like:
$ lxc image alias list
+------------+--------------+------------------------------------------------------------------+
|   ALIAS    |    IMAGE     |                        IMAGE FINGERPRINT                         |
+------------+--------------+------------------------------------------------------------------+
| centos     | ac243c26e5aa | ac243c26e5aa86796c747a4df0cf25744d568b4c098a47953efa264195188eac |
| debian     | f3f2bbd0054e | f3f2bbd0054ec4a727f22d6381a00ac5b7ffd6b440a632185637521b64731a76 |
| oracle_6_5 | 4935b6d422c3 | 4935b6d422c316444ba32b1a05191981bcaa5b3ba024f2f6575e1246ae32bbf7 |
| ubuntu     | 287d23d0650f | 287d23d0650f5cd572dff54390e8209f2e53466394c56ff36342881e29813ce2 |
+------------+--------------+------------------------------------------------------------------+

Is this what you had in mind @stgraber ?

@sl1pm4t that looks really nice.
For consistency, I'd just do the following changes:

Rename IMAGE to FINGERPRINT
Drop IMAGE FINGERPRINT (since it's just the long fingerprint and isn't typically displayed unless you do lxc image info)

I commited my client changes, and this PR was complaining of merge conflicts (on lxd.pot), so I rebased and force pushed. I hope I did it correctly.

Also, command output now looks like:
$ lxc image alias list
+------------+--------------+
|   ALIAS    | FINGERPRINT  |
+------------+--------------+
| centos     | ac243c26e5aa |
| debian     | f3f2bbd0054e |
| oracle_6_5 | 4935b6d422c3 |
| ubuntu     | 287d23d0650f |
+------------+--------------+

Hmm, looks like your change caused a test regression. Looks like you need to update the tests in tests/ to work with the new rendering of lxc image alias list.

Ok. Looking

Fixes for the tests has been committed.

Great, thanks!


Refuse to set unsafe header User-Agent

Hello everyone,
Issue Summary
I get an error since I upgraded to the last 1.9.4 version, in Chrome browser.
In the xhr.js I get an exception due to the commit https://github.com/TryGhost/SDK/commit/ae39516130bfa620d03299c1ab869e6cd97c6c99
I guess a fix would be to set this header only in NodeJs context.

Technical details:

Ghost Version: 1.9.4
Node Version: ?
Browser/OS: Chrome 101
Database: ?

Same here.

This one is slightly tricky. Firefox has dropped the User-Agent header from the forbidden headers list. Chrome on the other hand still drops it (source).
I think the best default behavior here would be following:

Do not set User-Agent by default in browser environment
Set it to default if userAgent: true parameter is passed
Set it to whatever value is passed in userAgent parameter if it's not boolean

This way the defaults will be slightly smarter and if anyone really need it in the browser environment like Firefox can still set it.

The above behavior would also play nicely with https://github.com/TryGhost/Toolbox/issues/301 allowing to set custom User-Agent headers


Girlfriend In front (LUA)

Describe your problem here.
Is it possible to place the girfriend in front of the boyfriend and the opponent in LUA plz ? (not in front of the HUD) thanks
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

you shoud add her like a bg image and then in the add lua sprite you need to change from false to true addLuaSprite('girl', true);

I tried it, but the danceLeft and danceRight thing can't be done

setObjectOrder() and getObjectOrder() (wiki)

danceLeft and danceRight thing can't be done

That's literally just using a variable

OR simply use setObjectCamera(‘girlfriend’, ‘camHUD’);

OR simply use setObjectCamera(‘girlfriend’, ‘camHUD’);
that's awful

setObjectOrder('girlfriendGroup', getObjectOrder('boyfriendGroup') + 1);

:(

OR simply use setObjectCamera(‘girlfriend’, ‘camHUD’);
that's awful

setObjectOrder('gfGroup', getObjectOrder('boyfriendGroup') + 1);

which function and where exactly (data/my song) ?


[tree widget] rename euronuts3

do we need to update the examples too? #1053

The examples come from the statVarSummary cache so if we need to update those, we would need to do it in the cache


Remove unncessary deps from RunTestsTask classpath

Purpose
Filter out all classpaths except for rt and runtime
Fixes #34348
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Tested Mocking and Code coverage general cases.

Tested against the following modules :

HTTP
GraphQL
GRPC


[BUG] When disabling revision counter, salvaging a faulty volume not work as expected

Describe the bug
The code is not right. This is a day 1 issue instead of a regression.
/pkg/controller/control.go#L552-L560
		repLastModifyTime, err := c.backend.backends[r.Address].backend.GetLastModifyTime()
		if err != nil {
			return err
		}

		repHeadFileSize, err := c.backend.backends[r.Address].backend.GetLastModifyTime() // this is incorrect
		if err != nil {
			return err
		}

To Reproduce

Expected behavior

Support bundle for troubleshooting

Environment

Longhorn version:
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of control plane nodes in the cluster:
Number of worker nodes in the cluster:

Node config

OS type and version:
Kernel version:
CPU per node:
Memory per node:
Disk type (e.g. SSD/NVMe/HDD):
Network bandwidth between the nodes (Gbps):

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context
cc @longhorn/qa

Clear copy/paste error.
But it also seems that it doesn't make sense to punt on one unhappy replica.  Skip it and go on.  If there are none that can even report a last modify time or head file size, we will fail the test for "there are no replica candidates."

I'll build and put up a PR for the first point for possible fast-tracking, and we can talk about the second one.

@james-munson have you tested the fix?

Not directly.  Just built and run unit tests.  No regressions or manual tests yet.  I will start a regression run with "-k salvage" to begin with.

Salvage tests passed as well.

Verified on master-head 20240123

longhorn v1.6.0-rc2 https://github.com/longhorn/longhorn/commit/7a40621dee1408bd90cf9bf92e41ed626f3d0221

The test steps
Referencing the documentation node-disconnection and the enhancement related to revision counter disable support PR: 20200821-add-revision-counter-disable-support:

Enable auto-salvage + DisableRevisionCounter.

Disable the Revision Counter via the Longhorn UI.
Deploy a new storage class: longhorn-test with disableRevisionCounter set to true in the parameters.

parameters:
  numberOfReplicas: "3"
  disableRevisionCounter: "true"
  staleReplicaTimeout: "2880"
  fromBackup: ""
  fsType: "ext4"

Deploy a PVC and POD using the pod_with_pvc.yaml example, replacing the storageClassName value with longhorn-test.

Create a StatefulSet using a new storage class named longhorn-test and write data.

Execute test case 3 under specific conditions.

Result Passed

After enabling auto-salvage + DisableRevisionCounter, create, attach, and detach other volumes to the recovered node. All volumes should function correctly.


Issue #871 sales revenue

Fixes # .
Changes proposed in this PR:

Not sure if we are going to do this, probably going to close

will probably not do this any time soon.

@mihaisc Are we also closing the related task on the subgraph that was needed for this? https://github.com/oceanprotocol/ocean-subgraph/issues/582


Add a watch face and a widget for NodeConf Remote

The bounds calculation still seems to be wrong, see the red dots in this picture

It's the second hand? I guess it could be a rounding issue as everything's a bit smaller than the original analog clock - the quick hack is just to increase the hand size for the bounding box:
addPt(r,handSizeSec+2,5);addPt(r,handSizeSec+2,-5);

Do you want me to merge this as-is, or do you want that fixed first?
Also, don't know if you're interested but you could have a smaller background image behind the clock, and could then render the NodeConf Remote text as a full-res 2 bit bitmap (which would get you nice greyscale).

Ready to merge?

Yes, ready, I rebased the commit.

Great - thanks!


Upstream refresh

Resolves #127
What is being addressed
This merge request pulls in the latest changes from the upstream branch as of about a week ago.
How is this addressed
There are a lot of changes here, I'm not expecting you to review them all. I've tested them, and Andrii has tested them.
The only known problem is with the Nexus machine hanging, but that happens in the upstream, it's not just us. I've tried enlarging the Nexus VM disk by hand, and after a bit of flakiness, it's now been working without problem for a few days. Not sure if that's the full resolution or not, TBD.
I've tested the airlock, that functions correctly. I've tested RStudio, Jupyter, Azure Storage Explorer, all on Linux, and that works.
Please take a (quick) look, and approve so we can get the merge.

I will take a look at this today and communicate my findings if any.