aeronmd.c Closing multi-publisher IPC publication

I observe unexpected behaviour -- when multiple publisher clients open the same "shared" IPC publication, the clients 2 and further don't seem to get publication "assigned" to them. To illustrate, see the code below (it's in Julia, but should be easy to follow).
I'm using C media driver, and I could reproduce this behaviour on version 1.43.0, and I have been able to reproduce it on 1.35 and 1.41 as well.
Setup
Important thing to note is that each publication (and subscription) has its own connection to AeronMD (via AeronConductor component).
using AeronClient

function make_conductor()
    c = AeronConductor()
    t = Timer(t -> aeron_poll(c), 0.0; interval = 0.1)
    c
end

function make_subscription(channel, stream)
    s = AeronSubscription(make_conductor(), channel, stream)
    t = Timer(0.1; interval = 0.1) do t
        aeron_poll(s) do m
            println("\nReceived: $(String(m))\n")
        end
    end
    wait(s)
    s
end

function make_publication(channel, stream)
    p = AeronPublication(make_conductor(), channel, stream)
    wait(p)
    p
end

case 1 -- closing second publication is ineffective (and returns error)
STREAM = 12345
p1 = make_publication("aeron:ipc", STREAM)
s = make_subscription("aeron:ipc", STREAM)
p2 = make_publication("aeron:ipc", STREAM)
# both messages received in subscriber
aeron_offer(p1, codeunits("p1_m1"));
aeron_offer(p2, codeunits("p2_m1"));
# p1 closes successfully, subscriber still has the image, as expected
close(p1)
# message received, so publication is still active
aeron_offer(p2, codeunits("p2_m2"));
# failure -> publication not found, subscriber still has the image
close(p2)
exit()

AeronMD log (v1.43.0)
[2212937.053159] DRIVER: NAME_RESOLUTION_HOST_NAME [38/38]: durationNs=6000 hostName=m75-office-003
[2212957.556878] DRIVER: CMD_IN_ADD_PUBLICATION [56/56]: 12345 aeron:ipc [1:2]
[2212957.556904] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 1 36
[2212957.559080] DRIVER: CMD_OUT_PUBLICATION_READY [98/98]: -1213691572:12345 38 -1 [2 2]
    "/dev/shm/aeron-sfokin/publications/2.logbuffer"
[2212957.824008] DRIVER: CMD_IN_ADD_SUBSCRIPTION [64/64]: 12345 aeron:ipc [-1][3:4]
[2212957.824019] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 3 39
[2212957.824022] DRIVER: CMD_OUT_SUBSCRIPTION_READY [28/28]: 4 -1
[2212957.824028] DRIVER: CMD_OUT_AVAILABLE_IMAGE [109/109]: -1213691572:12345 [40:4] "aeron:ipc" [2] "/dev/shm/aeron-sfokin/publications/2.logbuffer"
[2212958.109689] DRIVER: CMD_IN_ADD_PUBLICATION [56/56]: 12345 aeron:ipc [5:6]
[2212958.109697] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 5 41
[2212958.109700] DRIVER: CMD_OUT_PUBLICATION_READY [98/98]: -1213691572:12345 38 -1 [6 2]
    "/dev/shm/aeron-sfokin/publications/2.logbuffer"
# close(p1)
[2212974.879606] DRIVER: CMD_IN_REMOVE_PUBLICATION [40/40]: 2 [1:7]
[2212974.879612] DRIVER: CMD_OUT_ON_OPERATION_SUCCESS [24/24]: 7
# close(p2)
[2212988.861501] DRIVER: CMD_IN_REMOVE_PUBLICATION [40/40]: 2 [5:8]
[2212988.861529] DRIVER: CMD_OUT_ERROR [185/185]: 8 3 (-3) unknown publication
[aeron_driver_conductor_on_remove_publication, aeron_driver_conductor.c:3578] unknown publication client_id=5 registration_id=2
# exit()
[2213017.880451] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [5:9]
[2213017.880454] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [3:10]
[2213017.880454] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [1:11]
[2213018.097066] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213018.097070] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213018.097073] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213019.097068] DRIVER: REMOVE_PUBLICATION_CLEANUP [41/41]: sessionId=-1213691572 streamId=12345 channel=aeron:ipc

case 2 -- same as case1, but we close p2 first
STREAM = 54321
p1 = make_publication("aeron:ipc", STREAM)
s = make_subscription("aeron:ipc", STREAM)
p2 = make_publication("aeron:ipc", STREAM)
# both messages received in subscriber
aeron_offer(p1, codeunits("p1_m1"));
aeron_offer(p2, codeunits("p2_m1"));
# failure -> publication not found, subscriber still has the image
close(p2)
# message received, so publication is still active
aeron_offer(p1, codeunits("p1_m2"));
# p1 closes successfully, BUT subscriber still has the image, which is unexpected
close(p1)

AeronMD log (v1.43.0) -- clean start
[2213062.407485] DRIVER: NAME_RESOLUTION_HOST_NAME [38/38]: durationNs=3770 hostName=m75-office-003
[2213074.641719] DRIVER: CMD_IN_ADD_PUBLICATION [56/56]: 54321 aeron:ipc [1:2]
[2213074.641743] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 1 36
[2213074.643871] DRIVER: CMD_OUT_PUBLICATION_READY [98/98]:<PHONE_NUMBER>:54321 38 -1 [2 2]
    "/dev/shm/aeron-sfokin/publications/2.logbuffer"
[2213074.907829] DRIVER: CMD_IN_ADD_SUBSCRIPTION [64/64]: 54321 aeron:ipc [-1][3:4]
[2213074.907842] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 3 39
[2213074.907845] DRIVER: CMD_OUT_SUBSCRIPTION_READY [28/28]: 4 -1
[2213074.907851] DRIVER: CMD_OUT_AVAILABLE_IMAGE [109/109]:<PHONE_NUMBER>:54321 [40:4] "aeron:ipc" [2] "/dev/shm/aeron-sfokin/publications/2.logbuffer"
[2213075.196227] DRIVER: CMD_IN_ADD_PUBLICATION [56/56]: 54321 aeron:ipc [5:6]
[2213075.196235] DRIVER: CMD_OUT_COUNTER_READY [28/28]: 5 41
[2213075.196239] DRIVER: CMD_OUT_PUBLICATION_READY [98/98]:<PHONE_NUMBER>:54321 38 -1 [6 2]
    "/dev/shm/aeron-sfokin/publications/2.logbuffer"
# close(p2)
[2213081.004854] DRIVER: CMD_IN_REMOVE_PUBLICATION [40/40]: 2 [5:7]
[2213081.004879] DRIVER: CMD_OUT_ERROR [185/185]: 7 3 (-3) unknown publication
[aeron_driver_conductor_on_remove_publication, aeron_driver_conductor.c:3578] unknown publication client_id=5 registration_id=2
# close(p1)
[2213099.045283] DRIVER: CMD_IN_REMOVE_PUBLICATION [40/40]: 2 [1:8]
[2213099.045286] DRIVER: CMD_OUT_ON_OPERATION_SUCCESS [24/24]: 8
# exit()
[2213112.619825] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [5:9]
[2213112.619828] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [3:10]
[2213112.619829] DRIVER: CMD_IN_CLIENT_CLOSE [32/32]: [1:11]
[2213113.433502] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213113.433506] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213113.433508] DRIVER: CMD_OUT_ON_UNAVAILABLE_COUNTER [28/28]:
[2213114.434120] DRIVER: REMOVE_PUBLICATION_CLEANUP [41/41]: sessionId=2069360235 streamId=54321 channel=aeron:ipc

Would it be possible to reproduce this using the Java, C++ or C client and post a full code example?  We don't currently support the Julia client.  There is not enough information to determine if the issue is with the client or driver.
It is also not a typical approach to create a new client connection to the driver for each publication and subscription, generally a single client per process is the recommended approach.

Looks like a bug in Julia AeronClient library indeed.


[osgearth] Fix core installation

Test/fix core w/o blend2d. Needed for #31084.

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks!


Insert Curved Text

Hello there,
Is there a way for me to insert curved text into the image?
i am not sure that there is a function to make that but can i do it some other way like ... add text in image and then curve that image or something ... is that possible ?

I guess by curved text you mean text following a curved line. That is not possible, sorry.


Peer Endpoint Allowed IPs under Settings only allow /0 /8 /16 /24 /32 but not e.g /13

Describe The Problem
I want to route only private networks like <IP_ADDRESS>/13 but the dashboard do not allow that
Expected Error / Traceback
Peer Endpoint Allowed IPs Format Incorrect. Example: <IP_ADDRESS>/32 or <IP_ADDRESS>/32,<IP_ADDRESS>/32

Solution
Allow every suffix

I will add all other prefix in the next bug fix.


As a player, I want inventory, So that I may manage my items.

As a player,
I want inventory,
So that I may manage my items.
Acceptance criteria

[ ]

Definition of Done:

[ ] All acceptance criteria are met.
[ ] Code is reviewed and approved.
[ ] Necessary tests are written and pass.
[ ] Documentation is updated, if applicable.
[ ] Feature is deployed.

Mockup

Wireframe

Closed by d9b272a62325c7bc8613515079531b5160e28cb8


Fixer output in separate diff split

I watched https://www.youtube.com/watch?v=uUyRP8jRtnk&t=18m20s (18:20) and like the idea that the fixed output is in a split from which I can just pull what I want/need.
Usa case: Not every fix is proper (lack of sniff rule or issue)
Sure I can edit the fixer standard, but seeing all changes in a separate split is a great feature especially when the code is long, below the fold. In diff split I can review the fixes.
The mentioned funcionality in the video is available at https://www.drupal.org/files/issues/vimrc-phpcbf-2462969.patch
The relevant code should be
    " Make a copy of the current buffer and hit it with phpcbf.
    let copy = tempname()
    execute 'w' copy
    echom (command . ' ' . copy)
    call system(command . ' ' . copy)
    " Open the Fixed and Beautified copy in diff mode. Let me see the code even
    " when I have pulled in changes.
    execute 'vertical diffsplit' copy
    wincmd p
    setlocal foldlevel=1
    echo 'foldlevel set to 1. [c and ]c jump to prev, next diff; do accepts a change.'

I could hack it for myself, but I'm currently not enough skilled in VimL and I'm interested in some community opinions. It also might be great feature to the fixers.
I don't know how to run ALE phpcbf fixer https://github.com/w0rp/ale/blob/master/autoload/ale/fixers/phpcbf.vim
How to achieve running it and outputting the the copied buffer

No. If you want a diff of changes to your code, use version control tools like git, and maybe one of the associated plugins.

For information on running fixers, read the installation notes for the relevant program, and the online documentation with :help ale-fix.


[GPO settings on what to log]

Something you might want to add or note about enabling "Advanced Audit Configuration" via gpo and what events at a min you should turn on.   Or don't need based on what sysmon pulls.
The only thing I worry about is if you had an issue with sysmon and you had no "Advanced Audit Configuration" might be hard to find issues.
Noticed this in my testing since I did not apply my normal gpo for configuring event logging.  (Did want to add anything that would conflict with LME)   Was looking for normal user login events, and they were missing.

I have noticed a lot of the forwarded events in Chapter 1 Files > lme_wec_config.xml appear to come from here:
https://docs.microsoft.com/en-us/windows/security/threat-protection/use-windows-event-forwarding-to-assist-in-intrusion-detection#appendix-e--annotated-baseline-subscription-event-query
I would expect using the audit settings on the same page under "Appendix A - Minimum recommended minimum audit policy" would get you close to having the relevant event logged:
https://docs.microsoft.com/en-us/windows/security/threat-protection/use-windows-event-forwarding-to-assist-in-intrusion-detection#appendix-a---minimum-recommended-minimum-audit-policy

Closed due to project archive


fix: show ErrorSnackMessage in EventDetailsCoordinatorLayout

Fixes #1553
Changes:
Change the targetView of SnackBar to EventDetailsCoordinatorLayout to show the snackMessage without Scrolling
Screenshots for the change:

@ShridharGoel  @liveHarshit  please review

@liveHarshit  @ShridharGoel  could you review and help with codacy issue

resolve conflicts

@anhanh11001 @angmas1 Peer review please

@liveHarshit @iamareebjamal updated


python310Packages.oralb-ble: 0.14.3 -> 0.15.0

Diff: https://github.com/Bluetooth-Devices/oralb-ble/compare/refs/tags/v0.14.3...v0.15.0
Changelog: https://github.com/Bluetooth-Devices/oralb-ble/releases/tag/v0.15.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 209576 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.oralb
python310Packages.oralb-ble
python39Packages.oralb-ble


support appoint local freeline.zip

if your because the network can not download the freeline-version.zip in initFreeline process,
you can use command like:
gradlew.bat initFreeline -PfreelineZip=zipname
to appoint localfreeline.
The zip must be have in root project.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.lxl刘祥磊 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

@aa5279aa hello，需要先同意一下那个协议。
然后这次的commit记录确实有点问题...重新提个PR吧，这是一个会产生冲突的PR..最好能够提供一个干净一点的~


Update install-kubectl-linux.md

adding kubectl  to  PATH  ,  since kubectl  is copied to  ~/.local/bin/kubectl  directory PATH  should be set to ~/.local/bin/kubectl

This PR is similar with #32308, and the error in that page will be fixed in #32343.
/close


Translate heading and subheading of setup/independent/install-kubeadm/ in Japanese

This is a...

[x]  Feature Request
[ ]  Bug Report

Problem:
No heading and subheading of setup/independent/install-kubeadm/ in Japanese.
Proposed Solution:
Translate heading and subheading of setup/independent/install-kubeadm/ in Japanese.
Copy from en content and Update content/ja/docs/setup/independent/install-kubeadm.md.
Page to Update:
https://kubernetes.io/ja/setup/independent/install-kubeadm/

/remove-lifecycle stale

/assign

/close


npm error code EUNSUPPORTEDPROTOCOL

Describe the bug
node-v20.14.0
npm error code EUNSUPPORTEDPROTOCOL
npm error Unsupported URL Type "link:": link:./src/types
Reproduction
https://registry.npmjs.org/vite/-/vite-5.2.13.tgz
Steps to reproduce
npm install --prefer-offline --no-audit --global-style --install-strategy=shallow --no-save --omit=dev --no-package-lock --ignore-scripts
System Info
none

Used Package Manager
npm
Logs
npm error code EUNSUPPORTEDPROTOCOL
npm error Unsupported URL Type "link:": link:./src/types

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I created a Vite project with npm create vite and ran that command, but I don't get the error? The link: is used in devDependencies only and shouldn't be parsed by npm.

I created a Vite project with npm create vite and ran that command, but I don't get the error? The link: is used in devDependencies only and shouldn't be parsed by npm.

@bluwy
patch
https://github.com/nxhack/openwrt-node-packages/blob/dev/node-vite/patches/001-use_file.patch

I don't do this
npm create vite

I just do this
npm install

Can you provide a reproduction of the issue? I can run npm install but there's no project for me to run in, so I used npm create vite. (The https://github.com/nxhack/openwrt-node-packages repo itself it not a repro)

@esaaprillia please create a new issue with a proper minimal reproduction, I don't think this is actionable as stated by @bluwy (the patch link shared above is also broken).


Ivan/remove tlc

Re-opened the already existing pull request that was closed due to   branch renaming

Re-using the accept from @andrey-kuprianov given on the original PR to merge this one


Huge memory leak with Jest/Enzyme after upgrading to react-modal 3.X

Summary:
For a while now, I've been unable to upgrade to react-modal 3.X in my React 16 project with Jest/Enzyme because when I run tests Node itself actually crashes and I usually have to take some fairly nuclear steps such as force-quitting the process in my activity monitor app in order to use it again. The latest 2.X release of react-modal is totally fine.
Steps to reproduce:

Upgrade react-modal from 2.4.1 to 3.X
Run npm run jest
First few tests may run, but eventually the process hangs, and eventually these errors start to spill out:

<--- Last few GCs --->

[46548:0x102802400]    43615 ms: Mark-sweep 1402.9 (1437.6) -> 1402.9 (1436.6) MB, 1477.5 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1477 ms) last resort GC in old space requested
[46548:0x102802400]    45072 ms: Mark-sweep 1402.9 (1436.6) -> 1402.9 (1436.6) MB, 1456.0 / 0.0 ms  last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x38a049725ec1 <JSObject>
    1: read [/path/to/project/node_modules/react-dom/cjs/react-dom-server.node.development.js:~2206] [pc=0x181429965313](this=0x38a08a383211 <ReactDOMServerRenderer map = 0x38a00fd8cc29>,bytes=0x38a0125029a9 <Number inf>)
    3: renderToStaticMarkup [/path/to/project/node_modules/react-dom/cjs/react-dom-server.node.development.js:2512] [bytecode=0x38a0...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/Users/username/.nvm/versions/node/v8.9.0/bin/node]
 2: node::FatalException(v8::Isolate*, v8::Local<v8::Value>, v8::Local<v8::Message>) [/Users/username/.nvm/versions/node/v8.9.0/bin/node]
 3: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [/Users/username/.nvm/versions/node/v8.9.0/bin/node]
 4: v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [/Users/username/.nvm/versions/node/v8.9.0/bin/node]
 5: v8::internal::Runtime_AllocateInTargetSpace(int, v8::internal::Object**, v8::internal::Isolate*) [/Users/username/.nvm/versions/node/v8.9.0/bin/node]
 6: 0x18142970463d

*Note that I've replaced the actual path and username in this snippet.

@diasbruno have you had any success replicating this bug? I'm still unable to upgrade at all. Using latest versions of enzyme, enzyme-adapter-react-16, react-test-renderer and of course react-modal.

This can be related to the environment the modal is been rendered src/components/Modal.js#L187.
It will just return null if it can not use the DOM.

But why would returning null result in a big memory exception? Returning null is a perfectly valid render return value.

It's possible that it is not clean up resources properly. Can you write a small app with test? It would be better to debug.

Here you go @diasbruno: https://github.com/ryami333/react-modal-memory-exception
This is a 'Create React App' web-app, where I've done the following:

Installed enzyme, react-modal and enzyme-adapter-react-16.
Added a  to App.js.
Tried to render the 'App' component in App.test.js.

Simply run yarn install && yarn test (or npm install && npm test I suppose) to replicate.

The modal seems to be rendered. I'll try to see what is going on.
./node_modules/.bin/react-scripts --inspect-brk test --env=jsdom --runInBand

This could be an issue with jsdom I was seeing a similar bug when running integration tests (no enzyme render). This problem went away when I removed jsdom from the configuration.

@rharriso Thanks for the info. I'll try to dig in.

Some resources to help tracking this issue:
https://github.com/felixge/node-memory-leak-tutorial
https://github.com/jsdom/jsdom/issues/1705

It seems to be stuck in an infinity loop on:
ReactDOMServerRenderer.prototype.read = function read(bytes) {
  // bytes = Infinity
  ...

  var child = frame.children[frame.childIndex++]; // always $$typeof: Symbol(react.portal)
  ...
}

Well...you're test has passed. :)
 PASS  src/App.test.js (347.669s)
  â renders without crashing (342764ms)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        348.978s, estimated 1404s
Ran all test suites related to changed files.

Watch Usage
 âº Press a to run all tests.
 âº Press p to filter by a filename regex pattern.
 âº Press t to filter by a test name regex pattern.
 âº Press q to quit watch mode.
 âº Press Enter to trigger a test run.

Try to apply this patch on your project and tell me your results.
diff --git a/react-dom-server.node.development.js b/dd.js
index 2039700..d4832da 100644
--- a/react-dom-server.node.development.js
+++ b/dd.js
@@ -2227,7 +2227,8 @@ var ReactDOMServerRenderer = function () {
         }
         continue;
       }
-      var child = frame.children[frame.childIndex++];
+      frame.childIndex++;
+      var child = frame.children[frame.childIndex];
       {
         setCurrentDebugStack(this.stack);
       }

patch -p1 < patch.diff

node_modules/react-dom/cjs/react-dom-server.node.development.js

It's not a proper fix, I still need to investigate more, it's just to have an idea of what is going on...

cc @ryami333 Please let me know when you get a chance to try this patch.

@diasbruno I tried this out on my test suite. And it works!

@rharriso sorry, forgot to mention you. Thanks for trying it.

After some investigation, I reached the same state as facebook/react/issues/11565.

@diasbruno that patch works for me too! Does this mean you will create a PR for react itself?

Actually, it's not a fix. The things I wrote here are just for logging. That is really incorrect since it will skip the frame.children[0]. So, I'll still debugging this issue.
WARNING: it is not known the side-effects of this patch. :)

@ryami333 @rharriso Did you applied the patch on a real project?

Yes I did, one with over 200 render tests.

Interesting. I'll continue investigating to find the problem (if possible).

Sorry, I should qualify my response a little bit - the tests don't all pass, but they no longer crash.

@diasbruno, yes. It's a private company project with 600 tests

However, the error was see in our integration test suite, which renders with ReactDOM, not enzyme

@rharriso so you are using some kind of browser?

No, these tests just make sure the page responds with 200.
we injecting a request into the Hapi server, and looking at the response code.

There is a second patch I made, it should work with react-test-render (I still need to test it).
I'll publish on a fork of react.

@diasbruno  Let me know what you want me to test out.

The potential bug is:
while (React.isValidElement(child)) {. If the child has type react.portal,  React.isValidElement(child) will return false. Then it will return {child, context};.
So, it will end up in an infinity loop ({child: nextChild, context} = resolve(child, context));.
Note that in the current revision, it seems to have a fix $$typeof !== REACT_PORTAL_TYPE.
I ended up with this code before I looked the current version of react:
function isPortalElement(object) {
  return typeof object === 'object' && object !== null && object.$$typeof === REACT_PORTAL_TYPE;
}

  if (React.isPortalElement(child)) {
    child = null;
  }

  return { child: child, context: context };

@rharriso @ryami333 If you can't upgrade your react version, let me know so I can provide the patch to this version.

@diasbruno I probably can't this week. But probably soon after, I'll get back to you next week though!

@diasbruno - what do you mean by upgrading React? I've been experiencing this on the latesr version of React (16.2.0)

You are right. Totally forgot I was on the master branch.
I'm trying to finish the patch to submit to react.
The patch is available here.

@diasbruno Can you advice the best way to avoid this problem for now (until fix merged into next version of react)?

@asborisov I avoided this by removing jsdom from our integration tests. That may not be an option for unit tests though.

@asborisov @rharriso Follow the react issue I've mention here. Many ideas/workaround have been publish there.

I downgrade to 2.4.2 and looks it work now. Will wait until correct fix is merged


Upgrade babel

Pull Request Test Coverage Report for Build 363

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 94.783%

Totals

Change from base Build 359:
0.9%

Covered Lines:
80

Relevant Lines:
81

💛  - Coveralls


Istio Not Setting 'x-b3-sampled: 1' when sampling

Describe the bug
When Istio performs a trace sample, based on sampleProbability it always informs the receiver to not trace via x-b3-sampled: 0. This leads to incomplete traces as the receiver is never informed to perform trace.
Expected behavior
Client sends:

x-b3-sampled: 0: Istio and underlying receiver do not trace,
x-b3-samples: 1: Istio and underlying receiver do trace, ⚠ Istio ignores the flag, although it should potentially override it to avoid resource over-utilisation.
x-b3-sampled not set, Istio sampleProbability returns false: Istio and underlying receiver do not trace, ⚠ x-b3-sampled: 0 is sent regardless which is potentially an issue,
x-b3-sampled not set, Istio sampleProbability returns true: Istio and underlying receiver do trace, ❗ This is the case that is failing causing incomplete traces as x-b3-sampled: 0 is always sent.

Therefore, unless x-b3-sampled is specified by the client, if Istio decides to sample the trace it should inform the receiver to also sample.
Steps to reproduce the bug
GKE with Istio and Stackdriver tracespan configured (based on stackdriver.yaml) and underlying receiver using OpenCensus B3 Propagation.
Version
1.1.0-snapshot.6 (although this appears to be in 1.0.6 also)
Installation
Helm installation on GKE using Stackdriver tracing
Environment
GKE 1.11.7-gke.6
Cluster state
N/A

BTW, I understand that it's not good practice to allow the client to dictate the span behaviour as that could lead to increasing costs on the back-end and potentially hampering performance. However, Istio should be trusted to make a span sample decision which propagates.

Is there confusion here between tracing w/ the Stackdriver adapter, which has its own sampling impl, and the Envoy-generation of sampling headers?
The Stackdriver adapter handling should not impact the header propagation throughout the system.

@kyessenov would it be possible to use the route directive work to add headers based on adapter sampling decisions?  this probably isn't ideal, but might help in the case when envoy sampling and mixer adapter-based sampling diverge.

Not sure if it's related, but X-Request-Id also appears to not be propagated from the client and a new one created and overriding (#10875). This superficially looks like X-B3-Sampled: 1 from the client also being overridden.
Therefore, I'm wondering there are two issues here: the Stackdriver adaptor not setting the value when it samples and Envoy overriding it further down the chain.

Apologies, I think I've conflated with the X-Request-Id thing. It appears that X-B3-Sampled is always passed down to the underlying service via Envoy.
So, back on topic... I tried disabling all other tracers, except Stackdriver. I set:

Stackdriver trace.sampleProbablity: 0 - 0%
pilot.traceSampling: 10 - 10%
Now the X-B3-Sampled is correctly being set 10% of the time, but no data is now going into Stackdriver (expected for Istio, but not expected for underlying OpenCensus unless Stackdriver is not finding a valid x-parent-trace-id and throwing it away?!).

I also tried setting the default values:

Stackdriver trace.sampleProbablity: 0.1 - 10%
pilot.traceSampling: 1 - 1%

Now the app level tracing appears, stackdriver appears at 10% and X-B3-Sampled is set at 1% inline with the pilot config. However, the traces are still somewhat disjointed.
Therefore, I feel @douglas-reid is right that the Stackdriver adaptor should be setting the X-B3-Sampled header when it decides to sample, Pilot/Envoy and everything downstream will then just honour that.
At this time, I feel the only way we can get reliable traces is to set: 100% on trace.sampleProbablity, pilot.traceSampling and underlying service which won't scale well.

@dansiviter i think the correct "fix" involves a code change to enable the stackdriver adapter to respect sampling decisions made by the proxy -- and to control sampling via pilot.traceSampling. Mixer telemetry adapters don't have a way to set headers in requests -- and adding a sampling policy adapter just to sets headers doesn't seem like the best approach.
I'm gonna take a look at how to modify the Stackdriver adapter code / config now.

@douglas-reid I wasn't sure if that was possible but if sampling configuration is in one place then that's even better. I see there is some chatter around integration of OpenCensus into Envoy (envoyproxy/envoy#2456) which may help cover differed sampling types in the future too.

@dansiviter another possible workaround. hoping you can maybe test it out for me:

Set the handler config for the Stackdriver adapter to have a sampling rate of 1 (always sample).
Update the match condition on the handler to be:

(context.protocol == "http" || context.protocol == "grpc") && request.headers["x-b3-sampled"] == "1"

IMHO it sounds to me that the sampling decision needs to be made at the proxy if it were to propagate. Telemetry adapters run out of sync, and it's too late to force tracing from them. Therefore, we should probably recommend sampling rate is 100%, and let the proxy control sampling.
In the future, sampling could be made first-class config, so that this config can be acted upon at the proxy.

@dansiviter I tested the config in my own istio-on-gke cluster and it seemed to do the trick. I think that is what we should recommend moving forward.

@douglas-reid Yea, workaround does the trick for me too. Thanks

@dansiviter I think this issue has become one of documentation. I'm going to close this for now. Please feel free to re-open if you feel that is more appropriate.

@dansiviter another possible workaround. hoping you can maybe test it out for me:

Set the handler config for the Stackdriver adapter to have a sampling rate of 1 (always sample).
Update the match condition on the handler to be:

(context.protocol == "http" || context.protocol == "grpc") && request.headers["x-b3-sampled"] == "1"

please make sure it is using string "1" for the workaround .....


Add GetStringFromUTF8 and GetStringFromASCII to C#

Adds these methods to C#. Test code:
GD.Print(new byte[] { 65, 66, 67 }.GetStringFromUTF8()); // Prints ABC

Thanks!

Nice, will update the docs PR soon :D

Cherry-picked for 3.2.4.


Can't compile aztec_backend_wasm  from mv/to_bytes branch

Unable to compile aztec_backend_wasm  from mv/to_bytes branch. Seems CRS download logic to tangled into barretenberg_wasm to simply disable.

This isn't an issue as it doesn't handle downloading the CRS anymore.


Auto Materialization of source tables

Feature Description
Ability to get tables directly from source, more concisely, for SQL based databases
connection.yaml
name: prod
type: snowflake
options:
...

tables:
- orders
- reviews

or even
name: prod
type: snowflake
options:
...

tables:
- *

Goal of Feature

Faster DX when prototyping
The ability to more quickly specify sources in SQL connectors
faster visibility of the columns in your source database tables

Current Solution / Workarounds
sources/orders.sql
select * from orders

sources/reviews.sql
select * from review

Considerations

This would automatically do a select * which may be undesirable
Especially tables - *

Should likely limit the content also or this is a potential to blow up huge tables, or cost a lot.
Eg, imagine the costs of running this on a bigquery db with billions of rows. Could be quite expensive queries.
If you use datagrip or some other editor they usually limit to some fixed size (500 rows) for you automatically behind the scenes when you do stuff.
You could have a default limit, that is override-able
tables:
- orders
  - limit: 500
- reviews
  - limit: 0 #unlimited rows

Agree that this could be dangerous and we should think about how not to destroy people's DBs
However also automatic (unspecified) limits can also be confusing and lead to errors in analysis
(Easy to miss a dropped row if there were 505 rows in the table, and you excluded the last 5)
An alternative would be to refuse to run tables above x rows (eg x=100,000) unless the user specifies limit=0 (or rows=all or similar)

Although now I think about it, that would require a count(*) operation on the table which is sometimes (depending on DBMS) an expensive operation


Fix typo in managing-your-profile

Why:
misspelling

What's being changed (if available, include any code snippets, screenshots, or gifs):
"contro" changed to "control"

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Hi! We actually don't accept PRs on the translations. They're handled by an automation pipeline and we just use the git repo (at the moment) as our "database".


Support for JSON rendering of maps

As observed in #272 it seems like we cannot yet support serializing Map<String,Object> to JSON, even if the values are simple primitiveish types (like String). It would be useful to work out what the barrier is. Probably just some reflection configuration on HashMap or similar.

For the case I'm aware of - with the diskSpace endpoint - I've fixed it so the native-image returns the same thing. The issue was reflective access to the Health class that contains the details map. I guess that will fix many cases where the details are being missed.

Based on @aclement comment I think this is fixed please reopen if that's not the case.


fix: update to eye v11.2.5

fix: update to eye v11.2.5

:tada: This PR is included in version 18.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v18.1.2

Your semantic-release bot :package::rocket:


type(an error) should return the string "error"

Describe the bug
type(value) on a column with an error stored as a value raise the error instead of returning the string "error".
type() should never result in an error. At the worst, it should return an empty string or undefined (as it does now, most of the time). This is a utility function, and as such, it should be robust.
Current code (probably since a lots of years…):
https://github.com/OpenRefine/OpenRefine/blob/d7d567439eb39957e149e1b67576fe67c7836a10/main/src/com/google/refine/expr/functions/Type.java#L51-L66
To Reproduce
Steps to reproduce the behavior:

Go to 'Edit Cells -> Transform'
Type 'type(parseJson("a"))'
See error and not "error"

Current Results
An error is raised, interrupting the process.
Expected behavior
I think type() should also intercept errors like this (where value is a previously stored error):

Expression
Result
Expected result

value.type()
raise the error IF value is a stored error
"error"

type(parseJson("a"))
raise the error: parseJson failed: Unrecognized token 'a': was expecting ('true', 'false' or 'null') at [Source: (String)"a"; line: 1, column: 3]
"error"

type(vfdsf)
"undefined"
"undefined"

Screenshots

OpenRefine (please complete the following information):

Version 3.3

In the PR « Fix the true.type() == "boolean" #2586 », @wetneb noted that type() being a function, it will never have the opportunity to inspect its arguments if they contain errors. He suggested it « might » be converted to a control.
But, @tfmorris said that this choice would be breaking, as the Dot Syntax functionality would no longer work.
So, the idea of extending the Dot Syntax for control is now discussed (Extending the dot syntax for controls #2618). This would open a non-breaking path to this issue, by transforming it into a control, while still retaining its Dot Syntax functionality.
Regards, Antoine

Evaluation of parameters happens before the type() function gets control, so it's never going to see the error. Also, an error isn't a data type. Data types, which is what type() returns are things like String, Date, etc.
If you want to know whether an expression will result in an error (or have a cell which contains an error, rather than a value), use the isError() control. Because it's a control rather than a function, it doesn't have its operands evaluated ahead of time.


Schema Versioning Support

Requirements
As a user of the Guardian, there could a situation when a schema is already used in a policy, but I need to edit the schema. There may be new requirements, issues, etc. Therefore, we will add the functionality of schema versioning after finishing issue #45.
Pre-requisite
Cross blockchain, cross-database, cross schema Unit Testing
Definition of Done
Unit Test results
Acceptance Criteria:
Analyzing Unit Test Results

The mentality as per @sergmetelin and @danielnorkin  is that the history needs to be preserved. The verified credentials need to be given and connected to a specific version of the policy, not just TO the policy. We need to know that when the token is minted, it has been minted according to a certain version of the policy so that we can track it back. Everything needs to be linked to a version. This is not really about the version of the schema but the version of the policy. Next step is @Pyatakov  to analyze.

@anvabr We need some assistance flushing this ticket out. We will add to a later Sprint.

@anvabr Can we please update this? This is important to have updated ASAP with a timeline. cc: @danielnorkin @envisionblockchainpm

@anvabr @envisionblockchainpm can we please get an update on this?

Andrey needs to talk more with the Hedera Team as we need additional Discovery after just bringing this up on our Scrum. Sorry for the delay for some reason I hadn't seen this status request from you or Wes.

Step 1: @sergmetelin As discussed today with @anvabr on our call today, the next step is for @Mpinnola to find an example of a version update related to Verra/REDD and present notes in this ticket.

To summarise the main topics of our discussion on this and unanswered questions:

introducing versioning into the schema/policy syntax is relatively easy, it's just another line in the file.
the main difficulty is to design how different version of the schema/policy should be used/treated by the system.

I feel schemas/policies should be immutable, i.e. each 'version' of the policy is really a new policy self-sufficient artefact (similar to how smart contracts work in the traditional blockchains) referencing the previous artefact with the same name. Otherwise if we truly change the policy file (and change a version) we would lose the information about the previous version. Unless we somehow record the diffs, but that's more difficult to implement. But there are may be other consideration, @sergmetelin please comment?
what is the business logic wrt different version of schemas? does the old version gets obsolete immediately when a new one becomes available? Would the longer running project be assessed on the basis of the schema/policy it was created against, or would it have to comply with the newer version? Are token produced by a different versions of the same schema completely equivalent or should they be distinguishable? @Mpinnola.
also please see #85 which raises broader issues of difference and equivalence of versions, schemas and policies. @sergmetelin

@anvabr To provide some context, for the VCS Standard:

VCS Program editions are labeled with a version number and program documents are correspondingly version controlled
When a new version is released it's requirements take immediate effect unless specific grace periods are set for specific requirements
Program documents have a history in the appendix with previous versions, updates, grace periods, etc.
Example: VCS v4 was released 19 Sep 2019, and updated 9 Mar 2020. Registered projects and projects that complete validation on or before 19 March 2020 remain eligible to apply the crediting period requirements under VCS Version 3.
Project proponents will be informed of the updates and the updates will also be catalogued on the Verra website
Project proponents are responsible for using the most up to date versions
Verified carbon units (VCUs) are not labeled in the Verra registry with a specific version of the VCS Program
Methodologies, modules, etc. are also periodically updated
Upon renewing a monitoring period, project descriptions and baseline assessments shall be updated with the latest methodologies
In general, new requirements go into effect immediately unless otherwise specified.

I think we need to brainstorm the Guardian implications, but here are just some initial thoughts to start the conversion. Let's imagine a Verra REDD project becomes validated and verified under the methodology VM0007 REDD v1.6 and issued VCUs. Then they renew their monitoring period and during the subsequent monitoring period v1.7 is issued and requires a few new data points, effective immediately. A new policy (v1.7) will be created and the project would have to switch to the new policy, which may require slight changes, or no changes at all in certain contexts. Then at the end of the second monitoring period, the updated project description and monitoring report will be submitted to the VVB, in accordance with v1.7. The VCUs will all be the same (not labeled to a  specific version). Although the policy and schema may be 99% or even 100% the same in certain contexts, v1.7 will be treated as its own separate policy. A history of all monitoring periods, versions, etc. will be retained. The new policy will also include a history of versions, updates, grace periods, etc.
How does that sound? Just some initial thoughts, interested to hear yours.


Need help with external module "redcap-em-export-repeating-data"

Hi,
I am trying to link a repeating form within another repeating form (scenario 2 described in the text). I link form 3 with form 2 with the action tag FORMINSTANCE and INSTANCETABLE_REF on the form 3 (on the variable which identifies the form 2), but the result does not provide the desired output. Is there any contact through whom I can get help in using external module "redcap-em-export-repeating-data"? It would be great if I can get some! Thank you very much!

I can take at look if you want to send your data dictionary for these two forms.
Eileen
On Aug 2, 2022, at 3:09 AM, BPG-2010<EMAIL_ADDRESS>wrote:
[EXT]
Hi,
I am trying to link a repeating form within another repeating form (scenario 2 described in the text). I link form 3 with form 2 with the action tag FORMINSTANCE and INSTANCETABLE_REF on the form 3 (on the variable which identifies the form 2), but the result does not provide the desired output. Is there any contact through whom I can get help in using external module "redcap-em-export-repeating-data"? It would be great if I can get some! Thank you very much!
—
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsusom%2Fredcap-em-export-repeating-data%2Fissues%2F2&data=05|01|eloh%40stanfordhealthcare.org|24f6c85f2e194fc47edf08da746f2ddf|9866b506dc9d48ddb7203a50db77a1cc|0|0|637950318107177354|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=grGzD6wrVRfNe9aRmZ6He7J%2BtXUDK0Wn2S05W9q75ME%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEEKTSCW5DL5HDFLUINBIWLVXDXXLANCNFSM55KR7YCQ&data=05|01|eloh%40stanfordhealthcare.org|24f6c85f2e194fc47edf08da746f2ddf|9866b506dc9d48ddb7203a50db77a1cc|0|0|637950318107177354|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GJG36TN4Z17f6ZorS8OA5f2YYB838YEQfBeYE5gkXNA%3D&reserved=0.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you very much for the offer, Eileen! Recently I got to fix it myself, as it was due to some sort of error in the mentioning of FORMINSTANCE!


⚠ Guya.moe Proxied has degraded performance

In 7b7f537, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 13008 ms

Resolved: Guya.moe Proxied performance has improved in e878949 after 3 hours, 48 minutes.


If upgrade is not available on a cluster, '--output table' with produce error "Table output unavailable."

Under the heading 'Check for available AKS cluster upgrades', the command suggested is:
az aks get-upgrades --resource-group myResourceGroup --name myAKSCluster --output table

If there is an upgrade available, the table shows as expected and shown in the documentation.
If no upgrade is available, you will get:
Table output unavailable. Use the --query option to specify an appropriate query. Use --debug for more info.

The documentation should mention this to avoid user confusion.

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

ID: 7010bd58-42b1-5c90-3253-05deed855be8
Version Independent ID: c890efb2-cdaf-a3eb-d54f-0e7ccbd4314d
Content: Upgrade an Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/upgrade-cluster.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! We are currently investigating and will update you shortly.

I have created a PR with the updates and changes will be live shortly. https://github.com/MicrosoftDocs/azure-docs-pr/pull/88719
Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

I'm not sure this is true.  I get the error yet I have two upgrades listed in the JSON output (leave off the --output parameter).  The two upgrades also show in the Azure Portal Kubernetes Service.  See my output below
wma@Wess-MacBook-Pro ~ % az aks get-upgrades --resource-group aksResourceGroup --name wmAksCluster --output table
Table output unavailable. Use the --query option to specify an appropriate query. Use --debug for more info.
wma@Wess-MacBook-Pro ~ % az aks get-upgrades --resource-group aksResourceGroup --name wmAksCluster
{
"agentPoolProfiles": null,
"controlPlaneProfile": {
"kubernetesVersion": "1.13.12",
"name": null,
"osType": "Linux",
"upgrades": [
{
"isPreview": null,
"kubernetesVersion": "1.14.8"
},
{
"isPreview": null,
"kubernetesVersion": "1.14.7"
}
]
},

The error "Table output unavailable." still exists. Please reopen this documentation/cmdlet issue.


Added Missing Curly Braces in OnActivityResult

There are a couple of missing closing curly braces in the OnActivityResult method. This includes adding the closing one around the else statement, which appears to be consistent with other code sampes within the repository.

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


Populate Key Value pair for json files

Is there a way I get snippet trigger when type {} while editing json file in nvim?
        {
          "key": "value"
        },

Are you trying to trigger a particular snippet are just wanting the curly braces to expand?

Closing since no further discussion has happened. Feel free to reopen if you still need assistance :sunglasses:


Cleanup

Cleanup of various things:

Uses setupbase from jupyter/jupyter-packaging.
Uses @jupyter-widgets/base instead of jupyer-js-widgets (requires ipywidgets 7.0.0a4).
Tries to clean up pure model classes to not have views, vs DOM widgets.
Various tidying.

I still get some errors in the web console when I run unray.render_single_tetrahedron(), but at least they seem to all be normal errors.

Great, I'll check it out once I manage to install ipywidgets again. There may be bugs in the code committed during the weekend, the code from Friday should work.


How do I grant a pipeline permissions to deploy to AKS from CLI?

This web page describes the problem that I am facing (see below), but it proposed to solve this via the WebUI.  In today's automated world, I need a way how to do this from bash ("e.g. az devops...." command)
Problem:
Why do I see the message that I need to authorize a resource before the run can continue?
Expected solution:
some CLI commands

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

ID: 6f0ae7d0-e5be-03c9-5296-95a10984a795
Version Independent ID: 12d1cbdc-49ab-f4dd-2b21-1a1953bb1357
Content: Set pipeline permissions - Azure Pipelines
Content Source: docs/pipelines/policies/permissions.md
Product: devops
Technology: devops-cicd-policies
GitHub Login: @juliakm
Microsoft Alias: jukullam

@cvdabbeele Here's all the available pipelines Azure CLI commands: https://docs.microsoft.com/en-us/cli/azure/pipelines?view=azure-cli-latest. My understanding is that setting complex permissions with Azure CLI is not currently supported. For feature requests, please post in the Developer Community.

@juliakm thank you for your answer
This is really not clear in the documentation.  The docs always point to a GUI
Meanwhile a colleague helped me out.  I'm posting the answer here.
Hopefully this can help anyone else with the same issue
To allow a pipeline to deploy to AKS, I had to use the following command:
az devops invoke --area pipelinepermissions --resource pipelinePermissions --route-parameters project=${APP_NAME} resourceType=environment resourceId=${ENVIRONMENT_ID} --in-file params.json --http-method PATCH --api-version "6.0-preview"
with params.json:
{
      "pipelines": [
          {
              "id": "${PIPELINEID}",
              "authorized": "true"
          }
      ]
 }

and to get the ID of the environment:
az devops invoke --area distributedTask --resource environments --route-parameters project=${APP_NAME} --org ${DEVOPS_ORGANIZATION} --api-version "6.0-preview" -o json | jq -r --arg PROJECT_ID ${PROJECT_ID} '.value[] | select(.project.id=$PROJECT_ID) | .id'


django 1.10 adaptations

This change set should have gone into https://github.com/tabo/django-treebeard/commit/2edb72dfef9a732b244056b392f7856691333f63
For me, otherwise pip install fails over with:
(nexchange) olegs-mbp:nexchange beoleg$ python manage.py migrate --settings=nexchange.settings_test
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/core/management/__init__.py", line 367, in execute_from_command_line
    utility.execute()
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/core/management/__init__.py", line 341, in execute
    django.setup()
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/__init__.py", line 27, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/apps/registry.py", line 115, in populate
    app_config.ready()
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/contrib/admin/apps.py", line 23, in ready
    self.module.autodiscover()
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/contrib/admin/__init__.py", line 26, in autodiscover
    autodiscover_modules('admin', register_to=site)
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/django/utils/module_loading.py", line 50, in autodiscover_modules
    import_module('%s.%s' % (app_config.name, module_to_search))
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 986, in _gcd_import
  File "<frozen importlib._bootstrap>", line 969, in _find_and_load
  File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 665, in exec_module
  File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
  File "/Users/beoleg/virtualenvs/nexchange/lib/python3.5/site-packages/treebeard/admin.py", line 6, in <module>
    from django.conf.urls import patterns, url
ImportError: cannot import name 'patterns'

You doing this wrong. You just complitely drop support of older Django version.
You can't broke that already working (this is universal rule).

It should be made compatible with django 1.8+ (LTS).
Previous versions are not supported any more.
https://www.djangoproject.com/download/#supported-versions
(URL patterns are already deprecated in 1.8)
https://docs.djangoproject.com/en/1.10/releases/1.8/#django-conf-urls-patterns

To avoid waste your time for nothing. Looks #33
This patch do nothing.

Thanks very much
Le 29 sept. 2016 14:47, "Vladimir Osintsev"<EMAIL_ADDRESS>a
écrit :

To avoid waste your time for nothing. Looks #33
https://github.com/tabo/django-treebeard/pull/33
This patch do nothing.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tabo/django-treebeard/pull/36#issuecomment-250455884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD6lrtj0TiD1jlpFB_w8Fk-623M0BSMGks5qu7NfgaJpZM4KGBwA
.

@BeOleg #33 Is pretty solid - we are running production with it and have only run into 1 issue with MoveNodeForm which has been merged into #33.
Unless this is adding additional fixes - I think closing and supporting #33
Depending on your requirements and deployment setup, the safest way is to download a zip of #33 from https://github.com/fladi/django-treebeard/tree/django-1.10, bump the version number in setup.py (so something like heroku will realize the change) and reference in your requirements file like:
./deploy/requirements/django-treebeard-4.0.3unofficial.zip
@osminogin Is correct - the more people running against that branch and supporting that PR, the more bugs will be found, and the less PRs with overlap (suggest closing this one) the faster this can get merged when @tabo has availability.

Thanks but we're still supporting old versions of Django. Please take a look at #33


"What's New" for #3762 (unpinning Cartopy, Matplotlib, Proj)

#3762 was created before the ongoing revamp of the documentation, so does not include the architecture for the current "What's New" process. There has been much discussion of the future complication that can be caused by rebasing/merging from master onto a PR, so I suggest that we instead merge #3762 then write the relevant PR after merging (when our changes will be present alongside the latest "What's New" architecture).
Note from the Contribution categories that the "What's New" should also include a section for internal changes such as the testing changes we are making have made.

ping @stephenworsley

It's important that this work also addresses https://github.com/SciTools/iris/issues/3759#issuecomment-664199458

Closed by #3811


Fix a regression introduced in 71535104.

Since STLINK V1 cannot read voltage, and attempting to do so breaks the
USB communications, this patch will revert the behaviour to just blindly
doing 32-bit writes on these programmers.

I'm not sure, the indenting seems a bit odd.

I think github's diff generator makes it look weird, I followed the existing convention of 4 spaces.  Clone my repo first and have a look at it in context if you like.

PS I did step through it in a debugger and it works for me as expected.

Crossref for issue #439

Sorry to do this, but I'm going to rebase this (used wrong email) and create another pull request.  It is identical save that.

Moved to #441.


Copy-Item does not understand symlinks

Steps to reproduce
~/test |-/ mkdir folder
~/test |-/ touch file
~/test |-/ ln -s folder validfolderlink
~/test |-/ ln -s file validfilelink
~/test |-/ ln -s foobarbaz brokenlink
~/test |-/ Copy-Item ./validfolderlink validfolderlinkcopy
~/test |-/ ls -l validfolderlinkcopy
total 0
~/test |-/ Copy-Item ./validfilelink validfilelinkcopy
~/test |-/ ls -l ./validfilelinkcopy
-rw-rw-r--. 1 andschwa andschwa 0 Jan  5 13:29 ./validfilelinkcopy
~/test |-/ Copy-Item ./brokenlink brokenlinkcopy
Copy-Item : Could not find file '/home/andschwa/test/brokenlink'.
At line:1 char:1
+ Copy-Item ./brokenlink brokenlinkcopy
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Copy-Item], FileNotFoundException
    + FullyQualifiedErrorId : System.IO.FileNotFoundException,Microsoft.PowerShell.Commands.CopyItemCommand

~/test |-/

Expected behavior

validfolderlinkcopy should be a copy of the symlink validfolderlink
validfilelinkcopy should be a copy of the symlink validfilelink
brokenlinkcopy should be a copy of the symlink brokenlink

Actual behavior

validfolderlinkcopy is a copy of link's target folder
validfilelinkcopy is a copy of the link's target file
PowerShell throws an error when copying brokenlink to brokenlinkcopy

Environment data

> $PSVersionTable
v6.0.0-alpha.14
> uname -nro
andschwa-centos 3.10.0-514.2.2.el7.x86_64 GNU/Linux

This is similar to #804 where New-Item did not understand broken symlinks, since it was trying to follow it. The semantics on Linux of copy/move/new is to treat the symlink as a normal file, instead of following it, but PowerShell does not do this.

/cc @paulcallen

I assume this is covered by other bugs or fixed; and if not, at this point it hasn't been a big bug I guess.

I am a bit confused by the resolution of this issue. I am trying to do the exact same thing, copy a symlink but I get the same behaviour and I can't find an option to preserve the links when copying an item.
Is this issue really solved?

@joeyaiello Can you triage this please?

@andschwa this is still an issue (at least for me on Windows)
When I use Copy-Item to copy a Symlink, it creates a new folder instead of creating a new Symlink with the same Target

I have a script like this
- name: Copy License File
shell: powershell
run: |
$source = "${{ runner.temp }}\E.lic"
$destinationDir = "${{ vars[format('EASY_HMI_DEPLOY_PATH_{0}', matrix.site)] }}"
$destinationFile = Join-Path -Path $destinationDir -ChildPath "E.lic"
Write-Host "Copying $source to $destinationFile"
Copy-Item -Path $source -Destination $destinationFile -Force
      instead of copying it the file, it just creates a folder called E.lic


Add more unit tests for libcnb_data::buildpack::api

The tests have also been changed to exercise parsing of TOML rather than just using BuildpackApi::from_str(), since:

previously the TryFrom and serde Derserialize implementations were untested (when they are actually what's used in the library), and this approach now tests both
IMO we should drop support for parsing strings directly (ie: remove the FromStr implementation in favour of just having TryFrom), which we can now do in a later PR without needing to adjust the tests.

GUS-W-10222615.

Can we use serde_test here? This would remove the need to the TestBuildpack struct and toml as we would test Deserialize with serdes data model. We already have an example how to do this here: https://github.com/Malax/libcnb.rs/blob/5e0d592860d6c6a3924ab2565b5f31c05b0c5498/libcnb-data/src/newtypes.rs#L206-L213

Can we use serde_test here?

It's a lot cleaner, and I would like to eventually switch to that, however trying now it doesn't play nicely with the shadow struct:
---- buildpack::api::tests::deserialize_valid_api_versions stdout ----
thread 'buildpack::api::tests::deserialize_valid_api_versions' panicked at 'tokens failed to deserialize: invalid type: string "1.3", expected tuple struct BuildpackApiUnchecked', /Users/emorley/.cargo/registry/src/github.com-1ecc6299db9ec823/serde_test-1.0.130/src/assert.rs:164:19

I was going to remove the shadow struct for the buildpack API case (since it's unnecessary) in a later PR - ok if I switch to serde_test then? I'd rather avoid making non-test changes at the same time as adding tests here.

@Malax Could you approve the PR? I've opened #203 (currently based on this branch, will rebase once this merges), which removes BuildpackApiUnchecked, which unblocks switching to serde_test.


Fix multiple interfaces for enum appearing from multiple references to it

Track enum declarations by schema type like it's done for object types.
Fixes #17 .

Is there anything blocking this pull request?


iavl: make Dump testable with Fdump

Make Dump testable by making Fdump that takes in
an io.Writer instead. Dump in the past always
pushed output to stdout so testing this was
impossible without hijacking os.Stdout.

Thanks!


Add additional check for ${loggedInUser}

Please add an additional check to populate the ${loggedInUser} variable before the script looks to see if the variable exists. This will ensure that before the check, a recent variable check has been completed. Thank you!
loggedInUser=$( echo "show State:/Users/ConsoleUser" | scutil | awk '/Name :/ { print $3 }' )
https://github.com/dan-snelson/Setup-Your-Mac/blob/f195cb4ad5b9727aaf2455d320e9049a5556ea77/Setup-Your-Mac-via-Dialog.bash#L188

Thanks, Obi-@robjschroeder!

@robjschroeder: I'm seeing loggedInUser defined at Line 51; what am I missing?

Please advise the steps to reproduce. Thanks.

When using an Automated Device Enrollment workflow and SYM, it looks like the $loggedInUser variable is not getting populated when the script first checks on line 51, so when the script checks for the variable to be populated and not loginwindow on line 189, SYM thinks there is no loggedInUser because the variable is blank. With the additional chance for the $loggedInUser variable to be populated right before it decides to continue or exit, you have at that moment a chance to grab the correct variable, whether it is truly blank or if there is a logged In user.
This issue only seems to affect those using SYM and ADE workflows on versions 1.7.2 +

Please see: https://github.com/dan-snelson/dialog-scripts/issues/41

@robjschroeder:
Please test-drive 1.8.1-rc1; I'd be interested to see the client-side logs.
Thanks.

Thanks, @robjschroeder.
Addressed in 1.8.1.


Add support for "hot-code replace"

Describe the feature
In Metals v1.3.0 there is now support for socalled hot-code replace, see https://scalameta.org/metals/blog/2024/04/15/thalium/#debugger-improvements.
It would be very nice to have this available in Neovim with nvim-metals!
See also: https://discord.com/channels/632642981228314653/632841743435563028/1277531516821311560
Potential ways to implement
No response

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

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

I've reported both @WSA1k and @llowvxe for phishing.

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

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

So finally getting around to look at this @lervag I'm not actually seeing this as part of the DAP spec. So this may be VS Code specific behavior. I don't see any mention of it in nvim-dap as well.

Ah, I see. Perhaps we should ask for more information on the Discord channel for metals on this particular feature?

So I'm not sure Metals is actually the correct place for the discussion. I'd do the following:

Double check the DAP spec to see if it's part of it
If it is, then nvim-dap would need to add support for it
If it's not, then nvim-dap would still need to build it from scratch to mimic VS Code

I'm trying to find a specification for this. It's hard! It seems to be a custom DAP extension, or something like that.
I've found a few relevant references:

https://github.com/scalacenter/scala-debug-adapter/pull/553
https://github.com/puremourning/vimspector/issues/252

Perhaps you are right that this is something that should be added to nvim-dap?

But it seems like this may be a more relevant reference in case we want to implement this into nvim-metals directly.

I asked for some more in one of the related issues and got a reply there. For convenience, I'm quoting it:

@levarg Our implementation is the same as https://github.com/microsoft/vscode-java-debug which does not seem to be documented anywhere.
Here is a basic description:

recompile the project
collect all classes that changed
in each thread, if there is a frame in a changed class, unstack it (and all frames on top)
ask the JVM to reload the changed classes: the spec of this depend on your JVM, most JVMs don't accept new methods in a class
step into once


Doc - Contributing

Purpose: Anybody new onboarding can find the instructions to run this project.
Documentation

[x] Make a table of contents near the top of our README.md with links to various documentation in our project

Contributing

[ ] Details about bv gov organization requiring signed commmits and how to enable them

@marcellmueller should all the acceptance criteria be checked off on this one?


REQ: Upgradable MSI

Hello,
Can you make the MSI installation to be upgradable instead of uninstalling and installing ?
Thank you

Done.  Fix for the installer is in.  Next release will have it.


Rename markup to html

I found markup to be a little bit of a tricky name for the gulp task, it took me a little bit of thinking to realise that it meant HTML markup. While I understand that markup is technically correct and encompasses other markup languages as well as HTML, IMO HTML is a better name.
At the least I would suggest a comment on the markup task to explain it. I'm happy to make a PR for either option if you like.

I see your point, but I think markup fits best since the markup file doesn't have to be HTML. For example, in my projects the markup task copies WordPress PHP files which hold the HTML markup with some PHP logic, thus markup fits better in my case. Even the PHP files hold HTML and PHP code, I would search for HTML files when the task reads HTML.
It could also be a handlebar, xaml, xml or different markup similar language.
Anyway the task does nothing more than copying files at the moment. It could also be named "copy", as most grunt repositories do.

All valid points. Yeah, the plan was to actually make that markup task do something more than copy, but I never got around to it. When I (or someone else) does, I'm fine with renaming the task to html.

All valid points. Yeah, the plan was to actually make that markup task do something more than copy, but I never got around to it. When I (or someone else) does, I'm fine with renaming the task to html. Whatever markup language you're using, it's going to get compiled to html. So it makes sense to me, but don't feel strongly about it.

Coming soon in: https://github.com/greypants/gulp-starter/tree/2.0


NC DeMasi's phone

Per Mr. Carlisis, troubleshoot sticky keys or replace

Good afternoon Mr. DeMasi,
Mr. Carlisi let me know about your sticky keys on your old phone. We recently received a couple new backup telephones Mr. Steer ordered for this very purpose, so yours was replaced today. Note that the ‘virtual voicemail box’ follows the equipment so your voicemail needs a simple setup. Simply dial 1000# from the phone and it will talk you through a basic setup. If you want assistance with that in the morning, just give me a ring. Happy to help.
My Best,
Greg Abbott
State Assessment Coordinator & Technical Support
Mason Classical Academy Junior & Senior High School
<EMAIL_ADDRESS>


Reduce memory utilization in downstream projects creating multiple Alertmanager instances

This PR is a follow up of https://github.com/prometheus/alertmanager/pull/3092 (see rationale explained there, please).The PR 3092 reduced Mimir Alertmanager by -50%, and this last PR has the potential to reduce another -50% on top of it (see https://github.com/grafana/mimir/issues/3273).
The AlertmanagerAPI.Context() calls middleware.NewRoutableContext() passing the spec which then is analysed:
https://github.com/go-openapi/runtime/blob/dc39dfcab95e469a2034283ec1bc196ff94f09ac/middleware/context.go#L193-L197
I've added a new function NewRoutableContextWithAnalyzedSpec() to go-openapi/runtime (see: https://github.com/go-openapi/runtime/pull/258). This PR builds on that and adds a singleton cache for the analyzed spec too.
Since the file api/v2/restapi/operations/alertmanager_api.go is generated from swagger specs and we can't edit it, I've changed the code to manually create the swagger API client context, passing the analyzed spec.

I totally missed that api/v2/restapi/operations/alertmanager_api.go was auto-generated by swagger. Let me try a different option.

I totally missed that api/v2/restapi/operations/alertmanager_api.go was auto-generated by swagger.

I pushed a change: https://github.com/prometheus/alertmanager/pull/3114/commits/289b26e99d400fa074518aa61df8480dfda423ee


carthage 1.0.4

Make platform an input param, letting carthage to build for Mac as well.

Perfect, thanks for the contribution! 😉🚀


conda querynator package not found

Description of the bug
the github-action test nf-core CI / Run pipeline with test data (21.10.3) fails
the github-action test nf-core CI / Run pipeline with test data passes
mark could replicate the issue on his local machine.
I could not.
I suspect it might be due to querynator depending on packages from conda-forge. I have conda-forge added in my .condarc
Command used and terminal output
Error executing process > 'NFCORE_VARIANTMTB:VARIANTMTB:QUERYNATOR_CIVICAPI (test_sample)'

Caused by:
  Failed to create Conda environment
  command: conda create --mkdir --yes --quiet --prefix /home/runner/work/variantmtb/variantmtb/work/conda/env-8333bfb82ced3d481ee8f58aa4d2aef8 bioconda::querynator=0.4.1
  status : 1
  message:
    PackagesNotFoundError: The following packages are not available from current channels:

      - bioconda::querynator==0.4.1

    Current channels:

      - defaults
      - https://conda.anaconda.org/bioconda

Relevant files
No response
System information
nextflow version 21.10.3

the bioconda set up also specifies conda-forge to be added to the .condarc
I can replicate the error using conda:
 `.condarc`

channels:
  - defaults
channel_priority: strict

conda create -n querynator-test bioconda::querynator=0.4.1 fails with the issued error
conda create -n querynator-test bioconda::querynator=0.4.1 -c conda-forge works
I cannot replicate the error when running the pipeline. (neither nextflow-v23.10.1 nor nextflow-v21.10.6)

querynator works now. Tests now fail with the same issue at multiqc:
error message

[01/02ce65] Submitted process > NFCORE_VARIANTMTB:VARIANTMTB:BCFTOOLS_NORM (test_sample)
Creating env using conda: bioconda::querynator=0.4.1 -c conda-forge [cache /home/runner/work/variantmtb/variantmtb/work/conda/env-45e4beb0e59f8fa8b0c1e32b9558b591]
[ca/d1f23e] Submitted process > NFCORE_VARIANTMTB:VARIANTMTB:QUERYNATOR_CIVICAPI (test_sample)
Creating env using conda: bioconda::multiqc=1.14 [cache /home/runner/work/variantmtb/variantmtb/work/conda/env-26bbe61d6cbca5f71da2d7fcef2c52eb]
Error executing process > 'NFCORE_VARIANTMTB:VARIANTMTB:CUSTOM_DUMPSOFTWAREVERSIONS (1)'

Caused by:
  Failed to create Conda environment
  command: conda create --mkdir --yes --quiet --prefix /home/runner/work/variantmtb/variantmtb/work/conda/env-26bbe61d6cbca5f71da2d7fcef2c52eb bioconda::multiqc=1.14
  status : 1
  message:
    PackagesNotFoundError: The following packages are not available from current channels:

      - bioconda::multiqc==1.14

    Current channels:

      - defaults
      - https://conda.anaconda.org/bioconda

    To search for alternate channels that may provide the conda package you're
    looking for, navigate to

        https://anaconda.org/

    and use the search bar at the top of the page.

-[nf-core/variantmtb] Pipeline completed with errors-
Error: Process completed with exit code 1.

either add -c conda-forge to every conda use (ugly)
or find a way to specify conda-forge as one of the default channels to check.

maybe via the conda scope in the config file?
or outsource package specifications to environment.yml like sarek does

adding channels to .nextflow.config did not work like this:
.nextflow.config

...
// set conda channels
conda {
  channels = ["conda-forge", "bioconda", "defaults"]
}

commit 6a9ced0 resolves issue without changes in every conda call.


Git vars use latest change on tsu, tpr folder instead of whole repo

Updated the Testsuite parameters to reflect the last changes that happened on the tsu, tpr folder instead of the whole repo.

Closes #43

Close PR since it has been decided to stick to the previous commit information.


transaction_value_currency not picked up by solr for FI-3

Data reported by reporting_org FI-3 (Ministry of Foreign Affairs, Finland) do not provide the transaction_value_usd field, because the transaction_value_currency field is not correctly processed.

@sylvanr is ready for review correct?

also resolved by #2662. Will require reparse/reindex to reflect


Remove version in mmh3module.cpp

I plan to remove the line
PyModule_AddStringConstant(module, "__version__", "3.1.0");

in mmh3module.cpp in the next non-trivial update.
The __version__ constant was introduced in 2.1 (2013-02-25), since it was in vogue back then and the only (iirc) way to get the version number of a module from within a python script.
However, Python 3.8 officially introduced importlib.metadata, which is no longer provisional in Python 3.10. Python 3.7 still needs to pip install importlib-metadata, but it will be in EOL soon. Therefore, there is no need to keep the __version__ constant anymore. Plus, keeping the same info in multiple files is a bad (very bad indeed) engineering practice.
See also
https://stackoverflow.com/a/72168209
I will bump up the version to 4.0.0 then, because the removal breaks backward compatibility, however minor it is.

The removal of the constant has been done in version 4.0.0.


dxguids: guard Direct3D with d3d12_h

By doing this so that WINADAPTER_IID can be used without include of <directx/d3d12.h>
Signed-off-by: Yonggang Luo<EMAIL_ADDRESS>

There are merged in https://github.com/microsoft/DirectX-Headers/commit/4c2339cb02ae6d7dc7f6e56437ccff038ec8f126

@walbourn This is a different PR.

Pelease reopen it


Support for 12.5

i just want a support for the latest version.

Update, we now have 12.9

Checkout v0.6.5 for v2020.12.9s game version support

Can you add steps on how to compile this? maybe in the readme
Do I need Visual Studio 2019 ( a huge download for me, im stuck with Vs2012)? (which are you using?)

@gr01d unfortunately, to compile mod manually you need one of the latest VS version (mine is Microsoft Visual Studio Community 2019 Version 16.7.5). I don't know what is the minimal required version. I use C++17 and some features from C++20 that are definitely missing in VS 2015 and earlier.
P.S. Please, open new issue if you have question or suggestion. Comments can easily get lost.


Tech: linter

Add linter with rules for react and typescript

Hi @idodav,
I found this Medium article and I think it is relevant to your demand.
Can you take a look and tell me if this is what you asking for?
https://andrebnassis.medium.com/setting-eslint-on-a-react-typescript-project-2021-1190a43ffba
If yes, I would be happy to take this issue on me :)

@SID262000 Thank you very much for your contribution. Also @nkotzrenko for reviewing.
Closing this issue for now!

@idodav I think we should integrate these linters as part of the pre-commit hook to make sure the well-formatted code only gets pushed to the main codebase.
cc: @nkotzrenko

Think so as well!


lxd: Switch over to instance types

@stgraber this has been rebased on master now.

jenkins: test this please


distributed cache -migrate to csproj

fixes #2705

@GuardRex  please review

I'm pulling down the sample to repro it's all good.
FYI tho: Check that web.config ... It's got a wacky character at the start of the file that needs to be deleted.
Also note that server GC will be the default, so I think that runtimeconfig.template.json file can go adios: https://github.com/dotnet/cli/pull/5537

... and unused references and out of order usings ...
Program.cs unused ...
using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;

StartTimeHeader.cs unused ...
using System;
using System.Collections.Generic;
using System.Linq;

... and the usings are out of order.
Startup.cs unused ...
using Microsoft.Extensions.Caching.SqlServer;
using Microsoft.Extensions.Options;
using Microsoft.Extensions.Caching.Redis;

... usings out of order, too.

Let's just stick with the project.json to csproj conversions this pass

The sample doesn't run OOB ...
System.InvalidOperationException: Either absolute or sliding expiration needs to be provided.
   at Microsoft.Extensions.Caching.SqlServer.DatabaseOperations.ValidateOptions(Nullable`1 slidingExpiration, Nullable`1 absoluteExpiration)
   at Microsoft.Extensions.Caching.SqlServer.DatabaseOperations.SetCacheItem(String key, Byte[] value, DistributedCacheEntryOptions options)
   at Microsoft.Extensions.Caching.SqlServer.SqlServerCache.Set(String key, Byte[] value, DistributedCacheEntryOptions options)
   at Microsoft.Extensions.Caching.Distributed.DistributedCacheExtensions.Set(IDistributedCache cache, String key, Byte[] value)
   at DistCacheSample.Startup.Configure(IApplicationBuilder app, IDistributedCache cache)

Let me play with it a bit and see what I can work out ...
Either absolute or sliding expiration needs to be provided.

... doesn't sound toooo bad (famous last words :smile:)

I don't have a local SQL Server instance to fully test, but it looks like it will run with this change ...
Add/update Configure in Startup.cs:
var cacheEntryOptions = new DistributedCacheEntryOptions()
    .SetSlidingExpiration(TimeSpan.FromSeconds(30));
cache.Set("lastServerStartTime", val, cacheEntryOptions);

... and it's only throwing "You don't have SQL Server installed you GooberRex:tm:!"

Don't forget to kill that strange 1st character in the web.config file.

I'm not seeing the strange char in web.config with either VS17 or VS Code.

Open it in the GH UI online ... you'll see that character.

Everything else LGTM.
I'll have those samples updated to csproj to the "new-csproj" branch in a few hours, and I'll get the Pub IIS thing squared away, too.

@GuardRex  we see that character frequently (as the first char in the file) in MD files.  We haven't figured out where it's coming from.

You floated the idea that it might be when people work with the file using the GH UI, but I've been exclusively working with GH UI for PR updates and new docs, and I don't see that character cropping up when I work on the files.

moved to PR #2738

@Rick-Anderson X-Files: The Mystery of the Strange Character
Some believe that the character was inserted by 👽 to track human activity in advance of an invasion!
Actually, you probably knew or guessed it was the UTF8 BOM (ï»¿) ...

If I save a file out of Notepad in UTF8, upload it to a repo, the BOM is there, so it seems like someone or some software has saved/copied a file with UTF8 encoding.

We see that so frequently I'm a bit skeptical it from copy/paste with notepad.
cc/ @tdykstra @mairaw

@Rick-Anderson not with regular ANSI encoding, right? I think it makes sense. I also see sometimes the red marks for line endings too like in this commit that all I did was to add a space: https://github.com/mairaw/SCCMdocs/commit/ebbc2b250d651aa8befd7f2458d67bf0f4a8ff10#diff-3eb3e38b128dd94833c21d4a73c80a22

This turned out to be a funky spot for the BOM thing. Just for tracking purposes tho, I'll add it here:
I was just looking at the sample for https://docs.microsoft.com/en-us/aspnet/core/mvc/views/view-compilation in the GH UI and see it ...

@Rick-Anderson Curious: You did the sample (I guess) and was wondering how you prepared it? That might explain at least one source of the :alien: BOM. I mean if that software is saving UTF8 that would explain this case.

We see that frequently and have conjectured it's from web editing. We never figured out where it comes from.

I do a lot of web editing tho thru the GH UI ... e.g., most of those middleware docs and many of the IIS doc updates were all done via the GH UI ... and I didn't seem to be generating them. Could there be a difference when web editing on non-Windows platforms? Anyway ... this was prob a bad spot to ask/mention it. There isn't a dedicated issue on it AFAIK, so I just plopped that here with the rest of the comments.


Extended AzureException by CosmosClientException

CosmosClientException extends AzureException
Updated module-info:
** Removed slf4j as it is being brought in by azure-core.
** Added comment for java.logging module.

/azp run cosmos - java - tests

/azp run java - cosmos - tests

Multimaster multi region fast timed out because of slowness of the machine:
2020-04-21T18:40:44.1821940Z Downloading from ossrh: https://oss.sonatype.org/content/repositories/snapshots/com/github/spotbugs/spotbugs/4.0.0-beta3/spotbugs-4.0.0-beta3.pom
2020-04-21T18:57:23.8690138Z Downloading from central: https://repo.maven.apache.org/maven2/com/github/spotbugs/spotbugs/4.0.0-beta3/spotbugs-4.0.0-beta3.pom

See how this one pom file took 17 mins to download, causing heavy delay on the CI.


Conversion of °C/h

Hi,
as suggested in https://github.com/unitsofmeasurement/uom-se/issues/202#issuecomment-425394070 I've tested the same behavioiur using seshat.
When I try to parse °C/h I get the exception: The scale of measurement for “°C” unit is not a ratio scale.. What does that means?
I would expect to have something like (x+273.15)*3600 (K/s).
What do you think?
Thank you
Luca

The terminology follows the Levels of Measurement: nominal, ordinal, interval and ratio. A ratio scale is a scale without offset. For example m, km, K, N, etc. are ratio scales, but °C is not (it is an interval scale). When computing °C divided by something, there is currently no way to know if the value in °C is a quantity or an interval. In the former case 1°C/h = 374.15 K/h while in the later case 1°C/h = 1 K/h. Current Seshat implementation disable this case for avoiding surprising behavior, but we can revisit that if there is a need for it.
In the longer term, better support of such operations with °C would require an API change in JSR 385 in order to know if we shall interpret the numerator as a quantity or an interval. I think the subject is still on the table.

We hope to do this in Public Review if possible: https://github.com/unitsofmeasurement/unit-api/issues/95
There are two main focus points aside from bugs or improvements over the last version:

format
function

Arithmetics and conversions would be "function", formatting or parsing tickets or new features for 2.0 like "Compound Formatting" are under "format".


[DOCUMENTATION] Fiftyone on Ubuntu 22.04

URL(s) with the issue:
https://voxel51.com/docs/fiftyone/getting_started/troubleshooting.html#troubleshooting-linux-imports
Description of proposal (what needs changing):
At the moment fiftyone seems to work only with OpenSSL 1.1. On Ubuntu 22.04 only OpenSSL 3.x is through apt. Therefore, the line "On Ubuntu, libssl packages can be install with the following command: sudo apt install libssl-dev " should be replaced with something like
"For Ubuntu 21.10 or older, libssl packages can be install with the following command: sudo apt install libssl-dev".
Willingness to contribute

[x] Yes. I can contribute a documentation fix independently.
[ ] Yes. I would be willing to contribute a document fix with guidance from
the FiftyOne community.
[ ] No. I cannot contribute a documentation fix at this time.

Hi @victor1cea,
How should libssl-dev be installed on Ubuntu 22.04 then?
Thanks

@iraadit Afaik libssl-dev is required by FiftyOne to install its own MongoDB database only. On my Ubuntu 22.04 system I configured my own MongoDB instance and pointed FiftyOne towards it. See here how you can do this.
An alternative solution would be to manually compile libssl 1.1 from source and install in your Ubuntu 22.04 system, but 2 versions of libssl (1.1 and 3) will coexist. In my case, this lead to some issues with other components on my system.
I would personally go with the solution shown in the docs as it's less messy than having to juggle 2 versions of libssl.

Hi @victor1cea,
I didn't see your comment right away and finally compiled libssl 1.1 myself and installed it on the machine. I've been able to run FiftyOne afterwards. Hopping it will not create more problems down the line...
I cannot access the link you provided, it sends me to a https://login.united-internet.orglogin page
Thank you

@iraadit had a typo. try this instead https://docs.voxel51.com/user_guide/config.html#configuring-mongodb-connection


Adding potion effects doesn't work

When i open the editor in game and add some potion effects to the item, the potion effects do not work (they show up in nbt but do not apply when using the item and they do not show up when hovering over the item)

same issue

Fixed in v2.2.6


Put in place infrastructure for backwards compatibility for schema and generator

Feature Request
Description of Problem:
As a user, I want the profile/schema and the generator to be backwards compatible so that it will always be possible to use old versions of the schema and the generator even if several new versions of both have been released.
Acceptance Criteria

[ ] Store the accepted schema versions in the generator (for v1.0 of the generator, this will be v0.1 and v1.0 of the schema).
[ ] Update the generator to only try process schema versions from its list of compatible versions. If the schema version is not in the list, prevent further processing, unless an --unsafe-use-incompatible-schema-version flag (for example) is used.
[ ] Enforce updating the generator version every time the semantics or syntax of the schema/profile changes, for major, minor and patch releases.
[ ] Enforce updating the schema version every time the syntax changes.
[ ] Release version v1.0 of the generator and schema.

tasks.

[ ] get the schema version from the inputted profile (before reading the entire profile)
[ ] compare the schema version against list of accepted schemas (error if not accepted)
[ ] (optional) generate the list of accepted schemas from the schemas directory
[ ] select the correct schema from the schemas directory based on the schema version (and run schema validation against it)

these tasks should probably be on another ticket tbh as it's prework and a different scope to the intention of this ticket.

Assuming the max schemaVersion that the generator can support is v1.2:

Schema Version in file
Using vX.Y features
Version Check
Profile Validate
Runtime

1.1
1.1
Y
Y
Y

1.1
1.2
Y
Doesn't match 1.1 line A
Y

1.3
1.2
1.3 not supported
Cannot profile validate 1.3
Y

1,3
1.3
1.3 not supported
Cannot profile validate 1.3
Error

1.2
1.3
Y
Doesn't match 1.2 line A
Error

Sub-tasks for the "Enforce updating the schema version every time the syntax changes." requirement:
[ ] - Add an extra script to be run on the CircleCI which initially always passes.
[ ] - In the script, get profileschema folder and read all files from current branch.
[ ] - Get the same files, but for the origin/master branch.
[ ] - Find out whether the schema files (and their names) are identical, but ignore any new schema files.
[ ] - If not, fail the check.

Will no longer be implementing https://github.com/finos/datahelix/issues/1146#issuecomment-519829371 . The team has decided to not allow turning off the schema version checking.


Not entirely correct about replacing

"the dataset in your Power BI site will be replaced with the updated dataset from the Power BI Desktop file" is not entirely correct.  If you try to publish a report that is named the same thing as another report but one is import and the other is live connect, then it will not replace the first dataset.  It's more like it "updates" the dataset as much as it can.   If you want to truly replace an impoart dataset with another dataset that is direct query, then you'll need delete the first dataset.  Interesting distinction between "update" and "replace."

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

ID: ef5362a0-2148-3de0-fd60-a3fe5d06fe91
Version Independent ID: 1967384e-308d-cf6f-ad5f-cde6e5b741b3
Content: Publish from Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-upload-desktop-files.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

@precedence - thanks for posting and for the clarification. I'll check with the engineering team about how best to clarify what you've described, and meanwhile, others will benefit by having your post here in the article. Thanks for posting!


Rainbow Godsword Swings

Since you already have rainbow scythe swings, I think a cool idea for a future update would be rainbow Godsword swings. Seems like something that'd get used more since way more people have godswords than scythes and not that hard to implement since I imagine the code for scythe swing is based somewhat off the ones for godswords.

it's not really the same; it might be more work than the rainbow scythe swings were by quite a bit. Not 100% sure though.


new higher resolution favicon

Pull Request Test Coverage Report for Build 221

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.979%

Totals

Change from base Build 220:
0.0%

Covered Lines:
91

Relevant Lines:
99

💛  - Coveralls


Enhancement Idea - Scrolling data

It would be great if there was a way to have MMM-GoogleSheets scroll data using a defined range.
Let's say the screen only displayed up to row 15, but you have data down to row 20.  It'd be great to have it scroll the data after X seconds, moving one row up every Y seconds, and then looping back to row 1-13.
Also - I'd love to be able to donate some $ to you for your work on the current version.  Do you have a donation link?

@xSkydve1x this is a great idea! I will have to do some more research on exactly how I might make that happen but I will mark this as an enhancement for now.
You absolutely don't have to donate any money (I enjoy building these projects) but if you are insistent here is my buymeacoffee page.

@xSkydve1x this has been added! You can check out the readme for how to incorporate scrolling data. Let me know if you have any feedback!


[Optimize](Row store) allow to set row_store_page_size for tables, change default value to 16KB

Proposed changes

In current row storage format, under different page sizes, the compression effect of zstd on the hits data set (70 GB) is as follows.

no_compression
zstd (default, page size=4KB)
zstd ( page size=16KB)
zstd ( page size=32KB)
zstd ( page size=64KB)

105.76 GB
33.38 GB
22.07 GB
21.83 GB
18.29 GB

When the page size is 16KB, the optimization is the greatest (33.89%).


feat(e2e): use protractor api

Uses existing Protractor API to run it directly instead of using npm run e2e.
Also adds support for the following flags: --serve, --config,
--specs, --element-explorer, --webdriver-update.
Fix #4256
Fix #4478
Close #1054
BREAKING CHANGE: ng e2e no longer needs ng serve to be running.

@vikerman Please review this from the protractor side :)


Install custom dir

When running the install script if you select a custom directory e.g. "myserver" the apache vhosts file is not updated for the certificate path it still references the /var/www/haproxy-wi/app/certs/ path not /var/www/myserver/app/certs/

Also some of the directory switches for the deny access do not update the correct patch name still referencing /haproxy-wi/

Thank you, problem with certs I have fixed. Tell me please exactly which directories switched for deny access?

Hi, same file just referring to the directory sections lower down that control access to the directories.
The path does not update correctly on them. Also the key path referred to does not exist.

Ok, with the new install script all will be ok, I think so)
Отправлено из Mail.ru для Android среда, 14 августа 2019г., 00:35 +03:00 от terafirmanz<EMAIL_ADDRESS>:

Hi, same file just referring to the directory sections lower down that control access to the directories.
The path does not update correctly on them. Also the key path referred to does not exist.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .


Clarity load time for cloud environment

Sometimes, it may take more than a minute to start the environment.  Update the provided information/instruction to indicate that it may take more than a minute?
@Emily-Jiang obvsered this too:
it takes more than a minute to load up - I am wondering whether we can say it will take a couple of minutes instead of saying one minute

@yeekangc is it ok to add the message on the landing pages before/beside/below the Access cloud-hosted guide button?

Or fix in

SN issue: https://github.ibm.com/skills-network/labs/issues/1137


flashing on MSI x470 Gaming Plus

I recently attempted to use this software to make the LEDs on my motherboard roughly match my sapphire GPU (Since I cannot set the LED color on it in linux) the color of those LED's is a light blue cyan color, so I assumed the command I should use would be
sudo ./msi-rgb 80808080 80808080 ffffffff
to keep the color solid, but this resulted in my motherboard LEDs flashing cyan, then solid blue, back and forth. I was able to get something similar to what I wanted with
sudo ./msi-rgb 88888888 88888888 ffffffff
It seems that on this board it is using 8 color positions and only looking at one nibble per color each? Or is there something I am misunderstanding about the software?

also, to add onto this, I tried
sudo ./msi-rgb 00000000 00000000 0000000f
to see if it was going by nibbles instead of bytes, and this caused the led to blink full bright blue but be off most of the time, same result from
sudo ./msi-rgb 00000000 00000000 000000f0

The SIO only provides 16 levels each of r,g,b resp (single hex digit).
There are 8 'tweens' or animation frames per rgb loop, the speed of which
is controlled by 'duration'
To further make things more interesting, the SIO expects little endian
notation, so to get correct animation the input must be
r2r1r4r3r6r5r8r7 g2g1g4g3g6g5g8g7 b2b1b4b3b6b5b8b7
Welcome to the metal.
Dominic Cerisano
ca.linkedin.com/in/dcerisano
dcerisano@skype
NOTICE: Confidential message which may be privileged. Unauthorized
use/disclosure prohibited.
AVIS : Message confidentiel dont le contenu peut être privilégié.
Utilisation/divulgation interdites sans permission.
On Mon, Jul 15, 2019 at 2:37 AM segaretroboy<EMAIL_ADDRESS>wrote:

I recently attempted to use this software to make the LEDs on my
motherboard roughly match my sapphire GPU (Since I cannot set the LED color
on it in linux) the color of those LED's is a light blue cyan color, so I
assumed the command I should use would be
sudo ./msi-rgb 80808080 80808080 ffffffff
to keep the color solid, but this resulted in my motherboard LEDs flashing
cyan, then solid blue, back and forth. I was able to get something similar
to what I wanted with
sudo ./msi-rgb 88888888 88888888 ffffffff
It seems that on this board it is using 8 color positions and only looking
at one nibble per color each? Or is there something I am misunderstanding
about the software?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nagisa/msi-rgb/issues/70?email_source=notifications&email_token=AAMQZDMMV4QZ3PFHJAJ7BADP7QLKVA5CNFSM4IDTTTDKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7ERPWA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMQZDN5O52Z4VSRDTKLGADP7QLKVANCNFSM4IDTTTDA
.

alright, does this apply to any of the other motherboards or just this one? The README file states four tweens so maybe it needs updated?

Its not about the motherboard, it is about the specific SIO chip ms-rgb is coded for. Some mobos have it, some don't.


Replace generateIntensityMatrix

generateIntensityMatrix becomes redundant after use of clusterAlign

Has to be able to deal with empty peak lists

Functionality made and succesfully tested


Fails on buckets containing dots

For me, it seems I can successfully upload to buckets that don't contain dots in the name.
This is basically a duplicate of #53, which was apparently fixed by a third-party library update, but...it's now back?

Hmm, I think I somehow had an old version of this installed. Running on latest works now.

Ok, after a lot of painful investigation, I have discovered that there is an issue.
mapbox-tile-copy uses s3urls version 1.5.3 which fixed the dot issue.
But mapbox-tile-copy uses tilelive-s3 version 6.5.1 which uses s3urls^1.4.0 which in my case seems to mean that I got two versions of s3urls installed. tilelive-s3's version is what calculates the bucket name, so it gets it wrong.
Ohhh...I think it's because @mapbox/tilelive-s3 uses s3urls instead of @mapbox/s3urls.

Wow. The fix, after all that seems to be straightforward: just upgrade @mapbox/tilelive-s3 to 7.0.1. Although I haven't run the tests, so it's possible that breaks other things.

I ran into the same issue, and have decided to push to URL without dots and will migrate to proper s3 bucket afterwards.