nwjs: 0.23.6 -> 0.32.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/nwjs/versions.
meta.description for nwjs is: '"An app runtime based on Chromium and node.js"'.

Checks done (click to expand)

built on NixOS
/nix/store/kl2c3iafzywk3xkf3bmy5rgzy3wk9l14-nwjs-0.32.0/bin/nw passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.32.0 with grep in /nix/store/kl2c3iafzywk3xkf3bmy5rgzy3wk9l14-nwjs-0.32.0
directory tree listing: https://gist.github.com/5bab5c5fb40415b2241839ab062e191c
du listing: https://gist.github.com/86ca81bebd03935bc2d858b48bbcfe46

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 9d5e9f18a2c985988f8650571b46f0f98f49aa04 && nix-shell --pure -I nixpkgs=. -p nwjs

cc @offlinehacker for testing.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: nwjs
Partial log (click to expand)

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

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

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: nwjs
Partial log (click to expand)

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

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

Success on x86_64-linux (full log)
Attempted: nwjs
Partial log (click to expand)
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/ms.pak.info
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/bg.pak.info
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/id.pak
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/fa.pak
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/th.pak.info
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/mr.pak
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/es.pak
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/locales/it.pak
warning: working around a Linux kernel bug by creating a hole of 8192 bytes in ‘/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0/share/nwjs/nw’
/nix/store/pl2papy6zix13q7y217khv2s8xnbl3cn-nwjs-0.32.0

Closed in favour of #44676


youtube-dl: 2016.09.27 -> 2016.10.07

Motivation for this change
Updated youtube-dl.
Things done

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

[ ] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Is monthly not fast enough? I could do this more frequently if needed?

@goetzst sorry, was not aware you were maintaining this package. youtube-dl releases come out very frequently, often to adapt to changes in the websites that break downloading, so I think updates should happen more frequently than monthly.
I can try and help by checking for new releases, if that helps.

e.g. https://github.com/rg3/youtube-dl/issues/10900. 2016.10.07 was released 5 days ago and already there's a fix for broken downloads from Vimeo.

not a problem. You can do that or create a pull request if you think they are necessary I don't mind :D.


report attempts and errors in NewAuthorizerFromEnvironment chain

A user (@asw101) wants to use a device code authorizer when the others in NewAuthorizerFromEnvironment fail in some circumstances.
Can we aggregate attempts and errors from each authorizer tried in the chain? And print that out when all fail? That would at least provide basic info the user could parse and make a decision based on.
Can we provide a way to append or replace authorizers? To create a custom chain? That would be another way to address.

In https://github.com/Azure/go-autorest/releases/tag/v11.5.0 the primitives used to build NewAuthorizerFromEnvironment() have been exported so you can build your own authorization chains including device code.  Take a look at EnvironmentSettings.GetAuthorizer() for details.


Update logrus import path

See gemnasium/logrus-graylog-hook/issues/23

Looks like it's a duplicate of #384, which is merged, so I suspect the change to have been reverted at some point. /me investigating...


load-students.js doesn't work for a partially-empty roster registration

Some schools don't have all positions filled for the competition, so, for example, they only have 3 people in a team or have 2 people in an individual contest.
When this happens, a couple of errors may show up:
TypeError: Cannot read property 'push' of undefined

also
TypeError: Cannot read property 'push' of undefined

The errors depend on which roster registration you use -- I'm not sure why there are two different errors, but one happens with sampleRoster.csv and the other happens with 2021data.csv.

Okay, I will get on it today

Fixed and completed issue, I'm closing


1.13 Roadmap

Please report any issues/suggestions you have with the 1.13 build in the comments.
You might have noticed that there is a ''dev' branch. We are following the development of 1.13 to be compatible with it when it gets released.
Please know that this builds might be unstable and are not meant for production use.
We'll only support the latest snapshot, when a new one comes out, previous one will get dropped, please test it on the latest version before you report issues
Current compatible snapshot: 1.13-pre7
Downloads: BuildServer
TODO:

[x] Full block mappings check:

[ ] Fix north/south/east connectings for glass_pane/stone_wall/iron_bar/fences (This was handled client-sided in 1.12, server doesn't blockupdate fences around, will be a pain.) See #856 for progress.
[x] Fix invisible banners on placement

[ ] Full item mappings check

[ ] Smooth sandstone
[ ] custom name doesn’t show (unconfirmed)

[ ] Hunt down Advancements issue (Did the packet encoding/decoding change?)  (Disconnect message)
[x] Handle AreaEffectCloud changes
[x] Handle particle changes
[ ] Handle sound changes
[x] Scoreboard prefix/suffix?

Low prio:

[ ] Server sided translations
[ ] Statistics (worth it?)
[ ] Noteblock particle (is it client-sided based of state now?)
[ ] Investigate possibility to replace new blocks with air

Tried it and when I logged a 1.13-pre1 client, console spamming (probably tile entities) as I placed signs which said "unhandled" thing, if requested I can provide the log of it (tested yesterday) and sometimes when I break a block, a wrong sound/block particle is visible/playing. eg: anvil, breaks it, hears grass sound breaking/leaves particles. Hope it helps for testing :)

The team prefix and suffix were added again in 18w20a, so this should no longer be a problem. This change is already included in the commit for 18w20b by @creeper123123321

[ ] Fix sidebar title (May be related with MC-131382)
[ ] Think what to do with new nickname coloring based on team color instead of prefix last color

Tried Pre7 client with https://ci.viaversion.com/job/ViaVersion-DEV/139/ and it seems not to work. With pre8 it does, but looks like it hangs the server while it spams the console until I disconnect the client it goes fine ^^

The diagram on the readme file would need to be update aswell, I guess you've already know it :D

1.13.1 is coming soon already
https://minecraft.net/en-us/article/minecraft-snapshot-18w30a

@creeper123123321 Agreed (and sorry for late answer) but with worldedit it is possible. Probably the 1.12.2 server will deny it anyways, like you said^^

Closing this, we'll be following the missing specifics on separate issues :)


10.1016/j-ccell-2020-04-015

Witwoski et al.
Extensive Remodeling of the Immune Microenvironment in B Cell Acute Lymphoblastic Leukemia
https://www.sciencedirect.com/science/article/pii/S1535610820302142
Data:
https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSE134759

Hello,
Id like to work on this dataset


[Bug]: Conversation between bots never stop with parameter max_turns=4 for ConversableAgent.initiate_chat

Describe the bug
Conversation between bots never stop with parameter max_turns=4 for ConversableAgent.initiate_chat()
Steps to reproduce
pip install pyautogen
jack = {
    "config_list" : [
      {
        "model": "mistral-finetuned",
        "base_url": "http://localhost:11434/v1",
        "api_key": "ollama",
      }
    ],
    "cache_seed": None
}

rose = {
    "config_list" : [
      {
        "model": "phi3-finetuned",
        "base_url": "http://localhost:11434/v1",
        "api_key": "ollama",
      }
    ],
    "cache_seed": None
}

from autogen import ConversableAgent

Jack = ConversableAgent(
    name="Jack",
    llm_config=jack,
    system_message="You are Jack, you are a teacher, you have to answer all Rose's questions"
)

Rose = ConversableAgent(
    name="Rose",
    llm_config=rose,
    system_message="You are Rose，you are a student, you ask Jack a stupid question and keep asking Jack according to Jack's answers"
)

chat_result = Rose.initiate_chat(recipient=Jack, message="Hi Jack，I opened the fuel tank of my car, but why is there no email inside? Why?", max_turns=4)

Conversation would never stop.  it looks like "max_turns=4" doesn't work.
Model Used
mistral and phi3
Expected Behavior
No response
Screenshots and logs
No response
Additional Information
No response

Solved by myself.


An example in the taskrun doc is not runnable

This commit fixes #1855 - an example in the taskrun doc is not runnable.
Two changes were required to make this example in docs/taskruns.md
runnable:

"cat README.md" needed to be updated to "cat workspace/README.md"
because the git resource is checked out to /workspace/workspace

The TaskRun needed to be moved from above the resource and task
declarations to be the last resource in the document.

As an added bonus I've updated the example to use the script field
instead of bash -c $(args).
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

I would prefer to keep unrelated changes into a different PR.

No problem, I've removed the changes to go.mod


[Feature] Dark mode for website

Hi!
I am planning on implementing a dark theme for the website. I looked over the issues and it looks like no one else is working on this.
However, I'd like to make sure. Therefore @rachelnabors what do you think?

sounds good

We haven't had any requests for a Darkmode that I know of. Is it necessary? If you were able to help us update the component docs #1579 would be a far greater contribution! But if you submit a PR for Darkmode, I'll review it.

currently, react-native document is powered by docusaurus version 1.
docusaurus version 2 has support for dark mode by default.
Instead of trying to add dark mode support in the current document, I think it would be better to move on to docusaurus 2, once all the component docs are updated, and once docusaurus 2 supports versioning (which iirc not yet supported in the alpha version of docusaurus 2)

@Naturalclar good to know! I won't proceed with this then

Closing this issue. Let's consider this Yet Another Reason to Upgrade!


Add a way to reset the text area

One issue I have continuously with playpen is that in order to test/check/experiment on it I have to

open the site
delete the text area contents filled with whatever it had open the last time
manually write up a fn main() {}

I think it would be useful to have a button that does the last two step automatically, so that you can get right to writing up the code.

Where should this button be located?


On EARTH chosing English civ usually makes the game crash

The starting settler is (in most cases) placed somewhere near the arctic and immediately after the game crashes. Getting similar problems by choosing Greeks (same color).
Tried it with other civs and no such problems.
UPDATE: The game is very unstable on EARTH. Usual crashes after only a few turns.

I can imagine why this happens, but it's not intentional. The problem should be fixes along with #114.

Well... starting positions on EARTH are still random, and somehow the EARTH map invites the game to spawn civilizations on one of the poles which causes the game to try and center over the edges of the map.
I have taken precautions, that fixes this bug. Now I just need to get the EARTH start positions correct...


Scala2 => Scala3, Jvm and Js

1 Scala3Js is showing errors even though Scala3Jvm is building.
2 Scala3Jvm type inference on empty Arr case was failing although it was succeeding on ScalaJvm2.
3 Scala3Jvm is failing but Scala2Jvm is not. Dev/src/pSpace/Planets.scala line 30 Error with multiply.
posn = Pt2.circlePtClockwise(elapsed * 0.001 / math.sqrt(auRatio.cubed)).toDist2(dist)

The project will be reorganised to support the cross building Scala2Jvm, Scala2Js, Scala3Jvm Scala3Js and ScalaNat. So to simplify this task as a temporary measure, macros will go into their own module. The examples and the core parts of modules will be combined.
The Examples and Core will be separated again and the Macros module will be removed once Jvm and Js are fully working on Scala 3.

Raised issue for Scala here

Raised new issue on [epfl bug tracker(https://github.com/lampepfl/dotty/issues/13127)


[Bug] Hands animation no longer working

Background
After the upgrade of the glTF importer and the new rendering backend, the hands animation is no longer working. This is somewhat understandable as the way hands were being animated was a big hack.
glTF animation is well defined, but for some reason when exporting the default Oculus hands from Unity, the animations are completely mangled: instead of giving us keyframes between two states (hand open and hand closed) we get two animations, each with a single keyframe. We work around this by just creating a single animation and use each state (open and closed) as a keyframe.
TODO

[ ] Should we just get better models with correct animations?

@sinoth as discussed last week; what are your thoughts?

@sinoth as discussed last week; what are your thoughts?

I don't have any quality hands tucked away. This is something I would normally throw money at, but licensing hands to include in an engine might be a headache. I'm tempted to hack the Oculus hands to work as they are nicely stylized and great for prototyping. I'll try to find a different version of them or maybe fix the issue in Blender.

Mm, that's a fair point! Maybe fixing them in Blender is a good idea; I hadn't tried that previously as my animation skills are pretty lacking!

I think the two different animations are there because they represent non-linear actions. This makes sense if you consider more movements than only open and closed (eg. point with index finger).
This video was helpful for me to understand how non-linear animations work in Blender: https://youtu.be/tAo7HxxxA08


GitHub Releases have not latest version 1.0.0

Hello!
Version 1.0.0. was published 3 weeks ago, but in Releases chapter have no information.
Thank you!

Oops, so sorry! Totally an oversight on my part 😅  I've done this now! https://github.com/rails/solid_queue/releases/tag/v1.0.0


Add missing apt pacakge locales in Dockerfile

locale-gen command seems missing since uncertain point for a while:
Step 15/33 : RUN locale-gen en_US.UTF-8
 ---> Running in fae073c89c01
 /bin/bash: locale-gen: command not found
 The command '/bin/bash -o pipefail -c locale-gen en_US.UTF-8' returned
 a non-zero code: 127

This patch should fix the Docker image build

FYI, the failed CI jobs aren't related to the changes here.


Add operator __in to es.filter() for search in list.

This can fix #50 and good update for PR #51

Yes i think it would be a good idea but does it still work with strings ? .filter(X__contains='test') used to return documents with not analyzed fields like 'foo test bar'. If not we might need another lookup, because it wouldn't be backward compatible and i don't see a reason to break this.

@lauxley , I do not quite understand you.
In #50, I raised the question of how to work with lists. I think that it is necessary to share the search for relevant lists and string.
This pull request decide whether to #50.
In order to carry out a filtering of the list of line, I'll think separately.

Adding the __in lookup seems like a good idea, i didn't say otherwise so why do you say you don't understand me ? I can't merge the pull request just yet though, i want to check how it behaves with different types and re-check what problem it solves.
And a test would be nice :)

@lauxley, my English is not so good as I want it be. That's why I really can't understand the meaning of some phrases. I understand the words separetly, but the whole phrase sometimes seems difficult to get for me. But I have my wife, who helps me sometimes in English:)


fix doc typo in bn_dh.c

while RFC 2312 refers to S/MIME it doesn't actually declare any groups,
RFC 2412 actually talks about DH extensively and the group
defined in the code below is defined on page 47 of it
see also #12058

Checklist

[x] documentation is added or updated

This pull request is ready to merge

Merged. Thanks!


Please consider using Levenshtein Distance algorithm for command and param typos

Is your feature request related to a problem? Please describe.
Consider adding typo autocorrect suggestions.
Describe the solution you'd like
Currently a typo in command results in this standard information:
scala-cli copmile .
[error]  copmile is not a scala-cli sub-command and it is not a valid path to an input file or directory.
Try viewing the relevant help to see the list of available sub-commands and options.
  scala-cli --help

Ideally scala-cli would be able to guess and suggest the correct command (also less verbose):
scala-cli copmile .
Unrecognized command or path: copmile

Did you mean ...
   compile                Compile Scala code.

For more information, run:
  scala-cli --help

Tha same can be done for parameters, e.g.:
scala-cli --halp
Unrecognized argument: --halp

Did you mean ...
   --help                 List all available options

For more information, run:
  scala-cli --help

Describe alternatives you've considered
Alternative would be to print the standard help for typos in command, instead of telling user how to display it themselves.
Not doing it is also an alternative.
Additional context
E.g. cobra golang library uses this rule:

Every registered command that matches a minimum distance of 2 (ignoring case) will be displayed as a suggestion.

Thanks for reporting! I think I was talking about this for a while actually :D


Add sample application for PubSub Channel Adapter

Maybe Storage Resource Abstraction as well.

Regarding channel adapters, my test toy app lives in https://github.com/joaoandremartins/chatroom-springintegration/blob/master/src/main/java/com/google/springongcp/pubsub/PubSubApplication.java
Maybe we could use this as a starting point for a samples repo (new or existing) and link it from http://cloud.spring.io/spring-cloud-gcp/ ?
Brain dump of some more things we might want to showcase in our samples:

GCS resource abstraction,
Manual/automatic ACK mode (toy app only uses manual now),
(A)sync message sending (not yet built),
(Non-)Boot usage (toy app uses Boot heavily),
Programmatic topic/subscription creation (not yet built).

@jabubake do you have any advice for us here? Thanks!

Regarding samples repo : I'd advise keeping it under java-docs-samples : under a spring-cloud-gcp directory or just spring.
I was going to get started on the GCS resource abstraction showcase : create a sample , then a community article and Spring guide with it.  Happy to help if you want to do it yourself, if not, just file an issue under java-docs-samples, assigned to me.


Breaks on inline CSS with brace selector

Hi,
my site echos inline css in the page head. This CSS uses braces on some selectors, like a > span.
The plugin seams to cuts away all of the CSS beginning with the first > in finds.
Greetz

Yes, I the < and > is a problem if it's inside inline style or inline js. As a workaround for now, maybe you can move the css into your css file?

Ok, good to know.
As a workaround i disabled the plugin :-). Cause i need inline CSS for performance reasons.

That workaround should work as well. 😄
Related: https://github.com/jenstornell/kirby-html-minifier/issues/13


x3d: make ImageTexture's url attribute case insensitive for now

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

@mwitchwilliams knows where this is coming from.

Planning to merge Tuesday morning.


Add header dropdown menus

Adds dropdown menus to the header per @chriscubellis's mockups.
The styles are complete, but the actual links & routes are hard-coded & completely made up. They should obviously be made configurable.

@sneakysneakysneaky All the JS looks good to me.
I would like to see the menu moved into its own namespace or placed under Gazelle.menu with a function to add/remove menu items. This is so other packages could add their own menu items.

@ajax-wcd I will start working on the updates to the menu that we discussed last week. In the meantime, this PR can probably be merged unless there is more stylesheet feedback. The stylesheets probably won't change.

Rebased. Will let this PR simmer for a day or so, then merge. I've been working on some stuff for menus, & I would like this in master before i submit that feature.

You should definitely put screenshots/gifs with your PRs. Good practice.

By your command.


Add auto restart to systemctl file

https://ma.ttias.be/auto-restart-crashed-service-systemd/

Whoops this wasn't even in the repo lol


Resolve issues with foreground tasks getting cancelled

Summary

Adds dependency androidx.lifecycle:lifecycle-service:2.7.0 since SystemForegroundService relies on it (possible this was the problematic oversight)
Removes some unused imports
Adds PythonProvider class which implements AutoCloseable for use with try-with-resources, to provider the android context to the python side, and automatically ensure we don't create context memory leaks
Updates ContextUtil to utilize the PythonProvider if it's active
Refactors work manager worker logic into abstract base class and makes foreground worker a regular Worker instead of RemoteListenableWorker
Re-adds setForegroundAsync that was mistakenly deleted, but does not call it repeatedly-- calling it repeatedly seemed to have no benefit against the issue but to spam the logs
Goes back to having a single android notification for tasks, and since foreground tasks require an integer ID, it converts the task request UUID to an integer using CRC32
Removes WorkerService which was a RemoteWorkerService for running RemoteListenableWorkers
Adds observer pattern to task worker implementation so task update calls go through it, which is a Java object that should only exist in the same task's thread
Adds onProgressUpdate hook method to worker base class which is called when the task's progress is updated, and by default calls setProgressAsync and updates the notification (this may be the true fix)
Fixes order of operation issue after switching to // division instead of using math.floor(...) changed in https://github.com/learningequality/kolibri-installer-android/issues/204
Adds android.permission.POST_NOTIFICATIONS permission and check for it since Android Studio was complaining about it missing (I don't recall this happening previously)

References
Fixes https://github.com/learningequality/kolibri-installer-android/issues/205

Goes back to having a single android notification for tasks, and since foreground tasks require an integer ID, it converts the task request UUID to an integer using CRC32

Nice!


Unable to generate documentation

Following the instructions given under heading Building the documentation in the project main page, and executing given commands in the project directory ( ${PROJECT_SOURCE_DIR} ), after the last step make doc, no documentation appears to be generated at the location specified i.e. no index.html can be found in ${PROJECT_SOURCE_DIR}/doc/doxygen/doc/html/ currently. I am using CMake ver. 3.5.1 and Doxygen ver. 1.8.11 on Ubuntu 16.04 ( 64-bit ).
Please help me generate the documentation.

Thanks for the catch. I fixed the outdated documentation with https://github.com/ethz-asl/kindr/pull/102


Downloading multiple artifacts with the same file name corrupts them.

We've recently diagnosed a situation where artifacts appeared to be getting corrupted and discovered it's when multiple are created with the same name in different steps, then downloaded to a final step.
eg.
Step 1:    Uploads artifacts:

package_amd64.deb
package_all.deb

Step 2:    Uploads artifacts:

package_arm64.deb
package_all.deb

Step 3:   Downloads all artifacts from step 1 and 2:

package_amd64.deb
package_arm64.deb
package_all.deb    <-  will be corrupted combination of the two.

This took a while to diagnose because the corrupted file is the correct size and because it's a mixing of two similar files (in our case they were supposedly identical but because some file date stamps were different, the compressed content was unusable).
The buildkite agent code that downloads artifacts from S3 appears to grab a list of all artifacts and downloads them in parallel, it looks likely that this is where the problem happens.
I realise creating multiple artifacts with the same name is not recommended, but it's a very easy mistake to make.
Is it possible to either fix this behaviour or at least make it clearer in the documentation that this is an easy to make mistake.

hey @colincoghill! this is definitely not the intended behaviour - i suspect that the two artifacts are being erroneously downloaded into the same place and splinching with each other. i'll raise an escalation internally and try to get this fixed for you.


flagstaff, gunnings

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.1
logical conflict every: 100
sleep for: 300s
close stale after: 4 hours
[pullrequest]
requests per hour: 50
deps=[f,g]

/trunk merge


Add specific token ID to URL for direct linking

Would be great to link directly to a specific token ID, so that end users on a marketplace can go directly to a page on BCD to verify the token is on the blockchain.
Ex: https://minterpop.com/object/0
See "View on Blockchain" currently links to https://better-call.dev/mainnet/KT1AaaBSo5AE6Eo8fpEN5xhCD4w3kHStafxk/tokens
The particular token in question has:

Title: The Brand
Token ID: 0

If URL could end in /{token_id} then direct links could be added.
For the above example the URL would be: https://better-call.dev/mainnet/KT1AaaBSo5AE6Eo8fpEN5xhCD4w3kHStafxk/tokens/0

If this functionality already is possible and it needs to be added to front end, I can repost on that repo

Thanks for highlighting that @clcmusic ! This is a front end task indeed, will schedule for the next release


SendGrid no longer used?

I was reading through the Wiki and got to the step about setting up SendGrid to send emails. However, I'm not seeing a place to enter the API key. Has this functionality been removed? What emails does Blogifier send?

This going into plugin, sorry, I really need to update Wiki, there lots of changes in latest compared to previous version.


Variables exported in a C# script don't update during runtime

Godot version:
3.0
OS/device including version:
Windows
Issue description:
Updating exported variables in a C# script has no effect during runtime.
Steps to reproduce:

Create a project
Attach a C# script to a Node2D
Export a Vector2 from script
Update position on every frame based on exported variable

The position is not updated because variables don't update during runtime.

Do you mean the value you set in the inspector is never applied when you run the game?

@neikeq From how I have it understood based on the steps to reproduce, the variables from a C# script don't update at all in the Godot editor. I could be wrong though, maybe I forgot to mention that the game needs to be running

@divmgl Can you (or anyone else) still reproduce this bug in Godot 3.2.1 or 3.2.2beta4?

Closing due to lack of response. Please comment if you can still reproduce this bug.
PS: If no minimal reproduction project was included in the original bug report, please upload one as well to ease troubleshooting.


Broken links

On this page: https://docs.microsoft.com/en-us/powerapps/developer/data-platform/connection-entities?source=docs
Following links are broken:

https://docs.microsoft.com/en-us/reference/entities/connection.md
https://docs.microsoft.com/en-us/reference/entities/connectionrole.md

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

ID: b4140573-407b-8fb6-684e-c4b3ffe26272
Version Independent ID: 590125d4-25f4-8344-a876-4b280b0208a2
Content: Use connections to link records to each other (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/connection-entities.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @mayadumesh
Microsoft Alias: jdaly

@markdebhailis  Thanks for pointing that out. Should be fixed by EOD today


Add random-access methods to SlidingWindows

index(_:offsetBy:), index(_:offsetBy:limitedBy:), and distance(from:to:) for SlidingWindows.
The base.endIndex..<base.endIndex representation of endIndex made this a bit tricky, because neither of the bounds is a logical successor of the index before it. The offsetForward/offsetBackward dance alleviates this somewhat by ensuring that the limit is always on the correct side of i and never equal to it, removing some of the edge cases.
Checklist

[x] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

The base.endIndex..<base.endIndex representation of endIndex made this a bit tricky

Do you think it's worth considering a better implementation of endIndex in this case?

Do you think it's worth considering a better implementation of endIndex in this case?

That's always worth considering, but I don't think there is any! If we want endIndex to fit in naturally with the other indices, its lower and upper bounds would need to be the direct successors of the bounds of the index before endIndex. But since the index before it corresponds to the last window which has an upper bound of base.endIndex, there is no successor to that (in the general case).
Of course we could give SlidingWindows.Index an internal enum representation with an endIndex case, which I'm mostly indifferent towards, but that wouldn't get rid of any edge cases. It might make it less error-prone to work with, but with the tests passing I'm pretty confident this implementation is correct as well.
Having said all that, it's totally possible to write a protocol ValidIndexCollection: Collection { ... } that lets you write things like
struct CollectionOfTwo<Element> {
  let first, second: Element
}

extension CollectionOfTwo: ValidIndexCollection {
  enum ValidIndex: Comparable {
    case first, second
  }

  func element(at index: ValidIndex) -> Element {
    index == .first ? first : second
  }

  var firstValidIndex: ValidIndex? { .first }
  var lastValidIndex: ValidIndex? { .second }

  func validIndex(after index: ValidIndex) -> ValidIndex {
    .second
  }
}

where ValidIndexCollection handles all the endIndex edge cases for you. This could be useful for implementing collections that don't have a natural way of representing endIndex. SlidingWindows and ClosedRange come to mind. This could be worth considering if we end up with other collections that need to deal with endIndex separately.

Thanks, @timvermeulen! 👏


Remove shading

Shading stuff into a library meant for consumption by other applications is, frankly, a bad idea. For example, suppose a library that both my application and this library require exists (aka a transient dependency). If it is shaded into this lib, I cannot (for example) add it as a provided/compileOnly dependency, because it is shaded into this lib and therefore my application. Thanks to the Magic of Maven™, unshading the dependencies will not break anything else, because all it's dependencies will still be specified as transient dependencies.

Hold on, I don't quite understand. Surely if Sefilib is not shaded, servers without it installed as a dependency will not be able to use DML? Why would that not be the case?


India not updating.

Looks like india is no longer updating. Would be good to know if this is data related or a run time issue. It doesn't look like it is updating the status.csv.

20-10-26 08:51:06 INFO Getting regional data
2020-10-26 08:51:07 WARN simpleWarning in gsub("~m", msg, message, fixed = TRUE): argument 'replacement' has length > 1 and only the first element will be used

looks like it is legitimately broken

hmm is the warning here a broken log of some kind?


GCC 6: Fix java support for 19.09

Motivation for this change
Backport of #73291
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

@ofborg build gcj


A11y_SQLAzureDataStudio_ConnectToASQLServer_AdvancedProperties_ScreenReader: Hidden content is read by screen reader.

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
#A11y_SQLAzureDataStudio; #A11yMAS; #A11ySev2; #A11yTCS; #SQL Azure Data Studio; #Benchmark; #DesktopApp; #VoiceOver; #MacOs; #MAS1.3.1;
Environment Details:
Application Name: Azure Data Studio
Version: 1.15.0-insider
Commit: 3a8b74a
Date: 2020-02-11T05:49:41.181Z
VS Code: 1.42.0
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.3.0
Mac OS:10.15.3
Additional Details:
Mac OS:10.15.3
MAS References:MAS1.3.1
Screen reader: VoiceOver
Repro Steps:
1.Launch Azure Data Studio.
2.Select the Connections icon (if not already selected) from left bar.
3.Hover over the 'Servers' header and select the 'New Connection' icon.
4.Navigate till Advanced button under Connection details & hit enter.
5.Launch Narrator desktop app.
6.Check if hidden content is read by screen reader.
Actual:
After hitting Advanced button present on Connection details screen "Advanced Properties" screen is opened.
On "Advanced properties" screen on pressing shift+tab hidden content is being read by Voice Over.
Expected:
Hidden content on the screen should not be read by screen reader.
User Impact:
Users might get confused as the hidden content is read which is not present visually.
Recommendations:
Refer below link which is repository of bug fixes code snippets: https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
MAS Reference:
MAS 1.3.1
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={54f28d1f-a2d1-4dcd-84e1-5c9b87e8aba4}
Attachment link reference:
https://microsoftapc-my.sharepoint.com/:v:/g/personal/v-atkulk_microsoft_com/ESw-BRoMw_hDq78EeSFwGCcBuBWTMNjy_RIlLHTj8hEYLg?e=LQrAgT

The issue described here will be fixed if we set the focus to the newly opened dialog.
another issue i noticed, I am not able to get to all the text/dropdown controls using tab navigation on the advanced settings dialog.

This was fixed in a previous PR - the issue here was the focus wasn't being set correctly on open so it was escaping the dialog.

GitHubTags:#BM-TCS-SQLADS-MAC-Feb2020;


docs: fixed the sapphire docs

Hey if you're gonna throw shade at our framework then at least do it right LMAO.
Sidenote, I had to seriously rewind my brain to Sapphire v3, which is the version compatible with DiscordJS v13. I recommend updating your framework to DiscordJS v14 asap.

Hey if you're gonna throw shade at our framework then at least do it right LMAO.
Sidenote, I had to seriously rewind my brain to Sapphire v3, which is the version compatible with DiscordJS v13. I recommend updating your framework to DiscordJS v14 asap.

Hey @favna, thanks for the concern about being outdated, and sorry for being months late on the response (I didn't get any notif from github somehow)
This project would be archived soon in favour of a successor framework, with the state still being dependent on the old discord.js version so newcomers aren't encouraged to use it until I properly release the new one, would you still like this to be merged in?

I guess not but next time throw shade properly 🤣


nixos/acme: 20.03 -> 20.09 regression with opensmtpd

Describe the bug
When upgrading from 20.03 to 20.09, opensmtpd stopped working (apparently) due to ownership changes in #91121:
Oct 22 14:40:01 hostname systemd[1]: Started opensmtpd.service.
Oct 22 14:40:03 hostname smtpd[758]: warn:  /var/lib/acme/mx.example.com/fullchain.pem: not owned by uid 0
Oct 22 14:40:03 hostname smtpd[758]: smtpd: load_pki_tree: failed to load certificate file
Oct 22 14:40:03 hostname systemd[1]: opensmtpd.service: Main process exited, code=exited, status=1/FAILURE
Oct 22 14:40:03 hostname systemd[1]: opensmtpd.service: Failed with result 'exit-code'.

As you can see, opensmtpd seems to require the certificates to be owned by root.
I tried adding chown root fullchain.pem key.pem to security.acme.certs.postRun but it seems that the ownership resets to acme when rebooting.
To Reproduce

Configure acme to generate a certificate
Configure opensmtpd to use the certificate

Expected behavior
opensmtpd should start without errors.
Notify maintainers
@m1cr0man @lheckemann @mweinelt @flokli
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.72, NixOS, 20.09.git.4d3d4221cad (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.7`

Merely a workaround but you could probably copy them to another directory after each run. This isn't pretty but would at least unblock you - if this is a blocker at all.
In the long run we might want to consider supporting this and maybe OpenSMTPD can be taugtht to not enforce the uid 0 rule. Removing the code that does that specific check seems rather trivial: https://github.com/OpenSMTPD/OpenSMTPD/blob/a6a39cb0cb6ee72d194f84d6763997df7e0c0bf3/usr.sbin/smtpd/ssl.c#L117-L121. I wonder what they threat model looked like. Is their assumption having files as another uid is less/more safe?

Merely a workaround but you could probably copy them to another directory after each run. This isn't pretty but would at least unblock you - if this is a blocker at all.

Thanks, I will try this!

I wonder what they threat model looked like. Is their assumption having files as another uid is less/more safe?

Presumably, having a file as another uid is less safe, otherwise it wouldn't make sense to enforce that rule. One reason might be that if a file has 600 or 400 permissions and is owned by root, then it can only be read by root. If it is owned by another uid, then it can be read by that uid and root. But I don't know if this is the actual reason for the rule.

Presumably, having a file as another uid is less safe, otherwise it wouldn't make sense to enforce that rule. One reason might be that if a file has 600 or 400 permissions and is owned by root, then it can only be read by root. If it is owned by another uid, then it can be read by root and that uid. But I don't know if this is the actual reason for the rule.

The reason for enforcing the acme user is twofold:

We didn't want the ACME services to run as root as there was no technical reason to do so and it posed a security risk (you have a program contacting some HTTP service and writing files to disk, it should be minimally privileged)
In order to ensure that all certs could be created/updated by the ACME services it was necessary to enforce the acme user everywhere.

Copying them to another directory would be the best option here. The real thing you're fighting with regards to adding chown to postRun is the acme-fixperms.service, but even if that worked the 750 permissions would break the renew service entirely (as the acme user wouldn't be able to write new certs). I think this is a particularly unique issue, given that they are enforcing UID 0 for certs (to what end?), and having thought it through, allowing the user to be changed would be more cumbersome than simply copying the certs in the postRun script.

It might be possible to do some adaptation in the service scripts so that all the chown commands use $USER instead of acme. That would mean you could then override all the systemd service users by hand (via `systemd.services..serviceConfig.User`) and get what you need. However I don't see this solving any issues that a copy command in postRun doesn't solve, and is also more complex to use.

I'd also propose copying them to another directory for now, outside of the acme module.
Once 247 has landed, this should probably be revisited. It adds LoadCredential, which can be used to pass around credentials to processes without them having access to the bare files themselves.
Then, files will be made readable by the process that runs the service:

The data is accessible from the unit's processes via the file system, at a read-only
location that (if possible and permitted) is backed by non-swappable memory. The data is only
accessible to the user associated with the unit, via the
User=/DynamicUser= settings (as well as the superuser). […]

This will probably be root, at least currently, as we run opensmtpd.service take care of changing to a less privileged user.
In the more long-term, it might be a good idea to go into discussions with upstream of running opensmtpd, as a less-privileged user, and provide the necessary capabilities to bind on low ports, like we do with nginx.

I can confirm that copying to another directory seems to work for me. Thanks!

I'm running into the same issue with PostgreSQL (sanitized):
Oct 26 00:58:36 myhost systemd[1]: Starting PostgreSQL Server...
Oct 26 00:58:37 myhost postgres[19852]: [19852] FATAL:  private key file "/var/lib/acme/myhost.example.com/key.pem" must be owned by the database user or root
Oct 26 00:58:37 myhost postgres[19852]: [19852] LOG:  database system is shut down
Oct 26 00:58:37 myhost systemd[1]: postgresql.service: Main process exited, code=exited, status=1/FAILURE
Oct 26 00:58:37 myhost systemd[1]: postgresql.service: Failed with result 'exit-code'.
Oct 26 00:58:37 myhost systemd[1]: Failed to start PostgreSQL Server.

We do already run PostgreSQL as the user postgres, and it's unclear if LoadCredential will work for it from the systemd docs. Looking at the implementation it seems to try to use ACLs and otherwise fallback to file system ownership. Might just have to try it once we get systemd 247.

I added some pre-start code to do the "copy certs to a new directory" and it looks like PostgreSQL also cares about the permissions:
Oct 26 01:24:14 myhost systemd[1]: Starting PostgreSQL Server...
Oct 26 01:24:14 myhost postgres[20921]: [20921] FATAL:  private key file "/run/postgresql/certs/key.pem" has group or world access
Oct 26 01:24:14 myhost postgres[20921]: [20921] DETAIL:  File must have permissions u=rw (0600) or less if owned by the database user, or permissions u=rw,g=r (0640) or less if owned by root.
Oct 26 01:24:14 myhost postgres[20921]: [20921] LOG:  database system is shut down
Oct 26 01:24:14 myhost systemd[1]: postgresql.service: Main process exited, code=exited, status=1/FAILURE

Handling it in my pre-start code but something to keep in mind when developing a better solution.

Did not verify (yet) whether this still breaks.
If this is solved we should document how it was solved.

Totally forgot about this actually. It's been so long that @flokli's comment about using LoadCredential can and should be tested as a solution, as it's something that would work to fix the mentioned Postgres issues too.
I have to fix some bugs with the postfix tests too, was planning to do it this weekend and will do this at the same time if no one is in a rush for it.

For both opensmtp and Postgresql using LoadCredential is a viable solution. Shown below is a minimal opensmtpd config which loads:
{ pkgs, config, ... }:
let
  mailDomain = "example.com";
  certDir = config.security.acme.certs."${mailDomain}".directory;
in {
  services.opensmtpd = {
    enable = true;
    setSendmail = false;
    serverConfiguration = ''
    pki ${mailDomain} cert "cert.pem"
    pki ${mailDomain} key "key.pem"
    listen on localhost tls pki ${mailDomain}
    action act1 relay host smtp://<IP_ADDRESS>:10027
    match for local action act1
    '';
  };

  systemd.services.opensmtpd.requires = ["acme-finished-${mailDomain}.target"];
  systemd.services.opensmtpd.serviceConfig.RuntimeDirectory = "opensmtpd";
  systemd.services.opensmtpd.serviceConfig.WorkingDirectory = "/var/run/opensmtpd";
  systemd.services.opensmtpd.serviceConfig.LoadCredential = [
    "cert.pem:${certDir}/cert.pem"
    "key.pem:${certDir}/key.pem"
  ];
  systemd.services.opensmtpd.preStart = ''
    ln -s $CREDENTIALS_DIRECTORY/cert.pem .
    ln -s $CREDENTIALS_DIRECTORY/key.pem .
  '';

  security.acme.certs."${mailDomain}".postRun = ''
    systemctl restart opensmtpd
  '';
}

This results in cert.pem and key.pem having the correct permissions for opensmtpd to start up, and from testing they are correctly updated when the service is restarted. The last few lines (from LoadCredential down) are almost identical for Postgresql. There may be a way to avoid the symlinks and RuntimeDirectory change if there is a way to use environment variables in the smtpd.conf, or if the $CREDENTIALS_DIRECTORY can be predicted (it seems to be /run/credentials/$servicename, but I dont know hot reliable that is).
The only catch I can see with this right now is that LoadCredential doesn't appear to support service reloads. This is a problem for the likes of Postgres which accepts a SIGHUP for a configuration reload from disk (and thus refreshing certs) without dropping connections. That said, some users may be fine with simply restarting the service for cert renewals.
This feature is very new to Systemd and I can't find much discussion on it beyond the RFE so if someone knows if reloading/refreshing the files created by LoadCredential is possible please let me know! This does not stop us adding a useACMEHost-like option to opensmtpd right now though. :)

A bit of research into whether CREDENTIALS_DIRECTORY will always be in /run/credentials/$servicename led me to this section of the systemd source, where an assumption is being made that creds_path will always be underneath /run/credentials, and thus adds a mount to ensure /run/credentials exists. I can't see exactly where it picks the subdirectory path but I imagine at this point it is similarly hardcoded.
With that, the above opensmtpd configuration can be greatly simplified:
{ pkgs, config, ... }:
let
  mailDomain = "m1cr0man.com";
  certDir = config.security.acme.certs."${mailDomain}".directory;
  credsDir = "/run/credentials/opensmtpd.service";
in {
  services.opensmtpd = {
    enable = true;
    setSendmail = false;
    serverConfiguration = ''
    pki ${mailDomain} cert "${credsDir}/cert.pem"
    pki ${mailDomain} key "${credsDir}/key.pem"
    listen on localhost tls pki ${mailDomain}
    action act1 relay host smtp://<IP_ADDRESS>:10027
    match for local action act1
    '';
  };

  systemd.services.opensmtpd.requires = ["acme-finished-${mailDomain}.target"];
  systemd.services.opensmtpd.serviceConfig.LoadCredential = [
    "cert.pem:${certDir}/cert.pem"
    "key.pem:${certDir}/key.pem"
  ];

  security.acme.certs."${mailDomain}".postRun = ''
    systemctl restart opensmtpd
  '';
}

Now we don't need to set Runtime/WorkingDirectory or add preStart scripts. This is remarkably subtle to add to existing services where specific permissions are required for the cert files.

The issue with LoadCredentials is that it won't get updates during systemd service reload.

I'll opened a new issue


Bold, Italic, Strikethrough not working on demo site

Informations
Chrome 54
Windows 8.1
1920x1024
How to reproduce the bug?

Go to the [Trumbowyg demo page] (http://alex-d.github.io/Trumbowyg/)
Put the cursor in the editor. Click bold, italic, or strikethrough.
Start typing but the text is not formatted to what you selected.

It sort of looks like the buttons aren't staying active?
You can select text and then click bold and the selected text will change.

I'm sorry but I can't fix that. This seems to be a content editable bug.


Resource Metadata Editing:  Enhance Metadata Editing with Descriptive Text

Some of the items on the resource metadata editing page are not self explanatory.  Need some tooltip text on items that describe what they are or some text next to them on the page that describes what should be entered.  Examples include:

Manage access
Related resources

Could you provide the help text to go along these functions?

For "Manage access" use the following:
Use this button to share your resource with specific HydroShare users. You can give other users the ability to view or edit this resource. You can also add additional owners who will have full permissions.
For "Related resources", you have already implemented text below the button, so we should be good there.
For the "Add source" button on the "Related Resources" tab use the following:
"If this resource was derived from another resource, click the button above to create a link to the source resource from which this resource was derived. If the source resource is a HydroShare resource, you should paste in the URL to the source resource in HydroShare. If the source resource is something outside of HydroShare, you can paste in a URL or a full text citation to the external resource."

These help text can sometimes be too long. Do you think an icon that shows the help text when hovered would be better? Something like what we have next to Select a licence dropdown.

Yes I think that could work.

Is there more descriptive text to be added?

@mauriel133 - I think what you have is fine. What did you end up doing with the info icon on the manage access button when you moved it up to the toolbar at the top?

It's gone, but the icon has a hover text that reads "Manage who has access". Should I add it there or some other place?

Would it be lame to add it as the tooltip text when you hover over the button?

Like that?

I could instead add this message to the manage access interface window that pops up.

Why not both?

@mauriel133 Is this already done? Can this issue be closed?

This has been implemented and can be closed.

This is implemented on www.hydroshare.org. Closing.


Error on this query

Error on this Query :
SHOW TABLES FROM oswa_inv-313233220b LIKE "users"

https://github.com/bitsandbots/inventory


ba-edit error: This pool is no longer valid. Cannot fetch new objects

From @wjpepli:

Upgraded packages on server:

DBI: 1.1.1  → 1.1.3
dbplyr: 2.1.1  → 2.3.2
dplyr: 1.0.7 → 1.1.1
pool: 0.1.6 → 1.0.1

after noticing this error message got shuffled since our earlier version:

Extract pool validation out into own method (#161) · rstudio/pool@6bb3d48

Also applied fixes:

dropped global assignment of con after sussing Warning: Error in pool$close: The pool was already closed. · Issue #94 · rstudio/pool
added ck_con() to top of callback functions, a la r - dbPool object expiring - Stack Overflow


Remove zoom_reset shortcut in 2D editor

It was an alias to zoom_100_percent. It's no longer necessary since we support multiple events per shortcut. zoom_100_percent is now double shortcut: 1 and Ctrl + 0.

Thanks!


abacus-usage-collector tests fail with MongoDB

abacus-usage-collector-itest times out on Travis
Unit tests fail on Linux/Mongo 2.6 and Mac/Mongo 3.2.
Logs from Mac's run:
  1) abacus-usage-collector returns error doc due to pricing plan not found:
     Uncaught AssertionError: expected [Error: connection 1871 to localhost:27017 closed] to equal undefined
      at lib/test/test.js:737:26
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-batch/lib/index.js:87:30
      at _.map._.collect (/Users/hsiliev/workspace/cf-abacus/node_modules/underscore/underscore.js:172:24)
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-batch/lib/index.js:84:14
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-request/lib/index.js:341:11
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-transform/lib/index.js:100:9
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-request/lib/index.js:331:18
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-transform/lib/index.js:85:5
      at _.map._.collect (/Users/hsiliev/workspace/cf-abacus/node_modules/underscore/underscore.js:172:24)
      at Function.cbmap [as map] (/Users/hsiliev/workspace/cf-abacus/node_modules/abacus-transform/lib/index.js:83:3)
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-request/lib/index.js:324:19
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-transform/lib/index.js:100:9
      at /Users/hsiliev/workspace/cf-abacus/node_modules/abacus-request/lib/index.js:303:13
      at Request._callback (/Users/hsiliev/workspace/cf-abacus/node_modules/abacus-request/lib/index.js:173:14)
      at Request.self.callback (/Users/hsiliev/workspace/cf-abacus/node_modules/request/request.js:187:22)
      at Request.<anonymous> (/Users/hsiliev/workspace/cf-abacus/node_modules/request/request.js:1044:10)
      at IncomingMessage.<anonymous> (/Users/hsiliev/workspace/cf-abacus/node_modules/request/request.js:965:12)
      at endReadableNT (_stream_readable.js:975:12)
      at _combinedTickCallback (internal/process/next_tick.js:74:11)
      at process._tickCallback (internal/process/next_tick.js:98:9)

Seems that we have too many connections to MongoDB and this causes it to hit the configured OS limits:
2016-08-31T11:12:16.433+0300 I NETWORK  [initandlisten] connection accepted from <IP_ADDRESS>:52341 #2033 (2033 connections now open)
2016-08-31T11:12:16.433+0300 I NETWORK  [initandlisten] failed to create thread after accepting new connection, closing connection

We can:

reduce the search range in dataflow
work on mongoclient to close DB connections

I personally prefer the first approach since mongoclient has no clean way to know when a DB connection can be closed.

To fix the issue I'm going to:

allow end-date to be provided by the client
restrict the time window: end - beginning =< month + slackWindow

This should result in 2-3 connections per error list request.


Not being able to reconnect after network unavailability

Describe the bug
I have a client who often disconnects from the SRS server because of connectivity issues in their area. When the server attempts to reconnect automatically, SRS shows the following error message:
[2024-09-07 13:49:49.478][ERROR][15][40y6mec3][11] serve error code=1028(StreamBusy)(Stream already exists or busy) : service cycle : rtmp: stream service : rtmp: stream /live/livestream is busy thread [15][40y6mec3]: do_cycle() [./src/app/srs_app_rtmp_conn.cpp:263][errno=11] thread [15][40y6mec3]: service_cycle() [./src/app/srs_app_rtmp_conn.cpp:457][errno=11] thread [15][40y6mec3]: acquire_publish() [./src/app/srs_app_rtmp_conn.cpp:1078][errno=11](Resource temporarily unavailable)
Do you have any suggestions for avoiding this error? I'm considering whether SRS should disregard the current stream service and create a new one, but I'm not sure how to implement this.
Version
SRS  v5.0-r3 using default configuration
To Reproduce
With any stream client stream in rtmp, force a disconnection due to network and then reconnect

It will take a few seconds to let SRS remove the stream records.
One solution is before you republish the stream, query the streams.
curl http://localhost:1985/api/v1/streams/
get result:

{
"code": 0,
"server": "vid-05t951p",
"service": "0dpj008s",
"pid": "17211",
"streams": [
{
"id": "vid-o3r5w78",
"name": "test",
"vhost": "vid-7f9n237",
"app": "live",
"tcUrl": "rtmp://localhost:1935/live",
"url": "/live/test",
"live_ms":<PHONE_NUMBER>120,
"clients": 1,
"frames": 0,
"send_bytes": 4384,
"recv_bytes": 479328,
"kbps": {
"recv_30s": 0,
"send_30s": 0
},
"publish": {
"active": true,
"cid": "1x881deq"
},
"video": {
"codec": "H264",
"profile": "High",
"level": "3.2",
"width": 768,
"height": 320
},
"audio": {
"codec": "AAC",
"sample_rate": 44100,
"channel": 2,
"profile": "LC"
}
}
]
}

if you want to publish app/test, the stream {"name": "test", "app": "live",} match and conflict with your stream, then delete it by api:
curl -X "DELETE" http://localhost:1985/api/v1/clients/1x881deq
The 1x881deq come from there:

  "publish": {
    "active": true,
    "cid": "1x881deq"
  },

After this query and delete API, then I think you will never meet this error again.

Thanks will test it


CA documentation update

Main updates include,

1708 release CA updates
Screenshots update

AzSDKCCAutomationAccount was the name of older version of CA account (Before AzSDK 2.2.0).
This message was added to provide backward compatibility.


RelativeFormatters don't work with Swift Package Manager

This is a side effect of Swift Package Manager not working with bundled resources. So, the Formatters/RelativeFormatter/langs/ various json files aren't included and therefore aren't accessible when using Swift Package Manager.

Maybe it would be possible to bring these language mappings into the swift code so that it would work properly with Swift Package Manager?

at least it should not crash :) ATM it crashing in suitableFlavour localeData.flavours[Flavour.long.rawValue].

Gonna bump this. Is there a reason SwiftDate isn't working with bundled resources in SPM?

Yeah, because SPM doesn't support having bundled resources.

would some kindof "Run script" phase would help as a temp fix?

I don’t believe so. My understanding is that the resource files need to be codified.

Maybe if there was some script that you could tell the locales you wanted to support and it would copy the right codified files into place before the build. I am not sure how you would tie a script like this into the SPM build process. But might be worth looking into.

I was more thinking about app "Build Phase" "Run Script" that you have to add manually as a workaround

I know it can be a radical change and I can't foresee the consequences, but why not take into consideration the move from json files into swift files?

Update on SPM bundled resources: there is (as of this week) an official proposal now available for community comment.
Regardless, I agree with @robcas3 and @drewdeponte, the library shouldn't crash when a resource is missing. That seems like a doable fix.
Moving from those JSON files to Swift is quite the undertaking... Could that also hinder localization updates or additions?

for now you can add new build phase at the end
# Go to the build root and search up the chain to find the Derived Data Path where the source packages are checked out.
DERIVED_DATA_CANDIDATE="${BUILD_ROOT}"

while ! [ -d "${DERIVED_DATA_CANDIDATE}/SourcePackages" ]; do
  if [ "${DERIVED_DATA_CANDIDATE}" = / ]; then
    echo >&2 "error: Unable to locate SourcePackages directory from BUILD_ROOT: '${BUILD_ROOT}'"
    exit 1
  fi

  DERIVED_DATA_CANDIDATE="$(dirname "${DERIVED_DATA_CANDIDATE}")"
done

# Grab a reference to the directory where langs are checked out
LANGS_PATH="${DERIVED_DATA_CANDIDATE}/SourcePackages/checkouts/SwiftDate/Sources/SwiftDate/Formatters/RelativeFormatter/langs"

if [ -z "${LANGS_PATH}" ]; then
    echo >&2 "error: Couldn't find the langs in your checked out SPM packages; make sure to add the framework to your project."
    exit 1
fi

cp -f -R "${LANGS_PATH}" "${BUILT_PRODUCTS_DIR}/${PRODUCT_NAME}.app"

I'm getting this error when running unit tests on Xcode 12, it seems to be related to the same issue:
Details:  Error: Mismatch between existing container extension: <DVTExtension 0x7fc7aa5a49d0: Swift Package Folder (Xcode.IDEFoundation.Container.SwiftPackageFolder) from com.apple.dt.IDE.IDESwiftPackageSupport> and requested container extension: <DVTExtension 0x7fc7ad1b6800: Folder (Xcode.IDEFoundation.Container.Folder) from com.apple.dt.IDEFoundation> for file path: <DVTFilePath:0x7fc7b1d824f0:'/Users/mkalatrash/Library/Developer/Xcode/DerivedData/Travel-bavpgrmwbxqafacvgowqxhoryoeq/SourcePackages/checkouts/SwiftDate/Sources/SwiftDate/Formatters/RelativeFormatter/langs'>
Object:   <IDEContainer>
Method:   +_retainedContainerAtFilePath:fileDataType:workspace:options:error:
Thread:   <NSThread: 0x7fc7ad207c60>{number = 1, name = main}
Hints:

If I run unit tests through Xcode it will crash, the above is from the command line. @RolandasRazma I did add the run script but it didn't work.

Should we just not attempt to use SPM with SwiftDate ? Or is it only an issue under certain scenarios ?
@mohammad19991 Xcode only crashed for me when I had code coverage on.

@mohammad19991 script is for 6.1.0 version. 6.2.0 shouldn't need script but Xcode having problems...

I closed #758 but is it a new issue ? I can compile and run using SPM and execute unit tests. When I attempt to get code coverage then Xcode just crashes.


Write a Rust program to print numbers from n to 1

Description
Write a Rust program to print numbers from n to 1
Input  : 5
Output : 5 4 3 2 1

How to contribute

Save the solution in program/print_numbers_from_n_to_1.rs file
Add print_numbers_from_n_to_1.rs file in print-numbers-from-n-to-1 folder

!assign


effective use of fetchTimestamp methods

DO NOT MERGE this PR unless you intend to debug the problems that it creates. This PR is for illustrative purposes only. The problem fixed by this PR is described at https://github.com/USGS-VIZLAB/vizlab/issues/192#issuecomment-329018673.

did you mean to close this @aappling-usgs ?

Yes I did, because I wanted you to see this PR but to have to work really hard to accidentally merge it =)


[PSA] "write only" methods

"Write-only" methods. This particular choice will force most programs (except for the trivial ones) to keep separate metadata to keep track of the port/multicast_group/drop decisions and use these primitives only at the very end of the ingress control. For example, very often an ACL might want to match on an egress port for a unicast packet. Or, sometimes, an ACL might want to reverse a drop() decision. Neither is possible with the current set of PRE methods without having additional metadata carried alongside.

I think a better name to describe this isn't "write only", but "cannot be undone".  From all appearances in the spec, if you do a drop() action, it cannot be undone later.
I am sympathetic to @vgurevich's desire for defining what happens to a packet via explicit metadata fields.  Such fields would be both readable and writable throughout the ingress control block, and actions could be 'undone' if desired.  The behavior could be precisely specified in terms of pseudocode, e.g. something like:
if (std_metadata.drop == 1) {
    packet is dropped, i.e. no packet is written to the PRE packet buffer, and thus no packet will be sent to the egress control block later due to this ingress control block processing
} else if (std_metadata.multicast_group != 0) {
    // description of multicast behavior here
} else if ... more conditions here ...

Most of the functions in the PRE do allow 'undo' by simply setting a new value to the port/group/etc. And since you are setting a port/group/etc to a value, that value comes from variable that one can match against (or a constant). The style of programming that you give as an example encourages using global structures that the architecture needs to maintain in metadata -- for all the metadata possible, not just the one it is used (and then of course, to rely on the compiler to eliminate the unused metadata). It seems to me that this is against the trend in P4_16 of localizing state. Moreover, in your example you have a 1 bit drop. Some architectures may use several bits to implement a variety of drops which won't be expressible in the PSA standard metadata.
Undoing drop is indeed not possible in the current draft. However, I'm curious to hear if this is such a big issue. If ACL wants to undrop, why don't we check the negated condition and drop at that point. I do have a hard time understanding how come you made a decision to drop the packet and now that condition no longer holds. Is this just an artifact or a real need?

Whether all of these things are in visible metadata fields, or in 'hidden state' inside of the PRE object, the state is 'global to the ingress control block'.  Neither approach localizes this state more than the other, as far as I can tell.  One approach makes the state accessible only via method calls, the other via metadata variables.
That said, I don't know of a strong need to reverse drop decisions made earlier.  If someone wants to, they will have to maintain their own metadata about the decision, and delay calling drop() until the end of the ingress control block, as @vgurevich noted.

Yes, in the end it is a matter of taste. It would be great if other folks express their opinions one way or another and we can vote on the direction if we can not reach consensus online.

There are a couple of issues at hand:

Functional completeness. To me, every time I see set() without a get() and reset() it is usually a red flag
Efficiency. More complex programs that also need more resources will be required to carry extra metadata and execute additional code at the end to perform the resolution and call the required methods at the very end of the pipeline. Yes, I know that portability might have a certain price, the question is should we start paying it even in the simple cases
Delayed effect. When someone does "pre_metadata.drop_flag=1;" I think it is quite clear that the packet is not going to be dropped right away. When you do "pre.drop();" the impression is quite different. Yes, it will be documented, but still. Certain language constructs carry certain "first impressions" with them
More difficulties for platform-independent tools. For example how can a tool find out if there is a dependency between tables that call these various primitives?
A slightly funky interaction between them. For example, if one wants to ensure that the packet actually is sent to port, they need to do something like

action real_send_to_port(in PortId_t port) {
    pre.multicast(0);
    pre.send_to_port(port);
}

This raises a lot of questions (since actions are executed sequentially)... Also, I assume that multicast() and send_to_port() do cancel drop(), but I am not sure that has been mentioned explicitly.
The explicit metadata sure has its own set of problems, especially for implementations that decide to structure their metadata differently. The atomicity problem (5) will still be present too. However, they will be more visible and everyone will be able to see what's happening (including both tools and humans). In addition to that, it will lead to more efficient more "creative" code, although one can argue that in real data plane programs people will tend to abstract all these anyway...

Perhaps this issue can be closed now?

@jafingerhut ,
Given that we switched to "intrinsic metadata is the primary source of truth" approach, I agree -- we can close.


There is a compile warning!

This is the warning infomation:
ld: warning: linking against dylib not safe for use in application extensions: /Users/EricChen/Library/Developer/Xcode/DerivedData/ERCEngine-gjizomxiwiastihknrteecwkobuh/Build/Products/Debug-iphonesimulator/Alamofire.framework/Alamofire

Any chance you could provide a sample project that demonstrates this?

Additionally, how did you install Alamofire and which version did you install?

Please try to wipe your DerivedData and Xcode cache and try it again. We cannot reproduce this issue. If you end up not being able to fix the issue, please provide more info and possibly a sample project and I'll happily re-open the issue.


If needed sigaction in km needs to support SA_NOCLDSTOP, SA_NOCLDWAIT, SA_NODEFER, and SA_RESTART

The sigaction structure has a field named sa_flags.  km does not support all flags that can be passed in sa_flags.
The following flags need to be investigated and we need to decide what needs to be done:
SA_NOCLDSTOP
SA_NOCLDWAIT
SA_NODEFER
SA_RESTART

I found that the musl pthread cancel code uses the SA_RESTART flag for its SIGCANCEL (sig 33) handling.
So, we need to support this for musl.
We've been ignoring SA_RESTART up until now.


Monitoring gem should use the same authentication as other gems

The monitoring API doesn't take a keyfile in object creation. It also doesn't appear to use the environment variables the other gems use. All our gems should support the same authentication schemes.
Compare: https://googlecloudplatform.github.io/google-cloud-ruby/#/docs/google-cloud-logging/v1.0.1/google/cloud/logging?method=new-class
and
https://googlecloudplatform.github.io/google-cloud-ruby/#/docs/google-cloud-logging/v1.0.1/guides/authentication
to
https://googlecloudplatform.github.io/google-cloud-ruby/#/docs/google-cloud-monitoring/v0.24.0/google/cloud/monitoring/v3/metricserviceclient?method=initialize-constructor

@thagomizer We currently support this through the chan_creds parameter of the client object constructor (https://googlecloudplatform.github.io/google-cloud-ruby/#/docs/google-cloud-monitoring/v0.24.0/google/cloud/monitoring/v3/metricserviceclient?method=initialize-constructor). We don't necessarily intend to support the exact same format as the manually written surfaces as long as it's not overly complicated to configure client authentication using a keyfile -- it seems like the chan_creds should make this possible.
Does that seem reasonable?

We can take this offline but the expectation of the user community is that all of the gems should support the same authentication schemes. There is shared code that's available to be used for this.

Sure, let's discuss offline.

After some discussion in the team, we've decided to support passing a path to a keyfile for channel credentials. Tracking the code generator change as https://github.com/googleapis/toolkit/issues/1324.

Closing as code generator change is tracked in googleapis/toolkit#1324.


"assert" error

Steps to reproduce
$ pnpm create farm@latest
$ cd farm-project
$ pnpm i
$ pnpm run dev
Reproduce link
No response
What is actually happening?
Can't even try this project. I used the standard "Creating a Svelte Project" steps from https://www.farmfe.org/docs/frameworks/svelte
No modification of template whatsoever. This is what I'm getting.
><EMAIL_ADDRESS>dev /home/howe/ttt/farm-project
> farm

file:///home/howe/ttt/farm-project/node_modules/browserslist-generator/dist/esm/index.js:3
import * as compatData from '@mdn/browser-compat-data' assert { type: 'json' };
                                                       ^^^^^^

SyntaxError: Unexpected identifier 'assert'
    at compileSourceTextModule (node:internal/modules/esm/utils:337:16)
    at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:163:18)
    at callTranslator (node:internal/modules/esm/loader:430:14)
    at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:436:30)

Node.js v22.2.0
 ELIFECYCLE  Command failed with exit code 1.

I've seen similar issue on #921 but Node version is 22.2. Please advice.
System Info
$ uname -a
Linux gargamel 6.9.3-arch1-1 #1 SMP PREEMPT_DYNAMIC Fri, 31 May 2024 15:14:45 +0000 x86_64 GNU/Linux
$ node --version
v22.2.0

Dependencies:
$ pnpm list                                               254 ↵
Legend: production dependency, optional only, dev only

<EMAIL_ADDRESS>/home/howe/ttt/farm-project

devDependencies:
@farmfe/cli 1.0.2
@farmfe/core 1.1.13
@sveltejs/vite-plugin-svelte 3.1.1
@tsconfig/svelte 5.0.4
core-js 3.37.1
svelte 4.2.18
svelte-check 3.8.0
tslib 2.6.3
typescript 5.4.5

please try node 20 for now，we'll fix it soon

It did work, thanks for the prompt reply and fix.


Curl command cannot get the json file

hi there,
curl -O https://github.com/elastic/examples/blob/master/Machine Learning/Security Analytics Recipes/dns_data_exfiltration/machine_learning/data_feed.json
curl -O https://github.com/elastic/examples/blob/master/Machine Learning/Security Analytics Recipes/dns_data_exfiltration/machine_learning/job.json
curl -O https://github.com/elastic/examples/blob/master/Machine Learning/Security Analytics Recipes/scripts/reset_job.sh
all these commands cannot get the proper files but only the github html, please rectify.
thanks!

Use the -L option with curl
curl -OL https://github.com/elastic/examples/blob/master/Machine Learning/Security Analytics Recipes/dns_data_exfiltration/machine_learning/data_feed.json
to follow redirects.


add react-native-walkthrough-tooltip

https://github.com/CompanyCam/react-native-walkthrough-tooltip
https://medium.com/companydev/calling-attention-to-new-features-in-our-react-native-app-ba0d1daa380

Cool project @gianpaj ! I dig it.


Spelling grammar formatting fixes

Thanks for putting this tutorial together. Despite a lot of the math being over my head it was very helpful to see a "real world" implementation of this stuff. As I went through the tutorial, I made small updates to formatting, spelling, and grammar to make the tutorial more legible to future readers. Hope this helps! 🫡

Very cool! Thanks for the PR @bagelface.  If you want to take your STARK-skills to the next level, have a look at the follow-up tutorial:
https://github.com/aszepieniec/stark-brainfuck


[POS] CheckSync is not implemented in C#

Check against synchronized checkpoint is in stratisX which prevents reorgs longer than 500 blocks

Ah ok yeah we need to add that.

 [POS] CheckSync is not implemented in C#

I would suggest we put this limit in to the ChainHeadersBehavior
Just before this part
	if (this._PendingTip.ChainWork > this.Chain.Tip.ChainWork)
	{
		this.Chain.SetTip(this._PendingTip);
	}

Yes, but I'd still like to have the test for this working before I finish this.

Ah the test is working do you have issues with the test?

Yes, from day 1 the test does not work for me.

Ah ok I realise that's because we need to pull a fix from NStratis.

This test now works fine.

Yes, it is now better, except that it does not use ConsensusOptions.MaxReorgLength, which it actually should use.
Another thing is that the test don't pass even if I set MaxReorgLength manually to 10.

So, could you please check the latest version of #492?

I will probably introduce logs to ChainHeadersBehavior to see what's happening.

I mean, we certainly need that test to use ConsensusOptions - so I'd like to ask you to update the test code. And my task is to see why it does not work. I will analyze that and let you know.

OK, so I've advanced enough with the implementation (PR #492), so I believe that PR implements long reorg protection well now. However, the test Given__NodesAreSynced__When__ABigReorgHappens__Then__TheReorgIsIgnored does not currently work with this implementation. The test needs to be fixed:

There are two mining nodes and coincidentally as their mining tasks are run in parallel, they actually mine the same blocks. This prevents the two mined chains to be different enough (above the max reorg length). Therefore the reorg is accepted.

This seems to have an easy fix:
                    var t1 = Task.Run(() => stratisMiner.GenerateStratisWithMiner(10));
                    var t2 = Task.Run(() => stratisReorg.GenerateStratisWithMiner(12));

->
                    var t1 = Task.Run(() => stratisMiner.GenerateStratisWithMiner(10));
                    var t2 = Task.Run(async () => { await Task.Delay(2000); stratisReorg.GenerateStratisWithMiner(12); } );

A delay makes sure that first mined blocks won't be the same.

Once we fix the same mined block problem above, the test's WaitLoop fails (even in debug mode where it is given 15 minutes). The logs show why:

Node 1:

[2017-10-14 09:12:51.2312 33] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__70_0 ======Node stats====== 10/14/2017 09:12:51 agent StratisBitcoin:1.0.2
Headers.Height:        13       Headers.Hash:         c1ac14c3abbed3f8cac5fcab15348fc2cc8928b2c9cf5508d133fcba07bcde17
Consensus.Height:      13       Consensus.Hash:       c1ac14c3abbed3f8cac5fcab15348fc2cc8928b2c9cf5508d133fcba07bcde17
BlockStore.Height:     13       BlockStore.Hash:      c1ac14c3abbed3f8cac5fcab15348fc2cc8928b2c9cf5508d133fcba07bcde17
Wallet.Height:         No Wallet

------------------------------
Node 2:

[2017-10-14 09:12:54.8409 80] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__70_0 ======Node stats====== 10/14/2017 09:12:54 agent StratisBitcoin:1.0.2
Headers.Height:        13       Headers.Hash:         c1ac14c3abbed3f8cac5fcab15348fc2cc8928b2c9cf5508d133fcba07bcde17
Consensus.Height:      11       Consensus.Hash:       3eeb8ef82fbeac1b6b2bbfd06b7305c5df978cf3ce89d237e809f1a83f7324f9
BlockStore.Height:     11       BlockStore.Hash:      3eeb8ef82fbeac1b6b2bbfd06b7305c5df978cf3ce89d237e809f1a83f7324f9
Wallet.Height:         No Wallet

------------------------------
Node 3:

[2017-10-14 09:12:55.5154 80] INFO: Stratis.Bitcoin.FullNode.<StartPeriodicLog>b__70_0 ======Node stats====== 10/14/2017 09:12:55 agent StratisBitcoin:1.0.2
Headers.Height:        11       Headers.Hash:         3eeb8ef82fbeac1b6b2bbfd06b7305c5df978cf3ce89d237e809f1a83f7324f9
Consensus.Height:      11       Consensus.Hash:       3eeb8ef82fbeac1b6b2bbfd06b7305c5df978cf3ce89d237e809f1a83f7324f9
BlockStore.Height:     11       BlockStore.Hash:      3eeb8ef82fbeac1b6b2bbfd06b7305c5df978cf3ce89d237e809f1a83f7324f9
Wallet.Height:         No Wallet

As we can see the two mining nodes are where they should be, the problem is with the stratisSyncer node, which received (and rejected) the headers from stratisReorg, but it then did not receive any better headers from anyone.
There are several ways how to fix this. Either we should fix the node as once it rejects the headers, it should reset its headers to something it trusts, OR maybe after certain time (to make sure the reorg has been seen by the syncer), we should mine extra blocks with both stratisMiner and stratisSyncer and once stratisMiner's chain is longer than 13, the syncer's chain should switch to it (and should ignore stratisSyncer's chain, if if it is longer).

So, I've checked the second solution of problem 2) and it seems that it works - so I've basically mined blocks with both miners:
                    t1 = Task.Run(() => stratisMiner.GenerateStratisWithMiner(10));
                    t2 = Task.Run(() => stratisReorg.GenerateStratisWithMiner(12));

so the stratisMiner's chain is 21 blocks long and stratisReorg's chain is 25 in length and the stratisSyncer correctly follows stratisMiner's (i.e. shorter chain). So that fix would also work.
So I think it is possible to just update the test and it will work well.

And previously I forgot:

the test also have to to set MaxReorgLength in ConsensusOptions to 10, otherwise the default value is 500 and it won't work.

Ok sounds fine how you solved the test, you could also just trigger a sync fromstratisSyncer
I will have a better look at the code on Monday

bump

Yes sorry this was delayed for Breeze.


Missing "Calculated" data type

The online help, as well as Wiley's Access 2016 Bible, do not list the "Calculated" data type.

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

ID: a18b1de9-6947-586a-81e1-da7fc644d66a
Version Independent ID: 9aea7103-a646-5821-4cc7-e342a8d1a65a
Content: DataType property
Content Source: access/Concepts/Miscellaneous/datatype-property.md
Product: access
GitHub Login: @o365devx
Microsoft Alias: o365devx

@EdwardDice Do the topics under this topic help? https://docs.microsoft.com/en-us/office/vba/api/access.format.propertydate.time

Linda-Editor
The link you provided was for the format options under a Date/Time data type.
No, that does not address the issue.
I was talking about the Calculated data type of a field, as in a field of a table.
Hope you have a great holiday season
Merry Christmas
Ed

@EdwardDice In searching our documentation, it appears that the only "Calculated" items are under Excel. Hoperfully someone else will respond with more information. You might also try Stack Overflow (https://stackoverflow.com/questions/tagged/vba) to see if they have any ideas.

See Access: Add calculated data type to a table. Otherwise, try the Access developer community on Stack Overflow.


Mocking ui.Image for use in a unit test causes an engine crash

As part of #2198, I'm trying to mock out the ui.Image objects looked up in a mocked AssetBundle. These mock ui.Image objects are referenced from a build() function that returns an AssetImage.
Calling WidgetTester.pumpWidget() once with my build function initializes the appropriate ImageResources, but they're not populated yet. Calling pumpWidget() a second time causes the engine to crash.
Steps to Reproduce
Here's the mock ui.Image class:
class TestImage extends ui.Image {
  TestImage(this.scale);
  final double scale;
  int get width => (48*scale).floor();
  int get height => (48*scale).floor();
  void dispose() { }
}

There's a bunch of boilerplate I use to get this test image to be returned by my mock AssetBundle, but the end result is that the current canvas is passed this ui.Image subclass in its Canvas.drawImageRect() method.  At that point, the engine crashes.  Running with a locally built debug engine results in this assertion:
ASSERTION FAILED: image
../../sky/engine/core/painting/Canvas.cpp(260) : void blink::Canvas::drawImageRect(const blink::CanvasImage *, blink::Rect &, blink::Rect &, const blink::Paint &)
I assume this is because the mock class has a null in a field that is normally populated.  The relevant classes in the engine seem to be Canvas.h/.cpp and CanvasImage.h/.cpp.
@abarth mentioned that DartConverter is supposed to guard against malformed arguments passed into the native engine layer, but it's unclear to me as of writing where DartConverter would be used in the native method binding call sequence.

@Hixie

This is failing because the native fields of the TestImage were never set.
ui.Image inherits from NativeFieldWrapperClass2, and native code expects that a ui.Image will have values for the DartWrappable native fields.
However, ui.Image does not provide a constructor that is accessible from Dart and can fill in the native fields.  ui.Image is abstract and can not be instantiated from Dart, but there's nothing that prevents creating a subclass of ui.Image.
When DartConverterWrappable::FromArguments receives a handle to an instance of TestImage, it calls Dart_GetNativeFieldsOfArgument, which returns success and fills the native fields output with zeros.  The zero is then passed on as a null CanvasImage* to Canvas::drawImageRect.
DartConverterWrappable::FromArguments needs a check for this.

Thanks for looking further. Can you point me to where Dart_GetNativeFieldsOfArgument (or DartConverterWrappable::FromArguments) gets called in this case?

CanvasImage uses the DART_NATIVE_CALLBACK macro to generate the native functions for drawImageRect.
The generated function invokes DartCall -> DartArgIterator -> DartConverter<DartWrappable*>::FromArguments -> DartConverterWrappable::FromArguments


core: record store approximate size

use Mb as unit, add RegionSize and LeaderSize in StoreInfo

LGTM. Don't forget to update grafana metrics.


Macro: display atoms and bonds on canvas in macro view

Description
It needs to display atoms and bonds in macromolecules mode similar as it does in molecules mode but with some limitations. Atoms should be displayed as dots, without any text or other additional information. Bonds should be displayed as one line regardless which type of bond it is. Position of atoms and bonds should be the same in both modes. User settings should be ignored in macro mode.

Tests added to the automation table.


Add symbol double square brackets

I added a symbols for left/right double square brackets, unicode U+27E6 and U+27E7. For the properties I mimicked similar symbols such as floor/ceiling which were already included.
I tested with yarn start and got the error No character metrics for '⟦' in style 'Main-Regular'. I tried to see if I should address this and found the issues https://github.com/KaTeX/KaTeX/issues/374 and https://github.com/KaTeX/KaTeX/issues/1817 which gave me the impression I could overlook it.
The names \llbrack and \rrbrack are chosen to be consistent with KaTeX's \lbrack and \rbrack for single square brackets. They differ slightly from those found in the stmaryrd package for LaTeX (see Double square brackets @ TeX SE) which uses \llbracket and \rrbracket.
I would also like to make them into delimiters to work with \left and \right, but didn't see similar unicode symbols to the upper/lower corner and extension symbols for single square brackets (U+23A1, U+23A2, U+23A3).

Codecov Report

Merging #1947 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1947      +/-   ##
==========================================
+ Coverage   94.91%   94.91%   +<.01%
==========================================
  Files          79       79
  Lines        4835     4837       +2
  Branches      844      844
==========================================
+ Hits         4589     4591       +2
  Misses        222      222
  Partials       24       24

Flag
Coverage Δ

#screenshotter
89.47% <100%> (ø)
:arrow_up:

#test
86% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

src/symbols.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f948947...fbdff77. Read the comment docs.

@ben-dyer Nice job finding a LaTeX function name for that character. I once looked for such a function name and did not find it.
The KaTeX convention has consistently been to use pre-existing LaTeX function names whenever possible rather than inventing new function names. I think that using the names \llbracket and \rrbracket would be a better choice.
Also, you can avoid the error message by means of a somewhat sneaky macro.  Take a look at the code in macros.js. I suggest we modify that code so that it also includes your proposal. Something like this:
//////////////////////////////////////////////////////////////////////
// stmaryrd and semantic

// The stmaryrd and semantic packages render the next four items by calling a
// glyph. Those glyphs do not exist in the KaTeX fonts. Hence the macros.

defineMacro("\\llbracket", "\\html@mathml{" +
    "\\mathopen{[\\mkern-3.2mu[}}" +
    "{\\mathopen{\\char`\u27e6}}");
defineMacro("\\rrbracket", "\\html@mathml{" +
    "\\mathclose{]\\mkern-3.2mu]}}" +
    "{\\mathclose{\\char`\u27e7}}");

defineMacro("\u27e6", "\\llbracket");  // blackboard bold [
defineMacro("\u27e7", "\\rrbracket"); // blackboard bold ]

Also, could you add documentation to this PR? That would entail editing files docs/support_table.md and docs/supported.md.

Thank you for such helpful feedback! Yes I'll get working on these updates.

This sounds incredibly good! Can this be included in the coming release?

Can there be double-square braces (or even parenthesis)?

@ice1000 We could probably use a similar macro trick to emulate double-struck braces. Something like: \mathopen{\{\mkern-4.3mu|}. I don't see any such combination that would work for parenthesis. It looks like those will have to wait until someone can add new glyphs to the KaTeX fonts.
Until then, you can still use the Unicode characters ⦅ and ⦆. That technique has some drawbacks. KaTeX will get a glyph from a system font and put up a console warning. Also the height of the glyph might not be ideal.  But it might be better than nothing.

@ronkok I've added this as a PR, #1953.
Could you please take a look at that?

@ronkok I updated the documentation and changed the names to \llbracket and \rrbracket. I also made the change to macros as suggested, and verified it's working and the error message is gone.

Looks good to me.

@ben-dyer Nice work. Thank you for the contribution.

Closed??

@ice1000 My bad. Fixed now.


VST parameters back to default after reloading project when plugin uses states

Here's a way to reproduce the issue:

Load the VST version of ZamHeadX2, ZamSynth, ZamVerb or ZaMultiComp in Carla
Tweak the plugin's parameters
Save the project
Load the project

The parameters are now back at their default values.
This also happens in Renoise and Bitwig 1.3.15, but not in Ardour.
If "Use Chunks" is not ticked in the plugin's settings in Carla, the parameters are saved / loaded correctly, but the state is lost.
What's happening? :)

The relevant code is here: https://github.com/DISTRHO/DPF/blob/master/distrho/src/DistrhoPluginVST.cpp#L584
And I see what's going on, the parameter data is not being saved after the chunks.
I remember coding support for this... maybe in some other project.. hmm


Doesn't appear to account for custom table names

Some of my models are broken up into namespaces.  In order for me to more easily differentiate my database tables, I have prefixed table names with their namespace. For example:
class Acme::Project < ActiveRecord::Base
  self.table_name = 'acme_projects'

But for has_one relationships, consistency_fail will look for a unique index attached to project_id rather than acme_project_id.  Is there a way, when self.table_name is used, to take it into account?
Thanks,
Tim

This issue is fixed in the 0.3.4 release, thanks to @fw42's fix from #21.

(Thanks for reporting it!)


handle_system_req Memory includes total

@rnewson
Originally was done here: https://github.com/cloudant/chttpd
moved to this repo

How can I trigger this handler via HTTP API? So far I see there is no place in code that makes it available to handle incoming requests.

@kxepal: does seem to be missing from https://github.com/apache/couchdb-chttpd/blob/master/src/chttpd_httpd_handlers.erl :/

@rnewson Yes, I noticed that, but commentary for the handler says that it's only available for backdoor interface...or at least it should be.
I'll add it to default.ini where global httpd handlers are defined and ensure what it's available by /_node/X/_system path on cluster interface, ok?

ah, quite right. Let's just add it to :5986 for now

actually /_system is available on both front and back ports at Cloudant, I just checked, so we should do the same.

@rnewson ok, I'll made PRs for this.

Hm, not sure what's use case for this patch. Example output:
    "memory": {
        "atom": 801697,
        "atom_used": 793412,
        "binary": 1656328,
        "code": 23299745,
        "ets": 1753608,
        "other": -36439634,
        "processes": 8933048,
        "processes_used": 8919640
    }

Notice negative other value. I believe, total value should be shows with separate object field.

hmm...I wasn't sure if this would solve my problem or not. This was a response for:
https://cloudant.fogbugz.com/f/cases/47888/DBcore-memory-usage-displayed-in-admin-dashboard-is-wrong
Which is a bug in:
https://github.com/cloudant/wilson/blob/develop/wilson/views/clusters.py
The Wilson endpoint pulls info from graphite (and haproxy).

@jcounts Sorry, I don't have access to Cloudant fogbugz and private repositories. What's the case there is short which is fine to gather negative memory values?

@jcounts also, how about to backport those fogbugz ticket to CouchDB JIRA and bound it with this commit? Would be helpful to track the things.


Update npm package

Please consider updating your npm package with your latest commit , not working on angular 12 (last updated 6 years ago), I
replaced the js file on node modules with your latest commit, and worked again.

Yes, that would be nice.


Hot reload disconnect loops when SSR takes more than 1sec

Problem
When rendering SSR page after recompilation, and having the web page open trying to reconnect the web socket, if the rendering takes 1sec or longer, the connect/disconnect goes into a loop for a long time.
Steps To Reproduce
Make a server function call with use_server_future that takes more than 1 second to complete. When hot reload tries to reconnect over web socket, it times out before the rendering gets done, and tries to reconnect.
Even doing forced refresh in the browser doesn't break the loop.
This doesn't happen if the call takes less than 1sec.
Expected behavior
Should reconnect normally.
Screenshots

Environment:

Dioxus version: 0.5.0 alpha0
Rust version: 1.76
OS info: Win11, WSL2
App platform: fullstack

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

This could be fixed by introducing some backoff in the reconnect delay.

This looks like the same issue as https://github.com/DioxusLabs/dioxus/issues/2032. You can follow that issue for more updates. We shouldn't be running server functions on any routes that do not return HTML


Niet verder in behandeling genomen - dates are swopped

In cases where the assessment procedure is stopped without a formal decision (e.g. withdrawal of application), something goes wrong with the dates:
The date "Laatste beoordeling ontvangen op" seems to be swopped with the date "Afhandeling: Niet verder in behandeling genomen". This also leads to errors in the statistics.

For something to be swapped, there actually needs two things that swap places. That is not the case here, there is only one date with a misleading label. A correcter name would be 'Date review ended'; This normally corresponds to 'date last decision', but in this case the review wasn't ended because the last decision was received.
(Internally, the field is actually called date_ended).
That being said, that's just a explanation for this behavior. It also points us to a potential fix: a new field date_last_decision. This field should be updated every time a decision is made (hint: Decision.update_go); That way, the field will correspond to date_end when the review is closed normally, but will have a sensical name in the discontinue scenario.

Not sure if I understand this correctly. There can be two dates in case of 'niet verder in behandeling genomen', but this does not necessarily have to be the case. In principle, an application can be 'niet verder in behandeling genomen' at any time, also right after submission. In some cases, a first decision has already been sent, and that date should be the date the applicant was informed about that decision. The date 'niet verder in behandeling genomen' can be an entirely different date.


Forge: emit error message on common failures

Resolves #345.

Amazing, thanks

Released in 2.21.1


Update link 2017-41-04_17-11-20

:warning: Validation status: warnings

File
Status
Preview URL
Details

split/test_link_to_con.yml
:warning:Warning

Details

split/test_link_to_con.yml

Line 1: [Warning] Invalid file link:(~/2017-41-04_17-11-20.md).

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.


Add clear ipvs virtual server table when reset k8s

I used kube-proxy ipvs mode, after I reset the cluster, I found the virtual server table of ipvsadm still there.
So I want to improve the reset part for clear ipvs virtual server table.
Thank you.
cc #3465, #3416

@sufuf3 please check if CLA signed with same email commit has

@Atoms Thank you for the review.
I updated it. 😃

/lgtm
/approve


new binary image

Changelog category (leave one):

Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...

new binary image

Documentation entry for user-facing changes

[ ] Documentation is written (mandatory for new features)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

This is an automated comment for commit dd145e0a79ff44b81fe6f96e15eb68bc468a22a5 with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟢 success
Check nameDescriptionStatus
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success

This is an automated comment for commit dd145e0a79ff44b81fe6f96e15eb68bc468a22a5 with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟢 success
Check nameDescriptionStatus
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success