optimize diagonal

Summary
Optimize diagonal with bypass of parameter checking.
Details and comments
Parameter-checking in diagonal() is very slow if its qubit is large. It basically checks

check entry can be cast to complex (in DiagonalGate)
check absolute value of each entry is 1 (in DiagonalGate)
extract parameters (in Instruction)

These checking and processing can be skipped if

entry is 1 or -1 (skip 1 and 2)
all the entries is 1 or -1 or complex (skip 3)

I'm not sure a case when an entry includes Parameter and its absolute value is 1. But the original path (including 1, 2, and 3) is remained in this PR.

Performance of FourierChecking that calls two diagonal().
for qubit in range(5, 25):
    f = [ 1, -1 ] * (2 ** (qubit - 1))
    g = [ -1, 1 ] * (2 ** (qubit - 1))
    start = time()
    circ = FourierChecking(f, g)
    end = time()
    print ('{0}-qubit: {1}'.format(qubit, (end - start)))

qubit
original
check 3
no-check

20
7.51990724
3.62482285
2.53825712

21
15.0584638
7.26154184
5.0785079

22
30.1116669
14.6573582
10.2551303

23
59.671942
31.5060558
20.4238489

24
120.515799
62.253078
40.799298

(s)

Let me close this PR. This optimization is only effective if a diagonal operation is for many qubits by creating bypath to avoid calling validate_parameter with following codes:
        super().__init__("diagonal", int(num_action_qubits), [])
        self._params = diag

Though performance to generate FourierChecking circuits is improved with PR, other cases are not so often.

Let me close this PR. This optimization is only effective if a diagonal operation is for many qubits by creating bypath to avoid calling validate_parameter with following codes:
        super().__init__("diagonal", int(num_action_qubits), [])
        self._params = diag

Though performance to generate FourierChecking circuits is improved with PR, other cases are not so often.


implement passing parameters to render_wizard and finish_wizard_path

fixes #221

@schneems i added some details to the readme. See if that is clear enough. :)

great, thanks!


Epoch.time_slice() is crrently broken

currently, Epoch.time_slice() is broken:
import neo
import quantities as pq
Epoch = neo.core.Epoch(times=[1,2,3]*pq.s,durations=[1,1,1]*pq.ms,labels=['a','b','c'])
Epoch.time_slice(*(1,2)*pq.s)

## -- End pasted text --
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-1-d675ce8b57e8> in <module>()
      2 import quantities as pq
      3 Epoch = neo.core.Epoch(times=[1,2,3]*pq.s,durations=[1,1,1]*pq.ms,labels=['a','b','c'])
----> 4 Epoch.time_slice(*(1,2)*pq.s)

~/Dropbox/python/git_repos/python-neo/neo/core/epoch.py in time_slice(self, t_start, t_stop)
    261
    262         indices = (self >= _t_start) & (self <= _t_stop)
--> 263         new_epc = self[indices]
    264         return new_epc
    265

~/Dropbox/python/git_repos/python-neo/neo/core/epoch.py in __getitem__(self, i)
    177         obj._copy_data_complement(self)
    178         obj.durations = self.durations[i]
--> 179         obj.labels = self.labels[i]
    180         return obj
    181

TypeError: only integer scalar arrays can be converted to a scalar index

self.labels is a list, and indexing it with a np.array() causes the error. I am making a pull request and reference this issue

Thanks for  the report.
My guess is that it is already fix in this PR #472
Could test the branch related to this PR  to see if the bug is fixed ?
Best

my bad. All of this error boils down to me passing the labels as a list of strings and not a np.array of strings. Apologies ...


currentlyFocusedField is deprecated

Description
I have upgraded react native form 0.62.2 to 0.63 , and facing this issue
React Native version:
System:
OS: macOS 10.15.4
CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
Memory: 26.13 MB / 8.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 13.7.0 - /usr/local/bin/node
Yarn: 1.21.1 - /usr/local/bin/yarn
npm: 6.13.6 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.5, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK: Not Found
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
Xcode: 11.5/11E608c - /usr/bin/xcodebuild
Languages:
Java: 11.0.6 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.0 => 0.63.0
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
when I navigate from one page to another, I face this issue. I'm using react native router flux for navigation
Expected Results
resolve "currentlyFocusedField is deprecated" issue
Snack, code example, screenshot, or link to a repository:

Is currentlyFocusedField used by your code or the community module you're depending on? Either way... if it's deprecated then as a result of your upgrade you'll need to resolve that.

@chrisglein router flux is using react-navigation , so problem is there. so how we can resolve that ?

Open an issue on the react-navigation github.


Telnet to mapscii.me doesn't work (not enough xterm features)

From @timfel on May 30, 2017 9:44

Your Windows build number:  (Type ver at a Windows Command Prompt)
10.0.15063

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
Type telnet mapscii.me. This will connect to the server of this project: https://github.com/rastapasta/mapscii. The expectation is to see an ASCII map of the world.

What's wrong / what should be happening instead:
Expected:

Got:

Copied from original issue: Microsoft/WSL#2173

From @zadjii-msft on July 13, 2017 22:29
@timfel what font & size are you using? I believe that on Insider's builds, it works fine

Albeit, I couldn't find a font with the right characters

From @sunjoong on May 30, 2017 13:38
@timfel - It looks like to be able to draw shape of land but not to print names in my case on unbuntu and gentoo on WSL;

From @shanselman on January 19, 2018 22:49
@tara-raj good test case for font fallback.

We should check if this works with the font fallback support we have now and with the updates to VT we've done in course of getting the Terminal project going.


fix: deprecation of binom_test in scipy 1.12.0 and cryptic error occurring with incorrect VCFs in cyvcf2 0.30.26

This PR handles errors that arise when you upgrade to the newest versions of our dependencies, specifically for cyvcf2 and scipy
It resolves #206 and also handles the deprecation of scipy.stats.binom_test in favor of scipy.stats.binomtest: https://docs.scipy.org/doc/scipy/release/1.12.0-notes.html#expired-deprecations
Merging this PR will resolve failures in the CI checks for all of our other PRs

@LiterallyUniqueLogin, I decided to catch the error in all downstream tools (which turned out to just be dumpSTR and qcSTR), since it was easy enough to implement
Let me know if there are any other changes we need!

@LiterallyUniqueLogin, I'm realizing now that this error will probably crop up in any tool that uses cyvcf2, and not just in dumpSTR and qcSTR.
The proper way to handle this is probably to add a BrokenVCF-equivalent test to every tool, and then use those tests to figure out how to properly catch the exceptions. But tbh those sorts of changes would probably bring this PR beyond what I have the bandwidth for. Also, I don't think I'm familiar enough with TRTools to be able to write those tests.

Your reasoning sounds good. Fixing this shouldn't be held up by missing tests for other utilities.


Spec Hash Annotation and AutoHealing

Summary
Pipeline/ISBService need to auto heal if changed manually even if rollouts are unchanged by comparing annotation hash of spec.
Details
To avoid performing multiple expensive update calls on reconciliation, we will keep track of the following 3 hash strings:

hashA: previous PipelineRollout pipeline spec hash (the one stored in the PipelineRollout annotations)
hashB: current PipelineRollout pipeline spec that we calculate at reconciliation
hashC: hash of Pipeline spec on the cluster without the Lifecycle field (we need to calculate this everytime)

for the following scenarios:

If hashA == hashB && hashC == hashB: there are no changes made to the PipelineRollout nor to the Pipeline; therefore, we do not need to perform any updates
If hashA == hashB && hashC != hashB: there are no changes made to the PipelineRollout; however, the Pipeline on the cluster was changed. Therefore, we need to autoheal the Pipeline by applying the PipelineRollout pipeline spec to overwrite the Pipeline CR on the cluster.
If hashA != hashB the PipelineRollout pipeline spec has changed and we need to apply it. Note: no need to check hashC in this case.

NOTE: instead of computing hashC by reading the Pipeline everytime and calculating the hash, we considered gettting the hashC from the Pipeline since Numaflow already stores it. However, this includes the Lifecycle field which does not work for us since we want to exclude pausing state from the hash.
Therefore, we can either:

A. read the Pipeline CR and calculate hashC OR
B. read the Pipeline CR and compare the spec directly against the PipelineRollout pipeline spec.

Case B may not be any more performant than case A since for both cases we need to read the pipeline and, because we are using RawExtension type, we will probably have to perform marshalling/unmarshalling operations to convert to map and then perform deep comparisons. Hashing and comparing strings may be more performant. Appropriate performance testing should be performed if this is a great concern.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Thanks for all of this write up and in-depth analysis. I have some questions for clarification.
What are the advantages of this plan over the one I mentioned earlier?: Don't store annotation, and simply compare intended spec (including pause designation (so, this would be right before we're about to apply it)) with actual running spec (not the one that Numaflow stores in its annotation)?

I see you already have a PR so I don't want to mess up what you've already done, but here are my thoughts in case it makes sense to do some refactoring later at least (which I'd be happy to do).
In my mind, the true difference is "desired PipelineRollout spec" vs "running Pipeline". Those may be different because:

the PipelineRollout spec has changed
we need to auto-heal a manual change that was made to the Pipeline

In either case, why not safely make our change without data loss?
Therefore, we could simply compare those 2 things and not worry about the previously applied PipelineRollout spec.
As far as performance comparison:

In the "no hash annotation" model, there are no K8S API Update calls to update the Rollout spec
In the "hash annotation" model, in the case that the Rolllout spec has changed, there is no calculation of the current running child object and comparison between them

I believe the first one is more of an optimization. It is the saving of a network call, which is more impactful than a calculation. Also, the second one only helps in the case of the Rollout spec having changed, which I believe would be the minority of Reconcile() calls. More often, I would think we would see Reconcile() called because the status of a child object changed.
cc @xdevxy

@juliev0 I'm not sure what you mean by 1. In the "no hash annotation" model, there are no K8S API Update calls to update the Rollout spec. I think that even without hash annotations, if there are changes to either the PipelineRollout or the Pipeline, there will be a call to update the CR.
I think that what this boils down to is that we either simplify implementation or gain some performance improvements. However, has mentioned in the issue description, we may want to do performance testing to really understand how much we gain or lose with either implementations. I think at this time we do not have the bandwidth to do those tests.
If possible, you, me and @xdevxy should discuss verbally pros and cons of both ideas to clarify the auto healing and hashing features.

I think that even without hash annotations, if there are changes to either the PipelineRollout or the Pipeline, there will be a call to update the CR.

So, there will be an Update made to the PipelineRollout before the reconciler gets it (because ArgoCD applied it), then yes there is that call, but that's outside of us. But then here we are doing an additional Update to the Rollout when the Annotation changes.

Thanks so much for continuing the conversation with me.

If this is a concern, would it make sense to store the hash in the status (instead of the annotations) since we always update the status anyways?

That's a possibility. Honestly, I'm not terribly concerned about the Update(), but I mentioned it because the main reason for going with the other approach seemed to be performance.
I think I may be okay with keeping the existing logic that uses the annotation. But my thought is that our overall code structure should be:
If Pipeline doesn't exist, create it
else:
   if !(currentRolloutSpecHash == pipelineRollout.Annotations[apiv1.KeyHash] && currentRolloutSpecHash == pipelineSpecHash)   <-- this is the only place we need a comparison and not later on
     perform lifecycle management (i.e. either pause pipeline, continue waiting for it to pause, or unpause and update spec)
processPipelineStatus()

What do you think?

Thanks so much for continuing the conversation with me.

If this is a concern, would it make sense to store the hash in the status (instead of the annotations) since we always update the status anyways?

That's a possibility. Honestly, I'm not terribly concerned about the Update(), but I mentioned it because the main reason for going with the other approach seemed to be performance.
I think I may be okay with keeping the existing logic that uses the annotation. But my thought is that our overall code structure should be:
If Pipeline doesn't exist, create it
else:
   if !(currentRolloutSpecHash == pipelineRollout.Annotations[apiv1.KeyHash] && currentRolloutSpecHash == pipelineSpecHash)   <-- this is the only place we need a comparison and not later on
     perform lifecycle management (i.e. either pause pipeline, continue waiting for it to pause, or unpause and update spec)
processPipelineStatus()

What do you think?

That makes sense. For now, processPipelineStatus is called in applyPipelineSpec which is also called in multiple places in the reconcile func. This is what I meant earlier that we should change that logic and maybe have few functions to improve how and when to perform the operations we want. I'm honestly trying to make few small changes at the time but once I'm done with the urgent tasks for this sprint, I'll restructure that code. I also plan to make unit tests functions a bit more reusable now that we have more cases and some are similar.


mraa with firmata conflicts with modemmanager

modemmanager apparently causes Arduino 101 board to be reset in an endless loop. As the result firmata doesn't work. The workaround is to uninstall modemmanager (it might be also possible to remove some udev rules).
Platform: Intel NUC DE3815TYKE running Ubuntu 16.04 x86_64
MRAA version: 1.5.1-2 (installed from MRAA PPA repository)
Syslog:
Jan 13 14:36:24 skiselev-desktop kernel: usb 1-3: New USB device found, idVendor=8087, idProduct=0ab6
Jan 13 14:36:24 skiselev-desktop kernel: usb 1-3: New USB device strings: Mfr=1, Product=2, SerialNumber=3
Jan 13 14:36:24 skiselev-desktop kernel: usb 1-3: Product: Atlas Edge
Jan 13 14:36:24 skiselev-desktop kernel: usb 1-3: Manufacturer: Intel
Jan 13 14:36:24 skiselev-desktop kernel: usb 1-3: SerialNumber: 00.01
Jan 13 14:36:24 skiselev-desktop kernel: cdc_acm 1-3:1.0: ttyACM0: USB ACM device
Jan 13 14:36:24 skiselev-desktop mtp-probe[8815]: checking bus 1, device 56: "/sys/devices/pci0000:00/0000:00:14.0/usb1/1-3"
Jan 13 14:36:24 skiselev-desktop mtp-probe[8815]: bus: 1, device: 56 was not an MTP device
Jan 13 14:36:24 skiselev-desktop org.freedesktop.fwupd[673]: (fwupd:1720): libdfu-WARNING **: Failed to set new device: replugged device is not DFU-capable
Jan 13 14:36:27 skiselev-desktop org.gnome.evolution.dataserver.Sources5[1260]: ** (evolution-source-registry:1549): WARNING **: secret_service_search_sync: must specify at least one attribute to match
Jan 13 14:36:40 skiselev-desktop ModemManager[632]:   Creating modem with plugin 'Generic' and '1' ports
Jan 13 14:36:40 skiselev-desktop ModemManager[632]:   Could not grab port (tty/ttyACM0): 'Cannot add port 'tty/ttyACM0', unhandled serial type'
Jan 13 14:36:40 skiselev-desktop ModemManager[632]:   Couldn't create modem for device at '/sys/devices/pci0000:00/0000:00:14.0/usb1/1-3': Failed to find primary AT port

In CODK they simply uninstall modemmanager during the preparational target run. Not sure we'd want to do anything like that (and then again, this would only work for those who install from sources and not packages).
Sounds like something to add to the docs as a note to user to make sure it's uninstalled, what do you think, @arfoll?

Uhm it's a tricky one, modemmanager seems to just assume everthing in
/dev/ttyACMx is a modem, which uhm isn't an entirely bad assesment. Really
the arduino boards shoudl probably be asigned as something else. Likely the
best way to do this would be get users to create a udev rule to reassign
the firmata device node to smething like /dev/ttyFirmata0 rather than force
them to uninstall modemamanger.
On 15 January 2017 at 19:06, Alex T<EMAIL_ADDRESS>wrote:

In CODK they simply uninstall modemmanager
https://github.com/01org/CODK-M/blob/master/Makefile#L71 during the
preparational target run. Not sure we'd want to do anything like that (and
then again, this would only work for those who install from sources and not
packages).
Sounds like something to add to the docs as a note to user to make sure
it's uninstalled, what do you think, @arfoll https://github.com/arfoll?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/intel-iot-devkit/mraa/issues/630#issuecomment-272712255,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfZZnhLSS4jl-dohW7Px6FuD8eGJCTvks5rSmARgaJpZM4LjZoy
.

Oh, that's a good idea, I haven't looked too deep into this, but yeah, if all what ModemManager looks at is the device file name, your suggestion makes a lot of sense and is definitely better than asking people to uninstall it :)

Does this not effect the normal workflow for Arduinos too?

Confirmed, and fixed for my instance by disabling the ModemManager service. A custom udev rule would be a much nicer way to address this conflict.

Does this not effect the normal workflow for Arduinos too?

@tingleby I was wondering about that too, wouldn't this issue affect Arduinos in general, regardless of MRAA?

@w4ilun Yes Arduinos are blacklisted by ModemManager so they are ignored by it. We can add this upstream but could take some time
In the meantimeI have updated the ppa to mraa 1.6.1 and added some udev rules to the imraa package which should fix this.

So I tried on a fresh install with Ubuntu Server 16.04.2. Once I have all the packages installed, I plugged in the 101 and rebooted. I then tried using mraa with mraa.Gpio(512+13) with no luck. I then tried it with mraa.addSubplatform... and it worked, so that confirms Firmata has been flashed but something else was wrong. I checked the logs and got the following:
Apr 10 20:44:27 ubuntu-nuc systemd[1]: Starting imraa...
Apr 10 20:44:27 ubuntu-nuc libmraa[1003]: libmraa version v1.6.1 initialised by user 'root' with EUID 0
Apr 10 20:44:27 ubuntu-nuc libmraa[1003]: Platform not supported, not initialising
Apr 10 20:44:27 ubuntu-nuc libmraa[1003]: imraa: Failed to open lock file
Apr 10 20:44:27 ubuntu-nuc libmraa[1003]: libmraa initialised for platform 'Unknown platform' of type 98
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: dfu-util 0.9
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Copyright 2005-2009 Weston Schmidt, Harald Welte and OpenMoko Inc.
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Copyright 2010-2016 Tormod Volden and Stefan Schmidt
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: This program is Free Software and has ABSOLUTELY NO WARRANTY
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Please report bugs to http://sourceforge.net/p/dfu-util/tickets/
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: dfu-util: Invalid DFU suffix signature
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: dfu-util: A valid DFU suffix will be required in a future dfu-util release!!!
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Opening DFU capable USB device...
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: ID 8087:0aba
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Run-time device DFU version 0011
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Claiming USB DFU Interface...
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Setting Alternate Setting #7 ...
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Determining device status: state = dfuIDLE, status = 0
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: dfuIDLE, continuing
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: DFU mode device DFU version 0011
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Device returned transfer size 2048
Apr 10 20:44:29 ubuntu-nuc imraa[1003]: Copying data from PC to DFU device
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: [1.7K blob data]
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Download done.
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Sent a total of 62876 bytes
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: state(2) = dfuIDLE, status(0) = No error condition is present
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Done!
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: dfu-util: can't detach
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Resetting USB to switch back to runtime mode
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: dfu-util: error resetting after download
Apr 10 20:44:31 ubuntu-nuc systemd[1]: mraa-imraa.service: Main process exited, code=exited, status=1/FAILURE
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: imraa version is 1.4.0 good
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: dfu location: /usr/bin
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: lock file location: /tmp/imraa.lock
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Starting to flash board
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Ardunio 101 Device Node Path: /dev/ttyACM0
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Waiting for device...
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Waiting for device...
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Device found!
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: flash cmd: /usr/bin/dfu-util -d ,8087:0ABA -D /usr/share/mraa/firmata101.ino.bin -v -a 7 -R
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: ERROR: Upload failed on /dev/ttyACM0
Apr 10 20:44:31 ubuntu-nuc systemd[1]: Failed to start imraa.
Apr 10 20:44:31 ubuntu-nuc systemd[1]: mraa-imraa.service: Unit entered failed state.
Apr 10 20:44:31 ubuntu-nuc systemd[1]: mraa-imraa.service: Failed with result 'exit-code'.
Apr 10 20:45:11 ubuntu-nuc libmraa[1278]: libmraa version v1.6.1 initialised by user 'root' with EUID 0
Apr 10 20:45:11 ubuntu-nuc libmraa[1278]: Platform not supported, not initialising
Apr 10 20:45:11 ubuntu-nuc libmraa[1278]: imraa: Failed to open lock file
Apr 10 20:45:11 ubuntu-nuc libmraa[1278]: libmraa initialised for platform 'Unknown platform' of type 98
Apr 10 20:45:19 ubuntu-nuc libmraa[1278]: gpio525: init: Using sub platform
Apr 10 20:45:19 ubuntu-nuc libmraa[1278]: gpio525: init: Sub platform not initialised
Apr 10 20:45:23 ubuntu-nuc libmraa[1278]: gpio: init: pin 13 beyond platform pin count (0)
Apr 10 20:45:27 ubuntu-nuc libmraa[1278]: gpio525: init: Using sub platform
Apr 10 20:45:27 ubuntu-nuc libmraa[1278]: gpio525: init: Sub platform not initialised
Apr 10 20:45:35 ubuntu-nuc libmraa[1285]: libmraa version v1.6.1 initialised by user 'root' with EUID 0
Apr 10 20:45:35 ubuntu-nuc libmraa[1285]: Platform not supported, not initialising
Apr 10 20:45:35 ubuntu-nuc libmraa[1285]: imraa: Failed to open lock file
Apr 10 20:45:35 ubuntu-nuc libmraa[1285]: libmraa initialised for platform 'Unknown platform' of type 98
Apr 10 20:46:22 ubuntu-nuc libmraa[1285]: mraa: Added firmata subplatform
Apr 10 20:46:57 ubuntu-nuc libmraa[1285]: gpio525: init: Using sub platform

@w4ilun is this with Node.js and MRAA installed via NPM? If that's the case it will never work unless you manually add the subplatform, because node-gyp builds its own mraa.node with a bundled static libmraa. C/C++ and Python should add it automatically though.

@Propanu I tried this with C and Python after a clean install of Ubuntu and MRAA. I suspect it has something to do with the lockfile

@w4ilun, @tingleby, I figured it out. The udev rules that were added don't help in this case, because dfu fails on detaching the board, when switching back from DFU mode, as can be clearly seen from here:
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: dfu-util: can't detach
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: Resetting USB to switch back to runtime mode
Apr 10 20:44:31 ubuntu-nuc imraa[1003]: dfu-util: error resetting after download

And there's nothing preventing ModemManager to go bananas when that happens. In fact, if you look at the logs on a clean install you'll see the board keeps resetting.
The simplest solution I found would be to change 60-mraa-imraa.rules to:
# Arduino 101
SUBSYSTEM=="usb", ATTR{idVendor}=="8087", ATTR{idProduct}=="0ab6", ENV{ID_MM_DEVICE_IGNORE}="1"

That's it, after the change I was able to see the subplatform added consistently over several reboots. No bugs in imraa/firmata code.

@Propanu so...
# Arduino 101 in DFU Mode
SUBSYSTEM=="tty", ENV{ID_REVISION}=="8087", ENV{ID_MODEL_ID}=="0ab6", MODE="0666", ENV{ID_MM_DEVICE_IGNORE}="1", ENV{ID_MM_CANDIDATE}="0"
SUBSYSTEM=="usb", ATTR{idVendor}=="8087", ATTR{idProduct}=="0aba", MODE="666", ENV{ID_MM_DEVICE_IGNORE}="1"

Should be replaced with?
# Arduino 101
SUBSYSTEM=="usb", ATTR{idVendor}=="8087", ATTR{idProduct}=="0ab6", ENV{ID_MM_DEVICE_IGNORE}="1"

Does this not effect other usages of the 101 too?
The rules as they are now are based on https://github.com/01org/corelibs-arduino101/blob/master/scripts/create_dfu_udev_rule
Is that wrong?

I've noticed the Arduino IDE rules, but in our case flashing works just fine. It's after that we run into issues. They're not wrong, just not relevant to our flow. I tested the process with all 3 rules in there, and it works like that too.
TBH, I've never heard of anyone wanting to use the 101 as a modem and I'd say it's a very remote scenario (especially if you install IMRAA right?). Best we can do is document this properly in IMRAA's documentation and perhaps add an extra comment in the rules file mentioning it:
# The device won't be usable as a modem with this rule in place

What do you think?

So removing the previous rules might affect flashing the board manually, I've pretty much relied on the service in my case and that's a pass. It might be best to keep all 3 rules in there after all.

Ok this is being built on the PPA right now.

@skiselev does this look resolved to you now?


Fixed indentation in camel/test.properties

The indentation was so wrong, that the file could not be parsed

Found when running https://github.com/adoptium/aqa-tests/issues/5575#issuecomment-2341256974

The parsing error was caused by the updated strict parsing format from PROP_VALUE=cat $PROPERTY_FILE | grep "$PROP_KEY" | cut -d'=' -f 2-`` to  local PROP_VALUE=cat "${FILE}" | grep -v "^#" | grep "^${PROP_KEY}=" | cut -d'=' -f 2-, which requires the no space before the each property line. So probably we need also update some documents.


samples: led: consolidate LED samples under same directory

With over a dozen LED samples, grouping them in a common samples/drivers/led/ folder helps keep things tidy and lays the groundwork for further improvements in the way samples are showing up in the docs.

I'd like to see a samples/drivers/led/led_apa102c_bitbang/README.rst file created, but otherwise LGTM.

I'd like to see a samples/drivers/led/led_apa102c_bitbang/README.rst file created, but otherwise LGTM.

Ya, different topic :)
https://github.com/zephyrproject-rtos/zephyr/issues/27805


package.json file not parseable on install

I get the following error message when trying to install the package
Tell the package author to fix their package.json file. JSON.parse
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\SEANH_~1\AppData\Local\Temp\apm-install-dir-11713-14172-pu491m\npm-debug.log

Try to upgrade to 2.4.0


Deadlock with tsm1 engine

During a stress testing of InfluxDB 0.11 cluster, it got stuck and all queries timed-out. During this test, I was sending writes, read (and also drop series).
I could see that number of active queries were high and didn't reduced even long time after stopping the stress tools.
 This hang because the second thread is blocked on the Lock (https://golang.org/pkg/sync/#RWMutex.Lock : "a blocked Lock call excludes new readers from acquiring the lock")

To diagnose this issue, I've killed one a the InfluxDB node that got stuck with a SIGQUIT. Interesting part of the stacktrace is at https://gist.github.com/PierreF/7ed3ade69d56d04b48e8e22bfead43d5
Exact InfluxDB version : InfluxDB v0.11.1 (git: 0.11 57d625f804ca510220eba535568c9e635d588be3)
This issue should still exists in 0.12, since FileStore.KeyCursor still get a RLock and call FileStore.locations (thought KeyCursor) which also get a RLock.

Fixed via #6277


Cant checkout 0.2.0 automatically with Cocoapods

pod 'RevealingSplashView' is checking out 0.1.0

Thats weird. https://cocoapods.org/pods/RevealingSplashView shows that is 0.2.0
Can you run a pod repo update?

Perfect, thanks :)


[NFC][BACKEND] Remove dead code in Nvidia backend's ConvertLayoutOpConversion

@ThomasRaoux I won't merge this PR in the short term, no worries. There are some remaining stuff I need to solve


fix(resource-instance-form): disable "next" button when form is not valid (DEV-803)

resolves DEV-803

https://user-images.githubusercontent.com/60604010/163142676-940713f6-a6a6-4f92-882a-1ebd9f755d3c.mov
@kilchenmann I also added a progress indicator to clarify that something is happening in the background after the user selects something. If you want to try on your end, I recommend throttling your internet speed in the dev tools.


Commas missing in setup.py

Installing from the repository's root directory gives an error:
pip install -e .
Obtaining extraneous-activity-delays
  Preparing metadata (setup.py) ... done
ERROR: Could not find a version that satisfies the requirement pandassetuptoolslxmlnumpyscipyhyperoptlog_similarity_metrics (from extraneous-activity-delays) (from versions: none)
ERROR: No matching distribution found for pandassetuptoolslxmlnumpyscipyhyperoptlog_similarity_metrics

In setup.py https://github.com/AutomatedProcessImprovement/extraneous-activity-delays/blob/main/setup.py, commas are missing in install_requires to separate each package, so everything is read as one giant string.

@iharsuvorau, It should be solved with latest commit!


Add presence support

Adds customizable presence support, does need testing before being pushed out

I like this, with a bit of tweaking it does work.
Line 39: activity = discord.Activity(type=activity_type, name=activity_name)) isn't actually being used and can be cut.
Line 40: because this bot is mega simple I wrote it using 'client' instead of 'bot,' so instead of bot.change_presence it needs to be client.change_presence
Then, in the line 40 type and name fields, you can stick the activity_type and activity_name variables so it's easier to change.
Finally, apparently the discord listening activity uses, "Listening to" as a base, because right now it says, "Listening to to squeaks." So it can be shortened to just "squeaks."
Sorry for the nitpicking. :/

should be good now!


Invalid response JSON when response is null

Description
When the response object is null the response is invalid:
{
,
"error": []
}

fixed in https://github.com/Telenav/kivakit-extensions/pull/43


SEC-1811: Change pinned ssl certificate

JIRA
SEC-1811

Description

New GDS ssl certificate deployed is pinned now
Old fingerprints removed

Checklists
Development and testing:

[ ] The code changed/moved as a part of this PR has been covered with tests
[ ] All tests related to the changed code pass in development
[x] Pull request is ready for review

lgtm


dunst: 1.3.2 -> 1.4.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/dunst/versions.
meta.description for dunst is: '"Lightweight and customizable notification daemon"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.4.0 with grep in /nix/store/np8ng55wafj2ixy4w0f33bq8gr3lk0bf-dunst-1.4.0
directory tree listing: https://gist.github.com/3af08688b6a5f7337bfa3d418e725943
du listing: https://gist.github.com/6b68b2f145d6d1130ab21d729fd6c236

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
dunst

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/np8ng55wafj2ixy4w0f33bq8gr3lk0bf-dunst-1.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A dunst https://github.com/r-ryantm/nixpkgs/archive/10e5d0f3f91657391a3d37028c3096fd81a82d9d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/np8ng55wafj2ixy4w0f33bq8gr3lk0bf-dunst-1.4.0
ls -la /nix/store/np8ng55wafj2ixy4w0f33bq8gr3lk0bf-dunst-1.4.0/bin

cc @domenkozar for testing.

Built and tested locally :+1:
Only platforms.linux is supported, so we can merge this now.


Fix Private Mode Usernames

So it turns out that if you try to print a userid of someone that isn't in that server, such as in a dm channel, it simply displays as invalid user (Even though userids are unique across the entirety of discord, got to love that logic). So now the private message instead uses the .username and .tag attributes to provide the normal userid, like so: @GenericUser#1234
While this may be a tad confusing to those using nicknames, it identifies the exact individual doing the tipping, it just might require a little digging.

thank you!


fix: remove platform-specific uuid code to prevent bundler errors

Description
The Web Crypto API is available in all browsers, Node.js, and Deno. This PR assumes that covers the majority of the environments where this polyfill will be used. A simple semi-random v4 formatted string is provided for all other cases. These changes reduce the code and complexity, and should resolve bundler errors stemming from the platform-specific imports and top-level awaits that have now been removed.
Issues

Partial fix for #32

Notes
I had trouble getting things setup on my machine, so I haven't run the tests, though I was able to build at some point along the way. I am currently relying on CI to tell me if I broke anything.
Checklist

[x] run yarn build
[ ] tests and/or benchmarks are included (N/A)
[ ] documentation is changed or added (N/A)
[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

Next Steps
Once some version of this PR is fixed, I'll follow up with a second PR to address the JSON parse ocde also reference in #32.

Thanks for the fix @fabiancook! I wasn't aware that the Node implementation of randomUUID was unbound.


[WFLY-18585]:  SimpleTimerMDBTestCase.testTimedObjectTimeoutMethod in…

…termittently fails
Cleaning up the code, I couldn't reproduce after 500 iterations.
Jira: https://issues.redhat.com/browse/WFLY-18585

More information about the wildfly-bot[bot]

Thanks @ehsavoie


Can't install GPMDPU on Fedora 29

Hi all,
I migrated from Fedora 28 to 29 and now, I can't install the app through its package bundle Fedora-64 bit

I get the above error
ENV:
Fedora - 29
Gnome - 3.30.2

@marquesarthur There is also Flatpak, which is not maintained by the developer but is up-to-date.
https://flathub.org/apps/details/com.googleplaymusicdesktopplayer.GPMDP

It seems that there's an issue with electron not being up-to-date in 4.6.1 stable that's causing issues with the latest versions of Ubuntu and Fedora.
See #3391 and specifically this comment for more information and an .rpm that I can confirm is working for me on Fedora 29

@choephel This is what I was looking for, thanks. Are you involved in development? If so, are there any plans for this to make it into the main release?

Closing as duplicate of #3391

@yerbestpal Not involved in development, but I feel certain that it will be incorporated into the main release as it both fixes a major problem that multiple users have reported and it's currently an element of the latest unreleased builds.


Add shadow_color

Description
Add shadow_color, new in 1.21.4

Testing

Checklist
Things need to be done before this Pull Request can be merged.

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt
[x] Code does not produce any clippy warnings: cargo clippy
[ ] All unit tests pass: cargo test
[ ] I added new unit tests, so other people don't accidentally break my code by changing other parts of the codebase. How?

I don't think you need to rename it

You need to, because it is shadow_color and not shadowColor
see here

Mojang with insanity consistent naming conventions -_-

Ah i though it just uses the field name, I didn't saw camelCase is used.

Looks good then.
Thanks @Commandcracker


Is SPI slave mode possible for next version?

Now only SPI Master Mode supported, is there any plan for feature versions that make it possible for azure sphere board working as a spi slave ?

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

ID: 8f9eb269-0b66-ffee-7ebf-1271d958b460
Version Independent ID: b8f0641a-6c52-a96f-7e4e-977f3a80f7e0
Content: Azure Sphere Applibs spi.h reference
Content Source: AzureSphere/articles/reference/applibs-reference/applibs-spi/spi-overview.md
Product: azure-sphere
GitHub Login: @esalc
Microsoft Alias: azurespheredocs

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

@Lulixue Yes, Currently Azure sphere only support Serial Peripheral Interface (SPI) in master mode. I haven't found information regarding the same. Kindly ask you to open a new feature request on azure sphere feedback forum here. As this is out of the scope of this doc.  We will close the issue and follow-up with you on the feedback forum. Please share the link when you do so.

https://feedback.azure.com/forums/915433-azure-sphere/suggestions/38958154-enable-spi-slave-mode


Compatibility with JuypterLab 3.0

JupyterLab 3.0 is available. The migration guide is here: https://jupyterlab.readthedocs.io/en/latest/extension/extension_migration.html

hope the next update could use the pre-build feature of jupyterlab 3, thanks 🙏

hope the next update could use the pre-build feature of jupyterlab 3, thanks 🙏

I really enjoy this package, would also very much appreciate a version compatible with Jupyterlab 3. Thank you!!

I really enjoy this package, would also very much appreciate a version compatible with Jupyterlab 3. Thank you!!

A little gratitude would go a long way :wink:
I published 0.4.1 which should address this. I don't use this package anymore, so I don't actively work on this myself.

A little gratitude would go a long way :wink:
I published 0.4.1 which should address this. I don't use this package anymore, so I don't actively work on this myself.

Works like a charm. Thanks for still maintaining the library, even that you don't use it - I ❤ it

Works like a charm. Thanks for still maintaining the library, even that you don't use it - I ❤ it


Fixed h2spec 5.1-11 test when sending a data frame to the server on a closed stream

on a closed stream.

Before the fix:
16:27:23 homer:/usr/local/var/log/trafficserver$ h2spec  -t -k -p 4443 http2/5.1/11 -v
Hypertext Transfer Protocol Version 2 (HTTP/2)
  5. Streams and Multiplexing
    5.1. Stream States
           [send] SETTINGS Frame (length:6, flags:0x00, stream_id:0)
           [recv] SETTINGS Frame (length:12, flags:0x00, stream_id:0)
           [send] SETTINGS Frame (length:0, flags:0x01, stream_id:0)
           [recv] WINDOW_UPDATE Frame (length:4, flags:0x00, stream_id:0)
           [recv] SETTINGS Frame (length:0, flags:0x01, stream_id:0)
           [send] HEADERS Frame (length:15, flags:0x05, stream_id:1)
           [recv] HEADERS Frame (length:109, flags:0x04, stream_id:1)
           [recv] DATA Frame (length:531, flags:0x01, stream_id:1)
           [send] DATA Frame (length:4, flags:0x01, stream_id:1)
           [recv] RST_STREAM Frame (length:4, flags:0x00, stream_id:1)
           [recv] Timeout
      × 11: closed: Sends a DATA frame
        -> The endpoint MUST treat this as a connection error of type STREAM_CLOSED.
           Expected: GOAWAY Frame (Error Code: STREAM_CLOSED)
                     Connection closed
             Actual: RST_STREAM Frame (length:4, flags:0x00, stream_id:1)

Failures:

Hypertext Transfer Protocol Version 2 (HTTP/2)
  5. Streams and Multiplexing
    5.1. Stream States
      × 11: closed: Sends a DATA frame
        -> The endpoint MUST treat this as a connection error of type STREAM_CLOSED.
           Expected: GOAWAY Frame (Error Code: STREAM_CLOSED)
                     Connection closed
             Actual: RST_STREAM Frame (length:4, flags:0x00, stream_id:1)

Finished in 2.0229 seconds
1 tests, 0 passed, 0 skipped, 1 failed

After the fix:
16:27:33 homer:/usr/local/var/log/trafficserver$ h2spec  -t -k -p 4443 http2/5.1/11 -v
Hypertext Transfer Protocol Version 2 (HTTP/2)
  5. Streams and Multiplexing
    5.1. Stream States
           [send] SETTINGS Frame (length:6, flags:0x00, stream_id:0)
           [recv] SETTINGS Frame (length:12, flags:0x00, stream_id:0)
           [send] SETTINGS Frame (length:0, flags:0x01, stream_id:0)
           [recv] WINDOW_UPDATE Frame (length:4, flags:0x00, stream_id:0)
           [recv] SETTINGS Frame (length:0, flags:0x01, stream_id:0)
           [send] HEADERS Frame (length:15, flags:0x05, stream_id:1)
           [recv] HEADERS Frame (length:109, flags:0x04, stream_id:1)
           [recv] DATA Frame (length:531, flags:0x01, stream_id:1)
           [send] DATA Frame (length:4, flags:0x01, stream_id:1)
           [recv] GOAWAY Frame (length:8, flags:0x00, stream_id:0)
           [recv] Connection closed
      ✔ 11: closed: Sends a DATA frame

Finished in 0.0354 seconds
1 tests, 1 passed, 0 skipped, 0 failed

I haven't tested with the change yet, but it may break other cases. If I remember correctly, the block is not only for stream count error.
You may want to read https://github.com/apache/trafficserver/pull/1753 .

@zizhong Are you working on the issue?

@maskit Originally the change came from #1262

Another option is to only send stream errors to id that are over our last connection id.  We have a pretty good idea that connections under that last id that we can't find in the stream list are close.

I went back to read the RFC again.  After rereading section 6.1 (https://tools.ietf.org/html/rfc7540#section-6.1), I believe we are doing the correct behavior and h2spec is incorrect:
If a DATA frame is received
   whose stream is not in "open" or "half-closed (local)" state, the
   recipient MUST respond with a stream error (Section 5.4.2) of type
   STREAM_CLOSED.

I think the test is for closed state on section 5.1 (https://tools.ietf.org/html/rfc7540#section-5.1).
      An endpoint that receives any frame other than PRIORITY
      after receiving a RST_STREAM MUST treat that as a stream error
      (Section 5.4.2) of type STREAM_CLOSED.  Similarly, an endpoint
      that receives any frames after receiving a frame with the
      END_STREAM flag set MUST treat that as a connection error
      (Section 5.4.1) of type STREAM_CLOSED, unless the frame is
      permitted as described below.

We are treating the both cases above as a stream error, which is incorrect according to this paragraph. The h2spec test case is for the latter case and #1753 tried to fix it, but the change was treating the both case as a connection error, which breaks the former case.

@maskit The server isn't receiving a frame with END_STREAM, it is sending a frame with END_STREAM.  I still think that h2spec is wrong.

This sounds like potentially a 7.1.x back port too ?

@zwoop I think the behavior is already correct and h2spec is incorrect.

@bryancall h2spec is sending a HEADERS frame with END_STREAM flag.

[send] HEADERS Frame (length:15, flags:0x05, stream_id:1)

Thanks Bryan for filing https://github.com/summerwind/h2spec/issues/88.


Can stable-ts be optimized for multi-CPU and multi-threading?

I found that there is not much difference in the results when running on 5-core, 8-core, and 1-core CPUs.
It seems that the computing power of some CPUs has not been fully utilized.
If audio could be partitioned for processing, perhaps it could be several times faster.
By the way, I want to compliment that this project is great. I have used the custom sentence segmentation and merging methods, and it's very useful. Thank you.

Have you tried to specify the threads with pytorch?
torch.set_num_threads(5)

I tried it  (torch.set_num_threads(5)) and the processing time didn't decrease.

PyTorch uses all the available cores by default. It is slower when I set threads to a lower count so torch.set_num_threads is working on my end.
Batch processing is not supported but you can try something similar to https://github.com/openai/whisper/discussions/153#discussioncomment-3746713.

Thank you, I will use other multithreading solutions to address this issue. Using more virtual machines would also work the same.


Sign a text message with NBitcoin

Hello,
I could not find a way to sign a simple text message with NBitcoin.
Can you point me in the right direction so I can sign the message and verify?
thanks
Paulo Aboim Pinto

You can't its been deleted on main repo, use a fork of NBitcoin like blockcore or something

Why this feature was removed?

https://github.com/MetacoSA/NBitcoin/issues/1094

You are totally right imo, but for the sake of keeping politics out of code (unfortunately this is not always the case) just use a fork of NBitcoin or implement yourself the message signing, it's not that hard.
https://github.com/block-core/blockcore/blob/master/src/Blockcore/NBitcoin/Key.cs#L121

I agree ... politics should be out of the code ...
thanks!!

BIP322 has been merged which brings back supports through it (it is backward compatible) See #1224 (files) for examples

https://github.com/MetacoSA/NBitcoin/pull/1224


Typescript example doesn't include styled-components

Describe the bug
According to README, Typescript example is supposed to include styled-components. I'll be happy to add it if agreed.
Expected behavior
styled-components package should be included at install and template components should use it if needed.

Typescript example is in fact totally broken (uses old getSiteProps and getRouteProps vs new getSitedData getRouteData functions) I assume when v6 lands, there should be some template cleanup to bring these up to date.


fix fix result error when cast a round decimalv3 to double

Proposed changes
Issue Number: close #xxx
in round function, when the src data is decimalv3 and the scale parameter is large than src data's scale, the round result scale will be set to source data's scale, but it use scale parameter to create the result column. This will result in the round result is wrong.
CREATE TABLE `tb_round_decimal` (
          `id` int NOT NULL COMMENT '',
          `d1` decimalv3(9, 4) NULL COMMENT '',
          `d2` decimalv3(27, 4)  NULL DEFAULT "0" ,
          `d3` decimalv3(38, 4)  NULL
        ) ENGINE=OLAP
        UNIQUE KEY(`id`)
        DISTRIBUTED BY HASH(`id`) BUCKETS 10
        PROPERTIES (
        "replication_allocation" = "tag.location.default: 1"
        );

insert into tb_round_decimal values (1, 123.56789, 234.67895, 345.78956);
insert into tb_round_decimal values (2, 123.56789, 234.67895, 345.78956);

select cast(round(sum(d1), 6) as double), cast(round(sum(d2), 6) as double), cast(round(sum(d3), 6) as double) from tb_round_decimal ;

before:
2.471356	  4.693578	6.91579

after:
247.1356	469.3578	691.579

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

@Gabriel39 help to review, this pr is for doris 1.2

run buildall


package.json: remove option '--open' from script 'dev'

When yarn dev is executed inside a docker container, --open requires a browser to be installed and an X11 server to be shared. Instead, it is more idiomatic to just bind a port and remove the option.
Developers that want to execute the script on the host and have the browser opened automatically, can still do so: yarn dev --open.
In fact, the command inside the container needs to be yarn dev --host <IP_ADDRESS>. Therefore, an alternative solution would be to keep script dev and add another one (say dev-docker).

I'm fine with this solution overall. I don't love that the browser opens to an error screen before the docs are built. I'm open to any better solution.

ping @agjohnson

I'm fine with dropping --open. We won't official support the workflow you are describing, but I don't think the open flag was important either way. I have a PR that makes more changes to the development server in #908, which addresses some of the issues with the webpack build flow, though the open call still seems to have sometimes before test docs are built.


No new tiles on panning?

Stack: Xamarin.Forms + UWP app + Mapsui 2.0.0-beta.2 with OSM layer from Microsoft store. On pan map there are no new tiles retrieved.

I don't understand this. If you look at the last image, than you could see the road with "Im Siel", which is correct loaded. Perhaps it needs mor time to load?

had to cut the video a bit in the end due to max size :-( it's more about the tile above Im Siel

will try to use the OSM https version instead of the default http version with 307 internal redirect

There have been some changes in uwp because it use the gpu when idle. In current master there are no new tile requests at all on pan. But RefreshGraphics is called. There are new tile requests on mouse wheel zoom.

I pushed 2.0.0-beta.4 this morning with the fix in panning in uwp. I will close this. When discussing this yesterday there seemed to be some other issues with slow data fetching as well. Let's open een new issue for this if we have to clearer.


Enhancement: Update the code so that we do not need to keep replacing the storage if there is an update.

Update the code so that we do not need to keep replacing the storage if there is an update.

This is done.  Instead of using for loop, we used mutate: https://github.com/Xode-DAO/xode-blockchain/commit/1efe5c11eea54869f9c24ce0970a710f745c012e


loom: patch file sync failed: I/O error

I was running my ship in GroundSeg with the nativeplanet/urbit docker image and my ship crashed with the error below:
loom: patch file sync failed: I/O error
Assertion '!"loom: patch sync"' failed in pkg/noun/events.c:727
Assertion failed: !"loom: patch sync" (pkg/noun/events.c: _ce_patch_sync: 727)
pier: serf unexpectedly shut down

Restarting the ship worked, additional information if useful:
When I boot the ship:
urbit 1.22
boot: home is /home/nal/nallux-dozryl
loom: mapped 2048MB
lite: arvo formula 2a2274c9
lite: core 4bb376f0
lite: final state 4bb376f0
loom: mapped 4096MB
boot: protected loom
live: loaded: GB/1.247.805.440

When I pack and meld:
> |pack
>=
~nallux-dozryl:dojo> serf: pack: gained: MB/7.743.812
pier: pack complete
ames: czar pet.urbit.org: ip .<IP_ADDRESS>
> |meld
>=
~nallux-dozryl:dojo> hash-cons arena:
  root: B/160
  atoms (2766963):
    refs: MB/70.491.560
    data: MB/360.665.075
    dict: MB/174.264.720
  total: MB/605.421.355
  cells (33370076):
    refs: MB/781.763.380
    dict: GB/5.059.678.880
  total: GB/5.841.442.260
total: GB/6.446.863.775

pier: meld complete

Glad the ship was able to restart-- if you have a chance will you try to reproduce this?

@matthew-levan unfortunately I don't know what caused this to begin with. I'll continue to monitor my ship closely, if it happens again I'll retrace what I (or my ship) did prior to the error. Sorry I can't be more helpful as of right now.

Ok I'm going to close this for now. Please re-open it if you see it again.

So the issue happened again. The logs weren't showing anything particularly interesting prior to the crash. (just a bunch of vita-client stuff vita-client: not sending activity. already sent.)
Running Vere 2.5
loom: patch file sync failed: I/O error
Assertion '!"loom: patch sync"' failed in pkg/noun/events.c:728
home: bailing out
pier: serf unexpectedly shut down

I cannot reproduce it again, not sure if this is helpful.


I don't know what is wrong with this on HA0.78.3

18-09-28 02:14:27 ERROR (MainThread) [asyncio] Task exception was never retrieved
future: <Task finished coro=<WebSocketWriter.ping() done, defined at /usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py:617> exception=RuntimeError('unable to perform operation on <TCPTransport closed=True reading=False 0x73a4e030>; the handler is closed')>
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 621, in ping
    return await self._send_frame(message, WSMsgType.PING)
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 603, in _send_frame
    self.transport.write(header + message)
  File "uvloop/handles/stream.pyx", line 671, in uvloop.loop.UVStream.write
  File "uvloop/handles/handle.pyx", line 159, in uvloop.loop.UVHandle._ensure_alive
RuntimeError: unable to perform operation on <TCPTransport closed=True reading=False 0x73a4e030>; the handler is closed
18-09-28 02:14:32 ERROR (MainThread) [asyncio] Task exception was never retrieved
future: <Task finished coro=<WebSocketWriter.ping() done, defined at /usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py:617> exception=RuntimeError('unable to perform operation on <TCPTransport closed=True reading=False 0x719009e0>; the handler is closed')>
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 621, in ping
    return await self._send_frame(message, WSMsgType.PING)
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 603, in _send_frame
    self.transport.write(header + message)
  File "uvloop/handles/stream.pyx", line 671, in uvloop.loop.UVStream.write
  File "uvloop/handles/handle.pyx", line 159, in uvloop.loop.UVHandle._ensure_alive
RuntimeError: unable to perform operation on <TCPTransport closed=True reading=False 0x719009e0>; the handler is closed
18-09-28 02:14:37 ERROR (MainThread) [asyncio] Task exception was never retrieved
future: <Task finished coro=<WebSocketWriter.ping() done, defined at /usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py:617> exception=RuntimeError('unable to perform operation on <TCPTransport closed=True reading=False 0x7198d9e0>; the handler is closed')>
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 621, in ping
    return await self._send_frame(message, WSMsgType.PING)
  File "/usr/local/lib/python3.7/site-packages/aiohttp/http_websocket.py", line 603, in _send_frame
    self.transport.write(header + message)
  File "uvloop/handles/stream.pyx", line 671, in uvloop.loop.UVStream.write
  File "uvloop/handles/handle.pyx", line 159, in uvloop.loop.UVHandle._ensure_alive

Please fill out the entire issue template.

请填写整个问题模板。

I ran version 0.78.3 of Hass.io and this error occurs every time I initially run it in the system log of hassio. I don't know what it is wrong.


Sprig code editor V2 not saving things properly on firefox on linux

it sometimes saves and sometimes doesnt so i have to keep backups

Are you able to give reproducible steps so I can test this? Saving works for me. When you say "sometimes doesn't", is it erroring, getting stuck, or saying saved but not actually saving?

Ofc, it says saved but when i close firefox and then open it again it often does not save even if it says its saved at the top
to reproduce this use firefox in pop os (pop os comes with firefox but i think it is installed as a flatpack) and also pop os is based on ubuntu
you go to the new editor and you code i don't know if copy pasting things changes the behavior of the save, maybe it only
saves things when you type and not when you paste things, it doesnt seem to be that tough because i can copy paste
sometimes and it works, i will try to see if i see anything in the console

do you also get this?

i don't know what else i can show you because i don't know whats happening other than that

I haven't been able to reproduce this at all. @grymmy do you think you could sanity check me and give it a go as well.

ok ill try my best :

turn on computer with Linux on it
open Firefox installed as a flatpack
go to the new editor
start a project
save it
write code
close firefox
open it again and see if the newest changes were saved if they were turn off your computer(idk if that changes anything) and do the steps 6 to 8 again

I will share a screen recording with more details too

Screencast from 2023-03-16 06:37:26 PM.webm
i also copy pasted the yellow text :
Loading failed for the <script> with source “https://plausible.io/js/plausible.js”. [uS9x9ZogVUQOOm5F75aY:28:1](https://sprig.hackclub.com/~/uS9x9ZogVUQOOm5F75aY)
Content-Security-Policy warnings 4
Content Security Policy: Ignoring “'unsafe-inline'” within script-src: ‘strict-dynamic’ specified [anchor](https://www.google.com/recaptcha/enterprise/anchor?ar=1&k=6LdopcUkAAAAADUyrxhp32xJ1PAh9pS9mmX00UPE&co=aHR0cHM6Ly9zcHJpZy5oYWNrY2x1Yi5jb206NDQz&hl=en&v=Trd6gj1dhC_fx0ma_AWHc1me&size=invisible&cb=pzmdl0yfj7ge)
Content Security Policy: Ignoring “https:” within script-src: ‘strict-dynamic’ specified [anchor](https://www.google.com/recaptcha/enterprise/anchor?ar=1&k=6LdopcUkAAAAADUyrxhp32xJ1PAh9pS9mmX00UPE&co=aHR0cHM6Ly9zcHJpZy5oYWNrY2x1Yi5jb206NDQz&hl=en&v=Trd6gj1dhC_fx0ma_AWHc1me&size=invisible&cb=pzmdl0yfj7ge)
Content Security Policy: Ignoring “http:” within script-src: ‘strict-dynamic’ specified [anchor](https://www.google.com/recaptcha/enterprise/anchor?ar=1&k=6LdopcUkAAAAADUyrxhp32xJ1PAh9pS9mmX00UPE&co=aHR0cHM6Ly9zcHJpZy5oYWNrY2x1Yi5jb206NDQz&hl=en&v=Trd6gj1dhC_fx0ma_AWHc1me&size=invisible&cb=pzmdl0yfj7ge)
Content Security Policy: Ignoring “'unsafe-inline'” within script-src: nonce-source or hash-source specified [anchor](https://www.google.com/recaptcha/enterprise/anchor?ar=1&k=6LdopcUkAAAAADUyrxhp32xJ1PAh9pS9mmX00UPE&co=aHR0cHM6Ly9zcHJpZy5oYWNrY2x1Yi5jb206NDQz&hl=en&v=Trd6gj1dhC_fx0ma_AWHc1me&size=invisible&cb=pzmdl0yfj7ge)
Partitioned cookie or storage access was provided to “https://www.google.com/recaptcha/enterprise/anchor?ar=1&k=6LdopcUkAAAAADUyrxhp32xJ1PAh9pS9mmX00UPE&co=aHR0cHM6Ly9zcHJpZy5oYWNrY2x1Yi5jb206NDQz&hl=en&v=Trd6gj1dhC_fx0ma_AWHc1me&size=invisible&cb=pzmdl0yfj7ge” because it is loaded in the third-party context and dynamic state partitioning is enabled.

Some cookies are misusing the recommended “SameSite“ attribute 2
Cookie “hideHelp” does not have a proper “SameSite” attribute value. Soon, cookies without the “SameSite” attribute or with an invalid value will be treated as “Lax”. This means that the cookie will no longer be sent in third-party contexts. If your application depends on this cookie being available in such contexts, please add the “SameSite=None“ attribute to it. To know more about the “SameSite“ attribute, read https://developer.mozilla.org/docs/Web/HTTP/Headers/Set-Cookie/SameSite [editor.95a811ac.js:217:162](https://sprig.hackclub.com/_astro/editor.95a811ac.js)
Cookie “outputAreaSize” does not have a proper “SameSite” attribute value. Soon, cookies without the “SameSite” attribute or with an invalid value will be treated as “Lax”. This means that the cookie will no longer be sent in third-party contexts. If your application depends on this cookie being available in such contexts, please add the “SameSite=None“ attribute to it. To know more about the “SameSite“ attribute, read https://developer.mozilla.org/docs/Web/HTTP/Headers/Set-Cookie/SameSite [editor.95a811ac.js:217:2057](https://sprig.hackclub.com/_astro/editor.95a811ac.js)
downloadable font: no supported format found (font-family: "Roboto" style:normal weight:400 stretch:100 src index:1) source: (end of source list)
Storage access denied, but a redirect would be bad for UX [legacy-migration.2bb16353.js:1:806](https://sprig.hackclub.com/_astro/legacy-migration.2bb16353.js)

and heres the neofetch output
patcybermind@pop-os:~$ neofetch
             /////////////                patcybermind@pop-os
         /////////////////////            -------------------
      ///////*767////////////////         OS: Pop!_OS 22.04 LTS x86_64
    //////7676767676*//////////////       Host: Alienware Aurora R9 1.0.14
   /////76767//7676767//////////////      Kernel: 6.2.0-76060200-generic
  /////767676///*76767///////////////     Uptime: 2 hours, 20 mins
 ///////767676///76767.///7676*///////    Packages: 2249 (dpkg), 52 (flatpak)
/////////767676//76767///767676////////   Shell: bash 5.1.16
//////////76767676767////76767/////////   Resolution: 1920x1080
///////////76767676//////7676//////////   DE: GNOME 42.5
////////////,7676,///////767///////////   WM: Mutter
/////////////*7676///////76////////////   WM Theme: Pop
///////////////7676////////////////////   Theme: Pop-dark [GTK2/3]
 ///////////////7676///767////////////    Icons: Pop [GTK2/3]
  //////////////////////'////////////     Terminal: gnome-terminal
   //////.7676767676767676767,//////      CPU: Intel i7-9700 (8) @ 4.700GHz
    /////767676767676767676767/////       GPU: NVIDIA GeForce RTX 2080 SUPER
      ///////////////////////////         GPU: Intel CoffeeLake-S GT2 [UHD Grap
         /////////////////////            Memory: 8827MiB / 15821MiB

i think and hope this is enough!

@Patcybermind Thank you for the excellent video and explanation of your setup.  For those looking for the money shot in the vid, 2:15 shows the editor clearly saying text was saved, and then firefox is closed and reopened to the same URL, and boom Pat's changes are gone.
RE: the Storage access denied, but a redirect error, looking at the code it seems that log is emitted during the decision-making process of whether to display some migration UI notification - doesn't seem related to this issue, but @kognise might be able to double-check me there...
Moving this to the bug queue with high priority.

Thank you SO SO much for your reproduction! I was able to reproduce macOS Firefox. Some details:
Graham is correct, the Storage access denied error is unrelated.
I discovered that the editor is actually saving the file properly! This bug only occurs if "Open previous windows and tabs" is enabled in Firefox preferences; for some reason when the tab is restored the original content loads in rather than the new content. If you reload the tab without making any changes, the code will actually be restored!
Will do further investigation into this to figure out what layer this bug is at.

Looks like I fixed this in 47b2368 (PR incoming) but for future regressions: this can be easily reproduced/tested with Ctrl-Shift-T.

@kognise you are right i just need to reload the tab!

Seems like this issue is happening again on firefox (see issue #1225)

a user also reported this issue happening on Edge


Feature/#106 refactoring front end sliders

Moves Slider in seperate file

I will bypass branch protection and prevent outdating again, because it has been approved before being outdated, and the merge from dev doesn't relate to this feature.


Added additional street types from NY State Post-Address Types

NY State has posted their state address database along with the metadata standards for consuming it. Included in this metadata is a guide to abbreviations, incorporated in this pull request.
This adds additional dictionary items for future model trainings.

Thanks for the update! Merged with a few modifications:

Removed "Rue" as it's covered by the French dictionaries. I looked into this case specifically while building the English dictionaries, and in every instance I could find, if a US address uses "Rue" as the actual thoroughfare type (i.e. not something like "Rue Street"), it means that address is in fact written in French, which libpostal considers a perfectly valid regional language in the US (e.g. many streets in New Orleans have French names).
Not everything that's an official post office abbreviation is a street/thoroughfare type. Things like "Forest" may be part of the street name but are not a proper thoroughfare suffix like "Street", "Lane", "Circus." Since the dictionary type is used in parsing, it's best to keep abbreviations that are not (or are rarely) thoroughfare types in separate dictionaries. The convention I've been using is to put general abbreviations in synonyms.txt which can apply to any address field. If the word/phrase is often associated with a venue/landmark name, I put them in place_names.txt.


DIAGNOSTIC - DO NOT MERGE - partition test design with intentionally added failures

This is https://github.com/alan-turing-institute/sktime/pull/3137 with two intentionally added failures (in Catch22Classifier and NaiveForecaster), in order to test that the failures are indeed picked up by the partition design.

works!


PYIC-3661: Add ability to set "context" for a CRI in the journey map

Proposed changes
What changed

Allowed states in the journey map to pass context to BuildCriOauthRequest to request from the CRI with the context included
Enabled ProcessEventHandler tests to access test journey map

Why did it change

The CIC CRI is currently tailored for a photo ID journey, but it will also be used in No Photo ID journeys. The content on the screens needs to be adaptable based on the use case.

Issue tracking

PYIC-3661

Checklists
Environment variables or secrets

[ ] No environment variables or secrets were added or changed

As mentioned on slack - I think you should check this in dev if possible, to work out whether you need to update the step function to handle passing the context.

I tested this by:

adding context to each cri state
deploying it to my dev environment
watching the network tab in developer tools when trying to authorize with a cri
passing the request JWT into a temporary test method:

void temp() throws Exception {
    JWEObject jweObject = JWEObject.parse("eyJjdQn1...Lyt9Vg");
    jweObject.decrypt(new RSADecrypter(KeyFactory.getInstance("RSA")
            .generatePrivate(
                    new PKCS8EncodedKeySpec(
                            Base64.getDecoder().decode(RSA_ENCRYPTION_PRIVATE_KEY)))));

    SignedJWT signedJWT = SignedJWT.parse(jweObject.getPayload().toString());
    System.out.println(signedJWT.getParsedString()); // put this into jwt decoder
}

following this, I put the parsed string into https://jwt.io/ to visualise the decrypted jwt

Manual test shows that I could get context through to the request to the cri:
{
  "sub": "urn:uuid:4c294365-df42-4367-81f0-2257008f140f",
  "shared_claims": {
    "address": [],
    "name": [],
    "birthDate": []
  },
  "iss": "https://dev-mikec.02.dev.identity.account.gov.uk",
  "response_type": "code",
  "client_id": "ipv-core-dev",
  "govuk_signin_journey_id": "cd63a67a-674f-4c56-b7d3-02de322cf983",
  "aud": "https://dcmaw-cri.stubs.account.gov.uk",
  "nbf":<PHONE_NUMBER>,
  "context": "bank_account", <<<
  "redirect_uri": "https://dev-mikec.02.dev.identity.account.gov.uk/credential-issuer/callback/dcmaw",
  "state": "S2ClC1tPJM0nEZ3P06rpYq3iomVMeDn_m7zAhnaQj4Q",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>
}


Anyone used Cloudflare with WebSocketBundle?

Anyone had any success to run WebSocketBundle together with cloudflare?

Anyone?


error?

I installed Conda with anaconda2 and Windows Subsystem for Linux.
I have strictly followed the steps in instructions, and tried the example named bench_ocrf,py
Here is the error information:
DeprecationWarning: This module has been deprecated in favor of the model_selection module into which all the refactored classes and functions are moved. Also note that the interface of the new CV iterators are different from that of this module. This module will be removed in 0.20.
"This module will be removed in 0.20.", DeprecationWarning)
/home/sy/anaconda/envs/OCRF_ENV/lib/python2.7/site-packages/sklearn/grid_search.py:43: DeprecationWarning: This module has been deprecated in favor of the model_selection module into which all the refactored classes and functions are moved. This module will be removed in 0.20.
DeprecationWarning)
OMP: Error #100: Fatal system error detected.
OMP: System error #22: Invalid argument
Aborted (core dumped)

Hi, I don't think the deprecation warnings are a problem. Looks like it's more about OpenMP windows. Try to set the environment variable KMP_AFFINITY=disabled.
see: https://github.com/Microsoft/BashOnWindows/issues/785.


Edit Cluster Badge modal validation error

Setup

Rancher version: v2.10
Browser type & version: Chrome Version 129.0.6668.90

Describe the bug
Edit Cluster Badge modal has some validation wrong behaviours related to the colour.

The colour field will look disabled no matter if it's on or off.
However, a colour can be picked no matter if the field is on or off.
The "Apply" button will only be active if Cluster badge or Cluster comment are active and fulfilled.

To Reproduce
Go to a Cluster dashboard and click the brush button to open the Edit Cluster Badge modal.
Screenshots

Trying to validate in v2.10.0-alpha11, but when I click "Apply", the color does not get updated. The custom badge and cluster comment both update appropriately, so just the color isn't being accepted now. And the form validation itself is working, where they are disabled until the checkboxes are clicked and the Apply is available for any field update.

I recently tested this on head and was able to update the color without issue. I will need to follow up with a v2.10.0-alpha11 deployment to confirm that there is no issue, but the fix should be same in both Dashboard head and v2.10.0-alpha11.
Screencast_20241111_200831.webm

This is working based on this issue. I've opened https://github.com/rancher/dashboard/issues/12582 to resolve the other issue I saw.


Dropping old numpy versions

Astropy v2.0 is going to drop support for numpy 1.7 and 1.8, so I propose to drop support for them here, too. I'll convert this to a PR if you agree with this.

@bsipocz - Agreed. Please go ahead.


No way to create signals in C#.

Godot version:
Godot 3.0 beta 2
OS/device including version:
Windows 10
Issue description:
No way to create a signal in C#.
I have searched alot and was unable to find a way to add signals from C# even came across SignalAwaiter but to no avail it seems that Godot C# currently doesn't has a way for adding signals from C# so I would like to request addition of the feature until then I will be using GDScript.
Although receiving works just fine.

[x] I searched the existing GitHub issues for potential duplicates.

Pretty sure you can call add_user_signal() from C#, it should be available through the binding (it should be called AddUserSignal).

It's not that it's more like I can not create a Signal which will be visible through the inspector's Node tab.
Like in GDScript it was as simple as adding signal my_signal and then using emit_signal().

To create a custom signal, Godot.Object.AddUserSignal can be used to register the signal. Once registered, you can use Godot.Object.EmitSignal to emit the signal up the node tree.
You can then use Godot.Object.Connect to connect the signals to the appropriate event handler.
i.e.
Child pseudo code:
using Godot.Object;

public class Gem
{
    public override void _Ready()
    {
        AddUserSignal("gem_grabbed");
    }

    ...

    void PickedUp()
    {
        EmitSignal("gem_grabbed");
    }
}

Parent pseudo code:
using Godot.Object;

public class Main
{
    int _score;
    PackedScene _gem = ResourceLoader.Load(@"res://Gem.tscn") as PackedScene;

    public override void _Ready()
    {
        var g = _gem.Instance();
        AddChild(g);
        g.Connect("gem_grabbed", this, nameof(GemGrabbed));
    }

    public void GemGrabbed()
    {
        _score += 10;
    }
}

I certainly can add something like this if it feels needed. I personally don't think it is usefull though.
But now that I think of the Signal names: should I do some PascalCase to snake_case translation?

Automatic conversion to snake_case seems like a bad idea, or at least if there isn't a way to turn it off. In my opinion, having a way to override the name (as in my example above) or just writing your delegates as delegate void snake_case() would be better.

I too think that translation or conversion is not a good idea. And having the ability to override the name is I loved quite a bit while fiddling with CSTools.

Looks like this issue was solved in version 3.01 but I cannot find any information how it should be used correctly. Can anybody give me a short example for C# ?
Thanks

@Galileo53 I think something like this:
public class MyCoolScript
{
    [Signal]
    delegate void StoppedBeingCool(string reason);

    public void MakeUncool()
    {
        Emit("StoppedBeingCool", "You know why.");

        // or better (friendlier to refactoring tools)
        Emit(nameof(StoppedBeingCool), "You know why.");
    }
}

It is EmitSignal I think, but yes.
I should probably PR the documentation on this. Will do as soon as I have some time 🙂.

This should be more like it.
public class MyScript
{
    [Signal]
    delegate void StoppedAction(string whatHappened);
    [Signal]
    delegate void GivenChoices(string[] choices);

    public bool stopAction = false;

    public void Execute()
    {
        // IF action is action is stopped
        if (stopAction)
        {
            EmitSignal(nameof(StoppedAction), "INVALID INFO");
        }
        else
        {
            //using other types works too
            string[] strar = new string[] {"YES", "NO"};
            EmitSignal(nameof(GivenChoices), strar);
        }
    }
}

The documentation about this has been updated.
On another note @neikeq, I think #11956 is a duplicate from this one, no?

Thank you so much. It works now :-)

@paulloz Yep, that's a duplicate.

With your help I finally have ported the "Dodge The Creeps" (= "Your First Game" from the Godot Docs) to C#. As far as I know, this is the first successful porting - there have been some guys around trying it but they failed. Possibly this was because of the missing [Signal] support.
I would like to share this code because it took me a lot of time to create and I feel that it is a good collection of examples and it may help others to save time. Sorry if this is off-topic here but since I'm pretty new with Godot, I have no idea how I can post the code or send it to someone or if anybody is interested at all.

FYI, I've also ported this tutorial from GDScript to C# a few weeks ago
The source code can be found on gitlab
https://gitlab.com/GodotCSCourses/DodgeTheCreeps

Sorry, I did not know this.
So of course there is no need for another port.

No don't worry , that's not what I meant with my comment.
All jobs on Godot are good, specially on C#.
I also used another port to c# to created my code, because some informations can be hard to find in doc or support.
So having a C# code to look at is really helpfull/

If you stumble upon something that's missing or is inaccurate in the doc, feel free to report it on godot-docs :grin:

You're right ! Of course


ImageView example provides rel path not working

The ImageView example provides a relative path but Image.load_image
requires absolute path in Cocoa. This patch allows load_image to accept
a relative path.
Signed-off-by: William Clemens<EMAIL_ADDRESS>
Will not resolve relative paths for Cocoa.

Image widget didn't resolve relative paths with Cocoa. This caused the exmpale imageview app to be broken.

PR Checklist:

[X] All new features have been tested
[X] All new features have been documented
[X] I have read the CONTRIBUTING.md file
[X] I will abide by the code of conduct

@wesclemens Thanks for making those updates! Let me know when you are done testing and then we'll merge it in.

@danyeaw, @adamyala ran it on his mac book this morning. All looks good.

Thanks @wesclemens and @adamyala! :clap:


IntelliJ not finding dependency.

When I run an inspection Maven gives the following error: Dependency com.github.Xezard:XGlow:{latest_version} not found. How do I solve this?

{latest_version} replace this with latest version of XGlow.


[feature] add option to disable converting ISO strings to dates

This adds a new load option - disableAutoISOConversions. This turns off the automatic conversion of event fields where any strings that appear as an ISO8601 string (including just dates, e.g. YYYY-MM-DD) become Date objects.
Background
Through @segment/facade, any events passed to classic integrations had any ISO strings converted to Dates. I suspect this was intentional behavior because the way we cloned events (JSON.parse(JSON.stringify(event))) converted Dates to ISO8601 strings, so facade was converting them back to dates.
However, some users want their ISO strings to be passed to integrations as is, especially when the string represents a calendar date and not a time.
PR details
@segment/facade supports turning off this conversion by setting traverse to false, so the new option ultimately sets the traverse flag.
I also changed how we clone events so that users can still specify a property as a Date object and have that carried over to the cloned event regardless of the new setting. It looks like analytics.js classic also did a deep clone of the event instead of our JSON.parse(JSON.stringify(event)) trick, so this shouldn't cause any unintended consequences.
I made this field default to false, meaning users need to opt-in to this new behavior. I made it opt-in to prevent potentially impacting user-defined destination middleware.
[x] I've included a changeset (psst. run yarn changeset. Read about changesets here).

Very happy to get rid of JSON.parse(JSON.stringify(event))!


(void) show sometimes not working (depending on the situation )

This is limited.
//    UIWindow *window = [[[UIApplication sharedApplication] windows] lastObject];
//
//    //如果发现键盘windows 出现,就选键盘下一个 windows
//    if ([window isKindOfClass:[NSClassFromString(@"UIRemoteKeyboardWindow") class]]) {
//
//        window = [[UIApplication sharedApplication].windows objectAtIndex:([UIApplication sharedApplication].windows.count -2)];
//    }
//
//    [window addSubview:self];

use delegate.
[_delegate addSubview:self];
more effective

thanks,

can you tell me more detail about "limited";

I do not know the exact cause, but sometimes it does not appear on the screen and there is no response.
will continue to look for the cause.


Output Tweaking

Just a small feature request. Could you exclude the null chars in output? It breaks a lot of processing of the output.

Also, there seems to be a formatting issue with integer rendering when the hex value is 1 char. A space between 0x and the integer. I know they are small things, but, it helps a lot when trying to serialize for ingestion or post processing.

Will fix this today :)

should be fixed in #13.
Thanks for letting me know of these small things :)

Another formatting request.
For binary blobs, can you make the output a hex string [Exmple: '000000000400000028C6AB67E9AA6AB4...']?
The list of ints is harder to post process. Example of current output (PartitionTable, MBR, etc.):

added to #13. :)

@devgc added to 0.1.8 patch :shipit:


GetRequired Channel Extension

A handy extension method that throws an exception when the channel is not found!

Added! 😀 05a0020f


[FEATURE] Add RemoveElementTceFormsFractor

This Rule is currently not (easily) possible with the current approach as we only support one node but I need multiple nodes here or a good entry point selector which is also quite difficult.
What are your ideas?


for attribute in label does not correctly set focus on element

The label never focuses the select element, as the element is processed by the Select2 plugin and given a slightly modified ID.

@notmessenger picking this one up

There is an input element with the classes of select2-focusser select2-offscreen applied to it.  This element's id should be used for the label's for value.  This will mimic the focusing of the element when the label is clicked just like normal label for usage.  Do not open the control upon selection.

@notmessenger that is applicable for single select boxes, but not multiple. Multiple select boxes have a input with select2-input, but setting label for to this causes the selection to open when the label is clicked. I think if one type of select is going to open both should for consistency.

@joshforisha Want to lend your opinion?  Still tying into the click event which we're not liking so need to decide if multiple options auto-open, which might be the acceptable compromise here.  Using the label for focus is discussed in the Select2 repo at https://github.com/select2/select2/issues/2311

Sorry, should have clarified: the select2 focus event only gives focus, it doesn't open the selection. However, using for does open the selection on multiple select boxes.

Replaced by https://github.com/softlayer/sl-ember-components/issues/456


Fixed base_url definition

Changed conditional statements to avoid issues when there is more than one 'notebook' word in the URL.

Thanks @ingdanielguerrero for opening a PR.
I think my previous code was really a hack, so I'm not sure we should try to improve it, but rather think about a new solution.
See, having /voila/ in the URL doesn't mean we are in Voila, since it can just be a subdirectory where the notebook lives.

I guess we could also pass a get_base_url(window_url) function, that would return the base_url to use in xarray_leaflet (window_url would be the window URL that you see in your browser).
That would be a general solution to fall back when our solution doesn't work, what do you think?

@ingdanielguerrero I added a get_base_url parameter in #25, could you please try it?
You can pass a function such as:
def get_base_url(window_url):
    return my_custom_url

Perfect! Thank you @davidbrochart, that is a good generic solution! I just tested with my own callback and it's working well.

Perfect! Thank you @davidbrochart, that is a good generic solution! I just tested with my own callback and it's working well.

Thanks, good to know.

And you also removed the zoom prints ;).

Yes! These were some leftover.
I'll make a release soon.


Add viewer, protocol and crypto updates

Fix #58 and #57

Should be ready for review. The viewer should be moved out later on as it’s gotten a bit too big.


A trajectory with velocity only is not possible

I believe it used to be possible to publish a velocity-only command of type trajectory_msgs/JointTrajectory to a joint_trajectory_controller/joint_trajectory topic. That's not possible anymore:
Mismatch between joint_names size (7) and positions (0) at point #0.
That warning comes from here:  https://github.com/ros-controls/ros2_controllers/blob/9b344c7b5c06879b85cd4bcc135e35b5f7eedc2f/joint_trajectory_controller/src/joint_trajectory_controller.cpp#L1343
I tried commenting that vector size check but then the control node just crashes.
(I'm testing on Humble branch.)

Isn't allow_integration_in_goal_trajectories what you are looking for?

It looks like the logic related to allow_integration_in_goal_trajectories isn't correct. Note this isn't a big issue for me, just wanted to point out the issue.
    if (params_.allow_integration_in_goal_trajectories)
    {
      const bool all_empty = points[i].positions.empty() && points[i].velocities.empty() &&
                             points[i].accelerations.empty();
      const bool position_error =
        !points[i].positions.empty() &&
        !validate_trajectory_point_field(joint_count, points[i].positions, "positions", i, false); // SHOULD BE TRUE FOR ALLOW_EMPTY?

Why do you think?
If there are position/velocity/acceleration fields but not of the correct size, or all are empty, then it returns false.
If they are not complete but got accepted, they states will be integrated here
https://github.com/ros-controls/ros2_controllers/blob/f64c964b2ed772d7d75e8d7e5d46a1f35c8159bb/joint_trajectory_controller/src/trajectory.cpp#L172-L176

Thanks for the hint on [allow_integration_in_goal_trajectories](https://github.com/ros-controls/ros2_controllers/blob/humble/joint_trajectory_controller/src/joint_trajectory_controller_parameters.yaml#L62-L66). I've added that but weirdness is still happening.
I'm not sure what the cause is. I'll try to give a thorough bug report since you're asking for more detail.
ros2 control list_controllers shows the controller is active:  robin_velocity_controller joint_trajectory_controller/JointTrajectoryController  active
allow_integration_in_goal_trajectories is set to true now, thanks for that hint:
ros2 param dump robin_velocity_controller -> allow_integration_in_goal_trajectories: true
The velocity interfaces seem to be claimed as they should be. 7 are claimed:
command interfaces
	elev1_j1/position [available] [unclaimed]
	elev1_j1/velocity [available] [claimed]
	elev1_j2/position [available] [unclaimed]
	elev1_j2/velocity [available] [claimed]
	elev1_j3/position [available] [unclaimed]
	elev1_j3/velocity [available] [unclaimed]
	wrist_pitch/position [available] [unclaimed]
	wrist_pitch/velocity [available] [claimed]
	wrist_roll/position [available] [unclaimed]
	wrist_roll/velocity [available] [claimed]
	wrist_yaw/position [available] [unclaimed]
	wrist_yaw/velocity [available] [claimed]
	yaw1/position [available] [unclaimed]
	yaw1/velocity [available] [claimed]
	yaw2/position [available] [unclaimed]
	yaw2/velocity [available] [claimed]

Here's the code which sends out my action request to the correct 7 joints, it seems:
  traj_client_ptr_ =
      rclcpp_action::create_client<FollowJointTrajectory>(node_, "/robin_velocity_controller/follow_joint_trajectory");
...
BT::NodeStatus SendStopCommand::onStart()
{
  if (!traj_client_ptr_->wait_for_action_server())
  {
    RCLCPP_ERROR(node_->get_logger(), "Action server not available after waiting");
    return BT::NodeStatus::FAILURE;
  }

  trajectory_msgs::msg::JointTrajectoryPoint traj_point_msg;
  traj_point_msg.velocities = std::vector<double>(0.0, JOINT_NAMES.size());
  // TODO: design spec of the joints says they should be able to decelerate in 2s, so we use that value here.
  // Likely they could safely stop faster than that.
  // Try something like:
  // Find the joint which would take longest to decelerate.
  // Use that value as `time_from_start`
  builtin_interfaces::msg::Duration time_from_start;
  time_from_start.sec = STOPPING_SECONDS;
  traj_point_msg.time_from_start = time_from_start;

  trajectory_msgs::msg::JointTrajectory traj_msg;
  traj_msg.joint_names = JOINT_NAMES;
  traj_msg.points.emplace_back(traj_point_msg);
  traj_msg.header.stamp = node_->get_clock()->now();

  FollowJointTrajectory::Goal goal_msg;
  goal_msg.trajectory = traj_msg;

  // Send the goal and wait for the result
  traj_goal_future_ = traj_client_ptr_->async_send_goal(goal_msg);
  if (rclcpp::spin_until_future_complete(node_, traj_goal_future_) != rclcpp::FutureReturnCode::SUCCESS)
  {
    RCLCPP_ERROR(node_->get_logger(), "Failed to send goal");
    return BT::NodeStatus::FAILURE;
  }

  auto goal_handle = traj_goal_future_.get();
  if (!goal_handle)
  {
    RCLCPP_ERROR(node_->get_logger(), "Goal was rejected by server");
    return BT::NodeStatus::FAILURE;
  }

  // Wait for the traj to complete
  RCLCPP_INFO(node_->get_logger(), "Waiting for the traj goal to complete");
  return BT::NodeStatus::RUNNING;

I see this print from the traj controller:  [ros2_control_node-5] [INFO] [1733098757.168904128] [robin_velocity_controller]: Received new action goal
My app prints RCLCPP_ERROR(node_->get_logger(), "Goal was rejected by server");.

This is weird. The goal gets rejected only if validate_trajectory_msg returns false, and for every return false there should be something in the error log. (except for all_empty, I'll add a PR for that). Maybe you could build it from source and debug this method?


Superscript create new lines

Hi,
I have issue when I convert a docx to html.
When the docx contains a Superlink the layout is screwed.
Docx
Docx file
Hi I’m the 1st  and this should be on the same line.
Html Output
Html file
Hi I’m the   1
          st
            and this should be on the same line.

I found the culprit!
span { white-space: pre-wrap; }
Is there a way to work around this issue ?

TestSuperScript.docx

Here are the files for the issue
Issue 165.zip

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


why the time varies between different epoches in YOLOX

As the figure of time shown, the time between epochs varies a lot.
I have checked the features which just are used in YOLOX, like hooks and transforms, and no clue is found. Only "SyncRandomSizeHook" would change the size of the input, but it happens in different iterations.
Do you know any idea?

@RaymondByc Between each epoch, there will be closing dataloader multi-process and opening multi-process operation, which is very time-consuming. You can refer to the log we provide:
{"mode": "train", "epoch": 1, "iter": 1800, "lr": 0.00038, "memory": 3446, "data_time": 0.04779, "loss_cls": 2.66138, "loss_bbox": 3.53173, "loss_obj": 5.41322, "loss": 11.60633, "time": 0.53712}
{"mode": "train", "epoch": 2, "iter": 50, "lr": 0.00042, "memory": 3446, "data_time": 1.94365, "loss_cls": 2.60153, "loss_bbox": 3.59438, "loss_obj": 4.99105, "loss": 11.18696, "time": 2.48093}
{"mode": "train", "epoch": 2, "iter": 100, "lr": 0.00044, "memory": 3446, "data_time": 0.02042, "loss_cls": 2.60903, "loss_bbox": 3.56643, "loss_obj": 4.95299, "loss": 11.12845, "time": 0.55071}

You misunderstand what I mean. What I concern about is the changes between epochs instead of iterations.
[https://download.openmmlab.com/mmdetection/v2.0/yolox/yolox_tiny_8x8_300e_coco/yolox_tiny_8x8_300e_coco_20210806_234250.log.json](The log you post) do not have the changes because the tiny config does not have "Mixup".
You can check the changes between epochs in yolox_s_8x8_300e_coco.py.

The bug I found is that "update_dynamic_scale" in MultiImageMixDataset Class cannot really change the self._dynamic_scale.
You can found that when you print the self._dynamic_scale in get_item.

Change the dataset in dataloader can be done by "runner.data_loader.dataset.update_dynamic_scale()"? I think it would not works.

Change the dataset in dataloader can be done by "runner.data_loader.dataset.update_dynamic_scale()"? I think it would not works.

This is indeed a bug, you can refer to https://github.com/open-mmlab/mmdetection/pull/6144

Hello, why do I use yolox model very slowly?


Add Changelog

It would be nice if there was a place on the website where we could see changes between versions.

All changes are tracked in the releases https://github.com/Shopify/themekit/releases would you still like a file containing all of that information?

Well, I did state, "... on the website..." but that will do to. Thanks.

@tanema Awesome. Will this changelog be displayed on the site or just on GitHub in a file?

I have decided to keep the change information in the releases section. Generating a changelog from already existing and public resources seems like needlessly duplicating information and adding extra maintenance load to a repo I already have little time to work on.


Product with recently answered questions has inconsistent data on Smoothie

See details in this issue that was incorrectly filed on the server: https://github.com/openfoodfacts/openfoodfacts-server/issues/6290

Fixed by #929


Browserify bundling fails when using pixi from node_modules

Hi guys,
So I am facing annoying problem after upgrading to v4.1.0. If I require pixi from node_modules and not use binaries directly it ruins my browserify bundle throwing
>> Error: Cannot find module 'babelify' from 'C:\Users\Artjom\test\node_modules\pixi.js'
Happens because my browserify task sees 'browserify' property in pixis package.json and tries to do the transform, but babelify and its presets are included as dev dependencies only. And I am not using ES6 on my side.
Also, "main": "./bin/pixi.min.js", shouldn't it be pointing to entry point? Like is used to be before in v3
"main": "./src/index.js",?
Just want some clarification on this one :)

Hello, this is a problem for me too and is still there in v4.1.1. Manually editing the package.json in the manner @Zenext describe (moving babel to dependencies and changing "main") solves the problem, though it's not a very nice solution for lots of reasons. Can you please fix this to your next release?
And btw thanks for a awesome project. My professional life would be way less enjoyable without Pixi.

Some context behind why main points to the built file:
https://github.com/pixijs/pixi.js/pull/2981
https://github.com/pixijs/pixi.js/pull/3239
I think the solution here is to add a browser field so that browserify uses the source and everything else uses the built file. If you add "browser": "./src/index.js" does that fix your issue?

Yes that seams to work. Though babelify and it's preset is needed. Is there a problem with adding them as a dependency instead of dev-dependency?

This looks to be fixed via #3289, can anyone confirm?

This has been fixed in 4.2.2.


SSR mode?

If you are reporting a problem with Token Wizard, please include the following information:
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
Kovan
If you were able to create it, what is the URL of your crowdsale?
Later
Do you have screenshots showing the problem?
No, I just want ask ooooooooooooone question..Orz
Do you see errors in the dev console? If yes, please include a screenshot
To open the dev console in Google Chrome, press F12, or go to View -> Developer -> Developer Tools, and open the Console tab
If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.
this project is the SSR mode ? I recently studied Mobx and accidentally discovered this project and found it very good.

@xoxoj Token Wizard works in CSR mode only. Thus, to answer your question: no, it has no SSR mode. But you can contribute any time. Take a look the project PR policy https://github.com/poanetwork/token-wizard/wiki/Pull-Request-Policy.


tests of subdirectories are not skipped

I am using Mac. By looking at the source code, the '*_test.go' is in skip patterns by default.
However it doesn't work for tests in subdirectories (at least this is the case in Mac).
For example, if I run 'gas ./...' under /Users/xxx/Workspace/goprojects/src/a
then the results of the three patterns used in filelist.go are:
pathname:  /Users/xxx/Workspace/goprojects/src/a/b/timeTransformer_test.go
pattern 1: *_test.go - not matched
pattern 2: /Users/xxx/Workspace/goprojects/src/a/*_test.go - not matched
pattern 3: **/*_test.go - not matched
I think the quick fix is to make the third pattern join 'cwd' as well
// match file suffixes ie. *_test.go if matched, _ := filepath.Match(filepath.Join(cwd, "**", pattern), pathname); matched {
I've tested then the test in subdirectories is skip.

I can confirm this on windows, see #51


Update Mekanism API.

Fixes #124
Note, untested as I missed a lot of dependencies, should work with latest API.

I dont know how to update the Mekanism API  I dont know a lot about developement stuff.

This is a code update for the maintainer of Flux-Networks. If he/she accepts this he/she could release a new version which fixes the issues you're having. It's not something you could do that easily.

oh okay

Would be nice if somebody would do something with this...I can't update Mek in my pack until it's fixed unless I just want to remove FN and I don't really want to do that.

@xAlicatt We (team Mekanism) are working on a possible 1.10 release of Mekanism which doesn't crash.
This should still be merged to restore full functionality, but it allows users to update to the latest version of Mekanism.

@thommy101 that's good news... Will  be looking forward to seeing it, but hopefully someone over here will wake from the dead soon :p

@xAlicatt fix from Mekanism for 1.10 has been released

Thanks for PR. I've fixed this in my own workspace for SonarCore and FluxNetworks.


Huawei HarmonyOs

Hello all.
As you know last year Huawei devices start to come with new mobile system HarmonyOs. I dont have chance to try model viewer and augmented reality on thesee devices. and I know HarmonyOs have AR Engine too. Is it working on HarmonyOs too ? or do I have to add specific codes or scripts for it.
Anyone tried on HarmonyOs for model viewer's Augmented Reality ?
Thanks...

I have no idea, but I would also be interested in the results. I would expect the 3D mode to work fine, but AR is a bit more questionable.