Without pressing done button, app weirdly focus on textfield again

If I have two textfield, and moving from one to the other without press done to hide keyboard. App tries to show keyboard when tap a button to show dialog.
I tried to use FocusScope.of(context).unfocus();, but it doesn't always solve this problem.
I understand this happens probably because app still focus on the first textfield, but I feel this is not desirable action. Is there any way to prevent this?
update:

If I check my focus with focusnode and listener, it weirdly focuses only first textfield.
If I use requestFocus with textInputAction: TextInputAction.done by tapping 'done' button, it works as normal, but with textInputAction: TextInputAction.next by tapping 'next', it suddenly doesn't work.

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

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  Future<void> _neverSatisfied(BuildContext context) async {
    FocusScope.of(context).unfocus();
    return showDialog<void>(
      context: context,
      barrierDismissible: false, // user must tap button!
      builder: (BuildContext context) {
        return AlertDialog(
          title: Text('Rewind and remember'),
          content: SingleChildScrollView(
            child: ListBody(
              children: <Widget>[
                Text('You will never be satisfied.'),
                Text('You\’re like me. I’m never satisfied.'),
              ],
            ),
          ),
          actions: <Widget>[
            FlatButton(
              child: Text('Regret'),
              onPressed: () {
                Navigator.of(context).pop();
              },
            ),
          ],
        );
      },
    );
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Container(
        margin: EdgeInsets.all(20.0),
        child: Column(
          children: <Widget>[
            TextField(
              keyboardType: TextInputType.emailAddress,
              decoration: InputDecoration(
                hintText<EMAIL_ADDRESS>                labelText: 'Email Address',
              ),
            ),
            TextField(
              obscureText: true,
              decoration: InputDecoration(
                hintText: 'Password',
                labelText: 'Password',
              ),
            ),
            Container(
              margin: EdgeInsets.only(top: 25.0),
            ),
            RaisedButton(
              child: Text('Login'),
              color: Colors.blue,
              onPressed: () => _neverSatisfied(context),
            ),
          ],
        ),
      ),
    );
  }
}

flutter doctor:
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.14.6 18G95, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.42.1)
[✓] Connected device (1 available)

Can you try this on master?  I don't think I can reproduce this with the latest version.

@justinmc I tried with master, but It doesn't seem to resolve the problem.
flutter doctor:
[✓] Flutter (Channel master, v1.13.3-pre.11, on Mac OS X 10.14.6 18G95, locale ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.43.2)
[✓] Connected device (4 available)

I'm sorry. I didn't properly upgrade my version. I tried again, and it seems work without problem. Thanks @justinmc

Ah great, glad it's working!


pip release: ci autoupdate with latest master source

Also add documentation to pypi page

PyPi releases automatically when release is made since #151

And PyPi documentation can be made pretty as described here: https://packaging.python.org/guides/making-a-pypi-friendly-readme/ -- will add on next release.


Linking errors when building on windows

Description
@hawkinsp Follow the fix in https://github.com/google/jax/issues/14369#issuecomment-1429839230
There will still be linking error as in https://github.com/google/jax/issues/14369#issuecomment-1428121867

If all those fixed, you will get another link error as follows:
LINK : warning LNK4044: unrecognized option \'/lm\'; ignored
ffi.lib(ffi.obj) : error LNK2005: "struct XLA_FFI_Stream * __cdecl xla::runtime::ffi::GetXlaFfiStream(class xla::runtime::PtrMapByType<class xla::runtime::CustomCall,16> const *,class xla::runtime::DiagnosticEngine const *)" (?GetXlaFfiStream@ffi@runtime@xla@@YAPEAUXLA_FFI_Stream@@PEBV?$PtrMapByType@VCustomCall@runtime@xla@@$0BA@@23@PEBVDiagnosticEngine@23@@Z) already defined in executable.lib(executable.obj)
   Creating library bazel-out/x64_windows-opt/bin/external/org_tensorflow/tensorflow/compiler/xla/python/xla_extension.so.if.lib and object bazel-out/x64_windows-opt/bin/external/org_tensorflow/tensorflow/compiler/xla/python/xla_extension.so.if.exp
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'gpu_executable.lib(gpu_executable.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'send_recv.lib(send_recv.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'llvm_gpu_backend.lib(gpu_backend_lib.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'transpose.lib(transpose.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'bfc_allocator.lib(bfc_allocator.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'nvptx_compiler_impl.lib(nvptx_compiler.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'gpu_compiler.lib(gpu_compiler.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'cpu_runtime.lib(cpu_runtime.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'gpu_helpers.lib(gpu_helpers.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'pjrt_stream_executor_client.lib(pjrt_stream_executor_client.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'local_device_state.lib(local_device_state.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'profiler.lib(profiler.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'outfeed_receiver.lib(outfeed_receiver.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'py_client.lib(py_values.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'tfrt_cpu_pjrt_client.lib(tfrt_cpu_pjrt_client.obj)\'
LINK : warning LNK4217: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'allocator_registry_impl.lo.lib(cpu_allocator_impl.obj)\' in function \'"public: static void __cdecl tsl::profiler::TraceMe::InstantActivity<class <lambda_29f743e77e718fe99c3f5b22e598e942>,1>(class <lambda_29f743e77e718fe99c3f5b22e598e942> &&,int)" (??$InstantActivity@V<lambda_29f743e77e718fe99c3f5b22e598e942>@@$00@TraceMe@profiler@tsl@@SAX$$QEAV<lambda_29f743e77e718fe99c3f5b22e598e942>@@H@Z)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'pmap_lib.lib(pmap_lib.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'pjit.lib(pjit.obj)\'
LINK : warning LNK4286: symbol \'?g_trace_level@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_trace_level)\' defined in \'traceme_recorder_impl.lo.lib(traceme_recorder.obj)\' is imported by \'jax_jit.lib(jax_jit.obj)\'
LINK : warning LNK4217: symbol \'?g_annotation_enabled@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_annotation_enabled)\' defined in \'annotation_stack_impl.lo.lib(annotation_stack.obj)\' is imported by \'gpu_executable.lib(gpu_executable.obj)\' in function \'"public: __cdecl tsl::profiler::ScopedAnnotationT<0>::ScopedAnnotationT<0><class <lambda_aeb2b8c334a04b454d1eb165a0a6ffbd> >(class <lambda_aeb2b8c334a04b454d1eb165a0a6ffbd>)" (??$?0V<lambda_aeb2b8c334a04b454d1eb165a0a6ffbd>@@@?$ScopedAnnotationT@$0A@@profiler@tsl@@QEAA@V<lambda_aeb2b8c334a04b454d1eb165a0a6ffbd>@@@Z)\'
LINK : warning LNK4286: symbol \'?g_annotation_enabled@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_annotation_enabled)\' defined in \'annotation_stack_impl.lo.lib(annotation_stack.obj)\' is imported by \'gpu_executable.lib(sequential_thunk.obj)\'
LINK : warning LNK4286: symbol \'?g_annotation_enabled@internal@profiler@tsl@@3U?$atomic@H@std@@A (struct std::atomic<int> tsl::profiler::internal::g_annotation_enabled)\' defined in \'annotation_stack_impl.lo.lib(annotation_stack.obj)\' is imported by \'tracing.lib(tracing.obj)\'
bazel-out\\x64_windows-opt\\bin\\external\\org_tensorflow\\tensorflow\\compiler\\xla\\python\\xla_extension.so : fatal error LNK1169: one or more multiply defined symbols found

Cause of multiply defined tsl::profiler::internal::g_annotation_enabled
TF_COMPILE_LIBRARY is not properly defined
https://github.com/tensorflow/tensorflow/blob/959d1b144dc03fbda586f8a60dd4c117025e6c18/tensorflow/tsl/platform/macros.h#L61-L69
which if further caused by
parameter is_external of macro get_win_copts is not properly propagated in, in https://github.com/tensorflow/tensorflow/blob/959d1b144dc03fbda586f8a60dd4c117025e6c18/tensorflow/tensorflow.bzl
Fortunately, we can --copts=/DTF_COMPILE_LIBRARY to workaround this problem.
Cause of already defined xla::runtime::ffi::GetXlaFfiStream
xla::runtime::ffi::GetXlaFfiStream is first defined as weak symbol in ffi.cc
then defined as normal symbol in executable.cc and msvc does not support weak symbol.
As for dirty fix, removing the definition in ffi.cc and replacing it with decl will allow the linking to pass. After this, we can build a usable jaxlib whl
What jax/jaxlib version are you using?
jaxlib v0.4.3
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
No response

Seems like Triton has been changed such that the advice above is no longer valid (build will still fail for similar reasons).

Are both of these issues still current, or just the GetXlaFfiStream issue?

@ezhulenev do you have suggestions on how we might avoid the weak symbol here? MSVC apparently does not support weak symbols.

Still current. But the first one is mainly configuration issue. So only the second one need to be address ATM.

The weak symbol problem has been resolved on openxla/xla latest main.

Is this issue still a problem?
(I know it isn't an CPU, because the new Windows CPU CI build is mostly happy: https://github.com/google/jax/actions/workflows/windows_ci.yml )

The weak symbol problem has been resolved on openxla/xla latest main.

That is, the second is fixed in https://github.com/openxla/xla/commit/e634d4ab1067445c0f89f4b2bbe0bafaf0400051
The first one is still there, but as it can be workaround from outside, feel free to close this issue.


Guidelines implementation: Imports

[ ] Implement and apply rule awslint:from-signature: All fromXxx static methods should accept "scope, string[, props]" and return IFoo
[ ] Implement and apply rule awslint:from-attributes: Constructs should have at least one "fromXxx" static method that is not "fromExport", no type name prefix
[ ] Implement and apply rule awslint:from-account: The last argument of the fromAccount method should begin with QueryXxxx

Split into multiple issues


Query returns ERROR

Steps to reproduce the behavior (Required)

CREATE TABLE :
tpcds tables with 1G data;

SELECT:

select
  concat(
    cast(cast(null as VARCHAR) as VARCHAR),
    cast(ref_1.cc_city as VARCHAR),
    cast(cast(null as VARCHAR) as VARCHAR)) as c0,
  ref_2.r_reason_sk as c1
from
  (select
            ref_0.r_reason_sk as c0,
            ref_0.r_reason_sk as c1,
            ref_0.r_reason_id as c2,
            ref_0.r_reason_sk as c3,
            ref_0.r_reason_desc as c4
          from
            tpcds_1g.reason as ref_0
          where (false)
            and (ref_0.r_reason_id is NULL)) as subq_0
      right join tpcds_1g.call_center as ref_1
      on (subq_0.c0 = ref_1.cc_call_center_sk )
    left join tpcds_1g.reason as ref_2
    on (subq_0.c3 = ref_2.r_reason_sk )
where case when false then ref_1.cc_call_center_sk else ref_1.cc_call_center_sk end
     is not NULL
limit 80;

Expected behavior (Required)
Query returns the right result
Real behavior (Required)
Query ERROR:
ERROR 1064 (HY000): Dict Decode failed, Dict can't take cover all key :0
mysql> select
    ->   concat(
    ->     cast(cast(null as VARCHAR) as VARCHAR),
    ->     cast(ref_1.cc_city as VARCHAR),
    ->     cast(cast(null as VARCHAR) as VARCHAR)) as c0,
    ->   ref_2.r_reason_sk as c1
    -> from
    ->   (select
    ->             ref_0.r_reason_sk as c0,
    ->             ref_0.r_reason_sk as c1,
    ->             ref_0.r_reason_id as c2,
    ->             ref_0.r_reason_sk as c3,
    ->             ref_0.r_reason_desc as c4
    ->           from
    ->             tpcds_1g.reason as ref_0
    ->           where (false)
    ->             and (ref_0.r_reason_id is NULL)) as subq_0
    ->       right join tpcds_1g.call_center as ref_1
    ->       on (subq_0.c0 = ref_1.cc_call_center_sk )
    ->     left join tpcds_1g.reason as ref_2
    ->     on (subq_0.c3 = ref_2.r_reason_sk )
    -> where case when false then ref_1.cc_call_center_sk else ref_1.cc_call_center_sk end
    ->      is not NULL
    -> limit 80;
ERROR 1064 (HY000): Dict Decode failed, Dict can't take cover all key :0

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

mysql> select current_version();
+------------------------+
| current_version()      |
+------------------------+
| QA_TEST_MASTER 186e656 |
+------------------------+
1 row in set (0.01 sec)

the mini reproduce case:
select concat(NULL, col) from t0;


Speed up import of input files

From profiling, it seems that 67% of run is from importing input csv files. We should be able to speed this up with joblib, a library for parallellisation.

Alternatively, we could separate the running of the model and the reading in of inputs. Creating pickle files and reading the inputs from them will speed things up. More often than not, inputs do not change.

The disadvantage is that we then store the input data in three different ways on github (masterfiles, country files and pickle files). I would be open to this idea if it can get rid of the country files. Was that what you were thinking?

Probably the easiest (but ideal) win is to store the data by variable in a long format CSV file. It declutters the input folder a lot already and should speed up reading the inputs.

Do you mean we'd store one BCET, one MEWG etc? That would make the structure of the input files folders a lot easier. I don't know if it would be faster.. Pandas is apparently not great with larger files.


Allow provider to override template_name and context

Also an example provider which renders the user's initals against
a randomly colored background.

Coverage decreased (-0.9%) to 84.717% when pulling 55be600311844b470baed376572fe7b3d734e0c3 on reidransom:tmploverride into e5d488a08d89b05285ec41f53e6726aac0c603ca on grantmcconnaughey:master.


Snake Game

Want to add  Snake website, built using  HTML_CSS_JS
live demo here: https://snake-buzz.netlify.app/
@Ayush7614  please_assign me

I made this game before but it was using html & css & js now I want to rebuild it using react js can I get that chance

Go ahead @Waer1 But use react js and tell me you are participant of GSSOC22

Yes iam contributor

please is there tutorial to upload the done work ?


1051 rh UI re work first steps

Motivation and Context

First steps for creating the new rh-ui project.
What has changed

I've created the basic project structure, added a basic Flask app, with a Hello World page and some simple tests.
I've added the Dockerfile, set up logging, flake8 and Vulture.
How to test?

Check that all of the works and makes sense.
Check that the repo settings match our guidelines.
Check that the make commands work and make sense.
Links

https://trello.com/c/Tj9FLkF3/1051-rh-ui-re-work-first-steps-8
Screenshots (if appropriate):

Work in progress: I'm trying to set up the GH Actions secrets.


Replicate AI soldier movement more similar to how UE replicates crowd ZoneGraph movement

To avoid desync issues. See https://docs.google.com/document/d/1LhRWV-ZD_Cs3BzALpFKryQ4w6fyLARMli-4w1Ys7ktg/edit#heading=h.l6nqayknhnge for more.

Fixed in https://github.com/HaywireInteractive/OnAllFronts/commit/c246f33878cecc16faa3fc0dac983e50937e52f2


Is it possible to jump to the target?

Is it possible to jump to the target location when opening a file?
For example, if the header matches a file, click to jump to the header automatically

Yes, just like the jump function by Quick switcher++.

This issue will be closed for now, as there have been no comments from visitors for a month.

https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/pull/142 will resolve this issue.

@jerviscui
Released in 8.5.0 🚀


[threadpool] Add new port

Describe the pull request
This adds a new port for ThreadPool.

What does your PR fix?
Nothing.

Which triplets are supported/not supported? Have you updated the CI baseline?
Any triplets should be supported. I didn't update the CI baseline.

Does your PR follow the maintainer guide?
Yes.

LGTM, thanks for this PR!

/azp run

This seems to be a fork of the original repo. Can we use the original repo?

@dan-shaw: We could but it's more outdated and no longer maintained.

Ping @dan-shaw, is there any other review information?

It's still strange to ship a fork as if it were the parent. Perhaps you could add a new port with the fork under a different name?

Thanks for the PR!
I'm closing this for inactivity. If you'd like to make a new PR, we'd be happy to review it.


Queue Tools: Grouping comments by submission

Ref: #799
Starting this now because it's a thing that works, though there's a lot that could change with this depending on where we want to take it.
To discuss:

Keep option in settings, or put it on the page?
Style the groups? If not, we should ditch the wrappers.
If the queue is sorted by age descending (new at the top), the submission will always be at the bottom of its own list. Should we pull it up to the top and then have the comments within keep their existing order?
Currently the groups are sorted according to the thing that comes first in the group. Is that okay?
If a link isn't reported but several of its comments are, should the link preview be generated and put in for context?

Just as a note to everyone don't merge this yet. I want to release 3.6 first before we merge this.

I still need to check it out, but my thoughts so far on your questions.
Keep option in settings, or put it on the page?
Hrm, settings for permanent and possibly a button to do it on the fly?
Style the groups? If not, we should ditch the wrappers.
Probably, I'll need to check out how it looks now.
If the queue is sorted by age descending (new at the top), the submission will always be at the bottom of its own list. Should we pull it up to the top and then have the comments within keep their existing order?
That might be a good idea.
Currently the groups are sorted according to the thing that comes first in the group. Is that okay?
Not sure I understand this question? You mean that if you have this list

comment 1, thread 1
comment 2, thread 2
comment 3 thread 1

That it will first show the group for thread 1?  I think that is okay, I'll think it over for a bit and also play with it as well.
If a link isn't reported but several of its comments are, should the link preview be generated and put in for context?
Hrm... Not sure if it is really needed but it might provide some more context and clearly defines the groups.

Not sure I understand this question?

 NER is a thing ;)
Should be fairly straightforward though, you can probably make it just append to the already existing groups. NER seems tricky at first but once you get the hang of it you can make it work rather easily.
Probably adjust the current function to add a class "tb-group-processed" once it has processed the ".thing" and at the beginning of the function do a if (!$thing.hasClass('tb-group-processed')) {}

Yeah that sounds doable. Cool.

One more thing currently not working is sorting, when you resort the grouping gets undone.
A possible solution would be a "reset" function that undoes the wrappers and then call the group function again after sorting is done.

I also think that instead of adding a horizontal line as physical element border work better. Something like
.tb-comment-group {
    border: solid 3px #d2d2d2;
    margin-bottom: 5px;
    margin-right: 310px;
    padding-top: 5px;
}

Note that the right margin probably should be determined on the fly just as with .menuarea.modtools because of subreddit css and sidebars.

Er, dynamic sorting is a thing? I thought that involved a page reload
anyway... probably remembering the module wrong. Can't test anything rn but
that sounds like something I could do.
And yes, the plan was to move any actual styles to CSS instead of using
s in the future. I just added that as part of the mockup. (Pretty sure
it'd be padding-bottom, not padding-top, though.)
Honestly I think we'd be fine if we didn't even bother trying to figure out
what margin-right should be; if the sidebar is floated, the width of the
separator should adjust appropriately just like the links themselves cut
off, and if not, then it will just follow the same flow as the
links/comments themselves. (I think? It's actually been a while since I
worked with a floating sidebar design like that in-depth, I reserve the
right to be completely wrong :V )

Er, dynamic sorting is a thing?

Yessir.

Pretty sure it'd be padding-bottom, not padding-top, though.

Na padding top in this case because the first element is otherwise clinging to the top border. Just aesthetic kind of thing :) There are probably a few other methods we could try and style it, but I figured that since it is group I would do a border around the entire thing.

Honestly I think we'd be fine if we didn't even bother trying to figure out
what margin-right should be; if the sidebar is floated, the width of the
separator should adjust appropriately just like the links themselves cut
off, and if not, then it will just follow the same flow as the
links/comments themselves.

Nah that looks all ugly and you have to account for all sorts of things, besides the code to determine what the margin needs to be is already in there ;)

otherwise clinging to the top border.

Doesn't the margin-bottom handle that though...?
http://codepen.io/Geo1088/pen/LxMoBb

that looks all ugly and you have to account for all sorts of things

Fair enough, but will it be easy to only apply it to the area where the
sidebar restricts the content width? Otherwise the wrappers wouldn't let
their content flow underneath the sidebar on subs that allow that to happen.
I'll have to try it out with some other styles and see how things look.

The clinging part is the inner side of the wrapper. I should have made a screenshot I suppose.
Sidebar always restricts content width because it otherwise will flow under the sidebar obscuring the right border.

Sidebar always restricts content width because it otherwise will flow under the sidebar obscuring the right border.

I'm not even sure we're talking about the same thing? Content is supposed to flow underneath the sidebar unless the sub sets it to not.

I shall screenshot it :)

This branch will probably go stale now that I won't be able to actively work on toolbox for a while. If you guys want to add to this, you should have push access to this branch, but it'll require you cloning my copy of the repo and making changes from there.

Where are we on this, now that 3.6 is out?

I'll pick up work on this I guess, it is still in alpha stage in some regards.

Okay, I am merging this in so we can work on it now that 3.6 is out and I hotfixes have been done.


Attempt to set Date: Operation not Permitted

The model files are downloaded correctly, but show up as failed in the scraper. After setting the "debug" option the following error is thrown after every file is downloaded:
2023-08-20 06:15:52 - DEBUG - Media:2984954732 Post:704208492 Attempt to set Date to 2023-08-18 23:13
2023-08-20 06:15:52 - DEBUG - Media:2984954732 Post:704208492 [attempt 1/10] exception [Errno 1] Operation not permitted
2023-08-20 06:15:52 - DEBUG - Media:2984954732 Post:704208492 [attempt 1/10] exception Traceback (most recent call last):
File "/home/your_username/app/your_username/utils/download.py", line 366, in download
return await main_download_helper(c,ele,pathlib.Path(".").absolute(),username,model_id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/your_username/app/your_username/utils/download.py", line 405, in main_download_helper
set_time(path_to_file,newDate )
File "/home/your_username/app/your_username/utils/download.py", line 843, in set_time
pathlib.os.utime(path, (timestamp, timestamp))
PermissionError: [Errno 1] Operation not permitted
2023-08-20 06:15:52 - DEBUG - PID: 81 download process thread closing
Unfortunately, this means the scraper cannot mark the file as downloaded in the database and has to download all the files every time. Is there a fix for this? The script is running in Docker with the files on a mounted volume.

Can you try to pass in the owner of the folder
Are you on linux by chance

Yes, now running version 3.0.12+main.bfe776e as Docker container with unraid as host. Tried setting User ID 99 in Docker compose but getting error :

The files are downloaded, but the "Operation not permitted" error persists.

can I see your compose file

Sure, here is my current compose file that throws the mentioned "Operation not permitted" error:
version: '3'
volumes:
bin:
config:
data:
services:
ofscraper:
container_name: ofscraper
image: datawhores/of-scraper:main
# build:
#   context: .
#   dockerfile: Dockerfile
user: "${UID:-1000}:${GID:-1000}"
volumes:
# #change the host mountpoint
- /mnt/user/appdata/ofscraper:/home/ofscraper/.config/ofscraper
- /mnt/remotes/SpecialDownloads/onlyfans_script:/of_media
stdin_open: true # docker run -i
tty: true        # docker run -t
# use the following two lines if you want to interact with the script manually.
# bring the container up with docker compose up -d and then run it with
# docker compose exec -it ofscraper ofscraper --config /ofscraper/config/config.json
entrypoint: ["/bin/bash"]
command: ["-c","sleep infinity"]
# alternatively, if you want the script to run automatically with a set or arguments,
# comment out the lines above and instead do something like:
# command: "ofscraper --username ALL --posts all"

Afaik in unraid 99 is user "nobody" with group 100. This is default for most dockers and can be configured per-docker.
In ofscraper version 2.x the permission error did not occur.
Maybe this helps?
https://www.reddit.com/r/unRAID/comments/10lbudb/problem_with_puid_and_pgid_after_update_on/
Thanks for helping out!

I think someone had a similar error
or I'm not sure if your the same person
In either case please come to the discord
Closing issue


feat/HttpCacheStoragePaged

Status
IN DEVELOPMENT
Breaking Changes
NO
Description
Add new widget to manage paged cache

[x] initialize storage
[ ] create widget

Type of Change

New feature

Codecov Report
Base: 97.20% // Head: 96.36% // Decreases project coverage by -0.84% :warning:

Coverage data is based on head (f8c713a) compared to base (0fbfd00).
Patch coverage: 71.42% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   97.20%   96.36%   -0.85%
==========================================
  Files           9        9
  Lines         215      220       +5
==========================================
+ Hits          209      212       +3
- Misses          6        8       +2

Impacted Files
Coverage Δ

lib/src/http_cache.dart
93.79% <60.00%> (-1.37%)
:arrow_down:

lib/src/http_cache_storage.dart
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Cannot record video and stream simultaneously

I am not being able to record and stream video simultaneously.
When i send a record command, the video freezes. It unfreezes when i turn the recording off.
Also, is there a command to stop recording the video?

no.

gpCam.shutter(constants.stop)


move blog posts to top level

Deploy preview for gatsby-blog-starter-kit ready!
Built with commit 058991266c613b98768c5b299984e96b4a3a5730
https://deploy-preview-22--gatsby-blog-starter-kit.netlify.com


Changing CHIANTI files for thermal model?

Describe the feature
The CHIANTI files used for the thermal model are v7.1 sav files, i.e. sunxspex/sunxspex/thermal.py, whereas the OSPEX model has been using V9.0.1 geny files for a while. The latter files does change the lines a bit, i.e comparison model spectra but given the energy range and resolution of some of the spectra fitted by sunxspex it might not make a huge difference. The change to the current CHIANTI v10.0.2 is more minor update to v9.0.1.
Note also that the soon to be released CHIANTI 10.2 is changing the default coronal abundances and moving the older abundance files. That might cause problems in the generation of the files in ssw, and possibly the running of ospex/fvth model in ssw (not a sunxspex problem). But longterm would be good to have latest CHIANTI and updated abundances (possibly as the default?). Both sunxspex and ospex have these CHIANTI files saved with unity abundance so it can be chosen at time of model call.
Proposed solution
I can try and generate v10.0.2 files in the same format as the current v7.1 ones in ssw, and see how they change the model in sunxspex itself.
Once v10.2 is out can explore what want to do with those files - will likely impact OSPEX fvth model, so might want to replicate what it does.

Got the generation working and created an example using the thermal model in sunxspex with default v7.1 and new v10.0.2 files, i.e.
https://github.com/ianan/fvth_stuff/blob/main/python/test_thermal_sav.ipynb
For resolution of instruments using (just sub-keV) at worst ~20% different at some lines depending on T.
Note that new v10.0.2 file generated using fewer temperature bins (41 instead of 300 or 750) and over narrower range (logT 6-8 instead of 6-9) as otherwise line file would be too big (100s MB instead of 10s MB).

@ianan can you make a pull request that updates the .sav file download in the code?

@settwi Can do the PR but tried to use the new files for fitting and sunxspex crashes with IndexError: index 84 is out of bounds for axis 0 with size 84 in sunxspex/thermal.py looks like how the lines are weighted and added to the continuum, line 677. Something no doubt to do with the different temperature binning (and maybe more lines).
Example of thermal using existing v7.1 is here: https://github.com/ianan/fvth_stuff/blob/main/python/test_v7_fit.ipynb
Example of thermal using my v10 (which breaks) is here: https://github.com/ianan/fvth_stuff/blob/main/python/test_v10_fit.ipynb

I'll take a peek @ianan.
Maybe @DanRyanIrish can provide some support too. I think he wrote the thermal emission code.
-WS

Ok, dumb question: how do you get that pair of .sav files?

@ianan @DanRyanIrish
Hacked fix to get fit working
Change Line 4753 in sunxspex.sunxspex_fitting.fitter.py from _test_e_range = np.arange(1.6, 5.01, 0.04)[:, None] to _test_e_range = np.arange(1.6, 15.01, 0.04)[:, None].
Details
This appears to be a bug in the thermal code that is there without changing the CHIANTI files as well.
Running the thermal model with too few energy bins will cause this index error for some reason. I found this out a while back, I can't remember if this was ever brought up in the past, effectively I try to test that a given model is usable by running a quick test. I run the model with parameter values of 1 and energies of _test_e_range = np.arange(1.6, 5.01, 0.04)[:, None] where I pass np.concatenate((_test_e_range[:-1], _test_e_range[1:]), axis=1) to the model function (see these lines).
This isn't great, I know, but it's there as a check for now and can be dealt with in the re-factor. Anyway, the energy range was chosen to mimic NuSTAR energy bin resolution and starting at 1.6 keV but had to be up to 5.01 keV otherwise the same index error that Iain is seeing now would occur. I.e., even if I just changed the definition of _test_e_range to go to 5.00 keV then the index error would result.
I also managed to produce the same error with the new files and sunxspex.thermal.thermal_emission by changing the engs definition in Iain's Notebook (cell three) from engs=np.arange(1.01,21,0.01) to engs=np.arange(1.01,5,0.01). So it seems like the index issue is in the thermal model and not the fitting where the lowest number of bins it can handle is tied to the grid, I am unsure how though.

Can confirm @KriSun95 fix works - but as discussed might be other issues with f_vth that need to think about (i.e. max energy range shouldn't be limited by continuum file? But min energy should be limited by the lines/continuum files?).
As expected though the fit results for v7 vs v10 are very similar due to the energy resolution of the data we are working with.
Also, @settwi if you haven't found it already the sswidl code to produce the sav files is in the same repo but here https://github.com/ianan/fvth_stuff/blob/main/idl/make_new_ch_files_sunxspex.pro

@KriSun95 Just to check but the temporary bug fix for this hasn't been implemented yet? As L4756  is still _test_e_range = np.arange(1.6, 5.01, 0.04)[:, None]

@ianan, you are correct and apologies this quick fix has been held up. I've now opened a PR with this fix applied in #134.

fyi, I have better versions of the chianti database files to use when fitting the thermal model (either in ospex or sunkit-spex) available here https://github.com/ianan/fvth_stuff/tree/main/better_chxdb The issue is that the default sunkit-spex one uses the old v7 files (missing stuff) and the newer v9/v10 ones (v9 default in ospex) don't have enough temperature bins so can get "clustering" in the T results. These new ones are v10.1 and have enough temperature bins to avoid issues (I think?), and aren't too large a file.


Multi query alertmanager or grafana

Hello. I have a openshift cluster monitoring. All is OK.
I need a send metrics of CPU, memory when openshift POD is restarts. Help me please)

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


model drawer redone

Refactor ModelItem component to add modelName to card
add sortby function to model drawer
add fuzzy search with file name and model name

this closes https://github.com/11cafe/Internal_Roadmap/issues/6

嗯嗯好的 那我明白了～先这么用吧👌

From: Arslan Ablikim @.>
Sent: Tuesday, March 5, 2024 9:40:53 PM
To: 11cafe/comfyui-workspace-manager @.>
Cc: Weixuan Fu @.>; State change @.>
Subject: Re: [11cafe/comfyui-workspace-manager] model drawer redone (PR #235)
@arslan2012 commented on this pull request.

In ui/src/model-manager/models-list-drawer/ModelsListDrawer.tsxhttps://github.com/11cafe/comfyui-workspace-manager/pull/235#discussion_r1512842865:

     db: models.find(

       (model) => model.id === item.model_name + "@" + item.model_type,

     ),

嗯，对，我是这么想的，其实我还想的可以map一下id然后用bulkGet函数，但是get, bulkGet, toArray那个更快我也没找到相关的信息
—
Reply to this email directly, view it on GitHubhttps://github.com/11cafe/comfyui-workspace-manager/pull/235#discussion_r1512842865, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEMEEOOIT2FBDV6PVYXD6M3YWXDOLAVCNFSM6AAAAABEE6MNQ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSMJWHE3TENZZG4.
You are receiving this because you modified the open/close state.Message ID: @.***>


cacheFirst appears to fetch when the response is already cached

I've been examining the Network calls for an app using Workbox Service Workers with Webpack to cache static images. After activating the SW and visiting the page, I can confirm that the cache is created and the assets are in the cache. However, when revisiting the page, the network tab shows two calls for the image: an image returned from the SW (expected) and a fetch event from the disk cache originating from Workbox (and suggesting that the SW is making a request, which it shouldn't be).

What is the second unexpected fetch event and why is it being used in a cacheFirst strategy? I would expect just to see a single event for the image being returned from the SW.
If I remove the expiration constraints, this issue is resolved. Why is that?

Sample code below. Thanks!
Webpack config:
            new GenerateSW({
                skipWaiting: true,
                clientsClaim: true,
                include: [],
                runtimeCaching: [
                    {
                        urlPattern: /^https:\/\/.*\.cloudfront\.net\/.*/,
                        handler: 'cacheFirst',
                        options: {
                            cacheName: 'cloudfront',
                            cacheableResponse: {
                                statuses: [0, 200],
                            },
                            expiration: {
                                maxEntries: 100,
                                maxAgeSeconds: 600,
                            },
                        },
                    },
                ],
            }),

Generated:
workbox.routing.registerRoute(/^https:\/\/.*\.cloudfront\.net\/.*/, workbox.strategies.cacheFirst({
    cacheName: "cloudfront",
    plugins: [new workbox.cacheableResponse.Plugin({
        "statuses": [0, 200]
    }), new workbox.expiration.Plugin({
        "maxEntries": 100,
        "maxAgeSeconds": 600
    })]
}), 'GET');

This should have been fixed by #1422, which was deployed as part of the 3.1.0 release.
Please reopen if you're still having trouble after upgrading to 3.1.0!


Update all dependencies

WHAT
WHY
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

Landing with follow up
https://github.com/google/horologist/issues/2399


Add Confirmation Dialog when Adding Duplicate Combo

It would be good to have a confirmation dialog pop up when an editor tries to add a combo that is a duplicate of a pre-existing combo. I was thinking of something similar to the dialog that pops up when you delete a combo. A message like "The combo you are trying to add is a duplicate of combo [ID]. Would you still like to add this combo?" and then a yes/no answer. Yes adds the combo as normal, no takes you back to the "Add Combo" screen.
I think this would be good to have to ward out accidental duplicates before they are added and then (likely) need deleted. If we do need to have duplicates for some reason, we can still do so.

Today I tried working on this request without exploiting third party software, and I concluded that it costs too much dev efforts and time to get it done well. Django, the framework powering the backend, doesn't allow easy customization to implement a confirmation page that conditionally pops out based on related objects such as cards, templates and features in a combo, which need to be inspected in order to detect any duplicate.


Add process_exit(code) function

process_exit(1); // Exits the locks runtime with exit code

https://tldp.org/LDP/abs/html/exitcodes.html#EXITCODESREF


Refactor and zero-copy pervasively

We are interested in using this crate for wasm-bindgen. In investigating some issues that we are having, I decided to do a bit of refactoring. Note, this is a breaking change to the API (see below), but it looks like there are currently no public downstream crates, I don't believe this is a real issue.
Breaking changes:

The AST now borrows from the input string, it is no longer owned.
*I64 has been renamed to *Lit since these literals could contain non-valid i64 strings (e.g. 2^63).
A new inner type FloatValueLit was added for FloatLit::Value. This was to be a more consistent with other enums which didn't do any non-trivial parsing.
Identifier and ExtendedAttributeNoArgs are now tuple types. Again, this was done to be more consistent with other newtypes, namely literals.
UnionSingleType was renamed to NonAnyType and is now also used in SingleType.

The main other notable change is that I switched to using regex finds rather than captures. The captures weren't really needed, and they were causing the debug builds to be really slow.

This looks great. From your pull request, it was a great deal of learning for me. Thank you for putting your time to create this awesome pull request.
If you ever decide to actually incorporate this to wasm-bindgen, I would be happy to transfer the ownership to you/group (as I don't think I'll be having a go on this project for the foreseeable future).


Fix spelling gap in EventRegister.md

Removed whitespace on line #180

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Filter devices by serials, skip dialog if only one device is found

Teach the bcf tool to filter the list of discovered serial devices by
their serial strings. Devices whose serial strings do not start with
"bc-" will be silently ignored. A typical host will have a whole bunch
of serial ports and there is no need to present all of those to the
user.
If the filtered device list ends up with only one matching device, use
the device right away. There is no need to present the selection dialog
to the user in this case. This makes the bcf tool easier to use on
hosts where the name of the character device may change over time. The
bcf tool will always pick the only available BigClown device in this
case.

This PR introduced a bug. Will submit an updated version.


Remove or untight the npm package name validation

Describe the feature or problem you’d like to solve
Currently, the lstn to <npm_package_name> command validates the input npm package name.
To do so, we use a little utility Go library I wrote, go-npmpackagename.
This means that invalid npm package names never reach our verdicts API.
The problem is that on the npm registry there are packages whose name is considered invalid as of today. An example is the events package (more than 32M downloads/week).
Proposed solution
Barely we have two options here:

remove the npm package name validation totally
only block npm package names that were never valid npm names

Additional context
The go-npmpackagename can distinguish both currently invalid names, and names that were never valid ones.

An image is worth a thousand words.
I have a branch with (2) implemented. As you can see this way, the users can finally ask for packages like the events one (no more valid names, that were valid previously on the npm registry).
But, they cannot ask for names that were never valid.


LIME-718 updating Gov UK account to One Login

Proposed changes
What changed
References to UK Gov Account team have been updated to UK Gov One Login team.
Why did it change
To align with onboarding to One Login.
Issue tracking

LIME-718

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

[ ] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

May need to update the welsh translation tests as a part of this PR aswell @gds-adhenderson

Also should the error page not also be included in this?


Fix tests in Linux.

The directory listing order is different from OS X and so the tests are run in a slightly different order.  Other tests were leaving data bound to the "body" element, and this data was then propagated when new elements were appended, and thus messed up the data propagation test.
You can probably leave out the html("") changes as they were a red herring, but I don't think they do any harm.  Maybe we need a reusable "setup" function that clears the body element of any data and elements.

This fixes #329.

Excellent!!


inplace QR

I have incorporated the in-place QR add/del operations in bpdual. However, I suspect the CI tests will most likely fail because the updates to QRupdate.jl have not yet been released as a new version. @mpf would it be possible for you to create a new release of QRupdate? Thanks.

I like the idea very much. Can you run benchmarks that compare the old and the new version?

I just encountered a bug in QRupdate.jl and filed an issue there. I have a somewhat quick fix for it but it makes it a bit less memory efficient(still better than the original dynamic qrupdate). I can open up a PR there and fix it but I just emailed Nicolás Barnafi(the person who wrote the in-place qrupdate code) and he told me he's working on an updated implementation of this and it might fix the issue + keep it memory efficient. So I'm waiting on that.

@tinatorabi — I see that the unit tests are still failing with the latest commit. Please let me know if I can help.

Hi!
With this PR I wanted to incorporate the in-place qr operations into ASP but apparently there are some on-going issues in QRupdate that Nicolás is hoping to fix soon. Let's Ignore this PR for now until that's fixed. I reverted BPdual to the way it was before but added tests for OMP.

Hi! Now that the new QRupdate version is merged, everything is working just as we wanted and all the CIs just passed! I’ve updated both BPdual and omp to use in-place QR operations. :) @mpf, I think you can go ahead and merge this!


I'm updated my react native version to 0.73.4 and getInitialNotification() and onNotificationOpenedApp() does not work in iOS

I have updated my React Native version to 0.73.4. In Android, notifications are working fine in the foreground, background, and quiet states.
In iOS, foreground notifications are working, and when the user clicks on them, they navigate to the respective screen.
However, in the background and quiet states, notifications are coming through, but when the user clicks on them, neither getInitialNotification() nor onNotificationOpenedApp() is called, and no remoteMessage is received.
But with "@notifee/react-native": "5.7.0", , getInitialNotification() and  onNotificationOpenedApp() are working.
package.json
 "react": "18.2.0",
 "react-native": "0.73.4",
 "@notifee/react-native": "^7.8.2",
 "@react-native-firebase/app": "^19.1.1",
 "@react-native-firebase/messaging": "^19.1.1",

Same here
Can't get initial notification on press

Same here Can't get initial notification on press

check the release note of version 7.0.0 and i installed version "@notifee/react-native": "5.7.0",

Unfortunately, it's expected behavior https://github.com/invertase/notifee/blob/main/docs-react-native/react-native/docs/release-notes.md#600

getInitialNotification is deprecated on iOS in favour of the PRESS event received by the onForegroundEvent event handler
https://notifee.app/react-native/docs/events#app-open-events

@DeepakSharma04 separately, does triggering EventType.DISMISSED for onForegroundEvent() work for you when swiping up on the notification?


[Update control.py]: Add step_limit property

Could you explain why you want to get the step_limit? All the tasks in the control suite apart from lqr have a fixed episode length that defaults to 1000 steps.

I think I agree with you, except for lqr, all environment has fixed max episode length 1000. So perhaps a point is if there'd be near future plan to support additional standardized environments that this length might be varied. Also, since _step_limit is already defined in the class as private member, it might be good to allow the public access to the users. Although I'd agree that it's a bit API creeping according to current status.

Sorry for the long delay - for some reason I thought I'd already responded to this PR!
Overall I don't think the step_limit is something we should expose in control.Task, although of course feel free to subclass control.Task in your own code and do whatever you like with it :-). In the context of the control suite providing the step limit seems like redundant information, since all but one of the control suite tasks share the same limit of 1000 steps (and for the LQR task there is no upper bound on the length of an episode, so step_limit isn't meaningful in that case).  There's also a bug in your implementation - int(self._step_limit) would result in OverflowError in the default case where the step limit is Inf.


Build failure with GHC 9.2.1

Building library for atom-1.0.13..
[ 1 of 17] Compiling Language.Atom.Expressions ( Language/Atom/Expressions.hs, dist/build/Language/Atom/Expressions.o, dist/build/Language/Atom/Expressions.dyn_o )

Language/Atom/Expressions.hs:643:9: error:
    Ambiguous occurrence ‘And’
    It could refer to
       either ‘Data.Bits.And’,
              imported from ‘Data.Bits’ at Language/Atom/Expressions.hs:93:1-16
           or ‘Language.Atom.Expressions.And’,
              defined at Language/Atom/Expressions.hs:214:3
    |
643 | (&&.) = And
    |         ^^^

Language/Atom/Expressions.hs:781:3: error:
    Ambiguous occurrence ‘And’
    It could refer to
       either ‘Data.Bits.And’,
              imported from ‘Data.Bits’ at Language/Atom/Expressions.hs:93:1-16
           or ‘Language.Atom.Expressions.And’,
              defined at Language/Atom/Expressions.hs:214:3
    |
781 |   And      a b   -> uand      (ue a) (ue b)
    |   ^^^

v1.0.12 produces the same error.
As a Hackage trustee I have revised both versions to prevent users from running into these errors:
https://hackage.haskell.org/package/atom-1.0.13/revisions/

I've also revised v0.0.1, v0.0.2, v0.0.3 and v0.0.4 due to the same error.


Detect AWS temporary credentials

Hello!
The AWS access key currently only detects static credentials of IAM users.
It would be relevant to also detect AWS STS temporary credentials:

The access key ID matches ASIA[0-9A-Z]{16}
The secret key matches the same format as for static credentials
The session token

Thoughts? Interested by a PR?

Moved to https://github.com/IBM/detect-secrets/issues/34


Is discount factor wrongly implemented?

Hello, I have a question about the decay factor (discount factor) in N-step DQN (paper & implementation)
I found the following equation in the paper(page 6; first paragraph):

As far as I know, discount factor should be applied as following:

I checked the implementation of tsp2d(code/s2v_tsp2d/tsp2d_lib/src/tsp2d_lib.cpp), and in the Fit() function, former version of equation is implemented.
double Fit(const double lr)
{
    NStepReplayMem::Sampling(cfg::batch_size, sample);
    bool ness = false;
    for (int i = 0; i < cfg::batch_size; ++i)
        if (!sample.list_term[i])
        {
            ness = true;
            break;
        }
    if (ness)
        PredictWithSnapshot(sample.g_list, sample.list_s_primes, list_pred);

    list_target.resize(cfg::batch_size);
    for (int i = 0; i < cfg::batch_size; ++i)
    {
        double q_rhs = 0;
        if (!sample.list_term[i])
            q_rhs = cfg::decay * max(sample.g_list[i]->num_nodes, list_pred[i]->data());
        q_rhs += sample.list_rt[i];
        list_target[i] = q_rhs;
    }

    return Fit(lr, sample.g_list, sample.list_st, sample.list_at, list_target);
}

I'm currently studying the DRL, so I might be wrong... But since I couldn't also found the reason why you used discount factor in that way, I write this issue.
Thank you for providing nice implementation! I've really learned a lot from it!

Hi,
you are correct about the equation. But in our paper we mostly use one-step unrolling of bellman equation, and set the discounting factor to be 1. So it doesn't really matter.


[GitLab] Update branch accessors

Deprecated branch_for_merge in favour of branch_for_base and
add branch_for_head.

👍 looks good to me


cleanup 3rd-party assets

Look at docs https://shopify-vite.barrelny.com/guide/troubleshooting.html#how-to-cleanup-the-assets-folder
You mention this plugin https://www.npmjs.com/package/@by-association-only/vite-plugin-shopify-clean
But it doesn't support vite 5. And it seems like author is not going to do anything with it
May be this plugin would be better for this purpose
https://github.com/knjshimi/shopify-vite/tree/main/packages/vite-plugin-shopify-assets

Hi @truemoroz, I developed the vite-plugin-shopify-assets for that reason. The other plugin wasn't compatible with vite 5.
On top of that, vite-plugin-copy doesn't work on dev mode, and the plugin author said it's not a focus for them.
So in the end, vite-plugin-shopify-assets should cover everything that plugin copy and vite plugin cleanup do.
The only thing to keep in mind is that I haven't really put it in production for a long time. I'm starting to use it in my agency this week - so far so good!
If you want to give it a try and send me your feedback, I'd appreciate it. If you want to look at the code and think it could be better, let me know as well.
Thanks!

Hey @montalvomiguelo, do you think you could have a deeper look at my plugin to give some feedback?
I understand if you don't have enough time.

I'll take a look this weekend @knjshimi :)

Hi @knjshimi :)
I created this little example on Stackblitz. We have a typical Shopify theme folder structure because, let’s say, we need to keep things simple and working with the Shopify GitHub Integration.
Could vite-plugin-shopify-assets clean up the stuff that Vite generates without affecting other files in the assets/ directory? Ideally, with zero configuration, it’s nice for developers when things work out of the box.
Let’s say we don’t need to do any copy operations in the example. Is there a way that behavior can be extracted into its plugin?

Hey @montalvomiguelo thanks for looking into it!
I definitely put some default config in place, and it's possible to define if it should copy or not depending on the vite mode (build, watch, serve).
Let me do some testing when no configuration is set, and I will get back to you.

Hey @montalvomiguelo, I have just published a release (0.1.5) that fixes a few bugs when inferring default options. Here's a fork of your Stackblitz example with the latest release. I hope that one works better!
Note: Don't pass build: { emptyOutDir: false }
The plugin will take care of (and honour) that vite build option, meaning:

If it's false, it will not attempt to clear it.
If it's true, the plugin will under the hood turn that option to false, and do the cleanup itself based on the manifest.

Let me know how it goes :)

@knjshimi  In the original example, we have a very important asset, my-random-widget.js, that guy should not be deleted.
Actual:

@montalvomiguelo, at the moment this is the minimum configuration that we need to make it work:

It assumes:

public/ folder that holds static assets
assets/ folder for final shopify assets
targets: [*] plugin option

I'll see if there's an easy way to:

not require targets
use a single assets/ folder as both source and destination... but I'm struggling to see this working to be honest.

Hey @truemoroz, can you give me an idea of what you were expecting from that plugin (either vite-plugin-shopify-clean or vite-plugin-shopify-assets)? I want to understand your use case and likely folder structure.
Based on what we discussed in this thread so far, what are your thoughts?

Quick update: I think I found a way, which is more elegant and would solve the issue. It's likely to be more performant too.
It will take some time, but I'll let you know once it's done ☺

I want to keep my assets in target folder after rebuilding app

Hi @truemoroz, in that case vite-plugin-shopify-assets should do the trick. I have updated the readme with new documentation for zero config. Let me know if you are able to make it work with your folder structure ☺
@montalvomiguelo I wasn't able to make it work without using a separate static assets folder. It won't be possible because I can't find a way to differentiate old build files from static assets if they live in the same folder.

@knjshimi, the code of the previous plugin is public now 🎉 It contains a couple of ideas that your plugin could reference?
It'd be great to keep things simpler to help a wider range of projects like the one provided above that only need a simple way to clean up assets generated by Vite while preserving any other file that might be added by apps or via theme editor.

Hi @montalvomiguelo, thanks for that :) I will have a look and see if there's something I could adapt.

I still need to take a deeper look, but I can see a potential issue.
It differentiates old build files from by matching a glob pattern like 'name-*.ext', which mimics the default chunkFileNames pattern in vite '[name]-[hash].[ext]'
What if users use a different pattern via Vite options, eg: '-[name]__[hash].[ext]'?
What if users have two similar files in the same folder - one static and one processed? Eg: style-random.css and style-HuTrDF.css
I tried a similar path, at least in theory, in my recent experiments with no success because of the second case (the first one is not hard to overcome).
But maybe there is some small implementation detail in his method that I'm missing. I will give it another shot.

@montalvomiguelo do you have any idea why there is a check for "process.env.VITE_WATCH" on vite-plugin-shopify-clean?
I thought it was passed on vite watch mode, but it isn't (the correct way is this.meta.watchMode). Maybe it's a custom BAO thing?
Ideally we want this cleanup to work on watch mode as well.

Hey @montalvomiguelo maybe you want to close this issue now that vite-plugin-shopify-clean supports vite 5. cc: @truemoroz
I will still continue working on vite-plugin-shopify-assets anyway, and see if I can solve that use case you pointed out to me.
I got it working, at least partially, but I don't fully understand why yet. And I want to make sure it works on all modes (dev, build, watch) and it includes the copy feature.
Thanks for the help so far! Greatly appreciated 😄


Signin no longer working

Hi and happy easter.
In the last day or so I seem to have lost the ability to authorize. I get a "Post to authorize failed" response.
edit: sorry, the above was my own internal error message. I'm getting a "Could not resolve host: welcome.electrickiwi.co.nz" from CURL.
Has anything changed recently? Is there more information I can provide that will be helpful?

Sorry. I'm pretty sure this is a me problem. To do with container networking.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4869

Questions ?
Please refer to the Backport tool documentation


Release in tar.gz ?

Hi !
Is there a way to have a tar.gz release ?
I plan to add your binary like that in my Dockerfile :
ADD https://github.com/msoap/shell2http/releases/download/1.7/shell2http-1.7.linux.amd64.tar.gz /
But it has to be a direct link or a tar.gz
Thanx for reading

Agree, for Linux .tar.gz is a more appropriate format.
Done, see latest release: https://github.com/msoap/shell2http/releases/tag/1.8

Thanx lot


Change target_file to source_file

doh, thanks for catching that. Honestly a bit surprised it has managed to be there as long as it has. Missed this in the original rename iirc.


[pyAMQP] link_credit default value

#25451

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'KeyVault:0.3109458,Azure.Identity:0.065055385,Storage:0.045510672'


investigate using deno

We should do some research to see what it would take to support deno. I presume we will need to complete #35 and possibly use rollup to "polyfill" some of the node libs.

We removed vfile in #234 which should make this more viable.

@jbolda Deno supports .js files, also now that there is dnt you can very easily transform a deno package into a node package complete with shimming.


Post-install - in-cluster deployment fails on Rancher Desktop.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
After successfully installing the extension, there is a secondary install step that fails.

Expected Behavior
No response
Steps To Reproduce

Install Epinio Rancher Desktop Extension, via rdctl instructions
Use Install/Upgrade option in UI to install Epinio to the rancher desktop cluster.

Environment
- OS:
- Architecture:
- Epinio Version: 11
- Cluster:
  - Provider: Rancher Desktop
  - Options:
  - Kubernetes Version:

Logs
No response
Anything else?
No response

This could be related to a failed helm install, but errors aren't reflecting properly (separate issue).
I noticed that on windows, helm install failed and suggested running a helm repo update despite a direct reference to the remote tgz. On ubuntu/wsl on the same machine, there wasn't an issue. Since RD is running on windows, I ran the suggested helm repo update on windows and it resolved the issue.
Leaving open for better error handling


F1 does not show dartdoc or do anything else

Steps to repro:

Press F1

What happens:
Nothing
What should happen:
Something

It seems that pubspec.yaml possibly could not be found. Something more useful than silent failure would have been nice here. The same problem occurred with F3, jump to definition.

F1 and F3 appear to work only if the pubspec.yaml is at the root of the project folder. If it is deeper inside somewhere, then it can't be found, and these don't work.

Can you tell me a bit more about your project structure, and what folder you opened? We currently try and locate dart projects by scanning down through the opened top-level folders. We look two levels deep, and identify a dart project by looking for a pubspec.yaml file or a .packages file.

What might possibly help here is to decorate dart projects in the tree view. That would give the user an indication that we had successfully identified their dart projects.

Sure. My project folder is //src, and the Dart package is at //src/mojo/dart/mojom.

I mean, understanding this now, I could make sure I have a project directory for each dart package I'm working on, so this might be an education thing more than anything else.

Yeah, if you have multiple dart sub projects under a git repo, the editor gets confused sometimes.
How do you create a project directory

Closing; we have a separate issue for decorate items in the tree view.


Bug: Variant editor, changing attribute parameters, usability improvement

What are you trying to achieve?
The variant editor is hard to use with a lot of data and is frustrating for editors.
Once a value has been added using a dropdown for example, changing it requires clicking in to the field and deleting the value before it is possible to select a new value.
Steps to reproduce the problem
Create any product with variant attributes and try to modify a variant
What did you expect to happen?
Clicking into the field at least selects the existing value. Better: Show me all possible values instead of the currently selected one.
Logs
No response
Environment
Saleor SaaS

Transferring to saleor-dashboard as this is a UI issue.


Advise on learning and using PrimerLive

Hello Arthur,
I've chose PrimerLive for my Phoenix/LiveView project. Just a quick couple of questions:

You may think this an odd question, do you have any caution regarding my use of PrimerLive?
Do you have any advice, in general, on how to learn and use PrimerLive?
I see that it is based on GitHub's Primer Design System. Do I need to read the documentation for that?
At the Primer Design website, I see mention of Figma. Should I try to learn that?

When I started PrimerLive 2 years ago, I selected GitHub Primer because of its simplicity, focus on accessibility and completeness of documentation. The components were HTML-first and easy to enhance with CSS classes.
Soon after, things changed on the website: HTML-first got replaced by React and Rails, components got added with specs only, no code, and the Primer CSS repo got frozen.
Compare with the original website.
Implementing new components is harder, because there's no plain CSS file to base the code upon.
Regarding your first question: In its current state, PrimerLive is quite complete, but when GitHub implements new components I am unsure if the available code can be used to implement them in PrimerLive.
Learning a component library is best done doing. Take a component, pull up the documentation for that component, and see examples of use in the storybook app.
I think the original website (linked above) gives a better view of the documentation. Use it when you need styling that is not available out of the box.
Figma is useful if you want to create mockups or wireframes. Most of the time it is easier to just use a component.

Thanks


Missing fields k8s.pod.name and k8s.ns.name

Describe the bug
The Kubernetes pod and namespace name fields are not populated in emitted events, while Kubernetes support has been disabled in Falco. The Kubernetes client is not used in our environment to reduce costs and reduce the number of dropping events, this means that these fields need to be populated from the metadata available on the node as seen in https://github.com/falcosecurity/falco/issues/1585 .
Running version 0.31.0 with exactly the same settings resolves the problem. Also, enabling the flag "--disable-cri-async" does not result in the expected behavior in version 0.32.0.
How to reproduce it

Install Falco 0.32.0 on a Kubernetes node, without enabling Kubernetes support. Thus the arguments "-K" and "-k" should not be used.
Trigger any rule with %container.info, %k8s.pod.name and/or %k8s.ns.name in the output

Expected behaviour
The fields k8s.ns.name and k8s.pod.name are populated.
Screenshots
N/A
Environment

Falco version:

Falco version: 0.32.0
Driver version: 39ae7d40496793cf3d3e7890c9bbdc202263836b

System info:
{
"machine": "x86_64",
"nodename": "xxxxxx",
"release": "5.4.170+",
"sysname": "Linux",
"version": "#1 SMP Wed Mar 23 10:13:41 PDT 2022"
}

Cloud provider or hardware configuration: Google Kubernetes Engine version 1.21.11-gke.900

OS: Container-optimized OS

Kernel: 5.4.170+

Installation method: Daemonset on Kubernetes with either Docker or containerd nodes

Additional context
N/A

Hey @ruezetle

Is the container metadata correctly populated?
Are you using --k8s-node? (if yes, could you double-check the value passed is correct?)
Could you share your deployment files? Or just a minimal example to reproduce the issue?

Thank you in advance!
cc @jasondellaluce @FedeDP

Hi @ruezetle, we tried to reproduce your issue but without success. Everything works as it should.
Our environment is the following:
Nodes, k8s info:
NAME         STATUS   ROLES                  AGE    VERSION   INTERNAL-IP    EXTERNAL-IP   OS-IMAGE           KERNEL-VERSION      CONTAINER-RUNTIME
k8s-master   Ready    control-plane,master   116m   v1.21.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-39-generic   docker://20.10.7
node-1       Ready    <none>                 116m   v1.21.3   <IP_ADDRESS>   <none>        Ubuntu 22.04 LTS   5.15.0-39-generic   docker://20.10.7

Falco info:
* Setting up /usr/src links from host
* Running falco-driver-loader for: falco version=0.32.0, driver version=39ae7d40496793cf3d3e7890c9bbdc202263836b
* Running falco-driver-loader with: driver=bpf, compile=yes, download=yes
* Mounting debugfs
* Trying to download a prebuilt eBPF probe from https://download.falco.org/driver/39ae7d40496793cf3d3e7890c9bbdc202263836b/falco_ubuntu-generic_5.15.0-39-generic_42.o
curl: (22) The requested URL returned error: 404
Unable to find a prebuilt falco eBPF probe
* Trying to compile the eBPF probe (falco_ubuntu-generic_5.15.0-39-generic_42.o)
warning: the compiler differs from the one used to build the kernel
  The kernel was built by: gcc (Ubuntu 11.2.0-19ubuntu1) 11.2.0
  You are using:           gcc (Debian 5.5.0-12) 5.5.0 20171010
scripts/mod/modpost: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by scripts/mod/modpost)
scripts/mod/modpost: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by scripts/mod/modpost)
make[2]: *** [scripts/Makefile.modpost:134: /usr/src/falco-39ae7d40496793cf3d3e7890c9bbdc202263836b/bpf/Module.symvers]
* eBPF probe located in /root/.falco/falco_ubuntu-generic_5.15.0-39-generic_42.o
* Success: eBPF probe symlinked to /root/.falco/falco-bpf.o
Thu Jun 23 08:16:01 2022: Falco version 0.32.0 (driver version 39ae7d40496793cf3d3e7890c9bbdc202263836b)
Thu Jun 23 08:16:01 2022: Falco initialized with configuration file /etc/falco/falco.yaml
Thu Jun 23 08:16:01 2022: Loading rules from file /etc/falco/falco_rules.yaml:
Thu Jun 23 08:16:01 2022: Loading rules from file /etc/falco/falco_rules.local.yaml:
Thu Jun 23 08:16:01 2022: Starting internal webserver, listening on port 8765

Arguments passed to Falco Pod:
  - args:
    - /usr/bin/falco
    - --cri
    - /run/containerd/containerd.sock
    - --cri
    - /run/crio/crio.sock

Rule used to test the fields:
    - rule: Launch Package Management Process in Container
      desc: Package management process ran inside container
      condition: >
        spawned_process
        and container
        and user.name != "_apt"
        and package_mgmt_procs
        and not package_mgmt_ancestor_procs
        and not user_known_package_manager_in_container
      output: >
        Package management process launched in container (user=%user.name user_loginuid=%user.loginuid
        command=%proc.cmdline container_id=%container.id container_name=%container.name image=%container.image.repository:%container.image.tag k8s.ns=%k8s.ns.name k8s.pod=%k8s.pod.name container=%container.id)
      priority: ERROR
      tags: [process, mitre_persistence]

Rule output:
08:16:17.453463717: Error Package management process launched in container (user=<NA> user_loginuid=-1 command=apk container_id=6418a0d5a3fe container_name=k8s_tmp-shell-2_tmp-shell-2_default_54b83cfa-ab1a-4827-b2da-17ed96559207_0 image=nicolaka/netshoot:latest k8s.ns=default k8s.pod=tmp-shell-2 container=6418a0d5a3fe)

Could you provide us with more information about you environment and Falco configuration? Thank you!

We are also seeing this, in kubernetes on GKE with -K -k (set from the helm chart default values ) for some containers, but not all. These two events are with the exact same config, on the same cluster, within ~32s of each other
05:44:10.406740078: Error File below a monitored directory opened for writing (user=root user_loginuid=-1 command=sh -c ip link del calico_tmp_A || true file=/lib64/libc.so.6 parent=calico-node pcmdline=calico-node -felix gparent=calico-node container_id=8180fcd4b334 image=gke.gcr.io/calico/node) k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334 k8s.ns=kube-system k8s.pod=calico-node-np6pk container=8180fcd4b334

05:43:38.166628726: Error File below a monitored directory opened for writing (user=root user_loginuid=-1 command=start_runit /sbin/start_runit file=/lib64/libc.so.6 parent=<NA> pcmdline=<NA> gparent=<NA> container_id=472c06019de6 image=<NA>) k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6 k8s.ns=<NA> k8s.pod=<NA> container=472c06019de6

Another sample, now on falco 0.19.4. We are seeing these very often, more so since switching to COS_CONTAINERD on GKE.
08:00:20.868502159: Error File below / or /root opened for writing (user=<NA> user_loginuid=-1 command=systemd-udevd parent=systemd-udevd file=/<NA> program=systemd-udevd container_id=host image=<NA>) k8s.ns=<NA> k8s.pod=<NA> container=host

Server Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.12-gke.1500", GitCommit:"6c11aec6ce32cf0d66a2631eed2eb49dd65c89f8", GitTreeState:"clean", BuildDate:"2022-05-11T09:25:37Z", GoVersion:"go1.16.15b7", Compiler:"gc", Platform:"linux/amd64"}

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Performance of same op and workload in different model varies differently.

Compared two similar Bert models running on CPU with TVM, one is PyTorch model, the other is MXNet model. Due to the large performance difference, I did some profiling. The result shows the run time of the same operation(matmul) with same workload varies big.
ENV:

TVM: build with MKL.
Intel CPU
OpenMP: KMP_AFFINITY=compact,1,0 OMP_NUM_THREADS=24

Model inference time:
# mxnet model
TVM Mean inference time: 5.53 ms
# pytorch model
TVM Mean inference time: 23.05 ms

Profiling result:
# MXNet model
Node Name                           Ops.                Time(us)   Time(%)  Shape.  Inputs  Outputs
---------
fused_nn_dense_add_15        fused_nn_dense_add_1       308.926   5.58     (32, 768)      3       1
fused_nn_dense_add_11         fused_nn_dense_add_1       307.277   5.551    (32, 768)        3       1

# PyTorch Model
Node Name                           Ops.                Time(us)   Time(%)  Shape.  Inputs  Outputs
---------
fused_nn_dense_add_3        fused_nn_dense_add_3       1783.75    7.631    (32, 768)     3       1
fused_nn_dense_add_31      fused_nn_dense_add_3        1593.08    6.815    (32, 768)    3       1

IR code (same between PyTorch model and MXNet model)
          attr [0] "compute_scope" = "fused_nn_dense_add_3_compute_";
          attr [C: handle] "storage_scope" = "global";
          allocate(C, float32, [24576]) {
            attr [0] "extern_scope" = 0;
            @tir.tvm_call_packed("tvm.contrib.cblas.matmul", @tir.tvm_stack_make_array(placeholder, @tir.tvm_stack_make_shape(32, 3072, dtype=handle), 0, 2, 0f32, 0, dtype=handle), @tir.tvm_stack_make_array(placeholder_1, @tir.tvm_stack_make_shape(768, 3072, dtype=handle), 0, 2, 0f32, 0, dtype=handle), @tir.tvm_stack_make_array(C, @tir.tvm_stack_make_shape(32, 768, dtype=handle), 0, 2, 0f32, 0, dtype=handle), False, True, dtype=int32)
            for (ax0: int32, 0, 32) "parallel" {
              for (ax1: int32, 0, 768) {
                T_add[((ax0*768) + ax1)] = ((float32*)C[((ax0*768) + ax1)] + (float32*)placeholder_2[ax1])
              }
            }

However, when setting OMP_NUM_THREADS=1 the model inference time is same, seems it's a problem with multiple threads.
What may cause the difference?

I also got similar questions. Same models written in PyTorch (Bert for example) seem to be slower than TensorFlow.
As for Relay IR, models from different frontends should be the same.

Thanks @nolanliou , this is indeed an interesting observation. It might relates to other ops. It would be great if we can move some of the trouble shooting discusison to a https://discuss.tvm.ai/ thread, and print out the relay model to check their differences.

@tqchen Nobody bothers to answers questions on the forum my question has been pending for over an year. There is a limit to everything.

Hi @deimsdeutsch please note that the community relies on volunteers to answer help and answer questions that get arised. Many volunteers are also operate on their part-time. so it can indeed be possible that things get slipped through. Feel free to create new topics, we also welcome contributions to the community


postgresql reference Acorn.

Reference Postgresql Acorn file. Part of issue #28
Currently in progress.

library acorns being worked on and tracked separately


MB-15875 prime service item address change check

Jira ticket
Summary
This PR adds validations to ensure that the prime cannot use the mto_service_item_updater service object to add/update SITDestinationOriginalAddress or change an exisitng SITDestinationFinalAddress (prime is able to add SITDestinationFinalAddress if it hasn't been set yet).
SITDestinationOriginalAddress isn't in the swagger definition, so there isn't really a way to test service object validation directly, but hopefully the added tests are sufficient.
Verification Steps for the Author
These are to be checked by the author.

[x ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

run DEVSEED_SUBSCENARIO=sit_address_updates make db_dev_e2e_populate
Go to a SITUP2 and get the service item ID and etag for the DDDSIT service item
In postman, create a PATCH request to http://officelocal:3000/mto-service-items/:mtoServiceItemID with this body

{
  "id": {serviceItemID},
  "modelType": "UpdateMTOServiceItemSIT",
  "sitDestinationFinalAddress": {
    "streetAddress1": "7 Q St",
    "city": "San Antonio",
    "state": "TX",
    "postalCode": "78227",
  },
}
```  and make sure you receive an error
4. In your db, find the service item in the `mto_service_items` table and delete the `sit_destination_final_address_id`
5. create the same PATCH request in step 3, and make sure that the service item is returned with `sitDestinationFinalAddress` populated

### Backend

- [ ] Code follows the guidelines for [Logging](https://transcom.github.io/mymove-docs/docs/about/development/logging).
- [ ] The requirements listed in [Querying the Database Safely](https://transcom.github.io/mymove-docs/docs/backend/guides/golang-guide#querying-the-database-safely) have been satisfied.

Messages

:book:
:link: MB-15875

  Generated by :no_entry_sign: dangerJS against 5da53afa4f20b746136075ff2a5a2f93550bd7bf


JAX profiler does not trace tfds usage?

Hi JAX team,
Recently I tried to use JAX Profiler to do profiling, and bumped into an issue that it cannot show the iterations of tf dataset on separate threads, to have a very minimal example to illustrate the issue, please find:
https://colab.research.google.com/drive/1iS9EbiSr1uXDAfpWbYdCpoPE2dvVVHbx#scrollTo=7rI_fV1rlDXu
Here we can't see any dataset prefetch in another threads and the next and train are one after one.
While we would like to see is something similar to https://colab.research.google.com/github/tensorflow/tensorboard/blob/master/docs/tensorboard_profiling_keras.ipynb#scrollTo=lugpLFAflkeI
where tf_data_iterator is on another thread and we can see what's happening there.
From what I understood, JAX Profiler is using the xla_client.profiler, did I miss something or misconfigure so that I could not see the tfds info? Please help advise how to profile when we have tfds in the code. Thanks!

Hey @zhongwen ! I have to dig into this more, but I suspect that you'll have to separately use TF's profiling and JAX's profiling APIs. I'm not sure a single API can be used to generate a single trace including both JAX and TF events unfortunately. @hawkinsp had the idea to try the manual capture method via TensorBoard, starting both a TF and JAX server, and seeing if TB supports capturing from two servers at once. I'm not sure if this works though, I'll have to try it. I'll update this thread after I try some things myself and hopefully have more info.

I also ran into the same issue.
It took me a while to realize that I need tf's profiling tools to see details for tf.data. A note in JAX's profiling doc might save others' some time.
Would be great if the two profiling can be combined into one!

I tried starting both TF and JAX servers, but was not able to have tensorboard capture from the two at once.
I saw profiling JAX doc added Perfetto, in addition to tensorboard. Perfetto does not seem to support capturing from the two at once either, as jax/collect_profile.py only accept 1 port as command line argument.
(BTW, is perfetto added because it's better than tensorboard in some way? I assume the server side/training script and trace data  are the same between the two, the difference is in the UI to view the traces?)


Fixes #288 - Remove cranelift-codegen from the dependencies of the lucet-runtime components such as lucet-module

wasn't that difficult after all  :smiley:


list += operator generates Error

Feature

a = [3]
a += a
print(a)

The code above generates the following error:
(list += operator)
actual
Hyeockz:RustPython hyeockjinkim$ cargo run b.py
    Finished dev [unoptimized + debuginfo] target(s) in 0.11s
     Running `target/debug/rustpython b.py`
thread 'main' panicked at 'already mutably borrowed: BorrowError', src/libcore/result.rs:999:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

expected
[3, 3]

If this is issue is valid, can I take this issue?

Sure thing!


legal files: note[@type="intertext"]

Taking care of note[@type="intertext"]
In the legal files, note[@type="intertext"] indicates a text passage that has been quoted later in a fackel text. However, the relation is not 1:1. Some text passages are split into more than one element, with no indicator if or how they belong to one another and are quoted as a whole in the same fackel text or in parts and several fackel texts. What we know for sure boils down to: This legal file is quoted in a certain fackel text, doesn't get any more precise.
So: For every fackel text referring to a legal file - indicated in the legal file in one or more note[@type="intertext"] - create a statement like this one:
<https://sk.acdh.oeaw.ac.at/[text-ID]/relation/[n]> a ns1:INT3_IntertextualRelationship ;
        rdfs:label "Intertextual relation"@en ;
        ns1:R12_has_referred_to_entity <https://sk.acdh.oeaw.ac.at/[text-id]> ;
        ns1:R13_has_referring_entity <https://sk.acdh.oeaw.ac.at/[fackel-text-id]> .

The value for [text-id] is already in use in thy python script, take those.
The value for [n] needs to be one that has not been in use so far (many [n]s are in use already for other rs or quote elements).
[fackel-text-id] needs to be looked up:

Take the values from note[@type="intertext"]/@source
For each value, look up the range element in this file containing this @source value.
Take the text nodes of the preceding sibling textID-element. Something like //range[contains(., "[value]")]/parent::text/textID/text()
For each distinctive fackel text (aka textID) which is in this way indirectly referenced in a legal file's note[@type="intertext"]/@source, create one INT3-statement (see above, resp. example below).

Example from D_000010-003-000:
Contains exactly one relevant note element:
<note type="intertext" source="https://fackel.oeaw.ac.at/f/657,125">Wir besitzen Ihr w.
[...]
 wenden.</note>

So: 1 or more note[@type="intertext"] the @source of which refer to exactly 1 textID means: 1 statement should be created, in this case:
<https://sk.acdh.oeaw.ac.at/D_000010-003-000/relation/13> a ns1:INT3_IntertextualRelationship ;
        rdfs:label "Intertextual relation"@en ;
        ns1:R12_has_referred_to_entity <https://sk.acdh.oeaw.ac.at/D_000010-003-000> ;
        ns1:R13_has_referring_entity <https://sk.acdh.oeaw.ac.at/text12016> .

(Note 1: The [n] 13 is just a wild guess; let's just assume 12 text passages and segments with their respective intertextual relations have already been built from the respective elements and this is the first intertextual relation built for a relation from a fackel-text to this text. If this count is a problem, tell me - we'll figure out a new URI, something like https://sk.acdh.oeaw.ac.at/D_000010-003-000/relation/text12016.)
(Note 2: If there were, say, 7 note[@type="intertext"]/@source, referring to the range-element of exactly 3 distinct text elements in fackel_intertexts.xml, 3 such statements should be created. And so on.)
(Note 3: If its a problem to reduce the amount of statements created to the actually referenced textID - since usually there is more than one @source pointing at a text, AND there is trouble with the [n] count, then we can always go note element by note element and go with the URI suggested in Note 1, possibly creating a couple of identical statements. Redundancies are not a problem.)

@linxOD In the current version, I find these two statements:
<https://sk.acdh.oeaw.ac.at/D_000031-003-000/relation/0-99> a ns1:INT3_IntertextualRelationship ;
        rdfs:label "Intertextual relation"@en ;
        ns1:R12_has_referred_to_entity <https://sk.acdh.oeaw.ac.at/text12586> ;
        ns1:R13_has_referring_entity <https://sk.acdh.oeaw.ac.at/D_000031-003-000> .

and
<https://sk.acdh.oeaw.ac.at/D_000031-003-000/relation/1-99> a ns1:INT3_IntertextualRelationship ;
        rdfs:label "Intertextual relation"@en ;
        ns1:R12_has_referred_to_entity <https://sk.acdh.oeaw.ac.at/text12586> ;
        ns1:R13_has_referring_entity <https://sk.acdh.oeaw.ac.at/D_000031-003-000> .

Two issues here:

Minor: referred-to and referring entity are mixed up. Here, the Fackel refers to the legal doc.
These statements say the same thing twice, that there is the link between these two texts; it maybe has to do with the fact, that while there are seven relevant note elements in the document, they have only two distinct @source values. But: Both of them point at the same Fackel text, so there should actually be only 1 statement.
We can resolve this quickly and easily by changing the relation's URI: Instead of "[n]-99" just insert the Fackel text's ID as the last part. (aka Note 1 from above).


Feature request: reliable collection explorer

Currently, there's no way to view (or modify) reliable collection items.
Considering that a lot of data could potentially be stored in SFRC to replace a data store, this becomes a question of "if this is my data store, why can't I see the data".
Is this something that is planned and if so, what's the functionality it will support?

We have released a Backup Explorer that will allow you to view a Reliable Collections backup offline: https://github.com/microsoft/service-fabric-backup-explorer
We will be soon releasing a read-only viewer tool for viewing live data in Reliable Collections.

@SeanFeldman Please review backup explorer info shared in former commend and let us know it meets needs or your requirements. If yes, please close the issue and watch our release announcements for updates on RC viewer tool updates

Tool released as noted above and demoed in community call 48.


the question about random_shift_factor

Which code handles the sampling problem of random_shift_factor=32？

请问，空间感知采样策略的代码在哪里？？

Please find closer in the code.


Is there a way to control 'position' actuators?

I tried to run this with <position...> actuators, but it did not seem like mjpc tried to move my simulated actuators at all. It worked fine when I changed to <motor...> actuators.
Is this a known restriction, or is there some way to get mjpc to work with <position> actuators that I am not aware of?

Position actuators work, see the hand task. You may need to modify the cost function to work with different actuators. A common change is to penalize actuator_force instead of ctrl (example).
Do you have an example task we can look at?

Ok, using actuator_force might be it. Thanks for pointing me to the hand task; I will take a closer look at it.
Here is my task (gitlab) (<actuator> defined at the bottom of leg.xml, original position actuators commented out). I just copied the cartpole task and threw a custom model in it to see if I could get it to move around.
I modified the residuals to penalize velocity difference from 1.0, just to get the model to move. When I use position actuators, no motion occurs despite an elevated cost. The optimization  improvement graph is stuck at zero.
With motor actuators, the leg flails around as expected. Maybe my model's constraints are interacting with the optimizer in a funny way when I use position actuators.
Video of movement with motor actuators: https://youtu.be/wAMkcH5_GVk

Take a look at this branch.
I changed the position actuators to use ctrllimited and modified the way the residual is computed.
You can probably take it from here. Let us know if you encounter any additional blockers..

Thanks for the example!
Based on your example and some more testing, it looks like adding ctrllimited and explicit ranges allows planning and motion as expected with <position> actuators.
Also, thanks for demonstrating how to include data->actuator_force.


Add authorization to XDS connection

This adds basic authorization to the XDS connection. This consists of
extracting the identities and verifying the SA and NS are consistent
with the identity.
This is flag protected and on by default for testing - happy to turn it
off before merging.
As part of this work, I refactored some of the SPIFFE code to make it easier to extract the fields we need from a spiffe identity.
TODO (prior to merge):

add some testing of the auth function)
Figure out what to do for unauthenticated requests

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test release-notes_istio

lint_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test release-notes_istio

lint_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test lint_istio

unit-tests_istio
e11a4de5adaddfb99e5881a2498c860167e47904
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio/wg-security-maintainers PTAL


Transition to ES-syntax

As part of this issue, it might be worth switching to Prisma.
Also, as part of this, we can add TS support.

Only switched to TS. Prisma might be introduced later. Or not


[BUG] ModelID not shown in Module Identities List

Modules can use PnP as regular devices.
The Model Id should be a column in the Module Identities list

Updated in v0.14.7


Markdown refresh preview command resets scroll position

Issue Type: Bug
repo

Open a long markdown preview
Scroll down part way on the page
Be focused on preview
Run the markdown refresh preview command

Bug
Scroll position is reset to top
VS Code version: Code - Insiders 1.39.0-insider (55825e9ad2aa4b4b3f8b2290a54a517bc454a7a5, 2019-09-10T05:23:19.252Z)
OS version: Darwin x64 18.7.0

System Info

Item
Value

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

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
16.00GB (2.73GB free)

Process Argv
-psn_0_1016056

Screen Reader
no

VM
20%

See also #63690

Still happens.
Version: 1.39.0-insider
Commit: a655c0836de8b8e06f4d56fa4e581b0ced53f56f
Date: 2019-10-03T05:32:23.012Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 18.7.0

@Tyriar I tried with both side by side and single preview and I can't repo. Can you share your steps

@mjbvz steps are exactly as stated above, happens in both full preview and on side.

Command in gif is:

Do you have one of the scroll sync settings turned off? It looks like the text file is not scrolled along with the preview

Yes markdown.preview.scrollEditorWithPreview: false is what causes this