Update dependencies

Hello Michael!
Excellent work with Infinity!
This PR upgrades FastEmbed to the latest version which has a lot more models and some minor changes in the API to make it ready for sparse, image and other modalities.
This also upgrades Ruff, which gives speed improvements.

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

Comparison is base (0ef322b) 86.28% compared to head (15f44a7) 86.49%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
+ Coverage   86.28%   86.49%   +0.20%
==========================================
  Files          27       27
  Lines        1269     1266       -3
==========================================
  Hits         1095     1095
+ Misses        174      171       -3

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

Thanks for the suggestion! Yes, we're working towards adding export flows e.g. optimum as well. We'll make it an optional extra like you've done as well in all likelihood.


⚠ BCR Bot has degraded performance

In 7cb7f1c, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 904 ms

Resolved: BCR Bot performance has improved in 9e8965f.


Impossible to hide the "Courtesy Notice" about active virtualenv

Issue description
When using an existing virtualenv, we get this message:

Courtesy Notice: Pipenv found itself running within a virtual environment, so it will automatically use that environment, instead of creating its own for any project. You can set PIPENV_IGNORE_VIRTUALENVS=1 to force pipenv to ignore that environment and create its own instead. You can set PIPENV_VERBOSITY=-1 to suppress this warning.

Expected result
With export PIPENV_VERBOSITY=-1 the Courtesy Notice message should not be shown.
Actual result
The message still appear with export PIPENV_VERBOSITY=-1.

This was merged so i'll go ahead and close it, thanks for reporting!


Checked attribute for binary state components

This implements the checked attribute for checkboxes, radio buttons, and switches. This allows outside input to change the state of the checkbox, radio button, or switch (e.g., a button that toggles the checked state of a checkbox).

Thank you so much for the PR! This will ship out with v0.0.36 👍


Tfrs 2183

Modal.js refactor to use react based component

There are multiple other places where jquery is being used which is not related to Modal.js so removing reference of jquery from .html file would cause problem. for example SessionTimer.js is using jquery with different modal. bootstrap.js has jquery  references. to clean up all reference of jquery would required ticket on its own.


Does not open

*I double click Macro Deck to open it - nothing happens. I will wait a few minutes, still nothing so I right click and open and it says it's already running. So, I close it in task manager. Again try to open it, nothing happens. This is an endless cycle.
Windows 11 - 64bit
Version: 21H2
OS build: 22000.376

Please send me your profiles.db located in "%Appdata%\Macro Deck". After you sent me it, you can rename it (just put a .old) after the filename and then it should work.

How do I do that?

I think it's way easier to help you if you join the Discord server: https://discord.gg/yr7TRaXum8

I have been trying but something is wrong with my phone number on discord. :-(

I am just trying to get the software to open...

I uninstalled and reinstalled and it did nothing to help

Facing the same issue. Probably because of a plugin.
19:27:08 [Macro Deck] [Info] >> Successfully read configuration file 19:27:08 [Macro Deck] [Info] >> Set language to English 19:27:08 [Macro Deck] [Info] >> VariableManager: Initialize variables database... 19:27:08 [Macro Deck] [Info] >> VariableManager: 45 variables found 19:27:08 [Macro Deck] [Info] >> PluginManager: Loading plugins... 19:27:08 [Macro Deck] [Info] >> Loading plugin HWiNFO64 19:27:08 [Macro Deck] [Info] >> Loading plugin AutoHotkey Plugin 19:27:08 [Macro Deck] [Info] >> Loading plugin KSoft.MediaInfo 19:27:08 [Macro Deck] [Info] >> Loading plugin Macro Deck Media Controls 19:27:08 [Macro Deck] [Info] >> Loading plugin Windows Utils 19:27:08 [Macro Deck] [Info] >> PluginManager: Enabling plugins... 19:27:08 [Macro Deck] [Error] >> MacroDeck: CurrentDomainOnUnhandledException: System.NullReferenceException: Object reference not set to an instance of an object.    at Ize.HWiNFO64_Plugin.HWiNFO64Plugin.Enable() in C:\Users\Manuel\Desktop\Macro Deck Compiler\Temp\Ize.HWiNFO64\Main.cs:line 36    at SuchByte.MacroDeck.Plugins.PluginManager.EnablePlugins() in E:\Repos\Macro-Deck-WinForms\Macro-Deck\Plugins\PluginManager.cs:line 492    at SuchByte.MacroDeck.MacroDeck.Start(Boolean show, Int32 port) in E:\Repos\Macro-Deck-WinForms\Macro-Deck\MacroDeck.cs:line 447    at SuchByte.MacroDeck.MacroDeck.Main(String[] args) in E:\Repos\Macro-Deck-WinForms\Macro-Deck\MacroDeck.cs:line 415
If I delete "Ize.HWiNFO64" folder from "%Appdata%\Macro Deck\plugins" folder, Macro Deck runs once again


Add pyupgrade to pre-commit

https://github.com/asottile/pyupgrade

recheck

Oh, it looks like it conflicts with the change in OGR I did (https://github.com/packit/ogr/pull/468).

recheck


Test header bar - findings count fix

Fixes #3061
Per previous decisions around the active, verified status of findings, and also linked to the enhancement that will be brought by https://github.com/DefectDojo/django-DefectDojo/pull/3029 (showing more details about status findings per test) adjusting the query to display findings summary per severity, only filtering out duplicates from the count.
This way, the overall "Test" head bar will show a count of non-duplicates findings, whatever their status.
Actual details of findings status will be per test line, through the above linked PR.
When submitting a pull request, please make sure you have completed the following checklist:

[x] Give a meaningful name to your PR, as it may end up being used in the release notes.
[x] Your code is flake8 compliant.
[x] Your code is python 3.6 compliant (specific python >3.6 syntax is currently not accepted).
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[x] Add the proper label to categorize your PR.

i don't think I agree. A while ago we made a start to align the definitions of the statuses:

I think these kinds of data should only show active findings. I don't want to see findings from 3 years ago still being counted everywhere.

OK, so how do we convey the fact that there are findings to look at for developers that will see 0, 0, 0, 0?

So minor change, finding must be active, but can be verified or not.


CISv2 - There is no profile name in the ClusterScan CR when no value for scanProfileName is given

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

On an EKS cluster, run CIS v2 scan by deploying a ClusterScan CR with "scanProfileName" = ""
Scan is run and the report is generated.
But the report CR or the ClusterScan CR does not show which scan profile it has run. (View in YAML)

Expected Result:
There should be some field in the report CR or the ClusterScan CR to indicate which profile was run which can be displayed on the UI (easier for user to udnerstand which scan has run )
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - commit id: 8d9cedde931
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): EKS
Kubernetes version (use kubectl version):

1.17

This issue has been fixed and available to test via the new tag:

repository: rancher/cis-operator
tag: v0.0.4
The rancher-cis-benchmark chart has been updated to use this tag. https://github.com/rancher/charts/pull/586

Fix provided:

A new field 'status.lastRunScanProfileName' has been added to the ClusterScan CRD. This will be updated in the status of the  clusterscan after the scan runs. Few additional printer columns have been added to the ClusterScan CRD which will display this field in the kubectl output.

Example: Here is how the clusterscan fields will be displayed that shows the CLUSTERSCANPROFILE used
`
$ kubectl --kubeconfig ~/.kube/test_config_master  get clusterscans
NAME           STATUS   CLUSTERSCANPROFILE       TOTAL   PASS   FAIL   SKIP   NOT APPLICABLE   LASTRUNTIMESTAMP
rke-cis               fail        rke-profile-hardened               92      59         13      0       20                            2020-09-05T01:18:28Z
rke-cis-perm    pass     rke-profile-permissive             92       58         0       0       34                            2020-09-05T01:19:36Z
run-eks-1.0       error
`

Marking this as Review as the CIS v2 chart is still under review.
But can be tested using helm cli:
$ helm install clusterscan-operator ./rancher-cis-benchmark/charts/ --create-namespace=true -n cis-operator-system --kubeconfig ~/.kube/test_config_master

Verified on master-head - commit id: 3ca11da829

Deploy cis-operator from repo - https://github.com/prachidamle/charts.git branch: cis-v2-chart. chart Images:

  cisoperator:
    repository: rancher/cis-operator
    tag: v0.0.4
  securityScan:
    repository: prachidamle/security-scan
    tag: v0.1.20

Deployed an RKE cluster cluster. Run a scan with scanProfileName: ""
When the scan has finished, and status of the scan is Pass, the scan CR has lastRunScanProfileName: rke-profile-permissive


Tightly coupling between steps

It bothers me very much that the result of one step is handled in the next step.
In this way, I can not separate the steps and use them in whatever order I want, or wherever I want.
Step may also include a Handler function (although it does not really fit in with the source code), or StepContext will include lambda to handle the user's response (this can be done right now in a certain way).

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

ID: 04c5adf9-272a-e0f1-10f1-c8365e76035b
Version Independent ID: 1982af41-5a06-4398-0e95-abcc50864e0d
Content: Implement sequential conversation flow - Bot Service
Content Source: articles/v4sdk/bot-builder-dialog-manage-conversation-flow.md
Service: bot-service
GitHub Login: @JonathanFingold
Microsoft Alias: v-jofing

Hi Baruch,
Thanks for your feedback. Yes, this article focuses on the use of WaterfallSteps to control the flow of a conversation. That produces a very structured set of steps. It is equally possible to code your own control logic, evaluate each user input received, and branch control of your conversation based on what you just received. As a bot becomes more complex, this is often how it is coded. The capability is there to do this, it is just not discussed as part of this particular article.

Is there a doc where this custom method is specified?


Fixed exhentai to only load for the galleries

To reduce the unnecessary script loads on exhentai.
Probably would be good to do something like this for most of the sites, but I leave that for some other times.

Good idea. If you see this on more sites you use, feel free to send more pull requests. :)

Thank you!


Failure as NNlib.within_grad is deleted

Flux's downstream tests see a failure with latest NNlib, because this line calls a function which no longer exists:
https://github.com/FluxML/GeometricFlux.jl/blob/c306989f3aeb25d41e0fe00858880caf56d1e3f5/src/operation.jl#L46
Removed in https://github.com/FluxML/NNlib.jl/pull/434. One path would be just to upgrade here. Another would be to make an NNlib PR adding a deprecation for the removed function (which was assumed to be internal); this option would keep old versions of this package working with new NNlib.

@mcabbott Thank you so much for raising this issue!


-a amber -m all throws error

Hi David,
I tried alchemical_analysis.py with the amber test files provided in samples/amber/data
with
python2.7 ../../../alchemical_analysis.py -a amber -p "ti*/ti" -q out -o ~/test
the script runs through and produces the expected output for TI and TI-CUBIC.
if I add the option -m all to the above call, I get the following traceback:
Traceback (most recent call last):
File "../../../alchemical_analysis/alchemical_analysis.py", line 1184, in  main()
File "../../../alchemical_analysis/alchemical_analysis.py", line 1160, in main
Deltaf_ij, dDeltaf_ij = estimatewithMBAR(u_kln, N_k, P.relative_tolerance, regular_estimate=True)
NameError: global name 'u_kln' is not defined
Is this a bug or did I do something wrong? Any help appreciated.
Thanks,
Th.

TI and TI-CUBIC are the only methods for Amber as it does not output the energy differences between adjacent states ∆U_{i, i+1} needed for the FEP methods. So, yes, to some degree it is a bug because the exception catch is missing, but you are not supposed to call '-m all' with Amber.

Hm...maybe Im missing an important point - but in the amber mdout files there are sections starting with "MBAR Energy analysis" which print out what I always thought were the energies of the current configuration at different lamda values - and from this one could calculate the DeltaU{i,i+1} one would need...and indeed I could use the very same mdout files to generate input files for the pymbar routines to get the relative free energies via BAR or MBAR.

Yes, you are right, my bad.
When I played with the output files I was focusing on the "Summary of dvdl values" time series.
Because I didn't find similar time series for the DeltaU{i,i+1} energies, I concluded there wasn't any.
This will be fixed.

Perfect!
Thanks a lot,
Th.
Am 31.07.2015 um 21:15 schrieb FEPanalysis:

Yes, you are right, my bad.
When I played with the output files I was focusing on the "Summary of
dvdl values" time series.
Because I didn't find similar time series for the DeltaU{i,i+1}
energies, I concluded there wasn't any.
This will be fixed.
—
Reply to this email directly or view it on GitHub
https://github.com/MobleyLab/alchemical-analysis/issues/26#issuecomment-126788506.

@davidlmobley Shall we merge Hannes' PR?

I just had an e-mail from him the other day I need to follow up on. Hannes

you still have a little more you want to get in here it sounded like?

Thanks.
On Tue, Dec 1, 2015 at 5:28 PM, FEPanalysis<EMAIL_ADDRESS>wrote:

@davidlmobley https://github.com/davidlmobley Shall we merge Hannes' PR?
—
Reply to this email directly or view it on GitHub
https://github.com/MobleyLab/alchemical-analysis/issues/26#issuecomment-161150286
.

--
David Mobley
<EMAIL_ADDRESS>949-385-2436

But with respect to this issue - is this issue fixed now or is it still to be dealt with in one of your (Hannes' or Pavel) PRs?

Sorry, just found this now.  Is there a way to get automatic updates on issue reports?
I think the MBAR code in the AMBER parser now works fine although I fixed a bug just today, so after the merger.  The bug however did only trigger an error in the parser when there was no MBAR data in the output file.  MBAR data processing should be fine.

OK, so @halx and @foxth - I believe this issue is resolved and will mark it closed. @foxth , please reopen if you have further issues.
@halx - to get automatic updates on issue reports, one way is to just "star" the repository in which case you should get e-mail notifications of everything. There may be more fine-grained controls as well.


MINOR: Covering all epoch cases in add partitions to txn manager

Originally part of https://github.com/apache/kafka/pull/13608, Artem made a good point that this change was unrelated, so I'm making a minor PR to cover it.
Cleaning up the AddPartitionsToTxnManager and covering the 3 epoch cases more clearly.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Test failures appear to be unrelated.

Tests look unrelated. Will merge


Backend response explicit content-type

When content-type is not explicitly set, GAE tries to guess it after response has been produced, before it is sent to client.
Set it explicitly (json, plain text...) and measure performance gain.

Actually, it doesn't seem that GAE "tries to guess". When not explicitly set, GAE sets the header value to "text/html". Performance is unaffected.


Added Remove-AssetMaintenance Function

Created function Remove-AssetMaintenance as per https://snipe-it.readme.io/reference#maintenancesid

@snazy2000 let me know if I did this PR wrong!

Resolves #62


Feat flexovl elster flex 1542

FLEX-1542

SonarQube analysis reported 8 issues:

 8 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 Remove the unused method parameter(s) "deviceResponseHandler,ipAddress,domain,domainVersion,messageType,retryCount,isScheduled".
 Method has 10 parameters, which is greater than 7 authorized.
 Remove the unused method parameter(s) "ipAddress,domain,domainVersion,messageType,retryCount,isScheduled".
 Method has 10 parameters, which is greater than 7 authorized.
 Method has 11 parameters, which is greater than 7 authorized.
 The Cyclomatic Complexity of this method "convertToOslpSchedules" is 11 which is greater than 10 authorized.
 Method has 10 parameters, which is greater than 7 authorized.
 Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

All is well.

Closing this one, because the new code is also in #28


LICENSE

Please insert license.
Thanks you,
Best regards.

(There's an open PR that does include a MIT license.)

Added MIT license file


Remove FilePath

LGTM. It's strange the usages of user.dir on tests but that's clearly not important and you have a lot of PRs waiting for this.

It's strange the usages of user.dir on tests

They weren't really needed, I've stripped them out. Thanks for your time on this!


Fix uuids for multirepo imports

For multirepo imports, we currently use <repo-index>-<locale> as the uuids for sections that were created for repos. Problem is, these often go over the 32 character limit for the uuid field. It would be better to instead create a 32 character hash from the repo index and locale.

Ready for review.

:+1:


My Personal Project Work

Personal Project work
Travel-Agency website

add the issue number to the description.


Change stream and component IDs to Objects

I think it would be a positive change if stream and component IDs became generic Objects, rather than Strings, allowing any objects which implement .equals and .toString to be used.
Moving from integers to strings for IDs was a step in the right direction, however it is still prone to typos, or patterns such as:
static final String MY_STREAM_NAME = "something";
In our systems we declare enums, analogous to:
public static enum Streams {
MY_FIRST_STREAM,
MY_SECOND_STREAM
}
And can then iterate through those streams to declare outputs. This leads to patterns such as:
.shuffleGrouping(MY_BOLT,Bolt.Streams.MY_FIRST_STREAM.toString())
The .toString could be eliminated, and just the plain enum passed in.

This is an interesting idea. However, I think it makes makes the API confusing since you may want to read the stream id later on and you'd get back a string instead of an object. The topology structure for Storm is defined in Thrift in order to support non-Java languages, which precludes the use of arbitrary Java objects for the stream ids.
Something like this could belong in a Java-focused higher level abstraction on top of Storm.


Make rule matching when a tab gets activated optional

Currently FoF matches rules on the following occations:

When the page is reloaded:

DOMContentLoaded
load

When the user changes the tab

Make "changing to tab"  via settings and rule property.

Will be in 3.0.0


Clarified behavior of single-valued Monads

Fixes #892

Current coverage is 90.43%

Merging #920 into master will increase coverage by +0.01% as of dbe88d4

@@            master    #920   diff @@
======================================
  Files           81      81
  Stmts         8702    8702
  Branches      1725    1725
  Methods          0       0
======================================
+ Hit           7869    7870     +1
  Partial        215     215
+ Missed         618     617     -1

Review entire Coverage Diff as of dbe88d4

Powered by Codecov. Updated on successful CI builds.


How can I get the original file?

In my app , I'm downloading some pictures from the Internet and I noticed a strange behaviour on iOS that I'd like to understand.
Example:
When I download a picture, I choose to save it in the app folder with a filename MVIMG_20200306_095821.jpg that is stored in /Users/leo/Library/Developer/CoreSimulator/Devices/0B4E2BDF-5800-41A0-9AB9-FC0A2598C960/data/Containers/Data/Application/E93201C9-0DF2-4EFA-A15A-EF22A0444536/Documents/MVIMG_20200306_095821.jpg
When I load the recent list pic from your library, the corrispondend AssetEntity of that downloaded picture has:

title: null
titleAsync: IMG_0035.JPG
path of file: /Users/leo/Library/Developer/CoreSimulator/Devices/0B4E2BDF-5800-41A0-9AB9-FC0A2598C960/data/Containers/Data/Application/E93201C9-0DF2-4EFA-A15A-EF22A0444536/tmp/.image/IMG_0035.JPG
path of originFile: /Users/leo/Library/Developer/CoreSimulator/Devices/0B4E2BDF-5800-41A0-9AB9-FC0A2598C960/data/Containers/Data/Application/E93201C9-0DF2-4EFA-A15A-EF22A0444536/tmp/.image/IMG_0035.JPG

I suppose the file returned is the thumbnail file stored in temp, isn't it?
But then the question is: how can I actually get the original filename?
I need a parameter to actually link the two files (the downloaded and the one got from your lib): the filenames, the modified datetime are different and the creation datetime is not available on the File object in Dart.
Thanks

iOS and android are not the same, each of its apps runs in a sandbox with a separate file system, and you can't really get the original files stored in the system.
When you save, the system will 'automatically' assign a filename to your stored resources.
If you need to manage the files you download, you should use your own data storage system, or a simple json stored in SharedPreferences, or sqlite, hive or other databases.
If you need to associate an asset with a file you downloaded, you can use the asset's id to associate the foreign key.
The id of AssetEntity is the id of the item stored in the MediaStore in Android and the localIdentity of PHAsset in iOS. It can be associated as a unique association primary key.
You can design your database in the following way, but that's just a suggestion:

key_name
description

id
download id , primary key

create_dt
create date

path
file path

download_url
url

size
size

asset_id
id of AssetEntity

more columns...

If you need to associate an asset with a file you downloaded, you can use the asset's id to associate the foreign key.

That's what I need to do. I don't understand what is the foreign key you are referring to. I have this problem only on iOS bc on Android the title fo the AssetEntity is the filename of the downloaded file.
If my app is downloading 10 pictures in parallel while I'm taking other 10 from my camera, I'll get 20 new items from the function PhotoManager.getAssetPathList().
How do I recognize the ones downloaded vs the one took from the camera? And then, if the ones downloaded are not in order because of the connection or the size of the file, how do I link each one of them to the corresponding downloaded file?
Thanks

https://en.wikipedia.org/wiki/Foreign_key
Create your own relational database to maintain all downloaded data.
To distinguish whether a file is downloaded or not is to go to the database and find out if the id is there.
I can only give you one idea, the exact coding is up to you.

I know what is a foreign key but I don't understand your way to use it.
Are you saying that the ID of the file I download and the ID of the file you display are the same? also if their paths are different?

I mean: you need to maintain the correspondence between download and photo_manager yourself.

Closing without further responses.


Multiple integration support

Goal
Currently, the X-Ray daemon sends data to the AWS X-Ray service. This issue discusses the changes to be implemented to the existing design that supports multiple backends apart from the X-Ray service
Current design
The X-Ray daemon receives segments on the X-Ray daemon address. Each received segment has a daemon header. The current design utilizes a global memory pool known as buffer pool, (preallocated on initialization, default 1% of total memory) for receiving the UDP payload. A Ring buffer (RB) is a structure implemented using a channel and stores received segments using a goroutine. The size of the RB is 250 segments and each segment in the RB maintains a pointer to a piece of buffer allocated in the buffer pool. By default the buffer size is 64KB and we do not split large payload into multiple buffers. A Processor is on the receiver end of this RB channel and batches segments using a goroutine. A batch is ready to be sent by the processor to a Batch Processor, if it is large enough (default: 50 segments) or the processor goroutine has hit an idle timeout (default: 1 second), upon which the raw payload for the batch is serialized to strings and the buffer is returned to the buffer pool for reuse. The batch processor uses X-Ray client and sends batches to the X-Ray service using the PutTraceSegments API.
Modularization
We intend to decouple components of the X-Ray daemon, so the segments batched by the X-Ray daemon can be routed to the desired backend service. The changes to the design are backward compatible and support the X-Ray service by default.
Client
We create a X-Ray client instance to use the PutTraceSegments API that sends data to the X-Ray service. The X-Ray client implements XRay interface which contains X-Ray service API methods. We will have another interface Service (name yet to be finalized) which contains PutSegments() method. A Client structure will implement the Service interface for the desired backend service. The Client will be a bridge between the X-Ray daemon and the backend service.
Registering Client
In the current design,  during initialization of the Processor instance, the X-Ray client is created and set to the Batch Processor instance. When the batch of segments is ready to be sent, the Batch Processor instance uses the X-Ray client to send data to the X-Ray service. This part needs to be restructured and the Client/ X-Ray client will be created as a part of daemon initialization and passed to the Processor instance. Once the Batch Processor instance is configured with the Client/ X-Ray client, existing architecture will send the batch of segments to the configured backend service.
Note : These are initial thoughts on modularizing the X-Ray daemon. Your suggestions are welcome.

One suggestion:

Maybe a good name for Service would be Reporter because indeed it reports data to the backend.

One question:
- Will the agent do any sort of transformation with received data or will deliver data to the Service interface in the same way it receives it? The current code does nothing with data more than sending it directly to the backend, I think this is something that could be mentioned here.
Otherwise, great initiative @yogiraj07.

Hi @jcchavezs ,
Thank you for the feedback.
I think, we should deliver data to Service interface (name not yet finalized), in the same way we receive it.  Let the implementer of the interface handle the desired transformations.
However, this decision is also based on what kind of transformations, user expects and at what point of time in the pipeline. For example, is it during batching segments, or once the batch is ready to be sent, we do transformation on the batch.
Please let us know your motivation for the transformation on received data.
Best,
Yogi

I was more asking from the aws side, if we deliver data same way we receive then it will be easier to start using the agent with different formats :+1:. The no-transformation also opens the possibility for other encodings like msgpack for example.
The only transformation I can think of is the joining of jsons to report a batch over http. Let say you receive: [{"key":"value1", ...}, {"key":"value2", ...}] and in a second moment [{"key":"value3", ...}, {"key":"value4", ...}], you will most likely join them together and send to the server as [{"key":"value1", ...}, {"key":"value2", ...}, {"key":"value3", ...}, {"key":"value4", ...}] but that sort of combination could be left to the Service implementation.
Other sort of transformation could be dropping segments or traces based on different criterias (for example on firehose mode), let's say you only want to send traces with an error or with longer durations than a certain number, again that should be done in Service implementation.

Hi @jcchavezs ,
Can you please help me understand the following statements:
1)"Using other encoding formats like msgpack"
Do you mean, a possible transformation of received segments to msgpack format by the implementer of Service interface. For example, the following flow :
Received segment -> service interface -> transform to msgpack format -> send to desired backend (other than X-Ray service)
2)"Using agent with different formats"
Is the input to X-Ray daemon in different format, or the output?
Or is this the same concept covered in point 1 or you intended to say something else.
Please let me know if I am missing anything.
And for other mentioned transformations, we can let the implementer of Service interface decide.
Thanks,
Yogi

1)"Using other encoding formats like msgpack"
Do you mean, a possible transformation of received segments to msgpack format by the implementer of Service interface. For example, the following flow
Received segment -> service interface -> transform to msgpack format -> send to desired backend (other than X-Ray service)

Exactly that.

2)"Using agent with different formats"

If the agent might do some sort of validations instead of send what it receives it won't allow users to send data as msgpack or any other format. This is not so important.

Since the OpenTelemetry Collector is better suited for this goal of sending traces to different backends, and it already has the awsxrayreceiver and awsxrayexporter, closing this issue. OpenTelemetry Collector is a good solution for user wanting to export the data to other backends.


Encryption at rest and data flows unencrypted unclear

The comparison table says that the "Data flows encrypted between Compute and Storage" is not applied with the first 2 examples ("Encryption at rest")
However does this mean that the storage service (which is an HTTPS service) is utilizing HTTP? which seems very strange?
There must be Transport level encryption being applied.

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

ID: b964cc95-9074-55f1-c150-f74933aa9222
Version Independent ID: 90d48cea-1b05-d320-f939-179cd5e6b195
Content: Overview of managed disk encryption options - Azure Virtual Machines
Content Source: articles/virtual-machines/disk-encryption-overview.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@Oppedijk Thanks for the feedback! We are currently investigating and will update you shortly.

@Oppedijk Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@msmbaldwin Any update on this?

Please take a look at these descriptions, as these are the base for Security Standards (like CIS) which might rely on this wrong data

Can we get an update on this, please? We are 6 months further now
@AshokPeddakotla-MSFT @SumanthMarigowda @msmbaldwin

Any update on this? We are 9 months further now.
@AshokPeddakotla-MSFT @SumanthMarigowda @msmbaldwin

@Oppedijk Thanks for reaching out. I'm currently checking with the product team. I'll let you know what they come back with. :)

Thank you for your dedication to our documentation.
We sincerely apologize for the delayed response. After a careful review, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close

Thanks for your dedication to our documentation.
We sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary due to various constraints. Therefore, we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


Timeout option when specified breaks werf converge upgrade process with deadline exceeded

Check default timeout: should be infinite, got:
Waiting for helm hook job termination (300.00 seconds) FAILED

Fixed by the https://github.com/werf/werf/pull/3065

Fixed by the https://github.com/werf/werf/pull/3065


New line indents / formatting

Hi there.
First of all, thanks for the amazing extension, which we really love as an alternative for the ever-buggy FlashDevelop for our as3 projects.
We do suffer from a big bug, for which I'm unsure whether is fixable through your extension, or is VS Code related altogether.
The indents for new lines are not respected to be in class-level indentaton. That is, new line after a method or field/variable will always be positioned on location 0, instead of 2 tab indents in, on the level where all fields and methods live.
This causes many false-positive conflicts and file-changes when file is formatted in Vs Code versus FlashDevelop or Notepad++ for that matter.
It would be extremely helpful to have this fixed. Looking forward to at least a feedback, if this cannot be solved on extension level.
Thanks, Marko

Adding indentation when inserting new lines is definitely a behavior that is controlled by VSCode, and not by the extension.
However, what's strange is that I cannot reproduce the issue that you describe.
To test, I loaded the following ActionScript class in VSCode:
package
{
	import flash.display.Sprite;

	public class HelloWorld extends Sprite
	{
		public function HelloWorld()
		{
		}
	}
}

If I place the cursor after the constructor's } character, and I press the enter key to insert a new line, the new line is intended by two tabs. If I convert the file to spaces instead of tabs, it inserts the appropriate number of spaces instead.

Hi, Josh. Thanks for looking into this and replying so rapidly!
I see now that I completely forgot to mention and add the core - important part of the issue! I apologize for that!
The indent issue appears when doing "Refactor" command, causing all lines (even empty lines in between the empty method, if you'd add an empty line inside your "HelloWorld()" method).
Also, what's working even strangely and sporadicaly is, pressing Ctrl+S / Saving the document, seems to remove indents on empty lines above the current cursor. That is, making 3 new lines after a method, all keep indents. Saving the file causes the first two lines to lose indent.
I hope this is easier to understand and reproduce now. Once again, apologizes for missing the important bit in the original post!
Best regards, Marko

Edit: (edit button doesnt work for some reason on Edge browser...)
The latest seems to not be affected by save, but new lines in general. After every newline added it seems that previous ones lose indents...

After every newline added it seems that previous ones lose indents...

Yes, I see this behavior too. That's definitely controlled by VSCode. You should check to see if there is a setting that tells VSCode to preserve indentation on empty lines. Microsoft is pretty good about making little things like that configurable.


Create custom width property for flyout

Sometimes design specifications could require the Flyout to extend pass the five standardized sizes. Because the flyout has no method for extending beyond the size, this PR adds a width property to the Flyout component in order to allow the flyout to have custom sizes for when the Flyout must be a different size other than the standardized sizes.
Pros

allows for flexibility in designing components that use the fly-out
avoids having to add extra sizes for larger fly-outs (i.e. xxl, xxxl, xxxxl)

Cons

less consistency in fly-outs if developers and engineers choose not to use the standardized sizes

Hey @JediMasterRemington - this looks good. However, I think having both a width & a size property might be a little bit confusing. What would happen if somebody did the following?
<Flyout size="sm" width={800} ... />

Could you perhaps merge both properties together so that size also takes a number?

I've made it so size can now take a number


Feature request: send arbitrary R code transformations to the spark cluster?

I understand that's possible via library(sparkR). I also saw some info that spark_apply can do it, but does the latter accept any generic R-function?

@carioca67 Yes, it supported generic R-functions and even the use of packages and groups. Please take a look at https://spark.rstudio.com/articles/guides-distributed-r.html. would appreciate any feedback you might have on this article, thanks!

Thanks Javier!
On Fri, Jul 28, 2017 at 5:15 PM, Javier Luraschi<EMAIL_ADDRESS>wrote:

@carioca67 https://github.com/carioca67 Yes, it supported generic
R-functions and even the use of packages and groups. Please take a look at
https://spark.rstudio.com/articles/guides-distributed-r.html. would
appreciate any feedback you might have on this article, thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rstudio/sparklyr/issues/847#issuecomment-318751544,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APeqllg_CPJ_9cvn2wFTtXLsbdVeG188ks5sSkFpgaJpZM4OYxgt
.


build: add node v14

oh I see you've made one. I did not change any setting to create any approval condition. I'll delete the other branch I created.


TOS Page 404

The Bug
https://azuriom.com/en/terms page isn't working.
you can't access to read the TOS 📏

Hi, this page has been removed, normally all references to this page have also been removed. About the TOS, you can take a look at the GNU GPL v3.0 license :)


Change type Action to AnyAction

Thanks for sharing the light weight library what i searching for.
I'd like to offer to change type definition Action to AnyAction
export type ReduxListener = (dispatch: Dispatch, action: Action) => void;

Action has only 'type' property so if you have 'payload' you should do type casting Action to unknown (or AnyAction) to PlayloadAction
    listenMiddleware.addListener(secondStepDone({}).type, (dispatch, action: unknown) => {
      const data = action as PayloadAction<{ regionList: Region[] }>
      startProcess(data.payload.regionList)
    })

If i got action as AnyAction then all i just do is converting AnyAction to PayloadAction at parameter
    listenMiddleware.addListener(secondStepDone({}).type, (dispatch, action: PayloadAction<{ regionList: Region[] }>) => {
      startProcess(action.payload.regionList)
    })

If it's ok i'll make PR


Password Validator only validates for empty password

Describe the bug
Password validator only validates for empty password on signup page.
To Reproduce
Steps to reproduce the behavior:

Go to 'Signup page'
Click on 'enter a small password'
See error

Expected behavior
It should show "password is too short " when the length of password is short.
Screenshots

Are you working on this issue? (Yes/No)
Yes

Please assign it to me. I am working on it.


feat: Add support for edited events to run the action on every PR edit

Hi,
we found this action doing exactly what we wanted, but need support for PR edits like title changes. We tried this changed action and it worked, maybe someone else needs this too.
Closes #117

@Namchee the workflow references to an image (version) which doesn't exist. Is there any way to get my pull request accepted or rejected? Of course, we can also fork the project, but we would like to participate instead of maintaining a side fork

@TheGoderGuy, thanks for the PR!
I just noticed that it lacks edited event as this event is rarely fired for my personal use case 😅
As for the workflow run, I think I messed up the workflow somewhere in one of my old PR. I will fix this ASAP.

Alright, there we go! I guess I'm gonna try to edit this PR to mention #117. If self-test passes, I'll merge this.

Alright, looks good to me!

@all-contributors, please add @TheGoderGuy for code.

@all-contributors please add @TheGoderGuy for code


chore(master): release 1.1.67

:robot: I have created a release beep boop
1.1.67 (2023-09-04)
Miscellaneous

deps: update dependency phpstan/phpstan to v1.10.33 (d1e7ad4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.67 :sunflower:


Fix a failing test

This poor poor test was left behind at some point when [ConditionalTheory] broke and the baseline file got out of date.
Now that we've largely fixed this it's running again and failing since it wasn't updated when we made changes to the codegen.
I changed this to [ConditionalFact] as it's not a theory.

FYI only


Draft page revisions no longer copied in Wagtail 4+

Issue Summary
Page copy behavior changed in an unexpected way between Wagtail 3.x and 4.x. I'm opening this issue because this change doesn't seem to be documented anywhere, and so I'm not sure if it was intentional or inadvertent.
In Wagtail 3.x and below, copying a page creates a new page copy from the latest revision of the copied page. In Wagtail 4+, copying a page creates a new page copy from the latest published revision of the copied page (or, the first revision if the page was never published). This effectively means that there's no way to copy draft revisions in Wagtail 4+.
This change significantly impacts the workflow of Wagtail content editors who assume that copying a page will use its latest revision. On Wagtail 4+, editors can edit a page, be looking at an edit view with some latest unpublished content, copy the page they are looking at, and find that the page copy is unexpectedly for some older published version. There's no indication to editors that they aren't copying the latest version. Editors who work with a "save early and often" strategy who want to copy a page before publishing for the first time will find that all their saves except the first one are get lost when the page gets copied.
Steps to Reproduce
The easiest way to demonstrate how the behavior has changed over time is to run through these steps starting with Wagtail 3 and then trying with more recent versions:

Create a new virtual environment and pip install "wagtail=>3,<4" to install Wagtail 3.0.3.

To test more recent versions after going through these steps with Wagtail 3, try pip install "wagtail>=4,<5" and pip install wagtail==5.0rc1, making sure to ./manage.py migrate to update models after upgrading.

wagtail start myproject && cd myproject
Edit home/models.py as follows:from wagtail.admin.panels import FieldPanel
from wagtail.fields import RichTextField
from wagtail.models import Page

class HomePage(Page):
    body = RichTextField(blank=True)

    content_panels = Page.content_panels + [
        FieldPanel("body"),
     ]

Edit home/templates/home/home_page.html as follows:{% load wagtailcore_tags %}
{{ page.body | richtext }}

./manage.py makemigrations && ./manage.py migrate && ./manage.py createsuperuser
./manage.py runserver and navigate to http://localhost:8000/admin/, logging in with the superuser you just created.

Let's see what happens when we copy a page that's never been published but has multiple revisions.

Add a new HomePage, give it some title and set body content to "A", and then click "Save draft".
Now change the body content to "B" and click "Save draft" again.
In the three-dot menu in the header, click "Copy". Give the page a new title and slug and click "Copy this page".
Edit the page you just copied and note that, as expected, page content says "B" -- the copied page got the most recent revision of the page. If you go to the history of the copied page, you only see events for "Created" and "Copied from Home". You don't see the revision history.

On Wagtail 4+, the content says "A" instead - the first revision was copied, not the most recent one.

Let's see what happens when we copy a page that has been published and has a new unpublished revision.

 Change body content to "C" and select "Publish" from the footer menu.
Go back to editing that page again. Now change body content to "D" and click "Save draft".
In the three-dot menu in the header, click "Copy". Note that the copy page offers a checkbox "Publish copied page: This page is live. Would you like to publish its copy as well?". This is accurate; the page is live, but it has an unpublished draft. There's no indication here which content is going to get copied or published. Leave the checkbox unpublished for now. Give the page a new title and slug and click "Copy this page".
Edit the page you just copied and note that the content says "D" - again the latest revision was copied, not the published version. If you look at the history of the copied page, you only see events for "Created" and "Copied from Home". So to the editor it appears that the most recent revision was copied and there's no history before that, i.e. all previous revisions are unrecoverable.

On Wagtail4+, the content says "C" instead - the published live revision was copied, not the most recent draft. There was no indication to the user that this was going to happen.

What if we try to publish the page when we copy it?

Go back and copy your original page again (still saying "D" with an unpublished revision) but this time check the "Publish copied page" checkbox.
Edit the page you just copied and note that the content says "D" - again the latest revision was copied, not the published version. If you click on the "LIVE" link at the top-right and view the page, it says "C" - so the page draft version and live versions both exist. And indeed browsing the tree I see both "View draft" and "View live" links. But, looking at the page history, I can't see or recover those different versions. And in fact there's no way to edit the currently-live version of the page.

On Wagtail4+, the content again says "C" instead - the published live revision was copied, not the most recent draft. The page still has "View draft" and "View live" links, and is still in "LIVE + DRAFT" mode, but both versions say "C" - the "D" content is seemingly lost to the editor.

Technical Explanation
This change in behavior was introduced in #8482; since it is not mentioned on that PR as a deliberate choice, I suspect this was unintentional.
The specific issue is this change. Prior to this PR, Page.get_latest_revision() would query the list of page revisions in the DB and return the latest one. After this PR, the page's Page.latest_revision attribute is returned instead. The logic tries to keep that attribute updated when a new page revision is saved but, crucially, its value is not copied when the page is and the page copy logic doesn't set it.
The page copy logic does copy revisions, although there's no way for a page editor to access them, because the action log doesn't get populated with them. If you have access to the database and can look up the ID of the copied revisions, it is possible to compare and recover old revisions (for example), but regular Wagtail users can't do this.
Discussion
One fix to restore the old behavior would be to have the page copy logic set Page.latest_revision based on the most recent revision that was copied with the page. Additionally, the page action log could be populated with each old revision that got copied.
Whether these changes make sense depend on the answers to these questions:

What is the intended behavior when copying pages: should the latest revision be copied, or the latest published revision?
Should page revisions be copied when pages are, and, if so, for what purpose and with what visibility to the user?

My two cents, based on feedback from our content editors, is that the expectation is that copying a page should always copy the most recent page revision. If for some reason only the most recent live revision is copied, then at a minimum the copy view should warn the user about this. Or, perhaps that view could have a checkbox asking the user which version they want to copy.
For pages that have never been published, I think the most recent revision should always be copied. (I see https://github.com/wagtail/wagtail/pull/7755 which touches on this a bit and includes a comment suggesting that copying for translation might necessitate some different logic; I think that even in the translation use case you might still want to copy/translate unpublished content, for example to schedule simultaneous publishing of multiple languages. I'm less familiar with translation workflows so wonder whether my proposed fix above might break these somehow.)
I also tend to think that the revision history should continue to be copied over but that it should be logged and visible to the user. (If it's not visible or useable in any way, why is it being copied?) I wonder if there might be some complications here, though, for example: would it be confusing to have entries in the log that didn't actually happen. Perhaps the log event messages/author could be appropriate modified to reflect that these revisions were copied.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.11.1
Django version: 4.0.10
Wagtail version: 4.x, 5.0rc1

As a content editor, I wanted to add my two cents here if it's valuable. (If this issue is just an unintended bug, then what I'm saying here is largely moot, except as an explanation of need for a fix.)
There are a lot of cases where it makes sense to copy and re-copy drafts in progress, especially on larger websites. For one example, a project for a new section of a website may include building multiple child pages with a roughly parallel structure and sequence of blocks, easiest to replicate via a copy -- but that structure may also be complex enough to necessitate frequent saving before the draft is "ready," whether to ensure no loss of ongoing work or to pick up the next business day after the drafting has begun. While content is still in an experimental/prototype phase, that parallel structure may be tweaked, and again it may be easiest (and most consistent) to re-copy a source draft rather than replicate those tweaks manually across multiple pages.
As a content editor, I tend to lose track of the "initial revision/publication" version of a given page almost immediately, because in order to see that instead of the latest draft, I have to go through a page's version history and find the revision in question. It is worth emphasizing that this means that, if I select "Edit this page" or find a specific page and bring up the corresponding editor, what I will always see by default is the latest revision, whether published or unpublished. Having a button associated with that page labeled "Copy" thus creates an intuitive linear expectation for a content editor: that I'm going to copy the latest revision, whether it's published or unpublished.

Thanks for the report @chosak (and @csebianlander for the supporting info)! Have verified that I'm seeing the same behaviour from 4.0.x onward - this is definitely an unintended bug.
I think the problem may be due to the wagtail.actions.copy_page logic not being updated when the latest_revision field was introduced - latest_revision is listed in Page.default_exclude_fields_in_copy which means that it will be blanked in the copied page. After copying the revision records across, it creates a new revision which would previously have been based on the latest revision as determined by timestamp, but now takes it from the latest_revision field of the copy, which is blank here. As a result, it falls back on the real database record (which corresponds to the published revision if the page is live, or the very first version if not).
Working on a test case and fix for this now. (cc @laymonage)


Update dependencies

Updates package dependencies to fix (~800) vulnerabilities.

This PR also includes 2 minor fixes:

f39bdc7100de0c985aa3975b675dcd6ace3d9590 also fixes https://github.com/hellofresh/janus-dashboard/issues/313.
Fixes incorrect schema (trust_forwarded_headers should be trust_forward_header)

👍

+1

+1


File globbing

Fixes #126
@isaacs PTAL

Thanks!  That was definitely an oversight.  Landed and pushed to 1.0.3.


record call for live api

Can you add documentation for record calls for live api?

@vlad-borsh each ng-apimock client has the function recordRequests, which enables ng-apimock to record all api calls.
You can get the recordings using the function getRecordings or through the dev-interface recordings tab


fix jeff33 chain generation

As mentioned here, the generate_jeff_chain script fails when loading the NFY information from JEFF-33's file available at this adress
The reason for the failure is that this file is malformed and lacks a proper header line. This PR adds a few lines in the generate_jeff_chain script to fix the file before loading neutron fission yields.

Awesome thanks for this fix. I've triggered the tests


Attempt to load module index/hello

I just recently installed transcrypt on macOS Sierra using python3.5 and similarly to #115 I get this error when trying to build (with transcrypt -b) the demo hello.py (the one with the solar system - source code here
Error in program /usr/local/lib/python3.5/site-packages/transcrypt/demos/hello/hello.py, module hello:
    Attempt to load module: hello
    Can't find any of:
    ...
    # Looking for '__javascript__/hello.mod.js' at different locations

When I try to just run it (with transcrypt -r) I get another error:
Error: cannot run /usr/local/lib/python3.5/site-packages/transcrypt/demos/hello/hello.py using CPython: name 'chain' is not defined

Aborted

When I copy the source code from the example into a new file at another location I get yet another error using transcrypt -b:
Error in program what_should_i_eat/static/what_should_i_eat/js/index.py, module index:
    Attempt to load module: index
    Can't find any of:
    ...
    # Looking for '__javascript__/index.mod.js' at different locations

Using transscrypt -r with the newly created copy of hello.py outputs the same error as above ('chain' is not defined).
So my questions are:

Why does the demo not work? Why is transcrypt looking for a module file when there is no import other than itertools (which is built-in)?
How can the same source code produce different errors (when building)?

I would love to get this to work - pretty excited about using python for the browser. :)
Thanks is advance.

I'm on Mac OS X 10.11.6 so I can't test macos Sierra, but I don't think it should matter. Are you building from the transcrypt/demo/hello directory ? I can get the error message you do if I build form another directory. For example:
$> cd transcrypt
$> transcrypt demo/hello/hello.py
Transcrypt (TM) Python to JavaScript Small Sane Subset Transpiler Version 3.6.2
Copyright (C) Geatec Engineering. License: Apache 2.0

Error in program hello/hello.py, module hello:
	Attempt to load module: hello
	Can't find any of:
	/Users/.../transcrypt/Transcryptmaster/transcrypt/demos/hello/__javascript__/__init__.mod.js
        <More Madness>

But if I run from the hello directory it works:
$> cd transcrypt/demo/hello
$> transcrypt hello.py
(tspy) grumpy-2:hello callendorph$ transcrypt hello.py
Transcrypt (TM) Python to JavaScript Small Sane Subset Transpiler Version 3.6.2
Copyright (C) Geatec Engineering. License: Apache 2.0

Saving result in: /Users/.../transcrypt/Transcrypt-master/transcrypt/demos/hello/__javascript__/hello.js
Saving minified result in: /Users/.../transcrypt/Transcrypt-master/transcrypt/demos/hello/__javascript__/hello.min.js

Ready

I would definitely agree that this is not optimal but I'm not familiar enough yet to know what the complications are.

I just recently installed transcrypt on macOS Sierra using python3.5 and similarly to #115 I get this error when trying to build

Issue #115 is a little different actually. That issue is trying to build a script that uses the sys module. Currently, this project doesn't have a sys module yet. It can't possibly build. The hello script should definitely build.

Thanks for your fast reply. Your description also works for me, so thanks! :)
So does this principle always work? I mean do I always have to cd into the directory that (directly) contains the file I want to compile? That would make it more difficult to compile a whole bunch of files in different directories...
Or is the reason it worked that you cd'ed into the package directory?

Well - I know that the unit tests compile a bunch of sub-modules so I there is definitely a structured way to handle code in different files and directories. I'm pretty sure that the compiler traces through the imports and compiles as it goes even if they are separate directories. I think the compiled javascript for each module ends up in created subdirectory javascript in those separate directories.

Or is the reason it worked that you cd'ed into the package directory?

No I think it is because you were in the same directory as the "hello.py" file. The compiler compiles the hello.py file into a JS module in javascript/hello.mod.js. Then when it makes the top-level file that includes the transcrypt framework, it tries to include the hello.mod.js file from there. But because you aren't in the same directory - it can't find the module it just created.
I may have missed some details - @JdeH anything you want to add ?

Alright. Then I'm gonna play around with it a little and see how far I get.

Currently Transcrypt must be run from the directory that the main program is in.
Would be better if it could be run from anywhere.

For now labeled as postponed (see issue #258 for what this means)


Tests failing on hashed URL

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
ui5-test-runner --url https://ui5.sap.com/test-resources/sap/m/demokit/orderbrowser/webapp/test/integration/opaTests.qunit.html#
Expected behavior
Works but many errors are displayed (because of the trailing #)
Report
If possible, adds  report.html
Desktop (please complete the following information):
N/A
Additional context
Easy to reproduce

4.3.2


Finalize index.ts and export it in package.json

Currently, Tact API users must write imports in the following format:
import { SrcInfo } from "@tact-lang/compiler/dist/grammar/ast";

It would be more convenient if they could write one of the following:
import { SrcInfo } from "@tact-lang/compiler";
import { SrcInfo } from "@tact-lang/compiler/grammar";

To achieve this, we should finalize the exports in the index.ts files and add them to package.json. Here is an example:
  "exports": {
    ".": {
      "import": "./dist/src/index.js",
      "require": "./dist/src/index.js"
    },
    "./*": "./dist/src/*"
  },

Related: https://github.com/tact-lang/tact/issues/740 and https://github.com/tact-lang/tact/issues/543

and add them to package.json.

Somehow @ton/ton-core works without this step:

this is their index.ts
there is no exports section in their package.json

They somehow put index.js and index.d.js to the dist folder instead dist/src: https://www.npmjs.com/package/@ton/core?activeTab=code

In Tact, everything will work as expected. npm will place src/index.ts in dist/index.ts, with no additional actions required.
I encountered issues in Misti and Souffle.js because I included more than just src in my tsconfig.ts:
  "include": [
    "src/**/*",
    "test/**/*",
    "examples/**/*"
  ],

As a result, npm had to maintain that directory structure in dist, creating dist/{src,test,examples} directories.
Additionally, the TypeScript compiler checks if you import package.json in your code. If you do, it will generate the following structure: dist/{package.json,src}.


Development tests are broken

Working to reconstruct development and return to working condition.

Fixed by https://github.com/NREL/bifacial_radiance/pull/285/commits/9add655992f0f97e623de279bf3b2b9c8089d263


Fix local address remove

Fixed local interface address route remove by loading the address list for the interface and adding to all_allowed_ips
Using the logfile option for the FileHandler

Thanks for the contribution.
The logfile and fe80 fixes are good, I will merge them if you submit them separately.
For the other change, if I understand correctly, it fixes the case where you have IP addresses assigned on the wg interface, and remove_orphan_routes will remove the kernel routes in that case. We run wg without any IP address on the wg interface, so we don't run into this problem, but I agree it should be fixed. I have a few questions:

can you explain how your code is supposed to work? If <IP_ADDRESS>/24 is assigned to your wg interface, your code would add <IP_ADDRESS>/24 to all_allowed_ips. But the existing code would still remove the corresponding kernel route <IP_ADDRESS>/24, because we do a simple string comparison.
a simpler solution would be to just ignore routes that have proto kernel when listing routes, would this work for you?
as a last resort, if your solution is unavoidable, please refactor listing addresses in a separate function, similar to get_kernel_routes

Ok for the logfile and fe80
You are right for the fix, I was testing it with a /128 ipv6 but it does not work with a lower prefix. I will check how to filter proto kernel

Completed in https://github.com/zorun/wg-autoroute/pull/4 as discussed


Add this fix for sky island lod plz

https://www.reddit.com/r/NewYuzuPiracy/comments/13uath8/totk_sky_island_lod_improvement_mods_v111_test/
Looks great.

Already added ^^

Sorry that I didn't recognize them. You are doing really good man!


Redux bindActionCreators does not work with 3.2 (ActionType is broken?)

Description
Since typesafe-actions 3.2, some typing error has appeared that means Redux's bindActionCreators does not work in some cases.
export type ColorCycleAction = ActionType<typeof colorCycle>;
// ColorCycleAction is 'any'

Steps to Reproduce

Clone https://gitgud.io/nixx/twitch-extension-buzzword-boilerplate.git
cd twitch-frontend
npm install
npm run build -> this works
npm install<EMAIL_ADDRESS>npm run build -> error

Type error: Type 'PayloadAC<ColorCycleActionTypes.CYCLE, void>' is not assignable to type '(event: MouseEvent<HTMLButtonElement, MouseEvent>) => void'.
  Types of parameters 'payload' and 'event' are incompatible.
    Type 'MouseEvent<HTMLButtonElement, MouseEvent>' is not assignable to type 'void'.  TS2322

The typesafe-actions in that project is 3.0, but I've verified in a private project that the issue was introduced after 3.1.
You can notice a bunch of issues pop up, but I think the source of the problem is the ActionType thing.
In my private project, for example, it is bindActionCreators that blows up.
const mapDispatchToProps = (dispatch: Dispatch<RootAction>) => bindActionCreators(
  {
    openFile: fileActions.openFileFromInput,
    openFileFromDisk: fileActions.openFileFromDisk,
    importFile: importComponentsActions.loadFromDragAndDrop,
  },
  dispatch, // Argument of type 'Dispatch<RootAction>' is not assignable to parameter of type 'Dispatch<AnyAction>'.
);

Thanks for a nice report. I'll look into this later today.
On Fri, Mar 1, 2019, 12:07 PM nixxquality<EMAIL_ADDRESS>wrote:

Description
Since typesafe-actions 3.2, some typing error has appeared that means
Redux's bindActionCreators does not work in some cases.
export type ColorCycleAction = ActionType;// ColorCycleAction is 'any'
Steps to Reproduce

Clone https://gitgud.io/nixx/twitch-extension-buzzword-boilerplate
cd twitch-frontend
npm install
npm run build -> this works
npm install<EMAIL_ADDRESS>npm run build -> error

Type error: Type 'PayloadAC<ColorCycleActionTypes.CYCLE, void>' is not assignable to type '(event: MouseEvent<HTMLButtonElement, MouseEvent>) => void'.
Types of parameters 'payload' and 'event' are incompatible.
Type 'MouseEvent<HTMLButtonElement, MouseEvent>' is not assignable to type 'void'.  TS2322
The typesafe-actions in that project is 3.0, but I've verified in a
private project that the issue was introduced after 3.1.
You can notice a bunch of issues pop up, but I think the source of the
problem is the ActionType thing.
In my private project, for example, it is bindActionCreators that blows up.
const mapDispatchToProps = (dispatch: Dispatch) => bindActionCreators(
{
openFile: fileActions.openFileFromInput,
openFileFromDisk: fileActions.openFileFromDisk,
importFile: importComponentsActions.loadFromDragAndDrop,
},
dispatch, // Argument of type 'Dispatch' is not assignable to parameter of type 'Dispatch'.
);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/piotrwitek/typesafe-actions/issues/124, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAtHAzSHWTLHgSO4e2_zUBe_knhvZY7Hks5vSQoBgaJpZM4bYySK
.

I am facing the same issue

The bug here is because since 3.2, void is not consider as EmptyAC, so you get a PayloadAC<x, void> type for your request that's not compatible with MouseEvent.
@nixxquality  Here a workaround, is to replace void by undefined in your twitch-frontend/src/store/colorCycle/actions.ts file:
export const cycle = createAsyncAction(
  ColorCycleActionTypes.CYCLE,
  ColorCycleActionTypes.CYCLE_SUCCESS,
  ColorCycleActionTypes.CYCLE_FAILURE
)<undefined, string, Error>();

At this moment, all error should be gone.
@piotrwitek For typesafe-actions, I don't know what's the approach you want to have for void, but a quick fix could be this one in typesafe-actions/dist/type-helpers.d.ts:
export declare type ActionBuilderConstructor<T extends StringType, TPayload extends any = undefined, TMeta extends any = undefined> =
    [TMeta] extends [undefined]
        ? [TPayload] extends [undefined]
            ? unknown extends TPayload
                ? PayloadAC<T, TPayload>
                : unknown extends TMeta
                    ? PayloadMetaAC<T, TPayload, TMeta>
                    : EmptyAC<T>
            : TPayload extends void // Just add this condition
                ? EmptyAC<T>
                : PayloadAC<T, TPayload>
        : PayloadMetaAC<T, TPayload, TMeta>;

Or if you want to have a generic way:
export declare type IsEmpty<T> = T extends undefined ? never : T extends void ? never : T;
export declare type ActionBuilderConstructor<T extends StringType, TPayload extends any = undefined, TMeta extends any = undefined> =
    [TMeta] extends [undefined]
        ? [IsEmpty<TPayload>] extends [never]
            ? unknown extends TPayload
                ? PayloadAC<T, TPayload>
                : unknown extends TMeta
                    ? PayloadMetaAC<T, TPayload, TMeta>
                    : EmptyAC<T>
            : PayloadAC<T, TPayload>
        : PayloadMetaAC<T, TPayload, TMeta>;

Thanks for investigation, I just returned from vacations, I'll take a look in the evening

@MasGaNo @nixxquality Thanks for your reports based on them I have come to the conclusion that changing void constraint to undefined was a breaking change and should follow with a major version bump instead with a minor and a migration guide.
In order to fix that I will publish a fix for v3 with a suggested solution by @MasGaNo to bring back the backward compatibility, and then I'll switch to a new major version v4, that will only support undefined constraint. Using undefined is superior and I don't see any reason to support void any longer as it only adds complexity to the codebase, migration to v4 will be very simple - change all void type parameters in action creators to undefined.

In my opinion, void makes it clear that no argument should be supplied whereas undefined sounds like undefined behaviour.
I  think the correct terminology is worth the extra work, but it's your call in the end.

I do not agree, void was incorrect terminology, it is only valid in the context of expressions when you want to change a function return, and IMO it shouldn't be used in other not related contexts to avoid confusion. undefined is correct in case of the empty function argument.
You can learn more about the semantics of void operator here: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/void

I wasn't aware that Javascript has that usage of void, but do note that TypeScript has its own definition of it.
https://www.typescriptlang.org/docs/handbook/basic-types.html#void

void is a little like the opposite of any: the absence of having any type at all. You may commonly see this as the return type of functions that do not return a value

This lines up with the usage of void in other languages, such as C.
The ambiguity is weird, but it's not the only place where TypeScript and Javascript have different meanings of things...

Fixed in v3.4.0 release

@nixxquality good point but nevertheless I'm maintaining this library for a few years now and I know that this will be a big hindrance in the long run for me, I made this errors in the past and I would rather focus on new features than such details that IMO have minimal impact for DX, it's just a matter to learn a new simple convention: use undefined instead of void

Also, it makes it symmetric with a new feature which is an optional parameter when you use union like this number | undefined.

For the no parameter type, what do you think about using never comparing to undefined?
FYI: I have been using never for this case since 2.x.

never is supposed to be used in cases where the function will never return; for example, something inside it always throws an error.
https://www.typescriptlang.org/docs/handbook/basic-types.html#never


update example to membrane 1.0

Hey,
I was trying to follow the examples and I noticed that it is using some old versions
This PR try to fix that

Awesome, thanks


Unable to build code containing TypeScript "named tuples" -- upgrade to TS 4

Current Behavior
tsdx build failed with below error:
syntax error TS1005: ',' expected.

when building code containing "named tuples", like this:
/** Parameters that describe an arrow. */
export type ArrowDescriptor = [
  /** start point */
  sx: number,
  sy: number,
  /** control point for start point */
  c1x: number,
  c1y: number,
  /** control point for end point */
  c2x: number,
  c2y: number,
  /** end point */
  ex: number,
  ey: number,
  /** angle of end point */
  ae: number,
  /** angle of start point */
  as: number
]

the error pointed to the line sx: number,.
Expected behavior
tsdx build should build code containing "named tuples".
Suggested solution(s)
I guess there's something wrong in the rollup plugins tsdx is using. Probably upgrading them would fix.
Additional context
Staring from TypeScript 4.0, "named tuples" can be used to annotate parameters in an array.
Your environment

  System:
    OS: Linux 5.4 Manjaro Linux
    CPU: (4) x64 Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz
    Memory: 447.06 MB / 15.48 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 15.14.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.7.6 - /usr/local/bin/npm
  Browsers:
    Chrome: Linux
    Chromium: 96.0.4664.45
    Firefox: 95.0
  npmPackages:
    tsdx: ^0.14.1 => 0.14.1
    typescript: ^4.5.3 => 4.5.3
  npmGlobalPackages:
    typescript: 4.0.8

Staring from TypeScript 4.0

Duplicate of #926


JS/Flow: Consider adding SetType etc

Right now we have Type<TypeDesc>. I think the code would be cleaner if we added specific Flow types for:
ListType<V>
SetType<V>
RefType<V>
MapType<K, V>
...

nope


Stream Control

Extend Wowza to support stopping streams using external HTTP requests
Extend Wowza to verify streams with application server before a stream is allowed to published
Implement adapter and REST API for stopping stream from the application server

@sharminelow Can you help review this? Take notes of 2 places with TODO. We can arrange to merge your other PR first so I can help resolving conflicts with this if there are any.

Checked through, shouldn't be a problem. You can merge yours in after the TODO has been resolved.

Ok. Thanks :). I will review yours a while later since we need it to complete the TODO


Content2.npl

Hi!
I'd like to keep separate the mgcb for NoPipeline project and the mgcb project, and run mgcb twice when building the game.
When creating Content2.npl (or whatever name I put to the npl file) I always get this error
1>  O:/tbd/Game1/Game1/Content/Content.npl not found!
1>  Run with path to .mgcb or .npl config as an argument:
1>      NoPipeline.exe Content/Content.mgcb
1>  or
1>      NoPipeline.exe Content/Content.npl
1>

It never works, unless of course npl file is Content.npl , then it seems to work well.
Is it a bug or I have to do something extra in order to get a different name working?
thanks!

oh, wait, the mgcb must exist too and be correctly configured. Never mind, my fault.


Display issue at smaller browser sizes

Overview
Issue with the facility name displaying over the report and 'copy link' buttons.
Expected Behavior
When browser is at smaller sizes, report and 'copy link' buttons should display in the row under the name.
Actual Behavior

nal context

resolved, tested on chrome at various sizes with short and long facility names


Error generating alerts on Iris | 'NoneType' object has no attribute 'removesuffix'

Good morning,
An attempt has been made to run elastalert on Docker, version 2.2, with the following file:
es_host: ***************
es_port: *******
alert:
- debug
description: Test Rule
filter:
- query_string:
    query: (data.office365.Workload:AzureActiveDirectory AND data.office365.Operation:UserLoginFailed AND data.offic>
index: *******
name: Test Alert
priority: 3
realert:
  minutes: 0
type: any
alert:
- "iris"

iris_host: ***************
iris_api_token: *********************************************
iris_customer_id: 2
iris_description: 'Test alert from ElastAlert2'
iris_alert_note: 'Alert triggered by opened session'
iris_alert_tags: 'test, login, ssh'
#iris_alert_context:
#  username: data.office365.UserId
#  ip: data.office365.ClientIP
iris_iocs:

  - ioc_value: data.office365.ClientIP
    ioc_description: source ip address
    ioc_tlp_id: 1
    ioc_type_id: 79
    ioc_tags: ip-src

The first alert pops up in Iris, including its IoCs and everything is correct, but when the second alert pops up, it appears without IoCs and the following exception pops up
elastalert_error - {'message': "Uncaught exception running rule Test Alert: 'NoneType' object has no attribute 'remov                                                                                esuffix'", 'traceback': ['Traceback (most recent call last):', '  File "/usr/local/lib/python3.12/site-packages/elast                                                                                alert/elastalert.py", line 1324, in alert', '    return self.send_alert(matches, rule, alert_time=alert_time, retried                                                                                =retried)', '           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/                                                                                lib/python3.12/site-packages/elastalert/elastalert.py", line 1409, in send_alert', '    alert.alert(matches)', '  Fil                                                                                e "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 138, in alert', '    alert_data = self.                                                                                make_alert(matches)', '                 ^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/e                                                                                lastalert/alerters/iris.py", line 96, in make_alert', '    iocs = self.make_iocs_records(matches)', '           ^^^^^                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 67,                                                                                in make_iocs_records', "    record['ioc_value'] = lookup_es_key(matches[0], record['ioc_value'])", '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert                                                                                /util.py", line 165, in lookup_es_key', '    value_dict, value_key = _find_es_dict_by_key(lookup_dict, term)', '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/ela                                                                                stalert/util.py", line 88, in _find_es_dict_by_key', '    term = term.removesuffix(string_multi_field_name)', '                                                                                ^^^^^^^^^^^^^^^^^', "AttributeError: 'NoneType' object has no attribute 'removesuffix'"], 'data': {'rule': 'Test                                                                                Alert'}}

elastalert_error - {'message': "Uncaught exception running rule Test Alert: 'NoneType' object has no attribute 'remov                                                                                esuffix'", 'traceback': ['Traceback (most recent call last):', '  File "/usr/local/lib/python3.12/site-packages/elast                                                                                alert/elastalert.py", line 1324, in alert', '    return self.send_alert(matches, rule, alert_time=alert_time, retried                                                                                =retried)', '           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/                                                                                lib/python3.12/site-packages/elastalert/elastalert.py", line 1409, in send_alert', '    alert.alert(matches)', '  Fil                                                                                e "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 138, in alert', '    alert_data = self.                                                                                make_alert(matches)', '                 ^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/e                                                                                lastalert/alerters/iris.py", line 96, in make_alert', '    iocs = self.make_iocs_records(matches)', '           ^^^^^                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 67,                                                                                in make_iocs_records', "    record['ioc_value'] = lookup_es_key(matches[0], record['ioc_value'])", '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert                                                                                /util.py", line 165, in lookup_es_key', '    value_dict, value_key = _find_es_dict_by_key(lookup_dict, term)', '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/ela                                                                                stalert/util.py", line 88, in _find_es_dict_by_key', '    term = term.removesuffix(string_multi_field_name)', '                                                                                ^^^^^^^^^^^^^^^^^', "AttributeError: 'NoneType' object has no attribute 'removesuffix'"], 'data': {'rule': 'Test                                                                                Alert'}}

elastalert_error - {'message': "Uncaught exception running rule Test Alert: 'NoneType' object has no attribute 'remov                                                                                esuffix'", 'traceback': ['Traceback (most recent call last):', '  File "/usr/local/lib/python3.12/site-packages/elast                                                                                alert/elastalert.py", line 1324, in alert', '    return self.send_alert(matches, rule, alert_time=alert_time, retried                                                                                =retried)', '           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/                                                                                lib/python3.12/site-packages/elastalert/elastalert.py", line 1409, in send_alert', '    alert.alert(matches)', '  Fil                                                                                e "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 138, in alert', '    alert_data = self.                                                                                make_alert(matches)', '                 ^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/e                                                                                lastalert/alerters/iris.py", line 96, in make_alert', '    iocs = self.make_iocs_records(matches)', '           ^^^^^                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert/alerters/iris.py", line 67,                                                                                in make_iocs_records', "    record['ioc_value'] = lookup_es_key(matches[0], record['ioc_value'])", '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/elastalert                                                                                /util.py", line 165, in lookup_es_key', '    value_dict, value_key = _find_es_dict_by_key(lookup_dict, term)', '                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^', '  File "/usr/local/lib/python3.12/site-packages/ela                                                                                stalert/util.py", line 88, in _find_es_dict_by_key', '    term = term.removesuffix(string_multi_field_name)', '                                                                                ^^^^^^^^^^^^^^^^^', "AttributeError: 'NoneType' object has no attribute 'removesuffix'"], 'data': {'rule': 'Test                                                                                Alert'}}

In this example there is only one match and no error occurs, the alert is generated correctly.
es_host: *********************************************
es_port: *********************************************
alert:
- debug
description: Test Rule 2
filter:
- query_string:
    query: (data.type:utm AND data.subtype:ips AND data.severity:medium)
index: *********************************************
name: Test Alert
priority: 3
realert:
  minutes: 0
type: any
alert:
- "iris"

iris_host: *********************************************
iris_api_token: *********************************************
iris_customer_id: 2
iris_description: 'Test alert from ElastAlert2'
iris_alert_note: 'Alert triggered by opened session'
iris_alert_tags: 'test, login, ssh'
iris_iocs:

  - ioc_value: data.srcip
    ioc_description: source ip address
    ioc_tlp_id: 1
    ioc_type_id: 79
    ioc_tags: ip-src

In this example there is more than one hit and the same error is generated.
es_host: *********************************************
es_port: *********************************************
alert:
- debug
description: Test Rule 3
filter:
- query_string:
    query: (data.type:utm AND data.subtype:ips AND data.severity:critical)
index: *********************************************
name: Test Alert
priority: 3
realert:
  minutes: 0
type: any
alert:
- "iris"

iris_host: *********************************************
iris_api_token: *********************************************
iris_customer_id: 2
iris_description: 'Test alert from ElastAlert2'
iris_alert_note: 'Alert triggered by opened session'
iris_alert_tags: 'test, login, ssh'
iris_iocs:

  - ioc_value: data.srcip
    ioc_description: source ip address
    ioc_tlp_id: 1
    ioc_type_id: 79
    ioc_tags: ip-src

This happens when the following command is used:
elastalert-test-alert name-of-file.yaml --alert

Thanks for reporting this @rober-fuji. I reviewed the IRIS alerter and see a problem.
@malinkinsa, in the IRIS PR you submitted last year, there's a bug where the IOC fetcher loop is overwriting the rule configuration, causing subsequent alerts to throw errors and fail to alert.  Specifically, this line:
https://github.com/jertel/elastalert2/blob/23aab845c421411b1cc07f27b61659a0a78c831c/elastalert/alerters/iris.py#L67
is fetching a value from the matched document, and then storing that looked up value over the top of the original lookup key.
Given this config:
iris_iocs:
  - ioc_value: data.srcip
     ...

If that fetched value wasn't found in the document then a None is returned, and now the rule configuration looks like this:
iris_iocs:
  - ioc_value:
    ...

Notice how ioc_value no longer has a lookup key. Now all subsequent IRIS alerts will fail since ElastAlert 2 currently does not allow None (aka null) lookup terms.
I think the code should be using a deep clone of the IOC object and modifying/appending that, rather than using the original rule config.
CC: @gregorywychowaniec-zt, since you recently made a change in this area and may have some input or time to help prepare a fix.

Unfortunately, I haven't watched it yet. I'll try to look closer to the middle of the month :(

Unfortunately, I haven't watched it yet. I'll try to look closer to the middle of the month :(

Any update on this?

Unfortunately, I still haven't had the time to address this issue. :(

Same for me, I'm not on this topic at the moment. I will reopen this topic on my side in a few weeks, so if it was not fixed at that time, I will keep an eye on it.

Hello, I will restart IRIS project soon so I will look for this issue in the next few weeks

Thanks you @bvirgilioamnh for taking the time to fix this issue, and for the several other IRIS alerter improvements. Your efforts are appreciated!


Missing pc_histogram related files in Makefiles

hardware.mk line 92 missing VSOURCES += $(BSG_MANYCORE_DIR)/testbenches/common/v/vanilla_core_pc_histogram.sv
library.mk line 35 missing include $(LIBRARIES_PATH)/features/pc_histogram/simulation/feature.mk
Currently make debug.log is broken when running with hb_hammerbench
These two lines were accidentally dropped in PR #796 when consolidating platforms into common/dpi.

I remember fixing this a couple weeks ago in the latest PR. Have you pulled from main?

Oops thanks @mrutt92 ! Yeah I am on an older branch that does not have PR #832 ..
Do you think adding $(BSG_MANYCORE_DIR)/testbenches/common/v/vanilla_core_pc_histogram.cpp is equivalent to including $(LIBRARIES_PATH)/features/pc_histogram/simulation/feature.mk? On my side both of them fix the make debug.log though.. Just unsure of other hidden impacts?


Add culture support for rule output

We should plan for adding localisation support into rule definitions.
Rules may also want to provide localised output such as recommendations.

Related to #19

Split PowerShell informational message generated from error, warning, verbose cmdlets into separate issue #158.


IFRAME not taking all height on HTML mail preview

IFRAME height defaults to browser default which is 150px on chrome and in effect mail preview is limited to a small scrollable stripe.
My fix was to set IFRAME height to 100%.

Thanks @nexces - I tried with it set to 100% and it didn't seem to work consistently.
It currently attempts to resize the IFRAME when you select a message, so it breaks if you open developer tools, or resize the window, once its already in preview mode.
But aside from that, it should work - I'll have a look today.

I don't know how tabs are handled but div.tab-content has fixed height that is set just once (on page load) and does not update when window is resized.
If IFRAME is set to width: 100%; and height: 100%; it occupies whole parent (div.tab-content).
If there is some JS function that is responsible for setting up this height it should also be launched on 'resize' and 'orientationchange' window events.
That should do the trick.
Also with IFRAME inside div.tab-content there is no need for div.tab-content to have overflow-y: scroll;

resizePreview in MailHog-UI/assets/js/controllers.js is called in selectMessage and toggleHeaders, which updates the height of .tab-content and .tab-content .tab-pane
It might be a timing bug though - AngularJS doesn't (or, didn't) give any straightforward way to hook into "after rendering the changes i'm about to make", i.e., to be able to wait until AngularJS has rendered the tabs and panes before attempting to resize them. For me, pushing the resize call to the end of the event loop (with a timer set to 0) was enough to let the browser finish rendering, but that might not be enough for everyone.
The overflow-y: scroll is used for the other panes, e.g. plain text and MIME, which aren't contained in an IFRAME. That could all do with a bit of cleaning up!
You're right about resize - none of it hooks into any window events, which does mean resizing or opening developer tools etc will break it. The UI was mostly inherited from the old MailHog, and the recent rewrite probably didn't go far enough.
Ideally I'd rather handle all of this without needing Javascript to avoid all this mess :smile:
I'll have a look at it asap!

To be honest I didn't analyze how you've done tabs - is it your own solution or is it some kind of plugin/library.
If I would like to make resizeable content I would go with CSS absolute positioning.
But there are drawbacks - you need to assume that (in case of MailHog UI) left and top side of resizeable content (message preview) is static in terms of size.
For left side it's easy - it can be left as it is. However top side can cause troubles - "Subject" and "To" can go multiline and it breaks "static" assumption.
You can always go with text-overflow:ellipsis on that fields and provide some tooltip, but someone might want to copy all that field content (i think it's an edge case, but might not be the only case) and tooltip might not allow it.
So... that was me thinking out loud :D

The tabs are done with bootstrap, hooked into with angulars ng-click binding - AngularUI might provide a nicer solution! I think it was the message headers which caused me the problem - the show/hide full headers button - but I can't really remember.
I'll have another go at fixing it - there's a lot of old unnecessary code and CSS left in there from the previous UI which I've been meaning to remove, cleaning all that up might make it easier to fix!

I think this is fixed with the latest UI changes - reopen if its still a problem :smile:

I am still having height issues with the latest release.

Me too, specially when I click the browser notification, it leads me to an iframe with 0 height.

This is still an issue 4 years later unless I am missing something


Tokenization for downstream tasks

First of all thank you very much for your work.
I am working on the long text classification task, and given the spectacular results of MEGA for long sequence modelling I wanted to use it for this task. The only thing that I haven't figured out how to do is the tokenization of my text samples, so I was wondering if someone could help me out on how to tokenize my text with the dict that is obtained from a checkpoint like the LRA one from the text task.
Thank you very much for your time

Also I have seen that you plan to release a checkpoint for machine translation (WMT'14), which could be useful for my task as the encoder is an english LM. Around what date do you target to release this checkpoint?
Many thanks


DatePickerIOS Date Props issue in release build

Environment
React Native Environment Info:
System:
OS: macOS High Sierra 10.13.6
CPU: (4) x64 Intel(R) Core(TM) i7-2640M CPU @ 2.80GHz
Memory: 33.43 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 8.11.3 - /usr/local/bin/node
Yarn: 1.9.4 - /usr/local/bin/yarn
npm: 5.6.0 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
Android SDK:
API Levels: 23, 25, 26, 27, 28
Build Tools: 23.0.1, 26.0.3, 27.0.3, 28.0.2
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64, android-24 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.1 AI-173.4907809
Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.3 => 16.6.3
react-native: 0.57.8 => 0.57.8
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-git-upgrade: 0.2.7
react-native-image-picker: 0.26.10
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4
Description
Date props not show Specify (defined Initial date ) date in picker.In debug build it work fine but in release build picker show today date.
Mobile OS=IOS
Reproducible Demo code
import {View,DatePickerIOS} from 'react-native';
<View style={{flex: 1, justifyContent: 'center'}}>
<DatePickerIOS
date={new Date('12-30-1980')}
onDateChange={this.setDate}
mode="date"
/>

@MuhammadUzair new Date('12-30-1980') isn't a valid date, so it returns undefined. If you would like to parse dates like this, I would suggest checking out moment.

@ericlewis It is a valid date, I tried in console. For the moment what date format is required for DatePickerIOS Date Props (like DD/MM/YYYY)?

Thanks @ericlewis .I Fixed this issue by change the date formate in  YYY-MM-DD .


cmd/client: move pod create to pod run

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

👍 I'll run tests locally.

LGTM


Debug subs feature

Needs a feature to subscribe to debug messaging. since right now usually gerald fails silently.

Did domething


Add support for Soy Element Composition.

Add support for Soy Element Composition. It has the syntax in the form of
<{foo()}></>
This adds support to pass through allowEmptyTag and to support this mode in closetag.

Thanks for clarifying.

Thanks a lot! This was a great experience.


[FLINK-28778][SQL/API] Bulk fetch of table and column statistics for given partitions

What is the purpose of the change
Currently the statistics information about tables can only be fetched from the catalog by each given partition iteratively. Since getting statistics information from catalogs is a very heavy operation, in order to improve the query performance, we’d better provide functionality to fetch the statistics information of a table for all given partitions in one shot.
Based on the manual performance test, for 2000 partitions, the cost will be improved from 10s to 2s. The improvement result is 500%.
Brief change log

Implementations for new added methods will be done in all classes that implement the Catalog interface.
For the concrete HiveCatalog, HIVE_DEFAULT_PARTITION will be taken care of while fetching the column statistics. - - All currently supported types for partition column will be taken into consideration.
The logic of getting TableStates in FlinkRecomputeStatisticsProgram will be optimized from calling the catalog iteratively to bulk fetch.
Logic of TableStates conversion will be consolidated into the CatalogTableStatisticsConverter class to make the domain design a little bit cleaner.

Verifying this change
This change is already covered by existing tests, such as HiveCatalogHiveMetadataTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

Continue with #20501


Simplify Group::last_modified

See https://github.com/FrancisRussell/ferrous-actions/pull/1

Nice spot. Unfortunately I do not have the option to trigger CI for your PR, possibly because I had CI configured to run on push which I've now changed. Can you rebase on develop?

Can you rebase on develop?

Done!