Uncaught TypeError: Cannot read property 'removeAttribute' of null

[Enter steps to reproduce below:]

install two of supported json linters
delete https://atom.io/packages/linter-package-json-validator this one
receive error

After reinstalling nothing changes
Atom Version: 1.0.19
System: Mac OS X 10.10.5
Thrown From: linter package, v1.7.0
Stack Trace
Uncaught TypeError: Cannot read property 'removeAttribute' of null
At /Users/dsavitsky/.atom/packages/linter/lib/ui/message-element.js:20

TypeError: Cannot read property 'removeAttribute' of null
  at linter-message.updateVisibility (/Users/dsavitsky/.atom/packages/linter/lib/ui/message-element.js:41:130)
  at BottomPanel.setMessages (/Users/dsavitsky/.atom/packages/linter/lib/ui/bottom-panel.js:56:24)
  at LinterViews.render (/Users/dsavitsky/.atom/packages/linter/lib/linter-views.coffee:36:12)
  at /Users/dsavitsky/.atom/packages/linter/lib/linter.coffee:29:14
  at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:82:11)
  at MessageRegistry.updatePublic (/Users/dsavitsky/.atom/packages/linter/lib/message-registry.js:116:20)
  at UpdateMessages (/Users/dsavitsky/.atom/packages/linter/lib/message-registry.js:42:17)

Commands

Config
{
  "core": {
    "disabledPackages": [
      "language-csharp",
      "welcome",
      "background-tips",
      "dev-live-reload",
      "jsonlint"
    ],
    "themes": [
      "one-light-ui",
      "one-light-syntax"
    ]
  },
  "linter": {}
}

Installed Packages
# User
linter, v1.7.0
linter-jsonlint, v1.0.1
pretty-json, v0.4.1

# Dev
No dev packages

Restart the editor and let me know if this happens again

A patch release is up for download.

I'm getting this error in the Chrome's console in any website since I updated it last week to the version 51.0.2704.106 (64bit)
(program):5 Uncaught TypeError: Cannot read property 'removeAttribute' of null protectStyleSheet @ chrome-extension://cfhdojbkjhnklbpkdaibdccddilifddb/include.preload.js:393

@franciscotln Looks like you have a buggy extension, also this repository is many layers removed from having anything to do with Chrome 😛

Yes, you're right @Arcanemagus . I removed some extensions 2 days ago and I no longer have this issue

remove AdBlock


[RUNTIME][RPC]C++ RPC Server

Support of standalone C++ RPC Server
#1496
Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

@merrymercy @masahi  @srkreddy1238 @tqchen Could you please do the initial round of review. Thanks in advance.

@FrozenGene can you please help review this as well?

@tqchen Could you please help to clean the CI environment cache..  CI checkout is failing. Thanks.
> git rev-parse --is-inside-work-tree # timeout=10
Fetching changes from the remote Git repository
 > git config remote.origin.url https://github.com/dmlc/tvm.git # timeout=10
Fetching without tags
Fetching upstream changes from https://github.com/dmlc/tvm.git
 > git --version # timeout=10
using GIT_ASKPASS to set credentials tvmci
 > git fetch --no-tags --progress https://github.com/dmlc/tvm.git +refs/pull/1756/head:refs/remotes/origin/PR-1756 +refs/heads/master:refs/remotes/origin/master
Merging remotes/origin/master commit bde53033bdad08d87e0c40e128dd08ef523aecc3 into PR head commit b9c46b859ef99b9019ede0ff96cd7be28a878ac4
 > git config core.sparsecheckout # timeout=10
 > git checkout -f b9c46b859ef99b9019ede0ff96cd7be28a878ac4
 > git merge bde53033bdad08d87e0c40e128dd08ef523aecc3 # timeout=10
 > git config core.sparsecheckout # timeout=10
 > git checkout -f b9c46b859ef99b9019ede0ff96cd7be28a878ac4
Command "git merge bde53033bdad08d87e0c40e128dd08ef523aecc3" returned status code 1:
stdout: Auto-merging 3rdparty/dmlc-core
Adding as 3rdparty/dmlc-core~bde53033bdad08d87e0c40e128dd08ef523aecc3 instead
Auto-merging 3rdparty/dlpack
Adding as 3rdparty/dlpack~bde53033bdad08d87e0c40e128dd08ef523aecc3 instead
Auto-merging 3rdparty/HalideIR
Adding as 3rdparty/HalideIR~bde53033bdad08d87e0c40e128dd08ef523aecc3 instead
Automatic merge failed; fix conflicts and then commit the result.
stderr:

Thanks for the contribution. I have gone through the code. The key challenge here as raised in the #1756 is that certain part of code will require platform dependent API, specifically:
API specificity

Fork system call

Only supported in linux, there is windows equivalent, not supported by andoir

filesys dir query

Current implementation linux only, it is possible to get a windows/linux compatible version

socket related things

select API is relatively linux, however, we can port a windows/linux compatible select from https://github.com/dmlc/rabit/blob/master/src/socket.h#L423

Where to put the code
The general principle is everything under src/runtime need to be stable and works on all systems, the apps/cpp_rpc would be more relaxed, and make use of APIs like fork.
Given the current status of the code, I would recommend to encapsulate the RPCServer as a class and put most of the listen, accept and filesystem logic into apps/cpp_rpc first.  Ideally, I would  recommend to make it windows and linux compatible, but this is not a must as python version of rpc server can still be used.
There are still value to put certain code into src/runtime/rpc, specifically the tracker reporting logic, if we can encapsulate that as a class e.g. (TrackerClient) and encapsulate certain reporting feature there, we could put some of the tracker related logic into there.

Thanks @grwlf @merrymercy @liangfu @masahi @tqchen  for the comments.
The comments are reworked and the code is moved to apps/cpp_rpc
The below things are pending which i will do as anothe PR.

The Windows/android compatability & testing.
Proxy for android

@grwlf @merrymercy @liangfu @masahi please do another round of reviews at your convenience

I can't comment on RPC logic, sorry. I'm learning how RPC works in general :)

@tqchen  @liangfu @eqy Could you please review this PR once. Thanks

@siju-samuel the server seems to work for me. I run simple example on local machine.
Are there any plans to add cross-compile machinery to build the server for different architecture.

@siju-samuel would you please act on the comments?

Close due to inactive status.

why this PR was not merged? The C++ RPC is very useful in many embeded device.

@siju-samuel

In fact, I modify some bugs based on this pr, worked very well on Linux embedded devices. Because we can not have Python environment.

@FrozenGene do you mind to revive the thread and followup on #1496 to send a new PR?
Ok. I will handle it based on this PR.

One more question, this cpp_rpc depends on libtvm.so on device, so we need to cross compile tvm, it may be difficult sometimes. It's better to get its dependences compiled seperately like libtvm_runtime.so in apps/howto_deploy/.
@siju-samuel
@tqchen
@FrozenGene

One more question, this cpp_rpc depends on libtvm.so on device, so we need to cross compile tvm, it may be difficult sometimes(many armv7 or lower device's compiler don't support c++11 above). It's better to get its dependences compiled seperately like libtvm_runtime.so in apps/howto_deploy/.
@siju-samuel
@tqchen
@FrozenGene

One more question, this cpp_rpc depends on libtvm.so on device, so we need to cross compile tvm, it may be difficult sometimes(many armv7 or lower device's compiler don't support c++11 above). It's better to get its dependences compiled seperately like libtvm_runtime.so in apps/howto_deploy/.
@siju-samuel
@tqchen
@FrozenGene

Right. This PR's makefile should be improved. In fact, we only need libtvm_runtime.so, which could be cross compiled easily.

@snowolfhawk I am interested in contributing too. Let me know if you need any help.

I will apply the open source process of company next week about our modification of this PR. @tqchen

@yagnasrinath
Currently i apply this PR on armv8 and don't find any problems.
But just like my reply above, this cpp_rpc depends on libtvm.so on device, we need cross complie libtvm.so, it's related to most of the tvm's cpp file using >=c++11, so many armv7 or lower compiler don't support it.
We should keep cpp_rpc's dependences in a limited region, then fix these cpp file using <c++11 features. maybe @FrozenGene's PR will include these changes.

We should keep cpp_rpc's dependences in a limited region, then fix these cpp file using <c++11 features. maybe @FrozenGene's PR will include these changes.

Thanks. I will make the cpp_rpc use libtvm_runtime.so, not libtvm.so, which shouldn't need C++11. However, I haven't required cpp_rpc should use C++03, so in cpp_rpc's code, we still have std::to_string, auto and so on. unluckily, I am in the open source process of our BU, so I plan to keep this currently and could modify the code to C++03 next.


TinkerPop 3.0 support

Upgrade use of traversal sources for TP3, other TP3 syntax
Use orient-gremlin until OrientDB is officially supported
Dependency version upgrades
Other small improvements

Do we have any performance benchmarks between TP API and Neo4j native API?

This is not related with this PR but I just find it out. I am not able to build the project, it seems that dynamoDBLocal is not on Maven Central as I get this: No versions available for com.amazonaws:DynamoDBLocal:jar:[1.10, 2.0.0). Please add the fix to README.md

I updated the README.md with some preliminary benchmark results. OrientDB is not officially at TP3 yet so I did not run these tests. I made some interesting findings especially with regards to how we were setting custom vertex ids in Titan. It turns out that our original implementation of custom vertex ids never worked. I found this out by enabling assertions (-ea) with the JVM and saw Titan core crap out in many places. I fixed the issues with setting custom ids in Titan, but I am not sure we stand to gain much from custom ids, so I will play around and see if using auto-ids improves performance.

Shall I close that and create a new PR when this is ready?


Linter rule suggestion - Nested resource uses the same API version as the parent

We could potentially lint for situations like the following:
resource storageAccount 'Microsoft.Storage/storageAccounts@2021-02-01' = {
  // ...

  resource blobService 'blobServices@2021-02-01' = {
    name: 'default'

    resource container 'containers@2021-02-01' = {
      name: 'my-container'
    }
  }
}

In this example, the blobServices and containers resources doesn't need to declare an API version - if they doesn't, the API version is inherited from the parent resource.
I think it shouldn't be a linter warning to explicitly override the API version of the parent resource, though - there might be some edge case scenarios where the parent resource must be deployed using one API version and a child must be deployed using a different API version.

How do you suggest the linter rule behave?  I guess at best it would be turned off by default

Given many linter rules check for effectively redundant code/typing, I would have thought this sits in the same category and would be enabled by default.
Also, if the nested resource explicitly uses a different API version than its parent, then I think we should ignore it (i.e. not raise a linter warning) - that indicates the user made an explicit decision to use different versions.

Okay, so you're saying if the api versions match we should suggest removing it from the child.  Not what I got from the bug title.


Remove whatinput requirement introduced by #9174.

Remove whatinput requirement introduced by https://github.com/zurb/foundation-sites/pull/9174

At first, I did not understood the purpose of this line and had to git blame. A comment would be usefull, like at many places in the JS code :smile:.


clue scroll helper

whenever i get a coordinate clue, the helper box pops up but it only tells me to "travel to the marked location". which is definitely not easily noticeable or it there is not a marked location. love the client though otherwise.

If all goes well, this will be fixed next release. See #2064

Looks like it will be in the next version 100%, Adam merged it with the master branch.


Name executor operations based on schema structure

This PR changes the naming convention of the default executor operations. Prevents using the name of the step and uses schema index instead. Using the user provided step name does not guarantee the resulting op name is unique, hence steps having the same name are silently overwriting each other.
The issue can be easily reproduced with the following schema:
name: "This is a test"
stages:
  teststage:
  - name: "notUnique"
    commands:
    - echo "ONE"
  - name: "notUnique"
    commands:
    - echo "TWO"

I guess there could be many other naming conventions, I just opted for this trying to prevent very long names. The name step in schema could be as long as a sentence and felt odd to me using it, as it will later on be a key in a map.

mmm just realized this is a breaking change as op name could be use to refer to another schema...

Closing in favor of https://github.com/mudler/yip/pull/89


'HCODE-pvt' is not in list

Hi,
I am trying to use the run_Maze.py and run_Diabetes.py but when I do so I stumble on the error:
Traceback (most recent call last):
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/Utils/utils.py", line 246, in dynamic_load
    pos = abs_path.index('HCODE-pvt')
ValueError: 'HCODE-pvt' is not in list

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/run_Maze.py", line 112, in <module>
    main(mode='train')
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/run_Maze.py", line 89, in main
    config = Config(args)
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/config.py", line 66, in __init__
    self.env, self.gym_env, self.cont_actions = self.get_domain(args.env_name, args=args, debug=args.debug,
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/config.py", line 98, in get_domain
    obj = utils.dynamic_load(path, tag, load_class=True)
  File "/home/david/PycharmProjects/ImportanceSampling/UnO/Src/Utils/utils.py", line 256, in dynamic_load
    raise ValueError("Failed to dynamically load the class: " + name )
ValueError: Failed to dynamically load the class: Maze

What is HCODE-pvt and how can i get it? Does this dynamic loading have anything to do with julia? Can I avoid using julia if I want? I did try to install julia but still got the error message.

Thanks for raising this. HCODE-pvt was the name of my project when it was under development, which got later renamed to UnO for the public release.
Could you try replacing 'HCODE-pvt' with 'UnO' and see if that works? If it does not could you print(abs_path) before the error is raised and share the output of that?
(I am away from my work machine right now and will be able to push a fix only after a few days).

" Can I avoid using julia if I want?"
Only the data collection scripts are in python. The actual UnO code for the estimator/bounds is in Julia only.

Hi,
I was able to solve the issue by replacing it with UnO and then using the commented line with "/" instead of "\" (linux system vs windows I suppose).
I had a few more errors one of which is now resolved:

a circular import
from Src.Utils.Policy import Policy

In the Src/Utils/Policy.py file. Removing this line solved the issue.
-'No such file or directory: UnO/Experiments_2/Maze/ActorCritic/DataDefault/0/Checkpoints/actor.pt'. Do I need to make this file before running the script? Or what can I do to prevent this error?

Thanks for pointing these out. Looks like once I had the data cached, I did not clean up the data collection files properly. I will update these soon.

'No such file or directory: UnO/Experiments_2/Maze/ActorCritic/DataDefault/0/Checkpoints/actor.pt'.
In Src/config.py, if you change Line 37
self.paths['Experiments'] = path.join(self.paths['root'], 'Experiments_2')
to
self.paths['Experiments'] = path.join(self.paths['root'], 'Experiments')

I think that should fix the issue. But note that depending on the arguments of the code being executed, it might overwrite the data files in the Experiments folder.
Alternatively, if you would want to create a new parallel set of experiments, then you could create a folder named 'Experiments_2' at the root, and then save the trained policies and collect the data accordingly.
THanks for your patience!

Thanks, your suggestion for line 37 seems to work!


Remove blocking

Depends on #175
@mariofusco This is the re-do of what you did in #183

@mariofusco I've built something in now that when dependabot does an update, the checks also check that update against the downstream branches in this repo.
The downstream branches are essentially "solution" branches - the end results of what the users do in modules of the workshop. All changes made to main automatically flow to these branches without user intervention (unless there are merge conflicts).
I've updated the CI processes to also check and make sure that dependency updates don't cause failures in these other branches, which was the case with the langchain 0.22.0 update. It was fine on main but not on one of the other branches.
That won't happen again.

Hey @mariofusco I updated to the latest Quarkus LangChain4j (0.23.0.CR1), but I'm not sure things are working properly? When I look at the logs I see
2024-12-13 16:01:01,006 INFO  [org.par.res.ClaimWebsocketChatBot] (vert.x-eventloop-thread-1) Got chat query: ClaimBotQuery[claimId=1, claim=
    On January 2nd, 1955, at around 3:30 PM, a car accident occurred at the intersection of Colima Road and Azusa Avenue in Hill Vallet. The involved parties were Marty McFly, driving a silver Delorean DMC-12 (OUTA-TIME), and Biff Tanner in a blue Type 2 Volkswagen Bus (BIF-RULZ).
    Marty was heading south on Colima Road when Biff failed to stop at the red traffic signal on Asuza Avenue, causing a collision with Marty's vehicle. Both drivers exchanged information and took photos of the accident scene, which included damages to the front driver and passenger side of Marty's Delorean DMC-12 and the front driver's side of Biff's Volkswagen Bus. No injuries were reported.
    Marty has attached necessary documents, such as photos, a police report, and an estimate for repair costs, to his email. He requests prompt attention to the claim and is available at<PHONE_NUMBER> or<EMAIL_ADDRESS>for any additional information or documentation needed.
    , query=Should I approve this claim?, inceptionDate=1954-09-30]
2024-12-13 16:01:01,922 INFO  [com.git.tja.jla.ten.ope.TensorOperationsProvider] (vert.x-eventloop-thread-1) Using Native SIMD Operations (OffHeap)
2024-12-13 16:01:01,930 INFO  [com.git.tja.jla.mod.AbstractModel] (vert.x-eventloop-thread-1) Model type = Q4, Working memory type = F32, Quantized memory type = I8
2024-12-13 16:01:02,186 INFO  [com.git.tja.jla.mod.AbstractModel] (vert.x-eventloop-thread-1) Model type = Q4, Working memory type = F32, Quantized memory type = I8

It still shows that the query and jlama is running on the event loop thread? Is that what it should be? Shouldn't it be on an executor/worker thread?

@edeandrea I don't think that this is running on the event loop thread otherwise you should see tons of warnings (one every 2 seconds) that you're blocking it as it did before.
@geoand @cescoffier can you please confirm?

Yeah, it shouldn't be


Bridge Cosmos Wallet Connect

You should override this in the cosmos wallet connect theme settings. These are somewhat elusive and difficult to find, and took me a bit of time to initially find them to color the background black. Hopefully this helps:
Theme overrides can be set here:  https://github.com/forma-dev/hyperlane-bridge-ui/blob/main/src/features/wallet/context/CosmosWalletContext.tsx#L61
What can you override? good question. There are no docs on it. But after digging a lot, I found you can get some hints here:
https://github.com/cosmology-tech/interchain-ui/blob/main/src/styles/override/override.ts#L43
From there we can dig into the connect-modal overrides which leads to here:  https://github.com/cosmology-tech/interchain-ui/blob/main/src/ui/connect-modal/connect-modal.helper.ts and tells us we can use the shadow property to override it. value of none should work

see above comment

You should override this in the cosmos wallet connect theme settings. These are somewhat elusive and difficult to find, and took me a bit of time to initially find them to color the background black. Hopefully this helps:
Theme overrides can be set here: https://github.com/forma-dev/hyperlane-bridge-ui/blob/main/src/features/wallet/context/CosmosWalletContext.tsx#L61
What can you override? good question. There are no docs on it. But after digging a lot, I found you can get some hints here: https://github.com/cosmology-tech/interchain-ui/blob/main/src/styles/override/override.ts#L43
From there we can dig into the connect-modal overrides which leads to here: https://github.com/cosmology-tech/interchain-ui/blob/main/src/ui/connect-modal/connect-modal.helper.ts and tells us we can use the shadow property to override it. value of none should work

see above comment

The shadow property works for the outside box instead ( See image as I overrides shadow properties). As I check, we can only overrides following properties: background, shadow, color and borderColor.


Upgrade brave to 5.6.4

This PR starts the CircleCI tests to aid @adriancole in debugging after discussion on Gitter following:
https://gitter.im/spring-cloud/spring-cloud-sleuth?at=5d04d35b6f0ec85ade104498

Duplicated by #1373


IPv6 support

We should implement support for IPv6 - which in practice means that we should remove the IP and port inputs and replace them with a multiline textbox where each line is a IP/port pair, like this,
<IP_ADDRESS>:6881
[::]:6881

This means we can fully utilize IPv6 for those that have it enabled.

But how to enable IPv6?
Why isn't it enabled by default?

You can add the IPv6 network adapter in the Connections page in preferences


Extension data, Data export doesn't match webview data

Type: Bug

Go to extension I published: https://marketplace.visualstudio.com/manage/publishers/XXX/extensions/XXX/hub?_a=acquisition
Export data to excel
View data in the graph
Data doesn't match, also, it doesn't export all the same data.  Webpage shows "Install from Vs Code", "Download from Marketplace", "Page Views".  Export shows only "Install from VS Code" and "Page views".  And even still, the data doesn't match.

VS Code version: Code 1.76.2 (ee2b180d582a7f601fa6ecfdad8d9fd269ab1884, 2023-03-14T17:55:54.936Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

Howdy,
I wanted to see if there has been any movement on looking into this.

@xavierdecoster I was wondering about an update for this?  Can it be explained?  Is is a bug?  What is a data of truth?

Closing the issue since there has been no updates on the information requested. If the issue still persists, please feel free to reopen or create a new issue.
Regards,
VS Marketplace Team


Notifications sent via persistent connection don't arrive

I struggled with this issue for a long time, and finally discovered what was happening. I'm using version 2.2.1 of the gem.
I found that I could send notifications via the command line apn tool, but it was failing to send when integrating Houston into my code. After trying all sorts of different combinations I cut the code down until it was as simple as this:
apn = Houston::Client.development
apn.certificate = File.read("/path/to/apple_push_notification.pem")
token = "<ce8be627 2e43e855 16033e24 b4c28922 0eeda487 9c477160 b2545e95 b68b5969>"

notification = Houston::Notification.new(device: token)
notification.alert = "Hello, World!"

apn.push(notification)

I.e., just set a message and send. The notification would never arrive, and there was no error in notification.error. As a last attempt, I tried the Silent Notifications code in the README, meaning that I created a Notification with Houston::Notification.new(token: token, sound: ''), and it finally sent!
It seems like there's an issue with the default sound option. Maybe setting it to blank is the safest default?
Edit
Actually, I discovered in the end that it was down to invalid tokens in the database. I had registered some tokens from the iOS simulator, and in my Houston code (using the persistent connection format) was looping through all the devices sending out notifications, e.g.:
Device.each do |device|
  notification = Houston::Notification.new(device: device.token)
  notification.alert = "Hello, world"
  connection.write(notification.message)
end

It seems that all notifications sent after the one to the simulator would never arrive. After deleting the offending tokens from the database, the notifications would go through.
Is this an issue with the APN service? It seems like a bit of an issue though, as invalid tokens bring the whole thing to a halt. I don't know if you have any thoughts on this?
Thanks

Hi,
I finally had a chance to try and implement your suggestion @cognitiveflux, but didn't have any luck receiving anything at all from the Apple feedback service. Here's what I did:
threads = []
message = "This is a test message"
certificate = File.read(certificate_path)
passphrase = nil

devices = [
  "<valid token 1>",
  "<invalid token>,
  "<valid token 2>,
]

threads << Thread.new do
  connection = Houston::Connection.new(HOUSTON_URL, certificate, passphrase)
  connection.open

  devices.each do |device_token|
      notification = Houston::Notification.new(device: device_token)
      notification.alert = message
      notification.sound = "sosumi.aiff"
      notification.category = "INVITE_CATEGORY"
      notification.content_available = true

      puts "Sending to device: #{device_token}"
      p connection.open?
      connection.write(notification.message)
      if notification.error
        puts "Error for device #{device.id} (#{device.name}): #{notification.error}"
      end
  end
end

threads << Thread.new do
  Houston::Connection.open("apn://feedback.push.apple.com:2196", certificate, passphrase) do |connection|
    loop do
      while line = connection.read(38)
        p line      # Show me some errors!
      end
    end
  end
end

threads.each(&:join)

The first push notification would send correctly, the second one (to the invalid token) would silently fail, and the third, even though it's a valid token, would never be sent. You were right about connection.open? always returning true. If I removed the invalid token from the queue then both notifications would send.
However, the error reporting thread didn't receive anything from the feedback service - nothing is read from the socket, and nothing is printed out.
I'm really hoping I'm doing something wrong here! I also tried using the unregistered_devices method in the Houston library, but nothing is received there either.
Any ideas? Thanks again.

@joonty You could check out the suggestion I made over at https://github.com/nomad/houston/issues/85#issuecomment-71176096. It’s not as sophisticated as the solution in the blog post @cognitiveflux mentioned, but is based on what the houston client does as well as the assumption that invalid tokens don’t get encountered significantly often. It is able to catch and handle invalid tokens, though, and can be improved from there.

@joonty The issue is that you're using different connections on different threads, you need to use the same connection; two threads can simultaneously use the same connection as long one is strictly writing while the other is strictly reading.  Errors are only sent back on the same connection that the invalid notification was sent over, the feedback service is not for realtime error reporting.  You also don't need a separate thread for writing, in which case you do not want to join the reading threading.
High level example:
class ApnsRunner

 def initialize(...)
  @error_encountered = false
  @connection = Houston::Connection.new(HOUSTON_URL, certificate, passphrase)
  @connection.open
 end

 def start
  ... # make sure connection is open
  listen_for_errors
  send_notifications
 end

 def stop
  @thread.exit if @thread
  @connection.close
 end

 def listen_for_errors
  unless @thread && @thread.alive?
    @thread = Thread.new {
      read_socket =<EMAIL_ADDRESS># Use the SAME CONNECTION
      if error = @connection.read(6) # Block indefinitely until and error is received, hence why it is on a new thread
      ... # read error,
          # handle error (invalid token is just one type of error),
          # capture ID of the failed notification, add to error queue, etc.
      @error_encountered = true # Let the writing thread know that it should stop sending
       ... # handle connection reset
      end
    end
   }  # Do not join on main thread!
 end

 def send_notifications
   @some_array_or_queue_with_notifications.each do |notification|
     break if @error_encountered
     ...
     @connection.write(notification.message) # Again, using the SAME CONNECTION
     ...
   end
 end
end

A couple things to note, first you need to make sure the notification objects have a unique identifier to be able to determine when an error was encountered.  When an error is detected by the reading thread, it is possible for a couple notifications to still be sent by the writing thread, so that's where the notification IDs and queues come into play, you will need to "replay" the write from the point the notification failed.  Second, the error thread will stay open as long as the connection is open and an error has not been received, so you're not spawning threads all the time.  For many reasons it's in your best interest to use persistent long running connections (e.g. up to 24 hours).
@j4zz That will certainly work for a small number of notifications, although you can send a lot of notifications in one second.  Just a word of caution, one connection per notification for a large number of notifications is handled like a DoS attack and all connection attempts from your server will be blocked until you contact Apple (yes they do track and monitor it); I don't know where they define the threshold, but as long as your volume is low I suppose you can slip under the radar.

@cognitiveflux Just for clarification, my example is meant to send all notifications on a single connection, and afterwards checks the same connection for an error (that may have happened some time ago during sending). If so, resend all notifications after the erroneous one (since that one caused the connection to ignore all further notifications) on a new connection.

@cognitiveflux thanks very much for the detailed code - I'll try that. My first attempt was to use the same connection, but I must have been doing something wrong.
It seems that there are quite a few issues relating in some way to the inane way that Apple report on errors (and nothing to do with a problem with Houston), so feel free to close this issue. Then again, it may be helpful to others.
Thanks

I'd got a solution (os dependent - linux) which validates connection status before sending the notification. It works great with Sidekiq, sharing one connection with all threads. Does not open the connection until needs to send something (sharing code with front and queue servers)
https://gist.github.com/ferrous/d498971be300683c3bbd

I am experiencing this issue as well.  Have it down to a simple test script (same as @joonty's script more/less).  apn command line works fine.  apn ... also works fine (no surprise).  Just not getting any notifications sent using using Houston APN.
Anything else to verify, check?  I hacked in a bunch of debug logging in Houston client.rb and no errors are returned, etc.
ruby version: ruby 2.2.3p173 (2015-08-18 revision 51636) [x86_64-linux]
Houston gem: 2.2.3


Recommend new relevant products based on historical transaction records

In a business scenario, for user historical transactions, or sold out items, to recommend new items on the shelves can be based on feedback or labels.
In other words, for some items, participate in the calculation, but not participate in the recommendation.

棒！

棒！

master分支还不能用，缓存刷新还需要优化，需要等0.3发布哈


[RFC] optimized connect with dependencies

I was wondering if it is possible to implement easily on top of existing APIs such a concept - a connected component seting its dependencies (often in runtime, but mainly upon component creation only) so they can be grouped and handled internally, so the mere top state change is not enough to fire all subscription functions.
Possible use case - multiple dynamic lists in the app where each item is a connected one.
However from what I see connect and connectAdvanced are not sufficient for this, so more internal / undocumented APIs would have to be used.
What do you think about the concept?

I don't really understand what you're suggesting.

Sorry if I have phrased it in a confusing way, was writing it pretty late in the night.
The idea is to create such a connect which would be capable of tracking registered dependencies (via reselect for example) for components so they could be grouped internally. It would mean that not every state change would trigger each listener in the connect for each connected component, but it would only fire more relevant ones.
Imagine 2 lists of 100 items, each item connected. If we can scope them by some different dependency we could skip their listeners compeltely (if some irrelevant part of the state tree changes) or only fire 100 listeners (if only dependency of their interest changes and not the other one) instead of 200.

That is really up to your mapStateToProps function. Avoiding calling that listener wouldn't really net you much performance benefit. Most of the work comes from actually re-rendering your component. If the state mapped to your props doesn't change, that render doesn't have to happen. So, you're already doing this there, but just on the other side of the subscription. A few hundred function calls (especially when they happen often, meaning they get optimized by the compiler) are not going to have any notable impact on performance.

I realize that mapStateToProps's results are optimized under the hood, even on compiler level and not only in the lib itself. However surely even when thats get optimized to some extent those calls are still somewhat expensive if there are too many of them.
Recently in the redux-saga someone has noticed (after investigating the real life perf issue) that we subscribe internally each task (and they might be plenty of them) to the action stream (and situation is there somewhat worse than in case of react-redux cause we need to notify subscribers about all actions, not only after state change) and that caused him a perf issue cause he had hundreds of tasks not even depending on the actions. We'll be optimizing this, so we create only a single, global for redux-saga emitter of those actions so each task can subscribe to this independently - resulting in only some listeners in the opposite of the situation when each task got its own instance of the subscriber anyway.
This situation led to me think about this idea and when I realize it might be unnecessary for most apps, I think it might be worthwile to think about such a possibility. If you still think its completely futile leave it closed.


Does Javalin use an event loop for asynchronous?

How does asynchronous mode really work? I've been looking into Vert.x and it uses an event loop for handling requests, while Javalin uses a thread pool. But when I define a handler which returns a CompletableFuture, what does change? Instead of spawning a thread to handle that handler, does it go into an event loop mode, or does it dettach a thread from the main thread pool?
I want to consider this because I really liked Javalin and would like to continue using it (I love this simplicity), and I need to understand how it works in comparison to Vert.x's event loop.

Hi @ivanquirino. Javalin relies on Jetty which implements the Servlet spec. I'm on a business trip, so I can elaborate later, but in short you can Google how async servlets work.

@ivanquirino this article is about Spring Web Reactive, but it's basically the same: https://dzone.com/articles/understanding-spring-reactive-servlet-async
Javalin detaches/frees the the current request thread from the server's main threadpool. I've never looked into the details myself, I trust Jetty to fully take care of this. Here is a very simplified version of the request handling code:
if (ctx.resultFuture() == null) {
    writeResult(res)
} else {
    val asyncContext = req.startAsync()
    ctx.resultFuture().thenAccept {
        writeResult(asyncContext.response as HttpServletResponse)
        asyncContext.complete()
    }
}

So after the handler runs (app.get("/", userHandler)) Javalin will check what kind of result the user has set. If it's a not a Future (so a String or InputStream), the result will be written directly to the HttpServletResponse. If it's a Future, the request is released from the threadpool (req.startAsync()), and the result is written when the future resolves.

Thanks for the answer! So, it's not an event loop but a detached thread. I wonder if there's any interest of you to add Netty support so Javalin runs as a event loop (event-driven) mode.

@ivanquirino not much interest, I'm more or less married to Jetty at this point. My experience with other frameworks that support multiple underlying servers is that Netty is unstable and performs poorly. I don't know if that's because it's inherently like that, or because it's hard to write a good integration. Either way I'm very happy with Jetty. Have you actually run into any performance issues, or are you looking for a theoretical performance improvement ?

@tipsy No perfomance issues at all. I'm looking at a lightweight and efficient java web framework that uses less resources as possible because I'll be running image manipulation using BoofCV (a java computer vision library which has a great QRCode detector and image tools) and I found that like Node.js and Nginx, an event-driven loop instead of a thread pool consumes less resources, and that's exactly what Netty and Vert.x does (Vert.x is 100% on top of Netty only). Using the less expensive possible cloud deployments is a must in our organization.
I started this project's prototype using Javalin because It felt great, it has a nice API and simplicity which I enjoy a lot, but when thinking at a large and intense production deployment, I'm leaning more towards Vert.x. All of this depends if the current solution will be used at all, since we are evaluating a different solution using deep learning computer vision algorithms. If we change to the deep learning solution, I'll change my current server to be a image processor to help create our dataset, and I'll keep using Javalin for that, since it will not get to huge loads.
I may be underestimating Jetty capabilites tough, beucause I've been using Node.js and it's event loop for so long that I may be biased to that model.

when thinking at a large and intense production deployment, I'm leaning more towards Vert.x.

That's understandable, I felt the same way until someone added Javalin to the techempower benchmarks: https://www.techempower.com/benchmarks/#section=test&runid=a0d523de-091b-4008-b15d-bd4c8aa25066&hw=ph&test=plaintext&l=xan9tr-3&a=2
The default config (blocking) handles one million requests per second. I don't know how well the async version performs, but I imagine it's better. I'd be very interested in the results if you decide to do some benchmarking.

Using the less expensive possible cloud deployments is a must in our organization.

Then the JVM is probably not the best choice? You could look into GraalVM: https://javalin.io/2018/09/27/javalin-graalvm-example.html, which should increase performance a bit. Developer time is more expensive than hardware though, I'd be a bit careful about this.
Vert.x is a great choice though. If I ever run into performance issues with Javalin, that's what I'll use too.

The JVM is allowed in our requirements, since developer time is very valuable for us. Hey, thanks for your time into answering these questions. I'll keep using Javalin since I already got the hang of it and the docs are so well made and simple to follow and it is great for my actual usage.


Fixing unintended formatting changes

Description
When making the workload changes, I ran the xml formatter on the SDK.targets file to format tabs/spaces and indentation correctly, but it looks like that made some formatting changes that introduced bugs like adding newlines to property values, which would then break conditions when doing string comparisons. Thanks a lot @eerhardt for debugging and finding this out. The changes in this PR revert all the formatting changes made in that PR, which means the net difference since 8.1 will be the addition of the new target.
Fixes #5436
Checklist

Is this feature complete?

[ ] Yes. Ready to ship.
[x] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

After applying the changes in this PR locally, I can see that the issue is resolved:

/backport to release/8.2

@joperezr Is this ready to be merged? We're still experiencing the issue with 9.0 P4.


Monadic subscription handlers

Currently, subscriptions with reg-sub form an applicative functor with respect to input signals - mulitple input signals can be "fanned in" to a function that takes the same number of parameters, and returns a raw constant value. I was thinking it'd be nice if they were monads as well - essentially, if I could pass the contents of one subscription reaction as parameters to another subscription.

if I could pass the contents of one subscription reaction as parameters to another subscription.

Can you clarify what you mean by this? Maybe with some sample code to show roughly how this would look?

@glittershark  In the absence of further information, I'm going to close.  If you end up providing more information we can reopen.


Update container-instances-vnet.md

The network profile list command doesn't work as the network is virtual so we need to use network vnet subnet list. Please update.

@aryankhandal0 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Terminal

The terminal functionality, when a package is too large it will print it on a new line, not flushing the old one

This will still happen if someone makes the terminal really small, but its good enough. So I am marking as closed.


Remove special handling of config servers in ACL

The zones need to be bootstrapped by the next release anyway because of the changes to chef?


Clippers always damage plants

Description
Clippers currently transfer the damaged health to the seeds regardless of whether or not the plant is fully grown/harvestable.
Reproduction/Video
See https://discord.com/channels/310555209753690112/675078881425752124/1209771702578323466 on the Discord.
Clip any fully grown+harvestable plant and read its health value. This health value carries over to the seeds and the plant you can place with them.
Additional context
Pull 24851 rendered only harvestable plants able to be sampled, and it did not damage the seeds.
Pull 25326, the currently implemented one, allows sampling at more stages, but the health of the sampled plant carries over to the seeds. It carries over to the seeds of a harvestable plant as well. I don't believe this is intentional.

Hi, guy who wrote 25326 here. I did not consider that, it was a pretty hasty pr. I don't think you could argue this behaviour is unintentional per se.
Currently, it seems the one buff that seems to be considered as the first line of discussion would be adjusting the damage value.
Personally, I'm not good with balance, so I can't say if it's a good idea or one aligning with the maintainers' intentions, but I do like the concept of making cuttings from harvestable plants less damaged or not inheriting damage.

Ah, alrighty
I'll try to make a PR about this later


run --{example,bin} fake should give "unknown example" error

$ cargo new test
$ cd test
$ cargo run --example fake
a bin target must be available for `cargo run`

This isn't the real error, it should say "unknown example fake" (or similar).
The same thing is emitted when a bin target does exist, and for --bin fake.

Dupe of https://github.com/rust-lang/crates.io/issues/130 and PR at https://github.com/rust-lang/cargo/pull/1283


impl Clone for HistoryBuffer

I stumbled about the fact, that  Clone is implemented for Vec but not for  HistoryBuffer?
I didn't check the code and honestly, I didn't think about it in depth, but it would be nice to clone these as well.
Cheers!

HistoryBuffer::Clone exists in current version. This should be closed.


 This may be useful for applications that maintain a custom, application-specific index as persistent view.

Lost priority.


NetworkManager[496]:  [1450947554.023776] [nm-modem.c:532] ppp_stage3_ip_config_start(): (ttyUSB2): error starting PPP: (1) Could not find  "pppd" binary

Can't connect through the usb 3g modem.
That message appears in the log while NetworkManager trying to connect .
ppp package is installed.
nix-env -q => ppp-2.4.7
ModemManager is running.
usb_modeswitch is runned also.

Which NixOS version are you running? The package sets --with-pppd=${ppp}/bin/pppd so this shouldn't happen..

15.09.784.7b85b8a
Before I had use some previous branch of 15.09. Later I had upgrade to current 15.09 and did nixos-rebuild switch --upgrade, reboot and the problem still exists.

(triage) does this still happen?

Issue is old and silent -- closing.


No temporary loop for ASGI middleware

Fixes https://github.com/Azure/azure-functions-python-worker/issues/1139
get_event_loop() is deprecated in Python 3.10, but the replacement, get_running_loop() will raise a runtime error if there is no event loop, so asyncio.run() is a better solution, compatible with Python 3.7+.

"compatible with Python 3.7+"
Python 3.6 is failing. Are we still supporting 3.6?

3.6 isn't supported anymore, I could add a shim for this PR to get the 3.6 test passing, but it would add extra software debt to the project for no reason.

Suggest merging after #152

@YunchuWang this is passing now, ready to merge


Expand quiet_boot option

The Raspberry Pi ‘color test’ can be disabled by adding the line disable_splash=1 to /boot/config.txt.

Disable the various bits of output from the kernel and friends by adding consoleblank=0 loglevel=1 quiet to /boot/cmdline.txt.

Disable the login prompt by running systemctl disable getty@tty1 as root.

I'm taking these steps from https://yingtongli.me/blog/2016/12/21/splash.html and will open a splashscreen option ticket later, but the requests above are all related to "quiet booting" and a splashscreen in particular.

Perhaps quiet_boot can have a third value, e.g. 2, that silences absolutely everything (with steps above)?

Further:
Disable the Raspberry Pi logo in the corner of the screen by adding logo.nologo to /boot/cmdline.txt.

Disable the Raspberry Pi logo in the corner of the screen by adding logo.nologo to /boot/cmdline.txt.

:+1: already have that ;)

I also had to remove the black borders: disable_overscan=1 in /boot/config.txt.

Parts of this will be implemented in v1.5.2.

The following parameters will be implemented soon:
disable_splash=1
logo.nologo
disable_overscan=1

BTW: Set installer_retries="0" in the installer-retries.txt and see what happens! :)

BTW: Set installer_retries="0" in the installer-retries.txt and see what happens! :)

I suspect the 0 gets decremented and you get a very large number of retries.
So, do you need to specify 1 to get 0 retries as the first try is incorrectly treated as a retry?
I would, defensively set to 1 instead of 0 just in case my suspicion were justified.

Every installation attempt is a "retry". So installer_retries="0" means: no attempt left. Default is 3.

Every installation attempt is a "(re)try". So installer_retries="0" means: no attempt left. Default is 3.
So you can simulate what happens in case of too many tries if you set it to 0.

You're going to confuse English speakers with this view.  The prefix "re" indicates repetition.  The verb "to retry" means "to try again".  The first attempt cannot be a retry.
Unless you take the same view as Humpty Dumpty, your interpretation is just plain wrong.
(url)https://en.wikipedia.org/wiki/Humpty_Dumpty#In_Through_the_Looking-Glass
Using "(re)try" is understood to indicate that the prefix is only applied where appropriate.  However, the brackets don't appear in the variable's name.  Most people won't read any explanation, but expect it to take its usual meaning.  If they wanted to check after the first attempt if it was worth retrying, they would just set it to zero.
A few dictionary references...
retry https://en.oxforddictionaries.com/definition/retry
re-enter https://en.oxforddictionaries.com/definition/us/re-enter
attempt https://en.oxforddictionaries.com/definition/attempt
re- http://dictionary.cambridge.org/dictionary/english/re
I hope you enjoy the Humpty Dumpty link :-)

No reproof needed! :D
I first implemented this on another location in code and hadn't changed it afterwards.
If you worry about it, make a meaningful suggestion with "installer_" as prefix!

'installer_attemps'

Oups... Forget about it!
The wording "retries" was correct! 😆
The setting only gets important if and only if there gets something wrong. It will be decreased and the installer restarts. If the retry countdown is zero, there will be no further attempt/retry. That means, setting it to zero does nothing if no errors occur.
Sorry for the confusion and thanks for the attention!


clang-tidy linting third_party

Example: bwtree pr
Notes:
Previously, we fixed this for spdlog by changing -I to -isystem in here.
Attempted so far:

-isystem doesn't seem to be helping on clang in this case, some namespace stuff can't be found
we tried rewriting HeaderFilterRegex (whitelist) with no success

The underlying issue is that clang-tidy either checks all of a file, or it doesn't.
In this case, because it is in the test folder, we're saying: check all the headers.
There doesn't appear to be a way to say, check only some of the headers, and not the others.
i.e. I don't think the below is possible within a single file
#include "bwtree/bwtree.h" // don't lint me
#include "storage/data_table.h" // lint me

There are some open feature requests and PRs about it, but clang-tidy devs don't seem to care very much.
@mbutrovich

@lmwnshn Is this fixed?

Whoops, didn't see this. No. I'm not sure a clean fix exists.


Grant type support issue

.Net ADAL does not support UserPassword credential grant type, AcquireTokenAsync does not have constructor to accept userpassword credential parameter

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5a1179b0-7f3d-77de-239b-a33c9d30ff61
Version Independent ID: ab708f2a-3cb5-fcf3-9733-a2bc26b09e60
Content: Authenticate users and get an Azure AD access token for your application - Power BI
Content Source: powerbi-docs/developer/get-azuread-access-token.md
Service: powerbi
Sub-service: powerbi-developer
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

Thanks for your comment, @Athimoolaganesh. This issue has been assigned to the doc manager for review.
In the meantime, can you provide some additional info - is the issue an error in the docs or a limitation of the API?

@KesemSharabi

Hi @Athimoolaganesh,
Thanks for the information. We will review this article and amend it if needed.
In the meantime, you might find a solution in the Power BI Community. If not, try our support page.

This issue has been reported multiple times, but MS just diverts to the (totally useless) PBI Community page. If you are like me and having this issue, you may have luck with https://www.c-sharpcorner.com/article/how-to-get-access-token-for-powerbi-embedding-in-net-core/
I did not write and have no affiliation with that site - it's just the solution that worked for me.


Model to detect whether uploaded pdf contains malware or not

Overview
We need to create a machine learning model capable of analyzing uploaded PDF files to determine if they contain malware. This model will enhance our system's security by preventing malicious PDFs from being processed or accessed by users.
Objectives
To build a robust model that accurately classifies PDF files as either 'malware' or 'safe'.
To ensure the model can handle various types of PDF files and different forms of embedded threats.

Sure @DarshAgrawal14, go ahead, please mention the estimated time required for this, and please go through the project structure and the tutorial.

got it  , I think i can complete it by day after tomorrow .

@yashasvini121 , please let me know if any further changes are needed. Otherwise, I would appreciate it if you could merge the PR.


star commutes and thus derive commutes

Finally it is done :D
This follows after proving concat commutes: https://github.com/awalterschulze/regex-reexamined-coq/pull/170
Proving star commutes was simply a matter of destructing with the non empty condition to avoid recursion that we have setup weeks ago.

Congratulations @awalterschulze  - this is amazing 🎊

Congratulations all of us.  This has been a team effort <3


When using the Select component, pressing space key doesn't select the focused item

Current Behavior
https://github.com/huntabyte/shadcn-svelte/assets/44144724/19df4bca-0bd6-4d71-b16f-3b46a4fafaaf
Expected Behavior
I expect it to behave like the enter key and select the focused item.
Steps To Reproduce
.
Link to Reproduction / Stackblitz (reproduction template)
No response
More Information
No response

Pending: https://github.com/melt-ui/melt-ui/issues/729 /  https://github.com/melt-ui/melt-ui/pull/730


Prepare changes planned for official version 9.0

This pull request will officially promote the master branch to version 9.0 once completed. Changes will include:

Remove console (issue #5) which will be replaced by a plugin enabled console.
Rewrite plugin system (issue #3)
Rewrite event bus
Rewrite permissions

So far the following issues have been finished:

Remove console (issue #5)
Rewrite event bus
And the follow has been started on but needs finishing:
Rewrite plugin system (issue #3)
Rewrite permissions
The finally, not started:
Reimplement a command system that will support both remote and local interaction
Implement console as a plugin with will require the former

The command system is in the progress of being reimplemented and Query Server is also being implemented.


Charts resizing

Fix the charts so that they resize with the width of the window (always fit them into the screen)

Working on it.

From my understanding, the charts already are resizing according to the width of the screen. However, when the window size gets below a certain threshold, the charts disappear. Is that what you want to avoid happening?

is this issue solved?

Hello! hopes you are doing great... please I would like to join the project via googleSummerCode... but I find it difficult to elaborate my proposal related to this project... any helps would be really appreciates.
Thank you...


[BUG] Finding bugs in dogfooding

[ ] Run WCC on modern_graph occurs error

import graphscope as gs
from graphscope.dataset.modern_graph import load_modern_graph
g = load_modern_graph()
ret = gs.wcc(g)

[ ] ...

Can you tell us the detailed error message you have seen?
Thanks!

Can you tell us the detailed error message you have seen?
Thanks!

@sighingnow This issue acts as the problem tracker for the dogfooding process. I'll take care of most of that~.

Already leverage discussions to track issues.


Allow AMSAT status page upload for QSOs logged via API

Currently only manually logged QSOs via GUI are sent to AMSAT status page. This patch allows for API logged QSOs to be reported to the AMSAT SAT status page.

Refactored now and based on timestamps.


Code Quality: Introduce ViewModels for layout pages

What feature or improvement do you think would benefit Files?
Current codebase of layout pages are kind of mess. we have to separate code from code-behind to follow MVVM pattern(according to the guideline preface All code contributed to the repository must follow our Model-View-ViewModel design path).
Requirements

[x] Rename namespace LayoutModes with ContentLayouts
[x] Rename page classes (according to the guideline, section 1, item 11)

Rename BaseLayout with BaseLayoutPage
Rename IBaseLayout with IBaseLayoutPage
Rename ColumnViewBase with ColumnsLayoutBasePage
Rename ColumnViewBrowser with ColumnsLayoutPage
Rename DetailsLayoutBrowser with DetailsLayoutPage
Rename GridViewBrowser with GridViewLayoutPage

[ ] Move styles into separated ResourceDictionary file; ContentLayoutsStyles.xaml
[ ] Use ICommand instead of using Event Handler.
[ ] Introduce ViewModel classes.

Introduce BaseLayoutViewModel abstract class
Introduce IBaseLayoutViewModel interface
Introduce ColumnsLayoutBaseViewModel class
Introduce ColumnsLayoutViewModel class
Introduce DetailsLayoutViewModel class
Introduce GridViewLayoutViewModel class
Introduce GroupabeBaseLayoutViewModel abstract class

[ ] Move page-unrelated methods into each ViewModel.
[ ] Remove StandardLayoutMode class

Files Version
2.5.10
Windows Version
10.0.22000
Comments
No response

@d2dyno1 do you agree upon this?

@d2dyno1 @QuaintMako do you agree upon this?

Could you elaborate "Use ICommand instead of using Event Handler"? I think we should be careful where we do that since not all UI events should be handled in VMs
Is IBaseLayoutViewModel interface really necessary?
What classes would derive GroupableBaseLayoutViewModel? Perhaps using composition pattern would be better instead of nesting another abstractions?
Columns are a different story if you ask me. In an ideal world, all layouts (including Column View) would inherit from BaseLayout and use BaseLayoutViewModel. However, with that approach we'd have to eliminate all methods/properties from BaseLayout that depend on a list of Items since Column View uses blades and there's no single layout. I can only see two options to solve the aforementioned issue:

Create another level of abstractions that'd separate a Layout class with items and without.
Keep the BaseLayout as is (with the Items) and just update the collection to only reflect changes in the topmost blade

This should be done gradually. And to close this issue, we need to refactor so many classes including storage item classes and storage enumeration classes.


Method fileWrite always return false

Trying to upload save data to steam cloud. ANE work for achievements, stats and others things perfectly.
Result of fileWrite is always "false". Used writeObject, writeUTFBytes (from FRESteamWorks UNIT tests).
Rewinded ByteArray "position" to 0.
Tryed to write simple strings, not long JSON. Same result - false.
From SteamEvent.STEAM_RESPONSE i recieve event on fileWrite, with
responce = 1,
type = 1
which not informative at all.
Result of fileRead is true. But i have empty save, so it's not much useful.

Question: Do FRESteamWorks have fileWriteAsync? By the nature of actionscript, all blocking things must be async.

Found problem. To activate steam cloud, you must set quota in steamworks, save "cloud changes" on the bottom of page and publish your changes. After this, saves works.


variable "icdict" in line_predictor.py can't be resolved

I know it's defined in params.py but it's not linked to this variable

Sorry for this. Change it to params.icdict will work. This will be fixed soon.

got it, thanks a bunch


Adobe ADMX Templates have moved

Hello.  I have discovered that Adobe, who seems to think that we all have unlimited budgets for their products, have moved the ADMX files to a new location.
As you have the Adobe ADMX policies coming from the FTP, which seems to not getting updates and as FTP is blocked as a Security risk in most places, I wanted to share the information that I located after pulling my hair our in clumps (insert mental image  of Otto von Scratchensniff and the Warner's Animaniacs) with the information and the source of this information.
Source:  https://www.adobe.com/devnet-docs/acrobatetk/tools/DesktopDeployment/gpo.html#gpo-registry-template
Continuous Track
[Acrobat Continuous track]
(https://ardownload2.adobe.com/pub/adobe/acrobat/win/AcrobatDC/misc/AcrobatADMTemplate.zip).
Use this for Acrobat 32-bit and 64-bit, 64-bit Reader, and the unified installer.
[Reader Continuous track]
(https://ardownload2.adobe.com/pub/adobe/reader/win/AcrobatDC/misc/ReaderADMTemplate.zip)
Classic Track 2020
[Acrobat Classic track]
(https://ardownload2.adobe.com/pub/adobe/acrobat/win/Acrobat2020/misc/AcrobatADMTemplate.zip)
[Reader Classic track]
(https://ardownload2.adobe.com/pub/adobe/reader/win/Acrobat2020/misc/ReaderADMTemplate.zip)
Classic Track 2017
[Acrobat Classic track]
(https://ardownload2.adobe.com/pub/adobe/acrobat/win/Acrobat2017/misc/AcrobatADMTemplate.zip)
[Reader Classic track]
(https://ardownload2.adobe.com/pub/adobe/reader/win/Acrobat2017/misc/ReaderADMTemplate.zip)
Template preferences fall into these broad categories:
General enterprise settings: Features such as disabling updates and setting the default PDF handler.
Security: Application security features such as enhanced security, sandboxing, and JS controls.
TrustManager: Trusting Windows OS security zones as defined in Internet Explorer.
Digital Signatures: Adobe Acrobat Trust List integration.

@msfreaks Please close this issue.


Fix sql 380

:sparkles:

Coverage increased (+0.0%) when pulling 050056b1556c82323db689fb8924e095020e1bbd on fix-sql-380 into f17f9f6a724b9cac500aa43089afa80c43453878 on master.


fix: add support for local node_toolchain

Summary
This closes https://github.com/aspect-build/rules_js/issues/899 and it has all the context. Make the _target_tool_short_path path resolve when it’s in an external repo or repo-local.

 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.

Pushed some more commits to fix the node path in coverage.sh.tpl. The failure there also exposed an unset var there that I removed so actual errors can surface. Should go green now.

Thanks, @gregmagolan!


How use html in notifications?

How use html in notifications (Alert)?
Replace in Alert component 16 line from  {{ text }} to v-html="text"?

+1

Sorry for delay. Released version 2.0.13

I'm using
$dialog.notify.error(message, {
      position: "bottom-right",
      timeout: 5000,
    });

but my message value contains some string which is considered like html.  (e.g. <3442>)
I would like the text was considered as plain text is there a way to do that ?

You can use
 <3442>


Export analysis data to CSV

Allows custom designed and better looking graphs for reviews etc.

File -> Export as...
SaveFileDialog with "Zipped CSV files (*.zip)" format
one csv file for each timedomain and freqdomain chart (x,y pairs of points)
keyhistory.csv is a table as a chronological list of events that were captured
devices.csv is would contain data about devices stored in the kbi file


USB DFU support for STM32F1 series

Is this request related to a missing driver support for a particular hardware platform, SoC or board? Please describe.
Currently the samples/subsys/usb/dfu samples can't be compile for STM32F1 series SoC. Trying to compile will result in an error due to Kconfig warnings, as the F1 series is missing the MPU Kconfigs required by USB_DFU_CLASS.
warning: MPU_ALLOW_FLASH_WRITE (defined at C:/Users/yongc/zephyrproject_master/zephyr/arch/arm\core/aarch32\cortex_m\mpu/Kconfig:78) has direct dependencies ARM_MPU && CPU_HAS_MPU && CPU_CORTEX_M && !ARM64 && ARM with value n, but is currently being y-selected by the following symbols:
 - USB_DFU_CLASS (defined at subsys/usb/class/dfu/Kconfig:4), with value y, direct dependencies IMG_MANAGER && USB_DEVICE_STACK (value: y), and select condition IMG_MANAGER && USB_DEVICE_STACK (value: y)Parsing C:/Users/yongc/zephyrproject_master/projects/dfu/Kconfig
Loaded configuration 'C:/Users/yongc/GitLab/boards/boards/arm/ivm22_f103re/ivm22_f103re_defconfig'
Merged configuration 'C:/Users/yongc/zephyrproject_master/projects/dfu/prj.conf'
error: Aborting due to Kconfig warnings

To reproduce
Compile the sample for waveshare_open103z, which has usb defined in its dts.
Additional context
My STM32F1 custom board with custom firmware has USB DFU working using samples from ST. I'm guessing that USB DFU on Zephyr should be able to work without MPU?

The sample can be conpiled if MPU_ALLOW_FLASH_WRITE  is removed from USB DFU class's Kconfig. But dfu-util is not able to open the device after pluggin it into a Windows 10 machine.

After configuring these:

CONFIG_USB_DEVICE_VID=0x0483
CONFIG_USB_DEVICE_PID=0xDF11

It is recognized as "STM Device in DFU Mode" in Windows 10's device manager:

According to STM32 DFU example code and change usb_dfu.h's DFU_VERSION to 0x011A the device is now recognized by ST's DfuSe Demo software. But it says "Unable to find or decode device mapping... Bad Firmware"

I guess it is missing something like this?
uint8_t DFU_StringInterface0[DFU_SIZ_STRING_INTERFACE0] =
  {
    DFU_SIZ_STRING_INTERFACE0,
    0x03,
    // Interface 0: "@Internal Flash   /0x08000000/06*002Ka,250*002Kg"
    '@', 0, 'I', 0, 'n', 0, 't', 0, 'e', 0, 'r', 0, 'n', 0, 'a', 0, 'l', 0,  /* 18 */
    ' ', 0, 'F', 0, 'l', 0, 'a', 0, 's', 0, 'h', 0, ' ', 0, ' ', 0, /* 16 */

    '/', 0, '0', 0, 'x', 0, '0', 0, '8', 0, '0', 0, '0', 0, '0', 0, '0', 0, '0', 0, '0', 0, /* 22 */

    '/', 0, '0', 0, '6', 0, '*', 0, '0', 0, '0', 0, '2', 0, 'K', 0, 'a', 0, /* 18 */
    ',', 0, '2', 0, '5', 0, '0', 0, '*', 0, '0', 0, '0', 0, '2', 0, 'K', 0, 'g', 0, /* 20 */
  };

This is the debug log output:
--- 1 messages dropped ---
[00:08:41.087,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
--- 9 messages dropped ---
[00:08:41.087,000] <dbg> usb_dc_stm32.usb_dc_ep_read_wait: ep 0x00, 8 bytes, 0+8, 0x200027d8
[00:08:41.087,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x00, len 64
[00:08:41.087,000] <dbg> usb_device.custom_handler: bRequest 0x06, wIndex 0x0000
[00:08:41.087,000] <dbg> usb_device.usb_handle_std_device_req: REQ_GET_DESCRIPTOR
[00:08:41.087,000] <dbg> usb_dc_stm32.usb_dc_ep_write: ep 0x80, len 27
[00:08:41.087,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x80, len 0
[00:08:41.088,000] <dbg> usb_dc_stm32.HAL_PCD_DataInStageCallback: epnum 0x00
[00:08:41.088,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x80, status 0x02
[00:08:41.090,000] <dbg> usb_dc_stm32.HAL_PCD_SetupStageCallback:
[00:08:41.090,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
[00:08:41.090,000] <dbg> usb_dc_stm32.usb_dc_ep_read_wait: ep 0x00, 8 bytes, 0+8, 0x200027d8
[00:08:41.090,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x00, len 64
[00:08:41.090,000] <dbg> usb_device.custom_handler: bRequest 0x06, wIndex 0x0000
[00:08:41.090,000] <dbg> usb_device.usb_handle_std_device_req: REQ_GET_DESCRIPTOR
[00:08:41.090,000] <dbg> usb_dc_stm32.usb_dc_ep_write: ep 0x80, len 9
[00:08:41.090,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x80, len 0
[00:08:41.091,000] <dbg> usb_dc_stm32.HAL_PCD_DataInStageCallback: epnum 0x00
[00:08:41.091,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x80, status 0x02
[00:08:41.093,000] <dbg> usb_dc_stm32.HAL_PCD_SetupStageCallback:
[00:08:41.093,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x00, status 0x00
[00:08:41.093,000] <dbg> usb_dc_stm32.usb_dc_ep_read_wait: ep 0x00, 8 bytes, 0+8, 0x200027d8
[00:08:41.093,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x00, len 64
[00:08:41.093,000] <dbg> usb_device.custom_handler: bRequest 0x06, wIndex 0x0000
[00:08:41.093,000] <dbg> usb_device.usb_handle_std_device_req: REQ_GET_DESCRIPTOR
[00:08:41.093,000] <dbg> usb_dc_stm32.usb_dc_ep_write: ep 0x80, len 27
[00:08:41.093,000] <dbg> usb_dc_stm32.usb_dc_ep_start_read: ep 0x80, len 0
[00:08:41.094,000] <dbg> usb_dc_stm32.HAL_PCD_DataInStageCallback: epnum 0x00
[00:08:41.094,000] <dbg> usb_device.usb_handle_control_transfer: ep 0x80, status 0x02

To sum up the question: Could USB_DFU be supported w/o MPU support ?
@jfischer-no any insight ?

To sum up the question: Could USB_DFU be supported w/o MPU support ?
@jfischer-no any insight ?

Yes. But I do not have an overview of what MPU_ALLOW_FLASH_WRITE has for hidden effects.
I guess select MPU_ALLOW_FLASH_WRITE if ARM_MPU or imply MPU_ALLOW_FLASH_WRITE could resolve this issue.
diff --git a/subsys/usb/class/dfu/Kconfig b/subsys/usb/class/dfu/Kconfig
index 11a3fdacf2..f90e8ec348 100644
--- a/subsys/usb/class/dfu/Kconfig
+++ b/subsys/usb/class/dfu/Kconfig
@@ -3,7 +3,7 @@

 menuconfig USB_DFU_CLASS
        bool "USB DFU Class Driver"
-       select MPU_ALLOW_FLASH_WRITE
+       select MPU_ALLOW_FLASH_WRITE if ARM_MPU
        select POLL
        depends on IMG_MANAGER
        select IMG_ERASE_PROGRESSIVELY if SOC_FLASH_NRF

Yes. But I do not have an overview of what MPU_ALLOW_FLASH_WRITE has for hidden effects.
I guess select MPU_ALLOW_FLASH_WRITE if ARM_MPU or imply MPU_ALLOW_FLASH_WRITE could resolve this issue.

Great. @ycsin  Can you have a try and report back ?

Yes. But I do not have an overview of what MPU_ALLOW_FLASH_WRITE has for hidden effects.
I guess select MPU_ALLOW_FLASH_WRITE if ARM_MPU or imply MPU_ALLOW_FLASH_WRITE could resolve this issue.

Great. @ycsin Can you have a try and report back ?

Is this patch different from manually removing MPU_ALLOW_FLASH_WRITE in terms of the functionality? I think the ST's dfuse software and the dfu-util still won't work.

@ycsin Do you have a new status on this point ?

@erwango Nope.. This has became a 'good to have' feature in my current priority list (for F103) and I'm currently migrating from F103 to G0B1.

@ycsin Ok, when you have time please report back of the proposed solution is working.