Fix / Check grant reward UID

Description
Check reward uid tobe a valid uuid string

Type of change
Please check the options that are relevant. This PR introduces

[ ] Patch: Bug fix (non-breaking change which fixes an existing issue)
[ ] Minor: New feature (non-breaking change which adds new functionality)
[ ] Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Features Description

[ ] NA

Test Cases

[ ] NA

Documentation

Checklist:

[ ] I have added change type (major|minor|patch) in the PR
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Codecov Report
Attention: 17 lines in your changes are missing coverage. Please review.

Comparison is base (85f96c0) 11.83% compared to head (5e7e6ec) 11.83%.
Report is 17 commits behind head on master.

Files
Patch %
Lines

x/subaccount/types/query.pb.go
0.00%
10 Missing :warning:

x/subaccount/keeper/query_params.go
0.00%
4 Missing :warning:

x/reward/types/campaign.go
0.00%
2 Missing :warning:

x/reward/keeper/msg_server_reward.go
0.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #332   +/-   ##
=======================================
  Coverage   11.83%   11.83%
=======================================
  Files         346      347    +1
  Lines       44417    44423    +6
=======================================
+ Hits         5257     5259    +2
- Misses      38679    38682    +3
- Partials      481      482    +1

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


[8.x] Add cache-busting for public assets

This PR allows the use of cache-busting on public assets,
Using:
asset('filePath', version: true)

generates a URL for the asset and appends an id with this format:
https://your-domain.com/filePath?id=1639002822
the value of the id is a UNIX timestamp of the last time the file was modified.
So using this feature we can make sure the browser always requests the new file whenever the file is changed, even if the file was cached on the browser. also, it eliminates the annoying need for updating file names when they change to make sure clients get the newest file every time the file is changed.
This PR also doesn't break the normal usage of the asset helper.
So the function:
asset('filePath')

generates https://your-domain.com/filePath as expected
These changes don't introduce any security vulnerability as the files are already public and the only exposed info is the last modified time of the file.

I don't think this will play nice across multi-server deployments. It is more robust to use ETAG caching (default in cloudfront) or to version the URIs at compile time (mix supports this, easily), instead of using a query string param.

@GrahamCampbell @mhmdomer thanks for taking the time to review this.
I think I will just add a new helper function for this feature to my collection of helpers 😅

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Fix: #1988 fix clone

1. Why is this change necessary?
Because cloned entites, clones also the created at timestamp which is not true
2. What does this change do, exactly?
if you clone an entity via API, the create_at date getting overwritten, with the current date time instead of original one. so you know which clone is the new one
3. Describe each step to reproduce the issue or behaviour.
If you clone an entity, for example email template from administration, the created_at date getting cloned as well
4. Please link to the relevant issues (if any).
https://github.com/shopware/platform/issues/1988
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

It would also make sense to set the updatedAt column to null, wouldn't it? Just for a clean db entry.

@PheysX yes i had the same thoughts two minutes later :D


Fix wrong ruby negative int array encoded value

Issue
#27
Changes
In ruby, only treat int array as string if it's ord is > 0.

Thank you for the patch!


Detections for nuScenes mini dataset

Thanks for sharing your code!
I'm wondering if the CenterPoint and MMDetection detections are provided for the v1.0-mini version of the nuScenes dataset. As mentioned in the README, I changed SPLIT to mini-train / mini-val, but the CenterPoint and MMDetection folders from the google drive link contain detections for the (full) validation and test sets only. Am I missing something here? Thanks in advance!

Hi!

On NuScenes, if I remember correctly, the mini-train/mini-val are just subsets of the train/val splits, so the detections are exactly the same. To avoid duplicating data on disk, I simply renamed my val/test folders to mini-val/mini-test when running early experiments and then renamed them back when running experiments on full data. Alternatively, you can also just duplicate files.

The detections I extracted myself (not given by other implementations) are not provided for the training set - I never had them because the tracking framework just took detections as-is, without trying to train detector models.
If you need training set detections, you might try to find them in other implementations. Alternatively, the framework should work well with any detections of comparable quality, so maybe try something you already have?

For CenterPoint, look at the original model zoo, where I got my detections for the centerpoint_voxel_1440_dcn(flip) configuration. Maybe some other model config has training results.
For MMDetection, I used their model with this config from the model zoo:
config_file = 'configs/nuimages/cascade_mask_rcnn_x101_32x4d_fpn_1x_nuim.py'
checkpoint_file = '../checkpoints/cascade_mask_rcnn_x101_32x4d_fpn_1x_nuim_20200906_134611-47db31b0.pth'
model_name = "cascade_mask_rcnn_x101"

Hope this helps!

May I ask about where is configs/nuimages/cascade_mask_rcnn_x101_32x4d_fpn_1x_nuim.py？

Hi, it looks like either the link has changed or I linked to the wrong model zoo.
The link for 2D detection models from MMDetection is this https://github.com/open-mmlab/mmdetection3d/tree/master/configs/nuimages.
There, you will find the configs and the models available. The config I specified in the previous reply is correct (for Cascade Mask R-CNN | X-101_32x4d), but looks like the checkpoint and performance has improved. If you would like to use the exact checkpoint I have used, you will find it in the commit from September 8.

ok，Thanks a lot!

Hello, one quick question on the 2d detections results.
I have v1.0-mini version of the nuScenes dataset and centerpoint detections for the centerpoint_voxel_1440_dcn(flip) from the link you provided. To get the 2d detection can I run the Cascade Mask R-CNN | X-101_32x4d  model on the nuscenes data to get the 2d detections and plug these 3d and 2d to the tracking? Please correct me if Im wrong here.

@tiffanyyk  Can you please help me in understanding how to relate the detections downloaded from the drive to the images in the nuscenes mini version. Appreciate your help.


docs: update link to jscpd documentation

Readiness checklist

The jscpd documentation seems to have moved; I updated the URL to point to the correct location.
Pull request author tasks

[x] I checked that all workflows return a success.
[x] I included all the needed documentation for this change.
[x] I provided the necessary tests.
[x] I squashed all the commits into a single commit.
[x] I followed the Conventional Commit v1.0.0 spec.
[x] I wrote the necessary upgrade instructions in the upgrade guide.
[x] If this pull request is about and existing issue,
I added the Fix #ISSUE_NUMBER or Close #ISSUE_NUMBER text to the description of the pull request.

Super-linter maintainer tasks

[x] Label as breaking if this change breaks compatibility with the previous released version.
[x] Label as either: automation, bug, documentation, enhancement, infrastructure.
[x] Add the pull request to a milestone, eventually creating one, that matches
with the version that release-please proposes in the preview-release-notes CI job.

Thanks for taking the time to send this PR!


More stream services

There are some services thar I always ad on my settings when I install ElectronPlayer on a new computer that I think can be added as default. There is a list:
-Apple TV+
-HBO GO
-Telecine Play
-ESPN
-FOX
Thanks for the aplication!

I will add them to v3.0.0 whenever I get time to finish it.


gap: 4.11.1 -> 4.12.0

Description of changes
This is unpolished, a bit hacky and definitely not ready for review, but I want to see if it helps with our aarch64 deadlocks.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build sage

@ofborg build sage

The deadlocks seem to have been replaced by crashes! This is very good news. I'll investigate in two weeks, hopefully https://github.com/gap-system/gap/issues/5063 (which also affects x86_64) will be addressed by then too.

I'd be interested to know what these crashes are, is there a way to see them? -- people are successfully running GAP 4.12 on aarch64 as far as I know.

@ChrisJefferson The crashes happen during Sage doctesting. The build/test log is at https://logs.nix.ci/logfile/nixos/nixpkgs.192548/871aa396-d85f-4d6b-8737-0d2576d23fbe, and you can grep for capture_stdout to see the backtraces. Note that this is with https://trac.sagemath.org/ticket/34391 applied, which changed how CloseOutput is called.

There are other crashes in the same log, too (e.g. grepping for _cstr_to_str), perhaps they are related.

With GAP 4.11.1, we were seeing deadlocks during Sage testing instead, and I'm hoping they have a common cause. They're described in more detail at https://trac.sagemath.org/ticket/34297.

Note that the very last error/stack trace in the log is unrelated to GAP (it is tracked at https://trac.sagemath.org/ticket/34575).

@ChrisJefferson I changed the Sage function slightly to avoid a possible GC problem on the Sage side. The capture_stdout function on the Sage side is now this:
    try:
        GAP_Enter()
        s = NEW_STRING(0)
        output_text_string = GAP_ValueGlobalVariable("OutputTextString")
        stream = CALL_2ARGS(output_text_string, s, GAP_True)

        if not OpenOutputStream(&output, stream):
            raise GAPError("failed to open output capture stream for "
                           "representing GAP object")

        CALL_1ARGS(func, obj)
        CloseOutput(&output)
        return char_to_str(CSTR_STRING(s))
    finally:
        GAP_Leave()

Unfortunately, there are still some segfaults with backtraces which look like the following:
#0  0x0000fffff79740e8 in wait4 ()
#1  0x0000fffff5dc6b78 in print_enhanced_backtrace ()
#2  0x0000fffff5dc8190 in sigdie ()
#3  0x0000fffff5dcb1c0 in cysigs_signal_handler ()
#4  0x0000fffff7ffb7cc in __kernel_rt_sigreturn ()
#5  0x0000ffff99a0bf28 in ConvString ()
#6  0x0000000000000000 in ?? ()
#7  0x0000000000000000 in ?? ()
#8  0x0000000000000000 in ?? ()
#9  0x0000ffff99989930 in Pr ()
#10 0x0000ffff9998aa18 in CloseOutput ()
#11 0x0000ffff99884828 in capture_stdout () at /build/sage-src-9.7/pkgs/sagemath-standard/sage/libs/gap/element.pyx:154

For convenience, the Pr call, which is supposed to flush the buffer before closing, is made at https://github.com/gap-system/gap/blob/v4.12.0/src/io.c#L938, I think the ConvString call is made at https://github.com/gap-system/gap/blob/v4.12.0/src/io.c#L1126, although there are missing frames, and ConvString itself is at https://github.com/gap-system/gap/blob/v4.12.0/src/stringobj.c#L1249.
One funny thing I see over and over with our aarch64 builders is that they seem to hit a lot more GC-related problems then the x86_64 ones, even when the bug is not aarch64-specific. Is ConvString GC-unsafe somehow, is CloseOutput being called wrongly, or is the problem elsewhere?

I should also point out that there are only two callsites for this function
cdef str gap_element_repr(Obj obj):
    cdef Obj func = GAP_ValueGlobalVariable("ViewObj")
    return capture_stdout(func, obj)

cdef str gap_element_str(Obj obj):
    cdef Obj func = GAP_ValueGlobalVariable("Print")
    return capture_stdout(func, obj)

I wonder if calling GAP_ValueGlobalVariable without doing the GAP_Enter/GAP_Leave dance can cause GC problems.

I just saw the discussion on AArch64 crashes above. Some remarks:

"Is ConvString GC-unsafe somehow"  -- depends on what you mean? It can allocate which can trigger a GC, which can cause problems; but that holds for many GAP kernel functions. It never should raise an error and is pretty safe in this regard at least
"is CloseOutput being called wrongly" -- it looks right to me. but a warning: if the CALL_1ARGS should raise a GAP error, then you'll miss the CloseOutput invocation, which could cause problems
"I wonder if calling GAP_ValueGlobalVariable without doing the GAP_Enter/GAP_Leave dance can cause GC problems" -- it is no worse than ConvString (actually, it is likely to cause an allocation -- indeed only if the string passed to it has not yet been used as name of global variable. But you call it on "OutputTextString" so it won't ever trigger a GC.

However, what could in theory happen is that GAP starts a garbage collection, but does not "see" your references to objects. You have these:
    cdef Obj s, stream, output_text_string
    cdef UInt res
    cdef TypOutputFile output

But I am not sure if the GC "sees" all of them this way in your code... nominally GAP_Enter exists to ensure that. Hence I am surprised it works "better" without it ?!?. I know very little about Cython, though, so I can't judge at all how this happens. Probably this is something to discuss with @embray ?

Thanks for the aarch64 comments! I should have written another comment to clarify the current status instead of just an edit. Currently sage tests pass on aarch64, but I needed to patch Sage like so: https://github.com/NixOS/nixpkgs/blob/3be405a031dcc259de0d53ff2adbb50c6247f3a6/pkgs/applications/science/math/sage/patches/copy-string-before-gap-leave.patch. I was a bit busy the last few days but I will upstream this patch to Sage soon. In particular, this changes two things:

Sage was previously getting the C-style string backing GAP's text stream (which held the output of ViewObj or Print) and holding onto it past the GAP_Leave call. I fixed that by copying it to a Python string inside a GAP_Enter/GAP_Leave scope, otherwise I saw crashes.
The above was not sufficient to fix all crashes. I also needed to move the GAP_ValueGlobalVariable("ViewObj") and GAP_ValueGlobalVariable("Print") calls inside the GAP_Enter/GAP_Leave block, otherwise I'd also see crashes.

I think I expressed myself badly before, but to clarify: You're absolutely right that narrowing the GAP_Enter/GAP_Leave blocks makes things worse. I had to make sure relevant things were "protected" by those blocks to avoid crashes.

FYI, I just released GAP 4.12.1

Thank you! Sorry, I was busy the last week. Will push the changes now.

I've dropped the extra aarch64 patch needed for Sage tests to pass, since they were already broken before the GAP update. They will be part of a new PR (which will depend on this one) as soon as I submit the patch to upstream Sage. The update should now be in a good state thanks to invaluable help from @fingolfin, so I'll put this up for review from the other maintainers.


[FEATURE]: Add input state "wifi" for given apps as smarthings api doesn't report it correctly

I'm using sound bar Q990B, when I do AirPlay, the ha device would have empty input source showing, while other input sources like bluetooth or digital worked well. Here's some screenshot in HA:

entity state history:

should be a same issue as https://github.com/PiotrMachowski/Home-Assistant-custom-components-SmartThings-Soundbar/issues/52

Hi @bishengliu7
I know of this problem, and its the same thing with my soundbar (Q935). The thing is that the Smartthings API (for some reason I don't understand) is not returning wifi when using AirPlay or Spotify but "".
Thinking about it, I could make an extra step and still return wifi if the app is spotify or airplay.
The proposed changed would be something like this in the SoundbarDevice.py
    @property
    def input_source(self):
        if self.media_app_name() in ("airplay", "spotify"):  # added
            return "wifi"                                                             # added
        return self.device.status.input_source

This is a bit "hacky", but it could / should work.
I will try it out as soon as possible.

Hi @bishengliu7,
I released a new beta version, where I fixed this issue / feature. You can try it out and tell me if it works for you.
https://github.com/samuelspagl/ha_samsung_soundbar/releases/tag/0.3.0b2
For more information on how to install a "pre-release" please read the official HACS documentation.
Hope that helps.

Hi @bishengliu7,
I released a new beta version, where I fixed this issue / feature. You can try it out and tell me if it works for you.
https://github.com/samuelspagl/ha_samsung_soundbar/releases/tag/0.3.0b2
For more information on how to install a "pre-release" please read the official HACS documentation.
Hope that helps.

SGTM, i'll try that today. Thanks for the fast response!

seems not working, the value of input preset is still unknown

Can you try one thing for me please.
Do you have the SmartThings App on your mobile Device?
Could you open it, tap on your Soundbar (given that you stream music with AirPlay or Spotify) and take a look if something changes?
Interestingly I saw on my end that, the media_app, in my case AirPlay or Spotify sometimes is only being reported when the SmartThings App was opened.
Can you show me the complete state of the media_player entity?
Thanks a lot!

when soundbar is off:

when airplay

But the source now is listed correctly as wifi in the third image.

So this is again a case of "What the heck is the SmartThings API doing?" for me 😄. Until now I thought that it was maybe just my device, but as you can confirm the same behavior, it's official 😅.
So honestly I don't know how to make a change / fix this entirely. I know that there are API commands that control / fetch info for every aspect of the soundbar, but as they are not listed anywhere its kinda hard to guess.

yeah...seems the media type is not updated as soon as input source changed

I'm wondering if it is possible to add a default input source? so we can some how hack it in the HA automation, like treat the default input source as airplay. Currently it is unknown seems not able to be used as a condition in automation

Let me think about that. So I'm not sure about the naming, but I think that I would be fine to add the "unknown" state to the souce_list and specifically set it as a value if nothing is returned by the SmartThings API.


TypeError: Cannot read properties of undefined (reading 'header')

I currently getting this error on vue 3. This is my code

And this is the error show on my browser

Getting this too. And if you add the #header slot, you bump into other errors. It's because this.$scopedSlots is undefined, but I have no idea why.

Getting the same error, but I'm not willing to install Vuetify :)
Did you find another solution?

package version:
"vue": "^3.4.21", "vuedraggable": "^4.0.1"
code:
<draggable :list="myArray" item-key="name" ghost-class="ghost" @start="drag = true" @end="drag = false">
        <template #item="{ element }">

{{ element.name }}

        </template>
</draggable>

换个 "vuedraggable": "^4.0.1" 这个版本我这边正常, 没有出现这个错误了

this still not work with  "vuedraggable": "^4.0.1",

Updating Vue to latest will probably fix this, as it fixed mine.

I fixed it by turning shutting down dev server and then running npm run dev again. "vue": "^3.4.27", "vuedraggable": "^4.1.0"

I fixed it by shutting down dev server and then running npm run dev again. "vue": "^3.4.27", "vuedraggable": "^4.1.0"

Same here!

Just shut down the server and then rerun. If still not work, install to the  "vuedraggable": "^4.1.0", then rerun the server again

package version:
"vue": "^3.4.21", "vuedraggable": "^4.1.0"
solution:
delete node_modules and clean npm
it works for me!


add tmplr preview

templating recipes do irreversible stuff, for example removing the templating recipe file itself. this makes creating and testing templates a bit difficult. this short-coming could be solved with a preview command, where the contents of the current folder are copied into a temporary folder, where the templating recipe is executed.
e.g:
tmplr preview

could copy the contents of current folder into a .tmplr-preview folder, and run the templating recipe there.

I think it would be wise to reserve the test command for something similar but for automated testing a templating recipe.

Maybe a tmplr preview --dry-run could just output the steps that would be taken to the terminal.

This feature is added in 0.2.9. I didn't implement a --dry-run option, as the same could be achieved via running tmplr preview && tmplr clean.


Contrast on Mermaid Diagrams is too low

Currently some Diagrams using Mermaid have very low contrast, making reading them near impossible. An example of this is the mindmap diagram.

Thanks, I've never used mindmaps in Obsidian, so was unaware of this. I think I have solved this problem in the latest update, but, in searching, I did come across a post from the Obsidian forums indicating this is something that affects the default theme as well: https://forum.obsidian.md/t/text-colors-for-mermaid-mindmap-graph-are-unreadable-in-dark-mode/68674
Because I am able to replicate this issue in the default theme (and Minimal), I am a little concerned there might be a "reason" it doesn't work as expected there, but, if so, I am sure someone will let me know at some point. :)
I have made a few changes to how nodes are shown in the mindmap to standardize things a bit. Now it should use Catppuccin colors up to the eighth node in a mindmap (we could increase that number programmatically in sass if people regularly go beyond eight). I have changed the underline color to the node color for simplicity's sake, but if this creates an issue I am not aware of, or if something breaks with this update, please reopen and let me know!


DecorationImage removes shadow from a container

With image:

Without image:

child: Container(
                          width: 425.0 * AppScale.amount,
                          height: 225.0 * AppScale.amount,
                          decoration: BoxDecoration(
                            borderRadius: BorderRadius.all(
                              Radius.circular(Corners.med),
                            ),
                            color: theme.kTitleBarColor,
                            boxShadow: [
                              BoxShadow(
                                color: Colors.black.withOpacity(0.09),
                                spreadRadius: 0.0,
                                blurRadius: 8.0 * AppScale.amount,
                                offset: Offset(
                                  0.0 * AppScale.amount,
                                  2.0 * AppScale.amount,
                                ),
                              ),
                            ],
                              image: DecorationImage(
                                image: AssetImage('resources/test.jpg'),

                              )
                          ),

Flutter version: latest from stable channel

Hi @kirill-21
I cannot reproduce this issue on the latest master/stable channel using the code bellow.

Can you please provide your flutter doctor -v and a code that reproduce the issue?
Thank you

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        body: Center(
          child: Container(
            width: 425.0,
            height: 225.0,
            decoration: BoxDecoration(
              borderRadius: BorderRadius.all(
                Radius.circular(12),
              ),
              color: Colors.white,
              boxShadow: [
                BoxShadow(
                  color: Colors.black,
                  spreadRadius: 0.0,
                  blurRadius: 16.0,
                  offset: Offset(
                    0.0,
                    2.0,
                  ),
                ),
              ],
              image: DecorationImage(
                image: AssetImage('assets/images/logo.png'),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
/Users/pedromassango/Code/flutter_master/bin/flutter doctor --verbose
[✓] Flutter (Channel master, 2.3.0-2.0.pre.163, on macOS 11.2.3 20D91 darwin-x64, locale en-AO)
    • Flutter version 2.3.0-2.0.pre.163 at /Users/pedromassango/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 298f18b592 (31 hours ago), 2021-05-12 20:50:14 -0700
    • Engine revision a339374296
    • Dart version 2.14.0 (build 2.14.0-109.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/pedromassango/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • 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 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 56.0.5
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 11.2.3 20D91 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 90.0.4430.212

• No issues found!

flutter doctor -v
The code is in the first message

Hi @kirill-21
I've set the opacity to .15 using the code (Colors.red.withOpacity(0.15)) and I'm still able to see the color. Can you please provide a code that reproduces the issue?

Hi @kirill-21
I've set the opacity to .15 using the code (Colors.red.withOpacity(0.15)) and I'm still able to see the color. Can you please provide a code that reproduces the issue?

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        body: Center(
          child: Container(
            width: 425.0,
            height: 225.0,
            decoration: BoxDecoration(
              borderRadius: BorderRadius.all(
                Radius.circular(12),
              ),
              color: Colors.white,
              boxShadow: [
                BoxShadow(
                  color: Colors.deepPurple.withOpacity(0.15),
                  spreadRadius: 0.0,
                  blurRadius: 8.0,
                  offset: Offset(
                    0.0,
                    2.0,
                  ),
                ),
              ],
              image: DecorationImage(
                image: AssetImage('assets/test.jpg'),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

The image covers the shadow below itself

Hi @kirill-21
It doesn't, the thing is that you are not able to see the color because of the contrast of the image. If you build this on web and you zoom it, you will clearly see shadow on top and below the image.

Zoomed image - notice the shadow color partially visible

To fix this, you can simply increase the opacity of your shadow's color.

Hi @kirill-21
It doesn't, the thing is that you are not able to see the color because of the contrast of the image. If you build this on web and you zoom it, you will clearly see shadow on top and below the image.

Well, i've changed the image and opacity to .95. The shadow became visible, but it differs from an empty container, it's very light and it seems halfly cut.

This is still a contrast thing
Here I took a brush in photoshop and removed the lion

This is still a contrast thing
Here I took a brush in photoshop and removed the lion

I've created an extra slow animation to show how the image cuts the shadow, not recontrasts it

From the engine perspective, this seems WAI to me given the experiments related to making the shadow be of a different color to make it more apparent. cc @goderbauer for thoughts on whether the framework does anything differently in the case of the decoration image.

We took a closer looks at this and we don't think there is a bug here. The lack of the image makes the border more noticeable as was confirmed by a previous comment. The box shadow and its arguments are independent of the image being used and an identical shadow will be rendered by the engine. If you want to make the shadow more or less noticeable, you could perhaps modify the box shadow arguments.
Since there is no action item, I am closing this as WAI.

This is wild. It's got to be some sort of variant of the https://en.wikipedia.org/wiki/Chubb_illusion or the https://en.wikipedia.org/wiki/Checker_shadow_illusion maybe?
Feel free to reach out on #general on the Discord channel to chat about it more if you're not convinced. :-)


Loading model trained from scartch

Hi,
I am training a model from scratch using 'patch_tst_getting_started.ipynb' notebook. It seems that the model is saving checkpoints in "./checkpoint/patchtst/direct/train/output/" . How can I load the final model? I assume that the best model in terms of validation loss is saved in "./checkpoint/patchtst/direct/train/output/" ?
In my case, three checkpoint repos are created : "checkpoint-2597000", "checkpoint-2623500" and "checkpoint-2650000". How these checkpoints are selected?
Thank you,
Best regards

Once training has completed, you can run:
trainer.save_model("new_model_path")

Then, later you can use .from_pretrained, e.g. PatchTSTForPrediction.from_pretrained(new_model_path).


issue-10-rename-clickme

I'll make a new branch for the exiting

@hodgestar can I land this in the meantime? :)

:+1:


GH1287: Adds a try-catch around code that reads PATH environment variable

#1287

On second thought, also trims spaces and quotes from start and end of paths. Given that it's not unusual to find quotes in the PATH string (this is what tripped me up), trimming them seems like a reasonable way to accept paths that the user thinks are valid but that System.IO won't accept.
I also still think we should have the empty catch guard since we  probably never want to fail the build due to an external configuration like environment variables.

@daveaglick LGTM 👍 ,  your changes have merged and will be released in the next version of Cake. Thanks for contributing!


Example flake.nix has deprecated attributes extraModules and configuration

The example flake uses deprecated attributes extraModules and configuration, see https://nix-community.github.io/home-manager/release-notes.xhtml#sec-release-22.11-highlights
Putting both plasma-manager and home.nix in modules seems to work, though (at least it builds)...
Here is my flake:
{
  description = "Home-Manager Flake";

  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixos-unstable";

    home-manager.url = "github:nix-community/home-manager";
    home-manager.inputs.nixpkgs.follows = "nixpkgs";

    plasma-manager.url = "github:pjones/plasma-manager";
    plasma-manager.inputs.nixpkgs.follows = "nixpkgs";
    plasma-manager.inputs.home-manager.follows = "home-manager";

    nixgl.url = "github:guibou/nixGL";

    custom-any-nix-shell.url = "github:manuelbb-upb/any-nix-shell/nix_develop";
    custom-any-nix-shell.inputs.nixpkgs.follows = "nixpkgs";
  };

  outputs = inputs@{ nixpkgs, home-manager, plasma-manager, nixgl, custom-any-nix-shell, ... }:
  let
    system = "x86_64-linux";
  in {
    homeConfigurations.manuelbb = home-manager.lib.homeManagerConfiguration {
      pkgs = import nixpkgs {
        inherit system;
        config.allowUnfree = true;
        overlays = [ nixgl.overlay ];
      };
      modules = [
        inputs.plasma-manager.homeManagerModules.plasma-manager
        ./manuelbb_config/home.nix
      ];
      extraSpecialArgs = { inherit inputs; };
    };
  };
}

Will test a bit more and hopefully manage a PR.

I think this is fixed by now, at least I couldn't find any example still using extraModules.


Color image from T265 camera

Is it possible to get color images from T265? I am planning using T265 as a tracking (odom) device on my robot-car. And at the same time, processing the images from both cameras on T265 in my own particle filter based localization program where I need use color to detect land markers.

I don't think you can get the color image from T265 as it only has Grey scale fisheye lenses. I am pretty sure it doesn't contain RGB module.

Yes, the sensors of T265 are gray-scale

How to display the fish eye camera view of T265 with pyrealsense2 in Jetson nano


Attribute stringification

Vue version
3.4.28
Link to minimal reproduction
http://example.com
Steps to reproduce
In vue 3.4.28 in @vue/runtime-dom/runtime-dom.esm-bundler.js line 587 you added attribute stribgification for non-boolean attrs:
...
el.setAttribute(key, isBoolean ? "" : String(value));
...

This is not correct and leads to errors.
Attribute can be not a string, and this is not prohibited.
We are using objects for some attrs.
What is expected?
Is it possible to revert this patch?
What is actually happening?
Please, see above.
System Info
Please, see above.

Any additional comments?
No response

Invalid reproduction, could you please provide an actual reproduction?

yes, give me 2 hours, don't close issue pls

Are you using custom elements with value, checked, or selected props? This might be a duplicate of issue #11163.

I am using ExtJS web components:
<ext-grid :store="store" ... />

where store is instance of Store class.

Should I create and deploy reproduction code?
I think the problem is clear.

Sounds like a duplicate of #11163 ?

Should I create and deploy reproduction code? I think the problem is clear.

The problem is not really clear. Because the line you point to can't really be the problem on its own, here's why:
HTML Attributes are always converted to strings. That's part of the spec, and you can see a demo of this behaviour in this playground where I use vanilla DOM APIs to set an attribute to an obejct value without first converting it to a string manually. When you check the console, the attribute's value will be [object Object].
The patched line (https://github.com/vuejs/core/commit/537a571f8cf09dfe0a020e9e8891ecdd351fc3e4) you point to just makes sure that stringification works in a special case where the value is a Symbol - which also doesn't apply to your situation.
So clearly, something else must be going on, maybe some edge case or interaction with the way these extJS web components handle attributes/dom properties? Usually, web components expose all their attributes as DOM properties as well, which in turn ensures that Vue doesn't set them as attributes, but as DOM properties (see this check), but as that part of the code wasn't affected by 3.4.28, this also doesn't look related
So yes, we need code from you that we can run to reproduce the actual problem in a running app. If you can reproduce it in the playground, that's fine. If you need to provide a minimal github repo, that's also cool.

It works when I replace String(value) with just value in your package source code.
The error is related exactly to this change and attrs are not converted to the strings by default.
ExtJS process attrs after vue, so the problem should be fixed in vue.

I still don't understand what the problem is. We already established that the value will be stringified either way. So appearantly, the exJS component would re-convert the string into a store instance?
So do we have an issue where our implemenation does stringify that store object differently than the browser would?
As I explained, the change we did was to address an existing issue. If it causes a different issue for your situation, we need to understand why and how it causes an issue in order to find the best way to solve it without reverting the current fix, optimally.
We need  your help to get to that understanding. If you are not willing to provide that help, this issue will be closed.

Okay, addendum: I think I can guess what the specific issue is:
the extJS behaviour relies on the fact that the browser would call toString on objects passed to ``setAttrtibute()`. Likely, this ExtJS store provides this method that would serialize the store into a string which the web componen can re-convert into a Store instance.
String(store) doesn't do that.
Solution for keeping both the current fix and solve the reported issue:
} else {
    // attribute value is a string https://html.spec.whatwg.org/multipage/dom.html#attributes
-  el.setAttribute(key, isBoolean ? '' : String(value))
+  el.setAttribute(key, isBoolean ? '' : typeof value === 'symbol' ? : String(value) : value)
}

The problem is that store is the instance of the class, it is not a plain object, and can't be stringified / parsed from json.
I think your bugfix is not correct, you should not modify attrs values, or strictly limit the scope, where you do this.
If you need to convert Symbols to strings in some specific cases - you need to do this for your cases only, not for all cases.
At lease you can add type check to make sure, that attr value is a Symbol.
But most correct way - is to pass already stringified Sylmols to attrs, where you need it.

Yes. the code, you posted above, will work.

The problem is that store is the instance of the class, it is not a plain object, and can't be stringified / parsed from json.

But that's the part I don't understand. I just demonstrated to you that attributes will always be stringified by the browser. So ExtJS must re-parse a string into a Store Class instance.
But maybe that's something for users of those components to worry about.

vue pass object as attribute value to the custom web element.
ext component stores attr value in the property, if type of value is object.

I don't understand that explanation given what I demonstrated in the playground.

you pass object as attr value.
extjs has handler for set attribute value operation, it called before browser. If value is object - extjs stores it in property instead of attribute.

Ah, I see, thanks. So it overrides the default implementation setAttribute. that's an .... interesting approach.


.eslintrc.js file in the migration directory

My project has an eslint rule that requires file names to follow a regex rule. However the naming scheme changes in the migrations folder and I need to put an eslintrc file into that folder to overwrite it. I cannot find an option to have knex ignore the file. Any options here?

You could rename it to just .eslintrc, not .eslintrc.js.

sounds like viable solution, please comment / reopen if it doesn't work.

Unfortunately, while still usable via eslint, this is a deprecated feature and not allowed by our coding standards. So, while that would solve the problem, I don't think it is really a solution for your users :)

maybe you can add one extra directory to the path where migrations are and add the lint configuration in there...
src/otherlintingruleshere/migrations

Anyways there has been discussion about adding configuration option for migration extensions... with that you could for example set all migration files to .migration.js extension and lint configuration would be ignored.


feat(support): add methods reduce, chunk and findKey to ArrayHelper

This PR adds additional methods to the TODO list of https://github.com/tempestphp/tempest-framework/issues/543

Awesome! Thanks


The overload method is not available on dotenv-rails

I was wondering if there is a specific reason why the overload does not exist on the Rails version, is this an intentional or unintentional omission? I don't mind opening a PR if there's no objection to adding it. Seems like a pretty simple addition.
Steps to reproduce
Try calling Dotenv::Railtie.overload
Expected behavior
Method should exist and act similarly to how Dotenv.overload does.
Actual behavior
NoMethodError:
  undefined method `overload' for Dotenv::Railtie:Class

System configuration
dotenv version: 2.2.1
Rails version: 5.1
Ruby version: 2.4.1

@bkeepers shall I open a PR for this? Thanks

@bkeepers shall I open a PR for this? Thanks

@dkarter I'm wondering if you could overload using Dotenv instead since the railtie just used that under the hood?
require 'dotenv'
Dotenv.overload

@jonmagic I'm not sure how that would work given that dotenv_files, which Dotenv.load/overload are taking as an argument, is a privately defined method inside of Dotenv::Railtie.


Clarify When Support Packet Must be Sent

The current explanation of when to sent a format 3 support packet (in section 7.5) is not wrong, but it could be clarified to better explain the intent.
Before any trace packets are output, a support packet must be issued to provide the encoder configuration to the decoder.  Sending it any time before the first sync packet is acceptable, but the recommended time is when the encoder is enabled (TrTeEnable changes from 0 to 1).  This avoids having to generate two packets (support and sync-start), at the point tracing actually starts.
To inform the decoder that tracing has ceased, a support packet must also be sent when tracing stops for any reason (e.g. trace is disabled by setting TrTeEnable to 0, trace is stopped by a trace-off trigger, or because it no longer qualifies after applying any filtering, tracing ends due to halt or reset, etc.).

@pdonahue-ventana are you happy with this: https://github.com/riscv-non-isa/riscv-trace-spec/pull/164?


Support boolean not operator

What was wrong?
Fe doesn't yet support the boolean not operator. E.g. the following code doesn't compile:
if not covid_test.is_positive(person):
    allow_boarding(person)

How was it fixed?

Analyzer validates UnaryOperator::Not is used with bool only
Mapper maps it to iszero([operand])
Tests

Codecov Report

Merging #264 (d82cd13) into master (f2c2204) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #264   +/-   ##
=======================================
  Coverage   94.03%   94.03%
=======================================
  Files          54       54
  Lines        3791     3791
=======================================
  Hits         3565     3565
  Misses        226      226

Impacted Files
Coverage Δ

analyzer/src/traversal/expressions.rs
92.90% <ø> (ø)

compiler/src/yul/mappers/expressions.rs
97.24% <ø> (ø)

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 f2c2204...d82cd13. Read the comment docs.


Add WithTracing option to enable instrumentation with OpenTelemetry

When the Client is initialized with WithTracing, its underlying HTTP transport will be wrapped by otelhttp's Transport. This will create a tracing span and propagates its context via the provided request's headers before handing the request to the base RoundTripper.
This enables distributed tracing by ensuring that traces are propagated across service boundaries.
References:

https://opentelemetry.io/
https://pkg.go.dev/go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp#Transport

@marz619 ping! 🛎


Add demo site

Add a new page for describing and linking to the new demo site.
Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

Why are we adding a new html page, rather than mentioning this in existing docs?

@brian-brazil Because I want it promoted at the top bar. We're also planning to add additional demos based on K8s deployment.

I'm not sure this meets the bar to be up top, it's already got a fair few links. HTML pages also have additional maintenance costs which standard documentation does not, so we should try to avoid them.
This would seem better as a page under Overview.

While true, demos are special and distinct by nature, having its own top-level navigation element seems prudent.

So my two cents on the demo link at the top: as so often, I'm split :) This definitely makes sense for products where a demo really helps give completely new people a quick overview over how things work (in the sense of "a picture is worth a thousand words"). But a Prometheus demo by its nature doesn't really do that. I think most completely new users looking at that demo site wouldn't really know what to do with it, or what it means. Unless (maybe!) we really flesh it out with a lot of explanatory text and make sure they see Grafana first, etc. Absent of that I'm also tending more towards having it somewhere in the normal docs, but easily findable in there (like a "Demo" page under the "Introduction" section).

Unless (maybe!) we really flesh it out with a lot of explanatory text and make sure they see Grafana first, etc.

That's exactly why it doesn't point directly to prometheus instance, but to a portal site, where we can write explanatory texts etc.
Additionally, I am currently working on incorporating monitoring mixins to deploy them alongside basic prometheus stack. I think this way demo site will give a quick overview on how things should work.

That's exactly why it doesn't point directly to prometheus instance, but to a portal site, where we can write explanatory texts etc.
Additionally, I am currently working on incorporating monitoring mixins to deploy them alongside basic prometheus stack. I think this way demo site will give a quick overview on how things should work.

That would be great actually. But then I think we should hold off on adding a new top-level menu link until the demo site is really understandable for a newbie (in the purpose of the individual elements, getting a quick feel of things, etc.). Otherwise I can see a million new people clicking the link and not knowing what to do with it at all, and being somewhat disappointed... and users who already know Prometheus are not the ones who need the easily-findable demo link.

I wouldn't add the demo site to the top menu either, as it's already rather crammed, but it might be a good addition for a button on our main page next to "Get Started" and "Download". Adding a button there linking to the portal sight might be all what's necessary here? But https://demo.do.prometheus.io/ doesn't load for me.
How do you want to proceed here @paulfantom @SuperQ?


Can't evaluate multiline lists

If I position the cursor at the end of the block
[1,
2]

and evaluate, I get an error syntax: extra token "]" ....

Our block detection is indentation base, which means that
[1,
  2]

will work. Alternatively, you can also select the whole expression and evaluate then.


How to make it work in Wine / linux?

How to make it work in Wine / linux?

In my experience I haven't been able to make it work and have created this issue: https://github.com/EJWellman/QuarmTool/issues/22

I offer to pay 20$!

Or some kronos =)


Feature: Add support for watching folders in file navigator

Summary
This PR enhances the file system interactions, subprocess handling, and introduces a file watcher for real-time file change detection in our project. The focus is on improving performance, error management, robustness of the file operations, process management, and timely updates to file changes.

Changes Made

Optimized the loadFiles function to handle directory enumeration more efficiently using POSIX APIs.
Fixed the improper handling of file descriptors in subprocess execution, ensuring all file descriptors are valid before use in posix_spawn.
Enhanced error handling and logging in file system operations and process creation to provide better debugging and error traceability.
Modified recursive file loading to better manage memory and ensure thread safety.
Implemented a file watcher mechanism to monitor changes in the file system and update the application state in real-time.

TODO

Review the handling of maximum filename lengths and ensure proper null termination in file operations.
Implement further optimizations based on profiling results.
Additional unit tests covering edge cases for both file system operations, subprocess handling, and file watcher functionality.
Integration testing to confirm that the file watcher interacts correctly with other system components.

Motivation
The primary goal of this PR is to address performance bottlenecks, error management issues, and the lack of real-time file change detection in our handling of file system operations and subprocesses. These changes are necessary to improve the reliability and efficiency of our application, particularly in handling large directories, robust process management, and immediate responsiveness to file system changes.

Testing

Manual testing was conducted on the loadFiles function to ensure accurate directory traversal and file handling.
Regression tests were performed to ensure existing functionality was not adversely affected.
The file watcher was tested in various scenarios to ensure it accurately detects and responds to file changes in real-time.
The changes were profiled to confirm performance improvements.

Screenshots/GIFs
N/A

Checklist
Please ensure all of the following are completed before submitting the PR:

[X] Code has been tested and verified.
[X] Documentation has been updated to reflect the changes.
[ ] All tests pass successfully.
[X] Code follows the project's coding guidelines and style.
[X] The branch is up-to-date with the latest changes from the main branch.
[ ] Reviewed by at least one other contributor.

Related Issues
This PR addresses the following issue(s): []
Additional Notes
The changes introduced in this PR involve critical parts of the system; thus, extensive testing and careful review are recommended. We should also monitor the system's behavior in production closely following deployment to catch any unforeseen issues early.

Thanks for submitting a pull request!
We will review it as soon as possible.


Temp is 0 for AMD k10temp

As above, it's reading 0 for AMD for temps.
I can read the temp myself at: /sys/class/hwmon/hwmon6/temp1_input
I think line 292 of depend.go needs to be something like
if string(f) == "coretemp\n" || string(f) == "k10temp\n" {
Though after taking a second look I don't think the averaging code is correct as k10temp provides temps for each CCD as well as the hotspot temp so averaging them isn't correct. Should just take the hotspot (Tctl in the _label file).

I'll fix it tomorrow.

Created a PR https://github.com/ssleert/zfxtop/pull/14 to fix the temp values being averaged incorrectly

Close?


[URGENT] Journey not updating on Website

The token seems to be "expired" or removed. I hope you had set the token to never expire once it was created @Adarsh-jaiss . Hence, the journey's are not updating on the website, which is really BAD, please fix at the earliest @Adarsh-jaiss.
cc: @vikasganiga05

"authorization":"token [REDACTED]"}}}
SOURCE: https://github.com/collab-community/journey-book/runs/7747975742?check_suite_focus=true

I haven't created this token. It was created by you or vikas

I haven't created this token. It was created by you or vikas

Oh, request you to please create this token. @vikasganiga05 will guide you.

Kindly explain me the process and I will create it ASAP! ,
PLEASE ADD WHAT TO CHOOSE WHILE CREATING ACCESS TOKEN TOO

Kindly explain me the process and I will create it ASAP! , @vikasganiga05 PLEASE ADD WHAT TO CHOOSE WHILE CREATING ACCESS TOKEN TOO

even im not sure, lemme see.

@Adarsh-jaiss choose everything generate the token and put it in BOT_ACCESS_TOKEN environment secret and see if it works, if it doesn't then idk

Fixed yesterday, although now it's on my token instead of @Adarsh-jaiss.


export all print methods, closes #102

Checklist

[❎] update NEWS.md
[✅] documentation updated with devtools::document()
[ ? ] devtools::check() passes locally

Changes
updated the roxygen for head.FeatureLayer() and added it to NAMESPACE with devtools::document(). this is a really simple edit, so i'm not messing with updating the news (unless you think that's mission critical here).
Issues that this closes
Closes https://github.com/R-ArcGIS/arcgislayers/issues/102
Follow up tasks
Still getting the Error: object 'head' not found whilst loading namespace 'arcgislayers' in devtools::check(). I wonder if you have to register that function as an S3 generic on load since it comes from utils?

I stole this and added it to a more comprehensive PR that gets R CMD check to actually pass.... Thank you!
Please make a PR to add yourself as a contributor to the package! role = "ctb"


full music url

var spotify = new SpotifyClient("My Token Access");
var track = await spotify.Tracks.Get("6StaOnAGq2jhJeSZEy5CYY");
this is a code for get tracks , and i get just PreviewUrl . can we get full music or not? i want download the music.

Hi, that's not possible via the Spotify WEB API.
Other resources:
https://developer.spotify.com/documentation/web-playback-sdk/


Example on how to parse a row of the synthetic data and convert it to a Uk template

Note: this is just an example to convert a row of the data to the template. Other changes are likely needed to make this compile and work like :

Specifying the correct columns to select from
Specifying types of the columns
The multi event group changes Origin needs to make to take the run based on the spec
Changing the DataproviderIds either in the synthetic data or the EDPSimulators to match each other.

This change is


Is the process of from rgbd ros topic to lcm loss message?

sorry to bother you, I use your rgbd_ros_to_lcm package and Labelfusion to create 6d pose datasets, the collect lcm message process is first "roslaunch openni2_launch openni2.launch" second "rviz" third "rgbd_ros_to_lcm" and finally lcm-logger, but when I use the collected lcm message to 3d reconstruction the result is poor compare the collect lcm message process "./openni2-camera-lcm" and "lcm-logger".could you give me some advice on how to solve the problem, thanks for advance.

If you can use openni2-camera-lcm directly why are you using rgbd_ros_to_lcm? The purpose of rgbd_ros_to_lcm is to convert ROS messages into LCM format in cases where an LCM camera driver is not available.
My best guess is that your computing platform is not powerful enough to perform the conversion at the full frame rate, which results in partial data loss.

If you can use openni2-camera-lcm directly why are you using rgbd_ros_to_lcm? The purpose of rgbd_ros_to_lcm is to convert ROS messages into LCM format in cases where an LCM camera driver is not available.
My best guess is that your computing platform is not powerful enough to perform the conversion at the full frame rate, which results in partial data loss.

The reason why I don't use the process of "./openni2-camera-lcm" and "lcm-logger" to collect data is when I collect data I can't see the camera image as same time, as I move the camera I can't guarant the target always in the camera field of view, so I use the first process to collect data publish it as ros topic visual image in rviz and convert it  LCM format to make lcm logger

When I use you package to collect data, the post process of 3d Reconstruction performance is poor, I don't know why.Could you give me some advice on how to solve the problem, thanks for advance.

If you open lcm-spy what is the frequency of messages on the OPENNI_FRAME channel?


Please respect the BUILD_TESTING variable

Original report (archived issue) by yurivict (Bitbucket: yurivict).

There is no need to build tests by default, only with BUILD_TESTING=ON.
https://cmake.org/cmake/help/v3.0/module/CTest.html

This has been fixed by https://github.com/osrf/sdformat/pull/269. However, the behavior is to build tests unless BUILD_TESTING=OFF.


Update glass BSDF reference images

There is more grain now. :smiley:

Here there is, there's the explanation from the commit:
The previous reference images were wrong: a bug introduced in 672b0d7 ("Add SSE implementations of min/max_value() on spectra") essentially disabled russian roulette, leading to longer paths and less noise. The bug was subsequently fixed in e1069e1 ("Fix min_value() and max_value() for renderer::DynamicSpectrum when size=3").

Thanks! :)


Is the Air Shaft not supposed to be tube-like in behavior? Buildings cannot be built right against it.

The Air Shaft structure (the structure that the RoboDigger builds), both above and underground, should probably also be able to function as an "intersection tube".
Currently only Tube structures are able to be built right against it, but not regular buildings, even though I would assume it would carry these fluids/gases.
They do look very much alike (tube intersection and Air Shaft), of course, it isn't in the player's best interest to do so below ground, but it shouldn't be impossible (unless there is good reason to).
MapViewStateHelper has a function
checkStructurePlacement

which returns false.
Even though the Air Shaft is a Tube type structure and it has connector direction CONNECTOR_INTERSECTION, but it is not used when comparing the connector directions.

bool checkStructurePlacement(Tile& tile, Direction dir)
{
	Structure* _structure = tile.structure();
	if (tile.mine() || !tile.bulldozed() || !tile.excavated() || !tile.thingIsStructure() || !tile.connected() || !_structure->isConnector())
	{
		return false;
	}

	if (dir == Direction::East || dir == Direction::West)
	{
		if (_structure->connectorDirection() == ConnectorDir::CONNECTOR_INTERSECTION || _structure->connectorDirection() == ConnectorDir::CONNECTOR_RIGHT || _structure->connectorDirection() == ConnectorDir::CONNECTOR_VERTICAL)
		{
			return true;
		}
	}
	else // NORTH/SOUTH
	{
		if (_structure->connectorDirection() == ConnectorDir::CONNECTOR_INTERSECTION || _structure->connectorDirection() == ConnectorDir::CONNECTOR_LEFT || _structure->connectorDirection() == ConnectorDir::CONNECTOR_VERTICAL)
		{
			return true;
		}
	}

	return false;
}

This would allow an Air Shaft to also count as an intersection or is there logic that could fail in this setup?

Hmm, interesting. Kind of sounds like you've discovered an oversight bug to me. Maybe double check with @ldicker83 though.

I would also prefer the air shaft be considered a tube and found current behaviour a bit confusing.

@ldicker83 What is your opinion here?

You guys are the users here... if you think the air shaft should behave as a tube I have no objections. I based the behavior off the original game but I have no preference either way.
You can push a PR that changes this behavior and, assuming it passes the CI tests and doesn't break the game I'm good to merge.

Above ground:

Underground:

So far nothing seems to break.

I even found some pre-release screenshots from long ago (just tongue in cheek btw :) ) but they also allowed no tubes between buildings (and that's something I would not introduce):

Source: http://www.ibiblio.org/GameBytes/issue18/flooks/outpost.html
And even in the title screen, at one time they at least considered it: :)

Source: https://www.abandonware-france.org/ltf_abandon/ltf_galeries.php?id=1030&galerie=9

I suspect their artists didn't know the rules when they mocked these up. :D Not sure if that early screenshot is from actual gameplay or if someone just threw it together to give publications something to look at. Still though, fun to look at. :D


 tag - Dev hot-reload conflict

After building the snippet/importmap.liquid is first created and everything works as expected.  However during development the base tag <base href="//store-name.myshopify.com/cdn/shop/t/109/assets/"> is causing the code that the Shopify CLI injects for Hot Reload to connect to //store-name.myshopify.com/hot-reload instead of <IP_ADDRESS>:9292/hot-reload.
It would be helpful if during dev vite either base URL is correctly set to <IP_ADDRESS>:9292 or even if the importmap.liquid file was emptied.

@jonathanmoore Thanks for the catch, you're right. I think it would be better if the importmap file is just cleaned up during dev, because it's not necessary, vite will do everything for us. I'll check it tomorrow :)


Change s3 bucket for e2e tests, current default exists somewhere

What this PR does / why we need it:
Tests are failing due to bucket already existing in a different account.

/test pull-aws-iam-authenticator-unit


Feat: core/log_client - Supersede default rsyslog configuration (#488)

Forgot to update changelog

Shouldn't we do the same than NHC role ?
Remove this:
{% if log_client_static_configuration is defined and log_client_static_configuration is not none %}

{{ log_client_static_configuration }}

{% else %}

And add a filter on the main configuration file template, to check if not defined in log_client_configuration_files ?
This is close to what is proposed here, but I think simpler.

I am having trouble removing log_client_static_configuration:
In the nhc role, the configuration is written in /etc/nhc, but rsyslog expects /etc/rsyslog.conf.
Since the include directory must be defined in rsyslog.conf, maybe it can be added to the log_client_configuration_files ?
- name: template █ Install custom configuration files
  template:
    src: rsyslog.d_custom.conf.j2
    dest: "{{ item.path}}/{{ item.name }}"
    owner: root
    group: root
    mode: 0644
  loop: "{{ log_client_configuration_files }}"
  loop_control:
    label: "Install configuration file {{ item.name }}"

log_client_configuration_files:
  - name: local-rules.conf
    path: /etc/rsyslog.d
    content: |
      my configuration for local-rules
      can be multiline
  - name: forwarding-rules.conf
    path: /etc/rsyslog.d
    content: |

      my configuration for forwarding-rules
  - name: rsyslog.conf
    path: /etc
    content: |
      # Include all config files in /etc/rsyslog.d/
      $IncludeConfig /etc/rsyslog.d/*.conf

Good point.
I would default the path to /etc/rsyslog.d to ease the configuration for end users:
template:
    src: rsyslog.d_custom.conf.j2
    dest: "{{ item.path | default('/etc/rsyslog.d') }}/{{ item.name }}"

log_client_configuration_files:
  - name: local-rules.conf
    content: |
      my configuration for local-rules
      can be multiline
  - name: forwarding-rules.conf
    content: |

      my configuration for forwarding-rules

Plus an explanation in the readme for those who want to skip the default template. Something like "You can override the default templating of /etc/rsyslog.conf by adding the configuration below to log_client_configuration_files:"
  - name: rsyslog.conf
    path: /etc
    content: |
      # Include all config files in /etc/rsyslog.d/
      $IncludeConfig /etc/rsyslog.d/*.conf


Allow skipping the cost function calculation

Added a calc_cost parameter to the main function that can be set to False to skip the cost function calculation to reduce calculation time when the cost function values are not needed. Addresses the second part of issue #1.

I am going to close this PR since it's actually not needed. You can make the calculation time of the cost function negligible by:

Use np.sum rather than sum in theta and the cost calculation; since the values are numpy arrays, np.sum is significantly faster than sum.
Factorize the A matrix using the function splu from scipy.sparse.linalg and then use the factorization's solve method in the H function; since A is unchanged within the iterative portion, its factorization also never changes. It is effectively doing the same as spsolve does, but faster since the factorization is only performed once rather than every iteration. It would look like:

A_factorization = splu(A.tocsc())  # splu requires csc format
H = lambda xx: B.dot(A_factorization.solve(xx))

With those changes, the cost function takes almost no time. I don't want to flood your project with PRs so I'm not going to open another pull request with these changes, but feel free to incorporate these changes yourself. :)


[REQUEST] opence option to specify bazel cache directory

Is your feature request related to a problem? Please describe.
By default, bazel caches files it downloads to ~/.cache/bazel.  On the particular system where I build Open-CE, the home directory is small due to quota limits.  The Open-CE build eventually fails when it exhausts the available space in this file system.
Describe the solution you'd like
I would like to point bazel to use another cache directory.
I have been able to work around my problem by modifying the builds scripts that invoke bazel to set $TEST_TMPDIR like so:
diff --git a/recipe/build-pip-package.sh b/recipe/build-pip-package.sh
index 930d84b..d43facb 100644
--- a/recipe/build-pip-package.sh
+++ b/recipe/build-pip-package.sh
@@ -37,6 +37,9 @@ $SCRIPT_DIR/set_tensorflow_bazelrc.sh $SRC_DIR/tensorflow
 bazel clean --expunge
 bazel shutdown

+# tell bazel to use /dev/shm as its cache (rather than ~/.cache/bazel)
+export TEST_TMPDIR=/dev/shm/.cache
+
 bazel --bazelrc=$SRC_DIR/tensorflow/tensorflow.bazelrc build \
     --config=opt \
     --config=numa \

That variable is described here: https://docs.bazel.build/versions/master/output_directories.html
Could we add an option to the opence command to specify a different bazel cache directory?
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Rather than setting $TEST_TMPDIR in the build script for each feedstock recipe, I found there is an --output_user_root option in bazel that one can use to set this path.
https://docs.bazel.build/versions/master/output_directories.html#current-layout
https://docs.bazel.build/versions/master/command-line-reference.html#flag--output_user_root
This can be added to a global bazel resource file, e.g.,
echo "startup --output_user_root=/dev/shm/.cache" >> ~/.bazelrc

I'm happy with this as a fix for my problem, so feel free to close this out.  And in particular, redirecting bazel to download and build software in /dev/shm vs something like NFS also should speed up the build.
This might be a useful troubleshooting tip in the docs in case others run into a similar problem.

I and @jayfurmanek discussed this and decided that we will update the documentation for this. We will mention about $TEST_TMPDIR variable that can be used to change the bazel cache directory.

Done

I just noticed the documentation includes a note about $TEST_TMPDIR.  Thanks, @npanpaliya and @alexlang74 .
For people who need to take this step, it may be easier to set --output_user_dir in the bazel resource file.  That's what I have switched to.
https://github.com/open-ce/open-ce/issues/324#issuecomment-782761891
That can be done once at a top-level rather than modifying each feedstock.

Thanks, @adammoody !


HANAConfigMissingException:  A comma-separated list of topics is required to run

Hi All,
I'm currently facing a problem trying to get Kafka-Connect-SAP connector working properly.
I have done all the required configurations, including the JDBC driver on the same connector's directory.
This are my properties config:
name=kafka-connect-source
connector.class=com.sap.kafka.connect.source.hana.HANASourceConnector
tasks.max=1
topics=kafka_source_1,kafka_source_2
connection.url=jdbc:sap://hana/
connection.user=SYSTEM
connection.password=HXEHana1
kafka_source_1.table.name="SYSTEM"."KAFKA_SOURCES"

I have configure a docker image for Hana service, which is responding successfully from my Kafka containers.
$ ping hana
PING hana (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.095 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.142 ms

This is the current stacktrace.
com.sap.kafka.client.hana.HANAConfigMissingException: Mandatory parameter missing: A comma-separated list of topics is required to run the HANA-Kafka connectors
	at com.sap.kafka.connect.config.BaseParameters$class.getConfig(BaseParameters.scala:33)
	at com.sap.kafka.connect.config.hana.HANAParameters$.getConfig(HANAParameters.scala:13)
	at com.sap.kafka.connect.source.GenericSourceTask$$anonfun$1.apply$mcV$sp(GenericSourceTask.scala:35)
	at com.sap.kafka.connect.source.GenericSourceTask$$anonfun$1.apply(GenericSourceTask.scala:34)
	at com.sap.kafka.connect.source.GenericSourceTask$$anonfun$1.apply(GenericSourceTask.scala:34)
	at com.sap.kafka.utils.ExecuteWithExceptions$.apply(ExecuteWithExceptions.scala:40)
	at com.sap.kafka.connect.source.GenericSourceTask.start(GenericSourceTask.scala:34)
	at org.apache.kafka.connect.runtime.WorkerSourceTask.execute(WorkerSourceTask.java:157)
	at org.apache.kafka.connect.runtime.WorkerTask.doRun(WorkerTask.java:170)
	at org.apache.kafka.connect.runtime.WorkerTask.run(WorkerTask.java:214)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

It says that comma-separated list of topics is required to run the HANA-Kafka connectors, but they were specified on the properties.
Can you help me to figure this out?

Solved after rerun mvn commands


Would be great to use named placeholder

Eg. In some cases it might be way more readable if I can do
process.env
|> Object.keys(^)
|> filterUppercase(^keysList)
|> filterCamelCase(^upperKeys)

This allows me to express what is expected thing at given stage making it easier to read for fellow developer.

Thanks for the issue!
We thought about named topics and played around with some syntaxes, such as v |:foo> f(foo).
In the end, we decided to leave named topics to future proposals, making them out of scope of this proposal.
(Pipes aren’t meant to replace variable assignment, after all; we’re keeping this simple.)
const keysList = process.env |> Object.keys(^);
const upperKeys = filterUppercase(keysList);
filterCamelCase(upperKeys);

We’re leaving it as a possibility for future proposals—sorry if that’s disappointing, and thanks again for the issue. 👍

Note that pipeline is, in this respect, identical to method chaining:
process.env
	.keys()
	.filterUppercase()
	.filterCamelCase();

The implicit subject of each call isn't named at any point there, either, but that seems to be broadly acceptable to the JS community, and method chaining remains a very popular syntax.
And, just like method chaining, you can always break the pipe and assign the result to a variable when that would aid in readability, as @js-choi's example shows off.


ITMS-90338: Non-public API usage for public class while submitting my app using  0.9.15 for the module

When I upload the app to App Store, this gives exceptions as follow :
ITMS-90338: Non-public API usage - The app contains or inherits from non-public classes in XXX : NBPhoneNumberUtil . If method names in your source code match the private Apple APIs listed above, altering your method names will help prevent this app from being flagged in future submissions. In addition, note that one or more of the above APIs may be located in a static library that was included with your app. If so, they must be removed. For further information, visit the Technical Support Information at http://developer.apple.com/support/technical/

no one?


Added locks around static dictionary

Potentially avoids race conditions
@dilandau2001, any opinion?

Hi.
It seems pretty correct to me. With that change you make access to the static dictionary thread safe.
In the other hand, based on the last pull I did where I removed a static dictionary like that I would ask if it is really necessary to have it... right now is out of my understanding unless studying it deeper.


Refactor Phalcon\Dispatcher

Refactor Phalcon\Dispatcher

Resolved in https://github.com/phalcon/phalcon/pull/289


Generate CLI docs w/ PULUMI_EXPERIMENTAL flag

With this flag set, we will get docs generated for CLI commands like pulumi ai web. Fixes #9972

@thoward to add a section on experimental features to the CLI homepage
@justinvp any thoughts on documenting this publicly?


download file

not completely download my file and return code 200

Please provide a code sample and more information when creating an issue.

I'm having the same issue, but I think it's not something related to Dio, some iOS permission, idk. @229106948 any code?


Non-desired additional characters in branch on Debian Jessie

I have tried some themes (bobby, envy, powerline) and, in most of all, I have extra characters to the side of branch like "?:3","U:1":
|master ?:3 ✗|
|master U:1 ✗|
|master U:2 ✗|
Is there a setting to not show them?

run "git status", you will find untracked files or maybe other issues. consider adding this into your .gitignore. this is a feature, not a bug :)

In short, the themes are detecting that you are in a Git project folder, and it's showing you the current status.

"master" is the name of the Git branch that you're on
"?" shows the number of untracked files
"U" shows the number of unstaged files
The "X" shows that your workspace is dirty, i.e. that it has uncommitted changes.

I suggest you learn how to use this feature, as it's extremely useful.
If you still want to disable this feature, you can do so by defining the following in your ~/.bashrc or ~/.bash_profile (depending on your OS):
export SCM_GIT_SHOW_DETAILS=false

Thank you so much @nwinkler . I like this feature, but not the symbols used for that (?, U). Is there a way to change this symbols for something more informative like (UK, UG)? Anyway, I am closing the issue. Thank you.

You can change these as part of the theme. Check out the theme definitions - the default values are defined in the base theme here: https://github.com/Bash-it/bash-it/blob/master/themes/base.theme.bash#L33
You can override these values in your own theme to something that you like better.

Yeah!!, You are right @nwinkler. Thank you so much.


Creating a sharing link in ODB succeeds most times but others fails with 403 Fobidden

My team has been creating sharing links for a company's users' files, using ODB with an admin token. Most of the times, we can create sharing links successfully but other times, we get a 403 Forbidden response.
We are using this endpoint to create the sharing links.
I have not been able to determine in what circumstances creating the sharing links fails. It succeeds in some PDF, PPTX and even EXE files and it fails in others, for the same user.
If a feature needs to be enabled, like the error shows, what are the steps needed to enable such feature?
Sample response:
HTTP/1.1 403 Forbidden
Cache-Control: private, max-age=0
Transfer-Encoding: chunked
Content-Type: application/json;odata.metadata=minimal;odata.streaming=true;IEEE754Compatible=false;charset=utf-8
Expires: Thu, 07 Jul 2016 04:28:29 GMT
Last-Modified: Fri, 22 Jul 2016 04:28:29 GMT
Server: Microsoft-IIS/8.5
X-SharePointHealthScore: 0
SharePoint-CreateSharingLink-Error: Sharing by link is not enabled on the web, site, or tenant.
X-SP-SERVERSTATE: ReadOnly=0
ODATA-VERSION: 4.0
SPClientServiceRequestDuration: 491
X-AspNet-Version: 4.0.30319
SPRequestGuid: 9730929d-e08e-2000-0a07-31c08d44878a
request-id: 9730929d-e08e-2000-0a07-31c08d44878a
Strict-Transport-Security: max-age=31536000
X-FRAME-OPTIONS: SAMEORIGIN
X-Powered-By: ASP.NET
MicrosoftSharePointTeamServices: <IP_ADDRESS>07
X-Content-Type-Options: nosniff
X-MS-InvokeApp: 1; RequireReadOnly
P3P: CP="ALL IND DSP COR ADM CONo CUR CUSo IVAo IVDo PSA PSD TAI TELo OUR SAMo CNT COM INT NAV ONL PHY PRE PUR UNI"
Date: Fri, 22 Jul 2016 04:28:30 GMT

{"error":{"code":"notAllowed","message":"The feature has been disabled. Please contact your admin to get it enabled."}}

Do you see different behavior within the same drive/document library?

@ificator Yes. For the same ODB user, I was able to create a sharing link to file /sample.exe but not to file /subfolder/another.exe, both of which reside in their Documents library.

That does seem unexpected - we'll need to drill in and see what's going on

@ificator I have a report on sharing links that got created successfully and those that didn't. I could send it over in case it helps. I also have newer error responses in the logs in case they might be helpful. Let me know if I can help somehow. Thanks.

Hi @dluces, do you have a response from a request that succeeded, ideally in the same document library as the one that failed?

@ificator Unfortunately, I don't have a log for successful requests anymore. However, I do have a list of items for which we successfully created sharing links.
I'll see what I can do to get a log with successful requests and will get back to you. Thanks

Hello @ificator,
I've managed to log more responses for you.
Successful response:
HTTP/1.1 200 OK
Cache-Control: private, max-age=0
Transfer-Encoding: chunked
Content-Type: application/json;odata.metadata=minimal;odata.streaming=true;IEEE754Compatible=false;charset=utf-8
Expires: Sun, 10 Jul 2016 22:53:33 GMT
Last-Modified: Mon, 25 Jul 2016 22:53:33 GMT
Server: Microsoft-IIS/8.5
X-SharePointHealthScore: 0
X-SP-SERVERSTATE: ReadOnly=0
ODATA-VERSION: 4.0
SPClientServiceRequestDuration: 210
X-AspNet-Version: 4.0.30319
SPRequestGuid: 0467939d-402c-2000-ab2c-c63001f0590e
request-id: 0467939d-402c-2000-ab2c-c63001f0590e
Strict-Transport-Security: max-age=31536000
X-FRAME-OPTIONS: SAMEORIGIN
X-Powered-By: ASP.NET
MicrosoftSharePointTeamServices: <IP_ADDRESS>07
X-Content-Type-Options: nosniff
X-MS-InvokeApp: 1; RequireReadOnly
P3P: CP=\"ALL IND DSP COR ADM CONo CUR CUSo IVAo IVDo PSA PSD TAI TELo OUR SAMo CNT COM INT NAV ONL PHY PRE PUR UNI\"
Date: Mon, 25 Jul 2016 22:53:32 GMT

Error:
HTTP/1.1 403 Forbidden
Cache-Control: private, max-age=0
Transfer-Encoding: chunked
Content-Type: application/json;odata.metadata=minimal;odata.streaming=true;IEEE754Compatible=false;charset=utf-8
Expires: Sun, 10 Jul 2016 22:57:53 GMT
Last-Modified: Mon, 25 Jul 2016 22:57:53 GMT
Server: Microsoft-IIS/8.5
X-SharePointHealthScore: 0
SharePoint-CreateSharingLink-Error: Sharing by link is not enabled on the web, site, or tenant.
X-SP-SERVERSTATE: ReadOnly=0
ODATA-VERSION: 4.0
SPClientServiceRequestDuration: 505
X-AspNet-Version: 4.0.30319
SPRequestGuid: 4367939d-b0c7-2000-0a07-34928f7d648c
request-id: 4367939d-b0c7-2000-0a07-34928f7d648c
Strict-Transport-Security: max-age=31536000
X-FRAME-OPTIONS: SAMEORIGIN
X-Powered-By: ASP.NET
MicrosoftSharePointTeamServices: <IP_ADDRESS>07
X-Content-Type-Options: nosniff
X-MS-InvokeApp: 1; RequireReadOnly
P3P: CP=\"ALL IND DSP COR ADM CONo CUR CUSo IVAo IVDo PSA PSD TAI TELo OUR SAMo CNT COM INT NAV ONL PHY PRE PUR UNI\"
Date: Mon, 25 Jul 2016 22:57:54 GMT

{"error":{"code":"notAllowed","message":"The feature has been disabled. Please contact your admin to get it enabled."}}

These were both requests to create sharing links for .docx files, for a same user, in the same document library.

Thanks @dluces! These will be very helpful for debugging this.

Hi @dluces, can you confirm that for both the success case and the failure case that the same type of link was requested?

@ificator No. Actually, the one that failed was for anonymous access while the one that succeeded was for organization access. I just checked a few users and all the errors seem to be when sharing for anonymous access. That might be the feature that's disabled: external sharing. Sorry, I should have checked that earlier. If that's the case, could you point me to a resource about enabling such feature so I can pass it on to my client? Thank you.

Try the following:
https://support.office.com/en-us/article/Manage-external-sharing-for-your-SharePoint-Online-environment-c8a462eb-0723-4b0b-8d0a-70feafe4be85?ui=en-US&rs=en-US&ad=US
It seems to go into pretty good detail about sharing and the ramifications of the various options.

@ificator looks like this is not an issue anymore. The client decided to make the sharing links company-wide instead of enabling external anonymous sharing. I have been seeing a different error while retrying the creation of some of these links though. The server returns Unable to determine the user associated with the provided token. App-only tokens are not supported for this scenario. I've logged it as a different issue: #415


Add mode CLI args

Hi! I would like to add the -s switch in order to capture stdout/stderr. How can I do that with your package?
Thanks in advance.

have you tried typing literally -s from the menu? 😁

it looks like this in the menu:
 -s no output capture (--capture=no)

Which menu do you mean? I run that specific function using , t t in doom-emacs.

M-x python-pytest-dispatch
see the readme?

yes i see that, but how can i modify pyhon-pytest-function to have -s included? I don't want to call that menu every time.

this packages uses transient.el, which is also used by magit. you can use transient to save settings: https://magit.vc/manual/transient/Saving-Values.html

Ah! Thank you very much.

note that the transient.el change was relatively recent, and not everything may work perfectly, so there may be some rough edges here and there… 🙈

but it worked. for my purposes it works perfectly fine.

I have a similar issue where I (in doom) have mapped running some of this package's commands to shortcuts
E.g.:
(map! :leader :desc "Test file" :n "r f" 'python-pytest-file-dwim)

However, I would prefer if python-pytest-file-dwim was called with the verbose argument. I have taken a look at transient and am too much of an Emacs noob to understand it, but if you can suggest how I might achieve this some other way, that would be very helpful!
Thanks!

this packages uses transient.el, which is also used by magit. you can use transient to save settings: https://magit.vc/manual/transient/Saving-Values.html

To add to my comment above, I'd like to do this in a way where I don't have to save transient option defaults for all other calls to python-pytest-***. So this fix wouldn't work in my case.


Uncaught ReferenceError: nu is not defined

[Enter steps to reproduce:]

Opentreeview
Expand and Collapse a Directory

Atom: 1.40.1 x64
Electron: 3.1.10
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught ReferenceError: nu is not defined
At ~\AppData\Local\atom\app-1.40.1\resources\app\static\<embedded>:14

ReferenceError: nu is not defined
    at Directory.unwatch (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:2715593)
    at Directory.collapse (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:2717742)
    at DirectoryView.collapse (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:2722664)
    at DirectoryView.toggleExpansion (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:2722099)
    at TreeView.entryClicked (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:994693)
    at HTMLDivElement.<anonymous> (~/AppData/Local/atom/app-1.40.1/resources/app/static/<embedded>:14:989140)

Commands
  2x -7:16.9.0 settings-view:open (atom-notification.fatal.icon.icon-bug.native-key-bindings.has-detail.has-close.has-stack)

Non-Core Packages
atom-alignment 0.13.0
atom-beautify 0.33.4
drag-relative-path 1.1.3
emmet 2.4.3
file-icons 2.1.35
lang-pug 1.3.0
sort-lines 0.19.0

Thanks for the report!  I'm unable to reproduce on Windows 10 with 1.40.1 - can you reproduce in safe mode?

I've already reinstalled Atom so I can't check, probably a minor bug.

Thanks for the update - we'll go ahead and close for now, but if you find steps to reproduce in safe mode, let us know and we can take a look.


js2coffee example converts to coffeescript but errors when converted back to js

If you put this javascript code into js2coffee.org and convert it to coffeescript and then put it back into the conversion it says there is an error
javascript:
App.president = Ember.Object.create({
firstName: "Barack",
lastName: "Obama",
fullName: function() {
return this.get("firstName") + " " + this.get("lastName")
}.property("firstName", "lastName")
});
Converted coffeescript:
App.president = Ember.Object.create(
firstName: "Barack"
lastName: "Obama"
fullName: ->
@get("firstName") + " " + @get("lastName")
.property("firstName", "lastName")
)
It seems like the chaining of the .property on the function isn't picking up correctly with the indent?

That's correct, but it ain't an issue. Wrap your function in parens and you'll be good to go, chaining-wise.


Terminal suggest: The . builtin should explain what it is in the description

When we show . as a command like this:

We should explain that it's an alias for source. This is a little nicety to make using the terminal a little easier

we'll hopefully get most of these descriptions from fig specs


Return didDisappear stream in NavSource.didDisappear instead of didAppear

This fixes an issue where components fire appear and disappear immediately after appearing, and nothing when disappearing.

Oh, that's an embarrassing mistake, thanks for spotting it! I'll release a new version of this package

7.7.0


Edge Replace: _to and _from are required

rm Note: The attributes _from and _to of an edge are immutable

I'd rather write: Including _from and _to is mandatory.

We are using a Apache 2 CLA for ArangoDB, which can be found here: https://www.arangodb.com/documents/cla.pdf
We have not been able to set of an electronic system yet. Can you sign, scan and email it to cla (at) arangodb.com?
;-)

You've Got Mail ;)

mille gracie.


spyder not responding after installing spyder-terminal

Description of your problem
What steps will reproduce the problem?

win10, anaconda
install spyder-terminal
not responding when open spyder, stucking in connecting to kernel

What is the expected output? What do you see instead?
Please provide any additional information below
Versions and main components

Terminal Version:0.2.3
Spyder Version: 3.2.6-py36_0
Python Version:3.6
Operating system:window10

Duplicate of #119


error: overlap is not transmuted!

Hello I installed racon via conda conda install racon
I made my SAM file using bwa.
then I run racon:
racon -t 20 joined_reads.fastq UlvaAlignContIlu.sam ulvacomp.contigs.fasta > polished_contigs.fasta
I got this error message:
error: overlap is not transmuted!
why im getting this? I took a look into this: https://github.com/isovic/racon/issues/77 but after 3-4 years idk if this still being a solution
THanks for your time :)

Hello,
can you please verify which version you are using with racon --version? Also, please paste commands which produced the .sam file.
Best regards,
Robert

I had this issue come up with 4 out of >200 assemblies that I've run through Racon. I'm using v1.4.20. The one thing that I'm noticing is that they all contain a number of smaller contigs with abnormal coverage. I removed contigs smaller than 50kb from the assembly which seems to do the trick. It'd be nice though to be able to pass these through without getting the cryptic error.


Cf runtime recovery

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/cf-runtime-recovery/docs/


flutter test hangs when you have a missing dependency

Steps to Reproduce
Take a test suite. Remove "test" from the pubspec.yaml file and run pub get. Then run flutter test
Similarly, I took my test suite but commented out the dependency to import 'package:typed_mock/typed_mock.dart'; . Then I ran flutter test
Expected:
test fails immediately because of dependency issue
Actual result:
test hangs, and keeps waiting for a long time until a timeout. gives no information about the dependency issues

Looks like a dupe of https://github.com/flutter/flutter/issues/3305 ?


[Chronicles of Darkness] Add page for Conditions

Changes / Comments
This pull request adds a new page for storing character Conditions.
I restricted this page only for Second Edition of CoD, structured it with a CSS Grid and added some short hints about how to use it.
I also decided to stay close to the design shown in the books, so I used game-line colors for condition headers.
This required some changes in the CSS, but as far as I can see, those changes made CSS a bit more clear.
Also, updated translation files with new fields (I am not sure how this should be done, when new fields are added and not the actual translations — right now I added english fields into the translation files).

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Pinging @PrimalZed as he is an active developer of this character sheet.
Just FYI

Thanks for the ping. I think I'm mostly done with the sheet for now, even though I didn't get through everything in my list.
I skimmed through the changes here and didn't notice anything alarming. I didn't try pulling it down to run it myself. The consolidation of the theme color seems like a good idea.

Hello! Your Pull Request includes changes to translation files. We will not deny your Pull Request, but we would prefer sheet contributors use Crowdin for translations. Please email Cassie at<EMAIL_ADDRESS>and she will get you an invite to our Crowdin within 48 hours. Be sure to include the languages you would like to translate, and please use Crowdin for future translation updates! Thanks for all you do!


Error while building container for carla bridge

Hi @evshary! Thank you for the nice work!
I saw your comment here and I'm currently following the instruction for humble branch of this repository.
However, while building the container for carla bridge (by typing make prepare_bridge), I faced an error below:
...
lactic/lib/x86_64-linux-gnu:/opt/ros/galactic/lib:/usr/local/nvidia/lib:/usr/local/nvidia/lib64 ./python -E -m ensurepip \
                $ensurepip --root=/ ; \
fi
Looking in links: /tmp/tmpx4c31f6u
Requirement already satisfied: setuptools in /home/mins/autoware_carla_launch/pyenv/versions/3.8.10/lib/python3.8/site-packages (56.0.0)
Requirement already satisfied: pip in /home/mins/autoware_carla_launch/pyenv/versions/3.8.10/lib/python3.8/site-packages (21.1.1)
make[1]: Leaving directory '/tmp/python-build.20230611162332.223/Python-3.8.10'
Installed Python-3.8.10 to /home/mins/autoware_carla_launch/pyenv/versions/3.8.10
./script/dependency_install.sh: line 19: cd: external/zenoh_carla_bridge/carla_agent: No such file or directory
make: *** [Makefile:16: prepare_bridge] Error 1

I want to ask you some help to solve this error..
My OS is ubuntu 20.04, and I'm planning to use carla 0.9.13, so it seems there's no violation to the prerequisite..
Thank you in advance! I'm really appreciate to your work, since I could not have any expectation to run autoware universe humble with carla.. thank you so much.

I just found the solution:
Run git submodule update --init --recursive before the preparation.
I did not know there are submodules.

Thanks for your report! I think this is order issue.
I should run git submodule update --init --recursive before installing pyenv
https://github.com/evshary/autoware_carla_launch/blob/humble/Makefile#L17

I saw your PR for this issue. Thank you for the update!


Logging fixes (double logging & function caller obfuscation)

do not double log output in debug mode
properly show calling file & lineno for such messages

Before:

Now:

Note that the success/highlight/etc are printed as DEBUG messages, but the calling file+lineno are properly being displayed for debugging purposes, instead of the logger.py file.

Does that also work for --verbose? Also would it be possible to log these messages as Info log level so it is more visible (and I think it matches "display" better).

Honestly I think we should just get rid of --verbose and "INFO" level logging... we literally never use it. Either we want full debug mode or we want output, right? Then maybe during debug mode just display success/fail/display as "INFO"?

Agreed with --verbose, but I like to have a destinction between less and more important log messages though. Maybe keep it as it doesn't hurt? But I would agree with logging the rest as info


Route for custom location models

I installed the panel for my recent project and it work great... I try to add new model using panel:createmodel and the controller using panel:createcontroller because I want to move all my model to Models/ directory. I add new link with display Posts and link Models\Post.
It work and show the amount of Post in the dashboard. but I get error when try to show list or add new post
NotFoundHttpException in RouteCollection.php line 161:

do I miss some step for a custom location models? I tried re-read the documentation but seems can't find the answer, I know there something I miss out.. hope someone can pointing it out...
sorry for bad English

Hi, I'm having the same issue. Were you able to find a solution @glasnostra ?
Wondering if @serverfire could give us a hand.
Thanks in advanced.

Right now we only  support models in their original path . you can't put them in models folder

We added that, please install from master branch and give us feedback