kdeApplications.mbox-importer: fix hydra build

Motivation for this change
Build succeded locally but failed on hydra with "log limit exceeded". Disable verbose logging.
/cc ZHF #36453 - please backport
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3
shrinking /nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3/bin/mboximporter
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3/bin
patching script interpreter paths in /nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3
checking for references to /build in /nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3...
postPatchMkspecs
postPatchMkspecs
/nix/store/c47nwhz2pyhi5qmhs5knagyh58j0pyma-mbox-importer-17.12.3

Success on x86_64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/df33a4yf1rkzrhliq93bchm2dk2hnf62-mbox-importer-17.12.3
shrinking /nix/store/df33a4yf1rkzrhliq93bchm2dk2hnf62-mbox-importer-17.12.3/bin/mboximporter
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/df33a4yf1rkzrhliq93bchm2dk2hnf62-mbox-importer-17.12.3/bin
patching script interpreter paths in /nix/store/df33a4yf1rkzrhliq93bchm2dk2hnf62-mbox-importer-17.12.3
checking for references to /build in /nix/store/df33a4yf1rkzrhliq93bchm2dk2hnf62-mbox-importer-17.12.3...
postPatchMkspecs
postPatchMkspecs

Success on x86_64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3
shrinking /nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3/bin/mboximporter
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3/bin
patching script interpreter paths in /nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3
/nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3
checking for references to /build in /nix/store/kqda7l31wlwvrm1wdlbg7yylb489cc2v-mbox-importer-17.12.3...
postPatchMkspecs
postPatchMkspecs

Success on aarch64-linux (full log)
Attempted: kdeApplications.mbox-importer
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3
shrinking /nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3/bin/mboximporter
strip is /nix/store/j7d4mr0ikv974ig7yzhknpsq288js4bs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3/bin
patching script interpreter paths in /nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3
checking for references to /build in /nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3...
postPatchMkspecs
postPatchMkspecs
/nix/store/m9k8wnvwjdprcl51l7r3k35m9y5n9gr5-mbox-importer-17.12.3

Thanks!


Add ability to scroll to a section/question

ideally using hashtag location (#path/to/node)

don't think it's feasible any more as Enketo Express uses hashtags as enketo ID in offline-capable views


[DO NOT MERGE] Sample Fix for S2N_ERR_INVALID_SIGNATURE_SCHEME

DO NOT MERGE
Resolved issues:
resolves #2197
Description of changes:
Pick a certificate signature algorithm in TLS 1.3 even if client does not indicate support.
Call-outs:
Testing:
Unit tests
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2222 into main will decrease coverage by 0.58%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2222      +/-   ##
==========================================
- Coverage   83.57%   82.99%   -0.59%
==========================================
  Files         249      249
  Lines       16611    16627      +16
==========================================
- Hits        13883    13799      -84
- Misses       2728     2828     +100


Unable to release

Discussion https://github.com/webdriverio-community/wdio-electron-service/issues/37#issuecomment-1163550458_

I'll go revert some recent commits and see if I can release from a branch, there hasn't been much added.  I thought it was the upgrade to N16 but downgrading didn't have an effect.


Throw error for all undefined categories

Currently only throws an error for the first undefined category it finds
Could be useful to show all categories that are not defined

Problem with Imparse, not Constellation


"replication_pad1d" not implemented for 'Half'

This error report was obtained when I executed codes with function keras.layers.MaxPooling1D dynamically, but statically with shape output.
The version info is keras 3.5.0 with PyTorch 2.4.0
import os
import re
import torch
import numpy as np
os.environ['KERAS_BACKEND']='torch'
import keras

layer = keras.layers.MaxPooling1D(
    pool_size=2,
    strides=1,
    padding="same",
    data_format="channels_first",
    trainable=True,
    dtype="mixed_float16",
    autocast=True,
)

result_static = layer.compute_output_shape([3, 5, 4])

result_dynamic = layer(
    inputs=np.random.rand(*[3, 5, 4]),
)

For the static result, I got shape (3, 5, 4). However, for the dynamic one, I obtained this error report
...\torch\nn\functional.py:4552, in pad(input, pad, mode, value)
   4545         if mode == 'replicate':
   4546             # Use slow decomp whose backward will be in terms of index_put.
   4547             # importlib is required because the import cannot be top level
   4548             # (cycle) and cannot be nested (TS doesn't support)
   4549             return importlib.import_module('torch._decomp.decompositions')._replication_pad(
   4550                 input, pad
   4551             )
-> 4552 return torch._C._nn.pad(input, pad, mode, value)

RuntimeError: Exception encountered when calling MaxPooling1D.call().

"replication_pad1d" not implemented for 'Half'

I don't think it's reasonable for dynamic and static output to be different.
Furthermore, dynamic error messages are not clear enough, at least there is no Half in the codes.

This issue is at the level of the backend, PyTorch. Sometimes, some operations aren't supported by a specific backend, but are supported by others, and thus the backend-agnostic part of the code (like compute_output_shape) should not raise an error.


Typo in 7.0.6

chapter 3.3.3 Latter-day Saint Ordinances
"an earlier version of GDCOM" should read: an earlier version of GEDCOM

Fixed in ed16216b1778aa7f35df9778e8046e68e54e51ee


Locks not being cleared on SIGKILL

Our jobs are remaining locked after the worker process receives the SIGKILL (in our case from docker stop after the grace period elapses).
I'm not sure if this is by design or not, the README states (may being the keyword):

the process and may result in long-running jobs remaining locked until Delayed::Worker.max_run_time has elapsed.

In my tests they always remains locked when a job is running and Delayed also doesn't terminate with a SIGTERM.

Here is simple rake task I used to simulate the issue from Delayed
class TestWorker
  def start
    trap('TERM') { quit! }
    trap('INT') { quit! }

    500.times do |i|
      puts "Run #{i}"
      sleep 1.second
    end
  ensure
    on_exit!
  end

  def quit!
    puts 'quit!'
  end

  def on_exit!
    puts 'on_exit!'
  end
end

namespace :test do
  desc "Test signal interupts"
  task work: :environment do
    TestWorker.new.start
  end
end

If you run that as as a rake task, it won't terminate on a SIGTERM, only SIGKILL which won't execute the ensure block which in Delayed is what unlocks the jobs.

Whereas if I explicitly `exit` in the `quit!` method it works as expected.
class TestWorker
  def start
    trap('TERM') { quit! }
    trap('INT') { quit! }

    500.times do |i|
      puts "Run #{i}"
      sleep 1.second
    end
  ensure
    on_exit!
  end

  def quit!
    puts 'quit!'
    exit
  end

  def on_exit!
    puts 'on_exit!'
  end
end

namespace :test do
  desc "Test signal interupts"
  task work: :environment do
    TestWorker.new.start
  end
end

What's the recommended way to gracefully shut down a running job when the process is terminated?   Should a job implement its own traps or is there a hook that Delayed offers?

Rails 7.0.7
Ruby 3.2.2
Delayed 0.5.0

Hi @suwyn,
Thanks for reaching out!

I've modified your example a bit to add a break if stop? check:
class TestWorker
  def start
    trap('TERM') { quit! }
    trap('INT') { quit! }

    loop do
      3.times do |i|
        puts "Run #{i}"
        sleep 1
      end
      break if stop?
    end
  ensure
    on_exit!
  end

  def stop?
    @stop
  end

  def quit!
    @stop = true
    puts 'quit!'
  end

  def on_exit!
    puts 'on_exit!'
  end
end

This more closely mirrors how delayed responds to a SIGINT or SIGTERM, by attempting to finish the current set of jobs assigned to the thread pool before exiting. (I'm simulating that with 3 "jobs," repeated forever by the "worker" in a loop.)

Here's a sample output with a SIGINT (Ctrl+C) sent during the second set of jobs:
$ rake test:work
Run 0
Run 1
Run 2
Run 0
^Cquit!
Run 1
Run 2
on_exit!

It loops around to pick up more jobs until it receives SIGINT/SIGTERM, at which point it will finish the current pool and exit cleanly instead of picking up more jobs. However, if it receives a SIGKILL instead, it will immediately exit, without attempting to clean anything up. (This signal is special and is handled by the kernel/OS -- the worker never actually receives it so there would be no way for it to react.)

What's the recommended way to gracefully shut down a running job when the process is terminated?

Right, so to answer your question, the way to gracefully shut down is to first send a SIGTERM, give jobs a chance to finish, and then send a SIGKILL later if/when you need to fully stop the process. If you're not seeing SIGTERM produce a clean exit within a reasonable amount of time, it's likely due to long-running jobs. So, this means that jobs need to be short-lived enough to complete gracefully within that waiting period, otherwise the worker will exit ungracefully and the new workers will wait until locked_at + Delayed::Worker.max_run_time in order to be sure that no other worker is running the job.
In general, I'd suggest deconstructing jobs into shorter units of work, but keep in mind that once you send the SIGTERM, you know that the worker won't pick up any new jobs, so—depending on your deployment infrastructure—you could wait a very long time before sending a SIGKILL! (Perhaps even the entire max_run_time, at which point you'll know for sure that all jobs have either completed or timed-out-with-cleanup.)

Should a job implement its own traps or is there a hook that Delayed offers?

I hadn't really considered this before. Generally I think we've found that keeping max_run_time configured to the default of 20 min (or less)—and waiting after SIGTERM for jobs to finish gracefully—has produced the best overall results (in addition to making sure that jobs are all idempotent & re-runnable). Even if a worker can't exit gracefully, the longest we'd have to wait for that job to be picked up again is 20 minutes (and this typically only impacts longer-running jobs where we don't expect fast turnaround anyways).

Thanks for the explanation @smudge it all make sense.
You're correct in that the culprit for us is a long running job and that it should be broken down into shorter units of work and to keep max_run_time sensible.   We had been delaying that work and kept bumping the max_run_time up to compensate :fearful: and given that we're cleaning up containers 60 seconds after a deploy those jobs are staying locked/stalled for up to 3hours before they get cleared and picked up again.
While we do have an option to trap the SIGTERM in the job itself, it would feel cleaner if that came from the Job API (e.g. perhaps rescue_from) but I find myself agreeing with you again here and want to avoid that complexity.
tl;dr - We'll decompose our long running job so that it runs in multiple units, allowing us to keep max_run_time low enough to allow containers to be cleaned up in a timely manner.   Thanks!


[GenericPoseGraph] Use ceres::GradientChecker to check gradients.

Ba-Dum-Tss!

@wally-the-cartographer merge

Merge requested by authorized user pifon2a. Merge queue now has a length of 2.


Update com_github_curl to curl-7_83_0 (current: curl-7_82_0)

Fixes: #21031
Signed-off-by: Faseela K<EMAIL_ADDRESS>

cc @phlax

@moderation @phlax does the change look okay?

lgtm @kfaseela - leaving this for @moderation as he is assigned - cc @htuch as he may want to land sooner

/lgtm deps


VSCodeInsider_Settings_AI4D : Inappropriate toolbar structure provided for the "More Actions" control.

Evironment Destails

VSCode Version: 1.41.0-insider (user setup)
OS Version: Win10

GitHubTags: #A11yMAS; #A11yTCS; #Win32; #DesktopApp; #A11y_VSCodeEditor; #MAS4.1.2; #A11ySev2; #Benchmark
Steps to Reproduce:

Launch Visual Studio Code application.
Navigate to the "Settings" button in the left navigation bar and select "Settings" from the flyout that appears.
Select the "Extensions" option under the "Commonly Used" setting and expand.
Select any of the extensions from the list then details would appear at the right.
Launch the AI4D and inspect the content with the panel.

Actual:
The "More Actions" control is structured in a toolbar which is not appropriate as this control is a menu item and the toolbar is structured for a set of buttons.
This issue is observed in both the screen readers i.e. JAWS and NVDA.
Expected:
The "More Actions" control should not be structured in a toolbar and it should not be announced for screen reader users.
User Impact:
If any content is not structured properly then the screen reader users will have difficulty in understanding the content present on the screen.
Attachment:

Does this issue occur when all extensions are disabled?: Yes

It's essentially a toolbar with one button in it. It's the same as all other "..." type toolbar overflow menus. I'll see if there's a way to simplify it though.

@roblourens @isidorn is there something we want to do here? This issue is out of SLA so we need to report an update back to the accessibility team.

Settings editor has been restructured. We did not get negative user feedback regarding this.
Closing as no longer applicable / will not fix


Make sftp defaults and setup smarter

There are a number of defaults and behaviours of sftp that could work better.

If the port option is removed, default to the default ssh port (22).
Paths should accept ~ to refer to the current user's home directory, especially privatekey so keys in ~/.ssh/ can easily be used without hardcoding the absolute path.
When not specified user should default to the current user's name like ssh does.
When privatekey is not set remote-ftp should attempt to use the same key ssh uses (typically ~/.ssh/id_rsa) instead of allowing the connection to fail with an "Available authentication methods: publickey".
For settings like user, privatekey, etc... that are left as an empty string in the file created by remote-sftp an empty string should be considered equivalent to not being set. To disable the smart behaviour an explicit boolean false should be required.

+1

I mainly use FTP and sometimes SFTP when I absolutely need it. @dantman you seem like someone that deal with SFTP quite a lot. Are you up to the task and implementing some of the tweaks you are suggesting?


No data received for 5 minutes

Hey there,
I'm monitoring 2 database instances. 1 is working fine, but the other one is showing this alert: "No data received for 5 minutes". What does it mean? I'm sure that both of my instances are working fine. What should I do? Do I have to restart the zabbix dba service?

@ttamirt
Hello,
It looks like alive check doesn't work properly.
Do you receive any data from this database? Does alive has data (1 or 0) on "Latest data" tab?
Thanks.

It was still throwing "PGA Almost fully committed.", "Archivelog generation is too fast" notifications. So I think it was receiving data. I've restarted "/sbin/service zdba" it's back to normal now. I didn't check the alive on Latest data. thanks anyway.


[FEA] handle peer timeouts in shuffle

When UCX starts supporting peer error handling, this task is to pipe that in and cause a RapidsShuffleFetchFailedException or a RapidsShuffleTimeoutException.
This will help us report to the user, and the MapOutputTracker more accurate information at least for debugging/logs. Current logic in Spark indicates that a fetch failure from a peer will cause all output for such peer to be invalidated, but that certainly is not the case with the external shuffle service, but the error handling for non-ESS cases may change in the future, so we'll be prepared if we are implementing FetchFailedException as specified.

This depends on UCX 1.10.


io.copy() got rbd: ret=22, Invalid argument

	image,err := rbd.OpenImage(suite.storage.ioctx, "d1","")
	require.NoError(suite.T(), err)
	defer image.Close()
	lfile,err := os.Create("/tmp/d1.rbd")
	require.NoError(suite.T(), err)
	defer lfile.Close()
	wrriten,err := io.Copy(lfile,image)
	fmt.Println("Get File wrriten ",wrriten ," data to /tmp/d1.rbd")
	require.NoError(suite.T(), err)

Got
Get File wrriten  104857600  data to /tmp/d1.rbd
    TestCephRBDStorageSuite/TestGetFile: ceph_rbd_test.go:56:
        	Error Trace:	ceph_rbd_test.go:56
        	Error:      	Received unexpected error:
        	            	rbd: ret=22, Invalid argument
        	Test:       	TestCephRBDStorageSuite/TestGetFile
    --- FAIL: TestCephRBDStorageSuite/TestGetFile (0.63s)

my rbd image size is 100M ,and afert this func /tmp/d1.rbd  size is also 100M.
any good idea?

what about change
https://github.com/ceph/go-ceph/blob/d4440eb8c2966508ca4bf41240e6d3aefd144e97/rbd/rbd.go#L710-L712
to
	if ret < 0 {
        size,_ := image.GetSize()
        if uint64(image.offset) == size {
			return 0, io.EOF
		}
		return 0, RBDError(ret)
	}

Reading from an RBD image that hits EOF, should return a short read, or 0 in case nothing is read. I do not understand why you would get errno=22: Invalid argument. Unless io.Copy() causes the offset for reading to pass EOF somehow. This needs a little more debugging to understand what is going on, and wht rbd_read() returns an error that we do not expect.
Please provide the following details:

[ ] Ceph version of the client (local librdb.so)
[ ] Ceph version of the cluster (remote OSDs)
[ ] confirmation that your suggestion of the change prevents the problem
[ ] PR with a new test case using io.Copy() (expected to fail CI)

Could you try the following? This could be a workaround:
image, err := rbd.OpenImage(suite.storage.ioctx, "d1","")
lr := io.LimitReader(image, image.GetSize())
writen, err := io.Copy(lfile,image)

That way, there is an additional LimitReader that prevents going passed image.GetSize().

Thanks for your workaround! That works!
1.Client version  12.2.13-0ubuntu<IP_ADDRESS>
2.Cluster version ceph version 12.2.13 (584a20eb0237c657dc0567da126be145106aa47e) luminous (stable)
3.My suggestion works (of course, i think  your solution is better)
4.Test case need further recurrence.
This problem seems related to the image.  io.copy() other images with no problem.

I'm following up on a few issues. From my reading of this issue it sounds like the problem lies in the ceph libs rather than go-ceph. I also see that you are/were using ceph luminous client libs. If you could I'd like to suggest that you retest with newer ceph versions and if the issue does not reproduce we'll close this out.
I'll also point out that as of our v0.5.0 release, made earlier this week, we're officially deprecating luminous as a build target. In v0.6.0, due in October, building against luminous will be dropped.

Thank you！The issue did not reproduce.


When looking at downloads, it shows pins files that were downloaded over a year ago.

Description
Everytimg I open the app, I have to spend a minute scrolling to the very top because downloads always opens at items from this year.
please help
Steps To Reproduce
Click on downloads
Steps to fix
I dont know how to fix this
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.978
Windows Version
Edition	Windows 11 Pro Version	21H2 Installed on	11/7/2021 OS build	22000.978 Experience	Windows Feature Experience Pack 1000.22000.978.0
Relevant Assets/Logs

What happens if you reset the app?

I mean, I can't make a file look like it is 1 years old, but I have files from last week and it always opens at the beginning where the first files are. So, I am not exactly sure about year-old files but for weeks-old files, it always opens at the beginning. So in my case, I would consider this bug as invalid. As @yaichenbaum said, try resetting the app, and restarting your computer, as well as uninstalling and reinstalling the app. Also, I checked the bug on the PRE version. If your version is outdated, I suggest getting the latest updates first. If you are not to contribute in the community about this, do not download the PRE (preview) version as it can sometimes be unstable, (because that is the point - getting the latest updates for testing purposes).

Merging with #5825.
I believe it depends on how many items there are not if there grouped or not, I've had the scrolling issue when there was no grouping and in different layout modes. might be Files is loading items from B to Z first then items with symbols, number and A at the start which would lead items being loaded above the first viewpoint.

What happens if you reset the app?

I just did, no luck


Attaching this script removes the script from object and hides it

I've searched for an issue like this but I couldn't find anything like it. Sorry if you've seen it before.
Using Godot 3.2.3
Running Pop!_OS 20.10 based on Ubuntu Groovy
So I'm making a rather ambitious tool script to add properties for a game with waves of enemies and I want it to add new inspector properties for each part of the level. When I was working on this script it was working fine, but now after a few days it's started doing something fun. Attaching the script to an object adds the script, then removes it. It also sets the visibility to off and you can't make it visible again unless you reopen the scene. The visibility change shows in the inspector but not in the scene outline. Oddly enough you can still manipulate the properties in the inspector that the script adds.
Not sure what's going on but I managed to reproduce it in a new project and get a stack trace of the process. The trace is just me attaching the script, saving, and closing the editor.
Steps to reproduce:
Attach the script "testProperties.gd" to an object.
Project
weirdtool.zip

strace:
trace.txt

Your _get method returns false if property fails all your tests. That means every property Godot will ever try to get, will return false. That includes script and visible.

Documentation for _get() looks misleading on this. "Returns null if the property does not exist.", but you are actually expected to return null if you don't want to override the get behavior for a property.

@Zylann Thanks for the pointer. I can confirm this fixes my issue. But I agree with @KoBeWi, the language of the documentation isn't exactly clear. Though I mostly based it on code examples on the QA portal since the documentation is pretty hard to find. After searching I concluded this was undocumented and tried to figure it out on my own. Searching for "properties" or "setting properties" on docs.godotengine.org doesn't return the section on it, searching for "adding properties" returns the c++ documentation. Something this important to the engine should really have its own page on how to use it in gdscript.


⚠ PrivateHD has degraded performance

In c30dd7b, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3438 ms

Resolved: PrivateHD performance has improved in dd976b4.


Form reader support for encoding.TextUnmarshaler

This allows Context.Read to read components that support the text unmarshaler interface, like UUID libraries

Coverage decreased (-0.1%) to 94.034% when pulling aa09cd1d27d2deb3747cbc1ed485cb6c1a0dee27 on RangelReale:text-unmarshal into 4d40027cc89c573e422488c76a600c01649789cd on go-ozzo:master.

Good job! Thanks!


THRIFT-5816: Fix UUID for boost 1.86.0 (change in {{data}} member usage)

Tries to fix compiler issues with latest boost (1.86)

[X] Did you create an Apache Jira ticket?  (Request account here, not required for trivial changes)
THRIFT-5816
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[ ] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

@Jens-G
I have tested with Boost 1.86 and the following changes does fix the compilation and the unit tests still pass as expected


Add chromedriver to v16 images

We'll begin to switch to v16 images in the web CI soon and would need the chromedriver from v14 to run playwright smoke tests without any additional setup

We'll begin to switch to v16 images in the web CI soon and would need the chromedriver from v14 to run playwright smoke tests without any additional setup

That is already the case.
Did you already try it? If yes do you have a link to the failing pipeline? If not please close this issue.

Ah I see, sry for the noise


feat: use Alembic for migrations

To perform migrations automatically in the Docker image, see this example:
https://github.com/EPFL-ENAC/eesd-mast/blob/main/backend/Dockerfile#L14-L15


New network request: Canxium Testnet

Network information

Name: Cerium Testnet
ChainId: 30103
RPC http/s address: https://cerium-rpc.canxium.net
Chainlist: N/A
Blockexplorer: https://cerium-explorer.canxium.net

$ ts-node scripts/estimate.ts
{ chainId: 30103 }
{ estimate: '68131' }
{ gasPriceGwei: '200.99999999', gasPrice: '200999999990' }
✨  Done in 4.86s.

Thank you.


Feature Request: Please demonstrate how to switch versions of artifacts via environment/config

Obviously we can edit the artifacts.json files. It would be great if there is a way to specify versions for repo/share/tomcat/etc all using common environment variables that are used for things like selecting edition.

The main idea on the roadmap is that we maintain a matrix of artifacts compatible with each supported alfresco version and easily switch between them (internally tracked as OPSEXP-2849).
As of now you can edit any artifacts.json as you figured out, then the quickest way is to run ./scripts/clean-artifacts.sh && ./scripts/fetch-artifacts.sh which will cleanup every archives from the docker build directories and then fetching all the artifacts again - eventually leveraging already downloaded artifacts present in the artifacts_cache folder.

Thank you.
I suspect that clean-artifacts.sh could do with a little love. The strategy of just searching for certain file extensions without using the artifacts.json files seems a little brittle but OK. However the last line that uses xargs to rm seems very dangerous. I believe that can have dangerous consequences if there are spaces in paths. You are probably safer using -exec arguments to find or using -print0 on find and -0 on xargs.


Debug Console sometimes ignores logging of empty lines

VSCode Version: 1.11.1
OS Version: Debian GNU/Linux 8.7

Steps to Reproduce:

Write a small Node application like this:

for (let i = 0; i < 100; i++) {
    console.log();
    console.log("test", i);
}

Launch it in the debugger and check the output on the debug console. You'll see that sometimes there is no empty line between the "test" lines. The output is always different. So if you don't see the problem then try again or scroll up.

The problem can't be reproduced in the terminal, only in the debug console. console.log("") also doesn't work. Logging a single space character instead works.
Here is a screenshot showing the problem (Notice the missing line breaks between test65/test66 and test79/test80/test81):

Verified using the provided test case.


HADOOP-17936. Fix test failure after reverting HADOOP-16878 from branch-3.3

HADOOP-17936
Description of PR
After reverting HADOOP-16878 from branch-3.3, test TestLocalFSCopyFromLocal.testDestinationFileIsToParentDirectory started to fail since it depends on the behavior change in the JIRA. This fixed the test.
How was this patch tested?
Existing tests.

cc @bgaborg @steveloughran

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
9m 54s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
33m 51s

branch-3.3 passed

+1 :green_heart:
compile
18m 18s

branch-3.3 passed

+1 :green_heart:
checkstyle
0m 50s

branch-3.3 passed

+1 :green_heart:
mvnsite
1m 31s

branch-3.3 passed

+1 :green_heart:
javadoc
1m 31s

branch-3.3 passed

+1 :green_heart:
spotbugs
2m 22s

branch-3.3 passed

+1 :green_heart:
shadedclient
25m 58s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 56s

the patch passed

+1 :green_heart:
compile
17m 35s

the patch passed

+1 :green_heart:
javac
17m 35s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 46s
/results-checkstyle-hadoop-common-project_hadoop-common.txt
hadoop-common-project/hadoop-common: The patch generated 2 new + 1 unchanged - 1 fixed = 3 total (was 2)

+1 :green_heart:
mvnsite
1m 28s

the patch passed

+1 :green_heart:
javadoc
1m 31s

the patch passed

+1 :green_heart:
spotbugs
2m 30s

the patch passed

+1 :green_heart:
shadedclient
26m 51s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 45s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

162m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3478

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 53749a3faf37 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 2e3024bab21328c78bc9bb5f54f1c384ab2e6522

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/1/testReport/

Max. process+thread count
1259 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/1/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@steveloughran good question. it's not because we still have HADOOP-16878 / #1857 in the trunk so the test is passing there. Should we revert it there too? because the JIRA is re-opened.

Thank you guys. Could you merge this to branch-3.3 first, please?

Sounds good @ayushtkn . I've reverted them from trunk too, and open #3482 to fix the test there.

@dongjoon-hyun I'll wait until the fix is merged to trunk first, before merging this to branch-3.3

Got it~ No problem.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
33m 54s

branch-3.3 passed

+1 :green_heart:
compile
18m  3s

branch-3.3 passed

+1 :green_heart:
checkstyle
0m 48s

branch-3.3 passed

+1 :green_heart:
mvnsite
1m 30s

branch-3.3 passed

+1 :green_heart:
javadoc
1m 32s

branch-3.3 passed

+1 :green_heart:
spotbugs
2m 22s

branch-3.3 passed

+1 :green_heart:
shadedclient
26m 35s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 57s

the patch passed

+1 :green_heart:
compile
17m 36s

the patch passed

+1 :green_heart:
javac
17m 36s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 47s

hadoop-common-project/hadoop-common: The patch generated 0 new + 1 unchanged - 1 fixed = 1 total (was 2)

+1 :green_heart:
mvnsite
1m 29s

the patch passed

+1 :green_heart:
javadoc
1m 30s

the patch passed

+1 :green_heart:
spotbugs
2m 31s

the patch passed

+1 :green_heart:
shadedclient
26m 44s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 54s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

153m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3478

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 7ee738ceaa6e 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / b649c3c5ef1b290fc84f5dd161e808f1e29d2ab6

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~18.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/2/testReport/

Max. process+thread count
2661 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3478/2/console

versions
git=2.17.1 maven=3.6.0 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks all! I just merged this to branch-3.3


File server on Arduino board?

Hi,
Thank you for this library, it is running great. That is, if i host the file's on a traditional NAS or my pc. I want to deploy a project with a local file server for the OTA-pull file (no WAN available). So that is upload the binary once, leave the location, and whenever the clients come online, they update themselves. I was thinking on running this file server on a Arduino board. Would that be possible? Any thought on how to set this up?
Thanks in advance.
Paul

I think that would work, though it hasn't been tested outside of WiFi connection (see next issue).  I've never set up an Arduino as a file server, so can't be of much help there, but I can't see why it wouldn't work, as long as you get the the file with HTTP.


CLIMATE-653 - ocw.utils.calc_area_weighted_spatial_average is added

The current netcdf generator, dataset_processor.write_netcdf does not support multiple datasets and subregion information. A new module 'write_netcdf_multiple_datasets_with_subregions' complements write_netcdf.

Merged build triggered. Test Failed.

Merged build started. Test Failed.

Merged build finished. Test Passed.

I think the title on this one might have gotten mixed up with #217 @huikyole


Preparing for the next Explorer release

The next Explorer release will bring several changes that directly affect KinoExplorer.
This issue is to track all these changes and make sure KinoExplorer is in sync and ready for the next release.

[ ] Update DataFrame.arrange as DataFrame.sort https://github.com/elixir-explorer/explorer/pull/777
[ ] Add lists to the operations that support it
[ ] Ensure that all new types and lists are properly supported

@philss


[Test fix ] turn off hermetic build to verify the missing files in service

Turning off hermetic build to see if we can produce the artifacts that is missing in service.
OSError: Error no file named pytorch_model.bin, tf_model.h5, model.ckpt.index or flax_model.msgpack found in directory /app-root/embeddings_model.

/lgtm


fixes accuracy closes  #1351

Describe your changes
Describe any testing you have performed
Please add any demo scripts to CIL-Demos/misc/
Link relevant issues
closes #1351
Checklist when you are ready to request a review

[x] I have performed a self-review of my code
[x] I have added docstrings in line with the guidance in the developer guide
[x] I have implemented unit tests that cover any new or modified functionality
[ ] CHANGELOG.md has been updated with any functionality change
[x] Request review from all relevant developers
[x] Change pull request label to 'Waiting for review'

Contribution Notes
Please read and adhere to the developer guide and local patterns and conventions.

[x] The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in CIL (the Work) under the terms and conditions of the Apache-2.0 License.
[x] I confirm that the contribution does not violate any intellectual property rights of third parties

I think @KrisThielemans is right, however I notice that some tests are written thinking at integers, rather than floats.

https://github.com/TomographicImaging/CIL/blob/b70febf37b215dd6672ddc38b05effdecfb6a086/Wrappers/Python/test/test_DataContainer.py is full of assert_array_equal that should be checked.


feature: add support for env var interpolation in select blocks

Feature scope
Configuration (settings parsing, validation, etc.)
Description
Per discussion here:

https://github.com/meltano/meltano/issues/6727#issuecomment-1251272375

In cases where the select clause has to name a specific database or schema, and if that database or schema name differs across environments, it would make a plugin definition more DRY if we could support dynamic select clauses based on env variables.
Example case:
environments:
  - name: prod
    env:
     SOURCE_DB_ID: db1_live
  - name: uat
    env:
     SOURCE_DB_ID: db1_uat
plugins:
  extractors:
    - name: tap-mysql-db1
      inherit_from: tap-mysql
      select:
        - ${SOURCE_DB_ID}-table1.*
        - ${SOURCE_DB_ID}-table2.*
        - ${SOURCE_DB_ID}-table3.*
      metadata:
        "*":
          replication-method: LOG_BASED

Challenges
Cached catalog.json would not be compatible across environments
A challenge with this approach is that as of now, Meltano doesn't track separate environment-specific catalog files, and with the stream name having the db hardcoded as in the above example, a catalog from one environment simply would not work on any other environment.
Workaround
A workaround for this challenge would be to first deliver one or more of:

#2856
#2848

Then, the user would be informed that in order to use a dynamic catalog, we recommend disabling caching (preferred) - or else remembering to explicitly delete the cache before/after each switch between environment (much less ideal because it's prone to manual error/omission).
Related:

https://github.com/meltano/meltano/discussions/6757
#2856
#2848
https://github.com/meltano/meltano/issues/6727

Why do we want to limit this to select blocks, is there a technical reason? Seems like this is useful everywhere:
Carrying this over from Slack, Robert was looking for this, his use case:
plugins:
  loaders:
    - name: ga-target-s3-jsonl
      inherit_from: target-s3-jsonl
      env:
        PREFIX: the_prefix_here
      config:
        naming_convention: meltano/${PREFIX}/{stream}/export_date={date:%Y-%m-%d}/{timestamp}-{stream}.json

My personal use case for this would be
plugins:
  utilities:
    - ... # JUPYTER PLUGIN
      run_this_model:
              command: jupyter nbconvert ${NOTEBOOK_TO_RUN}

instead of writing multiple commands I thus can simply export a variable & then call my generic "run that model" command.

Clarification on current support for environment variable interpolation within meltano.yml here from @kgpayne.

all values read via the MeltanoYmlStoreManager are marked as expandable

My question is that if this issue is proposing adding support for this feature to select blocks, what sections of the meltano.yml currently have values read via MeltanoYmlStoreManager (i.e. have env var interpolation support)?

Still relevant

This is now supported thanks to https://github.com/meltano/meltano/pull/8268.


KeyError: 'AWP_ROOT222'

Do you have this variable set in your environment?  If not, then that's why you get the key error.  Try this:
import os
'AWP_ROOT222' in os.environ

It should report back True.

Hard to diagnose this w/o further information.


fixed layout for sw600dp and landscape category

Description:
Fixes the layout for landscape and tablets for the categories edit screen.
Screenshots:

[ ] Acceptance Criteria satisfied
[ ] Regression Testing

I compared the screen for landscape and portrait on a tablet and phone emulated device. The changes look good to me. App continues to behave as expected, and adjusts with rotation as expected.
👍


For sound slot, asset id type should be a number (or can be null) rather than string

We noticed the asset id in sound slots is typed as string but should be number. Checking the code, can also be null, so accounted for this also.
I confirm I have signed the Contributor License Agreement.

Well spotted. Thanks!


Option to create a textdraw.

An option to create a textdraw with preview directly in PawnPlus, without need to test it in SA-MP.

+1


Get rid of CMake export headers

This PR gets rid of the CMake generated ZycoreExportConfig.h header. It's annoying to deal with in non-CMake environments because CMake generates it for one specific compiler (so you can't just generate it once and use it everywhere) and also doesn't really provide much that we couldn't just as well detect at compile time.
Because the previous exports created by CMake were rather cryptic, I also took the opportunity to rename them:

ZYCORE_STATIC_DEFINE -> ZYCORE_STATIC_BUILD
Zycore_EXPORTS -> ZYCORE_SHOULD_EXPORT

There was a third define that we manually defined previously, ZYCORE_EXPORTS (mind the casing), which was seemingly unused, so I removed it.
Because we don't have CI here, I created a branch in the Zydis repo and updated the submodule. Builds just fine, results are here.
Once approved, I'll apply the same changes to Zydis.

Ah, could you please add something unrelated:
if (NOT DEFINED CMAKE_ARCHIVE_OUTPUT_DIRECTORY)
    set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR})
endif()
if (NOT DEFINED CMAKE_LIBRARY_OUTPUT_DIRECTORY)
    set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR})
endif()
if (NOT DEFINED CMAKE_RUNTIME_OUTPUT_DIRECTORY)
    set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR})
endif()

I found on Windows the .dll files are placed in sub-directories which makes it impossible to run/debug the applications in case of shared-build.
After adding the above lines, the libraries (shared and static) are always placed in the top-level binary directory.

Let's do that in a different PR after we figured out how to best do this in the discussion here.

I'm a bit late, but dropping GenerateExportHeader also means that all the header files will now be the same on all architectures, making it possible to install multiple copies of the zycore/zydis packages compiled for different architectures in Debian, useful in cross-compilation environments. Thanks!


[Doc Enhancement][Sbs tab v5][3061729]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/includes/get-started/azure-provisioning-instructions-tab.md
97
4
link
:white_check_mark:

msteams-platform/sbs-gs-javascript.yml
91
42
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Creating reference JSONS from netCDF file on Azure Blob hangs

I'm trying to use ReferenceMaker on files stored on Azure Blog storage after successfully doing the same for the same files on AWS S3. I'm using the following code:
import fsspec
from fsspec_reference_maker.hdf import SingleHdf5ToZarr
import json

url = 'az://noaa-goes16/ABI-L2-MCMIPF/2020/002/00/OR_ABI-L2-MCMIPF-M6_G16_s20200020000216_e20200020009524_c20200020010031.nc'

so = dict(
         mode="rb", anon=True, default_fill_cache=False, default_cache_type="none",
)

with fsspec.open(url, **so, account_name='goes') as inf:
        h5chunks = SingleHdf5ToZarr(inf, url)
        with open("test.json", 'w') as outf:
            outf.write(json.dumps(h5chunks.translate()).encode())

But this seems to hang forever (or at very least take longer than 20 minutes for a single file, after which I cancelled). This might be an issue with fsspec itself, as just opening a file with fsspec and xarray takes 10x as long on Azure as on AWS S3. See my related issue opened here https://github.com/intake/filesystem_spec/issues/681

(solved by adlfs upgrade)

Just to update here in case anyone else stumbles upon this issue:
intake/filesystem_spec#681 did solve the problem of the slow file read times. Running the code above (except using abfs==0.7.7,  goeseuwest instead of goes, and writing the json in mode 'wb' instead of 'w') takes ~40 seconds to generate a JSON. I chose goeseuwest since I am running on Microsoft Planetary Computer which is running in the West Europe region, so choosing the data hosted in the same region also serves to speed up access time.
import fsspec
from fsspec_reference_maker.hdf import SingleHdf5ToZarr
import json

url = 'az://noaa-goes16/ABI-L2-MCMIPF/2020/002/00/OR_ABI-L2-MCMIPF-M6_G16_s20200020000216_e20200020009524_c20200020010031.nc'

so = dict(
        mode="rb", anon=True, default_fill_cache=False, default_cache_type="none",
)

with fsspec.open(url, **so, account_name='goeseuwest') as inf:
        h5chunks = SingleHdf5ToZarr(inf, url)
        with open("test.json", 'wb') as outf:
            outf.write(json.dumps(h5chunks.translate()).encode())


Unused import Data.Char in basic-bytestring lexer

I get the following warning when using basic-bytestring, In this code.
The qualified import of ‘Data.Char’ is redundant
      except perhaps to import instances from ‘Data.Char’
    To import instances alone, use: import Data.Char()
52 | import qualified Data.Char

As far as I understand, this is a generated import.
Is this how it should be, or am I doing something wrong?

The generated code is not entirely warning-free.
In the test-suite we suppress some warnings, one of them is about unused imports: https://github.com/haskell/alex/blob/24d786c26f97d60251c9883dd79ba01cd5d664be/tests/Makefile#L42
Maybe making the code actually warning-free is too much effort.  A proper solution would be to make a DSL for lexers in Haskell, generate DSL code, extract import from this code, and then translate this into actual haskell.  However, the current implementation spits out Haskell directly and isn't suitable for introspection.
I suppose we could suppress these warnings with {-# OPTIONS_GHC -Wno-<WarningName> #-} pragmas put into the generated code.

I suppose fixing this minor issue does not warrant a release, does it?
The issue would then be fixed in the next regular release.


[rust] Some values show up incorrectly

I'm not sure how eager this project is in supporting Rust and I don't expect much at the moment, but since the Rust compiler outputs regular pdb's most of the features to work out of the box. However, in this particular case some values are displayed incorrectly.

Visual Studio for reference

My expectation would be that fence_value would be 1 (rust variables are immutable by default), and self would also contain a sensible value.

Fwiw, I've tagged this as rust but don't think this would be a rust bug per-se, though it may be triggering some corner cases here and there.

I've found rust has some issues with visual studio debugger as well. https://github.com/rust-lang/rust/issues/108972

@Jasper-Bekkers Can you try this again with the newest location info fixes in 0.9.6? Thanks.

Many relevant systems have been fixed & changed since this was posted, so this may have been resolved. I've not reproduced this exact Rust situation locally (as it seems easiest to identify in fairly contextual circumstances), so please re-open this issue if you notice the same bugs on newer builds.


Enable distributed mode on windows

Windows doesn't yet support all the features noble requires, but nothing prevents using noble "distributed mode" on it.
I fixed a couple of bugs in the distributed and websocket bindings and enabled distributed binding on Windows by default. This allows to create a noble application that runs on Windows and accepts connection from a remote client running on Linux.
You can test it by running the sample "advertisement-discovery.js" on a Windows machine and on a Linux machine start "node ws-slave.js ".
This will fix issue #201 and patches temporarily #84 until Windows will have a full BLE API.

@Loghorn thanks for the pull request, I've manually cherry picked https://github.com/Loghorn/noble/commit/44336451b4106e0cbdc33600aedd2f03b1f39f5d and https://github.com/Loghorn/noble/commit/db626602f2bbd21c3cd1ae6c25abb10e5ba6567e
What's the intended use case for this? I'm trying to understand if there is a broad scenario for usage. I'm concerned it will give false hope for noble actually supporting windows, when is doesn't.
cc/ @bragma

My use case is to run the application in windows while getting BT data from a remote device, in particular small IoT devices missing a proper debugging environment. I think Visual Studio Code is a great environment for node development, way better than using node debugger on a textual console. With this PR of @Loghorn I can debug in VSC while having ws-slave running on the device.
Please consider the PR.

@sandeepmistry my use case is similar to @bragma one. I develop for small devices without a display and debugging can be really painful. However, by running my app on windows and connecting it to remote devices using noble facilities already present and working (after the bugfix I made :-) I can leverage all the tools available on windows, like Visual Studio or the new Visual Studio Code.
Also, by enabling distributed mode on windows, you can build a working Bluetooth client that can connect to multiple remote devices or to emulated Bluetooth sensors.
I know that this does not mean that noble is fully working on windows, but this can be made clear in the documentation and/or by emitting a warning when the library is loaded on windows.

@bragma @Loghorn thanks for the explanation on your use case.
I'm going to hold off merging this until there is more demand. I anticipate this being a support nightmare ... so the pros out way the cons for me now.

@sandeepmistry I understand your concerns.
Thank you anyway for considering this PR.

@Loghorn thanks for understanding and the patches to fix up distributed/web socket mode!
Closing this for now ...


Emerging Programming Languages

covers each of the emerging programming languages, including descriptions, development details, advantages, usage examples, successful applications, and future prospects.

Emerging Programming Languages
1. Rust
Description:
Rust is a system-level programming language focused on safety, speed, and concurrency. It aims to provide memory safety without sacrificing performance.
Development Details:

Creator: Mozilla Research
First Released: 2010
Latest Version: Rust 1.51.0 (as of 2024)
Syntax: Similar to C++ but with modern language features

Advantages:

Memory safety without garbage collection
High performance comparable to C and C++
Strong support for concurrency and parallelism
Expressive and modern syntax

Usage Examples:
fn main() {
    let mut numbers = vec![1, 2, 3, 4, 5];
    for number in &mut numbers {
        *number += 10;
    }
    println!("{:?}", numbers);
}

Successful Applications:

Firefox Quantum: Parts of Mozilla's web browser
Dropbox: For file synchronization
Cloudflare: For their HTTP/2 and QUIC web servers

Future Prospects:
Rust's emphasis on safety and performance makes it ideal for system-level programming, and its adoption is growing in areas requiring reliable and efficient code, such as web browsers, operating systems, and blockchain technologies.
2. Go (Golang)
Description:
Go, also known as Golang, is a statically typed, compiled language designed by Google. It focuses on simplicity and efficiency, particularly in concurrent programming.
Development Details:

Creator: Google
First Released: 2009
Latest Version: Go 1.16 (as of 2024)
Syntax: Simple and clean, similar to C

Advantages:

Fast compilation and execution
Excellent support for concurrency with goroutines
Simple syntax and easy learning curve
Strong standard library

Usage Examples:
package main

import "fmt"

func main() {
    messages := make(chan string)

    go func() { messages <- "Hello, Go!" }()

    msg := <-messages
    fmt.Println(msg)
}

Successful Applications:

Docker: Containerization platform
Kubernetes: Container orchestration system
Hugo: Fast and flexible website generator

Future Prospects:
Go's efficiency in handling concurrent tasks makes it a preferred choice for cloud services, distributed systems, and microservices architecture. Its adoption is likely to continue growing in these areas.
3. Kotlin
Description:
Kotlin is a statically typed language that runs on the Java Virtual Machine (JVM). It is fully interoperable with Java and aims to be more concise and expressive.
Development Details:

Creator: JetBrains
First Released: 2011
Latest Version: Kotlin 1.5 (as of 2024)
Syntax: Modern and concise, reducing boilerplate code

Advantages:

Fully interoperable with Java
Concise and expressive syntax
Null safety features
Extension functions and smart casts

Usage Examples:
fun main() {
    val numbers = listOf(1, 2, 3, 4, 5)
    val doubled = numbers.map { it * 2 }
    println(doubled)
}

Successful Applications:

Android Development: Officially supported by Google as a first-class language
Spring Framework: Used in backend development

Future Prospects:
Kotlin's seamless interoperability with Java and its concise syntax make it a strong contender for Android development and backend services. Its adoption is expected to grow, especially in mobile and enterprise applications.
4. Swift
Description:
Swift is a powerful and intuitive programming language developed by Apple for iOS, macOS, watchOS, and tvOS app development. It aims to be fast, safe, and expressive.
Development Details:

Creator: Apple
First Released: 2014
Latest Version: Swift 5.4 (as of 2024)
Syntax: Modern and easy to read

Advantages:

Fast and efficient
Strong type safety and error handling
Interactive development with Playgrounds
Extensive standard library

Usage Examples:
let numbers = [1, 2, 3, 4, 5]
let doubled = numbers.map { $0 * 2 }
print(doubled)

Successful Applications:

iOS Apps: Widely used in iOS development
macOS Apps: Also used for desktop applications

Future Prospects:
Swift's strong performance and safety features, combined with Apple's backing, ensure its continued dominance in iOS and macOS development. Its potential for server-side development is also growing.
5. Julia
Description:
Julia is designed for high-performance numerical and scientific computing. It combines the ease of use of languages like Python with the performance of languages like C.
Development Details:

Creator: Jeff Bezanson, Stefan Karpinski, Viral B. Shah, and Alan Edelman
First Released: 2012
Latest Version: Julia 1.6 (as of 2024)
Syntax: Similar to MATLAB, but with more flexibility

Advantages:

High performance close to C and Fortran
Dynamic typing with a rich type system
Designed for parallelism and distributed computing
Extensive mathematical function library

Usage Examples:
numbers = [1, 2, 3, 4, 5]
doubled = map(x -> x * 2, numbers)
println(doubled)

Successful Applications:

Scientific Research: Used in various computational science fields
Machine Learning: Adoption in machine learning research and development

Future Prospects:
Julia's high performance and ease of use make it ideal for scientific computing, data analysis, and machine learning. Its adoption in academia and industry is likely to continue expanding.
6. TypeScript
Description:
TypeScript is a superset of JavaScript that adds static types. Developed by Microsoft, it aims to improve the development experience and code quality of large-scale JavaScript applications.
Development Details:

Creator: Microsoft
First Released: 2012
Latest Version: TypeScript 4.3 (as of 2024)
Syntax: Extends JavaScript with optional static types

Advantages:

Type safety and early error detection
Enhanced IDE support and tooling
Gradual adoption in existing JavaScript projects
Better maintainability of large codebases

Usage Examples:
function greet(name: string): string {
    return `Hello, ${name}!`;
}

console.log(greet("TypeScript"));

Successful Applications:

Angular: A popular web development framework
VS Code: The code editor itself is written in TypeScript

Future Prospects:
TypeScript's ability to improve JavaScript development makes it a preferred choice for large-scale web applications. Its adoption is expected to grow as more projects and frameworks embrace it.
7. Elixir
Description:
Elixir is a dynamic, functional language designed for building scalable and maintainable applications. It runs on the Erlang VM, known for its fault tolerance and distributed computing capabilities.
Development Details:

Creator: José Valim
First Released: 2011
Latest Version: Elixir 1.11 (as of 2024)
Syntax: Similar to Ruby but functional

Advantages:

Concurrency and fault tolerance
Easy to write and maintain
Strong ecosystem with the Phoenix framework for web development
Hot code swapping

Usage Examples:
defmodule Example do
  def greet(name) do
    "Hello, #{name}!"
  end
end

IO.puts Example.greet("Elixir")

Successful Applications:

Discord: The chat application uses Elixir for its real-time messaging system
Pinterest: Uses Elixir for some backend services

Future Prospects:
Elixir's strengths in concurrency and fault tolerance make it well-suited for real-time applications and distributed systems. Its adoption is likely to grow in industries that require highly reliable and scalable systems.

This document provides a comprehensive overview of these emerging programming languages, helping developers understand their benefits, applications, and future potential.


Update to 7

Summary
Provide a detailed description of all the changes present in this pull request.
Additional Context
Add any additional context about the problem here.

[ ] Root cause and the steps to reproduce. (If applicable)
[ ] Thought process behind the implementation.

Related Issues (if any)
Mention any related issues or pull requests.
Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified. (For example puppet apply)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: vladHulea:x: radudramba:x: tradeshift-renovate[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Improve reconcile in case of a landscaper upgrade

How to categorize this issue?

/area deployers
/kind bug
/priority 3
What happened:
Several installations failed due to an error of the landscaper. After the upgrade of the landscaper the failed installations were not reconciled and therefore remain in that state. This affected also the installations for the configured deployer.
The situation became even more complicated in case an agent is running. The installations for the deployer under the control of the agent were updated first but failed du to the bug in the agent. Then the agent was updated but its deployer were not reconciled again.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Landscaper version:
Kubernetes version (use kubectl version):
Others:

Closing this, as it has been moved to the new backlog project.


Player Kicked for "Invalid Player Move Packet Recieved"

Java version 8
Minecraft 1.16.2
Immersive Cursedness 1.1.1
Fabric API 0.20.1+build.401-1.16
Walking into an area with fake blocks causes the player to be kicked for invalid player packet received.

I'd say this is caused by #7, as you shouldn't be able to walk in an area with fake blocks


Compile error when using app:binding

I am getting the following error, when I try to use the app:binding tag
I have imported compile 'com.github.fabioCollini:twowaydatabinding:0.1' like in the demo, any idea whats wrong
Error:Execution failed for task ':app:compileDebugJavaWithJavac'.
> java.lang.RuntimeException: Found data binding errors.
****/ data binding error ****msg:Cannot find the setter for attribute 'app:binding' with parameter type ``java.lang.String.
file:\res\layout\activity_login.xml
loc:29:31 - 29:51
****\ data binding error ****

Hi, which object are using in the app:binding attribute? You should use an ObservableString, it seems that you are using a normal String.

That was the problem thanks :+1:


-v does not work when SQL file has comments

I have a SQL file like this
-- Comment
DROP DATABASE [$(DbName)]

When I run sqlcmd -i input.sql -v DbName=MyDatabase, I get error

Database '$(DbName)' does not exist. Make sure that the name is entered correctly.

If I remove the comment on line 1, the script executes successfully.
This used to work with the old sqlcmd.

It also works if the comment is after the line where the SQLCMD variable is, so the issue is only if the comment precedes the SQLCMD variable.

I could not reproduce this when the mentioned sql statements were the only SQL statements in the file.
I however could reproduce it with sql file similar to
-- Comment
CREATE DATABASE [$(Dbname)]
USE [$(Dbname)]
USE master
DROP DATABASE [$(DbName)]

On console
>.\sqlcmd.exe -i input.sql -v Dbname=T213
Msg 911, Level 16, State 1, Server LAPTOP-VPBI3ASB, Line 3
Database '$(Dbname)' does not exist. Make sure that the name is entered correctly.


Coverage test missing

Currently the coverage is commented out and has to be added.

The new code has a coverage test in place with coverage of 40%. This should be increased to a higher value, but I will open up separate issues for separate parts of the code.
addopts = "--cov --cov-report html --cov-report term-missing --cov-fail-under 40"


adding minimum and maximum limits for parameters.

eventhough the upper and lower limits for the following parameters were added to the system, it is not working.
Height,
Weight
Waist circumference
BMI
SBP
DBP
RBS
FBS
S.Creatinine
Peak Flow Meter
TC
HDL
LDL
TG

Will update asap
On Wed, 8 Feb 2023 at 15:37, kanchanabl @.***> wrote:

eventhough the upper and lower limits for the following parameters were
added to the system, it is not working.
Height,
Weight
Waist circumference
BMI
SBP
DBP
RBS
FBS
S.Creatinine
Peak Flow Meter
TC
HDL
LDL
TG
—
Reply to this email directly, view it on GitHub
https://github.com/lk-gov-health-hiu/chims/issues/301, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJLRQUBKNXYBHKPLMAKYFTWWNV5RANCNFSM6AAAAAAUVBQSAU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Optional gzip compression of responses

Issue #, if available: None
Description of changes:

Compress responses when the COMPRESSION env var is true, when the response is not already compressed, and when the content-type starts with text/ or is one of a few known types such as application/javascript
The list of mime types to compress is not configurable to align with the way CloudFront handles automatic compression if enabled - the list of mime types that should / should not be compressed is something that is largely configurable one time and does not need to be adjusted or excruciatingly specified by each consumer (like nginx requires)
Set the Content-Length response header to have the correct compressed length
Set the Content-Encoding response header to gzip if compression is performed

Motivations

This project is awesome and I'm going to incorporate it into the documentation and examples for my MicroApps framework as the suggested method for quickly converting web apps to run as Lambda functions
Node.js is exceedingly slow at performing gzip compression, particularly of large response bodies
Lambda functions exposed via CloudFront connected to Function URLs with SigV4 signing do not have another place where compression of the response can be guaranteed, particularly on non-cacheable responses (which CloudFront either never or sometimes does not compress)
Rust compressing the response should be substantially faster (I didn't even measure it as I've done extensive measurements of Node.js gzip performance for other projects... it's bad)
This is a feature I need to allow Lambda to tie or beat the current deployment pattern for an existing site

To-Do / Seeking Input

[x] Should the COMPRESSION env var name be the first name-spaced env var such as LAMBDA_ADAPTER_COMPRESSION?
[x] Do we have deployed integration tests that will confirm this works in the wild?  I wasn't sure where to put those
[x] I assume the lambda http crate is handling marking the response as base64 encoded and handling the base64 encoding when it sees the gzip content-encoding - could someone confirm that?  I see that nginx is sending gzip responses through the adapter so I assume this is all working correctly already
[x] The use of unsafe in one of the tests (not deployed code) was gross, but there does not appear to be a body function that takes a byte array so I had to fake it as a non-validated string - Any suggestions for that test would be welcome

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

Thanks for the contribution!
Here are my suggestions.

Let's use AWS_LWA_ATUO_COMPRESSION as the environment variable name.
We deploy a go-httpbin app in the pipeline, run e2e tests againt it in beta account, and tear it down when finished. You could add tests to the initial set of e2e tests here.
Yes, lambda http crate will base64 encode any response with non-empty content-encoding.
No need to convert the encoded bytes to a String. Hyper accepts Vec[u8] as body.

There are a few formatting issues. Please run cargo fmt. It will fix those issues.

There are a few formatting issues. Could you run cargo fmt? It will fix them.

Yup will do shortly!  Thanks for the quick feedback, that's very refreshing!

No need to convert the encoded bytes to a String. Hyper accepts Vec[u8] as body.

I should clarify: It's httpmock that only accepts a string for the mocked body response.  This is only in a unit test but there did not appear to be any other way to get httpmock to return unmodified bytes.

@bnusunny any chance this can be released quickly?  I'm really wanting to use it in a project.

Just need one minor update in the README. See my last comment above.

Just need one minor update in the README. See my last comment above.

I only see two comments?  One about cargo fmt and the one answering my 4 questions?

Just need one minor update in the README. See my last comment above.

I only see two comments? One about cargo fmt and the one answering my 4 questions?

Sorry, I forget to summit the review comments.

Just need one minor update in the README. See my last comment above.

I only see two comments? One about cargo fmt and the one answering my 4 questions?

Sorry, I forgot to summit the review comments.

No problem!  All issues addressed, thanks for the feedback!

Release v0.6.2 is out. The layers and docker images are published.

I can confirm this is working.  I'm using it in a few projects already!


84 double dash

Fixes #84.

This is branched off #86 so it includes commits from there.

How OK are we with linting pragmas?


Create one large final

Create a single file

File must be able to map sentence back to source file

This issue was fixed somewhere in this huge commit that should have really been several commits. Wops
https://github.com/jaysonmc/contradiction-crawler/commit/76710962dff4cd26abc1d04900c35bcf6b24a463

For the sake of record keeping, this approach was not taken. This was done instead: https://github.com/jaysonmc/contradiction-crawler/issues/7


Use temurin instead of adoptopenjdk

Motivation:
Update the URLs of OpenJDK in the documents because the new OpenJDK binaries from AdoptOpenJDK are no longer distributed and have been migrated to Eclipse Temurin.
It seems that this project have already used Eclipse Temurin on CI.
Modifications:

Update the documents and URLs

Result:

The URLs refer to Eclipse Temurin

Not completely related to this PR, but I think we can update this Java version in the documentation as well: https://github.com/line/armeria/blob/master/site/src/pages/docs/setup.mdx?plain=1#L11


chore(fleet): Fix incident-30484

What does this PR do?

Splits Windows tests from Linux installer tests
Adds more info in the command execution logs (date + test name)
Timeouts a loop that could run forever
Reverts https://github.com/DataDog/datadog-agent/pull/29232

Motivation
Fixing #incident-30484's remediation items
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes

/merge


Have a reverse proxy or a display page when the server is offline (?)

w/ the server automatically turning off if nobody is online, a reverse proxy may be needed so users aren't greeted with an HTTP 500 error, and have to refresh to see the website again. This is a flawed UI.

Honestly not a bad idea, it's a little goofy the way it's being handled right now. Could we use Nginx to serve an offline page?

Currently we are using shuttle and don't have a dedicated server, so some changes will be necessary for this.

See #44


Angular Animations not working in Ionic View

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[T] 3.x
[] 4.x
I'm submitting a ...  (check one with "x")
[T] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I've successfully implemented Angular animations as components in my Ionic 3 and they work perfectly in when running locally with Ionic Serve but when I try to emulate the app with Ionic View they don't work. Just wondering if anyone has had this issue or if it that is supposed to happen.
Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. This appears to be a question about Ionic View, which is part of the Ionic Pro set of offerings.
For Ionic Pro issues, please use the Ionic Pro Support Portal

Just wondering if anyone has had this issue or if it that is supposed to happen.

FWIW, I just tried one of my apps on Ionic View, paying attention to animations. All of the standard page transition animations work just fine. But... that is probably not really the same thing as your animations. You may need to supply a sample application when logging your issue.
Question: What happens if you load it on a device w/o using Ionic View? (IOW, use Xcode or ionic cordova android run depending on platform to load to your development device)?
Thanks for using Ionic!


list out of range

Traceback (most recent call last):
File "layer_channel_prune.py", line 273, in
compact_model2 = Darknet([compact_model1.hyperparams.copy()] + compact_module_defs, (img_size, img_size)).to(device)
File "/data1/visiondection/detection/yolov3-channel-and-layer-pruning-master/yolov3-channel-and-layer-pruning-master/models.py", line 221, in init
self.module_list, self.routs, self.hyperparams = create_modules(self.module_defs, img_size, arc)
File "/data1/visiondection/detection/yolov3-channel-and-layer-pruning-master/yolov3-channel-and-layer-pruning-master/models.py", line 57, in create_modules
filters = sum([output_filters[i + 1 if i > 0 else i] for i in layers])
File "/data1/visiondection/detection/yolov3-channel-and-layer-pruning-master/yolov3-channel-and-layer-pruning-master/models.py", line 57, in
filters = sum([output_filters[i + 1 if i > 0 else i] for i in layers])
IndexError: list index out of range


Error on Entity Death

Running:
forge-1.11.2-<IP_ADDRESS>2-universal
spongeforge-1.11.2-2282-7.0.0-BETA-2327
Prism-3.0.0-beta2-45-gfbbeb2b-all
LuckPerms-Sponge-3.0.68
griefprevention-1.11.2-<IP_ADDRESS>9
Nucleus-0.27.1-1.11.2-S6.0-plugin
https://pastebin.com/kt5FhVhm

This is a Sponge issue which gabizou has committed to fix, duplicate of #92


[3] Map out airflow tasks

What
We need to plan out the airflow tasks that will run the ETL. These tasks will handle the preparation and execution of the CLI tool. See the bitcoin ETL DAGs as an example.
Why
Airflow allows us to manage and monitor the ETL's runs cleanly. Mapping out the tasks makes the implementation easier.

We also will need to determine if we can create an airflow tasks that will export all ledger entries to BigQuery even if they are all in the same file, or if we will need to separate out changes by type/batch.

DAG organization

History archive DAG

exports ledgers, txs, ops
Load exported files into Google Cloud Storage
Load GCS files into BigQuery (append new rows)

Long Running Core DAG

Exports accounts, trustlines, offers
All in single command; long running stellar-core instance exports incrementally

Changes Export DAG

File sensors look at the location where ledger entry changes are exported in batches
As new files are created, they are uploaded to BigQuery and deleted from local storage
Files are used up update BigQuery tables (can create new rows, update existing rows, or delete rows)

Catchup DAG

Exports accounts, Trustlines, offers
Everything in separate commands; uses bucket list to catchup


Integration and unit tests

This desperately needs some automated testing

Wrote parsing tests in #22 , I'm calling this good for now


StreamHPC 2024-01-16

Note: this PR depends on https://github.com/ROCm/rocPRIM/pull/509
This PR does not include the updates from thrust 2.2. Those changes are intended to be provided in a follow-up PR.
Fixed a number of conversion warnings
In the partition algorithm, using transform iterator instead of transform
Using public API invoke_result from rocPRIM instead of the symbol from the detail namespace.

All tests pass, but one of the CI jobs for performance needs to be rerun


TeamViewer

Microsoft Reviewers: Open in CodeFlow

@samiranshah the hash validation failed. Is there a URL to a specific version of the installer?


[WIP] Adopt Zabbix protocol

Adopt Zabbix protocol

[ ] Update the documentation to include this new feature.
[ ] Tests(including UT, IT, E2E) are added to verify the new feature.
[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #6118
[x] Update the CHANGES log.

I removed the 8.4.0 for now, as we don't have the VM concept for now. FYI @wankai123 , this depends on your side process.

I removed the 8.4.0 for now, as we don't have the VM concept for now. FYI @wankai123 , this depends on your side process.

https://twitter.com/ASFSkyWalking/status/1355129922356830210
This should be a good example for you to match Zabbix meitrcs with @wankai123 's OTEL metrics for the VM. I hope those could be unified.

https://twitter.com/ASFSkyWalking/status/1355129922356830210
This should be a good example for you to match Zabbix meitrcs with @wankai123 's OTEL metrics for the VM. I hope those could be unified.

https://twitter.com/ASFSkyWalking/status/1355129922356830210
This should be a good example for you to match Zabbix meitrcs with @wankai123 's OTEL metrics for the VM. I hope those could be unified.

Sure. I will follow the metrics, and try to keep Zabbix metrics consistent.

https://twitter.com/ASFSkyWalking/status/1355129922356830210
This should be a good example for you to match Zabbix meitrcs with @wankai123 's OTEL metrics for the VM. I hope those could be unified.

Sure. I will follow the metrics, and try to keep Zabbix metrics consistent.

I have tested, and take a screenshot in the VM view.

It looks nice. Have you tested hybrid using Prometheus Node exporter and Zabbix? Does MAL support this(Same metrics in different MAL files)? @hanahmily

It looks nice. Have you tested hybrid using Prometheus Node exporter and Zabbix? Does MAL support this(Same metrics in different MAL files)? @hanahmily

Checked, no supported yet, add https://github.com/apache/skywalking/issues/6356 to enhance this.

Besides https://github.com/apache/skywalking/pull/6176#discussion_r573495258, the PR looks good to me.

UT fails, could you recheck?


CSS inconsistencies between docs/tutorials/kubernetes-basics and the rest of the pages

This is a Bug Report

Problem: The CSS looks different between docs/tutorials/kubernetes-basics and the rest of the website. It has its local CSS files here https://github.com/kubernetes/website/tree/main/content/en/docs/tutorials/kubernetes-basics/public/css. Maybe the main CSS diverged from the local for these pages that were integrated as a whole, see Integrate bootcamp into Kubernetes docs. #1203, or was just different from the start. Maybe that's not really an issue, I just noticed it because of the new Katacoda warning boxes that looked really different on these page and on the rest of the website.
For example, check the CSS differences between these pages:

https://kubernetes.io/docs/tutorials/hello-minikube/
https://kubernetes.io/docs/tutorials/kubernetes-basics/create-cluster/cluster-interactive/

Proposed Solution: Try to simplify the local CSS to avoid classes overrides that break the website look.
Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/public/css/styles.css
https://kubernetes.io/docs/tutorials/kubernetes-basics/public/css/overrides.css

/triage accepted
PRs are welcomed.

/area web-development

The ideal fix is that we can remove https://kubernetes.io/docs/tutorials/kubernetes-basics/public/css/*.css and instead have the appropriate style declarations be a part of the main website Sass (see https://github.com/kubernetes/website/tree/main/assets/scss)

/assign

Fixed in #34188
/close


Support an async pattern for the zip upload API

Some background
This issue is related to #1741
We also run into the timeout problem. As already noted in that bug thread, in our experience the problem isn't really the upload part (which is usually quick), but when the zip archive contains a huge number of files. After trying different clients it seems that this issue isn't really with Kudu nor the client, but with the load balancer in Azure which kills off idle connections after 230 seconds. If it takes the Kudu instance more than 230 seconds to expand the archive, the connection gets killed and we get the timeout error.
The exact same thing happens when uploading the zip file through a browser, the connection drops and you can see that the "paKman.gif" disappears.  If you use dev tools to monitor the request you'll see that it times out as well:

However, in all these cases it seems the unpack process keeps running in the background and eventually finishes.
If you want to test this for yourself, here's a zipfile (~18 mb) with 100k txt-files divided into 100 folders:
https://kudurocks.blob.core.windows.net/repro/timeout.zip
Or create one yourself by running (PowerShell):
1..100 | %{ "$_/100" ; $FolderName = $([Guid]::NewGuid().Guid) ; $null = New-Item -Path ".\$FolderName" -ItemType Directory ; 1..1000 | % { "$_" | Out-File .\$FolderName\$_.txt } }
It will create the folder structure in the "current folder" which you would then need to zip and then upload.
Feature request/workaround
A possible workaround for this problem would be to enhance the logging for the unpack process. Like some other Kudu processes would it be possible to create a log file when the process starts which is renamed when the process has finished/a new one is created when it's done? I think this is basically how the zip-api call works when packaging a folder, and the file also includes any problems that occurred during the packaging process.
This would allow us to basically do this in a more "async" way by polling that logfile for the current status if the original API call times out, and verify that it doesn't contain any errors when done.

@AWahlqvist the 230s timeout is an App Service limit which unfortunately we have no control over. The one way to avoid it is to keep some data flowing. i.e. write to the response to keep it live. That's what we do in the 'git push' scenario. But it's harder to do in an API that needs to return a proper http code, as the minute you write the headers, you lose the ability to change the code.
The proper solution would be to provide an async pattern for this API, which is what you are suggesting. But that's a fair bit of work.

I renamed the issue to better capture what it's about (the logging part is more of an implementation detail).

I agree with you, thanks for your quick response!
I was mostly looking at this from my perspective where a logfile would make it possible for me to build a workaround for this issue, and I'm guessing it's quite a lot simpler to implement. But I totally agree implementing a true async pattern for this API would be the best solution long term (the same thing applies for the "create zip API" I guess, we solved that by creating our own zipping function that support running async in the the Kudu session since we needed to apply some rules regarding what to zip up anyway).
Again, thanks for your response!

Yes, I think you can build your own workaround:

Instead of using zip API, upload a .cmd script (using vfs API) that does the unzipping, and write a marker file when it's done
Start your script using the Command API
Use the vfs API to poll for the marker file, so you know when it's done

A bit messy, but can be done :)

Yes, that's basically how the creation of the zip archive is done now :-)

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Requesting config file or pretrain weights.

Hello @yunshin
Thank you for your research and allowing your code to be open sourced.
Currently, I tried to reproduce SphericalMask with the S3DIS dataset.
Since SphericalMask is based on ISBNet, I used the config file used in ISBNet to run the S3DIS dataset.
However, the SpheircalMask result was much worse than the result shown in the paper.
Could you share the config file or pretrained weights used in this experiment?
Thank you for your time.

Hello,
Thank you for your interest.
We are going to upload the details of S3DIS accordingly with the conference at the latest.

Thank you for your response @yunshin !
I have some additional questions about your model.

When you train for Scannet benchmark, do you initialize with ISBNet benchmark pre-train checkpoints?

For training SPHMask benchmark can you kindly share the parameter values of epoch, repeat, learning-rate?

Thank you for your time.


Add chainer v3 test to travis

Since Chainer v4 has been released, Chainer installed by CHAINER_VERSION="chainer in travis has been changed to from v4 to v3. That means Chainer v3 is not tested in the current configuration. This PR adds a test with Chainer v4 to .travis.yml.
I did not remove configuration about Chainer v2 (CHAINER_VERSION="chainer<3)  because testing time in Travis is not a bottleneck in the current workflow. But I would be like opinions.

Codecov Report

Merging #154 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #154   +/-   ##
=======================================
  Coverage   77.54%   77.54%
=======================================
  Files          79       79
  Lines        3385     3385
=======================================
  Hits         2625     2625
  Misses        760      760

I think building miniconda takes most of the time. Does it build miniconda 12 times?

Does it build miniconda 12 times?

Yes, in total. The reason for using miniconda is that it is recommended way to install RDKit via it (if I remember correctly) and we expect most users install in this way.

I feel sometimes many jobs are queued (e.g. before releasing a new version), so it is better to reduce testing time.
Currently we test with v2, 4, and 5, I think at least we should replace v2 with v3, if we do not want to increase the number of configurations in the CI.

An alternative way is to remove testing with Python 3.5.

I remove tests with Python 3.5 as to keep testing time. It is just a work around and we need to consider a better way.

After offline discussion, we support the stable version of Chainer (v4 now) and test the stable and develop version of Chainer (v4 and v5) in Travis. So, the resulting change is to disable Chainer v2. I changed the PR title accordingly. Rel. #190.

Let me close this PR and resend it.