Question and a Bug

The format of your URL matters if you don't have meta data in your headers. This exact formatting is not clearly documented and I'd like more examples of this. I'm running Confluence 7.13.7 hosted on prem. I'm not sure if I'm using the wrong url format, but when I use this following format (and confirm the page works before deploying) with no meta data, mark crashes.
This URL format with no meta-data :
/pages/viewpage.action?pageId=
a command like this:
./mark -c ~/.config/mark/wiki-dev.cfg -l "https://<base_urL>/pages/viewpage.action?pageId=122781799" -f ~/test/my_doc.md --dry-run
or this
./mark -c ~/.config/mark/wiki-dev.cfg -l "<base_url>/pages/viewpage.action?pageId=122781799" -f ~/test/my_doc.md --dry-run
Throws this this error:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x20 pc=0x100b5ecf4]

goroutine 1 [running]:
main.processFile({0x1400001c780, 0x48}, 0x1008860f8?, {{0x16f5af560, 0x2d}, 0x0, 0x0, 0x0, 0x1, 0x1, ...}, ...)
        github.com/kovetskiy/mark/main.go:208 +0x1f4
main.main()
        github.com/kovetskiy/mark/main.go:159 +0x51

I get the same output if I use:
"<source_url>/pages/editpage.action?pageId=122781799"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x20 pc=0x10234ecf4]
goroutine 1 [running]:
main.processFile({0x16ddbf4c3, 0x1d}, 0x1020760f8?, {{0x16ddbf4c3, 0x1d}, 0x0, 0x0, 0x0, 0x0, 0x1, ...}, ...)
github.com/kovetskiy/mark/main.go:208 +0x1f4
main.main()
github.com/kovetskiy/mark/main.go:159 +0x510

Duplicate of https://github.com/kovetskiy/mark/issues/228
Please try again with a later version of mark and report back if the issue still exists.


Aubess not flashable

for more dan one day i try to flash an aubess mini switch V1.2.1
i followed the steps on: https://github.com/tuya-cloudcutter/tuya-cloudcutter/blob/main/HOST_SPECIFIC_INSTRUCTIONS.md
even the docker install option in folder tuya-cloudcutter. All with succes.
i followed the steps on: https://github.com/tuya-cloudcutter/tuya-cloudcutter/blob/main/INSTRUCTIONS.md
ran the script ./tuya-cloudcutter.sh -w wlan0 -s << myssd>> <>
After the scriptcycle and getting a succes
device is after the run disconected from tuya cloud but does not connect to my network.
tried again without extra parameters ./tuya-cloudcutter.sh
followd the steps but again a succes but device is not up in my network.
the scriptrun gives at some point an error on the wlan: can't connect to kernel. but gives overall succes with at the end an deviceid and local key. If i put the device in AP mode it shows up as A-728B but i cant connect to it in order to redirect it to the cloudcutter.
The device is, as mentioned not accessible with a tuya based app but also does it not show up in my network. i'm out of clues.
log:
Press any key to continue, or CTRL+C to exitBuilding cloudcutter docker image
[+] Building 1.7s (13/13) FINISHED
 => [internal] load .dockerignore                                                                                0.1s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load build definition from Dockerfile                                                                                0.1s
 => => transferring dockerfile: 457B                                                                                0.0s
 => [internal] load metadata for docker.io/library/python:3.9-slim-buster                                                                                1.4s
 => [base 1/2] FROM docker.io/library/python:3.9-slim-buster@sha256:c79e572f1ef9e9f30343980472f68986fcaf4c41fee67a3ef7536f20ca1f65ed                               0.0s
 => [internal] load build context                                                                                0.1s
 => => transferring context: 3.02kB                                                                                0.0s
 => CACHED [base 2/2] RUN apt-get -qq update && apt-get install -qy --no-install-recommends git hostapd rfkill dnsmasq build-essential libssl-dev iproute2 mosqui  0.0s
 => CACHED [python-deps 1/4] RUN pip install --upgrade pipenv                                                                                0.0s
 => CACHED [python-deps 2/4] COPY src/Pipfile /src/                                                                                0.0s
 => CACHED [python-deps 3/4] COPY src/Pipfile.lock /src/                                                                                0.0s
 => CACHED [python-deps 4/4] RUN cd /src && PIPENV_VENV_IN_PROJECT=1 pipenv install --deploy                                                                       0.0s
 => CACHED [cloudcutter 1/2] COPY src /src                                                                                0.0s
 => CACHED [cloudcutter 2/2] WORKDIR /src                                                                                0.0s
 => exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:1605f19d16f11fcc9bb97e30077c9dba6cc19a4283e9fc0b1a5220fa4afea2cc                                                                       0.0s
 => => naming to docker.io/library/cloudcutter                                                                                0.0s
Successfully built docker image
Loading options, please wait...
[?] How do you want to choose the device?: By firmware version and name
   By manufacturer/device name
 > By firmware version and name
   From device-profiles (i.e. custom profile)

[?] Select the firmware version and name: 1.2.1 - BK7231N / oem_bk7231n_control_switch
   1.1.71 - BK7231T / bk7231t_common_user_config_ty
   1.1.8 - BK7231N / oem_bk7231n_plug
   1.1.8 - BK7231T / oem_bk7231s_rnd_switch
   1.1.80 - BK7231T / bk7231t_common_user_config_ty
   1.1.9 - BK7231T / bk7231s_common_iot_config_ty
   1.1.9 - BK7231T / oem_bk7231s_ceiling_light_ty
 > 1.2.1 - BK7231N / oem_bk7231n_control_switch
   1.2.16 - BK7231N / oem_bk7231n_light_ty
   1.2.2 - BK7231N / oem_bk7231n_ceiling_light_ty
   1.2.7 - BK7231N / oem_bk7231n_light_ty
   1.3.1 - BK7231N / oem_bk7231n_light_ty
   1.3.2 - BK7231T / oem_bk7231s_ceiling_light_ty
   1.3.2 - BK7231T / oem_bk7231s_dltj

[?] Select the brand of your device: Aubess
 > Aubess
   Tuya Generic

[?] Select the article number of your device: 16A Mini Smart Switch
 > 16A Mini Smart Switch

Selected Device Slug: aubess-16a-mini-smart-switch
Selected Profile: oem-bk7231n-control-switch-1.2.1-sdk-2.3.3-40.00

================================================================================
Place your device in AP (slow blink) mode.  This can usually be accomplished by either:
Power cycling off/on - 3 times and wait for the device to fast-blink, then repeat 3 more times.  Some devices need 4 or 5 times on each side of the pause
Long press the power/reset button on the device until it starts fast-blinking, then releasing, and then holding the power/reset button again until the device starts slow-blinking.
See https://support.tuya.com/en/help/_detail/K9hut3w10nby8 for more information.
================================================================================

Scanning for open Tuya SmartLife AP
.....................................................
Found access point name: "SmartLife-728B", trying to connect...
Device 'wlan0' successfully activated with '67c22f7b-af8e-4eba-9a1b-9a5a8bf79562'.
Connected to access point.
Waiting 1 sec to allow device to set itself up...
Running initial exploit toolchain...
Exploit run, saved device config too!
output=/work/configured-devices/qitEJ7RZvr2p.deviceconfig
Saved device config in /work/configured-devices/qitEJ7RZvr2p.deviceconfig

================================================================================
Power cycle and place your device in AP (slow blink) mode again.  This can usually be accomplished by either:
Power cycling off/on - 3 times and wait for the device to fast-blink, then repeat 3 more times.  Some devices need 4 or 5 times on each side of the pause
Long press the power/reset button on the device until it starts fast-blinking, then releasing, and then holding the power/reset button again until the device starts slow-blinking.
See https://support.tuya.com/en/help/_detail/K9hut3w10nby8 for more information.
================================================================================

Scanning for open Tuya SmartLife AP
..
Found access point name: "SmartLife-728B", trying to connect...
Error: Connection activation failed: (53) The Wi-Fi network could not be found.
.....................
Found access point name: "A-728B", trying to connect...
Device 'wlan0' successfully activated with 'e0d9be3b-c401-4118-aa22-e1d7f3ae9341'.
Connected to access point.
Configured device to connect to 'cloudcutterflash'
Device is connecting to 'cloudcutterflash' access point. Passphrase for the AP is 'abcdabcd' (without ')
Checking UDP port 53... Available.
Checking UDP port 67... Available.
Checking TCP port 80... Available.
Checking TCP port 443... Available.
Checking TCP port 1883... Available.
Checking TCP port 8886... Available.
Cutting device off from cloud...

================================================================================
Wait for up to 10-120 seconds for the device to connect to 'cloudcutterflash'. This script will then show the activation requests sent by the device, and tell you whether local activation was successful.
================================================================================

Using WLAN adapter: wlan0
Configuration file: /dev/stdin
wlan0: Could not connect to kernel driver             <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< HERE!
Using interface wlan0 with hwaddr b8:27:eb:f8:d6:bf and ssid "cloudcutterflash"
wlan0: interface state UNINITIALIZED->ENABLED
wlan0: AP-ENABLED
Using PSK v1 - Received PSK ID version 01
Processing endpoint /v2/url_config
Processing endpoint tuya.device.active
Processing endpoint tuya.device.dynamic.config.get
Processing endpoint tuya.device.beacon.key.fetch
!!! Endpoint response not found, using default response - tuya.device.beacon.key.fetch (This is usually okay and safe to ignore unless something isn't working)
Processing endpoint tuya.device.uuid.pskkey.get
Device should be successfully onboarded on WiFi AP!  Please allow up to 2 minutes for the device to connect to your specified network.
Device MAC address: 1c:90:ff:5b:72:8b
Device Id: 6YewozfSajNEjet8CaNT
Local Key: 8rAHPFxtperGB5XE

the scriptrun gives at some point an error on the wlan: can't connect to kernel.

This is just an informational message output by the program that creates an AP, it is not a hard error.  I have not seen a machine that does not display this message.

The device is, as mentioned not accessible with a tuya based app but also does it not show up in my network. i'm out of clues.

Tuya-CloudCutter is successfully completing, as evidenced by the changing of the AP prefix as well as making it to the end without error.  If it is not joining your network, that is a network error of some sort beyond the control of Tuya-CloudCutter.  Some devices may need to be physically rebooted to join networks however.
There is nothing we can do any differently, we're just setting the ssid and password same as Tuya would be doing.  You'll need to investigate your network further to find potential pitfalls such as not accepting 2.4ghz on the ssid, special characters in ssid or password, or your network doesn't support the security modes of the device you are trying to add.

Thanks for the reply, had to be sure that flashing procedure was well done.
After playing around for a bit and physically pluged -> use switches, and unpluged the device for at least 10 times
it now suddenly poped up in my network.  It takes some time for changes get any affect i guess.
Great job!


Fix module loading

Fix some issues with the loading of modules.
Loaders must be included as packages to load, however this makes webpack-sandboxed more resilient to awkward directory configurations, like in example/.

Coverage decreased (-1.2%) to 88.75% when pulling 81cd1b13b2365d1ce1bc8567d350e63d32df04a7 on fix-module-loading into 5d71ac9f6f55c4303eab5713f32a77c2fea5e229 on master.

Coverage decreased (-1.2%) to 88.75% when pulling f4db21f1dc9d03a580e65ec462e565f733617e87 on fix-module-loading into 5d71ac9f6f55c4303eab5713f32a77c2fea5e229 on master.


Mechanism for basing accessor on another

For example to store Duration, you might store its ticks to PlayerPrefs (long) and then read them back. To do this DateTime accessor could use Long accessor to get/set the value.

Just create the other accessor within it, such as like CharPrefAccessor.


Drawer inadvertently intercepts events.

React doesn't actually add event handlers directly to the target nodes. Instead it adds all listeners on the document element.
Manually adding listeners here intercepts any listeners on child elements e.g.:
<Drawer>
    <DrawerContent>
         console.log('Never called')}>Broken Link
    </DrawerContent>
</Drawer

React automatically handles correct bubbling of events from child to parent to grandparent so it is safe to use standard props mechanism.

I've been playing with this library and testing different components as you've added them and like it so far and pretty excited to use it.
There was just this bug which got me tearing my hair out, going round in circles and diving deep into the react codebase to find out exactly how it handles events.
Hope this helps. If you any more help with testing other PRs etc. I'm happy to pitch in.

I really appreciate your opinion, thank you for your analysis. I've reread the React recommendations and found out that using of addEventListener is not advised.
I've changed commit message for changelog and merge everything to master


Racket failing with existing files

when typing in a new racket file everything works, syntax highlighting, repl integration, etc... but if I save that file and reopen I get no syntax highlighting and no repl. When I try to open a log or start a racket repl I get the following error
E5108: Error executing lua ...config/nvim/autoload/plugged/conjure/lua/conjure/log.lua:37: attempt to concatenate a nil value

when I remove the vim-racket plugin I get highlighting again but I still do not have the Conjure support. Please let me know if I am just doing something very dumb.

Thanks.

it was something dumb, I had

#lang racket/gui

which was throwing it off for some reason. I don't know if the #lang racket/gui is needed but it won't load with it


I keep get that error

Error
fend command error- Check installation path or fend config file
I Installed fend from the Plugins in flow launcher, but I keep getting the same issue
@IsaacTay

Hi, fend is separate program that this plugin interfaces with. Currently this plugin doesn't install fend automatically, but it will be added in the future.
For now, you can install fend through scoop, chocolatey, winget, the official msi installer, or  as a standalone binary.
I hope this fixes your issue!

Thank you for answering me
I did install fend, remove the Fend plugin from flow, and reinstalled it... but unfortunately, I keep getting the same error

We were having this issue, I fixed it by changing the fend path config to fend's file location ("C:\Program Files\fend\bin\fend.exe" for us) and now it works

Ah okay I see the problem. I mistakenly assumed that the fend program files were added to the system PATH by default, I am working on a fix right now so you won't have to manually configure it to point to the fend.exe. In the case that your fend installation isn't at Program Files, you can also do what @Ringlings said and configure it in the plugin config here:

I've added the support to automatically detect fend if it is located in the program files, and also a reset button that'll trigger the auto-detect. This should fix the problem, if not, feel free to open the issue again


Support suite command

Hi,
Will you support suite command as shelex package?

Hi @MDzyga ,
I will do that with pleasure. Could you please explain what is missing in more details?
Do uyou mean adding these commands in API ?
parentSuite(name: string)
suite(name: string)
subSuite(name:string)

do you know when you will be able to release a version with this/these method(s)?

in the meantime as a workaround you can use
cy.label('parentSuite', 'Parent Suite name here');
cy.label('suite', 'Suite name here');
cy.label('subSuite', 'Sub Suite name here');

ooo nice :)
We use it to group our tests on Suites panel.
Currently it looks like

When we used suite method from shelex it looks like

ooo nice :) We use it to group our tests on Suites panel.
Currently it looks like
When we used suite method from shelex it looks like

I got your concern,
Could also please send you test file structure
Actually in this plugin it should automatically add suite labels depending on your 'describes'
For example

describe('suite', () => {
  describe('child', () => {
    it('test', () => {
      //
    });
  });
});

should create report with suites suite -> child

ok, currently it looks like
describe
  describe
    it
      suite 1
    it
      suite 1
  describe
   it
      suite 2
   it
     suite 3
   it
     suite 4

So referring to screenshot is it actual structure in you spec file?
describe('Configuration', () => {
  describe('QLC-232 - Create or update configuration', () => {
    // tests
  });

  describe('QLC-235 - Get All configurations', () => {
      // tests
   });

  describe('QLC-237 - Get Configuration (by name)', () => {
      // tests
   });
  // ...
});

Not exactly
describe('Configuration', () => {
  describe('Get operations', () => {
    it('get one', () => {
       cy.suite('QLC-232 - Create or update configuration')
    })

    it('get all', () => {
       cy.suite('QLC-232 - Create or update configuration')
    })
  });

  describe('Save operations', () => {
    it('save', () => {
       cy.suite('QLC-235 - Get All configurations')
    })
   });
  // ...
});

@MDzyga 0.10.0 version has been released. You can check the documentation for details.

@MDzyga From what I see seems it is better to use epic/feature/story
After using that you will see results grouped on Behaviours tab
describe('Configuration', () => {
  describe('Save operations', () => {
    it('create', () => {
       cy.allure().story('QLC-232 - Create or update configuration')
    })

    it('update', () => {
       cy.allure().story('QLC-232 - Create or update configuration')
    })
  });

  describe('Get operations', () => {
    it('get all', () => {
       cy.allure().story('QLC-235 - Get All configurations')
    })
   });
  // ...
});

and in the same time they will be groupped by suites:

Thank you very much :)
I will discuss with tester about your proposal :)


Implement the WireFrames for UI for Analytics and Report Modules

Start designing and code the wireframe with sample UI components for the Interface of The Analytics and Report Modules website

CSS
HTML
JS files

Liam and Gleb made the previews and designs
Done wireframes


add package.json

How to npm install and truffle test without package.json ?

I believe it's in the repo?
I can see it... could you check again?

Please fork and make a pull request with a better package.json, for now you can just install the dependencies, there are not many...
Typically these are auto generated like the package-lock.json


[ENH] Add phase1/phase2 SDC support

Changes proposed in this pull request
Add support for phase1 and phase2 fieldmaps, related to #1169. This is just one idea for how to support them, another possibility would be to make a separate workflow.

Documentation that should be reviewed

Documentation probably doesn't need to be changed.

Please remove the [WIP] tag to indicate this is almost there, and I'll gladly review it :)
Thanks for the contribution!!

@oesteban have you ever worked with phase1, phase2 images?  I tried running some directly through the siemens2rads function, which looks like it handled these at some point, but the output looks wrong.

There is a script used here internally that handles these images, but it has two different equations depending on whether the readout was bipolar or monopolar (which isn't specified in the json sidecars). I'm wondering if supporting this kind of fieldmap is worth the trouble.

Hi, sorry for being missing all this time. I agree with @effigies that reverting stylistic changes will help review this PR substantially.
That said, why the phase maps cannot be just subtracted and passed over to the phasediff routines?

Here is the formula, assuming GE data with 12-bit ADC... Might want to check with 16-bit ADC
diffTE := abs(TE1sec - lTE2sec); // difference in phase, sec
scale := (2*pi/4096)/diffTE;
rescaleImg(-2048, scale); //intercept, slope

How close is it from being merged? There is at least one more user that would benefit from this work: https://neurostars.org/t/fmriprep-fieldmaps-keyerror-phase/3047

I missed some of the conversation in here... I think the main thing remaining is the exact scaling algorithm, as in this thread. I'm not very happy with the current rescale_image approach:
https://github.com/poldracklab/fmriprep/blob/f2e861d61782d54bd31331bb701af7e029b6fbbf/fmriprep/interfaces/fmap.py#L546-L559
as it ignores zero-points in known conventions, ending with a slightly positive shift in phase. Additionally, if the minimum and maximum values aren't present for some reason, it causes a stretch. For the former concern, given that we're dealing with phase differences, it should be fine as long as the same shift occurs; for the latter, I don't know whether that's a plausible situation, given the number of voxels and range of values. So perhaps it's fine, but I'm a little inclined to try to interpret the values exactly as intended by the scanner, if we can.

I think it would be good to support, even if people end up preferring to ignore fieldmaps and use SyN-SDC.

+1

I'll have a look ASAP

Please merge master to fix the doc build.

Any idea what's happening with Tcompcor?

Happened the same with my previous pr and the docs build. Two is a pattern
(not particularly good this one)
On Tue, Dec 18, 2018, 13:25 Chris Markiewicz<EMAIL_ADDRESS>wrote:

Ugh. Almost certainly my fault, but I'm annoyed that it didn't produce an
error until merged. I'll have a look when I can, but that might not be
until Thursday.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/poldracklab/fmriprep/pull/1359#issuecomment-448376197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkhxsLDl08bNPXNVnrrv70MZ21FkoXDks5u6V1PgaJpZM4YGfuW
.

I'll revert #1443.

@mattcieslak: the developmental branch of dcm2niix (v1.0.20181218) now inserts a "DiffusionScheme" tag into your BIDS sidecar. This requires the tag sDiffusion.dsScheme in the Siemens CSA header - I think this is in both the CMRR and Siemens product DICOMs with E11. I am not sure how to detect this for  other vendors or older Siemens data (though with earlier versions of the Siemens product, I think one could only select the Bipolar scheme). Regardless, if this tag is detected, the BIDS sidecar should include a tag that looks like this "DiffusionScheme": "Monopolar" or this "DiffusionScheme": "Bipolar".

Thanks @neurolabusc this will be very useful for the diffusion pipeline I'm working on. For these scans, whether it was bipolar or monopolar was specified in a tag called "ucReadOutMode" that was 2 if the scan bipolar or 1 if monopolar. I'm suspicious that the bipolar readoutmode was a non-standard variation that was some sort of a work-around at the time. Have you ever seen this vary on fieldmaps?

@mattcieslak please check the latest dcm2niix development branch commit. It will attempt to read the CSA tag sDiffusion.dsScheme (where 1=bipolar, 2=monopolar) and if that fails it will attempt to read ucReadOutMode (where 0x1=monopolar and 0x2=bipolar). If either is found, the BIDS file will report either "DiffusionScheme": "Monopolar" or "DiffusionScheme": "Bipolar".
I believe sDiffusion.dsScheme is used on E11 for both CMRR and Siemens product. I think the ucReadOutMode was used by CMRR prior to E11. I am pretty sure that prior to E11, Siemens product diffusion sequences were always bipolar (e.g. used a bipolar twice-refocused spin echo sequence).

@mattcieslak I think your method will work with all the Siemens images I have seen. I do not know how to detect these properties for GE and Philips. My sense is that monopolar is the default for Philips. @drmclem might know the answer.

@mattcieslak if it is Siemens and not detected by my software, it seems that the it is most likely bipolar. I think the Siemens sequences used to be only bipolar, and I think my new version detects bipolar/monopolar settings for CMRR sequences as well as the new Siemens sequences that allow one to select between bipolar and monopolar. For many DWI scans, one should be able to reliably infer bipolar or monopolar once knows the matrix size, partial Fourier and iPAT. For most DTI sequences users tend to select  the minimum possible echo time to maximize SNR. While bipolar sequences show less spatial distortion and are less sensitive to background gradients, they pay a large penalty with regards to TE. The one excpetion might be scans where users want to measure kurtosis, as these groups tend to prefer longer echo times, on the other hand they are also concerned with background gradients so the kurtosis sequences I have seen follow the trend of long TE and bipolar sequence.

I'm okay in either direction. Whether we reject anything that isn't explicitly monopolar or only reject things that are explicitly bipolar both seem like arguable choices. If we run into the problem that a lot of people with unlabeled bipolar fieldmaps are having issues, we can revisit it or perhaps encourage users to better label their data.

That makes sense for diffusion-weighted image series, but in this case the difference between monopolar and bipolar phase images doesn't just change the SNR/TE but the biploar readout also changes the values in the phase images. Bipolar phase images require that you scale the non-phase-encoding direction by its coordinate. This was previously corrected using
NP=`fslval ${outfroot}_rad2raw dim1`
3dcalc -a ${outfroot}_rad2raw.nii -expr "(i-$NP/2-1)*3.14159/$NP/2" -prefix ${outfroot}_rad2err.nii

which was checked against the monopolar sequence using a phantom.  I have no idea how general this solution would be, which is why I don't think it should go into fmriprep. It's also probably very rare to see a bipolar GRE scan.
The phasediff calculation in this patch works easily on the monopolar sequences, but the displacements that come out of the bipolar sequence greatly shift one hemisphere (you can see the shearing here: https://github.com/mattcieslak/example_fieldmaps/blob/master/figures/sub-1_task-rest_acq-v1_sdc_phase.svg).

OK, thanks for the clarification.

I'm hoping to finish this up. I merged in the latest fmriprep and am running into an issue with fmriprep-docker where --patch-fmriprep results in smriprep being missing. I'm using version 1.3.2. It this expected?

@mattcieslak We should probably add a --patch-smriprep option to fmriprep-docker. But you can use -i poldracklab/fmriprep:1.4.0a1 to use the latest pre-release, which should have a compatible version of smriprep.

Ok, this is working now on all my test datasets. Anything else I should add?

Thanks a lot. I'll need some time to go through this PR with enough depth. Since I'm at ISMRM right now, and we are about to release a new version with big changes, the review may take a little while.

This might also solve some of the issues we've been running into with these fieldmaps when they come from our 7T scanner. Also, I'd be happy to help with a refactor of the fieldmap workflows. I'm maintaining a diffusion pipeline that shamelessly copies them and would like to keep them up to date. Should we close this PR then?

This will be continued on poldracklab/sdcflows


Swate Alpha "Open file" could use json files

If you give me a way /ARCtrl api to easily understand what format i must parse?


Ports test suite runner from Bash to Haskell

Okay, so the tests are passing now (c.f. #14).
If you want to pass just one flag then the simplest command is
stack test --ta --sixten-args=-O2

If you want to pass multiple arguments then use
stack test --ta='--sixten-args "-O2 -Sasm"'

Changing either or both the quotes or putting an = sign after --sixten-args causes things to not work.
I've kept the shell script untouched for now.

Hey Varun!
Looks great! Here are some thoughts I had around the code:

If we made the --expected flag treat a missing file the same as an empty file we could get away with always passing --expected when running the success tests but wouldn't need to add any empty files. It would make it more convenient to add tests where you're e.g. only interested in it passing the typechecker since you wouldn't have to add the -expected file.
I'm guessing the intention of the TestInput type is to enable more code sharing between the different kinds of tests, but I think the code might be simpler without it. It's mainly used to compute input files and flags, so perhaps the function(s) that create the tests could take the directory to search and the flags as arguments instead. There could e.g. be one function for single-file tests and one for multi-file tests. To get the expected files right in the single-file case the flag argument could be a function FilePath -> Flags such that it can use the name of the input file when constructing the flags.
Add yourself to the authors list in the readme. :)

These are only suggestions so we could merge this now and save them for later unless you feel like working some more on it now.
Thanks for contributing!

For 1, there's two ways -- we could simplify the behaviour of the expected flag or we could add a check to the tests. I think the flag's behaviour is correct, so I've added a check to the tests, just like the Bash script.
I've almost addressed 2 but the issue is I need a signature like (v -> TestTree) to feed into askOption :: IsOption v => (v -> TestTree) -> TestTree which in turn calls the AskOption constructor. What I get with your suggestion is a function like v -> IO TestTree and I don't know what function to use to get
lift? askOption :: IsOption v => (v -> IO TestTree) -> IO TestTree

Thoughts?

Cool, yeah, I suppose adding a check is even better! :)
Is the reason that there's an IO TestTree that it needs to look for the files in the file system? Perhaps the test discovery functionality could be run outside of Tasty and return the arguments that sixten should be called with, and "inside" Tasty you could use askOption to add the additional extra args. Does that make sense?

Sorry if I was unclear in point 2 above.
What I meant was that there shouldn't be any need to do any dispatching (save for existence of -expected files) to determine the flags. These are things we know statically and it's fine to hard-code them. I'd expect the test discovery to be something along the lines of:
fmap concat $ sequence
  [ singles "success" []
  , singles "type-error" ["--expect-type-error"]
  , singles "syntax-error" ["--expect-syntax-error"]
  , multis "success-multi" []
  , multis "type-error-multi" ["--expect-type-error"]
  ]

No problem! I set it up that way so that the test suite would fail (undefined) in case you added files in some other folder which didn't have "success"/"type-error"/"syntax-error" in it and forgot to update the test suite to run those tests.

Hurrah! I've merged but Github just doesn't realise (maybe because of the autosquash?). Thanks! :)

Ah, did you merge locally? Yeah, in that case, it doesn't understand. I recall being puzzled by that too a few weeks back.

Yeah, I wasn't sure the web interface would do the autosquash properly. Do you know if it would have?

I have no clue.


[QUERY] Issue with running spring boot application on azure spring apps with key vault configuration by using service principal identity

Query/Question
I have a spring boot application, using key vault to store my mongodb credentials. I am using service principal authentication to connect with azure key vault. The application is currently running on local env with the below code attached after log in with az login service principal. But, when I am trying to deploy it on azure spring cloud, I am getting the below exception
java.lang.IllegalStateException: Failed to configure KeyVault property source
This is the Bean I kept in my mongodb config class. I am using the same service principal to deploy from local. And, it is getting deployed as well, but the application failed to start due to above exception. I tried with managed identity as well but no luck. I wanted to use service principal to connect with azure key vault as well. Here is the screenshot of the same service principal to the key vault.

@Value("${azure.keyvault.client-id}")
private String clientId;

@Value("${azure.keyvault.client-key}")
private String clientSecret;

@Value("${azure.keyvault.tenant-id}")
private String tenantId;

@Value("${spring.cloud.azure.keyvault.secret.property-sources[0].endpoint}")
private String keyVaultEndpoint;

@Bean
public SecretClient createSecretClient() {

    ClientSecretCredential clientSecretCredential = new ClientSecretCredentialBuilder()
            .clientId(clientId)
            .clientSecret(clientSecret)
            .tenantId(tenantId)
            .build();

    // Azure SDK client builders accept the credential as a parameter.
    return new SecretClientBuilder()
            .vaultUrl(keyVaultEndpoint)
            .credential(clientSecretCredential)
            .buildClient();
}

Here is the dependency I am using in my spring boot application.

com.azure.spring
spring-cloud-azure-starter-keyvault
5.11.0

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Query Added
[ ] Setup information Added

@kirankumar994 are you using the com.azure.spring spring-cloud-azure-starter-keyvault to retrieve the kv property source, instead of using the kv secret client and get the secret by yourself?

Hi @saragluna , here is the dependency I am using.
<dependency>             <groupId>com.azure.spring</groupId>             <artifactId>spring-cloud-azure-starter-keyvault</artifactId>             <version>5.11.0</version>         </dependency>

Hi @kirankumar994, what do you mean by The application is up and runnin after I login with az login service principal?
What's the properties you set on Azure Spring Apps, I guess that the KV property source is enabled, but without any credential to connect to the KV client in the property source. The bean you defined in the following code is a different client object from the KV client newed in the KV property source.
return new SecretClientBuilder()
            .vaultUrl(keyVaultEndpoint)
            .credential(clientSecretCredential)
            .buildClient();


Delete unused isInferredVal() methods

We don't use isInferredVal(), and removed it in this PR.

Hi @nnnkkk7, we always appreciate contributions from the community, thank you!
However, even though isInferredVal() is not used, we intentionally kept it there for InferredVal to be un-assignable by other structs because this interface only contains a very generic copy method. Having this method in the interface would prevent accidental assignment of unexpected structs for extra safety.
It is very similar to what Go stdlib did in its ast package: https://cs.opensource.google/go/go/+/refs/tags/go1.21.6:src/go/ast/ast.go;l=40

@yuxincs
Oh..I see.
I understood it!!
Thanks!


Setting an 'owner' fails if a user's group is not the same as username  e.g. nobody:nogroup

because default provider assumes the new owner belongs to the group with the same name e.g.
user:user. In my case I have group 'users' and user 'user1'.
around line 66
 owner new_resource.owner
 group new_resource.owner

'owner' is not being set properly
Platform RHEL
Chef: 11.12.4

Also ran into this problem.

@khvatov @ShibaBandit The 2.0.1 has resolved this by adding a property for group,which defaults to root's group.  Give it a try.


Error: pyppeteer.errors.ElementHandleError: Evaluation failed: TypeError: Cannot read property 'value' of null

I faced the following error. Any solution would be appreciated

Try latest version:
pip install -U deepl-cli

The error still occurs at the latest version

Now, this tool is not using pyppeteer. Check your CLI version.


Update logo for new networks on the mobile app

Description the task
Update logo for new networks on the mobile app

Recheck network:

[ ] XX Network
[ ] Watr Network
[ ] Watr Network - EVM
[ ] Subspace Gemini 3a
[ ] Aleph Zero Smartnet

Version test:v0.2.4
Validation passed.


Token displayed as QR

For a door control system, which already has a camera, it would be nice, that the application creates a qr code from the token. This way, the user can open the FreeOTP app, select the door systems entry, let the token be displayed by a qr code. then user holds the phone in front of the camera. Door system recognizes the qr code, verifies the token and if ok, opens the door.
Now need to add a key panel to the door system and let the user type in the qr code.

This is likely too far beyond the scope of what FreeOTP offers.


2.17.1 can't be install

please look at the image

This looks like a community maintained installer.

@bshlomo contact the maintainer of the installer.

@athomas
Who is the maintainer of this installer?

@bshlomo the answer is literally in that email: "GeKorm" is the maintainer. Just because I tried to help you before doesn't mean I maintain that installer.

@athomas
Thank you for past help and thank you for the current help
it is obvious


chore: add the logger base interfaces

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
This PR introduces a logging mechanism with a no-operation logger for development, integrates environment variable handling for a write key, and modifies build scripts. It allows conditional logging based on the presence of the write key and differentiates between server and client logging.
Detailed summary

Added noopLogger in noop.ts for development use.
Introduced WRITE_KEY in _writeKey.ts, set to undefined.
Created prebuild.ts and updated package.json scripts for the build process.
Implemented createServerLogger and createClientLogger functions.
Enhanced TrackEventParameters and EventLogger types in types.ts.
Added logic in index.ts to conditionally log based on WRITE_KEY.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

_writekey neat...


Constructing instances of type arguments

This issue was originally filed by @olostan

Currently when compiling such code:
class A {
}

class B<T> {
  T GetBySmth() {
    return new T();
  }
}

void main() {
  var b = new B<A>();
  var x = b.GetBySmth();
  assert(x is B);
}

does not compile with message:
Error: line 6 pos 16: type parameter 'T' cannot be instantiated
    return new T();
               ^

So proposal is to add specifications to inform VM/compiler that generic type does have constructor, so it could be invoked to initialize instance.
Syntax could be (as proposed here https://groups.google.com/a/dartlang.org/d/msg/misc/2vgshmBCZx8/FdI6iKQvEgYJ):
class B<T is new()> {}
class B<T is new(int arg1)> {}

or
class B<T has new(int arg1)> {}

also with named constructors:
class B<T has new.namedConstructor(int arg1)> {}

This will allow to build much more clean generic behaviors (e.g. generic caches, repositories, etc)

I guess https://github.com/gbracha/metaclasses/blob/master/proposal.md is supposed to lead to a solution (as reference for people finding this issue first).

Still new to Dart and I also ran into this same issue.  I see its closed (as obsolete).  I also see a link to  a proposed solution, but find no action on that proposal.
So, apologies for asking again, but how exactly am I supposed to instantiate a generic type?

Still new to Dart and I also ran into this same issue. I see its closed (as obsolete). I also see a link to a proposed solution, but find no action on that proposal.
So, apologies for asking again, but how exactly am I supposed to instantiate a generic type?

There is the reflectable, json_serializer packages etc
I use source generation

@jodinathan Thank you. I appreciate the pointer. Unfortunately, its a bit overkill for my requirements. I've already implemented a quite ugly workaround, but I'm still hoping for a language-supported fix.

@techfan101 wrote:

how exactly am I supposed to instantiate a generic type?

It is basically a questionable idea. However, a solution that will work in general is to pair up the type argument with a corresponding factory function:
class B<T> {
  final T Function() factoryOfT;
  B(this.factoryOfT);
  T getBySmth() => factoryOfT();
}

That factory function may need to embed a lot of decisions. For instance, if T is int then you'd have to return some existing integer rather than a "new" one; if T is a class that only has constructors that take many arguments and require some serious building effort in order to obtain an object that will actually work, then the factory function will have to perform that building work. If T is the empty type (Never) then the factory may be forced to throw an exception or loop forever: There is no way to satisfy the request "give me an element from the empty set".
C# allows for a constraint on type parameters that they must support new(), which is basically syntactic sugar for (1) requiring that the type argument denotes a class that has a constructor that accepts the empty argument list, and (2) passing a factory function around (or using a heterogeneous compilation strategy such that we get a fresh copy of the whole class for each distinct value of the type argument). This kind of syntactic sugar might be convenient in some situations, but it isn't very general, and the solution where we declare the factory function explicitly is much more general (e.g., it could take whatever parameters we'd want).

@techfan101 wrote:

how exactly am I supposed to instantiate a generic type?

It is basically a questionable idea. However, a solution that will work in general is to pair up the type argument with a corresponding factory function:
class B<T> {
  final T Function() factoryOfT;
  B(this.factoryOfT);
  T getBySmth() => factoryOfT();
}

That factory function may need to embed a lot of decisions. For instance, if T is int then you'd have to return some existing integer rather than a "new" one; if T is a class that only has constructors that take many arguments and require some serious building effort in order to obtain an object that will actually work, then the factory function will have to perform that building work. If T is the empty type (Never) then the factory may be forced to throw an exception or loop forever: There is no way to satisfy the request "give me an element from the empty set".
C# allows for a constraint on type parameters that they must support new(), which is basically syntactic sugar for (1) requiring that the type argument denotes a class that has a constructor that accepts the empty argument list, and (2) passing a factory function around (or using a heterogeneous compilation strategy such that we get a fresh copy of the whole class for each distinct value of the type argument). This kind of syntactic sugar might be convenient in some situations, but it isn't very general. The solution where we declare the factory function explicitly is more general (e.g., it could take whatever parameters we'd want, and we could use different factory functions for different instances of the enclosing class).

The syntactic sugar of C# is convenient in a lot of cases, if not most of them. And it does not stops you from using a factory function explicitly if needed.
I am guessing that you usually needs T() with business models and they are also usually have empty constructors. Our projects all use source generated generic factories with no argument, so a simple T() would have been nice.

@eernstg Thank you, but unfortunately your suggestion is a non-starter for my requirements.
@jodinathan Exactly what I need.  I am completely comfortable requiring a default/parameterless constructor.  While I have the occasional gripe, I think the new() constraint from C# (or some similar syntactic clue) strikes the right balance between convenience and insanity.
Short of some support within the language, I am reluctant to believe that other solutions will truly suffice. Honestly open to suggestions, but I need something broadly applicable that requires little additional plumbing.
I've been at this (engineering not Dart :)) for over 35 years.  I can always find ugly workarounds, which is (sadly) what I'm currently using.  That said, given that no "good" solution seems apparent, I'm a bit surprised this issue was closed.


Sliding Windows diagram has wrong unit

The sliding window diagram has Minute in Time (Minute) axis but code uses Seconds

@LoveCSharp To assist you better on your query could you provide us with the link to the document you are referring to ?
In case your query in not related to document content, open a new forum thread either MSDN, Q&A or StackOverflow as it could be better place to get help on your scenario where the community members could provide their expert guidance to you based on their experience.

@LoveCSharp We will now proceed to close this thread as we haven't heard back. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Could Not Execute Darwin / 7za on macOS

When trying to run this tool on macOS, I get:
Extracting HiyaCFW archive...
ERROR: Could not execute Darwin/7za
I am running macOS Mojave with the latest version of Python 3.

Does the 7za binary run directly? I mean, when you are at the helper's folder and run Darwin/7zadoes it gives you any output?

Yes, it gives me a list of commands.

*I should also point out that when running HiyaCFW Helper, in the Terminal window where I executed the Python script, I get:
objc[786]: Class FIFinderSyncExtensionHost is implemented in both /System/Library/PrivateFrameworks/FinderKit.framework/FinderKit (0x11a80b210) and /System/Library/PrivateFrameworks/FileProvider.framework/OverrideBundles/FinderSyncCollaborationFileProviderOverride.bundle/Contents/MacOS/FinderSyncCollaborationFileProviderOverride (0x1229dfdc8). One of the two will be used. Which one is undefined.

That warning is normal for tkinter apps. Please run a brew upgrade and try again. Meanwhile I'll update the script for printing the exception errors to the console.

Oh, and I also get "Error, Could not get HiyaCFW" when it tries to download the latest HiyaCFW release.

Replace the .py file with this one and tell me which errors you get.

How do I download that?

Nevermind, found it

OK, In the app I got "ERROR: Could not get HiyaCFW" and in the console, I got:
<urlopen error [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:646)>

OK, to avoid that TLSV1_ALERT_PROTOCOL_VERSION error you must install openssh via homebrew.

I installed openssh via homebrew and tried running it but I still get the same error.

Was your python installed via Homebrew?

You need to cd into the helper's folder.

Not a helper's issue. Closing...


Colab patch

Added a colab-configured notebook
Replaced binder badge with a collab badge

Codecov Report

Merging #46 (6843052) into master (ab9c9a7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #46   +/-   ##
=======================================
  Coverage   80.96%   80.96%
=======================================
  Files          41       41
  Lines        3373     3373
=======================================
  Hits         2731     2731
  Misses        642      642

Continue to review full report at Codecov.

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


[Service Bus] Message Lock Renewal PartitionedSubscription: Batch Receiver failing in nightly runs

Service Bus nightly test runs are failing with:

Error message:
Initial: Actual time Mon Oct 24 2022 12:04:25 GMT+0000 (Coordinated Universal Time) must be approximately equal to Mon Oct 24 2022 12:04:22 GMT+0000 (Coordinated Universal Time): expected false to equal true
Stack trace:
AssertionError: Initial: Actual time Mon Oct 24 2022 12:04:25 GMT+0000 (Coordinated Universal Time) must be approximately equal to Mon Oct 24 2022 12:04:22 GMT+0000 (Coordinated Universal Time): expected false to equal true
at Object.should.equal (/Users/runner/work/1/s/common/temp/node_modules/.pnpm/chai@4.3.6/node_modules/chai/lib/chai/interface/should.js:98:35)
at assertTimestampsAreApproximatelyEqual (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/renewLock.spec.ts:416:14)
at testBatchReceiverManualLockRenewalHappyCase (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/renewLock.spec.ts:185:5)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at Context. (/Users/runner/work/1/s/sdk/servicebus/service-bus/test/internal/renewLock.spec.ts:55:7)
+ expected - actual
-false
+true

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1939639&view=results

Recent runs are green


[SPARK-46514][TESTS] Fix HiveMetastoreLazyInitializationSuite

What changes were proposed in this pull request?

This PR enabled the assertion in HiveMetastoreLazyInitializationSuite
Why are the changes needed?

fix test intenton
Does this PR introduce any user-facing change?

no
How was this patch tested?

pass HiveMetastoreLazyInitializationSuite
Was this patch authored or co-authored using generative AI tooling?

no

cc @LuciferYang @dongjoon-hyun thanks.

Thank you all. Merged to master/3.5/3.4


added convert to gif script to make it easier to create a new gif for readme

This'll just make it way faster to update the gif of the gui in the readme to a new recording of it.
:)
added script in the tools folder :)

@ROBERT-MCDOWELL
Should make it easier on my going into the future with making the readme gui gif up to date with the current state of the gui
:)

Kool!


@W-14810956@ Update generator with options for slas private client

This change updates the pwa-kit-create-app project to include an option for setting up a SLAS private client.
Preset projects have been set up to default use a SLAS public client.
A new preset retail-react-app-private-slas-client has been added to the generator. This preset will set up an extensible project against the ODS zzrf-001. Ask @vcua-mobify for the client secret associated with using this preset
Testing the changes:

Check out the changes in this PR
Generate a project (you can either use the preset retail-react-app-private-slas-client or enter values manually)
Make sure you set the PWA_KIT_SLAS_CLIENT_SECRET environment variable to your client secret
Build and start the generated project
See that the project starts and is using the SLAS private client flow (no calls to SLAS /authorize)

The command for using the preset locally is below:
./scripts/create-mobify-app-dev.js --preset=retail-react-app-private-slas-client
Note: These changes apply to both an extensible project and a non-extensible project. Changes have been tested to work with both.

Note: This PR increases the number of _app-config/index.jsx from 1 to 3. The number of copies of ssr.js also goes up from 2 to 3.
The two new copies are templates that live in pwa-kit-create-app. One template (under the bootstrap folder) is used by extensible projects and the other template (under the template folder) is used by non-extensible projects.


Semantic Release + Commitizen

Should we put in Semantic Release and/or Commitizen to help with release automation and commit message uniformity?

Stories created in Jira.


Exploring Nostr

So far is post only, and vendored openssl isn't building for me with android targets. If I can get that building I'd like to try using it.
Is there a way we can avoid building an entire nostr client ourselves, while still providing a useful experience with our devices?
Can we hook into other clients?

I think that this will be cool but isn't needed before launch.. Frost bitcoin is cool by itself ;)

I think that this will be cool but isn't needed before launch.. Frost bitcoin is cool by itself ;)

This is what we also decided!


cmake: Remove ndk-build

Successfully ran VkCube.apk on my Pixel3
closes #850

CI Vulkan-Tools build queued with queue ID 60761.

CI Vulkan-Tools build # 1218 running.

CI Vulkan-Tools build # 1218 passed.


Create index.html

added this to redirect to the admin page instead of seeing the files

From the readme : There is no index page at the root of the install.

i added it because i used it 1 time and saw the i i open the root dir in browser it showed me all the file contents
so i added the index.html to redirect to the admin portal


Fix regression from #660

After updating to 0.10, I'm having issues with url_for in Flask, which I have traced to the changes made in #660. As noted by @davidism here the behavior that #660 was trying to add was kindof already supported:
with werkzeug==0.9.6
app = flask.Flask(__name__)
@app.route('/')
def index():
    return '42'

flask.url_for('index', x=[1, 2])  # '/?x=1&x=2'

with werkzeug==0.10
flask.url_for('index', x=[1, 2])  # '/?x=%5B1%2C+2%5D'

This PR will convert dict to MultiDict so that both will work now.

Oh dear... thanks for this patch! I applied it to 0.10-maintenance instead.


[eas-cli][metadata] use just a comma to concatenate keywords

Why
https://github.com/expo/eas-cli/issues/1963
How
Use just a comma to concatenate an array of keywords
Test Plan
Tests

/changelog-entry chore Use just a comma instead of ,  when concatenating keywords in eas metadata


Failure steps in running TempC example

I'm trying to confirm compatibility and operation of eeweather and eemeter using OpenEE docs.
C:\Users\micha>python
Python 3.6.8 (tags/v3.6.8:3c6b436a57, Dec 24 2018, 00:16:47) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import eeweather
import datetime
import pytz
ranked_stations = eeweather.rank_stations(35, -95)
station, warnings = eeweather.select_station(ranked_stations)
station
ISDStation('720627')
ranked_stations.loc[station.usaf_id]
rank                                     1
distance_meters                    32692.7
latitude                            35.283
longitude                            -95.1
iecc_climate_zone                        3
iecc_moisture_regime                     A
ba_climate_zone                Mixed-Humid
ca_climate_zone                       None
rough_quality                          low
elevation                            183.2
state                                   OK
tmy3_class                            None
is_tmy3                              False
is_cz2010                            False
difference_elevation_meters           None
Name: 720627, dtype: object
start_date = datetime.datetime(2016, 6, 1, tzinfo=pytz.UTC)
end_date = datetime.datetime(2017, 9, 15, tzinfo=pytz.UTC)
tempC = station.load_isd_hourly_temp_data(start_date, end_date)
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\stations.py", line 1458, in load_isd_hourly_temp_data
error_on_missing_years=error_on_missing_years,
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\stations.py", line 853, in load_isd_hourly_temp_data
for year in range(start.year, end.year + 1)
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\stations.py", line 853, in
for year in range(start.year, end.year + 1)
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\stations.py", line 715, in load_isd_hourly_temp_data_cached_proxy
data_ok = validate_isd_hourly_temp_data_cache(usaf_id, year)
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\stations.py", line 484, in validate_isd_hourly_temp_data_cache
store = eeweather.connections.key_value_store_proxy.get_store()
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\connections.py", line 120, in get_store
self._store = KeyValueStore()
File "C:\Users\micha\AppData\Local\Programs\Python\Python36\lib\site-packages\eeweather\cache.py", line 52, in init
raise ImportError("KeyValueStore requires sqlalchemy.")
ImportError: KeyValueStore requires sqlalchemy.

After receiving this error, I installed SQLAlchemy. The ImportError stopped, but I received the following when writing the next two example lines.

tempC.head()
Traceback (most recent call last):
File "", line 1, in
AttributeError: 'tuple' object has no attribute 'head'
tempF = tempC * 1.8 + 32
Traceback (most recent call last):
File "", line 1, in
TypeError: can't multiply sequence by non-int of type 'float'

I'm expecting to get tempC and convert to tempF outputs, like the example http://eemeter.openee.io/basics.html#loading-sample-data
Should SQLAlchemy be included in installation with eeweather, if it doesn't work without it?
OS: Windows 10
eemeter         2.7.0
eeweather       0.3.20
matplotlib      3.1.1
pandas          0.23.4
patsy           0.5.1
pip             19.1.1
python-dateutil 2.8.0
pytz            2019.1
SQLAlchemy      1.3.5
I'm new, so nothing is too obvious... Thanks.

@michaelsuhl11 Sorry for the very delayed response here, but what you'd want to do is call the following function in the following way, to capture the warnings:
+   tempC, warnings = station.load_isd_hourly_temp_data(start_date, end_date)
-   tempC = station.load_isd_hourly_temp_data(start_date, end_date)

Assuming you've moved beyond this by now, I'm going to go ahead and close this issue. Please feel free to reopen if this is not the case.

@michaelsuhl11 If you have a minute, would you mind filling our our new first-time issue contributor survey? The info we get from this will help us to grow to meet the needs of our community of users. https://forms.gle/H5jptzS5rL3hDcNJA Thanks!


Issues when running it on the Windows Server 2012 R2

Thanks for your interest in .NET Core.
Please note that this repo (dotnet/core) is not for filing product issues. Here are some repos where you can file an issue:

dotnet/cli - for CLI tools and questions
dotnet/corefx - for API issues and questions
dotnet/coreclr - for runtime issues
Issue is when I am trying to run it using command line. I am getting an error

dotnet.exe - System Error
The program can't start because api-ms-win-crt-runtime-l1-1-0.dll is missing from your computer. Try reinstalling the program to fix this problem.
OK
Please have a look at this issue

nuget/home - for NuGet questions and issues

Can you share which .NET Core version you were attempting to use?

This looks like 2.0. There is a hotfix for the OS that is required for the uCRT to be present. Can you make sure the OS is up to date? Was it a fresh install of the OS?

I believe this is the link https://support.microsoft.com/en-us/help/2999226/update-for-universal-c-runtime-in-windows

Does that work for you @fahad198 ?

Please install vc++2015


Unable to create a new Sveltekit project

Describe the bug
I just wanted to create a new Svelkekit project but am unable to run my project.
Am getting this error when I run npm run dev -- open
> svelte-kit dev "--open"

(node:3956) UnhandledPromiseRejectionWarning: file:///C:/Personal%20Files/www.kudadam.com.sveltekit/node_modules/@sveltejs/vite-plugin-svelte/dist/index.js:240
import { createMakeHot } from "svelte-hmr";
         ^^^^^^^^^^^^^
SyntaxError: The requested module 'svelte-hmr' does not provide an export named 'createMakeHot'
    at ModuleJob._instantiate (internal/modules/esm/module_job.js:92:21)
    at async ModuleJob.run (internal/modules/esm/module_job.js:107:20)
    at async Loader.import (internal/modules/esm/loader.js:179:24)
    at async file:///C:/Personal%20Files/www.kudadam.com.sveltekit/node_modules/@sveltejs/kit/dist/cli.js:975:19

(Use `node --trace-warnings ...` to show where the warning was created)
(node:3956) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:3956) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Reproduction
To reproduce,
npm install
npm run dev -- --open

Logs
No response
System Info
System:
    OS: Windows 7 6.1.7601
    CPU: (2) x64 Pentium(R) Dual-Core  CPU      E5300  @ 2.60GHz
    Memory: 361.25 MB / 1.95 GB
  Binaries:
    Node: 13.14.0 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Internet Explorer: 8.0.7601.19104
  npmPackages:
    @sveltejs/kit: next => 1.0.0-next.158
    svelte: ^3.34.0 => 3.42.4

Severity
blocking all usage of SvelteKit
Additional Information
No response

Node version is outdated, upgrade to the latest LTS version.

Thanks @ignatiusmb, You've made my day too, I just fixed my deployment on Cloudflare Page too


Introduce Test Project and simple unit test

I had to drop the dotnetcore dependency of ImageResizer from 1.0.1 to 1.0.0 to be able to reference the ImageResizer project from the xUnit ImageResizerTest project without causing version issues. Hope this isn't a problem.
Also to keep the ImageResizerTests project in this repository I had to put it in a subfolder of the ImageResizer project (tests) and then get the ImageResizer project to ignore this folder, and finally add the ImageResizerTests project to the solution.
I can now start increasing coverage of the code.

Cool -- a couple things:

Move non-test code into a "src" dir to match others' convention
Everything should use "netcoreapp1.1" at this point

On Feb 17, 2017 3:16 AM, "Mark Daniel Osborne"<EMAIL_ADDRESS>wrote:

I had to drop the dotnetcore dependency of ImageResizer from 1.0.1 to
1.0.0 to be able to reference the ImageResizer project from the xUnit
ImageResizerTest project without causing version issues. Hope this isn't a
problem.
Also to keep the ImageResizerTests project in this repository I had to put
it in a subfolder of the ImageResizer project (tests) and then get the
ImageResizer project to ignore this folder, and finally add the
ImageResizerTests project to the solution.
I can now start increasing coverage of the code.
You can view, comment on, or merge this pull request online at:
https://github.com/stevedesmond-ca/ImageResizer/pull/11
Commit Summary

Add in xunit testing framework
Create simple test for AspectRatio

File Changes

M ImageResizer.sln
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-0
(13)
M ImageResizer.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-1
(6)
A global.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-2
(3)
M project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-3
(13)
A tests/ImageResizerTests.cs
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-4
(33)
A tests/ImageResizerTests.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-5
(22)
A tests/project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-6
(19)

Patch Links:

https://github.com/stevedesmond-ca/ImageResizer/pull/11.patch
https://github.com/stevedesmond-ca/ImageResizer/pull/11.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stevedesmond-ca/ImageResizer/pull/11, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAL9ODEEGLOfkbXnHJ-GFulDaKGccdvEks5rdVdOgaJpZM4MD_Mn
.

I thought I was only creating test code. Can you be more specific?
I’ll try to upgrade to 1.1

On 18 Feb 2017, at 12:55 am, Steve Desmond<EMAIL_ADDRESS>wrote:
Cool -- a couple things:

Move non-test code into a "src" dir to match others' convention
Everything should use "netcoreapp1.1" at this point

On Feb 17, 2017 3:16 AM, "Mark Daniel Osborne"<EMAIL_ADDRESS>wrote:

I had to drop the dotnetcore dependency of ImageResizer from 1.0.1 to
1.0.0 to be able to reference the ImageResizer project from the xUnit
ImageResizerTest project without causing version issues. Hope this isn't a
problem.
Also to keep the ImageResizerTests project in this repository I had to put
it in a subfolder of the ImageResizer project (tests) and then get the
ImageResizer project to ignore this folder, and finally add the
ImageResizerTests project to the solution.
I can now start increasing coverage of the code.
You can view, comment on, or merge this pull request online at:
https://github.com/stevedesmond-ca/ImageResizer/pull/11
Commit Summary

Add in xunit testing framework
Create simple test for AspectRatio

File Changes

M ImageResizer.sln
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-0
(13)
M ImageResizer.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-1
(6)
A global.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-2
(3)
M project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-3
(13)
A tests/ImageResizerTests.cs
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-4
(33)
A tests/ImageResizerTests.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-5
(22)
A tests/project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-6
(19)

Patch Links:

https://github.com/stevedesmond-ca/ImageResizer/pull/11.patch
https://github.com/stevedesmond-ca/ImageResizer/pull/11.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stevedesmond-ca/ImageResizer/pull/11, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAL9ODEEGLOfkbXnHJ-GFulDaKGccdvEks5rdVdOgaJpZM4MD_Mn
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/stevedesmond-ca/ImageResizer/pull/11#issuecomment-280655561, or mute the thread https://github.com/notifications/unsubscribe-auth/AJ6wHBPfFoU11zPRDbPmCrWlBEUF70L8ks5rdabegaJpZM4MD_Mn.

--
n: Mark Daniel Osborne
e<EMAIL_ADDRESS>w: http://www.pcblues.com

You mentioned needing to drop the version number; I'd rather than TFM go up
On Feb 19, 2017 4:27 AM, "Mark Daniel Osborne"<EMAIL_ADDRESS>wrote:

I thought I was only creating test code. Can you be more specific?
I’ll try to upgrade to 1.1

On 18 Feb 2017, at 12:55 am, Steve Desmond<EMAIL_ADDRESS>wrote:
Cool -- a couple things:

Move non-test code into a "src" dir to match others' convention
Everything should use "netcoreapp1.1" at this point

On Feb 17, 2017 3:16 AM, "Mark Daniel Osborne"<EMAIL_ADDRESS>wrote:

I had to drop the dotnetcore dependency of ImageResizer from 1.0.1 to
1.0.0 to be able to reference the ImageResizer project from the xUnit
ImageResizerTest project without causing version issues. Hope this
isn't a
problem.
Also to keep the ImageResizerTests project in this repository I had to
put
it in a subfolder of the ImageResizer project (tests) and then get the
ImageResizer project to ignore this folder, and finally add the
ImageResizerTests project to the solution.
I can now start increasing coverage of the code.
You can view, comment on, or merge this pull request online at:
https://github.com/stevedesmond-ca/ImageResizer/pull/11
Commit Summary

Add in xunit testing framework
Create simple test for AspectRatio

File Changes

M ImageResizer.sln
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-0
(13)
M ImageResizer.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-1
(6)
A global.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-2
(3)
M project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-3
(13)
A tests/ImageResizerTests.cs
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-4
(33)
A tests/ImageResizerTests.xproj
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-5
(22)
A tests/project.json
https://github.com/stevedesmond-ca/ImageResizer/pull/11/files#diff-6
(19)

Patch Links:

https://github.com/stevedesmond-ca/ImageResizer/pull/11.patch
https://github.com/stevedesmond-ca/ImageResizer/pull/11.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stevedesmond-ca/ImageResizer/pull/11, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AAL9ODEEGLOfkbXnHJ-
GFulDaKGccdvEks5rdVdOgaJpZM4MD_Mn>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <https://github.com/
stevedesmond-ca/ImageResizer/pull/11#issuecomment-280655561>, or mute the
thread <https://github.com/notifications/unsubscribe-auth/
AJ6wHBPfFoU11zPRDbPmCrWlBEUF70L8ks5rdabegaJpZM4MD_Mn>.

--
n: Mark Daniel Osborne
e<EMAIL_ADDRESS>w: http://www.pcblues.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stevedesmond-ca/ImageResizer/pull/11#issuecomment-280906817,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL9OLsLmmHUh4ZtVShbHQY3CHDGWTQXks5reAr2gaJpZM4MD_Mn
.

No problems. I'll look at upgrading the netcoreapp version.
With respect to 1. Move non-test code into a "src" dir to match others' convention:
Where should I make the dir and which files should I put into it? I'm new to dotnetcore dev.

One more question. What if I can't get xunit to work with 1.1? I think I'll leave it for now.

just pushed a test project, which would supersede this


of2 does not run when installed using installation instructions

Environment : OSX
when I follow the instructions of2 command does not run.
When in the bin directory and I use
./of2

command executes correctly

Found the problem
I installed to
/Applications

and not to
~/Applications


Updated background image and font style

@Mohit049 are you going to make the changes?

@Mohit049 are you going to make the changes?


Move the CRDFinder to the cli-runtime repo

The CRDFinder and perhaps others in the util package are generic and very useful to be part of the cli-runtime https://github.com/kubernetes/cli-runtime repo

It turns out this is not very useful. I'm just going use bits of the code as reference


About the conditions in the diffusion model

Dear Author,
Your work has been a tremendous source of inspiration and assistance to me. Currently, I have a question pertaining to diffusion models: Does the scale of latent codes used as conditions influence the quality of image reconstruction? If I were to conduct an experiment to validate this, could you kindly advise on which specific sections or modules of the project I should concentrate on and potentially adjust? Your guidance would be immeasurably valuable in shaping the direction of my research.
I eagerly anticipate your precious insights and wish to express my deepest gratitude in advance for taking the time to read and respond amidst your busy schedule.

I'm not entirely clear on what you mean by "scale of latent codes." Are you referring to the number of dimensions? If that's the case, this paper in the appendix presents some findings on varying dimensionality. Typically, a high bitrate model necessitates more dimensions to achieve optimal distortion performance. However, the perceptual quality might differ, as a robust diffusion/GAN model can produce "fake" textures that appear perceptually convincing. I'm uncertain about the direction of your research. If your goal is to enhance the architecture, you might explore using transformers, attention mechanisms, etc. If you're aiming for a better compression ratio, more complex autoregressive entropy models or different quantization methods could be considered. For improved perceptual quality, refining the diffusion process could be beneficial. For instance, in my codebase, I experimented with using latent diffusion as the decoder. However, I discovered that while it improved perceptual quality, it tended to stray from "good distortion," leading me to abandon its use. There's indeed a trade-off between rate, distortion, and perception. Better perception does not equate to better distortion, and there's no definitive answer to this question. You might find this paper insightful.


Dataset keyframe annotation

I want to train ultrasound key frame extraction on thyroid ultrasound video, and specifically want to ask how to label ultrasound video key frames.

Sorry, I don't think this is a question the developers of this repo can help you with.


[Accessibility] Skip to main content link should be the first tab stop on the page

Currently the first tab stops are the languages in the upper right hand corner.

Fixed as part of accessibility audit


Make eslint configurable

In this fork, the Eslint construct accepts options (or undefined, in which case it falls back to the original config).
Here are the options:
interface EslintOptions {
  config: Linter.Config; // from definitely typed `@types/eslint`
  dependencies: Record<string, Semver>;
}

Here's an example of its usage:
import { Eslint } from './eslint';
import { JsiiProject } from './jsii-project';
import { Semver } from './semver';

const project = new JsiiProject({
+ eslint: false,
  name: 'projen',
  jsiiVersion: Semver.caret('1.5.0'),
  description: 'A new generation of project generators',
  repository: 'https://github.com/eladb/projen.git',
  authorName: 'Elad Ben-Israel',
  authorEmail<EMAIL_ADDRESS>  stability: 'experimental',
  dependencies: {
    'constructs': Semver.pinned('2.0.1'),
    'yaml': Semver.caret('1.9.2'),
    'fs-extra': Semver.caret('9.0.1'),
  },
  peerDependencies: {
    constructs: Semver.pinned('2.0.1'),
  },
  devDependencies: {
    '@types/fs-extra': Semver.caret('8.0.1'),
  },
  bundledDependencies: [ 'yaml', 'fs-extra' ],
  projenDevDependency: false, // because I am projen
  workflowBootstrapSteps: [
    { run: 'yarn bootstrap' },
  ],
});

+ new Eslint(project, /* ESLint options */);

project.synth();

@harrysolovay mind rebasing against master?

Will get around to this soon 👍

@eladb just updated. Also made sure it's working.

Gonna take another crack at this later. In the meantime, please let me know how you feel about the approach wherein a dictionary of pure override fns can be supplied. This could work for dependencies as well. The options interface would become...
interface EslintOptions {
  // would specify a generic identity fn type... but JSII throws error
  overrides: Record<string, any>;
}

Thoughts?

Closing for now. Feel free to pick up any time

Cool cool. @eladb please let me know your thoughts on the comments above.


test: remove incorrect error_t type

These functions return int, and error_t is only a Linux type, and not supported on FreeBSD.

Very strange. I'm not even aware of error_t type.


STORM-638:UI should show up process-id of the Worker to which an Executor is assigned

Pay Attention: conflicts  with https://github.com/apache/storm/pull/296

I'm not convinced having the PID on the UI is really that helpful, but it does provide more information for possible automation or monitoring, so I am OK with it.

@revans2 not quite comfortable with exposing pid on UI. UI is supposed to be available for all the users. In non-secure env it can be run without any filter so I am against exposing pid on the UI.

@harshach, If you are running the UI without a filter I would not consider that secure.  There is other more damaging information exposed on the UI already.  Also what is having the pid of the process going to let me do as an attacher?  If I cannot log onto the box the pid is really rather useless.  If I can log onto the box I can already get the pid by running ps -aef | grep '<topology-id>-<port-number>'.  Granted I am not a malicious individual so perhaps I am missing something or perhaps an attach will show up in the future that is somehow related to the pid.  It is totally possible, but I don't see it as being that likely.
@caofangkun, The generated thrift code no longer compiles.  It looks like you regenerated with thrift7, and we have moved on to thrift 0.9.2.  Please regenerate the code again.

Sorry it is not the code generation after all.  Although the  comment in the generated code with a 0.7.0 is off putting, and I would prefer it to be generated with 0.9.2.  by making the field optional it changes the constructor, so now backtype/storm/daemon/nimbus.clj:1325:47 is failing to compile, because the constructor is different.

unnecessary

-0
I don't really think this is a useful thing to add to the UI. If we want to add it to the JSON API, I'm okay with that, but let's not display it on the UI.

Agreed on @knusbaum.
It can reduce one ps command for actual users to operate (ps, kill, etc.) specific worker by hand, but I'm curious that there're other benefits.
I also think that since we're already defining "port", we would be better to encapsulate worker process into "port" and provide a way to operate such things via "port".
Or we can think about providing workers' information (usage of CPU, MEM, etc. but except secure things) into UI. Maybe we could provide a way to kill specific worker from UI, but I wonder there's such use cases.

Closing this.


Could NerdTree show file type?

What I mean is, ... to show if current file is a Class, in Interface, a Trait, or something like that. Is this a possibile?

Try this: https://gist.github.com/scrooloose/a60ebc0d685c6f5535a2
It will put flags beside class/interface files. The interesting part is how to check a file to determine what type it is.

The easiest way would simply be to use the full pathname - e.g. if we have a dir structure like this:  /path/to/project/class_files/foo.java then we can trivially match the filename.
My example in the gist greps every single file that is loaded into the tree to see if it is a class/interface. This is pretty hamfisted... but it does the job for demo purposes.
Let me know how you get on.


csv Error when "open(filename,"rb") as csvfile" with python 3.6.2

it is day16 practise:
with open(filename,"rb") as csvfile,temp_file:
i got error as below:
Traceback (most recent call last):
File "rw_CSV_function.py", line 56, in
edit_data(email="justin@hotmail.com",amount=99.99,sent="")
File "rw_CSV_function.py", line 38, in edit_data
writer.writeheader()
File "C:\Python36\lib\csv.py", line 144, in writeheader
self.writerow(header)
File "C:\Python36\lib\csv.py", line 155, in writerow
return self.writer.writerow(self._dict_to_list(rowdict))
File "C:\Python36\lib\tempfile.py", line 483, in func_wrapper
return func(*args, **kwargs)
TypeError: a bytes-like object is required, not 'str'
i don't know how to fix this problem, can u please help me?

I change code to this and it works on python 3.5.4 :
def edit_data(edit_id=None, email=None, amount=None, sent=None):
filename = "data.csv"
new_rows = []
temp_file = "data2.csv"
with open(filename) as csvfile:
    reader = csv.DictReader(csvfile, delimiter=';')
    fieldnames = ['id', 'name', 'email', 'amount', 'sent', 'date']
    for row in reader:
        # print(row['id'] == 4)
        if edit_id is not None:
            if int(row['id']) == int(edit_id):
                row['amount'] = amount
                row['sent'] = sent
        elif email is not None and edit_id is None:
            if str(row['email']) == str(email):
                row['amount'] = amount
                row['sent'] = sent
        else:
            pass
        new_rows.append(row)

with open(temp_file, "w", newline='') as tempfile:
    writer = csv.DictWriter(tempfile, fieldnames=fieldnames, delimiter=';')
    writer.writeheader()
    writer.writerows(new_rows)
shutil.move(temp_file, filename)

hmm I get
Traceback (most recent call last):
File "", line 6, in
NameError: name 'edit_id' is not defined
any idea why? @Sunip88
also did this code work for you @xiongdingding

You should not read and write to the same file at the same time:
with open(filename, "r") as csvfile, temp_file:
Even though you give this file two names, they are point to the same file in you disk, change it to:
with open(filename, "r") as csvfile, open('temp_file.csv', 'w') as temp_file:
Tried this its working #xiongdingding

Thanks alot @yusufshaikhai  it worked for me.

You should not read and write to the same file at the same time:
with open(filename, "r") as csvfile, temp_file:
Even though you give this file two names, they are point to the same file in you disk, change it to:
with open(filename, "r") as csvfile, open('temp_file.csv', 'w') as temp_file:
Tried this its working #xiongdingding

Are you have another solution? not working at me

A new version of this course is coming soon. Thank you.


Update main.yml

Description
[Description of the changes you are submitting]
Checklist

[ ] Have you tested the change you are submitting?
[ ] Is the commits history nice and clean?

@DZZN-DEV WTF

problem is tho, the matrix was specifically designed to build for devices known to work on droidboot, like vayu, setting the buildability to any device and releasing them is prone to unexpected stuff happening on said device

Please squash commits and provide justification for the changes in the squashed commit message and PR description. This PR is not justifiable in its current state.


Fix yaml as @ is forbidden in YAML 1.2

http://www.yaml.org/spec/1.2/spec.html#id2772075
(PR done on 2.5 branch, but this issue persists also in 2.6, 2.7 and master)

Thank you @jdespatis.


Typo

Hello,
There is a typo in the section called "End-User Experience" above and in the following text.
EXISTING TEXT:
"An example user is on the Contoso network, but is trying to access the Fabrikam instance of a shared SaaS application like Outlook online. If Contoso is a non-permitted tenant for that instance, the user sees the following page:"
CORRECTION REQUIRED:
The second line says: "If Contoso is a non-permitted tenant for that instance, the user sees the following page:"
Instead, it should say:  "If Fabrikam is a non-permitted tenant for that instance, the user sees the following page:"
RATIONALE:
If the user is trying to access the Contoso tenant from within the Contoso network, then the user will obviously need the ability to access the Contoso tenant as it is the same company.
However if the user is trying to access the Fabrikam tenant, which is another company's tenant, from Contoso's network, and if Fabrikam's tenant is not specified in the string "Restrict-Access-To-Tenants", then the user should see the error message whose picture is shown below the text.
Makes sense?
Thanks!

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

ID: 89ea5c2d-4602-25b8-3cdd-60cfb5c31895
Version Independent ID: 35d4af7e-72e4-7c40-9dd6-b8490ee73f0d
Content: Manage access to cloud apps by restricting tenants - Azure
Content Source: articles/active-directory/manage-apps/tenant-restrictions.md
Service: active-directory
GitHub Login: @barbkess
Microsoft Alias: barbkess

@ManojReddy-MSFT - Thanks for the update, but one minor correction is required. You spelt Fabrikam as "Fabricam", so just correct the spelling and the article would be perfect! Thanks a lot :)


zwave: Update Open-ZWave version

Fixes https://github.com/home-assistant/core/issues/36649 with https://github.com/OpenZWave/open-zwave/issues/2246
Diff: https://github.com/OpenZWave/open-zwave/compare/d2de6990389888850e936894080e4096c822e379...1661e647d42c3e5a59f8291eabbdf2854a938b97

This is actually not the latest version (at the time of the creation of this PR).
Any specific reason for that?

@frenck that was not my intention... I created this PR immediately after https://github.com/OpenZWave/open-zwave/pull/2247 was merged.  I'd be happy to update to the current HEAD if you'd prefer.

Yes, lets bump it to 6cec95bbb128e6a287ba8184d0281526f2edd195.
While you are at it, could you bump ozw-admin to 167180c9ac64ffd699f8c4048ecb8b6c5532d86d?
Please make sure to bump the version of the add-on a patch version and add both the changes to the changelog.
Thanks 👍

Alright, should be good-to-go.

I fixed merge conflicts.


Support for prelaunch_simulator to launch all simulators passed in devices

Feature Request
Motivation Behind Feature
When running a set of tests I can prelaunch a simulator but if we're testing on a large number of devices it would be helpful to prelaunch them all, otherwise prelaunching doesn't offer such a great time saving.
Feature Description
When calling run_tests with prelaunch_simulator set to true, all simulators passed in the devices should be launched.
Alternatives or Workarounds
I did a quick google but couldn't find anything. A workaround could be to have another step before run_tests that launches all the simulators you need.

Thanks @fastlane-bot! This hasn't been fixed yet though. Please don't close the issue :)

This would be a great improvement, indeed 🙏

You’re doing a great job Man,Keep it up.

Thanks @fastlane-bot! This hasn't been fixed yet though. Please don't close the issue :)


Any way to fetch user's Canned Email Responses?

Hi, I'm writing an Chrome Extension, and I'm looking to find a way to fetch all Canned Email Responses (it's an GMail feature that needs to be turned on through Settings>Labs and enables users to save predefined messages and then through compose window they can access them).
I couldn't find a method in your tool gmail.js, but I suppose someone could know a way around it maybe?

Sorry about the extremely slow response.
To answer your question: There might be, but I honestly don't think anyone on the core team will be working on it anytime soon.
Since I haven't heard anything new on this issue, I'm closing it. Please re-open it if you still have an issue you need resolved.


Include pattern is only applied when watching files

Include pattern is only applied when watching files.
I expect it also being applied when run in non-watching mode.

Thank you for finding this bug. Its fixed now and published to npm as version 1.2.3


Add ClickHouse event sink support

Replaces all "coursegraph" work with the new event-sink-clickhouse. Mostly this is renaming things, but it does install the event-sink-clickhouse plugin and configure it to talk to the Tutor-configured ClickHouse. Unfortunately this means that many downstream Superset assets needed to be deleted since they referenced the old names of things. New reports will be created as part of a separate set of tasks that Axim is contracting out.

@bmtcril Can you rebase this branch?

I know the testing instructions are using a local env instead of a development one but either way, I tried with one, and it's failing when initializing:
tutor dev do init
...
Running schema creation scripts...
Code: 210. DB::NetException: Connection refused (clickhouse:9000). (NETWORK_ERROR)

Is dev not supported yet?

@bmtcril I think you are missing adding the course_overview table schema in the clickhouse init job

@mariajgrimaldi I just saw the issue with clickhouse connection that you saw on dev. It happened because I was trying to init before clickhouse was actually started. I tried it 2 secs later and it worked. I'm not sure how to solve this without putting a health check in, which we've seen requires a newer Docker Compose. 🤔

I think this is ready for final review if anyone else wants to take a look before tomorrow morning.

@Ian2012 @mariajgrimaldi yeah everything I know of is fixed

@bmtcril Just one more thing, can you upgrade the event-routing-backends requirements to 5.3.0?


Domain not existing: publicapi.txdpsscheduler.com

Was the api endpoint changed or something? I started getting this a few days ago. With a quick nslookup I was able to confirm that the domain is non existing. Not a Node JS guy so I am not sure if I am looking in the right direction. Any help would be highly appreciated.

track issue at #156


To many Issues ?

When I generate the Changelog i get the following error:
Warning: Can't finish operation: GitHub API rate limit exceeded, change log may be missing
some issues. You can limit the number of issues fetched using the `--max-issues NUM` argument.

If i reduce the number of issues, the changelog generator is finishing, but than the changelog itself is missing some (random) issues. (that can easily be seen in the diff).
Is there an other way to configure the changelog generator ?
(i am working with daily / weekly releases, so i have a huge amount of releases & issues).
My repository is:
https://github.com/TheFive/osmbc

I have installed the newest version with gem install today.

My actual idea is to put all release notes before 1.0.0 to history.md and generate only the later one, but i did not find a configuration for the changelog generator to manage that.

I going to implement this logic in next releases.
Close in favor of #209.


[Attempt to Make New User Discoverable]

Hello, Gene
I add email as tag for new user so it could be discovered by others. I also explicitly set defacs for newly created user to "JWRPS" just in case server give incorrect default value.
Let me know your opinion.
Thanks.

You have removed the auto-login option login: true.
https://github.com/tinode/example-react-js/pull/5/commits/d051c84737bfb35ced930c013f4971b8761b31bf#diff-eacf331f0ffc35d4b482f1d15a887d3bL2510
Why? The result of form submission without the auto-login is confusing to the user.

Because in createAccountBasic() located in tinode.js, the login parameters for createAccount() is always set to true.
So even if we set {login: false} or not set it at all, newly created user would always be login.

I made some changes for code style and generalised the function. Please take a look.


Build a To-Do List app with React

I would like to start this but need to learn about cockroachdb, never heard of it. Anyway, I'm gonna complete it.

Here are some resources to help you learn cockroachdb:
https://www.cockroachlabs.com/cockroach-university/
https://www.cockroachlabs.com/docs/stable/

Please help me check my PR: https://github.com/cockroachdb/cockroachdb-todo-apps/pull/26

Go for it!


Empty nodepools do not report usage metrics or have a status

Description
Observed Behavior:
I have a nodepool responsible for autoscaling nodes to run tasks. When no tasks are needing to be run, the nodepool scales to 0 nodes. While there are 0 nodes in the pool, the nodepool does not have a status and does not report the karpenter_nodepool_usage metric. This causes some issues with monitoring, as alerts setup to monitor usage notify on no data whenever the nodepool scales to 0 nodes.
Expected Behavior:
While there are 0 nodes in the pool, I would expect the nodepool to show a status as follows:
{
    "resources": {
      "cpu": "0",
      "ephemeral-storage": "0Ki",
      "memory": "0Ki",
      "pods": "0"
    }
}

I would also expect it to report the karpenter_nodepool_usage metric with a 0 for the relevant resource types.
Reproduction Steps (Please include YAML):

Create a nodepool with disruption/consolidation enabled and a taint to only allow specific workloads to run on these nodes:

apiVersion: karpenter.sh/v1beta1
kind: NodePool
metadata:
  name: tasks-nodepool
spec:
  template:
    spec:
      requirements:
        - key: karpenter.k8s.aws/instance-category
          operator: In
          values:
            - c
            - m
            - r
            - t
        - key: karpenter.k8s.aws/instance-cpu
          operator: In
          values:
            - "4"
            - "8"
            - "16"
        - key: kubernetes.io/arch
          operator: In
          values:
            - amd64
        - key: karpenter.sh/capacity-type
          operator: In
          values:
            - spot
            - on-demand
      nodeClassRef:
        name: default
      taints:
        - effect: NoSchedule
          key: workload
          value: tasks
  limits:
    cpu: 1000
  disruption:
    consolidationPolicy: WhenUnderutilized
    expireAfter: 720h

Ensure no nodes are created in this nodepool:

> kubectl get nodes -l karpenter.sh/nodepool=tasks-nodepool
No resources found

Check to see the status of the nodepool, and ensure an empty object is returned.

> kubectl get nodepool/tasks-nodepool -o json | jq .status
{}

Check the metrics endpoint (karpenter.karpenter.svc.cluster.local:8000/metrics) and ensure the karpenter_nodepool_usage gauge metric does not include data for the tasks-nodepool nodepool.

Versions:

Chart Version: 0.32.4
Kubernetes Version (kubectl version): v1.28.4-eks-8cb36c9

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Is it empty nodepools or new nodepools?

Empty nodepools. Once nodes are created for a pool, metrics will be reported and the nodepool will have a status. When the nodes are removed due to being empty, the metrics are no longer reported and the status is gone.

Quoting from my comment on a Slack thread

Looking at the metrics code, the root cause is likely the nodepool status not containing resources when no nodes are in the pool. Seems the resourceCountsFor method responsible for setting the resources status doesn't handle the case when there are no nodes owned by this nodepool. https://github.com/kubernetes-sigs/karpenter/blob/v0.32.4/pkg/controllers/nodepool/counter/controller.go#L77-L94

This makes sense to me. One solve that fixes the status that's returned in the NodePool and what's returned by the usage metric. We should note that this is a breaking change, though (I doubt there is a large base of users that are relying on the empty resource status behavior, but it's something that we should callout when we make the change).

@mattsre Any chance you would be willing to open a PR with the fix?

@jonathan-innis I took a crack at this in #940
While making the PR I noticed there was a test that explicitly ensured the resources status was nil, figured I'd mention in case you knew of some reason this needs to nil when there are 0 nodes that I'm not aware of.

@jonathan-innis I took a crack at this in #940 While making the PR I noticed there was a test that explicitly ensured the resources status was nil, figured I'd mention in case you knew of some reason this needs to nil when there are 0 nodes that I'm not aware of.

I'm not aware of any. It's possible that we just wrote the test against the behavior ;)

/remove-lifecycle rotten

I'm seeing that the resources are 0 on my NodePool. Closing this out as solved
    resources:
      cpu: "0"
      ephemeral-storage: "0"
      memory: "0"
      nodes: "0"
      pods: "0"


gridfs as output stream

Hi there, thanks for the great package.
Can we use gridfs as output stream? I used to do it by writing to a temp location then piping into gridfs before. I suspect now it is possible (after upgrading to v2.0.1) to pipe directly to gridfs without temp file, but the following doesn't work for me.  Is it possible, if so how, what am I doing wrong?
"ffmpeg exited with code 1: pipe:1: Invalid argument"
var writeStream = gridFs.createWriteStream({ filename: updatedSaveLocation,
                    metadata: { actualFileName: fileName } });

new FFmpeg({source: sourcePath, timeout:0})
    .on('error', function(err) {
        console.log('An error occurred: ' + err.message);
    })
    .on('end', function() {
        console.log('Transcoded successfully!');
    })
.pipe(writeStream);

Hi,
What ffmpeg version are you using ? On which OS ?

You need to specify output format with .outputFormat('format')

@caglarcem yes, @phaux is right. You always have to specify the output format when using a stream.

Thanks guys it worked!
On May 12, 2015 9:28 PM, "Nicolas Joyard"<EMAIL_ADDRESS>wrote:

@caglarcem https://github.com/caglarcem yes, @phaux
https://github.com/phaux is right. You always have to specify the
output format when using a stream.
—
Reply to this email directly or view it on GitHub
https://github.com/fluent-ffmpeg/node-fluent-ffmpeg/issues/393#issuecomment-101251167
.


Replay an episode

When you want to watch an episode again, it's almost impossible to restart it.
After starting, you have to push rewind within 0.5 seconds. Otherwise it just stops, because the watch bar is at the end.
Maybe, when starting an episode, and there are just 5-10 seconds remaining, start it from the beginning.

I've just pushed a commit that will restart the video from the beginning if progress is > 95%. This is inline with how the rest of the apps / resume points works.
Fix will probably be included in 1.2.0 release but not yet sure when I'll release it.

Fixed in 1.2.0