Materialize Select not render when do the page responsive

Following the documentation I wrote the select tags as follows:

          <select name="s1" id="s1" required>
            <option value="" disabled selected>Elige una opción</option>
          </select>
          Subcategoría 1

in my

wrap your code in a document.ready function in jQuery

the code is into document.ready

and the problem continues

Then if you believe there is a bug with the library, please make a codepen illustrating it


liquibase: 3.5.3 -> 3.5.5

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5/bin/liquibase --help got 0 exit code
ran /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5/bin/liquibase help got 0 exit code
found 3.5.5 with grep in /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5

cc "@nequissimus"

Success on x86_64-darwin (full log)
Partial log (click to expand)
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
strip is /nix/store/4sdh09gmvl15cy0zb6i7mbvxh5syz206-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/rz4v22fq3i2h46qclmjn0dx0vd5yq2vf-liquibase-3.5.5/lib  /nix/store/rz4v22fq3i2h46qclmjn0dx0vd5yq2vf-liquibase-3.5.5/bin
patching script interpreter paths in /nix/store/rz4v22fq3i2h46qclmjn0dx0vd5yq2vf-liquibase-3.5.5
/nix/store/rz4v22fq3i2h46qclmjn0dx0vd5yq2vf-liquibase-3.5.5/bin/liquibase: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/pkjmwq7sqrvjg7cjiph6hq0khsmfl6p8-bash-4.4-p12/bin/bash"
/nix/store/rz4v22fq3i2h46qclmjn0dx0vd5yq2vf-liquibase-3.5.5

Failure on aarch64-linux (full log)
Partial log (click to expand)
Package ‘oraclejre-8u161’ in /var/lib/gc-of-borg/nix-test-rs-3/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-aarch64-community-3/pkgs/development/compilers/oraclejdk/jdk-linux-base.nix:71 has an unfree license (‘unfree’), refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
no Makefile, doing nothing
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5/lib  /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5/bin
patching script interpreter paths in /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5
/nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5/bin/liquibase: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/zqh3l3lyw32q1ayb15bnvg9f24j5v2p0-bash-4.4-p12/bin/bash"
checking for references to /tmp/nix-build-liquibase-3.5.5.drv-0 in /nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5...
/nix/store/hk54kvmrnqhyiqqzrdsbrhhqkwcwwzxi-liquibase-3.5.5


Allow bookmark url editing

Depends on https://github.com/amitsingh-007/bypass-links/issues/3166

Not required


Implement Neumorphic

Example: https://dribbble.com/shots/8297803-Skeuomorph-Mobile-Banking-Continuation
Example Use:
// Calculated Variable
View().neumorphic

// Function w/ Param Options
View().neumorphic(lightColor: .white, darkColor: .darkGray, angle: .pi / 4)

typo on neumorhic

Decided not to add after talking with the oneleif design team.


Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api'.

是否考虑将complie引入依赖的方式换成implementation或者api

希望支持implementation或者api


Fix repr issue with show_doc

Solves https://github.com/fastai/nbdev/issues/587 by having type_repr use _format_annos rather than get_name

The other option to consider is just completely change how get_name works and have it be _format_annos.inner
cc @jph00
(Also mark this as Enhancement)

Great!


I wanna dismiss button and click action

Can you tell me how to add  dismiss button and click action?

You can create a button or a anchor whith the suitable code and append it as another child to the wrapper... But this is somehow against the minimalistic approach.

+1 for this feature! Nice minimalistic cross in the right side of the message would boost UX a lot.

Hey guys. Sorry for being silent these last months. I like the idea of adding the close button at the top right of the Notyf. I'll work on this and add it to v3.x.

v3.0.0 has been released with a lot of cool things. I left this for 3.1.0 as I didn't really got time to add this feature, but it's on the roadmap for sure.

Closing as resolved in https://github.com/caroso1222/notyf/pull/51. This feature is now available in v3.3.0.


include option to use Excons retry and timeout features

E.g. to

not spend much time in a blocking request (do not wait for the default 1 minute timeout)
retry a number of times instead of raising on first problem

Implemented in https://github.com/betterplace/easy_compliance/pull/6
Thx for suggesting ;)


LGTM bug creating false alerts: "Module imports itself".

💥 Regression Report
This issue is to track a regression in LGTM and its effects on this skeleton project and all the inheriting child projects.
All __init__.py files are incorrectly being flagged with Module imports itself alerts.
These alerts can safely be ignored for now.  It is expected that the next version of LGTM will correct this behavior in the next couple of weeks.  We can expect these alerts to resolve at that time.
Please close this issue once the LGTM alerts in this project are resolved.
Example:

https://lgtm.com/projects/g/cisagov/skeleton-python-library/alerts/?mode=list

See:

https://github.com/Semmle/ql/issues/2771
https://github.com/Semmle/ql/pull/2802

Last working version
This is not a version-based issue.
Worked up to date: < 2020-02-06 02:44:10
Stopped working date: > 2020-02-06 02:44:10

Closing this since we have removed the LGTM integration from the organization. We have also migrated to CodeQL per #115.


Build aarch64-apple-darwin binaries on CI (take 2)

bors r=matklad

@lnicola You have to change bors.toml too :)

I think it's fine, we don't test it on CI (I've seen waiting times of 40-50 minutes on that platform).
I was confused by bors saying it merged a closed PR from a deleted branch.


anyOf does not work when used in additionalProperties

Description
The combination of anyOf and additionalProperties breaks connexion.
Expected behaviour
Multiple data types should be usable for additionProperties, as far as I can see, anyOf is the way to do this with openapi.
Actual behaviour
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.",\n  "status": 500,\n

Steps to reproduce
I constructed a minimal example. Using this api.yaml:
"openapi": "3.0.0"

info:
    title: Debug
    version: 1.0.0

servers:
  - url: 'http://<IP_ADDRESS>:9191'

paths:
  /debug:
    post:
      operationId: debug.debug
      responses:
        '200':
          description: OK
      requestBody:
        content:
          application/json:
            schema:
              x-body-name: order_list
              $ref: '#/components/schemas/OrderList'
components:
  schemas:
    OrderListConnexionBugWorkaround:
      allOf:
        - $ref: '#/components/schemas/OrderList'
    OrderList:
      type: object
      additionalProperties:
        anyOf:
          - type: number
          - type: string

which is being run by
import connexion

app = connexion.FlaskApp(__name__, specification_dir='.', port=9191)
app.add_api('api.yaml')

app.run()

and this request:
import requests
r = requests.post("http://localhost:9191/debug", json={'A': 1})

The server will return a 500 error code and the console will show:
ERROR in app: Exception on /debug [POST]
Traceback (most recent call last):
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/operations/openapi.py", line 322, in _get_typed_body_values
    prop_defn = body_props[key]
KeyError: 'A'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/decorators/decorator.py", line 68, in wrapper
    response = function(request)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/decorators/uri_parsing.py", line 149, in wrapper
    response = function(request)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/decorators/validation.py", line 193, in wrapper
    response = function(request)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/decorators/parameter.py", line 97, in wrapper
    request.files, arguments, has_kwargs, sanitize)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/operations/abstract.py", line 282, in get_arguments
    has_kwargs, sanitize))
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/operations/openapi.py", line 300, in _get_body_argument
    res = self._get_typed_body_values(body_arg, body_props, additional_props)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/operations/openapi.py", line 329, in _get_typed_body_values
    value = self._get_val_from_param(value, additional_props_defn)
  File "/home/anonymous/anaconda3/lib/python3.7/site-packages/connexion/operations/openapi.py", line 381, in _get_val_from_param
    if query_schema["type"] == "array":
KeyError: 'type'

Additional info:
What really boggles my mind is that the above example runs flawlessly when replacing  $ref: '#/components/schemas/OrderList' with  $ref: '#/components/schemas/OrderListConnexionBugWorkaround', also with more complex data types (I am actually using any of [arrays of strings, arrays of ints, ...]). I stumbled by accident upon that workaround.
Output of the commands:

python --version -> Python 3.7.4
pip show connexion | grep "^Version\:" -> 2.9.0 (has also been tested with 2.8.0)

Closing in favor of #1569.


Update README.md

@jonparrott, any reason this can't be merged?

No, I just forgot to ping @elibixby to do it. Eli, I don't have merge powers.


Fix iOS 13

Add method dataToHexString instead of using description of NSData.
HODL: waiting for confirmation from @gre to remove HODL label.

if that's what was used to gen current mobile bindings then yeah we already have merged that i think :)


fix repeat's output properties is not right

What problem does this PR solve?
Related PR: #37219
Problem Summary:
when child's output is hash distribution, repeat's output follow child only when all child's hash columns in all repeat set. otherwise, repeat's output should be any.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[x] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


F: Design a website for a car manufacturing company named Betta

Must include some basic information about cars and the features

Assign me @VidhiBhatt01 , I would like to work on this

@srinjoy-26 make the pr by tomorrow.

@srinjoy-26 make the pr by tomorrow.

My exams are on , I would surely make a PR on Saturday

@srinjoy-26 make the PR by today.


A question about the calculation of test metrics

Hello! Thank you very much for your work. I have a question while reading the code: why does the code return a Dice coefficient of 1 when there are pixels of that class in the predicted mask but not in the ground truth?
The code is as follows:
elif pred.sum() > 0 and gt.sum() == 0:     return 1, 0
I'm looking forward to your answer, and I really appreciate it!

The test metrics are borrowed from transunet, which measure the metrics in the 3D space but not in the image space. Such situation may not likely to happen when we test the metrics in the 3D space.

Got it, thank you for your answer.


fix(type): 修复 g renderer 的错误类型定义

👀 PR includes

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[x] Type optimization

📝 Description

🖼 Screenshot

Before
After

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.83%. Comparing base (ecdc9c8) to head (85f9eab).
Report is 453 commits behind head on next.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #2939      +/-   ##
==========================================
+ Coverage   75.77%   80.83%   +5.06%
==========================================
  Files         257      180      -77
  Lines       11994    10426    -1568
  Branches     2464     2365      -99
==========================================
- Hits         9088     8428     -660
+ Misses       1398      553     -845
+ Partials     1508     1445      -63

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:tada: This PR is included in version @antv/s2-v2.0.0-next.30 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.29 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create release.yml

Codecov Report

Merging #783 (3fbf7a3) into master (4c5c81b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 3fbf7a3 differs from pull request most recent head bfb41d8. Consider uploading reports for the commit bfb41d8 to get more accurate results

@@           Coverage Diff           @@
##           master     #783   +/-   ##
=======================================
  Coverage   83.81%   83.81%
=======================================
  Files          47       47
  Lines        3262     3262
=======================================
  Hits         2734     2734
  Misses        400      400
  Partials      128      128


add CutPrefixInString and CutSuffixInString functions

add CutPrefixInString and CutSuffixInString functions in basicalter package (check and trim prefix/suffix on the value of a string pointer)

Codecov Report

Merging #18 (d2988eb) into main (2b20974) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #18   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         7    +1
  Lines          137       165   +28
=========================================
+ Hits           137       165   +28

Impacted Files
Coverage Δ

basicalter/string.go
100.00% <100.00%> (ø)

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


General Mode in post filters now as per yaml

This PR fixes #217
In order to see how this affects the value of GeneralMode in the post filters there is a ctest in this branch.  There is no need to commit this ctest as it is not actually testing anything just adding a couple of print statements which need manually checking.
https://github.com/MetOffice/opsinputs/tree/feature/separateGenMode_withctest

Thanks for catching this. I assume your extra ctest works as expected?

Yes I tested it initially and it picked up the final filter (cx) number and used this for both.  I think this is because the code is compiled together and there is only one version of the Gen code and the flag is passed around by modules.  With this change the first filter (varobs) had a different value to the second filter (cx).


Prisoner laptops were showing "No Internet Access" when connecting to WiFi

ServiceNow details:
Prisoner laptops at HMP Garth and HMP The Mount were unable to connect to WiFi using their in cell technology.
When connected to WiFi they receive the error "No internet access". Prisoners affected by this issue will be unable to access any network resources using their In Cell laptops.
It has been reported that approx. 60% of prisoner In Cell laptops are unable to connect to WiFi at site. Other devices are connected without issue


typo in Credentials.swift

there is a typo at Sesssion with 3 's' :
public struct Sesssion: Credentials {}
should be :
public struct Session: Credentials {}

Oh shoot, well this is embarrassing! =D Thanks for noticing! Can't go around breaking people's code, so this will probably stay for the near future -- although I'll work on deprecating it and removing it for 2.0

Maybe renaming it and including a typealias for backward compatibility till version 2 hits?

Yep! It's been a while. Need to catch up on all of this Turnstile stuff. Should have time in ~2 weeks or so!

Fixed in #29


Remove 'okhttp' user agent as it is being used by home-assistant android app

https://github.com/JayBizzle/Crawler-Detect/blob/master/raw/Crawlers.txt#L800
bunkerized-nginx uses these lists, and it blocks access to home-assistant when trying to log in as:
2021/02/16 12:55:07 [warn] 24951#24951: *35 [lua] access_by_lua(main-lua.conf:166):80: [BLOCK] User-Agent okhttp/4.9.0 is blacklisted, client: <IP_ADDRESS>, server: www.example.com, request: "POST /auth/token HTTP/2.0", host: "www.example.com"

Additional information
https://github.com/home-assistant/android/search?q=okhttp&type=code

Thanks for the issue report.
Im 50/50 on what to do here. I understand this is causing an issue in a 3rd party app, but for the intended use here, it certainly seems like it should be classed as a bot visit.
😕

Thanks for the issue report.
Im 50/50 on what to do here. I understand this is causing an issue in a 3rd party app, but for the intended use here, it certainly seems like it should be classed as a bot visit.
confused

I'm trying to convince the home-assistant folks to customize the android app requests to avoid using okhttp and use a custom one instead https://github.com/home-assistant/android/issues/1364 :)

Okay, thanks, i'll keep an eye on it 👍

@JayBizzle we have now customized our user agent string to avoid using the default one provided by okhttp

Good stuff. Thanks for letting us know 👍


Update modified of vocabularies on concept import

Part of #91: If you import a set of concepts, the modified field of their concept scheme should be set to the maximum of each concept's modified value and its current value. Problem: what if the vocabulary information is updated/reimported from file?

What does the modified value of a concept scheme represent? Is it the date that the scheme itself (i.e. its information) was modified, or is it, like you are suggesting, the date of the last modification of a concept inside that scheme? If it's the latter, we could just say that the modified date will not be overridden by a scheme import, only by importing concepts.

The modified value of a concept scheme would refer to the date either its metadata or one of its concepts was last modified, depending which value is higher.

Okay, so maybe we need to add the modified value to the vocabularies in kos-registry and then, when a vocabulary is reimported into jskos-server, take the newer of those two values, right?

Could be implemented now in the import-refactor branch (maybe I should rename it to something else...).

Closing in favor of #120.


Introduce sass

i thought it would be great to use sass in your starter

@michaelknoch Hi! Thank you so much for your PR. Is there a way to make it be an optional thing instead of removing CSS totally? I'm not familiar with sass here, but I use LESS in my project currently. I have nothing against it except I don't know what other people will think about it? :)

i implemented two separate loaders for css and sass, now you are able to import scss or css files depending on your preference ;)

yay ! \o/
Thank you so much!

I use it already, works like charm. Thanks :+1:


Remove unnecessary conflicts, regenerate w/ abi 14

Thanks for the pr, I did some cleanup in addition to these two fixes


Step in only works during the first run when debugging through interactive

Applies To

[ ] Notebooks (.ipynb files)
[x] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
The contents of test.py:
# %%
import numpy

print(numpy.__version__)

Also:

I set up a breakpoint at import numpy in test.py above.
I have "jupyter.debugJustMyCode": false in the settings.
I run VS Code under Windows. I connect to a remote, which has Ubuntu. (Local Extension: Remote - SSH v0.80.0)

First run:

I click "Debug Cell". The debugger pauses in a finally-clause in interactiveshell.py (probably an uncaught exception)
I press F5 (Continue) to step out of interactiveshell.py. The debugger points at import numpy.
I press F11 (Step in) to step into the import statement. As expected, the editor opens __init__.py of the module, and I can trace the execution.
I stop the debugger

Subsequent runs:

[as above]
[as above]
I press F11 (Step in) to step into the import statement. The unexpected happens: instead of stepping into __init__.py, I find the debugger at interactiveshell.py:3457:
exec(code_obj, self.user_global_ns, self.user_ns)

After I click "Restart" in the interactive window, the expected behaviour restores for the following run, after which the bug comes back.
If when stepped in __init.py__ I set a breakpoint there, then this breakpoint works in first runs, but is ignored in subsequent runs.
VS Code Version
Version: 1.67.0-insider (system setup) Commit: a3d53f76790d8c88f835afe85225ff6442703409 Date: 2022-04-07T05:15:43.863Z Electron: 17.3.1 Chromium: 98.0.4758.141 Node.js: 16.13.0 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.19042
Jupyter Extension Version
v1.0.6
Jupyter logs
https://gist.github.com/avm19/201a983fb65ebbf543a2641b0030776d

Coding Language and Runtime Version
Python 3.7.7
Language Extension Version (if applicable)
v2022.5.10971003
Anaconda Version (if applicable)
conda 4.11.0
Running Jupyter locally or remotely?
 There are some fine tuning settings you can apply but they're not exposed as a setting.
See this issue for more information:
https://github.com/microsoft/vscode-jupyter/issues/8146#issuecomment-970926977

@rchiodo Thanks for the reference.
It is hard to believe that this is intended behaviour, because it works differently during the first run and during the subsequent runs. If one behaviour is expected then the other must be not.

Subsequent runs may be entering code differently. Depends upon the implementation of the kernel and whether or not it caches things.

The not stepping into __init__.py sounds more like the design of how packages work. Once a package is loaded into a python process it doesn't run __init__.py again.
This has some more information:
https://docs.python.org/3/tutorial/modules.html#tut-packages
So in this case if you debug a normal python file, you're starting a python process every time. You'd expect to be able to debug __init__.py each time.
If you debug from a kernel, it should only step into that file once, the first time the package is loaded into the kernel.

My bad, thanks for explaining!


Weak containers

Are there plans to offer weak versions of containers, e.g. a WeakMap equivalent of Map?

Unfortunately not since it's not possible to do within JavaScript. WeakMap requires knowledge of the garbage-collection state which isn't available outside of the VM itself.

Not even using the new experimental WeakSet and WeakMap under the covers?

Not that I've been able to figure out. I'm happy to take suggestions on how to do this, but every approach I've considered would either be very inefficient (copy the whole structure on every mutation) or leak memory (use a DAG with WeakMap leafs, but no way to know when to clean up without access to GC)
—
Sent from Mailbox
On Sat, Oct 3, 2015 at 7:41 AM, Paul Jolly<EMAIL_ADDRESS>wrote:

Not even using the new experimental WeakSet and WeakMap under the covers?
Reply to this email directly or view it on GitHub:
https://github.com/facebook/immutable-js/issues/639#issuecomment-145253965

Actually, I'm not sure a copy of a WeakMap is even possible as that would require interation which they do not support as that would leak GC state. So leaking memory would be the only path forward
—
Sent from Mailbox
On Sat, Oct 3, 2015 at 7:41 AM, Paul Jolly<EMAIL_ADDRESS>wrote:

Not even using the new experimental WeakSet and WeakMap under the covers?
Reply to this email directly or view it on GitHub:
https://github.com/facebook/immutable-js/issues/639#issuecomment-145253965

It would be worth investigating how far the current WeakRefs proposal enables immutable Weak Container primitives, and then also to incorporate any feedback for these immutable use cases into that spec process. It seems important that this use case is given adequate consideration in the specification process there.
Having a custom merge function seems like it won't be possible, as conflicts can't be detected for both WeakMap and WeakSet merges, but perhaps the design could factor in that limitation.

(or does WeakRefs get this problem no further forward as well...? Would be good to know either way!)


Cleanup

Cleanup unused imports/code

Sorry! This was meant to be a PR just on our fork, please ignore!


docs: Cookie based Consistent Hashing

Description:
Relates to https://github.com/envoyproxy/gateway/issues/2624

assign myself, docs can be added along with https://github.com/envoyproxy/gateway/issues/2069


adding pysoundfile dependency for librosa

as title

This does indeed fix librosa. @seibert the failures of hpat and fastparquet are unrelated, can you give this a review and merge if good?


How to use index(0,1,2) identifier in taiko?

There are 2 text fields with the same name, id and visual too. There is only one way to identify the object uniquely using index identifier but it is not working with taiko commands.
I used customized xpath.
$x("//[contains(@ng-reflect-name, 'lookBackDays')]")[0]
$x("//[contains(@ng-reflect-name, 'lookBackDays')]")[1]
But it is always focusing on first element.
Expected behavior
It should identify the right object based on index that we provide.
Actual behavior
It is always focusing on first element
Steps to reproduce

Node Version v10.13.0
2.Taiko Version Version: 0.3.0 (Chromium:71.0.3563.0)

Versions
Paste `node` and `taiko` versions here.

Accessing element using index is supported now. Try await text('something').elements() this return all matching elements which can be iterated.


Changed ocaml to serialize vars into QID.

Vars are now fully specified and uniquely named (OCaml):
{
      "rename": [
        [
          {
            "qid": [
              "EvenLoop",
              "loop",
              "i",
              0
            ]
          },
          {
            "qid": [
              "EvenLoop",
              "loop",
              "i",
              1
            ]
          }
        ]
      ]
}

I will merge this in conjunction with an accompanying change to the Haskell side which adds variable aliases.

@mertdumenci and I finished fixing the Ocaml and Haskell side, it should be ready to merge.


chore(ci): nicely formatted files and add a github action workflow to automate formating

Description
This pull request enhances the project's code quality and developer experience by

Improved Code Formatting: Enforces consistent and beautiful code formatting throughout the project. This improves code readability and maintainability for all developers.
Automated Formatting Workflow: Introduces a GitHub Action workflow to automate code formatting. This ensures consistent formatting remains effortless to maintain, saving time and reducing the risk of human error.

By implementing these improvements, this pull request lays the foundation for a well-organized and easy-to-contribute-to codebase.

Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Changes

Please which part of the project your changes affect ?

[x] Frontend (Templates, CSS, JS)
[ ] Backend (Dancer2, api-docs, Dockerfile)

@baimamboukar please resolve the conflicts as I have no clue what to keep and what ti ignore.

Hello 👋
Alright @manwar mentor
I'm on it.

Hello @manwar
The conflicts have been resolved already ✅

Good catch and thanks for your contributions. @baimamboukar


QueryInterceptor does not support async (IDbAsyncQueryProvider)

Hi David,
I noticed that your library does not support async functionality.
I forked your project and added support for IDbAsyncQueryProvider.
Can you please take a look and check if it's correctly implemented ?
If it's correct, can you please add this to your version and publish a new version in nuget ? Or do you want to create a new nuget package like "QueryInterceptorAsync" ?
Kind regards,
Stef

Any updates?


[Feature]:  Controllers enforce naming constraints on CFOrgs and CFSpaces

Blockers/Dependencies
#765
#767
Background
As a k8s user
I want the system to protect against duplicate names
So that I don't break the system by creating invalid CFOrg or CFSpace objects
In this story we will add duplicate name checks using the lease mechanism we use elsewhere.
(Note that space names need only be unique within an org)
Acceptance Criteria
GIVEN I am a cf-admin user with k8s access
WHEN I create two CFOrgs in the cf root namespace having the same name
THEN I see an error
GIVEN I am a cf-admin user with k8s access
WHEN I create two CFOrgs in the cf root namespace having the same name but different cases (e.g. 'foo' and 'FOO')
THEN I see an error
GIVEN I am a cf-admin user with k8s access
WHEN I create two CFSpaces in the namespace having the same name in the same namespace
THEN I see an error
GIVEN I am a cf-admin user with k8s access
WHEN I create two CFSpaces in the same CFOrg having the same name but different cases (e.g. 'bar' and 'BAR')
THEN I see an error
GIVEN I am a cf-admin user with k8s access
WHEN I create two CFSpaces in the namespace having the same name in the different namespaces
THEN I see that both spaces are created
Dev Notes
The name uniqueness webhook already exists for subnamespaceanchors; probably it can just be repurposed to look at CFOrgs & CFSpaces.

A/C passed


Template #1 (default): Adjusting paddings &/or margins

Description
Make formatting tighter with less spacing between content, again to save space.
Tasks

[x] 0. Develop an updated template.
[x] 1. You have a separate branch. A branch where the template has been updated to the "all_at_once" version. Or, a branch where it doesn't replace the original ppp template, but exists as a second, template--either as the new default, or non-default.
[x] 2. You make a pull request from your branch to the develop branch.
[x] 3. I will review, resolve any merge conflicts, merge into develop, and close this issue.
[x] 4. Then you can delete your feature branch if you want: git branch -D [FEATURE_BRANCH], git push origin --delete [FEATURE_BRANCH]
[x] 5. You will git fetch --all and git pull trunk/develop from inside your develop branch so that your develop branch is in sync with trunk.
[x] 6. If you have any other any other currently active feature branches, you can update them to be in sync with this new feature branch by switching to those branches, and doing git rebase develop or git rebase trunk/develop. You may experience some merge conflicts that need resolution. If so, it will be a good learning experience.

No good news :'(
I'll assume that "tighter formatting" == "saving space" == "less pages". Because after some changes it may seem that it's become really "tighter". But if the number of pages remains the same - no, it isn't
Original .xlsx (the one I have for tests) -> .html -> .doc == 34 pages. So it's a starting point.
Reducing font size a bit
1em -> 0.9em
Doesn't work. Same 34 pages.
Word breaking on '_' and other chars
In some cases long strings ('${husband_cohabit_start_recent}') produce extra space in a nearby cells, so it was worth to try...
Doesn't work. Same 34 pages.
Changing table proportions
Original columns' ratio was 50%/50%. But question cells contain a way more content then response cell (just forms), so why not to change it to 70%/30%? I really hoped it would work, and yeah, it seems it really become a little bit tighter (and it was quite a tricky task, for example if a colspan attribute is presented, then all tds' width will be ignored) , but...
Doesn't work. Same 34 page.
Misc.
Changing this and that, here and there...
Doesn't work, doesn't work.
Margins/paddings
Let's go wild:
* {
  margin: 0 !important;
  padding: 0 !important;
}

Doesn't work. But, perhaps, Word just ignore this wildcard. Tested each and every tag separately - and it worked. Hooray! So,
td,
p,
h1,
h2,
h3,
ul,
li,
tr,
em,
et cetera {
  padding: 0 !important;
  margin: 0 !important;
}

and... 27 pages! 20% less!
But I'm not sure it's the case. The main problem is an overall layout.
For example, it might make sense to use not this:
question|response
question|response
table layout, but
question
response
question
response
And yet again, there are these long vertical lists,
[ ] male
[ ] female
I really want to make them float, it will save so much space...
But I really don't want to make them in one line, it will reduce readability so much...
I mean this one is easy to use/submit:
[ ] January
[ ] February
[ ] March
[ ] etc.
and this is just a mess:
[ ] January [ ] February [ ] March [ ] etc.
So I guess readability should be always over the space saving...
In other words, what should I do next?
I guess:

Playing around with padding/margins only, anyway reducing page count by 10-15% will be not a bad result after all with options we have...
Adding to a todo list "think hard about further layout improvements".

Yeah, I think you're right. I don't think there's too much we can do here. Thinking back to the request that was made of us, I think it will be best if we just mess with the padding and font sizes a little bit, but not a lot.
70/30 may be a good idea though, as an alternative to 50/50. But I don't think we should change much more than that.
I think with the padding of 0, it just looks worse. But before this adjustment, there was barely any padding at all.
Can you do the following:

[ ] Produce a 70/30 .doc file example, but with the original padding.

After that, I'll share the following with my co-workers and get some feedback: (a) the original (50/50, default padding), (b) 70/30, 0 padding, and (c) 70/30, default padding.

Files in the zip:
original.doc - original doc with no modifications
original2.doc - actually, original is a little bit "buggy", so columns' ratio is not 50%/50%, but more like 48%/52%, because some content in a response cells is too long and occupy more than 50% of the document (like three inputs in a row), in this version it's fixed
font_reduced.doc - same as original2, but with font size reduced by 10%
padding_reduced.doc - same as original2, but with padding reduced
60_40_ratio.doc - same as original2, but with 60/40 table ratio
70_30_ratio.doc - same as original2, but with 70/30 table ratio
all_at_once.doc - all changes at once
docs.zip

@tulvit Hey Vitaly~.
IMO, I like the "all_at_once" version, which I think is--60/40, font reduced, without padding reduction.
The way I'd like to proceed with this ideally is:

You have a separate branch. A branch where the template has been updated to the "all_at_once" version. Or, a branch where it doesn't replace the original ppp template, but exists as a second, template--either as the new default, or non-default.
You make a pull request from your branch to the develop branch.
I will review, resolve any merge conflicts, and merge into develop.
You will git fetch --all and git pull trunk/develop from inside your develop branch so that your develop branch is in sync with trunk.
If you have any other any other currently active feature branches, you can update them to be in sync with this new feature branch by switching to those branches, and doing git rebase develop or git rebase trunk/develop. You may experience some merge conflicts that need resolution. If so, it will be a good learning experience.

@tulvit Closing this issue, as it and the PR are complete. GJ!


README.md: fix the file based separators internal link

The link to "file based separators" in the "Reading headers from a file" section when rendered to HTML does not work. The link is now fixed by providing a URL and link text, based on the existing link in the "Querystring parameters" section.
The problem

Example of a working link

Thank you, @rudolfolah 🙌


[bug] Unable to Build Tauri App on MacOS at all

Describe the bug
I'm using Tauri v2-rc and on my Macbook Pro M1 2021 (OS version macOS Sequoia 15.1 Dev Beta) with an NPM version of 10.8.2 and Node v22. I created my project using the cargo quickstart commands. The project is Vue.js + TypeScript + Rust + Tauri.
I'm following the v2.tauri.app documentation.
After creating the project, I ran the app using npm run tauri dev, which works fine. Then I tried the following commands:

npm run tauri build --bundles app
sudo npm run tauri build --bundles app
npm run tauri build
sudo npm run tauri build

All of which didn't build the application successfully. My repo is here: https://github.com/Type-32/dnd-utility-app
The logs for all command results above are as the following:
Log 1
(base) ~/Documents/RustRover Projects/dnd-utility-app
npm run tauri build --bundles app

><EMAIL_ADDRESS>tauri
> tauri build app

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
x Build failed in 287ms
error during build:
EACCES: permission denied, unlink '/Users/administrator/Documents/RustRover Projects/dnd-utility-app/dist/assets/index-CSwEWTC4.css'
    at unlinkSync (node:fs:1871:11)
    at _unlinkSync (node:internal/fs/rimraf:213:14)
    at rimrafSync (node:internal/fs/rimraf:194:7)
    at node:internal/fs/rimraf:252:9
    at Array.forEach (<anonymous>)
    at _rmdirSync (node:internal/fs/rimraf:249:7)
    at rimrafSync (node:internal/fs/rimraf:192:7)
    at Object.rmSync (node:fs:1247:10)
    at emptyDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:17081:19)
    at prepareOutDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:65402:7)
    Error beforeBuildCommand `npm run build` failed with exit code 1

Log 2
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
sudo npm run tauri build --bundles app

><EMAIL_ADDRESS>tauri
> tauri build app

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
dist/index.html                  0.48 kB │ gzip:  0.31 kB
dist/assets/index-CSwEWTC4.css   1.76 kB │ gzip:  0.68 kB
dist/assets/index-D8tvxMdj.js   63.03 kB │ gzip: 25.25 kB
✓ built in 327ms
error: unexpected argument 'app' found

Usage: cargo build [OPTIONS]

For more information, try '--help'.
    Error failed to build app: failed to build app

Log 3
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
npm run tauri build

><EMAIL_ADDRESS>tauri
> tauri build

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
x Build failed in 284ms
error during build:
EACCES: permission denied, unlink '/Users/administrator/Documents/RustRover Projects/dnd-utility-app/dist/assets/index-CSwEWTC4.css'
    at unlinkSync (node:fs:1871:11)
    at _unlinkSync (node:internal/fs/rimraf:213:14)
    at rimrafSync (node:internal/fs/rimraf:194:7)
    at node:internal/fs/rimraf:252:9
    at Array.forEach (<anonymous>)
    at _rmdirSync (node:internal/fs/rimraf:249:7)
    at rimrafSync (node:internal/fs/rimraf:192:7)
    at Object.rmSync (node:fs:1247:10)
    at emptyDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:17081:19)
    at prepareOutDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:65402:7)
    Error beforeBuildCommand `npm run build` failed with exit code 1

Log 4
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
sudo npm run tauri build

><EMAIL_ADDRESS>tauri
> tauri build

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
dist/index.html                  0.48 kB │ gzip:  0.31 kB
dist/assets/index-CSwEWTC4.css   1.76 kB │ gzip:  0.68 kB
dist/assets/index-D8tvxMdj.js   63.03 kB │ gzip: 25.25 kB
✓ built in 329ms
   Compiling proc-macro2 v1.0.86
   Compiling libc v0.2.158
   Compiling serde v1.0.210
   Compiling syn v1.0.109
   Compiling getrandom v0.1.16
   Compiling proc-macro-hack v0.5.20+deprecated
error: failed to run custom build command for `proc-macro2 v1.0.86`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro2-f63d1a9f619f407e/build-script-build` (signal: 9, SIGKILL: kill)
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-45c1d8e8e45cf624/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-bacd4364476b8ed6/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-f1a4924a62f72a9e/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `syn v1.0.109`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/syn-d49a4e4ef6da49ec/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-cad60ee301e7f6eb/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `getrandom v0.1.16`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/getrandom-2ef8440fc05af8e9/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `proc-macro-hack v0.5.20+deprecated`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro-hack-31d08b9743722662/build-script-build` (signal: 9, SIGKILL: kill)
    Error failed to build app: failed to build app

Is there anything I missed from the documentation? I have no experience with Rust whatsoever and this is my first time trying Rust through Tauri.
Reproduction
https://github.com/Type-32/dnd-utility-app

Use Cargo Commands to create the project

npm create tauri-app@latest -- --rc
cd tauri-app
npm install
npm run tauri dev

Try to build the app

npm run tauri build --bundles app
sudo npm run tauri build --bundles app
npm run tauri build
sudo npm run tauri build

Expected behavior
There should be a .app file in the build or somewhere around in the project files after building the app.
Full tauri info output
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
npm run tauri info

><EMAIL_ADDRESS>tauri
> tauri info

[✔] Environment
    - OS: Mac OS 15.1.0 arm64 (X64)
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.81.0 (eeb90cda1 2024-09-04)
    ✔ cargo: 1.81.0 (2dbb1af80 2024-08-20)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 22.8.0
    - pnpm: 9.5.0
    - yarn: 1.22.21
    - npm: 10.8.2
    - bun: 1.1.20

[-] Packages
    - tauri 🦀: 2.0.0-rc.10
    - tauri-build 🦀: 2.0.0-rc.9
    - wry 🦀: 0.43.1
    - tao 🦀: 0.30.0
    - tauri-cli 🦀: 2.0.0-rc.11
    - @tauri-apps/api : 2.0.0-rc.4
    - @tauri-apps/cli : 2.0.0-rc.12

[-] Plugins
    - tauri-plugin-shell 🦀: 2.0.0-rc.3
    - @tauri-apps/plugin-shell : 2.0.0-rc.1

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: Vue.js
    - bundler: Vite

Stack trace
Log 1

(base) ~/Documents/RustRover Projects/dnd-utility-app
npm run tauri build --bundles app

><EMAIL_ADDRESS>tauri
> tauri build app

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
x Build failed in 287ms
error during build:
EACCES: permission denied, unlink '/Users/administrator/Documents/RustRover Projects/dnd-utility-app/dist/assets/index-CSwEWTC4.css'
    at unlinkSync (node:fs:1871:11)
    at _unlinkSync (node:internal/fs/rimraf:213:14)
    at rimrafSync (node:internal/fs/rimraf:194:7)
    at node:internal/fs/rimraf:252:9
    at Array.forEach (<anonymous>)
    at _rmdirSync (node:internal/fs/rimraf:249:7)
    at rimrafSync (node:internal/fs/rimraf:192:7)
    at Object.rmSync (node:fs:1247:10)
    at emptyDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:17081:19)
    at prepareOutDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:65402:7)
    Error beforeBuildCommand `npm run build` failed with exit code 1

Log 2
```log
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
sudo npm run tauri build --bundles app

><EMAIL_ADDRESS>tauri
> tauri build app

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
dist/index.html                  0.48 kB │ gzip:  0.31 kB
dist/assets/index-CSwEWTC4.css   1.76 kB │ gzip:  0.68 kB
dist/assets/index-D8tvxMdj.js   63.03 kB │ gzip: 25.25 kB
✓ built in 327ms
error: unexpected argument 'app' found

Usage: cargo build [OPTIONS]

For more information, try '--help'.
    Error failed to build app: failed to build app

Log 3
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
npm run tauri build

><EMAIL_ADDRESS>tauri
> tauri build

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
x Build failed in 284ms
error during build:
EACCES: permission denied, unlink '/Users/administrator/Documents/RustRover Projects/dnd-utility-app/dist/assets/index-CSwEWTC4.css'
    at unlinkSync (node:fs:1871:11)
    at _unlinkSync (node:internal/fs/rimraf:213:14)
    at rimrafSync (node:internal/fs/rimraf:194:7)
    at node:internal/fs/rimraf:252:9
    at Array.forEach (<anonymous>)
    at _rmdirSync (node:internal/fs/rimraf:249:7)
    at rimrafSync (node:internal/fs/rimraf:192:7)
    at Object.rmSync (node:fs:1247:10)
    at emptyDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:17081:19)
    at prepareOutDir (file:///Users/administrator/Documents/RustRover%20Projects/dnd-utility-app/node_modules/vite/dist/node/chunks/dep-BaOMuo4I.js:65402:7)
    Error beforeBuildCommand `npm run build` failed with exit code 1

Log 4
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
sudo npm run tauri build

><EMAIL_ADDRESS>tauri
> tauri build

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
dist/index.html                  0.48 kB │ gzip:  0.31 kB
dist/assets/index-CSwEWTC4.css   1.76 kB │ gzip:  0.68 kB
dist/assets/index-D8tvxMdj.js   63.03 kB │ gzip: 25.25 kB
✓ built in 329ms
   Compiling proc-macro2 v1.0.86
   Compiling libc v0.2.158
   Compiling serde v1.0.210
   Compiling syn v1.0.109
   Compiling getrandom v0.1.16
   Compiling proc-macro-hack v0.5.20+deprecated
error: failed to run custom build command for `proc-macro2 v1.0.86`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro2-f63d1a9f619f407e/build-script-build` (signal: 9, SIGKILL: kill)
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-45c1d8e8e45cf624/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-bacd4364476b8ed6/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-f1a4924a62f72a9e/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `syn v1.0.109`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/syn-d49a4e4ef6da49ec/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-cad60ee301e7f6eb/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `getrandom v0.1.16`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/getrandom-2ef8440fc05af8e9/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `proc-macro-hack v0.5.20+deprecated`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro-hack-31d08b9743722662/build-script-build` (signal: 9, SIGKILL: kill)
    Error failed to build app: failed to build app

### Additional context

_No response_

Log 1 and 3 is your frontend failing, which makes sense since it doesn't have access to /Users/administrator/ without sudo (btw, why is everything in the admin dir?)
Log 2 is missing another -- for npm to forward the bundles arg to tauri: sudo npm run tauri build -- --bundles app
Log 4 i can only guess. Maybe macOS is preventing inner executables to run to protect the admin folder (sounds unlikely), or cargo spawns them without admin permissions making them fail.
I'm not much of a macos user but in my eyes coding projects have no reason to be in the admin acc/dir so i'm inclined to close this as wontfix and ask you to move the project to a normal user (unless of course there's a valid reason i'm not seeing).

My username on mac is "administrator." That's why it seems like everything is under an "administrator" direction, cuz I didn't bother changing the username.

Also, I tried the command sudo npm run tauri build -- --bundles app, and it still doesn't seem to work:
(base) ~/Documents/RustRover Projects/dnd-utility-app git:[dev]
sudo npm run tauri build -- --bundles app

><EMAIL_ADDRESS>tauri
> tauri build --bundles app

    Running beforeBuildCommand `npm run build`

><EMAIL_ADDRESS>build
> vue-tsc --noEmit && vite build

vite v5.4.3 building for production...
✓ 19 modules transformed.
dist/index.html                  0.48 kB │ gzip:  0.31 kB
dist/assets/index-CSwEWTC4.css   1.76 kB │ gzip:  0.68 kB
dist/assets/index-D8tvxMdj.js   63.03 kB │ gzip: 25.25 kB
✓ built in 493ms
   Compiling proc-macro2 v1.0.86
   Compiling libc v0.2.158
   Compiling serde v1.0.210
   Compiling syn v1.0.109
   Compiling getrandom v0.1.16
   Compiling proc-macro-hack v0.5.20+deprecated
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-45c1d8e8e45cf624/build-script-build` (signal: 9, SIGKILL: kill)
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `getrandom v0.1.16`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/getrandom-2ef8440fc05af8e9/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `serde v1.0.210`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/serde-f1a4924a62f72a9e/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-cad60ee301e7f6eb/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `proc-macro2 v1.0.86`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro2-f63d1a9f619f407e/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `proc-macro-hack v0.5.20+deprecated`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/proc-macro-hack-31d08b9743722662/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `libc v0.2.158`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/libc-bacd4364476b8ed6/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to run custom build command for `syn v1.0.109`

Caused by:
  process didn't exit successfully: `/Users/administrator/Documents/RustRover Projects/dnd-utility-app/src-tauri/target/release/build/syn-d49a4e4ef6da49ec/build-script-build` (signal: 9, SIGKILL: kill)
    Error failed to build app: failed to build app

I deleted the dist folder before running the command.

Gonna close this for now, seems clueless.

this is an issue on macOS, apparently: https://github.com/rust-lang/cargo/issues/8913

I got it working using the npm run tauri build -- --debug command.


Problem Compiling

Description of issue:
When I run npm run compile I get an error
Steps to reproduce:

Git clone the repo onto my local computer.
npm install the repo.
npm run compile

Error message:

// Console output:

```~/Documents/Overmind |  (dev) 👾 npm run compile

><EMAIL_ADDRESS>compile /Users/granthummer/Documents/Overmind
> rollup -c

Compiling Overmind... (deploy destination: none)
[!] TypeError: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received type number (0)
TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received type number (0)
    at Object.writeFileSync (fs.js:1380:5)
    at progress (/Users/granthummer/Documents/Overmind/node_modules/rollup-plugin-progress/dist/rollup-plugin-progress.js:25:8)
    at Object.<anonymous> (/Users/granthummer/Documents/Overmind/rollup.config.js:35:9)
    at Module._compile (internal/modules/cjs/loader.js:1200:30)
    at Object.require.extensions.<computed> [as .js] (/Users/granthummer/Documents/Overmind/node_modules/rollup/dist/shared/loadConfigFile.js:516:20)
    at Module.load (internal/modules/cjs/loader.js:1049:32)
    at Function.Module._load (internal/modules/cjs/loader.js:937:14)
    at Module.require (internal/modules/cjs/loader.js:1089:19)
    at require (internal/modules/cjs/helpers.js:73:18)
    at loadConfigFromBundledFile (/Users/granthummer/Documents/Overmind/node_modules/rollup/dist/shared/loadConfigFile.js:524:42)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>compile: `rollup -c`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/granthummer/.npm/_logs/2020-05-24T23_08_00_915Z-debug.log

Suggested fix (optional):

No idea
Other information:

Overmind version: 0.6.0
Latest dev branch

What does your config file look like? (Minus credentials)

How did you get to this point? I tried to install it and run into massive dump of cpp errors...
What is your setup? Node, npm?
Thanks.

same error with Overmind 0.5.2.
npm: 6.14.5
node: v14.5.0
OS: macOS Big Sur (20A5364e)

debug.log:
0 info it worked if it ends with ok
1 verbose cli [
1 verbose cli   '/usr/local/Cellar/node/14.5.0/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'compile'
1 verbose cli ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'precompile', 'compile', 'postcompile' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/mac/Overmind/node_modules/.bin:/usr/local/opt/libiconv/bin:/Users/mac/.pyenv/shims:/Users/mac/.cargo/bin:/opt/local/bin:/opt/local/sbin:/usr/local/opt/mysql-client/bin:/usr/local/opt/curl/bin:/usr/local/opt/curl/bin:/usr/local/sbin:/usr/local/opt/openssl/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin:/Library/Frameworks/Mono.framework/Versions/Current/Commands
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/mac/Overmind
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'rollup -c' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec compile script
13 verbose stack Error<EMAIL_ADDRESS>compile: `rollup -c`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:332:16)
13 verbose stack     at EventEmitter.emit (events.js:314:20)
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:314:20)
13 verbose stack     at maybeClose (internal/child_process.js:1051:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:287:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /Users/mac/Overmind
16 verbose Darwin 20.1.0
17 verbose argv "/usr/local/Cellar/node/14.5.0/bin/node" "/usr/local/bin/npm" "run" "compile"
18 verbose node v14.5.0
19 verbose npm  v6.14.5
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>compile: `rollup -c`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>compile script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

is this the dev branch? this requires rollup 2 to be installed. see https://github.com/bencbartlett/Overmind/blob/dev/CHANGELOG.md

It's the master branch.
<EMAIL_ADDRESS>is installed.

You have to use the compile Script instead of the build Script. Had the same problem ...
Hope that solves your issue

I had the same problem on commit 5eca49a0 and Node 14, once i downgraded to Node 12 though it worked fine.

Same issue here, Node 18, Overmind 0.5.2
PS C:\WoodenRobot\Screeps\Overmind> npm run push-main

<EMAIL_ADDRESS>push-main
rollup -c --environment DEST:main

Compiling Overmind... (deploy destination: main)
[!] TypeError: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received type number (0)
TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received type number (0)
at Object.writeFileSync (node:fs:2314:5)
at progress (C:\WoodenRobot\Screeps\Overmind\node_modules\rollup-plugin-progress\dist\rollup-plugin-progress.js:28:8)
at Object. (C:\WoodenRobot\Screeps\Overmind\rollup.config.js:30:9)
at Module._compile (node:internal/modules/cjs/loader:1256:14)
at require.extensions..js (C:\WoodenRobot\Screeps\Overmind\node_modules\rollup\bin\rollup:3015:24)
at Module.load (node:internal/modules/cjs/loader:1119:32)
at Module._load (node:internal/modules/cjs/loader:960:12)
at Module.require (node:internal/modules/cjs/loader:1143:19)
at require (node:internal/modules/cjs/helpers:121:18)
at C:\WoodenRobot\Screeps\Overmind\node_modules\rollup\bin\rollup:3022:32


LibWeb: Clear active targets before invoking them

Fixes crash on pixiv.net.
Resolve #2011

This actually fixes https://github.com/LadybirdBrowser/ladybird/issues/2475

Closing.


KORE

do not review

Jenkins, test this please.

Jenkins, test this please.

@dwightguth, @andreistefanescu, please review

looks okay although I agree we'll have to iterate over the rewriter interface. I'd like to have an answer to my question about it before we merge though.

I am okay with this being merged now. Should we still wait for @andreistefanescu to review, or try to find someone else to substitute as the second reviewer?

I added @andreistefanescu to get his opinion on the rewriter API, but we'll discuss and change it quite a bit anyhow. So I think we should merge for now.


ModuleNotFoundError: No module named 'bs4'

I'm actually getting this issue now when running updateHostsWindows.bat -a (seemingly out of nowhere as it was working about a month ago). I've located the beautifulsoup files to confirm that they were installed and even tried reinstalling them, to no avail. Could someone link me to the fix that led this to be closed? Couldn't seem to find it anywhere... Thanks!

Hi @funilrys,
My apologies. I forgot to specify that that is how I tried to install them. Every time I try to install the dependencies with that line, it simply returns with the following lines:
Requirement already satisfied: lxml>=4.2.4 in c:\users\ryank\appdata\roaming\python\python37\site-packages (from -r requirements.txt (line 1)) (4.2.5)
Requirement already satisfied: beautifulsoup4==4.6.1 in c:\users\ryank\appdata\roaming\python\python37\site-packages (from -r requirements.txt (line 2)) (4.6.1)

Only after I uninstall them then reinstall them with that line does it say that they are installed, but then running updateHostsWindows.bat -a simply returns with the same error.

Oh, you're on Windows!
Try to uninstall then install without the --user flag :thinking:

That was what I tried first, which also returned the same lines saying that they're installed already 😔

Just tried this script and got that same error. Might want to address this. Possibly a missing step in the instructions.


[KF 1.3.1] Use kubeflow/manifests for pipelines upstream

As described in the title, we used to point pipeline upstream directly at kubeflow/pipelines. The common approach should be use kubeflow/manifests.
Consider picking it up in Kubeflow 1.3.1 release

I feel that it's better we use kubeflow/pipelines manifests directly, to make it easier for users to upgrade/downgrade pipelines.
kubeflow/manifests can just be the reference
Or we could use one as default, but comment another as an option

I feel that it's better we use kubeflow/pipelines manifests directly, to make it easier for users to upgrade/downgrade pipelines.
kubeflow/manifests can just be the reference
Or we could use one as default, but comment another as an option

Closing this PR since we are going to adopt KFP directly from KFP repo.


Issue with ejected Expo Project

Hello,
I am trying to use react-native-version with and ejected expo project. The version in the app.json get incremented but not the one in the Android and in the iOS Project.
It seems that the Expo detection will not check if the project is ejected or not, only if the app.json contains an Expo key.
{
  "expo": {
    "name": "BAC",
    "description": "Description of the app",
    "slug": "exposlug",
    "sdkVersion": "32.0.0",
    "platforms": [
      "ios",
      "android"
    ],
    "version": "1.0.1",
    "orientation": "portrait",
    "icon": "./assets/icon.png",
    "updates": {
      "fallbackToCacheTimeout": 0
    },
    "assetBundlePatterns": [
      "**/*"
    ],
    "android": {
      "publishBundlePath": "android/app/src/main/assets/shell-app.bundle",
      "publishManifestPath": "android/app/src/main/assets/shell-app-manifest.json"
    },
    "ios": {
      "publishBundlePath": "ios/onvabosser/Supporting/shell-app.bundle",
      "publishManifestPath": "ios/onvabosser/Supporting/shell-app-manifest.json"
    },
    "isDetached": true,
    "detach": {
      "iosExpoViewUrl": "https://s3.amazonaws.com/exp-exponent-view-code/ios-v2.10.6-sdk32.0.0-e50ee83a-41bd-4965-b067-4c815a3b3fcc.tar.gz",
      "androidExpoViewUrl": "https://s3.amazonaws.com/exp-exponent-view-code/android-v2.10.8-sdk32.0.0-bca0f957-de78-44c8-91ed-f3151372d79d.tar.gz"
    },
    "scheme": "xxxxx"
  }
}

I believe that checking for the Expo key and for the isDetached could be a way to support Expo detached app and increment the version in the native folders.
Thanks

@jeremybdk Sorry for not replying earlier, I'm pretty busy atm, also in the middle of moving.
Seems like that could work. Would you like to submit a PR? The check whether the app is an Expo app or plain RN is here: https://github.com/stovmascript/react-native-version/blob/89fa937fa4f3eba5246a795bd7b540ca8319a10a/index.js#L69
It should be pretty easy to test. Maybe some other tweaks will be required to update both the Expo config and the RN app, when ejected - if that's what we want.

@stovmascript I did a bit search and I found out there's not a full-proof way to understand if a RN expo project is ejected (or bare flow) or not.
What I think of creating as PR is to have an additional option in CLI to force updating of native files even if expo has been detected. And leave it up to the user to fix his situation.
Or something like --ignore-expo totally.

@killerchip What about removing the expo key from app.json after ejecting? Is it needed for usage with the Expo SDK after ejecting? (Sorry, I don't use Expo.)

@stovmascript I'm using expo but not an expo expert myself.
My tests showed:

Using expo-cli to initiate an expo-bare workflow, it does create the expo key in app.json. Removing it has not issue. The app run fine.

Using the expo-cli to initial an expo-managed workflow, and then eject, cannot compile my app at all. So I could not tested it at atm.

Anyway, the solution I'm proposing is not obtrusive. It's an opt-in and removes the concern of managing edge cases with expo from your package, and gives this flexibility to user.
But it's your call in the end :-)

@stovmascript if the Pullrequest could be accepted it would be awesome!
I'm having the same problems with an ejected expo app, not actually updating the version numbers.

@stovmascript Even if expo is ejected, there are modules that use expo key in app.json. I think it is a good idea to introduce --ignore-expo option.
ex.) expo-updates - npm

Another option would be to include instructions on how to update the Android and iOS build steps to READ the version information from the "app.json" file.
I tried to find some examples of how expo already does this but haven't found it. It looks fairly easy to add this for Android.

I'm actually more inclined to do it the other way around and introduce some --expo option for Expo projects. That way, there wouldn't be a need for any "isExpoProject" logic. On the other hand, it would be a breaking change, but I guess it's not that big of a deal for people to add a flag to their setups.

I would absolutely love an --expo option. As of now, I am still using app.json information even though the project is ejected, so simply deleting does not do the trick.
In the meantime I am just bypassing the problem by adding sth. like this in my package.json scripts for example:
"bump:build": "yarn react-native-version -b && mv app.json app--.json && yarn react-native-version -b && mv app--.json app.json"

for iOS I found this suggested solution

@cjthompson I have tried below for iOS but its not working for me, version and buildNumber doesn't change.
// iOSVersioning.js, placed inside ios folder
/** @format */

const { exec } = require('child_process')

const APP_VERSION = process.env.APP_VERSION
const app = require('../app.json')
const versionName = app.version
const buildNumber = app.ios.buildNumber

if (APP_VERSION === versionName) {
  process.exit()
}

exec(
  `/usr/libexec/PlistBuddy -c "Set :CFBundleShortVersionString ${versionName}" -c "Set :CFBundleVersion ${buildNumber}" "${TARGET_BUILD_DIR}/${INFOPLIST_PATH}"`,
  (err, stdout, stderr) => {
    if (err) {
      console.log('Failed to apply the app version.')
      process.exit(1)
    }

    console.log(`Applied app version: ${versionName}, build number: ${buildNumber}`)
  }
)

and changed my Bundle React Native code and images build phase to:
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"
AppVersion=$(/usr/libexec/PlistBuddy -c "Print CFBundleShortVersionString" "${TARGET_BUILD_DIR}/${INFOPLIST_PATH}")
APP_VERSION="$AppVersion" node ${PROJECT_DIR}/../iOSVersioning.js
export NODE_BINARY=$(which node)
../node_modules/react-native/scripts/react-native-xcode.sh
../node_modules/expo-constants/scripts/get-app-config-ios.sh
../node_modules/expo-updates/scripts/create-manifest-ios.sh

can you share how you configured it for iOS?

Please apply one of these two PR's which would fix the issue:
--ignore-expo cli option by killerchip · Pull Request #150
Adds two CLI options: --is-bare-expo-workflow and --is-self-hosting-bundles by zibs · Pull Request #243

As a workaround, you may convert your app.json to a app.config.js file, which prevents react-native-version from detecting it as a managed Expo project.

I just tried this workaround and got it to work. Minor correction for future searchers. You'll want to convert your app.json into an app.config.json file.


Error when requesting singleton in unit test

cache.ts
import {singleton} from "aurelia-framework";

@singleton
export class CacheManager {
}

cache.spec.ts
import {Container} from "aurelia-framework";
import {CacheManager} from "./cache";

describe("Cache", () => {
	let cacheManager: CacheManager;
	beforeEach(() => {
		cacheManager = new Container().get(CacheManager);
	});

	it("doesn't work", () => {});
});

Error when running Karma:
    Error: Error invoking . Check the inner error for details.
    ------------------------------------------------
    inner error: TypeError: 'undefined' is not an object (evaluating 'target.hasOwnProperty')
        at :731
        at define (:52)
        at :223
        at invoke (:323)
        at invoke (:300)
        at invoke (:564)
        at get (:127)
        at get (:495)
        at :19
        at :75
        at get (:127)
        at get (:495)
        at :19
        at :75
        at invoke (:566)

If I remove the @singleton decorator, it works. Sorry about the sparse trace, I can't seem to get proper file names out of karma.

Of course it might help if I use @singleton() in stead of @singleton. Sorry, never mind! :blush:


Enable testing merge queues @ GitHub Actions CI/CD

This patch allows the project to start using merge queues. They will still have to be enabled in the branch protection settings.
Contributor checklist

[x] Included tests for the changes.
[x] PR title is short, clear, and ready to be included in the user-facing changelog.

Maintainer checklist

[x] Verified one of these labels is present: backwards incompatible, feature, enhancement, deprecation, bug, dependency, docs or skip-changelog as they determine changelog listing.
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

Hey @jezdez, could you go to the repo's branch protection settings, and enable merge queues there, once this PR is merged?

@webknjaz done, using squash commits

@webknjaz done, using squash commits

@jezdez please, keep all the options. Squashing is harmful for history.

@webknjaz It's not a multiple choice config, here are the options:

@jezdez then, use merge. It'll make them octopus. Also, I'd drop that checkbox for "only failing" — sometimes, several failing PRs composed will result in a green CI.

@webknjaz 👍🏻

@jezdez thanks! This will help reduce the need for people to "merge main into PR" all the time since the merge queue will take care of checking that several PRs play well together.

@webknjaz CI seems to have started failing after this pull request. Could you look into this?

That's unrelated. The scheduled nightly runs have a slightly different CI matrix which includes testing against the very last commit in the Pip's repository — that's what's failing.


Can't update settings

On latest release v2.16.0, on macOS Mojave 10.14.3
In settings page（设置）, clicking items under「其他配置」doesn't toggle the checkbox. Saving（保存）works fine. Developer console doesn't show any warning or error.

I test it. Really can't toggle the checkbox. I will solve this problem tomorrow.

@junjiah Already fixed. I don't release a new version. Redownload version 2.16.0 and try it, will works. 😆

confirmed it's fixed


Analytics and other remaining class validations

I have added, Analytics, Setting, Scheduladr APIs Class-validations

Looks good to me.


Fixed prompt not sending

Hi Micheal,
I encountered an issue where the prompt was unable to send and a new line was created instead, I added input_box.submit() to ensure that the prompt is sent instead.
In addition, I added a gitignore file.
Thank you

Hi Shen,
I have just seen the pull request, thank you for the contribution!!!


Update scalafmt-core to 3.0.5

Updates org.scalameta:scalafmt-core from 3.0.4 to 3.0.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch

Codecov Report

Merging #1831 (dc3a297) into master (3371143) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1831      +/-   ##
==========================================
- Coverage   86.02%   85.93%   -0.09%
==========================================
  Files          73       69       -4
  Lines        2526     2482      -44
  Branches      144      135       -9
==========================================
- Hits         2173     2133      -40
+ Misses        353      349       -4

Impacted Files
Coverage Δ

.../core/shared/src/main/scala/io/circe/Decoder.scala
83.37% <0.00%> (-0.26%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
87.71% <0.00%> (ø)

...main/scala-2.13+/io/circe/CollectionDecoders.scala

...rc/main/scala-2.13+/io/circe/LiteralEncoders.scala

...rc/main/scala-2.13+/io/circe/LiteralDecoders.scala

...c/main/scala-2.13+/io/circe/FactoryInstances.scala

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
100.00% <0.00%> (+2.70%)
:arrow_up:

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 3371143...dc3a297. Read the comment docs.


Embedding diagrams does not resize correctly if embedded within a Shadow DOM.

Expected
I expect an embedded diagram to have the correct height to maintain aspect ratio when embedded inside a shadow DOM.
Actual
The diagram has the height of the initial iframe, and does not ensure the correct aspect ratio is maintained.
Steps to reproduce
Rename index.txt to index.html.
Set the following lines appropriately for your test environment.
const root = "http://localhost:8081";
const workspaceId = "1";
const diagramId = "SystemContext-001";

Host index.html in a web-server somewhere (e.g. you could use a docker-compose.yml to host it in Nginx like so:)
services:
  # docker compose up -d --build --remove-orphans --force-recreate
  web:
    image: nginx:alpine
    volumes:
      - "./web:/usr/share/nginx/html/"
    ports:
      - "8080:80"

Then navigate to http://localhost:8080/index.html and you will see the Actual result.
Version/build information
onpremises 2024.06.25
Severity
Minor
Priority
I have no budget and there's no rush, please fix this for free
More information
I am writing a TechDocs plugin for Backstage that will embed a Structurizr diagram into a Backstage TechDocs page.
(This is not possible natively, because TechDocs removes iframes and scripts for security reasons).
TechDocs is a React application, and so renders the document inside a shadow DOM, like so (rename to index.html):
index.txt
However, the structurizr-embed.js script does not support looking in a shadow DOM so it is unable to find the iframe element.
One approach to fixing it is to update the structurizr-embed.js document to search the whole DOM, including shadow roots:
    function getElement(id, root) {
        root = root || document;
        if (root.shadowRoot) {
            const element = root.shadowRoot.getElementById(id);
            if (element) return element;
        }

        for (const child of root.children) {
            const result = getElement(id, child);
            if (result) return result;
        }

        return null;
    }

A full example is here (rename extension to .js).
structurizr-embed.txt
Another approach to fixing it would be to pass the id of the shadow root element as part of the embed URL, so that the script can find the element directly.

Thank you for the comprehensive details on this!

No probs - thanks for the quick response :)


Build and publish docs in CI

Since we do not want to store api docs in the repo (too big, change too often), we should create a nice place to host them.
For the beginning, we can request devs to generate locally, but this is a pain point and barrier to entry, which we want to make as easy as possible for anyone looking to build in the iov ecosytem.
yarn docs in the main package will build them all, we need to think of a hosting strategy, and then link them from the READMEs, so all npm packages have a link to current docs (on master? last tag??)

Not perfectly automated yet but the core of this issue is done. This is how you publish a docs update: https://github.com/iov-one/iov-core/wiki/Publishing


Grading Checklist - Aneesh Deevi

Self Grading Checklist
All self and by peer grading must be complete prior to Teacher review date(s).  Show a direct link to the tangible(s) for each line item.
Sprint1 Key Objectives
During the sprint you need to complete all of these key objectives and find and complete supporting hacks that show level of command you are declaring.
  In this review, pairs should rate each other base on their experience with the person on the line item.
Rank to Ratio, rank is an integer
1 = .55
2 = .75
3 = .85
4 = .90
4+Mastered-Y = .95

Skill
Mastered (Y/N)
Rank (1-4)
Ratio
Notes

Attendance
[Y]
4
0.95

Work Habits
[N]
4
0.9

Behavior
[Y]
4
0.95

Timeliness
[N]
4
0.9

Tech Sense
[N]
4
0.9

Tech Talk
[Y]
4
0.95

Tech Growth
[Y]
4
0.95

Advocacy
[N]
4
0.9

Comm & Collab
[N]
4
0.9

Integrity
[N]
4
0.9

Organized
[Y]
4
0.95

Total

0.0

Average

0.0

Analytics

Looks really good and the work is fully completed.

0.93

Showed great expertise in how everything works (remote theme, js, fullscreen)
Exceeded Hack with full screen snake
Great Issue


Error when creating client with authflow option with password auth on PocketMine-MP

My code
	// @ts-ignore
	const options: ClientOptions = {
		// @ts-ignore
		authflow: new Authflow(
			email,
			".cache",
			{
				password: password,
				flow: "msal"
			},
			null
		),
		username: email,
		authTitle: Titles.MinecraftNintendoSwitch,
		viewDistance: 1,
		connectTimeout: 10_000,
		skipPing: true,
		offline: false,
		host,
		port,
		version
	};
	const client: Client = await createClient(options);
	// @ts-ignore
	client.clientUserChainPayload = {};
	client.on("error", console.error);
	client.on("spawn", () => setInterval(() => client.queue("request_chunk_radius", {chunk_radius:Math.random()*100}), 1000));

Error
[Server thread/INFO]: [NetworkSession: <IP_ADDRESS> 55459] Session opened
[Server thread/ERROR]: [NetworkSession: <IP_ADDRESS> 55459] Bad packet: Error processing LoginPacket: JSON property "titleId" in class "pocketmine\network\mcpe\protocol\types\login\AuthenticationData" must not be NULL
[Server thread/INFO]: [NetworkSession: <IP_ADDRESS> 55459] Session closed: Packet processing error (Error ID: c09b-7257-248e)
[RakLib thread/NOTICE]: Blocked <IP_ADDRESS> for 5 seconds

When doing password based authentication we have to skip the title authentication part of the xbl flow due to insufficient privilege, normally this i ok but some servers may check specifically for title auth which is the case here. To resolve you would have to do device code authentication y removing the password field.

thank you

is there a way to custom implement title auth or is it not recommend?


v3 api 404 error

Version: v3.3.2
api: v3
problem: 404 error page is shown.

Why ?
But http://<IP_ADDRESS>:23790/v3beta/kv/range is accessable.

Hi @peng0208 /v3 will be available in 3.4+ or is now available now in master.
https://github.com/coreos/etcd/blob/master/CHANGELOG-3.3.md#added-grpc-gateway
ref: https://github.com/coreos/etcd/issues/9189


Promtool should warn when there's duplicated lines in the rules files

Accidentally, I found a case where the same rule name was used twice. That went without complaints by the server, and if both rule evaluations had resulted in different label sets, it might have just worked. However, they resulted in the same label set, so the results were funneled into the same time series, and now with our fancy monotonicity detector, we actually see warnings.
Question: Should we in general forbid to use the same rule name twice? Or are their use cases where you want to use the same rule name (but create different label values for the result)?
I tend to simply ban duplicate rule names, but perhaps I'm missing something.
@juliusv @brian-brazil

I'd say the current behaviour is correct.

Found more duplicate rule files at SC... happens quite often, and goes completely undetected so far.
@brian-brazil What's the use case for the current behavior?

As you said in the OP, when you have different label values for each.

Let me rephrase: What would be a use case for creating two rules with the same name that have different label values?

If different teams had per-job rules, those applying to the Standard Exports would be in this situation.

I guess that's a valid point... we have one server per team at SC, so basically one team manages all the rules on one server.
But we could detect if a rule is completely identical to another one. Would that make sense?

We could, but it'd break the abstraction that each rule runs every eval interval. It'll also cause oddities when we have ordered rule evaluation.

I mean, we could simply reject to load the rules file if we have an exact duplicate.

I'm not sure there's much value to that - should we also bail out if there's two rules that are effectively equal too?

It might be difficult to detect “effectively equal” rules.
But on at least three servers at SC, I found rule pairs that funneled into the same time series. One of them was basically a typo, i.e. two different expressions put their results into the same time series (bad). The other two were exact duplicates (copy&paste problem). The related time series had therefore duplicated samples, and with 0.15.0, those samples got rejected, and a counter got incremented on which you probably want to alert. So catching exactly duplicated rules will help to prevent a mistake that will later page you...

I did use this feature (no quotes) to feed different queries / metrics into one uber-metric with different labels (@beorn7 this is on a branch r/n, I can show you). I think we should keep allowing them, and we can't tell beforehand that the labelsets will be different.
I would vote to make a note of this in the documentation but allow people shooting themselves in the foot.

… and if people get paged for this then that's exactly the right point in time to take note of it, because it's the only point when we can be sure it is a problem.

Agreed, this sounds more like an example of bad external rule / config management than one with Prometheus per se. Would also vote to keep the current behavior.

Still, if i have two or more literally (in the literal sense of "literally") identical rules, there is no way that can be meaningful. So should we at least error in that case?

I'd be ok with erroring in that case. I'd say the usefulness is limited (slight decrease in computing resources, but the storage will just reject the second copy of identical samples anyways now), but it couldn't hurt.

I have adjusted the subject.

:+1: to being strict about such issues

This is probably something for a warning in promtool, once we add a general rule linter

How does this play with rule-groups?
Duplicated rule in rule-group would be an error as all rules in the rule-group are evaluated with the same timestamp. Also, duplicated rule across groups can also cause out-of-order appends.

Duplicated rule in rule-group would be an error as all rules in the rule-group are evaluated with the same timestamp. Also, duplicated rule across groups can also cause out-of-order appends.

Not necessarily, each rule could be one iteration of a state machine and you can construct those to avoid duplicate timestamps.

Can we add a duplicate rule name detection into promtool?

Just doing a bug scrub. If this is changed, it would be a breaking change, mandating 3.x. Promtool catching it might be good, though.

Per the above discussion there's no plans for any breaking changes, at most there'll be a warning from promtool.

Closed by #10435


Change license from MIT to Apache 2.0

Hey everyone. I want to relicense future releases from MIT to Apache 2.0. Noting this publicly in a PR to make sure all contributors have a chance to express their opinion. cc @matejklemen @novoselrok @dandxy89 @evanhaldane @nikdon

@inejc please describe your reason for this license switch

@plokhotnyuk I decided for the switch mainly because the Apache license is more dense and hence leaves less room for misinterpretation and because it contains explicit patent language. This might make for an easier industry adoption at some point if the project ever evolves that much. Note that the Apache license is as permissive as the MIT license wrt. the copyright properties so no changes in that sense. I realise that thinking about what I mentioned above at that stage is a bit of an overkill, but given the same amount of permissiveness I decided it's better to relicense now than later. Let me know if that's an issue in your eyes and thanks for the question.


Pass constant objective terms all the way to the solver

Fixes #111 (for the MOI interface). Requires https://github.com/JuliaOpt/LinQuadOptInterface.jl/pull/44
This currently contains a really gross workaround for the issues discussed in https://github.com/JuliaOpt/LinQuadOptInterface.jl/pull/44#issuecomment-409373755 . I'm happy to wait until we have a better solution for that before merging; I just wanted to get this PR opened so I won't forget about the changes.

x-ref: https://groups.google.com/forum/#!topic/gurobi/p2uJgx8Hn7Q

Even though this will break master (because of the LQOI bound in REQUIRE), I'm going to merge this because there are a few PR's backing up and I want to test them all in anticipation of the upcoming tags of MOI and LQOI.


yfinance 0.2.1: quarterly_balancesheet missing  'Total Stockholder Equity' attribute

df_quarterly_balancesheet.loc['Total Stockholder Equity']

KeyError: 'Total Stockholder Equity'

Does Yahoo actually have the data?

yes - just verified on another setup that uses 0.1.93 in which the ticker_data.quarterly_balancesheet has different attributes than the one in 0.2.1 .
0.1.93 has:
Index(['Intangible Assets', 'Total Liab', 'Total Stockholder Equity',
'Other Current Liab', 'Total Assets', 'Common Stock',
'Other Current Assets', 'Retained Earnings', 'Other Liab', 'Good Will',
'Gains Losses Not Affecting Retained Earnings', 'Other Assets', 'Cash',
'Total Current Liabilities', 'Short Long Term Debt',
'Other Stockholder Equity', 'Property Plant Equipment',
'Total Current Assets', 'Long Term Investments', 'Net Tangible Assets',
'Short Term Investments', 'Net Receivables', 'Long Term Debt',
'Inventory', 'Accounts Payable', 'Deferred Long Term Asset Charges'],
dtype='object')
0.2.1 has:
Index(['Total Assets', 'Current Assets',
'Cash Cash Equivalents And Short Term Investments',
'Cash And Cash Equivalents', 'Cash Financial', 'Cash Equivalents',
'Other Short Term Investments', 'Receivables', 'Accounts Receivable',
'Gross Accounts Receivable',
'Allowance For Doubtful Accounts Receivable', 'Inventory',
'Raw Materials', 'Work In Process', 'Finished Goods',
'Hedging Assets Current', 'Other Current Assets',
'Total Non Current Assets', 'Net PPE', 'Gross PPE', 'Properties',
'Land And Improvements', 'Buildings And Improvements',
'Machinery Furniture Equipment', 'Other Properties', 'Leases',
'Accumulated Depreciation', 'Goodwill And Other Intangible Assets',
'Goodwill', 'Other Intangible Assets', 'Investments And Advances',
'Long Term Equity Investment', 'Other Non Current Assets',
'Total Liabilities Net Minority Interest', 'Current Liabilities',
'Payables And Accrued Expenses', 'Payables', 'Accounts Payable',
'Total Tax Payable', 'Income Tax Payable',
'Pensionand Other Post Retirement Benefit Plans Current',
'Current Debt And Capital Lease Obligation', 'Current Debt',
'Current Deferred Liabilities', 'Current Deferred Revenue',
'Other Current Liabilities',
'Total Non Current Liabilities Net Minority Interest',
'Long Term Debt And Capital Lease Obligation', 'Long Term Debt',
'Long Term Capital Lease Obligation',
'Non Current Deferred Liabilities',
'Non Current Deferred Taxes Liabilities',
'Non Current Deferred Revenue', 'Tradeand Other Payables Non Current',
'Other Non Current Liabilities', 'Total Equity Gross Minority Interest',
'Stockholders Equity', 'Capital Stock', 'Common Stock',
'Retained Earnings', 'Gains Losses Not Affecting Retained Earnings',
'Total Capitalization', 'Common Stock Equity',
'Capital Lease Obligations', 'Net Tangible Assets', 'Working Capital',
'Invested Capital', 'Tangible Book Value', 'Total Debt', 'Net Debt',
'Share Issued', 'Ordinary Shares Number'],
dtype='object')

OK, I think figured it out, it moved from the 'quarterly_financials' to 'quarterly_balancesheet', and the attribute name was changed from 'Total Stockholder Equity' to 'Stockholder Equity'

There are a few others attributes that have changed on the move to 0.2.1, such as:
'Cash' ->  'Cash And Cash Equivalents'
'Total Liab' -> 'Total Liabilities Net Minority Interest'
'Short Term Investments'-> 'Other Short Term Investments'

Additional fields with slight attribute changes:
quarterly_cashflow.loc['Capital Expenditures']  ==> 'Capital Expenditure'
quarterly_cashflow.loc['Total Cash From Operating Activities'] ==> 'Cash Flow From Continuing Operating Activities'

Would this be better in a Wiki page? Anyone can edit that now.

good idea.
Can you open a topic with a link here,  and I and possibly others can update there as we come across changes?

Can I just link? I want Wiki to be a community-effort. https://github.com/ranaroussi/yfinance/wiki

I added a page (seemed like the first page in the wiki). Pls double check it looks in the spirit you wanted. thnx.

I just changed title. Content spirit is really for users to decide.

updated the wiki with additional fields that were relocated in release 0.2.7

The direct link to the wiki page is:
https://github.com/ranaroussi/yfinance/wiki/Financial-tables---attribute-changes-in-yfinance-0.2
@ValueRaider With the motivation to reduce usage of Ticker.info, should we consider moving also the following attribute from Ticker.info to Ticker.fast_info (or will it make it 'too long')?:
'beta', 'longName', 'sector', 'industry', 'heldPercentInstitutions', 'heldPercentInsiders', 'dividendYield', 'exDividendDate', 'trailingPE', 'forwardPE', 'pegRatio', 'priceToBook', 'enterpriseToEbitda', 'profitMargins', 'SandP52WeekChange'
All of them are 'flat data' (one value attributes with no 'history values'),  that seem to fit well with the data in fast_info, and may align well with the motivation to minimize the usage of (that is, if you don't think it will make fast_info too long).
currently users that do look for these fields need to make two queries, one for Ticker.info and one for Ticker.fast_info.

The purpose of fast_info is to be fast, faster than fetching info. E.g. history() is much faster than info, so use that for price data. It's about alternative data sources, not length.
I honestly don't know how to make this clearer.

@galashour I'm drafting an explanatory paragraph at top of README, does this clarify everything?
https://github.com/ranaroussi/yfinance/blob/doc/readme-explain-instability/README.md

Can this be closed?

yup. thnx


Add support for tagging AWS resources

I've been using Terratest in anger recently and have spotted a few things about the library.
Part of a pipeline that is building an AMI, I've found it quite common to add a tag to an AMI once it has passed testing. Adding a method to add tags to an AMI, along with other AWS resources, would be useful.

Could you use snapshot_tags in Packer?

The idea is to allow the adding a tag to mark the AMI as tested only after the AMI has passed testing although, after learning today that AMI tags aren't visible across accounts, the value is somewhat diminished.

Ah, I gotcha. An interesting idea and I'd be open to a PR for that assuming you haven't lost all enthusiasm for it :)


Module support for uvx

Currently uvx or uv tool only seems to support packages that provide binaries. However, many packages haven't properly published executables within them and you often need to call python -m package on these to run them.

A notable example is setuptools_scm or grpc_tools.protoc

It would be great if we could do something like:

uvx setuptools-scm -m setuptools_scm
instead of:
uv pip install setuptools-scm && python -m setuptools_scm

uvx grpcio-tools -m grpc_tools.protoc -I. --python_out=. example.proto
instead of:
uv pip install grpcio-tools && python -m grpc_tools.protoc -I. --python_out=. example.proto

Both of the above then pollute the env.

Context:
I've looked at uv run but this doesn't seem to be applicable because this is not for installed modules, rather it is for local scripts, and it also interacts with the current pyproject.toml (it seems to want to install things even if they are listed as extras, which may also be a bug?)
I specifically don't want to install the package I am running in the current env, e.g. I need to build a project using some deps, like running grpc compilation. But I don't want the extra grpc installed deps to conflict with later builds steps, e.g. PyInstaller.

uvx --from setuptools-scm python -m setuptools_scm is available today, but that is just a sidenote.

@bluss
Thank you for this

It seems sort of awkward to support this, I think it'd have to be uvx -m setuptools_scm setuptools so the arguments aren't ambiguous? I'm not sure if that offers much over the more verbose version.

@zanieb I think you are right. The syntax I proposed could conflict with arguments for the tools themselves. Unless order is respected. So this maybe isn't as intuitive.
Maybe the solution is just an addition to the docs with an example of @bluss's answer?
https://docs.astral.sh/uv/guides/tools/#installing-tools

Same issue as with uv run for the module argument, everything that follows -m is an external command, should not be parsed as arguments to uvx.