Made the modification

📋 Description
Changes in the css features of the website
colour, font size .weight, and many more
<-- #98   and issue no : #85 -->

✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[ yes] I have followed all the guidlines mentioned in CONTRIBUTING.md.
[ yes] My code follows the style guidelines of this project.
[ yes] I have performed a self-review of my own code.
[yes ] I have commented my code, particularly in hard-to-understand areas.
[ no] I have made corresponding changes to the documentation (if applicable).
[yes ] I have tested it locally and it works fine.
[ no] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

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

[ ] New feature (non-breaking change which adds functionality) ✨

[ ] UI enhancement (non-breaking change which enhances UI) 🎨

[ ] Documentation update 📚

Fixes #85

Thank you and please review and label me @Harshdev098

I am closing this PR, the changes are not much effective


[release/2.1] 2.1.1 fixes, combined

Rolls up https://github.com/dotnet/source-build/pull/598 (PR) plus(merged) fixes for #608, #609, #586.

There was a new 2.1.1 build (published to the 2.1 channel) that I've added a commit to uptake. I hear it's branding fixes for sdk and websdk.

I'm not sure yet why we weren't seeing the branding issue; from the changes I think we should have been and I'm continuing to look at it.  I think we also need to add <EnvironmentVariables Include="PB_IsStable=$(UseStableVersions)" Condition="'$(UseStableVersions)' != ''" /> here to be completely correct.

Ah, it appears the packages that are produced with the incorrect versions are unpacked and the contents included, so we never see the incorrect version.  For the roslyn-tools issue, I think the only place we would actually produce bad packages would be MSBuild, where we're manually setting the version anyway - so this change shouldn't be necessary for 2.1.1, although I think it would be good long-term.

If CI infra ends up too flaky and we want to go ahead with 2.1.1 without the last two commits, dabf6ed passed CI except an infra snag with CentOS7.1 Release. I kicked off a build to run the leg on that commit. (I don't think it'll update the CI status indicators because it's a manual build, unfortunately.)

@dotnet-bot test OSX10.12 Release please (java.nio.channels.ClosedChannelException)

This did get the versioning updates (that we didn't really need):
Successfully created package '/mnt/resource/j/workspace/dotnet_source-build/release_2.1/CentOS7.1_Release_prtest/src/websdk/bin/Release/Microsoft.NET.Sdk.Web.2.1.301.nupkg'.
Successfully created package '/mnt/resource/j/workspace/dotnet_source-build/release_2.1/CentOS7.1_Release_prtest/src/sdk/artifacts/Release/packages/Microsoft.NET.Sdk.2.1.301.nupkg'.

@dotnet-bot test OSX10.12 Release (java.nio.channels.ClosedChannelException)

@dotnet-bot test OSX10.12 Release please (looks like a timeout, but didn't take long enough for one.  no other error message.  machine lost without Jenkins realizing it?)


retry on #81

@lindsaycarr - review?
This PR competes with your #107. I'm proposing that we close #107 and merge this one instead.
This PR follows the same approach as #107 of modifying createProfile() rather than findProfileYaml(). The major difference is that this PR will accept a file path directly, with a default of ~/.vizlab and notes in the parameter description about other valid options. This offers the flexibility I think is needed to resolve #81.
I let some clutter into this PR, sorry. There are some minor edits in other files. The essential changes are all in the block of red and green that starts here: https://github.com/USGS-VIZLAB/vizlab/commit/51ec4544ab704cb92aa7c5743cd121345153257e#diff-020451bb865d7e219edcb3e30c710de4L22, plus some changes in that file to reduce code redundancy.

Coverage increased (+1.2%) to 80.902% when pulling 53fb788a2fc16dc6aa0935fea3a60e938a58da45 on aappling-usgs:master into beb668a8e88217f37a44bd2c101421b2e39044cb on USGS-VIZLAB:master.

@aappling-usgs this looks good to me! Could you update the documentation for findProfileYaml? It still references the home or relative options for createProfileYaml.

Yep, good idea!

Coverage increased (+1.2%) to 80.902% when pulling 5ffd6aa0617a42895e0c5e20704e2632d49d22a8 on aappling-usgs:master into beb668a8e88217f37a44bd2c101421b2e39044cb on USGS-VIZLAB:master.


Fix GitHub Actions deprecation warning

Background
Fixes some commands and jobs that have been deprecated or are scheduled to be disabled in GitHub Actions.
Details

Node.js 12 actions are deprecated

actions/checkout@v2
docker/setup-qemu-action@v1
docker/setup-buildx-action@v1
docker/login-action@v1
docker/build-push-action@v2

The set-output command is deprecated and will be disabled soon
The ubuntu-18.04 environment is deprecated

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


Add easing and overshoot

The final dial, which moves on its own, was a little bland.  This offers
some variation to its speed as well as a little "overshoot" motion so
that it feels more like a compass wavering around the item.
There is probably still some work to be done around nailing the easing
functions, right now it is using a cubic.  You can see a list of easing
functions over here:
https://gist.github.com/gre/1650294
as well as how they actually look over here:
https://easings.net/
The bounce isn't quite done using easing, but rather through
overshooting and wandering back recursively.
This commit also redoes the way that directionality was determined.  By
using -1 and 1 instead of 0 and 1 it becomes possible remove some
repeated code (since we were either adding or subtracting, now we can
just multiply by the direction in those cases).
This gif shows the code in action

This would resolve #27

@slifty Do you see these eslint warnings?

Compiled with warnings.

./src/scripts.js
Line 30:  Unexpected mix of '&&' and '||'      no-mixed-operators
Line 31:   Unexpected mix of '&&' and '||'      no-mixed-operators
Line 31:   Unexpected mix of '||' and '&&'      no-mixed-operators
Line 31:  Unexpected mix of '||' and '&&'      no-mixed-operators
Line 44:  Expected '!==' and instead saw '!='  eqeqeq


Adding pcdfilter to documentation index for indigo/jade/kinetic/lunar

Please see the error message in the Travis log on how to update your patch to pass. You can also run the tests locally.


error in ssh client

ssh client ask for password in all methods, even I already added it in the creation of ssh client.
it works correctly only in file_copy  method
ssh_client = j.clients.ssh.get(name="ssh_test_client", addr =IP, port=PORT, login=LOGIN,
passwd=password)
ssh_client.file_copy("/tmp/test", "/tmp/test")

in execute method for example it ask for password
ssh_client.execute(cmd="ls /home/")
<EMAIL_ADDRESS>password:

Should pass interactive parameter:
ssh_client.execute("ls /home/", interactive=False)
because the default of interactive = True resulting in asking for the password instead of passing it

works with me without identify the interactive flag
JSX> ssh_client = j.clients.ssh.get(name="ssh_test_client", addr='<IP_ADDRESS>', port=22, login='pishoy', passwd='xxxxx')
JSX> ssh_client.execute(cmd="ls /home")
backup	pishoy
Connection to <IP_ADDRESS> closed.
(0, '', '')

JSX> ssh_client.file_copy("testfileincontainer","/tmp/test")

jsx version

3BOTDEVEL:3bot:jumpscaleX_core: git log -1
commit 33253cf64e9a48fd24dbc7299fc1b0c5a0bf361b (HEAD -> development, origin/development, origin/HEAD)
Author: xmonader<EMAIL_ADDRESS>Date:   Sun Nov 10 17:24:35 2019 +0200

    don't start myjobs in jsx wiki-load, depends on 3bot to have it

interactive option doesn't exist in almost all methods in ssh_client.
Ex:

ssh_authorize.
download.
upload.
portforward_to_local.
portforward_kill.

and all of them require password when you execute them.

Issues that will be addressed:

[ ]  ssh_authorize, mosh : will add interactive option
[ ] download , upload: use j.sal.fs.copyDirTree(...) which do not provide interactive option
[ ] portforward_to_local, portforward_kill : cmd executed should had password passed in case of using password


Typescript Supporting?

Hi,
Is this library support for typescript? I want to use in Angular 2+. Thanks :)

There are no plans to offer support for typescript.

could i add a typescript definition for your library? :)

If you like to. Your PR is welcome.

Hi @commenthol,
I have created a new typescript definitions for your library and tried in my angular project.
I have created a PR to DefinitelyTyped. :)
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/35901

Wouldn't it be better to add the type definitions within this package, instead of adding this to an external dependency?

Closing due to issue resolved by PR in DefinitelyTyped package.


Karpenter replicas cause metrics service to wrongly route requests

Version

Karpenter: v0.16.1

Kubernetes: v1.22.11-eks-18ef993
Expected Behavior
When a controller is using replicas=2 or more the metrics should be available in any of the replicas.
Actual Behavior
By adding replicas in #2311 the idea was to implement the rolling update strategy and improve the availability of the Karpenter controller. When implementing this change we realised our Karpenter metrics were no longer properly collected by Prometheus. We discovered that the controller which is not currently active doesn't export metrics correctly and exports mainly zeroes and NaN. If the service route the traffic of http-metrics to the inactive controller, it wrongly exposes incomplete metrics causing metric collection disruption.
Steps to Reproduce the Problem
Add replicas=2 and check metric exposition at the service level.

Hi @tlk-kayrros, can you describe what the specific issue is here? If you are using a ServiceMonitor, Prometheus will just use the service for endpoint discovery, but will still scrape metrics from all pods so I don't believe the routing issue that you described above will occur with Prometheus ServiceMonitor

Hello @tlk-kayrros ,
I looked into this a bit further and this seems to be related to how our Prometheus scrape targets are configured.  The "karpenter" service is added as a static target which means we are not scraping each Karpenter pod individually. In short, only one of the pods will be scraped at each scrape interval.
Here is the config I'm referring to:  https://github.com/aws/karpenter/blob/main/website/content/en/preview/getting-started/getting-started-with-eksctl/prometheus-values.yaml#L10-L14
I think the best option here is use the pod or endpoint service discovery mechanism to scrape both pods at every scrape interval.  Then make sure the predefined queries for Karpenter are aggregating those metrics properly.

Hello tlk-kayrros@,
I just pushed up a PR which changes the scrape target to use endpoint discovery instead of a static scrape target.  Let me know if you think this will solve your problem.

It will probably solve the issue, yes. Thanks a lot

Closing this out as the PR is now merged.  Feel free to reopen if you feel the issue is not solved.


Search/filter functionality

I don't believe the search functionality works.
Using the demo at http://stylemark-bootstrap.surge.sh/ I type dropdown into the search box and hit return.
The page reloads with the URL http://stylemark-bootstrap.surge.sh/?filter=dropdown but nothing is different. The full page is loaded and at the top of the page.
Am I missing something?

Hey @ay13, the search filter in the sidebar is a live type-to-search that doesn't require you to press ENTER, but it looks like pressing ENTER refreshes the page when it shouldn't. Until that's fix, just avoid pressing ENTER after typing your search query.


Relay server crashes with LoopExit

When Sascha started the version 0.20.2 of the relay, it crashed with traceback:
2021-03-25 09:48:38,193 INFO     [main] trustlines: Starting relay server version 0.20.2
2021-03-25 09:48:38,217 INFO     [main] web3provider: Autodetect provider from uri ws://ws.ethereum.trustlines.tlbc.parity.trustlines-relay-0001
2021-03-25 09:48:38,219 INFO     [main] web3provider: Autodetected WebsocketProvider
2021-03-25 09:48:38,221 WARNING  [main] relay: No account configured
2021-03-25 09:48:38,221 INFO     [main] node: Assuming connected to parity node: Enabling parity-only rpc methods.
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/opt/relay/lib/python3.8/site-packages/sentry_sdk/integrations/threading.py", line 69, in run
    reraise(*_capture_exception())
  File "/opt/relay/lib/python3.8/site-packages/sentry_sdk/_compat.py", line 57, in reraise
    raise value
  File "/opt/relay/lib/python3.8/site-packages/sentry_sdk/integrations/threading.py", line 67, in run
    return old_run_func(self, *a, **kw)
  File "/usr/lib/python3.8/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/opt/relay/lib/python3.8/site-packages/web3/providers/websocket.py", line 38, in _start_event_loop
    loop.run_forever()
  File "/usr/lib/python3.8/asyncio/base_events.py", line 570, in run_forever
    self._run_once()
  File "/usr/lib/python3.8/asyncio/base_events.py", line 1823, in _run_once
    event_list = self._selector.select(timeout)
  File "/opt/relay/lib/python3.8/site-packages/gevent/selectors.py", line 201, in select
    self._ready.wait(timeout)
  File "src/gevent/event.py", line 163, in gevent._gevent_cevent.Event.wait
  File "src/gevent/_abstract_linkable.py", line 521, in gevent._gevent_c_abstract_linkable.AbstractLinkable._wait
  File "src/gevent/_abstract_linkable.py", line 487, in gevent._gevent_c_abstract_linkable.AbstractLinkable._wait_core
  File "src/gevent/_abstract_linkable.py", line 490, in gevent._gevent_c_abstract_linkable.AbstractLinkable._wait_core
  File "src/gevent/_abstract_linkable.py", line 442, in gevent._gevent_c_abstract_linkable.AbstractLinkable._AbstractLinkable__wait_to_be_notified
  File "src/gevent/_abstract_linkable.py", line 451, in gevent._gevent_c_abstract_linkable.AbstractLinkable._switch_to_hub
  File "src/gevent/_greenlet_primitives.py", line 61, in gevent._gevent_c_greenlet_primitives.SwitchOutGreenletWithLoop.switch
  File "src/gevent/_greenlet_primitives.py", line 65, in gevent._gevent_c_greenlet_primitives.SwitchOutGreenletWithLoop.switch
  File "src/gevent/_gevent_c_greenlet_primitives.pxd", line 35, in gevent._gevent_c_greenlet_primitives._greenlet_switch
gevent.exceptions.LoopExit: This operation would block forever
    Hub: <Hub '' at 0x7f4b5e8c0100 epoll pending=0 ref=0 fileno=7 thread_ident=0x7f4b5e85c700>
    Handles:
[]

An important change in between 0.20.1 and 0.20.2 is I upgraded the relay to make it work with python3.8 which means I had to update gevent from 1.4.0 to 21.1.2.
I can see related issues with the same problem https://github.com/gevent/gevent/issues/1698 https://github.com/kimbauters/ZIMply/issues/6 but I am not sure why we do monkey.patch_all(thread=False) with thread=False in the boot https://github.com/trustlines-protocol/relay/blob/master/src/relay/boot.py#L8
We cannot reproduce the problem locally or on the staging relay server. Ralf suggested that it could be due to Sascha running Sentry and us not.
It appears we run Sentry on the devel server, I could try the update there.
Ralf also mentioned the fact that web3 uses asyncio which could be a problem.
Another difference is that Sascha uses a webscoket URL while we use http

The problem was identified as coming from the use of websocket for the node rpc in the relay. Web3 uses asyncio to work with websockets and it does not cooperate well with gevent. I spent time reading about it and fiddling to try to find a solution but it is appears impossible.
I believe we should drop the support for websockets and revert to http.


Networkclick

First time submitting a PR for a non-python project, hopefully its not too sloppy.
Basically I wanted to be able to launch wicd-curses when I click on the network so I took the code from text.cpp and added it to network.cpp to allow configuration of click events in the config file. I then wanted the same thing for the cpu.cpp module so I could launch htop. Both appear to work well from my local testing and have added a ton of useful functionality to my polybar setup.
Ideally I wanted this to add the click events to the prefix (which I use mainly for icons in my polybar setup), however I couldnt get that to work. Happy to have another crack at it though if someone could point me in the right direction.

Closing due to inactivity
Feel free to reopen this if you still want to work on it.


drivers:can: Fix R-Car driver can_stop behaviour

Calling for can_stop() method on R-Car boards was not emptying FIFOs and tx_msgq.
We use controller reset mode to empty controller FIFOs.
We are emptying tx_msgq with "-ENETDOWN" return code.
Tested using can shell module ans obtained the following behaviour:
#Board disconnected from CAN bus
uart:~$ can start can@e6c30000
starting can@e6c30000
uart:~$ can send can@e6c30000 001 DE AD BE EF
enqueuing CAN frame #1 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 4
uart:~$ can send can@e6c30000 001 DE AD BE
enqueuing CAN frame #2 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 3
uart:~$ can send can@e6c30000 001 DE AD
enqueuing CAN frame #3 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 2
uart:~$ can stop can@e6c30000
stopping can@e6c30000
failed to send CAN frame #1 (err 141)
failed to send CAN frame #2 (err 141)
failed to send CAN frame #3 (err 141)
#Connect board to CAN bus
uart:~$ can start can@e6c30000
starting can@e6c30000
uart:~$ can send can@e6c30000 001 DE AD BE EF
enqueuing CAN frame #4 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 4
CAN frame #4 successfully sent

First three frames has been properly dropped and the last one has been properly sent.
Fix #50546

@aaillet ping

@aaillet ping

I haven't forgotten, I'm on vacation, I'll take care of it tomorrow !

So you were right, RX filters weren't kept with the changes I proposed (dropped by resetting controller).
Instead of switching to reset mode I'm now switching to halt mode and manually resetting RX FIFO.
My changes weren't passing can api test suite, new version is passing the test suite as before.
By testing the driver manually using can shell, I can confirm we are:

Dropping pending transmission, emptying RX FIFO and returning an error for each message in RX msg (ENETDOWN @henrikbrixandersen can you confirm it's the right error code to return ?) when stopping controller
Keeping RX filters when restarting controller (we are also receiving frames received when controller was stopped)
Not altering controller config as we are still able to send message after a controller restart (stop+start)

*** Booting Zephyr OS build zephyr-v3.3.0-2972-g3a08d9620ae5 ***
Starting Zephyr shell. (rcar_h3ulcb_cr7)
# Controller unlinked from can bus
uart:~$ can start can@e6c30000
starting can@e6c30000
uart:~$ can send can@e6c30000 001 DE
enqueuing CAN frame #0 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 1
uart:~$ can filter add can@e6c30000 001
adding filter with standard (11-bit) CAN ID 0x001, CAN ID mask 0x7ff, data frames 1, RTR frames 0, CAN-FD frames 0
filter ID: 0
uart:~$ can stop can@e6c30000
stopping can@e6c30000
failed to send CAN frame #0 (err 141)
# Link controller to can bus
uart:~$ can start can@e6c30000
starting can@e6c30000
uart:~$
     001   [1]  de
uart:~$ can send can@e6c30000 001 DE
enqueuing CAN frame #1 with standard (11-bit) CAN ID 0x001, RTR 0, CAN-FD 0, BRS 0, DLC 1
CAN frame #1 successfully sent
uart:~$ can stop can@e6c30000
stopping can@e6c30000
# Sending frames from Linux host while controller stopped
uart:~$ can start can@e6c30000
starting can@e6c30000
     001   [1]  de
     001   [2]  de ad
     001   [3]  de ad be
     001   [4]  de ad be ef
uart:~$
# Sending frames from Linux host while controller started
     001   [5]  de ad be ef de
     001   [6]  de ad be ef de ad

Only thing I don't understand is why frames are dropped with error code 141 while -ENETDOWN is error 100, as can shell is directly reporting error from can api can_send, I don't see where the translation is done.

Keeping RX filters when restarting controller (we are also receiving frames received when controller was stopped)

This doesn't sound right. That likely means the controller also sent ACKs for those frames while stopped? After calling can_stop(), the CAN controller must not interact with the network.

Dropping pending transmission, emptying RX FIFO and returning an error for each message in RX msg (ENETDOWN @henrikbrixandersen can you confirm it's the right error code to return ?) when stopping controller

Yes, -ENETDOWN is correct in this case.

Keeping RX filters when restarting controller (we are also receiving frames received when controller was stopped)

This doesn't sound right. That likely means the controller also sent ACKs for those frames while stopped? After calling can_stop(), the CAN controller must not interact with the network.

@aaillet Perhaps re-use what is done in can_rcar_set_timing() where registers are manually restored after entering/existing reset mode?

Keeping RX filters when restarting controller (we are also receiving frames received when controller was stopped)

This doesn't sound right. That likely means the controller also sent ACKs for those frames while stopped? After calling can_stop(), the CAN controller must not interact with the network.

I'll investigate on what is going on, I'm surprised that even in halt mode, the controller is ACKing frames.

I'll investigate on what is going on, I'm surprised that even in halt mode, the controller is ACKing frames.

@aaillet Ah, I think I know what is going on. When in "halt" after having called can_stop(), the remote node attempts to transmit a frame but receives no ACK. The remote node continuously attempts to retransmit the frame and once the local node exits "halt" after calling can_start(), it receives and ACKs the retransmission.
If you can confirm this behaviour (that the remote node does not receive any ACKs when calling can_stop() on the local node, using "halt" mode should be fine.

I'll investigate on what is going on, I'm surprised that even in halt mode, the controller is ACKing frames.

@aaillet Ah, I think I know what is going on. When in "halt" after having called can_stop(), the remote node attempts to transmit a frame but receives no ACK. The remote node continuously attempts to retransmit the frame and once the local node exits "halt" after calling can_start(), it receives and ACKs the retransmission.
If you can confirm this behaviour (that the remote node does not receive any ACKs when calling can_stop() on the local node, using "halt" mode should be fine.

@aaillet Can you confirm the above behaviour? It would be nice to have this fix in for v.3.4.0.

I'll investigate on what is going on, I'm surprised that even in halt mode, the controller is ACKing frames.

@aaillet Ah, I think I know what is going on. When in "halt" after having called can_stop(), the remote node attempts to transmit a frame but receives no ACK. The remote node continuously attempts to retransmit the frame and once the local node exits "halt" after calling can_start(), it receives and ACKs the retransmission.
If you can confirm this behaviour (that the remote node does not receive any ACKs when calling can_stop() on the local node, using "halt" mode should be fine.

@aaillet Can you confirm the above behaviour? It would be nice to have this fix in for v.3.4.0.

Yes, I need to figure out how to know if my remote node (linux using cansend with a canusb module) received the ack.
I'll get back to you ASAP !

Hey @henrikbrixandersen,
I just validated that when controller is in halt mode (after calling for can_stop) it doesn't send any acknowledgment to my remote node.
I also validated that my remote node is "repeatedly sending the frame on the CAN bus until there is an acknowledgement from at least one other node."
I can then confirm the behavior you were thinking about !
I do think now that my patch is correctly fixing can_stop behavior. (still passing can api test suite and working well when tested using can shell).


ContentView.swift:36:6 Referencing initializer 'init(_:configuration:filter:keyPaths:sortDescriptor:)' on 'ObservedResults' requires that 'Person' conform to 'Identifiable'

How frequently does the bug occur?
 Realm()
//// Persist your data easily with a write transaction
//try! realm.write {
//    realm.add(dog)
//}

struct ContentView: View {
    @ObservedResults(Person.self) var persons

    var body: some View {
        List {
            ForEach(persons) { person in
                Text(person.name)
            }
            .onMove(perform: $persons.move)
            .onDelete(perform: $persons.remove)
        }.navigationBarItems(trailing:
                                Button("Add") {
            $persons.append(Person())
        }
        )
    }
}

struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}

I follow the code in the ReadMe, but failed with error:
ContentView.swift:36:6 Referencing initializer 'init(_:configuration:filter:keyPaths:sortDescriptor:)' on 'ObservedResults' requires that 'Person' conform to 'Identifiable'
Stacktrace & log output
No response
Can you reproduce the bug?
Always
Reproduction Steps
No response
Version
newest
What Atlas Services are you using?
Both Atlas Device Sync and Atlas App Services
Are you using encryption?
No
Platform OS and version(s)
newest
Build environment
Xcode version: ...
Dependency manager and version: ...
build with error

This because the Dog class conforms to the ObjectKeyIdentifiable protocol it provides a default way for realm objects to conform to the SwiftUI Identifiable protocol.
To get your code working you can either make your class Person conform to ObjectKeyIdentifiable:
class Person: Object, ObjectKeyIdentifiable {
...
}

Or you can also make  your class Person conform to Identifiable:
class Person: Object, Identifiable {
    @Persisted(primaryKey: true) var _id: String
   var id: String {
    return _id;
}
    ...
}

You could also simply rename the _id field to id:
class Person: Object, Identifiable {
    @Persisted(primaryKey: true) var id: String
    ...
}

Closing this as @PhilippeWeidmann answer is right


Error when presenting HelpCentre

Hey,
Trying to update an older Titanium app, I came across your module for Intercom (which is ideal) - however I am getting the following error when trying to present the help centre, or messenger:
[INFO]  [Intercom] ERROR - Cannot present Messenger because no user is registered - you need register a user first -> Showing fallback error screen. Debug info: { has_identity: 1, has_user: 1, has_user_identification: 0 }
I have used the Intercom.registerUser function in the prior line, I noticed another project after googling suggested to the person to use the newer 14 SDK - I wondered if you have either came across this issue, or have plans to update to SDK 14?
Thanks in advance!

Just a note that it seems to work if I call Intercom.registerUser() without any data, but obviously that means that it's not being linked to the user?


Syntastic error when opening Vim 8.0

Hello! First of all thanks for a really amazing plugin... it's easily one of my favorites! 👍
I just upgraded MacVim to 8.0 using Homebrew (with --override-system-vim), and now I get an error when starting Vim:
MBPr:~ sam$ vim

Command terminated

syntastic: error: exception running system('uname'): Vim(let):E484: Can't open file /var/folders/9r/wxwcny8j3vvc4ld4y4s11kps73lqjk/T/vB6R69U/0
Error detected while processing function syntastic#util#system:
line   19:
E121: Undefined variable: g:_SYNTASTIC_DEBUG_CHECKERS
E116: Invalid arguments for function syntastic#log#debug
Press ENTER or type command to continue

My ~/.vimrc is down to the bare bones:
""""""" GENERAL
set nocompatible               " We're running Vim, not Vi!
filetype off                   " Disabled just for Vundle to do its magic.
" Set the runtime path to include Vundle and initialize.
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()

Plugin 'scrooloose/syntastic'

call vundle#end()              " Required for Vundle.

And here's everything in my ~/.vim (excluding stuff in ~/.vim/bundle since only syntastic should be loading):
MBPr:~ sam$ find ~/.vim | grep -v /bundle/
/Users/sam/.vim
/Users/sam/.vim/.netrwhist
/Users/sam/.vim/autoload
/Users/sam/.vim/autoload/pathogen.vim
/Users/sam/.vim/autoload/plug.vim
/Users/sam/.vim/autoload/vimclojure.vim
/Users/sam/.vim/bundle
/Users/sam/.vim/colors
/Users/sam/.vim/colors/darkspectrum.vim
/Users/sam/.vim/colors/inkpot.vim
/Users/sam/.vim/colors/inkpot.vim.old
/Users/sam/.vim/doc
/Users/sam/.vim/doc/checksyntax.txt
/Users/sam/.vim/doc/clojure.txt
/Users/sam/.vim/doc/syntastic.txt
/Users/sam/.vim/doc/tags
/Users/sam/.vim/ftdetect
/Users/sam/.vim/ftdetect/clojure.vim
/Users/sam/.vim/ftdetect/scala.vim
/Users/sam/.vim/ftplugin
/Users/sam/.vim/ftplugin/clojure
/Users/sam/.vim/ftplugin/clojure/completions-clojure.core.txt
/Users/sam/.vim/ftplugin/clojure/completions-clojure.set.txt
/Users/sam/.vim/ftplugin/clojure/completions-clojure.xml.txt
/Users/sam/.vim/ftplugin/clojure/completions-clojure.zip.txt
/Users/sam/.vim/ftplugin/clojure.vim
/Users/sam/.vim/ftplugin/python.vim
/Users/sam/.vim/indent
/Users/sam/.vim/indent/.scala.vim.swp
/Users/sam/.vim/indent/clojure.vim
/Users/sam/.vim/indent/python.vim
/Users/sam/.vim/indent/scala.vim
/Users/sam/.vim/plugin
/Users/sam/.vim/sessions
/Users/sam/.vim/sessions/armada.vim
/Users/sam/.vim/sessions/default.vim
/Users/sam/.vim/sessions/default.vim.lock
/Users/sam/.vim/sessions/k.vim
/Users/sam/.vim/sessions/kubectl-diff.vim
/Users/sam/.vim/sessions/y.vim
/Users/sam/.vim/syntax
/Users/sam/.vim/syntax/clojure.vim
/Users/sam/.vim/syntax/scala.vim

Any idea as to what might be going on?

This is a broken install, most likely inflicted by Vundle.  Please check that your syntastic folder is identical to the latest master HEAD.

On a side note: the plugin's address is now vim-syntastic/syntastic (that shouldn't have anything to do with your problem though).

Turns out that my bash itself was broken and I didn't even realize it. 😱 I fixed it with brew uninstall --force bash && brew install bash. Sorry to bother you - thanks for the help!


feat: add insecure support for oci repo

fix: https://github.com/helmfile/helmfile/issues/920

@Yeep could you help to test this feature? I will help you if you don't know how to build.

@Yeep I want to know the true use case. could you help me?

@Yeep got it.

@Yeep updated! waiting for your feedback.

@yxxhero That's fixed the login issue, thanks.
I'm now getting a certificate error on pull.
tls: failed to verify certificate: x509: certificate signed by unknown authority
I think this is because configuration for OCI registries isn't stored by helm so the helm pull command also requires the --ca-file flag.
ARGS:
  0: helm (4 bytes)
  1: pull (4 bytes)
  2: oci://docker.artifactory.local/my-chart (75 bytes)
  3: --version (9 bytes)
  4: ~0.1.0-0 (8 bytes)
  5: --destination (13 bytes)
  6: /tmp/helmfile625625529/namespace/my-chart/~0.1.0-0 (63 bytes)
  7: --untar (7 bytes)

@Yeep ok. got it.

@Yeep updated.

@Yeep ping

@yxxhero Sorry, I'm having some problems with the HTTPS certificate on helm registry login. I wanted to make sure it was definitely a problem with Helm or my infrastructure rather than Helmfile.
If I don't provide a username/password Helmfile now correctly pulls from an OCI registry with a private CA. If I do provide a user/password Helmfile fails with this error:
Error: Get "https://docker.artifactory.local/v2/": remote error: tls: handshake failure
But I get the same failure calling helm registry login directly and the arguments being passed to Helm look correct so I'm happy with Helmfile's behaviour here.

@mumoshu @xiaomudk tested locally. all is ok. please review.

@mumoshu ping

@mumoshu fixed conflicts. thanks so much.


Add prefer-reactive-destructuring rule

From the docs:

This rule triggers whenever a reactive statement contains an assignment that could be rewritten using destructuring. This is beneficial because it allows svelte's change-tracking to prevent wasteful redraws whenever the object is changed.
This svelte REPL example shows just how effective this can be at preventing bogus redraws.
<script>
  /* eslint svelte/prefer-reactive-destructuring: "error" */
  /* ✓ GOOD */
  $: ({ foo } = info)
  $: ({ bar: baz } = info)

  /* ✗ BAD */
  $: foo = info.foo
  $: baz = info.bar
</script>

I've noticed that sometimes what the rule report doesn't lead to a reduction in render times.
I think the documentation might need to be fixed.
See https://github.com/ota-meshi/eslint-plugin-svelte/pull/208#discussion_r942125022

@tivac What are the remaining tasks of the PR? I can do the remaining tasks.

@baseballyama Needs to be changed to a stylistic rule and any remaining comments need to be addressed (I think it's just capitalization nit?). Seems like either svelte has improved object access change tracking or my initial tests when I wrote these rules years ago were faulty 🤷🏻♂ So this is really a style preference over a perf thing.

I checked this rule well, and now I understand this rule doesn't have an impact on rendering frequency.
And already ESLint can report destructuring through prefer-destructuring rule.
So I think even in terms of styling, I don't think we need to add this rule.
I would like to clarify the needs of this rule.

Thank you for the explanation.

And already ESLint can report destructuring through prefer-destructuring rule.
So I think even in terms of styling, I don't think we need to add this rule.

You are right. So I will close this PR.


Coding style for conditions

I'm creating this issue to specifically tackle the subject of condition formatting already mentioned in #87. I've also added some suggestions I had in mind after seeing some of the library's code. I'm ready to fix these, but of course I'll only do it if there's a general agreement that it should be done!
1. Inverted conditions
In C# code, we usually write conditions in which the subject of the test is written on the left of comparison operators, and the value we compare it to comes on the right. So for example, we would write:
if (someObject.Method() > 7)

instead of:
if (7 < someObject.Method())

Today most of the library uses the second form. I propose we change them to use the standard .NET order.
2. Explicit boolean comparisons
The library usually uses explicit boolean comparisons, which is also unusual in C#, where we usually see implicit comparisons. Instead of:
if (someObject.Method() == true)

we would write:
if(someObject.Method())

This change doesn't really do anything else than making the code more concise, and make it closer than what we usually see in C#.
3. String comparisons
When looking for an exact equality between two strings, there is no difference in C# between string1 == string2 and string1.Equals(string2) (except in the case where string1 is null). Currently, the Fido2 library uses a lot of comparisons using Equals like we would use in Java.
In order to improve in comparisons across the library, I propose that we compare == instead of Equals, when there is no difference between the two.

There it is. What do you think ?

@abergs I believe we've all our condition logic to read right to left. Can we close this issue?

I agree


Switching between scenes with module installed makes scenes load as blanks.

Loading up my world for the first time will allow the initial scene to load, but switching between scenes will cause every new scene to load as blank.

This module is no longer supported in recent versions of foundry, because it's functionality was added to core.


tortoise: compute correct global and local threshold

In the codebase, we default to 6/10 and 2/10 for global and local thresholds respectively. We discussed the rationale behind these values on 1st of February (details are recorded in the call). The issue is to revisit these values based on that information.

changes discussed in https://community.spacemesh.io/t/requirements-for-local-and-global-thresholds/235 and https://community.spacemesh.io/t/uncounted-weight-and-the-global-threshold/228 will be implemented (sorry for private links)


merge perl advances

bump to 0.1.7

add optional loader nonstrict option to allow certain reader errors. see https://github.com/ingydotnet/yaml-libyaml-pm/issues/44

add new default dumper option indentless_map, for backcompat with yaml readers.
It changes the default indent behavior of map elements,
advance the indent the children of the map
according to the spec: 6.1 Indentation Spaces:
  "Each node must be indented further than its parent node.  All sibling nodes
  must use the exact same indentation level.  However the content of each
  sibling node may be further indented independently."
  http://yaml.org/spec/1.2/spec.html#id2777534

This make it readable by YAML.pm (perl),
and behaves the same as all other YAML writers.

reformat: return and goto in seperate lines, to be able to set breakpoints.

fix most pedantic compiler warnings, remove unused code

fix .travis: make test => check

spec-1.2 rule [53]. Support "/" for JSON compatability

The YAML 1.2 spec section 6.1 also says:

The “-”, “?” and “:” characters used to denote block collection entries are perceived by people to be part of the indentation. This is handled on a case-by-case basis by the relevant productions.

So "for backcompat with yaml readers" in fact means "for compatibility with YAML readers that do not understand some specific cases of YAML indenting rules".

Yes, I know that, but YAML.pm does know that, and libyaml is the only YAML dumper which interprets the specs for maps like this.
And the word “perceived by people” is much weaker than the sentence "Each node must be indented further than its parent node”.
Thus the default needs to be according to the specs and compatible with yaml dumpers and readers, even if some people see the -  as part of the indentation.
"perceived by some people" and "case by case" is not so important as the default reader for perl used in CPAN.pm, which errors on this unusual and breaking interpretation.

Since @ingydotnet rewrote master, this is in merge conflict. I'm closing this for now. Feel free to send revised pull requests for each of the points in your original description.


[Model] Support Starcoder2

This PR supports Starcoder2 model.
Here is demonstration:
tlopex@tlopex-OMEN-by-HP-Laptop-17-ck1xxx:~/mlc-llm$ mlc_llm chat dist/starcoder2-7b-q4f16_1-MLC \
>              --device "cuda:0" --overrides context_window_size=2048 \
>              --model-lib ./dist/libs/starcoder2-7b-q4f16_1-cuda.so
[2024-07-14 22:43:31] INFO auto_device.py:79: Found device: cuda:0
[2024-07-14 22:43:31] INFO engine_base.py:143: Using library model: ./dist/libs/starcoder2-7b-q4f16_1-cuda.so
[22:43:31] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "local", max batch size will be set to 4, max KV cache token capacity will be set to 8192, prefill chunk size will be set to 2048.
[22:43:31] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "interactive", max batch size will be set to 1, max KV cache token capacity will be set to 16384, prefill chunk size will be set to 2048.
[22:43:31] /home/tlopex/mlc-llm/cpp/serve/config.cc:668: Under mode "server", max batch size will be set to 80, max KV cache token capacity will be set to 135185, prefill chunk size will be set to 2048.
[22:43:31] /home/tlopex/mlc-llm/cpp/serve/config.cc:748: The actual engine mode is "interactive". So max batch size is 1, max KV cache token capacity is 16384, prefill chunk size is 2048.
[22:43:31] /home/tlopex/mlc-llm/cpp/serve/config.cc:753: Estimated total single GPU memory usage: 6213.544 MB (Parameters: 3972.174 MB. KVCache: 1112.598 MB. Temporary buffer: 1128.772 MB). The actual usage might be slightly larger than the estimated number.
You can use the following special commands:
  /help               print the special commands
  /exit               quit the cli
  /stats              print out stats of last request (token/sec)
  /metrics            print out full engine metrics
  /reset              restart a fresh chat
  /set [overrides]    override settings in the generation config. For example,
                      `/set temperature=0.5;top_p=0.8;seed=23;max_tokens=100;stop=str1,str2`
                      Note: Separate stop words in the `stop` option with commas (,).
  Multi-line input: Use escape+enter to start a new line.

>>> Hello, how are you?
Well. Nice to meet you
>>> Who are you?
I'm a helpful and honest assistant
>>> Good!
It's nice to meet you too

I'll then complete the tp function of this model after merged.

cc @tqchen @MasterJH5574


Update HelloWorld.cpp

test2

Summary by CodeRabbit

New Features

Introduced a new Java application that responds with "Hello Docker World!" at the root URL.
Added a main entry point to launch the application.

Bug Fixes

No changes to the existing C++ functionality, ensuring continued performance.

Preparing review...

Preparing PR description...


OneHotEncoder inverse_transform with datetime64[ns] does not preserve other column dtypes

I'm not sure if this is an issue with dask, dask-ml or something else altogether. I've noticed after calling inverse_transform where the dataframe contains a column of datetime64[ns], the non-categorical columns all become 'object' type.
As a side effect, you can't directly do df.to_parquet('output') on the inverse_transform dataframe. df.compute().to_parquet('output') works sometimes.
Version info:

dask.version
'2.10.1'
dask_ml.version
'1.2.0'

Errors I've seen so far:
ValueError: Error converting column "id" to bytes using encoding UTF8.

Dask Name: getitem, 275 tasks

Thanks for the example.
I'm not sure I'd expect the inverse_transform to operate when you have an array (which you are with the .values). That's going to be upcast to object dtype, so all the type information is lost.

A bit simpler example. Perhaps there is something that could be done, not sure

In [17]: df = dd.from_pandas(pd.DataFrame({"A": pd.to_datetime(['2000', '2001']), "B": pd.Categorical(['a', 'b'])}), 2)

In [18]: de = DummyEncoder()

In [23]: result = de.inverse_transform(de.fit_transform(df).to_dask_array(lengths=True))

In [24]: result.dtypes
Out[24]:
A      object
B    category
dtype: object

In [25]: result.compute()
Out[25]:
           A  B
0 2000-01-01  a
1 2001-01-01  b

Regardless, I'd recommend avoiding the conversion to object dtype.

Thanks for the quick feedback and explanation.
This is a bit of a rabbit hole for me since I ran into the same issue as in #362 (though the error message is worded slightly differently now.) So I tried the suggested workaround passing .values and then couldn't get it to output to parquet nicely.
For now I'll just recast the columns back to the dtypes I expect. Sorry for the noise!
P.S. any idea when #362 might be resolved?

I'm not sure about #362. If you're interested in working on it, we'd be grateful.

What's left in this issue?
We have https://github.com/dask/dask-ml/issues/362 about DummyEncoder handing non-numeric divisions.
I don't think we should encourage mixing datetimes and numeric dtypes into an object-dtype (which is what .values does).  Rather, a ColumnTransformer should be used.

Thanks. I'll take a look at 362 and see if there's something I can do there.


Add support for Symfony

Add latest Nodejs (18.x)
Add support for Vi editor

Hi @MagePsycho ,
I think there might be an easier way to achieve this?
Would it be possible to change https://github.com/cmtickle/docker-stack/blob/master/docker/nginx/m-servers.conf#L22
to :
root /var/www/htdocs/${http_host}/${root_subfolder}/;

Then introduce a new map in https://github.com/cmtickle/docker-stack/blob/master/docker/nginx/m-hosts.conf
like :
#################################################################
# Provide facility to change the root_dir to something else
#################################################################
map $root_subfolder $MAGE_RUN_CODE {
    hostnames;
    #example of a symfony host
    #symfony.loc public;

# DEFAULT - LEAVE AS LAST ENTRY IN MAP
    .local pub;
    .loc pub;
    .localhost pub;
    .test pub;
}

Hi @MagePsycho , I'm glad it's acceptable to you.
If you wanted to use an application which does not have a subfolder you could achieve this as below:
map $http_host $root_subfolder {
    hostnames;
    #example of a symfony host
    #symfony.loc public;
   #example of a host with no subfolder
   #wordpress.loc '';

# DEFAULT - LEAVE AS LAST ENTRY IN MAP
    .local pub;
    .loc pub;
    .localhost pub;
    .test pub;
}

I think at that point we'd need additional server blocks per your original suggestion.
But for now, I think we can limit the impact of your change by adding maps and the single edit per my response?

Tested locally and all works as expected. Thanks for your contribution @MagePsycho !


Add host-based routing constraints

Context
We want to split routing between the "School Placements" and "Track & Pay" services within the Rails application. In production, these URLs are likely to be manage-school-placements.service.gov.uk and claim-funding-for-general-mentors.service.gov.uk.

School Placements have confirmed that "Placements" is their desired top-level namespace.
Track & Pay have confirmed that "Claims" is their desired top-level namespace.

Changes proposed in this pull request

Add 2 routing files for the separate services, config/routes/claims.rb and config/routes/placements.rb
Update app/views/layouts/application.html.erb to use service name header based on current_service helper.

Guidance to review
By updating your local /etc/hosts file with the following lines, you can access both services without running a proxy server:
<IP_ADDRESS> claims.localhost
<IP_ADDRESS> placements.localhost

Link to Trello cards

https://trello.com/c/Mvow5DdE/50-add-route-constraint-for-track-pay
https://trello.com/c/US9I2gnJ/49-add-route-constraint-for-school-placements

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated or added to the Azure KeyVault

Turns out I was setting ENV["..._HOST"] to http://... :/

It appears this breaks logging in as a persona -- root_path is not defined.
no_root_path.mov

😅 Yup. It's fixed now :)


Does it work?

@sunloverz What do you mean? I'll close this for now. Please, if you're experiencing any problem open a more detailed issue, thank you very much :)

It doesn't work with angular 6.1
ERROR in src/app/reservations/reservations.component.ts(2,10): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'CalendarSchedulerEventAction'.
src/app/reservations/reservations.component.ts(2,40): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'CalendarSchedulerEvent'.
src/app/reservations/reservations.component.ts(2,64): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'SchedulerViewDay'.
src/app/reservations/reservations.component.ts(3,11): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'SchedulerViewHour'.
src/app/reservations/reservations.component.ts(3,30): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'SchedulerViewHourSegment'.
src/app/reservations/reservations.component.ts(3,56): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'subPeriod'.
src/app/reservations/reservations.component.ts(3,67): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'addPeriod'.
src/app/reservations/reservations.component.ts(3,78): error TS2305: Module '"/home/tyler/renting/node_modules/angular-calendar/angular-calendar"' has no exported member 'endOfPeriod'.
src/app/reservations/reservations.component.ts(17,11): error TS2304: Cannot find name 'CalendarPeriod'.
src/app/reservations/reservations.component.ts(30,21): error TS2304: Cannot find name 'endOfDay'.
src/app/reservations/reservations.component.ts(30,30): error TS2304: Cannot find name 'addMonths'.
src/app/reservations/reservations.component.ts(58,72): error TS2304: Cannot find name 'AppService'.
src/app/reservations/reservations.component.ts(91,22): error TS2304: Cannot find name 'CalendarPeriod'.
src/app/reservations/reservations.component.ts(141,14): error TS2339: Property 'refresh' does not exist on type 'ReservationsComponent'.

@sunloverz Sorry, readme is not updated, I wll update it now. However you have to use CalendarView
(part of angular-calendar library) instead of CalendarPeriod.
Moreover you've imported my component from angular-calendar but you must import them from angular-calendar-scheduler :)
Let me know if the issue is solved after that fixes.
Thank you


Update to beta.29

Updated the homebrew formula to use beta.29.

sorry I didn't notice this PR at the time. It's outdated now, so I'll close it. But thank you @suppenburger for the contribution. I'll make a better job at watching my github notifications next time.


Create playlist data visualisation page

Create a page with the following elements:

Sidebar displaying all playlists from a user in the form of a list
User is able to click on a playlist and select to preview its analytics
Main container where user can preview playlist analytics as plots

[ ] Sidebar displaying playlists
[ ] Main analytics container
[ ] Playlist buttons styling
[ ] Showing analytics for playlist

Made significant changes to the front-end with the use of the dash-bootstrap-components package. The package will be added to the requirements.txt when the branch is merged.

Dashboard now displays flexible dash plot that can show how often artists appear on a playlist.


Installer fails to launch after PC restart

After installing WolvenKit.Installer, the program instantly crashes when launched for the first time after each PC restart.
After launching a second time, WolvenKit 8 stable and nightly are unable to be installed via WolvenKit.Installer. The "Install" button is greyed out. WolvenKit 7 versions are available to install.
Lastly, older versions of WolvenKit.Installer seem inaccessible. I attempted to downgrade to an older version to see if that would work, but the appinstaller forces the latest (<IP_ADDRESS>) regardless of which appinstaller version is ran.

Installing WolvenKit versions fixed by deleting existing "c:\ModdingTools\WolvenKit" folder.
First launch crashing still persists.

Installer version is not currently supported :) Just extract


Make S3 overwrite fails visible

🚀 🚀 Pull Request
Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Overwrites on S3 can fail without raising errors but with errors in the server response.
Check the response so that the errors do not go unnoticed.

Things to be aware of

Things to worry about

Additional Context

can you add tests to cover the changes?


Bind syntax should work with slots

Is your feature request related to a problem? Please describe.
Currently, bind syntax is limited only to components that have children, but if we could extend this to allow for any function-like prop, we'd be able to make binding work with existing core components, such as <For /> and others, although these should probably be ported anyways.
Describe the solution you'd like
The syntax is definitely WIP, but something like the following should be possible:
view! {
  <For each key bind:item@view>
    <Fragment slot:view>

{item}

    </Fragment>
  </For>
}

Describe alternatives you've considered
All we need is a way to communicate with the view! {} macro that we need to associate a binding with a child...perhaps even something like this would be better:
view! {
  <For each key>
    <Fragment fn_slot:view bind:item>

{item}

    </Fragment>
  </For>
}

Additional context
N/A

I was confused by the title here: bind: does work with slots, it's just that it only works with a children prop.
See #1773 for an alternate way of doing this for For. Doing it in a non-breaking way would be great, of course! But adding the nested <Fragment> just for the sake of being able to use a slot seems less than ideal to me.

The idea is to allow non-children slots to be used:
#1484 fn Comp(fn_prop: impl Fn(Param) -> impl IntoView, children: Children) -> impl IntoView {
  /* ... */
}

view! {
  <Comp>

I go in the `fn_prop` prop: {param}

I go in the `children` prop

   </Comp>
}

Fragment is only needed in order to "name" where you want the slot to go. It's only needed if you need to group multiple elements/components. This is because the fragment symbol is <></>, which cannot have attributes, therefore < slot:fn_prop></> is invalid.

Fragment is only needed in order to "name" where you want the slot to go. It's only needed if you need to group multiple elements/components. This is because the fragment symbol is <></>, which cannot have attributes, therefore < slot:fn_prop></> is invalid.

I think we can refer to Vue, use <template>.  When slot:_ exists on <template>, discard all attributes except let:_.
#[component]
fn Comp(fn_prop: impl Fn(Param) -> impl IntoView, children: Children) -> impl IntoView {
  /* ... */
}

view! {
   <Comp>

I go in the `fn_prop` prop: {param}

I go in the `children` prop

   </Comp>
   <Comp>
    <template slot:fn_prop let:param>

I go in the `fn_prop` prop: {param}

------

------

    </template>

I go in the `children` prop

   </Comp>
}


Bug fix

Codecov Report
Patch coverage: 61.11% and project coverage change: -0.23% :warning:

Comparison is base (a574e63) 91.69% compared to head (8dbbd63) 91.46%.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #115      +/-   ##
============================================
- Coverage     91.69%   91.46%   -0.23%
- Complexity     1146     1148       +2
============================================
  Files           111      111
  Lines          2686     2696      +10
  Branches        263      266       +3
============================================
+ Hits           2463     2466       +3
- Misses          115      118       +3
- Partials        108      112       +4

Files Changed
Coverage Δ

...src/main/java/cn/crane4j/core/util/ClassUtils.java
71.42% <0.00%> (-11.91%)
:arrow_down:

...r/operation/TypeDynamitedDisassembleOperation.java
81.25% <25.00%> (-18.75%)
:arrow_down:

...ne4j/extension/spring/aop/AutoOperatePointcut.java
80.00% <50.00%> (-10.00%)
:arrow_down:

.../core/support/aop/AutoOperateAnnotatedElement.java
92.30% <66.66%> (-7.70%)
:arrow_down:

.../core/parser/TypeHierarchyBeanOperationParser.java
96.05% <100.00%> (ø)

...pport/aop/AutoOperateAnnotatedElementResolver.java
84.61% <100.00%> (+1.28%)
:arrow_up:

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


Pylance type detection could be smarter if the return type of a function depends on the argument type.

See this code:
from typing import Union
# The return type depends on x's type.
def multiply_by_2(x: Union[int, list[int]]):
    if isinstance(x, int):
        return x * 2
    elif isinstance(x, list):
        return [i * 2 for i in x]
    else:
        raise TypeError

Actual behavior:
Pylance just lists all the possible return types.

When I do type hints to narrow the type of foo, pylance reports an error.

And an ugly solution is to use an assert statement every time multiply_by_2 appears. Why do I call it ugly? What if there are many function calls in my code? There will be a lot of redundant assert statements.

Expected behavior:
Pylance can detect the return type according to the parameter type(that may be a little difficult)
Or don't report errors when I do type hints.

Pylance's behavior in this case is prescribed by the typing standard defined in PEP 484. Accordingly, other type checkers like mypy act the same.
Thankfully, PEP 484 considered this case, and it provides a way to annotate a function or method whose return type depends on the argument type(s). This mechanism makes use of a decorator called @overload. You would use it like this:
from typing import overload

@overload
def multiply_by_2(x: int) -> int:
    ...

@overload
def multiply_by_2(x: list[int]) -> list[int]:
    ...

def multiply_by_2(x: Union[int, list[int]]):
    if isinstance(x, int):
        return x * 2
    elif isinstance(x, list):
        return [i * 2 for i in x]
    else:
        raise TypeError

Given the above, I'm going to close this.


Crash in minecraft fabric 1.19.4

I don't know if the issue is from EEC or OWO. But I only have the required mods and it crashes.
All loaded mods:

Full log:
MultiMC version 0.7.0-3863.txt

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

I won't lie, installing a password-protected file doesn't feel good security-wise. So I won't do it.

REAL SECURITY ISSUE

Also it seems the user who just gave me this file (OtakuFra) has been hacked by the same file (judging by his comment on other issues)

Don't download this file, I hope it gets taken down fast.

Anyway, I think that the issue lies in the reach entity mod version, it seems it should be 2.3.2 not 2.4.0 (for miencraft 1.19.4)


Fix #693: Add project url to BOM

Why: the absence of a project url causes validation errors at Nexus
Type of change
Select the type of your PR

Bugfix

Description
Please describe your pull request
Additional Context
Why are you making this pull request?
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonar-Lint warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

I've verified in dry-run mode that the bom now gets signed.
find /var/folders/5s/v5576rp175bf4vrf15mgvffm0000gn/T/tmp.nSfJRywH/io/kroxylicious/ -name "*.asc" | grep bom
/var/folders/5s/v5576rp175bf4vrf15mgvffm0000gn/T/tmp.nSfJRywH/io/kroxylicious//kroxylicious-bom/0.3.0/kroxylicious-bom-0.3.0.pom.asc


EM and Dp

Hi Apollo team,
I would like to ask two questions. What is the role of EM(Expectation Maximization) here? I couldn't find its implementation in planning layer. Also, it seems that DP(Dynamic Programming) is being used in planning, what is the problem that DP is used to solve?
Thank you!

The current algorithm is composed of two DP and two QP(quadratic programming). The two DPs are used as a decision provider and QPs are used to generate smooth trajectories based on the decision results. The results of one planning cycle can be feed into the next planning cycle, and that is what EM refers to.

Thanks. I assume the result of current planning cycle is a trajectory and next planning cycle utilizes that (history). Initially, I thought there might be some stochastic formulation regarding the expectation but perhaps not. Thanks anyway!


Implement KEP-19 by distinguishing between app and operator version

What this PR does / why we need it:
The app version of an operator package is taken into account using operations that apply to packages. E.g., when installing a package, a user is able to provide a specific app version to install. To further distinguish app version and operator version, the 'version' field in operator packages and the package index has been renamed to 'operatorVersion'.
Package version resolution has been updated to take the app version into account; operator packages are first sorted by app version, then sorted by operator version. Furthermore, the app version is added to the package tarball name to avoid conflicts.
Notes for reviewers:
Most of the changes update the existing tests and their test data. Functional changes are:

Update CLI commands to have app-version and operator-version flags
Update package resolvers to take app version into account
Update the tarball writer and repo index to include app version in tarball names
Update the PackageVersions struct to be sorted first by app version, then by operator version

The part of the E2E test that tests the current operators repository is expected to fail because the repository needs to be updated to reflect that version has been renamed to operatorVersion.

locally TestAddContainers failed in integration-tests

this resulted in a package named Package created: /Users/kensipe/repo/cassandra-3.11.4_.tgz  :(

it's the operators fault!   on it.

Thanks @kensipe for taking care of providing a new repository!
Is there a branch for this in the operators repo? Asking because the e2e tests currently check out master of operators and tests the operators. As the operator definitions aren't compatible with the changes of this PR, the E2E tests currently fail. Do you plan to update the operators repo once this PR has landed?

e2e test failing due to out of sync operator repo.  fixed with https://github.com/kudobuilder/operators/pull/184
rerunning tests after merge


I am having a problem with live share

Product and Version [VS/VSCode]:
OS Version [macOS/Windows]:
Live Share Extension Version:
Target Platform or Language [e.g. Node.js]:
Steps to Reproduce / Scenario:

@Conman1119, can you please specify what is the problem that you are facing? Are you able to share/join into Live Share session?

@Conman1119: are you still having issues using VS Live Share?  I'm going to close this out for the moment but if you're still experiencing problems we'd love to help resolve them. If you can provide specific steps on the issue you're encountering we can reopen and triage this item.


Mock good practices

Hi @mvandervoord I would like to test some modules and I cannot figure out how to do.
Please consider I have the following modules :

Item module :  Item.c / Item.h / test_Item.c
ListItem module : ListItem.c / ListItem.h / test_ListItem.c

Item module define the following functions :

void InitItem(ITEM_t *Item) : function that initialise every field of ITEM_t typedef structure
int getItemData(ITEM_t *Item) : return the content of internal int Data variable
void setItemData(ITEM_t *Item, int Data) : int Data setter

ListItem module defines the following functions :
typedef struct{
   ITEM_t Items[3];
}LISTITEM_t;

void InitListItem(LISTITEM_t *ListItem)
{
   for(int i=0; i<3; i++)
      InitItem(ListItem->Items[i]);
}

ITEM_t GetItemFromList(LISTITEM_t *ListItem, int Index)
{
      return ListItem->Items[Index];
}

Then I want to test ListItem :
First, I mock Item.h in order to check InitListItem() calls InitItem() 3 times (with correct arguments in my real code).
Then I want to check GetItemFromList returns the Item corresponding to Index.
#include "unity.h"
#include "listitem.h"

#include "mock_item.h"

void setUp(void)
{
}

void tearDown(void)
{
}

void test_InitListItem(void)
{
    LISTITEM_t ListItem;
    for(int Index = 0; Index < 3; Index++)
        InitListItem_Expect(&ListItem.Items[Index]);
}

void test_GetItemFromList(void)
{
    LISTITEM_t ListItem;
    InitListItem(&ListItem);

    setItemData(&ListItem.Items[0], 0);
    setItemData(&ListItem.Items[1], 1);

    ITEM_t TestItem;
    TestItem = GetItemFromList(&ListItem, 1);
    TEST_ASSERT_EQUAL_UINT16(1, getItemData(&TestItem));
}

I have some issues with GetItemFromList() I would like to call Item module functions to initialise and access variables but I cannot use the real and mocked versions at the same time.
In this case : InitListItem(&ListItem); calls InitItem() and I can use InitItem_Ignore() but I cannot Ignore Getter and Setter and continue using them.
How do you usually figure it out ?

Using mock_Item.h is definitely the option you want. You don't want to include item.h also.
If you focus on what you are attempting to prove with each test, you learn what corners can be cut. With your test test_InitListItem, you are proving that the init function gets called three times, as expected. You also want to make sure that the items are getting stuffed in the correct order. That's it. That's the test.
To do that, mine might look something like this:
void test_InitListItem(void)
{
    LISTITEM_t test_items[3] = { { 11 }, { 22 }, { 33 } }; // It doesn't need to make sense, it just needs to be unique

    // If it's a reasonable number, I unwind loops in tests. We're not looking to duplicate the original code.
    // we're verifying it does what it should. we can be explicit
    InitListItem_Expect(&test_items[0]);
    InitListItem_Expect(&test_items[1]);
    InitListItem_Expect(&test_items[2]);

    // finally it calls the function being tested
    InitListItem(test_items);
}

Notice I didn't do anything that deals with the actual fields. I didn't require any knowledge about the internal workings of the type even... I just proved what I needed to. If I WANTED to initialize the type in a way that made sense, I COULD... but it's more maintenance and it's not actually validating anything further (thought it might reduce warnings from the compiler for the test itself).
In any case, the test for getting items is the same. I don't need to retest the get and set functions. Those are tested in test_Item.c. Here I just need to make sure it grabbed the correct item.
void test_GetItemFromList_SHOULD_ReturnCorrectItem(void)
{
    LISTITEM_t test_items[3] = { { 11 }, { 22 }, { 33 } };

    InitListItem_Expect(&test_items[0]);
    InitListItem_Expect(&test_items[1]);
    InitListItem_Expect(&test_items[2]);

    InitListItem(test_items);

    // In this case, we're testing ALL the possible getter indices... but I usually test 3-5 gets
    // one is the lowest index, one the highest, and then one or more in the middle.
    // I also like to put them in a random order to make sure it's not something order dependent
    TEST_ASSERT_EQUAL_MEMORY(test_items[1], GetItemFromList(test_items, 1), sizeof(LISTITEM_t));
    TEST_ASSERT_EQUAL_MEMORY(test_items[0], GetItemFromList(test_items, 0), sizeof(LISTITEM_t));
    TEST_ASSERT_EQUAL_MEMORY(test_items[2], GetItemFromList(test_items, 2), sizeof(LISTITEM_t));
}

I'd also write another test which verifies your desired behavior when you attempt to grab an out of range value. I don't know what your desired behavior IS. There doesn't seem to be an error code being returned and it's not a pointer, so it can't be null... so I went with it returning a default item:
void test_GetItemFromList_SHOULD_ReturnDefaultForInvalidRequests(void)
{
    // The default item might require you to be more correct with the structure
    LISTITEM_t test_default = { 5, 777, 22, "sandwhich" };
    LISTITEM_t test_items[3] = { { 11 }, { 22 }, { 33 } };

    InitListItem_Expect(&test_items[0]);
    InitListItem_Expect(&test_items[1]);
    InitListItem_Expect(&test_items[2]);

    InitListItem(test_items);

    TEST_ASSERT_EQUAL_MEMORY(test_default, GetItemFromList(test_items, -1), sizeof(LISTITEM_t));
    TEST_ASSERT_EQUAL_MEMORY(test_default, GetItemFromList(test_items, 3), sizeof(LISTITEM_t));
}

Does that make sense?

Thank you very much for your valuable help !
I thought that the easier way to make sure it grabbed the correct item was a field comparaison but that does make sense now : I have to test modules without depanding on others already tested.
This will require some workout =)

Yes, it's definitely a mental workout! :)
Best of luck!


Using debian package

After installing the debian package, some get this error:
chown: cannot access '/var/lib/honeytrap-agent..

The Debian package doesn't seem to create the /var/lib/honeytrap/ and /var/log/honeytrap directories.


Enable perception with a single VLP-16 LiDAR

System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04): Linux, Ubuntu 18.04
Apollo installed from (source or binary): Source
Apollo version (3.5, 5.0, 5.5, 6.0): 6.0
Output of apollo.sh config if on master branch:  APOLLO_VERSION: master-2021-11-01-539839c714

Steps to reproduce the issue:

I am trying to connect a physical Velodyne VLP-16 LiDAR in order to enable perception. All of the config files suggest that the VLP-16 alone is not used (or at least recommended) for perception as the configuration files have been preset to use the VLS-128 as the main LiDAR along with multiple VLP-16s surrounding it. Is it possible to use just a single VLP-16 for perception without any sensor fusion (and by bypassing the fusion compensator modules entirely)?
There is limited documentation available on the usage of the VLP-16 standalone or even on the usage of slightly different sensor configurations other than what is officially supported. It would be great if there is some guide or documentation detailing how the LiDAR drivers work and steps to enable our own custom sensor configuration (either a single LiDAR or more).

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


rick changes

updating links and removing page. merge at convenience

this was merged. you'll most likely want to delete and refork.

so I've been editing my fork since yesterday commits.


Update WEPP

The National Soil Erosion Research Laboratory has announced the release of WEPP 2024, which an update since the last public release of WEPP 2012.  The Daily Erosion Project has been a soft release dated 17 Jan 2023, so will try out the new version and see how things go.

No observed difference in execution time and negligible regression suite difference in output.  About to run a smoke test.

Smoke test looks good.  Will let production ride with it overnight...

Had 3,439 run failures out of the 960,496 daily runs we do.  A common error
forrtl: severe (174): SIGSEGV, segmentation fault occurred
Image              PC                Routine            Line        Source
libc.so.6          00007FBF5E03E6F0  Unknown               Unknown  Unknown
wepp20240930<PHONE_NUMBER>49565A  mltbtm                    144  mltbtm.for
wepp20240930<PHONE_NUMBER>493A39  frostn                    593  frostn.for
wepp20240930<PHONE_NUMBER>47EE54  winter                    324  winter.for
wepp20240930<PHONE_NUMBER>40E4F2  contin                    935  contin.for
wepp20240930<PHONE_NUMBER>458B4E  main                      322  main.for
wepp20240930<PHONE_NUMBER>4041AD  Unknown               Unknown  Unknown
libc.so.6          00007FBF5E029590  Unknown               Unknown  Unknown
libc.so.6          00007FBF5E029640  __libc_start_main     Unknown  Unknown
wepp20240930<PHONE_NUMBER>4040C5  Unknown               Unknown  Unknown

Will debug shortly.

No more segfaults, about to declare victory on this...


SQLite3

Contact Details
<EMAIL_ADDRESS>Template URL
https://railway.app/template/jOiNFt
What does this template deploy?
Solves #419
It deploys SQLite database with WebUI (https://github.com/coleifer/sqlite-web) for easy management.
It also has the support for init_db.sql.
Demo project
No response

Nice work, @ThallesP.  I like that you met the requirement for easy management by including the SQLite web interface.
I did find some minor issues to sort out.
Mainly, the entrypoint file isn't successfully executing all logic:
Running database init ./entrypoint.sh: line 18: shasum: not found Running database init ./entrypoint.sh: line 25: can't open init_db.sql: no such file
Looks like you just need to COPY it into the container.
Also, even if you sort the issue with shasum, I don't think the logic to handle checking the hash of the init file is going to work to run only if changed, since the hash is hardcoded and you're not updating it.. unless I'm misunderstanding your intent.  I feel like you can toss that logic and make it so that the init_db.sql only runs on initial deployment.  I like the idea of allowing people to manage their schema and data from that file, over multiple deployments, but I feel like it adds unnecessary complexity to the entrypoint script.  Open to hearing your thought process of course!
Lastly, could you please also add a sample schema in the init_db.sql?  It doesn't have to be complex at all, maybe just a CREATE TABLE IF NOT EXISTS $TABLE_NAME with a few different columns, and insert a few lines of sample data into the table?

Nice work, @ThallesP. I like that you met the requirement for easy management by including the SQLite web interface.
I did find some minor issues to sort out.
Mainly, the entrypoint file isn't successfully executing all logic:
Running database init ./entrypoint.sh: line 18: shasum: not found Running database init ./entrypoint.sh: line 25: can't open init_db.sql: no such file
Looks like you just need to COPY it into the container.
Also, even if you sort the issue with shasum, I don't think the logic to handle checking the hash of the init file is going to work to run only if changed, since the hash is hardcoded and you're not updating it.. unless I'm misunderstanding your intent. I feel like you can toss that logic and make it so that the init_db.sql only runs on initial deployment. I like the idea of allowing people to manage their schema and data from that file, over multiple deployments, but I feel like it adds unnecessary complexity to the entrypoint script. Open to hearing your thought process of course!
Lastly, could you please also add a sample schema in the init_db.sql? It doesn't have to be complex at all, maybe just a CREATE TABLE IF NOT EXISTS $TABLE_NAME with a few different columns, and insert a few lines of sample data into the table?

Thanks, i'll sort out the entrypoint issue.
Regarding the hash, the intention was to execute it only when the user specifically wants to. This is due to the fact that if the user opts to deploy the template via the Railway dashboard, the deployment occurs automatically, and the repository is created along with it. As a result, there is no window of time available for the user to make modifications to that file before the deployment takes place.
I can probably separate that in a seed_db.sql file that does the above and the init_db.sql will include the sample data. What do you think?

For sure!  I like the idea of enabling users to add data to the db from within a file in the repo, so I do appreciate the effort you put in to adding this.  It feels like asking a lot of you to add more logic to account for two files and all that.  I think just making it straightforward so that init_db.sql is run when the container is deployed from the template, which creates a table or two and seeds some sample data is plenty to get people going.  Then you could eliminate the logic looking at the hash and everything altogether.
That said, if you are enjoying the thought exercise, I won't tell you not to do the extra work to set up a seed mechanism :)  I think people may find it useful if they don't have a frontend or just want a way to bulk insert data. If you want a quicker dialog, let me know and we can chat on Discord

For sure! I like the idea of enabling users to add data to the db from within a file in the repo, so I do appreciate the effort you put in to adding this. It feels like asking a lot of you to add more logic to account for two files and all that. I think just making it straightforward so that init_db.sql is run when the container is deployed from the template, which creates a table or two and seeds some sample data is plenty to get people going. Then you could eliminate the logic looking at the hash and everything altogether.
That said, if you are enjoying the thought exercise, I won't tell you not to do the extra work to set up a seed mechanism :) I think people may find it useful if they don't have a frontend or just want a way to bulk insert data. If you want a quicker dialog, let me know and we can chat on Discord

Hi, just implemented that.
Here's a link: https://sqlite3-production-08ef.up.railway.app/
Password: 3h5piei9kpx-4$9u
hope that im not billed that much for the preview :pray:

@ThallesP beautiful!!  I deployed it from scratch and then seeded some more data.  It's as easy as can be, I think.  Also I love the trains table haha
I have created this repo and invited you as a maintainer: https://github.com/railwayapp-templates/sqlite
Could you push your code to this repo then update the Template config to point there?  After that, please shoot me an email at<EMAIL_ADDRESS>and we'll start the payout process.
PS let me know if you need some credits on your account.  Would love to see you submit some more stuff!


Code stop with errors

When run the 'inversion' workflow with ‘slurm_sm’ system, the following problems always happened, and it is independent of the clusters and models that I used.  And inversion with the same models and parameters (except the parameters for slurm_sm) work well if I use the system option 'multithreaded'.  So I think model parameters for the inversion are correct.
Anyone has come across the problems and has any idea to solve them? Thank you very much!

Starting iteration 21
Generating synthetics
Computing gradient
Computing search direction
Computing step length
trial step 1
Starting iteration 22
Generating synthetics
Computing gradient
Computing search direction
Computing step length
trial step 1
Traceback (most recent call last):
File "/tmp/slurmd/job1005876/slurm_script", line 24, in
workflow.main()
File "/home1/03419/liren/seisflows-master/seisflows/workflow/inversion.py", line 128, in main
self.line_search()
File "/home1/03419/liren/seisflows-master/seisflows/workflow/inversion.py", line 185, in line_search
self.evaluate_function()
File "/home1/03419/liren/seisflows-master/seisflows/workflow/inversion.py", line 212, in evaluate_function
path=PATH.FUNC)
File "/home1/03419/liren/seisflows-master/seisflows/system/slurm_sm.py", line 125, in run
+ '%s ' % PAR.ENVIRONS)
File "/home1/03419/liren/seisflows-master/seisflows/tools/tools.py", line 31, in call
subprocess.check_call(*args, **kwargs)
File "/home1/03419/liren/anaconda2/envs/obspy1.0.3/lib/python2.7/subprocess.py", line 186, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command 'srun --wait=0 /home1/03419/liren/seisflows-master/seisflows/system/wrappers/run  /work/03419/liren/stampede2/tests/marmousi_offshore/output solver eval_func  ' returned non-zero exit status 1
1
When this error happed, the residual file for source number 11 (/scratch/evalfunc/residuals/000011) is not built,
OSError: [Errno 17] File exists: '/work/03419/liren/stampede2/tests/marmousi_offshore/scratch/evalfunc/residuals'
srun: error: c495-044: task 11: Exited with exit code 1
When the second error happed,  the code

exits

subprocess.CalledProcessError: Command 'srun --wait=0 /home1/03419/liren/seisflows-master/seisflows/system/wrappers/run  /work/03419/liren/stampede2/tests/marmousi_offshore/output solver eval_func  ' returned non-zero exit status 1

I know this is a late answer but it may help someone else in the future. This kind of error is typically due to a mix-up between parallel processes when writing files. This is why some "sleep" commands have been added (see for example in seisflows/tools/unix.py function mkdir).
When facing this kind of messages (which magically disappear when using a debugger) try to add a few seconds more sleep time in mkdir or elsewhere. Type grep -rni "sleep" * in seisflows main folder to see some examples of what has been done.


feat: Remove document notes from the datatracker

Fixes #6013

Do we really want to remove the note from the database? (I've been on the fence about this). We have the option of removing the ability to edit/make new ones, but show old ones when they existed.
If we want do want to remove it from the database, do we want to move the existing notes somewhere (a dump kept in  long-term offline storage, or shifting them to be comments on the document (using DocEvent)? Or is it ok just to throw the information away?

No, let's keep them in the db. What do I need to change?

don't remove the column from the models and remove the migration that came from that change. Then look at the places where you removed how its shown - the emails and all can stay as you've changed them I think, but the html views should show them still if the note is non-empty.
The editing views should all stay removed as the PR currently removes them - if any need to change in the future, the secretariat can[^1] change them using the admin.

^1 : We have a separate issue that the secretariat's access to the admin is spotty and needs tuning.


chore: add test for NewOverlayFromEthereumAddress

added 4 test cases under TestNewOverlayFromEthereumAddress  to test NewOverlayFromEthereumAddress of crypto library
Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[x] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@Aviksaikat consider re-writing it as a "table" test: https://dave.cheney.net/2019/05/07/prefer-table-driven-tests

Looks like I have messed up my branches


Item Activities should include the Base Damage Type if none is specified in the Attack Damage Details tab.

Certain Features (such as Monk's Martial Arts) allow for an override of an Item's damage formula. That override includes the Item's Base Damage Type and if a new Type is not specified, will result in a damage roll that has no type.
so, Item Activities should include the Base Damage Type if none is specified in the Attack Damage Details tab.
eg, the Monk Attack shown below should inherit whatever the Base Damage Type was for the weapon (in this case, it was a Scimitar, so it should be Slashing):

currently, it has to be manually set to work properly:

The idea of "none" damage being an intrinsically invalid state, which signals to the damage workflow that it should find the "nearest appropriate damage type(s)", may also serve to address #3754 in a more holistic manner.


Problem when start pulsemeeter

I installed pulsemeeter, when I open it, it shows the error below.
[kureshio@linux-casa ~]$ pulsemeeter
ERROR: invalid json grep: warning: stray \ before #
grep: warning: stray \ before #
grep: warning: stray \ before #
invalid json from server
App sinks returned an error, audio backend probably crashed. Server will be closed.
Could not join subscribe_thread (maybe there is none)
closing client handler threads...
closing socket
sending exit signal to listener thread, it should exit within 2 seconds...
Pulseaudio is installed.
Distro: Archlinux

same/similar issue
`$ pulsemeeter
Could not start vumeter for Virtual_Input_3
invalid json from server
App sinks returned an error, audio backend probably crashed. Server will be closed.
Could not join subscribe_thread (maybe there is none)
closing socket
(pulsemeeter:116154): Gtk-CRITICAL **: 02:58:00.856: gtk_main_quit: assertion 'main_loops != NULL' failed`

#59
This is an issue we have in the older version. If you are running pulseaudio/pulseaudio over version 16 it is fixed in the nightly version. The guide on how to install is on https://github.com/theRealCarneiro/pulsemeeter/wiki/Installation#Nightly-installationupdate-script .


Mention users

How do you mention a user in a message? Like with @ and the user's name. I've tried @ + Phone number, @ + Formatted Name but nothing works.
The mention should turn blue on the message and the user should recieve a notification

How was this solved?

Basically, I'm calling this JS code directly:
var chat = Store.Chat.get(ID);
chat.sendMessage(message, {linkPreview : null, mentionedJidList : [mentionedID], quotedMsg : null, quotedMsgAdminGroupJid : null});

ID is the id of the chat, where you want to send the message to. (ending with<EMAIL_ADDRESS>or<EMAIL_ADDRESS>mentionedID is the ID of the person that you want to tag. (ending with<EMAIL_ADDRESS>message is the message that you want to send, which needs to contain "@" so that the tagging works. Don't inclunde the<EMAIL_ADDRESS>with the number.

let user = await Store.Contact.serialize().find(x => x.id.user === 'phonenumber_without@c.us');
then chat.sendMessage(message, {linkPreview : null, mentionedJidList : [user.id], quotedMsg : null, quotedMsgAdminGroupJid : null});


[FLINK-17739][network][tests] Fix unstable ResultPartitionTest#testInitializeMoreStateThanBuffer

What is the purpose of the change
Fix unstable ResultPartitionTest#testInitializeMoreStateThanBuffer
Brief change log
Fix unstable ResultPartitionTest#testInitializeMoreStateThanBuffer
Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review @pnowojski  and unrelated failures in azure, merging!


[AutoRelease] t2-signalr-2023-03-07-93559(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3895
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1033

issue link:https://github.com/Azure/sdk-release-request/issues/3895


Test Issue created at Fri, 26 Mar 2021 14:15:44 GMT

Test issue body  Fri, 26 Mar 2021 14:15:44 GMT

Comment on issue at Fri, 26 Mar 2021 14:15:47 GMT


CUDA out of memory

My computer has 24G memory, but it still has the problem of CUDA out of memory,Choosing smaller pictures doesn't help either.
the command is like this:
IMG_PATH=./test_img/37.jpg
CLS_WRD='shoes'
NAME='shoes_test'
python main.py
--base configs/stable-diffusion/v1-finetune_patch_picture.yaml
-t --actual_resume ./checkpoints/sd-v1-4-full-ema.ckpt
-n $NAME --gpus 0,   --logdir ./logs
--data_root $IMG_PATH
--reg_data_root $IMG_PATH --class_word $CLS_WRD
Excuse me, is there any solution?

Hi, Thanks for your interest in our work. We haven't worked on this yet, but it could be a possible solution for you to change the optimizer to AdamW8bit. Let us know if it works.

Ok, thank you for your reply

I have tested my solution by using A30 (GPU 24G only):

install bitsandbytes such as : pip3 install bitsandbytes-cuda116  ---- cuda 11.6 was installed before

modify the source file: .\ldm\models\diffusion\ddpm.py
2.1 disable DDP like:
# default to ddp
# trainer_config["accelerator"] = "ddp"
2.2 use Adam8bit like:
import bitsandbytes as bnb #try to reduce GPU memory
#opt = torch.optim.AdamW(params, lr=lr)
 opt = bnb.optim.Adam8bit(params, lr=lr)

Note that，there are 7 places need to be replaced in ddpm.py


Please, add "question" label

Hi!
Could you please, add WEB OTA, namely, when the bin located somewhere in Internet with version info in separate JSON. The MC should check own version with server version. If server version is greater then the own, let's firmware upgrade go on. Let's the OTA checking take place, once, at MC initialization. For OTA report, let's the MC write another JSON as report. How is the idea?

Does not make sense for this use case, where this is generic, waiting for the real image to be flashed 8especally, since the network configuration is hard coded, with no gateway...)


After building SwiftLint in Linux, a runtime error is generated when attempting to Lint a file.

Description
A runtime error is generated when attempting to run SwiftLint after building it (either in Debug or Release mode) under Linux. It appears to affect x86_64 and aarch64 platforms, albeit only on the nightly snapshots (the 5.9 Release branch is not affected).
The error generated at runtime is:
failed to demangle witness for associated type '' in conformance 'Swift.Array<SwiftParser.Parser.ExperimentalFeatures>: Sequence' from mangled name '?z??ySayxGG' - subject type x does not conform to protocol Collection
PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace.
Stack dump:
0.	While evaluating request ExportedSourceFileRequest(source_file "/home/user/s7/Plugins/SwiftLintPlugin/SwiftLintPlugin.swift")

Steps to reproduce
Running the following commands on the terminal window is sufficient to reproduce this issue:
mkdir SwiftLint
cd SwiftLint
git clone https://github.com/realm/SwiftLint . --depth 1
rm ./Tests/SwiftLintTestHelpers/*.swift
swift build -Xswiftc -static-stdlib
./.build/aarch64-unknown-linux-gnu/debug/swiftlint

If building for x86_64, the executable may be found inside .build/x86_64-unknown-linux-gnu/debug. Otherwise, search for the generated swiftlint executable.
In the above example, we are Linting the SwiftLint repository itself. It is sufficient to trigger the runtime crash, but for a minimal reproducer of files that require Linting, one can make new directory with a single file called Package.swift; the file only needs to contain // swift-tools-version:5.8 (note that a blank file won't trigger the crash).
Expected behavior
The program should Lint all specified files successfully.
Environment

Swift compiler version info: 2023-09-20a, 2023-09-26a nightly toolchains.
Xcode version info: N/A
Deployment target: Linux x86_64 (Ubuntu 18.04), Linux ARM64 (Ubuntu 20.04).

Additional context
Some additional packages may need to be installed (this list is not exhaustive):
apt install libcurl4-openssl-dev libxml2-dev
No issues were found when building for macOS; in fact the -Xswiftc -static-stdlib flag is not supported.
Here is a stack trace from an lldb run on Ubuntu 18, x86_64:
* thread #72, name = 'swiftlint', stop reason = signal SIGABRT
  * frame #0: 0x00007ffff6477e87 libc.so.6`__GI_raise(sig=<unavailable>) at raise.c:51
    frame #1: 0x00007ffff64797f1 libc.so.6`__GI_abort at abort.c:79
    frame #2: 0x0000555557a862f9 swiftlint`swift::fatalErrorv(unsigned int, char const*, __va_list_tag*) + 137
    frame #3: 0x0000555557a86383 swiftlint`swift::fatalError(unsigned int, char const*, ...) + 131
    frame #4: 0x0000555557a9ee4a swiftlint`swift_getAssociatedTypeWitnessSlow + 1274
    frame #5: 0x0000555557a9e945 swiftlint`swift_getAssociatedTypeWitness + 37
    frame #6: 0x00005555578faeb5 swiftlint`Swift.SetAlgebra.init<τ_0_0 where τ_1_0: Swift.Sequence, τ_0_0.Element == τ_1_0.Element>(__owned τ_1_0) -> τ_0_0 + 197
    frame #7: 0x0000555556b9f276 swiftlint`protocol witness for SetAlgebra.init<A>(_:) in conformance Parser.ExperimentalFeatures at <compiler-generated>:0
    frame #8: 0x00007fffc6f25523 libsourcekitdInProc.so`swiftASTGen.parseSourceFile(buffer: Swift.UnsafePointer<Swift.UInt8>, bufferLength: Swift.Int, moduleName: Swift.UnsafePointer<Swift.UInt8>, filename: Swift.UnsafePointer<Swift.UInt8>, ctxPtr: Swift.Optional<Swift.UnsafeMutableRawPointer>) -> Swift.UnsafeRawPointer + 291
    frame #9: 0x00007fffc6e7e72d libsourcekitdInProc.so`swift::ExportedSourceFileRequest::evaluate(swift::Evaluator&, swift::SourceFile const*) const + 509
    frame #10: 0x00007fffc7226f2d libsourcekitdInProc.so`llvm::Expected<swift::ExportedSourceFileRequest::OutputType> swift::Evaluator::getResultUncached<swift::ExportedSourceFileRequest>(swift::ExportedSourceFileRequest const&) + 397
    frame #11: 0x00007fffc7226bc3 libsourcekitdInProc.so`llvm::Expected<swift::ExportedSourceFileRequest::OutputType> swift::Evaluator::getResultCached<swift::ExportedSourceFileRequest, (void*)0>(swift::ExportedSourceFileRequest const&) + 435
    frame #12: 0x00007fffc7202848 libsourcekitdInProc.so`swift::ExportedSourceFileRequest::OutputType swift::evaluateOrDefault<swift::ExportedSourceFileRequest>(swift::Evaluator&, swift::ExportedSourceFileRequest, swift::ExportedSourceFileRequest::OutputType) + 24
    frame #13: 0x00007fffc7202826 libsourcekitdInProc.so`swift::SourceFile::getExportedSourceFile() const + 38
    frame #14: 0x00007fffc6e7d5fc libsourcekitdInProc.so`swift::Parser::parseTopLevelItems(llvm::SmallVectorImpl<swift::ASTNode>&) + 204
    frame #15: 0x00007fffc6eda36a libsourcekitdInProc.so`swift::ParserUnit::parse() + 90
    frame #16: 0x00007fffc56468e6 libsourcekitdInProc.so`void SourceKit::WorkQueue::DispatchData::callAndDelete<(anonymous namespace)::SwiftDocumentSyntaxInfo::parseIfNeeded()::'lambda'()>(void*) + 22
    frame #17: 0x00007fffcb7dca3a libsourcekitdInProc.so`executeBlock(void*) + 10
    frame #18: 0x00007fffcb7dca5a libsourcekitdInProc.so`void* llvm::thread::ThreadProxy<std::tuple<void (*)(void*), void*>>(void*) + 10
    frame #19: 0x00007ffff71d96db libpthread.so.0`start_thread(arg=0x00007fff967cc700) at pthread_create.c:463
    frame #20: 0x00007ffff655a61f libc.so.6`__clone at clone.S:95

This is definitely not sourcekit-lsp. SwiftLint is calling into sourcekitd, so it’s correct for this the issue to live in this repo.

I couldn't reproduce it with:
swift toolchain: swift-DEVELOPMENT-SNAPSHOT-2023-10-09-a-ubuntu20.04-aarch64.tar.gz
SwiftLint: 0934c18cba3de323033920c1d0c2b38399e2745a
It builds and runs successfully.
@fibrechannelscsi Could you teach me what the exact toolchain and SwiftLint hash you can reproduce the runtime crash?

I have a fresh (as of yesterday) installation of Ubuntu 20.04 on an x86_64 machine.
which swift
gives me the executable found here:
swift-DEVELOPMENT-SNAPSHOT-2023-10-08-a-ubuntu20.04/usr/bin/swift.
swift --version gives:
Swift version 5.11-dev (LLVM cdb60493231c0e1, Swift a2cc0d0468d689c)
Target: x86_64-unknown-linux-gnu

For SwiftLint itself, git log yields:
commit 0934c18cba3de323033920c1d0c2b38399e2745a (grafted, HEAD -> main, origin/main, origin/HEAD)
Author: Danny Mösch<EMAIL_ADDRESS>Date:   Tue Oct 3 18:05:31 2023 +0200

    Let RuleConfigurations be equatable by default (#5257)

I have a Ubuntu 20.04 Docker image running on an M1 Mac. The command which swift yields:
swift-DEVELOPMENT-SNAPSHOT-2023-10-09-a-ubuntu20.04-aarch64/usr/bin/swift
swift --version gives:
Swift version 5.11-dev (LLVM cdb60493231c0e1, Swift 2eb90c370a39c0a)
Target: aarch64-unknown-linux-gnu

The entry for git log is the same as above.
In both cases, swiftlint begins to lint the current directory, but soon generates the following error message:
failed to demangle witness for associated type '' in conformance 'Swift.Array<SwiftParser.Parser.ExperimentalFeatures>: Sequence' from mangled name '????ySayxGG' - subject type x does not conform to protocol Collection

Thanks I reproduced the crash. A wrong sourcekitdInProc.so was loaded.

This appears to be fixed on the 10-17 nightly. Thanks!


Test: Cloud Shell

For https://github.com/Microsoft/vscode-azure-account.

[ ] Linux @auchenberg
[ ] Windows @egamma
[ ] Mac @isidorn

Complexity: 1

Checkout https://github.com/Microsoft/vscode-azure-account (requires 1.18 API, hence not published yet).
Run extension from source.
Give "Open Cloud Shell" a try.

Note: when checking the vscode-azure-account make sure to chekcout the branch chrmarti/cloudconsole since master has no changes related to this testplan item.

Authentication dance worked fine, had trouble setting up cloud shell on azure which is captured in #37214
I see a cloud shell tutorial here https://azure.microsoft.com/en-us/features/cloud-shell/ however when I login with my ms account to our portal I can not create a new cloud shell since I am not offered a cloud shell when creating a new resource. What am I missing here?

Apologies for not mentioning the branch.
The Portal has the Cloud Shell button at the top of every page:

@chrmarti thanks, that was not easy to find since the portal has 100+ buttons.

@isidorn My bad, I checked with the Cloud Shell team and there is currently no deep link to the Cloud Shell in the Portal.

I couldn't get this to work under Windows :-(


Dataset

Hi,
Can you please kindly share the dataset used to train your model.

Hi @JiahuiSophieHU,
I am trying to reproduce Multitask Learning code and I found these datasets:

BioScope corpus: two .xml files with abstracts and full texts respectively

SFU Review corpus: data from '3. Negation and speculation annotations' section

but it would be nice if the authors could confirm this.


[Ubuntu] add php 8.2

Description
Add PHP 8.2 on Ubuntu
Related issue: #6743
Check list

[X] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Please note: I added PHP 8.2 to only ubuntu-18.04 and ubuntu-20.04 as after #6331, it cannot be added to ubuntu-22.04.

/azp run ubuntu1804,ubuntu2004


Remove the From<String> and From<&str> impl for IndexUid

It is a terrible footgun.
We used it extremely implictly in many places and it is extremely dangerous.
It assumes that the string is well-formed (with a ":") within.
We use it in countless places, especially in unit tests.

IndexUid should be IndexUid(String, Ulid) instead of IndexUid(String). It should only be a <index ID>:<incarnation ID> when serialized to JSON when using the file-backed metastore.
The issue is dealing with the tuple in protobuf message. I found a somewhat decent solution, please take a look at this PoC PR and tell me what you think.