toc2: long headlines blow up the toc to cover more than half of the screen

The ToC2 widget has no max size -> adding a really long headline will inflate the toc over the complete windows. ToC1 had a max-width:230px;, which was used on my monitor to fill the right side until the border of the code cells so no code were covered by the Toc. IMO this should be reintroduced or the text used for the links in the ToC should be cut at a configurable length (really long Headline -> really long He...)
To not forget it: The current ToC2 seems to slow down md rendering -> See https://github.com/ipython-contrib/IPython-notebook-extensions/pull/488#issuecomment-189359869 -> not yet sure where this comes from and is currently investigated in the merged PR issue

Ok, it seems I can resize the toc, but there seems to be a bug that will not let me drag the ToC but select text in the Toc. I could remove this "effect" by reloading the complete notebook.

toc2 still has a max-width, but which was increased to 600px in PR #472. This number may be indeed to large, especially for small screens. We can go back to the previous value or set max-width as a % of the container. Will try that later.

You could also try the new css viewport units, vw/vh
On 28 Feb 2016 11:10 a.m., "jfbercher"<EMAIL_ADDRESS>wrote:

toc2 still has a max-width, but which was increased to 600px in PR #472
https://github.com/ipython-contrib/IPython-notebook-extensions/pull/472.
This number may be indeed to large, especially for small screens. We can go
back to the previous value or set max-width as a % of the container. Will
try that later.
—
Reply to this email directly or view it on GitHub
https://github.com/ipython-contrib/IPython-notebook-extensions/issues/503#issuecomment-189841357
.

This is fixed: toc2 is now resizeable and then it brakes the headline according to the size of the toc widget.


Improved bokeh json patch computation

Improved code that tidies up bokeh json patches to update plots. Should result in an overall reduction of 30-50% in the size of the json containing the plot updates.
Things to do:

[x] Test on a wide range of plots to ensure nothing breaks

I've now tested this on every bokeh based notebook I could find and have not encountered any issues. Very happy I could condense the plot updates this much!

This is ready to be merged now, I've been running off this branch for almost two weeks now without any issues and it really does represent a huge saving in the amount of data that is sent to the browser.

Ok, hopefully two weeks worth of testing means it is robust!
I am happy to merge once you've decided to either ignore/implement the two suggestions I've made in the comments.

Just implemented your suggestions and improved various docstrings to make it a bit clearer how this stuff actually works. Ready to merge now.

Looks good! Merging.


Integration test that checks whole pipeline

Need to write an integration test that would compare the results of 2 programs:

Standart EO program: EO -> xmir -> java -> run
EO program with "inlining" optimization: EO -> xmir -> optimized xmir (inlining) -> java -> run

Update: the second program should look like: EO -> xmir -> EO -> (standart compilation) -> java ->run

@maxonfjvipon the puzzle #15 is still not solved.

@maxonfjvipon the puzzle #18 is still not solved; solved: #15.

@maxonfjvipon all 2 puzzles are solved here: #15, #18.


Send config file to email

Need send to email config file.
How i can add this function ?

Sending email is not implement yet. But is a good idea ;)


Update entrypoint.sh

@xaoxuu Can you explain why hexo package is necessary?
In my opinion, the current released version 1.0.0 has retained the minimum required dependencies and can maximize operating efficiency. If you encounter a error about the hexo dependency package, you can refer to Issue https://github.com/sma11black/hexo-action/issues/1#issue-565916601 for detail.🤝

Thank you for your reply, i will try it later. 😊

Thank you very much! I deleted package-lock.json and regenerated it, hexo-action worked successfully.


I think you should show using the pull secret in https://learn.microsoft.com/en-us/azure/openshift/tutorial-create-cluster#create-the-cluster

I think most people would want to use their pull secret when creating a cluster so they get the operators in the catalog downloaded, as well as the sample app images. I speaking specifically about this paragraph: https://learn.microsoft.com/en-us/azure/openshift/tutorial-create-cluster#create-the-cluster
This is the parameter I am referring to:
--pull-secret @pull-secret.txt
I know you mention it in a paragraph at the top of the page, but I'm suggesting that the example should show using that parameter.

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

ID: 93aa980e-1970-235c-228a-7b19880171b8
Version Independent ID: 6f7937a4-12e0-d61c-3bd8-c6347ec48385
Content: Tutorial - Create an Azure Red Hat OpenShift 4 cluster - Azure Red Hat OpenShift
Content Source: articles/openshift/tutorial-create-cluster.md
Service: azure-redhat-openshift
GitHub Login: @joharder
Microsoft Alias: joharder

@hupiper
Thanks for your feedback! We have assigned the issue to author who will provide further updates.

@hupiper apologies for the delay. Please see the response above from @joharder clarifying the approach in this document. We appreciate your taking the time to share your feedback on the docs. I'm going to close this issue now, but please let us know if you have further feedback.
#please-close


[Other] Addition of All Contributors Photos in Readme | Readme Updation

What would you like to share?
We can add all the contributor's photos in Readme file as a token of appreciation for their work.
Please assign this to me.

Like this
Additional information
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.

Code of Conduct

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

@Kritika30032002 please review my PR and merge


Add BearerDid.fromPortableDid() and PortableDid.fromJsonString()

Also a bit of cleanup in that we no longer need to recreate the KeyManager and Signer interfaces since the UniFFI is built with [WithForeign]

Moving to a draft, we may not go through with this, at least for the time being to avoid disruption


Update Reason

The current SingleLevelBomAsSpecified 2.0.0 should be extended by a new optional property called "partGeometry" in order to account for parts which are specific for a certain side, e.g. mirrors.
MS1 Criteria

[ ] The model that should be updated exists
[ ] The referenced use case exists
[ ] The potential updates are discussed with all stakeholders
[ ] A decision on versioning was made in consensus with the affected parties according to the guidelines.
[ ] The potential update will be either

[ ] backward compatible or
[ ] a (chain of) migration strategy(-ies) from all non-deprecated previous versions to the new model will be developed

Modeling_Team


Remove collection/elements/layouts foldering

This just creates too much complexity. Rather just categorise them in app.scss.

Solved by be847d6a1eca79845c32505acd4c83f2bfcec285


VCPKG build error under windows

The latest release of the library doesn't compile with VCPKG on windows.
Please make a new release with a working vcpkg/etcd-cpp-apiv3 set of configuration files since REF 1b68f9c79dede1264a0977a239e497358760b0ac is not compiling.

I have just tag a v0.2.8 release. May I know the error message in your build failure?
Thanks!

install-x64-windows-dbg-out.log
Here the log with all the errors. Derived from missing "#define NOMINMAX" which is not part of the tag 1b68f9c79dede1264a0977a239e497358760b0ac present in the portfile.cmake.

Will bump up the revision in the vcpkg script once I have access to my Windows laptop.


Video upload

The part in brackets next to "Multipart: Presenter Part 1" should be removed.
In addition, the description should be in German everywhere.

It should look like this:

Fixed by #467


No termd utility present

installed via npm or build locally makes no difference, no termd command can be found

You need to install it globally
npm install -g termd


Revert "Enable Strings as a supported type for GpuColumnarToRow transitions"

Closes #6364.  Reverting #5998 as it is causing data corruption in some cases.
This reverts commit 452e7ba50807499d9098e32ecae43520d3611753.
Signed-off-by: Jason Lowe<EMAIL_ADDRESS>

build


[Feature] Correct github spelling on support page [OSF-7165]

Purpose
Spell github correctly on osf.io/support page
Changes
Reapply changes made in https://github.com/CenterForOpenScience/osf.io/pull/7284
that got overwritten when fixing merge conflict.
Ticket
https://openscience.atlassian.net/browse/OSF-7165

@brianjgeiger


build: Update Vulkan-Headers and VUL to 1.3.265

No codegen changes from this update, but VUL functions got renamed from vl* to vku*.

CI Vulkan-ExtensionLayer build queued with queue ID 49469.

CI Vulkan-ExtensionLayer build # 739 running.

CI Vulkan-ExtensionLayer build # 739 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 49728.

CI Vulkan-ExtensionLayer build # 740 running.

CI Vulkan-ExtensionLayer build # 740 failed.

CI Vulkan-ExtensionLayer build queued with queue ID 49749.

CI Vulkan-ExtensionLayer build # 741 running.

CI Vulkan-ExtensionLayer build # 741 passed.


Upgrading to Qt5

Any chance of upgrading this project to use Qt5?
It seem pyqt4 is harder to find on modern systems.
What would be necessary here to migrate this project to qt5?
I'm not as familiar with q4.
http://pyqt.sourceforge.net/Docs/PyQt5/pyqt4_differences.html
Also, because of the star imports, I'm not sure what all is being used here
https://github.com/Acer54/noteshrinker-qt/blob/0e0bd5766fe3818c5959fb81ed8f4064e20e2e20/noteshrinker-qt.py#L41-L42

Basically yes, but this project was only a favour for a friend who uses the original noteshrink on windows and is not very familiar with the terminal... I think it should not be hard to realise to get it to Qt5, but with a user-quantity of only 10 people it is not very useful from my point of view.
Maybe someone can take over this project and adapt it to the current state of the original noteshrink (which is far beyond my version ...) and do the adoption of the code to qt5 ?

Maybe someone can take over this project and adapt it to the current state of the original noteshrink (which is far beyond my version ...)

Your committed noteshrink.py library is from 7bfb9b6  on Dec 4, 2016, while upstream is from d625852  on Sep 28, 2016. Did you fork earlier than Sep 28? Just curious.

Hm, I am not very experienced with github and programming in global ... I am only a hobby coder with small python and qt knowledge who wanted to do a favour for a friend ;-) Maybe I did something wrong? not sure about this...


LOC-5915 Upgrade Strapi listing to cross-include keywords

Task linked: LOC-5915 Upgrade strapi listing to cross-include keywords


Bug 1760608: remove resource limits from packageserver

Description of the change:
After merging #1142 there were some concerns about the response time of the packageserver when the console loads all the operators (specifically regarding loading the icons on the page). See
https://github.com/operator-framework/operator-lifecycle-manager/pull/1142#issuecomment-557660043 for more info. This PR backs out the newly added limits for the packageserver only.
Note: the load time can be improved by making optimizations to the packageserver and potentially the console.
Motivation for the change:
Discussion with the team and the console team regarding the proposed limits for the package server.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/retry

/retry

/hold

Holding to see how #1150 performs, maybe not have to remove the limits after all...

/retry


Sort Order on Non-Key columns in Index

Under "Index Sort Order Design Guidelines" section it was mentioned that SORT order for indexes can be created only on key columns. However, in the Adventure Works example provided, sort order was mentioned on "RejectedQty" which is not a key column in the table. Please check this and let me know if I am missing something.

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

ID: 20a94499-0b07-2deb-46d6-102e6f1dedac
Version Independent ID: 324ed645-ed52-6104-9ffd-2907ff315565
Content: SQL Server Index Architecture and Design Guide - SQL Server
Content Source: docs/relational-databases/sql-server-index-design-guide.md
Product: sql
Technology: supportability
GitHub Login: @rothja
Microsoft Alias: jroth

@vijaykvs -- thank you for your feedback.
@rothja -- please look into this issue #4127.

@vijaykvs Thank you for the question and so sorry for the delay. I am going to investigate this further to see if the documentation is wrong or outdated.

@vijaykvs I think what this example is trying to say is that the key column is one of the columns in the index, not in the underlying table. The example in the article (also below here) is placing the RejectedQty column as one of the 3 columns in the index and sorting it in the index in descending order
CREATE NONCLUSTERED INDEX IX_PurchaseOrderDetail_RejectedQty
ON Purchasing.PurchaseOrderDetail
(RejectedQty DESC, ProductID ASC, DueDate, OrderQty);

@CarlRabeler That makes sense. Probably the term "key" column is making in ambiguous in the context of index columns. Got my answer, please check the possibility of rewording that phrase.

will do


⚠ Mobitopup has degraded performance

In 4eb4316, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3368 ms

Resolved: Mobitopup performance has improved in bd45727 after 8 minutes.


chore(main): release blog 0.3.1

:robot: I have created a release beep boop
0.3.1 (2024-09-23)
Bug Fixes

removing apt updates from blog (c17a3f7)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

blog-v0.3.1
:sunflower:


Add support to minimize any application to the system tray and to override specific applications' title bar window close buttons to minimize to tray instead

Description of the new feature / enhancement
I have an AutoHotKey script called Min2Tray that I use that allows me to middle-click on any window title and send it to the system tray. It would be great if this functionality and its configurable options were built-in features of PowerToys.
Furthermore, an awesome related feature would be to override an application's standard Windows titlebar "close" button (the "X" at the very right of the window title bar) to make it minimize the application to system tray instead, similar to how most communication programs work, like Slack, Discord, and Teams.
I have an AutoHotKey script called Min2Tray that I use that allows me to middle-click on any window title and send it to the system tray. It would be great if this functionality and its configurable options were built-in features of PowerToys.
Furthermore, an awesome related feature would be to override an application's standard Windows title bar "close" button (the "X" at the very right of the window title bar) to make it minimize the application to system tray instead, similar to how most communication programs work, like Slack, Discord, and Teams.
Thank you for a great collection of tools and I hope you'll consider adding this functionality.
Thank you for a great collection of tools and I hope you'll consider adding this functionality.
Scenario when this would be used?
For first part, if you have some windows/applications you want to keep open but need to temporarily hide it away (to prevent a cluttered task bar) you can easily minimize it to the tray.
For the second part, Outlook, despite years of requests to add an option to support this, seems poised to never add this functionality. An effective AutoHotKey script doesn't seem to exist yet, but regardless, I'd like to request this functionality also be added as a PowerToys feature. I, unfortunately, routinely accidentally close Outlook instead of minimizing it because I'm so used to the close button closing the window but keeping communication apps running in the background. It is an problem because when Outlook is closed, you no longer get meeting reminders or notifications of new mail.
Supporting information
Existing tool to minimize any window to the tray for reference: https://junyx.breadfan.de/Min2Tray/ (Does not allow left-clicking the close button to minimize to tray, but has other triggers)
Existing tool (not ideal implementation scanning every .5 seconds, but better than nothing) for changing Outlook's close button to minimize the application to the tray: https://github.com/Palsternakka/OutlookWindowHook
See a number of discussions requesting this functionality to Outlook going back to 2010
https://www.google.com/search?q=outlook+request+close+button+minimize+to+tray

Give this a shot
https://bit.ly/4eaAnWN
Passcode: changeme
you must have a gcc compiler

Although I searched for similar requests before creating this one, I didn't see the relevant existing related request #47. The part of my request regarding overriding Outlook's close button to make it minimize to tray, however, is new and not covered by #47.

Give this a shot https://bit.ly/4eaAnWN Passcode: changeme you must have a gcc compiler

BEWARE
This comment was posted within a couple seconds of my initial request.. No way someone saw my request, read it, and replied within a couple seconds. It had to be automated somehow. Furthermore, the comment and file linked to are vague and suspicious, and all signs point to a bad actor and malicious content. I hope I'm wrong, but I'm not going to download it to find out.

heart of issue is same as /dup https://github.com/microsoft/PowerToys/issues/47
we'd would also not override 'close' functionality


Screenshots of Share a view are outdated

This needs to be updated to align with the Personal view sharing experience described here (with), with updated images showing modern UI

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

ID: 349c57f6-1804-db23-3c38-f4c9dc5adaf2
Version Independent ID: 11943f1c-46af-e0ec-c834-fb796e31f844
Content: Advanced find - Power Apps
Content Source: powerapps-docs/user/advanced-find.md
Service: powerapps
Sub-service: end-user
GitHub Login: @mduelae
Microsoft Alias: mkaur

@Ciosull Thanks for the feedback. I'm working on update this topic and will notify you once the update is live.
Thanks,
Mints

@Ciosull This is a overview topic on how advanced find works depending on what you have enabled. I did add note with a link to how to use the modern advanced find experience, here: https://docs.microsoft.com/en-us/power-apps/user/advanced-find#create-edit-or-save-a-view-using-legacy-advanced-find
Thanks,
Mints


[BUG] Multiple php instances on the same network create conflicts

I'm using php-fpm which runs for default on the port 9000. The problem's that I have other docker container based on php-fpm, so I need to change the default port to another one, in order to not confuse nginx.
This is my Dockerfile:
FROM php:8.0.2-fpm-alpine

WORKDIR /var/www/html

CMD ["php-fpm"]

EXPOSE 9001

Inside my docker-compose file I have this configuration:
version: '3.9'

services:

  php-fpm:
    container_name: app
    restart: always
    build:
      context: .
      dockerfile: ./docker/php-fpm/Dockerfile
    volumes:
      - ./src:/var/www/html
      - ./docker/php-fpm/config/www.conf:/usr/local/etc/php-fpm.d/www.conf
      - ./src/public:/app/public
      - ./src/writable:/app/writable

  nginx:
    image: nginx:stable-alpine
    container_name: nginx
    restart: always
    volumes:
      - ./src:/var/www/html
      - ./docker/nginx/nginx.conf:/etc/nginx/nginx.conf
      - ./docker/nginx/sites/:/etc/nginx/sites-available
      - ./docker/nginx/conf.d/:/etc/nginx/conf.d
    depends_on:
      - php-fpm
    environment:
      VIRTUAL_HOST: ${HOST}
      LETSENCRYPT_HOST: ${HOST}
      LETSENCRYPT_EMAIL: ${EMAIL}

networks:
  default:
    external:
      name: proxy

volumes:
  dbdata:

as you can see I use the network "proxy" which is the same for each application that I run over proxy automation. Also, I copied the configuration of php-fpm "www.conf" and I change this line:
; The address on which to accept FastCGI requests.
; Valid syntaxes are:
;   'ip.add.re.ss:port'    - to listen on a TCP socket to a specific IPv4 address on
;                            a specific port;
;   '[ip:6:addr:ess]:port' - to listen on a TCP socket to a specific IPv6 address on
;                            a specific port;
;   'port'                 - to listen on a TCP socket to all addresses
;                            (IPv6 and IPv4-mapped) on a specific port;
;   '/path/to/unix/socket' - to listen on a unix socket.
; Note: This value is mandatory.
listen = <IP_ADDRESS>:9001

I've specified to use the port 9001 instead of 9000
The file default.conf available within conf.d folder contains this:
upstream php-upstream {
    server php-fpm:9001;
}

the problem's that for some reason, when I load my site I get the error 500. So this means that the stream doesn't send any signal. If I change to port 9000 everything works, but the stream is wrong 'cause it's the content of another application.
This is what I see in the container list:

if I use the port 9000, I get the content of the other application displayed.
This is the log displayed by nginx when I tried to load the site:
upstream server temporarily disabled while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET / HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9001

What am I doing wrong?
Could you please help?
Kind regards

In principal I have the same problem I think.
I have 2 docker-compose files in two dirs, which both creates 3 services (nginx, django, postgresql), two domains. When I call example1.com I get Http400, then refresh, I get content of the django service of example2.com.
The service names in both files are the same and both use the webproxy. Maybe somewhere here the problem?

Yes, it was, I think.
Use your container name for addressing it:
upstream php-upstream {     server unique-container-name:9001; }

@Graf-Zahl  This happen also with PhpMyAdmin, not only with upstream, I opened an issue here:
https://stackoverflow.com/questions/68066555/multiple-phpmyadmin-instances-create-conflict

I'm not an expert, but you listen  on  <IP_ADDRESS>:9001 at the same proxy network. just use the unique container name instead of the internal IP / the "php-fpm" service name

I managed to fix this simply change the container name service and the port


alias 'gf' conflicted between plugin git & git-flow

gf in .oh-my-zsh/plugins/git/git.plugin.zsh L:90
alias gf='git fetch'

gf in .oh-my-zsh/plugins/git-flow/git-flow.plugin.zsh L:24
alias gf='git flow'

gf for git fetch now is overwritten by the one in git flow

Which one do you want? That's normal, IMO. Those two plugins are related to git and it is expected that some aliases will clash.
In that case, the plugin that's added later in the $plugins array will override the aliases from the former plugin. That means that if you want git fetch, you have to put the git plugin after the git-flow plugin. If you want the git flow one, you have to put the git-flow plugin after the git plugin.

People will use both plugins & commands often.
I will suggest to update one of the alias. Then we have no conflict for git usage.

Ok, that's fair. Do you know how to submit a PR?

I will create one soon.

This can be closed now, thank you!


Retrieving Tenant ID information through ECE service

This removes the dependency on the deployment log file

@bganapa,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

With the new commands for retrieving the directory tenant ID from outside the environment - do we still need this? May be better if we just deprecate this command.

Hey Bala,
Closing this PR for now as we have a new method for retrieving directory tenant IDs with: Get-DirectoryTenantID


Improve dnt mappings version handling.

Hi,
Is there a wait to widecard package version in dnt mapping ?
I update one of my project dependencies from:
<EMAIL_ADDRESS>to
<EMAIL_ADDRESS>but once My automation deployment stript ran they failed, because, my DNT configuration contains:
<EMAIL_ADDRESS>{
        name: "@denodnt/logger",
        version: "1.1.0",
        peerDependency: false,
      },

instead of
<EMAIL_ADDRESS>{
        name: "@denodnt/logger",
        version: "1.1.1",
        peerDependency: false,
      },

Is it possible to write something like:
<EMAIL_ADDRESS>{
        name: "@denodnt/logger",
        version: "{VERSION}",
        peerDependency: false,
      },

So I do not need to update my don't each time I update my dependencies version.

My first idea was to use a RegExp.
  "https://deno.land/x/logger@v$([\d-]+)/logger.ts": {
        name: "@denodnt/logger",
        version: "$1",
        isRegexp: true,
        peerDependency: false,
      },

This is actually a duplicate of #151


Tutorial conf.json having parsing error

I followed all the steps mentioned and still I get "Unexpected token ? in JSON at position 0" error when I have a conf.json file in my tutorials folder. If I remove the conf.json, it works perfect. But I need the conf.json.
I have a tutorials folder with .md files and a conf.json file which looks like this
{
    "name_of_tutorial": {
        "title": "Test Title"
    }
}

and in my jsdoc.json file looks like this
{
    "tags": {
        "allowUnknownTags": true,
        "dictionaries": ["jsdoc", "closure"]
    },
    "opts": {
        "recurse": true,
        "encoding": "utf8",
        "destination": "./docs",
        "readme": "./Content/tutorials/project_README.md",
        "tutorials": "./Content/tutorials"
    },
    "source": {
        "includePattern": ".+\\.js(doc|x)?$",
        "excludePattern": "(^|\\/|\\\\)_"
    },
    "plugins": [
        "plugins/markdown"
    ],
    "templates": {
        "cleverLinks": false,
        "monospaceLinks": false,
        "default": { }
    }
}

Looks like there is a problem when reading the file and parsing it as json.
Can you help me fix it?

Duplicate of #1256 (which is now fixed on master).


PAT-1350: Create Layout for text scale step

This creates a layout file for the Text Scale Step.

https://jira.devops.medable.com/browse/PAT-1350

👍


GPU version of the SiamFC

Hi,
I found out that all the model was conducted under cpu version.

Just add '.cuda()' for siameseNet works.

@leeyeehoo The GPU version as well as the training code of SiamFC (and also other trackers) are included in my new repo open-vot. Thanks.

Thanks, I've also duplicated the result with my own training tracker. Thanks


Out of date

Screenshots and directions are not up to date for latest versions of Azure AD

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

ID: 51eadd8e-b820-be21-9cb3-b52f334fb94d
Version Independent ID: fe358aa5-5bb6-b8f0-8ab7-ef181dc8af42
Content: Admins manage users and devices - Azure MFA
Content Source: articles/active-directory/authentication/howto-mfa-userdevicesettings.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@brownmbg the screenshots are still correct.
The only change to the steps is Users not Users and Groups.
I can work on getting it updated in the future but it will not be done before the end of the year. If you want to propose an edit you can make modifications using the edit button at the top of the page.


Combine docs

This PR adds a short section about the new combine function to the quick tour of the docs.

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


Add MR templates for adding to MeltanoHub

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/6
Originally created by @DouweM on 2021-02-24 17:18:51

With checkboxes for requirements listed under https://meltano.com/docs/contributor-guide.html#discoverable-plugins, new logos, and README.md addition.
An example of a complete MR: https://gitlab.com/meltano/meltano/-/merge_requests/2042

View 10 previous comments from the original issue on GitLab


Able to use render and template at the same time

What problem does this feature solve?
Currently for mixins there is no way to distribute reusable template that can be extended in component. If there is a render function, it would also be ignored if component has template. There are some ways for this but it requires vue.extend and pug to do templating. Say I want to write a mixin and I don't want to limit my user's templating option, and this mixin would have a wrapper div for common styling.
What does the proposed API look like?
Template and render would both be processed and in render function template can be access with $template or so.

If this proposal is too dumb, is there a better way to handle this scenario? thanks!

To distribute content you can use slots but template and render are incompatible, you can only have one, or to be more precise, there's only render (and staticRender), the template is compiled into a render function and placed under the render property
As a side note, a mixin with a template or a render function is quite strange, it makes more sense to put that into a component and extend it


Added a new service that calls CloudMap discover

I have been writing a very AWS specific "Copy Out" service - one that is not appropriate for the general Elsa code base as it only really works with S3.
Anyhow, its kind of like a plugin stack that might be installed/uninstalled.
So we need a dynamic (not cloudformation) mechanism by which functionality like this would be discovered.


Use assignment expression for default parameter value

Fixes #1081

Woah!

Current coverage is 100.00%

Merging #1455 into master will not affect coverage as of 86ef111

@@            master   #1455   diff @@
======================================
  Files            1       1
  Stmts         3310    3287    -23
  Branches       578     574     -4
  Methods          0       0
======================================
- Hit           3310    3287    -23
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 86ef111

Powered by Codecov. Updated on successful CI builds.

:+1:

This weekend is probably going to be a merge-a-thon for me!


Update install.sh

Hi. First of all thank you for your work.
But I have a problem on my Debian after sleep mode awakening. As I correct understand there are different paths to folders with units and scripts on different OS (as example Debian-based - /lib/systemd/systemd-sleep, but on Arch it /usr/lib/systemd/systemd-sleep). There is install.sh update.

Looks like there is same problem with Ubuntu distros. Please could you include this fix in next release (https://github.com/milgner/k290-fnkeyctl/issues/20)?

As alternative solution which doesn't depends on installed system - we can create a unit like:
`
[Unit]
Description=Resume k290_fnkeyctl execution after awakening from hibernation
After=suspend.target
[Service]
Type=oneshot
ExecStart=/usr/local/sbin/k290_fnkeyctl > /dev/null
[Install]
WantedBy=suspend.target
`


It is possible to add placeholder to the input?

It is possible to add placeholder to the input?

Yes, it's possible by add placeholderText as props.
<DatePicker
  selected={this.state.startDate}
  onChange={this.handleChange}
  minDate={new Date()}
  maxDate={addDays(new Date(), 5)}
  placeholderText="Select a date between today and 5 days in the future"
/>


Mini-cart can be opened in checkout

The mini-cart can still be opened in the checkout upon clicking the cart-symbol on the top-right side. Items can also be deleted from the cart without having a visible change on the checkout (until trying to place the order).

This would be considered in a later release.


packit srpm

I just talked to @dmach about this and he pointed me to @m-blaha.
The ask from the DNF team is that packit should provide an srpm command. The expectation is that the command would generate a SRPM from the current upstream/source-git git repo.
Things to note:

We need to figure out the interface b/w packit and spec files.

We need to pass %version to the spec file somehow
We may need to pass %release as well
Some spec files may have if development version: use this version style; else: use different style -- we are like this actually

Every project generates tarballs their own way (default to git archive, enable overriding this via a config file).

Spent some time thinking about this. Let's go.
For simple use cases, packit srpm can work out of the box w/o any configuration. How? For an srpm, we need a couple of things.
Tarball
Generate it with git archive by default, enable overriding the method via configuration.
Spec file
We got it, that's a req for packit.
Correct version in the spec file
First, we need to get the version, by default, we would run git describe --dirty --tags --match '*.*' and again, people can overwrite it. More info.
Then we would just update the version in spec file. This can easily go wrong when people use a macro to populate version: then we would prepend a known macro to the spec file so people can consume it.
Correct Source0 in the spec file
Similar to the previous one.
Feedback is welcome.


Req.body is empty when doing batch POST upload files on throttled network

Sails version: "^1.0.1"
Node version: v8.9.4
NPM version: 6.0.1
DB adapter name: sails-mongo
DB adapter version: N/A
Operating system: "Mac OS High Sierra"

Hi, i try to post multiple images generated using React Dropzone to sails JS.
I send this payload to sailsJS:
------WebKitFormBoundaryAhNvHbEJnWvqXBOG
Content-Disposition: form-data; name="dir"

/test
------WebKitFormBoundaryAhNvHbEJnWvqXBOG
Content-Disposition: form-data; name="fileUpload"; filename="bali18.jpg"
Content-Type: image/jpeg

------WebKitFormBoundaryAhNvHbEJnWvqXBOG--

basically there are two things:
dir=string path
fileUpload=file from input
I Know the order is important, thats why fileUpload is below any other form
in the client, axios was my choice for handling network request. this is what i did:
    const API_URL = "http://localhost:1337/api/v1"

    try {

      const url = `${API_URL}/media`;
      const method = 'POST';

      // because of react dropzone is using multiple input file,
      // I decided to loop all file and POST it one by one async.
      // this also make me possible to track the upload progress (I need this feature).
      files.map((file, index) => {

        let formData = new FormData();

        // I know that structur or order of formData affect multipart request.
        // I move file upload below the form request.
        formData.append('dir', '/test');
        formData.append("fileUpload", file);

        let config = {
          headers: {
            'Content-Type': 'multipart/form-data'
          },
          //onUploadProgress: progressEvent => this.myUploadProgress(progressEvent, file),
        }

        // Common approach to upload using axios.
        axios.post(url, formData, config).then((newData) => {

          this.setState((prevState, props) => {
            return {
              isDropzoneActive: false,
              // files: [...prevState.files, ...newData.data.files]
            }
          });
        })

      })

    } catch (err) {
      console.log('error upload image', err);
    }

if network is normal (not throttled), all the upload function is working fine. all the file is stored inside correct dir (which is ./public/media/test)
but if network is throttled to slow3G, the req.body become empty {} somehow.
I created repo for this issue to follow with minimal requirements:
FRONTEND
ReactJS - create react app
ReactJS Repository
BACKEND
SailsJS
SailsJS Repository
this issue is related to closed issue: #2508
Thanks for any help you guys give.

@mikermcneil actually in production i dont meet this issue yet.. lol.. will investigate further.

Hi @mikermcneil where can i upload multiple files in ration.io ?

Hi @ptdede You cant upload multiple files in ration.io (as of today). I would like to work on this bug. If you can give more details on this bug then I will do my investigation. Thanks.

@saranp Hi! you can find the repo at minimal setup to reproduce this issue from my post above. I don't know but this line files.map((file, index) => {}) seems wrong to me. Should it use Promise.all ?

@saranp Yes, i know it's not related to sails, but inconsistent behaviour like throttling the network makes this bug happen is. If you run the code from the repo, you will find it's working fine if the network is not throttled. If not (3G network), the req.body is empty. I don't know only on my machine or others also experiencing the same issue. :D

We can close this issue as this is not related to sails.


[Forwardport main] Fix datasources test cases' url assertion

Description
Fix datasources test cases' url assertion when security is enabled
Please merge after https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1023 so that we can verify the fix in CI.
Issues Resolved

Forwardport https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1024

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

nice, all cases are passed! 😃


Merge origin/main into resources branch

Handles conflicts from the wasi-http support for inbound requests.

Ah, I was already working on this. But my version has some test failures, and your version doesn't, so I think I'll just switch to your version here.


[Feature] remove deprecated columns

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
In June 2023, the connector was migrated to a new Shopify API version, in which the following relevant columns were deprecated

order_line.fulfillment_service
order_line.destination_location_* fields
order_line.origin_location_* fields
order.total_price_usd
order. processing_method
customer.lifetime_duration

we should mark them as deprecated in the src yml file and remove them from staging models
Describe alternatives you've considered
leave em
Are you interested in contributing this feature?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[ ] No.

Anything else?
No response

This is a duplicate of the Issue #60 created in the transform package. This is currently being worked on in the active sprint. I will close this issue out as a duplicate.


Authentication fails after ~ 24-30 hours running successfully

Checkboxes for prior research

[X] I've gone through Developer Guide and API reference
[X] I've searched for previous similar issues and didn't find any solution.

Describe the bug
Below is my setup. For first few hours it is working as expected. After 24 hours - 30 hours i am getting unauthorized exception. I believe token refresh is not being handled properly
`private constructor () {
this.configManager = ConfigurationManager.getInstance();
this.kafka = new Kafka({
clientId: this.configManager.getInitialConfig().managementPlane.activationKey,
brokers: this.constructBrokers(),
ssl: true,
sasl: {
mechanism: 'oauthbearer',
oauthBearerProvider: () => this.oauthBearerTokenProvider()
}
});
}
constructBrokers (): string[] {
const { kafkaServers } = this.configManager.getInitialConfig().dpActivatedResponseDTO;
return kafkaServers?.split(',') || [];
}
async oauthBearerTokenProvider (): Promise {
const { accessID, accessSecret } = this.configManager.getInitialConfig().dpActivatedResponseDTO;
process.env.AWS_ACCESS_KEY_ID = accessID;
process.env.AWS_SECRET_ACCESS_KEY = accessSecret;
const authTokenResponse = await generateAuthTokenFromCredentialsProvider({
region: 'us-west-2',
awsCredentialsProvider: fromNodeProviderChain()
});
console.log('authTokenResponse', authTokenResponse)
return {
value: authTokenResponse.token
};
}`
aws-msk-iam-sasl-signer-js library version used
1.0.0
Which Node.js version is this issue in?
18.7.1
Operating System and version
mac
Reproduction Steps
Added snippet
Observed Behavior
Token refresh is not handled properly
Expected Behavior
Token should for ever refresh
Possible Solution
No response
Additional Information/Context
No response

Can you share logs on the errors you are seeing here?

Not 100% sure due to the lack of error logs in the original bug, but this seems similar to what we are seeing. Although, we are seeing it roughly once per week as opposed to every 30 hours.
Everything seems to be working smoothly while authenticating with an MSK Kafka cluster using IAM auth, and then our applications log "InvalidIdentityTokenException: Couldn't retrieve verification key from your identity provider, please reference AssumeRoleWithWebIdentity documentation for requirements"
The line that fails is const authTokenResponse = await generateAuthToken({ region });
const kafkaConfig = {
      ...config,
      sasl: {
        mechanism: 'oauthbearer',
        oauthBearerProvider: () => oauthBearerTokenProvider(config.region as string)
      }
    }

Revelant snippet from oauthBearerTokenProvider
    log.info('Kafka Auth generating authorization token . . .')
    const authTokenResponse = await generateAuthToken({ region });

    log.info('Kafka Auth token successfully generated.')
    return {
      value: authTokenResponse.token
    }

The application appears to recover after, but worried this could be causing an issue we are not anticipating. I can potentially try to generate some more useful debug logs if any are needed and you direct me to which logs you'd like to see.
I'm wondering if this could be related to https://github.com/jmaver-plume/kafkajs-msk-iam-authentication-mechanism/issues/20. I did not test setting a session name but maybe that would be a good next step.
Thanks for any help!

Following up here, setting session name lowered the frequency, but we have still seen this once or twice in the past month.


Updated configs of Lefthook

Description
Updated configs of Lefthook.
Is this a breaking change (Yes/No):
No
Additional Information
To remove leftover husky settings, please run the following commands:
rm -rf .husky
git config --unset core.hooksPath

@koralle
By the way, don't you have spellcheck set up?

@koralle By the way, don't you have spellcheck set up?

@108yen
Sorry. I added spellcheck command to pre-commit hook now.


[BUG] Send transactions fails every time max amount is used

Describe the bug
When using the max button on the amount text field for send transaction. The flow always fails, my guess is that it is not accounting for gas when fetching the wallet balance.
To Reproduce

Choose a chain and tap on send.
Use the max button for the amount field and press continue.
Verify the details and tap next.
You will get the error message screen with "failToStart" text.

Expected behavior
Should send the maximum amount of crypto based on account balance.
Screenshots
After the max button is tapped

Error message

https://github.com/voltix-vault/voltix-ios/pull/390


Update wide_n_deep_tutorial.py

-mkdtemp is low-level, and creates a temporary file somewhere out of sight that requires users to delete the file themselves. For the use case of a tutorial, it is best to use a method that cleanups the file when finished
-TemporaryFile does such as described above, but not available in Python 2

Can one of the admins verify this patch?

@tensorflow-jenkins test this please


Create a details page for content to view the content "read-only"

Closed in https://github.com/oddbit/tanam/pull/360


Implement better fix for the OS X / findPtr issue

This is an alternative solution to https://github.com/snowleopard/hadrian/issues/614.
@Mistuke pointed out here that the solution I went with (always include findPtr/_findPtr) has its own problems, and suggested a better one. I implemented just that in https://github.com/alpmestan/ghc/commit/f78e0ae443f48bb2c58819f04d781a3d27c2d528.
Note that the hadrian build should "just" pick up the fix, since we already pass the debug flag whenever appropriate for our extra-library-flavours business. Let's see what the OS X builds say. When I get them to work, I'll submit the GHC patch, and when it's merged, we can just close this PR I suppose, since there's nothing to change in hadrian land. :-)

Sounds good @alpmestan, thank you!

Hmm... Circle CI fails with:
Command: _build/stage0/bin/ghc -Wall -hisuf hi -osuf o -hcsuf hc -static -hide-all-packages -no-user-package-db '-package-db _build/stage1/lib/package.conf.d' '-this-unit-id ghc-boot-th-8.7' '-package-id base-<IP_ADDRESS>' -i -i_build/stage1/libraries/ghc-boot-th/build -i_build/stage1/libraries/ghc-boot-th/build/autogen -ilibraries/ghc-boot-th/. -Iincludes -I_build/generated -I_build/stage1/libraries/ghc-boot-th/build -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181009/base-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181009/rts-1.0/include -I_build/generated -optc-I_build/generated -optP-include -optP_build/stage1/libraries/ghc-boot-th/build/autogen/cabal_macros.h -optc-fno-stack-protector -odir _build/stage1/libraries/ghc-boot-th/build -hidir _build/stage1/libraries/ghc-boot-th/build -stubdir _build/stage1/libraries/ghc-boot-th/build -Wnoncanonical-monad-instances -optc-Wno-unknown-pragmas -c libraries/ghc-boot-th/GHC/ForeignSrcLang/Type.hs -o _build/stage1/libraries/ghc-boot-th/build/GHC/ForeignSrcLang/Type.o -O0 -H64m -XHaskell2010 -XNoImplicitPrelude -ghcversion-file=/Users/distiller/hadrian/ghc/_build/generated/ghcversion.h -Wno-deprecated-flags
Exit code: 1
Stderr:
libraries/ghc-boot-th/GHC/ForeignSrcLang/Type.hs:1:1: error:
    Bad interface file: /usr/local/Cellar/ghc/8.4.3/lib/ghc-8.4.3/integer-gmp-<IP_ADDRESS>/GHC/Integer/Type.hi
        mismatched interface file versions (wanted "80720181009", got "8043")
  |
1 | {-# LANGUAGE DeriveGeneric #-}
  | ^
)

So does AppVeyor which shouldn't have been affected by this patch, looks like the stage0 compiler that stack is installing is broken?

Well, Circle CI uses the cabal script I think? And installs ghc from homebrew.

Usually when this happens there's some kind of ABI mismatch. I assume the builds aren't cached? The linux build doesn't seem to suffer the same issue. why would also AppVeyor be affected? this only changes the CircleCI script no?

Yes, and I don't know why appveyor is affected.

When I try to reproduce the problem I get... other failures. It's just one of those days/weeks...

I think I know where the circle ci failure comes from: --integer-simple... I'll remove it temporarily to see how things go. I wonder if https://github.com/ghc/ghc/commit/fc2ff6dd7496a33bf68165b28f37f40b7d647418 has anything to do with this.

Haha! New failure:
| Run Ghc CompileHs Stage1: libraries/ghci/GHCi/Message.hs => _build/stage1/libraries/ghci/build/GHCi/Message.o
| Run Ghc LinkHs Stage1: _build/stage1/utils/hpc/build/Main.o (and 11 more) => _build/stage1/bin/hpc
ld: library not found for -lgmp
clang: error: linker command failed with exit code 1 (use -v to see invocation)
`gcc' failed in phase `Linker'. (Exit code: 1)
shakeArgsWith    0.000s    0%
Function shake   0.432s    0%
Database read    0.002s    0%
With database    0.000s    0%
Running rules  932.053s   99%  =========================
Total          932.487s  100%
Error when running Shake build system:
* _build/stage1/bin/hpc
user error (Development.Shake.cmd, system command failed
Command: _build/stage0/bin/ghc -Wall -hisuf hi -osuf o -hcsuf hc -static -hide-all-packages -no-user-package-db '-package-db _build/stage1/lib/package.conf.d' '-package-id array-<IP_ADDRESS>' '-package-id base-<IP_ADDRESS>' '-package-id containers-<IP_ADDRESS>' '-package-id directory-<IP_ADDRESS>' '-package-id filepath-<IP_ADDRESS>' '-package-id hpc-<IP_ADDRESS>' -i -i_build/stage1/utils/hpc/build -i_build/stage1/utils/hpc/build/hpc/autogen -iutils/hpc/. -Iincludes -I_build/generated -I_build/stage1/utils/hpc/build -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/unix-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/time-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/bytestring-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/base-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/integer-gmp-<IP_ADDRESS>/include -I/Users/distiller/hadrian/ghc/_build/stage1/lib/x86_64-osx-ghc-8.7.20181015/rts-1.0/include -I_build/generated -optc-I_build/generated -optP-include -optP_build/stage1/utils/hpc/build/hpc/autogen/cabal_macros.h -optc-fno-stack-protector -odir _build/stage1/utils/hpc/build -hidir _build/stage1/utils/hpc/build -stubdir _build/stage1/utils/hpc/build -no-auto-link-packages -rtsopts -optl-lgmp -Wnoncanonical-monad-instances -optc-Wno-unknown-pragmas _build/stage1/utils/hpc/build/Main.o _build/stage1/utils/hpc/build/HpcParser.o _build/stage1/utils/hpc/build/HpcCombine.o _build/stage1/utils/hpc/build/HpcDraft.o _build/stage1/utils/hpc/build/HpcFlags.o _build/stage1/utils/hpc/build/HpcLexer.o _build/stage1/utils/hpc/build/HpcMarkup.o _build/stage1/utils/hpc/build/HpcOverlay.o _build/stage1/utils/hpc/build/HpcReport.o _build/stage1/utils/hpc/build/HpcShowTix.o _build/stage1/utils/hpc/build/HpcUtils.o _build/stage1/utils/hpc/build/Paths_hpc_bin.o -o _build/stage1/bin/hpc -O0 -H64m -XHaskell2010 -ghcversion-file=/Users/distiller/hadrian/ghc/_build/generated/ghcversion.h
Exit code: 1
Stderr:
ld: library not found for -lgmp
clang: error: linker command failed with exit code 1 (use -v to see invocation)
`gcc' failed in phase `Linker'. (Exit code: 1)
)

@snowleopard Could we arrange for libgmp to be available? At least to make CI work, then I can investigate the --integer-simple problem.

If I take the tip of ghc's master branch of today, revert https://github.com/ghc/ghc/commit/fc2ff6dd7496a33bf68165b28f37f40b7d647418 and build with --integer-simple, it works.

So... maybe
src/Settings/Builders/Ghc.hs:116:            , libraryPackage ? arg ("-this-unit-id " ++ pkgId)

should be updated to special-case integer-gmp/integer-simple? But shouldn't the ghc-options changes from both cabal files in the GHC commit I linked to take care of that?

But shouldn't the ghc-options changes from both cabal files in the GHC commit I linked to take care of that?

Yes, that would be my expectation. Since there are no changes to the Make build system, I suppose there shouldn't be any changes in Hadrian either... Oh wait, there is one change in compiler/ghc.mk. Do you think it's relevant?

I checked and indeed GHC receives two -this-unit-id args when building integer-simple modules and registering the package. One with just the typical package id, and the one from the ghc-options field of the .cabal file. So I made this patch:
diff --git a/src/Settings/Builders/Ghc.hs b/src/Settings/Builders/Ghc.hs
index 8212b5f..082e6c8 100644
--- a/src/Settings/Builders/Ghc.hs
+++ b/src/Settings/Builders/Ghc.hs
@@ -113,7 +113,12 @@ packageGhcArgs = do
     mconcat [ arg "-hide-all-packages"
             , arg "-no-user-package-db"
             , packageDatabaseArgs
-            , libraryPackage ? arg ("-this-unit-id " ++ pkgId)
+            , libraryPackage ?
+                -- those two packages specify 'integer-wired-in' as the unit id
+                -- in their .cabal files (ghc-options field), so we need to omit
+                -- this argument just for them.
+                not (package `elem` [integerSimple, integerGmp]) ?
+                  arg ("-this-unit-id " ++ pkgId)
             , map ("-package-id " ++) <$> getContextData depIds ]

 includeGhcArgs :: Args

but it apparently isn't enough, I've rebuild everything from scratch but still see this error:
$ hadrian/build.sh -j4 --flavour=quickest -o_quickest --integer-simple
[...]
Error when running Shake build system:
* _quickest3/stage1/lib/package.conf.d/ghc-boot-th-8.7.conf
* _quickest3/stage1/libraries/ghc-boot-th/build/libHSghc-boot-th-8.7.a
* _quickest3/stage1/libraries/ghc-boot-th/build/GHC/Lexeme.o
* _quickest3/stage1/libraries/ghc-boot-th/build/GHC/Lexeme.o _quickest3/stage1/libraries/ghc-boot-th/build/GHC/Lexeme.hi
user error (Development.Shake.cmd, system command failed
Command: _quickest3/stage0/bin/ghc -Wall -hisuf hi -osuf o -hcsuf hc -static -hide-all-packages -no-user-package-db '-package-db _quickest3/stage1/lib/package.conf.d' '-this-unit-id ghc-boot-th-8.7' '-package-id base-<IP_ADDRESS>' -i -i_quickest3/stage1/libraries/ghc-boot-th/build -i_quickest3/stage1/libraries/ghc-boot-th/build/autogen -ilibraries/ghc-boot-th/. -Iincludes -I_quickest3/generated -I_quickest3/stage1/libraries/ghc-boot-th/build -I/nix/store/0d2ajw9gi57cc1iv2hzfi07d54zfrfq8-ghc-build-environment/include -I/home/alp/WT/ghc-diffs/_quickest3/stage1/lib/x86_64-linux-ghc-8.7.20181015/base-<IP_ADDRESS>/include -I/home/alp/WT/ghc-diffs/_quickest3/stage1/lib/x86_64-linux-ghc-8.7.20181015/rts-1.0/include -I_quickest3/generated -optc-I_quickest3/generated -optP-include -optP_quickest3/stage1/libraries/ghc-boot-th/build/autogen/cabal_macros.h -optc-fno-stack-protector -odir _quickest3/stage1/libraries/ghc-boot-th/build -hidir _quickest3/stage1/libraries/ghc-boot-th/build -stubdir _quickest3/stage1/libraries/ghc-boot-th/build -Wnoncanonical-monad-instances -optc-Werror=unused-but-set-variable -optc-Wno-error=inline -c libraries/ghc-boot-th/GHC/Lexeme.hs -o _quickest3/stage1/libraries/ghc-boot-th/build/GHC/Lexeme.o -O0 -H64m -XHaskell2010 -XNoImplicitPrelude -ghcversion-file=/home/alp/WT/ghc-diffs/_quickest3/generated/ghcversion.h -Wno-deprecated-flags
Exit code: 1
Stderr:
ghc: panic! (the 'impossible' happened)
  (GHC version 8.7.20181015 for x86_64-unknown-linux):
	lookupGlobal
  Bad interface file: /nix/store/g5ggsbqkwqiiq3qkpw8ji42axrna117y-ghc-8.4.3/lib/ghc-8.4.3/integer-gmp-<IP_ADDRESS>/GHC/Integer/Type.hi
      mismatched interface file versions (wanted "80720181015", got "8043")
  Call stack:
      CallStack (from HasCallStack):
        callStackDoc, called at compiler/utils/Outputable.hs:1160:37 in ghc:Outputable
        pprPanic, called at compiler/typecheck/TcEnv.hs:132:32 in ghc:TcEnv

Please report this as a GHC bug:  http://www.haskell.org/ghc/reportabug

)

mismatched interface file versions (wanted "80720181015", got "8043")

Looks like we somehow manage to mix up interface files built in different stages -- I have no idea how...

Well, keep in mind that reverting https://github.com/ghc/ghc/commit/fc2ff6dd7496a33bf68165b28f37f40b7d647418 is sufficient for making everything work again. So we're probably not too far from a fix, I'm just failing to see what else we should change right now.

I figured it out. Will create an issue explaining the problem and the potential solutions.

Ping.

@alpmestan I lost track of this -- have you submitted the findPtr patch you mentioned? In any case, we should probably close off all leftover PRs here...

@Mistuke Oh right, the other fix got merged only recently. I suppose I now have to create a differential for https://github.com/alpmestan/ghc/commit/4c37000bd65befcfd8331ed19dd46eb478ecf683 -- and... that's it, I think. Hadrian already passes the right flag.

@Mistuke Could you try https://phabricator.haskell.org/D5356? I made you a reviewer.

@alpmestan thanks for making the diff! the patch will run through validation tonight with my other stuff, I'll post a reply tomorrow. Cheers.

@Mistuke reported that the Windows tests got fixed with the patch, woohoo!

Awesome, many thanks 🎉


404 "Not Found" for URL http://download.microsoft.com/download/2/d/6/2d61c766-107b-409d-8fba-c39e61ca08e8/vcredist_x64.exe

Cookbook version -
VC9
Chef-client version
14.5.33
Platform Details
windows server 2016 standard evaluation
Scenario:
trying to install VC C++ 2008
Steps to Reproduce:
"run_list": [
"recipe[vcruntime::vc9]",
"recipe[vcruntime::vc10]"
]
Expected Result:
VS C++ 2008 should be installed ### Actual Result:
getting error
         Error executing action `create` on resource 'remote_file[C:\Users\vagrant\AppData\Local\Temp\kitchen\cache\package\vcredist_x64.exe]'
         ================================================================================

         Net::HTTPServerException
         ------------------------
         404 "Not Found"

         Compiled Resource:
         ------------------
         # Declared in

         remote_file("C:\Users\vagrant\AppData\Local\Temp\kitchen\cache\package\vcredist_x64.exe") do
    action [:create]
    default_guard_interpreter :default
    source ["http://download.microsoft.com/download/2/d/6/2d61c766-107b-409d-8fba-c39e61ca08e8/vcredist_x64.exe"]
    cookbook_name "vcruntime"
    checksum "55bf99ad3f063165c3ee1626787b5d4c2b65b5001fef79dd58c3490369ed282f"
    path "C:\\Users\\vagrant\\AppData\\Local\\Temp\\kitchen\\cache\\package\\Microsoft Visual C++ 2008 Redistributable - x64 9.0.30729.17.exe"

Fixed by #27

Seeing a very similar issue to this with vc10.
remote_file[C:\TEMP\kitchen\cache\package\vcredist_x64.exe] action create[2021-04-21T20:41:38-04:00] WARN: remote_file[C:\TEMP\kitchen\cache\package\vcredist_x64.exe] cannot be downloaded from http://download.microsoft.com/download/A/8/0/A80747C3-41BD-45DF-B505-E9710D2744E0/vcredist_x64.exe: 404 "Not Found"


System doesn't accept crafting using charged solar panels.

(The panels I'm talking about are from the Solar Flux Reborn mod. "Using" a panel is actually placing it down in a place with sun, letting it charge and then breaking it.)
Issue description:
I'm trying to make my system craft Solar Panels tier V using used (i.e charged) Solar Panels IV.
What happens:
If I insert newly crafted Solar Panels (i.e panels recently made, with no charge in them), all the crafting goes well. However, if I insert used Solar Panels (the charged ones) the system will tell me that solar panels are missing. All the recipes were made using the oredict thingy.
What you expected to happen:
System to recognize the used panels the same way as the newly made panels.
Steps to reproduce:

Make patterns using solar panels, enable ore dictionary.
Insert newly made solar panels into the system and try crafting the pattern you've made (this won't work for the first tier of solar panels as there are no tiers before that), the crafting will work.
Insert used solar panels into the system (you know if it is charged or not by looking at the item description, it will show up as charged. The system will probably tell you that resources are missing!
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>76
Refined Storage: 1.5.33

Does this issue occur on a server? [yes/no] Yes.

RS only accepts the exact item used in the recipe, not "used" versions.


Hotspot Transfer not working?? Urgent

Hello,
I am trying to transfer a hotspot of mine to a friend, it shows that there is a 'transfer request pending click to cancel'.
I have tried cancelling this many many times to no avail, I deleted the app, redownloaded and logged back in and the message saying pending has gone, and shows transfer hotspot. I type the address of the receiver and click transfer and nothing happens still.
Please assist me with this, the hotspot name is Wonderful Oily Copperhead .
I can't do anything with this hotspot and nobody in the Discord can assist either.
Please help!

It also seems to be a bug with another hotspot of mine, both the hotspot names I will leave below.
Rough Arctic Cheetah
Wonderful Oily Copperhead

Rough Arctic Cheetah has now been transferred, Wonderful Oily Copperhead is still showing:

Also adding evidence that the transfer to my address was completed 2 days ag

o.

@heliumsupplies there is a bug that will be resolved soon with transfer v2. Can you please try again, it should be fixed now.


Possible memory leak when service is running

Steps to reproduce:

Open up client, and make sure service is running

Make sure server is not running. If it is, close it down to ensure no processing on the client is happening.

Open the Android monitor and select the crowd-ml:datasend process. View the allocated memory continuously increasing in the performance profiler.

This may be due to the wifi listener, because that is the only running thread in the :datasend process.

Is this in the tensorflow branch?

No, I can view it on master.

Was able to reproduce.


payload is empty

payload always empty, cuz message should be in parameter 'data', not 'payload'.
http://developer.android.com/google/gcm/server.html#payload
Optional. If you are including a payload in the message, you use the data parameter to include the payload. This applies for both HTTP and XMPP.

Didn't catch that, thanks for reporting it!


Fix wrong HTTP method in runScheule

Run-Scheule API use POST instead of GET.
ref: https://github.com/treasure-data/td-client-ruby/blob/master/lib/td/client/api/schedule.rb#L104

@kenchan Good catch. I confirmed the method is changed and the patch is working.
Thank you so much for feedback!

@kenchan We really appreciate your contribution and blogging ;) Thank you again!

Thanks for the merging! :star:


Overlay on HDMI input stream

Hi Hannes,
I have an Android box running Android 11 AOSP from Khadas.  It's the VIM4 and supports HDMI input.
https://www.khadas.com/vim4
I would like to overlay text and graphics on the live HDMI input.
I have built your example and it's working as expected. However, when I switch to HDMI input (which is selected from an option in system settings), the overlay dissapears when the HDMI video is shown.  It returns when the HDMI is dismissed.
This also happens if I use the camera app (which has the HDMI input as the feed).
I have enabled overlay on top of app for all apps.
Is the problem with the HDMI feed, or possibly overlaying on top of video?
Any help would be appreciated.  Thanks

Hi Hannes, so what I have found is that if I reboot the vim4 after loading your app, it seems to work i.e. it will overlay on top of the HDMI.  I could mark this as closed unless you think you understand why this might be, and can suggest a fix?


Conflicts are not always identified

The logic for determining if an alignment object conflicts, is not always identifying conflicts.
The function arrayIntersect(a,b) that it was based from, doesn't work as it should. For example, [1,2,3,4] and [5,4,3,2] Doesn't return [2,3,4], it returns [].
[1,2,3,4] and [4,3,2] Doesn't return [2,3,4], it returns [4].
Most likely it is an issue with the math in the function and likely not returning any results in some cases.

Addressed in d8a5b0eeb348ff7fe962d4b6d11ca2d5332ef820


[Optimisation] Constant/variable folding & propagation

Take the following code as an example:
#include <mlogevo/io.h>
#include <mlogevo/mlog_object.h>
extern struct MLogObject message1;

void main() {
    int a = 1 + 2;
    print("1 + 2 = ", a);
    printflush(message1);
}

It can be observed that the value of 1 + 2 can be determined at the compile time to be equal to 3.
Furthermore, it can be observed that therefore the value of a can be determined at compile time.
Currently the outputed code (with -O 3)
op add _a@main 1 2
print "1 + 2 = "
print _a@main
printflush message1
end

However, we can simplify this to:
print "1 + 2 = 3"
printflush message1
end

I believe one of the ways to do this is, at the abstract syntax tree optimization step, to observe when an operation is done on two (for binary operations or one for unary operations) literals or constants and precompute the calculation.
Note, that variable can be constant at some times, but not constant in others.
#include <mlogevo/io.h>
#include <mlogevo/mlog_object.h>
extern struct MLogObject message1;

void main() {
    int a = 0;
    print("a before: ", a, "\n");
    a = someFunction();
    print("a after: ", a, "\n");
    printflush(message1);
}

The first print statement can be optimized to print("a before: 0\n");. The second can not be optimized in this way.
Variable folding & propagation is a bit more difficult than doing the same on constants, but can optimize common patterns, such as assigning a set of bit flags to a variable:
int a = 1 | 2 | 3;

At the current state this compiles too two instructions, though it can be compiled to only one.

Also, consecutive print instructions on literals are not optimized. Seems like an easy thing to do imho. I might try to contribute.


Version info via the -h flag please and error message when executing 1.00 release

Hi Dave,
would it be possible to show the pmemtool version as part of the -h or as a parameter please?
Also since we updated to version 1.00 I get this error when running pmt:
xxxxxxxxxx:/usr/share/pmemtool-main # pmt
Traceback (most recent call last):
File "./pmt", line 239, in
main(sys.argv[1:])
File "./pmt", line 142, in main
i.dimms = i.parse_dimm()
File "/usr/share/pmemtool-main/ipmctl.py", line 233, in parse_dimm
if white_list[root[dimm][dimm_attr].tag] == 1:
KeyError: 'FWActiveAPIVersion'

Integrated into Release 1.01.0


Event stream off a materialised view

Not clear from the example of what I am asking for is already in pranadb.
It would be great if a client can subscribe to a event stream of changes on a materialised view

Hi @gedw99 , yes, we will be implementing what we call "push" queries shortly. There is an issue here for them already https://github.com/cashapp/pranadb/issues/220

yep #220 is exactly what i was thinking.
https://github.com/cashapp/pranadb/blob/main/docs/usermanual.md#sinks describes the same thing, but with the Materialised View change event being transported to a kafka topic. Then presumably a Client can subscribe to that topic satisfying #220.  Is that how you envisage it working ?
Using the message broker / queue to hold the event is IMHO the right way to ensure they eventually get the event.
Thanks btw for the decent docs. Its makes it way easier to contribute.

Push queries are more lightweight thank sinks, but they are both ways of extracting changes from a MV.

The docs are a bit out of date, tbh. But we intend on updating them before too long :)

Thank you for the answer. Makes sense.


The code has a little mistake

This line should adjust the order of parameters as follows:
trans = gen_trans_from_patch_cv(c_x, c_y, patch_width, patch_height, bb_width, bb_height, 1.0, 0, inv=True)

Sorry, I didn't notice the parameter inv=True


Web environment cannot obtain real-time data stream

Package
dio
Version
v5.1.1
Output of flutter doctor -v
[√] Flutter (Channel master, 3.10.0-10.0.pre.39, on Microsoft Windows [版本 10.0.18363.592], locale zh-CN)
    • Flutter version 3.10.0-10.0.pre.39 on channel master at C:\tools\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d186792c00 (2 hours ago), 2023-04-20 20:50:59 -0700
    • Engine revision 122c3b3820
    • Dart version 3.1.0 (build 3.1.0-26.0.dev)
    • DevTools version 2.23.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\John\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Users\John\AppData\Local\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.5.4)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.5.33530.505
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] IntelliJ IDEA Ultimate Edition (version 2023.1)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2023.1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart

[√] VS Code (version 1.77.3)
    • VS Code at C:\Users\John\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.62.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[√] Connected device (4 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • Windows (desktop)          • windows       • windows-x64    • Microsoft Windows [版本 10.0.18363.592]
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 112.0.5615.50
    • Edge (web)                 • edge          • web-javascript • Microsoft Edge 112.0.1722.39

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

! Doctor found issues in 2 categories.

Dart Version
Dart SDK version: 3.1.0-26.0.dev (dev) (Thu Apr 20 12:25:58 2023 -0700) on "windows_x64"
Steps to Reproduce
Future askAi(String text) async {
Response response = await dio.post(
'https://api.openai.com/v1/completions',
data: {
"model": "text-davinci-003",
"prompt": text,
"temperature": 0.6,
"max_tokens": 1024,
"stream": true,
},
options: Options(headers: {
'Authorization': 'Bearer ${Env.apiKey}',
'Content-Type': 'application/json'
}, responseType: ResponseType.stream),
);
response.data?.stream.listen((List data) {
String str = String.fromCharCodes(data);
print(str);
});
}
Expected Result
Use the "ResponseType. stream" method to call the OpenAI interface, hoping to obtain real-time responses from the OpenAI interface.
Using dio: ^ 5.1.1, it works normally in both Android and Windows environments, “response.data?.stream.listen((List data)” -- Multiple data can be received per second.
But not work in a web environment(Chrome 112.0.5615.50). It should not be a browser caching issue, as using the same interface, JS can receive stream data in real-time.
Actual Result
response.data?.stream.listen((List data)
only one data
It takes about a few tens of seconds to receive stream data, and only a large data packet can be get when completion, and the process cannot be get in real-time.

Closing this as duplicate of https://github.com/cfug/dio/issues/1740


Question: How to stop and resume the undomanager

My application is running some commands on ctrl+space. When the user presses Ctrl+space, it begins to search if the last word typed at the caret position is a command and replaces it with a template. My problem is that the undomanager is saving the commands. eg. "pk" is replaced by "primary key" and "pk" shows up again on ctrl+z / ctrl+y. How do i force the undomanager to ignore this replaceText(...).

You'll need to create your own custom EventStream and then rebuild the default UndoManager using methods in the UndoUtils class. Something like this should work:
EventStream<PlainTextChange> regularStream = area.plainTextChanges();
SuspendableYes allowChange = new SuspendableYes();
EventStream<PlainTextChange> stream = regularStream.conditionOn(allowChange);
// Recreate UndoUtils#defaultUndoManager
// but pass in `stream` as the EventStream
UndoManager<?> um = // creation code

// when want to do CTRL+Space in some other part of your code
if (userPressedCtrlSpace) {
    allowChange.suspendWhile(() -> {
        area.replaceText();
        // area will emit a change but that change
        // will be ignored because allowChange is now 'false' during its suspension
    }
}

I'm assuming this answers your question. Reopen if it's still unanswered.

For those who are wondering about the magic "Recreate UndoUtils#defaultUndoManager" line in the answer above, use the code below to create/set the new manager:
area.setUndoManager(UndoManagerFactory.unlimitedHistoryFactory().createMultiChangeUM(area.multiRichChanges().conditionOn(allowChange), TextChange::invert, UndoUtils.applyMultiRichTextChange(area), TextChange::mergeWith, TextChange::isIdentity));


Update tabs aria-controls for hidden tabs

Update spec based on APG outcome: https://github.com/w3c/aria-practices/issues/293

@adamduncan, The ARIA Authoring Practices will stay consistent with the ARIA specification. See my comments in w3c/aria-practices#293 and w3c/aria#531.
I don't think making a change here is a good idea. If there is a change, it should be in user agent requirements or expectations of assistive technologies.

Thanks for the insights @mcking65! Much appreciated. Closing.


run with coverage enabled but failed

env
uname: 5.19.17
os-release:
root@syzkaller:~# cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 12 (bookworm)"
NAME="Debian GNU/Linux"
VERSION_ID="12"
VERSION="12 (bookworm)"
VERSION_CODENAME=bookworm
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

qemu-system-x86_64 \
  -m 8G \
  -smp 4 -cpu host \
  -kernel /home/g0dA/kernel/linux-5.19.17/arch/x86/boot/bzImage \
  -append "console=ttyS0 root=/dev/sda earlyprintk=serial net.ifnames=0" \
  -drive file=/home/g0dA/kernel/debian/debian.img,format=raw \
  -net user,host=<IP_ADDRESS>,hostfwd=tcp:<IP_ADDRESS>:10021-:22,hostfwd=tcp:<IP_ADDRESS>:8888-:8080 \
  -net nic,model=e1000 \
  -enable-kvm \
  -nographic \
  -pidfile vm.pid

Error
root@syzkaller:~# ls
buzzer	sourceFiles  vmlinux
root@syzkaller:~# ./buzzer
......
2023/05/03 02:09:54 failed to init control unit: Run program did not succee

and have output like this:
program flaked1974.
Writing verifier log to "/tmp/verifier-log-302911802".
Writing eBPF binary to "/tmp/ebpf-binary-3260864109".
"strconv.Atoi: parsing \"?\": invalid syntax"

Make complaints....Compiling is really cumbersome and requires gcc 12+

Hi!
For the coverage failures, I might be wrong but I see you are using your own debian image? We haven't tested that setup yet, so far we have only used the syzkaller instructions to build a kernel + bullseye/stretch image.
for the other failures like

program flaked1974.
Writing verifier log to "/tmp/verifier-log-302911802".
Writing eBPF binary to "/tmp/ebpf-binary-3260864109".

this actually means the fuzzer is running, just that the parse verifier log strategy requires more work to be more accurate, maybe run with the flag --parsing_strategy=pointer_arithmetic?

No, I also use ./create-image.sh -d bullseye
And the second problem is why output "strconv.Atoi: parsing \"?\": invalid syntax"
by the way, what is --parsing_strategy=pointer_arithmetic,i do not find this flag in code

Sorry, i meant “—fuzzing_strategy=pointer_arithmetic”
i ran across the invalid syntax error today as well. In my case, the vmlinux file was outdated, i recompiled the kernel and forgot to re upload the file, couls you make sure you are using the up to date one?

Yes, I just compiled
make vmlinx && make bzImage

then scp this vmlinx and kernel/bpf/verifier.c to qemu

Can you make sure your compilation is generating dwarf4 symbols? (CONFIG_DEBUG_INFO_DWARF4)

nono it actually makes sense to mention this in the documentation :) I'll send a pull request soon

Hi, I still encounter the similar issue "strconv.Atoi: parsing \"?\": invalid syntax". And the CONFIG_DEBUG_INFO_DWARF4 is set. On the localhost:8080 web, there is no coverage info about the bpf related files I specified, but only mmap.c, sys_x86_64.c, and current.h.


Update Rust demos to gstreamer 0.16 bindings

Also bump async-tungstenite to 0.10, no change was needed to this code.

This repository moved to https://gitlab.freedesktop.org/gstreamer/gst-examples/-/tree/master/webrtc


Get App Working

Description
Latest changes on the repositories do not work because we were working last year on branches different than develop. We have to get all the repositories building and running again before anything else.
Related
work must be also made on h9-backend, see this issue
Notes
Make sure to document the way to get the app working

After running npm audit fix the app started to work


Mo tro 180-182

Refactored styled components issue #180, #181 , and #182.

@K0LdW0rLd - nice work.


Implement histogram plotting for DE-Analysis

Update the histogram code such that

a heatmap colour code is selectable in the config
throw error if a non-existing colour map is selected.
select upon highest log2fold, only those upon a threshold should be plotted, only those < configurable p-value

done with #88


[Feature] add more accordions

Description
I can try to add more accordions if possible
please assign this issue to me under gssoc 2024
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Which type of more accordions do you want to add @nagamukesh


Fix XpackWatch struct

I found XPackWatch struct's Action fields have wrong type specification.
XPackWatchActionStatus is representing Status corresponding action, which represent status of watch actions. However, watch get api returns specification of actions in XpackWatch.Actions position.

I also ran into this recently. @olivere would you mind taking a look?

@olivere any thoughts here?

@olivere any thoughts here?

I'm a bit cautious with something like map[string]map[string]interface{} because although okay, it's very cumbersome to use. The basic benefit of this library is to provide a Go-like DSL, and I'd really like to make it at least a bit easier than such a generic access pattern.
I didn't have the time to look into XPack APIs too much. But most of it is available just like any other part of ES, and it seems to use the same logic to create request/response structures. E.g. here is the code for the Slack action. It's just a bit of work to translate all of this into Go structs, and I didn't have the time for it recently.

@olivere what do you think of merging this PR to fix the bug with the wrong type being used?

Sorry for the delay. I've found a different issue with the API, added a test, and moved it over to #1362. It will be fixed in 7.0.18, hopefully due later today.


handle icon copy when file does not exist

I feel a little clueless on good practice to do here, do we know if upgrade file can contain use keyword on 1.6 ?
Should i change the scope of setStateIcons method to be able to use it in this upgrade file ?

@apacios please read description before reviewing most of your questions are what i am asking in the description

Thank you @zalexki


Adapt @with to work with minor change from JuliaSyntax.jl

The reference parser parses the = in @x(y=1) as Expr(:(=)) but the = in @x(y=1) do y end as Expr(:kw).  In JuliaSyntax, we've corrected this inconsistency as a minor change to the parsing rules, but this breaks the @with macro in this package (And one macro in Fluxperimental - the only two cases in General.)
Found as part of the integration work in https://github.com/JuliaLang/julia/pull/46372

Oops, sorry for the weird copy and paste in the message, I forgot to change it from the sibling PR at https://github.com/FluxML/Fluxperimental.jl/pull/9


User can login twice

Core Library
@azure/msal or msal
Core Library Version
1.4.4
Wrapper Library
@azure/msal-angular
Wrapper Library Version
1.1.2
Description
It's worth to mention, that issue is not constantly reproducible. The main issue, that a user can log in with different accounts in 1 browser, which leads to other issues.
Error Message
No response
Msal Logs
No response
MSAL Configuration
I can provide you my msal configs, but I'm not sure if it's secure to post them here.
Relevant Code Snippets
There is not really any code snippets, just the usual this.authService.loginRedirect(authParams);, where authParams is an object with some additional params, that we need to use after login. And usual b2c login form:

Reproduction Steps
I could reproduce it once like this:

Open 2 tabs.
In the first tab sign in with your b2c user.
In the second tab sign-up as a new user.
From the second tab you will be redirected to web site and in the first tab you are logged in as well as in the second tab you are logged in but with a different user. This may lead to other issues.

My testers could reproduce a similar issue, but just with the sign-in flow:

Open 2 tabs.
Sign in with 1 user.
Sign in with another user in some time (not sure how long you should wait?)
From the second tab you will be redirected to web site and in the first tab, you are logged in as well as in the second tab you are logged in but with a different user. This may lead to other issues.
(c) Testers

Tested only in Chrome.
Looks like SelfAsserted endpoint is not consistent and randomly fails when it comes to the simultaneous logins.
Expected Behavior
Usually, SelfAsserted fails with 400 error, which is fine. But it's not robust. It would be nice either to test it more with the simultaneous logins or restrict the user from navigation to this page if the user already opened another login page.
Identity Provider
Azure B2C Basic Policy
Browsers Affected (Select all that apply)
Chrome
Regression
No response
Source
External (Customer)

@aosyatnik To clarify, you don't want a user to be able to log in with multiple accounts in the same browser? If you want to prevent multiple accounts, your app should check if MSAL has a cached account, and not allow the user to initiate login if they are already logged in.
And please provide your MSAL configuration (this is not sensitive data), thanks!

@jasonnutter hi, thank you for such a fast reply.

you don't want a user to be able to log in with multiple accounts in the same browser?
Yes, exactly, that's what we want.

MSAL config for dev:
environment.ts
export const environment = {
  production: false,
  debug: true,
  endpoint: 'http://localhost:7071/api/',
  scopeUri: ['https://portmonsysdevb2c.onmicrosoft.com/c878f897-9c92-4ee1-abae-53311382f537/Backend'],
  clientId: '5d460340-2df0-4e1c-80ed-bb452620f735',
  b2cPolicies: {
    names: {
      signUpSignIn: 'B2C_1_Signup',
      resetPassword: 'B2C_1_PasswordReset',
      profileEdit: 'B2C_1_Edit_Profile'
    },

    authorities: {
      signUpSignIn: {
        authority: 'https://portmonsysdevb2c.b2clogin.com/portmonsysdevb2c.onmicrosoft.com/B2C_1_Signup'
      },
      resetPassword: {
        authority: 'https://portmonsysdevb2c.b2clogin.com/portmonsysdevb2c.onmicrosoft.com/B2C_1_PasswordReset'
      },
      editProfile: {
        authority: 'https://portmonsysdevb2c.b2clogin.com/portmonsysdevb2c.onmicrosoft.com/B2C_1_Edit_Profile',
      }
    }
  }
};

Used in app.module.msal.factory.ts:
import { MsalAngularConfiguration } from '@azure/msal-angular';
import { Configuration } from 'msal';
import { environment } from 'src/environments/environment';

export const protectedResourceMap: [string, string[]][] = [
  [environment.endpoint, environment.scopeUri],
  ['./assets', null] // Assets are not protected resources.
];

// Default route after msal login redirect.
const LOGIN_ROUTE = '/login';

export function MSALConfigurationFactory(): Configuration {
  return {
    auth: {
      clientId: environment.clientId,
      authority: environment.b2cPolicies.authorities.signUpSignIn.authority,
      validateAuthority: false,
      redirectUri: window.origin + LOGIN_ROUTE,
      postLogoutRedirectUri: window.origin,
      navigateToLoginRequestUrl: true
    },
    cache: {
      storeAuthStateInCookie: false
    },
    system: {
      loadFrameTimeout: 10000
    }
  };
}

export function MSALAngularConfigFactory(): MsalAngularConfiguration {
  return {
    popUp: false,
    protectedResourceMap
  };
}

Connected in app.module.ts as this:
@NgModule({
  declarations: [...],
  imports: [...],
  providers: [
      ....,
      {
      provide: MSAL_CONFIG,
      useFactory: MSALConfigurationFactory
    },
    {
      provide: MSAL_CONFIG_ANGULAR,
      useFactory: MSALAngularConfigFactory
    },
]);

We are using MsalGuard and as I said, usually, it works, but sometimes different users can log in with 1 browser.

Hi @aosyatnik ! As @jasonnutter suggests, your app needs to check if MSAL already has an account in the cache and block the login if that's the case. You can see our account APIs' documentation here. Please let us know if this helps so we can close the issue!

@hectormmg ok, yes, we can close the issue.


Add the modify form for threads and forum posts

Add the ability to modify forum threads.
Add the ability to modify forum posts.

Forum thread management is still missing


Compiles fine in Simulator but doesn't compile on iPhone 7 Plus device?

I'm using your code in my Swift 3 app and it works fine in the simulator but when I try to run it on my 7 Plus device, it doesn't compile, complaining that value of UIImage has no member dominantColors.
The weird thing is running your project as a stand-alone works fine on my device.
Xcode does see the DominantColor framework when I try to import it using import DominantColor, so I'm really at a lost.
Any clues?

This issue is likely no longer relevant, please try on the latest release (v0.2.1) and reopen the issue if it persists. Thanks!


Incorrect ES5 transpilation for try/catch + super

🔎 Search Terms
ES5, this alias
🕗 Version & Regression Information

This changed between versions 5.1.6 and 5.2.2

⏯ Playground Link
https://www.typescriptlang.org/play?target=1&ts=5.2.2#code/IYIwzgLgTsDGEAJYBthjAgYge2wg3gFAJLYB2kUArvNlABRgAW2VyAJgCpNTYDuALgQhcyAKbAyASgLESCAJYAzBIxZsuPfjKLz5ELXwRkxRgKJReDAOQAFcWjEIoY6AE9rUgNxySAXwQxZDAnXT1SCmxxADpkbABzemsAeQBpTx89Pzls7MIUNAwAIWAoQIAPCDEydgwcPDDYckoaCDp6HV8Ed1lwhDAqAAcxBmgqMW8ugNhgCFgmVTEhKjIAazJ+aV7wpsiYuMTrCyshawQAakWENARjuilogFsxdGB4iczwgHov51coNxdEgDYYMJTAYIfKY5Qh5ExGEoMbxAA
💻 Code
// @target: es5
// @showEmit: true

abstract class Foo {
  constructor(shouldThrow: boolean) {
    if (shouldThrow) {
      throw new Error('Please retry');
    } else {
      console.log('OK');
    }
  }
}

class Bar extends Foo {
  constructor() {
    try {
      super(true);
    } catch (e: unknown) {
      console.log('Error: ' + (e as Error).message);
      // retry
      super(false);
    }
  }
}

new Bar();

Target: ES5
🙁 Actual behavior
_this is not defined

🙂 Expected behavior
No error
Additional information about the issue
In TS5.2.2 output var _this = this; has been removed, but _this is still being used.

Probably related to as https://github.com/microsoft/TypeScript/issues/55637, as both involve a use of this in a class constructor which broke in 5.2.

Same bisect result, so probably the same thing as #55637


P-3128 Dateepicker font color

@gunjankothari I think we need correlated stylesPanelSections in the mappings file.

@gunjankothari This is close.

Needs the following additions/fixes:

For the labels (ie "Date") and any text that appears below, etc -- have it use the same font family.
Create a new config sizing props for any of the labels, additional text. ie "Label Size"
Create a new config color prop for any of the labels, ie "Label Color"
Create a new config color prop for the underline / border around the field (as shown in the image)

See comment

Ready for review

@gunjankothari Can you fix the listing errors? See the failing check.


TensorRT server  stucked when I run perf_client command

hello, I have trained an inception_resnet_v2 model(.pb), about 220MB, then i use function create_inference_graph() to convert to tf-trt FP32(bigger than 400MB), tf-trt FP16(bigger than 300MB),
then , I make pressure test with perf_client command, the result is,
when the server is runing a tf model, the return statics is normal. while when the server is running a tf-trt FP32 model, only the first GPU memory rise until occupy all memory, then the server stucked.
so what's the problem?
by the way, although I set CUDA_VISIBLE_DEVICES=0, when I run the tensorrt server, it occupy all GPUs, how to assign tensorrt server runing on some GPU?
my hardware is T4 with 4 cards.

@diligent321 You should probably look at your max batch size. The sizes for the converted models seems to be correct. When you use tensorrt, it has fixed size that it will occupy on GPU memory and it depends on your max batch size. You might have specified too big value for max batch. In TF case, GPU memory is utilized during run-time based on incomming request. Therefore, you might have observed this discrepancy for your tf vs tf-trt models. How are you setting CUDA_VISIBLE_DEVICES? Normally, I would set it when I run trtis docker container by passing env variable as:
$ docker run --rm --shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864 -e CUDA_VISIBLE_DEVICES=0 -p8000:8000 -p8001:8001 -p8002:8002 -v/path/to/model/repository:/models <tensorrtserver image name> trtserver --model-store=/models

Thans for your kindly reply. The out of memory error is because of the large batch size. now, it can run normally.  when CUDA_VISIBLE_DEVICES=0 is included in docker command, it occcupy only one gpu.
Now, i have another question, i run model in all 4 cards, when i make pressure test with perf_client, it seems only the first card runs, as the following image shows,

and the return statics are almost the same as running on one card, so how to distribute http posts on all 4 cards ? @bezero

@diligent321 With the above configuration your model will be running only on 1st GPU. You can add more GPU by CUDA_VISIBLE_DEVICES=0,1,2 for example. If you remove CUDA_VISIBLE_DEVICES it will use all GPU resources available. If you want to allocate all the available resources to TRTIS, but you want to limit your deployed model to be running on certain GPUs, you can set this option on your models' deployment congig.pbtxt by setting instance_group. For example:
instance_group [
  {
    count: 1
    kind: KIND_GPU
    gpus: [ 0, 1, 2 ]
  }
]

To be able to utilise all your resources, you could send multiple async requests, so that your data are being processed in parallel by different instances. As far as I know, you can not distribute single request execution to multiple GPU instances, but you have to verify this hypothesis.

@bezero thanks, maybe i did not express  my question clearly, i clarify it here again,
In my server machine, i run the following script,
nvidia-docker run --rm --shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864           -e CUDA_VISIBLE_DEVICES=0,1,2,3  -p8000:8000 -p8001:8001 -p8002:8002 -v/model_dir:/models d98ba9ab7916 trtserver --model-store=/models
the instance_group in config.pbtxt is as follows,
20 instance_group [
21   {
22     kind: KIND_GPU,
23     count: 4
24   }
25 ]
when i start the server, the GPU usage is as follows,

In my client machine, i run the following script,
perf_client -m  inception_resnet_v2_graphdef  -p30000 -t16 -v -u <IP_ADDRESS>:8000
the GPU usage is as follows,

in my opinion, the server is running normally on 4 cards, with 4 instances on each card, but why only the first card processes inference computation when i send https with 16 concurrencies?

@diligent321 Based on the Options available for perf_client
Options:
 -b: batch size for each request sent.
 -t: number of concurrent requests sent. If -d is set, -t indicate the number
     of concurrent requests to start with ("starting concurrency" level).
 -d: enable dynamic concurrent request mode.
 -l: latency threshold in msec, will have no effect if -d is not set.
 -p: time interval for each measurement window in msec.

I think, you should set concurrent request mode and set the number of concurrent requests. It should be something like this:
perf_client -d -t 4 -m  inception_resnet_v2_graphdef  -p30000 -t16 -v -u <IP_ADDRESS>:8000

However, I am not sure. You can try running with these options.

I tried this ,but with no luck, thanks all the same.  @bezero

Using perf_client -t16 should cause 16 concurrent requests to be sent to the server, and because you have 16 model instances across 4 GPUs you should see all 4 GPUs busy. What version of TRTIS are you using? Also, can you add "-f " switch to your perf_client run and share the resulting CSV file.

@deadeyegoodwin
TRTIS  version:  I pulled the docker image nvcr.io/nvidia/tensorrtserver:19.05-py3.
In the console log of the server, it prints log like the following ,
I0716 01:37:20.530781 1 base_bundle.cc:162] Creating instance inception_resnet_v2_graphdef_0_{i}_gpu{j} on GPU {j} (7.5) using model.graphdef"
i={0, 1, 2, 3}, j={0, 1, 2, 3}
obviously, it create 4 model instances on each of the 4GPUs.
In the client, i run the following script, but no resulting CSV file exists,
perf_client -m  inception_resnet_v2_trt_FP32_zhipeng_graphdef  -ftemp_result.csv  -p30000 -t16 -v -u <IP_ADDRESS>:8000

or
perf_client -m  inception_resnet_v2_trt_FP32_zhipeng_graphdef  -f  temp_result.csv  -p30000 -t16 -v -u <IP_ADDRESS>:8000

the log in the client console is as follows,

Thanks for the report. We'll try to reproduce on our end. It is strange that the CSV file is not created...

Thanks for your kindly help, i do experiment again, the result is the same. When running a tf model, the model instances can run on several gpus normally, however, when running a tf-trt model, just convert the tf model to a FP32/FP16 one, it only runs on the first gpu, even if i assign "gpus: [ 2, 3 ]" in the config.pbtxt.
Really appreciate your suggestion.

This is likely related to the same issue as in #147 (but no caffe2 specific). Tensorflow API/config has a property that is supposed to allow you to control which GPUs a model runs on, but it doesn't work. As a result TRTIS is forced to use a work-around which is likely not working correctly in all cases. It could be that the TF-TRT model is explicitly placing every node on device 0. The TRTIS work-around would likely not override this explicit placement and so the model will always run on 0 (but this is just a guess we will have to verify).

Different from object detection algorithm involving CPU and GPU ops, the model is inception_resnet_v2， simply a classfication model, which could be run on only GPUs,
so I think it is a bug of TRTIS, and also look forward to your verification.

@deadeyegoodwin
Hi,
the TRTIS has many advantages, like multiple framework support, dynamic batch and so on. I would like to replace my flask framework with TRTIS as soon as possible. However, TF-TRT model can only be run on the first GPU, which limit the use in product environment.
so, is there a way to let TR-TRT models run normally on multi GPUs, instead of only the first GPU. if you have any suggestions about what I should do to deal with this disadvantage?
Sincerely look forward to your answer.

@diligent321
Hi,
In our verification we found TF-TRT models specifies device field in GraphDef to a single GPU. TRTIS does sets a default gpu device id, but unfortunately this setting doesn't override the specification in graphdef. We have a workaround in progress to circumvent this issue. In the mean time you can try clearing this field in the graphdef and then try running trtserver with your modified graphdef.
However, ensure the model of gpus is identical to what the model was generated on. TF-TRT models are very specific to the model they were generated on.

Thanks very much,  i will try  this  method.
发自我的iPhone
------------------ Original ------------------
From: Tanmay Verma<EMAIL_ADDRESS>Date: Sat,Jul 27,2019 9:13 AM
To: NVIDIA/tensorrt-inference-server<EMAIL_ADDRESS>Cc: diligent321<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [NVIDIA/tensorrt-inference-server] TensorRT server  stucked when I run perf_client command  (#456)
@diligent321
Hi,
In our verification we found TF-TRT models specifies device field in GraphDef to a single GPU. TRTIS does sets a default gpu device id, but unfortunately this setting doesn't override the specification in graphdef. We have a workaround in progress to circumvent this issue. In the mean time you can try clearing this field in the graphdef and then try running trtserver with your modified graphdef.
However, ensure the model of gpus is identical to what the model was generated on. TF-TRT models are very specific to the model they were generated on.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@tanmayv25
It is true that TF-TRT models specifies device field in GraphDef to a single GPU, even if i set CUDA_VISIBLE_DEVICES=0,1,2, the GraphDef is shown as follows,

so, how to clear device field in the TF-TRT model, could you offer the tool script, or explain this in more detail？

@diligent321 ,
Please have a look at the PR "Overriding the gpu allocation in GraphDef models #507" which solves your problem. The feature will soon be available at the master.

Great, thanks very much， look forward to the new tensorrt server release.

@diligent321,
The PR "Overriding the gpu allocation in GraphDef models #507" has been merged into master. The fix will be available in 19.08 release. I am closing the issue.