decorator shows in async functions but not sync ones.

https://apiview.dev/Assemblies/Review/ee993c29a21746d5b3d98d496b538f93/5d0c16d86dc94dd5a85a8d8fe320014d?diffRevisionId=ab94b712f31a4736a71f6590bb236903&doc=False&diffOnly=False#azure.identity.aio.VisualStudioCodeCredential
We show @log_get_token_async for aio.VisualStudioCodeCredential.get_token(). But it does not show for sync one.

I think the issue is not sync/async but the fact that the sync decorator takes arguments and the async one does not.


feature request: "Browse_Media"

Hi
first of thanks for this awesome integration.
it would be great if the "browse_media" function could be implemented
clicking this on the native media_player integration opens HomeAssistant's media section (c.f. screenshot below -- not my screenshot). Would be great to have this in media_player.template as well

thanks again

I also want this feature to be implemented, but I guess it can take some time to make it, so I found a workaround of this problem. You can combine media_player.template with Universal media player integration to take advantage of the features of both.
Just use your media_player.template entity as a first child in Universal media player and make sure to properly override browse_media_entity and play_media command to the media player, that supports it.
Here is a small code snippet (media_player.android_tv_template is an entity from media_player.template integration):
- platform: universal
  name: My Universal Media Player
  unique_id: my_universal_media_player
  device_class: tv
  children:
    - media_player.android_tv_template
    - media_player.android_tv
  browse_media_entity: media_player.android_tv
  commands:
    play_media:
      service: media_player.play_media
      target:
        entity_id: media_player.android_tv
      data:
        media_content_id: "{{ media_content_id }}"
        media_content_type: "{{ media_content_type }}"


Implement too-many-stars bot

@iakunin the puzzle #52 is still not solved.

@iakunin the only puzzle #52 is solved here.


Non positive definite matrix (posterior of weights)

Issues happened due to extremely small/large scaling parameter. The same issue happens in SVR in sickout-learn.
Added try-catch that identifies to user that issue comes from scaling parameter.


Various console warning fixes

Change List

Bump mjolnir.js version - v2.1 fixes a console warning about passive listeners
Fix a console warning about missing component name

Coverage increased (+0.002%) to 62.289% when pulling 4cce5b03b97a1fd06676b1e2032447136f1c87d0 on x/warnings into 8802a6884a89a7f7a104bc7f642d7dea04c17294 on master.


Test Case metadata or something similar

Greetings,
I am wondering, if there is better way to add "metadata" to test case for later evaluation/parsing.
Right now I am forced to provide own test message and in it use some own format (json?) for mandatory information, as measured values and such.
I am looking for something like [pseudo code]
Test Case Board Voltages
    [Documentation]    Main supply voltages according to 1.5.3 section of document EAS5161 test requirement
    ${v3V3} =     Library.Measure      something
    ${result} =     test value somehow
    RECORD     "1.5.3 3V3 Power Supply"    ${v3V3}    ${result}
    ${v5V0} =     Library.Measure      something
    ${result} =     test value somehow
    RECORD     "1.5.3 5V0 Power Supply"    ${v5V0}    ${result}
   ...

And such.
Then look into XML output and be able to find these record and parse them out easily, with known format provided by RF, instead of I do it somehow by appending to test message, which could lead and leads to bloated test messages.

You can use Listeners (link).

v3 also allows you to read and modify the attributes of the test case

So you can edit the tests after they have run for example, so that the data you need is only in the output.xml file.
Or you can write it into other files, or do anything you want.

We are planning to add metadata to tests in RF 7.2 (#4409).


reduce healthcheck permission

What this PR does / why we need it:
reduce healthcheck permission on limited resources
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/retest


Use Inheritance So You Don't Repeat Yourself

Describe your problem and - if possible - how to reproduce it
The challenge description and code make reference to Dog & Bird (Bear and Cat) being sub-types of Animal, but there's no description or code to actually do that so one can be left wondering "How did that happen?" Furthermore, the code example won't actually run if done stand-alone since there Bear & Cat aren't actually sub-typed from Animal.
I think it would be better to include the sub-typing so it's clear what's going on
Add a Link to the page with the problem
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/object-oriented-programming/use-inheritance-so-you-dont-repeat-yourself
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: Version 69.0.3497.100 (Official Build) (64-bit)
Operating System: MacOS 10.13.6

If possible, add a screenshot here
Taken after the code changes to meet the challenge...

Hi,
I agree that the code doesnt make a lot of sense and example standalone doesnt work.
Though, these questions are answered in the next challenge when we connect the prototypes object for delegation inheritance.
What do you think of adding a note about the code being incomplete and the function constructors arent connected yet.

Closing as stale. Feel free to reopen and provide a suggestion here anyone.


Automated cherry pick of #28763: Fix default disk type for Azure

Cherry pick of #28763 on release-1.17.
#28763: Fix default disk type for Azure
For details on the cherry pick process, see the cherry pick requests page.

/assign @tengqm

Hi @ialidzhikov
You've opened the same cherry pick 4 times, so I'm going to close the other 3 first.

Hi @ialidzhikov
You've opened the same cherry pick 4 times, so I'm going to close the other 3 first.

What is wrong with you? They are for different release branches…

/lgtm
/approve


Prompting for LWC code returned an inscrutable error

How can I create a LWC component to surface a user record with small user photo

returned:
lua
nmp init --type=module

restating the question returned actual code:
How can I create a lightning web component to surface a user record with small user photo

returned:
<template>
  <template if:true={user}>

{user.Name}

{user.Email}

{user.Phone}

  </template>
</template>

@snugsfbay Thank you for posting the issue. Our current LLM is not yet ready to generate LWC code, so it behaves oddly when prompted for LWC generation. Please stay tuned. We'll release a new version once it excels at LWC completion.

@floralan I appreciate that, yet I was told in the Community Group to file an issue for this and so I have.

Does Einstein support generation of LWC code now ? It generates only the html and not the complete bundle.

@pavan95 We are now supporting LWC Inline Autocompletions for CSS, HTML, Javascript. Could you please update to the latest version and try it out? Thank you for your feedback!

Closing this issue out since we now support LWC for Inline Autocompletions. If you continue to have trouble please re-open this issue or create a new one. Thanks!


Admin View for human takeover

I plan to create sort of administration view as an extension of the current tracking view. This could enable a active intervention of a human as soon as the user gets stuck with the bot.

Hi Adrian. I would like to join in creating this administration view and have human intervention too.

Hei @juliomac
Sure, feel free to submit a merge request for it. I could imagine a new /admin screen where you could see all the existing conversations and respond to them.
Let me know if you need any help. The readme should describe the setup pretty well.


is_compatible function added

This PR makes multiple changes:

Add a basic version of the "is_compatible" method, it will grow gradually over future PRs
update BatchNorm + Mul/Add fusion code to work with the opset 7 version of Mul/Add ops
a fix in filling up the shape_dict, which fixes bugs
add a new unsqueeze remover optimization
update slice conversion function (issue #321 )
remove test dependency on caffe2, by storing input/output values as .npy files

I am merging the PR. Travis is green expected for python 3 which fails in installing caffe2. However, the converter no longer requires caffe2 and in a subsequent PR, that will be removed.


Convert to Typescript

Now that Agent 2 is out and it ships with tf.exe on windows and tf on linux I want to convert over to invoking that instead of rolling my own code. That should insure the same behavior and will make the whole system more extensible

[ ] Create generic TF task that invokes TF and passes additional arguments and optionally adds the collection and auth information

[ ] Windows
[ ] Linux (solve any parameter issues)

[ ] Convert existing Check-in task
[ ] Convert existing Add task
[ ] Convert existing Delete task
[ ] Convert existing Update Gated Changes task
[ ] Convert existing Undo task
[ ] Checkout task should not be needed, since the 2.0 agent explicitly always creates a local workspace

From there it should be easy to support

[ ] Map additional directories & get
[ ] Merge task
[ ] Branch task
[ ] Download from TFVC task (create workspace, map, get)

This is a great idea but the comment is almost 2 years old. Are you still planning to convert the task to Typescript? I just installed it and I get a warning after build:
##[warning]Task 'TFVCCheckin' (1.4.1) is using deprecated task execution handler. The task should use the supported task-lib: https://aka.ms/tasklib

If you'd pick the 2.* version of the task the warning will disappear.
I have few clients using tfvc at the moment, so not very many victims to
test against. Thus trying to not do too much high impact changes.
What would you use a typescript version for?
On Sat, 19 Oct 2019, 05:45 arman-m<EMAIL_ADDRESS>wrote:

This is a great idea but the comment is almost 2 years old. Are you still
planning to convert the task to Typescript? I just installed it and I get a
warning after build:
##[warning]Task 'TFVCCheckin' (1.4.1) is using deprecated task execution
handler. The task should use the supported task-lib:
https://aka.ms/tasklib
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jessehouwing/azure-pipelines-tfvc-tasks/issues/43?email_source=notifications&email_token=AA724S7DQFEHWHIHAPFFIWLQPJ7EDA5CNFSM4C4OT6Y2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXAGFY#issuecomment-544080663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA724SYYXGZ2E5IISQB44MTQPJ7EDANCNFSM4C4OT6YQ
.


Update README

Update README - Add missing info about installing pygments.rb and add other useful steps to run poroject

Hi,
I was plauing with your theme today. I had problem running it right away. The problem was that gem pygments.rb was missing.
Therefore I decided to write more in depth README.
I hope you will find that helpful, I do :)
Cheers!

Brilliant, thanks!


Enable successfulJobsHistoryLimit and failedJobsHistoryLimit to Cronjob Template

*Is your feature request related to a problem? Please describe.
We want to have the ability to delete pods and keep only few Successful/Failed pods.
Describe the solution you'd like
We would like edit the values in the cronjob template to define the number of job history to retain.
example -
cronjobs.spec.failedJobsHistoryLimit, cronjobs.spec.SuccessfulJobsHistoryLimit.
Describe alternatives you've considered
We tried to add annotation to Cronjobs to work with kube-janitor, however we do not have the access to do the same.
What version of eunomia are you using?
0.1.7

eunomia version:0.1.7
Additional context

This will be considered for the v1alpha2 API.


Bootstrapped Django 1.10.

Bumped version
Added stub release notes
Removed deprecated features
Increased PBKDF2 iterations

(to be merged after the stable/1.9.x branch is created in mid-September following Django 1.9 alpha)

I reviewed the entire PR, paying special attention to the deprecations that result from the multiple template engines project.
Besides the small problem I spotted in template responses and a few process-related suggestions, everything looks good.
Thanks!


[RDKDEV-721] XCast limit payload to avoid overflow

Fix for
https://jira.rdkcentral.com/jira/browse/RDKDEV-721

Internal ticket for tracking https://ccp.sys.comcast.net/browse/RDKCOM-3724

Hi @apatel859
Can you please review and merge this PR?


Add instructions for VS Code

Trying to rebase with your remote 2.0-rc1 branch, and my local 2.0-rc1 branch was returning multiple conflicts and I was more than a little worried about introducing error.
Since my PR was a single commit, I think cherry-picking it from my patch-1 branch into the 2.0-rc1 branch is safer.

Awesome, thanks!


Show generic error msg and hide stacktrace if feature flag is enabled

CATEGORY
Choose one

[ ] Bug Fix
[ x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Show a generic error message and hide stacktrace if SHOW_STACKTRACE feature flag is disabled
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

Tested by enabling and disabling SHOW_STACKTRACE feature flag.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0d248fe). Click here to learn what that means.
The diff coverage is 75%.

@@            Coverage Diff            @@
##             master    #7751   +/-   ##
=========================================
  Coverage          ?   65.77%
=========================================
  Files             ?      459
  Lines             ?    21952
  Branches          ?     2415
=========================================
  Hits              ?    14440
  Misses            ?     7391
  Partials          ?      121

Impacted Files
Coverage Δ

superset/views/base.py
70.44% <75%> (ø)

Continue to review full report at Codecov.

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


Option to skip decoding, while keeping LZW-compressed data

Builds on top of #153 and  #144. Implements https://github.com/image-rs/image-gif/pull/144#issuecomment-1492163801
This is a minimal implementation that keeps the current layering between decode_next and LZW decoding happening within.
I've also explored an alternative approach where decode_next only returned compressed Decoded::LzwData, leaving LZW decoding to an upper layer like fill_buffer. That seemed like a more elegant, layered design, but it meant that the lower-level decode_next part of the decoder would always consume full blocks of LZW data, before validity or end of LZW data were checked. It also made fill_buffer have to deal with excess LZW data to decode.

@HeroicKatora @fintelia Reminder about the PRs.
If you don't have the time to maintain the gif crate, can you add me as a maintainer? I'm probably the biggest user of this crate, given that I use it at Cloudflare and in gifski.

@kornelski I just sent you an invite granting you permissions to review/merge pull requests across the image-rs org.
If anyone else reading this is interesting in being a reviewer, please see: https://github.com/image-rs/organization/issues/15


feat: distribution of sessions in lifo order

Changed the order of distribution of the session from CacheSessionPool::getSession(). Fixed data for unit tests with this in mind.
Closes #2368.

@skuruppu based on our conversation yesterday - are you comfortable with us closing this out?

@dwsupplee, yes I think we can drop this PR given the constraints. Here's my comment in the issue linked to the PR.
@andrewinc I hope it's ok that we close this PR without merging.


how to fix this?

django_private_chat2 version:
Django version:
Python version:
Operating System:

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
What I Did
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

Exception inside application: 'ChatConsumer' object has no attribute 'channel_name'
Traceback (most recent call last):
File "/root/testchat/venv/lib/python3.8/site-packages/channels/staticfiles.py", line 44, in call
return await self.application(scope, receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/routing.py", line 71, in call
return await application(scope, receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/sessions.py", line 47, in call
return await self.inner(dict(scope, cookies=cookies), receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/sessions.py", line 263, in call
return await self.inner(wrapper.scope, receive, wrapper.send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/auth.py", line 185, in call
return await super().call(scope, receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/middleware.py", line 26, in call
return await self.inner(scope, receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/routing.py", line 150, in call
return await application(
File "/root/testchat/venv/lib/python3.8/site-packages/channels/consumer.py", line 94, in app
return await consumer(scope, receive, send)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/consumer.py", line 62, in call
await await_many_dispatch([receive], self.dispatch)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/utils.py", line 51, in await_many_dispatch
await dispatch(result)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/consumer.py", line 73, in dispatch
await handler(message)
File "/root/testchat/venv/lib/python3.8/site-packages/channels/generic/websocket.py", line 173, in websocket_connect
await self.connect()
File "/root/testchat/venv/lib/python3.8/site-packages/django_private_chat2/consumers.py", line 138, in connect
logger.info(f"User {self.user.pk} connected, adding {self.channel_name} to {self.group_name}")
AttributeError: 'ChatConsumer' object has no attribute 'channel_name'
hi how i can fix this?

Try add this in settigns.py
CHANNEL_LAYERS = {
'default': {
'BACKEND': 'channels.layers.InMemoryChannelLayer'
}
}

Closing as stale, please read https://channels.readthedocs.io/en/stable/ to learn about how Django Channels works.


subparser parents

one more request on this way .. the tool I want to wrap uses the parent feature of subparser, i.e. it defines subparsers that are reused in many other subbarsers.
check for the "parent" argument (in combination with add_help=False) here :
checkm.py.txt
I guess macros would be a great way to do this.
for now I will do this manually

Wow you were not kidding about a lot of argparse options. Oh my word.
Thanks for the example. I see that it's under GPL3, would you mind if I included it into the examples/ folder in this repository as an example of the complexity it can support?

Sure, I took it from bioconda checkm-genome=1.0.11 (most recent version in https://github.com/Ecogenomics/CheckM/blob/master/bin/checkm) and adapted it to include only the argparse stuff (and replaced some imports by the actual code).

Perfect, thanks!

since I just stumbled across this again, still no time for development work, but

I guess macros would be a great way to do this.

I've often considered how nice it would be if I could find a good AST parser (or three), and then identify large common subsets across 2+ tools, and automatically rewrite it into a macro. But that's almost a masters-time/level project and hard to fit that in.

I now "solved" this in a shell script generating the macros. Made small progress here: https://github.com/bernt-matthias/mb-galaxy-tools/tree/master/tools/checkm

I've often considered how nice it would be if I could find a good AST parser (or three), and then identify large common subsets (using the same code structure/same XML structure even if different parameter names) across N xml files and automatically rewrite it into a macro. But that's almost a masters-time/level project and hard to fit that in.

Great idea! I would like to see this as a planemo feature I think. It should be useful to a lot of tools. A master student for planemo development would be great!


Add Hermes engine as option for macOS

Related project board https://github.com/microsoft/react-native-macos/projects/4
Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
This adds the Hermes engine as an optional replacement to JavaScriptCore for macOS targets.

Hermes is enabled by virtue of adding the React-Core/Hermes dependency to one’s Podfile.

The engine is shipped as a prebuilt framework binary, optimised for release.

It comes with a separate debug symbols bundle (dSYM) for debugging/crash report symbolication.

Support to generate source-maps when building release builds has been added, both without Hermes enabled:

…and with Hermes enabled:

(Note that file/line is correct, but column not. At the moment I am brushing this off as a minor issue, but something to watch out for, especially when we move to Hermes master.)

Changelog
[macOS] [Added] - Add Hermes as (optional) JS engine.
[macOS/iOS] [Added] - Add ability to generate source-maps when building a release build with Xcode.
Test Plan

Update Podfile with use_react_native!(hermes_enabled => true) to add the dependencies and enable it as runtime engine.
Uncomment FORCE_BUNDLING in bundle build-script phase to test source-maps.
Verify that attaching a Chrome debugger works.

TODO

[x] Finish this PR: review it myself and finish the PR description.

[x] Rebase on latest RN macOS v0.62.0 branch.

[ ] The changes required to build the binary are currently still here in my fork. I am reaching out to the upstream team about incorporating those changes upstream and releasing npm packages with the binary from their account, especially going forward.
This could also be done in a follow-up PR, as long as we do so before we release v0.62.0 (preview); although in that case it might be preferable to move the tarball from my account to this MS repo?

Microsoft Reviewers: Open in CodeFlow

The current CI failures are identical to those on the fb62merge branch.


Local Storage does not work in Edge Browser

I am not able to get local storage working in Microsoft Edge browser but does work fine with Chrome. I was able to re-produce this by running the Blazor.Extensions.Storage.Test project first in Chrome to validate it was working, which it was and then in Edge. When you run it in Edge you see the following in the Console output.

Can you try the latest package please?

I've tested on current edge build and it works just fine. Closing for the lack of innactivity.


Documentation does not contain that columns array element can have sortMethod property

In documentation it is not mentioned that element in columns array can also have sortMethod property.

Located @ https://github.com/tannerlinsley/react-table/blob/master/docs/api.md#table-options-1 under the sortType option.

You bet. If you would like to improve the v6 documentation, please submit a pull request to the v6 branch. Thanks for your support!

@tannerlinsley I will send, I wanted to confirm if I have not misread something.


Prepare app for localization

Task/Issue URL: https://app.asana.com/0/856498667320406/1176971145104443
Tech Design URL:
CC:
Description:
Prepare for localization work:

Go through UserText and provide comments when needed.
Ensure special characters (like %) are included in text instead of being concatenated.
Switch to plural stiringsdict where needed.
Extract attributed labels from storyboards to UserText and apply these from the code.
Regenerate Localizable.strings from the code.

To resolve later:

About screen.

Steps to test this PR:
Smoke tests strings in the app, especially:

Dax dialogs.
Privacy protection trackers summary.

To ensure strings are translatable you can choose 'Accented Pseudolanguage' in the scheme while running the app.

Copy Testing:

[x] Use of correct apostrophes in new copy, ie ’ rather than '

Device Testing:

[x] iPhone
[x] iPad

OS Testing:

[x] iOS 10
[x] iOS 11
[x] iOS 12
[x] iOS 13

Internal references:
Software Engineering Expectations
Technical Design Template

@tomasstrba no, I've overlooked that, thanks for pointing this out. Fixed :)


fix: failed to return index stats if the first partition is empty

Codecov Report
Attention: Patch coverage is 95.45455% with 2 lines in your changes missing coverage. Please review.

Project coverage is 80.01%. Comparing base (685fe43) to head (0533d07).

Files
Patch %
Lines

rust/lance/src/index/vector/ivf/v2.rs
95.45%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2565   +/-   ##
=======================================
  Coverage   80.00%   80.01%
=======================================
  Files         209      209
  Lines       59864    59907   +43
  Branches    59864    59907   +43
=======================================
+ Hits        47896    47934   +38
- Misses       9143     9145    +2
- Partials     2825     2828    +3

Flag
Coverage Δ

unittests
80.01% <95.45%> (+<0.01%)
:arrow_up:

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

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

this failed checks are not related to this PR, merge it


share token is derived from asset with lower id

LGTM after conflicts are resolved

Closed in favor of #29


Update all.json

Site
Urlscan
Screenshot

collab-assets.github.io
talisman: https://urlscan.io/result/74c49898-9148-4bcd-9782-016f78b01d8a/
https://github.com/polkadot-js/phishing/assets/97024090/c7af7092-3f5d-4231-a9cb-bf95d471e7d0

community-system.in
talisman: https://urlscan.io/result/ec95e188-97f2-4ec4-a90a-c37ce28622ae/
https://github.com/polkadot-js/phishing/assets/97024090/e008494f-c127-44a5-9b2f-5ff71778fa88

friend-net.tech
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/758cf63a-2b2f-494f-a184-bc750bc50a9d

join-collab.land
talisman: https://urlscan.io/result/14022692-9875-4095-9d8f-02a6b436166c/
https://github.com/polkadot-js/phishing/assets/97024090/8cbc867b-419a-4008-90d4-5c91d3fa1c84

lidoi.fi
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/f9781d5a-1f47-4d86-b6cd-f5d553f86c8d

link-pool.com
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/f2b2f7ff-4f69-4975-882a-b73d4eb286b4

nodedebug.firebaseapp.com
https://urlscan.io/result/8a9096b2-3fa1-488d-9c0e-e93fa3cbef41/dom/
https://github.com/polkadot-js/phishing/assets/97024090/774659d2-59ce-436a-a0a2-3808fccc0bab

nodedebug.web.app
https://urlscan.io/result/adba31b0-ba6f-4f42-a748-d4b319785790/dom/
https://github.com/polkadot-js/phishing/assets/97024090/818597aa-41f5-4a8f-a0f2-eaade891e9c7

pondc0in.site
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/fdef148b-ff58-4187-a8f3-7de088db358f

prorectifier.com
https://urlscan.io/result/dfbce3da-81ab-4a45-b158-d4919fd366c1/
https://github.com/polkadot-js/phishing/assets/97024090/f7805cd3-9748-47f0-80a7-c0b38a2b2aa0

uniswap-v3.swapto.pro
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/15861a72-53ce-406e-9c22-66129f0993e2

webmainnetai.pages.dev
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/57168975-eb2c-4ba6-a504-75971eadbd62

xblur-io.net
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/ffba9779-7ec7-4273-b123-2d9e4fe4933b

zksync.gifts
subwallet
https://github.com/polkadot-js/phishing/assets/97024090/e1a974d8-b16b-44ca-b3d7-010419c5d0d3

@jacogr I keep getting these failed checks but I don't get whats going wrong...


Add GenShinGothic font

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Additionally, when adding a new cask:

[x] Checked there are no open pull requests for the same cask.
[x] Checked there are no closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

was verified as official when first introduced to the cask comments are too unspecific. OSDN is like Sourceforge, so more detail is needed in the comments.

Replied at #633


⚠ Bofrost.lu has degraded performance

In 0e36b67, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1994 ms

Resolved: Bofrost.lu performance has improved in f6fa4f7 after 29 minutes.


closes #427 - add CSRF tokens

closes #427

Nice @Knorrke - did you test all forms manually?

I only tested some of them (the Authorization and Entity forms), but could do this tomorrow if needed. But I had to do it manually, yes, is there a way to write an automated test for this?

But I had to do it manually, yes, is there a way to write an automated test for this?

I would love to have automated tests, unfortunately we need to come up with the whole toolchain for that because it requires integration tests, not unit tests. We can go through it together tomorrow!

@arekkas this should be finished now, please take a look. Espacially: I used a lot of view helpers to load the forms, is this ok, or should I change this somehow?
I also manually tested all pages, so there shouldn't be any problems in the functionality.

@arekkas As discussed I now removed the view helpers and passed the forms in the controller directly to the view. Looks a lot better now, thanks for the tip!
I sticked with the view helper for uuid trash and purge actions, because these forms are used in many controllers and I think it's much easier to use this way, than passing the form in every controller.
So again, ready for review ;-)


Add ignore flag

Add the ignore flag which make cargo-machete respect .ignore and .gitignore files when searching for files.
I've tried to make minimal changes. I've also tried not to make any breaking changes. The behavior should only change when passing --ignore. No guarantee of correctness.

I feel like this should be the default behavior. Maybe via a breaking change in a major release?

I'd say that doing so is a separate issue. I wouldn't want to block this capability on the discussion about whether it is the right time to do a major release, and what needs to go in that release. However, this is not up to me. That is up to the maintainer.

@bnjbvr pinging you in case you are not yet aware of this PR

Howdy! Sorry for not answering earlier, happy to see a PR for this, thanks. It might take a bit of time for me to review, since I'm in the middle of some big personal changes, but j should be able to get to it in the next week or so. In any case if someone from the community wants to review/test etc, feel free to do so!

I'm in the middle of some big personal changes

Good luck with that!

Can you add two tests please:

one where there's an ignored file that's filtered out because the ignore flag is set?
I believe this is covered by these lines: https://github.com/bnjbvr/cargo-machete/pull/95/files#diff-42cb6807ad74b3e201c5a7ca98b911c5fa08380e942be6e4ac5807f8377f87fcR305-R310. If skip_target_dir is false and respect_ignore_files is true, then no manifests are found because /target is in the .gitignore.

one where the ignore flag isn't set and machete successfully analyzes a crate that's in a directory that would be ignored if the flag was set?
I believe this is covered by https://github.com/bnjbvr/cargo-machete/pull/95/files#diff-42cb6807ad74b3e201c5a7ca98b911c5fa08380e942be6e4ac5807f8377f87fcR312-R317. If both skip_target_dir and respect_ignore_files are false, then at least one manifest is found.

I can split the test function into multiple test functions. I have not done so because the original already contained two cases.

(Oops, sorry I've forgotten about this PR, I'll try to get back to it soon™)


NoMethodError 'fully_formatted'

Subject of the issue
I am attempting to set up rspec to run only failures with rspec --only-failures. When I do this I get this error
 block in fully_formatted_failed_examples': undefined method 'fully_formatted' for #
 <RSpec::Core::Notifications::ExampleNotification:0x00007fce3766b108> (NoMethodError)

When I run be rspec --only-failures I get this error:
bundler: failed to load command: rspec (/Users/mikeheft/.rbenv/versions/2.5.0/bin/rspec)
I have config.example_status_persistence_file_path = './spec/examples.txt' set up in spec_helper.rb
The stacktrace says it's erroring on /.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/notifications.rb:114:in 'block in fully_formatted_failed_examples' with an undefinedmethod for fully_formatted
Your environment

Ruby version: 2.5.0
rspec-core version: 3.7.1

Steps to reproduce
Not sure of the steps to reproduce. I simply enter rspec --only-failures or  bundle exec rspec --only-failures
Expected behavior
"0 examples, 0 failures"
Actual behavior
One of two errors depending on what I enter into terminal.
With bundle exec rspec --only-failures I get :
bundler: failed to load command: rspec (/Users/mikeheft/.rbenv/versions/2.5.0/bin/rspec)
Along with
 block in fully_formatted_failed_examples': undefined method 'fully_formatted' for #
 <RSpec::Core::Notifications::ExampleNotification:0x00007fce3766b108> (NoMethodError)

And when I just enter rspec --only-failures The error is:
 block in fully_formatted_failed_examples': undefined method 'fully_formatted' for #
 <RSpec::Core::Notifications::ExampleNotification:0x00007fce3766b108> (NoMethodError)

Can you provide a full stacktrace?  It provides important context.

Run options: include {:last_run_status=>"failed"}

All examples were filtered out

Randomized with seed 36778
must have 0 Critical Security Issues (FAILED - 1)

Coverage report generated for RSpec to /Users/mikeheft/turing/bread_shop/coverage. 174 / 249 LOC (69.88%) covered.
Traceback (most recent call last):
	18: from /Users/mikeheft/.rbenv/versions/2.5.0/bin/rspec:23:in `<main>'
	17: from /Users/mikeheft/.rbenv/versions/2.5.0/bin/rspec:23:in `load'
	16: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/exe/rspec:4:in `<top (required)>'
	15: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:45:in `invoke'
	14: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:71:in `run'
	13: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:87:in `run'
	12: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:112:in `run_specs'
	11: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:81:in `report'
	10: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:171:in `finish'
	 9: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:191:in `close_after'
	 8: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:175:in `block in finish'
	 7: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:205:in `notify'
	 6: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:205:in `each'
	 5: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:206:in `block in notify'
	 4: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/formatters/base_text_formatter.rb:32:in `dump_failures'
	 3: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/notifications.rb:113:in `fully_formatted_failed_examples'
	 2: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/notifications.rb:113:in `each_with_index'
	 1: from /Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/notifications.rb:113:in `each'
/Users/mikeheft/.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/notifications.rb:114:in `block in fully_formatted_failed_examples': undefined method `fully_formatted' for #<RSpec::Core::Notifications::ExampleNotification:0x00007f890cc37cc8> (NoMethodError)

I have not edited the local installation of RSpec. I'm not sure how to put together an example for you to clone unless you clone the project that I'm working in. It seems to only error out when there are no errors. I was under the impression that if that were the case, that the response would be '0 examples, 0 failures'

I'm not sure how to put together an example for you to clone unless you clone the project that I'm working in.

For problems like this, I usually create an isolated, runnable example by doing something like:

Make a new branch.
Delete an arbitrary half the specs
Run the rspec command again and see if the same problem happens.  If the same problem happens, go back to step 2 and delete half the specs again.  If the problem goes away, it indicates that something in the half you deleted is causing the problem.  Switch which half you are keeping and which you are deleting, then run again the rspec command again to see if the problem happens again.
In this fashion, continue narrowing things down to the smallest set of specs you can that reproduce the example.
Do the same with your dependencies in your Gemfile (paring them down) and you lib/app code so that you get them down to a minimal set of dependencies and implementation code.
Create a new git repo with the minimal project you've created, and report here with instructions to clone and run the project.  (Or just paste a code snippet here if it can be repro'd with a single file of code).

Note that many times when I've gone through this exercise, the act of doing it has itself identified the source of the problem before I got things pared all the way down.  For example, if removing a particular gem makes the problem go away, it indicates that the gem was causing the problem, and you can either report that here or continue to look into it yourself.

hey @myronmarston thanks for the insight! Sorry for the delayed response, been having a hectic month. It turns out it was my brakeman gem. I deleted all tests except one and the error still persisted. Then I started on the Gemfile and removed gem brakeman. Now I need to figure out how that was cause the --only-failures flag to cause the fully_formatted to give a no method error.

Thanks for the confirmation, closing for now as I don't think theres anything further RSpec can do to help here.


Loading an external exolist.json file doesn't work with TV API >= 31

Version
Media3 1.1.1
More version details
No response
Devices that reproduce the issue
All TV devices with API >= 31.
For example, reproduced with TV emulator + 'S' Android TV image for x86 on Windows, and arm64-v8a on Mac.
Devices that do not reproduce the issue
TV API <= 30. For example, cannot reproduce on TV emulator + 'R' Android TV image for x86.
Phone or Tablet devices are not affected. Even API >= 31.
Reproducible in the demo app?
Yes
Reproduction steps

Open the project in Android Studio
click 'Run', 'Edit Configurations...'
Select the 'demo' configuration
In the 'General' tab, under 'Launch Options', 'Launch', change 'Default Activity' to 'URL'
In the 'URL' field, type https://github.com/androidx/media/raw/release/demos/main/src/main/assets/media.exolist.json
Save and run the 'demo' configuration

Or manually type the command:
adb shell am start -a android.intent.action.VIEW -c android.intent.category.BROWSABLE -d 'https://github.com/androidx/media/raw/release/demos/main/src/main/assets/media.exolist.json'

Expected result
The SampleChooserActivity should launch with the specified exolist.json
I ActivityTaskManager: START u0 {act=android.intent.action.VIEW cat=[android.intent.category.BROWSABLE] dat=https://github.com/... flg=0x10000000 cmp=androidx.media3.demo.main/.SampleChooserActivity} from uid 2000

Actual result
System toast message saying "You don't have an app that can do this".
I ActivityTaskManager: START u0 {act=android.intent.action.VIEW cat=[android.intent.category.BROWSABLE] dat=https://github.com/... flg=0x10000000 cmp=com.android.tv.frameworkpackagestubs/.Stubs$BrowserStub} from uid 2000

Media

This was also reported by NimaTajfar in old ExoPlayer issues #9437 and #9441, but was missing important details:

This is TV-specific, and API version-specific, but not ATV or GTV specific.
The exact 'am start' command included above.

Bug Report

[X] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Thanks for your report.
I think this is a duplicate of #438.
It works if you are using the specific action androidx.media3.demo.main.action.BROWSE that we added for this reason:
adb -s emulator-5554 shell am start -a androidx.media3.demo.main.action.BROWSE \
    -d 'https://github.com/androidx/media/raw/release/demos/main/src/main/assets/media.exolist.json'

If you want to use it from Android Studio like you explain above, then you can change the Launch options in Run configuration  to URL and add -a androidx.media3.demo.main.action.BROWSE as the Launch flags. With this in place I was able to start the app from AS to an TV emulator running API 33.

I added a not in the documentation about using androidx.media3.demo.main.action.BROWSE on TV devices.
https://developer.android.com/guide/topics/media/exoplayer/demo-application#2-loading
I'm closing this issue. Please open a new one if there are further issues regarding starting the app on ATV.

Thanks Marc! I always wondered what that custom action was for, but never bothered digging (I should have).
And sorry I missed #438 in my search (I must have searched in the old ExoPlayer repo only).


Pass environment variables during build

This could be done by adding an attribute like we do for @guku(generate) it could look like @guku(env=ENV_NAME)

implemented using @guku(env=ENV_NAME)


uom data conversion explanation and queries

Unit of Measure data conversion steps (powerpoint) and SPARQL. Fixes #1034.

I don't think I know how to do a git review and or approval, or if I know what I'd be looking at.  There appears to be a lot to look at.
On Apr 2, 2024, at 9:18 AM, Phil Blackwood @.***> wrote:
@philblackwoodhttps://github.com/philblackwood requested your review on: #1071https://github.com/semanticarts/gist/pull/1071 uom data conversion explanation and queries.
—
Reply to this email directly, view it on GitHubhttps://github.com/semanticarts/gist/pull/1071#event-12327812584, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAGJPVGOV7YTGKZJUL7HPLY3LD43AVCNFSM6AAAAABFTTDM3WVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGMZDOOBRGI2TQNA.
You are receiving this because your review was requested.Message ID: @.***>

You can make comments or modifications in a copy and then upload it into a GitHub comment.
From: Michael Uschold @.>
Date: Tuesday, April 2, 2024 at 3:27 PM
To: semanticarts/gist @.>
Cc: Rebecca Younes @.>, Mention @.>
Subject: Re: [semanticarts/gist] uom data conversion explanation and queries (PR #1071)
@philblackwoodhttps://github.com/philblackwood @rjyouneshttps://github.com/rjyounes I want to make comments in the PowerPoint deck, what is the best way for me to do that? One way is to make a separate copy and email it to Phil who can make any desired changes and then make another commit. How can I do it more publicly on git?
—
Reply to this email directly, view it on GitHubhttps://github.com/semanticarts/gist/pull/1071#issuecomment-2032943127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAM5PZFOCFM667RSYINZYHTY3MBCVAVCNFSM6AAAAABFTTDM3WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZSHE2DGMJSG4.
You are receiving this because you were mentioned.Message ID: @.***>


Add gem removal hook to remove YARD and yri data on uninstalling gems

Description
When you uninstall some gems, YARD and yri data should be removed as well as RDoc data(see
here).
Completed Tasks

[x] I have read the Contributing Guide.
[x] The pull request is complete (implemented / written).
[x] Git commits have been cleaned up (squash WIP / revert commits).
[x] I wrote tests and ran bundle exec rake locally (if code is attached to PR).

Coverage decreased (-0.04%) to 93.473% when pulling 020a46e5a0f4e659b3a844a69ef387caf56833ed on KitaitiMakoto:removal-hook into fdf71a4708e91d2c422ef2a815ee9c36a7c60710 on lsegal:master.

Does RDoc do this?

RDoc gem itself doesn't.
But rubygems lib bundled with Ruby removes RDoc's data on uninstalling:
https://github.com/ruby/ruby/blob/68c9c97f139c8e4f0543a3f431fdb567ad6b1ac3/lib/rubygems/uninstaller.rb#L271
As we see the result, by uninstalling gems, we remove documentation files. Don't you think it's nice if there is no documentation files including YARD's after uninstalling gems?

I'm fine with this, but I want to make sure we're doing what other tools are.

Thank you so much!


Reduced reusability because of MatDialogRef's requriments?

Proposal.
Suppose I have a component I want to reuse in different other components to be opened as a dialog.
MatDialogRef's current requirement to pass the "opener" component's type does not allow such use case.
If not using MatDialogRef, it have to be the openers's responsibility to close the dialog, forcing some kind of communication between the two components.
It would be nice to be able to instantiate a MatDialogRef derivative which requires no opener type, and just allows to close the dialog.
This is useful in frameworks like NGRX: I just issue a new action from my dialog and I just want it to close itself, the dialog opener does not need to know anything, as it can watch the application's state.

I think youre wrong, to instantiate the dialog ref you dont need the opener component you should use the component itself, i mean if your dialog component is DialogComponent you should instantiate dialog ref like this: public dialogRef: MatDialogRef<DialogComponent> in the constructor. You can see an example here.
The only thing a bit anoying is that you need to add the dialog component as an entry component, but if you have a shared module you can add it there and thats it, you can use that dialog anywhere

I don't also understand the request:

use MatDialogClose button component to close the dialog from within the dialog
use MatDialogRef.close() to close the dialog from outside (i.e. the opener) or inside (the dialog itself)
use MatDialog.closeAll() to close all dialogs without a MatDialogRef

Also for opnening a dialog, all you do is MatDialog.open(YourDialog) which does not require the opener at all and returns the MatDialogRef.
This should give you all possible options to close the dialog from either outside or inside the dialog without ever having a strict dependency between the opener and the dialog.

The overview: https://material.angular.io/components/dialog/overview#dialog-content
The API: https://material.angular.io/components/dialog/api#MatDialogClose
Anyways, you're welcome! Guess you can than close this ticket, when you have everything you need.


doesnt work

the issues section should be populated only with specific lines cited and examples of input along with expected output.

the issues section should be populated only with specific lines cited and examples of input along with expected output.

Ok fantastic point. I shall implement that methodology in my future posting of issues.

Great thanks sir. Chip Chip


Fix the issue on project ids which prefixed by organization

Some cloud project ids have organization as prefix, separated with colon. It makes problem on parsing configuration that passes by -instances parameter.
The observer error is :
errors parsing config:
googleapi: Error 403: The client is not authorized to make this request., notAuthorized
This fix handles this scenario and merge organization to project id again.

Thanks for raising the issue! I'd rather move forward with https://github.com/GoogleCloudPlatform/cloudsql-proxy/pull/76 though, as it shares code that already exists for this job.


Add blackbody PBR node implementations (#1356)

Address the missing targets for the PBR blackbody node implementation ( #1356 )
using the following approximation: Wikipedia: Planckian Locus Approximation
Add a simple Standard Surface material example using the above blackbody node for emission.
Reenable blackbody node tests.

This looks really promising, thanks @nicolassavva-autodesk.  Let us know if you run into any issues with the Autodesk CLA, and the Linux Foundation is usually very responsive if you file a support ticket using the link above.

Thanks Jonathan
I just re-authorized the Autodesk CLA with the account in question

Looks great @nicolassavva-autodesk , I just noted an issue with the clamps above.

Good catch Niklas.
Added a quick fix for the missing clamping

is there an MDL implementation missing now?

I hope I'm not opening a can of worms on a topic that has long since been settled, but is this function merely trying to return the color corresponding to a blackbody temperature, normalized to some standard luminance? Or is it supposed to be returning the actual emittance of the blackbody (W/m^2 or whatever)?


Add Lithuanian language support

A customer has kindly translated the page into Lithuanian for us. We need to drop these language files into the theme.
lt.json.txt

These are no added to the theme and released in 2.47.0 - We'll now look at adding this release to the app.


do not use maps for custom ui console driver

and do a sync.RLock while using it to make sure this don't go off the rails (altho that's should not be absolutely necessary)
https://issues.redhat.com/browse/SRVKP-5507
Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

I don't know how to reproduce since we don't have a clear case, how that race is happening but there is e2e for custom console driver in gitea

/lgtm


Version 20.2.1

Coverage remained the same at 84.346% when pulling 7f9f6d9f1779afd83e2ed9962f0b5e0dfdb2cc40 on version-20.2.1 into a60a76a14bddbba43033ad20b507137d18ab0248 on master.


added support for datatype declarations

Problem: We do not currently support datatype declarations

Support: This PR adds parsing for datatype declarations, along with pretty printing. We get stuff like:

λ ~/P/parse-sml on add-datatype-dec ⨯ ./main test/succeed/datatypes.sml                          02:55:14

datatype ('a, 'b) either = INL of 'a | INR of 'b
datatype foo = A

datatype even = Z | S1 of odd
and odd = S2 of even

datatype 'a tree = Empty | Node of 'a tree * 'a * 'a tree

datatype ('a, 'b, 'c) complicated =
    op Con1 of int * string
  | Con2 of int -> string
  | Con3
and complicate = op Cons1 | Cons2 of (int, string, bool) complicated
and 'a complex = Const
withtype bar = int
and 'a baz = string

Lexing succeeded.
Parsing...

datatype either = INL of 'a | INR of 'b
datatype foo = A
datatype even = Z | S1 of odd
and odd = S2 of even
datatype tree =
  Empty | Node of 'a tree * 'a * 'a tree
datatype complicated =
    op Con1 of int * string
  | Con2 of int -> string
  | Con3
and complicate =
    op Cons1
  | Cons2 of (int, string, bool) complicated
and complex = Const
withtypee bar = int
and baz = string

Parsing succeeded.

You might want to check me on the pretty printing stuff - I never actually read through all of the Wadler paper. For instance, the datatype tree in the above example might not be exactly what you want. I'm not sure of how to fix it, though.
Actually, it parses replicates too now, but it just doesn't pretty print them yet.

Nice! The pretty printing seems good. I've really just been using it to spot-check the parser.


fix typo in v2 manual.md demo code

What type of PR is this?
fix typo

[ ] bug
[ ] cleanup
[x] documentation
[ ] feature

What this PR does / why we need it:
fix typo

Which issue(s) this PR fixes:
typo

Special notes for your reviewer:
(fill-in or delete this section)

Testing
(fill-in or delete this section)

Release Notes
no

🙏 thanks!


Create test.js

@campoy Actually, it works on the new files. There is only one useful suggestion, however. I guess that's because redux model was cached with a wrong version of the JS driver initially.
@zurk Can you please repeat this tomorrow in our lab conditions.

Actually, all the three suggestions make sense to me, I can see the logic behind them.

With latest mater and bblfsh javascript driver v1.2.0 I get the same comments. So we are fine.
By the way, training+analyze takes only 92 sec. Great optimization since December.


Fix Beyla CTRL+C shutdown. Also simplify pipeline

In preparation for task #249 . The change is not big but it affects multiple files, as it has changed an ubiquitous type.
First, we have movedtransform.HTTPRequestSpan to request.Span in its own package.
Before this PR, eBPF tracers could send either []HTTPRequestTrace or []HTTPInfo slices. That mean that the Pipeline input channel had to be of the type []any and internally added a Codec that transformed each object to HTTPRequestSpan (In the Pipes library jargon, a codec is an component that automagically is attached to transform data when two connected pipeline stages do not match the output->input types).
Now the codec has been removed and the input channel is of the []request.Span instead of the []any type. Now each Tracer implementor has the responsibility to convert its input type to request.Span type, so the pipeline stages don't need to know about the eBPF types.
This will facilitate inserting the Service Name from the tracers, as we have now a unified type. It also will slightly improve speed, as we have removed a goroutine continuously iterating a channel.
This PR also fixes the sigint/sigterm handling, allowing Beyla to stop again without requiring a sigkill (bug introduced in PR https://github.com/grafana/beyla/pull/243).

@grcevski yes! I agree that at some point we'd need to expand Beyla for more event types. Another option is not make each event type to pass across the same pipeline but have their own pipeline, as there would be some nodes that wouldn't be useful in that case (for example, the log events maybe don't need the routes decoration node).


Fix validation errors by removing spaces from keys

$ref values must be RFC3986-compliant percent-encoded URIs

Hi @saerdnaer ,
thank you for your contribution.
These files however are automatically generated by the openrouteservice backend:
e.g. in line
https://github.com/GIScience/openrouteservice/blob/104371bb60298f90641908a1d11e3e6a6325eace/openrouteservice/src/main/java/org/heigit/ors/api/requests/routing/RouteRequestOptions.java#L27
by the package springfox-swagger2
https://github.com/GIScience/openrouteservice/blob/master/openrouteservice/pom.xml#L497-L499
So it would be best to fix these issues directly there


Update libqglviewer rules for debian.

The sourcedeb has stayed the same, however the binary packages have been renamed.
Sourcedeb: https://packages.debian.org/source/jessie/libqglviewer
Wheezy binaries:
https://packages.debian.org/wheezy/libqglviewer-qt4-dev
https://packages.debian.org/wheezy/libqglviewer-qt4-2
Jessie equivalents:
https://packages.debian.org/jessie/libqglviewer-dev
https://packages.debian.org/jessie/libqglviewer2
This will fix octoviz building on debian: http://<IP_ADDRESS>:8080/view/Kbin_dJ64/job/Kbin_dJ64__octovis__debian_jessie_amd64__binary/7/console @ahornung when this is merged if you could rerun bloom that would fix the above build.

+1

@tfoote Done!

@ahornung thanks! it's building now: http://<IP_ADDRESS>:8080/view/Kbin_dJ64/job/Kbin_dJ64__octovis__debian_jessie_amd64__binary/


Update dependencies

Dependencies include: Catch, .NET SDK, etc.

I'm reluctant to change these dependencies as they are tied to the VS ingestion process that is not tested here. I'll leave that for @DefaultRyan to handle as needed.


Planet Haskell's newest post is 12 months in the future

The actual date of the post is 2017-12-14. It would be less annoying if the post didn't remain at the top.

FYI, the problem is still present as of February 8, 2018 (more than 1 month after this issue was opened). I do realize this is not a high-priority bug, but still, it's sort of an ugly wart on a fairly visible surface, so if someone had time to take a look, I think that would be great.


Run as background process

This should run as a background process rather than blocking the shell running it.

We're all already familiar with it, but as a note here for this issue, check out forever.


Roll Engine from 541b6365633b to bcdadab28cdd (6 revisions)

https://github.com/flutter/engine/compare/541b6365633b...bcdadab28cdd
2022-04-07<EMAIL_ADDRESS>Soft revert of cd7a1e2f14415ab5b1e6d6f45ffe95cdd4c592d6 (flutter/engine#32465)
2022-04-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from kXsU99mbx... to ry_yHNxEs... (flutter/engine#32501)
2022-04-07<EMAIL_ADDRESS>Roll Skia from e4d4c9362b98 to eabed6ac6766 (10 revisions) (flutter/engine#32500)
2022-04-07<EMAIL_ADDRESS>[skia] Fix more transitive includes (flutter/engine#32430)
2022-04-07<EMAIL_ADDRESS>Roll Skia from 072e09b26dff to e4d4c9362b98 (13 revisions) (flutter/engine#32483)
2022-04-07<EMAIL_ADDRESS>Roll Dart SDK from 760722f8224b to 4f0b75865b3c (9 revisions) (flutter/engine#32459)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from kXsU99mbxJf9 to ry_yHNxEsx8B
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


minor change of description in makefile

In the makefile explanation in readme.md the option
"ENABLE_COPY_CHAN_TO_VFO       := 1       copy current channel into the other VFO."
gives the impression that the channel is copied to the other VFO. But it is copied to the same VFO.
E.g. if i use a channel in VFO A it is copied to VFO A, not VFO B.

VFO means frequency mode. There is also channel mode, hence chan to VFO.
I didn't choose that naming. It is hard to even name it properly, there is upper channel, lower channel, frequency channel, "channel channel".
It is what it is.

I understand your point, naming these features has become diffuse. Maybe I can clarify some things, but that is from a strict HAM view. But pls understand: in no way this is meant to be leading for your project.
From a HAM/technician perspective there are 2 VFO's, as in the old analoque era: VFO-A and VFO-B.
A VFO can be regarded as a seperate receiver, it is the Variable Frequency Oscillator for a mixer.
Wether you set the VFO freely by hand or from a fixed value in memory, it is still the same VFO.
Channels only excist in bandplans and define fixed points in frequency assignement to be used. The step size ( increment per up/down ) is often set equal to the channel distance. Like in CB where the stepsize of 10 enables one up/down press to go to the next channel. Channels can be numbered, CB channel 14 is the 14th channel/step counting from the beginning of the CB band.
Often the pre-programmed frequencies in memory are not displayed as frequency but as the channel number. So, to keep it in C, the channel number is the index for an array of stored frequencies: CB[14] = 27.125 .
On the HT the upper display is regarded as VFO-A and the lower as VFO-B. So for a HAM or trained radio-operator when tuning VFO-A 'the other VFO' is VFO-B.

I read again your suggestion and yes the description was misleading, I concentrated on the option name itself at first. I changed the description. Thanks.


Authentication failure when cloning repo

Hi
I am cloning the repo using git clone --recursive ...., but it fails to authenticate, after asking me twice to enter login details:
Cloning into '/home/../Desktop/MoravaEngine/MoravaEngine/src/Hazel'...
Username for 'https://github.com': TreeFruit
Password for<EMAIL_ADDRESS>remote: Support for password authentication was removed on August 13, 2021. Please use a personal access token instead.
remote: Please see https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ for more information.
fatal: Authentication failed for 'https://github.com/dtrajko/Hazel-dev.git/'
fatal: clone of 'https://github.com/dtrajko/Hazel-dev.git' into submodule path '/home/mm/Desktop/MoravaEngine/MoravaEngine/src/Hazel' failed
Failed to clone 'MoravaEngine/src/Hazel' a second time, aborting

How should i fix this, you have an ideas?

I've also tried cloning it normally using git clone ..... and then running git submodule update --init --recursive inside the folder, but it gives me another error:
Submodule 'MoravaEngine/src/Hazel' (https://github.com/dtrajko/Hazel-dev.git) registered for path 'MoravaEngine/src/Hazel'
fatal: No url found for submodule path 'vendor/FastNoise' in .gitmodules

Thanks , it now clones fine


empty attribute

When enable the modules in front office see empty attributes?why?
thanks

@metalzone82  Can you post a link to your website? Also please specify the magento version. I'm not able to reproduce this.

I'm closing this issue as it cannot be reproduced and there is not feedback from the OP in more than 10 days.


Invalid API response from NVIDIA.

An "Invalid API response from NVIDIA" problem occurs if you choose the "Studio" option while your card doesn't have any studio drivers available. After that, the program can't be started without deleting its settings file.

I see the problem. I never check if the card actually supports the driver type selected. Shouldn't be too hard to fix, thank you for the heads-up!

Where is this setting file please ?

Thank you, I already deleted it but it didn't fix the bug.

GTX 1060 maxQ with game ready driver

but don't worry i will wait patiently for a fix ;)

GTX 1060 maxQ with game ready driver

Oh, that is strange. Can you post a screenshot of EnvyUpdate on your computer?

This should be fixed in 2e4fa930ff5e1fb89d96c451f24e1e36eee6ec10


Consider use OrderMap instead of HashMap

benchmarksgame-rs used OrderMap, and got better performance.
Ref: https://github.com/TeXitoi/benchmarksgame-rs/commit/0396307b11399db15a65a114a161d7a297c7daa5

Build: ROCKSDB_SYS_STATIC=1 ROCKSDB_SYS_PORTABLE=1 ROCKSDB_SYS_SSE=1 make bench
CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz x4

ordermap
benching Set on raft cluster (channel), nodes: 1, value len:    8       ...    127,518 ns/iter (+/- 26,072)
benching Set on raft cluster (channel), nodes: 1, value len:  128       ...    127,710 ns/iter (+/- 22,830)
benching Set on raft cluster (channel), nodes: 1, value len: 1024       ...    140,110 ns/iter (+/- 20,358)
benching Set on raft cluster (channel), nodes: 1, value len: 4096       ...    190,476 ns/iter (+/- 69,319)
benching Get on raft cluster (channel), nodes: 1        ...     43,487 ns/iter (+/- 4,048)
benching Delete on raft cluster (channel),      nodes: 1        ...    134,027 ns/iter (+/- 29,724)
benching Set on raft cluster (channel), nodes: 3, value len:    8       ...    317,554 ns/iter (+/- 73,531)
benching Set on raft cluster (channel), nodes: 3, value len:  128       ...    287,588 ns/iter (+/- 41,608)
benching Set on raft cluster (channel), nodes: 3, value len: 1024       ...    306,094 ns/iter (+/- 43,780)
benching Set on raft cluster (channel), nodes: 3, value len: 4096       ...    346,072 ns/iter (+/- 80,190)
benching Get on raft cluster (channel), nodes: 3        ...     45,413 ns/iter (+/- 3,882)
benching Delete on raft cluster (channel),      nodes: 3        ...    289,999 ns/iter (+/- 49,032)
benching Set on raft cluster (channel), nodes: 5, value len:    8       ...    320,410 ns/iter (+/- 41,489)
benching Set on raft cluster (channel), nodes: 5, value len:  128       ...    321,195 ns/iter (+/- 41,137)
benching Set on raft cluster (channel), nodes: 5, value len: 1024       ...    342,628 ns/iter (+/- 70,342)
benching Set on raft cluster (channel), nodes: 5, value len: 4096       ...    438,669 ns/iter (+/- 70,880)
benching Get on raft cluster (channel), nodes: 5        ...     48,164 ns/iter (+/- 8,503)
benching Delete on raft cluster (channel),      nodes: 5        ...    360,020 ns/iter (+/- 62,654)
benching Set on raft cluster (tcp),     nodes: 1, value len:    8       ...    216,916 ns/iter (+/- 30,069)
benching Set on raft cluster (tcp),     nodes: 1, value len:  128       ...    211,395 ns/iter (+/- 14,380)
benching Set on raft cluster (tcp),     nodes: 1, value len: 1024       ...    217,181 ns/iter (+/- 19,421)
benching Set on raft cluster (tcp),     nodes: 1, value len: 4096       ...    269,963 ns/iter (+/- 79,548)
benching Get on raft cluster (tcp),     nodes: 1        ...    116,285 ns/iter (+/- 10,843)
benching Delete on raft cluster (tcp),  nodes: 1        ...    223,209 ns/iter (+/- 25,690)
benching Set on raft cluster (tcp),     nodes: 3, value len:    8       ...    558,185 ns/iter (+/- 99,868)
benching Set on raft cluster (tcp),     nodes: 3, value len:  128       ...    581,265 ns/iter (+/- 72,894)
benching Set on raft cluster (tcp),     nodes: 3, value len: 1024       ...    604,005 ns/iter (+/- 102,744)
benching Set on raft cluster (tcp),     nodes: 3, value len: 4096       ...    688,080 ns/iter (+/- 107,601)
benching Get on raft cluster (tcp),     nodes: 3        ...    118,565 ns/iter (+/- 23,202)
benching Delete on raft cluster (tcp),  nodes: 3        ...    595,063 ns/iter (+/- 47,854)
benching Set on raft cluster (tcp),     nodes: 5, value len:    8       ...    751,751 ns/iter (+/- 92,857)
benching Set on raft cluster (tcp),     nodes: 5, value len:  128       ...    676,497 ns/iter (+/- 100,801)
benching Set on raft cluster (tcp),     nodes: 5, value len: 1024       ...    730,324 ns/iter (+/- 131,106)
benching Set on raft cluster (tcp),     nodes: 5, value len: 4096       ...    799,189 ns/iter (+/- 103,713)
benching Get on raft cluster (tcp),     nodes: 5        ...    113,157 ns/iter (+/- 18,586)
benching Delete on raft cluster (tcp),  nodes: 5        ...    688,439 ns/iter (+/- 117,368)
benching tombstone scan with rocksdb    ...      50,672,245 ns/iter (+/- 48,829,050)

FnvHashMap
benching Set on raft cluster (channel), nodes: 1, value len:    8       ...    132,319 ns/iter (+/- 27,589)
benching Set on raft cluster (channel), nodes: 1, value len:  128       ...    132,374 ns/iter (+/- 21,381)
benching Set on raft cluster (channel), nodes: 1, value len: 1024       ...    143,712 ns/iter (+/- 18,494)
benching Set on raft cluster (channel), nodes: 1, value len: 4096       ...    189,253 ns/iter (+/- 55,715)
benching Get on raft cluster (channel), nodes: 1        ...     43,415 ns/iter (+/- 2,458)
benching Delete on raft cluster (channel),      nodes: 1        ...    117,632 ns/iter (+/- 8,877)
benching Set on raft cluster (channel), nodes: 3, value len:    8       ...    289,978 ns/iter (+/- 37,148)
benching Set on raft cluster (channel), nodes: 3, value len:  128       ...    287,917 ns/iter (+/- 50,632)
benching Set on raft cluster (channel), nodes: 3, value len: 1024       ...    304,041 ns/iter (+/- 40,824)
benching Set on raft cluster (channel), nodes: 3, value len: 4096       ...    343,476 ns/iter (+/- 64,309)
benching Get on raft cluster (channel), nodes: 3        ...     43,500 ns/iter (+/- 3,007)
benching Delete on raft cluster (channel),      nodes: 3        ...    305,520 ns/iter (+/- 48,974)
benching Set on raft cluster (channel), nodes: 5, value len:    8       ...    312,754 ns/iter (+/- 57,490)
benching Set on raft cluster (channel), nodes: 5, value len:  128       ...    318,167 ns/iter (+/- 46,408)
benching Set on raft cluster (channel), nodes: 5, value len: 1024       ...    342,766 ns/iter (+/- 44,255)
benching Set on raft cluster (channel), nodes: 5, value len: 4096       ...    385,620 ns/iter (+/- 66,329)
benching Get on raft cluster (channel), nodes: 5        ...     59,532 ns/iter (+/- 3,759)
benching Delete on raft cluster (channel),      nodes: 5        ...    318,489 ns/iter (+/- 43,496)
benching Set on raft cluster (tcp),     nodes: 1, value len:    8       ...    218,944 ns/iter (+/- 35,122)
benching Set on raft cluster (tcp),     nodes: 1, value len:  128       ...    210,884 ns/iter (+/- 23,055)
benching Set on raft cluster (tcp),     nodes: 1, value len: 1024       ...    224,679 ns/iter (+/- 30,564)
benching Set on raft cluster (tcp),     nodes: 1, value len: 4096       ...    274,936 ns/iter (+/- 119,347)
benching Get on raft cluster (tcp),     nodes: 1        ...    112,058 ns/iter (+/- 8,420)
benching Delete on raft cluster (tcp),  nodes: 1        ...    213,167 ns/iter (+/- 18,505)
benching Set on raft cluster (tcp),     nodes: 3, value len:    8       ...    542,626 ns/iter (+/- 41,048)
benching Set on raft cluster (tcp),     nodes: 3, value len:  128       ...    537,007 ns/iter (+/- 42,049)
benching Set on raft cluster (tcp),     nodes: 3, value len: 1024       ...    551,481 ns/iter (+/- 39,178)
benching Set on raft cluster (tcp),     nodes: 3, value len: 4096       ...    609,141 ns/iter (+/- 40,070)
benching Get on raft cluster (tcp),     nodes: 3        ...    114,330 ns/iter (+/- 10,087)
benching Delete on raft cluster (tcp),  nodes: 3        ...    556,115 ns/iter (+/- 35,643)
benching Set on raft cluster (tcp),     nodes: 5, value len:    8       ...    681,369 ns/iter (+/- 124,071)
benching Set on raft cluster (tcp),     nodes: 5, value len:  128       ...    688,422 ns/iter (+/- 124,169)
benching Set on raft cluster (tcp),     nodes: 5, value len: 1024       ...    696,046 ns/iter (+/- 152,589)
benching Set on raft cluster (tcp),     nodes: 5, value len: 4096       ...    788,007 ns/iter (+/- 130,074)
benching Get on raft cluster (tcp),     nodes: 5        ...    113,999 ns/iter (+/- 13,672)
benching Delete on raft cluster (tcp),  nodes: 5        ...    676,558 ns/iter (+/- 99,600)
benching tombstone scan with rocksdb    ...      53,643,533 ns/iter (+/- 36,259,630)

Looks like the over all performance doesn't have much difference. How about bench the OrderMap and HashMap directly (with different key count and size)?

Build: cargo bench
CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz x4
Bench results:
test collections::bench_fnv_map_large   ... bench:     177,892 ns/iter (+/- 32,867)
test collections::bench_fnv_map_little  ... bench:       1,468 ns/iter (+/- 56)
test collections::bench_ordermap_large  ... bench:     109,889 ns/iter (+/- 4,394)
test collections::bench_ordermap_little ... bench:         986 ns/iter (+/- 36)

Code: https://gist.github.com/overvenus/c9046aed30b684b5b0de3718a4dbb8d3

Great, seem that we can use OrderMap now.

Sure, but ordermap is not fully consistency with HashMap, I am working on it.
cc: bluss/ordermap#19


DSND-2612: Fix paper filed discrepancy

Describe the changes
This PR removes the check against the 4th char in the document
id. Previously, if the document id had an X in the 4th char, it
would be considered electronic and so paper filed would be false
and therefore not persisted. However, the Perl doesn't do this.
instead, if the barcode is empty/null but the delta still has
a document_id field that looks like this: "000X....", then
paper_filed is still set to true in the Perl. This commit makes the Java do
the same.
Related Jira tickets
DSND-2612
Developer check list
General

[ ] Is the PR as small as it can be?
[ ] Has the Jira ticket been updated with any relevant comments?
[ ] Is the README up to date?
[ ] Has the POM been updated for the latest versions of dependencies?
[ ] Has the code been double-checked against main?
[ ] Does the code adhere standards in this repository, including SonarQube checks?

Testing

[ ] Do the code changes have unit and integration tests with code coverage > 80%?
[ ] Has the code been tested locally and/or passed the API Karate tests on Tilt?
[ ] Have mandatory manual test cases been run?
[ ] Are extra Karate tests required?
[ ] Are all the test data resources up to date?

Release preparation
Where possible, add links to the respective Jira tickets when an item is checked

[ ] Have these changes been included in a release ticket?
[ ] Are changes required to the docker-chs-development deployment or test data?
[ ] Are any changes required to the environment added to chs-configs?

Notes to the tester
Add any testing hints or instructions here.

CI: No security warnings found

CI: No security warnings found


Kou verify

Official Website: https://www.kocaeli.edu.tr/
You can see our official email domain at (@kocaeli.edu.tr):
Student emails<EMAIL_ADDRESS>

@development1923 bin/main/swot/Compiler.kt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


Unit test info.json population of multiple-measure-dimension classes (and units) from info.json#67

1.QbMultiMeasureDimension column is defined with the names of the existing measures which are defined in csvwlib/csvqb/csvqb/tests/test-cases/configloaders/bottles-test-files/bottles-info.json correctly loaded.
2.QbMultiUnits column is defined with the names of the existing units which are defined in /Users/santhoshthangavel/projects/csvwlib/csvqb/csvqb/tests/test-cases/configloaders/bottles-test-files/bottles-data.csv correctly loaded.
3.URIs for the measures and units are the full URIs, not just their short identifiers.
4.CubeMetadata model has following properties.'base_uri', 'creator', 'description', 'issued', 'keywords', 'landing_page', 'license', 'public_contact_point', 'publisher', 'summary', 'themes', 'title' and 'Uri_safe_identifier'
5.Class CubeMetadata and JSON schema is referred to while adding a test case for all of the possible properties an info.json can have.
6.Creator - test case related to metadata property is fixed.

bottles-test-files directory contains bottles-data.csv and bottles-info.json files which is used to write test cases in /Users/santhoshthangavel/projects/csvwlib/csvqb/csvqb/tests/unit/configloaders/infojsontests.py

@robons Please, Can you review my pull request? Thank you.

I've added a few comments but I think it's a good idea for someone else who haven't been so involved in the process to review it too.
@canwaf could you review it please?

@canwaf finished Rob's PR comments. Please, can you review my PR with the recent changes. Thank you.

Approved for merging.

@canwaf Thanks for approving and merging the commit. @robons Thank you

Bit late to the party but thought I'd mention -
Seems like the .from_dict() method of the CubeMetadata refers to fields you'd find in an info.json. I wouldn't expect this class to have knowledge of info.jsons.
https://github.com/GSS-Cogs/csvwlib/blob/db18ae19261ba92ef0dd72f742c42d77297b6062/csvqb/csvqb/models/cube/cube.py#L44-L60

Seems like the .from_dict() method of the CubeMetadata refers to fields you'd find in an info.json. I wouldn't expect this class to have knowledge of info.jsons.

Fair point, we can move it at some point.


Rust 1.69-nightly binding for catalyst needs review

After a bit of holiday, I came back to the codebase and noted that clean builds of the XCFramework were failing. Multiple pieces appear to have shifted forward to the point that they're incompatible with Rust prior to version 1.70. In https://github.com/automerge/automerge-swift/pull/18, @munhitsu pinned Rust to version 1.69 and a specific nightly version to enable Catalyst support, and now it looks like we might need to review how that all goes together.
https://github.com/automerge/automerge-swift/pull/75 is a related PR that was updating to the latest Automerge crate, which also experienced a variation of this, but the issue also happens when just working from the main branch of the repo and invoking ./scripts/build-xcframework.sh

rustup update versions currently:
rustup update
info: syncing channel updates for 'stable-aarch64-apple-darwin'
info: syncing channel updates for 'nightly-aarch64-apple-darwin'
info: checking for self-update

   stable-aarch64-apple-darwin unchanged - rustc 1.73.0 (cc66ad468 2023-10-03)
  nightly-aarch64-apple-darwin unchanged - rustc 1.75.0-nightly (59edd6705 2023-10-09)

I don't know what issue @munhitsu in Rust/macabi compilation support that we were working around when we pinned to Rust 1.69 nightly: nightly-2023-02-02, so I took a stab (https://github.com/automerge/automerge-swift/pull/77) at the latest Rust nightly build to see if that would compile. It does look like that's doing the trick - at least there's no compilation failure.
Catalyst support is still quite low tier Rust support, so it looks like it does still require a nightly version and -Z build-std to get it operational, although I'll try with a more stock variation to see if that's also more resolved now.

I tried a more aggressive variation in that PR that removed NIGHTLY altogether - waiting for a clean CI environment to make sure it's all kosher to see how it works, but it may be the need to pin down to a specific NIGHTLY has been resolved. Kinda wish I'd figured out what was the fundamental issue earlier, but I'll take it however it works ;-)


List not rendering completely even with a small data set

I am using superselectfield with redux-form to show a dropdown with autocomplete of around 80-100 items. It works fine when scroll position of the dropdown is on top.
But when I scroll through the list only 3-4 options are visible on top.
And it gets worse the more I scroll down; it decreases to only 1-2 items being visible.
Any suggestions on how to fix this.

Can you give an example image or even better a video?

Are you setting the elementHeight property to something larger than the default? I was running into this same issue, then removed elementHeight={58}, and it resolved the issue for me.


Transparancy with each frame

When cropping a gif (the one from the example), there is a problem with each frame that is cropped, it looses color! This could be related to the https://github.com/jnordberg/gif.js component, or by some config, but this should be a major bug on this module.

Let me see!

There seems to be a problem with the GIF parsing library that gifuct-js relies on.

Is this issue fixable in this package, or can we "throw" this issue? it looks like its something related to this issue https://github.com/matt-way/gifuct-js/issues/29. Maybe @jeetiss could help with some integration?

It might solve my problem. I'll look into it.

Is this issue fixable in this package, or can we "throw" this issue? it looks like its something related to this issue matt-way/gifuct-js#29. Maybe @jeetiss could help with some integration?

It might solve my problem. I'll look into it.

Is this issue fixable in this package, or can we "throw" this issue? it looks like its something related to this issue matt-way/gifuct-js#29. Maybe @jeetiss could help with some integration?

My branch feat/fix-color-loose will solve this problem.

Is this issue fixable in this package, or can we "throw" this issue? it looks like its something related to this issue matt-way/gifuct-js#29. Maybe @jeetiss could help with some integration?

Thanks for the tip.

This problem has been fixed. But there is still a small bug with gif.js.If you pass the transparent parameter to gif.js (A built-in behavior). You will get the following effect.

If you need to fix this problem. You need to reset the configuration. Like this:
superImageCropperInstance?.crop({
 gifJsOptions: {
   transparent: null
 }
})

And then you can see the effect:

@STDSuperman is there a way to keep the transparency for gifs without background, but also solve this issue with
 gifJsOptions: {
   transparent: null
 }

??


Release Notes: check support of vert.x

Per chapter 2 and also 6.1, Vert.x is unsupported. Is that really the case?

@Ladicek Indeed id turns out That some Vert.x artifacts are supported thank you for catching this
I have confirmed with Rodney that the only Vert.X components that have the 3.4.2.redhat-006 version in the BOM are supported. I have updated the list release notes and marked the component list as Supported for supported components. The Vertx tech preview artifact list seems to be correct.
Updated RN here: https://access.qa.redhat.com/documentation/en-us/red_hat_openshift_application_runtimes/1/html-single/red_hat_openshift_application_runtimes_release_notes/

Thanks @inoxx03, LGTM now.

This has been fixed.


feat!: refactoring of orderBy parameter

This PR was generated using Autosynth. :rainbow:
Log from Synthtool
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:545c758c76c3f779037aa259023ec3d1ef2d57d2c8cd00a222cb187d63ceac5e
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/cloud/datacatalog/artman_datacatalog_v1beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/datacatalog-v1beta1.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
Skipping: samples/README.md
synthtool > Replaced '\\/\\*\\*\n \\* @namespace google\n \\*/' in src/index.js.
synthtool > Replaced '\\* @see \\[google.cloud.datacatalog.v1beta1.GcsFileSpec definition in proto format\\]{@link https://github.com/googleapis/googleapis/blob/master/google/cloud/datacatalog/v1beta1/gcs_fileset_spec.proto}\n' in src/v1beta1/doc/google/cloud/datacatalog/v1beta1/doc_gcs_fileset_spec.js.
npm WARN npm npm does not support Node.js v12.13.0
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
added 497 packages from 781 contributors and audited 994 packages in 9.161s
found 0 vulnerabilities

npm WARN npm npm does not support Node.js v12.13.0
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> eslint . --fix

synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

cc: @jskeet should I be concerned about this orderBy parameter shifting around?

That looks pretty odd to me, yes. On the other hand, it's a beta API, so you (presumably) haven't released a GA library for it, and so the change being a breaking one isn't too bad.
The YAML change just added the order_by parameter though, so I'm surprised it's changed things about the page size as well.

@alexander-fenster @xiaozhenliu-gg5 this seemed a little weird, any idea why the orderBy field seems to have been dropped by tests, and was moved around in the client; anything I should be concerned about.

@bcoe I don't see any problems here, the change only affected the test code and order of parameters in the documentation (which is defined by some internal gapic-generator's logic). I'm pretty sure there were no compatibility changes.


refactor: do some housekeeping

What does this change?

moves all the individual libs into their own directories to keep things a bit tidier
uses our latest eslint configs
updates our use of peerdeps to the lowest version in the peerdep's range (following https://github.com/guardian/source/pull/1215)
bumps deps
updates logger#debug to run whenever it's not on https://www.theguardian.com

addresses https://github.com/guardian/libs/security/code-scanning/2

Why?
it was getting a bit messy

Coverage increased (+0.2%) to 99.804% when pulling b515f7f8f0eb5079279b1849f097723f533d30bd on housekeeping into f3a5c2b120584f30db2b6c97d08240a8f67cbab1 on main.

@mxdvl is the change to logger ok with you?

I agree with this refactor, it does indeed make things tidier 😄
I wonder if, as we did with Source, we dispense with index.ts files under every feature directory and instead use filenames that reflect the name of the features (e.g. src/cookies/cookies.ts).
We could also use the clever preserveModules workaround to ensure unused features get tree-shaken.
Apologies if I've got the wrong of the stick on anything and I'm just adding noise to this PR 😅

I wonder if, as we did with Source, we dispense with index.ts files under every feature directory and instead use filenames that reflect the name of the features (e.g. src/cookies/cookies.ts).

👍 good idea. i might do in a separate PR though to keep things cleaner?

We could also use the clever preserveModules workaround to ensure unused features get tree-shaken.

this uses pkgu which still uses tsc to build, so no monolithic bundles here


refactor: remove unused  import

Thanks, @clarkdo . I think this is resolved in refacto-app branch :) (Will be merged soon)

ok, good 😸


soci: use Kerberos from darwin.apple_sdk_11_0.frameworks on aarch64-darwin

Description of changes
This is a draft PR to fix current soci in nixpkgs for aarch64-darwin, which currently fails, with this message. I found this issue while working on a local branch to fix nixpkgs rstudio for the same architecture. Please note this is my first PR. Happy to adapt according to your tips :-)
ld: file not found: /System/Library/Frameworks/Kerberos.framework/Versions/A/Kerberos for architecture arm64
clang-16: error: linker command failed with exit code 1 (use -v to see invocation)

Things done

added darwin.apple_sdk_11_0.frameworks to nativeBuildInputs.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[ ] Tested basic functionality of all binary files (usually in ./result/bin/)

24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


Please add register the feature : KubernetesConfiguration

When creating k8s extension in AKS, it shows error as below. Cx and we CSS all see the same object id: 8cbxxxxx, and after checking, it's  ClusterConfigToAKS. Normally cx won't aware of adding this feature.
Registering below solves the issue. Please kindly modify our doc to remind cx to register the feature in perquisition.
https://learn.microsoft.com/en-us/azure/aks/dapr#register-the-kubernetesconfiguration-service-provider

Please contact jiaxinliu if you need any further clarification. Thanks

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

ID: 8d81c8d7-02dd-76bb-0dc5-dc863f937410
Version Independent ID: dc17bdde-5092-78f5-6722-67b40427a63f
Content: Deploy AzureML extension on Kubernetes cluster - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-kubernetes-extension.md
Service: machine-learning
Sub-service: core
GitHub Login: @Bozhong68
Microsoft Alias: bozhlin

@Jessieliu111
Thanks for your feedback! We will investigate and update as appropriate.

@Jessieliu111
Please go through this below link for your reference.
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/azure-arc/kubernetes/quickstart-connect-cluster.md
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Hi team,
Clear enough. Thanks!
Best regards,
Jessie Liu
Support Engineer | Azure IoT&AI Support Team
Phone: (0510) 6665 8360
Mon-Fri 9:00-17:30/7:00-4:00 (UTC+8)
No.111, Linghu Road, Xin Wu District, Wuxi, Jiangsu, China
@.***
Delighting our customers is our #1 priority. We welcome your comments and suggestions about how we can improve the support we provide to you.
From: Naveenommi-MSFT @.>
Sent: Wednesday, October 12, 2022 6:19 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Jessie Liu @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Please add register the feature : KubernetesConfiguration (Issue #99762)
@Jessieliu111https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FJessieliu111&data=05|01|jiaxinliu%40microsoft.com|7c8eaf1b6c374ee0f22d08daac3b273c|72f988bf86f141af91ab2d7cd011db47|1|0|638011667353500973|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fijjI91ePwl%2B1kJlQkuF0jjzcMJ2UKFHUbhbPscF%2FUQ%3D&reserved=0
Please go through this below link for your reference.
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/azure-arc/kubernetes/quickstart-connect-cluster.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fblob%2Fmain%2Farticles%2Fazure-arc%2Fkubernetes%2Fquickstart-connect-cluster.md&data=05|01|jiaxinliu%40microsoft.com|7c8eaf1b6c374ee0f22d08daac3b273c|72f988bf86f141af91ab2d7cd011db47|1|0|638011667353500973|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=zQ2ZvtlQfKdEoMjOMOWI9y7rPezoCvts1qjmsJc0xF8%3D&reserved=0
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F99762%23issuecomment-1275932425&data=05|01|jiaxinliu%40microsoft.com|7c8eaf1b6c374ee0f22d08daac3b273c|72f988bf86f141af91ab2d7cd011db47|1|0|638011667353500973|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=CFwFA60qzYIo64ZL1LO3mKI8UDrrGykdqalArGyto3Q%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAVSBUJWFUFJYAUGSTHFWEO3WC2GARANCNFSM6AAAAAARCBOYSI&data=05|01|jiaxinliu%40microsoft.com|7c8eaf1b6c374ee0f22d08daac3b273c|72f988bf86f141af91ab2d7cd011db47|1|0|638011667353500973|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=JnI3%2BI4lvovBAdXhE0XByQKW1%2BalbcOIM1FMbDRI04E%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


AVOID_USING_CFSETTING_DEBUG is not configured correctly

CFLint outputs the error AVOID_USING_CFSETTING_DEBUG is not configured correctly
Seems like cflint.definition.json is missing the definition.

Fixed also added to xml config file and rule descriptions


My own app is crashed  and got error is fatal error: value failed to bridge from Swift type to a Objective-C type

Hi,
I am assigning yaxis custom max and min values dynamically.When i gave values statically to the Line graph it works fine but when i gave values dynamically to the Line graph  the app crashes at the following point
" _xAxis.labelWidth = widthText.sizeWithAttributes([NSFontAttributeName: _xAxs.labelFont]).width"
Please suggest me why the app is crushed at this point.
Please help me its very urgent for me.
Thanks
Smily

you have to give the crash log... looks like width is nan?

close as no update. Reopen if you are able to reproduce.


[BUG] openapi-nodegen-api-file.yml parsing issues on MacOs

Describe the bug
There seems to be a parsing issue for the generated openapi-nodegen-api-file.yml file on MacOS.
It produces the following error when running the app:
Promise {
  <rejected> [ParseException [Error]: Unexpected characters (
      }
    },
    "servers": [
      {
        "url": "{protocol}://{host}:{port}",
        "variables": {
          "host": {
            "default": "localhost"
          },

The error is fixed by just resaving the file.
To Reproduce
Steps to reproduce the behavior:

Generate the backend from a swagger file with generate-ti
Start the application

Expected behavior
The application rune without issue.
Node & NPM version
Node: 12.18.2
NPM: 6.14.8
Operating system:

OS: MacOS
Version: 11.2.2

Additional context
The error is fixed by just resaving the file.

That's really strange. If it's easily reproducible, that's good at least.
That yaml file is just the duplicated output of the input yml, so I don't think the core reads it for anything. Does the whole generation stop when you see this error, or only when you start the server? Which template are you using for the server generation?

It's not the generation but when running the API with swaggerUI enabled.

ok that sounds very strange but then not an issue with the core perhaps?
Sounds the issue could be located around about here:
https://github.com/acrontum/generate-it-typescript-server/blob/master/src/http/nodegen/routes/swaggerRoutes.ts#L12
Could you confirm please @Celleb  a more detailed jounrey up to when you see this happen please, also as it doesn't happen on generation i would vote this issue be moved to the typescrpt server repo too.

@Celleb as this happens on runtime of a specific set of templates, i will close this ticket unless you have any objections?

Just so we don't forget that this might still be an issue, moved it into the template.
We have very few people testing on mac, so it will be a difficult issue to nail down and debug. It seems fairly low prio (albeit mildly annoying), but I think we should keep a note of it somewhere