I'm going to reject this PR for now:

For something like this, it either needs to be discussed on the forum thread, or made optional wit hthe old method being the default option.
In V2World/Province.h, better to simply call it rgo instead of newRgoString, as it doesn't get built up into a more complete type. Likewise, the getter function should be renamed. As well, it should be placed in it's own space among the members, not separating the members relating to the province owner.
V2World/Province.cpp makes me suspect this won't work right. The former ignored item was "rgo" (with no warnings in the log), but the one you're trying to match is "goods_type". Are you matching an internal element of an rgo item? In that case, you need to create an RGO class to handle it and extract the data you want. You should even be able to record the workers in the RGO (more on that later). If you're doing what I think, you'll see from the log (and console) that you're skipping right past unhandled items.
If you move the existing code into an RGO class, you need to use a better name for an internal variable than rgoTest. That indicates quick'n'dirty code that wasn't cleaned up. You need to clean it up before submitting a pull request.
Vic2::State::getRgo() is fatally flawed. Do not use an array with no names attached to store data like that. Don't return a pointer to internal data of the class. And from a design standpoint, why are you using total province population instead of the workers employed by the RGO? In this case, either merge RGO classes together and return a container of them, or do something like HOI4World/HoI4State.h and map the value to the name and value. As well, the getter function should not calculate these. They should be calculated in advance and the getter should only return the results of the calculation.
There was no need to change how resources were handled in HOI4World/HoI4State.cpp. And if there were, you would still need to remove the existing implementation, not just bypass it and leave it there. If you need to clear the existing resources for your (optional) case, you can call resources.clear().

Sorry if this is harshly worded, but the quality needs to be greatly improved before I can accept it.

Eh, it's still more grumpily worded than i like from myself, but thank you for being gracious. And I think I misunderstood that this was just a demonstration.

Yeah, you'd have to create a whole class to handle RGOs, unfortunately.

I'm not sure where I'd prioritize incorporating this (I've got a lot of other things I want to implement), but let me know how well it works in your game so I can think about it!


Added KNSL №145

@olegoniy First stage of verification passed, official domain is lic145.kiev.ua as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@olegoniy Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Adds brakeman.ignore file

Adds brakeman.ignore file. Running brakeman against this app produces a false positive, and so I have created the ignore file and added the false positive to it. Details are in the README.
Related ticket

Closing to allow for Platform Health to take a more unified approach


Fail flutter run --release

can you help me?
do you know the error?
I try to run flutter run --release and flutter build apk
`C:\Users\52963\AndroidStudioProjects\itec>flutter run --release
Initializing gradle...                                              1.7s
Resolving dependencies...                                           7.3s
Launching lib/main.dart on Moto G 5 Plus in release mode...
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedResFolders(FileCollection)
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':multi_media_picker:verifyReleaseResources'.

java.util.concurrent.ExecutionException: com.android.builder.internal.aapt.v2.Aapt2Exception: Android resource linking failed
C:\Users\52963\AndroidStudioProjects\itec\build\multi_media_picker\intermediates\res\merged\release\values-v28\values-v28.xml:7: error: resource android:attr/dialogCornerRadius not found.
C:\Users\52963\AndroidStudioProjects\itec\build\multi_media_picker\intermediates\res\merged\release\values-v28\values-v28.xml:11: error: resource android:attr/dialogCornerRadius not found.
C:\Users\52963\AndroidStudioProjects\itec\build\multi_media_picker\intermediates\res\merged\release\values\values.xml:2601: error: resource android:attr/fontVariationSettings not found.
C:\Users\52963\AndroidStudioProjects\itec\build\multi_media_picker\intermediates\res\merged\release\values\values.xml:2602: error: resource android:attr/ttcIndex not found.
error: failed linking references.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1m 49s
Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                     110.6s
Gradle task assembleRelease failed with exit code 1`

Oh maybe that issue is related to mixed dependencies (AndroidX / Support library).
Let me update the plugin and see if it works.

Please can you try now?
Apologies for the wait.

Thanks, the problem was solved.


status_quo: bottlerocket canary hangs

I wanted to share some of my experience using Rust async. I tried to follow the instructions but following instructions isn't always my strong suit so let me know...

My push didn't go through when I made those comments. Now the push is actually there.

Thanks! I get it much better now.

Is anything else required for merging?

@webern Sorry for being slow! It's been a busy couple of weeks. I think this is looking good as is!


Table: cannot create an OBJECT array from memory buffer

OK, this might be several issues I experienced, but I am putting them here together.
From the NanoAOD file format of CMS, a TLorentzVector Table is reconstructed via:
 flatarray = uproot_methods.classes.TLorentzVector.TLorentzVectorArray.from_ptetaphim(
            arrays["%s_pt" % name].content, arrays["%s_eta" % name].content, arrays["%s_phi" % name].content, arrays["%s_mass" % name].content)
jaggedarray = awkward.Methods.mixin(uproot_methods.classes.TLorentzVector.ArrayMethods, awkward.JaggedArray).fromoffsets(
            arrays["%s_pt" % name].offsets, flatarray)

Then the other property (like Jet_EMFrac etc) are associated with this Table by
jaggedarray["EMFrac"] =arrays["Jet_EMFrac"]
This is OK for local test. But if I run the same code on LPC Condor, the same code crash:

File "/storage/local/data1/condor/execute/dir_20889/awkward/array/jagged.py", line 749, in setitem
self._content[where] = self._broadcast(what)._content
File "/storage/local/data1/condor/execute/dir_20889/awkward/array/jagged.py", line 764, in _broadcast
data = self._util_toarray(data, self._content.dtype)
File "/storage/local/data1/condor/execute/dir_20889/awkward/array/base.py", line 312, in _util_toarray
return cls.numpy.frombuffer(value, dtype=getattr(value, "dtype", defaultdtype)).reshape(getattr(value, "shape", -1))
ValueError: cannot create an OBJECT array from memory buffer

I also tried another approach, by assigning attribute to the Object(awkward.Table type) by
 setattr(jaggedarray, "EMFrac", arrays["Jet_EMFrac"])
It is my prefer way as it keeps the code development simpler, jet.EMfrac vs jet["EMFrac"]. It fixed the memory buffer issue above, but created another issue that the object selection Jet = Jet[Jet.Flag] will result in a new awkward.Table, and lost all the attribute.
So I am in a dilemma here ..

OK, there are unknown dependency issue. I was using the uproot come along with CMSSW, and awkward + uproot_methods from mine own. After I tared up and upload my own pip packages, the first issue disappear.

Okay, so the first, which results in an exception due to a dtype=object array in the mix, is a symptom of an old bug that's already been fixed. Updating uproot fixed it. (There were some old versions of uproot in CMSSW recently...)
But the second is that masking a Table that is also a Lorentz vector array loses table columns other than Lorentz vector kinematics? Okay, I'll take a look.

I've been touching up broadcasting and Table.__setitem__ in PR #99. I should have tested for your bug before these corrections, but I can't reproduce it after these corrections. Maybe it has been fixed?
import numpy, awkward, uproot_methods
pt = awkward.fromiter([[1.1, 2.2, 3.3], [], [4.4, 5.5]])
a = uproot_methods.TLorentzVectorArray.from_cylindrical(pt, pt*2, pt - 3, 0.511)  # fake 4-vectors
a
# returns <JaggedArrayMethods [[TLorentzVector(-0.64735, 0.88935, -1.9, 0.511) TLorentzVector(-0.67613, -2.0935, -0.8, 0.511) TLorentzVector(3.1358, 1.0281, 0.3, 0.511)] [] [TLorentzVector(-3.5688, 2.5736, 1.4, 0.511) TLorentzVector(0.024341, -5.4999, 2.5, 0.511)]] at 0x7a2c8feaa978>

a["scalar"] = 999
a["scalar"]
# returns <JaggedArray [[999 999 999] [] [999 999]] at 0x716be7e3c7b8>
a["flat"] = [100, 200, 300]   # becomes a Numpy array and broadcasted
a["flat"]
# returns <JaggedArray [[100 100 100] [] [300 300]] at 0x716be7e3c6a0>
a["jagged"] = awkward.fromiter([[1, 2, 3], [], [4, 5]])
a["jagged"]
# returns <JaggedArray [[1 2 3] [] [4 5]] at 0x716be7e3c860>

### mask this 4-vector-with-stuff!
b = a[[True, True, False]]
b
# returns <JaggedArrayMethods [[TLorentzVector(-0.64735, 0.88935, -1.9, 0.511) TLorentzVector(-0.67613, -2.0935, -0.8, 0.511) TLorentzVector(3.1358, 1.0281, 0.3, 0.511)] []] at 0x7a2c832087b8>
b["scalar"]
# returns <JaggedArray [[999 999 999] []] at 0x7a2c83208908>
b["flat"]
# returns <JaggedArray [[100 100 100] []] at 0x7a2c83208a20>
b["jagged"]
# returns <JaggedArray [[1 2 3] []] at 0x7a2c83208a58>

### index this 4-vector-with-stuff!
c = a[[2, 1, 1, 0]]
c["scalar"]
# returns <JaggedArray [[999 999] [] [] [999 999 999]] at 0x7a2c83208978>
c["flat"]
# returns <JaggedArray [[300 300] [] [] [100 100 100]] at 0x7a2c83208a58>
c["jagged"]
# returns <JaggedArray [[4 5] [] [] [1 2 3]] at 0x7a2c83208978>

Hi Jim,
Ok, the version I was using is awkward v0.8.8. And I run into issue with your example code.

>>> import numpy, awkward, uproot_methods
>>> pt = awkward.fromiter([[1.1, 2.2, 3.3], [], [4.4, 5.5]])
>>> a = uproot_methods.TLorentzVectorArray.from_cylindrical(pt, pt*2, pt - 3, 0.511)  # fake 4-vectors
>>> a["scalar"] = 999
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/uscms/home/benwu/.local/lib/python2.7/site-packages/awkward/array/jagged.py", line 759, in __setitem__
    self._content[where] = self._broadcast(what)._content
  File "/uscms/home/benwu/.local/lib/python2.7/site-packages/awkward/array/jagged.py", line 774, in _broadcast
    data = self._util_toarray(data, self._content.dtype)
  File "/uscms/home/benwu/.local/lib/python2.7/site-packages/awkward/array/base.py", line 316, in _util_toarray
    return cls.numpy.frombuffer(value, dtype=getattr(value, "dtype", defaultdtype)).reshape(getattr(value, "shape", -1))
ValueError: cannot create an OBJECT array from memory buffer

Switching to the issue-98 branch seems fixed this issue. But failed below (v0.4.3 uproot_methods)
>>> a["jagged"] = awkward.fromiter([[1, 2, 3], [], [4, 5]])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "awkward/array/jagged.py", line 756, in __setitem__
    self._content[where] = self.tojagged(what)._content
  File "awkward/array/jagged.py", line 774, in tojagged
    return self.copy(content=data._content[self.IndexedArray.invert((self.index + self._starts)._content)])
  File "/cvmfs/cms.cern.ch/slc6_amd64_gcc700/external/py2-numpy/1.14.1-gnimlf/lib/python2.7/site-packages/numpy/lib/mixins.py", line 25, in func
    return ufunc(self, other)
  File "/uscms/home/benwu/.local/lib/python2.7/site-packages/uproot_methods/classes/TLorentzVector.py", line 301, in __array_ufunc__
    raise TypeError("(arrays of) TLorentzVector can only be added to/subtracted from other (arrays of) TLorentzVector")
TypeError: (arrays of) TLorentzVector can only be added to/subtracted from other (arrays of) TLorentzVector
>>>

This can be fixed by setting as attribute to a , instead of a column like
>>> setattr(a, "jagged", awkward.fromiter([[1, 2, 3], [], [4, 5]]))
>>> a.jagged
<JaggedArray [[1 2 3] [] [4 5]] at 0x7f627e7dd110>

But this won't work after selection:
>>> b = a[[True, True, False]]
>>> b.jagged
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'JaggedArrayMethods' object has no attribute 'jagged'

Maybe because I used quite a bit of Pandas. Since this is a Table, I am expecting
Pandas dataframe like control , dataframe["name"] is equivalent to dataframe.name.
Hopefully this feature can be added to the awkward.Table. Thanks a lot!

I just realized that you're attaching it as an attribute (__setattr__), not an item (__setitem__)! It's not a bug: any kind of selection returns a new object, which doesn't have the old object's attributes.
The feature you want, attribute-style access of items, was removed from awkward after we find some fundamental problems with it. Not only can item names collide with method names (not a huge problem), but __getattr__ hides bugs: an exception in the __getattr__ code or something it calls gets reported with the wrong error message. This is something internal to Python itself. Pandas also has this problem, but they have a large user base and can't change the interface as freely. I remember seeing a deprecation notice, in which they hope to make that transition in Pandas 1.0.
For a little while, we had a syntax like jaggedtable.at.name, but while this seems like a good idea, it made physics calculations really confusing to read.
The fnal-columnar-tools package, a layer above awkward-array for physicists, implements the __getattr__ feature through mix-ins, similar to TLorentzVector.ArrayMethods itself. That provides a cleaner separation. You could use that package or do the same thing they do.
Back to the actual bug: are you running exactly the same code as I am and getting an exception?

Hi Jim,
The crash above TypeError: (arrays of) TLorentzVector can only be added to/subtracted from other (arrays of) TLorentzVector is indeed using issue-98 branch of awkward-array and v0.4.3 uproot_methods. These are the latest branch I can find.
Now I understand the concerns. It seems their JaggedCandidateMethods() does that. I will try similar but lighter. Thanks

I have s commit in uproot-methods's master branch that's not in a release yet, but I didn't think it's related to this error. I didn't check your version number against the latest one—is it the latest? Your symptom sounds familiar.

OK, the crash comes from uproot-methods installed by pip, version number 0.4.3.
Now I checked out the master branch from github, also (v0.4.3) and it fixed the issue.
I thought both are the same versions so I didn't check the master branch.

I rarely let the matter branch differ from the latest release—I only did it this time because I thought that the change wouldn't affect much. It's a check that raises an error if there are invalid inputs. Unless I'm missing something, it can only cause more exceptions (good ones), or earlier exceptions (closer to the point when the mistake is actually made). I can't think of how it could cause an exception not to happen. Let me look.
In either case, I guess I should give this a proper version number and deploy it, since it's obviously helpful.

There were more undeployed changes than I had been aware of, some relevant to your bug. I'm deploying it now as 0.4.4.

Sounds good. Thanks

You probably saw the notice from scikit-hep/awkward-array#99, but I'm deploying all of the pieces that fix the bugs you've been seeing.

Yeah, I saw it and will update my package. Thansk


Update isolated_labels.py

As mentioned in https://github.com/theislab/scib/issues/367, the isolated label metrics do not compute what they should.
By calculating the individual silhouette scores and then taking the mean over only the cells of the cluster, this should be fixed.

I guess you'll need to adapt the tests as well for this.

Please also adapt the perfect score test

ah bit too fast

@mumichae could you still review this please?


tcgaCompare seems to get the wrong samples in plot

laml.mutload = tcgaCompare(maf = laml, cohortName = 'Example-LAML')
1   I calculate the sample number in excel , it is 193,not 192
2  capture_size: capture size for input MAF in MBs. Default NULL. If provided plot will be scaled to mutations per mb.
tcga_capture_size   : TCGA capture size is assumed to be 50mb.
without default capture_size, how can calculate the Example-LAML TMB?  and people usually say tcga capture size is 38, bot 50
3  in maf file. do you count each variant in a sample without filtering, such as filter silent,and I think teh tmb you calculate is a little big. and has the tmb in plot been loged?

does the y-axis stands for the number of genes ,so 10^-1=0.1, how should I interprete this, thanks a lot

Hi,

Function removes samples with zero mutations be default. Setting rm_zero = FALSE shows 193.
Without capture size mutation burden is not normalized to per MB. They will be just median values. There is no solid way of knowing TCGA capture size, it depends on who you ask. Default value of 50MB is based on agilent sureselect kit. You can set it 38 or of your choice.
Only non-synonymous mutations are used.

For gistic results; negative sign indicates deletions. Its just 10^1 with sign reversed.

thanks a lot.
1 bubble is according to -log10 transformed q values, so can you give a q_value bar on the plot?
you said negative sign indicates deletions, so what the y-axis mean, can you describle it clearly
thanks a lot

I realised that plot had no scale for q values. I have added it now. Maybe it will help.
They are number of genes amplified (above zero) or deleted (below zero). Anyway I now made it to draw absolute values since it seems confusing. If you dont want log scale set log_y = FALSE
Its which bands to be labelled among all. By default it shows top 5 (based on q values). This argument doesnt take all as an option. You will have to provide names of the bands to highlight. Use getCytobandSummary() to get the summary.
I have made it more flexible now (you can set it to all or any number top bands or names of the cytobands.

so i nedd to install from tyhe github now, am I right?

Yes, that is correct.
remotes::install_github(repo = "poisonalien/maftools")

thanks a lot, I have tested that, wonderful, thanks a lot


Can't find 'basicsr.version'

I use Anaconda to run ESRGAN. I just tried to import the head file in 'train.py' . Then I run the 'get_env_info()' function. It reports that there is no module named 'basicsr.version' . Just like the picture below.
How can I solve this problem? What things I need to provide? Or is there anything I can do ? Thanks you!

Hello the error is not that important.
I modified the file as follow:

Hello,
This error is not really important.
You can find the version here : BasicSR-master/VERSION
By changing the function  get_env_info as follow :
Don't forget to change the path.

Hello,
This error is not really important.
You can find the version here : BasicSR-master/VERSION
By changing the function get_env_info as follow :
Don't forget to change the path.

Thank you for your reply. I delete all and rerun setup.py again . Now the question has been solved . It seems that is my personal question. Thank you again for your kindness . Have a nice day!


[Spec] More documentation is required for Rabbit MQ trigger

More spec documentation is required for Rabbit MQ trigger beyond just a sample spec.

I'll grab this and do on the docs repo

Docs are available here - https://keda.sh/scalers/rabbit-mq-queue/


python3Packages.paramiko: 3.4.0 -> 3.4.1

Description of changes
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please rebase on staging instead, merging into python-updates doesn't really make sense if there isn't a batch update at the moment.

Actually, I can do that for you!


the last update broke the youtube query

app version: X.X.X

[ ] twitch api key set
[x] youtube api key set

issue description:
With 1.0.4 Only one video will be obtained from the YouTube link, and it is not playable.
Downgrading to 1.0.2 fixed the problem.

kubectl logs vod2pod-64f65b5b9b-tphnw -f

Defaulted container "vod2pod" out of: vod2pod, podtube
2023-10-19T00:00:48.813Z INFO  [app] app version 1.0.4
2023-10-19T00:00:48.814Z DEBUG [app] cached app version Some("1.0.4")
2023-10-19T00:00:48.814Z DEBUG [app] set cached app version to 1.0.4
2023-10-19T00:00:48.814Z INFO  [app] starting server at http://<IP_ADDRESS>:8080
2023-10-19T00:00:48.814Z INFO  [actix_server::builder] starting 12 workers
2023-10-19T00:00:48.814Z INFO  [actix_server::server] Actix runtime found; starting in Actix runtime
2023-10-19T00:01:12.379Z INFO  [app] processing transcode at 320k for https://www.youtube.com/watch?v=-uHjDNm3sgM
2023-10-19T00:01:12.380Z DEBUG [vod2pod_rss::provider] using YoutubeProvider
2023-10-19T00:01:12.380Z DEBUG [app] received content range 0-
2023-10-19T00:01:12.380Z DEBUG [app] requested content-range: bytes 0-11359999/11360000
2023-10-19T00:01:12.380Z DEBUG [app] choosen seek_time: 0
2023-10-19T00:01:12.380Z DEBUG [app] seconds: 284, bitrate: 320
2023-10-19T00:01:12.380Z DEBUG [vod2pod_rss::provider] using YoutubeProvider
2023-10-19T00:01:12.381Z DEBUG [vod2pod_rss::provider::youtube] getting stream_url for yt video: https://www.youtube.com/watch?v=-uHjDNm3sgM
2023-10-19T00:01:13.682Z WARN  [vod2pod_rss::provider::youtube] error while parsing stream url:
input:
error: relative URL without a base
2023-10-19T00:01:34.113Z DEBUG [vod2pod_rss::provider] using YoutubeProvider
2023-10-19T00:01:34.114Z INFO  [vod2pod_rss::provider::youtube] trying to convert youtube channel url https://www.youtube.com/@dacongmovie
2023-10-19T00:01:34.119Z DEBUG [reqwest::connect] starting new connection: http://<IP_ADDRESS>:15000/
2023-10-19T00:01:34.119Z DEBUG [hyper::client::connect::http] connecting to <IP_ADDRESS>:15000
2023-10-19T00:01:34.119Z DEBUG [hyper::client::connect::http] connected to <IP_ADDRESS>:15000
2023-10-19T00:01:34.119Z DEBUG [hyper::proto::h1::io] flushed 94 bytes
2023-10-19T00:01:37.446Z DEBUG [hyper::proto::h1::io] parsed 6 headers
2023-10-19T00:01:37.446Z DEBUG [hyper::proto::h1::conn] incoming body is content-length (61593 bytes)
2023-10-19T00:01:37.447Z DEBUG [hyper::proto::h1::conn] incoming body completed
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::rss_transcodizer] parsed feed
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::rss_transcodizer] Title found: 大聪看电影
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::rss_transcodizer] Description found: 过度解读&脑洞解析
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::rss_transcodizer] image found, logo branch: x=Image { uri: "https://yt3.ggpht.com/Q9YQlHCRE8GV6_fhKSH0uQXqi1Sylqwmio_bTRX5jWNyFj1fKHGmG2TSDnYVvuJP8JjuMT6M8Q=s800-c-k-c0x00ffffff-no-rj", title: Some("大聪看电影"), link: Some(Link { href: "https://www.youtube.com/channel/UCkM0vZ9Q2ZxxRPORzTDI7eg", rel: None, media_type: None, href_lang: None, title: None, length: None }), width: None, height: None, description: None }
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::provider] using GenericProvider as provider
2023-10-19T00:01:37.449Z DEBUG [vod2pod_rss::rss_transcodizer] item title: 《阿凡達2》最終版預告解析！老二社死名場面！更多畫面細節曝光！#阿 凡達2 #阿凡达2
2023-10-19T00:01:37.450Z WARN  [vod2pod_rss::rss_transcodizer] no media_url found out of:
""
2023-10-19T00:01:37.450Z WARN  [vod2pod_rss::rss_transcodizer] links analyzed:
""
2023-10-19T00:01:37.450Z WARN  [vod2pod_rss::rss_transcodizer] regex used:
"^(https?://)?.+\\.(mp3|mp4|wav|avi|mov|flv|wmv|mkv|aac|ogg|webm|3gp|3g2|asf|m4a|mpg|mpeg|ts|m3u|m3u8|pls)$"
2023-10-19T00:01:37.450Z DEBUG [vod2pod_rss::provider] using GenericProvider as provider
2023-10-19T00:01:37.450Z DEBUG [vod2pod_rss::rss_transcodizer] item title: 🜃1899🜃你看懂了嗎！深度解析這一切到底發生了什麼！#1899 #暗黑
2023-10-19T00:01:37.451Z WARN  [vod2pod_rss::rss_transcodizer] no media_url found out of:
""
2023-10-19T00:01:37.451Z WARN  [vod2pod_rss::rss_transcodizer] links analyzed:
""
2023-10-19T00:01:37.451Z WARN  [vod2pod_rss::rss_transcodizer] regex used:
"^(https?://)?.+\\.(mp3|mp4|wav|avi|mov|flv|wmv|mkv|aac|ogg|webm|3gp|3g2|asf|m4a|mpg|mpeg|ts|m3u|m3u8|pls)$"
2023-10-19T00:01:37.451Z DEBUG [vod2pod_rss::provider] using GenericProvider as provider
2023-10-19T00:01:37.451Z DEBUG [vod2pod_rss::rss_transcodizer] item title: 星際異攻隊聖誕特別篇！彩蛋滿滿！星爵又有家人了！ #星際異攻隊 #银 河护卫队
2023-10-19T00:01:37.452Z WARN  [vod2pod_rss::rss_transcodizer] no media_url found out of:
""
2023-10-19T00:01:37.452Z WARN  [vod2pod_rss::rss_transcodizer] links analyzed:
""
2023-10-19T00:01:37.452Z WARN  [vod2pod_rss::rss_transcodizer] regex used:
"^(https?://)?.+\\.(mp3|mp4|wav|avi|mov|flv|wmv|mkv|aac|ogg|webm|3gp|3g2|asf|m4a|mpg|mpeg|ts|m3u|m3u8|pls)$"
2023-10-19T00:01:37.452Z DEBUG [vod2pod_rss::provider] using GenericProvider as provider
2023-10-19T00:01:37.452Z DEBUG [vod2pod_rss::rss_transcodizer] item title: 星戰《安多》！小人物大格局！更懂星戰精髓！#安多 #shorts
2023-10-19T00:01:37.453Z WARN  [vod2pod_rss::rss_transcodizer] no media_url found out of:
""

sorry, seems there is something wrong with my reverse proxy


ENH: Expose values of dual variables in milp/linprog

The function scipy.optimize._highs._highs_wrapper._highs_wrapper provides information about the solution to the dual problem through the attributes "lambda" and "margs_bnds", but that information is not carried through scipy.optimize._milp.milp; that is, in scipy.optimize._milp.milp, parts of what is in highs_res is not carried over to res.
Here's an example (taken from here):
from scipy.optimize import milp
from scipy import optimize

c = np.array([1, 2, 4, 1])
A = np.array([[-1, 1, 1, 10], [1, -4, 1, 0], [0, 1, 0, -.5]])
lb = np.array([-np.inf, -np.inf, 0])
ub = np.array([20, 30, 0])
constraints = optimize.LinearConstraint(A=A, lb=lb, ub=ub)
bounds = optimize.Bounds([0, -np.inf, -np.inf, 2], [40, np.inf, np.inf, 3])
res = milp(c=-c, constraints=constraints, bounds=bounds)

Here, in the execution of milp, highs_res has the following value:
{'status': 7,
 'message': 'Optimal',
 'unscaled_status': 7,
 'x': [17.5, 1.0, 16.5, 2.0],
 'slack': [0.0, 0.0, 0.0],
 'lambda': [-1.5, -2.5, -10.5],
 'marg_bnds': <MemoryView of 'ndarray' at 0x7f4674241220>,
 'fun': -87.5,
 'simplex_nit': 2,
 'ipm_nit': 0,
 'crossover_nit': 0}

where np.array(highs_res["marg_bnds"]) is
array([[0.  , 0.  , 0.  , 8.75],
       [0.  , 0.  , 0.  , 0.  ]])

and I would like to have this array, as well as highs_res["lambda"] (which is [-1.5, -2.5, -10.5]) available through the public interface.
This could be useful, for instance, for implementing branch and price solvers where the values of the dual variables of the reduced master problem is used to solve the pricing problem.
cc @mdhaber @mckib2

linprog does expose these when there are no integration constraints. See "marginals":
https://docs.scipy.org/doc/scipy/reference/optimize.linprog-highs.html#optimize-linprog-highs
milp is often for problems with integer constraints, in which case dual variables are not available.

@mdhaber Ah, thanks, linprog(...).ineqlin.marginals is indeed what I want, so that's great; sorry for missing that.
One thing that threw me off just now is that when using integer constraints, ineqlin.marginals is still there, but is just the all-zero array; I imagine that could be a bit misleading, and that it'd be better to leave it out altogether, or at least document that behavior? Similarly, highs_res["lambda"] is a zero array, but that's less of an issue since it's not part of the public API.

I agree; it would be better to leave out irrelevant things like that when there are integer constraints. PR welcome, all!

Any preference for whether we should leave out the meaningless keys entirely, or if we should keep the schema intact and use e.g. None for highs_res["lambda"], treating it as an Optional[np.ndarray]? It looks like there was an attempt at something in between in https://github.com/scipy/scipy/blob/6c4a3f3551f9ee52a75c8c3999a57bed8ea67deb/scipy/optimize/_linprog_highs.py#L381-L390 which handles the case that "lambda" is missing (which then doesn't actually happen even if there are non-continuous variables), and which uses None for e.g. ineqlin.marginals.


Dimensional Transceiver and (Insulated) Redstone Conduit

I think it would be really useful if Redstone Conduits, and Gas conduits could be connected via Dimensional Transceivers in the same way as Item, Energy and Fluid conduits. This would be particularly useful if the color system being used with Redstone Conduits would continue to work, allowing 16 redstone signals to be passed through the Transceiver.

I know this was a while ago, but is this still going to be implemented? A good way to send and receive Redstone signals is very hard to come by.

Redstone conduits need a complete rewrite before any features will be added to them. That is one of our TODOs for version 2.4. Gas conduits are suffering from the old "none of the active devs is using Mek himself" problems.

That makes sense. The redstone will definitely be more useful anyways. Keep up the amazing work. This is definitely one of my most used mods. :)


Adding ValidateNotNullOrEmpty attribute to parameters listed in #2672

Do changes to parameter metadata require tests?  If so, should they be located with the tests for each individual command, or would it be sufficient to write a single It block with -TestCases to hit all of the affected commands in this issue / PR?

I imagine we'd eventually have tests per cmdlet to keep it organized so I would prefer having individual tests scripts we would eventually build up for complete test coverage
CC @JamesWTruher

@dlwyatt Agree with @SteveL-MSFT and that's the approach we have used with #2542 and #2545 as well.

Perhaps we should add the relevant comments in the Provider cmdlets why we don't fix them to block accidental changes in future.

Get-Service was already fixed in https://github.com/PowerShell/PowerShell/commit/67638548fc9992f2a242ed33b48eea19ea06099c .

i believe we do need tests against metadata changes. Strictly speaking we should really have tests which ensure that we have the proper parameters and that their configuration (type/attributes/etc) is correct.

@JamesWTruher - are you fine with accepting the PR without tests?
If we do add tests, I'd rather see that we test behaviors, e.g. that Get-Service -Name $null throws, and not test that the metadata is what is "expected".
When I've seen test failures that validated implementation details, the fix was always to change the expectations in the test, which to me means the test wasn't catching anything, it was just duplicating the code in a test.


Clarify what all valid code means in readme

Fixes #555

Could we merge this? Lack of this information in the documentation caused me much confusion today

Just to note - this won't be true once this lands.


Change text

Can we update the text to Barlow throughout the app - to be universal with the website and other app.

What is the text that should be replaced?

All the text! Just changing the text font throughout the app to Barlow.

Closing as a legacy issue.


documentation

anyone have the documentation for this project?
iam trying to download the project http://confluence.highsource.org/display/HJ3/Purchase+Order+Tutorial
but tha page seems offline.

That old server regularly goes down so I have to restart it manually.
I've brought it back online.
http://confluence.highsource.org/display/HJ3/Home


Release v0.2

[x] Implement get_sdo & get_sdo_entry (#11)
[x] Implement get_pdo & get_pdo_entry (#13)
[x] Use new-type pattern for indexes (#14)
[x] Implement get_sync (bee0d75e22c5e2c63294003341e8f4deac0d0cfe)
[x] Add some basic debug logs (c3bd08288fc6379191bb9657873da5ae0ff17620)
[x] Improve errors (20c21269c1acb80b6c01bbe3c78a24daa8cc9645)
[x] ethercat-sys: generate ioctl numbers from master header (#18)
[x] Make it compile with upstream IgH Master (#20, #22)
[x] Install basic CI (#21, #19)
[x] include pregenerated bindings (#23, #29)
[x] release ethercat-types crate v0.3.0 and remove patch in ethercat/Cargo.toml
[x] Add CHANGELOG.md
[x] release ethercat-sys crate v0.2.0 (#28)
[x] Make sure all examples work with some real hardware (#3)
[x] Publish on crates.io

:+1: - I had no time to review the PRs due to holidays, but it all looks good.

done :)


Does raw packet sending work on OSX?

I'm unable to write raw packets, even root privileges and ipv4 forwarding enabled on OSX 10.7. Is this working? I think I was able to do so time ago on OSX 10.6 and upstream libnet.

I had the same issue. MacPorts has a patch that makes things work for me. Pull it?
https://trac.macports.org/browser/trunk/dports/net/libnet11/files/patch-configure.in

Oh, and I came across this issue when tcptraceroute didn't work on 10.7:
[lars@lars: ~] sudo tcptraceroute -d google.com
debug: tcptraceroute 1.5beta7, i686-apple-darwin11.2.0
debug: Compiled with libpcap 1.1.1, libnet 1.1.4 (API 110)
debug: entering getinterfaces()
debug: ifreq buffer set to 32
debug: Successfully retrieved interface list
debug: Using HAVE_SOCKADDR_SA_LEN method for finding addresses.
debug: Discovered interface lo0 with address <IP_ADDRESS>
debug: Ignoring AF_INET6 address on interface lo0
debug: Discovered interface lo0 with address <IP_ADDRESS>
debug: Ignoring AF_INET6 address on interface lo0
debug: Ignoring down interface gif0
debug: Ignoring down interface stf0
debug: Discovered interface en1 with address <IP_ADDRESS>
debug: Ignoring AF_INET6 address on interface en1
debug: Discovered interface en1 with address <IP_ADDRESS>
debug: Discovered interface en0 with address <IP_ADDRESS>
debug: Ignoring AF_INET6 address on interface en0
debug: Discovered interface en0 with address <IP_ADDRESS>
debug: Discovered interface p2p0 with address <IP_ADDRESS>
debug: Ignoring down interface fw0
debug: leaving getinterfaces()
debug: Determined source address of <IP_ADDRESS> to reach <IP_ADDRESS>
debug: entering finddev()
debug: finddev() returning en0
debug: Setting o_syn, in absence of either o_syn or o_ack
debug: debugoptions():
debug:         TEXTSIZE: 1024        SNAPLEN: 92     IPTOSBUFFERS: 12
debug: ALLOCATEID_CACHE: 512        datalink: 1    datalinkoffset: 14
debug:         o_minttl: 1          o_maxttl: 30        o_timeout: 3
debug:          o_debug: 1         o_numeric: 0          o_pktlen: 0
debug:       o_nqueries: 3        o_dontfrag: 0             o_tos: 0
debug:      o_forceport: 0             o_syn: 1             o_ack: 0
debug:            o_ecn: 0        o_nofilter: 0 o_nogetinterfaces: 0
debug:      o_trackport: 0      datalinkname: ETHERNET     device: en0
debug:       o_noselect: 1            o_dnat: 0               isn:<PHONE_NUMBER>
Selected device en0, address <IP_ADDRESS>, port 57382 for outgoing packets
debug: pcap filter is:
		(tcp and src host <IP_ADDRESS> and src port 80 and dst host <IP_ADDRESS>)
		or ((icmp[0] == 11 or icmp[0] == 3) and dst host <IP_ADDRESS>)
Tracing the path to google.com (<IP_ADDRESS>) on TCP port 80 (http), 30 hops max
debug: Generating a new batch of 512 IP ID's
libnet_write failed?  Attempted to write 40 bytes, only wrote -1

Merged, thank you.
https://github.com/sam-github/libnet/commit/65d4db36d678cadd80b1a0087302dcebeb6a0e04


Bottom Sheet

Feature request type
New component
Component name
No response
Is your feature request related to a problem?
No response
Describe the solution you'd like
Please Add Bottom sheet like Angular Material
https://material.angular.io/components/bottom-sheet/overview
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ahmadlashgari this is already possible with a MudDrawer:
https://mudblazor.com/components/drawer#api

Thanks! @dennisrahmen

Closing as solved by MudDrawer


Make PSQLDecodingContext generic

Motivation
Since we don't attach the JSONDecoder to the connection anymore and supply during decoding, we can now make the decoding generic over the supplied JSONDecoder.
Changes

Make PSQLDecodingContext generic over the JSONDecoder
Rename PSQLDecodingContext to PostgresDecodingContext

Result

We don't use an existential in the very hot code paths.

Codecov Report

Merging #217 (3bae404) into main (eaef208) will increase coverage by 3.03%.
 Consider uploading reports for the commit 08218a3 to get more accurate results

@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   40.69%   43.73%   +3.03%
==========================================
  Files         117      118       +1
  Lines        7709     7688      -21
==========================================
+ Hits         3137     3362     +225
+ Misses       4572     4326     -246

Flag
Coverage Δ

unittests
43.73% <49.60%> (+3.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

...resNIO/Connection/PostgresConnection+Connect.swift
0.00% <0.00%> (ø)

Sources/PostgresNIO/New/PSQLData.swift
0.00% <ø> (-100.00%)
:arrow_down:

Sources/PostgresNIO/Postgres+PSQLCompat.swift
0.00% <0.00%> (ø)

...es/PostgresNIO/Utilities/PostgresJSONEncoder.swift
0.00% <0.00%> (ø)

Sources/PostgresNIO/New/Data/Int+PSQLCodable.swift
20.26% <8.33%> (+2.61%)
:arrow_up:

Sources/PostgresNIO/New/PSQLCodable.swift
72.72% <14.28%> (-27.28%)
:arrow_down:

Sources/PostgresNIO/New/PSQLError.swift
57.74% <35.71%> (-26.04%)
:arrow_down:

...ces/PostgresNIO/New/Data/Decimal+PSQLCodable.swift
74.19% <50.00%> (+6.45%)
:arrow_up:

...es/PostgresNIO/New/Data/Optional+PSQLCodable.swift
61.11% <62.50%> (+14.05%)
:arrow_up:

...ources/PostgresNIO/New/Data/Bool+PSQLCodable.swift
95.34% <66.66%> (+4.65%)
:arrow_up:

... and 41 more


Receiving FaceLiveness.FaceLivenessDetectionError Code 1

Describe the bug
Some of my users are receiving
FaceLiveness.FaceLivenessDetectionError Code 1, which in the iOS SDK is a session not found error in FaceLivenessDetectionError.swift file,
public static let sessionNotFound = FaceLivenessDetectionError(         code: 1,         message: "Session not found.",         recoverySuggestion: "Enter a valid session ID."     )
We are creating a sessionID server side, passing the session ID to the FaceLivenessDetectorView constructor and receive the above error.
Steps To Reproduce
Follow steps above

Expected behavior
Faceliveness Detector view sequence starts and accepts valid sessionID.
Swift Liveness Version
1.2.14
Xcode version
15.4
Relevant log output
No response
Is this a regression?
No
Regression additional context
No response
OS Version
17.5.1
Device
iPhone 11 Pro
Specific to simulators
No
Additional context
This is intermittent. This happens to some users and we are not sure why.

Hi @haarj, thanks for opening this issue!
Would you mind telling us how you are triggering the creation of the session id on the server side? Also, do you have any relevant logs that you can share?
Thanks!

Hey @ruisebas sure thing. I can't reproduce it on my side. It happens to my users sometimes and it's hard to know why or acquire logs.
Here is how I am calling it server side.
`const rekClient = new Rekognition({
region: "us-east-1",
credentials: {
accessKeyId: rekAccessKeyID,
secretAccessKey: process.env.AMAZON_REKOGNITION_SECRET_KEY
}
});
const response = await rekClient.createFaceLivenessSession().promise();

const sessionId = response.SessionId;
console.log("SessionId:", sessionId);`

Thanks for providing the code. I assume you are triggering this code in your app, for example with a Amplify.API.post call, right?
Are you certain that you are receiving a valid response from the service before attempting to show FaceLivenessDetectorView?
Or putting it in another way, are you handling error responses so that FaceLivenessDetectorView is not being invoked with an old or default sessionId?

Hey @ruisebas my server side code is always returning a "valid" session Id, which is then passed back to the app and then passed to the FaceLivenessDetectorView immediately. What makes this a valid or invalid session ID I'm not sure. Is there a race condition happening where the sessionID I am creating server-side is not recognized by the detector view yet, maybe?

Hi @haarj, following up on this.
While a race condition would cause this, I think it's unlikely to happen since the session creation is performed on the backend itself. Are you sure you're passing the correct region to the Liveness component?
To get more information, would you be able to log these failing session ids and provide them, alongside their time of creation and region? We can then involve the service team and ask them to verify if the sessions are indeed created or what could be going on.
Thanks!

Oh and sorry to insist on this, but how are you handling errors when attempting to create the session?
const response = await rekClient.createFaceLivenessSession().promise();
const sessionId = response.SessionId;

I'm not really familiar with NodeJS, so what happens if createFaceLivenessSession returns an error?
Are you propagating it to your app and handling it inside a do catch block?

Hey @ruisebas I'll have to release a new version to capture this information. Standby on that front. I'll admit that I don't receive errors on the backend side of things when creating session Ids and so the apps always receive "valid" sessions Ids.

Thanks!
I think that might be the cause, createFaceLivenessSession () failing and the function returning an invalid/corrupted sessionId that is then attempted to be used.

Yea so it seems that every now and then a bad sessionID gets created. Anything I can do on my end to fix that?

To be precise, it's not like a bad sessionId is created, but rather that the CreateFaceLivenessSession call fails. So a session is never created.
This is outside the scope of the Liveness component, but what you should do is inspect the reason for the failure. You might want to propagate the error to the application so you can handle it gracefully (e.g. showing an error alert/screen instead of trying to show the Liveness component)

Well no a sessionID is always being created and I am not receiving any errors. Like you said, a corrupt sessionID is being created but it is still a sessionID that I am passing back to the app.JustinSent from my iPhoneOn Jul 29, 2024, at 12:31 PM, Sebastian Villena @.***> wrote:
To be precise, it's not like a bad sessionId is created, but rather that the CreateFaceLivenessSession call fails. So a session is never created.
This is outside the scope of the Liveness component, but what you should do is inspect the reason for the failure. You might want to propagate the error to the application so you can handle it gracefully (e.g. showing an error alert/screen instead of trying to show the Liveness component)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Ah, I see. Well, that is still outside the scope of the Liveness component and within the AWS Rekognition service, since they own the CreateFaceLivenessSession API.
I recommend you open an AWS Support case with them and provide the invalid sessionIDs you're receiving from the service call.

Ok thanks!


chore(control, planning): add maintainers

Description
add maintainers of following mofules:

control_performance_analysis
external_cmd_selector
joy_controller
obstacle_collision_checker
scenario_selector

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

write release note.


Explicitly call Python.

Should fix #8.
I have absolutely no experience with Kotlin, but it does compile and run under Linux... :-)

And now I tested in under Windows too. It seems like rust-android-gradle works under Windows too now.

it still won't work for Windows users

Heh, I literally tested this under Windows. Afaik, the .exe suffix is optional under Windows.

Missed this, but IIRC this is controlled by the windows PATHEXT environment variable, which includes .exe by default (and by all sane non-default configurations), so you shouldn't need the .exe unless something weird is going on.


Make sure the input reader is closed after loading an image from file.

Signed-off-by: David Calavera<EMAIL_ADDRESS>

test failures :(

@runcom compilation problem you mean :stuck_out_tongue_closed_eyes:. Although it's not pretty, the return part probably should not be changed :wink:

yeah I meant compilation, sorry

I'm going to close this because it's going to conflict with #18472 and I don't think it's worth the rebase for now. I can reopen it if that PR doesn't ship.


Text Protocol Formatter

Part of #174
Provides the code for reading/writing message batches using the "Text" protocol defined in the spec. It's not wired up to anything, just wanted to get it in and have tests. Next I'll do binary.

OK, I've reworked it to use the new Formatting/Parsing APIs. I didn't end up using TextFormat because it seems somewhat difficult to adapt to this purpose, when all I really need is an indicator of two different types. I also don't like data types like Message having their formatting/parsing logic inside the type, I'd rather that logic be extracted elsewhere.
The MessageBatchFormatter is the primary entry point into the parse/format APIs.

Updated. I removed the batch formatting logic since it's not really a pure formatting/parsing concern, there are things like incomplete messages (a message with endOfMessage being false, which must be buffered and assembled before transmission because unlike WebSockets we don't support that in this protocol, by design), or insufficient buffer space and partial writes/reads to be considered.

Investigating the travis failure, it appears to be unrelated (it'll need a fix in dev obvs.)

@davidfowl I'd like to close this out today if possible. Already working on the binary one and nearly ready to submit that one :).

:clap:


Add custom label buttons and test

I put buttons label into options like {'buttons': {'ok':'Ok Label','cancel':'Cancel Label'}}.
I had an issue into test comparing html template : the one loaded from file system have a "newline" that I'm not able to find and remove using text editor (vi, vim, oni, atom..). My solution was add a function that remove the new line before comparing html.

Coverage remained the same at 100.0% when pulling 4eda6a0a6ce8c8248537819ba49e722f8ed0328a on paologf:ButtonsLabel into a721d2ba4feb4a54f6c5897ada665b7630d736c0 on coderaiser:master.

Coverage remained the same at 100.0% when pulling 4eda6a0a6ce8c8248537819ba49e722f8ed0328a on paologf:ButtonsLabel into a721d2ba4feb4a54f6c5897ada665b7630d736c0 on coderaiser:master.

Thank you :), merged as 6cf7fc67. Landed in v3.1.0


Contradictory result about 3D CNN

First of all, thanks to the author's selfless dedication, your code is well written!
I have implemented two versions of 3D CNN(separable 3D and P3D) and want to calculate their FLOPs. I found that the FLOPs of separable 3D is much smaller than P3D version using your code (283.237G FLOPs VS 3.246T FLOPs). But when I calculate the inference time averaging in 100 forwards, the P3D version is much faster (0.8194s VS 0.3823s). Is there any problem when calculating computation of 3D CNN?

Can u share some example code?  THOP haven't considered much about 3D models, so some details might be missing.

Thanks for your quick reply! Can you provide your email address? I can provide you with network definition files and test code.

I recommend pasting here directly as it would help other people in the future.

OK. Thanks for your kindly help!
test.zip


Verifier implementation using an external service (based on #122, supersedes #223)

This implementation calls out the external Microservice ipaas-verifier for doing the verification of a connector and has the following advantages over the locally process forking implementation of #223:

Much faster because of an already 'prewarmed' running ipaas-verifier service
Does not eat up memory and CPU resources by starting external Maven processes which are also quite network intensive since a lot of dependencies needs to be downloaded
No local process management required
Simpler implementation
Doesn't need to call an external CLI command
Standard REST / Microservice technology
Supports a the full set of data that a Camel ComponentVerifier can return, in addition to code and description

Parameters which are affected by an error
Attributes holding additional error information (i.e. whether a parameter is missing or an unexpected parameter has been send.

Uses always both scopes: First verify the PARAMETERS before checking the CONNECTIVITY. Only if the initial parameter validation succeeds, a the real connection is called. This result in much faster responses if the given parameter set is unsound.
Does not add a GAV to the connector, which is factual wrong. GAVs are connected to Actions, a connector might reference an Action as its 'default verification action', but must not duplicate the (possibly contradicting) GAV of an action. Currently this association happens on the ipaas-verifier side. This had to be changed anyway.
Supports twitter, salesforce and http instead of only twitter (however could be easily added).

The disadvantage of this approach is, that it is currently much simpler and supports three connectors: twitter, salesforce and http which are pre-started in the service. There is no dynamic loading of additional connectors, since this is not in the scope of the current Sprint. The setup is however expandable to deal with dynamic connectors, too.

The LocalProcessVerifier code is still present as reference, however due to changes in the model does not fully work and needs to be adapted (e.g. not to use a connector GAV but an action GAV and adapt to the interface of the Verifier). However I doubt that we should add efforts into this for now. Actually I would recommend to remove the code as it has served its purpose of a POC well but might not be needed anymore right now.
The templates has been adapted, the image builds has been setup and the Twitter validation has been tested end-to-end (starting from ipaas-rest, ipass-ui integration is still pending). For testing the Salesforce verification I would need proper credentials for it.

Just want to make sure I highlight the long term problems that with running the verification code in process can encounter:
(1): The verification action code has to be compatible with all classes found in the classpath of the ipaas-verfifier.
(2): We need to make sure no system levels secretes are loaded into the ipaas-verfifier pod, as end user code will have access to it.
(3): We run the risk of an evil user defined connector stealing secrets that are being verified by another connector.

I completely agree with your remarks and we should carefully track them.

(1): The verification action code has to be compatible with all classes found in the classpath of the ipaas-verfifier.

As briefly mentioned in the ipaas-verifier README the goal is to extract the hard coded calls into external, pooled pods which only run version of one (or multiple) connectors.

(2): We need to make sure no system levels secretes are loaded into the ipaas-verfifier pod, as end user code will have access to it.

That's true (also for the forked approach since the filesystem is accessible here as well). Plus we need to ensure to secure the communication between ipaas-rest and ipaas-verifier (it's still http, although not exposed to the outside). For the final pod-pool approach however, the user code is run completely sandboxed within those pods with no secrets mounted.

(3): We run the risk of an evil user defined connector stealing secrets that are being verified by another connector.

Not really sure if understand this right, but even now the credentials coming in are not stored anywhere but coming in as input params and used for the check statelessly. At least for the connectors that we provide, we are save. Custom connectors are supposed to run in external pods as sandboxes.

The goal of this sprint is to provide a decent (== speedy) UX for validation of Twitter and Salesforce connections. We can add better isolation later if we target custom connectors (which is better than removing / changing stuff latter which nearly never happens if some time has passed)
@chirino Thanks a lot for the feedback! 'would be awesome if we could agree how to proceed today so that we could merge one of the PRs.

In general I do like the idea of spinning a new pod per user to do validation as that isolates the damage he can cause, But I'm not sure what's the plan to deal with the classpath issue.  Perhaps long term we should move the local process validator logic into the ipaas-verifier, and any unblessed (connectors we are not sure if they are compatible or not) get validated via forking?

Oh and by issue (3) I meant that I could craft a component which figures out how to install intercept subsequent HTTP requests to the verifier.  Perhaps via a servlet filter.

Yes, that could be an option, too. But I'm not have any idea how a custom connector should be realized at all.

How does an expert user define a customer connector ?
How is the custom connector build and made available for the verifier ?
How will a custom connector be made available for the UI ?
How enters the custom connector the DB ?
How are the custom connectors separated for each tenant ?

I think these are all question which should be tackled before thinking about how to execute such a custom connector. My gut feeling is that the answers to those questions will guide us in a certain direction we don't yet fully know. Let's do first things first.

I'm not too worried about (3): the SaaS solution will use only curated connectors, not user-provided. Self-managed deployments will be up to the user to ensure they review connectors for deployment.
I would prefer to go with this PR's approach right now. I haven't reviewed code - would appreciate it if @KurtStam or @iocanel could do so please.
The sooner we can get to the pooled pod approach the better, but this works for now.

LGTM

[merge]

[merge]


Calculation of Disconnection/Drop to Zero Percentage for different voltage ranges

Disconnection/Drop to Zero percentage for each zone at different voltage ranges are calculated. The percentage disconnection for low voltage conditions (Vmin<180V) is separately analyzed where the voltage condition prevails 5s or 10s or more before disconnection.
A bar chart is added in GUI as an output.

I am closing this pull request as the new pull request for "voltage visualization" was merged to the master repository.


Implement JSON metadata support for AsciiSignalIO (cf #579)

The JSON file currently has a parameter named filename. @apdavison  mentioned that:

The idea was to allow cross-checking, to make sure that the correct data file is paired with the correct metadata file. This cross-check is not implemented at the moment, however.

On further discussion, it was identified that there might be occasions (quite commonly; e.g. in a batch of data files) when we want to use the same metadata JSON to handle all the files in that batch. @apdavison  therefore proposed to make this an optional parameter.

This is OK for me.
tell me when you want I merge it.

It's ready to merge, please go ahead.


Additional rock textures

Added additional Rock Textures and updated directories

Not changed anything that should matter, just added some directories but would you mind having a look over to be sure haha tyty <3

Fab I'll look at this by the end of the day!


Create void fulfill function

Hello everybody
I find it annoying to have to add a Void value every time I need to fulfill a Void promise. It can also be confusing how to fulfill Void promises for people who are not familiar with how to instantiate a Void value.
This PR would allow users to replace this syntax
resolver.fulfill(())

with
resolver.fulfill()

Thanks!

Many thanks 👍🏻 I'm surprised we didn't already add this… much like yourself!


Spaces in property name get screwed up

When adding a description of a flow with spacing is out of wack.

Thank you for your interest in Threat Dragon.
This issue is in owasp-threat-dragon-core and is fixed in commit f30c579f309d300389d2f00791afe17a48b9e5d0 it will be merged into the next release of desktop.


Make Cupertino page transition linear during drag

So that the page's edge would track the finger's position very precisely.
#8726

BTW this was checked in when the tree was red. Please avoid checking in when the tree is red.

Got it. Watch for the red light strip


URLs not accepted in URL field (raised this before - seems to be closed but still issue)

This URL is not accepted again: https://www.commonspace.scot/articles/9846/will-they-deliver-six-commissioners-proposed-advance-land-reform
I've raised this issue before, don't see it here anymore so I guess someone closed it without it being actually fixed.

if you don't tell me where it's not going to be fixed....

It is everywhere, in all URL fields. The previous bug with the exact same problem I described in thorough detail. Then it was a URL field in an organizational profile, this time it was a news item. I am not going to repeat every bug that I report after it's been (unjustly apparently) deleted.
I am showing you the kind of URL the fields reject, and since it's apparently in every content type where we add a URL, it is a general behavior and it needs to be generally fixed.

Duplicate. Move interesting content to #112


Get server access from UVG

Goal: Get access for scripts and analysis

Update 8/9: UVG said we should get access within the next two days


Could we please have some Javadocs?

Most of the classes/interfaces/methods in this library have no Javadoc. This leaves my to guess what they are actually doing.
There is a usage example in the README, but it doesn't say what it is doing.
Could at least the most important classes/interfaces like AccessTokenBuilder, AccessTokens, AccessTokenConfiguration get some Javadocs for the important methods?

We welcome PRs :smile:

@hjacobs The problem is that for writing documentation one first has to know what these things are supposed to do. Due to missing documentation I don't know.
I guess I could try to reverse engineer this from the source code (I did this years ago for JSch), but this would likely take more time from me than from someone who wrote the code, or who designed the library.

Yes, documentation for this lib is a hot topic. I just met with @LappleApple and @jbellmann last week to discuss next steps we want to take, to improve this. There is the plan to add more test-cases that demonstrate usage scenarios, but also to add more docs, of course.

@harti2006 @jbellmann I just updated the README based on my chat with Andre and Joerg, but I think it still lacks what you're looking for, @ePaul. Here were the follow-up "to-do's" from Andre+Joerg:
need to add:
-- main dependency to the credentials directory and credential files, or do a small unit test to show how to use it
-- Under ‘Usage,” example with two different tokens with two different scopes. It’s a minimum configuration.
-- a test case with the “test credentials” in JSON files, that uses the normal setup, and mock the OAuth provider with some mock REST service
-- show how we use it with one example
-- document more config options

@ePaul @hjacobs @jbellmann @duergner @harti2006 Hm, so, these Javadocs have been on the request list for some time. What can we do to get them created? :)

They are merged to master. I suppose we would just have to create a release to get them pushed to Central

👍 to that, @duergner ...

Fixed with latest release.


[Dynamic Regions] raft-engine with v2 is slower than baseline due to append flow on GCP pd-ssd disk

Feature Request
Is your feature request related to a problem? Please describe:

Append flow impact the write duration
Describe the feature you'd like:

We need to enhance raft-engine, to eliminate or at least mitigate the performance penalty from the append flow.
Raft-engine should use an alternative approach to handle redo logs by requesting N static log files of M GB each to be continuously reused in a circular fashion. The completion of applying a log to a certain file can be safely identified and overwritten by indexing.
Describe alternatives you've considered:

no workaround
Teachability, Documentation, Adoption, Migration Strategy:

TiKV Config on GCP
      tikv:
        nodeSelector:
          node.kubernetes.io/instance-type: n2-standard-32
        storageClassName: premium-rwo
        requests:
          storage: 4096Gi
          cpu: 31000m
          # memory: 116Gi
        limits:
          cpu: 32000m
          memory: 64Gi
        replicas: 3
        baseImage: us-west1-docker.pkg.dev/qa-infra-dev/qa/tikv
        storageVolumes:
          - name: log
            storageClassName: local-path
            storageSize: 10Gi
            mountPath: /var/lib/tikv/log
          - name: raft-pv-ssd
            storageClassName: premium-rwo
            storageSize: 500Gi
            mountPath: /var/lib/raft-pv-ssd
        topologySpreadConstraints:
          - topologyKey: topology.kubernetes.io/zone
        config: |
          # FIXME
          # Baseline
          # [cdc]
          #   min-ts-interval = "2s"
          # [coprocessor]
          #   region-split-size = "256MB"
          # [raft-engine]
          #   dir = "/var/lib/raft-pv-ssd/raft-engine"
          #   enable = true
          #   enable-log-recycle = true
          #   purge-threshold = "10GB"
          # [raftstore]
          #   pd-heartbeat-tick-interval = "180s"
          #   raft-max-inflight-msgs = 1024
          #   store-io-pool-size = 1
          #   store-pool-size = 3
          # [resolved-ts]
          #   advance-ts-interval = "10s"
          # [rocksdb]
          #   max-background-jobs = 12
          # [rocksdb.defaultcf]
          #   compression-per-level = ["zstd", "zstd", "zstd", "zstd", "zstd", "zstd", "zstd"]
          # [server]
          #   grpc-concurrency = 8
          # V2
          [raft-engine]
            dir = "/var/lib/raft-pv-ssd/raft-engine"
            purge-threshold = "200GB"
            prefill-for-recycle = true
            enable-log-recycle = true
          [raftstore]
            store-io-pool-size = 4
          [rocksdb]
            write-buffer-limit = "8GiB"
            [rocksdb.defaultcf]
              compression-per-level = ["lz4", "lz4", "lz4", "lz4", "lz4", "zstd", "zstd"]
            [rocksdb.writecf]
              compression-per-level = ["lz4", "lz4", "lz4", "lz4", "lz4", "zstd", "zstd"]
          [server]
            snap-io-max-bytes-per-sec = "50MiB"
          [storage]
            engine = "partitioned-raft-kv"
            reserve-space = "0MB"

/severity critical

ON GCP pd-ssd 500GB, the IOPS limit is 50 K/s.
write_iops: (g=0): rw=randwrite, bs=(R) 4096B-4096B, (W) 4096B-4096B, (T) 4096B-4096B, ioengine=libaio, iodepth=64
fio-3.27
Starting 1 process
write_iops: Laying out IO file (1 file / 10240MiB)

write_iops: (groupid=0, jobs=1): err= 0: pid=516: Wed Mar 29 03:35:17 2023
  write: IOPS=50.3k, BW=197MiB/s (206MB/s)(11.5GiB/60001msec); 0 zone resets
    slat (usec): min=2, max=13477, avg=16.76, stdev=16.59
    clat (usec): min=387, max=23794, avg=1253.67, stdev=376.72
     lat (usec): min=398, max=23806, avg=1270.54, stdev=379.98
    clat percentiles (usec):
     |  1.00th=[  717],  5.00th=[  848], 10.00th=[  938], 20.00th=[ 1045],
     | 30.00th=[ 1123], 40.00th=[ 1172], 50.00th=[ 1237], 60.00th=[ 1287],
     | 70.00th=[ 1352], 80.00th=[ 1418], 90.00th=[ 1532], 95.00th=[ 1663],
     | 99.00th=[ 2212], 99.50th=[ 2638], 99.90th=[ 4621], 99.95th=[ 5932],
     | 99.99th=[14877]
   bw (  KiB/s): min=160697, max=256929, per=100.00%, avg=201545.38, stdev=23677.66, samples=120
   iops        : min=40174, max=64232, avg=50386.19, stdev=5919.43, samples=120
  lat (usec)   : 500=0.01%, 750=1.67%, 1000=13.77%
  lat (msec)   : 2=83.01%, 4=1.39%, 10=0.14%, 20=0.02%, 50=0.01%
  cpu          : usr=7.96%, sys=83.88%, ctx=57782, majf=0, minf=58
  IO depths    : 1=0.0%, 2=0.0%, 4=0.0%, 8=0.0%, 16=0.0%, 32=0.0%, >=64=100.0%
     submit    : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.0%, >=64=0.0%
     complete  : 0=0.0%, 4=100.0%, 8=0.0%, 16=0.0%, 32=0.0%, 64=0.1%, >=64=0.0%
     issued rwts: total=0,3020986,0,0 short=0,0,0,0 dropped=0,0,0,0
     latency   : target=0, window=0, percentile=100.00%, depth=64

Run status group 0 (all jobs):
  WRITE: bw=197MiB/s (206MB/s), 197MiB/s-197MiB/s (206MB/s-206MB/s), io=11.5GiB (12.4GB), run=60001-60001msec

Once the memtable change is done, let's see if it will help to reduce the rewrite traffic and ultimately help raft-engine performance

we also see the performance impact on IDC using nvme disk when the rewrite rewrite and rewrite append flow kick in.


Try to fix running tests in IntelliJ

Thank you for opening a pull request and contributing to AsciidoctorJ!
Please take a bit of time giving some details about your pull request:
Kind of change

[x] Bug fix
[ ] New non-breaking feature
[ ] New breaking feature
[ ] Documentation update
[ ] Build improvement

Description
What is the goal of this pull request?
It seems like it is not possible anymore to run or debug a single test in IntelliJ anymore since very recently.
This PR tries to fix this.
How does it achieve that?
My last status was that IntelliJ complained about the annotation org.osgi.annotation.Export not being available.
Changing the dependency type from compileOnly to implementation seems to solve this situation.
(Don't ask me why, I don't understand why compileOnly is not good here either)
@abelsromero As you were also facing this issue, could you please test this?
Issue
If this PR fixes an open issue, please add a line of the form:
Fixes #Issue
Release notes
Please add a corresponding entry to the file CHANGELOG.adoc

Changing the scope of the dependency makes Gradle add a dependency with runtime scope to the pom.
I am not sure right now if this is acceptable or not, right now we don't have a dependency from asciidoctorj-api to org.osgi:osgi.annotation.

It seems like adding a testRuntimeOnly dependency on osgi is sufficient to run tests in IntelliJ.
Plus it does not have the "downside" of adding a dependency to the pom.

Merging for now since we don't change the dependencies with it.
If we still feel that we should expose an optional dependency on osgi.annotation we can still add it later.

Did nothing to me :shrug: more the opposite

Hmmm.. is it complaining about everything in red?
The last missing piece for me to run a test was the osgi annotations.

Interesting enough dependencies are ok in the Gradle pannel, but in the code complains it cannot find junit. But in any case I can stll do a "Run all project tests" and debug with that. It's really odd.


Automated Package Upgrade

:tada: This PR is included in version 7.2.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unregister package "package-name", sorry created it by mistake

bower search package-name
Search results:
package-name https://github.com/tempusdominus/bootstrap-3.git
I apologies, used the wrong name for the package

no problem removed it


Enable manual run of build & test github action

WHAT
Enable manual run of build & test github action.
WHY
For some reason one of my pull requests did not trigger the github action, so I'd like to trigger it manually.
Alternatively to this solution, I could push an empty commit.
HOW
Add workflow_dispatch event to build.yml.
Checklist :clipboard:

[N/A] Add explicit visibility modifier and explicit return types for public declarations
[N/A] Run spotless check
[N/A] Run tests
[N/A] Update metalava's signature text files

just noticed that this might not work for the scenario described as github does not seem to list branches from forked repos


scale-transition with scale-out/scale-in not working within .halfway-fab

Description
The scale-transition effect using scale-out or scale-in does not work if the FAB is positioned with class halfway-fab, as seen in http://materializecss.com/cards.html.
Repro Steps
Add class scale-transition to the FAB of the card in http://materializecss.com/cards.html, section "FABs in Cards", add scale-out, then remove scale-out and add scale-in, you will see no transition effect. Remove class halfway-fab and apply the classes again. Ok, the button is not positioned as desired, but the transition will work.
See also http://codepen.io/Chocoliver/pen/dvJyrM.
Maybe there is a workaround for positioning?
Thank you in advance!

Fixed in f4693a9

Thank you very much for your fast reply! Let's test it...

Please close if you can confirm, that the fix is working

It works like a charm, thank you very much!


data-sveltekit-preload-data="hover" triggers navigation event

Describe the bug
I'm using data-sveltekit-preload-data="hover" on my body tag. This should trigger a page preload on hover, but instead it triggers an actual navigation event. The bug has appeared in the 1.27.6 version, 1.27.5 is fine.
Reproduction
Please see https://d3zcv1ysidbcu.cloudfront.net which is running 1.27.6. Hover over the blog link in the menu and it will actually navigate to the blog page.
In contrast please try the same on https://passlock.dev, which is running 1.27.5. This behaves as expected.
Unfortunately I can't reproduce the issue with the 1.27.6 version when running vite preview so it could be somehow related to the production adapter. I'm using the svelte-kit-sst adapter.
Logs
No response
System Info
System:
  OS: macOS 14.1.1
  CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
  Memory: 19.56 GB / 64.00 GB
  Shell: 5.9 - /bin/zsh
Binaries:
  Node: 18.15.0 - ~/.nvm/versions/node/v18.15.0/bin/node
  Yarn: 3.4.1 - ~/.nvm/versions/node/v18.15.0/bin/yarn
  npm: 9.6.6 - ~/.nvm/versions/node/v18.15.0/bin/npm
  pnpm: 8.10.5 - ~/.nvm/versions/node/v18.15.0/bin/pnpm
Browsers:
  Chrome: 119.0.6045.159
  Safari: 17.1

Severity
annoyance
Additional Information
No response

We need a minimal reproduction as code, not just your deployed site. Since this is only appearing for you when using a third-party adapter, some sort of investigation into whether this is a bug in SvelteKit or in the adapter would be helpful.

Ok, I'll try to create a minimal reproducible example and in the process hopefully narrow down the source of the problem.


PR: Fix zip archive in macOS app bundle so that pkg_resources will find entry points

Fixed zip archive in macOS app bundle so tat pkg_resources will find entry points
Movee patching function to setup file.
Eliminated inadvertent additional logging handler.
Fixes #17546
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @mrclary

There is also a temporary fix for #17551 and #17552 (installer only)

@ccordoba12 @dalthviz , macOS app tests should pass now with this PR.


Simplify Dockerfile

Remove upgrade of all packages since fixing Lows and Moderates is really
not necessary and just adds time to the build process. We can trust Red
Hat's SLAs around addressing meaningful vulnerabilities in the UBI images.
Remove contact address for the image owner, the team name should be
sufficient to identify the image internally. The distribution scope of this
image is private anyway.
Install syft directly from the URL without an extra curl step.

Remove upgrade of all packages since fixing Lows and Moderates is really not necessary and just adds time to the build process. We can trust Red Hat's SLAs around addressing meaningful vulnerabilities in the UBI images.

Mostly LGTM, but does it really add that much time to the build to check for security updates? "Only rebuilt every six weeks" seems rather infrequent, and even if they're only Moderates and lows, I'd feel more comfortable running an image that doesn't have any bugs.

Remove upgrade of all packages since fixing Lows and Moderates is really not necessary and just adds time to the build process. We can trust Red Hat's SLAs around addressing meaningful vulnerabilities in the UBI images.

Mostly LGTM, but does it really add that much time to the build to check for security updates? "Only rebuilt every six weeks" seems rather infrequent, and even if they're only Moderates and Lows, I'd feel more comfortable running an image that doesn't have any bugs. And I don't imagine we'll be rebuilding the image that often anyway.

Discussed this offline. It's OK to rely on image refreshes only.


Embedding Missing Thumbnails with AtomicParsley

When YouTube can't return any thumbnail through "maxresdefault", it returns a 404 which is unhandled by youtube-dl since it tries to embed a thumbnail which cannot, and has not been downloaded to the file, crashing AtomicParsley.
[debug] System config: []
[debug] User config: [u'--add-metadata', u'-o', u'%(upload_date)s - %(title)s.%(ext)s', u'-cf', u'bestvideo[ext=mp4][width<=1920]+bestaudio[ext=m4a]', u'--embed-thumbnail', u'--embed-subs', u'--exec', u'IFS=$n && j={} && k=`expr length $j` && l=`expr $k - 15` && notify-send "YouTube DL" --icon /usr/local/bin/icon.png "`expr substr $j 12 $l`"']
[debug] Command-line args: [u'https://www.youtube.com/playlist?list=PLARhCYccw3tb0ewDzW4eE28DNc9V_oIZE', u'-n', u'-v']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.06.15
[debug] Python version 2.7.9 - Linux-3.19.0-20-generic-x86_64-with-Ubuntu-15.04-vivid
[debug] exe versions: avconv 11.2-6, avprobe 11.2-6
[debug] Proxy map: {}
[youtube:playlist] Downloading login page
[youtube:playlist] Logging in
[youtube:playlist] PLARhCYccw3tb0ewDzW4eE28DNc9V_oIZE: Downloading webpage
[download] Downloading playlist: Immediacy
[youtube:playlist] playlist Immediacy: Downloading 6 videos
[download] Downloading video 1 of 6
[youtube] Downloading login page
[youtube] Logging in
[youtube] 49srojjrd4I: Downloading webpage
[youtube] 49srojjrd4I: Extracting video information
[youtube] 49srojjrd4I: Downloading DASH manifest
[youtube] 49srojjrd4I: Downloading thumbnail ...
WARNING: Unable to download thumbnail "https://i.ytimg.com/vi/49srojjrd4I/maxresdefault.jpg": HTTP Error 404: Not Found
[download] 20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.mp4 has already been downloaded and merged
[ffmpeg] Adding metadata to '20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.mp4'
...

...
[ffmpeg] There aren't any subtitles to embed
[atomicparsley] Adding thumbnail to "20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.mp4"
[debug] AtomicParsley command line: AtomicParsley '20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.mp4' --artwork '20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.jpg' -o '20150614 - Product vs Packshot  - Bloomex 3 _ The Checkout.temp.mp4'
ERROR:
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1548, in post_process
    files_to_delete, info = pp.run(info)
  File "/usr/local/bin/youtube-dl/youtube_dl/postprocessor/embedthumbnail.py", line 73, in run
    raise EmbedThumbnailPPError(msg)
EmbedThumbnailPPError

Thanks for the report. In the next version thumbnail embedding is skipped if thumbnail downloading fails.


Dont see option to add in integrations

Using latest version of home-asssitant, try to add but the option isnt in there
https://imgur.com/a/NC67iwv

Refresh your browser cache.

dude, i tried ingognito mode....

How did you install the component?


filename-case rule false positive

spec/iss47-spec.js:9:1
  ✖   9:1   Filename is not in kebab case. Rename it to iss-47-spec.js  xo/filename-case

Should we maybe make an exception here? I don't see why we would enforce the number to be independent.

It should accept either variant.

I raised a similar case in Gitter https://gitter.im/sindresorhus/ava?at=5726d196e10a59c06107afb2

test/configs/i18n.js - Filename is not in kebab case. Rename it to i-18-n.js  xo/filename-case

...files named i18n.js, l10n.js, or a11y.js numeronyms could be ignored, at least in my head I treat these as a single word, and i-18-n.js looks pretty weird

Nice, looks like #28 takes each of the above into account

Yeah, I saw it pass in the Gitter, so I added it ;)


[ENGOPS-3650] change Pentaho to Hitachi Vantara and update copyright date

WARNING!  FOR MERGE BY PROCESS ONLY PLEASE!

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'ui,plugins,core' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'ui,plugins,core' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui,plugins,core' -P '!assemblies' -amd build-helper:remove-project-artifact

Build Sensitive Files Changed
  plugins/lucid-db-bulk-loader/build-res/subfloor.xml
  plugins/ssh2/build-res/subfloor.xml
  plugins/star-modeler/build-res/subfloor.xml

Changed files
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_es_AR.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_es_ES.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_fr_FR.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_it_IT.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_ko_KR.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_nl_NL.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_pt_BR.properties
  core/src/main/resources/org/pentaho/di/i18n/messages/messages_pt_PT.properties
  plugins/lucid-db-bulk-loader/build-res/subfloor.xml
  plugins/lucid-db-bulk-loader/build.properties
  plugins/ssh2/build-res/subfloor.xml
  plugins/ssh2/build.properties
  plugins/star-modeler/build-res/subfloor.xml
  plugins/star-modeler/build.properties
  ui/src/main/resources/org/pentaho/di/ui/core/dialog/license/license.txt

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes.


MainLoop.NOTIFICATION_WM_FOCUS_OUT not properly triggered on Android

Godot version:
3.1 Alpha 3
OS/device including version:
Android 9.0/Nokia 7 plus
Issue description:
MainLoop.NOTIFICATION_WM_FOCUS_OUT is not called when window's focus is lost. Instead, it is called when the window's focus is returned, slightly before MainLoop.NOTIFICATION_WM_FOCUS_IN. This probably causes all sorts of negative side effects. One, that I noticed, is that if you're using audio in your game and you want that audio to mute when player switches to another app (and un-mute on first player's action, not immediately after MainLoop.NOTIFICATION_WM_FOCUS_IN), then you will notice that even though you muted your audio on MainLoop.NOTIFICATION_WM_FOCUS_OUT notification, there will still be a split second of your audio playing once the player returns back to your game.
Steps to reproduce:

Add audio to your game
Mute audio on MainLoop.NOTIFICATION_WM_FOCUS_OUT notification
Do NOT un-mute the audio on MainLoop.NOTIFICATION_WM_FOCUS_IN (let's assume you want to keep it muted)
Run the game on Android, switch to another app, return to your game and you will hear your audio for a split second, even though it should be muted

Minimal reproduction project:
notifications.zip

Can anyone still reproduce this bug in Godot 3.2.1 or any later release (e.g. 3.2.2-rc2)?

No answer, so closing. Please comment if you can still reproduce this issue as described above in Godot 3.3 or later.

Hi guys
I don't know if this is the right place to post this, but NOTIFICATION_WM_FOCUS_OUT is not working properly in android; is being triggered at the same time that NOTIFICATION_WM_FOCUS_IN when the application is resumed.
I'm trying to detect when the user exits the application (not by the "back" button but the "home" button)  or "minimized it"
Tested in Godot 3.3.2.stable on windows x64
Thank you

it seems fixed with 3.4.beta f80b15bebc
notification_test.zip

Im having the issue when running the game in Ubuntu. The game pauses on NOTIFICATION_WM_FOCUS_OUT, and doesn't unpause on NOTIFICATION_WM_FOCUS_IN. MainLoop and Node.
The code is something like:
if focus_in:
unpause
elif focus_out:
pause


Component isn't keyboard focusable

https://developer.salesforce.com/blogs/2020/01/accessibility-for-web-components


MAF alignment score

Hi,
I was wondering how is the MAF alignment score calculated exactly?
I was hoping to use score filtering to remove smaller/secondary matches.
Thanks,
Agnieszka

The MAF alignment score is simply the number of identical bases aligned together in an alignment. But it is defined differently in other methods.


Custom Transport receives "level" as a terminal colored string.

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: v14.17.0
Operating System? MacOS Monterey 12.0.1
Language? TypeScript 4.4.4

What is the problem?

The custom transport is receiving level as a colored string.
public override log(info: any, next: () => void): void {
  console.log(info)

  /* outputs
  {
    message: 'Websocket connected',
    level: '\x1B[32minfo\x1B[39m', // ❌
    timestamp: '2021-11-09T19:56:55.985Z',
    [Symbol(level)]: 'info',
    [Symbol(message)]: '2021-11-09T19:56:55.985Z [\x1B[32minfo\x1B[39m] Websocket connected'
  }
  */

  if (info.level === 'info') {
    // 💀 never happens 💀
  }

What do you expect to happen instead?
The level is a text string that can be used to determine what level it is.

I just ran into this too.


Add Contract estimate proxy.

Create an object on Contracts to proxy gas prices estimates for transaction functions.

Done.


Adding force orientation lock to Camera

Hi,
Sorry if this is the wrong way to ask for features,
But I do hope this feature is going to be implemented soon.
i.e choosing/forcing what orientation the camera should shoot in, either portrait or landscape or both.
Is there an alternative workable solution?
Thankyou and keep up the good work.
Hax

Cordova CI Build has completed successfully.
Dashboard - http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-camera/5/
Results

Windows 8.1 Store - Console Output - Test Results - Device Logs
Windows 10  Store - Console Output - Test Results - Device Logs
Windows 8.1 Phone - Console Output - Test Results - Device Logs
iOS - Console Output - Test Results - Build Artifacts
Android Mac - Console Output - Test Results - Build Artifacts
Android Windows - Console Output - Test Results - Device Logs

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 8.1 Store
Link
Link
Link

Windows 10  Store
Link
Link
Link

Windows 8.1 Phone
Link
Link
Link

iOS
Link
Link
Link

Android Mac
Link
Link
Link

Android Windows
Link
Link
Link

👍 for locking orientation of camera

I'm trying to find this feature documented somewhere, but cannot find it. Any pointers on how to use it? (for example, how can I lock the camera in landscape mode?). thanks

if if the orientation of the phone is Vertical ,
we should be able to force a landscape/horizontal target file or object.

Can anyone able to lock the camera rotation? then, please  let us know.

I cant find this feature too, anyone?

I don't think we have this feature in the plugin... Please correct me if am  wrong
I am also looking to lock the orientation to portrait mode  as I have my app locked in portrait mode
But opening the camera is messing things up and broking my UI


Can't succeed to build and run the project in the simulator !!

thanks for sharing this library.
but i faced some problem when i am going to run the project in the simulator.
it shows

Have you added the static library to your project as describe in the readme at step 3?

yes, already added

but still failed

@jf423 - that's weird! what version of the library and React Native are you using?

@brentvatne i think i am in the latest version.
it's keep showing the issue of RCTWebSocketDebugger.

@brentvatne i solve this problem.Turn out I just need to delete the link of RCTWebSocketDebugger in link libraries .
Thanks for your patient in my issue and your contribution in this resource.

Great :smile: thanks!


Add E2E test suite for cephfs

Added E2e test framework
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>Co-authored-by: Poornima G<EMAIL_ADDRESS>Note: Not up for review, need to optimize  the code

@poornimag PTAL

planning to use rook framework to deploy rook and to create cephfs and rbd pools
need to do code refractor in (https://github.com/rook/rook/blob/master/tests/integration) ( move some functions to a new go file) in rook to import rook test framework in ceph-csi
@travisn is this acceptable in rook?

Finally, being able to run the test for cephfs and rbd
Ran 2 of 2 Specs in 487.358 seconds
SUCCESS! -- 2 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestE2E (487.37s)
PASS
ok  	github.com/ceph/ceph-csi/e2e	487.491s

csi-test.log Attaching logs for more info

https://github.com/rook/rook/pull/3192

@Madhu-1 I would suggest squash commits of code in one and vendor updates in another commit, that will make it easy to review .

@humblec @ShyamsundarR PTAL

i will add  a readme and remove DNM  label later

Added readme @humblec @ShyamsundarR PTAL

@Madhu-1 Thanks for adding README and adding the test suite. Lets have this as the starting point and do rest or improve it later.!


Not able to fetch pick a color value

Hello,
I am using pick a color for one of my form.I have used it as -

But when i am trying to fetch the value of color picker on click of submit button, then it is showing me blank or empty string.I am fetching the value as -
alert($("#colorPicker").val());

You can add this to the top of the code module:
$.fn.getColor = function () {
return $(this).find('input').val();
}
then call by $("#colorPicker").getColor();


nested json object not supported when convert json to attr_container

when send request with a json file payload.  nested object in the json file is not supported.  What's the reason? is it difficult to calculate the size of the object?

We may use cbor to replace attr_container in the future.


Change old hiredis functions to make it work with hiredis 1.0.0

#46 This removes the following old function aliases, use the new name now: (See hiredis Changelog)
redisReplyReaderCreate  => redisReaderCreate
redisReplyReaderCreate  => redisReaderCreate
redisReplyReaderFree    => redisReaderFree
redisReplyReaderFeed    => redisReaderFeed
redisReplyReaderGetReply    => redisReaderGetReply
redisReplyReaderSetPrivdata => redisReaderSetPrivdata
redisReplyReaderGetObject   => redisReaderGetObject
redisReplyReaderGetError    => redisReaderGetError

Thanks! In a couple of weeks I should finally be able to find the time to work on these kind of outstanding issues in phpiredis, just give me that much time and I'll be more than happy to merge your pull request :)

Is this still considered?  Also - is this backwards compatible?

Are there plans to integrate this? RPMlint prevents phpiredis 1.0 to be built because of 64 bit portability issues in phpiredis.c. Replacing the two functions does fix the issue.

I'm going to merge #52 in the next few days since it already comes with merge conflicts resolved, thank you anyway for your PR 👍


No way to set the location of an offline player

Currently there is only one way to alter the position of an offline player, which is by writing to the NBT's. As this is really unhandy and (IMO) unspongy, I would request another option to do this.
I've looked into the User class, which is the class that resembles an offline player. In this class there are ways to set/get the inventory,... but no way to alter the location of the player. If this is a deliberate choice, would someone explain to me why you chose not to?
Other contributions are welcome too! 😃

Is there any update on this? :)

@gabizou shouldn't this be data?

User#getLocation() and User#setLocation() would be nice.

Thought about this for a bit, we can't realistically use a Location since you can have a player in an unloaded world (let's think about multi-dimensions). At most we can provide a position and world id, but certainly not a Location for offline Users. And @Faithcaio I'd rather it not be data, but it was an intention long ago when first writing the Data API and @Zidane shot me down on the idea :P

so smth like setLocation(Vector3d, WorldProperties) WorldProperties getOfflineWorld Vector3d getPosition?

WorldProperties or would it be a dimension id :-S

Why world properties? I thought you could get a world or world properties or whatever you wanted with just a uuid, so why wouldn't we want to use a UUID? It doesn't make sense to me to use anything different (especially how world properties can be created without having an actual world of which to focus on setting a player location for a world that doesn't exist.

I guess UUID would be fine.
But UUIDs can also be created without having a world backing it up and the UUID is also not what is saved in the Player.

What is saved on the player for their location?

NBTTagList nbttaglist = compound.getTagList("Pos", 6);
this.posX = nbttaglist.getDoubleAt(0);
this.posY = nbttaglist.getDoubleAt(1);
this.posZ = nbttaglist.getDoubleAt(2);
this.dimension = compound.getInteger("Dimension");

how about this @gabizou ? ^


Add more unit tests

With this PR, we have:

updated our README and hopefully, it's more informative now.
add more unit tests for helpers and bksecrets.
show a nice coverage for the modules at the end of the test run, currently it's:

..............................
----------------------------------------------------------------------
Ran 30 tests in 0.032s

OK
Name                                    Stmts   Miss  Cover   Missing
---------------------------------------------------------------------
bk_ssm_secrets/__init__.py                  0      0   100%
bk_ssm_secrets/__main__.py                 14     14     0%   1-28
bk_ssm_secrets/bksecrets.py                64      0   100%
bk_ssm_secrets/config.py                    4      0   100%
bk_ssm_secrets/helpers.py                  52      1    98%   24
bk_ssm_secrets/ssm_parameter_store.py      99     79    20%   40-44, 47-65, 68-94, 98-107, 110, 113-115, 118-146, 149-153, 156, 159, 162, 165
---------------------------------------------------------------------
TOTAL                                     233     94    60%

We have a plan to add more unit tests for __main__ and replace ssm_parameter_store.py.

cc @vgrigoruk, I have changed a little bit of logic in ACL handling here, hope that will not affect you.


Allow relative paths in IvfModel.save() and PqModel.save()

Tried to use a relative path in this API, and it interprets it as an absolute path:
ivf.save("ivf.index")

OSError: LanceError(IO): failed to shutdown object writer for ivf.index:
  Generic LocalFileSystem error: Unable to open file /ivf.index#1:
    Read-only file system (os error 30), /Users/runner/work/lance/lance/rust/lance-io/src/object_writer.rs:271:17

I suspect the same issue exists for the APIs:

IvfModel.load()
PqModel.save()
PqModel.load()

It would be nice if relative paths worked in all places. It would be even nicer if Path objects were accepted as well.
We should modify the code to accept a relative path, while still supporting absolute and remote paths (like s3://bucket/path).

+1.  I think the issue might be even deeper (in LanceFileWriter / LanceFileReader) and it would be good to fix those APIs.

Oh right. Yes I think those APIs also require absolute paths, and it would be good to fix those too.


docs: cleaned up docs and fixed typos

I noticed that you have been implementing deletion rules. So I'd advice you to hold off on merging this pull request (I will be making more changes once you have fully implemented them.)

I think it would help if you could provide a list of characters that need to be escaped (I think this will help other users) and any other special characters such as \n etc.
Also, I am a little confused by how the auto space and auto formatter works. I've tried it with Korean, but it doesn't seem to work. (Perhaps you could list the supported languages)
If you have time, please write a short explanation of how these things work; it would help a lot in writing the docs (and for me and others to fully know what this plugin is capable of).


Fix the Active Directory strict role mapping

Summary
Active directory settings were setup as strict, which was removing manually applied roles upon login. This fixes the login to leave manually applied roles alone except for the Drupal_Admin, which is the only group that should be maintained within AAD.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
yes

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
yes

Did you perform browser testing?
yes

Risk level
Low

Relevant links
RIG-136

Looks good once build passes
/approved


Improve Redirect Interceptor

What's in this PR?
The original PR by @ParkerK is here #208. However, due to lack of my attention over the time, it has been stale without me taking much care about it. 🙏 😢 😭
I think it might be best to revive it. And improve it a little along the way. With the suggestion from @SleeplessByte, I think we will have the right Redirect interceptor this time.
This PR should supersede #208. All credits go to him 🍺 🥂

@markGilchrist not at all, sir! I have fixed most of them but https://github.com/kittinunf/Fuel/pull/388#discussion_r202905881. Please advice 🥇

I wanna have @SleeplessByte to see this PR and hopefully say some few wise words 😄

There is one thing I haven't addressed which is the discussion in https://github.com/kittinunf/Fuel/pull/388#discussion_r203130180
I have created this #395 to support this.

I wanna get this thing rolling. Hopefully, @SleeplessByte is mildly satisfied when this. 🙃
Thanks so much for the reviews.

Does this work with FuelManager?


Buildscript

should we keep the LICENSE file ?  it was created when this repo was created, since we requested everything in this repo will be under Apache license.


Clarify that settings from the Anti-Spam policy are still honored

We should clarify that action taken on any message is determined by the setting configured in the antispam policy. Secure by default will not change that and messages with the high confidence phish action will just take effect as it normally would; although, we do recommend that tenants move to the quarantine experience.

@FrankPlawetzki Please mention which document you are referring to. Thanks

https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/secure-by-default?view=o365-worldwide

For clarification:
Although the documentation includes the sentence: "Email identified as high confidence phishing will be handled according to the anti-spam policy action." in my opinion it is not clear, that the anti-spam policy settings will still be honored and e.g. customers can still send high confidence phish to the junk mail folder instead of quarantine, although this is not recommended by Microsoft

@FrankPlawetzki Thank you for your feedback. You are correct,  We have added that part in the document -- Anti-spam policies are still honored as secure by default is not a full takeover of your available options for high confidence phishing messages in anti-spam policies. Although we recommend Quarantine, the other actions that have always been available are still available (Move to Junk Email folder or Redirect to an email address).
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @FrankPlawetzki Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.