Wrong results when doing paginated searches within single http-request scope

Hi,
We're developing a register using the fhir-server packages and cosmosdb. To meet business-requirements we've injected extra logic as mediatr-pipeline-behaviors (validation etc.). This has worked fine, but we've recently encountered problems after the cosmosdb started using multiple physical partitions and the results are spread across multiple pages.
As part of our business-logic within a pipeline-behavior we use the mediator to search for resources using business-identifiers. The searches are simple and expects 0 or 1 results, and we iterated using the continuation-token to handle cases where the result is not on the first page. The issue is that we now experience unexpected behavior, e.g.:

We have a custom-endpoint (our own controller) where we use the mediator to execute SearchResourceRequests, but queries are not respected, the internal searches always returns ALL resources.
We have a pipeline-behavior on SearchResourceRequest where we do multiple internal searches (sequentially): The first search gives the expected outcome and the following searches gives the same result as the first one, even if the queries and expected outcome are different.

We are aware that this probably is a use-case that you have not intended, and we are considering moving this functionality into some sort of API-gateway in front of the fhir-server API. However, we would very much appreciate if you have some insight into what is happening and how we are able to fix this.
Thank you.
Edit: I believe I have resolved the issue. When doing these "internal-searches" the bundle's next-link only contains the continuation-token, not the initial query-parameters. The next-page will therefore have unexpected results. It has been my understanding that the continuation-token itself contained enough information, but that seems to not be the case.

I am not sure I totally follow what you are doing here, but it does sound like some behavior you should really put in an API management layer instead of working it into the code like this. If for no other reason than it makes it hard for you to update your FHIR server code when it updates here and we may make some changes to the specific areas you have modified thus breaking your code. The only contract here is really the FHIR API, rest is an implementation detail.
Sounds like we might be able to close this for now? But I will let @brendankowitz comment first in case he has some thoughts on this.

I am not sure I totally follow what you are doing here, but it does sound like some behavior you should really put in an API management layer instead of working it into the code like this. If for no other reason than it makes it hard for you to update your FHIR server code when it updates here and we may make some changes to the specific areas you have modified thus breaking your code. The only contract here is really the FHIR API, rest is an implementation detail.
Sounds like we might be able to close this for now? But I will let @brendankowitz comment first in case he has some thoughts on this.

You may close it since we've been able to resolve it. We would still appreciate some feedback from @brendankowitz.
I totally agree that it would be better to have this functionality in a separate API-layer and the separation shouldn't be too difficult due to how the code is structured. However, the current solution works well enough and such a separation is therefore not a priority at the moment.

Just to get a clearer picture, is your project forked from our repository, or are you using a new web project and importing the nuget packages?
I think feedback from your project is valuable to to learn how our FHIR Server is being used to build new solutions. Having said that, we would not be able to provide support for customizations to our code, including extending or modifying the behavior, unless there was a bug that can be reproduced with the unmodified code. Beyond this, we haven't 'officially' released our code packages as libraries, so internal dependencies may break and change over time. As Michael suggested above, the only contract we are committed to is the FHIR endpoint itself.
I'm happy to hear that you are able leverage this project to build a solution, but please be cautious of the above. I'm open to continuing to hear your feedback as I'm sure that there will continue to be community projects based on the code here.

Just to get a clearer picture, is your project forked from our repository, or are you using a new web project and importing the nuget packages?

We're importing the nuget-pacakges.
As you've might have figured out I'm part of the same team as @ellgre and I'll be happy to answer any further questions regarding our project by email.

Closing this issue here for now


ConnectionRefusedError: [WinError 10061] No connection could be made because the target machine actively refused it

Hey, I am trying to use

v = pptk.viewer(points)

for i in range (100):
    points = updates_points(points)
    v.clear()
    v.load(points)
...

but I am getting the next error ConnectionRefusedError: [WinError 10061] No connection could be made because the target machine actively refused it.
I already disabled my firewall but it doesn't help.
I installed the package with pip install on anaconda env.
what can I do?

This is definitely no longer supported, hence no updates in 4 years. This seems to be an issue with the socket communication getting locked up somewhere. This seems to be independent of the number of points or size of data, it seems to happen after about 25 loads, but goes down to 14 if you have a wait() step as well.
For me the following will print to 25 every time before the viewer hangs:
import pptk
import numpy as np

points = np.zeros((1,3),np.float32)
v = pptk.viewer(points)

for i in range(50):
    print(i)
    v.clear()
    v.load(points)


[Docs]: Datasources Landing Page

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Discord/slack/intercom Link
No response
Describe the problem and improvement.
[Docs]: Datasources Landing Page

https://github.com/appsmithorg/appsmith-docs/pull/2112


Add PyPI package badge to README

Hi @danielhollas. Is there a reason to link through badge.fury.io rather than directly to the PyPi repo at https://pypi.org/project/tcpb/0.12.0/? I'd prefer to avoid redirects through third party apps :)

@coltonbh thanks, I guess there's no good reason so I switched it to https://pypi.org/project/tcpb/ (we want to link to the latest, not to a specific version).


Could you use Reduce here?

https://github.com/bakedoatmeal/js-data-challenges/blob/518c2cfba8e312147419dd3dee8bcd631a907d66/challenges-2.js#L77
Would it possible to use reducer here?

Done! It's cool using reduce to return an array or object, previously I had only used it to calculate single values


OverflowException: Value was either too large or too small for an unsigned byte.

LongDdbConverter (and few others) contains incorrect number conversion
System.OverflowException: Value was either too large or too small for an unsigned byte.
at System.Number.ThrowOverflowException(TypeCode type)
at System.Byte.Parse(String s, IFormatProvider provider)
at EfficientDynamoDb.DocumentModel.NumberAttributeValue.ToByte()
at EfficientDynamoDb.Internal.Converters.Primitives.Numbers.LongDdbConverter.Read(AttributeValue& attributeValue)
at EfficientDynamoDb.Internal.Metadata.DdbPropertyInfo`1.SetValue(Object obj, AttributeValue& attributeValue)
at EfficientDynamoDb.Internal.Extensions.DocumentExtensions.ToObject[T](Document document, DynamoDbContextMetadata metadata)
at EfficientDynamoDb.DynamoDbContext.ToObject[T](Document document)
To reproduce problem model should contains long field with value higher than byte range
I will create PR with bugfix in few minutes

@lezzi Could you publish nuget with  https://github.com/AllocZero/EfficientDynamoDb/pull/195 ?
Thank you


Fixed PowerShell syntax highlighting

Proposed changes

Adjusted markdown code snippets to use powershell rather than PowerShell, which appears to break the Rouge syntax highlighting engine

Added a code snippet for PowerShell to the Test page for easier visibility of syntax highlighting going forward.

Related issues (optional)
Closes #8167

Deploy preview for docsdocker ready!
Built with commit 0f0d1211ad58e295b60b9ab60cc6ba80253f917c
https://deploy-preview-8170--docsdocker.netlify.com

Before:

After:

Test Page


modtwonumbers

Please read the readme file properly. We are not accepting PR in this format. Refer to Contribution.md to understand how we accept PRs. Till then cheers and keep hacking.... !!!
NB: I'm closing all your PR as it is invalid.

THANKS FOR MAKING A VALID PR.
THANKS TO YOUR VALUABLE PR


New DOM Order Popup Doesn't Work with Overflow Hidden Container

Describe the bug
With the move to placing the popup directly next to the input in the DOM this causes issues based on the position and overflow values of the inputs ancestors.
I am currently trying to use the date picker in a table/grid header that is nested in several ancestors with an overflow value of hidden. This causes the calendar popup to appear cutoff. I unfortunately don't have much control over the markup and the styles are brittle.
This worked prior to v4 with the teleport.
To Reproduce
I made a very simple recreation here.
Expected behavior
A couple of options come to mind

Should teleport vs DOM order be an option?
Would something like Floating Vue be a good option to handle all cases and leverage how much it can be customized?
Could just the contents of the calendar popup be abstracted to a component and exported so that consumers could put it into something like Floating Vue?

Screenshots

Desktop & mobile (please complete the following information):

Browser [e.g. chrome, safari]
All
Library version [e.g. 3.0.0]
4.0.1

Yes, I know this can be an issue, however, some component libraries like headless-ui don't like when the components are in the teleport because it also uses teleport. Anyway, I'll return the teleport functionality alongside the dom order.

Encountered the same issue when updating to the latest, read the changelog and saw that the default way is changed, which is good, but It would be good to have it as an option to teleport to body, because it is a very useful feature to have especially with overflow hidden

hi, Does this feature have release time. cause I need it now. thanks

@anton-liam Wednesday


brew bump-cask-pr fails on tap

brew config output
HOMEBREW_VERSION: 3.2.0-95-g45b2b54
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 45b2b549ff188762a39f3614bcfb9adf95db6ea8
Last commit: 16 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 34284fe7bb4dcafc59d32f659d2ab352753fe40b
Core tap last commit: 13 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_DEVELOPER: set
HOMEBREW_EDITOR: nano
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit kabylake
Clang: 12.0.5 build 1205
Git: 2.32.0 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.5-x86_64
CLT: <IP_ADDRESS>.9
Xcode: 12.5.1

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause formulae that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  php
  six

Warning: You have uncommitted modifications to Homebrew/homebrew-core.
If this is a surprise to you, then you should stash these modifications.
Stashing returns Homebrew to a pristine state but can be undone
should you later need to do so for some reason.
  cd /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core && git stash && git clean -d -f

Uncommitted files:
  ?? Formula/graal.rb

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Update a cask in a personal tap.
What happened (include all command output)?
Instead I got a weird error that printed part of the cask.
[sean] /u/l/H/L/T/s/homebrew-saltstack (main)> brew bump-cask-pr --version=3003.1  smillerdev/saltstack/saltstack
Error: Cask 'i)
  end

  conflicts_with cask: ["saltstack-3001", "saltstack-3000"]

  pkg "salt-#{version}-py3-x86_64.pkg"

  uninstall pkgutil: "com.saltstack.salt"
end
' is unavailable: No Cask with this name exists.

What did you expect to happen?
Open a PR to update the cask
Step-by-step reproduction instructions (by running brew commands)
brew tap smillerdev/saltstack
brew bump-cask-pr --version=3003.1  smillerdev/saltstack/saltstack

My initial thoughts for what this problem was were wrong, given it only appeared on this tap and none of the others I tested afterwards. Now it seems like it's just choking on the comment at the top of the file and thus not replacing anything? If I remove the comment at the top of the SMillerDev/saltstack/saltstack file, brew bump-cask-pr works fine. 😕

Not sure if this warrants an audit here (I don't really know about Casks), given the CaskLoader expects a file to start with cask "name" do and not a comment, I fixed up the comments in the tap to be inside the Cask block. Alternative fixes (for if any other user does this in taps we don't know about) is to skip the comment lines or allow them optionally in the regex, but I'm not sure how big a problem this is and whether it warrants that. Although this was a pretty confusing bug to figure out.

Great find, I do see this as a bug in brew though. I really don't expect features to crash on code comments.

Improving utils/ast would probably help in avoiding having to develop a regex that works for all sorts of corner cases for bump-{formula,cask}-pr.

Does that mean https://github.com/Homebrew/homebrew-cask/pull/108383/files would also fail on a bump-cask-pr?


IAM and Network queries are breaking with wrong type error

Describe the bug
IAM queries such as iam_subscription_owner_more_than_1 and iam_subscription_owner_max_3 and Network queries like network_watcher_in_regions_with_virtual_network are breaking with the below error -
Error: attribute 7 of type azure_role_definition has wrong type

Error: attribute 9 of type azure_virtual_network has wrong type

Powerpipe version (powerpipe -v)
latest
Steampipe version (steampipe -v)
latest
Plugin version (steampipe plugin list)
latest main
To reproduce
Steps to reproduce the behavior (please include relevant code and/or commands).
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

@bigdatasourav @ParthaI This seems to be a plugin issue. I tried using version v0.55.0 of the Azure plugin, and it works fine. Could you please check. Thanks.


Stale reading value of the env variable LD_LRBRARY_PATH in jaxlib

I have installed the jaxlib with cuda11 from official repo. When I run the training program, this error occurs:
2022-12-06 21:28:14.287076: W external/org_tensorflow/tensorflow/compiler/xla/service/spmd/slice_auto_sharded_stages.cc:199] Unexpected marker_type: grad
(MeshHostWorker pid=141952) 2022-12-06 21:28:15.167848: W external/org_tensorflow/tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudnn.so.8'; dlerror: libcudnn.so.8: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /opt/rh/devtoolset-8/root/usr/lib64:/opt/rh/devtoolset-8/root/usr/lib:/opt/rh/devtoolset-8/root/usr/lib64/dyninst:/opt/rh/devtoolset-8/root/usr/lib/dyninst:/opt/rh/devtoolset-8/root/usr/lib64:/opt/rh/devtoolset-8/root/usr/lib

However, I have already changed the LD_LIBRARY_PATH. It seems that jaxlib reads a staled version.

This problem doesn’t happen on some of my machines, but happens for some other.

This seems to be an issue on your environment. Please make sure all nodes have the same environment (same CUDA/Python/BASH env a). You can do that via a shared file system.


GetIndexMapping API bugfix

Signed-off-by: Petar Dzepina<EMAIL_ADDRESS>Description
bug fix regarding "timestamp" alias
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] 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.

Codecov Report

Merging #293 (58963a0) into main (18a6e92) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main     #293      +/-   ##
============================================
+ Coverage     36.87%   36.89%   +0.01%
  Complexity      887      887
============================================
  Files           179      179
  Lines          7031     7029       -2
  Branches        858      857       -1
============================================
  Hits           2593     2593
+ Misses         4201     4199       -2
  Partials        237      237

Impacted Files
Coverage Δ

...search/securityanalytics/mapper/MapperService.java
4.61% <0.00%> (+0.04%)
:arrow_up:

...ch/securityanalytics/mapper/MappingsTraverser.java
82.35% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Authentication requires captcha input now?

I am getting this error while trying to log into grammarly from this extension:

Sounds like grammarly has changed something in their API?

The issue went away, so closing this.


outputting ball x,y and players 1 and 2's x,y and others at the moments of each impact

solved by combing through source code.

Hi are you able to share how you did this please ?


fetchSongs() pushes undefined

fetchSongs() is currently pushing undefined tracks if an asset type (song/episode) has more available than 1 set of pattern.frequency iterations

need to test this more, as local instance doesn't push any undefined tracks (though only using 1 song/1 episode vs 20 song/6 episode on live

turns out this isn't a bug, but a nice little feature :)


Add a Chef ScaleIO Demo

The README has a bunch of details. The Chef demo uses configurable "data bags", which can be modified to the specific environment without having to modify the lower level chef recipes. The demo includes a script which is mostly self-driving (but currently only runs on Mac).
The performs the following tasks:

Download the ScaleIO GUI files and extracts them
Performs a vagrant up to bring up Chef server & 3 client nodes (each with a persistent volume)
Deploys ScaleIO using Chef recipes and cluster configuration defined in data_bags
Launches GUI to monitor cluster
Automatically upgrades ScaleIO cluster from default 1 node to 3 node metadata cluster
Adds SDCs, SDSs and SDS devices

This demo also includes a fourth node, which is intended to show cluster upgrades at a later date.

Nice! can you rebase down to a single commit and i'll merge it?

@kacole2 You got it.


Add head-day slot to views

Hi,
I would like to change the default header in the scheduler view for a custom one. I tried with the scheduler-day-header slot, but this only adds an element below the default header.

@chemscobra Override it in which way? CSS? Element? I just want to understand before I go look at it.

@hawkeye64, I would like to replace entirely the day header with a custom element.

@chemscobra I'll see what I can do -- thanks, and stay tuned

@chemscobra How soon do you need this? Is there any rush? I have added several slots and I am working on examples (which could take a few more days).

will be in v2.1.1

@chemscobra How soon do you need this? Is there any rush? I have added several slots and I am working on examples (which could take a few more days).

@hawkeye64  There is no rush. That's fantastic, I can't wait to see the new features. :clap: .  Thank you so much for your help

Hi @hawkeye64, I could see that you added the function, it's great! But since I am using the npm version with a Vue CLI project,  the latest version available in npm is 2.1.0. How could I update it?thanks in advance

@chemscobra npm did not update correct;y? This has happened to me several times. Usually I have to do another release. I'll be having one shortly (tomorrow some time).


feat(theme-classic): add stable class for DocSidebarContainer

Motivation
Currently styling the docs sidebar requires use of wildcard selectors to work with the generated id e.g:
aside[class*='docSidebarContainer'] {
  border-right: 1px dashed var(--ifm-toc-border-color);
}

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Tested locally to ensure class applies. No linting errors introduced.
Related PRs
None/NA

Should be good now @Josh-Cena - apologies for that. Still not entirely sure why it used default prettier rules but fixed now and it uses the right rules!


vite not found at after deploying

After deploying my laravel 9 project in 2 different hosts , i have faced the same issue regarding vite .
vite not found at .
knowing that i have run both of npm install and npm run dev before deploying
and regarding running the commands (npm install , npm run dev ) on the host  , i am face the issue which is shared hosting does not provide node easily so on two different hosts i couldn't run my project .
are there any solutions for my issue ?

You should not be running npm run dev on deploy. You should run npm run build.
If you are unable to run npm commands on your shared hosting, you should run npm run build locally and commit  the assets in the public/build directory.

that's exactly what i have done
i ran npm run build locally and committed the changes but still I'm getting
the same error
On Mon, Aug 29, 2022, 3:55 AM Tim MacDonald @.***>
wrote:

Closed #43903 https://github.com/laravel/framework/issues/43903 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/43903#event-7274514628, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWD3YXI6QN23JULCHF6OZG3V3QKBVANCNFSM574B466A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you provide the stacktrace or a screenshot? Will need more information to provide context.


SAPUI5 - Add a list to the view - CSS fix still required?

Is the fix in /webapp/css/style.css (mentioned at http://www.sap.com/developer/tutorials/sapui5-webide-add-list.html > 2.) still required? If I add the code snippet to style.css my app is missing the title bar and I can't scroll the list. If I just leave style.css empty the app seems to work fine. I'm using SAP Web IDE innovation version, maybe the behavior is different than with "standard" SAP Web IDE.
BR Nils

That's an interesting problem.  The entire sample runs correctly on the standard WebIDE version (I tested it about 30 seconds ago).
It's possible that the innovation version is using a new release of SAP UI5.  If so, the CSS problem may have been fixed, or just changed.  The CSS for UI5 is not guaranteed to remain constant, and so this fix was only temporary for our tutorial.
Just to verify, however, I took the CSS out of the sample.  It works without it.  So I'll remove it from our build copy and push out the change as soon as possible.
Thanks, good catch!
If you find any other problems, feel free to reopen this case.


Move content from old site

See old content, especially blog posts, here: https://web.archive.org/web/20140811074709/http://bikeky.org/blog

May also try this: https://www.drupal.org/node/1984522


Use --no-cache-dir on all pip install commands in dockerfiles

Follow-up to #790
Removing pip cache purge and using --no-cache-dir where pip install is used

The failure in GPU CI is because NVT hasn't yet been updated to match the latest version of Core: https://github.com/NVIDIA-Merlin/NVTabular/pull/1758


added json attributes to filebeat_prospector

Filebeat v5.0.0-alpha1 added support for decoding JSON logs.
This update will allow users with an up to date Filebeat to utilize that feature. I have tested that adding these options to prospectors on v1.3.1 doesn't cause obvious errors.
Please review, and let me know if you have any questions.

@michaelm-spindance 👍


SCP errors in discovery process when deployed in an AWS Organization managed by Control Tower

When using Control Tower in an AWS Organization the discovery process can't write to the addAccounts GQL mutation. This is because the mutation makes an unnecessary call to the putConfigAggregator API, an API that is disallowed by the SCP guardrails that Control Tower adds to all accounts in an org.
The call is unnecessary because in AWS_ORGANIZATION mode, the Config aggregator is managed by the organization not Workload Discovery, so there is no need to manually update the aggregator using putConfigAggregator.

Fixed in v2.1.14


Disabled buttons can be selected when using the Head Tracking feature

Description
For the My Sayings category, we have disabled the 'Remove Category' button so that users cannot remove it. However, the button can still be selected and the category removed by using the cursor for head tracking. This is also true of the 'Show Category' and 'Edit Phrases' buttons when they are in the disabled state.
Steps to Reproduce

With head tracking turned ON
Navigate to the Settings > Categories and Phrases screen
Locate and select the '123' keypad category
Note that the 'Edit Phrases' button is disabled
Hover the head tracking cursor over the button until it is selected

Expected behavior
Nothing happens after you hover the head tracking cursor over the button long enough for it to be selected.
Actual behavior
Once the button is selected the user is transitioned to the edit category phrases screen for the keypad, which should not be possible.
Device Information

Device: iPad, iPhone
OS: iOS 14, 15

Test verified on:
iPhone 12 Pro Max, iOS 15
iPad 9th gen, iOS 15
landscape & portrait
Try to select disabled buttons with head tracking:

123 > Edit Phrases ✅
My Sayings > Remove Category ✅
Recents > Edit Phrases ✅
Keyboard > save button ✅
Main screen > pagination ✅


A - Get contract with Stripe in place

Acceptance Criteria

[x] Legal contact with Stripe established
[ ] Contract drafted
[ ] Contract signed by both parties
[ ] Alternatively: current contract deemed acceptable.

Contract currently being reviewed by Mozilla legal.

We have a signed contract.


e-antic 1.2.2

Merge only after success.
This pull request was auto-generated by rever

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Duplicate of #48


Leverage Toolkit

[ ] Compass
[ ] Job Manager

With the introduction of Carthage support, this process will now be most seamlessly possible.

The toolkit is now integrated, closing.


v1.14 Backports 2024-12-02

[ ] #36089 (@giorio94) :warning: resolved conflicts

Once this PR is merged, a GitHub action will update the labels of these PRs:
 36089

/test-backport-1.14

Note that the previous round of backports is still in-flight: https://github.com/cilium/cilium/pull/36284.


Provide an ordering for permission contexts

@gravityfox wants to be able to have some sort of region priority system -- and it some other cases it makes sense for some contexts to be placed before others. At its simplest, this would involve ajusting all the permissions query methods to accept a List rather than a Set

I thought this would have been up to the permissionService to define? but yes, some hinting that the permissionsService is free to ignore would probably help because at the moment it has no clue as to what contexts override what.

Well instead of asking the permission service for a particular permission with a specific context order I would rather add a priority option to the PermissionService something like : setPriority(Context,int).
Reason:

Plugin X does/should not know about Plugin Y's regions nor their priorities.
Plugin Y does not know about the verify important context of Plugin Z (arena) and the less important context of plugin W (informative region). Only each individual plugin can say how important each of its own contexts is and share the information with the permission plugin, which itself may allow some kind of adjustment of the contexts so they can be put in a proper order.
We use Permission calculators that do the context calculation that we could alter to provide a priority mapping

Workflow:

Get active/current contexts
send them to the permission service
let the permission service order them (it knows best how to do this)
get permission value from the permission service using the properly ordered contexts
use permission value

@ryantheleach contexts and determining which contexts go before which others is more the job of the context calculators --permissionservice implementations do not know any specifics about contexts.
@ST-DDT context priorities don't mean much when not placed in relation to each other, so I don't see any reason to have them exist outside of the ContextCalculator.


[Feature]  Internal password manager

General

[X] My feature does not exist in the latest version of Nexterm.
[X] I have checked that my feature has not been suggested by anyone else.

Your Idea
I want to include a way to auto-complete sudo-prompts on my servers using a simple overlay that, when clicked, fills in the sudo password for me and presses enter.

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Unify types between packages/app and packages/ui-extensions-dev-server-kit

See: https://github.com/Shopify/cli/pull/833


export receiveany as method for raw req socket

export the implemention of receiveany as method for raw req socket

need this function, pls merge it.

need this function, pls merge it.

we will merge it after the new release it out in the near feature days, and it will be bundled in the next release that will be out in about June.

This branch should be rebased to the latest master. I'm seeing conflicts during the rebase:
agentzh@glass ~/git/stream-lua-nginx-module 0 (PR/184)$ g rb master
First, rewinding head to replay your work on top of it...
Applying: export ngx.req.socket:receiveany
Applying: revert the or <IP_ADDRESS>rc0 中的更新
Applying: count + 1 for receiveany added
Applying: add receiveany doc copied from lua-nginx-module, mod version for stream-lua-nginx-module 0.0.8
Applying: add test for raw req socket receiveany
Applying: TLSv1 disabled, switch to TLSv1.1
Using index info to reconstruct a base tree...
M	t/129-ssl-socket.t
Falling back to patching base and 3-way merge...
Auto-merging t/129-ssl-socket.t
CONFLICT (content): Merge conflict in t/129-ssl-socket.t
error: Failed to merge in the changes.
Patch failed at 0006 TLSv1 disabled, switch to TLSv1.1
Use 'git am --show-current-patch' to see the failed patch

Resolve all conflicts manually, mark them as resolved with
"git add/rm <conflicted_files>", then run "git rebase --continue".
You can instead skip this commit: run "git rebase --skip".
To abort and get back to the state before "git rebase", run "git rebase --abort".


Margin for Divider with middle broken after v4 to v5 migration

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
After upgrading from MUI v4 to v5.4, <Divider/> with middle prop has unexpected behavior, i.e., lacking margin-x properties. I did a little bit of digging, and found out that it was changed as part of https://github.com/mui/material-ui/pull/25428. If this is something that is removed for good, it would be nice if it could be added to the migration document.
Expected behavior 🤔
The margins must be present OR the migration document must specify this under Divider section here: https://mui.com/guides/migration-v4/#divider
Steps to reproduce 🕹
Steps:

Add <Divider orientation='vertical' variant='middle' /> from v4.
Observe it has margin of 16px on left and right
Divider from v5 lacks this margin

Context 🔦
Breaks many layouts across the application after v4 to v5 migration
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

  System:
    OS: macOS 12.2
    CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz
    Memory: 845.97 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.2 - /usr/local/bin/node
    Yarn: 1.22.5 - ~/.npm/bin/yarn
    npm: 8.1.2 - /usr/local/bin/npm
  Managers:
    Homebrew: 3.3.3 - /usr/local/bin/brew
    pip3: 21.1.1 - /usr/local/bin/pip3
    RubyGems: <IP_ADDRESS> - /Users/danedavid/.rbenv/shims/gem
  Utilities:
    Make: 3.81 - /usr/bin/make
    GCC: 4.2.1 - /usr/bin/gcc
    Git: 2.32.0 - /usr/bin/git
    Clang: 13.0.0 - /usr/bin/clang
  Servers:
    Apache: 2.4.51 - /usr/sbin/apachectl
  Virtualization:
    Docker: 20.10.12 - /usr/local/bin/docker
  IDEs:
    Nano: 2.0.6 - /usr/bin/nano
    VSCode: 1.64.0 - /usr/local/bin/code
    Vim: 8.2 - /usr/local/bin/vim
    Xcode: /undefined - /usr/bin/xcodebuild
  Languages:
    Bash: 3.2.57 - /bin/bash
    Go: 1.16.5 - /usr/local/bin/go
    Java: 1.8.0_265 - /usr/bin/javac
    Perl: 5.34.0 - /usr/local/bin/perl
    Protoc: 3.17.3 - /usr/local/bin/protoc
    Python: 2.7.18 - /usr/bin/python
    Python3: 3.9.5 - /usr/local/bin/python3
    Ruby: 2.6.8 - /Users/danedavid/.rbenv/shims/ruby
  Databases:
    PostgreSQL: 12.3 - /Applications/Postgres.app/Contents/Versions/latest/bin/postgres
    SQLite: 3.36.0 - /usr/bin/sqlite3
  Browsers:
    Chrome: 97.0.4692.99
    Safari: 15.3

The PR you linked actually fixes a bug, as the margin should be applied indeed on the top and bottom. The Divider is not the component that should decide what should the spacing be between it and the other elements. It is however, as you mentioned a breaking change, so we could add an entry in the migration guide. Would you like to create a PR for it?

Can I work on this task? I will update the docs!

@mnajdova is anyone assigned to this?

@DiegoAndai
Can i work on this ??

It was fixed in https://github.com/mui/material-ui/pull/36840


terminal mac os web smoke tests failing

seems like a few consistent smoke test failures:
see build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=306954&view=results and https://dev.azure.com/monacotools/Monaco/_build/results?buildId=306866&view=results
  1) VSCode Smoke Tests (Web)
       Terminal
         Terminal Input
           Auto replies
             should automatically reply to a custom entry:
     Error: Timeout: get terminal buffer '#terminal .terminal-wrapper' after 20 seconds.
      at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:205:23)
      at async Code.waitForTerminalBuffer (/Users/runner/work/1/s/test/automation/out/code.js:176:9)
      at async Terminal.waitForTerminalText (/Users/runner/work/1/s/test/automation/out/terminal.js:262:13)
      at async Context.<anonymous> (out/areas/terminal/terminal-input.test.js:38:17)

see build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=306898&view=results and https://dev.azure.com/monacotools/Monaco/_build/results?buildId=306921&view=results and some more
anywhere from 1-14 tests failing like below
  1) VSCode Smoke Tests (Web)
       Terminal
         Terminal Shell Integration
           Process-based tests
             Decorations
               terminal.integrated.shellIntegration.decorationsEnabled should determine gutter and overview ruler decoration visibility
                 "before each" hook for "never":
     Error: Timeout: is active element ' .monaco-editor[data-uri$="settings.json"] .native-edit-context' after 20 seconds.
      at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:205:23)
      at async Code.waitForActiveElement (/Users/runner/work/1/s/test/automation/out/code.js:167:9)
      at async Editor.waitForEditorFocus (/Users/runner/work/1/s/test/automation/out/editor.js:68:9)
      at async SettingsEditor.openUserSettingsFile (/Users/runner/work/1/s/test/automation/out/settings.js:51:9)
      at async SettingsEditor.clearUserSettings (/Users/runner/work/1/s/test/automation/out/settings.js:42:9)
      at async Context.<anonymous> (out/areas/terminal/terminal-shellIntegration.test.js:54:25)

  2) VSCode Smoke Tests (Web)
       Terminal
         Terminal Shell Integration
           Process-based tests
             "after all" hook in "Process-based tests":
     Error: Timeout: is active element ' .monaco-editor[data-uri$="settings.json"] .native-edit-context' after 20 seconds.
      at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:205:23)
      at async Code.waitForActiveElement (/Users/runner/work/1/s/test/automation/out/code.js:167:9)
      at async Editor.waitForEditorFocus (/Users/runner/work/1/s/test/automation/out/editor.js:68:9)
      at async SettingsEditor.openUserSettingsFile (/Users/runner/work/1/s/test/automation/out/settings.js:51:9)
      at async SettingsEditor.clearUserSettings (/Users/runner/work/1/s/test/automation/out/settings.js:42:9)
      at async Context.<anonymous> (out/areas/terminal/terminal-shellIntegration.test.js:33:17)

  3) VSCode Smoke Tests (Web)
       Terminal
         Terminal Shell Integration
           Write data-based tests
             "before all" hook in "Write data-based tests":
     Error: Timeout: is active element ' .monaco-editor[data-uri$="settings.json"] .native-edit-context' after 20 seconds.
      at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:205:23)
      at async Code.waitForActiveElement (/Users/runner/work/1/s/test/automation/out/code.js:167:9)
      at async Editor.waitForEditorFocus (/Users/runner/work/1/s/test/automation/out/editor.js:68:9)
      at async SettingsEditor.openUserSettingsFile (/Users/runner/work/1/s/test/automation/out/settings.js:51:9)
      at async SettingsEditor.addUserSettings (/Users/runner/work/1/s/test/automation/out/settings.js:36:9)
      at async setTerminalTestSettings (out/areas/terminal/terminal-helpers.js:9:5)
      at async Context.<anonymous> (out/areas/terminal/terminal-shellIntegration.test.js:89:17)

  4) VSCode Smoke Tests (Web)
       Terminal
         Terminal Shell Integration
           Write data-based tests
             "after all" hook in "Write data-based tests":
     Error: Timeout: is active element ' .monaco-editor[data-uri$="settings.json"] .native-edit-context' after 20 seconds.
      at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:205:23)
      at async Code.waitForActiveElement (/Users/runner/work/1/s/test/automation/out/code.js:167:9)
      at async Editor.waitForEditorFocus (/Users/runner/work/1/s/test/automation/out/editor.js:68:9)
      at async SettingsEditor.openUserSettingsFile (/Users/runner/work/1/s/test/automation/out/settings.js:51:9)

cc. @lszomoru

@aiday-mar can you please look into this. The smoke tests work by modifying settings.json via this call:
https://github.com/microsoft/vscode/blob/f408139c28c2c653d673e9a85ba250d6c6a588be/test/smoke/src/areas/terminal/terminal-input.test.ts#L41
This seems to have broken/become flaky with the edit context work. This is what the file looks like after that runs (from artifacts):

We could just set the edit context setting to off here:
https://github.com/microsoft/vscode/blob/f408139c28c2c653d673e9a85ba250d6c6a588be/test/smoke/src/areas/terminal/terminal-helpers.ts#L9-L19
But I'm still worried about flakiness with that since the edit context would need to be swapped out without a reload and we would need to change the settings.json file originally via edit context.

Thanks @Tyriar for filing this issue. I decided to disable temporarily the edit context and look into this later, as today is testing day. Pipeline should be green again after the disablement.


change cdsLocation capitalisation for consistency

Change attribute name as per #589

Can we close given https://github.com/ga4gh/schemas/blob/master/src/main/proto/ga4gh/allele_annotations.proto#L115 ?


Added metadata and metacounter services to groupChannels.

Simply added the metadata and metacounter services to the groupChannels service in an analogous way to what was being done in openChannels.

@tjanela Thanks a lot!
I haven't maintained this project in a while, let me know if you need anything else.


Remove useless comment in #23114 (#23173)

Backport #23173
The isAdmin param is no longer used so the comment should be removed.
https://github.com/go-gitea/gitea/blob/d27d36f2f4dd389050e613967ad2a5d02d250acc/routers/web/explore/repo.go#L36-L37


Editing zone by several admins

When several admins edit one zone,  changes of all admins overwritten by changes of admin who last click 'apply'.
Check the serial number of the zone before saving,  would be nice

This is a big issue with this interface for me. Even with just 5 sysadmins, the likelihood that someone has an old tab open with an ancient copy of the zone is very high and kind of terrifying.

I'm pretty sure this is fixed in the latest versions of the code. When the records have changed one must refresh before applying new changes.


Update README.md

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

Review Error for Bonniemarie216 @ 2024-09-29 15:53:19 UTC
User must have write permissions to review

Review Error for Bonniemarie216 @ 2024-09-29 16:29:36 UTC
User must have write permissions to review


faster integration tests via pre-hook

this change forces each test to use the set firebolt_dont_wait_for_upload_to_s3=1 pre-hook. this will make tests run much faster (provided that the driver/SDK allows pre-hook statements)

No longer needed, as we've moved off JDBC and it will no longer work.


Group creation

Creating new group shows "Done" on the button (should say next?)

in ios app it also says "Done", as this is the last step in the process

Looking back at the chat, I didn't describe the issue properly. It looks like the problem was the spinner next to the done button was running indefinitely.

Im not able to reproduce this one...


[prim] foundry prim should be updated whenever a generic prim changes interface or params

Following pr #10735 our nightlies have stopped running. This is because the generic prim cells and the foundry prim cells have different parameter list.
VCS sees the problem but ignores it:
Warning-[AOUP] Attempt to override undefined parameter ...spyglass_220213/opentitan/hw/foundry/generated/lowrisc_prim_flop_en-impl_0/prim_flop_en.sv, 37   Attempting to override undefined parameter "EnSecBuf", will ignore it.
However Tessent is not so friendly and stops with an error:
//  Error: File: ...spyglass_220213/opentitan/hw/foundry/generated/lowrisc_prim_flop_en-impl_0/prim_flop_en.sv, Line: 37:  Too many parameter assignments in instance u_impl_tsmc40ulpef of design unit prim_tsmc40ulpef_flop_en defined in file ( ...spyglass_220213/opentitan/hw/foundry/ip/prim_tsmc40ulpef/rtl/prim_tsmc40ulpef_flop_en.sv ) at line ( 5 ).
//  Error: File: ...spyglass_220213/opentitan/hw/foundry/generated/lowrisc_prim_flop_en-impl_0/prim_flop_en.sv, Line: 37:  Parameter name ( EnSecBuf ) in instance ( u_impl_tsmc40ulpef ) of design unit ( prim_tsmc40ulpef_flop_en ) defined in file ( ...spyglass_220213/opentitan/hw/foundry/ip/prim_tsmc40ulpef/rtl/prim_tsmc40ulpef_flop_en.sv ) at line ( 5 ) does not match with parent.
//  Error: File: ...spyglass_220213/opentitan/hw/foundry/generated/lowrisc_prim_flop_en-impl_0/prim_flop_en.sv, Line: 37:  Elaboration failed for the instance: 'u_impl_tsmc40ulpef'.

In the future please any change to the prim generic interface/param list should be immediately reflected in the foundry prims as well.
thanks

@NoamGottlieb

DC also breaks:
        the parameters "EnSecBuf=1'h0,ResetValue=1'h0,Width=1". (HDL-193)
Error:  EnSecBuf is not a parameter in module prim_XXX_flop_en (VER-216)

XXX is the technology name used in foundry
setting P0 as this is a show stopper

Ah this prim has been updated very recently.
I'll make a fix for the foundry repo.

This has been addressed in the internal repo.
Please confirm and close this issue if it works for you.

Thanks @msfschaffner
All is back to normal and our nightlies stated working again.
Closing the issue.


topbar feature - set a diffrent background color of all submenus

New option to set a different background-color for the submenus of a menu in the topbar.
For example if the topbar-background is transparent.
new var $topbar-background-submenu

Thanks :)

:+1:


[WIP] Spike of automatic package.json merging.

TODO:

[ ] Tests
[ ] Review
[ ] Tests

I have been able to run this successfully on a number of addons that I am updating.  My workflow at the moment is basically:
ember-cli-update --ignore-conflicts
git checkout master package.json
ember-cli-update --package-json-only
git mergetool

Once I get tests in here we can consider running this automatically (or swap the flag to something like --process-package-json to run the normal update/patch then run the package.json merging code).

You can now tell git-diff-apply to ignore package.json

Brought to completion in #108


Implement --release option.

After some trial and error it finally worked.
Fixes #181.
Questions

Instead of env::var(rustflags) we should use build_config.extra_rustc_args or build_config.extra_rustc_env. But I couldn't figure out how they work, I don't consider important for that PR.
I added -C debuginfo=2 to all kind of builds, I assume that is not going to have any unintentional effects?

-C debuginfo=2 shouldn't have any effects unless a user tried to override it. But when tarpaulin relies on debug info to work I would advise against anyone doing that anyway.
This all looks good to me - tried it out myself and code looks good. So I'm going to merge this in, thanks for your work :+1:


Send Key - Not working on non-qwerty keyboard configuration

I do have an azerty keyboard. When trying to use "send key" and mapping a shortcut, it save the english equivalent, for exemple " \ " for " ù ".
Could be solved by implementing azerty keyboard support or by giving the option to setup keybind from a menu. (For exemple, Keyboard Manager from Windows Powertoys) :

I would be glad to give you feedback about it :)


Fix an issue when using both "ImGui" and "Common UI" plugins where "accept" button presses are consumed by the "CommonAnalogCursor" preprocessor.

At Surgent Studios we've been using both ImGui and Common UI but we've run into the above issue.
It can be fixed by adding an input preprocessor to ImGui that listens to inputs before CommonAnalogCursor has a chance to consume the button.

Thank you for your contribution!


Explicitly declare variables

Currently, classes do not have variables explicitly declared, which is an obstacle when it comes to static type checking and debugging.

Fixed but not ready to push repo yet...


Cannot start in ultra light mode?

Context
2.3.1
Summary
Some steps were moved out of the SwapEnable condition in this commit: https://github.com/ethersphere/bee/commit/1f0c3f6747b2f3fb9ed89727c06158dae99eb92d#diff-515b2d0691202c4ea5e578ec896258ae9c440c381b608ed1eb7116a27847057dL492
It seems InitChequebookFactory, or chequebookFactory.ERC20Address are called for ultra light mode now, shutting down the Bee node during startup sequence:
"time"="2024-12-21 18:42:50.040629" "level"="info" "logger"="node" "msg"="bee version" "version"="2.3.1-67cc9b77"
"time"="2024-12-21 18:42:50.098554" "level"="info" "logger"="node" "msg"="swarm public key" "public_key"="034dac0d64b646254228c6ff4d797bb24dc874f6734b345657f7b05a4983d16e93"
"time"="2024-12-21 18:42:50.154441" "level"="debug" "logger"="node" "msg"="using existing libp2p key"
"time"="2024-12-21 18:42:50.216656" "level"="debug" "logger"="node" "msg"="using existing pss key"
"time"="2024-12-21 18:42:50.216686" "level"="info" "logger"="node" "msg"="pss public key" "public_key"="0276a23a6dd136309ce972a189e502c624f5544be75159dab12d81e3049c1dda55"
"time"="2024-12-21 18:42:50.216697" "level"="info" "logger"="node" "msg"="using ethereum address" "address"="0xfB67fff961D0F8f9c94b2d44de32c59B15EA11b7"
"time"="2024-12-21 18:42:50.251070" "level"="info" "logger"="node" "msg"="using overlay address" "address"="254d0189262aff04c18772e5a83576a2deaea2ad6e84af4c4b2528bf2f8cc900"
"time"="2024-12-21 18:42:50.251098" "level"="info" "logger"="node" "msg"="starting with a disabled chain backend"
"time"="2024-12-21 18:42:50.251294" "level"="info" "logger"="node" "msg"="using chain with network network" "chain_id"=100 "network_id"=1
"time"="2024-12-21 18:42:50.271399" "level"="info" "logger"="node" "msg"="using default factory address" "chain_id"=100 "factory_address"="0xC2d5A532cf69AA9A1378737D8ccDEF884B6E7420"
"time"="2024-12-21 18:42:50.271440" "level"="error" "logger"="node" "msg"="got error, shutting down..." "error"="factory fail: disabled chain backend"
"time"="2024-12-21 18:42:50.271456" "level"="info" "logger"="node" "msg"="starting debug & api server" "address"="<IP_ADDRESS>:1633"
"time"="2024-12-21 18:42:50.271581" "level"="error" "logger"="node" "msg"="failed to build bee node" "error"="factory fail: disabled chain backend"

Expected behavior
Can start with a disabled chain backend
Actual behavior
Please see summary
Steps to reproduce
This is my configuration
api-addr: <IP_ADDRESS>:1633
verbosity: 5
swap-enable: false
mainnet: true
full-node: false
cors-allowed-origins: '*'
use-postage-snapshot: false
resolver-options: https://cloudflare-eth.com
data-dir: /Users/aron/Library/Application Support/Swarm Desktop/data-dir
password: 2984c394-3594-44e3-b9d2-f8c786b24675
storage-incentives-enable: false
chequebook-enable: false

Possible solution
https://github.com/ethersphere/bee/pull/4936/files
I can start in ultra light mode with this, but there might have been a reason these steps were moved out of the check in the first place.

@istae Could you have a look at this?


Install LDC compiler

This installs LDC 0.17.2 and LDC 1.0.0 (D compiler).
One of the steps to get LDC on https://d.godbolt.org/.
Thanks!
LDC can be found here: https://github.com/ldc-developers/ldc/releases

Thanks! I'll merge now, and start updating the compilers available to CE

Looks like do_strip breaks all the executables in this case; will try without.

@mattgodbolt Wow, amazingly quick response!
Pointers to other places where LDC needs to be added are welcome :)
Currently looking at: https://github.com/mattgodbolt/gcc-explorer/blob/master/etc/config/d.amazon.properties
I'm afraid some compile flags are a little different for LDC... Importantly:  -of <filename> instead of -o <filename. How hard is it to work with that?

Thanks! ldc might need special casing rather like '6g' (for go) does. Presumably it doesn't support -S ? In general I use the compiler's asm output instead of actually outputting a binary. The code to do this is in https://github.com/mattgodbolt/gcc-explorer/blob/master/lib/compile.js#L270 - but as you can see the way it's written doesn't make it very scalable for different compilers.
If you can get an example command line for how ideally you'd process a D file into assembly, let me know and I can see how it might fit in!

ldc2 -c -output-s source_code.d -of assembly_code.s
-g adds line information

perfect! for now, try hacking it locally to supply -of on line ~270ish of compile.js and see if everything looks like it's ok. add the ldc compiler path in your own d.<hostname>.properties file (or hack the d.defaults.properties -- if it all looks good then a very simple change should do it :)

(I am actually editing gcc-explorer files online ;)  haven't even tried running things locally yet)

@mattgodbolt It works with these changes :)  (only colourising doesn't work)
-            compileToAsm = compilerProps("compileToAsm", "-S").split(" ");
+            compileToAsm = compilerProps("compileToAsm", "-output-s").split(" ");
         } else {
             compileToAsm = compilerProps("compileToBinary", "").split(" ");
         }
         if (compilerInfo.isCl) {
             options = options.concat(['/c', '/Fa' + outputFilename, '/Fo' + outputFilename + ".obj"]);
         } else {
-            options = options.concat(['-g', '-o', outputFilename]);
+            options = options.concat(['-g', '-of', outputFilename]);

Awesome. If you feel up to making a full patch I'd be very happy to merge;
else I'll look at making the changes myself over the next few days
On Wed, Oct 19, 2016 at 10:15 AM, Johan Engelen<EMAIL_ADDRESS>wrote:

@mattgodbolt https://github.com/mattgodbolt It works with these changes
:) (only colourising doesn't work)

       compileToAsm = compilerProps("compileToAsm", "-S").split(" ");+            compileToAsm = compilerProps("compileToAsm", "-output-s").split(" ");
   } else {
       compileToAsm = compilerProps("compileToBinary", "").split(" ");
   }
   if (compilerInfo.isCl) {
       options = options.concat(['/c', '/Fa' + outputFilename, '/Fo' + outputFilename + ".obj"]);
   } else {-            options = options.concat(['-g', '-o', outputFilename]);+            options = options.concat(['-g', '-of', outputFilename]);

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/gcc-explorer-image/pull/7#issuecomment-254844283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmsdX6ocEjaq7uBsTTqaLNHeYiRi070ks5q1jQggaJpZM4Ka__O
.

https://github.com/mattgodbolt/gcc-explorer/pull/157


nit: Remove busy waiting on scheduler

Motivation
This PR includes 3 changes.

contributor guide has no instruction regarding build. I found it is in install page, and I added a link there.
Use setproctitle to change the process title for sglang. (it is contraversial, and we need to introduce a new dependency).
Scheduler has busy waiting on recv_pyobject even when there are no requests which makes scheduler have 100% cpu usage even when the engine is idle. I make it use poll API when there's no batch to run (we need busy waiting when there are batches because we need to keep running batches).

Modifications

Checklist

[ ] Format your code according to the Contributor Guide.
[ ] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

I just made a PR to start contribution. Not sure if change makes sense to sglang perspective...


Avoid undefined indexes in notifications

If you have a webhook waiting information, with code similar to :
/* parse the incoming request in a meaningful response */
$response = app()->paymentManager->getProvider('2checkout')->getGateway()->acceptNotification()->send();

And someone does a post request without any data, it will trigger warnings like Undefined index: sale_id because there is basically no check made to see if the given array keys do exist.
This pull request checks using isset to make sure the array keys do exist before trying to fetch their value. Initially i thought to set the values to empty strings if they are not set, but in my second commit i used null as it makes more sense in this context. I also included a test for it.
Please keep in mind i am not familiar at all with how your library works internally, so please review the changes carefully.
Thank you.

Coverage increased (+0.4%) to 64.858% when pulling b92c0c4386cfe6bb93df55d74d97841e0d3cce4b on twisted1919:master into c317aad071cddb0bd022fb9e3a6e87cad38de730 on collizo4sky:master.

Please merge @w3guy


Unable to open save file from Journey to the Savage Planet

I'm trying to figure out why my save files from Journey to the Savage Planet on Stadia aren't working on the Steam version, and so being able to compare them would be useful. Unfortunately there seems to be a bug preventing uesave from reading them:
$ uesave to-json < save.bin
Error: unknown property type: "TextLanguage"

I've attached a zip file containing the save file.
Journey to the Savage Planet_1001_gamesave.zip

This save is a pretty good test case, so thanks for uploading it! Finally took the time to property implement text properties from the ground (still incomplete as text localization stuff is huge, but the groundwork is there now).
It also features an FName array which gets serialized as a two-member struct with each member having the same key name. I made the assumption member names have to be unique so to fix it requires some pretty significant changes to the API which hampers JSON readability quite a bit. I implemented a possible solution (store property key/value pairs in a vec instead of in a map) here, but I'm hoping there's a better solution before merging it. For now, this save will parse without error but there will be a member missing from the TrackedQuestsNames array in the resulting JSON.


styled the sidebar's element selector

Describe the changes you have made in this PR

added 2 new fonts in sidebar
increased the margin of the circuit elements
added border and some minor padding changed

Screenshots of the changes (If any) -

@nitin10s @Shivansh2407  review this.

Coverage remained the same at 84.279% when pulling ff0ccd0f0c1beb1ded3ce93c14d5736be8c1eb4d on RohitKumarGit:sidebar-improvisation into a9a0b072ace01117f676af8f9ada5ddf3842e1e3 on CircuitVerse:master.

We've already increased the margins in another PR, what's the purpose of adding two fonts? The PR is fine but the thing is (since we have this as a project), we are wanting to work on this holistically, and not just adding little margins and changing fonts. So, let's put this PR on hold for now.

Sir I have also changed colors, It's not just two fonts

Closing this since we are revamping the simulator completely.


Add compilation from passed program string to bindings

Sorry, but I think this needs a least a couple of words of description. What is the input, and what is the output?
Do you want to have cairo => casm only or cover more steps?

@karol-bisztyga I think this is done


Importing a certificate into oscm-core container deletes existing oscm-app certificate

[Build] 17.5 2017/11/07
[Steps to reproduce]
carry out steps described in https://github.com/servicecatalog/oscm/issues/36
[Expected result]
existing certificates e.g. for oscm-app are not removed
[Observed result]
existing certificates e.g. for oscm-app IS removed
[Details]
It seems the same certificate is used for oscm-core and oscm-app containers. But if I try to load in a new
certificate, the existing (in my case oscm-app) certificate is deleted.

moved to https://github.com/servicecatalog/oscm-dockerbuild/issues/74


Contact-map

Hi
Thanks again for your great repositories;
I have a question; how did you calculate contact map in your pre-retrained model? Have you used any supervising datasets or just use the embedded representation of amino acids and pairwise scoring?
Thanks
Nasser

Hi Nasser,
Thanks for your question -- It's a logistic regression mapping the attention heads to the contact map, logistic regression weights trained on only 20 proteins. As described in our recent paper to appear at ICLR.

Thanks so much
Great


Non-zero exit code when updates are available

Hi, I did three sessions with the routeros-upgrader tool:
$ echo N |./routeros-upgrader -l MikroTik; echo $?
2024/07/22 08:27:42 checking installed packages
|UP> MikroTik: dude-7.13.4-arm => dude-7.15.2-arm
|UP> MikroTik: container-7.13.4-arm => container-7.15.2-arm
|UP> MikroTik: routeros-7.13.4-arm => routeros-7.15.2-arm
|UP> MikroTik: wireless-7.13.4-arm => wireless-7.15.2-arm
Install? [y/N]: 0

$ echo Y |./routeros-upgrader -l MikroTik; echo $?
2024/07/22 08:40:12 checking installed packages
|UP> MikroTik: dude-7.13.4-arm => dude-7.15.2-arm
|UP> MikroTik: container-7.13.4-arm => container-7.15.2-arm
|UP> MikroTik: routeros-7.13.4-arm => routeros-7.15.2-arm
|UP> MikroTik: wireless-7.13.4-arm => wireless-7.15.2-arm
Install? [y/N]: 2024/07/22 08:40:13 downloading "dude-7.15.2-arm.npk"
2024/07/22 08:40:13 downloaded "dude-7.15.2-arm.npk" (1319057 bytes)
2024/07/22 08:40:14 MikroTik: uploaded dude-7.15.2-arm.npk
2024/07/22 08:40:14 downloading "container-7.15.2-arm.npk"
2024/07/22 08:40:14 downloaded "container-7.15.2-arm.npk" (98449 bytes)
2024/07/22 08:40:14 MikroTik: uploaded container-7.15.2-arm.npk
2024/07/22 08:40:14 downloading "routeros-7.15.2-arm.npk"
2024/07/22 08:40:15 downloaded "routeros-7.15.2-arm.npk" (11557096 bytes)
2024/07/22 08:40:21 MikroTik: uploaded routeros-7.15.2-arm.npk
2024/07/22 08:40:21 downloading "wireless-7.15.2-arm.npk"
2024/07/22 08:40:31 downloaded "wireless-7.15.2-arm.npk" (2003089 bytes)
2024/07/22 08:40:33 MikroTik: uploaded wireless-7.15.2-arm.npk
Execute synchronized reboot? [y/N]: 2024/07/22 08:40:33 EOF
1

$ ./routeros-upgrader -y -l MikroTik; echo $?
2024/07/22 08:41:48 checking installed packages
|UP> MikroTik: dude-7.13.4-arm => dude-7.15.2-arm
|UP> MikroTik: container-7.13.4-arm => container-7.15.2-arm
|UP> MikroTik: routeros-7.13.4-arm => routeros-7.15.2-arm
|UP> MikroTik: wireless-7.13.4-arm => wireless-7.15.2-arm
2024/07/22 08:41:48 downloading "dude-7.15.2-arm.npk"
2024/07/22 08:41:52 downloaded "dude-7.15.2-arm.npk" (1319057 bytes)
2024/07/22 08:41:53 MikroTik: uploaded dude-7.15.2-arm.npk
2024/07/22 08:41:53 downloading "container-7.15.2-arm.npk"
2024/07/22 08:41:53 downloaded "container-7.15.2-arm.npk" (98449 bytes)
2024/07/22 08:41:54 MikroTik: uploaded container-7.15.2-arm.npk
2024/07/22 08:41:54 downloading "routeros-7.15.2-arm.npk"
2024/07/22 08:42:38 downloaded "routeros-7.15.2-arm.npk" (11557096 bytes)
2024/07/22 08:42:45 MikroTik: uploaded routeros-7.15.2-arm.npk
2024/07/22 08:42:45 downloading "wireless-7.15.2-arm.npk"
2024/07/22 08:42:45 downloaded "wireless-7.15.2-arm.npk" (2003089 bytes)
2024/07/22 08:42:46 MikroTik: uploaded wireless-7.15.2-arm.npk
2024/07/22 08:42:46 MikroTik: rebooting in 10s
0

If the tool returned a non-zero exit code when updates were available, it could be used to monitor/automate mass upgrade.
However, it would be worth taking care of standard input processing ;) and perhaps introducing the -n (force no) option.

Hi @Marco98,
this is a great change and it works:
$ ./routeros-upgrader -n; echo $?
2024/07/26 11:35:28 checking installed packages
|UP> MikroTik1: routeros-7.15.2-powerpc => routeros-7.15.3-powerpc
|UP> MikroTik1: wireless-7.15.2-powerpc => wireless-7.15.3-powerpc
|UP> MikroTik2: dude-7.15.2-arm => dude-7.15.3-arm
|UP> MikroTik2: container-7.15.2-arm => container-7.15.3-arm
|UP> MikroTik2: routeros-7.15.2-arm => routeros-7.15.3-arm
|UP> MikroTik2: wireless-7.15.2-arm => wireless-7.15.3-arm
2

$ echo N |./routeros-upgrader; echo $?
2024/07/26 11:37:55 checking installed packages
|UP> MikroTik1: routeros-7.15.2-powerpc => routeros-7.15.3-powerpc
|UP> MikroTik1: wireless-7.15.2-powerpc => wireless-7.15.3-powerpc
|UP> MikroTik2: dude-7.15.2-arm => dude-7.15.3-arm
|UP> MikroTik2: container-7.15.2-arm => container-7.15.3-arm
|UP> MikroTik2: routeros-7.15.2-arm => routeros-7.15.3-arm
|UP> MikroTik2: wireless-7.15.2-arm => wireless-7.15.3-arm
Install? [y/N]: 2

A small digression.
My routers are set to knock. When they are not reachable, the error code returned is 0 - which suggests that everything is OK and there are no pending updates, which is not true 😉:
$ ./routeros-upgrader -n; echo $?
2024/07/26 11:34:54 checking installed packages
|DN> MikroTik1: unreachable
|DN> MikroTik2: unreachable
2024/07/26 11:34:54 no action required - exiting
0

Hi @issrec,
yeah it makes more sense in this case with a non-zero exit code.
I've now added an error to this. Please see this prerelease

Hi @Marco98,
I thank you for the change. It looks better:
$ ./routeros-upgrader -n; echo $?
2024/07/29 07:37:47 checking installed packages
|DN> MikroTik1: unreachable
|DN> MikroTik2: unreachable
2024/07/29 07:37:47 no action required - exiting
2024/07/29 07:37:47 fatal error: one or more routers unreachable
1

$ ./routeros-upgrader -n; echo $?
2024/07/29 07:38:39 checking installed packages
|DN> MikroTik1: unreachable
|UP> MikroTik2: dude-7.15.2-arm => dude-7.15.3-arm
|UP> MikroTik2: container-7.15.2-arm => container-7.15.3-arm
|UP> MikroTik2: routeros-7.15.2-arm => routeros-7.15.3-arm
|UP> MikroTik2: wireless-7.15.2-arm => wireless-7.15.3-arm
2

@issrec thank you!
Do you need additional tweaks for your automation or can i release the changes?

Overall it's OK and that's enough for my automation.
For now I see one/two situations that are not recognizable:
$ ./routeros-upgrader -n; echo $?
2024/07/26 11:35:28 checking installed packages
|UP> MikroTik1: routeros-7.15.2-powerpc => routeros-7.15.3-powerpc
|UP> MikroTik1: wireless-7.15.2-powerpc => wireless-7.15.3-powerpc
|UP> MikroTik2: dude-7.15.2-arm => dude-7.15.3-arm
|UP> MikroTik2: container-7.15.2-arm => container-7.15.3-arm
|UP> MikroTik2: routeros-7.15.2-arm => routeros-7.15.3-arm
|UP> MikroTik2: wireless-7.15.2-arm => wireless-7.15.3-arm
2

$ ./routeros-upgrader -n; echo $?
2024/07/29 07:38:39 checking installed packages
|DN> MikroTik1: unreachable
|UP> MikroTik2: dude-7.15.2-arm => dude-7.15.3-arm
|UP> MikroTik2: container-7.15.2-arm => container-7.15.3-arm
|UP> MikroTik2: routeros-7.15.2-arm => routeros-7.15.3-arm
|UP> MikroTik2: wireless-7.15.2-arm => wireless-7.15.3-arm
2

I thank you @Marco98 for the changes you made.


[SPARK-24114] Add instrumentation to FPGrowth.

What changes were proposed in this pull request?
Have FPGrowth keep track of model training using the Instrumentation class.
How was this patch tested?
manually
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #90687 has started for PR 21344 at commit 40c204b.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3266/
Test PASSed.

Merged build finished. Test PASSed.

@ludatabricks thanks for taking a loo, I added logParams.

Test build #90696 has started for PR 21344 at commit 5600718.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3274/
Test PASSed.

Merged build finished. Test PASSed.

LGTM.

Test build #90696 has finished for PR 21344 at commit 5600718.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90696/
Test PASSed.

I'll take a look now

LGTM
It'll be nice when we move the implementation to spark.ml in the future so that we can log more info, but this is good for now.
Thanks @MrBago and @ludatabricks !
Merging with master

Whoops!  This was merged linked to the wrong JIRA.  It should have been: https://issues.apache.org/jira/browse/SPARK-24310 for the record.


Conform to golang standard generated code comment

Generated files should have a comment stating that they are auto-generated. This allows automated tools to make exclusions regarding these files.
From Go's documentation:

To convey to humans and machine tools that code is generated, generated source should have a line that matches the following regular expression (in Go syntax):
^// Code generated .* DO NOT EDIT\.$
This line must appear before the first non-comment, non-blank text in the file.

Do you need these files to be detectable by automated tools? This is an exception because the code generated can be human edited. You can generate your schema structs and use it as the source of truth by adding or removing columns from it and generating the necessary SQL migrations needed to reach that state.
https://bokwoon.neocities.org/sqddl#generate

The generate subcommand generates migrations needed to get from a source schema to a destination schema. The source is typically a database URL/DSN (same as the -db flag), while the destination is typically a Go source file containing table structs.

Thanks for the prompt response!
I was actually referring to the files created using sqddl tables and sqddl views.
I assumed those shouldn't be editable. Should I not use these commands directly?

Oh the files created by sqddl tables are editable, yes you can generate them but you can also create them from scratch by hand. sqddl views can't really be edited because you can't define SQL views as struct annotations, but it's missing the generated code comment for symmetry with sqddl tables.

Got it. Thanks! I think you can close the issue :)


Crash on closing extension project

If you open an extension project and then close it really quickly, the front-end can crash. The problem occurs if the process that updates the examples list has not completed before the project window is closed.

Fixed by 224961ec1791963f53a68e370234e3d74839391f


survey question checkbox export

see https://github.com/expfactory/expfactory-surveys/issues/50

This has been fixed.


train on a custom dataset

hi! thanks for sharing your work :)
i am trying to train using a custom dataset, with only 1 class. i keep getting the following error, i believe it's because the number of classes is different from coco:
2020-11-10 18:53:53,782 - mmdet - INFO - load model from: open-mmlab://resnet50_caffe
2020-11-10 18:53:53,868 - mmdet - WARNING - The model and loaded state dict do not match exactly

unexpected key in source state_dict: conv1.bias, layer1.0.conv1.bias, layer1.0.conv2.bias, layer1.0.conv3.bias, layer1.0.downsample.0.bias, layer1.1.conv1.bias, layer1.1.conv2.bias, layer1.1.conv3.bias, layer1.2.conv1.bias, layer1.2.conv2.bias, layer1.2.conv3.bias, layer2.0.conv1.bias, layer2.0.conv2.bias, layer2.0.conv3.bias, layer2.0.downsample.0.bias, layer2.1.conv1.bias, layer2.1.conv2.bias, layer2.1.conv3.bias, layer2.2.conv1.bias, layer2.2.conv2.bias, layer2.2.conv3.bias, layer2.3.conv1.bias, layer2.3.conv2.bias, layer2.3.conv3.bias, layer3.0.conv1.bias, layer3.0.conv2.bias, layer3.0.conv3.bias, layer3.0.downsample.0.bias, layer3.1.conv1.bias, layer3.1.conv2.bias, layer3.1.conv3.bias, layer3.2.conv1.bias, layer3.2.conv2.bias, layer3.2.conv3.bias, layer3.3.conv1.bias, layer3.3.conv2.bias, layer3.3.conv3.bias, layer3.4.conv1.bias, layer3.4.conv2.bias, layer3.4.conv3.bias, layer3.5.conv1.bias, layer3.5.conv2.bias, layer3.5.conv3.bias, layer4.0.conv1.bias, layer4.0.conv2.bias, layer4.0.conv3.bias, layer4.0.downsample.0.bias, layer4.1.conv1.bias, layer4.1.conv2.bias, layer4.1.conv3.bias, layer4.2.conv1.bias, layer4.2.conv2.bias, layer4.2.conv3.bias

i changed the config file with the path to my dataset, which is in coco format.
should i do something else or are this changes enough? apparently the training is running fine.
thanks in advance!

@amaanda Thanks for interests. Maybe num_classes should be 2, including objects and background.

thanks for the quick reply! =)
hm i see. my dataset contains only one class, do you consider background as a class inside your code or should i edit my json to include this class?

i just tried running by only changing num_classes to 2 but i keep getting my losses = nan. so i guess it's the second option then?

@amaanda Usually, donot need to change json.

ok. thanks once again!

Hey I am having the same issue but I my class number seems to be correct and the json seems file.
My config:
# model settings
model = dict(
    type='SipMask',
    pretrained='open-mmlab://resnet50_caffe',
    backbone=dict(
        type='ResNet',
        depth=50,
        num_stages=4,
        out_indices=(0, 1, 2, 3),
        frozen_stages=1,
        norm_cfg=dict(type='BN', requires_grad=False),
        style='caffe'),
    neck=dict(
        type='FPN',
        in_channels=[256, 512, 1024, 2048],
        out_channels=256,
        start_level=1,
        add_extra_convs=True,
        extra_convs_on_inputs=False,  # use P5
        num_outs=5,
        relu_before_extra_convs=True),
    bbox_head=dict(
        type='SipMaskHead',
        num_classes=2,
        in_channels=256,
        stacked_convs=3,
        feat_channels=256,
        strides=[8, 16, 32, 64, 128],
        loss_cls=dict(
            type='FocalLoss',
            use_sigmoid=True,
            gamma=2.0,
            alpha=0.25,
            loss_weight=1.0),
        loss_bbox=dict(type='IoULoss', loss_weight=1.0),
        loss_centerness=dict(
            type='CrossEntropyLoss', use_sigmoid=True, loss_weight=1.0),
        center_sampling=True,
        center_sample_radius=1.5))
# training and testing settings
train_cfg = dict(
    assigner=dict(
        type='MaxIoUAssigner',
        pos_iou_thr=0.5,
        neg_iou_thr=0.4,
        min_pos_iou=0,
        ignore_iof_thr=-1),
    allowed_border=-1,
    pos_weight=-1,
    debug=False)
test_cfg = dict(
    nms_pre=200,
    min_bbox_size=0,
    score_thr=0.03,
    nms=dict(type='nms', iou_thr=0.5),
    max_per_img=10)
# dataset settings
dataset_type = 'CocoDataset'
data_root = './dateset/blob/'
img_norm_cfg = dict(
    mean=[102.9801, 115.9465, 122.7717], std=[1.0, 1.0, 1.0], to_rgb=False)
data = dict(
    imgs_per_gpu=8,
    workers_per_gpu=2,
    train=dict(
        type=dataset_type,
        ann_file=data_root + 'train/annotations.json_coco.json',
        img_prefix=data_root + 'train/',
        img_scale=(1200, 1200),
        img_norm_cfg=img_norm_cfg,
        size_divisor=32,
        flip_ratio=0.5,
        with_mask=True,
        with_crowd=False,
        with_label=True,
        with_track=True
    ),
    val=dict(
        type=dataset_type,
        ann_file=data_root + 'valid/annotations.json_coco.json',
        img_prefix=data_root + 'valid/',
        img_scale=(1200, 1200),
        img_norm_cfg=img_norm_cfg,
        size_divisor=32,
        flip_ratio=0,
        with_mask=True,
        with_crowd=False,
        with_label=True
    ),
    test=dict(
        type=dataset_type,
        ann_file=data_root + 'test/annotations.json_coco.json',
        img_prefix=data_root + 'test/',
        img_scale=(1200, 1200),
        img_norm_cfg=img_norm_cfg,
        size_divisor=32,
        flip_ratio=0,
        with_mask=False,
        with_label=False,
        test_mode=True
    )
)

# optimizer
optimizer = dict(
    type='SGD',
    lr=0.005,
    momentum=0.9,
    weight_decay=0.0001,
    paramwise_options=dict(bias_lr_mult=2., bias_decay_mult=0.))
optimizer_config = dict(grad_clip=None)
# learning policy
lr_config = dict(
    policy='step',
    warmup='linear',
    warmup_iters=1000,
    warmup_ratio=1.0 / 80,
    step=[8, 11])
checkpoint_config = dict(interval=1)
# yapf:disable
log_config = dict(
    interval=50,
    hooks=[
        dict(type='TextLoggerHook'),
        # dict(type='TensorboardLoggerHook')
    ])
# yapf:enable
# runtime settings
total_epochs = 12
device_ids = [0]
#dist_params = dict(backend='nccl')
log_level = 'INFO'
work_dir = './dataset/blob/result/'
load_from = './models/vis_sipmask_ms_1x_final.pth'
resume_from = None
workflow = [('train', 1)]

I am tryinng on a single gpu:
        {
            "name": "Train model",
            "type": "python",
            "request": "launch",
            "program": "${workspaceFolder}\\tools\\train.py",
            "console": "integratedTerminal",
            "args": [
                "configs\\sipmask\\sipmask_r50_caffe_fpn_gn_ms_1x_blob.py",
                "--validate",
                "--launcher=none"
            ]
        },

What am I doing wrong? Thanks in advance.

I managed to convert my data to RLE but I cannot train the modell still.
Exception has occurred: TypeError
forward_train() missing 3 required positional arguments: 'ref_img', 'ref_bboxes', and 'gt_pids'
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\models\detectors\base.py", line 86, in forward
    return self.forward_train(img, img_meta, **kwargs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\core\fp16\decorators.py", line 49, in new_func
    return old_func(*args, **kwargs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 38, in batch_processor
    losses = model(**data)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 221, in _non_dist_train
    runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 60, in train_detector
    _non_dist_train(model, dataset, cfg, validate=validate)
  File "D:\Repos\SipMask\SipMask-VIS\tools\train.py", line 106, in main
    logger=logger)
  File "D:\Repos\SipMask\SipMask-VIS\tools\train.py", line 110, in <module>
    main()

@JialeCao001 Any ideas?

@Traderain I cannot get your point. What is your final error?

I managed to convert my data to RLE but I cannot train the modell still.
Exception has occurred: TypeError
forward_train() missing 3 required positional arguments: 'ref_img', 'ref_bboxes', and 'gt_pids'
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\models\detectors\base.py", line 86, in forward
    return self.forward_train(img, img_meta, **kwargs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\core\fp16\decorators.py", line 49, in new_func
    return old_func(*args, **kwargs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 38, in batch_processor
    losses = model(**data)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 221, in _non_dist_train
    runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
  File "D:\Repos\SipMask\SipMask-VIS\mmdet\apis\train.py", line 60, in train_detector
    _non_dist_train(model, dataset, cfg, validate=validate)
  File "D:\Repos\SipMask\SipMask-VIS\tools\train.py", line 106, in main
    logger=logger)
  File "D:\Repos\SipMask\SipMask-VIS\tools\train.py", line 110, in <module>
    main()

@JialeCao001 Any ideas?

@JialeCao001 This is the final issue. My annotations are ms coco (rle)

@Traderain dataset_type = 'CocoDataset' should be changed to dataset_type = 'YTVOSDataset'

So this cant be trained with ms coco?
On 2020. Dec 4., Fri at 2:56, Jiale Cao<EMAIL_ADDRESS>wrote:

@Traderain https://github.com/Traderain dataset_type = 'CocoDataset'
should be changed to dataset_type = 'YTVOSDataset'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JialeCao001/SipMask/issues/26#issuecomment-738504604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQ5LVFAN3BOJW4IHSQRGVTSTA6UVANCNFSM4TRBJVMA
.

@Traderain For image instance segmentation, you can use coco-style annotations with SipMask-benchmark or SipMask-mmdet. For video instance segmentation, Sipmask-VIS only supports the yotube-vis style annotations.


Adding User-Agent to the DCA client

What does this PR do?
For troubleshooting purposes: Having a User-Agent in the header of the calls made by the Cluster Agent will help identify queries faster.
Motivation
Debugging.
Additional Notes
Anything else we should know when reviewing?

@CharlyF I think we can close this PR. the user agent was added in another pr
WDYT?


Should onsend events work with the new OWA look ?

On send events are not firing with the new OWA look on Office 365. This appears to be a bug, I hope it is corrected before official release.

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

ID: 423fb893-bedd-e004-5102-c13c4bea8d50
Version Independent ID: f28b807f-0f3b-0bf5-c967-257fc55ec7f9
Content: On send feature for Outlook add-ins - Outlook Developer
Content Source: docs/add-ins/outlook-on-send-addins.md
Product: outlook
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@b1osborne Thanks for your question! On send events are supported in the old/classic OWA in Office 365 and I've updated the article with that info (the change should be pushed soon but see the linked PR for the details).  We are working on this feature for new OWA and when it becomes available, we will update this article.
This feature is also in preview on Windows and Mac but we are in the process of adding info for those platforms to this article.
I'll close this issue now but if you have any other questions about this, let us know and we can reopen it.
Thanks.


Update math.algebraic.d#abs to return correct integral type

The documentation states the return type for an argument of integral types (assuming under phobos: (u)byte,short,int,long) should equal its own type.
Under current implementation this is not the case. Exemptions are made for short and byte.
Using isIntegral instead yields the correct behavior, also for unsigned types (which simply return themselves).

I think the weird treatment of byte and short is a remnant from the transition to new integer promotion rules of unary minus, but since that transition is over, it's good to remove that code.
This needs a changelog entry or bugzilla issue.

I'm not sure how or where to add this (and have difficulty accessing bugzilla at all).
If truly necessary, I'd gladly have some help 😅.

I filed an issue: https://issues.dlang.org/show_bug.cgi?id=24278
If you include "Fix 24278" or "Fix issue 24278" in one of the commit messages, the bot will auto-close the issue and add it to the change log.

I'm not sure why 2 tests fail. Their error message suggests a different issue.

I'm not sure why 2 tests fail. Their error message suggests a different issue.

cc @kinke is this caused by your recent makefile changes?

is this caused by your recent makefile changes?

Yes; fix here: https://github.com/dlang/dmd/pull/15911


net/http: transport caches permanently broken persistent connections if write error happens during h2 handshake

The fix for https://github.com/golang/go/issues/34978 is not correct, the issue is still present.
What version of Go are you using (go version)?

$ go version
go version go1.13.12 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/cfriesen/.cache/go-build"
GOENV="/home/cfriesen/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/cfriesen/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/cfriesen/devel/h2bug/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build843802376=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Test-case is here: https://github.com/mikedanese/h2bug
What did you expect to see?
The persistent connection is not cached by http.Transport.
What did you see instead?
The persistent connection is cached by http.Transport.
More
Commit https://go.googlesource.com/go/+/88186e5e232625f9c91d639e0cb90a88c6cf1172  was submitted in order to fix https://github.com/golang/go/issues/34978, but it appears to be not quite correct due to how http2/transport.go gets included into http via h2_bundle.go with an "http2" prefix added.
If I run the above testcase using "dlv test", then add a breakpoint in src/net/http/transport.go right after the assignment to isH2DialError, and then run "print pconn.alt", it gives me

(dlv) print pconn.alt
net/http.RoundTripper(golang.org/x/net/http2.erringRoundTripper) {
        err: error(*errors.errorString) *{
                s: "broken pipe",},}

At the same time, "isH2DialError" is "false" when it clearly should be "true" for the bugfix to be correct.
Trying to explicitly assert the types gave the following interesting results:

(dlv) print pconn.alt.(http2.erringRoundTripper)
golang.org/x/net/http2.erringRoundTripper {
        err: error(*errors.errorString) *{
                s: "broken pipe",},}
(dlv) print pconn.alt.(http.http2erringRoundTripper)
Command failed: interface conversion: net/http.RoundTripper is golang.org/x/net/http2.erringRoundTripper, not net/http.http2erringRoundTripper

Without rearchitecting the fix for #34978, one option is to change this assert:
https://github.com/golang/go/blob/fa98f46741f818913a8c11b877520a548715131f/src/net/http/transport.go#L1688
To check an interface that both http2erringRoundTripper and http2.erringRoundTripper implement.

cc @fraenkel

I can't see a way of fixing this without changing the http2 side to something we can detect on both sides.

This bug is fixed, isn't it?
https://go-review.googlesource.com/c/net/+/243257
https://go-review.googlesource.com/c/go/+/243258

I don't see it in release branches other than go 1.16, is it possible to backport it to go 1.15, go 1.14 by any chance?
Thanks, cc @fraenkel

@gopherbot please backport to 1.15, as Kubernetes is affected.

Any chance for 1.14 as well? 😂 We are using Kubernetes 1.18 cc @networkimprov

Only the latest 2 releases (1.15 and 1.16) are supported

@networkimprov thanks for opening https://github.com/golang/go/issues/45076, would be good to get this fixed in 1.15 since it can't easily be worked around by callers

Jordan, you may want to note that on #45076, as I have no influence on backport decisions.


Add Cryptagon to sites made with React-Static

I just added a new entry to the list of sites made with react-static

Awesome!