[Feature Request] Localization List Alphabetized

Is there any way we can get the language list in localization remapping alphabetized?  Currently it seems to be a very long list of randomly listed languages.  It would be easier to navigate in order.
Alternatively, and probably more complicated, show only the languages used in the other translation tab.  Though not sure how feasible that is.

This seems like a fairly simple fix, I could take care of that.

That would be awesome!

I will have a look this evening :)

That is terrific, thank you, sir!

It seems to be fixed already (first entry in en, then it's alphabetical).


Move off of MoonJIT

MoonJIT has been deprecated for 4 years and we should switch to the latest LuaJIT 2.1

string.pack and related is a blocker
math.maxinteger, max.mininteger, math.tointeger(x), math.type(x), math.ult(m, n) are blockers, but mostly unused in rosaserver codebases. the FFI and LL/ULL literal suffixes which use the FFI
utf8 lib is a blocker, fixable with https://github.com/starwing/luautf8, but also mostly unused in rosaserver codebases
https://github.com/moonjit/moonjit/blob/master/doc/extensions.md#extensions-from-openresty-luajit2 has some highly uncommon stuff that doesnt transfer over

https://github.com/lunarmodules/lua-compat-5.3 could be used

I use string.pack a ton internally, however happy to swap over to FFI


use partition data source for arn formatting

what

Use aws_partition data source to construct ARNs.

why

This is required for govcloud users since govcloud uses a different IAM partition

@aknysh done

/codefresh run test

/codefresh run test

@aknysh this seems to be failing due to issues setting up the test harness unrelated to the code changes

@aknysh  any update on this? the build seems to be failing setting up the build harness

/codefresh run test

/codefresh run test

@RothAndrew is there something i can do to help get this merged? it's doesn't seem that the test failures are related to this code change.

Hi @woz5999 sorry for the delay. The repos are being migrated from Codefresh to github actions.
@osterman , can you do your magic to add the test PR to this repo?

/test all

/rebuild-readme

/test all

/test terratest


Crash, see description.

Assertion failed: (Val && "isa<> used on a null pointer"), function doit, file /tmp/llvm20150628-7874-1qnjbb2/llvm-3.6.1.src/include/llvm/Support/Casting.h, line 95.

When does it occur?

Happened once while I wrote code, but was unable to reproduce the crash. Hopefully we'll figure out which pointer it is.

Seems like the problem lies in either Diagnostic.cc or Cursor.cc. Did a grep isa *.cpp in clang/tools/libclang after cloning https://github.com/llvm-mirror/clang.git.

Or.. I'm not sure. Might be jucipp's source.cc that try read from libclang's translationunit or cursors while they are being processed. Might be an exception there I have not thought of yet:)

The problem was in source.cc, I had added a reference that sometimes was empty. Committing fix Tomorrow.

Fixed in latest commit to jucipp.


Terminal doesn't understand some git concepts

VSCode Version: 1.7.2
OS Version: Windows server 2012

Steps to Reproduce:

Open terminal in visual studio code in git repo
type in git reset --soft HEAD^ (should soft reset the last commit)
This is what I get:

More? y
fatal: ambiguous argument 'HEAD
y': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

From the error, I suspect this has nothing to do with VSCode and I confirmed that the example git command can be run correctly from the integrated terminal.
What happens if you run the same command in the project direcotry from a regular git enabled command prompt?

In Window's cmd.exe ^ is a special character that needs escaping HEAD^^. Alternatively HEAD~1 would work too. Not specific to VSCode.


Email Template Field Issue

On the site I'm using, a form submission gets emailed and shows the field entries in a table using the default mail template, however the labels are displayed exactly as they are in the field ID, with underscores in.
Is there any way I can give each field a different ID that gets shown as the field label instead?

Hello @mattpollard0
Currently is not possible from the plugin.
You can manually edit email template by going to: Settings > Mail templates > martin.forms::mail.notification and replace line 6.
Using twig replace function you can change labels underscores to spaces.
Hope it helps.

@skydiver Why not incorporate new tag like

So labels in mail will be taken from label tag if label tag exist. What do you think? It can be called data-label or data-maillabel etc. or maybe even just label.


Can't edit undefined values

Describe the bug
When the data contain undefined values, they can't be edited:

When I hover over the undefined value, the edit (and delete) buttons don't show
When I edit an existing value, I can't choose undefined as the new value

Expected behavior
I would expect I can edit undefined values in the same way I can edit null values.
Screenshots

Online demo
https://codesandbox.io/p/sandbox/json-edit-react-demo-forked-d2qmml

I'm not sure if this is a bug or a missing feature, but I think it makes sense to support editing both null and undefined values.

This was an intentional choice because undefined is not technically part of the JSON format. I could allow undefined values, but the problem is, if you use the "JSON text" edit feature, then what do we do with them?

This view just uses JSON.stringify() and JSON.parse() to present and parse the input, and that causes non-standard JSON values (like undefined) to be stripped out, which would be annoying.
I'm open to suggestions as to how to deal with this. There's a few possible options I can think of:

if there is an undefined (or any non-standard JSON) values, then we don't allow editing JSON text directly. But that adds some additional complexity and would still potentially be confusing
filter out non-standard values before JSON.stringify() and re-inject them after JSON.parse() of the input. Again, adds some complexity and potential confustion which I'm not sure is worth it.
temporarily turn undefined into "undefined" string when editing JSON, then turn it back when parsing. (Would mess up when someone actually has "undefined" as a string, but that's a pretty edge case)
I'm actually using JSON5 for parsing, to allow a "looser" input text. I could look at trying a different "loose JSON" library, or perhaps implement my own parser to handle non-standard values. But this would be a pretty big job, so would need to really justify it.

I probably won't do either of these without thinking about it some more, and getting a sense for what people would want/expect.
However -- i think you could probably implement a basic undefined editor yourself using Custom Nodes. You can add additional types, and use the condition to catch undefined values and render them as you see fit. I haven't tried this, but I'm happy to have a go if you're having trouble making it work.

First of all, thanks for the detailed answer!
I always thought of the value for this component (and other json-view-* components) as a JavaScript object, not a literal JSON value. If we go by the JSON definition, then the behavior makes perfect sense.
I played around with the editor a bit more and it looks like values that are no JSON-compliant are not editable (and will get removed when the whole value is edited as JSON). I think that's reasonable behavior.
I don't really need the functionality to edit undefined values, I've just noticed that it doesn't work as I expected. I just replaced the value with null for my use case, so from my point of view there's no need to add this functionality.
(Furthermore, in the case where the value passed to the editor could contain undefined, we can JSON.parse(JSON.stringify()) the value before passing it to the editor to avoid having an uneditable/undeletable values in the data.)
Feel free to close this issue (and thanks for the cool component!).

Thanks, good to know it's not a critical issue for now. Personally, I can't really see why you'd want to be having users edit data that has "undefined" values, as this would normally be used for things like configs, and undefined is not really a valid "value" (if it's "undefined", it shouldn't be there at all). But some people may have a use for it, I dunno.
I might just chuck in a Custom Node for handling undefineds though, leave it there as another example and if anyone wants to use it they can.

I'm going to close this one now. I've made a very simple little custom component and put it in the repo, which people can use if they want to be able edit undefined values:
https://github.com/CarlosNZ/json-edit-react/blob/main/demo/src/customComponents/Undefined.tsx
All it does is add "undefined" to the list of available types, and lets undefined values get changed to something else.


enable smoke tests

Description
enable smoke tests
Fixes CCXDEV-6166
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Testing steps
Check this pipeline, must work
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

lgtm


Add Department of Education Tasmania, Australia

Adds the three email domains belonging to Department of Education schools in Tasmania, Australia.
These domains are:

education.tas.gov.au
education.tas.edu.au
tased.edu.au

@sheabunge according to our rules (see https://www.jetbrains.com/student/license_educational.html) we can only add domains, that belong to educational institutions directly - no to Ministry of Education or similar entity. Therefore please reapply, providing the domain of the school, to which you belong. Thanks!

I understand what you're saying. however these are the domains that are used for student email - like<EMAIL_ADDRESS>The school websites look like http://education.tas.edu.au/academy/schoolname

@sheabunge could you somehow prove that? a relevant information available on webpage / document should suffice. Also please let me know if anyone but students / training staff can register emails in the mentioned domains. Thanks and sorry for the delay - the situation is somewhat complicated.

Nobody but the technical staff can register emails on the domain, not even regular staff or students.
Here's the website of one of the schools:
http://education.tas.edu.au/academy/elizabeth/Pages/Home.aspx
Here's the main page for Tasmanian colleges:
http://education.tas.edu.au/academy/Pages/Home.aspx
Here's a listing of all the school websites:
http://education.tas.edu.au/SitePages/Home.aspx
Hopefully this is sufficient. Thanks for working through this!

@sheabunge thanks for taking time on extra explanation - now it's all clear. Please mind the terms of use, when using the licenses though. I'm adding the domain.

Of course - thank you for sorting this out!


SSAO Banding and Lagging Severly

Godot version: 3.1.1 Stable
OS/device including version: Lenovo Ideapad 320 laptop, Intel UHD Graphics 620, Intel CORE i7 8th Gen, Windows 10
Issue description: Whenever I add the SSAO (Screen Space Ambient Occlusion) effect to my environment in 3D, the scene starts lagging beyond usability. (20-30 FPS in editor, 1-30 FPS in game.) Also, severe banding and artifacts appear especially when FPS is high and the camera is near meshes. Overall, this makes SSAO unusable. Plus, the effect looks really bad with lots of dots even without taking the artifacts into assessment. Unity works just fine.
Steps to reproduce:

Create a 3D scene.
Add a directional light, tilt it at a natural angle, and turn on shadows.
Add a few meshes occluding at sharp angles in shadow.
Add a WorldEnvironment node to the scene and turn on SSAO. The settings don't matter.
To increase the unwanted artifacts, up the Intensity properties.
Use a fly camera or orbit controls to view the artifacts, which are most visible in motion.

Video I took
Google Drive Link

Duplicate of #17568 and #34580.

Why do other engines do such a good job? In Unity, as long as I limit the vegetation, I could run SSAO at 60 FPS with little to no hitches, even on slightly lesser hardware. Godot dosen't even have vegetation. Also, Unity is pretty much artifact free, and light baking actually works on any mesh, so SSAO isn't even necessary. On Godot, only MeshInstances actually bake, and they are filled with artifacts. No baking for CSG stuff, or for plugins that generate mesh. Plus, the BakedLightmap node causes a crash on bake most of the time, and GIprobes are too slow for use in game, no matter the settings. Lastly, I can't even unwrap most mesh's UV2s, as the option often dosen't present its self.

Why do other engines do such a good job? In Unity, as long as I limit the vegetation, I could run SSAO at 60 FPS with little to no hitches, even on slightly lesser hardware. Godot dosen't even have vegetation. Also, Unity is pretty much artifact free, and light baking actually works on any mesh, so SSAO isn't even necessary. On Godot, only MeshInstances actually bake, and they are filled with artifacts. No baking for CSG stuff, or for plugins that generate mesh. Plus, the BakedLightmap node causes a crash on bake most of the time, and GIprobes are too slow for use in game, no matter the settings. Lastly, I can't even unwrap most mesh's UV2s, as the option often dosen't present its self.

@BlueCannonBall SSAO will be reimplemented in the Vulkan renderer (which will be in 4.0). It'll hopefully be more efficient there.


Various small improvements/cleanup

Coverage decreased (-0.009%) to 57.609% when pulling 74a0a9f2e03c50952b7124fa73a144c750fc6834 on dennisschagt:feature/misc-improvements into 9b289cd57bf8746bd665b17a1b06f798d6e47808 on newsboat:master.


Update Laravel51Compatibility.php for 5.5

Remove (array) requirement to allow this to work with laravel 5.5

https://github.com/backup-manager/laravel/pull/87
Already a more thorough request


Implement format/1 function

This PR implements format/1 function.
❯ jq 'format("html","uri","csv","tsv","sh","base64")' <<< '[1,2,3]'
"[1,2,3]"
"%5B1%2C2%2C3%5D"
"1,2,3"
"1\t2\t3"
"1 2 3"
"WzEsMiwzXQ=="

Awesome, LGTM! Thank you!


TINKERPOP-937 Extracted "provider docs" to their own book.

https://issues.apache.org/jira/browse/TINKERPOP-937
Moved from the standard reference documentation.  Added links between the docs where necessary. Published docs for review here:
http://tinkerpop.apache.org/docs/3.1.2-SNAPSHOT/dev/provider/
VOTE +1

@velo watching the appveyor build. looks like we got a clean one on the branch but a failed one on the pr...maybe just a fluke with hadoop? thought you might want to look at the logs....
if anyone is reviewing this, i wouldn't worry about the single appveyor failure - the other three builds were good.

Should be a race/timing issue...
Failed tests:
  GroovyProfileTest$Traversals>ProfileTest.g_V_sideEffectXThread_sleepX10XX_sideEffectXThread_sleepX5XX_profile:173 Duration should be at least the length of the sleep (59ms): 53

Tests run: 1722, Failures: 1, Errors: 0, Skipped: 821

VOTE +1

VOTE +1.


Attempt to switch from using non_generic to .type_params field on Function

This is a work-in-progress.

This won't work because the same type variable can be both generic and non-generic depending on the current context.


Update feature_request.md

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Update the template for feature request issue.
close #30
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual.
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Regression tests have to complete successfully.
[x] 5. If significant functionality is added, there must be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test.
[x] 6. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.

Job Test linux pull on 047c1aa : invalidated by @wangcj05

Just Git Feature Request template update, No code changes in this PR. It can be merged.


Add settings to support font size

Default font size is too small in atom 1.0.

+1
Small screen with large pixel density, it's hard to read.

It's supposed to be the same as your code. Can you share a screen shot?

Sure

Oh yeah this looks like a bug...

Yeah, I'm seeing the same thing.

+1 on this, really small on Retina display.

Yeah, forgot to mention I'm on a retina screen.

+1 here, retina screen as well

Not on retina display here.

Can some of you test the proposed line here https://github.com/webBoxio/atom-term2/commit/75c266e8b2680701c16ae9e2ed6dd0c42a2973a9#commitcomment-12635263 ? Thanks.


Add task whitelisting to pd run to only allow certain tasks to be run for testing

E.g. If i want to test symcts in the pipeline and inject a target, i really don't want to have to start all of the other tasks that are triggered by new targets. E.g. I don't care about building the target with other instrumentations, starting other fuzzers, etc. At the moment this overhead is still manageable but it won't be for long.

pd launch task job?

I'd like to test, e.g. target -> buildguy -> symcts -> asan_guy -> derive_path, are you suggesting I just launch them all in order? there can be many though. I'd like to basically say "execute this subgraph" of the pipeline.

We support disabling individual tasks (not exposed by the current interfaces but it's there), are you okay with that?

fe19399dad13f957687f2a8e0dd657c987e4c65a


Update Nodejs to newer runtime

Hi there!
Thanks so much for this PR and sorry for not getting back to you sooner. We've gone and actually bumped the version to 6.10 which was more recently supported. I am going to just close this PR but we really appreciate your involvement!
Thanks,

Chris<EMAIL_ADDRESS>


Update AboutUs page

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (e353ef3) compared to base (30ffec6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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

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


Changing box to bento/oracle-7.7

Change base box to bento/oracle-7.7
Add provisioner to add storage and install VBox Guest Additions
Add MEMORY parameter to move changes out of Vagrantfile
Add psadmin-plus to box during provision

This PR addresses #77, #67, #61.


Docs: make sure shell examples can be copied

... when showcasing shell commands to be run I think the $ prompts shouldn't be copy-pastable since one needs to remove them manually after clicking on the "copy" button ...
TODO: find all cases in the docs that have this issue and fix them :)

hi, can I help with this please, thanks


why not use socketpair to implement async?

Now uv_async_t must be initialized before uv_loop_t run. Why not use socketpair? We can new a new uv_async_t, and send to loop by socketpair.So we can post any uv_async_t  to loop.
### Why?Thanks!

There are more efficient ways than using socketpair. Look at how it's currently implemented: https://github.com/libuv/libuv/blob/v1.x/src/unix/async.c#L157 we use eventfd on Linux if possible, otherwise pipe() but saving a fd, etc.
You can initialize the async handle whenever you want, from the loop thread that is, so I'm not sure I fully understand that part of the question.

Now use async, we must init async before loop run. But sometimes, we  don't known how many asyncs do we need, so I need new a async when I need. Because QUEUE_INSERT_TAIL is not thread safe, I can't do this.
If use socketpair,I can new a async and set it to pAsync,   then I write pAsync to socketpair, in loop thread I  read this pAsync, so I post the pAsync between two threads safely.

You mean write the address of the handle to the write end of the socketpair and have the event loop thread read it from the read end?
The obvious problem with that approach is that socketpairs have finite (bounded) queues. When it's full, you either have to block until the other thread clears it or you lose the message.

Some os don't support SO_REUSEPORT. So I must bind in one thread, after accept client ,I put the client to other thread. In this way,how can I do?

I don't get how that's related... for general questions, please turn to https://github.com/libuv/help.


Add installation steps when running wiremock in docker

Summary
Please provide a docker run command example with the --extensions flag for this specific extension.
References
https://wiremock.org/docs/standalone/docker/#running-with-extensions

Unfortunately, there's no docker image for 3.0.0-beta available yet - and the extension uses the new extension API. Will definitely add it once there is a docker image with 3.0.0-beta .

merged


Resubmit: Wrap

This is re-submission of:
https://github.com/scalatest/scalatest/pull/1127
against branch 3.0.x.
One of the sections had an unescaped &, which trips up browsers when the
generated files are presented as XML/XHTML.
Preventively, this wraps the two longer

Sorry for the delay. I e-mailed the signed CLA today.

Thanks @biochimia , will move this PR under PR for Review list (from pending CLA).

I've also cherry picked the commit into 3.1.x. Pulling into 3.2.x has a problem currently, so didn't complete that yet.


Over-reverted commit

It seems 73f44245e5c7bfc37fbbd092ca9823a7c2c16e0a was reverted too much.
This and this are correct.

https://github.com/ThaUnknown/jassub/commit/03da58662ed58ddbaa34f511ee9b0f1a9d043690#comments
this comment was very missleading

thanks for putting up with my fuckups on this, been short on time recently and mostly doing things very blindly

published as 1.7.4

@dmitrylyzo is this issue now fixed? together with the corruption of bitmaps?


:sparkles: add claims and roles

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


Fixed #26293

Removed a tab from lines 61 - 68 so code will run even if PREPEND_WWW is
set to False.

Tests aren't passing (check $ ./tests/runtests.py middleware) and a new regression test to demonstrate that the issue described in the ticket is fixed is also required.

This is a bad patch by me.

Please send a new PR if you get something working, thanks!

Will do.


Support new mechanism of defining deprecated enum variants in OpenAPI

Description
Support new mechanism of defining deprecated enum variants in OpenAPI
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Seems like needs
  Run './gradlew :java-codegen:spotlessApply' to fix these violations.


Make ockam node stop (no args) interactive by asking the user to choose from a list of nodes to start (tuify)

Current behavior
When a user runs ockam node stop without any arguments, currently they are shown help text. Here's a screenshot of this.

Desired behavior
Change the behavior of this command so that it becomes interactive with the user when no arguments are passed. Note that this can only be allowed to happen if:

the terminal is interactive, and,
the --no-input or --quiet flags have not been passed, and,
no arguments have been passed in to the command.

This what the user flow should look like in interactive mode.

Use the tuify crate's select_from_list() function to ask the user to select from a list of nodes that have been started. They should be able to make multiple selections. If there are none, then simply display a message saying that none exist (not an error) and exit.

Once the user has made their selection(s), display a confirmation prompt letting the user know that that stop is about to be called with the nodes they have selected. Use select_from_list() to display this prompt. They should be able to make a single selection.

If they have made no selections then exit by displaying a message to the user letting them know they did not make any selection.

If they choose "YES", then run stop for each selected nodes, and display the results of the stop command for each nodes that they've selected.

If an operation succeeds then use the ✅ emoji as a prefix for the message.
And if it fails, then use the ⚠ emoji as a prefix for the message.

The following is a video showing a generic user flow that shows the user experience when a user is asked to choose a from a list of "things". Then an action is performed on each "thing". Finally status information is reported for each action that is performed on each "thing". Please note that this flow is not specific to this issue.

https://asciinema.org/a/612547

Implementation details

Please use the functions select_multiple, and confirm_interactively in this file.

Here's a similar PR to get some ideas from: https://github.com/build-trust/ockam/pull/6480

You can learn more about how to use the select_from_list() in the tuify crate here. Here is an example that is very similar to the steps you might have to take for this issue.

Here's a file you can look at to detect when a terminal is interactive or not.

Here's a file you can look at to see where the ockam node start command is implemented.

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

@nazmulidris I'll like to attempt this, please, can you assign it to me?

@Lumexralph That's awesome, this is all yours. Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds

@nazmulidris  I think I have finished working on this but I need to run it to check it out.
Is there a way I can do this?

@Lumexralph You can follow the instructions here to create some nodes and then start / stop them.
You can also follow this manual here to create nodes so that you can play w/ starting and stopping them.
To build and run Ockam command from source, you can check this out: README.md.

thanks @nazmulidris

@Lumexralph You are most welcome. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds 👍🏽


Jackson 2.4.0 conflicts with Spark

Unfortunately I think spark uses an older version of jackson, and you might bump into some issues using the jackson version (2.4.0) imported by wisp
See: http://vanwilgenburg.wordpress.com/2014/10/26/fix-the-classpath-in-spark/

Update: this may only affect older versions of spark. We are using jackson-2.4.0 to support scala 2.11, where as jackson-2.2.2 was the version used by spark compiled against 2.10

Any Jackson version after 2.2.2 has significant problems serializing or
deserializing certain classes (not just spark).  We must be critically
careful if/when we upgrade.
On Thu, Jan 8, 2015 at 1:28 PM, Austin Gibbons<EMAIL_ADDRESS>wrote:

Update: this may only affect older versions of spark. We are using
jackson-2.4.0 to support scala 2.11, where as jackson-2.2.2 was the version
used by spark compiled against 2.10
—
Reply to this email directly or view it on GitHub
https://github.com/quantifind/wisp/issues/4#issuecomment-69252454.

I am using this strategy to circumvent the problem:
    ("com.quantifind" %% "wisp" % "0.0.1")
      .exclude("com.fasterxml.jackson.module", "jackson-module-scala")
      .exclude("com.fasterxml.jackson.core", "jackson-annotations")
      .exclude("com.fasterxml.jackson.module", "jackson-module-jsonSchema")
      .exclude("com.fasterxml.jackson.datatype", "jackson-datatype-joda"),

I've elected to use json4s-native over fasterxml-jackson

FYI, in the spark-notebook, I've done that https://github.com/andypetrella/spark-notebook/blob/master/project%2FDependencies.scala#L47.
So it worked fine with 2.3.3.

set value of spark.executor.extraClassPath in SparkConf with path to updated jackson libs:
conf = SparkConf()
conf.set("spark.executor.extraClassPath", "/usr/hdp/<IP_ADDRESS>-169/spark/lib/jackson-annotations-2.4.4.jar:/usr/hdp/<IP_ADDRESS>-169/spark/lib/jackson-core-2.4.4.jar:/usr/hdp/<IP_ADDRESS>-169/spark/lib/jackson-databind-2.4.4.jar:/usr/hdp/<IP_ADDRESS>-169/spark/lib/jackson-module-scala_2.10-2.4.4.jar")
sc = SparkContext(conf=conf)


Allow to customize location of .eggs directory

Apparently the PYTHON_EGG_CACHE environment variable does not affect the location of the .eggs directory when building a Python package using a command like python setup.py build: https://github.com/pypa/setuptools/blob/38b9010bc3527e89561c7a985e0908a3071b7ff5/setuptools/dist.py#L745
My code lives in a read-only partition mounted into a Docker container and therefore the build fails:
...
  File "/usr/local/lib/python3.6/dist-packages/pkg_resources/__init__.py", line 1077, in obtain
    return installer(requirement)
  File "/usr/local/lib/python3.6/dist-packages/setuptools/dist.py", line 778, in fetch_build_egg
    install_dir = self.get_egg_cache_dir()
  File "/usr/local/lib/python3.6/dist-packages/setuptools/dist.py", line 747, in get_egg_cache_dir
    os.mkdir(egg_cache_dir)
OSError: [Errno 30] Read-only file system: './.eggs'

Setting the PYTHON_EGG_CACHE variable has no effect on this behavior, but I guess it should?

@tgpfeiffer Did you find a workaround? I would also benefit from such feature for slightly different reasons.
I have two docker containers that mount the same directory for local development purposes. They both do a pip install -e at the same time, but one them them fails because of a race condition; they try to create the setuptools egg at the same time which leads to the following error for one of them:
ERROR: Command errored out with exit status 1:
 command: /venv/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/app/setup.py'"'"'; __file__='"'"'/app/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info
     cwd: /app/
Complete output (27 lines):
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/app/setup.py", line 74, in <module>
    'Werkzeug[watchdog]',
  File "/venv/lib/python3.5/site-packages/setuptools/__init__.py", line 144, in setup
    _install_setup_requires(attrs)
  File "/venv/lib/python3.5/site-packages/setuptools/__init__.py", line 139, in _install_setup_requires
    dist.fetch_build_eggs(dist.setup_requires)
  File "/venv/lib/python3.5/site-packages/setuptools/dist.py", line 721, in fetch_build_eggs
    replace_conflicting=True,
  File "/venv/lib/python3.5/site-packages/pkg_resources/__init__.py", line 782, in resolve
    replace_conflicting=replace_conflicting
  File "/venv/lib/python3.5/site-packages/pkg_resources/__init__.py", line 1065, in best_match
    return self.obtain(req, installer)
  File "/venv/lib/python3.5/site-packages/pkg_resources/__init__.py", line 1077, in obtain
    return installer(requirement)
  File "/venv/lib/python3.5/site-packages/setuptools/dist.py", line 777, in fetch_build_egg
    return fetch_build_egg(self, req)
  File "/venv/lib/python3.5/site-packages/setuptools/installer.py", line 133, in fetch_build_egg
    wheel.install_as_egg(dist_location)
  File "/venv/lib/python3.5/site-packages/setuptools/wheel.py", line 102, in install_as_egg
    self._install_as_egg(destination_eggdir, zf)
  File "/venv/lib/python3.5/site-packages/setuptools/wheel.py", line 110, in _install_as_egg
    self._convert_metadata(zf, destination_eggdir, dist_info, egg_info)
  File "/venv/lib/python3.5/site-packages/setuptools/wheel.py", line 131, in _convert_metadata
    os.mkdir(destination_eggdir)
FileExistsError: [Errno 17] File exists: '/app/.eggs/setuptools_scm-3.3.3-py3.5.egg'

Because setuptools doesn't use PYTHON_EGG_CACHE in this case, I can't put the .eggs directory inside my containers which is what I'd really want to do. My current workaround is to create two directories .eggs_container0 and .eggs_container1 locally and mount them in the containers using volumes (docker run -v .eggs_container0:/app/.eggs). That's kind of clumsy and it messes up my local directories for no reason.
@jaraco What do you think is the proper fix? Would it be acceptable to create the .eggs directory in PYTHON_EGG_CACHE directory if the environment variable is set? If so, I could maybe submit a PR.

There's no relation between PYTHON_EGG_CACHE (the default cache location for extracting resources from zipped eggs) and the .eggs directory used by setuptools' fetch_build_egg for installing missing setup_requires/tests_require requirements.
setuptools is not multi-process / thread safe, you can't expect to be able to run concurrent commands without issues, which means that using editable installs is similarly not multi-process / thread safe, and you'll have to synchronize concurrent installs yourself.
Note that the issue is not just about .eggs, you could run into issues with concurrent generations of the egg-info metadata.

That makes sense. I'll try to find a synchronization mechanism then. Thanks.

@benoit-pierre Sorry for taking your time on this, however, by closing this issue, are you considering that installing .eggs directory somewhere else than the project directory is not a use-case setuptools maintainers want to address? Unless I'm missing something obvious, I don't think installing a python package in editable mode while its source being read-only is particularly illegitimate.
More specifically, if my current working directory is NOT where the setup.py lays, pip install -e /absolute/path/to/my/project/ would still create try to create the .eggs directory in the project's directory (and not in the current working directory). Isn't this a bug (maybe not)?

It's not a valid use case, because you need write support to generate the egg-info metadata. And yes, .eggs and .egg-info directory are generated in the source directory.

It's not a valid use case, because you need write support to generate the egg-info metadata. And yes, .eggs and .egg-info directory are generated in the source directory.

The point of this issue is that you could write them both in the current working directory (or somewhere else) instead of the project's directory if write isn't available there. Is there a specific reason why this would not be possible or why it would not be disable to implement it? Apart from saying "it's not a valid use-case" while not suggesting a portable solution to a use-case that does exists for at least two people (and I suspect a lot more).

@gilbsgilbs  @tgpfeiffer
I had a slightly different problem when specifying an install directory
$ easy_install --install-dir /opt/myapp  myapp.egg
<fail message>

it erred out with a message that it could only write to a valid site-packages directory.
As I workaround I set up a virtual environment and activated it
$ python3 -m venv --prompt myapp /opt/myapp
$ /source opt/myapp/bin/activate
(myapp)$ easy_install --install-dir /opt/myapp  myapp.egg
<success message>

As a result under /opt/myapp/lib/site-packages/lib were mostly many compact '.egg' files, and under '/opt/myapp/share were many compact wheel files.  The total memory used by the virtual environment only (containing python3, etc.) was 8mb, a small fraction of the total compact installed 'myapp' which came to 84mb.
Of course to call 'myapp' pre-requires another call to
'''
$ /source opt/myapp/bin/activate
'''
I don't grasp your exact usage requirements, but there seemed enough potential overlap I thought it might be worth posting this message.

I'd like to see this re-opened. I use separation between source and build directories for all my projects. "python -m build" with setuptools as a backend supports this workflow perfectly, except that the automatically generated .egg-info folder is created in my source rather than in my build directory. Since the egg-info is automatically generated, there is no reason for it to exist in my source directory. Of course I can add a line to my .gitignore, but it seems a cleaner and more modern strategy to simply not write anything into the source directory during the build process.

@wagenadl I'd like to add the use case of PyBind11 that creates a build directory for its C++ compilation and a top-level .egg-info directory for the wheel build. It would make for a much cleaner project setup if it was possible to generate the .egg-info where the other build stuff is.

There's some confusion about this issue. The reported issue is about the .eggs directory and not the .egg-info directory. The former is where Setuptools will download and build/install eggs for temporary use as defined by setup_requires and tests_require if not already installed. This functionality is deprecated and unused when using Setuptools as a build backend, and there's not going to be any changes accepted to expand the scope of this behavior.
There might be a separate issue regarding the .egg-info directory. I agree that Setuptools shouldn't be creating this file in the source folder. And in fact, it probably doesn't need to create it at all. Probably what should happen is the backend should be re-designed to generate an sdist with the metadata in a PKG-INFO file and/or pyproject.toml and maybe some other details gathered by the sdist command.
This work is probably blocked on first removing the legacy behaviors that are entangled with the backend build process (easy_install, package_index, ...). We can revisit that issue later after removing more of the tech debt.


Show creation code in trace

When new contracts are created, the creation code should be shown. The creation code is already returned by the trace RPC call, so this is a frontend-only change.

looks cool


I've got the same psnr during training unpaired HDR model

Hi, I've got the same problem as issue#54. I got almost the same PSNRs for test for different epochs, but I can't find any solution in issue#54! can anyone help me?

The reason is that there is no ground truth in the unpaired HDR experiment.

I've used the ExpertC images for the PSNR, isn't it right?


xsibatch can't find shaders defined in ArnoldShaderDef.js

As of right now, only physical_sky is defined in ArnoldShaderDef.js and that fails silently when rendering with xsibatch on Windows. Tested with testsuite, RoyalRender and just manual command. They all fail.
Why would this happen? Is it a known limitation of some sort?
@caron Do you have any idea?

The only piece of code I could find to explain this is shown below but I see no reason that wouldn't work...
https://github.com/Autodesk/sitoa/blob/52a0a4b35f51bccb6223a8206d04b40edc80071c/plugins/sitoa/sitoa.cpp#L103-L113

Can you explain the behavior a bit more so I can reproduce?

Can you explain the behavior a bit more so I can reproduce?

Just try and render a scene that have a physical_sky shader. The shader will not be exported to Arnold when rendering with xsibatch.
physical_sky is a special case where it exits early in auto shaderdef code for it to be manually defined in ArnoldShaderDef.js.
https://github.com/Autodesk/sitoa/blob/52a0a4b35f51bccb6223a8206d04b40edc80071c/plugins/sitoa/loader/ShaderDef.cpp#L554-L555
So far I've been unable to find why it's not working.

OK, I will try it later tonight.

So here's what I've found so far.

When running in interactive mode, ArnoldShaderDef.js is loaded first then Load() in ShaderDef.cpp because of the OnStartup callback.
When in batch mode, Load() is called directly which causes ArnoldShaderDef.js to be loaded after.

This is the only thing that differs the two.
Good news is that I think I have a work around, but I'm still not sure why the reverse load order is causing the shaders defined in ArnoldShaderDef.js to fail in batch.

I made a fix for this but shaders that are defined in ArnoldShaderDef.js needs to be explicitly skipped in ShaderDef.cpp. See the above commit for the fix.

I should note that in the process I removed most of the manual define of physical_sky from ArnoldShaderDef.js. Only the extra buttons at the bottom and the logic is defined there now, and the rest is handled by the auto shaderdef.
Now, only set_parameter is defined in js.


Non-module namespace merge

fixes #641
When namespaces with the same name are declared in non-module files, TS assumes they'll be merged somehow (the example in the bug wouldn't work as you can't import non-modules, but there are other ways they code could be merged).
This fixes that case by using an or assignment when a namespace is declared in a non-module file.
NS = NS or {}

Is there really a need to have any logic there? TS seems to use code equal to ns = ns or {} always when namespace is used.


Improve look-and-feel of the dashboard

General Goals:

apply new logo
update look-and-feel to work well with the logo
improve usability

:squid:


Feature: Storage Analysis

Idea
We can add a tool for storage analysis and drawing a graph helping users identify how files are taking up user's storage.
Similar products: TreeMapFs.
Files Version
v2.0
Windows Version
Windows 11 22449.1000

Good idea. Here is a another app to get an idea https://www.microsoft.com/store/productId/9PFXCD722M2C

Is this something that should be in the properties window?
Fyi @chingucoding

Fyi @mdtauk


[Dialogs] FileDialog - Progress indicator for all VFS modules

Currently, upload progress is only for uploads on OS.js deployed servers. Ever since alpha46 VFS modules all support this, but is not implemented in any way.
NOTE When copying from different sources (internal module => external module) VFS will first copy the file into memory then upload to destination. In these cases we would have to split the progress in two parts

Merged with #49


Fix parsing of a profile with 2 subsequent sliced properties

Greetings,
We're running into an issue while parsing the BEPractitionerRole profile (https://www.ehealth.fgov.be/standards/fhir/core/StructureDefinition-be-practitionerrole.html).
The error message is Invalid slice start before discriminator: SNOMED-CT (PractitionerRole.specialty:SNOMED-CT).
Diving deeper into packages/core/src/typeschema/types.ts where the error is thrown, it seems that the slicingContext of the previous property 'PractitionerRole.code' is only closed after handling the first 'PractitionerRole.specialty' element (in checkFieldExit). Because of this, no new slicingContext is opened for 'PractitionerRole.specialty' in checkFieldEnter.
I believe following changes fix this issue.
Kind regards

Hi @deklungel - thanks for submitting.  The fix looks good to me.  Would it be possible to create a more surgical test case / smaller StructureDefinition example?  The Medplum team can assist with that if it's ok with you.

Hey @codyebberson , I'll make an attempt, but honestly, I have no clue what the rules are for a StructureDefinition. Feel free to make changes :)

I've removed many elements that I believe are not relevant for the test case.
Feel free to update.
Kind regards


Client create fails with invalid key create_key

Hi,
We recently migrated from the official chef-server to goiardi and just stumbled upon an error when manually running "knife client create".
% knife client create delete_me -d -f delete_me.pem
ERROR: The data in your request was invalid
Response: Invalid key create_key in request body
% knife --version
Chef: 14.1.12

According to the chef-server API docs this create_key is valid [1], but goiardi does not know about it [2]. Currently this means for us that Terrform can not use the chef provisioner which is kind of a blocker for us. Any suggestion on how to quickly fix this? Or do you think this option could simply be added/allowed?
[1] https://docs.chef.io/api_chef_server.html#id24
[2] https://github.com/ctdk/goiardi/blob/f53c85261c0b82125ecb497370193eb68c2661a6/client/client.go#L371

We just compiled and run goiardi with this minimal fix of adding "create_key" to the validElements and everything seems to be in order now. If you prefer a pull request or anything just say so and we can send it to you within a day.

Omg, I only just saw this. (I seem to miss many github notifications in all the noise in my primary email account.) I think this is a newer allowed element is all, and to make it pass the old Chef 11 pedant tests I had to be pretty limiting in what I'd accept.
I'll add that and see if it breaks pedant horribly. If it does, I'll do something to make it work. Thanks for finding that.

I pushed it up yesterday, but that fix you made both works and doesn't appear to cause anything else to blow up. It's in both master and 1.0.0-dev now, and I'll follow it up with a teensy bugfix release as well. Thanks again for bringing it to my attention, good sir.


Add static analysis

This adds static analysis to all projects in the Agent.
The ruleset I picked is "All" from Microsoft. We can decide on a case by case basis what we want to remove.
For now, we will keep warnings as warnings but we should try to resolve everything and then enable Treat Warnings as Errors.
There is also this warning:
MSBUILD : warning : Post-build Code Analysis (FxCopCmd.exe) has been deprecated in favor of FxCop analyzers, which run during build. Refer to https://aka.ms/fxcopanalyzers to migrate to FxCop analyzers. [E:\github\vsts-agent\src\Agent.Service\Windows\AgentService.csproj] [E:\github\vsts-agent\src\dir.proj]
Which needs to be investigated.

Must be something wrong, from the logs I only see the Listener being analyzed. And they aren't surfacing as warnings in the UI.

Closing in favor of https://github.com/microsoft/azure-pipelines-agent/pull/2781


[fix #655] display error/warning count in EpubCheck result

fixes #655
To be able to implement this I had to change the Report interface and add two new methods getInfoCount() and getUsageCount().
However, this change may break existing Java implementations using the plain Report interface instead inheriting from MasterReport.
So this definately needs to be in a Minor release rather than a Bugfix release.

I'm confident with those changes and will merge them to master. No big changes anyways, but an Interface change (labeled it accordingly) which is okay since we release v4.1 with other Report interface changes as well...


V1.1

@artemrizhov Thanks! Now it is as it is supposed to be. I am relieved.
master and deploy branches are the leading branches and the old branch v1.1 is merged into master:


Expose ObservabilitySystem to Clients

This exposes ObservabilitySystem to clients by exporting it from the Workspace module.
Motivation:
When diagnostics were refactored, ObservabilityScope became necessary in order to do almost anything, but it is located in the Basics module, which is not formally a part of any of the library products. For now it is certain to be present and importable, but that is only because SwiftPM currently allows transitive dependencies to leak through to higher levels.

I don't have a problem with this particular change, but I wonder how much else is available only because Basics leaks through.  Is that intentionally something that isn't available to libSwiftPM clients?

@swift-ci please smoke test

I wonder how much else is available only because Basics leaks through

I do not really know. This was the only issue I encountered, but I did not attempt to audit all the possibilities. I am not sure there is an efficient way to do that at the moment.
If internal vs exported imports are ever fixed, this sort of thing will resolve itself.

I don't think we should be exposing all of Basics to clients since it mostly contains utility functions that are used internally.

If internal vs exported imports are ever fixed, this sort of thing will resolve itself.

I'm not sure that is true, since observability is something that clients need to deal with, so it'll always need to be part of libSwiftPM's public API.

I meant that when the compiler prohibits anything but an exported import from being used in public signatures, then all code will be naturally self‐auditing and @abertelrud’s question about other types will answer itself. (It seems pretty clear from various forum discussions how import is intended to eventually work; it just is not clear if it will ever actually happen.)

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

@swift-ci please smoke test macOS

thanks for bringing this up and the PR @SDGGiesbrecht, this is a good thing to finally clean up.
@abertelrud @neonichu and myself discussed several times how we could create a better separation of the public API for consumers of libSwiftPM vs. the one that has to be exposed due to the limitations in the module system visibility (ie across modules of the same package). generally speaking Basics is of the second camp - its types are public due to this limitation and not because they are designed to be part of the public API of libSwiftPM. Of course, this does not change  the question of how to use Workspace since it expects an ObservabilityScope in many of its APIs that can only be created via an ObservabilitySystem, leading to this PR.
Instead of exposing ObservabilitySystem which is designed to be internal, I wonder if a more future facing solution would be to have a new type and API in Workspace that is designed specifically for libSwiftPM users. Maybe something simple like (maybe too simple?):
extension Workspace {
    struct ObservabilityCollector {
        private let observabilitySystem: ObservabilitySystem

        init(_ handler: @escaping (String) -> Void) {
            self.observabilitySystem = ObservabilitySystem({ scope, diagnostic in
                handler(diagnostic.message)
            })
        }

        var topScope: ObservabilityScope {
            return self.observabilitySystem.topScope
        }
    }
}

It still leaks the ObservabilityScope type, but that is already leaked via Workspace APIs.
thoughts?

Yes, encapsulating it would be fine too. Do you want me to put it together? I am thinking a protocol would make it easier for the client to associate any accumulator storage. Something roughly like this:
public protocol DiagnosticObserver {
  func handleDiagnostic(_ diagnostic: String)
}

extension DiagnosticObserver {
  public func createObservabilityScope() -> ObservabilityScope {
    return ObservabilitySystem({ self.handleDiagnostic("\($0): \($1)") }).topScope
  }
}

protocol is nice so we can extend it in the future. not sure if creating an observability system every time you need a scope is ideal, maybe a combination of the concrete struct in my suggestion that take a protocol instead of a closure or something like that?


Version fixture migration

Replace the previous organic method of parameterising tests with version numbers with a method that is clearer and more scalable.
The documentation at the top of the versions fixture module explains the design of the new system.

(note that there are a few small TODOs that should be completed before this is merged, though the rest of the code can be reviewed before that point)

This is probably easier to review if #282 is merged first - #279 appears to have gone into a different integration branch (I've made the mistake of using too many branches and becoming confused as to which is which!)


Persian Font Using 'styled-components'

Dear all,
I must thank you very much for providing such a fantastic tool. However, I have an issue with Persian font face while aadding it under injectGlobal like this:
injectGlobal`
@font-face {
font-family: 'sans';
src: url('./assets/fonts/iran-sans.ttf');
}
`
when I import globalStyle from relevant path and reload the page, nothing has changed and style will be set as a default not iran-sans which I had set above.
What's wrong with this code?
Your prompt reply is highly appreciated in advance.

@GitHub1396 are you using it?
html {
    font-family: 'sans';
}

@evan-scott-zocdoc  this way works perfect. Thank you very much.

Use injectGlobal like this:
injectGlobal`
    @font-face {
        font-family: 'Your Font';
        src: url('/path/to/your/font.woff') format('woff');
    }

    html {
        font-family: 'Your Font';
    }
`

That did not work.

To reproduce: download any free font that is super visibly different from any default font. Use create-react-app to make a webpage. Display the custom local font. Now explain to me how you got it to display, because I cannot, nor can I find anything explaining why I cannot.


Disable custom benchmark caching

The benchmarks nightly job currently caches data using Github's cache. It turns out github-action-benchmark doesn't allow us to do this and publish to our slatedb-website repo; it's either/or. So, I'm disabling the custom download/save caching jobs, and removing external-data-json-path so that the GH action simply uses the JSON data that's stored in the slatedb-website repo when saving and comparing old results.

Optimistic merge.


Allow to provision masters for organization accounts

When provisioning masters they are saved under the cloud user account on the cloud. There should be also option to provision masters for organization accounts too.

kontena cloud platform join allows to do this.


The addon removes find features from the input gestures.

First, great addon!
I wanted to remap shortcuts for find features for browse mode to different shortcuts.
I typed the word "find" in the filtered by in the input gestures dialog, and the result was empty.
I removed the enhanced find dialog and searched again, and they appeared inside the browse mode.
Just out of curiosity, I reinstalled the addon, and searched again, and the result was empty.
I'm on nvda 2024.2.rc2.
Could you look into it?
Thanks so much!

Hello!
I also can reproduce it with NVDA 2024.2, tonight I will look on the code of NVDA 2024.2 to see if we missed something that was added to try to fix it.


Revel fails Hot Reload on Windows 10

Revel succeded to hot reload templates on changes, but fails to stop and restart after a change in the code (controllers, etc). Below -v output:
INFO  11:00:29 watcher.go:208: Watcher:Notify refresh state             Current Index=0  last error index=-1 force=false refresh=true lastError=false  INFO  11:00:29 watcher.go:257: Waiting for refresh timer to expire Changed detected, recompiling INFO  11:00:30    app.go:136: Killing revel server pid                 pid=16168  INFO  11:00:30    app.go:153: Waiting to exit ERROR 11:01:30    app.go:159: Revel app failed to exit in 60 seconds - killing. processid=16168 killerror=nil  INFO  11:01:30    app.go:129: Revel App exited                         state="exit status 1"  INFO  11:01:30    app.go:165: Done Waiting to exit INFO  11:01:30 harness.go:235: Rebuild Called INFO  11:01:30  build.go:285: Cleaning dir                             dir=tmp INFO  11:01:30  build.go:285: Cleaning dir                             dir=routes  INFO  11:01:30 source_processor.go:31: ProcessSource Parsing packages, (may require download if not cached)...INFO  11:01:30 source_processor.go:87: Reading packages                         parser=SourceProcessor packageList="[github.com/revel/revel/... github.com/revel/modules/static/... github.com/revel/modules/testrunner/...]"

Duplicated entry due to github 404.

Duplicated entry due to github 404.

Duplicated entry due to github 404.

Due to what?

Duplicated entry due to github 404.

Due to what?

#1521
Clicked two times the submit button because the first time I got a 404 page from GitHub instead of a confirmation page. So I closed #1520 and leaved #1521 open.

#1521
Clicked two times the submit button because the first time I got a 404 page from GitHub instead of a confirmation page. So I closed #1520 and leaved #1521 open.

Got it

Got it


Fixing conjugate based on scanner not the site

Fixing Complex Conjugate have to be triggered when processing GE scanner instead of Iowa site (Trigger by Scanner not Site)

Assigning to @Sarahhongqin for testing

Tested both (1) Yonni's Xe001 subject and (2) Duke subject. The results for both subjects look normal.


Languages Override Not Working

I'm using the new GRAV Beta ... suggestions?

Can you tell us a little more? Just changing the default_lang isn't necessarily enough. If your blog itself is multilingual, and your base language is en, then changing the default_lang to it won't change it.
I'm a new maintainer here and don't use Grav's multilingual features, but if you can give us some more details, I'll see what we can do.


hbx 版本在mac上 运行报错

16:42:06.517 ✘ [ERROR] Cannot start service: Host version "0.20.2" does not match binary version "0.20.1"
16:42:06.517 1 error
16:42:06.523 [plugin:vite:vue] The service was stopped


Add version: pnpm.pnpm.6 version 6.32.12

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run

@wingetbot waivers Add Validation-Executable-Error


Add custom data for glog.Logger

go 1.20, windows
gf version v2.4.1
I wish the "glog.Logger" module could add custom Fields like "logrus.fields" so I could output some data on the same line as the log content

try to use
// SetHandlers sets the logging handlers for current logger.
func (l *Logger) SetHandlers(handlers ...Handler)

usage in  manual:
日志组件-Handler

@niluan304 @920922786 Logger.SetPrefix


TagLinter: class doesn't exist /  aggregation doesn't exist

Describe the bug
Bug appears in VSCode extension.
TagLinter reports "class doesn't exist" when custom controls from a third-party library are added to XML views.
TagLinter reports "aggregation doesn't exist" when custom controls are added to XML views.
Also, the attributes are marked as invalid.
To Reproduce
Steps to reproduce the behavior:

Create a custom control
Include the custom control in the XML view

Expected behavior
The control, whether third party or own, should be recognized by the linter.
Additional context
The custom controls are written in TypeScript. The third-party custom controls in JavaScript.
In addition, the custom controls can be accessed in controllers without any problems and there are no other error messages either.

Hi,
Thanks for bug report.
Please provide minimal project example in order to reproduce the issue.

Structure:
├── frontend
|   ├── src
|   |   ├── components
|   |   |  ├── custom
|   |   |     ├── customControl1.ts
|   |   |     └── customControl2.ts

The custom controls are implemented in a single file with the following structure:
// Module

declare module "./customControl1" {
  interface $CustomControl1Settings extends $ControlSettings {
      //
  }

  export default interface ICustomControl1 {
      //
  }
}

// Class

/**
* @namespace project.frontend.custom
*/
export default class CustomControl1 extends Control {
  // control implementation
}

⇒ Usage in XML view:
<mvc:View
	controllerName="project.frontend.controller.test"
	xmlns="sap.m"
	xmlns:custom="project.frontend.custom"
>
  <Page>
    <VBox>
      <!--Taglinter: "CustomControl1" aggregation doesn't exist in "sap.m.VBox"-->
      <custom:CustomControl1 />
    </VBox>
  </Page>
</mvc:View>

As mentioned above, the controls are rendered and no other errors occur when they are used.
I hope this example is helpful.

I found the problem:
The file name of the custom control was written in Kebap case, changing the file name back to Pascal case, like the class name, solved the problem.

I found the problem:
The file name of the custom control was written in Kebap case, changing the file name back to Pascal case, like the class name, solved the problem.
Good. Keep the PascalCase, it's the same way as SAP uses for their classes :)
Are there any other problems? If no, this issue might be closed.


add json schema docs and json schema updates

update json schema

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.聿剑 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add an option in preferences to hide the titlebar from panes/dockwidgets if locked.

Description
This depends on #2854

@goanpeca  and @ccordoba12
Presently, the Lock panes features (Ctrl+Shift+F5) that exists under the View menu of the MainWindow only serves to prevent the dock widgets to be dragged around. I propose that instead of adding an option, we use this same Lock panes option to also :

Hide the titlebar
Hide the Close and Detach button

Changes to the code would be minimal to implement this feature, there is no need to create a custom title bar. Everything can be done within the Qt framework. I will do it if it is accepted. Below, i present a POC of what I am proposing.

I see this idea was already proposed/discussed in PR #2345, but as a new option on top of the lock panes feature.
I believe there is no need to add a new option.  Lock panes would need to be off by default however.

Jean, we agree the title is unnecessary for most plugins, but we show the current file path in the Editor title bar, and people is very accustomed to that. So you would need to add a label to the Editor plugin for that.
Other than that, I'd like to see a PR for this. I think connecting this to the Lock panes action is a terrific move and something none of us had thought before!

Alright thank you, I'll submit a PR soon.

Note to self: I'm waiting for PR #3824 to be closed before working on this one.

@jnsebgosselin, FYI, I just merged #3824 ;-)

@ccordoba12 Thanks for the heads up! :+1:


Fix ValueError in  get_submission() when doing int()

If the result of get_submissions() contains some invalid submissions, such as compile error,
it will cause get_submission() to get a non-integer value when fetching the runtime, which will cause a ValueError.

Hi @mbledkowski
PTAL

Thank you, for your commit. I currently do not have means to test this pull request. Have you tested it yourself, and you are sure that it works fine? @shownlin

I revised the commit.
We can see it works properly after add some logs for testing:


[BUG] NeoTree floating windows text is unreadable

Describe the bug

When i call any floating window in NeoTree (adding, deleting, etc.) it appears to be completely unreadable

Expected behaviour
Text in floating windows should be different color or something
Screenshots

this happened to me while setting transparent_background as true


backported fix for termiante task from #2056

Backported fix from #2056

@rickfish Please verify the changes made to the TERMINATE task here as per #2056. Pinging as per this comment.
Please let me know if you find any discrepancies.

Pull Request Test Coverage Report for Build 5076

54 of 56 (96.43%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 70.311%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/main/java/com/netflix/conductor/core/execution/WorkflowExecutor.java
30
31
96.77%

core/src/main/java/com/netflix/conductor/core/execution/WorkflowRepairService.java
1
2
50.0%

Totals

Change from base Build 5064:
0.03%

Covered Lines:
13525

Relevant Lines:
19236

💛  - Coveralls


Publishing views

Hi there,
Is there a description how to publish views properly ?
Publishing by  'artisan vender:publish'  works nice, but in which path I have to copy view files that Laravel takes the published views and not these from my Module/resouces/views folder. Views from packes will be published at "resouces/views/vendor/vendorname/packagename", is there an equivalent for modules ?

Nothing special going on - you would follow the same principles in your modules as you would a package - https://laravel.com/docs/5.7/packages#views
So, within your module's service provider you'd add something like the following to its boot() method:
$this->publishes([
    module_path('yourmodule', 'Resources/Views') => resource_path('views/module/yourmodule'),
], 'module');

Then you can run php artisan vendor:publish --tag=module and your views will be published to the views/module directory inside resources.


Enforce variable declaration coding style

Long time ago we agreed on multiple variable declarations for initialized variables, however we never made this style a rule. Some of the old code still uses the one variable declaration approach, same for a few submitted PR. So for consistency, let's enable the ESLint one-var rule to require multiple variable declarations for initialized variables per scope. Uninitialized variables can still be declared together (preferred) or separately.
I think this article describes pretty well the reasons of that move.

lgtm


I have a question about managing output

For example, print.js has been introduced in index.js. Why do I need to package and output print.js in the configuration file? Besides, I can still run normally without this print.js  in this demo.

Multiple entries should not only have one index.html？


Additional 12x12's for you.

Hello,
I'm also making 1bit 6x12 and 12x12, and 18x12 etc sprites for inclusion in a gaming-font for terminals. I love some of your sprites here.
I'm not convinced copyright is enforceable on 12x12 pixmaps but my work is declared CC0.  You may use any of these or just take ideas.
6x12_font_PUA_bitmaps_2023_06.zip
If I use some of your sprites what is your preferred attribution and name?  E.g. "Includes sprites from urizen by vurmux."

Hello!
Thank you for sharing. I will take some ideas from this tileset. But I prefer to draw tiles by myself, so I will not use tiles as is.
My tileset is under CC0 too. You can re-use it for free. If you will use tiles as is, I will be appreciated if you will write credits like "Includes sprites from Urizen 1Bit Tileset". But if you want, you can use them without credits.


Transferring pose or animation from one set of bones to another.

Possible to copy the pose of one rig to another, assuming it is an identical copy?

There is no built-in way to do this, but you could write a method that traverses the two node hierarchies and copies their transforms, and/or you could map the animation-target pairs provided by the asset to another node hierarchy. In my opinion, glTF's reliance on target node indices rather than node names makes this kind of retargeting harder than necessary, so although it is feasible in practice, making this use case easy is not a goal of this framework.


Add Android 11 support

Hello @jeff2900
Thank you for your Android 11 update! The app works beautifully again on the Shield. I tried to build your repo with Android Studio. I got errors when opening the build folder. Still missing files? Missing gradle configs to build? I was able to build the old repo without any problems. I'm not really familiar with Android apps, but I'm always trying something new.
Can you help me? regards

Hello,
Sorry, the build.gradle file commit is not done correctly. I did an update.
I compiled it with command ./gradlew task build  and Android-Studio 4.1.

Hello @jeff2900
Many thanks! Building works now... Regards


How to remove the cursor from touch devices

if (window.matchMedia('(pointer:fine)').matches) {
    const cursor = new MouseFollower();
}

See also #14

awesome thanks


Fixes - Footer #11

@JoshuaCode1
This looks great, the conflict was likely caused because of all the recent updates.
Let me know if you are facing issues with resolving the conflict

A newer pr was made by @JoshuaCode1 on the same issue. I am closing this one as outdated.


Replace all uses of MatcherAssert.assertThat()

Let's replace the MatcherAsssert.assertThat() in favor of Assertion (cactoos-matchers).

The job #944 assigned to @marceloamadeu/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat status

@0crat status (here)

@marceloamadeu This is what I know about this job in C3FFK3YAY, as in §32:

The job #944 is in scope for 1mon
The role is DEV
The job is assigned to @marceloamadeu/z for 18days
There is no monetary reward attached, it's a free job
The job doesn't have any impediments
The budget is 30 minutes/points
These users are banned and won't be assigned:

@fabriciofx/z: This user reported the ticket

Job footprint (restricted area)

@0crat refuse

@0crat refuse (here)

@marceloamadeu The user @marceloamadeu/z resigned from #944, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @marceloamadeu/z

The job #944 assigned to @lucasmessias9898/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat refuse

@0crat refuse (here)

@lucasmessias9898 The user @lucasmessias9898/z resigned from #944, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @lucasmessias9898/z

The job #944 assigned to @serranya/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job


Question about behavior of repetitions

Hi all,
I think there is a discrepancy between the documentation for repetitions in conjunction with code blocks and results of preceding labeled expressions and the actual behavior.
In particular the statement

Any non-number values, returned by the code block, will be interpreted as 0.

seems to be wrong.
A grammar like:
start = text:"a"|{ return 'some string' }|

does not return a number and hence should be equivalent to
start = text:"a"|0|

However, the parser accepts infinite input of "a".
This is probably due to the following lines, in particular |0 and the operator precedence?:
https://github.com/peggyjs/peggy/blob/d2f5bb928f5dbd7759e7d41601ea300e161fc362/lib/compiler/passes/generate-js.js#L490-L496
Any clarification is appreciated.

Yes, this is a bug. Need to wrap boundary|0 in parenthesis


[cdt] Add new port

Fixes https://github.com/microsoft/vcpkg/issues/28238.

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See docs/examples/adding-an-explicit-usage.md for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

I manually checked that the SHA in here is not affected by https://github.com/microsoft/vcpkg/issues/29288

All features are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Tested usage successfully by the cdt:x64-windows triplet
cdt provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(CDT CONFIG REQUIRED)
    target_link_libraries(main PRIVATE CDT::CDT)

All features are tested successfully in the following triplet

How did it find boost for you? 👀

All features are tested successfully in the following triplet

How did it find boost for you? 👀

Sorry, I tested the feature and usage of several PRs locally. After testing the usage of this port, I thought I had already tested his feature, so I wrote it like that and caused this error. All along, I was in Everything written on the user PR is tested locally. If the test fails, I will provide the user with an error log.
usage test:
1> CMake generation started for configuration: 'x64-Debug'.
1> The toolchain file has changed (CMAKE_TOOLCHAIN_FILE).
1> Command line: "C:\WINDOWS\system32\cmd.exe" /c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\ENTERPRISE\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 16 2019" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject42\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\test\source\repos\CMakeProject42" 2>&1"
1> Working directory: C:\Users\test\source\repos\CMakeProject42\out\build\x64-Debug
1> [CMake] -- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19045.
1> [CMake] -- The C compiler identification is MSVC 19.29.30146.0
1> [CMake] -- The CXX compiler identification is MSVC 19.29.30146.0
1> [CMake] -- Detecting C compiler ABI info
1> [CMake] -- Detecting C compiler ABI info - done
1> [CMake] -- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe - skipped
1> [CMake] -- Detecting C compile features
1> [CMake] -- Detecting C compile features - done
1> [CMake] -- Detecting CXX compiler ABI info
1> [CMake] -- Detecting CXX compiler ABI info - done
1> [CMake] -- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.29.30133/bin/Hostx64/x64/cl.exe - skipped
1> [CMake] -- Detecting CXX compile features
1> [CMake] -- Detecting CXX compile features - done
1> [CMake] -- Configuring done
1> [CMake] -- Generating done
1> [CMake] -- Build files have been written to: C:/Users/test/source/repos/CMakeProject42/out/build/x64-Debug
1> Extracted CMake variables.
1> Extracted source files and headers.
1> Extracted code model.
1> Extracted toolchain configurations.
1> Extracted includes paths.
1> CMake generation finished.

Build :
Build started...
1>------ Build started: Project: ZERO_CHECK, Configuration: Debug x64 ------
1>Checking Build System
2>------ Build started: Project: main, Configuration: Debug x64 ------
2>Building Custom Rule C:/Users/test/source/repos/CMakeProject42/CMakeProject42/CMakeLists.txt
2>CMakeProject42.cpp
2>main.vcxproj -> C:\Users\test\source\repos\CMakeProject42\out\build\x64-Debug\CMakeProject42\Debug\main.exe
3>------ Skipped Build: Project: ALL_BUILD, Configuration: Debug x64 ------
3>Project not selected to build for this solution configuration
========== Build: 2 succeeded, 0 failed, 0 up-to-date, 1 skipped ==========

Yeah it looks like the boost feature wasn't turned on in that test. Oh well :shrug:

@ex-purple, when I test the feature by command ./vcpkg install cdt[64-bit-index-type,as-compiled-library,boost]:x64-windows I get this error, please take a look:
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
    Command failed: F:/Feature-test/cdt/vcpkg/downloads/tools/cmake-3.25.1-windows/cmake-3.25.1-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j9
    Working Directory: F:/Feature-test/cdt/vcpkg/buildtrees/cdt/x64-windows-dbg
    See logs for more information:
      F:\Feature-test\cdt\vcpkg\buildtrees\cdt\install-x64-windows-dbg-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  ports/cdt/portfile.cmake:26 (vcpkg_cmake_install)
  scripts/ports.cmake:147 (include)

F:\Feature-test\cdt\vcpkg\buildtrees\cdt\install-x64-windows-dbg-out.log
[1/3] C:\PROGRA~1\MIB055~1\2022\COMMUN~1\VC\Tools\MSVC\1434~1.319\bin\Hostx64\x86\cl.exe   /TP -DCDT_EXPORTS -DCDT_USE_64_BIT_INDEX_TYPE -DCDT_USE_AS_COMPILED_LIBRARY -DCDT_USE_BOOST -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\x86-windows-dbg -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\include -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\extras /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1 /showIncludes /FoCMakeFiles\CDT.dir\src\CDT.cpp.obj /FdCMakeFiles\CDT.dir\ /FS -c F:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\src\CDT.cpp
FAILED: CMakeFiles/CDT.dir/src/CDT.cpp.obj
C:\PROGRA~1\MIB055~1\2022\COMMUN~1\VC\Tools\MSVC\1434~1.319\bin\Hostx64\x86\cl.exe   /TP -DCDT_EXPORTS -DCDT_USE_64_BIT_INDEX_TYPE -DCDT_USE_AS_COMPILED_LIBRARY -DCDT_USE_BOOST -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\x86-windows-dbg -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\include -IF:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\extras /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /GR /EHsc /MP  /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1 /showIncludes /FoCMakeFiles\CDT.dir\src\CDT.cpp.obj /FdCMakeFiles\CDT.dir\ /FS -c F:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\src\CDT.cpp
F:\Feature-test\cdt\vcpkg\buildtrees\cdt\src\1.2.0-dbb5670f32.clean\CDT\include\CDTUtils.h(49): fatal error C1083: Cannot open include file: 'boost/container/flat_set.hpp': No such file or directory
ninja: build stopped: subcommand failed.

@JonLiu1993 I added a patch
Please try again

[64-bit-index-type,as-compiled-library,boost] feature tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

but when I tested usage by cdt:x64-windows failed:
CMakeLists:
cmake_minimum_required (VERSION 3.8)

project(test)

find_package(CDT CONFIG REQUIRED)

add_executable (main "CMakeProject42.cpp")

target_link_libraries(main PRIVATE CDT::CDT)

CMakeProject42.cpp:
#include <iostream>
#include "CDT.h"

using namespace std;

int main()
{
	cout << "Hello CMake." << endl;
	return 0;
}

config:
1> CMake generation started for configuration: 'x64-Debug'.
1> Command line: "C:\WINDOWS\system32\cmd.exe" /c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\ENTERPRISE\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 16 2019" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject42\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/cdt/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\test\source\repos\CMakeProject42" 2>&1"
1> Working directory: C:\Users\test\source\repos\CMakeProject42\out\build\x64-Debug
1> [CMake] -- Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.19045.
1> [CMake] -- Configuring done
1> [CMake] CMake Error at F:/Feature-test/cdt/vcpkg/scripts/buildsystems/vcpkg.cmake:582 (_add_executable):
1> [CMake]   Target "main" links to target "Boost::boost" but the target was not found.
1> [CMake]   Perhaps a find_package() call is missing for an IMPORTED target, or an
1> [CMake]   ALIAS target is missing?
1> [CMake] Call Stack (most recent call first):
1> [CMake]   CMakeProject42/CMakeLists.txt:12 (add_executable)
1> [CMake] -- Generating done
1> [CMake] CMake Generate step failed.  Build files cannot be regenerated correctly.
1> 'C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\ENTERPRISE\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 16 2019" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject42\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/cdt/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\test\source\repos\CMakeProject42" 2>&1"' execution failed with error: ''C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\ENTERPRISE\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 16 2019" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject42\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/cdt/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\test\source\repos\CMakeProject42" 2>&1"' returned with exit code: 1'.

Pinging @ex-purple for response.

@JonLiu1993
Yes, cdt[boost] depends on Boost and I added boost to the dependencies. But I don't know how resolve this for the cmake config.
You just can prepend
find_package(Boost CONFIG REQUIRED)
before
find_package(CDT CONFIG REQUIRED)
and all must be fine.
If this should be solved in another way please show me how

Thank you for the PR !

Great to see CDT in vcpkg :partying_face:
Feel free to ping me if I can help with anything.


Fix/12784 name textfield moves out of visible area

Description
We re-adjust the layout based on keyboard notification twice once in the dynamicTableViewController and another in the TopBottomContainerViewController which causes the bug.
The current solution: introduces a flag to prevent keyboard subscription in the dynamicTableViewController if we know that the Viewcontroller will be part of a TopBottomContainerViewController
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-12784
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-11624
Screenshots
https://user-images.githubusercontent.com/15270737/165901369-5b853de6-1294-4805-be1b-efcb0db5a1a0.MP4

Does this also fix #4152?

Does this also fix #4152?

Yes it should

@Ein-Tim thanks for the heads up, the same approach should fix all similar issues, i just pushed a commit to fix your issue as well. 👍


Update README.md

Added POA Core, xDai Chain, POA20 Bridge from https://poa.net

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@igorbarinov thank you for your contribution!


swap EXPAND_FIT_WIDTH and EXPAND_FIT_HEIGHT

Proposal to fix TextureRect's EXPAND_MODE enumerations to make intuitive sense to end users
#75130

I think this needs some more organised discussion to avoid changing them without having arrived at a clear approach


Disconnect scribe response from storage completion

When using scribe, the zipkin-collector-process connects storage result to the scribe RPC result. For example, if storage fails, this results in a TryLater (and a log like.. "Sending TryLater due to NoHostsAvailableException").
The issue with this approach is that the connection to the scribe listener must be held until the storage operation succeeds or fails. This can overload, and has shown to in @gena01's site.
Another option could be to intermediate the scribe RPC from storage. For example, defer the act of receiving a scribe event from storing spans contained in it. If you can't issue an asynchronous request to store the spans, issue tryLater. This would allow the scribe call to return earlier.
This approach exists in the Kafka receiver (both scala and java), where processing tasks to store spans are deferred onto a fixed size pool of workers. This also exists in the http receiver (both scala and java) where a 202 is returned without blocking on storage completion.
There's a bespoke queue option in ZipkinQueuedCollectorFactory. This code is not used by the collector, but was used by lookout in the past in a custom build.

Note that if the underlying concern is overload, other work may be harder hitting.
For example, Kafka transport was easier to tune when it could accept multiple spans per message (and instrumentation sent multiple spans per message).
Also, the Cassandra layer currently works on one span at a time. This could be measured to see if batching helps reduce load or not.

current code is disconnected


[Carthage] Missing GDT symbols when installing Messaging and Remote Config

[READ] Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository
file a Github issue.

If this is a feature request make sure the issue title starts with "FR:".

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
Firebase support channel.

[REQUIRED] Step 2: Describe your environment

Xcode version: 11.2
Firebase SDK version: 6.7.0
Firebase Component: Core
Component version: 6.7.0

[REQUIRED] Step 3: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
This could be a description, log/console output, etc.
Today i updated xcode from 11.1 to 11.2, once I did this both of my projects now fail when building. I am getting the following errors:
  "_OBJC_CLASS_$_GDTTransport", referenced from:
      objc-class-ref in FirebaseCoreDiagnostics(FIRCoreDiagnostics.o)
  "_GDTLog", referenced from:
      -[FIRCoreDiagnosticsLog transportBytes] in FirebaseCoreDiagnostics(FIRCoreDiagnostics.o)
      _FIRPopulateProtoWithCommonInfoFromApp in FirebaseCoreDiagnostics(FIRCoreDiagnostics.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Relevant Code:
Cartfile:
# Firebase - FirebaseAnalyticsBinary must always be included #
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json"

Cartfile.resolved
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json" "6.7.0"
binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json" "6.7.0"

// TODO(you): code here to reproduce the problem

@paulb777

Paul is currently on vacation.
It looks like you're missing the GoogleDataTransport framework. Make sure GoogleDataTransport is linked along with the rest of the Firebase frameworks.

https://github.com/firebase/firebase-ios-sdk/blob/master/Carthage.md @morganchen12  which one of these binaries should I include

It should be included with the Analytics blob, so you should already have it in your project directory.

@morganchen12 exactly that's the issue. That's what I am trying to solve

please let me know how to fix this

Those frameworks should be located in the Carthage/Build/iOS directory. It looks like they may not be there because FirebaseAnalytics was somehow excluded from your Cartfile.resolved.
Can you share the output of running carthage install?

accidentally cropped it out here is an accurate .resolved binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json" "6.7.0" binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json" "6.7.0" binary "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json" "6.7.0"
response from running cartage update
*** Cloning lottie-ios
*** Cloning Kingfisher
*** Downloading binary-only framework FirebaseRemoteConfigBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json"
*** Cloning Alamofire
*** Cloning Mobile.iOS.MVI
*** Cloning gtm-session-fetcher
*** Downloading binary-only framework FirebaseMessagingBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json"
*** Downloading binary-only framework FirebaseAnalyticsBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json"
*** Cloning RxSwift
*** Checking out gtm-session-fetcher at "v1.2.1"
*** Checking out RxSwift at "4.0.0"
*** Checking out Mobile.iOS.MVI at "0.9.11"
*** Checking out Kingfisher at "4.10.1"
*** Checking out lottie-ios at "f7f7763dfb700d0c85b3f457cdcb377a1ad2ffd4"
*** Checking out Alamofire at "4.8.0"
*** xcodebuild output can be found in /var/folders/mr/9rj6sjr51qn_spbncxbqr2x8l1724v/T/carthage-xcodebuild.wSEFtH.log
*** Valid cache found for Alamofire, skipping build
*** Invalid cache found for FirebaseAnalyticsBinary, rebuilding with all downstream dependencies
*** Invalid cache found for FirebaseMessagingBinary, rebuilding with all downstream dependencies
*** Invalid cache found for FirebaseRemoteConfigBinary, rebuilding with all downstream dependencies
*** Valid cache found for gtm-session-fetcher, skipping build
*** Valid cache found for Kingfisher, skipping build
*** Valid cache found for lottie-ios, skipping build
*** Valid cache found for RxSwift, skipping build
*** Valid cache found for Mobile.iOS.MVI, skipping build
*** Downloading binary-only framework FirebaseAnalyticsBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json"
*** Downloading FirebaseAnalyticsBinary.framework binary at "6.6.0"
*** Downloading binary-only framework FirebaseMessagingBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json"
*** Downloading FirebaseMessagingBinary.framework binary at "6.11.0"
*** Downloading binary-only framework FirebaseRemoteConfigBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json"
*** Downloading FirebaseRemoteConfigBinary.framework binary at "6.11.0"```

running the update now

@morganchen12  that is an invalid command
Unrecognized command: 'install'. See `carthage help`.

Sorry, try running carthage bootstrap.

no worries, here's the result  carthage bootstrap *** Checking out Kingfisher at "4.10.1" *** Checking out gtm-session-fetcher at "v1.2.1" *** Checking out Mobile.iOS.MVI at "0.9.11" *** Checking out RxSwift at "4.0.0" *** Checking out Alamofire at "4.8.0" *** Checking out lottie-ios at "f7f7763dfb700d0c85b3f457cdcb377a1ad2ffd4" *** xcodebuild output can be found in /var/folders/mr/9rj6sjr51qn_spbncxbqr2x8l1724v/T/carthage-xcodebuild.COvmUT.log *** Downloading binary-only framework FirebaseAnalyticsBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseAnalyticsBinary.json" *** Downloading binary-only framework FirebaseMessagingBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseMessagingBinary.json" *** Downloading binary-only framework FirebaseRemoteConfigBinary at "https://dl.google.com/dl/firebase/ios/carthage/FirebaseRemoteConfigBinary.json" Failed to write to /Users/moriah.hammond/Documents/Mobile.iOS.App.Work/Carthage/Build/iOS/Protobuf.framework: Error Domain=NSCocoaErrorDomain Code=513 "“Protobuf.framework” couldn’t be removed because you don’t have permission to access it." UserInfo={NSFilePath=/Users/moriah.hammond/Documents/Mobile.iOS.App.Work/Carthage/Build/iOS/Protobuf.framework, NSUserStringVariant=(     Remove ), NSUnderlyingError=0x7fc1eff18ce0 {Error Domain=NSPOSIXErrorDomain Code=66 "Directory not empty"}}

Looks like cartahge bootstrap is failing. Once you fix the error, check your Carthage build directory for the GoogleDataTransport framework.

just updated

@morganchen12

@morganchen12 it's in my Carthage directory

Once it's in your Carthage directory, you should be able to add it to your target in Xcode. See step 3 in the Carthage documentation here.

@morganchen12 I did and added it, the project builds but I get this simulator error.  I also included my embedded frameworks to give a full picture

Try deleting your build cache and DerivedData and rebuilding.

ended up having to delete the Firebase.framework, the ensuring all other frameworks were set to Do not Embed, on the general tab under frameworks, libraries, and embedded content

Awesome, glad you could get it working!


Request: Textual Template

Requesting a template for Textual.

Cool! I hadn't seen Textual before. That would be awesome. I'll mark this with the "help wanted" label to see if anyone wants to try to implement it. (docs)


Please implement example app

I want to learn how it work from the example.
https://github.com/PhilJay/MPAndroidChart-Realm/blob/master/MPChartExample-Realm/src/main/java/com/github/mikephil/charting/realm/MainActivity.java

https://github.com/PhilJay/MPAndroidChart/tree/master/MPChartExample/src/com/xxmassdeveloper/mpchartexample/realm

This link is currently not working!!!


python311Packages.google-cloud-bigquery-datatransfer: 3.14.0 -> 3.14.1

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-cloud-bigquery-datatransfer-v3.14.1/packages/google-cloud-bigquery-datatransfer/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 289043 run on x86_64-linux 1

2 packages built:

python311Packages.google-cloud-bigquery-datatransfer
python311Packages.google-cloud-bigquery-datatransfer.dist


Clean Set Quota Fix

Clean up the Set quota to not generate super clocks.
Wait for all the nodes to complete, before returning success.

LGTM! You tested this right @arunthirupathi?

Yes, I will merge this in. I tested this on the staging environment, I posted some details on the internal RB about the testing.

Ok thanks (:


support table splitting process control when read table snapshot

This resolves a problem encountered in our production environment when there are too many tables to be read on the snapshot phase and each takes quite a lot of time ,  as a result, the process of data reading falls behind the process of chunk splitting too much.
To be more specific, say there are 100 tables to be read, the current process is:
1、 split all the tables into chunks one by one, which may last several hours, as it's a lightweight job.
2、read the chunks from procedure 1, which may last several days, as it's a heavyweight job.
When start reading the last split of the last table,  executing 'select from table where primary key > *** ',  too much data has accumulated for this chunk, because a long time has passed since it has been split, then the job will fail .
There are someone else who also encountered this problem (url) #1375
As a  solution,  we can control the speed of chunk splitting thread, to prevent it from going too much ahead. when it reaches the threshold value, it will wait for the data reading thread.

Hi @JustinLeesin , sorry for the delay of this PR. Could you please rebase it to latest master branch since there’s been lots of changes in Flink CDC repo since your original commit? Kindly reminder that com.ververica.cdc.connectors.mysql package has been moved to org.apache.flink.cdc.connectors.mysql.
cc @leonardBang


[harfbuzz] update to 1.7.4

Tested and compiles successfully. Should be able to close #2441 after this is merged.

Thanks!


Require https-proxy-agent only when actually needed

https-proxy-agent patches core Node methods in https for whatever reason, and that breaks unrelated code. In our app we don't use proxy, and the mere fact that puppeteeer requires the proxy package, it breaks other code.
See TooTallNate/node-agent-base#35 and sindresorhus/got#951

@googlebot I signed it!

Hmm it seems like the failing tests are not due to my changes apparently, can someone look at it ?

Thanks for putting together this patch! As you implied, this approach just moves the problem further down the line instead of fully solving it, so I'm closing this in favor of #5243 which does fully solve it! Hope that's okay :)

No problem at all, as long as it gets fixed i'm fine with whatever fix is used 😉


XML Import Error

While importing the XML file, I got the following error:

I have checked the files. But I can't find any duplicate files.
Thanks in advance.

Hey, I had a error before yours. Do you know how fix this?


cant see any enemy

hello , i played this game for 10 min but i didn't see any enemy! is the game have enemy?

Hi @babakmdi!
Thanks for the issue. You right! I didn't add any enemy for the game. Just added various platforms.