Rails 5

Hello! We upstreamed this change into Rails 5 in https://github.com/rails/rails/pull/17573. It would be nice if you can help review it and see if anything is missing. (And we could use some help with the documentation side of things if you have time.)
If it looks good to you, would you consider wrapping the this gem with a Rails version check? You can skip inserting the middleware for Rails >= 5 and emit a warning ("please remove this from your gemfile...") instead.

Thanks. Commenting for future Rails 4.2 users on CloudFlare because CF (silently) automatically convert strong ETags to weak ETags by prepending W/ in the header. Googling W/ is pretty ineffective and CloudFlare doesn't document this behaviour so hopefully this comment helps visibility. TLDR use Rails 5 or this gem.


HDDS-2992. Handle existing volume/bucket in contract tests

What changes were proposed in this pull request?
Make at most 5 attempts to create randomly named volume/bucket in integration tests if creation fails due to conflict (volume/bucket already exists).
https://issues.apache.org/jira/browse/HDDS-2992
How was this patch tested?
Faked random "volume already exists" and "bucket already exists" events, verified that creation is attempted at most 5 times.
https://github.com/adoroszlai/hadoop-ozone/runs/436426070

if (i == attempts) {
   LOG.info("$$$$$ JACKPOT $$$$$");
}

;-)

Thanks @elek for reviewing and committing it.


[Feature] Windowed aggregates

Description
Aggregates can be formed either over the current bucket (hour, day) or the entire lifetime of the underlying timeseries. In some cases, it would also be useful to aggregate over a limited window, e.g., the last 6 hours. That could look something like
  recent_max_price: BigDecimal! @aggregate(fn: "max", arg: "price", window: 6)

The current ways to aggregate would be expressed as window: 1 (default) and window: "all" (lifetime/cumulative)
There's a bunch of things that aren't entirely clear:

Is it enough if the window is always an integer multiple of the bucket? Or should we offer expressing arbitrary times there, like window: "90 minutes"
How would users express that a windowed aggregate should be done for one interval but not the others, e.g., if the above aggregation is done both hourly and daily, how do you express "only add this to the hourly aggregates"

@lutter do you imagine this being updated on every block, or on the hour?

@azf20 It would still only be updated on the hour/day, but would consider data reaching farther back than just that interval


Merge only for already existing packages?

I'm considering using sbommerge as a way to add information to the packages in an SBOM, rather than actually merging two lists of packages. The idea is the following:

I generate one SBOM from the actual source code, that gives me a list of packages that actually are in use
That SBOM is lacking some information about the packages. For example, none of the packages have a LicenseConcluded information.
On the other hand, I have another SBOM file with that missing information.
So the process would be: if the package appears only in the first SBOM file, just output it in the result. If the package appears in both files, merge the information. If the package appears only in the second SBOM file, ignore it.

Maybe this could be seen as enriching the SBOM with another SBOM.
Does that make sense?
It seems to me that sbommerge doesn't have that feature. Did I miss it?
Are there tools that already do exactly that?
A quick look at the code tells me that this shouldn't be difficult for me to implement and open a PR. Would you be interested in that?

@nodet Please feel free to suggest some code improvements to the tool.
The logic should be if the package is only in one of the SBOMs, it is copied across to the merged SBOM; if the package is in both SBOMs, an enrichment process will be done so that the superset of information will included by adding additional information from the second SBOM to the package information provided the versions are the same.


Modified feature extraction notebooks

Hi @haesleinhuepf,
I modified some notebooks sorted under feature extraction:

Quantitative image analysis
Statistics using SimpleITK -> I added a notebook Statistics using Scikit image
I added a schematic of feature categories
Comparison of measurements from different libraries (mostly added text here and explained Bland-Altman Plot)
Added link to Feature extraction blogpost in literature

Let me know if you want me to change something, I am happy to do so.
Best,
Mara

I now changed the gitignore file so that not all .csv and .png files are ignored.

Awesome. Big thanks! :heart:


NS7: Shared files get generated on NS-only projects.

Environment
OS: win32 x64
Node: 12.18.2
NativeScript: 7.0.11
Angular CLI: 11.0.5
Describe the bug
The isWeb method returns true if there is a nativescript.config.ts file:
https://github.com/NativeScript/nativescript-schematics/blob/c087d4843843e13c52e39fea3106349fd5fdb855/src/generate/utils.ts#L33
In NS7 projects the configs are all consolidated into a single nativescript.config.ts per default also in NS-only projects. Therefore the ng-CLI generates shared files in NS7 projects:
$ ng g m test
CREATE src/app/test/test.module.ts (190 bytes)
CREATE src/app/test/test.module.tns.ts (269 bytes)
CREATE src/app/test/test.common.ts (177 bytes)

Expected behavior
The isWeb method should check the shared attribute of the NativeScriptConfig in NS7 projects.

This is fixed in #314

I guess this issue can be closed now @NathanWalker 😃


pep8 fixes and no need to encode data for beautifulsoup

This might fix #9, if not than response.encoding needs to be passed along to BeautifulSoup :)

This commit is a bit too monolithic. ;)
However, I will look into using response.text instead of response.encode(...), but this needs further testing.

I'm closing this pull request for now. I might open an issue on general improvements in the near future


Use CUDAdrv instead of CUDArt

This package only needs a couple of types anyway, but generally CUDAdrv is better maintained (we even discussed deprecating CUDArt in order to unify things a bit more).

wrt. OwnedPtr, this is the first design iteration so it could be improved, but conceptually I didn't want downstream packages to use it because it is just an implementation detail to deal with resource finalization. And because the design is bound to change, as is evident from this use case.
But as you say, some users will require manual memory management to build whatever they need. What about an abstract Buffer with OwnedPtr being the struct CUDABuffer <: Buffer, and by-protocol having an unsafe_convert(::Ptr, Buffer and some others?

That could work. You could also expose struct Buffer(ptr::T, len::Int) or something similar, and guarantee the API of Buffer but not of the internal ptr type (other than that it can be passed to kernels).
Raw access to buffers seems essential, but I don't think we need anything more than that (e.g. pointer arithmetic, which we can do by storing an offset where needed).

I've tried removing OwnedPtr but I think it's going to be an uphill battle; CUBLAS uses pointer(xs) a lot as well as pointer arithmetic. It seems strange that pointer still returns an OwnedPtr if that's meant to be an implementation detail; and since it's not a subtype of Ref there's no interface we can use to handle it abstractly.
I think we should go ahead with this improvement (which is at least no worse than the original) and then figure out the right pointer design that handles these cases.

It seems strange that pointer still returns an OwnedPtr if that's meant to be an implementation detail

It ought to implement the same basic interface.

and since it's not a subtype of Ref there's no interface we can use to handle it abstractly.

I thought subtyping Ref had other consequences, wrt. ccall and such. Not sure though.

I think we should go ahead with this improvement (which is at least no worse than the original) and then figure out the right pointer design that handles these cases.

Was going to propose the same. We can iron out a buffer-like interface, or another design, when we have some more bandwidth to do so (hackathon). I'll try and keep OwnedPtr stable until then.

Great, thanks a lot. I don't think the stability restrictions on OwnedPtr are too tight, since we're only really relying on it to exist and behave in a vaguely pointer-like way.


Retry #4337: Make gce kubeconfig context include project

Updated gingkgo-e2e auth_config this time. Ran e2e tests, 19/20 passing ("Pods should contain environment variables for services" still flaky). cc @zmerlynn

Since link doesn't seem work from commit title, this is #4337 fixed to not break gingkgo e2e.


(docs) lib/puppet/type/exec.rb unless grep(1) anchors

The shown grep(1) example is missing begin‑of‑line/end‑of‑line anchors.

I cherry picked the commit from PR #8859, rebased against 6.x and updated the commit message.


Generating Handwritten Digits on MNIST using GANs

This PR is for discussing the issues and working on GANs notebook for MNIST dataset.

@zoq @kartikdutt18  When I am adding convulation layer, it seems to give an error that

/home/viole/anaconda3/envs/notebook/include/mlpack/methods/ann/ffn.hpp:290:36: error: no member named 'push_back'
Is it due to syntax error while using the modules or something else? I tried working around it, but don't know why its giving push_back not found(this function should be available).

your thoughts?

Hi Swain, Could you push the latest changes (if you have got the issue resolved).

Hi Swain, Could you push the latest changes (if you have got the issue resolved).

Sure...I will push it by Tonight (I don't have the computer right now )

I redirected the PR at #172 . Closing this one.


Add speed and remaining time when copying files

What's the Problem?
No response
Solution/Idea
The lack of speed in copying files is very annoying to me, I have no way of knowing how long it will take.
Please add speed and remaining time when copying files.
Thanks.
Alternatives
Add speed and remaining time when copying files
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

Merging with #4811


dataset visibility is consistent across all charts

When a label is clicked on any chart, it removes the dataset from every other chart. This makes it easier to compare, for example, parquet and vortex-compressed across all queries.


fix documentation: machines result inaccurately explained

The results of machine checks are inaccurately explained in the documentation. They are represented as tuples, which in case of filters might be right, but is wrong when talking about machines.

Fixed.


Add support for ninja to re-run mbs if the configuration changes

Need to implement https://github.com/vkottler/runtimepy/issues/95 first, but then it should be fairly straightforward to implement this?

Start by automatically producing "compiled config" output.

https://github.com/vkottler/runtimepy/issues/95#issuecomment-1704971546
Just implement this command as a CLI command in this package (not runtimepy).

Implemented on this PR: https://github.com/vkottler/yambs/pull/66.

Continue further work on config triggering to: https://github.com/vkottler/yambs/issues/67.


String update

#2738  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

Merge activity

Jun 18, 12:47 AM EDT: @tommylau523 started a stack merge that includes this pull request via Graphite.


Pages Link Fix

This pull request addresses the Pages Documentation, and specifically the Deploy a React application link. If the link is clicked directly from the main page, it results in a Not Found error. If accessed directly, it works fine, by redirecting to Deploy a React application in Framework guides. This pull request edits the link for the main Pages Documentation to lead directly to the updated Deploy tutorial, rather than the original, deprecated link.

Accidentally broke this PR, here is the new one: Remove Superfluous Docs Page


god mode is bugged

there is a issue on god mode where whenver you die from outside enemies (Who hit kills such as eyeless dog and forest giant) you simply you can't interact with anything,
such as:
starting ship, closing doors, grabbing loot, etc
please fix

Yeah, its been an issue for a while, i might look into it next week, if it is a quick fix i will do it, otherwise probably not, sorry

okay thanks its just that i test some things and then suddendly a eyeless dog comes out "kills" me with god mode and then i have to wait for the rest of the round

Sorry it took so long but i fixed it in version 1.1.92


Report fields - App crashes when disabling/enabling list values via checkbox & dropdown button

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

Version Number: v9.0.19-0
Reproducible in staging?: Y
Reproducible in production?: N
Found when validating PR : https://github.com/Expensify/App/pull/46938
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-internal team
Action Performed:

Go to staging.new.expensify.com
Go to workspace settings > Report fields
Click Add field
Select List from Type
Go to List values and add a few list values
Long press on the list value > Select.
Tap on the dropdown.
Tap Disable.

Expected Result:
App will not crash.
Actual Result:
App crashes when disabling/enabling list values via checkbox and dropdown button.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/e8e64cfe-6bf3-44df-981e-1a9a412ff2f6
Bug6569740_1723464002972!staging.new.expensify.com-1723463652050.txt

View all open jobs on GitHub

We think this issue might be related to the #wave-control.

Production
https://github.com/user-attachments/assets/c29feebd-daa6-4800-b6d1-968dc2b29efc

cc @nkdengineer @jasperhuangg @jjcoffee from the linked PR

Checking this now.

@mountiny This comes from this PR. When we disable all values, the options becomes an empty array. Then the App crashes here
https://github.com/Expensify/App/blob/4ef1202704270a84bb6bdc2a6bf22201a6699d76/src/pages/workspace/reportFields/ReportFieldsListValuesPage.tsx#L260
https://github.com/Expensify/App/blob/4ef1202704270a84bb6bdc2a6bf22201a6699d76/src/components/ButtonWithDropdownMenu/index.tsx#L53

@Krishna2323 @mananjadhav are you able to handle this?

@Krishna2323 great!
@mountiny do you wanna take over this one since you have more context and already engaged in the conversation?

Taking over to move deploy forward :D

@allroundexperts will be reviewing @Krishna2323 's PR - https://github.com/Expensify/App/pull/47244

PR merged, working on CP

CP'd, asking for test on staging

Working well on staging!

Thanks everyone 🙇

@twisterdotcom i believe @allroundexperts needs payment for C+ for https://github.com/Expensify/App/pull/47244 - but maybe only once the fix has been on prod for a week?

I think it has now been on prod for a week? Deploy was on the 12th: https://github.com/Expensify/App/issues/47219

Payment Summary:

@allroundexperts paid $250 via NewDot requests (C+)

$250 approved for @allroundexperts


add Semi Icons StreamGeometry

Would it be better to release the icons as a separate package?

Worth noting package size increase from 528kb to 677 kb

What is IconPatch file used for?

What is IconPatch file used for?

Oh, I see, it seems to be a backward compat for typos?


Clarify the method of selecting several lines for commenting in PR.

Why:
When I read this, I didn't understand where I needed to click to add a comment to multiple lines. So I suggest clarifying this.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Instead of just "click and drag" you should say where to click: "click on a line in the line number gutter area". In addition, there is another way to add a comment to several lines: "click on the blue comment icon and drag to select the range of lines".
Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@Denis-GH Thanks so much for opening a PR! I'll get this triaged for review ✨

@subatoi
Thak you for reply!
I have made the changes as per your suggestion.
I have also added this to another doc about this.

I'll go ahead and merge this shortly :)

Why:
When I read this, I didn't understand where I needed to click to add a comment to multiple lines. So I suggest clarifying this.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Instead of just "click and drag" you should say where to click: "click on a line in the line number gutter area". In addition, there is another way to add a comment to several lines: "click on the blue comment icon and drag to select the range of lines".
Check off the following:

[x]  I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x]  For content changes, I have completed the self-review checklist.


Fixing session information in ServrviceBusReceiverAsyncClient debug logs

Description
Fixing incorrect session information in ServrviceBusReceiverAsyncClient debug logs
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


goroot alias directory somehow?

In Bazel 8.0, the path separator will change from ~ to +. (See the 7.3.0 release notes and associated PR)
That means the documentation for various IDE settings (and I assume other bits?) will have to change. It would be nice to have some prettier, more stable alias for the GOROOT and similar things needed. I'm not sure how much power bazel provides for this, though, so it's possible this ticket just needs to be closed or turned into a update-the-docs-when-change-lands ticket.

I would like to recommend using https://github.com/buildbuddy-io/bazel_env.bzl instead of maintaining something like this in rules_go. It's still a somewhat new ruleset, but please give it a try and provide feedback.

Hm, I'm not sure how to make that work in the configuration of vscode workspaces, and so on where I would need it for my aliasing. Using an extension that loads direnv doesn't seem to work because it wouldn't necessarily fire before the Go extension settings are loaded.


freeze

It freezes on main thread when I call [TTTAttributedLabel intrinsicContentSize].
I could reproduce it if I run following branch's app and scroll down to the bottom.
It seems CTFramesetterSuggestFrameSizeForAttributedStringWithConstraints never return.
https://github.com/aTakeshiKominato/TTTAttributedLabel/commit/7b89cd02ee6f9c7865ce7ef5ea9b491adba27004
It won't occur If I remove following line. I still don't know why.. using UIFont with kCTFontAttributeName causes..?
https://github.com/aTakeshiKominato/TTTAttributedLabel/blob/7b89cd02ee6f9c7865ce7ef5ea9b491adba27004/TTTAttributedLabel/TTTAttributedLabel.m#L133
My environment:
Xcode Version 7.3.1 (7D1014) iPhone6S iOS9.3 Simulator

i have the same issue there is any solution for it


num_particles behaves differently on cpu and gpu

It gives the number of active particles on CPU
It gives the total number of particles on GPU

Not ideal...

See PR #40


Non-Instructors can log in

The app is not checking if a user is an instructor or not. This is an application only for instructors. Also, there's exception:

You should update the methods of DataAccess with the newest ones, and also the database with the latest .bacpac.

solved


LCK Challenger does not provide data.

LCK Challengers has not been working this season. Doesn't provide data.

Thank you for reporting this issue and sorry that I couldn't reply sooner.
Unfortunately the split is already over, but from the history it looks like the Events should have been there. I will keep an eye out for this issue next split. Closing this issue for now.

@zlypher Commenting here instead of opening another issue. The LCKCL feed does not seem to provide data as of now. The season starts next week but no events are added to the calendar (whereas LCK/LPL/LEC etc do have their events added).
Thanks for this project by the way; it provides a lot of value to me.


Publish latest version to App Store / PlayStore

Is it possible to get the latest version for both App Store and iOS Store ?
The current published version date from last year 😕

@kuashe the updated version is in the master branch. We have planned publishing new version in both stores pretty soon.

+1

@NickIliev Any update on this 😬 ?

@kuashe @vahidvdn  I've updated the source of the repository - stay tuned for updates in the app stores.

Closing , I'll stay tune for update on the App Store.


Add Elastic Rerank notebook

Test in Colab👇

👋 @joemcelroy can I go ahead and merge this notebook? :)


Missing built files in npm

Hi @cgewecke! Congratulations on shipping this plugin! 🎉
Unfortunately, it isn't working right now. I think you forgot to build the project before running npm publish, because the dist/ folder is missing. I guess compiling and publishing a patch version should be enough.
I can't believe npm doesn't fail to publish a package whose main doesn't exist. I'll try to do something about this in the plugin boilerplate project.

Oh my! Very sorry, thanks for catching this. I published late last night and completely forgot about building.
This is fixed in 0.1.1.

Thanks for the quick fix!
Does it work with ethers? Or do I need @nomiclabs/buidler-truffle5?

@alcuadrado Yes, ethers works:

I had difficulty getting my ethers project tests to work (in this plugin, not yours). It's a little tricky to test the test command - opening an issue about that rn.

Ah wait - there's a problem!  I have to look at why.


/assets/images/wood-2.png missing from GitHub hosting?

Seems like the image is committed correctly but is 404'ing on github.io

It looks like it’s located here: https://github.com/YaleDHLab/neural-neighbors/blob/master/assets/images/wood-2.png
Will review the URL to see if something is off…
Monica
On Sep 18, 2017, at 11:17 AM, Peter Leonard <notifications<EMAIL_ADDRESS>wrote:
Seems like the image is committed correctly but is 404'ing on github.iohttp://github.io
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_YaleDHLab_neural-2Dneighbors_issues_15&d=DwMCaQ&c=cjytLXgP8ixuoHflwc-poQ&r=8Jph9bImGaZYoxbGc_VxQ554-KwncjdHj50LJ2PVLlY&m=2gm11GUAArPD98S8JrkB3hPDOpiBVfSxID6bxL_fU3Q&s=DEQMJ3fy5tomgKmi0fZC51KiibA5u7vDpaYNQ-8z0Ik&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AABjjxV5otDzkhwYX-2DT-2DOD9UPTjZHdVLks5sjol2gaJpZM4PbEnx&d=DwMCaQ&c=cjytLXgP8ixuoHflwc-poQ&r=8Jph9bImGaZYoxbGc_VxQ554-KwncjdHj50LJ2PVLlY&m=2gm11GUAArPD98S8JrkB3hPDOpiBVfSxID6bxL_fU3Q&s=8tHWmA3kKoPxL5-MHt94YJTlDz2D54zMJ_G9xMlRhXM&e=.

I feel like the line in the CSS is correct and for some reason that asset just isn't making it to .io?

Hi, Peter -
Doug pointed me to using a long image address — I’ve created a pull request. Do you want to take a look and merge to see if it works?
Thanks,
Monica
On Sep 18, 2017, at 11:45 AM, Peter Leonard <notifications<EMAIL_ADDRESS>wrote:
I feel like the line in the CSS is correct and for some reason that asset just isn't making it to .io?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_YaleDHLab_neural-2Dneighbors_issues_15-23issuecomment-2D330265086&d=DwMCaQ&c=cjytLXgP8ixuoHflwc-poQ&r=8Jph9bImGaZYoxbGc_VxQ554-KwncjdHj50LJ2PVLlY&m=Uw6d7SGxRLfRgqWIeSSlOUTtrA7zLullZ586o-eLmXE&s=bf9ZCTOuoTn8yNhPzco8sLq9-ir7B0WDLbpNoUUPEis&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AABjjzjWYC0C67Z5whxvXeQgnwBQZWozks5sjpAJgaJpZM4PbEnx&d=DwMCaQ&c=cjytLXgP8ixuoHflwc-poQ&r=8Jph9bImGaZYoxbGc_VxQ554-KwncjdHj50LJ2PVLlY&m=Uw6d7SGxRLfRgqWIeSSlOUTtrA7zLullZ586o-eLmXE&s=8lNTXiR_UQ_m8eCDwcS9mLM4C0HSc6nOrIJNdvhYND4&e=.

Hosting images on Github Pages is a little funny, in part because the repository name is added as a directory path on the deployed domain, and in part because the path to image files needs to point to the raw image, so:
https://github.com/YaleDHLab/neural-neighbors/blob/master/assets/images/wood-2.png
needs to be changed to:
https://github.com/YaleDHLab/neural-neighbors/blob/master/assets/images/wood-2.png?raw=true

Thx! All set now...


Can't build also with Visual Studio components

Describe your bug here.
I can't build Psych Engine 0.7.3.
I installed Visual Studio 2019 with the components. I also tried "setup-msvc-win.bat".
Haxe list:
flixel-addons: [3.2.1]
flixel-tools: [1.5.1]
flixel-ui: [2.5.0]
flixel: [5.5.0]
flxanimate: [git]
hxCodec: [git]
hxcpp: [4.3.2]
hxdiscord_rpc: [git]
lime: [8.0.1]
linc_luajit: [git]
openfl: [9.3.2]
SScript: [7.7.0]
tjson: [1.4.0]
Command Prompt/Terminal logs (if existing)
BUILDING GAME
------------------------SScript 7.7.0 Macro------------------------
Special Thanks:
- CrowPlexus

Checking version...
You are using the 4.2.5 version of Haxe, SScript works best with Haxe 4.3.2 or newer. Update your Haxe version.
Continuing...
-------------------------------------------------------------------

D:\JOSUE\fnf mod\export\release\windows\obj>setlocal enabledelayedexpansion
Warning: Could not find Visual Studio VsDevCmd
Missing HXCPP_VARS
Error: Could not automatically setup MSVC

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No.

#14890 I solved it.
Apart of MSVC v142 - VS 2019 C++ x64/x86 build tools and Windows SDK (10.0.17763.0) I installed these components:

C++ Profiling tools
C++ CMake tools for windows
C++ ATL for v142 build tools (x86 & x64)
C++ MFC for v142 build tools (x86 & x64)
C++/CLI support for v142 build tools (14.21)
C++ Modules for v142 build tools (x64/x86)
Clang Compiler for Windows

#14890 I solved it. Apart of MSVC v142 - VS 2019 C++ x64/x86 build tools and Windows SDK (10.0.17763.0) I installed these components:

C++ Profiling tools
C++ CMake tools for windows
C++ ATL for v142 build tools (x86 & x64)
C++ MFC for v142 build tools (x86 & x64)
C++/CLI support for v142 build tools (14.21)
C++ Modules for v142 build tools (x64/x86)
Clang Compiler for Windows

nah this still didnt work for me 😭


Make optional classes pointers so that omitempty can take effect.

Without this change optional classes often serialise to '{}' and can't take advantage of the 'omitempty' option to prevent serialisation. This is required for some requests that only allow a single member to be specified.

hey @Sumo-MBryant, it's been some time but I'd like to discuss these two PRs :) Can you please rebase so I can review and discuss? thanks for the contribution!

I have now rebased this onto the latest golang branch.

merging this and updating in a subsequent commit


DiffOrig is genius

Just came here to tell you that I love your DiffOrig command. It's really handy for me since I've switched to persistent undo in my .vimrc so I can't just uuuuuu to remember what I changed since I started editing.
Thank you!
I featured your snippet here: http://vimrcfu.com/snippet/214
PS: I also snatched nerdy from your Dockerfiles :grin:

@florianbeer http://vimdoc.sourceforge.net/htmldoc/diff.html#:DiffOrig

Ya i cannot take credit for this genius :)


Once repo is public, turn on Zenodo for DOIs

Once this is public, turn on Zenodo so that each release will automatically generate a new citeable DOI for the code.

Zenodo is turned on. We'll start getting DOIs as soon as we cut a release.


Added Resize Functionality to Plots

added resize functionality
limit to 80% of the screen's width and height

Fixes #456

@abhishekpratapa This adds a lot of complexity to the code, and I'm not sure it's the right approach. Have we considered making sure all of the Vega specs will fit within the default window size, vs. resizing the window to fit our Vega specs? I think the former may be a better fix, since it's both simpler, and will result in a consistent UX (consistent window size regardless of data structure passed in). Users can always resize/zoom on their own if they want; this issue is about the initial design bug where the Vega spec mismatches the window size (and we chose both of those to begin with).

Feature request


Better logging of bundling errors

I noticed that bundling errors seem to fall silent.
The simplest workaround is to add e.g.
log.error("Bundle error:", error)

at https://github.com/showpad/karma-rollup-preprocessor/blob/master/lib/index.js#L44
I don't know Karma's internal workings well enough to speculate why done(error, null) does not report the error, but I noted the coffee plugin similarly logs a message with
      log.error('%s\n  at %s:%d', e.message, file.originalPath, e.location.first_line)

Cheers.

Implemented by #5


Fix bug in repayDebt(...) in DelayedRepayCollateral

This would make the transfer form consistent with the other examples plus with DefaultCollateral.

The example was outdated and, therefore, was removed.


[debug] Provide a general setting to configure the default debug console

You can see that more and more debuggers support terminals as program console, and they provide the same console option individually in launch configuration. See that 'java', 'python' and 'node' all define the same console option in their launch configuration. The console option seems to be a common requirement rather than language specific, so it's better to have VS Code provide a general option for it.
Also, it is convenient to have it as a user setting. If a user has a preference for using a specific console, they only need to change the settings.json once, rather than having to modify launch.json repeatedly for each different project.
// cc: @isidorn

Currently Java has introduced their own java setting for this. Which I thought does not really make sense, and we should consider making this into a general vscode setting, so Java could deprecate their own.

@testforstephen I understand the feature request, but we are no longer in a position to make console a "built-in" VS code property because that would break all extensions that already use console but in a slightly different way.
E.g. the C# debugger uses console but with more options. And the C++ extension uses externalConsole instead of console because they cannot support all console options.
Dealing with this zoo in a non-breaking way would probably mean to find a new unused property name, agree on a common feature set, and then all extension would have to adopt the new property and migrate the user's debug configurations. IMO this effort is no longer feasible.

@weinand got it, thanks for your concern. If there are already some extensions that have different implementation for console, then making it a common setting will be a little tricky.

@testforstephen even if it is no longer possible to promote the console property to become a standard ("built-in") VS code property, we could introduce more "standard" user settings for VS Code's built-in "debug console".
That means every debugger extension that wants to support the debug console will still have to add the console property to its schema, but it could automatically profit from the built-in "debug console" settings.
Are there any generally useful user settings for the debug console that you would like to use in the Java extension?

That means every debugger extension that wants to support the debug console will still have to add the console property to its schema, but it could automatically profit from the built-in "debug console" settings.

Could you explain more how we can benefit from the built-in "debug console" settings when we have to add the console to its schema by ourself?

Are there any generally useful user settings for the debug console that you would like to use in the Java extension?

On Java extension, we define a user setting java.debug.settings.console and also console property in launch.json schema. By default, the launch console inherits the value from settings, and user can customize it per debug session via modifying console in launch.json. The allowed values for console are  ["internalConsole", "integratedTerminal", "externalTerminal"].


fix: module graph ensureEntryFromUrl based on id

Description
Currently we check for a module existence via url in ensureEntryForUrl, but a url could map to the same resolved id that already exists in idToModulesMap (e.g. via a different url that also resolves to the same id).
This can currently happen for two URLs like:

/Users/foo/bar/project/src/index.js
/src/index.js

This PR changes to check the resolved id instead, and update the url to use the existing id & module if possible.
Additional context
Ref https://github.com/withastro/astro/pull/4378
Feels related to #9730 and #7845
Something similar has been happening in SvelteKit too

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Full ecosystem CI green for this PR. There was a glitch in Rakkas, but it is now solved.


GNOME xrdp session sometimes freezes on reconnect

On several different installations of Ubuntu 20.04 (both VMs and non-virtualized installs), I keep hearing from several users (but not some other users) that their GNOME desktops sometimes freeze when they reconnect to pre-existing xrdp sessions. One user uses MS Remote Desktop from a Mac to connect, another one uses MS Remote Desktop from Win 10. Upon connect, they can move their mouse cursor but not click on anything. Sometimes the end up with a black screen after a while. This can happen after dozens of successful reconnects.
I have found this issue very hard to debug, I'm not seeing any relevant error messages in xrdp.log xrdp-sesman.log (both say that the reconnection attempts succeeded, the failed attempts look just like the successful ones) or ~/.xorgxrdp.XX.log, and I couldn't pinpoint any pattern in what the users had been doing / running. I've been seeing this issue on previous versions of Ubuntu, too, and it has survived all upgrades since 18.04. These machines are all joined into an AD domain, but authentication seems to be successful: xrdp reports that the sessions get established just fine. So this might totally be a GNOME issue. I'm willing to hunt down this bug more systematically, but would need some pointers on that. Thanks!
Package: xrdp:
Version: 0.9.12-1
Package: xorgxrdp
Version: 1:0.2.12-1
Package: gnome
Version: 1:3.30+2
In the logfiles, connection attempts by user myuser1 until 13:01 were successful, but from 16:21 on they were frozen. At 16:55 we killed all of this user's processes.
.xorgxrdp.xx.log: https://pastebin.com/P4d8PsxH
xrdp-sesman.log: https://pastebin.com/PVcqR8Wk
xrdp.log: https://pastebin.com/WpZEWi2Y

I saw something similar some time ago when I was running as sysadmin for a group of developers (also running as part of an AD domain).
It turned out to be an application the developers were running had made a mouse grab, and as a result the mouse buttons could not be used by anything else. It might have been Eclipse, but it's some time ago so I can't be certain. You could try logging in over ssh, setting DISPLAY and running something which makes a mouse grab like xwininfo. That might tell you if this is the problem.
X11 doesn't log things like this, so we ended up killing X11 applications one-by-one until we found the culprit.
Hope that's useful.

Possible this is related.   https://github.com/neutrinolabs/xrdp/issues/1590

I'll put this here because it might be relevant, and it also might help anyone who lands here.
I was having this problem with xfce4. Whether the screen is black depends on whether its been idle for 10 minutes or more, and this is caused by the Xfce power manager. Go into the Xfce power manager and turn off, max out and disable all display and system time outs. Perhaps Gnome has a simliar feature?

I had the same issue and I found the following entries in the system log that were written while the session was disconnected:
xrdp[36752]: [ERROR] xrdp_sec_recv: xrdp_mcs_recv failed
xrdp[36752]: [ERROR] xrdp_rdp_recv: xrdp_sec_recv failed
xrdp[36752]: [ERROR] libxrdp_process_data: xrdp_rdp_recv failed
xrdp[36752]: [ERROR] xrdp_process_data_in: xrdp_process_loop failed
xrdp[36752]: [ERROR] SSL_write: I/O error
xrdp[36752]: [ERROR] xrdp_iso_send: trans_write_copy_s failed
xrdp[36752]: [ERROR] Sending [ITU T.125] DisconnectProviderUltimatum failed

It appears that a connection on the server was idle for too long and therefore was terminated. I could solve the issue by adding the following options to my /etc/ssh/sshd_config:
ClientAliveInterval 300
ClientAliveCountMax 1152

They tell the SSH daemon to send a keep-alive packet every 5 minutes for at most 1152 times (which equals 4 days in total).

The SSL library used by xrdp does not use /etc/ssh/sshd_config. I can't explain why editing this file should have any effect on xrdp.

@matt335672 is right. I just was lucky not to run into the issue for several days, but it has happened again although I've not removed my changes to sshd_config yet...

Experiencing this, with the same errors in xrdp-sesman.log, on Ubuntu 22.04, granted with a slightly older version - 0.9.17. Updating to current is in the plan for the near future.

I don't think @stefanuhrig 's issue is not the same with the original post because this log is not recorded in the @nobias 's log. I think it should be reported as separate issue.
xrdp[36752]: [ERROR] xrdp_sec_recv: xrdp_mcs_recv failed
xrdp[36752]: [ERROR] xrdp_rdp_recv: xrdp_sec_recv failed
xrdp[36752]: [ERROR] libxrdp_process_data: xrdp_rdp_recv failed
xrdp[36752]: [ERROR] xrdp_process_data_in: xrdp_process_loop failed
xrdp[36752]: [ERROR] SSL_write: I/O error
xrdp[36752]: [ERROR] xrdp_iso_send: trans_write_copy_s failed
xrdp[36752]: [ERROR] Sending [ITU T.125] DisconnectProviderUltimatum failed

Regarding @kronenpj 's issue, I don't see any errors in the @nobias 's xrdp-sesman.log. Should post the actual log instead the word "the same errors".


Publish package for Core library

https://github.com/microsoft/chat-copilot needs to consume package for integration

@TaoChenOSU could you check if we have automatic nightly builds being published? then we can close this one

Public nuget packages are in nuget.org.  Let's open bugs or new tasks for any additional work.


AutomationElement.Find***return wrong decendants arrangement.

Hi everyone.
Thanks for Reading my problem.
I am trying to get a custom control's decendants. but FindAllDescendants() return a wrong results.
there are two panes of a parent pane. When I get AutomationElement of one pane, returns AutomationElement of the other pane. Somewhere wrong?
Appreciate for your help.

paste my code, the scrolloutput  below is the wrong:
        var buildWindowsViewWindow = window.FindFirstDescendant(cf => cf.ByAutomationId("buildWindowsViewWindow"));
        buildWindowsViewWindow.Patterns.SelectionItem.Pattern.Select();
        System.Threading.Thread.Sleep(2000);

        var outputWindowsViewWindow = window.FindFirstDescendant(cf => cf.ByAutomationId("outputWindowsViewWindow"));
        outputWindowsViewWindow.Patterns.SelectionItem.Pattern.Select();
        System.Threading.Thread.Sleep(2000);

        var scrolloutput = outputWindowsViewWindow.FindAllChildren();

UISpy is return same value.


Implement interface improvement with mobile compatibility this time

@joshurtree, this looks good (I checked mobile too this time).  Can you resolve the conflicts and then we can merge this change?


agent class misses information if backup / restore is enabled

on the agent class, you can enable and disable backup / restores,
but you can't get the current status to check if backup is enabled or not.
Please add .is_backup_enabled to the agent class.

Hi,
Thanks for pointing this out. You are welcome to make the change, and open a PR, or let us know, and we can add this attribute.
-jatin

Hi Jatin,
sorry, i'm busy with developing a tool for our commvault enviroment with this cool sdk.
It doesn't hurt, but it would be great if you can add this. I can only report errrors at
work time, and my freetime is realy limited atm.
Thank you,
Daniel

Hi Daniel,
Sure, I understand.
I'll do it in the coming week.
Thanks,
Jatin

Hi Daniel,
The attributes have been added. Please try with latest code from master.


Add documents owner for ImplDef and SourceFile

When working on #3182, I found that ImplDef and SourceFile do not implemet DocCommentsOwer trait, and I tested it in cargo doc that impl could has some doc-comments.
I am not so sure about SourceFile case, but in theory if that file is a crate root, the doc comment of it should represent the whole crate documentation, right ?

the doc comment of it should represent the whole crate documentation, right ?

Or the inner doc commnent of the corresponding module.
bors r+


Localhost:8888  can't be reached.

Hi Apollo Team
I followed the steps to access DreamView, but the localhost cannot be reached.
everything seems good except two errors as following:
%-------------------------------------------------------------------------
gogators@gogators:~/apollo$ docker/scripts/dev_start.sh
[sudo] password for gogators:
[INFO] Start pulling docker image apolloauto/apollo:dev-x86_64-20180130_1338 ...
dev-x86_64-20180130_1338: Pulling from apolloauto/apollo
Digest: sha256:c4c153fa44480e284d3d843819e6ad367452e922dd665e3970a3e6972c3879fc
Status: Image is up to date for apolloauto/apollo:dev-x86_64-20180130_1338
modprobe: ERROR: could not insert 'nvidia': No such device
modprobe: ERROR: could not insert 'nvidia_uvm': No such device
localization_volume-x86_64-latest: Pulling from apolloauto/apollo
Digest: sha256:b56b7526484aa4e70f9e6c9f7df028c7560d7a3d8df0fee7de984b5f43fad14e
Status: Image is up to date for apolloauto/apollo:localization_volume-x86_64-latest
ff1c652d640618720b924ad0cc632e377573fe00bea751067778301d8c233768
[INFO] Starting docker container "apollo_dev" ...
c8d740f8639f0c5e319348374a2ec406957e02e6a8f4e34b68cb48b9c9f56764
Adding group gogators' (GID 1000) ... Done. Adding user gogators' ...
Adding new user gogators' (1000) with group gogators' ...
The home directory /home/gogators' already exists.  Not copying from /etc/skel'.
[ OK ] Finished setting up Apollo docker environment. Now you can enter with:
bash docker/scripts/dev_into.sh
[ OK ] Enjoy!
%-----------------------------------------------------------------------------------------------------------
Hope you guys can help me to fix it, Thanks a lot.

Did you build successfully? After that you should also run "scripts/bootstrap.sh start".

Yes,built successfully. and then run "./scripts/bootstrap.sh" as the introduction. Finally, shows:
Start roscore...
Started supervisord with dev conf
Dreamview is running at http://localhost:8888
But the localhost cannot be reached...

Hi Apollo team,
I've also met the same problem. I run Apollo version 2.0 on docker 17-12.0-ce.
walle-dl@in_dev_docker:/apollo$ bash scripts/bootstrap.sh run
Start roscore...
Started supervisord with dev conf
Dreamview is running at http://localhost:8888

And then I opened a the page http://localhost:8888 with firefox, but it shows that Unable to connect...
Hope you guys can help me to address this problem, thank you :)

@GooGators Hi, did you do build the apollo from source after you first step into the docker?
# To get a list of build commands
./apollo.sh
# To make sure you start clean
./apollo.sh clean
# This will build the full system and requires that you have an nVidia GPU with nVidia drivers loaded
bash apollo.sh build

After you build the source apollo code, have another try! Good luck to you :)

@ytzhao Yes, it has been built successfully. Do you have any idea to fix this problem?

I'm having similar issues. I am running Apollo on two computers, both build and run fine. On one computer, dreamview also is accessed successfully. On the other, while everything seems OK and the Start roscore... Started supervisord with dev conf Dreamview is running at http://localhost:8888
messgae appears after running bootstrap.sh. But http://localhost:8888 can not be accessed. I have looked into the "supervisord.log" and see the following:
2018-03-06 15:42:55,042 INFO RPC interface 'supervisor' initialized 2018-03-06 15:42:55,042 CRIT Server 'unix_http_server' running without any HTTP authentication checking 2018-03-06 15:42:55,043 INFO daemonizing the supervisord process 2018-03-06 15:42:55,043 INFO supervisord started with pid 99 2018-03-06 15:42:55,148 INFO spawned: 'monitor' with pid 103 2018-03-06 15:42:56,150 INFO success: monitor entered RUNNING state, process has stayed up for > than 1 seconds (startsecs) 2018-03-06 15:42:56,268 INFO spawned: 'dreamview' with pid 152 2018-03-06 15:42:56,641 INFO exited: dreamview (terminated by SIGILL (core dumped); not expected) 2018-03-06 15:42:57,645 INFO spawned: 'dreamview' with pid 243 2018-03-06 15:42:57,830 INFO exited: dreamview (terminated by SIGILL (core dumped); not expected) 2018-03-06 15:42:59,837 INFO spawned: 'dreamview' with pid 245 2018-03-06 15:43:00,020 INFO exited: dreamview (terminated by SIGILL (core dumped); not expected) 2018-03-06 15:43:03,025 INFO spawned: 'dreamview' with pid 247 2018-03-06 15:43:03,234 INFO exited: dreamview (terminated by SIGILL (core dumped); not expected) 2018-03-06 15:43:07,241 INFO spawned: 'dreamview' with pid 249 2018-03-06 15:43:07,432 INFO exited: dreamview (terminated by SIGILL (core dumped); not expected)
Any ideas or help much appreciated!

Supervisor is not a good way for development. In recent change (https://github.com/ApolloAuto/apollo/commit/67fad1f572b3f1973b695af5bab093546ee7fa0d), I have removed it from debugging mode.
However, if you are using Apollo for some production scenarios, Supervisor could provide a second chance to recover the modules, which makes the more robust. Pass "--prod_mode" when starting dreamview for such use case.
You should see more detailed information at data/log/dreamview.out if you still encounter this problem.

@GooGators hi guys,I have the same problem,did you find any way to fix it?I tried my best to fix it,but failed


provide self as context when evaluating default

Ability to define defaults based on the current hash. Pretty straight forward.

cc @dblock

I like it.

@dblock  any idea when we'll bump to 3.3.3/3.4.0 ? :D

You want to make a release? Go for it! Should be 3.4.0 since there're some API changes.

sounds good, will do tonight thx :)


Feature: Implement a terraform/opentofu remote state

Related to
Other
Impact
nice to have
Missing Feature
Implement the remote state storage api to avoid losing the terraform state which is currently stored within the working directory.
Implementation
Extend the api spray by the api endpoint documented by terraform to store the terraform state within the database is the terraform scripting haven't defined any other remote state.
Design
No response

Like I mentioned here as well. Alternative implementation could be: store it outside of the Docker container using a persistent volume and/or volume mount.

You could also just use external services like S3, Gitlab or https://github.com/webhippie/terrastate to store the state. But a really good goal would be to provide the state storage as part of Semaphore.


I tryed this plugin in Docker Compose. but error.

Hi, everyone.
As title, I tryed this plugin in docker-compose as well as docker-compose.yml
but, the following error is displayed
  ERROR: Cannot start container ff1440ebc042b22582d35fd0bd3add2e95c8f06fc5efba3a8709485423234809: [8] System error: no such file or directory

of cource, usuall docker-engin as well as docker run --volume-driver glusterfs ... is good result.
I want to using this plugin in docker-compose.
tanks.

hi everyone.
I fixed my problem.
point is volume configrate of base image(in dockerfile).
I use base docker image is drupal:7
this image is use /var/www/html directory.
and, this time included -v foo:/var/www in docker running configration.
I rewrited configration file like -v foo:/var/www/html
And, VICTORY!!
thanks.
fix.

closed


Exclude proptest-regressions from packaged files

This PR causes the proptest-regressions folder to be excluded from packaged files.
Closes #4061.

bors merge


Fix receiver lifetimes in Any methods.

The lifetimes in several Any methods are restrictive in a manner that's unlikely intended. To resolve this, this PR makes changes of the following nature:
-    pub fn as_bytes(&'a self) -> &'a [u8] {
+    pub fn as_bytes(&self) -> &'a [u8] {

The prior method signature requires that the reference to receiver have the same lifetime as the internal content of Any. What this means is that Rust will unify the lifetime of the receiver with that of the returned content. Since the receiver must have a shorter lifetime than the content, the lifetime of the returned content is shortened. This is unnecessary as there need not be any restriction on the receiver's lifetime.

Neither of the failures are related to this change.

Indeed, it's better to let the compiler infer the lifetimes here.
Applied, thanks!


Going from Talon to Foundation causes the talon card to go behind

This is what happens after the fact when you drag from the Talon to the Foundation.

========TALONVIEW========
class framework.core.mvc.view.PanelView
[H][F]	6 of HEARTS	[22]
[H][F]	4 of CLUBS	[21]
[H][F]	1 of HEARTS	[20]
[H][F]	13 of HEARTS	[19]
[H][F]	1 of SPADES	[18]
[H][F]	3 of HEARTS	[17]
[H][F]	12 of HEARTS	[16]
[H][F]	10 of DIAMONDS	[15]
[H][F]	2 of DIAMONDS	[14]
[H][F]	9 of CLUBS	[13]
[H][F]	5 of SPADES	[12]
[H][F]	13 of DIAMONDS	[11]
[H][F]	11 of DIAMONDS	[10]
[H][F]	7 of HEARTS	[9]
[H][F]	4 of SPADES	[8]
[H][F]	11 of SPADES	[7]
[H][F]	5 of CLUBS	[6]
[H][F]	10 of CLUBS	[5]
[H][F]	3 of SPADES	[4]
[H][F]	5 of HEARTS	[3]
[H][F]	13 of SPADES	[2]
[H][F]	12 of CLUBS	[1]
[H][F]	11 of CLUBS	[0]
========TABLEAUVIEW========
[V] [F]	11 of HEARTS	[0]
========TABLEAUVIEW========
[V] [F]	10 of HEARTS	[1]
[V] [B]	2 of HEARTS	[0]
========TABLEAUVIEW========
[V] [F]	5 of DIAMONDS	[2]
[V] [B]	6 of CLUBS	[1]
[V] [B]	8 of CLUBS	[0]
========TABLEAUVIEW========
[V] [F]	8 of HEARTS	[2]
[V] [B]	8 of SPADES	[1]
[V] [B]	6 of SPADES	[0]
========TABLEAUVIEW========
[V] [F]	4 of DIAMONDS	[4]
[V] [B]	12 of SPADES	[3]
[V] [B]	9 of SPADES	[2]
[V] [B]	6 of DIAMONDS	[1]
[V] [B]	3 of DIAMONDS	[0]
========TABLEAUVIEW========
[V] [F]	4 of HEARTS	[5]
[V] [B]	8 of DIAMONDS	[4]
[V] [B]	7 of DIAMONDS	[3]
[V] [B]	13 of CLUBS	[2]
[V] [B]	1 of DIAMONDS	[1]
[V] [B]	10 of SPADES	[0]
========TABLEAUVIEW========
[V] [F]	9 of DIAMONDS	[6]
[V] [B]	7 of CLUBS	[5]
[V] [B]	7 of SPADES	[4]
[V] [B]	9 of HEARTS	[3]
[V] [B]	3 of CLUBS	[2]
[V] [B]	2 of SPADES	[1]
[V] [B]	12 of DIAMONDS	[0]
========FOUNDATIONVIEW========
[V] [F]	1 of CLUBS	[1]
[V] [F]	2 of CLUBS	[1]
========FOUNDATIONVIEW========
========FOUNDATIONVIEW========
========FOUNDATIONVIEW========
=========OptionsPreferences=========
Draw Option: ONE
Scoring Option: NONE
Timed Game: false
Status Bar: false
Outline Dragging: true
Cumulative Score: false
Deck: DECK_10


[Suggestion] Immersive Vehicles Integration

Would it be possible to improve the compatibility with the Immersive Vehicles mod on 1.12.2? As of now, the maids CAN ride the vehicles (I got them to ride it using the Carry On mod), but they sit on top of the collision box as if it were a horse, instead of sitting on the intended vehicle seats, I just wanted to ride around  with my maido :(
In Immersive Vehicles, vehicle seat coordinates are set through JSON files in whichever vehicle pack they belong, maybe that could be useful, just an idea.

1.12.2 has stopped supporting


Support fetching only a subset of the metadata in the Python client

Proposed API after several design discussions with @tacaswell:
results = tree.select(metadata=[('start', 'plan_name'), ('stop', 'exit_status)])
for item in results.items():
    # type(item) is some limited proxy object that only implements keys(), items(), values(), ... for now

This is supported in the HTTP API (?select_metadata) but not yet exposed in the Python API.

Per discussion with @pbeaucage, maybe the best way to do this it to add a method like this:
for k, v in c.select_metadata(json_path).items():
    # k is the key, v is the metadata dict _only_

Another idea:
for k, v in c.raw_items().page_size(10).select_metadata(…):

Likewise for raw_values().
Compared the suggestions in my previous comments I like that

This just exposes the parsed JSON from the server, without wrapping it in a special Tiled client class. There could be a lot of uses for that.
The name is pretty clear.
This supports chaining methods as shown above, including those like select_metadata that only are supportable on raw_items() and those like page_size() that make sense of items() generally.

This issue may be of interest to @cjtitus.


Hide gene report generation button

Hides the broken gene report DL button mentioned in #103 (just commenting it out)

There was actually a long dragging pull request that Mutaamba merged and the report works.
So don't delete the button after all :)

There was actually a long dragging pull request that Mutaamba merged and the report works.
So don't delete the button after all :)

Alrighty, closing this.

Alrighty, closing this.


Converted item registry to Java

I tested and everything works

I can't use Github for the like of me ...


Testing: flag tests as "online"

Our test suite currently includes "online" tests, i.e. tests that require network connectivity and access to services for their functionality.
We should mark these tests as "online," so that a user can disable them for local-only testing.
Pytest supports CLI extensions for this purpose, like in this SO post.

To summarize: we're going to want to mark these test with @pytest.mark.online or something similar, and then add a --no-online or --skip-online flag to the pytest runner that users can pass when they want to disable any online tests.


irreps analysis with the NAC option

Dear Dr. Togo
I'm using phonopy together with abinit to calculate phonon dispersions. However, there seem to be two problems.

With the --nac option, the mesh and band calculations produce quite different phonon frequencies at the gamma point. Can I know why are they different?
The gamma-point phonon frequencies in the irreps analysis output file is not changing with the the --nac option and always stay with the zero Born effective charge values. Then how to analyze the raman/infrared activity once the --nac option is included?
Thanks a lot for your help!

Wishes
Shang

You can use --nac and --q-direction simultaneously to break symmetry at Gamma.

I will close this issue.

Dear Professor Togo,
Can I ask a question related to this issue? By including --nac and --q-direction command line options as you suggested above, I did indeed manage to get LO-TO splitting in both band.yaml and irreps.yaml files, but in mesh.yaml, for each irreducible representation, the LO mode is still degenerate with the two TO modes at gamma point. I think this leads to incorrect projected density of states. Can you please let me know how I can handle this issue?
Many thanks,
Wei

It is the effect at non-zero q-point but at limit of q->0. So I have no idea how to include at Gamma point for mesh case as implementation.


Add extensions for logging whole results

https://github.com/Rutherther/NosSmooth/blob/main/Core/NosSmooth.Core/Extensions/ResultExtensions.cs


License in Readme

The license text in the readme states regarding the license

CrossPlatformLibrary is Copyright © 2016 Thomas Galliker. Free for non-commercial use. For commercial use please contact the author.

This has the wrong project name (copy & paste?) and contradicts the license file that states that the code is licensed under MIT.

True, this is a typo, thanks for reporting. Concerning the contradiction: All of my open source projects come with dual license: As long as you use the project for non-commercial purposes, the MIT license applies. When used commercially, it‘s subject to a small usage fee. Hope that‘s not a no-go for your project. The aim is not to get rich, but to get a fair compensation if it really helps you :-) Always welcome to discuss: linkedin >> thomas galliker


Cannot distribute as IPA generation fails due to isar post Archiving

Steps to Reproduce
Build, then archive, then distribute as IPA
I am not even using isar, just added in pubspec.yaml and as the build was triggered in codemagic I noticed all the builds for iOS started failing
Code sample
ipatool failed with an exception: #<CmdSpec::NonZeroExitException: $ /Applications/Xcode.app/Contents/Developer/usr/bin/python3 /Applications/Xcode.app/Contents/Developer/usr/bin/bitcode-build-tool -v -t /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin -L /var/folders/__/yww5sl0s2zx_l_mf4jmk7cs80000gn/T/ipatool20220310-73262-1h0u74u/thinned-out/arm64/Payload/Goala\ Dev.app/Frameworks/Flutter.framework --sdk /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS15.2.sdk -o /var/folders/__/yww5sl0s2zx_l_mf4jmk7cs80000gn/T/ipatool20220310-73262-1h0u74u/thinned-out/arm64/Payload/Goala\ Dev.app/Frameworks/isar_flutter_libs.framework/isar_flutter_libs --generate-dsym /var/folders/__/yww5sl0s2zx_l_mf4jmk7cs80000gn/T/ipatool20220310-73262-1h0u74u/thinned-out/arm64/Payload/Goala\ Dev.app/Frameworks/isar_flutter_libs.framework/isar_flutter_libs.dSYM --strip-swift-symbols /var/folders/__/yww5sl0s2zx_l_mf4jmk7cs80000gn/T/ipatool20220310-73262-1h0u74u/thinned-in/arm64/Payload/Goala\ Dev.app/Frameworks/isar_flutter_libs.framework/isar_flutter_libs
Status: pid 73652 exit 1
Stdout:

Version

Platform: iOS
Flutter version: 2.10.3
Isar version: 2.2.1
Isar Generator version: 2.2.1

A little bump, screenshot attached, appstoreconnect validation passes but ad hoc ipa generation fails... I haven't submitted to the appstoreconnect yet as the app is not ready so cannot check that, I am completely block and cannot be producing test builds of the app... https://capture.dropbox.com/dgpeR0fxFb5C4C4Y

Does it work with other profiles?

Should be fixed by now. Please reopen if you still have issues.


Add default github workflow

Finally, I would recommend to replace the GH action of one of our policies with what would be generated by cargo generate. That way we would be 100% sure the action is working as intended.

I am fine doing it this way, but I would rather add yet another pipeline within this project that does that test, both after push/PR and in a scheduled way.

Of course, assuming we have a way to not include the "test the template" workflow inside the target project generated by cargo generate itself...

Just tested on a clean repo of my own. Everything worked fine and carg-generate now asks for a couple more placeholders with default values: registry (ghcr.io) by default and prefix of the image (kubewarden/policies by default).


Initial Rev of Framework Outline

This pull request provides a outline/rough draft for the application functions sdk for go. Filtering and transforming is rudimentary, data is triggered by HTTP at the moment.

Approving and merging in order to populate the holding repo. After that, formal review and discussion can occur. Not gonna manage all of that via PR.


Added FROM to UPDATE for Postgres flavour.

Thanks.

@hiddentao Any chance you could publish this change to NPM?

@akshayp done!

Thanks for the quick turnaround @hiddentao!


Resources created without @context gets effectively updated without updating rev

Steps to reproduce it:

Create a resource without context: {"@type": "Person", "name": "John", "age": 23 }
Fetch the resource. A @context is injected with the base and vocab values from the project

{
  "@context": [
    {
      "@base": "http://example.com/base20/",
      "@vocab": "http://example.com/vocab20/"
    },
    "https://bluebrain.github.io/nexus/contexts/resource.json"
  ],
  "@id": "john",
  "@type": "Person",
  "age": 23,
  "name": "John"
}

Change the project vocab and base value
Fetch the resource again

{
    "@context": [
        {
            "@base": "http://example.com/base30/",
            "@vocab": "http://example.com/vocab30/"
        },
        "https://bluebrain.github.io/nexus/contexts/resource.json"
    ],
    "@id": "http://example.com/base20/john",
    "@type": [
        "http://example.com/vocab20/Person",
        "Person"
    ],
    "age": 23,
    "name": "John"
}

That's a problem because effectively the payload has changed without any change in the revision of the resource, which is not consistent with the resources lifecycle.
Also there is the issue that the @type value has now two values (the original one and the one with the new @vocab.

The way we are going to fix is as follows:

Whenever a resource that does not contain a @context gets created, we will add (and store in the primary store) the following payload:

{
     "@context": {
            "@base": "{project_base}",
            "@vocab": "{project_vocab}"
     }
}
- Whenever a resource that does not contain a @context **gets updated**, we will take it's @context from its previous revision

This will fix the issue, while keeping the original context for updated resources (even if base and vocab gets updated in the project).


feat: add unsafe atom conversion to hocon_tconf:check_plain

By using hocon_tconf:check_plain(Schema, Map, #{atom_key => {true, unsafe}}), fields and hoconsc:map keys are converted to atoms unsafely, while keys in map() and their nested values therein are kept as binaries.

tagged 0.39.16


Centering play button within a Video element

The placement of the play button is a bit off-centred. Can this be fixed please?

Alternatively, we bring in the timestamp and adjust spacing accordingly. What do you think @davidpmccormick @gestchild ?

I think we should take the margin off the right of the icon and add it to the left of the timestamp, then everything should have appropriate spacing regardless of whether the timestamp exists.


Handle new non-numeric sasl events

Note that I've opened an issue on the irc library: https://github.com/jaraco/irc/issues/214 to inform them about the problem this change caused. I don't really expect any action to come from that; this fix should be merged regardless.
If their policy is that new name mappings for numeric events can be added in minor version bumps, it might be a good idea to also change the dependency in setup.cfg to irc >=19.0.0, <20.4, since there are still a few other numeric codes being used in network_room.py

I like the cleanup @kepstin suggested. If you could incorporate that I'll happily merge this. Thanks for figuring it out!


'Go to Implementations' does not work when referenced models are in same line

INTERLIS IMPORTS can be written in one line with the model names separated by commas, like
IMPORTS model_A,model_B,model_C,model_D;
With v0.1.6 I see the following behaviour:

when selecting the first model after the IMPORTS statement (model_A) the lookup is successful
when selecting one of the other models I get the  'No implementation found' hint
when deleting model_A, so that model_B is next to the IMPORTS statement the lookup of model_B is now successful

This and #27 look quite similar to me. Like 'is this line starting with IMPORTS, then do the lookup using the selected text as model name' (and therefore ignore everything between IMPORTS and the selected text)?


If StatusBar handles ,  is eaten for all Toplevels except Application.Top

Easily reproduced in the main UI Catalog window where opening the File menu, selecting Quit and hitting ENTER starts the selected scenario vs. exiting.
This is true of any key that StatusBar eats: E.g. if there's a StatusItem for Key.Left left arrow on Menu won't work.
I'm using ENTER this way, because ListView currently doesn't support ENTER to select an item, nor does it provide a "ItemDoubleClicked" event (#429).

ENTER key is working on Menu.

I'm going to create an OpenSelectedItems event for ListView right now. Invoked by the ENTER key and DoubleClicked.

ENTER key is working on Menu.

Not for me. It causes the selected Scenario to run instead of exiting. Have you tried reproducing on Windows? I can repro on Linux too.
(Note I just remembered OnKeyDown only works on Windows and thus UI Catalog scenarios currently can't exit on Linux...)

Yes I did. It's working on both. I'll checkout your PR and see if it's happen too.

I'm going to create an OpenSelectedItems event for ListView right now. Invoked by the ENTER key and DoubleClicked.

Done! #440

Pretty sure the underlying bug here is the same as what is causing #437.

@BDisp this is fixed, right?


PowerToys Run unit conversion result copy failed

Microsoft PowerToys version
0.64.0
Installation method
GitHub, PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

enable global unit conversion
enter the text 1 hour to ms
press enter to copy the text
receive the error: copy failed.

✔ Expected Behavior
copying to clipboard
❌ Actual Behavior
receiving error message: "copy failed"

report a bug zip:
PowerToysReport_2022-11-08-15-39-13.zip
Other Software
No response

/bugreport

/dup #21855


Some issues

Hello, thank you very much for sharing your code. However, I have noticed that some parts of the code do not align with the content of the paper.

The code following PAdaIN does not match the paper content.
I couldn't find the preprocessing method for generating reference images in the code.
The balance module and stretch module involve convolution, and the convolution parameters need to be trained before use.
Additionally, the essence of the paper is essentially based on supervised training using reference images, which is quite different from the title of the paper.

Hello,
Thank you for your interest in our work and for taking the time to review our code. We appreciate your feedback and understand your concerns. Let me address them one by one:

PAdaIN Code: The implementation of PAdaIN can be found in the file UDnet.py on lines 102-109. We believe this section of the code aligns with the content of the paper.

Preprocessing Method: The preprocessing method for generating reference images is implemented in the file label_gen.py.

Balance and Stretch Modules: The balance and stretch modules are trained online. This process is implemented on line 77 in the file UDnet.py.

Supervised Training: The paper relies on auto-generated reference images without ground truth, see dataset.py lines 71-115.

We hope this clarifies your concerns. Please feel free to reach out if you have any more questions or suggestions.


[Question]: RouterQueryEngine what is recommended way to access selected engine?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Hi,
what is recommended way to access selected tool/engine in RouterQueryEngine? As far as i can tell it's only logged via logger.info() but it's not stored anywhere. It would be great if either it can be returned or stored as property (or example 'last_selected_engine_ind`).
Thanks!

Such approach requires to sync custom implementation with any changes made RouterQueryEngine class (since it's not possible to call super().query()) - would it be feasible that I will prepare MR with such feature so it might be officially supported?

@h1Logic I think ideally this would get logged either in the response object (response.metadata?)  or would be part of our callback-based event logging.


Using GPG signatures

Q
A

Issue Type
Feature Request

Deployer Version
N/A

Local Machine OS
N/A

Remote Machine OS
N/A

Description
Using GPG signatures for release.
And support install with PHIVE
Ref:
https://akrabat.com/using-phive-to-manage-phpunit/

Phive now in really development stage. There is no docs about installation or adding own phar.
But adding GPG signatures is a good idea. Now we have only SHA1, and i'm looking forwart to adding GPG to releases.

What to sign with GPG? tag or phar archive?

No need for this right now. There is sha1 anyway. Maybe will return to this later.

Please reconsider. Phive is not yet 1.0, but it works quite well and it simplifies the tools management a lot. Basically you end up with:
git checkout someproject.git
cd someproject
phive install
tools/dep target

No need to know which tools in which version the project needs any more. Unlike composer it avoids conflicts between the tool and project classes. It will verify the signatures of each tool before installing.
What you would need to do:

Sign the phar (I suggest adding a task for that to the project :-) )
gpg -u<EMAIL_ADDRESS>--detach-sign --output deployer.phar.asc deployer.phar
Upload the phar to
https://deployer.org/deployer.phar.asc

This would allow to install the phar using
phive install https://deployer.org/deployer.phar
Optionally, add the phar and the signature to the GitHub release. Phive can use the GitHub
as a repository for released versions. It simplifies the installation to:
phive install deployphp/deployer

Ok, will reconsider it. Also did you know what we have distributed version of deployer?
https://packagist.org/packages/deployer/dist

Digged a little bit GPG signing. Really complicated and how do you verify public keys? :)
Even if GPG adds more security, how many deployer’s users are into that stuff?
Can we rely on https and sha1(can be updated) signing?

I am also interested to use Phive to install deployer
Maybe this article https://medium.com/@tfidry/create-and-deploy-secure-phars-c5572f10b4dd can help you.


Undefined symbols for architecture armv7: "GULAppEnvironmentUtil"

[REQUIRED] Step 1: Describe your environment

Xcode version: 11.3.1
Firebase SDK version: 6.21.0
Firebase Component: InAppMessaging
Component version: 0.19.1
Installation method: CocoaPods (cocoapods-rome plugin)

[REQUIRED] Step 2: Describe the problem
When building the framework I get the following error:
Undefined symbols for architecture armv7:
  "_OBJC_CLASS_$_GULAppEnvironmentUtil", referenced from:
      objc-class-ref in FIRInAppMessaging+Bootstrap.o
ld: symbol(s) not found for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Steps to reproduce:
What happened? How can we make the problem occur?
Unzip the file and run make (cocoapods-rome plugin must be installed to build).
InAppMessaging.zip

Workaround for now. Add these lines to your Podfile file:
post_install do |installer|
	installer.pods_project.targets.each do |pod|
		if pod.name == "FirebaseInAppMessaging"
			pod.build_configurations.each do |config|
				if config.name == 'Release'
					puts "Linking missing 'GoogleUtilities' framework to #{pod.name}"
					config.build_settings['OTHER_LDFLAGS'] ||= ['$(inherited)','-framework "GoogleUtilities"']
				end
			end
		end
	end
end

Thanks for sending the fix and workaround along with  the bug!
It's merged to master and is targeted for the M69-6.23.0 release.


added option for date format in \today

This seems to be pretty complicated compared to the fact that package babel usually deals with this kind of stuff. In addition, it is often more suiten to set the date by hand. Package datetie2 has some useful features but i see none that is helpful here to be honest.

Sounds fine to me.  This is just my personal solution for when I was applying to American as well as non-american jobs.

I agree with @johannesbottcher .
If someone want to change a format of the date, just edit this code with your own style. (Hard-coded date and the command with other date package are suitable 👍


How to put a @Reference instance to spring managed container?

I know that the @Reference annotated field will be injected as a rpc proxy but there are some another places where we need to put a singleton Reference bean to spring managed container and then get the refernce by ApplicationContext.getBean(SomeService.class). Because in some place the @Reference can not be scaned and injected such as some short life beans or domain entities. There was a existing issue but was dubbox.

So your meaning is that the @Reference will put the annotated field to the spring container?

@amit-gshe , definitely yes.


Parse Array ConstArg

To allow writing specs like
#[flux::sig(fn (head: [i32; N]))]
pub fn test1<const N: usize>(head: [i32; N]) {}

#[flux::sig(fn (head: [i32; _]))]
pub fn test2<const N: usize>(head: [i32; N]) {}

am going to merge having removed the shady bit!

Fixes #887


[Question] Current place in code to change floating point resolution

I'd like to reduce the number of floating point digits produced by dump().
Where in json.hpp do I do that?
This old issue https://github.com/nlohmann/json/issues/768 says to replace std::numeric_limits<number_float_t>::digits10 at  line 6701 but digits10 is no longer at that line. I tried replacing this symbol in dump_float() but it didn't do the trick and when I tried replacing every occurrence of this symbol in json.hpp I got assertion failures.
Thanks!

I think I found a workaround, if I switch from float to double and then do this based on https://stackoverflow.com/a/11209187 before calling dump()
double ToTwoDecimalPlaces(double d) {
int i;
if (d >= 0)
 i = static_cast<int>(d * 100 + 0.5d);

else
 i = static_cast<int>(d * 100 - 0.5d);

return (i / 100.0d);
}

doubleliterals don't have any suffix in C nor C++. Is that suffix "d" an extension of your compiler?
https://en.cppreference.com/w/cpp/language/floating_literal

I've just now removed the d suffix from the floating point literals in the code snippet since as you point out, it's meaningless.

Unfortunately, while this workaround has been working for me, it's not working reliably for my colleague. Presumably due to some numbers not being representable exactly enough as a double.

It turns out my colleague just made a typo.
Yesterday I tried about 50,000 floating point numbers with my code and they all were formatted with the correct number of decimal places
However these numbers are all in the range from -60 to 60 so I don't know for sure that my code works reliably for every number.

It's possible that once numbers get large enough that there is considerably less floating precision left to represent the part after the decimal point, my code starts failing. This could be tested with a loop that covers the representation range of the double type.


How do I update the header counter when navigating to other pages?

Laravel Version: 8.12
PHP Version: 7.3
Laravel-admin: 1.8

Description:
Hello everyone!
The following misunderstanding arose. I added a pointer to the new notification in the bootstrap.php file.
Admin::navbar(function (\Encore\Admin\Widgets\Navbar $navbar) {   $Content = '
                                                       '.\App\Models\News::countRead().'
';   $navbar->right($Content); });
After switching to this notification, the server marks it as read, but the number of new notifications does not change, even when navigating to other pages. The question is - how to update this number when navigating to other pages?
Thanks for any help!
Steps To Reproduce:

The question has not gone away. Do you have any recommendations?


XMLParser() item list

As described in:
http://stackoverflow.com/questions/35631886/django-rest-xmlparser-cannot-parse-item-list
It appears XMLParser() does not support item lists. When parsing an XML list, the parser returns the last list item only (as a dict()). This behaviour is unexpected, and from what i can tell, not documented.
Example code follows:
XML = """<?xml version="1.0" encoding="UTF-8"?>
                <root>
                <item>
                    <ID>item_1</ID>
                    <Description>first item</Description>
                </item>
                <item>
                    <ID>item_2</ID>
                    <Description>second item</Description>
                </item>
                </root>
            """
from django.conf import settings
settings.configure()
from django.utils.six import BytesIO
from rest_framework_xml.parsers import XMLParser

data_stream = BytesIO(XML)
parsed_data = XMLParser().parse(data_stream)

print parsed_data

Which returns:

{'item': {'ID': 'item_2', 'Description': 'second item'}}

Given XML lists are common, and the code does not throw a warning/error; I would consider this a bug.

I had this same problem. My solution was to subclass XMLParser, give it a list of tags to consider as "list elements", and then override _xml_convert:
class XMLParserAttributes(XMLParser):
    list_tags = [
        'Classification',
        'BulletPoint',
        'Job'
    ]

    # ...

    def _xml_convert(self, element):
        # ....
        if children[0].tag in self.list_tags:
            data = []
            for child in children:
                data.append(self._xml_convert(child))

Why not include a 'item_tag_name' class property like there is on the XMLRenderer to make it easier to subclass without having to reimplement _xml_convert method?


Report error summary in pretty mode during normal compilation

Alternative to  #28196

OK. Having seen this in our own output, I can confirm that I don't like having Found 0 errors printed to my console all the time. Can we not print this if there's no other output?


Navbar not responsive

Mobile view breaks some containment and does not trigger top level flex-column/flex-row breakpoint
On mobile (sm breakpoint) we should have some nice vertically stacked menu items and the inline form/national dsa icon at the bottom
Menu template is in stl_dsa/templates/menus and base template is in stl_dsa/templates
Sass in stl_dsa/static
Feature branch at
https://github.com/stldsa/site/tree/layout

This is the smallest width viewport (320px), can you please elaborate more on this issue, because I don't seem to get it.
What looks off to me with the website on strain is the spacing between its sections, everything looks condensed.
Concerning the navbar, I think it would be much better if its elements were horizontally centered.

Hi yamani!  Thanks for joining the discussion.  The issue is in regards to the latest redesign, with changes that are committed in the repo and not deployed on the live site.  If you pull the latest commits and follow the def instructions you'll see what I mean!


Properties of  isn't attached correctly.

In the section Even More Customization, I found that the properties of <td> doesn't seems to be correct after rendering.

This is what your rendered markup looks like? It is weird that you have capitalized  and  elements rendered... those should be Reactable.Td and Reactable.Tr components, which render regular  and  html elements. If you are using Reactable.Td, you shouldn't have to enter your own  elements. Also, if you are rendering a  inside a react component, it will only render the attributes that React supports, which are listed here:
http://facebook.github.io/react/docs/tags-and-attributes.html

@thechrisproject if you look closer it looks like @dimotsai is using the [react dev tools][1], which display both the capitalized component names and the DOM nodes rendered by those components. So this is still a valid issue report :smile: thanks for taking a look at this though.

This looks like a resurgence of #106 and #100. @dimotsai, can you verify you're using the latest Reactable version?

@glittershark - Oops! Thanks for pointing that out. Color me embarrassed.

@glittershark, Yes, it is the latest version.
Looks like there is a problem in this for-loop (https://github.com/glittershark/reactable/blob/master/src/reactable.jsx#L253-L258).
            // Attach any properties on the column to this Td object to allow things like custom event handlers
            for (var key in this.props.column) {
                if (key !== 'key' && key !== 'name') {
                    tdProps[key] = this.props.column[key];
                }
            }

Since this.props.column is always a string, this for-loop will take every character as a element and attach it to a <td>.

ah, alright. thanks for narrowing that down. I've just pushed a commit that (I think) fixes the issue, can you try this out again on master and confirm that?
thanks!

It works. Thanks for your fix. :thumbsup:

Cool! Just pushed version 0.10.1 with this fix.


Optional parameter with jsdoc does not be respected by salsa intellisense

When I use jsdoc to create constructor the intellisense work well. Except a bug that it must obey parameter. It will not know type if the parameter is not all passed in. even it marked as optional by jsdoc

As the screenshot. air is optional by jsdoc

this work but

not

Testing with the latest release-1.8 bits and we seem to have fixed this. Note the various different syntaxes supported for optional params below (p2 - p4 are all optional below):

Please try again with the latest bits in the release-1.8 branch, and if you are still not seeing optionality on params appear problem, comment with more details (ideally some code I can cut/paste), and I'll look into it.
Thanks!

@billti I think you may misunderstand the issue a little
The problem is registering intellisense to constructor
Here is the code I use for test
/**
 * @param {string} p0
 * @param {string} [p1]
 */
function Test(p0, p1) {
	this.P0 = p0;
	this.P1 = p1;
}

var test = new Test("");

And this is result. When parameter is missing it then make an object created from that constructor to not know what it is

@RyanCavanaugh  @billti If you close issue then does you get notification?

Yes, but it is the weekend (and Monday is a holiday).

@billti OK then I'm apologize. Just not sure that it is or not. Because it still closed

So this is a somewhat different issue than what's being described; we should discuss the implications of fixing it.
Currently in TypeScript if you try to invoke a function (or constructor) and zero overloads match, we issue an error and return unknownType (which is effectively identical to any). This is the general pattern we use everywhere because it stops downstream errors.
However, if there are any overloads, we could give a better Salsa (and arguably a better TypeScript) experience by choosing the last overload in the set instead of unknownType. That would give you back what we generally believe to be the "most general" return type of the function, and you could start seeing downstream errors.
This is probably the right thing to do everywhere. e.g. today, even in TypeScript, if you say
function makeString(n: number) {
	return 'the number is ' + n;
}

let j = makeString();
let k = j.subtr(4);

you get an error on the initializer for j, but nothing on the incorrect spelling of substr. That's bad for Salsa (as shown above) and bad for TypeScript because you really do have two errors (not one), and now need to go through two compilation cycles instead of one.
@ahejlsberg any thoughts on this change and/or this pattern in general?

Open question with this approach: what to do about generic signatures?
function fn<T>(x: T, y: T): T { ... }
let x = fn(3, ""); // was x: any, now x: ??


Update 3d meshing routine

Create notebook with meshing example on a 3d slab from the data.
Closes #70
Closes #76
TODO

[ ] Reinstate tetgen for macos python 3.8

Closing this in favour of #82 now that #77 was merged (which was based off this branch).


fix: add backend.groups.description variable to serverless_neg

It seems description variable is being used in module like below.
# modules/serverless_neg
resource "google_compute_backend_service" "default" {
...
  dynamic "backend" {
    for_each = toset(each.value["groups"])
    content {
      description = lookup(backend.value, "description", null)
      group       = backend.value["group"]

    }
  }
...
}

But there's no description in variables.tf, and documentation.

@Sonins plz update template here and follow this doc to generate the code and submit.

@imrannayer Just fixed it, can you check?

/gcbrun

/gcbrun


JSON can't contain int over 53 bits

Despite this fact, the API has a bunch of int64, knowing that JSON stores all numbers in double precision floats, precision will be lost for approximately the 4 last digits on a big int64 number.
I think it will be an issue at some point, do you plan to fix it ? (workaround by splitting in two int32 maybe, or try to fit officially in int53/aka double)

The library already stores int64 as String and int53 as Number (aka double). Have you found some int64 number which is incorrectly marked as int53 in the TDLib API?

You can see that in our TDLib API scheme, from which all binding, including JSON binding, are generated, int53 and int64 are two different types. This is done exactly to mitigate mentioned issue.

So you mean that if I'm using JSON API, I should be aware that int64 fields in TL file are in fact String field in JSON ? Seems okay for me.

Yes. This is documented in description of TDLib JSON interface.


include shaded sketches-core into this jar, javadoc fixes

Coverage remained the same at 96.174% when pulling 14d9024c07160a38cb2e56789491e77c757b8c46 on shade-core into a72e650c829e72b7f5ae19369bb12fba3a024a1c on master.


Problem with CUDA 7.5?

The following minimal code doesn't work even if cupy installation seems to be fine
import cupy as cp x_gpu = cp.array([1, 2, 3]) l2_gpu = cp.linalg.norm(x_gpu)
Error message:

CompileException: nvrtc: error: invalid value for --gpu-architecture (-arch)

Note that I can successfully initialize the array on GPU, i.e. the code runs fine if I remove cp.linalg.norm.
Running the code on a  TITAN X (Pascal) GPU,
here is the output of nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2015 NVIDIA Corporation Built on Tue_Aug_11_14:27:32_CDT_2015 Cuda compilation tools, release 7.5, V7.5.17
Thank you in advance!

Thank you for the report.
This issue is planned to be fixed in the next release. https://github.com/cupy/cupy/pull/1055
(Or, upgrading to CUDA 9.0 can also solve the issue)

This issue has been fixed via #1055


FILTEROP_ERROR When Applying Kafka L7 Policy

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Attempting to apply a Kafka-aware L7 policy drops all traffic to Kafka:
apiVersion: cilium.io/v2
kind: CiliumNetworkPolicy
metadata:
  name: my-topic-policy
  namespace: kafka
spec:
  endpointSelector:
    matchLabels:
      strimzi.io/cluster: my-cluster
  ingress:
  - fromEndpoints:
    - matchLabels:
        app: my-app
    toPorts:
    - ports:
      - port: 9092
        protocol: TCP
      rules:
        kafka:
        - role: "consume"
          topic: my-topic

Cilium Version
1.12.7
Kernel Version
5.15.92
Kubernetes Version
1.25.8
Sysdump
No response
Relevant log output
Thousands of these logs
{"level":"warning","msg":"[[C135995] Cilium Network::OnIO: FILTEROP_ERROR: 2 bytes","subsys":"envoy-filter","threadID":"966815"}

Anything else?
No response
Code of Conduct

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

This can be closed. It's the same issue as #21813


Max vcf file size?

Hi,
I am using the web tool with WGS data. I faced issues with one of our tumor samples.
I am getting an error/
<html>` <head><title>413 Request Entity Too Large</title></head> <body> <center><h1>413 Request Entity Too `Large</

/
I assume, my file is too large? What is the maximum file size that can be used?
Thanks in advance!

Hi @oksalau,
The maximum file size is 1GB. Thanks!


Official support for rust-analyzer?

Given that Rust-Analyzer is the official LSP implementation for Rust, I think this important.
Win-RT crate says this:

WinRT is developed with and tested for the rust-analyzer VS Code extension. We want WinRT-rs to work well with rust-analyzer, so if you run into issues, please file a bug report so we can look into it. There are, however, a few things to note about WinRT-rs and rust-analyzer:

Is there similar guarantee for windows crate? Right now it's really difficult to use this crate with rust-analyzer.

If I hit CTRL+Space in ID2D1Solid it won't auto complete.
If I hit Ctrl+Mouse Click it doesn't go to definition on D2D1_BITMAP_PROPERTIES1, it won't even show me what struct fields it has.

To me this is so huge downside compared to winapi crate, so much so that I can't use windows crate over that.
I have this in my settings.json:
{
    "rust-analyzer.procMacro.enable": true,
    "rust-analyzer.cargo.loadOutDirsFromCheck": true
}

Our desire is for good rust-analyzer support. The windows crate is in preview and there is a great deal of work yet to be done. I would expect that we will eventually have more time to focus on IDE support.
Having said that, many of the problems that I've seen with the rust-analyzer extension are actually limitations with the extension itself. So if you would like to contribute or even just report issues then please do so here: https://github.com/rust-analyzer/rust-analyzer/issues


fix(#820): Illegal character in filename

Also ran into this issue with the jdt:// URLs, as reported in #820. Instead of using JDT-specific workarounds like #1030 and #821, let's fix the root cause.
Given a directory structure like %dir1/%dir2/%actual_%file', the shortening process should eliminate the % escaping. It currently shortens it to %/%/%%actual_%%file' instead of the correct '%%/%%/%%actual_%%file'. The unescaped %/ is an illegal char sequence.
To address this, it would make sense to run the escaping after shortening has been completed.

lgtm, thanks for your contribution.


update UpdateIssue function on issues.go (#6)

Added fields (now the structure is complete) on UpdateIssue function :

Confidential
DueDate
DiscussionLocked

One small comment, looks good otherwise!

Right ! Done !

Thanks @antony360! LGTM!


Nodes are always more than 2,000 blocks behind

a few things could be causing this:

slow disks
slow network
bad peers

given this ticket is almost half a year old, feel free to reach out again if you still need help.


Fix environment in light of numerous problems with conda and deps

HIGH PRIORITY ISSUE
We have been experiencing severe conda-related issues see eg https://github.com/ESMValGroup/ESMValTool/issues/1241 that is reflected in a lot of similar issues reported on the conda gitHub page; also issues related to slowness at installing packages, updating them etc; in a nutshell conda is being conda and crapping up periodically.
On top of it all there is a new set of problems related to ncl and r-yaml (see below for details).
To @ESMValGroup/esmvaltool-coreteam
This PR solves these issues but you must test it (with this PR CircleCI is finally fully working again but even if all works for me on my platform and on CircleCI this is not a prerequisite that there are no more problems on other platforms, hopefully all goes fine :crossed_fingers: ). To test please do the following:

grab the latest Anaconda3 installer at https://repo.anaconda.com/archive/Anaconda3-2019.07-Linux-x86_64.sh
nuke all previous anaconda3 environments and repositories;
install the new Anaconda installer by doing bash Anaconda3-2019.07-Linux-x86_64.sh
activate the base environment by doing eval "$(PATH_TO_ANACONDA/bin/conda shell.bash hook)" if you have a bash shell (check the name of your shell by doing echo $SHELL);
use the environment.yml file in this PR and recreate the tool's environment from scratch by doing conda env create -n esmvaltool -f environment.yml;
activate environment and run all tests

Post replies here if all OK or not all OK (add problem report if not all OK).
Explaining the changes

pin tiledb>=1.6.0 since the latest ncl needs libtiledb.so.1.6
switch to new anaconda3 from miniconda3 in the CircleCI docker image since miniconda3 is obsolete (>3 months old) see https://hub.docker.com/r/continuumio/anaconda3 vs https://hub.docker.com/r/continuumio/miniconda3
telling the CI env to install r-yaml without which generates the issue seen by @jvegasbsc in PR #1042 which is generalized to current CI env
a couple full path pointers to pip so that the runtime env knows what to do

Benefits from changes

enviornment builds seamlessly;
conda update -y conda works fine (note that the new Anaconda installer comes with conda=4.7.10 which is much newer than the older 4.6.14);
much faster and more stable conda package installation via the new base
:beer:

Thanks @valeriupredoi ! I will test this PR at DKRZ. Do we need to adapt the CONTRIBUTING.md and the documentation for this (miniconda -> anaconda) in this PR?

cheers @bjoernbroetz - I think the documentation and CONTRIBUTING.md should be fine since a new user will always grab the latest anaconda or miniconda, which in this case are the anaconda installer I posted above and the miniconda https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh from https://repo.anaconda.com/miniconda/ which you can see it is the latest ie 4.7.10

the flake8 error is an old one - have seen it during installation for a while now. Can you post the actuall test failure related to that moved function pls? Both myself and CircleCI don't see that issue. Cheers for testing, man! :beer:

Can you post the actuall test failure related to that moved function pls?

See PR #1244

right, bloody Mary and Joseph, the fix to the CircleCI documentation test took a while - OK all stuffs work fine now - test, people, pls - Bjorn has already but I need more of you to test so we're sure this thing is solid on all sorts of platforms :grin: :beer:

@rswamina since you've been running in all sorts of environment issues, it'd be cool if you had time to test this as well :beer:

@valeriupredoi - Does this require a fresh installation of all environments or can I also create a separate env? I will be testing this on JASMIN.

@valeriupredoi - Does this require a fresh installation of all environments or can I also create a separate env? I will be testing this on JASMIN.

It requires downloading the new Anaconda shell installer, installing it, and creating a brand new esmvaltool environment, as per my instructions in the PR description (pls follow those instructions). Cheers for testing it! :beer:

@rswamina note that Jasmin is going down for reboot in 29min

I confirm I installed the new Anaconda, built the env and updated conda, installed the tool and ran the tests with no problem on Jasmin, I reckon this PR is good to go soon, unless we get more volunteers to test - :camel:

Just back from leave and I've forgotten everything. Just to confirm @valeriupredoi, is this a new installation procedure and we need to work through this in order to get ESMValTool working?

@ledm this is the same installation procedure as old but note that the environment file has changed and it is highly recommended that you don't update the environment (which will not work) but  grab the latest Anaconda installer (see my comment at the top of the PR) and install anaconda from scratch, build the esmvaltool environment with the new anaconda and install the tool as per usual :beer:


Question about secret lifetime in "Enforce recommended guardrails for Azure Key Vault" initiative

Describe the bug
I've a question about the secret validty handling in the "Enforce recommended guardrails for Azure Key Vault" policy initiative
As far as I can see and understand the following parameters are both set to the value "90"

secretsActiveInDays: 90 days
This parameter denies the creation of a secret with a lifetime greater than 90 days ( (PolicyDefinitionRefrence Id: Deny-KV-Secret-ActiveDays )

minimumSecretsLifeDaysBeforeExpiry: 90 days (PolicyDefinitionRefrence Id: KvSecretsLifetime)
This parameter audits the lifetime of a secret which lifetime is shorter than 90 days (PolicyDefinitionRefrence Id: KvSecretsLifetime)
BTW: The parameter description doesn´t fit, seems to me copy from another parameter ?

So if I deploy this policy initiative I will never be able to create a compliant secret ?
Screenshots

@jdrepo it can be confusing :)
We default to the policy default parameter values provided by the authors, and the intent is that customer customize these according to their organization/workload requirements.
In this yes, you can still deploy a secret (with an expiry date of less than 90 days), but the second parameter will automatically flag as not compliant, as this serves as a reminder to renew the secret before expiry.
It doesn't break anything, but you are right, you would probably not want to leave the values as they are :)
We considered reducing the minimumSecretsLifeDaysBeforeExpiry to 30 days, but decided against, as most organizations do not regularly review policy compliance as part of operations.