ARM64 Node has run away agetty process

What happened:
Customer created ARM64 cluster and node has agetty process consuming 100% CPU
What you expected to happen:
agetty not consuing 100% CPU
How to reproduce it (as minimally and precisely as possible):
Create cluster on ARM64
Anything else we need to know?:
Environment:

EKS Anywhere Release: v0.7.0
EKS Distro Release: v.1.21

https://github.com/aws/eks-anywhere/pull/1158#issuecomment-1030712948
https://user-images.githubusercontent.com/3286998/152662193-f155001e-490b-45d9-8c24-957a48d3963f.png

I am going to take a look at this one. Looks like a known issue with older versions of systemd/getty which almost def effects al2 which our kind image is based on.  Im suspicious its not an arm specific issue, but ill report back.

Yep I can't say for sure that it's an ARM64 specific issue, I was running on ARM64 and did not want to omit that from the report because it seems light on support. (The vast majority of things were working, but Homebrew itself isn't supported on arm64 linux and I had to compile it myself to get it working.)
This is Linux-on-ARM64 on bare metal, (not Linux via Docker Desktop on an M1 mac, in case that makes a difference too.)

@kingdonb totally thanks for the amazing details!
arm64 is something we "added" recently, but do not currently have e2e tests for it so have held off on officially "supporting" it.  This is why we havent published to the arm builds for linux or mac to homebrew or added anything to the docs.  Hearing you go through and replicate success is awesome to hear and adds motivation to prioritizing finalizing our arm support, at least for capd and bootstrap clusters.
For this specific issue, I was able to replicate on my amd64 mac and have a fix coming soon.  Ill expand on the details in my PR.

I think the work is done here, PR merged, available in next release.

Amazing, 🎉  I gave it a shot and this is much better:

This machine is running 10 capd workload clusters and a management cluster. Looks like there's room for at least another 40 before the control planes would start to overwhelm the node. It's 128GB of RAM with 32 cores, for reference.


nvm is confused by symlinks

I cloned the nvm repository into a directory, then sourced nvm.sh via a symbolic link to that directory.
This causes nvm to error out due to incorrect "contains path" logic:
$ nvm use v5.1.0
nvm is not compatible with the npm config "prefix" option: currently set to "/opt/src/nvm/versions/node/v5.1.0"
Run `npm config delete prefix` or `nvm use --delete-prefix v5.1.0` to unset it.

Here's one possible fix:
diff --git a/nvm.sh b/nvm.sh
index b4e871f..7cd1e1d 100755
--- a/nvm.sh
+++ b/nvm.sh
@@ -98,9 +98,9 @@ fi

 nvm_tree_contains_path() {
   local tree
-  tree="$1"
+  tree="$(readlink -f "$1")"
   local node_path
-  node_path="$2"
+  node_path="$(readlink -f "$2")"

   if [ "@$tree@" = "@@" ] || [ "@$node_path@" = "@@" ]; then
     >&2 echo "both the tree and the node path are required"

This is a duplicate of #617 - for now, $NVM_DIR can not be a symlink.
I'm relatively sure readlink in that fashion is not POSIX - meaning, it won't work on all of bash, zsh, dash, sh, and ksh. PRs welcome if you can convince me that it is!


Path validation abandon

Hi! I am working on a scenario that makes use of the path validation. I have the HTTP server running on a remote host and the HTTP client running on my laptop, which is connected to the internet via WiFi. Server and client are the ones provided in the examples directory. I made a little modification to the http3_client.py: I made the client performing 10 HTTP requests to the padding endpoint within the same QUIC connection, so that I can simulate a handover in between two consecutive requests.
Once the handover is done, I expect the server to initiate path validation upon the reception of a non-probing frame from the new client address. The server promotes the new address as primary path and sends the path challenge frame to the client. Now, what happens if either the path challenge or the path response frame is lost?
RFC 9000 specifies that "Endpoints SHOULD abandon path validation based on a timer". Is this implemented in the current version of aioquic? Because it seems that if one of the two frames is lost, the endpoint abandons immediately the path validation procedure. Is this right?

hello @francescobarbarulo
i have the same problem, did you find a solution?


MAINT: spatial: standard infinity handling in ckdtree

Use NumPy's NPY_INFINITY and npy_isinf instead of a custom global variable. Should fix #5281.

No objections for a week and not expected to be controversial; merging.

@larsmans See https://github.com/scipy/scipy/issues/5311

This code is correct but it tickles a bug in the g++ executable shipped with Xcode.


Implement A/B tests for showcase page

Implement A/B tests for hypotheses defined in https://github.com/datahq/frontend/issues/283
Acceptance criteria

[x] A/B tests are setup and working for hypotheses

Tasks

[x] draft experiments in google optimise
[x] setup objectives
[x] implement required changes to variants (eg, html, css implementation of a new version)
[x] start experiments

FIXED - experiments started on 28 May 2018


Fixed some causes for exception for not being able to download certain videos

Formatting fixed, few extra checks added
removed some tokens from song title which raises exceptions for opening a file with that name, dislikes, subscriber count etc maybe zero, hence the key may not be present in metadata

Example: This video doesn't have any subscribeText key hence it failts

@creepysta  Thanks for reminding me. I have fixed that two days ago on my local system. But forgot to apply the changes. Appreciate your efforts


babelfish: 1.0.0 -> 1.0.1

Motivation for this change
Packaging a new version that fixes a bug critical to https://github.com/NixOS/nixpkgs/pull/108947
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110334 run on x86_64-darwin 1

1 package built:

babelfish

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110334 run on x86_64-darwin 1

1 package built:

babelfish


NIFI-1868: Add PutHiveStreaming processor

The second commit is to (temporarily) remove ConvertAvroToORC (and all ORC references). The Hive processors (for 1.0) must work with Hive 1.2.1, which was before ORC was split into its own Apache project. In order for PutHiveStreaming (and the rest of the bundle) to compile against Hive 1.2.1, the Hive version was downgraded and all ORC references were removed.
NIFI-1663 was reopened to refactor ConvertAvroToORC to use hive-orc in Hive 1.2.1 rather than Apache ORC. That Jira will restore the ConvertAvroToORC processor.

Per an offline discussion with @bbende, we decided a good way to handle the outgoing flow files is to have a "success", "failure", and "retry" relationship. If a connection / environment error occurs, the incoming flow file will be routed to retry. Otherwise, all records put successfully to Hive Streaming will go into an Avro record on the "success" relationship, and any records failed to be written will go into an Avro record on the "failure" relationship.

Latest update is looking good... one thing I noticed, if you send in an Avro file that does not have the partition columns of the table, it throws an IOException around line 435 when trying to extract the partition fields from the Avro schema, but then it gets wrapped in a ProcessException and thrown out of onTrigger so the flow file sits in the incoming queue but can never be processed.
Could we look for ProcessException with a cause of IOException and route to failure (similar to the connection error handling)? or maybe create a specific exception type to look for since there could be other IOExceptions that we want to bounce out of onTrigger?

Yes that's a good idea, will update, thanks!

Error handling on the relationships is looking good, thanks for making those updates.
One more thing I ran into, when I set concurrent tasks to 2 on the PutHiveStreamingProcessor and I was generating a flow file every 500 ms, I would occasionally get the following error:
2016-08-02 20:50:17,860 ERROR [Timer-Driven Process Thread-1] o.a.n.processors.hive.PutHiveStreaming
java.lang.NullPointerException: null
	at org.apache.hive.hcatalog.streaming.StrictJsonWriter.write(StrictJsonWriter.java:79) ~[na:na]
	at org.apache.hive.hcatalog.streaming.HiveEndPoint$TransactionBatchImpl.write(HiveEndPoint.java:632) ~[na:na]
	at org.apache.nifi.util.hive.HiveWriter$1.call(HiveWriter.java:113) ~[na:na]
	at org.apache.nifi.util.hive.HiveWriter$1.call(HiveWriter.java:110) ~[na:na]
	at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[na:1.8.0_102]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [na:1.8.0_102]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [na:1.8.0_102]
	at java.lang.Thread.run(Thread.java:745) [na:1.8.0_102]

Yeah I think that's a bug in this version of Hive: https://issues.apache.org/jira/browse/HIVE-13725
For now I'd like to make the processor TriggerSerially (unless there are any objections), then when we upgrade the version of Hive we can revisit and hopefully remove this limitation.

TriggerSerially sounds fair knowing that we should be able to remove it in the future

Tested out the trigger serially and appears to be working as expected. I am a +1 to merge this in, do you want to squash the commits first?


Windows on ARM, Support?

Please let us know when can we have an ARM64 version for Windows on ARM OS.

Never, as this project is EOL and shouldn't be used anyway.


Expose rpc service for remote backup and restore

Signed-off-by: Naseem Ullah<EMAIL_ADDRESS>

Hi @jackzampolin, can you please take a look?

utACK, looks great! cc @timhallinflux


2024.12.0 Error Upgrade Firmware with HASSIO Esphome

INFO ESPHome 2024.12.0
INFO Reading configuration /config/esphome/m5stack-atom-echo-9f9bec.yaml...
INFO Updating https://github.com/esphome/esphome.git@pull/5230/head
INFO Updating https://github.com/jesserockz/esphome-components.git@None
INFO Generating C++ source...
INFO Updating<EMAIL_ADDRESS>INFO Updating<EMAIL_ADDRESS>INFO Updating submodules (components/esp-sr, components/esp-adf-libs) for<EMAIL_ADDRESS>INFO Updating<EMAIL_ADDRESS>INFO Updating<EMAIL_ADDRESS>INFO Updating<EMAIL_ADDRESS>INFO Compiling app...

-- Building ESP-IDF components for target esp32
Processing 1 dependencies:
[1/1] idf (5.1.5)
-- Configuring incomplete, errors occurred!
See also "/data/build/m5stack-atom-echo-9f9bec/.pioenvs/m5stack-atom-echo-9f9bec/CMakeFiles/CMakeOutput.log".

fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
CMake Error at /data/cache/platformio/packages/framework-espidf/tools/cmake/build.cmake:266 (message):
Failed to resolve component 'jsmn'.
Call Stack (most recent call first):
/data/cache/platformio/packages/framework-espidf/tools/cmake/build.cmake:308 (__build_resolve_and_add_req)
/data/cache/platformio/packages/framework-espidf/tools/cmake/build.cmake:603 (__build_expand_requirements)
/data/cache/platformio/packages/framework-espidf/tools/cmake/project.cmake:604 (idf_build_process)
CMakeLists.txt:3 (project)
========================== [FAILED] Took 3.99 seconds ==========================

adding this in esp32 has solved the issue (loads of warrning in the build still though)
  framework:
    type: esp-idf
    version: 4.4.8
    platform_version: 5.4.0

adding this in esp32 has solved the issue (loads of warrning in the build still though)
  framework:
    type: esp-idf
    version: 4.4.8
    platform_version: 5.4.0

Confirmed with my install.

where to add?

where to add?

Add to the esp32: section

Still an issue in 12.2 compiler and happens with the esp32-s3-box-3, didn't try on the atom echo but assume the same.  Weird how can flash fine from the web-flasher and then can't install an OTA update via HA?  These things are finicky for sure, maybe one day it'll all get figured out nicely.

Still an issue in 12.2 compiler with atom echo


Test commit

DO NOT MERGE, created only to test the repository configuration.

Closing the test PR: All tests are correctly marked as required and the merge queue is active.


Feature request: form to submit zip code

This app appears to only geolocate based on the user's IP address. That is great for the weather where you are located. However, sometimes you're traveling to another location and want to know the weather there ("I'm going to XZY tomorrow, do I need to bring a jacket/umbrella?"). So a simple one-field form to submit zip code (or city name, or both, whatever's easy) to get the weather there.

Another example is when the geolocation fails. In my case an error displays that I'm outside of the NWS region, which is not the case.


I cannot stop webspeaker manually

Hi,
Can you please help me to stop webspeaker manually
Thanks

is it working properly


Configure TrackEvent in UI

It would be helpful to be able to configure one or more TrackEvent data sources in the Perfetto UI.  Ideally the UI would have the option to query the current set of Producers to get a list of available processes, categories, and tags, and an easy way to filter accordingly.
Currently to use TrackEvent data sources, the trace must be executed outside of the UI in order to supply a handwritten TraceConfig.

Agree, this should definitely be on our radar :) Technically, the querying described by @rickvanprim should be possible after @octaviantu's improvements. We can already query the track event data source descriptor with available categories from the service. We'd probably want to extend it such that it also specifies available tags.

Would be great having the ability to configure track events from the UI. Currently we are forced to write the config separately and invoke from the terminal.

@primiano maybe you can fix this as part of your look again into recording?


Software launcher

The current software management solution is not very aesthetically pleasing and can be optimised by using similar software as in the example below
Website:
https://dawnlauncher.com/
https://github.com/fanchenio/Dawn-Launcher
Advantages: Can create subcategories (this allows for better classification)

Note: The international UI needs to be improved

I don't think that's open source, kind of defeats the purpose , doesn't it ?

So we're looking for similar software

The software is open source and supports English

Thanks for that! Will definitely take a look! ☺

I'll wait for an English translation to be released. Thanks, @fairycn! Will keep it open so I don't forget about it. :)

https://github.com/fanchenio/DawnLauncher
Updated and added English language UI

https://github.com/fanchenio/DawnLauncher
Update 1.3.5
Updated and added English language UI
@merces


WaddleParser tests

Codecov Report
Base: 66.84% // Head: 70.38% // Increases project coverage by +3.53% :tada:

Coverage data is based on head (92e624c) compared to base (f0d8946).
Patch coverage: 33.33% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #243      +/-   ##
============================================
+ Coverage     66.84%   70.38%   +3.53%
- Complexity      660      710      +50
============================================
  Files           111      111
  Lines          2371     2374       +3
  Branches        280      280
============================================
+ Hits           1585     1671      +86
+ Misses          676      602      -74
+ Partials        110      101       -9

Impacted Files
Coverage Δ

.../java/seedu/waddle/logic/commands/CopyCommand.java
8.33% <0.00%> (+8.33%)
:arrow_up:

.../java/seedu/waddle/logic/commands/FreeCommand.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...n/java/seedu/waddle/logic/parser/WaddleParser.java
89.74% <ø> (+43.58%)
:arrow_up:

...seedu/waddle/logic/parser/UnplanCommandParser.java
80.00% <100.00%> (+80.00%)
:arrow_up:

src/main/java/seedu/waddle/model/item/Day.java
85.55% <0.00%> (+2.22%)
:arrow_up:

...n/java/seedu/waddle/model/itinerary/Itinerary.java
65.69% <0.00%> (+4.65%)
:arrow_up:

src/main/java/seedu/waddle/model/item/Cost.java
77.77% <0.00%> (+11.11%)
:arrow_up:

...n/java/seedu/waddle/logic/commands/PdfCommand.java
13.33% <0.00%> (+13.33%)
:arrow_up:

...va/seedu/waddle/commons/core/index/MultiIndex.java
58.82% <0.00%> (+17.64%)
:arrow_up:

... and 5 more

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

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


Alternatives are very slow for LM/AStar+turn costs

map: bayern-210101.osm.pbf:
route using maps 2 client: http://localhost:3000/?point=49.583276%2C10.991626&point=48.455783%2C12.095752&profile=car
profile: fastest/car, u_turn_costs: 60

traversal mode
single route
alternatives (max=3)

node-based
13ms
~800ms

edge-based
25ms
~9000ms

So alternatives take 80 times longer than a single route for node-based. This seems very slow, but at least it still takes less than a second. But for edge-based it is even worse where the slow down is x360 and the response takes so long that its almost not useful anymore.
This is not just special for this route, I just gave an example. I actually saw routes that took tens of seconds, and I did not even try a larger map. Disabling LM makes it even slower.

Using the approach taken by @michaz in #2092 the query time for edge-based alternative seems to go down from ~9s to ~5s. However, for node-based the response becomes (slightly) slower, like ~950ms and also the single routes for edge-based seem to take longer (~40ms vs. ~25ms), but this was just a quick manual check.

@otbutz in #2567  you wrote:

I can vouch for Java Mission Control paired with -XX:+UnlockDiagnosticVMOptions -XX:+DebugNonSafepoints for getting a good grasp about where the bottleneck is :)

I am answering here as this is about speeding up alternative routes:
Thanks I will try. I am using the YourKit Profiler, usually. But in my experience this does not help much and the real issues are typically caused by the algorithms (i.e. normally something is called too often rather than some method is slow). In the case of AlternativeRoute I am pretty sure this takes very long, especially for large search trees:
https://github.com/graphhopper/graphhopper/blob/47febfc6278e6eebf15f2940e5bac0c77c0034b5/core/src/main/java/com/graphhopper/routing/AlternativeRoute.java#L321-L326
But I did not really look at the alternative route algorithm to figure out if this can be avoided without making the quality of alternative routes worse.

normally something is called too often rather than some method is slow

The flame view is immensely helpful IMHO.
https://www.youtube.com/watch?v=vVh_z34Gtn8

Yes, I like the flame view, too. YourKit has it as well. But often I saw cases where something looks like a bottleneck there, but in reality it wasn't. Overall I feel like I spent a lot of time looking at the profiler, but rarely it helped me finding important bottlenecks and a lot more often improving the algorithm lead to significant speed ups. But maybe I'm still just doing it wrong.
In other news: This: https://github.com/graphhopper/graphhopper/commit/19e2e16bfa9ff663846f99113bd17d23b12be487 is definitely an improvement, especially for edge-based routing without CH. And in this case the PriorityQueue#remove call actually shows up in the profiler, too :) I'll open a separate PR for this soon.


Google Chrome & Edge print_pdf not working

Describe the bug
We need to check if this only happens with Selenium 3 or also with Selenium 4.
s = b'{"value":{"error":"unknown error","message":"unknown error: PrintToPDF is only supported in headless mode\\n  (Sessi...7ff8080904e1 _pthread_start + 125\\n19  libsystem_pthread.dylib             0x00007ff80808bf6b thread_start + 15\\n"}}'
altchars = None, validate = False

    def b64decode(s, altchars=None, validate=False):
        """Decode the Base64 encoded bytes-like object or ASCII string s.

        Optional altchars must be a bytes-like object or ASCII string of length 2
        which specifies the alternative alphabet used instead of the '+' and '/'
        characters.

        The result is returned as a bytes object.  A binascii.Error is raised if
        s is incorrectly padded.

        If validate is False (the default), characters that are neither in the
        normal base-64 alphabet nor the alternative alphabet are discarded prior
        to the padding check.  If validate is True, these non-alphabet characters
        in the input result in a binascii.Error.
        """
        s = _bytes_from_decode_data(s)
        if altchars is not None:
            altchars = _bytes_from_decode_data(altchars)
            assert len(altchars) == 2, repr(altchars)
            s = s.translate(bytes.maketrans(altchars, b'+/'))
        if validate and not re.fullmatch(b'[A-Za-z0-9+/]*={0,2}', s):
            raise binascii.Error('Non-base64 digit found')
>       return binascii.a2b_base64(s)
E       binascii.Error: Incorrect padding

New option from Selenium 4: https://www.selenium.dev/documentation/webdriver/interactions/windows/#print-page

FIX in commit


Font type in Text extractor

Description of the new feature / enhancement
The text extractor should give a notification of the text copied just like the sniping tool does. Also, it should provide font details for the selected text.
Scenario when this would be used?
It would help programmers and designers to discover fonts from various websites and applications.
Supporting information
None

Hello and thanks for your message.
Text extractor to give the user feedback is...good feedback :) That would be a duplicate of issue #20305.
Font details is close to impossible, with the countless number of fonts, small adaptations and possibly very unique fonts. See #20670


http://www.domain.com/directory

Is it possible to have this type of configuration?
I did send the reverse proxy to /directory but i cannot see css, fonts and scripts. I am getting 404 for everything.

Did you set publicpath


Add app upgrade

This feature is not yet done.
There are still some problems with choosing specific target versions for an upgrade.
Also prompts, user interaction aswell as general code efficiency and hygiene needs some more work.
Any feedback is already welcome though!

Still Todo

[ ] fix up the "list-appupgrade-candidates" command to actually make it work
[ ] add app installation description to user dialogue where it make sense
[ ] make appinstallation selection interactive if none is given
[ ] make upgrade target version selection interactive if none given
[ ] add "--wait" flag
[ ] target version semver functionalities

This PR is ready to be merged after having gotten a second opinion on user dialogue and general functionality.
I merged every commit that was pushed into master since this branch was created and generated the readme but the readme check still fails, what am I missing?
About the two open todos:
Making the Project or App as an interactive selection would need a bit of work on the "withProjectId"/"withAppInstallationId" functions as to not bloat or overcomplicate things. I dont see it as a necessary part of this PR's feature.
I will tackle target version semver functionalities in a follow up PR at a later date.


question

hi , it is possible to persist index?

As the InstaTrie index is designed as a efficient runtime cache and index generation is quite fast, there is currently no persistence API for the InstaTrie index.
Currently the easiest solution for persisting the data would be to simply serialize/deserialize the collection of your models by any means available (e.g. XML serialization, or via a database) and then feeding the deserialized models into a fresh InstaTrie to recreate the index.
Depending on the efficiency of your word splitter and the average number of words in your models, this can be rather fast. We had used the TrieIndex with a few 100.000 models consisting of authors and titles, and index creation only took a few milliseconds.
Naturally, if you use it to index larger documents, index creation will take more time, but so would loading the index from any persisted state.
Feel free to ask if you need more information!

I have a doubt in this solution. Consider i want use that index in big data , maybe not only the index tree can be loaded in memory but index might be loaded partially when necessary.
I m thinking , for example a index present in mysql or cassadra is all in memory and at startup it load all the data present in tables for fullfilling the index tree?


Rails::Delegate should take into account that the receiver is not a class/module constant

Expected behavior
given the following code:
class A
  def some_method
    method_resolving_to_a_class.some_method
  end
end

I expect rubocop to realise that the receiver method_resolving_to_a_class  won't have a value or exist at the class level, only at the instance level and thus accept the above as being valid.
Actual behavior
It wants me to replace the above with:
  delegate :some_method, to: method_resolving_to_a_class

which won't work as method_resolving_to_a_class doesn't exist at the class level
Steps to reproduce the problem
No real steps, the code above should cause the cop to throw up a warning
RuboCop version
Output of rubocop -V:
$ rubocop -V
0.54.0

rubocop --auto-correct changes your example to:
class A
  delegate :some_method, to: :method_resolving_to_a_class
end

which will call method_resolving_to_a_class on an instance of A.

Unless I'm missing something, something was missed. Feel free to re-open if I'm wrong. 🙂

The problem is that when call delegate it expects method_resolving_to_a_class to exist but it doesn't because delegate is called in the context of the class even if the delegation only happens on an instanceas a result when you try to calldelegateit will crash becausemethod_resolving_to_a_classresults in aNoMethodError` because it doesn't exist when you try to delegate, it will only exist at the instance level, not the class level where delegate is called.

No, the problem is that the Cop shows up at all. Rubocop should realise that method_resolving_to_a_class is a method call, not a class constant that therefore may not exist in the context of code run in the class. Therefore doing delegate :some_method, to: method_resolving_to_a_class won't work. There's no ambiguity there, method_resolving_to_a_class only exists at the instance level, when you try to use it in delegate ruby will try to callmethod_resolving_to_a_class to resolve it's value but it won't work as it doesn't exist at the class level.

Rubocop isn't telling you to use
delegate :some_method, to: method_resolving_to_a_class

it wants
delegate :some_method, to: :method_resolving_to_a_class


How to remapping shortcuts?

The default shortcut combination is so hard for me to hit. A-M
hint_activation_key		string		"A-M-x"			"Activates hint mode."
grid_activation_key		string		"A-M-g"			"Activates grid mode and allows for further manipulation of the pointer using the mapped keys."
screen_activation_key		string		"A-M-s"			"Activate (s)creen selection mode."
activation_key			string		"A-M-c"			"Activate normal movement mode (manual (c)ursor movement)."
hint_oneshot_key		string		"A-M-l"			"Activate hint mode and exit upon selection."

I want to change it to another combination, like C-M or something else, which is more fit for me.
Is there a way to achieve this?

Yes, you can rebind all keys in ~/.config/warpd/config. Please see the man page for more details.

@LintaoAmons did you ever get it working? I stumbled upon rebinding via config do nothing, and I'm wondering if there's perhaps some magic trick, or is it just buggy in the latest released version.

yeah, my is working:
~ on ☁  (ap-northeast-1)
 cat ~/.config/warpd/config

hint_activation_key: A-M-j
activation_key: C-M-'
hint: f
exit: q
cursor_size: 13

@LintaoAmons what is your warpd version?


XPath expressions validity

XPath expressions are considered with logical OR. However it would be nicer to change that to AND too. Best it would be to select between OR and AND

Why not use XPath expressions to accomplish such cases?
(I am not sure about your use case)
For example this input xml:
<?xml version="1.0" encoding="UTF-8"?>
<root>
  <user>
    <login>user1</login>
    <name>User 1</name>
    <profile>
      <id>2</id>
    </profile>
    <profile>
      <id>1</id>
    </profile>
  </user>
  <user>
    <login>user2</login>
    <name>User 2</name>
    <profile>
      <id>4</id>
    </profile>
    <profile>
      <id>5</id>
    </profile>
  </user>
</root>

This xpath would succeed:
.//user[login='user1' and name='User 1' and profile/id[text()='1'] ]
This whould fail:
.//user[login='user1' and name='User 1' and profile/id[text()='5'] ]

That would work yes, though if the expression is too long I find it easier to read when they are several smaller.

I am afraid that, the UX might be complex.

I think we need just one radio button saying how to combine all the expressions - OR or AND.
Why need to be more complex?
Currently actually it is not really clear at first look how they are combined...

I like the idea.

Hi @milkotodorov.
Thank you for the suggestion.
I will take a look at this as soon as I can. Should be quite easy to implement.
Thank you.
Best regards,
Karl Dahlgren

Hi, did you has the change to look into this?


Details of inner exceptions are not displayed in console output

Inner exception details for both example and context exceptions are ignored.
We might need to consider using (or mimic) Exception.ToString instead of outputting stack trace and message separately.
Reproduce (NSpec 3.0.7):
public class illustrate_ContextInnerExceptionLost : nspec
    {
        public void given_FailureWithInnerException ()
        {
            act = () => { throw new Exception("Ctx:Outer", new Exception("Ctx:Inner")); };

            it["should display both inner and outer"] = () => { throw new Exception("Example:Outer", new Exception("Example:Inner"));} ;
        }
    }

Actual: Ctx:Inner or Example:Inner don't appear in console output:
**** FAILURES ****
nspec. illustrate ContextInnerExceptionLost. given FailureWithInnerException. should display both inner and outer.
Context Failure: Ctx:Outer, Example Failure: Example:Outer
   at nspec_debug.illustrate_ContextInnerExceptionLost.<>c.<given_FailureWithInnerException>b__0_0() in C:\_dev\other\nspec-debug\illustrate_ContextInnerExceptionLost.cs:line 13

Relevant

@dpoznyak just to let you know I'm going to close this shortly for inactivity


Ui  improvements to OAuth Configuration

ISSUE TYPE
Enhancement/Improvement Request
COMPONENT NAME
Component: API/ UI
CLOUDSTACK VERSION
Cloudstack version 4.19 rc 1,
SUMMARY
Ui  improvements to OAuth Configuration
Issue 1: Disable oAuth icon should be visible on top menu bar
Steps to reproduce the issue

Navigate to configutation  > Ouath Configuration

Screenshot

The disable action of account/ user in present in the top menu bar

Current behavior
End user has to edit the OAuth configuration and disable the configuration , which can be a little tedious

Issue2 : Extra Space is getting  adding in the UI  form of Oauth
Steps to reproduce the issue

Navigate to configutation  > OAuth Configuration.

Create a OAuth Configuration.

Copy and paste the client id, secret key and redirect URI

A extra space is getting added

End user has to make sure there is no space after they paste the credentials

@kiranchavala these are two issues, can you create separate issues for those, please? (personally I would refuse the second one as a user responsibility)

@DaanHoogland
@harikrishna-patnala requested to log both issues under the same ticket.
CC @harikrishna-patnala  could you please take this up when you are free, not a big priority as of now

Confirmed with Kiran that Issue 2 might not be a valid issue. So will provide the fix for Issue 1 only.

fixed in #9242


Cannot set property  on # which has only a getter

TypeScript Version:  4.0.0-dev.20200622

Search Terms:
Repo

In the VS Code code base
Revert this change in grid.ts
Recompile VS Code (yarn run watch
Start VS Code

Bug:
VS Code doesn't start due to the following exception
/Users/matb/projects/vscode/out/bootstrap-window.js:215 TypeError: Cannot set property GridViewSizing of #<Object> which has only a getter
    at Object.<anonymous> (/Users/matb/projects/vscode/out/vs/base/browser/ui/grid/grid.js:136)
    at Function.Module._invokeFactory (/Users/matb/projects/vscode/out/vs/loader.js:1040)
    at Module.complete (/Users/matb/projects/vscode/out/vs/loader.js:1050)
    at ModuleManager._onModuleComplete (/Users/matb/projects/vscode/out/vs/loader.js:1671)
...

This is triggered on a line that looks like:
    exports.GridViewSizing = Sizing;

@RyanCavanaugh This seems like a regression from 4.0.0-dev.20200615. Possibly related to #38809?

Wow, so it looks like there's a bug in our namespace export emit - namely, export namespace Sizing is adopting the reexport from
export { Orientation, Sizing as GridViewSizing, IViewSize, orthogonal, LayoutPriority } from './gridview'; - there shouldn't be a     exports.GridViewSizing = Sizing;, since the local Sizing namespace isn't exported as GridViewSizing. Here's a minimal repro:
// @filename: gridview.ts
export type Sizing = any;
export namespace Sizing {
	export const Distribute = { type: 'distribute' };
}
// @filename: index.ts
export { Sizing as GridViewSizing } from './gridview';
export namespace Sizing {
	export const Distribute = { type: 'distribute' };
}

I'm working on it now.


Create Dockerfile

The purpose of this just for production environment?

@lipemorais I think both guesses are valid and not exclusive. Am I wrong?

@cuducos, it's not exclusive. For now looks that the Dockerfile is aiming emulate "production" environment, since it's necessary build the image every time that a change is made.
To make development easier could be used a shared folder with source code in this Dockerfile or a docker-compose.yml doing the same and keep this Dockerfile.

Got it. IMHO the Dockerfile in the case of a pip installable is more useful for tests — if we need to prioritize, that would be my first guess…

Nice! Let me see if I got you too.
This is the kind of "test" that will help us keep the installation process always working, like running once a day or before release just to assure that everything are well.

— if we need to prioritize, that would be my first guess…
I didn't get what you mean with it. :/

Sorry about that. Rephrasing it:
If we need to prioritize between developing and testing the installation as the focus of this Dockerfile, testing the installation would be my first choice ; )

Just made a new version of this Dockerfile, now with Alpine.
Purpose? Make development with toolbox easier, specially when working with other Serenata projects.

Coverage remained the same at 48.247% when pulling fa5dbbf3705aee924dc255b8e3804c28c012ff8b on irio-docker into 854fcdc7af3f3a0fcedf58a6ed6e0f9d74422111 on master.

Coverage remained the same at 48.247% when pulling 4618d5553171d38c776bba9dee2d5f28b3d4d25c on irio-docker into 854fcdc7af3f3a0fcedf58a6ed6e0f9d74422111 on master.

The objective of this PR is to facilitate the way to run serenata-toolbox using Makefile.
By now, we can see that we can run make and the option.
To test this PR I:

First, I checked out to @Irio's branch:

git checkout -b irio-docker origin/irio-docker

Then I tested the make tasks:

make bootstrap

This bootstrap the enviroment.

and then:
make test

and ran the test suite.

And it LGTM ;)

Coverage remained the same at 48.247% when pulling cbf1f03e4012b7303d9cac3fee8bf9178fe2e3e7 on irio-docker into 854fcdc7af3f3a0fcedf58a6ed6e0f9d74422111 on master.

Coverage remained the same at 48.247% when pulling cbf1f03e4012b7303d9cac3fee8bf9178fe2e3e7 on irio-docker into 854fcdc7af3f3a0fcedf58a6ed6e0f9d74422111 on master.

I know that there is some recommendations for toolbox be used on Windows. Is it true for development too? If yes, make do not work on Windows environment as it do on Unix like ones. So this Makefile will not help who contribute from Windows.

cc @Irio @anaschwendler

Considering my review and @lipemorais's feedback: is it possible to rollback this PR?


Edit Tests

Alex said there is no need to include Try-Except statements in out tests

Will fix later
On Feb 13, 2015 12:36 PM, "bkhunter"<EMAIL_ADDRESS>wrote:

Alex said there is no need to include Try-Except statements in out tests
—
Reply to this email directly or view it on GitHub
https://github.com/CMPUT301W15T13/TravelPlanner/issues/93.

Removed all the uncessary Try-Catch exceptions from my tests. The necessary try-catch statements remain in the file (necessary for test).
Will look for others now


About the license for this model

Thank you for sharing your great code. :smiley_cat:
What is the license for this model? I'd like to cite it to the repository I'm working on if possible, but I want to post the license correctly.
https://github.com/PINTO0309/PINTO_model_zoo
Thank you.

Hey @PINTO0309,
Thank you for taking a look at it :). Happy to be able to help with your work.
I've just added a new commit with an MIT LICENSE file.
Thanks for your interest :)

Thanks!


Problems with saving layout

Normally when I restarted my PC, Rainmeter would remember my last settings.
However, there are several times - including the newest restart - it unloaded all of my suites and skins!
That's why I tried to save my layout in Rainmeter. One problem appeared. Rainmeter would only saved my preferences for the main hub or the sub hubs only.

When I saved Rainmeter layout in the Main Hub screen, all of the Sub Hubs things went to the top left position after I loaded it to test.
When I saved Rainmeter layout in the Sub Hubs screen, all of the Main Hubs things went to the top left position after I loaded it to test.

What should I do now?

This could be one of 3 things, or a mix of them.

 Unless you kept the option below checked during installation, Rainmeter will never open on computer startup.

You can identify if this setting was turned on or off by navigating to Startup Apps in your Windows Settings and see whether Rainmeter is on that list and toggled on.
If this is the issue, do a re-install of Rainmeter (no uninstall necessary) and make sure that option is checked with the re-install.

ToggleSwitch can be funky if your computer is too - on an old device of mines, it works prematurely and calculates idle time during computer startup since shutdown for at LEAST a tick, meaning if your computer was closed for longer than 10 minutes (or whatever setting you have timeout at) it auto-triggers on computer startup. You will know that this is an issue if:

You had closed your computer while in the main hub

Specifically, you closed your computer with ToggleSwitch active

You have correctly catalogued all your skins in the ToggleSwitch database
Rainmeter is set to launch on computer startup
During login, everything appears unloaded
In the Rainmeter client, when you select toggleon.ini, you can see it loaded (i.e. the button to Load/Unload currently says Unload, and variables like Coordinates are editable). You can double-check this by hovering over this Skin's supposed location, and it should unhide itself when you mouse over its area.

The cursor isn't visible, but ToggleOn is only visible right now because the cursor is right above it. You can also notice all the other skins being absent in its area.

If this is the issue, I can only recommend selecting the Change Timeout Value option in ToggleSwitch (ToggleOn does not have this option!) and setting the value to 0 or less to disable it, as your computer's idle time checker will not work with the auto-timeout feature. Alternatively, if this was a behavior you were unaware of and would LIKE this, then all the more better! Just leave it alone, and everytime you log in, click on ToggleOn hidden in the top right of your screen to load the layout!

When you save your layout, make sure you have "Exclude Unloaded Skins" UNCHECKED. Otherwise, when you save the layout, Rainmeter will "clean" your layout configuration by removing all Skins that have Active=0 (i.e. they are inactive), and thus during those skins' first load they will have the default configuration: loading at the top left of the screen, having a Normal Position instead of On Desktop, and so forth. You need layouts to remember the settings of unloaded skins as well; otherwise, skins like AudioVisualizer will break once you switch layouts and try to use the ToggleSwitch.

If this is the issue, I recommend loading the Default Layout that the .rmskin installation came with to "reset" everything including unloaded but relevant skins to their basic configurations, and you can readjust from there.

If any of these solved your issue, please specify which one did - that would help a lot! If none of these points fixed your issue or I misunderstood it completely, please clarify so I can try and think of a different solution.

I'm pretty sure my Rainmeter starts every time I open my PC.
I would consider my PC as one of those "high-end" PC and I had restarted my PC after everything seems unloaded after I turned on the PC, even though this won't rule out the chance of the ToggleSwitch problem.
Yeah I definitely always have that exclude checked so that was my bad.

That observation further increases my confidence in that it is issue 2), and following your explanation of your PC being on the high-end of things, I realized a possible explanation for the "strange" behavior:
I worded poorly that it takes a "funky" computer to exhibit the hidden-on-startup behavior. I am not well versed on the hardware side of things, but I recall reading somewhere that a PC when shut down doesn't actually fully shut down - with modern innovations a PC that is still receiving power has a hibernating component that allows computers to reboot extremely fast the next time it is opened. I don't know the exact keywords to search for to find a proper citation, but it just might add up - despite using a computer, I unplug the power supply after I shut down my computer, meaning my system is completely depleted of electricity and that 100% guarantees the computer is off, not sleeping at any level, especially after the night has passed. It is possible that if you keep your PC plugged in whether you're using it or not, it is preserving certain functions while off such as its internal clock while waiting for its next boot, which rolls into how ToggleSwitch "wrongly" believes your system has been idle for 10 hours instead of off. This would explain why it is happening on your high-end PC, and my low-end laptop is probably just mangled from the inside out after having gone through major repairs after 5+ years. However, as I said, I cannot properly cite my sources as I have forgotten them, so take this hypothesis with a grain of salt.
I'm also further convinced of this conclusion since you stated that "it appears unloaded once in a while", which sounds exactly like the behavior of ToggleSwitch's auto-hide feature.

I will close this issue as it looks like it was indeed a hibernating system interaction with the ToggleSwitch behavior, timing out the suite after 10 minutes of inactivity.


Allow manual control of label 'required' class

Added skip_required as a field option to prevent the usual checks
for a PresenceValidator and the addition of a 'required' CSS class
to the label element.
This is a fix for #205 and builds on the work done in #143.
(was:)

Added new field option label_required that, if present, will control
whether the required CSS class is added to the label element. If not
present, continue to use the existing logic that looks for a
PresenceValidator on the model object.

Hey @LucasAU!  Nice work on cleaning this up -- this feature had way more corner cases than expected.
A couple of thoughts:

did you have any other ideas on the parameter name?  "label_required" sounds to me more like the label is required vs the parameter.  With that said, not sure I can think of an alternative name other than simply using required :-)
For the documentation update, would it make more sense to use the negative case (i.e. disabling the required css class for a required attribute?).  The reason I say this is the same thing in the example could be accomplished with using the standard class: 'required' approach which might be a little cleaner.

Thanks for your work on this.

Thanks for the feedback. Good point about limiting scope to the negative case.
I'll have another go at it. I think skip_required: true may make the most sense. It sort of falls in line with skip_label.

All minor nits.  skip_required: true makes a lot of sense and per your point lines up nicely with skip_label.  Thanks @LucasAU!

Hi, thanks for all your work on this PR! I'd like to merge this in. Can you rebase it and fix the conflicts?

Rebase done, conflict resolution was straight forward.

Much appreciated! ✨


Timeout in typeclass synthesis of ∀ i, MvFunctor (_ i)

In the following coinductive type we get a timeout after updating in #1
data QpfTest α β where
  | A : α → α → β → QpfTest α β → QpfTree β → QpfCoTree (QpfTree (QpfTest α β)) → QpfTest α β

/-
failed to synthesize
  MvFunctor
    (MvQPF.Comp (TypeFun.ofCurried Shape)
      (Vec.append1
        (Vec.append1
          (Vec.append1
            (Vec.append1 (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs (Fin2.fs Fin2.fz)))) (MvQPF.Prj (Fin2.fs Fin2.fz)))
            (MvQPF.Prj Fin2.fz))
          (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs Fin2.fz)))))
        (MvQPF.Comp (TypeFun.ofCurried QpfCoTree)
          (Vec.append1 Vec.nil (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj Fin2.fz)))))))
(deterministic) timeout at 'typeclass', maximum number of heartbeats (20000) has been reached (use 'set_option synthInstance.maxHeartbeats <num>' to set the limit)
-/

Each of the individual typefunctions are known to be functors:
#synth MvFunctor (TypeFun.ofCurried (n:=5) QpfTest.Shape) -- succeeds
#synth MvFunctor (MvQPF.Prj (Fin2.fs (Fin2.fs Fin2.fz))) -- succeeds
#synth MvFunctor (MvQPF.Prj (Fin2.fs Fin2.fz)) -- succeeds
#synth MvFunctor (MvQPF.Prj Fin2.fz) -- succeeds
#synth MvFunctor (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs Fin2.fz)))) -- succeeds
#synth MvFunctor (MvQPF.Comp (TypeFun.ofCurried QpfCoTree)
          (Vec.append1 Vec.nil (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj Fin2.fz))))) -- succeeds

It is when we put them in a vector that the timeout happens:
private abbrev Gs : Vec (TypeFun 5) 5
  := (Vec.append1
    (Vec.append1
      (Vec.append1
        (Vec.append1 (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs (Fin2.fs Fin2.fz)))) (MvQPF.Prj (Fin2.fs Fin2.fz)))
        (MvQPF.Prj Fin2.fz))
      (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs Fin2.fz)))))
    (MvQPF.Comp (TypeFun.ofCurried QpfCoTree)
      (Vec.append1 Vec.nil (MvQPF.Comp (TypeFun.ofCurried QpfTree) (Vec.append1 Vec.nil (MvQPF.Prj Fin2.fz))))))

#synth ∀ i, MvFunctor (Gs i)

Solution
The implementation of (co)data relies too much on the VecMvFunctor typeclass hack. We should have it generate a definition of Gs as above, then synthesize each of the individual functor instances, and generate the final instance for the concerete vector Gs, instead of trying to teach the typeclass machinery about arbitrary vectors.

There are also smaller examples with a similar issue:
-- adding to Test/Misc.lean

data QpfTreeInTreeTest α where
  | A : QpfTree (QpfTree (QpfTreeInTreeTest α)) → QpfTreeInTreeTest α

import Test.List

data QpfListInListTest α where
  | A : Test.QpfList (Test.QpfList (QpfListInListTest α)) → QpfListInListTest α

Although I do not see timeout being stated in the error message, I still think that this might be related.

What is the error you were seeing?
Regardless, I've implemented the solution I described in the better-typeclass-synth branch, and it happily compiles both the original QpfTest example, and also QpfTreeInTreeTest and QpfListInListTest without errors :tada:

It's a very similar message for both:
▶ 34:1-35:1: error:
failed to synthesize instance
  MvQPF
    (MvQPF.Comp (TypeFun.ofCurried Shape)
      (Vec.append1 (Vec.append1 (Vec.append1 Vec.nil (MvQPF.Prj (Fin2.fs Fin2.fz))) (MvQPF.Prj Fin2.fz))
        (MvQPF.Comp (TypeFun.ofCurried Test.QpfList)
          (Vec.append1 Vec.nil
            (MvQPF.Comp (TypeFun.ofCurried Test.QpfList) (Vec.append1 Vec.nil (MvQPF.Prj Fin2.fz)))))))

▶ 34:1-35:1: error:
type mismatch
  MvQPF.Fix.mk ?m.112558
has type
  MvQPF.Fix ?m.112339 ?m.112342 : Type ?u.112337
but is expected to have type
  QpfListInListTest α : Type

▶ 34:1-35:1: error:
application type mismatch
  MvQPF.Fix.mk (Shape.A x✝)
argument
  Shape.A x✝
has type
  Shape ?m.112426 ?m.112427 (Test.QpfList (Test.QpfList (QpfListInListTest α))) : Type
but is expected to have type
  ?m.112339 (?m.112342 ::: MvQPF.Fix ?m.112339 ?m.112342) : Type ?u.112337

My guess is that the timeout issue happens somewhat "internally" here which makes deriving the outer MvQPF instance impossible...
Good to hear that you already fixed it! :)

Yeah, the important part of the error is that it failed to synthesize the instance. Admittedly, building these huge expressions and then handing them off to typeclass search was a bit of a brittle hack.
I've now merged the fix to master, which builds up the necessary instances at the same time as building the expression itself (because we know then exactly how to build them). If you're curious: the commit is https://github.com/alexkeizer/QpfTypes/commit/bfbd8310843b40a2844a525e33c50d929ac68942.

Thanks for the pointer. This makes sense to me :)


Fix Metal3 subchart keys

Fixes #10

metal3 prefix exists in some more places in the repo. In order to make the names consistent, can we replace the names everywhere?

I figured out that the issue #10 is caused by #14, so I propose to close this PR since when #14 will be solved this PR effect will be useless and subchart keys metal3-<subchart name> will work as expected.


LSP support

This is related to https://github.com/tlaplus/vscode-tlaplus/issues/153.
The current TLAPS/Toolbox integration involves document parsing on the Toolbox side (it uses Sany for that, as I understand). While moving to the VSCode, I would like to push as much logic out of the editor to the backend (TLAPS). The parsing is one aspect. The TLAPS has a parser implementation anyway, and the parser is used by the editor in the scope of the proofs, at least for locating the obligations (or a sub-tree of obligations), folding, and probably proof decomposition functionality.
Instead of implementing a specific protocol for the TLAPS integration in the VSCode, it may be worth trying to implement LSP (Language Server Protocol) internally in the TLAPS. E.g., if started with the --lsp option instead of the --toolbox, it would behave as an LSP server. It could then receive a document to process, handle commands (e.g., check a step, propose a decomposition, etc.), and output the relevant information.
It looks like there are some LSP building blocks that would be reused, e.g., https://github.com/ocaml/ocaml-lsp.
I hope it should not interfere with the existing functionality as VSCode supports multiple LSP servers for a single document.
What do you think about such a direction?

No worries about a conflict as the current vscode-tlaplus extension is not built with LSP.

Your suggestion sounds good to me, but it may require substantial changes to tlapm, which AFAIK is currently not expecting to interact reactively with the IDE.

I will experiment a bit and then return with the results.

Some LSP commands are working already, document synchronization and some fake diagnostics are shown.
The code is in OCaml; the initial draft is here: https://github.com/kape1395/tlapm/tree/lsp/lsp.
It is on top of PR #83 and #91.
Moving further depends on how to present the proof states and accept the user's actions.
The proof obligations could be shown as test cases like in the Dafny IDE.

Maybe you have other approaches to try? I checked:

Diagnostics could underline the obligations, but that makes the proof text less readable.
Code actions - a "lamp" along the proof line is probably not visible enough; provides no ways to show an overview.


Stream blocks to arrow

Stream blocks from clickhouse using RecordBatchStreamAdapter
Currently set dependency to my fork of the clickhouse-rs

Also, we could benefit from a little more context in the PR description.  Can you put in some more details including a why/motivation section.


Why this repository is showing ineligible in hacktoberfest?

same here, and dunnno repo owner is not responding as well :(

please check the readme, it has been updated.

please check the readme, it has been updated.

https://github.com/bhaveshlohana/HacktoberFest2020-Contributions/pull/1360
See this pull request made by me. I have followed all the rules but it is still not marked at "hacktoberfest-accepted" or it is not merged till now. I haven't made any small changes which will not add value to the project.
This project was used by our friends in the college to share files from one pc to another, and it was made by me.

@saurabhshalu I know, hacktoberfest-team has excluded this repo from Hacktoberfest. Check the readme or #1429

Okay


Be sure executable returns fail code on error

It looks like when db returns an error, the exe is not returning an error code. Check this and make sure it's behaving correctly.

added to wrong repo


bug: monitor keyword params fails to parse

Did you check existing issues?

[X] I have read all the tree-sitter docs if it relates to using the parser
[X] I have searched the existing issues

Tree-Sitter CLI Version, if relevant (output of tree-sitter --version)
tree-sitter 0.22.6
Describe the bug
Copied from the config here: https://wiki.hyprland.org/Configuring/Monitors/
monitor=eDP-1, 2256x1504@60, 0x0, auto does not parse properly
I get a similar error on lines like these:
bindl=...,switch:off:[switch name],exec,hyprctl keyword monitor "eDP-1, 2560x1600, 0x0, 1"
Steps To Reproduce/Bad Parse Tree

Expected Behavior/Parse Tree
params is parsed as strings (or some other type)
Repro

tree-sitter generate
echo 'monitor=eDP-1, 2256x1504@60, 0x0, auto' >conf
tree-sitter parse conf

(configuration [0, 0] - [1, 0]
  (keyword [0, 0] - [1, 0]
    (name [0, 0] - [0, 7])
    (params [0, 8] - [0, 38]
      (string [0, 8] - [0, 13])
      (ERROR [0, 14] - [0, 19])
      (string [0, 19] - [0, 27])
      (ERROR [0, 28] - [0, 31])
      (number [0, 31] - [0, 32])
      (string [0, 33] - [0, 38]))))

Sorry for the late reply, it should be fixed now but it may takes some times before the parser and highlights in nvim-treesitter get updated. But as it's a parser issue if think I can close this now.

Thanks!
I'll look out for the update from nvim-treesitter and test when I have a chance

Thanks!
I'll look out for the update from nvim-treesitter and test when I have a chance

I'll make a PR to nvim-tressitter later


Cannot find module 'buildStyleInterpolator'

Hey there!  Thanks for all your hard work mocking this all out.
In my app I'm using buildStyleInterpolator which is provided by React Native and made available via import buildStyleInterpolator from 'buildStyleInterpolator'... which I feel is a bit strange in the first place as most library utilities like Dimensions are imported like import {Utility} from 'react-native'.
Regardless, it looks like react-native-mock is not mocking out buildStyleInterpolator as well as various other utility functions that are exported from react-native/Libraries/Utilities/** (screenshot below from my node_modules).

Please help me out? (I'll take a look now to see if I can't create a PR myself)

I have been thinking on a nice way of implementing these fixes. I think the best way of doing it would be in the same way we override nodes require with react-native. We should be able to do the same for any other library.

I am hoping to start work on something like this soon, once a few more PRs are released. Hoping to bring it out inline with react-native 0.30 so we can match up the version numbers. Stay tuned for a fix!


How to user owl carousel on React JS

I wanna user owl carousel JQuery is not defined in global scope for my application.
When a remove the import of carousel, all goes right. I can call jQuery function and alert some text on the screen of my webapp.

============================= *
import $ from 'jquery';
// import owl from './assets/js/vendor/owl.carousel.min';
// import main from './assets/js/main';
$(document).ready(() => {
alert('Works !');
})
window.jQuery = window.$ = $;
================================*
However, when I import owl.carousel and main.js , appear that error =/ (  Line 238:  'jQuery' is not defined  no-undef ) the JQuery is not serve to the main and ow.carousel

well it works for me even I never configure in webpack, I simply import jquery in particular component like
import $ from 'jquery';
and if still you ound error then in componentdidmount lifecycle you can find jquery into window.jquery and you can assign it to any variable. then it should work.

Add jQuery as a script tag and then do this:
https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/template/README.md#using-global-variables
Seems like the easiest way?

Thanks Guys ! That solved my problem. In order to make  owl carousel and google maps works, I had to install react-google-maps and react-owl-carousel. I was trying to call then directly on the ./public/index.html . That was a mistake cause whenever I navigate through the website, and back again to the initial page ( where the owlcarousel is set )  it stopped working. XD !!!


Java: Add Zrandmember command. (Sorted Set Commands)

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Converted to draft: waiting on approval for the return type conversion design.


[FEATURE REQUEST] Function to return a secret, if it doesn't exist generate a random string, set it, and return it.

I wasn't sure where the best place for this discussion so figured I would just make a ticket.
Is your feature request related to a problem? Please describe.
A common use case we have is to initialize new clusters of applications that need secrets generated. We don't care what the secrets are as long as they are unique per environment. The way we have been doing this is with a custom execution module in salt called get_set_secret.
Describe the solution you'd like
vault.get_set_secret
This function has two arguments:

Path to a secret you want to read or write to
Dictionary of key values you want to generate if the secret doesn't exist.
If the value of one of the keys is something like "generate" we just generate a random string of say 30 characters and use that. Could add additional arguments to change this behavior.

Describe alternatives you've considered
Before using this method we would manually bootstrap and load secrets into our new clusters each time we redeployed.
Additional context
The main downside I see with this is salt needing write permissions to the paths would could be considered a security issue.
I am interested in any other approaches or advice as well!

We don't care what the secrets are as long as they are unique per environment.

Do you know about sdb.get_or_set_hash, which works with the vault SDB module? It takes a

Path to a secret you want to read or write to

+ a length and charset of the secret to generate if it does not exist at the path. It does not however generate multiple values per invocation.

The main downside I see with this is salt needing write permissions to the paths would could be considered a security issue.

Agreed, there's some potential for lateral movement, but you might be able to mitigate it somewhat by only allowing the creation of the secret, no updates. For multiple values, this would require one (Vault) path per secret though.

Do you know about sdb.get_or_set_hash, which works with the vault SDB module? It takes a

Path to a secret you want to read or write to

a length and charset of the secret to generate if it does not exist at the path. It does not however generate multiple values per invocation.

Oh no I did not know this. What is the reason to use vault with sdb vs just calling the execution module in the state?

It's just a different interface to the same backend, but it provides a similar function to the one you describe, no other reason. You can use both methods at the same time.
In general, you can specify SDB URI in places where config.get is used to transparently source (configuration) values from databases instead of writing them to the config in plain text. You can read about it in the Salt docs.

Ok thank you. I will give this a try!


Tests Failing

On main 5 tests are failing. Not sure if there is something else I need to setup. Full log (too long to put in a comment):
https://gist.github.com/jamesward/738f703601fdbac2ef64fa18ef3dd3ef

What OS are you on, and what is your java version?

JDK 17. Ubuntu Linux.

Hm, we test on that. Can you try adding env var LANG=en_US.UTF-8?

Same failures with that env var.

This appears to have been caused (somehow) by the bump to Gradle 7.6, which has been reverted. Can you confirm these tests no longer fail for you?

I don't think it was actually; James tested prior to us merging the Gradle verison bump


Support for the Zero Trust Gateway: Network Firewall Policy

Current Terraform and Cloudflare provider version
Terraform v1.3.9 with the cloudflare provider v3.35.0.
(I also double-checked the docs for the new cloudflare provider v4.0.0.)
Description
I'd like the ability to manage network firewall policies through terraform. The feature is available on the Zero Trust dashboard under the Gateway > Firewall Policies > Network tab.
Use cases
With multiple virtual networks and many cloudflare tunnel instances to manage, we'd appreciate being able to also capture our gateway policies in terraform.
Potential Terraform configuration
The cloudflare API seems to treat gateway rule policies as a string expression:
https://api.cloudflare.com/#zero-trust-gateway-rules-create-zero-trust-gateway-rule
The bare minimum might look like:
resource "cloudflare_zero_trust_gateway_rule" "allow_to_prod" {
  account_id = var.cloudflare_account_id
  name = "Allow access to prod tier"

  action = "allow"
  enabled = true
  precedence = 3000

  filters = ["l4"]
  traffic = "net.vmnet_id == \"3a87c214-FAKE-FAKE-FAKE-1b87d1d845b5\"" # not a real virtual network ID
  identity = "any(identity.groups.name[*] in {\"Foo Group\"})"

  rule_settings {
    check_session {
      enforce = true
      duration = "12h"
    }
  }
}

This is a bigger ask: it would be nice to be able to express these in a more terraform way, such that, for example, I could use references to other objects. I'm not sure if there's already a standard you all adopt for these expressions, but modeling it off AWS IAM policy documents, it might look something like:
data "cloudflare_zero_trust_gateway_rule_expression" "in_prod" {
  property = "net.vmnet_id"
  value = cloudflare_tunnel_virtual_network.prod, # be able to refer to them as objects instead of hard-coding network IDs
}

data "cloudflare_zero_trust_gateway_rule_expression" "foo_users" {
  property = "identity.groups.id[*]"
  values = [
    cloudflare_access_group.foo.id,
  ]
}

resource "cloudflare_zero_trust_gateway_rule" "allow_to_prod" {
  account_id = var.cloudflare_account_id
  name = "Allow access to prod tier"

  action = "allow"
  enabled = true
  precedence = 3000

  filters = ["l4"]
  traffic = data.cloudflare_zero_trust_gateway_rule_expression.in_prod.expression # and use the expression here as an object
  identity = data.cloudflare_zero_trust_gateway_rule_expression.foo_users.expression # same here

  rule_settings {
    check_session {
      enforce = true
      duration = "12h"
    }
  }
}

References
No response

Does this not already work with cloudflare_teams_rule?  I've created Network Policies like so:
resource "cloudflare_teams_rule" "allow_contractors" {
  enabled     = true
  name        = "Allow Contractors"
  account_id  = "foobaraccount"
  description = "Allow Contractors access to Foobar"
  precedence  = 1
  action      = "allow"
  filters     = ["l4"]
  traffic     = "net.sni.host in {\"example.com\" \"bar.example2.com\"}"
  identity    = "any(identity.groups.name[*] in {\"Contractors\"})"

  rule_settings {
    block_page_enabled = false
  }
}

Yup, the documentation is confusing. I guess Zero Trust was once named "Teams".

Thank you for pointing this out! Gonna close this now.
Out of pure curiosity, I looked through the source code that cloudflare_teams_rule calls the TeamsRule Go API:
https://github.com/cloudflare/terraform-provider-cloudflare/blob/1ff904481eb07b84e58a548fd68bf3ba4af4a339/internal/sdkv2provider/resource_cloudflare_teams_rules.go#L33-L37
which ends up doing an HTTP call to /accounts/{accountID}/gateway/rules/{ruleID} at:
https://github.com/cloudflare/cloudflare-go/blob/d3fcc6c0396ae4c039ec30a3c699e62d1fec73a8/teams_rules.go#L184-L185
which matches the URL in the API docs for "Zero Trust Gateway Rule Details".


Fix errors in SDL to be caught

Related Issues
Close #534.
Summary
Replace throw new with throw.

Thanks. Merging.


[HB-4992] Simplify SKAdNetwork Integration Process

Description
Helium Unity should have had some sort of reference as to how to integrate SKAdNetwork, this PR exists to address that and make it easier for developers to have such capability enabled.
TODO
We might want to add some checks in case the webrequests fail or timeout. This is just the initial PR 👍🏻

Lgtm. Looking for these is something. Nicely done.


[BUG] Fail to batch write decimal type with conversion in new row format

Describe the bug
batch write with decimal type will fail when using new row format
Spark and TiSpark version info

could you release v2.3.5 to maven repository? waiting for this fix, thanks!

could you release v2.3.5 to maven repository? waiting for this fix, thanks!
@birdstorm @marsishandsome

hi Dieken, v2.3.5 will be available tomorrow.

@Dieken v2.3.5 is available now https://github.com/pingcap/tispark/releases/tag/v2.3.5


Update for Faraday 2.0

According to the upgrade guide Faraday Middleware was deprecated with the 2.0 release. It seems we're leveraging it for faraday_middleware/multi_json which seems to have just been refactored to Json and promoted to the Faraday gem itself?
I'm happy to work on a solution for this but I have specs failing on Failure/Error: require 'active_support/per_thread_registry'

neo4j-core gem is discontinued and superseded by activegraph which does not use Faraday. But thank you for offering help.
We do have the per_thread_registry deprecation in activegraph though, so that one still needs to be fixed.


Sound/Volume is very faint

Hello. As noted in the issue title sound at the end of the time period seems extremely faint. Are oters experiencing this. Could it be fixed and perhaps an option to allow or not sound and at what level. Thank you.

Related to #19


Message should be clearer when bridging a guild

Ignoring the typo, the message here isn't very clear. I waited around for about 30m expecting to start getting invited to rooms from the guild I just bridged, only to realize I'd only start getting invites once a message was sent.
I've made issues about changing that behaviour anyways, but it'd be nice in this moment to explain what the user should expect from here onwards.
Something like: "Discord Puppet Bridge is now listening for messages in Guild BABSCon", which would at least clue the user in that a message needs to be sent in a room before the bridge will act upon it.

Ignoring the typo

whaaaa, how to english? a little help, please?
Also, maybe add "Type listchannels and click a channel pill to start talking"?

whaaaa, how to english? a little help, please?

"Is now being bridged" :)

Also, maybe add "Type listchannels and click a channel pill to start talking"?

Mm, I think we can just replace this step and tell people to listchannels instead (or have a way to auto-join all channels at once), no?

Fixed in https://github.com/matrix-discord/mx-puppet-discord/commit/24ef3804fa3b6485fb43ebe31b8c72cf135c4062

So does the bridge no longer invite you to channels as people talk in them? (I agree that this was confusing, so happy to see it removed if so).

It still does.

Oh, well if you want to leave that behaviour in, then the message should contain "You will be invited to guild channels as messages are sent in them", just to let the user know.

done, thanks


RuntimeError: DataLoader worker (pid(s) 49) exited unexpectedly when trying to test config files in Conv3D

Greetings. Congratulations on your work. I was trying to  run "python Sign_Isolated_Conv3D_clip_test.py" inside /Conv3D/ folder. I ran into the following error, I'm pasting the error traceback.
---Traceback Starts----
Using 2 GPUs
######################Testing Started#######################
ERROR: Unexpected bus error encountered in worker. This might be caused by insufficient shared memory (shm).
Traceback (most recent call last):
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 872, in _try_get_data
data = self._data_queue.get(timeout=timeout)
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/queue.py", line 179, in get
self.not_empty.wait(remaining)
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/threading.py", line 306, in wait
gotit = waiter.acquire(True, timeout)
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/_utils/signal_handling.py", line 66, in handler
_error_if_any_worker_fails()
RuntimeError: DataLoader worker (pid 49) is killed by signal: Bus error. It is possible that dataloader's workers are out of shared memory. Please try to raise your shared memory limit.
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "Sign_Isolated_Conv3D_clip_test.py", line 137, in
val_loss = val_epoch(model, criterion, val_loader, device, 0, logger, writer, phase=phase, exp_name=exp_name)
File "/home/smilelab_slr/Isolated_SLR/CVPR21Chal-SLR/Conv3D/validation_clip.py", line 13, in val_epoch
for batch_idx, data in enumerate(dataloader):
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 435, in next
data = self._next_data()
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1068, in _next_data
idx, data = self._get_data()
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1024, in _get_data
success, data = self._try_get_data()
File "/home/smilelab_slr/SLR_pytorch/env/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 885, in _try_get_data
raise RuntimeError('DataLoader worker (pid(s) {}) exited unexpectedly'.format(pids_str)) from e
RuntimeError: DataLoader worker (pid(s) 49) exited unexpectedly
----Traceback Ends----
Any insights will be appreciated.
Thanks.

set  docker  shm-size="12g"


Stimulus controllers lost on BACK and FORWARD [iOS]

If you set up a stimulus controller, on iOS Safari (latest GA) and go back and forward, the element behaves as if the stimulus controller is not attached.

Managed to attach safari, there was an import map CDN caching problem that was looking for the wrong JS file.

Nope, caching wasn't the problem. I'm able to reproduce the issue on my app, will try to make a public repro, but sure if any of the codepen style sharing sites will work with back and forward.

Can confirm that the problem is with form interactivity itself, not just the stimulus controller. I have a stimulus controller to do a confirm on form submit, works fine in all browsers, but hitting back and forward on iOS the submit buttons no longer do anything.

I just tested this on iOS safari with a form that has a stimulus attached to the submit event.  I'm unable to reproduce the reported issue.

Please reopen if you can boil this down to a reproducible case.


microFS problems on version 2.1.0 beta

i am now using  version 2.1.0 beta of microbit.hex which i flash using file copy.  how ever now that i have to use microFS to to copy my program and class definition i am experiencing a lot of failures with the error message Expecting 2 values to unpack, only found 1
i believe this is in relation to outstanding file system issues and not a problem with microFS.

This is an issue with microFS, this PR aims to fix it, but still needs some tests added before we can merge:

https://github.com/ntoll/microfs/pull/39

You can use it to patch your local installation of microFs, or if you like you can pip install -e . from that fork branch.


Nuxt 4 layers nested composables are not defined / auto imported

Environment

Operating System: Linux
Node Version:     v18.20.3
Nuxt Version:     3.13.2
CLI Version:      3.14.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-mag9gj-w7fy1u?file=layers%2Fauth%2Fapp%2Fcomposables%2Fservices%2FcomposableNested.ts,app%2Fcomponents%2Ftest-nesting%2FAppComponent.vue
Describe the bug
If you have a composable in a nested folder inside layers in Nuxt 4 (and possibly earlier?) then it will not be auto imported.

500  useComposableNested is not defined
See reproduction above, if you comment out this line then the non-nested layer composable will run fine.
app/components/test-nesting/AppComponent.vue
<script setup lang="ts">
// const { isComposableNestedWorking } = useComposableNested();

Additional context
I have tried adding this to layers/auth/nuxt.config.ts
imports: {
  dirs: ['~~/layers/auth/app/composables/services/**']
}

My unit tests also indicate this might also be breaking asyncContext in those layer nested composables. e.g they are unable to call useNuxtApp(), I get [nuxt] instance unavailable
Logs
No response

Maybe with resolve?
import { createResolver } from 'nuxt/kit';

const { resolve } = createResolver(import.meta.url);

export default defineNuxtConfig({
  imports: {
    autoImport: false,
    dirs: [resolve('./app/composables/services/**')],
  },
});

@dsvgl thank you resolve fixed it!
https://stackblitz.com/edit/github-mag9gj-tnstob?file=layers%2Fauth%2Fapp%2Fcomposables%2Fservices%2FcomposableNested.ts,layers%2Fauth%2Fnuxt.config.ts


The admin should not be able the edit the disabled fields by inspect element.

Terms

[x] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
The admin is able to edit the disabled fields of attributes by inspecting the element.
Preconditions
1. Admin should be logged in
2. Some attributes should be created by the admin.
3. admin should be on the attribute edit page.
Steps to reproduce
1. Go to the 'admin' panel
2. Click on 'Attributes' inside the catalog section.
3. Edit any attribute.
4. Reach on is_required section.
5. Inspect the element.
6. Change is_required = no.
7. See an error
Expected Result
The admin should not be able to edit the disabled fields of any attribute by inspecting the element.
Actual Result
The admin is able to edit the disabled fields of any attribute by inspecting the element.
Screenshots

Additional context
No response

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Remove "since" gremlin predicate since context accepts duration

https://softwarefactory-project.io/r/#/c/6911/


Use serde for deserializing instance metadata credentials

My attempt at fixing #1123

Looks great, thank you!


Are we making too many Kubernetes API request from the server?

I've done some cursory testing, and not found this to be the case.


fix: Fix initial glitch (Context solution)

#206

landed as 90f5784

This doesn't work for nested scenes because of #190


Roadmap listing or keep the simplicity likes the current state

Great work, thank you, it seems the only functional go game engine to
support cross-platform, AT, is any roadmap listing, maybe this could encourage
people to contribute some features, or the authors prefers keep
the current engine simplicity ?

Hi,
No concrete roadmap so far yet, but my vague plan is:

Nov 2020: Shaders

Now I'm working on this.

2020?: Porting to Nintendo Switch

I have developed go2cpp to support environments where Go doesn't support. Unfortunately, because of NDA, I cannot say the details even if I have succeeded to run Ebiten applications on Switch. However, I believe supporting Switch would be feasible.

Mar? 2021: Ebiten v2

No drastic change. I'd like to add some API change and remove already-deprecated features.

2021?: Vector graphics

Now there is vector package, but I need to add more features.

That's it. I'd like to keep the current engine simplicity. Especially after supporting shaders and vector graphics, there is no plan to add big features. Non-visible change like supporting more platforms (e.g., Raspberry Pi Zero by Dispman, ARM Windows by using DirectX, etc) will be more important. Even if we don't add big changes, there are a lot of things to do.
Contributions are always welcome!

Thanks for the detailed info.
Maybe this could go in readme.md since you already have typed it :D
I am doing more aiming to mobile platforms, so I am glad to have the actual
Ebiten support.
I am happy that vector is in the list, I miss those days that you could draw
interesting things via drawing api in flash.
Sure, I will see I would be able to contribute some, after I finish my first game with Ebiten!

Maybe this could go in readme.md since you already have typed it :D

Thanks, but I think I'd write a blog article, as I'd like to keep README as simple as possible.
I appreciate all your Ebiten usages, feedbacks and contributions!

https://ebiten.org/blog/roadmap2020.html Brushed up the roadmap


nginx: Node Red behind proxy - no sidebar icons and texfields that should be hidden

This is not a node-red bug, but I need some help from people who have actually set up node-red behind nginx as proxy.
What are the steps to reproduce?
Put Node-Red behind nginx as proxy:

location / {
proxy_pass http://red.mynetwork.com:1880; //mynetwork = my home network
proxy_set_header Host $host;
proxy_set_header X-Real-IP $remote_addr;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection "upgrade";
}

What happens?
When I now visit: https://red.mydomain.com, then for some reason I can see 90% of the UI. But it looks like some of the resources can't be loaded. (If I access it directly via 1880 everything is loaded - but I don't want to expose node red nor the port to the public)...
I could see that some icons are not loaded in the side bar (but I can see them in my flow). Also I see some text fields that are supposed to be hidden.
When I look at the Chrome Web Developer I can clearly see that it load almost everything... no issues with JS, CSS, HTML files.... but I can see that it actually does not even request certain icon files.
Missing Icons
Weird Text Boxes
What do you expect to happen?
Have the same experience like without a proxy.
Please tell us about your environment:

[ ] Node-RED version: 0.18
[ ] node.js version: latest
[ ] npm version: latest
[ ] Platform/OS: Raspi
[ ] Browser: Chrome - latest

Hi, thanks for raising an issue.

This is not a node-red bug, but I need some help from people who have actually set up node-red behind nginx as proxy.

If you check out our contribution guidelines you'll see we prefer to handle this sort of thing on the mailing list, slack team or you could ask a question on Stack Overflow and tag it node-red. That way the whole Node-RED user community can help, rather than rely on the core development team.
Thanks!

That's a good point,. I already added this link to reddit.
Any way there is still something that the core team could provide some input for:
Do you have an idea why only SOME icons are being requests when going through nginx? It's not that generally all items are blocked (that why I'd assume it's an nginx config issue).
This might help me to dig deeper (or add more information to my support threads).
Thank you very much, @knolleary!
BTW: Great work,. After my project is set up, I'll help you improving this great platform.

I already added this link to reddit.

Where exactly on reddit? If you want help from people who have done what you are trying, I do strongly suggest you ask within the node-red community at one of the links I provided.
I'm sorry, but I have not used nginx and cannot advise on how you should configure it.
I assume you've looked at the browser log to see what paths are failing to load to get a clue as to what's missing?

Hey @knolleary,
I think I might have found the issue - will try them as soon as I'm at home. My custom security headers that I use on all of my websites might block inline code.
I'll update/close this ticket as soon as I have confirmation.

Any news?

My location section looks like this. No missing icons or extra text fields.
location / {                 proxy_set_header Upgrade $http_upgrade;                 proxy_set_header Connection "upgrade";                 proxy_http_version 1.1;                 proxy_pass http://192.168.XXX.XXX:1880;         }

Thanks for this!
I needed to add
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection "upgrade";

for everything to work right. I was getting "lost connection to server" without it. (it still worked despite the error btw, just annoying)
If I understand correctly that's to make sure a persistant connection is kept open between client and node-red app running on server.
https://nginx.org/en/docs/http/websocket.html

We have solved this problem by using a stream block instead of location.

Can you explain how you did fix it with nginx stream module?

use stream header.
On Wed, Oct 17, 2018, 2:45 AM Debdut Biswas<EMAIL_ADDRESS>wrote:

Can you explain how you did fix it with nginx stream module?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/node-red/node-red/issues/1672#issuecomment-430352372,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYOUUqFVu4ccIA45JAdRNb8D1C6BMdwRks5ulilMgaJpZM4Sp8aO
.

Try this:
location /node-red/ {
    proxy_pass http://<IP_ADDRESS>:18880/;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "upgrade";
    proxy_http_version 1.1;
}


hosting weights on github repository release

Why not hosting pretrained weights in github repository release? You can upload files having GBs of size into repository release. And using a simple get request function, weights can be downloaded into local. It would free us from the unnecessary wandb dependency when downloading a pretrained weight.
I can show you an example if you desire to go on this direction 👍

Sounds really promising! Would you like to commit a pull request for a model in the transformer folder?

I feel like this would go great with the newest "general" dall-e models.

Okay well PR doesn't transfer releases 😅 should have seen this coming.
Anyway I've DMed you how do setup .pt files onto github via releases.
Here is 3E: https://github.com/johnpaulbin/DALLE-models/releases/tag/model

Okay well PR doesn't transfer releases 😅 should have seen this coming.
Anyway I've DMed you on discord how do setup .pt files onto github via releases.
Here is 3E: https://github.com/johnpaulbin/DALLE-models/releases/tag/model

May I ask, in what configuration was the model trained, and for how long was the 3EPOCH trained?

Sure!:)
The name-coding can also be found in https://github.com/robvanvolt/DALLE-models/readme.md
16L_64HD_8H_512I_128T_cc12m_cc3m_3E
means 16 layers, 64 head dimensions, 8 heads, 512 image dimensions, 128 text dimensions trained on cc12m and cc3m for 3 epochs, which was around one week on a 3090 RTX.


Ci failing with missing file error

I am experimenting with knapsack pro for a project with circleci. My tests are randomly failing with this error. am i missing some configurations ?
Errno::ENOENT: No such file or directory @ rb_sysopen - .knapsack_pro/tracker/prerun_tests_RSpecAdapter_node_1.json

This looks similar to the error from another user https://github.com/KnapsackPro/knapsack_pro-ruby/pull/165
Most likely you have a typo in the bind method called in spec helper:
Please add at the beginning of your spec/rails_helper.rb or spec/spec_helper.rb:
require 'knapsack_pro'
KnapsackPro::Adapters::RSpecAdapter.bind

This is from the installation guide: https://docs.knapsackpro.com/knapsack_pro-ruby/guide/
Please ensure the name of RSpec has 2 capital letters RS in the RSpecAdapter name.
Let me know if this helps.

That is unlikely to be the case as I have written the correct name

By any chance do you see errors about no permission access while writing to the directory .knapsack_pro?
knapsack_pro gem creates the .knapsack_pro directory for temporary files inside of your project directory.
I'm wondering if your issue could be related to this https://github.com/KnapsackPro/knapsack_pro-ruby/issues/158
Could you share all errors you see (full backtrace). If you don't want to share it publicly please send an email to the support https://knapsackpro.com/contact
I could also check what's recorded on our API side for your account by knowing your email address.

This issue has been solved. The root cause of the issue was that Dir.pwd was getting reset in one spec which caused the knapsack_pro gem process to not find the .knapsack directory as knapsack_pro uses a relative path to fetch config. Dir.chdir with a block in the user's codebase is not thread-safe by design.


Authorize running tests with action_policy

(i18n integration is not released yet in the gem, and it has a few quirks, but the code is much better with it)

@david-yz-liu I think I'll go with the flattening, to make the policy functionality easier to understand rather than mimic the policy logic exactly.
The text will be a bit repeated and christmas tree-like, that's the con side.

@david-yz-liu all done, but I'll have to file a bug report with the gem for the reasons not properly tracked between dependent policies

This is the first step to resolving Issue #3577


Use correct rpc account type for eth_getAccount

Describe the feature
use correct account type in rpc #9632
blocked by https://github.com/alloy-rs/alloy/pull/1075
Additional context
No response

@mattsse should I add the type in core.rs?


Skip init when precompiling

I thought __init__() was always skipped when precompiling, but this appears to not be the case (at least on Windows). This appears to fix https://github.com/stillyslalom/PyThermo.jl/issues/7, and may also fix https://github.com/cjdoris/PythonCall.jl/issues/266, https://github.com/JuliaPy/PythonPlot.jl/issues/25, and https://github.com/JuliaPy/PythonPlot.jl/issues/27.

@cjdoris could you approve the CI run?

What is missing to merge this commit? This bug that this commit fixes is causing issues with my co-workers when they are trying to install PythonPlot...

Pinging @cjdoris for any comments on what's blocking merge. He may just be on vacation, as I don't see any GitHub activity for about a month.

Hi yeah I've been away.
My worry with this is what happens if a package Foo that depends on PythonCall has an __init__ function which does pyimport("something")? Unless Foo.__init__ is also guarded in the same way, it assumes that PythonCall.__init__ has run already, and therefore will error because it hasn't.
Or have I misunderstood and it all works fine? I'm going to experiment a bit.

Also the linked issues all look like the same issue of conda install mysteriously failing during precompilation, which I think has been reported before but not resolved. Really efforts should be concentrated on fixing that.

I would love to figure out the underlying issue, but this band-aid PR makes this (very annoying) problem go away. Can we merge it as a minimally-intrusive stopgap?

It is not minimally intrusive, it actually very easily breaks any downstream packages in the way I suspected above.
If I make a package Foo containing just
using PythonCall
function __init__()
    pyimport("sys")
end

then precompiling Foo fails because it runs Foo.__init__() which calls pyimport("sys") which only works if PythonCall.__init__() has already run.
I don't think this general idea can work, so I'm going to close this PR. Another option may be to initialise Python not in __init__ but whenever pyimport and friends are actually called - but this would be a larger change to the codebase.

But Foo shouldn't call __init__() during precompilation in the first place - the fact that it does is a bug that will probably need to be resolved within Julia itself. PythonPlot.jl already uses the suggested escape hatch to avoid the same bug: https://github.com/JuliaPy/PythonPlot.jl/blob/63ed915e9a3abcffa24552129d00b7f1b9099a56/src/init.jl#L147


Really slow

Definitely some bad request magic going on on the backend.

Fixed using some special async optimizations


updating the redux skeleton

It'd be awesome if someone could update the redux skeleton on a regular basis. React especially because it's moving fast. I'd also add stage-0 babel to it, await / async is awesome.

async/await are on stage 3, so no need for stage-0. You are welcome to send a PR to appropriate repo:
https://github.com/brunch/with-redux
https://github.com/brunch/with-react

oh nice. For sure, once hit any barriers in my project I'll have to PR. thanks ❤

There is peerDependencies for this stuff, because constantly updating repos is not quite fun. But there are also compat risks implementing it.

@shvaikalesh btw I'm also having issue #1443 with this skeleton

Will take a look at it shortly.


Try to work around inability to auto-trigger CI checks on updatecli PRs

This PR follows up on #25, #24, and #19 by trying to work around the inability to automatically trigger required CI checks in branches/PRs opened by updatecli (e.g. as in #23). This is actually a security restriction (see https://github.com/orgs/community/discussions/55906 ). The solutions (making a PAT or defining+installing a new GitHub App in the PlanktoScope organization) would be a bureaucratic hassle, so this PR attempts to make it possible for me to manually trigger CI checks by enabling auto-merge.

I need to merge this PR to test its changes, so I will merge it now.


tinycc: refactor

get rid of rec

extreme use of recursive rec

format version string according to commit

cb89165fb9a5e80a05356af4d75ed826d7d417e0 (#280312, RFCs 107 and 147)

strictDeps
meta.mainProgram

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Next grand step: remove that dependency on darwin.applesdk.callpkg dark magic.


Separating test phase in different file & accepting test specific hyperparameters

🚀 Feature
Allow users to separate "Train + Validation" and "Test" phases into separate files and allow extra hyperparameters in Test phase.
Motivation
Often, research codes and trained models shared by researchers are consumed by developers who only care about testing on the trained model and not retraining it. It makes sense if users can break their PyTorchLightning module into two different files with the test file accepting extra hyperparameters specific to testing, e.g. "How many samples to draw from a trained VAE".
Pitch
One script train.py for training + validation:
class TrainPLModule(pl.LightningModule):
    def __init__(self, all, my, hps):
        self.save_hyperparameters()
        # all my hyper params can be accessed from self.hparams
    def training_step(self, ...): ...
    def validation_step(self, ...): ...

system = TrainPLModule(all=ALL, my=MY, ...)
trainer.fit(system)

.. another test.py for testing
class TestPLModule(TrainPLModule):
    def test_step(self):
        # self.hparams are all indeed accessible, BUT ..
        # I want new hparams specific to testing process
        self.hparams.test_specific_hp # NOT AVAILABLE

system = TestPLModule.load_from_checkpoint('/trained/model',
                                            all=NEW_ALL, # overwrites the original HP
                                            # but this WON'T show up in self.hp in test_step()
                                            test_specific_hp=12
trainer.test(system)

Alternatives
We can define all Hyperparameters in train.py and overwrite them in test file after loading the model, but testing process maybe large and have lots of testing specific HPs which are totally unnecessary for training (e.g. VAE latent space sampling/walking, creating UMAP/TSNE embedding from latent codes etc).
Additional context
Opening this feature request on @tchaton's suggestion on this discussion https://github.com/PyTorchLightning/pytorch-lightning/discussions/8581

Dear @dasayan05,
Definitely an interesting idea ! Would you be interested to work on this ?
Best,
T.C

@tchaton I would love to give it a try.
So, the reason .load_from_checkpoint(..) rejects unknown kwargs is clearly this line here:
https://github.com/PyTorchLightning/pytorch-lightning/blob/963c26764682fa4cf64c93c5a7572ae0040e9c32/pytorch_lightning/core/saving.py#L193
We can simply tweak some code around this to inject the unknown parameters into model.hparams. Is this a reasonable thing to do ? @tchaton do you have any other suggestion ?


If there is no current successful workflow, get a hash collision error.

I am using last-successful-build-actionv4 and it works fine if there is a successful workflow deployment. However on a new branch with a new deployment the stage will fail and think there is a hash collision.
I have tried with a fetch depth of 0 and 100.
The intended behavior would be to allow the step to pass, and give the current hash as described in the readme.md.
Error message:
Verifying git version...
Valid git version found: (2.40.0)
Running on a push event...
Error: Similar commit hashes detected: previous sha: d40191fadf586b8e5f5d68b1dc4619c005430c27 is equivalent to the current sha: d40191fadf586b8e5f5d68b1dc4619c005430c27.
Error: Please verify that both commits are valid, and increase the fetch_depth to a number higher than 50.
Error: Process completed with exit code 1.

@birkses  I've tried replicating this with a new branch and a new workflow that has not yet been run, which resulted in a workflow not found Error

I've also tried it with a new branch and a workflow that has not successfully completed yet (also one which has not yet run in that branch), which resulted in the current hash being returned

Are there any other details that may be necessary to reproduce this issue?

Gday, @domi2120,
Sorry for the late reply, I have been unable to replicate the issue myself as well. Closing this issue as cant repro.
Thanks for you time.
Regards


Expose common window resolutions

What problem does this solve or what need does it fill?
Users often need to set specific window resolutions. For example, a player will probably want a dropdown menu for a 3D game to tweak the resolution. A game designer might want to use a specific resolution for stylistic reasons. This is also relevant for 2D camera setups, see this chapter of the cheatbook.
As a result, lots of Bevy users will need to define their own resolution constants, resulting in unnecessary boilerplate.
What solution would you like?
Add resolution constants somewhere. My suggestion is adding constructors to WindowResolution like this:
let lowres = WindowResolution::from_360p();
let highres = WindowResolution::from_2k();
// etc.

But other options like associated constants would also work
What alternative(s) have you considered?
Leave it as is. Users can copy-paste their own resolution constants or create a third-party crate.
Additional context
We don't need to add all resolutions, that would be way too much. But something like the 10 most common ones should be feasible.
This issue was originally raised by @Sapein on Discord

I think an enum here is actually probably ideal, with an iterable list and then an Into UVec2.

@alice-i-cecile that's a pretty good idea. I like that better :).

Closing as Not Planned, per @IceSentry in #14158.