ARROW-2412: [Integration] Add nested dictionary test case, skipped for now

Adds a test case that contains a dictionary-encoded struct and list, both with dictionary-encoded children. As expected Java, C++ and JS all complain about the generated JSON.

@TheNeuralBit is this still relevant?

@emkornfield thanks for asking :) I've let this go way too long
This is supposed to test the use-case described here
This test is designed to fail.. but it's probably worth merging this PR without the generate call and create associated JIRAs to fix the places where it fails.

@trxcllnt could you take a look at this since I added changes to JS to support this use-case?

@TheNeuralBit does this mean dictionaries will be able to reference prior dictionaries? yay!

@trxcllnt yep!

I can work on this for C++ but I won't be able to do it until after my work on variable dictionaries (which will also allow implementing Delta dictionaries) for ARROW-3144 is completed, so it could be a week or more

@wesm understood. Would you want to hold off on merging this until we have it implemented in Java and C++ as well?

Yeah, I think we should have 2 implementations that can speak the correct binary with each other

Codecov Report

Merging #1848 into master will increase coverage by 2.51%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1848      +/-   ##
=========================================
+ Coverage   87.99%   90.5%   +2.51%
=========================================
  Files         773      74     -699
  Lines       96308    5456   -90852
  Branches     1251    1251
=========================================
- Hits        84747    4938   -79809
+ Misses      11325     510   -10815
+ Partials      236       8     -228

Impacted Files
Coverage Δ

js/src/ipc/metadata/message.ts
94.26% <100%> (+1.27%)
:arrow_up:

js/src/ipc/metadata/json.ts
96.73% <100%> (+4.34%)
:arrow_up:

python/pyarrow/ipc.pxi

cpp/src/arrow/csv/chunker-test.cc

cpp/src/parquet/column_page.h

cpp/src/parquet/bloom_filter-test.cc

cpp/src/arrow/array/builder_decimal.cc

cpp/src/plasma/client.cc

cpp/src/arrow/io/test-common.h

cpp/src/arrow/util/int-util-test.cc

... and 694 more

Continue to review full report at Codecov.

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

@TheNeuralBit btw this change looks good to me, but we should probably add some test cases to validate the reader and writer do the correct thing. We may also encounter some unanswered questions in the process. For example, are there requirements/guarantees that dictionary ids are preserved, or is the implementation free to rewrite the ids of dictionaries as necessary?
@wesm I want to double check, how are you thinking about dictionary deltas, and how it relates to variable dictionaries? The JS has deltas implemented as appending new values to an existing dictionary to create a ChunkedArray. I could see someone wanting to replace or partially update a dictionary, or being confused and assuming that's what the isDelta flag does, so I want to ensure JS and C++ are in alignment there.

@trxcllnt our arrays are immutable, so to avoid breaking immutability, when a delta is observed, the next RecordBatch produced will have a larger dictionary for the corresponding Field. What I'm working on for ARROW-3144 is making the dictionary part of the ArrayData data structure rather than the Schema/type object. You might contemplate doing something similar in JS if the change goes over well in C++ (I'm pretty sure it's the right thing to do...)

@wesm if I recall correctly, the JS implementation did have a special DictionaryData subclass of ArrayData, but we eliminated it (and all the other ArrayData subclasses) when we consolidated the functionality into the class we have now, though the isDelta flag was an important motivator also.
The Vectors in JS are also immutable, so I really meant we (zero-copy) concatenate the current dictionary with the delta dictionary to create a ChunkedArray.
I'm not sure how familiar you are with the JS, and obviously solve this however you need to in CPP, but in case there's ideas worth cross-pollinating, here's a quick description of how we're doing it in JS:
To ensure all Fields that reference the same dictionary id are always pointing at the same Vector instance, we store added a dictionaryVector pointer to the Dictionary DataType. Since this instance is part of the Schema, it's shared across all current and future RecordBatches. We then updated DictionaryArray to retrieve the dictionaryVector  from the shared Dictionary DataType instance, rather than each DictionaryArray having its own pointer to the dictionary.
When the Reader sees new delta DictionaryBatch, it updates the dictionaryVector pointer on the Dictionary DataType in the Schema.
This approach ensures that all DictionaryArrays point to the same underlying dictionary instance, even if you created the DictionaryArray before the delta batch was processed. This is safe, since DictionaryArrays that already exist before the delta dictionary is parsed by the reader won't have indices that point beyond the original DictionaryBatch. But if you have a pointer to a DictionaryArray that was created before the delta batch came in, calling oldDictArray.dictionary is guaranteed to return you the full, most up-to-date version of that Field's underlying dictionary.
Hope this helps, and let me know if we need to discuss further. I don't want the JS to deviate from the C++ behavior here, and am happy to make changes to align the features if necessary.

Ah, I just realized you were responding to my question about how to handle deltas. That's great, and sounds like we're already aligned on this :+1:

Can you help create follow up JIRA issues to actually implement this in each language?


feat: revamp store settings

What

store settings
adds a type property to the BodyCard component props type

Why

if the BodyCard is inside a form, then all events (buttons) will behave as type="submit" buttons. Instead, consumer of BodyCard should be able to alter this behavior as needed

I've opened a PR, that would allow you to put the <BodyCard /> inside <TwoSplitPane /> and achieve the same scrollable behavior when content expands the view height as seen on attached below:

Would love to hear what you think of that approach @zakariaelas 😊


Add content about default session data

Fixes #1082.

@NoraGDS @joelanman Hello!
Would love to get your input on this content update. I raised a (fairly meh) first draft a few weeks back, now I've had another go.
Is the example correct? Any other changes you'd make? :)

@joelanman Hi! 👋🏻
@NoraGDS and I have drafted the content update here. Can we get a review? :)

@EoinShaughnessy thanks for this! @NoraGDS and I will take this from here.
we should also review the content in the file itself:
https://github.com/alphagov/govuk-prototype-kit/blob/main/app/data/session-data-defaults.js


Not working properly with ocaml-4.08.1+rc3

After upgrade to ocaml-4.08.1+rc3 make test fails:
(cd _build/default/test && ../../install/default/bin/ocaml-mdx test empty_line.md)
ocaml-mdx-test: internal error, uncaught exception:
Failure("incomplete OCaml version")
Sys.ocaml_version;;

: string = "4.08.1+rc3"

Related to #144
@samoht what do you think of using https://github.com/dividat/ocaml-semver instead of parsing the ocaml version number ourselves?

Using a dedicated (and well tested library) is a good idea. In that case we should be using https://github.com/avsm/ocaml-version which is already used in production by the CI infrastructure.


net/http/httputil: improve compatibility with server sent events

According to https://html.spec.whatwg.org/multipage/server-sent-events.html#sse-processing-model
MIME type parameters should be ignored when Content-Typeheader value istext/event-stream`.
Related https://github.com/spring-projects/spring-framework/pull/24632

Gerrit isn't letting me comment on https://golang.org/cl/223118. ("Error 409: Change 223118 may not exceed 5000 updates.") I'm not sure why the change has 5000 updates on it, but I'm going to force abandon it since it's broken.
This change should use mime.ParseMediaType to parse the Content-Type header, rather than a prefix match.
I've created https://golang.org/cl/340529 with that change. If you'd like to make the change yourself, please feel free to create a new CL and send it to me for review and I'll abandon 340529.


Fix WEXT verification on some devices

On my rtl8814a adapter, iwr.u.name is "IEEE 802.11AC" (or disabling the 5ghz band will appear as "IEEE 802.11bgn"). Using IF_NAMESIZE, a maximum of 16 chars (null termination) is compared.

Merged, thanks.
Som Realtek drivers are really very special.


Setting Custom Headers for signoutRedirect

Hi there,
I have a question, I was hoping to set the Clear-Site-Data header when running UserManager.signoutRedirect.
I can see we can set custom query String parameters by passing an object to signoutRedirect({ foo: 'foo'})
Is it possible to set custom headers when calling signoutRedirect?
Thanks

No sorry, as the API redirects the browser to the end session endpoint.

Thanks!


Question: List all Datastore Clusters (StoragePod) available

How do I retrieve list of all Datastore Clusters which are attached to ALL hosts of a specific ESX cluster?
For instance, given an ESX Cluster Fooo (ClusterComputeResource), I want list all Datastore Clusters (StoragePod) that are connected to EVERY host in an ESX cluster.
My initial naive attempt was as follows:
			Map<String, ManagedEntity> storages = new LinkedHashMap<>();

			// Datastores Clusters
			ClusterComputeResource ccr = (ClusterComputeResource) cluster;
			Datastore[] datastores = ccr.getDatastores();
			for (Datastore datastore : datastores) {
			    if (datastore.getParent() instanceof StoragePod) {
			        StoragePod storagePod = (StoragePod) datastore.getParent();
			        storages.put(storagePod.getName(), storagePod);
			    }

            }

The problem with the above is that ClusterComputeResource.getDatastores() returns an aggregate of all datastores connected to the hosts in the cluster; but that doesn't necessarily mean that ALL hosts in the cluster have the datastore mounted.
Any ideas?

Hi @saamalik not sue how I missed this. What do you mean by  but that doesn't necessarily mean that ALL hosts in the cluster have the datastore mounted.

Hi @michaelrice,
For instance imagine the following:

Datastore Cluster A

Datastore 1

Datastore Cluster B

Datastore 2

ESX Cluster Foo

Host A (DS 1)
Host B (DS 1, DS 2)

If the user selects ESX Cluster Foo, then since both hosts A & B have DS 1 mounted, then Datastore Cluster A should be listed as an available DSC Cluster. However DSC B should not be listed since Host A does not mount DS 2.

Hi @saamalik,
I created a sample a couple of days ago that may help you here. It gets info about a given cluster using property collectors. You would need to expand upon it though. You would want to add properties like datastore to the HostSystem properties. Then you would want to add Datastore the way HostSystem and VirtualMachine have been aded and add properties too for the Datastore like name. Once you have that done it would be up to you to relate those objects it returns but it would give you what your after in a way that will scale no matter how large your inventory became.
I will admit it is going to seem like more work than what you've done so far but if your way did work on a small cluster it would start getting slow on very large inventories. Property Collectors are much harder but they are the only way to make the code fast and scaleable.

Thanks for the response @michaelrice. Ok, I'll give it a go. Thanks.

Im going to close this issue out since I havent heard anything else. If you need additional assistance please feel free to reopen.


support second mouse button interaction

Any updated on this? Or a workaround?

https://github.com/CartoDB/wax/issues/3

There has been no activity on this issue for several months and that's why are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


Cleanup all unused code and files

Pull Request check list

[ ] Proper tests/regressions included?
[ ] Documentation updated?

Affected functionality
N/A
Description of change
Removing code and files that we are not using in the PoC and that won't probably be used afterwards.
Which issue this PR fixes

Signed-off-by: Max Lambrecht<EMAIL_ADDRESS>

Any specific reason we are excluding the telemetry/prometheus folder?
In the MVP backlog we are tracking those metrics to be part of Galadriel.


Fix typo in typing example

@luxifer
I don't think it should be fixed. Please see the code comments.
It actually gives us a counter-example to illustrate type validation of CUE.
But still thank you !

Ah yes indeed I read it too fast. I'm gonna close it.
Le lun. 14 mars 2022 à 06:26, zhaoyang @.***> a écrit :

@luxifer https://github.com/luxifer
I don't think it should be fixed. Please see the code comments.
It actually gives us a counter-example to illustrate type validation of
CUE.
But still thank you !
[image: image]
https://user-images.githubusercontent.com/12080364/158110022-9f38c23a-4a64-4964-9154-4137cb60cdd7.png
—
Reply to this email directly, view it on GitHub
https://github.com/cue-lang/cuelang.org/pull/234#issuecomment-1066390461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADGKBTVGGNIABCVGLAKMVTU73EXPANCNFSM5PXYXWEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Florent Viel
*Software Engineer @ Cisco *.:|:.:|:.
@.**_vielblog.luxifer.fr


Bind options

Is it possible to bind an object of options?
E.g
<ng-quill-editor theme="bubble" ng-model="body" options="myOptions"></ng-quill-editor>

Ignore that, its called modules :)

what do you mean with options?
check readme:
https://github.com/KillerCodeMonkey/ng-quill#configuration
you can set modules configuration as object and formats


Flagger addon

Issue #, if available:
Description of changes:
working on flagger now and starting new pull request
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please remove the commits from the other PR, it creates a lot of noise and makes it hard to review. You can close the PR if needed and drop the branch, recreate and move the one file that you created back.

@Eli1123 do you mind addressing this ^^ one first? also merge from main.


fix cmake install directory (for real this time)

Rename 'develop' folder to 'nlohmann'
Use <nlohmann/*> headers in sources
Hack test/CMakeLists.txt to find the correct json.hpp
Change amalgamate config file

Should fix #928 and #942.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[x]  Changes are described in the pull request, or an existing issue is referenced.
[x]  The test suite compiles and runs without error.
[x]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x]  The source code is amalgamated; that is, after making changes to the sources in the develop directory, run make amalgamate to create the single-header file src/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.8 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

@patrikhuber I agree. src -> single_header/nlohmann for the sake of consistency.

It's not really necessary since we can still install single_header/json.hpp in a nlohmann directory.
But this might prevent drag and drop the json.hpp file, and encourage users to copy the enclosing nlohmann folder as well.
I'll push in a few minutes.

Here it is, I have installed with/without multiple headers.
I've patched test files to include <nlohmann/json.hpp> as well.
I haven't looked at the benchmarks folder, there might be some changes to make here as well. I don't know if this is mandatory for the release though.

It's not really necessary since we can still install single_header/json.hpp in a nlohmann directory.

But it prevents using the single header from a submodule in a consistent way with #include <nlohmann/json.hpp>. I mean - maybe that's not important - users who use the library from a submodule might as well use the non-single-header version under <nlohmann/json.hpp>.
I'll leave that up to you guys :-)

I finally went with your suggestion, it makes everything easier, even in the CMake :)

My 2 cents: I'm using json.hpp in my project (and I'd like to use json_fwd.h). I use git-submodules, but not for this library. For this library I'm downloading the .hpp from the release-site.
I wonder how many users are doing like I do, compared to users who do make install having this project as a submodules (or anywhere else) and are thus concerned about the forced directory-hierarchy - except those who develop this library. Sorry for my ignorance in advance ;-)

@pboettch So what do you propose?

What I tried to say is that maybe changing the paths inside this repo (and thus maybe breaking compatibility) is not a big deal, because only a few people are using it directly as compared to the multiple 10k (!) non-contributing/silent users who just use json.hpp as a copy. But this assumption might be completely wrong, I have no data.
In the intro-animation I read "What if JSON would be part of modern C++?" I hear would be part of standard C++ libraries. I'd expect to be able to do:
#include <json_fwd>

in header-files and in source-files
#include <json>

(similarly to iostream and iosfwd)
and this is exactly what I'm doing in my project(s) (*), I place the library's files so that exactly that works and I'm adding .hpp in my include statements.
(*) - I'm not yet using json_fwd.

I'm not sure following the C++ standard include model is a good idea.
The main point in having a top-level directory to include is to avoid clashes. This library is not part of the standard, we cannot assume that json.hpp is reserved to us.

@theodelrieu I see your point. A more generic approach then would be to prefix with nlohmann/. Then it would be just logic to have the same hierarchy of paths inside this repo and after a make install.
And that was exactly my original motivation when entering this discussion. I think there is no problem with moving files inside the repo to be coherent.


Update branding, links, and description in metainfo.xml

I am proposing the following changes to our metainfo.xml:

Change summary from "Offline education technology platform" to "Offline learning platform." Flathub suggests a 35 character limit for the summary text.
Add more URLs to complete the list at https://freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-url.
Replace the description text with the text from Kolibri's Play Store listing.
Change the dark mode branding colour to a deep blue, loosely derived from Kolibri's branding. Previously I was using a dark yellow, which appeared muddy against the icon. The light branding colour is unchanged. It's using a light yellow from Kolibri's colour palette.

Preview of the banners from the Flathub tool:

That description is the only thing keeping it from being featured on the homepage of Flathub, now. Looking forward to the next release!


libwebsockets: 4.3.2 -> 4.3.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/libwebsockets/versions.
meta.description for libwebsockets is: Light, portable C library for websockets
meta.homepage for libwebsockets is: https://libwebsockets.org/
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.3.3 with grep in /nix/store/zcankngm9gqbjb0llpkjavgfj3gzyxzw-libwebsockets-4.3.3
found 4.3.3 in filename of file in /nix/store/zcankngm9gqbjb0llpkjavgfj3gzyxzw-libwebsockets-4.3.3

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

16 total rebuild path(s)

16 package rebuild(s)

First fifty rebuilds by attrpath
ardour
ardour_7
collectd
domoticz
driftnet
ebusd
guacamole-server
janus-gateway
kismet
libwebsockets
mosquitto
owntracks-recorder
seafile-client
seafile-shared
ttyd
vhs

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/801b22a15181c2a14c2c6b2280c7c931c8134f4a#libwebsockets

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

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

16 packages built:

ardour
ardour_7
collectd
domoticz
driftnet
ebusd
guacamole-server
janus-gateway
kismet
libwebsockets
mosquitto
owntracks-recorder
seafile-client
seafile-shared
ttyd
vhs

Maintainer pings
cc @Mindavi for testing.

Upstream diff looks fairly safe, but I'll have to test this a bit soon-ish.


Static image wrongly mapped when update to new version in Android

Description
The images are showing up but they are wrongly map all over the screen [refer:image]. It occurs whenever user installed new update over the old version. Only happen in Android release build, totally fine in debug mode. Refer to screenshot
If user uninstall and reinstall, then the app appears fine. But it happen again when user install new update over it. I couldn't force user to uninstall and reinstall each time new update is released to store.
I'm using require(./image/xx.png) to pass in the <Image> source.
Was wondering is it something went wrong during assets bundling? I've been debugging this issue for whole day but to no avail, hence creating this issue.
Version
0.66.1
Output of npx react-native info
System:
    OS: macOS 11.6
    CPU: (8) x64 Apple M1
    Memory: 43.58 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 17.3.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.3.0 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
    Android SDK:
      API Levels: 26, 28, 29, 30, 31
      Build Tools: 28.0.3, 29.0.2, 30.0.2, 30.0.3
      System Images: android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
    Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_275 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.66.1 => 0.66.1
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

cd android && ./gradlew clean
./gradlew assembleRelease
Install new APK over existing version
Image show up wrongly

Snack, code example, screenshot, or link to a repository
Example usage:
const IMG_LOGO = require('../../../assets/image/auth/logo.png')

<Image source={IMG_LOGO} style={styles.imgLogo} resizeMode='contain'/>

请问你解决了吗  我也碰到了同样的问题， 男嗖！！

@zhoumi1324 I couldn't find a permanent fix, temporarily using uri to load image instead.
<Image source={{uri:'https://...'}} />

Same issue here

+1

Fixed using solution from #32671. Downgrade fresco version from 2.6.0 to 2.5.0
implementation 'com.facebook.fresco:animated-gif:2.5.0'

@anniewey can you please clarify where did you add that line exactly?
The link you shared they are using a different implementation, by having it in
configurations.all {
resolutionStrategy {
force "com.facebook.fresco:fresco:2.5.0"
}
}
and why animated-gif only?

@pierroo i'm using animated-gif only in my project for the gif support as documented here. that line is in android/app/build.gradle
dependencies {
    implementation 'com.facebook.fresco:animated-gif:2.5.0'
}

I am still running into this - tried adding all variants that I can find here configurations.all { resolutionStrategy { force "com.facebook.fresco:fresco:2.5.0" } }
and dependencies {     implementation 'com.facebook.fresco:animated-gif:2.5.0'  }
and
dependencies {     implementation 'com.facebook.fresco:fresco:2.5.0'  }
anybody another idea?

this is fixed on<EMAIL_ADDRESS>https://github.com/facebook/fresco/issues/2667#issuecomment-1476119354


Migrate e2e test for Cassandra to Go

Signed-off-by: aviadlevy<EMAIL_ADDRESS>
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added

Fixes #3231

/run-e2e cassandra*

So glad it passed. I only forgot to delete the old test

/run-e2e cassandra*

/run-e2e cassandra* Update: You can check the progress here

Seems like something was wrong with initiating the scaled object - https://github.com/kedacore/keda/runs/7144994425?check_suite_focus=true#step:6:220
Can you re-run the e2e?
P.S.
I noticed that the test cleanup is finished before all the resources are actually cleaned - https://github.com/kedacore/keda/runs/7144994425?check_suite_focus=true#step:6:541
With this behavior there is no point to retry. I opened an issue to fix that - https://github.com/kedacore/keda/issues/3322

/run-e2e cassandra*

/run-e2e cassandra*


Rely on conda "api" more directly in order to fetch repodata

Right now we have our own implementation for fetching repodata see https://github.com/Quansight/conda-store/blob/main/conda-store-server/conda_store_server/conda.py#L109-L161. While this does work it is not future proof. We should be using conda internally for this. While there is not an official "api" there are bits of conda which handle this logic for us.
We should be using SubdirData directly. In addition it has support for partial downloads now along with other features. https://github.com/conda/conda/blob/main/conda/core/subdir_data.py#L122

Edit, we have moved our conda utils to https://github.com/conda-incubator/conda-store/blob/main/conda-store-server/conda_store_server/_internal/conda_utils.py

FWIW, this is an example of how we use the SubdirData API to refresh the local cache for a channel and obtain the path to the JSON file:
https://github.com/conda/conda-libmamba-solver/blob/ea420d8c61c338a18613b874f59ebbb42cd741f5/conda_libmamba_solver/index.py#L248-L256
I think we could use this to:

periodically fetch the repodata in the background and have a 15min cache lifetime
run the solver with the local cache only (--use-cache or --offline both should work) to obtain the list of packages that need to be installed and generate a lockfile
download, extract and install the packages if needed


Upgraded Driver, FanSpeed error

Hello!
I have been using this for a while to give stats on GPU's.
Now, after some Nvidia upgrades, it seems the container complains, with the error: FanSpeed() error: nvml: Not Supported
I see in Grafana I get the other Data, but fan speed is no longer there.
Is that something fixable?

Maybe you're using a passively cooled GPU such as  T4? @ajcollett

that seems to happen for us too. Did you implement any fix?


Update README.md

typo in the git URL. 404 during running the commands

https://github.com/Azure-Samples/tailwind-traders is the accurate repo


Error Cannot use GraphQLSchema "[object GraphQLSchema]" from another module or realm.

I face this error. It seems due to the presence of extensions in import file names.
The library generating my schema uses ".mjs" files of graphql module, whereas graphql-query-complexity uses ".js" ones, and so graphql module believe that there are 2 different versions of it used.
For instance:
	import {
	  getArgumentValues,
	  getDirectiveValues,
	  getVariableValues,
	} from 'graphql/execution/values.js';

should be replaced with
	import {
	  getArgumentValues,
	  getDirectiveValues,
	  getVariableValues,
	} from 'graphql/execution/values';

I'm also facing the same issue. I'm guessing the problem is affecting all people that have their application built as esm (for example in my case, I'm using esbuild).
But I think it is a requirement for esm to specify the file extensions, so I think that's why it was there in the first place. So, what I ended up doing was creating an additional step after the build to replace graphql/execution/values.js with graphql/execution/values.mjs : https://github.com/slicknode/graphql-query-complexity/compare/master...andresusanto:graphql-query-complexity:master . I've published the fix here https://www.npmjs.com/package/@susanto/graphql-query-complexity in case you're interested @jer-sen
^ @slicknode is this something you would consider merging into your repo?

@andresusanto I don't see any issue to remove the file extensions, it's a serverside module https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import

@andresusanto I don't see any issue to remove the file extensions, it's a serverside module https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/import

I'm not quite sure what you meant by "serverside module", but perhaps check this one out?
https://nodejs.org/api/esm.html#mandatory-file-extensions

@andresusanto you are right. Relative and absolute specifiers must have a file extension (in the build directories) but this does not apply to bare specifiers. So graphql module should be imported without a file extension. Do you agree ?

@slicknode is this something you would consider merging into your repo?

@andresusanto it looks like this is a general issue with the ESM version of this library, so it is probably a good idea to fix this here. We just have to make sure we are not breaking things for particular graphql-js library versions. I think the 14.x versions didn't have .mjs exports, but since we dropped support for this it's probably safe to change it here.

thanks @ivome, I've confirmed that earlier v14 versions don't have the .mjs, but since 14.6 they all do, so I'll raise PR for this.

Any update on this issue ? I am facing the same problem with my graphql scalars. I tried to use your own version of the library @andresusanto but it still raises the same issue: Cannot use GraphQLNonNull \"ID!\" from another module or realm

Having the same issue.
Error: Cannot use GraphQLScalarType \"String\" from another module or realm.\n\nEnsure that there is only one instance of \"graphql\" in the node_modules\ndirectory. If different versions of \"graphql\" are the dependencies of other\nrelied on modules, use \"resolutions\" to ensure only one version is installed.\n\nhttps://yarnpkg.com/en/docs/selective-version-resolutions\n\nDuplicate \"graphql\" modules cannot be used at the same time since different\nversions may have different capabilities and behavior. The data from one\nversion used in the function from another could produce confusing and\nspurious results.
Anyone managed to solve this? I have tried the resolutions suggestion, ran yarn list --pattern graphql and made sure there is a single graphql version, still getting the error.

Can confirm have managed to solve the issue by removing .js from that import. We have initially manually done this within node_modules folder to test, then have automated that by adding graphql/execution/values.js': 'graphql/execution/values' to the alias webpack config property (we're using webpack)

hi @ivome , would you mind reviewing the PR above for the resolution? The idea is to keep it run without the extension so the user could config on their end to use their expected module type from both graphql-query-complexity and graphqljs


Debugging failure to render GeoJSON

I've got a very bizarre issue.  I am building a website off of MEAN.js (private repo currently, because of client) and I've added a number of modules for displaying data and graphs and such, and I'm getting into displaying maps now.
I've been trying to make ui-leaflet work with the GeoJSON data and it is failing to render the polygons in the browser, and does not generate an error on the browser's console.  I can look at the scopes using Batarang and see that they're correctly populated, but they just don't show.
When I make an example using a pristine instance of MEAN.js the polygons do render, so I'm unable to duplicate the problem.  I've added the additional JS libraries that are called in other parts of the project to my pristine MEAN.js instance and that seems to have no impact.  I've overwrittent the module in my problem project with the module from my pristine project and there is still no impact.
I'm out of ideas for how to find out what is causing this issue.  Here's my example where I fail to reproduce the issue: https://github.com/jjcad/leaflet_meanjs_example
I'm open to any and all ideas about how to solve this.

When you inspect the map with your browser dev tools, can you see the DOM elements for the polygons? They should be in an <svg> element with the class leaflet-zoom-animated.
Here's a screenshot from my project - we have a lot of GeoJSON polygons, so there's a lot of <g>...</g> elements in the SVG, one per each polygon:

In my linked example project & my problem project (which have identical map modules at the moment) I've downgraded to trying to show just one polygon (a square). In both of them, it is adding the <svg> and <g> elements.  I tried adding a screenshot from my problem project, but github is being disagreeable about that matter.  I'll mess with it a little more and see if I can add one or link to one.

I am loading d3 and nvd3 for another module in the project.  Is it possible that those are creating a conflict?  If so, I'm still not sure why my example would show the polygon, but my problem project would not, since they're both loading d3 and nvd3 right now.  The difference is that nowhere in my example project are d3 and nvd3 actually used...

I have used d3 in the past at it has some interesting low level issues if you muck with the date object. Your not by anychance using date.js? Just double checking as it will make d3 not work if your using a bad version of date..js
https://github.com/mbostock/d3/issues/1302

As for why polygons are not showing; I really need a reproducible example.

No date.js being used. I'm really wishing I could make a reproducible example, but all my attempts at making one that fails to render polygons have been thus far foiled. It seems like it only fails to work in my actual project, even when I import working code from a working example into my actual project.

I did not mean to close that. I'm still going to keep trying to make a reproducible example and some screenshots.

Alright, here's an example.
Also, I'm so sorry about the open/close thing. I keep hitting the wrong button when trying to post.

Is there a way that I could figure out if d3 is maybe hijacking the <svg> element from leaflet?

Sorry I can not derive anything from your image. Please make a plnkr or github repo that shows shows the problem. If we need to get into your server we can do that but that will need to be taken offline via email.

Believe me, I would love to reproduce the problem. Every reproduction I make fails to make the problem occur though.

I'm guessing one of your extra libraries is causing a conflict. Maybe in your reproductions, you're not using the same versions of each library? (e.g. maybe your reproduction uses D3 version 3.5.11 but the problem is with D3 version 3.5.10)
In any case, this is next to impossible for us to help you troubleshoot without being able to see the problematic code. Keep trying to make a 'working' (non-working) version that successfully reproduces the issue, paying close attention to each additional library and its version number, as well as the order in which each library is loaded.

@seadour Thanks for the good advice. I'll work on building that out.

After several more attempts at building a non-working example, I still can't reproduce this error. Closing issue as a result.


[Swif 6.0] Mark Bundle as Sendable

Cherry-pick of https://github.com/apple/swift-corelibs-foundation/pull/4963 for Swift 6.

@swift-ci test


Git: commit-graph and multi-pack-index builtins always show progress

The commit-graph and multi-pack-index builtins show progress during their output. There is no way to turn it off right now. This means that when we have an error, stderr is filled with progress indicators and the actual error message may be truncated in our logging.
Update git to allow --no-progress and use this argument in the background operations.

Also fix the percentage progress for the "Expanding reachable commits" which makes no sense.

Use the --[no-]progress trick similar to git fetch here: https://github.com/git/git/blob/master/builtin/fetch.c#L151


bump k8s-openapi 0.13.0 and revert option unpacking - fixes #581

reverts unpacking from #531 while bumping k8s-openapi for https://github.com/Arnavion/k8s-openapi/releases/tag/v0.13.0

Cargo-deny on tar:

Solution: No safe upgrade is available! :ok_man:

I'll open a PR to fix the CI (cargo deny). That was my fault. An advisory shouldn't block a PR.

@lfrancke It's not actually blocking it, and we're not using it as a required status, so it's OK. It's just a bit annoying that advisories can be released without fixes. We got an issue about the same thing which was reasonable.

Okay. In theory I thought github actions is set up to ignore errors from the advisories run but for some reason it doesn't.
We just hit the same today and fixed it like this.: https://github.com/stackabletech/agent-integration-tests/pull/53/files
I don't understand why the original/current version doesn't work but I'm happy to create a PR if you like.

Yeah, very happy to take a PR for only doing the license ban on PR if you have time. The red cross will show up in places and will be a bit annoying. The issues for advisories are probably sufficient.


scripts/generate/genesis.sh:  some more order in the business

This allows better automation in iohk-ops.  Been tested in the last week's three deployments.

This also depends on DEVOPS-255 (for example there is no more qanet)

Plus regenerated genesises on top of current master

https://hydra.iohk.io/eval/62859


Write manual test cases for "AustraliaPage: https://prayersconnect.com/regions/provinces?id=43&cid=3" and "NewZealandPage: https://prayersconnect.com/regions/provinces?id=51&cid=3".

Please write a manual test case for "AustraliaPage: https://prayersconnect.com/regions/provinces?id=43&cid=3" and "NewZealandPage: https://prayersconnect.com/regions/provinces?id=51&cid=3" which involves breaking down the functionality of the webpage into specific steps that can be tested.
Please use this Google Sheet for your test cases
https://docs.google.com/spreadsheets/d/1TPTDXvuPvRhw-U4YBsxJ58iATH1P4MCsE98E3ydKECQ/edit?usp=sharing

completed
https://docs.google.com/spreadsheets/d/1TPTDXvuPvRhw-U4YBsxJ58iATH1P4MCsE98E3ydKECQ/edit#gid=1628079725

Thank you.


Returns duplicate tweets

twint is returning duplicate records.
Eg: When I scrape all tweets for "nbcnews" with default config parameters, it returned total of 142997 records out of which only 41 were unique tweets.
twint was installed using pip3 install --user --upgrade git+https://github.com/yunusemrecatalcam/twint.git@twitter_legacy2
Below is the code used to get tweets from nbcnews:
c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
twint.run.Search(c)
tlist = c.search_tweet_list

The problem exists even with c.Profile_full = True
Can someone help me with solving the duplicate tweet problem?

c.Limit = 1
And dont forget
c.Since = startdate
c.Until = enddate

c.Limit = 1
And dont forget
c.Since = startdate
c.Until = enddate

This return 20 unique tweets by default. Is there any parameter to increase the number of unique tweets?

c.Limit = 1
And dont forget
c.Since = startdate
c.Until = enddate

As mentioned, c.Limit = 1 would return 20 unique. But, we could use "Since" and "Until" differently.
One "not so efficient" way of getting more tweets is to iterate over the date and get the tweet for one day at a time.
The following returned only 33 unique tweets for 9 days:
c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
c.Limit = 1000
c.Since = "2015-12-01"
c.Until = "2015-12-10"
twint.run.Search(c)
tlist = c.search_tweet_list

Where as the below code that that gets tweet for same date range returned 266 unique tweets:
import twint
from datetime import datetime
from datetime import timedelta

c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
c.Limit = 50

def get_tweet(config):
    twint.run.Search(config)
    tlist = config.search_tweet_list

    return tlist

start_date = "2015-12-01"
stop_date = "2015-12-10"

start = datetime.strptime(start_date, "%Y-%m-%d")
stop = datetime.strptime(stop_date, "%Y-%m-%d")

output_tweets = []
while start < stop:
    next_day = start + timedelta(days=1)

    c.Since = str(start)
    c.Until = str(next_day)
    start = next_day

    output_tweets+=get_tweet(c)

c.Limit = 1
And dont forget
c.Since = startdate
c.Until = enddate

As mentioned above, c.Limit = 1 would only return 20 unique. But, we could use "Since" and "Until" differently.
One "not so efficient" way of getting more tweets is to iterate over the date and get the tweet for one day at a time.
Example
The following returned only 33 unique tweets for 9 days:
c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
c.Limit = 1000
c.Since = "2015-12-01"
c.Until = "2015-12-10"
twint.run.Search(c)
tlist = c.search_tweet_list

Where as the below code returns 266 unique tweets for same date range:
import twint
from datetime import datetime
from datetime import timedelta

c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
c.Limit = 50

def get_tweet(config):
    twint.run.Search(config)
    tlist = config.search_tweet_list

    return tlist

start_date = "2015-12-01"
stop_date = "2015-12-10"

start = datetime.strptime(start_date, "%Y-%m-%d")
stop = datetime.strptime(stop_date, "%Y-%m-%d")

output_tweets = []
while start < stop:
    next_day = start + timedelta(days=1)

    c.Since = str(start)
    c.Until = str(next_day)
    start = next_day

    output_tweets+=get_tweet(c)

Thanks for your post. I fixed it from this perspective

@HegdeChaitra this has been solved in #944.
Please close this issue.

I am using a terminal to execute twint. I have the same problem and I tried to use 'since' and 'until' but it does not work for me.

Also, if I use the method you have given above. I get an error like this. Please help me!
AttributeError: partially initialized module 'logging' has no attribute 'getLogger' (most likely due to a circular import)

Also, if I use the method you have given above. I get an error like this. Please help me!
AttributeError: partially initialized module 'logging' has no attribute 'getLogger' (most likely due to a circular import)

Can you give details on how you have installed it and what exactly is the code you are running?

I install twint by using git
git clone --depth=1 https://github.com/twintproject/twint.git cd twint pip3 install . -r requirements.txt
To be safe, after I installed it, I do it again by using pip
pip3 install twint
After that, I open a python file in \twint\twint, and run the code you have given.
`import twint
from datetime import datetime
from datetime import timedelta
c = twint.Config()
c.Search = "from:@nbcnews"
c.Store_object = True
c.Limit = 50
def get_tweet(config):
twint.run.Search(config)
tlist = config.search_tweet_list
return tlist

start_date = "2015-12-01"
stop_date = "2015-12-10"
start = datetime.strptime(start_date, "%Y-%m-%d")
stop = datetime.strptime(stop_date, "%Y-%m-%d")
output_tweets = []
while start < stop:
next_day = start + timedelta(days=1)
c.Since = str(start)
c.Until = str(next_day)
start = next_day

output_tweets+=get_tweet(c)`

As soon as I run the code, the error occured.
AttributeError: partially initialized module 'logging' has no attribute 'getLogger' (most likely due to a circular import)

After that, I open a python file in \twint\twint, and run the code you have given.

Seems to me that this might me the reason you are getting the circular import error.
Try to create the .py file outside the twint package directory.
Once you have installed twint you can access it from anywhere on your system so there's no need to stay in the twint package directory.
Also first try to uninstall twint then do a fresh install using
pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint
After that simply run the following line in your terminal and see if that works
twint -u realDonaldTrump

@HegdeChaitra @yangyangdotcom
One more thing,
There is no c.search_tweet_list in the master branch of twint.
The code you are using is outdated as it was for the proposed Pull Request #917 which wasn't actually merged into the master because it was returning incomplete tweets at that moment.

After that, I open a python file in \twint\twint, and run the code you have given.

Seems to me that this might me the reason you are getting the circular import error.
Try to create the .py file outside the twint package directory.
Once you have installed twint you can access it from anywhere on your system so there's no need to stay in the twint package directory.
Also first try to uninstall twint then do a fresh install using
pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint
After that simply run the following line in your terminal and see if that works
twint -u realDonaldTrump

I followed your instruction but it does not work for me. It does scrape some tweet of Donald Trump succesfully. But the following error occured halfway through the process and twint stoped scrapping.
CRITICAL:root:twint.run:Twint:Feed:noData'tweet'
sleeping for 15 secs

I also try to execute it using IDE, no tweets were scraped and following error occured.
CRITICAL:root:twint.run:Twint:Feed:noDataExpecting value: line 1 column 1 (char 0)
sleeping for 1.0 secs

Does these happened to me only?

After that, I open a python file in \twint\twint, and run the code you have given.

Seems to me that this might me the reason you are getting the circular import error.
Try to create the .py file outside the twint package directory.
Once you have installed twint you can access it from anywhere on your system so there's no need to stay in the twint package directory.
Also first try to uninstall twint then do a fresh install using
pip3 install --user --upgrade git+https://github.com/twintproject/twint.git@origin/master#egg=twint
After that simply run the following line in your terminal and see if that works
twint -u realDonaldTrump

I followed your instruction but it does not work for me. It does scrape some tweet of Donald Trump succesfully. But the following error occured halfway through the process and twint stoped scrapping.
CRITICAL:root:twint.run:Twint:Feed:noData'tweet'
sleeping for 15 secs

I also try to execute it using IDE, no tweets were scraped and following error occured.
CRITICAL:root:twint.run:Twint:Feed:noDataExpecting value: line 1 column 1 (char 0)
sleeping for 1.0 secs

Does these happened to me only?

As mentioned in the first commend on this issue, above method works when you install the twint using pip3 install --user --upgrade git+https://github.com/yunusemrecatalcam/twint.git@twitter_legacy2


[Revise subclass] Alzheimer disease 2 (MONDO_0007089) no early-onset AD

Mondo term (ID and Label)
Alzheimer disease 2 (MONDO_0007089)
Suggested revision and reasons
This term is currently a child of early-onset autosomal dominant Alzheimer disease (MONDO_0015140). However, according to the information in the page of the related OMIM term OMIM_104310, this is a late-onset alzheimer's disease. Therefore, it seems that it should be moved under that subtype of AD (like EFO_1001870 in EFO) or the general alzheimer's disease term (MONDO_0004975).
cc @zoependlington

Thank you @nicolevasilevsky


Store prompt answer

Store all prompt answers in Yeoman config.
So other author can reuse the field.

This won't really allow you to read what the answer to the question was from outside generator-license.
Instead, you should read the value inside package.json

I though I can get the .yo-rc.json.
Only realise it won't write only it finish.
Scrap that.
Thank for reply, guess have to write basic generator class and extend it instead.

@kievechua why not read it from package.json? This generator will updated the field with the correct value.

@SBoudrias That feature is what I'm trying to do, should read through the code.
Basically building first generator to scaffold an empty project (no package.json), so come upon this project. Then though trying to be clever so user don't need to type twice for name, email and etc.
I think my own code override the package.json created, that's why I didn't see it.
Still learning the best practise and how other build it.
thanks again.

@kievechua you can ask these questions in your generator, then pass them as option to generator-license.

Yes, planning to do that.
Just searching around to see if anyone have a GeneratorStarter asking those question.
So I can just extend it instead.

@kievechua you can have a look at https://github.com/ek9/generator-licensor as an example. It uses this and generator-license-cc to create licenses. Perhaps that will help you writing your own generator.


Update help feature in UG

Codecov Report

Merging #126 (708fd4f) into master (6d0235f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #126   +/-   ##
=========================================
  Coverage     63.58%   63.58%
  Complexity      497      497
=========================================
  Files            90       90
  Lines          1711     1711
  Branches        186      186
=========================================
  Hits           1088     1088
  Misses          568      568
  Partials         55       55

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

LGTM


Improve error message when provisioning local kubeflow resources with a non-local container registry.

Describe changes
Improve error message when provisioning local kubeflow resources with a non-local container registry.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table.
[ ] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

I implemented the reverse order essentially: Instead of telling them that zenml stack up isn't possible with non-local components, the error message tells users to change to a local container registry if they want to provision the local kubeflow deployment (the kubeflow orchestrator itself is agnostic to local/remote settings).

@strickvl Is this good to merge in your opinion?


Make enabling the wire-spy package environment aware through an env variable

As with automated testing you don't want the wire-spy to be active. But there could also be other situations where you don't want the wire-spy to be active. With this adjustment, you can set an env variable to switch off wire-spy

Thanks @ErikDohmen 🙌


Customization on bootstrap-selectsplitter

This is exactly what I want.
My data is two level select that is exactly this component does.
But, It looks so simple and can't be applied any style to it.
It always appears in two vertical boxes, not in two columns. Even, I added either col-md-6 or col-lg-6 in javascript file.
Beside, I found that this plug-in http://rickardn.github.io/listgroup.js/ would make it looked better.
So I add class='list-group' inside your javascript.
Unfortunately, it's still not working.
I would appreciate if you kindly provide more way to customize your wonderful component here. :+1:
Cheers.

Hello,
I doubt I can improve the styling of  tags cross browser without heavy changes, so I prefer to keep the default behavior (Look at bootstrap  --> they are also very lightly styled).
Unless I missed something, the plugin that you linked does not provide the ability to scroll down ; which is necessary and essential for selectsplitter :-).
For your problem for the two vertical boxes could you provide me a jsfiddle so that I see the problem?
Kind regards


add rocky 9.1 repos

I have no idea why this test broke, both short names exist.
- name: Assert that RPM package repository list can be filtered to multiple
assert:
that:
- rpm_package_repos_filtered | length == 2
- rpm_package_repos_filtered[0].short_name == 'centos_stream_8_baseos'
- rpm_package_repos_filtered[1].short_name == 'docker'
vars:
rpm_package_repo_filter: docker centos_stream_8_baseos

I have no idea why this test broke, both short names exist. - name: Assert that RPM package repository list can be filtered to multiple assert: that: - rpm_package_repos_filtered | length == 2 - rpm_package_repos_filtered[0].short_name == 'centos_stream_8_baseos' - rpm_package_repos_filtered[1].short_name == 'docker' vars: rpm_package_repo_filter: docker centos_stream_8_baseos

It's using multiple filters to match the repos:
rpm_package_repo_filter: docker centos_stream_8_baseos

New docker repo means there are now 3 matches. Please add the new one to the assert.


Dijkstra's Algorithm

Overview

Write code for Dijkstra's Algorithm  in C++.
Add that code into a directory named "Graphs", if the directory doesn't exist, create one and add the code file.

Please take care of these things while creating the pull request.

The code file was added to the correct directory.
Code quality looks good.
The programming language used is the same as mentioned in the issue.

I can contribute here. consider assigning me to this issue.  Thanks. @vashuteotia123

Hello, I'm a participant in Hacktoberfest 2022. I want to contribute to this issue.
Thank you,
@vashuteotia123

Hello, I'm a participant in Hacktoberfest 2022. I want to contribute to this issue.
Thank you,
@vashuteotia123

This issue is already been assigned.


How is using this any different than the default behavior of flutter?

I'm just not clear on how this is different than what I achieve by using the default flutter widgets which then recompose themselves as either android or ios widgets. I know that in some cases I may want to compose completely different UI or at times different behaviors for the varying platforms. Also, how will this impact projects that support one of the desktop modes (macos, windows, linux) or the web?
Thanks in advance for the clarification.

Flutter provides either a material themed library or a cupertino themed library which a developer has to make the choice on what widget to use. There are only a couple of widgets that are adaptive to the target platform but the rest don't render that way..
If you want pretty much the same layout and pages between your Android material app and your Cupertino iOS app then this library may help reduce the amount of conditional logic in your code to display the correct platform themed widget,
As for Web and desktop, currently if you target android, web, windows or linux it will use the material theme and iOS or macos it will use the cupertino theme.
If your want to build a custom themed app that is the same look between platforms then this library is not the best choice


Report Errors During Conversion

From the ReadMe

Report possible errors during conversion (eg. referencing the variables doc or op in global functions without previously declaring them)

They are listed as "ideas for the future" for a reason, as they're not trivial to implement.


Problem with the session_name

The session name is obtained from configuration, but later it is used to combine it with a hash of the path where grav is installed. This causes that when deploy the code to the server if we move the path a new session name is generated, and lost all the sessions.
$session_name = $inflector->hyphenize($config->get('system.session.name', 'grav_site')) . '-' . substr(md5(GRAV_ROOT), 0, 7);
I think if you need another name, you have the option for change it, but generate magically modifications on explicit configuration is very wrong.

What is the use case for sessions persisting across machines where the filesystem layout is different?  If you had a cluster would your filesystem not be the same, and therefore the md5 of GRAV_ROOT be identical?

Hi @rhukster,
For example when use a deploy release strategy, with capistrano or similar.
More information: https://capistranorb.com/documentation/getting-started/structure/
What is the case for which it was thought that it was necessary to reinvent the configured session name when changing the directory that can not be resolved by simply changing the configuration?

The use case is a very simple one.  Having multiple sites on one server in multiple sub-directories.  This is an extremely common use case for Grav.
What we really need is a way to uniquely identify an installation, and I have an idea, so let me try it out and let you know.

Ok, I've made a commit in the 1.6 branch of Grav that uses an md5 of the security salt (which is unique per installation).  It will change if you delete your user/config/security.yaml file of course, but that's not a common thing to do.

Updated with this commit: https://github.com/getgrav/grav/commit/3c2a8510f91e422a264856dcea4bac4443e15dd0
Now it defaults to PATH (so you don't get logged out during admin update!), but use the new system.session.uniqueness option to pick salt rather than path if you wish.

mmm.... I like it!


Error upon saving changes to project data

The application is generating an error message nearly every time a change is made to project data.  However, the data appears to be saved, nevertheless.
Reproduce:

Navigate to a project in the app

Click edit

Change a field (e.g., change "The project must be maintained" to "unmet")

Click the Save and Exit button

Error message is generated

Click the back arrow on the browser to return to the form

Another message is displayed that says that another user made a change

Scroll to the field that was changed above and confirm that the change is still there

I've never heard of this problem before!
The "other user" triggers when different people edit the same project at the same time - did that happen here?
Can you give me the time this happened (as accurately as you can) and the project number? If there's anything else you remember about the incident, please let me know that too. We'd like to track this down & fix it.

By the way, I've been using the badging app for many years and this is the first time I've seen this issue.  So, that might imply that the bug is the result of a change to the app in say, the past year.

Thanks so much for reporting this. That's obviously a bad thing.
@andrewfader - I may need your help on this one. Any insights you have would be really appreciated.

The other thing to note is that, despite both messages, the change is successfully made.  I never saw an instance where the change to the field did not happen.

Note: David McBride was specifically making a change to the maintained_status and maintained_justification fields. I'm not if that makes any difference.

I will take a look at it

The cause of this issue is our email system @david-a-wheeler
2023-06-20T22:15:58.611659+00:00 app[web.1]: method=GET path=/zh-CN/projects/5321 format=html controller=ProjectsController action=show status=200 duration=32.83 view=23.18 db=5.91
2023-06-20T22:15:59.184876+00:00 app[web.1]: Delivered mail<EMAIL_ADDRESS>(705.9ms)
2023-06-20T22:15:59.185186+00:00 app[web.1]: method=POST path=/en/projects format=html controller=ProjectsController action=create status=500 duration=2381.81 view=0.00 db=18.25 uid=27615
2023-06-20T22:15:59.185768+00:00 app[web.1]:
2023-06-20T22:15:59.185770+00:00 app[web.1]: Net::SMTPFatalError (550 The from address does not match a verified Sender Identity. Mail cannot be sent until this error is resolved. Visit https://sendgrid.com/docs/for-developers/sending-email/sender-identity/ to see the Sender Identity requirements
2023-06-20T22:15:59.185771+00:00 app[web.1]: ):
2023-06-20T22:15:59.185771+00:00 app[web.1]:
2023-06-20T22:15:59.185771+00:00 app[web.1]: app/models/project.rb:347:in `send_new_project_email'
2023-06-20T22:15:59.185772+00:00 app[web.1]: app/controllers/projects_controller.rb:250:in `block in create'
2023-06-20T22:15:59.185772+00:00 app[web.1]: app/controllers/projects_controller.rb:248:in `create'
2023-06-20T22:15:59.185772+00:00 app[web.1]: config/initializers/canonical_trailing_slash.rb:30:in `call'
2023-06-20T22:15:59.192929+00:00 heroku[router]: at=info method=POST path="/en/projects" host=bestpractices.coreinfrastructure.org request_id=46937a72-1af2-41e1-8b9f-4062fc38eaf2 fwd="2600:4041:5267:cb00:958b:b5ae:f730:5d4f, 2600:4041:5267:cb00:958b:b5ae:f730:5d4f,<IP_ADDRESS>" dyno=web.1 connect=0ms service=2398ms status=500 bytes=2529 protocol=https

@david-a-wheeler I triggered a verification email to<EMAIL_ADDRESS>and you'll need to approve it to proceed with the fix


[Feature Request] Support user based beta projects

Is your feature request related to a problem? Please describe.
I want to be able to automate beta projects owned by users
Describe the solution you'd like
Add a user input field
Describe alternatives you've considered
Tried other actions, they don't support beta projects
Additional context
Add any other context or screenshots about the feature request here.

Thank you for your feature request. I'll have a look at this in the next few days.

I can confirm that it is not possible to hand over the user as an organization for the automation.
The workflow responds with:
{{{
PROJECT_ID: null
ITEM_ID:
STATUS_FIELD_ID:
status_value_OPTION_ID:
gh: Could not resolve to a node with the global id of 'null'
{data:{updateProjectNextItemField:null},errors:[{type:NOT_FOUND,path:[updateProjectNextItemField],locations:[{line:8,column:9}],message:Could not resolve to a node with the global id of 'null'}]}
}}}
Even if the values are set correctly:
{{{
Run<EMAIL_ADDRESS>with:
gh_token: ***
organization: leonsteinhaeuser
project_id: ***
resource_node_id: I_kwDOGWypss4-jzii
status_value: Todo
}}}

I've finished the first draft for this feature. It worked quite well in my tests. I would like to ask you to take a look at the README file and give me your opinion

Works great, thanks!


Make examples runnable

Add main method to both examples.
Provide package defining executables for each example.
Update examples to use SwiftParser.

The medium-term plan was for the examples to be migrated to Swift 5.7 snippets, which would let users run the examples using swift run <snippet> without the need for main methods or full-blown targets. Essentially, this 'migration' only involves renaming the folder from Examples to Snippets to have SPM pick up on these scripts:

#531

This was already implemented in #477, but reverted due to an Xcode issue, and therefore currently put on hold.
cc @ahoppen

Thanks for the idea but I agree with @fwcd that we should be using SwiftPM’s snippets to make sure the examples compile one they are available in Xcode. That way we don’t spam the targets in Package.swift. Also, I think the primary used case for these example should be to show how the API can be used and not a set of scripts should be easy to run.

I accept the decision to keep everything as it is, but I also have some thoughts I would like to share:

In their current state, the examples do not work anymore. Some outdated APIs are used.
Even though the intention was not to provide small scripts, the AddOneToIntegerLiterals example parses command line arguments.
It is natural for a user discovering the project to run the examples and experiment with them. Quite some manual work is required (basically writing the Package.swift) to do so.
SwiftPM's snippets might be perfectly suitable. As long as their application here is on hold, there is no benefit of being a perfect fit. Better have something easily usable than waiting for the perfect solution.
The main Package.swift is not touched at all. There is a new manifest solely for the examples that only references the main manifest.

I think the central idea is that these scripts are really intended to be more of a documentation on how to quickly get started using the API rather than standalone programs. So ideally they'd have as little noise as possible around them (main methods, argument parsing, ...).
I definitely agree that being able to run them directly can be very useful and from what I understand we also do want to have them as part of the build pipeline, e.g. to make sure that they stay in sync with the (rapidly evolving) API.
Snippets let us do all of this without the added complexity of another package layer. For CodeGeneration an extra package was necessary to address specific problems with bootstrapping, but here I believe we should keep it simple and hopefully get the Xcode issues sorted out soon.

Regarding the outdated APIs: Fixes are definitely welcomed there, I think especially the Format part has undergone some changes recently, but I haven't tracked it super closely, so @ahoppen probably has a better idea of how the API is to be used idiomatically.

Oh, sorry, I somehow missed that you also adjusted these scripts, I inferred too much from the PR title, sorry.

The change to use SwiftParser and the typo fixes are definitely good changes that I would like to merge.
IMHO the change to use @main adds noise to these examples that draws attention away from the important bits

Created #929 to address the accepted changes. I'm still not happy with the examples not being easily executable. Hope the issue Xcode is having with the Snippets will be resolved shortly.
On the other hand, why not just name the folder back to Snippets? That would make them runnable at least with SwiftPM. Or is a Snippets folder already causing problems in Xcode?

Created #929 to address the accepted changes. I'm still not happy with the examples not being easily executable. Hope the issue Xcode is having with the Snippets will be resolved shortly.

What would your use case of having these example being runnable be? I can’t really see any but I might be missing something. What I would like to avoid is that these examples start becoming a grab-bag of scripts based on SwiftSyntax that people rely on in production use cases.

On the other hand, why not just name the folder back to Snippets? That would make them runnable at least with SwiftPM. Or is a Snippets folder already causing problems in Xcode? This comment assumes the rename happened inadvertently. Did it?

Yes, exactly, if you rename the folder to Snippets, building the package in Xcode fails with
Expressions are not allowed at the top level

in the snippets

What would your use case of having these example being runnable be? I can’t really see any but I might be missing something. What I would like to avoid is that these examples start becoming a grab-bag of scripts based on SwiftSyntax that people rely on in production use cases.

I see two: First off, you can easily run them to verify they still compile and produce reasonable results, in other words, they are not outdated.
Secondly, people discovering the project can readily use them to get an idea on how the stuff works. They may change their content experimenting a bit with the API. I expect most of the people to write their own Package.swift to do so if they don't give up right away. You might take a guess what I've done when preparing and verifying #929. 😉

Thinking over it for a few days, maybe I was wrong to close this. I think we should move to snippets as soon as they are available but in the meantime a separate package is probably right after all.
If you want to re-open this, could you modify build-script.py to build the examples package? That way CI would ensure that the examples always build. I hope that the changes in this repo should be fairly straight-forward. You will need to also add the package to contents.xcworkspacedata in the apple/swift repo. Note that this file is not really an Xcode workspace, it’s just a file that specifies the SwiftPM packages that should get built in CI (it’s for the --multiroot-data-file option to make sure we don’t need to build a package twice if two packages depend on it).

Sorry if my comments came across a bit harsh.
As long as we keep the examples simple scripts, having a package manifest for them probably doesn't hurt, that's true. Ultimately making them snippets would be nice, but being able to run them now benefits everyone and also makes it easier to test against regressions.

No worries! Happy I was able to convince you in the end. 😉
I've updated my branch incorporating the newly added example and rebasing it onto main. Could you please re-open the PR? I'm not able to do that.
I guess, the changes in apple/swift can be done as soon as this PR is merged?

I've updated my branch incorporating the newly added example, letting the build script know about the examples and rebasing it onto main. Could you please re-open the PR? I'm not able to do that.

I can’t re-open it either. It says ”The runnable-examples branch was force-pushed or recreated.”. Could you just open a new PR?

I guess, the changes in apple/swift can be done as soon as this PR is merged?

Sure.

I can’t re-open it either. It says ”The runnable-examples branch was force-pushed or recreated.”. Could you just open a new PR?

Ah, I should not have updated the branch. Created #1007.

You could have force-pushed the old commit, reopened and then force-pushed back the new commit :D But yeah, a new PR works too


CommonJS support added

UMD support added to be compatible with Browserify (and others), like they recommend here for JQuery plugins. Maybe it would be good to add something like this to the documentation.
This library is CommonJS compatible, so you can use it in this way:
var jquery = require('jquery'),
    moment = require('moment'),
    fullcalendar = require('fullcalendar')(jquery, moment);

    jquery('#calendar').fullCalendar({
          weekends: false // will hide Saturdays and Sundays
    });
    ...

Really this PR is not needed, but I think you need to generate the distribution file again. Differences:
intro.js
(function(factory) {
	if (typeof define === 'function' && define.amd) {
		define([ 'jquery', 'moment' ], factory);
	}
	else if (typeof exports === 'object') { // Node/CommonJS
		module.exports = factory(require('jquery'), require('moment'));
	}
	else {
		factory(jQuery, moment);
	}
})(function($, moment) {

dist/fullcalendar.js
(function(factory) {
	if (typeof define === 'function' && define.amd) {
		define([ 'jquery', 'moment' ], factory);
	}
	else {
		factory(jQuery, moment);
	}
})(function($, moment) {

Thanks for the PR. The dist files will be re-generated when the next version is released. I will also submit this to the NPM registry after that happens.

Nice :), thank you


Feature Request - IPC namespaces

I would like to run a Mininet simulation that allows for IPC namespaces.
I need this to be able to run a simulation that has multiple Quagga routers with different configuration files.
The current behavior results in vtysh on all routers connecting to and reading the config file for the last router that was started by the network.

Hi! Its possible to run multiple Quagga instances with different configuration files. All you need is:
privateDirs = [ '/var/log', '/var/run', ('/etc/quagga', 'configs/%(name)s/') ]
    host = partial(Host, privateDirs=privateDirs)
    net = Mininet(host=host)

This will make private folders for each mininet host instance, with the config form 'configs/%(name)s' directory. In my case (a config folder with hostname subfolders with config files in the same folder) this configs folder is in the same folder with the mininet script.


Problems in inspect_model.ipynb

I get my jupyter restart when i excute '  _ = model((batch_imgs, batch_metas), training=False)  ',after dataset loaded. If i don't excute this line  ,i got a error  in load weights procedure. The sreenshots of these problems are as belows.

Hi @kuailedagongzai
This is how eager mode works in TensorFlow2.
If you don't run the network at first, this model wouldn't know the structure of network, causing this "a model with 0 layers" problem.

Thanks for advices,but everytime i try to feed the data into the model,my jupyter will restart.How could i avoid this situation?By the way, I tried this scripts in vscode ,then my vscode restart. I stored the data in anothor directory , so i made a little modification to coco.py .Does this matter ?These are my modifications.

Maybe you could check out your GPU memory? One reason to cause this crash is OOM.
You can try this scripts in Terminal and figure it out~

Actually,the GPU in my notebook could not be used. Therefore , i use CPU only. There are 12G memory in my systerm.And when i run this script in the terminal ,it report a 'Segmentation fault'

It can't work in my CPU either.
Maybe it's a bug?

I fix this bug in this commit
Now the weight initialization method is Xavier(default) instead of He


Make evox pluggable

Create a way to add modules to evox externally

I’m writing this from my phone while at a family event so this is probably very scattered, but my initial thought on how to do this is to have an evox∷init function that loops through a vector of function pointers, and have a register function.


Configurable structure that will hold the most common confirmation messages

It could be an Enum, or maybe better a config file.
TBD

we have now config/labels.php


[Feature] Improve async resource loading by adding async/await support.

Hi there! I think it'd be better to make [https://docs.rs/rg3d/0.11.0/rg3d/engine/resource_manager/struct.ResourceManager.html#method.request_texture_async](this method) supports rust async/await.

Hello, I need to learn async\await in Rust before doing that 🙂
Resource manager still lacks proper async resource loading support, I'm currently busy with editor for engine and UI library, so have no idea when I'll have time to improve this.

I'm closing this, because recently I added async/.await texture loading. See #40 for more details.


feat: [IAC-2080]: Support for IACM default pipelines

Describe your changes
This commit addes a new resource and data source to allow management of IACM default pipelines.
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck
gitleaks: trigger gitleaks

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


Activity Explorer does not show sensitivity label changes for office apps

Your documentation states:-
Sensitivity label activities and Retention labeling activities from Office native applications, Azure Information Protection add-in, SharePoint Online, Exchange Online (sensitivity labels only)
But in my testing only updates done through sharepoint and the web apps show up, not if you edit them with the office apps - however if I check the Unified event log in powershell - they are captured there !

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

ID: 3a3d3633-893d-9354-9402-c47ed7dec790
Version Independent ID: fc12340b-403f-b531-04be-91f00a8f0af6
Content: Get started with activity explorer - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/data-classification-activity-explorer.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

@technonath Thank you for your feedback. I'm currently testing this to see if this is a bug in your tenant or by design.  I'll update you shortly.
Thanks
Sri

Thank you, just to be clear it was when editing the office documents from
the same sharepoint location that has been synchronized with onedrive.
On Tue, 14 Dec 2021, 06:39 Sriraman M S, @.***> wrote:

@technonath https://github.com/technonath Thank you for your feedback.
I'm currently testing this to see if this is a bug in your tenant or by
design. I'll update you shortly.
Thanks
Sri
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7038#issuecomment-993206743,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF4HK2JGNEZ7KNNHUA3S7UTUQ3RBTANCNFSM5JENZPMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@chrfox Please help us in resolving this issue. Thanks/ Email sent with details: 12-19-2021

Hi @technonath , thanks for your comment. I have sent an inquiry to the product team. I'll update the content if necessary.  In the meantime, I suggest that you open a Support ticket.


your home page icons looks not very good

hei, I'd like to try some icons, but something wrong is happed on your webpage
look

Not for me mhm did you reload your page :$

all right, still not working,
by the way can you add a tool which is called
back to top
It's simple and usefulI think you should consider it

What browser are you using?

Due to lack of feedback/information, I'll be closing this issue. If you can give us more information we can reopen it.


Theme submission: DPSG

Theme submission
Link to my theme repository: https://github.com/pfadfinder-konstanz/hugo-dpsg
This is a theme incorporating the CI of the largest German scouts association.
I made sure that...

[x] the repository contains a good README.md describing my theme
[x] an open source license has been added to LICENSE.md
[x] all metadata have been added to theme.toml
[x] screenshots have been added in the images/ folder with the required dimensions
[x] in case I'm using a customized demo via the exampleSite folder that

[x] https://example.com is set as base url in exampleSite/config.{toml, yaml, json} to avoid the abuse of unused domains
[x] I tested that my theme's demo works with the content directory of gohugoio/HugoBasicExample

[x] I tested my theme against the gohugoio/HugoBasicExample

[x] I've checked the developer tools' console in my browser for error messages

[x] in case my theme is using Hugo Pipes features like toCSS and PostCSS that I have committed the /resources directory with all generated assets, for my theme to work in the basic version of Hugo

New themes will usually be promoted on Hugo's official Twitter account. If you would like to be mentioned in the tweet please add your Twitter username to this submission.
Link to my Twitter account (optional): https://twitter.com/mxmehl

Your theme is already live on Hugo's theme site. Next, I'll promote your theme on Hugo's official Twitter account.


Dzwelling/close toast after five seconds

Alaska Airlines Pull Request
Fixes: #13
Summary:
Automatically close the toast after 5 seconds have elapsed.
Type of change:

[x] New capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

@blackfalcon I pushed new changes if you'd like to take another look.


Added Email Existence Check via OTP and Verification Code During Sign-up

Changes:
1- Email Verification via OTP:
Integrated Node-Mailer to send an OTP for verifying the user's email during sign-up.
Users will receive a verification code in their email, ensuring that only valid email addresses can be used for account creation.
2- OTP Verification and Validity:
Added OTP validation logic in the backend:
The OTP must match the one sent to the user's email for successful verification.
OTP is valid for 10 minutes to enhance security.
Updated the user model schema to include a verified field, which is set to true only when the OTP is correctly entered.
3- New OTP Input Page:
Added a new page that appears after the user presses the "Sign-up" button, allowing the user to enter the OTP received via email.
Impact: This feature enhances the security of the sign-up process by ensuring that users can only register with valid, verified email addresses.
NOTE -
Make sure that the .env file is setup properly ( RAZRORAPY, GMAIL ID AND PASSWORD FOR Node-mailer to function )
and firebase details as well in client folder .env
Thank you

@PorePranav , you can check the code, I have written
i was facing many issues with email service
so took a few days to resolve them. I have created a new file in the utils folder, named ( emailService.js ), there I have written the complete code for sending the otp and handling errors as well.
I have tested on my local system, and its working completely fine.
Install all the packages before trying to run. Thank you

The functionality is working well, however there are some issues that are needed to be addressed. Please check the inline review comments.
Thank you!

So, you will merge the code and accept my Pull Request.

The functionality is working well, however there are some issues that are needed to be addressed. Please check the inline review comments.
Thank you!

Hey, @PorePranav I have made the changes in the inline comments you added.
However, for the emailService.js file inside, Util
I will have to create a lot of changes in my code and It may create a mess.
I wanted to add a 6 digit OTP verification, for email. with a Proper message that the mail is from Campus-Unify
That is the reason, I created a separate JS file

Hey @sidgureja7803,
Thank you for your contribution and for taking the time to open this PR. Unfortunately, I'm unable to merge it at the moment due to some existing issues. I'll review it in more detail later, and we can revisit it when those concerns are addressed.
Thank you!

Hey @sidgureja7803, Thank you for your contribution and for taking the time to open this PR. Unfortunately, I'm unable to merge it at the moment due to some existing issues. I'll review it in more detail later, and we can revisit it when those concerns are addressed.
Thank you!

Please merge my PR, I have spent a week working on this.
I will create one more PR, if you allow me to and this time, I will make sure i don't repeat the mistakes you pointed out in inline comments.


seldon-core-operator fails to build during integration tests

Seems like this charm cannot be built during integration tests, resulting in the following error message:
RuntimeError: Failed to build charm .:
Packing the charm.
Launching environment to pack for base name='ubuntu' channel='20.04' architectures=['amd64'] (may take a while the first time but it's reusable)
charmcraft internal error: LXDError(brief="failed to inject host's snap 'charmcraft' into target environment.", details=None, resolution=None)
Full execution log: '/home/runner/.local/state/charmcraft/log/charmcraft-20230726-124415.757660.log'

Logs and info here and in the latest CI run of #184

The error message is not very clear and it is not showing exactly what is failing in the CI, after closer inspection to the CI logs and several runs, the error is pretty inconsistent. In a effort to minimise possible errors due to limited storage and to isolate test environments, #187 introduces certain changes that could probably fix the issue since:

GH runners are limited in storage and the SeldonDeployments may be requiring more than the machines can give, causing them to never deploy or halt mid deployment. For that, increasing the storage of the runner is included in the aforementioned PR.

There are two test files building and deploying the same charm (seldon-core-operator), which can cause issues because there is no cleanup in between the two tests executions, leading to potential errors when deploying the same application.

I will keep looking into this error if it is not fixed by #187, but I'm almost certain this will do it.


#29: Removed logging to STDOUT and adding logrotate to stdout/stderr …

…files
Test Evidence : https://gist.github.com/bbaugher/e7d868bd3f13b4759f0d

+1

+1

Got enough +1s, merging


Add actor example

Description
Adds a simple actor example that is easy to experiment with
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to
implementation.
#6
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Tests pass
[ ] Created/updated tests
[ ] Extended the documentation

Codecov Report

Merging #87 (b1192e6) into main (98dfdd3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #87   +/-   ##
=========================================
  Coverage     73.76%   73.76%
  Complexity      396      396
=========================================
  Files            49       49
  Lines          1456     1456
=========================================
  Hits           1074     1074
  Misses          382      382

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 98dfdd3...b1192e6. Read the comment docs.


Minimize need for repeated generic arguments.

Using many gadgets requires repeating type parameters: once for the native version, and once for the constraint version. However, this seems redundant as usually there's only a single gadget for a particular primitive. So, it makes sense to tie the gadget to the native impl. We can do this via an extension trait, as follows:

pub trait CRHGadget<ConstraintF: Field>: CRH {
	type ParametersVar: AllocGadget<Self::Parameters>;
	type InputVar: AllocGadget<Self::Input>;
	type OutputVar: AllocGadget<Self::Output>;

	fn evaluate(params: &Self::ParametersVar, input: &[UInt8]) -> R1CSResult<Self::OutputVar>;
}

impl CRHGadget for PedersenCRH { ... } // Note: this is implemented for *PedersenCRH*, not PedersenCRHGadget.

We can invoke this as

fn hash<H: CRHGadget<ConstraintF>, F: Field>(...) {
	H::evaluate(&params, &input)?;
}

We've reduced the number of type parameters greatly. This works for Gadget traits which don't contain variables themselves. What about Var traits, which contain variables? We can still use them, as follows:
trait FieldExt {
	type Var: FieldVar<...>;//figure out what to put in the type parameters.
}

impl FieldExt for FpN {
...
}

This would allow us to access the "unique" variable type for a given field element, without introducing multiple type parameters.
Feedback on this idea is very welcome!

should input be &Self::InputVar instead of &[UInt8]? (or rather, Borrow<Self::InputVar>)
Yeah this API looks good (if possible). One minor issue is that we have to input var is an AllocVar of Self::Input, which might not always be the case ([UInt8] is not an AllocVar for [u8], for example). It makes sense to bound OutputVar, but do we really need to bound InputVar?


FIX: Improve the command

This should allow us to revert https://github.com/conda-forge/staged-recipes/pull/16711/commits/34d48a52cc6d89cde4dbb9ab82d5d6028b15e7d4. Currently running the command will silently download the sample dataset, which is not good. It also has no --help option, so the default entry_point actually tried to display -- which is problematic for the conda-forge recipe.

Thank you @larsoner. This entry-point was more meant for testing than for normal use. I did not know that it is invoked by conda-forge.
Should we add the --help then in this stage?
A future feature is planned where calling mne-qt-browser (or mne browse_raw) where you can open the browser without any file first and then select the file from a dialog. Then the --help would indicate that behaviour

Should we add the --help then in this stage?

This PR at least allows things to be tested with mne-qt-browser --help and doesn't ever download sample itself, which are both improvements. But I'm also fine with removing the command altogether if it's really just for debugging.

A future feature is planned where calling mne-qt-browser (or mne browse_raw) where you can open the browser without any file first and then select the file from a dialog. Then the --help would indicate that behaviour

To me this should just be mne browse_raw where you have MNE_BROWSE_BACKEND=pyqtgraph in your config, no need for a separate command.
So I think overall I'm +1 on removing mne-qt-browser as a command. Let me know if you agree and I'll change this PR to do it.

Yes lets remove it.

Done @marsipu

Thank you @larsoner


Bump NonEmpty

This should make it 4.2-compatible.

Will update after cutting a 4.2 Linux-compatible release of NonEmpty...


ADVI is not up-to-date on the AD side

Raised by Paul Myers on Slack:
julia> Turing.setadbackend(:reversediff)
┌ Warning: `ADBACKEND` and `setbackend` are deprecated. Please specify the chunk size directly in the sampler constructor, e.g., `HMC(0.1, 5; adtype=AutoForwardDiff(; chunksize=0))`.
│  This function has no effects.
│   caller = ip:0x0
└ @ Core :-1

julia> advi = ADVI(10, 1000)
ADVI{AdvancedVI.ForwardDiffAD{0}}(10, 1000)

The "hotfix" is to do::
Turing.AdvancedVI.setadbackend(:reversediff)

because AdvancedIV.jl is not updated to support ADTypes.jl, etc.

This should now be solved in AdvancedVI.jl v0.2.5 (https://github.com/TuringLang/AdvancedVI.jl/pull/56)


Ensure text search also searches specialties

Developer specialties, launched today #803, gives developers an official way of letting businesses know what they are interested in. This isn't enabled in production yet, though.
I'm planning on allowing developers to add their specialties for a few weeks before opening up filtering to businesses. To help ease the transition, I'd like to make sure the text search also searches specialties.
A PR addressing this issue should:

[ ] With a paid business plan, when searching by text on /developers, include developers who have specialties with the same name

For example, if I search for "hotwire" I should see profiles that mention "hotwire" in their hero and/or bio and if they have the "hotwire" specialty selected.

hi @joemasilotti   I can take on this task

Excellent! All yours @nnahgodspower122. Let me know if you run into any issues or have questions. I'm thinking we will need to modify the pg_search query and index on Developer but maybe not.
I'll reach back out in 2 weeks if I haven't heard from you.

Thanks @joemasilotti would leave a comment here if I have any questions

hi @joemasilotti please see attached video if this is the expectation for this issue
https://user-images.githubusercontent.com/94062261/226338421-2be39911-d608-4743-91c9-b60cc917a8fb.mp4

That is looking good! Is the text input still searching developer hero/bio fields, too?

yes it still searches for developer hero/bio

Excellent! Please submit a PR whenever the code is ready for review.

Closed via #823.


remove unnecessary if statement

Remove unnecessary if statement and improve coverage

I fear there is a misunderstanding:
Line 58 updates the value of widget_key, which is then re-evaluated. In theory, the second if could be moved into the first one, but other than that the code should be correct as it is.
Or do you see some other reasons for the need of your change?

Oh! Sorry, that's true! Maybe I've confused about how this if statement works.
Your last comment clarified it. Could I change with another commit?


VerificationEnroll function does not return phrasesList in NET 6.0

Describe the bug
Hi,
I am enrolled in the Speaker Recognition capability and very eager to try out this functionality. However when I try to compile the example method VerificationEnroll I get the following error in Visual Studio for Mac 2022 Preview:

If I change phraseResult into a normal variable within the function scope, the code compiles but crashes on the "Speak the passphrase" console printout since phraseResult.Phrases does not exist.
I get these results for client, profile and phrasesResult
client -> {"Properties":{},"IsDisposed":false}
profile -> {"Id":"","Type":2,"IsDisposed":false}
phrasesResult -> {"ResultId":"DC8162328FA142ED85EFF0265A8D7672","Reason":1,"Phrases":null,"Properties":{}}
What am I doing wrong?
regards,
Djim
To Reproduce
Steps to reproduce the behavior:

Use the VerificationEnroll example from the Getting Started Documentation
Run the code

Expected behavior
I expect the phrasesList to return phrases so they can be used to start enrolling a speaker profile
Version of the Cognitive Services Speech SDK
Microsoft.CognitiveServices.Speech 1.19.0
Platform, Operating System, and Programming Language

OS: macOS Monterey 12.1
Hardware: x64
Programming language: C# .NET 6.0
IDE: Visual Studio for Mac 17.0 Preview

Well, I'll have to apologize for the QuickStart doc being wrong. The VoiceProfilePhraseResult doesn't implement IDisposable and can't be in a using statement.
Past that, the retrieval request was canceled (Reason:1 where ResultReason.Canceled = 1)
You can get a more detailed reason for the cancelation with code like:
?VoiceProfilePhraseCancellationDetails.FromResult(phraseResult)
{Reason:Error ErrorDetails:<Authentication error (401). Please check subscription information and region name.>}
    ErrorCode: AuthenticationFailure
    ErrorDetails: "Authentication error (401). Please check subscription information and region name."
    Reason: Error
    voiceProfileEnrollmentResult: {ResultId:3aa7b736135b472691d6b5d16840c38a Reason:Canceled Recognized Json:{"error":{"code":"401","message": "The Pass Phrases Operation under Speaker Verification is not supported with the current subscription key and pricing tier CognitiveServices.S0."}}}

(From the VS immediate window)
Which leads to me a guess at why the request may have been canceled. Speaker Recognition is gated service that requires prior approval to use. The details for this can be found here.

Closing the item since the QuickStart doc has been corrected and no further details received. Please open a new issue if more support is needed.


Add a new template for portfolio

#4
simple portfolio template made using HTML, CSS and JS.

@anu-shree LGTM! I will review and merge soon.


CB-15710 Add java.io.tmpdir property for NiFi. Added property java.ar…

…g.7, to set additional java argument java.io.tmpdir for NiFi process.
See detailed description in the commit message.

Change looks okay, but we'll only merge this after the next hotfix in line is merged:
https://github.com/hortonworks/cloudbreak/pull/12085

@vbrodetskyi Just a few sanity checks:

Does this really have to land in CB 2.51 instead of master (2.52)?
Is there possibly any prerequisite regarding runtime version support (I mean possibly requiring some recent build of 7.2.14)?

@vbrodetskyi Just a few sanity checks:

Does this really have to land in CB 2.51 instead of master (2.52)?
Is there possibly any prerequisite regarding runtime version support (I mean possibly requiring some recent build of 7.2.14)?

@lajosrodek  Richard told me that i just need to create PR for 2.51.0. After that it will be merged to master.

ping Laszlo to merge to the relesase branch and we will merge back
no need for master branch pr

No, there is not any requirement on CM/CDH version, for this property. Only CFM-<IP_ADDRESS> required. It should be default CFM version for CDP 7.2.14, so i think everything should be fine.

Change looks okay, but we'll only merge this after the next hotfix in line is merged: #12085

@lnardai @doktoric looks like https://github.com/hortonworks/cloudbreak/pull/12085 already merged, so could you please also merge current changes? Thank you.