CIGAR operations out of range after splice junction correction

Hi,
Since the update in #14 to update transcripts after correcting splice junctions, I get the following error. I think there may be a bug in the rescueNoncanonicalJunction function, which results in a CIGAR string incompatible with the sequence (i.e. more match/insert/sub operations than there are characters in the sequence), but I haven't been able to trace the exact source of the error. Could you please look into it if you have the chance?
Thanks!
  File "bin/TranscriptClean_fork/TranscriptClean.py", line 1042, in <module>
    main()
  File "bin/TranscriptClean_fork/TranscriptClean.py", line 171, in main
    writeTranscriptOutput(noncanTranscripts, sjDict, oSam, oFa, transcriptLog, genome)
  File "bin/TranscriptClean_fork/TranscriptClean.py", line 184, in writeTranscriptOutput
    outSam.write(Transcript2.printableSAM(currTranscript, genome, spliceAnnot) + "\n")
  File "/scratch/mrstone3/pacbio_iPSC/bin/TranscriptClean_fork/transcript2.py", line 294, in printableSAM
    self.NM, self.MD = self.getNMandMDFlags(genome)
  File "/scratch/mrstone3/pacbio_iPSC/bin/TranscriptClean_fork/transcript2.py", line 360, in getNMandMDFlags
    currBase = self.SEQ[seqPos]
IndexError: string index out of range```

Hi Matt,
Have you tried running the TranscriptClean 2.0 version on this data instead of the patch update? I am hopeful that my test cases covered this issue in the new version of the code, although if this error happens there too I would certainly want to know so that I can address it. Another benefit of the most recent version is that you can run it on multiple cores for a better runtime.
-Dana

Update: I was able to find an example in some data I had of a noisy read that triggered this same problem during non canonical splice junction correction. I fixed the problem and added more tests. The patch will be included in v2.0.2. Hope this helps!

Hi Dana,
Thanks! The patch fixed that issue. Sorry to bother you again, but I found another bug that I was hoping you could take a look at. I'm not sure if it's related to the problem you found, so I thought I'd post it here.
I'm finding that after correcting noncanonical splice junctions, some transcripts have all of their canonical splice junctions shifted by 1 bp. I've attached an example of an affected transcript here. The issue appears when correcting with respect to either the gencode reference junctions or a set of junctions from Illumina sequencing of the same sample. Please let me know if you can't reproduce and I can share the gencode file I'm using.
Thanks,
Matt
transcriptclean_example.tar.gz

Hi Matt,
Thank you for bringing this to my attention. I will take a look at the files you sent and see if I can reproduce the issue.
Best,
Dana

Hi Matt,
So I tried to reproduce the issue with the latest TranscriptClean version (including reducing it down to a small test case of one junction), and I have not been able to get the same result. Here is the link to a UCSC genome browser session I am using to visualize the reads and get to the bottom of what is going on. The uncorrected and corrected files shown are the ones you sent me, and the one marked 'dana_run_corrected' is the corrected file I obtained when I ran my local version of TC on your file. Using the genome browser, I noticed in your corrected file that the sequence looks messed up right after the leftmost splice junction ends, which made me think that maybe the sequence was being handled incorrectly in some way that led to everything after the junction being shifted over. But as I say, I haven't been able to reproduce the problem.
Let me know what you think- it would be great to get to the bottom of this!
Best,
Dana


Show button for goal creation if issue does not exist

Currently a user can see the note box even if the issue does not exist in the open-sauced-goals repo.
A use should be prompted to create that issue if it does not exist. They should not be able to see the note box if it doesn't exist.

#348


Update README.md

Edited for readability
Syntax suggestion ("lower" to "no-upper")

Coverage remained the same at 73.035% when pulling 5fa01b65f82a22ac0c8ac510ce7c63c41b15660d on saphryn:patch-1 into c4c7dc366c10a96a22a6e0dfa7e657f5ee70d7f6 on YusukeHirao:master.


Can anserini be used for German documents?

If I have a set of German documents, can anserini be used there?

Yes, for example, check out: https://github.com/castorini/anserini/blob/master/docs/regressions-clef06-fr.md
The analyzer can be controlled by -language fr.


Fix setting username

reported in https://discuss.elastic.co/t/transaction-setuser-not-working-properly-when-populating-username/137092

Codecov Report

Merging #106 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #106      +/-   ##
============================================
+ Coverage     77.13%   77.14%   +<.01%
- Complexity      926      927       +1
============================================
  Files            96       96
  Lines          3132     3133       +1
  Branches        273      274       +1
============================================
+ Hits           2416     2417       +1
  Misses          570      570
  Partials        146      146

Impacted Files
Coverage Δ
Complexity Δ

.../elastic/apm/servlet/ServletTransactionHelper.java
78.75% <100%> (+0.26%)
22 <0> (+1)
:arrow_up:

Continue to review full report at Codecov.

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


Is it possible to set the FTP upload destination?

Describe your question, attach logs, images or anything that might help to better understand it.
Everytime I use the action-upload with a file, the filename is the path from the computer it's rendered on. It is possible to change the folder it's uploaded to in the post-render object?

I guess the best answer can be found in the docs regarding the ftp module.
https://github.com/inlife/nexrender/tree/master/packages/nexrender-provider-ftp
Since any params passed in the object will be directly passed into the node-ftp, it can be used to put some additional options. I'm assuming there must be a way to provide a target upload directory, or maybe some other way of achieving that.

Actually seems like from the .put description, the src field can be used to put file in the specific destination, however, the way it is used in our ftp and upload modules doesn't allow it to be provided from outside, that seems to be a bug.

I guess the best way to solve this would be adding and "output" field to the action-upload settings, that would be used to both form the key for AWS S3 module (unless there is already a "key" field defined in params), and a destination for FTP module.


Improving the docs for email config?

I thought since I was still having only limited success with alertmanager sending emails, I should check whether I'm doing it right, and at the same time try and get something up here for others to copy from until the docs are the definitive source of info...
Reading from various articles about the SMTP AUTH world, including https://www.fastmail.com/help/technical/ssltlsstarttls.html, I concluded that the likely combinations are:
Example config for TLS email server:
global:
  smtp_smarthost: 'xxxx.com:578'
  smtp_from<EMAIL_ADDRESS>  smtp_auth_identity<EMAIL_ADDRESS>  smtp_auth_secret: 'nnnnnnnn'

Example config for SSL email server:
global:
  smtp_smarthost: 'xxxx.com:465'
  smtp_from<EMAIL_ADDRESS>  smtp_auth_username<EMAIL_ADDRESS>  smtp_auth_password: 'nnnnnnnnnn'

Example config for plain SMTP email server:
global:
  smtp_smarthost: 'xxxx.com:25'
  smtp_from<EMAIL_ADDRESS>
Example config for SMTP with auth:
global:
  smtp_smarthost: 'xxxx.com:25'
  smtp_from<EMAIL_ADDRESS>  smtp_auth_username<EMAIL_ADDRESS>  smtp_auth_password: 'nnnnnnnnnn'

Then a simple email receiver config:
receivers:
- name: 'mail'
 email_configs:
 - to<EMAIL_ADDRESS>   send_resolved: true
   require_tls: false
 - to<EMAIL_ADDRESS>   send_resolved: true
   require_tls: false

Please correct me if anything is wrong, I'll edit the config snippets - I'm not sure if I've missed any options.
Where do smtp_auth_identity and smtp_auth_secret fit into the above configs? Are there more permutations than those I have listed?
I have received errors like WARN[0600] Notify attempt 1 failed: EOF                  source=notify.go:193 when it tries to send the email. I think this sort of error, once we know the reason for it should be caught and a useful error message printed...
I'm happy to try and fix docs and code, but I'm not an expert in email or go!
Thanks in advance to anyone who can help!

I updated the example configs a bit, I have now successfully sent emails from alertmanager using the SMTP with auth config above. I have had issues with TLS/SSL emails as the error reporting is almost nonexistent.

@estechnical - are you sure
global:
  smtp_require_tls: false

works?
In latest prom/alertmanager image from Docker Hub, this setting did not work. Instead adding this setting as part of email config worked -
email_configs:
    - to<EMAIL_ADDRESS>      require_tls: false

Indeed, there is no global TLS option. See #433

I have updated the config snippets in my post according to these suggestions. Thanks :)

Indeed,a more detail doc is necessary.

DEBU[0122] Notify attempt 2 failed: EOF                  source=notify.go:546
ERRO[0122] Error on notify: EOF
I think it make users feel confused. It's better to have more certain error.

@estechnical you're my f'n hero with the 'require_tls: false'.  Wasted a day because that flag seems to not be documented ANYWHERE.

I notice the "help wanted" tag... how would I help? :)

@estechnical could you extend the existing email_config documentation in the Promtheus documentation? You can find the repository here.

I started to work on this: prometheus/docs#917

Hello all. I have a question related to this config but the docs is not clear enough. How can I send to multiple email, I've tried some format of the but they didn't work:
1.
to: a<EMAIL_ADDRESS>2.
to<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Thanks in advance.

@vietthang207 you should just be able to separate them by comma:
receivers:
- name: 'email'
  email_configs:
  - to<EMAIL_ADDRESS><EMAIL_ADDRESS>
For better debugging of the email provider make sure to start Alertmanager with the --log.level=debug flag. As a test server you can use sudo python -m smtpd -n -c DebuggingServer <IP_ADDRESS>:25 in case you have python installed.
In general, please open up usage questions on the<EMAIL_ADDRESS>mailing list.


[Toggle PSR-11 Factories] Add static version to PostFeatureFactory

It closes #133 and also adds the phpspec/prophecy mocking library as commented here. The only drawback is that checking in this issue if it was possible the mock of the final class CreateFeature I realise that it was not possible :(
If the benefit of mocking final classes was the point to move on this library, we can remove the library and use mocks with PhpUnit as we've done with others ;)

Done @kpicaza ;)


Use Pusher

Rather than keep polling an endpoint to see if a CSV has finished processing, we should party like it's 2016 and use Pusher like modern people. This should also cut down on server load, which is nice.

Coverage increased (+0.03%) to 96.944% when pulling ae69a38671aee573a8fb7d552332b296eda5aa90 on feature-pusher-real-good into 3914fcbfbda585eff7a44f107e07702a3d9cd85b on master.

Coverage increased (+0.03%) to 96.944% when pulling e7cb4b0158f1f077a4fc6f4d3d5ef153f4620595 on feature-pusher-real-good into 3914fcbfbda585eff7a44f107e07702a3d9cd85b on master.

Right, tested this in the PR build and it works. This is good to go


Generate User APIs

View user profile GET /user/
Update profile POST /user/
Deactivate user POST /user/deactivate
Follow User user POST /user/follow

where  can be user_id or username, which ever we finalise .

I will take this up.

this tasks needs https://github.com/coderplex/coderplex-backend/pull/10  to be merged. waiting on it.


it's a bug in code?

lass BBoxTestMixin(object):
"""Mixin class for test time augmentation of bboxes."""
def merge_aug_bboxes(self, aug_bboxes, aug_scores, img_metas):
    """Merge augmented detection bboxes and scores.
    Args:
        aug_bboxes (list[Tensor]): shape (n, 4*#class)
        aug_scores (list[Tensor] or None): shape (n, #class)
        img_shapes (list[Tensor]): shape (3, ).
    Returns:
        tuple: (bboxes, scores)
    """
    recovered_bboxes = []
    for bboxes, img_info in zip(aug_bboxes, img_metas):
        img_shape = img_info[0]['img_shape']
        scale_factor = img_info[0]['scale_factor']
        flip = img_info[0]['flip']
        flip_direction = img_info[0]['flip_direction']
        bboxes = bbox_mapping_back(bboxes, img_shape, scale_factor, flip,
                                   flip_direction)
        recovered_bboxes.append(bboxes)
    bboxes = torch.cat(recovered_bboxes, dim=0)
    if aug_scores is None:
        return bboxes
    else:
        scores = torch.cat(aug_scores, dim=0)
        return bboxes, scores

in this code it use  img_shape, it should be pad_shape right?

Hi, @kellenf  This is correct because the pad is in the lower right way.

because it's lower right way, so in flip it's not right use img_shape to flip bboxes, right?

because it's lower right way, so in flip it's not right use img_shape to flip bboxes, right?

That's not what I meant. In the lower right pad style, no need to remove the pad pixels first.


[FEATURE] Fix Navbar Overlapping Content

Feature Request: Fix Navbar Overlapping Content
Description
The website’s navbar is currently overlapping the content below it, making it difficult for users to interact with or view the page information. Additionally, the navbar is too large, taking up excessive vertical space, which further reduces the overall user experience.

Problem Statement
The overlapping navbar obstructs the view of the main content, causing usability and accessibility issues. Its large size contributes to a poor user experience, as users cannot easily access the information or interact with the content without scrolling or resizing the window. Fixing these issues is essential to improve website navigation and content accessibility.

Proposed Solution
The navbar should be fixed at the top of the page without covering the content beneath it. This can be achieved by:

Adjust the CSS to use position to relative or fixed for the navbar.
Adding proper padding or margin to the content below the navbar to ensure it doesn’t overlap.
Ensuring the z-index is appropriately set to keep the navbar visible without obstructing the content.
Decreasing the height of the navbar to reduce its size, ensuring it occupies less vertical space.
Example: The navbar should remain at the top, and the main content should start right below it, maintaining appropriate space between them.

Alternatives Considered
One alternative considered was setting a fixed height for the navbar and reducing the content margin. However, this might not be scalable across different screen sizes. The proposed solution ensures better responsiveness.

Additional Context
Here is an example of the current issue

Checklist

[x] I have checked the existing issues and this feature request is not a duplicate.
[x] I have starred this repository.
[x] I am willing to help implement this feature (optional).

@gauravsingh1281 I request you to please assign this task to me

Can you assign me this issue?


Multi-select row disabled inconsistency

Starting from this record page:

click the Add link for the Specimen Collection table.
In the modal dialog, open the facet panel, go to the Specimen Collection facet, and select the  Histology of Human and Mouse embryonic and fetal kidney option.
Next, open the Species facet and select Mus Musculus

The table should have 25/33 records shown with the first one having metanephros (TS18-TS28) as the Anatomical Sources value. This row should be disabled

Change the limit dropdown to be 50.

The order of the displayed data shouldn't change, but sometimes this first row (including metanephros (...)) is no longer disabled
if it is not disabled, it becomes selectable (Select All can select it) and submitting the data will cause an error to be thrown.
following these steps is how the error from this issue was caused

After clicking on Add link for the Specimen Collection table, we're generating two requests
R1: A page of Specimen that we are going to show to the users.
R2: All the Specimen that have a Collection with the given RID=Q-3K5C, and have the same sorting and paging as the R1 request.
We show the result of R1, and use R2 to disable the existing values. This only works if R2 is a subset of R1 request. Which means that it has the same sorting and paging settings of the R1, alongside the same facets. The issue is that the function that I wrote for this feature doesn't support facets. So if you change the facets for getting R1, the R2 won't be the subset of R1 anymore.
I am going to change that function to support facets.

@ljpearlman @hongsudt I've pushed a fix for this to ermrestjs. Link to the commit:
https://github.com/informatics-isi-edu/ermrestjs/commit/cc2f8bc340a5b4e1b1394580dd0137f7395c474a
Can you please verify that this has solved the issue?

I am going to close this issue due to inactivity. We should create a new issue if the bug still exists.


Clean up isolation segment specs

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:
Cleaned up some testing after https://github.com/cloudfoundry/cloud_controller_ng/pull/2406 was merged since we thought it would look nicer

An explanation of the use cases your change solves

Links to any other associated PRs

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

It's not your fault you merged!  We should have put it into draft mode or given some other indication.  Thanks for looking at this


feature: dm-191 tag resources

Tag all newly created AWS resources from DM
💭 Description
Resources are now tagged by the app name and environment
✅ Checklist

[X] My code follows the code style of this project.
[X] My changes have been adequately tested locally.
[X] All automated tests are passing.
[X] I have updated/added required automated tests.
[X] Pre-commit checks are passing.

😪 Anything Else

Almost there, but there are supposed to be two tags. Like this ->
{"Key": "App", "Value": APP_NAME},
{"Key": "Environment", "Value": APP_ENV}
I would declare the tags in settings, so if we ever need to add an additional tag, it's as simple as changing one place.

updated


Finnish BIS (Independend Publisher) - Fixed response schema for Contact details

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.
Please note that we are experiencing a significant delay due to a high volume of connector submissions. It may take 1-2 weeks for us to provide comments for your connector.

[x ] I attest that the connector works and I verified by deploying and testing all the operations.
[x ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Fixed response schema. Contact details section value was typed as integer but now fixed to be a string.

@tpertila Please resolve the branch conflicts of the PR to get it approved and merged.

Scandic characters are removed now also from the apiProperties.json file.

@tpertila Can you please resolve the branch conflicts to get it approved and merged in the certification process?

[[certify-connector]]

@tpertila Can you please resolve the branch conflicts to get it approved and merged in the certification process?

Changed validation failed because of the correction itself. (ContactDetails value property type changed from integer (wrong) to string (proper)).

"the type has been changed for the output field. Existing flows may be broken if they are using this field and actual payload has been changed. ValuePath: operation/CompanyByBISCode/responses/200/schema/results/items/contactDetails/items/value/type Source: NewSwagger"
Currently, flow fails if there is any value on contactDetails value property. Becouse it has wrong datatype in original swagger file. Any change this can be merged?


The second image folder will not display icon

[x] I'm sure this issue is not a duplicate.

[ ] I'm submitting an icon request:

Type: ``
Icon Name: ``
Sample original Icon:
Extensions: ``
Filenames: ``
Language ids: ``

Language Extensions:

More info:

You will have to use the customization feature for that (https://github.com/vscode-icons/vscode-icons/wiki/FineTuning#some-examples). vscode does not support glob expressions.


add "open" popover styling to buttons

Changes
This PR adds "open" state styling to buttons that have a popover (or dropdown menu) that is currently open. Right now, it just reuses hover state styling, which is a subtle, non-offensive choice. (Can be made more noticeable in the future if desired)
Builds upon #2075.
CSS
The hover state is toggled on when the popover is open. This is done at the iui-button level (we can think about adding it to other fields in the future if needed).
I used a data attribute (data-iui-has-popover) and also added :has(+ :popover-open) as a hook that we can rely on in the future if we use the popover API in more places.
React
A new PopoverOpenContext is exported. Its value is provided in multiple components where a trigger element is present.
Button and IconButton read from this context and set the data attribute accordingly.
Testing
Added unit tests that either directly or indirectly check the affected code paths.
Updated affected visual test images. (Not sure why the focus moved in some cases; probably some flakyness).
Manually tested a whole bunch of stories that are affected. I encourage reviewers to do the same. (Start with DropdownButton for example)
https://github.com/iTwin/iTwinUI/assets/9084735/3dc04b2d-5684-40fd-8688-d52d2ecf88cc
Docs
Added changesets for CSS and React.

Waiting for @FlyersPh9 to create a PR moving icon fills to field CSS (see thread), because that change will likely affect visuals in this PR.

#2088 is merging any minute.


New cmdlets for FlowLog feature

FlowLog feature was created around 3 years ago. It allows customer to view information about ingress and egress IP traffic through an NSG. It used to be a proxy child resource of NetworkWatcher. Customer could enable/disable it using POST ConfigureFlowLog call. The problem with this approach was that customers couldn’t deploy flowLogs through template and couldn’t easily track and update them.
So, it was decided to make flowLog a tracked ARM resource and add PUT/GET/DELETE operations.
Now we need to add corresponding cmdlets for CREATE/GET and DELETE
Parameters for CREATE cmdlet should be the same as for this cmdlet: https://docs.microsoft.com/en-us/cli/azure/network/watcher/flow-log?view=azure-cli-latest#az-network-watcher-flow-log-configure
Here is documentation on existing cmdlets (for POST calls): https://docs.microsoft.com/en-us/cli/azure/network/watcher/flow-log?view=azure-cli-latest
Swagger link: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/networkWatcher.json
Starting from: "/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/networkWatchers/{networkWatcherName}/flowLogs/{flowLogName}": {
And here is an example for PUT operation: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/examples/NetworkWatcherFlowLogCreate.json
ETA
First half of March
Additional context

One problem we encountered during discussion is what we should do about current az network watcher flow-log show cmdlet. Right now on the backend it makes a POST call and returns this object:

{
"targetResourceId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourceGroups/cleanupservice/providers/Microsoft.Network/networkSecurityGroups/rg-cleanupservice-nsg5",
"properties": {
"storageId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourceGroups/MyCanaryFlowLog/providers/Microsoft.Storage/storageAccounts/immutablelogfltestsa",
"enabled": true,
"retentionPolicy": {
"days": 0,
"enabled": false
},
"format": {
"type": "JSON",
"version": 1
}
},
"flowAnalyticsConfiguration": {
"networkWatcherFlowAnalyticsConfiguration": {
"enabled": false,
"workspaceId": "a9ea48c6-2184-419a-9e39-31a913279979",
"workspaceRegion": "eastus",
"workspaceResourceId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourcegroups/flowlogsv2demo/providers/microsoft.operationalinsights/workspaces/hossamws",
"trafficAnalyticsInterval": 60
}
}
}
In a new cmdlet for GET call we need it to return
{
"name": "Microsoft.Networkcleanupservicerg-cleanupservice-nsg5",
"id": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourceGroups/NetworkWatcherRG/providers/Microsoft.Network/networkWatchers/NetworkWatcher_eastus2euap/FlowLogs/Microsoft.Networkcleanupservicerg-cleanupservice-nsg5",
"etag": "W/"149be128-02f3-4165-a8b2-3204792e425c"",
"properties": {
"provisioningState": "Failed",
"targetResourceId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourceGroups/cleanupservice/providers/Microsoft.Network/networkSecurityGroups/rg-cleanupservice-nsg5",
"targetResourceGuid": "fedd4848-4ce8-49ae-b308-824efdc42b14",
"storageId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourceGroups/MyCanaryFlowLog/providers/Microsoft.Storage/storageAccounts/immutablelogfltestsa",
"enabled": true,
"flowAnalyticsConfiguration": {
"networkWatcherFlowAnalyticsConfiguration": {
"enabled": false,
"workspaceId": "a9ea48c6-2184-419a-9e39-31a913279979",
"workspaceRegion": "eastus",
"workspaceResourceId": "/subscriptions/56abfbd6-ec72-4ce9-831f-bc2b6f2c5505/resourcegroups/flowlogsv2demo/providers/microsoft.operationalinsights/workspaces/hossamws",
"trafficAnalyticsInterval": 60
}
},
"retentionPolicy": {
"days": 0,
"enabled": false
},
"format": {
"type": "JSON",
"version": 1
}
},
"type": "Microsoft.Network/networkWatchers/FlowLogs",
"location": "eastus2euap"
}

add to S166.

@irrogozh Is ther min API version restricted?

@irrogozh Besides, I think it's possible to merge the response of GET call return into the previous response of POST call.
It takes care of backward compatibility, meanwhile supporting new response content.

What do you think?

@haroldrandom, it's a good idea as a temporary approach. But I don't think it's going to work in a long run. The reason is that

Customer will not see essential properties of the resource. Like, provisioningState, for example. So, he will not be aware if creating of resource succeeded or not.
Customer will not be able to use 'show' cmdlet, and, then, modify resource using 'update' cmdlet with this approach, right?

I start thinking that we will need 'update' cmdlet as well like it is done in CLI for other resources.

About your question on min API version. Corresponding API were added into swagger for 2019-11-01 version. But on the backend PUT/GET/DELETE calls will work for any early version.

Sync decision from Teams discussion: we decided not to mess up the output of current show command (via HTTP POST call). Instead, we decided to:

add another option like --arm to let user explicitly give us instruction that they want new formated output of what ARM returns (via HTTP GET call). The default vaule is false for now.
Add deprecation/warning info to current show command to warn user that the current output format will be changed.
Also add deprecation/warning info to --arm option to warn user that the default value of this parameter will be changed from false to true. Such that user don't have to provide this parameter since show command supports HTTP GET call only (ARM).
After 6 month, or service decide to finally retire old command, we will apply those changes and remove deprecation info.

Sorry, I found --arm is not necessary, new API GET call requires watcher_name and flow_log_name, that's already enough.


swift-inspect: enable dump-arrays on Windows

Adjust the implementation of DumpArrays to enumerate the allocations
first before processing them as the heap may alter in between.  This
significantly impacts performance on Windows.
Hoist iterateHeaps into the RemoteProcess protocol, requiring an
implementation on all platforms.  If the platform is unable to implement
heap traversal, it would be possible to simply leave the callback
uncalled.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @mikeash

@swift-ci please test


Add twitter stream loader and example

Implements #3.
The example code is similar to the twitter_streaming_lang in jubatus-example but uses first 1,000 tweets for training; so no dataset is required to try this out!

Coverage increased (+0.3%) to 78.139% when pulling 1b7a5ce45f9a394921e9cecaa1b7ff214c9b9dd5 on add-twitter into d1e602341d54bfd5f976aff492178880c1883922 on master.

Great works!! 👍


Unable to find 0.2.1 version of signals package on Hackage

Hi @markus-git
As per the co-feldspar.cabal file, the build dependencies include signals package with version >= 0.2.1 but we couldn't find the updated package on Hackage.
The latest one available at Hackage is <IP_ADDRESS>.
Can you guide me as to where I need to look for the >= 0.2.1 signals package.

Hello @emilaxelsson, can you look into this if possible?

@kalindkaria, I'm afraid I don't have access to anything besides what's in the public signals repo. Hopefully Markus can dig it up.

The commit bf598c8 fixes the above issue. Also the signals repository is updated to its latest version of 0.2.1.
Thanks @markus-git !


Universe extension JSON schema

For https://github.com/dagger/dagger/issues/3206
Note: the three required fields to add an extension to your dagger project today are: repo, ref, and path.

repo is the git repository where the extension lives
ref is the git ref for the version of the extension you want to load
path is the path to the cloak.yaml, which is the root of the extension in a repo. For most extensions, this will be at the repo root

In all cases below, extension owner can be inferred from the github org where the extension lives. Verifying the quality of the extension, the ownership, and README contents are our responsibility when a PR is submitted to any of these files.
Single File Option
In the universe repo, we have one extensions.json which looks like
[
{
  "name": "Terraform",
  "repo": "https://github.com/kpenfound/dagger-terraform",
  "path": "cloak.yaml",
  "ref": "v1.0.0",
  "tags": [ "infrastructure", "cloud", "aws", "gcp", "azure" ],
  "feathericon": "cloud", <-- optional
  "iconbackground": "purple" <-- optional
},
{
  "name": "Rails",
  "repo": "https://github.com/kpenfound/dagger-rails",
  "path": "cloak.yaml",
  "ref": "main",
  "tags": [ "ruby", "rails" ]
},
{
  "name": "Vercel",
  "repo": "https://github.com/slumbering/dagger-vercel",
  "path": "cloak.yaml",
  "ref": "v1",
  "tags": [ "web" ]
}
]

Pros:

May be easier to manage early on
May be easier to load the showcase

Cons:

May encounter merge conflicts with a lot of activity
Might become large and unwieldy

Multi-file Option
In the Universe repo, we have one json per extension owner
extensions/kpenfound.json:
{
  "Terraform": {
    "repo": "https://github.com/kpenfound/dagger-terraform",
    "path": "cloak.yaml",
    "ref": "v1.0.0",
    "tags": [ "infrastructure", "cloud", "aws", "gcp", "azure" ],
    "feathericon": "cloud", <-- optional
    "iconbackground": "purple" <-- optional
  },
  "Rails": {
    "repo": "https://github.com/kpenfound/dagger-rails",
    "path": "cloak.yaml",
    "ref": "main",
    "tags": [ "ruby", "rails" ]
  }
}

extensions/slumbering.json:
{
  "Vercel": {
    "repo": "https://github.com/slumbering/dagger-vercel",
    "path": "cloak.yaml",
    "ref": "v1",
    "tags": [ "web" ]
  }
}

Pros:

Easier to manage at a larger scale

Cons:

May be more difficult to load the showcase
May need to define a second data source for ordering

Open Questions

Can/Should we load extra repo info from the specified git repos in the showcase view? Or should we define everything we need for showcase view in the json?
Does the cloak.yaml snippet need to be defined in json, or are we able to render the repo's readme and require that the readme has a copyable snippet?
Are there additional fields that we can't load from the repo which we need to display extensions?
Do we provide a predetermined list of tag options for more logical groupings on the showcase?

@jpadams @slumbering @mircubed

Can/Should we load extra repo info from the specified git repos in the showcase view? Or should we define everything we need for showcase view in the json?

What's the showcase view?

Here is an example of third-party catalog that does not rely on custom metadata files: https://deno.land/x

What's the showcase view?

Showcase view is the main view that lists all extensions. Showcase is the term that docusaurus uses, but maybe there's a more universal term for it?

How does this work with planned https://github.com/dagger/dagger/issues/3016 ?

The proposed flow looks something like:

Add an extension to this structured data to have it listed on the universe site
User finds package on universe
User copies provided snippet to paste in their dagger.gql (or to add to their playground)

Single file option is the quickest option to implement for the front-end but we do not want to deal with merging conflict and huge file (hopefully we'll have this problem one day), do we? I think Multi-file option would be a nice option.
I was thinking of the multi-file differently though. Instead of creating a file per extension owner (kpenfound.json, slumbering.json...), I would rather see a file per extension like vercel.json, rails.json and so forth. When browsing the repo you can get an exhaustive list of available extensions and each one can get more than one source.
extensions/rails.json
[
  {
   "author": "kpenfound",
   "name": "Rails",
   "repo": "https://github.com/kpenfound/dagger-rails",
    "path": "cloak.yaml",
    "ref": "main",
    "tags": [ "ruby", "rails" ]
  },
]

extensions/vercel.json
[
  {
   "author": "slumbering",
   "name": "Vercel",
   "repo": "https://github.com/slumbering/dagger-vercel",
    "path": "cloak.yaml",
    "ref": "v1",
    "tags": [ "web" ]
  }, {
   "author": "AnotherUser",
   "name": "Vercel",
   "repo": "https://github.com/AnotherUser/dagger-vercel",
    "path": "cloak.yaml",
    "ref": "v1",
    "tags": [ "web" ]
  },
]

If we have more than one source for an extension like Vercel above, we could differentiate them by users, like Vercel by slumbering.
Below a proposal for a very simple card that display Terraform extension.
Notice icons in the footer. One that expand the card to get code snippet and the second one linked to the Github source repo. Cloud playground icon will also be here.

Here is an example of third-party catalog that does not rely on custom metadata files: https://deno.land/x

relies on a custom db somewhere, right?


Make UserHandler a 'service' class that is accessible from within the AbstractController

UserHandler should be a class that has one instance alive for each request.

This is fixed by the userHandler being accessible from within the Nacho class which every AbstractController gets. - closing


MySQL table really big

Hello.
I suggest that the tables could be exported in chunks, some MyPhpAdmin doesn't have really big import.
My import is restricted to 100 MB and i have 343774 KB
Make it into chunks that is smaller than 100 MB would be good incase your on a shared hosting or hosting solution.

Plugin territory. See wiki page Possible With A Plugin.


Advertise Debian / Ubuntu package in our docs

Our documentation does not mention yet that users of Debian derivatives (such as Ubuntu) can install OpenRefine with APT (apt install openrefine). This is much more convenient than downloading the archive, so it ought to be documented! I guess that could also be on our website too.
https://packages.debian.org/bookworm/openrefine
https://launchpad.net/ubuntu/+source/openrefine/3.5.2-1

@priyanshupardhi how is it going?

Hey @wetneb I was just setting up developer environment,
reading contribution guide.

not able to install openrefine through "apt install openrefine" command
Error: unable to locate package openrefine.

This is only available on some recent versions of Debian and Ubuntu: by following the links above, you should be able to find out which versions to support it.


Add test artifacts generated in Gradle build to gitignore

When running a Gradle build, a number of test artifacts are created in the pg directory.  Adding to gitignore.

Merged, thanks.


Make prompt tracking top level in send_event

ya it was just forgotten to be added with https://github.com/autoblocksai/python-sdk/pull/122

oops thanks 👍


for Pauli multiplication adding Pauli vectors element-wise

Description
this fixes PR #160
Motivation and Context
multiplying Paulis was not working
How Has This Been Tested?
from qiskit.tools.qi.pauli import Pauli
pauli_1 = Pauli([1,0,0],[1,0,0])
pauli_2 = Pauli([1,1,0],[1,0,1])
print(pauli_1)
print(pauli_2)
mult_pauli = pauli_1*pauli_2
print(mult_pauli)
maybe a testing routine should also go into test.python.test_qi.TestPauli, compare to PR #156
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

This works:
from qiskit.tools.qi.pauli import Pauli
import numpy as np
pauli_1 = Pauli(np.array([1,0,0]),np.array([1,0,0]))
pauli_2 = Pauli(np.array([1,1,0]),np.array([1,0,1]))
print(pauli_1)
print(pauli_2)
mult_pauli = pauli_1*pauli_2
print(mult_pauli)
Do you think it is worth to change it? Have you found occurrences of creation of Paulis without numpy arrays? In https://github.com/QISKit/qiskit-sdk-py/blob/master/qiskit/tools/apps/fermion.py
for example I use numpy arrays.

@antoniomezzacapo can we at least have a test and documentation that says used np.arrays.

Indeed, I was not aware that Paulis have to be numpy arrays.
I do not have a feeling about how much the change slows down Pauli multiplication when numpy arrays are used.

a short timeit with random rectors vectors with size 100000 and 100 repetitions
print(min(timeit.repeat('mult_pauli_np = pauli_1_nppauli_2_np', setup=setup, number=100)))
print(min(timeit.repeat('mult_pauli = pauli_1pauli_2', setup=setup, number=100)))
old code:
np arrays
0.33755043400014983 seconds
new code:
np arrays
13.35699049699997 seconds
lists
4.498526385000332 seconds
I will revert the code change and add some documentation on using np arrays.


feat(MatIconRegistry): "Wildcard-URL" support for svg icon sets

Feature Description
The current approach of adding svg icons either results in loading a large svg sprite regardless how many icons are actually used (approach 2) or requires to register every icon individually (approach 1). Approach 1 is not viable because it requires developers to maintain a list of all used icons. Alternatively all icons could be added manually. This is an improvement over the sprite as users only download the used icons. The downside is that still all icon "metadata" has to be included in the app, even if 95% of icons are not used.
The ultimate goal would be to only include the icons in the build that are actually used. But it would be sufficient if the webapp (as seen from the user) only downloads the necessary icons (and metadata), even if the build includes all icons. Having the option to configure a directory for a svg icon set (where each icon resides as a single svg file) would solve this issue. An icon defined as <mat-icon svgIcon="foo"></mat-icon> would look up the icon foo.svg in the configured directory.
Current configuration of an SVG icon set:
// Approach 1: either add all icons individually
for (...) {
  matIconRegistry.addSvgIcon('foo', domSanitizer.bypassSecurityTrustResourceUrl('foo.svg'));
}

// Approach 2: or the icon set
matIconRegistry.addSvgIconSet(domSanitizer.bypassSecurityTrustResourceUrl('foo-set.svg'));

Desired configuration:
// Approach 3
matIconRegistry.addSvgIconSetWithPath('/assets/icons/');

Use Case
The primary goal is to reduce the size of the app the user loads. Popular svg sprites are large and most of those items are not used in apps.
Adding icons individually (approach 1) works, but requires to maintain a list of all used icons or having a list of all possible icons (e.g. a string array of icon names) which again adds unnecessary data that has to be downloaded.
There should be a third approach that solves the issue and only actually loads the icons and metadata that are used by an app.

We use the URL to fetch the icon using an HTTP request. I'm not sure how that would work out when passing in a pattern. We also have support for defining an icon as a string so you could use Webpack to import the icons as a pattern (e.g. require.context('./icons', true, /\.svg$/)) and then register the values yourself.

We use the URL to fetch the icon using an HTTP request. I'm not sure how that would work out when passing in a pattern.

The icons would have to reside in a directory (which "defines" the icon set), e.g. "assets/icons". The URL would have to be composed by using the path and the icon name plus the svg file extension (e.g. ${iconSetPath}/${svgIconName}.svg). It would only require the file names to be identical with the icon name you pass to the svgIcon attribute.

We also have support for defining an icon as a string so you could use Webpack to import the icons as a pattern (e.g. require.context('./icons', true, /.svg$/)) and then register the values yourself.

Yes but, as in my described approach 1 developers still would have to take care of maintaining "a list" of the icons they used, right? Otherwise, if I'd just put 1000+ icons in a directory, webpack would include them all and register them regardless of whether I only used 5 of those icons or all of them. This still leads to a user downloading more code than necessary because the code that registers 1000 icons is always executed, even if the corresponding icons are never used.
Having the possibility to just tell the icon registry "In directory X are my icons, please look them up there." would solve the issue. No individual registration required (less code) and only used icons are downloaded.

What do you think of this approach, @crisbeto?

I think that it would make more sense to allow for a function to be passed in that allows people to construct the URL themselves, rather than coming up some kind of templating. I'd wait until https://github.com/angular/components/pull/18654 gets in before changing anything since it moves some things around.

@philmtd it's been a while since we discussed it, but I finally got back around to this issue. I've submitted #21431 which adds the ability to register a function that resolves the icon URL at runtime. Can you take a look if this is something that would've worked for your use case?

@crisbeto Thanks, I took a look at your changes and they look good to me.
Getting back to my original example I can now do the following if I want to resolve all icons in the default namespace manually:
matIconRegistry.addSvgIconResolver((name, namespace) => {
  return namespace === "" ? domSanitizer.bypassSecurityTrustResourceUrl(`/assets/icons/${name}.svg`) : null;
});

Can't wait to use it once it's released!


Linting/Documentation/Refactoring on gui.py

There should be no functional changes in this PR.  It is meant to only be a linting/refactoring pass on gui.py with some improvements on the docstrings as well.

functionality is the same, but pylint gave me these errors:
gui.py:21:0: R0902: Too many instance attributes (17/7) (too-many-instance-attributes)
gui.py:25:4: R0913: Too many arguments (6/5) (too-many-arguments)
gui.py:43:8: W0201: Attribute 'window' defined outside init (attribute-defined-outside-init)
gui.py:49:8: W0201: Attribute 'vid_count_text' defined outside init (attribute-defined-outside-init)
gui.py:51:8: W0201: Attribute 'storage_text' defined outside init (attribute-defined-outside-init)
gui.py:53:8: W0201: Attribute 'device_text' defined outside init (attribute-defined-outside-init)
gui.py:55:8: W0201: Attribute 'recording_text' defined outside init (attribute-defined-outside-init)
gui.py:57:8: W0201: Attribute 'time_text' defined outside init (attribute-defined-outside-init)
gui.py:59:8: W0201: Attribute 'error_text' defined outside init (attribute-defined-outside-init)
gui.py:61:8: W0201: Attribute 'ip_text' defined outside init (attribute-defined-outside-init)
gui.py:63:8: W0201: Attribute 'solar_text' defined outside init (attribute-defined-outside-init)
gui.py:73:8: W0201: Attribute 'frames' defined outside init (attribute-defined-outside-init)
gui.py:124:8: W0201: Attribute 'elapsed_time' defined outside init (attribute-defined-outside-init)
gui.py:206:4: R0913: Too many arguments (6/5) (too-many-arguments)
gui.py:223:0: R0903: Too few public methods (1/2) (too-few-public-methods)
gui.py:368:31: W0613: Unused argument 'controller' (unused-argument)
gui.py:394:0: R0903: Too few public methods (1/2) (too-few-public-methods)
the only one i tried to fix was gui.py:368:31: W0613: Unused argument 'controller' (unused-argument) but upon fixing it it throws an error elsewhere due to how frame variable for all the pages requires the controller to be there, so i think it makes sense to keep it. i think these pylint things are fine but just in case we wanted to capture that they're thrown at least in my pylint version


Parse instance name from ADO connection string, regardless of platform

Sql Browser works fine on non-windows platforms (at least Linux), so there is no reason to limit parsing out the instance name to Windows.

Oh it does? Is there any other places in code that should get changed now. Did you test connecting to a browser from a linux workstation?

I tested it on a Linux machine by manually setting the instance_name on the parsed Config object. It was able to connect to the browser just fine. The only place I could find that needed changed was simply in the ADO string parser.


.NET Core February 2020 Update - 2.1.16, 3.0.3 and 3.1.2

3.1.2 Release Notes
3.1.2 Download
3.0.3 Release Notes
3.0.3 Download
2.1.16 Release Notes
2.1.16 Download
Blog posts
.NET Core
Please report any issues you find with 2.1.16, 3.0.3 or 3.1.2, either responding to this issue, creating a new issue or creating a new issue in one of the following repos:

aspnet/Home - for ASP.NET Core questions
aspnet/EntityFrameworkCore - for Entity Framework Core questions
dotnet/cli - for CLI tools and questions
dotnet/runtime - for runtime, API, or installer issues and questions
dotnet/winforms - for Winforms issues
dotnet/wpf - for WPF issues

We are experiencing delays in pushing Linux feeds. Here is the list of repositories we are trying to push and as the release is progressing, we will tick them off as each is resolved.

Distro
3.1
3.0
2.1

Ubuntu 16.04
X
X
X

Ubuntu 18.04
X
X
X

Ubuntu 19.10
X
X
X

Centos 7
X
X
X

Debian 9
X
X
X

Debian 10
X
X
X

Fedora 30
X
X
X

Fedora 31
X
X
X

OpenSUSE 15
X
X
X

SLES 12
X
X
X

SLES15
X
X
X

RHEL 7 / Oracle Linux 7
X
X
X

Here is a list of issues that we are tracking and working to resolve with the Feb 2020 release. We'll tick them off as each is resolved. See any linked issue for progress or resolution details.

[x] Missing Microsoft.EntityFrameworkCore.Relational 3.1.2 on nuget.org
[ ] The type or namespace name 'Pipelines' does not exist in the namespace 'System.IO' (https://github.com/dotnet/aspnetcore/issues/19133) - more info
[x] dotnet/announcements (https://github.com/dotnet/announcements/issues/151), the link to the Release Notes of .NET Core 3.1.2 doesn't work.
[x] No binaries published for macOS/x64 for the new 3.1.102 SDK (missing sdk osx.tar.gz from releases.json)(https://github.com/dotnet/core/pull/4289#event-3052723721)
[ ] SqlClient issue on ASP.NET Core app update from 3.1.1 to 3.1.2
[ ] Flakiness in CentOS 7 Linux package repository, encountered >5 times per day (https://github.com/dotnet/core/issues/4298)

Hi, we're having issues installing 2.1.16 on Ubuntu 16.04.  Are the some dependency issues here?
ubuntu@ip-172-31-52-133:~$ sudo apt-get update && sudo apt-get install -y dotnet-sdk-2.1
Hit:1 http://us-east-1.ec2.archive.ubuntu.com/ubuntu xenial InRelease
Hit:2 http://us-east-1.ec2.archive.ubuntu.com/ubuntu xenial-updates InRelease
Get:3 http://us-east-1.ec2.archive.ubuntu.com/ubuntu xenial-backports InRelease [107 kB]
Ign:4 http://us-east-1.ec2.archive.ubuntu.com/ubuntu precise InRelease
Get:5 http://security.ubuntu.com/ubuntu xenial-security InRelease [109 kB]
Ign:6 http://archive.ubuntu.com/ubuntu trusty InRelease
Hit:7 https://packages.microsoft.com/ubuntu/16.04/prod xenial InRelease
Hit:8 http://download.mono-project.com/repo/debian wheezy InRelease
Hit:9 http://download.mono-project.com/repo/debian wheezy-apache24-compat InRelease
Hit:10 https://deb.nodesource.com/node_8.x xenial InRelease
Hit:11 http://repo.saltstack.com/apt/ubuntu/16.04/amd64/2019.2 xenial InRelease
Hit:12 http://us-east-1.ec2.archive.ubuntu.com/ubuntu precise Release
Hit:13 http://archive.ubuntu.com/ubuntu trusty Release
Hit:15 https://packagecloud.io/github/git-lfs/ubuntu xenial InRelease
Fetched 216 kB in 0s (259 kB/s)
Reading package lists... Done
W: http://us-east-1.ec2.archive.ubuntu.com/ubuntu/dists/precise/Release.gpg: Signature by key 630239CC130E1A7FD81A27B140976EAF437D05B5 uses weak digest algorithm (SHA1)
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 dotnet-sdk-2.1 : Depends: dotnet-runtime-2.1 (>= 2.1.16) but it is not going to be installed
                  Depends: aspnetcore-runtime-2.1 (>= 2.1.16) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

Seeing the exact same situation on Ubuntu 19.10:
ator@ubuntu:~$ sudo apt-get install dotnet-sdk-3.1
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 dotnet-sdk-3.1 : Depends: aspnetcore-runtime-3.1 (>= 3.1.1) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

Hey folks, Keep an eye on the Linux repo checklist at the top of this issue. The upload service is really slow today and we're babysitting everything through.

@TaraOverfield in repository dotnet/announcements (issue 151, February 2020 .NET Updates), the link to the Release Notes of .NET Core 3.1.2 doesn't work. It currently links to a file in tag 3.1.1, where it should be 3.1.2.
The correct link is: https://github.com/dotnet/core/blob/master/release-notes/3.1/3.1.2/3.1.2.md

@TaraOverfield in repository dotnet/announcements (issue 151, February 2020 .NET Updates), the link to the Release Notes of .NET Core 3.1.2 doesn't work. It currently links to a file in tag 3.1.1, where it should be 3.1.2.
The correct link is: https://github.com/dotnet/core/blob/master/release-notes/3.1/3.1.2/3.1.2.md

@rbhanda needs to take a look

Going from 3.1.1 to 3.1.2, our app failed to start:
Could not load file or assembly 'System.Data.SqlClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.

Event log
crit: Microsoft.AspNetCore.Hosting.WebHost[6]
      Application startup exception
System.IO.FileNotFoundException: Could not load file or assembly 'System.Data.SqlClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.
File name: 'System.Data.SqlClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'

Ok so I am running into this over five times a day, each time it seems to work if I try again in a few minutes. It's turning out to be a major productivity problem when we have multiple image builds per day on cluster - each one has to be manually checked and rerun when they fail. I am just wondering if the RPM repo will ever be reliable:
Downloading packages:
https://packages.microsoft.com/centos/7/prod/netstandard-targeting-pack-2.1.0-x64.rpm: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=packages-microsoft-com-prod clean metadata
Trying other mirror.

Error downloading packages:
  netstandard-targeting-pack-2.1-2.1.0-1.x86_64: [Errno 256] No more mirrors to try.

@yoshiya8 check out https://github.com/dotnet/core/issues/4167 for general status on our effort to get the owners of the service to improve its reliability (and updates when we have them).
That said... we haven't heard of flakiness the way you're experiencing it... the typical scenario is a period of downtime (up to 2 hours is "normal") but then back up for everyone. I've opened https://github.com/dotnet/core/issues/4298 based on your comment to discuss more since it doesn't sound related to this release.

When migrating from .NET Core 2.2 to 3.1, CORS does not appear to be working properly. I now receive an error saying

Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://localhost:5000/users/login. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing).

In the ConfigureServices method, I have:
services.AddCors();
In the Configure method, I have:
app.UseRouting();

app.UseCors(options => options
        .AllowAnyOrigin()
        .AllowAnyHeader()
    );

app.UseAuthentication();

app.UseEndpoints(endpoints =>
{
    endpoints.MapControllers();
});

Note: The documentation on Microsoft's website is using outdated code, such as; UseMvc and IHostingEnvironment.
https://docs.microsoft.com/en-us/aspnet/core/security/cors?view=aspnetcore-3.1

hi,
could you please provide upgrade steps from .net core 2.1 to 2.1.16

Seeing the exact same situation on Ubuntu 19.10:
ator@ubuntu:~$ sudo apt-get install dotnet-sdk-3.1
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 dotnet-sdk-3.1 : Depends: aspnetcore-runtime-3.1 (>= 3.1.1) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

Experiencing the same issue

Is 3.1.2 is deployed to azure. I don't see it in selection.

Hello,
On .NET Core 3.1 page release date of 3.1.2 is changed from February to 16 March 2020.
https://dotnet.microsoft.com/download/dotnet-core/3.1
This was done by mistake? Or maybe there is new invisible release?
Size of runtimes and hosting packges still the same as in February.

How can install .NET 3.1.1 on ubuntu and not 3.1.2 ? Becasuse 3.1.2 has breaking changes

@dicksonkimeu Based on a couple of Stack Exchange answers, you can do the following:

sudo apt update to refresh the list of available versions
apt policy dotnet-runtime-3.1 to see the available package versions.
sudo apt install dotnet-runtime-3.1=3.1.1-1 to install the desired version of the package.
dotnet --info to verify you installed the right thing.

Replace the dotnet-runtime with a different dotnet- package depending on what exactly you need.

Seeing the exact same situation on Ubuntu 19.10:
ator@ubuntu:~$ sudo apt-get install dotnet-sdk-3.1
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 dotnet-sdk-3.1 : Depends: aspnetcore-runtime-3.1 (>= 3.1.1) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

I experience exactly the same issuet on Ubuntu 20.04. Does anyone know a fix for this?

@michalbilinski See https://github.com/dotnet/core/issues/4360. Ubuntu 20.04 is currently not supported, packages for it should be available on 12 May. In the meantime, you can try the workarounds mentioned in that issue.

@michalbilinski See #4360. Ubuntu 20.04 is currently not supported, packages for it should be available on 12 May. In the meantime, you can try the workarounds mentioned in that issue.

You are right, thanks!


JSBin is down - 502 Bad Gateway

Even though the site is down for me, and http://www.isitdownrightnow.com confirms that it is down for everyone (not sure how accurate is this), http://status.jsbin.com/ informs me that 'All System Operational'

Yeah, gotta kill that status page and current in the process of a server move, so hopefully this helps.


Performance Issues with predict Call in XGBoost (v2.1.1)

Hello XGBoosters!
I was looking for the official forum (https://discuss.xgboost.ai/) to discuss this issue, but it seems the link is no longer working. Does anyone know if it's still active or if there's another community hub for discussions?
Now, to the issue:
I’m running a live simulation using an XGBoost model in Python. During the simulation, I call the following code to make predictions:
# Make prediction
prediction = model.predict(dmatrix)

After profiling, I found that this predict call takes ~400ms on average, which is significantly impacting the performance of my application. I’m using XGBoost v2.1.1.
My goal: I need to reduce the prediction time to around 30ms. A 10x speed-up is essential for my use case.
What I’ve considered so far:

Downgrading XGBoost: Some have reported that v1.7.6 is faster.
Using ONNX Runtime: Converting the model and running it with ONNX Runtime.
Switching to Another Language: Implementing predictions in languages like Go or C for better performance.
Caching Solutions: Investigating caching mechanisms for repeated predictions.

Here's a screenshot of the profiling results for reference:

How I can run 10x more fast this predict? like about 30ms? Some solutions I found but have not tried yet:

Change to old versions of XGBoost, V1.7.6 seens fast.
Use onnxruntime solution
Use another language like Go, C...
Some way to cache

Questions:
Has anyone else encountered similar performance issues with model.predict in XGBoost v2.1.1?
Are there specific settings or optimizations within XGBoost that could reduce prediction time?
Which of the options above would you recommend trying first? Or are there alternative solutions I haven’t considered?
Thanks in advance for any advice or insights!

Could you please extract a snippet of your inference code and the parameters used to train the model, along with data shape and sparsity?
This needs to be resolved case by case, for example, https://github.com/dmlc/xgboost/issues/10882 is caused by inspecting the pandas dataframe.
If you prefer a dedicated inference engine, you might want to try the FIL project (disclosure: I work for the same team as cuML).
cc @wphicks


Improve FAQ answer.

retest this please

retest this please

@sascala, updated! Let me know what you think.

@ssk2 That makes so much more sense! 👍


Prefix issue labels

Common colours should have common prefix - easier to find. For example:

language: c
language: ruby

etc

Done!


[Bug]: New Citation in Citation Annotator shows Topics component rather than Citations list

Steps to reproduce the bug
1. Open any radial that shows Citations.
2. Delete all existing citations.
3. Now add a new citation.

Expected results: The new citation gets listed.
Actual results: The new citation doesn't appear, instead a Topics chooser/list shows with nothing listed.

Screenshot
No response
Expected behavior
No response
Additional Screenshots
No response
Environment
Development (docker)
Sandbox Used
Also shows up in sandbox
Version
08addf274e5abd25b297723f4190aa555e1569e5
Browser Used
No response

@mjy, if I remember correctly, this behavior was intentional. After create the citation, load the citation and display list of topics for that citation. This has changed?

I'm not sure my pr is what you want, but I'll add some more context as to what made it seem like a bug to me (a naive new user):
When the first citation is added, I see:

None of the icons to remove or edit the citation I added, nothing showing the name of the citation I added - if I now reload the page and go back to the citations radial, I see this:

The topics section no longer appears and the list with my citation and the ability to edit or delete it appears, which is what I was (perhaps naively) expecting when the citation first got added.
Also, with the current behavior, if I start with no citations, add one, now the topics section is showing, now if I add a second citation I get the popup saying the citation was added but the display doesn't change (maybe it would if the second citation had different topics than the first?) and there's no visual indication that i have more than one citation.
And if I load the citation annotator with already existing citations, it starts off showing the list, but now if I add a new one the topics pane replaces the list and I can't see how to get back to the list except to reload the page. Maybe I'm missing something?

Sorry for the slow trickle as I figure some of this out, but now I notice that when I do choose the green edit icon from the list for a citation, it displays the Topics pane for that citation, but there's no indication (?) of which citation I'm editing, and no way to get out of the topics pane and back to the list. Maybe that's what this issue is actually about?

I think the issue is that @kleintom , it wasnt easy to know if the user was creating or editing a record. I pushed a change for it, I think it should be more clear now

Yeah, I didn't realize that was edit mode - thanks for the update!


(PUP-7030) Fix bug causing lexer hick-up on two adjacent unicode chars

The lexer was only capable of recognizing the first of several adjacent
unicode characters. This commit fixes that problem.

CLA signed by all contributors.

TIL: hickup is the archaic spelling of hiccup


Pass env vars to cdn build

main

#184   👈

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Feature/issue 1221 OpenCL implementation of primitive mdivide_left_tri

Summary
This adds the use of OpenCL lower_triangular_inverse in mdivide_left_tri. A specialization for double was added for both mdivide_left_tri(A) and mdivide_left_tri(A,b) to avoid any issues with fwd mode.
Tests
Adds new tests to check that the CPU and OpenCL versions are the same. All added tests are in prim/mdivide_left_tri
Side Effects
/
Checklist

[x] Math issue #1221

[x] Copyright holder: Rok Češnovar, Univ. of Ljubljana

[x] the basic tests are passing

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Speedups:
The V100 shows its a real beast here.

Files:
titanxp.txt
v100.txt
r9.txt
i7.txt

@serban-nicusor-toptal another Jenkins hiccup I noticed this morning:
This branch failed on the Upload doxygen phase (http://d1m1s1b1.stat.columbia.edu:8080/blue/organizations/jenkins/Math Pipeline/detail/PR-1222/3/pipeline/263).
Restart fixed that issue, but not sure what would be the cause.

Waiting for next available executor
i-0a7b57d754220ce86 was marked offline: Connection was broken: java.io.EOFException
	at java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2671)
	at java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:3146)
	at java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:858)
	at java.io.ObjectInputStream.<init>(ObjectInputStream.java:354)
	at hudson.remoting.ObjectInputStreamEx.<init>(ObjectInputStreamEx.java:49)
	at hudson.remoting.Command.readFrom(Command.java:140)
	at hudson.remoting.Command.readFrom(Command.java:126)
	at hudson.remoting.AbstractSynchronousByteArrayCommandTransport.read(AbstractSynchronousByteArrayCommandTransport.java:36)
	at hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:63)
Caused: java.io.IOException: Unexpected termination of the channel
	at hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:77)

Somehow the EC2 machine just died ?
To be honest I don't really get it why this happened, It's just like the machine timed out while running.

Thanks for digginig into that.

Is this waiting for review?

Yep.

@SteveBronder Finally managed to come back on this. Sorry for the delay.
I changed lower_triangular_inverse to tri_inverse with the triangular_view and its almost done.
What is missing is how to prevent the use of TriangularViewCL::Entire with tri_inverse without making two separate specializations for Lower and Upper. One option is making it a run-time error but would really want to make it a compile time error. Do you have any ideas/help on how to do that?
The only other remaining thing is just to clarify one the inplace version (see comment).

I'm working from home tomorrow so ping me whenever

@SteveBronder ready for re-review. Thanks.

Thanks for the review! I added the missing doxygen comments and responded to the RNG comment.


Halon-326: Workaround high memory grows unless better solution will be available.

Created by: qnikst

Closing as an obsolete


Wrong sign in control surface hingLineRotation for port-side wing

According to the CPACS documentation the hingeLineRotation of a control surface path is defined to be mathematically positive around the vector pointing from the inner hinge point to the outer hinge point:

The deflection path of the control surface is defined within the hinge line coordinat system. This is defined as follows: The x-hinge coordinate equals the wing x-axis. The y-hinge coordinate equals the hinge line axis (see above; positive from inner to outer hinge point). The z-hinge line is perpendicular on the x-hinge and y-hinge coordinate according to the right hand rule. The rotation of the control surface is defined as rotation around the positive y-hinge line.

For a starboard trailing edge device, a positive hinge line rotation means the device is rotated downwards for descend and for a port wing it would be the other way around: A positive hinge line rotation means the device is rotated upwards for ascend.
If a port-side wing is defined in CPACS (i.e. it is not a mirror of the starboard wing using the symmetry attribute) the hinge line rotation in TiGL has the wrong sign.
I suppose the error is somewhere in CTiglControlSurfaceTransformation::CTiglControlSurfaceTransformation.
Thanks @Cliersch for the find.

I suppose this is related both to the ongoing discussion about symmetry in CPACS, see https://github.com/DLR-SL/CPACS/issues/609 and the discussion on coordinate systems in TiGL, see https://github.com/DLR-SC/tigl/discussions/911.
If we follow the CPACS definition precisely (which of course we should), we have a different sign of the hingeLineRotation for mirrored wings and non-mirrored wings on the port-side.


Workspace settings not respected

VSCode Version: Code 1.12.2 (19222cdc84ce72202478ba1cec5cb557b71163de, 2017-05-10T13:20:36.315Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

angulardoc-vscode
AngularDoc
2.2.2

tslint
eg2
0.15.0

vscode-great-icons
emmanuelbeziat
2.1.6

Angular2
johnpapa
2.3.1

vscode-add-angular-native-files
joshdsommer
0.0.8

cpptools
ms-vscode
0.11.2

csharp
ms-vscode
1.10.0

team
ms-vsts
1.118.2

angular2-inline
natewallace
0.0.17

tns-cli
pkosta2006
0.0.3

Ruby
rebornix
0.12.1

code-settings-sync
Shan
2.8.1

nativescript
Telerik
0.7.2

nativescript-ng2-snippets
wwwalkerrun
2.0.1

Steps to Reproduce:

Change workspace integrated terminal to something different: eg "terminal.integrated.shell.windows": "C:\\Program Files\\Git\\bin\\bash.exe"
Open an integrated terminal ctrl+`
Integrated terminal still has the terminal configured in user settings (In my case I have bash.exe as my user terminal and made an attempt to set a workspace setting for another executable for a different bash and also did a sanity check with PowerShell)

I believe this is already fixed, can you see if you can reproduce in the Insiders build?

@Tyriar It looks like it's working! Thank you! My apologies for the duplicate, I wasn't able to find another issue reported for this :)

Would you happen to have a link to the pull request? I peeked at the code, but wasn't sure where to begin. I'd be curious what the fix was!

@jonest here's the issue https://github.com/Microsoft/vscode/issues/26178, the issue was wrongly falling through a switch statement (bad copy and paste) https://github.com/Microsoft/vscode/commit/4ba913058376d23a9641448e5ef56b010d092e46
Also in this area for v1.13, actions to control this are surfaced in the command pallete https://github.com/Microsoft/vscode/issues/23362


This does not work for azure functions 5, for example host start --port  command line arguments, just simply do not work.

[This document is out of date, these command line arg do not work with azure 5 functions, for example host start --port, causes an "unknown command"]

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

ID: 369aefa6-74da-3001-f835-15abdfffd5d6
Version Independent ID: 2cc2a2db-d059-3252-7a1a-4261b7d3c37c
Content: Develop Azure Functions using Visual Studio
Content Source: articles/azure-functions/functions-develop-vs.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @jwisener - Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

There is no "Azure Functions 5" and this article doesn't mention host start --port. If you have created this comment from the wrong article, please reopen it from the correct article. #please-close


Spinner errors

Hey. I wanted to try using your chat. Installed through npm. I added the necessary parameters, but when sending a message, I get an error from the spinner. What could be wrong?
Even when I just try to run your code and another question.
<template>
  <chat-window
    :currentUserId="1234"
    :rooms="rooms"
    :loadingRooms="loadingRooms"
    :messages="messages"
    :messagesLoaded="messagesLoaded"
  >
  </chat-window>
</template>

<script>
import ChatWindow from 'vue-advanced-chat'
import 'vue-advanced-chat/dist/vue-advanced-chat.css'

export default {
  components: {
    ChatWindow
  },

  data() {
    return {
      rooms: [],
      loadingRooms: true,
      messages: [],
      messagesLoaded: false
    }
  },

  mounted() {
    this.rooms = [
      {
        roomId: 1,
        roomName: 'Room 1',
        unreadCount: 4,
        lastMessage: {
          content: 'Last message received',
          sender_id: 1234,
          username: 'John Doe',
          timestamp: '10:20',
          date: 123242424,
          seen: false,
          new: true
        },
        users: [
          {
            _id: 1234,
            username: 'John Doe',
            status: {
              state: 'online',
              last_changed: 'today, 14:30'
            }
          },
          {
            _id: 4321,
            username: 'John Snow',
            status: {
              state: 'offline',
              last_changed: '14 July, 20:00'
            }
          }
        ],
        typingUsers: [4321]
      }
    ]

    this.messages = [
      {
        _id: 12345,
        roomId: 1,
        content: 'Heeeey',
        sender_id: 4321,
        username: 'Tu',
        date: '13 November',
        timestamp: '10:20',
        seen: true,
        disable_actions: false,
        disable_reactions: true
      }
    ]

    this.loadingRooms = false
    this.messagesLoaded = true
  }
}
</script>

I cannot reproduce this error.
Could you give me access to your repo so I can have a look? Or a minimal example that I can run to reproduce the error.

https://github.com/temirlankasen/naimi_admin_chat
I take data from api. data format from the documentation in the repository

I cannot run the project, there is an error.
I cloned the project, ran yarn install then npm run dev and get:
Error: EROFS: read-only file system, mkdir '/static'

Please let me know how I can run the project without any error.

I'm closing this issue until I can reproduce the error.


Fix: Webapp columns can be resized to "negative"

It is possible to drag the right-border of the column to the left of the left border (as if the width was "negative", so-to-say), which breaks the UI in multiple ways.
Demonstration:

Expected behavior:
Limit the minimum column size.

Hmm, the rust back-end should probably reject a negative width? @nhatcher do you think this should be handled on the UI or the "backend"? I am happy to make my first PR on this task.

Same issue exists with rows.

Hi @enjoythecode , this is a great first task. It should be handled in both places.
Around:
https://github.com/ironcalc/IronCalc/blob/main/base/src/worksheet.rs#L300
insert:
if width < 0.0 {
    return Err(format!("Cannot set a negative width: {width}"));
}

you will need to add a regression test in:
https://github.com/ironcalc/IronCalc/blob/main/base/src/test/test_column_width.rs
In the front:
https://github.com/ironcalc/IronCalc/blob/main/webapp/src/components/worksheet.tsx#L107
Maybe something like:
if (width > 0) {
      model.setColumnWidth(sheet, column, width);
      worksheetCanvas.current?.renderSheet();
}

Something similar for rows.
Please add an entry to the CHANGELOG under "fixed".
Many thanks!

This is fixed now


Design breaking changes

Should we re-evaluate the thing we consider as breaking changes?
I had some feedback concerning the new design of the disabled input and input validation.

New disabled styles were not technically breaking, but visually. Should this have been a breaking change?
The information was there, but projects are not happy with the change
Versioning rules are just about code, not about design. Create the possibility to add a major version even if the code is not breaking
Projects are not happy with the current disabled styles (strikethrough)
We changed disabled styles for accessibility reasons so it's not possible to revert to a non-accessible state, maybe people are also using it wrong
Disabled elements are not accessible either way
Disabled buttons were implemented a little wrong, so the design change broke their page. A minor change resulted in a lot of work. With a major change, expectation would have been different
We could not predict that this would break with incorrect usage
Depends on how often we do design breaking changes (should not be too often)
Inform Design Team about pending major changes so design changes can be implemented as well

Actions

Take major design changes into consideration when deciding for major release versions


New Game Initiate

Version 1 was better. Version 1.3 won't let me start a new game. I have to exit the app and load it up.

That's my bad, I should have made a changelog mentioning the changes.
You now have to hold the new game button for less than a second to start a new game, this is for avoiding starting a new game by accident.
I'll make a more detailed post on itch, but which button does what and how to press them is explained in the updated tutorial, by pressing the '?' button.
Hope this helps.

I find it confusing that i have to hold the button even after I won a game. There is not much more to do after winning a game than to start a new game, no?

I think this would be much more intuitive if the button graphics changed to a "pressed" state when you tap it, with a little button sound effect maybe.  that would encourage experimentation, I feel.

You could choose to replay the same deck or copy the seed code?
I think it would be helpful if there was a visual indicator around the button to show the press-and-hold timer action.


ARTEMIS-2317 Reuse file buffer wrapper instances to reduce allocations

While measuring throughput of Page::read introduced by https://github.com/apache/activemq-artemis/pull/2633 I'm not getting the same performance of the original mmap read due to the increased allocation rate: reusing the file buffer wrapper makes the new Page::read on par (but without long TTSP issue) of the original version.

The speed of Page::read has been measured by a handrolled test added on org.apache.activemq.artemis.tests.unit.core.paging.impl.PageTest:
   @Test
   public void testReadSpeedWithNIO() throws Exception {
      recreateDirectory(getTestDir());
      NIOSequentialFileFactory factory = new NIOSequentialFileFactory(getTestDirfile(), 1);
      SequentialFile file = factory.createSequentialFile("00010.page");

      Page impl = new Page(new SimpleString("something"), new NullStorageManager(), factory, file, 10);

      Assert.assertEquals(10, impl.getPageId());

      impl.open();

      Assert.assertEquals(1, factory.listFiles("page").size());

      SimpleString simpleDestination = new SimpleString("Test");

      final int numberOfElements = 10 * 1024 * 1024 / 100;

      addPageElements(simpleDestination, impl, numberOfElements);

      final int fileSize = impl.getSize();

      System.out.println("fileSize = " + fileSize);

      impl.sync();
      impl.close();

      file = factory.createSequentialFile("00010.page");

      file.open();

      impl = new Page(new SimpleString("something"), new NullStorageManager(), factory, file, 10);

      final int tests = 10;
      final int times = 100;
      NullStorageManager manager = new NullStorageManager();

      for( int t = 0; t<tests;t++) {

         long totalTime = 0;

         for (int m = 0; m<times;m++) {
            final long start = System.nanoTime();
            List<PagedMessage> msgs = impl.read(manager);
            final long elapsed = System.nanoTime() - start;
            totalTime+=elapsed;
            Assert.assertEquals(numberOfElements, msgs.size());

            Assert.assertEquals(numberOfElements, impl.getNumberOfMessages());

            for (int i = 0; i < msgs.size(); i++) {
               Assert.assertEquals(simpleDestination, msgs.get(i).getMessage().getAddressSimpleString());
            }
         }

         final long totalBytes = fileSize * times;
         final long megaPerSec = (totalBytes * 1000L)/ totalTime;
         System.out.println(megaPerSec + " MB/sec");
         System.gc();
      }

      impl.delete(null);

      Assert.assertEquals(0, factory.listFiles(".page").size());
   }

Although it is a naive one, this test stress the best case scenario of Page::read ie when a file page is already into the OS page cache.

@wy96f Please take a look :+1:

@clebertsuconic Please don't merge it yet: I'm working on a test failure: I will add both test and final solution first

I've addressed the test failure, now it should be ok
@wy96f thanks for the review! Let me know if now the performance is much similar to the original version :+1


googlechrome.fireeye issue on install

googlechrome.fireeye v2<IP_ADDRESS>
googlechrome.fireeye package files upgrade completed. Performing other installation steps.
WARNING: 'C:\Program Files\Google\Chrome\Application\chrome.exe' does not exist. If it is not created the shortcut will
not be valid.
ERROR: The term 'C:\Program Files\Google\Chrome\Application\chrome.exe' is not recognized as the name of a cmdlet, funct
ion, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path i
s correct and try again.
The upgrade of googlechrome.fireeye was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\googlechrome.fireeye\tools\chocolateyInstall.ps1'.
 See log for details.

Looks like Chrome 32-bit is installed and not 64-bit that this is looking for.

Hi @zaqes,  I believe we have an update for this. Please give it another shot and let me know if you still have issues.

This still an issue I'm afraid.
You have googlechrome.fireeye v2<IP_ADDRESS> installed. Version 2<IP_ADDRESS> is available based on your source(s).
Progress: Downloading googlechrome.fireeye 2<IP_ADDRESS>... 100%
googlechrome.fireeye v2<IP_ADDRESS>
googlechrome.fireeye package files upgrade completed. Performing other installation steps.
WARNING: 'C:\Program Files\Google\Chrome\Application\chrome.exe' does not exist. If it is not created the shortcut will
not be valid.
ERROR: The term 'C:\Program Files\Google\Chrome\Application\chrome.exe' is not recognized as the name of a cmdlet, funct
ion, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path i
s correct and try again.
The upgrade of googlechrome.fireeye was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\googlechrome.fireeye\tools\chocolateyInstall.ps1'.

@zaqes,
Can you try uninstalling googlechrome.fireeye and googlechrome packages, then install the latest version of googlechrome.fireeye?

That worked, thanks!


Fix Filter Values By Type solution

Replaced the usage of is_integer/1 and is_float/1 with is_number/1
Handled empty list for test case

Thank you!


Sidekiq 4.x scope side effects

I was doing a code review of this gem before adoption and I noticed that there isn't any code to clear thread locals between sidekiq jobs.  Since sidekiq 4.x (and maybe earlier?) reuse Processor threads to run more than one job (https://github.com/mperham/sidekiq/blob/master/lib/sidekiq/processor.rb#L66-L68), if I use Rollbar.scope! to mutate the current scope I think there may be side effects for the next job.  I would expect to see sidekiq middleware that calls Rollbar.reset_notifier! before calling yield.
Note that the easiest way to test this would be to run sidekiq with only one Processor thread.

Hey @ryansch, thanks for creating the issue, we'll take a look at this. Will Rollbar.scoped help you with your issues? You can always create a new notifier with Rollbar.scope and use the returned object instead of using Rollbar.

We have a multi-tenant app.  I'd much rather be able to create a piece of sidekiq server middleware to inject the current tenant information into the scope with Rollbar.scoped! instead of having to pass scope objects around.  We have many workers that call different service objects and it's infeasible to change all that code.
We're evaluating rollbar while we're currently using honeybadger.  With their stuff I can just call Honeybadger.context and it works great.  I'm looking for the same experience here.

Thank you very much for the information @ryansch, we'll try to fix this as soon as possible. We'll contact you in this issue.
Thanks!

Hi @ryansch, we've fixed and closed this issue. Thank you very much for reporting this, since it was a real bug.
We hope you integration with Rollbar is successfuly, and don't doubt in open any other issue if you have questions, problems with it or you want to purpose something new, improvements, etc...
Thanks again.


MODCLUSTER-456 Fix compilation on JDK 6

Jira
https://issues.jboss.org/browse/MODCLUSTER-456

Triggering build using a merge of e0ed2b9d3c70d8987cec98e7fdd9669ea836eadb on branch master:
Private: https://jenkins.mw.lab.eng.bos.redhat.com/hudson/job/mod_cluster-1.3.x-pull-player-executor/

Build 146 outcome was SUCCESS using a merge of e0ed2b9d3c70d8987cec98e7fdd9669ea836eadb on branch master:
Private: https://jenkins.mw.lab.eng.bos.redhat.com/hudson/job/mod_cluster-1.3.x-pull-player-executor/146
Public: https://hudson.jboss.org/hudson/job/mod_cluster-1.3.x-pull-player-executor/146


SDK - Tx Storage & Ark SDK func update

This adds storage for transactions and vtxos into SDK sqlite storage.
This refactors ark sdk api by adding GetTransactionEventChannel which can be used to listen for new events, it replaces PaymentNotification that was discussed before, this to be closed.
It also updates GetTransactionHistory to read txs from db.
Main idea is to add listenForVtxos function that polls the server every 1(or smtn diff) seconds for new VTXOs,
Func listenToVtxoChan will listen for new vtxos and with help of current state in db will create tx history by using existing
vtxosToTxsCovenant/vtxosToTxsCovenantless funcs.
I pushed changes to the same PR in which i emphasised potential bug in how we handle covenant transaction history, PSB:
Although, this could chain in near future, ATM, async(pending) payments are related to BTC chain(covenantless)
while regular(NON-PENDING) payments are related to Liquid chain(covenant).

This adds test case, for covenant GetTransactionHistory, that check bellow scenario(bellow u can see
expectations as well):
1. before onboarding => empty history
2. after sending sats to the boarding address => 1 PENDING boarding tx
3. after claiming the boarded funds => 1 NON-PENDING boarding tx
4. after sending N sats to bob => 1 NON-PENDING sent tx, 1 NON-PENDING boading tx
Test is failing cause current code seems that it only supports BTC chain as it ignores non pending payments, here.
For covenant GetTransactionHistory returns only boarding txs.

@bordalix let me know if u agree that this is bug.
@tiero @louisinger @bordalix @altafan please review.

is the title correct? Only touches the covenant version? We should mention a new big feature like adding a SQLite storage. Also how that will impact WASM wrapper?

@sekulicd we made some changes to covenant's GetTransactionHistory() yesterday.
Please merge master into your branch and check if error persists.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In 6476980, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 5362 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in 5f4b027.


New install - 4.19.1 on Centos Stream 9, database upgrade fails.

ISSUE TYPE

Bug Report

COMPONENT NAME

Core startup

CLOUDSTACK VERSION

4.19.1

CONFIGURATION

Separate database server:
mysql> select @@version
-> ;
+-----------------+
| @@version       |
+-----------------+
| 10.5.22-MariaDB |
+-----------------+
1 row in set (0.00 sec)
OS / ENVIRONMENT

CentOS Stream 9
SUMMARY

First startup, system detects that database is not correct revision ...
2024-09-06 22:05:47,734 INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) (logid:) DB version = 4.0.0 Code Version = <IP_ADDRESS>
2024-09-06 22:05:47,735 INFO  [c.c.u.DatabaseUpgradeChecker] (main:null) (logid:) Database upgrade must be performed from 4.0.0 to <IP_ADDRESS>
2024-09-06 22:05:47,751 DEBUG [c.c.u.DatabaseUpgradeChecker] (main:null) (logid:) Running upgrade Upgrade40to41 to upgrade from 4.0.0-4.1.0 to 4.1.0
During the upgrade of the database schema, there is an invalid operation attempted (altering a foreign key column) which is not allowed unless you turn of foreign key constraints.
2024-09-06 22:05:47,796 DEBUG [c.c.u.d.ScriptRunner] (main:null) (logid:) alter table storage_pool modify id bigint unsigned AUTO_INCREMENT UNIQUE NOT NULL
2024-09-06 22:05:47,817 ERROR [c.c.u.d.ScriptRunner] (main:null) (logid:) Error executing: alter table storage_pool modify id bigint unsigned AUTO_INCREMENT UNIQUE NOT NULL
2024-09-06 22:05:47,817 ERROR [c.c.u.d.ScriptRunner] (main:null) (logid:) java.sql.SQLException: Cannot change column 'id': used in a foreign key constraint 'fk_storage_pool_details__pool_id' of table 'cloud.storage_pool_details'
2024-09-06 22:05:47,819 ERROR [c.c.u.DatabaseUpgradeChecker] (main:null) (logid:) Unable to execute upgrade script
java.sql.SQLException: Cannot change column 'id': used in a foreign key constraint 'fk_storage_pool_details__pool_id' of table 'cloud.storage_pool_details'
at com.cloud.utils.db.ScriptRunner.runScript(ScriptRunner.java:185)
STEPS TO REPRODUCE

EXPECTED RESULTS

I expected the web site to come available.

ACTUAL RESULTS

HTTP ERROR 503 Service Unavailable

URI: | /client/
-- | --
503
Service Unavailable
-

@rongage , centos stream is not explicitely supported by ACS. I do not know how far forked it is from the likes of Oracle Linux 9, Alma Linux 9 and Rocky Linux 9, which are all supported. It could be that it is a matter of a package version to control better (the DB for instance). The mariaDB I think, is only supported for version 5 ; MySQL 8.0 (or equivalent compatible DBMS).

I tested mariadb-server 10.6.18-0ubuntu<IP_ADDRESS>, it worked well.
10.6.18 is a more recent release (released in May 2024) than 10.5.22 (released in August 2023)
can you please try a newer version ? @rongage


Implementing DevOps Practices for Project Efficiency and Reliability

[x] Continuous Integration Setup: Establish a CI system (like GitHub Actions or Jenkins) to automatically build and test the codebase upon each commit or pull request.
[ ] Continuous Deployment/Delivery: Set up a CD pipeline to automate the deployment of code changes to staging and production environments.
[ ] Containerization: Utilize Docker to containerize the application, ensuring consistency across different environments and ease of deployment.
[ ] Infrastructure as Code (IaC): Implement IaC using tools like Terraform or AWS CloudFormation to manage and provision infrastructure through code.
[ ] Monitoring and Logging: Integrate monitoring tools (like Prometheus or Grafana) and set up robust logging for the application to track performance and troubleshoot issues.
[ ] Performance Testing: Incorporate performance testing in the CI/CD pipeline to ensure scalability and stability under load.
[ ] Documentation: Update the project documentation to reflect the new DevOps processes and guidelines.

Progress Update on Implementing DevOps Practices
Task Completed: Continuous Integration Setup

Successfully created and tested GitHub Actions workflows for both the Django backend and React frontend.
Integrated Docker to containerize both applications, ensuring consistency across different environments.
Set up Docker Compose to facilitate local development and testing of the multi-container application.

Next Steps:

Moving forward to Continuous Deployment/Delivery (CD).
Plan to set up a CD pipeline using GitHub Actions to automate the deployment process.
Aim to include manual triggers in the workflow for greater control over deployment timings.
Will ensure secure handling of credentials and sensitive information using GitHub Secrets.

This marks a significant milestone in enhancing the project's DevOps capabilities and streamlining the development process. Looking forward to the next phases of implementation.

Progress Report for Issue 14
Tasks Completed:

DevOps Principles Implementation: We explored various workflows suitable for the project's Django backend and React frontend. Discussed and selected GitHub Actions workflows for both parts of the application.
Dockerization: Successfully created Dockerfiles for both backend and frontend, and verified their functionality.
GitHub Actions for CI/CD: Set up and tested GitHub Actions workflows for continuous integration and deployment. This included linting, building, and pushing Docker images to a registry.
Database Configuration for Heroku: Configured the project to use PostgreSQL in the production environment on Heroku, while retaining SQLite for development. Ensured proper installation and setup of psycopg2-binary for PostgreSQL support.
Heroku Deployment: Addressed several deployment issues, including database migrations and environment-specific settings.
CSRF Error Resolution: Investigated and provided detailed steps to resolve a CSRF verification issue, emphasizing the importance of CSRF token handling in templates and AJAX requests.

Next Steps:

Continue monitoring and troubleshooting any deployment or runtime issues on Heroku.
Implement additional features or improvements as required.
Regularly update and maintain the CI/CD workflows.

Additional Notes:

Emphasis on testing both locally and in the Heroku environment to ensure stability and performance.
Consider implementing additional security measures and optimizations for both backend and frontend.


[Feat]: Add ctrl + enter combined key press to have a task done

This Pull Request closes this issue

excellent, merged!


Port to new deps, ng2, sui 2

Now that angular2 is released we should port cayenne over to ng2, since it's a small codebase it shouldn't be too bad. We should also take the time to factor out game-specific content and keep it as bare as possible, but extendable.

Angular 2 won't be happening any time soon. We need autobahn to be compatible with TypeScript and Angular2 (right now it's a huge pain to get working) or get an Autobahn replacement which is compatible. I'll stick with ng1 for now and at least bump the other deps

It appears that nwjs has fallen quite behind in keeping up to date with Chromium. Also its documentation is absolutely atrocious. Gonna close this and think about a better roadmap.


Update map docs

This PR updates the map docs for processing text to include return_tensors="np" to make it run faster (see #4676).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Resizable UART, Auto-load Prebuilt ROM, UART Send Byte

Makes the UART Terminal resizable (rows, cols)
Fixes EOL for UART Terminal (\n is treated as a new line)
Add "Send Byte" with a Decimal value (useful for testing)
Introduce ?r=latest parameter

this is being introduced lazily for wider testing
once verified, we can dynamically update the URL to reflect the selected ROM

handful of CSS adjustments

Thanks a lot for this PR! Looks good to know, I left a small comment about swapped columns and rows

Thanks, it's merged!


puppet-server support

Hi all,
are there plans to implement support for running tests with puppet-server and beaker?
Or is that support already there and am i just completely blind?
reason for asking is that an issue was reported [0] on my Logstash module.
Thanks
[0] https://github.com/elasticsearch/puppet-logstash/issues/193

I'm going to have to do more investigation here and get back to you.

@electrical
We do test puppet-server using beaker.
Unfortunately the state of documentation for acceptance testing puppet-server is somewhat poor but we are aware of the need to improve it and have it scoped in upcoming work:
https://tickets.puppetlabs.com/browse/SERVER-317
This is our current documentation:
https://github.com/puppetlabs/puppet-server/blob/master/acceptance/README.md
We'd really appreciate any feedback or improvements you might have to suggest :)

Closing this out, you can follow up through the doc and to @waynr .


lint issue in azure core

The pipeline task PyLint failed In this PR: https://github.com/Azure/azure-sdk-for-python/pull/11949.
I found below in the task detail log:
************* Module azure.core.polling._poller
azure/core/polling/_poller.py:34: [W0611(unused-import), ] Unused Tuple imported from typing
It seems will block all PR, shall we have an urgent fix on it?

@scbedd is there a reason why pylint is failing here, and was not failing before?

@lmazuel I spent a couple hours yesterday trying to track this down. This does not repro locally, and as far as I can tell the environment dep and python versions are identical.
Submitted a PR that has since been supplanted by Xiang, but could have unblocked CI with an approval and a merge. Check teams channel for further discussion.
I'm closing this issue in favor of this other issue that details the problems.


⚠ Avatar API Endpoint has degraded performance

In fef05e2, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4311 ms

Resolved: Avatar API Endpoint performance has improved in 5932ea0 after 5 minutes.


Cow: Watch sjp.ac.lk

Cow requests the watch of the watch_keyword sjp\.ac\.lk. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sjp\.ac\.lk has been seen in 1 true positive, 1 false positive, and 0 NAAs.

Approved by Ollie in Charcoal HQ