RuntimeError: Error llama2_7b_int4_vulkan_rdna2_unknown.vmfb

Error occurs when using model "llama2_7b => meta-llama/Llama-2-7b-chat-hf" with device "AMD Radeon RX 6600M => vulkan://0"
`Configuring for device:vulkan
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in C:\Users\rodri.cache\llama2_7b_int4_vulkan_rdna2_unknown.vmfb.
Saved vic vmfb at  C:\Users\rodri.cache\llama2_7b_int4_vulkan_rdna2_unknown.vmfb
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Loading module C:\Users\rodri.cache\llama2_7b_int4_vulkan_rdna2_unknown.vmfb...
    Compiling Vulkan shaders. This may take a few minutes.

::: Detailed report (took longer than 2.5s):
+18.04518699645996ms: Mapping device id: 0
+26.04389190673828ms: ireert.get_driver()
+130.61070442199707ms: ireert.create_device()
+134.61041450500488ms: ireert.Config()
+150.99406242370605ms: mmap C:\Users\rodri.cache\llama2_7b_int4_vulkan_rdna2_unknown.vmfb
+154.99424934387207ms: ireert.SystemContext created
+2805.0849437713623ms: module initialized
apps\language_models\scripts\vicuna.py:441: UserWarning: To copy construct from a tensor, it is recommended to use sourceTensor.clone().detach() or sourceTensor.clone().detach().requires_grad_(True), rather than torch.tensor(sourceTensor).
::: Detailed report (took longer than 5.0s):
+2.000570297241211ms: Load to device: torch.Size([1, 1])
+3.9997100830078125ms: Load to device: [1, 32, 24, 128]
+3.9997100830078125ms: Load to device: [1, 32, 24, 128]
+3.9997100830078125ms: Load to device: [1, 32, 24, 128]
+3.9997100830078125ms: Load to device: [1, 32, 24, 128]
+3.9997100830078125ms: Load to device: [1, 32, 24, 128]
+3.9997100830078125ms: Invoke function: second_vicuna_forward
Traceback (most recent call last):
File "gradio\queueing.py", line 388, in call_prediction
File "gradio\route_utils.py", line 219, in call_process_api
File "gradio\blocks.py", line 1437, in process_api
File "gradio\blocks.py", line 1123, in call_function
File "gradio\utils.py", line 503, in async_iteration
File "gradio\utils.py", line 496, in anext
File "anyio\to_thread.py", line 33, in run_sync
File "anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 807, in run
File "gradio\utils.py", line 479, in run_sync_iterator_async
File "gradio\utils.py", line 629, in gen_wrapper
File "ui\stablelm_ui.py", line 252, in chat
File "gradio\helpers.py", line 528, in next
File "apps\language_models\scripts\vicuna.py", line 1817, in generate
File "apps\language_models\scripts\vicuna.py", line 443, in generate_new_token
File "shark\shark_inference.py", line 150, in call
return self.shark_runner.run(function_name, inputs, send_to_host)

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "shark\shark_runner.py", line 110, in run
return get_results(

       ^^^^^^^^^^^^

File "shark\iree_utils\compile_utils.py", line 557, in get_results
result = compiled_vm[function_name](*device_inputs)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "iree\runtime\function.py", line 137, in call
File "iree\runtime\function.py", line 162, in _invoke
RuntimeError: Error invoking function: C:\actions-runner\w\SRT\SRT\c\runtime\src\iree\hal\drivers\vulkan\native_semaphore.cc:134: RESOURCE_EXHAUSTED; overflowed timeline semaphore max value; while invoking native function hal.fence.await; while calling import;
[ 1]   native hal.fence.await:0 -
[ 0] bytecode module@1:102460 -
Traceback (most recent call last):
File "gradio\queueing.py", line 388, in call_prediction
File "gradio\route_utils.py", line 219, in call_process_api
File "gradio\blocks.py", line 1437, in process_api
File "gradio\blocks.py", line 1123, in call_function
File "gradio\utils.py", line 503, in async_iteration
File "gradio\utils.py", line 496, in anext
File "anyio\to_thread.py", line 33, in run_sync
File "anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 807, in run
File "gradio\utils.py", line 479, in run_sync_iterator_async
File "gradio\utils.py", line 629, in gen_wrapper
File "ui\stablelm_ui.py", line 252, in chat
File "gradio\helpers.py", line 528, in next
File "apps\language_models\scripts\vicuna.py", line 1790, in generate
File "apps\language_models\scripts\vicuna.py", line 425, in generate_new_token
File "shark\shark_inference.py", line 150, in call
return self.shark_runner.run(function_name, inputs, send_to_host)

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "shark\shark_runner.py", line 110, in run
return get_results(

       ^^^^^^^^^^^^

File "shark\iree_utils\compile_utils.py", line 557, in get_results
result = compiled_vm[function_name](*device_inputs)

         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "iree\runtime\function.py", line 137, in call
File "iree\runtime\function.py", line 162, in _invoke
RuntimeError: Error invoking function: C:\actions-runner\w\SRT\SRT\c\runtime\src\iree\hal\drivers\vulkan\native_semaphore.cc:134: RESOURCE_EXHAUSTED; overflowed timeline semaphore max value; while invoking native function hal.fence.await; while calling import;
[ 1]   native hal.fence.await:0 -
[ 0] bytecode module@0:6470 -`

seems no enough resource in your system.
RuntimeError: Error invoking function: C:\actions-runner\w\SRT\SRT\c\runtime\src\iree\hal\drivers\vulkan\native_semaphore.cc:134: RESOURCE_EXHAUSTED; overflowed timeline semaphore max value; while invoking native function hal.fence.await; while calling import;


Installation error

I used the latest installation package “one-click-installers-tts-4.0” for installation, but encountered an error during the final installation.

Is this error related to an exception during installation? How can I solve this problem?

Hmm, this could be the result of some change in another project, let's see how to fix it. I will attempt to do a clean install, meanwhile, could you please post the whole log if you try to run install command again?
If you right click on the window, you can do Edit > Select All, Copy and Paste here

For reference, here's a clean install that worked for me:
install-log.txt

Several installation improvements have happened, including for fairseq, to ensure that the installation is smoother. Closing the issue for now, if there are more problems, please reopen it.


fix misleading typo in README: notes -> detail

Fixes https://github.com/ruby-grape/grape-swagger/issues/648

thanks @kjleitz


Fix incorrect field type issue

[x] The purpose of this PR is explained in this or a referenced issue.
[x] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[x] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[x] Apache v2 license headers are included in each file.

https://github.com/Azure/azure-sdk-for-go/issues/5339

Can one of the admins verify this patch?

Note that this code is generated from swagger files so modifying it directly won't fix the underlying issue and will be lost in future updates.  @ArcturusZhang can you please follow up on this?

Because this code is auto-generated from swagger, this fix will be overridden in next code generation.
And in the swagger, this enum is clearly declared as string (actually all enum in azure is modeled as string).
The root cause of this error lies in service side.
I submit a new issue in the swagger repo about this.
Closing this pr.

So sad, almost a month passed,  no one is working on that issue https://github.com/Azure/azure-rest-api-specs/issues/7268

So sad, almost a month passed, no one is working on that issue Azure/azure-rest-api-specs#7268

Hi @metrue sorry about the service team does not responds with this issue. Have you tried other api-versions? There are 2016-12-01 and 2016-06-01 api version for backup.
And there is a workaround, I am working on this. After I complete, I will post the workaround here.

@ArcturusZhang
There is not JobsClient in API version 2016-12-01 and  2016-06-01 as I remember.

@ArcturusZhang
There is not JobsClient in API version 2016-12-01 and 2016-06-01 as I remember.

I see. The direct workaround is not very easy. Basically rewriting every method related with this List method. For your personal usage, you can always fork the whole SDK and apply your changes in your fork then use you fork as dependency.

That's what I am doing right now, but I think it's a must-fix bug for this SDK that should be fixed officially.


[Suggestion] Replace the automatic scrolling feature of preview cards for selected notes to the feature of manual scrolling of preview cards for all folder notes

This change can have a great benefit on improving the user experience and speed of execution of tasks, in this way it is possible to access the sequential selection and show the note options simultaneously and with only one long press, instead of the current method that requires 1-long press, then 2-select a note, then 3- Select the required notes, and then 4- Click on Options, all these steps (actions) will be shortened and executed in the same screen (The place).

Oh, damn, nice idea! :)

And all this already sorted/ordered and already filtered! Damn, damn... :))

Could you please elaborate more? @AI-IMED

We can take advantage of long press on any note of any folder to show all notes of that folder in manual scrollable left and right and selectable previews instead of showing only that note.

Please, let me know if you still need this after the latest beta, where the app differently handles dragging, and selection.

I do not hide from you, for me I did not like this method, I began to see more buttons that accompanied this method and that distorted the interface of the application, I do not say my idea is the best, but the current method is not ideal for the user experience

@uli-on thoughts?

Functionally, the current version is working nicely for me re this topic's criticism.
A portion of that, though, that's still contained, is the auto-started slider that's  forcing me to interact with it in order to stop moving and stop distracting my attention from what I actually want to do. I could gladly have it optional but have the drawer slide up completely so I don't need to swipe it upwards in order to see whether all tools are there.

What slider are you specifically talking about? I'm having a bit of difficulty understanding.
Do you mean you want to have all the tools available upon selection, and without having to open the drawer? Or is this about something else?

Sorry, I probably mixed up terminologies. I mean the sliding representation (preview portion) of the selected notes that move along the tool drawer's top edge one by one. With this info, does my post make more sense now?

It does make sense now. Thank you!
Re-reading your answer, it has two parts:

A portion of that, though, that's still contained, is the auto-started slider that's forcing me to interact with it in order to stop moving and stop distracting my attention from what I actually want to do.

You would like a way to stop the auto-scroll behavior for notes.

I could gladly have it optional but have the drawer slide up completely so I don't need to swipe it upwards in order to see whether all tools are there.

You want the drawer to be completely visible, and not requiring to swipe up manually.
Please, correct me if I'm wrong.

I'd love it if the auto-moving representation of notes would be optional (turn on or off in prefs). This would create space to open the tools drawer completely. I, personally, don't need to see my selection of notes reflected at the drawer's top edge, but I also do not want to take anybody that animation thing away, maybe there's use in having it.


a href inside html tag of markdown of notebook not rendering properly in sphinx

Here is a source:

https://github.com/Neuraxio/Neuraxle/blob/master/examples/Introduction to Time Series Processing.ipynb

Here is a result:

https://www.neuraxle.org/stable/time_series_processing.html

The  doesn't render in the result.
Note: that'd be extra cool to support youtube's embedded video iframes directly in notebooks.

Can you please post the relevant Markdown text here?

Here @mgeier:

      [Watch video]


error=HTTP status client error (400 Bad Request) for url (https://api.aliyundrive.com/v2/file/list)

问题描述
run aliyundrive-webdav in WSL1 ubuntu, windows 10 LTSC 21H2 map network drive. When play media file, report
aliyundrive_webdav::vfs: list_files_and_cache failed path=....../file.mp4 error=HTTP status client error (400 Bad Request) for url (https://api.aliyundrive.com/v2/file/list)

重现步骤
always
版本
1.10.1
运行平台
python venv on WSL1 Ubuntu
refresh token 获取方式
网页版
日志
2022-09-28T00:36:17.599027Z ERROR aliyundrive_webdav::vfs: list_files_and_cache failed path=.mp4 error=HTTP status client error (400 Bad Request) for url (https://api.aliyundrive.com/v2/file/list)

I have this issue too. Do you guys has any suggestion?


docs: supplement route English documents

What type of PR is this?
docs
What this PR does / why we need it (en: English/zh: Chinese):
en:  supplement route English documents，add a demonstration for registering static file routes
zh:  补充路由的英文文档，添加用于注册静态文件路由的使用

LGTM


When tom pars=0 tti recv=acous recv vals

When Thomsen parameters are 0 we expect the acoustic solution to be the same as the tti solution. Receiver values are updated to (1/3)(u+u+v) for 3D and (1/2)(u+v) for 2D from u+v originally.
Also corrected the 2D Gzz kernel to check if sin_theta == 0. When sin_theta == 0 the Zero object has no attribute dy. This was already fixed in the 3D kernel.
I tested and confirmed these changes in the attached jupyter notebook.
test_tti_modifications.ipynb.zip
It is basically a merger of the tti example and acoustic example with a layered medium and Thomsen parameters set to 0.

Any updates on this ones? @dimitri-voytan ?


Make wagtail api renderer_classes configurable

This PR originates from https://github.com/wagtail/wagtail/issues/6066  and acts as a proposal on how we can make the renderer_classes configurable from the settings in a similar way as DRF works today.
This is my setting proposal;
WAGTAILAPI_DEFAULT_RENDERER_CLASSES = [
    'rest_framework.renderers.JSONRenderer',
    'rest_framework.renderers.BrowsableAPIRenderer',
]

Some considerations I’ve had while doing this:
Using the built in settings in DRF. (REST_FRAMEWORK.DEFAULT_RENDERER_CLASSES)
This works by just removing the renderer_classes in the View definition, it works well but will most likely introduce bugs for users using custom renderer_classes that will start to affect the wagtail api.
Using the setting design proposed in the originating issue. (WAGTAIL_API.DEFAULT_RENDERER_CLASSES)
I’ve decided against using a multi key dictionary for definition since there are already flat level  WAGTAILAPI_* settings in the api app.

I think it's best to close this issue now, it's been a while and the original issue referenced has since been closed as done.
The solution was different to what's proposed here, instead of a setting there is better documentation for the existing approach that worked as is. See https://github.com/wagtail/wagtail/pull/9902
Thanks for raising this @marteinn and if you feel that it's worth a fresh discussion, can I ask that you raise a new issue with an overview of the proposal.  Alternatively, a discussion about how to better make the API endpoints configurable may be good.


Colours.h file not found

I cloned the repo and after doing pod install, I still get an error saying "Colours.h" file not found. Can you help?

I have the same problem :(


Cleanup scheduler integration tests

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Cleans up the scheduler integration test to use the patterns we settled on as of late.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

L346 L356 are also needed to refine.

Thanks, done.

fixed the flake as well: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/216/pull-kueue-test-integration-main/1514094346826158080

/lgtm
Thanks. @ahg-g

fixed the flake as well

Was the flake only for this PR? or should we cherry-pick to release-0.1?

No cherrypicking needed, this is a flake caused by the test framework.


Lock show/hide by target element

@dobbinx3 looks good? @mauyr

methods show/hide continue working cleaner, e.g.

MnBackdrop.show()
MnBackdrop.hide()

methods show/hide accept option target to lock the work of these methods based on a element, e.g.

MnBackdrop.show({target: element})
MnBackdrop.hide() // dont do nothing
MnBackdrop.hide({target: element}) // works

@darlanmendonca This is a great approach. :+1:

nice!!, merging
thanks


how can deploy or build in windows or linux?

+1

Please see notes on local build here https://github.com/chidiwilliams/buzz/blob/main/CONTRIBUTING.md


SSR broken in latest version of NextJS when setting locale manually

When trying to load translations on the server side from within getServerSideProps, the props are loaded into the store, but fail to be applied to the html. When the client takes over, the translations are loaded. This also causes a mismatch error when fallbackLng is set to supportedLngs as in the example.
The issue is present in a fresh vanilla NextJS installation with files as follows:
pages/_app.tsx
import '../styles/globals.css'
import type { AppProps } from 'next/app'
import { appWithI18Next, useSyncLanguage } from 'ni18n'
import { ni18nConfig } from '../ni18n.config'

const App = ({ Component, pageProps }: AppProps) => {
  useSyncLanguage('en')
  return <Component {...pageProps} />
}

export default appWithI18Next(App, ni18nConfig)

ni18n.config.ts
import type { Ni18nOptions } from 'ni18n'

const supportedLngs = ['es', 'en']

export const ni18nConfig: Ni18nOptions = {
  fallbackLng: supportedLngs,
  supportedLngs,
  ns: ['translation'],
  react: {
    useSuspense: false,
  },
}

pages/index.tsx
import Head from 'next/head'
import { useTranslation } from 'react-i18next'
import { loadTranslations } from 'ni18n'
import { ni18nConfig } from '../ni18n.config'
import { GetServerSideProps } from 'next'

export default function Home() {
  const { t } = useTranslation()
  return (
    <>

        <title>Test nI18n</title>

Hello!

{t('key')}

    </>
  )
}

export const getServerSideProps: GetServerSideProps = async (props) => {
  return {
    props: {
      ...(await loadTranslations(ni18nConfig, props.locale, ['translation']))
    }
  }
}

It's worth noting that the prop ni18n_server is being loaded with the translations, but whatever I try, I've not been able to get SSR to pick them up.
 __ni18n_server__: { resources: { es: [Object], en: [Object] }, ns: [ 'translation' ] }
As can be seen here, the SSR response shows 'key' instead of the text 'Translation in English'. Once the client takes over, the translation is corrected in the view and the error is displayed.

package.json as follows:
{
  "name": "n18n",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@next/font": "13.0.7",
    "@types/node": "18.11.17",
    "@types/react": "18.0.26",
    "@types/react-dom": "18.0.9",
    "i18next": "^22.4.6",
    "next": "13.0.7",
    "ni18n": "^1.0.5",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "react-i18next": "^12.1.1",
    "typescript": "4.9.4"
  }
}

Try to downgrade your<EMAIL_ADDRESS>into<EMAIL_ADDRESS>

@bombillazo can you send your package.json ?

@bombillazo  thanks, I had with ^
I am passing Spanish in loadTranslations and English is displayed. I am using the approach from the example - https://github.com/JCQuintas/ni18n/blob/main/examples/simple/pages/server-side-props.js. Was there such a problem?
Did you use useSyncLanguage?

I had exactly the same error and I just used the @bombillazo suggestion, use the exact version of the react-i18next.
"react-i18next": "12.0.0",
But I think it's a temporary solution because the react-i18next will continue to growing if the next.js will change API later

It seems someone reported a related problem in the react-i18next repo, possibly caused by the same bug. 👀

Hi, as far as I can tell this seems to have been caused by a version of i18next. Make sure you are using the latest versions below. I won't add these to peerDependencies just now because it would be a breaking change, and I need more time to research.
    "i18next": ">=22.0.0",
    "react-i18next": ">= 12.0.0"

Make sure you are using the latest versions below.
    "react-i18next": ">= 12.0.0"

Hey JC. Unfortunately, using any version greater than<EMAIL_ADDRESS>causes the problem. I am on the latest version of i18next with<EMAIL_ADDRESS>and this was the only way to get it to work. I recommend using the exact version while the bug is fixed.

Sorry, I misread, this is happening in development only. I was testing in production. Will try to debug it then


Support receiving File to srcs declared with ctx.action.declare_directory

Presently, if the srcs attr receives a generated File declared with ctx.actions.declare_directory the directory path is passed directly to the compiler, which fails. By using ctx.actions.args we get the directory contents auto-expanded:

Each directory File item is replaced by all Files recursively contained in that directory, unless expand_directories=False is specified.

https://docs.bazel.build/versions/master/skylark/lib/Args.html#add_all
For context, the D protoc plugin produces non-predictable outputs, which makes declare_directory necessary, rather than declare_file. To support building the generated files, rules_d needs to support both file and directory File to its srcs attr.

Could you add a test using the generated files? This will clarify your use case and assure it will not break in the future.

Sure, test added! I should have done that in the first place, sorry

Is this ever likely to merge?

Can you check the conflicts?

Done


GUI buttons don't appear in Linux on low screen resolutions

My laptop has a 1336x768 resolution, and it seems that the GUI is not resizing right at that resolution. The buttons at the bottom of the main window don't appear, even when the window is maximised. When I use an external monitor with a higher resolution, the buttons appear when I move the window to it and maximise it, but they disappear again when I resize the window below a certain vertical size.
On startup, the following error messages appear in the terminal:
(fluxengine-gui:25061): Gtk-WARNING **: 11:04:55.514: Negative content height -7 (allocation 5, extents 6x6) while allocating gadget (node button, owner GtkButton)

(fluxengine-gui:25061): Gtk-WARNING **: 11:04:55.514: Negative content height -7 (allocation 5, extents 6x6) while allocating gadget (node button, owner GtkButton)

(fluxengine-gui:25061): Gtk-WARNING **: 11:04:55.514: Negative content height -7 (allocation 5, extents 6x6) while allocating gadget (node button, owner GtkButton)

(fluxengine-gui:25061): Gtk-WARNING **: 11:04:55.514: Negative content height -7 (allocation 5, extents 6x6) while allocating gadget (node button, owner GtkButton)

(fluxengine-gui:25061): Gtk-WARNING **: 11:04:55.514: Negative content height -7 (allocation 5, extents 6x6) while allocating gadget (node button, owner GtkButton)

That's weird --- at 768 height, there should be loads of space. Can you attach a screenshot?
(There is a known issue where if the window is too small for the content, it's not handled particularly gracefully. I'm still thinking what to do about that.)


Code example for rounded inputs was changed to wrong example.

https://purecss.io/forms/#rounded-inputs
The code example I'm seeing is for disabled inputs.

I'm not sure how I got it in this state, but it looks like something changed the element in the DOM. The source came across the network correctly, but by the time I reached that part of the page, it had been changed.

I can't seem to replicate this. What browser and device are you using?

I don't know. It may have been a bug in my browser or an extension.


Consolidate pillar and allow for multiple splunk endpoints

All designed to be backwards compatible
I suggest hitting view on the pillar.example, the way github is doing the diff doesn't make the new format obvious.

looks ok


Update terra-functional-testing to 4.0.0, updated ALL screenshots

Summary

What was changed:
Updated to consume the new version of terra-functional-testing (4.0.0).
Why it was changed:
For compatibility with Apple silicon devices.
Testing

This change was tested using:

[x] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-XXXX

Thank you for contributing to Terra.
@cerner/terra

Do we have a timeline on when this will get merged? TIA.

Tests work on intel devices


Tower not cycling during RACE sessions

The tower is not swapping between the different mode of display during the race

IT F***ING WORKED WOOOOOOOOOOOOOOO


Start a animation with javascript function

Hi
Thanks for your amazing work :-)
I'd like to start a specific animation with a javascript function ( event listener : click )
How can I do this ?
Thanks for your help :)

@quentin35
If you don't want to initialize AOS on DOMContentLoaded event, you can pass your own event name and trigger it whenever you want. AOS is listening for this event on document element.
AOS.init({
startEvent: 'someCoolEvent'
});

Hey @quentin35, thanks! Unfortunately this is not that kind of library @quentin35, the only workaround might be what @georgetrad proposed, but that would init animation on all elements at once.


Automatic PR from RestAPI-PR1384

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1384

Codecov Report

Merging #1294 into master will decrease coverage by 0.05%.
The diff coverage is 15.09%.

@@            Coverage Diff             @@
##           master    #1294      +/-   ##
==========================================
- Coverage   55.83%   55.77%   -0.06%
==========================================
  Files        2622     2624       +2
  Lines       70003    70105     +102
==========================================
+ Hits        39086    39101      +15
- Misses      30917    31004      +87

Impacted Files
Coverage Δ

...tions/server_azure_ad_administrators_operations.py
15.31% <ø> (ø)
:arrow_up:

...ure/mgmt/sql/models/transparent_data_encryption.py
71.42% <ø> (ø)
:arrow_up:

azure-mgmt-sql/azure/mgmt/sql/models/__init__.py
100% <100%> (ø)
:arrow_up:

...e/mgmt/sql/models/database_blob_auditing_policy.py
35.71% <100%> (ø)
:arrow_up:

...-mgmt-sql/azure/mgmt/sql/models/database_update.py
15.62% <15.62%> (ø)

...re/mgmt/sql/operations/elastic_pools_operations.py
9.46% <3.84%> (-0.68%)
:arrow_down:

...t-sql/azure/mgmt/sql/models/elastic_pool_update.py
35.71% <35.71%> (ø)

.../azure/mgmt/sql/operations/databases_operations.py
18.58% <6.45%> (-0.4%)
:arrow_down:

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 7b54091...fe145c2. Read the comment docs.

in https://github.com/Azure/azure-sdk-for-python/pull/1413


Error while loading some of the scenes

Operating system or device - Godot version:
Ubuntu Linux, f55211ae0dc202cc015c247495af8e05af81b24b
Issue description:
It's impossible to open some of the scenes in newest build:

Steps to reproduce:

Download reproduction project
Try to open Enemy.tscn

Link to minimal example project:
FSMDemo3.zip

72436956ddcdfdcb9edea5c447363391c23552c4 is the first bad commit

Enemy.tscn contains PoolFloatArray which should be PoolRealArray
IMO it's not an issue because Pool*Array is introduced in 3.0.

Yes, that should be one of the last scene-breaking changes for 3.0 :) The PoolFloatArray was wrong, it's PoolRealArray everywhere else in the source, so it was changed accordingly.
Run s/PoolFloatArray/PoolRealArray/g on your scene files and it should work fine.

oh I see, thank you guys :+1:


Dashoffset not properly animating in ios 8.1

For some reason when animating a svg  dashoffset the animation will stutters.  Settings the values through css works fine, but not when using the animate api.
<svg width="200" height="200" viewBox="0 0 100 100" preserveAspectRatio="xMidYMid">
    <circle cx="50" cy="50" r="40" stroke="white" fill="none" stroke-width="10" stroke-linecap="round"/>
    <circle id="line" cx="50" cy="50" r="40" stroke="#808080" fill="none" stroke-width="6" stroke-linecap="round"/>
  </svg>

  <script>
    var line = document.getElementById("line");

    line.animate(
      [
        {strokeDashoffset:0},
        {strokeDashoffset:502}
      ],
      { duration: 2000, iterations: Infinity }
    );
    line.animate(
      [
        {strokeDasharray:"150.6 100.4"},
        {strokeDasharray:"1 250"},
        {strokeDasharray:"150.6 100.4"}
      ],
      { duration: 2000, iterations: Infinity }
    );
  </script>

or jsbin
http://jsbin.com/wecabo/1/edit?html,output

Your demo doesn't work as you're using href rather than src on the script element.
But yeah, there seems to be a bug here. The polyfill is only setting strokeDasharray to be the first & final values - no interpolation.

Seems this property is missing a property handler. It should use the same one as "left" I think.
http://jsbin.com/paxokuyifi/1/edit?html,output - hacked this together. Seems to work.

@jakearchibald Im assuming in not missing anything - is this what your fix was? #85

Looks right to me

Can we get a release with this code in it?


Fix logging console log image.

The first image on the logging document was intended to show the custom message written to console, but the message is not shown at all. Update the image.
Reference:
https://github.com/aspnet/Docs/blob/b48611fffd5042a01ce4e41738a84ab9d5895d0f/aspnet/fundamentals/logging.rst

Hi @karataliu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

@ardalis can you take a quick look?

looking

:shipit:


Tweaked cache key when getting localizer from basName/location:

Cleaned up code a little to make it easier to understand (maybe)
#138

@ryanbrandenburg  @rynowak

:shipit: - seems simple enough. Should we have a test that we can retrieve the same one twice (with/without the resource name set)?

Sounds like a good ide, will add a test.

:shipit: simple cleanup it seems to me.


How to put photo between name and info columns ?

I know it's not expected to put photos to CVs in US, but in Europe we usually do that.
I though about making it \doublecol and put the image there but it's not located correctly. I'm now trying with the \singlecoland try to tweak it but any input is welcome.

What does it look like with either option? I'd imagine \doublecol to be harder to adjust to since it doesn't leave much room on the header for you to insert a photo.

With \doublecol I did those modifications:
\usepackage{graphicx}

...

\def \photo {\includegraphics{Dwight_Schrute.jpg}}

...
%In double col definition
...
<EMAIL_ADDRESS>        {} & {\small#1} \\
        {} & {\small#2} \\
        {\photo} & {\small#3} \\ %photo added here
        {} & {\small#4}
      \end{tabular}

The outcome is this:

The image is from wikipedia: https://en.wikipedia.org/wiki/Dwight_Schrute

I just pushed a branch that has semi-hackish support for a circular photo to the left of the name; you can likely follow the same logic within in order to put it between the name and contact info. Personally, I think it suits the left side pretty well.
Check it out here: 3d7ad27

I looked at the output of the branch. Looks good to me, I think I can close this issue now. Thanks for your help.


Support for HTML5 input elements

Hey Everyone,
I thoroughly enjoy Axis and this has greatly improved my workflow and I cannot thank you all for that. I think it would be beneficial if we could extend the functionalities with styling radio and slider to include different input5 types (eg: number), dropdown and checkbox maybe? I think it would help ensure same looks across browser and lots of time usually spent looking those design properties up.
Thanks!

Hey @Infinitism. Agree!
Here is state of the union on these things currently.
Dropdown (select)
Ready to go (just needs to be added to axis with tests):
http://codepen.io/dbox/pen/ZGZzmW?editors=110
Demo here (demo is scss but same ideas for stylus):
http://s.codepen.io/dbox/debug/xwNMZd
Checkboxes
Needs conversion to stylus, a little more testing, and axis tests:
http://codepen.io/dbox/pen/dYBBxb
Number
No progress.
Happy to accept a PR on any of these, otherwise I will get to it when I can.
Hope this helps a little!

Hey @dbox thanks for a quick response. I'll try to get on some of those features implemented with Axis and reach out to you guys with a PR soon.
Thanks for the links:)

available in 2.0 (sans number)


Callout: Responsiveness updates

This PR updates Callout to be consistent with Upsell by:

Expanding the button link when on small display
Centering text when on small display
Shifting icon from left of text to above of text on small display
Moving primary button to top when on small display

Before:

After:

Test Plan
Updated snapshot tests, verified that component looks as expected.

Hey Erica, on md we should center the icon. Now it's going to the top.


Find in the Blockly workspace doesn't center on blocks

The new find feature doesn't center on the blocks it finds; sometimes they end up in the corner of the workspace and are very hard to spot
If the block is on the edge of the workspace, we should center on it.

This is not blocking release. If you get to it lets, take the fix.


Add ability to get parent for an Api and fix OOM when serializing large objects

Add field to expose the parent docId for a given Api
Encountered an OOM when serializing DotNetCatalog. Fix this by adding methods to write/read from stream rather than return a string.

LGTM


Is your feature request related to a problem?

Describe the solution you'd like
New Provider and SDK.
Describe alternatives you've considered
No response
Additional context
If you don't have time, I can provide support.

We also have a need for sign message

Also need


Support the use command?

A lot of text adventures allow players to use an item in their inventory to some effect. Currently the drop command allows players to use an item (such as purchasing a ticket with a dollar in the Carnival game), but that seems counter intuitive as dropping something usually indicates discarding an item, rather than using it for some benefit.
An example I was thinking of:
$ use dollar on ticket stand
=> You purchase a ticket with the dollar. You can now enter the main square!
This is a really cool project!

that's really helpful! I'm not super familiar with patterns as i don't actually spend a ton of time playing them myself 🤣 that also simplifies/expends logical possibilities in terms of locking mechanisms and payment etc.

implemented on master, the engine is still bare but the command is there


Release 0.4.2

Codecov Report

Merging #108 (254ce7d) into main (e57ce2e) will decrease coverage by 0.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   90.75%   90.05%   -0.71%
==========================================
  Files          16       16
  Lines        1471     1578     +107
==========================================
+ Hits         1335     1421      +86
- Misses        136      157      +21

Impacted Files
Coverage Δ

google_nest_sdm/google_nest_subscriber.py
64.49% <0.00%> (+3.66%)
:arrow_up:

google_nest_sdm/device_manager.py
100.00% <0.00%> (+5.88%)
:arrow_up:

Continue to review full report at Codecov.

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


Remove contributors list from template and update README files

Contributors list is already out of date. We will need to explore a different way of crediting contributions.
This PR:

removes the list from the component view template
updates all the README files with latest changes

has there been work on the component template?

Possibly related to https://github.com/alphagov/govuk-frontend/commit/4c01202842352f6f8488065cab4d9dd1db976b1c#diff-841b0a984418ab47f365fb13d1a0ac89?

ok will fix that first then and check markdown compiler


add makeHorizontalCoordinates(with geographicCoordinates: GeographicCoordinates) to CelestialBody Protocol

A very small API suggestion...
The Sun object includes the function makeHorizontalCoordinates(with:)which is convenient for folks wanting to plot the location of the sun in the sky.  The CelestialBody protocol already has all the information one needs to compute rectangular coordinates (RA and HA), so why not add a default implementation of the function to that protocol so all objects (not just the Sun) get access to it?
While I'm here, I'd just like to say that this library is AWESOME!  It's already saved me tons of time and has taught me a lot about celestial coordinates as well. I'm looking forward to using it in my next app. Thanks!!!

Hi. Thanks for your message. Indeed, this could be added.
I've spent so much time trying to factor out everything, and play with this powerful Swift pattern which is protocol-oriented programming. But sometimes, I missed a few details in the global view.
I am glad it helped you in different ways. Don't hesitate to support it on patreon, it if you which. ;-)

I am currently moving the func makeHorizontalCoordinates(with geographicCoordinates: GeographicCoordinates) -> HorizontalCoordinates function from the Sun class to the CelestialBody protocol.
The little change is that it uses self.equatorialCoordinates and not self.apparentEquatorialCoordinates
Hence to be consistent, I kept the method in the Sun class, with the existing implementation, but renaming it more correctly makeApparentHorizontalCoordinates(with geographicCoordinates: GeographicCoordinates) -> HorizontalCoordinates

I'll wait for closing #100 and #101 before making a patch release.

Already done.


Error: Failed to parse PDF document (line:0 col:16 offset=8): No PDF header found

Hi there,
I am using pdf-lib module to edit header and footer of the pdf file.
I am using this module in spfx webpart.
When user clicks on pdf file from document library, then custom header and footer should get added in the pdf.
Below is the code snippet for the same but I am getting error as Error: Failed to parse PDF document (line:0 col:16 offset=8): No PDF header found.
const existingPdfBytes = file.pdfFileName; (Note - file.pdfFileName is the name of my pdf)
// Load a PDFDocument from the existing PDF bytes
const pdfDoc = await PDFDocument.load(existingPdfBytes)
// Embed the Helvetica font
const helveticaFont = await pdfDoc.embedFont(StandardFonts.Helvetica)
// Get the first page of the document
const pages = pdfDoc.getPages()
const firstPage = pages[0]
// Get the width and height of the first page
const { width, height } = firstPage.getSize()
// Draw a string of text diagonally across the first page
firstPage.drawText('This text was added with JavaScript!', {
x: 5,
y: height / 2 + 300,
size: 50,
font: helveticaFont,
color: rgb(0.95, 0.1, 0.1),
rotate: degrees(-45),
})
// Serialize the PDFDocument to bytes (a Uint8Array)
const pdfBytes = await pdfDoc.save()
}
}
Task to achieve - Create spfx webpart. Add custom header and footer on click of pdf document.
Reference link - https://github.com/Hopding/pdf-lib
Quick help would be appreciated.

Issue is closed.
Converted pdf to arraybuffer and it worked.
const url = "https://XXXX.sharepoint.com/sites/XXXX/printToPdf/pdf_customHeaderFooter.pdf";
const existingPdfBytes = await fetch(url).then(res => res.arrayBuffer())
var bytes = new Uint8Array(existingPdfBytes);
const pdfDoc = await PDFDocument.load(existingPdfBytes)

So what of the "var bytes = new Unit8Array(existingPdfBytes)" line you referenced?

I am also curious, were you able to resolve?

negative, I instantiated a shell script to handle what I needed but would have liked to use pdf lib

Unfortunately I ended up with the same error. When I run your code it tells me that PDFDocument.load() expects a Uint8 or ArrayBuffer. So I did this instead.
   const url = '../dump/template.pdf'     const existingPdfBytes = await fetch(url).then((res) => res.arrayBuffer)     var bytes = new Uint8Array(existingPdfBytes)     const pdfDoc = await PDFDocument.load(bytes) // <- using bytes instead of existingPdfBytes     const pages = pdfDoc.getPages()     pages[0].drawText('asdadasdasd')     const pdfBytes = await pdfDoc.save()
This unfortunately gives me the same error

I'm still stuck on this. The compiling of nextjs breaks something with loading the PDF file and I can't get it to load a pdf from a url.

Somehow I got a very different result when running in nextjs dev vs running after a nextjs build:
  const save = await currentPdf.save()
  const saveBase64 = await currentPdf.saveAsBase64()
  console.log({ save, saveBase64 })
  //  "saveBase64" has a value of "T0Y="
  // Load it back in as "save" works
  currentPdf = await PDFDocument.load(save)

I had the same error, in my case the solution was just to import the .PDF file outside the async function (I'm using pdf-lib), getting this:
`
import pdf from '../assets/docs/plano_de_trabalho_teletrabalho_ordinario_inicial_e_renovacao.pdf'
async function modifyPdf() {
const url = pdf
const existingPdfBytes = await fetch(url).then(res => res.arrayBuffer())
const pdfDoc = await PDFDocument.load(existingPdfBytes)
const helveticaFont = await pdfDoc.embedFont(StandardFonts.Helvetica)

const pages = pdfDoc.getPages()
const firstPage = pages[0]
const secondPage = pages[1]
const { width, height } = firstPage.getSize()

}
`

I have also encountered this issue before, but I managed to solve it. The key to the problem is the file path referenced in the fetch, it should not be a relative path.
const url = '/1.pdf'; const existingPdfBytes = await fetch(url).then((res) => res.arrayBuffer());

I have also encountered this issue before, but I managed to solve it. The key to the problem is the file path referenced in the fetch, it should not be a relative path. I moved the PDF file to the public folder, and the response returned the correct content instead of HTML. You can view the response in devtool network to check whether the requested file path is correct.
- const url = './assets/sample.pdf';
+ const url = '/sample.pdf';
const existingPdfBytes = await fetch(url).then((res) => res.arrayBuffer());
const bytes = new Uint8Array(existingPdfBytes);
const pdfDoc = await PDFDocument.load(bytes);

Did you write unit test for this? I am getting file.arrayBuffer() is not a function. Anyone?

If use puppeteer convert the html to pdf file whih will be stored in local OS or Docker, pdf-lib using the fs.readFileSync, and the merge local pdf-file to one pdf which can be upload the AWS s3 successfully.  Codes like the follwoing:
const mergedPdf = await PDFDocument.create();             // 逐个任务进行pdf处理, 进行pdf合并             for (let j = 0; j < examStudents.length; j++) {                 const examStudent = examStudents[j];                 const htmlUrl =${htmlUrlDomain}?token=${task.token_code}&exam_id=${task.exam_id}&student_id=${examStudent.student_id};                 const pdfName = ${examStudent.username}_${examStudent.remarks}                const pdfPath = await htmlToPdf(htmlUrl, taskMergePdfDir, pdfName, browser);                 if (pdfPath) {                     const existingPdfBytes = fs.readFileSync(pdfPath);                     const document = await PDFDocument.load(existingPdfBytes);                     const copiedPages = await mergedPdf.copyPages(document, document.getPageIndices()); 		            copiedPages.forEach((page) => mergedPdf.addPage(page));                  }             }             const mergedPdfFile = await mergedPdf.save();             fs.writeFileSync(taskMergePdfPath, mergedPdfFile);


updated Code of conduct

from https://www.contributor-covenant.org/ version 2.1

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

Thanks @mustaphazorgati . Awesome 🤩 🤩 🤩


GR: Add test universe generation script and tests for patch generation

Made a go script that creates a universe file for the MockResolutionClient (#909) from the current real deps.dev & OSV data used during in-place / relock / relax computation.
This has let me add some real-world in-place & relax patch generation test cases to test for regression without it breaking constantly due to new versions. I've only added one universe because the files are pretty large.
The in-place/relax patch generation tests are basically end-to-end tests - it's not really written in a way that makes testing only the patch computation functionality possible. (This might be a code smell, but I don't think it's practical to structure the code in such a way).

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

Project coverage is 64.53%. Comparing base (e2adeb5) to head (5a17651).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #967      +/-   ##
==========================================
+ Coverage   63.71%   64.53%   +0.82%
==========================================
  Files         146      146
  Lines       11958     9294    -2664
==========================================
- Hits         7619     5998    -1621
+ Misses       3875     2842    -1033
+ Partials      464      454      -10

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


Static Verification: Add support for comparing Spring MVC implementation to Hand-written RAML documents

The generator will be used to build a RAML model from the endpoints implemented in a project. This will be then compared to an existing RAML file that is hand written for discrepancies.
This check should be available at runtime based on beans discovered by the application context, as well as statically based on compiled code. This latter could present some issues in dependency detection

Basic Static analysis complete. More rules will be added in future releases


Bindings does not pass Tuple input parameters

Hello
Using Net7.0 and AzureFunctions v4 the following type of Function declaration does not work as expected runtime. tuple.Item1 and tuple.Item2 are both null in the body of the function.
(This worked fine on Net6 in-process)
Sending one of the tuple items individually works fine. It is when coupling them in a Tuple that doesn't.
[Function(nameof(SendMsgToDynamicQueue))]
[ServiceBusOutput("{tuple.Item1.QueueName}", Connection = "CONNECTION_PREFIX")]
public static string SendMsgToDynamicQueue(
    [ActivityTrigger] Tuple<string, MetaDataMessage> tuple,
    FunctionContext executionContext)
{ }

        await orchestrationContext.CallActivityAsync<string>(
            nameof(SendMsgToDynamicQueue),
            (msgpayload, metadata));

I've just run into the same issue, with .NET 6, converting a function app to isolated functions v4 and passing ValueTuple parameters to an Activity function no longer works

Duplicate of #1218
(They are both serialization issues).

In-proc and out-of-proc use different default JSON serialization libraries (Newtonsoft.Json and System.Text.Json respectively). STJ may not support deserialization of Tuple.
Also, this line (msgpayload, metadata) is a ValueTuple, which is different from a Tuple. https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/builtin-types/value-tuples#tuples-vs-systemtuple


Error messages

Every time that my code tries to print an error message, I get an error, and the message is shown in the console instead. How can a print it so the user can see it?

hey @sidneylopes. Each time you ask a question like My code doesn't work and you don't share your code I can't really help you because I've never seen your code. Can you please be more respectful of my time and provide your code first.

@sidneylopes I was able to replicate and fix the issue, please check https://github.com/troublediehard/magento-react-native/commit/cac6929cca36ae2c097ca86a206153e9d09dc7cb

I tried, but no different results

@sidneylopes what exactly did you tried?
I was able to reproduce the issue and the fix is working for me.

in the index, I changed "Promise.reject(errorResponse);" to "reject(errorResponse);", but nothing happened. Should I try printing something else?

Probably js bundler cache, try to restart it and remove the app from the simulator.

It worked, thanks. Sorry about yesterday!

no problem, glad you solved it.


"Telega", "Telega", "Buștenari", "Doftana", "Melicești" and "Țonțești" villages are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Type "Telega" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Telega, PRAHOVA (Telega)".

Step 2
Type "Boșilcești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Boșilcești, PRAHOVA (Telega)".

Step 3
Type "Buștenari" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Buștenari, PRAHOVA (Telega)".

Step 4
Type "Doftana" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Doftana, PRAHOVA (Telega)".

Step 5
Type "Melicești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Melicești, PRAHOVA (Telega)".

Step 6
Type "Țonțești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Țonțești, PRAHOVA (Telega)".


Line 33 of specification.markdown contains "XML Declaration"

Line 33 contains "XML Declaration" with an capital 'D'.
Is that intended to be the case? (I am not objecting if that is how it was intended to be.)

Seems to be intentional after I read some more of the specs.


Scrollbar color in tree view

Hi, I noticed this is the scrollbar's appearance in the tree view, using a dark syntax scheme:

Would it be possible to have it match the color and behaviour of the main editor scrollbars?
Thank you and keep up the great work,
N

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/15401. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Adjust for change in state change publishing in homegear

Variable state changes are no longer published to "event", but are now published to:
homegear/HOMEGEAR_ID/plain/PEERID/CHANNEL/VARIABLE_NAME
homegear/HOMEGEAR_ID/json/PEERID/CHANNEL/VARIABLE_NAME
homegear/HOMEGEAR_ID/jsonobj/PEERID/CHANNEL/VARIABLE_NAME
Probably further changes in the homegear util are required.

Fixing for variable state changes in new homegear versions.

Coverage remained the same at 100.0% when pulling ee1eb46d7893a9d5901b74cbe2f44c62d259e923 on Aietes:patch-1 into 2af440ec2e611f712200fbc95d9e9ae82a286084 on Shirk:master.


Fix bug with bindings inside of  elements

Fixes #2441
I didn't add an SSR-specific test because it's evident that the problem is with <title> elements in general and the test for <title> was broken before the fix was applied.

Just wondering when this will be released? I can't find a release plan within the scope of this project.

Just wondering when this will be released? I can't find a release plan within the scope of this project.

We will do a release in the next day or two. We don't have a regularly scheduled release process yet, and do releases on-demand, but may move to a weekly schedule at some point.


[Anaconda] Address vulnerabilities GHSA_jfhm_ghh_2f97;  GHSA_94vc_p8w7_5p49;  GHSA_v68g_wm8c_6x7j

Address vulnerabilities as mentioned in issue #90 in Anaconda Dev container.

Cryptography package https://github.com/advisories/GHSA-jfhm-5ghh-2f97
Imagecodecs package https://github.com/advisories/GHSA-94vc-p8w7-5p49
Transformers package https://github.com/advisories/GHSA-v68g-wm8c-6x7j

Updated to the required versions in Dockerfile and test.sh files. Was able to run tests successfully.

Wrongly raised pull request.
Needed only one vulnerability remedial per pull request.
Thus, closing this PR.


Linewise iteration over compressed file

With gzip.open() the following is possible:
with gzip.open(filename, 'rb') as f:
    for line in f:

How do I achieve this with lz4framed? (And I think this question is general enough that this should be integrated into the library)
There's ByteIO, but this only supports an initial chunk. The question is essentially: How do I stream the chunks from lz4 into a ByteIO so that I can read line-by-line?

See either gzip implementation on how to do this or e.g. something like (just an example, won't debug it myself if doesn't work properly!):
https://gist.github.com/vtermanis/739e34b8a4e4b323932996a288fa1001#file-lz4framed_file_reader_example-py-L126

You're welcome - maybe worth looking at exactly how you want/need to read the data and make a more simple specific solution for your application (e.g. using the compress_(begin|update|end) calls directly).
If you think there's a bug in the Compressor class (not part of example), feel free to raise as another issue.
Ultimately, because I work full-time (we open-sourced this project just so others can use it) and because this module is just supposed to expose the C API, don't expect too much support in terms of adding stuff on top, sorry.
I'll close this issue for now. If you or anyone else come up with something you think is worth adding to this module, feel free to open a pull request!

@DreamFlasher - I found a bug in the gist I posted (SEEK_END for BytesIO) - it should not stop reading when decompressing anymore.

Amazing, thank you very much!


Only perpetual gauges to be target of pool-incentives

closes #230

Codecov Report

Merging #296 (f52c694) into main (e3f1cae) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main     #296   +/-   ##
=======================================
  Coverage   18.37%   18.38%
=======================================
  Files         134      134
  Lines       21094    21099    +5
=======================================
+ Hits         3877     3878    +1
- Misses      16580    16582    +2
- Partials      637      639    +2

Impacted Files
Coverage Δ

x/pool-incentives/keeper/distr.go
75.67% <33.33%> (-4.04%)
:arrow_down:

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 e3f1cae...f52c694. Read the comment docs.

@antstalepresh Can you add a test for this?

Closed in favor #298


Use tox-uv in the CI

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

Project coverage is 85.04%. Comparing base (b51427c) to head (3c25673).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   85.04%   85.04%
=======================================
  Files          45       45
  Lines        2508     2508
  Branches      495      495
=======================================
  Hits         2133     2133
  Misses        309      309
  Partials       66       66

Flag
Coverage Δ

pytest
85.04% <ø> (ø)

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.


Exclude from the BackFace function, objects that have a specific suffix

When our objects are petals or leaves we do not want to apply a backface culling by mistake. For this reason, it would be interesting to be able to exclude objects with a specific suffix from the BackFace function.
Example: Flower_01_leaves_reticobfc

mmh I don't think it's the right way to do. You better have to put your objects in a dedicated collection, and just not select them when clicking on BackFace button, isn't it?

I've tried to make better use of the collections to do that and indeed it's working pretty well, thanks.


Link in guide redirects to the same page

https://gyazo.com/915942dba9dc7773c4034229d8e27d18
"If you have a developer (“PANDA” or “SNAKE”) console, check out the standard Guide"
The link redirects to the same page.

https://github.com/Plailect/Guide/commit/f038e599a3337311b632175e0282ce57beb5b23d


Disable base64_encoded by default

Since all new clients created are not base64 encoded, it does not make
sense to keep this turned on by default.

Setup wizard fine.


[Task]

Description
添加push方式的数据采集，类似Prometheus的pushgateway
为什么需要添加push模式？

方便用户自定义需要监控的数据
对于内网的应用，hertzbeat部署在外网无法访问内网的应用

Task List
具体描述：

[ ] 前端创建对应的push monitor，生成monitorId作为push模块的key
[ ] 新增push module，暴露接口供用户调用push监控数据，push模块存储和定时删除数据。
[ ] collector模块定时拉取push模块的数据，并复用数据采集的部分逻辑，进行存储（内存和磁盘）和删除（暂时使用定时删除，有能力可以根据当前使用内存进行判断是否需要删除）
[ ] 前端调用展示用户自定义的数据

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Description
Add push data collection, similar to Prometheus's pushgateway
Why do you need to add push mode?

It is convenient for users to customize the data that needs to be monitored
For intranet applications, hertzbeat can be deployed on the external network without access to intranet applications

Task List

[ ] The monitoring interface is exposed, the Http method is tentatively determined, and other SDK methods may be supported in the future (user push mode to push data)
[ ] collector collects push data (pull mode pull)

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

not support，It's still implementing

I can do this.
Please assign it to me.

collector模块定时拉取push模块的数据，是不是无法解决hertzbeat部署在外网无法访问内网的应用。
应该要有个client，比如redis_client，定期采集redis的监控数据，往hertzbeat推送
@Ceilzcx @tomsun28

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

The collector module regularly pulls the data of the push module. Is it impossible to solve the application that hertzbeat is deployed on the external network and cannot access the internal network.
There should be a client, such as redis_client, which regularly collects redis monitoring data and pushes it to hertzbeat
@Ceilzcx @tomsun28

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

of

Yes, the client written by the user should be required to push data to the push module.
If the push module is deployed on the external network, the gateway only needs to release the egress traffic. If the push module is on the intranet, an ingress traffic needs to be released so that the collect module can access the push module

collector模块定时拉取push模块的数据，是不是无法解决hertzbeat部署在外网无法访问内网的应用。 应该要有个client，比如redis_client，定期采集redis的监控数据，往hertzbeat推送 @Ceilzcx @tomsun28

是的，应该需要用户编写的client向push module推送数据。
push module如果部署在外网的话，网关就只需要放行出口流量。如果push module在内网则需要放行一个入口流量，以便于collect module访问push module
@gcdd1993

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

The collector module regularly pulls the data of the push module. Is it impossible to solve the application that hertzbeat is deployed on the external network and cannot access the internal network. There should be a client, such as redis_client, which regularly collects redis monitoring data and pushes it to hertzbeat @Ceilzcx @tomsun28

Yes, the client written by the user should be required to push data to the push module.
If the push module is deployed on the external network, the gateway only needs to release the egress traffic. If the push module is on the intranet, an ingress traffic needs to be released so that the collect module can access the push module
@gcdd1993

collector模块定时拉取push模块的数据，是不是无法解决hertzbeat部署在外网无法访问内网的应用。 应该要有个client，比如redis_client，定期采集redis的监控数据，往hertzbeat推送 @Ceilzcx @tomsun28

是的，应该需要用户编写的client向push module推送数据。 push module如果部署在外网的话，网关就只需要放行出口流量。如果push module在内网则需要放行一个入口流量，以便于collect module访问push module @gcdd1993

随之而来的一个问题就是如何复用现有的collect模块，以及如何管理client的配置（下发配置至client），众多的采集协议或许可以用pf4j重构一波

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

The collector module regularly pulls the data of the push module. Is it impossible to solve the application that hertzbeat is deployed on the external network and cannot access the internal network. There should be a client, such as redis_client, which regularly collects redis monitoring data and pushes it to hertzbeat @Ceilzcx @tomsun28

Yes, the client written by the user should be required to push data to the push module. If the push module is deployed on the external network, the gateway only needs to release the egress traffic. If the push module is on the internal network, an ingress traffic needs to be released so that the collect module can access the push module @gcdd1993

A subsequent problem is how to reuse the existing collect module, and how to manage the configuration of the client (deliver the configuration to the client). Many collection protocols may be available with [pf4j](https://github.com/ pf4j/pf4j) reconstruction wave

collector模块定时拉取push模块的数据，是不是无法解决hertzbeat部署在外网无法访问内网的应用。 应该要有个client，比如redis_client，定期采集redis的监控数据，往hertzbeat推送 @Ceilzcx @tomsun28

是的，应该需要用户编写的client向push module推送数据。 push module如果部署在外网的话，网关就只需要放行出口流量。如果push module在内网则需要放行一个入口流量，以便于collect module访问push module @gcdd1993

随之而来的一个问题就是如何复用现有的collect模块，以及如何管理client的配置（下发配置至client），众多的采集协议或许可以用pf4j重构一波

复用应该是可以做到的，步骤一生成monitorId就是为了复用的，collector模块原本的数据就是放在一个Map里面的，key就是monitorId，当时我和tom讨论了一下，应该如何有效的让用户获取这个monitorId，同时将他作为push的类似key的效果，查询monitorId是否有效（在数据库中），确保用户上传的数据正确

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

The collector module regularly pulls the data of the push module. Is it impossible to solve the application that hertzbeat is deployed on the external network and cannot access the internal network. There should be a client, such as redis_client, which regularly collects redis monitoring data and pushes it to hertzbeat @Ceilzcx @tomsun28

Yes, the client written by the user should be required to push data to the push module. If the push module is deployed on the external network, the gateway only needs to release the egress traffic. If the push module is on the internal network, an ingress traffic needs to be released so that the collect module can access the push module @gcdd1993

A subsequent problem is how to reuse the existing collect module, and how to manage the configuration of the client (deliver the configuration to the client). Many collection protocols may be available with [pf4j](https://github.com /pf4j/pf4j) Refactor a wave

Reuse should be possible. The monitorId generated in step 1 is for reuse. The original data of the collector module is placed in a Map, and the key is monitorId. At that time, I discussed with Tom how to effectively let users Obtain this monitorId, and at the same time use it as a key-like effect of push, query whether the monitorId is valid (in the database), and ensure that the data uploaded by the user is correct

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

@tomsun28 One question still needs to be considered, whether push needs to be deployed separately, because prometheus is through an exporter, and it is normal for pushgateway to be a single module, and our push function can actually be merged. Consider the benefits of separate deployment, after all, separate deployment and configuration increase user operations

@tomsun28 有一个问题还是要考虑一下的，push需不需要单独部署，因为prometheus是通过一个exporter的，pushgateway单独一个模块很正常，我们的push功能其实合并也是可以的。要考虑一下单独部署有什么好处，毕竟单独部署和配置增加了用户的操作

我们第一版本可以把push做成一个模块，暂时不单独部署，和其它模块一样all in one的形式。单独部署我觉得可以在以后的集群环境下，多采集器和PUSH接收器再支持多进程单独部署。感觉目前没有这个必要，单独部署增加了用户配置操作成本。

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

@tomsun28 One question still needs to be considered, whether push needs to be deployed separately, because prometheus is through an exporter, it is normal for pushgateway to be a single module, and our push function can actually be merged. Consider the benefits of separate deployment, after all, separate deployment and configuration increase user operations

In our first version, push can be made into a module, and it will not be deployed separately for the time being. It is in the form of all in one like other modules. Separate deployment I think that in the future cluster environment, multi-collectors and PUSH receivers can support multi-process separate deployment. I feel that this is not necessary at present, and separate deployment increases the cost of user configuration operations.

Hi all，想问下这个模块的开发工作还有在进行吗😂

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Hi all, I would like to ask if the development of this module is still in progress 😂

吗😂

您好，我在负责这个模块的开发，怎么了？

Hi all，想问下这个模块的开发工作还有在进行吗😂

您好，我在负责这个模块的开发，怎么了？

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

Hi all, I would like to ask if the development of this module is still in progress 😂

Hello, I am in charge of the development of this module, what's wrong?

没有，我只是想问我可以帮到什么么，23333

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

No, I just want to ask if I can help, 23333

没有，我只是想问我可以帮到什么么，23333

啊，还好，我这应该没什么需要帮忙的。（

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

No, I just want to ask if I can help, 23333

Ah, it's okay, I shouldn't need any help here. (

没有，我只是想问我可以帮到什么么，23333

啊，还好，我这应该没什么需要帮忙的，可以去看看有无其他issue？

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

No, I just want to ask if I can help, 23333

Ah, it’s okay, I don’t think there’s anything I need help with, can I go and see if there are any other issues?

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑🤝🧑👫🧑🏿🤝🧑🏻👩🏾🤝👨🏿👬🏿

@l646505418 This requirement is carried out as a task of the summer of open source, and no one else can commit. If you are interested, you can take a look at other ones, similar to [#1003](https://github.com/dromara/hertzbeat /issues/1003) is also a similar design class, you can take a look if you are interested

没有，我只是想问我可以帮到什么么，23333

啊，还好，我这应该没什么需要帮忙的，可以去看看有无其他issue？

你好，请问这个模块已经上线了吗？
我现在更新到了版本v1.5.0，从上一个版本开始，我看文档上面就已经有了关于“推送方式监控”的介绍，在docker镜像的配置文件目录中给也已经找到了 app-push.yml 的配置，并且添加配置文档也会提示名称重复，但是在操作页面中一直就没能找到配置入口。


About page - Localization

Please localize the following text on the About page:

Details button
If you have any questions, please feel free to contact:

Fixed in version v1.1.


Need a quick page showing “under maintenance"

Will pop up a quick page when the service is under maintenance, popping up a notice instead of services.

Will adopt @Varunram 's work soon. https://github.com/Musicoin/orbiter/pull/40/commits


Underscores in words

As described in Github flavoured markdown, it makes no sense to italicise parts of a word.
Underscores in words usually refer to something like a variable or field name so it is particularly bad when these are parsed out and converted to italics.
Please can showdown be changed to use the github rules for this.
Thanks.

We're running into this too. Is it an easy fix to make to the parser?  There should be a non-character (space, or beginning of the line) before the underscore in order for it to trigger the italicized output.

@zachad thanks for posting the work around!

@pdeschen It's not a bug, it's by design

As described in Github flavoured markdown, it makes no sense to italicise parts of a word.

Showdown is vanilla markdown. As stated in the original specifications:

Emphasis can be used in the middle of a word:
un*frigging*believable

But if you surround an * or _ with spaces, it’ll be treated as a literal asterisk or underscore.

The correct way to prevent the behavior (or to create a literal asterisk or underscore) is to escape it with backslash, as stated in the specs:

To produce a literal asterisk or underscore at a position where it would otherwise be used as an emphasis delimiter, you can backslash escape it:
\*this text is surrounded by literal asterisks\*

Also I could not reproduce the issue @shhQuiet refers to!
Please provide an example where showdown incorrectly parses links and images with underscore so we can have a better understanding of the issue.

Showdown is vanilla markdown

But vanilla markdown is unmaintained and broken in this behaviour. Could an option be added to support github flavoured markdown?
Thanks.

@SystemParadox @pdeschen https://github.com/showdownjs/showdown/issues/164

http://www.google.com


Building types and checks for external functions : Future Plans

I saw in the recent release announcement that there was some great work done on reading regular @type/@spec definitions... https://elixirforum.com/t/typecheck-fast-and-flexible-runtime-type-checking-for-your-elixir-projects/32886/95#building-types-and-checks-for-external-functions-3 by @orsinium
I understand that at this point, it is more for manual usage, but that it might be further integrated for simplifying the override of remote types. Is the idea that at some point you might be able to do something like...
defmodule MyMod do
  use TypeCheck, fetch_external_types: true # maybe this is the default

    @spec! user_older_than?(User.t(), integer()) :: boolean()
    def user_older_than?(user, age) do
      user.age >= age
    end
  end

In this example, User.t() may or may not be annotated with a @type! t (with a bang) specification. If there is a TypeCheck specification it will use it. Otherwise, it would try to find an external @type t. Is that how you are envisioning it?

Yep, there is a lot more that can be done. I also thought about having something like what typeguard has for checking all types in specific packages during tests. Something like doctest but for types (typetest?) I'm not sure if I personally will have time to bring any of it, though. I hoped to become an Elixir developer and focus my contributions on Elixir ecosystem, but I haven't found any Elixir job :(

Is that how you are envisioning it?

Yes, exactly!
It is not always what you want (in other words: I do not see manual type overrides to fully go away) because sometimes you do need/want to make the type that is used for runtime type-checking (and/or spectesting) more precise than what is available in the non-TypeCheck @type/@spec. But in many cases it will be a very nice and simple way to integrate TypeCheck with existing code.
Some other ideas are to for instance use it to add checks to implementations of a behaviour (c.f.  #106 ), where we could support both TypeCheck and non-TypeCheck behaviours.
And I'm pretty sure that there are other things that could be done as well :smile:.

Closing this discussion issue for now :blush: .


Road segmetns need to be regenerated on changing scene

Right now if you switch between scenes, the geometry isn't generated/is lost.

On initial examination, it appears that redraw is not the issue. That is merely a symptom of some other problem.
Here is the message displayed in the console when switching scenes:

modules/gdscript/gdscript_functions.cpp:775 - Instance was not valid on sid: 44017-44018
core/variant_call.cpp:1222 - Attempted method call on a deleted object.
res://addons/road-generator/road_network.gd:130 - Attempt to call function 'check_rebuild' in base 'previously freed instance' on a null instance.

So, the trick will probably be to figure out why it's trying to rebuild a deleted object.

Hey @TheDuckCow,
Regarding RoadNetworks not redrawing when there are multiple scenes:
When the Scene is changed, the outgoing Scene exits the tree. In the process, the segid_map dictionary becomes stale. It's also worth noting that RoadNetwork._exit_tree() includes some code to cleanup the segid_map. But, it's commented out because, apparently, it caused other issues.
When an inactive Scene is brought back into focus, it tries to rebuild the Segments using stale data and fails. Thus, manifesting as an inability to redraw.
The simplest solution is to add "segid_map.clear()" to the _exit_tree routine. This works great. When the scene becomes active, it rebuilds the segid_map with current data and all is well. Did you comment out the segid_map code in _exit_tree? What are your thoughts on the proposed solution?

@bdog2112 Thanks for investigating and sharing these details, definitely helps to more clearly understand what is happening. If you want to make a PR for what you see as the fix, I'll do some though testing of it in both some test projects as well as a live runtime game. It's possible the error I was seeing was during a game run and not in the editor, but I don't specifically remember.

This has been solved since before the alpha release 0.3.1.


down: add option to delete namespaces

@landism: I don’t see any docs for this feature. Can you please point me to it or add this to the docs so that users know that this feature exists.

@victorwuky don't worry, the documentation in the command also got auto-generated to https://github.com/windmilleng/tilt.build/pull/512

Thanks for pointing me to that @nicks . I'd like to link to the docs in release notes (and not commit), since the release notes should help a user learn about the feature and use it. See https://github.com/windmilleng/tilt/releases/tag/v0.13.1.
I'll be sure to update the release notes once the docs update is merged.
cc @maiamcc .

@victorwuky release notes updated, good catch!


Issue: not able to work with codecov-action

Looking mostly for guidance. We are trying to use wretry.action with codecov-action and things don't work as expected. Am I missing something or this is not supported at all? Here is the PR action logs and the PR. Any insight would be appreciated.

Hello, @williamfgc
I have a lot of work and cannot fix it quickly. Sorry for that. I will fix it later.

Hello, @williamfgc
I found a few hours to work with the issue.
I wrote the test with similar inputs and action found the files of the actions.
https://github.com/Wandalen/wretry.action/pull/76/files
Later, I will write another experimental workflow to check the action.
Please, give me an answer.
Could the previous steps in your workflow delete files of the action with the paths in https://github.com/QMCPACK/qmcpack/actions/runs/3586046379/jobs/6036204370#step:9:679?


How to use this FloatingSearchView with ArrayList ?

I tried using the sample app of this FloatingSearchView and it was working fine. What I want is to use this FloatingSearchView with ArrayList    How can I do it ? Is it possible or not ? Please provide a sample code if possible. Thank You.

GitHub bug tracker are for issues, not for seeking help about Android. Please ask on StackOverflow.


@CsatariGergely The Development Section is admittedly a bit light.  The Maven documentation  provides information on how to compile and where the results are located.  Do you have any specific suggestions beyond referencing the Maven documentation?

Closing the issue as there are not enough specifics to action


In some browsers copy + paste don't work as expected

Work around: Using the menubar (as in Edit > Paste) works in this case.
 Front conversations

related to #704


Support SVG Voids

Description
This one is actually supported in the first minor release of markdown and liquid as thus this issue is merely here for prosperity reasons. Containing SVG tags accept both void and block type token expressions. This means that the following are valid:
<svg>
  <path d="xxxx"></path>
</svg>

<!-- OR -->

<svg>
  <path d="xxxx" />
</svg>

<!-- OR -->

<svg>
  <path d="xxxx">
</svg>

Handling of these structures would fail in the beta versions but support is now available and also an accompanying rule. The new rule and handling will gracefully work these structures and give you the option to convert or omit enders.
New Rule selfClosingSVG
The new markup will determine the existing structures and can remove enders, for example when the rule is enabled which it is by default this:
<svg>
  <path d="xxxx"></path>
</svg>

Will be converted to:
<svg>
  <path d="xxxx" />
</svg>

Partially complete here, though I suspect some minor defects occurring with complex use cases.
Tests
https://github.com/panoply/esthetic/blob/next/tests/cases/html/svg-structures.test.mjs

Supported in v0.6.1-beta.1
Will likely expand upon this in later releases pending stress tests.


[Question] Is any way make navigation from view model?

I think that navigation from VM is more clean way, because in my mind composable more about UI, not logic. But navigation is related to logic in various cases.

Same as with official compose navigation, you cannot pass a NavController (in our case you can't pass DestinationsNavigator either) into ViewModels.
You can workaround this if you really want to, something like this where you depend on a component that just emits some event and then in the UI you can listen to it and do the actual navigation:
https://proandroiddev.com/jetpack-compose-navigation-architecture-with-viewmodels-1de467f19e1c


500 on empty file uid|label|reaper upload

Fixes #993
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Codecov Report

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

@@            Coverage Diff            @@
##             master     #996   +/-   ##
=========================================
  Coverage          ?   90.23%
=========================================
  Files             ?       49
  Lines             ?     6690
  Branches          ?        0
=========================================
  Hits              ?     6037
  Misses            ?      653
  Partials          ?        0

I think we would rather 400 (client messed up) than 500 (we messed up).

We'd want to change the existing 500s as well. Inside placer.py there is a check state, and we have requireTarget and requireMetadata, would we also want to write one for requireFile? Not 100% sure that's the best way forward but would probably be cleaner.

Our of band, Harsha: we can't access file_fields that way because it breaks the ability of the Placer interface to incrementally process a metadata stream. Megan and I worked on some alternatives, check with us.

@hkethi002 and I went through moving the check back into the process_upload path.


Docker Engine section in Add Host for Packet is on same line as Add Label

Version 0.50.1
Steps;

Go to Host Page
Click on Add Host
Select Packet

Results:  The Docker Engine section is messed up and showing up on same line as add label

Expected:
Should show up on next line like all other add pages.

These are under an advanced options toggle now.

Version - master 3.20
Verified fixed


error related to recent wine-update (wine 5.9-1)

Hi,
on my Arch Linux pc there’s a problem with airwave since the recent update of wine to version 5.9-1. when I try to open a project in Reaper DAW with bridged vsts the program is stuck loading it; the terminal output goes:
[udaemon@x270 ~]$ reaper 002c:fixme:ver:GetCurrentPackageId<PHONE_NUMBER>21FD10<PHONE_NUMBER>000000): stub Airwave host endpoint, version 1.3.3error: wrong number of arguments: 6usage: /usr/bin/airwave-host-32.exe.so <vst path> <port id> <log level> <log socket path>003c:fixme:ver:GetCurrentPackageId<PHONE_NUMBER>21FD10<PHONE_NUMBER>000000): stub
can someone else confirm this issue on their setup? are there any ideas to solve it?
thank you!
take care

Just chiming in to say that this is an issue with winegcc's implementation in Wine 5.7+. I ran into this as well while I was working on my own Wine VST bridge. The workaround is to explicitly specify the correct calling convention. I created a bug report here:
https://bugs.winehq.org/show_bug.cgi?id=49138

Weird behavior, two machines  (ubuntu 18.04) both with wine 5.9 one with the airwave deb file installed 8 months ago and the other building airwave with cmake a couple of days ago. In the first maschine (deb) the plugins never open similar to what udaemon reports. On the second maschine (cmake build) only the 32-bit vsts work

It seems a little weird to me that Wine made this change, but going on the comment made by Zebediah Figura in the link posted by @robbert-vdh, it seems like a lot of code in the Wine tree was updated to explicitly use this calling convention.
See here: https://source.winehq.org/git/wine.git/commit/ed6a5e97e0

That PR solves the new crash, but 64-bit plugins that used to work are no longer working.

@xdrie Could you be more specific?

Which plugins aren't working?
What error messages if any do you see (and where do you see them)?

@xdrie Could you be more specific?

Which plugins aren't working?
What error messages if any do you see (and where do you see them)?

all 64-bit plugins (in this backtrace IK multimedia tape mashine 80) produce a wine error. on the other hand 32 bits (addictive drums work). This issue existed before your commint ( I dont now if its being accepted)
backtrace.txt

I am currently having both troubles with 64-bit plugins and with Reaper stucking upon plugin load. I have current master build. Since PR #112 is not merged yet i am assuming that problem with 64-bit plugins is not caused by #112

Yes, I tried building the package before your patch, and that issue (64-bit plugins not working) still exists.
However, your patch does appear to solve the issue with the 32 bit plugins on my system.
The problem with 64-bit plugins is probably separate.

I'm unable to reproduce that error with #112 . Addictive Drums 2 (64-bit) takes about 30 seconds to load, but it does load, and it does work.
For reference, I'm running on Arch Linux (kernel 5.6.14-arch1-1), wine version 5.9-1, DAW used for testing is Bitwig 3.1.3.
I also had zero issues with 64-bit plugins before this issue was opened. Seems like #109 is more relevant.
@durutti and @DiligentKeyPresser : Only hint I can offer you is that I used a different version of the VST SDK than the one recommended by @durutti. I found it in the issue tracker for this project: https://github.com/psycha0s/airwave/issues/92#issuecomment-442582285

#112 also fixed the issue for me.
I did a fork that added a "testing" component that shows Wine logs that helps pinpoint the issue: https://github.com/asb2m10/airwave
Don't forget to build with the debug flag otherwise the Wine stacktrace is useless; eg:
cmake -DCMAKE_BUILD_TYPE="Debug"
Then load the plugin from the command line:
$ export WINEDEBUG=fixme-all,warn+all
$ cd $AIRWAVE_HOME/bin
$ ./airwave-tester ~/.vst/$YOUR_PLUGIN

If it loads from here, you can discard wine compability issues... otherwise you can see the Wine logs.

@j-n-f  could you provide the SDK you used because I cannot download that SDK from the link provided in the specific comment
@asb2m10 I rebuilded airwave (from rdlie git) following your recommendation but I dont see airwave-tester in the bin directory. I still have an error message while scanning for a 64-bit plugin on mixbus


updating catch-targets-biomass

Shouldn't this file be updated? It appears that historical columns are out of date.

If you have correct info that is great, it should be fixed. I think I just copied the values from the 2015 assessment when I built this in 2016.

If you have correct info that is great, it should be fixed. I think I just copied the values from the 2015 assessment when I built this in 2016.

kk, i can update things, just wanted to double check.

kk, i can update things, just wanted to double check.

updated realized catch column and added 2020 with 949b471

updated realized catch column and added 2020 with 949b471


ci: support version strategy for partial Ivy compilation

Angular CLI >=13.0 only allows to partially Ivy compile Angular library bundles, that is these Angular CLI versions cannot build View Engine-compatible library bundles.
A few major Angular CLI versions from now (currently at Angular 13 at the time of writing), the Angular Compatibility Compiler will be removed and only the Angular Linker will remain, removing any support for View Engine-compatible Angular libraries.
We need something like the following:

Version 2.x with support for Angular versions 9 through <last major version with NGCC>
Version 3.x with support for Angular versions 12.2 through <last major versions without breaking changes in the partially compiled Angular Package Format (APM)>

Option A

Features and patches are released across both versions 2.x and 3.x.

Option B

Version 2.x is left as-is in a branch with the possibility of hotfix patches.
Features are released to version 3.x only.

For now, we will only actively maintain one major version of Lumberjack, synced with the latest version of Angular.


Callbacks

It would be useful to be able to call Julia functions from within R, similar to PyCall's facility, e.g.
rcall(:plot, x -> x*x)

However there doesn't appear to be an immediately obvious way to call C function pointers from R, at least as far as I could tell. Any ideas?

AFAIK the only way of calling C functions from R is to write and compile wrappers.  If all your callbacks had a fixed signature you could create an R function that called C/C++ code that takes an external pointer and sets up the call.  Rather messy but I think that is about the best that can be done.  The Rcpp package and the inline package for R have ways of doing the compilation on the fly.

Having dug around some of the undocumented headers, I think I may have found a way to do this.

@dmbates I'm trying to understand more about the internals: why do you manually reassign vv.p and vv.pv here? Doesn't this break the doubly-linked list used for R's memory management?

The unsafe_load call allocates a Julia vector and copies the values pointed to by p into the Julia memory space.  It is the elements of that vector that are being overwritten, not the original memory locations allocated by R.

Ah, I has just reached that conclusion. Thanks!

My comment was poorly phrased.  There are two calls to unsafe_load and I am referring to the second one.
vv = unsafe_load(convert(Ptr{jtyp},p))

Also, the values are in a Julia type, not a vector.

I have implemented it in RCalling.jl, which is obviously inspired by rpy2.
Basically, we want to convect a julia function to a R function. It could be done in the following steps.

register an external method via R api (do_julia in my code) which takes julia objects (the function to be called and the arguments) and eval the call. This will allow us to call .External(".RCall") in R.
Check https://github.com/randy3k/RCalling.jl/blob/master/deps/external.c

covert the input julia function call to the R function-call object (via rlang): function(...) {.External(".RCall", POINTER, ...)}
where POINTER is the pointer to the array of julia objects (the julia function to be called and the arguments).

I may have some time tomorrow to work on this.

I actually have some code as part of my other PR I'm going to put up soon, after a bit more testing.

@simonbyrne cool :100:

Okay, I've got it working fairly reliably now. See ce059fe0e30b3ee510f6981ca7d60a953f50291d, which is part of #59.

Fixed in #59.


Improve Institution user creation API

This improvement was done along with the fix for #227 to add multiple users.


Add support for django-hijack

Closes #9

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

Comparison is base (a3aeb2c) 100.00% compared to head (03d7a45) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          195       195
  Branches        22        22
=========================================
  Hits           195       195

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

Is this something that also should be taken into account in the upstream django two factor auth library? Or is it really specific to MM?

Really specific to our particular combination of django-2fa and django-hijack.
Of course, users of django-two-factor-auth can always use this library as a source of inspiration :D


Add fast ExtractToDirectoryAsync extension method on IArchive

As mentioned in #728 (and I believe some other issues), the IArchive.WriteToDirectory(...) extension method can be very slow for .7z files. This extension utilized IArchive.ExtractAllEntries() and IReader so is fast for .7z files.
Being async has nothing to do with the performance, rather, it's because I use this extension method from an async method and need it to perform the long-running work asynchronously. Ideally IArchive.ExtractAllEntries() and IReader.WriteEntryTo(Stream) would be async as well and then the whole Task.Run wouldn't be needed and then it would make more sense to be async.
It currently doesn't support ExtractionOptions or reporting progress through IArchive's events (although it does report progress through a callback). But, it works great as-is for my use case, which I suspect is pretty common, and I wanted to share.

I've removed the Task aspect in the implementation.
It still feels a bit odd to me since it has its own progress reporting and doesn't fire the progress events on IArchive.


use blocks_without_votes and solana-common source to speed up

Use solana foundational module: https://substreams.dev/streamingfast/solana-common/v0.2.0

change input to 'blocks_without_votes'
add blockFilter to only match blocks with program ID: whirLbMiicVdio4qvUfM5KAg6Ct8VwpYzGff3uctyCc

Since the solana foundational module (maps and indexed) are already cached on StreamingFast's substreams servers, this gives a good performance improvement (about 4x)
Data output is identical

Hi @sduchesneau,
Thank you for the PR, it's much appreciated! The changes look good to me. Is this ready to be merged, or do you have any additional updates?


Fixed issue with dock panel/preview on secondary monitor

Fixed issue where dock panel and preview would not display on secondary monitor

Thanks for correcting！


Error page for variants not found in ExAC

When searching for specific variants, if the variant is not present in ExAC the use would get a page that stated it was not present and also provided the coverage for that region.  This was helpful because you could see if it was a poorly covered region or whether coverage was good and the variant itself has never been seen before. Now there is only an error page when a specific variant is not present in the database.

This is definitely an issue. I am working on analysis of several thousand rare variants that fall into this category.  GnomAD also had this issue as well for a while, but it was fixed on my request.  The same fix should be applied to ExAC.

Thanks for the follow up and all your efforts in maintaining this great resource!
Sent from my iPhone
On Jun 5, 2017, at 8:16 PM, Douglas Smith <notifications<EMAIL_ADDRESS>wrote:
This is definitely an issue. I am working on analysis of several thousand rare variants that fall into this category. GnomAD also had this issue as well for a while, but it was fixed on my request. The same fix should be applied to ExAC.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/konradjk/exac_browser/issues/316#issuecomment-306345905, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbZ0v3_ubxUPZmZEdfqiriql-BFw2Acyks5sBJpWgaJpZM4NcqIi.


Changes to the B team code at BRB

Changes

Adds epic 20 point penalty auto

This is overcome by events


Jikun/start or deploy from outside

Supported "swa start app1" or "swa deploy app1" outside the app1 folder.
For example, under the current working dir containing two projects "app1" and "app2", the above commands will work if there is a "swa-cli.config.json" in "app1" folder, while in the past users must run "swa start" under the app folder.

Just checked the code in details. It seems we got a config path option in SWA CLI's global configs. If Cx runs SWA outside of a project folder, we expect Cx to specify the config path as an extra option. This may be the appropriate way to handle this from a PM's perspective.


Re-add ability to view block and template source code

i would like to see this https://youtrack.jetbrains.com/issue/IDEA-65879


enum class error. :(

Hi.
I am solving under message.
I using C++17 in ubuntu 22.04.
include/CppLinuxSerial/SerialPort.hpp:89:13: error: expected identifier before numeric constant
89 |             OFF,
What can I do for?
Thank you.

Hmmm, I don't think there is a syntax error in the header file. Do you potentially have a preprocessor macro assigned to "OFF" that replaces it with a number (probably 0) passed in at compile time?


environment requirement

Glad to see this project!Can cuda=11.4 environment deploy this project?

Thank you for your interest in our project! We haven't tried that and there might be compatibility issues. We recommend using CUDA 11.6 or 11.7 for the best compatibility.


updated readme with --kubeconfig option

This PR updates the readme to address using the kubeconfig option as an alternate to the defaults.

thanks @rjsadow !


Change Name of Generic Document

Change the name of a generic document in the UI to call it simply "Document." This will change in a subsequent release to a number of new document types such as Living Will, Power of Attorney, DNR, Health Plan, etc.. Refer to related issue #94.

Also merged in #104.