legion_prof crash

I am seeing a crash with legion_prof:
(/pscratch/sd/s/seshu/cunumeric-cuda) seshu@perlmutter:login06:/pscratch/sd/s/seshu/2023/09/07/095158> legion_prof legate_*.prof
Reading log file "legate_0.prof"...
Reading log file "legate_2.prof"...
Reading log file "legate_3.prof"...
Reading log file "legate_1.prof"...
Matched 55611 objects
Matched 49128 objects
Matched 48719 objects
Matched 49847 objects
No Legion Spy data, skipping postprocess step
Sorting time ranges
Generating interactive visualization files in directory "legion_prof"
thread '<unnamed>' panicked at 'Error: Copy can not find src_inst:0x8000004000500037', src/backend/common.rs:915:21
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

(/pscratch/sd/s/seshu/cunumeric-cuda) seshu@perlmutter:login06:/pscratch/sd/s/seshu/2023/09/07/095158> legion_prof --archive legate_*.prof
Reading log file "legate_0.prof"...
Reading log file "legate_2.prof"...
Reading log file "legate_1.prof"...
Reading log file "legate_3.prof"...
Matched 55611 objects
Matched 49128 objects
Matched 48719 objects
Matched 49847 objects
No Legion Spy data, skipping postprocess step
Sorting time ranges
Created output directory "legion_prof.1"
Writing level 0 with 1 tiles
thread '<unnamed>' panicked at 'assertion failed: sample_util <= 1.0', src/backend/data_source.rs:600:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'assertion failed: sample_util <= 1.0', src/backend/data_source.rs:600:13
Rayon: detected unexpected panic; aborting
Rayon: detected unexpected panic; aborting
Aborted

These are the two modes I care about right now.
The logs are here: http://sapling2.stanford.edu/~seshu/rixs/logs/
I should be using this commit of legion:
commit 3fbed8ed8771f931e463400390edcd9484fa8537 (HEAD -> shardrefine, origin/shardrefine)
Author: Mike Bauer<EMAIL_ADDRESS>Date:   Tue Sep 5 00:48:39 2023 -0700

    legion: remove advisement operations since they are no longer meaningful with the new refinement data structures and replace them with reset operations

I played with the log files and it turned out the instance 0x8000004000500037 is not logged, that's why the copy complains that there is no such instance 0x8000004000500037. Here is the CopyInstInfo that contains this instance:
src_mem: 0x1e0000000000000a  dst_mem: 0x1e0001000000000a  src_fid: 0  dst_fid: 0  src_inst: 0x8000004000500037  dst_inst: 0x8000801000a00050  fevent: 0x800080100de00018  num_hops: 1  indirect: False

I think we need @lightsighter to take a look at this issue.

Run with the latest control replication first.

With the latest control replication I still see a crash with archive mode of legion_prof, but the default mode works.
Created output directory "legion_prof.1"
Writing level 0 with 1 tiles
thread '<unnamed>' panicked at 'assertion failed: sample_util <= 1.0', src/backend/data_source.rs:600:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread '<unnamed>' panicked at 'assertion failed: sample_util <= 1.0', src/backend/data_source.rs:600:13
Rayon: detected unexpected panic; aborting
Rayon: detected unexpected panic; aborting
Aborted

I think @elliottslaughter needs to look at this. The logs are here: http://sapling2.stanford.edu/~seshu/rixs/logs/

I did another run and I hit the missing copy error again too on control_replication.
commit 61a919f8b2dce55619b6318de19b63aa5c1c56d9 (HEAD -> control_replication, origin/control_replication)
Author: Mike<EMAIL_ADDRESS>Date:   Wed Sep 6 17:02:41 2023 -0700

    legion: make sure to clear references to the initial future when all-reduce operations are deactivated

Not sure if they are useful but profile logs are available here: http://sapling2.stanford.edu/~seshu/rixs/logs/missing_copy/

Are we sure that we are doing all the detach operations on external allocations? Any instances that Legion allocates will require those instances be deleted and have their profiling responses come back to the runtime before it can shutdown or it will hang. All that leaves left are the instances made for external allocations with attach/detach.
If you want me to look at it I will need a reproducer on sapling.

I think I found a root cause for https://github.com/StanfordLegion/legion/issues/1546#issuecomment-1711981596, but there is a design decision that needs to be made here.
When you have a mix of dynamically and statically-sized memories, the profiler currently takes the sum of the static memories to compute an upper bound. (This is for the sysmem overview across all nodes.) That is obviously wrong. However, what is the correct bound?
At the point we are in the code, it would be easy to compute a dynamic bound on all memories. However, that may be inaccurate because the static memories aren't necessarily fully used.
I can't see an easy way to split it out (e.g., sum all static memory sizes and then add the dynamic memory allocations on top of that). If we really want that as an answer, I'll have to re-architect the code to achieve it.

@syamajala Apply this patch to Legion and then try to generate the profile logs again. When it crashes report the backtrace with line numbers:
diff --git a/runtime/legion/runtime.cc b/runtime/legion/runtime.cc
index 00db22fb5..d221d793d 100644
--- a/runtime/legion/runtime.cc
+++ b/runtime/legion/runtime.cc
@@ -8252,6 +8252,13 @@ namespace Legion {
                                                              constraints,
                                                              dim_order);
       Realm::ProfilingRequestSet no_requests;
+      if (runtime->profiler != NULL)
+      {
+        const RtUserEvent unique = Runtime::create_rt_user_event();
+        Runtime::trigger_event(unique);
+        runtime->profiler->add_inst_request(no_requests,
+            implicit_provenance, unique);
+      }
       Realm::RegionInstance::create_instance(eager_pool_instance,
                                              memory,
                                              layout,
@@ -8272,9 +8279,7 @@ namespace Legion {
     //--------------------------------------------------------------------------
     MemoryManager::~MemoryManager(void)
     //--------------------------------------------------------------------------
-    {
-      if (eager_pool_instance.exists())
-        eager_pool_instance.destroy();
+    {
       delete eager_allocator;
     }

@@ -8413,6 +8418,8 @@ namespace Legion {
         free_legion_instance(it->first, it->second);
       pending_collectables.clear();
 #endif
+      if (eager_pool_instance.exists())
+        eager_pool_instance.destroy();
     }

     //--------------------------------------------------------------------------
diff --git a/runtime/realm/inst_impl.cc b/runtime/realm/inst_impl.cc
index 33f959907..c3665d9ec 100644
--- a/runtime/realm/inst_impl.cc
+++ b/runtime/realm/inst_impl.cc
@@ -435,6 +435,7 @@ namespace Realm {
                                                     const ProfilingRequestSet& prs,
                                                     Event wait_on)
     {
+      assert(!prs.empty());
       return RegionInstanceImpl::create_instance(inst, memory, ilg, 0,
                                                 prs, wait_on);
     }
@@ -446,6 +447,7 @@ namespace Realm {
                                                              const ProfilingRequestSet& prs,
                                                              Event wait_on)
     {
+      assert(!prs.empty());
       return RegionInstanceImpl::create_instance(inst, memory, ilg, &res,
                                                 prs, wait_on);
     }
@@ -777,6 +779,7 @@ namespace Realm {
                                                         const ProfilingRequestSet& prs,
                                                         Event wait_on)
     {
+      assert(!prs.empty());
       MemoryImpl *m_impl = get_runtime()->get_memory_impl(memory);
       RegionInstanceImpl *impl = m_impl->new_instance();
       // we can fail to get a valid pointer if we are out of instance slots
diff --git a/tools/legion_prof.py b/tools/legion_prof.py
index d9ff0c41c..8182f18c9 100755
--- a/tools/legion_prof.py
+++ b/tools/legion_prof.py
@@ -5666,6 +5666,9 @@ def build_state() -> Optional[State]:
         print('No matches found! Exiting...')
         return None

+    import pdb
+    pdb.set_trace()
+
     # if number of files
     if state.num_nodes > len(file_names):
         print("Warning: This run only involved %d nodes, but only %d log files were provided. If --verbose is enabled, subsequent warnings may not indicate a true error." % (state.num_nodes, len(file_names)))

At the point we are in the code, it would be easy to compute a dynamic bound on all memories. However, that may be inaccurate because the static memories aren't necessarily fully used.

@elliottslaughter I would compute the utilization of the static memories as normal based on their total size. For dynamic memories, I would compute their utilization based on the total size of all the live instances in them at any point. So 100% utilization of a dynamic memory is the point(s) where it hits its high water mark.

Maybe I'm not explaining myself clearly.
The profile provided by @syamajala has 4 nodes. It has 8 sysmems total. Of those, 4 (1 per node) is a normal sysmem (what I called "statically sized" above, because it has a fixed size). Then there are 4 more external sysmems (what I called "dynamically sized", because the memory pool isn't fixed).
When we generate the utilization plot for a single memory, everything works. The logic in the profiler today is correct.
The problem is rendering the overview utilization plot. This covers the entire machine, so all 8 sysmems (both static and dynamic). Basically, to generate this utilization plot we pretend that this memory is a single aggregate memory across the entire machine, and we add up the sizes of the individual memories. Because the aggregate memory size is not 0 (it contains at least 1 static memory), it does not take the dynamic memory code path. So we successfully add up the sizes of the static memories, but then if there are enough dynamic allocations, that pushes us over.
You can see the problematic code here:
https://github.com/StanfordLegion/legion/blob/7b5ff2fb9974511c28aec8d97b942f26105b5f6d/tools/legion_prof_rs/src/backend/common.rs#L367-L376
To recap, I can see a few different options:

Calculate the aggregate size as the static capacities + the maximum size of instances in dynamic memories. This is the theoretically optimal approach, but the code isn't designed to do it, and it would impose overheads in either computation or memory relative to what we do now.
Pretend that the entire aggregate memory is dynamic and just calculate its size based on used instances across the entire machine. This is easy to do today but not accurate, because we throw away information about the sizes of the static memories.
Take the max of the static size and dynamic size. A bit odd, but arguably a better approximation than (2).
Split the overview into 2 parts. Now we'll have a "sysmem" and a "sysmem external" utilization plot in the overview. Arguably might be better than (1) since users might care about this detail.
Stop providing overview utilization for memories for the entire machine. The problem goes away, at the cost of this feature.

Option (1) was exactly what I was proposing. You can decide if you think it is worth it or not to do that.

I suppose I wouldn't mind doing option 5 also since I feel like that is misleading anyway since some memories (like sysmems) are usually not full, while other memories (GPU framebuffers) are very close to capacity.

I am not hitting any assertion when I run with @lightsighter runtime patch, but  I'm hitting the issue Elliott is looking at with dynamic memories though.

@syamajala Do the profiles you generate still fail with this error message when running with the patch?
thread '<unnamed>' panicked at 'Error: Copy can not find src_inst:0x8000004000500037', src/backend/common.rs:915:21

No. They are failing with: thread '<unnamed>' panicked at 'assertion failed: sample_util <= 1.0', src/backend/data_source.rs:600:13

And that is true over repeated runs?

You can try cherry-picking https://gitlab.com/StanfordLegion/legion/-/merge_requests/920 to work around the dynamic memory capacity issue, on the off chance that helps expose the other bug.

The work around for the dynamic memory issue works for me and I am able to generate profiles now.

I pushed a fix for the second failure mode to the control replication branch. @syamajala please pull and try again.


Prevented which xmllint on Windows

I feel like writing a CONTRIBUTING guide now.
This repository follows the issue-discussion-PR pattern.
May as well do an exception again. AFAIK, the which command does exist on Windows. What is the point of this? What bug(s) does it solve?

What happened with the windows when you run that original script? I think you can do the similar test with where on Windows, but I don't have a windows to test it with.

I looked it up, and it looks like the which command is not present on Windows (replaced instead with where ).
Disabling a feature makes little sense if it can be made to work with minimal effort.
if (`which xmllint` || `where xmllint`)

I am not sure if this would even work. I only have access to windows machines on week days.

I can run it on GNU/Linux if you test it first on Window and make sure it works there.

Didn't knew about the where command. I will test it out.

Both suggestions work but, in my opinion, its better only to execute one command than to test two of them.

OR test || only executes the left and if it fails, then it the right one, not both by default. The first test should be which as it sounds to me a bit pervert to run PHP on Windows :D

True, but we exist. xD

Sorry for the delays, college systems were a mess today. Will test it out tomorrow.


Is there a reason that pre/postUpdate run on killed entities?

Commit 4d1d27f3d40b0cf7809fa39dae835a442f897763 added the updateOnlyExistingChildren flag to groups, which blocks running the update loop on entities that have exists = false. However, it seems that preUpdate and postUpdate still call on all of these children. Disabling those didn't seem to have an impact that I could tell. Is there something I am overlooking as to why these still need to run?

I'm not sure. I've also disabled those without any apparent effect.
Component.Core.preUpdate itself exits early for exists=false or parent.exists=false. For all objects it handles pendingDestroy and updates previousPosition and previousRotation.
It sort of makes sense that pendingDestroy should work even on killed objects.
Component.Core.postUpdate handles customRender regardless of the exists value. It also calls postUpdate on any children. Some display objects (BitmapText, Graphics, TilemapLayer) have special postUpdate methods.
Component.PhysicsBody.postUpdate skips nonexisting objects.

Hmm, okay so it wouldn't make sense to just use updateOnlyExistingChildren the same on post/pre I suppose? Would it then make sense to add a different flag? We've done further testing and at least with our setup can't find any issues when modifying to use updateOnlyExistingChildren on pre/post, and we get ~25% decrease in tick time since we have hundreds of dead entities in groups waiting to be recycled.

I guess there's no harm in adding preUpdateOnlyExistingChildren and postUpdateOnlyExistingChildren, though I wish I could describe the consequences better. It feel like pendingDestroy should be handled in any case, so that part would have to be moved.
On the other hand, anyone interested enough could just overwrite these methods like you and I have.
I'm curious, do you know where most of that extra/wasted time is spent?


In 46

Closes #46
Implementation

Add two new integer types i16 and i32.
Check the values and promote them accordingly to the next higher value.
After each binary operation check if the result is in bounds, otherwise promote the result to next higher dtype.
In binread, for integer constants, check the length and values (if they conflict) and then assign them appropriate integer types.

All integration tests and yamtests are passing. Merging this.


prefetch-npm-deps: fix reproducibility

Description of changes
v1 lockfiles can contain multiple references to the same version of a
package, and these references can contain different integrity values,
such as one having SHA-1 and SHA-512, while another just has SHA-512.
Given that HashMap iteration order isn't defined, this causes
reproducibility issues, as a different integrity value could be chosen
each time.
Thanks to @lilyinstarlight for discovering this issue originally, as well
as the idea for the sorting-based implementation.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

By the way, I've now confirmed all 46 npm FODs in nixpkgs still reproduce to the same hash after this change (and now some like open-stage-control.npmDeps won't randomly get a different hash sometimes)

FODs checked, by attr (required some local changes not in nixpkgs to get all of them exposed via attrs):
ansible-language-server.npmDeps
audiobookshelf.npmDeps
authelia.web.npmDeps
bitwarden.npmDeps
cdxgen.npmDeps
deltachat-desktop.npmDeps
docker-compose-language-service.npmDeps
dot-language-server.npmDeps
element-desktop.keytar.npmDeps
emscripten.nodeModules.npmDeps
evcc.npmDeps
forgejo.frontend.npmDeps
geph.gui.gephgui.npmDeps
hred.npmDeps
iosevka-comfy.comfy-duo.npmDeps
iosevka-comfy.comfy-fixed.npmDeps
iosevka-comfy.comfy-motion-duo.npmDeps
iosevka-comfy.comfy-motion-fixed.npmDeps
iosevka-comfy.comfy-motion.npmDeps
iosevka-comfy.comfy-wide-duo.npmDeps
iosevka-comfy.comfy-wide-fixed.npmDeps
iosevka-comfy.comfy-wide.npmDeps
iosevka-comfy.comfy.npmDeps
iosevka.npmDeps
kaufkauflist.npmDeps
lv_img_conv.npmDeps
matrix-appservice-irc.npmDeps
mongosh.npmDeps
mpvScripts.webtorrent-mpv-hook.npmDeps
open-stage-control.npmDeps
paperless-ngx.frontend.npmDeps
photoprism.frontend.npmDeps
pnpm-lock-export.npmDeps
sharing.npmDeps
sunshine.ui.npmDeps
terminal-stocks.npmDeps
torq.web.npmDeps
twspace-crawler.npmDeps
uptime-kuma.npmDeps
vaultwarden.webvault.npmDeps
vieb.npmDeps
vsce.npmDeps
webcord-vencord.vencord.npmDeps
webcord.npmDeps
zigbee2mqtt.npmDeps
zinc.webui.npmDeps

This is not in 23.05 atm right?

This is not in 23.05 atm right?

This was merged before the 23.05 release, so it definitely is in 23.05. Are you having an issue still?
Feel free to open a new issue on nixpkgs if you are and ping me on it


Use favicon with white background

The current template has a favicon with transparent background. This makes it very hard to see in dark backgrounds.

This PR updates favicons with white backgrounds to make them look better.

@tqchen


Can I have a configuration to call multiple formatters automatically instead of using :NextFormatter?

If you have multiple formatters installed that are supported for some filetype, vim-autoformat tries all formatters in this list of applicable formatters, until one succeeds

Can I configure somehow to something like until all suceeds? I can create an augroup that calls :NextFormatter to acomplish this, but then everytime is run I get a "selected formatter" message. I would like to know if there's an elegant way to do it (instead of using multiple :NextFormatter and :silent calls).

Would it be possible to create a bash script that chains the tools that you want to use? That way you have a single program that you can use as formatter.

I solved this by chaining multiple formatters like this:
let g:formatdef_multiple = '"isort - | black -q -"'
let g:formatters_python = ['multiple']

Would it be possible to create a bash script that chains the tools that you want to use? That way you have a single program that you can use as formatter.

It's possible, but I was thinking about a configuration to avoid creating external bash scripts to acomplish this.
But since you're the maintainer of this project, feel free to close the issue. I'll try @Ch00k's suggestion as well. Thanks!

Yes, if you're in an environment where shell syntax works, then @Ch00k 's suggestion is superior to creating external scripts. This is also how I envisioned this to work when I wrote this tool, but I later learned that it may not work on windows.

I'm using this
function! s:autoformat_all() abort
  let l:formatter_global_var_name = 'g:formatters_' . &filetype

  if !exists(l:formatter_global_var_name)
    Autoformat
    return
  endif

  let l:formatter_buffer_var_name = 'b:formatters_' . &filetype
  let l:formatters = []
  execute 'let l:formatters = ' . l:formatter_global_var_name

  for l:formatter in l:formatters
    execute 'let ' . l:formatter_buffer_var_name . '= ["' . l:formatter . '"]'
    Autoformat
  endfor

  execute 'unlet ' . l:formatter_buffer_var_name
endfunction
command! AutoformatAll call <SID>autoformat_all()
noremap <Leader>a :<C-u>AutoformatAll<CR>


SSHD startup should not be asynchronous

Amends #19 as per comments there.
@reviewbybees @olivergondza

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@jglick, I was experiencing this with 1.6 so before the fix (I had no idea already existed). I have retested against 1.11 it has the desired effect. Thanks!

@reviewbybees done

@oleg-nenashev, I have not realized Jesse had already achieved that objective in more predictable way.


Add global allocator for SGX enclave

Current dependencies on/for this PR:

main

PR #5

PR #6   👈

This comment was auto-generated by Graphite.


403 Permission Denied (uploading to account)

Me and @Brottweiler got that error when trying to upload to an account today, it was definitely working a few days ago. Now the 403 response says the token is invalid.
Current workaround:

delete the credentials.conf
remove imgur_acct_key and imgur_secret from settings.conf
follow the instructions when running imgur-screenshot -c

The only change to handling authorized uploads was commit dc8eaa6638c357a4fa5e2ee2960726ee4c92e942, I don't think that should have made anything worse.
Let's see if this happens again with new generated auth keys.

Following the instructions fixed it. Can you add it to the README for more visibility until its fixed?
Thanks.

This workaround fixed it for limited time, today right now I am getting the same error again.

Can confirm. I'm not sure why this happens. Will have to read the API docs again but I feel like it's imgur causing it.

Btw, you only need to go to https://imgur.com/account/settings/apps and press "generate new secret" and  copy paste the secret to your settings.conf, then you need to run it once to get the PIN and enter that. You don't need to register new application.

Breaks for me every day.

Looks like it is a problem with token expire time. My solution is add to cron
0 0 * * * imgur-screenshot -c > dev/null

The bug has actually been there since the feature was introduced in b71655eb03220382379d2d0c6490fc61dd0910b1, but another bug #70 caused it to have no impact because it would refresh the token every time the script ran.

Fix released in v1.7.4.
Homebrew: https://github.com/Homebrew/homebrew-core/pull/874

Looking at it again, the commit doesn't fix it. In bash, a return value of 0 means true / success, anything greater is false / error:
[ 1 -gt 0 ]
# return value: 0

However, bash's arithmetic evaluation returns the result of the computation. In cases of a binary result, it returns it as 0 or 1. Thus 0 means false and 1 means true:
$((1 > 0))
# return string: 1

I.e. the expiry check was correct. I have reverted to 1.7.3 locally and double checked token expiry detection works as intended.
But I'm also able to upload to my account. Is this still an issue for others as of today?

@jomo I think I've reset my token 3 times total, but that 3rd time was maybe two-three days ago, and it's still working even before your fix today. So, right now it's working, and I'll update this if it stops working again.

v1.7.4 works fine for me while v1.7.3 losts authentication every day.

Yeah, it has worked for me quite a while now.

@ValdikSS did you use v1.7.3 recently? I'm asking because I believe the issue was a temporary problem at imgur and not related to the code. I'm using v1.7.3 locally and it correctly requested a new token after (and only after) it expired. I'm able to upload to my account without any issues.
As explained in my previous comment, v1.7.4 shouldn't handle token expiration correctly.

@jomo, yes, I switched to 1.7.4 3 days ago after 4 days of 1.7.3 which made me repeat steps to fix it every day.

Interesting, I've used v1.7.3 for about a month. The token actually expired today and I didn't have any issues so far.

@jomo I can reproduce the issue with 1.7.4. After resetting everything next day it's the same issue. Please fix it.

@jomo  Now I am using 1.7.3 and I can still reproduce this.

I've not had any problems in a long time now.

It is happening wih me for both 1.7.3 and 1.7.4. It works if I reset authentication. But every-time after the token expire, it seems it can not get new token and hence getting permission denied.

Sorry, I can't reproduce either. v1.7.3 should be working, see here.
Please try deleting your credentials.conf and remove imgur_acct_key + imgur_secret from settings.conf. Run imgur-screenshot -c, then follow the instructions and try again.

@jomo Any specific reason for removing imgur_acct_key + imgur_secret from settings.conf? Because I still want to use my own acc_key.
Update: Ok. After removing everything and reconnecting it is working for now. Hope it will still work tomorrow. Thanks.

@jomo

Please try deleting your credentials.conf and remove imgur_acct_key + imgur_secret from settings.conf. Run imgur-screenshot -c, then follow the instructions and try again.
After reconnect this error strikes again after few days.

I wanna point out that i had this same issue today, the workaround fixed it, lets see if sticks (was using it for a few weeks at most)


window type grid, newline as first character in JSON 'text' of line 0

You can almost see this issue on the documentation page Adventure sample output:
http://www.eblong.com/zarf/glk/remglk/docs.html
Even thought the window is defined as having a "height" of 1 line, and this is line 0, the line actually starts with a newline character (char[0]) and would technically require two lines:
 { "line":0, "content":[{ "style":"normal", "text":" At End Of Road
 Score: 36    Moves: 1      "}]}

It is a character before the word "At".  Is this some quirk with Glk's concept that all status lines must have a blank line at start if less than two, or just an accidental newline that RemGlk is inserting?
For clarity: I did encounter this 'first char is newline' on Git interpreter linked with RemGlk using advent.blb on the latest Github source of RemGlk and Git. I was trying to track down why my status line kept coming out two lines in height.

The first character of that status line output is a space, not a newline. The room name starts in the second column, due to the I6 library code that invokes MoveCursor(1, 2). (That's row, column in 1-based coordinates.) Therefore a space appears before the "A".

ok, thank you.  Maybe citing the doc was a bad idea. Because with a Git + advent.blb  the output isn't identical to the docs, because it has "Moves: 1" also in the status line.
This exact story file: http://mirror.ifarchive.org/if-archive/games/glulx/advent.blb md5sum: 8e3ace47c395bf1acd4c5f02dd6befa3
I'm using a somewhat modified stdout on Android and I guess it's possible it's altering this one character - but I've run thousands of story lines through RemGlk and it would seem odd to pick this exact textgrid line every time in different stories to be a newline.
I spent at least 3 days getting stdin to be reliable on all the models of test phones I had.  I guess I'm going to have to add a string hex dump down in the RemGlk C code to a file or something and see if I can figure out how I'm getting a newline only on line 0 of grid windows.

I have been testing with glulxe rather than git.
There should never be a newline in the text line data of any window at all. I'd say it was a bug in git, but it shouldn't be possible for remglk to output that, really.

ok, sounds like stdout writing or reading corruption on my part. I'll close and re-open if I find otherwise. Thank you.

Ok. I just re-tested it with git+remglk, and got the same result. That section of the output looks like
 {"id":21, "lines": [
  { "line":0, "content":[{ "style":"normal", "text":" At End Of Road                                      Score: 36    Moves: 1      "}]}
 ] },

ok, I found a bug in my code that was adding the 81st char before the char1 space. Sorry about this. Thanks for looking into it.


[5.2] Inaccurate Chinese character length problem

mb_strlen second parameter to add utf-8 support Chinese string length calculations.

This is a breaking change?

sorry,I do not understand your development process, but I hope this issue can be resolved.

I think you can set the default multibyte encoding http://php.net/manual/en/function.mb-internal-encoding.php

But other mb_ function you specify the encoding, why the need to manually set it?


Document how max levels are determined for multilevel DWT and SWT routines.

This PR improves the dwt_max_levels docstring as discussed in #306
Thanks to @JohnLunzer for the suggested improvement to the phrasing used here.
I opted to enable the MathJax extension in Sphinx to get a LaTeX rendering for the docs (and IPython interactive help):

A  drawback, is that the LaTeX code for the above is pretty illegible when perusing a plain text docstring, so I left a more legable plain text rendering as a comment ( https://github.com/grlee77/pywt/blob/2a7ef8857046355e6a7f969a16df8295ead596e7/pywt/_dwt.py#L44) .  Is this a reasonable compromise?
closes #306

Codecov Report

Merging #309 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
+ Coverage   86.22%   86.25%   +0.03%
==========================================
  Files          21       21
  Lines        3092     3092
  Branches      535      535
==========================================
+ Hits         2666     2667       +1
+ Misses        373      372       -1
  Partials       53       53

Impacted Files
Coverage Δ

pywt/_extensions/_swt.pyx
75.96% <ø> (ø)
:arrow_up:

pywt/_dwt.py
98% <ø> (ø)
:arrow_up:

pywt/_extensions/c/wt.template.c
87.38% <0%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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

Is this a reasonable compromise?

Yes I think so. Another think to consider is the LaTeX style. Especially things like \left can often be left out to make the plain text version more readable.

Docs build fine for me and text looks good, so merging. Thanks @grlee77


api/state/daily Is out of date with spreadsheet

Looks like the data in the api is not being updated correctly. The website shows newer data than the APIs. See the Georgia data for an example.

The website uses the API for its data. I'm thinking this was a cache issue on an edge server. Please reopen if you notice it again. Thanks!

Thanks


Update whatsapp.js

string conversion to send messages which contains only number
fix for issue #43

if this PR will pass can someone create a new Release to get a propper update for my supervised home assistant instance. Currently version 1.3.3 is installed.
Otherwise i have to reinstall the addon.

@giuseppecastaldo
Van you merge this PR and create a new release?
Thank you :)


Deprecate LegacyGsonTypeHandlerAdapter

Deprecates LegacyGsonTypeHandlerAdapter and removes its usages.
The following usage(s) in UIFormat
https://github.com/MovingBlocks/Terasology/blob/5d17468a400f9335ec36fff4beeee348848adcf8/engine/src/main/java/org/terasology/rendering/nui/asset/UIFormat.java#L108-L110
are replaced with a single GsonTypeSerializationLibraryAdapterFactory.
The usage in UISkinFormat
https://github.com/MovingBlocks/Terasology/blob/5d17468a400f9335ec36fff4beeee348848adcf8/engine/src/main/java/org/terasology/rendering/nui/skin/UISkinFormat.java#L69
is replaced with a GsonTypeHandlerAdapterFactory with a ColorTypeHandler.
Known issues
TypeSerializationLibrary logs a few errors when the UI is loaded, but these errors are irrelevant and everything functions as expected (needs confirmation). This error logging will be stopped in a future PR.

Hooray Jenkins reported success with all tests good!


[FLINK-13859][docs] JSONDeserializationSchema spell error

What is the purpose of the change
The doc spell error
Brief change log
JsonDeserializationSchema -> JSONDeserializationSchema
Verifying this change
none
Does this pull request potentially affect one of the following parts:
none
Documentation
none

@1u0 thanks to review


mzR not installing in devel_mscore2:

There is a build error:
** testing if installed package can be loaded
Error: package or namespace load failed for ‘mzR’
in dyn.load(file, DLLpath = DLLpath, ...): unable to load shared object '/usr/local/lib/R/site-library/mzR/libs/mzR.so': /usr/local/lib/R/site-library/mzR/libs/mzR.so:
undefined symbol: _ZNK2H58CommonFG15getObjnameByIdxB5cxx11Ey

in
https://hub.docker.com/r/bioconductor/devel_mscore2/builds/b5m6spuitr4cbpvbx4envjr/

The symbol should be in libhdf5_cpp.a from Rhdf5lib:
sneumann@msbi-corei:~$ nm /vol/R/R-devel/library/Rhdf5lib/lib/libhdf5_cpp.a | grep _ZNK2H58CommonFG15getObjnameByIdxB5cxx11Ey
0000000000003056 T _ZNK2H58CommonFG15getObjnameByIdxB5cxx11Ey

or demangled H5::CommonFG::getObjnameByIdx[abi:cxx11](unsigned long long) const
Remember, man nm  says: "T" The symbol is in the text (code) section., i.e. present in the lib*.a.
Instead, inside the container the closest thing I get is:
00000000000038e0 T _ZNK2H510H5Location15getObjnameByIdxB5cxx11Ey

or demangled H5::H5Location::getObjnameByIdx[abi:cxx11](unsigned long long) const.
There has been talk that "HDF5 class hierarchy was slightly changed in hdf5-1.10.1"
(https://github.com/mannau/h5/issues/56), but the container is still on 1.10.0-patch1+docs-3+deb9u1.
So no idea yet. Yours, Steffen

Fixed since mzR_2.15.5 ,
build succeeded in https://hub.docker.com/r/bioconductor/devel_mscore2/builds/ba3j6esc5lunh3p8v5zgxjh/


Wrong example command line

The examples format shown how to disable or enable this setting per product is wrong.
This is not correct
-Value "Disabled"
-Value "Enabled"
this is correct
-Enabled $false
-Enabled $true
Found the correct answer from here https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7234

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 5082f7ed-ad40-04c1-5c6c-59e67b253257
Version Independent ID: 3d8b109d-285c-952f-2fa4-9fa244c2a3f9
Content: Use AllowSelfServicePurchase for the MSCommerce PowerShell module
Content Source: microsoft-365/commerce/subscriptions/allowselfservicepurchase-powershell.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@hharsunen Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/12889
Once the author approves, the changes will be live.
Thanks
Sri

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


fix: revert revert and add ssh back into guide docs

#4042 broke our docs website, #4060 reverted the change.

There are 2 problems in there. There are 4 back ticks at the end of one block. Changing the line “output” to  “console” should fix it

Thanks to @mterhar, I think we found the issue.
We're working with @BrunoQuaresma to prevent issues like this in the future.
Related:

https://github.com/cdr/code-server/pull/4042
https://github.com/cdr/code-server/pull/4060
https://github.com/cdr/code-server/discussions/4057

Checked the preview. Looks good! @BrunoQuaresma any idea why the docs are making this shield round?
When I look at it in the browser, it's square like the others. See here

@jsjoeio I will check it. Probably a random CSS class. Also, when adding badges, it is recommended to make it a link, even if it is only an image. When it is an image, you can make it a link pointing to the image itself. It is how GH does to make their badges inline. You can inspect how our README.md looks like on GH inspecting the badges.

@jsjoeio also, looks like we are not closing a code block here:

It looks like the bash block is inside of another block.

When it is an image, you can make it a link pointing to the image itself. It is how GH does to make their badges inline. You can inspect how our README.md looks like on GH inspecting the badges.

Ah okay, I'll match that.

also, looks like we are not closing a code block here:

Nice catch! I'll fix that too

When it is an image, you can make it a link pointing to the image itself. It is how GH does to make their badges inline. You can inspect how our README.md looks like on GH inspecting the badges.

Ah okay, I'll match that.

I just checked and these are already links 🤔 Not sure what else we can do then.

Ready for another review! @BrunoQuaresma

@jsjoeio I sent a PR just fixing the link on the terminal badge.

@jsjoeio I need to tell, the ngrok URL changes every time, as well as the port. One time you run ngrok, you get 0.tcp.ngrok.io and port 19028, but the next time you run it, you will get 2.ngrok.io and 10320, and it goes on. So, add in this case, to the ngrok guide where the port is mentioned and add a note that the port and the URL always changes so you have to use the URL and the port given on the terminal.

Ahh good call @im-coder-lg - thanks for mentioning that!

:+1:


ImportError: /../lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found

Environment:

System: WSL, Ubuntu 22.04.2 LTS
Python 3.7.17, 3.8.18, 3.9.18

Reproduce:
conda create -n football python=3.8
conda activate football

sudo apt-get install git cmake build-essential libgl1-mesa-dev libsdl2-dev \
libsdl2-image-dev libsdl2-ttf-dev libsdl2-gfx-dev libboost-all-dev \
libdirectfb-dev libst-dev mesa-utils xvfb x11vnc python3-pip

python3 -m pip install --upgrade pip setuptools psutil wheel
python3 -m pip install gfootball

python3 -m gfootball.play_game --action_set=full

And then I get the error below:
libgame.so _gameplayfootball.so && popd
Traceback (most recent call last):
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/runpy.py", line 185, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/runpy.py", line 111, in _get_module_details
    __import__(pkg_name)
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/__init__.py", line 17, in <module>
    from gfootball.env import scenario_builder
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/env/__init__.py", line 21, in <module>
    from gfootball.env import config
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/env/config.py", line 27, in <module>
    import gfootball_engine as libgame
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball_engine/__init__.py", line 43, in <module>
    from _gameplayfootball import *
ImportError: /home/carlos/anaconda3/envs/football/bin/../lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball_engine/_gameplayfootball.so)

I've tried methods suggested in 'GLIBCXX_3.4.30' not found for librosa in conda virtual environment (after trying out a lot of solutions)?, Anaconda ImportError: /usr/lib64/libstdc++.so.6: version GLIBCXX_3.4.21 not found: conda install -c conda-forge libstdcxx-ng=12 and conda install -c conda-forge libstdcxx-ng, respectively.
However, I get another error:
libgame.so _gameplayfootball.so && popd
Traceback (most recent call last):
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/runpy.py", line 185, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/runpy.py", line 111, in _get_module_details
    __import__(pkg_name)
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/__init__.py", line 17, in <module>
    from gfootball.env import scenario_builder
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/env/__init__.py", line 21, in <module>
    from gfootball.env import config
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball/env/config.py", line 27, in <module>
    import gfootball_engine as libgame
  File "/home/carlos/anaconda3/envs/football/lib/python3.8/site-packages/gfootball_engine/__init__.py", line 43, in <module>
    from _gameplayfootball import *
SystemError: initialization of _gameplayfootball raised unreported exception

Would you please help me solve the problem? Thanks!

I have a similar problem, and my error message is
Traceback (most recent call last):
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/runpy.py", line 188, in _run_module_as_main
mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/runpy.py", line 111, in _get_module_details
import(pkg_name)
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/site-packages/gfootball/init.py", line 17, in
from gfootball.env import scenario_builder
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/site-packages/gfootball/env/init.py", line 21, in
from gfootball.env import config
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/site-packages/gfootball/env/config.py", line 27, in
import gfootball_engine as libgame
File "/home/yuechao/anaconda3/envs/grf/lib/python3.9/site-packages/gfootball_engine/init.py", line 43, in
from _gameplayfootball import *
ImportError: /lib/x86_64-linux-gnu/libwayland-client.so.0: undefined symbol: ffi_type_uint32, version LIBFFI_BASE_7.0
Hope to receive a reply

I encounter the same issues. Anyone solve this?

Same issue.
(grf) zxh@ubuntu-AS-4124GS-TNR:~$ python3 -m gfootball.play_game --action_set=full Traceback (most recent call last):   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/runpy.py", line 185, in _run_module_as_main     mod_name, mod_spec, code = _get_module_details(mod_name, _Error)   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/runpy.py", line 111, in _get_module_details     __import__(pkg_name)   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/site-packages/gfootball/__init__.py", line 17, in <module>     from gfootball.env import scenario_builder   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/site-packages/gfootball/env/__init__.py", line 21, in <module>     from gfootball.env import config   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/site-packages/gfootball/env/config.py", line 27, in <module>     import gfootball_engine as libgame   File "/home/zxh/anaconda3/envs/grf/lib/python3.8/site-packages/gfootball_engine/__init__.py", line 43, in <module>     from _gameplayfootball import * SystemError: initialization of _gameplayfootball raised unreported exception

How to solve this issue
I have solved this problem in Ubuntu,  miniconda3. First, you need to check whether your system has the correct version of "GLIBCXX". You can do this by running:
strings /usr/lib/x86_64-linux-gnu/libstdc++.so.6 | grep GLIBCXX

This should display a list of GLIBCXX versions. You need to check whether you have GLIBCXX 3.4.30. If it exists, you can proceed to the next step. Otherwise you probably need to update your system.
Next, check what's wrong with your conda environment by running:
strings /path_to_your_environment/lib/libstdc++.so.6  | grep GLIBC

In my case, conda only has GLIBCXX 3.4.29. This is why it doesn't work. We then need to link the library. We can run:
ln -sf /usr/lib/x86_64-linux-gnu/libstdc++.so.6 /path_to_your_environment/lib/libstdc++.so.6

After doing this, the engine can be compiled. (For me, I need to run "pip install six" before it works).
Other problems you might encounter
Here are other information you might need. This package is old, so we need to install older version of gym. To do this, we need the older versioins of pygame,  pip and setuptools. Try running:
pip install pygame==2.1.2
pip install --upgrade pip setuptools==57.5.0
python -m pip install pip==20.0.1
pip install gym==0.20.0


Reduce amount of stuff shown by default for details pane

Is your feature request related to a problem? Please describe.
The details pane can get pretty crowded when there are a few things in each section. This is really really rough on mobile, but also can be bad on desktop

Describe the solution you'd like
Each section should probably just be put into a tab, and when a tab is selected, a param should be set for it in the URL (see #214)

Maybe something like vscode

Questions:

Should the tabs be vertical or horizontal? It seems like a lot to double up the [details|views] tabs with this

Describe alternatives you've considered
Additional context
Technical ideas

It might be a bit awkward to have these tabs in addition to the details/views tabs.
I realize that vscode actually handles this by defaulting all tabs to collapsed, and this might be an easier solution for us:

With that, we probably would want:

scrolling per section + no scrolling for the whole pane, instead of no scrolling per section + scrolling for the whole pane
indication of whether or not there's something to look at in the section; either use outline instead of solid for the section icon, or use "(x)" where x is the number of things in that section (e.g. x justification nodes, comments, research nodes). could do both. icon is probably slightly better because it pairs with the functionality that indicators on nodes use
clicking a node indicator to open the details pane with the relevant section expanded

Think something like this seems pretty good:

Notes:

probably fine to have empty space when the selected details section doesn't take up all the space

Questions:

should tabs have the names of their sections as well?

try it out and see how it looks

what to do with type-specific sections e.g. Question's "Asks About" and "Potential Answers"?

seems ok to put this under the "Notes" section?

maybe section would ideally be named differently but then the indicator might be awkward

maybe there could be a separate section for "associations"? that probably wouldn't need indicator anyway

should there be an option to still show all sections at once?

seems like showing one at a time is better for reading
but showing all seems better for editing, to retain one-click-access for adding any kind of nuance
for now it's probably fine to not have an option and see how it feels, adding later if need be
it's possible that adding any kind of nuance can be streamlined without showing all sections at once, e.g. via a + button that expands into each kind of nuance

how should this look on mobile?

think there probably should be a separate ticket for showing the pane on mobile... probably should take up the entire screen, since there's not really enough space for it to be useful otherwise

This made the ontology topic not show justification/research anymore in its default view. I think that's ok, but we should have a way to share a Topic with a specific diagram part's details tab selected, e.g. to specifically show a part's Questions. Created https://github.com/amelioro/ameliorate/issues/608 for this.


Png files update

PNG files etx. unified (lowercased)

OPS Build status updates of commit c3909d9:
:clock10: Preparing: average preparing time is 41 sec(s)

OPS Build status updates of commit c3909d9:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit c3909d9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/actionable-messages/email-dev-dashboard.md
:white_check_mark:Succeeded

docs/actionable-messages/images/adaptive-action-set.png
:white_check_mark:Succeeded

docs/actionable-messages/images/adaptive-column-pixel-width.png
:white_check_mark:Succeeded

docs/actionable-messages/images/adaptive-image-properties.png
:white_check_mark:Succeeded

docs/actionable-messages/images/adaptive-visibility-collapsed.png
:white_check_mark:Succeeded

docs/actionable-messages/images/adaptive-visibility-expanded.png
:white_check_mark:Succeeded

docs/actionable-messages/images/connector-dev-dashboard/connect-screenshot.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/create-webhook.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/group-menu.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/incoming-webhook.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/postman-setup.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/select-group.png
:white_check_mark:Succeeded

docs/actionable-messages/images/get-started/webhook-url.png
:white_check_mark:Succeeded

docs/actionable-messages/images/invoke-mha-add-in.png
:white_check_mark:Succeeded

docs/actionable-messages/images/reference/expanded.png
:white_check_mark:Succeeded

docs/actionable-messages/images/reference/heroimage.png
:white_check_mark:Succeeded

docs/actionable-messages/images/reference/heroimagebanner.png
:white_check_mark:Succeeded

docs/actionable-messages/images/reference/ismultiselect.png
:white_check_mark:Succeeded

docs/actionable-messages/images/reference/startgroup.png
:white_check_mark:Succeeded

docs/actionable-messages/images/teams-actionable-message.png
:white_check_mark:Succeeded

docs/actionable-messages/images/teams-context-menu.png
:white_check_mark:Succeeded

docs/actionable-messages/invoke-add-in.md
:white_check_mark:Succeeded

docs/actionable-messages/message-card-reference.md
:white_check_mark:Succeeded

docs/actionable-messages/send-via-connectors.md
:white_check_mark:Succeeded

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

docs/add-ins/contextual-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/add-from-file.png
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/addin-prompt-configure.png
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/addin-settings.png
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/addin-taskpane.png
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/button-and-pane.png
:white_check_mark:Succeeded

docs/add-ins/images/addin-tutorial/yeoman-prompts.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-detected-entity-card.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-detected-entity-highlight.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-sideload-desktop-add-from-file.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-sideload-desktop-store.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-sideload-web-add-from-file.png
:white_check_mark:Succeeded

docs/add-ins/images/outlook-sideload-web-manage-integrations.png
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-button.png
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-task-pane.png
:white_check_mark:Succeeded

docs/add-ins/images/quick-start-yo-prompts.png
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-graph-permissions.png
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-api-details.png
:white_check_mark:Succeeded

docs/add-ins/images/single-sign-on-web-details.png
:white_check_mark:Succeeded

docs/add-ins/index.md
:white_check_mark:Succeeded

docs/add-ins/quick-start.md
:white_check_mark:Succeeded

docs/add-ins/sideload-outlook-add-ins-for-testing.md
:white_check_mark:Succeeded

docs/payments/get-started.md
:white_check_mark:Succeeded

docs/payments/images/payment-form-initial.png
:white_check_mark:Succeeded

docs/payments/images/payment-pane-confirmation.png
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support terminal for displaying table when evaluating metrics

Right now the Evaluate class only support displaying in Colab. It would be nice if we also support terminal output given it is already supported in pandas.

There was a recent discussion of this, I forget where.
I think this could be nice as well.

Yeah, I think we will need this feature to display when running in CI and output to markdown or github comments

Happy to merge a PR on this, sounds like it's already in pandas


Redirect from non-ssl to ssl on any non-ssl request

Can you please add this feature as a flag. Basically what would be nice to have would be a redirect from any non-ssl request/URL to same request/URL but with SSL.

This will be covered in #1021.
If we missed anything in #1021, please comment on it.


[FEATURE] Adding join for switching channels

This pull request...

[ ] Fixes a bug
[x] Introduces a new feature
[ ] Improves an existing feature
[x] Boosts code quality or performance

Description
Added JoinCmd to switch channels when music plays for djs.
Also added intellij stuff to gitignore and shortened something in DJCommand
Purpose
If you are a dj and want to switch channels and don't have move permissions you couldn't switch the music bots channel without disconnectiong which closes the queue. Now you can just switch chanels with join!
Relevant Issue(s)
This PR closes a thing on the road map.

still waiting for new review by jagrosh c:

@UnrealValentin Hello! Are you alive & well? Apologies for the late review. There are small things that need fixing. Once they're fixed, this should be good to merge! If you wish to work on fixing these yourself, leave a quick comment. If there's no answer within the next few days, I will fix & commit these myself.

@UnrealValentin Hello! Are you alive & well? Apologies for the late review. There are small things that need fixing. Once they're fixed, this should be good to merge! If you wish to work on fixing these yourself, leave a quick comment. If there's no answer within the next few days, I will fix & commit these myself.

yes I am. just busy a bit lately ^^ Forgot this pr still existed tbh ^^
But I gonna continue work on this this week

But I gonna continue work on this this week

Apologies, but I already have made & committed all the changes I requested recently. As far as I know, this PR does not need any changes. Jagrosh will double-check & merge if everything looks good.

Sounds good 👍

Should we do it like that?


User invitation expiry

The user invite to join an organization or to collaborate on a project has a time limit of 6 hours. Currently, the invite expires before 6 hours.
To recreate the error-

Click on Your organizations
Click on Manage for a specific organization
Click on Teams
Click on Manage for a specific team
Click on Members
Scroll down to the bottom of the page and select a user in the drop down list
Click on Add

The user invitation expires before 6 hours. If the invite has expired, it should not be possible to revoke the invite.

On the first part, this requires a little investigation to reproduce.
The second part seems to be a lacking equivalent to the task we use to update RoleInvitation statuses:
https://github.com/pypi/warehouse/blob/39daea188d2e1e6494c50753cd48cb5a8da3e8c4/warehouse/manage/tasks.py#L18-L31


Update paella player to 6.2.4

This PR updates paella to 6.2.4 (that version fixes #1123)
Also reverts the temporal fix (PR #1131), as it is causing problems with @dependabot
Your pull request should…

[x] have a concise title
[x] close an accompanying issue if one exists
[x] be against the correct branch (features can only go into develop)
[ ] include migration scripts and documentation, if appropriate
[x] pass automated testing
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits
[ ] have appropriate tags applied

Sorry @lkiesow, now the click button should work

Works now → merged


Removed: the type specification from right side when using generics

Removed type specification from the planet pojos package

hi @ymaheshwari1,
are you planing to do this for more packages? Project wide "Inspect and Transform" using netbeans gives me over 150 hits for "can use diamond". I noticed you did something similar recently in #77. Maybe we can aggregate more in this PR, otherwise this won't ever end :)
feel free to set this PR to draft if you planing to add more so that it isn't integrated before its done.

hi @ymaheshwari1,
are you planing to do this for more packages? Project wide "Inspect and Transform" using netbeans gives me over 150 hits for "can use diamond". I noticed you did something similar recently in #77. Maybe we can aggregate more in this PR, otherwise this won't ever end :)
feel free to set this PR to draft if you planing to add more so that it isn't integrated before its done.

Sure, I will convert the PR to draft and will update the same in other packages as well.

Hi @ymaheshwari1
Is it done?

Hi @ymaheshwari1
Is it done?

Hi,
Some of the packages are yet to be updated, I will complete them as soon as possible.

Hi @ymaheshwari1
Is it done?

Hi,
I have removed all the type specifications please review.

@ymaheshwari1 excellent, thank you. Changes look good.
If @snoopdave is ok with this i would like integrate this PR. Should we squash all 21 commits to one or leave it as is with a simple merge?

@mbien looks good. Usually, I prefer to squash but I don't have a strong opinion.

squashed + merged. thank you for the contribution @ymaheshwari1


LAP: usehooks-ts

A library acceptance proposal for usehooks-ts
While building the membership management system, I ended up using the above library to handle debouncing useDebounce.
The library provides several other useful hooks, docs here. The proposal is to assess which hooks are useful and if this should be a standard issue in the Anomaly web development pack.

Should cross reference #21 to see if react-query can help.

rooks seems to be a popular group of hooks that developers are using. It's worth investigating the library as well


Improve the IOError condition description.

Important to note that the io error is the cause the cause for pausing, and that it comes from the low-levels.
What this PR does / why we need it:
Explain a condition better
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

@awels @mhenriks @acardace  is this something you can help with?

/retest
/lgtm
/approve

/override pull-kubevirt-apidocs

/retest


IObit Uninstaller Pro Crack <IP_ADDRESS> + Key Download [Latest]

IObit Uninstaller Pro Crack <IP_ADDRESS> + Key Download [Latest]
IObit Uninstaller Pro Crack Free Download is a powerful uninstall application that helps you easily remove unwanted programs, even if Windows “Add or Remove Programs” fails. In addition to removing unwanted applications, it’s easy to scan and delete residues. The most appealing part is that it generates a recovery image before each uninstall. Using the newly added toolbar to uninstall modules, it helps you completely remove unwanted toolbars.
IObit Uninstaller Pro full version crack can powerfully remove them and delete all the leftovers, including the registry and file directories of some stubborn antivirus programs. It will continue the unfinished uninstall process caused by the system reboot; and it can also create a system restore point before each uninstall to prevent accidents. Sometimes users forget to delete the rest after uninstalling.
Download Now

✅▶▶DOWNLOAD IObit Uninstaller Pro Latest
✅▶▶MIRROR IObit Uninstaller Crack


Automata example: show how to run it

It can be a bit surprising that the binary is under build/src.
There does not seem to be an example on how to run it. If one tries the binary as is, it throws an ugly error directly from C++.

It can be a bit surprising that the binary is under build/src.

This is on purpose, as one of the tasks in the exercise is to browse the build tree and find how it is laid out in comparison with the source tree. I decided not to set CMAKE_RUNTIME_OUTPUT_DIRECTORY and co. to avoid overloading the number of new variables presented at once.

There does not seem to be an example on how to run it. If one tries the binary as is, it throws an ugly error directly from C++.

I am amending this.

+1 for point 1.


function tests don't work with new hypar-cli... generator github repo is out of date?

I might just be missing something but... while upgrading to utse the new elements library I needed to upgrade a function by re-runninghypar init and ran into problems. Thought I could get through it, realized I needed new Amazon references for using Amazon.S3 to work.  Guessed and got the latest using dotnet add package AWSSDK.S3
However, there are still things wrong with my now outdated test methods.  This test method is from the old generator and doesn't work.  I think this Generator repo is old.  Generating new hypar funcs also doesn't work (which makes sense, because it just clones this repo)
If i'm right that this repo is out of date updating this repo would be great.  It looks like I should start using output.Result for the computed properties as well as the model.
However, these changes get me to a successful build, but not a successful test. The JSON deserialization of the sample data at the top of the test fails.  It's not in the correct format.  GUIDs where there should be Int64s... bummer.
here's the code where I replaced output["Element_count"] with output.Result.Element_count and similar changes to get a good build
        [Fact]
        public void Test()
        {
            // Execute the function.
            var func = new Function();
            var output = func.Handler(_data, null);

            Assert.NotNull(_data.Model);

            // Check that the computed values are as expected.

            var computed = (Dictionary<string,object>)output["computed"];
            Assert.True(Math.Abs((double)computed["element_count"]) > 0.0);

            var model = Model.FromJson(output["elements"].ToString());
            model.SaveGltf("model.gltf");

Apologies for the delay in picking this up. I believe this has now been brought up to date with the latest and greatest that the CLI creates. Can you verify that this repo's test functions as expected?

ok, looks better / different, not able to get a test though...

hypar new mister-web-system
remove the .git stuff, don't need in my repo situation
remove license, preview, readme stuf
edit hypar.json
run hypar init src  NOTE: instructions should say to run hypar init with src
restart VSCode, seems like build can't find the new Input.g.cs files
!!! function.csproj wont' build  !!!
problems in Function.g.cs

missing using Elements.Serialization.glTF; and using Elements.Serialization.JSON;
-wrong serialization method.  It doesn't seem ready for Elements version 0.2.2. line 56  model.SaveGlb is deprecated, and the new method is model.ToGlTF with a parameter for serializing to binary.

problems in Input.g.cs

wrong namespace for ModelConverter line 55.  needs to be Elements.Serialization.JSON.ModelConverter

The HyparDotnetStarter.cs file exists... with errors... and I don't think I need it, so I deleted it.  would be nice if it wasn't there.

function.csproj builds, yay!

function-test.csproj won't build :-(
problem in FunctionTest.cs

the namespace of my function is missing, had to add it to access Input class
needed to comment out the assert in the test function, because output.Area is no longer valid with my edits to hypar.json

can now run dotnet test from top directory without build errors
Test fails!!!

steping through as far as line 39 in Function.g.cs error in UploatGlTFAsync occurs because output.Model was not set in my function...

for my test function I simply had return new Output(input.Model, 0); passing a model straight through, this resulted in null value error
creating a new model and adding that to the output instead seemed better... this particular problem makes sense now, although currently there's no way there is a model on the "Input" so maybe that property shouldn't exist?

now have a problem still in model.ToGlTF() lin 57 in Function.g.cs, expcted buffer size > 0 ???

Ok, add a space to the fake model I've created, that gets to next error

part of filepath writing to ../../../live/models/model.glb doesn't exist.  yup, the models directory doesn't exist

make that directory

Test Passes!!!  phew!
In the end, got it to run.  These are all my notes of the process I went through hope it helps.  Adding actually interesting function code next :-)

Thanks Eric! I think many of these issues arise from the fact that we don’t generate test infrastructure, which would be a really smart thing for us to do ;) Code generation by the CLI and cloning the repo from github are currently in opposition. I’ll smooth this out.
Ian Keough
@ikeough

On Apr 10, 2019, at 11:23 AM, wynged<EMAIL_ADDRESS>wrote:
ok, looks better / different, not able to get a test though...
hypar new mister-web-system
remove the .git stuff, don't need in my repo situation
remove license, preview, readme stuf
edit hypar.json
run hypar init src NOTE: instructions should say to run hypar init with src
restart VSCode, seems like build can't find the new Input.g.cs files
!!! function.csproj wont' build !!!
problems in Function.g.cs
missing using Elements.Serialization.glTF; and using Elements.Serialization.JSON;
-wrong serialization method. It doesn't seem ready for Elements version 0.2.2. line 56 model.SaveGlb is deprecated, and the new method is model.ToGlTF with a parameter for serializing to binary.
problems in Input.g.cs
wrong namespace for ModelConverter line 55. needs to be Elements.Serialization.JSON.ModelConverter
The HyparDotnetStarter.cs file exists... with errors... and I don't think I need it, so I deleted it. would be nice if it wasn't there.
function.csproj builds, yay!
function-test.csproj won't build :-(
problem in FunctionTest.cs
the namespace of my function is missing, had to add it to access Input class
needed to comment out the assert in the test function, because output.Area is no longer valid with my edits to hypar.json
can now run dotnet test from top directory without build errors
Test fails!!!
steping through as far as line 39 in Function.g.cs error in UploatGlTFAsync occurs because output.Model was not set in my function...
for my test function I simply had return new Output(input.Model, 0); passing a model straight through, this resulted in null value error
creating a new model and adding that to the output instead seemed better... this particular problem makes sense now, although currently there's no way there is a model on the "Input" so maybe that property shouldn't exist?
now have a problem still in model.ToGlTF() lin 57 in Function.g.cs, expcted buffer size > 0 ???
Ok, add a space to the fake model I've created, that gets to next error
part of filepath writing to ../../../live/models/model.glb doesn't exist. yup, the models directory doesn't exist
make that directory
Test Passes!!! phew!
In the end, got it to run. These are all my notes of the process I went through hope it helps. Adding actually interesting function code next :-)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

you've done great work with the function and test generator with the CLI


[Feature Proposal] a PhoneField for a good presentation of phone numbers

Hi !
I've wrote a field for phone numbers which displays it as a  link.
If the number is from the phone locale it'll be formatted like a national number, if not international formatting will be set. The default phone locale is 'FR' (french) but it can be changed by setting a prop (see the example below).
It's based on the  phone lib of halt-hammerzeit based itself on the "Google Android's famous libphonenumber library"
For more information about the lib : https://github.com/halt-hammerzeit/libphonenumber-js
Example :
<PhoneField source="personal_phone" />
renders a french number as national:
06 23 45 67 89  => 06 23 45 67 89
and an US number as international:
+1<PHONE_NUMBER> => +1<PHONE_NUMBER>
<PhoneField source="personal_phone" locale="US" />
// renders a french number as international:
+336 23 45 67 89 => +336 23 45 67 89
and an US number as national:
(234) 456-5656 =><PHONE_NUMBER>
So what do you guys think, is this something you might be interested in adding to AOR?

That's great! It's very specific though and I remember libphonenumber to be quite heavy. Would you mind publishing it as an add-on ?

And make a documentation pull request?

Thank's !
Ok then I'll do that.


expired token still fetching data

i am still facing same issue before, the plugin does not validate token from database.
Application.php in cakephp 5 using authentication/authorization plugin for handing the login/logout
$service->loadIdentifier('Authentication.Token');             $service->loadAuthenticator('Authentication.Token', [                 'queryParam' => 'token',                 'header' => 'Authorization',                 'tokenPrefix' => 'Bearer'             ]);
in my database table there is token_expiration field and token field even token_expire field but still does not invalidate token which one have already expired

Do you have your code somewhere online? I would take a look.

Login Method

Token generate method for 1 minute only for testing purpose

Application.php
Authorization

please sir help me.. why my token not expired after one minute

Do you have your code somewhere online? I would take a look.

i shared screenshots

You do not have to use $service->loadAuthenticator('Authentication.Token')
This is automatically done by the plugin.
Instead of this you need a config/apiTokenAuthenticator.php file described here: https://github.com/rrd108/api-token-authenticator?tab=readme-ov-file#changing-the-default-settings
I think the way you do is overwrite the plugins behaviour.


fix: add virtual environment creation for local development, include …

Include help option for makefile
Virtual environment creation

Thanks but we don't plan to add specific support for venv in the Makefile for now.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(package): add main from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/*-test.mjs' as in template
chore(package): set $.ava.extensions='mjs' as in template
chore(package): remove unknown value for main ({{main}})

Pull Request Test Coverage Report for Build 1642

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.399%

Totals

Change from base Build 1639:
0.0%

Covered Lines:
301

Relevant Lines:
302

💛  - Coveralls


Splitting functions into separate library

What are your thoughts on splitting the Perl components out to a separate Perl module (e.g. ArkServerTools.pm, which would have functions that return the desired information), and perhaps also either split the bash functions out into a separate ArkServerTools.sh or convert some of those functions to Perl.
Something similar was originally posited by @pandabear41 in pandabear41@bc9efa8.

I think it could work

This probably would be best left until v1.6 - we probably want to try to get v1.5 stabilized.

I think it might be best to push this back to a later version, and look at releasing v1.6


Dynamic theme capability

What problem does this feature solve?
Following https://github.com/ant-design/ant-design/issues/9366 (which has a solution on react aparently).
It would be nice to have such flexibility on angular implementation too, as I'm not a less expert I didn't find a way to do that without ejecting webpack (which is impossible with current angular-cli version).
What does the proposed API look like?
It would simply provide a way to call less to modify variables and have it loaded properly on runtime.

we already support it, you can check the bottom right of our site.
you can use less.js to change theme if you want.

@vthinkxie I see the implementation for the docs site and if feels quite hard to maintain, especially because you seem to import the whole ant style in this file because it needs to be regenerated in the browser, but then I can't make sure I'm up to date in this file with the latest ant design changes.

As you see, it is the only way to dynamic change style in runtime without eject angular-cli to webpack.

Yeah that was my initial guess, but I thought maybe I was missing something, thanks for the details.
N.B: do you have a script or something to generate color.less based on current version of the theme? it seems like I don't have the same theme with the same colors if I'm using this file instead of the classic theme system.

@Supamiu you can check antd-theme-generator package in npm


Move dendrocoil out of mod

Placeholder issue for future feaure.
The dendrocoil is a very niche block that is mostly used by developers and is wasted on the majority of players.  It creates a hard dependency(however proxied) to computer craft that developers have to deal with.
This block will be moved and made available in the "Magic Cracker Factory" mod(currently unreleased on curseforge but available on github).  MCF contains other creative mode only computercraft peripherals for use in admin world creation and is a good fit for it.

The Dendrocoil has been moved to Magic Cracker Factory.  Freeing the base code from a computercraft dependency makes it easier for Dynamic Trees add-on developers.


Update Gradle Enterprise Gradle plugin to v3.12

This PR updates the GE Gradle plugin to v3.12
Note that build scans cannot be published at the moment, because https://ge.gradle.org/ is not yet updated to GE 2022.4.
Publishing build scan...

The request was rejected.
Gradle Enterprise plugin version 3.12 is newer than the newest version supported by Gradle Enterprise 2022.3.7 which is 3.11. Please update to a newer version of Gradle Enterprise.

The new GE Gradle plugin has integrated support for test retry. But this means, that the Test Retry Gradle plugin has to be removed from this build build. For the time being, we disable the test retry functionality in GE by setting the system property gradle.enterprise.testretry.enabled to false.
See also: https://docs.gradle.com/enterprise/gradle-plugin/#test_retry_migration

@bot-gradle test this

For the time being, we disable the test retry functionality in GE by setting the system property gradle.enterprise.testretry.enabled to false.

Is there a downside to this? Are you going to take care of removing the test retry plugin?

For the time being, we disable the test retry functionality in GE by setting the system property gradle.enterprise.testretry.enabled to false.

Is there a downside to this? Are you going to take care of removing the test retry plugin?

As test retry in Gradle Enterprise is a new feature, make use of this feature should be done in a separate PR.
I don't see a downside to use the bundled version of test retry though - apart from losing some flexibility when you'd like to use a new feature, which was already released in the Test Retry Gradle plugin, which was not delivered in a new GE plugin version yet.

@bot-gradle test and merge


[PRTL-60] security audit

– Rename function arguments
– Add Purchase event
– Improved checks when buying trees

Work on the branch is still in progress

@Maxirus2 I think that's it :)


e2e directory inside

@bond007007 - what is your issue?

I'm going to close this issue, b/c it does not describe a bug.  Feel free to reopen if you have details on the issue.


Marks for sliders end up as unassociated text in screen readers

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behaviour 😯
If you use labeled marks on a  component they will be spoken by the screen reader before the sliders and will not be associated with anything.
Expected Behaviour 🤔
I don't know of any way to associate the marks with the slider. And most likely the thumb has minimal information and the marks are an aid for visual users to provide expanded information. For screen reader users I think the best practice is for the aria-valuetext to contain the full information that may be truncated in the thumb label for visual users.
In this case the marks are only there for visual users and we should probably use aria-hidden={true} to hide the marks from screen readers.
Steps to Reproduce 🕹
For example if you have a slider with
My slider
<Slider value={0} aria-labelledby="a" marks={[{value: 60, label: '1:00'}, {value: 120, label: '2:00'}]} />.
The screen reader will read to you heading level 2, My slider, followed by One zero zero Two zero zero, then finally be followed by the slider with something like Zero, My slider, slider.

@dantman I can reproduce the behavior with VoiceOver:

I agree I don't see how the marks announcement would help visually, cognitive, hearing or motored impaired users.
I would propose the following diff:
diff --git a/packages/material-ui/src/Slider/Slider.js b/packages/material-ui/src/Slider/Slider.js
index fbfb32f3d..e580fd1e7 100644
--- a/packages/material-ui/src/Slider/Slider.js
+++ b/packages/material-ui/src/Slider/Slider.js
@@ -657,6 +657,7 @@ const Slider = React.forwardRef(function Slider(props, ref) {
               })}
             />
             <span
+              aria-hidden
               style={style}
               className={clsx(classes.markLabel, {
                 [classes.markLabelActive]: markActive,

Do you want to submit your first pull request? :)

Do you want to submit your first pull request? :)

Not right now, I'm leaving for a long family trip with no internet very soon.

Enjoy your summer trip!

@oliviertassinari can I make the pr ?

Sure :)


Optimize code for firefly marquee

Optimize code for firefly Interactive marquee

Resolves: MWPW-138426
Test URLs:

Before: https://main--cc--adobecom.hlx.page/drafts/ruchika/firefly/firefly?martech=off
After: https://stage-FF--cc--adobecom.hlx.page/drafts/ruchika/firefly/firefly?martech=off

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

Comparison is base (f6d219d) 68.96% compared to head (e3d6c98) 77.68%.

Files
Patch %
Lines

...ativecloud/features/firefly/firefly-interactive.js
89.28%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #114      +/-   ##
==========================================
+ Coverage   68.96%   77.68%   +8.72%
==========================================
  Files           2        2
  Lines         261      242      -19
==========================================
+ Hits          180      188       +8
+ Misses         81       54      -27

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

Additional comments -

Arrange media by viewport size
Remove font size and add bocy-class where possibe (eg .enticement-text)
Combine the handleclick event listener

All has been done except point 2 because we needed same size 28/35 for enticement text in tablet and desktop however in milo heading xl value changes for desktop view. hence using the direct values now.


Integrate networking security

K3S is using flannel under the hood. Maybe we can leverage cilium support
https://cilium.io/

Cilium has an issue when running on WSL2 due to missing kernel features. It can be fixed by recompilinh your WSL kernel but that leads us a bit too far


UI error in Presets options

I don't know if this happens only to me.
in the PRESETS section, when creating a new preset or editing an existing one.
by activating "Display Normally-Hidden Options?" It can no longer be disabled and clicking anywhere will open the window to load the "prompt Images"
The "Choose File" button will always be activated no matter where you click
The solution is in the CSS sheet
Assign to:

the value:
position: relative !important;
sorry for my bad English

Already reported in #316 and fixed, so you should update your installation 😄

Already reported in #316 and fixed, so you should update your installation 😄

I update from the update-windows.bat file
is that correct? Why does the problem keep appearing to me?
Or do I have to update it through some other command?
From already thank you very much

... well, now I'm very confused. It was recently fixed in the prior issue, I tested and confirmed at the time... but it is indeed broken again. I don't know how that happened, but, uh -- re-fixed, and re-tested to once again be fixed.
And yes update-windows.bat is the correct way to update


Update "Report an Issue" template

We got feedback that characters like < made it harder to read the comments they surround (Fixes https://github.com/Microsoft/vscode-azureappservice/issues/782) and @StephenWeatherford has mentioned he wants the call stack several times. Here's my reasoning for other changes:

Added 'IMPORTANT' callout as suggested in above issue
Made the comments "hidden" once the issue is posted with <!--. This matches the VS Code issue template and most other templates I could find. I think the comments just make the issue ugly
Add "1." and "2." to maybe (?) make it more obvious that there's a bunch of whitespace that needs to be addressed. Also matches VS Code issue template.

Example:

NOTE: I decided not to add call stack to unit tests. The stack is dependent on the machine and the compiled code, so it just didn't seem worth it.
Fixes https://github.com/Microsoft/vscode-azuretools/issues/152

Great, I just added callstacks, too. :-(


lodash: signatures of the method _.union changed

https://lodash.com/docs#union
https://lodash.com/docs#_ (description of the chainable wrapper)

thanks mate!


WIP - test collapsibles

FYI: This repository will be archived shortly. Please send this PR against https://github.com/datawire/ambassador/tree/master/docs.


Add semi-automatic derivation for ADT enumerations

Currently the derived codecs for case objects result in either an empty object, or a field with an empty object value, depending on whether the value is statically typed as an ADT leaf or root:
scala> import io.circe.generic.auto._, io.circe.syntax._
import io.circe.generic.auto._
import io.circe.syntax._

scala> sealed trait Base; case object Foo extends Base
defined trait Base
defined object Foo

scala> Foo.asJson.noSpaces
res0: String = {}

scala> (Foo: Base).asJson.noSpaces
res1: String = {"Foo":{}}

This will probably always be the default behavior, since it's always safe and unambiguous, and since having derived encoders that return JSON values that aren't objects complicates things significantly.
There is a common use case for case objects that can be pretty easily supported, though. If we're defining an enumeration as a sealed trait of case objects, we often want the values to be represented as strings. This PR adds support for this representation to io.circe.semiauto:
import io.circe.{ Decoder, Encoder }
import io.circe.generic.semiauto.{ deriveEnumerationDecoder, deriveEnumerationEncoder }
import io.circe.jawn._, io.circe.syntax._

sealed trait Suit extends Product with Serializable
case object Club extends Suit
case object Heart extends Suit
case object Spade extends Suit
case object Diamond extends Suit

object Suit {
  implicit val decodeSuit: Decoder[Suit] = deriveEnumerationDecoder
  implicit val encodeSuit: Encoder[Suit] = deriveEnumerationEncoder
}

And then:
scala> decode[Suit]("\"Club\"")
res0: Either[io.circe.Error,Suit] = Right(Club)

scala> List(Heart, Club, Diamond).asJson
res1: io.circe.Json =
[
  "Heart",
  "Club",
  "Diamond"
]

If we had any case classes that extended Suit, the derivation would fail to compile.
Note that this must be explicitly invoked via deriveEnumerationX, and will not collide with any automatically derived decoders.

Superseded by #429.


Catch the rain that drips from the sky in your bucket! You need to drink something right?
How do you play your game?
A and D to move your bucket and catch the rain! Your goal is to get as high a score as you can. use the I key to reset your score.
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[x] The image is in the /games/img directory.
[x] The name of the image matches the name of your file.

Thanks for your PR!

Thanks for PR! Where did you hear about Sprig?

Hi! I heard about it from a friend and I found it very interesting, so i decided to give it a go since i already had some JS knowledge.

Sorry me for slow answer! Is it endless, right?

Sorry me for slow answer! Is it endless, right?

Yes!

Hi Lucas! Can I claim my sprig?

Hi Lucas! Can I claim my sprig?

Happy new year! Congrats:) If you're a teen, you can get your sprig here! https://sprig-order.hackclub.dev/
Hack Clubbers would love to see this! Just post a link and 1-2 sentences talking about your game to our #ship channel:)

Thanks so much! Happy new year!

Happy new year! Congrats:) If you're a teen, you can get your sprig here! https://sprig-order.hackclub.dev/
Hack Clubbers would love to see this! Just post a link and 1-2 sentences talking about your game to our #ship channel:)

Thanks so much! Happy new year!


Break Archiver.SaveFile() method down into smaller ones

Breaks SaveFile() down into

SaveFile()
reloadFileIfChanged()
saveChunk()
waitForResults()
updateNodeContent()

I think this increases readability drastically. The old function was doing way too many things on way too many different levels of abstraction and was therefore hard to understand.

GitHub is terrible again and displaying the diff...
Best to just look at the new file to see what's happening now.

I'm having a look. Btw: using https://github.com/codegram/vim-codereview together with https://github.com/chrisbra/vim-diff-enhanced makes viewing such changes much more pleasant:
Before (with myers diff algorithm)

After (with patience diff algorithm)

The pictures have been stitched together from two screenshots each, and I'm not using Terminus at 8 points ;)


Hardware input is not implemented

Support for System.Device.Gpio is not implemented yet. This supports a momentary push button for selection and a double sided momentary toggle switch for menu movement.

In-progress


Python Version Ask

Sir, what version of Python are you using and does it matter?

use 3.7 or higher


Cannot run mittens with tensorflow 2.1

After installing tensorflow 2.1, I cannt run GloVe any more - "fit" function gives the following errors:
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
2020-03-01 17:54:01.126392: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-03-01 17:54:01.127263: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.127355: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.127418: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128038: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128072: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128111: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128134: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128168: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.128184: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.129233: E tensorflow/stream_executor/cuda/cuda_blas.cc:238] failed to create cublas handle: CUBLAS_STATUS_NOT_INITIALIZED
2020-03-01 17:54:01.129268: W tensorflow/stream_executor/stream.cc:2041] attempting to perform BLAS operation using StreamExecutor without BLAS support
Traceback (most recent call last):
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1367, in _do_call
return fn(*args)
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1352, in _run_fn
target_list, run_metadata)
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1445, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.InternalError: 2 root error(s) found.
(0) Internal: Blas GEMM launch failed : a.shape=(4, 1), b.shape=(1, 4), m=4, n=4, k=1
[[{{node Tensordot_1/MatMul}}]]
[[Sum/_5]]
(1) Internal: Blas GEMM launch failed : a.shape=(4, 1), b.shape=(1, 4), m=4, n=4, k=1
[[{{node Tensordot_1/MatMul}}]]
0 successful operations.
0 derived errors ignored.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/richard/Documents/AI/XCS224U/similarity_methods.py", line 28, in
embeddings = glove_model.fit(cooccurrence)
File "/home/richard/anaconda3/lib/python3.7/site-packages/mittens/mittens_base.py", line 240, in fit
X, fixed_initialization=fixed_initialization)
File "/home/richard/anaconda3/lib/python3.7/site-packages/mittens/mittens_base.py", line 84, in fit
fixed_initialization=fixed_initialization)
File "/home/richard/anaconda3/lib/python3.7/site-packages/mittens/tf_mittens.py", line 83, in _fit
self.log_coincidence: log_coincidence})
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 960, in run
run_metadata_ptr)
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1183, in _run
feed_dict_tensor, options, run_metadata)
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1361, in _do_run
run_metadata)
File "/home/richard/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1386, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InternalError: 2 root error(s) found.
(0) Internal: Blas GEMM launch failed : a.shape=(4, 1), b.shape=(1, 4), m=4, n=4, k=1
[[node Tensordot_1/MatMul (defined at /anaconda3/lib/python3.7/site-packages/mittens/tf_mittens.py:151) ]]
[[Sum/_5]]
(1) Internal: Blas GEMM launch failed : a.shape=(4, 1), b.shape=(1, 4), m=4, n=4, k=1
[[node Tensordot_1/MatMul (defined at /anaconda3/lib/python3.7/site-packages/mittens/tf_mittens.py:151) ]]
0 successful operations.
0 derived errors ignored.
Original stack trace for 'Tensordot_1/MatMul':
File "/Documents/AI/XCS224U/similarity_methods.py", line 28, in
embeddings = glove_model.fit(cooccurrence)
File "/anaconda3/lib/python3.7/site-packages/mittens/mittens_base.py", line 240, in fit
X, fixed_initialization=fixed_initialization)
File "/anaconda3/lib/python3.7/site-packages/mittens/mittens_base.py", line 84, in fit
fixed_initialization=fixed_initialization)
File "/anaconda3/lib/python3.7/site-packages/mittens/tf_mittens.py", line 56, in _fit
self._build_graph(vocab, initial_embedding_dict)
File "/anaconda3/lib/python3.7/site-packages/mittens/tf_mittens.py", line 151, in _build_graph
tf.tensordot(self.bw, tf.transpose(self.ones), axes=1) +
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/ops/math_ops.py", line 4106, in tensordot
ab_matmul = matmul(a_reshape, b_reshape)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/util/dispatch.py", line 180, in wrapper
return target(*args, **kwargs)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/ops/math_ops.py", line 2798, in matmul
a, b, transpose_a=transpose_a, transpose_b=transpose_b, name=name)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/ops/gen_math_ops.py", line 5626, in mat_mul
name=name)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/framework/op_def_library.py", line 742, in _apply_op_helper
attrs=attr_protos, op_def=op_def)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 3322, in _create_op_internal
op_def=op_def)
File "/anaconda3/lib/python3.7/site-packages/tensorflow_core/python/framework/ops.py", line 1756, in init
self._traceback = tf_stack.extract_stack()

double submissions - please see previous issue.


Add Support for Searching in Encrypted Columns

Package
filament/filament
Package Version
v3
Laravel Version
v10
Livewire Version
v3
PHP Version
PHP 8.3
Problem description
There is currently no built-in support or documented method in Filament for searching in encrypted columns. When attempting to add a Closure function to the searchable function, I am unable to achieve the desired result.
Expected behavior
I request the addition of a feature or a documented method in Filament to search in encrypted columns. This should include an improvement that allows defining custom search queries in encrypted columns, for example by adding a Closure function to the searchable function.
Steps to reproduce
Make a field encrypted in the model and then attempt to search for that field in the table.
Reproduction repository
https://github.com/batuhanhazarguler/filamentphp-encrypted
Relevant log output
No response

This is not a bug. Please do not open issues for feature requests.
Please investigate ways to search encrypted data outside of Filament, the same techniques in Eloquent apply. However, I do not think it's something that is easily possible.


[Bug Report]:  panic when handle css contains calc

System Info
System:
OS: macOS 12.5.1
CPU: (10) arm64 Apple M1 Pro
Memory: 2.95 GB / 32.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.18.0 - ~/Library/Caches/fnm_multishells/71636_1680514176616/bin/node
npm: 8.19.2 - ~/Library/Caches/fnm_multishells/71636_1680514176616/bin/npm
Browsers:
Chrome: 111.0.5563.146
Firefox Developer Edition: 111.0
Safari: 15.6.1
npmPackages:
@rspack/cli: workspace:* => 0.1.4
Details

when i removes calc from css the panic disappear
.App-header {
  background-color: #282c34;
  min-height: 100vh;
  display: flex;
  flex-direction: column;
  align-items: center;
  justify-content: center;
  font-size: calc(10px + 2vmin);
  color: white;
}

Reproduce link
No response
Reproduce Steps
cd examples/cra && pnpm build

because of https://github.com/swc-project/swc/blob/ae2362e97bb358a95b99b5c6a4d2255b488c2349/crates/swc_common/src/syntax_pos.rs#L580-L592, called by https://github.com/swc-project/swc/blob/ae2362e97bb358a95b99b5c6a4d2255b488c2349/crates/swc_css_minifier/src/compressor/calc_sum.rs#L956

introduced by https://github.com/swc-project/swc/pull/6291


Support variable interpolation in Request URL path

This PR partially solves https://github.com/emqx/emqx-web-hook/issues/224.

Hi @z8674558 It seems a very cool feature, thanks for your contribution. We will review it later :)

Hi, @z8674558 We Plan to release this PR to v4.2.3
Could you rebase your branch from dev/v4.2.3 first ? That will keep this PR clear enough :)