Storybook failing in Internet Explorer 11

SCRIPT1002: Syntax error
vendors~main.661e6dc9dcb24086488e.bundle.js (183508,1)

/**
 * @author mrdoob / http://mrdoob.com/
 * @author Mugen87 / https://github.com/Mugen87
 */

// BoxGeometry

class BoxGeometry extends Geometry {
...

Close issue.
Wont fix IE-related issues. Don't want to introduce workarounds for slow/unpopular/outdated browsers.


Fixes bug in lax.con_shape_tuple when using (None,) as part of an input shape.

I came across this bug this morning: https://gist.github.com/nirum/6f7f345bc6b46224f36cc40c73a8147c
The error is due to the dtype of the lhs_shape argument being 'object' instead of a numeric type, which causes onp.ceil to fail.
This fix casts the output of true_divide to float before the call to onp.ceil

Thanks for identifying this issue!
I'm not sure this is the right fix, though. I don't think we want to support None in the input_shape argument to a stax init_fun. It should be a tuple of integers. We can use -1 when we don't want to commit to a batch shape, though even that's a bit sloppy (because we really want to create a shape variable, but since the abstract evaluation shape rules were written to act on tuples of integers, using -1 is a hacky shortcut).
Is there a place in our docs or examples where we suggest a None in the input_shape? If so, we should revise it, because None shouldn't be allowed there.

Ah, ok, cool--I didn't realize None wasn't supported. I didn't see it in the docs, I just tried it and it worked when I used padding='VALID', but it makes sense to use -1 instead.


trivial addition of a twitter adapter factory test. largely to valida…

…te coveralls

Coverage increased (+0.08%) to 23.134% when pulling be33baa169491285dee19328a0ece73f42fb4b76 on feature/trivial-twitter-test into 2ac7ee5ce00e9f8d624d857176fb9918284fd6c2 on master.


Instrument project with DiagnosticSource

In order to better identify and root cause issues, it would be helpful to get in depth logs as to what the system is doing. DiagnosticSource would be ideal for this. I haven't used it personally, but wanted to track a work item to add support for it within the project.

@twsouthwick Hi there. I've wanted to open new issue here but it might be not appropriate, so let me ask my question here.
Is there any approach to get depth, lineNumber or position of current node?
As it's done using native XmlTextReader: https://docs.microsoft.com/en-us/dotnet/api/system.xml.xmltextreader.lineposition?view=netcore-3.1
while (reader.Read()){
       switch (reader.NodeType){
         case XmlNodeType.Element:
           Console.Write("{0} {1},{2}  ", reader.Depth, reader.LineNumber, reader.LinePosition);
           Console.WriteLine("<{0}>", reader.Name);
           break;
       }
    }

I have tried to get Depth of OpenXMLReader, but it seems that it is returning depth of element in some stack which is not real depth of current xml node.
I would appreciate any answer. Thank you in advance.

Feel free to open a new issue if you find a limitation of the sdk. I'm not available to look into it right now, but maybe others will. We want to keep issues scoped to single issues


Image displayed with wrong date and wrong location

I wonder which information is used for location and data. I assume the metadata embedded in the image.
I saw an image displayed to be taken on 15.09.2020 in Adiri (Libyen), but in reality the image was taken in November 2018 at Gran Canaria. I checked the data with XNView MP where EXIF Aufnahmedatum is 18.11.2018 and GPS is shown to be Gran Canaria.
Transferred blured image for debugging purpose. FIle and EXIF modification date is now, but EXIF Aufnahmedatum and GPS is original.
I assume, that's the image but of cause, maybe your tool found a duplicate which I'm not aware of...
Pls. give a note, when downloaded the image, that I can remove the link above...

Thanks for your report, I've downloaded the image.

I assume not connected, but I don't want to flood with issues.
Just got this
this-week-in-past  | thread 'actix-rt|system:0|arbiter:0' panicked at 'called `Result::unwrap()` on an `Err` value: SqliteFailure(Error { code: DatabaseBusy, extended_code: 5 }, Some("database is locked"))', src/resource_store.rs:130:14
this-week-in-past  | stack backtrace:
this-week-in-past  | note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
this-week-in-past exited with code 139

I have started the app via docker-compose:
...
    environment:
      BIGDATA_CLOUD_API_KEY: xxx
      WEATHER_ENABLED: true
      WEATHER_LANGUAGE: de
      OPEN_WEATHER_MAP_API_KEY: xxx
      RESOURCE_PATHS: /resources/2022,/resources/2021,/...
      RUST_BACKTRACE: 1
      TZ: Europe/Berlin
      PUID: 33
      PGID: 33

A chance to provide helpful debug info?

I actually found a bug in gps date / lat, long reading. Thank you for reporting that.
The other thing is also interesting.
Was you viewing the "website" while the scanning / indexing was in progress?
If so, there might be also a bug.

I just pushed a fix for the first bug reported from you, thank for the test photo, it helped me a lot.
I'll most likely merge this fix this evening. I'll keep you up-to-date.

Was you viewing the "website" while the scanning / indexing was in progress?

I don't think so. AFAIK the webpage stays black till the scan is done...
But meanwhile I made the debug from 1 to full and when I started the app I got this:
this-week-in-past  | [2023-09-14T13:31:08Z INFO  this_week_in_past::scheduler] Found 20121 resources
this-week-in-past  | [2023-09-14T13:31:09Z INFO  this_week_in_past::scheduler] Purging resources store
this-week-in-past  | [2023-09-14T13:31:10Z INFO  this_week_in_past::scheduler] Cleanup cache
this-week-in-past  | [2023-09-14T13:31:11Z INFO  this_week_in_past::scheduler] Inserting new resources
this-week-in-past  | [2023-09-14T13:31:14Z INFO  this_week_in_past::scheduler] Cleanup database
this-week-in-past  | thread 'actix-rt|system:0|arbiter:1' panicked at 'Insertion of 5c28f8d86e713cccf4179a189d5de970_2560_1415 failed', src/resource_store.rs:83:33
this-week-in-past  | stack backtrace:
this-week-in-past  |    0:           0x6fa498 - <unknown>
this-week-in-past  |    1:           0x570894 - <unknown>
this-week-in-past  |    2:           0x6da06c - <unknown>
this-week-in-past  |    3:           0x6fbb48 - <unknown>
this-week-in-past  |    4:           0x6fb2e0 - <unknown>
this-week-in-past  |    5:           0x6fc328 - <unknown>
this-week-in-past  |    6:           0x6fbd8c - <unknown>
this-week-in-past  |    7:           0x6fbd00 - <unknown>
this-week-in-past  |    8:           0x6fbcf4 - <unknown>
this-week-in-past  |    9:           0x408d6c - <unknown>
this-week-in-past  |   10:           0x4fc830 - <unknown>
this-week-in-past  |   11:           0x4ed764 - <unknown>
this-week-in-past  |   12:           0x4e4ab4 - <unknown>
this-week-in-past  |   13:           0x4231a4 - <unknown>
this-week-in-past  |   14:           0x4296a4 - <unknown>
this-week-in-past  |   15:           0x431ad8 - <unknown>
this-week-in-past  |   16:           0x4d2af8 - <unknown>
this-week-in-past  |   17:           0x70f354 - <unknown>
this-week-in-past  |   18:           0x537160 - <unknown>
this-week-in-past  |   19:           0x535acc - <unknown>
this-week-in-past  |   20:           0x534694 - <unknown>
this-week-in-past  |   21:           0x6fd1d4 - <unknown>
this-week-in-past exited with code 139

You can also set RUST_LOG=debug.
Ok I have a feeling what went wrong...
How much folders did you set in the "RESOURCE_PATHS" variable?
Can you please try to set only one, and check if the error persists?

Can you please try to set only one, and check if the error persists?

Changed to one folder... BUT: The problem does not occur every time. Therefor not sure, when it is safe ;)
It now ran stable for one hour.

I've pushed a test docker image tag: 16957a
Would be nice if you could test, if this test version fixes your location / date issue.

I assume, you ask me to pull like...  -  which I did right now
version: '3.9'
services:
  this-week-in-past:
    image: rouhim/this-week-in-past:16957a

But do you have an idea how to test?
The images had been shown because of the bug. Now I should not see them anymore. Could take a while to be reasonably sure. And if the aren't shown anymore, I can't check the location...

Maybe copy that one image that was broken to a new folder, then only set this new folder in RESOURCE_PATHS

Okay, did so.
GPS: Now the location is named San Bartolomé de Tirajana which is correct. But I need to google to check if it is near Maspalomas (which is the common name for that place).
Doublechecked that place with the API...
-> Solved and (now) works like designed.
Maybe extending the displayed information by principalSubdivision? Not sure, if this is always useful, but in that case it would state Las Palmas, which is more common.
Date: The date is now display correct, although I wonder, why I see pictures taken in November.
-> Solved
Pls. ping me, if you publish a new version; otherwise I would stay on the tagged one 😉

The date is now display correct, although I wonder, why I see pictures taken in November.

To cite myself:

If no images from the calendar year are found, random images are displayed.

Pls. ping me, if you publish a new version; otherwise I would stay on the tagged one 😉
Ok, this is a temporary tag, but if there is a final release I'll let you know.

Ups, now there is an image shown without a place, although it has embedded GPS data. Can share the image, if useful...
Yes please

Okay, regarding the test I did... Got the RTFM 😇
BUT, now I have nearly 20k images in several resources (folders) and still see random images, although there are enough memories.
One example for an image where no location had been displayed.
For your performance statistics: RPi4 with 8 GB and 1 TB ssd:
this-week-in-past  | [2023-09-15T07:28:30Z INFO  this_week_in_past::scheduler] Found 19877 resources
this-week-in-past  | [2023-09-15T07:28:30Z INFO  this_week_in_past::scheduler] Purging resources store
this-week-in-past  | [2023-09-15T07:28:30Z INFO  this_week_in_past::scheduler] Cleanup cache
this-week-in-past  | [2023-09-15T07:28:31Z INFO  this_week_in_past::scheduler] Inserting new resources
this-week-in-past  | [2023-09-15T07:28:33Z INFO  this_week_in_past::scheduler] Cleanup database
this-week-in-past  | [2023-09-15T07:28:38Z INFO  this_week_in_past::scheduler] Job done in 181s!

Nice, thanks for your help :)
I will nevertheless take a look at the broken picture.
Thanks for sharing the log :+1:
It is always exciting to see how the programme performs on different devices.

Nice, thanks for your help :)

Very welcome. Like your app.
Found 33011 resources -> Job done in 289s! - Same Pi.

An idea how to embed the app output (see displayed images) to a lovelace widget?

One example for an image where no location had been displayed. (But don't spend to much time in analyzing the image. Might have been a kind of interference - app restarted and browser not refreshed.)

Great, found another little issue there :) Thank you very much!

An idea how to embed the app output (see displayed images) to a lovelace widget?

I'm using this: https://www.home-assistant.io/dashboards/iframe/

The fix is merged and should be available on the "latest" tag in about 10 mins.

Also the hidden folder feature is out.

Pulled the new image. How would I check, that this is the right version?
Maybe you could extend the log...

Again had a:

thread 'actix-rt|system:0|arbiter:0' panicked at 'Insertion of f17babb634fb545e6792fcbd7fff1231_2560_1415 failed', src/resource_store.rs:83:33
stack backtrace:
   0:           0x6fa498 - <unknown>
   1:           0x570894 - <unknown>
   2:           0x6da06c - <unknown>
   3:           0x6fbb48 - <unknown>
   4:           0x6fb2e0 - <unknown>
   5:           0x6fc328 - <unknown>
   6:           0x6fbd8c - <unknown>
   7:           0x6fbd00 - <unknown>
   8:           0x6fbcf4 - <unknown>
   9:           0x408d6c - <unknown>
  10:           0x4fc830 - <unknown>
  11:           0x4ed764 - <unknown>
  12:           0x4e4ab4 - <unknown>
  13:           0x4231a4 - <unknown>
  14:           0x4296a4 - <unknown>
  15:           0x431ad8 - <unknown>
  16:           0x4d2af8 - <unknown>
  17:           0x70f354 - <unknown>
  18:           0x537160 - <unknown>
  19:           0x535acc - <unknown>
  20:           0x534694 - <unknown>
  21:           0x6fd1d4 - <unknown>

Got an image displayed to be taken at 02. October, although EXIF says 19. September.
Can share, if useful.

pulled to early ;)
Now I see log messages like because it contains a .ignore file


Question about DistributedDataParallel

I only need to train my network with InplaceABN in single GPU and I have already read about https://github.com/mapillary/inplace_abn/issues/84.
Now I have some problem about implementing DistributedDataParallel in my code. I wonder if it is necessary to use it when I only use InPlaceABN in my code.

@geekboood no that's not needed by InPlaceABN. It is required if you want to use the synchronized version.

@rotabulo Thank you for the answer! By the way, I encounter a CUDA OOM problem when I change the learning rate, and I have never meet that when I using traditional BatchNorm. Do you have any idea about that? Is this InplaceABN need to consume more memory when I change the learning rate?

@geekboood no there is no relation at all, so you probably have other issues in the code.

@rotabulo OK, thanks you for the answer. It turns out that although this InplaceABN could increase the batchsize and the performance somehow may benefit from that, in semantic segmentation scenario the performance of ResNeXt101 is worse than SE-ResNeXt50. Our dataset is relatively small and we must utilize pretrained models. Can I load the pretrained model for BN into a InplaceABN modified model? Will it save some memory when I disable the activation in InplaceABN and use it as a  normal BN?


Debian installation instructions still use /etc/apt/trusted.gpg.d for storing the key

This violates https://wiki.debian.org/DebianRepository/UseThirdParty, which states:

... The certificate MUST NOT be placed in /etc/apt/trusted.gpg.d or loaded by apt-key add.
If future updates to the certificate will be managed by an apt/dpkg package as recommended below, then it SHOULD be downloaded into /usr/share/keyrings using the same filename that will be provided by the package. If it will be managed locally , it SHOULD be downloaded into /etc/apt/keyrings instead.

Please update the instructions accordingly.
Maybe also take into account that Debian 13 and Ubuntu 24.04 will default to the s.c. "deb822" style *.sources files in favor of the old *.list files in /etc/apt/sources.list.d. See man 5 sources.list for the details.

From a quick glance at the third-party repository link, we want mkdir -p /etc/apt/keyrings and to place the certificate there?

Yes, except I'd make that [[ -d /etc/apt/keyrings ]] || mkdir -p /etc/apt/keyrings.
Oh, and btw., ...| tee somefile > /dev/null can be replaced with ... >somefile.

Or, even better: Also provide a package for managing the keyring file going forward, in which case the instructions for the user would change to [[ -d /usr/share/keyrings ]] || mkdir -p /usr/share/keyrings (and place the bootstrap keyring file there).


[Discussion] Should RandomGenerator be public or internal?

It seems to me that the RandomGenerator class maybe shouldn't be public, but rather it should be internal since it seems to be (at least almost) a complete duplicate of GetRandom. GetRandom uses this class exclusively to do all of its random generation, so exposing RandomGenerator could become confusing to certain users.
Is there something I'm not seeing? Should things be kept as-is to avoid any breaking changes? Should this be changed to internal and released as version 7.0.0?

If it's a duplicate, then obsolete or internal. I would prefer obsolete because why have 2x versions of the same code?
yes, those 2 options are breaking changes, etc. which is fine with a major semver release IMO.

@PureKrome I'll have to do some more looking into this and create a list of shared functionality. Initial glance at least revealed a significant amount of shared functionality.
My current preference is that whatever exists in RandomGenerator but not in GetRandom should be ported over, then RandomGenerator deleted. I would be in favor of this breaking change and releasing a new major version as per semver.
I'll do further looking at it this week and post back with some updated findings and maybe an updated proposal.

@crmckenzie Are you available to address the lingering PRs? :)


Bridge request for mail.tm

Bridge request
mail.tm is a free to use Mail Service. It automatically deletes Mails older than 10 Days. It also has an API, so it could be more easy to implement this. Using this Bridge we could subscribe to Newsletter, which are only over mail, with our RSS clients.

General information

Host URI for the bridge (i.e. https://mail.tm):

Which information would you like to see?
The E-Mail with Date, subject and content, maybe also links to attachments.

Which of the following parameters do you expect?

[X] Title
[ ] URI (link to the original article)
[X] Author (Who Send the Mail?)
[X] Timestamp
[X] Content (the content of the article)
[X] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[X] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

https://github.com/heithemmoumni/mail.tm

https://docs.mail.tm/


Test: Added a --showCommands option

Add a new writter SSHMetaLogs where all commands executed over SSH are
saved in the commandslogs variable. By default
test_results/${testname}/cmds.logs
To simplify the last AfterEach also created a new helper function
CreateLogFile.
On the other hand, I added a new helper Writer, where all the data is
stored in the buffer and streamed to a new io.Writer. This allows use to
have --showCommands options in the cilium config and stream the data to
os.Stdout.
Fixes #2843
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

Is this actually fixing #2843? It seems like you are not printing out command output, only what commands were executed, while the issue was about adding command output to stdout.

The output is on test_suite, where the ShowCommands flag changes the SSHCommandslogs io.Writer to os.Stdout

test-me-please

test-me-please

test-me-please

Test failure looks unrelated, as this change is related to the test framework itself.

test-me-please


feat: Add expectNextN to StreamTestKit for javadsl.

Motivation:
Add an expectN method to StreamTestKit which supports java List

is there a test that could be added here or elsewhere in the repo that uses this method?


Amended staging disk space as it had run out.

Describe this PR
Amended the RDS staging disk space because it had run out.
What is the problem we're trying to solve
Database is unavailable because of lack of disk space.

Closed because after checking (well Mirela checking) we realised that prod doesn't need such a big DB so something's wrong with staging - so will see if we can work out why the bucket is filling up rather than just giving it a bigger bucket.


training fails

running the next command :+1:
python3 ./models/train_rels.py -m sgcls -model motifnet -order leftright -nl_obj 2 -nl_edge 4 -b 6 -clip 5
-p 100 -hidden_dim 512 -pooling_dim 4096 -lr 1e-3 -ngpu 1 -ckpt checkpoints/vgdet/vg-24.tar
-save_dir checkpoints/motifnet2 -nepoch 50 -use_bias
and getting the next error, did you encounter this error?
Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 199, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 110, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) == 0)) #b == 0))
File "./models/train_rels.py", line 142, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 593, in getitem
return self(*batch[0])
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py", line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 538, in forward
boxes.data, result.boxes_all)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py", line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 333, in forward
box_priors=box_priors,
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 187, in dispatch_line
self.user_line(frame)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 408, in user_line
self.interaction(frame)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 376, in interaction
show_exc_dialog=show_exc_dialog)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 2118, in call_with_ui
return f(*args, **kwargs)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 2362, in interaction
self.event_loop()
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py", line 2322, in event_loop
keys = self.screen.get_input()
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py", line 336, in get_input
keys, raw = self.parse_input(None, None, self.get_available_raw_input())
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py", line 436, in get_available_raw_input
codes = self._get_gpm_codes() + self._get_keyboard_codes()
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py", line 510, in _get_keyboard_codes
code = self._getch_nodelay()
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py", line 644, in _getch_nodelay
return self._getch(0)
File "/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py", line 554, in _getch
return ord(os.read(self._term_input_file.fileno(), 1))
TypeError: ord() expected a character, but string of length 0 found

No clue what’s going on, the error seems to be in the debugger rather than
in the code...
On Tue, Sep 4, 2018 at 5:42 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

running the next command 👍
python3 ./models/train_rels.py -m sgcls -model motifnet -order leftright
-nl_obj 2 -nl_edge 4 -b 6 -clip 5
-p 100 -hidden_dim 512 -pooling_dim 4096 -lr 1e-3 -ngpu 1 -ckpt
checkpoints/vgdet/vg-24.tar
-save_dir checkpoints/motifnet2 -nepoch 50 -use_bias
and getting the next error, did you encounter this error?
Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 199, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 110, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) == 0))
#b == 0))
File "./models/train_rels.py", line 142, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 593, in
getitem
return self(*batch[0])
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",
line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 538, in forward
boxes.data, result.boxes_all)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",
line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 333, in forward
box_priors=box_priors,
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 187, in dispatch_line
self.user_line(frame)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 408, in user_line
self.interaction(frame)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 376, in interaction
show_exc_dialog=show_exc_dialog)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 2118, in call_with_ui
return f(*args, **kwargs)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 2362, in interaction
self.event_loop()
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",
line 2322, in event_loop
keys = self.screen.get_input()
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",
line 336, in get_input
keys, raw = self.parse_input(None, None, self.get_available_raw_input())
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",
line 436, in get_available_raw_input
codes = self._get_gpm_codes() + self._get_keyboard_codes()
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",
line 510, in _get_keyboard_codes
code = self._getch_nodelay()
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",
line 644, in _getch_nodelay
return self._getch(0)
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",
line 554, in _getch
return ord(os.read(self._term_input_file.fileno(), 1))
TypeError: ord() expected a character, but string of length 0 found
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWJxzUwhVC_Rjr8o5XBoxUIsCwotqgcks5uXnUVgaJpZM4WY6AL
.

Please take a look at the next comment i left.
On Tue, 4 Sep 2018, 17:46 Rowan Zellers<EMAIL_ADDRESS>wrote:

No clue what’s going on, the error seems to be in the debugger rather than
in the code...
On Tue, Sep 4, 2018 at 5:42 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

running the next command 👍
python3 ./models/train_rels.py -m sgcls -model motifnet -order leftright
-nl_obj 2 -nl_edge 4 -b 6 -clip 5
-p 100 -hidden_dim 512 -pooling_dim 4096 -lr 1e-3 -ngpu 1 -ckpt
checkpoints/vgdet/vg-24.tar
-save_dir checkpoints/motifnet2 -nepoch 50 -use_bias
and getting the next error, did you encounter this error?
Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 199, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 110, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) == 0))
#b == 0))
File "./models/train_rels.py", line 142, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 593, in
getitem
return self(*batch[0])
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 538, in forward
boxes.data, result.boxes_all)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 333, in forward
box_priors=box_priors,
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 187, in dispatch_line
self.user_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 408, in user_line
self.interaction(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 376, in interaction
show_exc_dialog=show_exc_dialog)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2118, in call_with_ui
return f(*args, **kwargs)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2362, in interaction
self.event_loop()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2322, in event_loop
keys = self.screen.get_input()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 336, in get_input
keys, raw = self.parse_input(None, None, self.get_available_raw_input())
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 436, in get_available_raw_input
codes = self._get_gpm_codes() + self._get_keyboard_codes()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 510, in _get_keyboard_codes
code = self._getch_nodelay()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 644, in _getch_nodelay
return self._getch(0)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 554, in _getch
return ord(os.read(self._term_input_file.fileno(), 1))
TypeError: ord() expected a character, but string of length 0 found
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABWJxzUwhVC_Rjr8o5XBoxUIsCwotqgcks5uXnUVgaJpZM4WY6AL
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418394861,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbMDMmC5-95lBQjZ_R9anQV2Af3zbO51ks5uXpIvgaJpZM4WY6AL
.

It doesn’t look like you posted more than 1 comment...
On Tue, Sep 4, 2018 at 7:48 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

Please take a look at the next comment i left.
On Tue, 4 Sep 2018, 17:46 Rowan Zellers<EMAIL_ADDRESS>wrote:

No clue what’s going on, the error seems to be in the debugger rather
than
in the code...
On Tue, Sep 4, 2018 at 5:42 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

running the next command 👍
python3 ./models/train_rels.py -m sgcls -model motifnet -order
leftright
-nl_obj 2 -nl_edge 4 -b 6 -clip 5
-p 100 -hidden_dim 512 -pooling_dim 4096 -lr 1e-3 -ngpu 1 -ckpt
checkpoints/vgdet/vg-24.tar
-save_dir checkpoints/motifnet2 -nepoch 50 -use_bias
and getting the next error, did you encounter this error?
Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 199, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 110, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) ==
0))
#b == 0))
File "./models/train_rels.py", line 142, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 593, in
getitem
return self(*batch[0])
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 538, in
forward
boxes.data, result.boxes_all)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 333, in
forward
box_priors=box_priors,
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 187, in dispatch_line
self.user_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 408, in user_line
self.interaction(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 376, in interaction
show_exc_dialog=show_exc_dialog)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2118, in call_with_ui
return f(*args, **kwargs)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2362, in interaction
self.event_loop()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2322, in event_loop
keys = self.screen.get_input()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 336, in get_input
keys, raw = self.parse_input(None, None,
self.get_available_raw_input())
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 436, in get_available_raw_input
codes = self._get_gpm_codes() + self._get_keyboard_codes()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 510, in _get_keyboard_codes
code = self._getch_nodelay()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 644, in _getch_nodelay
return self._getch(0)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 554, in _getch
return ord(os.read(self._term_input_file.fileno(), 1))
TypeError: ord() expected a character, but string of length 0 found
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22, or mute the
thread
<

https://github.com/notifications/unsubscribe-auth/ABWJxzUwhVC_Rjr8o5XBoxUIsCwotqgcks5uXnUVgaJpZM4WY6AL

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418394861>,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AbMDMmC5-95lBQjZ_R9anQV2Af3zbO51ks5uXpIvgaJpZM4WY6AL
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418395640,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWJx6scwgp4DcPfOSvw9CVMABGgWafOks5uXpKogaJpZM4WY6AL
.

Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 200, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 111, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) ==
0)) #b == 0))
File "./models/train_rels.py", line 143, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 591, in
getitem
return self(*batch[0])
File
"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",
line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 568, in forward
), 1))
RuntimeError: The size of tensor a (38) must match the size of tensor b
(51) at non-singleton dimension 1
On Tue, Sep 4, 2018 at 6:59 PM Rowan Zellers<EMAIL_ADDRESS>wrote:

It doesn’t look like you posted more than 1 comment...
On Tue, Sep 4, 2018 at 7:48 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

Please take a look at the next comment i left.
On Tue, 4 Sep 2018, 17:46 Rowan Zellers<EMAIL_ADDRESS>wrote:

No clue what’s going on, the error seems to be in the debugger rather
than
in the code...
On Tue, Sep 4, 2018 at 5:42 AM Gal Sadeh Kenigsfield <
<EMAIL_ADDRESS>wrote:

running the next command 👍
python3 ./models/train_rels.py -m sgcls -model motifnet -order
leftright
-nl_obj 2 -nl_edge 4 -b 6 -clip 5
-p 100 -hidden_dim 512 -pooling_dim 4096 -lr 1e-3 -ngpu 1 -ckpt
checkpoints/vgdet/vg-24.tar
-save_dir checkpoints/motifnet2 -nepoch 50 -use_bias
and getting the next error, did you encounter this error?
Training starts now!
Traceback (most recent call last):
File "./models/train_rels.py", line 199, in
rez = train_epoch(epoch)
File "./models/train_rels.py", line 110, in train_epoch
tr.append(train_batch(batch, verbose=b % (conf.print_interval*10) ==
0))
#b == 0))
File "./models/train_rels.py", line 142, in train_batch
result = detector[b]
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 593, in
getitem
return self(*batch[0])
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 538, in
forward
boxes.data, result.boxes_all)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/torch/nn/modules/module.py",

line 325, in call
result = self.forward(*input, **kwargs)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 333, in
forward
box_priors=box_priors,
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/home/sgalk87/neural-motifs/lib/rel_model.py", line 221, in
edge_ctx
obj_embed2 = self.obj_embed2(obj_preds)
File "/usr/lib/python3.5/bdb.py", line 48, in trace_dispatch
return self.dispatch_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 187, in dispatch_line
self.user_line(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 408, in user_line
self.interaction(frame)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 376, in interaction
show_exc_dialog=show_exc_dialog)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2118, in call_with_ui
return f(*args, **kwargs)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2362, in interaction
self.event_loop()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/pudb/debugger.py",

line 2322, in event_loop
keys = self.screen.get_input()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 336, in get_input
keys, raw = self.parse_input(None, None,
self.get_available_raw_input())
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 436, in get_available_raw_input
codes = self._get_gpm_codes() + self._get_keyboard_codes()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 510, in _get_keyboard_codes
code = self._getch_nodelay()
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 644, in _getch_nodelay
return self._getch(0)
File

"/home/sgalk87/neural-motifs/venv/lib/python3.5/site-packages/urwid/raw_display.py",

line 554, in _getch
return ord(os.read(self._term_input_file.fileno(), 1))
TypeError: ord() expected a character, but string of length 0 found
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22, or mute the
thread
<

https://github.com/notifications/unsubscribe-auth/ABWJxzUwhVC_Rjr8o5XBoxUIsCwotqgcks5uXnUVgaJpZM4WY6AL

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418394861
,
or mute the thread
<

https://github.com/notifications/unsubscribe-auth/AbMDMmC5-95lBQjZ_R9anQV2Af3zbO51ks5uXpIvgaJpZM4WY6AL

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418395640>,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABWJx6scwgp4DcPfOSvw9CVMABGgWafOks5uXpKogaJpZM4WY6AL
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rowanz/neural-motifs/issues/22#issuecomment-418423064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbMDMsaaY3SsAdoV6l7HkKJA4ha0AG7Fks5uXqNagaJpZM4WY6AL
.

in my version, lib.rel_model has 560 lines, not 568. so I don't know where you're getting the error.


When transforming WikiImage leaves {caption} placeholder when content is empty

Category
[X] Bug
[ ] Enhancement
Problem Area
[ ] Page Transformation: Error during the setup/use of the Page Transformation UI solution (did you check our troubleshooting guide?)
[X] Page Transformation: Error during the use of page transformation from PnP PowerShell
[ ] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[ ] Modernization Scanner: something went wrong...
Expected or Desired Behavior
Empty web parts should be empty or removed (see ref #156), in transformations I have seen placeholders in the image web part
Observed Behavior
When transforming publishing pages, when the source is empty, an empty web part contains {caption} text when using the SharePointPnP.Modernization.WikiImagePart mapping, currently working around with type: Microsoft.SharePoint.WebPartPages.ImageWebPart, Microsoft.SharePoint, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=71e9bce111e9429c

Let's solve this via the implementation of #156

Workaround - Specify the web part mapping file for WikiImagePart, Under Properties, add:
<Property Name="Caption" Type="string" />

then in page layout mapping file, under WebParts > Field PublishingPageImage, specify:
<Property Name="Caption" Type="string" Functions="EmptyString()" />

This moves the control to the page layout mapping file which for the moment I have specified as empty. This clears the issue when specifying the -WebPartMappingFile with the adjustments.
Secondary Issue
One thought tho, I noticed in my user profile for the default xml files for page/web part mappings, are we handling updates to the file when we release a new version with modifications? Or how/where are the defaults specified?

Nice workaround but I'll plan to actually do a proper fix for this as I don't want to encourage creating a custom web part mapping file for this reason. Each release has the latest mapping files in the nuget package and in PnP PS, but it's up the user to merge changes for the web part mapping files.

Hey @pkbullock ,
Did check this and I don't think this is an issue. In the default page layout mapping file we use the following construct:
<Property Name="Caption" Type="string" Functions="ToImageCaption({PublishingImageCaption})" />

without the Caption field being set in the webpartmapping file. The caption field value is always set to "" for non-publishing page transformations. For publishing pages you would either use the default approach as shown above which will return in an empty value if the field provided into ToImageCaption is null or empty or alternatively use your approach with the EmptyString() function in case you do not want to take over image captions altogether.
Can you verify that things work for you with the default web part mapping file?

Closing this one as we now anyhow skip empty web parts + the above mentioned approach works when there's actual content but no caption value. If this still is an issue then let's create a new issue for it.


Congestion hashmap implementation

Working towards https://github.com/mapbox/directions/issues/501

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d4aa6ea). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   43.32%
=========================================
  Files             ?       11
  Lines             ?      457
  Branches          ?        5
=========================================
  Hits              ?      198
  Misses            ?      259
  Partials          ?        0

Impacted Files
Coverage Δ

src/types.hpp
0% <ø> (ø)

src/hashmap.hpp
0% <0%> (ø)

src/nodejs_bindings.cpp
46.37% <0%> (ø)

src/hashmap.cpp
0% <0%> (ø)

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 d4aa6ea...85336ab. Read the comment docs.

Btw are we using Mason for deps? I can see the hashmap lib vendoered in third_party.
It is already bundled with mason => https://github.com/mapbox/mason/tree/master/scripts/sparsehash/2.0.2 could make sense to just use this one.

@daniel-j-h we are using Mason, but we are actually using https://github.com/greg7mdp/sparsepp, not sparsehash. And sparsepp not vendored in Mason yet. We had a conversation with @springmeyer and decided not to bring it into Mason during the logistics sprint because of time constraints.

@ghoshkaj please drop a time on my calendar for getting sparsepp packaged - we can do a screenshare and it will likely take 15 min. Make sure to run through https://github.com/mapbox/mason/#trigger first.


possible API change for compcor

this thread needs to be monitored: https://github.com/nipreps/fmriprep/issues/2384

looks like the derivatives change is going to move forward, but has not yet been implemented. Would be useful to test prior to fmriprep release, so the support is rolled out as close as possible to the fmriprep release date, at least on the dev version of the package.


Certificate installation and renewal

As a long time C/C++ programmer I know just enough about Python to be extremely dangerous.  In-spite of that handicap, I have gotten the sample_client.py working to list all of my site devices, but I don't see any clues as to how to install or renew a TLS Certificate.
Has that part of the API been coded yet?

No, sorry, updating the certificate via the API wrapper isn't a feature we have yet.
It's probably not extremely hard to add - however, at the moment I can't renew certs even through the web interface because it doesn't seem to support the certificates I get out of Let's Encrypt any more. I haven't investigated why yet.
I also only know just enough about Python to be extremely dangerous... If you want to have a go at adding the feature, please do! The Omada API documentation is incomplete, so you might have to observe what the web interface does and duplicate that.
I can see value in adding this support, so I will probably get round to doing it eventually.

I am using a private SmallStep CA on a RPi4 to generate my certs.   I have noticed that even though the web interface claims to allow the PEM format for both the cert and the key, I must convert my PEM certs to PFX format using openSSL, which then uploads just fine to my OC200.

Thanks. I tried pfx and it worked for me.
The API for PFX upload seems pretty trivial. I will try to add support for it...

This feature has been added in 1.3.13. Please give it a try.
Note, I only had the patience to add .pfx support, given that I couldn't get .der format to work anyway. It wouldn't be hard to add other certificate formats if they could be shown to work. I'll leave that as an exercise for the reader...

sample_client.py is a very basic early test harness.
Using the omada cli that is included in the package should get you somewhere a lot faster.

Hey Mark,
OK, I have finally got a complete automatic cert renewal working for my OC200!  Whooo-Hooo!
Cron executes a script at midnight and 6 am; thus make sure that the cert never expires before renewal.
This script renews the certificate from my Smallstep private CA, converts it to .pfx, calls the cli to upload the renewed certificate, and finally calls the cli to reboot the machine.
Right now it has my machine names hard-coded in the script, but I am working on parameterizing the script and adding some documentation.  I hope to post it here for possible inclusion in the repo.
Cheers,
Rich


chore(master): release 1.14.32

:robot: I have created a release beep boop
1.14.32 (2023-11-24)
Bug Fixes

deps: update dependency pnpm to v8.11.0 (1d9ae59)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-node/releases/tag/v1.14.32 :sunflower:


Docs: Add more section labels for referencing

Allows for sections to be linked from other projects using Intersphinx.

No objection from me: @sigmavirus24?

:+1:

Thanks @awiddersheim! :sparkles: :cake: :sparkles:


Calculate elapsed time values using FrameStatistics source instead of clock

Until now, we were reading ElapsedFrameTime from threads' clock, but only 10 times per second. This means it could be inaccurate if frame times were different between these polls.

Now considers every frame
Uses FrameStatistics.CollectedTimes instead of ElapsedFrameTime. This is a prerequisite for making the elapsed time display actually work with single threaded execution. In practice, this means a small framework overhead may be excluded from the calculation, but if anything this makes the display more correct in its context.

After showing and hiding the display:

After showing and hiding the display:


PTO should trigger an ack-eliciting packet, but we don't if there's no data to be sent

cc @janaiyengar

While discussing if we should (or if there are alternative approaches), we have noticed that the way we mark things to be retransmitted in mark_packets_as_lost is inadequate:

The function should not touch max_lost_pn, as the role of the value is to express the largest PN that have been "confirmed" lost. We should stop touching the value and might also rename the value to reflect the concept of loss confirmation.
The function marks the first one or two packets in the sentmap that are inflight and ack-eliciting as lost. This leads to undesirable behavior when more than one PTO fire consecutively; instead of sending the "oldest" unacked data1, we end up sending the oldest data marked inflight.
Use of quicly_sentmap_update(EVENT_LOST) is incorrect - as the packet that is newly marked as lost might only have contained frames that no more needs to be restrasmitted (e.g., a STREAM frame of a stream that has been cancelled). In such case, we would end up sending nothing even though there is inflight data.

Considering these aspects, we agreed that @kazuho should work on adding a new event type to sentmap named "retransmit", that stages something for transmission without changing the state of the sentmap (or it's entries). The mark_packets_as_lost function (which should be renamed to "quicly_sentmap_prepare_retransmit" or something) would iterate from the oldest entry in the sentmap until the frame-level callbacks of the sentmap indicate that sufficient amount of data has been staged for retransmission.
1: Ideally, we should be asking the stream scheduler to provide the most urgent data that needs to be sent among those that are considered inflight - however that's difficult. Therefore we are approximating it using sent time. Note also that this behavior is not beneficial for SPTOs; we need to consider that separately.

As of #257, we send PING when PTO timer fires and if there's nothing to send.


Add a FIPS enabled RHEL8 buildbot

Addition of RHEL 8 buildbot with FIPS mode enabled.
This config is experimental, and Python will possibly not compile at all initially with it. Adding it to the unstable list, and feel free to turn off the notifications from it.

cc @tiran

New config deployed. Let's see how Python likes FIPS.


Issue using Nominatim

Nominatim generated a permissions error. Changing the user_agent from 'myGeolocator' to another (unique) name fixes this problem. You might want to add a README file to tell the user this, or put it in a comment in the notebook.

I tested the code but didn't get any permission errors. Would you happen to have a screenshot of the error message so I can look? Thanks!

I tested the code but didn't get any permission errors. Would you happen to have a screenshot of the error message so I can look? Thanks!

I think it is because Nomatim needs a unique user_agent name (not sure how they check) and you have the myGeolocator name. Just changing the name to something random made the error go away for me. I don't have a screen shot, but the issue is described here: https://stackoverflow.com/questions/51503389/geopy-exc-geocoderinsufficientprivileges-http-error-403-forbidden

Thanks for the detail! Just update the fix to the main notebook by adding one more comment to replace the myGeolocator to a unique name.


Specify h5py string type to match Homer

Codecov Report

Merging #364 (8930019) into main (a0e7cd2) will decrease coverage by 2.62%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##             main     #364      +/-   ##
==========================================
- Coverage   95.92%   93.29%   -2.63%
==========================================
  Files          49       49
  Lines        1765     1789      +24
  Branches      229      234       +5
==========================================
- Hits         1693     1669      -24
- Misses         30       74      +44
- Partials       42       46       +4

Impacted Files
Coverage Δ

mne_nirs/io/snirf/_snirf.py
92.46% <76.19%> (-3.13%)
:arrow_down:

mne_nirs/io/snirf/tests/test_snirf.py
95.50% <85.71%> (-1.83%)
:arrow_down:

...e_nirs/visualisation/_plot_nirs_source_detector.py
20.83% <0.00%> (-75.00%)
:arrow_down:

mne_nirs/visualisation/tests/test_visualisation.py
77.45% <0.00%> (-22.55%)
:arrow_down:

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 a0e7cd2...8930019. Read the comment docs.

Just got word from the Homer camp that this was an issue with their reader, not with our writer. So closing this.


[5.7] Added the ability to send an array to get json method

Added the ability to send an array to getJson method.
Like:
$this->getJson('path/to', ['param' => 'test'])
It's a continuation of: #23955
I've made the appropiate changes to pass the tests.

I still think this is not worth it, we are introducing a breaking change and the gain from this change is not much.


Handle membrane-backend session data with PostgreSQL

Currently, the session data in membrane-backend is managed using a "filesystem" storage mechanism. This solution is not scalable, posing risks for data loss and limiting the ability to share session states across multiple instances.
Objective
Migrate the session storage to use a PostgreSQL database to improve scalability, persistence, and data integrity.
Tasks

[ ] Evaluate current session handling code in membrane-backend.
[ ] Design a schema for storing session data in PostgreSQL.
[ ] Implement the schema in PostgreSQL.
[ ] Update session handling in membrane-backend to use the PostgreSQL database.
[ ] Test the new implementation to ensure data consistency and reliability.
[ ] Update documentation to reflect the new changes.

Acceptance Criteria

Session data should be stored in a PostgreSQL database.
Session handling should work seamlessly with the new database setup.
Documentation should be updated to explain the new session handling mechanism.

good work detailing this plan!


show amount of owned stocks, improve responsive, don't reset input

closes #60
closes #68

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7cc6061). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #80   +/-   ##
=========================================
  Coverage          ?   44.12%
=========================================
  Files             ?       58
  Lines             ?      791
  Branches          ?       87
=========================================
  Hits              ?      349
  Misses            ?      431
  Partials          ?       11

Impacted Files
Coverage Δ

...rc/components/Charts/FinancialDevelopmentChart.tsx
62.5% <ø> (ø)

src/views/Market/StockCard/BuyOrSellView.tsx
41.17% <ø> (ø)

src/views/Market/StockCard/StockDetails.tsx
40% <ø> (ø)

src/views/Market/StockCard/StockCard.tsx
83.33% <ø> (ø)

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 7cc6061...85921a0. Read the comment docs.


Uncaught Error: Can't save buffer with no file path

I have this error when I try to see the preview of the html:
[Enter steps to reproduce below:]

...
...

Atom Version: 0.190.0
System: Mac OS X 10.9.5
Thrown From: atom-html-preview package, v0.1.6
Stack Trace
Uncaught Error: Can't save buffer with no file path
At /Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:128

Error: Can't save buffer with no file path
  at TextBuffer.module.exports.TextBuffer.saveAs (/Applications/Atom.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:904:15)
  at TextBuffer.module.exports.TextBuffer.save (/Applications/Atom.app/Contents/Resources/app/node_modules/text-buffer/lib/text-buffer.js:899:19)
  at TextEditor.module.exports.TextEditor.save (/Applications/Atom.app/Contents/Resources/app/src/text-editor.js:556:26)
  at AtomHtmlPreviewView.module.exports.AtomHtmlPreviewView.renderHTMLCode (/Users/jeancarloperez/.atom/packages/atom-html-preview/lib/atom-html-preview-view.coffee:95:76)
  at AtomHtmlPreviewView.module.exports.AtomHtmlPreviewView.renderHTML (/Users/jeancarloperez/.atom/packages/atom-html-preview/lib/atom-html-preview-view.coffee:92:8)
  at /Users/jeancarloperez/.atom/packages/atom-html-preview/lib/atom-html-preview.coffee:54:25
  at _fulfilled (/Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:794:54)
  at self.promiseDispatch.done (/Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:823:30)
  at Promise.promise.promiseDispatch (/Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:756:13)
  at /Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:516:49
  at flush (/Applications/Atom.app/Contents/Resources/app/node_modules/q/q.js:110:17)
  at process._tickCallback (node.js:357:13)

Commands
     -3:31.5 project-manager:toggle (atom-text-editor.editor)
     -3:30.6 core:confirm (atom-text-editor.editor.mini)
  2x -3:22.2 core:paste (atom-workspace.workspace.scrollbars-visible-always.theme-atom-dark-syntax.theme-atom-dark-ui.find-visible)
     -0:37.0 grammar-selector:show (atom-text-editor.editor)
     -0:36.0 core:confirm (atom-text-editor.editor.mini)
  5x -0:28.1 core:copy (atom-text-editor.editor)
     -0:27.2 application:new-file (atom-text-editor.editor)
     -0:26.8 core:paste (atom-text-editor.editor)
     -0:26.5 emmet:insert-formatted-line-break-only (atom-text-editor.editor)
     -0:26.5 editor:newline (atom-text-editor.editor)
     -0:25.4 core:undo (atom-text-editor.editor)
     -0:24.2 grammar-selector:show (atom-text-editor.editor)
     -0:22.9 core:confirm (atom-text-editor.editor.mini)
  2x -0:20.8 find-and-replace:show (atom-text-editor.editor)
  2x -0:11.5 core:move-left (atom-text-editor.editor.mini)
     -0:01.1 atom-html-preview:toggle (atom-text-editor.editor)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "atom-dark-syntax"
    ]
  }
}

Installed Packages
# User
atom-html-preview, v0.1.6
autocomplete-paths, v1.0.2
autocomplete-plus, v2.7.1
emmet, v2.3.6
file-icons, v1.5.3
minimap, v4.7.5
project-manager, v1.15.5

# Dev
No dev packages

Same error.
System: Linux Fedora 21 x84_64
Atom version: 0.194.0
Thrown From: atom-html-preview package, v0.1.6

is there a work around for this?

I can confirm that what @liridoni "you have to save the document first. Then run the preview" would work.

yes that works for me!

It would be nice to have a more clear error message than just a link to this bug.

The solution of @liridoni  help me. Thanks.

I'm saying that I think this bug should be re-opened because it's annoying to have to come here to find out how to fix it. Even if there's a solution for you, it's still a bug.

Yap, I agree to @calebegg, this error still exists!


Add tests and missing parsing features and bump node version

This PR adds new parsing features and associated tests:

Scope access symbol: ::.
Parenthesis wrapped typedef and typeset.
functag and funcenum support.
__nullable__ support for methodmap.
More variable tests, including cases which are not handled.
Support for macros.
Improved define statement parsing.

This PR is still a WiP as the variable tests I have implemented do not pass yet. I am having a hard time making the new operator optional on variable declarations.
a;

is a valid variable declaration in Sourcepawn, however making old type optional in the grammar creates a conflict when building.

Thanks for the contribution :heart:
I didn't had the time too look deeply into it, but will see if I can provide some feedback to you :smiley:
Keep up the great work :+1:

Reopened the PR here #2 .


renames

Current coverage is 99.18% (diff: 100%)

Merging #70 into master will not change coverage

@@             master        #70   diff @@
==========================================
  Files            27         27
  Lines           368        368
  Methods           0          0
  Messages          0          0
  Branches         83         83
==========================================
  Hits            365        365
  Misses            3          3
  Partials          0          0

Powered by Codecov. Last update bf7ed9b...4d4fb1e


Differentiate model tests and model evaluation for experiments

In our current architectural diagram, we only have an "Evaluator" component. However, we actually have two different tasks:
a) For production usage, we want to do quality-checks on models, test performance on pre-defined test cases etc.
b) For experiments, we probably want to collect some form of test score.
These are two separate tasks and should probably be handled by two separate components.

We disregard an integration check component (for now). It's important to keep in mind and a motivation for dynamic datasets instead of continous finetuning, but from a research perspective not the focus of Modyn right now.

(And the evaluator is tracked by #15)


Merge master branch to prTesting1_202004201924 branch

Docs Build status updates of commit e86d6d7:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: Docs Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit e86d6d7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestRef1.md
:warning:Warning

Details

split/updateTestRef1.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.
Line 1: [Suggestion] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 04/01/2020.

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


Remove space between plots on the distribution grid

The current distribution grid has large gaps between the plots (see first image), can we remove the gaps so that lit looks more like the second image, search google scatter matrix to see more examples.

done


Honor CodeSystem.caseSensitive during validation

Is your feature request related to a problem? Please describe.
One of the PDEX PlanNet examples includes a code that has casing which does not match the Display value in the Codesystem.
Currently, our validator considers this an error.
The $validate-code operation (both ValueSet and CodeSystem variants) says this on the subject:

Whether displays are case sensitive is code system dependent

When I look at the CodeSystem resource, I see CodeSystem.caseSensitive but the description makes it sound like its just for the codes and not for the display values:  https://www.hl7.org/fhir/codesystem-definitions.html#CodeSystem.caseSensitive
Assuming this should apply to the display value too, then I think we should follow their recommendation on this:

If this value is missing, then it is not specified whether a code system is case sensitive or not. When the rule is not known, Postel's law should be followed: produce codes with the correct case, and accept codes in any case.

Describe the solution you'd like

When CodeSystem.caseSensitive is populated, we should honor it when we validate both the codes and their display values
When CodeSystem.caseSensitive is NOT populated, we should follow the guidance in the specification and perform a case-insensitive match

Optionally, in case number 2, we could chose to emit a warning (or not).
Describe alternatives you've considered
We could ignore this guidance and chose to be more rigorous, but being more strict that the other validators makes me nervous.
Additional context
One complicating factor is that this caseSensitive field is on the CodeSystem (and not the ValueSet).  This could be very bad for us since we lean more heavily on ValueSet checks.  We may want to cache this value somewhere (maybe as part of the existing code system cache we have) so that we don't need to read the CodeSystem resource to do validation.

I don't think this will be too much of an issue for us in terms of determining case sensitivity and the implications for performance. I think this will likely be isolated to DefaultTermServiceProvider which is where the value set cache is built up. I think we would have a change in two spots: 1. normalize the case when building the value set cache (if the code system is case insensitive) and 2. normalize the case of the code we are checking (if the code system is case insensitive). We also want to handle this when doing a Lookup operation.


Add TomTom comparison

Adding TomTom to the comparison tool.
Now the output will look something like this:
origin,destination,departure_time,google_travel_time,tomtom_travel_time,tt_travel_time,error_percentage_google,error_percentage_tomtom
"52.200400622501455, 0.1082577055247136","52.21614536733819, 0.15782831362961777",2024-09-20 07:00:00+0100,606.0,775.0,956.0,57,23

Here are the different types of travel times TomTom can calculate. By default, it includes traffic delays, as we do, so I don't think we need to modify it in any way.

Got these results:
2024-09-17 16:15:14 | INFO | Mean relative error compared to GOOGLE API: 16.87%
2024-09-17 16:15:14 | INFO | 90% of TravelTime results differ from GOOGLE API by less than 31%
2024-09-17 16:15:14 | INFO | Mean relative error compared to TOMTOM API: 14.99%
2024-09-17 16:15:14 | INFO | 90% of TravelTime results differ from TOMTOM API by less than 26%

after running with this setup:
python src/traveltime_google_comparison/main.py --input inputs/United_Kingdom.csv --output output.csv --date 2024-09-20 \
    --start-time 07:00 --end-time 20:00 --interval 180 --time-zone-id "Europe/London"

Nice!
Last remark - the top section of Readme is now outdated. We should add the information that we can compare with TomTom as well. Maybe let's leave the repository name as is (with just google in the name), but change description in Readme?

Yea sure, I'll update the README for now.
Later we probably should change the repo name (and pip package name), but we can do that at any point


FancyZones: move to last known zone doesn't work on secondary monitors

Environment
PowerToys version: 0.14.1
PowerToy module for which you are reporting the bug: FancyZones

Steps to reproduce
Make sure Move newly created windows to their last known zone option is ON.
Select a layout/zoneset on a secondary monitor.
Move the command prompt to a zone in the secondary monitor.
Close the command prompt window.
Relaunch command prompt.
Expected behavior
The command prompt window to open on the secondary monitor in the last known zone.
Actual behavior
The command prompt window opens on the primary monitor.

The code flow:
IFACEMETHODIMP_(void) FancyZones::WindowCreated(HWND window) noexcept
=>
GetAppLastZone
```.

Currently, we only check the primary monitor's history:
```cpp
        if (auto monitor = MonitorFromWindow(window, MONITOR_DEFAULTTONULL))

But we should get all active monitors and check all of their histories as well.

Would also be nice to check if the monitor is on or off.
When connected to HDMI, usually a monitor off is not listed, but when connected with DisplayPort it may be listed.

moving to a v1.0 milestone bucket item.

The fix is now available in 0.16 https://github.com/microsoft/PowerToys/releases

😍


Get babel.dev.js working with Mocha

Hi, I'm used to having a .babelrc and start mocha with: mocha --compilers js:babel-core/register, but I would like to reuse the .babel.dev.js in config/. Any hint in the right direction would be appreciated, thanks!

You can probably do something like
./mochaHook.js
var babelConfig = require('react-scripts/config/babel.dev');
require("babel-register")(babelConfig)

and then try mocha --compilers js:./mochaHook.
I’m not sure it would work but worth giving a try.
Note: this is not officially supported now. We may move or change react-scripts/config/babel.dev without warning in any version.

Perfect, it works. Thanks

This has now been extracted to a babel preset: "babel-preset-react-app", in case anybody else finds their way here.
https://www.npmjs.com/package/babel-preset-react-app
https://github.com/facebookincubator/create-react-app/tree/master/packages/babel-preset-react-app

Also, to be clear, we've since added a built-in testing solution so you shouldn't need to run Mocha—unless you really want it over Jest for one or another reason.


Ease the training grid search

[x] Edit README.md to show that --input_shape command line arg tuple needs to be put into quotation marks
[x] Add command line args in the run_training.py: max_iterations etc.
[x] Change how the exported model weights files are named: config.filename_brief() fn
[x] Create parallel subdirectories ( /logs /docs /models ) to records training session with time stamp
[x] Add .gitignore mentions of the training subdirectories: /logs /docs /models

@quantumjot This PR facilitates the encoder training. However, it may not be fully compatible with the later stages (temporal model etc.). Perhaps we can consider keeping this at a separate branch?

Hi Kristina - think this looks like a good start, but we may want to fix a couple of other issues first (most importantly #28)

Also - it's probably easier to have the config written directly to the model_dir directory.  That way we can (later on) make a method to load the whole thing.  When doing a grid search, you just specify a new model_dir to represent the new model?

These commits remove the hard-coded values (resolves #28) & organise the model_dir to store both the trained encoder / decoder weights.h5 files as well as the 'ConfigHyperParams.json' which can be use to load the weights from the config afterwards.

Hi Kristina - a couple of thoughts while you're working on this:

I'd rather keep the projector and encoder training separate.
You should look at the dataclasses.asdict() functionality for loading/saving the config as a JSON file

It would be good to add some of the functionality of this PR. Is it worth splitting it up a little?


Part 2 of the Alexa skill builder prototype

With this, we should be closer to a testable state

Pull Request Test Coverage Report for Build 1292

97 of 139 (69.78%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 65.707%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

routes/developer_alexa.js
88
130
67.69%

Files with Coverage Reduction
New Missed Lines
%

routes/my_api.js
1
63.57%

Totals

Change from base Build 1278:
-0.1%

Covered Lines:
5757

Relevant Lines:
8253

💛  - Coveralls


Pagination error because a  before pagination's cols

Hi, I'm inserting a databable as the basic example and I'm getting this:

This div is the cause:
https://github.com/AllenFang/react-bootstrap-table/blob/master/src/pagination/PaginationList.js#L119
Maybe it's because I did something in my custom css?

Maybe it's because I did something in my custom css?

I think so, if you have no idea for fixing it, you can custom the pagination panel by yourself,
check https://github.com/AllenFang/react-bootstrap-table/blob/master/examples/js/custom/pagination/custom-pagination-panel.js for learning how to custom the pagination panel.
Please let me know if you have any concern, thanks


panel_uperf is missing | latest Uperf

GoodEvening @yc9559
I have an issue regarding the installation of uperf..
I installed successfully the uperf (the latest one) from magisk, but i can't find the panel_uperf in the system.. in that case i can't change performance mode automatically in startup..
Device:
Poco M3 (citrus_global)
SnapDragon 662

running on MIUI | Xiaomi.Eu custom rom

Hope you would fix it soon, Love you 💕

Default power mode is located at /sdcard/yc/uperf/cur_powermode in uperf v2. Have a nice a day!


Plugin does not install on ES 2.0

There are 2 issues to address for ES 2.0 with the existing plugin:

The installation step using --install no longer works in 2.0:

Change:
./bin/plugin --install elasticsearch/elasticsearch-support-diagnostics

To:
./bin/plugin install elasticsearch/elasticsearch-support-diagnostics

Plugin does not install via the plugin command above because it is missing the now-required plugin-descriptor.properties

This should be fixed in the 2.x release of this utility. If not please reopen the issue.


Ember mouseEnter deprecation

https://deprecations.emberjs.com/v3.x/#toc_component-mouseenter-leave-move

Great thanks, will be in the next release

I'm getting loads of TypeError: this.get is not a function when I mouse over a droparea now, without dragging anything. Which versions/deprecations is this for?

@nruth oops that looks like a bug. Sorry I didn't realize this PR had already been merged and released. I will put up a Pr with a fix

Thanks for looking into this.
It's happening in our Ember 3.8 app (bit behind with upgrades). I don't know if it happens with newer versions.

On 10 Sep 2020, at 16:31, Josiah Campbell<EMAIL_ADDRESS>wrote:
I'm getting loads of TypeError: this.get is not a function when I mouse over a droparea now, without dragging anything. Which versions/deprecations is this for?
@nruth https://github.com/nruth oops that looks like a bug. Sorry I didn't realize this PR had already been merged and released. I will put up a Pr with a fix
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mharris717/ember-drag-drop/pull/171#issuecomment-690368764, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAGMLTOREA2V6JPD72ZN7TSFDWORANCNFSM4M3S5XOQ.


Release 0.2.0

https://github.com/coiled/coiled-runtime/pull/488 bumped package versions in preparation for a new runtime release. Given https://github.com/coiled/coiled-runtime/issues/520 we should try to release ASAP. @ncclementi are you aware of any outstanding issues?

Yes, we have a couple of problems:

We are seeing this constantly https://github.com/coiled/coiled-runtime/issues/468
There are valid regressions and we do not know where do they come from see https://github.com/coiled/coiled-runtime/issues/500 more specifically https://github.com/coiled/coiled-runtime/issues/500#issuecomment-1314505305

Based on the comment above plan of action:

Do a patch release (0.1.1) of runtime pinning bokeh to avoid dashboard problems.
Keep studying problems to bump all the packages for release 0.2.0 see https://github.com/coiled/coiled-runtime/issues/500 and https://github.com/coiled/coiled-runtime/issues/468

That sounds like a good plan to me. I'd suggest pinning bokeh=2.4.3

this means that, e.g., test shuffle got ~30% slower (with high degree of confidence)?

@ncclementi
As we discussed -- running another A/B test that looks at bumping everything excepting the fsspec packages and then making a decision seems OK.
Let's also confirm you can successfully complete a conda build with both of the build candidate, and the one you intend to test.
Let's aim for a decision by midday tomorrow.
If pyarrow==10.0.xmakes its way onto condo-forge in early December (as expected) we can do a patch release.

Shuffle specifically we can expect to be slower (30% sounds about right to me) with queuing on by default.
If we're worried about versions of (transitive) dependencies having affected performance, @ncclementi maybe you should run the same A/B test, but turn off queuing for 0.2.0. If that looks the same/better, then we can attribute the wall-clock difference to queuing.

Hmm, so it looks like some changes are definitely not due to queuing. The H2O q9 change seems like the most important to investigate, but I wonder why all of H2O seems a little slower consistently.

I was looking into the specific case of q9 and I think the regression we see between 0.1.1 and the new 0.2.0 was introduced by dask/distributed at some point
Notice that on latest q9-5GB was almost always averaging between 7-10 s (always using 2022.6.0), except the big bump when the version of dask was upgraded.

Now if look at upstream the earliest values we have are for 2022.7.01 and it's already higher by ~50% and this continues pretty consistently. See figures below
2022..7.1

2022.9.2

We were back to the good times on 2022.10.0 but then back again to the regression.

In conclusion, this regression is being with us since 2022.7.1, it is probably worth investigating but it shouldn't block the release given all the other benefits that come with newer versions of dask.

As we discussed -- running another A/B test that looks at bumping everything excepting the fsspec packages and then making a decision seems OK.

The results of this experiment did not have anything surprising. The behavior is the exact same as including the updated version of all the packages.

Let's also confirm you can successfully complete a conda build with both of the build candidate, and the one you intend to test.

I was able to do this locally with no problems.

Let's aim for a decision by midday tomorrow.
For regressions discussion see https://github.com/coiled/coiled-runtime/issues/526#issuecomment-1334094911
Got a PR open to kick this in https://github.com/coiled/coiled-runtime/pull/570

cc: @hayesgb

This looks good to me.  I would move forward with the newer release, and create an issue to investigate the Q9 issue separately.


UI should jump to the project page when completed

What problem does this feature solve?
when we completely create a project, we shouldn't be at the create page, This leads to the misconception that our project is not yet created.

What does the proposed API look like?
UI page should jump to the project page when completed.

Not repro now. I'll look into it. close temporarily.


Add clarifiication on uncle headers

Reverts a previous accidental commit to master and then adds clarifying comment defining uncle headers using the devp2p definition for list_of_uncle_headers found here

What should happen with this PR?


[Sozialpass] Add multiple cards (Design)

Please provide a design for multi cards view

Swipe area to navigate between cards and barcodes
Delete Button/Icon to delete the particular card

relates to #1084

Design-proposal can be found here

version 2 of the design proposal can be found here: 🟣 @f1sh1918 @sarahsporck

@hauf-toni
Here is the verification result modal. I think we could just adjust the headline and make a second action

@hauf-toni Can you make the design publicly available?

@hauf-toni i'm all good with the design alternative that you proposed :)
Not sure if we have primary and secondary button like that but the delete button should be highlighted anyways

@michael-markl your requests have to be accepted first, so that the design files can be viewed. unfortunately, the project cannot be made publicly available to everyone. of course, every staff member who wants to will get access to the designs.

unfortunately, the project cannot be made publicly available to everyone

Is this a technical limitation of Figma for our license model or so?
It seems, for files that I create on Figma, I could set the visibility to "Everyone with the link can view".

It seems, for files that I create on Figma, I could set the visibility to "Everyone with the link can view".

good point; i'll make sure that the link is always set like this.
this way everyone who belongs to the team "digitalfabrik" can see the file. for this membership you have to be approved.

@hauf-toni there are some small adjustments missing. If we need to clarify sth let me know!


How to get started? leveldb missing.

I am attempting to run this on windows.
I get:

dotnet neo-cli.dll

Unhandled Exception: System.DllNotFoundException: Unable to load DLL 'libleveldb' or one of its dependencies: The specified module could not be found. (Exception from HRESULT: 0x8007007E)
at Neo.IO.Data.LevelDB.Native.leveldb_options_create()
at Neo.Persistence.LevelDB.LevelDBStore..ctor(String path)
at Neo.Shell.MainService.OnStart(String[] args)
at Neo.Services.ConsoleServiceBase.Run(String[] args)
at Neo.Program.Main(String[] args)
PS C:\Users\DarrellTunnell\Downloads\neo-cli-portable\neo-cli>
In the readme it says I need to install leveldb from here: https://github.com/erikzhang/leveldb - but there are no instructions in that leveldb repo about how to install that dependency on windows.

@dazinator Just build the leveldb project and put leveldb.dll into the path of neo-cli and then retry
dotnet neo-cli.dll.

Replace
NeoSystem = new NeoSystem(protocol, Settings.Default.Storage.Engine, string.Format(Settings.Default.Storage.Path, protocol.Network.ToString("X8")));
with
NeoSystem = new NeoSystem(protocol, null, null);
In MainService.css for CLI


Support for tsp decimal/decimal128 types

TypeSpec defines decimal/decimal128 as part of its built-in numeric types. Since Go doesn't have anything equivalent in its standard library, we'll need to decide how we want to handle them.

We might just emit them as string types, similar to how we treat durations, and leave the parsing up to consumers.


controllers: change resourceManager interface

With the current resourceManager interface there is no way to
communicate the reconcile.Result back to the reconcilePhases as it only
returns the error.
change the interface signature to return the reconcile.Result as well
with the error.
Why?
The change is required for ocs to ocs where we want to communicate the
reconcile.Result to the reconcilePhases.
While making GPRC calls we will get different error codes and based on
those error codes we will decide whether we need to requeue or not if
requeue when to requeue and this can be done with reconcile.Result. With
the existing design, we can return only errors and reconcile will get
requeue. That's why we are changing the interface to return the
reconcile.Result also.
Usecase:
let's say a user will create a storageCluster CR with the wrong
connection string and while making an onboarding request we will get an
error that says invalid connection string. This means there is no point
in re queueing at all as this can not be solved without user interaction.
Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

/hold

/hold cancel

/lgtm


Improve docstrings formatting

Now that online documentation was setup, some effort to make sure all docstrings use a correct numpy style formatting is necessary.
For instance, most methods under "API reference" of the documentation, are currently not very well rendered.
Any help on this would be greatly appreciated. Instructions on how to build the documentation locally can be found in https://github.com/bsmurphy/PyKrige/pull/48

I'll be honest, as I was originally making the docs I didn't really have any idea about what kind of formatting was appropriate... so this definitely needs some work. Also might be a thing where gradual changes as they come up would be best...

@bsmurphy +1 for gradual changes.  Also, it's not actually that much changes, just small formatting fixes: the docstrings are already really well documented, thanks, to your efforts )

Closed with PR #84.


How can I change the approving user?

I would like to know how in which part of the application the approving user can be changed, or if there is any functionality that an approving user can have for each one who uses the application. Waiting for your early reply. Thank you so much.
Regards

The approving user is based on the "Manager" field that is populated in the users Azure Active Directory profile.
You will need to change the name in the field to change the "approver".

Build 1.12 (variation created by the community) allows the definition of approvers per space. That is, you can create spaces and for each one define a different person that will be responsible for approval. I also have another variation that allows a two stage approval process with a custom approver first then a building approver if you want.
Updated Package (v1.12)
https://1drv.ms/u/s!AnR50DmsM8Jwl9kWzDgeuBgB40XJxQ?e=8LZw4R

Build 1.12 (variation created by the community) allows the definition of approvers per space. That is, you can create spaces and for each one define a different person that will be responsible for approval. I also have another variation that allows a two stage approval process with a custom approver first then a building approver if you want.
Updated Package (v1.12)
https://1drv.ms/u/s!AnR50DmsM8Jwl9kWzDgeuBgB40XJxQ?e=8LZw4R

How would you set a space to have approval but the rest of the spaces in the Building to be auto-approved?

Auto-approval is managed by the whole building occupation. The approval will only be triggered big the occupation threshold is reached.

From: kf-paulbrown<EMAIL_ADDRESS>Sent: Wednesday, October 7, 2020 8:42:34 AM
To: microsoft/powerapps-tools<EMAIL_ADDRESS>Cc: Cristiano Goncalves<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [microsoft/powerapps-tools] How can I change the approving user? (#447)
Build 1.12 (variation created by the community) allows the definition of approvers per space. That is, you can create spaces and for each one define a different person that will be responsible for approval. I also have another variation that allows a two stage approval process with a custom approver first then a building approver if you want.
Updated Package (v1.12)
https://1drv.ms/u/s!AnR50DmsM8Jwl9kWzDgeuBgB40XJxQ?e=8LZw4Rhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2F1drv.ms%2Fu%2Fs!AnR50DmsM8Jwl9kWzDgeuBgB40XJxQ%3Fe%3D8LZw4R&data=02|01||7c122748309c4833a0ea08d86ab615db|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637376677560680810&sdata=XsFijwiVR5%2Bjqr4YNIrb5998njKO3oCuGVOdcp%2FWzwk%3D&reserved=0
How would you set a space to have approval but the rest of the spaces in the Building to be auto-approved?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fpowerapps-tools%2Fissues%2F447%23issuecomment-704878665&data=02|01||7c122748309c4833a0ea08d86ab615db|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637376677560690805&sdata=0I%2BQceO0aq%2BkFJLXv3%2FG4pgkY7ZTDrfTv3WSOhS%2BcCQ%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALSAPKX522UPC3ZQD2NT5VLSJRH2VANCNFSM4QR5QDKQ&data=02|01||7c122748309c4833a0ea08d86ab615db|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637376677560700801&sdata=XvZYTF005SrswfJTPI2rTr6wTzbcbHZbc4XnGkmkt9I%3D&reserved=0.


How to use it in another react project where the react version might be different by adding react and react dom as dependency?

Hi, react and react-dom have been added as peerDependencies in v2. here is the new r2wc


Add ability to specify fluentd service annotations

Add ability to specify fluentd service annotations.
This is useful when we need to create an ELB with a TLS certificate or to specify that the ELB is "internal" and not exposed to WWW

Please review my PR?

Can someone assist me to figure out how to solve the "invalid-owners-file" issue in my PR?
#15318

/help-wanted


Update 055-aggregation-grouping-summarizing.mdx

Describe this PR

Changes

What issue does this fix?

Any other relevant information

@ryands17 I resolved a merge conflict (we had removed the admonition about MongoDB not being supported after you opened this) and went ahead and pushed to your PR, then merged it. Thanks, lemme know if you prefer a different process for merge conflicts.


Ghost branded generic tool - remove link to retired measure

https://openprescribing.net/sicbl/06H/ghost_generics/
This page contains a link "view change over time" to a measure which has since been retired.

https://github.com/ebmdatalab/openprescribing/pull/4606

Merged - closed by 4606


Configure auto-updates for nvidia-container-toolkit (via Renovate)

This updates the nvidia-container-toolkit version, which is used to determine the version to download for these packages: libnvidia-container1, libnvidia-container-tools, nvidia-container-toolkit-base, and nvidia-container-toolkit

Example PR from my fork: https://github.com/ganeshkumarashok/aks-gpu/pull/18


Introducing Val_data in Fit function

Hello guys,
I was trying NSL out, but I was unable to run .fit with my validation data. What is the correct way to do it?
The following seems to be working just fine:
adv_model.fit({'feature': Xtrain_array, 'label': Ytrain_array},epochs=20)

Xtrain_array is a np.array (shape= (20410, 70, 7, 1))
Ytrain_array is a np.array (shape= (20410,4))

I tried converting the training and test arrays to a Dataset class using the following code but I'm getting an error "KeyError: 'feature'":

x_train=Xtrain_array
y_train=Ytrain_array
x_val=Xtest_array
y_val=Ytest_array
batch_size=32
train_data = tf.data.Dataset.from_tensor_slices({'input': x_train, 'label': y_train}).batch(batch_size)
val_data = tf.data.Dataset.from_tensor_slices({'input': x_val, 'label': y_val}).batch(batch_size)
val_steps = x_val.shape[0] / batch_size
adv_model.fit(train_data, validation_data=val_data,validation_steps=val_steps, epochs=2, verbose=1)

@pascoa-pand, thanks for the question.
What is your Tensorflow and NSL version?
I am able to run .fit with validation data (set to a tf.data.Dataset object) using TF 2.1.0. The following code (based on the example here) works on Google Colab:
adv_model.fit(train_set_for_adv_model,
              validation_data=test_set_for_adv_model,
              validation_steps=10,
              epochs=2)

And the output includes validation losses:
Epoch 1/2
1875/1875 [==============================] - 22s 12ms/step - loss: 0.0700 - sparse_categorical_crossentropy: 0.0240 - sparse_categorical_accuracy: 0.9923 - adversarial_loss: 0.2296 - val_loss: 0.0820 - val_sparse_categorical_crossentropy: 0.0344 - val_sparse_categorical_accuracy: 0.9844 - val_adversarial_loss: 0.2382
Epoch 2/2
1875/1875 [==============================] - 17s 9ms/step - loss: 0.0574 - sparse_categorical_crossentropy: 0.0196 - sparse_categorical_accuracy: 0.9940 - adversarial_loss: 0.1892 - val_loss: 0.0837 - val_sparse_categorical_crossentropy: 0.0462 - val_sparse_categorical_accuracy: 0.9875 - val_adversarial_loss: 0.1875

Closing this issue for now. Feel free to reopen if the error occurs again.


Handle RemovedInDjango110Warning Notices

Running the test suite for py34-django-latest we get a number of RemovedInDjango110Warning notices...
django_filters/filterset.py:104: RemovedInDjango110Warning:

  'get_field_by_name is an unofficial API that has been deprecated.
  You may be able to replace it with 'get_field()'
      rel, model, direct, m2m = opts.get_field_by_name(parts[-1])

django_filters/filterset.py:90: RemovedInDjango110Warning:

  'get_field_by_name is an unofficial API that has been deprecated.
  You may be able to replace it with 'get_field()'
      rel = opts.get_field_by_name(name)[0]

django-filter/tests/test_filterset.py:195: RemovedInDjango110Warning:

  'get_field_by_name is an unofficial API that has been deprecated.
  You may be able to replace it with 'get_field()'
       f = User._meta.get_field_by_name('comments')[0]

django-filter/tests/test_filterset.py:222: RemovedInDjango110Warning:

   'get_field_by_name is an unofficial API that has been deprecated.
   You may be able to replace it with 'get_field()'
       f = Worker._meta.get_field_by_name('employers')[0]

django-filter/tests/test_filterset.py:204: RemovedInDjango110Warning:

  'get_field_by_name is an unofficial API that has been deprecated.
  You may be able to replace it with 'get_field()'
      f = Book._meta.get_field_by_name('lovers')[0]

django-filter/tests/test_filterset.py:213: RemovedInDjango110Warning:

    'get_field_by_name is an unofficial API that has been deprecated.
    You may be able to replace it with 'get_field()'
        f = DirectedNode._meta.get_field_by_name('inbound_nodes')[0]

django-filter/tests/test_filterset.py:186: RemovedInDjango110Warning:

   'get_field_by_name is an unofficial API that has been deprecated.
   You may be able to replace it with 'get_field()'
       f = Account._meta.get_field_by_name('profile')[0]

django-filter/.tox/py34-django-latest/lib/python3.4/site-packages/django/test/testcases.py:229: RemovedInDjango110Warning:

   SimpleTestCase.urls is deprecated and will be removed in Django 1.10.
   Use @override_settings(ROOT_URLCONF=...) in GenericClassBasedViewTests instead.
       self._urlconf_setup()

django-filter/tests/urls.py:13: RemovedInDjango110Warning:

   django.conf.urls.patterns() is deprecated and will be removed in Django 1.10.
   Update your urlpatterns to be a list of django.conf.urls.url() instances instead.
       (r'^books/$', FilterView.as_view(model=Book)),

django-filter/.tox/py34-django-latest/lib/python3.4/site-packages/django/conf/urls/__init__.py:89: RemovedInDjango110Warning:

  Support for string view arguments to url() is deprecated and will be removed in Django 1.10
  (got django_filters.views.object_filter).
  Pass the callable instead.
      t = url(prefix=prefix, *t)

django-filter/.tox/py34-django-latest/lib/python3.4/site-packages/django/template/utils.py:37: RemovedInDjango110Warning:

  You haven't defined a TEMPLATES setting.
  You must do so before upgrading to Django 1.10.
  Otherwise Django will be unable to load templates. "unable to load templates.", RemovedInDjango110Warning)

django-filter/.tox/py34-django-latest/lib/python3.4/site-packages/django/test/testcases.py:229: RemovedInDjango110Warning:

   SimpleTestCase.urls is deprecated and will be removed in Django 1.10.
   Use @override_settings(ROOT_URLCONF=...) in GenericFunctionalViewTests instead.
        self._urlconf_setup()

this is a _meta API refactor issue


Trouble running devhotloader

Running devHotLoader of a clean repo pull and opening up localhost:3000 gives me a error such as this:
Invariant Violation: renderToString(): You must pass a valid ReactElement.
I thought this was a peerdependency problem so I added react and reactdom into the devDependencies in Package.json but that didnt resolve the problem. Is the hot loader not running on 3000? I know the assets are proxied from 3001/assets

@kkotwal94 did you end up finding a solution to this? I'm running into the same issue.

Not at all actually, I asked on chat too. I tried searching it up and figuring out myself and this is what i found https://github.com/rackt/react-router/issues/2139
@choonkending any insight?

Ah. Thanks for replying so quick. I've tried updating the router from rc to 1.0.0, but no luck there either. I'll keep digging.

Ill try as well right now.

Might be the webpack config for dev being outdated. I will be looking to
using react-transform-hmr in the next few weeks.
On Thu, 19 Nov 2015 14:38 Karan Kotwal<EMAIL_ADDRESS>wrote:

Ill try as well right now.
—
Reply to this email directly or view it on GitHub
https://github.com/choonkending/react-webpack-node/issues/76#issuecomment-157942198
.

@kkotwal94 HMR is in! :) Closing this issue, but feel free to open a new issue if you find any bugs/improvements that can be made.


DNS debugging article should be extracted from DNS configuration doc

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
https://kubernetes.io/docs/tasks/administer-cluster/dns-custom-nameservers/
This article contains two topics which I think should be separate:

Customizing DNS service
Debugging DNS

These two are not necessarily related.
Proposed Solution:
Move Debugging DNS into a separate article.
Page to Update: https://kubernetes.io/docs/tasks/administer-cluster/dns-custom-nameservers/

@ahmetb 👋 Agreed, Debugging would be better as its own article.

Is it ok to take up this issue and work on this?

sure. @scriptonist

Closing since Issue resolved by PR #7436

Apparently this is fixed now.


Potential typo in wiki (Initializing Window)

On the Initializing Window  section, the example given is:
swapi_createwindow(L"window", 0, 0, 600, 600);
If I am not mistaken, shouldn't it be:
swapi_initwindow(L"My Window", 0, 0, 600, 600);
According to your documentation and demo, swapi_createwindow() doesn't have arguments.

Yes, it is true. I changed a bit the code, so we would have 3 functions. init_window, create_window, and show_window. There only was create_window and show_window, so it was a typo. Thanks a lot for noticing the typo!

Now I'm going to remove the init_window function, and combine init_window with create_window, same as how it was before. Thanks again for opening this issue!