pynio backend broken in python 3

pynio recently released a development python 3 port (see https://github.com/NCAR/pynio/issues/10). The pynio backend in xarray needs a slight fix to use this though; the get_variables() method in the NioDataStore class tries to access the iteritems() method of the variables object, which no longer exists in python3. The simple fix is to replace iteritems() with items() in this method. I've made this fix on a local version of the code, and it seems to work as expected. I would be happy to put together a PR if nobody has done such, and if someone could point me to any documentation or recommendations on the preferred conventions for structuring PRs for the project (I have not contributed before). Thanks!

@brhillman - that would be great. I don't think anyone has worked on this yet and we'd be happy to have you contribute here.
We have some contributor guidelines in the works (see #1485).
Generally, we'll want to add the pynio package to one or more of the python3 test suites (hre would be a good place to start).
Put your changes on a branch (e.g. feature/python3_pynio), run the test suite (py.test), follow pep8 formatting, and then push to your fork and open a PR. The template checklist in the PRs helps us make sure we're catching the main things.

@jhamman okay I'll take a look and put this together. Would you prefer a single PR to both fix the issue and add a pynio test, or separate PRs for those two changes? I've got a bit of reading to do to get up to speed on adding a test I think.

One PR is fine. I don't think you'll need to add any tests, just add pynio to the conda environment file I linked to above and the existing test base should run for python 3 with pynio.
We may need to ping @ocefpaf to help us get a conda-forge build of pynio for python3 repo.

Do you know if the py3k support made into pynio 1.5.0 release? If so I can "fix" the conda package easily, if not I am unsure what source I should use to enable the Python 3 builds.

@ocefpaf no, I don't think so. I think the only version of pynio that has py3 support is the dev snapshot on the ncar conda-forge channel. This is the version I used to get pynio to work in my python 3 environment so that I could use xarray to read ERA-Interim grib files. I installed the dev snapshot via conda install -c ncar -c conda-forge pynio=dev python=3.

Yep, just saw that in https://github.com/conda-forge/pynio-feedstock/pull/30
So we need to wait... Hopefully they will release it soon. Thanks!

@ocefpaf @jhamman should I wait to put together the PR then until they release an official version of pynio with py3 support?

@brhillman  - A PR to fix the iteritems issue can happen now. You could also add a test to the allowed_failures section:
https://github.com/pydata/xarray/blob/c748062ef4353f5e249553b06a198a44c71083c4/.travis.yml#L42-L61
that pulls the ncar dev build down. That would allow us to work out some of the kinks while they get their final python3 release ready.
More broadly, if we don't have a viable way to install the package, perhaps your effort would be better spent helping PyNio get Python3 into a full release.

Okay @jhamman excuse my ignorance on the test suite, I've never worked with pytest or Travis CI before, but if I understand correctly, this would involve adding a section like:
- python: 3.6
   env: CONDA_ENV=py36-pynio-dev

to xarray/.travis.yml, and then add a corresponding ci/requirements-py36-pynio-dev.yml file to list the dependencies for that test environment? I've gotten the tests to run and pass on my environment just by adding an entry to ci/requirements-py36.yml that reads
 - pynio=dev python=3 and adding an entry that reads   - ncar to the "channels" section, but maybe this is the wrong place to add this?
FYI I also had to make a change to xarray/tests/__init__.py to import pynio correctly (the test would have never run, regardless of whether or not pynio because it tried to import the wrong package name).

Yep, it sounds like you have it about right. Why don't you open a PR and we can iterate further there.

@jhamman okay I figured it out, added the tests, confirmed that the Nio tests get run, and they all pass (other than a dask test, but I think that's expected right now?). But, I'm getting some strange results from git diff upstream/master | flake8 --diff, on lines that I did not modify, and some lines that I copied the syntax over from existing files (i.e., ci/requirements-py36-pynio-dev.yml, where it complains about indentation). Is this a concern? PR ready to go other than that. My branch is here:  https://github.com/brhillman/xarray/tree/fix-py3-pynio-backend.


HD and FHD For ADN (AnimationDigitalNetwork)

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
France
Example URLs
https://animationdigitalnetwork.fr/video/naruto-shippuden/7579
Provide a description that is worded well enough to be understood
have the possibility to download the episodes in hd/fhd like this link which has hd/fhd but yt-dlp does not detect it, I can share my cookies if necessary.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
> yt-dlp -vU -F --cookies "cookie.txt" "https://animationdigitalnetwork.fr/video/naruto-shippuden/7579"
[debug] Command-line config: ['-vU', '-F', '--cookies', 'cookie.txt', 'https://animationdigitalnetwork.fr/video/naruto-shippuden/7579']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [615a84447] (pip)
[debug] Python 3.12.0 (CPython AMD64 64bit) - Windows-11-10.0.22631-SP0 (OpenSSL 3.0.11 19 Sep 2023)
[debug] exe versions: ffmpeg N-112972-g5256b2fbe6-20231214 (setts), ffprobe N-112972-g5256b2fbe6-20231214
[debug] Optional libraries: Cryptodome-3.20.0, brotli-1.1.0, certifi-2024.02.02, mutagen-1.47.0, requests-2.31.0, sqlite3-3.42.0, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets
[debug] Loaded 1803 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[ADN] Extracting URL: https://animationdigitalnetwork.fr/video/naruto-shippuden/7579
[ADN] 7579: Downloading player config JSON metadata
[ADN] 7579: Downloading access token
[ADN] 7579: Downloading links JSON metadata
[ADN] 7579: Downloading vostf mobile JSON metadata
[ADN] 7579: Downloading m3u8 information
[ADN] 7579: Downloading vostf sd JSON metadata
[ADN] 7579: Downloading m3u8 information
[ADN] 7579: Downloading additional video metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for 7579:
ID        EXT RESOLUTION │   FILESIZE  TBR PROTO │ VCODEC      ACODEC
────────────────────────────────────────────────────────────────────────
vostf-416 mp4 640x360    │ ~ 67.44MiB 416k m3u8  │ avc1.4d001f mp4a.40.2
vostf-834 mp4 853x480    │ ~135.28MiB 835k m3u8  │ avc1.640028 mp4a.40.2

cookies don't work on ADN, you need to provide username and password
https://github.com/yt-dlp/yt-dlp/issues/9067#issuecomment-1910695509


makeDOMDriver does not handle being called before DOM Ready

From @WorldMaker on October 20, 2015 16:19
I created a test bundle of all my scripts (using jspm bundle-sfx if you are curious, but that shouldn't matter) and when attempting to run as a bundle I hit the case where the browser was running the makeDOMDriver function prior to the DOM Ready. (Interestingly the DOM in this case was just the single DIV I'm currently using as a container, so kind of amazing the browsers manage to run the scripts before evaluating the DIV...)
Does it make sense for makeDOMDriver to handle the case where it is being called before DOM Ready?
Copied from original issue: cyclejs/cycle-dom#56

Yeah I'll mark this as a bug because makeDOMDriver should indeed start doing its stuff after the DOM is ready, but nothing will change on the API side. You will still call makeDOMDriver whenever you want.

I'll be working on this.
This is simply about adding document.addEventListener(DOMContentLoaded, () => {/**/}) at the right place in makeDOMDriver right?

@chadrien I think we should wait for a major rewrite to Cycle DOM before doing this. In other words, let's pause any development on Cycle DOM master, unless it's a critical bug. Check out this branch https://github.com/cyclejs/dom/tree/diversity where we are rewriting Cycle DOM in TypeScript and Snabbdom. It'll be a breaking change because virtual-dom API differs from snabbdom just a tiny bit.
I'll remove help wanted from this issue for now. I'll check other issues to see if help wanted is for real.

See also comments of duplicate https://github.com/cyclejs/core/issues/301

@staltz got a fix but TypeScript doesn't like Property 'readystate' does not exist on type 'EventTarget'.
might be a fix here but then I'm not sure how I can make a test that would be reasonable!! basically I've been no help!!
But here is the PR

if this is still a bug I can work on this...

Take my PR as a reference if you like @artfuldev 😄
I've not bothered learning TypeScript as I went towards PureScript instead! So this bug is still open

I saw the PR @Cmdv - you're almost done.

in this case it might be quicker to do that and just submit your own PR 😄  @artfuldev

Did you start working on this, @artfuldev ?

I can complete this? I want to start contributing to Cycle.js somewhere, and this looked like low hanging fruit... I didn't hear from staltz, so I didn't proceed much. But this shouldn't take much time. Please, can I work on this?

The dom library is undergoing some heavy refactoring which is waiting for a merge, so it might be better for you to wait until then to avoid massive merge conflicts. @artfuldev

Please point me to any other low hanging fruit then @Frikki
As an aside, not able to contribute from a Windows machine. Should a Mac be alright?

I don’t know about Windows, but any unix-like system would surely do.
You can filter issues by size to search for small issues. You can also just wait a day or two, then the merge will surely have happened.

Has the merge happened?

No, you can ignore the big refactor PR for the time being. Frikki would do incremental refactors.

Fixed finally in the unified branch. Not yet released.


Incompatible with SpongeForge

Hello,
while trying to use SpongeForge, it failed to load. After adding mods 1 by 1 to the folder, I found that movingworld prevents Sponge from loading. It DOES NOT crash.
Here is a log with ONLY SpongeForge and movingworld installed: https://pastebin.com/AcAai1ZH
NOTE: I am using movingworld-1.12-6.342-full (And any version of spongeforge)

MovingWorld is embedding an older version of Mixin, and is getting loaded first.
Try using SpongeBootstrap to guarantee that Sponge is loaded first.


Drone: Can't Connect Without Private SSH Key

Relevant section of my .drone.yml
  ssh:
    image: plugins/ssh
    host: example.com
    user: isaac
    port: 22
    script:
      - echo `date` > test.txt

Drone Outputs this:
$ docker images -q -f dangling=true
[info] Pulling image plugins/ssh:latest
Error: Can't connect without a private SSH key.
[info] build failed (exit code 1)

I've copy-pasted the Public Key that shows under the Settings tab into my ~/.authorized_keys on my host machine, and it really looks like there are private keys in the /var/lib/drone/drone.sqlite database.

Did you set your private key?
drone secret add --image=plugins/ssh your/repo_name PLUGIN_SSH_KEY $(cat ~/.ssh/id_rsa.pub)

It sounds like you try to use the 0.5 plugin on a 0.4 server. The 0.5 server doesn't provide any ssh key.

Is there any way to use the 0.4 plugin? Tagging :0.4 fails

plugins/drone-docker is 0.4, plugins/docker is 0.5 :)

@tboerger I probably wasn't very clear :) Is there a way to use the equivalent of plugins/ssh:0.4 since I'm using a 0.4 server?

Just use plugins/drone-ssh, for 0.4 we had the convention to name the images plugins/drone-foo while it have been changed for 0.5 to plugins/foo. All plugins should have a release/0.4 branch and for this plugin you can see the docs at https://github.com/drone-plugins/drone-ssh/blob/release/0.4/DOCS.md
I will close this issue since it's just mixing 0.5 plugins with a 0.4 server. If you still encounter problems feel free to join the Gitter chat and ask there for support.


add command for retrieving plugin status

the setting page shows some details, but no error messages
https://translate.dokuwiki.org/plugin/dw2pdf/settings
Add command that displays these errors. Store all error messages and not only latest. Seems the mailed message is stored as message, 250 characters is not enough.
Workaround:
select name,state,url,branch,from_unixtime(lastUpdate),errorMsg,errorCount
from repository
where name='yourplugin';

ah, there is already a dokuwiki:showStats. Shows basic stuff for all extensions. Error output is still lacking. Also add filter for all extensions with number of error >0

dokuwiki:showStats is renamed with dokuwiki:showInfo and will show the entire error messages if a single repository is shown.
i.e.  dokuwiki:showInfo plugin example . Implemented in #112


Update documentation after refactor to copier

Checklist:

[ ] Added tests for changed code where applicable.
[x] Documentation reflects the changes where applicable.
[x] Updated the CHANGELOG.md file with your changes.
[x] My PR is ready to review.

Resolves: #175
Description of the changes being introduced by the pull request:

Codecov Report

Merging #213 (bd235da) into master (da0fbe3) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #213   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          377       380    +3
=========================================
+ Hits           377       380    +3

Impacted Files
Coverage Δ

fastapi_mvc/core.py
100.00% <100.00%> (ø)

fastapi_mvc/generators/loader.py
100.00% <100.00%> (ø)

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


Update Universe Package to v2.0.0

@djosborne can this be closed?

Probably not until it's accepted into the DC/OS Universe: https://github.com/mesosphere/universe/pull/872


Fix 'A closing curly bracket must not be preceded by a blank line.' issue in LearnFiles\Basics\WhileLoops.cs

CodeFactor found an issue: A closing curly bracket must not be preceded by a blank line.
It's currently on:
LearnFiles\Basics\WhileLoops.cs:32
Commit 924fd1e4a1fe34f64d5a161521583dca4defb1f6

Removed Unnecessary Spacing


wasm-pack support missing.

I want to use the package with webassembly, unfortunately there is no support (I guess) for wasm-pack's compiler wasm32-unknown-unknown. Is there any plan to add support in the close future? Thanks
I am using:
Ubuntu 18.04.2 lts
rustc 1.32.0
wasm-pack 0.6.0
$ wasm-pack build
--> /home/regl/.cargo/registry/src/github.com-1ecc6299db9ec823/dirs-1.0.4/src/lin.rs:41:18
 |

41 |     use std::os::unix::ffi::OsStringExt;
|                  ^^^^ could not find unix in os
error[E0432]: unresolved import unix
--> /home/regl/.cargo/registry/src/github.com-1ecc6299db9ec823/dirs-1.0.4/src/lin.rs:6:5
|
6 | use unix;
|     ^^^^ no unix in the root

storage access from browser to local file system does not really make sense..


[Authy] ...

Extension
https://www.raycast.com/guga4ka/authy
Description

Error:
Error: Invalid base32 characters

uu:OtpListItem.tsx:72:38
fu:search-otp.js:42:4668
Ir:index.js:6:5510
    at ray-list

Steps To Reproduce
Initial installation from the Raycast store. The error occurred while trying to use the extension for the first time.
Current Behaviour
No response
Expected Behaviour
No response

i have the same problem but happened when syncing the extension today
Error: Invalid base32 characters

uu:OtpListItem.tsx:72:38
fu:search-otp.js:42:4668
Ir:index.js:6:5510
    at ray-list

System:
Mac OS Ventura 13.3.1
i7

It seems to be related to #5880. Despite that issue suggesting that the problem was sorted, it remains even on a fresh install.

lets track it in one place #5880
@raycastbot close this issue


HI，GREAT WORK!

HI，GREAT WORK!
i have serval questions, is there a edge dataset exit in your data?
or how can i got this edge data?

no, it is not required. you just need to remove it.

发件人: gayuyu @.>
发送时间: 2023年7月14日 18:08
收件人: zwbx/DFM-Net @.>
抄送: Subscribed @.***>
主题: [zwbx/DFM-Net] HI，GREAT WORK! (Issue #12)
CAUTION: External email. Only click on links or open attachments from trusted senders.

HI，GREAT WORK!
i have serval questions, is there a edge dataset exit in your data?
or how can i got this edge data?
—
Reply to this email directly, view it on GitHubhttps://github.com/zwbx/DFM-Net/issues/12, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJFDSLYDM66RR4FFRYAI3KLXQEAQBANCNFSM6AAAAAA2KAWRRE.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


I converted the resource pack to use the new 1.14 feature (custom_model_data), may need a few changes to the datapack.

I converted the resource pack to use the new 1.14 feature (custom_model_data), it may need a few changes in the datapack though:
MechResources_v2.0.6b.zip

Doesn't work, due to this bug: https://bugs.mojang.com/browse/MC-138426.

I see you have made your own implementation of it. No worries, it is nice that you are making progress in making this modpack work more efficiently using 1.14 mechanics.


HADOOP-13075 adding sse-c and sse-kms

Updating fedecz changes to work with changes in trunk.  Refactored methods to be clearer as to purpose and added more integration tests.

Accidently included others commits due to trying to resolve unpushable commit.  Update to the pull request is in e7303f9eff316d3d4329de9011cfd4348ac97a23.  Will correct this tomorrow and resubmit cleaner patch.

Overall, production code looks good —just some minor tweaks needed. Test code all LGTM, as do docs. As stated on the JIRA, I'd really like tests which verify that sse-c credentials are making it up. Anything we can do to expand coverage of the actual S3 state helps the users, and it helps catch any regressions in our code or the SDK library.

As stated on the JIRA, I'd really like tests which verify that sse-c credentials are making it up.

The ITestS2AEncryptionSSEC tests currently just check to make sure that md5 of the encryption key is filled in.  I'll change it to do a md5 on the key to verify it.  The AWS UI won't show those files as encrypted in the properties or metadata.  But the Java SDK can see if that one field is filled in with the md5.

Closing because changes are already pushed in.


Add Endpoints to Container page

Rancher versions: 6/5
Steps to Reproduce:

Create a container with endpoints
Go to Container page

Results:  Add Endpoints to main container page like we have for scaling groups.

Need API field...

Version - 2.0 master 8/5
Verified fixed


Moving BranchLinkUsed to intent extra

BranchLinkUsed used to added to the Uri. Since this has issues with
partners and fragmented uris adding BranchLinkUsed as an intent extra.
@EvangelosG @aaustin
Testing with Test Bed app

Create a new link with test bed app
Click the link and open the testbed app ( Expected : app should open and  should show the auto deep linked activity)
Navigate back to MainActivity by clicking back button
Press home button to background the app
Relaunch the app from background (Expected : App should not show the auto deep linked activity)

@mparul ^

my test: https://drive.google.com/a/branchmetrics.io/file/d/0B0mNoy6xg6lmN2N5V3NNOUNnTmM/view?usp=sharing

👍


Normalize python version

setuptools will force a version to be 'normalized'
which means that it removes the dash.
This will probably cause some weirdness on the version
numbers in the windows build; might have to re-work
my patches on cx_freeze: https://bitbucket.org/jobevers/cx_freeze/commits/all

osx builds are so far behind its dumb, merging as windows and linux passed


fix: OPTIC-1087: Label Config Preview not handling editor lifecycle correctly for updates

PR fulfills these requirements

[X] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[X] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[X] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[X] Frontend

Describe the reason for change
When changing values in the Labelling Config via the live editor, the correct lifecycle methods were not firing on update and there would be a missed step to clear out shared stores on the current instance. It would then proceed to try and create another AnnotationStore again, and end up with a case where the shared store had a reference in the module scoped Stores and StoreIds Map(s), but the local instance of the mobx model did not, and would try to re-register the exact copy once again which was failing. Once failed, this error would propagate across navigations to the next time a task would attempt to be loaded and would fail to load a task with a Taxonomy tag, only clearing up on a refresh of the browser.
Which logical domain(s) does this change affect?
Config Preview, Taxonomy

:rocket:
This description was created by Ellipsis for commit 0ae42ceb9314764fd5d953bd577f8457689ad30a

fix: OPTIC-1087: Correct Label Studio lifecycle in Config Preview
Summary:
Fixes lifecycle handling of Label Studio in Preview.jsx to prevent errors in the live editor.
Key points:

Fix: Correct lifecycle handling of Label Studio in Preview.jsx.
Behavior: Moved cleanup logic to a separate useEffect with an empty dependency array.
Prevents Errors: Ensures proper destruction of Label Studio instance on component unmount.
Frontend Impact: Affects live editor functionality in Config Preview.
Issue: Prevents errors related to shared stores and task loading.

Generated with :heart: by ellipsis.dev

@ellipsis-dev review this


AMA Removal Tool Documentation Fixes

The link can be found here:
https://learn.microsoft.com/en-us/azure/azure-monitor/agents/azure-monitor-agent-mma-removal-tool?tabs=single-tenant%2Cdiscovery
Prerequisites: The resource group, UAI and the corresponding permissions are created by the script and not necessarily a prereq
Set up the utility: 1. The script is MMARemovalUtilityConsolidatedSetup.ps1 not MMARemovalUtilitySetupConsolidated.ps1
Set up the utility: 3. Bicep deployment consistently fails with Bad requests at MMARemovalenvironmentsetup at the steps MMARemovalUtility-ScopeResolver-3ee3d/MSDeploy, MMARemovalUtility-ExtensionRemovalStatusCheck-3ee3d/MSDeploy and MMARemovalUtility-ScopeResolverTrigger-3ee3d/MSDeploy with failure in the deployment template: "Code 429: Cannot acquire exclusive lock to create or update this server farm. Retry the request later." - Bicep CLI version used: 0.25.3
Set up the utility: 3. Location property seems to be required - not sure about the domain controller reference in the description?
Run the utility: "Update-AzTSMMARemovalUtilityDiscoveryTrigger" cannot be found, presumably because of the failures at setting up the utility.

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

ID: b6492b16-82f3-73da-99bf-471277fbd74b
Version Independent ID: 7e5fbd8f-4699-c65a-e147-5ed5564db19f
Content: MMA Discovery and Removal Utility - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-mma-removal-tool.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@gkgab-ptr
Thanks for your feedback! We will investigate and update as appropriate.

@gkgab-ptr - - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can review and update accordingly.

@MonikaReddy-MSFT , Is there any update on this? I'm still having the following error
Update-AzTSMMARemovalUtilityDiscoveryTrigger: The term 'Update-AzTSMMARemovalUtilityDiscoveryTrigger' is not recognized as a name of a cmdlet, function, script file, or executable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

@JeffreyWolford please advise.

There was an issue in the script that has been fixed.  Please can you please downloaded latest zip package from here
https://github.com/azsk/AzTS-docs/blob/main/TemplateFiles/AzTSMMARemovalUtilityDeploymentFiles.zip

@JeffreyWolford / @guywi-ms Thank you for the response; it worked. During the cleanup, I encountered the below warning. However, I already have the owner permission for the RG, and it did not perform the cleanup.
WARNING: User should have Owner permission to perform deletion activity.

Hello,
I managed to complete all the steps for discovery and removal, and I don't see any error. However, I don't see any of shared dashboard widget is updating except "Progress: Configured scopes triggered for discovery phase." and I don't se the MMA removed for the scope that I set.
PowerShell results for Removal :
================================================================================
Configuring MMA Removal utility removal phase trigger...
Updating extension removal scheduler configurations...
Checking if WorkItemScheduler function app [MMARemovalUtility-WorkItemScheduler-4de94] exists...
WorkItemScheduler function app [MMARemovalUtility-WorkItemScheduler-4de94] exists.
MMA 'Removal' phase has been enabled.
Successfully Updated extension removal scheduler configurations.
Completed MMA Removal utility removal phase trigger.
I am happy to share the details in private.

Hello All,
Is there any update on my issue?

Please share the details with me, so we can triage the problems.

Hello,
As I shared it in my previous email, I have done all the steps suggested in
the documentation However, It's not removing or the dashboard not getting
updated. Please let me know what details you require?
Best Regards,
Arjun Rajan Selvaraj
On Fri, Apr 5, 2024 at 5:17 AM Jeffrey Wolford @.***>
wrote:

Please share the details with me, so we can triage the problems.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/119980#issuecomment-2038239109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMGGEI443K4AMGAPG2DGI3Y3W7NHAVCNFSM6AAAAABDPH6TNGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZYGIZTSMJQHE
.
You are receiving this because you commented.Message ID:
@.***>

Hi, the issue still occured to us. The function is in the MMARemovalUtilitySetup.ps1 file, but in step 1 you only load the MMARemovalUtilityConsolidatedSetup.ps1 file. If you enter this commmand: . .\MMARemovalUtilitySetup.ps1 and then run the update-AzTSMMARemovalUtilityDiscoveryTrigger command it should work.

.\MMARemovalUtilitySetup.ps1 and then run the update-AzTSMMARemovalUtilityDiscoveryTrigger command it doesn't work.
$ExecutionContext.SessionState.LanguageMode
FullLanguage
Bicep --version
Bicep CLI version 0.26.54 (5e20b29b58)
$PSVersionTable
Name                           Value

PSVersion                      5.1.26100.268
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.26100.268
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Same results when follow the instruction online "term not recognized"


Robustify handling of numerical issues in samplers

WIP
This is a draft that addresses NCT issue 538.
Per @danielturek , it creates checkLogProb, which has two purposes - one to replace NaN values with -Inf (since we generally want to take a calculation that produces NaN as a rejection), and to print a warning message when an Inf is found.
I've modified various samplers and utility functions to use the new function. It feels a bit kludgey in that we handle logProb calculations (and some degree of checking already) in various ways in our samplers. I've tried to interject checkLogProb in the most reasonable locations in the samplers, but my choices are open to other ideas. Given the accretion of the various components of acceptance/rejection, it was hard to do this in clean, consistent way. My goal is simply to make the samplers more robust.
@danielturek I'd particularly like your review of this.
Will be interesting to see what happens with testing. Given we haven't seen a lot of tickets related to numerical issues, I'm hoping these changes will have little effect. However, I am a bit worried that there have been cases of using samplers where Inf values might arise (such as the stickbreaking nimble-users post that inspired all this) where the sampling might give invalid results.
I'll also note that PR #1509 is related to this.
Other things to consider:

[ ] perhaps use checkLogProb in the new code in sampler_CRP in PR #1509.
[ ] look at SMC samplers to see if we want to do this type of work there
[ ] look at HMC samplers as well

@paciorek Thanks for tagging me on this.
This feels like I'm overlooking something for certain, but how is it that decide is defined as a regular R function (as opposed to a nimbleFunction), and is used extensively in run function code of samplers?  What am I overlooking?
Would another call to checkLogProb be appropriate for line 372 of MCMC_samplers.R?  This is in the core RW sampler, where we separate the logProbs for the target node and dependent nodes:
            logMHR <- logMHR + model$calculateDiff(calcNodesNoSelf) + propLogScale

How about on line 121 of MCMC_utils.R, which is in the function setAndCalculateOne ?
       lp <- model$calculate(calcNodes)

Same for line 164 of MCMC_utils.R, which is in the function setAndCalculate ?
Same for line 180 of MCMC_utils.R, which is in the function setAndCalculateDiff ?
(I know that sometimes the result of these functions in the setAndCalculate family are later wrapped in checkLogProb.  But from my review, this isn't always the case, so my suggestions above are defensive).
For as much as I've tried to standardize things, looking over this has really shown me what a hodgepodge of different structures and approaches the various samplers use ...
Overall, this PR feels like a good approach, given that we want to handle this corner case.

@danielturek thanks, I will look at your suggestions in detail.
As far as decide, it's also a C++ function in Utils.cpp.

Thanks for catching the missing checkLogProb in sampler_RW.
As far as potential uses in those locations in MCMC_utils.R, my logic was to use checkLogProb just before the result would be used for a accept/reject decision or something similar, and not just whenever we call calculate. So I'm inclined to leave them out of MCMC_utils.R except in decideAndJump.

@paciorek Any chance you'd be up to run a very simple benchmark or two? It seems like it shouldn't cost much, but if it is called so much it could be worth checking. One idea would be to code it directly in C++, so it is added to the DSL, and then we could mark it as inline, encouraging the compiler to always insert it in place. The message itself could be contained in another function, in order to keep it out of inlining, since it is only needed when there is a problem.

Yeah, good idea. I took a dead simple normal-normal model with one observation and a single (mean) parameter and tried RW and slice under current nimble and with this change.
It was surprisingly hard to get robust timings, but it looks like for the RW it's about 5% slower and for the slice 8-10% slower.
Since that is for a worst case scenario with the nearly the least amount of calculation relative to the new checking that seems possible, I'm inclined to just merge it in. We could always improve efficiency in a future release.
Thoughts?

@perrydv I went ahead an inlined checkLogProb. Please take a quick look and see if there's anything that is not what you were thinking. Also do you know if it should be inline or static inline?

Also, I meant to say that when I run the simple normal-normal case with the inlining, the time seems to be the same as without checkLogProb at all, so the inlining does seem to help.

Thanks @paciorek that looks good to me and good to hear the performance improved.
I think it's better to use inline rather than static inline. The internet has some interesting explanations on this. Basically we don't need the static.

@perrydv in Utils.h we have static inline for isTRUE: static inline bool isTRUE(bool x) {return(x);}.
Should I remove 'static'?


Motoko Interop

The simplest way to get Sudodb working on Motoko is to implement it in Motoko. I could do that, but I'd rather use the Rust code I've already written. Because Motoko and Rust deployed to the IC will both run in a Wasm environment, we should be able to share libraries between projects written in the different laguages. I think Sudograph will have a harder time because of the resolver generation and query resolving, but Sudodb might work well imported into Motoko. I just need to understand how the data structures are supposed to map over.

Cross-canister calls should make this relatively simple


update README

I'm not sure how well we're doing with "Incremental", "Extensible", and "Mutable" goals. Maybe we should pare back expectations for this? We should also point at the docs, and maybe add some badges.

I agree, I  think we should remove these three goals.


Add CodeClimate (Logo)

Name with link
codeclimate-logo
Description
The CodeClimate logo as opposed to the banner
Link to the vector file, preferably from an official source
Couldn't find an official SVG provided from branding, however, I COULD get this path from their maintainability badge

    <g fill="#FFF" transform="translate(6 5)">
      <polygon points="11.135 0 8.018 3.089 9.872 4.926 11.135 3.674 14.718 7.226 16.571 5.389"/>
      <polygon points="7.518 3.893 5.613 2.065 .754 6.729 0 7.453 1.914 9.29 2.667 8.566 5.131 6.201 5.613 5.738 7.518 7.567 9.312 9.29 11.226 7.453 9.431 5.73"/>
    </g>

Hey @exoRift
The Code Climate logo is already available.

Apologies for reopening the issue, @gilbarbara , I thought I was clearer when I added "as opposed to the banner" to the description.
The logo you currently serve is a banner, not the actual logo itself (which is the progress chart-looking-thing)

I'm not sure I understand what you mean.
If you navigate to https://codeclimate.com/, you'll see that we have the same logo in this repo. It's not a banner or badge.
Do you want to add the logomark (the icon) besides the full logo?

Yes, the icon. Sorry for the confusion

Added in b8f7d66ff934a70f78a850993286146656825d8b
Thanks


Add WeightNorm wrapper layer

Adds a wrapper for Weight Normalization as requested in #14070 & #10125 .
Contains optional data dependent initialization for eager execution, and works on both keras.layers and tf.layers
I struggled to figure out where to place this Wrapper, as no other layers in contrib appear to subclass anything from tf.Keras, but going forward I believe this is the direction TF is headed.  Please advise if it should go somewhere else (maybe a wrappers module in contrib?)
Collab Example:
https://colab.research.google.com/drive/1nBQSAA78oUBmi9fhnHJ_zWhHq2NXjwIc#scrollTo=au25bSP75hdr
Wrapped graph:

Nagging Assignee @case540: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

+1
Can this be reviewed/merged?

Nagging Assignee @case540: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 75 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 90 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @case540: It has been 105 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

+1
I'm already using this, it would be nice to see it in an official release. Thanks for all the work from contributors and reviewers.

This will be moved to a PR for tensorflow/addons.
Closing for now to stop the nagging, but I'll reference this issue within a couple of weeks.

a weight normalization wrapper is available in https://github.com/tensorflow/addons/blob/master/tensorflow_addons/layers/wrappers.py


Added disabled prop to types and makes possible to have a disabled Picky

Currently, the Button component is not consider the disabled prop.
I added and added also the disabled prop to types.

Thanks!


Cleaning up permissions

Permissions Issue
The permissions service in ghost is not as good as it could be.
One of the biggest issues is that it has a lot of work offloaded to the model layer in permissible functions, this makes the logic for permissions scattered and hard to determine/audit from a single place. But there are some smaller issues we can tackle to help make fixin that easier.
What could be done
Nested loops
Due to the permissions service being quite complex, it is quite hard to see what is going on and reason about it. This is in part due to the API the permissions service provides e.g. canThis(context).edit.posts(model, unsafeAttrs) It need to do numerous loops to build up the data structure, only for it to be used once and thrown away.
If we were to make a change to the canThis api along the lines of:
canThis(context, 'edit', 'post', model, unsafeAttrs);

I think it would result in huge improvement of readability and mantianablity of the module.
External context
As the permissions service has grown it has become necessary to add permissions to unauthenticated requests, we've done this via an external property on the context - this has now split permissions outside of the permissions service. If we were able to bring those permissions back into the permission service, we can again make it more maintaniable, and easier to make future changes.
I think the solution here is to add an Unauthenticated or Anonymous role. To begin with we would just give this role permissions for add and edit actions on the subscriber object.
Owner permissions
We also have scattered through the permissible functions, references and check to the Owner role, this again moves permission logic outside of where it belongs.
To address this we should probably give the Owner role all permissions
(There would still need to be checks to ensure the owner could not remove their role or delete themselves)
App context
We have an app context, that has further complicated the permissions service, as we have to check the permissions of the user AND the app. From initial searching - I don't believe the functionality is used anywhere in ghost, nor has it been documented.
Can we delete it?

@allouis Did you see https://github.com/TryGhost/Ghost/issues/9043?
I am not sure we need to issues for this topic?

:+1:

@allouis Wanne merge your ideas into #9043? :)

aw yis i would love to @kirrg001


Display help when not given a subcommand. Fixes #193

@bors: r+ a66b2e04bae5ac7a3826bd5143e4947bb18909e9
Ah, of course!

:hourglass: Testing commit a66b2e0 with merge 93ab4d2...

Let's see if bors actually gets around to this one.

:boom: Test timed out


bug: rust not installed into treesitter when using rust extra

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.4
Operating system/version
macos 14.1
Describe the bug
I've installed the rust and neotest extras in my lazyvim config. When trying to open neotest i see the following error:

Checking TSInstallInfo shows:

Steps To Reproduce
Here's my enabled extras if that helps:

Expected Behavior
Reading the rust extra source it seems like everything should work out of the box as far as treesitter and neotest is concerned.
Repro
No response

Do you have somewhere in your config something like
{
"nvim-treesitter/nvim-treesitter",
opts = {
ensure_installed = {
...
}
}
}

?

I do! Great thinking. This was what i had when trying to enable nix support:
return {
  {
    "nvim-treesitter/nvim-treesitter",
    opts = {
      ensure_installed = {
        "nix",
      },
    },
  },
  {
    "neovim/nvim-lspconfig",
    opts = {
      servers = {
        ["nil_ls"] = {
          settings = {
            formatting = {
              command = { "nixpkgs-fmt" },
            },
          },
        },
      },
    },
  },
}

What would have been the appropriate way to add a plugin like i attempted above?

See this comment. If you use vim.list_extend then you won't have such problems in the future, since the final treesitter spec will merge your configuration with the LazyVim one (which also includes the treesitter fragments of spec throughout the extras).

I don't think so, since both examples are needed to explain what is happening in each situation. One would overwrite the default configuration and the other would just extend it. It's up to the needs of the user which one he wants to use. Some users prefer overwriting the default and creating their own file hierarchy structure where they define things according to exactly what they want. Some of LazyVim's defaults might not be wanted. So, you just end up overwriting things according to your needs.


nomad_1_0: 1.0.10 -> 1.0.11, nomad_1_1: 1.1.4 -> 1.1.5

Motivation for this change
See commit messages for links to changelogs.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 139435 at 0593b0a4 run on aarch64-linux 1

2 packages built successfully:

nomad (nomad_1_1)
nomad_1_0


[help me]{"id":"go.micro.client","code":500,"detail":"OK: HTTP status code 200; transport: missing content-type field","status":"Internal Server Error"}

sometimes 200 ok,simetimes 500 , I do not known why! I set the content-type,but change nothing,who can help me?

ok,my mistake, just replace micro.NewService with grpc.NewService. the error resolved

What grpc package are you using? I'm dealing with the same issue. Using grpc.NewService didn't resolve it for me.

if you wan help - please provide full example, without code we can't answer

Here is my go mod:
github.com/golang/protobuf v1.3.5
github.com/micro/go-micro/v2 v2.4.0
github.com/micro/go-plugins/broker/rabbitmq/v2 v2.3.0
github.com/micro/go-plugins/registry/consul/v2 v2.3.0

That's how I set up my server:
import (
	"time"

	micro "github.com/micro/go-micro/v2"

	"github.com/micro/go-micro/v2/client/grpc"

	"github.com/micro/go-micro/v2/registry"
	"github.com/micro/go-micro/v2/server"
	consul "github.com/micro/go-plugins/registry/consul/v2"

	"gitlab.com/lootspot/api/misc"
	"gitlab.com/lootspot/api/users/pkg/service"

	opb "gitlab.com/lootspot/api/offers/pkg/opb"
	pb "gitlab.com/lootspot/api/users/pkg/upb"
)

func main() {
	reg := consul.NewRegistry(registry.Addrs("consul:8500"))

	svc := micro.NewService(
		micro.Name("go.micro.srv.users"),

		micro.RegisterInterval(time.Second*10),
		micro.Registry(reg),

		micro.Client(grpc.NewClient()),

		micro.Server(
			server.NewServer(
				server.Name("go.micro.srv.users"),
				server.WrapHandler(misc.RecoveryWrapper),

				server.RegisterInterval(time.Second*10),
				server.Registry(reg),
			),
		),

		micro.Address(":5000"),
	)
...

That's how I set up my client:
import (
	"flag"
	"fmt"
	"net/http"
	"os"
	"os/signal"
	"syscall"
	"time"

	cclient "gitlab.com/lootspot/api/chats/client"
	oclient "gitlab.com/lootspot/api/offers/client"
	pclient "gitlab.com/lootspot/api/payments/client"
	uclient "gitlab.com/lootspot/api/users/client"

	"github.com/micro/go-micro/v2/broker"
	"github.com/micro/go-micro/v2/client"
	"github.com/micro/go-micro/v2/client/grpc"
	"github.com/micro/go-micro/v2/registry"

	micro "github.com/micro/go-micro/v2"

	rabbitmq "github.com/micro/go-plugins/broker/rabbitmq/v2"
	consul "github.com/micro/go-plugins/registry/consul/v2"

	"github.com/gorilla/mux"
	"github.com/rs/cors"
)

func main() {
	var (
		httpAddr   = flag.String("http.addr", ":8000", "Address for HTTP (JSON) server")
		consulAddr = flag.String("consul.addr", "consul:8500", "Consul agent address")
		rabbitAddr = flag.String("rabbit.addr", "amqp://admin:password@rabbit:5672/", "Rabbit broker address")
		//retryMax     = flag.Int("retry.max", 3, "per-request retries to different instances")
		//retryTimeout = flag.Duration("retry.timeout", 500*time.Millisecond, "per-request timeout, including retries")
	)
	flag.Parse()

	// register gateway service, connect to consul and rabbitmq
	reg := consul.NewRegistry(registry.Addrs(*consulAddr))
	b := rabbitmq.NewBroker(broker.Addrs(*rabbitAddr))

	service := micro.NewService(
		micro.Name("go.micro.srv.gateway"),

		micro.Broker(b),

		micro.Client(grpc.NewClient()),

		micro.RegisterInterval(time.Second*10),
		micro.Registry(reg),
	)
	service.Init()

What grpc package are you using? I'm dealing with the same issue. Using grpc.NewService didn't resolve it for me.

emm，I use the last version（not v2）；I had try to use v2 before my problem resolved, but it seems not work ，so, I have to use v1 , all right , it works just like the issue mentioned above;

Removing this part fixed it for me
micro.Server(
			server.NewServer(
				server.Name("go.micro.srv.users"),
				server.WrapHandler(misc.RecoveryWrapper),

				server.RegisterInterval(time.Second*10),
				server.Registry(reg),
			),
		),

use grpc.NewServer  instead server.NewServer
micro.Server( 			grpc.NewServer( 				server.Name("go.micro.srv.users"), 			), 		),


fix removal of pre-existing setting in neo4j.conf

This PR fixes issue #156.
Proposed solution: when removing pre-existing keys in neo4j.conf start search a begin of line.

Unfortunately build is failing for Neo4j 3.3 and 3.5, but as I have no access to TeamCity, I can't see the reason why. Maya someone provide details of build failures?

@lweller dude, I'm so sorry we haven't gotten back to you here. We're reorganizing ownership a bit for the docker stuff and this fell through the cracks of that. This is very nicely spotted, and a nice clean fix; thank you for taking the time to write it up.
If you don't mind, could you sign our CLA - it's super fast, just one email and then you're signed up to contribute to all neo repos - the details are at the end here: https://neo4j.com/developer/cla/
If you do that, I'll rebase and do the rest to get this in

I've rebased my fix on master and agreed to CLA. Hopefully now everything is redy to merge this PR.

Thank you!

@lweller I think there may have been an email hiccup on the CLA - would you mind terribly sending an email agreeing to the CLA again, and double check it goes to<EMAIL_ADDRESS>


Whats wrong with this

Before i use this service is clearly enjoy, now this service is freakin freak lol, I have do reconnect and refuse connection after 10 second when I connect again, not statisfying, so now I use loophole.cloud service now they is enjoy, but the problem is, loophole exposed my local 80 port on curr uri after domain, and this is make sense if use any api I cannot get, post, or anything basically crud or curl method, hmmmm when I have some enjoyable private server without any trouble like this, just need 2 hour for 179 X, I have clicking my up curssor and hit the enter for reconnect on localtunnel again :( oh yeah before my internet speed is 50MB/s (Up1/3Down), just not have Public IP on my internet package that's why I use this service LoL.
This is LoL message error I've see 179x is:
throw err;
^

Error: connection refused: localtunnel.me:44817 (check your firewall settings)
at Socket. (C:\Users\xxx\AppData\Roaming\npm\node_modules\localtunnel\lib\TunnelCluster.js:52:11)
at Socket.emit (node:events:527:28)
at emitErrorNT (node:internal/streams/destroy:157:8)
at emitErrorCloseNT (node:internal/streams/destroy:122:3)
at processTicksAndRejections (node:internal/process/task_queues:83:21)

This may just be a connection hickup, there's a PR open with a retry policy: #311.
Maybe you can upvote it to bring it to the maintainers attention?


Web facet set to 2.4 instead of 3.0 for simple web project

I have a simple web project and I do a 'gradle build' on it and then import it into eclipse as an existing Gradle project.  If I use the 2.1.1-SNAPSHOT the Dynamic Web Module facet is set to 2.4 instead of 3.0.  If I use the 2.1 release then the Dynamic Web Module facet is set to 3.0 as expected.

Fixed by #212


Regression: CoinExtensions.cs

Minor regression inside CoinExtensions.cs , line 20, causes the project not to compile.
Original: ExchangeName[] onlyOnTheseExchanges = null)
New:      ExchangeName[] onlyOnTheseExchanges)
Seems that it should be reverted back to the original unless you had something else in mind.

I changed it again since then.  It's now IEnumerable instead of an array (allowing to pass an array or a list, but we lose the params support).
The examples should be compiling now.  Let me know if there's more issues.
thanks,
nick.


feat: Publish objc_access module

Relates #70
This pull request proposes to make obj_access module public.
It is needed to support the views which are not supported in cacao yet without forking/extending cacao.

Thanks!

Thank you for the quick merge!


Doesn't work for pined tabs

FF 61.0.1 64bit Linux
Doesn't work for pined tabs. And stops working for regular tabs after trying to detach pined tab

FF 62.0, 64bit Windows10
Also doesn't work for pinned tabs.

Fixed: e91a5a2

Great! Thank you! When will it be released?

I uploaded a new version to AMO yesterday, so it should be available right now.


Cannot set/change league.

In settings, the League field is greyed out. Clicking it does NOT reveal dropdown menu. Typing does nothing. Nothing does anything when it comes to interacting with that field.
Probably related, I can't use it to price check - item price refreshes forever.

Mysteriously fixed itself today.
Before opening the issue, I already tried manually updating both in POE Overlay and through Overwolf settings.
But today it seemed to have updated again and UI elements positions have slightly changed. The league was set to Ritual by default and everything works.
Thanks!

Mysteriously fixed itself today.
Before opening the issue, I already tried manually updating both in POE Overlay and through Overwolf settings.
But today it seemed to have updated again and UI elements positions have slightly changed. The league was set to Ritual by default and everything works.
Thanks!


Does this project support compiling on the windows platform? Or whether to support msvc compiler

问题描述
请问该工程支持在window平台编译吗，或者是否支持msvn？

I don't have enough experience to work on Windows. But PRs are welcome.


能不能像datagrip一样可以调整默认运行最小的sql语句

Azure Data Studio Version:

Steps to Reproduce:
能不能像datagrip一样可以调整默认运行最小的sql语句，不是全部，而且有delete这种危险操作没有加where语句时可以提示是否执行

what do you mean by "shortest sql statement"? ADS will only run the selected statement by default.

@sunmicosun

I'm sorry, I find it, just press ctrl + F5
thank you！
在 2019-01-04 10:29:01，"sunmico"<EMAIL_ADDRESS>写道：
I'd like to say this.
just like datagrip
在 2019-01-04 02:21:26，"Alan Ren"<EMAIL_ADDRESS>写道：
what do you mean by "shortest sql statement"? ADS will only run the selected statement by default.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


init() missing 1 required positional argument: 'sessionID'

Hi,
I'm running into an issue while executing the PoC. I use Python 3.8.10, Twisted-22.2.0and installed PyRDP from source (Without GUI and ffmpeg dependencies).
(venv) yassineaboukir@Yassine-2 CVE-2019-0708 % python3 exploit.py victim.com -rp 3389 <my_ip>
Traceback (most recent call last):
  File "exploit.py", line 545, in <module>
    main()
  File "exploit.py", line 535, in main
    exploit = Exploit(reactor, args.rdp_host, args.rdpport, args.backdoor_ip, args.backport)
  File "exploit.py", line 58, in __init__
    self.state = RDPMITMState(config)
TypeError: __init__() missing 1 required positional argument: 'sessionID'

This may be an error in the process of writing exp.
Line 58 in exp passed in
    config = MITMConfig()
    self.state = RDPMITMState(config)

And in pyrdp.mitm.state The RDPMITMStatefunction is defined as follows
class RDPMITMState:
    """
    State object for the RDP MITM. This is for data that needs to be shared across components.
    """

    def __init__(self, config: MITMConfig, sessionID: str):

https://github.com/GoSecure/pyrdp/blob/master/pyrdp/mitm/state.py

Hi,
I'm running into an issue while executing the PoC. I use Python 3.8.10, Twisted-22.2.0and installed PyRDP from source (Without GUI and ffmpeg dependencies).
(venv) yassineaboukir@Yassine-2 CVE-2019-0708 % python3 exploit.py victim.com -rp 3389 <my_ip>
Traceback (most recent call last):
  File "exploit.py", line 545, in <module>
    main()
  File "exploit.py", line 535, in main
    exploit = Exploit(reactor, args.rdp_host, args.rdpport, args.backdoor_ip, args.backport)
  File "exploit.py", line 58, in __init__
    self.state = RDPMITMState(config)
TypeError: __init__() missing 1 required positional argument: 'sessionID'

This may be an error in the process of writing exp.
Line 58 in exp passed in
    config = MITMConfig()
    self.state = RDPMITMState(config)

And in pyrdp.mitm.state The RDPMITMStatefunction is defined as follows
class RDPMITMState:
    """
    State object for the RDP MITM. This is for data that needs to be shared across components.
    """

    def __init__(self, config: MITMConfig, sessionID: str):

https://github.com/GoSecure/pyrdp/blob/master/pyrdp/mitm/state.py

@flag0 I sent the following PR https://github.com/RICSecLab/CVE-2019-0708/pull/2 to fix it - it works fine for me now.
Connected to RDP server
Sending X224ConnectionRequestPDU...
Connection confirmed
Sending MCSConnectInitialPDU...
The server selected 1
Got MCSConnectionResponsePDU
The server selected EncryptionMethod.ENCRYPTION_NONE
rdpdr <---> Channel 1004
RDPSND <---> Channel 1005
MS_T120 <---> Channel 1006
Sending MCSErectDomainRequestPDU...
Sending MCSAttachUserRequestPDU...
Got MCSAttachUserConfirmPDU
[...]

did any one get this to work


Yes button on Level Editor Yes/No prompts does not close prompt

Describe the bug
When using the level editor, pressing "Yes" on a Yes/No dialogue correctly loads the content, but does not close the dialogue
Expected behavior
Close the dialogue
Screenshots and logs
[Error  : Unity Log] IndexOutOfRangeException: Index was outside the bounds of the array.
Stack trace:
tk2dRuntime.TileMap.RenderMeshBuilder.BuildForChunk (tk2dTileMap tileMap, tk2dRuntime.TileMap.SpriteChunk chunk, tk2dRuntime.TileMap.ColorChunk colorChunk, System.Boolean useColor, System.Boolean skipPrefabs, System.Int32 baseX, System.Int32 baseY) (at <9086a7372c854d5a8678e46a74a50fc1>:0)
tk2dRuntime.TileMap.RenderMeshBuilder.Build (tk2dTileMap tileMap, System.Boolean editMode, System.Boolean forceBuild) (at <9086a7372c854d5a8678e46a74a50fc1>:0)
tk2dTileMap.Build (tk2dTileMap+BuildFlags buildFlags) (at <9086a7372c854d5a8678e46a74a50fc1>:0)
tk2dTileMap.Build () (at <9086a7372c854d5a8678e46a74a50fc1>:0)
LevelEditor.BuildAllTiles () (at <9086a7372c854d5a8678e46a74a50fc1>:0)
LevelEditor.RefreshChunks () (at <9086a7372c854d5a8678e46a74a50fc1>:0)
LevelEditor.PressedYesButton () (at <9086a7372c854d5a8678e46a74a50fc1>:0)
UnityEngine.Events.InvokableCall.Invoke () (at <a5d0703505154901897ebf80e8784beb>:0)
UnityEngine.Events.UnityEvent.Invoke () (at <a5d0703505154901897ebf80e8784beb>:0)
UnityEngine.UI.Button.Press () (at <d5bb9c19c2a7429db6c6658c41074b11>:0)
UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at <d5bb9c19c2a7429db6c6658c41074b11>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at <d5bb9c19c2a7429db6c6658c41074b11>:0)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at <d5bb9c19c2a7429db6c6658c41074b11>:0)
UnityEngine.EventSystems.ExecuteEvents:Execute(GameObject, BaseEventData, EventFunction`1)
Rewired.Integration.UnityUI.RewiredStandaloneInputModule:ProcessMousePress(MouseButtonEventData)
Rewired.Integration.UnityUI.RewiredStandaloneInputModule:ProcessMouseEvent(Int32, Int32)
Rewired.Integration.UnityUI.RewiredStandaloneInputModule:ProcessMouseEvents()
Rewired.Integration.UnityUI.RewiredStandaloneInputModule:Process()
UnityEngine.EventSystems.EventSystem:Update()

What operating system are you on?
Windows 64-bit
What version of the game are you running?
v95
What version of RogueLibs are you running?
v3.x.x
Installed mods
RL, SpritePackLoader, BTHarmonyUtils, CCU, DemolishThatFreakingWall

RogueLibs v3 will no longer be updated. Reopen the issue if the issue persists in v4+.


Bootloop after reboot to recovery

Hi, first, thank you for your great work on GammaOS.
Today, I was preparing to upgrade my RG405 GammaOS to version 1.3 (was on 1.2). Then, follwing the instructions, I rebooted it to recovery ( hold down the power button and choose the Restart option, then choose the Recovery option.). Then it started bootlooping on Ambernic Logo and the usual messages are shown:
INFO: LOCK FLAG IS: UNLOCK!!
Warning: Lock Flag  is: Unlock, Skip Verify!!!
I recorded a video: https://www.youtube.com/watch?v=FJmCkqdZDNo

I had to flash back the original firmware using Black-Seraphs.
Now its working again but I'll have to configure everything again.

Hi, sorry about this. This may be related to your SD card, if you see this issue again, try ejecting your SD card.

Nice! Never thought it could be the SD card. Next time, should I remove the SD card before try to update?

Yes that would be the workaround

Hi, I have an rg505 same boot loop owing stop vibrating and screen flashing.
There was no sad card inside mine when it started.
Any help with re install firmware using a Mac? Unfortunately I have available


Use buffer not string

Lots of calls to getText on atom editor. Also our languageServiceHost specifically ScriptSnapshot is working on string. Make it use nodejs Buffer. Also do this when reading files etc.

Blocked by : https://discuss.atom.io/t/getting-changes-in-textbuffer/14581

Debounce linter further
Use on did change to just copy edit buffer.
And Debounce errors further
It is just might make autocomplete work as it is.

Debounce linter further
And Debounce errors further

Not needed.

Use on did change to just copy edit buffer.

Investigating.

It seems like this is not the issue. The program.languageServiceHost.updateScript(filePath, options.editor.getText()); call only takes under 1ms on a 1000 line file (considered a largish file for most purposes).
However the complete call to requestHandler in autocomplete plus is around 100ms

With getCompletion taking the majority of time.

So moving this to the wishlist category.

TODO: investigate if getCompletions makes calls to the language service host and consider optimizations if that is what is taking too long.

Probably a good thing to do after https://github.com/TypeStrong/atom-typescript/issues/130

We are not going to use a buffer. Instead we will bring in the changes from TSServer


Autofix via documentFormattingProvider

Commonly, LSPs provide formatting capabilities (documentFormattingProvider).
For typos, it could be useful be able to autofix misspellings via LSP-formatting. Essentially what typos --write-changes does, only triggered from inside the IDE.

Heya, I hadn't thought of this because typos is often used with source code, which typically has its own document formatter. So in most cases I would expect the default formatter to be specific to the file type (eg: prettier for javascript). I can't think of a use-case or file type where you would want to use typos instead of its source code formatter?

No, not using typos instead of a source code formatter, but rather using typos in addition to the source code formatter. Not familiar with other editors, but in nvim, when I run vim.lsp.buf.format(), all attached LSPs that have formatting capability will run their formatting

Ah thanks for the elaboration. You are using vim. I use vscode which AFAIK doesn't have an "apply all formatters" command, so I wasn't aware this was a thing!
Because of the difference between editors, I don't think I'd want this enabled by default, but I can see a new config option that if set would offer the documentFormattingProvider. I presume you'd want this enabled for all file types?

Yep, enabling it via setting should be good


Add metric package containing Gauge support

@bogdandrutu PTAL
I also removed the metricexporter.Registry to free up the name "Registry". I plan to incorporate its functionality into metric.Registry when I submit the stats-to-metrics PR.


Plugin E2E: Pass testinfo to models

What this PR does / why we need it:
The Playwright testinfo class contains information about the currently running tests. It's passed as the second argument in tests, and the third argument in fixtures. Among many other things, testinfo allows you to conditionally fail a running test.
Today, all APIs of plugin-e2e are backwards compatible to Grafana 8.5. This is great, but I'm afraid we won't be able to commit to that for every new feature we build into plugin-e2e in the future. If for example we'd like to add support for testing Correlations at some point, we can't keep backwards compatibility as this feature was introduced pretty recently. We then need some way communicate to the user of plugin-e2e that those APIs don't work for the current Grafana version.
This PR injects the testinfo to all models. Even though we don't need it today, it will give us more flexibility when adding support for new APIs in the future. Adding testinfo to the models is a breaking change, so I rather do it now than after we've released 1.0.
Here are a few examples of how it can be used:
export class DataSourceConfigPage extends GrafanaPage {
  async addCorrelation() {
      this.ctx.testInfo.fail(
        semver.lt(this.ctx.grafanaVersion, '10.0.0'),
        'Correlations are not supported in this version of Grafana. Please skip this test for this version of Grafana.'
      );
      ...
  }
}

const createDsReq = await request.post('/api/datasources', datasource);
  if (!createDsReq.ok()) {
    this.ctx.testInfo.fail('Could not create data source');
  }

If we don't have a way to force a test to fail, it will fail on something else with possibly weird error messages that we don't control. We can write to the console, but that easily gets lost in the test report.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

I guess this will cover the scenario where a plugin developer is conditionally providing functionality in tier plugin based on Grafana version. So for me this makes sense. The only thing is that I think we should fail the test and let the plugin author (who writes the test) add logic to skip the test since they have added logic to conditionally use the feature/functionality.

Yep totally agree!


Added game FruitCatcher

PR Description 📜
Added a new game
Fixes #4159

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @VijaySamant4368 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit please merge this pr

Thank you  @VijaySamant4368 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @VijaySamant4368,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


Configurable service ip range

Add the ability to configure your kubernetes service ip range
Sorry, this is dependent on the previous PR

Will resubmit this with master changes merged in.


Enable reactions-avatars on more pages

Fix #3978
Test URLs

Discussion: https://github.com/tophf/mpiv/discussions/50
Release: https://github.com/sindresorhus/refined-github/releases/tag/21.9.3

Screenshot

The problem is that hasComments automatically uses its own listener, you can see it in features/index.tsx
It's probably best to create 2/3 inits. One for hasComments, one with the observer for the discussions, one static for the releases

The problem is that hasComments automatically uses its own listener, you can see it in features/index.tsx

I forgot that

It's probably best to create 2/3 inits. One for hasComments, one with the observer for the discussions, one static for the releases

That's what I did initially, I just thought this is simpler


Implemented settings from previous installation initializes settings for new installation

If Notepads is installed under a Microsoft account, the settings is synced with this account. For a new installation under this account the default settings value will be from these synced settings
PR Type
What kind of change does this PR introduce?
Feature

I thought this roaming thing is going to deprecated, can you confirm?

I thought this roaming thing is going to deprecated, can you confirm?

This article says it is deprecated but the functionality isn't removed. Should I close this or will you implement it now and remove this when roaming is removed??

I thought this roaming thing is going to deprecated, can you confirm?

This article says it is deprecated but the functionality isn't removed. Should I close this or will you implement it now and remove this when roaming is removed??

Let's not implement it since it is scheduled to be discontinued. It will cause confusion in the future.


Question about masking rate and missing rate

Hi, thank you for presenting such a nice paper and project! Since I'm not familiar with tsl code format, I have several questions about the missing rate and masking rate settings in your paper:

In METR-LA point missing case, you randomly drop 25% of the original data to construct an incomplete dataset (this is controlled by the 'p_noise' parameter). Then during traning stage, more missing data is generated for each input batch by setting 'whiten_prob' parameter. I'm confused about whether the 25% of missing data exists in the training data? If that is the case, the input contains missing data generated from both 'p_noise (25%)' and 'whiten_prob (e.g., 80%)' , which leads to a very sparse traning set. Is the loss only computed on the 'whiten_prob'  parts during training?
If I want to train the model with training data only contains 'whiten_prob', and test it on different 'p_noise' levels, like the settings in Tab. 2 of your paper. But the difference is that, I do not add 'p_noise (25%)' to training data and would like to add different levels of 'p_noise' data to testing data, how could I achieve this goal based on your code?

Thanks in advance for your help!

Hi, thanks for your interest in our work! See inline.

I'm confused about whether the 25% of missing data exists in the training data?

I just wanted to let you know that you understood the masks correctly. While in principle you can consider as valid missing observations that 25% of injected missing data in the training set, we simulated the case that the underlying data-generating process is affected by this missing rate, not only at test time. Given the difficulty in evaluating imputation performance, this sounded like a nice solution to us, since starting from a dataset with few (really) missing data you can then test an imputation algorithm on a precise missing-data distribution.

If that is the case, the input contains missing data generated from both 'p_noise (25%)' and 'whiten_prob (e.g., 80%)' , which leads to a very sparse traning set. Is the loss only computed on the 'whiten_prob' parts during training?

Yes, it is a very sparse training set, but it is required for our algorithm as it is only trained on the points masked out following whiten_prob. Still, test performance is comparable to SOTA and makes SPIN more robust to missing-data distribution shifts.

If I want to train the model with training data only contains 'whiten_prob', and test it on different 'p_noise' levels, like the settings in Tab. 2 of your paper. But the difference is that, I do not add 'p_noise (25%)' to training data and would like to add different levels of 'p_noise' data to testing data, how could I achieve this goal based on your code?

You can do it by manually changing the masks, e.g., by setting an all-valid mask during training and then updating the masks as done here:
https://github.com/Graph-Machine-Learning-Group/spin/blob/2320695ff03b23606e73b05ac87f3ddff9d74c0c/experiments/run_inference.py#L138-L147

Thanks for your reply ! Your suggestions are very helpful !


Migrations (arrow, geos, libarchive, openssl)

Saw this feedstock due to https://github.com/conda-forge/heavydb-ext-feedstock/pull/32 (I'm following the migration a bit).
Strictly speaking the migrator doesn't make sense here (because arrow is pinned anyway), but I thought why not bump arrow here while we're at it.
Also, this recipe has an old openssl pin that should be removed, and there are two other open migration PRs I thought I'd pick up (the migration for https://github.com/conda-forge/heavydb-ext-feedstock/pull/25 closed already, so it comes through the pinning).
Builds on #31
Closes #32
Closes #31
Closes #25

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

@conda-forge/heavydb-ext PTAL :)

After downgrading to arrow 9.x, this is now blocked on https://github.com/conda-forge/arrow-cpp-feedstock/pull/900

@pearu
This should be good now, PTAL. :)


Change charging amps

Hi
Sorry if this in already implemented, but I cannot work out how to do it.
I see we have control to start and stop a charge, but is there a way to change current charge amps? #

Please search the issues. This has been asked and answered multiple times.

Oh sorry I didn't see it initially. But now I see it here https://github.com/alandtse/tesla/issues/86


Use dev.fuzz branch for fuzz tests

This change replaces go-fuzz with Go fuzz tests from dev.fuzz branch.
Also now it should be easier to add new fuzz targets in any package. To generate a seed corpus
# Build Go from dev.fuzz branch.
go install golang.org/dl/gotip@latest && gotip download dev.fuzz
# Fuzz for 1 hour, adding interesting inputs to testdata/corpus.
gotip test -v -run=^Fuzz -test.fuzztime=1h -test.fuzz=. -test.fuzzcachedir=testdata/corpus

Alternatively, it’s possible to call (*testing.F).Add in fuzz target to add existing input at runtime (e.g. from testdata for other tests).
Closes #397

@jhchabran yes, I’ve intentionally committed seed corpus. IIRC I built that manually by extracting strings from tests cases. It serves as starting point for fuzzer, and go test runs these inputs as regular tests if fuzzer is not enabled. If fuzzer finds a crasher, it’ll also write it to testdata/corpus/{{ fuzz function name }}/{{ sha256(file data) }}.
The CI workflow runs tests and fuzzer for 5m on each commit (plus ~3m to build gotip). This setup was good enough to find crashers when we’ve added initial fuzzing configuration. The downside is that each time the fuzzer starts from scratch, and sometimes 5 minutes may not be enough.
Then there is genjidb/go-fuzz-corpus repo that would run fuzzer for 5h on a daily schedule. That potentially generates a lot of interesting inputs we’d like to keep in corpus but Git and some filesystems don’t like lot’s of small files in a single directory, and that would also clutter the main repository. Currently they are stored in GitHub Actions cache, but ideally should be committed to go-fuzz-corpus repo (that should be possible with GitHub token permissions from scheduled workflow run).
Note that, unlike go-fuzz, Go tooling keeps interesting inputs somewhere in GOCACHE and doesn’t add them to testdata directory, but there is a fuzzcachedir flag that does that.
See also https://golang.org/s/draft-fuzzing-design

Thank you for the detailed explanation, it's much clearer now!


The Nucleus servers gets deployed on a public subnet instead of private and issues with Nginx setup

"/usr/share/nginx/html/omni/web3/index.html" is not found (2: No such file or directory), issue due to nginx deployment using stcack. Also the stack deploys Nucleus server on public subnet instead of private subnet.

Hi @anandtge are you able to share more information about the nginx error you are experiencing?

Hi @anandtge, it looks like nginx on the reverse proxy did not configure correctly. Could you try terminating the reverse proxy instance and allow the auto scaling group to launch a new one?
This process may take 15-30 minutes. Please let me know the results of this test.

Closing due to inactivity.


quad_form_sym: reverse specialization does not return symmetric matrix

Description
quad_form_sym should return a symmetric matrix. See https://mc-stan.org/docs/2_24/functions-reference/dot-products-and-specialized-products.html
For reverse mode, there are instances where it can return a non-symmetric matrix. The primitive version for the same values of matrices results in a symmetric matrix.
Example
pseudocode, but pulled from C++ example
A =  1.25882993696386514 -0.03325949401909023
-0.03325949605426004  1.85523447220884385
B =  2.55474619740069508  0.66362927717720988
-1.92917223922387349  2.01853255256721731

M = quad_form_sym(A, B);
M(1, 0) - M(0, 1) = 1.31006e-08 // this should be 0 and is 0 for primitive

Expected Output
M(1, 0) - M(0, 1) = 0

Current Version:
v3.3.0

I'm working on branch bugfix/issue-2095-quad_form_sym and something is off.
In the primitive version, it's not picking up that the input matrix is not symmetric, but it is for the reverse version. I'm going to dig into check_symmetric.

thanks, @kmckee90, for providing an example that triggered this behavior!


Fix some static code analyse errors with PHPStan level 1

Hi,
i've run PHPStan over the 4.x branch and found a lot of wrong Interface usages.
For the first step i fixed some simple errors

Coverage decreased (-0.1%) to 97.653% when pulling 3769e1e570e0cf2abf87b06993b80fefd9090558 on marcelthole:fix-static-code-analyse-errors into 5b5675451ae78eab35ca5c58fb4a24bb547fcfb6 on slimphp:4.x.

oh sorry. There is already a phpstan PR #2505 - my fault

PHPstan support has now landed. Thanks for looking at this though.


Fixed isn't a Semiring

Quickcheck reveals the following properties don't actually hold:

Associativity of multiplication
Left and right distributivity

We might want to consider dropping Semiring and only providing a (commutative) additive Monoid. Multiplication would have to be provided as a function, just like we do for division.
We could perhaps provide an unsafe Semiring around a newtype for compatibility purposes, but I doubt many people are using this library anyway.

On reflection I'd actually prefer to have Fixed keep its Semiring instance, and even all of its instances up to EuclideanRing and perhaps even DivisionRing. Those instance wouldn't be any more dodgy than the same instances for Number; at least Fixed is a genuine commutative monoid under addition, unlike Number. We should just document the circumstances in which it fails to satisfy the laws.


Core Forgers Network Impl

Core Forgers Network:

PeerConfidence.Forger
Peer limits and logic for Forger Network
Prioritized connection
Prioritized broadcasting

Simple Flag:

send a boolean flag in the handshake message to tell that the node is a forger

I'd also like to give you some food for thought that are not request changes: we're introducing now the name forger inside Sparkz, which at its level should mean nothing. So, what do you think, instead of forger, to use another name to denote special treated entities? Like TopTier, VIP, Elite, the GOATs (jk); this way we introduce the concept of elite peers without calling them forgers. But as I said at the beginning, if you think that's fine, I won't object

We discussed this a bit, and the winning argument was that we would like to import Sparkz into SDK at some point in the future and get rid of unnecessary generalization, as it would only confuse people working with it, until they discover that this special node is in reality just a forger.


Serde derive in tests

Manual parsing Json is tedious and error-prone. It contained a bunch of
unwraps and was hard to read.
This replaces manual Json parsing with serde_derive implementation.
 I guess my machine didn't even get to compile tests. I suspect I just have full RAM but no time to free it up rn.


Add Python program to print 1 to 100

This is a one-liner python code to print 1 to 100 without using numbers.

Your file name is something else and you have updated contributors file. That's invalid!


feat(util): use relative url instead of hardcoded

Make this plugin compatible with all sorts of URL rewrites.
graffiti currently expects a /graphql endpoint absolute to the current URL.
This patch fixes that, so that graffiti can be used with spooky URL rewrites.
This also theoretically enables to rename /graphql without breaking graphiql (I think)

Thank you! I'll release soon.


Updated product name.

Changed "Apps" to "App" to reflect actual product name.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@jamesche75 : Thanks for your contribution to the Azure documentation! The author, @naziml, has been notified to review your proposed change.

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

@jamesche75 This is already fixed in the most recent version. Thanks! #please-close

@jamesche75 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


hidedecorations! is comparatively slow

Hi! @SimonDanisch recommended I raise an issue here
begin
       using CairoMakie
using TimerOutputs
	t =TimerOutput()
	fig = Figure()
	ax = fig[1,1] = Axis(fig)
	@timeit t "heatmap" heatmap!(ax,1:500,1:500,rand(500,500))
	@timeit t "hidedecorations" hidedecorations!(ax)
	fig
end

results in longer time for hidedecorations! than the actual heatmap. (hidespines! is super fast though). Any idea what is going on?
 ──────────────────────────────────────────────────────────────────────────
                                   Time                   Allocations
                           ──────────────────────   ───────────────────────
     Tot / % measured:          280ms / 23.0%           57.5MiB / 38.3%

 Section           ncalls     time   %tot     avg     alloc   %tot      avg
 ──────────────────────────────────────────────────────────────────────────
 hidedecorations        1   38.3ms  59.4%  38.3ms   12.7MiB  57.8%  12.7MiB
 heatmap                1   26.2ms  40.6%  26.2ms   9.31MiB  42.2%  9.31MiB
 ──────────────────────────────────────────────────────────────────────────

hidedecorations! sets 12 attributes which all change the layout variables of the Axis, so because of the immediacy of the Observables system you get 12 relayouts per Axis, and 300 for all Axes combined. Because each relayout triggers some computations again for each affected Axis, that becomes super slow. I will do some testing but the remedy will probably be to block layout updating during this function call. This can of course just prohibit 11 of the 12 updates, so you'll still get 25 updates overall, but that should still be much better. Additionally, we can check what makes relayouting itself slower than it needs to be, there are probably low hanging fruit there also.


Add back-ticks in AzurePipelinesCredential section of the Identity TSG

Without these, the text within < > doesn't render.

API change check
API changes are not detected in this pull request.


Add Outplayer

Awesome! Thanks for your contribution!

Any idea when I'll be able to see this in an Opener release?

@Enzime right now! You'll see it if you open the app.

awesome 🎉


requests package issue in pip10 version

When I try to uninstall requests package using pip10, below error I am getting.
Cannot uninstall 'requests'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.

That's interesting given that requests uses setuptools and has for years now. It sounds like the version of requests you have must be incredibly old.

Yes, It works in the latest version of requests==2.18.4.


I made a change myself that caused this.


Build failure: lime3ds

Steps To Reproduce
Steps to reproduce the behavior:

build lime3ds-2118.2

Build log
log
Notify maintainers
@arthsmn
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
hive@library : ~ ඞ
» nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.10.9-zen1, NixOS, 24.11 (Vicuna), 24.11.20241009.5633bcf`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.8`
 - channels(root): `""`
 - nixpkgs: `/nix/store/60sn02zhawl3kwn0r515zff3h6hg6ydz-source`

Add a :+1: reaction to issues you find important.

This is a problem with the ffmpeg version, the latest version in nixpkgs is 7.0.2, too new. I'll open a PR to downgrade to ffmpeg_6, this fixes the build.


Load FMU with Femyou not working

Hello!
I have a program in C#, running on Visual Studio 2019, with target framework netcoreapp3.1.
I used Open Modelica v1.22.3 (64-bit).
My code:
using System;
using System.Collections.Generic;
using System.Text;

namespace TesteFMU
{
    public class Program
    {
       public static void Main(string[] args)
       {
          TestFMU test = new TestFMU();
          test.Calculation();
          Console.Write("Calculation concluded.");
       }
    }

using System;
using Femyou;
using System.IO;

namespace TestFMU
{
    public class TestFMU
    {
        public void Calculation()
        {
            string fmuFolder = File.ReadAllText(@"C:\FMUTests\");
            using var model = Model.Load(Path.Combine(fmuFolder, "TestFemyou.fmu"));
            using var instance = Tools.CreateInstance(model, "test");
            var x = model.Variables["x"];
            var y = model.Variables["y"];
            var z = model.Variables["z"];
            Console.WriteLine("x = " + Convert.ToString(x));
        }
    }
}

However, when I try to load the FMU, it shows this error:

System.IO.FileNotFoundException: 'Could not find or load the native library: C:\Users\10376\AppData\Local\Temp\Femyou\TestarFemyou.fmu\binaries\win64\TestarFemyou.dll

I don't understand why it tries to load a native library and why it fails despite the DLL being available when I manually check the folder.
Could it be a compatibility issue?
The example BouncingBall.fmu works just fine.
Thank you!

My best guess here would be to check the dependencies of TestarFemyou.dll and make sure that they are all included in TestarFemyou.fmu
Missing dependencies is a common case of dll loading error.
I have not used MS Windows for a long time but, if I remember correctly, a free tool such as DependenciesWalker could help : https://dependencywalker.com/


Fixed an issue with the Guid hashcode

The hashcode for a GUID should be the same across instances if the underlying value is the same. This behavior is inline with that of C# where separate GUIDs constructed with the same value generate the same hashcode
The current hashcode implementation represents the identity of the object but is not tied to the value
Updated the hashCode property to leverage existing Uuid mechanisms for hashcode generation
This fix allows for the Guid type to be used as Map keys with the desired/expected behavior
Updated unit tests

@arakurk @mbelie please review :)


[security] Clients connect to server without using user/pass

Why I need to set the property "ALLOW ANONYMOUS PROPERTY_NAME" even by setting the authenticator?
I ended up taking time to identify the problem. This could be set to false when set an authenticator

There are certain use cases where you might want to allow anonymous user connections. By default it's been set to true. If you wish to disallow anonymous users, set allow_anonymous to false in your moquette configuration file.
This is well documented in here

Understand,
I just found it a bit strange this behavior.
Thank you for the explanation.


cx_Freeze kivy application

Is it possible to use cxfreeze on a kivy project to turn it into an .exe?
If so, is there a tutorial on how to correctly compile it?
If not, is there an alternative way to compile a kivy project to a Windows executable?

There are no kivy docs for cx_freeze usage, but you might have luck adapting the pyinstaller one. http://kivy.org/docs/guide/packaging-windows.html
You could use pyinstaller though, or nuitka, but the latter is largely untested.
https://github.com/kivy/kivy/wiki/Packaging-Kivy-apps-written-in-Python-3,-targeting-Windows-using-Nuitka


Analyzer: Ensure to use Microsoft.VisualStudio.TestTools.UnitTesting.DescriptionAttribute and not System.ComponentModel.DescriptionAttribute

It is very easy to use  System.ComponentModel.Description instead of a Microsoft.VisualStudio.TestTools.UnitTesting attribute.
Having an analyzer which ensure that on a TestMethod we use the right attribute will be good.

Yeah good idea!

This is great. I was hit by them same just this week.

awesome, cannot wait for next release,
Thanks @engyebrahim  and @Evangelink  !


Generate tool config

Summary

This PR adds support for generating a tool config whenever a user adds a new function
Depends on
Test plan

Screenshots

Checklist

[x] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

Thank you for the pull request, @homanp! Here's my review:

In the SuperagentAgent class, the _get_tools method is modified to return an empty list instead of [ChitChatTool]. Is this intentional? If so, please provide a brief explanation in the code comments.
The get_agent method in the SuperagentAgent class is currently empty. Could you please provide an implementation for this method?
In the create function in libs/superagent/app/api/tools.py, there is a new asyncio.create_task call to run the run_generate_tool_config function. This seems fine, but please make sure to handle any exceptions that may occur during the execution of run_generate_tool_config.
In the list function in libs/superagent/app/api/tools.py, there is a loop that checks if tool.toolConfig is a dictionary and converts it to a JSON string. Please make sure this conversion is necessary and handle any exceptions that may occur during the conversion.

Overall, the changes look good. Thank you for your contribution!


Error on detection layer~ can u fix it?

Traceback (most recent call last):
File "D:/Keras-RFCN-master/Fashion_Test.py", line 138, in
model_dir=os.path.join(ROOT_DIR, "logs") )
File "D:\Keras-RFCN-master\KerasRFCN\Model\Model.py", line 42, in init
self.keras_model = self.build(mode=mode, config=config)
File "D:\Keras-RFCN-master\KerasRFCN\Model\Model.py", line 184, in build
[rpn_rois, classify_output, regr_output, input_image_meta])
File "C:\Anaconda3\lib\site-packages\keras\engine\topology.py", line 619, in call
output = self.call(inputs, **kwargs)
File "D:\Keras-RFCN-master\KerasRFCN\Model\Model.py", line 836, in call
self.config.IMAGES_PER_GPU)
File "D:\Keras-RFCN-master\KerasRFCN\Utils.py", line 620, in batch_slice
output_slice = graph_fn(*inputs_slice)
File "D:\Keras-RFCN-master\KerasRFCN\Model\Model.py", line 835, in
lambda x, y, w, z: refine_detections_graph(x, y, w, z, self.config),
File "D:\Keras-RFCN-master\KerasRFCN\Model\Model.py", line 722, in refine_detections_graph
rois, deltas_specific * config.BBOX_STD_DEV)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\ops\math_ops.py", line 894, in binary_op_wrapper
return func(x, y, name=name)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\ops\math_ops.py", line 1117, in _mul_dispatch
return gen_math_ops._mul(x, y, name=name)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\ops\gen_math_ops.py", line 2725, in _mul
"Mul", x=x, y=y, name=name)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 2958, in create_op
set_shapes_for_outputs(ret)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 2209, in set_shapes_for_outputs
shapes = shape_func(op)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 2159, in call_with_requiring
return call_cpp_shape_fn(op, require_shape_fn=True)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\common_shapes.py", line 627, in call_cpp_shape_fn
require_shape_fn)
File "C:\Anaconda3\lib\site-packages\tensorflow\python\framework\common_shapes.py", line 691, in _call_cpp_shape_fn_impl
raise ValueError(err.message)
ValueError: Dimensions must be equal, but are 1000 and 4 for 'mrcnn_detection/mul' (op: 'Mul') with input shapes: [1000], [4].

This error looks strange. But I will check it.
Would you provide more information?

Did you use the project on your own dataset?

Your Config?

Hi @parap1uie-s ,
I am also getting the same error in Fashion_Test.py (Dimension must be equal)
I have used my dataset and Config is same as provided.
Also, is there a way of saving the model with the weight instead of saving just the weights.

Bug fixed, issue close.

@parap1uie-s, Have you added the feature to save a model with its weights?


Implement consumer group patch (reset offsets) "dry run" option

Implement consumer group patch (reset offsets) "dry run" option required by #960

@hemahg , this should give you what is needed for dry-run in #960 . Simply pass "dryRun": true in a "meta" consumer group patch object. You will receive a response with the array of offsets, just like is returned from the consumer group GET operation.

Thanks @MikeEdgar, I will implement the dry-run page on top of this PR


Increased Test Coverage for UI elements

Issue Description
What did you expect to see happen?
The test coverage of RAVEN should be above 80%
What did you see instead?
The test coverage is currently less than 80% as none of the UI components are currently being tested by the regression system.
Do you have a suggested fix for the development team?
The regression test system should at least load and exit the UI elements so that we can test parts of these files. This could be done as a proof of concept first and then followed up with more robust testing of the UI components until full coverage is reached.
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.
No error.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[ ] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

This has not been addressed on the appropriate test machine since the code coverage operates on an headless machine. It is possible to redirect the display to a virtual framebuffer using something like Xvfb, however currently this coverage script is running on a MacOS machine. The reason this is relevant is because Qt uses the native painting engine in MacOS and it is not trivial to force Qt to use X11 via XQuartz. If we are able to change the coverage test to run on a linux machine I can forward the display using Xvfb.


Issue about slot

Describe the bug

Expected behavior
render out the slot in markdown file
Which version do you use:
"@vuese/markdown-render": "^2.2.7"
"@vuese/parser": "^2.1.6"

Describe the bug

"

"

<template>
  <div>
    <template #searchSlot>
      <!-- search slot -->
      <slot name="searchSlot"></slot>
    </template>
  </div>
</template>

same issue🤕

Duplicate of #119

will be fixed by #161


Histogram triggers JS error when zooming in/out and changing number of buckets

Context
I was running some smoke tests.
Steps to Reproduce

Create a map and add a layer with some features.
Add an histogram widget (you can select cartodb_id as the column)
Change the number of buckets, zoom in an out and change number of buckets in between.

Current Result
There's a JS error in the console:
Uncaught TypeError: Cannot read property 'max' of undefined
    at N.d._getMaxValueFromBinIndex (common_editor3.js:12)
    at N.d._calculateDataDomain (common_editor3.js:12)
    at N.d._generateFillGradients (common_editor3.js:12)
    at N.d._setupFillColor (common_editor3.js:12)
    at N.d._onChangeData (common_editor3.js:11)
    at s (vendor_editor3.js:38)
    at r (vendor_editor3.js:38)
    at m (vendor_editor3.js:38)
    at N.d.k.trigger (vendor_editor3.js:38)
    at N.d.set (vendor_editor3.js:38)

I found this error in track.js too -> https://my.trackjs.com/messages/filtered?hash=238bd01efd1edeab
Expected result
Histogram works like a charm and there're no JS errors.
cc: @noguerol @xavijam

I feel this ticket should go in deep-insights...

Yes, this is definitely a deep-insights.js issue.

But, I guess we can leave it here since it's linked to the smoke tests spreadsheet.

Well, up to you, but take into account this problem is not related with any smoke tests pass, it is happening right now in production.

Yes, I tried in production too and it's also failing.

Fix already merged, PR created to update dependencies.

Deployed!


remove seed from client builder and wrap build client in build faucet

This kind of refactor is critical.   I am very happy about this.


Issue 1547: table view for vividus

https://github.com/jdi-testing/jdn-ai/issues/1547

Not implemented due to the project being mothballed and lack of time for development:

LocatorsTreeColumnView:
- The general horizontal scroll functionality has not been implemented for the table view of locators (LocatorsTreeColumnView). As a temporary workaround, individual horizontal scrolls for each locator have been added, which are activated on hover.
- The spacing between elements is not PixelPerfect as per the design mockups.

Automatic Disable: The automatic disable functionality for the PageObjGenerationSettings page has not been implemented.

Test Updates: The tests will need to be updated since some of them started failing after changes to the interfaces.