GATK AddOrReplaceReadGroups activation (hw 6)

Hello :)
After a successful installation of GATK, I tried to run this command:
$ gatk AddOrReplaceReadGroups -I "SRR1569760_vs_S288C.HQ.sort.bam" -O "out.bam" -LB READS -PL ILLUMINA -PU 1 -SM SRR1569760 --CREATE_INDEX
but I got this error -
Using GATK jar /home/nofar/gatk-<IP_ADDRESS>/gatk-package-<IP_ADDRESS>-local.jar
Running:
    java -Dsamjdk.use_async_io_read_samtools=false -Dsamjdk.use_async_io_write_samtools=true -Dsamjdk.use_async_io_write_tribble=false -Dsamjdk.compression_level=2 -jar /home/nofar/gatk-<IP_ADDRESS>/gatk-package-<IP_ADDRESS>-local.jar AddOrReplaceReadGroups -I SRR1569760_vs_S288C.HQ.sort.bam -O out.bam -LB READS -PL ILLUMINA -PU 1 -SM SRR1569760 --CREATE_INDEX
Traceback (most recent call last):
  File "/home/nofar/gatk-<IP_ADDRESS>/gatk", line 511, in <module>
    main(sys.argv[1:])
  File "/home/nofar/gatk-<IP_ADDRESS>/gatk", line 177, in main
    runGATK(sparkRunner, sparkSubmitCommand, dryRun, gatkArgs, sparkArgs, javaOptions, debugPort, debugSuspend)
  File "/home/nofar/gatk-<IP_ADDRESS>/gatk", line 360, in runGATK
    runCommand(cmd, dryrun)
  File "/home/nofar/gatk-<IP_ADDRESS>/gatk", line 416, in runCommand
    check_call(cmd, env=gatk_env)
  File "/home/nofar/miniconda3/envs/mamba/lib/python2.7/subprocess.py", line 185, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/home/nofar/miniconda3/envs/mamba/lib/python2.7/subprocess.py", line 172, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/home/nofar/miniconda3/envs/mamba/lib/python2.7/subprocess.py", line 394, in __init__
    errread, errwrite)
  File "/home/nofar/miniconda3/envs/mamba/lib/python2.7/subprocess.py", line 1047, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory

The bam file as it named in the command line is accurate to my directory. I don't understand what is the problem, and I would like to get help :)
Thanks in advance,
Nofar :-]

@ranelmalka100 I assume this has been resolved for you


Dashbaord not draggable

if the app is show as window then  i cannot drag the dashboard.

Yeah, but  there  reason for that was,  there is notification section on the header so it was not draggable 😀. but also now you can  drag the window onhovering  the title rather than that empty space 😊


Added frame rate and relative frame delay to span metrics

:scroll: Description

Added new measurement frames.relative_delay to spans. This metric is the frame delay divided by the span duration. It provides a relative measure of frame delay to better assess the impact of frame delay.
Added new measurement frames.rate to spans. This metric estimates the frame rate based on the span duration and the number of frames.

:bulb: Motivation and Context
The goal of this MR is to add some new metrics to improve the user's ability to prioritize poor performing spans.
In reviewing the documentation of frames delay, I noticed the example prioritization scenarios don't account for how long the span was.
For example if Scenario 1 had a frame delay of 2400ms but hypothetically the span duration was 5x the span duration of Scenario 3, wouldn't that mean that over the entire span, Scenario 3 actually had less frame delay?
Hence the metric of relative_delay.
I added frames.rate as the variables needed to calculate it were present.
:green_heart: How did you test it?
Wrote one test for relative frame delay
Otherwise, tested locally to evaluate the reported estimated frame rate.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes*
[x] No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
[ ] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

we don't need to estimate the frame rate, we already know the frame rate since we fetch it from the native layer
maybe you have some thoughts on the relative delay proposal @philipphofmann

I'm happy that you're looking into frame delay, @YamenSarah. We don't calculate such metrics on the client side; instead, we plan to calculate them on the backend because we have all we need on the backend to do so, and adding it on the client increases the payload size. Our new mobile module already shows that data. We still need to display it properly in all performance-relevant places. This will happen in the upcoming months.

@buenaflor Where is the frame rate reported currently?
@philipphofmann That sounds reasonable. Just to confirm, I'm not talking about average frame delay. I'm talking about frame delay relative to the span duration.
I've been unable to do this in the discover page myself as your Equation feature does not let you use aggregate fields in an Equation, and I want to see (AVG(frame_delay) / AVG(span_duration)) etc..

Where is the frame rate reported currently?

We do not report the frame rate right now, it's only known internally.
If you need the frame rate we can expose it or you can do the calculation as you've done before

I've been unable to do this in the discover page myself as your Equation feature does not let you use aggregate fields in an Equation, and I want to see (AVG(frame_delay) / AVG(span_duration)) etc..

If you want to send custom measurements you can also do so in the beforeSend hook:
options.beforeSendTransaction = (transaction) {
  // calculate your relative delay
  transaction.measurements['frames_relative_delay'] =
      SentryMeasurement('frames_relative_delay', relativeDelay);
  return transaction;
};

We don't have frame delay yet available in Discover. We still need to add it, but I can't give you an ETA at the moment.


misc/main_lenslet_array_eris.py

That file is importing routines from the following external modules:

ccdproc
photutls

should we add them to the setup.py? or is the module not useful (there are references to local files not included in the distribution)?

Oops didn't see that it has already been removed tonight.


make port selection slightly more idiomatic

Codecov Report
Patch coverage: 83.33% and project coverage change: +0.03 :tada:

Comparison is base (db77faf) 60.48% compared to head (3fa5b29) 60.51%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #414      +/-   ##
==========================================
+ Coverage   60.48%   60.51%   +0.03%
==========================================
  Files          34       34
  Lines        5276     5283       +7
==========================================
+ Hits         3191     3197       +6
- Misses       2085     2086       +1

Impacted Files
Coverage Δ

zingocli/tests/integration_tests.rs
84.84% <ø> (ø)

zingo-testutils/src/lib.rs
85.77% <69.23%> (-1.04%)
:arrow_down:

zingolib/src/blaze/update_notes.rs
91.48% <100.00%> (+0.09%)
:arrow_up:

zingolib/src/lightclient.rs
62.42% <100.00%> (+0.28%)
:arrow_up:

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


Swap CtyInstanceState with PlanResourceChange flag

An experiment to see if PlanResourceChange can substitute for CtyInstanceState while still handling wafv2_web_acl resource correctly. See https://github.com/pulumi/pulumi-aws/issues/3361

This slightly regressed on the ability to cleanly refresh the state for the example program  https://github.com/pulumi/pulumi-aws/issues/3361  - but the unclean refresh now matches TF actual changes. It's in a broad category of issues of this sort, we need to figure out the policy for it. So I slightly am leaning on recommending to merge this in so we can retire CtyInstanceState.

The unclean refresh is rather unfortunate but probably not a big deal.
I think I saw somewhere that it only happens once and then doesn't happen during subsequent up/refreshes, is this correct?
Does refresh run Diff? Could we add some diff customization for these properties to say null and empty are the same?

I think I saw somewhere that it only happens once and then doesn't happen during subsequent up/refreshes, is this correct?

Yes.
I still think we might want to work that one out to be more friendlier but it is indeed somewhat tolerable.


default blueprint

Adds a default addon blueprint to install the dropzone bower dep. Now during installation, the blueprint will be ran and will install dropzonejs for inclusion.
Additionally ember install ember-cli-dropzonejs can be used to run the default blueprint.

Thanks for this 👍


Offer

It seems you're much more interested in continuing development of forkphorus than I am. If you want, I'm willing to just give you write access to forkphorus/forkphorus.
Just going to leave that offer here. If you want to continue your own fork that's perfectly fine too.

Thanks for that offer, but I don't think that's a good idea.
This is actually my first time using node.js and typescript and I still haven't figured out how most parts of the forkphorus work.  Also I'm not even planning to do any long-term development on this fork. As for the current changes, I am planning to do pull requests with detailed explanations once I consider those branches done, which should hopefully happen soon.
Somewhat unrelated, but I would like to ask, are you interested in new-webgl branch?

Sounds good. Just send them when you're ready.
I'm personally not going to spend time on improving the WebGL renderer, but if someone like you wants to, that's sounds great


updated retroshare (0.6.0,20160206-7ded128b)

Closes #18368.

Merged as 234e64faa70512df9cac771f090136a0b011ef7b.


Add Chinese translate

Hello, I am a localized author. This is a Chinese language file that I and the team have discussed and hope to adopt,plssssss

Thanks for the contribution! I'll be sure to include it in a release soon :)


Add responsive float classes

Fixes #13690.
Also removes .pull-left and .pull-right since they're redundant to .pull-xs-left and .pull-xs-right.
CC: @mdo

@mdo SGTY?

Woof. Yeah, I suppose. Honestly still not keen on all the responsive tiers for these classes, but I know it's been requested and I believe I've done custom media queries a few times to do the same thing.


Windows 10 volume overlay artist delay

Checklist:

[x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[x] I have double checked and can reproduce the issue

OS:
Windows 10 Home 1809, build 17763.55
GPMDP Version:
4.6.1
Issue Descriptions:
My issue is for the Youtube Music player of GPMDP.
The "Show track information in Windows 10 volume overlay" artist has a delay of 1 song. If I switch back and forth between two different songs they have the wrong artist, see the GIF for an example.

Steps to Reproduce:

Play any song
Change Windows volume with the vertical slider (so not the one in your taskbar) and look at the current song's artist, or use any playback keys you have on your keyboard that shows the track information if you press on it.
Switch songs either through the player, through the track information playback buttons or the playback keys you have on your keyboard
Change Windows volume with the vertical slider (so not the one in your taskbar) and look at the current song's artist, or use any playback keys you have on your keyboard that shows the track information if you press on it.

Duplicate #3180


Fastify major version update

Fastify has been on version 2 for a while and it seems the generated template is compatible with it, so I think the version in the _package.json template can be update to the current version ^2.10.0.

fastify has been updated to v2 by https://github.com/nuxt/create-nuxt-app/commit/952230b1227da7b05182667385eaabc4e3127281, this change will be valid in next release


Default values not supported on SE queries

 Issue by kbrandwijk
Tuesday Aug 29, 2017 at 22:45 GMT
Originally opened as https://github.com/graphcool/api-bugs/issues/247

What is the current behavior?
It's not possible to define default values for query parameters in a SE. The value of the parameter inside the function is null
Please share the relevant part of your GraphQL schema and all functions, permissions or other project settings for easier reproduction
extend type Query {
  add(a: Int = 2, b: Int!): AdditionPayload
}

What is the expected behavior?
Default values should be used.

The current workaround is to simply handle the null case in code.
It would be nice to enable a default value in the extension schema. Is the proposed syntax valid GraphQL?

Yes: https://launchpad.graphql.com/73kmp8ppj

That's awesome - thanks!


chore(dashboard_rbac): bump superset-ui 0.17.27

SUMMARY
bump for new superset-ui version to support needed extra_jwt for the dashboard_rbac ff
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Link the PR for transparency: https://github.com/apache-superset/superset-ui/pull/1021


Python 2.6 support

There is at least one simple syntax issue with using httq on Python 2.6.
https://travis-ci.org/jayvdb/anyhttp/jobs/60478689
I didnt investigate further.
I dont mind a wontfix resolution, as I workaround it by excluding it from py2.6 builds
(https://github.com/jayvdb/anyhttp/commit/34a3a79f2dc7d046b00c46b447c9166cf99bc20d) , and believe anyone using python 2.6 is crazy.

Python 2.6 support has now been added.


Very noob question

The output name gets changed eg. 'Mp4_Avc_Aac_16x9_1280x720p_24Hz_4.5Mbps_qvbr.mp4'.
Is there anyway to have customized output name eg. VideoName+CustomName?
Thank You

The solution creates a series of MediaConvert Templates to create the set of desired outputs. Each output is named with the source file name + a name modifier for each preset. This is defined in the template for each preset as the NameModifier, for example:
   {
      "Preset": "_Mp4_Avc_Aac_16x9_1280x720p_24Hz_4.5Mbps_qvbr",
      "Extension": "mp4",
        "NameModifier": "_Mp4_Avc_Aac_16x9_1280x720p_24Hz_4.5Mbps_qvbr"
    }

You will need to update/ replace the templates. The name of the templates is stored as an environment variable for the input validate Lambda function.
Does that help?

closing as inactive


Recording rules groups

This PR creates a new go module within hack/tools.
Moves controller-tools from the root module into hack/tools so it's not part of the prod build anymore. (As a consequence we use go build instead of go run and then run the binary from the root folder, otherwise controller-gen complains with target folder being outside available modules, and same change is done for staticcheck).
Introduces promtool into the hack/tools module.
Adds a new make command promtool which check the syntax for the Prometheus recording rules we install including the rules expression. This is run by verify so CI will block on it.

xref https://issues.redhat.com/browse/HOSTEDCP-246

/hold cancel


Request: add Feather icons by Cole Bemis

There are two free Feather icon sets:
➖ Absent in Iconify: Feather icons by Cole Bemis (github)
✅ Present in Iconify: feathericon by Megumi Hano (github)
Similar names cause confusion, but the bigger problem is that Feather icons by Cole Bemis are much more popular and well-established, than feathericon by Megumi Hano.
So, I think, Iconify would become better when Feather icons by Cole Bemis are included.
If there's a special place to add such requests, I'll be happy to move it there.
Thank you in advance!

As explained by @cyberalien:

It is available: https://icon-sets.iconify.design/lucide/
Feather Icons have been abandoned for many years, Lucide is a fork that is maintained by community. It has all icons from original set + many new icons.


VRTnu subtitles no longer hardcoded, text subtitles aren't downloaded

Used latest v2018.12.03.
VRTnu has changed the way subtitles are shown, previously they were hardcoded into the video stream, now they are separate text files.
The youtube-dl --all-subs --skip-download --id "..." does not download them.
E.g. : https://www.vrt.be/vrtnu/a-z/er-was-eens/1/er-was-eens-a5-een-model-voor-de-natuur/

HLS VTT subtitles (used by vrtnu) are available in the youtube-dl fork yt-dlp : https://github.com/yt-dlp/yt-dlp/


Helm development version is set to appropriate version.

Set helm development version to:

0.0.0 if master is the current branch
< ver > if release-< ver > is the current branch
otherwise Chart.yaml version/appVersion fields are not changed

How can this be tested?
Execute make bundle command on the following branches:

master
release-9.99.999
feature/set-helm-version

Chart.yaml version/appVersion fields should be updated according to the requirements.
Checklist
- [ ] Unit tests have been updated/added

[x] PR is labeled accordingly

tested it and got the following
on master:              0.0.0 👍
on release-0.7:       0.7 ❓
on feature branch:  nothing happened 👍
shouldn't it be 0.7.0  on release-0.7 ?
also something we should tackle after this PR: if we have a 0.7.1 release we will not have a separate branch for that, but the Chart.yaml should be updated to 0.7.1, how should we go about that ?
@chrismuellner


documentation: point out the User Function is defined on the selected database

documentation change: point out that  AQL User Functions are defined on the selected database (not globally)

Thanks!


Add UI for new nudge notifications

see https://github.com/lbryio/internal-apis/pull/1608

Improve UX (drive directly to verification, presumably)

Duplicate at https://github.com/lbryio/lbry-desktop/issues/6021 with more info.


Potential changes for major revision

Hey, thank you so much for this awesome library - I've been looking for something like this for so long.
However, I've seen parts of the structure of the code that I think that they can be improved, the main point being that it was a bit hard to follow what is a component doing as it uses class with method overriding, etc.
For that, I forked this repo, added some of the potential changes I thought they would make sense, and implemented just a few components to check that everything works as expected:

App
Window
VerticalBox
Entry
Area

Area.Group
Area.Rectangle
Area.Bezier

I documented the reasoning and results in here: PotentialChanges - I could try to paste it down here, but it's some markdown format I think it makes more sense to use github's reader.
Please, don't take this as an attack of some sort, I think  it's all good work and I just wanted to point out things where I thought there was room for improvement. I'd be more than pleased to help if you want to transition some of those ideas into your project.

Hi,
This looks great and I agree that the old hierarchy is due for a change. It
worked better when there were fewer exceptions, but it increasingly turned
large. I'll look this over this weekend more thoroughly.
On Tue, Apr 16, 2019 at 4:18 AM Víctor Oliva<EMAIL_ADDRESS>wrote:

Hey, thank you so much awesome library - I've been looking for something
like this for so long.
However, I've seen parts of the structure of the code that I think that
they can be improved, the main point being that it was a bit hard to follow
what is a component doing as it uses class with method overriding, etc.
For that, I forked
https://github.com/voliva/proton-native/tree/rendererRework this repo,
added some of the potential changes I thought they would make sense, and
implemented just a few components to check that everything works as
expected:

App
Window
VerticalBox
Entry
Area

Area.Group
Area.Rectangle
Area.Bezier

I documented the reasoning in here: PotentialChanges
https://github.com/voliva/proton-native/blob/rendererRework/PotentialChanges.md
Please, don't take this as an attack of some sort, I think it's all good
work and I just wanted to point out things where I thought there was room
for improvement. I'd be more than pleased to help if you want to transition
some of those ideas into your project.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kusti8/proton-native/issues/211, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJW73P88kc5AdK9Z70iwsW4inUtaQef1ks5vhYdxgaJpZM4cxoy_
.

I'm currently working on a v2 with a major rewrite of the project. I want to revisit this when I get closer. I agree with your idea, I just want to clean up the code so it is easier to read.

I'm going to be posing questions as I get to parts. About the top-level App component:
I agree that having windows there but not being showed goes against react, so I want to eliminate that. However, I don't think that eliminating it as a React component all together is a good idea. The way I see it if you have multiple windows with separate Windows being applied to a single root App by registering them similar to ReactDOM, then you loose a lot of the communication between those windows AFAIK.
I want to be as similar to the react-native syntax as possible in this rewrite. Some things will inevitably have to change since apps don't support windows etc. React-native has a single place where you register your app component: AppRegistry.registerComponent(appName, () => App);. I'm thinking of keeping that single registry, having a top level App component and then have nested Windows that can be updated like normal components (meaning no windows that are in the tree, but not shown).

I've finished up most of the rewrite using a lot of your code and its much nicer. I can probably clean it up more later, but I'll track anything more in #213


core[minor]: Add from_env utility

Add a utility that can be used as a default factory
The goal will be to start migrating from of the pydantic models to use from_env as a default factory if possible.

from pydantic import Field, BaseModel
from langchain_core.utils import from_env

class Foo(BaseModel):
   name: str = Field(default_factory=from_env('HELLO'))

Force merged -- all unit tests and lints passed but CI was hanging


Fix hawkins sutton smoke tests, linting

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGES.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

Fixes some Hawkins-Sutton behaviour that fails under xarray v2024.5.0

Does this PR introduce a breaking change?
No, AFAIK.

coverage: 90.815% (-0.03%) from 90.843%
when pulling ca4822bccc1a0c1d2aa616c7067e59f3f313a476 on fix-hawkins-sutton
into edf1e13b7d47e4ead94506cdbb3bce1f977376f9 on main.

@huard
I've set the test to be ignored for this specific xarray version. The upstream dev version seems to be working fine, so I don't want to risk changing outputs for a bug in the current xarray.

coverage: 90.588% (-0.3%) from 90.843%
when pulling dc80f45d66e7e37bc5613df204adab20f944c9fb on fix-hawkins-sutton
into edf1e13b7d47e4ead94506cdbb3bce1f977376f9 on main.

coverage: 90.588% (-0.3%) from 90.843%
when pulling 26bf8f8f604a1ac5b4dac5e45158d758b1eacace on fix-hawkins-sutton
into edf1e13b7d47e4ead94506cdbb3bce1f977376f9 on main.


Git autodeploy some time does not trigger

We have now two slots : production and staging. Only the staging connected to git - bitbucket.
What we are trying to achieve is the following :
1 . push code to repo
2. the staging will automaticly detect a change and pull changes
3. it will stop tomcat server
4. pull the changes
5. rerun tomcat
6. when its ready do a swap
It seems that for 3,4,5 we just go for touch web.config. But for some reason not all of the time the wepbapp is aware of change in the repo. I cannot put my finger on what is the cause but sometimes it capture repo update and sometimes is not
Any ideas how this subject?
Thanks

If the deployment doesn't get triggered at all, is it fair to say that everything in your steps relating to tomcat and later swapping the slots (i.e. everything after step 2) is not relevant to the question?

I Pushed  21 minute ago to bitbucket

But in azure

Try looking at the WebHook history on the bitbucket side. It should show if it was sent. And on Kudu side look under D:\home\LogFiles\kudu\trace to see if there was any activity.

In bitbucket repo there is no WebHook installed but there are services related to azure. It should do post to the webapp

Services allow you to extend what Bitbucket does when the repository changes (for example, new code is pushed or a pull request is merged).

But they(bitbucket) do say that "webhooks will replace these services"

Yes, sorry, I meant Services. Though it seems they changed their UI and no longer show history.
Did you check the Kudu logs?

Just did, it weird. It does get the post - but it always one commit behind. If I do git pull in the kudu repo it fetches the last commit.
Found this between the logs. maybe can help

<step title="Incoming Request" date="2016-08-29T21:35:38.649" instance="...." url="/deploy?scmType=BitbucketGit" method="POST" ...... >
    <step title="Cleanup Xml Logs" date="2016-08-29T21:35:38.680" />
    <!-- duration: 48ms -->
    <step title="FetchHandler" date="2016-08-29T21:35:38.789" >
        <step title="Scm is not enabled, reject all requests." date="2016-08-29T21:35:38.946" />
        <!-- duration: 20ms -->
    </step>
    <!-- duration: 176ms -->
    <step title="Outgoing response" date="2016-08-29T21:35:38.980" type="response" statusCode="403" statusText="Forbidden" Server="Microsoft-IIS/8.0" Cache-Control="private" X-AspNet-Version="4.0.30319" />
    <!-- duration: 13ms -->
</step>
<!-- duration: 344ms -->

If I do git fetch --dry-run in kudu repo I see there is a fetch to be done

I tried to remove bitbucket and reset it. It does trigger on deploy. at least the staging slots is restarting. But it does not fetch the lastest version

And you're sure you're hooping up the right branch?
Can you share your web app name, either directly or indirectly? This will help us investigate. Thanks!

Sure, here is a dummy name :) http://dummya.azurewebsites.net/
and about the branch we have azure branch. Moreover and get fetch --dry-run in the repo shows there is a fetch available

There are two other sites there, including one with a slot. Is it the one with '3' in the name or the other?

The one that ends with 3

If you look at the Kudu trace under D:\home\LogFiles\kudu\trace, you can see the full sequence of git commands that Kudu executes. With some reformatting (and I tokenized the org/repo name to not reveal it):
git remote rm origin
git remote add -t azure origin 'git@bitbucket.org:{yourorg}/{yourrepo}.git'
git branch
git fetch origin --progress
git update-ref refs/heads/azure origin/azure
git checkout azure --force

I don't want to affect your Kudu repo, but can you try going to D:\home\site\repository and running these commands yourself? Does that result in the same issue? If so, fo you see anything wrong with the steps that would cause this?

sure, give me a mintue

The git branch should include the branch does not have any parameter?

No, that's ok. It's just a call we make to test what the current branch is. I could have left it off the sequence, but I just listed every git invocation that was made.

ok, I ran them. Those commands run ok. slow, but ok

By 'run ok', you mean that they got the git head to the right commit? If so, it's very puzzling, as it's exactly what we run when the bitbucket hook fires.

Yes, It gets the right commit. Now the site/repo folder is newer than the webapp folder in one commit

OK, I did some digging:

I deleted the logs at trace folder
I started our deployment process - We have our own built deployment process
I checked what happens with the logs

Here are the results:
In our deployment system, we always have an extra tag for quick rollback, we called it  last_development
The tag is being deleted and recreate each time a new deploy process is starting. The tags also being push to remote.
So here is what happend, for some reason the push of the remove/add tag triggers the deployment cycle at kudu, Since this is one heavy java/tomcat/hibernate app it takes it time to boot. Meanwhile  our development process has finished to run compile + run tests and it pushes the update to repo. This triggers another deployment cycle at kudu. But, this get reject (forbidden) I guess is due the fact that the system is still down
What do you think? What would pushing tags triggers the bitbucket services?

I see that bitbuckets has two options

use service wheneven there is a change in repo, this makes some kind of sense that pushing tags triggers the service
using webhook this allows more customization when to trigger the event

Seems like service will be deprecated in the near future, moreover it makes sense to use webhook only when the branch has changes and not on every repo change, doesn't it?

Still, I don't quite understand the behavior. What I would have expected is:

You push your tag
bb fires the hook
Kudu sees that it's not for the relevant branch, and makes it a no-op
You push the branch and normal things happen

But here, it seems that the tag push is triggering a full deployment, which I don't understand. We need to get to the bottom of that.

could you provide me with the git commands in your deployment process
Especially the ones interact with the remote repository in their exact order (ie)
git push origin
git push origin --tags

that will help me to reproduce the issue

@davidebbo
It seems like kudu does not understand this is not relevent for the brnach and trigger full deployment
@watashiSHUN, here are the list of remote commands

#delete last fast recovery tag
git tag -d last_development || true
git push --delete origin last_development || true
#create last_development tag for quick recovery
git tag  -a last_development -m "quick recovery tag"
git push --tags
#push current branch changes to remote
git push origin master

Note that the azure deployment cycle starts before the last command so the last command fails. To be sure, I switched the order and do moved all tags stuff to the end. This make things work as expected.

@oak-tree
as you mentioned above, "I switched the order and do moved all tags stuff to the end. This make things work as expected.", this just proves that kudu will not trigger development because of the tag deletion and recreation.
what happened here is that when kudu receives the notification about git push --delete origin last_development, it ran a series git command to check if there's an update in the gitbucket repo
git remote rm origin
git remote add -t azure origin 'git@bitbucket.org:{yourorg}/{yourrepo}.git'
git branch
git fetch origin --progress
git update-ref refs/heads/azure origin/azure
git checkout azure --force

In most cases, this should be a no-op, however, since we ran git push origin master right after git tag updates,the gitbucket repo will already have the tags updated and the newest commits while kudu is still processing the tag removal. This is why you see that kudu triggers the deployment right after it receives the first gitbucket request
while I am working on a fix, you can add a custom delay in your script as a workaround
::delete last fast recovery tag
git tag -d last_development
git push --delete origin last_development

::create last_development tag for quick recovery
git tag -a last_development -m "quick recovery tag"
git push --tags

:: this quick and dirty fix should keep your workflow unchanged
timeout 5

git push origin master

@oak-tree (I have updated my comment)
as you mentioned above, "I switched the order and do moved all tags stuff to the end. This make things work as expected.", this just proves that kudu will not trigger development because of the tag deletion and recreation.
what happened here is that when kudu receives the notification about git push --delete origin last_development, it ran a series git command to check if there's an update in the gitbucket repo
git remote rm origin
git remote add -t azure origin 'git@bitbucket.org:{yourorg}/{yourrepo}.git'
git branch
git fetch origin --progress
git update-ref refs/heads/azure origin/azure
git checkout azure --force

In most cases, this should be a no-op, however, since we ran git push origin master right after git tag updates,the gitbucket repo will already have the tags updated and the newest commits before kudu finish processing git fetch for the tag removal request. This is why you see that kudu triggers the deployment "prematurely"
However. this should not cause your website to be 1 commit behind... after you finished running your git script, can you go to kudu and check for the commit ID?
it should be under D:\home\site\deployments\ folder, where you would find a list of commit hashes, try to compare the most recent one against your gitbucket repository

@oak-tree
you are seeing one commit behind because of the auto swap
here is what happened

you ran your script of git commands
gitbucket sends to kudu notifications about tag delete, create, git push
kudu process it one by one
kudu fetch remote repo and realized that there are new commits from git push
kudu thus starts an azure deployment cycle
after the deployment, azure performs an auto swap
after the swap, changes from the latest deployment is moved to your production site and the staging site now looks like the production site before deployment(which is one commit behind)

step 4 and step 7 combined makes people think that "kudu performs a redeployment on tag delete"
Could you verify that after deployment, your production site is ahead of the staging site by one commit?
you can check for commit hash in D:\home\site\deployments\active file of their scm site

Hi, we were confused in a similar way in case of staging slot connected to github and autoswap to production slot which wasn't connected anywhere. From the azure portal, it's actually not possible to see (or at least I didn't find it out) what's currently deployed on production slot. The web app section "Deployments" is visible only on the staging slot, but not on production, where it offers us to configure deployment source:

I guess it would make sense in the Azure Portal, to split it into two sections:

One could be "Deployment Source" and it would allow the users to configure the deployment source or disconnect it.
The other one "Deployments" would show the deployments no matter if there is actually any deployment source connected at the moment.

As of now, e.g. if we have to see the deployment log, we have to go to KUDU, check the deployments JSON, find the the log url there and visit it. I understand that this request should be better suited in Azure User Voice, but wanted to share it here anyway.


Feature/twing extension

This PR introduces the following:

Adds an NPM package @studiometa/twig-toolkit which is a port of the Composer package in TypeScript based on Twing
Adds an NPM package @studiometa/storybook-twig to easily use the extension from @studiometa/twig-toolkit for component development
Remove the template/ folder and the @meta namespace (BC)

This feature is abandoned, we will use Storybook with its server framework.


AnimationTree does not apply Call Method Track for a method in an AudioStreamPlayer script.

Godot version:
3.3
OS/device including version:
Windows 10 Home, version 20H2, build 19042.928, HP Pavilion Gaming Laptop 16-a0xxx.
nVidia GeForce GTX 1660 Ti Max-Q, driver version 457.63, any backend.
Issue description:
Apologies, this is a weird one.
What happens: AnimationTree state machine does not apply method calls to an AudioStreamPlayer script in a Call Method Track in an Animation when another Animation exists on the same AnimationPlayer and which is alphabetically after the current animation and has an Audio Playback Track.
(yes, all of those details are important, see steps to reproduce below)
What I expected: The Call Method Track should get called regardless of other animations' Audio Playback Tracks and alphabetical status.
Steps to reproduce:
Gosh, where to begin...
Setup:

Create a new Node2D scene with a child Sprite node to animate (optional but useful for debugging).
Add AudioStreamPlayer node to the scene with some audio stream imported.
Attach a script to the AudioStreamPlayer with a simple method that prints a debug message or something.extends AudioStreamPlayer

func play_bloop():
    print("hi")
    play()

Add an AnimationPlayer node to the scene.
Create a simple animation named default which translates the Sprite (optional) and which has a Call Method track pointing to the AudioStreamPlayer. Optionally, make this looping.
Add a keyframe to the Call Method track which calls the AudioStreamPlayer's method which was added in step 3.

Create a second animation named zzz which has an Audio Playback track with a keyframe that plays a stream to the AudioStreamPlayer (edit: actually, the keyframe is optional, only the track itself needs to exist).

Add an AnimationTree node to the scene.
Point the AnimationTree to the AnimationPlayer, mark it as Active, and create a new state machine tree root.
In the tree root, add a single node to play the default animation.
Attach a script to the scene root node which starts the AnimationTree's state machine at the default node on ready:extends Node2D

func _ready():
    #$AnimationPlayer.play("default")
    $AnimationTree["parameters/playback"].start("default")

Actual bug reproduction:

Play the scene. Observe that the default animation is playing, but the method in the Call Method track is not being called.
Do any one of the following:

Change the root node script to play the animation via the AnimationPlayer directly, without using the AnimationTree.
Remove the Audio Playback track from the zzz animation.
Rename the zzz animation to aaa.

Play the scene. Observe that the method is now being called as expected.

Additional notes:

If there are multiple other animations which are similar to zzz, they must all be "fixed" in order to get the method calls to function.
This does not affect a Call Method track which targets a node other than the AudioStreamPlayer.
[screaming]

Minimal reproduction project:
AudioAnimationTreeTest.zip

Here is suggestion for AnimationTree improvement.
First of all, there are two solutions to the problem of duplicate paths being removed.

Generate a TrackCache for each animation
Append the TrackType to the path

The first one is not good from a performance point of view, because it expands the number of elements. So we should pick the latter.
Next, consider the original reason for being: to remove duplicate paths. Perhaps we want to prevent cases like the one in the image below.

In this case, it is not possible to determine which track should be preferred, so we need to validate. However, this should not be an implicit validation, but rather a UI warning when a track is added.
The most important thing to keep in mind is that we don't know if blending between things with different TrackTypes will work. A blend between a BezierTrack and a ValueTrack may work fine with some fixes, but there is no way to ensure that a blend between a ValueTrack and a MethodTrack will work.
As far as AudioTracks are concerned, forcing the blend to override the AudioStream's volume is also problematic - it may be useful for crossfading background music, but it's inconvenient when playing sound effects in OneShot. We think it should be selectable.
With all this in mind, I suggest that the blending issue should be resolved.


FIX Run TextCollectorTask as part of push instead of pull

Issue https://github.com/silverstripe/silverstripe-tx-translator/issues/11
After merging, create a 1.1 branch from 1 and tag 1.1.0

Updated


Consultation adding command

Closes #26

Codecov Report
Base: 65.33% // Head: 59.59% // Decreases project coverage by -5.73% :warning:

Coverage data is based on head (6687328) compared to base (fb60a8f).
Patch coverage: 30.46% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #56      +/-   ##
============================================
- Coverage     65.33%   59.59%   -5.74%
- Complexity      456      493      +37
============================================
  Files            87      101      +14
  Lines          1549     1849     +300
  Branches        170      215      +45
============================================
+ Hits           1012     1102      +90
- Misses          492      695     +203
- Partials         45       52       +7

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
68.18% <0.00%> (-3.25%)
:arrow_down:

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

...ain/java/seedu/address/logic/commands/Command.java
100.00% <ø> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <ø> (ø)

...java/seedu/address/logic/commands/EditCommand.java
97.01% <ø> (ø)

...ands/consultation/exceptions/CommandException.java
0.00% <0.00%> (ø)

...ss/logic/commands/tutorial/AddTutorialCommand.java
85.71% <ø> (ø)

...logic/commands/tutorial/DeleteTutorialCommand.java
100.00% <ø> (ø)

...commands/tutorial/exceptions/CommandException.java
100.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
75.00% <0.00%> (-3.95%)
:arrow_down:

... and 21 more

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

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


transaction.get() is undocumented and buggy

I can't see .get() documented here: https://googleapis.dev/nodejs/datastore/latest/Transaction.html
Like all annoying Node.js methods, even if you provide a single key to it, it returns an array with that entity
It seems to work if you provide an array of keys to it, but instead of returning an array of entities, it returns an array of an array that have the first entity in it's 0th slot (This is a bug, it shouldn't work at all)

Some bonus questions to whoever designed this entire API:

Why datastore.key(['Kind',key]) instead of datastore.key('Kind',key)
For the regular datastore.get (and this one too), providing a single key returns an array including that single element, can you sleep at night?

Coming from a Python NDB background, I really don't understand these decisions, even from a JS standpoint, they don't make sense

I mean, since transaction.get() doesn't accept an array of keys like datastore.get() - it should've at least returned a single bare entity, but it also returns a 1 Length array with the entity inside - I mean it just doesn't make any sense ...

I mean, since transaction.get() doesn't accept an array of keys like datastore.get() - it should've at least returned a single bare entity, but it also returns a 1 Length array with the entity inside - I mean it just doesn't make any sense ...

I can't see .get() documented here: https://googleapis.dev/nodejs/datastore/latest/Transaction.html

@AVaksman do you know how to get that method in there?

if you provide a single key to [transaction.get()], it returns an array with that entity
providing a single key returns an array including that single element
it should've at least returned a single bare entity, but it also returns a 1 Length array with the entity inside
It seems to work if you provide an array of keys to it, but instead of returning an array of entities, it returns an array of an array

@AVaksman Is promisifyAll forcing the array / is it possible to work around it?

transaction.get() doesn't accept an array of keys like datastore.get()

It seemed to accept two keys and return both matching entities in my testing:
const key = datastore.key(['Company', 'Google']);
const key2 = datastore.key(['Company', 'Google2']);

datastore.save({key, data: {url: 'www.google.com'}});
datastore.save({key: key2, data: {url: 'www.google.com'}});

const transaction = datastore.transaction();
await transaction.run();

const [entities] = await transaction.get([key, key2]);
console.log(entities[0]); // { url: 'www.google.com', [Symbol(KEY)]: [Key] }
console.log(entities[1]); // { url: 'www.google.com', [Symbol(KEY)]: [Key] }

Why datastore.key(['Kind',key]) instead of datastore.key('Kind',key)

We could probably add support for that style of usage. Are you interested in sending a PR?

I can't see .get() documented here: https://googleapis.dev/nodejs/datastore/latest/Transaction.html

@AVaksman do you know how to get that method in there?

if you provide a single key to [transaction.get()], it returns an array with that entity
providing a single key returns an array including that single element
it should've at least returned a single bare entity, but it also returns a 1 Length array with the entity inside
It seems to work if you provide an array of keys to it, but instead of returning an array of entities, it returns an array of an array

@AVaksman Is promisifyAll forcing the array / is it possible to work around it?

transaction.get() doesn't accept an array of keys like datastore.get()

It seemed to accept two keys and return both matching entities in my testing:
const key = datastore.key(['Company', 'Google']);
const key2 = datastore.key(['Company', 'Google2']);

datastore.save({key, data: {url: 'www.google.com'}});
datastore.save({key: key2, data: {url: 'www.google.com'}});

const transaction = datastore.transaction();
await transaction.run();

const [entities] = await transaction.get([key, key2]);
console.log(entities[0]); // { url: 'www.google.com', [Symbol(KEY)]: [Key] }
console.log(entities[1]); // { url: 'www.google.com', [Symbol(KEY)]: [Key] }

Why datastore.key(['Kind',key]) instead of datastore.key('Kind',key)

We could probably add support for that style of usage. Are you interested in sending a PR?


Direction of integration for cumint?

For aesthetic reasons I'd like to plot a barotropic streamfunction using an integral starting at the top rather than the bottom. (For an idealised southern ocean channel model.) Looking at the method-object-docs, I couldn't see a way to do this with cumint..? I briefly played with reindexing via strides but that feels un-xgcm like (and I didn't see how to make it work!)
Thanks for a handy package!

Hi @chrisb13, thanks for raising this issue.
I think this is related to #337 and earlier #184.
I agree this is a needed feature, but currently you would have to flip your values (not the coordinates see #605) apply the cumsum and flip them back (which is not ideal).
Would you be interested in working on this feature at all?


fix links to Julia's Twitter and Github profiles

The GitHub username was correct, but the link is a 404. On Twitter, "jlsmp" seems to be someone else.

Thanks @pdurbin!


Revert "Fix beginModalSheetForWindow warning." to fix #152

Reverts macvim-dev/macvim#80

What problem does this fix?


add api-gateway

Only and must have one lambda "service", can use api-gatway
TODO: custom domain & cert if needed

Code changes in this PR have been merged in PR #3 Am closing this PR


Synonyms Generator

Develop a Javascript Synonyms Generator app.
Description 📜
We have a JavaScript project titled Synonyms Generator. This issue aims to add a new project that allows users to find synonyms for words they input. The synonyms feature will enhance the usability of the application by providing users with richer language options.
Live Demo https://synonyms-ketan-sonar.vercel.app/
Requirements 🛠

Integrate a synonyms API or database to fetch synonyms for the input words.
Display the synonyms for the user's input word(s) in a user-friendly manner.
Implement error handling for cases where synonyms cannot be found.
Ensure that the synonyms feature works seamlessly with the existing project.

Bonuses ✨

Implement an autocomplete feature that suggests synonyms as the user types.
Allow users to click on a synonym to replace the original word in the input field.
Provide an option for users to toggle between synonyms and antonyms.
Implement caching to improve the speed and efficiency of synonym retrieval.
Add unit tests to ensure the reliability of the synonyms feature.

Project location 👉🏻 https://github.com/ketan-sonar/synonyms
👨🏻💻 Happy Coding 👩🏻💻

hey @ketan-sonar i would like to contribute to this issue can you assign it to me!!!

Hi @ketan-sonar, I am Parmesh Bhatt.
I hope you are fine.
Can you assign this issue to me?

Hi @Vidhanvyrs @ketan-sonar ,
Sure you can work on it, this issue is open to all, just make sure you are following proper directory structure mentioned in readme file.


arm-linux build blocker

currently blocked by
err: Package atk was not found in the pkg-config search path.

tauri requires atk >= 2.18 but it can not be found

tauri related


CSS excercises/Update solution to reflect desired outcome pic

Both the self-check section of the Readme and the solution css file did
not reflect the desired outcome picture. The .title-section has a bold
font in the picture but this is not mentioned in solution or Readme.
Update both files to reflect the desired outcome.

Complete the following REQUIRED checkboxes:

[X] I have thoroughly read and understand The Odin Project Contributing Guide
[X] The title of this PR follows the location of change: brief description of change format, e.g. 01-flex-center: Update self check

Complete the following checkbox ONLY IF it is applicable to your PR. You can complete it later if it is not currently applicable:

[X] I have ensured that the TOP solution files match the Desired Outcome image

1. Because:

Small fix/editorial, according contribution guide. Neither the self-check section in Readme.md nor the solution reflected the bold font that is visible in the desired outcome picture. This is fixed.
2. This PR:

Updated Readme with instruction that the .title section should be bold.
Updated solution.css with font-weight: bold in the appropriate class.

3. Additional Information:

Hi @thatblindgeye I think I resolved the conflicts.

I fear I messed this one up. I'm closing it and opening a new one.


Maximum database size on 32bit architectures

I am interested in using bolt on ARM devices.
https://github.com/boltdb/bolt/blob/master/bolt_arm.go as well as bolt_386 mentions a maximum mmap size of 256MB. Is the mmap size correlated to the maximum database size, or can these architectures have databases that are larger than 256MB?
Maybe this could be clarified in the readme Limitations section?

Bumped up to 2GB: https://github.com/boltdb/bolt/pull/281
I'll have to add a section in the docs separately.

2GB on 32 bit sounds perfect, thank you!

Are there any plans to allow larger sizes on 32-bit?
I would really like to switch over to using bolt for the metadata storage for btcd, but as you can imagine, the database size of Bitcoin can grow quite large.  This is especially true when you start to deal with optional address indices.  It will easily exceed 2GB which would make it unusable on 32-bit platforms.

I guess not, because bolt relies on memory mapped files just like MongoDB. Allowing a Database bigger than 2GB would require serious workarounds as this is the maximum size for mmaped files on 32-bit architectures. http://blog.mongodb.org/post/137788967/32-bit-limitations
The only option on your side would be to split the data into multiple databases.

@davecgh As @mbertschler mentioned, 32-bit architectures have a limited address space so we can't physically map the data file into that space. Bolt files mmap to a contiguous block of memory and on a 64-bit address space that's not a problem because it's a ridiculously huge space. For 32-bit machines you can likely get into other issues of memory fragmentation before you reach the full available address space.

Rather than make a new issue, I decided to bump this one.
Are there any plans to support >2GB database sizes on 32bit machines? Our users are starting to run into issues, and it's not an option for us to keep the database below 2GB anymore. Our alternative is to drop support for 32bit machines.
Thanks, and I understand if there's no plans to add support for this.

@DavidVorick I'd love to support >2GB databases on 32-bit systems but unfortunately since Bolt is mmap'd there's limited memory address space to use. It wouldn't be feasible to support larger databases without completely rewriting the database.


Gradle kills parent process unexpectedly

Current Behavior
Gradle kills parent process(i.e. cmd.exe) unexpectedly.
Expected Behavior
Gradle does not kill parent process.
Context (optional)
I guess this https://github.com/gradle/gradle/issues/7603 and related PR is the cause.
Steps to Reproduce
Let's say we have "sample.cmd" like:
echo "clean"
gradlew clean

echo "assemble"
gradlew assemble

echo "finish"

then open a terminal and run this.
If the task is successful, then the terminal should be end with "finish", however it doesn't. You might see: "Process finished with exit code 0".
actual output may like:
"clean"
BUILD SUCCESSFUL in 6s
1 actionable task: 1 executed

"assemble" task didn't run because gradle killed process tree, when it had finalized the task "clean".
expected output should like:
"clean"
BUILD SUCCESSFUL in 6s
1 actionable task: 1 executed

"assmble"
BUILD SUCCESSFUL in 6s
1 actionable task: 1 executed
"finish"

Gradle version
7.6, 7.6.4
Build scan URL (optional)
No response
Your Environment (optional)
Windows
Edition	Windows 10 Pro
Version	22H2
OS Build	19045.4046
Java
OpenJDK Runtime Environment Temurin-11.0.21+9 (build 11.0.21+9)
gradle
affected versions:

gradle-7.6-bin
gradle-7.6.4-all

Your issue appears to be a question about how to use Gradle. They should be asked in a different channel, so we will close it. Your question might be answered in Gradle documentation or on our community Slack or forums. Find links to these resources on help.gradle.org.
If you think our analysis is wrong, please provide us with more detailed information explaining why.

I think this is expected behavior with batch files. See, for example, this SO question. You should use call.

@ov7a
No, It's not typical behavior of shell script.
If you write same script above but in .sh(git bash or MINGW in windows), it will run to end.
echo "clean"
./gradlew clean

echo "assemble"
./gradlew assemble

echo "finish"

and its output may like:
clean
BUILD SUCCESSFUL in 7s
1 actionable task: 1 up-to-date

assemble
BUILD SUCCESSFUL in 9s
5 actionable tasks: 5 executed

finish

Once again, this is same but only difference is earlier one(.cmd) is recognized as windows environment at runtime while the other is recognized as *nix.

If you concern, this is not typical behavior of cmd also.
Let's say we have another .cmd like:
echo "this exists normally"
java --version

echo "this exists abnormally"
java -jar some_nonexist.jar

echo "end of .cmd"

and its output may like:
"this exists normally"
java --version
openjdk 11.0.21 2023-10-17
OpenJDK Runtime Environment Temurin-11.0.21+9 (build 11.0.21+9)
OpenJDK 64-Bit Server VM Temurin-11.0.21+9 (build 11.0.21+9, mixed mode)

java -jar some_nonexist.jar
Error: Unable to access jarfile some_nonexist.jar

"end of .cmd"

this file ends with "end of .cmd", because .cmd runs regardless of its child process's result.

I added example out to the origianl post for more clarity.

@finejustice

(ba)sh and bat are completely different in terms of implementation and behavior, this is expected.
The problem is related to including bat stript into bat script. So, the java example does not demonstrate the same situation. gradlew is a bat script.

@ov7a
sorry for late reply.
Again, it's not a problem of terminal type: it's a problem of gradle(w).bat.
In fact, gradle is a just java application(jar), so I can run it as a plain java application(e.g. java -jar gradle-launcher-7.6.jar clean, see the file "directly.cmd" in attachments).
if doing so, I do not experience the issue above.

However, If I use gradle(w).bat, the issue still occurs:

not for gradle(w):

I attached sample project and screenshots for clarification.
gradle-bug-28511_9dc067f055eb2bd340d4a66e4a5bf1e22dbd154f.zip

Again, it's not a problem of terminal type: it's a problem of gradle(w).bat.

I still think it's not related to gradlew.bat, but to the way bat files work. Did you try to use call as suggested in SO question I referenced?


Fix package name and bump version.

This fixes #1.
Might want to remove the misnamed package from PyPI, too.

@scopatz Ping. 🙂

Whoops Sorry! Don't know how I wasn't following this repo!

BTW, version numbers are bumped by rever: https://regro.github.io/rever-docs/

Should be all fixed up now!

Thank you!


No longer need to exclude language client from webpacking

It looks like we no longer need to be excluding the language client from webpacking: https://github.com/microsoft/vscode-docker/issues/2601#issuecomment-759718618
Both @dbaeumer and I tried it out webpacking with this PR in place and did not encounter any obvious issues. It also unblocks us from fixing https://github.com/microsoft/vscode-docker/issues/2601.
I'd like to hear from @StephenWeatherford first on why this exclusion was present originally; I don't know the history here.

If it works there's no reason to continue excluding.  There are various reasons for components having been excluded (usually you can tell by errors during the build, although sometimes things work anyway - frequently this is due to dynamically finding resource or source files in a way webpack can't deal with).  Given that vscode is also doing webpacking, it's quite possible and even likely that they intentionally fixed this component to be webpack-compatible.
Note that the cosmos extension was one of the most complex in that it creates two different output components from webpacking, so if you want to sanity check anything that might be the most logical (in particular mongo scrapbooks).

If it works there's no reason to continue excluding.  There are various reasons for components having been excluded (usually you can tell by errors during the build, although sometimes things work anyway - frequently this is due to dynamically finding resource or source files in a way webpack can't deal with).  Given that vscode is also doing webpacking, it's quite possible and even likely that they intentionally fixed this component to be webpack-compatible.
Note that the cosmos extension was one of the most complex in that it creates two different output components from webpacking, so if you want to sanity check anything that might be the most logical (in particular mongo scrapbooks).

Let's hope then! Nothing glaringly obvious broke; I tried out several scenarios with the webpacked build.

Let's hope then! Nothing glaringly obvious broke; I tried out several scenarios with the webpacked build.

Probably fine then.

Probably fine then.


/exporter/trace/zipkin fix example

The example used an incorrect import path: go.opencensus.io/trace/adaptor/zipkin
It should be: go.opencensus.io/exporter/trace/zipkin

I signed it!

Thanks much!


Svelte and Xaml ❓

Hello,
I've been interested in svelte for a while. I also found avalonia for desktop programming. As a result of my research, I realized that axaml is an extension that belongs to this repository. While asking myself why a different xaml engine is needed. I found out that xamlx engine compiles IL code on ast. The same logic is valid in svelte, the room generates pure javascript file through ast code.
I wonder if I can generate xaml using xamlx and svelte?
Svelte Compiler
Converting xaml using javascript is a silly idea, of course, but I'm asking because I'm curious.

I think the main problem would be the difference between XAML language and javascript component is that XAML is completely declarative. You can't have imperative code in a xaml file, it's just some instructions of how to assemble an object tree.
React, Angular, Vue, Svelte and others rely on having inline JavaScript. So such components can't be converted to XAML.
XAML is also supposed to be used with MVVM paradigm.

Thank you for the answer. It was enlightening for me.


Wire up more stuff in loader.rs

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

New commit adds support for structs.


Throttle Arlo API calls

Currently, we are hammering the Arlo api with every component making a call. This results in frequent warnings such as
2017-11-15 08:35:03 WARNING (MainThread) [homeassistant.setup] Setup of alarm_control_panel is taking over 10 seconds.
2017-11-15 08:35:03 WARNING (MainThread) [homeassistant.setup] Setup of camera is taking over 10 seconds.
2017-11-15 08:35:05 WARNING (MainThread) [homeassistant.components.alarm_control_panel] Setup of platform arlo is taking over 10 seconds.
2017-11-15 08:35:05 WARNING (MainThread) [homeassistant.components.camera] Setup of platform arlo is taking over 10 seconds.

Thanks Paulus for the fix!!
Related issue (if applicable):
https://community.home-assistant.io/t/insteon-hub-pro-help/31276/8
https://community.home-assistant.io/t/hassio-io-10-second-warnings/23714/11
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

NICE!


Billing Page iteration one

Pull Request Description
Tl;dr
Closes: enso-org/cloud-v2#1158
This PR adds a redirect to the stripe page where the user can manage his subscription
Demo Presentation

https://github.com/enso-org/enso/assets/61194245/360fdc7e-46ff-49fa-9936-e5c61fe6f917

Context:
Our first iteration was to add our billing page but after a few iterations, we decided to postpone it in favor of more important features.
This Change:

creates a link for a private user session
redirect the user to that page (open in a new tab) when the user clicks on the billing tab

Test Plan:
Go over how you plan to test it. Your test plan should be more thorough the riskier the change is. For major changes, I like to describe how I E2E tested it and will monitor the rollout.

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.

The related lambda is merged. This PR is ready for the QA. @somebody1234 , @PabloBuchu

@MrFlashAccount any way to show team plan when I hit update plan button? currently I can only buy multiple solo plans which doesnt make much sense

@PabloBuchu This can be configured in the Stripe settings: https://docs.stripe.com/products-prices/pricing-models#flat-rate


Not working with new "math.div" Sass method

I'm trying to use this with Sass's math.div method for division, but am getting
error: [plugin: postcss2] There is no module with the namespace "math"

Did you import it with @use?

Thanks, that worked.


Support ERC-20 deposits in x/rollup

Adds support for ERC-20 deposits by parsing the deposit tx data. ERC-20 tokens are minted on L2 in the erc20/{L1 ERC-20 address} format.
Safeguards are in place to ensure that the deposit tx originated from the L1CrossDomainMessenger so that users can't forge tx data to mint unlimited ERC-20 tokens.
x/rollup integration tests and e2e tests with ERC-20 deposits were added to verify the deposit was successful.
Closes https://github.com/polymerdao/monomer/issues/210

Force-pushed to rebase on main to get the docusaurus CI job passing

Force-pushed to rebase on main and resolve merge conflicts

Force-pushed to format commits for merge


Fix GroupBy.nunique not to update the internal state.

GroupBy.nunique should not update the internal state.

Actually I suspect the current behavior is a pandas' bug..

Codecov Report

Merging #1467 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1467   +/-   ##
=======================================
  Coverage   93.76%   93.76%
=======================================
  Files          36       36
  Lines        8405     8409    +4
=======================================
+ Hits         7881     7885    +4
  Misses        524      524

Impacted Files
Coverage Δ

databricks/koalas/groupby.py
88.63% <100.00%> (+0.08%)
:arrow_up:

Continue to review full report at Codecov.

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


Release js-lite to 2.0.0

This is an old PR and probably should have been merged a while back... Better late than never! 🙈

Merged! Re https://github.com/PostHog/posthog-js-lite/pull/57 - should we now push a release manually?

We should add a changelog for web too, it exists for the other 2 I think!
@mariusandra I think that got included in 2.0.0 since this PR was merged after.


Quote matrix versions (because YAML is ambiguous)

YAML parses these versions as floating point values, and then reformats 11.0 as 11 in string interpolation. 😑

Thank you 👍


Question on Supervised Dim Reduction

I am working on a project where I have 100D navigation vectors of Wikipedia articles and I plan to cluster these vectors and then visualize the clusters. I wanted to use labels acquired from clustering in the process of reducing dimensions with UMAP.
But I have no idea if it it works or not. So my questions are:

Is there anyway to have any idea about the magnitude of the loss?
Is there anyway to see the definition of the loss function either in code or mathematically so I
can see how it utilizes the class labels?
If we cannot view the loss how would one approach parameter tuning? Right now I gauge the quality of my visualization by measuring trustworithness and checking it by eye.

Is the loss function listed here?
Kindly awaiting your answers! Thank you

The loss is given in the piece you link, the question is more "loss in terms of what?". If you want it in terms of input variables then you are going to get a very complex expression which is not at all edifying (partly why such things aren't really written down). Computing the loss itself is actually O(N^2) which doesn't scale well, and hence it is usually never actually fully computed. In principle you can compute the loss by looking at the graph_ attribute of the umap object after fitting and constructing the relevant equivalent matrix from the pairwise distance matrix of the embedding (if d is you distance matrix then you want 1 / (a + d ** (2*b))) where a and b are the a_ and b_ attributes of the umap object. The loss in the link then applies with the sum being over all matrix entries.

Thank you for the prompt response, I have one more question. In the loss shown how does it account for the labels point might have? At the high level I understand it tries to balance the weighting of edges in the higher dimensional space with that in the low dimensional space but how does it account for labels?

What is the range of the loss? I get a huge number. This is how I get the loss
def get_loss(umap_fit):
    dist_mat = umap_fit.graph_
    dist_mat = dist_mat.toarray()
    a = umap_fit._a
    b = umap_fit._b
    smooth_fam = 1.0 / (a + dist_mat ** (2*b))
    return np.sum(smooth_fam)

I got
Loss: 7254961.0397050595

I don' believe your loss function looks quite right. Regardless the loss value is simply a number and is entirely dataset dependent. There is no apriori "good" range or answer (other than smaller is better).


tests: Add test for Secret::from.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 93.201%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
8348

Relevant Lines:
8957

💛  - Coveralls


[Bug]: Documentation on dockerhub doesn't mention WHERE to place configuration YAML file

Describe the bug
The documentation  on docker hub states the following:
Just like opensearch.yml, you can pass a custom opensearch_dashboards.yml to the container in the Docker Compose file.

But doesn't say where this file should be mounted or what it's contents can be.
To reproduce
visit docker hub site
Expected behavior
No response
Screenshots
If applicable, add screenshots to help explain your problem.
Host / Environment
No response
Additional context
No response
Relevant log output
No response

[Triage] Thanks for your input. We will be working to add more documentation to it.


Improve "missing final expression" error

In long traces, the explanation or tip can go unnoticed:
── MISSING FINAL EXPRESSION ───────────────────────────────────────── main.roc ─

I am partway through parsing a definition, but I got stuck here:

53│                  subItemWalker =
54│                      List.walk subsItems {invalidEncounters: Bool} \walkerState, walkerItem ->
55│                          if walkerState.invalidEncounters == Bool.true then
56│                              {invalidEncounters: Bool.true}
57│                          else
58│                              itemSplit = Str.split walkerItem " "
59│                              countStrRaw = List.get itemSplit 0
60│                              countStr =
61│                                  when countStrRaw is
62│                                      Ok val -> val
63│                                      Err OutOfBounds -> crash "Nope, doesn't work"
64│                              count =
65│                                  when Str.toU32 countStr is
66│                                      Ok val -> val
67│                                      Err InvalidNumStr -> crash "Nope, doesn't work"
68│                              # at this point, we got the count of cubes for this specific color.
69│                              colorStrRaw = List.get itemSplit 1
70│                              colorStr =
71│                                  when colorStrRaw is
72│                                      Ok val -> val
73│                                      Err OutOfBounds -> crash "Nope, doesn't work"
74│                              # we also got the color name.
75│                              when colorStr is
76│                                  "red" ->
77│                                      if count > 12 then
78│                                          { invalidEncounters: Bool.true }
79│                                      else
80│                                          { invalidEncounters: Bool.false }
81│                                  "green" ->
82│                                      if count > 13 then
83│                                          { invalidEncounters: Bool.true }
84│                                      else
85│                                          { invalidEncounters: Bool.false }
86│                                  "blue" ->
87│                                      if count > 14 then
88│                                          { invalidEncounters: Bool.true }
89│                                      else
90│                                          { invalidEncounters: Bool.false }
91│                                  _ -> { invalidEncounters: Bool.false }
                                                                           ^

This definition is missing a final expression. A nested definition
must be followed by either another definition, or an expression

    x = 4
    y = 2

    x + y

I would use alloc.tip for the "This definition ..." section.

[ ] Further "missing final expression" errors in this thread should be analyzed for improvement as well.


Header not parsing correctly

I'm encountering a bug in this GOTV demo http://www.hltv.org/match/2300904-dollarhouse-rize-nordic-masters-season-2-finals
Header.PlaybackFrames, Header.PlaybackTime, Header.PlaybackTicks, Header.SignonLength are all 0.  I tried the dust2 and mirage matches and both had this bug.  I also tried another matchup from the same tournament, but didn't encounter the problem, so I don't know how widespread it is.

Found some other matches from the same tournament with the same problem, so it's not just an isolated incident.

But you can play demos in cs go ?
And in the demoui in the game, which information get you for the number of ticks , frames, and duration ?

Yes, i can play the demo in csgo and everything looks normal on demoui.  Ticks and time duration is displayed properly.

And for this demo in particular how much time and ticks?

For the Dust2 demo:
Ticks -  507904
Time - 1:06...
I did notice something else though that might be helpful.  Right after selecting the demo there's a splash screen other demos seem to display the total ticks and time immediately, but this demo doesn't display them properly until the demo is fully loaded.

Has anybody made any progress with this problem? I've seen it now myself "in the wild", and it's pretty annoying.

I did not had time to search about it but if they return 0 for ticks, duration,  etc these demos seems to be corrupted.
Maybe make a binary analyses the files to verify the 0 values.
In this case:
On old version of the game engine, corrupted demos can't be read. Maybe recent engine extrapolates these values directly from demo's datas (last in-game tick,  etc)

https://github.com/SteamRE/SteamKit/blob/master/Resources/Protobufs/csgo/netmessages.proto#L245
It's open for grabs now, whoever wants to parse the Server-Message is open to do so :)

The bitstream is still a bit magic to me, but I was under the impression these types of messages don't get parsed?

@econoraptor We do parse netmessages for several important purposes. We used to use protobuf-net for them, but this turned out to be quite slow - so as of #26 some additional boilerplate code is unfortunately required. I'd recommend using e.g. NETTick (proto - impl) as a template since the general concept is very simple and repetitive.
As you can see, bitstream interaction is limited to just ReadProtobufString() and ReadProtobufVarInt() which are straightforward and do exactly what it says on the tin (and ChunkFinished but if you just copy the basic structure, you don't even need to think about it - unless you want to, of course).

@econoraptor That being said, there's obviously the alternative of just using the old SLOW_PROTOBUF mode which is still available (for debug purposes mostly) just like #65 does. You wouldn't have to implement a FastNetmessage then, but do consider that most DemoInfo users would not benefit from this then as our NuGet releases don't enable SLOW_PROTOBUF for performance reasons.
Whatever you choose, we would be happy about any contribution!

Ok, I'll take a look and see if I can figure out how to implement it.

Right at the start when I try to read the key I'm getting a NotSupportedException from this condition in ReadProtobufVarIntStub: if ((count == 9) ? (b != 1) : ((b & 0x7F) != 0x7F)) .
The problem the error is raised on 'var desc = bitstream.ReadProtobufVarInt();'
http://pastebin.com/vF9bc0fX

Alright, I found two potential issues:

The CRC fields are fixed32 instead of int32, so they're not encoded as varints. Just use ReadInt() instead.
The ugc id field is 64 bits long. So far, we've only seen 32-bit fields used in csgo, so 64 bits are not yet implemented. I can have a look at this if you don't want to mess with protobuf internals.

Changing "(uint)bitstream.ReadProtobufVarInt();" to "bitstream.ReadInt(32);" as you suggested seems to have gotten things running.  In the language guide (https://developers.google.com/protocol-buffers/docs/proto) it shows for c++ that fixed32 maps to uint32, so I figure it's probably the best bet for C#.  I don't think I'll be messing around with trying to implement 64 bits, so I'll probably just comment the field out.  It doesn't seem all that important anyways.
I still need to tie in tick_interval to the parser, but it's bedtime for me, so I'll finish up tomorrow.


Issue/4738 user management (#5638)

Pull request opened by the merge tool on behalf of #5638

Merged in<PHONE_NUMBER>146260978e871fba3828800de10120


v1.9: Revert broken tests

This reverts the commits that introduced by the premature merge of #16224.
Draft for now to kick off the tests.

test-backport-1.9

Oops, pushed to wrong repo. Closing.


Samsung tablet crash

Started map view with focus to Berlin, zoom 0, zoomed in up to zoom 15, then back; closed activity, started again, repeated 5-6 times. Activity crashed with attached log.
samsung_tablet_crash.txt

Unable to reproduce this with the latest SDK version, thus closing this for now. This could have also been related to previous Galaxy Tab S2 firmware version, as we have have updated the device now to Android 6.0.1


[Todoist] ...

Extension
https://www.raycast.com/doist/todoist
Description
Menu Bar command completely failing on load.
Steps To Reproduce
Occurred on start up of Raycast with the command in the Menu bar.
Current Behaviour
Error: Worker terminated due to reaching memory limit: JS heap out of memory

NodeError:errors:393:5
[kOnExit]:worker:277:26
Worker.<computed>.onexit:worker:199:20

Expected Behaviour
No response

Thanks for bringing this up. It appears that this is a duplicate of issue https://github.com/raycast/extensions/issues/6562, so I'll mark it as such.


[DOCUMENTATION][KAFKA] fix typo in kafka/OffsetRange.scala

this is to fix some typo in external/kafka/src/main/scala/org/apache/spark/streaming/kafka/OffsetRange.scala

Can one of the admins verify this patch?

@microwishing can you have a look at the rest of the doc around this for any other typos? even small PRs have nonzero overhead, so I'm encouraging people to not just open a PR for single typos as they see them, but take a little time, if they're bothering, to evaluate similar issues around the change. Maybe there aren't any.

@srowen thanks for your tips. I happened to work on some kafka project recently so I went through source codes including comments for spark kafka streaming part today.
I spent some time setting up the spark dev environment and feel good so far due to rich doc. Those are indeed trivial typos.  I can make check-ins next time after fixing some real bugs there.

Test build #2163 has started for PR 10121 at commit 97dfb8b.

Test build #2163 has finished for PR 10121 at commit 97dfb8b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master/1.6


Clean up ae-to-chat messages when done with combat

I noticed that a fairly large part of my chat db was ae-to-chat messages, so I figured it might be a good idea to clean them up once irrelevant. I.e. hooking on createCombat or deleteCombat. I think createCombat is slightly better as an ae-to-chat message might possibly be relevant right after a combat, but isn't likely to be relevant once the combat ends.
So, something like this perhaps? Not too fond of filtering on 'ae-to-chat header', but I couldn't think of anything better.
Also, it's neither well tested nor particularly efficient, thus an issue rather than a PR. :)
Hooks.on("createCombat", function (data, delta) {
	game.messages.entities.filter(x => x.data.content.includes("ae-to-chat header")).map(y => y.id).forEach((async z=>ui.chat.deleteMessage(z, {})));
});

Good idea!
It's going to be a couple weeks until I can get to it (as I've got one more
0.8 module update to do, and things going on the next couple weekends).
(Just my own notes here for when I get back to this)
My thinking is to throw a flag on the chat messages as they're created, and
then filter on that flag, which should be a slightly neater solution and
won't break if I change the template in future.
Setting the flag to the combat ID for the start of turn messages would
allow only deleting the ones for a particular combat at the end of that
combat, which might be handy if someone had multiple combats running at
once for some reason. Don't think I can do the same with the "on
application" messages in a neat way (as it's hard to associate them with a
particular combat, and for linked tokens they might not be for the token
instance that's actually in the combat due to some shortcuts I'm taking
elsewhere).
Maybe also an option to do it every new round instead, but that depends on
the combat handing library that I'm vaguely planning as a future project.
On Tue, 25 May 2021, 10:13 Jonas Karlsson, @.***> wrote:

I noticed that a fairly large part of my chat db was ae-to-chat messages,
so I figured it might be a good idea to clean them up once irrelevant. I.e.
hooking on createCombat or deleteCombat. I think createCombat is
slightly better as an ae-to-chat message might possibly be relevant right
after a combat, but isn't likely to be relevant once the combat ends.
So, something like this perhaps? Not too fond of filtering on 'ae-to-chat
header', but I couldn't think of anything better.
Also, it's neither well tested nor particularly efficient, thus an issue
rather than a PR. :)
Hooks.on("createCombat", function (data, delta) {
game.messages.entities.filter(x => x.data.content.includes("ae-to-chat header")).map(y => y.id).forEach((async z=>ui.chat.deleteMessage(z, {})));
});
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BadIdeasBureau/ae-to-chat/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIVM42BBHWTAZ3IOZUG3EVLTPNS2HANCNFSM45O4XWGA
.


error when displaying process instances

I could see my processes but i cant diplay the proess instances in cockpit, i got the error "Communication Error : The application received an unexpected 400 response from the server. Try to refresh the page or login and out of the application" instead.
Is there a way to know the cause, i'm working in karaf container

Hey @elkhadirzyad
well, as I mentioned before, there is no code in Cockpit that makes it OSGi aware. According to the error message I suppose Cockpit tries to use the REST API. Is this intentional? And does your process engine in Karaf expose itself via REST?
PS. is #63 fixed for you?

ok , i dont know how to fix it, #63 is fixed.

also i have the error Could not fetch the resource for Could not fetch the resource for 'Open tasks' : Error: Conflicting setter definitions for property "config": com.fasterxml.jackson.databind.ObjectMapper#setConfig(1 params) vs com.fasterxml.jackson.databind.ObjectMapper#setConfig(1 params) at [Source: (org.camunda.bpm.engine.rest.filter.EmptyBodyFilter$1$1); line: 1, column: 1]
in human tasks

Hey @elkhadirzyad
this doesn't look very OSGi specific. Have you tried asking in the camunda forums?


v0 TODO's

[x] Get PR link into review python func
[x] Get .patch into string in python file
[x] OpenAI API take input
[x] figure out good prompt
[ ] output review
[ ] only works if you're collaborator

Me too

try

issue

ok


Fix logging and a bug in instance config.

The instance config should reuse the instance config file if it exists.
The logger should not propagate message to the system logger.
Remove duplicate logging to console from a running daemon.
Add a debug mode for executing scripts and daemons.

More debug logging can be added as appropriate.

LGTM


Update to restic 0.12.1

Pls upgrade restic version.
Pls tagged version. Latest is not sufficient.

Since updating restic is pretty easy and I'm not following any regular release cycle, I suggest to fork the repo and build with your required restic version.
If someone like to help with maintaining this repo feel free to contact me.

Update the restic container with the following command in Terminal/Powershell connecting over ssh:
$ docker exec -ti resticcontainer restic self-update

Since updating restic is pretty easy and I'm not following any regular release cycle, I suggest to fork the repo and build with your required restic version.
If someone like to help with maintaining this repo feel free to contact me.

Hey @niondir! First, thank you for this super useful image :)
I have been "maintaining" (meaning adding new versions as soon as they are released, nothing more) in this fork, and I could help with this repo maintenance if you want.

@vfauth I'm pretty bad at following the public repos besides the daily work. Help with Maintenance is very welcome.
If you are still interested I would add you as maintainer. Just give me a shot notice.
Sorry for the very late reply.

@niondir Yes, I would still be interested in helping maintaining this repo.

Great, I added you as maintainer.

I updated to the latest. mailx in alpine made some issues and is now called "mail". Maybe someone who uses the feature can test it. Automated tests are still an open issue.


[10.x] Ensure headers are only attached to illuminate responses

Backport https://github.com/laravel/framework/pull/52789

Static analysis failure is unrelated

@timacdonald thank you!


Hats for crabs, penguins, skellys and slimes

About the PR
The people have requested hats, so with the guidance of Old Dance Jacket, I've added hats to several spawnable species that lacks them.
I want to do Ian and Xenos, but Rane has cited engine issues preventing that.
Changelog
:cl:
-Hats added for penguins and crab AI's. These are strippable by players.
-Hats added for skeleton and slime people.

All 3 inventory templates seem basically the same; can you compress them into 1?


Get &[u8] or Vec.

I want to get back &[u8] or Vec containing the .exe binary code from pe and then save that to a file..
let file = FileMap::open("./name.exe").unwrap();
let mut pe = PeFile::from_bytes(file.as_ref()).unwrap();

I tried:
pe.image()

But it produces a value 900 +- bytes larger than the original.
Help me, please.
Thank you in advance.

And how can I edit the certificate, versions, and resources?

This library is not designed to let you edit the file conveniently.
To get the original bytes, just go back to the original file:
let file = FileMap::open("./name.exe").unwrap();
let mut pe = PeFile::from_bytes(file.as_ref()).unwrap();
// Do things with pe...
let bytes = file.as_ref();

PeFile and friends just lightly wrap a &[u8], you are not meant to store these types long term (recreate them if necessary). Simply access the underlying storage.
FileMap is a readonly view over the file, so if you wish to modify the bytes you have to use std::fs::read to read the contents into a Vec then process that. After you have done your processing, collect all the edits you wish to make and apply them after the wrappers (like PeFile) are no longer used then save the resulting Vec back to disk with std::fs::write.


https://gitlab.kitware.com/f3d/f3d/-/issues/242

--no-background does not work, either because of a VTK regression or an issue in our code.
It works in 1.0, but not in 1.1

https://gitlab.kitware.com/f3d/f3d/-/issues/242


Cors headers only coming on certain routes

I've setup the package using version 0.7.x on laravel 5.1.x.
I'm also using the jwt-auth package as middleware.
I'm setting up a RESTful API, so I'm just using resource routes and returning JSON.
Interesting enough, CORS seems to work on my POST routes but not GET routes. Using Postman to examine my headers, this seems to be the case for the controller methods I've tested.
Here's my routes.php and cors.php:
https://gist.github.com/GrahamWood/f282f0907c4709106f4d
Any idea where I should start looking to solve this?

@GrahamWood I think you logged this issue to the wrong library; this cors library is for node.js, not PHP


Enable abi-encode to work with structs / tuples

Component
Cast
Describe the feature you would like
If I have a function that takes a struct as input:
struct animal {
  string name
  uint256 id
}

function getAnimal(animal myAnimal) public {

And I want to encode my arguments:
cast abi-encode "getAnimal((string,uint256))" bobby 123

I get an error:
Error:
Failed to parse tokens

Context:
- Invalid data

Additional context
When you get the function signature of a function that takes a struct as a type, the function signature is generated from adding a second set of parenthesis around the parameters.
This could also just be a documentation update where we add this as an example.

cast abi-encode "getAnimal((string,uint256))" "(bobby,123)"
should work.
def need another example for this

I'm idiot. Thanks. I'll make a docs PR.


[Update Package] Microsoft/DotNet/SDK/3_1/3.1.421 new CDN urls

[Update Package] Microsoft/DotNet/SDK/3_1/3.1.421 new CDN urls
( #202037 )
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


 After this action the whole OBJ is processed.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15382/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15382/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15382/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15382/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15382/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15382/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/testResults/webgpuplaywright/index.html

I don't think this is the right approach. I think the right way to display lines is to change the draw mode (fillMode in Babylon materials) to a line. Using wireframe to display lines is very strange.
Obj allows lines and triangles to coexist on a single mesh, so ideally, we should create submeshes with a MultiMaterial to do this properly. Here is an example of what I mean: https://playground.babylonjs.com/#5I0VSE#3.
This is obviously a much bigger change though.

Using wireframe to display lines is very strange.

Yes it is. I agree.
This is just a quick fix for the parser. There are several other issues with the OBJ loader. These issues can be adressed in the future, but let's quickly stop the bleeding from this wound at least.

why not simply converting them to degenerate triangle for now? So we don't have to use wireframe

The whole algorithm of parsing the obj elements and builidng the mesh is tightly coupled to the wireframe material. The parser has many flaws and should be rewritten from scratch to correctly handle non-existig referred elements such as uvs, normals. I'll gladly contribute and create a more robust parser but I can start to work on it only in September. That's why I created this quick fix which adresses the issue from the forum for now.

Works for me!

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15382/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Section header only getting set at the top (CursorAdapter implementation)

Problem - The section header is there and it works, but it is always at the top, so if a new list item comes on the screen that should have a different header - the header on the top gets changed instead of a new header getting placed in between
I am using this library with CursorAdapter (extends CursorAdapter implements StickyListHeadersAdapter) to section my notes into different periods (Today, Yesterday, Last Week, Earlier)
Following is my implementation of the methods (let me know if you require anything else along with this)
    @Override
    public View getHeaderView(int position, View convertView, ViewGroup parent) {

        HeaderViewHolder holder;
        if (convertView != null)
            holder = (HeaderViewHolder) convertView.getTag();
        else{
            convertView = mInflater.inflate(R.layout.notes_row_header, parent, false);
            holder = new HeaderViewHolder(convertView);
            convertView.setTag(holder);
        }

        String newDateString = getCursor().getString(getCursor().getColumnIndex(NotesEntry.COLUMN_LASTEDIT_DATETEXT));
        //get the new date and format it accordingly
        DateTimeFormatter formatter = DateTimeFormat.forPattern("yyyy-MM-dd");
        DateTime newDate = formatter.parseDateTime(newDateString);
        int diffSinceMonday = UtilityFunctions.getDateDiff(newDate.withDayOfWeek(1));
        int diffDates = UtilityFunctions.getDateDiff(newDate);
        //set appropriate code
        if(diffDates == 0)
            dayDiff = HEAD_TODAY;
        else if(diffDates == 1)
            dayDiff = HEAD_YESTERDAY;
        else if(UtilityFunctions.isBetween(diffDates, 1, diffSinceMonday))
            dayDiff = HEAD_THIS_WEEK;
        else dayDiff  = HEAD_EARLIER;
        Log.v("IN_ getHeaderView", "ID IS "+ dayDiff + " " + newDateString);

        String headerText= "" + HEADER_TEXT[dayDiff];
        Log.v("IN_ getHeaderView", "HEADER IS "+ HEADER_TEXT[dayDiff] + " " + dayDiff);
        holder.headerText.setText(headerText);
        return convertView;
    }

    @Override
    public long getHeaderId(int position) {

        String newDateString = getCursor().getString(getCursor().getColumnIndex(NotesEntry.COLUMN_LASTEDIT_DATETEXT));
        //get the new date and format it accordingly
        DateTimeFormatter formatter = DateTimeFormat.forPattern("yyyy-MM-dd");
        DateTime newDate = formatter.parseDateTime(newDateString);
        int diffSinceMonday = UtilityFunctions.getDateDiff(newDate.withDayOfWeek(1));
        int diffDates = UtilityFunctions.getDateDiff(newDate);
        //set appropriate code
        if(diffDates == 0)
            dayDiff = HEAD_TODAY;
        else if(diffDates == 1)
            dayDiff = HEAD_YESTERDAY;
        else if(UtilityFunctions.isBetween(diffDates, 1, diffSinceMonday))
            dayDiff = HEAD_THIS_WEEK;
        else dayDiff  = HEAD_EARLIER;
        Log.v("IN_ getHeaderId", "ID IS "+ dayDiff + " " + newDateString);
        return dayDiff;
    }

I wanted to know if I have to somehow initialize the header in newView or bindVIew implementations.
I am also aware that having the date calculation part in getHeaderView and getHeaderId might be redundant, I have kept them at both places as I am not sure what would work (it definitely didn't work when it was just in getHeaderId).
If anyone has an example of cursor adapter implementation of this library I would appreciate if I got a chance to review it.
Please disregard the log messages - they were just for testing.

I know this is very late but did you end up finding a solution? I'm facing the same problem when trying to use a Cursor Adapter that implements StickyListHeadersAdapter

Unfortunately no, I ended up using Recycler view in conjunction with some
other sticky header lib.
On Tue, Jul 28, 2015 at 6:46 AM, oseparovic<EMAIL_ADDRESS>wrote:

I know this is very late but did you end up finding a solution? I'm facing
the same problem when trying to use a Cursor Adapter that implements
StickyListHeadersAdapter
—
Reply to this email directly or view it on GitHub
https://github.com/emilsjolander/StickyListHeaders/issues/394#issuecomment-125399595
.

Do you have any contact info I can reach you at? I'd like to discuss your solution if possible. There's no contact info available on your github profile.

My email ID should now be visible on my profile.
On Tue, Jul 28, 2015 at 7:14 AM, oseparovic<EMAIL_ADDRESS>wrote:

Do you have any contact info I can reach you at? I'd like to discuss your
solution if possible. There's no contact info available on your github
profile.
—
Reply to this email directly or view it on GitHub
https://github.com/emilsjolander/StickyListHeaders/issues/394#issuecomment-125402877
.

Try to put this code in the first line of getHeaderView() and getHeaderId()
mCursor.moveToPosition(position);

Can confirm that @gersonmendes answer solved it


chore: add new option Server_ssl to doc and changelog

Coverage remained the same at 87.573% when pulling 19983ced400e28b5286707feeb0fbc4ecbbd973a on chore/add_doc into a2f7412d374eed7adf18304f4b3732fa43f4f93d on master.


updated label + added UMLS mapping

Fixes #4344
Note: I did not add the label as synonym as I couldn't find evidence that this name is used.

I think this must have come from OMIM, they cite Kuhn on the page:
https://omim.org/entry/115000
We don't want to delete old labels, can you make it an exact synonym and mark it as A synonym that is historic and discouraged (has synonym type -> synonym type property