[Feature request] resizable table

Just wanted to make resizable columns in a table so i built this:
https://jsfiddle.net/xodtjy2v/2/
it's not that advance, it's simple lightweight and don't depend on any javascript at all. doe it's not as good as any javascript solution that can resize a hole column at any y-axis.
doing this css trick on x-axis was really buggy and not even worth implementing without a js solution.
either way. i thought it would be worth implementing this into bootstrap core:
.table.table-resize-y thead th {
  overflow-y: auto;
  resize: horizontal;
}

 /* optional */
.table.table-resize-y ::-webkit-resizer {
    background-color: transparent;
}

huge benefit with little css and no extra DOM elements 😉

It doesn't seem to work with Firefox at all (in both ESR and Nightly).
Can you confirm this, and maybe suggest a solution? This seems interesting, but can't be added without working on supported browsers.

The other only option then is to use a wrapper element
with that solution you can decide for yourself which columns should be resizable and which shouldn't. Also notice it works much better to resize on the y-axis too.
new demo: https://jsfiddle.net/ywb9e4s7/
the other only thing that is weird is when you have a resize: both the other cells don't follow.

(trying to add min-width: 100%; had a super weird behavior)
it could be such a generic class that it don't only have to apply to table's resize, resize-x, resize-y could work for any element too

Closing as a won't fix given the quirks. Super cool idea though!

I'm fine with closing this, it's a bit quirky. too bad i could not modify the resize handle to stretch or move it in some ways. mostly just limited to changing background


Question about the wikitext2 data loader

Thanks for your nice work. When running the sample script, I'm getting the following error message regarding the wikitext2 loader. Would you kindly check it?
File "/ssd2/bkkim/FLAP/lib/data.py", line 81, in get_wikitext2
    traindata = load_dataset('text', data_files='datasets/wikitext/wiki.train.raw', split="train")
FileNotFoundError: Unable to find '/ssd2/bkkim/FLAP/datasets/wikitext/wiki.train.raw'

By the way, I found a workaround for the issue by using the comment lines in the script. Is this an appropriate solution?
# traindata = load_dataset('wikitext', 'wikitext-2-raw-v1', split='train')
# testdata = load_dataset('wikitext', 'wikitext-2-raw-v1', split='test')

Thank you for your time.

@bokyeong1015  Due to my server's inability to directly access HuggingFace for downloading datasets, I opted for local loading during my experimentation. You can replace it with the two lines of code commented in the script, which should resolve this issue.


Races for VRS Global Endurance Series are not available

Thanks for letting me know. I've run a fresh scrape, hope that's sorted everything 👍  (you may need to force cache refresh)

No @tmoitie , that doesn't run, I don't find the serie in the race week list after refreshing the application cache !

Are you sure? The next scheduled race week for Global Endurance is the 6th so you might need to adjust the date to see the line item


Run on a Bare-Metal Platform (static_library)

Hi,
I am trying to build and run IREE "Static Library" sample without building IREE from scratch. I've noticed that the IREE "Static Library" sample has its own CMake so I've used this cmake (cmake , cmake --build) to create the executables (static_library_demo_test, static_library_demo_c_test). The problem is that after I've used this cmake file no executables were created.

I think you want to look into https://iree-org.github.io/iree/deployment-configurations/bare-metal/. Further, https://github.com/iml130/iree-bare-metal-arm is doing what you want: We use a snapshot and only cross-compile the runtime for the bare-metal target.

I think you want to look into https://iree-org.github.io/iree/deployment-configurations/bare-metal/ and https://iree-org.github.io/iree/building-from-source/riscv/#bare-metal-execution-example (for RSIC-V). Further, https://github.com/iml130/iree-bare-metal-arm is doing what you want: We use a snapshot and only cross-compile the runtime for the bare-metal target.

Yes I've seen this links before. They are not related with my issue. Ι only need to a make a NN executable for x86. The above instructions build IREE from scratch and then generate static_library_demo executable.

I only wont to generate static_library_demo executable and not build IREE (takes time). So I've used this CMake without any success. Can I use the above cmake to create static_library_demo? And if I can what would be the steps to do it ?

You might also want to take a look at https://github.com/iree-org/iree/tree/main/experimental/web/sample_static, particularly the CMakeLists.txt and build_sample.sh files. That uses a prebuilt iree-compile in the script and then links the output of that in via the CMake file.

You might also want to take a look at https://github.com/iree-org/iree/tree/main/experimental/web/sample_static, particularly the CMakeLists.txt and build_sample.sh files. That uses a prebuilt iree-compile in the script and then links the output of that in via the CMake file.

So I can't use the cmake that I mentioned to generate an executable without compiling IREE from scratch?

You can start with that file, but you'll need to make edits if you want to avoid compiling IREE from scratch. The checks at the start depend on building the iree-compile compiler binary and a few CMake options being set: https://github.com/iree-org/iree/blob/99caaec7072c02997434e711edd6e722dfca8ff9/samples/static_library/CMakeLists.txt#L7-L14
Further down there are more checks for optional features in the build: https://github.com/iree-org/iree/blob/99caaec7072c02997434e711edd6e722dfca8ff9/samples/static_library/CMakeLists.txt#L97-L99
Some debugging with CMake's message() and configuring should help you see what parts of the CMake file are running and which values they use.

You might also want to take a look at https://github.com/iree-org/iree/tree/main/experimental/web/sample_static, particularly the CMakeLists.txt and build_sample.sh files. That uses a prebuilt iree-compile in the script and then links the output of that in via the CMake file.

So I can't use the cmake that I mentioned to generate an executable without compiling IREE from scratch?

You can start with that file, but you'll need to make edits if you want to avoid compiling IREE from scratch. The checks at the start depend on building the iree-compile compiler binary and a few CMake options being set:
https://github.com/iree-org/iree/blob/99caaec7072c02997434e711edd6e722dfca8ff9/samples/static_library/CMakeLists.txt#L7-L14
Further down there are more checks for optional features in the build:
https://github.com/iree-org/iree/blob/99caaec7072c02997434e711edd6e722dfca8ff9/samples/static_library/CMakeLists.txt#L97-L99
Some debugging with CMake's message() and configuring should help you see what parts of the CMake file are running and which values they use.

Thank you. I am closing this issue.


init() of blanked imports are ignored

Hi, I have a project that contains multiple imported packages, some of them are blanked because I'm using only the init() function, the problem is that in go-callvis they are not displayed. I use
go-callvis -group type,pkg ../myproject |dot -Tpng -o out.png
I'm missing something?

Hey, there is a -focus flag which by default uses main package to focus only on calls going from or to that package. That's why you don't see those init() functions from other packages.
You could try focusing on the package that has the init() you are interested it. However you could also checkout the recent enhancement for go-callvis, currently opened in PR.
The call graph output is now served via HTTP server in SVG format and you can click on packages to change focus.


Add AU region

What are you changing in this pull request and why?
Adding new MT region, this updates NAT IPs for external facing documentation.

Checklist

[ ] Review the Content style guide and About versioning so my content adheres to these guidelines.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding new pages (delete if not applicable):

[ ] Add page to website/sidebars.js
[ ] Provide a unique filename for the new page

Removing or renaming existing pages (delete if not applicable):

[ ] Remove page from website/sidebars.js
[ ] Add an entry _redirects
[ ] Ran link testing to update the links that point to the deleted page

@dtardoin This looks good. When does it ship?

@runleonarun the instance is up now (so these IPs won't change), but it's not live until Feb 7th.

This is for any customer using the AU MT instance (au.dbt.com)

ah gotcha! i think i didn't ask correctly. so in terms of region, should it be APAC region? or would customers using the AU MT instance only be Australian? I'm just double checking whether the region is AU or APAC


golang/tools/cmd/vet vs golang/go/tree/master/src/cmd/vet, what is the correct one to use?

Hi Brad,
commit 7454f53604b9953b5b7a3897c6b855957f911e66 in its message says:
No more changes to golang.org/x/tools please. This should not have
happened (and letting it happen was partly my fault).
What does it mean? github.com/golang/tools is becoming read only? Or is just no more updates to golang.org/x/tools packages which is, well, exactly the same.
Just wondering, which repository for vet tool is recommend/correct to use?
Jan

@bradfitz

This isn't a bug report, so I'm closing it. (Unlike some Github projects, we don't use our bug tracker for Q&A)
Please bring questions to the golang-nuts@ mailing list or the Go forum: https://forum.golangbridge.org/

The message means that new changes to cmd/vet should be made to the one in
master go repository. It doesn't mean the whole x/tools repository will be
readonly.


Fuse how to handle concurrent open() create()

Is your feature request related to a problem? Please describe.
A file may be reading or writing with multiple open(), create() concurrently.
Alluxio cannot support reading while writing.
Alluxio cannot support double writing.
Deal with those cases, error out if needed and gives clear information.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Urgency
Explain why the feature is important
Additional context
Add any other context or screenshots about the feature request here.

https://github.com/Alluxio/alluxio/pull/14846


fix: support identity characters

handle the various characters found in a user identity that does not
work in a GCE instance name
fixes #53

for_each =<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
  + member1 = {
      +<EMAIL_ADDRESS>= {
          + condition = []
          + etag      = (known after apply)
          + id        = (known after apply)
          + member    = "user:hime@examplecom"
          + project   = "nkb-bootstrap"
          + role      = "roles/notebooks.viewer"
        }
      +<EMAIL_ADDRESS>= {
          + condition = []
          + etag      = (known after apply)
          + id        = (known after apply)
          + member    = "user:test2@examplecom"
          + project   = "nkb-bootstrap"
          + role      = "roles/notebooks.viewer"
        }
      +<EMAIL_ADDRESS>= {
          + condition = []
          + etag      = (known after apply)
          + id        = (known after apply)


Use python stdlib logging

Proposal: Use python stdlib logging
Author: Adrian Likins<EMAIL_ADDRESS>Date: 2016-10-10

Status: New
Proposal type: core design
Targeted Release: 2.3
Associated PR: <link to GH PR in ansible/proposals if PR was submitted>
Estimated time to implement: working prototype exists, a few weeks of iterations to square it up

Motivation
Python stdlib logging is a well known and common approach to logging, with a flexible and extensible configuration, wide 3rd party vendor support, and offers integration with common python modules.
Problems
What problems exist that this proposal will solve?

Current logging support is very limited
Current logging approach is unusual
Current logging approach makes it difficult to take advantage of 3rd party tools that support stdlib logging
Current approach ('display') has grown organically and has an unusual API
Current logging approach makes it very difficult to support structured logging
Current logging approach does not offer find grained configuration (for ex, enabling debug level output for only 'ansible.parsing.vault')
Current logging approach only offers a limited amount of context associated with records and some fields/info are wrong or misleading:

the 'pid' used in the logger name is only the pid of the main process, and does not reflect what process made the original call.
the logger name is non-standard
the ''pathname', 'filename', 'module', funcName' and 'lineno' log attributes are wrong and only point to display.py, not the originating call

Solution proposal

Use 'logging' module throughout the code base, replacing use of 'display.debug', 'display.warn' etc

eliminates the need to format messages in calling code

defers 'str/repr' of objects until log record is formatted. For example, a log.debug('template=%s', template) would never need to str/repr(template) if the handlers log level is 'INFO'

Use standard logging.getLogger(name) logger setup

where useful, also setup per class loggers
This allows fine grained control of logging, on a per module or per class level

Make use of 'log.exception()' for logging exception info from except clauses (including tracebacks)

eliminates 'hidden' exceptions that are caught and ignored

Use a config file based logging setup (either yaml->dictConfig or 'fileConfig')
provide log handlers for files, syslog, journald, console

optional: log handlers for windows logging

provide a log handler for supporting concurrent logging from subprocess
set up and use a log handler attached to 'ansible.*' logger namespace

allow attaching other specific loggers to our handlers (for ex, 'urllib3', 'requests')
allow attaching the root logger to our handler and enabling propagation to see all loggers process wide

provide log formatters for current display output style, a basic line oriented format for log files, a detailed 'debug' formatter for troubleshooting and debugging.

optional: formatters for JSON output for use with ELK or other log indexing tools

provide log filters and adapters for adding contextual info to logging events (Very useful for debugging and troubleshooting by devs, users and support).

some examples of info that could be added to records:

sys.argv
curent user
local hostname
remote_addr/hostname (for logging task/connection info)
remote_user
playbook/play/block/task context

provide log filters for handling 'unsafe' text (censoring passwds etc)
optional: provide a callback plugin that uses stdlib logging (either modifying existing 'default' or providing a new one_/
prototype branch: https://github.com/alikins/ansible/tree/display_logging_notes
'logging' callback prototype branch: https://github.com/alikins/ansible/tree/stdlog_callback

Example logging use
import logging

log = logging.getLogger(__name__)
<...>
        log.warning("provided hosts list is empty, only localhost is available")

Output with added context info:
2016-10-10 13:21:08,278 user=adrian cmd_name=ansible-playbook <@> [ansible.cli.playbook WARNING] (pid=31566) tid=139879600539392:MainThread run:146 - provided hosts list is empty, only localhost is available

log.exception() example on a playbook yaml parsing error
2016-10-10 13:28:26,343 user=adrian cmd_name=ansible-playbook <@> [ansible CRITICAL] (pid=32217) tid=139760017659648:MainThread <module>:130 - Syntax Error while loading YAML.

The error appears to have been in '/home/adrian/src/ansible/ping.yml': line 12, column 1, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

#- hosts: me,localhost,swoop
^ here
Traceback (most recent call last):
  File "/home/adrian/src/ansible/bin/ansible-playbook", line 121, in <module>
    exit_code = cli.run()
  File "/home/adrian/src/ansible/lib/ansible/cli/playbook.py", line 161, in run
    results = pbex.run()
  File "/home/adrian/src/ansible/lib/ansible/executor/playbook_executor.py", line 85, in run
    pb = Playbook.load(playbook_path, variable_manager=self._variable_manager, loader=self._loader)
  File "/home/adrian/src/ansible/lib/ansible/playbook/__init__.py", line 53, in load
    pb._load_playbook_data(file_name=file_name, variable_manager=variable_manager)
  File "/home/adrian/src/ansible/lib/ansible/playbook/__init__.py", line 76, in _load_playbook_data
    ds = self._loader.load_from_file(os.path.basename(file_name))
  File "/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py", line 124, in load_from_file
    parsed_data = self.load(data=file_data, file_name=file_name, show_content=show_content)
  File "/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py", line 104, in load
    self._handle_error(yaml_exc, file_name, show_content)
  File "/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py", line 205, in _handle_error
    raise AnsibleParserError(YAML_SYNTAX_ERROR, obj=err_obj, show_content=show_content)
AnsibleParserError: Syntax Error while loading YAML.

The error appears to have been in '/home/adrian/src/ansible/ping.yml': line 12, column 1, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

#- hosts: me,localhost,swoop
^ here

Same example using a 3rd party json logging formatter (logomatic.JsonFormatter in this case)
{"asctime": "2016-10-10T13:32:27Z+0000", "name": "ansible", "processName": "MainProcess", "filename": "ansible-playbook", "funcName": "<module>", "levelname": "CRITICAL", "lineno": 130, "module": "ansible-playbook", "threadName": "MainThread", "message": "Syntax Error while loading YAML.\n\n\nThe error appears to have been in '/home/adrian/src/ansible/ping.yml': line 12, column 1, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n#- hosts: me,localhost,swoop\n^ here\n", "exc_info": "Traceback (most recent call last):\n  File \"/home/adrian/src/ansible/bin/ansible-playbook\", line 121, in <module>\n    exit_code = cli.run()\n  File \"/home/adrian/src/ansible/lib/ansible/cli/playbook.py\", line 161, in run\n    results = pbex.run()\n  File \"/home/adrian/src/ansible/lib/ansible/executor/playbook_executor.py\", line 85, in run\n    pb = Playbook.load(playbook_path, variable_manager=self._variable_manager, loader=self._loader)\n  File \"/home/adrian/src/ansible/lib/ansible/playbook/__init__.py\", line 53, in load\n    pb._load_playbook_data(file_name=file_name, variable_manager=variable_manager)\n  File \"/home/adrian/src/ansible/lib/ansible/playbook/__init__.py\", line 76, in _load_playbook_data\n    ds = self._loader.load_from_file(os.path.basename(file_name))\n  File \"/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py\", line 124, in load_from_file\n    parsed_data = self.load(data=file_data, file_name=file_name, show_content=show_content)\n  File \"/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py\", line 104, in load\n    self._handle_error(yaml_exc, file_name, show_content)\n  File \"/home/adrian/src/ansible/lib/ansible/parsing/dataloader.py\", line 205, in _handle_error\n    raise AnsibleParserError(YAML_SYNTAX_ERROR, obj=err_obj, show_content=show_content)\nAnsibleParserError: Syntax Error while loading YAML.\n\n\nThe error appears to have been in '/home/adrian/src/ansible/ping.yml': line 12, column 1, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n#- hosts: me,localhost,swoop\n^ here\n", "cmd_name": "ansible-playbook", "remote_user": "", "remote_addr": "", "cmd_line": "/home/adrian/src/ansible/bin/ansible-playbook -i hosts ping.yml", "user": "adrian", "timestamp": "2016-10-10T13:32:27Z+0000"}

An example of a debug log with contextual info about remote hosts being targetted:
2016-10-10 13:44:11,142 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: ansible.cfg set ssh_args: (-C)(-o)(ControlMaster=auto)(-o)(ControlPersist=60s)
2016-10-10 13:44:11,142 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: ansible_password/ansible_ssh_pass not set: (-o)(KbdInteractiveAuthentication=no)(-o)(PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey)(-o)(PasswordAuthentication=no)
2016-10-10 13:44:11,143 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: ANSIBLE_REMOTE_USER/remote_user/ansible_user/user/-u set: (-o)(User=vagrant)
2016-10-10 13:44:11,143 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: ANSIBLE_TIMEOUT/timeout set: (-o)(ConnectTimeout=10)
2016-10-10 13:44:11,143 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: PlayContext set ssh_common_args: ()
2016-10-10 13:44:11,143 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: PlayContext set ssh_extra_args: ()
2016-10-10 13:44:11,143 user=adrian cmd_name=ansible-playbook<EMAIL_ADDRESS>[ansible.plugins.connection DEBUG] (pid=1505) _add_args:128 - SSH: found only ControlPersist; added ControlPath: (-o)(ControlPath=/home/adrian/.ansible/cp/ansible-ssh-%h-%p-%r)

Dependencies (optional)
For basic use, there are no external dependencies. The 'logging' module exists in all python versions supported. For concurrent logging to a single file, the 'multiprocess' module is required.
Additional log handlers, formatters, and filters may have additional deps. For example, log handlers for logging to 3rd party cloud based logging services may need 'requests' or a app specific dep.
If a yaml based logging config supports is needed on py2.6, a backport of 2.7's 'logging.dictConfig' may be needed.
Testing (optional)
Logging focused testing is recommended.
Additionally it will be useful to include tests that compare old style display based logging output to 'logging' output.
Testing of an assortment of logging configurations is recommended (custom formats, custom handlers, logging to syslog/journal/etc).  Testing of log file handling and rotation. Testing of multiprocess concurrency correctness and stability.
Documentation (optional)
This will need user and developer focused documentation.
Users will have significant logging options available. Examples configurations of some common cases (file logging, syslog, journald, remote logging, ELK stack, etc).
Developers will need documentation on proper use of 'logging' module (although the module is well documented elsewhere). This will include any ansible specific changes (our handlers, formatters, filters, and adapters will need documentation on their use).
Anything else?
Related:

Example of adding stdlib file based logging to module_common: https://github.com/alikins/ansible/commits/module_common_logging  (note this only logs info to a file on the remote node. It does not currently foward logs back to the manager).
Example of adding file based stdlib based logging to module_utils/basic.py: https://github.com/alikins/ansible/commits/module_logging (note this only logs info to a file on the remote node. It does not currently foward logs back to the manager).

What is this status of this proposal? I have third party Python modules using the logger class that I am trying to wrap Ansible around and have no idea how to set up or configure so that Ansible will get those logger messages written somewhere.
The proposal opening says there is a "working prototype", is there something I can use to help validate third party code using logger class can be configured and work inside and Ansible playbook?

I am wondering how many years do we need until Ansible logging would be done right. As of today, even the latest 2.13.1 does enforce wrapping of warnings, which hinders console display and makes impossible to process the output. Effectively making it worse for both humans and machines.

@cidrblock Maybe we can push a little bit more to sort this out? That bug alone hinders usability our most development tools

@ssbarnea  'done right' has many different opinions, this proposal has been discussed many times always ending in an impasse.
While the core team now has more consensus on the approach going forward, it still needs to be backwards compatible with the current 'display', otherwise all 3rd party plugins using it would cease to function.
In any case, if you want to bring it up for further discussion, we have monthly IRC meetings exactly for this purpose. https://github.com/ansible/community/issues?q=is%3Aissue+is%3Aopen+label%3Acore+label%3Ameeting_agenda


Is it compatible with different driver versions and cuda versions

What happened:All previous Gpus of the cluster were 515 version of the driver and cuda11.7.Rencently I add a machine with L20(only support driver 535 at least and cuda12,  then I ran into a problem that the gpus were not recognized correctly:

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

The output of nvidia-smi -a on your host
Your docker or containerd configuration file (e.g: /etc/docker/daemon.json)
The hami-device-plugin container logs
The hami-scheduler container logs
The kubelet logs on the node (e.g: sudo journalctl -r -u kubelet)
Any relevant kernel output lines from dmesg

Environment:

HAMi version:
nvidia driver or other AI device driver version:
Docker version from docker version
Docker command, image and tag used
Kernel version from uname -a
Others:

can you re-submit the task with env 'CUDA_DISABLE_CONTROL'=true , can see if it reproduces this error?


The white space

The white space between the elements inside the container need to be wider and there's much white space in the left and the right side so you can use them to making the balance

Can you attach a screenshot please, to show what you mean?

Closing this for now, please reopen with more explanation.


Add documentation about commit limits

(Only 500 entities per commit.)

I have been thinking about this, and I think we should automatically batch for Insert requests. Insert does not make any promises about transactional logic, and if the user cared strongly, they would be using a transaction.

I'd far rather be consistent, to be honest. It gets complicated really quickly in terms of retries and the like. It also means the library doesn't need to be changed if Datastore's limits change, for example.

(Have asked internally for comment from the Datastore team.)

Confirmed that this is really an application concern. (I know that feels like a cop-out, but the number of potential options required and inconsistency between inside/outside transactions is enough.)


Remove unused staticfile dependency

Should help address https://github.com/rust-lang/docs.rs/issues/656 Nope.
r? @pietroalbini

We might also want to add a cache header to the static files -- to avoid requests from CloudFront -- but I suspect that can be handled later. (Previously, the staticfile crate did set ETag and Last-Modified on our end).
https://github.com/iron/staticfile/blob/0.4.0/src/static_handler.rs#L212-L214
Otherwise, this looks good to me, though I don't know what our process these days is for merging -- r=me :)

I think we do that already, actually: https://github.com/rust-lang/docs.rs/blob/master/src/web/file.rs#L25

I don't know what our process these days is for merging -- r=me :)

We don't really have a process ... I don't have admin access to the repo so I can't turn on protected branches.

Please also update the Dockerfile to not touch public_html.

Looks like tests are failing, but feel free to merge whenever.

After testing locally, it looks like this breaks something with the favicon. I don't feel comfortable merging this until we figure out what's going on there.


Add csrf option to Dream.form and Dream.multipart

Part of #11

Thanks, this looks good, and I'm glad the diff is minimal (no extra whitespace changes, etc.)!
One question: is this motivated by need in your case (i.e. you wanted to use the form helpers without the CSRF check)?

Hi! Yes it is, i am playing with Dream and have an endpoint that expects a Content-Type:  application/x-www-form-urlencoded, but couldn't consume it with curl neither test it with Dream.test, so i searched a way of disable the csrf and found the issue

Great, thanks for telling. Helps actually motivate it (as I was only suspecting it would be useful, hadn't actually observed a use!)


[FEAT] Pagination

Virtual/Infinite scrolling is awesome but sometimes you need to give perspective to where you are at in the actual page structure so a user can leave and come back and easily find the item they were viewing.
I've implemented something like this on https://github.com/Swimlane/ng-table but I'd love to work with you to integrate this into your grid to make the ultimate grid! :)

Interesting. Am in pub at the mo. Will check out ng-table over the weekend.
Where you based??
On 6 Mar 2015 18:39, "Austin"<EMAIL_ADDRESS>wrote:

Virtual/Infinite scrolling is awesome but sometimes you need to give
perspective to where you are at in the actual page structure so a user can
leave and come back and easily find the item they were viewing.
I've implemented something like this on
https://github.com/Swimlane/ng-table but I'd love to work with you to
integrate this into your grid to make the ultimate grid! :)
—
Reply to this email directly or view it on GitHub
https://github.com/ceolter/angular-grid/issues/5.

The demo doesn't work that great but you can review the code.  I'm working on getting it better but i use it internally fine.
It has jqueryui dependency i'd love to remove and has some missing features and it can be slowish sometimes hints why interested in your project.
The really cool thing about it is its directive driven!
I'm based in South Florida US

Ah Florida, I am 5 or 6 hours ahead of you in London!!
On 6 Mar 2015 18:54, "Austin"<EMAIL_ADDRESS>wrote:

The demo doesn't work that great but you can review the code. I'm working
on getting it better but i use it internally fine.
It has jqueryui dependency i'd love to remove and has some missing
features and it can be slowish sometimes hints why interested in your
project.
The really cool thing about it is its directive driven!
I'm based in South Florida US
—
Reply to this email directly or view it on GitHub
https://github.com/ceolter/angular-grid/issues/5#issuecomment-77614112.

Hi, ceolter ! I have checked your “Angular Grid”， It is greate! but the example does not have "Client side Pagination"??   which I think is  the basic functions of a grid.  I hope that I can found this function soon.   Now I'm trying it...

Hi Ppneo, this feature isn't in yet, on my to do list. However the grid can handle very large data sets (tested up to 100k rows), why do you need pagination, why not load everything into the GUI and allow user to filter?

@ceolter I'm not sure throwing 100k is a good suggestion, maybe should give a demonstration on how to server side page based on scroll position and grid height.

lazy loading of rows based on scroll wasn't something i had in mind, grid doesn't support it :( . . . . but am curious what the use case is for paging - if your report has more than 1000 rows, then the user is not going to go through each row of the report, typically they would filter / search / aggregate (either in the client or the server). my gut feeling is paging is from old-style applications, where you couldn't handle large data sets in the client. one of the strong points of angular grid is you can manage large datasets, and filter / aggregate you way through it!

I think @ceolter sees the light after our discussion on gitter ;)

+1

+1

+1

+9001
(:D)

it's coming it's coming!!!

I am with you ceolter. I don't see the point for pagination neither

Yeay!!!! At least one other person agrees with me!!!!
Unfortunately (for me!!) I'm still having to implement it, even if I'm
never going to use it myself!!!
I have the core of pagination and infinite scrolling working now, I'm just
patching it all up before merging into trunk. Will probably be in this
weekend.
On 11 April 2015 at 16:34, nelreina<EMAIL_ADDRESS>wrote:

I am with you ceolter. I don't see the point for pagination neither
—
Reply to this email directly or view it on GitHub
https://github.com/ceolter/angular-grid/issues/5#issuecomment-91869935.

--
The information transmitted is intended only for the person or entity to
which it is addressed and may contain confidential and/or privileged
material. Statements and opinions expressed in this e-mail may not
represent those of the sender. Any review, retransmission, dissemination or
other use of, or taking of any action in reliance upon, this information by
persons or entities other than the intended recipient is prohibited. If you
received this in error, please contact the sender immediately and delete
the material from any computer.

@ceolter I'll give you a use case:
Fill in the table with results coming from a paginated search (e.g. using ElasticSearch), because you hell don't want to load large bulks of data. In general, pagination is recommended for goal-oriented finding tasks.
Sources:

http://www.nngroup.com/articles/infinite-scrolling/
http://ux.stackexchange.com/questions/15982/regular-pagination-vs-infinite-scroll
http://ux.stackexchange.com/questions/33406/infinite-scroll-vs-pagination-in-e-commerce-websites

Don't also forget that providing an infinite scroll feature is more difficult than what you think, because it's not "just" infinite scrolling but also much more features to make it usable:

http://blog.adrianroselli.com/2014/05/so-you-think-you-built-good-infinite.html

so-you-think-you-built-good-infinite
http://blog.adrianroselli.com/2014/05/so-you-think-you-built-good-infinite.html
.
. . . . just read!!! . . . . mostly concerned about scrolling a webpage
though, not a grid.
funny, everyone else just asked for pagination, you asked for the best one
in the world!!!!
baby steps, baby steps, give my first iteration a try when it's ready :)
On 11 April 2015 at 17:37, Sébastien Heymann<EMAIL_ADDRESS>wrote:

@ceolter https://github.com/ceolter I'll give you a use case:
Fill in the table with results coming from a paginated search (e.g. using
ElasticSearch), because you hell don't want to load large bulks of data. In
general, pagination is recommended for goal-oriented finding tasks.
Sources:

http://www.nngroup.com/articles/infinite-scrolling/

http://ux.stackexchange.com/questions/15982/regular-pagination-vs-infinite-scroll
http://ux.stackexchange.com/questions/33406/infinite-scroll-vs-pagination-in-e-commerce-websites
Don't also forget that providing an infinite scroll feature is more
difficult than what you think, because it's not "just" infinite scrolling
but also much more features to make it usable:

http://blog.adrianroselli.com/2014/05/so-you-think-you-built-good-infinite.html
—
Reply to this email directly or view it on GitHub
https://github.com/ceolter/angular-grid/issues/5#issuecomment-91877238.

--
The information transmitted is intended only for the person or entity to
which it is addressed and may contain confidential and/or privileged
material. Statements and opinions expressed in this e-mail may not
represent those of the sender. Any review, retransmission, dissemination or
other use of, or taking of any action in reliance upon, this information by
persons or entities other than the intended recipient is prohibited. If you
received this in error, please contact the sender immediately and delete
the material from any computer.

@ceolter I can't wait for it ;)

ok pagination and infinite scrolling is in! going to leave this issue open, i'm sure am going to get hammered with "you forgot this and that", so please just add to this about how i did it wrong rather than raising new issues!!!!!

no feedback . . . . . :( . . . . . might have to close the issue . . . . .


Added https://tom.org.nz

Request to add my own site https://tom.org.nz

I'm so sorry I forgot to add a link back to the Webring! Updated "about" page to add the Webring SVG icon, thank you!

Why did you close it?


UX usability in images and collections

Is your proposal related to a problem?
When a user wants to change all images in one of the collections they experience an annoying problem. You select the collection select an image, do some stuff (change name etc.) hit save then you are resetted to the root collection.
Describe the solution you'd like
Can we stay in the collection when hit save? It will improve the User Experience

Likewise, when you are in the collection's page, and select a collection to filter the list, you'd expect the same collection to be preselected when you hit "add an image".

See #9177 for a more generic request.


Dump of current DB from server.

Not just the dump, but also a cheat-sheet for creating them.

Will add the uploaded images, as well. Just a sec, please.

@kleins05 Here you have a dump from the current production DB. Would you be able to do a timely review or should I ask someone else?
I suggest as review to try it locally following the first added section in the README. Some code changes in the config might require a little explanation. The respective commit messages might help. Or we could have a brief video chat.

@mdrie Thanks a lot! I could review by trying to import the provided database tomorrow. If you need it quicker, you need to ask someone else. I cannot review the export for lack of ssh access.

@mdrie How about using a custom management command for django to do the import of the snapshot. That reduces the README section to one line
python manage.py loaddbsnapshot 2023-04-27T141616Z --settings=config.settings.local
Take a look at branch database-import-command for a quick implementation.
Also, I would like to include the migration in the filename, i.e. prod_database_{migration}_{date}, so the correct migration can be restored automatically by the management command. What do you think?
Finally, what does that section in README.md:168 about migrations mean "If snapshot you want to use is based on an older model version, ...". Is that not already mentioned in the script above?


Typo of require for application.css

Thanks. Did it in an another commit https://github.com/outpunk/evil-icons/commit/e614041f944335335d271e524958aa97e60da4f6


Add "--leopard-url" CLI option

Enables testing a custom Leopard build locally, or providing any other alternate Leopard library host.
The default value is the same one that is used by default in toLeopard.ts. It would be nice not to duplicate this definition, but unless toLeopard defined it as an export (this would probably be awkward) or stops providing a default at all, it is what it is.
This option is used in both leopard and leopard-zip output formats, and ignored in other formats.
Only fully qualified URLs are supported, i.e. you can't just do /leopard/dist. This is to make the output folder work regardless where it's hosted (provided the specified URL is accessible there), and to avoid confusing a path on a hypothetical file server with a path on your own system.
There's no magic about e.g. taking a specified path to Leopard and copying out the dist/ files at time of sb-edit run (to make a self-contained folder/zip). That's out of scope for this PR, and may require adjustments to toLeopard internals (e.g. supporting relative imports there).
Typical development setup 1:
# terminal 1
cd leopard
npm run dev

# terminal 2
cd leopard/dist
npx vite --port 6201

# terminal 3
npx vite --port 6200

# terminal 4
sb-edit -i /path/to/proj.sb3 -o demo \
  --leopard-url http://localhost:6201/

# in browser, navigate to:
# http://localhost:6200/demo/

Typical development setup 2:
# terminal 1
cd leopard
npm run dev

# terminal 2
npx vite --port 6200

# terminal 3
rm -rf demo
sb-edit -i /path/to/proj.sb3 -o demo \
  --leopard-url http://localhost:6200/leopard/dist/

# in browser, navigate to:
# http://localhost:6200/demo/

Merged, thanks! I'm going to push 0.15.0 with this and other recent merges.


Restrict generate! paths to paths within crate boundaries

Crates using generate! with paths leading outside the crate boundary cannot be dependent upon in cargo vendor use cases.
E.g. see https://github.com/rvolosatovs/wasi-vendor-repro for a reproducer
Refs https://github.com/bytecodealliance/preview2-prototyping/issues/133
A draft PR will follow

Note, that the fix enabling cargo vendor use cases is actually trivial - symlinks https://github.com/bytecodealliance/preview2-prototyping/pull/134

Thanks for the report! I'm not sure if there's much that can be done within this crate, though, about that?

Thanks for the report! I'm not sure if there's much that can be done within this crate, though, about that?

The proposal here is to disallow path references like this https://github.com/bytecodealliance/preview2-prototyping/blob/083879cb955d7cc719eb7fa1b59c6096fcc97bbf/wasi/src/lib.rs#L7, which lead outside the crate
generate! would fail with a descriptive error in this case. I have a WIP PR ready for it, but adapting tests, which rely heavily on this feature in this repository is quite time-consuming

I'm not sure that disallowing outside-root paths is a way to handle this, though, since not all crates will make their way to crates.io which would cause this to be an issue. Personally I feel like it's better to be more flexible with the macro, but still have facilities available for those publishing to crates.io.

I'm not sure that disallowing outside-root paths is a way to handle this, though, since not all crates will make their way to crates.io which would cause this to be an issue. Personally I feel like it's better to be more flexible with the macro, but still have facilities available for those publishing to crates.io.

Well, this does not break the crates being published, rather it breaks all downstream consumers of these crates using the standard cargo functionality: cargo vendor. Whether those crates downstream consumers depend upon are published to a particular crate registry or just consumed via git is irrelevant here.


Fuse unmount hit device busy, cause fuse not exit correctly in k8s environment

Alluxio Version:
2.7.2 + Fluid 0.6
Describe the bug
We are using fuse as a sidecar container for ai platform, and find some pod stays in 'Terminating' state, which cause resource not released in Prod environment.
After training task finish, we need fuse container to exit to stop the whole pod, we are using 'Alluxio-fuse unmount' command to do that.
For those 'terminating' pod, we found sidecar container of fuse can not exit, since it can not unmount the mount point. If we try unmount that mount point, will receive device busy.
And if we use linux 'umount' command with -f flag, the terminating pod is deleted soon.
root@jssz-coeus-live-gpu-01:~ # umount /runtime-mnt/alluxio/ai/openimages-298661/fuse
umount: /runtime-mnt/alluxio/ai/openimages-298661/fuse: target is busy
        (In some cases useful info about processes that
         use the device is found by lsof(8) or fuser(1).)

To Reproduce
Confirm it is related to file handle not closed. If file handle is opened on some fuse file and try to umount fuse, then fuse will think it is busy
Suggested fix
Provide flag/config to Alluxio unmount command

Tested in plain machine
Scenario one:

Mount fuse
use vi to open file without closing it
run alluxio-fuse umount

Alluxio-fuse receives the kill -15 term call, Alluxio begin to umount, and throws FuseException that device is busy

After 2min, alluxio-fuse force kill AlluxioFuse process with -9

AlluxioFuse process is killed but the fuse mount point is left in device which is unacceptable.

I will address this scenario when also taking https://github.com/Alluxio/alluxio/issues/13239 in mind
Scenario two

mount fuse
vi file to open a file without closing it
umount  throws the following exception which is operating system logics (not AlluxioFuse code logics)

umount(/Users/alluxio/alluxioFolder/alluxio/standaloneFuse): Resource busy -- try 'diskutil unmount'

umount -f  AlluxioFuse process exited and fuse mount unmounted.

Fuse unmount logic ideally should be:

alluxio-fuse umount -s <mount_path> should soft kill fuse, which means that if device is busy, umount will fail
alluxio-fuse umount <mount_path> will make sure fuse process is killed and fuse mount is unmounted. In alluxio docker container, maybe we can use the umount  command to help kill fuse mount point.

Just want to confirm with @LuQQiu that current Alluxio-fuse command don't support hard unmount.

@Nizifan the current command doesn't support hard unmount, and as i mentioned in scenario one, the force kill will kill the Fuse process but left the fuse mount point in the operating system which is unacceptable


snap package

Includes a wrapper script and snapcraft.yaml  to build snap. I'm still running tests and will make some updates along the way.

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

@mikeroyal thanks for opening this PR.  Based on Microsoft policy all contributors are required to sign a Contributor License Agreement before we can merge.  Please see the previous bot comment for additional context.  Thanks!

@mikeroyal it has been a while, do you still want to merge the PR?

hi @alanrenmsft, I'm interested I just never heard back from everyone after the previous comment. Also, I would need to make some updates for the snap to work probably for the latest Ubuntu 20.04 LTS. Right now the Ubuntu 20.04.1 is scheduled for August 13th, which is when Ubuntu 18.04 LTS users will be prompted to upgrade to the 20.04 LTS release.

Thanks for this proposed contribution!  We don't have plans to add a Ubuntu snap package in the near future, so cleaning up older PRs.  We may reactivate in the future if we plan to introduce this deployment option.


Added pagination to entity single select component

1. Why is this change necessary?
This is related to https://github.com/shopware/platform/issues/835. It is actually possible to display more then 25 entries, but it is quite tricky to reach them with the search or keyboard.
2. What does this change do, exactly?
It extends the sw-entity-single-select component with a version of the sw-pagination, enabling browsing through select fields with pagination if there are too many entries for one 'page'.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

I reintegrated the block as well as the slot, so should be non breaking anymore.

Hey @kiplingi, thanks for your contribution 💙 Sorry for the late reply 🙂  Is this still relevant? I know that we actually have infinite scrolling on the sw-entity-single-select, but I don't know if we already had that at the time you created the PR.

Hi @bneumann97, no this can be closed, infinite scrolling solves it.


Quotes in column names - postgresql

Description
Quotes in column names seems to either be broken or not supported. I'm using an existing database and tables so I did not create the table though the interface
Steps to Reproduce the Problem
With the given table definition with quoted column
CREATE TABLE users (
    id integer NOT NULL,
    name text,
    "admin" boolean DEFAULT false
);

And the following model details
type User struct {
    ID int `json:"id" db:"id"`
	Name string `json:"name" db:"name"`
	Admin string `json:"admin" db:"admin"`
}

and alternatively with attempted quoted strings
type User struct {
    ID int `json:"id" db:"id"`
	Name string `json:"name" db:"name"`
	Admin string `json:"admin" db:"\"admin\""`
}

Expected Behavior
I expected to get the json results pulling from the table which I do receive if I quote out the admin field in the struct. The issue only appears with columns with quoted names
Actual Behavior
For the first model I get the following sql statement showing in buffalo dev
SELECT admin_users.admin, admin_users.name, admin_users.id FROM admin.users AS admin_users

which fails in pgadmin because the quotes are missing.
and then the following error:
{
    "error": "ERROR: column admin_users.admin does not exist (SQLSTATE 42703)"
}

With the alternative model the following sql statement is shown in buffalo dev
SELECT admin_users."admin", admin_users.name, admin_users.id FROM admin.users AS admin_users

which I can query successfully through pgadmin as the quotes are there.
But also generates the following error
{
    "error": "missing destination name admin in *[]models.User"
}

I of course may be trying to handle the quotes incorrectly so any guidance would be greatly appreciated. I was unable to locate any mention of how to handle this in the documentation. I also was not able to find any one else reporting the issue here or on slack channel suggest in the support section of the documentation
Info
rhel8, Pop 5.3.2 version through Buffalo v0.16.23

updated to pop 5.3.4 through buffalo v0.16.23. No change.


Calculate and log average fitness

It's very hard to get an idea of how the repair is progressing, since we only say anything if we find something. Calculating the average fitness of a generation and outputting it would be nice.

Added in 2c0e5f3 in #50


I'm from China and I have sighed up your DATASET, but I can't get a link.My count<EMAIL_ADDRESS>Thank you in advance!

Hi Jason, you'll have to sign up with an academic email. Sorry for any inconvenience!


feat: really use ruff instead of ruff_lsp (#4)

This commit prevents the following error:
/usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:800: Spawning language server with cmd: { "ruff-lsp" } failed. The language server is either not installed, missing from PATH, or not executable.
and will allow the attachment of the new ruff server when editing Python files.

Ah, overlooked that one, thx


intellisense for js suddenly stopped working

Issue Type: Bug
Cannot reproduce, I open any project and doesn't autocomplete anymore (js projects). Ctrl + space don't show anything useful now
VS Code version: Code 1.28.0 (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-04T16:40:40.180Z)
OS version: Darwin x64 17.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.54GB free)

Process Argv
.

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

vscode-svgviewer
css
1.4.6

vscode-eslint
dba
1.6.0

prettier-vscode
esb
1.6.1

auto-close-tag
for
0.5.6

(1 theme extensions excluded)

whenever I hit ctrl + space, I see this line in the window log:
[2018-10-09 11:19:25.745] [renderer1] [error] An unknown error occurred. Please consult the log for more details.
[2018-10-09 11:19:42.461] [renderer1] [error] An unknown error occurred. Please consult the log for more details.

Does this still happen if you disable all extensions and restart VS Code. Also please look at the typescript server trace :

Set "typescript.tsserver.trace": "verbose"
Reproduce the problem
Open the output panel and share the output from the typescript output section.

Stumbled upon this issue since Intellisense for my Ember.js JavsScript project had also suddenly stopped working after the update to 1.28.0. I eventually resolved it by manually installing @types/ember from npm, despite the fact that it was working fine without installing types before. I checked my Window log and I've noticed the exact same log entries around the time Intellisense wasn't working.
The problem still isn't fully resolved though since my JavaScript auto-imports are now doing an in-place dynamic import (import("../path/to/module").ModuleName) where I press Ctrl+Space on a symbol instead of the regular ES2016 style import at the top (import ModuleName from "../path/to/module";) which is very annoying and completely useless for me since it's against our code style.
I've tried disabling / reinstalling every Ember.js related extension but the problem persists.

 Request took 29 ms. Success: false . Message: Error processing request. Cannot read property 'some' of undefined
TypeError: Cannot read property 'some' of undefined
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:94664:59
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:108298:21
    at forEachExternalModule (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:108311:21)
    at Object.forEachExternalModuleToImportFrom (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:108296:13)
    at getSymbolsFromOtherSourceFileExports (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:94651:28)
    at getGlobalCompletions (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:94548:21)
    at tryGetGlobalSymbols (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:94471:25)
    at getCompletionData (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:94350:22)
    at Object.getCompletionsAtPosition (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:93542:34)
    at Object.getCompletionsAtPosition (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:114798:35)
    at IOSession.Session.getCompletions (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:122583:64)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:121645:61)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:123061:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:123052:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:123061:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:123081:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:124344:27)
    at emitOne (events.js:116:13)
    at Interface.emit (events.js:211:7)
    at Interface._onLine (readline.js:282:10)
    at Interface._normalWrite (readline.js:424:12)
    at Socket.ondata (readline.js:141:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)
[Error  - 2:54:23 PM] 'completionInfo' request failed with error.

Thanks. from the log this looks like this upstream issue: https://github.com/Microsoft/TypeScript/issues/27640
Will continue investigating upstream. If you have a small example project that reliable reproduces this issue, please share it as well


Added translation for login attributes

Hi!
In order to have a sensitive default translation for any failed login attempt (e.g. 'username is required' or 'password is required'), I added the commited attributes.

Hello @tobysommer ,
problem with styleci

sorry, fixed

Danke


Fixes Rails 7.2 dep warning

Same fix at the last but for release 3.6

Thanks, I addressed it in main and 3.6.4 which is now out


Android XHR longpoll "reconnects" each time

Hello guys, I've seen that there are couple more topics on the subject, but I still couldn't find the andswer to the question. So here's the problem: Android continiously reconnects to the server as the new client.
Little bit more details on the subject.
Client

Device: Samsung Galaxy (GT-I9000)
Android version: 2.2
Build number: FROYO.XWJP6

Server

Node.js version: v0.6.1 running on Windows 7
Socket IO version: 0.8.7

List of installed modules (npm ls)
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+-T<EMAIL_ADDRESS>¦ +--<EMAIL_ADDRESS>¦ +--<EMAIL_ADDRESS>¦ +--<EMAIL_ADDRESS>¦ L--<EMAIL_ADDRESS>L-T<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  L-T<EMAIL_ADDRESS>    +--<EMAIL_ADDRESS>    +--<EMAIL_ADDRESS>    L--<EMAIL_ADDRESS>
The server code:
var express = require("express");

var app = express.createServer();
var io = require('socket.io').listen(app);

app.use(app.router);
app.use(express.static(__dirname + "/public"));

app.get("/", function(req, res) {
    res.sendfile(__dirname + "/public/dbg.html");
});

var maxUserId = 0;

// This limit is here to check if desktop browser behaves the same
// way. No it is not.
io.set('transports', ['xhr-polling']);
io.sockets.on('connection', function (socket) {
    var userId = maxUserId++;
    var userName = "User " + userId;

    socket.emit('message', { text: "You have connected" });

    socket.broadcast.emit('user-joined', {name: userName});

    socket.on('disconnect', function () {
        socket.broadcast.emit('user-left', {name: userName});
    });
});

app.listen(80);

The client code
dbg.html that is served as static from the server
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8" />
<meta name="viewport"
	  content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no, target-densitydpi=device-dpi"/>

<script src="/socket.io/socket.io.js"></script>

<script>
	function init() {
		var socket = io.connect();
		socket.on('message', function (message) {
			console.log(message.text);
		});

		socket.on('user-joined', function (user) {
			console.log(user.name + " joined")
		});

		socket.on('user-left', function (user) {
			console.log(user.name + " left");
		});
	}
</script>
</head>
<body onload="init()"></body>
</html>

Observed behaviour:
When I connect to the server with the desktop browser, everything is running fine, I'm getting single "You have connected" message to the console, no matter how much time I leave the page opened. When I connect from the device, it reconnects as the different client all the time.  At the same time my desktop browser is reporting that clients are connecting and disconnecting after 20 seconds:
You have connected
http://localhost/:18User 2 joined
http://localhost/:22User 2 left
http://localhost/:18User 3 joined
http://localhost/:22User 3 left
http://localhost/:18User 4 joined

Logcat output:
> adb.exe logcat -v raw browser:I *:S
--------- beginning of /dev/log/main
--------- beginning of /dev/log/system
Console: You have connected http://<IP_ADDRESS>/:14
Console: You have connected http://<IP_ADDRESS>/:14
Console: You have connected http://<IP_ADDRESS>/:14
Console: You have connected http://<IP_ADDRESS>/:14
Console: You have connected http://<IP_ADDRESS>/:14
...

Note that I'm not navigating around or reloading the page... It just stays opened
My blind guess (was wrong).
It seems that android client code fails to provide the same identifier when he makes the new connection to the server. The server thinks that this is the entirely new client, and fires the "new client" routine. That's a blind guess. I'm trying to debug the client code now, but it is kinda slow on Android.
If someone knows how to get rid of this issue, please let me know.

I have similar issues with a HTC Sense and a Sharp IS03 using port 80 (japanese carrier softbank only allows http requests over port 80 in their android phones..).
Mine disconnects far sooner than 15 seconds though, more like 0-2. Only happens over xhr-transport.
Any ideas as to why this might happen?

Try to debug in on server-side and see what is the reason of disconnects. Put few breakpoints in Transport class: Transport.prototype.onSocketEnd, Transport.prototype.onSocketClose, etc. Also try to check the connectivity on WiFi - does it show the same problems?
As the last resort - try to simplify the case even further and write the test case without socket.io, but with plaint http server (or express).
I know that this is not the answer that you're looking for, but if you follow these directions you could probably investigate another problematic case and describe the root cause of the problem to the other people who may be facing it.

@faeldt, try using wireshark + tcpdump(on server side) and you'll get precious answer on your question.

Thank you for your answers. I will investigate further and try to come up with something useful.

I have same issue on Android. socket.io client 0.8.7

Founded the cause of this problem - simultaneous posts!
Here is my fix - https://github.com/LearnBoost/socket.io-client/pull/388


Arbitrary precision fixed-point support

Dear Team,
This is awesome work. I’m wandering do we have arbitrary precision fixed point version of this framework or other counterparts?

We are going to push later this week an update to the library that allows the selection, at HW generation time, to select the desired precision on each layer.

Closing since the updates are already available on the repo.


Is there a way to add meta information to a Resource Object or an Identifier?

I need to qualify a relationship with some data, which can be accomplished with the meta object.
However, adding a meta object to an Resource Object Identifier (using the mockHandlers's examples option) results in a ValidationError: "meta" is not allowed, and the server omits the offending Resource Object. The same error is received when attempting to create a Resource Object with meta information.
jsonApi.define({
  resource: "example",
  attributes: {
    qualified_relationship: jsonApi.joi.many("other"),
  },
  examples: [
    {
      type: "example",
      qualified_relationship: [
        {type: "other", id: "1", meta: {quantity: "2"}}
      ],
      meta: {
        description: "It should also be possible to add a `meta` object to a document."
      }
    }
  ]
})

jsonApi.define({
  resource: "other"
  examples: [
    {type: "other", id: "1"}
  ]
});

Absolutely, its a straight forward addition. I'll see what I can do this weekend :+1:

@Artanis does this behave as you'd expect?

It does.
The only concern I have is that the meta member should be an object, and the pull types it as Any.

Good call. I've also added some changes and test cases for creating and updating both resources and relationships metadata to prove it flows down to the handlers. This allows implementations to either accept changes to metadata, silently ignore invalid properties or respond with errors.


Additional "Magnet" feature?

Precheck

Do not use the issues tracker for help or support (try VRTK Slack channel, Stack Overflow, etc.)
For proposing a new feature, please check existing open and closed issues before creating a duplicate
For bugs, do a quick search and make sure the bug has not yet been reported
Finally, be excellent to each other, and party on dudes!

Environment

Source of VRTK (Unity Asset Store or Github)
Version of VRTK (Unity Asset Store/Github release number) (Github master commit hash)
Version of Unity3D (e.g. Unity 5.4.4f1)

Steps to reproduce
Attempt to recreate the issue in a VRTK example scene and provide steps to reproduce if possible. Include code samples, errors and stacktraces if appropriate.
Expected behavior
Briefly describe what you expect should happen
Current behavior
Briefly describe what is actually happening

It looks like something went wrong here?


Microphone method is still a bit too broad

Camera shows microphone dot when it shouldn't

- After calls, microphone dot remains (seemingly ignoring the code addressing this?)

The call issue has been fixed in v1.0.8. The primary mic method is not perfect, but it is accurate enough, so I am closing this issue.

The call issue has been fixed in v1.0.8. The primary mic method is not perfect, but it is accurate enough, so I am closing this issue.


support DELETE /v1/teams/:team_name/posts/:post_number/watch #26

Close #26

Codecov Report

Merging #95 (e3728e9) into main (5092ac4) will decrease coverage by 0.33%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   97.37%   97.04%   -0.34%
==========================================
  Files          25       25
  Lines        1029     1049      +20
==========================================
+ Hits         1002     1018      +16
- Misses         18       22       +4
  Partials        9        9

Impacted Files
Coverage Δ

esaapi/watch/types/response.go
66.66% <0.00%> (-33.34%)
:arrow_down:

esaapi/watch/types/parameter.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Question: Parsing CLI Output

Thanks for putting this together. I stumbled on this after cobbling this all together myself.
Any ideas on how to parse the CLI output to fail the build or deploy on any SFDX error?
In otherwords, if the deploy command responds with "ERROR running force:mdapi:deploy:  The metadata deploy operation failed.", how do I mark the task as failed in bamboo?
Thanks again!

I don't know if this is still a problem for you but it may help others:
Here's how I handled it (I'm using this with a scratch org so not using force:mdapi:deploy)
parse the result with jq into a variable.
export PUSHRESULT="$(eval sfdx force:source:push --json | jq '.status')"
switch on the result variable.
if [ "$PUSHRESULT" == 0 ]; then
echo Push Succeeded
elif [ "$PUSHRESULT" == "1" ]; then {
echo "Error in push";
exit $PUSHRESULT;
}
elif [ "$PUSHRESULT" == "2" ]; then {
echo "Error in push - bad syntax";
exit $PUSHRESULT;
}
elif [ "$PUSHRESULT" == "127" ]; then {
echo "Error in push - File not found";
exit $PUSHRESULT;
}
else {
echo "Unknown error in push - Check log";
exit $PUSHRESULT;
}
fi
This will only work for those CLI commands that have an option to view results as JSON.
Here's another option:
sfdx force:org:create --setdefaultusername --definitionfile config/enterprise-scratch-def.json --wait 10 --durationdays 1
if [ ${#} != 0 ]; then
echo "Scratch org could not be created";
exit $?;
fi
Also - this assumes you have a runner set up to run the script.  In that runner batch file, you run your script like this:
docker run --cap-add=NET_ADMIN --name=$1 salesforce
if [ "$?" != 0 ]; then
echo "Error running the deploy script in the salesforce container.  Check the log"
exit -1;
fi
This will allow you to check if the docker run command was successful or not.  If the result of the docker run is unsuccessful, $? will not be equal to 0 and you can have it exit.  This will cause bamboo to stop and you can check your log.


Error when updating the event's actual date overlapping the next event in line

How to reproduce

Create any type of work
Put actual date in any event which would be greater than the next event's date

Expected

It should get saved and push the subsequent events according to the user choices

Actual

Throwing error


Bump Dependencies #11766178173

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/11766178173

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


extract_msi fails if Windows is below version 1903

Microsoft PowerToys version
0.37.0
Running as admin

[X] Yes

Area(s) with issue?
Installer
Steps to reproduce
Attempt to extract the msi on a windows version below 1903. I was using Windows Server Core 2016.
PowerToysSetup-0.37.0-x64.exe --extract_msi
✔ Expected Behavior
MSI to be extracted without a Windows version check.
❌ Actual Behavior
MSI is not extracted because the Windows version is too old.
Other Software
No response

@w0
starting with 0.37, the minimum version supported is 1903 https://github.com/microsoft/PowerToys#requirements

@enricogior
Msi should be able to be extracted regardless of the version. This is not an install scenario.

@aravindvnair99 https://github.com/microsoft/PowerToys/issues/10563

Resolved.


AQCU-1143: changed sort order of extremes values before selecting val…

…ue, need to select first occurrence whereas previously was selecting last

Coverage remained the same at 98.8% when pulling ab69a6d62aaef0c4254122d4d141a76ef9fdfd85 on mhines-usgs:AQCU-1143 into 32796ccb486bb780cc62126309d5d1a78f24361d on USGS-R:master.

Testing this locally now


Support Python 3.8

Closes #8
Tested on Python3.8-Python3.11.
Added explicit handling of List, Union types to make the tests pass. The proper handling of all such types should be handled in a separate issue #10.

I will take a closer look at this today.

Thanks! I'm not sure what to do with the 100% coverage requirement. I wouldn't pursue it, but if it's a must, maybe you can assist with that.

I am afraid the 100% requirement is a must, yes. Do you need help with an idea for the test?

Understood. In most of my projects, I combine the coverage from all CI runs, so 100% for different python versions is easily reachable.
Please, take a look whether pydantic duality is the same. I would assume so.

FYI, the cumulative coverage (after running coverage combine) is 100%.

@zmievsa, Python 3.8 support is critical for our project. We could depend on the commit in main but prefer to see a release with this PR. Any plans about the next release date?

I'll try to release it today.

Verified and published it.


How to call AppIntro at any time I need?

Situation:
I have MainActivity and onCreate(Bundle savedInstanceState) I'm calling:
appManual myAppManual = new appManual();         myAppManual.init(savedInstanceState);
But my app crashes on launching MainActivity.
What I'm doing wrong?

Im getting error:
java.lang.RuntimeException: Unable to start activity ComponentInfo{APP.PACKAGE.NAME/APP.PACKAGE.NAME.MainActivity}: java.lang.NullPointerException

And other one:
 Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'void com.github.paolorotolo.appintro.PagerAdapter.notifyDataSetChanged()' on a null object reference

If you've read the readme, you need to build an intent that launches the intro activity. In the activity's init method, you can add your slides and decide animations. The way your initializing your intro is what your problem is.


Remove faulty debug config

The previous debug config caused a runtime error. Simply launch the application.py instead.

LGTM


Support python setuptools package-data

I'm building a python project that includes jinja templates and css files. Unfortunately it seems like the dream2nix pip module doesn't support setuptools package-data as those files are not included in the final derivation.
Are there any plans to support this or workarounds to get these non-python files included in the final derivation?

It should be supported. Could you open an issue with a minimal reproducer flake where it doesn't work?


Remove leading and trailing spaces from secret values

We had an issue where copy/pasting a value in the editor introduced a space at the beginning of the secret value.
This situation caused an error in our app because of this invalid character.
Could we at least trim the leading and trailing spaces and maybe detect other invalid characters (like bullets copied from an email or a Word document)?

Hi, this is a very good idea. The philosophy of the Secret Editor is that it allows you to save whatever you want, and only displays warnings if a validation fails. E.g. if the secret value is a malformed JSON, you get a notification, but you can save it, if you really want it. Based on this, I would not trim the values automatically, but I can imagine a new warning notification if there is a whitespace at the beginning or at the end.

Hi, I hope the new features in the 1.16.0 version will help your issue.


cluster_logs_management/send_logs-s3.sh -- add region to aws s3 sync commands

Describe the bug
AWS_DEFAULT_REGION / region of S3_BUCKET incompatibility raises error when the cluster_logs_management/send_logs_s3.sh runs
Error message:
download failed: s3://XXXXXX/soca-YYYYY/ An error occurred (IllegalLocationConstraintException) when calling the GetObject operation: The eu-central-1 location constraint is incompatible for the region specific endpoint this request was sent to.
To Reproduce
My bucket is in eu-central-1; SOCA is in il-central-1.  The il-central-1 region uses region-specific endpoints.  I imagine the same error would happen when there is a similar bucket-region/soca-region incompatibility and the soca-region in the account uses region specific endpionts
Please complete the following information about the solution:

[ ] Version: 2.7.5

[ ] Region: il-central-1

[ ] Was the solution modified from the version published on this repository? No

[ ] If the answer to the previous question was yes, are the changes available on GitHub? NO

[ ] Have you checked your service quotas for the services this solution uses?  (<--there's still a typo in "sevices").  N/A

[ ] Were there any errors in the CloudWatch Logs? N/A

Bug fix
Modify the code as follows:
S3_BUCKET="s3://$SOCA_INSTALL_BUCKET/$SOCA_CONFIGURATION/cluster_logs/"
S3_BUCKET_REGION=$(aws s3api get-bucket-location --bucket $SOCA_INSTALL_BUCKET --output text)
.
.
.
/usr/bin/aws s3 sync $SCHEDULER_SERVER_LOGS $S3_BUCKET'server_logs/' --region $S3_BUCKET_REGION
/usr/bin/aws s3 sync $SCHEDULER_SCHED_LOGS $S3_BUCKET'sched_logs/'  --region $S3_BUCKET_REGION
/usr/bin/aws s3 sync $COMPUTE_HOST_LOG $S3_BUCKET'compute_host_logs/'  --region $S3_BUCKET_REGION

Thanks @tammy-ruby-cherry  for the report and proposed change.
As we are in the process of refreshing this section, I'd like to know if sending these logs to CloudWatch (via LogAgent) would make sense to you?

Closing ticket. Changes have been made to the internal repo and will be made available publicly on the next release.


[incubator/cassandra] Add way to initialize cassandra keyspace

When I install Cassandra via the incubator/cassandra Helm Chart, there are no keyspaces in the deployed Cassandra cluster. I want to deploy a service that I have no direct control over. This service requires a Cassandra Keyspace to exist in the cluster.
The obvious solution to the problem is to manually do something like
kubectl exec -ti cassandra-0 -- cqlsh -e "CREATE KEYSPACE IF NOT EXISTS my_keyspace WITH replication = {'class': 'NetworkTopologyStrategy', 'DC1': 3};" <IP_ADDRESS> 9042

Instead, I would prefer if I could get rid of this manual step. Maybe a configurable postStart handler could be provided that can be configured with one or more cql statements?

This is a valid feature request, I think.

Still valid and I'm still interested.

Still valid and I'm still interested.

⏳


Android Hostname missing from request Origin, causing different origins between ios and android

Hi,
We are migrating our ionic 2 app to ionic 4.
With the latest ios requirements for migrating to WKwebview, ios scheme must differ from http, so we want to unify both android and ios platforms origins to be the same (scheme://Hostname), so that we wont need to deal with 2 different origins.
ios default is ionic://localhost
android default is http://localhost.
problem is, when assigning
<preference name="Scheme" value="httpmyscheme" /> for android
<preference name="iosScheme" value="httpmyscheme" /> for ios
and <preference name="Hostname" value="myhostname" /> in config.xml, the origin on android devices is always "httpmyscheme://" without the hostname "myhostname" defined in config.xml .
In ios devices, it works perfectly fine.
Ionic:
Ionic CLI                     : 5.4.16 (C:\Users\Shay\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework               : @ionic/angular 4.7.1
@angular-devkit/build-angular : 0.803.25
@angular-devkit/schematics    : 8.3.25
@angular/cli                  : 8.3.25
@ionic/angular-toolkit        : 2.2.0
Cordova:
Cordova CLI       : 9.0.0<EMAIL_ADDRESS>Cordova Platforms : android 9.1.0-dev
Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 7 other plugins)
Utility:
cordova-res : not installed
native-run  : not installed
System:
NodeJS : v10.15.0 (C:\Program Files\nodejs\node.exe)
npm    : 6.4.1
OS     : Windows 10

This is a webview/chromium "feature", I reported it long ago and they closed as won't fix because "it works as expected" according to them.
There is nothing we can do about it and google engineers have told us to discourage people from using custom schemes on Android, see https://github.com/ionic-team/cordova-plugin-ionic-webview/issues/484


docs

snake-charmer set project version from 0.11.1 to 0.12.0. If you do not want to release a new version with this PR remove the release label from this PR


Adding trigonometric functions

I would like to add trigonometric functions such as:

sin
-cos
tan
cot
sec
cosec

Please assign me this issue and also add hacktoberfest label in it.
Thank you.

https://github.com/DhanushNehru/calculator/pull/62

I have created the pull request.
kindly merge it
#70


Move private key input from cli arg to keyfile

https://github.com/carver/trin/pull/35#discussion_r661832189

Closing since we don't have any immediate plans to support a keyfile


How to handle dynamic typedStorage keys with TypeScript ?

How my schema should look then like?
const memoryStorage = new MemoryStorage();
export const typedStorage = new TypedLocalStore<Schema>({
  fallbackStorage: memoryStorage,
});

const BOARD_KEY = `gameBoard_${gameID}`;
const savedBoard = typedStorage.getItem(BOARD_KEY);

I didn't tested it but I guess something like this:
interface Schema {
  [key: `gameBoard_${number}`]: string; // number can be the type of your gameID
}

Source: https://stackoverflow.com/a/65879197
Let me know if it helped.


Utility Pages type and spacing

body text line height: 24/25px ( will effect all pages with this style)
bottom margin: 80px

note font size need not change

bottom margin remains to be done.


Sky region plot

In #76 (I think) I argued to remove the contains and plot method on SkyRegion objects, because it can be ambiguous how to transform the sky region to a pixel region and it's better to let the user be explicit:
sky_region.to_pix(wcs).plot(...)
sky_region.to_pix(wcs).contains(...)

Now using it myself, and seeing others plot many regions in Gammapy, I find it annoying to have to do sky_region.to_pix(wcs).plot(ax=ax) all the time, and would prefer sky_region.plot(ax=ax) to work directly and do whatever DS9 does to plot, presumably already what our to_pix(wcs).plot(ax=ax) does.
@astrofrog or anyone - is that possible? Does the ax have a link to the wcs when using wcsaxes, so that we can add a one-lineer in a method SkyRegion.plot in the base class to make this work?
Another issue I have with region plot is that if I call PixRegion.plot and then plt.legend, it never shows up in the legend. Only if I call region.as_artist and ax.add_artist does it show up in the legend. Does someone know if it's possible to plot regions and have them added to the legend in a simple way? Would be nice to extend the https://astropy-regions.readthedocs.io/en/latest/plotting.html page with an example.

This sounds like it should be possible and I like the approach you've suggested for the skyregion plot api.
The second issue, with the legend, sounds like a bug, and I don't understand it.

@cdeil - if you are using WCSAxes, then ax.wcs should give the WCS

The approach you've taken looks fine, that's what I'd do.


Jump to correct anchor for onboarding schools

Hi @urbanwide I've merged this down so I can release on test with a couple of other PR's.  hope that's all ok.


Add more patterns/examples for deploys

I would like to see more patterns/examples for deploys.

Ansible's docs usually contain a useable snippet as a starting point. Could the pyinfra docs be expanded to include examples? For example, the apt.packages could have an example of what a typical code snippet would look like.

Would like to see examples on using the "when" or a conditional task, so as to make deployments quicker/idempotent.

For example, the "if CentOS" in deploys.rst could show how to also use the "when" format.
Another example would be to check for the existence of a file then use that as a "when" condition.
Yet another example, would be to run a remote shell command and use the output from that as a "when" condition in a task. (a good example of this would be check if docker apt key is installed before needing to download that key)

For 1 I think this is a fantastic idea - I think adding examples to each operartions docstring would be a huge improvement to the documentation especially people new to pyinfra - I think the current list of arguments can sometimes be quite overwhelming, especially when most are often not required.
As for 2 - I think the when argument may be worth deprecating/removing entirely - and normal Python conditionals should be used; ie:
if host.fact.linux_name == 'CentOS':
   dnf.packages(...)

IMO this is easier to read (any more Python-y) than the old when syntax (which dates back to pyinfra not handling conditionals properly):
dnf.packages(
    ...
    when=host.fact.linux_name=='CentOS',
)

For a file check this can already be achieved with:
if host.fact.file('/path/to/file') is None:
    files.put(...)

And for the apt key example - ideally the apt.key operation should handle this internally (it does not currently). Ie always calling apt.key(...) in the operation and pyinfra figures out whether the key exists, and if so does nothing. This is how most modules work where possible (ie apt.packages('iftop') will do nothing if iftop is already installed).
There is also a documented example for dynamic execution of commands during a deploy.

I agree on the "when" part. Thanks for sharing the "if" with the "file" fact. I'll leverage that for stuff in the future.
It is funny that I try hard to find an example where a "system" command is needed and you show that it should be handled internally. (which I agree, but I'm just providing an example that I found... lol)
So, I took a stab at the apk doc. See it here:
https://github.com/Fizzadar/pyinfra/compare/master...mkinney:add-doc-examples?expand=1
A few questions:

Why does that show all of my prior commits? I thought it would just go from master and show the two commits. I wonder if my refresh upstream commands of:

git fetch upstream
git checkout master
git merge upstream/master
git push

Is missing some step. Any ideas?

I also added and "examples" (perhaps it should be "standalone_examples" or something) and added that same code snippet. That is duplicated effort. Any way to say... go include that code snippet in that dir from inside the .rst file?

How do you view the .rst when doing dev? I am using MacView, but it does not render correctly?

Might be answered in the answer for above, but how do you locally generate the doc?

Do you want to include this pattern?

https://github.com/hoh/pyinfra-formulas/blob/master/formulas/nodejs/__init__.py#L33

I also need some help with processing a simple command. For instance, if I have something like this:

server.shell('apt-key list')
How do I get the stdout?

Is there a way to get all facts?

What is the list of the meta info from a command (like changed)?

@mkinney the APK docs look good - one thing is the docs/modules/*.rst are autogenerated so you'll want to add the example in the Python module itself; ie under https://github.com/Fizzadar/pyinfra/blob/master/pyinfra/modules/apk.py#L48. (On that note I'm going to investigate not having the autogenerated docs in git, and generated on doc build).
I've answered your questions below:

That should be fixed now - I think it was pre my merge of the Mech branch?

It is possible but with the above change it won't be.. it might be worth putting the APK example in examples/packages.py. Basically the idea is the examples directory is full deploys, whereas examples for single operations would live in the Python docstrings.

3 & 4. If you have everything installed in a virtualenv you should be able to run scripts/build_docs.sh to generate all the docs, and then load up docs/build/index.html in a browser.
5 & 8. Yep! The meta output can be really helpful here; and is very briefly documented here - the implementation is @ https://github.com/Fizzadar/pyinfra/blob/master/pyinfra/api/operation.py#L44 - basically keys are changed, commands and hash (currently).

This is not currently possible - but if you use host.fact.command('apt-key list') that will return the stdout.

Not currently! They can be listed (pyinfra --facts) but no way to fetch them all as is.

Let me know if you've any more questions - this is incredibly helpful for improving the pyinfra documentation and massively appreciated!

Thanks for looking at 1 above.
I tried (but failed) to get the doc updated. The scripts work, but I could not get the locally built files to reflect changes. I was trying to use:
    Example:
    .. literalinclude:: ../../examples/apt.py

I think I'm just going to build some standalone examples for now. Let me know if you figure out how to get code snippets to be included.
FYI, I did something like this to get all facts:
pyinfra <IP_ADDRESS> --user vagrant --password vagrant --facts | tail -n +2 | awk '{ tnf=split($0,x); for (i=0;i < tnf;i++) { s=x[i]; sub(",", "", s); all[s]=1 }}; END { for (e in all) { if (e != "") { print e }}}' | sort | awk '{ printf("%s ", $1) } END { print ""}'
Which returned:
apk_packages apt_sources block_devices brew_casks brew_packages command cpus date deb_package deb_packages directory docker_container docker_containers docker_image docker_images docker_network docker_networks docker_single_mixin file find_directories find_files find_in_file gem_packages git_branch git_config home initd_status ip6tables_chains ip6tables_rules iptables_chains iptables_rules ipv4_addresses launchd_status link linux_distribution linux_name lsb_release memory mounts mysql_databases mysql_user_grants network_devices openvz_containers os pacman_packages pip_packages pkg_packages postgresql_databases rcd_status rpm_package sha1_file short_fact_base socket sysctl systemd_enabled upstart_status
Then I removed the ones that required args:
pyinfra <IP_ADDRESS> --user vagrant --password vagrant fact apk_packages apt_sources block_devices brew_casks brew_packages cpus date deb_packages docker_containers docker_images docker_networks gem_packages git_config home initd_status ip6tables_chains ip6tables_rules iptables_chains iptables_rules ipv4_addresses launchd_status linux_distribution linux_name lsb_release memory mounts network_devices openvz_containers os pacman_packages pip_packages pkg_packages postgresql_databases rcd_status sysctl systemd_enabled upstart_status
The reason for this is so I could look for any fact that might contain a string (like "amd64").

init was not obvious to me that it was for services. Not sure how to show that in the docs.

Hmm - for the docs build what error were you getting? I'll try a fresh virtualenv later to see if I can replicate.
For the facts, I believe the arch fact would give you the amd64 string. I've also added an issue to get all facts in one go as it'd be nice to have.
8 - interesting, I think this makes sense. service.systemd vs init.systemd I suppose. v1 is coming up which could break this; or service could be an alternative name for the same module. Will have a think about this!

I've added an example for documentation-based examples here: https://github.com/Fizzadar/pyinfra/commit/12bf145ec0971a578546deeccae83cca79ab1da1 which can be seen here: https://pyinfra.readthedocs.io/en/latest/modules/apk.html#apk-packages.
Keen to keep the inline examples as short as possible (no imports/etc) or the docs will get out of hand. This also nicely separates examples/ vs the docs.

Nice! That looks great. Thanks.
I still think it would be good to have a "standalone scripts" for each in examples somewhere. That way, people don't get tripped up on what the includes would need to be or anything. Plus they could show various patterns.

Agreed - it'd be neat to have examples that cover using every operation in a given module. It's also much more explicit than some of the current example names!

Working on it... it is slow going, but I think that's ok. I've found a few bugs and will probably need your help with some things. (as evidenced by some new issues)
See https://github.com/Fizzadar/pyinfra/compare/master...mkinney:add-examples?expand=1 for progress, if you're curious.

Just had a quick read through - looking great so far 👍

I was unable to test pacman as the Arch instance I was using kept erroring on gpg keys. I've asked the maintainer wouldn't mind re-spinning the image. (https://github.com/mrlesmithjr/packer-templates/issues/54)
I was unable to test pkg.packages() as neither freeBSD 11 nor 12 would work. (errors were something like "pkg_info" and probably should be "pkg info"). I tried openbsd, but it would take longer than 10 seconds to connect. I tried hacking the connection timeout, but ran into a paramiko error. So, not sure if you have any better luck with validating pkg.packages().

mrlesmithjr got the Arch image working so I was able to get a pacman standalone tested/uploaded.

I tried using the generic/openbsd6 and it works so I was able to get the pkg.packages() example added.


chore: update import path for messages in cypress to ee path

Description
Updates the import path to get messages from ee/constants where we have both re exported ce messages and added ee messages.
Fixes #
Type of change

Please delete options that are not relevant.

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=fadf09d


Send and receive FDs

I added two functions read_bufs_fds() and write_bufs_fds() to send and receive file descriptor over Unix domain socket. It uses the sendmsg() and recvmsg() system call. I plan to add this feature to the tokio framework, in the future.

mio-uds is deprecated at this point.  I encourage you to work with the mio folks and see if it fits into their native UDS support in 0.7.x.


Why isn't PDF-LIB hosted at deno.land?

Do you believe your library would get a lot more exposure if it was the only PDF library hosted at deno.land? Because it would be the only PDF library available to everyone looking for extensions.

pdf-lib relies on https://www.skypack.dev to convert its compiled source into something that Deno can work with. In particular, the import syntax produced by the build is only compatible with Node (or common JS loaders). At least, that was the state of things last I checked. That being said, it would be cool to be listed on deno.land, so I've added it to the roadmap: https://github.com/Hopding/pdf-lib/discussions/998. Thanks for the suggestion!


git_deps: document the script and handle existing files

Very nice ! Thanks for for your update ! :-) I merge.


[BUG]: Not able to import the COE Starter Kit core solution into a Dataverse for Teams environment

Describe the bug
New day, new error. This time, the solution import fails with the error message "The imported solution and the existing solution must both be managed for upgrade.", see screenshot

The log file shows an error about missing capacity, but this is wrong as the next screenshot shows:

CenterofExcellenceCoreComponentsTeams_1_9_managed.zip_import (1).txt
Somehow the CoE Kit and Teams don't like each other in my environments...

Dataverse for Teams environments don't consume capacity from your tenant capacity, instead capacity is allocated based on user licenses in the tenants. If you select Microsoft Teams on the the screenshot you showed above, how many environments and capacity does it say you have available?

It looks like you have 0 capacity and 0 environments allocated to your tenant.
Are you able to import other solutions and is only the CoE one failing? Would you be able to try importing another managed solution and see if that succeeds? As your failure happens before import, we need to confirm if its something in the kit or something in your tenant/product causing the issue.

You can learn more about Teams capacity enforcements here https://docs.microsoft.com/en-us/power-platform/admin/about-teams-environment#enforcement as to me it looks like you are hitting those limits.

Well, as a Microsoft employee I need to show this to partners and customers. I only have access to Demo and Trial environments (e.g. from demos.microsoft.com) so I think I've hit a barrier here which limits us in showing the CoE kit to anyone who would be interested.

Unfortunately that may be true. We have many customers who are now using the CoE in Teams and have been able to import the solution, so I'm afraid the issues you're experiencing are likely not with the solution itself. Sorry we've not been able to help you further!


Method to specify dash in title?

Some ISO Standards have a seemingly 4-part title, where 2 parts are set in the main title element.
However, none of — or — work. Both just display verbatim in the resulting Word doc.
Thoughts?

Yes, this is the Asciidoc document header, which is very intolerant of any markup: anything in there passes straight through as is. The only way to get a non-Ascii character in there is to enter it as raw Unicode, not as an entity. So for dash, you have to enter option-shift-hyphen on the Mac:
Standard—Subtandard.
I'll document.


Use option pattern to configure function blocks

To cleanup the configuration it might make sense to use the option pattern.
https://www.sohamkamani.com/golang/options-pattern/
Example usage:

UploadConfiguration(WithOutputFrittingMask(..), WithOutputWatchdogTimeout(..)) // setting two values explicitly, others default
UploadConfiguration(WithOutputFrittingMask(..), OutputWatchdogMask(..)) // setting two values explicitly, others default
OutputWatchdogMask(WithOutputWatchdogTimeout(..) // setting only one value explicitly, others default
WithOutputWatchdogTimeout() // all values default

https://github.com/ci4rail/io4edge-client-go/blob/a0a653b0d9c2d1aa0b8224ec5278b0204d9dbf76/binaryiotypea/client.go#L68

Great idea!

@siredmar: Like this: https://github.com/ci4rail/io4edge-client-go/blob/functional_option_pattern/binaryiotypea/client.go?

@popples lgtm

@popples However, i would use Configuration instead of fspb.ConfigurationSet as parameter and hide the pb type.

you are right. never mind

Fixed with 7acf976dfb301d7911a492864dac65c4f05ee0fe


Finetuned openai azure models cost calculation

Description:
Add cost calculation for fine tuned Azure with relevant unit tests.
see https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/fine-tuning?tabs=turbo&pivots=programming-language-studio for more information.
this PR is the result of this PR: https://github.com/langchain-ai/langchain/pull/12190
Twitter handle: @nirkopler

@mspronesti This is the fix for finetuned Azure models

thank you @nirkopler @mspronesti!


fix insta regex

pulling in node deps causes pre-commit hook to fail as well.
biome pulled in as node dependency will just not work running nix, because it relies on dynamic linking, which obviously will lead for any system not adhering to the standard FHS to fail.


wait-for-processing fails on pull requests from forks

The addition of the flag to wait-for-processing to complete causes pull requests from forks to fail. We are working on a fix for this internally. This is the public-facing version of the issue and we will close this issue when the internal issue is closed.

Any update on this open issue @aeisenberg ?
cc. @abstractj

There is still some work to do.

Hi @aeisenberg , is there any update on this issue?
We are approximating the results limit:
https://github.com/keycloak/keycloak/runs/6075094447?check_suite_focus=true#step:7:5635
Given the unexpected delay in the server-side implementation of it, would you reconsider merging a PR on the lines of #960 ? The implementation is super straight forward and it won't introduce technical debt.
cc. @abstractj

Apologies for the wait. All of the work is complete. I am just verifying that the changes are all rolled out.
In the meantime, you can try adding back wait-for-processing in your codeql-action/analyze step. I am not sure if it is ready yet. All the bits have been fixed, but just not sure if they are deployed yet.

Thanks, @aeisenberg for the feedback!

This has been fixed for a while now. Forgot to close this issue.