[pyspark] Support stage-level scheduling for training

XGBoost training task requires taking the whole GPU exclusively, so we have limited there's only 1 task running at the same time even for the ETL stages, which typically decreases the parallelism of spark tasks, and finally has some side-effect for the whole performance of xgboost end-2-end training.
The stage-level scheduling we are using in this PR does not change executor resources and hence does not require dynamic allocation to be enabled.  The original executors are retained. The features used there, however, are new in Spark 3.4.  We also changed CPU task resources therein to ensure only 1 task per gpu just for that barrier stage.

@WeichenXu123 @trivialfis @eordentlich please help to review, Thx

I did the performance test
Env:
Spark standalone cluster with 2 Nodes each having 12 CPU cores and 64G mem, and 1 TITAN V GPU with 12 G mem.
Dataset:
Mortgage,  total 255, 962, 232 rows, 27 features + 1 label
Result,

W/O stage-level scheduling,

    .config("spark.executor.memory", "60g")
    .config("spark.executor.cores", 12)
    .config("spark.task.cpus", 12)
    .config("spark.executor.resource.gpu.amount", 1)
    .config("spark.task.resource.gpu.amount", 1)
    .config("spark.sql.execution.arrow.maxRecordsPerBatch", 2000000)

The training times are:  532.36, 534.46, average: 533.41

W/ stage-level scheduling

    .config("spark.executor.memory", "60g")
    .config("spark.executor.cores", 12)
    .config("spark.task.cpus", 1)
    .config("spark.executor.resource.gpu.amount", 1)
    .config("spark.task.resource.gpu.amount", 0.08)
    .config("spark.sql.execution.arrow.maxRecordsPerBatch", 2000000)

The training times are: 383.34, 383.11, average: 383.225
The ratio is 533.41/383.225 = 1.39, which means the PR can have 39% speedup.
The ETL phase is just reading the parquet file and doing the repartition, if we involve more real ETL operations, I guess, the stage-level scheduling PR can bring more speedup.

Any update on the status of this PR?

Hi @WeichenXu123, Could you help to review it again? Thx very much.

Hi @trivialfis, Could you help trigger CI ?

Could you please provide an example usage that gets everything right (with no warning), and add comments on the example for each configuration with an explanation for why the configuration is set in such a way? If I were a user I would struggle to understand and address all potential warnings.

I will file a follow-up PR to add related doc


Trim tags to only include blocks being modified

For example, instead of re-adding every block mineable by a pickaxe to the minecraft:mineable/pickaxe tag, it adds only the blocks that are affected by this datapack (glass, etc.).
This does still work, of course - at least on 1.19.4.
(and i sorted the block names in the JSON files to make them prettier ;) )

Haha that will make it a lot easier to maintain thanks 😅


Separated the events files to make them work

Changes proposed
Previously the events were not showing up on my LinkFree page. Now I made separate files for each of the events and I hope they will work fine.
I deleted this file 2022-10-24-kubecon.json and created the following two files. 👇

2023-04-18-kubecon-cloudnativecon-europe-2023.json
2022-10-24-kubecon-cloudnativecon-north-america-2022.json

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Cheers for that. I haven't checked the JSON but assuming it is what we spoke about this should be okay.

I have deleted this file "2022-10-24-kubecon.json" and created two more files but now it is asking me to resolve the merge conflicts but the resolve conflicts button is not showing up. Maybe I don't have the write access. It is not clear to me.

Works great cheers.


Add documentation for Copy-UserInternationalSettingsToSystem cmdlet

A while back, we added the documentation for the newly created powerShell cmdlet "Copy-UserInternationalSettingsToSystem".
However, we removed it because the cmdlet had not been released in Windows yet, and was causing confusion for users who were trying the cmdlet.
This cmdlet has now released for Windows 11, so we can add the documentation back.
NOTE that it is only for Windows 11 and higher. I have added a note on this on the documentation
This cmdlet copies the current user's international settings (Display language, Input language, Format/locale and Location/GeoID) to one or both of the following:

Welcome screen and system accounts
New user accounts
Depending on which parameters are set to true.

Docs Build status updates of commit b67a199:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fb9c11d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Thanks for adding this @erik0686! 👍

Docs Build status updates of commit c94205c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 11ef330:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@jefgen Hopefully yes :) However I don't have merging authorization so I think someone else from the documentation team has to merge it :)

@e0i - could you please help to move this PR forward? Many thanks.

Docs Build status updates of commit c4a4899:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 26ce838:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/international/Copy-UserInternationalSettingsToSystem.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

docset/winserver2022-ps/international/International.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


fix: fix wrong distribution

Hi, I just found the source code supports signedReleaseFiles while the distribution file lib/main.js is the old one. So, actually signedReleaseFiles and related logics do not work. So, I build it and also correct the typo in the readme.

Bump, just ran into this as well


Extract data from MongoDB

My string format is as below
MongoDB\Model\BSONDocument Object
(
    [storage:ArrayObject:private] => Array
        (
            [_id] => MongoDB\BSON\ObjectId Object
                (
                    [oid] => 5b05c37eb826ef78a204e4d0
                )

            [type] => Archive
            [imei] =><PHONE_NUMBER>15044
            [Version] => 1.20,H02
            [TimeStamp] => 23/05/2018 07:42:22
            [utcDate] => 2018-05-23
            [utcTime] => 07:42:22
            [Status] => A
            [Lati] => 23.033645
            [LatiDir] => N
            [Long] => 72.524738
            [LongDir] => E
            [Altitude] => 54.2
            [Speed] => 9.17
            [Acceleration] => 0
            [Breaking] => 0
            [Heading] => 209.04
            [GSMSignal] => 23
            [GSMCellID] => 2BAE
            [ExtBatt] => 1
            [IntBatt] => 3.84
            [TempOnBoard] => 49.0
            [Odometer] => 2.555
            [DI1] => 1
            [DI2] => 1
            [DI3] => 1
            [DI4] => 1
            [DO1] => 0
            [DO2] => 0
            [AI1] => 0.00
            [AI2] => 0.00
            [Switch1] => 0
            [Switch2] => 0
            [Switch3] => 0
            [Switch4] => 0
            [Switch5] => 0
            [created_date] => 23-05-2018 07:39:42 PM
        )

)

How can i extract value of some parameter like
[ExtBatt] => 1
[IntBatt] => 3.84
[TempOnBoard] => 49.0
[Odometer] => 2.555

and can show it on dashboard by graph

I assume the output above was produced with print_r().
Since MongoDB\Model\BSONDocument extends ArrayObject and specifies the ARRAY_AS_PROPS flag, you can access individual fields within the document using either the array accessor (e.g. $document['ExtBatt']) or property accessor (e.g. $document->ExtBatt).
If you are asking about limiting the fields returned by a query, I would suggest reviewing the Project Fields to Return from Query tutorial in the MongoDB manual, which has code examples available in PHP syntax. There, you'll see how the projection option for MongoDB\Collection::find() can be used to filter out unneeded fields from documents returned from a query.

Closing due to inactivity. Feel free to follow up if needed.


expansion doesn't trigger syntax highlighting

I use zsh-syntax-highlighting, and expanding a suggestion seems to 'break' syntax highlighting until another key is pressed.

This is not expected behavior. Can you give some details about your setup (OS, zsh version, plugin version)? It would also be helpful if you could provide minimum steps to reproduce the problem.
For example, I'm not seeing the issue with Mac OS X, zsh 5.2, z-asug v0.3.1, z-sy-h v0.4.1, and the following:
% zsh -f
%% source zsh-syntax-highlighting.zsh; source zsh-autosuggestions.zsh
%% echo hello
%% echo <expand suggestion>

I'm using Ubuntu 16.04 and zsh 5.1.1 with oh-my-zsh. I also have another machine running Arch and zsh 5.2 + omz that does the same thing, both plugins are git pulled daily via cron on all machines.
Even without omz, and manually sourcing as per your example above, expanding doesn't trigger syntax highlighting of the expanded text..

Same issue here. The expanded part is not being syntax-highlighted after accepting a suggestion with <Ctrl>+<Space>.
Adding another space after expansion to triggers syntax-highlighting correctly afterwards but most of the times you just want to run the suggested command as next step.

FWIW, while I haven’t tried to further identify the cause so far, I was able to work around this issue by calling zle redisplay; zle redisplay; (yes, twice) after using the autosuggest-accept widget.

@eigengrau Can you please paste the patch for others to use the workaround ?

I didn’t patch the sources, only wrapped them in my .zshrc like this:
my-autosuggest-accept() {
    zle autosuggest-accept
    zle redisplay
    zle redisplay
}

zle -N my-autosuggest-accept

ZSH_AUTOSUGGEST_IGNORE_WIDGETS+=my-autosuggest-accept

bindkey '^ ' my-autosuggest-accept

@eigengrau Thank you.


Add Login page link button

In this situation, there should be a button below the "You are not logged in" message that points to the Login Page.
Please do this for the following pages:

Resume Analyzer
Job Tracker

Please assign me this issue under gssoc24

@SurajSanap I tried to implement that but can't found to do that as in streamlit there isn't a way to do so. I have tried streamlit-extras as well but there are errors. So that's why i can't do it. Please unassign


Bump notary version to v0.3.0-RC1

This PR mainly bumps the notary version up to v0.3.0-RC1.
There are 2 Notary API changes relevant to Docker in this change:

NotaryRepository.Update now returns just an error
NewNotaryRepository now takes a trust pinning configuration - we are just providing an empty one here, so that behavior does not change (TOFU is enabled), but a future PR should probably add trust pinning configuration to Docker itself (to only trust roots signed by a particular CA, or to otherwise disable TOFU)

This PR also removes the pkcs11 tag for Dockerfile.s390x as per #22007, since that depends upon GCCGo 5.3, which supports the Go 1.4 standard library.  Notary + pkcs11 (the yubikey support) requires an interface that was added in Go 1.5.

LGTM

(just rebased - also bump :)  once this gets merged we have another RC we'd like to bump to, but would love to split this until multiple PRs because the dependency changes are fairly large)

LGTM

@LK4D4 Thank you!


DataRecords

Include where customers go next
Include number of people at node when they leave

Fixed with commit b8f270be5a7cf75811abb06b85d77331825cb255


Back Arrow Interaction - Find Records in both pre-board search locations

In Board Vessel search and the Find Records search from the homepage: Back arrow should take users back where they just came from as per https://docs.google.com/document/d/1P3mzpbmd1iaXudXnmArLdawOrDCdmhG4WMwXPC0682c/edit#bookmark=id.tjt6bu9b3qv8

@Sheeri
This is native android behaviour to hide keyboard.
On Android we don't have another mechanism to hide keyboard. Thats why if user press back arrow it hides automatically. Next tap on back will close the screen

The back arrow shouldn’t have the same exact functionality as the triangle Android bottom button. Please let the triangle button has its own functionality, and the back arrow goes back to the previous screen on the first tap.
So if a user wants the keyboard to go away they can hit the triangle button, and if they want to go back they hit the back arrow at the top.
Note that if the user does make the keyboard go away, the back arrow at the top should still be there.


Using proxy server with Atom is very complex

You can configure Atom (OSX) to use a HTTP proxy as follows:

Add proxy info to APM:
apm config set proxy http://localhost:9999
apm config set http-proxy http://localhost:9999
apm config set strict-ssl false # only add if needed - may be required if your proxy uses a non-globally-accepted SSL cert

Run HTTP_PROXY="http://<IP_ADDRESS>:9999" HTTPS_PROXY="http://<IP_ADDRESS>:9999" atom --proxy-server="http://<IP_ADDRESS>:9999"

You can now update Atom & packages/themes from within Atom.

(sidenote: git clones still won't work and require proxy configuration in $HOME/.gitconfig or $HOME/.ssh/config.)
This is reallllllly complex, and AFAICT not documented anywhere. I cobbled together this information from:

atom/atom-shell#584
atom/atom#1807 (very helpful)
atom/atom-shell#183

This ticket is about the following issues:

Atom needs proxy config specified three ways (should only need to specify this data in once)
There's no documentation of this process (steps should be mentioned in the manual)

Edit $HOME/.atom/.apm/.apmrc and add:

Your .apmrc file should go in $HOME/.atom/.apmrc, $HOME/.atom/.apm/.apmrc is the path to the global file that is a fallback.

Ah, thanks, edited my original comment.

It appears the http_proxy/https_proxy env vars are used by whatever grabs the avatar icons, so you do need to specify those. Edited original comment again.

+1
Also my system uses a .pac file for internet access meaning when I'm not on the corporate network everything still works. Having a fixed proxy means I need to keep adding and removing depending on where I am. It would be nice for it to fallback if the proxy servers fails to respond

as far as I can tell, atom doesn't respect http_proxy nor https_proxy environment variables, and there other issues logged about it still open:

Not using env: https://github.com/atom/apm/issues/398 and https://github.com/atom/atom/issues/1807
Icons go missing: https://github.com/atom/atom/issues/9346
Problem with custom corporate CA signing certs: https://github.com/atom/apm/issues/340

https://github.com/atom/apm/pull/447 is a recent fix for getting the upper case HTTP_PROXY envrioment variables, but not sure about lowercase...

I would love if Atom just worked worked with OSX's proxy auto discovery

I believe the original issue is fixed based on the latest documentation. At least, I feel that the instructions for apm config set are much more understandable than hunting down a specific configuration file.
@alexjurkiewicz, do you agree?

@lee-dohm, I like the apm config set info, thanks for adding that. I'll edit the OP with the update. However as per OP I had to set proxy config in three places. The documentation you link to refers to just one.
#9346 shows that even with proxy added to APM config, some web resources fail to load. They work when the env vars and/or command-line option is used (I forget which).
I don't work in a proxied environment so I'll unsubscribe from this ticket soon, but IMO this issue is not fixed, for two reasons:
a) the current documentation is insufficient
b) even sufficient documentation still leaves my original complaint that "Using proxy server with Atom is very complex"

Thanks for replying back and for updating the original description.

@alexjurkiewicz I've tried doing the steps you mentioned for Windows 7 Enterprise Edition 64 bit. When trying to search and "Install packages", I get this error :
tunneling socket could not be established, cause=socket hang up

Sorry, I can't troubleshoot your setup. I suggest trying minimal changes as per the docs and only attempting what my post suggests if the official instructions fail.

@alexjurkiewicz Nevermind. I have found a solution that worked for me over here :
http://stackoverflow.com/a/31174009/1186038
Summary: Configuring for http-proxy or HTTPS_PROXY did not do anything for me.
Only setting proxy and 'strict-ssl=false' did.

Still not working with the system's .pac file? Can it be done so it is more easier to work with Atom on a Windows environment inside a corporate?

To help move the focus of this away from narrow solutions such as environment variables, Firefox provides a great example of concise and complete connection settings options along with a sensible default (Use system proxy settings). An equivalent solution in Atom would be a fantastic resolution to this issue.

As of Atom v1.15, it uses system proxy settings by default. The proxy settings, if one needs to set them, have been documented for quite a while now. So I believe everything from this issue has been covered. It is true that we don't have a GUI for editing proxy settings ... but that's not something we want to offer really when using the system settings should work for the vast majority of people.

I keep getting network error even though chrome works. i guess this comes from the fact proxy is configured from a tac file


Revert "Remove hardcoded timeout in favor of relying on global ZK timeout"

Reverts slackhq/astra#1130

It appears that occasionally on new cache initialization, larger ZK clusters the initialized event is either lost or not correctly triggered in Curator.


Problem With Spring Native Query Count total only last date

https://docs.jboss.org/hibernate/orm/6.0/userguide/html_single/Hibernate_User_Guide.html#sql
maybe some help

asking in stackoverflow:
https://stackoverflow.com/questions/57201205/spring-jpa-count-in-native-query-wrong-result

already find better solutions with using thymeleaf aggregates


refactor: Replace the duplication of '\s\t>' regular expressions with single source of truth

Reviewing #953, I see 4 regular expressions needed to be changed in order to add support for callouts and block quotes:

This is an unnecessary duplication of core logic, and a pit-fall for maintainers, as it is easy to not change all locations.
I would like there to be a single source of truth for the identification of the start of a task line.

Possible strategy for this:

In Task.ts define constant strings for INDENTATION = "^([\\s\\t>]*)", BULLET = "[-*]", CHECKBOX = "\\[(.)\\]", AFTER_CHECKBOX = " *(.*)"
In Task.ts change taskRegex to new RegExp(INDENTATION + BULLET + " +" + CHECKBOX + AFTER_CHECKBOX, 'u');
In CreateOrEdit.ts change nonTaskRegex to new RegExp(INDENTATION + BULLET + "? *" + CHECKBOX + "?" + AFTER_CHECKBOX, 'u');
In ToggleDone.ts change listItemRegex to new RegExp(INDENTATION + "(" + BULLET + ")"); and the un-named one to line.replace(new RegExp(INDENTATION), '$1- ');

That looks excellent.

We could even move all the regex definitions into Task.ts to make them easier to cover with automated tests.

I like that idea. Having them all together does seem sensible.

Question about this strategy: Is there any performance cost to doing new RegExp instead of using literals? I do not know how this works in JS.

I don't know. Once sytone's logging code is hooked up, we can see how the measurement code works.
But I strongly favour readable, safely maintainable code - and we are making so many other speed improvements to the code, plus the vast bulk of the time is in rendering, not text manipulation, that my assumption is that this will be fine.

There is this method for combining regexes, but I'm unsure it's useful in this case:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/source


Add SMS based voting

SMS voting could allow question authors to list questions that are less likely to be voted on more than once by a user.
SMS voting would disable the web based form and allow only one vote per phone number per question. If a user texts a vote more than once then their vote is updated as per the website behaviour.

This sounds interesting. I'll setup a test Twillio account and see how it goes.
However this requires a few changes in the UI that I'll think of and communicate with you.

Did anyone end up working on this? I could use this feature for a project I'm working on.
I'm open to uncompleted forks or feature branches 😄

Could this be expanded on to allow each slice of the graph to be selected and expanded, revealing a comment section from those votes, explaining their individual reasoning? People love reading comments and to have it selectively organized like that would be a very good way to "self-educate" ourselves with other points of view--which would allow a period of time to recast that vote differently from the same registered device?


grpc reverse proxy

Hi,
I would like to get more information about the grpc-proxy you built. can it also be used for Tensorflow serving (it uses grpc request / response)? I want to read the header and re-write it and forward it to to TensorFlow Serving and read the response and forward it to the grpc client.
grpc client <-> grpc proxy <-> TensorFlow Serving
Thanks.

I have the same question which you have meet.We can have a touch by wechat

I use envoy proxy for now

I use envoy proxy for now

can you share a sample config? thanks

Hi,
I would like to get more information about the grpc-proxy you built. can it also be used for Tensorflow serving (it uses grpc request / response)? I want to read the header and re-write it and forward it to to TensorFlow Serving and read the response and forward it to the grpc client.
grpc client <-> grpc proxy <-> TensorFlow Serving
Thanks.

hello，i have the same question. Have you found a solution？

Hi,
I would like to get more information about the grpc-proxy you built. can it also be used for Tensorflow serving (it uses grpc request / response)? I want to read the header and re-write it and forward it to to TensorFlow Serving and read the response and forward it to the grpc client.
grpc client <-> grpc proxy <-> TensorFlow Serving
Thanks.

hello，i have the same question. Have you found a solution？

@peterpunk  @aalansehaiyang
I used envoy proxy in front of reserving:
Dockerfile:
FROM envoyproxy/envoy:v1.14.1

RUN apt-get update && apt-get -q install -y \
    curl
ADD ./front-envoy.yaml /etc

EXPOSE 8080

CMD /usr/local/bin/envoy -c /etc/front-envoy.yaml --service-cluster front-proxy

front-envoy.yaml:
static_resources:
  listeners:
    - name: api
      address:
        socket_address:
          protocol: TCP
          address: <IP_ADDRESS>
          port_value: 8080
      filter_chains:
        - filters:
            - name: envoy.filters.network.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                codec_type: auto
                stat_prefix: ingress_http
                #access_log:
                #  - name: envoy.file_access_log
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
                #      path: /dev/stdout
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.config.accesslog.v2.FileAccessLog
                #      path: /Users/tien.nguyenvan/Downloads/access.log
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: www
                      domains:
                        - "*"
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            cluster: tfs
                http_filters:
                  - name: envoy.filters.http.router
  clusters:
    - name: tfs
      connect_timeout: "0.25s"
      type: logical_dns
      lb_policy: round_robin
      dns_lookup_family: V4_ONLY
      #http_protocol_options: {}
      http2_protocol_options: {}
      #tls_context: {}
      hosts:
        socket_address:
          address: "tfserving"
          port_value: 8500

admin:
  access_log_path: "/dev/null"
  #access_log_path: /Users/tien.nguyenvan/Downloads/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 8081

@peterpunk  @aalansehaiyang
I used envoy proxy in front of reserving:
Dockerfile:
FROM envoyproxy/envoy:v1.14.1

RUN apt-get update && apt-get -q install -y \
    curl
ADD ./front-envoy.yaml /etc

EXPOSE 8080

CMD /usr/local/bin/envoy -c /etc/front-envoy.yaml --service-cluster front-proxy

front-envoy.yaml:
static_resources:
  listeners:
    - name: api
      address:
        socket_address:
          protocol: TCP
          address: <IP_ADDRESS>
          port_value: 8080
      filter_chains:
        - filters:
            - name: envoy.filters.network.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                codec_type: auto
                stat_prefix: ingress_http
                #access_log:
                #  - name: envoy.file_access_log
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
                #      path: /dev/stdout
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.config.accesslog.v2.FileAccessLog
                #      path: /Users/tien.nguyenvan/Downloads/access.log
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: www
                      domains:
                        - "*"
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            cluster: tfs
                http_filters:
                  - name: envoy.filters.http.router
  clusters:
    - name: tfs
      connect_timeout: "0.25s"
      type: logical_dns
      lb_policy: round_robin
      dns_lookup_family: V4_ONLY
      #http_protocol_options: {}
      http2_protocol_options: {}
      #tls_context: {}
      hosts:
        socket_address:
          address: "tfserving"
          port_value: 8500

admin:
  access_log_path: "/dev/null"
  #access_log_path: /Users/tien.nguyenvan/Downloads/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 8081

@peterpunk @aalansehaiyang
I used envoy proxy in front of reserving:
Dockerfile:
FROM envoyproxy/envoy:v1.14.1

RUN apt-get update && apt-get -q install -y \
    curl
ADD ./front-envoy.yaml /etc

EXPOSE 8080

CMD /usr/local/bin/envoy -c /etc/front-envoy.yaml --service-cluster front-proxy

front-envoy.yaml:
static_resources:
  listeners:
    - name: api
      address:
        socket_address:
          protocol: TCP
          address: <IP_ADDRESS>
          port_value: 8080
      filter_chains:
        - filters:
            - name: envoy.filters.network.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                codec_type: auto
                stat_prefix: ingress_http
                #access_log:
                #  - name: envoy.file_access_log
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
                #      path: /dev/stdout
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.config.accesslog.v2.FileAccessLog
                #      path: /Users/tien.nguyenvan/Downloads/access.log
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: www
                      domains:
                        - "*"
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            cluster: tfs
                http_filters:
                  - name: envoy.filters.http.router
  clusters:
    - name: tfs
      connect_timeout: "0.25s"
      type: logical_dns
      lb_policy: round_robin
      dns_lookup_family: V4_ONLY
      #http_protocol_options: {}
      http2_protocol_options: {}
      #tls_context: {}
      hosts:
        socket_address:
          address: "tfserving"
          port_value: 8500

admin:
  access_log_path: "/dev/null"
  #access_log_path: /Users/tien.nguyenvan/Downloads/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 8081

Thank you very much. I'll try

@peterpunk @aalansehaiyang
I used envoy proxy in front of reserving:
Dockerfile:
FROM envoyproxy/envoy:v1.14.1

RUN apt-get update && apt-get -q install -y \
    curl
ADD ./front-envoy.yaml /etc

EXPOSE 8080

CMD /usr/local/bin/envoy -c /etc/front-envoy.yaml --service-cluster front-proxy

front-envoy.yaml:
static_resources:
  listeners:
    - name: api
      address:
        socket_address:
          protocol: TCP
          address: <IP_ADDRESS>
          port_value: 8080
      filter_chains:
        - filters:
            - name: envoy.filters.network.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                codec_type: auto
                stat_prefix: ingress_http
                #access_log:
                #  - name: envoy.file_access_log
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.extensions.access_loggers.file.v3.FileAccessLog
                #      path: /dev/stdout
                #    typed_config:
                #      "@type": type.googleapis.com/envoy.config.accesslog.v2.FileAccessLog
                #      path: /Users/tien.nguyenvan/Downloads/access.log
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: www
                      domains:
                        - "*"
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            cluster: tfs
                http_filters:
                  - name: envoy.filters.http.router
  clusters:
    - name: tfs
      connect_timeout: "0.25s"
      type: logical_dns
      lb_policy: round_robin
      dns_lookup_family: V4_ONLY
      #http_protocol_options: {}
      http2_protocol_options: {}
      #tls_context: {}
      hosts:
        socket_address:
          address: "tfserving"
          port_value: 8500

admin:
  access_log_path: "/dev/null"
  #access_log_path: /Users/tien.nguyenvan/Downloads/admin_access.log
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 8081

Thank you very much. I'll try


Fix flaky test in BraveServiceIntegrationTest

Motivation:
I found the follow exception while testing
BraveServiceIntegrationTest > notFound FAILED
    org.junit.runners.model.TestTimedOutException: test timed out after 5 seconds
        at java.base@13.0.2/java.lang.Class.isPrimitive(Native Method)
        at java.base@13.0.2/java.lang.invoke.MethodHandles.constant(MethodHandles.java:3371)
        at java.base@13.0.2/java.lang.invoke.InnerClassLambdaMetafactory.buildCallSite(InnerClassLambdaMetafactory.java:211)
        at java.base@13.0.2/java.lang.invoke.LambdaMetafactory.metafactory(LambdaMetafactory.java:329)
        at java.base@13.0.2/java.lang.invoke.LambdaForm$DMH/0x0000000800be1440.invokeStatic(LambdaForm$DMH)
        at java.base@13.0.2/java.lang.invoke.Invokers$Holder.invokeExact_MT(Invokers$Holder)
        at java.base@13.0.2/java.lang.invoke.BootstrapMethodInvoker.invoke(BootstrapMethodInvoker.java:127)
        at java.base@13.0.2/java.lang.invoke.CallSite.makeSite(CallSite.java:307)
        at java.base@13.0.2/java.lang.invoke.MethodHandleNatives.linkCallSiteImpl(MethodHandleNatives.java:259)
        at java.base@13.0.2/java.lang.invoke.MethodHandleNatives.linkCallSite(MethodHandleNatives.java:249)
        at app//com.linecorp.armeria.server.ServerConfig.<init>(ServerConfig.java:229)
        at app//com.linecorp.armeria.server.ServerBuilder.build(ServerBuilder.java:1467)
        at app//com.linecorp.armeria.server.brave.BraveServiceIntegrationTest.init(BraveServiceIntegrationTest.java:81)
        at app//brave.test.http.ITHttpServer.setup(ITHttpServer.java:58)
        at app//com.linecorp.armeria.server.brave.BraveServiceIntegrationTest.setup(BraveServiceIntegrationTest.java:55)

Modifications:

Set globalTimeout to 15 seconds

Result:
No more flaky

@adriancole is fixing the test. https://github.com/openzipkin/brave/pull/1117
So we probably do not need this. 😉

@adriancole is fixing the test. openzipkin/brave#1117
So we probably do not need this. 😉

That is nice 👍 Thanks for sharing!

ps not sure the brave thing will have anything to impove this as somehow it took 15 seconds in the setup method. this method wasn't affected by brave 1117

https://github.com/line/armeria/blob/master/core/src/main/java/com/linecorp/armeria/server/ServerConfig.java#L229 is the line.. any smoking gun?

/core/src/main/java/com/linecorp/armeria/server/ServerConfig.java@master#L229  is the line.. any smoking gun?

The line just flattens the nested lists. 🤔

I can't see anything in the stack trace that makes any sense.. wonder if verbose:gc or something? because setup has no reason to take longer than 5 seconds..

my guess is next step might be to either trust gradle's servers and use their build scan thing https://scans.gradle.com/ or add some JVM flags and/or archive gc logs at the end of a run.
when we were talking about a second or two, sure a test could take a bit to warm up, but this seems like something else. cc @anuraaga for advice.
PS I already tried the --profile thing, it isn't useful as it has no GC stats

FWIW I've heard of reports of a 10s timeout failing in setUp, but different project and CI. I don't know if there is anything really related though, other than our surprise at slow setUp methods :) I had a suspicion maybe the stack trace is wrong but didn't investigate when 20s timeout fixed it.

I will change brave to 20s and mention in the note that this isn't about tests taking long, rather that it is defense against CI hosts under heavy load and/or extreme GC. I think with such a comment folks won't remove it.. our "poll" ops can stay the same as far as I can tell.

I can't see anything in the stack trace that makes any sense.. wonder if verbose:gc or something? because setup has no reason to take longer than 5 seconds..

Yes. the stack trace has no useful information. Let me try to get other pieces of evidence.


Q module with Node-github

Hi. Thanks for the great module. It's covering basically all my needs here but one.
I have to list a list of issues by ids ( like 567, 555, 523 ) and it seems that I can't do this filter so I was thinking in use the Q module to make N requests ( N as how many issues ) and them returning it as an array to the response. But it seems that you don't work with promises so this is not working:
var issuesArray = [567, 565];
var promiseArray = [];
for(var i = issuesArray.length; i--;) {
    promiseArray.push(getIssues(github, obj, issuesArray[i]));
}

Q.all(promiseArray).then(function(issues){
    console.info(issues)
}).done();

function getIssues(github, obj, issueid) {
    return github.issues.getRepoIssue({
        user: obj.issue.user,
        repo: obj.issue.repo,
        number: issueid
    });
}

Does anyone did something like this? Or do you have an endpoint that I can filter by IssueId?
Thanks for the attention.

You should be able to use bluebird promisifyAll: http://bluebirdjs.com/docs/api/promise.promisifyall.html

Merged in a more flexible solution that allows you to use Promise library of choice: #262


Logical Model: intended performer

#128
Be-assigned-Performer	intendedPerformer	1 .. *	0 .. *	Must be "1 .. *" in FIHR Model. (The prescribers knows the discipline of performer and must specifiy it)

WG: duplicate of #132

In a national standard?
This means tha if there is a valid prescription where the physician doesn't
know or care about the intended performer, this prescription can never be
exchanged.
Isn't there any, not even one, use case where intended prescriber is not
known?
On Thu, 28 Jul 2022, 13:05 Bart Decuypere (eHealth), <
@.***> wrote:

#128 https://github.com/hl7-be/referral/issues/128
Be-assigned-Performer intendedPerformer 1 .. * 0 .. * Must be "1 .. *" in
FIHR Model. (The prescribers knows the discipline of performer and must
specifiy it)
—
Reply to this email directly, view it on GitHub
https://github.com/hl7-be/referral/issues/131, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3HUUF4YGR3WD5CWOAO7J3VWJZR5ANCNFSM545AVYOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

The opinion of RIZIV/INAMI (as expressed in document #128) is that a referral prescription has no sense if the discipline or specialty of the caregiver referred to is unknown. In the system as conceived here, the prescription would not be retrievable by a caregiver (a caregiver has only access to prescriptions addressed to his discipline/specialty).
This does not mean that the person must be known to whom the patient is referred to. The patient has the free choice to choose a caregiver, but at least he must know to which specialty/discipline his GP refers him to.


feat(remix): Add support for Remix SPA Mode

Description
WIP
This PR aims to add support to Remix SPA Mode.
Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

@clerk/chrome-extension
1.0.19-snapshot.vbc54be6

@clerk/clerk-js
5.7.1-snapshot.vbc54be6

@clerk/elements
0.8.0-snapshot.vbc54be6

@clerk/clerk-expo
1.2.2-snapshot.vbc54be6

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/remix
4.2.0-snapshot.vbc54be6

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/elements
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

!snapshot

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

@clerk/chrome-extension
1.1.1-snapshot.ve39e3a5

@clerk/clerk-js
5.7.3-snapshot.ve39e3a5

@clerk/clerk-expo
1.2.4-snapshot.ve39e3a5

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/remix
4.2.0-snapshot.ve39e3a5

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact

!snapshot

Hey @anagstef - the snapshot version command generated the following package versions:

Package
Version

@clerk/chrome-extension
1.1.1-snapshot.vdf60950

@clerk/clerk-js
5.7.3-snapshot.vdf60950

@clerk/clerk-expo
1.2.4-snapshot.vdf60950

gatsby-plugin-clerk
5.0.0-beta.45

@clerk/remix
4.2.0-snapshot.vdf60950

Tip: Use the snippet copy button below to quickly install the required packages.
@clerk/chrome-extension
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-js
npm i<EMAIL_ADDRESS>--save-exact

@clerk/clerk-expo
npm i<EMAIL_ADDRESS>--save-exact

gatsby-plugin-clerk
npm i<EMAIL_ADDRESS>--save-exact

@clerk/remix
npm i<EMAIL_ADDRESS>--save-exact


Add lightbend reactive platform frameworks (Play/Akka/Lagom) and scala language

Add Play, Akka, Lagom to frameworks and Scala language.
Logos for reference on lightbend.com

Added Lightbend to languages & frameworks

Hi @kikiya please find in 0.9.4 https://github.com/cncf/landscape/blob/master/landscape/CloudNativeLandscape_v0.9.4.jpg

Sorry, we removed all the programming languages and frameworks from the newest version, whether they are open source or proprietary.


Linter: Update implementation code

Update Dapr implementation code to adhere to new linter rules. All test files have been purposefully omitted to ensure that the implementation code is still functionally correct. A follow-up commit will contain the update to all test code, and removal of the "only new changes linted" from the github linter workflow.

/ok-to-test

/ok-to-test

/ok-to-perf

Thanks @elena-kolevska @mikeee, please take another look!


allow custom entrust eku and digicert use_csr_fields

Currently, it's not possible to create a lemur or digicert cis cert with custom ekus. This change adds the ability to specify entrust EKU settings via configuration and to defer to the csr ekus for digicert cis

coverage: 75.339% (+0.06%) from 75.275%
when pulling a9854f515af67015254b756e9cbb6ec223db7418 on jmcrawford45:PS-5508-entrust
into c90d6b2a6ac787fb88136d7b45dd2c84401ae36c on Netflix:main.


set modified time when modifying file

Gulp plugins that create/modify output files are responsible for updating file modified timestamps as of Gulp 4 https://github.com/gulpjs/gulp/issues/2193#issuecomment-399168738
This change simply updates the file stat appropriately.
Fixes #706

Welp, it fails the tests, but the idea is right.

Would be nice if this actually got merged ;)

A colleague and I just spent 2.5 hours debugging an issue that was ultimately caused by gulp-sass not updating the mtime of the output files. It would be really nice if this was merged (after fixing the linting issue I guess).

Either of you know how to fix this so the tests pass?

Close in favor of https://github.com/dlmanning/gulp-sass/pull/763

Released in v4.1.0


Describe the "reasons" config-field

Is your feature request related to a problem? Please describe.
As it is a very handy feature to not alert on every pod shutting down/killing/whatever event, the reasons field could be very handy. However, there is
a) no mentioning about  it except this github repo (its missing on the page)
b) there is no documentation about it (e.g. about what potential values are)
This makes it very hard to start using it
Describe the solution you'd like
Having  a config-field which expects  certain enums should document the enum-values

Hey! I got a look on this subject in hope to improve filter of Error.
concretely for now, kwatch grab the reason field from the kubernetes-provided field for container.State.*.Reason :
cf https://github.com/abahmed/kwatch/blob/main/controller/controller.go#L196
For a such field, I was unable to find some explicit docs from kubernetes, field is considered Experimental so they don't provide more details and it sounds this field can be provided by the container itself. We can only find that "reason is a unique, one-word, CamelCase"
from a few search here a list (probably incomplete and not up to date however ) :
For Waiting container status it is possible to get one of following reason:

ContainerCreating,
CrashLoopBackOff,
ErrImagePull,
ImagePullBackOff,
CreateContainerConfigError,
InvalidImageName,
CreateContainerError.

For Terminated container status there are different list of reasons:
OOMKilled,
Error,
Completed,
ContainerCannotRun,
DeadlineExceeded

if it can help someone!


feat(cli): add -r and -v flags for safer usage

rm will fail if given a directory without the -r flag (similar to unix rm)
rm will print the removed files/folders when given the -v flag

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


fix(EVM): Fix DUP1 underflow check

What ❔

Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.

Closed in favor of https://github.com/matter-labs/era-contracts/pull/1134


** (FunctionClauseError) no function clause matching in Livebook.LiveMarkdown.Import.grab_leading_comments/1

Environment

Elixir & Erlang/OTP versions (elixir --version): master
Operating system: macOS
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc): livebook CLI
Livebook version (use git show if running with mix): master

Current behavior
I'm running latest Livebook, installed with $ mix escript.install github livebook-dev/livebook
Importing this notebook:
https://github.com/wojtekmach/notebooks/blob/main/req_0.2.0.livemd
fails with this in the server log:
11:10:51.142 [error] #PID<0.385.0> running LivebookWeb.Endpoint (connection #PID<0.366.0>, stream id 6) terminated
Server: localhost:8080 (http)
Request: GET /import?url=https%3A%2F%2Fgithub.com%2Fwojtekmach%2Fnotebooks%2Fblob%2Fmain%2Freq_0.2.0.livemd
** (exit) an exception was raised:
    ** (FunctionClauseError) no function clause matching in Livebook.LiveMarkdown.Import.grab_leading_comments/1
        (livebook 0.3.1) lib/livebook/live_markdown/import.ex:289: Livebook.LiveMarkdown.Import.grab_leading_comments([{:cell, :markdown, [{:comment, [], ["vim: syntax=markdown"], %{comment: true}}]}, {:metadata, %{"persist_outputs" => true}}])
        (livebook 0.3.1) lib/livebook/live_markdown/import.ex:259: Livebook.LiveMarkdown.Import.build_notebook/4
        (livebook 0.3.1) lib/livebook/live_markdown/import.ex:18: Livebook.LiveMarkdown.Import.notebook_from_markdown/1
        (livebook 0.3.1) lib/livebook_web/live/home_live.ex:399: LivebookWeb.HomeLive.import_content/3
        (livebook 0.3.1) lib/livebook_web/live/home_live.ex:231: LivebookWeb.HomeLive.handle_params/3
        (phoenix_live_view 0.17.5) lib/phoenix_live_view/utils.ex:369: anonymous fn/5 in Phoenix.LiveView.Utils.call_handle_params!/5
        (telemetry 0.4.3) /private/var/folders/xx/q05b_dv958b3ts198z66l56h0000gn/T/mix-local-installer-fetcher-qU4WAw/deps/telemetry/src/telemetry.erl:272: :telemetry.span/3
        (phoenix_live_view 0.17.5) lib/phoenix_live_view/static.ex:261: Phoenix.LiveView.Static.call_mount_and_handle_params!/5

Expected behavior
Importing works.

<!-- livebook:{"persist_outputs":true} -->
<!-- vim: syntax=markdown -->

# Req v0.2.0

Ah! So technically custom comments should go at the very beginning, and the livebook metadata should go straight before the element it concerns, in this case before the heading1. The import shouldn't definitely break, but the vim comment is not in a valid place, so we would probably skip it, I'll have a look.

Thanks, moving comments around fixed it!


Waveshare Pico LCD 1.3 Support

Can you please add support for the Waveshare display 1.3 LCD to the Micro GUI?  I can get it loaded, and even running, but it is off center and isn't behaving quite right.  I don't know if it's a config thing or a driver thing.  I can send you a photo.

Unfortunately I don't have one of these displays.
The problem here is that different manufacturers wire up the ST7789 in different ways,necessitating a dedicated display 3-tuple. If you look at this code you will see examples, some contributed by users. You may find that one of these works for your display: try them, e.g. by passing display=PICO_LCD_2 to the constructor. I suggest starting with this one as it's a Waveshare display.
If none of the provided tuples works, try experimenting, starting with whichever provides the best results. See drivers doc.
To evaluate a tuple, use the quick hardware check. If this produces pixel-perfect results you can be sure that the GUI will work.
Please post your results and I will update the driver and doc.

Sweet! Thanks! I'll give it a gander after work this evening. The 1.14 inch, 240x240 hardware_setup is good enough to get the UI to work on the 1.3 device and 3 of the inputs are functional, so it's close, but not quite there. I'll let you know; thanks again.
On a personal note, I was perusing your GitHub and I just wanted to say that I am grateful for people like you who have contributed significantly to the foundation of work related to micropython and controllers in general. If my buddy and I ever produce more than two devices related to our project, I will personally send you a pair.

Here's how it looks right now, just an FYI. Sorry about the picture quality. You know how it is. Anyway, I'll have another peek tonight with fresh eyes. Thanks!

Greetings Again Mr. Hinch,
You were absolutely right.  The magic combo for the display 3-tuple was (0, 0, 16) for this hardware: https://www.waveshare.com/wiki/Pico-LCD-1.3 .
Hardware test

GUI Demo

Thank you again for your hard work and your time.  It will be invaluable to my friend and my pet project.

Excellent, thank you for letting me know. I've added a constant WAVESHARE_13 to the code and driver doc.


Merge master to fixgraph branch

Description of Change
As title
Checklist

[X] Added description of change
[X] I acknowledge that all my contributions will be made under the project's license.

Notes:

Please do not create branches on the main repo. Use forks to create pull-requests. This will prevent any inadvertent errors cleaner process. Thanks.

closing this Pr as it is affecting the main repo CI checks.
@fhlasek please create this PR from a branch in your fork. Thank you.


Kotlin native could not find "openssl"

I am trying to create project using this library with kotlin native on linux 64bit, but i am getting strange error about missing openssl. I have installed openssl and libssl-dev but the error still persists.
e: Could not find "openssl" in [/home/fjerabek/mqttTest, /home/fjerabek/.konan/klib, /home/fjerabek/.konan/kotlin-native-prebuilt-linux-x86_64-1.7.0/klib/common, /home/fjerabek/.konan/kotlin-native-prebuilt-linux-x86_64-1.7.0/klib/platform/linux_x64]

Execution failed for task ':compileKotlinNative'.
> Compilation finished with errors

Am i missing some other library on my system ?

Hello, how did you include the dependency in gradle? Also it looks like you are using Kotlin 1.7.0, I have yet to try to compile with that version but I will check soon.

I am including the dependency by adding this to the build.gradle.kts file into the sourceSets block.
commonMain {
    dependencies {
        implementation("com.github.davidepianca98.KMQTT:kmqtt:0.3.2")
    }
}

The resulting build file looks like this. It is the default template that jetbrains is using for kotlin native projects only with the added dependency.
plugins {
    kotlin("multiplatform") version "1.7.0"
}

group = "org.example"
version = "1.0-SNAPSHOT"

repositories {
    mavenCentral()
    maven(url = "https://jitpack.io")
}

kotlin {
    val hostOs = System.getProperty("os.name")
    val isMingwX64 = hostOs.startsWith("Windows")
    val nativeTarget = when {
        hostOs == "Mac OS X" -> macosX64("native")
        hostOs == "Linux" -> linuxX64("native")
        isMingwX64 -> mingwX64("native")
        else -> throw GradleException("Host OS is not supported in Kotlin/Native.")
    }

    nativeTarget.apply {
        binaries {
            executable {
                entryPoint = "main"
            }
        }
    }
    sourceSets {
        commonMain {
            dependencies {
                implementation("com.github.davidepianca98.KMQTT:kmqtt:0.3.2")
            }
        }
        val nativeMain by getting
        val nativeTest by getting
    }
}

For now i am trying out the library in jvm and kotlin 1.7.0 and it working great so far, but cannot get it to work with native.

I must be missing something :thinking:. I added the dependency to the openssl library and looks like it loaded correctly because i can use openssl library methods, but now the problem is not in compileKotlinNative task but in linkDebugExecutableNative.
e: Could not find "openssl" in [/home/fjerabek/mqttTest, /home/fjerabek/.konan/klib, /home/fjerabek/.konan/kotlin-native-prebuilt-linux-x86_64-1.7.0/klib/common, /home/fjerabek/.konan/kotlin-native-prebuilt-linux-x86_64-1.7.0/klib/platform/linux_x64]

Execution failed for task ':linkDebugExecutableNative'.
> Compilation finished with errors

When i add the klib file to one of those locations and change the name to openssl.klib gradle complains about caching
/home/fjerabek/.gradle/caches/modules-2/files-2.1/com.github.davidepianca98.KMQTT/kmqtt-linuxx64/0.3.2/704c50f7b3bf5fd394f044a485dbba73cdc81f59/kmqtt-linuxx64-0.3.2 is going to be cached, but its dependency isn't: /home/fjerabek/mqttTest/openssl

This is weird. The same build script works on windows (I tried it on my windows machine) but not on linux. I tried cleaning the build cache and still no difference. The gradle versions are the same. I also tried it inside wsl to try other "linux" system and i cannot get it build in wsl too.

You are right, I have no idea what's going on, because linkReleaseExecutableNative works but linkDebugExecutableNative doesn't. I will continue to try to find the issue but it could be something wrong with the Kotlin build system.

Solved in 0.3.3

I'm experiencing this with Kotlin Multiplatform, on Linux build:

Task :shared:compileKotlinLinuxX64 FAILED
e: KLIB resolver: Could not find "openssl" in [/home/qus/StudioProjects/dms-agent/linuxX64App, /home/qus/.konan/klib, /home/qus/.konan/kotlin-native-prebuilt-linux-x86_64-2.0.0/klib/common, /home/qus/.konan/kotlin-native-prebuilt-linux-x86_64-2.0.0/klib/platform/linux_x64]

KLIB resolver: Could not find "openssl" in [/home/qus/StudioProjects/dms-agent/linuxX64App, /home/qus/.konan/klib, /home/qus/.konan/kotlin-native-prebuilt-linux-x86_64-2.0.0/klib/common, /home/qus/.konan/kotlin-native-prebuilt-linux-x86_64-2.0.0/klib/platform/linux_x64]
Is there anything I can do? The prebuilt binary doesn't seem to be there anymore...


Feature for contributors/reviewers to privately contact curator

@wrynearson @deborahUAH
Description
Currently, users can use thread mentions to comment amongst each other - contributors, reviewers, and curators.  However, all users associated with a document can see all comments.  Some comments with the curator may need to be private and therefore not seen by everyone.
Tasks:

[ ] Determine/discuss the best approach (simplest) way to allow users to privately interact with the curator (e.g., option for private comments within thread mentions - user selects who sees comments or perhaps an option to send email to the curator via a link)
[ ] Implement approach that is decided on

Acceptance Criteria:

[ ] Users have a method to successfully interact with the curator privately (both contributors and reviewers need to be able to do this)

@deborahUAH could you give an example of when private communication would be helpful for the author/reviewer? In the spirit of openness (open source, open repository for ATBDs), I'd advocate for only having private communication if there's a compelling reason.
cc @batpad

Conditions have changed and this will no longer be required.    Closing issue


How to invert black and white in image? Can't make bitwise_not operation work

Hi,
If I want to invert black and white in my image using python, I can do it like this:
inverted_image = cv2.bitwise_not(image)

But when I try that in Rust I get an error: Error { code: "StsAssert, -215", message: "OpenCV(4.5.4). This is the code:
let mut inverted_black_and_white: Vector<u8> = Vector::new();
bitwise_not(&image, &mut inverted_black_and_white, &Vector::<i32>::new()).expect("Bitwise operation should work");

am I missing something?

What is the full error message that you're getting? But just a hunch, can you try using Mat instead of Vector?

@twistedfall it works if I use Mat
let mut inverted_black_and_white = Mat::default();
bitwise_not(&image, &mut inverted_black_and_white, &Vector::<i32>::new()).expect("Bitwise operation should work");


Legacy Migration Guide (APIF to Sauce)

Description

See https://saucedev.atlassian.net/browse/TCD-2188.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[x] I have read the contributing document.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@kimsauce if Component is not supported user will see something like that

Anyway, the Components not supported anymore are: Update Input, JDBC, Signature, Fake


Document new SSL functionality

 SSL certificates

You need to deploy after changing the manifest.

And I think its convox ssl add port crt key

 :facepunch:

You could also demonstrate how introspecting SSL configuratino works.
$ convox ssl
PORT  EXPIRES            DOMAINS
443   9 months from now  *.foobar.com

@nzoschke Great suggestion. I'll add that.

:+1:


[FLINK-29508][legal] Improve detection of non-deployed modules that need a NOTICE

Fix for flink-fs-hadoop-shaded, flink-s3-fs-base and flink-table-planner-loader-bundle not being covered by license checks as they got auto-ignored due to not being deployed.
The checker now only ignores non-deployed modules if they are not bundled by another deployed module.
Note that this approach will not work if a non-deployed module is bundled by another non-deployed module which is then bundled by a deployed module.
This is currently not a problem though.
I noticed this problem when I saw this in the CI output of a build, despite the NOTICE being correct:
	flink-table-planner-loader-bundle:
		 These issues are mistakes that aren't legally problematic. They SHOULD be fixed at some point, but we don't have to:
			Dependency org.scala-lang:scala-compiler:2.12.7 is not bundled, but listed.
			Dependency org.scala-lang:scala-library:2.12.7 is not bundled, but listed.
			Dependency org.scala-lang:scala-reflect:2.12.7 is not bundled, but listed.

What happened is that the NOTICE was still found, but we already threw out the information about bundles dependencies for that module.
We really need some tests for this...

Let's maybe first get https://github.com/apache/flink/pull/20970 in before making further changes.


Feature/rename global colors

GitHub Issue
Add a link to the GitHub issue.
Changes
A summary of the changes in the code in this PR. If making any UI changes consider adding screenshots or gifs.
Packages Added
Brief list of any packages added, and if folks need to rerun yarn install to run locally
Checks
Before making your PR, please check the following:

[ ] Critical lint errors are resolved
[ ] App runs locally
[ ] App builds locally (run the build command for any impacted package and check for any errors before the PR)

Implementation doesn't follow what Radix had suggested for uses. Because of this, the names of our color variables can be confusing. Ex. button border = primary.hoverElementBg. This could wait until after mcon.


Add Table of Contents for the Operating Page

Signed-off-by: Yash<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Added Table of Contents
Fixed some spelling mistakes and Grammar
Fixed markdown order

Verification
Tested locally

Looks like linkings in TOC are not working. You can test in  https://deploy-preview-2656--thanos-io.netlify.app/operating/troubleshooting.md/

Weird, I tested in my vscode and it seems to be working, let me fix that up!

Can we instead aim for automatic tool as per https://github.com/thanos-io/thanos/pull/2689 ? (:
@yashrsharma44 @povilasv @soniasingla

@yashrsharma44 @bwplotka @povilasv
As I'm working on the automation already, I believe we can avoid adding toc manually, wdyt? :)

Great! If we can automate the process, that would certainly remove the manual work involved. +1 for this!


agvtool in "Sync Version Numbers" Run Script is making it impossible to build

I'm not sure if I'm the only one or not, but I'm unable to build.
Xcode will not throw an error or anything, the build will simply be "cancelled"

I found this stackoverflow question related to the issue, which says it's a problem with agvtool in run scripts, I tested this by removing the "sync version numbers" script and that seems to have fixed it.

Had to do the same thing to build

Yes, I think it's because Xcode will reload the project when it is changed by avgtool.
Are you sure it is the Sync Version Numbers script and not the Increase Build Number script?

After some more testing, seems like it's either one of the scripts.
If I remove either "Increase Build Number" or "Sync Version Numbers", clean and then build it will fail at first attempt, but if I build a second time after that it will succeed.


Structured Logging Phase 2

resolves #4260
Description
This is a feature branch. There will be lots of individual PRs contributing to this branch to achieve the checklist of features outlined in the ticket.
Checklist

[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change

We've decided not to use a feature branch for this phase, since each task in the ticket can be merged into main independently without breaking anything.


Fix Utilities.GetMacAddress may return wrong result

Utilities.GetMacAddress may return empty byte[] that crashes MessageIdGenerator.MessageIdGenerator() and MessageIdGenerator's type initializer.
https://github.com/apache/rocketmq-clients/blob/33caa017045d6dc0339fc32d61d8f8c7169068af/csharp/rocketmq-client-csharp/MessageIdGenerator.cs#L46

This PR is stale because it has been open for 30 days with no activity. It will be closed in 3 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this PR.

reply for keeping alive


[KYUUBI #2033] Hive Backend Engine - GetCrossReference

Why are the changes needed?

Hive Backend Engine - GetCrossReference.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #2194 (01a0065) into master (0489623) will increase coverage by 0.31%.
The diff coverage is 92.80%.

@@             Coverage Diff              @@
##             master    #2194      +/-   ##
============================================
+ Coverage     61.21%   61.52%   +0.31%
  Complexity       69       69
============================================
  Files           329      331       +2
  Lines         15739    15861     +122
  Branches       2010     2013       +3
============================================
+ Hits           9635     9759     +124
+ Misses         5287     5283       -4
- Partials        817      819       +2

Impacted Files
Coverage Δ

...ine/flink/operation/FlinkSQLOperationManager.scala
85.00% <0.00%> (-2.18%)
:arrow_down:

...engine/trino/operation/TrinoOperationManager.scala
86.36% <0.00%> (-4.12%)
:arrow_down:

...org/apache/kyuubi/operation/OperationManager.scala
90.32% <ø> (ø)

...a/org/apache/kyuubi/metrics/MetricsConstants.scala
100.00% <ø> (ø)

.../apache/kyuubi/client/KyuubiSyncThriftClient.scala
89.28% <81.81%> (+4.01%)
:arrow_up:

...apache/kyuubi/server/api/v1/SessionsResource.scala
70.76% <84.61%> (+3.24%)
:arrow_up:

...in/scala/org/apache/kyuubi/server/api/v1/dto.scala
70.76% <85.71%> (+1.80%)
:arrow_up:

...rg/apache/kyuubi/server/BackendServiceMetric.scala
93.44% <88.88%> (-2.72%)
:arrow_down:

...rg/apache/kyuubi/operation/GetCrossReference.scala
93.75% <93.75%> (ø)

...uubi/engine/hive/operation/GetCrossReference.scala
100.00% <100.00%> (ø)

... and 13 more

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 0489623...01a0065. Read the comment docs.

@yaooqinn , could you please help to merge?

thanks, merged to master


restore getRouteResponses (for now), add ResponsesForRouter

fixes #191
@Gabrola mentions that

In order to provide this functionality publicly we'll need to distinguish between inferring the Input schema and Output schema for Zod depending on whether the type will be used by the client or server; kind of similar to how trpc has inferRouterInputs and inferRouterOutputs.

For example for responses, we'd need to use the Zod Output on the client, and Zod Input on the server. For everything else (query, body, etc.) it's vice-versa.

I do eventually think we should take this idea and run with it eventually, however, for the sake of brevity, we need to do the following:

we should restore the original method
add a "better" way of using the original method
document the release versions that this is affected by

We should be careful moving forward on removing methods without proper communication/documentation.

Codecov Report

Merging #192 (ca48500) into main (864d2f0) will not change coverage.
The diff coverage is 50.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #192   +/-   ##
=======================================
  Coverage   62.79%   62.79%
=======================================
  Files          11       11
  Lines         293      293
  Branches       68       68
=======================================
  Hits          184      184
  Misses         97       97
  Partials       12       12

Impacted Files
Coverage Δ

libs/ts-rest/core/src/lib/client.ts
83.67% <50.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[workloadmeta/kubeapiserver] Fix return type in Replace()

What does this PR do?
Fixes a bug in the Replace function of the reflector stores used by the kubeapiserver workloadmeta collector.
Unset events notified from Replace() had the expected workloadmeta kind but were always of type *workloadmeta.KubernetesPod instead of the expected type. This mismatch caused a panic when using workloadmeta filters like workloadmeta.IsNodeMetadata.
I added the no-changelog label because this bug doesn't affect any released version.
Describe how to test/QA your changes
The bug is difficult to trigger manually. The unit test added should cover it, but we can also check that the panic that we saw no longer appears:
panic: interface conversion: workloadmeta.Entity is *workloadmeta.KubernetesPod, not *workloadmeta.KubernetesMetadata

/merge

/merge


Fix problem with overreleased error

Make @autoreleasepool local, because it releases the error [NSError errorWithDomain:TAR_ERROR_DOMAIN code:TAR_ERROR_CODE_BAD_BLOCK userInfo:userInfo]; and app crashes

This looks like a legit fix, shouldn't this be merged? :-)

Thanks for bringing this to my attention again :)
I will look at it soon.


✨ Provide serialize context to response models

Hi @ilyatotl 👋
Thank you for your time and interest in helping us with your contribution. 🙇
Before we can continue reviewing this PR. Can you add some tests for the context you added and fix the lint? 🤓
Thanks again in advance!🚀

@ilyatotl  it seems someone else created a PR before you to add this feature #11634.
Since there is another PR older with the same contribution to add their feature, We are going to keep the oldest one and continue there. 🤓
Thank you again for your time and for taking care of this feature! 🚀


Purrr.jl or similar?

I've been using TidierDB to query databases recently (this was the last feature that I was missing in Julia compared to R), and things are going great. Thanks for all the work on Tidier!
Now I am writing some pipelines in Julia using REST APIs and I am dealing more with JSON files. I was thinking if there will be a "purrr"-analogue in Julia do manipulate lists (dicts? arrays? and the sort of nested objects). In R, I use functions like "list_flatten", "modify_if" all the time to get non-relational data from jsons and be able to write them in a relational database, but they are very slow compared to more "loop-based" manipulations. In Julia we won't have this problem.
I believe that many purrr functions will be """easy""" to translate to Julia without using macros, and I will be happy to contribute.
What do you think?

Thanks for using Tidier and for sharing thoughts on the functionality from the purrr package.
Although I have worked a bit with ragged JSON-like data, I always have to look up how to do things in purrr mainly because I never firmed up my mental model for the full purrr package. Because Julia has performant loops, this is less of an issue (as you point out), but the purrr syntax is still much more concise.
We don't have anyone working on a purrr-equivalent in Tidier.jl. I'd be open to adding such a repo to the Tidier family.
I would suggest starting a repo on your own GitHub account. When it has sufficient functionality that it's ready for a first release (doesn't have to be feature complete but should have enough functionality to be useful), let me know and I'll review. At that point, we would transfer the repo to TidierOrg (with you as a member) so that you could continue to lead work on it with others hopefully pitching in to help with documentation.
Keep us posted on whether you decide to take this on!

Hi, @kdpsingh !
Here is a first version o "JPurrr", the Julia version of purrr:
https://vituri.github.io/JPurrr.jl/dev/
I will write the examples later, but all functions are documented and trivial to use if you know the purrr package.
I implemented the majority of map_*, imap_* and pmap_* functions, along with modify and keep, but I will need some thinking regarding list_flatten. As you know, in R named lists and dataframes are all lists, so we can always iterate as usual. But what does it means in Julia to "flatten" a "list"? Maybe adding more methods to cover Julia dicts, arrays and dataframes in a nice way? I don't know yet.
There is a very nice cheat-sheet here that we can try to mimick:
https://github.com/rstudio/cheatsheets/blob/main/purrr.pdf

This is great! Amazing that you whipped this up so quickly. Haven't played with it yet but the docs look familiar. Excited to try it out soon when I get some downtime.
Can we modify the package name to fit the Tidier naming scheme? My goal has been to make the package names more obvious with respect to their purpose, and rooted in Julia.
How about one of these:

TidierMap (short and to the point. package also includes walk functions but Julia users will recognize map())

Or

TidierCollections (descriptive, but it's a bit long, which I don't like)

Thoughts?
Once we finalize name, will have you transfer the repo to TidierOrg and will be invite you to be a member of the org if that is something you're okay with.

TidierMap sound really good! According to the purrr docs, its goal is to provide "a complete and consistent set of tools for working with functions and vectors", and the word "map" gives this idea. There are even some "adverbs" which are very useful: https://purrr.tidyverse.org/reference/index.html#adverbs .
Here is the new doc:
https://vituri.github.io/TidierMap.jl/dev/
and the package address:
https://github.com/vituri/TidierMap.jl
I'll be happy to help work on this package within your org!

It's hard to get a single name that makes sense for all of purrr, since it's more "three packages in a trenchcoat" than it is one single package,  but I would suggest TidierFunctions.jl - the core functionality of purrr to me is the ability to work with functions: the various map and walk variants, quietly, safely, etc.
In the other TidierX names, the X is the thing we're working with (Plots, Data, etc.)

Thanks @rdboyes. They main issue I have with the name TidierFunctions is that in R, the two main functions (map and walk) are intended to operate on R's lists (using functions). I agree that safely/possibly do work on functions, but safely/possibly are essentially there to make it possible to apply map/walk without a single bad iteration breaking the whole thing. So I still view map/walk as the main functionality.
You could view map/walk as applying functions, but the lists are the star of the show.
Julia doesn't have the equivalent structure as a list -- named lists in R are kind of like nested named tuples in Julia. But non-named lists could be represented by types or arrays in Julia. So any "ragged array"-type object is the object of interest for this package (like JSON objects).
Agree that TidierMap doesn't quite do this justice but it's closer IMO than TidierFunctions.
Open to discussion and other suggestions. Happy to move this discussion to our Julia Slack channel.


Make sure that bots do not ever get information put into SceneView as…

… they can't use it. Also make sure to fully clear out SceneView in case it isn't disposed properly by the GC later.

Looks good. Thanks!

There may be a problem with this commit; when bots first appear in a region, it seems no updates are sent, the bot does not rez for the owner standing only 2m away. (If I roll the mouse wheel to zoom out/in, they appear.)

I have confirmed the problem and fixed it (plus some other additional IsBot checks) in a follow-up PR #197


Laravel migration can't change mysql auto increment primary key column type without resorting to raw sql query

Laravel Version
11.x-dev
PHP Version
8.2.8
Database Driver & Version
MySQL 8.0.33
Description
(only tested with -and probably only relevant for- mysql)
There's currently no syntax to change auto incrementing primary key column datatype (for example from unsigned medium integer to unsigned big integer).
Using $table->bigIncrements('id')->change(); results in error:

   Illuminate\Database\QueryException

  SQLSTATE[42000]: Syntax error or access violation: 1068 Multiple primary key defined (Connection: mysql, SQL: alter table `post` modify `id` bigint unsigned not null auto_increment primary key)

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:795
    791▕         // If an exception occurs when attempting to run a query, we'll format the error
    792▕         // message to include the bindings with SQL, which will make this exception a
    793▕         // lot more helpful to the developer instead of just the database's errors.
    794▕         catch (Exception $e) {
  ➜ 795▕             throw new QueryException(
    796▕                 $this->getName(), $query, $this->prepareBindings($bindings), $e
    797▕             );
    798▕         }
    799▕     }

Steps To Reproduce
example repository: https://github.com/nanaya/laravel/tree/increment-change
(use mysql database)
(literally just two new migration files as per https://github.com/nanaya/laravel/commit/863ae5e2b0df0b463bc0154c2570f97c002d1ca7 )

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


feature request: "skip-tables", "structure-tables", more options for db backups

drush offers many options which are not available to "terminus site backup" command.
Since commands like skip-tables and structure-tables are not supported, synching the pantheon-hosted database with non-pantheon environment (e.g. local) is exceedingly tedious.
if offering equivalent functionality is not possible, what about providing a pantheon-specific drush command to extend compatibility for sql-dump and/or sql-sync?

@aaronbauman There is some momentum coalescing around tools for syncing and piping databases in #235. I think your suggestions would contribute well to that discussion.


Add custom helpers update for cookiecutter package-template

helpers needs to be updated a custom way for the template:
https://github.com/astropy/package-template/blob/cookiecutter/cookiecutter.json#L27

I close this now as a won't fix.


[1.10] Fix build on Windows; Fix machines randomly rotating and losing state; Tiny optimization

I thought this makes sense as 3 separate pull requests but can't figure out how to do that.
See commit messages...

To create three pull requests you need three branches.
Create a new one like this: (assume origin is Mekanism github and ricket is yours)
git checkout -b <featureName1> origin/1.10
git cherry-pick <sha of commit you want>
git push ricket

Repeat for each of the three commits.

Whaat.. Those other commits I simply pushed to my fork, they definitely aren't supposed to be part of this pull request. I don't know how this works.
I see the message now, "Add more commits by pushing to the 1.10 branch on Ricket/Mekanism." That's definitely not what I intended.

The way GitHub works is that if you create a PR from a branch, any future commits to that branch get added to the PR until it is merged or closed. That's why it's best to have separate branches for each PR you wish to make.

Any chance you can retry the original pull request? Think it would be quite nice to have it as there are still a lot of sync/desync issues.
If you don't have time I can try and cherry pick them into a new pull request.


Add location.delta

Hey. It would be great to know the direction of the transition thus we can set a different animation (with react-router) for  forward/back actions. I first though that I can depend on action but browser back/forward buttons are always POP. I would appreciate if we can add this feature, thx?

I think I'd prefer to have something a bit more general, like location.delta that would give us the n the user supplied to go(n). When going back, location.delta would be -1. Likewise, when going forward location.delta would be 1.

Even better :).

This would be a great improvement! I'm also relying on location.POP to detect back transitions and perform a different animation, but it is not reliable because of the (less frequent) forward transitions. Is this planned?

I think this would make a great addition to the library, but haven't had time to get to it. I would love it if someone else could step up and take a shot at this. It's a feature that a lot of people seem to want.

This is superseded by #334. If we can figure out how to provide location.index, users can derive the delta for themselves.


Document I can see in remote couchdb, does not show up in alldocs after syncing

Just starting to debug this issue that has happened sporadically to me in an app I'm working on using this plugin.
I have a basic app using this with a few documents, that syncs to a remote CouchDB instance.
Right now I have a document with ID '2016-11-2' that doesn't show up in alldocs on PouchDB, even though it's there on the remote server. PouchDB reports the sync was successful, and other types of changes are still flowing correctly back and forth.
If I hit the CouchDB changes API manually I get an entry like this for the document '2016-11-2', which appears like there is a conflict:
{
  "seq":41,
  "id":"2016-11-2",
  "changes":[
    {"rev":"1-1b2b09d47ea243554a79d7b3af29a811"},
    {"rev":"7-ce1394ede863930bd45746f8bd1f933e"}
  ]
}

Using couch API, or futon, to fetch that document, selects rev 1-1b2b09d47ea243554a79d7b3af29a811 as the winner and I get a response like this:
{
  "_id":"2016-11-2",
  "_rev":"1-1b2b09d47ea243554a79d7b3af29a811",
  "foo": "bar"
 }

If I specifically fetch the other rev, it comes up deleted:
{
  "_id":"2016-11-2",
  "_rev":"7-ce1394ede863930bd45746f8bd1f933e",
  "_deleted":true
 }

Dumping the keys in AsyncStorage got me this,  which appears like something has caused PouchDB to select the other, deleted revision, as the winner:

Storage Key
Value

my-app!ÿby-sequenceÿ0000000000000017
{"foo":"bar","_id":"2016-11-2","_rev":"1-1b2b09d47ea243554a79d7b3af29a811"}'

my-app!ÿby-sequenceÿ0000000000000018
{"_id":"2016-11-2","_rev":"7-ce1394ede863930bd45746f8bd1f933e"}

my-app!ÿdocument-storeÿ2016-11-2
{"id":"2016-11-2","rev":"7-ce1394ede863930bd45746f8bd1f933e" ...}

So then later when I do the all docs, it considers the doc deleted and won't include it.
Is this a bug in the AsyncStorage driver or am I doing something wrong?

Could you try to reproduce it locally (in terminal, not on a device/emulator) with react-native-mock, please?
{
  "private": true,
  "scripts": {
    "start": "babel-node index"
  },
  "dependencies": {
    "babel-cli": "^6.18.0",
    "babel-preset-react-native": "^1.9.0",
    "pouchdb-react-native": "^6.1.20",
    "react": "^15.3.2",
    "react-native": "^0.36.1",
    "react-native-mock": "^0.2.7"
  }
}

// index.js
'use strict';

require('babel-register')({
  presets: ['react-native'],
  ignore: path => {
    return path.indexOf('pouchdb') === -1 && path !== (__dirname + '/app.js');
  }
});

require('./app');

// app.js
import 'babel-polyfill';
import 'react-native-mock/mock';
import PouchDB from 'pouchdb-react-native';

(async () => {
  const db = new PouchDB('mydb');
  const doc = await db.put({ _id: '1', foo: 'bar' });
  console.log(await db.get('1'));
})();

Hmm yeah thanks for the framework for testing. I'll try and get a better repro. I'm still not 100% sure how I've caused it even.

I was able to reproduce a similar type of error in the playground, see my fork
https://github.com/JDeuce/pouchdb-react-native-playground/commit/28a82ef96c9bc9d07dc969d8eb51bc683ad7bc3b
I basically just introduce a simple conflict by inserting a document with the same ID into 2 local databases,
and then replicate it to a local PouchDB, and to a remote CouchDB.
My expectation is that after syncing in the same manner,  the local PouchDB destination would match the remote CouchDB destination. However, in some cases they are selecting different winners.
Here is the run output from 2 separate runs:
https://gist.github.com/JDeuce/40e590604af9637dbf00de764094262c
In the first run, they select the same winning conflict, but in the second run they select different ones.
Run it like
npm start -- http://myserver.com:5984/test

Could you also add assertions with any assertion library like assert, please? Sorry for not clarifying it in the first message

Here's another test case with some assertions. I was also able to replicate the issue this time with local databases only.
https://github.com/JDeuce/pouchdb-react-native-playground/commit/7f341d1e27784b7f56d2badc8d20464351de9929
It seems like the winning rev is being determined by the order that replication happens, when it should happen deterministically based on the rev ids of the document.

Here is the same test with the basic browser version of PouchDB in jsfiddle:
https://jsfiddle.net/s3aoc8a3/
The rev ids are consistent every time.
Feels like this is a definite issue in pouchdb-react-native.
Let me know thoughts.

Send a pull request #41 Hopefully it doesn't break anything beyond what is under the tests now
@stockulus Have you had a chance to look at the issue?


FEATURE: 添加图标选择器组件

Version
Vben Admin V5
Description
添加图标选择器组件
Proposed Solution
添加图标选择器组件
Alternatives Considered
No response
Additional Context
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.

mark


SQ610RF - Humidity extraction

Hi
Firstly, thank you for pulling together this HA integration, it has massively improved the usage of the setup.
I have temperature extracting well from my SQ610RF but cant seem to find humidity - see extract from my HA states below:

I have pulled down the code and installed the integration manually today and wondered what i was doing wrong as i have seen others are able to extract it?
Thank you in advance.

I have the same problem too.

Hi,
I had the same issue to get humidity from my SQ610 with Home assistant (docker image)
It turns out that the model name returned by the gateway is SQ610(WB) instead of SQ610 (or SQ610RF).
In order to get the humidity, I had to change the line 504 of the file pyit600/gateway.py from that :
       if model == "SQ610" or model == "SQ610RF":

to that :
       if model is not None and "SQ610" in model:

Hope it will help
Michael

Hi Michael,
i am using a SQ610RF Model on Homeassistant. Can you please let me know, where is the pyit600/gateway.py file located in HA? I am using a VM edition...
Many Thanks....

Hi @yah996
I'm using home assistant in Docker so I don't know if the following information will be applicable for a VM edition !
The file gateway.py is located here :
/usr/local/lib/python3.9/site-packages/pyit600/gateway.py

thanks, I found it.

Hi @jvitkauskas
Do you have time to include in the next release the solution proposed by @mdelalan ?
Thanks

thanks, I found it.

Can you tell me how did you find it? I am in the same situation as you and do not seem to be able to find the file. Thanks.

are you using the terminal add on in HA?

Yes, I have it installed.

ok, try this in the terminal add-on
shell to the homeassistant server, name='yourhostname'.
Use this commands:
docker exec -it $(docker ps -f name=homeassistant -q) bash
bash-5.1# find / -name gateway.py
/usr/local/lib/python3.10/site-packages/pyit600/gateway.py
Use vi or so to modify the gateway.py, this should be work.

Hi @jvitkauskas Do you have time to include in the next release the solution proposed by @mdelalan ? Thanks

Hello,
I've noticed that the temperature returned by the gateway is not always the same as the one displayed on the SQ610 :
It seems that the value is not updated in realtime !
I played a little bit with the Android app Smart Home in Wifi local mode and I found the command that force the gateway to update the temperature/humidity with the one displayed on the SQ610 :
await self._make_encrypted_request(                                                                                "write",                                                                                {                                                                                "requestAttr": "write",                                                                                "id": [{"data": device["data"],"sIT600I":{"SetCommand_d":"42 37 34 00 00 00 00"}} for device in devices]                                                }                                                                                )
Maybe this can also be added in the next release ??

Thank you yah966, but my HA is installed in a VM on a Synology NAS. And I am not an IT guy, I did it following some YouTube tutorials.

Hi @jvitkauskas
I've updated my salus integration with 0.51 but from what I see there still no humidity. There are no entities for that ...

Thanks

Thank you for the update, humidity appeared in states (developer tools - states - attributes).
@DragosFlorea From there you can add it as an attribute in any of your cards in YAML.

Hello,
I use HASO in VM and I am not able to see the SQ610RF device. I see all other devices I have in the IT600 system but not the SQ610RF Thermostat.
Also I don't have this file   /usr/local/lib/python3.9/site-packages/pyit600/gateway.py
Any suggestion from you please? Thank you !

Hi,
The humidity seems to be available, but cannot be tracked. Like we can see the temperature over a certain period of time, can we also have the humidity being tracked?

hello, would like to track humidity as was mentioned in last comment - is there some way to it? thanks


Connect OSMNX to a local OSM databasae

Is your feature proposal related to a problem?
Maybe.
A clear and concise description of what the problem is.
I need to connect OSMNX to my local OpenStreetMap server installed by overv/openstreetmap-tile-server Docker.
I have a closed network running secret stuff and need to cut off any contact to external world.
So I have my own OSM server and data, updated daily.
Describe the solution you'd like to propose
I'd like to use OSMNX in this closed environment, so I need to know if there is any configurations or additional tools I must use.
A clear and concise description of what you want to change in OSMnx and how you plan to implement it.
I don't know what to change.
Describe alternatives you've considered
There are no alternatives.
A clear and concise description of any alternative solutions or features you've considered.
I'm planning to:

Install a local Overpass API ( https://hub.docker.com/r/wiktorn/overpass-api )
Somehow (plase tell me how) edit https://github.com/gboeing/osmnx/blob/main/osmnx/settings.py to point to my local API in overpass_endpoint parameter.

I don't know how to do the step 2 because I'm using an already cooked OSMNX Docker container ( exposing this file, maybe? ).
Additional context
Seems doing that I'll have another copy of same data. Unfortunatly Overpass don't use PBF or PostGIS... ( AFAIK )

Somehow (plase tell me how) edit https://github.com/gboeing/osmnx/blob/main/osmnx/settings.py to point to my local API in overpass_endpoint parameter.

Hi. Thanks for using OSMnx. Please ask this usage question on StackOverflow. We reserve this issue tracker for bugs and feature development. Also, see the documentation for details on using the settings module and configuring the endpoint.


New version: LX-Systems.WinMute version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Automatic Validation ended with:

No errors to post.

(Automated response - build 813.)

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Programs\LX-Systems\WinMute\WinMute.exe returned exit code: -1073741515

(Automated response - build 821.)

Manual Validation ended with:


Command line option for search and replace

It is nice that you can open Visual Studio Code from command line and provide the folder to open as well (code folder). With an option code --diff file1 file2 you can right away get the comparison view.
I would like to be able to execute code folder --search "some search string" and code folder search "old string" --replace "new string" to open Visual Studio Code in Search tab with appropriate fields filled in and maybe additional parameter --execute to perform the search and/or replace immediatly.

I don't think this is likely to happen. Better to use a CLI tool like ripgrep directly.


password still remembered while "remember password" was turned off

password still remembered while "remember password" was turned off

Looks like a duplicate of #66, will track it there since it was first.  Thanks for reporting.