fix(formly-form): added originalModel and formOptions to getFormlyFie…

…ldLikeLocals function
added originalModel and formOptions to getFormlyFieldLikeLocals function
issue : https://github.com/formly-js/angular-formly/issues/616

Current coverage is 96.00%

Merging #626 into master will not affect coverage as of 8042d2a

@@            master   #626   diff @@
=====================================
  Files           16     16
  Stmts         1150   1150
  Branches         0      0
  Methods          0      0
=====================================
  Hit           1104   1104
  Partial          0      0
  Missed          46     46

Review entire Coverage Diff as of 8042d2a

Uncovered Suggestions

+0.34% via ...alidationMessages.js#25...28
+0.26% via src/test.utils.js#38...40
+0.26% via ...s/formlyUsability.js#18...20
See 7 more...

Powered by Codecov. Updated on successful CI builds.

Awesome! Mind adding a test or two that verifies these changes work (and to prevent them from breaking in the future?)

Sure, why not? Some tips on how to go about it would be useful :)

Sure! :+1:
I'd probably put it in the section of tests that test hideExpression because that's one of the main places that leverages this feature. Find that here.
I would take the existing beforeEach and two it blocks and nest them in another describe with a title of something like: behavior when the model changes
Then I would create another describe block for your tests with the title of something like as a string. And then add an it with the message: should have access to almost all the same properties as an expressionProperties string expression.
Then in your test, you'll want to initialize scope.fields, scope.model, and scope.options (scope.options should have a formState object). And for scope.fields you can use the getNewField helper function (see other places for how that's used. It's pretty much just a helper so you can omit the template/type/etc.).
It may be a bit tricky because you can't really do assertions in string expressions, but the returned value of the string expression is assigned to the field's hide property. So you might try something like this:
scope.model = {}
scope.options = {formState: {}}
scope.fields = [getNewField({
  hideExpression: `
    options === options.data.field &&
    index === 0 &&
    formState === options.data.formOptions.formState &&
    originalModel === options.data.originalModel &&
    formOptions === options.data.formOptions
  `, // can't reliably test `formId` unfortunately
  data: {
    originalModel: scope.model,
    formOptions: scope.options,
  }
})]
scope.fields[0].data.field = scope.fields[0]

Then invoke compileAndDigest and assert that the scopes.fields[0].hide is true.
Let me know what you think or feel free to change whatever you like. Looking forward to seeing this. Thanks!

Awesome! I will get cracking on the tests right away :+1:

What is the field's data property actually used for? The documentation is a bit vague about it...

It's for anything you want. Because angular-formly validates all of the properties you provide, if you have any other things you need to provide, they either go in templateOptions (if it's specific for the template or type that you're using) or in data if it's just general data that you need to get your job done (like in this case).

Thanks. Just a couple more questions:

We can also pass a function to hideExpression to evaluate it, but there are no tests for it. Should I add another test for it?
Should I commit these tests to a separate commit or should I squash it with the previous commit into a single one?

I'm fine with either way. Feel free to add more tests in this pr, feel free to squash if you like.

If you could fix that comment and the name of that data property (to parentScope maybe), then this is great and I'll merge it. Thanks!

I have updated the PR with the changes you requested.

Perfect. Thank you!


(chaos-operator): Add unit test cases for function engineRunnerPod()

UNIT TEST
What happened:

Write the unit-test

What you expected to happen:

The test case should be written for function engineRunnerPod() of file https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller.go

Anything else we need to know?:

Issue related to Repository: https://github.com/litmuschaos/chaos-operator
Reference:- https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go

The chaosengine controller code has undergone significant refactor as part of effort to be compliant with the betterCodeHub coding standards.
A similar/corresponding unit test is available here: https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go#L32


Improper url validation

Improper url validation leads to error:
2021/05/04 17:00:45 Get "https://rust-lang.orglearn": dial tcp: lookup rust-lang.orglearn: no such host
It's better to use net/url to avoid this behaviour

Fixed in [1]


Documentation> "Legacy Select" link> "Not Found" error message is appearing upon clicking the link

Describe the bug
"Not Found" error message is appearing upon clicking the link "Legacy Select".
To Reproduce
Steps to reproduce the behavior:

Go to https://react-magma.cengage.com/version/2.5.7/api/select/#basic_usage
Click on "Legacy Select " link
Verify the error

Expected behavior
Appropriate page should appear upon clicking the link "Legacy Select".
Screenshots
https://somup.com/c3nXlIZRIQ
Desktop:
OS: [Win 10]
Browser [chrome]
Version [Chrome version: Version 98.0.4758.81 (Official Build) (64-bit)]

"Legacy Select" link is displaying as plain text in Selects page.
Selects Link : https://next--upbeat-sinoussi-f675aa.netlify.app/api/select/
See Screen-cast : https://www.screenpresso.com/=Q5FG
Hence, Reopening the ticket.
Thanks!

The link is no longer needed.  Please verify "legacy Select" is text only.

The issue is fixed.
See Screen-cast : http://somup.com/c36Ihrv6mB
Thanks!


On-premises deployment of your machine, step 7

I guess command to import log collector configuration should be run in elevated command prompt? It would be good to include this information in the article.

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

ID: e4f7c304-9777-e8d6-1e49-e80ea333c072
Version Independent ID: c659f6d0-9011-fd40-d33e-090a719e82da
Content: Roll out continuous reports for Cloud App Security using a Docker on Windows
Content Source: CloudAppSecurityDocs/discovery-docker-windows.md
Service: cloud-app-security
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

I am not going to go through all of these emails.
What are you all for then?? I was told to sit right here and wait for
everything!! I have been waiting for several years too!! I still want my
cornbread from Mrs. Kay.. Cornbread with cheese and bacon..the entire pan
is mine!! And one of her son's or daughter in law's must give it to me..I
don't know what Tawanna is supposed to be doing??  Why does her name keep
coming up in my information?? I don't actually know her that well..
Mrs. Heather Rocio Gonzalez Bonner
On Mon, Feb 4, 2019, 10:24 AM pawp81<EMAIL_ADDRESS>wrote:

I guess command to import log collector configuration should be run in
elevated command prompt? It would be good to include this information in
the article.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com
http://docs.microsoft.com ➟ GitHub issue linking.

ID: e4f7c304-9777-e8d6-1e49-e80ea333c072
Version Independent ID: c659f6d0-9011-fd40-d33e-090a719e82da
Content: Roll out continuous reports for Cloud App Security using a
Docker on Windows
https://docs.microsoft.com/en-us/cloud-app-security/discovery-docker-windows
Content Source: CloudAppSecurityDocs/discovery-docker-windows.md
https://github.com/Microsoft/CloudAppSecurityDocs/blob/master/CloudAppSecurityDocs/discovery-docker-windows.md
Service: cloud-app-security
GitHub Login: @rkarlin https://github.com/rkarlin
Microsoft Alias: rkarlin

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/CloudAppSecurityDocs/issues/96, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AbiHox7EV_wQO8yLWh9VhCv-v_yYKO9wks5vKF7YgaJpZM4ahjE8
.

Hi @pawp81 , thanks for your comment. In step 2, number 1, we say: Open a PowerShell terminal as an administrator on your Windows machine.
Is there some other information you thought you needed?
thanks

#please-close


Fails to Load on 1.21.2/3

Newest update breaks the shaders. Here is the error dump from client log when you load the pack.
[11:06:12] [Render thread/INFO]: Caught error loading resourcepacks, removing all selected resourcepacks
java.util.concurrent.CompletionException: java.lang.RuntimeException: Failed to load required shader programs:

minecraft:core/rendertype_entity_translucent (VertexFormat[Position, Color, UV0, UV1, UV2, Normal]): Could not find shader: minecraft:rendertype_entity_translucent (VERTEX)
minecraft:core/rendertype_armor_cutout_no_cull (VertexFormat[Position, Color, UV0, UV1, UV2, Normal]): Could not find shader: minecraft:rendertype_armor_cutout_no_cull (VERTEX)
at java.base/java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:315) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:320) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:722) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:482) ~[?:?]
at awb.a(SourceFile:68) ~[1.21.3.jar:?]
at bry.d(SourceFile:164) ~[1.21.3.jar:?]
at bsc.d(SourceFile:23) ~[1.21.3.jar:?]
at bry.B(SourceFile:138) ~[1.21.3.jar:?]
at bry.bA(SourceFile:123) ~[1.21.3.jar:?]
at fmg.c(SourceFile:1307) [1.21.3.jar:?]
at fmg.f(SourceFile:933) [1.21.3.jar:?]
at net.minecraft.client.main.Main.main(SourceFile:265) [1.21.3.jar:?]
Caused by: java.lang.RuntimeException: Failed to load required shader programs:
minecraft:core/rendertype_entity_translucent (VertexFormat[Position, Color, UV0, UV1, UV2, Normal]): Could not find shader: minecraft:rendertype_entity_translucent (VERTEX)
minecraft:core/rendertype_armor_cutout_no_cull (VertexFormat[Position, Color, UV0, UV1, UV2, Normal]): Could not find shader: minecraft:rendertype_armor_cutout_no_cull (VERTEX)
at gmc.a(SourceFile:174) ~[1.21.3.jar:?]
at gmc.a(SourceFile:37) ~[1.21.3.jar:?]
at awa.a(SourceFile:14) ~[1.21.3.jar:?]
at java.base/java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:718) ~[?:?]
... 9 more

You probably should no longer use this shader, as vanilla now supports custom armor textures. Fo other uses, update would still be needed

You probably should no longer use this shader, as vanilla now supports custom armor textures. Fo other uses, update would still be needed

I am not aware of any vanilla custom armor textures... Can you link anything? I have not seen anything of the sort, unless you're talking about trims?

https://www.minecraft.net/en-us/article/minecraft-java-edition-1-21-2
Search for minecraft:equipable and Custom Equipment Layers

https://www.minecraft.net/en-us/article/minecraft-java-edition-1-21-2 Search for minecraft:equipable and Custom Equipment Layers

Thank you very much. I will work with this going forward. :)

You probably should no longer use this shader, as vanilla now supports custom armor textures. Fo other uses, update would still be needed

I definitely agree with this advice. If possible avoid shaders, since they carry a lot of baggage. There are however some niche use cases in which fancyPants can still be useful.


not getting transcript even tho its available

DO NOT DELETE THIS! Please take the time to fill this out properly. I am not able to help you if I do not know what you are executing and what error messages you are getting. If you are having problems with a specific video make sure to include the video id.
To Reproduce
Steps to reproduce the behavior:
What code / cli command are you executing?
For example: I am running
    def transcribe_video(self) -> str:
        youtube_url = self.inputs[0].value
        try:
            video_id = self.extract_video_id(youtube_url)
            transcript = YouTubeTranscriptApi.get_transcript(video_id, languages=['en', 'hi'])
            full_transcript = " ".join([entry['text'] for entry in transcript])
            return full_transcript
        except Exception as e:
            return f"Error: {str(e)}"

Which Python version are you using?
Python 3.12.4
Which version of youtube-transcript-api are you using?
youtube-transcript-api 0.6.2
Expected behavior
I expected to receive the english transcript
Actual behaviour
Error:
Could not retrieve a transcript for the video https://www.youtube.com/watch?v=yeCguJWHKVY! This is most likely caused by:

Subtitles are disabled for this video

If you are sure that the described cause is not responsible for this error and that a transcript should be retrievable, please create an issue at https://github.com/jdepoix/youtube-transcript-api/issues. Please add which version of youtube_transcript_api you are using and provide the information needed to replicate the error. Also make sure that there are no open issues which already describe your problem!

list transcript answer
For this video (yeCguJWHKVY) transcripts are available in the following languages:
(MANUALLY CREATED)

en ("English")[TRANSLATABLE]

(GENERATED)

en ("English (auto-generated)")[TRANSLATABLE]

Hi @Uddhav07,
does this happen for all videos or only this specific one? If it happens for all videos you are most likely encountering the same issue as in #303

Hi @Uddhav07,
does this happen for all videos or only this specific one? If it happens for all videos you are most likely encountering the same issue as in #303

Yeah im using cloud server and therfa ing this on all videos

Okay, then this is a duplicate of #303 .


Rewrite multipart parser in Swift

Rewrites parser in Swift and removes dependency on C library.
This also fixes an existing parsing bug where a nested multipart body would see its boundaries stripped of the "\r\n--" prefix. (replaces #56)

We probably don't need API docs if we don't expose the Parser. In the same vain I think we can keep the error package protected for now and expose it later if it makes sense. Easier to go that way than the other way!

Wrt "research better approach for reading and slicing ByteBuffers that doesn't involve a separate copy just for the slicing" maybe @weissi could offer some pointers?

Final comment I think we can roll any improvements into #55 unless anyone spots any blockers. Since all the tests are passing and we've had a few eyes on it I'd be comfortable with merging and tagging

Wrt "research better approach for reading and slicing ByteBuffers that doesn't involve a separate copy just for the slicing" maybe @weissi could offer some pointers?

@siemensikkema What would you need there? ByteBuffer can be sliced without copying the bytes either with or without consuming the bytes.
Some examples (all assuming we have a buffer: ByteBuffer), none of them copy any bytes:

buffer.readSlice(length: foo): will conume foo bytes from buffer and return the consumed portion as a ByteBuffer
ByteBuffer(buffer.readableBytesView.prefix { $0 != "$" }): scan the readable bytes until we find a $ character and construct a ByteBuffer which contains all readable bytes until (excluding) the $ character
ByteBuffer(buffer.readableBytesView.drop { $0 == " " }.prefix { $0 != " " }): trim the `ByteBuffer from leading/trailing spaces

@weissi this parser's state machine operates on a byte by byte basis in order to support arbitrary "cuts" in the incoming data which makes decoding from streaming input possible.
But reading one byte at a time the way I do it consumes them so I resorted to keeping another copy of the buffer around which I use to send slices of data via the callback closure once I've determined the offsets.
I need to look into using the techniques you suggested for parsing the body sections while keeping the rest of the parser as is (I don't need slicing for the headers). I think I can get by with a single buffer that way.
But things get tricky with regards to detecting the boundaries when you're never sure whether the current buffer (streamed chunk) suddenly runs out of bytes and what started as a boundary was in fact a part of the body after all.

But reading one byte at a time the way I do it consumes them so I resorted to keeping another copy of the buffer around which I use to send slices of data via the callback closure once I've determined the offsets.

Yup, seems fine. These "copies" won't actually copy the data, so I think you're fine here. ByteBuffer is a value type so you can hold onto as many copies as you want, it'll only make a copy when you actually write into the bytes (which you'll never do). Moving reader/writerIndex doesn't count as a write.

I need to look into using the techniques you suggested for parsing the body sections while keeping the rest of the parser as is (I don't need slicing for the headers). I think I can get by with a single buffer that way.

I think so too but regardless, I wouldn't worry about copies.

@siemensikkema we good to merge and tag then?

Original comment before preparing for release:

Rewrites parser in Swift and removes dependency on C library as part of this SSWG proposal and specifically to address this comment.
This also fixes an existing parsing bug where a nested multipart body
would see its boundaries stripped of the "\r\n--" prefix.

[x] handle illegal characters while parsing headers
[x] simplify body parsing
[x] add more tests for the parser specifically to test certain edge cases
[ ] research better approach for reading and slicing ByteBuffers that doesn't involve a separate copy just for the slicing
[ ] add API docs
[ ] consider expanding MultipartParser.Error with more precise error cases and making it public?

These changes are now available in 4.0.1

How is the relative performance? Out of interest.

@mxcl I want to do a comparison between the old one, the current one and one with a potential performance improvement (using an approach mentioned above) that I have locally because according to #59 the performance has a lot to be desired.
I didn't get around to setting up the comparison yet.


fix allowedMethods function return

I think the middleware function named allowedMethods should return the result.
It must return the result of 'yield *next', so that the next middleware function can get the value.
Sorry for my poor English.
Wish you work happily.

Seems like a return statement inside a generator function is not necessary according to mdn. In the example given on mdn it is a bit confusing in my opinion, that the return results in done: true, although there can be further yield keywords.

@marvinhagemeister

There are some differences. Like here

function* yieldAndReturn1() {
    return yield "Y"; // there is a 'return'
}

function* yieldAndReturn2() {
    console.log(yield* yieldAndReturn1()); // Y
    return "R";
}

const gen = yieldAndReturn2();
let ret = gen.next();
while (!ret.done) {
    ret = gen.next(ret.value);
}

function* yieldAndReturn1() {
    yield "Y"; // there is no 'return'
}

function* yieldAndReturn2() {
    console.log(yield* yieldAndReturn1()); // undefined
    return "R";
}

const gen = yieldAndReturn2();
let ret = gen.next();
while (!ret.done) {
    ret = gen.next(ret.value);
}

I used a execution function, just like co. When there is no return, 'value' can not be transmitted. So the return value is important.


added missing array unique export in index

Good catch :+1:. Somehow I managed to update the builds w/o updating the typings.


Make options configurable per embed

Allow configuration of settings on a per-embed basis (maybe as a query string or something after the path to the examples folder?)
For example, module is a very nice option to have:

Which module to open by default. Multiple paths comma separated are allowed, in that case we show them as tabs.

Workaround: Add multiple Remark plugin configuration blocks with different protocols (eg. embedded-html-sandbox:// for configuration for html files, embedded-react-sandbox:// for React configuration)

Maybe a query string would also be a possibility...
Got the idea from the awesome remark plugin: https://github.com/kevin940726/remark-codesandbox


Issue#SB-15803 feat: report other issue pop up form validations and binding and reactive form setup

multiple pr


Chess: Move E1 to D1

Please do not change the title. Just click "Submit new issue". You don't need to do anything else :D

@Ke123-design done! Successfully played move e1d1 for current game.
Thanks for playing!


added AR, MA, ARMA model

🛠 Related Issue

Closes #502

📝 Description

Added Time Series Analysis using AR, MA, ARMA models.
🔍 Type of PR

[ ] 🐛 Bug fix
[X] ✨ Feature enhancement
[ ] 📚 Documentation update
[ ] 🛑 Other (specify): _______________

✅ Checklist

[X] I have performed a self-review of my code.
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@Akshat111111 Kindly review.


Change from F to C

Hi,
I'm trying to change for the weather from F to C and it doesn't work. I've tried setting the locale to UK but no luck, any idea what i'm missing?
P.S. i'm not a dev so probably is something obvious
Thanks

NVM i figured it out.


[Help]: What do we pass for the config.json for default Facodec Training

I see that config.json is a required argument for training Facodec. Are there any examples of what this should look like including the default config.json that the pretrained model was trained on that we can use?

@Plachtaa Could you please help with this issue?

Hi @amuvarma13 you can use egs/codec/FAcodec/exp_custom_data.json as the default training config


TCA analysis into the paper

Addition of TCA analysis of simple model

This is great, thanks Umar!


Fossil Support

I don't actually use Fossil, so I can't say this is super important to me.  However, Fossil has always looked like a very interesting project, and I know it has some very happy and dedicated users.  For the sake of completeness, I think it would be good to support Fossil here.
Here's the link to where Fossil describes how their "versionable settings" are setup:  http://fossil-scm.org/xfer/doc/trunk/www/settings.wiki
The important line is this:  "For example, to set the list of ignored files, create a .fossil-settings/ignore-glob file where each line contains a glob for ignored files."

Fossil has quite different logic - the rules are described in the .fossil-settings/ignore-glob file which is outside the project files scope.
All of the handled ignore files starts ignoring from the current working directory and cannot access other tree or parent scope.

Interesting.  I was going to say, "This feature should be pretty easy!" and then I slapped myself, and left the analysis of whether it was possible or how easy to someone who actually knows something about it...  :D  Well, like I said, I don't think support for this is super important.

I'll not reject this feature request for now..
There is something simmilar to the rules defined out of the scope issue: Git allows to define the ignore rules common for all of the Git projects.
Reference: https://help.github.com/articles/ignoring-files/#create-a-global-gitignore
I have a plan to handle this case too, so Fossil is quite simmilar to this one.

Introduced with release v1.1.


failing test tests.test_openeo_gfmap.test_cloud_masking.test_bap_quintad[Backend.TERRASCOPE]

tests.test_openeo_gfmap.test_cloud_masking.test_bap_quintad[Backend.TERRASCOPE] currently fails
with this in the error logs:
  File "/opt/venv/lib64/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1584, in apply_process
    return process_function(args=ProcessArgs(args, process_id=process_id), env=env)
  File "/opt/venv/lib64/python3.8/site-packages/openeo_driver/ProcessGraphDeserializer.py", line 1009, in aggregate_temporal
    return data_cube.aggregate_temporal(
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/geopysparkdatacube.py", line 846, in aggregate_temporal
    return self._apply_to_levels_geotrellis_rdd(
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/geopysparkdatacube.py", line 186, in _apply_to_levels_geotrellis_rdd
    pyramid = Pyramid({
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/geopysparkdatacube.py", line 187, in <dictcomp>
    k: self._create_tilelayer(func(l.srdd.rdd(), k), l.layer_type if target_type==None else target_type , k)
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/geopysparkdatacube.py", line 847, in <lambda>
    lambda rdd, level: pysc._jvm.org.openeo.geotrellis.OpenEOProcesses().aggregateTemporal(rdd,
  File "/opt/spark3_4_0/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py", line 1322, in __call__
    return_value = get_return_value(
  File "/opt/spark3_4_0/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/protocol.py", line 326, in get_return_value
    raise Py4JJavaError(
py4j.protocol.Py4JJavaError: An error occurred while calling o3178.aggregateTemporal.
: org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 21.0 failed 4 times, most recent failure: Lost task 0.3 in stage 21.0 (TID 171) (epod052.vgt.vito.be executor 6): org.apache.spark.api.python.PythonException: Traceback (most recent call last):
  File "/opt/spark3_4_0/python/lib/pyspark.zip/pyspark/worker.py", line 830, in main
    process()
  File "/opt/spark3_4_0/python/lib/pyspark.zip/pyspark/worker.py", line 822, in process
    serializer.dump_stream(out_iter, outfile)
  File "/opt/spark3_4_0/python/lib/pyspark.zip/pyspark/serializers.py", line 146, in dump_stream
    for obj in iterator:
  File "/opt/spark3_4_0/python/lib/pyspark.zip/pyspark/util.py", line 81, in wrapper
    return f(*args, **kwargs)
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/utils.py", line 54, in memory_logging_wrapper
    return function(*args, **kwargs)
  File "/opt/venv/lib64/python3.8/site-packages/epsel.py", line 44, in wrapper
    return _FUNCTION_POINTERS[key](*args, **kwargs)
  File "/opt/venv/lib64/python3.8/site-packages/epsel.py", line 37, in first_time
    return f(*args, **kwargs)
  File "/opt/venv/lib64/python3.8/site-packages/openeogeotrellis/geopysparkdatacube.py", line 582, in tile_function
    result_array = result_array.transpose(*('t' ,'bands','y', 'x'))
  File "/opt/venv/lib64/python3.8/site-packages/xarray/core/dataarray.py", line 2154, in transpose
    dims = tuple(utils.infix_dims(dims, self.dims))
  File "/opt/venv/lib64/python3.8/site-packages/xarray/core/utils.py", line 726, in infix_dims
    raise ValueError(
ValueError: ('t', 'bands', 'y', 'x') must be a permuted list of ('t', 'y', 'x'), unless `...` is included

Did some initial digging and I think this is going on:
there is a UDF https://github.com/Open-EO/openeo-gfmap/blob/main/src/openeo_gfmap/preprocessing/udf_rank.py used in apply_neighborhood that takes a t-bands-y-x array and returns a t-y-x array (eliminating the band dimension). Our code seems to expect an array with bands dimension.

@soxofaan @jdries Seems like an OpenEO issue, as if I'm adding the band dimension manually to the DataCube before running the UDF, I get the error:

openeo.metadata.DimensionAlreadyExistsException: Dimension with name 'bands' already exists

Also before running the UDF, the following line is done, and the job doesn't complain before entering the UDF
score = score.rename_labels("bands", [BAPSCORE_HARMONIZED_NAME])

@GriffinBabe to provide some more context on the issue (e.g. full exception)

@GriffinBabe The error that you show is on this line in the openEO backend code:
result_array = result_array.transpose(*('t' ,'bands','y', 'x'))
Where result_array refers to the output of your udf_rank.py UDF.
Apply_neighborhood expects your output to have these dimensions ('t' ,'bands','y', 'x') but this line in the UDF creates a dataarray with ('t', 'y', 'x') dimensions:
bap_score = array.sel(bands="S2-L2A-BAPSCORE")
To solve it you can use this line instead:
bap_score = array.sel(bands=["S2-L2A-BAPSCORE"])

@soxofaan assign to me


Fixed listing nodes and obtaining config values

This commit:

Enables the nodes with specific label_selector to be listed.
Prevents Cerberus from erroring out when key-values are not
set in the config

Can one of the admins verify this patch?


DFSClient holds its own copy of ugi, DFSClient needs to be created wi…

…thin the DoAs context

LGTM good job.


Update axon-guice to be compatible with Axon 2.1

Axon 2.1 changes some things used in axon-guice. One of these is that
ParameterResolverFactory is now an interface, so GuiceParameterResolverFactory
can no longer extend it (it was an abstract class).
Please see following commits :
 - https://github.com/AxonFramework/AxonFramework/commit/1bcb3468df5f448388d470c02c1caa542a0ac00a
 - https://github.com/AxonFramework/AxonFramework/commit/cb5110af1b8098fb1b4ab4283eac285ddd679c07
 - https://github.com/AxonFramework/AxonFramework/commit/61af06976b3ddf33ba4bcb9061f7a70bf1567b1c

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 11 Mar 2014 at 9:33

Thanks for the migration to github. I see that the depencies were changed to use Axon 2.4.2 in Maven, but I don't see any code change. Can this really work?


Code refactory and UI improvements for DualView sample

Great job, I love what you did with the toolbar, the string utils and the javadocs! I left only some small comments, please take a look when you have time:)

Great job, I love what you did with the toolbar, the string utils and the javadocs! I left only some small comments, please take a look when you have time:)


⚠ Young Discoverers has degraded performance

In 68e456c, Young Discoverers (https://youngdiscoverers.org.au) experienced degraded performance:

HTTP code: 200
Response time: 5763 ms

Resolved: Young Discoverers performance has improved in 6d49fb7.


Add spill_count and spilled_bytes to BaselineMetrics, test sort with spill

Which issue does this PR close?
Closes #1611 and #1573 .
Rationale for this change
Report spill_count and spilled_bytes in BaselineMetrics since these might be common metrics as we proceed with more memory consumers.
We could also test sort spill with these newly introduced metrics.
What changes are included in this PR?

Add spill_count and spilled_bytes in BaselineMetrics
Report the above two metrics in the sort while spilling.

Are there any user-facing changes?
No.

Hi @alamb, could you please help review this?

Thank you @yjshen  and @liukun4515  for the review ❤


more metadata for the arms images

Now that the image filenames and URLs are accessible from the plutof download as harvested into github (associated_data_[observatory], the information therein needs to be supplimented
In particular we would like

the plate number and side (and if there are >1image for a plate and side, that is OK, maybe add an iterator tho)
if this is a field image instead

To check out how possible this is to do on the current crop of images

are the filenames presented well enough to get this info from there?
are enuf events provided with an image-explanation spreadsheet, well enough formatted, that this can be used
perhaps may need to do some manual work, e.g. indicating where to use the filename, where to use a CSV file, and perhaps even myself editing those CSV files where they are not quite right
what other metadata from the images pages on plutof do we want to add to this spreadsheet, so they can be made visible (as the metadata page on plutof for each image is not downloadable from outside, only the actual image is). is there more metadata there than is in the .jpg images themselves anyway?

If yes to the above, then @kmexter should go ahead and do the necessary preparation and the @cedricdcc will write the script to impliment these actions

We have added a bit of code that extracts plate number and face to the metadata, when those info are available. Waiting for user feedback on whether more is wanted


using a non-integer number instead of an integer will result in an error in the future

peak_detection.py
line 136: "ridge.shape[1] / 2" is not necessary an integer.

Thanks for your feedback!
This issue was fixed by converting float to int for indexing ridge.
Please see this commit for details.


Fix the container registry name and apikey name in Makefile

Bug: #104
In this PR:

Update the apikey name to be as in devcontainer
Fix the container register namespace

IBM Cloud toolchain: Delivery Pipeline deployed products to nyu-devops-products, including commit 1e844af1088c4eaf0b2b34b668cc0fa159a53511


Fix default account status

Fixes Issue #27
Change default account status to 'unverified' and reordered unique where status in constraint to match Rodauth default table setup: http://rodauth.jeremyevans.net/rdoc/files/README_rdoc.html#label-Creating+tables

Looks good, thanks!


Unexpected keyword argument 'batch_label' error when calling AnnDatasetFromAnnData() function

I have made a docker container that uses a Python 3.6 base image and I use conda to install the following packages:
name: myenv
channels:
  - conda-forge
  - bioconda
dependencies:
  - python=3.6
  - numpy
  - scipy
  - pandas
  - matplotlib
  - scvi
  - awscli
  - boto3
  - scanpy
  - dataclasses
  - anndata
  - pytorch

Within my container, I have a python script that has the following line:
from scvi.dataset import AnnDatasetFromAnnData

scviDataset = AnnDatasetFromAnnData(adata,batch_label="batch_indices")

When I try to run my container, that line raises the following error: TypeError: __init__() got an unexpected keyword argument 'batch_label'
This error made me think there might be a version incompatibility issue, so I checked the version number of anndata, which is Version('0.7.1'). Which version of anndata is scVI expecting? Is that what's causing the above error?

Hi!
Could you provide the scVI version in the container and the full traceback?

Hi! The scVI version is '0.4.1'. And attached is a snapshot of the Traceback!

Thanks! As of today, scVI does not support python 3.6. Could you upgrade to 3.7?

Ah also this functionality was not available in 0.4.1. We recommend upgrading to the newest version (0.6.3)

@adamgayoso Ahh okay, I figured it might be that. I'll try upgrading to that newer version first, and then if it still doesn't work, I'll upgrade to 3.7!

Great, you should be able to run scVI on python 3.6 if you install dataclasses.

Oh okay, I just tried building the container with scVI=0.6.3, but then I got a whole bunch of package conflicts (too long to post here), and the dataclasses package was included already in my YML file. Looks like Python 3.7 is my only option?

Possibly, please let me know if there is an issue with 3.7!

@claudiadast I also updated the conda recipe so 3.6 should work now


Inference is fast but saving is slow.

I noticed something when I tried to use torch.compile (similar to the way it's used here) on RunPod. I am using it on an A100 80gb.
What I noticed is that the inference time is really fast, but the time to actually decode and save is way slow. I did some digging into it and realized that it is actually the time to move the tensors from GPU to CPU that took the time. The funny thing is, during the first run (which is when the compilation happens), because it takes time, the saving of the image is really fast but after that, inference is fast but saving is slow. It is more like the saving tries to account for the speed gained while inference.
I've been battling with this all day and I'll really appreciate if anyone can help with this.
Here's the code I ran (yeah I didn't quantize)
import torch
from diffusers import FluxPipeline
from PIL import Image
import numpy as np
import time
from torchvision.transforms import ToPILImage

torch.set_float32_matmul_precision("high")

model = "dev"

pipe = FluxPipeline.from_pretrained(f"black-forest-labs/FLUX.1-{model}", torch_dtype=torch.bfloat16, local_files_only=True).to("cuda")

torch._inductor.config.conv_1x1_as_mm = True
torch._inductor.config.coordinate_descent_tuning = True
torch._inductor.config.epilogue_fusion = False
torch._inductor.config.coordinate_descent_check_all_directions = True

print("Done Loading")

pipe.set_progress_bar_config(disable=True)

pipe.transformer.to(memory_format=torch.channels_last)
pipe.transformer = torch.compile(pipe.transformer, mode="max-autotune", fullgraph=True)

pipe.vae.to(memory_format=torch.channels_last)
pipe.vae.decode = torch.compile(pipe.vae.decode, mode="max-autotune", fullgraph=True)

print("Done Compiling")

prompt = "A dinosaur in modern suit"

num_images = 4

def tensor_to_pil(tensor: torch.Tensor) -> list[Image.Image]:
    """
    Convert a batch of PyTorch tensors to a list of PIL Images.

    Parameters:
    - tensor: torch.Tensor - The tensor to convert. Assumes the tensor has a batch dimension.

    Returns:
    - list[PIL.Image.Image]: The list of tensors as PIL images.
    """
    # Convert to fp16 and move to CPU
    # ToPILImage Transform does not support bfloat16 or some other formats
    tensor = tensor.to(dtype=torch.float16, device="cpu")
    print("Done")

    transform = ToPILImage()
    images = [transform(t) for t in tensor]
    return images

for _ in range(num_images):
    print()
    start_time = time.time()
    out = pipe(
        prompt=prompt,
        guidance_scale=3.5,
        height=1024,
        width=1024,
        output_type="pt",
        num_inference_steps=28
    ).images

    print(f"Inference Time: {time.time() - start_time}")

    print("Starting to save")
    start_time = time.time()

    pil_images = tensor_to_pil(out)
    for i, img in enumerate(pil_images):
        # Save the image
        img.save(f"image{i}.png")

        print(f"Save Time: {time.time() - start_time}")

    print("Saved")

And here's the output
Inference Time:<PHONE_NUMBER>0526428
Starting to save
Done
Save Time: 0.3608579635620117
Saved

Inference Time: 11.806470394134521
Starting to save
Done
Save Time: 0.36004066467285156
Saved

Inference Time: 3.705930471420288
Starting to save
Done
Save Time: 7.542477369308472
Saved

Inference Time: 3.5726981163024902
Starting to save
Done
Save Time: 7.612122297286987
Saved

The funny thing with this output is that the ratio just flips in such a way that the total time are approximately equal (11s).

If you're running on GPU you're using the async launch API of PyTorch, so you need to use synchronize to get accurate timings. See https://pytorch.org/docs/main/notes/cuda.html#asynchronous-execution

If you're running on GPU you're using the async launch API of PyTorch, so you need to use synchronize to get accurate timings. See https://pytorch.org/docs/main/notes/cuda.html#asynchronous-execution

Oh thank you very much. I actually tried this earlier but then I lose out on the speed 🥲. 1 image on an A100 8gb now use 10.5s on average.
From the benchmarks here, I see I'm suppose to get like 4s on average.

Oh nevermind me, I think @sayakpaul used H100 for most of those benchmark figures as I am getting similar results now.
I also noticed that I couldn't use some of the fp8 quantization on A100 as the compute capability is low (8.0). The fp8 used here (fp8dq= fp8e4nv) works with an Nvidia GPU with ARCH/Compute capability >= 8.9. So, I found these GPUs to be well suited: H100, RTX 6000 ADA, and RTX 4090.
I also noticed fp8dqrow  needs a GPU with compute capability of 9.0 (I could only find H100 as at writing this comment).
Please, if there's a work around to making this work on other GPUs (like A100), do share. If there's not, I think we can update the benchmark with these details i.e. GPUs with the respective precisions they support.
Thank you and as usual, awesome work guys!

Thanks!

Please, if there's a work around to making this work on other GPUs (like A100), do share. If there's not, I think we can update the benchmark with these details i.e. GPUs with the respective precisions they support.

I think we already mention that FP8 related things have compute restriction. But feel free to send over a PR if you think any info is missing.
Cc: @drisspg for FP8 as well.


Loading a single locale makes noticeable improvements in the load time of test suites.
This PR sets en as the default locale for the library, and skips loading all the other locales available in faker.
Faker docs
Scope
This PR only updates graphql-fixtures because it's what I managed to test in Shop, but we probably should apply this change to other libs of quilt using faker default export unless we intentionally want to load all locales for some reason.
Happy to include those changes in this PR or open a follow up PR for doing so.
Shop
In Shop, this PR reduced the test suite time from ~360s to ~280s in local environments.
Type of change
Not so sure here... if for some reason someone is relying on graphql-fixtures loading faker's locales then this should be considered a breaking change. This would be very rare though, and we might want to consider this change a minor instead?

[x] graphql-fixtures Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Credits
@notjosh for the research and finding of faker affecting tests load time

Great find! If you don't want to take on updating the other uses of faker, could you open an issue please?

Hey, happy to do the update!
I'll add the changes to this PR and request the review again, unless you think it's better to do that in a separate PR?
Also, regarding type of change, any thought on this being a breaking change vs minor?

@vsumner changed other places as well.
Another question, regarding changelog, I was going to add an entry in the changelog of the affected packages... but then decided not do so, as they don't really expose functionality based on faker. graphql-fixtures seems to be the only package to do so.

@vsumner changed other places as well.
Another question, regarding changelog, I was going to add an entry in the changelog of the affected packages... but then decided not do so, as they don't really expose functionality based on faker. graphql-fixtures seems to be the only package to do so.

Makes sense.

@BPScott do you know if there's anything wrong with the CLA check? Seems to be hanged in Waiting for status to be reported


Have I Been Pwned API Support

Feature Request
An optional flag for running a given password against the HIBP API. Presence of the password in the HIBP database would result in a lowered score and a custom message. To save on requests to the HIBP API it could run after the basic checks have completed—no need to run "password1!" against HIBP. Requests to the API could fail silently.

A wrapper library which calls zxcvbn and then HIBP would make more sense than integrating the two directly (imo). I'd venture that someone out there must have built such a thing (whether as a library or just in their application code) - anyone want to open source it (and link from this issue)?

Guess who knows all your code stacktraces and stashes? Me. The girl you hacked.
BadFood Sucks.
Cheers criminal!

In my react web app i use a function that does exactly that: it calls zxcvbn and returns the usual object, but if the security score is greater than 2 the password is also checked agains HIBP using the k-anonymity apis, and the result is adjusted.
It's definelty not clean code, but if anyone is interested the code is here

Janek Bevendorff @phoerious addressed this:
https://keepassxc.org/blog/2018-02-24-pwned-database/
On Keepass I use the (open-source) HIBPOfflineCheck plugin that tries to match the SHA1 against a downloaded database dump at https://haveibeenpwned.com/Passwords
It has the benefit of not supplying the hash to an external API but eats quite a lot of space (22+ GiB).

Janek Bevendorff @phoerious addressed this:
https://keepassxc.org/blog/2018-02-24-pwned-database/

Off topic disagreement: HIBP integration is useless only if you are building a password manager.
As the article points out HIBP is not very effective for measuring password strength, but combined with zxcvbn in a login form It can detect reused passwords, or common passwords that are not in the zxcvbn list. It's pretty useful

If anyone is going to create a wrapper, this might be a useful reference implementation:  https://github.com/jamiebuilds/havetheybeenpwned/blob/master/lib/implementation.js


Lunar 2021 resets after cup removed

"With the Acaia Lunar 2021, there is a flow-rate that displays after you remove the cup from the scale after brewing... when I remove the cup, I briefly see the flow rate indicator, then the scale zeroes out."

"Pretty sure that my problem was that Auto Reset was turned on for the default 30 seconds. That would explain the inconsistency in behavior as it would behave as expected some of the time when I removed the cup relatively quickly after brewing. I turned the setting off and will do more testing.
Okay, yup. The auto reset is what it was. Sorry for the false alarm!" - rhtrevino — 08/11/2024 6:17 PM


Add Rakudo Star binaries cataloger

I seem to be missing something here as I can't get it to detect the binary

Taking a look at the binary you selected and the regex you have I also don't see the version embedded in the binary:
$strings classifiers/bin/rakudo-star/2024.08/linux-amd64/rakudo | grep -i rakudo
ERROR: RAKUDO_HOME is invalid: %s
--rakudo-home=
RAKUDO_HOME

$ strings classifiers/bin/rakudo-star/2024.08/linux-amd64/rakudo | grep -i 2024
<no results>

Maybe this version is embedded not as a string? If that's the case then detecting the version might not be possible with this approach.

I have been looking at rakudo and it's a lot more complex than most of the binaries I have done. The binaries do not seem to include any version info and instead it is embed in the moarvm. I was able to detect moarvm though

@gdams @LaurentGoderre It looks like the path forward is cataloging the moarvm binary as mentioned above -- is any help wanted with this? Or does that not quite fit what you're trying to accomplish @gdams ?

sorry - yes I'll likely need to drop this one for now as I've been dragged into other things


Check cophi-toolbox.preprocessing

check if cophi-toolbox.preprocessing == dariah-topics.preprocessing

there is redundancy, but dariah-topics.preprocessing also includes preprocessing functions specifically tailored for topic modelling.
add_token2id
create_document_term_matrix()
read_matrix_market_file()
read_model()
read_token2id()

has there been separate development in the common functions?

nope :)


[BUG] The admin audits page froze after clicking on the user or status columns

Overview of the issue
The admin page froze after clicking on the user (or status) column. The dropdown on entity and admin on the navbar are also stuck.

Motivation for or Use Case
In the chrome console there is an error (see screenshot below)

Reproduce the error

Go on the admin audits page
Click on the user (or status) column to sort
Try to click on the entity or admin menu (on the navbar) --> KO

Related issues
None

Suggest a Fix
I will try to do a PR next week to fix this

JHipster Version(s)

JHipster configuration
Same as #404

Entity configuration(s) entityName.json files generated in the .jhipster directory

Browsers and Operating System

[x] Checking this box is mandatory (this is just to show you read everything)

I didn't succeed to reproduce my own issue today.. That's really weird (I didn't update anything)...
Is someone able to reproduce it ?

I couldn't reproduce but I do see a different error if the dates are the same:
vue.esm.js?a026:628 [Vue warn]: Duplicate keys detected: '2019-07-05T14:42:57Z'. This may cause an update error.
found in
---> <JhiAudits> at src/main/webapp/app/admin/audits/audits.vue
       <App> at src/main/webapp/app/app.vue
         <Root>

I'm closing this issue as I'm not able to reproduce it.
@ruddell I didn't succeed to reproduce your issue with same dates


interactive mode

would be great to have a interactive mode where construct asks the user for all required (and also optional information) with some sane defaults pre-selected

You could use http://symfony.com/doc/current/components/console/helpers/questionhelper.html

:+1: for this

Feature added on latest release: https://github.com/jonathantorres/construct/releases/tag/v1.7.0 :dancer:


Add Application to FHIR Role step missing?

[Enter feedback here]
The steps to test the Get Patient via API returns unauthorized after following all the steps to add the app registration and grant the API permissions.
Once the App is granted any FHIR Role (Reader / Writer) via the Access control/ RBAC on the FHIR service, the call succeeds.
I believe this step is missing in the tutorial and should be added for clarity.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 74d0dae2-ed92-bad6-1a97-4fdcda016c07
Version Independent ID: ec5ca2c6-319a-1af5-eda3-95e84c7fac8d
Content: Web App Tutorial - Test connecting to Azure API for FHIR
Content Source: articles/healthcare-apis/tutorial-web-app-test-postman.md
Service: healthcare-apis
Sub-service: fhir
GitHub Login: @CaitlinV39
Microsoft Alias: cavoeg

@pantanuj Thank you for the feedback. We are actively investigating and will get back to you soon.

@pantanuj This is being assigned to the content author to evaluate and update as appropriate.

@pantanuj This is the third step in the tutorial. Did you configure the CORS And also register your public application as described in these steps:
https://docs.microsoft.com/azure/healthcare-apis/tutorial-web-app-fhir-server
https://docs.microsoft.com/azure/healthcare-apis/tutorial-web-app-public-app-reg
Let me know if you are still having issues after following these configuration steps first.

@AnuragSharma-MSFT - I believe this can be closed unless there are other questions on this.

Thank you @CaitlinV39 for helping here.
@pantanuj We are going ahead and closing this feedback and please get back to us if any other info is required.


Anchor link that navigate into a page without refresh

Inspired from flamethrower but rather than intercepting the default click event handler in HTMLAnchorElement (), you must be explicit which link that cause SPA navigation.
About

About

No <render-scope> is necessary

Note
Using attribute is="a-page" is recommended than using custom tag  because it provide fallback when the script not yet executed

...very short content
About

others yet same:

Iles Turbo

No need to keep and track the scroll since it can be solved by structuring the page to utilize #29


pip install errors (versions)

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
opentelemetry-exporter-otlp-proto-http 1.23.0 requires opentelemetry-exporter-otlp-proto-common==1.23.0, but you have opentelemetry-exporter-otlp-proto-common 1.29.0 which is incompatible.
opentelemetry-exporter-otlp-proto-http 1.23.0 requires opentelemetry-proto==1.23.0, but you have opentelemetry-proto 1.29.0 which is incompatible.
opentelemetry-exporter-otlp-proto-http 1.23.0 requires opentelemetry-sdk~=1.23.0, but you have opentelemetry-sdk 1.29.0 which is incompatible.
gradio-client 0.14.0 requires websockets<12.0,>=10.0, but you have websockets 14.1 which is incompatible.
crewai 0.22.5 requires langchain-openai<0.0.6,>=0.0.5, but you have langchain-openai 0.1.7 which is incompatible.
crewai 0.22.5 requires python-dotenv==1.0.0, but you have python-dotenv 1.0.1 which is incompatible.

conda or os python ?

Python 3.10.11
Windows 10

installed in its own virtual env


Highlighting Actions in Sublime Text

Hi Richard (@cpsolver),
While searching through the dashrep_compiler_source_code.txt for the string "required-operand-count-for-action-", I noticed a slight inconsistency in how Sublime Text flagged the "action" portion following the string.
Note: In the examples below, I am using "Quick Search" (<Alt><F3>) to highlight the search string.
Most the "action" phrases are marked with an italic type font:

However, in a few cases, they are marked in cyan blue:

There are 117 actions in total, of which the following 26 phrases are not marked the same as the other actions in the list. I have given below the description of the "exception" phrases from the Dashrep.Org website.
 I will therefore make a PR to change the syntax highlighting in Sublime Text to reflect that fact.
Kind Regards,
Liam

In my testing, I found that I had missed "size-of-file" from the list above. Since it is an action, I will amend the PR to take account of this phrase.

In another place, "Update Sublime highlighting for selected actions #20," I requested moving a few more actions into the correct font style.  (I'm still learning how to use GitHub.  In hindsight I probably should have written that comment here.)

Hi Richard (@cpsolver),
That's not a problem! I will work on the list that you mentioned in https://github.com/cpsolver/Dashrep-language/pull/20#pullrequestreview-2267413834. I will not be able to do so for the next few days as I will be away from my computer. But I will get to it as soon as possible following my return.
Kind Regards,
Liam

Again, thank you Liam!  I've merged your changes so I'm closing this issue.


update mock-fs. fixes #31

Coverage remained the same at 92.818% when pulling fa88004e638160ecad8e0180cfe9160c1b0e5822 on fix-tests-node4 into bb8c84a038e8a0f345a56514a400e83f91937b51 on master.


Extend commons.py to support a more complete G-code specification

Mirror of #26 to get the full test run before merge

@giannissc we may have to address this https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&pullRequest=27&id=voneiden_cq-cam&open=AYfEFVc0M9bAMZl_3TMA but no hurry.
Need to research a bit what the best practice would be for large number of keyword arguments. Maybe somekind of configuration object or TypedDict


fixtures: update resource types from Zenodo

:heart: Thank you for your contribution!
Description
Please describe briefly your pull request.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings (for relevant code).
[ ] I've followed the CSS/JS and React guidelines (for relevant code).
[ ] I've followed the web accessibility guidelines (for relevant code).
[ ] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

agreed with @slint to close this PR, since not all the resource types introduced in the PR should be loaded by default


New Google

For noting: the "new" Google Podcasts experience has a useragent of something like Mozilla/5.0 (Linux; Android 8.1.0; Pixel XL Build/OPM2.171019.029; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/65.0.3325.109 Mobile Safari/537.36 GSA/<IP_ADDRESS>.arm64 - "GSA/" would appear to be the user-agent to search for ('Google Search App'?) - the version number appears to match the version of the Google app that's installed on my phone. Ask your podcast host to add this now to their analytics.

via https://podnews.net/update/google-new-podcasting-trolls-weather

TODO: update useragents csv in repo, and find missing stuff.

Google Podcasts on Android: \bGSA\/
Google Podcasts on smart speakers: \bGoogleChirp\/
https://developers.google.com/search/docs/data-types/podcast#analytics

I see the GSA in there, I think we need to add GoogleChirp

Note: didn't actually see any GoogleChirp in bigquery.  Must be pretty uncommon?
We also split GSA matches between Android and iOS.


Fix: staking reward calculation

I think there is a little mistake in the staking reward calculation.

Dependencies added
hast-util-is-element

Descriptionhast utility to check if a node is a (certain) element
AuthorTitus Wormer
LicenseMIT
ContributorsTitus Wormer

rehype-katex

Descriptionrehype plugin to transform inline and block math with KaTeX
AuthorJunyoung Choi
LicenseMIT
ContributorsJunyoung Choi, Titus Wormer

remark-math

Descriptionremark plugin to parse and stringify math
AuthorJunyoung Choi
LicenseMIT
ContributorsJunyoung Choi, Titus Wormer

@bdeneux Based on your work, I have slightly restyled the table of inflations to show the values better, and also used katex (provided as a plugin for docusaurus)  to express the general calculation formula. Let me know if it okay for you.

@ccamel Nice, it's ok for me ! 👍


Fix typo in README

The method to init the UITableView version of SLKTextViewController is named initWithTableViewStyle:, not initWithStyle:.

Whoops. dzenbot, I think I'm wrong actually. Hadn't tested. Will make a pull request to undo.

Oh really. Okay, never mind then. Just having some trouble getting up and running.


WSL2: Seek of directory entry by lseek does not work on v9fs

Build number: Microsoft Windows [Version 10.0.19603.1000]
Seek of directory entry by lseek does not work on v9fs.
Code: a.c
#define _GNU_SOURCE
#include <dirent.h>
#include <fcntl.h>
#include <stdio.h>
#include <unistd.h>
#include <stdlib.h>
#include <sys/stat.h>
#include <sys/syscall.h>

struct linux_dirent {
    long           d_ino;
    off_t          d_off;
    unsigned short d_reclen;
    char           d_name[];
};

#define BUF_SIZE 1024

int main()
{
    int fd, nread;
    char buf[BUF_SIZE];
    struct linux_dirent *d;
    off_t off;
    int bpos;

    fd = open("./xxx/", O_RDONLY | O_DIRECTORY);
    syscall(SYS_getdents, fd, buf, BUF_SIZE);
    off = lseek(fd, 0, SEEK_SET);
    if (off < 0) {
        return 1;
    }

    while ((nread = syscall(SYS_getdents, fd, buf, BUF_SIZE)) > 0) {
        for (bpos = 0; bpos < nread;) {
            d = (struct linux_dirent *)(buf + bpos);
            printf("%s\n", d->d_name);
            bpos += d->d_reclen;
        }
    }
    return 0;
}

Prepare directory and running
mkdir ./xxx/
seq -w 0000001 0000040 | xargs -i touch ./xxx/{}.txt
ls xxx/ | wc -l
#=> 40

gcc a.c
./a.out | sort

Actual output
0000031.txt
0000032.txt
0000033.txt
0000034.txt
0000035.txt
0000036.txt
0000037.txt
0000038.txt
0000039.txt
0000040.txt

Expected output
.
..
0000001.txt
0000002.txt
0000003.txt
0000004.txt
0000005.txt
0000006.txt
0000007.txt
0000008.txt
0000009.txt
0000010.txt
0000011.txt
0000012.txt
0000013.txt
0000014.txt
0000015.txt
0000016.txt
0000017.txt
0000018.txt
0000019.txt
0000020.txt
0000021.txt
0000022.txt
0000023.txt
0000024.txt
0000025.txt
0000026.txt
0000027.txt
0000028.txt
0000029.txt
0000030.txt
0000031.txt
0000032.txt
0000033.txt
0000034.txt
0000035.txt
0000036.txt
0000037.txt
0000038.txt
0000039.txt
0000040.txt

On ext4 we get the expected result.
Strace
execve("./a.out", ["./a.out"], 0x7ffdc9af5ce0 /* 20 vars */) = 0
brk(NULL)                               = 0x556e019c6000
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
access("/etc/ld.so.preload", R_OK)      = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=25456, ...}) = 0
mmap(NULL, 25456, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7fe00ba2c000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\3\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\260\34\2\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0755, st_size=2030544, ...}) = 0
mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7fe00ba2a000
mmap(NULL, 4131552, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7fe00b41b000
mprotect(0x7fe00b602000, 2097152, PROT_NONE) = 0
mmap(0x7fe00b802000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1e7000) = 0x7fe00b802000
mmap(0x7fe00b808000, 15072, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7fe00b808000
close(3)                                = 0
arch_prctl(ARCH_SET_FS, 0x7fe00ba2b4c0) = 0
mprotect(0x7fe00b802000, 16384, PROT_READ) = 0
mprotect(0x556e0166a000, 4096, PROT_READ) = 0
mprotect(0x7fe00ba33000, 4096, PROT_READ) = 0
munmap(0x7fe00ba2c000, 25456)           = 0
openat(AT_FDCWD, "./xxx/", O_RDONLY|O_DIRECTORY) = 3
getdents(3, /* 32 entries */, 1024)     = 1008
lseek(3, 0, SEEK_SET)                   = 0
getdents(3, /* 10 entries */, 1024)     = 320
fstat(1, {st_mode=S_IFCHR|0620, st_rdev=makedev(136, 0), ...}) = 0
brk(NULL)                               = 0x556e019c6000
brk(0x556e019e7000)                     = 0x556e019e7000
write(1, "0000031.txt\n", 120000031.txt
)           = 12
write(1, "0000032.txt\n", 120000032.txt
)           = 12
write(1, "0000033.txt\n", 120000033.txt
)           = 12
write(1, "0000034.txt\n", 120000034.txt
)           = 12
write(1, "0000035.txt\n", 120000035.txt
)           = 12
write(1, "0000036.txt\n", 120000036.txt
)           = 12
write(1, "0000037.txt\n", 120000037.txt
)           = 12
write(1, "0000038.txt\n", 120000038.txt
)           = 12
write(1, "0000039.txt\n", 120000039.txt
)           = 12
write(1, "0000040.txt\n", 120000040.txt
)           = 12
getdents(3, /* 0 entries */, 1024)      = 0
exit_group(0)                           = ?
+++ exited with 0 +++

The following code reproduces the same problem.
#include <stdio.h>
#include <dirent.h>

int main() {
    DIR *dh;
    struct dirent *de;

    dh = opendir("./xxx/");
    readdir(dh);

    rewinddir(dh); // does not work

    while (de = readdir(dh)) {
        printf("%s\n", de->d_name);
    }

    return 0;
}

seq -w 0000001 0001030 | xargs -i touch ./xxx/{}.txt

ls xxx/ | wc -l
#=> 1030

gcc a.c
./a.out | sort
#=> 0001023.txt
#=> 0001024.txt
#=> 0001025.txt
#=> 0001026.txt
#=> 0001027.txt
#=> 0001028.txt
#=> 0001029.txt
#=> 0001030.txt

Nice post. Confirmed WSL2 19603.
As a work-around, I confirmed cifs is okay. You'll have to do without metadata though, natch.

Something like:
$ sudo apt install cifs-utils
$ mkdir ./mnt
$ sudo mount -t cifs -o username=you //xxx.xxx.xxx.xxx/Users/you ./mnt

Curious why this is closed since it does not seem to have been repaired. PHP maintainers believe this is the cause of the following issue: https://github.com/microsoft/WSL/issues/6081 which affects Docker for Desktop and makes this a chore. Is there any appetite to fix this with v9fs? I do not see a bug in the v9fs bug tracker regarding this.
https://bugs.php.net/bug.php?id=80227

Any update on this?  This breaks popular PHP applications that are likely to be run in WSL2 for development purposes, including XenForo.

I got problems with phpunit that does not discover all test files in a large directory. Find it strange that this still isn't fixed.

I there any temp workaround?

Suggestion: If wouldn't mind using VSCode it's actually better fully switch development into WSL and use VSCode as frontend editor more on that here. There is no overhead between translating different file systems and works like a charm and there is no speed/ram/cpu overhead.  I am running a PHP docker app with its code sticking out for development.

Any bro has been resolved?
I try to run the swoft project on docker on Windows, but some file can not read when loading.

Same problem with symfony/finder class, which uses php's RecursiveDirectoryIterator class.

Same problem when using squizlabs/php_codesniffer

Confirming the same issue with phpunit/phpunit and zircote/swagger-php
Windows 10 Pro 21H2

Could we add a bug label to this?

@benhillis, could you put a bug label on this so it doesn't get lost?  This is a pretty serious issue that prevents quite a bit of software from working within WSL2.

I have encountered the same problem. Has the error from 3 years ago not been fixed yet

Still preventing me from launching docker containers with Windows FS volume mounts 😞, it's been years now.

I needed to set up my project for emergencies on a laptop. However this bug makes Docker Desktop on Windows 11 Home edition unusable when combining it with PHPStorm. Simple solution on Windows 11 Pro edition is to switch to Hyper-V. But no, for some reason you can't do that in Home edition. So no point in using Windows for me I guess.

@selfer-work That is why I switch to the Ubuntu app and now I'am using projector, phpstorm runs in projector in ubuntu, the client in windows uses the weblink. And yes it is not ideal. Works better than the windows for docker.

(a quick "fix" is to put all source files "in" WSL, then open that directory in your PhpStorm (or whatever), the WSL files should be available at \wsl.local if I am not mistaking)

"dev drive" also can be used as "quick fix" for this problem: https://learn.microsoft.com/en-us/windows/dev-drive/

Great behavior. Just ignore the issues without reply. FU.

I've requested that this be reopened: https://github.com/microsoft/WSL/issues/12416


Add read-line command

Currently read-line can be implemented in terms of read-upto, but it would be nice to have a dedicated read-line command too.

Let me start by saying I haven't used MS Windows as a developer in more than two decades. So perhaps the rest of my comment is ill-informed regarding best practices for splitting text on line boundaries on MS Windows. In fact, I've found a few articles that talk about this. Such as https://www.theregister.co.uk/2018/05/08/windows_notepad_unix_linux_macos/ from two years ago which notes that Windows Notepad application finally does something reasonable when opening a file with UNIX line endings.
The problem with implementing read-line as a function built in terms of read-upto is that it seems impossible to make it work correctly on UNIX and MS Windows. Some of the difficulty in doing so was recently ameliorated by the introduction of the $platform:is-unix and $platform:is-windows variables. Those can be used to ignore \r on UNIX platforms.
The difficulty is how to handle the \r\n sequence with full fidelity on MS Windows if either char of the pair is missing. You can't just read-upto a \n then strip the \r, if any, that is in the returned string AFAIK to be strictly conforming with the MS Windows definition of a line. Perhaps being strictly conforming with old versions of MS Windows (and MS DOS) is no longer appropriate. And splitting on any of the chars individually, and the \r\n pair, is now acceptable and/or a best practice.

I am inclined to recommend that Elvish implement a read-line that accepts \r, \n or \r\n. Furthermore, that implicit splitting of command byte output be done using the same criteria. Regardless of the platform. Where more control is needed the read-upto command should be used. Although that represents difficulties since if you do x = (read-upto "\n") you presumably don't won't any trailing \r that remains to be removed.
It seems like this ultimately boils down to how easily we want Elvish to be able to process arbitrary binary data without using mechanisms like the slurp command (which might also need to be special-cased).

It is not possible to implement a command that accepts \r and \r\n at the same time (it wouldn't know whether to continue going after \r). None of the platforms supported by Elvish uses \r, so read-line will just support \n or \r\n.

Adding this to 0.14 since read-line can replace use of take 1 for reading a single line from byte input, and #923 is in 0.14.


Support Catalyst

After integrate RXDataSources via pod or SPM receive error:

Maybe for now no support Catalyst?

any on this?


How to distinguish "unsupported manifest" error from other 400 Bad Request errors on manifest push?

Remote registries return 400 Bad Request error on manifest push when:

The pushed manifest is malformed
The layers referenced by the manifest do not exist
The registry does not support the media type of the pushed manifest

Currently, oras-go is unable to distinguish case 3 from case 1 and 2 on manifest push.
This blocks the below scenario:

A client builds an artifact manifest and pushes is to a remote registry
The client finds that the registry does not support artifact manifest
The client fallback to push an image manifest to the registry

Related code:
https://github.com/oras-project/oras-go/blob/1969551cc3c77b46f09090f00360e8465871fe8f/registry/remote/repository.go#L1048-L1051

What would ECR return if we push a manifest, whose media type is unsupported by ECR?

ECR returns a http.StatusBadRequest error, with a sub-error code of UNSUPPORTED`.


Calling List external methods

Even after successfully setting a List ref (as shown in the tree.js example), I seem to be unable to successfully trigger a remeasuring of row heights using public methods (also shown in the tree.js example).
https://plnkr.co/edit/V2JhJL8ZGknrbYgRWDhf?p=preview

Hello! 😄
For this type of question, I suggest using Stack Overflow with a #react-virtualized tag or asking in the react-virtualized Slack channel. Issues are reserved for bug reports and feature-requests.
This helps keep the project somewhat more maintainable for me and hopefully gets you an answer faster (since more people check those places than monitor GitHub issues). 😄

Thanks! I'm asking here because I'm wondering whether there is a bug.

I understand. But since this project has a single maintainer (me) and since I'm pretty strapped for time lately, it's better (for me and you both) if we start with Slack / Stack Overflow. If, after doing so, you're convinced it's a bug then it's totally fine to open an issue here with repro steps, etc.
I don't think this is a bug though probably because there other are examples of doing this sort of thing. For example, this Twitter client which has its source here.

Thanks! The example was helpful.


Create a pi user and disable initial-setup

This branch is inspired in the work of @adammck in issue #3
If enabled with HEADLESS=1 in the settings.conf, it will create a pi user with password raspberry, like in Raspbian.
This user is member of the wheel group, so you can directly ssh to your Rpi and sudo from there.

Why try to be like raspbian with a 'Pi' user.
Why not 'raspi2' or something unique to fedora  ?

Thanks for the PR.  I'm not against having a have a default "pi" user as Raspbian does.  A few thoughts:
The script is basically reimplementing useradd pi + passwd pi.  Can we just write a service file, like initial-setup-text.service, to run on first boot that will run the commands to add the pi user?
We should probably configure the account so that the password must be changed on first login for some added security.

There is probably better methods to do this. I just wanted to share my modification if someone can find them useful.


Messages are not dismisable

Describe the bug
Whenever there is error on the graphql schema or even a message that tells me to input the fields I get a popup message that cannot be dismissed
To Reproduce
Steps to reproduce the behavior:

Download altair for windows using choco install altair-graphql or install altair chrome extension on brave
Update the app as per the request on the package
Input your graphql endpoint, look at the docs, import a mutation
See error

Expected behavior
Dismissable popup when clicking on the X
Screenshots

Desktop (please complete the following information):

OS: WINDOWS 10
Browser Brave
Platform: Chrome extension, choco install altair-graphql and updated app
Version <IP_ADDRESS>4

Additional context
Other dismissable popups not included in this report are not dismisable either

Hello @achatainga I don't see the issue you're referring to. The messages are dismissible for me.

@imolorhe I have the same issue as described.

In my case I just add a new window and save it to a collection. And that Added query to collection. message already cannot be dismissed.
At the same time the Something wrong with your query s {message: "Syntax Error: Unexpected <EOF>", message appears in the console right after the new window is added. So maybe this has something to do with the problem.

Does it happen regularly? That is, if you restart Altair, does it happen again?

This happens all the time, there was not a single case when I could dismiss the message.

App Version: 4.0.0
Windows 10 (18363.1379)
Tested on:

Firefox 88.0 (64 bit)
Chrome 90.0.4430.72 (Official Build) (64-bit)
Standalone for Windows

Reproducible in all of them.
Tested also on Mac. It was working there. So I suppose this is related to the OS.

Hmm.. wonder what's up with windows. Perhaps the toast package doesn't support something in windows - which I find weird, but I'll dig a bit deeper. Thanks for the info!

I'm also seeing this issue on macOS (Big Sur 11.4). Installed via Homebrew. However, I'm not seeing any errors in the console. Let me know if I can provide any further information.

@mloenow Yeah I'll need more information. What do you do before that happens? Does reloading the page fix it? Do you have a screenshot?

I'm also seeing this issue. Happens all the time with every Message. Didnt use Altair for a long time. Happened in Chrome with the plugin. Then installed the Windows Client and it also happens there. Force reload does fix it.

This seems to be fixed for me on macOS after https://github.com/altair-graphql/altair/pull/1643 👍

Yes had to replace the toast library

Fixed on Windows 10 App & Chrome extension on Brave/Opera/Chrome browsers


⚠ Bofrost.at has degraded performance

In d056bcc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1871 ms

Resolved: Bofrost.at performance has improved in 4a65ec6 after 6 minutes.


Document postgres --version option

Release Playbook

[ ] Rebase against master
[ ] Code review
[ ] Merge into master
[ ] Verify changes at http://site-staging.convox.com
[ ] Promote release on site-production in the convox/production Rack

 Document postgres --version option


Markdown highlighting doesn't work as it should

Hello,
I have been trying to ask this on IRC, but I'll write my question here too.
So, why Markdown syntax highlighting is not working? In Codemirror, when you surround text with - for example - two asterisks to make text bold, it becomes bold, but when doing same in Brackets, it has no effect.
See Markdown-mode demo here:
http://codemirror.net/mode/markdown/index.html
I write a lot using Markdown and I would like to have proper syntax highlighting for it.

You could create a Language in Brackets to use the CodeMirror mode, but I prefer using the Markdown Preview extension to see both the raw and formatted text at the same time.

I got it partially solved by installing Brackets Themes -extension and choosing theme right theme. So it seems to be theme problem partially. But then, if I hard-wrap words, the highlighting breaks, at least with lists so it's not problem only with theme.

I agree with @mkoske here, I wonder why Brackets doesn't add syntax highlighting to Markdown formatting when all the other Codemirror Markdown editors seem to do it? I use Markdown Preview too, but raw markdown should receive the syntax highlighting styles:
As in
**hello world** should show up in editor as

instead of

Oh, it's a bug:
See brackets_codemirror_overrides.less
/*
 * Temporarily override bold and italic syntax highlighting until
 * SourceCodePro supports them in a fixed pitch
 */
span.cm-em {
    font-style: normal;
}
span.cm-header, span.cm-strong {
    font-weight: normal;
}
span.cm-emstrong {
    font-style: normal;
    font-weight: normal;
}

Not sure if it has been fixed yet, the bolded letters seem to be very blurry but italics look okay?

I use Ttcn theme, and it works to some extent, e.g. I get bold text and italics too. But when I try to have list with hard-wrapped text, like this:
- item 1
- item 2. which is long
  and is hard-wrapped and intended,
- and this is next item

highlighting breaks. Hard-wrapped lines and item after that are not highlighted correctly. It works on the Codemirror demo-page I linked to in my first message.
So there's a bug somewhere.


Make AppConfig an interface

archaius-pull-requests #135 FAILURE
Looks like there's a problem with this pull request


Build fails on Sonoma 14.1.2 intel mac

Swift info
swift --version
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.108 clang-15<IP_ADDRESS>)
Target: x86_64-apple-macosx14.0

Output of swift build
swift build
warning: 'vulkan': ignoring declared target(s) 'Vulkan' in the system package
warning: 'vulkan': system packages are deprecated; use system library targets instead
Building for debugging...
error: unknown argument: '-fprofile-instr-generate'
[0/7] Compiling SPIRV-Cross-static spirv_msl.cpp

Not sure how to get around this

I am currently looking for a way to fix this issue.
As a simple workaround, you can open Package.swift in Xcode and build it, but IIRC, you will get an error because Float16 is not available on Intel Macs.
This issue has not been resolved yet. ( I have not tested it on the latest macOS-sonoma.),

I've fixed the issue, it was a deprecated linker option that was causing the problem. I'm closing this issue now, if you have any other issues please open a new one.

@Hongtae  Thanks for working on this. You are right it doesn't work for intel macs due to Float16 errors. Is intel mac not a target ?

As long as macOS continues to be released on Intel Macs, Intel Macs are a valid target.
However, since there is no way to distinguish CPU architecture in Swift, I need to find a way to support Float16 on Intel only. As far as I know, CPU distinction is only possible in C/C++/Objective-C.
I'm backlogged with other work right now, so it's hard to look into this issue in detail, I apologize.


Accessibility improvement: Screen Reader focus does not highlight TOS/Privacy Policy links

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
On sign up/sign in page, Screen Reader focus should move to "Terms of service and Privacy Policy" links.
Actual Result:
On sign up/sign in page, Screen Reader focus does not move to "Terms of service and Privacy Policy" links.
Action Performed:

Launch the app/webapp
Prerequisite: Turn ON the Screen reader.
Mobile: Navigate using Swipe gesture(From Left to Right) and verily.
Web: Navigate the page using arrow keys then verify.

Platform:
Windows/JAWS/Chrome
MACOs/Voiceover/Safari
Android/Talkback
iOS/Voiceover
MacWebapp/Voiceover
Notes/Photos/Videos:
https://user-images.githubusercontent.com/37308300/119418350-d4d15580-bcc5-11eb-89af-952a0a721cff.mp4
View all open jobs on Upwork
Job posting on Upwork: https://www.upwork.com/jobs/~0148dd67804d9a00f0

Triggered auto assignment to @arielgreen (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Hey, I'd like to work on this issue. Here's my proposal
I'm planning to resolve this issue by following the tips mentioned in this blog post (1st item is related to the problem we're facing here.)
https://callstack.com/blog/react-native-android-accessibility-tips/
We will modify the following module to add accessibilityRole="link" to Text components and we also need to split the phrase into 2 pieces but putting them into separate View components to make the links accessible separately.
https://github.com/Expensify/Expensify.cash/blob/c602342664a193a671399b7484d8721150895982/src/pages/signin/TermsAndLicenses/TermsWithLicenses.js
We may also need to use AccessibilityInfo.isScreenReaderEnabled to make it possible to access links by pressing Enter key.

@tugbadogan Thank you for your proposal. I think it is a good one, so you have a 🟢 to move forward with this.
I would not change your proposal, but I did have a thought that at some point it would be interesting to see if we could make some kind of accessibility component that makes implementing the screenreader functionality a bit easier. I don't really like having the duplicated openURL() calls.

@tugbadogan I just sent over an offer via Upwork.

Thanks for assigning this issue to me, I will send the PR soon.

@arielgreen PR caused some minor regressions

Hmm, this regression is caused by moving these texts out of nested Text component to make them accessible from Safari and iOS devices. We have two options to solve this regression, which one do you recommend here @roryabraham @tgolen @arielgreen  ?
Option 1) Split this long phrase into smaller phrases, so they can fit to single line, so other Text components can be positioned in  the same line https://github.com/Expensify/Expensify.cash/blob/3683d39ef4d0f5ab8dc7763ae67dfadfdeb4dff4/src/languages/en.js#L224
        phrase5_1: 'Money transmission is provided`
        phrase5_2: 'by Expensify Payments LLC',
        phrase5_3: '(NMLS ID:2017010) pursuant to its',

Option 2) Put these Texts and TextLinks into a nested Text component as before and add onPress prop to parent Text component, so the licenses link will be accessible in Safari and iOS devices and text wrapping will be handled properly.
https://github.com/Expensify/Expensify.cash/blob/83a06dd485f86ea08f8bd6fb116b5a24b0ce109a/src/pages/signin/TermsAndLicenses/TermsWithLicenses.js#L25-L31
will be replaced as follows
    <Text
        accessibilityRole="link"
        onPress={() =>
            AccessibilityInfo.isScreenReaderEnabled().then(
              enabled =>
                enabled && openURLInNewTab(CONST.LICENSES_URL),
    )}>
        <Text style={[styles.loginTermsText]}>
            {translate('termsOfUse.phrase5')}
        </Text>
        <TextLink style={[styles.loginTermsText]} href={CONST.LICENSES_URL}>
            {translate('termsOfUse.phrase6')}
        </TextLink>
        <Text style={[styles.loginTermsText]}>.</Text>
    </Text>

Solution 2 seems much better to me. Are there any downsides to that approach?

I realised that links in nested text component are now accessible in Safari. It might be changed by recent version upgrade for react-native. The only downside was duplicated links and codes in text component. But they are not required now. I attached the PR and it is ready for review.

Reopening, will close again upon payment June 16th.

Paid via Upwork.


setup your ide

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[ ] Improve performance

[x] Ehancement

Related issues

I think you misunderstand my point. We have source code in two places, one is GitHub, the other Apache source release. They are different, Apache soure release doesn't need git submodule thing, because we did that in source release procedure.

I think you misunderstand my point. We have source code in two places, one is GitHub, the other Apache source release. They are different, Apache soure release doesn't need git submodule thing, because we did that in source release procedure.

Hi, @wu-sheng . I got it. So here's the case with code clone from GitHub.

If you clone the codes from Github, please make sure that you had finished step 1 to 7 in section Build from GitHub

If download Apache soure release, we don't need that.

If download Apache soure release, we don't need that.

You still need that to generate some source codes from .proto files

If download Apache soure release, we don't need that.

@dmsolr You still need that to generate some source codes from .proto files

Yes. There are Run ./mvnw compile -Dmaven.test.skip=true to compile project and generate source codes. Because we use gRPC and protobuf in the steps.
But it is ok that we say it twice.


Mandatory params defined in NetworkControllerAdapter not shown for get cmdlet

Repro steps:

help new-NetworkControllerState

NAME
Get-NetworkControllerState
SYNTAX
Get-NetworkControllerState [-ConnectionUri ] [-CertificateThumbprint ] [-Credential ]
[-PassInnerException]  []
Mandatory parameter defined in NetworkControllerAdapter.cs ConnectionUri is shows as optional in ps.

TFS:8384570

This issue was moved to MicrosoftDocs/windows-powershell-docs#55


NotificationManagerCompat group/channel method missing

Why NotificationCompat.Builder has method to set channel id but NotificationManagerCompat can't create a channel?
https://developer.android.com/reference/android/support/v4/app/NotificationCompat.Builder.html#NotificationCompat.Builder(android.content.Context, java.lang.String)
https://developer.android.com/reference/android/support/v4/app/NotificationManagerCompat.html

It seem that the mothod setChannel in NotificationCompat.Builder don't work. Use Notification.Builder instead.

@zunpiau Thanks bro ... save my day

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


Refactored base code

Refactored base code

Any ideas why this is failing?

The flake8 tests are failing, stricter settings were set recently.
I'm a bit hesitant to merge style-only PRs, but we are planning a cleanup sprint and your PR is pretty clean. thx!

Done


bug fix: labelSelectorForVolume

labelSelector should add req and assgin back.
fixes: #24

/ok-to-test

/lgtm
/approve


Compile different runtime targets to different directories

This allows us to have multiple runtime targets in a useful way.

Users of the location of this library still need to be updated.


Can not download weights.

The pretrained_models.zip can not be downloaded.
wget https://polybox.ethz.ch/index.php/s/yR74R0XFRXAfUiU/download -O pretrained_models.zip
--2023-04-24 12:33:43--  https://polybox.ethz.ch/index.php/s/yR74R0XFRXAfUiU/download
Resolving polybox.ethz.ch (polybox.ethz.ch)... <IP_ADDRESS>
Connecting to polybox.ethz.ch (polybox.ethz.ch)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 503 Service Unavailable
2023-04-24 12:33:45 ERROR 503: Service Unavailable.

Could you please provide another link such as Google Drive to download this file?

Hello, thank you for your interest.
I just put it here: https://drive.google.com/file/d/1qSREgs8evfJQg8rfTTrn5dniNX1DY-ly

Thanks!


Sponsorship/Donation

Completely just floating this idea. Not saying this is going to happen. I was thinking it might be good to have a way for companies to donate towards the project aside from just hours of development time.

example: Marketing Company A might be super excited about CrankShaft, but feel their team can’t contribute code-wise due to current workload. They, however, do have money they could contribute if they knew that the money would be spent directly to further the project. A possible contributor in this fashion could even be HubSpot itself if they see the value in the project.

CrankShaft’s money would be used to reimburse individuals or companies that spend a significant amount of time contributing. The intent not being that companies or individuals are to profit, but offset their costs. A business that has a full-time employee, if they dedicate say 6 hours in a week, that’s almost a full business day, that the employee was not doing billable work. The company still has to pay that employee. This also incentivizes development to some degree. If a company had a slow month, or an individual wanted to make side cash, they could by furthering the project.
Actual logistics of how donations would work, how sponsorships would work, how disbursements of money are approved, the fairness of the system, etc. all of those details can be figured out(There are tools like Open Collective for this). We could do "bounties" on issues, or something. But I don't want to discuss that stuff until hearing peoples views on the idea itself. At this time I'd just want to figure out what people think to see if it's worth researching and investing time into.
My inkling is that a sponsorships system could put real fire behind the project accelerating it more rapidly because time spent on it would not be viewed as “lost business time”.
An example of an open source project this has been hugely successful for is VueJS. Evan You quit his Job because donations for the project were enough to live off - he’s not rich but he’s living about the same life as he had with a real job(no I’m not planning to do that). Because of his ability to increase time spent on the project, there are now tons of spin-off projects he’s helping, and he’s also more able to project manage the whole thing. Now the real benefit is that third party devs have actually gone and spent full work days, and in one case a dev took a vacation from work, and dedicated days to working on Vue, and the Vue project was able to reimburse that time spent. Making it less of a risk or financial burden on the companies and individual contributors.
Then sponsors also benefit from having their name associated with the projects, example VueMastery has donated the most money to Vuejs, and are now the top place to learn VueJS.
If everyone hates the idea, that's totally okay It was just a thought I had after listening to some podcasts.

again lets focus this discussion on IF we should have something like this, not the specifics of how it would work, we'd settle that later.

I've applied for access to the GitHub Sponsorship Beta.
Please let me know if you have any feedback at all.
For now, I've added a Paypal donation link for this repo. Any funds that get added to it will be used to further this project.
I don't think Paypal is the right funding method for us just because of the structure we'd want to ensure the funds go toward the project transparently, I just set it up to get something up there, we will continue to improve upon it. Once the individual sponsor system is in place I will consider removing this, if that system alone seems like a good start.
If you apply to join the github sponsorship beta, if you are active in this repository we can discuss adding you to the list of contributors in the funding area to encourage sponsoring you.
As stated in the past OpenCollective is an option for a process/mechanism of donations.

On a separate note - Luke has said he could help facilitate if I want to pitch having HubSpot Sponsor/ provide help in some way. I will collect thoughts on this and I think we should make more progress so it's easier to show it's potential before pitching it.


how to change version

I have mcprotocollib 1.12 how to make it possible to join on 1.8.8 version.

@Steveice10

You have to use an earlier build of the library for the Minecraft version in question.


depends_on & healthcheck

What should we add or change to make your life better?
Some kind of depends_on functionality where services don't immediately come up, and rather wait to come up until other services are either up successfully, or, responding to a successful healthcheck
Why is this important to you?

I have a scenario where a specific project cannot recover from being started before a dependent container is both up completed it's startup.
It is not clear without diving into logs whether the entire system is up or not and in a ready state. It would be nice if a healthcheck could be configured and then exposed to both depends_on and to the dashboard.

Some kind of depends_on functionality where services don't immediately come up, and rather wait to come up until other services are either up successfully, or, responding to a successful healthcheck

I think this is nice to have from a developer POV but bad in practice if you don't provide the same when you deploy. I think your application has to be resilient to this.

It is not clear without diving into logs whether the entire system is up or not and in a ready state. It would be nice if a healthcheck could be configured and then exposed to both depends_on and to the dashboard.

I think the dashboard could be improved to show what's running and healthy and what's not. Replicas and restarts do that but in a very subtle way.


fix: patch version

Changes

 fix type in build.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
195

Relevant Lines:
195

💛  - Coveralls


NIFI-6457 Add Upgrading NiFi section to Admin Guide

Drew, this is incredibly helpful and will be of use to a lot of community members. Thanks for doing this.
+1, merging.


C++ division operator overloading unwanted cast

When wrapping a class like so
# vec3.pxd
cdef extern from "math3d_c.h":
    cdef cppclass Vector3[T]:
        T x, y, z
        Vector3() except +
        Vector3(T x, T y, T z) except +

        Vector3[T] operator*(T &b)
        Vector3[T] operator/(T &b)

# math3d.pyx
from . cimport vec3

cdef class Vector3:
    cdef vec3.Vector3[double] c_vec

    def __init__(self, double x=0, double y=0, double z=0):
        self.c_vec.x = x
        self.c_vec.y = y
        self.c_vec.z = z

    def __mul__(Vector3 a, double scalar):
        return new_vector3_from(a.c_vec * scalar)

    def __truediv__(Vector3 a, double scalar):
        if scalar == 0:
            raise ZeroDivisionError("Vector3 division by zero")
        return new_vector3_from(a.c_vec / scalar)

# new_vector3_from just constructs a Python Vector3 object from a C++ Vector3 object, unrelated

The code generated by Cython for __mul__ is correct, however for __truediv__, Cython unnecessarily casts scalar to a Vector3.
// mul
 __pyx_t_1 = ((PyObject *)__pyx_f_6extlib_6math3d_new_vector3_from((__pyx_v_a->c_vec * __pyx_v_scalar))); if (unlikely(!__pyx_t_1)) __PYX_ERR(0, 30, __pyx_L1_error)

// truediv
__pyx_t_2 = ((PyObject *)__pyx_f_6extlib_6math3d_new_vector3_from((__pyx_v_a->c_vec / ((Vector3<double> )__pyx_v_scalar)))); if (unlikely(!__pyx_t_2)) __PYX_ERR(0, 43, __pyx_L1_error)

Which, of course, causes an error if the wrapped class doesn't provide an overload for that method.

That looks wrong. Might be a problem with the code that tries to find a common floatish type for true division.
Please also see
http://docs.cython.org/en/latest/src/userguide/special_methods.html#arithmetic-operators

I encountered the same behavior. All other arithmetic operators I used in my code work as expected, only overloading __truediv__() somehow leads to this unwanted cast.
I'm happy to provide more detailed information if necessary to help track the problem down, but the original text pretty much sums up the unwanted behavior.


Feedback to remove auto assigning label when mouse cursor out of canvas

As seen from the video
https://user-images.githubusercontent.com/33477318/121283838-62d84d80-c90e-11eb-8389-3b599e0e9028.mov
when a mouse cursor move away from canvas, the label auto assigned to the first option.
there's feedback about how keeping the label window open is preferable rather than auto assigning
Reasons being

its common to multi tasking , thus have mouse cursor to move away from current task of labelling
when came back to the current task of labelling, might overlook the current bounding box that already auto assign , the label assign might not be intentded
the similar situation can be seen when a user have multiple monitors, where cursor easily waver elsewhere unintentionally

Desired behaviours:
label window stays open even when window cursor move away

@amalsyahmi-certifai up to you to assign who to look into this . thank you

reflected in v2_alpha


X-Ray traces have no log information in CloudWatch

This is not a bug, just a configuration question. We've migrated from Java X-Ray SDK to AWS OTEL. And the thing we are facing now is the absence of log records while analyzing traces. This worked with X-Ray SDK:

However, with OTEL, the logs are not mapped (we use Fluentbit). The only difference we've found in the metrics/traces messages is the absence of the following key:
...
Segments: [
 {
  ...
  "document": {
   ....
   "aws": {
     ...
     "cloudwatch_logs": {
         "log_group": "our log group"
     ....

The aws.cloudwatch_logs.log_group is missing when using OTEL. The documentation says it translates from aws.log.group.names (https://aws-otel.github.io/docs/getting-started/x-ray#otel-span-cw-logs-metadata-translation).
We've tried configuration using environment properties, in OTEL_RESOURCE_ATTRIBUTES with no luck. What are we missing? Is it possible to map Fluentbit logs (visible in Logs/Log groups under the desired group and containing AWS-XRAY-TRACE-ID) with OTEL traces in a way we see the traces with the correlated log records?
Thank you.

Moved to Java instrumentation repo (https://github.com/aws-observability/aws-otel-java-instrumentation/issues/205)