Import throw error "instance() takes at least 1 argument (0 given)"

Hi,
when I import bingads it throws this error: "instance() takes at least 1 argument (0 given)"
>>> from bingads import *
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "[MYPATH]\.venv\lib\site-packages\bingads\__init__.py", line 6, in <module>
    import bingads.bulk  # noqa
  File "[MYPATH]\.venv\lib\site-packages\bingads\bulk\__init__.py", line 8, in <module>
    from .file_reader import *
  File "[MYPATH]\.venv\lib\site-packages\bingads\bulk\file_reader.py", line 2, in <module>
    from .entities.bulk_entity import BulkEntity
  File "[MYPATH]\.venv\lib\site-packages\bingads\bulk\entities\__init__.py", line 13, in <module>
    from .ad_extensions import *
  File "[MYPATH]\.venv\lib\site-packages\bingads\bulk\entities\ad_extensions\__init__.py", line 3, in <module>
    from .bulk_call_ad_extensions import *
  File "[MYPATH]\.venv\lib\site-packages\bingads\bulk\entities\ad_extensions\bulk_call_ad_extensions.py", line 9, in <module>
    _CallAdExtension = type(_CAMPAIGN_OBJECT_FACTORY.create('CallAdExtension'))
  File "[MYPATH]\.venv\lib\site-packages\bingads\service_client.py", line 286, in _create_with_cache
    copied_obj = _suds_objects_deepcopy(obj)
  File "[MYPATH]\.venv\lib\site-packages\bingads\service_client.py", line 275, in _suds_objects_deepcopy
    new = origin_type()
TypeError: instance() takes at least 1 argument (0 given)

I'm running on Python 2.7, with bingads 9.3.2

Hi, Alicor
Would you check if you also have installed the other forked suds library?
you can check by running
pip list | grep suds

The requirement is there is one row like
suds-jurko (0.6)

if there are more rows, please uninstall other suds variants, because they share the same namepsace 'suds'.

Ok, I had also suds (0.4) installed.
I removed both suds and suds-jurko and re-installed suds-jurko to make it works
Thank you so much for your quick reply!


How does react-native-dotenv decide

Hi,
I wonder how does react-native-dotenv deciede which .env file to fetch data (.env.product or .env.dev), can someone explain me please ?

@OguzhanKazak I'm using Expo and it's supposed to do it via the production mode switch.  But, it's always in production mode for me and I can't seem to get it to run in development mode no matter what.

Same Issue here.

You have to reset your cache after making any changes.
Stop the bundler and run:
react-native start --reset-cache
Then react-native run-ios

I am having the same issue, then it's pretty much useless with expo?


Broadcast

Issue that this pull request solves
Closes: #360
Proposed changes

[x] Currently few of the styles are present in the js file. Keep all styles in CSS file
[x] Change the heading from Broadcasts to All Broadcasts
[x] Increase the width of the search-box and aligned with the cards
[x] Enhancement in filter buttons

Brief description of what is fixed or changed
Types of changes

JS style converted into CSS style
Header changed from broadcast to all broadcast
Changed design of filter buttons that has a gradient, animation, and scales 1.1 larger when hovered
Search box length same with card length and left-aligned.

Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes do not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of a change in the user interface

The video that shows the animation and effect of the button when hovered:
https://user-images.githubusercontent.com/67599130/107194196-b2733f00-6a2a-11eb-89d5-51088b75ea33.mp4
Other information
Any other information that is important to this pull request

I am under DWOC Thanks!


Task IDs parity between paper and code

Currently, there is a mismatch between task IDs mentioned in the paper and the ones used in the code.
For instance, in the code "Measurement" task has ID 10-*,

> ScienceWorldEnv("").getTaskNames()
['task-1-boil',
 'task-1-change-the-state-of-matter-of',
 'task-1-freeze',
 'task-1-melt',
 'task-10-measure-melting-point-(known-substance)',      <---Mismatch
 'task-10-measure-melting-point-(unknown-substance)',    <---Mismatch
 'task-10-use-thermometer',                              <---Mismatch
 'task-2-power-component',
 'task-2-power-component-(renewable-vs-nonrenewable-energy)',
 'task-2a-test-conductivity',
 'task-2a-test-conductivity-of-unknown-substances',
 'task-3-find-animal',
 'task-3-find-living-thing',
...

and in the paper it is ID 2

This PR addresses this issue by removing the prefix "task-ID-" from the task names. Also, this PR changes the env.load(...) function to accept a proper task ID to load a task, e.g. env.load("2-3").
Last, a Python script is available to list all task names, their IDs, and number of variations.
python scripts/list_scienceworld_task.py
TASK LIST:
   1-1                                                boil      (  30 variations)
   1-2                                                melt      (  30 variations)
   1-3                                              freeze      (  30 variations)
   1-4                       change-the-state-of-matter-of      (  30 variations)
   2-1                                     use-thermometer      ( 540 variations)
   2-2             measure-melting-point-(known-substance)      ( 436 variations)
   2-3           measure-melting-point-(unknown-substance)      ( 300 variations)
   3-1                                     power-component      (  20 variations)
   3-2  power-component-(renewable-vs-nonrenewable-energy)      (  20 variations)
   3-3                                   test-conductivity      ( 900 variations)
   3-4             test-conductivity-of-unknown-substances      ( 600 variations)
   4-1                                   find-living-thing      ( 300 variations)
   4-2                               find-non-living-thing      ( 300 variations)
   4-3                                          find-plant      ( 300 variations)
   4-4                                         find-animal      ( 300 variations)
   5-1                                          grow-plant      ( 126 variations)
   5-2                                          grow-fruit      ( 126 variations)
   6-1                                       chemistry-mix      (  32 variations)
   6-2               chemistry-mix-paint-(secondary-color)      (  36 variations)
   6-3                chemistry-mix-paint-(tertiary-color)      (  36 variations)
   7-1                            lifespan-(longest-lived)      ( 125 variations)
   7-2                           lifespan-(shortest-lived)      ( 125 variations)
   7-3        lifespan-(longest-lived-then-shortest-lived)      ( 125 variations)
   8-1                              identify-life-stages-1      (  14 variations)
   8-2                              identify-life-stages-2      (  10 variations)
   9-1                      inclined-plane-determine-angle      ( 168 variations)
   9-2            inclined-plane-friction-(named-surfaces)      (1386 variations)
   9-3          inclined-plane-friction-(unnamed-surfaces)      ( 162 variations)
  10-1                    mendelian-genetics-(known-plant)      ( 120 variations)
  10-2                  mendelian-genetics-(unknown-plant)      ( 480 variations)

@aphedges hopefully this PR is not breaking anything on your side. You can still use the former task name convention. Let me know if that's not the case.

@aphedges hopefully this PR is not breaking anything on your side. You can still use the former task name convention. Let me know if that's not the case.

It might break some stuff, but we have enough internal breakage that we already have scripts to do task name translation.
ISI is currently on break, so it won't be something we will be discussing until January.

Once I'm done with https://github.com/allenai/ScienceWorld/pull/37, there will be some breaking changes too. I plan to make ScienceWorld compatible with Gymnasium.


Missing function Element.closest(selector)

Every web browser I've tested other than MS Edge supports JavaScript statements of the form:
variable = Element.closest(selector)
Example: :    var targetElem = this.nodeField.closest('div');   // Get field's closest containing div
The list of supporting browsers includes Mozilla, Chrome, and Safari on desktop, iOS, and Android, and IE 11 on Windows.
However, executing the above JavaScript statement in MS Edge results in the following error:
SCRIPT438: Object doesn't support property or method 'closest'
This forces me to add a conditional function prototype for "closest" to my application so that it will run successfully on MS Edge, which I would much prefer not to do.

Looks like this is an API defined by the DOM:
https://developer.mozilla.org/en-US/docs/Web/API/Element/closest
Please open this issue on Edge as described here.

Not an issue in ChakraCore as indicated by dilijev.

As requested by dilijev, I’ve opened new Issue #10320716  Edge Browser missing function Element.closest(selector) at https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/10320716/
I appreciate your response, and  apologize that it was not obvious to me that this is an Edge platform issue rather than a ChakraCore issue.
From: EdMaurer<EMAIL_ADDRESS>Sent: Tuesday, December 27, 2016 4:44 PM
To: Microsoft/ChakraCore
Cc: ericf01; Author
Subject: Re: [Microsoft/ChakraCore] Missing function Element.closest(selector) (#2286)
Not an issue in ChakraCore as indicated by dilijev.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Microsoft/ChakraCore/issues/2286#issuecomment-269386911 , or mute the thread https://github.com/notifications/unsubscribe-auth/ARD3J9t1KWLxVxXNr2E4DmiXzftxkZDJks5rMYa8gaJpZM4LVc2E .  https://github.com/notifications/beacon/ARD3J4L-1XQttoEnsOiZAaepmRf_BcyAks5rMYa8gaJpZM4LVc2E.gif

Thanks @ericf01- and no reason to apologize, it's not always obvious whether something is a Edge issue or a Chakra issue 😄 Thanks again for reporting the issue to both us and Edge!


activate system service before factory app launch (#1351)

Co-authored-by: mthiru270<EMAIL_ADDRESS>Co-authored-by: Michael Fiess<EMAIL_ADDRESS>

 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.


Setting to pause/resume background music if Playnite is minimized/in-focus

Please add an option under settings to pause the background music when Playnite is minimized and resume when Playnite is back in focus. Ideally the music should just be paused and not be completely stopped, and then unpaused from where it left off again.

I very, very much would want this feature and came here to request it. I love the idea of the background music and it's great in Fullscreen mode but having no way to automatically pause it when I switch to a different window is pretty annoying.

Hey i just released a new version with this option implemented


Add generic typing for endpoints body

Since I started using Notion API and this SDK, I wanted to work around typing.
Because the notion API format is quite heavy, it's easy to do mistakes when filling properties, at page update for example.
I opened a pull request #103 last week on this repository to show an example of what could be done to enable body validation by typescript during development.
This PoC shows how we could use generic typing to validate the page update method inputs based on the page definition.
Would be glad if you can give me some feedbacks !

Having some similar issues too. I'm using this package with strict TypeScript checks.
Some typings on the request APIs should be updated include only part of them, e.g. notion.blocks.children.append should not mark id, created_time, last_edited_time, has_children as required properties on children because those are created at server side.
Some typings do not match the verification logic. For example, when creating page with notion.pages.create, to set the page title, in the properties, title: { type: 'title', text: { ... } } would pass TS check but throws error at runtime validation instructing me to update it to such undocumented structure:
title: {
  type: [
    {
      type: "text",
      text: { content: "Test Title" },
    },
  ],
}

Although it runs without throwing runtime errors, the title is not properly set.

I've not looked into the implementation details of this SDK, but logging the issues here first. Will see what I can do later.

Hello @blackmad @aoberoi
Before I invest some time in doing the same work for every method, I would really like to have your feedback on what I did with this PR.
Don't hesitate to tell me if you want more details around this.

@pllnk is Notion's resident type wizard and will be thinking about the future architecture of the client library, I'll let him chime in here!

As the types are now being auto-generated, I'm going to close this issue. Hopefully the generated types are more complete 🙏


Update Compiler.php

Allow to append custom C classes to extension. Example configuration:
"extra-classes": {
    "iterator": {
        "source": "own/test/iterator.c",
        "header": "own/test/iterator.h",
        "entry": "own_test_iterator_ce",
        "init": "Own_Test_Iterator"
    }
}

Is is really important when zephir functionality is not enough.

Could you please use fix the PSR2 style? https://travis-ci.org/phalcon/zephir/jobs/65245051#L2116

@ovr @andresgutierrez
I will be grateful for review.

@baszczewski
https://travis-ci.org/phalcon/zephir/jobs/65291681#L2132

@ovr ping

I think you should rebase & squash the commits.
Also maybe the commit message should have a short summary in the title
(first 75 characters) of the changes insteadof a generic "Update Compiler.php".

Thanks


chore(deps): add resolutions for minimist

This PR addresses the vulnerability with minimist
Changes

Added resolutions.minimist in package.json
Updated @sapphire/eslint-config, eslint and typedoc to latest
Added rimraf (and @types/rimraf) as dev-deps

TIL security vulnerabilities can be ignored


Reply with stickers

Teached to flame with stickers
When sending a sticker in a private chat to the bot it replies with its sticker file_id

Looks good to me.


Update nouislider dep

@vomchik thanks for this! It came in as part of #52, #58 & #60
This has also been published to npm as 1.0.0-rc.1, please test with ember install ember-cli-nouislider@next and please open any issues that you encounter.


[WIP] Implement basic AWS CloudHSM support for root CA creation + rewrite "FulcioCA" to "PKCS11CA"

This adds basic support for self-provisioning a root CA with AWS CloudHSM through the existing PKCS11 support. Since the "FulcioCA" option now encompasses both BYOD HSMs and AWS CloudHSM, it's been renamed to the more all-encompassing "PKCS11CA".
Signed-off-by: Mark Bestavros<EMAIL_ADDRESS>How it works
AWS CloudHSM provides a PKCS#11 compliant library for interacting with instances of its CloudHSM product. For the most part, this serves as a drop-in replacement for the SoftHSM PKCS#11 library; as long as keys are set up beforehand, just as Fulcio expects to be done when using SoftHSM, it (mostly) works with no code changes.
The exception is that CloudHSM does not support certificate storage on their HSMs:

AWS CloudHSM does not store certificates on the HSM, as certificates are public, non-confidential data.

In its current state, Fulcio does attempt to store the root CA it creates as part of the createca command within the HSM it's using, and this causes an error when running with AWS CloudHSM.
I've explored many possible workarounds for this limitation, and run into just as many dead ends. While those might change in the future, for now, this is the most simple working implementation of something that works with AWS: effectively, during root CA creation when running on AWS, instead of attempting to store the root CA in the HSM or in some other certificate storage service, it's just stored on-disk as a PEM file. Then, later on, when running fulcio serve, another option is exposed to load the root CA from disk instead of the HSM. This works around the lack of certificate support on AWS' HSMs.
Setup
AWS provides a rather comprehensive setup guide for getting a CloudHSM instance deployed and setting up an EC2 instance that can interact with it. For a complete setup, you'll want to make sure the CloudHSM Management Utility, Key Management Utility, and CloudHSM's PKCS#11 library are all installed on your EC2 instance.
First, you'll need to make sure your HSM is set up using the Management Utility using this guide. Next, you should provision keys within the HSM for Fulcio to use. Do this with the Key Management Utility:

Log in: loginHSM -u CU -s <username> -p <password>
Generate a keypair with appropriate label and ID: genECCKeyPair -i 14 -l PKCS11CA -id 1

With these steps done, your HSM is set up!
Next: clone Fulcio with this patch included onto the EC2 instance that has the AWS PKCS11 library installed. You'll also need to adjust config/crypto11.conf as such, in order to make it work with the CloudHSM setup:
{
    "Path" : "/opt/cloudhsm/lib/libcloudhsm_pkcs11.so",
    "TokenLabel": "cavium",
    "Pin" : "<username>:<password>"
}

The Path variable replaces the softhsm PKCS11 library with AWS CloudHSM's. On AWS CloudHSM, the TokenLabel must always be cavium - or things won't work. The Pin takes the format of a crypto user and corresponding password on the HSM, as seen here.
With this done, you're ready to use Fulcio! Here are some example commands:
fulcio createca --org=acme --country=US --locality=SomeTown --province=SomeProvince --postal-code=XXXX --street-address=XXXX --hsm-caroot-id 1 --out myrootCA.pem --hsm=aws`

This command creates a new root CA using the private key stored in AWS CloudHSM and stores it in the myrootCA.pem file locally.
fulcio serve --ca pkcs11ca --hsm-caroot-id 99 --aws_hsm_root_ca_path myrootCA.pem

And this command uses the generated myrootCA.pem file to run the Fulcio server.

wow, now that is how to write up a PR!
This looks good to me Mark, only thing I would add is to make additions to the README (such as using libcloudhsm_pkcs11), I understand this may be pending though as its an WIP PR.
@dlorenc / @bobcallaway on ideas on the root CA being stored locally, I see no is security impact as the priv key is in the HSM. This is not to different to what we have with the GCP public good instance. I guess the main aspect is if someone wants this to be highly available. I am ok with addressing that later when we get user feedback to suggest adoption, we can then look to use an ec2 bucket api.

@lukehinds @bobcallaway I've just pushed a few changes that address most of the minor comments people have made. I've also amended my setup guide and put it in its own .md file for posterity.
On another note, I think that it would be helpful to reorganize/simplify the various pieces of documentation in a follow up PR. As is, they're kind of just sitting around.


Map cb servervar

Completed adding the OrderedMap for the structs along with their basic unit tests

OrderedCallbacks
OrderedEncodings
OrderedExamples
OrderedHeaders
OrderedLinks
OrderedMediaTypes
OrderedResponses
OrderedStrings
OrderedParameters
OrderedRequestBodies
OrderedResponses
OrderedSchemas
OrderedSecuritySchemes

These structs are created with Ordered prefix, and the old ones are also renamed to follow this pattern.
In the upcoming PRs, adding marshal and unmarshal to these new Ordered... structs

@casualjim missed adding you as a reviewer.

excellent work so far!


Add workflow already completed error to client

What changed?
Adding the new error type WorkflowExecutionAlreadyCompletedError to the client repo

Why?

How did you test it?

Potential risks

Pull Request Test Coverage Report for Build 38afaae8-61bf-4dee-bf31-5fa5c924ae60

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 75.669%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/internal_workflow_testsuite.go
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/internal_task_pollers.go
2
75.25%

Totals

Change from base Build 46e1e459-04ae-42d7-8f81-827891cefcd7:
0.0%

Covered Lines:
10148

Relevant Lines:
13411

💛  - Coveralls


Edge case: printing with MultiProgress after all bars are finished

The Issue
When using a MultiProgress, you need to use a bar's println to avoid the terminal getting messy.
However, if all bars finish, you can't println, even though you could if using a solo ProgressBar.
Possible Solutions

Add a println to the MultiProgress itself.
Add some other type of join to MultiProgress, which continues even after all bars are done.
Add another method to control the pump inside MultiProgress::join, a user can query if it's already done, and switch to regular println!. Something like MultiProgress::single_event(), which returns if the last event was final.
Add another method that lets other threads wait for MultiProgress to finish, which could then switch over to regular println!.
Some combination of the above.

Out of all of these, 1 seems the simplest, and I'd be happy to work on a PR if y'all think it's an appropriate solution.
Background
I'm working on two adapters for indicatif:
One to work in an async runtime that dispatches messages to another thread. Without a way to query if MultiProgress is done, println calls could be dropped, or I'd have to wrap all print calls in another mutex over MultiProgress.
That's complicated. And unless I haven't seen the solution, it's impossible to do without race conditions.
The other adapter will let you use indicatif's bars as a Writer for tracing_subscriber. It more or less has the same issue.

Somehow I totally missed #351. I think keeping this open might make sense if there's discussion to be had for the other solutions I mentioned.

I didn't see the removal of join in main! This is all handled, or at least will be once #351 is merged.


Add getCurrentRoute() method, fix group middleware

Q
A

Is bugfix?
✔

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Fixed issues
fixed adding group middleware to a route

I'm working on the router/Group middleware test.
This bugfix is critical. Accept PR as soon as possible.

Thanks!


[Cypress] - Better result reporting

Until now, results are only available when you go to Github Action and check the Cypress run step.
Maybe we can export the result as an HTLM page and make it easier to find/read.

Made progress using this pluging:
npm i mochawesome
It creates an artifact with screenshots and videos when tests fails.
Local results:
https://user-images.githubusercontent.com/37271841/192831213-27cbd68b-34f3-4cd7-aa6f-ea85f195c7cc.mov
However it seems to have an issue in ci finding the right paths:

Will continue working on a solution.

Managed to copy screenshots and videos within mochawesome-report directory and read the artifact from there:
https://github.com/epinio/epinio-end-to-end-tests/actions/runs/3177952134

I will polish this tomorrow

I will finally merge the artifact approach here: https://github.com/epinio/epinio-end-to-end-tests/pull/237
As a summary this this approach, as explained above, creates an artifact with copies of videos and screenshots but also:

It always displays the videos (even if succeeds and
It displays screenshots if fails.
It creates 1 html page per runner.

This is how it looks: mochawesome-report-light.zip

And here the new artifact on CI:

I tried to merge all pages into one using this approach: https://medium.com/swlh/publish-your-cypress-test-report-with-github-actions-47248788713a
This worked well when tests are not parallelized, but since we do it overrides all the time the html generated on each test. We can always revisit this issue in the future
Furthermore, it can also be revisited the attempt to change directly the report on the own gh action page with the use of peaceiris/actions-gh-pages@v3


request: new Kind value for "monitoring"

Hello!
While there is a separation between signals that are important for the measurement ("normal", and "hinted"), configuration signals ("config") and those who should not be read out ("omitted"), it would be great to have another type: "monitor".
Especially for large detectors, it is sometimes required to provide a monitoring data stream, e.g. 2d frames at a reduced frequency. However, these signals should not be considered as relevant data for the file writer (normal / hinted), nor configuration data.
Therefore, I would propose to add a new type to Kind:
class Kind(IFBase):
    """
    This is used in the .kind attribute of all OphydObj (Signals, Devices).

    A Device examines its components' .kind atttribute to decide whether to
    traverse it in read(), read_configuration(), or neither. Additionally, if
    decides whether to include its name in `hints['fields']`.
    """

    omitted = 0b000
    normal = 0b001
    config = 0b010
    hinted = 0b101  # Notice that bool(hinted & normal) is True.
    monitor = 0b110

Would you be open for these changes? If so, I could prepare a PR.

This seems reasonable on first consideration, but we should think about if there is a better name.  monitor collides a bit with the bluesky message Msg('monitor',....) and I worry that there could be assumptions that if you flag a signal as kind monitor it will be automatically monitored (and produce an additional data stream in the run).
This is something that down-stream tools can use to identify things they may want to set up subscriptions too (and possibly reduced rates)?  Would this also be used for something like motor temperature or queue-size on an AD plugin?  I could see wanted to flag those as "things to watch but you really don't want to write this down verbatim in the data stream".
Are there signals that may need to be both normal and monitor?  I could see the case for that for most position-like values ("I want to write down what the temperature is / where the motor is in every event but also show the user a wiggly line of it moving in between measurements").
maybe "live_status" or "preview" might be better names?

hmm, I never thought about using it for temperatures, I only had reduced data rates in mind. But you are right, calling it "live_status" could open it up to more use cases.
Regarding your use case for normal and (let's call it) live_status: I think if you can handle the entire data stream of a signal, you could simply add a subscription to it and plot the readback value, no? On every read event, you would still get a reading from both temperature and motor and in between you would see the live updates. However, I can see that a combination could make it more flexible and allow users to poll themselves instead of receiving data through callbacks.

maybe "live_status" or "preview" might be better names?

I prefer the new kind to be called "preview" - it indicates it is not meant for measuring, only for preview.
The implementation of the signal reading will have to deal with this kind - maybe it comes from EPICS auto-monitor,
or maybe it comes from a zmq stream or whatever... I would not mix 2 different use cases in one signal.
Examples:

for a 2D detector we could have an image signal with kind preview, it would give last frame (best effort) upon reading
for a MCA, there would be a live_spectrum signal with kind preview, to show the accumulation while measuring

it would be different compared to spectrum with kind normal, for reading acquisition result

diagnostic is one potential shed colour.


Handle unexpected JSON schema of Well responses from DoT API

Child of #101
Requires #91
Requires #92
The following fields are problematic in the JSON schema of a well from DoT:
"dTimCreation": "2018-11-29T14:46:03.165Z",
 "dTimLastChange": "2018-11-29T14:46:03.165Z",

These fields should not be at the object root, but should be in another field.
Add a conditioning step to the JSON to XML ValveTranslator that will fix this JSON schema before parsing the JSON as a WitsmlObject.
This step should be in a function conditionWellJSON(rawJsonResponse) that returns the conditionedJsonResponse so that the function can be easily changed and tested in the future.
Please also write tests cases for this.

@cherrera2001 at your leisure, please provide the expected transformation for the above fields.
Thanks!

Ok, latest news is that the DoT API will no longer require this. Marking this done


typings/helpers.d.ts should be in distributed npm package

The built NPM package is missing './typings/helpers, so our build pipelines, validating the typings, fail...

Hey @siebertm, thanks for reporting this one!
We'll prepare & publish a fix soon 🔧

Hello @siebertm!
Your issue should should be resolved as of version 1.2.1 of this package - the helpers.d.ts is now properly included into the npm package..
Feel free to update i18next-phrase-in-context-editor-post-processor in your project, give it a try and come back with a feedback 🚀
Have a nice Monday!

Works like a charm! Thanks Jakub!


Add normal mode for input.

I've seen #1.
But I see value in having ESC exiting insert mode.
Pressing ESC again would then close the prompt.
I guess it wouldn't be too alienating for many users since this is his telescope works as well.
Ideally this would be an option which could be set in get_config for input.
On another note, is there a filetype assigned to the input buffer which can be used to apply custom mappings?
This might not "fix" but help with #2.

I like both of these suggestions! Will add them shortly.

Added an option to disable the ' in insert closes' mapping and set the filetypes of DressingInput and DressingSelect on the built-in modals.

Awesome, thank you 🙏


[DOCS] Contributing E2E Test

Notes for Reviewers
Initial draft for Contributing to E2E Test
This PR fixes #11230
Staging:
https://66bb7080be029a47eabee379--meshery-docs-preview.netlify.app/project/contributing/contributing-ui-tests
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 99d3ac629c56bffe47acf707e71043a3bcde30a9 at: https://66bb7080be029a47eabee379--meshery-docs-preview.netlify.app

Thank you for using the do-not-merge label to help communicate review readiness.

@jerensl let's discuss this PR and get some feedback during the Meshery dev meeting call, if you would. Please add it as an agenda item to the meeting minutes.

🚀 Preview for commit 450ecf38d5c2e8b173f3f8858e07fe267c99571d at: https://66bc91feab6023820c659ab3--meshery-docs-preview.netlify.app

🚀 Preview for commit 23228fefb8f927e5e54228e3792e1eaa063020a2 at: https://66be17ffa001ab4c81801a1c--meshery-docs-preview.netlify.app

🚀 Preview for commit 5ed66f08838d7d8b5688bb6c5a02c090a993a830 at: https://66bf4609ac8fdb1ba5807b1f--meshery-docs-preview.netlify.app


[PluggableDevice] Cross-device copies support

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.04
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): v1.12.1-69032-gfb972503145 2.8.0-dev20211220
Python version: 3.9
Bazel version (if compiling from source): 3.7.2

Describe the current behavior
When using a PluggableDevice plugin with 2 physical graphics cards, the memcpy_dtod function is invoked. This function, however, only has a single SP_Device as a parameter, even when the src and dst memory regions are from different devices.
Describe the expected behavior
The memcpy_dtod function should have 2 SP_Device parameters in case the copy is done cross-adapter to give the plugin a chance to handle cross-device copies.
Contributing

Do you want to contribute a PR? (yes/no): yes
Briefly describe your candidate solution(if contributing): Adding a memcpy_dtod_cross_adapter function for cross-device memory copies, or adding a second SP_Device parameter to memcpy_dtod (although the latter wouldn't be backward compatible).

Standalone code to reproduce the issue
import tensorflow as tf

tf.compat.v1.disable_eager_execution()

a = tf.compat.v1.placeholder(dtype=tf.float32, shape=[3,1])
a_values = [[1],[2],[3]]

b = tf.compat.v1.placeholder(dtype=tf.float32, shape=[3,1])
b_values = [[4],[6],[11]]

c = tf.compat.v1.placeholder(dtype=tf.float32, shape=[1,3])
c_values = [[2,4,5]]

with tf.device("/device:CUSTOM:0"):
    y1 = tf.raw_ops.AddN(inputs=[a,b], name="MyAdd")
with tf.device("/device:CUSTOM:1"):
    y2 = tf.raw_ops.MatMul(a=y1, b=c, name="MyMultiply")

with tf.compat.v1.Session(config=tf.compat.v1.ConfigProto(log_device_placement=True)) as s:
    print(s.run(y2, feed_dict={a:a_values, b:b_values, c:c_values}))

@PatriceVignola
In order to expedite the trouble-shooting process here,Could you please fill the issue template,
Thanks!

@sushreebarsa Thank you, I updated the issue with the template.

@PatriceVignola Thank you for the update!
We see that you are using TF v1.12 which is not actively supported ,could you please upgrade to TF v2.4. or later(FYI Latest stable TF v2.7.0 )?
This tf.compat.v1.Session API was designed for TensorFlow v1. Please check this details on how to migrate from this API to a native TensorFlow v2 equivalent. See the TensorFlow v1 to TensorFlow v2 migration guide for instructions on how to migrate the rest of your code.
As TensorFlow version 1.x is out of support window so for any further queries  please post this issue in TF Forum where there is a larger community to help .Thank you!

I'm not using 1.12, I'm using 2.8.0-dev20211220 (I was also using 2.7.0 and having the same problem). This is a PluggableDevice specific issue, which is a TF 2 feature.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


azure network nsg show query shows "null" when there are multiple CIDR blocks in SrcAddrPrefix

Environment summary
Install Method: Interactive Script
CLI version: 2.0.20
OS version: 14.04.5
Shell Type: Bash on Windows
Running a query such as:
 az network nsg show -g myresourcegroup -n mynsg -nsg --query '[defaultSecurityRules[],securityRules[]][].{Name:name,Desc:description,Access:access,Direction:direction,DestPortRange:destinationPortRange,DestAddrPrefix:destinationAddressPrefix,SrcPortRange:sourcePortRange,SrcAddrPrefix:sourceAddressPrefix}'

Will display "null" as the below when you put multiple CIDR blocks into the "source" in the Azure portal. This is supported and works, but doesn't seem to fare so well in displaying with the 2.0 cli. When displaying this in table format, the output is simply blank.
  {
    "Access": "Allow",
    "Desc": null,
    "DestAddrPrefix": "*",
    "DestPortRange": "*",
    "Direction": "Inbound",
    "Name": "myrule",
    "SrcAddrPrefix": null,
    "SrcPortRange": "*"
  },

If you show the NSG without the query do the address prefixes show?  If so, there is an issue with your query syntax.
If I understanding your scenario correctly, you are using the query in order to generate a table format. Is that right?

You're right, it looks to be an issue with the query. The docs on this page should probably be updated, which prompted this ticket:
https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-manage-nsg-arm-cli
Feel free and close this issue, though I'd as to open an issue with whomever is responsible for the docs on that page.


[HAL][NRF51822] enable I/O mapping and peripheral dynamic assignment for I2C and SPI

to enable any digital I/O of nRF51822 to be used as I2C's SDA/SCL or SPI's MISO/MOSI/SCK.

to avoid conflict between I2C and SPI by dynamically assigning peripheral
The nRF51822's I2C_0 and SPI_0 (I2C_1, SPI_1 and SPIS1) share the same address. They can't be used at the same time. So we use I2C_0 and SPI_1 as default. See nRF51822 address information at nRF51822_PS v2.0.pdf - Table 15 Peripheral instance reference.

See discussion at #573

Could someone pretty, PRETTY, please make these same changes for the NRF51-DK platform? They're different target platforms but are ultimately for the same chip...

@peter100191 Having any problems? nrf51-dk platform uses the same hal sources, therefore should be functional the same as for any other?

I'm confused on the implementation of this fix... By the looks of it the selection of SPI_0 vs SPI_1 or I2C_0 and I2C_1 is done internally, yet the following code is causing my NRF51-DK to crash:

SPI device(p25, p28, p29);
I2C i2c(p30, p7);

It's crashing on the initialization of the second object, regardless of order. If I put this before the main function it won't even execute the first line.

Just to be sure: You are using a recent mbed lib? Not an old one you get often when importing example programs? (If you use the online compiler, right mouse button, update).

Latest mbed version, platform "Nordic nRF51-DK". If I instantiate SPI & I2C within the main() void, if I init SPI first it will crash, and if I init I2C first it will not crash but SPI will not work. I've tried using the default I2C / SPI pins as well as others. Same effect.


Readme with prominent public transit and isochrone links&images

Also mention setup for both in quick start (with sources)

Closing via https://github.com/graphhopper/graphhopper/commit/5d602f1e16b5dfed42fdfca9ea895f5557941334 and https://github.com/graphhopper/graphhopper/commit/5378bae67115bf53e82779084094652fa1497644


Admin - upload con note

As an admin I should be able to upload a copy of the con note when the delivery is complete

As an admin I am able to upload a copy of the con note when the delivery is complete by scanning the packages Unique Identifier when the delivery is complete.

I think this should actually be generated by the server to be honest.
@me4502 thoughts?


Difference between 1.0.2 and 1.1.0

Hi team!
Are there any meaningful differences between these two versions? I am asking because using 1.0.2 would allow us to postpone compileSdk bump to the newest version as it is causing build issues for us.
Thanks!

Cool thanks for such a quick turnaround! Actually managed to get it sorted so now looking forward to those survey component on mobile 🚀


javascript interoperability

How hard would it be to be able to execute JS functions? Let's say for sake of simplicity on node.
I've seen in the source code stuff like (get console 'log),
but wasn't able to achieve much more. It was possible to do (require 'fs) and the whole list of functions of this module pops up, but I wasn't able to do much more

I think you're nearly there! Lumen functions are JavaScript functions, so you can invoke them in the same way. The biggest hurdle is the syntax, because JavaScript makes things like require('fs').mkdir(...) easy to write, but in Lumen it would look pretty awkward: ((get (require 'fs) 'mkdir) ...).
I think the convention in Lumen is to bind function objects to variables before using them. This file in the source code might be the most helpful to consult: https://github.com/sctb/lumen/blob/master/system.l. At the REPL, you could go further with your (require 'fs) by doing something like:
> (define fs (require 'fs))
...
> (define slurp-file (get fs 'readFileSync))
...
> (slurp-file "my-file.txt" 'utf8)

Closing.


Log from BeakerStepLock at WARNING level

Should this be printed to the click logger? Do we still have that?

Should this be printed to the click logger? Do we still have that?

We still have that (it's the cli_logger now), but the downside would be that those messages wouldn't show up when using Tango programmatically unless the our initialize_logging() function was called with enable_cli_logs=True.

WARNING is our cheap way of getting both at the same time? I think it's OK in this case.

WARNING is our cheap way of getting both at the same time? I think it's OK in this case.

Right

This is good, but tests don't pass.

Looks like Beaker is still broken 🙂


Microsoft.ManagedIdentity/userAssignedIdentities/federatedIdentityCredentials

Bicep version
azure-cli   version 2.48.1
bicep code in github workflow, results in the same "bug"
Describe the bug
If i run my code, where i expect the result to be, Federated credential scenario : Kubernetes Accessing Azure Resources , then i get the Other scenario " Configure an identity managed by an external OpenID Connect Provider to access Azure resources as this application"
I happens both in bicep code, see snippet below
resource privateFederation1 'Microsoft.ManagedIdentity/userAssignedIdentities/federatedIdentityCredentials@2023-01-31' = {
name: priv1.name
parent: priv1
properties: {
audiences: [
'api://AzureADTokenExchange'
]
issuer: aksPrivateOicd
subject: 'system:serviceaccount:${namespaceInKubernetes}:${priv1.name}-serviceaccount'
}
AND also when i try using az cli commands. Like this one below
az identity federated-credential create --name myFicName --identity-name myIdentityName --resource-group myResourceGroup --issuer myIssuer --subject mySubject --audiences myAudiences
Additional context
If i re-run existing projects, with our generic code in order to re-federate credentials for existing managed identities, it keeps the right Federated Credential scenario, regarding Kubernetes Accessing Azure Resources, but all new federation get the Other result.
But mores to it, If i do a federation list from the one that is created to be Kubernetes but is results to be Other according to the azure portal.
az identity federated-credential show --name myFicName --identity-name myIdentityName --resource-group myResourceGroup
It will list the exact output from what i got after running the create command
az identity federated-credential create --name myFicName --identity-name myIdentityName --resource-group myResourceGroup --issuer myIssuer --subject mySubject --audiences myAudiences

This appears to be an RP bug, not on the Bicep side. I will bring this to the Managed Identity team's attention, but if you would like to streamline this, please open a support ticket with the Managed Identity team.


v1.0.0 roadmap

I think it's the time for a full upgrade including the core and the features as well.
My plan is to address these issues in the v1.0.0 release (Not necessary in same order, and not necessary in the first release, but can be added in patches):

Full rewrite of the dashboard queries to low down the number of queries and make it faster. #71 (done in efd7461d85718c25beae5cfb68f184bf186e19d1)
Full rewrite of the settings database and Settings management panel
Tickets filters by department, user, agent, category, ..etc
Sub-tickets #16
boilerplate compatibility #26 (I don't know, but i will give it a try)
Ability to Disable Priorities and Categories #27
Admin master template separate of the front master  #28
Add Ids and classes to Ticketit templates divs and elements #29
Relationship restrictions #53
Create tickets through webform #69
Create tickets through email #69
Attachments to tickets #70  (done at #108)
Private comments #94 (done in c2e462f8b4170467a161a8bd1699c9d050e37ee0)
Agent teams/groups (while the auto assign solves a part of it, may be we need to add permission for agent to work on his department issues) #94
Ability for agent to assign owner of ticket for customers that call in by phone #94
Service Level Agreements (SLAs) where specific actions/escalations are taken based on ticket timelines #94
ticket timeline (audit)
more agent restriction permissions (ex. agent can view category tickets but can't edit, can view and edit, can view all categories tickets but can't edit, ..etc)
Make a config file for fallback, so if any thing goes wrong and can't access web interface, user can change configuration in the config/ticketit.php file then issues php artisan db:seed --class="Kordy\Ticketit\Seeds\SettingFallbackSeeder" to reset the ticketit settings in the database #78 #91 (done at e87270c9314da14c4c14317b672a3996e77fa8af )
Handle errors and exceptions and provide errors logging

On number 11 what were your initial thoughts on that? Should it be some sort of controller page that allows services like mailgun to pipe json to it, or were you thinking more along the lines of the application pulling emails from an imap account?

or were you thinking more along the lines of the application pulling emails from an imap account more along the lines of this: https://github.com/barbushin/php-imap

That's it, and I think we can use a package like that one.
To be more clear, we would have a new controller that depends on a third party email piping (like barbushin/php-imap) .. this controller would check the support email inbox every period of time for new emails, if found, it will create a new ticket with the subject of the email as ticket subject, and the body is the ticket body, and the sender as ticket owner.
Some points to think about:

How it would work with a sender who is not registered?
Attachments
We will need to filter email body to remove signature and history quotes (if user is replying to ticket notification).

This is awesome, especially the ticket creation via email - thats a thing i thought of first. If i can help in anyway - you're welcome

My thoughts on the points provided:

I think there should be a checkbox option to either auto create accounts for unrecognized emails or not accept them. It could additionally be limited to only auto create accounts based on the domain address of the sender. So if<EMAIL_ADDRESS>sends an email they get an account created. If<EMAIL_ADDRESS>sends one it is ignored/dropped, or potentially dropped into a category for unregistered accounts.
I think attachments should be added as an easy click link to download/view. Should definitely have some blocks/ignores on certain file types like .exe and .zip etc...
I imagine this one could be tricky to remove their signature. I think mailgun open sourced their code for doing that though so that might be a place to look first.

@jetwes Thank you for your feeling. You can help by coding, translating, testing, or feedback, any of them would be very appreciated :)

@tmcdon89 Thanks for the work time PR.
I agree with you on the first and the second points, and I don't think we should waste time on trying to remove the signature, but it's better (and it might be easier) to focus on removing quoted notification.

it should search first on the incoming email subject, if present, then it's only a new comment, if not present then create a new ticket.
a default category for the tickets created by email
I think it's better to start on attachments in tickets first then to go to the email. As we will need it ready when we will work on the email

I am weighing the pros and cons of where to store the attachments. I definitely think its more straight forward to store them on the file system and simply reference their path from the database but it would also be nice having everything contained within the database to easily move between hosts.  What are others thoughts on where to store the attachments?

+1 - store them on the file system and simply reference their path from the
database.
On Mon, Dec 14, 2015 at 9:39 AM, tmcdon89<EMAIL_ADDRESS>wrote:

I am weighing the pros and cons of where to store the attachments. I
definitely think its more straight forward to store them on the file system
and simply reference their path from the database but it would also be nice
having everything contained within the database to easily move between
hosts. What are others thoughts on where to store the attachments?
—
Reply to this email directly or view it on GitHub
https://github.com/thekordy/ticketit/issues/95#issuecomment-164306998.

yes - i think it's the better option to just reference the path

on the file system and simply reference their path from the database
:+1:

to easily move between hosts
You can use relative path in the database, and the attachment class would transform it to the current path

I'm making the assumption that we would want these files stored within the base folder of the Laravel installation? Something like: base_path() . '/Attachments/foo'
Where foo is where the attachment came from, such as "Comments" or "Email"

I would prefer storage_path().'/app/user/ticketit/attachments' - since the storage is for that type of files.

added new capability to the installer, so the system will check at the start up for any new migrations or settings, if any found, will force the upgrade window for only administrators 8a26f3d7a794022762e33d31afb4983e962a8373

Looking forward to number 11 however i am thinking there is normally a reply within that ticket with a hash number in it and replying to this adds it to the ticket

Hi! Ticketit looks really simple, and easy to use - this is exactly the kind of thing I'm after for a client. However, ticketing-by-email is a must for me, and so I'm a bit lost at this stage (no decent systems out there based on Laravel, which is also a requirement).
Any feedback yet on this roadmap?


Show better error message for Windows abnormal termination.

This helps display better error messages when there is an abnormal termination on Windows.
If rustc crashes, there was a slight mistake in #6270, where the error code was actually negative, so the message was getting hidden behind the --verbose flag.  This changes it show that the abnormal error is always shown if rustc crashes in an unusual way.
This also changes cargo test to display a better message if the test crashes.  Previously:
running 1 test
error: test failed, to rerun pass '--bin crash'

New:
running 1 test
error: test failed, to rerun pass '--bin crash'

Caused by:
  process didn't exit successfully: `D:\Temp\crash\target\debug\deps\crash-b3c2389529da3d3e.exe` (exit code: 0xc0000374, STATUS_HEAP_CORRUPTION)

I didn't add any tests because testing on this on Windows seems a little precarious.  AFAIK, the exact error message depends on the processor (like x86 vs i686), and Windows version.  I was concerned about chasing down every little nuance, though I'm willing to try if it seems important.
Fixes rust-lang/rust#65692 (I think).

:pushpin: Commit 78b14005e9d98e75ece59a8c5c2fa5a515ae84cf has been approved by alexcrichton

:hourglass: Testing commit 78b14005e9d98e75ece59a8c5c2fa5a515ae84cf with merge d8465428ab17de4648ab9b2e593ccf010554ee56...

:broken_heart: Test failed - checks-azure

@bors: retry

:hourglass: Testing commit 78b14005e9d98e75ece59a8c5c2fa5a515ae84cf with merge 79e4d13bef348822d571b4a266995ecb29e90bcd...

:broken_heart: Test failed - checks-azure

@bors: retry
maybe today?

:hourglass: Testing commit 78b14005e9d98e75ece59a8c5c2fa5a515ae84cf with merge d5f87f113315383b9e053e39bfe92254d2f95218...

:broken_heart: Test failed - checks-azure

@bors: retry

:hourglass: Testing commit 78b14005e9d98e75ece59a8c5c2fa5a515ae84cf with merge 79bb06ba64722939f63109c0052e8eee91a42f4e...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 79bb06ba64722939f63109c0052e8eee91a42f4e to master...


Bug

Make it Comkies less
I don't know how to get and how to set please remove the sign option

Ask YouTube to remove the restriction.


Transaction generator: send transactions to multiple nodes

Currently tx generator is able to connect to one single node (of the cluster) and send transactions to this node only. It would be useful to be able to send transactions to multiple nodes in the cluster (because it's more real-life case).
There are two possible ways how to do it. For example, if we have a cluster from 3 nodes (a1, b1 and c1), we could:

run one instance of tx generator with an ability to connect to a1, b1 and c1 and send transactions to all three nodes (sequentially or randomly).
run multiple instances of tx generator, for example tx_gen_a1, tx_gen_b1 and tx_gen_c1, so tx_gen_a1 will send transactions to a1, tx_gen_b1 - to b1 and tx_gen_c1 - to c1.

AFAIU, the first way is assuming some changes in the tx generator's code. The second way is assuming some changes in cluster's configuration.

Note that a node will already send incoming transactions to other nodes via the tx-submission protocol.
Will clients in practice also send their transactions to multiple nodes?

node will already send incoming transactions to other nodes via the tx-submission protocol.

Of course! Currently one node receives all transactions and just share them with other nodes, but we want to be able to test situation when different nodes receive transactions and share them with other nodes. It's more real-life case (transactions are sending to the different nodes of the cluster, not to the same node).

Will clients in practice also send their transactions to multiple nodes?

I don't know. :-)

node will already send incoming transactions to other nodes via the tx-submission protocol.

Of course! Currently one node receives all transactions and just share them with other nodes, but we want to be able to test situation when different nodes receive transactions and share them with other nodes. It's more real-life case (transactions are sending to the different nodes of the cluster, not to the same node).

Ah, I misread. That makes sense indeed.

first step:

accept list of target nodes from command line argument (by name/IP address)
demonstrate on local setup that multiple nodes are receiving txs from the tx generator through "node2node" protocol via TCP/IP

second step:

install the above changes on the cluster
demonstrate that the tx generator is sending txs to multiple nodes

accept list of target nodes from command line argument (by name/IP address)

Not by IP address, but by node ids (as --node-id 0). Corresponding IP addresses and ports will be taken from topology file.

https://github.com/input-output-hk/cardano-node/pull/293


[Dependency Scan] Remove redundant recombination of arguments in 'initCompilerInstanceForScan' into a string, before being re-parsed.

This does not seem to serve a purpose other than corrupting arguments with whitespaces - they get merged into one large string where the whitespace boundary between arguments and whitespaces within arguments are blurred.
Part of rdar://98985453

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

https://github.com/apple/swift-driver/pull/1159
@swift-ci smoke test


[cherry-pick][6774] fix snowflake bug

cherry-pick #6774  6774 fix snow flake bug.

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.0.0-release-prepare@4e812e9). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             2.0.0-release-prepare    #6752   +/-   ##
========================================================
  Coverage                         ?   37.63%
  Complexity                       ?     3209
========================================================
  Files                            ?      648
  Lines                            ?    26383
  Branches                         ?     2849
========================================================
  Hits                             ?     9930
  Misses                           ?    15540
  Partials                         ?      913

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 4e812e9...ccc90e1. Read the comment docs.


Improve Avatar Group

Need to improve this component on Avatar/src/avatar-group.html
Improvements required :
Make the width fit of the new followers section
Any reference image?

You can checkout this one for better understanding.

I would like to work on this issue


Credit

I have used your beautiful svg in my first WordPress plugin, it is free and I want to tag you , wondering how can I do that.
https://wordpress.org/plugins/matrix-pre-loader/

Hi, that's super cool, if you'd like you can throw a link in the readme or just leave it how you have it, either works for me :)


feat: create an append_message_output method

Closes #579 by creating an append_message_output method that works similarily
to append_variable_output.

@mohammadfawaz I'm not sure I understood the context for this so I didn't include Sway tests as I would usually do, do you have some Sway code to show me where this API could interact?

@iqdecay 2 points:
1.) My draft PR is just waiting for this PR to land and be released so I can test it correctly!
Similar to how the tro(transfer to output) opcode requires that a variable output be appended in order to work, the smo opcode (send message output) requires a message output be appended to the call in order to work.
2.) we should consider if there's any other way to approach testing, as we've already seen that there's a circular dep issue between the stdlib (the tests use the SDK) and the SDK (which has some tests which use sway (and the stdlib).
It might be that we work on features like this one in tandem, and you can run your tests pointing at a stdlib branch other than master where the supporting changes have been made already. Once the SDK changes are released, the stdlib changes can merge to master, and then the SDK tests can be updated to use the master stdlib branch...
Open to ideas here.


Renamed the semantics lib to analyzer.

This renames the fe-semantics library to fe-analyzer.
To-Do

[ ] OPTIONAL: Update Spec if applicable

[x] Add entry to the release notes (may forgo for trivial changes)

[x] Clean up commit history

Codecov Report

Merging #207 (d03ed4a) into master (e108a74) will increase coverage by 16.76%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #207       +/-   ##
===========================================
+ Coverage   77.50%   94.27%   +16.76%
===========================================
  Files          47       49        +2
  Lines        3308     3355       +47
===========================================
+ Hits         2564     3163      +599
+ Misses        744      192      -552

Impacted Files
Coverage Δ

analyzer/src/errors.rs
77.27% <ø> (ø)

analyzer/src/namespace/events.rs
100.00% <ø> (ø)

analyzer/src/namespace/operations.rs
90.32% <ø> (ø)

analyzer/src/namespace/scopes.rs
95.29% <ø> (ø)

analyzer/src/namespace/types.rs
85.58% <ø> (ø)

analyzer/src/traversal/_utils.rs
100.00% <ø> (ø)

analyzer/src/traversal/assignments.rs
96.87% <ø> (ø)

analyzer/src/traversal/contracts.rs
100.00% <ø> (ø)

analyzer/src/traversal/declarations.rs
95.65% <ø> (ø)

analyzer/src/traversal/expressions.rs
95.51% <ø> (ø)

... and 47 more

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 e108a74...d03ed4a. Read the comment docs.

Codecov Report

Merging #207 (d03ed4a) into master (e108a74) will increase coverage by 16.76%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #207       +/-   ##
===========================================
+ Coverage   77.50%   94.27%   +16.76%
===========================================
  Files          47       49        +2
  Lines        3308     3355       +47
===========================================
+ Hits         2564     3163      +599
+ Misses        744      192      -552

Impacted Files
Coverage Δ

analyzer/src/errors.rs
77.27% <ø> (ø)

analyzer/src/namespace/events.rs
100.00% <ø> (ø)

analyzer/src/namespace/operations.rs
90.32% <ø> (ø)

analyzer/src/namespace/scopes.rs
95.29% <ø> (ø)

analyzer/src/namespace/types.rs
85.58% <ø> (ø)

analyzer/src/traversal/_utils.rs
100.00% <ø> (ø)

analyzer/src/traversal/assignments.rs
96.87% <ø> (ø)

analyzer/src/traversal/contracts.rs
100.00% <ø> (ø)

analyzer/src/traversal/declarations.rs
95.65% <ø> (ø)

analyzer/src/traversal/expressions.rs
95.51% <ø> (ø)

... and 47 more

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 e108a74...d03ed4a. Read the comment docs.


Update ARM template JSON snippet

Correct the missing extensionProfile.extensions section in the ARM template snippet for KV VM extension for CSES.

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

@gachandw,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

#sign-off

@surbhijain16: I'm sorry - only the author of this article, @shisriva, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@surbhijain16 @prabkumdev @gachandw - The edited code block won't render correctly in the live view; it must be indented to make it nest inside the numbered list. Can you fix this, or would you like us to fix it before merging?


Rename TV shows episodes doesn't rename the episode number

TMM v<IP_ADDRESS>
build date: 2016-03-18 14:25
Windows 10 64-bit version 1511
JDK: 1.8.0_45 x86 Oracle Corporation
What is the actual behaviour?
Every single episode has been scraped and nfo files updated. When I select rename/cleanup function, it moves all files to the correct directory according to titles and season, but incorrectly rename the files itself.
For the episodes title, I use this format: $N - $E - $T, but the $E (episode number) is not renamed.
What is the expected behaviour?
Using the format above, one example of episode title should be 'Flashpoint - 01 - Broken Peace.mp4', but instead it renames to 'Flashpoint - $E - Broken Peace.mp4'.
Steps to reproduce:

Scrape the new episodes.
Go to Settings > TV Shows > tab Renamer > change format of episode file name to $N - $E - $T.
Right click on the title of TV shows in TV shows list > rename/cleanup selected TV shows/episodes.

Additional
Have you attached the Logfile from the day it happened?
tmm_logs.zip

we've done a punch of renamer changes for TV show to make it more stable.
Could you try the latest nightly on test data?
http://nightly.tinymediamanager.org


Building/use conan2 for building

Updated the CMakeLists.txt and the Readme to work with conan2

Removed the obsolete conanfile.py as the recipe is maintained in conan-center
added conanfile.txt to handle dependencies

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

Comparison is base (e93ed4f) 95.28% compared to head (2ce2873) 97.84%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   95.28%   97.84%   +2.55%
==========================================
  Files          16       16
  Lines         276      278       +2
==========================================
+ Hits          263      272       +9
+ Misses         13        6       -7

see 1 file with indirect coverage changes

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

I updated the macos runner to macos-12 as the original runner macos-10.5 is no longer supported.


Highlight search matches

Replaced by https://clusterhq.atlassian.net/browse/FLOC-505
I've searched for "move" on salt.readthedocs.org and then gone to a result, which highlights matches.
http://salt.readthedocs.org/en/latest/ref/modules/all/salt.modules.schedule.html?highlight=move#salt.modules.schedule.move
It would be nice to have this in our docs

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-505. You are welcome to file additional issues in GitHub if that's easier for you.


update graph

The graphs should include:
a percentage index on the horizontal
The actual number in the line of the graph (ie. 200 Hospitals)
The full number of healthsites should be published in the Title (Rwanda data - 506 healthsites)
https://redpen.io/ege170f16ac17fc0ea

It is already implemented in this card:
#243


[MonorepoBuilder] Autoload paths defect when merging under Windows

Given the following monorepo-builder.yml
parameters:
  package_directories:
    - src/Company/Something

  merge_sections:
    - 'require'
    - 'require-dev'
    - 'autoload'
    - 'autoload-dev'

and the following composer.json inside one of the packages
{
  "autoload": {
    "psr-4": {
      "Company\\Something\\ImportantBundle\\": ""
    }
  }
}

the merge command will create the following merge autoload section:
{
    "autoload": {
        "psr-4": {
            "Company\\Something\\ImportantBundle\\": "src\\Company\\Something\\ImportantBundle\\"
        }
    }
}

Please notice the slashes of the path which is Windows-style but needs to be Linux-style whatever the current operating system may be.
Current result should be:
{
    "autoload": {
        "psr-4": {
            "Company\\Something\\ImportantBundle\\": "src/Company/Something/ImportantBundle/"
        }
    }
}

Thanks for testing this on Windows :+1:

Feel free to send PR, closing for now


Automorphisms. Explain better. Also, how to set Galois Keys?

For the simple case of an Automorphism on a polynomial, no batching business, an example or a few would be greatly helpful.

Specifying the full domain domain for every variable would be great appreciated, for example

1. 3^i       % M corresponds to a cyclic row rotation i steps to the left
2. 3^(N/2-i) % M corresponds to a cyclic row rotation i steps to the right

Clearly, the powers of case 1 are increasing and the powers of case 2 are decreasing for increasing positive i...so, they will overlap...where's the divide? For example, replace i by N/2-1 in case 2 and by 1 on case 1. We then have the same power of 3, but interpretations that are not equivalent.
3. cyclic rotation by N/2-1 steps to the right is not equivalent to having done 1 rotation to the left.

Also, what if I want to rotate by more than N/2 positions to the right, or the left?
Now, a practical issue, the compiler keeps yelling at me:
GaloisKeys gen_gal_keys( KeyGenerator &keygen, int len ) {

    vector< uint32_t > galois_elts;
    galois_elts.reserve( len );

    for (int i = 0; i < len; i++)  galois_elts.push_back( 1<< i );
    return keygen.galois_keys( galois_elts );
}

Compiler:
src/file.cpp: In function ‘seal::GaloisKeys gen_gal_keys(seal::KeyGenerator&)’:
src/file.cpp:67:30: error: could not convert ‘seal::KeyGenerator::galois_keys(const std::vector<unsigned int>&)(galois_elts)’ from ‘seal::Serializable<seal::GaloisKeys>’ to ‘seal::GaloisKeys’
   67 |     return keygen.galois_keys( galois_elts );
      |            ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
      |                              |
      |                              seal::Serializable<seal::GaloisKeys>

What am I doing wrong? And, how do I fix this? Also, are the values of the Galois elements meant to be odd integers?

For example, replace i by N/2-1 in case 2 and by 1 on case 1. We then have the same power of 3, but interpretations that are not equivalent.

They are equivalent interpretations.

Also, what if I want to rotate by more than N/2 positions to the right, or the left?

3^{N/2} modulo 2N equals to 1. It rotates back to the original state. It means that we define (N/2) automorphisms here. Elements 3^i and 3^j have no difference if i is congruent to j modulo N/2 (or equivalently 3^i is congruent to 3^j modulo 2N).

On the compiler error:

Call galois_keys_local instead. Because galois_keys returns a seal::Serializable type.
The galois_elts that you choose are not valid. They should be powers of 3 modulo 2N, for example, if N=8, then the elements should be chosen from {1, 3, 9, 11}. If by 1<<i you meant "steps", you should use vector<int> rather than vector<uint32_t>.

On the interpretation piece, instead of pointing out sutff, here's a concrete example:
Say N=8 for simplicity, and consider  f(x) =  1 + 2x + 3x^2 + 4x^3 + 5x^4 + 6x^5 + 7x^6 + 8x^7.

1. (1)             |-->   2 + 3x^1 + 4x^2 + 5x^3 + 6x^4 + 7x^5 + 8x^6 + 1x^7
2. ( N/2 - i = 3 ) |-->   6 + 7x^1 + 8x^2 + 1x^3 + 2x^4 + 3x^5 + 4x^6 + 5x^7

So, rotation to the right by 3 is equivalent to rotation to the left by 1?
How exactly are those two polynomials the same?

As for the compiler error, that works. Thanks.

The automorphisms in SEAL rotate message slots (batched with BatchEncoder or CKKSEncoder) rather than coefficients. What you described is a mapping that sends f(x) to f(x)*x^i mod q mod x^N+1 which is not an automorphism of ring ZZ_q[x]/(x^N+1). If you want to use SEAL without batching, then GaloisKeys and Evaluator::rotation* should not be used at all.

But then, what does
In the polynomial view (not batching), a Galois automorphism by a
Galois element p changes Enc(plain(x)) to Enc(plain(x^p)).

mean?
That is in the documentation.

Here p is an Galois element, a power of 3 modulo 2N.

Could you give a simple example of this process using a small N, say N=8?

Say N=8 and q=17 for simplicity, and consider  f(x) =  1 + 2x + 3x^2 + 4x^3 + 5x^4 + 6x^5 + 7x^6 + 8x^7.

To rotate message slots 3 steps to the right, choose p = 3^3 mod 16 = 11. The automorphism sends x to x^11.

Now you have f'(x) = 1 +  2x^11 + 3x^22 + 4x^33 + 5x^44 + 6x^55 + 7x^66 + 8x^77
                   = 1 + 15x^3  + 3x^6  + 4x    + 5x^4  + 6x^7  + 7x^2  + 9x^5 // mod 17 mod x^N+1

To rotate message slots 1 step to the lest, choose p = 3^-1 mod 16 = 11. The automorphism sends x to x^11, too.

are the 15 and the 9 as coefficients typos?

No, x^11 is -x^3 mod x^N+1, so 2x^11 becomes -2x^3, then reduction modulo 17 gives 15x^3.

Got it. It took me a second to compute its modulus 17 on the coefficients. Now this makes a lot more sense.
So, say I have a ciphertext, with say N=2048, and I wanted to perform an automorphism to send x |--> x^p(i) where i is the steps to shift, say shift to the right, and p(i) is the correct galois element:

Can I do that in SEAL?
How would I do that in seal?

Thanks.

You probably need to check examples 2_encoders and 5_rotation to know how message slots are created.
The automorphism is evaluated by calling Evaluator::apply_galois* or GaloisTools::apply_galois* methods. The required input galois_elt can be calculated from a step by calling GaloisTools::get_elt_from_step. This only evaluates the automorphism on a ciphertext or a polynomial, leaving the output ciphertext not decryptable with the original secret key. Evaluator::rotate* methods evaluate the automorphism and performs a key switching so that the output is decryptable with the original secret key.

There is no documentation for GaloisTools. The constructor for that class is
GaloisTool (int coeff_count_power, MemoryPoolHandle pool)

What is coeff_count_power? What value should I give it?
By the way, you had a constructor from previous versions that build the galois keys directly from passing it the a vector with the steps to shift...why did you get rid of that? Now you have added this extra layer with another class that needs to be initialized and then call this get_elt_from_step() function.

By the way, you had a constructor from previous versions that build the galois keys directly from passing it the a vector with the steps to shift...why did you get rid of that?

We did not. It is here.

What is coeff_count_power? What value should I give it?

It is log 2 of poly_modulus_degree. For example, it should be 12 if poly_modulus_degree is set to 4096.

I had missed it in the docs, but indeed the function is still there. And yes, that works. Thanks for all the explanations.

Say N=8 and q=17 for simplicity, and consider  f(x) =  1 + 2x + 3x^2 + 4x^3 + 5x^4 + 6x^5 + 7x^6 + 8x^7.

To rotate message slots 3 steps to the right, choose p = 3^3 mod 16 = 11. The automorphism sends x to x^11.

Now you have f'(x) = 1 +  2x^11 + 3x^22 + 4x^33 + 5x^44 + 6x^55 + 7x^66 + 8x^77
                   = 1 + 15x^3  + 3x^6  + 4x    + 5x^4  + 6x^7  + 7x^2  + 9x^5 // mod 17 mod x^N+1

To rotate message slots 1 step to the lest, choose p = 3^-1 mod 16 = 11. The automorphism sends x to x^11, too.

Say N=8 and q=17 for simplicity, and consider  f(x) =  1 + 2x + 3x^2 + 4x^3 + 5x^4 + 6x^5 + 7x^6 + 8x^7.

To rotate message slots 3 steps to the right, choose p = 3^3 mod 16 = 11. The automorphism sends x to x^11.

Now you have f'(x) = 1 +  2x^11 + 3x^22 + 4x^33 + 5x^44 + 6x^55 + 7x^66 + 8x^77
                   = 1 + 15x^3  + 3x^6  + 4x    + 5x^4  + 6x^7  + 7x^2  + 9x^5 // mod 17 mod x^N+1

To rotate message slots 1 step to the lest, choose p = 3^-1 mod 16 = 11. The automorphism sends x to x^11, too.

Hello @WeiDaiWD , I am learning the Galois automorphism in FHE and I find this thread quite helpful, does what you are elaborating here come from the BGV paper (https://eprint.iacr.org/2011/277.pdf)? And as a further request, could you elaborate how this process affects the keys?

@KaneX Many papers, like the one you posted, point to the same knowledge. You can also check this one.
The quoted example is an automorphism f'(x) = phi( f(x) ) which maps a polynomial f(x) to f'(x). Let a ciphertext contain a pair of polynomials ( a(x), b(x) ) that is decryptable with the secret key s(x). We know that if we apply the automorphism phi on a(x), b(x), and s(x) we will have phi( a(x) + b(x) * s(x) ) = phi( a(x) ) + phi( b(x) ) * phi( s(x) ) = a'(x) + b'(x) * s'(x). So after applying to automorphism on a ciphertext, to decrypt it, you need a different secret key s'(x). The naïve way is to compute a number of secret keys each of which corresponds to an automorphism/rotation. You will need to keep track of what automorphism/rotation has been performed on the ciphertext to identify which secret key to use for decryption. More significantly, it will be hard to compute two ciphertexts that are resulted from different rotations. Therefore, a better solution that involves key switching is preferred. We perform key switching (from s'(x) to s(x)) on the ciphertext ( a'(x), b'(x) ) to generate a new ciphertext which can be decrypted to the same plaintext with the original secret key s(x). Each Galois key is a key switching key corresponding to an automorphism.


[WFLY-16449] Upgrade ASM to 9.3

https://issues.redhat.com/browse/WFLY-16449

asm.asm module dependencies from searching in EE-9/dist/target:
io/smallrye/config/main/module.xml
org/jboss/as/webservices/server/integration/main/module.xml
org/apache/cxf/ws-security/main/module.xml
org/apache/cxf/main/module.xml
org/apache/cxf/services-sts/main/module.xml
org/apache/cxf/impl/main/module.xml

@scottmarlow  Upgrade to ASM 9.3 is good for webservice. Thanks.

Is ASM upgrade fine from Smallrye PoV @kabir ?

Thanks @ropalka


feat: update golang-github-tklauser-numcpus to 0.3.0

Go module to get the number of CPUs (library)
Issue: https://github.com/deepin-community/sig-deepin-sysdev-team/issues/552
Log: update repo

/lgtm
/approve


[SPARK-34355][SQL] Add log and time cost for commit job

What changes were proposed in this pull request?

Add some info log around commit log.
Why are the changes needed?

Th commit job is a heavy option and we have seen many times Spark block at this code place due to the slow rpc with namenode or other.
It's better to record the time that commit job cost.
Does this PR introduce any user-facing change?

Yes, more info log.
How was this patch tested?

Not need.

Test build #134868 has started for PR 31471 at commit def94f1.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39455/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39455/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39455/

Test build #134873 has started for PR 31471 at commit 185ec52.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39459/

Test build #134868 has finished for PR 31471 at commit def94f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39459/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39459/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134868/

Test build #134873 has finished for PR 31471 at commit 185ec52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134873/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39483/

Test build #134899 has started for PR 31471 at commit ff2cce2.

Actually I have been thinking about this - while I think this helps to track down the elapsed time on committing job, there's still another problem end users confuse the huge delay of commit job with being stuck unless they took the stack trace. Unfortunately Hadoop layer leaves log messages with DEBUG level, hence no log message may be written during commit phase. (I've added latency information on DEBUG log message in Hadoop side via MAPREDUCE-7317, but will be available in 3.3.1+ and the log level is still DEBUG.)
Probably we'd need to print out some informative log message during commit phase periodically - not sure where is the right place to fix, Hadoop vs Spark.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39483/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39483/

@HeartSaVioR thanks for the sharing. Yea, agree. It's better to provide a progress-like stuff during commit job, but seems it's hard to do this at Spark side.

Test build #134906 has started for PR 31471 at commit 9d6eec7.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39488/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39488/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39488/

Test build #134899 has finished for PR 31471 at commit ff2cce2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

OK. I'll leave this till early next week and merge if there's no further comment.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134899/

Test build #134906 has finished for PR 31471 at commit 9d6eec7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134906/

retest this, please

Will merge once either Jenkins or GA passes.

Test build #134999 has started for PR 31471 at commit 9d6eec7.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39582/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39582/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39582/

Test build #134999 has finished for PR 31471 at commit 9d6eec7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Thanks! Merging to master.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134999/

thanks all !


GDScript is unable to do type checking on variables typed with const X = preload("script.gd")

Godot version
Godot 4 beta10
System information
Windows 10 64 bits NVIDIA GeForce GTX 1060
Issue description
In Godot 3 it was possible to type-hint a variable using constant preloaded script. But in Godot 4 it seems GDScript treats those variables as if they were variants with no known type, even though all the information is available.
Lines will not be highlighted as "safe", and attempting to use : to auto-hint variables assigned from properties or methods of the object will cause errors.
Steps to reproduce
body.gd
extends RigidBody3D

main.gd
const MyBody = preload("./body.gd")

func _ready():
	var body := MyBody.new()
	print(body.name)

	# This causes an error
	#var pos := body.position

	body.freeze = true # This is supposed to be safe
	body.queue_free() # This is supposed to be safe

Minimal reproduction project
GDScriptSafeLineConstType.zip

CC @vonagam

I confirm this issue on master. (b29bb11)
This is weird because the editor suggests the right types.

This is a regression. Could not reproduce in beta 8, but could in beta 9.
I'm currently trying to find the culprit commit.

@rune-scape I suspect #69471 have something with this.


viewer initially only shows black screen in revealjs

Some viewers (not all) initially only show a black screen when embedded in a revealjs presentation.
The app is correctly displayed after one of the following:

resize the window
click e.g. the options or full-screen button (buttons are visible, just not the protein)
enter / exit revealjs full screen (press F)

When I used to embed molstar manually I would use iframes of an external html file, in which case the issue didn't seem to occur. So what is happening here?

additionally, the pointer position in the molstar viewer is offset from the mouse (down and right). This must be and interaction between molstar and revealjs's scaling function


[Bug] can't download files

Describe the bug
Files are not downloadable
How to Reproduce
Go to https://f-droid.org/
Download the f droid app from that website.
It will seem Android States to download it since there is such a notification. But in fact nothing happens and eventually the file can't be found anywhere and the notification disappeared.
Expected behavior
File should be downloaded and named correctly.
Everything works fine on fennec or Firefox.
I've been having that issue for a few months
Environment
- DDG App Version: 5.109.0 ml

Device: Xiaomi Redmi note 9s
Android 11

It looks like duckduckgo forwards the download to the download manager of the phone which in my case is:
com.android.providers.downloads.ui
Version <IP_ADDRESS>
The app looks like a built in app provided by Xiaomi but the app Id doesn't mention Xiaomi.

Problem actually seems vpn related. If vpn is disabled in Android settings, the download manager starts downloading.

Thanks @tenzap. We are aware of some issues around downloads (some of them when the VPN in enabled). We are currently working on a project that will improve downloads 👍

Adding a comment to this so I see updates. :)
I encountered a problem trying to download a file today (I do have a VPN enabled). While I didn't try a lot of different files, the ones from this folder on google drive kept giving download failed messages:
https://drive.google.com/drive/folders/1BfeK39boriHy-9q76eXLLqbCwfV17-Gv
On the other hand (doing a minimal amount of troubleshooting), downloading a file from github worked perfectly.
Also on 5.109.0 (multiple devices, multiple android versions).

With release 5.113.1 it is still problematic

I'm currently using DuckDuckGo as my default browser.
Using version 5.113.1
Device - Pixel 2
Android version - 11
I can download from https://f-droid.org/

This has been fixed as per https://github.com/duckduckgo/Android/releases/tag/5.113.1

As I said, with release 5.113.1 I'm still unable to download: as soon as I am on a VPN, download doesn't start. This is the case with 5.113.1 and was also the case at the time I submitted the bug report.
BTW, I mentionned fdroid just as an example, the issue is not related to APK or fdroid, it happens with any file type (PDF...)
Shall I open a new issue?
Device: Xiaomi Redmi note 9s
Android 11


Fixed a bug where volume would not remain same when song was changed.

Basically added UpdateVolume() method to keep previously selected volume on correct level when user switches to next/previous song.

That was really helpful xD, thank you.


Better ajax support with error handling and arbitrary body

I created a custom ajax-post procedure that allows to post an arbitrary string body. I don't want to use http-request which makes things complicated backend side. I want to send scheme values as strings.
Also, it would be nice to have exception support to handle error cases.
WDYT?

There should be a way to specify entire post body. But I'd prefer extending existing function rather than adding a new one.
So how about this?
(http-post path alist)  ; The existing API
(http-post path string) ; Specify the post body directly by `string`

For error cases, it is difficult to support them with exceptions because BiwaScheme does not have exception handling features right now. However, it is easy to extend http-post to return error value.
(http-post path str) ;=> Return string (response body) when succeeded
(http-post path str) ;=> Return something like '(404 "
Not Found
") for error cases

it is easy to extend http-post to return error value.

Indeed. That is what I will do.
It is unclear to me whether biwascheme.js should offer (deep) integration with the browser or nodejs. The alternative I have in mind is good Scheme support with great FFI. The latter is already the case in my opinion.


CPL items do not open with Powertoys Run

Microsoft PowerToys version
0.62.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
If I am trying to open any Control Panel Item (.cpl), it does not work. PowerToys Run offers to open in a web browser. For example, ncpa.cpl:

✔ Expected Behavior
It should open the control panel item, like the Run dialog box does.

❌ Actual Behavior
If I am typing the control panel item command, it offers to open a web page.

Other Software
No response

@Sumitdhiman, you can open them using the Shell plugin like this: >ncpa.cpl
The easiest solution would be to fix/update the alternative tags in settings plugin and add the missing cpl names. An alternative would be to add the to the run commands of program plugin which might be complicated because of the cpl format.
@jaimecbernardo, I prefer updating settings plugin. What do you think?

@Sumitdhiman, you can open them using the Shell plugin like this: >ncpa.cpl
The easiest solution would be to fix/update the alternative tags in settings plugin and add the missing cpl names. An alternative would be to add the to the run commands of program plugin which might be complicated because of the cpl format. @jaimecbernardo, I prefer updating settings plugin. What do you think?

Thanks, that is helpful.
The list is not so big, so adding the missing CPL names would be a good idea.
https://support.microsoft.com/en-us/topic/how-to-run-control-panel-tools-by-typing-a-command-bce95b4d-e8c2-1cd0-ee0d-027679d520a6

This was implemented in release 0.68. Thank you for the bug report!


Link up mentions of network parameters

Tasks

[X] Link up network parameter mentions in existing docs
[X] Look for additional opportunities to link network parameters in existing docs
[ ] Link up network parameter mentions for new pages when Candida's PR is merged

GitHub issue: https://github.com/vegaprotocol/documentation/issues/247
We have a network parameter component which allows us to embed parameter names and values in docs, so we don't have to send someone off to block explorer.
There's a PR to improve the component: https://github.com/vegaprotocol/documentation/pull/177
Once this is done, someone needs to hunt down all the references to network parameters in docs and replace them with the shiny new component. They are not in the autogenerated docs, mostly in tutorials and concept pages, and Candida is working on a PR that will add many more.
Stuff that can be passed to the component:
frontMatter={frontMatter} – every instance should have this, it determines if the parameter values should come from testnet or mainnet. This is derived from the front matter, so make sure it's defined there.
param="blockchains.ethereumConfig" – every instance should have this, it determines which parameter we're referencing
name="Minimum length of voting period" – replace the name of the parameter with something else, e.g. a human-readable name
hideValue={true} – hide the value of the parameter, e.g. when you just want to highlight that a parameter exists
hideName={false} – hide the name of the parameter, e.g. when including in a sentence
suffix="tokens" – append text to the end of the component, in this case the word "tokens"
formatter="governanceToken" – Use governanceToken for token values ("0.000000000000000000000001" will be displayed as "1") and percent for percentages ("0.66" will be displayed as "33%")
And don't forget to import the component at the top of the page, too.
Use the parameter names from this doc: https://lb.testnet.vega.xyz/network/parameters as capitalisation on the public network parameters page won't work.
Afterwards, we could also flip things around and check to see if there are any parameters which aren't mentioned in docs, and look for places where doing so might be beneficial.
Mainnet parameters: https://explorer.vega.xyz/network-parameters
Testnet parameters: https://explorer.fairground.wtf/network-parameters

Neil:
Candida has added the new pages and says there are network parameters in this section which need to be added: https://docs.vega.xyz/docs/testnet/concepts/trading-on-vega

Pull request: https://github.com/vegaprotocol/documentation/pull/274


Can we change the default installation path of selenium2library

Hi,
I have python installed at following location :
C:\apps\qsi_robot\python-2.7.2
I am using Custom batch file to run robot test cases and my python class path is set to following location:
C:\apps\qsi_harness\robot_libs
to import all robot libraries.
Is there a way that i can install selenium2 library in C:\apps\qsi_harness\robot_libs folder instead of
C:\apps\qsi_robot\python-2.7.2\Lib\site-packages

As per your instructions i installed Selenium2Library at C:\apps\qsi_harness\robot_libs.
Now the Selenium2Library keywords are displaying in RIDE but when i run the test case it gives me error message that failed to import library. SyntaxError: 'with' will become a reserved keyword in Python 2.6 (webdriver.py, line 549)
Could you please look into this issue, thanks. bellow is the error message in complete.
Invalid syntax in file 'C:\apps\demo_scripts\test_suites\Selenium_New_Tests\QSIWeb.tsv' in table 'Setting': Importing test library 'Selenium2Library' failed: SyntaxError: 'with' will become a reserved keyword in Python 2.6 (webdriver.py, line 549)
PYTHONPATH: ['C:\apps\qsi_harness\robot_libs\Selenium2Library\lib\decorator-3.3.2', 'C:\apps\qsi_harness\robot_libs\Selenium2Library\lib\selenium-2.8.1\py', 'C:\apps\qsi_harness\lib\robot\robotframework-2.6.0.jar\Lib\robot\libraries', 'C:\apps\qsi_harness\lib\robot\Lib', 'C:\apps\qsi_harness\lib\robot\robotframework-2.6.0.jar\Lib', 'classpath', 'pyclasspath/', 'C:\apps\qsi_harness\\robot_libs', 'C:\apps\qsi_harness\\doc', '.']
CLASSPATH: C:\apps\qsi_harness\lib\robot\robotframework-2.6.0.jar;C:\apps\qsi_harness\bin\qsi_harness_library.jar;C:\apps\qsi_harness\lib\common\activation-1.1.jar;C:\apps\qsi_harness\lib\common\aopalliance-1.0.jar;C:\apps\qsi_harness\lib\common\apache-mime4j-0.6.jar;C:\apps\qsi_harness\lib\common\commons-codec-1.4.jar;C:\apps\qsi_harness\lib\common\commons-io-2.0.1.jar;C:\apps\qsi_harness\lib\common\commons-lang-2.6.jar;C:\apps\qsi_harness\lib\common\commons-logging-1.1.1.jar;C:\apps\qsi_harness\lib\common\commons-net-3.0.1.jar;C:\apps\qsi_harness\lib\common\groovy-all-1.6.7.jar;C:\apps\qsi_harness\lib\common\hsqldb-<IP_ADDRESS>.jar;C:\apps\qsi_harness\lib\common\httpclient-4.0.3.jar;C:\apps\qsi_harness\lib\common\httpclient-cache-4.1.1.jar;C:\apps\qsi_harness\lib\common\httpcore-4.0.1.jar;C:\apps\qsi_harness\lib\common\httpmime-4.0.jar;C:\apps\qsi_harness\lib\common\j2ssh-ant-0.2.9.jar;C:\apps\qsi_harness\lib\common\j2ssh-common-0.2.9.jar;C:\apps\qsi_harness\lib\common\j2ssh-core-0.2.9.jar;C:\apps\qsi_harness\lib\common\j2ssh-daemon-0.2.9.jar;C:\apps\qsi_harness\lib\common\java-getopt-1.0.13.jar;C:\apps\qsi_harness\lib\common\javassist-3.6.0.GA.jar;C:\apps\qsi_harness\lib\common\jms-1.1.jar;C:\apps\qsi_harness\lib\common\jmxri-1.2.1.jar;C:\apps\qsi_harness\lib\common\jmxtools-1.2.1.jar;C:\apps\qsi_harness\lib\common\jolokia-client-java-0.95.jar;C:\apps\qsi_harness\lib\common\log4j-1.2.15.jar;C:\apps\qsi_harness\lib\common\mail-1.4.jar;C:\apps\qsi_harness\lib\common\ojdbc-14.jar;C:\apps\qsi_harness\lib\common\ostermillerutils-1.7.0.jar;C:\apps\qsi_harness\lib\common\postgresql-8.4-702.jdbc4.jar;C:\apps\qsi_harness\lib\common\scannotation-1.0.2.jar;C:\apps\qsi_harness\lib\common\sqljdbc4-3.0.jar;C:\apps\qsi_harness\lib\common\xmlunit-1.3.jar;C:\apps\qsi_harness\lib\arcgis\arcobjects-10.0.jar;C:\apps\demo_scripts\test_data\FileChooserDemo.jar;C:\bison\REL_BISON_V<IP_ADDRESS>\lib\startup.jar;C:\bison\REL_BISON_V<IP_ADDRESS>\lib\workspacemanager.jar;C:\ExtractViewer\lib\extractviewer.jar
Traceback (most recent call last):
File "C:\apps\qsi_harness\lib\robot\robotframework-2.6.0.jar\Lib\robot\utils\importing$py.class", line 86, in non_dotted_import
File "C:\apps\qsi_harness\robot_libs\Selenium2Library_init.py", line 8, in
from keywords import *
File "C:\apps\qsi_harness\robot_libs\Selenium2Library\keywords_init_.py", line 3, in
from _browsermanagement import BrowserManagementKeywords
File "C:\apps\qsi_harness\robot_libs\Selenium2Library\keywords_browsermanagement.py", line 4, in
from selenium import webdriver
File "C:\apps\qsi_harness\robot_libs\Selenium2Library\lib\selenium-2.8.1\py\selenium\webdriver_init.py", line 18, in
from firefox.webdriver import WebDriver as Firefox
File "C:\apps\qsi_harness\robot_libs\Selenium2Library\lib\selenium-2.8.1\py\selenium\webdriver\firefox\webdriver.py", line 21, in
from selenium.webdriver.remote.webdriver import WebDriver as RemoteWebDriver

From your error message it looks like you're running Python version 2.5. The Selenium Python Bindings require 2.6 or 2.7. See:
http://code.google.com/p/selenium/wiki/PythonBindings
I've updated https://github.com/rtomac/robotframework-selenium2library/blob/master/INSTALL.rest to explicitly state that Selenium2Library has the same requirements as the Selenium Python Bindings (since it uses them).

Thanks for your reply by i am using python 2.7.2 version instead of 2.5 (never installed 2.5 version).
Which is installed at following location on my machine:
C:\apps\qsi_robot\python-2.7.2
I am using jython-2.5.2 at following location on my machine:
C:\apps\qsi_robot\jython-2.5.2
SeleniumLibrary (1) is still working properly in my environment but SeleniumLibary2 is giving me this error message.
Could you please help me more to solve this issue.

Hi nadeemsharif,
Sorry for the slow reply. Jython 2.5 implements the same language features as CPython 2.5, so you are effectively using Python 2.5. Is there a reason you can't run this through the CPython 2.7 installation you have there instead of running it through Jython?
The requirement for Python 2.6/2.7 is a requirement of the Selenium Python libraries, so unfortunately there's nothing I can do about in this wrapper library.
Ryan


Server manager bot

  Hello, please help me solve the bot problem! I've been suffering for a week, I can't understand what's wrong! The bot is on the server e, it is active and sends me a menu of commands, but when I give it a command, it refuses to execute it. Server restart notifications also do not come to the channel.      The bot is an administrator, all rights are granted, the channel is registered, the token is also registered, the server mod is connected