Add bundled analytics support in Inspector

[x] I've included a changeset (psst. run yarn changeset. Read about changesets here).
Corresponding Inspector PR - https://github.com/segmentio/segment-inspector/pull/24

Looks good, except for the build error -- I assume this isn't done?

I think the build is causing a typescript error because of this line, which is conflicting with ambient declaration merging:

This is the correct declaration:
declare global {
  interface Window {
    analytics: AnalyticsSnippet
  }

All this is complicated by the fact that there's a circular dependency here with analytics-next and  '@segment/inspector-webext'. Publishing inspector-webext's types as a separate dependency or just removing that dependency altogether from analytics-next might be a good idea (and doing the typing locally).


Track service end typo in Python Tables Storage error message

There are two tests for invalid table names in tables storage failing in Python - async test, sync test
We are testing with a list of invalid storage table names, some are failing("1table", "a//", "my_table"), not all of them.
Here's one of the response we got:
'message': 'The specifed resource name contains invalid characters.\n'             'RequestId:6783e6ea-a002-006a-3c48-c912e9000000\n'             'Time:2022-09-15T21:15:28.7300879Z\n' 'ErrorCode:InvalidResourceName'
You can find there's a typo in response message so that we can't catch it and convert it to a ValueError to users.

The typo exists for a while, it breaks all of our live pipelines. Please give some priorities to fix it.

The typo exists for a while, it breaks all of our live pipelines. Please give some priorities to fix it.

@YalinLi0312 to help me understand better, do you mean this is a service issue? Or a storage issue?
Thanks.

@YalinLi0312 to help me understand better, do you mean this is a service issue? Or a storage issue?
Thanks.

The typo exists in the error message which returned from service end, I need service team to help to resolve it.

Thanks for the clarification.
Given it is not a bug in SDK, we can close the issue or if you want, use the issue to track the service change request (let's update the title to make it clearer that this is used to track the issue).

Hi @klaaslanghout, I find you as the bot tagged your name above. I know you were working on Storage from your GitHub description, do you know anyone in Storage team that can help to resolve this issue?


Time is money (time based coin)

I think since "time is money" there should be a coin that is minted based on the time that has elapsed

Like I noted in the matchthat tickets it might be good to have a client that keeps track of time of something being worked on that equates to coins.


add function to convert to image folder structure

What does this PR do?
Fixes #56
Todo:

[x] Create a copy of the Oxford dataset in the ImageFolder format
[x] Create a child class of the ImageFolder dataset and add support to export a CSV
[x] Add an example config for the new ImageFolder dataset
[ ] Support all places where class_names are referenced
[x] Support dataset splits

New structured used in training run here: https://wandb.ai/jeremytjordan/flowers/runs/1i8zve7g?workspace=user-jeremytjordan

Added class_names to the model's checkpoint so that we can always have the correct mapping back to human-readable class names at inference time.
Next steps:

Retrain top performing models to include new checkpoint
Upload artifacts to Github as a release
Update Streamlit to reference the new model attribute in place of importing a list of oxford102 class names

This will allow us to trivially expand to more classes as we grow our dataset.


Change license

Hello,
I have signed https://cla-assistant.io/uber/jaeger-client-java with my personal details and now I would like to change it and sign with company details.

@pavolloffay unfortunately it is not possible yet to change the signature.
CLA assistant does "basic" verification of signatures based on github user accounts. So we do not support differentiation between private and corporate CLAs. Some repositories/organisations (like uber) use so called "custom fields" as workaround to collect different data, but there is no functional support on our side to handle this data.
Sorry for that :(
I would suggest to contact uber directly and provide your agreement with company details on an informal way.


install evilginx 2.3.2

hi, can seomeone share to me how to install the new release evilginx 2.3.2

why become like this
root@ubuntu-s-1vcpu-1gb-sgp1-01:~/go/src/github.com/kgretzky/evilginx2# make
build flag -mod=vendor only valid when using modules
make: *** [Makefile:8: build] Error 1

hi, can seomeone share to me how to install the new release evilginx 2.3.2

i can help you with your<EMAIL_ADDRESS>

I have the same error to

i can help you with your<EMAIL_ADDRESS>


vscode extension recommendations

found a repo that was boilerplate for new lit projects and it had .vscode which made recommendations on plugins to install. We do this in class manually telling people so just another DX thing to avoid having to communicate this manual process for machine setup.

easy and found some new good ones. win win.


Switch to using fixedRate() instead of fixedDelay() for Reactor

In BaseReactor I noticed that the method used to generate the Stream[IO, Command] of states is Stream.fixedDelay[IO]. Switching to using Stream.fixedRate[IO] might result in smoother motion with less microstutters. At least I expected the frames to be generated at a fixed rate and not with a fixed delay.

Thanks!


clap v4 support

As the fine folks of clap have released clap v4, friends and me tried to go ahead and support clapv4 in klask.
Here's the fork: https://github.com/rust-nbg/klask
Doesn't work yet, but we'll be working on it on-and-off. Also I don't know if it would be possible/desirable to support both v3 and v4 somehow.

For now no progress. First thing will be to remove usage of deprecated clap 3.x.y features, then i will try to port.

Thanks for starting to work on this! I'm going to try and help with moving from v3 to v4 in the near future! I would like to fix a few remaining small issues and publish everything as a new version. This would be the last one that supports clap v3. I don't think supporting both version at once is realistic... :P

Perhaps we should release klask v2 (or v4?!) with clap v4 support and keep v1 alive for a while...

I am quite interested in the clap v4 version. I have a bioinformatics cli app coming and this would give access to many researchers who are unable to use a shell.

[Note] some progress on removing usages of deprecated clap v3 features: https://github.com/MichalGniadek/klask/pull/58

Maybe we could do something similar to cansi and split this into modules klask::clap3, klask::clap4 etc., not sure how problematic this would be. Probably also add a cargo feature for each module.

As the fine folks of clap have released clap v4, friends and me tried to go ahead and support clapv4 in klask.
Here's the fork: https://github.com/rust-nbg/klask
Doesn't work yet, but we'll be working on it on-and-off. Also I don't know if it would be possible/desirable to support both v3 and v4 somehow.

I forked your fork and switched to clap v4. All the tests and examples are passing and it seems to work well with my cli app which has several subcommands. First I wasn't planning on contributing, so I just went ahead and removed some fields from the ArgKind::MultipleStrings implementation I thought were unnecessary for clap v4, however I am not actually 100% sure if this is the case.
But I hope some the code can be of use down the line: https://github.com/xosxos/klask

@xosxos just loaded up your fork and it seems to be working well enough! Hard to say for me as my app doesn't seem to really work with klask in general unfortunately. I'm going to submit an issue and see if there's a way to make it work.

@xosxos Just noting: when using this clap 4 fork (awesome btw!) it used to work, but now I get this error message:
error: The platform you're compiling for is not supported by winit
  --> /home/rafael/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.28.7/src/platform_impl/mod.rs:67:1
   |
67 | compile_error!("The platform you're compiling for is not supported by winit");
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

@barafael Sorry! I somehow did not realize it was being used. I accidentally broke it last week when updating to egui 0.23.0 and I forgot to fix it straight away! But as far as I know, it should work now.

@xosxos love your work in getting the dependencies updated.
I'm trying to track down a particular oddity on Mac which shows up on egui 0.23 (and 0.24) but not on the original 0.18, and I'm hoping you or someone else in the thread might have have an idea.
When I run the app (--example showcase will do) it initially launches with NSMenuItem in the menu bar.

But upon taking it full screen, the name showcase appears where expected (and remains there afterwards)

@xosxos Any chance of upstreaming clap v4 support anytime soon?


chore: Update ServiceWire package to version 5.6.0

Updated the ServiceWire package version in Directory.Packages.props from 5.5.4 to 5.6.0.

PR Details
The new package resolves System.Text.Json vulnerabilities, and some bug fixes, summary:
https://github.com/tylerjensen/ServiceWire#namedpipeserverstreamfactory-and-other-improvements-560
NamedPipeServerStreamFactory and Other Improvements 5.6.0

Contributed fix where accepting TCP clients synchronously may block new clients from being accepted until the terminating request is received on the synchronous client.
Contributed NamedPipeServerStreamFactory to allow greater level of permissions control in using named pipes.
Introducted injectable ILog and IStats across channels and clients with default NullLogger and NullStats, making InjectLoggerStats obsolete.
Code improvements for code consistency and eliminating outdated frameworks from tests and supporting projects.
Updated several dependencies in supporting projects.
Updated System.Text.Json to 9.0.0 to resolve known vulnerabilities in previous versions.

Related Issue
n/a
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have built and run the editor to try this change out.

Thanks !

@Eideren, could you revert this PR? It would need some work from xen2 so we can leave it open or WIP till someone can look into #2566. Thanks.


Change Color Title columns

Hi,
thanks for your great theme;)
I would like to change the color of the title in black at least for the light version.
Is it possible?
I've attached a screen shot to explain better...
Thanks

In the config.yml change the color for text to any color you'd like!


Emacs auto-lang mode to detect Tamil

Auto lang mode needs a list of most common Tamil words
https://github.com/emacsmirror/auto-lang

We have list of most common Tamil words from sorting Wikipedia lists, or even standard TVU dictionary.


USHIFT-852: assets.yaml declaring manifests to copy from releases

Adds asset.yaml that lists all the files in assets/. Focused only on sourcing the assets (either copying from STAGING_DIR or using git restore). It does not include any modifications (like yq) to the manifests (yet).
Its purpose is to have common place for both rebase procedure and rebase presubmit.
Introduces new way of copying manifests from staging dir based on contents of assets.yaml file.
Commands such as cp, git restore, rm -rf are removed from rebase.sh's functions dealing with manifests (assets/).
Other changes include:

wrapping {{ .template  }} inside quotes to get correct yaml file, so many sed commands are removed
formatting lvms/topolvm manifests with oc create --dry-run=client (includes sorting) (a consequence of using oc create --dry-run on all lvms/topolvm manifests, not just the copied ones)
changing method of creating "lvmd" ConfigMap from templating to overriding ConfigMap's data (because templating "{{ .lvmd }}" would keep the " resulting in incorrect data)
removed Test_renderLvmdConfig because it was testing approach based on renderTemplate() which is no longer used

/retest
@pmtk, can you add the fix description to the PR? There seem to be some formatting changes, so it's not clear what needs to be reviewed.

@pmtk, can you add the fix description to the PR? There seem to be some formatting changes, so it's not clear what needs to be reviewed.

I copied (and slightly tweaked) descriptions from individual commits

/cc @copejon
PTAL on the lvms/topolvm related changes

/test e2e-reboot

When I tried to run this version of the rebase script it replaced some of the assets with "Script exited with error.". Maybe you can prepare a separate PR showing the output of this work?

It looks like that's the same problem with oc requiring credentials that we've discussed elsewhere.

When I tried to run this version of the rebase script it replaced some of the assets with "Script exited with error.". Maybe you can prepare a separate PR showing the output of this work?

Sure.
I ran contents of  https://github.com/openshift/microshift/pull/1362 last_rebase.sh on this branch.
Here's the branch it produced: https://github.com/pmtk/microshift/tree/rebase-4.13.0-0.nightly_amd64-2023-02-13-194759_arm64-2023-02-14-014641
I created PR against rebase branch: https://github.com/openshift/microshift/pull/1365 although the diff is weird because it shows the same changes like it would be against main branch

When I tried to run this version of the rebase script it replaced some of the assets with "Script exited with error.". Maybe you can prepare a separate PR showing the output of this work?

Sure. I ran contents of #1362 last_rebase.sh on this branch. ~Here's the branch it produced: https://github.com/pmtk/microshift/tree/rebase-4.13.0-0.nightly_amd64-2023-02-13-194759_arm64-2023-02-14-014641 I created PR against rebase branch: #1365 although the diff is weird because it shows the same changes like it would be against main branch~
Here's PR between today's rebase PR and same refs produced by this branch #1367 (although the diff looks like it's against main, maybe if I'd rebase my branch on top of CI PR branch the diff would be "correct" - but you can see same changes with some reformatting).
Edit2: Here's PR between 1362 and 1367 rebased on top of 1362: #1368

OK, that all looks good. This needs a rebase, but then I'll LGTM it.

/retest


PLT-749: Added rountrip ToJSON/FromJSON and ToField/FromField property tests

Added rountrip ToJSON/FromJSON and ToField/FromField property testsy tests for all orphan instances in marconi-chain-index, and for StorableResult datatype in the UTXO indexer

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated
[ ] Important changes are reflected in changelog.d of the affected packages

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

@kayvank Normally, this PR would solve the datum issue in the PR #947


Use Pixi.js v3

I know, I know. Pixi v3 has a while to go before it is bug-free enough for you to want to merge something like this, but I have been using this for a new project and just wanted to share how it could be done.
Basically this switches from using a copied in-place pixi.js to using pixi.js from npm. The build methods stay the same, it just uses the prebuilt pixi.js that is packaged in npm.
Feedback welcome.

Aside from quite a few local changes we've made to Pixi 2 I'm curious how many things would this add back in that we don't ever use? (resource loader? event handling? filters? etc?) - I know if Phaser was properly modular it could chose not to use them, but that isn't happening in v2s lifetime.

@photonstorm Totally, and it probably adds a bunch. Alternatively a build script can run gulp build -e loader -e interaction -e filters to build the pixi file that is then imported.
That keeps you able to easily use the pixi package, but minimize your footprint? Just a possible solution.

Oh thanks for this. I'm currently prototyping for an upcoming project where I need Phaser instead of a Pixi only environment. I always wondered how many places I would have to change in Phaser to use Pixi.js V3. To my surprise this looks more or less like the default Pixi v2 to v3 converting process. I'm looking forward to the moment where pixi is bug free enough to get merged.

Closing off old PRs that are now fixed, no longer required, locally merged or were just for testing.


Major changes and some translation corrections

This texts seemed to be directly translated from English. There were some errors, as well as, bad concepts and malformed sentences. I fixed all of them.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Highlight Errors in CSV on Purchase Order and Inventory Review Page

What is the motivation for adding/enhancing this feature?
This enhancement will provide users with immediate visual cues to address issues within the CSV data.
What are the acceptance criteria?

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information

Suggested UI: https://www.figma.com/file/wK5FMs4z2zXQR9VfHV9ycE/Dhiraj's-Ionic-6-Material-UI-Kit-(Community)?type=design&node-id=28890%3A153154&mode=design&t=kUWR7ayCsXLkTDnw-1

Can you assign me this issue

can you assign me this issue ?

Sure @2004rohanbhati467 go ahead

can you assign me this issue ?

@Preetvani07 please pick another issue


chore: remove unuseful patch message from instrumentations

#2107 removed the diag prints on InstrumentationNodeModuleDefinition and InstrumentationNodeModuleFile, patch and unpatch callbacks, as this message is now hoisted to be emitted consistently from the base class. The scope of this PR was narrow to handle specific types of messages that became redundant, but it was noted that there are other diag prints for patching and unpatching specific function from moduleExports.
This PR is a followup to #2107

Cleanup diag patch messages for trivial function which created noise and added no real value for troubleshooting to users.
Document guidelines on using diag channel to emit patch messages, based on the examples I observed in the repo.

@JamieDanielson @trentm @pichlermarc  thanks again for the great feedback and discussion on the previous PR.
Since those changes were intentionally left out of the previous one, I opened a follow-up to apply the issues we talked about in a dedicated PR. I also added a section to the GUIDELINES.md file for future reference to capture the decisions.
Please let me know if this work correctly reflects the discussion in #2107 🙏

Thank you @JamieDanielson
Your suggestions are always valuable. addressed all the comments 🙏


Update azure-storage-blob-cryptography to 12.0.0 for GA

Update the version of azure-storage-blob-cryptography to 12.0.0 for GA

/azp run java - storage - ci


Rendering webcomponents on the server side

Is there a way to render web-components on the server side?

Yes, you can use the snabbdom-to-html module.
Pass a vnode to the toHTML that the module exports and you get an HTML string you can use to generate a server response. (You need some extra markup, <html><body>... etc. to insert your components into, you have to do that yourself).

@acstll That won't work for webcomponents. For instance if I have a custom input box such as —
h('x-custom-input-box')

The inner HTML would remain unknown to both the browser as well as the server. In the browser we can you document.registerElement('x-custom-input-box', {prototype}) but I feel it would have been great if something like this was available to snabbdom as well.
I think this can be done via some external plugin and needn't be a part of snabbdom.

@tusharmath you're right, I missed that one, sorry!


Remove tests from Veracode Scanner

Only production code need to be scanned.
Added only needed files into zip.
Add copyright. Small script refactoring.
Relates-to: OLPEDGE-1555
Signed-off-by: Yaroslav Stefinko<EMAIL_ADDRESS>

Codecov Report

Merging #212 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #212   +/-   ##
=====================================
  Coverage      90%    90%
=====================================
  Files          57     57
  Lines        1540   1540
  Branches      181    181
=====================================
  Hits         1387   1387
  Misses         93     93
  Partials       60     60

Impacted Files
Coverage Δ

...olp-sdk-dataservice-read/lib/client/QueryClient.ts
91.8% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


fixed validation publish condition

FEDX-1599

Pre dart 3, running dart pub publish --dry-run would transitively execute dart analyze --fatal-infos under the assumption that all info lints should be resolved before publishing a dart package
There is an escape hatch --force where this check was ignored.
This was decided overkill, and just dart analyze is now run in dart. This change was applied in dart 3
Since we commonly index/publish dart 2 packages in gha-dart-oss, we have to support both approaches, and --fatal-infos is a stopgap for the action.
As a solution, we've decided to write custom validation steps for dart 2 for the criteria that we want to enforce (specifically the changelog update), and for dart 3, continue to use dart pub publish --dry-run
On the publish side of things, we need to --force publish for dart 2, but dart 3 should be able to publish without issue

Security Insights
No security relevant content was detected by automated scans.
Action Items

Review PR for security impact; comment "security review required" if needed or unsure
Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

QA +1

See test run on this release PR: https://github.com/Workiva/dependency_validator/pull/129


Adding Python Crop Simulation

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
Crop is a fundamental problem for humans on this planet.
I've not used the full power of PCSE --- more to come.
How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

I guess the License is too restrictive here :s it's a bit borderline here since it's a benchmark and not an optimizer, but it safer to avoid adding any...

I guess the License is too restrictive here :s it's a bit borderline here since it's a benchmark and not an optimizer, but it safer to avoid adding any...

Would a wget in the code be ok ?

I guess the License is too restrictive here :s it's a bit borderline here since it's a benchmark and not an optimizer, but it safer to avoid adding any...

Would a wget in the code be ok ?

I don't think so, but I may be incorrect

Let me check this.


<EMAIL_ADDRESS>update error

Upgrading anaconda3 from 2019.03 to 2019.07 removes 2019.03 without prompting and does not preserve existing environments.
Checking hash of Anaconda3-2019.07-Windows-x86_64.exe ... ok.
Uninstalling 'anaconda3' (2019.03)
Running uninstaller... done.
Removing shim for 'python'.
Removing shim for 'pythonw'.
Removing shim for 'python3'.
Unlinking ~\scoop\apps\anaconda3\current
Installing 'anaconda3' (2019.07) [64bit]
Loading Anaconda3-2019.07-Windows-x86_64.exe from cache.
Running pre-install script...
Installing Anaconda 3. This can take up to 30 minutes on an HDD.
Running installer... done.
Linking ~\scoop\apps\anaconda3\current => ~\scoop\apps\anaconda3\2019.07
The system cannot find the path specified.
Creating shim for 'python'.
Creating shim for 'pythonw'.
Creating shim for 'python3'.
Persisting envs
'anaconda3' (2019.07) was installed successfully!

It persists the envs directory. Are there other directories that need to be persisted?

I have installed anaconda since before the persist feature exists. What I
did when upgrading previously was to copy/move env directory from old
version to new version manually. However when upgrading to 2019.07, scoop
just deleted the old version without prompting, hence my env directory is
not preserved.
On Tue, 6 Aug 2019, 4:18 am Ross Smith II<EMAIL_ADDRESS>wrote:

It persists the envs
https://github.com/lukesampson/scoop-extras/blob/16994326a31d071262c28c61ad518db622f3b330/bucket/anaconda3.json#L32
directory. Are there other directories that need to be persisted?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lukesampson/scoop-extras/issues/2572?email_source=notifications&email_token=ADA625AJB7JYHK6CB3MSKKLQDCDKRA5CNFSM4IJHF2AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3S6NXY#issuecomment-518383327,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADA625BOYKBF5X4FWJFDDVDQDCDKRANCNFSM4IJHF2AA
.

Yuk. Sorry you lost data. If you have a backup, you can copy it into the new envs directory, and you should be OK on future upgrades.

It's ok. By the way, is it the intended behavior to delete older version
when updating to newer version? This doesn't seem to happen when I update
other applications.
On Tue, Aug 6, 2019 at 9:41 AM Ross Smith II<EMAIL_ADDRESS>wrote:

Yuk. Sorry you lost data. If you have a backup, you can copy it into the
new envs directory, and you should be OK on future upgrades.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lukesampson/scoop-extras/issues/2572?email_source=notifications&email_token=ADA625GMEGGJJGZE4YWZPXTQDDJENA5CNFSM4IJHF2AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3TRVGI#issuecomment-518462105,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADA625E7SSEGYDQO2EKFX23QDDJENANCNFSM4IJHF2AA
.

No, it's not usual way to delete previous version. This only be done after you run scoop cleanup. Updating only redirects current link.

@edisoncks Can you check if the directory C:\users\you\scoop\apps\anaconda3\2019.03\envs exists?

@rasa Nope, the entire 2019.03 directory is gone.

That’s because the uninstall function using their installer here.
What’s weird is the 2019.07 update didn’t change anything that should cause your issue.
The persist and uninstaller were added here in 2018.12.
@edisoncks Can you check if the directory C:\users\you\scoop\persist\anaconda3\envs exists?

@rasa The ~\scoop\persist\anaconda3\env directory exists. The envs that I have recreated are located in this directory.


(maint) Updated mac spec test to use a supported mac platform

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4215/

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4216/

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4218/

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4220/

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/4232/


Uncaught TypeError: hook.apply is not a function on using onEnter in react-router

Below is the code snippet:
function requireAuth(nextState, transition, callback) {
  console.log('hey');

  callback();
}

export const AdminList = {
  'path': 'admin-area/admin-list',
  getComponent(location, cb) {
    require.ensure([], (require) => {
      cb(null, require('components/admin/admin-list/admin-list').default);
    }, 'admin-list');
  },
  'onEnter': {requireAuth}
};

On entering the path /admin-area/admin-list, I get the error mentioned in the title from the file TransitionUtils.js.
I am not sure what is causing this error. Can anyone please help?
StackOverflow: hook.apply is not a function

onEnter needs to be a function

So that's it, issue closed? How about adding a PropTypes or other validation to provide a concise error message?

That's not even going through the React route definition above.

That's not even going through the React route definition above.

Sorry, didn't get what you mean.

It's not a React component, so there's no PropType validations to be done.


Simplify ToggleButton coloring for easier class overrides

[x] I have followed (at least) the PR section of the contributing guide.

Hi. I stumbled across this issue when I attempted to override the background color of the selected ToggleButton classes property. The color came back faded. I spent a lot of time trying different things to override the styling, until I decided to look through the source.
The change submitted appears to style the default components as before, while providing a simpler means for overriding those default styles, and with fewer lines of code.

@DonBrody I agree with the direction this change is taking. I would also change something else. I would increase the specificity for the select CSS.

I'd be happy to make some more changes. How would you recommend increasing specificity without making the overrides more complicated for the end user?

Ok I see what you mean. I'm happy to make that change if you would like, but I believe that if you increase the specificity in that way that the user will have to declare their color values as !important in order for them to take precedence over the styles with higher specificity. Please correct me if I'm wrong about this.

@DonBrody No, no need for !important: https://material-ui.com/customization/overrides/#overriding-with-classes.

Thank you for sharing. I've pushed that change.

@DonBrody It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!


go/types, types2: report better errors when type sets are empty

Error messages are confusing when type sets are empty. For instance, for:
func _[T interface{comparable; ~int; ~string}](x T) {
        _ = x == x // ERROR cannot compare x == x
}

the error doesn't say why the comparison is not possible. The root cause is that T's type set is empty.
The same applies to other operations, such as + in this case.
cc: @findleyr

Addressed for == in 1.19. Moving to 1.20 for the rest.


Support CircleMarker shapes

Added support for creating CircleMarker shapes.

Found a bug when removing circle markers with right click. When I toggled edit mode off it triggered a Null Pointer Exception. Looking into it now, added WIP to the title.

HI @cosme-benito , thank you for your work on this 👍
Do you mind rebasing this on the latest develop? I've added a testing framework and maybe you could even add some tests to this PR.
I hope to get some time to manually check your addition in the next days

Merged with the develop branch again, added support for the toolbar (including SVG) and tried to extend the Marker classes when possible.
I'm working on the cypress tests but I'm having a hard time getting it to work. It seems cypress is trying to click on the map to add the circle markers before it's ready to accept clicks. Hopefully I can add them soon.

Basics tests added.

Merged with the develop branch again.

Just came across this when I found that my circle markers wouldn’t move.  Seems this might fix it?  If so I might have to replicate for a few other types of markers I’ve created (triangles and stars at the moment).  I might be better just merging in to a simple extended marker with options.
Is this PR in a state where it should be tested/merged?

Leaflet PM doesn't support CircleMarkers, that's why it doesn't work. This
code is ready for testing and it should fix your problem for CircleMarkers
in particular. Adapting this PR to other kind of markers shouldn't too
hard, extending the Circle markers classes will do most of the work.
On Fri, Apr 26, 2019, 18:35 James Patterson<EMAIL_ADDRESS>wrote:

Just came across this when I found that my circle markers wouldn’t move.
Seems this might fix it? If so I might have to replicate for a few other
types of markers I’ve created (triangles and stars at the moment). I might
be better just merging in to a simple extended marker with options.
Is this PR in a state where it should be tested/merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeofsumit/leaflet.pm/pull/282#issuecomment-487139313,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACKARDOW6BU26RI2GEKGHLDPSM4O7ANCNFSM4ETH2ULA
.

Reviewing this PR and adding Circle Marker Support to leaflet.pm is next on my list

Thanks for the feedback, I've implemented all your suggestions except snapping in edit mode.
I'm having a hard time having it to work, the marker is getting "stuck" after hitting the first snap point.

@cosme-benito just tried it. It's crazy difficult. I'm building it right now and will PR it to your branch.

I have created a Pull Request to your branch:
https://github.com/cosme-benito/leaflet.pm/pull/1

Thanks a lot for your help, I don't think I could have figured the problem on my own! From my testing everything appears to be working as intended.
I didn't notice the new support for translation files, that's nice. I'll create a pull request for the portuguese language soon. I added the translations for the circle marker texts to every language except russian and romanian. The text will be in english for those languages until someone more apt than me can translate it.
Also added some tests to ensure the labels are properly translated.

I added the NL language file that was merged into the dev branch.
For Dutch, Romanian and Russian languages I just copied the labels for the "Markers" which I feel is better than just writing it in English. Hopefully someone will translate it in a better way.

Merged with dev again.
@codeofsumit Am I missing any request change?

@cosme-benito No you don't, sorry for not replying sooner. I got married last weekend and I'm now on honeymoon. I'll check this again when I'm back!

Congratulations mate!!!
On Sat, Jun 8, 2019 at 5:56 PM Sumit Kumar<EMAIL_ADDRESS>wrote:

@cosme-benito https://github.com/cosme-benito No you don't, sorry for
not replying sooner. I got married last weekend and I'm now on honeymoon.
I'll check this again when I'm back!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeofsumit/leaflet.pm/pull/282?email_source=notifications&email_token=ACKARDKHBOJYZDA6LTLCNSLPZPQETA5CNFSM4ETH2ULKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXHYTUA#issuecomment-500140496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACKARDJRAHXZCQ5YPWNURI3PZPQETANCNFSM4ETH2ULA
.

Would be great to see some love on this PR and finally get it into production.

I agree. Everything seems to be working @cosme-benito - I will merge it now :-)
Thanks so much for your patience!!! 🙏


Doubles the e-shotgun charges

About the PR

HOS energy shotgun can now shot 16 times.
Why / Balance

After removing auto charge e-shotgun became useless, even the enforcers and kamerers are better, because you can reload them in combat and use different type of ammo, like rubber or uranium
Technical details

Just changed his max charge values in prototypes
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

tweak: HOS energy shotgun max charges doubled

personally, I think doubling it is too much.

personally, I think doubling it is too much.

It can be EMPed and become useless, nukies, agents and even chemists can make EMP, but if it will be too OP, we can decrease charges or shot speed.

i mean.. we could just make it emp proof

EMP proof weapons would be an interesting concept, especially with eshotgun having stun mode

EMP proof weapons would be an interesting concept, especially with eshotgun having stun mode

EMP proof stunmeta, lets go. :trollface:

Bring back self recharge. HoS should have a reliable weapon, it only makes sense.

Bring back self recharge. HoS should have a reliable weapon, it only makes sense.

I think, maintainers will not accept self recharge PR.

While I agree that the shotgun needs a buff, I don't think simply doubling the charges is the way to go about it.
Obviously I would prefer it if #32104 got merged (I made that) but if it does not, this might help but we'd have to see.

Closed because i think #32104 is a better solution


T2w-ACPC image not being output

I have a BIDS-valid dataset with T1 and T2 images. The qsiprep boilerplate suggests that

The additional T2w reference image was registered to the T1w-ACPC reference image using an affine transformation in antsRegistration

However, I do not see any T2 images in my output directory:
/nese/mit/group/gablab/data/BRAID/bids/derivatives/qsiprep_0.20.0/sub-BRAID003P/anat/
├── sub-BRAID003P_acq-vNavNorm_from-orig_to-T1w_mode-image_xfm.txt
├── sub-BRAID003P_desc-aseg_dseg.nii.gz
├── sub-BRAID003P_desc-brain_mask.nii.gz
├── sub-BRAID003P_desc-preproc_T1w.nii.gz
├── sub-BRAID003P_dseg.nii.gz
├── sub-BRAID003P_from-MNI152NLin2009cAsym_to-T1w_mode-image_xfm.h5
├── sub-BRAID003P_from-T1wACPC_to-T1wNative_mode-image_xfm.mat
├── sub-BRAID003P_from-T1wNative_to-T1wACPC_mode-image_xfm.mat
└── sub-BRAID003P_from-T1w_to-MNI152NLin2009cAsym_mode-image_xfm.h5

QSIPrep completed without errors:
singularity run -e -B ${scratch},${cache_dir} ${qsiprep_img} $scratch/data $scratch/data/derivatives participant --participant_label ${subject:4} -w $scratch --fs-license-file ${scratch}/data/code/license.txt --denoise-method patch2self --pepolar-method TOPUP+DRBUDDI --unringing_method mrdegibbs --recon-spec $scratch/data/code/recon_spec.json --output-resolution 1.25 --freesurfer-input ${scratch}/data/derivatives/freesurfer/ --fs-license-file $scratch/license.txt --skip-odf-reports --notrack --mem-mb $mem_mb --nthreads 16 --omp-nthreads 8
I've noticed this with 0.19.0 and 0.20.0
Thanks for your attention,
Steven

Ah! I forgot to add a datasink to the T2w. Thanks for opening a bug report.
Also, have you taken a look at the html report using the TOPUP+DRBUDDI? If you have "rpe_series" data then you'll see a new FA-alignment figure in there.

I don't see the new FA-alignment figure in the HTML unfortunately - happy to share any data that may help

And you're sure it's multiple dwi series in multiple phase encoding directions? The section header (I realize now it's not very descriptive) should be
Susceptibility distortion correction
Results of performing susceptibility distortion correction (SDC) on Fractional Anisotropy (FA) images.

It's one DWI series and a RPE fmap.

ahh ok then the FA SDC part won't run and there won't be a report. The T2w should still get sent to the derivatives directory though. do you see much movement in the DRBUDDI html report?

TOPUP (lots of correction)

DRBUDDI (not as much, but still noticeable)
Before

After

Very interesting. I haven't tried TOPUP+DRBUDDI with just an epi fmap before. It looks reasonable offhand, but it will be interesting to see how downstream things are affected.


:sparkles: feedback command

Closes #87

Has to be rebased, I believe configuration update is already merged into develop with #86

Don't forget to add commands to suggestions via @BotFather on release


exec.start is not return out

I have container created from dockerfile
FROM ubuntu:latest

RUN apt-get update;

RUN apt-get install -y curl \
        && apt-get install -y git

RUN curl -sL https://deb.nodesource.com/setup_9.x | bash -
RUN apt-get install -y nodejs
RUN npm install -g n

When i want to run exec command with simple commands like ls, echo it works and response command output, but when I try to call for example n 8 it does not return a response stdout
Example code (Typescript), runs with (ts-node):
import * as Docker from 'dockerode';

(async() => {
  const docker = new Docker();

  const container = await docker.getContainer('569632765d9e8386cc7f67a6741a64958421a0044df868541b95b42f3a84e036');
  const exec = await container.exec({
    Cmd: [ 'n 8'],
    AttachStdin: true,
    AttachStdout: true,
    Detach: false,
    Tty: true
  });

  exec.start({stdin: true}, (err, stream) => {
    console.log('err', err); //err null
    stream.on('end', (e) => {
      console.log('on end', e) // called once with empty arg
    })
    stream.on('data', (d) => {
      console.log('on data', d); // not called
    });
  });

})().catch(e => console.error('error', e));

Is it n specific case or dockerode bug ?

Sorry, StdinOnce solved my issue


jpeg2000 encoder

@olivervogel I'm not able to replicate this locally, I updated docker-compose to match the CI:
command: bash -c "composer update --prefer-stable --prefer-dist --no-cache && ./vendor/bin/phpunit"

but tests are passing like this here

@deluxetom I will take a look.

@deluxetom For some reason the jp2 media type was not detected correctly in the testing environment. Although it worked for me locally too. I now check the two magic byte signatures of the jp2 format manually.


Add option to remove bold escape codes

I've been playing around with using ghcid with plan9's acme editor. It displays some ugly cruft when the build displays bold output.
This adds an option to turn off trying to display bold output.

Thanks! I'll review this, probably over the weekend.

Thanks, seems like a valuable thing to have, but rather than restrict it to bold, I suggest we follow the same approach as HLint at https://github.com/ndmitchell/hlint/blob/master/src/CmdLine.hs#L76. The essentially names the flag color, and tries to default to something sensible if it can.

I've pushed a patch that gives --color=never to disable the Bold. In particular I'll be allowing more color in the near future, so will make sure --color=never keeps suppressing it. Furthermore, if your terminal correctly advertises it doesn't support ANSI then hopefully just doing nothing will still work.

Thanks for fixing this, Neil! Sorry I didn't get a chance to incorporate your suggestions.
I'll be on a setup where I can give acme a go again soon, so I'll be pleased to have this


IndexOutOfRangeException: Invalid kernelIndex (0) passed, must be non-negative less than 20.

@aras-p
Hello ,when I import 3DGS pointcloud to unity ,error:"IndexOutOfRangeException: Invalid kernelIndex (0) passed, must be non-negative less than 20. UnityEngine.ComputeShader.GetKernelThreadGroupSizes (System.Int32 kernelIndex, System.UInt32& x, System.UInt32& y, System.UInt32& z) (at <5cd59f99245548bdbfa6531b54530320>:0)
So,how do i solve it? thanks

@yuancaimaiyi I'd need more details to be able to tell what's wrong.

Which Unity version?
What OS? (Windows, Mac, Linux)
Which project are you opening?
What is your GPU?
What is the full callstack of that error (it is shown in lower part of Unity Console window when you select that error)

@aras-p
Sorry, I forgot to mention my detailed environment.
platform: linux /ubuntu 18.04
unity version : Unityhub has tried various unity versions, 2022, 2023, etc.
project : main branch
GPU : GTX 1660 Ti
detil error: IndexOutOfRangeException: Invalid kernelIndex (0) passed, must be non-negative less than 20. UnityEngine.ComputeShader.GetKernelThreadGroupSizes (System.Int32 kernelIndex, System.UInt32& x, System.UInt32& y, System.UInt32& z) (at <5cd59f99245548bdbfa6531b54530320>:0) GaussianSplatting.Runtime.GaussianSplatRenderer.InitSortBuffers (System.Int32 count) (at UnityGaussianSplatting/package/Runtime/GaussianSplatRenderer.cs:423) GaussianSplatting.Runtime.GaussianSplatRenderer.CreateResourcesForAsset () (at UnityGaussianSplatting/package/Runtime/GaussianSplatRenderer.cs:408) GaussianSplatting.Runtime.GaussianSplatRenderer.Update () (at UnityGaussianSplatting/package/Runtime/GaussianSplatRenderer.cs:614)

Hi aras-p: Big thanks on your work on this project. I'm experiencing the same problem in this comment.
Platform: Win11, Ubuntu 22.10
project: 0.8.0
GPU: rtx 4070
I'm trying to use this to develop for an Meta Quest 3. Problem occurs when development set for Quest. No issues when running without Quest settings.
Thanks again. I've really enjoyed following this project.

@michnaugh1 that sounds like the editor is in active platform (Android?) that has OpenGL ES (and not Vulkan) setup as the graphics API, i.e. issue #26 -- currently OpenGL or OpenGL ES is not supported.


How to pass list of lat long.

How to pass a list of lat long to find a list of address.

We try to map the API as directly as possible, neither Android Geocoder or iOS CLGeocoder supports this. The solution would be to just call it multiple times, if you need it more often you might even try to create a helper function.
Also good to know is that a single lat/lng location might return multiple addresses.


GH-44982: [C++] Add support for building system OpenTelemetry

Rationale for this change
We don't need to disallow -Dopentelemetry-cpp_SOURCE=SYSTEM.
What changes are included in this PR?
Remove forcing opentelemetry-cpp_SOURCE=AUTO.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

GitHub Issue: #44982

@github-actions crossbow submit -g cpp -g linux -g wheel

@github-actions crossbow submit -g cpp -g linux -g wheel -g r

@github-actions crossbow submit test-fedora-39-cpp

I'll merge this.


Add network name to the network specs widget

Add the network name to the widget that displays networks everywhere for better readability.

ping @algirdasp


private-internet-access fails

➜  ~  brew doctor
Your system is ready to brew.

➜  ~  ruby -v
ruby 2.0.0p353 (2013-11-22 revision 43784) [x86_64-darwin13.0.0]

➜  ~  brew cask install private-internet-access
Error: wrong number of arguments (1 for 0) while loading '/usr/local/Library/Taps/caskroom/homebrew-cask/Casks/private-internet-access.rb'
Please report this bug:
    https://github.com/caskroom/homebrew-cask/issues
<main>:5:in `name'
<main>:5:in `<class:PrivateInternetAccess>'
<main>:in `<main>'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/source/path_base.rb:55:in `eval'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/source/path_base.rb:55:in `rescue in load'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/source/path_base.rb:52:in `load'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/source/tapped.rb:28:in `load'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask.rb:124:in `load'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli/install.rb:19:in `block in install_casks'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli/install.rb:17:in `each'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli/install.rb:17:in `install_casks'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli/install.rb:6:in `run'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli.rb:81:in `run_command'
/usr/local/Cellar/brew-cask/0.45.0/rubylib/cask/cli.rb:121:in `process'
/usr/local/bin/brew-cask.rb:42:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Library/brew.rb:58:in `require?'
/usr/local/Library/brew.rb:142:in `<main>'

You’re using an old version of homebrew-cask. Please brew update && brew upgrade brew-cask && brew cleanup && brew cask cleanup and try again.
Please always do this before opening new issues.

Haha oh man. I was running into this issue my desktop computer I never use. I did this:
brew update && brew cleanup && brew cask cleanup

Like 10 times before making the issue. I'm sure you see immediately why I was still having trouble.
Thanks :+1:

Hm. I'm struggling to install PIA as well, have run all the commands posted above. Currently getting this:
$ brew cask install private-internet-access
==> Downloading https://www.privateinternetaccess.com/installer/installer_osx.dmg
######################################################################## 100.0%
==> No checksum defined for Cask private-internet-access, skipping verification
==> Running installer script Private Internet Access Installer.app/Contents/MacOS/runner.sh
Password:
==>
==> No matching processes were found
==> LSOpenURLsWithRole() failed with error -10810 for the file /Applications/Private Internet Access.app.
🍺  private-internet-access staged at '/opt/homebrew-cask/Caskroom/private-internet-access/latest' (74 files, 21M)

Any ideas?


Weiners Attack

TODO- Documentation.
Make the code more elegant.
Will be done by tomorrow.

Make sure that your local code is up to date with the main repository before issuing PRs.
As you can see here, a lot of conflicts exist because you haven't done this.
From the root of the project:

git checkout master; git fetch origin
git merge origin/master

I'm assuming that you've cloned this repository directly.

git checkout -b weiner (or some other appropriate branch name)
Make changes, ensure everything's working
git push origin weiner

Then raise a PR against master.

Will follow it from next time!
On Jan 24, 2016 6:17 PM, "Kishor Bhat"<EMAIL_ADDRESS>wrote:

Make sure that your local code is up to date with the main repository
before issuing PRs.
As you can see here, a lot of conflicts exist because you haven't done
this.
From the root of the project:

git checkout master; git fetch origin
git merge origin/master

I'm assuming that you've cloned this repository directly.

git checkout -b weiner (or some other appropriate branch name)
Make changes, ensure everything's working
git push origin weiner

Then raise a PR against master.
—
Reply to this email directly or view it on GitHub
https://github.com/chinmaydd/NITK_IEEE_SaS/pull/8#issuecomment-174293432
.


Scrobbling has stopped after new lastfm update.

Also scrobbler dissapeared from Opera apps store.

completely stopped - no nothing but tampermonkey (tube), no answers, a lot of complaints?

Did you try reauthenticating with last.fm?

Yes i tried. Few times. Still no scrobbling.

@iamrazor This is weird, people have reported issues with lastfm itself, but I never had any. Could you show us the background page logs?

The scrobbler was intentionally taken down from the Opera store. It's because Opera is no longer supported. For more info see #503
Do I get it right that you're still using Opera? The extension is no longer updated and probably not working there.

All is working great (Opera is overrated :) I know you peeps are busy, the big question everyone has is exporting listened to tracks From L-FM, seems people don't like Libre?
I have never have tried it, someone had a site that put all scrobbling history into zip files, opens in excel, but read only, so an easier question is do any other media player (other than Spot) support export/import function without major glitches, no biggie a yes or no in a few week would be cool, I hope is okay to post a question here, its not an issue for me.
Thanks David & others, you are a great service

@pink999 Import/export will probably not be implemented, as it goes too far from this extension's core features and main purpose.
If you're talking about Libre.fm, then we already have PR for that - #756

I think I'll give a try or likely, its getting funky, mean, I honestly
think a merge with bandcamp would be perfect, that is just me,
But; either way Thank you so much David!
Hey if you like rock and fun which you do - this rocks,
so funny and big names

https://www.youtube.com/watch?v=vc5PMA6poio

On Mon, Oct 12, 2015 at 11:11 PM, David Šabata<EMAIL_ADDRESS>wrote:

@pink999 https://github.com/pink999 Import/export will probably not be
implemented, as it goes too far from this extension's core features and
main purpose.
If you're talking about Libre.fm, then we already have PR for that - #756
https://github.com/david-sabata/web-scrobbler/pull/756
—
Reply to this email directly or view it on GitHub
https://github.com/david-sabata/web-scrobbler/issues/739#issuecomment-147615583
.


Are US extension-followed supported? Seems to be common thing to have.

Hey,
was browsing through a bunch of phone number validation libraries but stopped at yours because it looks  the most actively developed, mature and easy to use. Well done :+1:
But, unfortunately, I haven't found the support for US business numbers with extensions (like "1-258-955-9931 x59734" or "1-258-955-9931 ext. 59734", both are important to support for me). It seems to me (from reading issues of this repo) that for some of European countries you do support extensions.
Can you give me any useful info on enabling the support of such numbers to be plausible? (Now, they are marked as non-plausible).

Hi @vittorius!
It may be a common thing to have, but as the summary states in the first word, it's a library for handling E164 numbers (in Phony's case with leniency wherever feasible). I would suggest you write a small wrapper method for Phony that removes clearly marked extensions before checking for validity (and possibly releasing it as a gem).
Cheers! 🙂
Florian

I see. Yes, I've read the disclaimer... OK, thanks for a kind response anyway )

Sure! It should not be hard to extend, assuming x or ext. are always used on extensions 😊
def plausible_with_extensions? number, hints = {}
  ext_removed = number.sub(/\s?(?:ext|x).+$/, '')
  Phony.plausible?(ext_removed, hints)
end


Move DesignTimeServiceCollectionExtensions to Microsoft.Extensions.DependencyInjection namespace

Is there any reason to not have DesignTimeServiceCollectionExtensions in Microsoft.Extensions.DependencyInjection namespace as it is recomended here to have all ServiceCollection extensions in one namespace ?
We recommended that apps follow this convention. Place extension methods in the Microsoft.Extensions.DependencyInjection namespace to encapsulate groups of service registrations.

It's "hidden behind a namespace" since it's an advanced scenario and won't/shouldn't be used by most apps.

We have other extension methods hidden behind the Infrastructure namespace for the same reason.


GOSSIP-91 Random test failure in SharedDataEventTest

This looks good to me! Tests running clean. Thanks.


Reorder requests

Closes #876
Very basic re-ordering functionality added to the requests under an itinerary.
@tonysurma This was accidently based off of a branch from an earlier PR so I'll need to rebase it from master once all others are merged. Just let me know and I'll get that done when you're ready.

doesn't look it needs a rebase @stevejgordon Let me know if I should merge

now it needs a rebase :)

I've rebased locally but it's not quite working so I'll need to debug it later and validate the behaviour. Something in the merge has broken it!

Please don't accept until I've squashed those bugs.

I think this is okay to go now. Checked it out at the start of my lunch break and can't replicate the issue I saw first thing. I've tweaked the UI though so it's a bit (not much) more pleasing!


dagger develop  traces don't seem correctly grouped

Problem
dagger develop doesn't seem to be using proper grouping in the telemetry spans. Here's the output of an example call:
dagger develop
✔ connect 0.2s
✔ cache request: mkfile /schema.json 0.0s
✔ load cache: mkfile /schema.json 0.0s
✔ cache request: blob://sha256:3f96dc98cb3827315b7c4f8e21be0fec3eabd82f4c09031d83f87650b1b5519c 0.0s
✔ load cache: blob://sha256:3f96dc98cb3827315b7c4f8e21be0fec3eabd82f4c09031d83f87650b1b5519c 0.0s
✔ moduleSource(refString: "."): ModuleSource! 0.0s
✔ ModuleSource.kind: ModuleSourceKind! 0.0s
✔ ModuleSource.resolveContextPathFromCaller: String! 0.0s
✔ ModuleSource.resolveFromCaller: ModuleSource! 0.9s
✔ ModuleSource.asModule(engineVersion: "latest"): Module! 4.7s
✔ Module.sdk: String! 0.0s
✔ ModuleSource.resolveFromCaller: ModuleSource! 0.8s
✔ ModuleSource.sourceSubpath: String! 0.0s
✔ ModuleSource.resolveFromCaller: ModuleSource! 0.8s
✔ Module.generatedContextDiff: Directory! 0.0s
✔ Directory.export(path: "/home/marcos/Projects/dagger"): String! 0.4s

On the contrary, dagger functions is correctly collapsed
dagger functions
✔ connect 0.2s
✔ initialize 12.4s

Solution
Improve dagger develop output so we set proper span hierarchy to avoid overloading the output with unnecessary information.
cc @jedevc

I think init has the same issue? Maybe also query/etc - should probably look through other commands as well to see where we see this issue.

It would be great to solve this one :)

Sure, feel free to give it a show @mcruzdev


Add a title to the calendar

This is just a suggestion, but the calendar can have a title that shows the Year of that calendar
Something like: {Year} Calendar

Seems like it was mentioned in Issue #27 and address in PR #29
Will close this issue


fix(attach): end comments in nested mapping

Fixes #69

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #70   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          26     26
  Lines         571    587   +16
  Branches      119    121    +2
=====================================
+ Hits          571    587   +16

Impacted Files
Coverage Δ

src/attach.ts
100% <100%> (ø)
:arrow_up:

src/utils.ts
100% <100%> (ø)
:arrow_up:

src/transforms/document.ts
100% <100%> (ø)
:arrow_up:

src/transform.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Issues to be fixed

Right-Click Menu Customization
Implement a Discord-specific right-click menu instead of relying on the browser's default context menu.

Voice Level Adjustments
Allow users to permanently adjust the volume levels for both:
Individual users in the call.
The stream audio during screen sharing or streaming sessions.

Exiting Call by Clicking on Members
Prevent accidental call disconnection when clicking on members.
Instead, implement proper functionality for:
Opening the member card.
Direct messaging (DMs).

(If not possible, just add a warning message)

Feedback After Changes
Add a mechanism to provide feedback after making changes. (for example, give a confirmation notification when a file got uploaded or on profile picture change etc.)

Pasting Text with Enter
Fix the issue where pressing Enter pastes the text block as a single line, instead of splitting it across lines.

Shift+Enter Not Working
Resolve the bug where Shift+Enter (to move to a new line without sending) is not functioning properly.

Part 2:
Ability to use voice channel and text channel at the same time (if not possible, make it so that there is a warning when you are leaving video/voice channel)
Upload file function correction. (uploading a file shouldnt require you to click upload twice, also implement "upload successful" notification)
Voice Channel chat does not show which member posted which message
Cant upload images in voice channel
The plus next to text/voice/video channels is named "manage members" but has "create channel" functionality.

Part 3:
Navigation between different servers disconnects you from voice/video channel.
The server you are not currently in shows as thought there are unread messages, even though there is no notification system implemented nor is there any new messages
Provide full screen functionality to screen share.

Implement a notification/unread messages function
Allow for minimization of server navigation tab while in text/voice/video channel
Add personal voice level change function (for example ability to change output device level, what decibel range is picked up etc)
Add personal sound level change function (input device level)

Part 4:
FIX SOCKET (soketin düzelmesi)

Part 5:

Revise members tab in servers so that is shows online and offline members seperately.

Add pinging (@username) function in text channels.

When starting private calls with another user, make it so that previous chats remain visible.

Make the deafult setting of "turn camera on while joining a video call" off. (Turning on the camera should be prompted, not immediate)

Make it so that whenever a private call is ended, the PM is displayed rather than the call saying you are "disconnected".

Add notification/unread messages function to all text channels.

Change "start video call" in PM's to "start call".

Add permanent PM's instead of server PM's.


[BUG] Some functions not working after upgrade rancher to version 2.7.0

Rancher Server Setup

Rancher version: 2.7.0
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.23.14
Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? Imported

User Information

What is the role of the user logged in? Admin

Describe the bug

When install or upgrade version of a service from Cluster tools (Ex: Longhorn). Click on Edit button but it shows nothing without errors. When using develop tools, some errors on page shows that:

@tommy04062019
When you say " it shows nothing without errors" are you talking about a blank, white page when trying to update apps?

@tommy04062019 When you say " it shows nothing without errors" are you talking about a blank, white page when trying to update apps?

Yes, it is. As you can see bellow. I have three clusters:

When I go to Clusters Tools to install or edit any app

It just shows an blank page like this

@tommy04062019 Thanks we have more reports of this and I will raise this internally, referring to this issue.

@tommy04062019 What cloud provider is this running on? We have reproduced this on AKS so far..


Allow game.askForNumber prompt to use screen's width

Allow game.askForNumber prompt to span the width of the screen, instead of restricting it to the number pad's width.
Before:

After:

Commits are unverified; will resubmit shortly.


Allowing edge from BidAdjustmentGroup to TargetingDimension

Allows for more flexibility

Can one of the admins verify this patch?


Add/remove part for people icon in taskbar

Hi,
I've added a part for hiding the new (my) people in the taskbar. I believe this button show up in the Creator Update.
Cheer,
Nerothos

Interesting. I don't have this functionality at all in my language version. Will merge the PR for next release. Thanks.

I'm using the english language (US version IIRC) so maybe it's not for all for now.


Fix cloudwatch logs' response error

As Cloudwatch logs mock have error with response, fixed it.
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/client.py", line 312, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/client.py", line 588, in _make_api_call
    operation_model, request_dict)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/endpoint.py", line 141, in make_request
    return self._send_request(request_dict, operation_model)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/endpoint.py", line 170, in _send_request
    success_response, exception):
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/endpoint.py", line 249, in _needs_retry
    caught_exception=caught_exception, request_dict=request_dict)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/hooks.py", line 227, in emit
    return self._emit(event_name, kwargs)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/hooks.py", line 210, in _emit
    response = handler(**kwargs)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 183, in __call__
    if self._checker(attempts, response, caught_exception):
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 251, in __call__
    caught_exception)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 269, in _should_retry
    return self._checker(attempt_number, response, caught_exception)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 317, in __call__
    caught_exception)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 223, in __call__
    attempt_number, caught_exception)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/retryhandler.py", line 359, in _check_caught_exception
    raise caught_exception
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/endpoint.py", line 204, in _get_response
    proxies=self.proxies, timeout=self.timeout)
  File "/Users/kawasakitoshiya/.pyenv/versions/3.6.0/lib/python3.6/site-packages/botocore/vendored/requests/sessions.py", line 573, in send
    r = adapter.send(request, **kwargs)
  File "/Users/kawasakitoshiya/workspace/moto/moto/packages/responses/responses.py", line 308, in unbound_on_send
    return self._on_request(adapter, request, *a, **kwargs)
  File "/Users/kawasakitoshiya/workspace/moto/moto/packages/responses/responses.py", line 261, in _on_request
    status, r_headers, body = match['callback'](request)
  File "/Users/kawasakitoshiya/workspace/moto/moto/core/utils.py", line 170, in __call__
    result = self.callback(request, request.url, request.headers)
  File "/Users/kawasakitoshiya/workspace/moto/moto/core/responses.py", line 115, in dispatch
    return cls()._dispatch(*args, **kwargs)
  File "/Users/kawasakitoshiya/workspace/moto/moto/core/responses.py", line 183, in _dispatch
    return self.call_action()
  File "/Users/kawasakitoshiya/workspace/moto/moto/core/responses.py", line 257, in call_action
    response = method()
  File "/Users/kawasakitoshiya/workspace/moto/moto/logs/responses.py", line 92, in get_log_events
    "events": [ob.__dict__ for ob in events],
  File "/Users/kawasakitoshiya/workspace/moto/moto/logs/responses.py", line 92, in <listcomp>
    "events": [ob.__dict__ for ob in events],
AttributeError: 'dict' object has no attribute '__dict__'

Nice fix, @toshitanian - apologies for the huge delay here.


smoother_step

hi, I restart a new issue, because I'm also confused about the timesteps of interleaved-frame smoother. The paper says that : "interleaved-frame smoother is performed on predicted RGB frames at timesteps {30, 31} by default" , But your code use 19, 20 as default. How is this parameter determined， Why not do smoother on all DDIM steps？Have you done ablation for this part of the experiment? I don't seem to see it in the paper.

Since DDIM inverison (50 steps in total) starts from t=50 to t=0, and {30,31}  claimed in paper is indices and actually corresponds to {50-30, 50-31}={20,19} timesteps. We will fix it in our paper.
In our experiments, 2 or 4 steps are enough to smooth the video, and using more will increase the time cost. We have put all ablation results on smoother in our appendix, which will be public later.


Error: couldn't parse 'bool $(as-instr, tpause %ecx)': unknown tokens in line - Newer RHEL Kernels

kconfiglib is throwing a consistent error, pictured below, when attempting to parse newer RHEL, CentOS 8, and CentOS 8 Stream Kconfigs.

I have been able to isolate this error.  It is caused by the macro within the file [Kernel Source]/arch/x86/Kconfig.cpu, pictured directly below on line 479.  The specific config is AS_TPAUSE.

To reproduce this error, obtain recent kernel source code which includes the AS_TPAUSE configuration in Kconfig.  An example of this is the most recent CentOS 8 Stream kernel source.  Then appropriately set your environment variables for kconfiglib and attempt to initialize a kconfiglib object as pictured below.

It appears that kconfiglib can neither handle nor support Kconfigs with AS_TPAUSE at this time.
Thank you for your assistance!

+1 we're also running into this while attempting to build a RHEL 8.6 kernel


Network conection

So, I tried to install some new wallpapers. After a while, it show this notification

I know it must be my connection problem. However, my connection is still normal that I can still post this issue here.
I also tried to reinstall Doki theme, run Restore asset command, restart my computer, but it isn't work.
What should I do?

Few questions:

What kind of VSCode are you using? Is it the desktop version, the browser version (code server) or something else?
Are you able to see the image using this link? https://doki.assets.unthrottled.io/backgrounds/wallpapers/emilia_light.png

What it should look like (but please follow the link)

Few questions:

What kind of VSCode are you using? Is it the desktop version, the browser version (code server) or something else?
Are you able to see the image using this link? https://doki.assets.unthrottled.io/backgrounds/wallpapers/emilia_light.png

What it should look like (but please follow the link)

I am using VSCode Desktop version
I followed the link on my browser and it still appear

Ohh thanks, It work again.  Then, this morning, I turn on the computer and try again, and it work. I still don't know how it was solved


NIFI-12122 Fixed bug where parameter context descriptions were not loaded on NiFi startup and overwritten as empty

Summary
NIFI-12122
When creating a parameter context and adding a description it gets persisted to the flow.json.gz flow definition. If NiFi is shut down, and then restarted, the parameter context is loaded by the controller and FlowManager, but the description is not loaded. The flow is then saved to disk, loosing the parameter context description in the process.
This PR solves that by modifying the FlowManager interface and adding a description parameter to the createParameterContext method. Then in AbstractFlowManager, a .setDescription() step is added, to add the passed description value to the created parameter context.
Also added a test to verify that the parameter context that is imported to the flow manager contains a description if the flow.xml.gz flow file had a description for it.
The rest of the edits are to make existing tests and function calls compatible with the updated FlowManager change.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing
- [ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
- [ ] New dependencies are documented in applicable LICENSE and NOTICE files
Documentation
- [ ] Documentation formatting appears as expected in rendered files

Rebased and fixed checkstyle warning

Thanks for addressing this issue @Hexoplon. One general note: the flow.xml serialization format is in the process of being removed from the main branch, so the test method should use the JSON format instead.

@exceptionfactory The test now uses the JSON format

@exceptionfactory Great! It's been rebased now, and the test modified to be similar to the other modified tests


Add support for @Video directive

Bug/issue #, if applicable: 97715316
Summary
Add support for DocC @Video directive in Documentation pages.

Dependencies
This PR extends the work done the PRs below, and should be rebased after they are merged into main.
https://github.com/apple/swift-docc-render/pull/404
https://github.com/apple/swift-docc-render/pull/406
DocC PR - https://github.com/apple/swift-docc/pull/381
Testing
We need proper DocC support to be able to test this out, but you can add this JSON to the content of a random RenderJSON file from a doccarchive, and it should work:
Video JSON
                {
                  "type": "video",
                  "identifier": "video-reference.mp4",
                  "metadata": {
                    "abstract": [
                      {
                        "type": "text",
                        "text": "Video Caption"
                      }
                    ]
                  }
                }

Reference JSON inside references
    "video-reference.mp4": {
      "poster": "reference-to-poster-image",
      "type": "video",
      "identifier": "video-reference.mp4",
      "alt": null,
      "variants": [
        {
          "traits": [
            "1x",
            "light"
          ],
          "url": "/path/to/video.mp4"
        },
        {
          "traits": [
            "1x",
            "dark"
          ],
          "url": "/path/to/video~dark.mp4"
        }
      ]
    }

Steps:

Assert a replayable video is rendered with a "Play" button beneath it.
Assert Video can be rendered with a caption below it, but also without.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

I think we should mark this PR as a draft until the two dependent PRs have been merged, and this has been rebased.

Yes, I just wanted to get things rolling.

@swift-ci test

@swift-ci test

@swift-ci test

My only minor concern is that the "Play" icon is the same icon as the "Replay" icon

You have a point Marcus, I changed that because it's a good point anyway.

I will check with Design if they want me to just show full native controls all the time, instead of just on mobile devices. I think we discussed that we may want just the minimal UI of the tutorials.

@swift-ci test

@swift-ci test


Always use the subpackage name as suffix

Should fix the check statuses label for all jobs

I am not really sure I want to merge this PR 😅 I thought it could fix the statuses check report, where for some statuses the name of the sub-package is not present, but this changed nothing... now I am quite sure that for those statuses the full identifier is missing.
So I keep this open for a while but probably I will close it.

The problem was the serialization/deserialization for the JobConfig object. I close this PR .


Bike shed and find uniform format for deprecated note field

Can we have a uniform format for
#[deprecated(since = "TBD", note = "format of this field is under discussion")]
Currently we have:

Capitalized with full stop
Capitalized without full stop
Lowercase with full stop
Lowercase without full stop
Lowercase without full stop and multiple "sentences"

I don't personally care what it is but just that its uniform so I don't have to think every time I write one.

We should check if there's any Rust-wide standard first.

We should check if there's any Rust-wide standard first.

In Rust's standard library:

everything starts with lowercase
no period at the end
two most common patterns:

“replaced by abc”
“use abc instead”

So perhaps “use abc instead“ would be suitable, given that this is the wording mostly used in rust-bitcoin. The fix would be only to make it lowercase and remove periods.

rg -trust -I --trim "deprecated.*note"

#[deprecated(since = "1.3.0", note = "renamed to join", suggestion = "join")]
#[deprecated(since = "1.6.0", note = "replaced by `std::thread::sleep`")]
#[deprecated(since = "1.6.0", note = "replaced by `std::thread::park_timeout`")]
#[deprecated(since = "1.16.0", note = "this option can only be set before the socket is bound")]
#[deprecated(since = "1.16.0", note = "this option can only be set before the socket is bound")]
#[deprecated(since = "1.8.0", note = "other methods of this trait are now preferred")]
#[deprecated(since = "1.6.0", note = "replaced by `std::sync::Condvar::wait_timeout`")]
#[deprecated(note = "moved to `std::ops::Bound`", since = "1.26.0")]
#[deprecated(since = "1.26.0", note = "use inherent methods instead")]
#[deprecated(since = "1.8.0", note = "other methods of this trait are now preferred")]
#[deprecated(since = "1.37.0", note = "should be unsafe, use `pre_exec` instead")]
#[deprecated(note = "use `align_of` instead", since = "1.2.0", suggestion = "align_of")]
#[deprecated(note = "use `align_of_val` instead", since = "1.2.0", suggestion = "align_of_val")]
#[deprecated(since = "1.39.0", note = "use `mem::MaybeUninit` instead")]
#[deprecated(note = "no longer an intrinsic - use `ptr::drop_in_place` directly", since = "1.52.0")]
#[deprecated(since = "1.42.0", note = "use the Display impl or to_string()")]
#[deprecated(since = "1.81.0", note = "this never returns anything useful")]
#[deprecated(since = "1.59.0", note = "use `IntoIterator::into_iter` instead")]
#[deprecated(since = "TBD", note = "replaced by the `MIN` associated constant on this type")]
#[deprecated(since = "TBD", note = "replaced by the `MAX` associated constant on this type")]
#[deprecated(since = "TBD", note = "replaced by the `RADIX` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `DIGITS` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `EPSILON` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_POSITIVE` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_EXP` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX_EXP` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_10_EXP` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX_10_EXP` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `NAN` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `INFINITY` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `NEG_INFINITY` associated constant on `f32`")]
#[deprecated(since = "TBD", note = "replaced by the `RADIX` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `DIGITS` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `EPSILON` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_POSITIVE` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_EXP` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX_EXP` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MIN_10_EXP` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `MAX_10_EXP` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `NAN` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `INFINITY` associated constant on `f64`")]
#[deprecated(since = "TBD", note = "replaced by the `NEG_INFINITY` associated constant on `f64`")]
#[deprecated(since = "1.0.0", note = "renamed to is_sign_positive")]
#[deprecated(since = "1.0.0", note = "renamed to is_sign_negative")]
#[deprecated(since = "TBD", note = "replaced by the `MIN` associated constant on this type")]
#[deprecated(since = "TBD", note = "replaced by the `MAX` associated constant on this type")]
#[deprecated(since = "1.39.0", note = "use the `?` operator instead")]
#[deprecated(since = "1.52.0", note = "rustc-serialize is deprecated and no longer supported")]
#[deprecated(since = "1.52.0", note = "rustc-serialize is deprecated and no longer supported")]
#[deprecated(since = "TBD", note = "replaced by the `MIN` associated constant on this type")]
#[deprecated(since = "TBD", note = "replaced by the `MAX` associated constant on this type")]
#[deprecated(since = "1.51.0", note = "use hint::spin_loop instead")]
#[deprecated(since = "1.4.0", note = "use lines()/Lines instead now")]
#[deprecated(since = "1.29.0", note = "use `get_unchecked(begin..end)` instead")]
#[deprecated(since = "1.29.0", note = "use `get_unchecked_mut(begin..end)` instead")]
#[deprecated(since = "1.4.0", note = "use lines() instead now", suggestion = "lines")]
#[deprecated(since = "1.33.0", note = "superseded by `trim_start`", suggestion = "trim_start")]
#[deprecated(since = "1.33.0", note = "superseded by `trim_end`", suggestion = "trim_end")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]
#[deprecated(since = "1.13.0", note = "use `std::hash::DefaultHasher` instead")]

I like @jirijakes suggestions and I prefer the imperative "use abc instead" in place of the passive "replaced by abc".

Can we also make a habit of sticking a PR # or issue # in there somewhere?
(For people making new PRs -- it's mildly annoying to do, but you can just look at the existing PRs/issues list to find the most recent number, and add +1 to it to guess your PR number before pushing. This will almost always work because it's rare to have two new PRs within minutes of each other, and easy to fix after the fact.)

@apoelstra we could also replace it during release when TBD is being replace. I think it should be scriptable.
Anyway, ACK using std's imperative style.

@Kixunil oo, interesting.
In principle it is scriptable but I think it'd require having an index of commit IDs to PRs. (If we had such a thing, I suppose we could also sanity-check our changelog to insist that every change that added a deprecation showed up there somewhere..).
I've got such an index in my label-pr tool which has served me well but it's not currently designed to be used within scripts, and also it's extremely slow on massive repos.

Github GUI can tell you in which PR a commit was, I'd be surprised if it's not accessible from API. I think we could also get away with scanning which merge contains adding of deprecation - we only need to scan since last release and there shouldn't be so many merges for it to take too long during release.

Nice work @jirijakes!

Push #2137 forwards as part of this.

#3475 changes all the existing deprecated note fields to the “use abc instead” format.
I was going to add the PR number to the note too using git blame, until I came across one which is "automated rustfmt nightly".  If there are only a few like that I can find the original PR with git log.
Is the following format ok?
"use abc instead (see PR #xxxx)"

I don't think we should add PR numbers to the code base.

#3475 made all of the exiting deprecated notes uniform and #3476 adds a section to CONTRIBUTING.md stating the agreed format of
#[deprecated(since = "TBD", note = "use `alternative_method()` instead")]
Suggest to close this Issue.


Late submissions of non programming exercises

Describe the issue
I'm not 100% sure. Either the issue is that when the student submits that his hold submission gets overridden or the issue is that if you submit late the submission gets still overridden.
To Reproduce

Submit a exercise (Text, Modeling) multiple times until the deadline
Submit after the deadline
Assess the submission
Check the participation of the student and see that there is only one submission and it's a late one

Expected behavior
Either you have multiple submissions for one participation and the last one before due date counts or you can't override the submission after due date but must return an error.
I don't know why it's implemented this way but I think it's unfair if you try until the end and get punished for it because you don't know the exact server time.

I tried the following setup:

Create a new Text exercise with the duedate in some minutes.
Submit multiple times until the "Submit" button gets deactivated
Deadline was 19:00

However when I assess it, it counts as a normal (not late submission) in contrary to what is stated in the screenshot

However this is not what a student has experienced (4 seconds late, graded with full points but doesn't count). I'll update this issue as soon as I know more.

Fixed by #7258


Change metrics scripts to use MetricSender instead of ZaggSender

https://github.com/openshift/openshift-tools/pull/1752 from stg
@twiest @joelddiaz

👍 #1752 in stg


Updated flutter_map_dragmarker for flutter_map v6 support

I wanted to try out flutter_map v6 in one of my projects, and I didn't want to wait, so I took the initiative to update the flutter_map_dragmarker myself. I already sent a PR to flutter_map_dragmarker. Please take a moment to update the dependency after flutter_map_dragmarker's update.

Thanks for both the PRs, it's really helpful. It may be a couple of days before I get chance to test. Would it be possible to update the version numbers as well to 6.0, I'll try and keep them in line with FM

Great, thanks!

Closing this since a new pull request is sent #41


openapi: add spec and code generation

This will implement #518 .
Posting early to get feedback on the yaml files and the Makefile hookup. Will implement the full API before asking it to be merged.

Could we use go's support for generating source code? oapi-codegen seems to support that.

I don't think this applies to our use-case. Unless I'm misunderstanding something here :)

[...] The docs on that link doesn't seem to be what we want, [...]. For now I suggest we merge it as is though, so we don't block on figuring that out.

Oops, sorry about that. I should've given a bit more detail, too: Integrating it with go build means we can keep using the %gobuild macros in the spec files. Committing the generated files has the same effect, of course. Let's go with that for the first iteration.

Thanks for all the input! This is now partially superseded by #583.


Widget tests starts to fail when using PdfViewer widget

Hi,
First I would like to thank you for this amazing package, my problem with it is that my widget tests started to fail once I started to using it, replacing pdf_render. The problem seems to be related with a timer not being disposable correctly inside the package, maybe is related to the https://github.com/espresso3389/pdfrx/issues/246. The code is quite simple, I pasted below as long with the error message.
I tried with both PdfViewer.data and PdfViewer.file I got the same error for both.
class TicketPdfPreview extends StatelessWidget {
  const TicketPdfPreview({
    super.key,
    required this.data,
  });

  final Uint8List data;

  @override
  Widget build(BuildContext context) {
    return PdfViewer.data(
      data,
      sourceName: '',
      params: PdfViewerParams(
        backgroundColor: Colors.transparent,
        errorBannerBuilder: (context, error, stackTrace, documentRef) {
          return ErrorWidget(error);
        },
      ),
    );
    ;
  }
}

Widget test
testWidgets('TicketPreview renders.', (tester) async {
    await tester.pumpWidget(
      widgetFramework(
        child: TicketPreview(
          ticket: 'html',
          width: 3,
          printerUtils: ts.buildPrinterUtils(),
        ),
      ),
    );

    await tester.pumpAndSettle();

    expect(find.byType(TicketPdfPreview), findsOneWidget);
    expect(find.byType(PdfViewer), findsOneWidget);
  });

Exceptions
════════════════════════════════════════════════════════════════════════════════════════════════════
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following assertion was thrown running a test:
A Timer is still pending even after the widget tree was disposed.
'package:flutter_test/src/binding.dart':
Failed assertion: line 1542 pos 12: '!timersPending'

When the exception was thrown, this was the stack:
#2      AutomatedTestWidgetsFlutterBinding._verifyInvariants (package:flutter_test/src/binding.dart:1542:12)
#3      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1049:7)
<asynchronous suspension>
<asynchronous suspension>
(elided 3 frames from class _AssertionError and package:stack_trace)

The test description was:
  TicketPreview renders.
════════════════════════════════════════════════════════════════════════════════════════════════════
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following message was thrown:
Multiple exceptions (2) were detected during the running of the current test, and at least one was
unexpected.
════════════════════════════════════════════════════════════════════════════════════════════════════

1.0.93 contains the fix.


Start 1.9.0

No review needed.

coverage: 33.944%. first build when pulling 66168b386fc918fa014af11278f05199cb454e0c on start_1.9.0 into 588ab599d24dfed76d765d96f2e7ca9afad9c6de on master.


[Feature Request]

I have array of customers  and each customer object has several properties that are unique.
Customer = {   Id : 0, RegistrationNumber : 1, NationalCompnayNumber : 1 }
so is it possible to search by Id, RegistrationNumber  or  NationalCompnayNumber  ?

Yes, with custom search. The explanation is in https://github.com/monterail/vue-multiselect/issues/319


Updaete SHA example to use newer risc0 with canonical byte ordering

In 3224c3c64e38bb42fca4df86a95157e2b04e84b8 the byte ordering of the SHA digest was changed to make
it use the canoncial, big-endian, ordering. This PR includes updates to the sha crate to make it
compatible with that ordering change.

Will be superseded by a PR from @3lkn


Channel Not Opened & Cannot Remove Pending Channel

Background
A few weeks ago, I tried opening a channel to a peer. The channel failed to open but remained as a pending channel. The opening transaction was never broadcast and the channel point on the pending channel does not correspond to a transaction on the blockchain or a pending one.
I tried to forceclose it and it moved to waiting_close_channels, where it has been for about a week. Trying to close it again give response [lncli] rpc error: code = Unknown desc = channel is already in the process of being force closed
Any way to remove this pending channel?
Your environment
0.6.0-beta commit=v0.6-beta-rc3-10-ga2aeb646e7b6fe3f1c2af5ccafd13d785431bb86
Linux 4.14.35-128
bitcoind v0.17.1
Expected behaviour
This pending channel should be cleared-out automatically at some point.
Actual behaviour
Pending channel remaining indefinitely.

Seems related to #2948. Can you paste the logs for the funding and force close transactions?

Could not find opening log, but here is when I try to close:
Apr 16 14:51:28 thundroid lnd[2816]: 2019-04-16 14:51:28.822 [INF] HSWC: Removing channel link with ChannelID(5d90da315c748d34cc08b370d67e7ac856b41185aad9cf7fd7b414872e96b38f)
Apr 16 14:51:28 thundroid lnd[2816]: 2019-04-16 14:51:28.822 [INF] CNCT: Attempting to force close ChannelPoint(8eb3962e8714b4d77fcfd9aa8511b456c87a7ed670b308cc348d745c31da905d:1)
Apr 16 14:51:28 thundroid lnd[2816]: 2019-04-16 14:51:28.822 [WRN] CNCT: Unable to disable channel 8eb3962e8714b4d77fcfd9aa8511b456c87a7ed670b308cc348d745c31da905d:1 on close: edge not found
Apr 16 14:51:28 thundroid lnd[2816]: 2019-04-16 14:51:28.823 [ERR] RPCS: unable to force close transaction: channel is already in the process of being force closed

Do you recall why the channel failed to open? Was it due to a double spend or low fees during the fee spike? Can you find the channel's funding transaction in your bitcoind's mempool?

bitcoin-cli getrawtransaction does not find it No such mempool or blockchain transaction.. I don't think it made it to my node to be broadcast. The channel opening just stayed in pending_open_channels for most of a day before I checked the txid and it didn't exist on-chain or in the mempool. It stayed there until I tried to force close it and it has stayed in waiting_close_channels since.

Here's the channel info:
 "waiting_close_channels": [
        {
            "channel": {
                "remote_node_pub": "02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b",
                "channel_point": "8eb3962e8714b4d77fcfd9aa8511b456c87a7ed670b308cc348d745c31da905d:1",
                "capacity": "1000000",
                "local_balance": "989324",
                "remote_balance": "0"
            },
            "limbo_balance": "989324"

If the transaction failed to broadcast due to being invalid, then the funds should not be locked up. If you believe they still are, you can force a rescan from your wallet's birthday. See here for more details on how to do so. I would only do this if you're absolutely sure the transaction cannot confirm in the chain.
You can attempt to use the abandonchannel command with a debug build of lnd to remove the pending channel record. To obtain a debug build, just run make in lnd's source directory and use the binary created there: ./lnd.

I'm certain no funds were moved in regards to this so I'll go ahead and remove it.
Thank you for your help!

Sweet. Will close for now then, feel free to comment or ask on our Slack if you have trouble doing the above.


Problems with scripts/dd_ibs_test.sh and scripts/dd_obs_test.sh

First, thanks for the handy scripts! Needed to tweak them to make them work though...
Testing this with bash 4.3 on Raspbian, the scripts silently exit without running the tests.
Seems the following line is the problem:
[ -e "$TEST_FILE" ]; TEST_FILE_EXISTS=$?
This test appears to be entirely redundant (and for some reason it causes an early exit as well).
With that, the last line would also simplify from
[ $TEST_FILE_EXISTS -ne 0 ] && rm $TEST_FILE
to
rm $TEST_FILE
Now, I wonder what the purpose of that problematic line above was, exactly? It seems like if a file exists 'ibs' will early exit anyway (with a message), and 'obs' doesn't even have the (working) existence check seen in 'ibs'. Thoughts?

Hey Marty!
Thanks for pointing this out. I wonder if it's something that changed with a more recent version of Bash, because I swear this used to work.
I think the intention of the check was to prevent users of the script from clobbering their files. An error message of some sort would be useful, huh?
This script is a little funky because there are version in a couple of places. @rugk ran into the same problem as you and submitted a fix here: https://github.com/tdg5/blog_snippets/pull/3
Maybe that will help for now?
Ultimately, I need to do a better job of maintaining these scripts. They get more use than I expected and they warrant revisiting. My plan is to break the dd scripts out into their own dedicated repo. I'll keep you posted as to when that happens and with any luck you can verify that the updated scripts meet with your expectations.
In the mean time, thanks for raising this issue!

That's the thing - in dd_ibs_test.sh there is such a test, complete with error message.
More oddly, there are two copies of the scripts... the ones here (of which only one got fixed):
https://github.com/tdg5/blog_snippets/tree/master/lib/blog_snippets/articles/tuning_dd_block_size
And the ones here which I opened the issue on:
https://github.com/tdg5/blog/tree/master/_includes/scripts
I was able to work around it as I described above, though I might submit a patch (or patches) myself now that I understand what the logic is (the merged patch makes the script work, but can be simplified and made common).
As for moving both into a single place, now that I'm aware it exists I favor how blog_the snippets repo is (README and all)... of course, the folder here in tdg5/blog ought to either point to that or be removed to avoid confusion.

My problem isn't actually an issue, I just need a little help. Sorry if I shouldn't be asking here. I want to use the script before I install my OS from usb, to an empty drive. Can someone post a piece of the code, before and after, for an example of which piece that I need to copy and paste over? I'm aware that I would have to change the drive path, to my own.