Long training time

Why did the training time for 10 batches increase several times when replacing the backbone network from CrossFormer-S to SMT-T, despite SMT-T having much fewer parameters and computations than CrossFormer-S?

Hello, our SMT-T model should have a higher throughput speed than CrossFormer-S (833 vs. 672 images/s). Theoretically, the training time should not differ significantly. Could you please provide more training details?

I am using the RefineDet detection model, training it on an RTX 3080 GPU with a batch size of 12. The only difference is the choice of the backbone network.

Hello, our SMT-T model should have a higher throughput speed than CrossFormer-S (833 vs. 672 images/s). Theoretically, the training time should not differ significantly. One possible reason is that depth-wise convolution is not very GPU-friendly during training, and our MHMC utilizes multiple depth-wise convolutions with relatively large kernels and channel shuffling, which may impact the actual training speed.  Thank you for bringing it to our attention. If longer training times are not feasible for you, we suggest either reducing the number of heads in MHMC or decreasing the number of blocks in the first two stages to achieve higher efficiency.

Thank you for your suggestion


Language is not persistent on preview page

When changing the language, the preview page should keep the language persistent, but it currently doesn't keep that information in the URL.

the attempt #349 by @SergeyPrimary
did not fix the issue. Directly after switching the language, the URL briefly shows the new value, but as soon as the data is updated, the value in the URL is reset to the old one


Delete unused timer variables

These variables I think are vestiges from when these addons cleared a timer as a way of staggering updates. Now they simply check the data variable itself.

Coverage increased (+0.008%) to 72.638% when pulling 2f168da1dc753dfec7ddbf4ad07a837284ffc79c on vincentwoo:unused-timer into 575c10114be5002b45d4d7842a6f8741ba87f5e3 on sourcelair:v3.


Encrypted py2py3

Tested: full compatibility with python 2 and 3 for encrypted method. The automatic merge is not possible because of the lines that I fixed with the other pull request. You can safely substitute thos three lines with the three in this branch. Please let me know if you want me to modify anything.

Should be fixed now. It used to work before that and it is continuing to work after both with py 2 and 3.


Trix/master/sa opae int

could also be solved by #include adapter.h in opae_int.h but that did not seem like the intent of opae_int.h

Coverage increased (+0.02%) to 81.185% when pulling 6cdec275bea9e18b51c2dbdd3b57414b8158c720 on trixirt:trix/master/sa-opae_int into 71ac5d4a7923826bc3b6c8e971b5b3b48a08044d on OPAE:master.


Question regarding PRIDICT Sequences Format and Source Code for Retraining

I am currently working on reproducing the results of the following work: 'Predicting the efficiency of prime editing guide RNAs in human cells.' To run the PRIDICT tool, the format of the sequence used as input should have the following characteristics: 'xxxxxxxxx(a/g)xxxxxxxxxx.' A minimum of 100 bases up and downstream of the brackets are needed. Unchanged edit-flanking bases should be placed outside of the brackets (e.g., xxxT(a/g)Cxxx instead of xxx(TAC/TGC)xxx).
I have noticed that in library 2 of the article's datasets, the PRIDICT sequence format(excel sheet) given does not follow this criterion. I would like to be informed if the sequences are being extended through the training process. If they are, would it be possible to send me the source code to retrain the tool?
Thank you for your time!

Hi! Quick follow up! Is there any update on the training code?
Thanks!
Peiheng

Hi Peiheng,
Sorry for the late reply!
We have added cleaned up training code for the updated model PRIDICT2.0 (access it here) where you'll find a few notebooks and can run the training workflow within them.
Hope this helps!
Best,
Nicolas


KOGITO-6854 Added create-your-first-workflow-service guide

JIRA: https://issues.redhat.com/browse/KOGITO-6854
This PR depends on: https://github.com/kiegroup/kogito-examples/pull/1266

@ricardozanini can you please review this PR?

Thanks for your review @akumar074, @ricardozanini, @MarianMacik.
I pushed the requested changes.

Great work @hbelmiro Just a couple of minor comments. Basically to clarify the purpose of the "default" dependencies.

Many thanks for the feedback, @fjtirado.
I pushed the suggested changes.

@hbelmiro Thank you for making the changes in the guide. The description of workflow is much better than I was hoping for. Great work there. I might have confused you when I was giving the examples of Understanding JQ expressions, Understanding workflow error handling and Handling events on workflows - I didn't meant to add them to the list of What's next items. I think "Getting Familiar With Our Tooling" and "Orchestration of OpenAPI Based Services" should be enough. Thank you

@hbelmiro Thank you for making the changes in the guide. The description of workflow is much better than I was hoping for. Great work there. I might have confused you when I was giving the examples of Understanding JQ expressions, Understanding workflow error handling and Handling events on workflows - I didn't meant to add them to the list of What's next items. I think "Getting Familiar With Our Tooling" and "Orchestration of OpenAPI Based Services" should be enough. Thank you

Many thanks for your feedback @ibek.
I removed those items from "What's next" and updated the guide with your other suggestions.

@hbelmiro Thank you. Could you please add the changes/commits to the PR so I could check them? I will then approve the PR.

@hbelmiro Thank you. Could you please add the changes/commits to the PR so I could check them? I will then approve the PR.

Sorry. I thought I had done.
Pushed now. Thanks.

np and thank you @hbelmiro  It looks great

Thanks for the review @krisv.
I pushed the suggested changes.

@akumar074 @amadhusu can we safely ignore these CI errors?

@hbelmiro This PRs branch does not have updated .github/workflows/sw-docs-pr.yml
Please compare
[1] https://github.com/hbelmiro/kogito-docs/blob/8a94411aede305e7ede4b53c904ded876247fd16/.github/workflows/sw-docs-pr.yml#L24
and
[2] https://github.com/kiegroup/kogito-docs/blob/e767491702eb4bcf4160211e6b1bd307572e3432/.github/workflows/sw-docs-pr.yml#L24

@hbelmiro This branch does not have an updated package.json file. It is missing the scripts key. You can compare it with the upstream/main branch.

we can maybe add the package-lock.json on gitignore if it is not needed.

+1
@akumar074 WDYT?

we can maybe add the package-lock.json on gitignore if it is not needed.

+1
@akumar074 WDYT?

@hbelmiro @spolti I think we should commit package-lock.json. package-lock.json serves a good purpose to keep dependency in check. Right now we just have 3 dependencies, but it may later increase.

@hbelmiro @ricardozanini I've added changes to the content. Please take a look and let me know if you're good with the changes.

Thanks for your improvements, @hmanwani-rh.
Looks great!

@hmanwani-rh it's ok, you can merge this PR.


Use file templates to improve security, reproducibility

By git-ignoring the files that will hold passwords, creating associated template files for them, and then conditionally creating the files with a couple of new make targets, we can help people install their configuration in a reproducible manner without compromising on security. The commits are separated logically, so you can cherry-pick what you see fit.

Nice contribution!


Obsolete, incomplete and otherwise non-working community code interfaces

Some of the interfaces in AMUSE are obsolete, incomplete or otherwise not working.
These interfaces should be updated, completed or removed rather than linger on.
Some examples of incomplete interfaces:

ramses (nowhere near complete, see #503)
krome (downloading fails)
mocassin (building fails, test doesn't include evolve step)
sphray (tests work but don't include the evolve step which fails)

Obsolete interfaces:

adaptb (made obsolete by Brutus)
octgrav (made obsolete by Bonsai)

As a rule, I think each community code should

have a working example script
have a test script that includes at the very least one evolve step and a test for each parameter

As a rule, I think each community code should

have a working example script
have a test script that includes at the very least one evolve step and a test for each parameter

krome is fixed now (downloads and builds)

and tests also work ok

I think sphray should work ok

otherwise, an error report would be handy

adaptb can probably be deleted
mocassin need to be checked
octgrav I am not sure (since I don't know well enough when bonsai works (ie which hardware) vs octgrav?)

see above


Adding gifs to readme.md and fixed bugs related to pdf, analytics commands

uploaded gifs to docs folder to attach in readme.md
fixed 2 bugs related to pdf and analytics commands.

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

Comparison is base (d38af5b) 81.25% compared to head (3596090) 81.28%.
Report is 9 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #57      +/-   ##
==========================================
+ Coverage   81.25%   81.28%   +0.03%
==========================================
  Files          27       27
  Lines        2192     2191       -1
==========================================
  Hits         1781     1781
+ Misses        411      410       -1

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


Move DNSProvider capabilities out of g/g

How to categorize this issue?

/area open-source
/kind enhancement
What would you like to be added:
Managing DNSProviders should be moved from gardener/gardener to gardener/gardener-extension-shoot-dns-service. The following approach was discussed:

[ ] Introduce a new feature gate to disable any handling of DNSProviders in g/g. Turning off DNSProviders should only work if UseDNSRecords is enabled.
[ ] Enhance DNSConfig.service.dns.extensions.gardener.cloud/v1alpha1 by providers.
~Add RawExtension field to shoot.spec.dns.providers in order to support options that are specific to a DNS extension (e.g. rate limit #5207).~
[ ] Add mutating webhook to gardener/gardener-extension-shoot-dns-service which replicates provider from shoot.spec.dns.providers to the DNSConfig as well as shoot.spec.dns.providers[].secretName to shoot.resources.
[ ] The DNS extension creates a shoot domain based DNSProvider based on the external DNSRecord object.
[ ] For additional DNS providers (non default-domain providers) the DNS extension is supposed to continue working with the managed-dns-provider role label (ref) to avoid orphaned providers when the switch happens.

Why is this needed:
Gardener has been transitioned from using the DNS APIs of gardener/external-dns-management to a slimmed down API dnsrecords.extensions.gardener.cloud which will eventually become the only option to create infrastructure related DNS records (e.g. internal, external, owner domains) once the corresponding feature graduates to GA (today beta #5219) and the feature gate is removed.
To accomplish this plan we need to move the DNSProvider handling from gardener/gardener to gardener/gardener-extension-shoot-dns-service, so that end-users can still use the DNS providers that are configured in their shoot clusters.

Hi @timuthy,
thanks for opening the issue and the summary! I have only one question:

Add RawExtension field to shoot.spec.dns.providers in order to support options that are specific to a DNS extension (e.g. rate limit Support configuration of rate limits for external and additional DNSProviders #5207).

Do we really need this? Can't the .spec.extensions[].providerConfig field be used for this?

@rfranzke the point about "advanced" configuration (2nd point in list) is a little bit different from what you proposed here. Any strong objections to not add a RawExtension field to shoot.spec.dns.providers? I know that your proposal more matches the general approach we took with extensions. However, I think due to historical reasons DNSProviders are a bit different and duplicating advanced configuration options in shoot.spec.extensions is not very user friendly.

@timuthy I think we should rather go back to exactly one provider in the .spec.dns section since gardenlet uses exactly one domain/provider for a shoot cluster. Any additional providers and any additional metadata information is solely related to shoot-dns-service. I think something like this would be a proper target picture:
spec:
  dns:
    domain: <shoot-domain>
#   provider: # specify only when no default domain
#     type: aws-route53
#     secretName: <secretName>
  extensions:
  - type: shoot-dns-service
    providerConfig:
      apiVersion: shoot-dns-service.extensions.config.gardener.cloud/v1alpha1
      kind: Configuration
      providers:
      - type: azure-dns
        primary: false
        zones:
          include:
          - zone-id-1
          exclude:
          - zone-id-2
      - primary: true # primary=true allows additional configuration for the same provider like in `.spec.dns`, there is no type field needed (only if it's not the default domain, i.e., only if `.spec.dns.provider` is provided)
        domains:
          include:
          - my-custom-domain.com
          - my-other-custom-domain.com
          exclude:
          - yet-another-custom-domain.com
        ...

Does this make sense? I understand that it's "a little inconvenient" to not have the configuration right at the same place like .spec.dns, though, I guess this is acceptable since the impact is quite low (you'd have to go to .spec.extensions.providerConfig anyways when configuring additional providers).
Keeping .spec.dns.providers and allowing a providerConfig encodes information that there is a DNS extension (different from the "native" DNSRecord extensions) that MIGHT make use of this into the core API. This still makes DNS extensions special although they are "just" regular Extension-based extensions. This doesn't seem right to me.

I think when we have introduced the core.gardener.cloud the gardener-apiserver has taken care of the conversion of fields that previously were "native" fields in garden.sapcloud.io but moved to providerConfig fields in core.gardener.cloud. Probably we can follow the same approach again and relax the extension from caring about this. In this case, it can only care about its providerConfig.
We can then plan to promote v1beta1 to v1 and deprecate v1alpha1 and v1beta1 (finally), and remove them eventually (and with it this conversion code).

It's definitely easier but based on some assumptions/facts:

The type of the extension is shoot-dns-service
The configuration is of type Configuration.shoot-dns-service.extensions.config.gardener.cloud because otherwise we don't know how to edit the config.
Gardenlet has to know how providers look like in Configuration.shoot-dns-service.extensions.config.gardener.cloud which prevents us from getting rid of that logic for a longer time (as you said, until we drop v1beta1).

I think we should anyways add a validating webhook for the enhanced config in the shoot DNS extension, so having an additional mutation won't hurt?!

OK, sure, valid points. If you prefer doing the conversion with a mutating webhook then it's even better for g/g. Thanks!

Please see also my comment about using secret binding resources: https://github.com/gardener/gardener/issues/3846#issuecomment-1014262097

Thanks @timuthy and @rfranzke for your considerations here. I like your proposals and what you already agreed on :)
One question about this step:

Actual provider migration: Gardenlet transfers ownership to extension by removing its finalizer after extension finalizer has been set (DNSEntries remain unchanged in the meantime).

I currently only see the dns.gardener.cloud/compound finalizer on DNSProvider objects. I assume, it is added by the dns-controller-manager. So what do you mean by "gardenlet [...] removing its finalizer"?

I currently only see the dns.gardener.cloud/compound finalizer on DNSProvider objects. I assume, it is added by the dns-controller-manager. So what do you mean by "gardenlet [...] removing its finalizer"?

My mistake. You are right, only the dns-controller-manager is adding a finalizer. Transferring ownership to the extension does not need any action from the Gardenlet. The DNSProvider should be left as it is and the extension takes care of updating or deleting it.
Can we use the annotation gardener.cloud/timestamp to identify DNSProviders created by the Gardenlet?

Can we use the annotation gardener.cloud/timestamp to identify DNSProviders created by the Gardenlet?

As discussed offline, we can use the managed-dns-provider role label for the clean-up logic that Gardenlet already uses today (ref).

I noticed another thing: The gardener/gardener-extension-shoot-cert-service extension or the backing gardener/cert-management component uses DNS01 challenges by creating DNSEntries. This will result in a direct dependency from gardener/gardener-extension-shoot-dns-service to gardener/gardener-extension-shoot-cert-service when the DNS capabilities are removed from gardener/gardener. (cc @MartinWeindel)

Good catch @timuthy, I think that's fair (to build an extension on top of the functionality of another extension). We should clearly document this in gardener/gardener-extension-shoot-cert-service and potentially add some start-up checks to ensure the gardener/gardener-extension-shoot-dns-service extension is installed as well (for example, it could check whether the dns.gardener.cloud/v1alpha1.DNSEntry CRD exists).

Small addition:
Once, the DNSProvider capabilities are moved, we should stop accepting DNSProvider in core.ControllerRegistration.Spec.Resources[].Kind.
There is probably some more code related to references to DNSProvider in ControllerRegistrations that can be cleaned up as well. For example, we can cleanup some stuff in pkg/gardenlet/controller/extensions.

DNSProvider management implementation has been already released with version v1.18.x of the shoot-dns-service extension. The new admission webhook of this extension is already deployed to all landscapes.
Next step is validation on a large landscape (canary) by activating the feature gate DisableDNSProviderManagement for the Gardenlet and enabling the DNS provider management in the shoot-dns-service.

Feature gate DisableDNSProviderManagement  has been enabled on both canary and live landscapes.
@timuthy
The task "Verify whether above changes work as expected on large landscapes" is completed.

Thanks for the update @MartinWeindel. Can we then proceed with the last open task which is

Drop dns.gardener.cloud related coding from g/g

from your point of view? Or should we wait for something else?

@rfranzke
From my point of view it is ok to proceed with "Drop dns.gardener.cloud related coding from g/g"

/kind cleanup

Before we cleanup dns.gardener.cloud related code in this repository, the DisableDNSProviderManagement feature gate should be promoted to GA (currently still alpha). Otherwise, it doesn't make sense to have it, because disabling it won't work anymore.
@MartinWeindel what are your plans for promoting the feature gate?

I have added a PR to promote DisableDNSProviderManagement feature gate to beta: #6142
Then I would suggest to set it to GA two releases later, i.e. with v1.52. Does this make sense?


503 DNS resolution failed for service: secretmanager.googleapis.com:443

Environment details

OS type and version: Ubuntu 22.04.1
Python version: 3.7.16
pip version: 22.0.4
google-cloud-secret-manager version: 2.7.2

Steps to reproduce

Run the code given below

Code example
from google.cloud import secretmanager
client = secretmanager.SecretManagerServiceClient()
name = f"projects/{project_id}/secrets/{secret_name}/versions/latest"
payload = client.access_secret_version(name=name).payload.data.decode("UTF-8")
print(payload)

Stack trace
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/google/api_core/grpc_helpers.py", line 67, in error_remapped_callable
    return callable_(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 946, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "/usr/local/lib/python3.7/site-packages/grpc/_channel.py", line 849, in _end_unary_response_blocking
    raise _InactiveRpcError(state)
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
	status = StatusCode.UNAVAILABLE
	details = "DNS resolution failed for service: secretmanager.googleapis.com:443"
	debug_error_string = "{"created":"@1677255939.419684061","description":"Resolver transient failure","file":"src/core/ext/filters/client_channel/client_channel.cc","file_line":1324,"referenced_errors":[{"created":"@1677255939.419665546","description":"DNS resolution failed for service: secretmanager.googleapis.com:443","file":"src/core/ext/filters/client_channel/resolver/dns/c_ares/dns_resolver_ares.cc","file_line":359,"grpc_status":14,"referenced_errors":[{"created":"@1677255939.419472016","description":"C-ares status is not ARES_SUCCESS qtype=A name=secretmanager.googleapis.com is_balancer=0: Timeout while contacting DNS servers","file":"src/core/ext/filters/client_channel/resolver/dns/c_ares/grpc_ares_wrapper.cc","file_line":698}]}]}"
>
The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/google/api_core/retry.py", line 188, in retry_target
    return target()
  File "/usr/local/lib/python3.7/site-packages/google/api_core/grpc_helpers.py", line 69, in error_remapped_callable
    six.raise_from(exceptions.from_grpc_error(exc), exc)
  File "<string>", line 3, in raise_from
google.api_core.exceptions.ServiceUnavailable: 503 DNS resolution failed for service: secretmanager.googleapis.com:443

I found a reference to the same error here, so please check if it's the same issue.

 It looks like a temporary failure.

I'm not able to produce this error. 5XX errors are usually related to the backend and in some cases can be transient. I'm going to close this issue but please open a new issue if the problem re-appears and we can escalate to the API team.


Support modifying command names

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 94.493%

Totals

Change from base Build<PHONE_NUMBER>0:
0.02%

Covered Lines:
652

Relevant Lines:
690

💛  - Coveralls


Update screenshots to latest version of PowerAI Vision

Should be usable as-is, but some screenshots look out-of-date.

closed with #38


Add minMaxInvalid flag to avoid unnecessary needPreprocess

Segment build: If min/max value is invalid (based on some definitions we have set like presence of , or empty), we skip putting the min/max value in segment metadata.properties.
Segment load: If either min or max is not available in segment metadata.properties, the ColumnMetadata for it returns null for both getMin and getMax. If both these values are null, but ColumnMinMaxGeneratorMode != NONE, we will mark this as "needPreprocess" and process the segment during segment load (either when it is ingested, or during server restart). As part of the preprocess however, we will again skip persisting the min/max value, since it is still invalid, and will always be for a segment with the same crc.
As a result, in the above combination (invalid min or max value), we always incur preprocessing cost during restart/load, even though the actual preprocessing is a noop. This may not seem much in Pinot using LocalSegmentFSDirectory, but can become very expensive in other implementations, where initing the SegmentDirectory might involve more expensive means. So we want to avoid getting into the else of this code when it is unnecessary. From BaseTableDataManager:
try {
      // If the segment is still kept by the server, then we can
      // either load it directly if it's still consistent with latest table config and schema;
      // or reprocess it to reflect latest table config and schema before loading.
      if (!ImmutableSegmentLoader.needPreprocess(segmentDirectory, indexLoadingConfig, schema)) {
        LOGGER.info("Segment: {} of table: {} is consistent with latest table config and schema", segmentName,
            _tableNameWithType);
      } else {
        LOGGER.info("Segment: {} of table: {} needs reprocess to reflect latest table config and schema", segmentName,
            _tableNameWithType);
        segmentDirectory.copyTo(indexDir);
        // Close the stale SegmentDirectory object and recreate it with reprocessed segment.
        closeSegmentDirectoryQuietly(segmentDirectory);
        ImmutableSegmentLoader.preprocess(indexDir, indexLoadingConfig, schema);
        segmentDirectory =
            initSegmentDirectory(segmentName, String.valueOf(zkMetadata.getCrc()), indexLoadingConfig, schema);
      }

Adding a flag in metadata.properties to detect this invalid min/max value case, so we can use it to avoid returning needPreprocess=true when preprocess would be a noop.

Codecov Report

Merging #9238 (edc8bd9) into master (6fddffe) will increase coverage by 35.02%.
The diff coverage is 81.81%.

@@              Coverage Diff              @@
##             master    #9238       +/-   ##
=============================================
+ Coverage     28.60%   63.63%   +35.02%
- Complexity       53     4743     +4690
=============================================
  Files          1844     1806       -38
  Lines         98570    96960     -1610
  Branches      15004    14828      -176
=============================================
+ Hits          28200    61702    +33502
+ Misses        67667    30710    -36957
- Partials       2703     4548     +1845

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.10% <81.81%> (?)

unittests2
15.39% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../columnminmaxvalue/ColumnMinMaxValueGenerator.java
73.68% <0.00%> (+73.68%)
:arrow_up:

...a/org/apache/pinot/segment/spi/ColumnMetadata.java
80.00% <0.00%> (+80.00%)
:arrow_up:

...java/org/apache/pinot/segment/spi/V1Constants.java
12.50% <ø> (+12.50%)
:arrow_up:

...ment/creator/impl/SegmentColumnarIndexCreator.java
78.93% <100.00%> (+78.93%)
:arrow_up:

...segment/spi/index/metadata/ColumnMetadataImpl.java
89.63% <100.00%> (+89.63%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1521 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Review Issue/Bug Template

Describe the solution you'd like

Templates for various parts of the project

Describe alternatives you've considered

Issues
Test
Standard
Help
Pull Requests

Additional context

The templates are currently the basic ones from Github
I'd like them to be reviewed and changed to be more suitable for the project
Feel free to add any so that we can review as a team to see whether they're applicable

@dcdarrell9 @treilly94 Is this still valid. I can see templates for issues/bugs in the repo already?

@emilioward yeah still valid, more about refining them and then adding anything that's missing

Leave them as is, change as we go.


Package request: HTTPie Desktop

Project description
HTTPie Desktop — cross-platform API testing client for humans. Painlessly test REST, GraphQL, and HTTP APIs.
Metadata

homepage URL: https://httpie.io/desktop
source URL: https://github.com/httpie/desktop#linux
license: unfree
platforms: linux, darwin

This package is currently closed-source I think. So literally no way to start it :)


Grad through a quantity function

I tried this thinking it might work and was surprised that it throws an error -- is this a bug or not something we can support at the moment?
import jax
import astropy.units as u
from unxt import Quantity
from unxt import experimental

length = u.get_physical_type("length")
time = u.get_physical_type("time")
velocity = u.get_physical_type("velocity")

@jax.jit
def test_ad1(x: Quantity[length], t: Quantity[time]) -> Quantity[velocity]:
    return x / t

experimental.grad(test_ad1, units="si", argnums=1)(
    Quantity(15.0, u.m), Quantity(1.0, u.s)
)

...
UnitConversionError: 'm' (length) and 's' (time) are not convertible

We should figure out how to support this!
The following does already work.
experimental.grad(test_ad1, units=(u.m, u.s), argnums=1)(
    Quantity(15.0, u.m), Quantity(1.0, u.s)
)

But just "si" would be simpler.


Status line shows file name as [No Name]

goneovim version: nightly-1-g531049e

settings.toml:
[Statusline]
Visible = true
Left = [ "mode", "filename" ]
# Right = [ "message", "git", "filetype", "fileformat", "fileencoding", "curpos", "lint" ]

P.S. I know, it should not be here, but worth adding status line colors configuration. White status line in dark goneovim window looks strange.

It start work if switch to another file:

goneovim test_file.txt -> [No Name]
':e test_file2.txt' -> test_file2.txt

Close this issue with #475.


Add Elm support

This adds support for the Elm language. The definitions are derived from the existing ones for Haskell, since the two languages are syntactically very similar.
@evancz (the designer of Elm), can you double check I got the keyword lists right?

LGTM, thanks @jvoigtlaender!

Thanks, merged!

Thanks for merging it in!


Linker errors with Gurobi

I have compiled or-tools from the source code to integrate it with the Gurobi solver. I have run the tests and examples and everything seems working properly.
I have also solved a simple LP problem in C with Gurobi using its C interface. However, when I try to solve the same LP problem with C++, I obtain the following linker errors:
simple_program.obj : error LNK2019: símbolo externo "public: __cdecl GRBEnv::GRBEnv(bool)" (??0GRBEnv@@QEAA@_N@Z) sin resolver al que se hace referencia en la función "void __cdecl operations_research::RunTest(enum operations_research::MPSolver::OptimizationProblemType)" (?RunTest@operations_research@@YAXW4OptimizationProblemType@MPSolver@1@@Z)
simple_program.obj : error LNK2019: símbolo externo "public: __cdecl GRBEnv::~GRBEnv(void)" (??1GRBEnv@@QEAA@XZ) sin resolver al que se hace referencia en la función "void __cdecl operations_research::RunTest(enum operations_research::MPSolver::OptimizationProblemType)" (?RunTest@operations_research@@YAXW4OptimizationProblemType@MPSolver@1@@Z)
simple_program.obj : error LNK2019: símbolo externo "public: __cdecl GRBModel::GRBModel(class GRBEnv const &,class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const &)" (??0GRBModel@@QEAA@AEBVGRBEnv@@AEBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@Z) sin resolver al que se hace referencia en la función "void __cdecl operations_research::RunTest(enum operations_research::MPSolver::OptimizationProblemType)" (?RunTest@operations_research@@YAXW4OptimizationProblemType@MPSolver@1@@Z)
simple_program.obj : error LNK2019: símbolo externo "public: __cdecl GRBModel::~GRBModel(void)" (??1GRBModel@@QEAA@XZ) sin resolver al que se hace referencia en la función "void __cdecl operations_research::RunTest(enum operations_research::MPSolver::OptimizationProblemType)" (?RunTest@operations_research@@YAXW4OptimizationProblemType@MPSolver@1@@Z)
simple_program.obj : error LNK2019: símbolo externo "public: void __cdecl GRBModel::optimize(void)" (?optimize@GRBModel@@QEAAXXZ) sin resolver al que se hace referencia en la función "void __cdecl operations_research::RunTest(enum operations_research::MPSolver::OptimizationProblemType)" (?RunTest@operations_research@@YAXW4OptimizationProblemType@MPSolver@1@@Z)
bin\\simple_program.exe : fatal error LNK1120: 5 externos sin resolver
tools\make: *** [bin/simple_program.exe] Error 2

Can we have the linking command line ? I would say there is a missing linkage with gurobi libraries...

I overwrote the simple_program from the getting started section, so the command I´m using is tools\make ccc EX=simple_program.cc. I have tried the flags -L and -l to index the library gurobi_c++md2017.lib but the command prompt keeps responding that it does not recognize them.
The installation of or-tools with Gurobi does not index its c++ libraries? If so, should I modify the file Makefile.user to include them?
Thanks!

You must define WINDOWS_GUROBI_DIR when compiling from source.
We currently use this:
https://github.com/google/or-tools/blob/d44fb1b423f9d6658c142c041143a4f54b5106d3/makefiles/Makefile.win.mk#L131-L143
and
https://github.com/google/or-tools/blob/d44fb1b423f9d6658c142c041143a4f54b5106d3/makefiles/Makefile.win.mk#L166
note: Currently we don't have CI jobs for compiling from source with gurobi support since we can't use it in Appveyor-CI (i.e. proprietary license issue)...

Note that the makefile expects gurobi<GUROBI_LIB_VERSION>.lib, no other
name.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 26 oct. 2018 à 11:24, Tresmele<EMAIL_ADDRESS>a écrit :

Yes, after executing tools\make third_party I defined both
WINDOWS_GUROBI_DIR and GUROBI_LIB_VERSION in the Makefile.local file and
then I executed tools\make cc.
I think it might be something related to the C++ interface with gurobi, as
the C interface seems to work properly.
Thank you very much for the rapid response.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/903#issuecomment-433345753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17UYwZ7QCzL-QOsE7YWbiNQtT9DYrks5uotS3gaJpZM4X6mxM
.

Yes, I just checked. I defined the constants as follows:
WINDOWS_GUROBI_DIR = C:\gurobi810
GUROBI_LIB_VERSION = 81

Yes but what are the .lib in the gurobi installation?
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le ven. 26 oct. 2018 à 11:38, Tresmele<EMAIL_ADDRESS>a écrit :

Yes, I just checked. I defined the constants as follows:
WINDOWS_GUROBI_DIR = C:\gurobi810
GUROBI_LIB_VERSION = 81
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/903#issuecomment-433349814,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17RD7QWAjzPXKKZBeOU1tJHBIOMWtks5uotf9gaJpZM4X6mxM
.

Inside the lib folder of the gurobi installation there are the following .lib files:
gurobi_c++md2015
gurobi_c++md2017
gurobi_c++mdd2015
gurobi_c++mdd2017
gurobi_c++mt2015
gurobi_c++mt2017
gurobi_c++mtd2015
gurobi_c++mtd2017
gurobi81

Tested everything with 7.2.
Should work now/


Remove or Add Steps Conditionally

Make sure these boxes are checked before submitting your issue - thank you!

[X] Check in the issue tracker whether
this issue wasn't already reported.
[ ] If it's a bug report check that clear steps to reproduce,
stacktrace, expected behaviour and other needed details are provided.
[ ] Feature requests contain a description of the feature
you're requesting as well as a brief scenario explaining
under what circumstances this might be useful.

I have a step wizard with 8 steps, and the last step is a summary of everything where the user can click on each information and go direct to this step.
Therefore, when the user select some specific services on the 4 step. I need to show or hide 3 further steps.
However, I cannot find a good way to implement this behavior.
case 0:
                final CompanyFragment client = new CompanyFragment();
                Bundle bClient = new Bundle();
                bClient.putInt(CURRENT_STEP_POSITION_KEY, position);
                client.setArguments(bClient);
                return client;
case 1:
                final BuildingFragment building = new BuildingFragment();
                Bundle bBuilding = new Bundle();
                bBuilding.putInt(CURRENT_STEP_POSITION_KEY, position);
                building.setArguments(bBuilding);
                return building;
case 2:
                final ShiftFragment shift = new ShiftFragment();
                Bundle bShift = new Bundle();
                bShift.putInt(CURRENT_STEP_POSITION_KEY, position);
                shift.setArguments(bShift);
                return shift;
case 3:
                final ServiceFragment service = new ServiceFragment();
                Bundle bService = new Bundle();
                bService.putInt(CURRENT_STEP_POSITION_KEY, position);
                service.setArguments(bService);
                return service;
case 4:
                final ServiceDetailsFragment serviceDetails = new ServiceDetailsFragment();
                Bundle bServiceDetails = new Bundle();
                bServiceDetails.putInt(CURRENT_STEP_POSITION_KEY, position);
                serviceDetails.setArguments(bServiceDetails);
                return serviceDetails;
case 5:
                final CarrierFragment carrier = new CarrierFragment();
                Bundle bCarrier = new Bundle();
                bCarrier.putInt(CURRENT_STEP_POSITION_KEY, position);
                carrier.setArguments(bCarrier);
                return carrier;
case 6:
                final VehicleFragment vehicle = new VehicleFragment();
                Bundle bVehicle = new Bundle();
                bVehicle.putInt(CURRENT_STEP_POSITION_KEY, position);
                vehicle.setArguments(bVehicle);
                return vehicle;
case 7:
                final PlateFragment plate = new PlateFragment();
                Bundle bPlate = new Bundle();
                bPlate.putInt(CURRENT_STEP_POSITION_KEY, position);
                plate.setArguments(bPlate);
                return plate;
case 8:
                final SummaryFragment summary = new SummaryFragment();
                Bundle bSummary = new Bundle();
                bSummary.putInt(CURRENT_STEP_POSITION_KEY, position);
                summary.setArguments(bSummary);
                return summary;

I need to hide the case 5, 6 and 7.

HistepperLayout.setAdapter(stepperAdapter, step)


Change project name getter command

What

Use gcloud config get-value to get project name

Why

Because the command exists for getting the project name already

@dorbin @plumpy I added the missing ). Take a look please.


[DOC] MUI change default font family example not working

Documentation issue
MUI Theme overriding for typography is not working. fontFamily is not changing
https://refine.dev/docs/api-reference/mui/theming/#overriding-variables
Describe the thing to improve
For typography override, we need to recreate the theme using createTheme
const typographyOptions: TypographyVariantsOptions = {
  fontFamily: [
    "Montserrat",
    "-apple-system",
    "BlinkMacSystemFont",
    '"Segoe UI"',
    "Roboto",
    '"Helvetica Neue"',
    "Arial",
    "sans-serif",
    '"Apple Color Emoji"',
    '"Segoe UI Emoji"',
    '"Segoe UI Symbol"',
  ].join(","),
};

const overridedLightTheme = createTheme({
  palette: RefineThemes.Blue.palette,
  typography: {
    ...typographyOptions,
  },
});

Describe the solution (optional)
No response

Hey @pentolbakso, thank you for reporting this and for providing the correct way! Do you want to send a PR for the changes? We'll be happy to see your contribution 🙏 🚀

@pentolbakso Thank you for helping us 🚀 We updated our documentation  🙌


Scarabs missing from fragments tab

I've just tested the changes from #910, and can not see any of the scarabs I have. I can still see other fragments such as sacrifice fragments, divine vessel etc, but none of the scarabs.
I've tried putting a scarab in a regular tab, and that shows up correctly and has the correct hover.
@aydjay @thailyn

Ah, it looks like we don't have any ItemDisplay in the XAML file for them.  Do you have more information on how the scarabs are laid out?  Right now @aydjay and I don't know what goes where.  Otherwise I could make a PR on top of @aydjay's (since he said he's unavailable for Christmas now) just putting them anywhere, and we can reorganize them as necessary, later.

Ok, from the few scarabs I have, and eyeballing the tab in game with the scarab art on the wiki, I think I've been able to work out all the placements.
As you're probably already aware, there are 12 different types of scarabs (Cartography, Divination, Sulphite, etc) and each of these have 3 quality levels- Rusted, Polished and Gilded.
I'll refer to the fragment tab and try explain the positioning,

The slots for the scarabs have been broken into 3 sections, each 3x4 slots. The columns are repeating in each of these sections, and correspond to the quality levels, from left to right Rusted, Polished and Gilded - the first column of the first section will be rusted, and so will the first column of the second section.
The rows for each section are for the type of scarab, and do not repeat per section - the first row in the first section will be a different type to the first row in the second section.
Now for the placements, below I've listed the type per row, per section. The ones with an asterisk are ones I've guestimated, the rest are ones I can confirm in my tab.
Section 1

1 - Bestiary
2 - Reliquary*
3 - Torment*
4 - Sulphite

Section 2

1 Ambush*
2 Shaper
3 Perandus
4 Cartography

Section 3

1 Harbinger*
2 Elder*
3 Divination*
4 Breach

Hopefully this is clear enough, and I haven't misplaced any of the ordering!

Nice work, can confirm that my scarabs are showing up in the correct spots after the merge


bug: overview does not update when switching tabs

steps to reproduce:

have lagrange open with 2 tabs on different pages
open a sidebar
set sidebar to show the headings on the site (overview)
switch between tabs

expected behavior:
overview updates to show the headings for the new site
actual behavior:
overview does not change
version (from about:lagrange):
Version 1.12.1

Fixed in v1.13.0.


webpack解析出错

./node_modules/dxf-viewer/src/DxfViewer.js 764:60
Module parse failed: Unexpected token (764:60)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
|         const material = materialFactory.call(this.viewer, this.viewer._TransformColor(color),

                                          instanceBatch?.GetInstanceType() ?? InstanceType.NONE)

|
|         let objConstructor

How did you get this error? Actually there are loaders configured here: https://github.com/vagran/dxf-viewer-example-src/blob/master/babel.config.js
and here: https://github.com/vagran/dxf-viewer-example-src/blob/0b16e6fa5365b4896e0c03e26ed428fa27b4b91c/vue.config.js#L5
Running npm run build should work.

@LIKEADOGANDCAT yes, if you use dxf-viewer in your own project, you need add transpileDependencies in your webpack config file, because ?? is new  operator in es2020

How did you get this error? Actually there are loaders configured here: https://github.com/vagran/dxf-viewer-example-src/blob/master/babel.config.js
and here:
https://github.com/vagran/dxf-viewer-example-src/blob/0b16e6fa5365b4896e0c03e26ed428fa27b4b91c/vue.config.js#L5
Running npm run build should work.
My project is a React project,I 'm not sure it has a 'transpileDependencies',,,I think it's unique to VUE projects.Then what should I do to use dxf-viewer in my REACT(umi) project?

How did you get this error? Actually there are loaders configured here: https://github.com/vagran/dxf-viewer-example-src/blob/master/babel.config.js
and here:
https://github.com/vagran/dxf-viewer-example-src/blob/0b16e6fa5365b4896e0c03e26ed428fa27b4b91c/vue.config.js#L5
Running npm run build should work.
My project is a React project,I 'm not sure it has a 'transpileDependencies',,,I think it's unique to VUE projects.Then what should I do to use dxf-viewer in my REACT(umi) project?

fixed in a new react project ：） 3q


Set up issue auto-assigner gh workflow

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


Dosent Run

When trying to run the Batch file i get the error message
Remove-ItemProperty : A positional parameter cannot be found that accepts argument ''.
At D:\jrose197\Desktop\me\Policies-Be-Gone-master\DisablePolicys.ps1:39 char:1

Remove-ItemProperty -Path "HKCU:\Software\Microsoft\Windows\CurrentVe ...

  + CategoryInfo          : InvalidArgument: (:) [Remove-ItemProperty], ParameterBindingException
  + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.RemoveItemPropertyCommand

This is expected behaviour after running the program multiple times, could you please confirm that it works still?

Was my fault sorry


(SERVER-534) Bump puppet submodule to 3.7.5

I think the "puppet version" test in master_spec.rb will need to be bumped up to "3.7.5".

Ahh yes, of course. Thanks, @camlow325

Are there any changes to the Gemfile or Gemfile.lock files that we should make to keep in sync with 3.7.5?

Also, should our facter submodule be bumped as well - not sure if a new one was shipped along with 3.7.5?

Spec tests all currently pass with the current lock file, and I'm guessing that if there hasn't been any change to our acceptance tests then it should be fine as is. Does this sound correct?

Looks like the Puppet Gemfile.lock specifies Facter 2.4.1. Does that sounds like a sane version to bump to?

Spec tests all currently pass with the current lock file, and I'm guessing that if there hasn't been any change to our acceptance tests then it should be fine as is.

It's probably ok.  I was just thinking that it would be best for all of our puppet-server CI jobs to be running with the same versions of gems as would the corresponding puppet platform CI jobs for 3.7.5.  For example, same beaker version.

Ahh ok, that makes sense.

Looks like the Puppet Gemfile.lock specifies Facter 2.4.1. Does that sounds like a sane version to bump to?

Good question.  When I do an install of 'puppet' from the latest package repos right now, I get version 2.4.3 of Facter.
@kylog For our "stable" branch of Puppet Server - 1.x series supporting Puppet 3.x - what version of Facter would you suggest we test along with the Puppet 3.7.5 version that we're bumping to?

Yeah I'd probably bump to 2.4.3. It just has misc fixes on top of 2.4.1 so should be low risk.
If it does give you heartache relative to 2.4.1, please let us know!

Cool, facter's been bumped up to 2.4.3

:+1: Looks good to me.  @nwolfe - you good with this?

:100:


Get-AdvancedThreatProtectionDocumentReport deprecated?

Hi folks,
Can you please confirm if this commandlet is deprecated?
There is no mention of it becoming deprecated in the docs, and the commandlet has not provided any warning that the command will become deprecated in the future (as has been the precedence with other deprecated commandlets).
At the moment the command does not run and results in an error from outlook.office365.com:
We cannot currently process your request, please wait a few minutes and try again.
FullyQualifiedErrorId : [Server=MEAPR01MB2421,RequestId=85b81df5-a926-4090-a3bd-8887c12a69f7,TimeStamp=10/03/2022 12:40:12 am] [FailureCategory=Cmdlet-FfoReportingException] 60318414,Microsoft
.Exchange.Management.FfoReporting.GetATPDocumentReport
There appears to be no other way to get this information exported into a useful format for reporting.
Can you please advise an alternative command that generates the same output?
Cheers,
Anthony

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

ID: f0268c7f-8a78-c46d-717a-dc88b4e9307a
Version Independent ID: 56e6b1ac-5304-0f03-ba51-52b75687bae1
Content: Get-AdvancedThreatProtectionDocumentReport (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Get-AdvancedThreatProtectionDocumentReport.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@anthonynz, checking.

Thanks Chris,
I should also clarify...

this error has persisted for the past couple of weeks.
O365 support suggested the command is deprecated.
I'm sure it was working in December.

@chrisda, wouldn't you know it, I had changes stating that this cmdlet was being deprecated (and other related updates) that were parked in a branch waiting to be published. I published them (PR will be referenced in this issue).
Thanks for the feedback.
Closing

Yes:

Get-AdvancedThreatProtectionDocumentDetail: This cmdlet will be deprecated. Use the Get-ContentMalwareMdoDetailReport cmdlet instead.
Get-AdvancedThreatProtectionDocumentReport: This cmdlet will be deprecated. Use the Get-ContentMalwareMdoAggregateReport cmdlet instead.

Brilliant thanks, you have been more informative than Office 365 support.
I look forward to seeing the two new commands released to production, is there an ETA on them going live?
I can see they're physically present in the ExchangeOnlineManagement V2 module, but currently do not function.
Get-ContentMalwareMdoAggregateReport
Get-ContentMalwareMdoDetailReport
From: Chris Davis @.>
Sent: Friday, 11 March 2022 8:54 am
To: MicrosoftDocs/office-docs-powershell @.>
Cc: Anthony Ross @.>; Mention @.>
Subject: Re: [MicrosoftDocs/office-docs-powershell] Get-AdvancedThreatProtectionDocumentReport deprecated? (Issue #9079)
Yes:

Get-AdvancedThreatProtectionDocumentDetail: This cmdlet will be deprecated. Use the Get-ContentMalwareMdoDetailReporthttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fpowershell%2Fmodule%2Fexchange%2Fget-contentmalwaremdodetailreport&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=uO5xNaG4zYr2VHdmbGbMDOZ4KN0H2orIFdGW%2FfhrZfs%3D&reserved=0 cmdlet instead.
Get-AdvancedThreatProtectionDocumentReport: This cmdlet will be deprecated. Use the Get-ContentMalwareMdoAggregateReporthttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fpowershell%2Fmodule%2Fexchange%2Fget-contentmalwaremdoaggregatereport&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=E66V0l2QF2ZlPdHZx9ST2hcq9Stvbhj%2BXgtzCfYUg2I%3D&reserved=0 cmdlet instead.

Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Foffice-docs-powershell%2Fissues%2F9079%23issuecomment-1064446062&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=f0ezMdEZZ384bRSQOrBc8QFr3pNhqzs%2BOsRqTQXMSSU%3D&reserved=0, or unsubscribehttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAID3YRHVQTZG5CNFF7B4GLU7JHPLANCNFSM5QLEATCQ&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=AmpoDGzQup9GvYcEDh2l%2F6EtHnitaTTicVEnheuP%2Fn0%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=r9RxrilLc2SkoM7Ol6IckMLwccoUteFW7xcprUqVWng%3D&reserved=0 or Androidhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|anthonyr%40datacom.co.nz|ab805e117c744a4027f408da02cfcb90|866c7a4c8a594bd3ad9f8512a581efc0|0|0|637825388746267512|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=M%2BzHbOTT038Pu4skyesFp5KuIr6yuIGpekG8YqLE95U%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

The new ContentMalwareMdo cmdlets are likely in the process of being rolled out. I see them in my stock PROD E5 tenant (no flighting or Preview), so if you don't see them yet, I presume you'll see them shortly.


Prevent timeout value from skirting limit check

Description of the Change
Original PR partially fixed the problem of specifying a timeout value that was too large;
unfortunately, it did not account for string-based timestamp translation.
This corrects that, and updates the function documentation.
Alternate Designs
N/A
Why should this be in core?
N/A
Benefits
All timeout values are now range limited, as originally intended.
Possible Drawbacks
None.
Applicable issues
Fixes #1652
semver-patch

Coverage increased (+0.09%) to 90.446% when pulling 96f5c18ed87bbed00c4054d802d267b7cfea8631 on plroebuck:issue/1652 into 741c4c794c89f9cf7c590e8c220c8c16162c0465 on mochajs:master.


some issue  on website build stage

[~] Website build with :
^CTraceback (most recent call last):
File "/app/omnisci3nt/omnisci3nt.py", line 299, in
programming_languages, technologies, javascript_libraries, web_server = analyze_website(website_url)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/app/omnisci3nt/buildwith.py", line 21, in analyze_website
technologies = builtwith.builtwith(url)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/builtwith/init.py", line 65, in builtwith
if contains(html, snippet):
^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/builtwith/init.py", line 111, in contains
return re.compile(regex.split('\;')[0], flags=re.IGNORECASE).search(v)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyboardInterrupt
Exception ignored in: <main.Tee object at 0xffffa3ee6ed0>
AttributeError: 'Tee' object has no attribute 'flush'

Seems like a keyboard interruption. we have tested it works fine for us. I don’t see any specific error you need to provide more info. if you are using on Windows use terminal(WSL2) instead of cmd/PowerShell otherwise, we recommend using it on Mac/Linux.

Hey
Yes of course I use Mac keyboard only.
What additional information would you like to have?


Other Changes List

"Custom command-line arguments can be flowed into IDesignTimeDbContextFactory, allowing applications to control how the context is created and initialized."

Please elaborate; how is this done?

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

ID: 32360b50-1129-b3d2-2d71-c6b89cd9f2c6
Version Independent ID: 4f18f8dc-5cd5-cb37-0f59-fb1767609b2b
Content: What's New in EF Core 5.0
Content Source: entity-framework/core/what-is-new/ef-core-5.0/whatsnew.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @ajcvickers
Microsoft Alias: avickers

@byronwjones See https://docs.microsoft.com/en-us/ef/core/cli/dbcontext-creation?tabs=dotnet-core-cli#args


Perf: use manual masking on state flags to avoid boxing

While profiling something unrelated i noticed that the highest allocation was boxing of an enum that was being used with HasFlags. This is netcore so it should be using the optimized code path but as noted in https://github.com/dotnet/SqlClient/pull/1054 this isn't happening. This PR changes from using HasFlag to manual masking.
before:

after:

LGTM.

I thought we were all in favor of HasFlag to be the recommended solution in this case.
Do we compromise performance over allocations?

In netfx flags usage should always be by masking because the compiler generates box instructions and the jit wasn't able to understand the pattern enough to remove them. Marking means that the enum is treated as it's integer base type and those don't require boxing. Here's an example of simple use:
using System;
public class C {
    [Flags]
    public enum MyFlags
    {
        None = 0,
        Flag1 = 2,
        Flag2 = 4
    }

    public bool Masked(MyFlags flags) {
        return (flags & MyFlags.Flag1) == MyFlags.Flag1;
    }

    public bool HasFlag(MyFlags flags)
    {
        return flags.HasFlag(MyFlags.Flag1);
    }
}

If you look at the IL generated for the masked case you see this:
        IL_0000: ldarg.1
        IL_0001: box C/MyFlags
        IL_0006: ldc.i4.2
        IL_0007: box C/MyFlags
        IL_000c: call instance bool [System.Private.CoreLib]System.Enum::HasFlag(class [System.Private.CoreLib]System.Enum)
        IL_0011: ret

and those box instructions are the problem, each one is a heap allocation that is short lived.
If you look at the assembly on netfx difference between the mask and hasflags approaches there's a clear difference and masking is superior:
C.Masked(MyFlags)
    L0000: test dl, 0x2
    L0003: setnz al
    L0006: movzx eax, al
    L0009: ret

C.HasFlag(MyFlags)
    L0000: push rdi
    L0001: push rsi
    L0002: sub rsp, 0x28
    L0006: mov esi, edx
    L0008: mov rcx, 0x7ffc50029748
    L0012: call 0x7ffca8682540
    L0017: mov rdi, rax
    L001a: mov [rdi+0x8], esi
    L001d: mov rcx, 0x7ffc50029748
    L0027: call 0x7ffca8682540
    L002c: mov rdx, rax
    L002f: mov dword [rdx+0x8], 0x2
    L0036: mov rcx, rdi
    L0039: call System.Enum.HasFlag(System.Enum)
    L003e: movzx eax, al
    L0041: add rsp, 0x28
    L0045: pop rsi
    L0046: pop rdi
    L0047: ret

In netcore 2.1 (i think, certainly around that timeframe) the jit team make Enum.HasFlag a jit intrinsic which means that the jit has a known pattern that it emits for that instruction sequence and this allows simplification of the surrounding box instruction interpretation by the jit. So the assembly on netcore is considerably better:
C.Masked(MyFlags)
    L0000: test dl, 2
    L0003: setne al
    L0006: movzx eax, al
    L0009: ret

C.HasFlag(MyFlags)
    L0000: test dl, 2
    L0003: setne al
    L0006: movzx eax, al
    L0009: ret

In general value.HasFlag(MyFlags.Flag1) is more easily read and understood than ((value & MyFlags.Flag1) == MyFlags.Flag1). I know I've made the mistake of missing a subtle `!=' in the mask version.
Because of this the advice in netcore is to use HasFlag wherever possible. The code changes in the netcore vesion of this library originate from the work I did when it was part of the corefx repo and was netcore only and at that time I was advised to use HasFlag instead of masking.
Today when profiling under netcore I can see that the HasFlag version is allocating which means that the jit intrinsic is being put into an instruction sequence that means it isn't possible to eliminate the boxes. Changing to masking will eliminate the boxes and will use the mask asm as seen above which is good.
Reproducing the problem in a way that the jit team can sensibly investigate will require running sqlclient on a custom built checked runtime so that the jit debug flags allowing dumping of compilation debugging information are available. I'm capable of doing this in theory but it's a daunting and time consuming process and even if it is fixed in NET7 that wouldn't improve perf on netcore3.1 to net6. Masking will.
So the short answer is that it should work but doesn't for a simples reason that it's hard to track down. Masking reduces allocations improving memory performance and the cost is in readability. I think everyone working on this library is used to dealing with much more complex code patterns than masking vs hasflags.

Today when profiling under netcore I can see that the HasFlag version is allocating which means that the jit intrinsic is being put into an instruction sequence that means it isn't possible to eliminate the boxes. Changing to masking will eliminate the boxes and will use the mask asm as seen above which is good.

Are you sure you weren't running in Debug or something? I'm not seeing any allocations with HasFlag, at least not on .NET 6 - see benchmark code below (with MemoryDiagnoser).
I mean, if the idea is to avoid HasFlag because you want to share code between netfx and netcore, and want netfx to be optimized, that's one thing (though as I previously wrote - with the general perf you get from netfx, I do doubt that this would be significant). But if you really think you're seeing a problem with HasFlag on netcore, it's probably worth confirming exactly what's going on before making a decision based on that.
BenchmarkDotNet=v0.13.0, OS=ubuntu 21.04
Intel Xeon W-2133 CPU 3.60GHz, 1 CPU, 12 logical and 6 physical cores
.NET SDK=6.0.100-rc.1.21379.2
[Host]     : .NET 6.0.0 (<IP_ADDRESS>802), X64 RyuJIT
DefaultJob : .NET 6.0.0 (<IP_ADDRESS>802), X64 RyuJIT

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

HasFlag
0.2700 ns
0.0094 ns
0.0079 ns
-
-
-
-

Bitwise
0.0118 ns
0.0050 ns
0.0041 ns
-
-
-
-

Benchmark code
BenchmarkRunner.Run<Program>();

[MemoryDiagnoser]
public class Program
{
    public MyFlags Flags { get; set; }

    [Benchmark]
    public bool HasFlag()
        => Flags.HasFlag(MyFlags.Flag1);

    [Benchmark]
    public bool Bitwise()
        => (Flags & MyFlags.Flag1) == MyFlags.Flag1;
}

[Flags]
public enum MyFlags
{
    None = 0,
    Flag1 = 2,
    Flag2 = 4
}

Yes. I'm sure I'm using the release build. I know HasFlag is supposed to generate non boxing code on netcore but I can see that in this case it is generating boxing code. In every non-sqlclient case I've tried it has generated non-boxing code. I've discussed it with some of the runtime team and in order to debug it needs a clear replication which is why I'd need a checked runtime.
This method is portable between netfx and netcore and despite being slightly less readable it will perform well. I don't really understand why I'm getting so much pushback. If you don't trust my profiling results then do your own and see what you find.

@Wraith2 regardless of the specific discussion here, from some Npgsql internal discussions (with @ninofloris and @vonzshik) we have seen some cases where a profiler shows allocations where BDN does not; this may be due to tiered compilation somehow not working under the profiler (but definitely working under BDN). Bottom line: when in doubt, I'd advise confirming that allocations indeed occur via BDN.

How do I identify if a specific allocation from all the other required allocations is taking place using BDN?
If the profiler(s) aren't working then that needs to be identified and reported. It's their job to get this stuff right. I'm profiling against a release build from the repo build script used through the nuget package so there are no weird debug or visual studio things going on. It's an as-release environment.

How do I identify if a specific allocation from all the other required allocations is taking place using BDN?

You're right about that - that needs a close-to-minimal repro, where you can compare two pieces of code which are identical except for the thing you're trying to test (or a really minimal piece of code which should have zero allocations).

If the profiler(s) aren't working then that needs to be identified and reported. It's their job to get this stuff right.

Agreed, but the first step for that is to make a reproducible code sample - which only you can do as you're the one running into it. The moment some sample program using SqlClient shows that particular allocation when profiled, that can be investigated.
Note that I don't have a confirmed case of profiler malfunction, just some possible weirdness.

@cmeyertons you reproduced a similar behaviour in https://github.com/dotnet/SqlClient/pull/1054 , is there any chance you could see if you can repro this one? It's a simple benchmark I'll paste below and I can see it boxing and unboxing SnapshottedStateFlags constantly on netcore:
	[MemoryDiagnoser]
	public class NonQueryBenchmarks
	{
		private string _connectionString = "Data Source=(local);Initial Catalog=Scratch;Trusted_Connection=true;Connect Timeout=1";

		private SqlConnection _connection;
		private SqlCommand _command;
		private CancellationTokenSource _cts;

		[GlobalSetup]
		public void GlobalSetup()
		{
			_connection = new SqlConnection(_connectionString);
			_connection.Open();
			_command = new SqlCommand("SELECT @@version", _connection);
			_cts = new CancellationTokenSource();
		}

		[GlobalCleanup]
		public void GlobalCleanup()
		{
			_connection.Dispose();
			_cts.Dispose();
		}

		[Benchmark]
		public async ValueTask<int> AsyncWithCancellation()
		{
			return await _command.ExecuteNonQueryAsync(_cts.Token);
		}
	}

Issue opened on dotMemory: https://youtrack.jetbrains.com/issue/PROF-1157
The answer to getting the disassembly was to add the DisassemblyDiagnoser with a large depth and then sift through the output to find what I wanted. Codegen was/is:
; Microsoft.Data.SqlClient.TdsParserStateObject.get_HasPendingData()
       movzx     eax,byte ptr [rcx+19F]
       test      al,2
       setne     al
       movzx     eax,al
       ret
; Total bytes of code 16

Which is fairly self explanatory and clearly doesn't box (no helper calls).
Regardless of that; I think that this update is portable between netfx and netcore and has only a small reduction in readability so I'd still like to proceed.

CI is green, no conflicts. Anyone?

1 file, 7 lines changed, CI is green, what is blocking this please?

I'm guessing uncertainty.
(Preface: I'm just here trying to mediate. I don't know whether we should merge this or not, but I do appreciate the contribution.)
From what I understand, this is a reversing of a previous perf/readability change and there is uncertainty around the scope of the perf problem. HasFlags is the forward-looking and recommended pattern for .NET and is much easier to read than masking. If it's incorrectly boxing in some scenarios, it will eventually be fixed to not box in the future, right? Perhaps even in an update? So then some time in the future we would want to reverse this reversal?
I can see that in your run and the scope you are showing, there appears to be a significant difference in allocations. However, what is the overall perf improvement in an end-to-end use case? Are we arguing over tens of thousands of allocations among millions? 10,000 rows/sec versus 10,001 in some hypothetical scenario? I'm honestly not sure which way to argue for this PR. I know we try really hard to squeeze perf out where we can, but given the discussion, it doesn't sound like everyone was convinced.
Regards,
David

These flags will be checked multiple times per query and on each field access because the contain various pieces of important state. They are masked together instead of being individual boolean fields for two reasons 1) it's makes the state object smaller and 2) they are copied as a group into async snapshots and the save/restore time is noticeable (but not a dominator) on profiles.
Every field access will need to check if the reader has advanced to the metadata and will cause a boxed enum value on every call to reader.Get* or reader.GetfieldValue<*> so the more you do the more garbage gets generated. They're gen0 allocations and very short lived so they're cleaned up quickly, however the optimal case for garbage collection is for it to never happen. The more time spend in GC the less time you can do user work.
In netfx up to current and last version using enum.HasFlags will box. On netcore 3.1 and later where tiered compilation is available+enabled the tier0 (quickly produced, relatively unoptimized) code generated by the jit will box in the same way as netfx. The tier1 code is produced once a method execution count hits a specific level and the in the tier1 compilation there is an optimization that recognises the HasFlags pattern and removes the box. Manual masking will generate optimal code even at tier0 on netcore and will generated box free code on netfx.
I would prefer to use HasFlags if readability were the only concern. I acknowledge that the manual masking version is slightly less readable but I feel that the slight drawback for the small number of people who work on this code is heavily outweighed by the efficiency that all users of this library gain. It's a case of <10 people taking a few extra seconds to think through the masking vs many thousands of people spending less electricity to query their data.

In netfx up to current and last version using enum.HasFlags will box. On netcore 3.1 and later where tiered compilation is available+enabled the tier0 (quickly produced, relatively unoptimized) code generated by the jit will box in the same way as netfx. The tier1 code is produced once a method execution count hits a specific level and the in the tier1 compilation there is an optimization that recognises the HasFlags pattern and removes the box. Manual masking will generate optimal code even at tier0 on netcore and will generated box free code on netfx.

This .NET Core angle doesn't make sense to me. Tiered compilation is a completely standard feature of JITing, and trying to micro-optimize something which only occurs in tier0 is, well, odd. The whole point of tiered compilation is that the slow code gets optimized automatically by the JIT after a few iterations, so any perf gains are completely negligible and don't justify worse readability...
For .NET Framework, if you believe reducing these gen0 allocations justify worse readability, then this may make sense. But I'd argue that .NET Framework is already inefficient in so many ways (compared to .NET Core), that this is again unlikely to matter. Anyone who cares even a little about perf should already be in .NET Core for a thousand reasons which are more important than this allocation.

On netfx we still use individual with internal access and touch them directly. The change to add the properties and implement them using flags was made in netcore only because at the time it was part of corefx. netfx currently doesn't incur the cost of boxing here and it will be a reduction in performance when the two are merged to use the common codebase.
In general I don't put a lot of effort into netfx perf improvements for exactly the reason you stated, there are bigger problems. Not using an approach which is superior on all targets in various degrees just because it benefits netfx slightly is punitive and discriminatory. There are people who use and will continue to use netfx and if we can make their code more efficient with minor effort on our part we should.

You can define a private bool GetSnapshottedState(...) method that parallels SetSnapshottedState to address the readability concern.
FWIW, Enum.HasFlags vs. manual masking is a personal style choice. I do not think we have a guideline anywhere that HasFlags is the recommended pattern. In fact, the guideline has been exact opposite for a quite while since the JIT was not optimizing it as intrinsic. The JIT in .NET Core/5+ is optimizing it now, but this optimization is not guaranteed to happen. In addition to tiered compilation, there are other situations (e.g. too complex method) where this optimization may not happen. If you want to write a code that is fast by construction, avoid Enum.HasFlags.

@jkotas Not to digress too much from the main topic, but what was the reason that the Enum.HasFlags optimization in .NET Core/5+ was implemented as a JIT intrinsic instead of a compiler intrinsic?

I assume that you mean C# compiler (Roslyn) intrinsic. Roslyn has very minimal understanding of the base class library behaviors. It does not expand any methods as intrinsics, with a very few exceptions of methods on core types e.g. == operator on strings. It is intentional design decision to avoid duplicating logic from the runtime libraries.

Ah, yes, that's what I meant. Got it, thanks! 🙂

Updated to use an accessor function GetSnapshottedState as suggested. CI is failing on unrelated mars things as if often does. 8 lines, coming up to 6 months to review them. If we're working at one line per month have i just added another 4 months wait time? or are { }'s free in which case it's just 2?

Yay!


Use new Figure component

incorporates new Figure component from components, which can be used to wrap a figure (or table) component to (a) avoid breaking on print and (b) standardize vertical margin and spacing between component and caption

references new components version with Figure and updated Caption (https://github.com/carbonplan/components/pull/136)

Addresses figure concerns in https://github.com/carbonplan/research/issues/206

looking really good! big push here, but a very nice clean up


Constants import path

I think you miss "./" in the path to constants.js file
In the file ReactCursorPosition.js, line 6
import * as constants from 'constants';
The correct path is:
import * as constants from './constants';
I know you way is also accepted, but it can affect aliases created by webpack

@tekbreak - Thanks for reporting this issue! I really appreciate it. I think it is corrected in release v2.4.1. Please have a look! If you find react-cursor-position useful, please consider starring it on GitHub. Starring helps to grow the project and that benefits everyone who uses it. Thanks again!

Closing as fixed in v2.4.1


Unable to refer the json file for Security profiles

API-Mgr. CLI version

Please make sure to use the latest available version before reporting a bug.
7.7
API-Manager and Service-Pack Version
7.7

Expected behavior
To refer the json file for Security profiles updation
Actual behavior
Unable to refer the json file for Security profiles, getting Cant parse JSON-Config file(s)
Log-Output in Debug
ERROR         APIImportApp| Cant parse JSON-Config file(s)
Test case sample

Please describe the steps you have done to replicate the issue

We have added the below line in config.json, PFA for reference.
"securityProfiles": "MSC2.json",
"outboundProfiles": "requestMscJWT-noAuth.json",
We have validated the json files and all are valid one.

All the json files are available under same repo folder, We can able to refer the Swagger.json but getting issue while referring SecurityProfiles json.

Hi @meharunissa-Basheer, What you try to do is unfortunately not supported by the CLI.

Hello Cwiechmann,
Will it be supported in the upcoming versions, If yes kindly give us the tentative month when it will be released.
Do we have a simple way to refer the multiple security profiles, outbound profiles while deploying an API. Kindly suggest.

Hi @meharunissa-Basheer, There is currently no plan to add support for this requirement. All belonging profiles must be part of the API-Configuration file.
Of course, you may use a script/java-tool that is preparing the JSON-File by resolving external references before finally handing it over to the APIM-CLI. This might run, as an extra step in your CI-Pipeline.

Closing this issue now. Feel free to re-open if you have any further questions @meharunissa-Basheer.


Adjust max-connections configuration.

We need the variable to adjust the configurations of AmazoneMQ.
https://docs.aws.amazon.com/amazon-mq/latest/developer-guide/rabbitmq-defaults.html
such as: max-connections, max-queues,...

we can refer to:
https://github.com/hashicorp/terraform-provider-aws/issues/4846

Claude.ai
Here is how you can adjust the max-connections setting for an AWS MQ broker using Terraform:

Find the aws_mq_broker resource in your Terraform config that is creating the broker. It will look something like:

resource "aws_mq_broker" "example" {
  #... other config
}

Add a configuration block inside the resource:

resource "aws_mq_broker" "example" {

  #... other config

  configuration {
    id       = aws_mq_configuration.example.id
    revision = aws_mq_configuration.example.latest_revision
  }

}

Create an aws_mq_configuration resource to define the configuration:

resource "aws_mq_configuration" "example" {
  name           = "example-config"
  engine_type    = "ActiveMQ"
  engine_version = "5.15.0"

  data = <<DATA
maxConnections={desired max connections}
DATA
}

Set the maxConnections value inside the data block to your desired max connections limit.

This will attach the configuration to the broker and set the maxConnections value when it is created/updated.
A few key points:

The configuration block links the broker to the config resource.
The aws_mq_configuration resource defines the actual config.
Make sure the engine type/version matches your broker.
Adjust other settings as needed in the data block.

This allows managing AWS MQ broker configs through Terraform.

We can't change the max-connection configuration via terraform.

closed


Search in keywords by default + Allow searching in specific field like authorName.

Telerik UI should be shown when searching for listview (we should search in keywords)
uatmarket.nativescript.org?search=authorName:Zdravko should return all plugins created by someone with Zdravko in its authorName (same with 'description', 'keywords', 'displayName', 'name', 'isVerified', 'version')

I'm not sure whether 'isVerified' is working.


There is a girl who did this to me just now successfully, through Fiverr, do you want me to send you the link for her, talk to her and give her the required information and she will book for you

Hello
There is a girl who did this to me just now successfully, through Fiverr, do you want me to send you the link for her, talk to her and give her the required information and she will book for you

YESS pls

YESS pls

https://www.fiverr.com/dr_spiritual/book-you-a-place-in-nz-miq
CONTACT WITH HER NOW

https://www.fiverr.com/dr_spiritual/book-you-a-place-in-nz-miq

I'm not saying this is a scam, but be careful about scams everyone!

YESS pls @nz-matt

Any luck?

I'm not saying this is a scam, but be careful about scams everyone! Also, this could is kind of marketing so I'm not sure if this 'issue' should perhaps be deleted by admins

I'm unsure about this either. Generally, I believe access to MIQ should be free and equal to everyone, and no one should make money with it. However, a few people have reported being unable to use these tools (lack of skills, time or technology, for example) and have asked for paid help.
So I think people offering to reserve spots at a reasonable price can be useful to some people. But then again, I'm not sure if the repo issues section is a good place to collect services either.
With this particular one, I would be careful too. Generally, there is little information about the process in that offer. From the offer:

If you want the process quickly and you cannot wait for the delivery days, talk to me, I will speed up the process for you and I will send you another offer

I'm unsure how one could speed the process up even more.
What are your opinions?

I'm not saying this is a scam, but be careful about scams everyone! Also, this could is kind of marketing so I'm not sure if this 'issue' should perhaps be deleted by admins

I'm unsure about this either. Generally, I believe access to MIQ should be free and equal to everyone, and no one should make money with it. However, a few people have reported being unable to use these tools (lack of skills, time or technology, for example) and have asked for paid help.
So I think people offering to reserve spots at a reasonable price can be useful to some people. But then again, I'm not sure if the repo issues section is a good place to collect services either.
With this particular one, I would be careful too. Generally, there is little information about the process in that offer. From the offer:

If you want the process quickly and you cannot wait for the delivery days, talk to me, I will speed up the process for you and I will send you another offer

I'm unsure how one could speed the process up even more.
What are your opinions?

I looked at a few more on Fivver and they mostly had references. Honestly this particular one just seems a little bit iffy.
As to the issue of paying to secure a voucher it's something I have been exploring for some of the reasons you mention (lack of time and skills mostly).
I'm cognizant that there are issues of fairness involved and maybe some people potentially unable to afford to pay but frankly when it comes down to it I'm just getting desperate and rapidly running out of both times and options--I can't leave it to chance, might sound melodramatic but my kids well-being is at stake.

Ps. None of which is to say I'm not massively grateful for this tool either because I am so thanks very much, I've been using it but it just hasn't worked out well for me I always seem too slow (I don't think I'm in a great time-zone for it either)

Listen all of you here, I just shared with you the girl who helped me, it's normal for someone to help you for money, also if you don't know and want someone to do it for you quickly I shared with you the girl who can open your problems to all of you, either a scam or something like that, the site Fiverr protects you from this, you will pay the money and will give you time, for example, 7 days and will give you your account with the reservation, if the girl does not succeed in this Fiverr will refund the money to you, where is the problem,,
Help for money, you should free your mind, don't criticize when you haven't tried anything

I'm not saying this is a scam, but be careful about scams everyone! Also, this could is kind of marketing so I'm not sure if this 'issue' should perhaps be deleted by admins

I'm unsure about this either. Generally, I believe access to MIQ should be free and equal to everyone, and no one should make money with it. However, a few people have reported being unable to use these tools (lack of skills, time or technology, for example) and have asked for paid help.
So I think people offering to reserve spots at a reasonable price can be useful to some people. But then again, I'm not sure if the repo issues section is a good place to collect services either.
With this particular one, I would be careful too. Generally, there is little information about the process in that offer. From the offer:

If you want the process quickly and you cannot wait for the delivery days, talk to me, I will speed up the process for you and I will send you another offer

I'm unsure how one could speed the process up even more.
What are your opinions?

Listen all of you here, I just shared with you the girl who helped me, it's normal for someone to help you for money, also if you don't know and want someone to do it for you quickly I shared with you the girl who can open your problems to all of you, either a scam or something like that, the site Fiverr protects you from this, you will pay the money and will give you time, for example, 7 days and will give you your account with the reservation, if the girl does not succeed in this Fiverr will refund the money to you, where is the problem,,
Help for money, you should free your mind, don't criticize when you haven't tried anything

@simimoha cool, but she wasn't good enough for you to leave a review on Fiverr?

(deleted a double post)

Hi everyone, I just want to add to the comment about paid services, verses trying to do the booking by yourself. I liken this service to using the MIQ booking app (which by the way is a fantastic tool to use considering NZ governments lack of fair play for getting back home). That being said the MIQ tool gives an advantage over the average non pc literate person trying to get home and we are all very comfortable using it just as if a paid service can assist in booking a spot then why not.
Until the NZ government comes up with a better fairer means of trying to get back to the country these systems are there to help get an advantage over the average kiwi looking for a spot.


Is it possible to have both sticky and normal headers together?

As from title.

Not foreseen, so not supported.
You can however create a Header with Expandables below where the expandable item (parent) is visually another header.


Add screenshots to setup instructions

It seems some users are trying to register end-user app before login in the distributor

outdated


Use Required flag for CLI parameters where needed

See #2852, we can simplify some code by relying on Required flag (remove error checks), but a thorough review of whole CLI command set is needed.

We've tried with #3278, and turns out that Required functionality of urfave/cli not completely suits our needs. See the comment. Thus, we're closing this issue in favour of https://github.com/nspcc-dev/neo-go/issues/3097.

There's a way to validate flags with the new version of urfave/cli, ref. https://github.com/nspcc-dev/neo-go/issues/3097#issuecomment-2191090318, we can combine Recuired functionality with validation to reach our goal.


Getting two socket replies instead of one

My code:
_socket = PhoenixSocket(url, socketOptions: _socketOptions())..connect();
_socket.openStream.listen(_onStreamEvent);

void _onStreamEvent(PhoenixSocketOpenEvent event) {
    channel = _socket.addChannel(topic: 'user:${user.id}');
    channel?.join().future.then((response) async {
      if (response.isOk) {
        onChannelJoin?.call();
        await for (final message in channel!.messages) {
          onChannelMessage?.call(message);
        }
      }
    });

    onConnected?.call();
  }

When I receive a reply from socket, there are two messages - one is PhoenixChannelEvent(phx_reply) and another is PhoenixChannelEvent(chan_reply_2) (with ref id)
Is that intended behavior or I am doing something wrong? Both of them responds to event.isReply, so I had to add an additional condition to prevent my code running twice.

Ok, there was an issue in my code. Solution is:
    _socket = PhoenixSocket(url, socketOptions: _socketOptions())..connect();

    _socket.openStream.listen(_onStreamEvent);

    channel = _socket.addChannel(topic: 'user:${user.id}');
    channel?.join().future.then((response) async {
      if (response.isOk) {
        onChannelJoin?.call();
        await for (final message in channel!.messages) {
          onChannelMessage?.call(message);
        }
      }
    });

     void _onStreamEvent(PhoenixSocketOpenEvent event) {
        onConnected?.call();
     }

I've also started to see some unexpected behavior like this, I'm wondering if the exact implementation of Phoenix has changed the protocol a bit.
What version of Phoenix are you using?

What version of Phoenix are you using?

I use 1.6.11 Phoenix and 2.1.1 PubSub

the same issue

Took me a little digging around to figure out how to distinguish the two types of response
extension PhoenixChannelReplyIdentification on PhoenixChannelEvent {
  static const String __chanReplyEventName = 'chan_reply';
  static const String __replyEventName = 'phx_reply';
  bool get isChanReply => value.startsWith(__chanReplyEventName);
  bool get isPhxReply => value.startsWith(__replyEventName);
}

If I inspect the websocket messages I only see the phx_reply, there's no sign of the chan_reply messages which leads me to believe it's client side somehow.

Fix is in the 0.7.2 release


Support multiple addresses per listener

Via https://github.com/envoyproxy/envoy/issues/11124. This would avoid mass listener duplication in cases where a listener config needs to be bound to multiple addresses.

This may be required for some QUIC use cases. If Envoy is terminating QUIC and sets a Server Preferred Address, packets will start arriving on a different listener, but are part of the same QUIC connection. So packets from different listeners will need to go to the same QUIC connection router.
Same story for QUIC connection migration.
This will also remove duplication if both HTTP/2 and HTTP/3 are needed for the same configuration (UDP vs TCP listeners).

FYI the same for contour which also uses envoy https://github.com/projectcontour/contour/pull/1162

I'm trying to work out a PoC for this issue

This may be required for some QUIC use cases. If Envoy is terminating QUIC and sets a Server Preferred Address, packets will start arriving on a different listener, but are part of the same QUIC connection. So packets from different listeners will need to go to the same QUIC connection router.
Same story for QUIC connection migration.

I think supporting dual-stack ipv6 sockets and QUIC server migration are different things. The former only needs a 2nd listener with the same config but listening on a different socket. But QUIC server migration requires one connection lands on the same listener. #19367 seems to implement the former, but I don't see how it helps with the latter. QUIC server migration might be easier to implement in L4 load balancer rather than requires one listener to listen on both sockets. Google is planning to experiment server migration, but we don't have a good story to tell yet.


feat: support nullable api response data

need new test case to cover the change between 'NULL' and null

Yes, this PR should wait for changes in databend.


feat(collector): added support for<EMAIL_ADDRESS>

<EMAIL_ADDRESS>is using aws-sdk@3 under the hood.
FYI: When using the latest version of @aws-sdk/client-sqs to send a message to the receiver, it does not work and throws an error. Reported.

Added batch cases & support.


Naive Bayes | Python

Title -
what will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Code
[x] Improving Documentation
[x] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[x] Python
[x] C++
[x] Java
[x] C
[x] Go
[x] Other language

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding,

assigned, dont forget to add it in the respective folder


Enable cascading deletes on accession child tables

The initial version of the app didn't allow deleting accessions, so I erred on the
side of safety and didn't put ON DELETE CASCADE clauses on the foreign-key
constraints on the various child tables.
Now we support deleting accessions, though, so it makes sense to modify the
constraints to auto-delete rows in tables that exist purely to hold lists of
values in accessions. There is never a situation where, e.g., we'll want the
existence of a viability test on an accession to cause us to be unable to delete
the accession.
However, we do want that behavior for accession photos: if we haven't deleted
an accession's photos from the file store yet, we shouldn't be able to delete the
accession since it would cause the files to be orphaned.

Current dependencies on/for this PR:

main

PR #546

PR #547

PR #548   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: j30sQfEOmb5qXpOUSK81


BUG REPORT Add a counting animation to the 'About Us' page.

Description
Add a counting animation to the 'About Us' page to display values when scrolling.
Screenshots

Additional information
Benefits of the Correction:

Enhances the professionalism of our website.

I am a GSSOC Contributor.
What browser are you seeing this bug on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have read the Code Of Conduct
[X] The changes don't break the code

@Muskansahuincredible, please assign this issue to me!

Idea looks great. Go ahead!
@nikhildeshmukh170


Deleting email invite fails for unkown users

Clickling the delete button on an email invite fails if there is no user object.
The error reads: 'NoneType' object has no attribute 'first_name',
Exception Location: | /usr/local/lib/python3.7/site-packages/rdmo/accounts/utils.py in get_full_name, line 14
How to reproduce:
In RDMO 1.5.5 invite some email address not yet known to the system. Delete that email from invites afterwards.

Hi @ubbo , thanks for reporting, I will look into it.


Upstream VQLS into Qiskit Prototype

Victoria
from IBMQ suggested that we could upstream VQLS to the Qiskit prototype : https://github.com/qiskit-community/prototypes

Issue opened :)


choose photos that are on cloud

Are you planning on adding support to pick images that are on cloud (such as from Google Photos)?

Would be nice to have it

bump, for me this seems to be an essential feature..
will try to check how the native picker does this


Allow subpixel center on view

This change reverts #3933 and brings back subpixel exact view centering. The JSFiddle linked in #3933 works after applying the changes from this pull request, indicating the underlying issue has been fixed elsewhere (probably #3964) in the meantime.
Fixes #5596.

Looks good to me.


Treating async operations as first-class citizens

Hi and thanks for writing this cool library !
I've been looking at monad libraries for a while now and they all seamed unnecessarily complex to me. I came to the conclusion that the main (only?) benefit of Either types was to have typed Errors. When I read your blog article, I thought "right on !".
There is one thing that bugs me though and it has to do with handling async operations.
Your (great) library offers a Result type with a bunch of great api methods like map, mapErr, andThen, match, etc. but as soon as an operation is async, there aren't as many options.
I like the chain solution but it doesn't offer the same chainability as we would get with the synchronous Result.
I understand this has to do with the fact that you don't have control over the api of Promise<Result> but I believe there might be an elegant solution: having a wrapper around that Promise. A ResultAsync<T> type that would have more or less the same api as Result<T> and could offer full chainability.
Furthermore, I was thinking that whatever the Result could do, the ResultAsync could do too. Therefor, I'm not even sure we need both types, only the async version. In the Real World, there's always an async operation somewhere in the chain anyway, right ?
The method signatures would look something like this:

map<U>(fn: T => U | T => Promise<U>):  ResultAsync<U, E>
mapErr<U>(fn: E => F | E => Promise<F>):  ResultAsync <T, F>
andThen<U>(fn: T => ResultAsync <U,E>): ResultAsync <U, E>
match<A>(
  okFn: (t:  T) =>  A,
  errFn: (e:  E) =>  A
): Promise<A>

then(cb: (T | E) => any) // a method to make the ResultAsync a thenable which can have the result extracted by calling either .then() or await

And a code example:

const newUserId = await api.checkAvailability()
                           .andThen(api.createNewUser)
                           .map({ id } => id);

// newUserId is of type string | Error
if(newUserId instanceof Error){
   // newUserId is an Error
   console.log(newUserId.message);
}
else{
   // newUserId is a string
   console.log("new user id is ", newUserId)
}

Thanks again and looking forward to your thoughts !

Hey @paduc,
Thanks for the kind words!
I like the idea, and this has been something that has crossed my mind in the past. So it's great to hear that others are thinking about it too.
I recently recalled that BlueBird implements its own Promise API that behaves much like native promises, except it adds a bunch more useful methods.
I'm not familiar with how BlueBird, or other libs, implement a awaitable object. If this is something that interests you, by all means I would support a PR that implements this.

I do, however, I am not totally in agreement with your current API suggestion.
I think when you do:
await api.checkAvailability()
  .andThen(api.createNewUser)
  .map({ id } => id);

The resulting value should be a Result instance.

Great ! I'm glad you're open to the idea !
Regarding what you call making Result awaitable, it's actually very easy. If an object or function defines a then() method, it is called thenable and can be used in conjunction with await or then (see specification). I included it in my original post at the end of the method signatures.
As for the result of such an await, I thought you might prefer it returning a Result instance.
I'm dabbling with the idea that there could be no more Result at all (only ResultAsync). In french, we have a saying "Qui peut le plus, peut le moins" which would translate to "He who can do the most, can do the least". If we have a ResultAsync capable of handling async and sync operations, we wouldn't really need Result which is only capable of handling sync operations.
Do you have a use-case where you would want to wait for a ResultAsync to get a Result ? In my mind, I would only wait for a ResultAsync at the very top of my call stack, where I unpack the value or the error. I would see no need for the capabilities of the Result type.
I remain open-minded to having both Result and ResultAsync types of course, I'm just try to push the boundaries a bit.
I'm getting to work on a PR for the ResultAsync type.

My apologies for missing your original then method!
I wasn't aware how simple it would be to make something awaitable. I haven't put in much thought around how things should work. But in a perfect world, Result would be enhanced to be awaitable without modifying the behaviour of the rest of the Result api (i.e. new features added without a breaking change).
Otherwise, I would like there to be a new ResultAsync function (in its own file - result-async.ts with accompanying tests) that duplicates the Result API. This is, again, very nice because it is not a breaking change for others.

No problem !
Making Result awaitable would indeed introduce a breaking change because await would be necessary to access the underlying value or error.
I'm moving forward with a new ResultAsync (with it's own file and tests) that mostly duplicates the Result API. I think Result.asyncMap would return a ResultAsync instead of the current Promise<Result<U,E>> so we could continue chaining.

I think Result.asyncMap would return a ResultAsync instead of the current Promise<Result<U,E>> so we could continue chaining.

Sounds good. As long as it's completely non-breaking!