Validation on custom component with after does not work properly

Versions

vee-validate: 2.2.7
vue: 2.6.10

Describe the bug
I've encountered 2 problems with my custom datepicker component with after rule.
Firstly, validation with after rule does not work until I change its value.
Secondly, when using cross-field validation,  a field with after rule uses old value of a field with ref.
I found workaround for these problem that use $nexttick, but that is not a clean way.
Is there any way to avoid these ?
To reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/xlwpx8xwqq?fontsize=14

first problem

Set startDate which is after endDate
Validation on endDate does not execute
Set endDate
Validation start to work

second problem ( after validation for endDate start to work)

Set startDate which is after endDate
There appears no error.
Set startDate which is after endDate again
There appears after error.

Demo link
https://codesandbox.io/s/xlwpx8xwqq?fontsize=14

The first problem is by design, By default, vee-validate will not validate the target field unless both inputs have been interacted with by the user at least once. You can use .immediate modifier to force validation on mount.
As for the second problem, since vee-validate uses the value prop to validate, it sometimes gets behind in sync, since Vue applies updated asynchronously. Your workaround mitigates that with the next tick, a better approach tho is to add a $_veeValidate object on the component and handle value retrieval yourself like its being done with the Text input.
Instead of using a computed prop, I used the local component state to store the current input value in innerValue, then expose it after parsing to vee-validate:
  $_veeValidate: {
    name() {
      return this.label;
    },
    value() {
      return parse(this.innerValue);
    }
  },

Here is an updated example:
https://codesandbox.io/s/88pmwvqjz8

@logaretm
I understood the solution.
But I do not  want to my small component like Date.vue in example to depend on vee-validate internally.
So, I decide to use nexttick solution.
Thanks for your advice anyway 👍

@simezi I understand, you could take a look at ValidationProvider component. It doesn't have the shortcomings of the directive and solves a lot of issues.

@logaretm
Sorry for bothering you repeatedly.
I've tried ValidationProvider , and it almost worked.
But I found a little weird behavior.
Here is my sample code.
https://codesandbox.io/s/1r0kvz6v5j
When I change the value of Start Date , validation on Start and End works well.
However, When I change the value of End Date, It seems only validation on End Date works.
Is this an expected behavior?

@simezi This is a bug, due to the order of the fields registering, I fixed this and it should be in the next release in a day or two.

I appreciate your great support.
Thank you so much ❤


fixed internal server error when gathering subscriptions for stream without recipient id

Since Stream.recipient_id field can be null, gathering subscribers will always cause internal server error if you have such stream. It is better to return not full data and investigate why my stream was created without recipient as a separate task.
It fails few rows below at
recipient_ids = sorted([stream["recipient_id"] for stream in target_stream_dicts])
with error
TypeError: '<' not supported between instances of 'NoneType' and 'int'

I don't think recipient_id should ever be null for a stream in practice; while the database model allows that, it's intended only for the brief window during stream creation where the Recipient object hasn't been created yet.  I think any work we do to address this should aim to making that state not occur, because a lot of other code would ideally be able to rely on stream.recipient_id being set.
Can you talk about the background here -- e.g. has there been manual editing of the database on the system where you saw this state?
@mateuszmandera FYI

@timabbott thank you for quick reply! This state occurred to me after creating stream via API. That endpoint (subscribe users to non-existing stream) responded with 500, but stream was created in database, which is strange. Then after login all clients started seeing 500 because of original issue that I reported.
I've got your point and tomorrow I will dig into error, which happened while creating stream and let you know, or even update my PR if needed. Please do not close this PR yet
Thank you!

@timabbott Ok, so my issue was that on every deployment I apply database dump to zulip database and it was resetting sequence for Recipient pk, so just after creating stream this statement
 recipient = Recipient.objects.create(type_id=stream.id, type=Recipient.STREAM)
failed with error:
django.db.utils.IntegrityError: duplicate key value violates unique constraint "zerver_recipient_pkey"
So it's not an issue for me anymore, but it is possible to happen in the wild, so in my humble opinion we should merge it so users will be able to use zulip if it occur to someone again. It's up to you.
With this fix users will not see the broken stream in list
Currently users can not use zulip at all: just seeing Internal server error page
Thank you!

I think for anyone with a corrupted database like this, we probably do want things to 500 and get that fixed.  I'm going to close this PR, but we should do a separate PR to prevent that IntegrityError failure mode from resulting in a Stream without a Recipient object.
@abhijeetbodas2001 @mateuszmandera perhaps what we want to do here is add a @transaction.atomic in create_stream_if_needed, covering the start through the RealmAuditLog step.


Hey there I want to work on this issue.

hey! I want to work on this issue.

@sonukmrsah either choose an existing issue or create a relevant issue explaining what you are interested in contributing.

I want to fix typos or readme fixes basically it will be a non code contribution!

@sonukmrsah in that case whether or not your contribution will be accepted depends on the effort put in. We are currently not accepting very small non code fixes.

No problem!


Fix syntax errors of registry.md

Change plugin to plugins. Otherwise the configuration is ignored silently.

this repo is closed/archived .. we merged into the containerd/containerd project

note.. for service purposes we made the release/1.4 branch the current active branch for this archived repo vs master/main.. if you look at the release/1.4 branch you'll see the commit you made is already there :-) Cheers, Mike


Optional raw data on emitText

My XML contains HTML in some data nodes that is escaped on the emitText function.
What would be the best way to implement this (I can do and send a pull request)

add an option parameter (object) to return unescaped text?
expose the escape function?

Or any better way?

why not just wrapping your content with cdata?
example:
<![CDATA[ __your___html___content ]]>

It beats me why cdata is not used at all but I'm not the owner of the format. These are TMX files used for data exchange in TMS (Translation) systems as any change might break on some platforms I'm trying to stick to the samples.


[5.7.0] Disable match-module-cache-with-compiler.swift test on Linux

Cherry-picks https://github.com/apple/swift/pull/58818 to release/5.7.0.

Explanation: SourceKit/Misc/match-module-cache-with-compiler.swift started failing since we started supporting Ubuntu 22.04 in release/5.7.0. The test failure is because in CI builds, the clang compiler odes not live next to the swiftc compiler but in the llvm-linux-aarch64 build directory (see https://github.com/apple/swift/pull/58852 for more explanation), so it only manifests in CI and should not be an issue in released toolchains. So, just disable the test on release/5.7.
Scope: Just disables a test
Issue: rdar://98818484
Risk: Zero (just disables a test and the failure should not be happening with released toolchains)

@swift-ci Please test


feat: change hello to Hello

What does this PR do?
This pr fixes the "hello" to "Hello"
Fixes #269
How should this be tested?
Go to the home page and scroll down to the "The power of a type system" and check "Hello"
Mandatory Tasks
Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.
yes , I have self-reveiwed the code.

aww, this was intentionally lower case in the design 😅


Implement TO extensions plugin in Golang

https://github.com/apache/trafficcontrol/pull/2513

#2513 should have closed this.


[Help Request]sumo.tools.traci.exceptions.FatalTraCIError: Connection closed by SUMO.

High Level Description
When I am training in the comp-1 branch, the program always breaks, how to solve this problem?
Version
comp-1
Operating System
ubuntu22.04
Problems
Traceback (most recent call last):
File "train_ppo.py", line 83, in
state, dones = env.reset(), {"all": False}
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/core.py", line 251, in reset
return self.env.reset(**kwargs)
File "/data/zhangdi/SMARTS/smarts/env/multi_scenario_env.py", line 341, in reset
obs = self.env.reset(**kwargs)
File "/data/zhangdi/SMARTS/smarts/env/hiway_env.py", line 269, in reset
env_observations = self._smarts.reset(scenario)
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 410, in reset
raise first_exception
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 403, in reset
return self._reset(scenario, start_time)
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 440, in _reset
self.setup(scenario)
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 470, in setup
provider_state = self._setup_providers(self._scenario)
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 1112, in _setup_providers
new_provider_state = self._handle_provider(provider, provider_error)
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 1171, in _handle_provider
raise provider_error
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 1110, in _setup_providers
new_provider_state = provider.setup(scenario)
File "/data/zhangdi/SMARTS/smarts/core/sumo_traffic_simulation.py", line 334, in setup
self._traci_conn.simulation.subscribe(
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/sumo/tools/traci/_simulation.py", line 671, in subscribe
Domain.subscribe(self, "", varIDs, begin, end)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/sumo/tools/traci/domain.py", line 188, in subscribe
self._connection._subscribe(self._subscribeID, begin, end, objectID, varIDs, parameters)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/sumo/tools/traci/connection.py", line 295, in _subscribe
result = self._sendCmd(cmdID, (begin, end), objID, format, *args)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/sumo/tools/traci/connection.py", line 231, in _sendCmd
return self._sendExact()
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/sumo/tools/traci/connection.py", line 137, in _sendExact
raise FatalTraCIError("Connection closed by SUMO.")
sumo.tools.traci.exceptions.FatalTraCIError: Connection closed by SUMO.
Exception ignored in: <function SMARTS.del at 0x7fc409614940>
Traceback (most recent call last):
File "/data/zhangdi/SMARTS/smarts/core/smarts.py", line 856, in del
TypeError: 'NoneType' object is not callable
Exception ignored in: <function _ShowBaseInstance.del at 0x7fc406e73940>
Traceback (most recent call last):
File "/data/zhangdi/SMARTS/smarts/core/renderer.py", line 140, in del
File "/data/zhangdi/SMARTS/smarts/core/renderer.py", line 136, in destroy
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/direct/showbase/ShowBase.py", line 601, in destroy
TypeError: 'NoneType' object is not callable
Exception ignored in: <function BulletClient.del at 0x7fc433673c10>
Traceback (most recent call last):
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/pybullet_utils/bullet_client.py", line 43, in del
TypeError: catching classes that do not inherit from BaseException is not allowed

Hi @DOPEMANdd,

The comp-1 branch was used for 2022 NeurIPS Driving Smarts competition and has since been archived, with no further updates expected.

Going forward, please use the latest code version from the master branch. The latest code version sidesteps the sumo.tools.traci.exceptions.FatalTraCIError: Connection closed by SUMO error.

If you are interested in training and benchmarking your agents, please consider using the latest Driving Smarts 2023.1 & 2023.2 and Driving Smarts 2023.3 benchmarks.

@Adaickalavan thanks for your reply,I tried ,but this new error occurred in the master branch, where there is hiway-v1 registered in smarts.env.
Traceback (most recent call last):
File "train.py", line 66, in
env = gym.make('smarts.env:hiway-v1', scenario="sumo/intersections/1_to_2lane_left_turn_c_agents_1",
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 184, in make
return registry.make(id, **kwargs)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 105, in make
spec = self.spec(path)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 161, in spec
raise error.DeprecatedEnv(
gym.error.DeprecatedEnv: Env hiway-v1 not found (valid versions include ['hiway-v0'])

@Adaickalavan I deleted my previous version and the following error appears again.
Traceback (most recent call last):
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 130, in spec
importlib.import_module(mod_name)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 961, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'smarts'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "train.py", line 62, in
env = gym.make('smarts.env:hiway-v1', scenario="sumo/intersections/1_to_2lane_left_turn_c_agents_1",  # 可以场景迭代
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 184, in make
return registry.make(id, **kwargs)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 105, in make
spec = self.spec(path)
File "/data/zhangdi/anaconda3/envs/smart/lib/python3.8/site-packages/gym/envs/registration.py", line 133, in spec
raise error.Error(
gym.error.Error: A module (smarts.env) was specified for the environment but was not found, make sure the package is installed with pip install before calling gym.make()


undefined method `authenticate_user!'

There are some other tickets around this issue, and I have tried to follow the answers there
https://github.com/lynndylanhurley/devise_token_auth_demo/issues/4
https://github.com/lynndylanhurley/devise_token_auth/issues/182
But no luck ... I can't seem to make it work. I'm on Rails 4.2.
class Api::BaseController < ActionController::API
   include DeviseTokenAuth::Concerns::SetUserByToken
   include ActionController::ImplicitRender

class Api::SentencesController < Api::BaseController
   before_filter :authenticate_user!

Rails.application.routes.draw do

   namespace :api, defaults: { format: :json } do
      mount_devise_token_auth_for 'User', at: '/auth'
      resources :users, :only => [:show, :create, :update, :destroy]
      resources :sentences
   end
end

Any assistance, or advice on how to debug would be really appreciated. Great libraries by the way :-)

It is happen because namespace inside routes. You can use authenticate_{namespace}_user! instead using standard authenticate_user!, for your case it should be authenticate_api_user!, for another helper methods you need to add namespace:

current_api_user
api_user_signed_in?

Thanks for the tip. I did actually try the *authenticate_api_user! *but no
luck.
How would I debug this issue?
On Thu, Apr 16, 2015 at 7:50 PM, Agung Prasetyo<EMAIL_ADDRESS>wrote:

It is happen because namespace inside routes. You can use
authenticate_{namespace}_user! instead using standard
authenticate_user!, for your case it should be authenticate_api_user!,
for another helper methods you need to add namespace:

current_api_user
api_user_signed_in?

—
Reply to this email directly or view it on GitHub
https://github.com/lynndylanhurley/devise_token_auth/issues/219#issuecomment-93800327
.

what about to puts binding.pry somewhere inside your controller, check and see ?

@brain64bit it worked for me. thanks

@brain64bit thanks. It worked :)

It seems like this works too, outside the api namespace:
mount_devise_token_auth_for 'User', at: 'api/auth'

I'm sure there are caveats to this approach, just not sure yet, since I just started playing with it.

@stevehanson did this ever work out for you?  I am going to try it as well.
is there any reason that mount_devise_token_auth_for (in routes.rb) doesn't pass along the opts to the devise_for method?  Devise has support for this using the :as option or :singular option, but this isn't passed through in the mount_devise_token_auth_for method.
I suppose there could be some devise options that cause conflicts with devise_token_auth, but it seems viable to pass through the opts and then exclude or fix the options that cause conflicts as they are discovered.

@andoq I ended up going with another approach with my app and don't think I really explored it further. Sorry I can't be of more help.

Hi there @spheregenomics    ,
In an effort to cleanup this project and prioritize a bit, we're marking issues that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this issue. Obviously feel free to re-open it at any time if it's the right time or this was done in error!
If you are still having the issue (especially if it's a bug report) please refer to our new Issue Template to provide some more details to help us solve it.
Hope all is well.


Containment on by default in STP

CSS Containment is now on by default in Safari Technology Preview.

Super happy this is on by default now and to see you here 🎉, but if #6148 were to be merged this change would be part of it 0:-)

Ah, yes, please merge in @Schweinepriester 's PR! And ignore this one. Thanks for your work, Kai.

Thanks Jen! @Schweinepriester 's PR has been merged.


Update build status location

This build has been migrated to travis-ci.com as per their recommendation.

Coverage remained the same at 68.286% when pulling e5b966b5c0c1068671490d7cb17f2deec6712eaa on sghill:travis-ci-com into 1cb3ce7c789b668711e1e4a7d541010dc113a9a5 on Netflix:develop.


[REQ] CI reduce the number of running nodes

Is your feature request related to a problem? Please describe.
CI reduce the number of running nodes

can node3 be removed?

Describe the solution you'd like

Describe alternatives you've considered

Additional context

resolved by 3.0.0 release


Prevent player suicidal move for chess

Thank you for your patience, I haven't had time to check this for a while. As I was reviewing your codes, I saw the problem with the minimax function that you've told me before and I cannot fix it =)))). But I also found another bug, sometimes, the minimax function broke and just let the player plays on both sides. I still haven't found a solution nor the cause of it :')))


Players Trend on graph is broken

Browser name and version: Firefox 92
Device (mobile or desktop): Desktop
Page where the issue happens: https://steamdb.info/app/412220/graphs/

Describe the bug
Players Trend on graph is broken since the switch to hourly player statistics (compared to the old 10 min interval). The line does not correspond to the actual player statistics.

Fixed.
You can login to see the higher density data as before.


Tensorflow cpu memory profiling

System information

TensorFlow version (you are using): 2.3.1
Are you willing to contribute it (Yes/No):

Describe the feature and the current behavior/state.
Tensorflow cpu memory profiling: we want to get detailed profiling as in GPU memory profiling.
(Beginning with a single thread.)
Currently: when running the memory profiler in CPU mode we do not see any memory profile data captured.
Will this change the current api? How?
no
Who will benefit with this feature?
Anyone profiling memory on CPU and trying to understand the memory behavior of operators.
Any Other info.

Are you following this guide https://www.tensorflow.org/guide/profiler?

@bhack  yes

@saar-eliad,
Tensorboard Profiler works with CPU. Please find the Gist of working code.
The code used in the Colab is taken from the documentation of the Profiler and did the changes mentioned below:

Removed the code related to GPU
Executed the Colab with Runtime as None i.e.. CPU

Please find the screenshot below:

@rmothukuru what you are showing from the gist/colab is CPU execution/trace, which works.
However, the problem we are discussing is with the memory profiler tool.

@saar-eliad,
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

@saar-eliad Can you select CPU in the dropdown at https://www.tensorflow.org/guide/profiler#memory_profile_summary ?

@bhack when running with GPU, I am able to select CPU in the dropdown menu (the cpu info is very limited compared to the gpu information).
when running without GPU, i.e., CPU only, there is no menu at all (and even the gunziped json file in tensorboard dir is empty).

Do you see the https://www.tensorflow.org/guide/profiler#memory_profile_summary on CPU?

@bhack  I do not see the memory profile summary on CPU. (it says something like - no profile).

It seems a duplicate of https://github.com/tensorflow/tensorflow/issues/42123. @MarkDaoust can we add a claim in the Docs?

@rmothukuru since I can't share the whole code, here is a copy & paste of the relevant parts.
(we got this from a 3rd party)
Do you have an example of a working CPU memory profiler without GPU? I did not find any.
If so let's take the model from there.
The 3rd party we are working with claims Tensorflow can profile only CPU memory - it did not work according to my check, hence I believe this feature does not exist.
(from looking around the TensorFlow code: seems like the GPU memory profiling is based on CUPTI from NVidia for GPUs, so it is probable that in CPU mode it simply does not run)
    os.environ["CUDA_VISIBLE_DEVICES"] = "-1"

    def tf_profile(self, iterations=10):
        """ Runs Profiled and Traced steps of the pipeline """
        with tf.profiler.experimental.Profile(self.profiling['logdir'],
                                              options=tf.profiler.experimental.ProfilerOptions(
                                                  **self.profiling['options'])):
            self.run_traced_steps(iterations=iterations)

    def run_traced_steps(self, iterations=10):
        """ Run pipeline steps Traced by the TensorFLow's Profiler """
        for step in tqdm(range(iterations)):
            with tf.profiler.experimental.Trace('Step', step_num=step, _r=1):
                outputs = self.run_step()

I think that situation is still the same as 2020 https://github.com/tensorflow/tensorflow/issues/42123#issuecomment-675047711. It is why probably a claim in Doc is the better solution to avoid new tickets on this topic.

@bhack I saw this issue before opening this one, and I checked: even in tf 2.3.1 there is an output of gunzip-ed memory profile in tensorboard directory - it has a lot of information there regarding GPU profiles.
(for GPU: we converted it to xlsx and using it instead of the tensorboard frontend)
I believe that in tf 2.4 the fix was reading and displaying this file (frontend fix).
In CPU mode, however, this file is completely empty.
Will CPU-only profiling in TensorFlow work in any version?
Can you point to the code?
I currently believe that there is no such ability without CUPTI.

If you read that comment was:

This tool reports memory consumption on the device (i.e. GPU or TPU); not on the host (CPU).

I think that also now it rely on CUPTI/GPU profiler and It needs a clarification in the docs.

@bhack Absolutely. This clarifies things beyond all doubt.
(Some of the first comments here gave me false hopes that I might be missing something.)
So it's a valid feature request then.

CPU memory profiling does not work on https://colab.research.google.com/gist/rmothukuru/db189c30d7acd81df1b2b75423315238/tensorboard_profiling_keras.ipynb#scrollTo=dFWOMyaHkUX5
even with TF 2.9

+1. This is a feature request, I believe it'd be really beneficial to users if profiling team can support this.

I think it should be mentioned in the documentation, that profiling while running on only CPU isn't supported. Currently, there is no info, and it is misleading. I wasted some time trying to get the memory profile from tensorflow-serving running on the CPU.


Depth-wise deformable convolutions are slow

Hi
I noticed that the implementation of deformable convolutions is really slow when using depth-wise convolutions (groups = in_channels). See profiling output below and profiling script here.
If there is any chance, the current code could be made faster for these depth-wise convolutions, that would be fantastic.
# Output of deform_conv_speed.py

bs: 01, in-channels: 512, groups: 1
Time normal conv:                0.37 ms
Time torchvision deform conv:    0.37 ms
Time mmdet deform conv:          0.36 ms
----------------------------------------
bs: 01, in-channels: 512, groups: 512
Time normal conv:                0.19 ms
Time torchvision deform conv:   15.23 ms
Time mmdet deform conv:         13.67 ms
----------------------------------------
bs: 16, in-channels: 512, groups: 1
Time normal conv:                1.15 ms
Time torchvision deform conv:    2.26 ms
Time mmdet deform conv:          2.19 ms
----------------------------------------
bs: 16, in-channels: 512, groups: 512
Time normal conv:                0.32 ms
Time torchvision deform conv:   15.06 ms
Time mmdet deform conv:         13.76 ms
----------------------------------------

Motivation
I am using deformable convolutions to shift feature maps in the spatial dimension similar to the approach described in Spatiotemporal Sampling Networks. Using regular convolutions would introduce more parameters than necessary which is why I opted for depth-wise convolutions which reduce the amount of parameters by a factor of in_channels. Unfortunately this makes the network very slow, both in training but even more so during inference.
Related resources
I started a thead in the PyTorch forum to ask for advice concerning this issue. In the mean time I found out that I can at use the weights trained with depth-wise convolutions for inference with normal convolutions if I initialize the kernels as in the code snipped below (see this post for more information).
# model.deform_conv.weight.shape = [512, 512, 3, 3]
# pretrained_kernels.shape = [512, 1, 3, 3]

for i in range(model.deform_conv.weight.shape[0]):
    model.deform_conv.weight[i, i, :, :] = pretrained_kernels[i, 0, :, :]

I was wondering if the same approach could not also be used on cuda level to speed up the grouped convolutions for the special case where groups = in_channels?
Unfortunately I have zero experience with cuda code but I am happy to help if I get some guidance.

Hi @MauroPfister
Thanks for bringing this up. I will take a look into it.

Hi @xvjiarui
Thanks for considering my suggestion. Did you already have time to look at it?
Cheers

Hi @MauroPfister
Sorry for the late reply. Actually, I didn't find an elegant way to fix it for good. Since DCN is implemented on CUDA without cudnn support, depthwise is slow.

Oh that's too bad. I guess I'll have to work with my hack from the first post then.
Thanks anyways for looking into it and please keep us posted if you manage to speed it up.

Hi,
I just noticed, that depthwise deformable convolutions are still slow. Is there any update on speeding them up?
Thanks!


AsciiDoc formatted table cells do not follow theme's grid color

In some recent update in the asciidoctor-pdf main branch, the table grid color around AsciiDoc formatted table cells is no longer consistent with regular table cell grid color of the same table.
For example, the AsciiDoc syntax below produces the following screenshot as output, where the a| cell is surrounded by a darker grid color. This example uses the default asciidoctor-pdf theme, but the problem also appears when a custom grid color is defined.
It would be great to have AsciiDoc formatted table cells use the same grid color as the rest of the table (as was the case several weeks ago).
= Table Example

*Simple Table*

[cols="50%,50%",options="header"]
|===
|Column 1
|Column 2

|Row 1 left
|Row 1 right

|Row 2 left
|Row 2 right

|Row 3 left
|Row 3 right
|===

*Table with AsciiDoc Formatted Cell*

[cols="50%,50%",options="header"]
|===
|Column 1
|Column 2

|Row 1 left
|Row 1 right

|Row 2 left
a|AsciiDoc formatted cell

* List bullet
* List bullet

|Row 3 left
|Row 3 right
|===

It would be great to have AsciiDoc formatted table cells use the same grid color as the rest of the table

That is certainly the intended behavior.
I'm not able to reproduce this problem using either 2.0.0.alpha.1 or the code from the main branch. I have tried it using both the default theme and a custom theme, and I don't get the result you shared in either case.
What code are you running? If you are using code from git, are you sure that you have fetched all updates cleanly? Can you share the relevant sections of the custom theme file you are using? Can you share a project that reproduces the problem in isolation?

Please reopen if you can provide a reproducible example. As far as I can tell, this is not an issue with the code as it stands right now.

Just to close the loop on this one, it seems I was using an older dependency in my Gemfile. I replaced:
gem 'prawn-table', github: 'prawnpdf/prawn-table', branch: 'HEAD'

with:
gem 'prawn', github: 'prawnpdf/prawn', branch: 'HEAD'

and the table output is back to normal. Or maybe it was just resetting the Ruby packages that did the trick. But it's working again.
Thanks!

Thank you for following up and sharing that diagnostic information.
I have even better news for you. You can remove the prawn-table gem declaration entirely. We have moved the patch for it into Asciidoctor PDF so that you no longer need to declare it. And you don't need to declare prawn as a dependency either.
I'll add this point to the release notes.


Prometheus.Client.Abstractions nuget package

Hello,
I can't seem to find a nuget package with the Abstractions of this client. Can you modify your CI in order to be able to build Prometheus.Client.Abstractions nuget packages ?
--Theodore

We have not separated Abstractions into the package, but this definitely makes sense. I'll mark this issue as "enhancement" and we will include it into next release.
For now you have to reference the library itself.
Thank you for your contribution.


Dls 9608 - Questions pages showing hint text rather than paragraph

As discussed in refinement, putting on hold till after discussion with UCD


Typo in ODR_t?

It looks like there is a typo in the enum ODR_t in CO_ODinterface.h.  Of the following
/** SDO abort 0x08000022 - Data can't be transf. (present device state) */
ODR_DATA_DEV_STATE = 23,
/** SDO abort 0x08000023 - Object dictionary not present */
ODR_OD_MISSING = 23,

should ODR_OD_MISSING be 24?

Have missed that, fixed now, thank you.


Lenovo Thinkpad x1 Yoga gen 3

Ouput of linux-enable-ir-emitter -v -d /dev/video1 configure

configure command output
INFO: Ensure to not use the camera during the execution.
INFO: Warning to do not kill the process !
CRITICAL: Cannot access to /dev/v4l/by-path/pci-0000:00:14.0-usb-0:8:1.0-video-index1
ERROR: The configuration has failed.
INFO: Do not hesitate to visit the GitHub ! https://github.com/EmixamPP/linux-enable-ir-emitter/wiki

Output of v4l2-ctl --list-devices
Integrated Camera: Integrated C (usb-0000:00:14.0-8):
        /dev/video0
        /dev/video1
        /dev/media0

Ouput of v4l2-ctl -d /dev/video1 --list-formats-ext

v4l2 device output
ioctl: VIDIOC_ENUM_FMT
        Type: Video Capture

Additional info

Computer (or camera) model: Lenovo Thinkpad x1 Yoga gen 3
Version of linux-enable-ir-emitter: linux-enable-ir-emitter 4.5.0

It seems that your infrared camera is not recognized by the kernel. It is not listed in v4l2-ctl --list-devices, the one you see is the regular one

Closing because we can't do anything until this infrared camera is recognized by the kernel :confused:


[IMPROVEMENT] Hideable "Sticker Button"

It would be a great idea to make the sticker button hideable by the user, since not everyone uses it (and sometimes it might confuse people, as it uses the same icon as the "Gift Nitro" one)

That is indeed a bug that I did not notice.


[js-testing] write more tests for javascripts/SqlLab/components/QueryAutoRefresh.jsx

current: 32%. bring to 70%.

Notice: this issue has been closed because it has been inactive for 311 days. Feel free to comment and request for this issue to be reopened.


Is there a plan to release CogVideoX-2B-I2V

Feature request  / 功能建议
open CogVideoX-2B-I2V
Motivation / 动机
release CogVideoX-2B-I2V for resource limited finetuning
Your contribution / 您的贡献
.

The model weights have been open-sourced and can be found in the readme. They are free to use for academic research.

The model weights have been open-sourced and can be found in the readme. They are free to use for academic research.

It's CogVideoX-5B-I2V, not CogVideoX-2B-I2V ?

We did not create CogVideoX-2B-I2V (we did not train this model); we only trained the 5B version, because the 5B version has better performance

We did not create CogVideoX-2B-I2V (we did not train this model); we only trained the 5B version, because the 5B version has better performance

thanks for your reply.


mutation testing frameworks/tools

I like your list and I got here because I have been looking for a mutation testing framework / tools. unfortunately there is nothing listed right now. I think there is nothing to find right now. but maybe, at any time, we have something that should get listed up here.

What do you mean by that?

What do I mean by mutation testing? Mutation testing can be described as a method, with which you mutate the production code (e.g. "if (a>b)" will become "if (a<b)") and then run your automated test suite. With each mutation at least one test should fail, so that this mutation is killed. Otherwise you may have a lack in your code coverage (or other reasons).
so this is a method to measure the quality of your test suite and goes beyond code coverage metrics.
One concrete tool might be Visual Mutator (https://visualmutator.github.io/web/). But this is not working with current Visual Studio versions.
Working tools/frameworks for other languages are PIT (http://pitest.org/) for Java or Stryker (http://stryker-mutator.io/) for JavaScript.

Oh, I wasn't aware of such concept, my bad. I tried to see how much it's widespread and it looks to me that adoption of it is not widespread as well as there aren't many libraries/frameworks in .Net supporting it. So I'd rather avoid creating a new specific category and I'd suggest to put it to the Testing category where we have frameworks and libraries related to unit/integration/younameit testing.

Yes, this is the best fit. I did not want to suggest a separate category, but, at any time when available, to list such a framework in the testing category. ;)
On Twitter I got into a conversation with one of the maintainers of Stryker, which I mentioned earlier in this issue. They are planning to start to develop a tool for c#. Sounds promising!

Am 19.03.2018 um 17:53 schrieb Vasili Puchko<EMAIL_ADDRESS>Oh, I wasn't aware of such concept, my bad. I tried to see how much it's widespread and it looks to me that adoption of it is not widespread as well as there aren't many libraries/frameworks in .Net supporting it. So I'd rather avoid creating a new specific category and I'd suggest to put it to the Testing category where we have frameworks and libraries related to unit/integration/younameit testing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@skorzinetzki interesting... PR with new category seems like the right next step

The Stryker mutation testing framework has gained inital support for C#. See https://github.com/stryker-mutator/stryker-net.

@philippw yes, I had a first try and it worked in my very simple example. I think it can be listed right now. Or do you wait until it has left it's preview status?

Okay, I can create a pr today. :)

Am 29.11.2018 um 09:26 schrieb Philipp Weissenbacher<EMAIL_ADDRESS>@skorzinetzki Adding it to the list now, has the added benefit of getting more people interested and maybe invested. It'd be great if we can attract more outside contributors.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm closing the issue since all questions have been covered and PR has been merged. For now let's keep mutation testing under the Testing section, later on we can move it to separate if there will be a plenty of similar projects. Please reopen the issue if needed.


chore(templates): fixes bug in e-commerce

Description
When following the documentation to run the E Commerce template locally, you are asked to run yarn stripe:webhooks to work with webhooks.
However, when checking out your cart and a webhook is triggered, your terminal receives the following error:
[ERROR] Failed to POST: Post "http://localhost:8000/stripe/webhooks": dial tcp <IP_ADDRESS>:8000: connect: connection refused

I believe this is because the port is wrong, and it should be port 3000. There is no reference to a port 8000 anywhere in the code base for this template, including in the docker-compose.yml file.
Making this changes allows webhook requests to be forwarded correctly:
--> customer.created [evt_...]
<-- [200] POST http://localhost:3000/stripe/webhooks [evt_...]

This PR makes this small change.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Change to the templates directory (does not affect core functionality)

Checklist:

[x] Existing test suite passes locally with my changes

Thanks for the pr @yosufali


Key-mon plans for  Windows

Hello,

First of all, THANKS a lot for developing key-mon.
I use it on Ubuntu Maverick and I am really satisfied :-)

I am wondering whether or not are there any plans to port key-mon on Windows as
well?

On Windows, Camstudio, which is probably the most widespread open-source screen
recorder, lacks key-mon features.
At present, I have read Camstudio's developers are considering adding the
options to show key combination [1] through  Hotkeys [2].

In my opinion, it would be great to have key-mon working on Windows as well.
This being said, I don't know how difficult would be to develop an ".exe"
self-installer for key-mon :-(

Anyway, thanks indeed for your work for Linux users :-)

Best regards,

Silvio

[1] http://camstudio.org/forum/comments.php?DiscussionID=424&page=2
[2] http://www.autohotkey.com/

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 24 Oct 2010 at 8:06

The code is very much tied to xlib and linux. Would be hard to port to Windows.


Add new updated_by/on fields

Script for Robert Knight to update the schema in td-meta1
Emailed tigerdata distribution list. Waiting on confirmation.
See also https://github.com/pulibrary/tiger-data-app/pull/291

This has been applied to td-meta1 and it's ready to be merged


After pressing enter to create graphics all the input values are cleared from the pane - ArcMap

In Distance and Direction after entering all your parameters and pressing Enter to create your graphic, all the inputs you used are cleared out.  I would like to see those values stay there until you explicitly change them. I realize that the graphics are labeled, however you still immediately lose information about the center point of things like ellipses until your export the graphics and get the info from the attribute table
It looks like this behavior happens on all the tabs

[ ] Line tab
[ ] Circle tab
[ ] Ellipse tab
[ ] Rangs tab

We are not going to do this based on the level of effort and it is the current design.


WebGL x regl different results

Thank you Jeremy for this amazing work!
I found this project a few years ago. Finally, last week, I started my journey to adapt your matrix rain as Vivaldi background start page, as I did before with a simpler matrix rain, and then adapt the shader for Unity so we can have something like the Apple Vision Pro matrix effect.
On the Vivaldi browser adaptation, my first problem was a CORS related policy on Vivaldi that prevented regl from compiling the shaders. So I switched to webGL and noticed a huge difference in the bloom effect on my computer:

Investigating the bloom shader, I noticed a slightly different implementation from the regl shader, but even matching the shader code and decreasing the bloomStrength value, I didn't get the crisp text as in regl variant. So, testing the animation without the bloom effect, I noticed yesterday that webgl animation has a blur in the text:

I will investigate this blur today, but I decided to bring this issue up to see if you know what can be the cause of this issue.
Thanks for sharing this amazing matrix effect! 🫂

Hi! Thank you for your interest, and your kind words. 😃 I'll look into this in more detail, but off the top of my head, I can confirm that the bloom implementation in my regl and webgpu regions of this project are different, produce slightly different results, and therefore may contain different bugs. 😆

🎶 we come from the land of the ice and snow, from the midnight sun, where the code rains flow 🎶
Speaking of subtle whoopsies, it looks like you misspelled bloomStrength in the WebGPU tab of your second screenshot, and so it may still be applying the bloom pass. That doesn't rule out a bug, but it might explain the blur in there.
I'll keep you posted. Thanks for pointing this out, and also for pointing out that what I do for free is selling for fifteen bucks on Apple's headset. 😆💸

Hi! Thank you for your interest, and your kind words. 😃 I'll look into this in more detail, but off the top of my head, I can confirm that the bloom implementation in my regl and webgpu regions of this project are different, produce slightly different results, and therefore may contain different bugs. 😆

I was wondering that. Can you confirm that the webgpu implementation produces different results with the default parameters? I will check that on my wife's Windows laptop.

🎶 we come from the land of the ice and snow, from the midnight sun, where the code rains flow 🎶

Lol! Nice catch! Was on point! :metal:

Speaking of subtle whoopsies, it looks like you misspelled bloomStrength in the WebGPU tab of your second screenshot, and so it may still be applying the bloom pass. That doesn't rule out a bug, but it might explain the blur in there.

Oh, thanks! I misspelled bloomSize. Let me fix the url and reduce the column number:

If we pick a 7 with similar brightness from each side, it will be easier to see that the font on the WebGL implementation is blurry:

I think this is related to a problem that I faced these past 2 days on the Unity port and is related to the regl rounding of symbol brightness on symbol.r = clamp(screenPxDistance + 0.5, 0.0, 1.0). On Unity, and I believe the same problem is happening on WebGL,  this calculation results in a smooth transition from the font shape to the outside, creating the blur effect. I solved that by rounding the symbol.rg. I will try that on  webgl later this week, and I will let you know.

I'll keep you posted. Thanks for pointing this out, and also for pointing out that what I do for free is selling for fifteen bucks on Apple's headset. 😆💸

You're welcome! But you still have time. You can publish for the Meta store focusing on Quest 3. Let me know if you want access to my Unity port.
My goal is to make a PoC Unity shader for the free productivity VR app called Immersed. Hopefully, they can create an environment with the shader, and I will finally be able to work inside the Matrix :sunglasses:
Today, I figured out the main problems on Unity to make it work with raindrop and palette pass. Porting the bloom effect today :rocket:

Can you confirm that the webgpu implementation produces different results with the default parameters?

The parameters passed into them are the same, but the webGPU bloom and the regl bloom were implemented separately.
bloomBlur.wgsl and bloomPass.blur.frag.glsl have the same responsibilities but I don't think their kernels are even the same size.
bloomCombine.wgsl and bloomPass.combine.frag.glsl are also quite different. I might even have different size image pyramids in here.
Either of those large differences would be reason enough for the blur to look different between my two renderers. One other theory, though, is that the WebGPU bloom pass may be setting a different base resolution than the regl pass.
From the perspective of a Unity port, I would suggest that you go with your gut when implementing the bloom. My regl code supports WebGL 1.0, and whatever version of Unity you're working in is way past that, so it might as well play outside the confines of an API designed in, what, 2008? Better bloom awaits us all.
As for the symbol brightness shader code you mention, that's almost verbatim MSDF rendering stuff.
void main() {
	// Bilinear sampling of the distance field
	vec3 s = texture2D(sdf, P).rgb;
	// Acquiring the signed distance
	float d = median(s.r, s.g, s.b) - 0.5;
	// The anti-aliased measure of how "inside" the fragment lies
	float w = clamp(d/fwidth(d) + 0.5, 0.0, 1.0);
	// Combining the two colors
	gl_FragColor = mix(outsideColor, insideColor, w);
}

—that's from Chlumsky's master's thesis. It's a fun one to skim. A-Frame's got it in theirs, too, so if your wife is using her laptop for something important, this particular tree probably isn't worth barking up just yet. 😉

Can you confirm that the webgpu implementation produces different results with the default parameters?

The parameters passed into them are the same, but the webGPU bloom and the regl bloom were implemented separately.
bloomBlur.wgsl and bloomPass.blur.frag.glsl have the same responsibilities but I don't think their kernels are even the same size.
bloomCombine.wgsl and bloomPass.combine.frag.glsl are also quite different. I might even have different size image pyramids in here.

Yeah, I noticed that too. I also adjusted the WebGL code to have a similar behavior to the regl implementation, but I didn't get the desired results :smiling_face_with_tear:

Either of those large differences would be reason enough for the blur to look different between my two renderers. One other theory, though, is that the WebGPU bloom pass may be setting a different base resolution than the regl pass.

If this is the case, reducing the bloomStrength solved the problem for me.

From the perspective of a Unity port, I would suggest that you go with your gut when implementing the bloom. My regl code supports WebGL 1.0, and whatever version of Unity you're working in is way past that, so it might as well play outside the confines of an API designed in, what, 2008? Better bloom awaits us all.

:rocket:

As for the symbol brightness shader code you mention, that's almost verbatim MSDF rendering stuff.
void main() {
	// Bilinear sampling of the distance field
	vec3 s = texture2D(sdf, P).rgb;
	// Acquiring the signed distance
	float d = median(s.r, s.g, s.b) - 0.5;
	// The anti-aliased measure of how "inside" the fragment lies
	float w = clamp(d/fwidth(d) + 0.5, 0.0, 1.0);
	// Combining the two colors
	gl_FragColor = mix(outsideColor, insideColor, w);
}

—that's from Viktor Chlumsky's master's thesis. It's a fun one to skim. A-Frame's got it in theirs, too, so if your wife is using her laptop for something important, this particular tree probably isn't worth barking up just yet. 😉

I was reading his master's thesis yesterday to try to understand my problem with Unity. I'm sure it's related to float precision since I'm not able to control it that well on HLSL, like define a global precision as you did on GLSL. I'm not familiar with Unity, and most of the big problems that I had I solved by changing a Unity option :man_facepalming:
The latest one was the MSDF rendering part generating a square in the border of the texture. I was almost one day trying to discover the problem. In the end Unity was reading the MSDF glyph texture with the wrong format. After changing to RGBA Float and doing the rounding, it fixed the problem for me :smiling_face_with_tear:

Circling back to the problem, I tried the solution that I used on Unity, but it didn't help. It just reduced the antialias, but I still have a blur in the font using webgpu renderer. When I have more time, I will return to find the root of this problem and submit a PR.
I finished the Unity port for the 2D version of the shader and a simple tri-planar mapping strategy for the same texture, but using the same 2D texture strategy for 3D doesn't generate a seamless effect. Now, I'm adapting the code for something more in this line: https://github.com/IRCSS/MatrixVFX/tree/master?tab=readme-ov-file
I'm happy with the results so far. Thank you again for your work and for sharing valuable tips and resources :pray:


Added option to add custom HTTP-Header for scraping

Hi together,
I and apparently a few others would like to have the ability to add our own custom HTTP headers for scraping.
For example, if additional headers are needed when authorizing on the API.
So far I have only read that this was possible via proxies (e.g.: https://github.com/prometheus/prometheus/issues/1724 ).
This is a bit inconvenient, which is why I added the possibility to add the custom headers to the scrape config.
With my customizations, the optional entry to add new HTTP headers could look like this:
global:
  scrape_interval:     15s
  evaluation_interval: 15s

scrape_configs:
  - job_name: 'my_job'
    scrape_interval: 2m
    scrape_timeout: 1m
    scheme: https
    metrics_path: '/foo/metrics'
    proxy_url: 'any-proxy'
    oauth2:
      client_id: 123456
      client_secret: 123456
      scopes:
      - openid
      token_url: https://foobar/as/token.oauth2
    custom_header: #<- this and the next two lines are new
        My_new_custom_Header: 'foobar'
        My_new_custom_Header2: 'foobar'
    static_configs:
    - targets: ['any-api']

This is my first pull request, so please forgive potential mistakes

Thanks for your contribution.
We have added a lot of flexibility lately in the authorization header. We added the ability to change the credentials type (which was hardcoded to Bearer), and we have added oauth2 support.
We should not have custom headers in the scrape configs because that would raise a lot of security concerns. The next logical request would be to be able to relabel them, which would also not be secure.
It could also raise other issues if you set overwrite standard http headers.
Please see https://github.com/prometheus/prometheus/issues/5107 https://github.com/prometheus/prometheus/pull/8512 for the updates we have done to the authorization header.


Cursor can move past last line in file

When moving cursor down past last line in file revi will panic.   Index out of bounds.

This Issue was fix with PR e013293f5fca061e04f9a8340878125bd2ffb807


Smart Mirror always shows that it is offline

The smart mirror always says that it is offline on the Alexa app and on the Alexa website does anyone know how to fix it or is it meant to show as offline all the time?

I have the same problem.
Anyone can helpus?

I have the same problem. Help us

Hi
if you got a black screen you dind't install the files correctly
just delete the file MMM-awesome-elaxa and start over
i followed this discription and now it works fine:
https://www.howtogeek.com/423233/how-to-add-alexa-to-your-smart-mirror/
my problem is that i only can ask 1 question
and if you got too the poin: The terminal will display a custom built link. Copy that and paste it into your browser. Then log in with your Amazon account, if prompted. Click Allow on the next screen.
coppy the whole link and paste it in too you're browser and press enter.
klik allow if youre finish and see the github screen koppie the url in too notepad you'll see:
https://magic-mirror-avs.github.io/Alexa-Web-Helper/authresponse?code=**********************&scope=alexa:all
the * is you're code
good luck!

The smart mirror always says that it is offline on the Alexa app and on the Alexa website does anyone know how to fix it or is it meant to show as offline all the time?


diplaying the fist few rows of datasets

The UnicodeDecodeError usually occurs when there’s a character in the file that doesn’t match the UTF-8 encoding. To fix this, try specifying a different encoding when reading the CSV file. The most commonly used encoding for such issues is ISO-8859-1 (Latin-1) or sometimes cp1252 for Windows systems.
Alternatively, if ISO-8859-1 doesn’t work, try encoding='cp1252'

The UnicodeDecodeError usually occurs when there’s a character in the file that doesn’t match the UTF-8 encoding. To fix this, try specifying a different encoding when reading the CSV file. The most commonly used encoding for such issues is ISO-8859-1 (Latin-1) or sometimes cp1252 for Windows systems.
# Loading the IMDb dataset with specified encoding
imdb_data = pd.read_csv(r"C:\Users\Lenovo\Documents\GitHub\TheUltimate pandas bootcamp\MovieMind-Rating-Prediction\Data\IMDb Movies India.csv", encoding='ISO-8859-1')

# Displaying the first few rows of the dataset
imdb_data.head()

Alternatively, if ISO-8859-1 doesn’t work, try encoding='cp1252'.


Deduplicate found components or include more information

Perhaps related to https://github.com/anchore/syft/issues/32
Basically, after running Syft on an image primarily containing NPM packages I got a pretty large SBOM, and when looking into it, I saw a lot of duplicates, anywhere between the range of 1-10 duplicates for a lot of packages. Some of course were different versions of a particular package, but many times it was the exact same, ex.
(CycloneDX format)
<component type="library">
      <name>1to2</name>
      <version>1.0.0</version>
      <licenses>
        <license>
          <name>MIT</name>
        </license>
      </licenses>
<EMAIL_ADDRESS>    </component>
    <component type="library">
      <name>1to2</name>
      <version>1.0.0</version>
      <licenses>
        <license>
          <name>MIT</name>
        </license>
      </licenses>
<EMAIL_ADDRESS>    </component>

My assumption is that Syft is picking up sub-dependencies or some such, which I'd say is a good thing.
But it would be nice if there could either be a flag that we could set to avoid getting these duplicates, or if the components could contain some additional information to make their inclusion meaningful.

@PapoyEdits thanks for the issue! Do you have an image we can use to reproduce the duplicates? I think there is some identifying information in the core syft data format that we can reflect up into cyclonedx to give more nuance to these findings. Having a concrete example to start from helps so we make sure we're fulfilling your need rather than just adding all of the possible data we can from the start.


Config/ini to enable/disable patches

Sure I can do that.  Should everything be enabled by default still?

-shrugs- entirely up to you.

Done.


Apparently unable to use |>> operator

I am trying to use operator |>> inside a function.
Please see line 25 of:
https://github.com/Muzietto/geiesidioms/blob/master/kep/kep-source/lib/khepri-idioms.kep
Compiler returns: undeclared symbol |>>.
What is the case here?

I managed to use <<|
https://github.com/Muzietto/geiesidioms/blob/master/kep/kep-source/lib/khepri-idioms.kep#L24
But still |>> should be available.

Fixed. Had ||> defined in the builtins instead of |>>


i'm getting "error 65"

when I do "make install", it'll go through everything and then say it failed due to;
> ** BUILD FAILED **
>
>
> The following build commands failed:
> 	CompileSwift normal x86_64
> 	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
> (2 failures)
> make: *** [build] Error 65

Yeah well this hasn't been updated in a while, and as the Swift syntax changes quite fast... This is what we get.

Okay fixed it all, if somewhat late. See #31. It will compile, but if it doesn't work, see #29.

@perfaram Awesome, thanks!

#31 has been merged which fixes compilation for Xcode 8 thanks to @perfaram. With one additional change (https://github.com/beltex/SMCKit/commit/f04ebca9e66f98557ea92463f5738ac19d24d7c2), should compile now on Xcode 9 as well.


Add ambient profile to profile validate test

Please provide a description of this PR:
Adding 'ambient' missing profile value to validate test cases.

Hi @ericdbishop. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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-sigs/prow repository.

/ok-to-test


Move Execution Runtime from Mesh repo

Should we merge fusion-runtime and gateway-runtime?

Not for now. Because gateway-runtime has HTTP stuff and fusion-runtime deals with the execution only.
We can think of refactoring later.


New rule: Disallow expression1?.let { expression2 } ?: run { expression3 } when expression2 is nullable

Expected Behavior of the rule
The rule should disallow expression1?.let { expression2 } ?: run { expression3 } like code as run block can be executed either when expression1 is null or expression2 returns null which is accidental most of the time. Most of new comers replace if else with null check with the above expression and they face this issue very later in the cycle
Some example codes as below
fun main() {
    val a: String? = "Non null string"
    a?.let {
        println("a is not null case")
        nullable()
    } ?: run {
        println("a is null case")
    }
}

fun nullable(): String? {
    return null
}

above code(playground) prints both a is not null case and a is null case and hence is problematic
Compliant code should be as below(playground) if simple null check is intended
fun main() {
    val a: String? = "Non null string"
    if (a != null) {
        println("a is not null case")
        nullable()
    } else {
        println("a is null case")
    }
}

fun nullable(): String? {
    return null
}

Hi @cortinico, @BraisGabin can we pick this in detekt 2.0.0? I find recurring issues due to this type of code snippet. I can give this a try

I understand the idea but I feel it is way too specific. Don't get me wrong. I love the rules that discourage bad patterns with scope functions. But we should find a more general pattern. For example, is the run important here? Or just with an elvis operator is enough?
This also seems to step a bit on unnecessaryLet. One of my favorite rules. But it is really difficult to define clear bounderies in this type of rules.

Hi, @BraisGabin for your query,

 is the run important here? Yes even in cases when run is important(for example multi lines expressions) this issue will prevail. I have just taken one line expression to make the example shorter.

fun main() {
    val a: String? = "Non null string"
    a?.let {
        println("a is not null case")
        nullable()
    } ?: run {
        callSomeOtherFun()
        println("a is null case")
    }
}

fun callSomeOtherFun() = println("callSomeOtherFun")
fun nullable(): String? {
    return null
}

Even in the above example(playground link), all the print statements will be executed
2. Or just with an Elvis operator is enough? issue will reproduce with Elvis operator as well playground link. Here the issue is that most people associate ?.let { <expr1> } ?: <expr2> as null check if else which is not correct
3. This also seems to step a bit on unnecessaryLet. If you are suggesting adding this logic to unnecessaryLet then that will make unnecessaryLet to other things than checking if let is unnecessary or not and overcomplicated that rule

I don't think Brais was trying to be dismissive of your rule suggestion. I also think the way you described it it is too specific.  I think we all agree, that the code in question really should be flagged as hard to understand and error prone. The question remains: What is the pattern here that makes it "bad"?
a?.apply {
    println("a is not null case")
    nullable()
} ?: run {
    callSomeOtherFun()
    println("a is null case")
}

Does this have the same problem even if there is no let?  How about this:
a?.run {
    println("a is not null case")
    nullable()
} ?: callSomeOtherFun()

or this
a?.let {
    println("a is not null case")
    nullable()
}?.run {
    callSomeOtherFun()
    println("a is null case")
}

It is similar but also quite different. According to your initial post the one with apply would not be a problem as the run  statement does not depend on the outcome of nullable() (even though I would still consider this code hard to read).  But the last 2 examples would in fact be a violation.  Can you think of a clear set of rules to apply?

Yes, that's exactly what I was asking for. I like the idea of the rule, but I would like to know exactly what are we looking for so we can formulate the rule correctly.
Maybe this rule is not about let. It is about any function that can return a null. Example:
foo?.maybeNullable() ?: doSomething()

No let involved but the same issue is present, I think. Should we look for this case too?
 I say "usually" because there are some trivial cases where ReplaceSafeCallChainWithRun is a bit annoying. And probably the rule that we are talking about will have the same "problem". It could help a lot to reduce complexity but in some trivial cases it could be "annoying".

Hi @BraisGabin, @marschwar thanks for your inputs. Regarding What is the pattern here that makes it "bad"? I think the code snippets shared by you should also qualify for the rule. I think I listed down the most common pattern which bit me multiple times.
@BraisGabin regarding foo?.maybeNullable() ?: doSomething() I think this would be a little bit restrictive considering examples like foo?.maybeNullable() ?: doSomethingIfAnyChainIsNull(). While scoping operators should be fine to be caught while looking through the code we generally focus on scoping operators to understand code flow(ignoring the last statement of each scoping block). Or should we make this configurable?

Then, what is the  difference between
foo?.maybeNullable() ?: doSomething()

and
foo?.let { maybeNullable() } ?: doSomething
// or
foo?.run { maybeNullable() } ?: doSomething

Are we only going to raise when it is a scoped function? Are we going to raise when the return type of the function is a generic? I mean: let returns T while maybeNullable() on the first example could return String?. One is a generic and the other is a known value.
These type of questions are the ones that I want to solve before this is implemented.

My opinion: I wouldn't make this rule only for scope functions. The underlying problem happens on both cases. But I also agree that maybe that's a bit too restrictive. For that reason I will add something to make it configurable. But when it is "clearly wrong" and when it is "maybe wrong but acceptable". And how will we call that configuration so the users understand when the issue will be raised?

Hi, @BraisGabin sorry for the late reply. To answer your question of Then, what is the difference between, it just that generally, people consider foo?.let { maybeNullable() } ?: doSomething as an alternative to if else on condition that foo is null or not null.
This consideration is not very common across other scope operators. With this rule, I wanted to tackle that issue.

Now I understand your request! But I think that we should flag that even if we change maybeNullable() for alwayNoNullable(). You shouldn't use let as an if/else.
let is a PITA. It's so difficult to draw a line when a usage of let have sense and make the code better and when let just obfuscates the code.
UnnecessaryLet needs a reimplementation in my honest opinion.  But I always end up in the same spot: where should I draw the line?
So, I would vote to close this issue and open an issue to talk about UnnecessaryLet. Create a list of valid and unvalid usages of let. And an explanation for each of them. And then we reimplement UnnecessaryLet.

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

I kind of agree with you @BraisGabin. But TBH let really feels very convenient tool to use as if not null/else. And it works well and makes the code readable except in the above case that I mentioned. Are you suggesting making if not null/else as invalid usage for let? I think that would make all users turn off the UnnecessaryLet rule. I think even Kotlin doc suggests(source) using let for non-nullable form of an object

.

It depends. Not all the null checks are the same. There are some that have sense. But as I said, draw the line is really difficult.

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

This issue was closed because it has been stalled for 7 days with no activity.

Not stale, as PR is open for review


Fix gateway get request without params

Failing to get gateway details when submitting without parameters (--params)

fix(gateway): issue with params


New version of cimgui is available

Hi there, I was curious if we could get this merged with the upstream repo? The following resolves an issue that is currently causing panic when building font ranges, an issue that I have also been dealing with.
https://github.com/AllenDang/cimgui-go/pull/241
thanks for your consideration!

Just wanted to post an update that the latest commit on cimgui-go now includes access to ImNodes. https://github.com/AllenDang/cimgui-go/pull/244 and would be super awesome to have access to! :)

Hello! I think I'll have time to look into it on this weekend.

Hey there, before you start looking at this, let's put this on hold, or close it for now. I've actually moved onto putting together a new cimgui-go backend similar to https://github.com/AllenDang/cimgui-go/blob/main/glfw_backend.go which enables ebiten and controlling how the ui/game are composed together with additional features. If you don't mind, I'd like to use this repository as reference for dumping Imgui's draw commands, would that be okay with you?


An ability to customize binaryFolder

In
https://github.com/bamlab/flashlight/blob/8aee5ec25e13511bb9c20f1d43cd3ff94b02e407/packages/platforms/android/src/commands/platforms/UnixProfiler.ts#L29-L31
we kind of predefine the path for cpp binaries. It works really well when we have an access to node_modules and run locally, in this case global.Flipper === false and we add ../../../ prefix.
But we have a custom pipeline/e2e test runner, so when we run the e2e tests on AWS we pack everything into a single .zip archive and then execute our tests based on the content of this archive.
The problem here is that we still add ../../../ prefix and when we unpack the content we search for binaries in a root folder of a runner (and of course we can not copy files there because of security reasons).
To overcome this problem I fixed it in following way:

copy binaries into zip folder under bin folder name;
modified condition for lookup to be (global.Flipper || process.env.AWS_RUN)

Just curious if I can submit this fix to the repository (maybe use a different name like prcoess.env.USE_BIN_FOLDER)?

Hi @kirillzyusko, I'm not sure I understand the issue correctly, how are you packaging the .zip and then installing on AWS?
Technically the CI on this repo is running on AWS https://github.com/bamlab/flashlight/blob/main/package.json#L30 with our undocumented AWS Device Farm plugin https://github.com/bamlab/flashlight/blob/main/packages/plugins/aws-device-farm/src/bin.ts#L11 and we don't seem to have an issue there

how are you packaging the .zip and then installing on AWS?

This is the script for installation + run:
      - name: Configure AWS Credentials
        uses: aws-actions/configure-aws-credentials@v4
        with:
          aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
          aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
          aws-region: us-west-2

      - name: Schedule AWS Device Farm test run on main branch
        uses: realm/aws-devicefarm/test-application@7b9a91236c456c97e28d384c9e476035d5ea686b
        id: schedule-awsdf-main
        with:
          name: App E2E Performance Regression Tests
          project_arn: ${{ secrets.AWS_PROJECT_ARN }}
          device_pool_arn: ${{ secrets.AWS_DEVICE_POOL_ARN }}
          app_file: zip/app-e2eRelease.apk
          app_type: ANDROID_APP
          test_type: APPIUM_NODE
          test_package_file: App.zip
          test_package_type: APPIUM_NODE_TEST_PACKAGE
          test_spec_file: tests/e2e/TestSpec.yml
          test_spec_type: APPIUM_NODE_TEST_SPEC
          remote_src: false
          file_artifacts: |
            Customer Artifacts.zip
            Test spec output.txt
          log_artifacts: debug.log
          cleanup: true
          timeout: 5400

Into zip we are packing apks (two apk to check performance regressions between them) + node script (for tests execution) into zip archive:

app-e2eRelease.apk
app-e2edeltaRelease.apk
testRunner.ts <- our custom test runner (i. e. it's not Detox, not Appium).

Then we upload it and execute (tests/e2e/TestSpec.yml) our test as:
node testRunner.ts -- --mainAppPath app-e2eRelease.apk --deltaAppPath app-e2edeltaRelease.apk
The testRunner.ts is a compiled TS file.
I hope these code snippets will give you a better understanding of what we are doing 😅

But you are also packing the node_modules (which should contain the @perf-profiler/android package for instance) inside App.zip right? Shouldn't it also include the binaries? 🤔

@Almouro this it the problem that we are not packing node_modules 🙈 We are using compiler and in the output we have a big single .ts file that contains all the necessary code (including code from node_modules) and in this case it doesn't include binaries (we have to copy it themselves).

ah I see!
We're using npm-bundle to package everything https://github.com/bamlab/flashlight/blob/main/packages/plugins/aws-device-farm/src/zipTestFolder.ts#L7 but I'm guessing a bundler would improve the general execution speed in the AWS machine?
What bundler are you using?
I think in this case it would make more sense to use a process.env.FLASHLIGHT_BINARY_PATH variable to make it more flexible and fallback on default values, what do you think?
Feel free to submit a PR! You can also get rid of the flipper condition since we don't maintain the flipper package anymore

What bundler are you using?

We are using ncc. Not sure why we've chosen such way, maybe for faster uploading to AWS, because packing all node_modules seems to be extremely expensive operation.

I think in this case it would make more sense to use a process.env.FLASHLIGHT_BINARY_PATH variable to make it more flexible and fallback on default values, what do you think?

Yep, I agree 👍

Feel free to submit a PR! You can also get rid of the flipper condition since we don't maintain the flipper package anymore

Done! Opened a PR here: https://github.com/bamlab/flashlight/pull/300


Port this feature to Recode?

Hey, this looks like a neat feature, is it possible to port this to https://github.com/hrzndhrn/recode?

I had some experiments with sourceror (you can see them in livebooks/). For now, I don't plan on doing so, this package is highly experimental and tailored for my use-cases. From what I'm seeing, there's already sorter module there.


Not able to read all the metrics from S3

I am trying to read all the metrics from Thanos S3 for 1 min via thanos remote-read(its connected to thanos store)
Thanos store downloads the whole block has it in memory, but the query fails with the below message. Can you please check.
ts=2021-03-02T12:35:36.80453472Z caller=main.go:278 level=info traceID=00000000000000000000000000000000 msg="thanos request" request="min_time:1598898600000 max_time:1598898659999 matchers:<type:RE name:\"__name__\" value:\".*\" > aggregates:RAW "

level=debug ts=2021-03-02T14:32:54.694630715Z caller=bucket.go:993 msg="stats query processed" stats="&{blocksQueried:1 postingsTouched:1 postingsTouchedSizeSum:8348668 postingsToFetch:0 postingsFetched:1 postingsFetchedSizeSum:8348668 postingsFetchCount:1 postingsFetchDurationSum:173430545 cachedPostingsCompressions:1 cachedPostingsCompressionErrors:0 cachedPostingsOriginalSizeSum:8348668 cachedPostingsCompressedSizeSum:948850 cachedPostingsCompressionTimeSum:39563631 cachedPostingsDecompressions:0 cachedPostingsDecompressionErrors:0 cachedPostingsDecompressionTimeSum:0 seriesTouched:2087166 seriesTouchedSizeSum:7759035328 seriesFetched:2087166 seriesFetchedSizeSum:7787482272 seriesFetchCount:1 seriesFetchDurationSum:77909345526 chunksTouched:1995842 chunksTouchedSizeSum:291363676 chunksFetched:1995842 chunksFetchedSizeSum:145903464063 chunksFetchCount:55948 chunksFetchDurationSum:866512903290886 getAllDuration:243967687800 mergedSeriesCount:979396 mergedChunksCount:983014 mergeDuration:0}"
err="rpc error: code = Unknown desc = send series response: rpc error: code = Canceled desc = context canceled"

err="rpc error: code = Unknown desc = send series response: rpc error: code = Canceled desc = context canceled"

This message is when client cancels the call. So either Querier, Querier client (remote read proxy?) or remote read client aborted the call. Usually it's when you just Ctrl+c client curl OR refresh browser OR client timeout passes 🤗
Hope that helps. Essentially server did nothing wrong - someone canceled this request.


Fix layout issues with LF Projects header and broken favicon

Signed-off-by: John Mertic<EMAIL_ADDRESS>

@mandy-chessell Try this PR instead ;-)

The build is failing because the RAT check is failing - this means there are new files without propertly licenses in them (or not excluded)


[question] TensorFlow Lite

Hi, @hollance san,
Have you ever evaluate TensorFlow Lite on iOS/iPadOS for GPU(I mean not for Neural Engine)?
As you had pointed out at some article, Core ML is slower than MPSCNN.
So I expected Metal delegate of TensorFlow Lite and tried but am disappointed it’s performance.
If you have some insights, could you tell me some?
Thanks.

I haven't really used TFLite very much, so I couldn't say. I also think Core ML is the go-to choice these days, I would only try MPSCNN if there was a really good reason for it.

Thank you for your comment!
I know what you meant in some article, but also it is true even today.
Core ML on GPU is twice or more slower than OpenCL implementation on macOS in another case I tried.
I want to get best performance on both ANE and GPU and expected it from TensorFlow Lite but…
Anyway thank you!
Please take care of yourself in COVID-19 period.

Note that TF Lite's ANE support actually runs the model on Core ML. For Metal they use their own GPU kernels, plus maybe some MPSCNN (it's been a while since I checked).

Sure, so I had expected GPU boost by TF Lite since Core ML on GPU is slow while it keeps speed in the case which ANE is available.
Thank you for your comment.


jnp.where: provide fill_value for single-argument form

Extends #6910 & #7592. Padding with 0 can collide with actual zero indices and cause problems. We could specify the pad value, i.e. as -1.

Thanks for working on this!


Db objects now support update

@jlongstaf
Issues:
Fixes #251
Problem: We had not previously implemented update for Db objects, but they are
supported.
Analysis: Implemented and tested.
Tests: sys/test_db::TestDb::test_update

Ship it!


flutter build ios-framework ignore plugin vendored_frameworks

Is there an existing issue for this?

[X] I have searched the existing issues
The same issue is closed due to customer no response. https://github.com/flutter/flutter/issues/72909
[X] I have read the guide to filing a bug

Steps to reproduce
download attach demo,  run  flutter build ios-framework --no-debug --no-release .
in fair plugin, it uses FairDynamicFlutter.framework.
fair.podspec as following:
#
# To learn more about a Podspec see http://guides.cocoapods.org/syntax/podspec.html.
# Run `pod lib lint fair.podspec' to validate before publishing.
#
Pod::Spec.new do |s|
  s.name             = 'fair'
  s.version          = '0.0.1'
  s.summary          = 'A new Flutter plugin.'
  s.description      = <<-DESC
A new Flutter plugin.
                       DESC
  s.homepage         = 'http://example.com'
  s.license          = { :file => '../LICENSE' }
  s.author           = { 'Your Company' =><EMAIL_ADDRESS>}
  s.source           = { :path => '.' }
  s.source_files = 'Classes/**/*'
  s.public_header_files = 'Classes/**/*.h'
  s.dependency 'Flutter'
  s.platform = :ios, '8.0'

  # s.preserve_paths = 'FairDynamicFlutter/Products/FairDynamicFlutter.framework'
  s.vendored_frameworks = 'FairDynamicFlutter/Products/FairDynamicFlutter.framework'
  s.xcconfig = {
    'OTHER_LDFLAGS' => '-ObjC'
  }
  s.pod_target_xcconfig = { 'VALID_ARCHS' => 'x86_64 armv7 arm64 arm64e armv7s' }
  s.requires_arc = true
  s.static_framework      = true

end

Expected results
FairDynamicFlutter.framework is in build/release folder.
Actual results
FairDynamicFlutter.framework is not in build/release folder.
Code sample
flutter_module.zip
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.7.12, on macOS 12.6 21G115 darwin-x64, locale zh-Hans-CN)
    • Flutter version 3.7.12 on channel stable at /Users/zmtzawqlp/Documents/flutter/3.7.12
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (8 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/zmtzawqlp/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • ANDROID_HOME = /Users/zmtzawqlp/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.2

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

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

I can reproduce the issue using the code sample and command provided above.

tree output of directory after running flutter build ios-framework --no-debug --no-profile
build/ios/framework/
└── Release
    ├── App.xcframework
    │   ├── Info.plist
    │   ├── ios-arm64
    │   └── ios-arm64_x86_64-simulator
    ├── Flutter.xcframework
    │   ├── Info.plist
    │   ├── ios-arm64
    │   └── ios-arm64_x86_64-simulator
    ├── FlutterPluginRegistrant.xcframework
    │   ├── Info.plist
    │   ├── ios-arm64
    │   └── ios-arm64_x86_64-simulator
    ├── fair.xcframework
    │   ├── Info.plist
    │   ├── ios-arm64
    │   └── ios-arm64_x86_64-simulator
    └── url_launcher_ios.xcframework
        ├── Info.plist
        ├── ios-arm64
        └── ios-arm64_x86_64-simulator

16 directories, 5 files

flutter doctor -v
[✓] Flutter (Channel stable, 3.7.12, on macOS 13.3.1 22E261 darwin-arm64, locale en-GB)
    • Flutter version 3.7.12 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (8 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.9862592/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode-14.3.0.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2022.1)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9619390/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.9862592/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at /Users/nexus/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 73.0.4
    • Dart plugin version 231.8109.91

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.8109.175/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (3 available)
    • Pixel 7 (mobile) • 28291FDH2001SA • android-arm64  • Android 13 (API 33)
    • macOS (desktop)  • macos          • darwin-arm64   • macOS 13.3.1 22E261 darwin-arm64
    • Chrome (web)     • chrome         • web-javascript • Google Chrome 112.0.5615.137

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[!] Flutter (Channel master, 3.10.0-12.0.pre.29, on macOS 13.3.1 22E261 darwin-arm64, locale en-GB)
    • Flutter version 3.10.0-12.0.pre.29 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 60077ff979 (3 days ago), 2023-04-23 07:09:16 -0400
    • Engine revision fcf46af2d3
    • Dart version 3.1.0 (build 3.1.0-35.0.dev)
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.9862592/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
    • Xcode at /Applications/Xcode-14.3.0.app/Contents/Developer
    • Build 14E222b
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2022.1)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/221.6008.13.2211.9619390/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.9862592/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at /Users/nexus/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 73.0.4
    • Dart plugin version 231.8109.91

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at /Users/nexus/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.8109.175/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (4 available)
    • Pixel 7 (mobile)     • 28291FDH2001SA            • android-arm64  • Android 13 (API 33)
    • Dean’s iPad (mobile) • 00008103-000825C811E3401E • ios            • iOS 16.4.1 20E252
    • macOS (desktop)      • macos                     • darwin-arm64   • macOS 13.3.1 22E261 darwin-arm64
    • Chrome (web)         • chrome                    • web-javascript • Google Chrome 112.0.5615.137

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.


Immutable Metadata for Routing

In https://github.com/microsoft/reverse-proxy/pull/1639 Matchers property has been changed from IReadonlyList to Array.
https://github.com/microsoft/reverse-proxy/blob/e684ba52dded6598260fa17cec8c1769088597dd/src/ReverseProxy/Routing/IHeaderMetadata.cs#L16
https://github.com/microsoft/reverse-proxy/blob/e684ba52dded6598260fa17cec8c1769088597dd/src/ReverseProxy/Routing/IQueryParameterMetadata.cs#L16
I'm just curious about the reason of this change. Isn't it better for routing metadata to be immutable?
If it's going to remain array, the read-only part should be removed from these API docs.
https://github.com/microsoft/reverse-proxy/blob/e684ba52dded6598260fa17cec8c1769088597dd/src/ReverseProxy/Routing/HeaderMatcher.cs#L53-L56
https://github.com/microsoft/reverse-proxy/blob/e684ba52dded6598260fa17cec8c1769088597dd/src/ReverseProxy/Routing/QueryParameterMatcher.cs#L51-L54

All of these types are internal so it would only be modifiable by code within this library. Do we actually modify it anywhere?
I assume we're using an array for efficiency?

I assume we're using an array for efficiency?

Yes. With an array, you avoid bounds checks and virtual calls for every element when enumerating.
And no, we don't modify these anywhere.

Oh my bad, I missed the internal part! I was thinking about eventually these might move to ASP.NET Core repo, and there all the new metadata is immutable as far as I see. The issue could be closed.


cixtrix workspace app protection not installed

Describe the bug
The app protection feature is not installed.
Steps To Reproduce

# configuration.nix
environment.systemPackages = with pkgs; [ cixtrix_workspace ];
nixos-rebuild switch # as root, if that makes a difference

Expected behavior
app protection feature is enabled and where citrix session enforces the use of app protection the session is successful.
Screenshots

Additional context
During nixos-rebuild switch, here is a snippet of the output.
\nSelect a setup option:\n\n 1. Install Citrix Workspace app for Linux <IP_ADDRESS>\n 2. Remove Citrix Workspace app for Linux <IP_ADDRESS>\n 3. Quit Citrix Workspace app for Linux <IP_ADDRESS> setup\n
Enter option number 1-3 [1]: \nPlease enter the directory in which Citrix Workspace app for Linux is to be installed.\n[default -i /nix/store/w2qp0y2kmch3sqkfi812v7v96jw14p1q-citrix-workspace-<IP_ADDRESS>/opt/citrix-icaclient] \nor type "quit" to abandon the installation: \nThe parent directory -i /nix/store/w2qp0y2kmch3sqkfi812v7v96jw14p1q-citrix-workspace-<IP_ADDRESS>/opt does not exist.\nDo you want to create it? [default y]: \nYou have chosen to install Citrix Workspace app for Linux <IP_ADDRESS> in -i /nix/store/w2qp0y2kmch3sqkfi812v7v96jw14p1q-citrix-workspace-<IP_ADDRESS>/opt/citrix-icaclient.
\nThis is not the default location. You must set environment variable\nICAROOT to this location after installation has completed, for example:\n\nexport ICAROOT=-i "/nix/store/w2qp0y2kmch3sqkfi812v7v96jw14p1q-citrix-workspace-<IP_ADDRESS>/opt/citrix-icaclient"
\nProceed with installation? [default nWARNING: One of the prerequisites required for installation is not present- "libwebkit2gtk-4.0.so.37" not found. Citrix Workspace app (Self Service) may not work.\nDo you want to proceed with the installation..[default n]:
Installation proceeding...
\nChecking available disk space ...
\n\tDisk space available 15665964 K \n\tDisk space required 422748 K
\n\nContinuing ...
Creating directory -i /nix/store/w2qp0y2kmch3sqkfi812v7v96jw14p1q-citrix-workspace-<IP_ADDRESS>/opt/citrix-icaclient
Core package...
Setting file permissions...
Failed to connect to bus: No medium found
Created symlink /build/tmp.4cdwlXX3by/.config/systemd/user/default.target.wants/ctxcwalogd.service → /build/tmp.4cdwlXX3by/.config/systemd/user/ctxcwalogd.service.
Unit /build/tmp.4cdwlXX3by/.config/systemd/user/ctxcwalogd.service is added as a dependency to a non-existent unit default.target.
Failed to connect to bus: No medium found
Integrating with browsers...
\nIntegration complete.
No GStreamer directories were found, skipping GStreamer integration.
linuxx64/hinst: line 1447: killall: command not found
You must have root privileges to install the app protection component.
USB support cannot be installed by non-root users. Run the installer as root to access this install option.
\nSelect a setup option:\n\n 1. Install Citrix Workspace app for Linux <IP_ADDRESS>\n 2. Remove Citrix Workspace app for Linux <IP_ADDRESS>\n 3. Quit Citrix Workspace app for Linux <IP_ADDRESS> setup\n
Enter option number 1-3 [3]: Quitting Citrix Workspace app for Linux <IP_ADDRESS> setup.

Notify maintainers
@pmenke-de
@michaeladler
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.97, NixOS, 22.11 (Raccoon), 22.11.2999.a7cc81913bb`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@michaeladler @pmenke-de @affieuk Any status on this issue?


src/stan/version.hpp should have numbers available to the preprocessor

Currently the version is stored in const std::string. These aren't available at preprocessor time. This means that it is impossible to write conditional code that compiles against more than 1 stan version with minor header incompatibilities.

[ ] add it to the language parser to check for reserved words
[ ] add to the manual's list of reserved words.
[ ] define STAN_MAJOR
[ ] define STAN_MINOR
[ ] define STAN_PATCH

Rationale: Currently there is a potential CRAN race condition. Suppose StanHeaders is upgraded first. Then OpenMx will fail to install until it is updated. Same problem is OpenMx is upgraded first. So both packages would need simultaneous upgrades on CRAN to avoid the situation where OpenMx was uninstallable for a short time.

@jpritikin, can you take it a step further and describe what macro should be defined? should we split it into 3 version numbers (major, minor patch) or a single version? or not?

this page lists a lot of existing macros:
http://nadeausoftware.com/articles/2012/10/c_c_tip_how_detect_compiler_name_and_version_using_compiler_predefined_macros

should we split it into 3 version numbers (major, minor patch) or a single version?

Yes, given that the preprocessor can only make simple comparisons. You might want to follow something like, gcc with "__GNUC__ and __GNUG__ contain the major version number, and __GNUC_MINOR__ and __GNUC_PATCHLEVEL__ the minor version and patch level numbers (e.g. "4", "6", and "1" for version 4.6.1). The __GNUC_PATCHLEVEL__ macro was introduced in version 3.0."

[re-subjected to grab Ben's attention]
I seem to recall Ben telling me that once we're on CRAN,
our hands are going to be tied in not letting us update versions
that break other packages.
Ideally, we'd not use "__" because it's reserved for system
variables in C++, but we might want to anyway because we do elsewhere
without problems anywhere and it's already built-in as a reserved
symbol in the language

Bob

On Mar 16, 2015, at 1:42 AM, Joshua Pritikin<EMAIL_ADDRESS>wrote:
should we split it into 3 version numbers (major, minor patch) or a single version?
Yes, given that the preprocessor can only make simple comparisons. You might want to follow something like, gcc with "GNUC and GNUG contain the major version number, and GNUC_MINOR and GNUC_PATCHLEVEL the minor version and patch level numbers (e.g. "4", "6", and "1" for version 4.6.1). The GNUC_PATCHLEVEL macro was introduced in version 3.0."
—
Reply to this email directly or view it on GitHub.

Ideally, we'd not use "__" because it's reserved for system variables in C++

Yeah, I think it would be fine without the _ prefix and postfix. Just STAN_VERSION, etc would be fine.

Without the "__" sufix, we'll also need to

[ ] add it to the language parser to check for reserved words, and
[ ] add to the manual's list of reserved words.

Bob

On Mar 16, 2015, at 11:00 AM, Joshua Pritikin<EMAIL_ADDRESS>wrote:
Ideally, we'd not use "__" because it's reserved for system variables in C++
Yeah, I think it would be fine without the _ prefix and postfix. Just STAN_VERSION, etc would be fine.
—
Reply to this email directly or view it on GitHub.

I see this still isn't addressed in stan 2.7. That is disappointing.

@jpritikin, I added a todo list to the top comment. Can you check and see if that's sufficient?

Also just a heads up that Daniel is going to be pulling the math library
into its own Git repo for the 2.7 release and we hope to keep the paths,
namespaces, etc., stable after this.  Sorry for all the disruption en route.
It's a general rule that it takes 3 tries to get a library right, so hopefully
this one will stick for awhile.

Bob

On Jun 8, 2015, at 4:16 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
@jpritikin, I added a todo list to the top comment. Can you check and see if that's sufficient?
—
Reply to this email directly or view it on GitHub.

@syclik Looks good.

Daniel is going to be pulling the math library into its own Git repo for the 2.7 release

Sounds good. Will that be reflected in the R packaging too? Will you split StanHeaders into StanMathHeaders and StanHeaders?

@jpritikin, I hope not. Our plan is for Stan to package up everything it needs, which includes the math library.

It'd be nice to split out math.  I could imagine someone
writing a wrapper around Stan Math that let you define a C++
function and then wrapped it to provide evaluation and gradients
(or higher order) back to R.

Bob

On Jun 9, 2015, at 12:10 AM, Joshua Pritikin<EMAIL_ADDRESS>wrote:
@syclik Looks good.
Daniel is going to be pulling the math library into its own Git repo for the 2.7 release
Sounds good. Will that be reflected in the R packaging too? Will you split StanHeaders into StanMathHeaders and StanHeaders?
—
Reply to this email directly or view it on GitHub.

@bob-carpenter Yeah, that's what I was thinking. But it's not a big deal either way.

On Mon, Jun 08, 2015 at 09:44:44PM -0700, Bob Carpenter wrote:

It'd be nice to split out math. I could imagine someone
writing a wrapper around Stan Math that let you define a C++
function and then wrapped it to provide evaluation and gradients
(or higher order) back to R.

Yeah, that would be very cool :-)