Problem with grunt-sass?

Hi,
I'm trying to get this work with grunt-sass but everytime I'm running the grunt-sass task I'm getting this error:

One or more elements from green-element have already been created

My test file:
@import "bem-constructor";

@include object('test') {
	background: red;

	@include element('green-element') {
		background: green;
	}
}

Did you ever run into that issue?
EDIT:

grunt-sass version: 1.1.0
bem-constructor version: 1.0.0

Interesting. Thanks for reporting the issue. I'll look into it and get back to you.

Thx :+1: :smile:
I'm testing around, too ...

Looks like (yet another) Libsass bug. Ruby Sass 3.4.21 works as expected. Will report the issue in to the Libsass team.

The issue has been reported in the Libsass repo sass/libsass/issues/1944

Thank you! Great to hear that :plus1:

The Libsass issue has been fixed in the new release. Closing this issue.


Update sign in/sign up and sign out UI

The UI doesn't match the rest of the UI. We need to update it to look similar to the landing page.

I created a css sheet for only the account section of the template and connected to the base.html of the accounts directory


Seems missmatch codestyle waypoint.yaml in manifests

 volumeMounts:
        - name: workload-socket
          mountPath: /var/run/secrets/workload-spiffe-uds
        - mountPath: /var/run/secrets/istio
          name: istiod-ca-cert
        - mountPath: /var/lib/istio/data
          name: istio-data
        - mountPath: /etc/istio/proxy
          name: istio-envoy
        - mountPath: /var/run/secrets/tokens
          name: istio-token
        - mountPath: /etc/istio/pod
          name: istio-podinfo

Looks incorrect style subsequence of paris name, mouthPath in first element of array
https://github.com/istio/istio/blob/89d73dc1f639d1625135923c381e4ec222247059/manifests/charts/istio-control/istio-discovery/files/waypoint.yaml#L221

I don't think it particularly matters what order its in...?

I as inexperienced was confused . Yes, there is no difference functionally, but avoid unclear moment


remove deprecation warning fixed_lim

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b0e8364) 83.30% compared to head (ab93fde) 83.31%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #321   +/-   ##
=======================================
  Coverage   83.30%   83.31%
=======================================
  Files          36       36
  Lines        4248     4244    -4
=======================================
- Hits         3539     3536    -3
+ Misses        709      708    -1

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


Planes should only expose "interesting" data

Overview
The audio resampler output is incorrect.
Expected behavior
The audio resampler output should be consistent with ffmpeg's command line tool.
Actual behavior
The audio resampler output is incorrect/corrupted.
Investigation
If I only do decoding without resampling, the output is correct.
Reproduction
The corrupted output can be generated with the following code.
import av
resampler = av.AudioResampler(format='fltp', layout='mono', rate=16000)
container = av.open('Silent_Light.m4a')
stream = next(s for s in container.streams if s.type == 'audio')
with open('out_fltp_16k.pcm', 'wb') as f:
    for packet in container.demux(stream):
        for frame in packet.decode():
            frame.pts = None
            out_frame = resampler.resample(frame)
            if out_frame is not None:
                f.write(out_frame.planes[0])

Versions

OS: Ubuntu 18.04
PyAV runtime:

PyAV v6.2.0
git origin: git@github.com:mikeboers/PyAV
git commit: unknown-commit
library configuration: --prefix=/home/zhaosheng/anaconda3 --disable-doc --disable-openssl --enable-shared --enable-static --extra-cflags='-Wall -g -m64 -pipe -O3 -march=x86-64 -fPIC' --extra-cxxflags='-Wall -g -m64 -pipe -O3 -march=x86-64 -fPIC' --extra-libs='-lpthread -lm -lz' --enable-zlib --enable-pic --enable-pthreads --enable-gpl --enable-version3 --enable-hardcoded-tables --enable-avresample --enable-libfreetype --enable-gnutls --enable-libx264 --enable-libopenh264
library license: GPL version 3 or later
libavcodec     58. 18.100
libavdevice    58.  3.100
libavfilter     7. 16.100
libavformat    58. 12.100
libavutil      56. 14.100
libswresample   3.  1.100
libswscale      5.  1.100

PyAV build: (installed with Anaconda)
FFmpeg:

ffmpeg version 4.0.2 Copyright (c) 2000-2018 the FFmpeg developers
built with gcc 4.8.2 (GCC) 20140120 (Red Hat 4.8.2-15)
configuration: --prefix=/home/zhaosheng/anaconda3 --disable-doc --disable-openssl --enable-shared --enable-static --extra-cflags='-Wall -g -m64 -pipe -O3 -march=x86-64 -fPIC' --extra-cxxflags='-Wall -g -m64 -pipe -O3 -march=x86-64 -fPIC' --extra-libs='-lpthread -lm -lz' --enable-zlib --enable-pic --enable-pthreads --enable-gpl --enable-version3 --enable-hardcoded-tables --enable-avresample --enable-libfreetype --enable-gnutls --enable-libx264 --enable-libopenh264
libavutil      56. 14.100 / 56. 14.100
libavcodec     58. 18.100 / 58. 18.100
libavformat    58. 12.100 / 58. 12.100
libavdevice    58.  3.100 / 58.  3.100
libavfilter     7. 16.100 /  7. 16.100
libavresample   4.  0.  0 /  4.  0.  0
libswscale      5.  1.100 /  5.  1.100
libswresample   3.  1.100 /  3.  1.100
libpostproc    55.  1.100 / 55.  1.100

Research
I have done the following:

[x] Checked the PyAV documentation
[x] Searched on Google
[x] Searched on Stack Overflow
[x] Looked through old GitHub issues
[ ] Asked on PyAV Gitter
[ ] ... and waited 72 hours for a response.

Additional context
The attached file Silent_Light.zip contains the input audio file I used. I used the following command to generate the reference output with ffmpeg:
ffmpeg -i Silent_Light.m4a -ar 16000 -ac 1 -acodec pcm_f32le Silent_Light.wav

Silent_Light.zip

I haven't run anything yet, but I vaguely remember someone saying something was up with audio planes in certain circumstances. There was a lot of work at some point to assert that video planes (or at least the numpy arrays) were reasonable.

It is as I suspected.
# first frame
<av.AudioFrame 0, pts=None, 1024 samples at 44100Hz, stereo, fltp at 0x1086b42f0>

# resampled
<av.AudioFrame 0, pts=0, 355 samples at 16000Hz, mono, s16 at 0x1086b4440>

# the plane
<av.AudioPlane 744 bytes at 0x7f840e71c850>

The resampled plane has 355 samples, which should be 355x2 = 710 bytes, but the plane is 744.
So... I guess we should be trimming the planes to be reasonable lengths or something.

@mikeboers Thank you for your reply. I am now able to use this workaround:
import av
resampler = av.AudioResampler(format='fltp', layout='mono', rate=16000)
container = av.open('Silent_Light.m4a')
stream = next(s for s in container.streams if s.type == 'audio')
with open('out_fltp_16k.pcm', 'wb') as f:
    for packet in container.demux(stream):
        for frame in packet.decode():
            frame.pts = None
            out_frame = resampler.resample(frame)
            if out_frame is not None:
                f.write(memoryview(out_frame.planes[0])[:4 * out_frame.samples])

This is definitely the same as #539. I'm gonna edit this a bit to represent the problem and close the other.

I'm not sure we should get too involved with how FFmpeg lays out the memory, there are probably always going to be cases we cannot catch, especially with non planer formats.


nginx.conf points to mashape syslog server

Summary
I think this shouldn't be in there?
cat /usr/local/kong/nginx.conf
...
error_log syslog:server=kong-hf.mashape.com:61828 error;
...

Additional Details & Logs

Kong version: 0.8.3 in the official kong:0.8.3 docker image

That is the error reporting and anonymous usage data for Mashape's use, nothing wrong here. You can disable this feature by turning off anonymous_reports.

Sorry, glanced over the docs too fast. Thanks for the clarification.


Image is not saved.

I have used the code as you say but it is not saving cropped images in any directory either in my custom folder or DCIM (inbuilt). help! @ekimual

@sarang1996 have you updated to v1.1.2? Can I see some source code / logs and device information..

I have solved it just I am using as your documentation but when we are initializing a folder there is one '/' needed in last. please updated. @ekimual


Property [from] does not exist on the Eloquent builder instance.

Laravel Version: 8
PHP Version: 7,3
Database Driver & Version:

Description:
On joinSub method get error  Property [from] does not exist on the Eloquent builder instance.
{
error: true,
message: "Property [from] does not exist on the Eloquent builder instance.",
file: "/var/www/lumen/vendor/illuminate/database/Eloquent/Builder.php:1379",
trace: [
"#0 /var/www/lumen/vendor/illuminate/database/Query/Builder.php(366): Illuminate\Database\Eloquent\Builder->__get('from')",
"#1 /var/www/lumen/vendor/illuminate/database/Query/Builder.php(342): Illuminate\Database\Query\Builder->prependDatabaseNameIfCrossDatabaseQuery(Object(Illuminate\Database\Eloquent\Builder))",
"#2 /var/www/lumen/vendor/illuminate/database/Query/Builder.php(328): Illuminate\Database\Query\Builder->parseSub(Object(Illuminate\Database\Eloquent\Builder))",
"#3 /var/www/lumen/vendor/illuminate/database/Query/Builder.php(506): Illuminate\Database\Query\Builder->createSub(Object(Illuminate\Database\Eloquent\Builder))",
"#4 /var/www/lumen/vendor/illuminate/support/Traits/ForwardsCalls.php(23): Illuminate\Database\Query\Builder->joinSub(Object(Illuminate\Database\Eloquent\Builder), 'z', 'z.id', '=', 's.zone_id')",
"#5 /var/www/lumen/vendor/illuminate/database/Eloquent/Builder.php(1419): Illuminate\Database\Eloquent\Builder->forwardCallTo(Object(Illuminate\Database\Query\Builder), 'joinSub', Array)",
"#6 /var/www/lumen/app/Services/NetworkStatisticsService.php(185): Illuminate\Database\Eloquent\Builder->__call('joinSub', Array)",
"#7 /var/www/lumen/app/Http/Controllers/NetworkStatisticsController.php(28): App\Services\NetworkStatisticsService->networkByDate(Array)",
"#8 [internal function]: App\Http\Controllers\NetworkStatisticsController->networkByDate(Object(Laravel\Lumen\Http\Request))",
"#9 /var/www/lumen/vendor/illuminate/container/BoundMethod.php(37): call_user_func_array(Array, Array)",
"#10 /var/www/lumen/vendor/illuminate/container/Util.php(40): Illuminate\Container\BoundMethod::Illuminate\Container\{closure}()",
"#11 /var/www/lumen/vendor/illuminate/container/BoundMethod.php(95): Illuminate\Container\Util::unwrapIfClosure(Object(Closure))",
"#12 /var/www/lumen/vendor/illuminate/container/BoundMethod.php(39): Illuminate\Container\BoundMethod::callBoundMethod(Object(Laravel\Lumen\Application), Array, Object(Closure))",
"#13 /var/www/lumen/vendor/illuminate/container/Container.php(596): Illuminate\Container\BoundMethod::call(Object(Laravel\Lumen\Application), Array, Array, NULL)",
"#14 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(386): Illuminate\Container\Container->call(Array, Array)",
"#15 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(352): Laravel\Lumen\Application->callControllerCallable(Array, Array)",
"#16 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(326): Laravel\Lumen\Application->callLumenController(Object(App\Http\Controllers\NetworkStatisticsController), 'networkByDate', Array)",
"#17 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(279): Laravel\Lumen\Application->callControllerAction(Array)",
"#18 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(259): Laravel\Lumen\Application->callActionOnArrayBasedRoute(Array)",
"#19 [internal function]: Laravel\Lumen\Application->Laravel\Lumen\Concerns\{closure}(Object(Laravel\Lumen\Http\Request))",
"#20 /var/www/lumen/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(48): call_user_func(Object(Closure), Object(Laravel\Lumen\Http\Request))",
"#21 /var/www/lumen/app/Http/Middleware/GuardMiddleware.php(40): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Laravel\Lumen\Http\Request))",
"#22 /var/www/lumen/vendor/illuminate/pipeline/Pipeline.php(167): App\Http\Middleware\GuardMiddleware->handle(Object(Laravel\Lumen\Http\Request), Object(Closure))",
"#23 [internal function]: Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}(Object(Laravel\Lumen\Http\Request))",
"#24 /var/www/lumen/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(30): call_user_func(Object(Closure), Object(Laravel\Lumen\Http\Request))",
"#25 /var/www/lumen/vendor/illuminate/pipeline/Pipeline.php(103): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Laravel\Lumen\Http\Request))",
"#26 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(423): Illuminate\Pipeline\Pipeline->then(Object(Closure))",
"#27 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(260): Laravel\Lumen\Application->sendThroughPipeline(Array, Object(Closure))",
"#28 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(166): Laravel\Lumen\Application->handleFoundRoute(Array)",
"#29 [internal function]: Laravel\Lumen\Application->Laravel\Lumen\Concerns\{closure}(Object(Laravel\Lumen\Http\Request))",
"#30 /var/www/lumen/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(48): call_user_func(Object(Closure), Object(Laravel\Lumen\Http\Request))",
"#31 /var/www/lumen/app/Http/Middleware/CorsMiddleware.php(74): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Laravel\Lumen\Http\Request))",
"#32 /var/www/lumen/vendor/illuminate/pipeline/Pipeline.php(167): App\Http\Middleware\CorsMiddleware->handle(Object(Laravel\Lumen\Http\Request), Object(Closure))",
"#33 [internal function]: Illuminate\Pipeline\Pipeline->Illuminate\Pipeline\{closure}(Object(Laravel\Lumen\Http\Request))",
"#34 /var/www/lumen/vendor/laravel/lumen-framework/src/Routing/Pipeline.php(30): call_user_func(Object(Closure), Object(Laravel\Lumen\Http\Request))",
"#35 /var/www/lumen/vendor/illuminate/pipeline/Pipeline.php(103): Laravel\Lumen\Routing\Pipeline->Laravel\Lumen\Routing\{closure}(Object(Laravel\Lumen\Http\Request))",
"#36 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(423): Illuminate\Pipeline\Pipeline->then(Object(Closure))",
"#37 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(172): Laravel\Lumen\Application->sendThroughPipeline(Array, Object(Closure))",
"#38 /var/www/lumen/vendor/laravel/lumen-framework/src/Concerns/RoutesRequests.php(109): Laravel\Lumen\Application->dispatch(NULL)",
"#39 /var/www/lumen/public/index.php(28): Laravel\Lumen\Application->run()",
"#40 {main}"
]
}

Steps To Reproduce:
run code
     $queryStats = MyModel::query()
            ->select([some columns])
            ->from('some table as s')
            ->joinSub($zonesQuery,'z', 'z.id', '=', 's.zone_id')
            ->whereBetween('s.date', [$dateFrom, $dateTo])
            ->groupBy(['s.date']);

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


compatibility hack for jobs created pre rails 4.2

Upgrading rails to 4.2 breaks delayed_jobs when its worker is trying to run jobs created in pre 4.2 with the following error:
NoMethodError: undefined method `fetch_value' for #<Hash:>
from /path_to_gems/activerecord-4.2.3/lib/active_record/attribute_methods/read.rb:93:in `_read_attribute'

This PR tries to make the upgrade process works more smoothly.

looks like the fault falls in rails side and is fixed by https://github.com/rails/rails/commit/438662ab32715d3b780fb54b0743cc2e19843086
sorry for the noise..


DateRangePicker keyboardType

Adds keyboardType parameter to DateRangePicker.
Fixes https://github.com/flutter/flutter/issues/121912.

@Renzo-Olivares Good point, it would be weird to use the emailAddress keyboard type in a date picker...  The original DatePicker class doesn't limit it though.  I can't say we should necessarily stop someone if they explicitly set it to something weird.  I wouldn't be surprised if there are some edge cases where setting the keyboard to a non-date-related keyboard type makes sense.
If you have a specific idea let me know though.

Google testing appears stuck, but it shouldn't be a problem.  Merging.


Update linker-support-for-delay-loaded-dlls.md

"Statically linking with DLL" is not a correct terminology. "Implicit linking" should be used to describe linking via lib-file, such as in this topic: https://docs.microsoft.com/en-us/cpp/build/linking-an-executable-to-a-dll?view=vs-2017#implicit-linking

@MSDN-WhiteKnight : Thanks for your contribution! The author, @, has been notified to review your proposed change.


add Operation::push_back_region

Hi! Thanks for making pliron! It's awesome!
I'm trying to use pliron in my https://github.com/greenhat/ominzk (Wasm -> various ZK VMs compilation framework). Although Wasm and ZK VMS are stack-based (non-SSA), I believe pliron could be an excellent fit for defining IRs and transformation passes.
I'm writing a func op for the wasm dialect, and I cannot add a region to an operation (wasm.func), so I added Operation::push_back_region.

Hi @greenhat, thank you for the PR!
I had a look at MLIR's API, and w.r.t the regions in an Operation, they don't seem to allow adding or removing regions after an Operation is created. So would you be opposed to, instead of adding the push method you added, changing Operation::new to accept an additional num_regions argument and create that many regions during Operation creation?
Would this alternate work for your use-case?

Sure! Makes total sense. I should have looked into MLIR API first.

@greenhat Shall I do that change or do you want to update the PR?

@greenhat Shall I do that change or do you want to update the PR?

Thanks! I'm on it. I'll update this PR.

Hi @greenhat, thank you for the PR!
I had a look at MLIR's API, and w.r.t the regions in an Operation, they don't seem to allow adding or removing regions after an Operation is created. So would you be opposed to, instead of adding the push method you added, changing Operation::new to accept an additional num_regions argument and create that many regions during Operation creation?
Would this alternate work for your use-case?

Done. I also swap direct Operation::regions access with get_region in builtin dialect. Would you consider moving dialects into separate crates? This way, it'd be easier to ensure they are not getting access to the "inner" mechanics of Operation, etc.

Thank you @greenhat !. I have some minor comments.
Also, a "big picture" thank you, this is the first non-me contribution the project gets :1st_place_medal:

Hi @greenhat, thank you for the PR!
I had a look at MLIR's API, and w.r.t the regions in an Operation, they don't seem to allow adding or removing regions after an Operation is created. So would you be opposed to, instead of adding the push method you added, changing Operation::new to accept an additional num_regions argument and create that many regions during Operation creation?
Would this alternate work for your use-case?

Done. I also swap direct Operation::regions access with get_region in builtin dialect. Would you consider moving dialects into separate crates? This way, it'd be easier to ensure they are not getting access to the "inner" mechanics of Operation, etc.

Would you consider moving dialects into separate crates? This way, it'd be easier to ensure they are not getting access to the "inner" mechanics of Operation, etc.

I've wanted to do that, but have been postponing it. I'll get to it sometime soon.

Thank you @greenhat


fix(webhook): wrong namespace used for serviceaccount

mutated object namespace must be used to inherit  vault policies

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

License
Apache 2.0

What's in this PR?
With the new feature serviceAccount, it is possible to choose which serviceAccount is used by the webhook to connect to Vault. The problem for me is that this serviceAccount is searched/used in the vault-webhook namespace instead of the mutated object namespace.
Why?
In our case, vault policies are namespaced and with this fix, vault-webhook namespace doesn't need to have access on all secrets...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.David Quagebeur seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

One question, does this break the previous functionality regarding the location of the secret it expected?

I'm not sure what Tim has in mind when he implements this feature :
1- the secret has to be in the vault-webhook namespace (it is just an alternative service account): in that case, yes, there is a break
2- the secret has to be in the mutating object namespace, so the webhook works like vault-env with Pod : in that case, it is not a break, but a bug fix.
I have asked my teammates, and for everyone (including me), the logical answer is 2. Do you agree with us ?

Yes TBH after giving some thought to it I agree, that option 2 is the better solution here. And yes that better represents the vault-env experience here.
Since this feature is not yet released we don't break anything just the original PR authors use-case :)
Can we make it configurable so by default it would search the Secret's namespace and if the users would like to have it otherwise it would check that namespace? I would suggest to transform this annotation this way:
Your way:
vault.security.banzaicloud.io/vault-serviceaccount: blabla # The annotated objects namespace will be searched for the blabla SA.

Tim's way:
vault.security.banzaicloud.io/vault-serviceaccount: mynamespace/blabla # The mynamespace will be checked for the blabla SA.

WDYT?

TBH I don't really see a use case on a production cluster. And it would be complicated regarding rights management, as namespace A secret could use a reference to namespace B service account to access secrets of namespace B.
At least, it required a global configuration attribute to enable/disable the support of namespace in service account name.

I also agree, my bad, that I haven't cached this at the initial PR of this feature.

No problem, thanks for the merge.


4.1.0

Clean up library, remove duplication, make response objects behave like Ruby objects rather than JSON hashes.

Could you make the circle.yml file so we can test this on a few rubies?

Also, are the dependencies correct in the gemspec?

Let's talk about the tests in this when you are back.  Grepping out the assertions
$ cat test/**/*.rb | grep assert
    assert_equal Array, response.class
    assert_equal Array, response.class
    assert_equal true, candidate.save
    assert_equal candidate.name_first, 'Chris'
    assert_equal candidate.deleted, true
    assert_equal response.class, BlockScore::QuestionSet
    assert_equal response.class, BlockScore::QuestionSet
    assert_equal response.class, Array
    assert_equal response.class, Array
    assert_equal response.class, Array
    assert_equal response.class, BlockScore::QuestionSet
    assert_equal response.class, resource_to_class(resource)
    assert_equal resource, response.object
    assert_equal Array, response.class
    assert_equal Array, response.class, msg
    assert_equal Array, response.class, msg

the majority of the tests only assert some type checking.  We should be exercising much more of the client's behavior.

Note on method_missing: Whenever you catch a method you should define it. Neglecting that produces a significant performance hit:
Calculating -------------------------------------
       define_method    96.754k i/100ms
      method_missing    21.436k i/100ms
-------------------------------------------------
       define_method      4.324M (±12.9%) i/s -     42.475M
      method_missing    686.767k (±12.4%) i/s -      6.752M

Comparison:
       define_method:  4324036.9 i/s
      method_missing:   686767.5 i/s - 6.30x slower

Bench code:
require 'benchmark/ips'

class MethodDefiner
  def metaclass
    class << self; self; end
  end

  def method_missing(method_name, *args, &blk)
    if method_name =~ /^foo_bar$/
      metaclass.instance_eval do
        define_method(:foo_bar) { true }
      end
      send(method_name)
    else
      super
    end
  end
end

class MethodCatcher
  def method_missing(method_name, *args, &blk)
    if method_name =~ /^foo_bar$/
      true
    else
      super
    end
  end
end

@definer = MethodDefiner.new
@catcher = MethodCatcher.new

Benchmark.ips do |x|
  x.config(:time => 10, :warmup => 2)

  x.report("define_method") do
    @definer.foo_bar
  end

  x.report("method_missing") do
    @catcher.foo_bar
  end

  x.compare!
end

Look into adding some more tests for errors, specifically what you are expecting the formatting to look like.

Running reek turns up some code that could probably be confused.  Try it out and take a look at what it complains about

Read up on some of the smells it turns mentions

10x Duplicate Method Call
10x Irresponsible Module
4x  Too Many Statements
4x  Nil Check
3x  Uncommunicative Variable Name
3x  Class Variable
3x  Nested Iterators
2x  Boolean Parameter
2x  Utility Function
2x  Unused Parameters
1x  Data Clump
1x  Uncommunicative Parameter Name
1x  Feature Envy


TypeError: unlink() got an unexpected keyword argument 'missing_ok'

os.unlink() was given a keyword argument, missing_ok, in Python 3.8 but NiWorkflows 1.5.x still supports Python 3.7.
Commit 518b360679780fc916cc1a3a180a2322430faf0f introduced this keyword, causing an error to be thrown in  nipreps/nirodents#55, whose Docker image is still in 3.7. Exemplary error trace from one of the three errors:
"""
Traceback (most recent call last):
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/plugins/multiproc.py", line 344, in _send_procs_to_workers
self.procs[jobid].run(updatehash=updatehash)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
result = self._run_interface(execute=True)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
return self._run_command(execute)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
result = self._interface.run(cwd=outdir)
File "/usr/local/miniconda/lib/python3.7/site-packages/nipype/interfaces/base/core.py", line 419, in run
runtime = self._run_interface(runtime)
File "/usr/local/miniconda/lib/python3.7/site-packages/niworkflows/interfaces/bids.py", line 672, in _run_interface
out_file.unlink(missing_ok=True)
TypeError: unlink() got an unexpected keyword argument 'missing_ok'
"""
Are there plans to stop supporting 3.7 (meaning I should bump NiRodents to 3.8) or is there a workaround?

Ugh. I didn't realize that I was depending on new behavior. I thought we were testing all supported versions. I'll submit a PR.
That said, it's probably a good idea to bump up to 3.9 or even 3.10 when you get the opportunity. 3.7 will be end-of-life in another year or so.

nope - 3.7 isn't EOL til mid 2023 (https://endoflife.date/python)
we should probably replace those with:
# 3.7 backport
try:
    out_file.unlink()
except FileNotFoundError:
    pass

Sorry to be the bearer of bad news... Thanks for the speedy replies. I'll look to bump NiRodents up anyway, I guess mid-2023 will come round before we know it.

nope - 3.7 isn't EOL til mid 2023 (https://endoflife.date/python)
we should probably replace those with:
# 3.7 backport
try:
    out_file.unlink()
except FileNotFoundError:
    pass

edit - 5 mins late on this...

Sure - Considering the bugfix is so cheap, I'd go with it.
However, I think we should stick with Numpy's deprecation schedule, which dropped Python 3.7 last December. Our Docker images will start breaking soon if some change preempts the use of cached layers (see MRIQC's setup.cfg for an example on the efforts to keep support for 3.7 ongoing)


fix(scanoss): Fix issue on form file field #7320

Form field "file" was missing while sending wfp file

Thanks @scanoss-qg for the update. Please squash all your commits into one, and force-push to the source branch of this PR as we prefer atomic commits. This will also finally make the DCO check pass, as all (i.e. the only) commits in the PR are signed off.

Also, the commit message title and body need to be split across different lines, like
fix(scanoss): Fix issue on form file field

This fixes #7320.

Signed-off-by: scanoss-qg<EMAIL_ADDRESS>

@scanoss-qg something went wrong with squashing commits.

Sorry Sebastian! Was my bad. I Will fix ASAP

Sorry Sebastian! Was my bad. I Will fix ASAP

No worries 😉 Now the signoff is again missing in the commit, and the spaces are back 😁 Please just amend your existing commit and force-push again instead of adding a new commit.

Great News! Thanks for the guiding!


Use LogicalMarker channels in a sequence

The user should be able to generate square pulses from the marker channels at will, not just special cases like digitizerTrig.

This is very likely caused by the fact that we use magic entry keys (namely, PatternUtil.TAZKey and PatternUtil.markerHighKey... allow the latter is only checked implicitly) to encode information on marker channels. We need to make the code more flexible.

I'm interested to see if the recent refactoring has improved this situation here. We have moved heavily away from relying on the magic TAZKey.


printing error instead of crashing in case of syntax error

when your template file contains an error, like a syntax error, broccoli will crash. I am try-catching this and printing an error instead, but possily emitting am error event would be preferable.
with the current test setup, i did not see how i could include a meaningful test for this case. To reproduce this, it's easiest to just put in some garbage html (like an unclosed tag) into an jsx file.

I'm not sure that this is a good idea. Your broccoli build should fail if there is a jsx error.

Yes, it should. but right now, it tears down a whole broccoli watcher with it. I have a setup where i create my own watcher (think "broccoli serve", mostly), and it crashes whenever i have a syntax error.

note: this might be new behaviour: Broccoli 0.16.3 introduces this commit:
https://github.com/broccolijs/broccoli/commit/9923afe8f7c9ad7a05b5ebaf9a3233114214fdcd

I am on broccoli 0.13, but I also create my own watcher, but I don't have the issue of it being torn down. As soon as I fix the jsx error, it continues on to re-build. Here is how my watcher is configured:
watcher.on('error', didError);

function didError(error) {
  console.log(error);
}

This is, why i am specifically mentioning the change in 0.16.3. It would not have happened in 0.13, because this line specifically was not present and your error is just swallowed somewhere in the Promises.
Now, in case of error, there is an error handler that will notify the user that an error has occurred.


How-To Reference a dependent Reference?

Greetings,
I am struggling with creating an simple interface that has 70 Institutions (Account lookups) that references over 250 Programs.  I did not use the Intended Programs listing since it would be too much for users to scroll a list of 250+ programs; therefore, filtering by an Institution would separate the Program listing significantly.

My first attempt had 2 reference questions with the following queries:

Question1. SELECT Name FROM Account WHERE RecordType = 'Institution'
Question2. SELECT  FROM Program WHERE Department_Name__c = (SELECT TextValue__c FROM QuestionResponse where Question = Question1.ID)

Of course that did not work, since I could not generate a query that would get the value of the previous question's response on the same page.

On my second attempt, I tried a brute-force approach by creating 70 Picklist holding the queries of each Institution:

SELECT Name FROM Program WHERE Department_Name__c = 'My College Name"

This approach worked for the first 25 widgets/controls, but since there is a 25 widget/control limitation on Visualforce pages, add more generated a failure.  Also, I realized the increased storage used, since 70 possible questions for each submitted application was saved with the 1 question response/answer.

Lastly, I tried to store the results in the Application object with hopes of referencing it later:

Question1. SELECT Name FROM Account WHERE RecordType = 'Institution' (stored in Intended_Institution__c)
Question2. SELECT  FROM Program WHERE Department_Name__c = :application.Intended_Institution__c (stored in Intended_Program__c)

This approach failed with run-time error stating the variable did not exist.  (I tried other variables that I found in the apex classes, which failed also included app, appControl, applicationID, etc.)
I need help.  I do not know if Holding Place or URL Parameter usage will help, but I cannot find examples in the User Guide.
Thanks.

@softwareangel Why not use one of the filtering picklists (i.e. School/College)? You could add picklist values for each institution and select the relevant institution on each program. You might need to make a small modification to the area around line 250 in the ApplicationCreate class. By default, EASY doesn't restrict the program list when users pick values and instead displays an error message when they try to create the application with an invalid program. The reason for this is a business one: a common complaint by admissions staff is that users call the office when they can't find the program they want after it's filtered out of the list. There's nothing stopping you from changing this behavior, however.

Thanks for your quick feedback.
I will investigate and attempt this method.   Note that the Institution is
also a lookup for the Program | Department__c field and there are
common/duplicate Program names such that Alabama and Florida may have a
"Master of Psychology" program.
To make sure I understand correctly, I am adding the Institution names to
the School/College filter of a particular Program.  This should reduce my
Program list of 270, since there are similar Programs across Institutions.
If so, I would need feed the Program filter the Institution name from the
question response (which is my initial issue that I am having difficulty
tracking).
My interface has

Question 1: Semester - lookup for Academic Term object
Question 2: Institution - lookup (or pick list) for Account object
Question 3: Program - lookup/filter based for Program object (based on
Institution Question 2 Response)

I will let you know if I am successful.
On Tue, May 14, 2019, 6:04 AM Kyle Schmid<EMAIL_ADDRESS>wrote:

@softwareangel https://github.com/softwareangel Why not use one of the
filtering picklists (i.e. School/College)? You could add picklist values
for each institution and select the relevant institution on each program.
You might need to make a small modification to the area around line 250 in
the ApplicationCreate class. By default, EASY doesn't restrict the program
list when users pick values and instead displays an error message when they
try to create the application with an invalid program. The reason for this
is a business one: a common complaint by admissions staff is that users
call the office when they can't find the program they want after it's
filtered out of the list. There's nothing stopping you from changing this
behavior, however.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/EASY-Open-Source-Solution/issues/11?email_source=notifications&email_token=AK23UORJR5XYYZZXT5YQ4NTPVKFETA5CNFSM4HMU4TAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVK7U3A#issuecomment-492173932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK23UOS5D73T2PLI22P2RYDPVKFETANCNFSM4HMU4TAA
.

Greetings,
I implemented your suggestion, by using the school/college filter for
Institutions.  Unfortunately, this revealed another issue with invoking too
many queries to the Program object.  It appears that Question
object/Reference record type invoke queries when the Checklist object is
loaded in order to populate the picklist.  I assumed there was trigger that
did this in real time upon selection (and type-ahead has never worked as
advertised, just blank results).
When reviewing the interface, the logs show 72 SOQLs invoked, in real-time,
the logs show SOQLs exceeding the 101 limitation (with a possible 210 SOQLs
for the entire application).
I cannot used the available Intended Term/Program feature
(ApplicationCreate class - line 250), since this does not allow the
applicant to modify at-will according to given requirements; therefore, a
mirrored-version that can display in the Checklist was built.
I am looking to add a trigger to the Question object so that the Reference
record type will populate picklists upon selection.  This may impede
performance, but will remove the SOQL errors while loading the Checklist.
Thanks for your help.
Raul R. Edwards - smartphone
ENOVATIONAL CORP
On Tue, May 14, 2019, 6:42 AM Raul R. Edwards<EMAIL_ADDRESS>wrote:

Thanks for your quick feedback.
I will investigate and attempt this method.   Note that the Institution is
also a lookup for the Program | Department__c field and there are
common/duplicate Program names such that Alabama and Florida may have a
"Master of Psychology" program.
To make sure I understand correctly, I am adding the Institution names to
the School/College filter of a particular Program.  This should reduce my
Program list of 270, since there are similar Programs across Institutions.
If so, I would need feed the Program filter the Institution name from the
question response (which is my initial issue that I am having difficulty
tracking).
My interface has

Question 1: Semester - lookup for Academic Term object
Question 2: Institution - lookup (or pick list) for Account object
Question 3: Program - lookup/filter based for Program object (based on
Institution Question 2 Response)

I will let you know if I am successful.
On Tue, May 14, 2019, 6:04 AM Kyle Schmid<EMAIL_ADDRESS>wrote:

@softwareangel https://github.com/softwareangel Why not use one of the
filtering picklists (i.e. School/College)? You could add picklist values
for each institution and select the relevant institution on each program.
You might need to make a small modification to the area around line 250 in
the ApplicationCreate class. By default, EASY doesn't restrict the program
list when users pick values and instead displays an error message when they
try to create the application with an invalid program. The reason for this
is a business one: a common complaint by admissions staff is that users
call the office when they can't find the program they want after it's
filtered out of the list. There's nothing stopping you from changing this
behavior, however.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/EASY-Open-Source-Solution/issues/11?email_source=notifications&email_token=AK23UORJR5XYYZZXT5YQ4NTPVKFETA5CNFSM4HMU4TAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVK7U3A#issuecomment-492173932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK23UOS5D73T2PLI22P2RYDPVKFETANCNFSM4HMU4TAA
.


[Update term] DevSecOps for En

1. Please describe why this term needs to be updated
Why this term needs to be updated? (briefly)

In the Problem it addresses section, the term continuous deployment currently links to the continuous delivery page. I believe this need to be corrected.
Perhaps we need to incorporate both continuous deployment and continuous delivery into the description.

2. Checklist

[X] I understand that maintainers will confirm if the term should be updated. (Please be patient)
[X] I want to work on this suggestion and confirm that I meet the criteria as outlined in "who can contribute".. (If not checked, maintainers will assign another volunteer)

(Note)
Maintainers will assign a label after assessment procedure.
(from triage/awaiting to triage/accepted or triage/not accepted)
A contributor can be assigned to this issue after the issue get the triage/accepted label.
The assignee needs to check Style Guide.

/assign

triage/accepted

I want to work on this issue

triage/accepted

@Nitishupkr, only maintainers can accept or decline terms. Also, as you can see @tico88612 would like to work on this.
@tico88612, sorry, just seeing this. Let me get back to you shortly.

Hi @tico88612, thanks for this. I've accepted triage, so feel free to open a PR with your suggested changes.


v33.0.0: Failed to upgrade CRD

Welcome!

[x] Yes, I've searched similar issues on GitHub and didn't find any.
[x] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What version of the Traefik's Helm Chart are you using?
v33.0.0
What version of Traefik are you using?
v3.1.6
What did you do?

Upgrade traefik helm to v33.0.0 via fluxcd reports but failed
Manually update CRD: kubectl apply --server-side --force-conflicts -k https://github.com/traefik/traefik-helm-chart/traefik/crds/

What did you see instead?
fluxcd helm ugrade reports:
traefik	32.1.1	False	Unknown	Running 'upgrade' action with timeout of 5m0s
traefik	32.1.1	False	False	Helm upgrade failed for release traefik/traefik with chart<EMAIL_ADDRESS>failed to apply CustomResourceDefinitions: failed to update CustomResourceDefinition(s): failed to replace object: CustomResourceDefinition.apiextensions.k8s.io "grpcroutes.gateway.networking.k8s.io" is invalid: status.storedVersions[0]: Invalid value: "v1alpha2": must appear in spec.versions

Manually update CRD:
customresourcedefinition.apiextensions.k8s.io/accesscontrolpolicies.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apiaccesses.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apibundles.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apiplans.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apiportals.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apiratelimits.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apis.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/apiversions.hub.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/gatewayclasses.gateway.networking.k8s.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/gateways.gateway.networking.k8s.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/httproutes.gateway.networking.k8s.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/ingressroutes.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/ingressroutetcps.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/ingressrouteudps.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/middlewares.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/middlewaretcps.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/referencegrants.gateway.networking.k8s.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/serverstransports.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/serverstransporttcps.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/tlsoptions.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/tlsstores.traefik.io serverside-applied
customresourcedefinition.apiextensions.k8s.io/traefikservices.traefik.io serverside-applied
The CustomResourceDefinition "grpcroutes.gateway.networking.k8s.io" is invalid: status.storedVersions[0]: Invalid value: "v1alpha2": must appear in spec.versions

What is your environment & configuration?
K8S version: v1.31.2+k3s1
Install traefik helm: v32.1.1
Additional Information
No response

same problem here when upgrading from v32.1.1 to v33.0.0 on k0s and talos clusters.

There is no v1alpha2 in crds of v33:
$ grep -rn v1alpha2 traefik/crds/
$

It looks like that an other component in your environment is using a different (previous ?) version of Gateway API.
Considering helm known limitation on CRDs, there is a WIP to move CRD into a dedicated chart, see #1141 and PR #1223.
In the mean time, release notes of v1.2 of Gateway API contains instructions on how to upgrade when you have v1alpha2 reference:

Note that even if you've been using v1 in your YAML manifests, a controller may still be using v1alpha2 which would cause it to fail during this upgrade.

If you ran into issues installing these CRDs, it likely means that you have v1alpha2 in the storedVersions of one or both of these CRDs. This field is used to indicate which API versions have ever been used to persist one of these resources.

Unfortunately, this field is not automatically pruned.

Wdyt about following instruction provided on Gateway v1.2 release to fix it ?

thank you, this worked and fixed it for me now.
i think it makes sense to include this in the release notes for people to are aware of this process. 👍


Feature request: run before.update only once with multi: true

I'd love to be able to run a before.update hook only once for a { multi: true } update.
Use case:
I change the modifier to set two fields on one of my collections, lastModifiedBy and lastModifiedDate, but it runs for every affected document, which is unnecessary, I don't need to know what any of the documents look like, just need to be sure that they get their last modified timestamp and userId updated.

Yeah, this is a side effect of the update hooks not taking place as a single function call. I think we were talking about this in another issue as well, but can't find that at the moment.

You could try debouncing it, either manually with a setTimeout 0 that you just cancel on each iteration:
Meteor.clearTimeout(t);
t = Meteor.setTimeout(function () {
  // do that update to your other collection for lastModifiedBy and lastModifiedDate
}, 0);

Or use _.debounce with delay of 0 -- just to group all those calls into a single one that runs on next tick.
This of course will only work if all those updates fire during the same tick, which I assume they do (fingers crossed)


opt: use conflict column IDs rather than names

PR: https://github.com/cockroachdb/cockroach/pull/45280
From release notes:

INSERT..ON CONFLICT index column names can now be specified in any order, rather than only in the same order as the index. [#45280][#45280] {% comment %}doc{% endcomment %}

The limitation lifted by https://github.com/cockroachdb/cockroach/pull/45280 was not previously documented, and it doesn't look like the PR otherwise changes any expected behavior or syntax of the INSERT ... ON CONFLICT statement.
Closing this issue. @andy-kimball please reopen if you disagree.

It turns what was previously an error case into a working case. And fixes a compat issue with PG.


Add sorting by station signals

Please add sorting by station control signals (compare them as character string). I have few hundreds of stations, sorting will help to debug wrong signaling of stations - if list is sorted, I can easily detect any inconsistency in signals.

In for v0.4.0.


Configurable company details

This fixes #37 by adding a new option to the company: hash for customizing what details are displayed in the company section. Conveniently, it also allows for reordering of those details as well.
Naming isn't exactly right yet, but I just wanted to get a prototype together.

@zaibacu this is what I had in mind.

Yeah, should so it 🙂thanks!


[FEA] New Github workflow to add new issues to spark-rapids project

Is your feature request related to a problem? Please describe.
I would like to have a workflow added to add any new spark-rapids issue to the spark-rapids project: https://github.com/orgs/NVIDIA/projects/4.  The workflow can follow examples listed here: https://github.com/actions/add-to-project.
Exact syntax would be something like this:
name: Add new issues to spark-rapids project

on:
  issues:
    types:
      - opened

jobs:
  add-to-project:
    name: Add issue to spark-rapids project
    runs-on: ubuntu-latest
    steps:
      - uses: actions/add-to-project@RELEASE_VERSION
        with:
          # You can target a repository in a different organization
          # to the issue
          project-url: https://github.com/orgs/NVIDIA/projects/4
          github-token: ${{ secrets.ADD_TO_PROJECT_PAT }}

for github-token: xxx
please enable the github action, and try ${{ secrets.GITHUB_TOKEN }} first in your forked repo to see if the default token could cover the permission for Github Projects ops. e.g.
github-token: ${{ secrets.GITHUB_TOKEN }}

if not we could a dedicated token for this action later.


added controlled vocabulary yaml

Added file with the controlled vocabulary LinkML model
@chris-grove would you please review and let me know if it needs any corrections

@sierra-moxon
Could you just take a look at the header for this new 'controlledVocabulary.yaml' file and see that is looks OK?

looks great - just make sure to add the new yaml file to allianceModel.yaml -- as that is the "root" of our project that is used by all the generators to traverse our yaml files.
https://github.com/alliance-genome/agr_curation_schema/blob/main/model/schema/allianceModel.yaml

@sierra-moxon
OK, thanks for catching those! Running 'make all' now to check and will merge after that.


ci: bump pnpm/action-setup to 2.2.4

This PR:

[x] Bumps pnpm/action-setup to v2.2.4 to remove the GitHub Actions warnings related to save-state.

https://github.com/hybridly/hybridly/actions/runs/5337303313/jobs/9673163709#step:3:9

CI failure appears to be unrelated to this PR.

Thanks!


⚠ Samsung has degraded performance

In c420c4f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2273 ms

Resolved: Samsung performance has improved in 847563a.


"IMPROVE THIS DOC" button doesn't work

The button "improve this doc" doesn't work on some tutorials. When you click it, 404 is caused.
For example, on the tutorial, the button redirects you here, which is 404.

Thanks for the report, there is an issue tracking that here: https://github.com/resin-io/docs/issues/563 , so I will close this one.


xds: fix logging all errors as timeouts for delta XDS

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
true
/test integ-telemetry-istiodremote

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-istiodremote_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
true
/test integ-telemetry-istiodremote

integ-telemetry_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
true
/test integ-telemetry

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

/retest

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
true
/test integ-telemetry

integ-telemetry-istiodremote_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
true
/test integ-telemetry-istiodremote

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

/test integ-telemetry-istiodremote

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry_istio
b8f334fea3aa564d60f29ef5e2de24a2ee89999b
link
unknown
/test integ-telemetry

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

/test integ-telemetry


DeviceGuard WMI class is available in Win10 Pro

This statement-
"The Win32_DeviceGuard WMI class is only available on the Enterprise edition of Windows 10 and Windows 11."
is inaccurate.  That WMI class appears to be present in Windows 10 Professional Edition.

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

ID: 1d4b8d75-5d09-e0d6-1a90-45175fd1043d
Version Independent ID: dfb10d10-a514-6ea5-1307-df865db530db
Content: Enable virtualization-based protection of code integrity - Windows security
Content Source: windows/security/threat-protection/device-guard/enable-virtualization-based-protection-of-code-integrity.md
Product: windows-client
Technology: itpro-security
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

@Embrik , I will check it again.

@Embrik and @RAJU2529 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/windows-itpro-docs/pull/11051
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @Embrik Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Develop

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@13665e8). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master   #135   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?    199
  Lines             ?   3430
  Branches          ?    505
=======================================
  Hits              ?   3430
  Misses            ?      0
  Partials          ?      0

Impacted Files
Coverage Δ

...ProblemsDiagnosisDetail/ProblemsDiagnosisDetail.js
100% <ø> (ø)

...lemsDiagnosisDetail/ProblemsDiagnosisDetailForm.js
100% <ø> (ø)

...Allergies/AllergiesDetail/AllergyDetailMainForm.js
100% <ø> (ø)

...components/pages/UserProfile/forms/PersonalForm.js
100% <ø> (ø)

src/components/containers/TopHeader/TopHeader.js
100% <ø> (ø)

...pages/Allergies/AllergiesDetail/AllergiesDetail.js
100% <ø> (ø)

...Medications/MedicationsDetail/MedicationsDetail.js
100% <ø> (ø)

.../components/pages/UserProfile/forms/ContactForm.js
100% <ø> (ø)

...cations/MedicationsDetail/MedicationsDetailForm.js
100% <ø> (ø)

...ages/UserProfile/panels/ContactInformationPanel.js
100% <100%> (ø)

... and 28 more

Continue to review full report at Codecov.

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


abigen skip type alias - when we are using vector of that new type

In this case, we define a new type new_type to be an alias of uint32_t. And then we declare the table struct called sample that contains vector of the new_type.
#include <eosiolib/eosio.hpp>

using namespace eosio;
using namespace std;

CONTRACT test : contract {
  public:
      using contract::contract;

      typedef uint32_t new_type;

      TABLE sample {
        uint64_t p;
        vector<new_type> vector_of_new_type;
        auto primary_key () const { return p; };
      };

      typedef eosio::multi_index<"sampletable"_n, sample> sample_table;

      ACTION hi( name user ) {
        sample_table _sample_table(_self, _self.value);
        _sample_table.emplace(_self, [&](auto& sample) {
          sample.p = _sample_table.available_primary_key();
          new_type x = 5;
          sample.vector_of_new_type = {x};
        });
        print( "Hello");
      }

};

EOSIO_DISPATCH( test, (hi) )

When we generate the abi, the abigen doesn't detect the new_type, i.e.
{
    "____comment": "This file was generated with eosio-abigen. DO NOT EDIT Wed Oct 24 14:58:53 2018",
    "version": "eosio::abi/1.0",
...
"types": [],
...
}

And hence, when we are pushing an action to that contract, we will get
Error 3015004: The type defined in the ABI is invalid
Error Details:
new_type[]

resolved by PR #208


[query]weixi[wechat]demo failed

wxdemo.zip
I was trying to make a simple demo: open WeChat app window which is in dock and then search someone, then click the search result in WeChat app and input a word in chat box. but in the very first step, it failed.
not sure what went wrong

Sorry @riverbuilding, I was away the past few months - https://github.com/tebelorg/RPA-Python/issues/144
Can I check if you have followed the installation instructions here - https://tagui.readthedocs.io/en/latest/setup.html
Make sure OpenJDK is installed so that the visual automation engine can run. Your automation script looks correct.
Can you also share the contents of the log file tagui/src/tagui.sikuli/tagui_windows.log or tagui.log (macOS/Linux)?

(closing issue for now till further inputs)


[Regression Issue] - git-lfs command not found in v8.35

In the latest release (v8.35), the git-lfs command not found bug has resurfaced. The old workaround still resolves the issue:
launchctl setenv PATH $PATH
open -a SourceGit

It's worth noting that this bug is not present in v8.34.
Related issues: #168, #507

Currently, I've removed the code that setting PATH for git subprocess. Instead, SourceGit will try to read PATH env from zsh at startup. If it is failed, open -a SourceGit from commandline will still work.
PS: launchctl setenv PATH $PATH is not needed

I have pushed my final solution on this issue. The solution no longer depends on the user's environment, but instead reads the PATH variable from a file.
You can download the latest CI build from https://github.com/sourcegit-scm/sourcegit/actions/runs/11435321999
After download it, you must run echo $PATH > ~/Library/Application\ Support/SourceGit/PATH to generate the custom PATH file for SourceGit. Once it is created, SourceGit will use the content of it for PATH env used by git subprocess.


lodash is still referenced in the compiled output of the quicktype-core package

It seems like lodash is being referenced in the compiled (dist/) output. If you attempt to use the package, it will throw an error.

Error: Cannot find module 'lodash'
Require stack:

node_modules/quicktype-core/dist/language/Dart.js
node_modules/quicktype-core/dist/language/All.js
node_modules/quicktype-core/dist/Run.js
node_modules/quicktype-core/dist/index.js

To replicate, start a new repo:

npm install quicktype-core
grep lodash node_modules/quicktype-core/dist/**/*.js

You'll find the line in dist/language/Dart.js

It seems that https://github.com/quicktype/quicktype/commit/679b7bcd9d11fc06b449b16b200ae05a6307767f wasn't published to npm. @dvdsgl I'm sorry for pinging you here, but do you have power of publish new working package?

Is there a workaround for this other than using a fork?

Is there a workaround for this other than forking?

Install lodash as a dependency
npm i -D lodash

#1870


Some minor tweaks

feel free to cherry pick any you want

Hi Simon - thanks a lot for your contribution.


python.pythonPath in user settings not working for venv

VS Code version
1.63.2
Extension version
v2021.12.1559732655
OS type
macOS
OS version
10.15.7 (Catalina)
Python distribution
Other
Python version
3.9.7
Language server
Pylance
Expected behaviour
If I set the python.pythonPath setting in my workspace settings to a local venv, I'd expect VS Code to use that venv as the selected Python interpreter.
However, what is happening for me on exactly one of my projects is that this setting is not being respected.  Note that all my other projects, this is working as expected.
Actual behaviour
In my Workspace settings I have:
{
    "python.pythonPath": "/Users/adam.parkin/temp/sandbox/myproject/.venv/bin/python",
}

But (even after a restart) the status bar is showing that the selected Python interpreter is a different environment (not a venv), and VS Code is also telling me that tools that are installed in that venv (black, pylint, etc) aren't installed.
When I have activateTerminal set to true & open up a terminal, I see it point to a pyenv shim:
$ which python
/Users/adam.parkin/.pyenv/shims/python

Steps to reproduce
I don't know how to reproduce as this is only happening on one project and is persisting past restart, but this image shows my confusion (sensitive bits are blurred out):

Logs
I don't see Python as an option in the Output Panel for this project:

Code of Conduct

[X] I agree to follow this project's Code of Conduct

And magically now after filing this issue it works as expected.  Apologies for the confusion.


removeHs not removing H from SetAtomicNum(1)

Description:
I'm trying to replace dummy atoms with hydrogens with SetAtomicNum(1) and then remove the hydrogens with RemoveHs, and I've come across a case where it's not working as I expect it to.

RDKit Version: 2018.09.1

s = '[CH3]\[CH]=[CH]\[*]'

mol = Chem.MolFromSmiles(s)
for a in mol.GetAtoms():
    if a.GetSymbol() == '*':
        a.SetAtomicNum(1)
mol = Chem.AddHs(mol)
mol = Chem.RemoveHs(mol)
print(Chem.MolToSmiles(mol))

// output: [H]/C=C\C

I was expecting the output to be [CH2]=[CH][CH3], (three carbons, no hydrogens, no double bond stereochem). Instead I find that the output still has a hydrogen as well as double bond stereochem. Is there a proper way to remove that lingering hydrogen?
Interestingly, if the input is s = 'C\C=C\*', I find that the output is C=CC. Not sure how to reconcile these observations.

I believe what’s happening here is that the RDKit is not removing the H atom because it is determining the stereochemistry of the double bond.

Thanks for your response Greg. In my molecule though, the lingering hydrogen should not be used in defining the stereochemistry of the double bond, since the other atom that the parent carbon is connected to is another hydrogen, so I would expect that double bond to no longer have any stereochemistry after replacing the dummy atom with a hydrogen. I think the problem is that the double bond no longer had any StereoAtoms, but the neighboring bonds still had BondDirs of ENDUPRIGHT/ENDDOWNRIGHT. Using AssignStereochemistry() did not fix it either.
I ended up implementing a workaround that involved cleaning up any lingering BondDirs, running AssignStereoChemistry with cleanIt=True, and then one final RemoveHs:
def print_bond_info(mol):
    print()
    for b in mol.GetBonds():
        print(b.GetStereo(), list(b.GetStereoAtoms()), b.GetBondDir())
    print()

s = '[CH3]\[CH]=[CH]\[*]'

mol = Chem.MolFromSmiles(s)
print("Original: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

for a in mol.GetAtoms():
    if a.GetSymbol() == '*':
        a.SetAtomicNum(1)
print("Change * to H: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol = Chem.AddHs(mol)
print("Add H's: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol = Chem.RemoveHs(mol)
print("Remove H's: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

bonds_to_change = []
for b in mol.GetBonds():
     if b.GetStereo() != Chem.BondStereo.STEREONONE and not list(b.GetStereoAtoms()):
            for a in [b.GetBeginAtom(), b.GetEndAtom()]:
                neighs = a.GetNeighbors()
                for neigh in neighs:
                    b_of_interest = mol.GetBondBetweenAtoms(a.GetIdx(), neigh.GetIdx())
                    if b_of_interest.GetBondDir() != Chem.BondDir.NONE:
                        bonds_to_change.append(b_of_interest)

for b in bonds_to_change:
    b.SetBondDir(Chem.BondDir.NONE)
print("Change BondDirs: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

Chem.AssignStereochemistry(mol, force=True, cleanIt=True)
print("AssignStereochemistry: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol = Chem.RemoveHs(mol, updateExplicitCount=True)
print("Final Remove H's: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

The final output of C=CC was what I expected. Here is the full output:
Original:  */C=C/C

STEREONONE [] ENDDOWNRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDDOWNRIGHT

Change * to H:  [H]/C=C/C

STEREONONE [] ENDDOWNRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDDOWNRIGHT

Add H's:  [H]C([H])=C([H])C([H])([H])[H]

STEREONONE [] ENDDOWNRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDDOWNRIGHT
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE

Remove H's:  [H]/C=C\C

STEREONONE [] ENDDOWNRIGHT
STEREOE [] NONE
STEREONONE [] ENDUPRIGHT

Change BondDirs:  [H]C=CC

STEREONONE [] NONE
STEREOE [] NONE
STEREONONE [] NONE

AssignStereochemistry:  [H]C=CC

STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE

Final Remove H's:  C=CC

STEREONONE [] NONE
STEREONONE [] NONE

AssignStereochemistry  really should help with this.
Here's what I observe:
In [36]: mol = Chem.MolFromSmiles('C/C=C/[*]')

In [37]: mol.GetAtomWithIdx(3).SetAtomicNum(1)

In [38]: mol = Chem.AddHs(mol)

In [39]: Chem.AssignStereochemistry(mol,force=True,cleanIt=True)

In [40]: Chem.MolToSmiles(Chem.RemoveHs(mol))
Out[40]: 'C=CC'

Which, I think, is what's expected. Can you please try that?

Okay yes, I got what you got. I see what you did differently than me was that you ran AssignStereochemistry after adding H's and before removing H's. It looks like in the final mol, one of the BondDirs is still ENDUPRIGHT though. Not sure how important that is, or if I should set the BondDir to NONE.
mol = Chem.MolFromSmiles('C/C=C/[*]')
print("Original: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol.GetAtomWithIdx(3).SetAtomicNum(1)
print("Change * to H: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol = Chem.AddHs(mol)
print("Add H's: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

Chem.AssignStereochemistry(mol,force=True,cleanIt=True)
print("AssignStereochemistry: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

mol = Chem.RemoveHs(mol)
print("Remove H's: ", Chem.MolToSmiles(mol))
print_bond_info(mol)

Output:
Original:  */C=C/C

STEREONONE [] ENDUPRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDUPRIGHT

Change * to H:  [H]/C=C/C

STEREONONE [] ENDUPRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDUPRIGHT

Add H's:  [H]C([H])=C([H])C([H])([H])[H]

STEREONONE [] ENDUPRIGHT
STEREOE [0, 3] NONE
STEREONONE [] ENDUPRIGHT
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE

AssignStereochemistry:  [H]C([H])=C([H])C([H])([H])[H]

STEREONONE [] ENDUPRIGHT
STEREONONE [] NONE
STEREONONE [] ENDUPRIGHT
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE
STEREONONE [] NONE

Remove H's:  C=CC

STEREONONE [] ENDUPRIGHT
STEREONONE [] NONE

Thanks again for your help!

I'm closing this because it looks like the question is answered. Please reopen if necessary.


1898

I could not easily create a one commit

Admin, can CI be enabled?

Did I get it right that squashing commits didn't succeed? What problem did you encounter? That's not such a bit deal with this issue PR, but with any larger PR having a clean history is important. If you are new to squashing, it would be a good idea to practice with this one.
Also, is this or #2000 currently active? Could you close the one that's not?

@pekkaklarck yes squashing did not succeed easily as i did  not have dedicated branch for this issue. Learning for me for the future is to create branch first :)
Closed not needed pull request.

Yes, I've just learned to do pretty much everything in a branch myself.
I assume this is the branch to be merged. Can you take a look the latest line notes I added and then try squashing this branch?

@wojciechtanski, do you think you could get this PR finalised today? We'd like to get it into 2.9a2 that we are planning to release tomorrow.


Manage WSL2 hostnames on Windows side where necessary

The Problem/Issue/Bug:
How this PR Solves The Problem:
Manual Testing Instructions:
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

Rebased and fixed a couple of linting errors.

This doesn't currently build for windows at all,
$ make windows_amd64
make: Circular .gotmp/bin/windows_amd64/ddev.exe <- windows_amd64 dependency dropped.
building .gotmp/bin/windows_amd64/ddev.exe from ./cmd/... ./pkg/...
# github.com/drud/ddev/pkg/exec
pkg/exec/exec_windows.go:33:28: not enough arguments in call to "golang.org/x/sys/windows".CreateFileMapping
	have ()
	want ("golang.org/x/sys/windows".Handle, *"golang.org/x/sys/windows".SecurityAttributes, uint32, uint32, uint32, *uint16)
pkg/exec/exec_windows.go:34:24: not enough arguments in call to "golang.org/x/sys/windows".MapViewOfFile
	have ()
	want ("golang.org/x/sys/windows".Handle, uint32, uint32, uint32, uintptr)
pkg/exec/exec_windows.go:41:31: undefined: event
make: *** [.gotmp/bin/windows_amd64/ddev.exe] Error 2

But I guess we only care about how it builds for linux? It does build on linux, haven't tried the functionality yet.

Manually testing this on WSL2, it seems not to change behavior at all; we still have the exact same problem. I haven't stepped through the code yet.

Rebased.

Given that MS is supposed to provide native X11 support "real soon now", it's going to be simpler for WSL2 users to use the browser inside wsl2.
IMO we should keep things fairly simple instead of trying to solve this problem with everything we have. I'm afraid that the launching into Windows would add complexity that we'd never unravel.
So if it's OK, I'd like to abandon this approach right now.  Closing for now, but we can reopen if we see fit.


[Python] add docs about how to run tests locally (or even simplify it if it's too complex)

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Related to https://github.com/apache/skywalking-python/pull/214#pullrequestreview-1026894959 .
Our contributor has to look around the repo for demo scripts to run the tests locally. Sometimes they have to try over and over. I'd expect the guide to run tests locally should be clearly documented step by step.
We should add a new section in https://github.com/apache/skywalking-python/blob/master/docs/en/contribution/PluginTest.md
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I write a guide to local test in this commit. Do you have any suggestions for this before I pull request.

You could send a pull request directly. It is fine if more polish could be done.

Got it.

Closed by apache/skywalking-python#222


Non-git repository causes Code to grab all files on Mac

When opening a non-git repository in Code, the Git tab seems to pull in every file on my Mac. In Windows, if the folder isn't a git repository, a button to Initialize a git repository is displayed.

What do you mean by grab and pull in? Can you be more descriptive? Can you provide a screenshot?

As shown in the picture, I opened my Documents folder, which contains 2 sub-folders, each with 3 or 4 files. However, the git tab shows there are 20444 files that have yet to be committed. From what I can tell, it is showing every file on my computer.

You must've accidentally created a git repository in your user home folder.
Open a Terminal, and type ls ~/.git. This will confirm that. You can remove it with rm -r ~/.git.

Perfect.   Thank you!
Cory Johnson
801-360-7864
On Wed, Jan 6, 2016 at 7:03 AM -0800, "João Moreno"<EMAIL_ADDRESS>wrote:
You must've accidentally created a git repository in your user home folder.
Open a Terminal, and type ls ~/.git. This will confirm that. You can remove it with rm -r ~/.git.

Reply to this email directly or view it on GitHub:
https://github.com/Microsoft/vscode/issues/1817#issuecomment-169346535


js name space

Hi,
I have a problem with the js namespace and, while debugging, I saw that the jquery plugins are defined directly referencing window.jQuery (text_widget.js) and window.$(in all other files).
It would be better, for managing collisions, with other js plugins, put all the definitions inside a classic
(function($){
$.fn.yourlabsRegistry = function(key, value) {...}
}(yourlabs.jQuery))
or something like this.
I can write a patch, if needed.
Thank you for your work.

Agreed.
A pull request for this would be great!

+1 !

+1 !
I think you can submit patches easy on GitHub ie. on https://github.com/yourlabs/django-autocomplete-light/edit/master/autocomplete_light/static/autocomplete_light/addanother.js
Then travis will test it for you ;)
Nice JS skills BTW B)

Hi ! Next release will be issued in ~2-3 weeks, so you still have time to get your patch in if you want to benefit from updates.

Hi!
I was moved to another project and I have no time to work on it... I'm
sorry!
On Tue, Apr 7, 2015 at 4:58 AM, James Pic<EMAIL_ADDRESS>wrote:

Hi ! Next release will be issued in ~2-3 weeks, so you still have time to
get your patch in if you want to benefit from updates.
—
Reply to this email directly or view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/355#issuecomment-90329013
.

Thanks @steeeveb for reporting back and the initial idea, which still makes sense.
I'll reopen it therefore.

Moved to jquery-autocomplete-light#2


[#PCC-2601] Repositories Data cleanup

Added script that removes project details from disabled repositories

Signed-off-by: Harold Wanyama<EMAIL_ADDRESS>

How will you build this tool? I don't see an entry in the Makefile.

I have added the entry in the Makefile


"Both Modified" files are not handled

Just encountered this when I tried to solve merge conflicts.
Currently this is not handled at all.

they are now


Avoid unrelated git histories

Fixes #1082

Codecov Report

Merging #1084 into master will decrease coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #1084      +/-   ##
==========================================
- Coverage   62.66%   62.65%   -0.01%
==========================================
  Files         183      183
  Lines       10225    10227       +2
==========================================
  Hits         6407     6407
- Misses       3021     3022       +1
- Partials      797      798       +1

Impacted Files
Coverage Δ

pkg/backend/rcs/git/cli/git.go
64% <0%> (-0.74%)
:arrow_down:

pkg/action/clone.go
52.5% <100%> (ø)
: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 91975d8...dbbac39. Read the comment docs.


Issue with TfLiteMobileNetSsd-Armnn

Dear all,
I tried to run the TfLiteMobileNetSsd-Armnn example with release 19_02 but I have the following error
WARNING: Layer of type Activation is not supported on requested backend CpuAcc for data type Unsigned8 (reason: Neon: Logistic Activations unsupported with QAsymm8 data type.), falling back to the next backend.
ERROR: Layer of type Activation is not supported on any preferred backend [CpuAcc ]
InferenceModel error : Optimize returned nullptr
Is this example fixed in release 19_02?
Regards

That's odd, are you sure that you're using the 19.02 Compute Library?

Ah got it, Logistic is on CpuRef and GpuAcc, not on CpuAcc yet:
Logistic support for TensorFlow Lite parser.
Logistic support for GpuAcc backend.
So if you add CpuRef to the end of your list of compute devices, it should work fine. You'll need to do that anyway because the SSD Detection Postprocess layer is provided by CpuRef.

Thanks @MatthewARM ,
using both CpuAcc and CpuRef as compute device solve the issue


Fix intermittent failure related to TickWheelExecutor shutdown

I think I broke this in #1332.

Going to merge this one pretty aggressively.  It's just tests, and the flakiness I introduced is breaking other builds.


ReadItemStreamAsync with ResourceToken throws UnauthorizedException

Describe the bug
When using ReadItemStreamAsync and a ResourceToken, if the token does not have access to the resource DocumentClient is throwing an exception
                    if (!isTokenAvailable)
                    {
                        throw new UnauthorizedException(string.Format(
                           CultureInfo.InvariantCulture, ClientResources.AuthTokenNotFound, resourceAddress));
                    }

given that a Stream is being used my expectation is for the failure to be in the HTTP status code and not have do deal with the exception.
Additionally, given that UnauthorizedException is internal to Microsoft.Azure.Cosmos.Direct, it cannot be effectively caught.
Environment summary
SDK Version: Source
OS Version Windows 10

This is a bug. This exception should have been converted to a ResponseMessage somewhere higher in the stack. The caches have not been converted to be exception-less yet. The current workaround around is the code should catch the exception and convert it. It looks like during the lazy client initialization that loads the caches there is path that is not correctly handling it.

Closing as inactive/old and not reproducible on latest versions


Add transition to example CSS and add keyword

This adds a masterful transition to the sliders and also deku-component keyword to make it searchable on http://dekucomponents.com/.

Looks really masterful to me.


[BUG]: crash with Mod Manager by @Ony and Mod Profile Manager i guess

Which Mod?
Mod Profile Manager
Description of the bug/crash
just a crash that happened when clicking the mods tab in the main menu with the 2 mods enabled ( i couldnt find my player.log )
Steps To Reproduce
activate both mods and then access the mod menu
Player.log
Error in ModsScreen.ModsScreen.OnSpawn at (960.0, 540.0, 0.0)
System.NullReferenceException: Object reference not set to an instance of an object
at SaveGameModLoader.AllPatches+ModsScreen_AddModListButton.Postfix (ModsScreen __instance) [0x00176] in <8978e4f10b1b42ec8e484da70e77baf4>:0
at (wrapper dynamic-method) ModsScreen.ModsScreen.OnActivate_Patch6(ModsScreen)
at KScreen.Activate () [0x00017] in <1e2b7d5db95c4d6b84eabb7ca0270927>:0
at KScreen.OnSpawn () [0x0004e] in <1e2b7d5db95c4d6b84eabb7ca0270927>:0
at KMonoBehaviour.Spawn () [0x0005d] in <1e2b7d5db95c4d6b84eabb7ca0270927>:0
Build: U50-583750-S

@Nick56731 On Mod Profile Manager's steam page the mod manager by Ony is listed as incompatible, which makes sense. That is why it is crashing. You have to make a choice on which you want.

onys mods are sadly not built with any compatibility in mind, mod manager just rips out the entire mod screen and replaces it with that cluttered custom version.
cant fix that


Fix adapter imports

Fixes #1262

Current coverage is 92.97%

Merging #1263 into master will not affect coverage as of 1eb9319

@@            master   #1263   diff @@
======================================
  Files           84      84
  Stmts         5265    5265
  Branches       956     956
  Methods          0       0
======================================
  Hit           4895    4895
  Partial         11      11
  Missed         359     359

Review entire Coverage Diff as of 1eb9319

Powered by Codecov. Updated on successful CI builds.


Update jinja2 compiler syntax

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The recipe license should not include the word "License".

Looks like the link may no longer be valid. Appears kind of like a mirror URL. So maybe a different mirror should be used.

this is already done in the recipe now.


feature request: support startRandomMotion with lip-sync

I want to play a random motion in a group with lip-sync. And I am trying to call the motion method like this:
this.model.motion("shake", undefined, undefined, "https://url.to/sound.mp3", undefined, undefined);

The question is it does not play the given mp3.
Notice that the current implementation drops the sound parameter if the index is undefined:
https://github.com/RaSan147/pixi-live2d-display/blob/39cf653a632b5696635875af838810631ec8068a/src/Live2DModel.ts#L213-L217
Though there is a workaround is to pass a 0 as the index (but this limit to the first motion in the group), is it possible to make the startRandomMotion supports lipsync?

Working on it, thanks

yes you can do it now.
only use model.speak("url.to/audio.mp3")
this will play in the current state (the random motion)
see Readme examples

random motion shouldn't have sound, if you specify motion it'll no longer use random motion. I should remove the priority as well
(as for you know unlike Python, JS doesn't;t support positional arg, like skipping one, so skipping motion index means skipping everything else in the rest of the arg list, so the rest of the parts are redunded)


Filled SMBIOS part in DTB file

As UEFI for arm64 is available by u-boot, for the purpose of unified platform retrieval for different architectures, DMI tables should be available for IOT2050/ARM64 device as in the x86_64 devices.

DMI types are available under this link: DMI types
If it is possible, please add as many as of them.
Therefore, please fill smbios part of DTB file for IOT2050 device (smbios.txt).

Thanks in advance.

Sorry for the delay for this "simple" thing, we are on it.
A first upstream-aiming patch set can be found at https://github.com/siemens/u-boot/commits/jan/iot2050. An open question is which fields we must have in addition to this:
# dmidecode 3.3
Getting SMBIOS data from sysfs.
SMBIOS 3.0 present.
7 structures occupying 319 bytes.
Table at 0xFCED0020.

Handle 0x0000, DMI type 0, 24 bytes
BIOS Information
        Vendor: U-Boot
        Version: 2023.07-rc3-00038-g6409bcf6933-dirty
        Release Date: 07/01/2023
        ROM Size: 64 kB
        Characteristics:
                PCI is supported
                BIOS is upgradeable
                Selectable boot is supported
                Targeted content distribution is supported
                UEFI is supported
        BIOS Revision: 23.7

Handle 0x0001, DMI type 1, 27 bytes
System Information
        Manufacturer: Siemens
        Product Name: SIMATIC IOT2050 Advanced PG2
        Version: Not Specified
        Serial Number: Not Specified
        UUID: Not Settable
        Wake-up Type: Reserved
        SKU Number: Not Specified
        Family: SIMATIC IOT2050

Handle 0x0002, DMI type 2, 14 bytes
Base Board Information
        Manufacturer: Siemens
        Product Name: SIMATIC IOT2050 Advanced PG2
        Version: Not Specified
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Features:
                Board is a hosting board
        Location In Chassis: Not Specified
        Chassis Handle: 0x0000
        Type: Motherboard

Handle 0x0003, DMI type 3, 21 bytes
Chassis Information
        Manufacturer: Siemens
        Type: Desktop
        Lock: Not Present
        Version: Not Specified
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Boot-up State: Safe
        Power Supply State: Safe
        Thermal State: Safe
        Security Status: None
        OEM Information: 0x00000000
        Height: Unspecified
        Number Of Power Cords: Unspecified
        Contained Elements: 0

Handle 0x0004, DMI type 4, 48 bytes
Processor Information
        Socket Designation: Not Specified
        Type: Central Processor
        Family: Unknown
        Manufacturer: Unknown
        ID: 00 00 00 00 00 00 00 00
        Version: Unknown
        Voltage: Unknown
        External Clock: Unknown
        Max Speed: Unknown
        Current Speed: Unknown
        Status: Unpopulated
        Upgrade: None
        L1 Cache Handle: Not Provided
        L2 Cache Handle: Not Provided
        L3 Cache Handle: Not Provided
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Part Number: Not Specified
        Characteristics: None

Handle 0x0005, DMI type 32, 11 bytes
System Boot Information
        Status: No errors detected

Handle 0x0006, DMI type 127, 4 bytes
End Of Table

@BaochengSu

@jan-kiszka For the fields, I think we have to refer to the standard x86 IPCs and try to use the same field when possible.
And most fields are sourcing from the scratchpad iot2050-info, so I am working a sysinfo driver for iot2050 so that this information could be easily passed into the dmitable. Will send out the patch for reviewing and upstreaming once done, (now blocked by a sync data abort exception issue ...)

Will that driver cover static elements from a DT input like above as well?
In any case, the driver alone does not answer what we fill on, only how.

It will, the sysinfo uclass in u-boot allows you to add the static elements in the DT, together with the dynamic elements from somewhere such as the eeprom to feed to the dmitable. I've tried that in my dirty branch, it worked.
see smbios.txt, esp. the example.

FWIW, we finally have some driver and integration that should address this, see https://github.com/siemens/u-boot/commits/jan/iot2050. It's a preview and also still needs to be run via upstream, but feedback on the chosen (or missing) fields is welcome. Here some example:
root@iot2050-debian:~# dmidecode
# dmidecode 3.4
Getting SMBIOS data from sysfs.
SMBIOS 3.7.0 present.
# SMBIOS implementations newer than version 3.5.0 are not
# fully supported by this version of dmidecode.
Table at 0xFDF0E020.

Handle 0x0000, DMI type 0, 26 bytes
BIOS Information
        Vendor: U-Boot
        Version: 2024.10-rc2-00138-gdd5a8638ce15
        Release Date: 10/01/2024
        ROM Size: 64 kB
        Characteristics:
                PCI is supported
                BIOS is upgradeable
                Selectable boot is supported
                Targeted content distribution is supported
                UEFI is supported
        BIOS Revision: 24.10

Handle 0x0001, DMI type 1, 27 bytes
System Information
        Manufacturer: SIEMENS AG
        Product Name: SIMATIC IOT2050
        Version: 9MC1110-7UG00-0AA0
        Serial Number: R3S75099
        UUID: 1c557e38-cdf3-4b4d-a4c9-1d39377c6a00
        Wake-up Type: Unknown
        SKU Number: Not Specified
        Family: Not Specified

Handle 0x0002, DMI type 2, 15 bytes
Base Board Information
        Manufacturer: SIEMENS AG
        Product Name: A5E520802600AB02
        Version: IOT2050-ADVANCED-SM
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Features:
                Board is a hosting board
        Location In Chassis: Not Specified
        Chassis Handle: 0x0003
        Type: Motherboard
        Contained Object Handles: 0

Handle 0x0003, DMI type 3, 21 bytes
Chassis Information
        Manufacturer: Not Specified
        Type: Desktop
        Lock: Not Present
        Version: Not Specified
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Boot-up State: Safe
        Power Supply State: Safe
        Thermal State: Safe
        Security Status: None
        OEM Information: 0x00000000
        Height: Unspecified
        Number Of Power Cords: Unspecified
        Contained Elements: 0

Handle 0x0004, DMI type 4, 48 bytes
Processor Information
        Socket Designation: Not Specified
        Type: Central Processor
        Family: Unknown
        Manufacturer: Not Specified
        ID: 00 00 00 00 00 00 00 00
        Version: Not Specified
        Voltage: Unknown
        External Clock: Unknown
        Max Speed: Unknown
        Current Speed: Unknown
        Status: Unpopulated
        Upgrade: None
        L1 Cache Handle: Not Provided
        L2 Cache Handle: Not Provided
        L3 Cache Handle: Not Provided
        Serial Number: Not Specified
        Asset Tag: Not Specified
        Part Number: Not Specified
        Characteristics: None

Handle 0x0005, DMI type 32, 11 bytes
System Boot Information
        Status: No errors detected

Handle 0x0006, DMI type 127, 4 bytes
End Of Table

@huaqianli

If it is possible, I would say to fill "Processor Information" part more. For example, it would be nice to get CPU Max Speed from this. @jan-kiszka @huaqianli


Prevents logger from emitting: 'No handlers could be found for logger mutagenerate'.

Prevents logger from emitting: 'No handlers could be found for logger mutagenerate'.
Very low priority fix.

Coverage remained the same at 0.0% when pulling e2849be7c430caf3a1f62cfa192b193312455454 on mlschneid:master into 84a0c0c75b1772aaa83cb052dceb4b98f8513c65 on gtnx:master.

Coverage remained the same at 0.0% when pulling dbeeadc48d3ea194a6b5639bbef2ff898f8880c6 on mlschneid:master into 84a0c0c75b1772aaa83cb052dceb4b98f8513c65 on gtnx:master.

Coverage remained the same at 0.0% when pulling dbeeadc48d3ea194a6b5639bbef2ff898f8880c6 on mlschneid:master into 84a0c0c75b1772aaa83cb052dceb4b98f8513c65 on gtnx:master.

Coverage remained the same at 0.0% when pulling 73752cb3f0461ff775ea779eee483d89a51781a6 on mlschneid:master into 84a0c0c75b1772aaa83cb052dceb4b98f8513c65 on gtnx:master.


image-loader doesn't work with WkWebView

Hi,
Everything worked great before using WkWebView for my ios application https://github.com/apache/cordova-plugin-wkwebview-engine
I'm confused, because on simulator everything is ok too !! and no errors on debugging can be found.
on production and after installing on real devices images do not load.
can you help please ?

Hey there,
WkWebView doesn't allow access to anything outside the www root directory. This module stores the cached images outside the www directory (in the app's cache directory).
I'm thinking to solve this issue I can add a fallback for WkWebView and return base64 data instead of a file URI.

Thank you man ,
I really appreciate it, if there could be hot fix for this.

As you mentioned WkWebView doesn't allow access to anything outside the www! so I tried to set _cacheDirectoryName to www/cache and it worked for me.
Thank you

@mmirzaee did you do that through ImageLoaderConfig ?
When I looked into this I remember finding out that the cordova-plugin-file cannot write to the www directory (see here: https://github.com/apache/cordova-plugin-file#ios-file-system-layout). That directory is read-only on iOS platform. How did you manage to write files to it?

honestly I don't know much about cordova-plugin-file, I just updated 'image-loader-config.js' , line13 to:
this._cacheDirectoryName = 'www/cache';
and then it worked like a charm :)

@mmirzaee yeah that's the same as injecting ImageLoaderConfig and calling setCacheDirectoryName. This is strange because it's still being saved under the cache directory. (see https://github.com/zyramedia/ionic-image-loader/blob/master/src/providers/image-loader.ts#L188).
Thanks for the update though, I will keep this mind in case others have the same issue.

I tried using other dir names ( like 'cache','www2/cache') to find out why it's really working!
the cool fact was that 'www' must be in cache path ! maybe it's a bug of wkwebview ! ( I really dont know )
this._cacheDirectoryName = 'www/cache'; // works fine this._cacheDirectoryName = 'www'; // works fine this._cacheDirectoryName = 'www2'; // doesn't work this._cacheDirectoryName = 'anythingelse'; //doesn't work

@mmirzaee that's good to know. Thanks for the update!

Unfortunately that didn't solve it for me
Using setCacheDirectoryName is just setting the directory within the Librarycache folder
What @mmirzaee faced was probably the images loading/showing initially (as they're loaded from url) but on a page reload/refresh, WkWebView will still fail to load locally cached files.


Improve existing commands for adding more roles when SC is ready

New roles for collection ESDT when issuing and special roles.
more info: https://github.com/juliancwirko/elven-nft-minter-sc/issues/36

handling token properties added in v1.26.0
special roles will be added in: https://github.com/orgs/ElvenTools/projects/6?pane=issue&itemId=47243916


Question that is not really related to the product, but why using extensions?

Hi Developers in Ramotion,
I was reading the example code included, and noticed that you are using extensions a lot. For example in DemoTableViewController, every methods are written as extensions.
Is there any reason/benefit why you are writing it this way?
BR
AnChin

I wanna know as well..

Swift 1 did not have pragma marks, which made extensions become the standard for code organization at the time.
The guidelines do not enforce this anymore, but it's still widely used.
https://swift.org/documentation/api-design-guidelines/

it's just coding style, read this article https://www.natashatherobot.com/using-swift-extensions/
... and it's more readable


Migrate usage of Ethers to Alloy in Steel

With Ethers being deprecated it makes a lot of sense to move Steel to alloy.

Addressed with https://github.com/risc0/risc0-ethereum/pull/183


update and test with new https:// services

cc @AlexanderPico (just FYI)

Done!
https://github.com/egonw/rwikipathways/pull/3

Thanks!


No auto-detect in notebook cell language picker

Testing #131451

Have "workbench.editor.languageDetection": true.
Open a .ipynb file with the Notebook editor.
Click in a python cell.
Open the language picker for the cell.

@alexr00 just to confirm... you tested this with the latest insiders? I added it yesterday which is why I ask.

Works for me

I think the latest build hadn't been released yet. Works now that I'm on yesterday's build!


sokol_app.h: fix for assignment within conditional expression

Oops how embarrasing :) I need to figure out why I didn't get a warning. Thanks for the fix!

...ok I didn't get a warning because it's Win32 specific code, it's a warning-level-4 warning on MSVC and I only compile the sokol-sapp samples with warning level 3. I'll see what it takes to make the code warning-level-4 clean on MSVC.