jest ignores .babelrc when using babel 7

🐛 Bug Report
Following the docs for using with babel 7 did not work. The babel config was ignored so it barfed on import.
To Reproduce
Follow the jest documentation for babel 7.
Expected behavior
I expect jest to work with babel 7.
Link to repl or repo (highly encouraged)
https://github.com/dadamssg/jest-babel-7
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: macOS High Sierra 10.13.6
    CPU: x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  Binaries:
    Node: 8.11.1 - ~/.nvm/versions/node/v8.11.1/bin/node
    Yarn: 1.6.0 - /usr/local/bin/yarn
    npm: 5.6.0 - ~/.nvm/versions/node/v8.11.1/bin/npm
  npmPackages:
    jest: ^23.6.0 => 23.6.0

Please follow the guide and use babel.config.js instead of .babelrc to make Babel transpile code inside node_modules

@thymikee I tried that initially. I updated the repo. It now uses babel.config.js and still does not work.

Finally figured it out. It appears that {modules: false}  in my babel.config.js was the problem.


Using popen instead of proc_open and trying to simplify things with curl

GitHub Issue: Part of https://github.com/Islandora-CLAW/CLAW/issues/978
What does this Pull Request do?
Bypasses ApiXMiddleware and CmdExecuteService entirely and uses popen to simplify the number of streams we're managing.  This is just for @seth-shaw-unlv to test.  If it actually works, i'll update CmdExecuteService accordingly.  Otherwise you can just ignore this.
Interested parties
@seth-shaw-unlv

Closing for now since https://github.com/Islandora-CLAW/Alpaca/pull/58 does this trick and this just obfuscates things.


syzkaller regression "failed to run executor"

Hello,
I updated to latest master and found that with commit a38fb99b3fbff0c988e64bf4bf277071e18b18a a regression was introduced causing continuous error-messages à la:
2024/05/24 15:42:10 vm-2: crash: SYZFATAL: failed to run executor version: failed to run executor version: failed to run ["/syz-executor" "version"]: exi

Please let me know if I can be of any help to debug this.

Was there anything after "exi"?
Could you please build "make executor" and run "bin/linux-amd64/syz-executor version" on your target machine? Does it fail? How?

Was there anything after "exi"?

Nope, nothing!

Could you please build "make executor" and run "bin/linux-amd64/syz-executor version" on your target machine? Does it fail? How?

It works:
$ bin/linux_amd64/syz-executor version
linux amd64 3efdd6dbffe8a6c3c20a04e835fd00c973fcb9a2 a10a183e260f0ea1a0c37e84ca5c60f28c13e3fd

Humm... This is strange. I don't have any ideas at the moment. syz-fuzzer just executes "syz-executor version" in the VM...


Add bindings for XICCEncodingStyle, copied from /usr/include/X11/Xutil.h

Edit:
I tried defining a rust enum for this but there's already a XICCEncodingStyle: c_int decl for this in xlib.rs which I didn't notice before sending the PR.
I've changed this to just exporting the XICCEncodingStyle variants as constants.
Thanks!

Don't merge just yet, I think I made a typo.

Fixed.
I had typo'd the values as flags (XUTF8StringStyle = 1<<3) rather than ints (XUTF8StringStyle = 4).

Heya!
Can you push a new crates.io version for this?
Thanks!


Bring carplay support to iPad/iPod

A good feature to add is to bring carplay support to iPads and iPods. Carplay is disabled/removed from them for no reason. Thanks for the free and very good tweak.

it's not really possible as most of the CarPlay underlaying implementation is inside frameworks that do not exist in the iPad and iPod dyld shared caches


Fix report generation

With the changes in #135, the schema has changed, which will break the report generation. Update the report generator so that it works with the current schema.

This is made obsolete by 179.


v11

Hi,
Any chance for a update to the v11 ?
Thanks

+1

does it not work on v11? or is it just a version bump in the manifest

nono is work just fine, it just for avoid the warning and made it searchable on the modules page

should be fixed


TypeError: NodePath.set_shader() argument 1 must be Shader, not NoneType

I have the following error traceback when trying to use complexpbr:
"C:\Program Files\Panda3D-1.10.13-x64\python\python.exe" D:\Workspace\Python\project-naer\main.py
Known pipe types:
  wglGraphicsPipe
(all display modules loaded.)
AL lib: (EE) ALCwasapiPlayback_resetProxy: Audio client returned buffer_len < period*2; expect break up
:shader(error): Could not find shader file: D:\Workspace\Python\project-naer\library\complexpbr\ibl_v.vert
:express(warning): Filename uses Windows-style path: D:\Workspace\Python\project-naer\library\complexpbr\output_brdf_lut.png
:express(warning):   expected Unix-style path: /d/Workspace/Python/project-naer/library/complexpbr/output_brdf_lut.png
:gobj(error): Texture::read() - couldn't read: D:\Workspace\Python\project-naer\library\complexpbr\output_brdf_lut.png
:express(warning): Filename uses Windows-style path: D:\Workspace\Python\project-naer\library\complexpbr\output_brdf_lut.png
:express(warning):   expected Unix-style path: /d/Workspace/Python/project-naer/library/complexpbr/output_brdf_lut.png
:gobj(error): Unable to find texture "D:\Workspace\Python\project-naer\library\complexpbr\output_brdf_lut.png" on model-path /d/Workspace/Python/project-naer;/c/Program Files/Panda3D-1.10.13
-x64/etc/..;/c/Program Files/Panda3D-1.10.13-x64/etc/../models
complexpbr message: Defaulting to dummy LUT.
Traceback (most recent call last):
  File "D:\Workspace\Python\project-naer\main.py", line 213, in <module>
    main = Main()
  File "D:\Workspace\Python\project-naer\main.py", line 56, in __init__
    library.complexpbr.apply_shader(self.render)
  File "D:\Workspace\Python\project-naer\library\complexpbr\__init__.py", line 165, in apply_shader
    complexpbr_rig_init(node, intensity=intensity, lut_fill=lut_fill)
  File "D:\Workspace\Python\project-naer\library\complexpbr\__init__.py", line 128, in complexpbr_rig_init
    node.set_shader(base.complexpbr_shader)
TypeError: NodePath.set_shader() argument 1 must be Shader, not NoneType

Process finished with exit code 1

Here is my code (simplified):
class Main(direct.showbase.ShowBase.ShowBase):

    def __init__(self) -> None:

        super().__init__(self)
        library.complexpbr.apply_shader(self.render)

Hi there, thank you for submitting this error report. It looks like you are importing complexpbr from a different module you've called "library". I actually haven't tested complexpbr for this use case, so I may need to do so and then iterate before this will work as expected. You can reference https://github.com/rayanalysis/Panda3D-Arena-FPS-Sample-Program to see the "standard" way of importing and using complexpbr.

Thanks for your reply!
I also tried to import it from the same folder as my main.py file and the result is the same.

If I may, what happens when you the Arena FPS sample program?

Interesting! I have the same error.
I tried with complexpbr located in Python's site-packages folder.

Here is a .zip of the current complexpbr wheel. Uninstall any complexpbr versions you may have, and then use "pip install" with this .whl file once you unzip it.
You may have copied the complexpbr folder itself into your site-packages, which would make this the second time it's happened (sigh), and is not a supported way of installing complexpbr. I will get this on pypi (pip) sometime but I've been too busy this year so far.
panda3d_complexpbr-0.5.3-py3-none-any.zip

Nevermind, I see the issue now. It was probably created when I accepted the last PR. I will correct and update the repo.

Excellent news! I'll wait for the update!

Thanks, I have just checked in that reversion commit. I've attached a new .whl below or you can build yourself from the latest commit.
Let me know how it goes, I know there is a bug on Python 3.8 so please test it with a recent Python version, IE 3.11.4
panda3d_complexpbr-0.5.3-py3-none-any.zip

Thanks for the update! It's better: now it runs!
Now, I just need to figure out why my scene is very dark, almost like there are no lights (I can still see objects).
At the moment, I am using the default 3.7 provided in P3D's SDK, which type of bug are you referring to?

I haven't taken the time to fully diagnose the issue on Python <=3.8 but it involves node transformation issues. I highly recommend using Python 3.11 or higher.
Closing as the problem has been solved. Feel free to open another issue if any come up and are not related to Python version support.

I experience the same error in my project. Here is the PR with the changes I made: https://github.com/ekiefl/pooltool/pull/144
I tested out complexpbr with arena.py and no error is observed (though I'm getting whiteout graphics, probably due to me using an 8 year old mac).


Update of Kitti 360

When will the validation and test set be available? Thanks.

I would be interested too @yiyiliao


Fix enter playmode exception caused by search

Summary
Fix null reference exception thrown when entering play mode with non-zero search hits.
Details
Explanation of the root cause can be found in https://github.com/sabresaurus/PlayerPrefsEditor/issues/5#issuecomment-777046984 and in the commit e60aaf0bfe1a5b0ba0acfbbe21be3dec6eb2fcd9 message. Unity does serialization of PlayerPrefsEditor since it inherits EditorWindow.
How to test
Checkout target branch. Open editor, search "unity", make sure search results are not empty. Enter playmode and observe exception in the console.
Checkout source branch. Repeat steps and observe no exception. Check that search results are persistent when stopping play mode. Check editor behaves as before when opening and closing.
Tested with 2019.4.8f1 on Windows.
Would be great if @tracer8 could test source branch as well.
Closes #5

Thanks for checking. I see the problem. For macOS before in OnGUI both deserializedPlayerPrefs was assigned and GUILayout.Label was added based on last write time of plist file. I need to think how to move that back to OnGUI since drawing depends on plist file path.

Hey @rfadeev looks good, thanks!

@sabresaurus You are welcome 😊


Need to handle AstraDB timeouts

I got this AstraDB exception:
17:34:53.524 [main] INFO  a.l.a.v.c.CassandraAssetsManagerProvider -- Executing: CREATE TABLE IF NOT EXISTS "doc_qa_demos"."chatbot2" (
filename TEXT,
chunk_text_length TEXT,
chunk_num_tokens TEXT,
chunk_id TEXT,
contents TEXT,
name TEXT,
embeddings_vector VECTOR<FLOAT, 1536>,
PRIMARY KEY (filename, chunk_id));
17:34:57.982 [main] INFO  a.langstream.impl.nar.NarFileHandler -- Closing classloader NarFileClassLoader{name='langstream-vector-agents-0.1.1-SNAPSHOT-nar.nar'}
17:34:58.007 [main] INFO  a.langstream.impl.nar.NarFileHandler -- Closing classloader NarFileClassLoader{name='langstream-kafka-runtime-0.1.1-SNAPSHOT-nar.nar'}
com.datastax.oss.driver.api.core.DriverTimeoutException: Query timed out after PT2S
        at com.datastax.oss.driver.api.core.DriverTimeoutException.copy(DriverTimeoutException.java:34)
        at com.datastax.oss.driver.internal.core.util.concurrent.CompletableFutures.getUninterruptibly(CompletableFutures.java:149)
        at com.datastax.oss.driver.internal.core.cql.CqlRequestSyncProcessor.process(CqlRequestSyncProcessor.java:53)
        at com.datastax.oss.driver.internal.core.cql.CqlRequestSyncProcessor.process(CqlRequestSyncProcessor.java:30)
        at com.datastax.oss.driver.internal.core.session.DefaultSession.execute(DefaultSession.java:230)
        at com.datastax.oss.driver.api.core.cql.SyncCqlSession.execute(SyncCqlSession.java:54)
        at ai.langstream.agents.vector.cassandra.CassandraAssetsManagerProvider$CassandraTableAssetManager.execStatements(CassandraAssetsManagerProvider.java:111)
        at ai.langstream.agents.vector.cassandra.CassandraAssetsManagerProvider$CassandraTableAssetManager.deployAsset(CassandraAssetsManagerProvider.java:102)
        at ai.langstream.api.runner.assets.AssetManagerAndLoader$1.lambda$deployAsset$2(AssetManagerAndLoader.java:58)
        at ai.langstream.api.runner.assets.AssetManagerAndLoader.executeWithContextClassloader(AssetManagerAndLoader.java:26)
        at ai.langstream.api.runner.assets.AssetManagerAndLoader$1.deployAsset(AssetManagerAndLoader.java:58)
        at ai.langstream.impl.deploy.ApplicationDeployer.setupAsset(ApplicationDeployer.java:127)
        at ai.langstream.impl.deploy.ApplicationDeployer.setupAssets(ApplicationDeployer.java:101)
        at ai.langstream.impl.deploy.ApplicationDeployer.setup(ApplicationDeployer.java:86)
        at ai.langstream.runtime.tester.LocalApplicationRunner.deployApplicationWithSecrets(LocalApplicationRunner.java:134)
        at ai.langstream.runtime.tester.Main.main(Main.java:159)

Steps to reproduce:
cd examples/applications/azure-document-ingestion
langstream docker run test -app . -s ../../secrets/secrets-azure-document-ingestion.yaml --langstream-runtime-docker-image public.ecr.aws/y3i6u2n7/datastax-public/langstream-runtime-tester --langstream-runtime-version latest

Run the pipeline below:
assets:
  - name: "langstream-keyspace"
    asset-type: "astra-keyspace"
    creation-mode: create-if-not-exists
    config:
      keyspace: "${secrets.astra.keyspace}"
      datasource: "AstraDatasource"
  - name: "langstream-docs-table"
    asset-type: "cassandra-table"
    creation-mode: create-if-not-exists
    config:
      table-name: "${secrets.astra.table}"
      keyspace: "${secrets.astra.keyspace}"
      datasource: "AstraDatasource"
      create-statements:
        - |
          CREATE TABLE IF NOT EXISTS "${secrets.astra.keyspace}"."${secrets.astra.table}" (
          filename TEXT,
          chunk_text_length TEXT,
          chunk_num_tokens TEXT,
          chunk_id TEXT,
          contents TEXT,
          name TEXT,
          embeddings_vector VECTOR<FLOAT, 1536>,
          PRIMARY KEY (filename, chunk_id));
        - |
          CREATE CUSTOM INDEX IF NOT EXISTS ${secrets.astra.table}_ann_index ON ${secrets.astra.keyspace}.${secrets.astra.table}(embeddings_vector) USING 'StorageAttachedIndex';
name: "Embeddings processor"
pipeline:
  - name: "Azure blob source"
    type: "azure-blob-storage-source"
    configuration:
      container: "${secrets.azure.container}"
      endpoint: "https://${secrets.azure.storage-account-name}.blob.core.windows.net/${secrets.azure.container}"
      storage-account-name: "${secrets.azure.storage-account-name}"
      storage-account-key: "${secrets.azure.storage-access-key}"
  - name: "Extract text"
    type: "text-extractor"
  - name: "Normalise text"
    type: "text-normaliser"
    configuration:
      make-lowercase: true
      trim-spaces: true
  - name: "Split into chunks"
    type: "text-splitter"
    configuration:
      splitter_type: "RecursiveCharacterTextSplitter"
      chunk_size: 4000
      separators: [ "\n\n", "\n", " ", "" ]
      keep_separator: false
      chunk_overlap: 2000
      length_function: "cl100k_base"
  - name: "Convert to structured data"
    type: "document-to-json"
    configuration:
      text-field: text
      copy-properties: true
  - name: "prepare-structure" # This step assumes that the filename for the doc follows the naming convention.
    type: "compute"
    configuration:
      fields:
        - name: "value.product_name"
          expression: "fn:split(properties.filename, ' ')[0]"
          type: STRING
        - name: "value.product_version"
          expression: "fn:split(properties.filename, ' ')[1]"
          type: STRING
  - name: "compute-embeddings"
    id: "step1"
    type: "compute-ai-embeddings"
    configuration:
      model: "${secrets.open-ai.embeddings-model}" # This needs to match the name of the model deployment, not the base model
      embeddings-field: "value.embeddings"
      text: "{{ value.contents }}"
      batch-size: 10
      # this is in milliseconds. It is important to take this value into consideration when using this agent in the chat response pipeline
      # in fact this value impacts the latency of the response
      # for latency sensitive applications, consider to set batch-size to 1 or flush-interval to 0
      flush-interval: 500
  - name: "Write to Astra"
    type: "vector-db-sink"
    configuration:
      datasource: "AstraDatasource"
      table-name: "${secrets.astra.table}"
      keyspace: "${secrets.astra.keyspace}"
      mapping: "filename=value.name,chunk_text_length=value.chunk_text_length, chunk_num_tokens=value.chunk_num_tokens, chunk_id=value.chunk_id, contents=value.text, name=value.name"

with this configuration.yaml:
configuration:
  resources:
    - type: "vector-database"
      name: "AstraDatasource"
      configuration:
        service: "astra"
        clientId: "token"
        secret: "${ secrets.astra.token }"
        token: "${ secrets.astra.token }"
        database: "${ secrets.astra.database }"
        environment: "PROD"
        password: "${ secrets.astra.token }"

Issue is transient. (Doesn't always occur.) But, it's more likely to occur if the table needs to be created.
Desired behavior:
AstraDB queries should be retried when creating new assets.
I'm not sure if this should be handled by the pipeline or lower in the DB interaction layer.
Perhaps we should allow the user to specify if it should be retried on failure or not?

When you run in k8s the pod is restarted.
With https://github.com/LangStream/langstream/pull/615 when you are running in docker mode the container will gracefully exit.


we can add hourly weather forecast panel for current day.

we can add hourly weather forecast panel for current day.

I'm willing to work on it , kindly assign this to me

only one issue at a time :)


cobra cmd errors use logger error instead of default std err

This is more to start a discussion. Basically I would prefer changing cobra command field RunE to Run and handle errors (using logger.Error) inside command. The reason is that cobra uses default std err for all outputs, even for help. (try kube-aws --help 2> /dev/null). So by overriding output (there is no option for err and out), help will be in red if color=true option is set.
To test this pull request you can run kube-aws --color=true init
Let me know what you think.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1352   +/-   ##
=======================================
  Coverage   36.84%   36.84%
=======================================
  Files          70       70
  Lines        4359     4359
=======================================
  Hits         1606     1606
  Misses       2526     2526
  Partials      227      227

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 3c38f29...9e1eb09. Read the comment docs.


Update write_metadata_file discussion in documentation

Makes minor changes to the DataFrame-Parquet documentation in light of recent PRs (like https://github.com/dask/dask/pull/8988).

@jcrist - Do you think it makes sense to add a subsection on calculate_divisions here?
I don't want this page to get to long (I can add a link to an "Advanced Parquet Options" doc at some point, but the initial landing page should probably only cover the "must-know" info)


[OnlineStateTracker]: Could not reach Cloud Firestore backend. Connection failed 1 times. Most recent error: Status{code=UNAVAILABLE, description=Unable to resolve host firestore.googleapis.com, cause=java.lang.RuntimeException: java.net.UnknownHostException: Unable to resolve host "firestore.googleapis.com": No address associated with hostname

Hey guys, I am getting this error suddenly. The app was running fine . Then suddenly I opened back the emulator, I was getting this error .
It is happening for all my screens where I have integrated firebase too

This probably indicates a network connection error or a glitch at firebase server side, so I don't think Flutterfire can do anything about it.
I suggest you to look at https://github.com/firebase/firebase-android-sdk/issues for similar issue or raise it there or reach out to firebase support for further resolution. https://firebase.google.com/support/troubleshooter/contact
Closing from here, as this doesn't look like flutterfire / cloud_firestore issue.


Azure Free Accounts limited to 4 BS Series vCPUs

Lab 2 IaaS deployment scales the BS series vm's up to  8 total vCPUs. Azure Free Accounts have the vCPU quota set at 4 total cores and cannot be increased without changing the account to Pay-as-You-Go. The Lab documentation should have attendees scale the initial 2 VM's back down to Standard_B1s before scaling out to 4 VM's.

Fixed


Implement test framework support for example testing

See https://docs.google.com/document/d/1ilZUZvuH_OaebxC7jdpySmJdEju_3neytMaamIqy9fI/edit?ts=5cd49196# for details.

Still working on this.

What is left to do here? Write tests?

This was implemented by https://github.com/istio/istio/pull/17119


Update winit version

Winit v0.30.0 was released awhile back which altered the event loop API to be trait-based. This would require a larger refactor and can't be done until some upstream dependencies have been updated as well.
Release: https://github.com/rust-windowing/winit/releases
blocked on: https://github.com/emilk/egui/issues/4437
PR: https://github.com/lukexor/tetanes/pull/283

Fixed in https://github.com/lukexor/tetanes/pull/315


High CPU usage

I've noticed some very high CPU usage when using wishlist as a library. Unfortunately I can't reproduce it or add much debug context, but I noticed, that I had +500% CPU usage sometimes with a lot of wishlist (wishbox to be precise) processes running even though, I had no active session at that moment.

I was a 100% sure I had replied to this, seems like I didn't, sorry.
Anyway, I did tried to reproduce this, and couldn't.
Since no one else reported it, and wishlist changed a lot since this was opened, I think I'm going to close it.
Feel free to re-open if it still happens or if you have more details!
Thanks

Hi,
I have the same issue on my box, the VM I am using has 2 cores and wishlist eats nearly all of it (180% of 200%).
I link the logs from the app, the output of ps -aux and a screen of htop. If you need anything else, I would be glad to give it to you.
Hoping this will resolve this problem.

logs.txt
ps.txt


Update of new_labels.md on a bug and corresponding workaround in OpenShift 4.x

A new section is added to new_labels.md on a bug in OpenShift 4.x. The method of updating output image labels during the s2i build process is not possible currently in OpenShift 4.x and a corresponding workaround is described.

/ok-to-test
/lgtm

thanks!


Optimize to handling of large outputs in notebooks

Fixes #11031

Codecov Report

Merging #11350 (186bc70) into main (01481b2) will decrease coverage by 0%.
The diff coverage is 83%.

@@          Coverage Diff           @@
##            main   #11350   +/-   ##
======================================
- Coverage     63%      63%   -1%
======================================
  Files        476      476
  Lines      33925    33915   -10
  Branches    5496     5494    -2
======================================
- Hits       21556    21539   -17
- Misses     10324    10328    +4
- Partials    2045     2048    +3

Impacted Files
Coverage Δ

...c/kernels/execution/cellExecutionMessageHandler.ts
77% <83%> (-1%)
:arrow_down:

src/platform/common/process/baseDaemon.node.ts
50% <0%> (-4%)
:arrow_down:

src/kernels/installer/productInstaller.node.ts
78% <0%> (-2%)
:arrow_down:

src/kernels/helpers.ts
66% <0%> (-2%)
:arrow_down:

src/notebooks/controllers/kernelRanking/helpers.ts
58% <0%> (-1%)
:arrow_down:

src/platform/msrCrypto/msrCrypto.js
19% <0%> (-1%)
:arrow_down:

src/kernels/raw/launcher/kernelProcess.node.ts
74% <0%> (+<1%)
:arrow_up:

src/platform/common/utils/decorators.ts
91% <0%> (+<1%)
:arrow_up:

.../notebooks/controllers/vscodeNotebookController.ts
82% <0%> (+1%)
:arrow_up:

src/kernels/jupyter/jupyterKernelService.node.ts
76% <0%> (+1%)
:arrow_up:

... and 1 more

🎉🎉🎉


[VIS] Modularity of sections

The 3 separate qtop sections displayed should be modular, the user should be able to display additional information/sections, provided in a config file

This is done for the existing code, the new features will be built with that in mind


Updated fpdi, fpdf and mpdf

1. Why is this change necessary?
FPDI 1.6.2 is deprecated and will no longer be updated. It will throw some warnings like: "Function each() is deprecated since PHP 7.2; Use a foreach loop instead."
2. What does this change do, exactly?
Update fpdi, fpdf and mpdf
3. Describe each step to reproduce the issue or behaviour.

4. Please link to the relevant issues (if any).

5. Which documentation changes (if any) need to be made because of this PR?

6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @MaximilianKresse,
unfortunately merging of Documents seems to be broken now. Can you check that please? It generates blank pages.

🤦♂ I'm sorry...
Can you check again please?

Wohoo looks good for me :tada:

Thanks for your Contribution! :blue_heart:


gifs not supported

Hi,
I am currently having issues with gif images and AWS CloudFormation (lambda), as it seems that serverless-image-hander version 6.0.0 is using sharp 0.27.0, which does not support gif images: https://github.com/aws-solutions/serverless-image-handler/blob/main/source/image-handler/package.json
Are there any plans to update sharp for serverless-image-handler please? 0.30.1 should support gifs.

Would future support include animated gif support or static gif only?
Will thumbnor eventually support webm, mp4, or webp? Thanks

Since sharp v0.30.0 added support for gif output, most likely animated gif support will be added in the Serverless Image Handler too. We will look into webm, mp4, or webp and add it to our backlog.

Also waiting on gif support please 🙏

I long for this to be :pray:

Same here :)

Also waiting on gif support please :)) 🙏🙏

Same here

Same, GIF support would be great :)

Would like to see support for this as well 👍

+1

I switched to cloudflare images instead - it’s significantly more powerful
Ryan

On 27 Apr 2022, at 00:19, Ermia Qasemi @.***> wrote:

Same here :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

gif and animated gif support should be available in the current v6.1.0 release made today, please file any issues that you may encounter


Update project to xcode 10

Updated the project to xcode 10.
Currently, the version of ReactiveCocoa is fixed to 8.0.0.

Close this because this is included in #7


Add sane doc reports

Used for the automation that produces docx reports.
Currently should be in dev mode.

@agamm build is failing. It seems that sane_src is empty.

@agamm plz note build fails here

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>6
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>6

Docker Metadata

Image ID: sha256:26dd613035fa95c804a693676e1894ba16b6f4be8894c9e6dbea1579ad6f5e35
Created: 2019-05-30T14:05:17.619226879Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:34763ddbe763036dd9777dd6ea5bf0a9f7af6eb4
org.opencontainers.image.version:<IP_ADDRESS>6

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>9
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>9

Docker Metadata

Image ID: sha256:91da32e9ddad03ec1209148960881dce4091a96f22d6c386c49bd43ada158393
Created: 2019-06-02T11:18:03.988658687Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:3800ebfc96ace0a01350e31adbfcddcfe517f6dc
org.opencontainers.image.version:<IP_ADDRESS>9

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>0
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>0

Docker Metadata

Image ID: sha256:3ce3dc8e965fdb2e975d0633d0a5ab19a29c4635ea7ca9fe63a9e3613108f9c9
Created: 2019-06-02T15:17:44.931974797Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:869e42d41785b68bd1f7165601211a754d463929
org.opencontainers.image.version:<IP_ADDRESS>0

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>7
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>7

Docker Metadata

Image ID: sha256:f6d57ab3ef14178689eb9987ab60cee8f402f91f20579d93f900eb9a070af8db
Created: 2019-06-10T13:58:21.083656956Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:b64a6a5c966952e87f9c2a6e784c6c5801604ccd
org.opencontainers.image.version:<IP_ADDRESS>7

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>8
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>8

Docker Metadata

Image ID: sha256:56e2c6080d18e200596eeefdeb9324a014cced7b1518c8004a1c5771d5cd64e8
Created: 2019-06-11T13:37:07.407380237Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:7cc16d673054bfc0dff860b2d807dff066219f3d
org.opencontainers.image.version:<IP_ADDRESS>8

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-doc-reports:<IP_ADDRESS>9
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-doc-reports:<IP_ADDRESS>9

Docker Metadata

Image ID: sha256:1610ac066a29ce110883e0453b91fe77286fa294b45c22c715b5eb758bfb0821
Created: 2019-06-11T14:44:26.702096032Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:d593681153fc9a745eed656ed238581d993c6916
org.opencontainers.image.version:<IP_ADDRESS>9

Docker Image Ready - Production
Docker automatic build at CircleCI has deployed your docker image: demisto/sane-doc-reports:<IP_ADDRESS>0
It is available now on docker hub at: https://hub.docker.com/r/demisto/sane-doc-reports/tags
Get started by pulling the image:
docker pull demisto/sane-doc-reports:<IP_ADDRESS>0

Docker Metadata

Image ID: sha256:c0a3c3c20d579cd76fcb040f92b707934fdc95ad86bd58bdf15503c7c9894fd8
Created: 2019-06-11T15:13:55.30496471Z
Arch: linux/amd64
Command: ["python3"]
Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=0D96DF4D4110E5C43FBFB17F2D347EA6AA65421D
PYTHON_VERSION=3.7.3
PYTHON_PIP_VERSION=19.1

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:d00dcdcabda414a94cf8eff8fab0891ec2382f90
org.opencontainers.image.version:<IP_ADDRESS>0


Resolving IEnumerable should return an empty list if nothing is registered

Repro:

Resolve IEnumerable<INotRegistered> where nothing is registered

Result

The AspNet seems to return null

Expected

Return Enumerable.Empty<INotRegistered>() or anything not null

It seems all other implementations work fine with this expectation.

+1

Calling GetServices does this. Covered by https://github.com/aspnet/DependencyInjection/blob/dev/test/Microsoft.Framework.DependencyInjection.Tests/ServiceProviderExtensionsTest.cs#L138-L150


How to use a connection string with @azure/data-tables

Add example how to use a connection string with NPM package @azure/data-tables in Azure functions.

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

ID: 6548247e-f52f-8a4d-9ca5-b15314a7bb33
Version Independent ID: f671c75c-8275-4e9a-7c4d-1508691139f7
Content: Storage connection strings - Azure Data Explorer
Content Source: data-explorer/kusto/api/connection-strings/storage.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@vincesp Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hi @vincesp, thanks for reaching out.
Response from @ArielYehezkely:
This document lists the storage connection strings supported by Kusto.
Kusto doesn’t support connection strings of tables.

@vincesp thanks for being part of our contributor community! Please feel free to @mention me with any further questions or open up a new issue if needed. Since this issue has been answered, I'm going to proceed to close it at this time.
#please-close


Example seems to miss a named parameter

Running the example
Set-NpsRadiusClient -Name "WirelessAP" -Address "<IP_ADDRESS>" $False -SharedSecret "1234567890"
gives me an error, presumably because -Enabled is missing in front of the Boolean.

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

ID: bb23b7dc-17aa-fa0c-d7a8-ebd3dc831821
Version Independent ID: 19a7def1-340b-153b-486b-36d020b31807
Content: Set-NpsRadiusClient (nps)
Content Source: docset/windows/nps/set-npsradiusclient.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Hello, @3ter  thank you for providing feedback.
Note that the form that you have submitted this feedback from is intended for product documentation issues only.
Please consider checking any relevant resources and asking the community about your problem via the following link.
PowerShell Community
Thanks.

@officedocsbot close

@officedocsbot open

@officedocsbot reopen

@3ter Thank you for sharing your feedback. Can you please format the whole command that works for you so that we can further look in this issue.

Set-NpsRadiusClient -Name "WirelessAP" -Address "<IP_ADDRESS>" -Enabled $False -SharedSecret "1234567890"

@3ter  With the help of your feedback, the issue has been reviewed and an update to the documentation has been published. Thanks for your feedback.

@officedocsbot close


#12883: Add initial unit tests for N300

Tracking Issue : #12883
Currently we don't have any tests for CCL to be run in N300, this PR will add initial tests to be run as post commit
Checklist

[x] Post commit CI passes - https://github.com/tenstorrent/tt-metal/actions/runs/10955272562
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

I forgot to mention in my review the but tests have not yet been enabled in the post-commit run in your PR. Please add that.


comma: 1.4.0 -> 1.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nix-community/comma/releases.
meta.description for comma is: Runs programs without installing them
meta.homepage for comma is: https://github.com/nix-community/comma
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.1 with grep in /nix/store/j361ifwhprknw2675q9zg5af792mjrsf-comma-1.4.1
found 1.4.1 in filename of file in /nix/store/j361ifwhprknw2675q9zg5af792mjrsf-comma-1.4.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
comma

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j361ifwhprknw2675q9zg5af792mjrsf-comma-1.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A comma https://github.com/r-ryantm/nixpkgs/archive/0bfbf10d459460e80af91bd103b52c32c756f8e5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j361ifwhprknw2675q9zg5af792mjrsf-comma-1.4.1
ls -la /nix/store/j361ifwhprknw2675q9zg5af792mjrsf-comma-1.4.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

comma

Maintainer pings
cc @Enzime @Artturin @marsam for testing.

Result of nixpkgs-review pr 214112 run on aarch64-darwin 1

1 package built:

comma


cloud-provider(aws): Need to correct content on cluster id in aws tagging section

Fail to map aws resource due to wrong tag entry ,Currently doc says CLUSTERID can be any string you choose   but CLUSTERID should be  required   k8s cluster id .(eg:  c-jqb69) .due to this end users got confused and  raised a concern on this. Please update the doc.
eg:
 Sample API call " https//mysite.rancher.com/v3/clusters/c-jqb69" , Here c-jqb69 is the cluster Id  )

Open this page https://rancher.com/docs/rke/latest/en/config-options/cloud-providers/aws/

Check the below content:

Key=kubernetes.io/cluster/, Value=owned
 can be any string you choose. However, the same string must be used on every resource you tag. Setting the tag value to owned informs the cluster that all resources tagged with the  are owned and managed by this cluster.
Please change content from any string  you choose to required k8 cluster id

Although the PR linked above does address some issues with the AWS docs, the wording update suggested here still needs to be made. The current version:

<CLUSTERID> can be any string you choose. However, the same string must be used on every resource you tag. Setting the tag value to owned informs the cluster that all resources tagged with the <CLUSTERID> are owned and managed by this cluster.

As noted in OP:

Please change content from any string you choose to required k8 cluster id

Revisiting this.
In the bigger context, I actually think this is ambiguously worded.

When you create an Amazon EC2 Cluster, the ClusterID is automatically configured for the created nodes. Other resources still need to be tagged manually.
Use the following tag:
Key = kubernetes.io/cluster/CLUSTERID Value = owned
CLUSTERID can be any string you like, as long as it is equal across all tags set.

So, if the user created a EC2 cluster, they need to use the automatically configured ClusterID when they tag other resources manually, to make sure that the tags match. Otherwise, they can use any string, as long as they're consistent in using that string to tag every relevant resource. Is that correct?


Fix cycle date + add dkorpel to the list of foundation members

cc @CyberShadow

Deployed :+1:

Thanks!


Roadmap for 8.0.0

Collecting all the items left to complete before an 8.0.0 release can be created.

[x] Switch to new API generator (https://github.com/elastic/enterprise-search-python/pull/100)
[x] Deprecate body, params and transport parameters (https://github.com/elastic/enterprise-search-python/pull/108)
[x] Change parameters to be keyword-only (https://github.com/elastic/enterprise-search-python/pull/100)
[x] enterprise_search.get_stats include parameter serialization (https://github.com/elastic/enterprise-search-python/pull/117)
[x] Add new test cases for async clients (https://github.com/elastic/enterprise-search-python/pull/111)
[x] Check that a correct version of elastic-transport is installed (https://github.com/elastic/enterprise-search-python/pull/120)
[ ] Add proper type hints for responses

There is now a pre-release of 8.1.0 available: https://github.com/elastic/enterprise-search-python/releases/tag/v8.1.0a1


[Event Request] Codeunit 21 Item Jnl.-Check Line - OnEntryTypeIsNotOutput

Hi, would it be possible to get an event OnEntryTypeIsNotOutput in Codeunit 21 Item Jnl.-Check Line, like this

?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


side bar

make a side bar, I will provide the template/skeleton

it is updated to the branch attached to this issue


ZeroDivisionError: float division by zero

getting ZeroDivisionError: float division by zero on function bbox_iou while running python yolo3_one_file_to_detect_them_all.py -w yolo3.weights -i dog.jpg. Iam not sure if i missing anything

it was due to python 3.. modified code to support python 3 and its working now

@chintan5750 can you shed any light on what was giving you issues?

Any update on this?

it was due to python 3.. modified code to support python 3 and its working now

Could you please tell what is the changes that you made ?

Did anyone found out what to change?
@chintan5750 @chriskaschner @AGrosserHH @kt-smitha

did you manage to solve it? i tried with python 3 and 2. but still getting ZeroDivisionError: float division by zero on function bbox_iou while running predict.py and on training after early stopping
@chintan5750 @chriskaschner @AGrosserHH @kt-smitha @andreasmarxer


chore: deletes unused code; dependencies

Going through package.json, checking if a dependency is used. If not delete it. If it's used in legacy code, see if I can delete that. If it's used in a single place, see if I can refactor it away.
Working my way through some of the low hanging fruit. More to come.

As soon as we can delete those 3 one-off editorials, so much code is going away.


python311Packages.type-infer: 0.0.17 -> 0.0.18

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.type-infer is: Automated type inference for Machine Learning pipelines
meta.homepage for python311Packages.type-infer is: https://pypi.org/project/type-infer/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.18 with grep in /nix/store/6gp13vjihs5igwzvk6wan7s6a53swv02-python3.11-type-infer-0.0.18
found 0.0.18 in filename of file in /nix/store/6gp13vjihs5igwzvk6wan7s6a53swv02-python3.11-type-infer-0.0.18

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.dataprep-ml
python310Packages.mindsdb-evaluator
python310Packages.type-infer
python311Packages.dataprep-ml
python311Packages.mindsdb-evaluator
python311Packages.type-infer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6gp13vjihs5igwzvk6wan7s6a53swv02-python3.11-type-infer-0.0.18 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.type-infer https://github.com/r-ryantm/nixpkgs/archive/fe01b8ad9ec4dc72a5b219f8002e66d2fa8632f6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/fe01b8ad9ec4dc72a5b219f8002e66d2fa8632f6#python311Packages.type-infer

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6gp13vjihs5igwzvk6wan7s6a53swv02-python3.11-type-infer-0.0.18
ls -la /nix/store/6gp13vjihs5igwzvk6wan7s6a53swv02-python3.11-type-infer-0.0.18/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages failed to build:

python310Packages.dataprep-ml
python310Packages.mindsdb-evaluator
python311Packages.dataprep-ml
python311Packages.mindsdb-evaluator

2 packages built:

python310Packages.type-infer
python311Packages.type-infer

Maintainer pings
cc @mbalatsko for testing.

@ofborg build python310Packages.dataprep-ml

Already on master.


Some issues about aliyun driver

Rancher versions:
rancher/server or rancher/rancher:
v2.0.1-rc1
Docker version: (docker version,docker info preferred)
17.03.2-ce
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 16.04.3
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Aliyun
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes

The Security Group drop-down box should show the security group name for easy lookup

We should support custom mirroring, otherwise I have not been able to create clusters using rancheros

If the Available Zone is not empty in the template and vpc and vswitch are empty, adding a cluster fails

2018/05/16 05:29:06 [INFO] Generating and uploading node config test1
E0516 05:29:21.837480       1 generic_controller.go:204] NodeController c-q58q5/m-p6q7g [node-controller] failed with : Error setting machine configuration from flags provided: test1 | aliyunecs driver requires both the --aliyunecs-vpc-id and --aliyunecs-vswitch-id for Virtual Private Cloud

Modify aliyun node template, all the drop-down box values become the default value
Private Address Only is set to x.x.x.x when I add a template. Using this template to launch the cluster, I see that the instance's IP is randomly assigned.
I think we should cancel the privateAddressOnly option, because after setting true, the instance cannot access the public network and cannot pull the image.
In Aliyun template, I don't know how to use "Route CIDR". I tested setting Route CIDR to "x.x.x.x/x" and pvc and vswitch are empty. Failed to add a cluster using this template

2018/05/17 01:57:46 [INFO] Generating and uploading node config hailong-test-cidr1
E0517 01:58:01.891051       1 generic_controller.go:204] NodeController c-q6kvt/m-9jw2v [node-controller] failed with : Error setting machine configuration from flags provided: hailong-test-cidr1 | aliyunecs driver requires both the --aliyunecs-vpc-id and --aliyunecs-vswitch-id for Virtual Private Cloud

After adding the aliyun template and setting the Available Zone and VPC, all VSwitches are displayed in the VSwitch drop-down box. This is wrong because each vswitch belongs to only one Available Zone

9. Like the above problem, different Available Zone will have different templates. Not all templates should be displayed in the Instance Type drop-down box
10. "Available Zone" should be placed on the right side of "Region"

I will check for 7 and 3

4. Modify aliyun node template, all the drop-down box values become the default value has not been fixed, I can't close this issue yet

@loganhz can you look at this? ^^ do you have the PR that would fix it?
@kingsd041 It might not have made it into 2.0.1/2 due to timing.

Tested with rancher/rancher:master 06/11
4. Modify aliyun node template, all the drop-down box values become the default value has been fixed, I will close this issue


chore: update dependencies

Signed-off-by: Xiaochao Dong (@damnever) <EMAIL_ADDRESS>

Codecov Report

Merging #6 (62ec3e9) into development (a5cf56d) will decrease coverage by 0.84%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development       #6      +/-   ##
===============================================
- Coverage        88.74%   87.89%   -0.85%
===============================================
  Files                1        1
  Lines              471      471
===============================================
- Hits               418      414       -4
- Misses              36       38       +2
- Partials            17       19       +2

Impacted Files
Coverage Δ

pool.go
87.89% <0.00%> (-0.85%)
:arrow_down:

Continue to review full report at Codecov.

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


issue/7068:Add dedicated set_fact and set_parameter endpoint

Description

add one dedicated endpoint for parameters
add one dedicated endpoint for facts

closes #7068
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Processing #7318.


callback 'onNodeClick' was triggered when clicking the collapse/expand button in Safari

Describe the bug
callback 'onNodeClick' was triggered when clicking the collapse/expand button in Safari
To Reproduce
Steps to reproduce the behavior:

wirte some code in onNodeClick function
Click on expand button
the code was excuted

Expected behavior
I wanted to jump to other page when clicking the node, but  it also jumped to other page when I was just clicking the expand/collapse button in Safari,
Screenshots

Desktop (please complete the following information):

OS:  macos 10.15.7  macos 12.2.1
Browser safari
Version 15.3

I got the same issue in safari as well

For the workaround, it's possible to add a custom onclick listener to the content inside and use it as onNodeClick replacement

For the workaround, it's possible to add a custom onclick listener to the content inside and use it as onNodeClick replacement

yes, i bind the window.clickCb to the nodeContent html string

@zhoufenfens @bumbeishvili can you show an example of what you're talking about?

I think the fix for this is to add event.stopPropagation() to the end of onButtonClick to prevent the button clicks from also triggering the node click handler.
The work-around @bumbeishvili mentioned is to override the onButtonClick handler yourself, and add the event.stopPropagation() at the end. So, after the chart's render() function is called, add this:
chart.onButtonClick = function(event, d) {
      const attrs = this.getChartState();
      if (attrs.setActiveNodeCentered) {
          d.data._centered = true;
          d.data._centeredWithDescendants = true;
      }

      // If childrens are expanded
      if (d.children) {
          //Collapse them
          d._children = d.children;
          d.children = null;

          // Set descendants expanded property to false
          this.setExpansionFlagToChildren(d, false);
      } else {
          // Expand children
          d.children = d._children;
          d._children = null;

          // Set each children as expanded
          if (d.children) {
              d.children.forEach(({ data }) => (data._expanded = true));
          }
      }

      // Redraw Graph
      this.update(d);
      event.stopPropagation();
}

While we're at it, onNodeClick should stop propagation too so that users can select text, or so we can do something else with the mouse events. And to make it so we can add event.stopPropagation() to an overridden onNodeClick, it needs the event as a parameter...Right now we just get the node id.

good idea, it works in safari

Agreed, I will add it in the next release (don't know exactly when though)


Create initial scripts and docs for deploying the TRE solution into an existing Azure subscription

User story
As a cloud administrator
I need basic instructions on how to clone the repo and how to install the core TRE pieces into my subscription
So that I can participate in the test cycle from the beginning.
Acceptance critiera

[x] Initial instructions on how to install the TRE solution, created in the docs section of the repo
[ ] Initial scripts for installing the TRE into an existing azure subscription created and available within the repo
[x] Cloud admin can follow the instructions and use the scripts to repeatedly install the baseline TRE solution

@christoferlof let me know if you think what is merged is sufficient for now.


Performance improvements

Improved performance by:

Avoiding closures where possible
Using HashSets instead of Lists for searches
Using more explicit typing
Avoiding duplicate map lookups

Before these changes the ring api device averaged 18ms, now it is 8ms
Also changed:

Won't log that websocket is connected until a msg is received
Silence a warning for an unsupported new msg type
Don't log that hub is being refreshed after websocket is connected

"Before these changes the ring api device averaged 18ms, now it is 8ms"
That is fantastic!


fix: units no longer making network requests

Description
Fix for https://github.com/shapeshift/web/issues/725
I spent quite some time trying to implement nock globally but it was a real pain. The difficulty came from initializing nock globally while some react-app-rewired things are happening when starting tests.
I settled in implementing nock in only one file, where a test failed because of no network connection. Let me know if you prefer, I can remove it.
Afterwards, I fixed the more subtle network calls we can find by launching tests with --detectOpenHandles, those were fixed basically by mocking shapeshiftoss/market-service.
Notice
Before submitting a pull request, please make sure you have answered the following:

[X] Have you followed the guidelines in our Contributing guide?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Do all new and existing tests pass? Does the linter pass?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
If applicable, please link to the github issue and put closes #XXXX in your comment to auto-close the issue that your PR fixes.
Testing
Please outline all testing steps

reproduced the issue
tried to implement nock globally
Implemented nock locally on unit tests of interests
Investigated which modules needed to be mocked (use of --detectOpenHandles when launching tests)
Found fix and applied it

Screenshots (if applicable)

@cjthompson removed nock


Set long_description in setup.py

Optuna doesn't provide the description at PyPI.
https://pypi.org/project/optuna/
Caution: We can use markdown-style long_description from setuptools v38.6.0 because it started to support PEP566
https://setuptools.readthedocs.io/en/latest/history.html#v38-6-0

Hmm... I don't know why test-python35 is failed.

FYI

Hmm... I don't know why test-python35 is failed.

I don't know the exact reason of the failure too, but in my environment (python-3.5.3), the following diff fixed the problem.
diff --git a/setup.py b/setup.py
index 2e058c6..152c5c1 100644
--- a/setup.py
+++ b/setup.py
@@ -100,6 +100,7 @@ setup(
             'storages/rdb/alembic/versions/*.*'
         ]
     },
+    data_files=[('.', ['./README.md'])],
     install_requires=get_install_requires(),
     tests_require=get_extras_require()['testing'],
     extras_require=get_extras_require(),

Thank you! I'll check later.
BTW, I notice that Optuna is in PyPI trending projects now. It looks a lot of users download optuna today.
https://pypistats.org/packages/optuna

Congratulation!

Great news🎉
Thanks for letting us know!

Hmm... Although I applied your patch, tests-python35 is failed. Basically, It doesn't need to specify README.md at data_files because setuptools include it at following lines:
https://github.com/pypa/setuptools/blob/8aeff6b2c9a64e47ad2a22533d7e65c08cd4103f/setuptools/command/sdist.py#L40-L41
I couldn't reproduce this error in my macOS environment even if I install optuna in python3.5 without specifying data_files argument of setup function:
$ python3.5 -m venv venv35
$ source venv35/bin/activate
$ python setup.py sdist
$ pip install ./dist/optuna-0.12.0.tar.gz
...
Successfully installed optuna-0.12.0

I'll investigate the reason of this error later by using login with ssh feature of circleci.

Although I applied your patch, tests-python35 is failed

Hmm...

Sorry, I couldn't find the reason why README is not found on Python3.5. Provisionally, I added an exception handling. Please close this PR if it needs to build sdist package in Python3.5.

I noticed another problem to render Optuna's logo image. It feels better to close this.

the reason why README is not found on Python3.5.

Maybe the cause is the combination of old setuptools and the CircleCI environment. When I add an update command of setuptools to the install step of CircleCI, I can successfully build the sdist package using Python 3.5.
Please take a look at the following branch:
https://github.com/pfnet/optuna/compare/set-long-description-repro
Changes

Add update command of setuptools
Revert get_long_description

I noticed another problem to render Optuna's logo image at pypi.org because this specifies relative path.

Thank you for pointing it out. I think we can use the absolute path as follows:

Maybe the cause is the combination of old setuptools and the CircleCI environment. When I add an update command of setuptools to the install step of CircleCI, I can successfully build the sdist package using Python 3.5.

Oh, I see! Thank you @toshihikoyanase !
It looks your changes good. I'll cherry-pick your changes and specify absolute path.

@toshihikoyanase I rebased on your branch and updated a link URL of the logo image.

It's OK! I pushed 6300ce71897c28ff23dc6e196499f08224496fe0 👍


Error 1406: Data too long for column 'longurl' at row 1

cloumn longurl of table url need longer length;

refer http://www.cnblogs.com/doit8791/archive/2012/05/28/2522556.html

fix at eb240dbe0910ff4538032cc1a898e3fd32f7f606


[5.2] Update Str::plural() to use intval()

A lot of the time when you use str_plural() you get the $count value from a form request which returns the number as a string and therefore this won't work as expected. This small change should make sure the $count is in integer format.
Many thanks to @barryvdh for suggesting to do this change deeper in the code.

Thank you, but could you please update this to use the (int) casting rather than calling the intval function.

@GrahamCampbell Thank you for your suggestion, I have made the requested changes.

@GrahamCampbell Could you please explain why (int) casting preferable over intval function?

@a-komarev from what I understood, if you're using a base of 10, it's a lot faster and therefore preferred. intval() gives you more options though.


glGetTexLevelParameter not implemented

https://www.opengl.org/wiki/GLAPI/glGetTexLevelParameter
it's present in core since 1.0 till 4.5, so I guess it's kinda ubiquitous. Not a trace of it in GL20/GL30 though. Is there any reason for the lack of it, or it's just a random omission?

It was not added to OpenGL ES until 3.1 as far as I can tell. GL20 and GL30 abstract OpenGL ES 2.0 and 3.0 respectively, so wouldn't have been part of the API.

I checked it, you're obviously right; I wouldn't have expected that a call so old wouldn't be present at least in ES 3.0, but yeah, it's only recently added by 3.1.
Still, it means that currently the only way to get it on supporting platforms is to call backend GL directly, e.g. org.lwjgl.opengl.GL11.glGetTexLevelParameter() or android.opengl.GLES31.glGetTexLevelParameter()? We don't have any bindings for ES 3.1 yet, do we?


chore: move load examples to the fixture [WIP]

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

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

Codecov Report

Merging #10545 into master will decrease coverage by 3.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10545      +/-   ##
==========================================
- Coverage   63.86%   59.87%   -4.00%
==========================================
  Files         767      413     -354
  Lines       36302    13430   -22872
  Branches     3426     3427       +1
==========================================
- Hits        23186     8041   -15145
+ Misses      13003     5196    -7807
- Partials      113      193      +80

Flag
Coverage Δ

#cypress
?

#javascript
59.87% <ø> (-0.02%)
:arrow_down:

#python
?

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

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


Desires to transmit data can be obtained when each callback transmission data

when use function _send_data, body is a function, want known send offset.

Sorry I can't parse this question. Do you wish to know how much of the request body is sent? Can you suggest a way this could work or send a pull request?


Whitespace in .gitattributes linguist-language (again)

Hi, I wonder how to specify a language name with spaces in .gitattributes for linguist-language. This question had been asked in issue #2751, but as the issue seemed to be solved differently, the question was not answered.
More specifically, I want to colorize this file with the language Motorola 68K Assembly. I found that name in languages.yml, but I'm stuck as spaces don't seem to be allowed in .gitattributes values.
BTW, to avoid confusion, it would be nice if GitHub could display the name of the currently detected language used for colorization.

Unfortunately, this is a limitation of the gitattributes(5) format itself; the best you can do is use an alias that doesn't contain whitespace.
Motorola assembly doesn't have any aliases defined yet, but m68k would be a good one. You're more than welcome to submit a pull-request to add it. 👍 Bear in mind the changes won't take place until the next release of Linguist (which will be soon, I believe — late this month).

@Alhadis Thanks for your feedback! I have just created the pull request #4683, as requested.

Aaaaand it's already been approved. 😉 Will get merged once GitHub staff (i.e., @lildude) gives it the final review.
Thanks!

Many thanks, it works fine now to colorize my assembler files on GitHub 😀
https://github.com/Vretrocomputing/atari-devpac2-afficher_image/blob/master/IMAGE.S


Failed to run build-wrapper.sh within MSYS2

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Windows 10
scrcpy version: 1.12 (master branch)
installation method: manual build
device model: none
Android version: none

Describe the bug
I am trying to build scrcpy from source without prebuilt server. Thanks to the docs, it is not painful to set up the environment. But still, I encountered something confusing when executing ninja -Cx.
MSYS2 reported errors:
SOMEBODY@SOMEWHERE MINGW64 /d/workspace/scrcpy_gh
$ ninja -Cx
ninja: Entering directory `x'
[26/45] Generating server/scrcpy-server with a custom command
/bin/bash: D:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh: No such file or directory
[31/45] Compiling C object app/scrcpy.exe.p/src_util_intr.c.obj
FAILED: server/scrcpy-server
"bash" "D:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh" "D:/workspace/scrcpy_gh/server" "server/scrcpy-server" "release"
[44/45] Compiling C object app/scrcpy.exe.p/src_sys_win_process.c.obj
ninja: build stopped: subcommand failed.

I dug it out and found some code in .\x\build.ninja that related to it:
build server/scrcpy-server: CUSTOM_COMMAND  | D$:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh PHONY
 pool = console
 COMMAND = "bash" "D:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh" "D:/workspace/scrcpy_gh/server" "server/scrcpy-server" "release"
 description = Generating$ server/scrcpy-server$ with$ a$ custom$ command

But if i run it manually, there was nothing wrong:
SOMEBODY@SOMEWHERE MINGW64 /d/workspace/scrcpy_gh
$ "bash" "D:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh" "D:/workspace/scrcpy_gh/server" "server/scrcpy-server" "release"
Downloading https://services.gradle.org/distributions/gradle-7.0.2-all.zip
..............10%...............20%...............30%..............40%...............50%...............60%...............70%..............80%...............90%...............100%

Welcome to Gradle 7.0.2!

Here are the highlights of this release:
 - File system watching enabled by default
 - Support for running with and building Java 16 projects
 - Native support for Apple Silicon processors
 - Dependency catalog feature preview

For more details see https://docs.gradle.org/7.0.2/release-notes.html

...

Can anyone give me some help? Why did the scripts fail here?
Update: It seems the path D$:/workspace/scrcpy_gh/server/scripts/build-wrapper.sh is not recognized correctly. But I still don't know why.

scrcpy version: 1.12 (master branch)

1.21?

I am trying to build scrcpy from source without prebuilt server.

Does it work at least with the prebuilt server? (for me it works)
However, I don't have a Windows VM with everything ready to build the server with Android currently.
@Biswa96, could you reproduce?

My apology, I did not understand the issue. msys2 does not provide any Java environment. I only build the portable  version for my use. What setup do I need to reproduce the issue? I am not very familiar with Java.

@Biswa96 Sorry, I notified you because I remembered that you have some experience with MSYS2/Windows, without much context :)
The author of this issue reports that a full build (including the server) from MSYS2 fails.
On Windows, I can build with a prebuilt server without issue, but I currently could not test without a prebuilt server (i.e. building the Android server also from MSYS2).
I asked in case you already had an environment with everything available, but if you don't, I'll take time to install an Android environment in my Windows VM.

After reading some Stackoverflow answers, I can now build scrcpy-server with AndroidStudio. I ran the commands as your documentation suggests. Ninja can successfully executes gradle build job.

@rom1v

Does it work at least with the prebuilt server? (for me it works)

Right now, I build it with a prebuilt server and it works. (But for some reasons, I still need to build without the prebuilt server.)
SOMEBODY@SOMEWHERE MINGW64 /d/workspace/scrcpy_gh
$ meson x --buildtype release --strip -Db_lto=true -Dprebuilt_server="scrcpy-server-v1.21"
The Meson build system
Version: 0.60.2
Source dir: D:/workspace/scrcpy_gh
Build dir: D:/workspace/scrcpy_gh/x
Build type: native build
Project name: scrcpy
Project version: 1.21
C compiler for the host machine: cc (gcc 11.2.0 "cc (Rev5, Built by MSYS2 project) 11.2.0")
C linker for the host machine: cc ld.bfd 2.37
Host machine cpu family: x86_64
Host machine cpu: x86_64
Found pkg-config: C:\tools\msys64\mingw64\bin/pkg-config.EXE (0.29.2)
Run-time dependency libavformat found: YES 58.76.100
Run-time dependency libavcodec found: YES 58.134.100
Run-time dependency libavutil found: YES 56.70.100
Run-time dependency sdl2 found: YES 2.0.18
Library ws2_32 found: YES
Checking for function "strdup" : YES
Checking for function "asprintf" : YES
Checking for function "vasprintf" : YES
Configuring config.h using configuration
Program cp found: YES (C:\tools\msys64\usr\bin/cp.EXE)
Program scripts/run-scrcpy.sh found: YES (bash D:/workspace/scrcpy_gh/scripts/run-scrcpy.sh)
Build targets in project: 3

scrcpy 1.21

  User defined options
    buildtype      : release
    strip          : True
    b_lto          : true
    prebuilt_server: scrcpy-server-v1.21

Found ninja-1.10.2 at C:\tools\msys64\mingw64\bin/ninja.EXE

SOMEBODY@SOMEWHERE MINGW64 /d/workspace/scrcpy_gh
$ ninja -Cx
ninja: Entering directory `x'
[45/45] Linking target app/scrcpy.exe

1.21?

Sorry, I should leave the commit id: 5704ec6967c03510dd7d206e5f59c1e4fd6e10a0 (on master branch)

After reading some Stackoverflow answers, I can now build scrcpy-server with AndroidStudio. I ran the commands as your documentation suggests. Ninja can successfully executes gradle build job.

@Biswa96 Sounds like good news to me! Thanks for your support, and would you like to share the steps or problems encountered?
 And it also takes me some time to find out and purge the .gradle cache files.)

These command may differ in your case. I have installed Android Studio from ZIP file. I just added these environments.
export ANDROID_SDK_ROOT=/d/AndroidSDK
export JAVA_HOME=/d/AndroidStudio/jre
export PATH=/d/AndroidStudio/jre:$PATH

And then run meson and ninja command as mentioned in docs.

@rom1v @Biswa96 thank you two for helping me. After reading ninja docs and do some tests, finally, I find the reason.
When running the following scripts (.\x\build.ninja):
build server/scrcpy-server: CUSTOM_COMMAND  | D$:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh PHONY
 pool = console
 COMMAND = "bash" "D:/workspace/scrcpy_gh/server/./scripts/build-wrapper.sh" "D:/workspace/scrcpy_gh/server" "server/scrcpy-server" "release"
 description = Generating$ server/scrcpy-server$ with$ a$ custom$ command

Ninja strangely starts using the bash from WSL2 instead of MSYS2, so it fails to locate build-wrapper.sh and reports No such file or directory.

I don't have a Windows VM with everything ready to build the server with Android currently.

Would it be better if a CI script is added for msys2 windows build? Though the .exe file will be dynamically linked with ffmpeg and sdl.

In theory, maybe, but I don't have a CI yet (for any platform). There is #1709, but there were still issues (if checkstyle was enabled it did not work for some reason), and I did not work on it since then.

Just realized that I can reproduce this issue in my PC. I generally delete bash, ssh etc. from System32 to workaround any conflict with msys2 and cygwin. Here is the output if system32/bash.exe present:
Please enable the Virtual Machine Platform Windows feature and ensure virtualization is enabled in the BIOS.
For information please visit https://aka.ms/wsl2-install
FAILED: server/scrcpy-server
"bash" "D:/scrcpy/server/./scripts/build-wrapper.sh" "D:/scrcpy/server" "server/scrcpy-server" "release"
ninja: build stopped: subcommand failed.

GitHub Actions CI also shows same error with msys2 environment. I assume that the issue comes from find_program() in server/meson.build file. Is it possible to invoke gradle from meson instead of going through a shell script?


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 12:42:29 on 2023.07.14
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 1, errors out: 0
Original problem ID: 32648
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=32648

Problem has been resolved in 3m 0s at 12:45:29 on 2023.07.14
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 32648


[e2e]: go tag main config-ldflags-assets-tag slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.69
Run: https://github.com/slsa-framework/example-package/actions/runs/2793934294
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.69
Date: Thu Aug  4 03:28:07 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.70
Run: https://github.com/slsa-framework/example-package/actions/runs/2801117429
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.70
Date: Fri Aug  5 03:47:12 UTC 2022
Tests are passing now. Closing this issue.


Corrected bug on getting deafult relation style.

Description:
A new relation without a default style was used in the test of ontograph, which showed that there was an error in this part.
A new test has been added that will eventually cover this, by accepting only that one warning, i.e. when runnig pytest -Werror this test will pass.
Type of change:

[x] Bug fix.
[ ] New feature.
[ ] Documentation update.

Checklist:

This checklist can be used as a help for the reviewer.

[ ] Is the code easy to read and understand?
[ ] Are comments for humans to read, not computers to disregard?
[ ] Does a new feature has an accompanying new test (in the CI or unit testing schemes)?
[ ] Has the documentation been updated as necessary?
[ ] Does this close the issue?
[ ] Is the change limited to the issue?
[ ] Are errors handled for all outcomes?
[ ] Does the new feature provide new restrictions on dependencies, and if so is this documented?

Comments:

Codecov Report

Merging #561 (d613a19) into master (5d89725) will increase coverage by 0.00%.
The diff coverage is 80.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #561   +/-   ##
=======================================
  Coverage   65.85%   65.86%
=======================================
  Files          16       16
  Lines        3099     3105    +6
=======================================
+ Hits         2041     2045    +4
- Misses       1058     1060    +2

Impacted Files
Coverage Δ

ontopy/utils.py
62.33% <75.00%> (+0.08%)
:arrow_up:

ontopy/__init__.py
80.00% <100.00%> (ø)

ontopy/graph.py
61.63% <100.00%> (ø)

ontopy/ontodoc.py
67.51% <100.00%> (ø)

ontopy/ontology.py
70.27% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

changed the name.


Matrix is not positive definite

I'm using the Covariance object on my own dataset. Here is the data.
(5245, 258)
Traceback (most recent call last):
  File "train_metric.py", line 61, in <module>
    X_trained = ml.transform(X)
  File "/home/andrey/anaconda2/lib/python2.7/site-packages/metric_learn/base_metric.py", line 46, in transform
    L = self.transformer()
  File "/home/andrey/anaconda2/lib/python2.7/site-packages/metric_learn/base_metric.py", line 29, in transformer
    return inv(cholesky(self.metric()))
  File "/home/andrey/anaconda2/lib/python2.7/site-packages/numpy/linalg/linalg.py", line 612, in cholesky
    r = gufunc(a, signature=signature, extobj=extobj)
  File "/home/andrey/anaconda2/lib/python2.7/site-packages/numpy/linalg/linalg.py", line 93, in _raise_linalgerror_nonposdef
    raise LinAlgError("Matrix is not positive definite")
numpy.linalg.linalg.LinAlgError: Matrix is not positive definite

Will it be better if the code use pinv instead of inv?

The issue here is that your data does not cover all 258 dimensions, which results in a rank-deficient covariance matrix. If you preprocess your data with PCA and take the principal components explaining 99% of the variance, you only need 5 dimensions:
import numpy as np
from sklearn.decomposition import PCA

x = np.loadtxt('data.csv', delimiter=',')  # shape = (5245, 258)
px = PCA(n_components=0.99).fit_transform(x)  # shape = (5245, 5)

Now the covariance of px is full-rank (5) and everything should work as intended.
I think it's outside the scope of the Covariance method to do this sort of preprocessing, but it might be useful to have an example like this in the docs somewhere.

Thanks! So I should always apply PCA before using any metric learning algorithms implemented here?

Most of the algorithms here either handle this case natively or give an option to apply PCA before learning (see https://github.com/all-umass/metric-learn/blob/master/metric_learn/rca.py#L64), so it varies case-by-case.
I would say that it's unlikely to hurt, though, especially with your particular dataset.


Is it possible to use it in the form builder, not the admin panel ?

For some reason i get this when i use it in filament form builder:
Alpine Expression Error: phoneInputFormComponent is not defined
Expression: "phoneInputFormComponent({
getInputTelOptionsUsing: (intlTelInput) => ({"allowDropdown":true,"autoPlaceholder":"aggressive","customContainer":"","customPlaceholder":null,"dropdownContainer":null,"excludeCountries":[],"formatOnDisplay":true,"geoIpLookup":"ipinfo","initialCountry":"auto","localizedCountries":[],"nationalMode":true,"onlyCountries":[],"placeholderNumberType":"MOBILE","preferredCountries":["us","gb"],"separateDialCode":false,"displayNumberFormat":"NATIONAL","inputNumberFormat":"E164","focusNumberFormat":false,"utilsScript":"/filament/assets/intl-tel-input-utils.js"}),
state: $wire.entangle('phone').defer,
inputID: 'phone',
})"
<input type="tel" x-data="phoneInputFormComponent({
getInputTelOptionsUsing: (intlTelInput) => ({"allowDropdown":true,"autoPlaceholder":"aggressive","customContainer":"","customPlaceholder":null,"dropdownContainer":null,"excludeCountries":[],"formatOnDisplay":true,"geoIpLookup":"ipinfo","initialCountry":"auto","localizedCountries":[],"nationalMode":true,"onlyCountries":[],"placeholderNumberType":"MOBILE","preferredCountries":["us","gb"],"separateDialCode":false,"displayNumberFormat":"NATIONAL","inputNumberFormat":"E164","focusNumberFormat":false,"utilsScript":"/filament/assets/intl-tel-input-utils.js"}),
state: $wire.entangle('phone').defer,
inputID: 'phone',
})" dusk="filament.forms.phone" id="phone" class="block w-full transition duration-75 rounded-lg shadow-sm focus:border-primary-600 focus:ring-1 focus:ring-inset focus:ring-primary-600 disabled:opacity-70 border-gray-300">
handleError @ module.esm.js:416
mo

Please check this discussion

`As you see I don't use the form feature of Filament yet.
I'm thinking new way to usage with form feature it in the next version of this package. Maybe just we can add assets on the top of application's layout. Just like below
components/layouts/app.blade.php`
Is that mean, that we can't use it in Filament's form builder yet and we need to wait for v3 ? Thanks


Return errors from the plugin functions to host

Issue #, if available:
Fixes #15
Description of changes:
This fix sends error message occurrence sign within high bit of the offset/len field.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

left some comments, but looking good!


OVSSwitch: use the OVSDB protocol directly for batch operations

net: group batch operations based on switch type
ovsdb.py: new file

I send this pull request as an alternative to #286.  Similarly to that, this pull request attempts to improve mininet's performance during startup and shutdown.
I measured 'mn --test=none --topo=linear,k=K' with different K values:
   k  orig  ovsbatch  ovsdb_batch
  25    51    37       34
  50   113    87       74
 100   273   189      171
 150   503   318      285
 200   787   458      395
 300  2261   883      703

measuring just the 'starting switches' phase:
   k  orig  ovsbatch  ovsdb_batch
  25    16     7        4
  50    35    16        9
 100   101    35       17
 150   212    66       25
 200   353   102       30
 300  1352   258       45

measuring just the 'stopping switches' phase:
   k  orig  ovsbatch  ovsdb_batch
  25     9     5        5
  50    21    17       12
 100    54    36       30
 150   104    63       50
 200   167    71       67
 300   439   109      148

This this version (ovsdb_batch) is slower than ovsbatch (#286) because it calls deleteIntfs in the end (which might be unnecessary)
So performancewise this version is quite good, but currently it has some drawbacks:

The license of the ovsdb library is unknown.  (But I can extract the parts that the pull request really needs.  These parts were written mostly by me.)
It probably doesn't follow the coding standards of mininet.
It sends one large ovsdb transaction.  Probably, it is better to configure the network in smaller chunks (say, 100 nodes per transaction.)

However, I think, at least the implementation of starting/stoping switches in a batch is more robust than that of #286.
My original goal with this work was to familiarize with the ovsdb protocol, but, as usual, if there's an interest in this pull request, I'd be happy to incorporate feedback.

@nemethf thanks for looking into this - it's definitely interesting work and it may be useful in the future.
For now I think it doesn't make sense to burden Mininet with the ovsdb protocol and to just use ovs-vsctl. But it is an interesting approach and I think the work might be useful for some applications.


[DO NOT MERGE] : CIEDEV-2144: Expand Version Manager service to enable publishing of releases

When the version-manager service is called, for official releases, release service publish is invoked and DB entry is created.
Unit test for official release (Ex: v1.0.0):

Unit test for development release (Ex0.0.0-dev):

https://github.com/terrariumcloud/terrarium/pull/53 introduced a change in protocol definitions, it's a matter of renaming release PublisherServer to ReleasePublisherServer to avoid a clash with PublisherServer that already exists for modules. This error is most likely because the ReleasePublisherServer is not registered yet with terrarium grpc server (as seen here). Once this line is uncommented, we'll need to refactor a couple of things in backend to support this change, it's just a matter of renaming things, not changing the functionality. We'll see whether this can be done in 2148 or there will be a separate PR to address this change.


Fix fatal error caused by switch cases without any statements (#473)

The issue can be reproduced on main by formatting the following snippet:
switch x {
case y:
default:
  print("hi")
}

The code isn't valid Swift of course, however it shouldn't cause a fatal error, and in my opinion it shouldn't prevent formatting of the code (because intent is clear). However, I'm not sure of the official position of this project on how invalid code should be handled.
Note that adding a comment in the y case doesn't effect whether a fatal error occurs.
I have added a test that covers formatting of empty cases and after my relatively minimal (and commented) changes it no longer crashes. The issue occurred because in the situation where a case is empty, the opening break and open token are added after the same syntax token as the closing break and close token. These two sets of tokens are added one after another, and because of the implementation of after, the two sets end up getting swapped when constructing the token stream. Thus, the closing break and close token end up preceding the opening break and open token causing the fatal error.

@allevato Would you be able to review this PR and give feedback if necessary? It’s a relatively small and safe PR. I’m not sure if you’re the right person to ping for a review, but you seem to be the most active maintainer (please direct me to the correct person if I’m wrong).
If there are any other steps I need to follow before getting the PR reviewed, please let me know.

Thanks, out of interest what do you mean by the companion pr in swift-syntax?

Thanks, out of interest what do you mean by the companion pr in swift-syntax?

We don't have swift-ci directly set up on swift-format yet, but I can kick it off indirectly by linking to this PR from this dummy PR that uses the trigger.
I don't want to dirty up swift-syntax with multiple dummy PRs though, so I need to wait for each one to finish and report the results before starting the next.


[release-1.4] Add analyzer rules for VirtualService destination ports

This is an automated cherry-pick of #18674

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test lint_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test lint_istio_release-1.4

integ-security-k8s-tests_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test integ-security-k8s-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test integ-security-k8s-tests_istio_release-1.4

lint_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test lint_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test integ-security-k8s-tests_istio_release-1.4

lint_istio_release-1.4
2a7752390d16f038a03ee2dc4fd3a9e903f079e7
link
/test lint_istio_release-1.4

integ-new-install-k8s-tests_istio_release-1.4
7a508941f8956e0ccf7e2db94f4d3408c49767ca
link
/test integ-new-install-k8s-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Strange behavior of select options

I recognized some strange behavior  in a select input, using providing a pre-selected value.
If I provide the key as Integer, angular adds a new empty option. If the key is passed as string, everything works as expected.
This behavior appears in Angular 1.3.2 and above, but it works in Angular  1.3.1
Example:
https://jsfiddle.net/pu27bL7v/3/

This is working as expected. When you iterate over an object's keys, those keys are strings. This is not angular specific but just how JavaScript object keys are stored. In Angular 1.3.2, I believe that we switched over to strict equality for comparing the option value to the model.
You have a couple of options to deal with this:

Use strings in your model. See https://jsfiddle.net/g56dokf0/
Convert the number to a string and back again using a $formatter/$parser pair via a directive. See https://jsfiddle.net/kd1shg1w/

Hope that helps.

ok, thnx.  but it seem's like it's not done in all places, because if I use an array of objects  it works again.
see https://jsfiddle.net/3ywvgk6y/

In this second fiddle, you are not iterating over an object's properties; you are iterating over an array of objects. The point is that a property's name is always a string. In your case the "value" of the id property is a number.


VendorTxCode refund param being unset on refund()->send()

Heyo,
Im having an issues where my VendorTxCode is being emptyed/removed on the send() from a refund.
Here the code im using
    /**
     * _processSagePayRefund
     *
     * @param Request $request
     * @param Order   $order
     * @param User | false | null    $user
     *
     * @return RedirectResponse
     */
    private function _processSagePayRefund(Request $request, Order &$order, User | false | null $user)
    // : RedirectResponse
    {
        $sagepay_vendor = config('services.sagepay.vendor');
        $sagepay_sandbox = config('services.sagepay.sandbox');

        // Initialize the Sagepay payment gateway instants.
        $gateway = Omnipay::create('SagePay\Server');
        $gateway->setVendor($sagepay_vendor);
        $gateway->setTestMode($sagepay_sandbox);

        $vpsTaxId = $order->VPSTxId;
        $reference = json_decode($vpsTaxId, true);
        // Get the refund amount from order.
        $refundAmount = $order->total;
        // In case user has specified an amount.
        if ($request->has('amount') && $refundAmount >= $request->input('amount')) {
            // Set that amount.
            $refundAmount = $request->input('amount');
        }
        $vendorTxCode = "REF-" . $refundAmount . "-" . $order->ref_id;

        $params = [
            // Transaction type. Should be in Capital letters.
            'TxType' => "REFUND",
            'VendorTxCode' => $vendorTxCode,
            'amount' => $refundAmount,
            'currency' => $order->currency,
            'description' => substr('Refund for Order #' . $reference['VPSTxId'], 0, 90),
            'VPSTxId' => $reference,
            'transactionReference' => json_encode($reference),
            'TxAuthNo' => $reference['TxAuthNo'] ?? null,
        ];
        $params['transactionId'] = $vendorTxCode;
        $params['vendor'] = config('services.sagepay.vendor');
        $params['VPSTxId'] = json_encode($reference);

        try {
            $response = $gateway->refund($params)->send();

        } catch (\Exception $e) {
            Alert::warning('Error', "Refund failed: " . $e->getMessage());
            \Log::info($order->id . " _processSagePayRefund - Failed to issue a refund: " . $e->getMessage());
            return back()->with('error', "Refund failed: " . $e->getMessage());
        }
        // Check if refund was successful.
        if ($response->isSuccessful()) {
            // Get the ID of the refund transaction reference.
            $refundTransactionId = $response->getTransactionReference();

            \Log::info($order->id . " _processSagePayRefund - isSuccessful -" . $refundTransactionId);
            // Update status on order record.
            $order->update(['order_status' => Order::REFUNDED]);
            // Save the refund transaction reference.
            $order->transactions()->create([
                'transaction_status' => OrderTransaction::REFUNDED,
                'transaction_number' => $refundTransactionId,
                'payment_result' => "success",
            ]);
            // Return success to user.
            return back()->with('success', "Refund successful. Transaction ID: " . $refundTransactionId);
        } else {
            // Refund failed return back to user.
            \Log::info($order->id . " _processSagePayRefund - isSuccessful - failed " . $response->getMessage());
            return back()->with('error', "Refund failed: " . $response->getMessage());
        }
    }

Here a screen shot of the params array and the $gateway->refund($params)->send() in a dd()

Im completely lost on what to do here. I get a 3009 : Missing VendorTxCode from the sagepay server response

@JJonesFK Does it work if you change this line
to $data['VendorTxCode'] = $this->getParameter('VendorTxCode'); ?

Hi jamieburchell,
I dont want to change the code in side the vendors folder as it will get over written at somepoint.
Cleaned up the code to make it more readable but still have the same issue
    /**
     * _processSagePayRefund
     *
     * @param Request $request
     * @param Order   $order
     * @param User | false | null    $user
     *
     * @return RedirectResponse
     */
    private function _processSagePayRefund(Request $request, Order &$order, User | false | null $user)
    // : RedirectResponse
    {
        $sagepay_vendor = config('services.sagepay.vendor');
        $sagepay_sandbox = config('services.sagepay.sandbox');

        // Initialize the Sagepay payment gateway instants.
        $gateway = Omnipay::create('SagePay\Server');
        $gateway->setVendor($sagepay_vendor);

        // Just two checks for none testing.
        if (!$sagepay_sandbox) {
            // If not in sandbox mode.
            if (env('APP_ENV', "local") == "production") {
                // In production disallow none sandboxed mode.
                $gateway->setTestMode(false);
            } else {
                // In local allow none sandboxed mode.
                $gateway->setTestMode(true);
            }
        } else {
            // Always allow none sandboxed mode.
            $gateway->setTestMode(true);
        }
        // $order->VPSRxId == {"SecurityKey":"#########","VPSTxId":"{#######-####-####-####-########}","VendorTxCode":"1500000064","TxAuthNo":"1#4#1#3#7#"}
        $reference = json_decode($order->VPSTxId, true);
        $refundAmount = $order->total;

        if ($request->has('amount') && $refundAmount >= $request->input('amount')) {
            $refundAmount = $request->input('amount');
        }

        $vendorTxCode = "REF-" . $refundAmount . "-" . $order->ref_id;
        $params = [
            'TxType' => "REFUND",
            'vendorTxCode' => $vendorTxCode,
            'amount' => $refundAmount,
            'currency' => $order->currency,
            'description' => substr('Refund for Order #' . $reference['VPSTxId'], 0, 90),
            'transactionReference' => $order->VPSTxId,
        ];
        $params['vendor'] = config('services.sagepay.vendor');
        // $params['vendorTxCode'] = $gateway->getParameter('VendorTxCode'); // This returned Null

        dd([$params, $gateway->refund($params)->send()]);
    }

Heres the dd from the end of the code

I dont want to change the code in side the vendors folder as it will get over written at somepoint.

It was just to test temporarily.

@JJonesFK
Looking at this again, I can see that the value for VendorTxCode (as presented to Opayo) is derived from the transactionId parameter.
TxType is not required, it's set by the refund() method. vendorTxCode is not required.
Try this as your params array:
$params = [
   'transactionReference' => $order->VPSTxId,
   'amount' => $refundAmount,
   'currency' => $order->currency,
   'description' => substr('Refund for Order #' . $reference['VPSTxId'], 0, 90),
   'transactionId' => $vendorTxCode
   ];