[SPARK-45515][CORE][SQL] Use enhanced switch expressions to replace the regular switch statement

What changes were proposed in this pull request?
This pr use enhanced  switch Expressions to replace the regular switch statement in Spark Java code refer to JEP 361
Example:
double getPrice(String fruit) {
  switch (fruit) {
    case "Apple":
      return 1.0;
    case "Orange":
      return 1.5;
    case "Mango":
      return 2.0;
    default:
      throw new IllegalArgumentException();
   }
 }

Can be changed to
double getPrice(String fruit) {
  return switch (fruit) {
    case "Apple" -> 1.0;
    case "Orange" -> 1.5;
    case "Mango" -> 2.0;
    default -> throw new IllegalArgumentException();
  };
}

This pr does not include parts of the hive-thriftserver module.
Why are the changes needed?
Using JEP 361: Switch Expressions can bring the following benefits:

More concise syntax: switch can be used as an expression, not just a statement. This makes the code more concise and easier to read.

Safer: In switch expressions, if we forget the break, there will be no unexpected fall-through behavior. At the same time, the compiler will check whether all possible cases are covered. If not all cases are covered, the compiler will report an error.

Easier to understand: The new switch expression syntax is closer to our decision-making pattern in daily life, making the code easier to understand.

Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions
Was this patch authored or co-authored using generative AI tooling?
No

JEP 361

Merged into master, thanks @srowen


Implement regression test suite for R scripts

[ ] Create test oracle files (host in DataSync, see RiboViz/bbsrc-nsf#5)
[ ] Write R functions, comparable to those in compare_files.py, to compare files output by R scripts.
[ ]  bam_to_h5.R
[ ]  generate_stats_figs.R
[ ]  collate_tpms.R

See #11 for background.

Requirements for regression, and unit, tests now added to issues to refactor individual R scripts: #32, #32, #61


played around with pygrametl yesterday. pretty nice! wish it was on github officially

What steps will reproduce the problem?
1. only use google to search pygrametl code base.
2. can't easy to submit patch and do code reivew.
3.

What is the expected output? What do you see instead?
moving to github, use social communication to enhance pygrametl user group.

What version of the product are you using? On what operating system?

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 5 Apr 2014 at 8:42

Would love to see pygrametl be on GitHub as well.


[Usersnap] "Back to the list" goes too far back

Context & versions
staging.sanchonet
Steps to reproduce

Actual behavior
GovTool takes you back to the main GovTool page
Expected behavior
GovTool should take you back to the category page.

This has been fixed. Closing


update macOS/chrome installation requirements

Informs users of pinentry-mac requirement, and necessary edits to gpg.conf and gpg-agent.conf.
Closes #205 .

Thanks! I changed your text a little bit, namely:

Chrome -> browser since the issue is exactly the same in Firefox.
There are some people who don't want to install gnupg (they have some alternative provider), for them it is important to make sure /usr/local/bin/gpg exists, for example as a symlink. If they have no admin rights to create symlink, it is also possible to patch the browser launcher.

Looks good to you? Merging after you confirm.

Yup, all looks good. Didn't realize it was an issue with Firefox too.
Your own user typically owns /usr/local/bin/ on macOS, so hopefully the workaround won't be necessary.

Good news - and thanks once again!

I still can't manually build this from source Mac, I can't find the instructions.
I tried to brew install dep but then I get this error when running make on v2.0.22 tagged release branch:
λ ~/browserpass/ make
yarn
yarn install v1.9.4
[1/4] 🔍  Resolving packages...
success Already up-to-date.
✨  Done in 0.30s.
dep ensure
/Users/jack/Dropbox/chromium-extensions/browserpass is not within a known GOPATH/src
make: *** [deps] Error 1

Because it's golang, projects cannot be cloned in arbitrary locations, golang has a concept of GOPATH - a special place for all projects.
This should have been mentioned in CONTRIBUTING.md, I'll add this later today.


[feat] Add batch rerun and stop for process instance

expand entry point executors/execute and make it support batch operator

Codecov Report

Merging #10013 (6c19691) into dev (bc1c15b) will decrease coverage by 0.04%.
The diff coverage is 15.00%.

@@             Coverage Diff              @@
##                dev   #10013      +/-   ##
============================================
- Coverage     40.81%   40.77%   -0.05%
  Complexity     4604     4604
============================================
  Files           834      834
  Lines         33832    33875      +43
  Branches       3745     3756      +11
============================================
+ Hits          13810    13811       +1
- Misses        18701    18742      +41
- Partials       1321     1322       +1

Impacted Files
Coverage Δ

...inscheduler/api/controller/ExecutorController.java
28.30% <5.55%> (-11.70%)
:arrow_down:

...uler/api/controller/ProcessInstanceController.java
72.09% <100.00%> (ø)

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <100.00%> (ø)

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.81% <0.00%> (-0.71%)
:arrow_down:

...er/server/master/runner/WorkflowExecuteThread.java
8.04% <0.00%> (-0.21%)
:arrow_down:

...nscheduler/service/process/ProcessServiceImpl.java
30.85% <0.00%> (ø)

Continue to review full report at Codecov.

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


Chapter 3: db.t2.micro instance type is deprecated

For creating a MySQL database on RDS, it is no longer possible to create databases using the db.t2.micro instance type. It worked for me after changing to db.t4g.micro.

Thanks for the report. AWS occasionally deprecates instance types in certain regions; using newer instance types as you did is the right solution!


Implement simplest pizza slicing algorithm

All infrastructure issues are almost completed, so it's time to implement the actual algorithm. See TaskDescription.pdf. An MVP is an algorithm that slices the example.in and passes checks on the task submission page.

A draft in a draw io file and descriptions in the project's wiki Simplest-pizza-slicing-algorithm(bruteforce)


Inconsistent/confusing permission constants

There are a number of inconsistencies with the API I have come across:

Currently, to request read AND write access on iOS, we use constants called READ_CALENDAR and READ_REMINDERS. This is quite confusing, since there are constants named WRITE_CALENDAR and WRITE_REMINDERS, which implies the read constants are in fact read-only.

The API documentation says that the READ constants "Represents the permission state for reading", when in fact they represent the permission state for both reading AND writing on iOS. This adds to the confusion.

Looking at the source code, in the current implementation the reminders constants do nothing (on iOS at least). Full access is requested with either constant.

On Android, requesting READ_CALENDAR permission does NOT provide write permission as well, in contrast to iOS.

The fix
I strongly suggest creating a separate set of permission constants for iOS and Android along the lines proposed above, that clearly indicate exactly what the permission is providing.
For example:
IOS_READ_WRITE_CALENDAR
ANDROID_READ_CALENDAR
WRITE_ONLY_CALENDAR  // works the same on both platforms
You may be thinking, "That requires the developer to have separate code for each platform." If we want both read and write permission (without request all permissions), we have to use separate code in any case.

@ebarooni Any thoughts on this?

@aparajita I see your point and I was actually thinking about your remarks. The problem is the behavior of iOS and Android are slightly different here. on iOS we have write only and full access (which is read and write permissions together). But on Android we have read only and write only.
Im always in the favor of providing exactly the same experience for both iOS and Android (or as much as possible). In this case I found what Android provides, a bit more convenient. Suppose we had defined write only and full access permissions (instead of what is implemented now), a developer on Android that only needs read access (for any reason) is forced to also define and request write access. On the other hand a developer on iOS will not find him or herself in a similar situation.
I would propose the following:
Firstly to provide further information on platform specific behavior of both readCalendar and writeCalendar. Then to implement two new methods, requestWriteOnlyCalendarAccess and requestFullCalendarAccess as alternatives to the already existing implementation. Both methods return Promise<void> and throw an error if unsuccessful. In this case developers can go for either or mix and match the solutions. Finally over the time either to drop either of the solutions or let both coexist.
For the reminders also you are right. There is less flexibility on iOS and there is in reality only full access or nothing. But i thought I try to match the defined permissions from the calendar by faking them. And finally I propose requestFullRemindersAccess which similarly returns Promise<void> and throws an error in case failed.
Let me know what you think about this proposal.

Im always in the favor of providing exactly the same experience for both iOS and Android

Me too, but Capacitor itself always separates config properties by platform when there is divergent behavior.

Firstly to provide further information on platform specific behavior of both readCalendar and writeCalendar

The way it is now is creating too many opportunities for mistakes by the developer, because the burden is on them to read and remember every detail of how the constants work on different platforms.

Then to implement two new methods, requestWriteOnlyCalendarAccess and requestFullCalendarAccess as alternatives to the already existing implementation

You would have to add one more, requestReadOnlyCalendarAccess, which would throw unimplemented on iOS. That is possible on Android.
If you replace the existing functions (and the corresponding check* function) with these functions, you don't need the constants at all. To me this is a much more elegant solution, as it avoids the confusion with the constants, and either works identically on both platforms or doesn't work at all on one platform (like selectCalendarsWithPrompt).

Both methods return Promise and throw an error if unsuccessful

There are two choices:

Return a boolean to indicate if the access was granted or denied, and throw an error if an actual error occurred; or

Throw an error and put an enum string constant in the .data property of the error, which indicates what caused the error. CapacitorException supports a free-form .data object, which could be checked to figure out what the result was.

If you imagine the code to handle those cases, the first option seems much cleaner to me both from the perspective of the plugin code and the user's error handling code.

i thought I try to match the defined permissions from the calendar by faking them

There is no need to force symmetry between calendar and reminder permissions. On the surface it seems to simplify matters, but if there is divergent behavior, in practice it will only complicate usage. I would propose defining separate functions that conform to the actual permission being requested (along the same lines as what I proposed for the calendar) and skip the constants.

Here's an example of why the calendar constants are quite unfriendly. If I'm just interested in read/write access to the calendar, I have to do all of this:
// Check if the user has granted permission to read and write to the calendar.
export async function checkFullCalendarPermission(): Promise<PermissionState> {
  if (!Capacitor.isNativePlatform()) {
    return 'denied'
  }

  try {
    const { result } = await CapacitorCalendar.checkPermission({
      alias: PluginPermission.READ_CALENDAR,
    })

    // If read or write permission is denied, return 'denied' immediately.
    if (result === 'denied') {
      return 'denied'
    }

    const readPermission = result
    let writePermission: PermissionState

    // On iOS, READ_CALENDAR includes write permission.
    // On Android, we have to check write permission separately.
    if (Capacitor.getPlatform() === 'ios') {
      writePermission = readPermission
    } else {
      const { result: writeResult } = await CapacitorCalendar.checkPermission({
        alias: PluginPermission.WRITE_CALENDAR,
      })

      // If read or write permission is denied, return 'denied' immediately.
      if (writeResult === 'denied') {
        return 'denied'
      }

      writePermission = writeResult
    }

    // Resolve the permissions into a single permission state.
    // 'denied' has already been handled. At this point the permission
    // will either be 'granted' or 'prompt*'. 'prompt*' takes priority.
    if (readPermission.startsWith('prompt')) {
      return readPermission
    } else {
      return writePermission
    }
  } catch (error) {
    captureException(error)
    return 'denied'
  }
}

@aparajita I think its not a good idea to remove the constants at this point because it would be considered a breaking change. However, like we discussed I will implement the following methods:

requestWriteOnlyCalendarAccess
requestReadOnlyCalendarAccess  (Android only)
requestFullCalendarAccess
requestFullRemindersAccess

But instead of returning true or false like you suggested, I would return PermissionState which is almost used everywhere. I believe a boolean would not be sufficient to cover all the possibilities.

I think its not a good idea to remove the constants at this point because it would be considered a breaking change

You could deprecate them.

But instead of returning true or false like you suggested, I would return PermissionState which is almost used everywhere

Makes sense. Better than throwing.


Support multi-character delimiter

Currently only single-character delimiters are supported. It would be an overhaul of the parser (Tokenizer), but consider adding support for multi-character delimiters.
Originally raised as Feature Request 14 on SourceForge.

Last time I looked at this feature there was only a minor performance degradation. So fine by me.


fix: JSDoc - fixes #28

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[x] file not found errors are benign in lint

It's pretty normal for files to be missing on disk but present in source control.
Thanks @dariorussi for reporting!

Is dario using sparse checkout? Seems fine but maybe we should log that a file is being skipped because its contents aren't present and I wonder if there's a way to ensure that this is a hard error in a place like CI? (assuming we want to ensure that we don't have a sparse checkout in CI)

No, his issue was a missing file in the middle of a rebase.

@bors-libra r=bmwill

:pushpin: Commit c10cb48 has been approved by bmwill

:hourglass: Testing commit c10cb4852fc064e8a4e98902764fdc2c89bd19f4 with merge d8dadc4d901dd85707b1abc719c5afd2b0baf546...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: bmwill
Pushing d8dadc4d901dd85707b1abc719c5afd2b0baf546 to master...


[add-resource] New Plugin/Theme

I would like to add my new plugin/theme to the Grav Repository.
https://www.twelvetone.tv/docs/developer-tools/grav-plugins/grav-admin-media-move-plugin
Here are the project details: https://www.twelvetone.tv/docs/developer-tools/grav-plugins/grav-admin-media-move-plugin

Will add this soon.

URLs were not pointing to github but i found it.  Added.

Thanks, I pasted the same url twice...
https://github.com/Flamenco/grav-admin-media-move


Godot editor does not work with El Capitan dual full screen feature

El Capitan (latest OSX) features a custom full screen feature where you can run two applications at the same time in a split split full screen mode.
Godot Editor does not support this because it does or cannot reorganize the UI dynamically . All the components and the layout is fixed and stiff. It should squish itself as much as possible create a horizontally thinner more compact look instead.

I'm out of OSX so, any contribution welcome
On Wed, Jan 13, 2016 at 10:04 AM, trollworkout<EMAIL_ADDRESS>wrote:

El Capitan (latest OSX) features a custom full screen feature where you
can run two applications at the same time in a split split full screen mode.
Godot Editor does not support this because it does or cannot reorganize
the UI dynamically . All the components and the layout is fixed and stiff.
It should squish itself as much as possible create a horizontally thinner
more compact look instead.
[image: screen shot 2016-01-13 at 8 01 41 am]
https://cloud.githubusercontent.com/assets/6987027/12294956/02bcaaf6-b9cc-11e5-9450-20faf0e4e889.png
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3337.

Does the issue occur when you just squash the window manually? (e.g. no fullscreen, etc.)

@bojidar-bg Yes .

I guess Godot Window does not like being squashed that much :)
On Wed, Jan 13, 2016 at 11:15 AM, trollworkout<EMAIL_ADDRESS>wrote:

@bojidar-bg https://github.com/bojidar-bg Yes .
[image: screen shot 2016-01-13 at 9 15 03 am]
https://cloud.githubusercontent.com/assets/6987027/12296512/23df35f0-b9d6-11e5-8be8-23b01608d5b0.png
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3337#issuecomment-171302467.

@trollworkout hm.... that's strange. I can't reproduce it this way under linux, unless I go to sub-768 px size (e.g. when top row buttons start overlapping)
Maybe you can somehow try on a more recent commit?
@trollworkout ~ Also, is this only on very squashed window sizes, or on all sizes smaller than the initial? (also, is it reproducible in the other direction?)

@reduz I guess not :)
@bojidar-bg
I can try a new commit but I think from my knowledge I am running the latest OSX one available unless I have to build my own TT_TT
Also looking closer at the screen it seems there is a graphical glitch as well. All the way at the top of the window there is a 20 pixel or so band  of gray space before the engine interface is laid out.

Also looking closer at the screen it seems there is a graphical glitch as well. All the way at the top of the window there is a 20 pixel or so band of gray space before the engine interface is laid out.

That's fixed in master. I guess you'll have to try building the engine yourself ;) It's relatively easy on Linux and Windows, so I'd expect it to be fine on OSX too.

Can't reproduce on linux either. This is the minimum size without this behaviour:

@akien-mga  I will wait around for a more fresh build or see if I can build one myself before trying again. Is not imperative that I have this feature although would be quite handy in doing two things at the same time :)  I like to watch movies or gameplays while I do work

I wish I was on linux so much easier to compile and most apps are always fresh . Macs tend to lag behind kinda bothering me right now.

Guys please keep this open until I get the newest possible version running somehow. It may actually be fixed . I have to figure out how to do that first.

@trollworkout try one of the builds I have: http://purpleorangegames.com/godot/
You might need to chmod +x on the one you download. Let me know if you need any help.
Directly link for 64 bits build:
http://purpleorangegames.com/godot/godot.osx.opt.tools.64
cd Downloads
sudo chmod +x godot.osx.opt.tools.64
./godot.osx.opt.tools.64

I am worry to have wasted ur time guys TT_TT This bug has been fixed in the latest build but was not available for download on OSX so I was behind.

@purpleorangegames  works perfect thank you. I have not only did what you said but also replaced the Godot executable inside the app bundle with the one you gave me 64 bit opt osx . Works fine!

@trollworkout If it is fixed, close it.

@bojidar-bg  I closed it is fixed


Build.DefinitionName ?

Both $(Build.DefinitionName) and $(BuildDefinitionName) are mentioned. Is the first one a typo ? If not, could you explain the difference ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a57f8545-bb15-3a71-1876-3a9ec1a59b93
Version Independent ID: 28c87c8d-c28d-7493-0c7c-8c38b04fbcd7
Content: Run (build) number - Azure Pipelines
Content Source: docs/pipelines/process/run-number.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@AndrewR66 Thank you for the feedback.
Please refer to the correct usage as $(Build.DefinitionName).
The documentation has been updated and is pending review, see about becoming a contributor.


[New Port Request] Effekseer

Library name
Effekseer
Library description
Effekseer--Particle Effect Creation tool
Source repository URL
https://github.com/effekseer/Effekseer
Project homepage (if different from the source repository)
https://effekseer.github.io/en/
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
This software is a particle effect editing tool. By incorporating runtimes and plug-ins, effects created in Effekseer can be played back in your application. Effekseer is intended to be used primarily for games, allowing the designer, not the programmer, to be in charge of editing particle effects.

I'm trying to make a port but I'm encountering some issues and I don't think I can cover all the platforms.
A help would be much appreciated!

You can tell me what's wrong with you in detail, but I'm afraid can't help more. I'm not very familar with this problem either, let me try to deal with it, too.

You can tell me what's wrong with you in detail, but I'm afraid can't help more. I'm not very familiar with this problem either, let me try to deal with it, too.

Basically I'm trying to run my little example using Vulkan and shaderc keeps mismatching linking options with effekseer
if I choose one, effekseer doesn't work, if I choose the other shaderc doesn't link
Effekseer has too many dependencies so it's not easy
I should try using a DirectX application to see if at least the port works.

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Fix inputing multiple operator.

Also some refactor for reducing unnecessary code.

I'll redo this :)


Optional Parameters

In the join monster example RootQuery there is an users element, that returns a list of users, and an user element, that returns the user by ID. I wonder if there is no way to have say a users element that returns a list when given no parameters, and a single user when given an ID.
Or maybe I am missing the point and this is a best practice. I don't know. I just feel like optional parameters would be powerful for defining all types of queries.

Closing this issue as the suggestion is technically already possible in Join Monster.
As suggested, a field that returns a list of users or a single user based on a provided identifier can be implemented. GraphQL permits optional arguments. Join Monster's where function can return a  clause or return a falsy value to indicate no WHERE clause. The caveat is that the type of this field must be a GraphQLList. I suggest checking out this Stack Overflow question regarding this general issue.


Could single_space_after_construct be extended with short-open-tags?

Could single_space_after_construct be extended with short-open-tags, so that <?=$body; ?> and <?=    $body; ?> will both be changed to <?= $body; ?>?

Hi and thanks for your question,
Currently the rule already support this through open_tag_with_echo
$ php php-cs-fixer fix test.php --dry-run --rules=single_space_after_construct --diff
Loaded config default from "/home/possum/work/PHP-CS-Fixer/.php-cs-fixer.dist.php".
Paths from configuration file have been overridden by paths provided as command arguments.
   1) PHP-CS-Fixer/test.php
      ---------- begin diff ----------
--- Original
+++ New
@@ @@
-<?=   $body; ?>
+<?= $body; ?>


TypeError: can only concatenate str (not "bytes") to str

请求
curl http://localhost:8090/v1/chat/completions
-H "Content-Type: application/json"
-H "Authorization: Bearer 111111abc"
-d '{
"messages": [
{
"role": "user",
"content": "Hello!"
}
]
}'
服务报错：
ERROR:    Exception in ASGI application
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/httptools_impl.py", line 426, in run_asgi
result = await app(  # type: ignore[func-returns-value]
File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 84, in call
return await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 292, in call
await super().call(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 122, in call
await self.middleware_stack(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in call
await self.app(scope, receive, _send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/cors.py", line 83, in call
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in call
await self.app(scope, receive, sender)
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 20, in call
raise e
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 17, in call
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in call
await route.handle(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 66, in app
response = await func(request)
File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 273, in app
raw_response = await run_endpoint_function(
File "/usr/local/lib/python3.10/site-packages/fastapi/routing.py", line 192, in run_endpoint_function
return await run_in_threadpool(dependant.call, **values)
File "/usr/local/lib/python3.10/site-packages/starlette/concurrency.py", line 41, in run_in_threadpool
return await anyio.to_thread.run_sync(func, *args)
File "/usr/local/lib/python3.10/site-packages/anyio/to_thread.py", line 33, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 877, in run_sync_in_worker_thread
return await future
File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 807, in run
result = context.run(func, *args)
File "/app/open-api.py", line 110, in create_chat_completion
openai_response = next(resp)
File "/app/adapters/azure.py", line 51, in chat_completions
response = post(url, self.headers, req_args)
File "/app/utils/http_util.py", line 20, in post
raise e
File "/app/utils/http_util.py", line 16, in post
raise Exception("响应异常：" + resp.content)
TypeError: can only concatenate str (not "bytes") to str

fixed，应该是请求出错了，你看下对应的日志


any example for DocSpace document recommendation

does any example for DocSpace document recommendation, I find every except that
I form the input format like follow
roger federer loses <tab> venus williams wins <tab> world series ended
i love cats <tab> funny lolcat links <tab> how to be a petsitter

but it does not work

Hi @HuangZhanPeng, what do you mean by it does not work?
The examples in wikipedia is similar to DocSpace: for instance, if you change the trainMode from 3 to 1 in this: https://github.com/facebookresearch/StarSpace/blob/master/examples/wikipedia_sentence_matching.sh
It essentially becomes an example for DocSpace.


Added pv-1.6.0 RPM spec

Hey, @andyone
I am interested in having pv utility in the repository because of --average-rate option. It provides data transfer average rate counter during on command execution. So I prepared a RPM spec based on source RPM from the official website. I have not well tested package yet, but I hope it works properly.

Looks good, merged. Thanks!


Player Indicators Plugin suggestion

I think it would be a wonderful feature to add a players combat level above their head in addition to their name. If possible, it would also be cool to see their clan and clan rank.

Duplicate of #6567
iirc the plugin already shows icon of the clan rank for the clan you're in. The client has no way of getting users of a clan that you're not in.

The main point of this was combat level next to name


rio dashboard Abnormal startup

Describe the bug

rio dashboard Abnormal startup
uname -a
Linux rio 5.4.0-54-generic #60-Ubuntu SMP Fri Nov 6 10:37:59 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
To Reproduce

rio dashboard

Expected behavior
root@rio:~# rio dashboard
- Waiting for dashboard service to be readyINFO[0083] Opening browser to https://dashboard-rio-system.pl9fpl.on-rio.io/dashboard/?setup=qvq4xtsjpt6rcgrwp4bhb54xslmzc2zchkjgb7d6dmwsr842fmjkrf
FATA[0083] exec: "xdg-open": executable file not found in $PATH

Kubernetes version & type (GKE, on-prem): kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4+k3s1", GitCommit:"2532c10faad43e2b6e728fdcc01662dc13d37764", GitTreeState:"clean", BuildDate:"2020-11-18T22:11:18Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4+k3s1", GitCommit:"2532c10faad43e2b6e728fdcc01662dc13d37764", GitTreeState:"clean", BuildDate:"2020-11-18T22:11:18Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}

Type:
Rio version: rio info
Rio Version: v0.8.0 (af7ad687)
Rio CLI Version: v0.8.0 (af7ad687)
Cluster Domain: pl9fpl.on-rio.io
Cluster Domain IPs: <IP_ADDRESS>
System Namespace: rio-system
Wildcard certificates: pl9fpl.on-rio.io(true)

Additional context
rio system logs
output:
rio-controller | time="2020-12-11T13:06:17Z" level=info msg="Starting rio-controller, version: v0.8.0, git commit: af7ad687"
rio-controller | time="2020-12-11T13:06:17Z" level=info msg="Updating CRD services.rio.cattle.io"
rio-controller | time="2020-12-11T13:06:17Z" level=info msg="Updating CRD stacks.rio.cattle.io"
rio-controller | I1211 13:06:17.451063       1 leaderelection.go:241] attempting to acquire leader lease  rio-system/rio...
rio-controller | time="2020-12-11T13:06:17Z" level=info msg="listening at :443"
rio-controller | I1211 13:06:17.456606       1 leaderelection.go:251] successfully acquired lease rio-system/rio
rio-controller | time="2020-12-11T13:06:17Z" level=info msg="Starting /v1, Kind=ConfigMap controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting apps/v1, Kind=StatefulSet controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting /v1, Kind=Endpoints controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting admin.rio.cattle.io/v1, Kind=Certificate controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting apps/v1, Kind=DaemonSet controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting tekton.dev/v1alpha1, Kind=TaskRun controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting /v1, Kind=Secret controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting /v1, Kind=Service controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting apps/v1, Kind=Deployment controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting admin.rio.cattle.io/v1, Kind=ClusterDomain controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting extensions/v1beta1, Kind=Ingress controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rbac.authorization.k8s.io/v1, Kind=ClusterRoleBinding controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rbac.authorization.k8s.io/v1, Kind=ClusterRole controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rio.cattle.io/v1, Kind=Router controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rio.cattle.io/v1, Kind=Service controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rio.cattle.io/v1, Kind=ExternalService controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting rio.cattle.io/v1, Kind=Stack controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting gitwatcher.cattle.io/v1, Kind=GitCommit controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting admin.rio.cattle.io/v1, Kind=PublicDomain controller"
rio-controller | time="2020-12-11T13:06:19Z" level=info msg="Starting gloo.solo.io/v1, Kind=Settings controller"
rio-controller | 2020/12/11 13:06:20 [INFO] [*.pl9fpl.on-rio.io] acme: Obtaining bundled SAN certificate
rio-controller | 2020/12/11 13:06:21 [INFO] [*.pl9fpl.on-rio.io] AuthURL: https://acme-v02.api.letsencrypt.org/acme/authz-v3/9228390615
rio-controller | 2020/12/11 13:06:21 [INFO] [*.pl9fpl.on-rio.io] acme: authorization already valid; skipping challenge
rio-controller | 2020/12/11 13:06:21 [INFO] [*.pl9fpl.on-rio.io] acme: Validations succeeded; requesting certificates
rio-controller | 2020/12/11 13:06:23 [INFO] [*.pl9fpl.on-rio.io] Server responded with a certificate.

kubectl logs dashboard-55f6fb4fc9-vpccc -n rio-system
time="2020-12-11T13:14:23Z" level=info msg="Rancher version dev is starting"
time="2020-12-11T13:14:23Z" level=info msg="Rancher arguments {Config:{Kubeconfig: UserKubeconfig: HTTPSPort:443 HTTPPort:80 Namespace: WebhookConfig:{WebhookAuthentication:false WebhookKubeconfig: WebhookURL: CacheTTLSeconds:0}} AuditLogPath:/var/log/auditlog/rancher-api-audit.log AuditLogMaxage:10 AuditLogMaxsize:100 AuditLogMaxbackup:10 AuditLevel:0 Features:}"
time="2020-12-11T13:14:23Z" level=info msg="Starting API controllers"
I1211 13:14:23.686350       7 leaderelection.go:241] attempting to acquire leader lease  kube-system/cattle-controllers...
I1211 13:14:23.686548       7 leaderelection.go:241] attempting to acquire leader lease  kube-system/cloud-controllers...
time="2020-12-11T13:14:24Z" level=info msg="Starting apiregistration.k8s.io/v1, Kind=APIService controller"
time="2020-12-11T13:14:24Z" level=info msg="Refreshing all schemas"
time="2020-12-11T13:14:24Z" level=info msg="Starting apiextensions.k8s.io/v1beta1, Kind=CustomResourceDefinition controller"
time="2020-12-11T13:14:25Z" level=info msg="Refreshing all schemas"
time="2020-12-11T13:14:25Z" level=fatal msg="unable to retrieve the complete list of server APIs: tap.linkerd.io/v1alpha1: the server is currently unable to handle the request"

Hi there I'm getting the same error "level=fatal msg="unable to retrieve the complete list of server APIs: tap.linkerd.io/v1alpha1: the server is currently unable to handle the request". Is there a work around for this?

Hi there I'm getting the same error "level=fatal msg="unable to retrieve the complete list of server APIs: tap.linkerd.io/v1alpha1: the server is currently unable to handle the request". Is there a work around for this?

so ,I alway use rio commands.

so ,I alway use rio commands.


CASMINST-6949: create k8s auditing volumes properly

Description
When k8s api server auditing is turned on, two volume mounts need to be added to the config. The original script didn't handle existing volume mounts which caused the k8s upgrade to fail. This PR provides the fix for the issue.

Checklist

[x] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[x] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.5

/backport release/1.6

/backport release/1.5


Make recurrent policies get their recurrence from RecurrentFeaturesExtractors

Decided against doing this PR and keeping it in learned-planners instead, to avoid throwing away good code.


operator project-quay (3.10.0-nightly.20230823)

Add 3.10.0-nightly.20230823 to community operators

/merge possible

/merge possible

/merge possible


HTTPClient.syncShutdown() check currentEventLoop is not nil doesn't work on NIOTS

This produces differing results between macOS and Linux

Check on current event loops on NIOTS must be willing to tolerate false negatives. If they are intended to be assertions, they should use the appropriate assert primitives.

However, the check in question uses currentEventLoop. This functionality cannot be implemented on NIOTS: it is not possible to ask what event loop you’re on. The check as provided literally cannot be duplicated, and so I think this is NTBF.

@Lukasa is there any way for syncShutdown to check if we are running on an EventLoop when using NIOTS?

No, Dispatch provides no way to check if you are on a DispatchQueue (because the question makes no sense in the Dispatch model). The only way to make this solution 100% bulletproof is to keep a registry of all EventLoops in the program, and as EventLoop supports arbitrary conforming types we can’t even guarantee to do that.

Ok I guess this Issue is going nowhere. Should I close?

Sorry we can’t do any better here!


New RLABO code ENV2022-01-14

Request by an Irish data submitter:
I’d like to request a new RLABO code for the lab below (Could I suggest using GEOM?):
GEOMAR, Helmholtz Centre for Ocean Research Kiel and Christian-Albrechts University Kiel
Wischhofstraße 1-3, Geb 12, Raum 305
24148 Kiel, Germany
It looks like the EDMO code is 2947 from what I can see in the VOCAB.
Here is a link to the website: https://www.geomar.de/

new code GEOM is created


Unplanned Maintenance Signals

Which issue this PR addresses:

Fixes https://issues.redhat.com/browse/ARO-1933
What this PR does / why we need it:

For resource health check, we have to emit a metric when we're doing unplanned maintenance on a cluster.  This code changes accomplishes this by:

Adding a maintenance signal middleware. For now, this just emits the "unplanned maintenance" signal.
Using that middleware for write admin operations (ex: start VM).

Test plan for issue:

Unit tests
INT test

Is there any documentation that needs to be updated for this PR?

No - everything is automated

/azp run e2e

Rebased with master due to merge conflict.


DRILL-4390: Uses Resource where Drill favicon is located for static assets

Drill Webserver uses the first jar containing a rest/static directory to
find its static assets. In case of another jar containing this directory, it
might cause the webserver to return 404 errors.
This configures the server to use the resource containing the Drill favicon
as the place to look for all static resources.

LGTM +1


make sure mocha callback is only called once

Mocha calls the callback in rare cases twice when we abort tests and this means that we register the exception handler also twice.

Current coverage is 93.70% (diff: 100%)

Merging #96 into master will increase coverage by 0.31%

@@             master        #96   diff @@
==========================================
  Files            22         22
  Lines           560        588    +28
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            523        551    +28
  Misses           37         37
  Partials          0          0

Powered by Codecov. Last update 80bd082...4ec9749


feat: add some lemmas for Haar measures

We prove some lemmas that will be useful in following PRs #6832 and #7037, mainly:
theorem Basis.addHaar_eq {b : Basis ι ℝ E} {b' : Basis ι' ℝ E} :
     b.addHaar = b'.addHaar ↔ b.addHaar b'.parallelepiped = 1

theorem Basis.parallelepiped_eq_map (b : Basis ι ℝ E) :
     b.parallelepiped = (TopologicalSpace.PositiveCompacts.piIcc01 ι).map b.equivFun.symm
       b.equivFunL.symm.continuous

theorem Basis.addHaar_map (b : Basis ι ℝ E) (f : E ≃L[ℝ] F) :
    map f b.addHaar = (b.map f.toLinearEquiv).addHaar

Co-authored-by: Eric Wieser<EMAIL_ADDRESS>

bors r+


What do you mean by "send history command"

Sorry I do not understand how to send commands with the program, the cmd prompt doesn't let me do anything.

Nevermind, I had the accounts backwards.


Improve README

I've rewritten the README to contain (hopefully) the same information, but in a way that's much easier to read.  This is mostly small tweaks to grammar/sentence structure, but I've also put the config parameters in a table.  Resolves #94: @farzadmf, @peterlitszo, @Leland-Kwong is this better?

Looks good, thanks for the update 👍

Nice, thank you

REALLY COOL! But one more thing, could you please add the dot-files' position? Like:
https://github.com/neoclide/coc-prettier/issues/94#issuecomment-999971857


Adding docker-config-update

https://github.com/sudo-bmitch/docker-config-update
Utility to update configs and secrets to simplify deployment in a compose file.
By submitting this pull request I confirm I've read and complied with the below requirements.
Yep, I actually read it.

I have read and understood the contribution guidelines

The project submitted conforms to the quality standards outlined in the contribution guidelines

Go to the contribution guidelines and read it again.

RULES (because you probably didn't read the guidelines):
Links will be tested with awesome_bot. Please make sure your URL to be listed is unique (do some search queries), otherwise put a link at the end of README.md file.

FORMAT

Use the following format: [Name](Link) - Description.
[Name] should be the title of the project
(Link) should be pointing to a GitHub repository, not to a website
Description. should end with a full stop/period
For GitHub repositories, add the name of the author with a link to its GitHub account after the description, like By [@author](GitHub account)

ORDER: The entries are in lexical order (English letter order)
LABEL (Optional): Add one of the following icons next to the project name like [Name](Link) :icon: - Description.

:heavy_dollar_sign: indicates if one needs a paying subscription to use the project.
:construction: indicates that the project is a work in progress/beta release.
:skull: indicates that the project is obsolete, but still worth mentioning it.

@sudo-bmitch thank you 🙏


Member fixes and removal

putting dylan back up.

@graytwentythree Something to know for the future, you can use git revert to undo existing commits. This is opposed to manually reverting each change.


Use rust syntax highlighting for .nr files in this repo

Related issue(s)
No issue exists for this - it is a minor, quick change.
Description
Summary of changes
Noir files (*.nr) currently have no syntax highlighting when viewing them on github. This PR hints to github that it should use rust syntax highlighting for these files instead - which should be close enough.
Checklist

[ ] I have tested the changes locally.

Cannot test these changes locally, it is on github only. It works for me in other repositories though.

[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[x] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Does it means that noir programs will count as rust language or could we define the noir language in github by duplicating the rust definition?

Within this repo, github will see noir files as rust language files for the purposes of the "Languages" proportion bar. The only way to create a new language for github to recognize as far as I'm aware is to go through the official process with submitting a request/change for the linguist. So this is the only method I'm aware of to get (a rough estimate of) syntax highlighting locally.


Updating docs and scripts to use kustomize overlay by default

The structure of service deployment changed in a way that we can deploy all the services using overlay wether there is a change/patch in overlay or not to keep an standard of deployment across all the services.
Related PRs:
https://github.com/rackerlabs/genestack/pull/564 (creating kustomize base and overlay structure)
https://github.com/rackerlabs/flex-host-baseline/pull/31/files (includes patches needed for the services in some envs)

Do you need to also rename all the "base" directories, to "overlay"?

I don't think we need to rename the base directories, in /etc/genestack/kustomize we do have the following stucture:
├── service-name
│   ├── base -> /opt/genestack/base-kustomize/service-name/base (symlink)
│   ├── aio -> /opt/genestack/base-kustomize/service-name/aio (symlink)
│   └── overlay
│       └── kustomization.yaml
│       └── service-patches.yaml

Also,vI've reviewed nearly all instances of base in the repo, and have converted all Kustomize-related base references to overlay where they relate to deployment. The only exceptions are those intended as examples in the documentation.
Please let me know if there is something missing here.

Currently there's no overlay directory in any of our kustomize folders, and nothing here looks to be creating it. RE: https://github.com/rackerlabs/genestack/tree/main/base-kustomize/keystone - we have
├── keystone
│   ├── base (directory)
│   ├── aio (directory)

Yeah, we didn't make any changes on /opt/genestack structure because we wanted to keep all the environment specific patches under /etc/genestack (which is a symlink to flex-host-baseline/env-name), so we used bootstrap.sh to create that structure for us, here is the PR:
https://github.com/rackerlabs/genestack/pull/564/files#diff-99d5a2a8ca4290b61836b3ad37941f2b925e9ff3151c90dcd5ea82364e4122a1

Currently there's no overlay directory in any of our kustomize folders, and nothing here looks to be creating it. RE: https://github.com/rackerlabs/genestack/tree/main/base-kustomize/keystone - we have
├── keystone
│   ├── base (directory)
│   ├── aio (directory)

@cloudnull Yeah, we didn't make any changes on /opt/genestack structure because we wanted to keep all the environment specific patches under /etc/genestack (which is a symlink to flex-host-baseline/env-name), so we used bootstrap.sh to create that structure for us under /etc/genestac/kustomize, here is the PR: https://github.com/rackerlabs/genestack/pull/564/files#diff-99d5a2a8ca4290b61836b3ad37941f2b925e9ff3151c90dcd5ea82364e4122a1

fair enough, I was looking at this from the lens of bases being deprecated in kustomize in favor of resources. https://kubernetes.io/docs/tasks/manage-kubernetes-objects/kustomization/#bases-and-overlays - but this makes more sense now. So +1


Lizards now metabolize chocolate

About the PR
Lizards were formerly unable to metabolize chocolate, meaning they were never poisoned drinking reagents containing it, the primary being hot chocolate. You could drink your stomach to the full 250u capacity and it would never metabolize the hot chocolate into theobromine
Fixes 32142
Fixes 32146
Why / Balance
Hot chocolate wasn't poisoning lizards like tea and coffee via metabolism
Technical details
Chocolate has been flagged as consumable by reptilians, meaning when they drink reagents with chocolate, it now poisons them. They cannot however eat chocolate bars still.
Media
https://github.com/user-attachments/assets/15999205-c920-42ee-8840-567365da17fc
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
Should break nothing
Changelog
:cl:

fix: Lizards are now poisoned by hot chocolate

Hey, I had a look at the code and it seems this only makes the chocolate bar item (which contains some theobromine) edible for lizards and does not fix metabolism at all. This means #32142 is not fixed by this PR like you claim.
Whether or not lizards should be able to poison themselves by eating wrong food is a separate discussion, but fine in my opinion. Either adjust your PR description so it fits the changes done or fix the mentioned bug, please.

Fixed the description. Also lizards still can't eat chocolate bars. That's somewhere else in the code I never found. This just flags the hot chocolate reagent as something their specific stomach component can metabolize to make it metabolize it like tea and coffee

I have fixed both 32142 and 32146 now. Thank you for the pointers to fix things

I don't think the failed test is my fault. I think this is an upstream issue because it only failed after I merged upstream

I have fixed both 32142 and 32146 now. Thank you for the pointers to fix things

Awesome, I will give it a review and an in-game test later :)

I don't think the failed test is my fault. I think this is an upstream issue because it only failed after I merged upstream

That test has been randomly failing lately, not your fault. I will re-run it for you.

Looks good now. Thanks for the fix and your first contribution!


The linkhttp://hls.ucsd.edu/ is not reachable

The link http://hls.ucsd.edu is broken.

ERR_CONNECTION_TIMED_OUT

Yes, some security issues with the machine that is hosting my website. It will hopefully be up and running in a few days.

Sorry, but this issue is not fixed yet, thus it can not be closed yet. It would communicate fase truth.


Schematicannon crashes and causes bugged lighting with Botania, LazyDFU, and Lithium installed

Describe the Bug
When building schematics in singleplayer with the Schematicannon with Botania, LazyDFU, and Lithium installed, the game crashes after a few blocks have been placed.
In multiplayer, it doesn't seem to crash (I haven't tested extensively), but when you leave the area and come back, the chunk's lighting is messed up (see screenshot). I've also had the lighting bug happen a couple of times while testing in singleplayer, but I can't seem to make it happen reliably.
Reproduction Steps

Have Create, Botania, LazyDFU, Lithium, and deps installed.
Place a Schematicannon with a Creative Crate (I'm fairly sure it also happens pulling from a normal chest, but I haven't extensively tested it).
Load a schematic and build it. The crash should happen within the first 30 seconds or so. Sometimes it doesn't happen — try generating a new world and trying again.

Expected Result
I expected the Schematicannon to build my schematic without crashing the game.
Screenshots and Videos
The bugged lighting that sometimes shows up.

Crash Report or Log
https://crashy.net/0MirtTFifSWuYG9Da0KA or https://bytebin.lucko.me/2yr0thuCkO
Operating System
Windows 11 Home 10.0.22000
Mod Version
0.5.0c
Minecraft Version
1.18.2
Other Mods
Patchouli-1.18.2-71.1-FABRIC.jar
trinkets-3.3.1.jar
lazydfu-0.1.2.jar
Botania-1.18.2-434-FABRIC.jar
lithium-fabric-mc1.18.2-0.7.10.jar
fabric-api-0.58.0+1.18.2.jar
notenoughcrashes-4.1.6+1.18.2-fabric.jar
Additional Context
This bug also happens with Create v0.4.1+637.
Here's the schematic I've been testing with. I don't think there's anything special about it, it's just using vanilla and Create redstone. The crash happens very early, when it's still placing the stone bricks.

Is this your entire mod list? I think this is two separate issues.
The actual crash seems to be caused by Botania, and I don't see how Create could make it happen.
There's several known issues with lighting, but they're caused by having Phosphor.

Yes, this is the entire smallest modlist where I got the crash to happen.

My guess is botania caused the crash and the crash messed up lighting. Don't think it's create's fault, and even if it is, this isn't reproducible.


Doc - Coordinate Conversion: Supported vs Non-Supported coordinate notation

@dfoll @Dbarnes1 @adgiles FYI.
Widget
Coordinate Conversion
Version of widget
16 Jan 2017 10:41
Because we may not be able to modify the widget to use the same RegEx statements that the addins use (see https://github.com/Esri/solutions-webappbuilder-widgets/issues/727#issuecomment-272920738), I went ahead and performed a comparison between the supported notations between the Pro/ArcMap addins and the widget. Shown below are the results. The screenshots are what is supported by the Pro/Arcmap addins. If text is green then addin notation is also supported in widget. If text is red then addin notation is NOT supported in widget. See comments for issues and other info
Decimal Degrees (DD)

[ ] ISSUE #800: Widget partially supports +/- as suffix. For example...
Widget will create correct output for: 45.0+ 100.0- and 45.0+ 100.0+, but will create incorrect output for 45.0- 100.0- and 45.0- 100.0+. Either we fix this for this release or remove +/- suffix support.
[ ] ISSUE #801: Widget supports hemisphere prefix [N|S|E|W] for DMS, but not for DD or DDM. We should fix this.
[x] Widget will support spaces between the lat/long value and hemisphere suffix [N|S|E|W] for example 45.0 N 100.0 W. This is not supported by addins. Adjust content in doc table accordingly.

Degree Decimal Minutes (DDM)

[ ] ISSUE #800: Widget partially supports +/- as suffix. For example...
Widget will create correct output for: 45 0.0+ 100 0.0- and 45 0.0+ 100 0.0+, but will create incorrect output for 45 0.0- 100 0.0- and 45 0.0- 100 0.0+. Either we fix this for this release or remove +/- suffix support.
[ ] ISSUE #801: Widget supports hemisphere prefix [N|S|E|W] for DMS, but not for DD or DDM. We should fix this.
[x] Widget will support spaces between the lat/long value and hemisphere suffix [N|S|E|W] for example 45 0.0 N 100 0.0 W. This is not supported by addins. Adjust content in doc table accordingly.

Degree Minute Seconds (DMS)

[ ] ISSUE #800: Widget partially supports +/- as suffix. For example...
Widget will create correct output for: 45 0 0.0+ 100 0 0.0- and 45 0 0.0+ 100 0 0.0+, but will create incorrect output for 45 0 0.0- 100 0 0.0- and 45 0 0.0- 100 0 0.0+. Either we fix this for this release or remove +/- suffix support.
[x] Widget will support spaces between the lat/long value and hemisphere suffix [N|S|E|W] for example 45 0 0.0 N 100 0 0.0 W. This is not supported by addins. Adjust content in doc table accordingly.

[ ] The widget supports using the Diaeresis ¨ (U+00A8) character for seconds mark (addins do not at all) only when used in conjunction with any of the degree-mark and minute-mark. If this character is used for seconds-mark and no degree-mark and minute-mark is used then widget can't interpret the coordinate. Add this character to the seconds-mark table in the doc and add a note about this limitation.

Military Grid Reference System (MGRS)

Same info from Pro/ArcMap addins apply to the widget.
[x] We need to put a note in the doc that if any coordinates that may be in the MGRS/USNG/UTM output textboxes are invalid when input coordinate is in the polar regions (outside of 84N and 80S).

United States National Grid (USNG)

Same info from Pro/ArcMap addins apply to the widget.
[x] We need to put a note in the doc that if any coordinates that may be in the MGRS/USNG/UTM output textboxes are invalid when input coordinate is in the polar regions (outside of 84N and 80S).

Universal Traverse Mercator (UTM)

Same info from Pro/ArcMap addins apply to the widget.
[x] We need to put a note in the doc that if any coordinates that may be in the MGRS/USNG/UTM output textboxes are invalid when input coordinate is in the polar regions (outside of 84N and 80S).

This one is a duplicate: https://github.com/Esri/solutions-webappbuilder-widgets/issues/752

#800 and #801 have been logged as separate issues, and we should discuss them in war room tomorrow.

@elinz almost complete. Can you verify this is still an issue. I wasn't able to reproduce:
The widget supports using the Diaeresis ¨ (U+00A8) character for seconds mark (addins do not at all) only when used in conjunction with any of the degree-mark and minute-mark. If this character is used for seconds-mark and no degree-mark and minute-mark is used then widget can't interpret the coordinate. Add this character to the seconds-mark table in the doc and add a note about this limitation.

@lfunkhouser I tested the latest build and I'm not having any issues with the Diaeresis ¨ (U+00A8) for seconds-mark. You can use the Diaeresis ¨ (U+00A8) in combination with all the other degree-mark and minute-marks.
But now I'm having problems using the Masculine Ordinal Indicator º (U+00BA) for degree-mark.
All combinations of minutes/seconds using Masculine Ordinal Indicator for degree do not work except for:
Masculine Ordinal Indicator º (U+00BA) Minute Sign ′ (U+2032) Double Acute Accent ˝ (U+02DD)
45º  0′ 0.0˝N 100º  0′ 0.0˝W
So what we should do is remove the Masculine Ordinal Indicator º (U+00BA) from supported degree-mark for the widget.

@lfunkhouser FYI. I check the last few builds and I don't see any of the doc changes listed in this issue in the .pdfs.

Next steps:
Remove masculine ordinal indicator
Kick off publish of doc
Kick off build
@elinz will verify and close

@elinz
My mistake I stupidly copied the characters being used on the pro help page that are not correct, the symbol they are using for Degree Sign and Ring Above are the same. The widget should now except the following three symbols: °˚º

@adgiles I'm assuming these Unicode characters:
Degree Sign ° (U+00B0)
Ring Above ˚ (U+02DA)
Masculine Ordinal Indicator º (U+00BA)

Correct

@topowright @Dbarnes1 Don't remove the Remove masculine ordinal indicator degree symbol yet from the doc. @adgiles made a fix. I'm testing all the combinations of degree, minutes, seconds symbols for DD, DDM, and DMS.

@topowright @Dbarnes1 I finished testing all the combinations of symbols. They are all working so leave the masculine ordinal indicator degree symbol in the doc.

@elinz, I'll make sure to put it back in. Thanks for testing!

ready to verify in the next build.

@elinz and I just made some final changes to the doc regarding coordinate support, now it should be ready for final verification.

Verified in web app builder .pdf build 19 Jan 2017 13:18 #1155. Closing issue.


Smoothly capture several sequential frame

Description
When I use "capture 10 sequential frame(s)",  the game will be lag or forzen. Is there a way to smoothly operate the game when apply "capture 10 sequential frame(s)"?
Environment

RenderDoc version: 1.31
Operating System: Windows 11
Graphics API: OpenGL

Capturing a frame is an expensive process, except in very small or trivial rendering examples. It will take many seconds to capture a frame so no there is no way to capture 10 frames without any interruption.


enforce_once triggering more than expected

In a 2 node cluster our logs are showing a period where the erleans enforce_once callback is being triggered over and over for the same elements.
The Erleans code logs when it deactivates a grain after an enforce_once triggers: https://github.com/SpaceTime-IoT/erleans/blob/master/src/erleans_pm.erl#L38-L60
I've now noticed that the problem may be unrelated to enforce_once because I'm seeing that the size of the sets for the elements are at times as large as 24. Clearly an issue in erleans that it is letting 24 grains register with the same name when there are only 2 nodes, meaning even on the same node a grain id is able to be registered many times.
But I also noticed that for a grain that has a set size of 2 there being logs in the same second for deactivate_dups called 3 times. So still seems excessive even if the more important problem is unrelated to enforce_once.

I think what you really want here is instead of {cardinality, 2}, you want {strict, {cardinality, 1}} which will fire when it's not only 2, but when it's moving from 1 to 2.
Can you try that?

Yea, I can change it to use that.

Please reopen if you run into more issues with this.


TypeError: html is undefined

@shivamgupta211 Can you provide details on this issue? I've gotten the same error, which lead me here


Latex equation broken in RZZGate documentation

Environment
Online docs
What is happening?
On the RZZGate docs page: https://docs.quantum.ibm.com/api/qiskit/qiskit.circuit.library.RZZGate
An equation that is a helpful reference for getting factors of 2 correct does not render. Mousing over gives the tooltip shown with the error message suggesting to use renewcommand instead of newcommand:

How can we reproduce the issue?
https://docs.quantum.ibm.com/api/qiskit/qiskit.circuit.library.RZZGate
What should happen?
Equation should render without an error
Any suggestions?
May just need to replace \newcommand with \renewcommand

Oh, come to think of it, I think the Sphinx build used MathJax as its backend when it lived in Terra, but it's now clearly using KaTeX.  KaTeX includes a \th command in its default preamble, so it looks like we're going to have to change this from the Terra side after all, so I'll transfer it back.
This is one place where the docs-build split is potentially going to cause nuisances, though - the MathJax parsing and rendering engine supports a totally different set of features to KaTeX (imo KaTeX is better).


Connection to MS SQL Server Failed. URI Worked on python console

Make sure these boxes are checked before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[X] I have reproduced the issue with at least the latest released version of superset.
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version
0.28.1
Expected results
I am expecting database connection is ok.
Actual results
I encountered an issue when adding a ms sql server to superset. I tried all the hints or solutions on Stack overflow and on here. Nothing works for me. I am using Windows and run the server under development.
The URI I am using is 'mssql://@SERVER/DATABASE?driver=ODBC+Driver+13+for+SQL+Server'.  Its weird that when I put the same URI in Python console to create engine using sqlalchemy.create_engine(URI), it works perfectly. It doesn't work when I put the URI in Adding database through Superset.
sqlalchemy.exc.OperationalError: (pyodbc.OperationalError) ('08001', '[08001] [Microsoft][ODBC Driver 13 for SQL Server]SQL Server Network Interfaces: Error Locating Server/Instance Specified [xFFFFFFFF].  (-1) (SQLDriverConnect); [08001] [Microsoft][ODBC Driver 13 for SQL Server]Login timeout expired (0); [08001] [Microsoft][ODBC Driver 13 for SQL Server]A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections.
Could anyone help me with this?
Steps to reproduce

@wangym1993  I am using pymssql and this is the connection string I am using to connect to MS SQL
mssql+pymssql://:@localhost:1433/<db_name>.


Added fix to read frame ID from image message header.

This PR change the marker publishers to use the frame IDs from the incoming camera frames instead of a rosparam.

LGTM. I'm surprised this wasn't already the case.


Internationalization (i18n) guide and recipe should be merged

📚 Subject area/topic
Internationalization (i18n)
📋 Page(s) affected (or suggested, for new content)
https://docs.astro.build/en/guides/internationalization/
https://docs.astro.build/en/recipes/i18n/
📋 Description of content that is out-of-date or incorrect
The guide Internationalization (i18n) Routing lacks several tips that would make the Astro feature more attractive. Personally, I had given up on using Astro's i18n until I realised the following points, which made my experience so much better:

Every example block shows a directory structure with a folder per locale, in which you have to duplicate every .astro file:

It seems inconvenient and off-putting, I can't imagine a case where I'd like to do this. The docs suggests in the text that you could create a /[locale]/ folder, but I must admit that when skimming through the page, I missed it and only saw the several examples like in the screenshot. I think a more proeminent example block with /[locale] and/or /[...locale] (see 2.) would better showcase the flexibility of the feature.

An info is missing from the guide and the alternative recipe (Add i18n features): What do you do when you have a defaultLocale without a prefix like /en/? (It's the default behaviour in the new feature) The answer is simple: you use /[...locale] and handle the undefined value in getStaticPaths. Pretty simple, but if an Astro beginner isn't aware of this possibility, the guide and the recipe appear to be missing critical info. Here's an example of what should be done:
// Store this in a utils file
export const langParams = [undefined, "fr"]
// In every [...lang]/*.astro file
export function getStaticPaths() {
	return langParams.map((lang) => ({
		params: { lang: lang },
	}))
}

By itself, the guide about the Astro feature does not give enough examples to get you started. You have to use /[...locale] and adapt the following tips from the recipe Add i18n features to have a working website:

The clever functions in Translate UI strings (you can use them with the new Astro feature like so: useTranslations(Astro.currentLocale)),
The LanguagePicker example is nice to have, because in Static mode, I don't think there is any other way to switch languages or to detect the client's preferred locale.

The Astro feature is very flexible, so I don't see why people would want to not use it. That's why I feel like the recipe page is "legacy" and could be merged into the new guide for easier access to these clever tips.

🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
No response

Thank you for this amazingly detailed feedback, @aureliendossantos !
We have a bit of a legacy problem here, and I do have updating i18n API docs on my plate, and probably including better linking to and discovery of that guide.
So the backstory is:
This recipe was written well before we had the i18n Routing API. It's based on how we created our i18n system for Astro Docs, and then Starlight, both of which are older than our i18n Routing API. (And which have been updated to take a little from, but still do not yet fully use this API for routing)
So, the guide is a bit more complete about setting up an i18n site as a whole, because that's what we did! And the i18n Routing guide contains information about only the routing part of it (ignoring any "extra features" like translating UI strings because it's not specific to routing, but it is a huge part of Astro Docs/Starlight implementations).
So the current situation is:
The two don't perfectly align! And as you've correctly identified, we don't right now have a unified "here's how to set up a site" story. Add to the mix that our experimental Content Layer API is dropping in the 5.0 beta NEXT WEEK, which will probably open up to even more content management options (like pulling in your translations from external sources), and we certainly don't have a full i8n story around that yet to tell!
All this is to say that this would be an amazing addition to our current docs that we don't currently provide in an all-in-one format. I would certainly be open to anyone wanting to help contribute to that! It's on our wishlist, but not our immediate plate with lots of new docs currently written to support the new features in and upgrading to v5.
BUT, this post will be a super helpful kick off for that, so I hope people see this, continue to leave feedback about what will be helpful, and maybe propose some contributions!

Just bumping this for freshness in case anyone would like to "freshen up" our i18n guide that pre-dates our i18n routing API features!

I agree the recipe could be merged with the guide because many expect to find this kind of information there (translating UI strings or routes). However, I see two caveats:

There is no official user guide and the implementation depends on each person's needs. In my opinion the recipe works very well for small sites with some adjustments if necessary; Starlight on its side now uses i18next with a function provided through a middleware. Others might have other methods that work well with their project.
Recommending the use of [locale] (or [...locale]) rather than creating folders for each locale doesn't seem ideal to me. Again, it will depend on the project I think.

It seems inconvenient and off-putting, I can't imagine a case where I'd like to do this.

Translate the slug?
With:
/src/pages
└── [locale]/
    ├── about.astro
    └── contact.astro

Considering that the site is available with the en, es and fr locales, this will generate:

/en/about
/es/about
/fr/about

Meanwhile, with:
/src/pages
├── es/
│   ├── sobre-nosotros.astro
│   └── contacto.astro
├── fr/
│   ├── a-propos.astro
│   └── contact.astro
├── about.astro
└── contact.astro

We can have:

/en/about
/es/sobre-nosotros
/fr/a-propos

From both a user and SEO perspective, it's better.
Perhaps the file names in the example should be translated to make it more meaningful.
My example can probably be rewritten using getStaticPaths with the following structure:
/src/pages
└── [...locale]/
    └── [...slug].astro

But it might be more complicated to explain to newcomers... especially without knowing where their content comes from and if they want different page templates.

To be clear, I think if there's any improvement/solution to be had here, it's in refreshing the recipe and NOT moving the content into the guide page, for similar reasons: a recipe is an opinionated suggestion of a "base dish".
Also, the Internationalization page is about using the API which is a routing API only and should stay focused on the connection between creating/validating route path not actual translation. (Which again, will be very much up to an individual project.)
The current recipe as it exists works, and I think the question is, can it be made more helpful by acknowledging the existence of the newer routing API? The answer might be "no", but that is the question to be asking here, and the way we will evaluate potential improvements!

I think the recipe could be simplified somewhat, at least the getLanguage() function can be replaced by Astro.currentLocale. I've had positive feedback from this example I made to help others, though I do understand that [locale] is not for everyone.

Sure, I could try to make a PR this week!

Thank you, @sasoria ! I know this is one inconsistency (between these two pages) that often gets mentioned!

@sasoria Are you still interested in updating the i18n recipe with Astro's i18n features like Astro.currentLocale?
In general, we will otherwise keep the recipe as is but just update to use the new i18n features!

Just a small little detail about routing on this example:
/src/pages
└── [...locale]/
    └── [...slug].astro

That would not work as expected in on-demand pages (SSR). Rest parameters ([...name]) can match any number of path segments and as a consequence using two rest parameters in the same routes makes parsing the path ambiguous. For example, a request to /en/blog/amazing-article could be parsed in any of the following ways:

{locale: undefined, slug: 'en/blog/amazing-article'}
{locale: 'en', slug: 'blog/amazing-article'}
{locale: 'en/blog', slug: 'amazing-article'}
{locale: 'en/blog/amazing-article', slug: undefined}

It can be used to generate the page, so for prerendered pages, this is fine since the routes are generated from getStaticPaths. Astro provides no guarantees about which of those interpretations will be used during parsing. We use a library to turn the file path into a parsing function, which also doesn't guarantee the behavior when facing this ambiguity (in fact, the behavior changed during a patch release earlier this year).
So if you do want to have an optional locale in the URL and use SSR the recommendation would be to have the entire tree twice, once on src/pages and once on src/pages/[locale]


documentation: Incorrect Information in DGS Framework Documentation Regarding WebSocket Sub-Protocols

What are you missing in the docs

The documentation for the DGS Framework currently contains incorrect information regarding the WebSocket sub-protocols graphql-transport-ws and graphql-ws. Specifically, the documentation states that graphql-ws is the deprecated sub-protocol, which is inaccurate. graphql-transport-ws is the deprecated sub-protocol, while graphql-ws is the newer protocol supported by the framework.
Proposed text

Update the correct subprotocol names.

Thanks for pointing out the confusion. We are referring to teh sub protocol names here, which are admittedly confusing. Please see https://www.apollographql.com/docs/react/data/subscriptions/#websocket-subprotocols
The deprecated protocol is referred to as subscriptions-transport-ws uses graphql-ws as the sub protocol.

https://github.com/Netflix/dgs/pull/179


Managing the app-registry-secret reference passed into scheduled workloads

Description
We recently tried to implement secret rotation with the immutable secret pattern through kapp managed  "versioned secrets" (documented here). Ultimately, the app-registry-secret reference we passed into Eirini was not updated on the app workload statefulSets on redeployment of Eirini. Do y'all have any opinions about how we should manage that secret and potentially insight into how Eirini monitors references to secrets and configMaps in the workloads it schedules?
Steps to reproduce
Deploy cf-for-k8s with a google service account granted GCS storage admin (the permissions required to use GCR as an app registry) configured as the app-registry-secret passed to Eirini, updated to 'app-registry-secret-ver-1' with kapp's versioned annotation. (Note that to avoid the use kapp in this reproduction flow, we could just directly update the Eirini config to point at a new secret reference explicitly)
What was expected to happen
Eirini updated its reference to the app-registry-secret and all references to that secret in workloads it had scheduled.
What actually happened
Only the secret reference in Eirini was updated, so new app workloads got the correct reference, but references in existing workloads remained stale.
Potential fix (optional)
In a comment on a related cf-for-k8s issue, we considered the exclusion of the imagePullSecret from the statefulSet definition of scheduled workloads, instead relying on the inheritance of a serviceAccount imagePullSecret in the namespace. This assumes any namespace into which we schedule workloads has configured its default serviceAccount appropriately.
Thanks,
Andrew

Hi @acosta11 - sorry for slow response, we discussed this in IPM and have spun out and prioritised a story to watch the ConfigMap and update secrets.

Hi @acosta11,
We started implementing a custom controller that watches changes to our ConfigMap and copies the secret over to the app namespace if the secret name was updated. This works fine if there's only one eirini ConfigMap, but that, as we found out, is not the case with cf-for-k8s -- kapp creates a new CM every time there's a change. This means that the CM is never effectively updated which causes issues with our approach.
We were wondering what's the use case of CM versioning in cf-for-k8s. Would you be open to not versioning the Eirini CM?
BR,
Georgi & @georgethebeatle

Hi Georgi,
Sorry about the delay here and thanks for the follow-up.
Intuitively, I think we want to continue using versioned ConfigMaps. We originally versioned ConfigMaps and Secrets to force Pod rolls, which forces reinitialization of the runtime config for applications that aren't written to watch the files to which their configuration is mounted in a Kubernetes pod context and enables transparent, fast feedback on the update of those resources and their volume mounts.
If Eirini can provide guarantees around the consumption of updated ConfigMaps at runtime without a pod roll, I think we would consider and exception to that general pattern.
That said, we've also considered the original question of whether imagePullSecrets in general need to follow this pattern because the pod/node will update those on demand as they are needed. So we've stopped versioning the app-registry-secret and haven't run into any issues yet. So we could potentially avoid this issue by defining the contract to make that secret name immutable?
Thanks,
Andrew

Hi @acosta11!

So we could potentially avoid this issue by defining the contract to make that secret name immutable?
Yes, we think this would work. Keeping the ConfigMaps versioned while maintaining the imagePullSecret immutable is probably the best way to go. Closing!

Yes, we think this would work. Keeping the ConfigMaps versioned while maintaining the imagePullSecret immutable is probably the best way to go. Closing!

Sounds good, thanks for thinking through this problem with us and spiking out the config map watcher!
Thanks,
Andrew


module 'nui.utils.autocmd'

处理 /Users/zerduo/.config/nvim/init.lua 时发生错误:
E5113: Error while calling lua chunk: /Users/zerduo/.config/nvim/lua/configs/base/ui/select.lua:2: module 'nui.utils.autocmd' not found:
no field package.preload['nui.utils.autocmd']
no file './nui/utils/autocmd.lua'
no file '/usr/local/share/luajit-2.1.0-beta3/nui/utils/autocmd.lua'
no file '/usr/local/share/lua/5.1/nui/utils/autocmd.lua'
no file '/usr/local/share/lua/5.1/nui/utils/autocmd/init.lua'
no file './nui/utils/autocmd.so'
no file '/usr/local/lib/lua/5.1/nui/utils/autocmd.so'
no file '/usr/local/lib/lua/5.1/loadall.so'
no file './nui.so'
no file '/usr/local/lib/lua/5.1/nui.so'
no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
[C]: in function 'require'
/Users/zerduo/.config/nvim/lua/configs/base/ui/select.lua:2: in main chunk
[C]: in function 'require'
/Users/zerduo/.config/nvim/lua/core/funcs.lua:2: in main chunk
[C]: in function 'require'
/Users/zerduo/.config/nvim/lua/core/init.lua:6: in main chunk
[C]: in function 'require'
/Users/zerduo/.config/nvim/init.lua:9: in main chunk

Neovim version, OS?
Try :PackerSync

I am having this issue too. I already am using lvim for a month and half. But I messed up my ~/.config/nvim and the ~/.local/share/nvim so I decided to rm -rf those and start again by cloning. But then this error pops up. I worked around it by checking out tag 2.2.08, then launch nvim, let it install.... Then checkout production branch do a PackerSync. Now for me it's fine again, but this stuff is not my cup tea. So I'll let you pro's handle this one ;)

Oh btw:
nvim --version
NVIM v0.8.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

I'm running Arch™ BTW.

@pizsd  @dhensen Thanks for the report!
Try the last commit. I think this problem is solved.
I would appreciate a reply.

@lvim-tech after PackerSync and restart of nvim this error message no longer pops up :+1:

@dhensen Thanks!


About how to use matching routes

The following code, does not generate routes.
# config/routes
get '/hoge/:id', to: 'hoge#show'

Correctly,
get '/hoge/:id', to: 'hoge#show', as: :hoge  #=> `hoge_path` is generated.

I was puzzled by this behavior :confounded: , so I added it to the README :memo:

Thanks for the patch! Could you fix the Rubocop warnings?

As your spec, it seems that get '/apps/foo', to: 'apps#foo' generates a route.

What is the difference between get '/hoge/:id', to: 'hoge#show' (described in the pullrequest) and get '/apps/foo', to: 'apps#foo'?

@yuku-t Thanks

it seems that get '/apps/foo', to: 'apps#foo' generates a route.

It is because I thought that
get '/apps/foo', to: 'apps#foo'
generates
function apps_foo_path(params) { return '/apps/foo'; } without as option.
But, My tests does not prove this..
I'll try to fix

i had too much left this pr... sorry 🙇
This pr seem to unnecessary  .. so close.


Plotting error

I ran your Example notebook code and got the following error:

ValueError                                Traceback (most recent call last)
 in
----> 1 main.main(data_smiles=data[['smiles1','smiles2']], # SMILES input
2           data_prop=data[['median']], # Property of interest
3           data_err=data[['min', 'max']], # Error on the property
4           data_name=data_name,
5           data_units=data_units,
/fwd/SMILES-X/SMILESX/main.py in main(data_smiles, data_prop, data_err, data_extra, data_name, data_units, data_label, smiles_concat, outdir, geomopt_mode, bayopt_mode, train_mode, pretrained_data_name, pretrained_augm, model_type, scale_output, embed_bounds, lstm_bounds, tdense_bounds, nonlin_bounds, bs_bounds, lr_bounds, embed_ref, lstm_ref, tdense_ref, dense_depth, bs_ref, lr_ref, k_fold_number, k_fold_index, run_index, n_runs, check_smiles, augmentation, geom_sample_size, bayopt_n_rounds, bayopt_n_epochs, bayopt_n_runs, n_gpus, gpus_list, gpus_debug, patience, n_epochs, batchsize_pergpu, lr_schedule, bs_increase, ignore_first_epochs, lr_min, lr_max, prec, log_verbose, train_verbose)
1046
1047             # Plot prediction vs observation plots per run
-> 1048             visutils.plot_fit(trues=[y_train_clean_unscaled, y_valid_clean_unscaled, y_test_clean_unscaled],
1049                               preds=[y_pred_train_mean_augm, y_pred_valid_mean_augm, y_pred_test_mean_augm],
1050                               errs_true=[y_err_train, y_err_valid, y_err_test],
/fwd/SMILES-X/SMILESX/visutils.py in plot_fit(trues, preds, errs_true, errs_pred, err_bars, save_dir, dname, dlabel, units, fold, run, final, model_type)
302             # (N, ) for symmetric errors, (2, N) for asymmetric errors
303             if err_bars is not None:
--> 304                 err_true = error_format(true, err_true, err_bars)
305
306             # Legend printing for train/val/test
/fwd/SMILES-X/SMILESX/visutils.py in error_format(val, err, bars)
442             # Switch from min/max range to the lengths of error bars
443             # to the left/right from the mean or median value
--> 444             return np.abs(val-err).T
445         # If three error values are given, they are treated as [std, min, max]
446         elif err.shape[1]==3:
ValueError: operands could not be broadcast together with shapes (5,) (5,2)
Do you know how to fix it ?

I have fixed the bug for me.
Could you please pull the most recent version and confirm that it runs for you?

Resolved and closed due to inactivity.


Feat: Fix memory issue by replacing io.ReadAll with io.Copy (#4017)

What this PR does / why we need it:
Fixes an out-of-memory (OOM) issue that occurs when downloading large files from Google Cloud Storage (GCS).
Replaces io.ReadAll() with io.Copy() to stream data and prevent excessive memory usage.
Which issue(s) this PR fixes :
Fixes #4017
Type of changes

[x] Bug fix (non-breaking change which fixes an issue)

Feature/Issue validation/testing:
Test A : Replaced io.ReadAll() with io.Copy() in the GCS download function.
Test B : Tested downloading large files from GCS to ensure memory usage remains stable.
Test C : Verified that existing functionality is unaffected by the change.

Logs

{"severity":"INFO","timestamp":"2024-10-25T05:59:19.932967054Z","caller":"logging/config.go:80","message":"Unable to read vcs.revision from binary"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933186058Z","caller":"agent/main.go:146","message":"Initializing model agent with config-dir /mnt/configs, model-dir /mnt/models"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933205723Z","caller":"agent/syncer.go:37","message":"Syncing from model dir /mnt/models"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933291351Z","caller":"agent/main.go:311","message":"Starting puller"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933308112Z","caller":"agent/main.go:161","message":"Starting agent http server..."}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933419621Z","caller":"agent/watcher.go:95","message":"Start to watch model config event"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933462608Z","caller":"agent/main.go:336","message":"Building server user port 8080 port 9081"}
{"severity":"INFO","timestamp":"2024-10-25T05:59:19.933486366Z","caller":"agent/watcher.go:133","message":"Watching /mnt/data/models.json"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.561580441Z","caller":"agent/watcher.go:109","message":"Processing event CREATE        \"/mnt/configs/..data\""}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.561973675Z","caller":"agent/watcher.go:180","message":"adding model my-model"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.562043687Z","caller":"agent/puller.go:144","message":"Worker is started for my-model"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.562067421Z","caller":"agent/puller.go:152","message":"Downloading model from gs://my-bucket/models/my-model"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.562145489Z","caller":"agent/downloader.go:47","message":"Downloading gs://my-bucket/models/my-model to model dir /mnt/models"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:27.567088747Z","logger":"modelAgent","caller":"storage/gcs.go:39","message":"Downloading model ","modelName":"my-model","storageUri":"gs://my-bucket/models/my-model","modelDir":"/mnt/models"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:48.268679916Z","logger":"modelAgent","caller":"storage/gcs.go:157","message":"Wrote  to file /mnt/models/my-model/1/my-model.safetensors"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:48.31997657Z","logger":"modelAgent","caller":"storage/gcs.go:157","message":"Wrote  to file /mnt/models/my-model/config.pbtxt"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:48.320341167Z","caller":"agent/downloader.go:73","message":"Creating successFile /mnt/models/my-model/SUCCESS.c355724640c567783469f4df19fa01faffea8df1a7c422910c722d9a83e95b2f"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:56.840635114Z","caller":"agent/puller.go:177","message":"Successfully loaded model my-model"}
{"severity":"INFO","timestamp":"2024-10-25T06:00:56.84074305Z","caller":"agent/puller.go:137","message":"completion event for model my-model, in flight ops 0"}

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[x] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Fix out-of-memory issue when downloading large files from GCS by replacing io.ReadAll with io.Copy to stream data and optimize memory usage.

I think it is a signed commit, is that correct?
I'm not very familiar with signed commits since it's my first time.

Hi @ops-jaeha, thanks for the fix. Can you please sign your commit?

I think it is a signed commit, is that correct?
I'm not very familiar with signed commits since it's my first time.

when pushing the commit you need to sign it with the -s option.
As the PR is already created you can updated it by using git commit -s --amend

I have made a signed commit and confirmed that it has been verified. What should I do next to proceed with CI?

I have made a signed commit and confirmed that it has been verified. What should I do next to proceed with CI?
Did you push?

Sorry for the commit mess, it’s my first time working with signed commits...
I'm finished DCO

Can you approve workflows?

@yuzisun @terrytangyuan can you help with this?

@spolti Can you approve workflows?

approved

Hello @spolti @sivanantha321
Due to a recent system change on my end, it seems that I need re-approval for this PR. I apologize for the extra step, but could I kindly ask for your approval again?
Also, I’m curious: when the workflow fails, will I need to request another approval every time I make a commit to address the issues, or is it retained?
Thank you for your patience!

Hello @spolti @sivanantha321 Due to a recent system change on my end, it seems that I need re-approval for this PR. I apologize for the extra step, but could I kindly ask for your approval again?
Also, I’m curious: when the workflow fails, will I need to request another approval every time I make a commit to address the issues, or is it retained?
Thank you for your patience!

Yeah. It needs approval every time you make a push. This is only applicable for the first time contribution. Also can you help sign off the commits ? The DCO check is failing.

Hello @spolti @sivanantha321 Due to a recent system change on my end, it seems that I need re-approval for this PR. I apologize for the extra step, but could I kindly ask for your approval again?
Also, I’m curious: when the workflow fails, will I need to request another approval every time I make a commit to address the issues, or is it retained?
Thank you for your patience!

Yeah. It needs approval every time you make a push. This is only applicable for the first time contribution. Also can you help sign off the commits ? The DCO check is failing.

I'll try again.

Hello @spolti @sivanantha321 Due to a recent system change on my end, it seems that I need re-approval for this PR. I apologize for the extra step, but could I kindly ask for your approval again?
Also, I’m curious: when the workflow fails, will I need to request another approval every time I make a commit to address the issues, or is it retained?
Thank you for your patience!

Yeah. It needs approval every time you make a push. This is only applicable for the first time contribution. Also can you help sign off the commits ? The DCO check is failing.

I'll try again.

I’ve completed up to the DCO.

@spolti @sivanantha321
All workflows have successfully passed.

/lgtm
/approve

This was my first time contributing to a large open-source project, and thanks to @spolti and @sivanantha321,
I was able to experience this for the first time. I truly appreciate your kindness and support.

You also need to rebuild and push the KServe Agent Docker file. Please handle that.

You also need to rebuild and push the KServe Agent Docker file. Please handle that.

Should be handled by the KServe CI, thanks.

Should be handled by the KServe CI, thanks.

I just noticed that workflows are not triggered if it is automerged by the github-actions-bot.


Refactor isAuthorized utility function

It's just a minor refactoring, which brings back the old function signature and rearranges the code in the for loop.
The refactoring of the for loop was negotiating the if statement and moving the "else" part into the body, significantly simplifying the code.
Improve readability by:

Simplifying nested if conditions
Removing unnecessary continue statements
Adding overloaded function to support optional parameters

To be discussed
Do we need the episodeRoleId argument? Wouldn't the mediapackageId be sufficient?
If it's null, we skip the check.
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits
[ ] explain why it needs to be merged into the legacy branch, if it is targeting the legacy branch

@gregorydlogan, I pulled out the argument, and it's ready for review.


Hi i know its stupid but

Can i use without proxy if yes how ?

No without proxy what's the meaning of this bot


[tool_crash] FileSystemException: Cannot create file, OS Error: Input/output error, errno = 5

Command
flutter pub get

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot create file, OS Error: Input/output error, errno = 5
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.createSync (dart:io/file_impl.dart:273:5)
#2

This could be many things, but I don't think it is Flutter related.
Check this:
https://github.com/flutter/flutter/issues/15711

@danthienphuc
Is that the complete log you get ? If not, can you provide the entire error log ?
Do you get this error everytime you run flutter pub get ? Can you also provide flutter doctor -v ?

Check again!


Simplify executor UX

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

Closes #771

Something funny is happening with codecov. The "Impacted Files" list appears to be the entire repo.

I see that, very weird. The UI change renders Codecov essentially useless. I think you'll need to open a support ticket with Codecov.
The Codecov annotations on this PR appear to be accurate tho and you can use those to increase the patch coverage.


Ecodevice and node red

In a node red stream, I capture the state changes of the ecodevice to detect possible ecodevice failures.
However, I notice that the information provided by the node red node 'The Poll State node' is not updated.
In particular, the fields:
last_changed: "2024-09-30T17:09:15.436Z"
last_reported: "2024-09-30T17:09:15.436182+00:00"
last_updated: "2024-09-30T17:09:15.436Z"
always display the same values.
And the field 'timeSinceChangedMs' displays a value in milliseconds of 6554849 while the node is polled every two minutes, i.e.: 120000 ms.
I use:
Core 2024.9.3
Supervisor 2024.09.1
Operating System 13.1
Ecodevice 5.2.0
Node red 18.0.5
Node red companion: v4.1.1

I don't think those value are managed in this integration but more in home assistant core :thinking:
Here I don't managed any datetime, I just tell to HA the last value of each entity

Thanks for your reply.


HealthCheckTest: Fix intermittent failures

The HealthCheckTest runs netcat in some containers via TemporaryJobs and
then probes those netcat instances to make sure they're reachable. When
netcat is probed, it usually exits immediately. Generally we wouldn't care
since the test ends right after the probe.
However, if TemporaryJobs noticed that the job exited while the test was
still running, it would throw an exception due to the job being unhealthy.
The test would then fail.
So now, we run netcat with -k so it sticks around as a persistent server
and the container doesn't exit prematurely.
Fixes #529.

:+1:

:+1:


[google_maps_flutter] Android: update myLocationButton preference on map load

Related Issues

https://github.com/flutter/flutter/issues/31930

@amirh I added a test for this at https://github.com/flutter/plugins/pull/1350, once we have tests running on Android, we will be able to catch errors like these before hand.