Migrate bower + npm to only npm for dependency management

No reason to have both of these dependency managers around, especially given #245 we stand to gain much from using npm.
This task is broken up into two parts:

[ ] Transfer bower dependencies to package.json
[ ] Ensure necessary dependencies are injected into dependency partials (gulp task)

cc @chalettu @dclarizio @martinpovolny @himdel @AparnaKarve

So it seems there are a few ways to skin this 😸 .  One is to use yarn, (yarn install instead of npm install and bower install) but there are a few issues presently.  Another is to completely rip out bower and use a combination of commonjs require and webpack/bowerserify to include the bundled assets in index.esj (leaning webpack)
If/when we move to angular 2, some key dependencies are published on npm but not bower (mainly https://ui-router.github.io/ng2/) .
I am mildly concerned  that the trend of dependencies not being published on bower will cause trouble down the road. 🙈

I agree bower is dying. What is your recommendation for how to proceed?

We can sit down and look at all our current dependencies, see if they are in npm.   We can then look into integrating webpack and the combo above.

They are, already checked 🤘🎷🐯
On Oct 15, 2016 12:42 PM, "Chris Hale"<EMAIL_ADDRESS>wrote:

We can sit down and look at all our current dependencies, see if they are
in npm. We can then look into integrating webpack and the combo above.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManageIQ/manageiq-ui-self_service/issues/251#issuecomment-253995437,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVSbKwD8jq0moSVAJt6qDT3N4GyqMWGks5q0QJfgaJpZM4KXZJE
.

I wonder if the priority is to get it done here or in the operations UI or both? @himdel?
I started putting together list of tasks related to splitting the various UIs here: https://www.pivotaltracker.com/n/projects/1619601 and the npm migration is among those.

Agreed, we should migrate both, rather soon ideally.  (In fact, I've already started looking into migrating manageiq, and there, we're missing a few dependencies.. angular-patternfly-sass, jquery-1.8, jquery.observe_field, kubernetes-topology-graph, manageiq-ui-components, patternfly-timeline, spice-html5-bower, xml_display - but I can fix that a bunch of those on Monday,)
As for yarn, on one hand it's very new, on the other, it does look good, is faster both when installing from scratch and when doing nothing, and since it uses the same node_modules and package.json and so can effectively be replaced by npm any time, I'd say what the hell, too tempting not to try :). I wonder if we'll want to .gitignore yarn.lock though, to get the same semantics as before.. (and I'm not yet clear when yarn updates that file - if it's only on explicit add or upgrade, maybe keeping the lock wouldn't be so bad..).
(Using yarn with bower.json does not work for the ops ui at all (it sorts it (whcih would break gulp inject BTW), reads .bowerrc, ..and deletes everything under vendor/assets/bower_components :)).)
Agreed with webpack, even angular2-cli is switching from system,js to webpack, so no question there.. (Investigating webpack-rails for the ops ui)

@himdel yeah, going to get @AllenBW and @chalettu to get it done this week (both the npm move and yarn)

🤘💪👦🍍🐯
On Oct 16, 2016 10:45 AM, "Chris Kacerguis"<EMAIL_ADDRESS>wrote:

@himdel https://github.com/himdel yeah, going to get @AllenBW
https://github.com/AllenBW and @chalettu https://github.com/chalettu
to get it done this week (both the npm move and yarn)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManageIQ/manageiq-ui-self_service/issues/251#issuecomment-254054487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVSbFVoCQ_tq86M7wAb96dXvZnnMSeuks5q0kaggaJpZM4KXZJE
.


Added graphql run method for query objects

This would close #232 if approved.
Usage:
query = nb.extras.graphql_queries.get("example")
query.run(
    variables={"foo": "bar"})
)

Let's add something to the docs as well, please.


feat(theme-default): improve navbar and sidebar options

This PR improves types of navbar and sidebar by making them stricter and more accurate.
E.g.: When users are nesting navbar to 3 levels, type issue is thrown.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.545%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1010

Relevant Lines:
1600

💛  - Coveralls


ITK 5.0.0 build fails with -DBUILD_TESTING=ON

With the following CMake command
cmake -C /var/tmp/portage/sci-libs/itk-5.0.0/work/itk-5.0.0_build/gentoo_common_config.cmake -G Unix Makefiles -DCMAKE_INSTALL_PREFIX=/usr -DBUILD_SHARED_LIBS=ON -DITK_USE_SYSTEM_DCMTK=ON -DGDCM_USE_SYSTEM_OPENJPEG=ON -DITK_USE_SYSTEM_DOUBLECONVERSION=ON -DITK_USE_SYSTEM_CASTXML=ON -DITK_USE_SYSTEM_HDF5=ON -DITK_USE_SYSTEM_JPEG=ON -DITK_USE_SYSTEM_PNG=ON -DITK_USE_SYSTEM_SWIG=ON -DITK_USE_SYSTEM_TIFF=ON -DITK_USE_SYSTEM_ZLIB=ON -DITK_USE_KWSTYLE=OFF -DITK_BUILD_DEFAULT_MODULES=ON -DITK_COMPUTER_MEMORY_SIZE=1 -DWRAP_ITK_JAVA=OFF -DWRAP_ITK_TCL=OFF -Ddouble-conversion_INCLUDE_DIRS=/usr/include/double-conversion -DExternalData_OBJECT_STORES=/var/tmp/portage/sci-libs/itk-5.0.0/work/InsightToolkit-/.ExternalData -DBUILD_TESTING=ON -DBUILD_EXAMPLES=OFF -DITK_USE_REVIEW=OFF -DITK_INSTALL_LIBRARY_DIR=lib64 -DITK_WRAP_PYTHON=OFF -DCMAKE_BUILD_TYPE=Gentoo -DCMAKE_TOOLCHAIN_FILE=/var/tmp/portage/sci-libs/itk-5.0.0/work/itk-5.0.0_build/gentoo_toolchain.cmake  /var/tmp/portage/sci-libs/itk-5.0.0/work/InsightToolkit-5.0.0

I get the following build.log failure.
With:
cmake -C /var/tmp/portage/sci-libs/itk-5.0.0/work/itk-5.0.0_build/gentoo_common_config.cmake -G Unix Makefiles -DCMAKE_INSTALL_PREFIX=/usr -DBUILD_SHARED_LIBS=ON -DITK_USE_SYSTEM_DCMTK=ON -DGDCM_USE_SYSTEM_OPENJPEG=ON -DITK_USE_SYSTEM_DOUBLECONVERSION=ON -DITK_USE_SYSTEM_CASTXML=ON -DITK_USE_SYSTEM_HDF5=ON -DITK_USE_SYSTEM_JPEG=ON -DITK_USE_SYSTEM_PNG=ON -DITK_USE_SYSTEM_SWIG=ON -DITK_USE_SYSTEM_TIFF=ON -DITK_USE_SYSTEM_ZLIB=ON -DITK_USE_KWSTYLE=OFF -DITK_BUILD_DEFAULT_MODULES=ON -DITK_COMPUTER_MEMORY_SIZE=1 -DWRAP_ITK_JAVA=OFF -DWRAP_ITK_TCL=OFF -Ddouble-conversion_INCLUDE_DIRS=/usr/include/double-conversion -DExternalData_OBJECT_STORES=/var/tmp/portage/sci-libs/itk-5.0.0/work/InsightToolkit-/.ExternalData -DBUILD_TESTING=OFF -DBUILD_EXAMPLES=OFF -DITK_USE_REVIEW=OFF -DITK_INSTALL_LIBRARY_DIR=lib64 -DITK_WRAP_PYTHON=OFF -DCMAKE_BUILD_TYPE=Gentoo -DCMAKE_TOOLCHAIN_FILE=/var/tmp/portage/sci-libs/itk-5.0.0/work/itk-5.0.0_build/gentoo_toolchain.cmake  /var/tmp/portage/sci-libs/itk-5.0.0/work/InsightToolkit-5.0.0

The build works, as seen in this build.log.
What can I do to allow ITK 5.0.0 to successfully build with testing support enabled?

If you want to do an offline build with testing, get ITKTestingData and unzip it into /var/tmp/portage/sci-libs/itk-5.0.0/work/InsightToolkit-/.ExternalData, or wherever you configure ExternalData_OBJECT_STORES to point to.

@TheChymera thanks for the continued work on packaging :+1:
To just download the data required for a given release, download the InsightData tarball and unpack it in the same directory that the sources were unpacked. For example,
https://github.com/InsightSoftwareConsortium/ITK/releases/download/v5.0.1/InsightData-5.0.1.tar.gz

@thewtex are you sure for 5.0.0 this shouldn't be https://github.com/InsightSoftwareConsortium/ITK/releases/download/v5.0.0/InsightData-5.0.0.tar.gz ?

Ok, @dzenanz , thanks for the pointer. I tried it again, and I get this build.log (CMake command at the beginning).
It still fails, thought the data seems toe be there:
darkhost /var/tmp/portage/sci-libs # tree itk-5.0.0/work/InsightToolkit-5.0.0/.ExternalData/
itk-5.0.0/work/InsightToolkit-5.0.0/.ExternalData/
|-- MD5
|   |-- 000361a85ff53cec8df5e6f00fcb3454
|   |-- 0005313c352ea289a61da9f412016a9d
|   |-- 0010e97b9d899be916a4e82cb6267c71
|   |-- 00364cfde89513a86de8228b0bd2e4ea
|   |-- 00507edaa64607d62de40375d48fe0f9
|   |-- 007603baa7aed28d8040164d93502443
|   |-- 00835c337749f68b63fb7bbd7c009e94
|   |-- 00bd97f6008825bab8e117837777e9fe
|   |-- 00bf1523c6751b4c2f0db7ddecf25d99
|   |-- 00d9bbe450140a8b30f02950dd00dea2
...

you get the idea

Those .tar.gz files only contain files index by MD5 hashes, and the error log contains complaints about missing sha512 hash. Corresponding .md5 file is just next to it.
Can you try adding to CMake configure line also -DITK_FORBID_DOWNLOADS=ON? Maybe that will force it to look for MD5 index before trying to download it.

@thewtex are you sure for 5.0.0 this shouldn't be https://github.com/InsightSoftwareConsortium/ITK/releases/download/v5.0.0/InsightData-5.0.0.tar.gz ?

I would recommend packaging 5.0.1 -- if I recall correctly, the 5.0.0 InsightData tarball was missing a few entries, which would explain the failure.

The log looks fine to me. ITK_COMPUTER_MEMORY_SIZE only controls whether some tests requiring large amount of memory (4-5GB) are run.

Thank you all for your help! :)


Login and Registration Form

Related Issuse

Info about Issue or bug

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

Original
Updated

original screenshot
updated screenshot

@nikhilkotiya Awesome work!! But few bugs-

Logout button redirecting to the wrong URL.
Display First Name and Last Name before logout button when user is logged in. (Example: Welcome, Akshat Khanna)
After Signup redirect to log in page.
When password is incorrect show login page with error.

This is the best contribution till now on this project. Great work buddy🙌🙌

This is the best contribution till now on this project. Great work buddy🙌🙌

Thank you sir

Just do the required changes and I'll merge the PR

Who created the Frontend for it?

@nikhilkotiya Awesome work!! But few bugs-

Logout button redirecting to the wrong URL.
Display First Name and Last Name before logout button when user is logged in. (Example: Welcome, Akshat Khanna)
After Signup redirect to log in page.
When password is incorrect show login page with error.

Sir I can't find bug to displayay First Name and Last Name .
Other bug removed

Who created the Frontend for it?

@sakshi

@nikhilkotiya Awesome work!! But few bugs-

Logout button redirecting to the wrong URL.
Display First Name and Last Name before logout button when user is logged in. (Example: Welcome, Akshat Khanna)
After Signup redirect to log in page.
When password is incorrect show login page with error.

Sir bug 2 is in the base.html file but I did not commit this file. what did I do?

Who created the Frontend for it?

@sakshi

Where is the PR for Front end

@nikhilkotiya Awesome work!! But few bugs-

Logout button redirecting to the wrong URL.
Display First Name and Last Name before logout button when user is logged in. (Example: Welcome, Akshat Khanna)
After Signup redirect to log in page.
When password is incorrect show login page with error.

Sir bug 2 is in the base.html file but I did not commit this file. what did I do?
Error is in line 64 ( user to user.first_name)

Can you can do that in base.html?

@khannakshat7  sir can you please merge this branch because I can't take the latest code from master. There is an error when I do git pull origin master(maybe because I forked your project). I make a different Pull of base.html file bug.

Who created the Frontend for it?

@sakshi

Where is the PR for Front end

She is not answering

@nikhilkotiya Awesome contribution dude🙌

Don't forget to star⭐ the repo and follow me on GitHub

@nikhilkotiya Awesome contribution dude🙌

Sry sir due to lack of knowledge in git my work slow

@nikhilkotiya That's how to learn. No issues keep going!!

Sir my master branch is now up to date and I just commit base.html file but why all files comes when i pull


Add a link to created GitBook

Changing "Help" to "Guides" button in the footer and adding the external link that  @l-br1 will provide to open in a new tab.

This has been deployed on BETA env. (https://sancho.constitution.gov.tools/), based on the link Intersect team provided. Once the Guides go public, we will change the link.
cc @bosko-m


[v1.1 review] Should MQTT transport sender broker_topic support 'auto'?

I don't feel strongly either way, I just noted that an MQTT sender's "broker_topic" and "connection_status_broker_topic" don't support 'auto', whereas the other MQTT transport params do support 'auto', and so does e.g. WebSocket senders' "connection_uri" - which broadly corresponds to the MQTT destination/topic properties combined.
(The same doesn't apply to receivers, and that'd be consistent with WebSocket receivers' "connection_uri".)

Closing after discussion.


Don't delete funding account if it has changes to public keys

Basically implicit account is only deleted in case when it would be re-created intact on repeated transfer.

Should address https://github.com/near/near-wallet/issues/1428 for cases when user is already using given implicit account id.

@kcole16 @Patrick1904 please test creating multiple accounts from same Ledger. If at any point you import implicit account directly it shouldn't get deleted on next funding.


cleanup files after validation

cleanup files after validation, those are not needed after the install
Fixes: https://github.com/sigstore/cosign-installer/issues/21

this breaks my workflow: I have the pub key commited in the root of my repo, and using this action deletes it, which causes a dirty working tree...

got it @caarlos0 i will update the action to create the directory before to not mess with the root directory

@cpanato , why not download into the tmp folder of the worker?
On the otherhand, i think cosign should have http on verify-blob too? Similar to verify-docker
https://github.com/sigstore/cosign/blob/9906181a17ee51c9ead65379f15e28d53fdac013/cmd/cosign/cli/verify_dockerfile.go#L68
What do you think?

it have @dirien but we need to download the signature as well, but I will check

ah okay, i did not saw it.


Add support for mainnet syncing to Trinity

What was wrong?
No support for syncing with the mainnet.
How was it fixed?
Added ability to sync with mainnet via trinity cli.  Also modified some internals to use network_id as the primary identifier for choosing what network you want to use.
Cute Animal Picture

Shall we merge this?

Notes for self

Up next:

some form of tests that provide some guarantees that running trinity works
evaluate what RPC endpoints work, which ones don't, which ones won't and ensure they behave in sane ways (raising proper exceptions, returning good error messages, etc).
evaluate the CLI options compared to geth/parity and see which things are clearly missing.
support for custom genesis parameters (probably integral for testing)
rework console output to output less information and more useful information.


Use MFEM's METIS mirror instead of glaros.dtc.umn.edu

Resolves #3188.
See also the discussion at the end of https://github.com/mfem/mfem/issues/698.

@pazner, do you think we should also replace the  "homepage" field in config/vcpkg/ports/metis-mfem/vcpkg.json?

Tzanio: this is a copy of their GitHub repo?

Tzanio: this is a copy of their GitHub repo?

Nope. It is a copy of the release tarballs that used to be on their website.

Tzanio: this is a copy of their GitHub repo?

Nope. It is a copy of the release tarballs that used to be on their website.

OK. Their GitHub version has some issues since February. I made it work, but it is not really satisfactory.

I don't see any approve buttons here.

Hence this change 😁

I don't see any approve buttons here.

Click on "Review Changes" here: https://github.com/mfem/mfem/pull/3196/files

@pazner, do you think we should also replace the "homepage" field in config/vcpkg/ports/metis-mfem/vcpkg.json?

We could change it to their GitHub repo. I imagine there will also be a working homepage at some point in the future

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Merged in next for testing...

http://glaros.dtc.umn.edu/gkhome/metis/metis/overview is back on 😭

I made some adjustments in the text in light of https://github.com/mfem/mfem/pull/3196#issuecomment-1245438593. Please take a look.

Re-merged in next for testing...


Challenge Missing for Headquarters Base

Base: Headquarters Base
Level: 5
Challenge #: 12
Name: Dangerous Comment
ID: W0050
I already know the solution but some people may not, all you need to do is post "<IP_ADDRESS>; cat escape_plan.txt" in the comment form.

I noticed that the name for the current walkthrough page was incorrect, that has been updated. I've also added another step to specifically say what needs to be typed to get the flag. Let me know if anything else is needed for that challenge.


Cassandra support for chat history using CassIO library

Overview
This PR aims at building on #4378, expanding the capabilities and building on top of the cassIO library to interface with the database (as opposed to using the core drivers directly).
Usage of cassIO (a library abstracting Cassandra access for ML/GenAI-specific purposes) is already established since #6426  was merged, so no new dependencies are introduced.
In the same spirit, we try to uniform the interface for using Cassandra instances throughout LangChain: all our appreciation of the work by @jj701 notwithstanding, who paved the way for this incremental work (thank you!), we identified a few reasons for changing the way a CassandraChatMessageHistory is instantiated. Advocating a syntax change is something we don't take lighthearted way, so we add some explanations about this below.
Additionally, this PR expands on integration testing, enables use of Cassandra's native Time-to-Live (TTL) features and improves the phrasing around the notebook example and the short "integrations" documentation paragraph.
We would kindly request @hwchase to review (since this is an elaboration and proposed improvement of #4378 who had the same reviewer).
About the init breaking changes
There are many options when creating the Cluster object, and new ones might be added at any time. Choosing some of them and exposing them as __init__ parameters CassandraChatMessageHistory  will prove to be insufficient for at least some users.
On the other hand, working through kwargs or adding a long, long list of arguments to __init__ is not a desirable option either. For this reason, (as done in #6426), we propose that whoever instantiates the Chat Message History class provide a Cassandra Session object, ready to use. This also enables easier injection of mocks and usage of Cassandra-compatible connections (such as those to the cloud database DataStax Astra DB, obtained with a different set of init parameters than contact_points and port).
We feel that a breaking change might still be acceptable since LangChain is at 0.*. However, while maintaining that the approach we propose will be more flexible in the future, room could be made for a "compatibility layer" that respects the current init method. Honestly, we would to that only if there are strong reasons for it, as that would entail an additional maintenance burden.
Other changes
We propose to remove the keyspace creation from the class code for two reasons: first, production Cassandra instances often employ RBAC so that the database user reading/writing from tables does not necessarily (and generally shouldn't) have permission to create keyspaces, and second that programmatic keyspace creation is not a best practice (it should be done more or less manually, with extra care about schema mismatched among nodes, etc). Removing this (usually unnecessary) operation from the __init__ path would also improve initialization performance (shorter time).
We suggest, likewise, to remove the __del__ method (which would close the database connection), for the following reason: it is the recommended best practice to create a single Cassandra Session object throughout an application (it is a resource-heavy object capable to handle concurrency internally), so in case Cassandra is used in other ways by the app there is the risk of truncating the connection for all usages when the history instance is destroyed. Moreover, the Session object, in typical applications, is best left to garbage-collect itself automatically.
As mentioned above, we defer the actual database I/O to the cassIO library, which is designed to encode practices optimized for LLM applications (among other) without the need to expose LangChain developers to the internals of CQL (Cassandra Query Language). CassIO is already employed by the LangChain's Vector Store support for Cassandra.
We added a few more connection options in the companion notebook example (most notably, Astra DB) to encourage usage by anyone who cannot run their own Cassandra cluster.
We surface the ttl_seconds option for automatic handling of an expiration time to chat history messages, a likely useful feature given that very old messages generally may lose their importance.
We elaborated a bit more on the integration testing (Time-to-live, separation of "session ids", ...).
Remarks from linter & co.
We reinstated cassio as a dependency both in the "optional" group and in the "integration testing" group of pyproject.toml. This might not be the right thing do to, in which case the author of this PR offer his apologies (lack of confidence with Poetry - happy to be pointed in the right direction, though!).
During linter tests, we were hit by some errors which appear unrelated to the code in the PR. We left them here and report on them here for awareness:
langchain/vectorstores/mongodb_atlas.py:137: error: Argument 1 to "insert_many" of "Collection" has incompatible type "List[Dict[str, Sequence[object]]]"; expected "Iterable[Union[MongoDBDocumentType, RawBSONDocument]]"  [arg-type]
langchain/vectorstores/mongodb_atlas.py:186: error: Argument 1 to "aggregate" of "Collection" has incompatible type "List[object]"; expected "Sequence[Mapping[str, Any]]"  [arg-type]

langchain/vectorstores/qdrant.py:16: error: Name "grpc" is not defined  [name-defined]
langchain/vectorstores/qdrant.py:19: error: Name "grpc" is not defined  [name-defined]
langchain/vectorstores/qdrant.py:20: error: Name "grpc" is not defined  [name-defined]
langchain/vectorstores/qdrant.py:22: error: Name "grpc" is not defined  [name-defined]
langchain/vectorstores/qdrant.py:23: error: Name "grpc" is not defined  [name-defined]

In the same spirit, we observe that to even get import langchain run, it seems that a pip install bs4 is missing from the minimal package installation path.
Thank you!

Trying to fix linter/build errors I see only here in the PR discussion.
Locally when I make lint only a Python 3.10 run starts, is there a way to run the same checks (incl. "build") which I see are then triggered in the PR? I'm not very familiar with some of the tools involved, sorry.

Thank you for taking action @rlancemartin !
However, I am afraid an unresolved <<<<<<< HEAD resulted in the poetry.lock (from the force-push) which is what is making all tests above fail.

Thanks! I just rebased on master and can merge once tests pass.

Thank you for taking action @rlancemartin ! However, I am afraid an unresolved <<<<<<< HEAD resulted in the poetry.lock (from the force-push) which is what is making all tests above fail.

Ya, re-generating it now :)


Unit 9 Team Teach

Review Ticket
Lesson Plan: Unit 9 - Inheritance and Shapes
1. Introduction

Objective: Students will learn about inheritance in object-oriented programming (OOP) using shapes as an example. They'll understand how inheritance allows for code reuse and the creation of more complex structures from simpler ones.
Why It’s Important: Understanding inheritance is fundamental to writing efficient and maintainable code. It helps in creating hierarchical relationships and promoting code reuse, which is crucial in managing large codebases.

2. What Students Will Learn

Concept of Inheritance: Definition and purpose of inheritance in OOP.
Base and Derived Classes: How to create base classes and derive new classes from them.
Polymorphism: How inherited methods can be overridden to provide specific functionality.
Practical Application: Implementing shapes (e.g., Circle, Rectangle) using inheritance to demonstrate the concept.

3. Why We Want to Learn It First

Foundational Knowledge: Inheritance is a core concept in OOP and understanding it early will make it easier to grasp more advanced topics like polymorphism and abstract classes later on.
Code Efficiency: Learning how to use inheritance can simplify code and make it more modular and easier to maintain, which is essential as projects grow.

4. Engagement and Hands-On Learning

Interactive Examples: Start with a live coding demonstration of creating a basic Shape class and then derive Circle and Rectangle classes from it. Show how methods and attributes are inherited and overridden.
Hands-On Exercise: Have students implement their own shape classes based on a given base class. They can choose different shapes and implement methods specific to those shapes (e.g., area calculation).

Nice pull request


Support .pdf files

It's not that tough to embed a .pdf in a web document, .pdf uploads should be supported, also web documents could be saved as .pdf and repuloaded

This looks good: https://github.com/mozilla/pdf.js


pre command question about backup up mongo db?

Hi
I was wondering if someone else has had this issue before currently i can use the pre command to backup mysql and postgre but cant seem to get it working on mongo this is what i have so far
           docker exec -i 4minitz-mongo-1 /usr/bin/mongodump --gzip --archive=db.archive | restic backup --tag meetingsdb --stdin --stdin-filename db.archive
but it gives an our saying invalid
but if i run it manually it works fine
Thank you

I think you are confusing | and &&

@nettnikl No, he is not. He wants to pipe the mongo dump into restic for backing it up, which is possible.
The problem is probably the escaping of the command. As a quick solution, I propose putting the command into a bash script and mounting the script into the restic container so you can run it instead.
So, something like this:
  backup:
    # ...
    environment:
      PRE_COMMANDS: |-
        /usr/local/bin/backup_mongo.sh
    volumes:
      - ./backup_mongo.sh:/usr/local/bin/backup_mongo.sh

Thank you


bug: @ionic/react IonLoading component does not dismiss correctly

Ionic version:
"@ionic/react": "^4.8.0-rc.0",
Current behavior:
It does not dismiss with setShowLoading(false) after http fetch.
It only work when wrapping inside a timeout :(
Expected behavior:
It should dismiss when calling setShowLoading(false); after http fetch
Steps to reproduce:
Show the loading and run the http fetch. After the promise resolve or reject, I need to hide the loading
Related code:
// This will not work
setShowLoading(true);

loadProducts().then((data) => {
    setShowLoading(false);
    console.log('Yaay!', data);
}).catch((e) => {
    setShowLoading(false);
    console.log('Oops!', e);
});

// This will work
setShowLoading(true);

loadProducts().then((data) => {
    setTimeout(() => {
        setShowLoading(false);
    }, 1000);
    console.log('Yaay!', data);
}).catch((e) => {
    setTimeout(() => {
        setShowLoading(false);
    }, 1000);
    console.log('Oops!', e);
});

Other information:
N/A
Ionic info:
Ionic:

   Ionic CLI       : 5.2.5 (/usr/local/lib/node_modules/ionic)
   Ionic Framework : @ionic/react 4.8.0-rc.0

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   NodeJS : v10.15.3 (/usr/local/Cellar/node/9.5.0/bin/node)
   npm    : 6.10.3
   OS     : macOS Mojave

It seems that this is my fault, It caused by some caching.
Please delete this issue.

@liamdebeasi ,
I figured out the issue. It only work when wrapping inside a timeout.
Please check this issue.

@liamdebeasi , please re-open this issue.
I updated the new description above.


Check stdout in fn render and eval tests with non-zero exit code

Currently fn eval and render tests only check of the stdout if exit code is non-zero. Output to stdout of non-zero cases also must be verified.

Currently fn eval and render tests only check of the stdout if exit code is non-zero. Output to stdout of non-zero cases also must be verified.

This has been done in #2062


give ark cluster-admin privileges

Fixes #73
Signed-off-by: Steve Kriss<EMAIL_ADDRESS>

Tested locally & it got rid of the escalation errors.


PR for exposing the 'set' functionality of ProgressBarJS

Hi, would you be interested in a PR that makes it so you can call the 'set' function on the progress bar before the animate function? Here's the code I wrote for it:
didRender(...args) {
    this._super(...args);

    const { progressBar, progress, setProgress } = getProperties(
      this,
      'progressBar',
      'progress',
      'setProgress'
    );

    if (isPresent(setProgress)) {
      progressBar.set(setProgress);
    }

    progressBar.animate(progress, () => this.sendAction('onAnimationComplete'));
  },

I'd write a test for it!

Sure!
On Thu, Mar 15, 2018, 4:25 PM Robert Webb<EMAIL_ADDRESS>wrote:

Hi, would you be interested in a PR that makes it so you can call the
'set' function on the progress bar before the animate function? Here's the
code I wrote for it:
didRender(...args) {
this._super(...args);
const { progressBar, progress, setProgress } = getProperties(
  this,
  'progressBar',
  'progress',
  'setProgress'
);

if (isPresent(setProgress)) {
  progressBar.set(setProgress);
}

progressBar.animate(progress, () => this.sendAction('onAnimationComplete'));

},
I'd write a test for it!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/patience-tema-baron/ember-progress-bar/issues/7, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACk2SJz9UbRwjeA4gx-T2O_PFsBgKEb0ks5tes5RgaJpZM4Ss1E8
.

Thanks for the PR! Just released this under 1.0.0.


ci: update to node 18

:tada: This PR is included in version 1.31.14 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


🚀[FEA]: Extending Diagnostic Models to take files as inputs/Being able to work with Ensembles

Is this a new feature, an improvement, or a change to existing functionality?
Improvement
How would you describe the priority of this feature request
Low (would be nice)
Please provide a clear description of problem you would like to solve.
Hi,
I've enjoyed the Earth2Studio framework so far (seems to be better than Earth2MIP from experience). One interesting use case that I'm encountering is the analysis of cyclones/AR. ClimateNet originally intakes .nc files (converts them to a custom dataset) and analyzes them. The current workflow in Earth2Studio is all together, and if I'm not mistaken you have to give the Prognostic model rather than being able to use the file. It would be a great addition if we do diagnostic on ensembles (getting potential cyclone tracks...etc.)

Hi @david5010
If I'm understanding you correctly, you're looking to feed data directly from a file into ClimateNet?
The diagnostics can be worked with directly rather than using the coupled workflow you're referencing. The interface for each takes in a tensor and a coordinate system, so if you have some data on disk you can:

either use the built in Xarray datasets or develop your own to read from the files you have
use the fetch_data util to then generate the tensor and coordinate system
feed this into the ClimateNet model directly and get the outputs

This will be writing your own workflow, but this is what we want users to do. Have a look at the corrdiff example for a reference on directly working with a diagnostic: https://nvidia.github.io/earth2studio/examples/04_corrdiff_inference.html#sphx-glr-examples-04-corrdiff-inference-py

Sounds good, IO is one of the more complicated parts... largely due to saving the meta data (coordinate systems) in a meaningful way but not the data itself! Definitely look at existing samples and docs to better understand how we go about doing things.
Alternatively, you can always skip IO and save things yourself manually however you'd like. Its all just tensors and dictionaries at the end of the day : )

Thank you! I spent quite some time today analyzing the code and it seems that I'm getting confused on NetCDFBackend and XarrayBackend. I'm assuming that the xr.py for datasource is meant to work with XarrayBackend. I'll try to follow closely to some of the run.py example to see if I'm understanding things better. Thank you again for all the help!

@NickGeneva Thank you! I'm still trying it out but running into quite a few issues (might be my lack of understanding).

I don't fully understand the xr.py (DatasetFile) specifically. I tried loading the ensemble into DatasetFile but I'm a bit confused on the array_name. I've tried a single variable to a list of variable but it doesn't seem to fully work.

I might experiment more or try a hybrid version of ensemble and the diagnostic workflow

Oh I see, so it's simply a place holder until the data source is called again, right?

You are correct the implementation in Earth2Studio expects variable to be a coordinate in the array.
If you want to keep your data how it is now. I would recommend writing your own that works for this data format. As long as you expose the expected data source interface then it will work.
Something along the lines of:
def __call__(
        self,
        time: datetime | list[datetime] | TimeArray,
        variable: str | list[str] | VariableArray,
    ) -> xr.DataArray:

  arrays = []
  for var in variable:
      da = self.dataset[var].isel(ensemble=0, lead_time=0).sel(time=time)
     # Will need to add a var dim to data and variable coord system
     da = da.assign_coords(variable=[var])
     da = da.expand_dims(dim={"variable": 1}, axis=1)
     arrays.append(da)

  return xr.concat(arrays, dim='variable')

@NickGeneva Hi,
I still couldn't get it to work by mimicking built-in workflows so I tried to work directly with the tensors as you suggested. I reformatted by ensemble forecast into tensors that ClimateNet takes (shape (batch, variables, lat, lon) where variables are ordered = ["tcwv", "u850", "v850", "msl"] and gave the climatenet input coords). The issue now is that everything shows up as nan. I checked the scaling and got some really high values 1e36...etc. Could it be that ClimateNet doesn't take that ordering?

Climate should take those variables based on the code. Is your input data normalized at all? The model will normalize / de-normalize the values for you.
Also to confirm, I'm assuming that the model loaded successfully from the default package as well?

@NickGeneva I don't think the values were normalized since I saw that the function normalizes inside. However, I did notice a possibility for the nan output.
I first forecasted FCN using Spherical Gaussian as an ensemble for the variables "tcwv", "u850", "v850" and "msl". I did notice that for some reason, I have extreme values at some point:
[1.00369484e+05, 1.00360547e+05, 1.00354516e+05, ...,
1.00358773e+05, 1.00379492e+05, 1.00367367e+05],
[9.96920997e+36, 9.96920997e+36, 9.96920997e+36, ...,
9.96920997e+36, 9.96920997e+36, 9.96920997e+36]]]]],
I don't know if this is a common issue. I used the built-in ensemble workflow too.

@david5010 fcnv2 small is an older version of SFNO thats smaller and fine tuned to deterministic accuracy. SFNO is the same architecture as FCN V2 that we have seem better performance for ensembling with... but I leave the full assessment up to you. Thanks!

@NickGeneva Interesting enough, it seems that SFNO didn't perform as well as FCNV2 from earth2mip for cyclone analysis. When applying ClimateNet, SFNO results showed a lot of noise. But I do think there might be other factors at play like how I set the perturbation for SFNO...etc.


Add function to generate "keys" for test results.

More work on the stock item testing functionality.
References

https://github.com/inventree/InvenTree/issues/808
https://github.com/inventree/InvenTree/pull/811

The stock-item test result page now displays which templated tests have been completed (or otherwise)


Cant start luigid in background with Python 3.8

After a recent update (luigi and system), i got the following error:
"""
$ pip install --user --upgrade luigi
....
...
Successfully installed luigi-2.8.11
$ luigid --background --logdir ~/logs
Traceback (most recent call last):
File "/home/csaintje/.local/bin/luigid", line 8, in
sys.exit(luigid())
File "/home/csaintje/.local/lib/python3.8/site-packages/luigi/cmdline.py", line 33, in luigid
luigi.process.daemonize(luigi.server.run, api_port=opts.port,
File "/home/csaintje/.local/lib/python3.8/site-packages/luigi/process.py", line 101, in daemonize
ctx = daemon.DaemonContext(
File "/home/csaintje/.local/lib/python3.8/site-packages/daemon/daemon.py", line 280, in init
detach_process = is_detach_process_context_required()
File "/home/csaintje/.local/lib/python3.8/site-packages/daemon/daemon.py", line 801, in is_detach_process_context_required
if is_process_started_by_init() or is_process_started_by_superserver():
File "/home/csaintje/.local/lib/python3.8/site-packages/daemon/daemon.py", line 778, in is_process_started_by_superserver
if is_socket(stdin_fd):
File "/home/csaintje/.local/lib/python3.8/site-packages/daemon/daemon.py", line 743, in is_socket
file_socket = socket.fromfd(fd, socket.AF_INET, socket.SOCK_RAW)
File "/usr/lib/python3.8/socket.py", line 544, in fromfd
return socket(family, type, proto, nfd)
File "/usr/lib/python3.8/socket.py", line 231, in init
_socket.socket.init(self, family, type, proto, fileno)
OSError: [Errno 88] Socket operation on non-socket
$ uname -a
Linux delirium 4.19.97-1-MANJARO #1 SMP Sat Jan 18 00:17:24 UTC 2020 x86_64 GNU/Linux
"""
Launching it with $ luigid --logdir ~/logs yields no error.

https://pagure.io/python-daemon/issue/34
Seems like a depended lib issue. Thanks for reporting.
I should also point out we haven't done any test under 3.8 and Luigi doesn't officially support anything beyond 3.7.

Bumping for any recent activity. Currently have a 3.8 build that ideally shouldn't be rolled back.


Error 400 upon authorization

The problem
I'm getting the following error when I try to add the integration:
Access blocked: Nest’s request is invalid
Error 400: invalid_request
The out-of-band (OOB) flow has been blocked in order to keep users secure. Follow the Out-of-Band (OOB) flow migration guide linked in the developer docs below to migrate your app to an alternative method.
Request details: redirect_uri=urn:ietf:wg:oauth:2.0:oob flowName=GeneralOAuthFlow
What version of this integration (ha-nest-protect) has the issue?
3.12
What version of Home Assistant Core has the issue?
2024.5.5
Device / Model
Nest Protect
Diagnostics information
None
Home Assistant log
None, didn't get that far.
Additional information
No response

I just realized there's another thread and found the solution there.  I installed the beta and used the alternative method of generating a token.  The integration works now.

I just realized there's another thread and found the solution there. I installed the beta and used the alternative method of generating a token. The integration works now.


Make Toisto self-updating

See https://github.com/RhetTbull/applecrate

Not worth the trouble as updating is as simple as uv tool update toisto


SOLR-17431: Deprecate -p parameter where it doesn't refer to a port.

https://issues.apache.org/jira/browse/SOLR-17431
Description
Deprecate -p for all options where it doesnt' refer to a port.
Solution
add deprecation warnings.
Tests
ran tests
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

@janhoy  can I get an LGTM on this?


Queen anne fc

created in error -- I didn't realize the link from my terminal submitted a pr into your repo; I thought mine was the upstream.


Error: Call to undefined function Sentry\mb_strpos()

Welcome,
I have issue when try to add sentry to php project. I install composer and go with instructions:
I add require ‘path to autoload.php’
install composer require sentry/sdk:2.0.3
add Sentry\init([‘dsn’ => my dns]);
add error:
try {
$this->functionFailsForSure();
} catch (\Throwable $exception) {
Sentry\captureException($exception);
}

but it make my app stop working, so I add Sentry\captureException($exception); in try catch and get Error: Call to undefined function Sentry\mb_strpos().
I have installed mb_strpos.
How to fix it?

ext-mbstring is a requirement for this package, so you have to have it installed for it to work. In the latest version of this lib, I can see only 2 usages of that function:

https://github.com/getsentry/sentry-php/blob/c9b253229b95f8e5bbf6a3661a170a0be0f80563/src/Stacktrace.php#L155
https://github.com/getsentry/sentry-php/blob/c9b253229b95f8e5bbf6a3661a170a0be0f80563/src/Stacktrace.php#L285

It seems fine to me, are you sure that you have the extension enabled?

php -m shows that it is.

@GeSup there is a difference between running php -m and a web request are you testing everything by running php from the command line or are you getting the errors from a web request (as in in your browser when accessing your code)?
If you get the errors on a web request you could check using phpinfo() to make sure the mbstring extension is also enabled in the web environment.

I getting errors form a web request. phpinfo() shows there is no mbstring. I using docker that why it dosen't work. Thanks.

No problem, I'm not sure what's happening there but maybe the docs of your container can provide you with the instructions of enabling the correct extension.
I'll close this issue, feel free to re-open or create a new one if something else comes up!


Clean up files in parallel to avoid large recursive delete

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10027/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10030/
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10038/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10103/
Test PASSed.

LGTM

alluxio-bot, merge this please


is_data_master? Blocking Nginx Restarts on Frontends for nginx Dispatch

https://github.com/opscode/chef-server/issues/37

Passed here: http://wilson.ci.chef.co/job/chef-server-12-trigger-ad_hoc/288/downstreambuildview/

I added a CHANGELOG entry, and when taking one last look at this PR I realized that what we probably want here is:
   notifies :restart, 'runit_service[nginx]' unless backend_secondary?

to ensure that we don't attempt to restart nginx on the secondary.  Seems a bit crazy that we need to guard nearly every service restart with that same conditional, but that seems like the easiest path for this particular PR.


Block Pages

Hello! An option needed from time to time in my company ist the ability to "block" a page after it has been approved (similar to rejecting the display of a page as long as an approved version is not availabe). This is quite crucial for quickly removing information in standard procedures and updating it with new information (e.g. the dose of a medication) an then approving at again. I was thinking of some sort of "disapprove" button...
In addition, showing a message instead of a draft for pages which have no approved version yet would be another useful option.

The second feature should apply only to pages that haven't been approved at all or also to those which latest revision is not approved yet (but have been approved previously)?

The second option would only appay to those documents, which haven't been approved at all. For example: In my company (a hospital) we have to define procedures precisely, e.g. medication doses. So we collaborate while designin the initial document, nobody should see this because it could result in the application of a wrong dose. Also, sometimes it is necessary to block a document quickly if a crucial operating procedure contains an error. I hope this explains the intention of my post.
At this point I do wish to thank you a lot for your work. Your plugin ist one of the most important of all, as it enables us to fulfill quality definitions in healthcare systems.


Base modal template

Add base modal template for all modals used in dashboard.
Closes #1352
Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1440   +/-   ##
======================================
  Coverage    83.4%   83.4%
======================================
  Files         128     128
  Lines        5791    5791
  Branches      664     664
======================================
  Hits         4830    4830
  Misses        803     803
  Partials      158     158

Impacted Files
Coverage Δ

saleor/dashboard/discount/views.py
69.62% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


The detr model trains own dataset, ap is all 0

I have the same problem as you, is config not set correctly？

What did you modify in the config? How many GPUs did you use for training? Please follow the issue template to provide more details.

What did you modify in the config? How many GPUs did you use for training? Please follow the issue template to provide more details.

File coco_detection and the data settings have been used before. There should be no problem. DERT's config only modifies num_classes, which can be run but mAP is always 0.

DERT is hard to train. The hyperparams in the config are based on 8GPUs. If your batch size is different, please modify them to a suitable setting.

Has this problem been solved for AP?

Has the problem been solved？


Support for limited 2012-2022 timezone data

Nice library! date-fns really needs this timezone support built into the library.
Is there any support for only including the 2012-2022 timezone-support data?
Currently it looks like imports are coming from the timezone-support index which includes all of the data by default.

Yes, I'm afraid that you're right. You cannot decide, what time zone data will be used on the date-fns-timezone level.
Actually, I didn't expect, that the 2012-2022 data would be wanted so much :-) I didn't consider it important, because dates are entered in form fields im my application and it is impossible to predict how old date will be entered there. Hence always all time zones...
timezone-support is a bunch of functions and no object instance to initialize and send around. I will need to think something up, either some context object with the time zone data, or a global initialization.

Very nice library, thanks! But yes, we have a similar problem: we need to use a custom timezone data bundle generated by timezone-support. Any suggestions on how to add this? I'd be happy to put in a PR.

Any updates?

If using webpack you can alias timezone-support to one of the limited data versions.
resolve: {
  alias: {
    'timezone-support$': path.resolve(
        __dirname,
        'node_modules/timezone-support/dist/index-1970-2038.js'
      )
  }
}


Cant execute

Hey man,
nice project!
I'm new to github so i'm not sure if this is the right way to contact you.
I cant get TQCE executed, could you point me in the right direction?

I'm also wondering how I would run this?

I'm also wondering how I would run this?

I just ran the app with roughly the following.
cd Application
mvn package
mvn exec:java -D exec.mainClass="tq.character.editor.TqCharacterEditor"

I was able to browse to the save file and open it, though the skills tab was blank, which is what I was hoping to reset.

I just ran the app with roughly the following.
cd Application
mvn package
mvn exec:java -D exec.mainClass="tq.character.editor.TqCharacterEditor"

I was able to browse to the save file and open it, though the skills tab was blank, which is what I was hoping to reset.

Hey! Incredibly sorry that I missed this issue, had notifications for github turned off and haven't worked on it in a while. I haven't released anything that can be executed stand-alone as I've been running it from inside my IDE during development. @periodic mentioned how you can execute it as a workaround but it does require Maven setup locally. I will try to release and update the documentation later today.
@periodic Yeah, sorry my build doesn't have support for that yet. I have it planned whenever I get more free time again. Check out https://github.com/epinter/tqrespec for a full character editor.

@lprawetz better late than never I guess, but I've created an executable jar so you can at least start the application. Start it with "java -jar Application-1.0-SNAPSHOT-exec"
I will close the issue, please re-open if you still can't start it.


Code breaks when upgrading from 2.0.7 to 3.x

I have a matrix that refuses to work if I upgrade the library to the 3.x series. Bottom half is set to green, while the top half works just fine. Reverting to 2.0.7 solves the issue.

Check clock speed and clkphase values


error creating NodePool: googleapi: got HTTP response code 404

Problem
When trying to create a gcp.container.NodePool I now got this message :
 gcp:container:NodePool ([clustername]-cluster-pool-main):
    error: error creating NodePool: googleapi: got HTTP response code 404 with body: <!DOCTYPE html>

      <meta charset=utf-8>
      <meta name=viewport content="initial-scale=1, minimum-scale=1, width=device-width">
      <title>Error 404 (Not Found)!!1</title>

404. That's an error.

The requested URL /v1beta1/projects/[project_name]/locations/europe-west1-b/clusters/projects/[project_name]/locations/europe-west1-b/clusters/[cluster_name]/nodePools?alt=json&prettyPrint=false was not found on this server.  That's all we know.

This comes from a code I use since more than a year, everything was still fine like 2 months ago.
I didn't change a thing, I just needed to create a new stack/cluster, so that's likely something outside of my code. No error is reported during update on existing stacks/clusters.
Seems like the v1beta1 URL does not work anymore ?
I don't know if that's directly related but there's a deprecation warning and shutdown notice here https://cloud.google.com/kubernetes-engine/docs/how-to/protecting-cluster-metadata?hl=en_US#disable-legacy-apis
I already talked about this issue on the Slack channel https://pulumi-community.slack.com/archives/C84L4E3N1/p1580991257027200
Versions
pulumi
v1.10.0
npm
"@pulumi/gcp": "2.5.0",
"@pulumi/kubernetes": "1.5.0",
"@pulumi/pulumi": "1.10.0",
"@pulumi/random": "1.5.0",

gcloud CLI
Google Cloud SDK 279.0.0
beta 2019.05.17
bq 2.0.53
core 2020.01.31
gsutil 4.47

Other
Node Version : 1.15.8-gke.2

After some digging, I think the problem is that the formatted URL is wrong :
/v1beta1/projects/{project_id}/locations/{location}/clusters/projects/{project_id}/locations/{location}/clusters/{cluster_id}/nodePools
but as stated in the doc https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1beta1/projects.locations.clusters.nodePools/list
URL should be :
/v1beta1/projects/{project_id}/locations/{location}/clusters/{cluster_id}/nodePools
And this one is actually working as expected.
I don't know when/why it changed, and it seems not to be in the pulumi source code, maybe from a third party like terraform ?

Looks like it comes from https://github.com/terraform-providers/terraform-provider-google/blob/31926fa61be523a515ac97cc5dfc0f498b4c3b16/google/resource_container_node_pool.go

I'm hitting this with a new Pulumi install, using Python and
✗ pulumi version
v1.10.1

✗ pip freeze
Arpeggio==1.9.2
attrs==19.3.0
certifi==2019.11.28
chardet==3.0.4
dill==<IP_ADDRESS>
grpcio==1.27.1
idna==2.8
parver==0.2.1
protobuf==3.11.3
pulumi==1.10.1
pulumi-gcp==2.6.0
pulumi-kubernetes==1.5.3
requests==2.21.0
semver==2.9.0
six==1.14.0
urllib3==1.24.3

I can provide more diags if that would be useful.

@sylv3r I resolved this in my own deploy -- try this:
const pool = new gcp.container.NodePool(
  poolName,
  {
    cluster: cluster.name,
...

The cluster arg of the NodePort should refer to the cluster name, not sure if cluster.id in your code is returning the same thing.

@paultiplady Indeed - I was just testing out the same thing as that looked like it might be the cause.  It seems as if the beta provider changed the id format from being just the name (like test-cluster) to being a qualified name (like projects/pulumi-development/locations/europe-west1-b/clusters/test-cluster).  As far as I can tell, this hasn't change in the last 1+ years, so I'm not sure exactly how this code was working for you previously but is not any more.  But it does appear to be by design (though a little unfortunate).  Given that, I'm going to close this out - thanks for the help getting to the bottom of this.

Thanks @paultiplady and @lukehoban
Fun fact, I just thought during the night to check at the morning the value returned by cluster.id
I'll follow up on the slack thread about this change of returned value.


[SPARK-12495][SQL] use true as default value for propagateNull in NewInstance

Most of cases we should propagate null when call NewInstance, and so far there is only one case we should stop null propagation: create product/java bean. So I think it makes more sense to propagate null by dafault.
This also fixes a bug when encode null array/map, which is firstly discovered in https://github.com/apache/spark/pull/10401

cc @marmbrus @viirya

Test build #48233 has started for PR 10443 at commit de214af.

Test build #48233 has finished for PR 10443 at commit de214af.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #48242 has started for PR 10443 at commit ac3a88b.

Test build #48242 has finished for PR 10443 at commit ac3a88b.

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

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

Merged build finished. Test PASSed.

Test build #48389 has started for PR 10443 at commit a6b826c.

Test build #48389 has finished for PR 10443 at commit a6b826c.

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

Merged build finished. Test PASSed.

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

Thanks, merging to master.


Usage Instructions for Gitlab

Hi We have about 200 gitlab repos- could you please help us with usage instructions with gitlab.

config.yaml reference for gitlab would also help.

@eth0izzle Like- I can use this docker image docker pull eth0izzle/shhgit and command like below
shhgit --local [Directory]
Currently we use community edition of gitlab.

Ay,
I don't know if it can help but here is my .gitlab-ci.yml file I use in my project to use ShhGit :
---
# Run ShhGit Security Scanner
shhgit-security-scanner:
  image: gcr.io/changecommerce/proximis/tools/shhgit
  stage: test
  script:
    - /app/shhgit -config-path ./ -local ./
  retry:
    max: 2
    when: runner_system_failure

Ay,
I don't know if it can help but here is my .gitlab-ci.yml file I use in my project to use ShhGit :
---
# Run ShhGit Security Scanner
shhgit-security-scanner:
  image: gcr.io/changecommerce/proximis/tools/shhgit
  stage: test
  script:
    - /app/shhgit -config-path ./ -local ./
  retry:
    max: 2
    when: runner_system_failure


Tutorial not working with ver 3.5

I am trying to work with the basic tutorial involving Kafka:
https://github.com/streamsets/tutorials/blob/master/tutorial-2/directory_to_kafkaproducer.md
It's not working. It seems that you user interface(UI) has changed or evolved but the tutorial is still referring to the old user interface and steps.
The moment I am at this step:
Defining the Kafka Producer
There is no way I can follow it as the user interface and the navigation is not in sync with this documentation.
Products like stream sets which are having sucha heavy duty user interface and the navigation should be having almost the perfect documentation and updated as per the latest versions. Or may be the tutorial should clearly specify with what versions it is going to work?

Hi @ashutoshsharmaindia,
Thanks for pointing it out. We're doing our best to keep the tutorials up-to-date and will add this one to the list for sure. In the meantime... I did a quick comparison between the UI presented in this tutorial and the latest released version (3.5) and the only discrepancy I saw under "Defining the Kafka Producer" was "Data Format" which has been moved to its own tab. See below:

Other than that, you should be able to follow along. If not, please let me know.
Cheers,
Dash

yes, a total of waste time, because of the out-dated documents. I wish the docs are able to be updated, so that new version of SDC can work.

@wiwengweng Where do you see the discrepancy? @iamontheinet updated the documentation over a year ago. I just checked the tutorial with the current version and it looks ok to me.

@ashutoshsharmaindia I see no screenshot in your second comment


Possible bug in chunk transformation from 4.16.0 onwards

A recent commit introduced another bug as I suppose. For small chunk (<1000) there is no buffer transformation - chunk remains still as a string.
I think this is an improper behaviour. In my app I overwrite end function and for small responses I've got string instead of buffer in chunk parameter:
res.end = function (chunk) {
	//some logic here
	oldEnd.apply(res, arguments);
};

Is it a bug or is it on purpose?

It is on purpose. The end function is part of Node.js API and it can br called with a string or Buffer. It previously would have been just a Buffer if the cann came through res.send but res.end is a valid API that middlewares can call and with strings.
Here is the doxumentation for res.end : http_response_end_data_encoding_callback
Make sure if you are patching merhods to implement all the argument support the original method had.

Sorry, here is the doc link : https://nodejs.org/dist/latest-v8.x/docs/api/http.html#http_response_end_data_encoding_callback

Thanks for the answer. I've already aligned my method to support also string.

No problem, happy to help :)


prompt: update powerlevel10k submodule to v1.5.0

Release notes: https://github.com/romkatv/powerlevel10k/releases/tag/v1.5.0

LGTM! Thanks!


not include node_modules/@types in dev build

I installed @types/cordova and ionic build works fine.
but ionic serve output:

[11:19:42]  ionic-app-scripts 0.0.36
[11:19:42]  watch started ...
[11:19:42]  build dev started ...
[11:19:42]  clean started ...
[11:19:42]  clean finished in 1 ms
[11:19:42]  copy started ...
[11:19:42]  transpile started ...
[11:19:43]  lint started ...
[11:19:45]  lint finished in 2.36 s
[11:19:47]  typescript: D:/Develop/Project/team.app/src/pages/home/home.ts, line: 89
        Cannot find name 'cordova'.
  L88:  handler: ()=> {
  L89:      cordova.plugins.settings.openSetting("location_source", ()=> {
  L90:              createAlert.dismiss();

[11:19:47]  transpile failed
[11:19:47]  copy finished in 4.88 s
[11:19:47]  watch ready in 4.98 s

@izee,
Can you make a sample repo showing your problem? I have not seen this behavior in my testing.
Thanks,
Dan

Maybe... declare var cordova: any; after your imports?

@danbucholtz demo is here

ionic build --dev always build prod.
npm run build --dev or ionic serve will show the issue.

Hello, thanks for using Ionic! I unfortunately cannot reproduce this issue with the latest version of app-scripts. Because of this I am going to close this issue for now. I would recommend asking around on our forum for further help.

@jgw96 please checkout this repo, the issue is still exist.

Can confirm the issue still exists with ionic-cli 2.2.1.


fix cron bug

'gendo.cron' generator occurs an Exception.
'next' is CronTab's method.

that's a good idea, this will probably break the build


Update library example to use up-to-date API

@olafurpg The changes should be applied to doc/readme/README.md first, then reported in the README at the root via sbt tut, to make the CI happy.

Thanks @alexarchambault, I've updated doc/readme/README.md

Cool, thanks!


Some pretrained models

Could you provide some pretrained models? Thanks.

Hello Zijundeng,
Can you please provide FCN trained on cityscapes (vgg-gcn) model.


Cross has to be pressed twice to delete data from Search field

System information:

Operating system (distribution) and version Mac
DBeaver version 7.3.0EA

Steps to reproduce, if exist:

Open DB Navigator/Database Task panel
Enter a value in Search field (actual for both fields in database task panel)
Press the cross on the right side of the field

Expected: the value is deleted
Actual result: no visible changes
Workaround: double-click the cross

This is a well-known bug in SWT. We'll wait for a fix.


[Code Addition Request]: Addition of Memory Typing Game

Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Latest Merged PR Link
https://github.com/UTSAVS26/PyVerse/pull/832
Project Description
Players have to memorize and type words that briefly appear on the screen. This game helps improve both memory and typing skills.
Full Name
V Pratheek
Participant Role
Contributor at GSSOC , Hacktoberfest

@UTSAVS26 Please assign me this issue ASAP ..... I will start working on it


chore: Add Kamelet rest-openapi-sink

New Kamelet loads OpenAPI specification from (http/https URL) and calls an operation on the HTTP service. The request is generated respecting the rules given in the OpenAPI specification (e.g. path parameters, Content-Type).

Thanks @christophd


[GHSA-952p-6rrq-rcjv] Regular Expression Denial of Service (ReDoS) in micromatch

Updates

Affected products

Comments
Only the 4.0.6 version is NOT vunerable

this is not correct 4.0.6 is vulnerable

Hey there, thanks for the PR , but am I reading that thread correctly in that 4.0.6 is incompatible with the other 4.x releases? It might be better to be technically incorrect on this advisory if so and update it with the fixed and compatible 4.0.8 when that releases (which looks like soon maybe?)

Hey @darakian !
Yeah, it seems to be the case, that the v4.0.6 has been "mislabeled", that should be v5.0.0 and v4.0.7 does not contains the fix.
I also opened a PR https://github.com/micromatch/micromatch/pull/266 for a v4.0.8 that contains the fix for the CVE. I hope it will be merged soon.

Gotcha. Ya, I'd really rather avoid having dependabot tell however many millions of people Hey go update to a version that'll break everything if possible. Would you be ok waiting on 4.0.8 to release and then we can add a patched version to the advisory rather than changing the vulnerable range?

Yeah @darakian , I'll close this PR, and once 4.0.8 has been realesed either with https://github.com/micromatch/micromatch/pull/266 has been merged or other, I'll open a new one with the patched version.

Yeah @darakian , I'll close this PR, and once 4.0.8 has been realesed either with https://github.com/micromatch/micromatch/pull/266 has been merged or other, I'll open a new one with the patched version.

Sounds good. I'll keep my eyes 👀 peeled for another PR so this can get resolved ASAP.

Also, this issue has been around for a while, but I think it is not as severe as it had been set up now...

Ya, severity is a weird topic. Denial of service in particular is usually a low concern for modern web app stuff where the infra just reboots a server when it locks up/crashes/whatever, but maybe it's a high concern for a car on a highway.
We really need per-project severity adjustments IMO, but failing that moderate is pretty low in the CVE world. If you look at a breakdown of CVSS 3 scores over half are high/crit as per NVD rating
https://nvd.nist.gov/general/nvd-dashboard

Hey @darakian !
The fix has been merged! 🚀
https://github.com/micromatch/micromatch/pull/266

Advisory is updated 👍
https://github.com/advisories/GHSA-952p-6rrq-rcjv


security: check if shared memory belongs to current user and is only …

…read/writeable for them
Fixes #33

Any updates on merging this PR?


Fixes #184

see https://github.com/doctrine/dbal/blob/master/lib/Doctrine/DBAL/Schema/Table.php#L158

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Has tests?
yes

Fixed tickets
#184

Thank you!


Add tests directory to sdist tarball

pyproject.toml:
Change tool.poetry.include to include the tests directory recursively
for the sdist format.
Fixes #251
Signed-off-by: David Runge<EMAIL_ADDRESS>

hmm, something about this is making the tests sad. Let me poke around

Alrighty! I should be able to merge and release this later today. Thanks!


fixes OBOE, fixes #317

Short Description:
see #317
Changes:
changes line 51 to fix off-by-one-error and make loading of configuration by parameter possible again
see #317
@OpenPoGo/maintainers

:+1:

:+1:

Actually, this fix is in #270 as well. Makes sense just to close this and merge that when it's done (which is pretty much now).

Agreed, @andythorne


Release schedule/cycle

Hi, @KharitonOff
Sorry for asking here (not sure if this is the right place), but how often do you people release (github release)?
I see that there are a lot of bugfixes since last release and it's better for everyone to use a specific github release instead of deploying from your latest master (not sure how stable is that).

@syndbg thanks for asking. We don't have any release schedule.
Sorry for not triggering the github release function last time. I'll fix it immediately. You are very welcome to provide any further feedback.

@KharitonOff I see that new releases are made now. Thank you very much :tada:
I think the issue can be closed.


math: p_abs, p_exp: Remove const args from impl.

Follow up of #9

Travis found another affected function: p_sincos - fix amended into commit.

LGTM


Add a Security best practices to the docs

see https://groups.google.com/forum/#!topic/docker-user/-VWmjLTgbHc

i think @ewindisch has added something right?

see https://github.com/docker/docker/pull/9855 but they can always be improved with a PR

Thanks. Yes, I added some flavor to the docs and with our expanded security team are working on new docs, blogs posts, and whitepapers.

wowww shinyyy I await the PR
On Thu, Feb 26, 2015 at 5:51 PM, Eric Windisch<EMAIL_ADDRESS>wrote:

Thanks. Yes, I added some flavor to the docs and with our expanded
security team are working on new docs, blogs posts, and whitepapers.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/issues/4485#issuecomment-76316392.


Bug in [HttpPut] sample code

if (item == null || item.Id != id)
{
return BadRequest();
}
The above will always trigger a bad request. Need to remove that section from the sample.

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

ID: caaef794-56f4-43bd-3db7-55e818614abd
Version Independent ID: 4c3ff8b2-1ffd-78d7-6877-e218ea993666
Content: Create a Web API with ASP.NET Core and Visual Studio for Mac
Content Source: aspnetcore/tutorials/first-web-api-mac.md
Service: unspecified
Product: asp.net-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Dup of #6424


Usability analysis (event schedule)

Steps for analysis following usability sessions on event schedule:

[x] 1) Fill in transcripts - go through each pp recording and fill in quotes.

[x] 2) postit creation - mainly quotes, some observations. 1 question at a time (across participants). Write one quote per postit, adding pp number.

[x] 3) Group analysis with @jennpb  & @Heesoomoon  - look at each q, looking for groupings, themes, and label each grouping.

Feedback ready to share with content and experience team. Slide deck here: https://wellcomecloud.sharepoint.com/:p:/s/wcdigitalexperience/EcmCVZ2fwuZMh-iNyXS_7-4BTmp--zmP5Q1G0OyE-V2sMg?e=QbIwdx


[aws-alb-ingress] Add ALB Cognito

Example
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  namespace: testcase
  name: echoserver
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: echoserver
    spec:
      containers:
      - name: echoserver
        image: gcr.io/google_containers/echoserver:1.4
        imagePullPolicy: Always
        ports:
        - containerPort: 8080
---
apiVersion: v1
kind: Service
metadata:
  namespace: testcase
  name: echoserver
spec:
  ports:
  - port: 80
    targetPort: 8080
    protocol: TCP
  type: NodePort
  selector:
    app: echoserver
---
apiVersion: v1
kind: Secret
metadata:
  namespace: testcase
  name: odic-secret
data:
  clientId: XXXX
  clientSecret: XXXX
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  namespace: testcase
  name: echoserver
  annotations:
    kubernetes.io/ingress.class: alb
    alb.ingress.kubernetes.io/scheme: internet-facing
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTP":80},{"HTTPS":443}]'
    alb.ingress.kubernetes.io/certificate-arn: 	arn:aws:acm:us-west-2:126450723953:certificate/2e2e7768-ef18-450d-87d9-20b8ffa65655
    alb.ingress.kubernetes.io/actions.ssl-redirect: '{"Type": "redirect", "RedirectConfig": { "Protocol": "HTTPS", "Port": "443", "StatusCode": "HTTP_301"}}'
    alb.ingress.kubernetes.io/auth-type: oidc
    alb.ingress.kubernetes.io/auth-idp-oidc: '{"Issuer":"https://goruha.auth0.com/","AuthorizationEndpoint":"https://goruha.auth0.com/authorize","TokenEndpoint":"https://goruha.auth0.com/oauth/token","UserInfoEndpoint":"https://goruha.auth0.com/userinfo","SecretName":"odic-secret"}'
spec:
  rules:
  - host: auth.us-west-2.testing.cloudposse.co
    http:
      paths:
      - path: /
        backend:
          serviceName: echoserver
          servicePort: 80

    alb.ingress.kubernetes.io/certificate-arn: 	arn:aws:acm:us-west-2:126450723953:certificate/2e2e7768-ef18-450d-87d9-20b8ffa65655
currently hardcoded
would prefer using ACM to generate -- or caller uses a data source?

un-hardcode:
"SecretName":"odic-secret"
which references
apiVersion: v1
kind: Secret
metadata:
  namespace: testcase
  name: odic-secret <<<<
data:
  clientId: XXXX
  clientSecret: XXXX

in the example helmfile (found in this PR's description), we should un-hardcode:
"SecretName":"odic-secret"
which references
apiVersion: v1
kind: Secret
metadata:
  namespace: testcase
  name: odic-secret <<<<
data:
  clientId: XXXX
  clientSecret: XXXX

I've updated the PR description, correcting presumed typo odic-secret -> oidc-secret
@goruha can you confirm this was a typo?

Answering my own questions above:

The example kubectl file that is this PR's #Example is fine with hardcoded arn:aws:acm.
No additional ACM work is required here.

The example hardcoded secrets are also fine.


Radial Menu Update

The radial hand menu is a pretty powerful tool for adding functionality to an app, but the existing implementation is still kinda partial. In particular, Jumbli's Idea Engine has done some really nice iteration on the UX.

[x] Reduce accidental activation (look at)
[x] Reduce accidental activation (in field of view?)
[x] Visual polish
[x] Icon support
[x] Extensibility (toggles?)
[x] Ambidextrous
[x] Live editing menus at runtime
[x] Child menu indicator

Old version looked kinda like this:

Now looks like this!

This is now in v0.3.7-preview.2!


Number of sms/obd calls sent over time

This PR will create the "message.{message_format}.sum" metrics when a message is submitted to the message sender.

@rudigiesler please review, thanks.

:+1:


refactor: break down server initialisation in different parts

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.981%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5351

Relevant Lines:
7332

💛  - Coveralls


Allow browser spell checking for markdown raw content.

As reported here:
https://twitter.com/jbarnoud/status/717354016624214017

This is tricky because CodeMirror does not use ContentEditable, hence built-in browser spell checkers don't work.


cell providing ability

CellProvider is disposable because of ByTargetIndex inside. The provider will be disposed at ModelContext (DataPad) for example.

Why CellProvider is Disposable?
Nice to document what HasParentWrapper is in the javadoc. Nice to include a motivating example for it there.
SrcT in CellProviders
BaseProjectionalSyncToCellMapping. Why we have Base prefix? It's not a base class for anything. Nice to rename to ProjectionalSyncToCellMapping. As far as I see it's wrapper for projectional synchronizer. I don't like such an architecture. What if we want to have several features for synchronizers? We won't be able to wrap BaseProjectionalSyncToCellMapping with another decorator.


Close button for transaction confirmation actionsheet works but isn't visible

There seems to be a button at the top left that closes the actionsheet (good), but it isn't visible.

Ah! We're trying to solve this one. It seems it's a theming issue and is solved in the darkmode theming. It always works for me (Emulator, Xiaomi and Samsung) but other Android phones it does not.

I will add a universal fix for this release


script title gets shortened for nested github proejct

Somewhere in pxt, a heuristic is current out the name of the extension name, jacdac-button -> jacdac for some reason.

when does this happen?

Somehow when opening jacdac/humidity in makecode

From: Michał Moskal<EMAIL_ADDRESS>Sent: Thursday, February 11, 2021 8:06:45 PM
To: microsoft/jacdac<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/jacdac] script title gets shortened for nested github proejct (#239)
when does this happen?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fissues%2F239%23issuecomment-777719727&data=04|01|jhalleux%40microsoft.com|a2bad8f76c814d4b8f0508d8cec02d4f|72f988bf86f141af91ab2d7cd011db47|1|0|637486672080975995|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=25gNPOvwIxihrrNeKqMqJ0LhIqVB75Xup3FzNzI0ROA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKOXSURJZLUNSCQMFCTS6QTELANCNFSM4XPEJEOA&data=04|01|jhalleux%40microsoft.com|a2bad8f76c814d4b8f0508d8cec02d4f|72f988bf86f141af91ab2d7cd011db47|1|0|637486672080985991|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IMsSCJS%2FyhKEC1%2FWt89VoGWapYU1tLpWoQ5R%2BfSFPkE%3D&reserved=0.

I think i fixed it.


Details individual device

As a member of the Support team I want to be able to see all details of an individual device in order to be able to troubleshoot issues more quickly.

Implemented in the REST service: http:///devices/


[mac] add platform macro for ACK FP

The commits add a new platform macro for ACK FP:

platforms that set ACK FP correctly should
#define OPENTHREAD_CONFIG_PLATFORM_SET_ACK_FP_CORRECTLY 1
other platforms should #define OPENTHREAD_CONFIG_PLATFORM_SET_ACK_FP_CORRECTLY 0

The commits check if OPENTHREAD_CONFIG_PLATFORM_SET_ACK_FP_CORRECTLY is set to 1. If not, it uses source match to determine ACK FP for data requests, which is highly likely to be correct.
The direct reason for introducing this flag is because CC2538 OT failed to pass 4 TH tests cases due to redundant empty data frame caused by PR #4131 . PR #4131 relies on platform radio driver to set ACK FP correctly, unfortunately this is not the case for most platforms (only nrf52840, posix, and cc2538 with PR #4263 set ACK FP correctly).
We expect this issue to impact all platforms that do not implement ACK FP properly yet. So, this PR is a workaround for this issue on other platforms.
Platforms that latter implement ACK FP correctly should change OPENTHREAD_CONFIG_PLATFORM_SET_ACK_FP_CORRECTLY to 1
This RP conflicts with PR #4263 on examples/platforms/cc2538/radio.c

I am fine with this change as as a work-around to pass the TH tests (on all platforms), but I think for longer term we should aim to add support for FP to be set correctly on all radio platforms.

Agree. @abtink
Also, PR #4131 is not the root cause of this issue. The root cause is to be identified.

I am fine with this change as as a work-around to pass the TH tests (on all platforms), but I think for longer term we should aim to add support for FP to be set correctly on all radio platforms.

Agree. @abtink
Also, PR #4131 is not the root cause of this issue. The root cause is to be identified.

If we want to force platforms to implement this properly, is it better not to introduce this config option?

@jwhui I fully agree. I also do prefer if it is addressed in platform radio code. I am fine with adding such a work-around if it's urgent to unblock.
The radio code itself can use the same work-around, i.e. if driver itself does not provide the info about FP in the emitted ack, it can possibly use a src match table lookup associated with the dst address of the frame to determine/guess this.

If we could force platforms to implement this properly, it would be definitely a better option.
@jwhui How could we make it happen? Raise one issue to note that individual platform should resolve this to get Thread Certified?

@jwhui How could we make it happen? Raise one issue to note that individual platform should resolve this to get Thread Certified?

I think it makes sense to raise a GitHub Issue describing the problem and listing platforms that are currently affected.

Submitted #4358 as discussed above.


SinTest - training images / testing images

I've been dabbling in this space for a couple of weeks.  perhaps we can help each other.
I love how your code is very simple and elegant.
I see you like to stay away from Surge and other frameworks - rightly so.
Keeping things simple.
I would say though - I'm getting some good mileage with EasyImagy (needs carthage) / it can build images by pixel arrays safely.  I threw this code together(which is a mash up of other code) which builds out 1000 images / 1000 angles. https://github.com/johndpope/Bresenham-Line
here's the gist
https://gist.github.com/johndpope/87b58509d134b256a54869df533e1834
working on integrating machine learning stuff slowly.

fyi - https://github.com/koher/EasyImagy

I think I have most of the ducks lined up - just need to correctly configure the neural network.
Given 1000 images - 100 x100 - with an angle drawn and corresponding angle in another array - build a neural network to infer any image with a line in it.
Would it make any sense to just use this structure?
let network = Network(layerStructure: [1,1000,1], learningRate: 0.1)

import Cocoa
import EasyImagy
import Foundation
import Surge

typealias Byte = UInt8

extension Image where Pixel == UInt8 {
    func byteArray()->[Byte]{
        var pixelIterator = self.makeIterator()
        var pixelArray:[Byte] = []
        while let pixel = pixelIterator.next() {
            pixelArray.append(pixel)
        }
        return pixelArray
    }
}

extension CMKViewController {

    func generateTrainingImage(_ angle:Double,_ width:Int,_ height:Int,_ thickness:Double)->[Byte]{
        var image = Image<RGBA<UInt8>>(width: width, height: height, pixel: RGBA.transparent)

        let x_0 = Double(width / 2)
        let y_0 = Double(height / 2)
        let c = cos(angle)
        let s  = sin(angle)
        for y in 0...height-1{
            for x in 0...width-1{
                let w1 = abs((Double(x)-x_0)*c + (Double(y)-y_0)*s)
                let h1 = -(Double(x)-x_0)*s + (Double(y)-y_0)*c
                if (w1 < Double(thickness / 2) && h1  > 0){
                    image[x,y] = RGBA(red: 255, green: 0, blue: 0, alpha: 127)
                }
            }
        }
        let nsImage = image.nsImage
        print("nsImage",nsImage)
       let grayscale: Image<UInt8> = image.map { $0.gray }
        return grayscale.byteArray()
    }

    func testAndTrainData(){

        print("BEGIN TEST AND TRAIN DATA")
        let width = 100 // Image width
        let height = 100 // Image heigth
        let thickness:Double = 1.0 // Line thickness

        let numTrain = 1000
        let numTest = 1000
        var testImages:[[Byte]] = []
        var testAngles:[[Double]] = []
        var trainImages:[[Byte]] = []
        var trainAngles:[[Double]] = []

        for _ in 0..<numTrain{

            let angle = Double.pi * Double.random(in: 0..<1)
            trainAngles.append([angle])
            let image = generateTrainingImage(angle,width,height,thickness)
            trainImages.append(image)
        }

        for _ in 0..<numTest{

            let angle = Double.pi * Double.random(in: 0..<1)
            testAngles.append([angle])
            let image = generateTrainingImage(angle,width,height,thickness)
            testImages.append(image)
        }

        let imageData = trainImages.map{ return $0.map{ return   Double($0) / 255 }}

        let network = Network(layerStructure: [1,1000,1], learningRate: 0.1)
        // Put setup code here. This method is called before the invocation of each test method in the class.

        network.train(inputs: imageData, expecteds: trainAngles, printError: true)

    }

    // Returns encoded angle using specified method ("binned","scaled","cossin","gaussian")
    func encodeAngle(_ angle:Double,_ method:String)->[Double]{
        var X:[Double] = []
        if (method == "binned"){ // 1-of-500 encoding
            X = [Double](repeating: 0, count: 500 )
            X[Int(round(250*(angle/Double.pi + 1)))%500] = 1
        }else if (method == "gaussian"){ // Leaky binned encoding

            for i in 0...500{
                X.append(Double(i))
            }

            let idx:Double = 250*(angle/Double.pi + 1)
            let Y = X-idx
            //            X = exp(-Float.pi*(Y)**2.0) //TODO
        }else if (method == "scaled"){ // Scaled to [-1,1] encoding
            X = Array([angle/Double.pi])
        }else if (method == "cossin"){ //Oxinabox's (cos,sin) encoding
            X = Array([cos(angle),sin(angle)])
        }
        return X
    }

    // Returns decoded angle using specified method
    func decodeAngle(_ X:[Double],_ method:String)->Double{
        var M:Double = 0
        var angle:Double = 0
        if (method == "binned") || (method == "gaussian"){ // 1-of-500 or gaussian encoding
            M = X.max()!
            for i in 0...X.count{
                if abs(X[i]-M) < 1e-5{
                    angle = Double.pi*Double(i/250) - Double.pi
                }
            }
            // angle = pi*dot(array([i for i in range(500)]),X)/500  # Averaging
        } else if (method == "scaled"){ // Scaled to [-1,1] encoding
            angle = Double.pi*X[0]
        }else if (method == "cossin"){ //# Oxinabox's (cos,sin) encoding
            angle = atan2(X[1],X[0])
        }
        return angle
    }

    func test(){
        // Add test image
        let image = Image<RGBA<UInt8>>(nsImage: NSImage(named: "fast_1.png")!)

        print("image.xRange:",image.xRange)
        let testImage = NSImage(named: "fast_1.png")!
        let testIV = NSImageView(image: testImage)
        testIV.frame = CGRect(x:0,y:0,width:image.width, height:image.height)
        self.view.addSubview(testIV)

        let testView = TestView()
        testView.frame = CGRect(x:0,y:0,width:image.width, height:image.height)
        self.view.addSubview(testView)

        let points = Bresenham.pointsAlongCircle(xc: 0, yc: 0, r: 3)
        print("points:",points)

    }

}


[SPARK-28798][DOC][SQL]Document DROP TABLE/VIEW statement in SQL Reference.

What changes were proposed in this pull request?
Added doc for DROP TABLE and DROP VIEW sql command

Why are the changes needed?
For reference DROP TABLE  or DROP VIEW in spark-sql

Does this PR introduce any user-facing change?
It updates DROP TABLE  or DROP VIEW reference doc

How was this patch tested?

 Attached the Snap
DROP TABLE

DROP VIEW

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@shivusondur Could you please attach the screenshot of the page to make it easier for reviewers ?

DROP TABLE

DROP VIEW

@shivusondur Could you please attach the screenshot of the page to make it easier for reviewers ?

@dilipbiswal
attached the screenshot

@shivusondur A few related DOC PRs were merged by @gatorsmile
https://github.com/apache/spark/pull/25525
https://github.com/apache/spark/pull/25523
Could you please make changes by following the format of these PRs ? A few things to note:

Formatting of the parameters under dl tag now.
Examples should contain output. Please refer to the INSERT PR for examples.
Add some related statements - statements users are likely to visit. For example after a drop users may want to do a show tables to check the catalog or be interested in some other view commands.

@dilipbiswal @srowen @gatorsmile
Updated the PR according to #25525 and #25523 PR's
Please review kindly check for merging.

Can one of the admins verify this patch?

@dilipbiswal
Thanks for your time.
i handled all the comments and updated the screen shot.
Please check

ok to test

Test build #111093 has started for PR 25533 at commit 194a0b0.

Merged build finished. Test PASSed.

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

Test build #111093 has finished for PR 25533 at commit 194a0b0.

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

Merged build finished. Test PASSed.

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

@srowen
Handled your comments, please check

Test build #112863 has started for PR 25533 at commit df23c95.

Merged build finished. Test PASSed.

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

Test build #112863 has finished for PR 25533 at commit df23c95.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

@srowen
Thanks for reviewing.
I handled your comments.

Test build #112950 has started for PR 25533 at commit 2ca9a6e.

Test build #112950 has finished for PR 25533 at commit 2ca9a6e.

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

Merged build finished. Test PASSed.

Merged to master


[PE-D][Tester C] Non-standard names not supported by app

I have clients called RAJU S/O DINESHWAR and I would like to save his name as such:
add n/Raju S/O Dineshwar p/222 e/raju@test i/888Z l/SKK1000A a/Test
However, this is not allowed by the app, which only allows alphanumeric names. Not supporting non-standard names might result in less functionality for users with clients with non-standard names.
I think would be good to implement this feature (or actually better to write this as a "Planned Enhancement" for actual PE).

Labels: severity.Low type.FeatureFlaw
original: conradsoon/ped#14

Change to allow /


fix invalid setting priority error message

This commit fixes an invalid error, because priority parameter may be
undef for list and pref type setting files.

@daenney: Thanks for taking a look. Test failure is unrelated. The beaker test can't download a gpg key. Can you restart the test?

I rescheduled the build, lets see what happens now.

Tests are all green now.

This appears to be fixed in stdlib, and puppet 3 is EOL. And closing for inactivity. Feel free to reopen if this is still an issue!


Add Domain/Account Assignment at Instance Creation

ISSUE TYPE

Enhancement Request

COMPONENT NAME

UI

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

N/A

OS / ENVIRONMENT

KVM

SUMMARY

Add Account/Domain Assignment at instance creation.  At this time, you need to go through the instance creation wizard and then assign the account later on.
STEPS TO REPRODUCE

N/A - Workflow Enhancement

EXPECTED RESULTS

When creating an instance being able to assign Domain/Account out of the gate.  Proposed to add these as dropdowns under #8 Details Section in the wizard.

ACTUAL RESULTS

Not possible today unless done via API.  You need to create the instance and then circle back to assign domain/account.

@BryanMLima Yes, this will work.  I didn't think to search for VM since they are called Instances now.  Thanks for the response.

fixed in #8919


Documentation on Configuring Asp.Net Core SDK!

2.3.0 version of Application Insights Asp.Net Core SDK made a bunch of improvements to the configuration story of the SDK. The post is to quickly highlight recent changes, and also can serve as a quick reference.

Adding Application Insights SDK to your Asp.Net Core Application
Step1:
Edit .csproj to add the following line
<PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.3.0" />
Step2:
Add the following line to ConfigureServices method of your Application's Startup class.

public void ConfigureServices(IServiceCollection services)
{
......
 services.AddApplicationInsightsTelemetry("myinstrumentationkey");  // Passing ikey here is optional as mentioned below.
...
}

Its also possible (and preferred) if instrumentation key is supplied via appsettings.json file as shown below. In this case services.AddApplicationInsightsTelemetry(); should be used to enable Application Insights.
{
  "ApplicationInsights": {
    "InstrumentationKey": "11111111-2222-3333-4444-555555555555"
  }
}

As an alternate to appsettings.json, ikey can also be set in ENV variable ApplicationInsights:InstrumentationKey

Removing Telemetry Modules (which does auto-collection)
To remove any of the existing telemetry modules, please use the following in ConfigureServices method after the services.AddApplicationInsightsTelemetry() method.

var performanceCounterService = services.FirstOrDefault<ServiceDescriptor>(t => t.ImplementationType == typeof(PerformanceCollectorModule));
if (performanceCounterService != null)
{
 services.Remove(performanceCounterService);
}

Replace the name of the module to the one which you wanted removed. The following lists the default TelemetryModules automatically enabled with application insights. The above example can be used to remove any of these modules.

DependencyTrackingTelemetryModule
PerformanceCollectorModule
QuickPulseTelemetryModule
AppServicesHeartbeatTelemetryModule
AzureInstanceMetadataTelemetryModule

Configuring Telemetry Modules
Any of the above listed modules can be configured using the new extension method ConfigureApplicationInsightsTelemetryModule as shown below.

public void ConfigureServices(IServiceCollection services)
{
......
services.AddApplicationInsightsTelemetry("ikey");
services.ConfigureTelemetryModule<DependencyTrackingTelemetryModule>( module => module.SetComponentCorrelationHttpHeaders = false);

//sets AutheticationApiKey for QuickPulse
services.ConfigureTelemetryModule<QuickPulseTelemetryModule>( module => module.AuthenticationApiKey = "YOUR-API-KEY-HERE");
...
}

Adding new Telemetry Initializers
Use the following in ConfigureServices method.

public void ConfigureServices(IServiceCollection services)
{
......
services.AddSingleton<ITelemetryInitializer, MyCustomTelemetryInitializer>();
services.AddApplicationInsightsTelemetry("ikey");
....
}

Removing existing Telemetry Initializers
Use the following in ConfigureServices method.

public void ConfigureServices(IServiceCollection services)
{
..
services.AddApplicationInsightsTelemetry("ikey");
// Remove a specific built-in TelemetryInitializer
var tiToRemove = services.FirstOrDefault<ServiceDescriptor>(t => t.ImplementationType == typeof(AspNetCoreEnvironmentTelemetryInitializer));
            if (tiToRemove != null)
            {
                services.Remove(tiToRemove);
            }

// or Remove all
services.RemoveAll(typeof(ITelemetryInitializer)); // this requires importing namespace using Microsoft.Extensions.DependencyInjection.Extensions;
..
}

Adding Telemetry Processors
Additional TelemetryProcessors can be added to the TelemetryConfiguration by using the extension method AddApplicationInsightsTelemetryProcessor as shown below

public void ConfigureServices(IServiceCollection services)
{
    // ...
    services.AddApplicationInsightsTelemetry();
    services.AddApplicationInsightsTelemetryProcessor<MyFirstCustomTelemetryProcessor>();

    // If you have more processors:
    services.AddApplicationInsightsTelemetryProcessor<MySecondCustomTelemetryProcessor>();

    // ...
}

6.. Overriding TelemetryChannel.
By default, ServerTelemetryChannel will be setup by the SDK with its defaults. If user wants to use a different channel or customize ServerTelemetryChannel itself, then use the following example.
Add the required channel to DI in ConfigureServices() method of your application's Startup class before AddApplicationInsightsTelemetry() call. For eg:, the following sets up InMemoryChannel with the specific MaxTelemetryBufferCapacity
public void ConfigureServices(IServiceCollection services)
{
...

services.AddSingleton(typeof(ITelemetryChannel), new InMemoryChannel() {MaxTelemetryBufferCapacity = 19898 });
services.AddApplicationInsightsTelemetry("ikey");
....
}

Version Info
SDK Version :
.NET Version :
How Application was onboarded with SDK(VisualStudio/StatusMonitor/Azure Extension) :
OS :
Hosting Info (IIS/Azure WebApps/ etc) :

The documentation of all the new extension methods is code only https://github.com/Microsoft/ApplicationInsights-aspnetcore/blob/883589c8a99fded1b754c5ed688734274c9243e9/src/Microsoft.ApplicationInsights.AspNetCore/Extensions/ApplicationInsightsExtensions.cs
Awesome isn't it !?
No, seriously all documentation found on several places is outdated or won't work on
For example, this one is outdated https://docs.microsoft.com/en-us/azure/azure-monitor/app/api-filtering-sampling
Another one is good 'enough'
https://docs.microsoft.com/en-us/azure/azure-monitor/app/asp-net-core-no-visualstudio


Lookup customization (like template specialization)

It's like template specialization in C++, but essentially it's just as registering a custom function with a mangled name into the global vtable, nothing's fancy. You will notice that a partially applied "template" (higher-kinded function) could be registered with a custom function too. Powerful features for free.

OOP is redesigned now (#109).


Support N64DD

This is a big task, might need to break it down. But the gist is, add support for N64DD.
First step is to implement the protocol, serve data from external flash for convenience.
Later we might want to stream the disk contents from the SDCard.

Lots of good info here https://github.com/LuigiBlood/64dd/wiki/Memory-Map

Don't forget about disk swapping. Maybe use a companion app and connect to it over WiFi?
Apparently Doshin 2 requires disk swapping, and Mario Artist also supports import/export from each other via disk swapping.


Add some way to include a high level API description

I see a info_description markdown div but no way to pass this in?

A work around for this that I have been using is to override the index.html template file. Not ideal but it gets the job done.

Thanks @oden65, I'll give it a shot!


GMD: Gradle plugin Helm task execution for install chart command

:tada: This issue has been resolved in version 0.12.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Generate Schema Documentation

https://app.quickdatabasediagrams.com/#/
Looks promising - export to svg then embed into wiki doc

Closing - open individual tasks to document specific sections of the schema / featureset