add support for icmp

add support for icmp

I signed it

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

Googlers: Go here for more info.

I signed it

Hi,
This is a synthetic repository where we don't accept pull requests. Please send this to the main repo https://github.com/google/gvisor.
This repo is synthesized from the main one for folks who want to use netstack independent of gVisor but we do not commit to this one directly.
-bhasker


Roll engine to 3021e8f6c3251712dbac7d15ad1538e287c10f49

Is there a way to test this?

In the engine code, yes. iOS doesn't provide an API for firing native key events, though we might be able to work something out :)


Fixed Night Mode in BlocksUI

Signed-off-by: Blacklion14<EMAIL_ADDRESS>Changes
I changed ThIs css file and added a z-index so that the blocks which were behind the container of bootstrap-dark have came to front
Also i changed this bootstrap-dark.css file a bit in order to make close Button in blocks details div to be visible clearly in dark mode. This change may not create any ambiguity because in general we will always willing to have a white contrast text colour inside a button in dark-mode . And this will change colour of only text inside button not the background colour.
If some one wants to change colour to some other white contrast they can directly change it from here.
Verification
I ran the UI and it is running clearly without any errors or bugs.

Nice work @Blacklion14 :muscle: Two things from me that will help reviewers with this PR:

Does this PR close an issue?
Could you provide some screen grabs of before and after your changes? That way we have a visual idea of what has changed.
Thanks!

Hello @ianbillett

I actually don't know if this closes a issue because i was running Blocks UI and i saw in night mode blocks were not visible.
So i changed the css a bit and asked @onprem on slack giving all the screenshots and changes and he guided me to create a PR.

Previously on using Night mode in Block UI we were unable to see blocks clearly (as in Screenshot)

But as i changed css a bit which made it look like this

Also i fixed the close button(in the section which appears by clicking on any block) which was not visible in darkmode.
These are the changes I made.

Thank you @ianbillett for reviewing 🙇🙇
I am also waiting for @onprem 's review..

@Blacklion14 please rebase on the latest main and don't forget to do make assets :muscle: let's get this into the next RC

Sure @GiedriusS

@Blacklion14 please rebase on the latest main and don't forget to do make assets  let's get this into the next RC

Hello @GiedriusS
I have tried to rebase my branch (fix-night-mode-bucketUI) with following commands

But i am facing merge conflicts issue in pkg/ui/bindata.go file.
Please guide me how can i resolve this issue and rebase my branch properly.

@Blacklion14 please rebase on the latest main and don't forget to do make assets  let's get this into the next RC

Hello @GiedriusS
I have tried to rebase my branch (fix-night-mode-bucketUI) with following commands

But i am facing merge conflicts issue in pkg/ui/bindata.go file.
Please guide me how can i resolve this issue and rebase my branch properly.

You could resolve this by accepting the incoming changes by opening that file with your favorite editor and then create another commit after running make assets

@Blacklion14 please rebase on the latest main and don't forget to do make assets  let's get this into the next RC

Hello @GiedriusS
I have tried to rebase my branch (fix-night-mode-bucketUI) with following commands

But i am facing merge conflicts issue in pkg/ui/bindata.go file.
Please guide me how can i resolve this issue and rebase my branch properly.

You could resolve this by accepting the incoming changes by opening that file with your favorite

@Blacklion14 please rebase on the latest main and don't forget to do make assets  let's get this into the next RC

Hello @GiedriusS
I have tried to rebase my branch (fix-night-mode-bucketUI) with following commands

But i am facing merge conflicts issue in pkg/ui/bindata.go file.
Please guide me how can i resolve this issue and rebase my branch properly.

You could resolve this by accepting the incoming changes by opening that file with your favorite editor and then create another commit after running make assets

@GiedriusS I have done the changes you said . Can you please review....

@GiedriusS I don't know why the above test is getting failed ....
Please review 🙇🙇


Erroneous example Ruby code snippet

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/webhooks/using-webhooks/securing-your-webhooks#ruby-example
What changes are you suggesting?
The first argument of the call to OpenSSL::HMAC.hexdigest should be just a string describing the requested algorithm. Diff is pretty self-explanatory. 😁
Additional information
No response

@ameuret Thanks for opening an issue and submitting a PR! ✨

Closing based on https://github.com/github/docs/pull/28259#issuecomment-1724112534


Tracing using Langfuse

Can we trace the page usage using Langfuse or any other framework ?
Purpose:
To track page usage and view in dashboard.

@hexapode , is my understanding correct that currently to track the cost/pages used in llama-parse there's no API in place to pull that data?

we have a usage API: https://docs.cloud.llamaindex.ai/API/usage-api-v-1-parsing-usage-get
also a parsing history API: https://docs.cloud.llamaindex.ai/API/get-parsing-history-result-api-v-1-parsing-history-get


ReferenceManyField pagination

ReferenceManyField supports Datagrid
but there is no pagination
it would be nice to have pagination prop like in List

Duplicate of #561


asp with html and javascript

Hi,
I am glad that the asp syntaxt is working, but when I set the syntax to asp, the html and javascript syntax is not working anymore.
Is it possible to let it work for html and javascript syntax with asp syntax at same time?

Hello. To be honest I do not know. Unfortunately I'v just wrapped existing ASP.tmLanguage file to work from VS Code. I do not have experience with tmLanguage format, and cannot fix it now. All pull requests or suggestions for new ASP syntax files are welcome.


Why is warning AZC0016 triggering?

@maririos commented on Tue Sep 28 2021
We had to suppress AZC0016 // All parts of ServiceVersion members' names must begin with a number or uppercase letter and cannot have consecutive underscores in the DocumentAnalysisClientOptions.ServiceVersion in public repo, while it didn't in the private repo.
Two follow-ups:

Are analyzers out of sync between repos?
What is this warning about?

Fixed by #29808.


blueprints's name collision

When trying to generate docs, I encounter this error:
AssertionError: A blueprint's name collision occurred between <flask.blueprints.Blueprint object at 0x7f89153c7e90> and <flask.blueprints.Blueprint object at 0x7f8915383710>.  Both share the same name "bootstrap".  Blueprints that are created on the fly need unique names.
I just used the method given in the doc.

Hey guys, I also encounter this problem. How did you handle it?

I probably never did. Long time ago


Funders Page

Initial requirement:

Create placeholder

To be discussed:

Do we actually have one since we might have the funders everywhere on our webpage?

References:

N/A

Funders page was combine with the Projects page. Closing this issue


Selecting a gltf animation

Hi there :-)
I implemented an animated .glftf file with the animation-mixer component:

add the script

<!-- Load animation-mixer component -->
<script<EMAIL_ADDRESS>

add animation-mixer property

<a-gltf-model
          rotation="90 0 0"
          position="0 0 0"
          scale="0.2 0.2 0.2"
          src="#avatarModel"
          animation-mixer>
</a-gltf-model>

Now here is my problem: how can I select a specific animation? When I open the file in gltf view it shows that there are multiple animations (1-4). Is there a way to select one of these for the animation mixer property?
Thanks,
Nahuel

Hi, you can use the clip argument of the animation-mixer component like it says in the docs, as an example:
<a-gltf-model src="#my_model" animation-mixer="clip: name_of_animation"></a-gltf-model>


Add plan scripts

Signed-off-by: matthew-frahry<EMAIL_ADDRESS>

@wenlock, tests are passing. This is the last one for today! I'll try and finish the last two tomorrow!

Lgtm


mapStateToProps ownProps undefined

I'm using the new "all in one"-connect (Great work by the way!)
I don't get ownProps passed to mapStateToProps. It's undefined.
Debugger printout:
mapStateToProps(globalState, ownProps) globalState = Object {auth: Object, products: Object, productTypes: Object, fields: Object, counter: 0…}, ownProps = undefined

Oops. I've never used that parameter so I missed it. Fixed in v3.0.3.
Thanks for pointing this out.

Thank you! That was fast.

No no no, don't do this, this is very inefficient: https://github.com/erikras/redux-form/commit/a62f9fdb5f566b943a62a574fb09a1a05084f8c0.
React Redux will now rebind the action creators on every prop change.
Previously it only did that once because you passed a function with single argument.
React Redux looks at the functions you pass to connect. If they accept two arguments (and have ownProps), it enters “slow mode”. If they accept just one argument, it works in “fast mode”.
To stay fast, your library needs to do the same thing. If mapDispatchToProps and mapStateToProps accept a single argument, pass a single-argument function to connect(). Otherwise, and only then, pass a function with two arguments.
I know it's a bit implicit, but then it's a bit unusual to have libraries wrapping React Redux and mimicking its API, so that's why it isn't thought out.

Ok, if we don't get ownProps how do we get to our url parameters in mapStateToProps?

I'm not saying Redux Form shouldn't pass ownProps. I'm saying it should pass ownProps only if the mapStateToProps/mapDispatchToProps specified by the user have .length > 1. This an implementation detail to speed it up: I'm not proposing an API change.

Is there an example how to use connect and reduxForm? Should I be using wrapMapStateToProps instead? I'm trying to access values outside of the form, like auth state and pass auth value to the formSubmitHandler. Thank you!!

I'm getting the same issue now in redux-form v6.6.3 - ownProps is undefined.


Fix NaNs (Closes #86)

Implement dealing with np.nan, closes #86
Every function in the library now handles NaNs correctly.
Implemented through decorator @handle_nans in new file _helper.py.
Tests added in test_nan.py
As we went through the whole library anyways, argument "input" was renamed to "s" in some functions to be in line with the others.

Wow! That's cool stuff! 👍
The part I'm confused with is: why do we need to decorate any function?
Given the example in handle_nans :
s = pd.Series(["a", np.nan])
s.str.replace("a","b")

gives the exact same result as
s = pd.Series(["a", np.nan])
@handle_nans
def replace_a_with_b(s):
   return s.str.replace("a", "b")
replace_a_with_b(s)

But the first solution is faster (as there is no copy), cleaner and shorter.
In which functions do we really need to use such a decorator? I would say we should use it only in extreme cases (as it makes a copy ...) or even better adjust such functions so that the decorator is not even necessary.
You might find this interesting. By doing some tests, I found that np.nan and pd.NA are not exactly the same thing. If you want to know more: API: distinguish NA vs NaN in floating dtypes

Thanks, we really did not know that pd.str. ... already takes care of NaNs :facepalm: -> we now removed it from most of the functions in preprocessing :cold_sweat:  . It's now only there if we really need it.
Concerning the performance: we'll have to do a copy at some point anyway if we want to pass the non-nan values to a function (like apply, or a Vectorizer from sklearn, and not s.str.... which handles all nans for us) as this is the behaviour:
>>> import pandas as pd
>>> import numpy as np
>>> s = pd.Series(["test", np.nan])
>>> def f(x):
...        # If we want to keep the NaNs, we'll have to do this.
...        x[~x.isna()] = x[~x.isna()].str.replace("t", "i")
...        return x
>>> f(s)
0    iesi
1     NaN
dtype: object
>>> # Function call without copy changes the argument!
>>> s
0    iesi
1     NaN
dtype: object

We looked at many methods to do this with one less copy but could not find any (without manipulating the input). Additionally, with e.g. PhrasesTransformer (and PCA etc.) we get an array of new values anyway that uses up the same amount of memory.
So we will need a second Series with only the non-NaNs anyway for some functions. For example in tokenize_with_phrases  when we use the PhaseTransformer, we need to pass it the input Series without NaNs, but in the result we want to re-insert the NaNs, and as shown above we can't use s[~s.isna()] = ... as it would change the input itself.
So we believe our decorator is as performant as we can be if we want to keep the NaNs. It also frees developers later on from thinking about NaNs.

By doing some tests, I found that np.nan and pd.NA are not exactly the same thing

It also catches this so we're good in this regard :sweat_smile:

After this, we'll then continue working on updating the documentation and should then have finished part 1 of #85  (and the " All function to deal with np.nan #86") :+1:

Thanks! I will look into this in more detail on Monday (leaving now). Sounds cool that you are about to finish with part 1!  🎉

Very short feedback:
For functions that call apply:
s = s[~s.isna()].apply(lambda x: ...)

should do the job (or very similar code, without a new copy, and without the need of the decorator), I'm missing something?
regards,

I'll try to formulate the problem a bit clearer (we probably spent a little too much time on this the last days, at one point we dug into the Pandas source code to figure out how s.str... handles the NaNs :woozy_face: )
We have NaNs in the input, e.g. s = pd.Series(["test", np.nan, pd.NA, "test2"]) . We now want to apply some function to the input and keep the original NaNs where they were. There are basically 3 cases:

The function is implemented with functions that already don't touch the NaNs, e.g. s.str.replace(...). We don't have to do anything in this case :heavy_check_mark:
The function at some point loops through the Series (e.g. stem loops through with apply). We can then just handle the NaNs manually in the loop with e.g. if text is np.nan or text is pd.NA (we implemented this e.g. in stem), along the lines of

s = s.apply(lambda x: x.upper() if x is not pd.NA and x is not np.nan else x)

We cannot do s = s[~s.isna()].apply(lambda x: ...) as we would lose the original NaNs (cells that were NaN in s originally would now be lost completely).
We cannot do s[~s.isna()] = s[~s.isna()].apply(lambda x: ...) (which seems great as it only changes the not-NaNs and keeps the original values) as doing this inside a function changes the original Series.

The function at some point needs all the (non-nan) values to do something (e.g. use TfidfVectorizer). We have the same problems described in 2., but we can't just check in a loop if something is a NaN and handle it there. Of course we'd like to do

tf = CountVectorizer(
    max_features=max_features, tokenizer=lambda x: x, preprocessor=lambda x: x
)

# Option 1 -> changes original Series
s[~s.isna()] = pd.Series(tf.fit_transform(s[~s.isna()]).toarray().tolist(), index=s.index)

# Option 2 -> loses the original NaNs from the input
s = pd.Series(tf.fit_transform(s[~s.isna()]).toarray().tolist(), index=s.index)

# Option 3: the best we found, it's basically what the decorator does (if one would write it into the function directly)
# As you can see, it's just Option 1, but not changing the input anymore
s_result = s.copy()
s_result[~s_result.isna()] = pd.Series(tf.fit_transform(s_result[~s_result.isna()]).toarray().tolist(), index=s_result.index)

Hey @henrifroese!
Thank you for your detailed answer. You are doing such a great job, really! I know you are right, I'm just asking since I would like to really understand the code and the reasons behind some choices. In about 1 month you will be probably less available, and I want to make sure I can have a full understanding of the codebase as well as argue all choices we made! ;)
Now the reason why s[~s.isna()] = s[~s.isna()].apply(lambda x: ...) (1) does not work is much clear. Thank you!
Some minor question/feedback:

Why tfidf is commented on the tests?
Why count_sentence returns now an object Series instead of an int Series? As output values are either nan or int, dtype=int64 is more natural
It is correct that remove_diacritics has a the .astype("unicode")? This question does not have much to do with this PR. Depending on your answer, I might need to open another issue to solve/check that.
Renaming all input to s is clearly a good idea. Next time, this kind of change should be preferably done in another PR. The simple reason is that the moment we will merge this PR, the other pull requests will have probably some conflicts.
handle_nan

is written in the docstring "and in pca there is no return", is that right?
As discussed, Examples should probably show an example of a function that makes use of apply
As I had the question of why simple not use (1), other contributors will have the same. I think we should add more information to the docstring to make sure it's really clear the purpose of this wrapper

@wrapt.decorator do we really need to use this external library? Can we not simply use the wrapper from the stlib functools? (Adding dependencies is great when is really necessary.)

Thank you again for such a great job!! 🎉 🎉

Why tfidf is commented on the tests?

I have no idea! :facepalm:  I uncommented it.

Why count_sentence returns now an object Series instead of an int Series? As output values are either nan or int, dtype=int64 is more natural

Yes, that's a little annoying, we can solve this by doing s_return = s_return.astype(output[0].dtype) to get the results type correct. We'll think about this some more.

It is correct that remove_diacritics has a the .astype("unicode")? This question does not have much to do with this PR. Depending on your answer, I might need to open another issue to solve/check that.

We did not change this I believe (or maybe I'm missing something?), so if there's something wrong maybe a new Issue would be good.

handle_nan is written in the docstring "and in pca there is no return", is that right?

That's weird, we'll think about what we meant there again, sorry about that!

As discussed, Examples should probably show an example of a function that makes use of apply

I'm not sure what you mean by this? Sorry

As I had the question of why simple not use (1), other contributors will have the same. I think we should add more information to the docstring to make sure it's really clear the purpose of this wrapper

Yes, will do!

@wrapt.decorator do we really need to use this external library? Can we not simply use the wrapper from the stlib functools? (Adding dependencies is great when is really necessary.)

Will check if that's possible!

As discussed, closing this as we'll do this differently in separate PRs.

Yes, see #123


Can't click Save

I can't click save when trying to save the blueprint. Whenever I press it, it does nothing. I've selected images and changed the name, I've tried not doing both of these. I'm using the mod Via the Foundry Mod Manager (0.5.7)

Edit: Image added for context of which particular save button

Thanks for reporting it.  The latest hotfix changed a function name.  Fixed in v0.4.23.

Thank you!


Documented all non-privileged intents needed by events

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[x] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
I have added documentation for all event implementations and package-info files regarding their
requirements for non-privileged intents. I worked off of logic to write these, so please do point out
any mistakes I made documenting them

Any update on this?

I think he waits for the UserFlags PR to be merged to then add missing info, unless I should add this.

Bump

Due to inactivity, this was closed. Feel free to open a new pull request if you have time to respond.

Ahem. I have been reminded that this exists

Re-Opened on Nue's request. waiting for updated code tho

Apologies for letting this die

Is this ready, or what? Been 2 weeks

I don't get why it still says "Changes requested" when I resolved everything

Some of the events say "Additionally, this event also requires blah blah". I'm removing "also", it's run-on

Good catch, sorry about that
On Tue, Nov 3, 2020 at 4:48 AM Florian Spieß<EMAIL_ADDRESS>wrote:

@MinnDevelopment commented on this pull request.
In
src/main/java/net/dv8tion/jda/api/events/guild/invite/GenericGuildInviteEvent.java
https://github.com/DV8FromTheWorld/JDA/pull/1277#discussion_r516575991:

@@ -27,6 +27,11 @@

Every GuildInviteEvent is derived from this event and can be casted.

Can be used to detect any GuildInviteEvent.

Requirements

These events require the {@link net.dv8tion.jda.api.requests.GatewayIntent#GUILD_INVITES GUILD_INVITES} intent to be enabled.

This event will only fire for invite events that occur in channels where you can {@link net.dv8tion.jda.api.Permission#MANAGE_CHANNEL MANAGE_CHANNEL}.

First you say these events, then you say this event. Decide on one please.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DV8FromTheWorld/JDA/pull/1277#pullrequestreview-522376933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHQUR4WXO2TXX5JZZZWGNK3SN7NYNANCNFSM4MQQ5RKA
.


Telotaod

In Tres Rdulaol's song "Telingolem," the chorus goes: "Alii kau a chad el diak omnguis / el telaod a churam / Telotaod a rengum / diak modengei el kmo ke mong ke mei."  Telotaod appears to be a v.redup from telaod (v.r.s.), with the phrase "telotaod a rengum" meaning to be indecisive or dishonest.

yes...indecisive; forked like the snake's tongue :-)
On Wed, Jul 24, 2024 at 10:42 AM JimGeselbracht @.***>
wrote:

In Tres Rdulaol's song "Telingolem," the chorus goes: "Alii kau a chad el
diak omnguis / el telaod a churam / Telotaod a rengum / diak modengei el
kmo ke mong ke mei." Telotaod appears to be a v.redup from telaod (v.r.s.),
with the phrase "telotaod a rengum" meaning to be indecisive or dishonest.
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/508, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2DJJEEBDL62YHJPQXEN6TZN7RRFAVCNFSM6AAAAABLM6VWXGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQZDQMJUGIYDMMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Compatibility with firebase 8.x ?

Firebase JS SDK just release their version 8.x
When can we see an update to firebase-ngrx-user-management for compatibility?
In my project when updating to firebase 8 I am getting following error:
ERROR in<EMAIL_ADDRESS>796:18-22
"export 'auth' was not found in 'firebase/app'

Thank you

Same problem here, any solution about this topic?

This project seems dead. I highly recommend another state management solution. I recently discovered Akita and loving it, much less boiler plate code than NGRX. They also have integration with firebase.

This project seems dead. I highly recommend another state management solution. I recently discovered Akita and loving it, much less boiler plate code than NGRX. They also have integration with firebase.


Fix(eos_designs): Allow to return a single host subnet when IPv4/32 or IPv6/128

CAUTION: The code makes offset and subnet offset not being used. Probably should document this somewhere?
Related Issue(s)
Fixes #2705
Component(s) name
arista.avd.eos_designs
Proposed changes
When the pool subnet size is a /32 and the required subnet is a /32 for IPv4, the current code is failing as it skips the netwok_address in the pool which is the only available. (same for /128 for IPv6). This code allows to not skip the network address in this scenario and returns the only available /32 or /128.
How to test
molecule
Checklist
Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

Both subnet_offset and ip_offset are 0-based. So you are asking for the second /32 in the pool. When I call with 0,0 it works. So if we need to change something, I think we should modify the calling function somehow.

so in this case it means modifying all the calling functions, in which we ignore the offset then

Closed in favor of https://github.com/aristanetworks/ansible-avd/issues/2719


Allow the same page to belong to multiple excerpts

Use digwork index id for page base id instead of using digwork source id (which causes collisions when the same page belongs to more than one excerpt)

Codecov Report

Merging #425 (ae0a8a6) into develop (a67aff1) will increase coverage by 0.98%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop      #425      +/-   ##
============================================
+ Coverage    99.01%   100.00%   +0.98%
============================================
  Files          114         5     -109
  Lines         5778        77    -5701
  Branches         6         6
============================================
- Hits          5721        77    -5644
+ Misses          57         0      -57


Create Release workflow fails with "not currently on a branch."

 You can look around, make experimental
2023-04-20T14:01:24.6673651Z changes and commit them, and you can discard any commits you make in this
2023-04-20T14:01:24.6674194Z state without impacting any branches by switching back to a branch.
2023-04-20T14:01:24.6674534Z
2023-04-20T14:01:24.6674877Z If you want to create a new branch to retain commits you create, you may
2023-04-20T14:01:24.6675755Z do so (now or later) by using -c with the switch command. Example:
2023-04-20T14:01:24.6676120Z
2023-04-20T14:01:24.6676490Z   git switch -c
2023-04-20T14:01:24.6676772Z
2023-04-20T14:01:24.6677049Z Or undo this operation with:
2023-04-20T14:01:24.6677341Z
2023-04-20T14:01:24.6677552Z   git switch -
2023-04-20T14:01:24.6679605Z
2023-04-20T14:01:24.6679943Z Turn off this advice by setting config variable advice.detachedHead to false
2023-04-20T14:01:24.6680303Z
2023-04-20T14:01:24.6680776Z HEAD is now at 2b8984c Merge pull request #23 from CAI-Software-Inc/MH0329
2023-04-20T14:01:24.6726803Z ##[endgroup]
2023-04-20T14:01:24.7265189Z [command]"C:\Program Files\Git\bin\git.exe" log -1 --format='%H'
2023-04-20T14:01:24.7701847Z '2b8984ca755c4b2547587da08fe349d9e9639fdc'
2023-04-20T14:01:24.8446186Z ##[group]Run $ErrorActionPreference = "STOP"
2023-04-20T14:01:24.8446730Z $ErrorActionPreference = "STOP"
2023-04-20T14:01:24.8447046Z Set-StrictMode -version 2.0
2023-04-20T14:01:24.8447316Z git checkout -b
2023-04-20T14:01:24.8447569Z git config user.name buzzwick
2023-04-20T14:01:24.8447914Z git config user.email<EMAIL_ADDRESS>2023-04-20T14:01:24.8448282Z git commit --allow-empty -m "Release branch "
2023-04-20T14:01:24.8448555Z git push origin
2023-04-20T14:01:24.8506752Z shell: C:\Windows\System32\WindowsPowerShell\v1.0\powershell.EXE -command ". '{0}'"
2023-04-20T14:01:24.8507112Z env:
2023-04-20T14:01:24.8507592Z   ALGoOrgSettings:     {
"useCompilerFolder": true,
"doNotPublishApps": true,
"artifact": "https://bcartifacts.azureedge.net/sandbox/22.0.54157.55210/us"
}
2023-04-20T14:01:24.8508058Z   ALGoRepoSettings:
2023-04-20T14:01:24.8508385Z ##[endgroup]
2023-04-20T14:01:25.1648327Z error: switch `b' requires a value
2023-04-20T14:01:25.5887503Z [detached HEAD d9833d8] Release branch
2023-04-20T14:01:25.6449036Z fatal: You are not currently on a branch.
2023-04-20T14:01:25.6450577Z To push the history leading to the current (detached HEAD)
2023-04-20T14:01:25.6451116Z state now, use
2023-04-20T14:01:25.6451416Z
2023-04-20T14:01:25.6453795Z     git push origin HEAD:
2023-04-20T14:01:25.6454258Z
2023-04-20T14:01:25.8046329Z ##[error]Process completed with exit code 1.
2023-04-20T14:01:25.8305357Z Post job cleanup.
2023-04-20T14:01:26.1769047Z [command]"C:\Program Files\Git\bin\git.exe" version
2023-04-20T14:01:26.2201540Z git version 2.40.0.windows.1
2023-04-20T14:01:26.2318862Z Temporarily overriding HOME='D:\a_temp\50b6c1dc-b67f-447a-b779-1b8ece43fde6' before making global git config changes
2023-04-20T14:01:26.2319735Z Adding repository directory to the temporary git global config as a safe directory
2023-04-20T14:01:26.2334533Z [command]"C:\Program Files\Git\bin\git.exe" config --global --add safe.directory D:\a\CAI-Software-Precious-Metals-Solution\CAI-Software-Precious-Metals-Solution
2023-04-20T14:01:26.2845089Z [command]"C:\Program Files\Git\bin\git.exe" config --local --name-only --get-regexp core.sshCommand
2023-04-20T14:01:26.3458364Z [command]"C:\Program Files\Git\bin\git.exe" submodule foreach --recursive "sh -c "git config --local --name-only --get-regexp 'core.sshCommand' && git config --local --unset-all 'core.sshCommand' || :""
2023-04-20T14:01:27.1098017Z [command]"C:\Program Files\Git\bin\git.exe" config --local --name-only --get-regexp http.https://github.com/.extraheader
2023-04-20T14:01:27.1491771Z http.https://github.com/.extraheader
2023-04-20T14:01:27.1544595Z [command]"C:\Program Files\Git\bin\git.exe" config --local --unset-all http.https://github.com/.extraheader
2023-04-20T14:01:27.2032478Z [command]"C:\Program Files\Git\bin\git.exe" submodule foreach --recursive "sh -c "git config --local --name-only --get-regexp 'http.https://github.com/.extraheader' && git config --local --unset-all 'http.https://github.com/.extraheader' || :""
2023-04-20T14:01:27.9416248Z Cleaning up orphan processes

Got it. Only three. That worked. Thanks.


Track hits to Dr. Elephant which are coming from azkaban.

This change enables us to track the links originating from azkaban in piwik dashboard.
Notes:
Azkaban is https and the outgoing link to Dr. Elephant is http and hence by policy "referrer" header won't be set by the browser. An alternate way will be to set the meta tag in the relevant page. Going by the above approach since it doesn't require any code change.

Great! LGTM

Since this pull request is outdated, may we close this pull request?

Closing...


New form-api-011.html for Kong v0.11 release

KONG API will release next version 0.11 soon. Actually, it has the 0.11RC already, so please add new form to add new api for this version.
Thanks.

@here Any news on upgrading. Im using it with yesterday's 0.11.0 stable release and most things works except editing an api entry. plugins editing and other things work.

@edwardjrp , @thucnc ,
I'm going to work on the 0.11.* integration tommorrow probably.
Cheers

Check out v0.8.3 release.


Could you clarify the build configuration?

Hello,
I'm looking to build this against the Oculus unreal fork and wanted to double check the following steps:

Set HandGameplay as the start-up project and Development Editor as the configuration.
Hit F5 to build and debug the project

Are you saying here that I should build Development Editor|x64?  Will that actually work? Across Oculus Link?   I think that would be awesome but as far as I know the Hand APIs won't work using the link.  You do go on in the next step to use F5 to debug so...
Or should this be Development | Android?
Or does Development Editor | Android?

Thanks,
Sean

Closing issue.  Let me know if you have any more questions.


How can use ChatGPT 4o

I have an ChatGPT account that I have used to connect to OpenAI but the answer it gives me seems from an older version.
Thx for your tool

Hi @cognominal
Can you go to https://platform.openai.com/docs/api-reference/introduction and log in? You should be able to generate you own key


Image to pdf

Worked on Issue Number: #542
Issue Name: Image to PDF
I am a GSSoC Contributor.

@heet-kumar @Shahzaibfardeen @manisha-27 @abhijeet007rocks8 please check and review


data: project addition by Himanshu kumar

Add a new project to the list
I want to add a project created by me
which is a landing page of  a insurance company
Teckstack used - HTML, Tailwind css, React
Code of Conduct

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

Based on the guidelines of GSSoC'23, it appears that this particular issue has not been resolved within the past few days. As it involves a simple pull request, we would like to kindly inform you that we will be closing this issue.
If you still wish to work on this project, we encourage you to create a new issue and submit a pull request accordingly. Please ensure to review the contributing guidelines
Thank you for your understanding.


Slider Widget in Bottom sheet isn't responsive

Can't drag the thumb of the slider when inside in the bottom sheet. Value changes only if bottom sheet is closed and opened again.

Put Slider Widget inside Stateful Widget


Fix DC spike in refactored signal meter

While still a (major) work in progress, the updated signal meter is unable to compensate for DC offset spikes anymore.  Have a look at this: https://www.rtl-sdr.com/wp-content/uploads/2017/02/Getting-rid-of-that-center-frequency-spike-in-gnuradio_07.pdf?ffccfa&ffccfa, and see if it's possible to filter that out from the FFT plot without changing the device's center frequency.
Neither the HD Radio or DAB demodulators allow for offset tuning, which is what caused this. An alternative approach would be to add that support for both HD Radio and DAB.

Fixed via https://github.com/djp952/pvr.rtlradio/commit/7aa08ebee0781f949df08e6dbffa7e986b265237 for analog signals (FM / Weather). The concern also affects HD Radio, but the signal meter will fail to "lock" onto an HD Radio signal that has a DC spike, so I'm ignoring that for now since it would require changes to NRSC5.


[WIP] Support running on mac

Run the function:

LGTM
We need a Mac (with the Intel chip) volunteer to verify this change, thanks!

Merge it for now , I'll test later


Company activities do not support the new activity labels

Is it the same for educational events?

Labels were implemented on the Activity object, not on the base Event object, so yes

So we can just default these categories to Career and Education labels respectively right?

That would seem logical yes


Add short flag

@bradfitz @rsc
Please ignore the bonus commit there from the other PR. Thanks, GitHub.

Sure. Push directly please.


Failed to activate the atom-coderoad package

[Enter steps to reproduce below:]

... Install latest (0.12.1
... Try to activate it

Atom Version: 1.9.9
System: Microsoft Windows 10 Pro
Thrown From: atom-coderoad package, v0.12.1
Stack Trace
Failed to activate the atom-coderoad package
At Path must be a string. Received null

TypeError: Path must be a string. Received null
    at assertPath (path.js:7:11)
    at Object.join (path.js:506:7)
    at packageJson (C:\Users\Zappa\.atom\packages\atom-coderoad\lib\modules\setup\package-json\index.js:11:44)
    at combination (C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\redux\lib\combineReducers.js:117:29)
    at dispatch (C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\redux\lib\createStore.js:179:22)
    at C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\core-coderoad\node_modules\redux-thunk\lib\index.js:14:16
    at dispatch (C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\redux\lib\applyMiddleware.js:45:18)
    at C:\Users\Zappa\.atom\packages\atom-coderoad\lib\modules\setup\actions.js:14:9
    at C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\core-coderoad\node_modules\redux-thunk\lib\index.js:11:18
    at dispatch (C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\redux\lib\applyMiddleware.js:45:18)
    at C:\Users\Zappa\.atom\packages\atom-coderoad\lib\modules\setup\actions.js:5:9
    at Object.dispatch (C:\Users\Zappa\.atom\packages\atom-coderoad\node_modules\core-coderoad\node_modules\redux-thunk\lib\index.js:11:18)
    at new Main (C:\Users\Zappa\.atom\packages\atom-coderoad\lib\index.js:15:25)
    at Object.<anonymous> (C:\Users\Zappa\.atom\packages\atom-coderoad\lib\index.js:41:18)
    at Module._compile (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\native-compile-cache.js:103:30)
    at Object.defineProperty.value [as .js] (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\compile-cache.js:208:21)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Module.require (module.js:367:17)
    at require (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\native-compile-cache.js:50:27)
    at Package.module.exports.Package.requireMainModule (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\package.js:715:27)
    at Package.module.exports.Package.activateNow (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\package.js:173:16)
    at C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\package.js:795:25
    at Function.module.exports.Emitter.simpleDispatch (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\node_modules\event-kit\lib\emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\node_modules\event-kit\lib\emitter.js:125:28)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\command-registry.js:241:20)
    at C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\node_modules\atom-keymap\lib\keymap-manager.js:580:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\node_modules\atom-keymap\lib\keymap-manager.js:388:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (C:\Users\Zappa\AppData\Local\atom\app-1.9.9\resources\app.asar\src\window-event-handler.js:98:36)

Commands
     -0:02.6.0 cr-viewer:toggle (atom-text-editor.editor.is-focused)
     -0:00.1.0 blur (atom-text-editor.editor.is-focused)

Config
{
  "core": {}
}

Installed Packages
# User
atom-beautify, v0.29.10 (inactive)
atom-coderoad, v0.12.1 (inactive)
browser-plus, v0.0.76 (active)
elm-format, v2.0.1 (active)
language-elm, v1.5.0 (active)
language-elmx, v1.2.7 (active)
language-racket, v0.4.0 (active)
linter, v1.11.16 (active)
linter-elm-make, v0.17.0 (active)
script, v3.9.0 (inactive)
Sublime-Style-Column-Selection, v1.7.2 (active)
atom-dark-syntax, v0.27.0 (inactive)
atom-dark-ui, v0.51.0 (inactive)
atom-light-syntax, v0.28.0 (inactive)
atom-light-ui, v0.43.0 (inactive)
base16-tomorrow-dark-theme, v1.1.0 (inactive)
base16-tomorrow-light-theme, v1.1.1 (inactive)
one-dark-ui, v1.3.2 (active)
one-light-ui, v1.3.2 (inactive)
one-dark-syntax, v1.2.0 (active)
one-light-syntax, v1.2.0 (inactive)
solarized-dark-syntax, v1.0.2 (inactive)
solarized-light-syntax, v1.0.2 (inactive)
about, v1.5.2 (active)
archive-view, v0.61.1 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.11.1 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.31.0 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.27.0 (inactive)
autosave, v0.23.1 (active)
background-tips, v0.26.0 (active)
bookmarks, v0.41.0 (active)
bracket-matcher, v0.82.1 (active)
command-palette, v0.38.0 (inactive)
deprecation-cop, v0.54.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.39.0 (active)
fuzzy-finder, v1.3.0 (active)
git-diff, v1.1.0 (active)
find-and-replace, v0.198.0 (inactive)
go-to-line, v0.31.0 (inactive)
grammar-selector, v0.48.1 (active)
image-view, v0.58.0 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.0 (active)
link, v0.31.1 (inactive)
markdown-preview, v0.158.1 (active)
metrics, v0.53.1 (active)
notifications, v0.64.1 (active)
open-on-github, v1.2.0 (inactive)
package-generator, v1.0.0 (inactive)
settings-view, v0.238.2 (active)
snippets, v1.0.2 (active)
spell-check, v0.67.1 (active)
status-bar, v1.4.1 (active)
styleguide, v0.47.1 (active)
symbols-view, v0.113.0 (inactive)
tabs, v0.98.1 (active)
timecop, v0.33.1 (active)
tree-view, v0.208.0 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.34.0 (active)
whitespace, v0.32.2 (active)
wrap-guide, v0.38.1 (active)
language-c, v0.52.1 (active)
language-clojure, v0.21.0 (active)
language-coffee-script, v0.47.0 (active)
language-csharp, v0.12.1 (active)
language-css, v0.36.2 (active)
language-gfm, v0.86.0 (active)
language-git, v0.13.0 (active)
language-go, v0.42.0 (active)
language-html, v0.44.1 (active)
language-hyperlink, v0.16.0 (active)
language-java, v0.22.0 (active)
language-javascript, v0.119.0 (active)
language-json, v0.18.0 (active)
language-less, v0.29.3 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.35.0 (active)
language-php, v0.37.0 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.0 (active)
language-ruby, v0.68.5 (active)
language-ruby-on-rails, v0.25.0 (active)
language-sass, v0.52.0 (active)
language-shellscript, v0.22.3 (active)
language-source, v0.9.0 (active)
language-sql, v0.21.1 (active)
language-text, v0.7.1 (active)
language-todo, v0.28.0 (active)
language-toml, v0.18.0 (active)
language-xml, v0.34.8 (active)
language-yaml, v0.26.0 (active)

# Dev
No dev packages

If you type atom.project.rootDirectories[0].path into your developer console inside of Atom, is the result the same as the path to your project folder?

I was trying to toggle before i had created the project folder.
I get i load now and am able to start.
When I click on  FUNCTIONAL SCHOOL 1.1.0
i get an errpr:
Uncaught Error: Cannot find module 'process-console-log'
C:/Users/Zappa/.atom/packages/atom-coderoad/node_modules/react/lib/ReactErrorUtils.js:52

Check for tutorials does nothing.

run npm install process-console-log and whichever modules come up. I'll try to put out a fix for that soon.

Sorry for the late update. The latest version of Atom-CodeRoad should resolve this issue.


Flicking through GUI Scale seems to trigger an update to the Smooth Tree Lighting setting

This is inconsistent, it seems. It seems when flicking through GUI Scale the Smooth Tree Lighting toggles sometimes changes. I've noticed this seems to mostly occur around Large or Auto.

In the process of rebuilding and cleaning up the Video Settings for v3.5, splitting the options based on whether they configure the screen or world rendering into Visual Settings and Screen Settings, the bug was patched


Add line break support for paragraphs

Is your feature request related to a problem? Please describe.
I think label-studio is a wonderful work to simplify the labeling process. But I'm a bit confused when I use paragraphs tag for dialogue data. For each turn, there're line breaks and indents such as codes and list of items in the original data. However, they disappear in the front-end which make the content hard to read.
Describe the solution you'd like
I believe it's better to preserve what original data looks like. What do you think?

any updates on this issue?

any updates on this issue?

I found the solution is using the style with the 'white-space' to change it.
for example:

Closed as a non-active issue, and it seems like it has been resolved.


Whisper ONNX audio-classification

Partially fixes https://github.com/huggingface/optimum/issues/1672

Merging as tests pass.


Design QA: Trading Page - Order Ticket

Designs:
https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=102%3A3991
Header (Buy Shares, Sell Shares)

[x] Check type sizes
[x] Make hover states green for buy, orange for sell

Outcome Select

[x] Update states to match design
https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=1957%3A727

[x] update dropdown menu style if not already done in the app

Inputs

[x] make 32px height - same as the outcome selector
[x] Update label styles
[x] Update text styles of labels within inputs
[x] Update input text style
[x] Update states. Design for orderform input states: https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign?node-id=8219%3A266044

percentage and clear button

[x] give hover state design: https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=1774%3A9278

Max cost of...

[x] update text styling

Advanced button

[x] make arrow animation instant
[x] Give hover state: https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=1774%3A3055

Advanced order dropdown

[x] Fix the width to the largest text size (good till cancelled). It shouldn't decrease in width for the other items
[x] give correct hover and active style: https://www.figma.com/file/fLWVwmanAwetVZbujQquEi/Market-Page?node-id=1950%3A38
[x] Update dropdown menu style

Order Expiration

[x] Input should follow the same state rules (active, hover etc) as the other inputs in the order form. Only difference is it's smaller and less padding  around text
[x] Date and time buttons should have hover states
[x] Timezone picker button needs be wider to show full text
[x] Date, time and timezone button should all be the same button style:

[ ] Date, time and timezone pickers should appear above the button so they don't get cut off the bottom of the screen

BUY/ SELL Button

[x] Dynamically move up and down 16px from the nearest above element:

[x] Update text styles
[x] Update opacity for states: https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign?node-id=8223%3A4544

Moving ticket back into ui dev incoming....remaining items are lower priority then current needs....this can be finished working on after higher priority issues are done

I haven't figured out how to

Date, time and timezone pickers should appear above the button so they don't get cut off the bottom of the screen

@bthaile Could you help this item?

This is low priority, not needed right now

@rubyweb3
Everything in the form looks good thanks, only suggestion is creating enough room for the time to sit on one line. See image from build:

all good. closing


Does this feature exist? Selection dot always jumps back to the end when releasing it

Current behaviour
Selection dot always jumps back to the end of the graph when I release it. It does NOT stay at its position where I release it.
Desired behaviour
Selection stays at its position when releasing it. (At least having a prop that can enable this functionality)
I checked the example app and the source code, and could not find a prop that enable this feature.

would love to see this


Clicking on directories does not update the URL to include the default file

I've a directory structure like:
docs
.
├── 00_Folder_0
│   ├── 00_File_0.md
│   ├── 01_File_1.md
│   └── 02_File_2.md
├── 01_Folder_1
│   ├── 00_File_0.md
│   ├── 01_File_1.md
│   └── 02_File_2.md

In config.json, the option inherit_index is set to true.
What happens is that if the link for Folder_0 is clicked for example, the browser displays the contents of 00_File_0.md as it should -- EXCEPT the images are not displayed. The URL in the address bar is <base_url>/Folder_0. However, if the contents of the folder are first expanded in the navigation pane and then the link for the file 00_File_0.md is clicked directly, then everything is displayed correctly. In this case, the URL in the address bar is <base_url>/Folder_0/File_0. Note that this problem occurs only for images stored locally and not for those included via remote URLs.
Any ideas how to resolve this?
PS: If inhert_index is set to false, clicking on the folder link just expands the tree, and then the link for 00_File_0.md can be clicked. This solves the issue temporarily as it at least avoids displaying pages without images, though it is not ideal from an UX perspective.

Just realized that this could be related to #374. Sorry for the duplication if that is the case. For now, I'm happy turning off the inherit_index.

Hi,
the management of images is done separately from links so no problem it's not a duplicate ;-)
Can you please check in the inspector to see what url is used to load the images with and without the inherit_index option ?
Thanks in advance

The site is accessible at http://localhost/daux/
With the inherit_index option set to true, and the folder is clicked, the console shows:
http://localhost/daux/image_1.jpg Failed to load resource: the server responded with a status of 404 (Not Found)
http://localhost/daux/image_2.jpg Failed to load resource: the server responded with a status of 404 (Not Found)

Additional info:
baseURI:"http://localhost/daux/folder_1"
currentSrc:"http://localhost/daux/image_1.jpg"
With the inherit_index option set to true, and the file is clicked, there is no error in the console. The image properties show:
baseURI:"http://localhost/daux/folder_1/file_0"
currentSrc:"http://localhost/daux/folder_1/image_1.jpg"
With the inherit_index option set to false, clicking on the folder does not display the file at all. So the problem does not arise. When file link is clicked, it is exactly same as the 2nd option above.
Does this help?

Hello, I'm sorry this took so long, I pushed a fix in the development branch, can you give it a try ?

Pushed the fix to master but I want to ensure the problem is fixed before closing

I don't think it's working. Checked with both branches. Still get the same errors are before.

I didn't generate a new daux.phar yet, so you will have to do a composer install to use the source from lib

Thanks. So looks like the inherit_index option in config.json has no effect any more.
I can confirm that clicking on the directory now expands the menu rather than displaying the contents of the first file. The only exception is when an index.md file is present in the directory. This avoids the problem with the images not being displayed.

Okay, so it seems to work fine.
Yes, I just saw about the option never read, because with the changes I made, the is now in html and not in live anymore because it also works with generated documentations now.
And the fallback that read the old value from existing configurations was done in the wrong order, this is now fixed.

I'm sorry, I don't really follow because I'm not aware of the inner workings. Are you saying that the inherit_index option is now deprecated?

On the contrary, the inherit_index option was in the live object before :
{
  "live": {
    "inherit_index": true
  }
}

Because this option was only available in the live mode.
With the new way this works, it also is possible to set this option for the static mode.
Thus I moved this option over to html:
{
  "html": {
    "inherit_index": true
  }
}

But the code to support the legacy configuration option was in the wrong order and the $config["live"]["inherit_index"] was never read, only $config["html"]["inherit_index"].


Checkout continuously reloads when shipping method not in selected shipping methods

Hi,
We want to make use of the local_pickup option in WooCommerce next to the My Parcel plugin.
So we want My Parcel frontend only be showed when the flatrate option is selected. That is currently possible with the settings.
But when the user selects the local_pickup shipping method, the checkout is being reloaded. This is because of this else statement.
https://github.com/myparcelnl/woocommerce/blob/0d4ab8af3b78812de182dac4adcc3f9f566c65c1/assets/js/wcmp-frontend.js#L75-L94
Could you look in to it?

Maybe you could check if the shipping_method is in "Ongefrankeerd".

@ricklambrechts : This should be fixed in v4.0.7 that will be released later this week.


chore: lift can wrap 1-arity function

When I run unit test, R.trim throw an test error, as follows:
1109 passing (4s)
  1 failing

  1) trim
       trims all ES5 whitespace:

      AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected

+ '"

Hello, World!\\t\\n\\v\\f\\r   "'
- '"Hello, World!"'
      + expected - actual

      -"

Hello, World!\t\n\v\f\r   "
      +"Hello, World!"

      at strictEqual (test/shared/eq.js:8:10)
      at Context.eq (test/trim.js:13:5)
      at processImmediate (internal/timers.js:456:21)

I think Ramda should add a CI/CD procedure,  so that only those Pull Requests passed CI/CD's unit test can be merged.

I found Ramda have a setting file: .github/workflows/build.yaml, but it seems not working now.

Agreed.  I would like to get 0.28 out the door in the next week and then 1.0 within two months.  Then I want to revisit all the infrastructure, build, test, etc.

Agreed. I would like to get 0.28 out the door in the next week and then 1.0 within two months. Then I want to revisit all the infrastructure, build, test, etc.

Great. If there is anything I can do, let me know. I'll do my best to help.

Your build is failing because of linting errors which have been fixed in #3206. If you rebase your branch against master it should be fixed.

Your build is failing because of linting errors which have been fixed in #3206. If you rebase your branch against master it should be fixed.

Done.

@adispring I think you only reverted the change to the GH workflow so this should be good to go as all other changes were previously approved already I believe. I'm happy for this to be merged.


Add message functionality for owners

Users should be able to change their message at their location. Possibly it can replicate the holder form entirely, to also allow future use cases.

Holder form is in the works, but not ready for production yet. It will be an update for a later version.


Crashes for apks built with minSdkVersion 24 or greater

Hi,
I found this crash in the Flank test runner project, but it appears to be coming from the dependency with this project. I get the following crash on apks built with minSdkVersion 24 or greater:
DexMagic(dex=[100, 101, 120], newline=10, version=[48, 51, 53], zero=0)
	at com.linkedin.dex.spec.DexMagic.validate(DexMagic.kt:31)
	at com.linkedin.dex.spec.HeaderItem.validate(HeaderItem.kt:62)
	at com.linkedin.dex.spec.DexFile.<init>(DexFile.kt:35)
	at com.linkedin.dex.parser.DexParser$Companion$readDexFiles$1$4.invoke(DexParser.kt:79)
	at com.linkedin.dex.parser.DexParser$Companion$readDexFiles$1$4.invoke(DexParser.kt:23)
	at kotlin.sequences.TransformingSequence$iterator$1.next(Sequences.kt:147)
	at kotlin.sequences.SequencesKt___SequencesKt.toCollection(_Sequences.kt:630)
	at kotlin.sequences.SequencesKt___SequencesKt.toMutableList(_Sequences.kt:660)
	at kotlin.sequences.SequencesKt___SequencesKt.toList(_Sequences.kt:651)
	at com.linkedin.dex.parser.DexParser$Companion.readDexFiles(DexParser.kt:80)
	at com.linkedin.dex.parser.DexParser$Companion.findTestMethods(DexParser.kt:58)
	at com.linkedin.dex.parser.DexParser.findTestMethods(DexParser.kt)
	at com.walmart.otto.Flank.getTestCaseNames(Flank.java:268)
	at com.walmart.otto.Flank.start(Flank.java:71)
	at com.walmart.otto.Flank.main(Flank.java:118)

As an upstream user of this project, thanks for all your hard work! By the way I posted the Flank issue here https://github.com/TestArmada/flank/issues/165

Thanks for the report, this is useful! I probably won't have time to take a look at this until after Google I/O next week, but I'll follow up.

Confirmed! I got the same error using minSDK greater than 24. In order to check I changed to minSDK 18 and it works smoothly. Any idea?

Hi, I have checked the code and I think that I found the bug source. The problem is caused because in android 7 the magic dex version has changed from 035 to 037 and in Android 8 it changes again from 037 to 038. here is further information about it https://source.android.com/devices/tech/dalvik/dex-format#dex-file-magic I have forked the repo and I going to try just to change the version to see if it works.

Implemented a simple solution in PR #14


Conflict detection

I would like to detect conflicts ( by push and probably by pull ) . I mean situations when:

I try to commit and push a file which was already modified prior by another user in GIT
I try to do a pull and my file was modified.
How can I list this files, and mark them as merged after I edited them?

I'd probably take a look at https://github.com/centic9/jgit-cookbook/blob/master/src/main/java/org/dstadler/jgit/porcelain/MergeChanges.java , it shows how to get conflicts when doing a merge and push/pull basically are doing merge/rebase internally.
However please stick to what the issue-template and the README state and discuss questions of "how do I do XXX in JGit" elsewhere:
Note: Please use sites such as http://stackoverflow.com for general questions about JGit usage, not issues in this project. Issues should be used for problems with snippets and suggestions of missing snippets. Snippets from good answers on stackoverflow can then be included here, naturally.

Thank you for your help. I created an StackOverflow thread, soon will be active.
Also, I tested this code snippet after commit and before push, and the merge.getConflicts() returns a null value.
ObjectId mergeBase = git.getRepository().resolve("changes");
                        MergeResult merge = git.merge().
                                include(git.getRepository().resolve(Constants.HEAD)).
                                setCommit(true).
                                setFastForward(MergeCommand.FastForwardMode.NO_FF).
                                setMessage("Merged changes").
                                call();
                        System.out.println("Merge-Results for id: " + mergeBase + ": " + merge);
                        for (Map.Entry<String,int[][]> entry : merge.getConflicts().entrySet()) {
                            System.out.println("Key: " + entry.getKey());
                            for(int[] arr : entry.getValue()) {
                                System.out.println("value: " + Arrays.toString(arr));
                            }
                        }


Customers db

You can edit the company details by going on the certain company on the grid and edit that particular record


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Bicep
Module Classification?
Pattern Module
Module Name
avm/ptn/azd/monitoring
Module Details
Please add a Pattern Module named avm/ptn/operational-insights/monitoring, this module will create the following three resources:

Log Analytics Workspace.
Application Insights instance based on an existing Log Analytics workspace.
Portal Dashboard for an Application Insights instance.

For details, please refer to azd:
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/monitor/monitoring.bicep
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/monitor/loganalytics.bicep
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/monitor/applicationinsights.bicep
https://github.com/Azure/azure-dev/blob/main/templates/common/infra/bicep/core/monitor/applicationinsights-dashboard.bicep
CC: @jongio
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
jongio
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@Menghua1, thank you for proposing this module and thank you for your patience on triaging it! Many of us on the AVM core team have been OOF in the last couple of weeks, but we're back now! 😊
As this is categorized as a pattern module, its approval will require a more thorough assessment. We'll reach out to @jongio internally for more context and details as this proposal may require some adjustments to fit AVM's pattern definition and intent. The AVM core team will get back to you soon. Thank you for your patience!

Hi @jongio,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER

Hi @jongio,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.
[x] Issue template file updated.

Thank you for your work, @jianingwang123! The module index will be updated with this information.


[Bug] Failed to run Kyuubi Server on K8S

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
When I was ready to run Kyuubi Servers on our K8S cluster as a deployment, I came across a problem below.  The reason is the Kyuubi user failed to perform application because of permission.
Affects Version(s)
master
Kyuubi Server Log Output
starting container process caused: exec: "./bin/kyuubi": stat ./bin/kyuubi: permission denied: unknown

Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

How do you build your docker image?

this is not a proper way to define a bug without any description, I will move it to discussions


Send events to redis from a cronjob

Currently we try to send events to redis and also write to the DB in case redis is not available. Then we periodically check if any events were not able to be sent and have accumulated in postgres and try to resend them.
We also have to keep deleting events from postgres that were successfully sent.
We should clean this up and write all events to postgres and then periodically flush them to redis via a cronjob. We should extend the functionality of the existing cronjob that syncs with Keycloak to do this as well.

This would delay event sending more than necessary, I am not sure it is a desirable change.

@Panaetius @olevski Like stated, above this will delay messages by up to two minutes, which I don't think is acceptable. Why not use background task from Sanic for this specific purpose (sending messages to redis)?
See: https://sanic.dev/en/guide/basics/tasks.md

like we discussed a while back, it shouldn't be a cronjob so this issue is poorly worded.
A background task raised some concerns about taking up async runtime from dealing with regular requests (I personally don't think this becomes a concern any time soon, though) and stability (a catastrophic error in one fails both pieces).
For now I've just made a draft that has the message sending as a separate container that sends it every 1 second, but using the same image as we use for keycloak sync cronjob. So having one image for all our background needs that can run different jobs based on the container arg.

btw. in current, deployed data svc it's already a sanic background job, just every 30 secs and only if sending a message failed: https://github.com/SwissDataScienceCenter/renku-data-services/blob/main/bases/renku_data_services/data_api/main.py#L98
Having a separate container on k8s just for this may be a bit nicer but does increase complexity a bit as well

If there is a concern about starving server worker process with background tasks, we can start a dedicated non-server worker for this. See: https://sanic.dev/en/guide/running/manager.md#background-tasks

that's neat, I didn't know about that :+1:

@Panaetius Here is a basic POC which can be expanded -> https://github.com/SwissDataScienceCenter/renku-data-services/pull/214

I don't think using a multiprocessing queue is workable for delivery guarantees. It has to be through postgres and in the same transaction as entities that are changed to properly guarantee that messages are delivered at least once. At least unless we want some two-phase commit that makes everything else more complicated.
Also I'd really appreciate it if you would talk to the person assigned first before working on an issue that already has an assignee.

I'm not saying the messages should not be stored on the database, but rather that the trigger for processing can be done and handled from a IPC queue to ensure fast delivery to redis (except when it is down).

so you're proposing essentially keeping the current approach of saving to database, then sending the message (just through ipc and then redis instead of redis directly) and deleting from the DB if it worked, with the same safety job that resend messages if delivery failed that we have now?
I don't see how that has any meaningful difference over sending it to redis directly as we do now. Other than IPC being slightly faster so reducing latency of requests a bit, but sending to Redis is already not a huge bottleneck (it takes like 5 ms).
If instead you just save (entity and message) to db directly and return, and the background job picks up messages from the db and sends them every 1 second, you still get very reasonable delivery times, but with less complexity and less logic needed to ensure consistency (the db transaction takes care of that).

My suggestion is to move the background job which is sending pending messages to redis from a separate container into a non-server process in the same container.
You will still have a regular check for late messages to be sent, but you can also have an immediate path by using a trigger message to that process. The contents of the messages to be sent to Redis are not going through the IPC queue.
This concept should require very minimal changes from what you have now because it's almost the same concept, except that there is a path for sending messages almost instantaneously.

and I'm saying that if we have a background job that sends messages from DB periodically with a low enough period, that's good enough for us without needing that added complexity of the fast path for a minor latency gain. Things like more logic in the request/decorator for creating messages, dealing with concurrency between "fast" and "slow" path to not send messages twice if possible, less cognitive load on what goes on in general.
As mentioned, I don't think there's a difference between sending to redis directly and having a late message job, versus sending to ipc directly which sends to redis directly and having a late message job, except for an additional layer of indirection.
Whereas I prefer to have the late message job handle everything and requests just storing to the db. Then there's only one way for a message to go, less moving pieces, at the cost of a message taking like a second to be delivered (we can always tweak this if we need less) vs 10ms with your suggestion, which I think is something we can live with and is perfectly good enough for our use-cases.

Fine then.


Add VS solutions that can leverage the MSVC 2019 runtime

ONNXRuntime is considering adopting mimalloc, but in order for that to work, we need to be able to build mimalloc on windows under VS2019

I think I did that, isn't the case of just opening the solution and retargeting?

Yes that works fine as well, but the project I'm trying to integrate it into has mimalloc as a submodule and it'd be nice not to have to add cmake steps to patch the solution on every build

Thanks Kirsten! I can't try it out yet but will install vs2019 soon :-)

@kile0 what is the reason you're targeting the visual studio solutions rather than the cmake project? Very curious.


Enable to export old versions

needs https://github.com/bioimage-io/spec-bioimage-io/pull/335
This PR enables

exporting resource packages without format version update
export resource package with specified format version

also updates core to sync with latest bioimageio.spec changes (d63d17e11690893b594ea02c3f75b07b863a63c7)

Tests pass now and comments are addressed, I will go ahead and merge this.


How does the size of glyphs work?

I'm new to font rendering so this may be an obvious misunderstanding on my side of things.
I am attempting to build a font renderer that first creates an atlas of MSDFs. This of course requires me to create many images of MSDFs with tight bounding boxes.
Using the ttf_parser crate I can load a font and gather information about the size of glyphs. For some reason, ttf_parser reports the size of glyphs being far bigger than the size msdf-rs renders them at. How does the scale of the MSDF compare to that of what ttf_parser reports? Is there a way to control the size msdf-rs renders at?
Here's an example of the letter A from the Roboto-Medium.ttf in this very repository.
According to (Face::glyph_bounding_box)[https://docs.rs/ttf-parser/0.15.2/ttf_parser/struct.Face.html#method.glyph_bounding_box] the glyph has the following dimensions:
[src/graphics/text.rs:52] &size = Rect {
    x_min: -37,
    y_min: -437,
    x_max: 1016,
    y_max: 1536,
}

But a 16x16 image produced by msdf-rs looks like this.

Clearly, msdf-rs is interpreting these numbers as something scaled down, although I don't what that factor is.

The scale factor is 64.
This was done to match the behavior of the msdfgen command line utility, which acts differently to the msdfgen library. This behavior is probably wrong, and I was planning on removing this scaling factor in a breaking v0.2.0 release. For the time being, you should be able to get around this by dividing the coordinates by 64.0.

I had actually gotten very close to figuring this out through brute force guess and check (I was guessing about 65).
Thank you for that clarification. I'm looking forward to v0.2.0.

Thank you for that clarification. I'm looking forward to v0.2.0.

v0.2.0 has been published and is available on crates.io.


Excludes completed tasks/projects from autostaffing.

We experienced an issue with autostaffing where a StaffbotRequest for a task in a completed project was in the DONE_SENDING_INQUIRIES state rather than the COMPLETED state. This led autostaff to repeatedly try (and fail) to staff the task.
This fix excludes RequestInquiries for completed tasks / projects from staffbot's consideration. It refactors logic we've previously used to filter those requests out of the Available Tasks table.

Coverage increased (+0.001%) to 94.376% when pulling 0d3c88f5ab4e92fae275be0448561314c62b773a on autostaff-fix into 2ed265843d3905d1ed55ba3ee80a83391cb23276 on main.


Remove a few more instances of pinning on m_pEEType

Replaced by ref locals + S.R.CS.Unsafe

@MichalStrehovsky PTLA


only works once per install, raspios64

[!] ValueError: could not convert string to float at lines 1507, 1232, 1189

the problem  is with
1507   │         main_menu()
1232   │     requirements()
1189   │             if float(version) > float(zipver):
i have the same problem

Clone from here

still not working frm what u have given new git cloning ....!!!!!!

@cyb3rwolf-coder Attach your screenshot of git cloning, that may have some clues


Updated the Add use case to use the InMemoryJourneyAccessObject instead

Good news! The Add use case now uses the JourneyAccessObject to access the graph, and it works perfectly.
Moreover, the addedToGraph variable of the corresponding Node object will be updated.


strip ?request=getcapabilities from request before add request=getmap

not sure if this is an openlayers or wegue issue
if you add a wms layer by url example.com/ows?service=wms&request=getcapabilities
the viewer adds; request=getmap to the querystring, without removing request=getcapabilities
just to verify, the getmap request is send to the submitted getcapabilites url or the onlineresource as advertised in capabilities?

I can reproduce it with
{
    "type": "WMS",
    "lid": "ahocevar-wms",
    "format": "image/png",
    "layers": "topp:states",
    "url": "https://ahocevar.com/geoserver/wms?SERVICE=WMS&REQUEST=GetCapabilities",
    "XXXurl": "https://ahocevar.com/geoserver/wms",
    "transparent": true,
    "singleTile": false,
    "projection": "EPSG:3857",
    "attribution": "",
    "isBaseLayer": false,
    "visible": true,
    "displayInLayerList": true
  },

As far I can see from the code (https://github.com/wegue-oss/wegue/blob/7f53a55545015d37c5b4fc73c8f9dba334240b7d/src/factory/Layer.js#L107) the url is taken as is and no parsing of the capabilities is done.
Does it work for you to just skip the query params of your provided url?

Yes it works without the query, a typical use case is to find a record in a catalogue, take the wms url from the record as-is, this url often has the ?request=getcapabilities postfix, splitting at ? Is not a good idea, because some wms urls include a required parameter, like mapserver; map=foo.map


Is it possible to add copy snippet feature from the docs just by one click?

The docs have some snippet explained how to use it. I think there should be a copy option and when we click we have the copied snippet.

Hi @syedsimanta03 it would be nice indeed, but the doc is actually generated from the components README files in this repo, so i'm afraid it's not possible


Run prettier on SUMMARY.md

My markdown formatter (prettier) seems to like adding newlines lines between bulleted lists and doesn't have a way to configure this. (This is also how I prefer it but I don't feel strongly about it.)
Can we leave these formatting changes the way prettier likes it for now? And then once the docs are a little further along we can decide what conventions we want to follow and stick to them then. I can either selectively ignore files from being formatted with prettier or turn off prettier entirely.

Sounds good


Replace pandas friendly times normalization

behavior changes from pandas implementation:

handles tz aware datetime arg
behaves intuitively in these cases:  '2014 ', '2014-02 ', '2014-2'
(trailing space, trailing space, no zero pad on month)
includes fairly comprehensive tests

Few minor comments, but overall :+1: from me.

Can you also remove the pandas dep from setup.py and meta.yaml?

Looks good! :+1:

@cowanml Thanks!

Yes, thanks, @cowanml! This is much better.


Add array cursor optimized for navigation

This is a quick experiment in trying to improve performance (related) by adding a new Cursor implementation for arrays that's optimized for navigation (as opposed to modification). Because decoders generally only navigate arrays, this does help out, although not as much as I'd hoped (not really much for speed, and allocation rates are only 8-10% lower in the decoding benchmarks).
There are no changes to the public API, but there is some extra complexity, so I'm not sure I want to merge this, but I wanted to post it here for possible discussion.

Current coverage is 69.64%

Merging #109 into master will decrease coverage by -0.67% as of d01f551

Powered by Codecov. Updated on successful CI builds.

Something like this ended up happening in #459.


Add mise configuration for tool autmation

Using mise to setup kind and kubectl.

Mise is a dev environment setup tool,  kind of like pipenv or npm, but for general tools as opposed to a particular language. Its based on the asdf plugins, but also supports other installation backends.
In short, with this file in place, and with mise installed, going into the directory and typing mise install causes the tools listed to be installed and made available in $PATH. IF the needed versions of the tools are already available locally, just CD-ing into the repo dir causes the right versions to be in $PATH.
Mise also support env variable, hooks and scripts that may come handy in the future to make the experience of using this repo nicer.

Let's create an issue to add it.

Let's create an issue to add it.

Added #175


run_diagnostic fails when re-running the diagnostic script.

Hi,
the run_diagnostic fails when re-running the diagnostic script. This is because the cfg["write_netcdf"] and cfg["write_plots"] are failing at the os.mkdirs stage, and these folders already exist. The simplest solution is to add an os.path.exists() before trying to make the directory.
`
# Create output directories
if cfg['write_netcdf']:

    if not os.path.exists(cfg['work_dir']):

        os.makedirs(cfg['work_dir'])

if cfg['write_plots']:

    if not os.path.exists(cfg['plot_dir']):

        os.makedirs(cfg['plot_dir'])

`

This code was added here:
https://github.com/ledm/ESMValTool/commit/10d90c9a5579cb16a02f2c5d63017109cc252476

@ledm The problem with this solution is that it may accidentally overwrite data the user has created manually while analysing the reason their script crashed. Also, the diagnostic script may behave differently/unexpectedly if files are already present in it's output directory.
A better solution would be to check if the directories exist, and if this is the case, warn the user that they should be deleted manually before trying to re-run the script.

@bouweandela @ledm this is inconvenient in the case of heavy diagnostic development, my local fix is to add a check in _base.py if.os.path.exists( yada yada...but it's still inconvenient since with each new installation I need to remember to place the check. I have raised this elsewhere as well, I think it's totally up to the user if they are negligent enough that's their problem, we need this thing so we can rerun diags without waiting for all the preproc stuff

I'm not sure this is related to #348. I'll add some nicer warning messages and a -f flag to the diagnostic script to force emptying the output directories before a rerun.

Perfect, thanks a lot!
Also, please remember to add the -f flag into the logger message following the message "To Re-run this script..."


Why does youtube-dl always modify cookies file

I write an app to manage cookies files and use youtube-dl to download from youtube. When pass the cookies,  youtube-dl always modifies this file, so I have some following problems:

Is it safe if I call 2 youtube-dl using the same cookies file (Or another process updates this file)
Can I ask youtube-dl not to modify the cookies file directly ?

Thank you !

No.
No, since it has to modify cookies to reflect potential changes by remote party during extraction process.

No.
No, since it has to modify cookies to reflect potential changes by remote party during extraction process.


Does FastSurfer support non-T1 modalities for improved pial surfaces?

Hello, FreeSurfer supports --flair or --t2 modalities. Does FastSurfer also support this? Just wondering because I do not see anything in the documentation about it.

No it does not. In the next release we add a flag for T2 as it can help the hypothalamus module (which we'll also add). It would be easy to include support for using the T2 also for the pial, but in the past we have seen cases where it did more harm than good ( in earlier Freesurfer versions, maybe it is fixed now).


add warning message for users

Regrid.pl can't be used with new boundary conditions v09 that contain snow albedo information.
This PR is to let users know that regrid.pl  is in process of being phased out and replaced with remap_restarts.py.  We need this for PR#618 on GEOSgcm_GridComp)

@gmao-rreichle is this what you had in mind?
This is how it looks on screen:

@biljanaorescanin: Thanks! I edited the wording of the message a bit.  Please double-check that I didn't mess up the syntax. Should then be ready to go after sorting out who is in charge (@sdrabenh, @mathomp4 ?) and what is needed for the CHANGELOG.md file, if anything.  The change seems so trivial that it may not be appropriate for the log.  The bigger point is that we have to start phasing out regrid.pl in earnest.

@gmao-rreichle I'll have it skip the changelog for this (not sure why the trivial label didn't pick it up).
It's probably good to get this in at the same time as #303. Do a "regrid release" as it were...


docs(Card): fix readme examples

Fixing readme  example UIKit.Card to Card.

It's already fixed :)


1.3.2

Release notes - Planorama - Version 1.3.2
Bug
PLAN-440 Sometimes the name shown in the session conflict isn't correct
PLAN-439 Loading session flyout sometimes causes browsers to freeze
PLAN-438 Schedule page takes ~20 seconds to load
Story
PLAN-442 change version numbers to 1.3.2

What's Changed

Main fix brakeman by @balen in https://github.com/ChicagoWorldcon/planorama/pull/260
Fixes build action by @balen in https://github.com/ChicagoWorldcon/planorama/pull/261
PLAN-439 PLAN-440 Fix conflict display and performance side effects by @balen in https://github.com/ChicagoWorldcon/planorama/pull/262
Plan-438 Stag grid performance fixes by @balen in https://github.com/ChicagoWorldcon/planorama/pull/264
Plan-442 update version numbers by @balen in https://github.com/ChicagoWorldcon/planorama/pull/266

Full Changelog: https://github.com/ChicagoWorldcon/planorama/compare/v1.3.1...v1.3.2


scaled_joint_trajectory_controller missing implementation

Hello, I tried to build the driver according to the readme
I used a container
FROM ubuntu:20.04

ENV DEBIAN_FRONTEND=noninteractive
# update packages and install dependencies
RUN apt-get update && apt-get install -y \
    wget

RUN apt-get update && apt-get install -q -y \
    bash-completion \
    dirmngr \
    gnupg2 \
    lsb-release \
    python3-pip \
    curl

# Setup environment
ENV LANG C.UTF-8
ENV LC_ALL C.UTF-8

RUN pip3 install colcon-common-extensions rosdep vcstool

RUN apt-get update && apt-get install --no-install-recommends -y \
    autoconf \
    automake \
    cmake \
    libtool \
    libltdl-dev \
    git \
    mercurial \
    && rm -rf /var/lib/apt/lists/*
RUN apt update && apt install curl gnupg2 lsb-release
RUN curl -sSL https://raw.githubusercontent.com/ros/rosdistro/master/ros.key  -o /usr/share/keyrings/ros-archive-keyring.gpg
RUN echo "deb [arch=$(dpkg --print-architecture) signed-by=/usr/share/keyrings/ros-archive-keyring.gpg] http://packages.ros.org/ros2/ubuntu $(lsb_release -cs) main" | tee /etc/apt/sources.list.d/ros2.list > /dev/null
RUN apt update && apt install -q -y ros-foxy-ros-base

and the following steps
docker run -t -v "$PWD:/work" ros2_ur5 .
cd /work/
mkdir -p /work/src/
git clone https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver.git src/Universal_Robots_ROS2_Driver
vcs import src --skip-existing --input src/Universal_Robots_ROS2_Driver/Universal_Robots_ROS2_Driver.repos
source /opt/ros/foxy/setup.sh
rosdep init
rosdep update
rosdep install --ignore-src --from-paths src -y -r
colcon build --cmake-args -DCMAKE_BUILD_TYPE=Release

The problem I hit is:
--- stderr: ur_controllers
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp: In member function ‘virtual controller_interface::return_type ur_controllers::ScaledJointTrajectoryController::update()’:
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:58:10: error: ‘is_halted’ was not declared in this scope; did you mean ‘is_halted_’?
   58 |     if (!is_halted) {
      |          ^~~~~~~~~
      |          is_halted_
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:59:7: error: ‘halt’ was not declared in this scope
   59 |       halt();
      |       ^~~~
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:93:38: error: ‘joint_position_state_interface_’ was not declared in this scope; did you mean ‘joint_state_interface_’?
   93 |     state_current.positions[index] = joint_position_state_interface_[index].get().get_value();
      |                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                      joint_state_interface_
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:94:39: error: ‘joint_velocity_state_interface_’ was not declared in this scope; did you mean ‘has_velocity_state_interface_’?
   94 |     state_current.velocities[index] = joint_velocity_state_interface_[index].get().get_value();
      |                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                       has_velocity_state_interface_
/work/src/Universal_Robots_ROS2_Driver/ur_controllers/src/scaled_joint_trajectory_controller.cpp:128:9: error: ‘joint_position_command_interface_’ was not declared in this scope; did you mean ‘has_position_command_interface_’?
  128 |         joint_position_command_interface_[index].get().set_value(state_desired.positions[index]);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |         has_position_command_interface_
make[2]: *** [CMakeFiles/ur_controllers.dir/build.make:63: CMakeFiles/ur_controllers.dir/src/scaled_joint_trajectory_controller.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:78: CMakeFiles/ur_controllers.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
---
Failed   <<< ur_controllers [2.44s, exited with code 2]

Is this a known problem? Should I use a different working branch?

Is it possible that you are using a cached version of the repository? Check that you have this in your Universal_Robots....repos file:
ros2_controllers:
type: git
url: https://github.com/ros-controls/ros2_controllers.git
version: 0.2.1

Ok, but it seems to be pulling in a different version (which is why you are seeing the error that you are). The other thing to try is to use $COLCON_WS as is used in the instructions. For whatever reason this seems important to set that environment variable (as in when I did things manually without setting $COLCON_WS, I couldn't get things to build in docker, even though I was sure I was doing the equivalent operations). Sorry I can't be more helpful.

it seem a pull was all I needed
From https://github.com/UniversalRobots/Universal_Robots_ROS2_Driver
   c5b84db..38a467e  main       -> origin/main
Updating c5b84db..38a467e
Fast-forward
 ur_bringup/launch/ur_control.launch.py                         | 10 ++++++++++
 ur_robot_driver/doc/features.md                                | 53 +++++++++++++++++++++++------------------------------
 ur_robot_driver/include/ur_robot_driver/hardware_interface.hpp |  1 -
 ur_robot_driver/src/hardware_interface.cpp                     |  4 ----
 4 files changed, 33 insertions(+), 35 deletions(-)

Now it's ok
Finished <<< ros2_controllers [1.03s]
Finished <<< ur_controllers [19.8s]
Summary: 30 packages finished [3min 46s]
Thank you


Suggestion: Configurable log file names

Hey Mike,
While troubleshooting my issue (#80), I thought on the idea of being able to use a log file name that had the (market pair) in the name. i.e. (pycrypto-ETHUSDT.log).
This hack is handy if you are running multiple bots under one install using subprocesses.
So, i got it working on my version of the code base:
[config.json]

adding a new key to the (config) array

"logfile_prefix" : ""
[models/PyCryptoBot.py]

added a default configuration value in case the key does not exist in the (config.json) file.
added logic to check for key in config file and to scrub it and use it.

self.logfile_prefix = 'pycryptobot'
if 'logfile_prefix' in config:
     if isinstance(config['logfile_prefix'], str):
        self.logfile_prefix = config['logfile_prefix'].strip()
        if self.logfile_prefix == "":
           self.logfile_prefix = "pycryptobot"
        else:
           self.logfile_prefix = config['logfile_prefix']

[pycryptobot.py]

added new variable (logfile_name)
set variable to to use (logfile_prefix) property concatenated with (market) value and hardcoded file extension
the file extension can be made configurable as well
logic could be added to pre-pend a date string before the file extension

try:
    # initialise logging
    log_filename = app.logfile_prefix + "-" + app.getMarket() + ".log"
    logging.basicConfig(filename=log_filename, format='%(asctime)s - %(levelname)s: %(message)s', datefmt='%m/%d/%Y %I:%M:%S %p', filemode='a', level=logging.DEBUG)

Here are examples of the files created using the logic
-rw-r--r--   1 admin  staff   3519 Apr 28 14:21 pycryptobot-ADAUSDT.log
-rw-r--r--   1 admin  staff   3464 Apr 28 14:21 pycryptobot-ETHUSDT.log
--

Good suggestion. Though this might not be ideal for dockerised bot. Another approach is to pass the log filename as a parameter OR mount the logs on docker-compose like the one below
Option 1: No code change, run bot on docker
version: "3.9"

services:
  # coinbase pro
  ethdai:
    image: pycryptobot
    container_name: ETHDAI
    volumes:
      - ./market/ETHDAI/config.json:/app/config.json
      - ./market/ETHDAI/pycryptobot.log:/app/pycryptobot.log
      - ./market/ETHDAI/graphs:/app/graphs
      - /etc/localtime:/etc/localtime:ro
    deploy:
      restart_policy:
        condition: on-failure

  btcdai:
    image: pycryptobot
    container_name: btcdai
    volumes:
      - ./market/BTCDAI/config.json:/app/config.json
      - ./market/BTCDAI/pycryptobot.log:/app/pycryptobot.log
      - ./market/BTCDAI/graphs:/app/graphs
      - /etc/localtime:/etc/localtime:ro
    deploy:
      restart_policy:
        condition: on-failure

Option 2: Code change neeed, pass the logfile as parameter or as environment variable
python3 pycryptobot.py --market BTC-GBP --granularity 3600 --live 1 --verbose 0 --selllowerpcnt -2 --logfile ./btc-gbp.log

inside docker-compose.yml
version: "3.9"

services:
  # coinbase pro
  ethdai:
    image: pycryptobot
    container_name: ETHDAI
    volumes:
      - ./market/ETHDAI/config.json:/app/config.json
      - ./market/ETHDAI/graphs:/app/graphs
      - /etc/localtime:/etc/localtime:ro
    environment:
     - LOGFILE:btc-gbp.log
    deploy:
      restart_policy:
        condition: on-failure

  btcdai:
    image: pycryptobot
    container_name: btcdai
    volumes:
      - ./market/BTCDAI/config.json:/app/config.json
      - ./market/BTCDAI/graphs:/app/graphs
      - /etc/localtime:/etc/localtime:ro
   environment:
     - LOGFILE:eth-dai.log
    deploy:
      restart_policy:
        condition: on-failure

@whittlem would u like me to work on this one?

Hi @letsautom8 ,
I agree with you that simply passing the log file name as a parameter is an easier implementation.
I am running the bot within a wrapper script so, dynamically modifying the log file name and passing it as a parameter serves the same purpose.
I believe the bot script is already heading towards a version that supports multiple bot instances which will lend itself to needing instance specific logging, etc.

@letsautom8, yes please. If you wouldn't mind that would be great. Thanks.

And thanks @github-ptaylor. Great idea.

Here you go #87

@letsautom8 i like how you coded that, super clean.
i get to try something new from a fresh copy because i totally "fubar'd" my previous install :)

@github-ptaylor, if you "fubar'd" the code just type "git reset --hard" and then "git pull". Should put you back to normal :)

@letsautom8 is this feature working only on docker ?

@letsautom8 is this feature working only on docker ?

@delphsmith it is working on the non-docker install as well.
i am using the feature with the command-line args

oh ok ...not working for me, but i put it in my config.json file.
I did not precise a folder tho, thinking it would put it in the working folder by default, but maybe i'm missing that.
I'll try with the cli args, thanks

@delphsmith
i just confirmed that it is NOT configurable from the config.json file.
@letsautom8 @whittlem i can add the code the pull request method if you like

github-ptaylor that would be great :) thanks


[Android] Application crashes on startup when building with last Flutter 1.12.13

Hi! We got a problem after updating to the latest version of Flutter 1.12.13, there is a crash on launch the application after successful build.
Steps to Reproduce

upgrade flutter with

flutter upgrade

upgrade dependences and cache

flutter pub cache repair
flutter pub upgrade
flutter pub get

Build and launch any application with the Pushy SDK dependecy, pushy-demo-flutter for example

flutter clean
flutter build apk
flutter install

Crash at application startup.

Logs
through Logcat
E/Pushy: Failed to invoke getFlutterView() on parent activity
    java.lang.NoSuchMethodException: getFlutterView []
        at java.lang.Class.getMethod(Class.java:1981)
        at java.lang.Class.getMethod(Class.java:1637)
        at me.pushy.sdk.flutter.g.a(:55)
        at me.pushy.sdk.flutter.g.a(:46)
        at b.a.c.a.a(:14)
        at me.pushy.example.MainActivity.onCreate(:11)
        at android.app.Activity.performCreate(Activity.java:6706)
        at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1122)
        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2837)
        at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2949)
        at android.app.ActivityThread.-wrap12(ActivityThread.java)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1655)
        at android.os.Handler.dispatchMessage(Handler.java:110)
        at android.os.Looper.loop(Looper.java:203)
        at android.app.ActivityThread.main(ActivityThread.java:6472)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1114)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:975)

Additional
Reproduced both for io.flutter.app.FlutterActivity and io.flutter.embedding.android.FlutterActivity
flutter doctor
[√] Flutter (Channel stable, v1.12.13+hotfix.5, on Microsoft Windows [Version 10.0.18362.476], locale ru-RU)
    • Flutter version 1.12.13+hotfix.5 at D:\flutter
    • Framework revision 27321ebbad (2 days ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at D:\Users\Egor\AppData\AndroidSDK
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: D:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.5)
    • Android Studio at D:\Program Files\Android\Android Studio
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] Connected device (1 available)
    • M6T • 811HEBT623EQ8 • android-arm64 • Android 7.0 (API 24)

• No issues found!

Please check updates for using the platform_channel in the Flutter SDK, thanks!
https://github.com/flutter/flutter/tree/master/examples/platform_channel

Hi @EgorGrisina,
Thank you so much for taking the time to report this issue and provide such exact reproduction steps! 💯
We've reproduced and released a fix in https://github.com/pushy-me/pushy-flutter/commit/55c319be13369e0fcfc8f46c579a295bec3b2689 which has been published as pushy_flutter: 1.0.3.
Please edit the pubspec.yaml in the root directory of your project and update the plugin version referenced to the following:
pushy_flutter: 1.0.3

Flutter now comes preconfigured with ProGuard. Please make sure the following lines are present in your android/app/proguard-rules.pro file (please create this file if it doesn't exist yet):
-dontwarn me.pushy.**
-keep class me.pushy.** { *; }

Then, run:
flutter pub get
flutter clean
flutter build apk
flutter install

Your app should run now 🎉


Extend support for BYTE_STREAM_SPLIT to FIXED_LEN_BYTE_ARRAY, INT32, and INT64 primitive types

Please correct me if I'm wrong! It seems arrow-rs has added BYTE_STREAM_SPLIT support for float types, but not for other numerical data types like INT32.
Since then, the Parquet spec has been expanded to extend BYTE_STREAM_SPLIT encoding to other numerical primitive types: https://github.com/apache/parquet-format/pull/229. The C++ PoC is here: https://github.com/apache/arrow/pull/40094.
It would be good for arrow-rs to support BYTE_STREAM_SPLIT encoding additionally for FIXED_LEN_BYTE_ARRAY, INT32, and INT64.

@anjakefala are you working on this? If not I have some free cycles to devote to this.

@etseidl I am not! I would love it if you picked it up. =)

Thank you so much @etseidl and @alamb! =)

Thank you so much @etseidl and @alamb! =)

Also @mapleFU  🙏

@anjakefala please let me know if you have performance issues, particularly with FIXED_LEN_BYTE_ARRAY data. That was particularly problematic on the read side. 😦

label_issue.py automatically added labels {'parquet'} from #6159