Flake Review

This looks really great. I'm inclined to say we're more or less set here (unless there is anything else you want to add @MatthewCroughan ). I'm able to build and run this, and the run the x86-64 checks. LGTM.
Two requests:

Could you run this through a formatter? Indentation is a bit wonky near the end and there's a couple of trailing spaces and a few tabs. (you probably already fixed this)
Could you add a bit of documentation? Mostly interested in the fancy checks and use of coercedTo

Did another pass. Everything is super clean. Can you resolve the with final; ... issue for the package first?

I'm going to upstream this with the current code, because I think the usage of with final is justifiable in the overlay. It is a large overlay in which I never intend to call prev. If this is an issue, it can be fixed later by a PR to Kiwiirc, which is better socially anyway since then we are working with the upstream project.
I'm not sure how to use inherit in the way suggested.


Middleware: Fixing Server Group and Server relationships delay when saving into inventory

fixes issue https://github.com/ManageIQ/manageiq/issues/10900
cc @abonas
@miq-bot add_labels providers/hawkular, bug, darga/no

@miq-bot assign @josejulio
@josejulio please review

LGTM.
Also tested and works well.

@miq-bot assign @chessbyte

fixes https://github.com/ManageIQ/manageiq/issues/10900


Models can't have multiple models with the same type

We can't do (as of v2.0.3):

@Table class Foo {
  @PrimaryKey long id;
}

@Table class Bar {
  @Column public Foo foo1;
  @Column public Foo foo2;
}

This is difficult because fully-qualified names (e.g. Foo.id) are no longer reliable. Needs the table-alias allocator.

Hi, really well designed library can I say. I have another manifestation of this issue:
@Table class SharedFoo {
@PrimaryKey long id;
}
@Table class Foo1 {
@PrimaryKey long id;
@Column SharedFoo shared;
}
@Table class Foo2 {
@PrimaryKey long id;
@Column SharedFoo shared;
}
@Table class Bar {
@Column public Foo1 foo1;
@Column public Foo2 foo2;
}

Thanks to a good example. Will add it to tests.
I'm planning tol resolve this issue in a few weeks.

When will the fix be available?

Not sure, but this issue is the most important for the next major release.

Any estimated release date?

This is working in progress.

I have released v3.0.0-rc2, which fixes this issue.
Please check v3.0.0-rc2 works for your project. Thanks.

I get build errors because *_Schema constructors are not public, and cannot be accessed by other schemas.
Error:: MyModel1_Schema(Aliases.ColumnPath) is not public in MyModel2_Schema; cannot be accessed from outside package

That happens with models that are not in the same package.


Exception :  Sizes of tensors must match except in dimension 1

Hi ,
@andimarafioti
when i run the  "python s2s_pipeline.py" with mentioned arguments in teh readme file ,  When i started to speak I am getting the below exception
(/home/user/Downloads/speech-to-speech/sts) ~/PycharmProjects/speech-to-speech git:[multi-language]
python s2s_pipeline.py 	--recv_host <IP_ADDRESS> 	--send_host <IP_ADDRESS> 	--lm_model_name microsoft/Phi-3-mini-4k-instruct 	--init_chat_role system 	--stt_compile_mode reduce-overhead 	--tts_compile_mode default
/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/df/io.py:9: UserWarning: torchaudio.backend.common.AudioMetaData has been moved to torchaudio.AudioMetaData. Please update the import path.
from torchaudio.backend.common import AudioMetaData
[nltk_data] Downloading package averaged_perceptron_tagger_eng to
[nltk_data]     /home/user/nltk_data...
[nltk_data]   Package averaged_perceptron_tagger_eng is already up-to-
[nltk_data]       date!
Using cache found in /home/user/.cache/torch/hub/snakers4_silero-vad_master
2024-09-05 14:53:45,040 - STT.whisper_stt_handler - INFO - Warming up WhisperSTTHandler
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50360]] which creates a conflict. forced_decoder_ids will be ignored in favor of task=transcribe.
The attention mask is not set and cannot be inferred from input because pad token is same as eos token.As a consequence, you may observe unexpected behavior. Please pass your input's attention_mask to obtain reliable results.
2024-09-05 14:54:06,376 - STT.whisper_stt_handler - INFO - WhisperSTTHandler:  warmed up! time: 21.336 s
2024-09-05 14:54:07,457 - transformers_modules.microsoft.Phi-3-mini-4k-instruct.5a516f86087853f9d560c95eb9209c1d4ed9ff69.modeling_phi3 - WARNING - flash-attention package not found, consider installing for better performance: No module named 'flash_attn'.
2024-09-05 14:54:07,457 - transformers_modules.microsoft.Phi-3-mini-4k-instruct.5a516f86087853f9d560c95eb9209c1d4ed9ff69.modeling_phi3 - WARNING - Current flash-attention does not support window_size. Either upgrade or use attn_implementation='eager'.
Loading checkpoint shards: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:02<00:00,  1.16s/it]
2024-09-05 14:54:11,049 - LLM.language_model - INFO - Warming up LanguageModelHandler
/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:567: UserWarning: do_sample is set to False. However, temperature is set to 0.0 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.
warnings.warn(
The seen_tokens attribute is deprecated and will be removed in v4.41. Use the cache_position model input instead.
2024-09-05 14:54:11,113 - transformers_modules.microsoft.Phi-3-mini-4k-instruct.5a516f86087853f9d560c95eb9209c1d4ed9ff69.modeling_phi3 - WARNING - You are not running the flash-attention implementation, expect numerical differences.
2024-09-05 14:54:11,263 - LLM.language_model - INFO - LanguageModelHandler:  warmed up! time: 0.214 s
2024-09-05 14:54:11,383 - parler_tts.modeling_parler_tts - WARNING - Flash attention 2 is not installed
2024-09-05 14:54:11,551 - TTS.parler_handler - WARNING - Parler TTS works best with flash attention 2, but is not installed
Given that CUDA is available in this system, you can install flash attention 2 with uv pip install flash-attn --no-build-isolation
You set add_prefix_space. The tokenizer needs to be converted from the slow tokenizers
/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/torch/nn/utils/weight_norm.py:134: FutureWarning: torch.nn.utils.weight_norm is deprecated in favor of torch.nn.utils.parametrizations.weight_norm.
WeightNorm.apply(module, name, dim)
2024-09-05 14:54:37,955 - TTS.parler_handler - INFO - Warming up ParlerTTSHandler
2024-09-05 14:54:37,990 - parler_tts.modeling_parler_tts - WARNING - prompt_attention_mask is specified but attention_mask is not. A full attention_mask will be created. Make sure this is the intended behaviour.
2024-09-05 14:55:00,466 - TTS.parler_handler - INFO - Warmed up length 128 tokens!
2024-09-05 14:55:45,284 - TTS.parler_handler - INFO - Warmed up length 64 tokens!
2024-09-05 14:55:46,336 - TTS.parler_handler - INFO - Warmed up length 32 tokens!
2024-09-05 14:55:46,995 - TTS.parler_handler - INFO - Warmed up length 16 tokens!
2024-09-05 14:55:48,776 - TTS.parler_handler - INFO - Warmed up length 8 tokens!
2024-09-05 14:55:49,682 - TTS.parler_handler - INFO - Warmed up length 4 tokens!
2024-09-05 14:55:49,682 - TTS.parler_handler - INFO - ParlerTTSHandler:  warmed up! time: 71.728 s
2024-09-05 14:55:49,682 - connections.socket_sender - INFO - Sender waiting to be connected...
2024-09-05 14:55:49,683 - connections.socket_receiver - INFO - Receiver waiting to be connected...
2024-09-05 14:55:57,632 - connections.socket_receiver - INFO - receiver connected
2024-09-05 14:55:57,632 - connections.socket_sender - INFO - sender connected
USER:  Hello, how are you?
/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:567: UserWarning: do_sample is set to False. However, temperature is set to 0.0 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.
warnings.warn(
ASSISTANT: ('Hello!', 'en')
Exception in thread Thread-12 (generate):
Traceback (most recent call last):
File "/home/user/Downloads/speech-to-speech/sts/lib/python3.10/threading.py", line 1016, in _bootstrap_inner
self.run()
File "/home/user/Downloads/speech-to-speech/sts/lib/python3.10/threading.py", line 953, in run
self._target(*self._args, **self._kwargs)
File "/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
File "/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/parler_tts/modeling_parler_tts.py", line 3316, in generate
input_ids, model_kwargs = self._prepare_decoder_input_ids_for_generation(
File "/home/user/Downloads/speech-to-speech/sts/lib/python3.10/site-packages/parler_tts/modeling_parler_tts.py", line 2923, in _prepare_decoder_input_ids_for_generation
inputs_embeds = torch.cat([prompt_hidden_states, inputs_embeds], dim=1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 1 in the list.

I'm looking into it

Fixed it, thank you for raising it 🙏


fix(kit): tuiTabs detect changes in children tab

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
Closes #431
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Hello @imitrakov
This issue already has a PR on review, sorry, it wasn't linked to the issue so you didn't see that :(
I will close this one in favor of the other because it's almost finished. On your approach — activeElement is index dependent and index is wrong by the time it gets updated + you create a new subscription on each change detection :)


Remove also watch from calling setUI with 'clock' (alternative 1 of 2)

'clock' - called for clocks. Sets Bangle.CLOCK=1 and allows a button to start the launcher

There is another way to do this as well, will do a separate PR for that so you can choose.
https://github.com/espruino/BangleApps/pull/2684

Thank you! I think I like the 2nd alternative, but thanks for the choice even so


After_create problem with acts_as_gmappable

I built an app that creates a user model and automatically creates a profile model and a location model using the after_create callback method in the model. The model profile belongs to users, and users has one profile, and the model location belong to profile and profile has one location.
The model called location, use the gem geocoder with the current_ip address of the client machine, to pull out the current location of the user, and then save the address, the latitude and longitude in the same model.
Everything works perfectly and fine, except, when I try to add to the location model, acts_as_gmappable, then during the registration process, the same model is not more automatically created, and this prevent the creation of the database column address and obviously consequently, this will bring to an error of gmapsforails.
How I can use the after_create callback in the profile model, and then automatically create the location model, and its column (longitude, latitude, address, ip_address) and at the same time using after the acts_as_gmappable?
I think it's a problem related with which method is called before: the after_create or the gmapsforrails one...this because, if I cut from my location model the call to acts_as_gmappable, register the user, and after past back in the location model the acts_as_gmappable, everything works fine, and the google map is showed.
Thanks to everyone that could in someway highlight my mind...

Please askquestions on stavkoverflow (and show code there)


🚔 copy-paste police: gettingStarted.ts and walkthrough.ts should share code

Yes, walkthroughs is a snapshot of getting started from earlier this iteration before we overhauled getting started to look like welcome. Definite debt item here as we figure out more of the story.


Test invalid type aliases

This is a separate PR to add more failing tests for other issues, related to type aliases.
PhantomTypeAliases is a regression test for #1110. It passes in master.
MissingTypeVariables is a currently failing test for #1146. I have bundled them together to show that fixing one has not fixed the other.

@laszlopandy had me activate a thing where we can't merge anything into master unless the tests pass. So I guess it makes sense to split this up?

I don't think merging the passing tests is a high priority. It was more meant to demonstrate that the bug persists despite a similar bug being fixed.
I think it would be reasonable to merge my branch into the dev branch for this bug (whenever you get around to it) and merge tests and bugfix together.

I think these are examples of the kind checking stuff. I'd like to just track it all in that issue.


Configure not implemented?

CrossSimpleLogger.Configure("mylog", 3, 100, LogLevel.Warning) not implemented?

No, it is implemented in https://github.com/jirimatejka/xamarin-logger/blob/master/SimpleLogger/Plugin.SimpleLogger.Abstractions/SimpleLoggerBase.cs

Sorry but I don't see an interface to Configure. How do you access it?

You're right, I always rely on the default Configure called from within the logger constructor. Have just added it to the ISimpleLogger interface. Please try the latest code.

Don't you also need to create the public static method Configure in the public class CrossSimpleLogger?
public static void Configure(string logFileNameBase, int maxLogFilesCount = 3, int maxLogFileSizeKb = 100, LogLevel level = LogLevel.Warning)
{
   var ret = Implementation.Value;

   if (ret == null)
   {
      throw NotImplementedInReferenceAssembly();
   }
   else
   {
      Implementation.Value.Configure(logFileNameBase, maxLogFilesCount, maxLogFileSizeKb, level);
   }
}

No, I don't think so. You can now call
CrossSimpleLogger.Current.Configure(your options)
ideally on application start.

You may want to update your wiki? https://github.com/jirimatejka/xamarin-logger/wiki
Usage
CrossSimpleLogger.Configure("mylog", 3, 100, LogLevel.Warning)

Definitely! Have just updated, thanks for your comments.


Improve the SSLValve so it is able to handle the ssl_client_escaped_cert header from Nginx

The problem solved here is exemplified by https://stackoverflow.com/questions/64911070/clients-certificate-authentication-issue-in-tomcat-in-7-0-100.
Nginx ssl_client_escaped_cert was introduced by https://trac.nginx.org/nginx/ticket/857 and that ticket mentions that

The $ssl_client_cert variable should be considered deprecated now

(force-pushed to rebase)

(force-pushed to rebase)

HTTPd BZ issue: https://bz.apache.org/bugzilla/show_bug.cgi?id=65169

(force-pushed to rebase and update correct section of changelog.xml)

Note that Joe Orton is working in HTTPd to make this possible too A new set of headers will contain certs in DER form encoded with Base 64.

The change on HTTPd is now on trunk. It does not use the NGINX approach.

Hi. Anything prevent a committer from merging this PR (besides time)?

(force-pushed to rebase)

(force-pushed to rebase and update correct section of changelog.xml after latest release)

Could someone have a look at merging this, if it's ok?

(force-pushed to rebase and fix conflicts in changelog.xml)

(force-pushed to rebase and fix conflicts in changelog.xml)

(force-pushed to rebase and fix conflicts in changelog.xml)

There are many complaints against Ehsaas Programpeople are not receiving money


Feature Request: Allow crow to spawn at game border

Use Case:
Outline the use case and/or provide a user story.

As a crow, I want to be able to catch the player no matter where they hide

Description:
Describe the feature you'd like to see added.

Currently the crow only moves completely within the playable area. The z-pos code needs to be changed so that it can spawn along the fence boundaries making it so that it can get the player at all areas onscreen

Code that needs changed:

private int GetRandomZPos() => Random.Range(2, 9); //Modify this range

Closing issue as resolved in:

#30


Check for the correct tab name

Fixes #689 (again) (probably wasn't ever fixed in the first place)

coverage: 32.901%. remained the same
when pulling 52d12ce1a683bcf0ce234701dd6469a1ea3279a2 on ginger/image-refresh
into 4f5308e36eabe63ba089aef4f444ee918b1b824b on dev.


chore(release) fix an issue while cleaning up build dir

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7536

Relevant Lines:
8280

💛  - Coveralls


Update ExportRowsCommand.java

Fixes #{issue number here}
Changes proposed in this pull request:

Hi, can you explain us what brought you to proposing this change? It seems to be reverting a security fix we made recently, so I am quite surprised you are proposing that.


Is there an existing issue for this?

[X] I have searched the existing issues

Project Name
WinGet Automation
What happened?
Winget repo shows version is 7.0.8 when its now up to 7.0.10

Updated in: https://github.com/microsoft/winget-pkgs/pull/115697


Dead Image Link

Please see the image on this next to last blog post:
https://blog.stackexchange.com/2015/06/welcome-nicolas-chabanovsky-and-stack-overflow-in-russian/
The second image has a dead link.

Fixed in https://github.com/StackExchange/stack-blog/commit/d401008539e0feddb6f04f69a50eb433125a7a05


Qgis - PyQt5- Backwards Incompatible Changes ?

Hello @dgoedkoop @pcav  I am having issues with the plugin on this error.... reffering to python incompatibilty ?  I wondering if it is a personal installation problem or just an issue connected to
Backwards Incompatible Changes
ImportError: PyQt4 classes cannot be imported in QGIS 3.x.
Use PyQt5 or the version independent qgis.PyQt import instead.

ImportError: PyQt4 classes cannot be imported in QGIS 3.x. Use PyQt5 or the version independent qgis.PyQt import instead.
Traceback (most recent call last):
File "C:/OSGEO4~1/apps/qgis-ltr/./python\qgis\utils.py", line 334, in startPlugin
plugins[packageName] = package.classFactory(iface)
File "C:/Users/info/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\joinmultiplelines_init.py", line 3, in classFactory
from .joinmultiplelines import joinmultiplelines
File "C:/OSGEO4~1/apps/qgis-ltr/./python\qgis\utils.py", line 744, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
File "C:/Users/info/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\joinmultiplelines\joinmultiplelines.py", line 61, in
import resources
File "C:/OSGEO4~1/apps/qgis-ltr/./python\qgis\utils.py", line 744, in _import
mod = _builtin_import(name, globals, locals, fromlist, level)
File "C:/Users/info/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\go2mapillary\resources.py", line 10, in
from PyQt4 import QtCore
File "C:/OSGEO4~1/apps/qgis-ltr/./python\qgis\utils.py", line 742, in _import
raise ImportError(msg)
ImportError: PyQt4 classes cannot be imported in QGIS 3.x.
Use PyQt5 or the version independent qgis.PyQt import instead.
Versione Python: 3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:59:51) [MSC v.1914 64 bit (AMD64)]
Versione di QGIS: 3.10.3-A Coruña A Coruña, 0e1f846438

It seems a problem of my local installation.... :- (

I found the problem.. it is connected to the installation of another plugin... go2mapillary  Whenever I do install it... this pugin "joinmultiplelines" starts not to function anymore with the above error.....

@alpterlab I'm the developer of go2mapillary.
having a look to exception trace it appear that is thrown by resources.py of mapillary plugin.
but that file is non longer needed and should be deleted manually.
Even if I will soon release a new version deletin resurces files, @dgoedkoop I found very strange that your plugin import resources https://github.com/dgoedkoop/joinmultiplelines/blob/f9c201e87357b2237abea99e4de9f8b93908b798/joinmultiplelines.py#L61 without using it in the code and without having resources.py in your path.
I think there is some sys.path issue. You try to import resources without any need and I left an old version of the file forgetting to delete it.

The resource file is used to display the icon on the toolbar, in this line:
https://github.com/dgoedkoop/joinmultiplelines/blob/f9c201e87357b2237abea99e4de9f8b93908b798/joinmultiplelines.py#L69
I think I could make this code more robust by using sys.path.insert(0, ...) instead of sys.path.append(...).

Why not simply get rid of resources and load directly the icon:
self.action = QAction(QIcon(os.path.join(os.path.dirname(__file__),"icon.png")), "Join multiple lines", self.iface.mainWindow())
and remove sys.path.append(os.path.dirname(__file__)) ?
I think that the issue is caused by sys.path runtime mismatch.


ParagraphStyle needs a font size

...and maybe a TextStyle in general?
Otherwise, we end up using some arbitrary internal default font and font size for the paragraph, and the lineHeight is relative to it, which is confusing.
Does the font-family matter? I'm not sure. It probably affects the default positions of the baselines?

Done.


Missing the ip's for the canada central live stream

Hello,
what are the ip for Central Canada for the live stream ?
thanks
ID: 3942b937-b212-3ceb-d356-450b7f598f68
Version Independent ID: 8b015b42-cf02-3942-3cc3-6e0f977d191a
Content: IP addresses used by Application Insights and Log Analytics
Content Source: articles/azure-monitor/app/ip-addresses.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@eazykiel thanks for your question. we are currently investigating and will get back to you shortly. thanks for your patience.
Cheers.

reassigning to @mrbullwinkle

@eazykiel the doc has the current up-to-date list of Live Metrics endpoints for Application Insights. As the engineering teams adds more we will update the doc accordingly. We definitely have had times where we have lagged in updating the doc in the past, so thank you for reaching out to confirm.
At the moment, there is no dedicated Live Metrics endpoint for Canada. We don't currently have a 1:1 relationship between Live Metrics endpoints and regions where Application Insights is supported though you can use Live Metrics in all regions where App Insights is supported.
Does this answer your question?

Yes it does. Thank you!
---- On Tue, 26 Nov 2019 16:50:00 -0500<EMAIL_ADDRESS>wrote ----
@eazykiel the doc has the current up-to-date list of Live Metrics endpoints for Application Insights. As the engineering teams adds more we will update the doc accordingly. We definitely have had times where we have lagged in updating the doc in the past, so thank you for reaching out to confirm.
At the moment, there is no dedicated Live Metrics endpoint for Canada. We don't currently have a 1:1 relationship between Live Metrics endpoints and regions where Application Insights is supported though you can use Live Metrics in all regions where App Insights is supported.
Does this answer your question?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I am going to close this out, but if you have any follow-up questions just let us know. Thanks again!

#please-close


Add identity policy

A policy that does nothing but pass calls through, to allow disabling execution policies all together (feature flag)

Turns out it was already there: Policy.noop :)


Compatible with PHP8（curl_init return CurlHandle)

fix #154   兼容PHP8.0。完善process_response方法的PHP-Doc.理论上process_response只应该接收resource|\CurlHandle|false三种参数类型，但是tests/OSS/Tests/HttpTest.php文件里有一处方法调用实参写了null，为了兼容，所以在PHP-Doc中也加入了null类型。

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


clean: Accept hardcoded /tmp

Motivation for this change
This fixes a build failure caused by a /tmp path being detected as impure. Tested the compiler on some of my homework.
ZHF: #97479
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I think the more proper fix would be to patch usages of /tmp to point to $TMPDIR

Since some usages are in const strings in C files that seems like too much
work.
On Wed, Sep 9, 2020, 19:42 Jon<EMAIL_ADDRESS>wrote:

I think the more proper fix would be to patch usages of /tmp to point to
$TMPDIR
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/97519#issuecomment-689715768, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS7M4XPARCS3Y7FB4P6VO3SE65A5ANCNFSM4RBUQYEQ
.

I don't think we'll need this.  Those strictness checks were reverted (ATM on master and staging-20.09), because such unintended cases were breaking more stuff in practice than anticipated.  A new version is planned in PR #97597 but it should be less strict and not give this problem.

Yes, this became irrelevant, closing.


article about programmer's career life

https://blog.steatoda.com/stop-promoting-programmers

https://github.com/mefengl/what-i-read/commit/19e81fa1d009848275a482e6c73fba84a30a0dd8


[Draft]: Dash ACL test

In the previous implementation of DASH ACL SAI, the ACL rule will always be added to a single ACL table, Because the ID of single ACL table was hardcode into the function sai_create_dash_acl_rule.
In this PR, I combined all dash ACL tables with same table name and added one ACL rule to all ACL tables.

Hi! Please consider putting your test plan to the standard location: test/docs/testplans/acl.md
See my open PR #127.
Thank you!

Hi! Please consider putting your test plan to the standard location: test/docs/testplans/acl.md See my open PR #127. Thank you!

Hi @aputriax , thanks for your reminding, I will move it.

Hi @Pterosaur For some reason the CI pipeline seems stuck in "Run PTF Tests." This is true in your fork as well as the PR. See here: https://github.com/Azure/DASH/actions/runs/3480389093/jobs/5820119909. It's been stuck in this step w/o any output for almost 2 hours. Have you successfully executed make run-all-tests in your dev environment?
Also, note that https://github.com/Azure/DASH/pull/281 is in review but once it is merged, you will need to change your PTF test base classes to SaiHelperSimplified. It should not have any functional impact, but it will bring it in alignment with other tests. Thanks for working on this!

@Pterosaur if this is to fix https://github.com/Azure/DASH/issues/232 please indicate in description, it will create a cross-reference. Thanks.

@Pterosaur if this is to fix #232 please indicate in description, it will create a cross-reference. Thanks.

@chrispsommers No problem.

Hi @aputriax , thanks for your comments. I will rebase my PR to the latest once these pre-PRs were merged.


how to make image assets determined during runtime?

I have the following piece of code, the image is not determined until the code is running in the browser. I noticed that when I compiled the code, I have to set the real file name of the image. So how can I achieve the feature of my code? Thank you so much!

                    {{module[0]}}

https://stackoverflow.com/questions/35737540/angular-2-dynamic-images

Thanks. It can be solved by using


Adding the first environment variable keeps showing message "There are no environment variables"

When adding the first environment variable to an app, the "There are no environment variables" message isn't removed from the page. See screenshot:

Thanks, its a known bug, for which I believed there already was an issue, but seems to be not there. Thanks!


Initial Polymorphic type.

Coverage decreased (-0.3%) to 93.071% when pulling 8a846f9095859e0c83e8d1abe4a1b15d567518f2 on careo:polymorphic into 3a2fecbcc869e361274a9d6eafd3c3936c858c78 on rightscale:master.

Coverage decreased (-0.1%) to 93.205% when pulling 2f8eb449d96e7ff18239d53160ba91a094d45eb5 on careo:polymorphic into 3a2fecbcc869e361274a9d6eafd3c3936c858c78 on rightscale:master.

Coverage decreased (-0.06%) to 93.271% when pulling cd6a0ae7902a8b4b854deb18449ed739a7952197 on careo:polymorphic into 3a2fecbcc869e361274a9d6eafd3c3936c858c78 on rightscale:master.

Coverage increased (+0.009%) to 93.338% when pulling de43a38bf9636218b2b3d85caa0f335688e78588 on careo:polymorphic into 3a2fecbcc869e361274a9d6eafd3c3936c858c78 on rightscale:master.

Coverage increased (+0.03%) to 93.36% when pulling dd8d62ec8842dc015e900abcf99263ea1d3e052b on careo:polymorphic into 3a2fecbcc869e361274a9d6eafd3c3936c858c78 on rightscale:master.

LGTM


Fixes ES8 composer installation instructions.

Purpose of this pull request
This pull request (PR) fixes the composer command to install the Elasticsearch 8 module
The original instruction doesn't work and is wrong:
$ composer update magento/module-elasticsearch-8 --update-with-all-dependencies

  The "--update-with-all-dependencies" option does not exist.

The correct command is to use require and not update, then it works as expected:
$ composer require magento/module-elasticsearch-8 --update-with-all-dependencies
Info from https://repo.packagist.org: #StandWithUkraine
./composer.json has been updated
Running composer update magento/module-elasticsearch-8 --with-all-dependencies
Loading composer repositories with package information
Updating dependencies
Lock file operations: 4 installs, 4 updates, 1 removal
  - Removing magento/module-elasticsearch-7 (100.4.6)
  - Upgrading composer/composer (2.5.4 => 2.5.5)
  - Locking elastic/transport (v8.6.0)
  - Upgrading elasticsearch/elasticsearch (v7.17.1 => v8.5.3)
  - Upgrading laminas/laminas-i18n (2.21.0 => 2.22.0)
  - Locking magento/module-elasticsearch-8 (100.4.0)
  - Locking php-http/httplug (2.3.0)
  - Locking php-http/promise (1.1.0)
  - Upgrading webonyx/graphql-php (v15.2.3 => v15.2.4)
...

Affected pages

https://experienceleague.adobe.com/docs/commerce-operations/upgrade-guide/prepare/prerequisites.html#upgrade-elasticsearch

Links to Magento Open Source code
N/A

@hostep, please sign the Adobe CLA so I can merge.


Link to facebook from hamburger menu doesn't work.

Clicking on facebook icon on the hamburger menu on the right takes user to https://www.facebook.com/pages/manage and shows a message 'This page isn’t working'.

Unable to duplicate - tried with Mac Chrome & Firefox as well as Win Chrome & iOS Safari
rendered markup looks correct.

@lthamizhmani Can you go back and try again? I have come across some other issues that were reported where if they go back and try the link works fine. Not sure why that is happening.


Adding/Chaning Translations for #133 and Fixed #156 in addition added Info Messages

I tested the InfoMessages on the CreateDatabase and ChangePassword Dialog (worked and no errors occured)
I tested the Translations on the PasswordChart (worked no errors occured)
No Compilation errors

tested it

fixed: resw File because before you could not compile anymore
added: correct translations for #133

Works now

Ok, then I can merge?

Wait I tripple check

Yes it works


HOTFIX: Do Not use unlimited num messages

Removed readKeyValues() that give UNLIMITED_MESSAGES which will doom to exhaust all wait time, as all its callers actually do provide the expected number of messages.

@dguy @mjsax @enothereska for reviews.


Typos and minimal refactoring

Putting a variable that wasn't used outside the loop inside and renaming it, and mainly typos.
Let me make a request here too: Please do a release. It's much easier to include this plugin that way. Thanks for your work, anyway, it was exactly what I was looking for.

@JJ As this project seems not to be maintained anymore, I built an external.js based on this version, but with many improvements (e.g. like recursive includes). My version can be found here: https://github.com/janschoepke/reveal_external. I tried to implement your suggestions into the initial commit.

I'll check it out then. Feel free to close this PR.


gcloud function deploy fails with incompatible dependencies

When deploying the cloud function, the cloud build process fails with:
gcloud functions deploy CVSCapacityEventManager --entry-point CVSCapacityManager_alert_event --trigger-topic CVSCapacityManagerEvents --runtime=python39 --region=europe-west1 --service-account $serviceAccount --env-vars-file .temp-event.yaml
Deploying function (may take a while - up to 2 minutes)...⠛
For Cloud Build Logs, visit: https://console.cloud.google.com/cloud-build/builds;region=europe-west1/e65e237f-a586-4acc-9f0f-1490eb82d50b?project=823997568320
Deploying function (may take a while - up to 2 minutes)...failed.
ERROR: (gcloud.functions.deploy) OperationError: code=3, message=Build failed: found incompatible dependencies: "packaging 21.2 has requirement pyparsing<3,>=2.0.2, but you have pyparsing 3.0.4."; Error ID: 85cde8bb

I was able to reproduce the issue.
Looks like cloud build is including the python modules "packaging" and "pyparsing" internally. Packaging released new version on Oct 21 which seem to break dependencies.
Until "packaging" is fixed, a workaround is to pin the version of "pyparsing" in requirements.txt to 2.4.7.


ENH:  support for scientific notation and decimal numbers in si role

So far the si role only supported integer numbers. With this PR it becomes possible to use decimal and scientific "e" notation. Additionally a negative sign "-" at the start is also accepted

@fwkoch I updated this PR as I thought it would be a good place to start looking at roles. Would you be happy to give this another review?
I haven't given much thought to what the renderer should do with this AST yet, e.g. use KaTeX to render the math text.

@rowanc1 we should do something about this. Is there a reason to consider it blocked, or is it mergeable?


negative forecasts

Detailed Description
Would be good to make sure we cant return negative forecasts
Context

we used to have this, but it got removed, as it was causing the API to error (correct thing to do)

This makes sure we dont return negative values, of course -0.0001 is fine, but if its bigger, its really obvious and should not be shown to the user

Possible Implementation

clip forcasts in forecaster to 0.0
in api, loop over forecasts and adjust to >=0, or perhaps we can do this in sql code?

Ill close thisn for now


Build a Java App with CockroachDB Doc Update

Re: Build a Java App with CockroachDB
Issue Description
This doc only talks about connecting to an insecure cluster,   while connect to secure cluster, it fails with "org.postgresql.util.PSQLException: ERROR: cleartext connections are not permitted"
Suggested Resolution
Should list example about connecting to secure clusters: how CAs are generated and used; only use user and password to connect a secure cluster, etc.

Thanks for opening this issue, @249550148. We definitely plan to add secure examples to the build an app tutorials. We're tracking that work here: https://github.com/cockroachdb/docs/issues/631


How to have single Fetch instance?

Hello, I want to add to a single download queue from different activities and have single listener, is it possible?

@sough This can easily be done. Fetch does not depend on the Activity context, but on the Application context. You can create a single instance of Fetch in a wrapper class and share it throughout you application. You can do something like this:
Example:
public final class DownloadManager {

    /*Do not worry about the lint warning for context. We are using the Application context.*/
    private static Fetch fetch;

    private static final FetchListener fetchListener = new FetchListener() {
        @Override
        public void onUpdate(long id, int status, int progress, long downloadedBytes, long fileSize, int error) {
            //D0 CHECKS HERE
        }
    };

    public static Fetch getSingleFetchInstance(Application application) {

        if(fetch == null) {
            fetch = Fetch.getInstance(application);
            fetch.addFetchListener(fetchListener);
        }

        return fetch;
    }

}

Hope this helps

Thx, I just made all settings + listener in CustomApplication class, as I understood that Fetch depends only from AppContext and every call of Fetch.getInstance(context) will return that instance from CustomApplication


Display fractional star ratings

Feature request type
Enhance component
Component name
Rating
Is your feature request related to a problem?
No response
Describe the solution you'd like
I would like for the current Rating component to be able to support fractional ratings.
An example of this would be when a rating of 4.5 is provided, it would display 4 and a half stars.
Have you seen this feature anywhere else?
An existing Rating implementation in Angular that supports fractional ratings (Angular Bar Rating)
https://github.com/MurhafSousli/ngx-bar-rating
A live demo of this feature in action (At the bottom of the page)
https://murhafsousli.github.io/ngx-bar-rating/
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Any chances to have this one on a next milestone of MudBlazor ?


On load, columns are narrow, then resize properly

The issue is that on some existing code that is been used okay for years, has begun to momentarily size columns that contain other components (ng-multiselect-dropdown v0.3.5 is one) to around 10 pixels. Once the table has loaded (half a second or so), the column change to their proper size.
The component, when initially built, was looking lovely but over time, as the table has been added to and gets more data, the problem has got worse.
It is also intermittent, sometimes the table is okay, sometime not
I'm submitting a ... (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

While this is not a showstopper, is is annoying to the user.
Reproduction of the problem
Here is a screenshot of the bug:

and an example of the code:
ngx-datatable code.txt
The software is used on a variety of systems, mostly modern Chrome/Firefox/IE running on Win 10.
ngx-datatable v ^18.0.0
Angular v10

Did you ever find a solution for this?

In a word NO. As this error is very intermittent (only a pain when network traffic slows the system down) we are living with it for now.
We are going to use Material Design in the future because we are using other ‘ngx-‘ plugins and they have annoying little bugs too and no one seems to support this.
Anthony Barrett
Senior Developer (UI)
T       0845 603 9000
fax:
E       @.***
Liaison Court
|
Vincent Road
|
    Worcester

|
WR5 1BW
@.***
@.***
@.https://www.linkedin.com/company/liaison-group-uk @. https://twitter.com/Liaison_Group
www.liaisongroup.comhttp://www.liaisongroup.com/
Click here for support, best practice and helpful guides in our Covid-19 Hubhttps://liaisongroup.com/covid-19-hub/
http://www.liaisongroup.com/
Email disclaimer:
This email and any files transmitted with it are private and intended solely for the use of the individual or entity to whom they are addressed.
If you are not the intended recipient, the email and any files have been transmitted to you in error and any copying,
 distribution or other use of the information contained in them is strictly prohibited.
Liaison Group is the trading name of Liaison VAT Consultancy Ltd (Reg. 2314163tel:2314163) and Liaison Financial Services Ltd (Reg. 6426660tel:6426660)
Registered Office: Estate House, Evesham Street, Redditch, Worcestershire B97 4HP.
Liaison Financial Services Limited, copyright 2021.
From: Pablo Quemé @.>
Sent: 21 September 2022 19:45
To: swimlane/ngx-datatable @.>
Cc: Anthony Barrett @.>; Author @.>
Subject: [EXTERNAL] Re: [swimlane/ngx-datatable] On load, columns are narrow, then resize properly (Issue #2050)
Did you ever find a solution for this?
—
Reply to this email directly, view it on GitHubhttps://github.com/swimlane/ngx-datatable/issues/2050#issuecomment-1254092556, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWZTVQBNEXOFVBLTRNDQFALV7NJRTANCNFSM5JOY6SDQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Release 4.10.0

Release 4.10.0
Next Dev version 4.10.1-SNAPSHOT
@saig0 : wanted to assign this to christian, but cannot find him on github anymore

Hi @jangalinski,
Currently the Camunda Runtime team (team leads @romansmirnov and myself) has the responsibility of making community extension releases. You can ask us to make a release by raising a CAM task issue in JIRA. I have done this here for you: https://app.camunda.com/jira/browse/CAM-10238 and will trigger the release in a minute :)
On a side note, Christian is no longer working for Camunda.
Cheers,
Thorben

@jangalinski the release has been built and is available on Maven central: https://search.maven.org/artifact/org.camunda.bpm.extension.mockito/camunda-bpm-mockito/4.10.0/jar


Port mapping does not update in UI after updating

I created a workload with random port mapping, and then update it to a fixed port. but the UI still display the old one.

Useful
Info

Versions
Rancher v2.0.0 UI: v2.0.41

Access
local admin

Route
containers.index

:(, I found the actual port mapping does not change, still 31311...

I can reproduce this issue in rancher/rancher:master 05/11.
Steps:

create a workload with a random NodePort
upgrade the workload with a specific node port

Results:
The new port is updated in API but publicEndpoints doesn't get updated.

We are not updating node port because this issues here https://github.com/kubernetes/kubernetes/issues/58892.
It will cause api server crash when updating node port.
Code in here https://github.com/rancher/rancher/blob/master/pkg/controllers/user/workload/workload.go#L177
cc @alena1108

According to the last comment in that issue (https://github.com/kubernetes/kubernetes/issues/58892#issuecomment-360947235) it is already fixed. Can the code for changing nodeport be uncommented now, or is there some other issue that's preventing that?
Btw. Rancher silently ignoring the nodeport change really took me by surprise and I spent a while figuring it out. I'm still in the phase of learning all the quirks so thanks for your comment and a link to the relevant part of code.

@dankeder the fix is backported to k8s 1.8.6 that we currently use as a vendor dependency, going to update rancher + put a proper fix

Tested with v2.0.3-rc1
This issue is still reproducible.
When I remove the node scheduling rule as part of upgrade ,I dont see pods getting recreated.
Also when I view the workload details,  the node scheduling rules are still present.

Tested with v2.0.3-rc1
Able to upgrade a workload created with random port mapping to use a fixed port.


Allow styles and images from same site doesn't work

Using 0.81.1 on Firefox 35. Ruleset imports fine, and I enabled and saved, but testing on various sites has requests still blocked.

Try using 0.19 Pre3, then remove old rule and re-import https://github.com/futpib/policeman-rulesets/blob/master/general/allow_styles_and_images.ruleset


Comments tab doesn't work + v2.0.5 says it's v2.0.4

Esteem surfer is very cool app! I've just used it for the first time today.
I've found two problems:

"Comments" tab shows nothing. Other tab (Blog, Replies, Wallet) works fine.
It says 2.0.4 in the status bar, while I installed with https://github.com/esteemapp/esteem-surfer/releases/download/2.0.5/esteem-surfer-2.0.5.dmg

App version: 2.0.5 (might be 2.0.4, see above)
Version: MacOS 10.14.2 (Mojave)

fixed. closing..


Fix travis build for 2.12

Travis build has been failing for a while because it keeps trying to run the Scala 2.12 builds on Java 7 or lower. In addition, we should really run the tests on openjdk8 as well.

Fixed.
Unfortunately I had to remove java 6 builds as these are no longer supported.


Style the instructional placeholder/comments

This is an indicator that guides users where to enter code in exercises.
// STEP 1: Put constructor here.
// TODO: Put constructor here.
Goal: allow users quickly identify where code should go
Currently we used comments for this purpose

It would be good to further style this placeholder

Status of this issue is also tracked in https://github.com/orgs/flutter/projects/3.

As part of the new, simplified version of the DartPad frontend, the exercise functionality this was related to was removed.
If anyone would like to see similar functionality brought back, please check out https://github.com/dart-lang/dart-pad/issues/2702 or open a new issue. Thanks so much!


Cannot Install Extension

Let me know if I'm wrong in assuming the installation needs the assets from codings.net or not and whether the installation process downloads files from that place automatically - and is accessible for everyone.
My installation of this extension simply sits idly for 10 minutes and still failed to install. Since there are barely any logs for whatever's happening/downloading from extensions I can't say what's wrong. Those assets are accessible via wechat/tencent unless the extensions downloads them without such need.

did you install from webui extensions?
depending on your Internet speed, assets is a package which is about 40M. If you go through the extension installation from webui, the entire repo is about 400M something and will automatically download assets for you

*** Error completing request
*** Arguments: ('https://github.com/miaoshouai/miaoshouai-assistant.git', ['ads', 'localization', 'installed'], 0, 'assis') {}
    Traceback (most recent call last):
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\objects\submodule\base.py", line 1240, in module
        repo = git.Repo(module_checkout_abspath)
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\repo\base.py", line 282, in __init__
        self.working_dir: Optional[PathLike] = self._working_tree_dir or self.common_dir
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\repo\base.py", line 363, in common_dir
        raise InvalidGitRepositoryError()
    git.exc.InvalidGitRepositoryError

    The above exception was the direct cause of the following exception:

    Traceback (most recent call last):
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\objects\submodule\base.py", line 608, in update
        mrepo = self.module()
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\util.py", line 144, in wrapper
        return func(self, *args, **kwargs)
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\objects\submodule\base.py", line 1245, in module
        raise InvalidGitRepositoryError("No valid repository at %s" % module_checkout_abspath) from e
    git.exc.InvalidGitRepositoryError: No valid repository at E:\AIs\stable-diffusion\tmp\miaoshouai-assistant\assets

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "E:\AIs\stable-diffusion\modules\call_queue.py", line 55, in f
        res = list(func(*args, **kwargs))
      File "E:\AIs\stable-diffusion\modules\ui_extensions.py", line 384, in install_extension_from_index
        ext_table, message = install_extension_from_url(None, url)
      File "E:\AIs\stable-diffusion\modules\ui_extensions.py", line 357, in install_extension_from_url
        submodule.update()
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\objects\submodule\base.py", line 666, in update
        mrepo = self._clone_repo(
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\objects\submodule\base.py", line 301, in _clone_repo
        clone = git.Repo.clone_from(
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\repo\base.py", line 1308, in clone_from
        return cls._clone(
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\repo\base.py", line 1219, in _clone
        finalize_process(proc, stderr=stderr)
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\util.py", line 419, in finalize_process
        proc.wait(**kwargs)
      File "E:\AIs\stable-diffusion\venv\lib\site-packages\git\cmd.py", line 604, in wait
        raise GitCommandError(remove_password_if_present(self.args), status, errstr)
    git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)
      cmdline: git clone -v -n --separate-git-dir=E:/AIs/stable-diffusion/tmp/miaoshouai-assistant/.git/modules/assets -- https://e.coding.net/miaoshouai/miaoshou-stable-diffusion-webui/assets.git E:\AIs\stable-diffusion\tmp\miaoshouai-assistant\assets
      stderr: 'Cloning into 'E:\AIs\stable-diffusion\tmp\miaoshouai-assistant\assets'...
    POST git-upload-pack (185 bytes)
    POST git-upload-pack (227 bytes)
    error: RPC failed; curl 56 OpenSSL SSL_read: Connection was reset, errno 10054
    error: 5273 bytes of body are still expected
    fetch-pack: unexpected disconnect while reading sideband packet
    fatal: early EOF
    fatal: fetch-pack: invalid index-pack output

Finally had an error in console. And yes I installed from the webui itself.


Lock travis.ci go version to 1.10.3 and add soft tag to revision

/assign @lpabon @jsafrane
travis.ci needs to be locked to 1.10.3 because previously it was defaulting to a beta 1.11.x version that had go fmt issues

/lgtm
I am sorry, I can't merge in this repo.
@lpabon?


Add source to span details

Fixes https://github.com/dotnet/aspire/issues/3711
While making other repository changes I noticed that the source was on the trace instead of span. Moved it, and added the UI to display a context section to span details. The new section includes the span's source (in .NET land, this is the activity source name such as Microsoft.AspNetCore or System.Net.Http), parent id and trace id.

Microsoft Reviewers: Open in CodeFlow

/azp run

Please review 🙏

main rolling build broke:
D:\a\_work\1\s\tests\Aspire.Dashboard.Tests\TelemetryRepositoryTests\TraceTests.cs(147,13): error CS0117: 'GetTracesRequest' does not contain a definition for 'ApplicationServiceId' [D:\a\_work\1\s\tests\Aspire.Dashboard.Tests\Aspire.Dashboard.Tests.csproj]
##[error]tests\Aspire.Dashboard.Tests\TelemetryRepositoryTests\TraceTests.cs(147,13): error CS0117: (NETCORE_ENGINEERING_TELEMETRY=Build) 'GetTracesRequest' does not contain a definition for 'ApplicationServiceId'
D:\a\_work\1\s\tests\Aspire.Dashboard.Tests\TelemetryRepositoryTests\TraceTests.cs(145,47): error CS9035: Required member 'GetTracesRequest.ApplicationKey' must be set in the object initializer or attribute constructor. [D:\a\_work\1\s\tests\Aspire.Dashboard.Tests\Aspire.Dashboard.Tests.csproj]
##[error]tests\Aspire.Dashboard.Tests\TelemetryRepositoryTests\TraceTests.cs(145,47): error CS9035: (NETCORE_ENGINEERING_TELEMETRY=Build) Required member 'GetTracesRequest.ApplicationKey' must be set in the object initializer or attribute constructor.

I included a fix in another PR: 5acbf8f (#4764)

I included a fix in another PR: 5acbf8f (#4764)

Cool, assuming that it is ready to merge once CI is green. Thanks!


Dialog blink twice

When I implement the datetimepicker using  tag, when click it blink twice. It happen on the demo site too.
But if I implement on  tag, it works.
The issue is I want to implement using  so it will POST by  but it will cause double blinking effect. Not sure is just me or anyone having this issue as well?
Is there any workaround? Thanks.

Hey thanks for Reporting issue.
It was happening on DateTimePicker Website.
If you are looking for a quick fix, just replace line 318 in _inputFieldClick function,
if($(this).prop("tagName") !== 'input')
with
if(!dtPickerObj._compare($(this).prop("tagName"), "input")).

Awesome ! Tested and it is working. Thanks !


Provide access to UIImageView

Would it be possible to open up access to the Card backgroundIV and iconIV in a future release to allow for easy integration with SDWebImage?

If you need to access imageViews you could try subclassing Card and make a getter for private vars.
Something like this:
class MyCard: Card {

  public var backgroundImageView: UIImageView {
    get {
      return backgroundIV
    }
  }

}

@PaoloCuscela  Doing this, it says 'backgroundIV' is inaccessible due to 'internal' protection level
https://image.prntscr.com/image/vBhfM2wTQZeHzHu72_WnNg.png

I have the same error


Add script to fetch congresspeople details

I talked to @joniarroba about useful data to create analysis that is available at Camara's API (e.g. age) but isn't fetched by the scripts in src/.
So this PR adds a new script that fetches and creates a pandas.DataFrame with birth_date, civil_name, congressperson_id, gender.
This script is "forked" from src/fetch_civil_names.py, but uses Camara's API (as noted on #224).

IMHO contributions in that sense should enhance or completely replace src/fetch_civil_names.py — and not add another script that generates (partially) duplicated data. Also we need to document this script in CONTRIBUTING.md.
TBH with this PR I believe we don't need src/fetch_civil_names.py anymore.
So what I suggest is:

[ ] Document this new script in CONTRIBUTING.md
[ ] Mark src/fetch_civil_names.py as deprecated in CONTRIBUTING.md
[ ] Generate a dataset and share the link with us so we can add to the serenata_toolbox list of datasets (the list that is downloaded in the setup process)

What do you think about it?

Document this new script in CONTRIBUTING.md
Mark src/fetch_civil_names.py as deprecated in CONTRIBUTING.md

Done (9b2331a)

Generate a dataset and share the link with us so we can add to the serenata_toolbox list of datasets (the list that is downloaded in the setup process)

https://we.tl/yrnPxCmQFA
The next step is probably refactoring anything that depends on src/fetch_civil_names.py to use src/fetch_congressperson_details.py, right?

The next step is probably refactoring anything that depends on src/fetch_civil_names.py to use src/fetch_congressperson_details.py, right?

That's about right, AFAIK there isn't anything depending on it... But if there is, yes, a refactor is in order.

I'm sorry @caiopo — it took a while for me to get back to this PR and the WeTransfer link is not working anymore. Can you re-upload it?
Besides that, do you confirm no refactor is needed as in:

The next step is probably refactoring anything that depends on src/fetch_civil_names.py to use src/fetch_congressperson_details.py, right?

Next I think this is good to merge ; ) Many  many thanks

Hi! :smile:
According to a grep -ri "civil-names" develop/ (searches for civil-names in files inside develop/), no files depend on that dataset, so I deleted it on d7aaa824f133e89f0eee5c0d81a4bc2d1709bc71.
Here's the new WeTransfer link: https://we.tl/9WAO5WVpUW
I also fixed the conflicts so the branches can be automatically merged. :)

Many thanks, @caiopo 🎉


combined oxidative phosphorylation deficiency (MONDO:0000732) definit…

…ion is truncated
Closes #8216

@sabrinatoro It is difficult to find a definition for the grouping term. Based on the definitions for term 'combined oxidative phosphorylation defect type 8' and 'combined oxidative phosphorylation defect type 9' the original truncated definition attributed to Chris would be correct if the words, "are affected" are added, i.e. A mitochondrial oxidative phosphorylation disorder in which multiple mitochondrial respiratory chain complexes are affected. See for more evidence: https://pmc.ncbi.nlm.nih.gov/articles/PMC2563189/
The OMIM definition "A multisystem disorder with variable manifestations resulting from a defect in the mitochondrial oxidative phosphorylation (OXPHOS) system." [OMIM:609060] does seem to describe MONDO:0016387 'mitochondrial oxidative phosphorylation disorder'
Not to open a can of worms, but if we have combined oxidative phosphorylation deficiency shouldn't we also have isolated oxidative phosphorylation deficiency?
Please let me know your thoughts and I will update the PR. Thank you :)

I completely agree! If we have "combined" we should have "isolated"
This publication might have answers. Looking (very briefly) at Table 2, it looks like a variation in 1 gene (most often from the nuclear DNA) can affect more than one respiratory complex.
In summary, what you originally had makes sense (I didn't have the background to understand). The answer to my original question, Are multiple complexes affected in one disease? (even if there is only 1 gene affected), is therefore yes.
After all this back-and-forth (sorry about that), I think that

the original definition (1 disease with multiple complexes affected) is correct.
since we are here, let's update the definition for the parent term "MONDO:0016387 'mitochondrial oxidative phosphorylation disorder'"
just FYI, there is a "mitochondrial disease" group who contacted us and will ask for new terms. We can ask them to review the classification.
we need to add "combined/isolated" to our patterns to review.

Thank you @katiermullen, sorry for the back and forth.


The Update API App endpoint is missing

There are currently SDK methods to create a new app, list all apps and delete an app, but you cannot currently update app settings.

Here is a workaround using the RestSharp request library to update white_labeling_options:
(```
static void UpdateAccount(string apiKey, string client_id)
{
var key = apiKey;
var httpClient = new RestClient("https://:@api.hellosign.com/v3/");
httpClient.Authenticator = new RestSharp.Authenticators.HttpBasicAuthenticator(key, "");
string urlString = "api_app/" + client_id;
var request = new RestRequest(urlString, Method.POST);
request.AddParameter("white_labeling_options", "{"primary_button_color": "#7DA26B", "link_color": "#7DA26B", "primary_button_color_hover": "#7DA26B", "secondary_button_text_color": "#7DA26B", "secondary_button_text_color_hover": "#7DA26B", "legal_version": "terms2"}");
IRestResponse response = httpClient.Execute(request);
var content = response.Content;
Console.WriteLine(content);
})

Please add this in the meantime since the open-api is not released for production yet.


xfig: 3.2.7b1 -> 3.2.8

Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110223 run on x86_64-linux 1

1 package built:

xfig

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110223 run on x86_64-linux 1

1 package built:

xfig


Myt

Sorry about that. I just fixed the CI failed at the master branch.
Please make a new commit to trigger the CI again.


Throw function should return by default never instead of returning void

VS Code Version: 1.54.1
OS Version: Windows 10 20H2 Build 19042.572

Steps to Reproduce:

Example below

It should recognize that it return never

Another case Infinity loop the editor should warning user it infinity loop and set return type to never
function infinityLoop(): never {
    while (true);
}

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

This feature is powered by the TypeScript project. Please file a bug upstream


one-time code exchange - noob question

Hey there, first of all. Thank you for this great utility. I was hoping you could elaborate on the token exchange on the back-end. I have been able to successfully use option b for a while but I would really like the refresh token.
I'm using node express and I've gone through all of the examples provided on the google-api-nodejs-client repo but I can't figure out how to exchange the one time token for the access and refresh token. Any help would be greatly appreciated!

Ah... I missed a really key piece
{ code: authorizationCode, redirect_uri: 'postmessage' }
I was just trying to send the authorizationCode and not the redirect_uri and was getting a redirect uri mismatch error. Thanks


Add Support for Parsing of Races not included in the system, e.g. Troll

Add Support for Parsing of Races not included in the system, e.g. Troll
Originally posted by @Lavaeolous in https://github.com/Lavaeolous/PF1-StatBlock-Converter-Module/issues/447#issuecomment-850955504

Support added in 3.3.0, but still need to collate a list of non-playable races.


add gitlab issues

data in _raw_gitlab_api_issues(raw layer)

data in _tool_gitlab_issues(tool layer)

data in issues(domain layer)


Add Kindle book of documentation to release package

Attached is the v 9.7.0 draft that I’ve just copied over for perusal and then, once the release manager is anointed, we can update.  I still need to look at doing this automatically, but, actually, just doing changes from this point is probably easier and will give us the opportunity to flex a little and also provide more of an introduction to the groupings that I refer to in the name of the file as ‘constellations’.
2021-06-10_gist_10.0.0_constellations.md

@pwin Include in this issue: Add file to list of files in bundle.yaml at line 187:
  includes:
    - "ReleaseNotes.md"
    - "MajorVersionMigration.md"


Create Emojis

Emojis
Summary

Changes

Closes:
Task list

[ ] For workflow changes, I have verified the Actions workflows function as expected.
[ ] For content changes, I have reviewed the style guide.

🤗🤗


Upgrade to new build 0.4.0

this PR is a sub-task of https://github.com/crate/tech-writing-domain/issues/353
changes:

Pins the docs-build to 0.4.0
Replaces Travis CI docs tests with a new GitHub Actions workflow
Adds a Mergify configuration

@infoverload can you please review this? ty! <3

this PR swaps Travis CI for GitHub Actions. accordingly, the branch protections for master need to be updated like so:

[x] Disable the Travis CI checks
[x] Enable the new GitHub Actions checks ("Build docs on macos-latest" and "Build docs on ubuntu-latest")

the "docs/readthedocs.org" check should be enabled if it isn't enabled already and any other checks should be left as they are
the .mergify.yml configuration (added in this PR) identifies the correct tests to include (lines starting with status-success)
can one of the @crate/sysadmins please do this for me? merging is blocked on this. thanks so much!

@autophagy do you know why there's a ghost PR check on this? I am wondering if the branch protection is misconfigured and is requiring a PR check that no longer exists


SR-2356 : Remove temporary APIs from Linux overlay

Remove temporary APIs from Linux Dispatch overlay;
swift-corelibs-foundation builds without them and
they are not present in the Darwin Dispatch overlay.

I'll create a matching PR for the swift-3 branch as soon as this one is merged to master.


Adding new entry to apps section - ipblog

IpBlog is an anti-censorship site. Users can write articles and then view them as a list.
https://github.com/Khushmeet/ipblog

Thanks, @Khushmeet. I think that this could use a bit more polish - for instance, a fuller README, and example website, instructions on how to set it up, and so on. Feel free to resubmit later when there has been significant usability work improvements.


[HOTFIX] Fix python tests after #9527

#9527 missed updating the python tests.

Merged build triggered.

Merged build started.

/cc @rxin

Test build #45274 has started for PR 9533 at commit 0b436e5.

Test build #45274 has finished for PR 9533 at commit 0b436e5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * abstract class Writer extends BaseReadWrite \n  * trait Writable \n  * abstract class Reader[T] extends BaseReadWrite \n  * trait Readable[T] \n  * case class GetInternalRowField(child: Expression, ordinal: Int, dataType: DataType)\n  * case class Expand(\n

Merged build finished. Test PASSed.


Consuming Sierra APIs from AWS

How does DDS talk to Patron API (SOAP) and PINTEST (REST-ish) from AWS?
Test both these APIs externally to check that they are callable.

Platform already calls various Sierra services from AWS without a VPN, so we just need to test that the ones we need are accessible.
Do that by deploying something that uses them, when we get to it.

For the spike in current work plan - make a POC page in new DDS, a form that takes your username and pin/password and returns some info from Sierra about you (v minimal, just to exercise the pintest and patron (SOAP) APIs to uncover any difficulties in comms with Sierra from AWS. No data storage required.

To exercise the flow here, we are going to need a bit of configuration in DLCS to allow it to be tested in parallel with the live site.
Here's an image with Clinical login required:
https://dlcs.io/iiif-img/wellcome/5/b21072061_0001.jp2
The auth service provided by this info.json is here:
https://dlcs.io/auth/2/clinicallogin
(This is a row in the DLCS AuthServices table, that includes labels for consuming UIs to display)
This is where a IIIF client will send the user in a new browser tab/window, and wait for the interaction to complete, without any knowledge of what might be happening in that window.
The DLCS immediately delegates auth to the DDS, by redirecting the user to https://wellcomelibrary.org/iiif/dlcslogin.
This URL is configured in the DLCS - this is what we need to be more flexible about, see later...
In the new IIIF Builder this redirected URL will be https://iiif.wellcomecollection.org/iiif/dlcslogin (or something along those lines).
The user logs in, the DDS establishes a DDS session for the user, and issues a token that represents that session.
The user is then redirected back to the DLCS with this token. The DLCS calls the DDS using this token, to acquire the user's roles. The DLCS is now the only consumer of the user's DDS session (no more wl.org use). The DLCS maintains a session of its own for the user, to authorise tile requests for access-controlled content, without having to talk to the DDS per-request.
The DLCS might periodically refresh its session by redirecting the user to the dlcslogin page, which will redirect straight back if the DDS session is still good, or show the login screen again if it has expired.
For milestone 3, we don't need to change the DLCS - we can prove the Sierra connectivity, and the MVP the DDS needs to provide, simply by maintaining a session for a user that has been authenticated via Sierra's APIs, and listing the roles they have. We don't need any other info - no personal info at all needs to be retrieved from Sierra, let alone stored anywhere.
But we will need to change the DLCS to allow us to integrate with this DDS session.
Observations:

Whatever we do here is a temporary version, to be later replaced by a Cognito-backed store. However, this DDS version represents the minimum viable auth, so is a useful reference for the later version. Easier to see what needs doing here.
On current DDS the session is maintained in the user database. This new version doesn't need to know anything at all about the user. Session could be maintained in something very lightweight, doesn't have to be a DB table.
We're going to need to have the DLCS send the user to one URL for space 5 images, and another URL for space 6 images. We need to further explore how best to do this.

It could be a space default - the delegated login screen is configured on a space basis - but it might be better to associate the redirect URL with the role registered for the image, so the provider of login and of roles is associated with each image.
We can name our test roles differently for images in space 6 (and have the test DDS return the renamed roles). The DLCS redirects to a role-specific login page (what happens if an image has more than one role, though?)
(note to self - might be worth building an admin UI in the portal (or parallel portal 2) to reason about how this is configured.

See Slack discussion:
https://digirati.slack.com/archives/CBT40CMKQ/p1596196188406500
The PINTEST API is not available over the public internet, so we should use the new API for this:
https://techdocs.iii.com/sierraapi/Content/titlePage.htm
This looks like the equivalent operation:
https://sandbox.iii.com/iii/sierra-api/swagger/index.html#!/patrons/Validate_patron_by_barcode_and_PIN_post_11
For this we'll need keys, which @pollecuttn can generate for us (we think).

Created 2 new roles in DLCS for testing this:

https://api.dlcs.io/customers/2/roles/testClinical - pointing at iiif-test. See https://dlcs.io/iiif-img/wellcome/6/b10003332_0004.jp2
https://api.dlcs.io/customers/2/roles/localClinical - pointing at local (via ngrok). See https://dlcs.io/iiif-img/wellcome/6/b10003332_0007.jp2

At some point we will need to update DLCS roles, so that existing images use the new login at https://iiif.wellcomecollection.org/roleprovider/dlcslogin
Leave this open for that!

This is working, with Patron REST API (no need for SOAP and Pintest any more).
But we need to revisit later after Wellcome's Sierra is patched to 5.2, to fix a few quirks.
So:

[ ] Revisit and test after Sierra 5.2 upgrade
[ ] Bulk update all authed images to a roleprovider associated with the new login page ^
[ ] Remove Wellcome.Dds.MillenniumClient from IIIF-Builder entirely!
[ ] Remove SierraPinTest from Wellcome.Dds.Auth.Web

Much later:
Refactor so that DDS is a client on Cognito. => Cognito requirements - it must be able to give us the role info used in https://github.com/wellcomecollection/iiif-builder/blob/master/src/Wellcome.Dds/Wellcome.Dds.Auth.Web/Roles.cs#L52
Moving this issue to Milestone 5, to come back to.

(update - wait for SSO work to finish then integrate with that - we have a working Direct Sierra route as a fallback if timing is an issue).

(update - wait for SSO work to finish then integrate with that - we have a working Direct Sierra route as a fallback if timing is an issue).

@jtweed shall we move this off the critical path?
It is done and was tested before we stopped after part 1 of this work, although would need a fair amount of re-testing.
Later, we would replace direct Sierra calls with SSO integration.
But at launch, we don't have any IIIF content at a higher access level than Open With Advisory, which does not need Sierra auth.

If we have nothing that needs Sierra auth (which I think is the case currently) then let's drop this for now.
I can see auth against SSO becoming a thing at some point, to provide eg staff access to restricted video or potentially for access to restricted born digital.
But we can cross that bridge when it comes and it will be against SSO, not Sierra.

Closing as no work to do now.


How to make the test.py work without CUDA and CPU only?

哥们儿，没有显卡只使用CPU，怎么搞？

Please refer to this: https://blog.csdn.net/moshiyaofei/article/details/90519430


Lab 4 Course toString() method

Description:
I'm not sure how to go about implementing the toString() method in the Course class. I tried using values() but it returns square brackets instead. Any help is appreciated, thanks!
Code Snippets
Attach code snippets (if any)
Screenshots
my current output:

I also just found out that we are to allowed to use values(), can i get other suggestions on how to implement toString() then?

you can try looping through each item in the immutablemap object, there's an example on how to do that in the exercise

you can try looping through each item in the immutablemap object, there's an example on how to do that in the exercise

hi, thanks for the help! i tried that too but im stuck at how i should add commas only between adjacent elements 😭

check if the string is empty, if its not empty, add a comma then the element, if its empty, just add the element

check if the string is empty, if its not empty, add a comma then the element, if its empty, just add the element

solved! thanks so much!


Add copyright notices

Fixes the CI test for copyright information.
We need to confirm the copyright status of those files that were copied or edited from rmw_fastrtps_cpp files.

@clalancette Do you know what the accepted practice is for copyright notices on files that were

copied from another project and used as is, or
copied from another project and modified.

@clalancette Do you know what the accepted practice is for copyright notices on files that were

copied from another project and used as is (example here), or
copied from another project and modified (example here).

The typesupport class definition is probably the most egregious copy from rmw_fastrtps :X

Should we just merge this? I'm not too sure on the practices for copyright

@clalancette Do you know what the accepted practice is for copyright notices on files that were
1. copied from another project and used as is ([example here](https://github.com/osrf/rmw_zenoh/blob/main/rmw_zenoh_cpp/src/impl/guard_condition_impl.hpp)), or

For files that are directly copied, the license should remain the same and the copyright holder should remain the same.  Additionally, we also usually provide a permalink to the file that we originally copied from; one example is here.

2. copied from another project and modified ([example here](rmw_zenoh_cpp/src/rmw_guard_conditions.cpp)).

The license has to remain the same.  For the copyright holder, it kind of depends on how much was changed was changed.  If it was minor/trivial, then leave the copyright holder as-is.  If it was fairly major changes, then add Open Source Robotics Foundation, Inc as a second copyright holder.  I will point out that our copyright linting tools don't actually support this case currently, but they should.
Let me know if that makes sense.


Duplicates and hyphens in wordlist

Hello! Thanks for this, was about to write something exactly like this :partying_face:
I am randomly generating numbers and encoding them with this library - and a test case failed because
unnerve-strongly-step-quartet-felt-tip
failed to parse with the UnknownWord error - because felt-tip is in the word-list the parsing based around .split("-") falls over
There was a few others like yo-yo and drop-down which are pretty common words non-hyphenated so I've just removed the -, but a few others like "felttip" didn't work so I've changed them entirely, and same with the duplicates

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you so much. I'm a bit embarrassed I didn't catch these myself!
I think I can issue this as a 0.2.1 release instead of bumping it to 0.3 -- any IDs that used these words were almost certainly broken (except by chance with the duplicates). While I'd normally consider a wordlist update a breaking change, this most likely only fixes things.
I'll get it shipped out shortly.


Browser mode

Love all of the ideas here, but would love for this to be server/browser (add browserify or webpack).

I'd love to modernize this generator, so I'm open to changes :)


OpenVDB Points Convert SOP Fixes

Fixed a bug where the auto voxel transform was being calculated on the packed geometry in the OpenVDB Points Convert SOP
Fixed a bug where points representing packed geometry were also being converted in the OpenVDB Points Convert SOP

Signed-off-by: Nick Avramoussis<EMAIL_ADDRESS>

@danrbailey : Isn't this such a filter?
parms.add(hutil::ParmFactory(PRM_STRING, "group", "Group")
    .setChoiceList(&hutil::PrimGroupMenu)
    .setTooltip("Specify a subset of the input point data grids to convert.")

Or do you mean to also filter the primitives to unpack?

@danrbailey : Isn't this such a filter?
parms.add(hutil::ParmFactory(PRM_STRING, "group", "Group")
    .setChoiceList(&hutil::PrimGroupMenu)
    .setTooltip("Specify a subset of the input point data grids to convert.")

Or do you mean to also filter the primitives to unpack?

Well, this parameter is the primitive group filter for conversion in the other direction (VDB -> Houdini). It's hidden for this direction (Houdini -> VDB) because it doesn't make a lot of sense to have a primitive group filter for points. However, it would at least make sense to include a points group filter, but whether or not that's a good idea or not, I'm not sure. I notice that some of the other VDB sops that handle points (such as VDB from particles) also don't expose a points group filter, so maybe it's just not convention to do this? I can't remember if we talked about this at some point and then decided against it.

I think I decided against the same named parameter being used as different filter types.  Having a separate ptgroup filter does make sense here though.  It could also subsume some of the implicit logic that is going on, like rejecting points that are part of packed primitives.

It could also subsume some of the implicit logic that is going on, like rejecting points that are part of packed primitives.

Are you suggesting we shouldn't always be ignoring points representing packed geo? I was going to suggest we extend this behaviour for points representing other geo too (i.e. volumes). Would converting these points ever be useful/wanted?

I think we should always ignore points of packed geo.  But it definitely is the case one could imagine other sets of points to be removed as well.  Having a group makes this easier to do without an extra blast operation.  (Which also may avoid a defragment, so be a performance win)

Will merge this soon


Unit test support

Example (with NUnit + NSubstitute)
Ref: http://jglue.org/cdi-unit-user-guide/#mocks
public class MyComponent
{
    [Inject] IDbSet<User> _users;
    [Inject] OtherComponent _otherStuff;
    public void DoSomething()
    {
    }
}

[CormoTest]
public class MyTest
{
    [Inject] MyComponent _component;
    [Produces, Mock] IDbSet <User> _users;

    [Test]
    public void Blah()
    {
        _component.DoSomething();
        _users.Received().Add(user);
    }
}

ok, perhaps xunit. For initial implementation I'll probably not even try to make it portable (e.g. nunit/xunit, nsubstitute/moq), I'll just pick xunit and nsubstitute.
Cdi-unit is portable (junit, testng, mockito, easymock).


Dataframe styling adjustments

The groupby and aggregate can look a bit weird with flex-end

The groupby and aggregate can look a bit weird with flex-end

Thanks for catching this! I've fixed this.

I was trying to make the trash icon for deleting a form row appear inline with the controls. I think it mostly works now.
To make it work I changed the FormMessage to be a tooltip in one place. Not tied to using a tooltip, but otherwise I was finding it hard to align everything. It's possible I regressed something, could use your help if I did.
Finally, I could use your help in repositioning the Drop NA for GroupBy, if possible. It'd be nice if it showed up in the bottom left or the top left, not in line with columns and aggregations as it does now.


[WIP] [AIRFLOW-356] [AIRFLOW-355] [AIRFLOW-354] [AIRFLOW-276] Refresh new dags

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-356
https://issues.apache.org/jira/browse/AIRFLOW-355
https://issues.apache.org/jira/browse/AIRFLOW-354
https://issues.apache.org/jira/browse/AIRFLOW-276

Testing Done:

Unittests are required, if you do not include new unit tests please
specify why you think this is not required. We like to improve our
coverage so a non existing test is even a better reason to include one.

Reminders for contributors (REQUIRED!):

Your PR's title must reference an issue on
Airflow's JIRA.
For example, a PR called "[AIRFLOW-1] My Amazing PR" would close JIRA
issue #1. Please open a new issue if required!
Please squash your commits when possible and follow the How to write a good git commit message.
Summarized as follows:

Separate subject from body with a blank line
Limit the subject line to 50 characters
Do not end the subject line with a period
Use the imperative mood in the subject line (add, not adding)
Wrap the body at 72 characters
Use the body to explain what and why vs. how

Is this abandoned? Seems dupe of #1678

@criccomini yes, I will close this


[WIP] Entity specific analytics (#1323)

Addresses: https://github.com/restarone/violet_rails/issues/1292

@Pralish some feedback:

Rename Visitor Analytics to Analytics

unify dropdown style between the two dropdowns shown below. It should be clear to the user that they are dropdowns

@Ayon95  with this analytics PR coming together nicely, we need a way to test it.
Could you add analytics instrumentation in the testnet (http://violet-test.net) to the spots marked below?

@Pralish Some feedback:

there are events as indicated in Legacy Analytics,

but they are not shown in Analytics V2

Please show events in a similar UI as Clicks and Watch Time (name it Events) :

Clicking on each event should lead to dashboard/events/:event_slug

Do not show Clicks / Watch Time / Events if none exist, instead render dynamic docs:

For docs, be sure to show an example each for tracking: section views, countable events, watch time, clicks

API Form submissions are not shown, they should be shown in a similar UI to Clicks and Watch time where each namespace/submission count should lead to the API Namespace #show page

I've added the necessary data attributes to the navbar links and car page links.
Car links:

Navbar links:

@Pralish some feedback based on your latest commits (note there's still some feedback here pending your implementation):

@Ayon95 nice work on adding the clickables, can you add watchables now? Please create another app like the car app (https://violet-test.net/car-renderer) but with car videos so we can populate the section marked below:

here are some car videos to get you started (please download them with a TikTok downloader and upload to the car video app)

https://www.tiktok.com/@donrestarone/video/7170127580790574342?lang=en
https://www.tiktok.com/@donrestarone/video/7189748678171593989?lang=en

@donrestarone Sure, I'll do it. Can you give me access to API namespaces so that I can create an API namespace and add videos? Thanks.

Just gave you access to https://violet-test.net @Ayon95 -- I had forgotten to restore your access level after testing a ticket

Hey @Pralish , this is looking great. Some feedback:

The scoping by page seems to be broken. The user expects to see a subset of the analytics events (eg: clickables, trackables, viewables, events) when they narrow down their search by page2.

For watch time should not depend on the existence of social share metadata

form submissions should be instrumented with sensible defaults and should have the ability to be overridden (like you have provided here, but they need to be instrumented by default)

tooltips are great. They should be a bit more dynamic and show the percentage gain/loss

@Pralish latest round of revisions look good. My final feedback:

Video tracking should be done in-near-real-time. Currently, Im observing that view time is tracked only when the user hits pause on the video. In most cases, people will watch a video and close the page without doing anything else. In cases like this, we want to track exactly how much of the video was elapsed (before the connection closed)
some of the show pages for the events are broken. I have marked them below:

another issue is mobile responsiveness, currently it looks like this:
https://user-images.githubusercontent.com/35935196/215172868-3065349d-32b0-4e55-9ed2-4db71926bef3.MP4

The responsiveness isn't working properly in phone due to events below:

https://user-images.githubusercontent.com/55496123/215235603-c2e7c13b-e447-4292-94fd-258c914b624d.mov

We can add loader while changing the pages from  the dropdown so it will be a smooth loading of data when we change the pages.

API Form submissions are shown, but each namespace/submission count isn't leading to the API Namespace #show page

This text is hard coded.

Do we need to add the following showing gain/loss percentage?

@ajaystha42 , @donrestarone ,thanks for the feedbacks

Do we need to add the following showing gain/loss percentage?

That was the design requirement.

API Form submissions are shown, but each namespace/submission count isn't leading to the API Namespace #show page

@donrestarone, didn't we decide to not implement that? Wouldn't having redirection on count on form submission make the design inconsistent with the rest of the section?

@Pralish

@donrestarone, didn't we decide to not implement that? Wouldn't having redirection on count on form submission make the design inconsistent with the rest of the section?

yup! The current implementation is fine
about the analytics mapping you will need to add the permission for it (please call it allow analytics mapping):

@Pralish found a bug, the custom events are not scoped by time period (eg: 1 month, 3 months etc):

@Pralish I took a look at the latest commits and the Date dropdown is broken on mobile/desktop and there is broken HTML at the bottom of the page:
https://user-images.githubusercontent.com/35935196/215337839-29f16d3d-ed6b-40e7-aea9-dc332be4da8f.MP4

@Pralish I back merged master into this branch (https://github.com/restarone/violet_rails/commit/70526b515158318b88da1b99bfcdb582dbc754ee) we should always keep feature branches caught up with master in the later stages of the development to avoid unforeseen issues

⚠  horizontal scrolling on iPhone on the event show page:
https://user-images.githubusercontent.com/35935196/215371047-4f42eb0b-9b81-4ab8-ac75-ab87ed192744.MP4
⚠  scoping of the events by date range (fix WIP):


Add an example as a python package for general CI tools

Hi @2bndy5 the following example you mentioned in the discussions yesterday is very instructive.
Someone who uses other CI tools like Travis CI, CircleCI, Jenkins, etc instead of GitHub Actions can also use this repo code if their code is hosted on GitHub.
I'm thinking whether updating this example code to README may help.
# Requires clang-format, clang-tidy installed

# Install linter package
python3 -m pip install git+https://github.com/shenxianpeng/cpp-linter-action

# Generate compilation database
mkdir build && cmake -Bbuild src -DCMAKE_EXPORT_COMPILE_COMMANDS=ON

# Run linter as a package
# --version arg is useless on Windows - it will use whatever version is installed
# --ignore the build/* artifacts generated by CMake
# --database indicates the path of the generated database
python3 -m cpp_linter.run --ignore=build --database=build --thread-comments=true

# Fail fast?!
if [ $? -ne 0 ]; then
    echo "Some files failed the linting checks!"
fi

Funny you opened this just now. I'm in the process of opening a PR to merge the needed modifications for windows-based runners. I'm literally in the process of updating the README for Windows-based runners.
As for using this code on other CI services (non-github), I think a fork of this repo is currently required since this action's src code heavily relies on github-specific env vars. While similar env vars might be available on other CI services, they might be named differently (which is a breaking point).
It would take a lot more research and testing to adapt the source code for other CI services' env vars, but it is definitely possible. I don't have a bitbucket account, but I do have a gitlab account (never really used though) and a travisCI account (haven't used it in years though).

OK, this needs much effort. I just realized if their code is hosted on GitHub why not use GitHub Actions :zany_face:

It is a good idea though. I didn't think to use it for other CI services like this. Additionally, this is starting to address one of your original design goals: Make it compatible with bitbucket pipelines.
Now that we can take advantage of python, I think this idea is better than maintaining a different fork for each CI service.

Then the current source code folder cpp_linter can be moved from shenxianpeng/cpp-linter-action to cpp-linter/cpp-linter. then this cpp-linter-action can install linter python package with python3 -m pip install git+https://github.com/cpp-linter/cpp-linter

I can tell you from experience that transferring the repo from one account to another will not screw up existing workflows that still use the old account. Github does some clever redirecting in the event of repo transfers, although that might be limited to a year or so.

So you prefer to transfer to cpp-linter org?
If we transferred, I can also do the following steps to minimize the impact

use my account to fork cpp-linter/cpp-linter-action
update the child repo README deprecated, add a link to the parent repo
then make the child repo shenxianpeng/cpp-linter-action as archived

If you want to transfer the repo, then wait till the PRs are closed or merged because I've heard horror stories about transferring repos with open PRs.

OK, let's wait until PRs are merged.

use my account to fork cpp-linter/cpp-linter-action

update the child repo README deprecated, add a link to the parent repo

then make the child repo shenxianpeng/cpp-linter-action as archived

I think making this repo into an archived fork of the transferred repo is a bad idea. It might confuse github into disabling the auto-redirect behavior. Not to mention confusing people about which action code is actually being used (archived or not).

Technically, you could also transfer the docker image repo to the @cpp-linter org as well. I'm not sure if that would have implications on dockerhub though.

Thank you for your suggestions. I'm going to transfer, and it seems need necessary changes after transfered

I created a branch named transfer-update for transfer ownership update.


Johnson's Algorithm in C, C++, Java and Python

Feature = Johnson's Algorithm

Johnson's algorithm is capable of finding the shortest paths between all pairs of vertices, even in the presence of negative edge weights.
It uses the Bellman-Ford algorithm to handle negative edge weights and identify negative cycles
By transforming the graph and modifying the edge weights, it ensures that Dijkstra's algorithm can be applied effectively.

Appraoch :
1.Graph Transformation:

Add a new vertex to the graph and connect it to all existing vertices with zero-weight edges.
Assign zero weights to these new edges.

Bellman-Ford Algorithm:

Use the Bellman-Ford algorithm to calculate the shortest distances from the newly added vertex to all other vertices in the graph.
If a negative cycle is detected during this step, terminate the algorithm as it indicates that the graph contains no shortest paths.

Graph Modification:

Modify the weights of the original graph based on the results of the Bellman-Ford algorithm.
For each edge (u, v) in the graph, update its weight as the sum of its original weight and the distance from the newly added vertex to u minus the distance from the newly added vertex to v.

Dijkstra's Algorithm (Multiple Times):

Apply Dijkstra's algorithm to each vertex in the modified graph to find the shortest paths from that vertex to all other vertices.
This step is performed multiple times, once for each vertex in the graph.

Attribute:

To be added in C, C++, Java and Python.
Added comments to explain the flow and logic of code to the reader.

I would like to work on this issue. Can you assign it to me under SSoC '23 ?
Thank You :)

Can I contribute in this

Assigned!
@harshita-kaushik2 : C, C++, Java and Python

@harshita-kaushik2 What is your status on this issue?

@harshita-kaushik2  if you don't want to contribute then @Kumar-laxmi  can you assigned me i would like to do this algorithm


Defined new revisions for golang dep: github.com/heketi/tests

Ran component through the scanners since I noticed it had not been harvested yet. PR matches curation site. Closing.


Migrate existing repos to flow-docs.json

We'll need to update each repo with a flow-docs.json (the file containing the menu definition and other user-defined metadata).

[ ] Remove call-refresh-doc.yml from all repos.

Add flow-docs.json to the following repos:

[x] fcl
[ ] emulator
[ ] cli
[ ] go sdk
[ ] js sdk
[ ] flow-cadut
[ ] js-testing
[ ] kitty-items
[ ] onflow/flow* This one might require some thought as there are multiple side-menus for this content
[x] cadence
[ ] sdks
[ ] etc ... ?

cadence:

https://github.com/onflow/cadence/pull/1908
https://github.com/onflow/cadence/pull/1909

fcl:

https://github.com/onflow/fcl-js/pull/1378

https://github.com/onflow/fcl-js/pull/1378/files

static builds https://github.com/onflow/docs


Forward real client IP using a reverse proxy for websocket

Environment

OS: Ubuntu Xenial 16.04.3 LTS
Erlang/OTP: 20 [erts-9.1]
EMQ: 2.2
Nginx: 1.10.3

Description
I'm using Nginx to proxify the websocket stream to my EMQ broker using the following configuration.
Note that nginx and EMQ are running on the same host
# Nginx config
location / {
  proxy_connect_timeout 7d;
  proxy_send_timeout 7d;
  proxy_read_timeout 7d;
  proxy_set_header Host $host;
  proxy_set_header X-Real-IP $remote_addr;
  proxy_set_header X-Scheme $scheme;
  proxy_set_header X-Forwarded-Proto $scheme;
  proxy_set_header X-Forwarded-Port $server_port;
  proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
  proxy_pass http://<IP_ADDRESS>:8083;
  proxy_http_version 1.1;
  proxy_set_header Upgrade $http_upgrade;
  proxy_set_header Connection "upgrade";
}

This works great, I'm able to connect to the EMQ host using the javascript client.
But, I'am expecting to see the real client IP on the EMQTT HTTP Api.
Instead, I see <IP_ADDRESS> (the loopback of the nginx server).
# GET http://emqtt/api/clients
{
  "currentPage": 1,
  "pageSize": 1,
  "totalNum": 1,
  "totalPage": 1,
  "result": Array[1][
    {
      "clientId": "Client-23649",
      "username": "undefined",
      "ipaddress": "<IP_ADDRESS>",
      "port": 39146,
      "clean_sess": true,
      "proto_ver": 4,
      "keepalive": 60,
      "send_msg": "undefined",
      "recv_msg": "undefined",
      "connected_at": "2017-11-13 09:21:54"
    }
  ]
}

Is it actually a real issue ?
Does the broker need the real client IP address ?
(I understand that Nginx is in charge of NAT/PAT to forward packets to the client)
Does EMQ support HTTP X-* Header ?

@franquis The MQTT/WS listener still not support to extract the source address from HTTP X-* Headers. We plan to implement the feature in 2.4-beta.1, thanks.

Hi, @franquis This Feature has implemented and published at v2.3.1.


Demo broke in REPL ('default' is not exported)

There appears to be an error in your code that's not allowing me to see a demo in Svelte REPL
'default' is not exported by<EMAIL_ADDRESS>imported by<EMAIL_ADDRESS>
https://svelte.dev/repl/ef46aaaa519548faa05aef83ba95dd43?version=3.48.0

Thanks for reporting this issue. A fix was sent in v1.2.2.