[CORE-9842] Add Hardcoded InterOp Group Policies to GroupPolicyReader

Adding hardcoded interop group policy to enable early development. This includes the addition of the interop member role, implementation of the new interop group policy and changes in the group policy provider to handle the new group policy type.

Summary: In general the PR conflates "hardcoded data" to allow working from get-go with a possible research what GroupPolicy data is needed for Interop/LinkManager, and both aims are not reached. As this seems structurally wrong approach I deferred from minor comments and just put 2-3 of them, my general questions:

The first DoD (definition of Done)  bullet:
"* Addition of new ‘InterOp’ Member role to be used in the Member Context"
Probably code change is ok, but doesn't really introduce any functionality now.
We created new InteropGroupPolicy which is sibling of MemberGroupPolicy and provides exactly the same getters (as the both are alias for GroupPolicy interface.)
As researched before, we don't need MGM Provided info or static memebers, howeverexisting  Group Policies Parsers and base classes treat them optionally, so in theory we can reuse existing parser/data at start (starting with superset).

There is second DoD bullet which I'm not sure what is current rational for it: "When the GroupPolicyReader needs to fetch the Group Policy for a Holding Identity, it will check if the MemberInfo is an InterOp Identity and if so calls a different class to the GroupPolicyParser "
We created a lot of code with parallel hierarchy of parsers and grouppolicies, but frankly or this is exactly the same code - in some cases we dropped members in static group (but this is anyway non production code), or we just internally hardcocded some values meaning this is not even a proper parser (it's a parser returning the same values regardles of input). Also having completely separated hierarchy will be hard to maintain, what e.g. they introduce new value for "sessionKeyPolicy" and our hardcoded one in parser becomes obsolete, we have no way to detect this unless we run real deployment and LinkManager start refusing to work with our values. What is new or different that existing parsers couldn't do? Again we have identical data class to the existing MemeberGroupPolicy so I guess we should be able to use exactly the same parser?
If we are introducing a parser we should add a real working parser which could show - interop group JSON is different and we need to parse it differently. Also we use our new code only during registration event, but I'm not sure are trigger any registration now, as we don't have any code to initiate registration, the point was to hardcode whenever data is read, not to implement writing of our data.

We don't have the original source of information (VirtualNodeInfo) which writes data to VirtualNodeInfoReadServiceImpl.
Because of this our "pseudo-membership-group" will never appear in GroupPolicyProviderImpl, whenever LinkMnager asks of a HoldingIdentity belonging to this pseudo-group.
Therefore the quick and hardcoded fix is as follows: whenever GroupPolicyProviderImpl fails to find something when asked via API/interface method, then it defers to in memory map with 1-2 Hardcoded GroupPolicies.
This would be sufficient for the last bullet of DoD:
"* A hardcoded GroupPolicy object is returned with default values and fixed GroupName where MGM is Null."

This PR lacks such code - out of box we don't have any hardcode values e.g. "GoldBond" group policy (ang. e.g. For "Alice@GoldBond" and "Bo@GoldBond").

I guess this is somewhat aligned with my specific comments too.
Unless I misunderstood @szymonsztuka .

I guess this is somewhat aligned with my specific comments too. Unless I misunderstood @szymonsztuka .

It's aligned with your first comment, it's a bit orthogonal to your second comment with hard-coded keys as strings (I think we need to produce a pair of PrivatePublic keys, not only have public ones),  the actual way of hard-coding is to be debated

Hi @szymonsztuka
I just want to clear up what is meant by : "When the GroupPolicyReader needs to fetch the Group Policy for a Holding Identity, it will check if the MemberInfo is an InterOp Identity and if so calls a different class to the GroupPolicyParser"
As you mention in your third point we don't have access to vnode info meaning we cannot retrieve the InterOpGroupPolicies in the same way as we do for normal member group policies. The intension is that we modify GroupPolicyProvider to be able to provide both InterOp and regular group policies. GroupPolicyParser is used for specifically getting metadata from the Virtual Node info to parse the cpi's group policy. As we'll be reading the InterOp data from a new kafka topic it makes sense to add a new class to read it from the queue. The original hope was that if we could assert that the Holding Identity's MemberInfo had a specific Interop Role associated with it then we would be able to logically know where to look for the group policy but from the Pr i can see it hasn't worked out that way.


deprecationSite.getFileName is not a function

Since the last Atom update when I run beautify I get a deprecation error and the beautify do not happen:

Sorry for the screenshot: I cannot copy/past the text.

I think this is a duplicate of #9350, since it's the same error.


enable panics if interrupts are disabled.

As seen in #16, sd_softdevice_enable HardFaults if interrupts are disabled.
It'd be cool to detect that and panic with a human-friendly error message so the user can figure out what's wrong.

We could use https://docs.rs/cortex-m/0.6.3/cortex_m/peripheral/struct.NVIC.html#method.is_enabled on the required interrupts.


Support for displaying and searching for account names

This PR depends on #1290.
Provide UI support and some hooks for account names.
Additionally, names for Pontus-X are loaded from their data source. (But only if someone browsers that layer.)
Searching for names is also supported. (But currently produces results only on Pontus-X, since there are no names on other networks.)
Depends on #1396 and #1397.

Closing this in favor of #1398, which can handle more data sources, and has been refactored.


Upgrade to AndroidX as it does not work RN => 0.59

Please refer this link to upgrade to AndroidX.
https://github.com/react-native-community/discussions-and-proposals/issues/129#issuecomment-495686282

i ve used this package  with all new features with react native greater than 0.60 support  react-native-receive-sharing-intent

@ajith-ab This package worked => 0.59.0 version. I found a solution. It is there in the document. But thanks for your suggestion.


Auto clear buffer broken when switching between relative and absolute

original bug report
-absolute mode seems to interfere with the other mode, once you set absolute keys, moving the camera doesnt flush the cache of the relative ones, The only way to do it is to slowly manually scrub. And even once you clear the absolute keys, and go back to relative mode,  the flush cache when u move the camera is broken, even when u toggle the renderer on and off.

not reproduceable


Add "Known and custom plugins" section to Plugins doc

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2316
Originally created by @DouweM on 2020-08-28 16:12:24

We need a better place to link "known to Meltano" to than https://meltano.com/docs/contributor-guide.html#known-plugins.

View 3 previous comments from the original issue on GitLab


bug fix for back button issue

Hi @dileep-gadiraju
I have tested above pr . now back button functionality is available on continue scan page .


Examples of licensed projects not detected by licensee

All of these projects on GitHub are not detected by licensee, but there is Good Reason™ to believe that they are actually licensed. There's also a Good Reason™ that most of them aren't detected by licensee, but I thought there might be some patterns in there that we are missing.

Some patterns I found:

License as part of the readme file (VERY common, but not trivial to detect)
COPYING file (VERY common, as far I know this is generally considered to be a valid synonym for LICENSE)
LICENSE or COPYING with a file name suffix

.txt
.md
(other) (rare)

Scannable metadata files which may include license information:

bower.json (example: https://github.com/webcomponents/webcomponentsjs/blob/master/bower.json)
package.json (example: https://github.com/TooTallNate/node-bindings/blob/master/package.json)
PKG-INFO (example: https://github.com/RDFLib/pyrdfa3; no idea where this file comes from)
Files with suffix .cabal. This comes from Cabal, a package manager for programs written in Haskell. Specification: https://www.haskell.org/cabal/

Implementing scanning those metadata files will probably be different since these files usually follow a strict syntax. I think doing reserach on other metadata files (esp. programming language specific ones) can yield some helpful results as these can be a way to unambigiously specify a license.
Unusual files (only seen once):

https://github.com/OpenKinect/libfreenect uses GPL2 and APACHE20 as files
LICENSE.markdown

Oddly, there seem to be some projects on the list with a LICENSE file but they are still on the list. Example:

https://github.com/square/cubism

All of these projects on GitHub are not detected by licensee

Some of these are now detected. If you'd like, you can use the following script help diagnose:
#!/usr/bin/env ruby

require 'open-uri'
require 'fileutils'

url = "https://gist.githubusercontent.com/bkeepers/8c003ed1aa2da39743ac/raw"
repos = open(url).read.split("\n")

temp_dir = File.expand_path "./tmp", File.dirname(__FILE__)

repos.each do |repo|
  path = File.join temp_dir, repo
  FileUtils.rm_rf path
  FileUtils.mkdir_p File.dirname(path)
  license_file = matched_file = nil

  begin
    Timeout::timeout(5) do
      puts "#{repo}:"
      system "git clone --depth 1 --quiet https://github.com/#{repo} #{path}"
      system "bin/licensee #{path}"
    end
  rescue Timeout::Error
    puts "Ain't nobody got time for that. Moving on."
  end

  puts ""
end

puts "Cleaning up..."
FileUtils.rm_rf temp_dir

@Wuzzy2 Thanks for all this. Responding in line below.

License as part of the readme file

This can be turned on. See https://github.com/benbalter/licensee/blob/master/docs/customizing.md#matching-project-readme-license-references.

COPYING file, LICENSE or COPYING with a file name suffix

This is currently supported.

Scannable metadata files which may include license information:

Currently Ruby and Node support can be turned on. See https://github.com/benbalter/licensee/blob/master/docs/customizing.md#matching-package-manager-metadata. I'd be glad to add review a PR to add support for other package managers.

GPL2 and APACHE20 as files

We currently don't support those file names. We could but I'm hesitant to, because I'd prefer we push the community towards consistency, rather than support every possible way to represent a license.

Oddly, there seem to be some projects on the list with a LICENSE file but they are still on the list. Example:

This is an example of https://github.com/benbalter/licensee/issues/103.

GPL2 and APACHE20 as files

We currently don't support those file names. We could but I'm hesitant to, because I'd prefer we push the community towards consistency, rather than support every possible way to represent a license.

I agree, this practice is very obscure and I only saw it once.


(bug): permission denied throws error

Navigating to a folder to which the current user does not have sufficient permissions throws a
Can't open directory X: Permission denied error. Opening files with insufficient permissions does not work as well.

Will be fixed in the next release 👍🏻


Test videocomposer two steps

[x] control randomness of inputs: remove randomresize from misc_transforms; change start_index to deterministic index; use only one video as the training sample.
[x] control randomness of model: change dropout to zero. VAE encode deterministic
[x] save ckpt for every step(epoch, since one epoch has one step);
Running command:

bash run_train_two_steps.sh

Change --ms_mode  in run_train_two_steps.sh, graph mode takes too long. Now I use Pynative mode.
This will save some intermediate .npy and .pkl during the forward pass in order to align the intermediate results with torch_npu. Checking gradients is not relevant to these files, so you may ignore them.

This PR was important for localize the precision issue of VC. But it's not appropriate to merge into the main model code. What about leave it in personal repo if there is no time to wrap into an independent debug script?

Of course. I did not intend to merge this pr into the main branch. It was for other colleagues to take as a reference. I am about to close this PR.


Bug: SMTP throws AttributeError on teardown

It seems smtp server isn't compatible anymore with the recent aiosmtpd vesrsion.
I'm getting the following error when teardown the smtp server. Before everything worked fine.

        """

        # This mostly copies the implementation from Controller.stop(), with two
        # differences:
        # - It removes the assertion that the thread exists, allowing stop() to
        #   be called more than once safely
        # - It passes the timeout argument to Thread.join()
        if self.loop.is_running():
>           self.loop.call_soon_threadsafe(self._stop)
E           AttributeError: 'Server' object has no attribute '_stop'

__pypackages__/3.9/lib/pytest_localserver/smtp.py:133: AttributeError

Darn, well thanks for pointing this out! Would you be interested in fixing this yourself? If not, I can work on it sometime soon, maybe this weekend or next week.

Sure I can take a look. Seems that private method (_stop), which was removed. Not sure if it just can be replaced with the public stop method?

I guess you might as well try it and find out. I'm not super familiar with the aiosmtpd API; I've never used it myself aside from this project.

Do you know why pytest-localserver actually was using the private method in the first place?

https://github.com/pytest-dev/pytest-localserver/blob/master/pytest_localserver/smtp.py#L133

I will take a look tomorrow. Here it's already 1:00 am.

No problem, thanks for being willing to help!
I see the comment in the source code says I copied the implementation of aiosmtpd.controller.Controller.stop(), with a couple changes to keep the behavior of the Server.stop() method consistent with older versions of pytest-localserver. So I don't think it calls _stop() for any reason other than that Controller.stop() does. Of course it would have been better to use a public API instead of the private method, but I couldn't find a suitable API in aiosmtpd, and I was just trying to do something quick to break our dependency on the deprecated smtpd module from Python's standard library. I seem to remember that if I didn't call it, something went wrong, but I don't remember what exactly that was - it's been a while.

It was quite easy to fix . In aiosmtpd the _stop method was replaced by cancel_tasks.


Add: SBT Interior [BR]

Channel Name
SBT Interior
Alternative Names (optional)
No response
Network (optional)
Sistema Brasileiro de Televisão
Owners (optional)
No response
Country
BR
Subdivision (optional)
BR-SP
City (optional)
Araçatuba
Broadcast Area
s/BR-SP
Languages
Por
Categories (optional)
General
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://sbtinterior.com/
Logo
https://directostv.teleame.com/wp-content/uploads/2017/11/SBT-Interior-ao-Vivo-Online.png
Notes
No response

No link provided, but found just the same. It will be included in the next release.

No link provided, but found just the same. It will be included in the next release.

https://cdn.jmvstream.com/w/LVW-10801/LVW10801_Xvg4R0u57n/playlist.m3u8


SnowSplitter output cube names are now the same as their StaGE couterparts

Addresses https://metoffice.atlassian.net/browse/EPPT-650
While testing the EPP Precipitation Type workflows, I have found that the output cube names for rain/snow rate/accum data are not the same as the same data found on StaGE. For accumulation data, the rain and snow outputs had the names of the precipitation cube. This PR modifies the new cube name generation, which is generated from a precipitation cube name, so that:

"precipitation" is replaced with either "rainfall" or "snowfall" (not "rain" or "snow")
"lwe_", which means liquid water equivalent, is removed if the output is rain.

Testing:

[x] Ran tests and they passed OK
[x] Updated KGOs: https://github.com/metoppv/improver_test_data/pull/42
[x] Added new tests for the new feature(s)

These changes look fine, however I think additional changes are required to the acceptance test data to accommodate them. At the moment this PR causes the snow_splitter CLI acceptance tests to fail as the kgo cubes have incorrect cube names.

Oh dear. I must have been falling asleep to miss this. Acceptance Test PR added: https://github.com/metoppv/improver_test_data/pull/42


NEXT-39679 - fix paymentstatus cannot be exported

1. Why is this change necessary?
Payment status in import/export can't be exported.
2. What does this change do, exactly?
Adds the OrderTransactionDefinition Class to the FieldSerializer.php
3. Describe each step to reproduce the issue or behaviour.
Create an export with transactions.stateMachineState.translations.DEFAULT.name
This Value is then empty
4. Please link to the relevant issues (if any).
NEXT-39679
5. Checklist

[ ] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfill them.

 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.


charmrun ++local dies without error message on non-existent program

Original issue: https://charm.cs.illinois.edu/redmine/issues/1782

Calling charmrun with a non-existent program produces a useful error message:
jim`belfast$/Projects/namd2/charm-6.9.0/netlrts-linux-x86_64-smp-iccstatic/bin/charmrun /foo
Charmrun> scalable start enabled.
Charmrun remote shell(abijan.0)> Cannot locate this node-program: /foo
Charmrun remote shell(abijan.0)> Exiting with error code 1
Charmrun> Error 1 returned from remote shell (abijan:0)

Except with the ++local option:
jim`belfast$/Projects/namd2/charm-6.9.0/netlrts-linux-x86_64-smp-iccstatic/bin/charmrun ++local /foo
Charmrun> scalable start enabled.
Killed

Original author: Dong Hun Lee
Original date: 2018-01-26 09:54:02

~~https://charm.cs.illinois.edu/gerrit/#/c/3575/~~ https://github.com/UIUC-PPL/charm/commit/d553faa0530a46602ac96036cc3ebbb305971570
The error comes from that after execve in line 5368 returns, indicating that exec was unsuccesful, the following line's fprintf isn't printed.
I tried fflush but does not seem to work.
Instead, before spawning processes in PE's, a quick check is done to see if the program path is valid.
Original error messages are in ssh_script function (line 4507), but this isn't called when ++local option is used.

Original date: 2018-01-26 15:10:04

Passing a file that is not executable is also handled badly:
jim`sunnyvale$/Projects/namd2/charm-6.9.0/netlrts-linux-x86_64-smp-iccstatic/bin/charmrun ++local Make.config
Charmrun> scalable start enabled.
Killed

We need to figure out why an error message is not displayed.

Original date: 2018-01-26 17:16:31

Dong Hun Lee wrote:

~~https://charm.cs.illinois.edu/gerrit/#/c/3575/~~ https://github.com/UIUC-PPL/charm/commit/d553faa0530a46602ac96036cc3ebbb305971570
The error comes from that after execve in line 5368 returns, indicating that exec was unsuccesful, the following line's fprintf isn't printed.
I tried fflush but does not seem to work.
Instead, before spawning processes in PE's, a quick check is done to see if the program path is valid.
Original error messages are in ssh_script function (line 4507), but this isn't called when ++local option is used.

See my comments in code review.  The code dups stdout (1) instead of stderr (2).


[Bug] Creating near infinite amounts of folders

PoB version: 1.4.64
OS: Windows 10 64-bit.
I was reorganizing my folders and I accidentally dropped the same folder on top of itself and I got the error: "Couldn't delete 'C:\Users\Epsynus\Documents/Path of Building/Builds 3.0 builds/HC Viable/Duelist' : Directory not empty"
When I hit Ok. It repeats the message but adds another Duelist:
"Couldn't delete 'C:\Users\Epsynus\Documents/Path of Building/Builds 3.0 builds/HC Viable/Duelist/Duelist' : Directory not empty"
The more I hit the more directories it create!
Going through File explorer my current Duelist folder looks like:
"C:\Users\Epsynus\Documents\path of building\Builds\3.0 builds\HC Viable\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist\Duelist"
I can repeat this as much as I want by just dropping another folder on top of itself and hitting the Ok button a ton.
It isn't that horrible of a bug since all you need to do is just force close the program and not drop a folder on itself in the first place. But thought I'd let you know.

Fixed in the next update.


1.4.0-RC2: cannot load a plugin that depends on sbt-dependency-graph transitively

steps
Depend on a plugin that depends on sbt-dependency-graph transitively (e.g. https://github.com/lightbend/sbt-paradox-dependencies)
problem
[error] Some keys were defined with the same name but different types: 'moduleGraphStore' (sbt.Task[net.virtualvoid.sbt.graph.ModuleGraph], sbt.Task[sbt.internal.graph.ModuleGraph])
[error] Use 'last' for the full log.
[warn] Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore? (default: r)

From this error, it's not to clear what's going on, and even if you are aware of the issue, it's not so easy to see which sbt plugin has this transitive dependency.
notes
This is a consequence of folding sbt-dependency-graph into sbt 1.4.0.
It might be fine (there are probably few plugins that have this transitive dependency, and for those it's probably not a problem to introduce a new version where the transitive dependency is dropped), but I wanted to create this issue to make sure we're aware of it and it can come up in a web search for someone who runs into this problem :)

@raboof Thanks for the report! Maybe I can change the intermediate key names to avoid this?

Looks like this came up with the community build too: https://github.com/scala/community-build/pull/1223#issuecomment-698060910. Renaming the intermediate keys sounds like a good approach.


fixes, cleanup and lint

We use your great project a lot now at x.ai, so a huge thanks from the team.
What I've done:

removed Procfile as Heroku can figure it out by looking at "start": "node app.js" in package.json.
fixed some bugs.
the code now gets a ✓ from jshint.com.
if it looses database connection, it will retry every 5 seconds.

Sorry for the super delayed reply, I'm not sure why I'm not getting these notifications anymore..
A huge thank you for your time to do some clean-up work, I'll review it and get back to you! Looks really nice so far!

I am here to help a little, @webjay if you rebase/squash this I will merge it.

This one may be too outdated to be merged in. Major file structure changes happened since this PR.

Yeah, I think a lot of good things here can be redone over the current state of affairs. All I can tell @webjay is that you'll get a responsive group of maintainers now if you are ok doing the work twice!

I cherry picked things from here in https://github.com/andrewvy/slack-pongbot/pull/22 and https://github.com/andrewvy/slack-pongbot/pull/19.


fix: APIScan reports Documentation Not Found errors

This PR aims to fix APIScan Documentation Not Found errors in order to bring our prebuilt version of zeromq into compliance.
Changes
The APIScan errors come from wepoll, which is an upstream dependency of zeromq-prebuilt:
zeromq-prebuilt -> zeromq.js -> libzmq -> wepoll.
As a result, this PR is fairly involved and involves:

Creating a patch for libzmq that adjusts the CMake file and wepoll source code to remove the KeyedEvent references.
Modifying the zeromq.js build script to apply the libzmq patch.
Modifying zeromq.js/binding.gyp to add a new dependency on synchronization.lib
Modifying the zeromq.js patch to contain those new changes.

Next steps and blockers
I have set this PR to July due to the following blockers:

We haven't tested this change in Jupyter, and I'd rather merge this in early next month in case of compatibility issues or issues with the new wepoll implementation.
We're currently blocked on waiting for APIScan to support our current Azure infrastructure setup. I have commented out the APIScan tasks for now, but we don't currently have the ability to check whether this PR fixes the APIScan Documentation Not Found errors.

For the alpine failures, you can change to mcr base images mcr.microsoft.com/mirror/docker/library/alpine:3.16 and download node binaries from https://github.com/microsoft/vscode-node/releases/tag/v20.14.0-282653 to avoid the rate limits.

I'm able to run the Alpine steps locally on Windows with pwsh, but the steps fail in CI.
Example command that works on Windows: docker run --volume ${pwd}:/app --workdir /app --privileged builderx64 sh -c "$env:DOCKERCMD" where pwd=C:\Users\raymondzhao\work\zeromq-prebuilt\zeromq.js, DOCKERCMD=apk add --no-cache pkgconfig curl tar python3 make gcc g++ cmake musl-dev npm patch && npm i -g pnpm@latest-8 && pnpm install && npm run prebuild
I could try using a Windows agent and pwsh tomorrow, but let me know if you've other ideas on what might be happening @deepak1556

Seems like a permission issue, try creating a sudo user and run the container steps via that user https://github.com/nodejs/docker-node/blob/2358b05a03119c1342a4a1cb8ffb21b026737029/18/alpine3.19/Dockerfile#L5-L6. Also see if updating to the latest npm helps

I have fixed the Alpine builds.
There are still a few action items I'd like to complete before I merge this PR:

[ ] Wait for Guardian to update their APIScan to the newest version so that we can use our Service Connection to authenticate into it. Currently, the APIScan task is commented out on all of our pipelines. ETA 1 to 2 weeks from now.
[ ] Confirm with a working APIScan run that the patch removes our dependency on undocumented APIs.
[ ] Perform another round of testing in wepoll, libzmq, and maybe even jupyter to confirm that the patch works as expected.

Moving to August. I increased the parameters of test-reflock.c and found out that my current implementation deadlocks.
I noticed a very interesting feature of NtReleaseKeyedEvent, which is that it actually waits for another thread to call NtWaitForKeyedEvent (source). Therefore, NtWaitForKeyedEvent and NtReleaseKeyedEvent wait for each other to be called before proceeding.
I'm currently experimenting with adding condition variables to each reflock struct instead of using WaitOnAddress to represent the reflock_unref() threads and the reflock_unref_and_destroy() thread waiting for each other.

I'm a lot more confident in my changes now, though would still like to wait until APIScan is enabled again to check that this PR resolves the Documentation Not Found issue.
I plan to check the wepoll test coverage from libzmq and jupyter during debt week.

I have confirmed that this PR resolves the Documentation Not Found issue!

Verification build https://dev.azure.com/monacotools/Monaco/_build/results?buildId=287328&view=results
I also haven't yet confirmed test coverage of wepoll in libzmq and jupyter, but I did test the wepoll changes using wepoll's test_reflock.c and confirmed that there's no deadlock (the older WaitOnAddress implementation had deadlocks).

@deepak1556 this PR is ready to review. There are still Missing Symbols APIScan results, but I'd like to resolve those in a future PR.

The last verification build failed due to a GitHub outage.
Latest verification build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=288203&view=results


Fix issues/259

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Change HTTP to HTTPS for REST API tests. Basically, test class is the client, it must have truststore to validate the SSL handshake with the broker. Hence set truststore location and password via system properties.
Resolves: #259
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

How does is work with the CLI tool? AFAIR we do not set those system properties when we use the CLI tool.


Allow element attribute class

Currently only className is supported.

Currently all the jsx properties translate to properties of DOM objects, rather than HTML attributes.  Hence className, and innerHTML.  But it probably makes sense to support some well-known html attributes in addition.  This is easy to do, and I don't see much reason not to.
Can you think of any other than class?

Can't think of any others offhand.

I made the change in df8f99c.  I didn't make any documentation changes.  I haven't published the npm package or sandbox, but it will be in the next release/publish.


Automate Lemur release

Lemur's current release publishing is done manual, which comes with overheads and slows down the release cycle.
Automating this operation would allow Lemur to make more frequent releases, for instance at least once a week to pick up the latest dependency updates.

Coverage decreased (-0.02%) to 60.689% when pulling 8d44ab2124f9e5a7c06f4ddb6fa3efd605b5498a on hosseinsh-publish-release into df17337075fee5ab7dbb414fa0019751760613aa on master.

Coverage decreased (-0.02%) to 60.689% when pulling 8d44ab2124f9e5a7c06f4ddb6fa3efd605b5498a on hosseinsh-publish-release into df17337075fee5ab7dbb414fa0019751760613aa on master.


[3.6][Bug][Has Fix] Couple of issues with table field

Hi guys, I have these problems with table field. I'm the only one with issues below?
Case1: With casting to array.

Issue: If you create a new record and add two row to table field and the second row is empty it will save the second (empty) row to json too. If you edit this record you get a json in your hidden input like this:

[{"db":"value","type":"value","manufacturer":"value"},[]]
instead of this
[{"db":"value","type":"value","manufacturer":"value"},{}]
Now if you add values to second row it won't be saved!

Issue: If you add the table column to your show() method and if you have empty field(s) in your table field row you get the following error when you open show view:

Undefined index: type (View: /home/brightstaff/laravel_backpack/vendor/backpack/crud/src/resources/views/columns/table.blade.php) (View: /home/brightstaff/laravel_backpack/vendor/backpack/crud/src/resources/views/columns/table.blade.php)
Where type is the first field that you leaved empty in the row.
Case1: With casting to object or without casting

Issue: The empty row problem is gone away when you use casting to object or you don't setup casting.
Issue: If you add the table column to your show() method get the following error when you open show view:

Cannot use object of type stdClass as array (View: /home/brightstaff/laravel_backpack/vendor/backpack/crud/src/resources/views/columns/table.blade.php) (View: /home/brightstaff/laravel_backpack/vendor/backpack/crud/src/resources/views/columns/table.blade.php)
I use:
PHP 7.1
Laravel 5.7
BackPack 3.4

I go with casting as object and override the table.blade.php column type with an if isset to solve this temporary:
@foreach($columns as $tableColumnKey => $tableColumnLabel)
		<td>
			<?php if (isset($tableRow->$tableColumnKey)):?>
			{{ $tableRow->{$tableColumnKey} ?? $tableRow[$tableColumnKey] }}
			<?php endif; ?>
		</td>
@endforeach

Thanks for the bug report @danielbidala !   Happy to test & merge it if you do. You should be able to easily do this one from the Github interface.
Let me know.
Cheers!
PS. Let's leave this open until it's solved.

Hi @danielbidala ,
You're totally right - thank you. I've just merged the fixes into 3.6.28, it'll be available with a composer update later today,  after I merge a few more non-breaking changes and tag it.
Thanks again! And sorry for taking so long to fix this.
Cheers!


sys/sys/param.h in freebsd/main loses comment after __FreeBSD_version

Commit 9e0ba9536b7cbba0e81e561458e95aafb65ee485 removed the /* Master, propagated to newvers */ comment after __FreeBSD_version in sys/sys/param.h in FreeBSD's main branch aka 14.0-CURRENT.
synth now believes I'm running FreeBSD 1.0. I temporarily doctored my /usr/include/sys/param.h and that will hold up until the next time I upgrade my "LiveSystem".
The same commit says this is proper way to extract __FreeBSD_version:
awk '/^\#define[[:space:]]*__FreeBSD_version/ {print $3}'
I believe there's a small mistake in the regexp and it should be ^\#define[[:space:]]+__FreeBSD_version.
At any rate, get_osversion_from_param_header in src/replicant-platform.adb should be changed to cope with the missing comment.

Thank you for a very detailed explanation with links.
At first glance, the get_osversion_from_param_header should not have been affected by removing the comment and the spaces that precede it.  But obviously your experience says otherwise.  I'll take a harder look.

okay, I see the logic flaw now

Okay, I believe I fixed it and created a new release, 2.11.
Can you let the port maintainer know so he can bring it in?  And close the issue when that's done?

[port maintainer mode on]:  landed in main (https://cgit.FreeBSD.org/ports/commit/?id=a53b9c858eb613ea9f1e79d0b31655d747cc5275) and quarterly (https://cgit.FreeBSD.org/ports/commit/?id=8527b435b418ca72092601127e9229f949663cb8) branches

Synth 2.11 works as expected in freebsd/src/main post commit 9e0ba9536b7cbba0e81e561458e95aafb65ee485.


different parameter in one batch

i want use generate() function with batch inputs, and each input in batch inputs with different parameter, such as temperature, presence_penalty.

This is not possible at the moment. If you want to use batch effectively, you will need to use same parameters. Otherwise, I suggest to do multiple requests in parallel.


XIVY-9398 disable script validation temporarily

disable validation, while it doesn't work as expected in this project

hmm yes it fails in the compile phase... as the validation of ivy-processes is exectuted in this phase... with the idea, that it will reveal similar problems:

compiler for java exposes java source code issues
validation for processes exposes ivy-script scriptlet issues


@Zaehiel maybe can you try and find out what tool could be used to convert like that male-idle.glb model was? I tried with numerous tools, I even got an expert in blender from fiverr to try and sort it out but could not make it :)

          @Zaehiel maybe can you try and find out what tool could be used to convert like that `male-idle.glb` model was? I tried with numerous tools, I even got an expert in blender from fiverr to try and sort it out but could not make it :)

Originally posted by @Radutiu-Horatiu in https://github.com/readyplayerme/visage/issues/33#issuecomment-1583285652

Hi @Radutiu-Horatiu,
I hope this message finds you well. I wanted to take a moment to address the question you raised and provide some clarification regarding the matter at hand. Firstly, I want to express my gratitude for your active participation and engagement with our repository. Your contributions are highly valued, and we appreciate your efforts in making it a vibrant community.
However, upon carefully reviewing the details you provided, I must inform you that the issue you mentioned appears to be unrelated to the repository's specific focus. As a result, it will be necessary for us to close this particular matter. Please understand that this decision is not a reflection of the importance or relevance of your question, but rather a means to maintain a streamlined and organized repository environment.
We strive to ensure that our repository remains dedicated to addressing issues directly tied to its purpose, thereby benefiting all users who visit and engage with the platform. To that end, I would suggest seeking assistance or guidance in a more appropriate forum or platform where you can receive the specialized support you require for the matter in question.
I want to emphasize that your inquiry is valuable, and we encourage you to continue participating actively in our repository. Please feel free to raise any concerns or questions that align with the intended scope of the repository, and we will be more than happy to assist you to the best of our abilities.
Once again, I sincerely appreciate your understanding in this matter. Your involvement plays a crucial role in fostering a collaborative and supportive community, and we value your continued contributions. If you have any further inquiries or concerns regarding the repository's guidelines or related topics, please don't hesitate to reach out, and we'll be glad to provide any necessary assistance.

This answer looks like ChatGPT generated :))) Ok mate, I will figure it out on my own.


Set structure?

I was wondering since you have a StrMap in here if you planned to add a Set or could point me to an implementation that works well enough and is well-typed

Immutable.js has a Set implementation that's very well-typed and mostly fantasy-land compliant. I'm working on making it fully fantasy- and static-land compliant and integrated with fp-ts, but you can mostly just start using it.
Most of the implementations in fp-ts are naive, so for heavy duty collections, you're usually better off with one of the Clojure style implementations anyway.
Immutable.js (really good types)
Mori (more functional, no types)

I've had a really bad time with using Immutable.js in any context, so I'll be looking forward to seeing your compatibility layer sometime

Oh really? If you tell me some of the issues you had, I could look into improving some of them in the compatibility layer.

Speaking of native, you'll have the ES6 one for reference as well!

I've been mulling this one over and I think the best course of action is two-fold:
One, give ES6 Sets the same treatment we give to Arrays, so there's a well-typed Set available.
Two, implement lattices (as well as join- and meet-semilattices). Set would be a join-semilattice, but not a lattice I think.
Lattices are cool abstractions that are often forgotten for some reason. They're like monads for idempotent operations. I think figuring out how they work might give us some cool new toys for fp-ts.
Tree operations tend to be semilattices too IIRC

Set would be a join-semilattice, but not a lattice I think.

@SimonMeskens if Set<A> represents the collection of all subsets of A (i.e. the power set of A) then is a lattice with respect to intersection (meet) and union (join)

Ah okay, I see now. I'll have to make some time in my schedule to write this module.


Properly handle "inline"-encrypted multipart/alternative

Several tools, e.g. Mailvelope, produce multipart/alternative-messages in which both parts contain the ciphertext; and in the html-part it's enclosed in a little HTML.
Example:
Content-Type: multipart/alternative; boundary="94eb2c074e843f8e250550e728d4"

--94eb2c074e843f8e250550e728d4
Content-Type: text/plain; charset="UTF-8"

-----BEGIN PGP MESSAGE-----
Version: Mailvelope v1.7.2
Comment: https://www.mailvelope.com

wcFMA2zW/XKwDLjKAQ//eVs/Ng+IcfUt+QuYLHZbZAsKwD9qhXs06DWRh6zY
1sO2hI59CMvoZ1P+DzMOPBqpBmQ438y3+kxEzOJVh1KxKJukFEv39m/QZDOk
[...]
j15W9B36eQcqqwz9MJ53/V0k5kvGiTTdQt+tH09+NwMHSjQVBFrhat8bVCdM
K3GxDIArXwi/yxRdg0IEqBdZvBxzcL8A3JVS0xzuOUXiErcQ5jTRt9IaiM/d
=wXg6
-----END PGP MESSAGE-----

--94eb2c074e843f8e250550e728d4
Content-Type: text/html; charset="UTF-8"

-----BEGIN PGP MESSAGE-----
Version: Mailvelope v1.7.2
Comment: https://www.mailvelope.com

wcFMA2zW/XKwDLjKAQ//eVs/Ng+IcfUt+QuYLHZbZAsKwD9qhXs06DWRh6zY
1sO2hI59CMvoZ1P+DzMOPBqpBmQ438y3+kxEzOJVh1KxKJukFEv39m/QZDOk
[...]
j15W9B36eQcqqwz9MJ53/V0k5kvGiTTdQt+tH09+NwMHSjQVBFrhat8bVCdM
K3GxDIArXwi/yxRdg0IEqBdZvBxzcL8A3JVS0xzuOUXiErcQ5jTRt9IaiM/d
=wXg6
-----END PGP MESSAGE-----

--94eb2c074e843f8e250550e728d4--

gpgme chokes on the html-part when mail-gpg tries to decrypt the body (GPGME::Error::NoData).
In my eyes it's mail-gpg's reponsibility to find decryptable ciphertext or deal with non-decipherable parts. But should undecryptable ciphertexts be stripped? Or be left untouched in the mime-structure? And how to communicate the failed attempt to the calling code?
A little fix that just leave mime-parts of type 'text/html' alone is already done. If you would like it as a pull request, please let me know.

I just noticed that mail-gpg inconsistently checks for inline-pgp-data.
In encrypted_inline? it uses
part.body.include?('-----BEGIN PGP MESSAGE-----')

In signed_inline? it uses
mail.body.to_s =~ /^-----BEGIN PGP SIGNED MESSAGE-----/

The latter check doesn't match the HTML-wrapped inline-ciphertext, so signed-but-unencrypted messages are not affected.
The previously reported bug could also be "solved" by strickter checking for inline-ciphertext. mail-gpg would then claim that such a message is not encrypted — that's not really correct, but at least it wouldn't raise exceptions.

A little fix that just leave mime-parts of type 'text/html' alone is already done. If you would like it as a pull request, please let me know.

Are you not interested or did you just miss this?

I missed that. Please go ahead with the PR :)

thanks for the PR!


Azerbaijan large update

update holidays
observed dates calculation completely rewritten

Seems something wrong with test workflow on Github

Seems something wrong with test workflow on Github

Yes, the coverage is broken. This https://github.com/dr-prodigy/python-holidays/pull/862 might help.

Coverage remained the same at 100.0% when pulling b30298d7b1787e0d88787dbde98af55fbe44eb0e on KJhellico:upd-azerbaijan into 6a04aab15fc9d4a0dc422acb1f65e5441bfb0642 on dr-prodigy:beta.


single quote bug

Looks like you are checking in a failing regression test for a bug discovered
possibly during a recent bug bash, similar to the "josh.wang/fail_single_quote" P.R.
onto "master" branch.  I'd suggest getting the test and it's fix P.R.'d onto master
branch.  After that, we'd cherry-pick the change onto "2.0.x" .  The "1.5.x" is
frozen since we've moved past this set of versions.

yeah was just checking to see if the bug was just on the customer version. made a similar PR on master already.
https://github.com/optimizely/objective-c-sdk/pull/236


Docker: running wdi5 tests in a selenium grid or standalone

moved from old repo setup

docker setup with single container
docker setup with selenium grid

@vobu would we like to have the two docker executions in github pipeline? I suggest to postpone this to -> opensource ;)


Change prettier workflow to git hook

Change request from #24

@viniciuskneves I think this will do the trick :+1:


Prisms!

Need to write as many interesting prisms over HsDecl as we can. Import lists, export lists, patterns, bindings, use sites, weird cases I've missed thus far, etc etc.
These should be in the form Prism' HsDecl RdrName or Prism' HsDecl [RdrName] so they can be trivially added to queries.

I'd love to take a look at this, though I might need to ask a few questions. I've used Lenses in a couple of different Scala libraries, and briefly played around with lens-aeson, but it's all still pretty new to me in Haskell.

Feel free to spend some time on this! The result probably won't be immediately mergeable, but we'll be able to use any prisms you write to build filters and queries down the track.
Take a look at the ones that already exist (there's like five of them, they do all the searching in hgrep): https://github.com/thumphries/hgrep/blob/master/src/Language/Haskell/HGrep/Query.hs#L34
Take a look at a bit of related work I was doing, the topic/filter branch is probably a better starting point: https://github.com/thumphries/hgrep/commit/91f93a02a40d68e58a914c1f19467fea5e087cd0
In general, if you can write a buncha prisms from HsDecl to RdrName or [RdrName] and clearly label what they're for, preferably with some example Haskell source alongside, that will be really really useful. Create PRs against topic/filter instead of master and I'll try my best to answer any questions you have.
Note that I barely know lens, just enough to be dangerous.

I'd also quite like to set up something like tasty-golden for these prisms.
For each Prism' HsDecl [RdrName], have a bunch of Haskell files to use as inputs, and verify that all the right names get matched. That'd also be quite useful, if it sounds more approachable or interesting.

Thanks for all the pointers! I'll give you a shout if i get stuck / have any questions

hi @thumphries i've been pretty ill with a chest infection so i'm only just getting round to looking at this. apologies for the delay!

actually forget that, i think i've found it!

Don't apologise, there's no hurry! Thanks for spending your time looking and thinking about this, it will be really useful when I try to add complex filtering to hgrep.

I could well be missing something but for example, import lists, don't seem to be exposed by HsDecl. It exposes foreign imports, but i don't seem to see anything to find standard imports.

exactprint gives us a HsModule, so prisms from HsModule to names are what we'd want for things like import lists. You can see the definition of HsModule in HsSyn: https://downloads.haskell.org/~ghc/8.0.2/docs/html/libraries/ghc-8.0.2/HsSyn.html

Hi @thumphries if you don't mind i have a lens question. I was looking at the export lists yesterday. I have the following prism:
_HsModule . _2 . _Just . _unloc . folded . _unloc . _IEModuleContents . _unloc . to (compareModuleName name)

I added some debug to see the result of applying this to the module that I get from ParsedSource. I can see that it's just checking the first entry in each export list. I'm guessing the issue is with how i'm using folded, but I'm not sure what i need to do to rectify it.
Sorry if this is basic stuff, but any pointers would be great :)


Circular queue

Have you read the Contributing Guidelines on Pull Requests?
yes
Description
Circular queue using C
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
#662
Fill up the DCP form
If you are a part of DCP 21, fill up this form after the PR is merged.
The links provided in the form will be checked thoroughly, if someone tries fooling around, strict actions will be taken against the individual.

Done. Sorry, Idk y they both were coming together in both the pr's

@RC2208 you mistakenly deleted the priority queue program as well. Please add it back. The overall PR should just add one file. You changed 2 files , added one and removed one

I added it back. Can you check now?

Looks great !! Nice work @RC2208


Added a new MIDIDriver class

New MIDIDriver class to handle MIDI hardware.
Also implements a new Input event class, InputEventMIDI.
Works with:
OS X using CoreMidi
Linux using ALSA
Windows using WinMM
MIDI events can be used on _input(event):, for example this is me playing two notes on my midi keyboard (printing the pitch and velocity):

PD: This is only for receiving events from MIDI hardware, sending events to MIDI hardware is not implemented in this PR.

Should the InputEvent have different casing from the class name? MIDI vs Midi.

We already have class names like HTTPRequest, so it should be MIDIPlayer.

As I mentioned many times, I don't think MIDI has any use case as engine core.
The situations where something like this would be needed are so few, if any, that it makes more sense to support something like this ad a GDNative add-on than in the core engine.

@aaronfranke @Calinou I've just renamed it to MIDIDriver
@reduz This doesn't implements any MIDI playback code though. This adds support for MIDI hardware such as MIDI controllers, MIDI guitars, etc. This code in my opinion belongs to the core.

Code is completed now, can be reviewed.
Note: If anybody tests this on Linux only one application can be actively connected to the rawmidi ALSA driver.

After thinking about it, i guess this makes sense for the whole music games genre, like rock band instruments (which are MIDI), games that teach you to play piano, and that kind of stuff. PR does not seem too big either, so it may be OK to merge if anyone else think it's ok.

Great work, thanks a lot!

Does this support Android currently?


Local test is impossible

GET https://localhost:3443/socket.io/?peerId=dlrfs8m8&roomId=k6tbbiwr&EIO=3&transport=polling&t=NYBxcAI net::ERR_CONNECTION_REFUSED
I got this error when I try to join the room in the local development environment.
What is the reason? I can't go now

Try looking for errors in the edumeet server logs using:
docker-compose logs -f  edumeet

Seems like SSL certificate problem to me...
Browsers no longer allow self-signed certificates if not properly configured.
I suggest  https://mkcert.dev/

You can visit the :8443 url first, accepting the certificate exception, and then you can visit the :3443 endpoint.


Update test-file-04.md

@v-caxian : Thanks for your contribution to the Azure documentation! The author, @TinaLiuGirl, has been notified to review your proposed change.

@v-caxian : Thanks for your contribution to the Azure documentation! The author, @TinaLiuGirl, has been notified to review your proposed change.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/Articles/test-file-04.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@v-caxian : Thanks for your contribution to the Azure documentation! The author, @TinaLiuGirl, has been notified to review your proposed change.

#please-close


could you help add JMA indicador?

it seems JMA behaves better than EMA in some situations. and pandas_ta already has JMA indicator(pandas_ta.jma).
here are the formula info:
https://br.tradingview.com/script/nZuBWW9j-Jurik-Moving-Average/
http://jurikres.com/catalog1/ms_ama.htm

Yea can look to add JMA, (https://c.mql5.com/forextsd/forum/164/jurik_1.pdf)

Yea can look to add JMA, (https://c.mql5.com/forextsd/forum/164/jurik_1.pdf)

sure, i will try to implement incremental JMA indicator, if finished testing, i will post my JMA draft here.
but my programming skill is not good, neither my math, it might take some time for me to do it.

did I misunderstand you? you wanna me add JMA to Hexita?

@iSkywalker168, I have added JME, It's very accurate to pandas_ta, although some decimal places may differ. This implementation will also create JMA with only one candle available, so first few (period length) will be volatile.
Closing this for now, can be re-opened if needed.


FIX 2. missed included in installed nana consumed with cmake

from https://cmake.org/cmake/help/v3.0/command/target_include_directories.html

Include directories usage requirements commonly differ between the build-tree and the install-tree. The BUILD_INTERFACE and INSTALL_INTERFACE generator expressions can be used to describe separate usage requirements based on the usage location. Relative paths are allowed within the INSTALL_INTERFACE expression and are interpreted relative to the installation prefix. For example:

target_include_directories(mylib PUBLIC
  $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include/mylib>
  $<INSTALL_INTERFACE:include/mylib>  # <prefix>/include/mylib
)

This works, no more include errors.
You should not commit commented code though. There is already a lot of commented code in cmake files.

Travis CI XVFB Breaking Change

Use #define NOMINMAX just before including windows.h, you may also need to add #define WIN32_LEAN_AND_MEAN.

Thanks for your PR!


decode function potential bug

Hi guys, just wanted to run this by here. Would the body of the second if statement ever be hit? Doesn't seem like it would. Is this a bug?
            if isinstance(f, Group):
                assert cursor.val >= header.value['blockLength']
                if cursor.val < header.value['blockLength']:
                    format_str_parts.append(str(header.value['blockLength'] - cursor.val) + 'x')

Link to line of the code

In 0.2.8 it was:
         if isinstance(f, Group):
                assert cursor.val <= header.value['blockLength']
                if cursor.val < header.value['blockLength']:
                    format_str_parts.append(str(header.value['blockLength'] - cursor.val) + 'x')

There is open PR about this https://github.com/kizzx2/sbe-python/pull/5/files

looks good in the PR https://github.com/kizzx2/sbe-python/pull/5
closing

looks good in the PR https://github.com/kizzx2/sbe-python/pull/5
closing


Add fsck to dracut modules

This is still not working (have to figure out why..)
#1323
Missing local validation (doesn't work still) and integration tests
Signed-off-by: Ettore Di Giacinto<EMAIL_ADDRESS>

mmm seems the tpm.mount in tumbleweed is in /usr/lib/systemd/system/tmp.mount as opposed to leap where is in /usr/share/systemd/tmp.mount

who did forgot to bump initrd? me!

ok, tested locally and seems to work just fine, I've enabled it now by default on the default bootargs.cfg that we ship. Will try now to add a test for it :crossed_fingers:

oh my, tests went green

mmm seems the tpm.mount in tumbleweed is in /usr/lib/systemd/system/tmp.mount as opposed to leap where is in /usr/share/systemd/tmp.mount

Yes this is just because Leap has not done the switch to tmpfs mount in /tmp by default yet. This is something we should check for teal now that I am thinking about it, whast is SLE for Rancher using for /tmp by default?


zigbee events

Hi
I've not managed to get any device activity reported by the node on any of the 15 echos on my account.
before I spend a lot of time trying to work out why this is happening (or not) given that @Apollon77 updated his library to match the new API, are zigbee events reported by the activity node?  e.g. can you extract the temperature and humidity reported by zigbee devices paired with echos?
thanks

What are "zigbee events"? Activity means "VOICE activity"! More is not possible and was never. Data from smart home devices is a different story and APIs

thanks Ingo - I had not appreciated that distinction from the documentation and, as I mentioned, I've had no activity reported from any of my echos so didn't have data to make the distinction between voice activity vs other activity.
zigbee events would be, for example, received temperature and humidity from a zigbee device paired to an echo device acting as a controller.   Ultimately I will use matter.js when you release all the templates that are in dev.

What you see as "zigbee events" are smart home devcie data" and as said these are no activities and never were.
Did you have enabled inthe node that activity is automatically requested?

yes, for sure have auto-query enabled.  I've (just) tried adding a user-agent.


Allow Background Customization in Sheet Overlay Component

Prerequisites

[X] This feature already exists in shadcn/ui - if not, it won't be considered here so don't continue with your issue.

Describe the feature
Current Behavior
Currently, the Sheet Overlay component has a hardcoded background style (bg-black/80) that cannot be effectively overridden using the class prop due to CSS specificity.
Proposed Solution
Remove the default background style from the base component, allowing users to:

Either specify their own background color through the class prop
Or use a new optional background prop to toggle the default background

Example Usage
svelte

<Sheet.Overlay class="bg-blue-500/50" />

<Sheet.Overlay class="bg-transparent" />
Alternative Solution
Alternatively, add a new boolean prop:
svelte

<Sheet.Overlay background={true} />

<Sheet.Overlay background={false} />
Benefits

More flexible component that can be used in different contexts
Better customization options for users
Maintains backward compatibility if implemented as a new prop

Additional Context
This change would be particularly useful for cases where the overlay needs to be transparent or have a different styling than the default black semi-transparent background.
sheet-overlay.svelte
<script lang="ts">
	let {
		ref = $bindable(null),
		class: className,
		background = true, // Add new prop
		...restProps
	}: SheetPrimitive.OverlayProps = $props();
</script>

<SheetPrimitive.Overlay
	bind:ref
	class={cn(
		'fixed inset-0 z-50 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0',
		background && 'bg-black/80', // Conditionally apply background
		className
	)}
	{...restProps}
/>

or remove the bg-black/80 by default from component

Where does this feature exist in the original?
You can feel free to update the component code yourself to expose whatever props make your life easier, that's the whole point of the project 😃

Where does this feature exist in the original?
You can feel free to update the component code yourself to expose whatever props make your life easier, that's the whole point of the project 😃

Thanks for the thoughtful response! It's truly inspiring to see how approachable and flexible your work is, encouraging developers to tailor components to their unique needs. This mindset really showcases your commitment to creating tools that empower the community.
By the way, I just wanted to take a moment to express my admiration for everything you’ve built so far—it’s clear you have a gift for breaking down complex concepts into accessible and impactful solutions. That said, I (and I'm sure many others) really miss your YouTube videos. It’s been about four months since your last upload, and your content was such a great resource for learning and inspiration. Your way of teaching and sharing ideas had a way of making development feel exciting and accessible, which is something not everyone can achieve.
If there’s any chance you’ve considered returning to YouTube, I’d love to see you dive back in—whether it’s sharing updates on your projects, teaching new techniques, or just discussing your approach to development.  Whatever you choose to do, thank you for everything you've already contributed. It’s clear you’re making a difference in the dev world, and we appreciate you!

Thanks for the kind words @mieaw ❤
I'd like to create some more videos/livestreams in the coming year! I've just been much more interested in writing code/maintaining these projects than creating content recently, but perhaps that will change! 😃
Going to close this issue now!

Okay am happy now in both statements you do the right *


Mouse control for translation improvements

We should try to maintain a 1-to-1 correspondence with the mouse's translation and the point on the object/scene that the user clicked on. This is more intuitive for the Node translation mechanism, but the camera translation should probably be similar-ish to how it works now.

Can you elaborate? I'm not sure I understand the first sentence :-/

There are some more general issues about making sure that the controls match the typical behavior of 3D editing programs. Such as rotation axes based upon where the user clicked.

Done and improved with Vlado's recent PRs.


Fix hex parse

Implements splitting of strings when needed after two hex digits in hex escape sequences.

I tried to fix this, but the linux test fails. In particular, the new examples/StringsAndBytes.act fails. I cannot see the exact reasons for failure in the Details, so I need help.

Interesting; not quite sure why it fails and yeah, there's no printout of the error itself. I tried to add that but don't understand how it's supposed to work with this Tasty library. For successful tests we want to suppress output; only when a test fails do we want to print some stuff but I don't see how.
Anyway, I checked out the branch on my machine and after rebasing on main it runs just fine. I (force) pushed the rebased version... let's see what CI thinks of that.

@sydow in #673 I implemented so that the test suite now prints output on test failures. I have rebased this branch so we can now see the failure, see https://github.com/actonlang/acton/runs/6830353409?check_suite_focus=true here included for convenience:
 ERROR: actonc return code (ExitFailure 1) not as expected (ExitSuccess)
STDOUT:
STDERR:
actonc: /__w/acton/acton/examples/StringsAndBytes.act: hGetContents: invalid argument (invalid byte sequence)

I don't get it at all. Could this depend on things like locale which might be different in CI than from my local Linux machine?

I have now reduced examples/StringsAndBytes.act so that it passes also the Linux tests. The hex escape fix that was the aim of this whole push request remains but all tests of en- and decoding, switching betwwen non-trivial unicode  strings and their bytes representations are removed. I will now merge anyhow, and think of how to reintroduce more tests next week...


Terraform-Compliance

This brings more in-depth testing to the infra catalog. This particular type of testing is new to me, but it seems well suited for certain types of testing and hardening.

The process takes a state file or a plan file, and then compares the configuration within that file with some expected values.
An additional benefit, now that we are running a full terraform plan on each sample is that certain errors that did not present previously will present now in CI/CD. (Reduces our reliance on manual testing.)
Currently, the pipeline will print compliance warnings but will not fail on them. This gives the developer and/or code reviewer a helpful resource, while not blocking promotions unnecessarily.

In what I've added here for POC, I'm not actually running terraform apply but I'm getting output from terraform plan which can then be tested against a set of rules.
(Example rules which this can support: buckets should not be public, security groups shouldn't allow ssh from public internet, etc.)
Blockers:

[x] Some module samples which require secrets will fail the plan step when they do not have configurations files creates.

[x] Add *.template files for all required secrets files in the samples folder, so that we can then use those templates as dummy/test files when running plan.
[x] In CI/CD, auto-rename (or copy) all "*.template" files in order to remove the ".template" suffix. The values would not be actual secrets but since we aren't deploying, that part shouldn't matter.

[x] To be more user-friendly, secrets parsing should be updated to automatically use the secret name as the key name in the file, unless the the secret location string is delimited by ":", in which case the key name in the file would be the key within the file.

Partially resolves #23

/docs


x/sys/unix: Prctl should return (int, error)

What version of Go are you using (go version)?

go1.13.4

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Linux on amd64
According to http://man7.org/linux/man-pages/man2/prctl.2.html the signature of prctl is
int prctl(int option, unsigned long arg2, unsigned long arg3, unsigned long arg4, unsigned long arg5);

unix.Prctl on the other hand is not returning the int. For some options that return value is relevant, for example PR_GET_NO_NEW_PRIVS and others (search for as the function result in the man page).
Could unix.Prctl be changed to return (int, error)?

I'm not sure we should touch the existing Prctl, as that would break existing programs.  But we should probably add a variant that returns (int, error).


FATAL error System.ArgumentNullException: Value cannot be null

2017-08-31 16:49:12|ArchiSteamFarm-12240|INFO|ASF|InitASF() ASF V<IP_ADDRESS> (abf2b357-0fb8-4a42-8881-8bde385d55a7)
2017-08-31 16:49:13|ArchiSteamFarm-12240|INFO|ASF|CheckForUpdate() ASF will automatically check for new versions every 1 day.
2017-08-31 16:49:13|ArchiSteamFarm-12240|INFO|ASF|CheckForUpdate() Checking for new version...
2017-08-31 16:49:14|ArchiSteamFarm-12240|INFO|ASF|CheckForUpdate() Local version: <IP_ADDRESS> | Remote version: <IP_ADDRESS>
2017-08-31 16:49:14|ArchiSteamFarm-12240|INFO|main|Start() Starting...
2017-08-31 16:49:14|ArchiSteamFarm-12240|INFO|main|Connect() Connecting...
2017-08-31 16:49:14|ArchiSteamFarm-12240|INFO|shadow|Start() Starting...
2017-08-31 16:49:15|ArchiSteamFarm-12240|INFO|main|OnConnected() Connected to Steam!
2017-08-31 16:49:15|ArchiSteamFarm-12240|INFO|main|OnConnected() Logging in...
2017-08-31 16:49:15|ArchiSteamFarm-12240|FATAL|ASF|OnUnhandledException() System.ArgumentNullException: Value cannot be null.
Parameter name: method
   at System.Linq.Expressions.Expression.Call(MethodInfo method, Expression arg0)
   at System.Runtime.CompilerServices.CallSite`1.CreateCustomNoMatchDelegate(MethodInfo invoke)
   at System.Runtime.CompilerServices.CallSite`1.MakeUpdateDelegate()
   at System.Runtime.CompilerServices.CallSite`1.GetUpdateDelegate(T& addr)
   at System.Runtime.CompilerServices.CallSite`1.Create(CallSiteBinder binder)
   at ArchiSteamFarm.ArchiWebHandler.<>c__DisplayClass40_0.<GetServerTime>b__0()
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at ArchiSteamFarm.ArchiWebHandler.<GetServerTime>d__40.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at ArchiSteamFarm.MobileAuthenticator.<GetSteamTime>d__22.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at ArchiSteamFarm.MobileAuthenticator.<GenerateToken>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at ArchiSteamFarm.Bot.<OnConnected>d__136.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
2017-08-31 16:49:15|ArchiSteamFarm-12240|ERROR|ASF|Exit() Exiting with nonzero error code!
2017-08-31 16:49:15|ArchiSteamFarm-12240|INFO|shadow|Stop() Stopping...
2017-08-31 16:49:15|ArchiSteamFarm-12240|INFO|main|Stop() Stopping...
2017-08-31 16:49:15|ArchiSteamFarm-12240|INFO|main|OnDisconnected() Disconnected from Steam!

Error appeared right after Windows Update procedure, so I think it's connected to this patch:
2017-08 Cumulative Update for Windows 10 Version 1703 for x64-based Systems (KB4034674)

It happen to be a duplicate of issue 634


Allow controllers to be either callable or constructible

(Branched off of this comment of mine)
TL;DR: Controllers should be anything executable: arrow functions, object methods, classes, etc.
If you have any ideas/opinions, good or bad, please share them! :smile:
Edit: Pulled the second half out to #995.

This would make both arrow functions and ES6 classes valid controllers. The problem at hand is that controllers, due to internal design, can only be ES5 functions, since they are both called and constructed by Mithril. If you have ES6 support enabled in your browser, you'll notice that in the case of child components, they're called, not constructed. This is inconsistent with the documentation, which says class constructors should be valid controllers.
This would require wrapping all controller constructions into this function below:
function init(controller, args) {
  if ("prototype" in f) {
    return new (f.bind.apply(f, [undefined].concat(args)))()
  } else {
    var inst = {}
    var ret = f.apply(inst, args)
    return ret !== undefined ? ret : inst
  }
}

@barneycarroll suggested creating a new branch for developing this and #995 further and see where that takes us. Also, feel free to discuss any ideas you have about this here. :smile:

/cc @lhorie @tivac @barneycarroll @pygy @ArthurClemens @StephanHoyer @mindeavor @ciscoheat

My 2 cents: 1) and 2) seem totally unrelated. Is there a reason why you're putting both together?

for mithril-objectify users, it's not possible to convert m(component, arg1, arg2) into something meaningful

With regards to the breaking change in 2), I don't actually mind that it is a breaking change, but I'm thinking it might be a good idea to log a warning for at least a version if the diff engine encounters that usage pattern.

@lhorie Ok.  You have a good point, and I almost did that myself beforehand.

@lhorie #995

I want to see benchmarks, we've only seen a single issue on this so far.

@tivac I doubt it would cause much of a perf hit after the initial load or a full route change in practice, because controllers aren't frequently instantiated. This issue is more about ergonomics, and should hardly affect performance in practice (unless you're frequently making lots of top-level vdom modifications, like in route changes).

This seems like such an edge-case thing, especially since it can only be hit if you're running raw ES2015. Doesn't really seem worth a perf hit to me, which is why I'd like to see the work put in to justify that it isn't a big deal.

@tivac FWIW, Mithril already doesn't consistently call vs construct functions. See this pen for classes not working and this pen for callables working in the first level down, but this pen for classes working and this pen for callables not working at the top level. Note that your browsers need ES6 support to run these.
That's my other concern. My idea is to allow both, but even picking one is better than nothing. And performance-wise, it's probably going to spend more time actually running the controller function/class in practice than trying to construct it with that init helper. This would be almost instant 90% of the time, anyways, and because this isn't a frequent occurrence (on a given component, view might be called 1000 times a second, while controller might be called once every 5 seconds most of the time), I'm not worried after the first render of a route.
Another thing is that bind is unavoidable for ES6 constructors unless you're willing to use spread syntax.

And if you're just writing a Chrome or Firefox app, or if you plan on using Mithril 2 years down the road when ES6 support is ubiquitous, this is relevant.

my 0.2 cent:
I think controllers should be simple functions to prepare data. A function without a prototype should be enough, in the end you only need a simple object for the view, not a throw away instance of a function.
Anyway the current behavior is very confusing while using a recent browser and mixed syntax (or simply ES6/2014).
The arrow functions used as controllers should work.
So I think the nice this convenience could be removed in favour of the more verbose return object pattern. It would not create more issues in the future and would be far more simple to reason about it.
const component = {
  controller: () => ({
    data: 'Hello World',
  }),
  view(ctl) {
    return m('div', ctl.data);
  }
}

const component = {
  controller() {
    return {
      data: 'Hello World',
    };
  },
  view(ctl) {
    return m('div', ctl.data);
  }
}

To be perfectly honest, I'm leaning towards the purely-callable controllers myself (still having a this). It's just as easy to implement, and it still allows arrow functions and ES6 methods.
The one concern I have is that TypeScript users might have a few issues migrating, but it should be straightforward to solve normally.
This was in part to invoke discussion on the topic, because I've been seeking an answer to some degree for a few months, now. The current functionality has been broken for that long. And since there are class die-hards as well as callable die-hards (@barneycarroll :wink:), I threw this out to generate some discussion on this.
Now, always returning an object would be impractical, because too many people rely on this. It might make sense if you use arrow functions, but not if you use this.

Not sure if it makes anyone more comfortable, but we could forcibly apply constructor return logic as follows:
function init(f, args) {
  if ("prototype" in f) {
    return new (f.bind.apply(f, [undefined].concat(args)))()
  } else {
    var inst = {}
    var ret = f.apply(inst, args)
    return ret instanceof Object ? ret : inst // [1]
  }
}

I'm at work at the moment (where jsbin is blocked by the corporate proxy), but I'm fairly sure this would lead to universally predictable outcomes for controller outputs regardless of function subtype.

@barneycarroll Not quite. Try typeof ret === "object" && ret !== null instead of ret instanceof Object. (Mine isn't correct, either. I misremembered.)

I keep on forgetting instanceof isn't fit for purpose. But. Javascript can produce entities that pass typeof ret === "object" && ret === null?!

@barneycarroll yes.
> typeof null
> "object"

@barneycarroll Try new function () { return function f() {} }. That actually returns the inner f, since according to spec, it's just an object with a [[Call]] method.

That's the other difference.

@tivac Actually, the perf difference for that init function will be significant, but can be alleviated for most cases with this helper (which tries to avoid the otherwise expensive bind call):
function constructWithArgs(C, args) {
    // Try to fast-path with a direct call first.
    switch (args.length) {
    case 0: return new C()
    case 1: return new C(args[0])
    case 2: return new C(args[0], args[1])
    case 3: return new C(args[0], args[1], args[2])
    case 4: return new C(args[0], args[1], args[2], args[3])
    default: return new (C.bind.apply(C, [undefined].concat(args)))()
    }
}

since controllers are gone in rewrite, I'm closing this


Only one Gateway with wildcard certificate works in the browser

Hello,
There is HTTP ERROR 404 codes when opening in the browser any host with wildcard certificate after accessing some with the same certificate.
When using browser incognito mode or curl the issue is gone.
Steps to reproduce the bug
Create Gateways with some VSs and try to access them one by one in the same browser.
---
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: testgw1
  namespace: istio-ingressgateway-int
spec:
  selector:
    istio: ingressgateway
  servers:
  - port:
      number: 443
      name: https
      protocol: HTTPS
    tls:
      mode: SIMPLE
      credentialName: wildcard-letsencrypt
    hosts:
    - "test1.domain.zone"
---
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: testgw2
  namespace: istio-ingressgateway-int
spec:
  selector:
    istio: ingressgateway
  servers:
  - port:
      number: 443
      name: https
      protocol: HTTPS
    tls:
      mode: SIMPLE
      credentialName: wildcard-letsencrypt
    hosts:
    - "test2.domain.zone"

Log entries from ingressgateway:
[2021-05-03T12:05:15.832Z] "GET / HTTP/2" 200 - via_upstream - "-" 0 2 67 66 "XXX.YYY.240.246" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.93 Safari/537.36" "de426a9d-8ed4-4aee-b694-ca086f860812" "test1.domain.zone" "XXX.YYY.188.237:8080" outbound|80||test1.default.svc.cluster.local XXX.YYY.72.67:44042 XXX.YYY.72.67:8443 XXX.YYY.240.246:19241 test1.domain.zone -

[2021-05-03T12:06:43.078Z] "GET / HTTP/2" 404 NR route_not_found - "-" 0 0 0 - "XXX.YYY.240.246" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.93 Safari/537.36" "9e0d5299-de10-4e39-bc87-93f2195be03b" "test2.domain.zone" "-" - - XXX.YYY.72.67:8443 XXX.YYY.240.246:19241 test1.domain.zone -

Version
client version: 1.9.2
control plane version: 1.9.2
data plane version: 1.9.2
How was Istio installed?
Using istioctl install.
Environment where the bug was observed (cloud vendor, OS, etc)
AWS EKS.
Please advise.
Thanks!
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
[ ] Upgrade

See https://istio.io/latest/docs/ops/common-problems/network-issues/#404-errors-occur-when-multiple-gateways-configured-with-same-tls-certificate


Added mupdf-tools desc

Added git-fixup too :smile:

Deleted adobe-air-sdk and fishfish

You're welcome :+1:

Merged. Thanks!

One note for future pull requests: It's easier for me to squash into one commit if you keep it as one pull request and just keep adding to it, rather than making multiple branches. Not a huge deal, but FYI.

I wil make this way in future commits, sure, sorry.

Not a huge deal at all. Just something I noticed when working with these two pull requests versus the last.
More importantly though: Thank you for all your recent contributions to brew-desc.


Problem in filtering library by discipline

As you can see in the screenshot below, not all cards with Spiritus appears if I apply the filter.
I didn't tested if other disciplines have the same problem.
I can help with fixing the problem if needed

Some filters were broken after last update (especially when using multi-filtering with [+] button), I'll try to fix in in the next few days.
If I'll feel I need help I'll let you know, thanks!

Fixed in 78d5a217e27e95206cac79c43f53080d27fd8588 (not yet at https://vdb.im, will be there in the next update).


Relational Sorting

Is relational sorting implemented ? if so, is there documentation.
http://localhost/api/v3/listing?sort=client.id

http://jsonapi.org/format/#fetching-sorting
"It is recommended that dot-separated (U+002E FULL-STOP, ".") sort fields be used to request sorting based upon relationship attributes. For example, a sort field of author.name could be used to request that the primary data be sorted based upon the name attribute of the author relationship."

I believe the answer is no
/**
 * @return \NilPortugues\Api\JsonApi\Http\Request\Parameters\Sorting
 */
public function getSort()
{
    $sort = $this->getQueryParam('sort');
    $sorting = new Sorting();

    if (!empty($sort) && is_string($sort)) {
        $members = \explode(',', $sort);
        if (!empty($members)) {
            foreach ($members as $field) {
                $key = ltrim($field, '-');
                $sorting->addField($key, ('-' === $field[0]) ? 'descending' : 'ascending');
            }
        }
    }

    return $sorting;
}

@apburton84 I believe you're right.

@nilportugues Any progress on this ?

Because this is not really a bug but an optional part of the implementation, changed from bug to enhancement.
Still no progress. Got the idea on how to resolve it, but haven't code it.


I get a script error when using the attack button on Weapons.  DnD 5e character sheet

I expect I'm missing something, as I don't see anyone else with this issue.  However, I have inserted my weapon information as:
warhammer      STR        0       6        1d8         'check mark'     4      bludgeoning      -------    20
and when I try to use the attack macro button I recieve a syntax error:
SyntaxError: Expected "(", ".", "[", "abs(", "ceil(", "d", "floor(", "round(", "t", "{", [ |\t], [+|-] or [0-9] but end of input found.
Is there something I'm missing, or something I should be doing before I click the attack button.  Granted, I am just in the initial processes of setting things up.  I have a clean map, I created the character, added a token and associated it, and tested it.  No additional tokens, no background, nothing yet.
Any help would be appreciated.
Thanks.
Dave

Your crit damage field must be the same as your damage dice field.

As mark mentioned, you need to have something filled in on the crit damage field.  Previously this was auto calculated (and not displayed) to match exactly what you had in the damage dice field but was split out to allow for features like extra damage on a crit and so on that is found in racial/class features or feats.
If you dont have anything that modifies your critical damage then the crit damage field should exactly match the damage dice section.
Hope this helps
Actoba


Error while deploying video-chatgpt locally

I followed the instruction of how to set up the model inference locally, but I was given the error:

Can you help me? Thanks.

I solved it. Sorry to disturb you.


One-page "quick start"

Add a quick start to the docs, for new users who keep asking "how do I run a game?" in the forums or Discord.

move programs into the right place (as image or loose files)
start the emulator
load and run a program

Also, discuss how to organize different programs into directories in your emulator folder, using scripts to launch games from the command line.

Would you like this for Windows or also for Linux and Mac users?

This was mostly a reminder for me to work on it, since there are some new features in the dev fork that are not published in an official release yet (specifically the -fsroot switch and the new file handling.)
However, if you want to take it on, then that would be great.
The basic process would be the same for all platforms:

Install the emulator into some place appropriate: I use /home/x16/emulator on Linux and C:\cx16\emulator on Windows.
Create a directory for the files (/home/x16/fsroot or C:\cx16\fsroot)
Create a shell script that starts the program with something like /home/x16/emulator/x16emu -fsroot /home/x16/fsroot -scale 2 -echo -rtc
The art of unzipping and organizing files into directories into /home/x16/fsroot
Start the emulator, DOS"CD:/FOLDER", LOAD "GAME", RUN

And I'd write this separately for each platform: Windows, MacOS, and Linux. It might also be useful to write a fourth version for Steam Deck, if it different significantly from the general Linux case.  (I have a Steam Deck, so I can validate the Linux steps and add the SteamDeck specific stuff.)

I've created  a proposal for the quickstart guide for Linux, see PR #25
Eventually the README itself has to link to the quickstart guides I guess, otherwise they're kinda hard to find?


Repair-DbaDbOrphanUser Contained Database Message

Looks like we are getting a message that if a database is Contained then it can't have orphaned users.  But if the user was mapped as a normal user to a login at the server level, then it can definitely be orphaned.  Not all users in a Contained database have to be contained, it just gives them the option to be so.
WARNING: [10:24:16][Repair-DbaDbOrphanUser] Database '[blah]' is a contained database. Contained databases can't have orphaned users. Skipping validation.
To reproduce, create a login at the instance level using SSMS and go into User Mapping and add it as a user in a database.  Then go delete the login at the instance level.  You can see that the user still exists in the database and is now orphaned.  If you then go recreate the login but don't map it to the user, then the user in the database is still orphaned.  You can then run the cleanup to remap them.
My use case is that I was running the Copy-DbaLogin and then Copy-DbaDatabase and needed to cleanup any orphaned SQL Native logins that may now exist.

@ClaudioESSilva any concerns if I pick this up off your plate?

No @sirsql . Go ahead.
Thanks


New IPv6 Documentation Created with IPv6 PM Team

Describe the summary, scope, and intent of this PR:
Working to get Azure Networking in the Azure Architecture Center per Working Group feedback.
This is to showcase and evangelize the IPv6 Designs that can be done in Architecture format
Help from IPv6 PM team from AzNet to help reviewed and provided technical feedback

Insert links(s) to any related work item(s) or supporting detail:
Contact Sherri Babylon or the IPv6 PM team for more info.

AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Cloud Architecture Content Team (CACT). Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors

Topic
Guidance

Draft PR
If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation.

ms.date metadata
Don't update an article's "ms.date" metadata property unless you've done a full freshness review of the content. A full freshness review includes changes required to correct or improve the full technical accuracy of the article.Don't update "ms.date" if you're doing targeted changes to improve non-technical aspects of the article, such as editorial quality, art improvements, article template alignment, etc.If you've changed any "ms.date" properties for work that wasn't part of full review for freshness, please reset them to their previous value.

Placement and linking
If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s).

PR build
After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved.

Publishing
Following a successful merge, most repos publish to the live site at least once per (business) day, usually around 10am Pacific.

Additional resources
Learn.Microsoft.Com contributor guide

Additional guidance for private repos and internal contributors

Topic
Guidance

PR size
If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/Contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps.

PR title prefix
Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:[Quality Check] - maintenance work related to content quality (edit passes, art improvements, template alignment)[LinkFix] - recurring/adhoc PRs to correct link URLs[Pipeline] - new/updated contributor success pipeline content[WIP] - a work-in-progress draft requiring several days/weeks

PR preview
Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review.

PR sign-off (public repo)
If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off.

PR sign-off (private repo)
After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:If your PR is in draft mode, remove "[WIP]" from the title and select Ready for review button at the bottom of the PR.Enter "#sign-off" in a new comment. This comment indicates that you're confident the work meets or exceeds Microsoft's standards for publication, and will trigger the review process.Your PR may be selected for initial review by the CACT. Following CACT review, you may receive questions or requests for additional changes. You should have initial feedback from CACT review within a few business days. If you have an urgent request or need to contact the team, please mention @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers in your PR and someone will get back to you. After CACT review is complete, a CACT #sign-off will be added.Final review/merge is done by the PR review team. The PR team may also respond with feedback, categorized as "Blocking" (requires action from you), or "Non-blocking" (to be addressed in a future PR).

Additional resources
Learn.Microsoft.Com internal contributor guideAuthoring templates: architecture-center-pr, well-architected-prTo contact the CACT use e-mail, or @mention our GitHub team in your PR comments using: @MicrosoftDocs/cloud-architecture-content-team-pr-reviewers

Thank you @WernerRall147 for your content. Can you please reopen this in architecture-center-pr. This pubic repo is not for FTEs to contribute with.  Please reach out to me if you have any access questions/challenges.
#please-close

Continued @ MicrosoftDocs/architecture-center-pr#10176
Thanks @WernerRall147!


gridtracker: init at 1.22.1226

Description of changes
GridTracker is a popular amateur radio application. Requested in #201213.

GridTracker listens to traffic from WSJT-X/JTDX, displays it on a map, and has a sophisticated alerting and filtering system for finding and working interesting stations. It also will upload QSO records to multiple logging frameworks including Logbook of the World.

Paging @Marc12O for testing. I've been using this for a little while in my own config and it works great.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thank you!
I will try, if I am able to
Marc

@lasandell @numinit
Gridtracker is popular as a companion to WSJT-X, would you be able to assist with review?

Thanks for the PR, I've been wanting to try GridTracker for a while! I've added my review and suggested changes. However, there is one remaining issue that may be specific to NixOS or my own setup, or it could be a bug in GridTracker itself, which seems less likely. When I close the application via the X button in the GNOME titlebar rather than CTRL-C in the console, the chromium/nw.js process continues to run and consumes an entire CPU core (its name shows as exe in top). I'm curious if anyone else can reproduce this behavior. I only found this one old GitHub issue pertaining to this issue that may or may not be relevant: https://github.com/nwjs/nw.js/issues/6549. If something like that solves the issue we could possibly include a patch for it.

Nevermind about my last comment. On further investigation, this behavior is specific to my config. It has something to do with nw.js picking up my chromium policies. When I disable chromium policies, the issue goes away.
I would still like to see my other suggested changes, since I think they are an improvement.

@lasandell Great suggestions, thanks for your review :heart:

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/793


XML containing non-printable ASCII characters parsing issue

If there are some non-printable ASCII characters inside a CDATA section in a XML document, the document object will miss some content.
Suppose I get this from the server.
<blog>
<title><![CDATA[Example]]></title>
<commentCount>0</commentCount>
<body>
<![CDATA[
	XXX
	This  line  contains non-printable ascii characters.
	XXX
]]>
</body>
<author><![CDATA[Author]]></author>
<pubDate>2015-07-21 20:24:36</pubDate>
</blog>

The response ONOXMLDocument object will be
<blog>
<title><![CDATA[Example]]></title>
<commentCount>0</commentCount>
<body>
 line  contains</body></blog>

This case happens only when non-printable characters are inside a CDATA section.

use like [[[aElement childrenWithTag:@"text"] firstObject] stringValue]   can fix this bug

Thanks for the suggested fix, @zhuhejin!
@aetern: Please let me know if you are still having an issue here, and I'd be happy to take another look.


Compatibility with numpy 2.0 issue (removal of np.unicode_)

Not sure if this should be raised here...
File [~/miniconda3/envs/solar/lib/python3.11/site-packages/eispac/core/read_cube.py:144](http://localhost:8889/lab/tree/~/miniconda3/envs/solar/lib/python3.11/site-packages/eispac/core/read_cube.py#line=143), in read_cube(filename, window, apply_radcal, radcal, abs_errs, count_offset, debug)
    142 val = np.array(f_head['index[/](http://localhost:8889/)'+key])
    143 if type(val[0]) == np.bytes_:
--> 144     val = val.astype(np.unicode_) # convert bytes to unicode
    145 if val.size == 1:
    146     val = val[0]

File [~/miniconda3/envs/solar/lib/python3.11/site-packages/numpy/__init__.py:397](http://localhost:8889/lab/tree/~/miniconda3/envs/solar/lib/python3.11/site-packages/numpy/__init__.py#line=396), in __getattr__(attr)
    394     raise AttributeError(__former_attrs__[attr])
    396 if attr in __expired_attributes__:
--> 397     raise AttributeError(
    398         f"`np.{attr}` was removed in the NumPy 2.0 release. "
    399         f"{__expired_attributes__[attr]}"
    400     )
    402 if attr == "chararray":
    403     warnings.warn(
    404         "`np.chararray` is deprecated and will be removed from "
    405         "the main namespace in the future. Use an array with a string "
    406         "or bytes dtype instead.", DeprecationWarning, stacklevel=2)

AttributeError: `np.unicode_` was removed in the NumPy 2.0 release. Use `np.str_` instead.

Thanks for the heads-up, Andy! I have fixed this issue here on GitHub and will push an updated package to PyPI sometime soon.