Masking behavior of concatenation should be documented

System information

nothing is applicable except:

Keras version: 2.3.1

Describe the current behavior
The behavior of the Concatenate layer with respect to masks on its inputs is not documented.
Describe the expected behavior
It ought to be documented. If I am reading the code correctly, the behavior is what you would hope it is: a position is masked out if it is masked out in any of the entries along the final axis of the concatenation result.  However, it would be nice not to need to check the source to confirm this. :-)
More generally, it would be useful if behavior with respect to masking were a standard part of layer documentation (especially when it is not trivial)
Code to reproduce the issue
n/a
Other info / logs
n/a

@gabbard  Moving this issue to closed status as there has been no recent activity, in case you still face the error please create a new issue.Thanks!

@sushreebarsa: This issue notes a gap in the documentation. That gap still seems to be present.

Hey there @gabbard, would you be interested in contributing this?

@LukeWood I want to contribute to this.
can you tell me how do I contribute to documentation

Hey @Krystal225!  The API doc page linked above is generated from the class docstrings.
You can see this specific example at:
https://github.com/keras-team/keras/blob/master/keras/layers/merge.py#L429
Note we use doctests:
A brief overview... statements that output a value are tested against lines without prepended >>>:
>>> 'hello'
'hello'

passes
>>> 'hello'
'another value'

Would fail.  You can use this to demonstrate and validate behavior, as you'll see in https://github.com/keras-team/keras/blob/master/keras/layers/merge.py#L457
Please also be sure to review our Contributors guide!
https://github.com/keras-team/keras/blob/master/CONTRIBUTING.md
Let me know if you need any more information!

@Krystal225
here's an official documentation guide:
https://www.tensorflow.org/community/contribute/docs_ref

I have an issue
I tried doctest on https://github.com/keras-team/keras/blob/master/keras/layers/merge.py#L429
for demonstration. the test failed and said:
Failed example:
    print(x)
Expected:
    [[[ 0  1  2  3  4]
      [ 5  6  7  8  9]]
     [[10 11 12 13 14]
      [15 16 17 18 19]]]
Got:
    [[[ 0  1  2  3  4]
      [ 5  6  7  8  9]]
    <BLANKLINE>
     [[10 11 12 13 14]
      [15 16 17 18 19]]]
Trying:
    y = np.arange(20, 30).reshape(2, 1, 5)
Expecting nothing
ok
Trying:
    print(y)
Expecting:
    [[[20 21 22 23 24]]
     [[25 26 27 28 29]]]

need help on how to fix BLANKLINE

For testing I needed to import numpy as np at the beginning of merge.py
Should I directly add it to code and send a PR?

@LukeWood : I believe this should be assigned to @Krystal225 instead of me.

@LukeWood : I believe this should be assigned to @Krystal225 instead of me.

ah thanks -  I updated it.

For testing I needed to import numpy as np at the beginning of merge.py
Should I directly add it to code and send a PR?

Can you elaborate on what you mean here exactly?  If I remember correctly numpy is pre-imported in doctests (I may be wrong)

on my system I had to add
import numpy as np

in the beginning of merge.py for doctest to work.
other wise all tests that had .np in their code failed.
@LukeWood
Also is the BLANKLINE because of which the test at  https://github.com/keras-team/keras/blob/master/keras/layers/merge.py#L429 failed okay?
or we need to correct it?

Hello, Thank you for reporting an issue.
We're currently in the process of migrating the new Keras 3 code base from keras-team/keras-core to keras-team/keras.
Consequently, This issue may not be relevant to the Keras 3 code base . After the migration is successfully completed, feel free to reopen this Issue at keras-team/keras if you believe it remains relevant to the Keras 3 code base. If instead this Issue is a bug or security issue in legacy tf.keras, you can instead report a new issue at keras-team/tf-keras, which hosts the TensorFlow-only, legacy version of Keras.
To know more about Keras 3, please read https://keras.io/keras_core/announcement/


⚠ Samsung has degraded performance

In 6ef1db0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3092 ms

Resolved: Samsung performance has improved in a22c6b4.


Update quickstart example

Changes:

update example using bulk requests for medias and media objects creation

the changes in this PR were added with the hari uploader interface PR: https://github.com/quality-match/hari-client/pull/7


Adds support for interactive mode for disambiguation prompts. Closes #5053

Adds support for interactive mode for disambiguation prompts.
Linked Issue
Closes: #5053
Result
https://github.com/pnp/cli-microsoft365/assets/58668583/c9ca739f-fbe7-46e8-b831-9b742374d522

Hi @Adam-it, what a great and huge job this is! You rock! 🎉 of course this can't go without a few comments, but I trust you already expected that 🤙

Thanks @martinlingstuyl for the awesome review. I will amend those and add the missing 3 commands from the issue

@martinlingstuyl I:

resolved your comments, thanks for the awesome feedback 👍
added another command search externalconnection remove which was mentioned in the issue as 'problematic'
merged with main and resolved conflicts

I didn't:

add the site apppermission add and site apppermission set (also mentioned in the issue). TBH I am having some troubles with those and seem to always come to a point that I need to have nested Promise.Resolve which is rather hard to maintain 🤔. Instead of adding this functionality to the existing implementation I would rather suggest refactoring to async/await making it a looot easier, BUT since the refactor is not part of 50 other commands from this PR I would suggest opening a separate issue (I may already self-assigned myself and find a slot to implement it next month) in which we will refactor those two commands to async/await and add the disambiguation prompt as well. What say you? 😉

@martinlingstuyl I:

resolved your comments, thanks for the awesome feedback 👍
added another command search externalconnection remove which was mentioned in the issue as 'problematic'
merged with main and resolved conflicts

I didn't:

add the site apppermission add and site apppermission set (also mentioned in the issue). TBH I am having some troubles with those and seem to always come to a point that I need to have nested Promise.Resolve which is rather hard to maintain 🤔. Instead of adding this functionality to the existing implementation I would rather suggest refactoring to async/await making it a looot easier, BUT since the refactor is not part of 50 other commands from this PR I would suggest opening a separate issue (I may already self-assigned myself and find a slot to implement it next month) in which we will refactor those two commands to async/await and add the disambiguation prompt as well. What say you? 😉

sorry @martinlingstuyl for reposting but did you see my last response? is that acceptable for you?
is there anything else I may do (support you) in reviewing/rechecking this PR?

Oh, and also: could you fix the merge conflict as well? thanks in advance!

@martinlingstuyl ready for another round? 🙂

Hi @Adam-it, Ok so I went through all the files. There's quite some places where an array item is being updated. That's a quick solution, but i do have code-smell feelings about that. 😄 I think we better just return the data at once.
What do you think? Do you Agree with me on this?

Sure. Not actually sure I did the overwrites 🤔 but if it will work ok I will update the code

If you rebase on the latest main you'll also get my PR with listiteminstance ID deletions. They are in 8 files or so. Could you make sure those are executed above the code of the prompt?

👍

There are quite some comments, but we're getting close now. 😉

@martinlingstuyl did you also:

rechecked the changes for every command locally if works as expected?
take a look at the tests?

sorry for the additional questions but we aim to have it as part of v7 together with making interactive as default and I would like to avoid more turn arounds if possible 🙂

@martinlingstuyl did you also:

rechecked the changes for every command locally if works as expected?
take a look at the tests?

sorry for the additional questions but we aim to have it as part of v7 together with making interactive as default and I would like to avoid more turn arounds if possible 🙂
also @martinlingstuyl could you recheck your comments and add a bit more clarity to them 🙏.
Sorry for maybe some stupid comments 🙂 but my intention is to try to really understand the path you want me to take and then I am ready to 🌶🚀 on this 👍
BTW thanks for another extensive review and sorry for a hard PR to handle 😉

@martinlingstuyl I saw you answered to one comment but did you notice the other answer as well?
Also please double check the above so we may try to point out all required changes in a single (I hope last 😅) go instead of having a couple of turnarounds for a single comment which result every time in refactoring 100 files 😜🙂

Also please double check the above so we may try to point out all required changes in a single (I hope last 😅) go instead of having a couple of turnarounds for a single comment which result every time in refactoring 100 files 😜🙂

Hi @Adam-it, I did a random spot check, so I did not execute all commands. Seeing the amount of commands and things I'd need to set up here to make that happen that seemed too much. But I did enough to be confident of your work.
I also checked the tests of course.
I understand you don't want too many turnarounds. The reason this happened is that the first few turnarounds were quite big redirections in my eyes, so I gave it back to you without checking everything, as too much was going to change as it was. I'm confident we'll make it in time though. 👍

Also please double check the above so we may try to point out all required changes in a single (I hope last 😅) go instead of having a couple of turnarounds for a single comment which result every time in refactoring 100 files 😜🙂

Hi @Adam-it, I did a random spot check, so I did not execute all commands. Seeing the amount of commands and things I'd need to set up here to make that happen that seemed too much. But I did enough to be confident of your work.
I also checked the tests of course.
I understand you don't want too many turnarounds. The reason this happened is that the first few turnarounds were quite big redirections in my eyes, so I gave it back to you without checking everything, as too much was going to change as it was. I'm confident we'll make it in time though. 👍

Ok good news and thanks for the confirmation. I started to get the impression that you come back with always a single find which in general is fine for me 🙂 but in this PR every change is a couple hour/100+ files refactor + rebase hell 😅. And it's hard for me to plan other stuff to do over the weekend if this returns multiple times instead of one or two 🙂.
Ok I will start working on those review comments. Please also double check my answers what was the intention: return object (hard part) or wrap around () and return .id (easy) 🙏

Ok good news and thanks for the confirmation. I started to get the impression that you come back with always a single find which in general is fine for me 🙂 but in this PR every change is a couple hour/100+ files refactor + rebase hell 😅. And it's hard for me to plan other stuff to do over the weekend if this returns multiple times instead of one or two 🙂.

Hi @Adam-it, yes I get your point. 👍 We could have done one iteration less if I'd seen the message/errorMessage sooner. sorry for that 🙏

Ok I will start working on those review comments. Please also double check my answers what was the intention: return object (hard part) or wrap around () and return .id (easy) 🙏

I think I responded on one of the comments. to repeat:
"I personally dislike calling functions and using properties from the response in the same line. Less readable in my eyes.
But maybe this is a difference in tastes in code-styles. Feel free to ignore..."
Also: I made some mistakes in my suggestions (returning an entire object instead of just the property) you've already discovered that. Just fix those as you see fit.
Across the board there are two issues I'm commenting on:

updating array items
returning the property in the same line as the await.

Feel free to ignore the last.

Ok good news and thanks for the confirmation. I started to get the impression that you come back with always a single find which in general is fine for me 🙂 but in this PR every change is a couple hour/100+ files refactor + rebase hell 😅. And it's hard for me to plan other stuff to do over the weekend if this returns multiple times instead of one or two 🙂.

Hi @Adam-it, yes I get your point. 👍 We could have done one iteration less if I'd seen the message/errorMessage sooner. sorry for that 🙏

Ok I will start working on those review comments. Please also double check my answers what was the intention: return object (hard part) or wrap around () and return .id (easy) 🙏

I think I responded on one of the comments. to repeat:
"I personally dislike calling functions and using properties from the response in the same line. Less readable in my eyes.
But maybe this is a difference in tastes in code-styles. Feel free to ignore..."
Also: I made some mistakes in my suggestions (returning an entire object instead of just the property) you've already discovered that. Just fix those as you see fit.
Across the board there are two issues I'm commenting on:

updating array items
returning the property in the same line as the await.

Feel free to ignore the last.

Sir yes sir 🙂👍.
All clear now. Thanks 🙏👍

@martinlingstuyl ready for another go 👍
I took the approach you recommended 👍

Check, I'll review it asap!


CDIO Cyber Security Vulnerability Management Service

This PR is for the creation of the Vulnerability Management Service information page, providing details of the service and how to onboard to it. This will act as a central point for providing CO departments with further information.
Author: jonmichaelpitt<EMAIL_ADDRESS>

This feels more like a product page than guidance. Is there a better place for this content to live than the GDS Way? Related to #711.

@jonmichaelpitt: I know there's an internal-facing Cyber microsite being built. Can we close this PR now, assuming the content will end up on that site?


Arm64 alpine images

Feature Request
Support arm64 alpine images
Was there anything blocking this in the work for multi-arch support?

Yes support upstream, arm64 alpine port was only merged into Java 16 which the Jenkins project doesn't officially support.
It's been backported into Zulu and Bellsoft Java 11 which are different vendors to what we normally use but possibly could be done if someone were to PR it.
Eclipse Temurin which is our standard distribution has said they will backport it to 11 but I don't think it has happened yet

As of May 1, 2022, Eclipse Temurin only provides amd64 images for Alpine images.  Still blocked by lack of an upstream image to support it.

this is possible now, not blocked on upstream anymore

As announced here and seen here, Eclipse Temurin’s JDK21 Alpine images are now available for arm64 architecture.

Reopening this issue until the next weekly release with the official Eclipse Temurin images as base, providing arm64 architecture.

Forgot to close this issue, alpine arm64 images are available.


Reloading the UI makes the VM stats go away

⌘+R makes the stats go away:

This was fixed in a previous version. CMD + R is no longer available as a hotkey. Since the UI automatically updates upon any changes there shouldn't be a reason to refresh.
Thanks for the bug report!


refactor(patches): move static observable stubs from Observable.ts to…

… patch files
Related to #1284, and I'm fine with rebasing once #1284 gets merged.
cc @masaeedu @kwonoj

:+1: Thanks for PR, let's take #1284 first, hope you do not mind. (I know you already mentioned it's ok ;) )

cc @blesh also for visibility, @blesh , this PR changes organizations of static method as below

now Observable.ts does not include static method stubs by default
as same as patch operator, importing patch will dynamically import static method as well as its types
No changes for Rx / Rx.KitchenSink

consumer perspective 2 is major change, now user who imports Observable only for partial operator import would need to import static method as well with this PR. Question would be if we want this or not. In terms of 'everything is modular' it's somewhat expected way though.

One thing to note, today they still have to import the static method, but a TypeScript consumer will see that the method "exists" then when they try to use it they'll get a method not found error.
This change makes it obvious that method doesn't exist and that they need to import it.

Yes, my terms are not precise. @david-driscoll 's correct.

Will this all work correctly if the consumer is using an older version of TypeScript? (say 1.7 or 1.8?)

I believe some of the changes are being back ported to 1.8 but I would have to confirm that.  This would also affect #1284.  @masaeedu do you know if this is being back ported to 1.8?  I thought I saw something about that.

I believe it should work at least on 1.8, for earlier version need to verify it. Primary goal is not to regress and consumer should not care about typescript version dependency. (this, #1284 both)

https://github.com/Microsoft/TypeScript/issues/6731 is tagged as version 1.8.2

Just tried and verified this change won't work with previous version of typescript, i.e 1.7. So if typescript side fix are in 1.8.2, supported min version will be 1.8.2. This feels bit breaking - will there be user want to stay with 1.6, or 1.7?

This change won't work with TypeScript 1.7.5, no. You need to have a version of the compiler with the external augmentations feature enabled in order for #1284 and this PR to work, so you need to have either the nightlies or 1.8 (which is indeed being back ported to).

My personal opinion is that there aren't any very serious reasons for a developer to want to stick to TS 1.7, since the breaking changes between 1.7-1.8 are so few, and the new features so many. That said I appreciate that breaking changes should be taken seriously, and will leave it to the maintainers to decide how to proceed.

I personally living edge of versions so wouldn't mind to upgrade and would vote for this changes, but due to that reason I'm not able to represent actual consumer perspective. @blesh , what do you think? If version support is concern, I think we can hold off these changes for now since this is not blocking any functional scope of current codebase.

TypeScript is currently published at version 1.7 and 1.8 is beta. So we need to support 1.7, unfortunately.

published at version 1.7 and 1.8 is beta. So we need to support 1.7

: That seems quite legit, valid point. @masaeedu / @david-driscoll , in that case I'd like to suggest to hold this changes until TypeScript release bumps up to 1.8 to avoid versioning issues - any suggestions around?

Let's have the Angular folks chime in on this too, since this is their big ask:  @IgorMinar @jeffbcross @vsavkin @robwormald

definitely :+1:  on this PR in general, but we do need to hold until this functionality is released properly. we can't ask users to install a specific TS edge version (not to mention then having to override their editor to use it as well...)

I agree it's hard to recommend consumers to tie with bleeding edge version of infrastructure.
Once I got agreement from @david-driscoll and @masaeedu , I'll put on hold both of PR and re raise once TypeScript bumps up its stable version includes this support. Hope it comes soon, I also quite like this changes and it possibly can bring additional benefit to codebases.

Oh it's fine to hold it. We can't all ride on the bleeding edge. :+1:

@kwonoj No objection here, waiting for 1.8 makes sense.

Note for when we get back to this: Microsoft/TypeScript#6858 fixes module imports so that re-exports aren't needed. TL;DR, all those import 'foo' -> import 'foo'; export * from 'foo'; changes are no longer required.

Great news :)

@david-driscoll Since this only works in bleeding edge TypeScript anyways, I think you can also take advantage of Microsoft/Typescript#6858 and get rid of the export * froms. Imports are fixed so those are no longer needed.

@masaeedu did you pull them out in your branch?  I can do that no problem.

@masaeedu Is that feature has planned tagged version? I.e 1.8.2 has module aug and if those comes later (1.9 or something else), it might need to be separated.

@kwonoj Yup, it'll be in 1.8.

There are a bunch of changes in https://github.com/Microsoft/TypeScript/commits/release-1.8 (the release tracking branch that they are migrating changes into).

https://github.com/Microsoft/TypeScript/pull/6846
https://github.com/Microsoft/TypeScript/issues/6762
https://github.com/Microsoft/TypeScript/pull/6906

@david-driscoll Haven't had time, will get to it on the weekend. No rush, right? :cry:

@masaeedu just want to make sure we're lined up! :smile:
Now just watch them release 1.8 tomorrow. :scream:

This looks good to me but we need to coordinate merging this with a TS release and an Angular release. We can't ask people to be on unstable release of typescript.

TS 1.8 has released (https://github.com/Microsoft/TypeScript/releases/tag/v1.8.2). Might need align with ng2 still though.

as of ng2beta7 we're 1.8 friendly I believe.

as of ng2beta7 we're 1.8 friendly I believe.

: Lovely :heart:

@david-driscoll , I'm closing this PR per #1388. Let me know if this is incorrect.


[BUG] Primitive Alloy Smelter Doesn't Serialize burn time

Just needs adding to saveAdditional and load

I'm going to go ahead and take a stab at this.


Add more basic attacks

There are many possible attacks that have not yet been implemented.
Some of these include:

Phrase-level attacks

Invert part-of-speech order
Change tense

Replacing words with homonyms, or symbols that are pronounced as a homonym (ate -> eight, for -> 4)
Surrounding characters (or other alternating patterns) (bank -> (b)(a)(n)(k))

homonyms lookup - http://www.speech.cs.cmu.edu/cgi-bin/cmudict


Can't find EVP_MD_GET_size on FreeBSD 13

I don't know where the problem is.
A temporary workaround is to add this flag -L/usr/local/lib to LFLAGS in Makefile.

This is no longer a problem


Semver docs endpoint

Bump version for App Catalog Release

Merging despite Snyk Security Fail as that will be addressed via other PRs


Active portals not working

Thread was weirdly locked without a fix
https://github.com/quek04/undergarden/issues/189#issue-2175127530 is the original thread link
Can activate the portal fine, however when walking inside of it/standing inside of it. Nothing happens, every other dimensional portal works except the Undergarden. This is on the RAD2 pack specifically but the error occurs separate to the pack as well
Occurs in multiple versions, notable 1.16.x and 1.21.x
Other dimensional mods within the pack work concurrently without any issues at all, the issue solely resides in this mod.. even when running by itself

Thank you for reopening this and providing more details. Not sure why the previous one was closed. If you find anything else out at any point, please post it.
Can't say I've been able to reproduce in 1.21, but we will look into it.

Undergarden 0.5.5 on 1.16.5:
https://github.com/user-attachments/assets/935ce5b0-36fa-428c-87e1-50a3c9862847
Undergarden 0.8.21 on 1.21.1:
https://github.com/user-attachments/assets/4761e2b5-0b36-4aa9-a2df-0e80b8023fa5
don't know what you want from me

I'll try that mod pack later to see if I have the same problem. I'll report back here once I do.


Condor provider has a hard-coded cmd_timeout.

This timeout should be configurable and probably default to something much longer.
https://github.com/Parsl/parsl/blob/1d8c9e35752274af6ae2ce2f07107474ece4561c/parsl/providers/condor/condor.py#L225
cc @ZhuozhaoLi who noted this in a comment to #889

Also reported by @lgray


Dependency on XStream, even when not using any XStream serializers

It's currently not possible to use Axon without XStream getting configured, even when XStream is not used for any serializers.
<dependency>
    <groupId>org.axonframework</groupId>
    <artifactId>axon-messaging</artifactId>
    <version>${version.axon}</version>
    <exclusions>
        <exclusion>
            <groupId>com.thoughtworks.xstream</groupId>
            <artifactId>xstream</artifactId>
        </exclusion>
    </exclusions>
</dependency>

JpaSagaStore:361

private Supplier serializer = XStreamSerializer::defaultSerializer;

Caused by: java.lang.NoClassDefFoundError: com/thoughtworks/xstream/mapper/CannotResolveClassException
at deployment.war//org.axonframework.modelling.saga.repository.jpa.JpaSagaStore$Builder.>(JpaSagaStore.java:361)

Same Issue was with QuartzEventScheduler :
Axon 4.5
[#1685] Remove QuartzEventScheduler hard dependency on XStream #1690
Remove hard dependency on XStream in QuartzEventScheduler #1685

I'll provide a pull request for this issue.

The Change would be like in https://github.com/AxonFramework/AxonFramework/pull/1440/files

@smcvb A fix is provided in https://github.com/AxonFramework/AxonFramework/pull/1932

I've changed the type from "bug" to "enhancement," as I wouldn't describe this as a framework bug at this stage.
Of course, it is not ideal, hence why a pull request is warranted.
From that end, thanks for the effort, @mvonrenteln!
I'll take a look ASAP.


Recipe Deployment is broken

While I was on vacation the deploy action broke.
About 2 weeks ago it started raising an error like this:
Submitting job...
Calling subprocess with cmd = ['pangeo-forge-runner', 'bake', '--repo=.', '--json', '-f=/tmp/tmp5num_3t5.json', '--Bake.job_name=a618127503-8331357441-1']
The virtual environment was not created successfully because ensurepip is not
available.  On Debian/Ubuntu systems, you need to install the python3-venv
package using the following command.

    apt install python3.10-venv

You may need to use sudo with that command.  After installing the python3-venv
package, recreate your virtual environment.

I have merged a few dependabot updates (#105 #106 #107 #108 #104), but that did not resolve the issue. I am not sure if this is a change somewhere upstream or in -runner that would cause this error.

Fixed via https://github.com/pangeo-forge/deploy-recipe-action/pull/28


Fix update to overwrite the value

When updating the value it was failing
Fixes #

@pflorek  I'm afraid a recent breaking change was introduced in AWS SSM that cause the CustomResource from @pepperize/cdk-ssm-parameters-cross-region to throw The parameter already exists. To overwrite this value, set the overwrite option in the request to true. which causes UPDATE_ROLLBACK_FAILED on stack update


feat(congregation): allow multi-edit of person assignments

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


Update Makefile to Support VCS Optionally

Motivate of the pull request

[ ] To address an existing issue. If so, please provide a link to the issue:
[x] Breaking new feature. If so, please describe details in the description part.

Describe the technical details
What is currently done? (Provide issue link if applicable)

Optionally add VCS support to run tests on both FPGA-level and Unit-level. Make sure to override the SIM=modelsim option with SIM=vcs under dv makefile: https://github.com/RapidFlex/alkaid_efpga/blob/2144f6b980e807a6677c8bf27118496ebc5ac6fc/dv/Makefile#L11

What does this pull request change?

Which part of the code base require a change

[ ] RTL
[ ] Gate-level netlists
[ ] Design Verification
[ ] Physical Design
[ ] Timing characterization
[ ] Documentation
[ ] Regression tests
[ ] Continous Integration (CI) scripts

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

@tangxifan These changes works for me on both unit-level and fpga-level. I am able to run tests with both iverilog and VCS. please try running test at your end using modelsim and see if that works for you as well. Once you give me green signal, i will update all the Makefiles accordingly.
PS: I am able to run on pl and gl netlists, getting fabric_netlists.v file missing error. Please double check it on your side

@doc build errors have been fixed in #30 . Please merge the latest master to your branch.

@anaszameer-rs Checked on local servers using VCS. Working well. Thanks!


/fdfe/acquire

it seems starting with Google Play Store version 11 (2018), that /fdfe/acquire is now being used instead of /fdfe/purchase. does this project have any plans to update to the new request?

https://github.com/EFForg/rs-google-play/issues/18


[feature] RSS feed for each single change-detection-url

Is your feature request related to a problem? Please describe.
I propose a RSS feed for every change-detection-url. Currently one can only view a RSS feed containing all monitors or filter by tag-id.
Describe the use-case and give concrete real-world examples
tba
Additional context
tba

hmmmm but this could get extremely cpu expensive, because for every change (row) in your RSS feed it would need to re-calculate all the differences
so if you have 100 watches, and each watch has 100 changes, then the machine has to calculate 10,000 differences every time you load the RSS

so at best, I think it just needs a list of the 'history' and not show any changes in the RSS, but you need to be more clear about what it is that you are trying to solve in your RSS feed i think
what is the goal here?

closing - idle

@dgtlmoon hello!
i have similar request. My use case:
I have page https://www.nasdaqomxnordic.com/news/listings/firstnorth/2024
And want to create rss from this page.
Example
<item>
<title>Canatu</title>
<pubDate>17 Sep 2024 00:00:01 GMT</pubDate>
</item>

<item>
<title>Solar Foods</title>
<pubDate>9 Sep 2024 00:00:01 GMT</pubDate>
</item>

...

I am not sure if it's right use case for your project. Maybe i should go with rss-bridge or rsshub. I just preferred your project cause it's python (and not php (rss-bridge) or node.js (rsshub))


Design improvements

Initially I wanted to do minor tweaks of some colors, fonts, line height, etc. But I could not stop in time, and ended up changing quite a lot of code!
In my eyes, it all makes sense; however I do understand you might have different vision. Feel free to reject, of course. Still, curious to hear your feedback!
Here is the comparison video. Your repo is on the left; my fork is on the right.
https://github.com/user-attachments/assets/17db29df-532f-467a-b734-918c26ce2c4a

testing with more themes...
https://github.com/user-attachments/assets/ff418f00-53e6-4188-aa6c-7a147afa5b02

What! This is so cool, I'll play around with this!

In retrospect, I should have been much more granular in my suggestions. The recent upstream update introduces many conflicts, which are probably not worth resolving. I'll close this PR and maybe will send some other PRs with very local changes that would be much easir to review.

Thank you for being so considerate @vladstudio


Generate SBOM and Provenance metadata buildx attestations

What would you like to be added?
Ratify publishes images to GHCR. Ratify should generate and attach SBOM + provenance metadata to the published images as docker build x attestations
Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

There are 2 approaches here:

Use docker buildx's --attest capability to generate Provenance and SBOM intoto attestations. These are attached to the image index as OCI images. This does NOT use the referrer method. However, multiple projects including GK already use this approach. It is also the simplest to implement.
Generate SBOM SLSA provenance manually using corresponding tools and then use ORAS to attach to the image

Closing since buildx attestation support has been added. New issue will be created to add SBOM and Provenance metadata as referrers artifacts attached to image.


feat: migrate to Anteater API

This PR shouldn't have been necessary since in an ideal world we would have skipped API-Next, but oh well.
Summary

Set up codegen for types using openapi-typescript, remove ppapi-next-types, and fixup existing usage.
Move fetching and most processing logic to backend, and set up tRPC routers and procedures.

Notably, we now rely only on the API cache layer for WebSoc, since we don't do anything fancy with it and a 5 minute TTL is probably reasonable. Because the backend now makes the requests, everyone fetching the same endpoint with the same params will hit cache, and as a result we can "invalidate" the cache by passing in a new dummy param.
Still caching grades on the frontend though since there's some Interesting logic there.

Move away from GraphQL and nuke dead code.

Test Plan
Tested locally, server related things need staging for tests.
Also someone else should run through their normal routine and verify that everything still works as expected.
Things That Still Need Doing

[x] Migrate fuzzy search.
[x] Once icssc/anteater-api#30 gets merged we can remove the local declarations for the prerequisite tree.

Future Follow-Up

Replace Arktype with bare TS types and Zod where validation is actually necessary, since all our other projects are using Zod. (Sorry Brian.)
There's a bug where the API tries to return enrollment history data objects for courses we don't have enrollment data for. This needs to be resolved on the API end but for now I've added a shim in the tRPC router.

Now tracking this in icssc/anteater-api#31.

#1030
#802 take 2

Some searches aren't working:
"ICSH32"/"ICS H32" return nothing.
"MAE93" returns nothing, even though it works right now.
"ICS139W" seems to work with fuzzy, but then no results show up

Should replace these messages about PeterPortal API with Anteater API


feat(colcon-build): enable cpu resource control

Description
Follow up from:

https://github.com/autowarefoundation/autoware-github-actions/pull/308

Related:

https://github.com/autowarefoundation/autoware.universe/pull/8498

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

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

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

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

[ ] The PR follows the pull request guidelines.

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

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

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

Ok it works:

https://github.com/autowarefoundation/autoware.universe/actions/runs/10456320423/job/28953375853?pr=8498#step:13:348

I'm open for variable name changes. Ready for review.


Third homework about

only about page for now

Hey, @ryvik13, please add all files from the task to the pull request. I'm changing it to draft, undraft it, when everything is ready to review.


THRIFT-4399 plugin.thrift t_const_value is not used as a union in C++…

… code -- fix this
plugin.thrift defines t_const_value as a union.  But in plugin_output.cc
and plugin.cc, the converters clearly either (a) SET NEITHER of
identifier_val & enum_val, or (b) SET BOTH.  But these are two different
fields in the union.  So clearly, the type t_const_value isn't being
treated as a union.
I think we need to fix Thrift's treatment of unions, but independently,
the plugin should use Thrift's type system in a correct manner.  This is
easy-to-fix, but since the current plugin relies on a bug, the fix will
be a breaking change.

BTW, this fix is necessary if I'm going to fix unions so they transport exactly one of their fields.  B/c (of course) right now, the plugin code depends on that not being the case.
I have the C++ code (compiler + lib) ready-to-go-and-tested, but (obv) can't submit a PR until this code goes thru, b/c it would break the build.


However, it seems like you're lacking manners more. Regardless of that, you can install the missing package yourself using the pip tool.


Reg chan

connect https://github.com/resin-os/resinos/issues/309

With this change, we are able to control the regulatory domain (country) of the wifi interface, or interfaces, attached to a board. The request to set a given domain is sent at boot time. This is controlled by the new "country" setting. A config.json for a fresh new board which sets this feature might look something like this:
{"persistentLogging":false,"hostname":"resin","uuid":"xxx","deviceApiKey":"xxx","country":"GB"}

@agherzan can this be merged?

Sure, I approved this PR some time ago (5 days ago) and don't see any changes since. All good from me.


Board Ameba_AMB82-MINI (platform AmebaPro2, package realtek) is unknown

Under a new ubuntu22.04 environment.
Arduino: 1.8.19 (Linux), Board: "AMB82-MINI, Disable, Disable, Normal Mode, Yes, Yes, NA"

Board Ameba_AMB82-MINI (platform AmebaPro2, package realtek) is unknown

Error compiling for board AMB82-MINI.

This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Thanks for the post. It is an know issue and going to be fixed by next release.

@Oliver0804  resolved by 06b716d1da1c02bb4521e4383f75456fe42bdeb7


Appveyour times out on master

Appveyour is currently timing out on master. We should trim it a bit for it be be under the 1h limit. Then we might move it out of appveyor.

Here's an example of a full log: https://ci.appveyor.com/project/AngularCLI/angular-cli/builds/24627859?fullLog=true. It times out at 89/95 tests.
If you search for Last step took you can see how long each test took. Tests that do many builds can take up to 6 minutes, like the update tests.


Disable a character for new memory choice if they don't have any (more)

As it is, the game lets you choose a modded character to unlock a new memory for, even though that character has no memories, or you already unlocked all memories it has (if it has less than 3). This confuses new modded players.

Done in upcoming Nickel 1.0.0.


Carousel doesn't stop dragging when releasing the button outside of the container

Describe the bug
Carousel doesn't stop dragging when mouseup event is triggered outside of carousel container.
To Reproduce
Steps to reproduce the behavior:

Go to https://brainhubeu.github.io/react-carousel/docs/examples/simpleUsage
Start dragging the carousel
While still pressing the button move the cursor outside of carousel container
Release the button
Notice carousel is still dragging while you're not pressing the button

Expected behavior
Carousel should stop dragging when releasing the button (no matter if it's in the container or not).
Screenshots

Desktop (please complete the following information):

OS: [macOS 10.14.6]
Browser [Chrome]
Version [79.0.3945.88]

IssueHunt Summary

 sinchang has been rewarded.
Backers (Total: $5.00)

 piotr-s-brainhub ($5.00)

Submitted pull Requests

#180 Fixed Carousel doesn't stop dragging when releasing the button outside of the container

Tips

Checkout the Issuehunt explorer to discover more funded issues.
Need some help from other developers? Add your repositories on IssueHunt to raise funds.

IssueHunt has been backed by the following sponsors. Become a sponsor

@sinchang
I have a question: are you able to withdraw any funds from the issuehunt?
I ask because after writing to<EMAIL_ADDRESS>asking them for an invoice and waiting more than one week, I still have no reply.
And in order to withdraw any funds, it's required to send an email to<EMAIL_ADDRESS>as well.


playlist: fix EXT-X-PLAYLIST-TYPE=VOD bug

EXT-X-PLAYLIST-TYPE=VOD to
EXT-X-PLAYLIST-TYPE:VOD

thanks for noticing and fixing the bug.


[FEAT] Have an optional to reset from when close modal inside useModalForm

Is your feature request related to a problem? Please describe.
Hi, I would like to have option to reset the form value to defaultValues when use close() in useModalForm hooks.
There are 2 issues, I'm facing:

When using create useModalForm({refineCoreProps:{action: 'create'}) I want to reset the form when user just close() (cancel by chance) and when user open modal again, the previous value is still there.
When using edit useModalForm({refineCoreProps:{action: 'edit'}), when close modal and open another modal by call show(id), the previous data is still inside the new modal and waiting to replace after fetch is completed. This will make user experience confuse sometimes when the internet is slow.

To bypass the issue, we call reset() then close().
Describe alternatives you've considered
It would be nice to have another option inside useModalForm when user want to reset form after closing the form not just by summitted. For example
useModalForm({
    refineCoreProps: { action: 'create' },
    defaultValues,
    modalProps: {
      autoResetFormWhenClose: true // <-- something like this
    }
    resolver: valibotResolver(schema)
  });

Additional context
N/A
Describe the thing to improve
N/A

Hey @Dominic-Preap and @alicanerdurmaz ,
I would like to work on this issue. Can you assign it to me? But please note that I do need to take some time to understand the code base and to go through the contributing guidelines. Thanks

@Dominic-Preap and @alicanerdurmaz, I am interested in this issue. Can you please assign it to me? Also, can you please guide me on the solution

@Dominic-Preap I'd like to work on this one. Please assign it to me, I'd be happy to contribute.

Hey @GANES1998 assigned issue to you, if we don't hear back from you, we'll assign it to @Xajid

Hi @BatuhanW, Thanks for assigning to me. I'll work on this.

Hey @Dominic-Preap , I am interested to work on this issue. Can you please assign it to me if it is still unsolved ?

@bhargavpshah98 are you still interested working on this issue?

Hello @BatuhanW , Yes I am interested

Hey @Dominic-Preap  and @alicanerdurmaz , if this is still available can u assign it to me ?

Hey @bhargavpshah98, it seems the notification lost and didn't see your comment. Just assigned to you, looking forward to your PR!
@abdulqdaer-q since @bhargavpshah98 asked earlier, I assigned to him. We can assign to you if he won't work on this one.

hey @BatuhanW, is this issue still open? if so please assign it to me, I'd like to contribute.

Hi @BatuhanW, I can work on this issue, can you assign this to me if it's still unresolved?

Hey everyone! I had some time to break down the issues here.
For the action: "create" persisting the dirty values; I think useModalForm({ resetOnClose: boolean }) can be added for this behavior 🤔
I hope understood correctly @Dominic-Preap, when calling close(); show("1"); modal is kept open and values are reset to the previous value for a moment (until the next fetch is complete). I think when setting the initialValues in useModalForm().formProps we currently do:
https://github.com/refinedev/refine/blob/013e975f15087561435b9e31c813b9d119d19d39/packages/antd/src/hooks/form/useForm.ts#L318
By default, queryOptions.keepPreviousData is set to true by Refine. By changing this to false, I was able to clear the previous values while fetching the new record.  🚀
@paoloLigsay Can you also check if setting keepPreviousData: false helps in your case as well? 🙏
I think the change can be done here in useForm hook of @refinedev/antd and work for useModalForm too.
https://github.com/refinedev/refine/blob/013e975f15087561435b9e31c813b9d119d19d39/packages/antd/src/hooks/form/useForm.ts#L256
If this resolves the issue, we're open to contributions for the fix 🙏

autoSubmitClose doesn't appear to be wroking either

@BatuhanW
can you please assign this to me i will love to contribute .

Hey @aqir-sapaad assigned to you.

sorry it was my work email can you please assign to me here @BatuhanW

Ran into this issue myself. I did a little playing around and if I use a create action but still set an id, for example and empty string "", the reset on cancel works.
I traced that id around in code for a bit and it appears that this happens because the resetFields is called in this useEffect:
https://github.com/refinedev/refine/blob/26d4065117001290a6f8d94f3f78c37e213b8947/packages/antd/src/hooks/form/useForm.ts#L275
I think this useEffect is triggered when setId(undefined) is called in the handleClose of useModalForm here:
https://github.com/refinedev/refine/blob/26d4065117001290a6f8d94f3f78c37e213b8947/packages/antd/src/hooks/form/useModalForm/useModalForm.ts#L292
When the action is set to edit you have to specify an id so setting it to undefined triggers the useEffect. But on create when you don't set an id it's already undefined and doesn't trigger.

Doing this does appear to have the problem of defaultFormValues not being set if I then re-open the create modal though

Hey, @Dominic-Preap  I'd want to contribute to this issue  🚀

Hey @ChevrierDev I've assigned this issue to you.

just faced with this issues. Tried to edit and save, and then open the same item, the form is empty. But if i open other items, the value is there again. It seems like when i open the same item (this is probably reading from id), the value didn't get inserted into the form.
And, I tried to edit some value and then cancel; then when i open the same item again, the value is the value i cancelled before, not value from data fetching

Hello @BatuhanW, could you assign this issue to me? I'd love to fix it once and for all.

I somehow solved this problem by setting cacheTime:0 and structuralFileSharing:false in the queryOptions

Hi there.
If this issue is still unsolved. I would like to work on this feature to add an optional state reset when closing the modal inside useModalForm. Could you please assign this issue to me?
Thank you!

Hello @BatuhanW, could you assign this issue to me? I'd love to fix it once and for all.

Assigned issue to you.
@imHardik1606 if we don't hear back from @abkr08, we'll assign it to you.

Thanks, @BatuhanW. I will get it sorted.

@Dominic-Preap I would love to contribute to this issue.

@Ananmayd assigned issue to you.
Please make sure to thoroughly read and follow our contribution guide. PRs that doesn't follow our contribution guide, will be closed. Thanks.
https://refine.dev/docs/guides-concepts/contributing/


version 2.0.0 seems to ignore format string

sneak@pris:~$ CoreLocationCLI -once YES -format '{ "lat": %latitude, "lon": %longitude }'
<+37.REDACTED,-80.REDACTED> +/- 134.13m (speed -1.00 mps / course -1.00) @ 10/8/16, 00:05:58 Eastern Daylight Time

sneak@pris:~$ CoreLocationCLI -once yes -format '%latitude : %longitude'
works
sneak@pris:~$ CoreLocationCLI -once yes -format 'x %latitude : %longitude'
works
sneak@pris:~$ CoreLocationCLI -once yes -format '"lat" %latitude : %longitude'
fails
sneak@pris:~$ CoreLocationCLI -once yes -format 'lat %latitude : %longitude'
works
sneak@pris:~$ CoreLocationCLI -once yes -format '{ lat %latitude : %longitude'
fails

really would defaulting to json output be so bad?

Hmmm interesting that the other ones work but that one fails.
I would love JSON output everywhere. But convention seems to be that command line tools do not output JSON. Maybe we can make a -json option, but it would have to be streaming json.

Convention is wrong now that jq -r exists. Also, I always use -once.

That makes sense, -json can imply -once. I'm not sure how jq -r works.

I'm not sure how jq -r works.

@fulldecent See the manual for jq.
@sneak I don't think jq can be considered a standard/convention by now. It is still far too unknown.

Why doesn't CoreLocationCLI output all information via JSON (like accuracy, speed, direction)?

Why doesn't CoreLocationCLI output all information via JSON (like accuracy, speed, direction)? Right now, -json is a showstopper for me because it doesn't give me accuracy.

bump.

Also, it seems like you made backwards-incompatible changes with v2.2.0 😢


Support Google Cloud SQL connect sockets

Support Google Cloud SQL connect sockets, fulfill README claim [2], and fix for issue #132.
We use this PyPI package import at work now, and I see some other PR's I'd like to integrate here, so I'm willing to take over maintenance at my fork. I can also rename to use a different PyPI package name if desired.

Obsoleted by https://github.com/jazzband/dj-database-url/pull/181 which is now merged


Add events messages

Hi I would like to work on this

Great! I'll assign to you! :smiley:

What file do you want the events on? Sorry, I'm new to github

You can put a JSON file in the src folder, for example ;)

I can't do this without the commands being created though correct? Or maybe its bc im noob :/

Don't worry, the commands is not exists yet, but it's possible to create a list of messages or create a class with some consts with the messages to be use later, or something like that :smiley:

Fixed!


Add ability to safely call functions

Add support for calling functions within the current scope. Should allow specifying in-scope variables as arguments to these function calls.
call myFunc(localVar, 4, "foo")

Comment moved here from PR #331:
I did think a bit about calling functions (without writing any code so this is purely philosophical) this is what I thought would be the pain points:
a. panics
b. breakpoints getting hit while evaluating a function
c. our goroutine getting scheduled out -- which makes (b) worse.
Regarding (a), one solution is to set a breakpoint on panic and abort the evaluation as soon as it is called, before it starts unwinding the stack. This is not ideal, the panic could be recovered by a frame below our dummy frame, but given how panics are generally used by go code it should be a big deal.
The other possibility would be to abort the call only when the stack unwind gets to our dummy frame, possibly modifying the g struct to slip in our own fake _defer object?
Regarding (b) and (c) I have two possible solution
Orthodox solution
For (b) we disable all user breakpoints. (c) is hard: I'm not sure it's possible to suppress co-operative scheduler calls, maybe abort on them? Even pre-emptive scheduler calls may be hard because they seem to be at least in part intertwined with stack resizes (which we can not suppress).
Unorthodox solution
We call Continue in the middle of Call, any breakpoint that gets hit is reported to the user normally, the rest of Call is executed when its temp breakpoint (which would have a condition on the goroutine id) gets hit, in a callback executed by Continue.
This is easier to implement because we don't have to worry about (b) and especially (c) but it's weird for the user.
> main.somefunction() somefile.go:10
(dlv) print a
3
(dlv) print double(a)
Command failed: function calls are not supported

The user must use a different command to enable function calls, this command will have a behaviour similar to continue and use a similar API, let's call this command exec:
(dlv) exec double(a)
> main.someotherfunction() somefile.go:50

the function call did not complete, the user got a breakpoint somewhere else in the program, the user can interact with the debugger:
(dlv) continue
> main.somefunction() somefile.go:10
    double(a) = 6
(dlv)

after another continue the original exec call completes and the user gets the evaluation result: double(a) = 6.

@derekparker @aarzilli Do we need some changes in Go itself for this issue?

@ignatov: likely yes. On top of the design problems in my previous message there is also another thing to be considered. Let's say we want to initiate a call, from the debugger, to a function F from goroutine G.
We shall call the activation frame currently on top of the stack for G the "dangerframe"

We can't overwrite memory in the dangerframe because the garbage collector could run while F is executing and collect live object because we overwrote the dangerframe, so we need to add memory do G's stack (because goroutines stack are tight): how?

Let's say there was enough space on G's stack to push our arguments, space for the return value and some fake return address. Now we change RIP and restart the process to execute F. If the garbage collector runs and tries to unwind G's stack it will get hopelessly confused once it gets to the dangerframe because we made dangerframe extra fat by adding one extra arguments area at the end of it.

Even if this doesn't happen the dangerframe is not stopped at a safe point so the garbage collector wouldn't have a good pointer bitmap to determine which pointers on the dangerframe are live.

@aarzilli maybe we should file an issue in https://github.com/golang/go/issues? We had a conversation with @rsc, and he told us that Go team is going to provide a good basement for debugger at least.

@ignatov Did you create an issue at https://github.com/golang/go/issues ?
I'm new to Go and was trying to run a func in a delve session and I came across this issue.
I'm asking, since I would like to file an issue in the Go repo if you haven't. Especially now that Go is collecting feedback on stuff that should go into Go2, I think a better debugging experience should be of utmost importance and having an issue open with them would greatly help.
cc @derekparker @aarzilli

@komuW

Did you create an issue at https://github.com/golang/go/issues ?
No, I didn't. Could you please do it?

Pardon my jumping in here, but could we side-step all the GC related issues by disabling it during the function call?
I can think of 2 risks to doing this.

function call goes nuts and consumes all memory. I think the risk here is pretty minimal, and acceptable if it gets us function calls.
function call explicitly triggers a GC. I'm not sure how the delve interacts with the Go runtime, but it seems like we might be able to set a breakpoint on runtime.GC() so that we can clean up the stack, or make it a noop.

Is it supported now?

GoLand from JetBrains is blocking a feature because of this issue:
https://youtrack.jetbrains.com/issue/GO-3433
Just an FYI

Given that https://go-review.googlesource.com/c/go/+/109699/ is merged into go 1.13, which has been released, does there now exist a reasonable estimate when this feature will be ready?

@coriolinus this feature is already implemented in the latest release of Delve, however it has been left open as there are still some improvements to be made.

That's excellent to know, thanks!
On Thu, Mar 7, 2019 at 6:29 PM Derek Parker<EMAIL_ADDRESS>wrote:

@coriolinus https://github.com/coriolinus this feature is already
implemented in the latest release of Delve, however it has been left open
as there are still some improvements to be made.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/go-delve/delve/issues/119#issuecomment-470619306, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHdeTnM3kaO_e8ChiWlkH-OH9xiGqyHFks5vUUyFgaJpZM4EOwJX
.

Going to officially close this out as we now have this functionality. We will continue adding more improvements and @aarzilli has done a ton already.


update version string to 1.4.0

update version string to "1.4.0"

Codecov Report
Patch coverage has no change and project coverage change: -0.02 :warning:

Comparison is base (ebdfc6c) 79.99% compared to head (4b407ae) 79.98%.

Additional details and impacted files
@@                Coverage Diff                 @@
##           releases/1.4.0    #1107      +/-   ##
==================================================
- Coverage           79.99%   79.98%   -0.02%
==================================================
  Files                 257      257
  Lines               29031    29031
  Branches             5689     5689
==================================================
- Hits                23224    23219       -5
- Misses               4478     4480       +2
- Partials             1329     1332       +3

Flag
Coverage Δ

macos-11_Python-3.8
?

ubuntu-20.04_Python-3.8
79.98% <ø> (+<0.01%)
:arrow_up:

windows-2019_Python-3.8
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


dataTakeID parameter not included in the MPH of RAWS_ products

According to the MPH Definition document v2.3, the bio:dataTakeID parameter:
/bio:EarthObservation/om:procedure/eop:EarthObservationEquipment/eop:acquisitionParameters/bio:Acquisition/bio:dataTakeID

is mandatory for RAWS<PID>_<PC> products.
I used the following configuration for the simulation of the Level-0 Step 1 processing:
{
    "mission": "biomass",
    "scenarios": [
        {
            "name": "20170225T094527 - LPPFStep1_025",
            "file_name": "L0PFStep1",
            "processor_name": "L0PFStep1_025",
            "processor_version": "01.00",
            "task_name": "L0PFStep1",
            "task_version": "01.00",
            "log_level": "debug",
            "data_takes": [
                {
                    "data_take_id": 6632654,
                    "start": "2017-02-25T09:46:04.620Z",
                    "stop": "2017-02-25T09:58:34.884Z"
                }
            ],
            "begin_end_position_from_toi": true,
            "outputs": [
                {
                    "type": "RAWS025_10",
                    "metadata_source": ".*RAW_025_10.*"
                }
            ]
        }
    ]
}

Procsim generates 9 output products but the bio:dataTakeID parameter is not present in the MPH.
Please find in attachment the working directory for the test: WD-L0PFStep1.zip.

We don't have version 2.3 of the MPH document we only have 2.1. Is this changed?
According to v2.1, the datatakeId is only mandatory for L0, L1, AUX_ATT and AUX_ORB. In fact, not any acquisition parameter is needed for RAWS.

With the merge of PR #44, RAWS products can now be generated with data take parameters. The data take ID should be included in their MPH.

I confirm that RAWS products may contain more than one data take. E.g. consider the case where an input RAW contains two data takes (DT1 and DT2) which are separated by 32 seconds (minimum data take separation). The temporal gap is shorter than a slice, thus it may happen that one slice contains final part of DT1 and initial part of DT2.
The issue is fixed!


failed build init app rn 0.70

<EMAIL_ADDRESS>(124, 18): Unresolved reference: deleteRecursively
FAILURE: Build completed with 2 failures.
1: Task failed with an exception.

What went wrong:
Execution failed for task ':callstack_repack:compileDebugKotlin'.

Compilation error. See log for more details

Can you tri this: https://github.com/callstack/repack/issues/184#issuecomment-1172448429 and confirm if that works?

Closing as issue became stale and this version is no longer supported anyways.


Prevent dependency of a PR on itself

Currently it is possible to create a PR and mark it to have a dependency on itself.
For example when I create the PR #5 and then add a comment to this PR /depends on #5 then dep-bot changes the commit status with the message "Blocked by #5".

Is this still applicable after #11 is merged? Since that's a big refactor in progress, it may be best to wait until that's complete for a change on this.

Yes, it's applicable.

it may be best to wait until that's complete for a change on this.

👍

I think the fix is trivial but it does not make any sense for a user to do this.
It handles an edge case that is unnecessary IMO


Fix error with large decimals in parseWithBigInt

What was changed
If JSONBigNative.parse(content) fails, catch and return JSON.parse()
Why?
A bug in json-bigint throws an error on decimals over 15 digits. This catches the error and falls back to JSON.parse()
Unit tests were added
Fixes issue #1171

Just noting that a single JSON document can technically contain both a large decimal number AND a big integer. In that case, big integer will loose precision.
But I don't think this is that much of an issue.

Yeah good call out. I want to get this fix out now and then work on a fork that handles both.


Unnecessary cases created in switch

Playground: https://rescript-lang.org/try?version=v10.1.2&code=DYUwLgBAzg7glmAxgCwKIA8CGBbADqCAXggAo4A7XAVzAC4IKwBKIgPggG8AoCaeJZA0o1OPXhAD0EiOQD2YOIhAzZEAKwQQ6OFDBQIyEACcQAQjEAfCABY2NyxACMABjsuHjgExvPDgPp2zmIAvlzBQA
Notice how in ReScript I provided 3 specific cases + a default one, while the transpiled code contains much more.

This should probably be in the compiler repo instead: https://github.com/rescript-lang/rescript-compiler


Missing functions on Zipper

[x] drop current
[x] updateCurrent
[x] filter?
[x] focus (by predicate)
[x] append / prepend (without changing the focus)

released in 1.1.0 https://github.com/turboMaCk/non-empty-list-alias/releases/tag/1.1.0


Set up CI to run tests and build site to review pull requests

The template came with a CircleCI file that should serve as a starting point.

To test the build of the website, I was running this...
#https://github.com/gjtorikian/html-proofer
htmlproofer --assume-extension --check-html --http-status-ignore "429"  ./_site

Would rather it check for 429 errors (too many requests), but I'm not sure how to use htmlproofer to slow down page requests. It currently errors on pages from twitter since I'm on a Twitter fast until Easter

I have it working. The only things that are still cludgey are the 429 and I only look for 4xx errors. There are a few pages that exist when I manually enter them (e.g. amazon, rdp) but that seem to be timing out, even though I give them extra time to load.


allow import types exported by ng-redux.d.ts

I am trying to define a function selector in my app, but I want to be able to type it . accordingly like:
import { FunctionSelector, NgRedux } from '@angular-redux/store';

export const mySelector: FunctionSelector<IAppState, boolean> = ({ user }) => user !== null;

But the type FunctionSelector is not exported inside  '@angular-redux/store'
It would be really handy to allow users to reuse those type definitions.

Shouldn't be to hard to add this, will try and address it soon.

Yup I like this idea.

Coming in 6.2, soon.

This has been implemented in @angular-redux/store 6.2.0. Let me know if you run into any issues.

Thanks guys! I'll try it out


rethink search

(1)
currently, search requests must always be dispatched explicitly by clicking the "magnifying glass" button. e.g. adjusting the date range slider, will not by itself dispatch a new request. similarly, when clicking a keyword "chip" in the list view, or in a details panel, the selected keyword will be added to the search autocomplete, but will not dispatch a search request (EDIT: however, in the "list all entities" screen, clicking an entity will dispatch a search request). the same is true for adjusting the search options, e.g. changing the query mode from "intersection" to "union".
are we happy with this behavior? or should we change to dispatch requests "on change", not "on submit"?

(2)
currently, the search form allows setting some basic search options. one of those settings is selecting specific entity types:

however, while it may seem that this allows configuring which entity types will be included in the result set, this actually just configures which entity types will be excluded from the options displayed in the autocomplete dropdown -- which is an unusual pattern for search UI.
i would vote for removing this config, and generally making clearer that search will always return passages (i.e. entities of type stelle). for example the autocomplete placeholder text should not read "Search for Authors, Passages, Keywords, Case Studies, or Places". similarly, the initial help text with recommended search terms should not read "Search our database for medieval authors, passages, keywords (such as names of peoples) or case studies."
afaiu by default, search always returns a filtered set of passages (or spatial coverages associated with a passage in the map view, or keywords associated with a passage in the graph/wordcloud view). it is only possible to search for other entity types via the dedicated "List all entities" screen, which has a different, non-configurable search form.

(3)
currently it is possible to search over multiple selected case studies - do we want to keep this? there is also some tension between these search views, and the dedicated case study entry point

(4)
it's not totally clear that some of the search config settings affect not only the main search form, but other requests as well - e.g. requests in the detail panels will respect the has_usecase setting.

(5)
the search autocomplete also doubles as an "author" select for the compare authors graph

solved via port to nuxt


Swal not closing only in IE11.

After clicking ok on a simple SWAL, the box is removed however the background overlay still remains and page remains faded out and unresponsive.
swal(
                'ERROR!',
                'Please check email and/or captcha',
                'error'
            ).catch(swal.noop)

See Image:

For bug reports, please include:

a jsfiddle in order for us to be able to reproduce the issue.
expected behavior of the plugin
actual behavior

JSFiddle template: https://jsfiddle.net/limon/ad3quksn/

See Contributing.md


Change database name

From mern-starter to something more related to the project...

This is done.


Basic GATE support

Done to teach myself how to make modules from scratch, not incredibly useful at the moment aside from dodging Daigoro in slice is right
TODO: tie stuff to animations where possible instead, especially in slice is right

I don't know. seems very unfinished. not even sure yet if the typhon attacks aren't serverside until the player is frozen in place

Work finished in #300


Consider adding formatting

It might be useful to add custom formatting for some types, e.g. TimeSpan.  The default display for TimeSpan isn't that good, the Humanize method looks much better,  Use case:
Measure-Command { dir }

The output should be, well, humanized.  In my own personal profile, I use something like:
<View>
<Name>Timespan</Name>
  <ViewSelectedBy>
    <TypeName>System.TimeSpan</TypeName>
  </ViewSelectedBy>
  <CustomControl>
    <CustomEntries>
      <CustomEntry>
        <CustomItem>
          <ExpressionBinding>
            <ScriptBlock>
$d = $_.Days; $h = $_.Hours; $m = $_.Minutes; $as = "","s"
$(if ($d) { "{0} day{1}," -f $d, $as[$d -gt 1] }
  if ($h) { "{0} hour{1}," -f $h, $as[$h -gt 1] }
  if ($m) { "{0} minute{1}," -f $m, $as[$m -gt 1] }
  "{0}.{1}s" -f $_.Seconds, $_.Milliseconds.ToString('D3')) -join " "
</ScriptBlock>
          </ExpressionBinding>
        </CustomItem>
      </CustomEntry>
    </CustomEntries>
  </CustomControl>
</View>

Thanks Jason, will do.

The format for TimeSpan is finished. I'm trying to get a format for DirectoryInfo and FileInfo, but I'm struggling.
Something like:
Mode   LastWritten    Length (MB) Name
----   -----------    ----------- ----
d----- 3 months ago               addins
d----- 2 months ago               appcompat
d----- 2 months ago               AppPatch
d----- 23 hours ago               AppReadiness
d-r--- 3 days ago                 assembly
d----- 3 months ago               bcastdvr

Neat idea.  PowerShell Community Extensions does the "Length (MB)" thing, but not the cool LastWritten.
You can look at how they did it: http://pscx.codeplex.com/

Getting closer. Works for either DirectoryInfo or FileInfo, but not both. Back to reading the help.
posh> ls -file

Mode   LastWritten   Length Name
----   -----------   ------ ----
-a---l 3 months ago   61 KB bfsvc.exe
-a--s- 6 hours ago    66 KB bootstat.dat
-a---- 6 days ago     10 KB cfgall.ini
-a---- 4 months ago    1 KB cfgrs.ini
-a---- 4 months ago   134 B cfgrs_ex.ini
-a---- 2 months ago    6 KB comsetup.log

Looking good

Look at C:\Windows\System32\WindowsPowerShell\v1.0\FileSystem.format.ps1xml - you'll see SelectionSetName - it lets you specify formatting for a group of related types.  Also note EntrySelectedBy - this can be used to do formatting on just one of the types in the group.


InvalidArgumentError when calling model as layer with ConvLSTM2D

Hey guys,
I have problems when I call a model that contains a ConvLSTM2D layer. The code works fine when I do not hand over the initial states for the ConvLSTM2D layer, but when I do I get an InvalidArgumentError error.
Here is an minimal example to reproduce the error:
import keras.backend as K
from keras.layers import Input, ConvLSTM2D
from keras.models import Model
import numpy as np

# create actual network
network_input = Input(batch_shape=(1, 1, 3, 3, 1), dtype='float32', name='network_input')
initial_state_1 = Input(batch_shape=(1, 3, 3, 1), dtype='float32', name='initial_state_1')
initial_state_2 = Input(batch_shape=(1, 3, 3, 1), dtype='float32', name='initial_state_2')
conv_lstm_out = ConvLSTM2D(filters=1,
                            kernel_size=(2, 2),
                            padding='same',
                            return_sequences=True,
		            #stateful=True)(network_input) # with this line instead the code works
                            stateful=True)([network_input, initial_state_1, initial_state_2])
main_network = Model([network_input, initial_state_1, initial_state_2], conv_lstm_out)

# create input placeholders and call network as layer
net_input = K.placeholder(shape=(1, 1, 3, 3, 1), name='net_input')
net_states_1 = K.placeholder(shape=(1, 3, 3, 1), name='net_states_1')
net_states_2 = K.placeholder(shape=(1, 3, 3, 1), name='net_states_2')
network_op = main_network([net_input, net_states_1, net_states_2])

# create output function to run 'network_op'
predict_function = K.function([net_input, net_states_1, net_states_2], [network_op])

# create input data
net_input_data = np.arange(9, dtype=np.float32).reshape([1, 1, 3, 3, 1])
net_states_1_data = np.arange(9, dtype=np.float32).reshape([1, 3, 3, 1])
net_states_2_data = np.arange(9, dtype=np.float32).reshape([1, 3, 3, 1])

# run output functions (here the error occures)
net_output = predict_function([net_input_data, net_states_1_data, net_states_2_data])[0]
print('output', np.squeeze(net_output))

The exception that is thrown is:
tensorflow.python.framework.errors_impl.InvalidArgumentError: You must feed a value for placeholder tensor 'initial_state_2' with dtype float and shape [1,3,3,1] [[{{node initial_state_2}} = Placeholder[dtype=DT_FLOAT, shape=[1,3,3,1], _device="/job:localhost/replica:0/task:0/device:CPU:0"]()]]
Does anyone knows a fix to this? Is this a bug in keras itself or in my code?
Keras: 2.2.4
TensorFlow: 1.12.0
PS:
In general, if you want to pass the initial states to a ConvLSTM2D layer as symbolic tensors you need to uncomment the first two lines in __call__ method of class ConvRNN2D(RNN) in  keras/layers/convolutional_recurrent.py so that the code looks as following:
def __call__(self, inputs, initial_state=None, constants=None, **kwargs):
        #inputs, initial_state, constants = _standardize_args(
        #    inputs, initial_state, constants, self._num_constants)

See #9761 for more information.

I also meet this bug and cannot feed state as initial_state even using reset_state() or the parameter of initial_state. There is _standardize_args() should be passed at least twice when Using ConvLSTM2D. I find the initial_state would  get the value from input as the following code at first _standardize_args() because the type of initial_state is a list .
if isinstance(inputs, list):
    assert initial_state is None and constants is None
    if num_constants is not None:
        constants = inputs[-num_constants:]
        inputs = inputs[:-num_constants]
    if len(inputs) > 1:
        initial_state = inputs[1:]
    inputs = inputs[0]

But at second _standardize_args() , the type of inputs is still list and the inital_state is not None because it has been get value at first _standardize_args(). then  isinstance(inputs, list) is True but the initial_state  is not None. I guess this is the reason that the assert is error

I modify source code : 'output = super(RNN, self).call(full_input, **kwargs)' to replace 'output = super(ConvRNN2D, self).call(full_input, **kwargs)'  at keras.layer.convolutional_recurrent.py 321th line so that it can run without any error information.


Insufficient permissions

So recently I discovered, that I can restrict specific blocks for areas trough permissions. But this all fails, when I use global areas or stacks:
a) For global areas there is no permission option in Front-end drop-down, neither in Blocks and stacks admin area (tab is nor present).
b) There is a permission option for Stacks (I can again restrict blocks) in admin area, but when adding stacks in FE, again, I can't restrict specific stacks to Areas.
So whenever I use global areas or stacks, it's that thing where Editor can add everything or nothing.
v8.1.0
P.S. - Any tips, hooks, advice highly appreciated.
P.S.S. - it's not just about preventing editors from doing something stupid, it's making choice lists shorter, more logical and thereby user-friendlier.

In version 7 there was a possibility to give permissions for global areas. This function is missing in 8.x - I opened a ticket in bugtracker for this: http://www.concrete5.org/developers/bugs/8-1-0/permission-settings-missing-for-global-areas/

@typoman76 what about Stacks? Was there an option to limit stack's to an Area?

I have resolved the issue reported by @typoman76 – you can now give advanced permissions to global areas through the global area interface in the Dashboard (this will be a part of 8.2.) Regarding the ability to limit stacks the same way you can block types in the front-end advanced permissions dialog – this would be a nice addition.

@aembler thanks for the fix and enhancement status. Greatly appreciated.


Fix ca crash (1.0.2)

Two related issues were causing a seg fault in the ca application. Firstly it is possible to create a certificate with an empty Subject. Secondly if ca reads an index.txt file with a missing subject in an entry then it will crash.
This is the 1.0.2 version of #5114.
Fixes #5109

Empty subjects DNs (empty RDN sequences) are actually valid, and are normally printed as "/" on output, and were accepted via the -subj / option on input.  In particular, per RFC5280:
   The subject field identifies the entity associated with the public
   key stored in the subject public key field.  The subject name MAY be
   carried in the subject field and/or the subjectAltName extension.  If
   the subject is a CA (e.g., the basic constraints extension, as
   discussed in Section <IP_ADDRESS>, is present and the value of cA is
   TRUE), then the subject field MUST be populated with a non-empty
   distinguished name matching the contents of the issuer field (Section
   <IP_ADDRESS>) in all certificates issued by the subject CA.  If the
   subject is a CRL issuer (e.g., the key usage extension, as discussed
   in Section <IP_ADDRESS>, is present and the value of cRLSign is TRUE),
   then the subject field MUST be populated with a non-empty
   distinguished name matching the contents of the issuer field (Section
   <IP_ADDRESS>) in all CRLs issued by the subject CRL issuer.  If subject
   naming information is present only in the subjectAltName extension
   (e.g., a key bound only to an email address or URI), **then the subject
   name MUST be an empty sequence** and the subjectAltName extension MUST
   be critical.

I should also point out that some mainstream CAs (e.g. Let's Encrypt) include only a DNS name "CN" in the subject DN, but this is limited to ~64 bytes, and longer CNs are disallowed.  In such cases one is forced to use an empty subject DN.  Therefore, this PR is incorrect, and the change must be reverted.
Instead the ca code MUST handle empty subject DNs gracefully.  Of course such DNs cannot be required unique, so they should be excluded from the uniqueness checks.

Closing this in favour of tracking it as part of the original issue, #5109.


Grouped sensors graph not working for kWh

Originally reported here: https://forum.mycontroller.org/topic/327/issues-with-group-sensor-graph/3

Fix is available in SNAPSHOT version


[Logging] Supporting logging presets (pre-configured sinks)

Need to be able to quickly configure common cases for logging output. Trim excess metadata, possibly re-order keys. Presets might need to be composable. We can define presets in code using the fluent interface, but then also need to be able to dump them to config to be used as templates for more complex custom logging requirements.

Done in #666


Issue27

Fixes #27

Looks good.  Does this work with the new Controllers in master?

It does now 😺


Move Features out of proposals

We believe Features are now in a state to be merged into the actual spec rather than a proposal. Also fixed a few typos and consistency issues.

Corresponding containers.dev PR: https://github.com/devcontainers/devcontainers.github.io/pull/62.

Great point @samruddhikhandale!
I've now also opened https://github.com/devcontainers/features/pull/243 and https://github.com/devcontainers/feature-template/pull/14.


Support for Fedora Server 21

Hi and thank you for the excellent work!
With the latest forever-service running on Fedora Server 21 32bit, I saw the following message
forever-service version 0.4.6

This platform is not yet supported by forever-service
To help us add this platform, please contibute to https://github.com/zapty/forever-service

Fedora Server uses Systemd, but in /etc/init.d/README is stated that:
Note that traditional init scripts continue to function on a systemd
system. An init script /etc/rc.d/init.d/foobar is implicitly mapped
into a service unit foobar.service during system initialization.

To test it, I've modified this line: https://github.com/zapty/forever-service/blob/master/templates/sysvinit/installer.js#L11 by adding |(Fedora):
if( contents && contents.match(/(Amazon Linux)|(Red Hat)|(CentOS)|(Fedora)/g) ){

and then when I ran forever-service again I was able to install the service and everything is working fine. I opened the issue to let you know that you can add Fedora to the list of supported systems.

@alexbbb Thanks for testing this and pointing out, I will add Fedora support in next build.

I tried forever-service on an old test-machine and got this message of "not yet supported", too.
So I looked after my /etc/os-release file and there was no such file.
Then I looked into /proc/version and it told me my distro and kernel version.
And best of all, on a fedora machine it says that it is a Red Hat one.
Maybe you should check /proc/version instead of /etc/os-release, but I haven't tested this on other machines.

@stillagainst which release of Fedora it is on your test server?

I don't have access to it right now, but it looked sth like this one:
Linux version <IP_ADDRESS>-115.fc12.i686<EMAIL_ADDRESS>(gcc version 4.4.3 20100127 (Red Hat 4.4.3-4) (GCC) )
Just found out that some CentOS behaves similar:
Linux version 2.6.32-71.el6.x86_64<EMAIL_ADDRESS>(gcc version 4.4.4 20100726 (Red Hat 4.4.4-13) (GCC) )

I have added code pointed by @alexbbb, could not test in earlier version of Fedora, closing this for now. If someone wants earlier version of Fedora to be supported, please open another issue with exact version and similar details.


changes to add starting points as features

Related to issue: # https://devtopia.esri.com/runtime/kotlin/issues/4502

Description:
Support the user workflow of adding the starting points as a list of features
Summary of changes:

Add fun addStartingPoints() that takes features to be added as starting points for a trace

Pre-merge Checklist

a vTest Job for this PR has been run

[ ] link: https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/151/

Unit and/or integration tests have been added to exercise this PR's logic, and the tests are passing:

[ ] Yes
[ ] No

To test it you can modify the micro app
MainScreen.kt

if (viewModel.traceState.initializationStatus.value == InitializationStatus.Initialized) {
        LaunchedEffect(viewModel.traceState) {
            coroutineScope.launch {
                viewModel.performQueryAndGetFeatures().forEach {
                    viewModel.traceState.addStartingPoint(it)
                }
            }
        }
    }

TraceViewModel.kt
suspend fun performQueryAndGetFeatures(): List<ArcGISFeature> {
        val featureService = (arcGISMap.operationalLayers[0] as GroupLayer).layers[3] as FeatureLayer
        val serviceFeatureTable = featureService.featureTable as ServiceFeatureTable
        serviceFeatureTable.load().getOrThrow()
        val queryParameters = QueryParameters().apply {
            geometry = serviceFeatureTable.extent
            whereClause = "1=1"
        }
        val features = serviceFeatureTable.queryFeatures(queryParameters).getOrThrow()
        val arcGISFeatures = features.map { it as ArcGISFeature }
        val arcGISFeaturesList = listOf(arcGISFeatures[0], arcGISFeatures[1])
        arcGISFeaturesList.forEach { it.load().getOrThrow() }
        return arcGISFeaturesList
    }


Update Go to v1.19

Fix several CVEs from Go 1.17
Update buildenv image to use golang:1.19.7-buster
Update golangci-lint to v1.51.2

This started as a cherry-pick of https://github.com/GoogleContainerTools/kpt-config-sync/pull/465, but some different choices were made previously when upgrading golangci-lint, so there weren't any more lint failures to fix. And we don't need to use the default exclusions, because we already have exclusions configured in .golangci.yaml.

/retest

/retest
One more time with feeling

/retest

/lgtm
/approve

Realized the cherrypick was actually downgrading the linter compared to head, so i fixed it.

/lgtm


elf_parser: Fix invalid-null-argument

When running
bpftrace -e 'kprobe:do_nanosleep { printf("PID %d sleeping...\n", pid);

with ASAN enabled, it was failing asan with UndefinedBehaviorSanitizer: invalid-null-argument:
src/ast/elf_parser.cpp:46:19: runtime error: null pointer passed as argument 1, which is declared to never be null

This change makes sure shdr->sh_size is non-zero before attempting memcpy

Is the command correct? It's missing a trailing }' so it's not even a valid bpftrace command.
FWIW, if I run
bpftrace -e 'kprobe:do_nanosleep { printf("PID %d sleeping...\n", pid); }'

with ASAN, I get no errors.

Is the command correct? It's missing a trailing }' so it's not even a valid bpftrace command.
FWIW, if I run
bpftrace -e 'kprobe:do_nanosleep { printf("PID %d sleeping...\n", pid); }'

with ASAN, I get no errors.

Yeah, I made a typo when I copied/pasted the command.
And ASAN was an gross approximation... it is a UBSAN error: https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html


Update careers

@phutchins  actually i just noticed some css issues; I'll fix it today and request PR sometime today.

@phutchins  please review; made some modifications to job descriptions/ fixed css issues.


How to debug this handler?

How can I run this handler's binary from the command line in order to debug if it actually can connect to the influxdb database? I assume that it expects something like:
echo {} | sensu-influxdb-handler -d monitor
I tried various formats of output, but without success.
Can anyone give me an example?

found a correct format


OpenIdConnectOptions Event Handler Question

Is there an example anywhere in this repository that shows how to wire up a custom OpenIdConnect authentication event handler.  In particular, I want to add a AuthorizationCodeReceived handler for the purpose of implementing the hybrid flow.
I see that there is an open issue (#456) regarding out-of-the-box hybrid flow support--looks like in RC2.  However, I was hoping to get something working sooner.
Thanks

Not currently, but it should look a lot like this other sample:
https://github.com/aspnet/Security/blob/dev/samples/SocialSample/Startup.cs#L102-L117

Thanks Chris.  I just wasn't getting the correct syntax.  The example you
referenced did the trick.
On Monday, January 4, 2016, Chris R<EMAIL_ADDRESS>wrote:

Not currently, but it should look a lot like this other sample:
https://github.com/aspnet/Security/blob/dev/samples/SocialSample/Startup.cs#L102-L117
—
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Security/issues/637#issuecomment-168851546.


Fail to compile code for Lidar point cloud ground segmentation

Thanks for sharing the code! When I run preprocess.py, I have the following error:
Traceback (most recent call last):
File "preprocess.py", line 11, in
from lib.grndseg import segmentation
ImportError: cannot import name 'segmentation' from 'lib.grndseg' (unknown location)
Does it mean I didn't compile the code in lib/grndseg successfully? Thanks!

Yes that'd be my guess.

Yes. I had the same issue. For my case, it was caused by the `GLIBCXX_3.4.32' not found in my anaconda environment.


[DO NOT MERGE] V11

Thank you for your contribution to Go-AutoRest! We will triage and review it as soon as we can.
As part of submitting, please make sure you can make the following assertions:

[ ] I've tested my changes, adding unit tests if applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, except in the case of urgent bug fixes warranting their own release.
[ ] If I'm targeting master, I've updated CHANGELOG.md to address the changes I'm making.

Have we updated any samples to use the v11 branch to test the changes you've made against the services? If so, could you link us those changes?

Also what is our timeline for expecting to merge this?

I have local changes, will push them later today.

Closing as stale


Update of rustc-ap and related crates

After a bit of digging for #1208 I think this can be fixed by updating rustc-ap-syntax and related crates. Would it be worth while for me to update the crate and attempt fix all problems, or is someone else working on this?

We will update rustc-ap-* crates before releasing 1.4.0.


Nuxt 3 module: Custom directive not working - using example from docs

Hello there,
thanks for creating such an amazing library! I love it ❤.
I'm not able to create custom directives in Nuxt using the example mentioned in the docs:
{
  // nuxt.config.js
  motion: {
    directives: {
      'pop-bottom': {
        initial: {
          scale: 0,
          opacity: 0,
          y: 100
        },
        visible: {
          scale: 1,
          opacity: 1,
          y: 0
        },
      }
    }
  }
}

When I try to use the custom directive like:
<p v-motion-bottom-pop>I am supposed to be animated by a custom directive using @vueuse/motion!</p>

I get the following error:
[Vue warn]: Failed to resolve directive: motion-bottom-pop
Using presets works without any issue.
I recreated the issue here: https://stackblitz.com/edit/nuxt-starter-7pqo1q?file=app.vue
Nuxt: 3.5.2
Vueuse/motion: 2.0

@PatrikBird The directive you defined in nuxt.config.js is pop-bottom, but you use <p v-motion-bottom-pop> instead.
Also I create a PR for exposing config to clients, it should fix the issue.

Oh okay sorry. I messed the repro... it's corrected now and I can't get it working either.. same error.
Am I doing something wrong?
I declared this:
motion: {
  directives: {
    'pop-bottom': {
...

I use it like this: <div v-motion-pop-bottom>
And get this: [Vue warn]: Failed to resolve directive: motion-pop-bottom (local env and stackblitz).
Again, the presets work.
Appreciate your quick response!

This also doesn't work for me. Same exact example code from above.
No animation is triggered whatsoever

It looks hardly maintained.


Supported Output Formats of .xls

how to get output file ext '.xls' ?

You can't - they're totally different things, and only the xml based .xlsx is supported.

@Rycochet  okey , thanks

If I want to add this feature in this library, what should I need?
Thanks.

They're completely incompatible and unrelated formats - so to start with you'd need to research the binary file format used, then start looking into the internal data layout used, then write a translation layer between the xlsx (zipped xml) based storage used here and the format in xls - then finally write a binary translation output that can be used to create a file either in the browser or via NodeJS...
In other words... A complete rewrite.

@Rycochet awsome

Is there any progress in this work？

@SnowyEvening nope :(

:(   An unfortunate news.


Add support for videos from Streamable

I recently found a video hosting site called Streamable and I think it looks pretty nice. Do you think adding support for it would be worth it?

Actually, I'd love to support as many playing techs as possible, but sadly, most of them don't want to be embeddable. There is an API for streamable (https://streamable.com/documentation), but there is no tech available for videojs.
I will start to create some techs one day, but I currently have too many projects to do that right now.. Maybe you can start one?

I can't program, so unfortunately I can't help you with that.

Closed for now, maybe we'll add this tech later.


Auto tune FAST DMD & RGB flow control buffers

We should look at the backlog of queued commands for the RGB and DMD processors and auto tune the hz for LEDs and DMD to support whatever the hardware can handle.
NET is fine as it is

Implement asyncio send loop

will not fix


environment configuration

dear author
thanks for your great work，I want to replicate your work，but Some files not found。for example ，segment_anything_hq

Sorry for my mistake. You need to run pip install segment-anything-hq to install it after downloading that github repo.

Thank you very much for your answer


entrypointbaseUrl does not work as intended on load method of web bootstrapping

Steps to reproduce
I could not reopen the issue i had,
Steps are mentioned in this issue - https://github.com/flutter/flutter/issues/151181
I was expecting the solution provided by @eyebrowsoffire to work.. But I am some how unable to load the main dart js file.
I am getting this error
Refused to execute script from 'https://my-app-domain/a/main.dart.js' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled
Expected results
The expectation is that the main.dart.js file reference should've been taken from a/{{buildVersion}} based on the entryPointBaseUrl provided
Actual results
It's trying to reference the file from the mainJSPathlocation that is injected by flutter_build_config.
Code sample
Code sample
window.addEventListener('load', function(ev) {
      {{flutter_build_config}}
      _flutter.loader.load({
        entryPointBaseUrl: "/a/{{buildVersion}}",
        serviceWorker: {
          serviceWorkerVersion: serviceWorkerVersion,
          serviceWorkerUrl: "{{buildVersion}}/flutter_service_worker.js?v=" + serviceWorkerVersion
        },
        onEntrypointLoaded: function(engineInitializer) {
          engineInitializer.initializeEngine({
            assetBase: "{{buildVersion}}/"
          }).then(function(appRunner) {
            appRunner.runApp();
          });
        }
      });
    });

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.2, on macOS 14.4.1 23E224 darwin-arm64, locale en-BR)
[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/to/macos-android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] IntelliJ IDEA Community Edition (version 2023.3.7)
[✓] VS Code (version 1.79.2)
[✓] Connected device (3 available)
! Error: Browsing on the local area network for Mickey’s iPhone. Ensure the device is unlocked and attached with a cable or associated with the same local area network as
this Mac.
The device must be opted into Developer Mode to connect wirelessly. (code -27)
! Error: Browsing on the local area network for iPhone. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources
! Doctor found issues in 1 category.

entryPointBaseUrl: "/a/{{buildVersion}}",

@Suman085 Here can you try by passing `main.dart.js to see if it helps ?
For the error you are getting, see if this issue is helpful along with underlying comments https://github.com/flutter/flutter/issues/152995
If not, I will reopen the linked issue for better tracking.

Thanks for the update. I will reopen the original issue for better tracking.
Closing this one.


Wrong year in changelog

https://github.com/JetBrains/js-graphql-intellij-plugin/blob/master/CHANGELOG.md
is: [3.4.0] - 2022-02-07
should be: [3.4.0] - 2023-02-07
Not sure if it's worth mentioning, but I thought I'd let you know :)

Just saw, there is already a pull request. So I close it.

Hi, thank you for the info! I just merged a PR. It shouldn't affect anything, just misleading a bit.


Change build scripts from Batch to Powershell.

Is your feature request related to a problem? Please describe.
Ducktape currently has both .sh and .bat scripts, so the scripts work on all platforms. Switching to using Powershell will simplify this, because with one script you can support all platforms, as Powershell is cross-platform.
Describe the solution you'd like
Remove batch and shell scripts and make them Powershell scripts.

👍


Fix: Use closeEvent() cf. hideEvent() to identify when window closed

Description
I think this bug must have been around for a while and it's rather embarrassing. It turns out we've been using the main window's hideEvent method to check for when the window is closed. While it is run when the window is closed, it also turns out it's invoked when the window is minimised (on Windows at least). This means that currently if you minimise the window, all of the connected devices will disconnect, as they are supposed to do when the window closes.
I'm not sure whether closeEvent wasn't working for me on Linux before (it seems fine now, although admittedly I am using a weird tiling window manager), or if I went with hideEvent for the symmetry with the showEvent handling we have next to it. In any case, I didn't think it would be invoked when the window was minimised.
Oops.
Type of change

[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation (non-breaking change that adds or improves the documentation)
[ ] Optimisation (non-breaking, back-end change that speeds up the code)

Key checklist

[x] Pre-commit hooks run successfully (pre-commit run -a)
[x] All tests pass (pytest)
[ ] The documentation builds without warnings (mkdocs build -s)
[x] Check the GUI still works (if relevant)
[x] Check the code works with actual hardware (if relevant)
[ ] Check the pyinstaller-built executable works (if relevant)

Further checks

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests have been added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

Indeed :facepalm:


feat: Added a naming route system

Also we can add @Routes.Back() => JSRuntime.InvokeVoidAsync('history.back');


In development, use git-describe for appVersion

Before:

After:

Normally appVersion is set from package.json's version field.
This looks goofy right now since package.json is set to 1.0.0 but the 1.0.0 git tag hasn't been created yet.
closes #718

How can I test it? Running with npm start, I get this:

And with npm run pack:

😕

@matheuss hmm, interesting. My screen shots are from Help > About which I now see in menu.js doesn't exist for OSX.
I did some more research. The OSX about menu is determined by Info.plist which is in build/ in our repo. We point Electron to this file in package.json:
  "build": {
    "appId": "co.zeit.hyperterm",
    "asar": false,
    "extend-info": "build/Info.plist",
    "linux": {
      "target": [
        "deb",
        "AppImage"
      ]
    },

However, when you do npm start it's using the default Info.plist that comes with the prebuilt Electron. When you do npm run pack it merges in our custom Info.plist with the default one.
So I need to figure out how to:

Use a custom Info.plist even in development
Inject git-describe into a custom Info.plist for development

I flipped the !== in menu.js, and it still gives me the default Electron one 😛
Since this is working for you/Linux, I'm merging it 😄

Thanks @flybayer 🙌