go-guerrilla doesn’t support “RCPT TO: ”.

RFC 5321
   Any system that includes an SMTP server supporting mail relaying or
   delivery MUST support the reserved mailbox "postmaster" as a case-
   insensitive local name.  This postmaster address is not strictly
   necessary if the server always returns 554 on connection opening (as
   described in Section 3.1).  The requirement to accept mail for
   postmaster implies that RCPT commands that specify a mailbox for
   postmaster at any of the domains for which the SMTP server provides
   mail service, as well as the special case of "RCPT TO:<Postmaster>"
   (with no domain specification), MUST be supported.

   o  The reserved mailbox name "postmaster" may be used in a RCPT
      command without domain qualification (see Section <IP_ADDRESS>) and
      MUST be accepted if so used.

   Syntax:

      rcpt = "RCPT TO:" ( "<Postmaster@" Domain ">" / "<Postmaster>" /
                  Forward-path ) [SP Rcpt-parameters] CRLF

                  Note that, in a departure from the usual rules for
                  local-parts, the "Postmaster" string shown above is
                  treated as case-insensitive.

Test
Preset
"allowed_hosts": ["a.com"]

Expectation
Case # 1
RCPT TO<EMAIL_ADDRESS>250 2.1.5 OK

Case # 2
RCPT TO<EMAIL_ADDRESS>454 4.1.1 Error: Relay access denied: not-a.com

Case # 3
RCPT TO: <poSTmAsteR>
250 2.1.5 OK

Case # 4
RCPT TO: <"po\ST\mAs\t\eR">
501 5.5.4 Invalid address

Current
Case # 1 ✅
RCPT TO<EMAIL_ADDRESS>250 2.1.5 OK

Case # 2 ✅
RCPT TO<EMAIL_ADDRESS>454 4.1.1 Error: Relay access denied: not-a.com

Case # 3 ❌
RCPT TO: <poSTmAsteR>
454 4.1.1 Error: Relay access denied:

Case # 4 ✅
RCPT TO: <"po\ST\mAs\t\eR">
501 5.5.4 Invalid address

Fix (not tested)

Validate the syntax per the ABNF. This must be done prior to the step 2, or it would break on the case # 4.
If the Host is empty and the User is case-insensitively Postmaster, say OK.

https://github.com/flashmob/go-guerrilla/blob/51f7dda326b1e9878e5f679ccb34a134127951b0/server.go#L492-L503
↓
				if s.allowsHost(to.Host) || (to.Host == "" && strings.toLower(to.User) == "postmaster") {
					client.PushRcpt(to)
					rcptError := s.backend().ValidateRcpt(client.Envelope)
					if rcptError != nil {
						client.PopRcpt()
						client.sendResponse(r.FailRcptCmd, " ", rcptError.Error())
					} else {
						client.sendResponse(r.SuccessRcptCmd)
					}
				} else {
					client.sendResponse(r.ErrorRelayDenied, " ", to.Host)
				}

The ABNF has already been parsed in parse.go
TODO:

modify the parser to evaluate the quoted local-part values as the expression is being parsed.
add a new string to the Address struct to store the evaluated result (we want to store both the raw value and the evaluated one)
modify the if s.allowsHost line as suggested above. However, we should also modify the allowsHost to match the evaluated value. (The case 4 above is a valid address <"po\ST\mAs\t\eR"> - no?)
add a test case for RCPT TO:  and <"po\ST\mAs\t\eR">

There's a question what Address.host should we default to? Should we fill any value there? Leave blank, or add a new config option, or just use 'localhost' ?

The RFC seems to be a bit unclear for "po\ST\mAs\t\eR" and<EMAIL_ADDRESS>inputs, but what you’ve said would be the right path to go, I guess; edited.
And for the Address.Host of <Postmaster>: we could simply leave the Host blank (as is) and modify the stringifying steps for use by processors (#199).

Hi @issuefiler, Please see PR #202 - it's ready for your acceptance test :-)
It works a little different from what was specified above (https://github.com/flashmob/go-guerrilla/issues/201#issuecomment-557957171)
It simply captures quoted local-parts without the escape characters, and then the String() function knows if it was quoted or not. So if the local-part is "test" " then it will be stored as "test " ".
So, allowsHost didn't need any modification after all.
When testing:
Note that if  is used in the RCPT TO (without a host), then new functionality was added to assume that the host is assumed to be the Hostname setting for the Server. Therefore, we must ensure that the Hostname setting also exists in the AllowedHosts config setting. (The server will log a warning if it detects this case)


v3: Add robotics module

This is the third version of the robotics module. Hopefully I've addressed all concerns from the previous versions (https://github.com/solettaproject/soletta/pull/1420 and https://github.com/solettaproject/soletta/pull/1264, respectively.)

Aside from a very specific and easy to change points we've discussed on IRC, +1
So if you decide to just change and push direct to master, seems ok for me.

What happened to this PR? Made its way to 'The Land of Forgotten Pulls'?

@edersondisouza hehe, I believe I'll merge it.
I understand this code is good to be merged
If @lpereira decides to make other changes later it's fine.

merged, thanks


Plugin makes my app auf Deutsch

Hey, I have no trouble myself with german but I guess our apps users are not that good with german as I am. The plugin changed totally my apps language.. All the popups etc. (Native ios) popups are in german. (I even tried to remove the plugin)
Atte

I am sorry to hear that!
When switching my iOS 10 device to English, my test app shows the correct (English) phrases. To investigate, I need more details:

Are the native iOS popups in English when not using this plugin?
Where do you see the German phrases? E.g. German keyboard, German "Cancel" button when clicking on a file input, ...
How are you building your app? (using PhoneGap Build which is the main target for this plugin, or locally, or...?)

Btw I have added the following to my config.xml - maybe you could give it a try?
<config-file platform="ios" parent="CFBundleDevelopmentRegion" mode="replace">
	<array>
		<string>en-US</string>
	</array>
</config-file>
<config-file platform="ios" parent="CFBundleLocalizations" mode="replace">
	<array>
		<string>en</string>
		<string>de</string>
	</array>
</config-file>

Yes, in english(in other apps, the language is totally changed in the app even when the plugin is removed)
keyboard, popups in german
3.cordova, iOS

Could you try the config file settings I posted above? (If it doesn't work, maybe you can change these settings manually in xcode? I am not familiar with it but this could help as well.)
Also: Please double check that no other plugin is causing these problems (maybe by creating a minimal project that only includes my plugin and the settings from above). After removing a plugin, the changes it made shouldn't affect the app anymore... 😃

I am closing this issue since there was no activity within the last month. If you still experience problems or have any questions, feel free to comment on this issue or to create a new one.

Hey! We got it working by removing the image feature. (It might have been working with your config but we were lazy :p). Now I’ve launched another project and now I truly need your plugin. The language problem was fixed with that tip you gave me but for some reason I can’t add your plugin. Tried it from the config file and through Cordova plugin add [git address.git]. But it shows me an error:
Error: Failed to fetch plugin https://github.com/MBuchalik/cordova-plugin-ios-permissions.git via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Could you help me?
Atte

MBuchalik<EMAIL_ADDRESS>kirjoitti 14.6.2017 kello 10.58:
Closed #1 https://github.com/MBuchalik/cordova-plugin-ios-permissions/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MBuchalik/cordova-plugin-ios-permissions/issues/1#event-1122781555, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5TeiVD4O3lN37QA1swmFuB772_pQfxks5sD5K4gaJpZM4NTSvQ.

I have never built apps locally (only through PhoneGap Build) so I am not sure if I can really help you with your problem. Have you tried adding the plugin to your config.xml like so?
<plugin name="cordova-plugin-ios-permissions" spec="https://github.com/MBuchalik/cordova-plugin-ios-permissions.git#v1.0.0" source="git" />

What error does cordova report to you when adding this line?
Are you able to add other plugins hosted on github?
Also: When building locally - can you download the plugin and add it from a local folder?

This is the error:
Discovered plugin "cordova-plugin-ios-permissions" in config.xml. Adding it to the project
Failed to restore plugin "cordova-plugin-ios-permissions" from config.xml. You might need to try adding it again. Error: Failed to fetch plugin cordova-plugin-ios-permissions@https://github.com/MBuchalik/cordova-plugin-ios-permissions.git#v1.0.0 via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 235 Error output:
npm ERR! addLocal Could not install /var/folders/3w/v5vmg3k97h35k87n4rh113vm0000gn/T/npm-45890-1a5bd3e1/git-cache-f372a0e9/abace2141717916071eb0b5a6de7e96cd5e88e3c
npm ERR! Darwin 16.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "cordova-plugin-ios-permissions@https://github.com/MBuchalik/cordova-plugin-ios-permissions.git#v1.0.0"
npm ERR! node v6.10.1
npm ERR! npm  v3.10.10
npm ERR! code EISDIR
npm ERR! errno -21
npm ERR! syscall read
npm ERR! eisdir EISDIR: illegal operation on a directory, read
npm ERR! eisdir This is most likely not a problem with npm itself
npm ERR! eisdir and is related to npm not being able to find a package.json in
npm ERR! eisdir a package you are trying to install.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/attepohjanmaa/projects/knowtheshow-cordova/node_modules/npm-debug.log
Discovered plugin "cordova-plugin-ios-permissions" in config.xml. Adding it to the project
Failed to restore plugin "cordova-plugin-ios-permissions" from config.xml. You might need to try adding it again. Error: Failed to fetch plugin cordova-plugin-ios-permissions@https://github.com/MBuchalik/cordova-plugin-ios-permissions.git#v1.0.0 via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 235 Error output:
npm ERR! addLocal Could not install /var/folders/3w/v5vmg3k97h35k87n4rh113vm0000gn/T/npm-45938-167f97ec/git-cache-edf785d0/abace2141717916071eb0b5a6de7e96cd5e88e3c
npm ERR! Darwin 16.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "cordova-plugin-ios-permissions@https://github.com/MBuchalik/cordova-plugin-ios-permissions.git#v1.0.0"
npm ERR! node v6.10.1
npm ERR! npm  v3.10.10
npm ERR! code EISDIR
npm ERR! errno -21
npm ERR! syscall read
npm ERR! eisdir EISDIR: illegal operation on a directory, read
npm ERR! eisdir This is most likely not a problem with npm itself
npm ERR! eisdir and is related to npm not being able to find a package.json in
npm ERR! eisdir a package you are trying to install.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/attepohjanmaa/projects/knowtheshow-cordova/node_modules/npm-debug.log

MBuchalik<EMAIL_ADDRESS>kirjoitti 16.6.2017 kello 11.36:
I have never built apps locally (only through PhoneGap Build) so I am not sure if I can really help you with your problem. Have you tried adding the plugin to your config.xml like so?

What error does cordova report to you when adding this line?
Are you able to add other plugins hosted on github?
Also: When building locally - can you download the plugin and add it from a local folder?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MBuchalik/cordova-plugin-ios-permissions/issues/1#issuecomment-308968079, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5TetUN2prc4VY0mmrME8HLqTvSpWSsks5sEj6AgaJpZM4NTSvQ.

Could you download the plugin to a local directory and install it from there?

Hi! Tried that, got a following error:
Error: Invalid Plugin! plugins/cordova-plugin-ios-permissions needs a valid package.json

MBuchalik<EMAIL_ADDRESS>kirjoitti 17.6.2017 kello 16.20:
Could you download the plugin to a local directory and install it from there?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MBuchalik/cordova-plugin-ios-permissions/issues/1#issuecomment-309214842, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5TeiQltDDqn8wimc-HG3iPzRgO-gNGks5sE9K2gaJpZM4NTSvQ.

Ok! I was able to get it working after some hustling with the plugman! The language is in German now but I guess I can progress by myself. Thanks buddy!
A

Atte Pohjanmaa<EMAIL_ADDRESS>kirjoitti 21.6.2017 kello 18.57:
Hi! Tried that, got a following error:
Error: Invalid Plugin! plugins/cordova-plugin-ios-permissions needs a valid package.json

MBuchalik<EMAIL_ADDRESS><EMAIL_ADDRESS>kirjoitti 17.6.2017 kello 16.20:
Could you download the plugin to a local directory and install it from there?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MBuchalik/cordova-plugin-ios-permissions/issues/1#issuecomment-309214842, or mute the thread https://github.com/notifications/unsubscribe-auth/AF5TeiQltDDqn8wimc-HG3iPzRgO-gNGks5sE9K2gaJpZM4NTSvQ.

No problem, I am glad it works now :smiley:


Fix TO Go to mask field

What does this PR do?
Fix TO Go to mask field
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2463/
Test PASSed.


ISBN regexp improvement

An ISBN regexp improvement allowing to extract more polluted citations like 902 198 84 X, 1-57488-530-8, and {{ISBN|978-83-7435-239-0}} ones.
In colleague's work - he reported to extract 30-40% percentage more valid citations after the fix.
(Also contains minor code cleanup and fixes)

Do you have any example revisions where pipes are used? Would be nice to link to them in the commit message.

This one was ours inspiration: Editing Dies irae (section).
Contains pipes in citations, similar form like these in the test code:

''Słownik muzyki'', red. Wojciech Marchwica, Wydawnictwo Zielona Sowa, Kraków 2006; {{ISBN|83-7435-239-6}}, {{ISBN|978-83-7435-239-0}}.. W [[Zwyczajna forma rytu rzymskiego|nowym obrządku]] nie jest już powszechnie używana (można ją wykonywać w ostatnim

You are right https://regex101.com/r/uekqPD/1, I have corrected the regex. Leftover of try with alternative | matching.

Thanks for the pull request!


docs: add a note that brew install doesn't work on Apple Silicon M1

Bug Report
I have issued
brew install --build-from-source dvc

on a macOS Big Sur system. Homebrew downloaded the required packages.
==> Installing dependencies for dvc: icu4c, boost, libffi, swig, llvm@9, brotli, gflags, glog, autoconf, automake, libtool, abseil, c-ares, protobuf, re2, grpc, lz4, gmp, isl, mpfr, libmpc, gcc, openblas, gdbm,
 readline, sqlite, tcl-tk<EMAIL_ADDRESS>numpy, rapidjson, snappy, thrift, zstd and apache-arrow

However while building swig I got
==> Installing dvc dependency: swig
==> Pouring swig-4.0.2.arm64_big_sur.bottle.tar.gz
/opt/homebrew/Cellar/swig/4.0.2: 727 files, 5.5MB
llvm@9: The x86_64 architecture is required for this software.

BTW pip version seems to be installed fine.

@iesahin Transferred the issue to our docs repo.

@iesahin Transferred the issue to our docs repo.

For the record: looks like arm bottle is coming soon https://github.com/Homebrew/homebrew-core/pull/69837#issuecomment-767935617

For the record: looks like arm bottle is coming soon https://github.com/Homebrew/homebrew-core/pull/69837#issuecomment-767935617

Hi @iesahin ! Looks like it should be available now https://github.com/Homebrew/homebrew-core/pull/69837 , could you give it a try, please?

Hi @iesahin ! Looks like it should be available now https://github.com/Homebrew/homebrew-core/pull/69837 , could you give it a try, please?


V2 - reformat the profile screen

Based on AnMaree's excellent design - (and our current utter lack of design effort on the profile screen) - I was wondering if we could take a stab at reformatting this a bit before the V2 release?
This image isn't exactly what we need....but it is darn close.
Notes about this image:
We still need a place to put our 'For Help' link...somewhere.... (note: we can probably get rid of the 'For Map').  Maybe (for now) instead of all the icons on the right of the screen - just include ONE "?" icon that goes to the "For Help" link?
@JonathanKoerber - your '# of Trees' (https://github.com/TreeMama/iSeaTree-React-Prototype/issues/55) relates to this. You could use the Deciduous/Conifer icons available here;
https://drive.google.com/drive/u/1/folders/1Sla8AUaH1mANN7KrrUiWo6Nz4v9sFZnZ

I could take this one on. Is the above image how the screen would like to look? Also, do we have the icons? Will that NavBar carry over through out the app?

I could take this one on. Is the above image how the screen would like to look? Also, do we have the icons? Will that NavBar carry over through out the app?

@JonathanKoerber - I'm working on the icons now....ignore the nav bar changes in the graphic for now. They are V3 items. V2 would be the screen items for the profile page....

@JonathanKoerber - I'm working on the icons now....ignore the nav bar changes in the graphic for now. They are V3 items. V2 would be the screen items for the profile page....

Punting this to V2.1. It's a high priority...but will not block the V2 release.

Punting this to V2.1. It's a high priority...but will not block the V2 release.


Fix M5stickC Plus2 backlight blinking when inactivity timeout

First of all, thank you for your project. I'm using it very well.
But backlight doesn't turn off on the m5stickc plus2.
so set Brightness to 0.
https://github.com/user-attachments/assets/cece9d6e-220e-4219-a836-24e37692dcb5
I've tested it and it works fine on me. But you need to check how it works on other devices.

@hijae Thank you for the PR.
Hmm, I do not see this behaviour with my M5StickC Plus2.
Can you share a few more details on your setup (eg. firmware version) so I can try to reproduce?
I would like to avoid disabling the backlight completely, otherwise it is difficult to tell if the device is off or sleeping.

@hijae Thank you for the PR. Hmm, I do not see this behaviour with my M5StickC Plus2. Can you share a few more details on your setup (eg. firmware version) so I can try to reproduce?
I would like to avoid disabling the backlight completely, otherwise it is difficult to tell if the device is off or sleeping.

Ah, I found it.
The web installer is using an old beta version for some reason.
This was fixed in #86.
Hmm, I need to find out why the web installer is not up to date with v2.0.0.

OK, I messed up recently when playing with the web installer deployments.
I have rolled it back to the correct v2.0.0+1 deployment (it was previously on v2.0.0-beta+1).

@hijae Thank you for finding and raising this, I appreciate your time and effort.
Closing as "Oh dear, I messed up".


Ubunut Docker installation - Need help by link

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
I have installed Bookstack under ubuntu and docker compose. I have configured everything so far. When I now call up the website, I always land on this example website: https://bookstack.example.com/login. Apache2 does not give any errors. I have adapted 000-default.conf with the path of my installation. In the docker-compose.yml I have specified the app url, tried once with https:/xxx.xx and once with the IP: 192.... Unfortunately both without success I always end up on the example page of Bookstack when I call up the homepage, does anyone have an idea? I have rented an Ubuntu server and have installed the latest Ubunut version and run Plesk on it.
Exact BookStack Version
newest from homepage
Log Content

Hosting Environment
Ubuntu 22.04 mit Plesk 18 Obsidian
PHP 8.2.17

Hi @Gumblfreak-R,

Please ensure the APP_URL is set exactly to the base URL you'd expect to access BookStack on.

Ensure you're re-creating the containers (compose down & up) rather than just restarting after changing this.

I don't understand how Apache is used with the shown docker setup here, they don't look connected at all. The apache looks set-up for a sub-path setup leading to a on-instance install, whereas you've got a docker-based install for which I'd expect to see a proxy-based setup.

Can you explain the expected URL format, and all the parts you have in play?

Do you intend to use a sub-path?
Is apache currently leading requests to the docker setup somehow? If so, how?
Are you using a port number in the URL when accessing it? If so, do you intend to?

I have an Ubuntu server incl. Plesk installation. I have installed Docker and Bookstack as follows:
curl -fsSL https://get.docker.com | sudo sh
First create the .yml file in your desired installation path
as an example the path would be /docker/bookstack
then open the compose file here with vi:
vi docker-compose.yml
Copy the content below and adapt it to your bookstack environment. Be sure to replace the example passwords!
services:
bookstack:
image: lscr.io/linuxserver/bookstack
container_name: bookstack
environment:
- PUID=1000
- PGID=1000
- APP_URL=https://bookstack.example.com
- DB_HOST=bookstack_db
- DB_PORT=3306
- DB_USER=bookstack
- DB_PASS=
- DB_DATABASE=bookstackapp
volumes:
- ./bookstack_app_data:/config
ports:
- 6875:80
restart: unless-stopped
depends_on:
- bookstack_db
bookstack_db:
image: lscr.io/linuxserver/mariadb
container_name: bookstack_db
environment:
- PUID=1000
- PGID=1000
- MYSQL_ROOT_PASSWORD=
- TZ=Europe/Berlin
- MYSQL_DATABASE=bookstackapp
- MYSQL_USER=bookstack
- MYSQL_PASSWORD=
volumes:
- ./bookstack_db_data:/config
restart: unless-stopped
Now exit the editor with :wq!
Then start your Bookstack Docker environment with
docker compose up -d
You can use docker ps to check whether your Docker environment is running.
Your URL is then https://bookstack.example.com:6875 or https://192.168.178.XX:6875

i chenged the url at the composer.yml with my url. but i always land on the example page...
by apache2 configs i have entered this one:
<VirtualHost *:80>
...

# BookStack Configuration
Alias "/bookstack" "/var/www/bookstack/public"

<Directory "/var/www/bookstack/public">
  Options FollowSymlinks
  AllowOverride None
  Require all granted

  RewriteEngine On
  # Redirect Trailing Slashes If Not A Folder...
  RewriteCond %{REQUEST_FILENAME} !-d
  RewriteRule ^(.*)/$ /$1 [L,R=301]

  # Handle Front Controller...
  RewriteCond %{REQUEST_FILENAME} !-d
  RewriteCond %{REQUEST_FILENAME} !-f
  RewriteRule ^ index.php [L]
</Directory>

<Directory "/var/www/bookstack">
  AllowOverride None
  Require all denied
</Directory>
# End BookStack Configuration

...

here i have updated my path and then saved

@Gumblfreak-R That apache setup appears in no way linked to the docker setup? Do you expect it to be?
Do you have a domain name you're using to access BookStack? Or an IP address?

i have a domain name. but i also tried it with the ip adress


Bug: I2C lock up

Product: (OC-SDR, OC-LTE, OC-LTE, OC-Power, others)
OC-SDR
Category: (Hardware, Software/stack, Firmware/bootloader, Documentation)
Firmware/OCWare
Describe the issue found
This is a hardware issue that we've seen on multiple boards that I'd like get to the bottom of. Firmware will need to be made robust enough to handle SCL being held low anyway , but on SDR 33, the bus locks up if we don't read from the EEPROM first. Very strange, and a bit worrying that the entire bus can be taken down but partially fixed with software if things are done in the correct order.
To repro, run latest Rev_C_Changes on SDR 33 - POST will stop shortly into SDR. To fix, simply add
char tmp_char;
eeprom_read(OC_SS_SDR, OC_SDR_INVEN_EEPROM, OC_SDR_BOARD_INFO, &tmp_char, 1);

in sdr_init() right before the call to sdr_fx3_reset();
Please provide all necessary steps to reproduce the issue
A clear and concise description of steps to reproduce the bug.

[Originally commented by imgraham]
Seems that this is related to our EEPROM chip - I can get i2c7 to lock up simply by trying to write to one of the GBC's EEPROM while WP is asserted. By the data sheet, the IC doesn't ACK any of the data bytes in this scenario. This is easily reproducible, so I'd like to probe the lines to see what's up. It's really unfortunate if it's that easy to screw up Tiva - we might have to muck around with the TI drivers afterall.


Increase RAM limit on migration service

See:
https://github.com/ITISFoundation/osparc-simcore/pull/4418#issuecomment-1604529841

https://monitoring.osparc-master.speag.com/prometheus/graph?g0.expr=container_memory_usage_bytes{image%3D~"itisfoundation%2Fmigration%3Amaster-github-latest.%2B"}&g0.tab=0&g0.stacked=0&g0.show_exemplars=0&g0.range_input=1h

https://github.com/ITISFoundation/osparc-ops-environments/pull/247


Deployment Automation with Builds

Trigger automatic deploy (maybe for different platforms) when merging to master with GitHub actions

Done


Fix performance of update_variant_names

I want to merge this change because it fixes the performance of update_variants_names task.
Local database (over 2500 affected variants)
Before: 26.7s ❌
After: 1.9s ✅
⚠ This is a port of https://github.com/saleor/saleor/pull/10657

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: CLOUD-2021 Optimize update_variant_names which times out on RT-stage


dbt_profile_dir

Use a better way to change and keep configuration for dbt_profile_dir

hey @akadiryigit thanks for opening this pr up, it's a great callout! for OS-specific or terminal-specific configurations, w recommend other documents, rather than specify this on our docs site for maintenance reasons. This document has various shell examples.
I'll go ahead and close this pr, but we really appreciate your contribution and we hope to see you opening up a pr soon! 😃

closing pr due to slack thread here: https://dbt-labs.slack.com/archives/C017GDLAF7D/p1674224184865989?thread_ts=1674139914.744859&cid=C017GDLAF7D


no Graph after Unfold

Before:

Folded:

After:

Refresh graph doesn't help.

Hi @532910,
Thanks for reporting this! Indeed, I do reproduce the issue. I'll come up with a fix for this.

I confirm that 0.5 works fine for me


Analytics - look over options and pick one

http://answers.io/answers ?

old repo - clearing issues


Update Example App

Update the example App to show better.

[ ] Default to Style Page
[ ] Add sub-pages for Dynamic Type examples
[ ] Make existing pretty styles Dynamic Type
[ ] Make the whole app 1 tableview
[ ] Remove fake content size category simulation.
[ ] Move GNUPlot code to a unit test
[ ] Restore RZ Red
[ ] Shorten Figure Style examples

RZ Red:
[UIColor colorWithRed:0.927
                           green:0.352
                            blue:0.303
                           alpha:1.0];

What do you think of moving the figure style examples to the bottom of the list and checking this off the list? I think it's in pretty good shape.

Works for me. We may end up putting the constraint examples down there as well.


Step 3 has incorrect sas token

Using the sas token in step 3 resulted in an authorization error.  After looking at Data Access section on the Seattle Safety Data page  I realized that the sas token should just be r"".

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

ID: 355fba0e-958e-f572-18c2-892c91d20b13
Version Independent ID: 7fe76247-c969-2068-2c3c-bf327fbbe39e
Content: Quickstart - Run a Spark job on Azure Databricks Workspace using Azure portal
Content Source: databricks/scenarios/quickstart-create-databricks-workspace-portal.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@ssoongcodes Thank you for sharing your feedback. We are actively validating it. We will share updates soon.

@mssaperla Could you please validate feedback and update as appropriate to update SAS? Thank you.

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and this outdated content has been removed. We are closing the issue.
#please-close

#please-close


Fix NavigationView pane not closing in minimal/compact mode when selecting an item

Description
This PR fixes a regression introduced with commit 5d444b6 which prevented the NavigationView pane to be closed in minimal/compact mode after selecting the Settings item or selecting an item programmatically (using NavigationView.SelectedItem).
Also added an interaction test to prevent this regression from occuring in the future.
Motivation and Context
Fixes #2244.
How Has This Been Tested?
Tested visually and added interaction test.

/azp run


Co-simulation framework

You can also look at the spike changes here: https://github.com/lowRISC/riscv-isa-sim/commit/d3cd47117e47bef6864d8e5c84f38a9cde0b42ae (it's in a branch in our Spike fork) you can comment directly on that commit.
This doesn't include the simple system integration by previous PR did as I've just been working on the UVM environment recently. I do intend to add a simple system integration once we've got this PR merged.
I've run a full regression with a good pass rate (over 90%) though I've been fiddling with it since so may have broken a few more things. In particular it seems turning on the icache causes issues with instruction errors and a recent RTL change I did seems to be hitting an assertion in ibex_controller.
I'll be looking at these issues in parallel (and doing another full regression) but wanted to get this PR out finally!

Closing/reopening to see if the linter errors are now shown on the right line.


Once logged the login button still apearrs

Create a superuser
Login
The wellcome page appears with the login button

Thank you for finding this. Expected behaviour is to redirect to the home page. Will work on a fix.

The login/sign-up workflow is now improved. Login takes you to the feed now.


Week 2 - Raiden Review Ticket

Individual Github Pages(3 points)

[X] Organize files, directories and menus for the first 3 weeks.
[X] Write Factorial function using classes, providing implementation of call.
[X] Select your own Math function. Write it in Imperative and OOP form.
[X] Extra Credit(1 point).  Write Palindrome function using classes, must provide implementation of call.  Must indicate where error occurs on front and back comparison.

Evidence

File Organization
Factorial
Math Function
Extra Credit Palindrome

Crossover Grader: Nathan Shih
Overall Grade 4/3

Has organized files that make it easy to find everything for the first 3 weeks
Factorial function uses classes and call and works nicely
Used the lcm math function and wrote in both ways
+1 extra credit point wrote palindrome and works.


Added "Stock Span" funny algorithm with story in markdown for Hacktoberfest 2023

Hii Sir, This contribution is for Hacktoberfest 2023. Kindly merge it.

Hey @marlonchalegre @ErickSimoes my PR is open from quite a lot of time. Kindly suggest me if any changes are required from your side. I will be more than happy if you merge it soon.


Record Label Inferences do not work in Sub-Module

If you define a module with a type inside, and do not open the module, when you instance a type from that module you either have to include the fully-qualified name, or declare the object as having the fully-qualified type.
Repro steps
    module Thing =
        type SomeType = { A : string; B : int }
    let someType = { Thing.SomeType.A = "Test"; B = 5 }
    // The above errors at "B":
    // Error : FS0039 : The record label 'B' is not defined.
    let someType : Thing.SomeType = { A = "Test"; B = 5 }
    let someType = { Thing.SomeType.A = "Test"; Thing.SomeType.B = 5 }
    // Both of the above mitigate the issue
    open Thing
    let someType = { A = "Test"; B = 5 }
    // The above also mitigates the issue, though the `open` is required.

Expected behavior
I would expect it to not error on the B label.
Actual behavior
It errors on the B label, with: "FS0039 : The record label 'B' is not defined."
Known workarounds
Define the object with the full type-name, or fully-qualify the type on all record labels.
Related information

Windows 10 Pro
Stable
.NET 4.6.1
Visual Studio 2017 15.4.0
Minor / Convenience

The current behavior seems reasonable to me.

@EBrown8534 It's currently by spec, but please open an fslang-suggestion.
thanks
don


Automation Account Variable Encryption Recommendation in Azure Security Center

Having enabled the VM Start / Stop solution in Log Analytics / Automation and deployed it successfully, Security Center issues recommendations for each of the variables as a "High" issue saying that they should be encrypted (See https://docs.microsoft.com/en-us/azure/security-center/recommendations-reference#recs-computeapp)
What I don't know is if I recreate each of the 17 variables myself (variables can only be encrypted at creation) is whether I will break the solution? Or can encryption be build in at deployment stage ?
If it is possible to correct manually, could guidance be provided in the instructions ?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 225c9d05-83dd-b006-0025-3753f5ab25bf
Version Independent ID: 9eecef0c-b1cb-1136-faf7-542214492096
Content: Azure Automation Start/Stop VMs during off-hours overview
Content Source: articles/automation/automation-solution-vm-management.md
Service: automation
Sub-service: process-automation
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@alistairl Thank you for your query, our team will review and get back to you at the earliest.

Sorry for the delay @alistairl .
@SwathiDhanwada-MSFT , @KrishnaG-MSFT  - Can you please take a look at this and followup with the Automation PG group if needed?

@alistairl - The Advisor rule to evaluate all the variables in your Automation account is unable to take into account variables, which don't contain sensitive information and don't need to be encrypted. In the case of the variables created by Start/Stop, there isn't anything sensitive stored in the variables (unless your security team considers the variables that store the webURI, webhook URI, and Automation Account name as hosting sensitive information), and you can configure exceptions for each of the variables created by Start/Stop so Advisor no longer triggers and flags them as non-compliant. In the meantime, I'll bring this up with the PM for the Start/Stop solution to see if they plan on encrypting any of the variables the solution generates and if a customer recreates them as encrypted, will it impact the solution from functioning correctly.  I haven't personally tested that scenario, I believe it should work because the runbooks are using the Get-AutomationVariable cmdlet, which supports working with encrypted variables.
Hope that helps. Because this isn't a documentation issue, I'm going to proceed with closing this GitHub issue, but I will monitor this thread if you have any further questions or will add additional information after I follow up with the team who owns the solution as previously mentioned.  #Please-close.


C++: more conservative resolveClass

This PR ensures that resolveClass never returns more than one result and does not attempt to match classes by name if they are templates or are nested in a name space or another class. This fixes the problem on LGTM where ChakraCore analysis went into an infinite loop in getQualifiedName because a class appeared to be nested within itself due to name clashes.
A more thorough fix would be to match classes by fully-qualified name, either with a string or a newtype, but at this point in the release cycle I'm proposing a more minimal fix.
The code change here is the same as in #198, where the CPP-Differences job found no query changes. I've extended the tests to try and show the effects of each commit.

That job compared an ODASA build to itself. I've started another one here: https://jenkins.internal.semmle.com/job/Query-Changes/job/CPP-Differences/412/

Good catch! Does the CPP-Differences job not respect qlSubmoduleShaOverride? Or did I put the wrong SHA?

It doesn't know about qlSubmoduleShaOverride (and if it did, there ought to be a qlSubmoduleShaOverride too).

The query changes LGTM. They have the following summary:
=== Overall changes (new / fixed) ===

                                                                           10 / 2

=== Changes per project (new / fixed) ===

linux                                                                      10 / 0
boost                                                                       0 / 2

=== Changes per query (new / fixed) ===

FEfferentCoupling.ql                                                       10 / 1
IncorrectPointerScalingChar.ql                                              0 / 1

I assume the change to IncorrectPointerScalingChar.ql in boost is just the inverse of the change we saw in #7. It's in a file where there are multiple types named implementation but with different qualifiers, and these changes make it so some of them are considered different. I don't understand the code well enough to tell whether it's the old or new behaviour that's correct, but the new behaviour is certainly more conservative, which is the intention with this PR.
The FEfferentCoupling.ql changes are hard to judge. The alerts say "Outgoing dependencies per file too high (N). It is a warning for the value to be higher or equal to 120" for various values of N close to 120. It seems reasonable to me that this query wobbles a bit under these changes.


Using dimple with webpack

Hi all,
I'm trying to write a React component for Dimple, and am trying to package it using Webpack. Has anybody tried doing this? I managed to do the same with the dygraphs package, but I'm having some issues with dimple.
I tried installing d3 and dimple via npm, and then adding this line to my JSX file:
import 'd3/d3.js';
import dimple from 'dimple-js/dist/dimple.latest.js';

Then try this inside the file:
var svg = dimple.newSvg("#chartContainer", 590, 400);

This is giving an error that dimple doesn't have the newSvg method.
Any ideas how to go about this?
Thanks!

I'm trying this as well, if anyone has figured it out please post

The change made in c3a10e5 allows you to import dimple from 'dimple
That change however, is not available in the 2.1.4 version of dimple that is currently available in npm.
In the meantime, npm install -S git+https://github.com/PMSI-AlignAlytics/dimple.git should pull the latest and do the trick

@ijoel Any ideas when the version in NPM will be updated? It seems like it's still at 2.1.4.


Redirect requests instead of using content scripts

I think redirecting YouTube URLs with a webRequest listener in the background script is better than injecting a content script into every page and checking if the user follows a YouTube link every time a click event occurs. So I made a few changes (and tested them thoroughly). The functionality has stayed absolutely the same.

Oh I forgot, I added the protocol handler manually. I will have to fix that before merging.


MINOR: Use string interpolation in FinalizedFeatureCache

This is a small change. In this PR, I'm using string interpolation in FinalizedFeatureCache at places where string format was otherwise used. This just ensures uniformity, with this change we ensure that throughout the file we just use string interpolation.
Test plan:
Rely on existing tests since this PR is not changing behavior.

Failed test is unrelated. Merging to trunk.

@dajac thanks for the review :)


Need to merge optimal vaspinteractive settings to main

Mark as TODO for branch tt-vaspinteractive-test

#35 solves PR.
TODOs

[ ] fix oal example
[ ] fix speedup example

done


JSON messages with RFC 3339 / ISO 8601 nanosecond timestamps not partitioned as expected to S3

When I analyse the files placed on S3 by secor I find there are messages being placed incorrectly with regards to date partitioning. Each JSON message has a top-level timestamp property which can be in either 2006-01-02T15:04:05.999999999Z format or, occasionally, 2006-01-02T15:04:05Z (as in, without nanoseconds, from systems which do not support such resolution).
Note: nanoseconds may not be a culprit here as the title suggests, that's just the majority of values in my dataset.
I've tried this with 0.22 from maven, and master (aee073b6a105f4365f50cabac1248747781a7a9a) from github.
It seems to work fine for the majority of events. However, when I scan the files for the dates of each message, I get results like...
# aws s3 ls --recursive s3://BUCKETNAME/secor-test-9/ | tr -s ' ' | cut -d' ' -f4 | sort | xargs -I{} sh -c 'echo {}; aws s3 cp s3://BUCKETNAME/{} - | zcat | jq ".timestamp[0:10]" | sort | uniq -c'

secor-test-9/athena-integration-1/dt=2017-01-31/1_22_00000000000002657819.gz
  19921 "2017-01-31"
      2 "2017-02-01"

secor-test-9/athena-integration-1/dt=2017-02-01/1_0_00000000000002675808.gz
  19302 "2017-02-01"
      3 "2017-02-02"
      1 "2017-02-03"

secor-test-9/athena-integration-1/dt=2017-02-01/1_11_00000000000002717065.gz
      2 "2017-02-02"
      2 "2017-02-03"
      5 "2017-02-04"
      3 "2017-02-05"
      2 "2017-02-06"

secor-test-9/athena-integration-1/dt=2017-02-01/1_12_00000000000002655329.gz
      3 "2017-01-31"
  33460 "2017-02-01"
      1 "2017-02-02"

secor-test-9/athena-integration-1/dt=2017-02-04/1_3_00000000000002720791.gz
      2 "2017-02-02"
  12869 "2017-02-04"

For one specific example above secor-test-9/athena-integration-1/dt=2017-02-01/1_11_00000000000002717065.gz the timestamps of each line are
"2017-02-02T22:33:03.12533900Z"
"2017-02-02T22:46:12.26030700Z"
"2017-02-03T00:49:04.27147800Z"
"2017-02-03T05:21:52.79127500Z"
"2017-02-04T00:12:43.81382200Z"
"2017-02-04T04:39:16.33926200Z"
"2017-02-04T04:41:33.62567100Z"
"2017-02-04T05:21:55.08586200Z"
"2017-02-04T07:39:26.89454900Z"
"2017-02-05T07:13:58.89362700Z"
"2017-02-05T07:59:05.27761600Z"
"2017-02-05T23:16:37.77798300Z"
"2017-02-06T19:20:29.98021100Z"
"2017-02-06T19:44:06.95325400Z"

Note that these nanosecond timestamps are handled correctly though for the vast majority (see: tens of thousands of correctly partitioned events).
My normalised (comments removed etc.) config file is:
aws.proxy.http.port=
aws.proxy.isEnabled=false
aws.region=
aws.role=
aws.secret.key=
aws.sse.customer.key=
aws.sse.kms.key=
aws.sse.type=
cloud.service=S3
hive.table.prefix=
kafka.consumer.auto.offset.reset=smallest
kafka.consumer.timeout.ms=10000
kafka.dual.commit.enabled=false
kafka.fetch.message.max.bytes=
kafka.fetch.min.bytes=
kafka.fetch.wait.max.ms=
kafka.offsets.storage=zookeeper
kafka.partition.assignment.strategy=range
kafka.rebalance.backoff.ms=
kafka.rebalance.max.retries=
kafka.seed.broker.host=...
kafka.seed.broker.port=9092
kafka.socket.receive.buffer.bytes=
kafka.zookeeper.path=/kafka-kafka
message.timestamp.id=1
message.timestamp.input.pattern=
message.timestamp.name.separator=
message.timestamp.name=timestamp
message.timestamp.required=true
message.timestamp.type=i64
monitoring.blacklist.topics=
monitoring.prefix=secor
ostrich.port=9999
partitioner.finalizer.delay.seconds=3600
partitioner.granularity.hour=true
partitioner.granularity.minute=false
qubole.api.token=
secor.compression.codec=org.apache.hadoop.io.compress.GzipCodec
secor.consumer.threads=7
secor.enable.qubole=true
secor.file.extension=
secor.file.reader.writer.factory=com.pinterest.secor.io.impl.DelimitedTextFileReaderWriterFactory
secor.generation=1
secor.kafka.group=secor-test-9
secor.kafka.topic_blacklist=
secor.kafka.topic_filter=^athena-((?!production).)*$
secor.kafka.upload_at_minute_mark.topic_filter=
secor.local.log.delete.age.hours=-1
secor.local.path=/tmp/secor-test-9
secor.max.file.age.seconds=120
secor.max.file.size.bytes=20000000
secor.max.message.size.bytes=100000
secor.message.parser.class=com.pinterest.secor.parser.Iso8601MessageParser
secor.message.transformer.class=com.pinterest.secor.transformer.IdentityMessageTransformer
secor.messages.per.second=100000
secor.offsets.per.partition=10000000
secor.offsets.prefix=offset=
secor.parser.timezone=UTC
secor.qubole.timeout.ms=300000
secor.s3.alter.path.date=
secor.s3.alternative.path=
secor.s3.bucket=...
secor.s3.filesystem=s3a
secor.s3.path=secor-test-9
secor.s3.prefix.md5hash=false
secor.topic_partition.forget.seconds=600
secor.upload.manager.class=com.pinterest.secor.uploader.S3UploadManager
secor.upload.minute_mark=0
secor.zookeeper.path=/secor-test-9
statsd.hostport=
tsdb.hostport=
zookeeper.quorum=...
zookeeper.session.timeout.ms=3000
zookeeper.sync.time.ms=200

This could be a bug, or it could be a misconfiguration on my behalf, but I've played around with this config for the better part of a day using the available documentation and can't improve it beyond this. Hourly granularity doesn't seem to work either in this setup, but that's another issue I suppose.

Take a look at the code in Iso8601MessageParser to see if it can handles
your two time formats.
On Thu, Feb 9, 2017 at 5:05 PM, gwilym<EMAIL_ADDRESS>wrote:

When I analyse the files placed on S3 by secor I find there are messages
being placed incorrectly with regards to date partitioning. Each JSON
message has a top-level timestamp property which can be in either
2006-01-02T15:04:05.999999999Z07:00 format or, occasionally,
2006-01-02T15:04:05Z07:00 (as in, without nanoseconds, from systems which
do not support such resolution).
Note: nanoseconds may not be a culprit here as the title suggests, that's
just the majority of values in my dataset.
I've tried this with 0.22 from maven, and master (aee073b
https://github.com/pinterest/secor/commit/aee073b6a105f4365f50cabac1248747781a7a9a)
from github.
It seems to work fine for the majority of events. However, when I scan the
files for the dates of each message, I get results like...
aws s3 ls --recursive s3://BUCKETNAME/secor-test-9/ | tr -s ' ' | cut -d' ' -f4 | sort | xargs -I{} sh -c 'echo {}; aws s3 cp s3://BUCKETNAME/{} - | zcat | jq ".timestamp[0:10]" | sort | uniq -c'
secor-test-9/athena-integration-1/dt=2017-01-31/1_22_00000000000002657819.gz
19921 "2017-01-31"
2 "2017-02-01"
secor-test-9/athena-integration-1/dt=2017-02-01/1_0_00000000000002675808.gz
19302 "2017-02-01"
3 "2017-02-02"
1 "2017-02-03"
secor-test-9/athena-integration-1/dt=2017-02-01/1_11_00000000000002717065.gz
2 "2017-02-02"
2 "2017-02-03"
5 "2017-02-04"
3 "2017-02-05"
2 "2017-02-06"
secor-test-9/athena-integration-1/dt=2017-02-01/1_12_00000000000002655329.gz
3 "2017-01-31"
33460 "2017-02-01"
1 "2017-02-02"
secor-test-9/athena-integration-1/dt=2017-02-04/1_3_00000000000002720791.gz
2 "2017-02-02"
12869 "2017-02-04"
For one specific example above secor-test-9/athena-
integration-1/dt=2017-02-01/1_11_00000000000002717065.gz the timestamps
of each line are
"2017-02-02T22:33:03.12533900Z"
"2017-02-02T22:46:12.26030700Z"
"2017-02-03T00:49:04.27147800Z"
"2017-02-03T05:21:52.79127500Z"
"2017-02-04T00:12:43.81382200Z"
"2017-02-04T04:39:16.33926200Z"
"2017-02-04T04:41:33.62567100Z"
"2017-02-04T05:21:55.08586200Z"
"2017-02-04T07:39:26.89454900Z"
"2017-02-05T07:13:58.89362700Z"
"2017-02-05T07:59:05.27761600Z"
"2017-02-05T23:16:37.77798300Z"
"2017-02-06T19:20:29.98021100Z"
"2017-02-06T19:44:06.95325400Z"
Note that these nanosecond timestamps are handled correctly though for the
vast majority (see: tens of thousands of correctly partitioned events).
My normalised (comments removed etc.) config file is:
aws.proxy.http.port=
aws.proxy.isEnabled=false
aws.region=
aws.role=
aws.secret.key=
aws.sse.customer.key=
aws.sse.kms.key=
aws.sse.type=
cloud.service=S3
hive.table.prefix=
kafka.consumer.auto.offset.reset=smallestkafka.consumer.timeout.ms=10000
kafka.dual.commit.enabled=false
kafka.fetch.message.max.bytes=
kafka.fetch.min.bytes=kafka.fetch.wait.max.ms=
kafka.offsets.storage=zookeeper
kafka.partition.assignment.strategy=rangekafka.rebalance.backoff.ms=
kafka.rebalance.max.retries=
kafka.seed.broker.host=...
kafka.seed.broker.port=9092
kafka.socket.receive.buffer.bytes=
kafka.zookeeper.path=/kafka-kafkamessage.timestamp.id=1
message.timestamp.input.pattern=message.timestamp.name.separator=message.timestamp.name=timestamp
message.timestamp.required=true
message.timestamp.type=i64
monitoring.blacklist.topics=
monitoring.prefix=secor
ostrich.port=9999
partitioner.finalizer.delay.seconds=3600
partitioner.granularity.hour=true
partitioner.granularity.minute=false
qubole.api.token=
secor.compression.codec=org.apache.hadoop.io.compress.GzipCodec
secor.consumer.threads=7
secor.enable.qubole=true
secor.file.extension=
secor.file.reader.writer.factory=com.pinterest.secor.io.impl.DelimitedTextFileReaderWriterFactory
secor.generation=1
secor.kafka.group=secor-test-9
secor.kafka.topic_blacklist=
secor.kafka.topic_filter=^athena-((?!production).)*$
secor.kafka.upload_at_minute_mark.topic_filter=
secor.local.log.delete.age.hours=-1
secor.local.path=/tmp/secor-test-9
secor.max.file.age.seconds=120
secor.max.file.size.bytes=20000000
secor.max.message.size.bytes=100000
secor.message.parser.class=com.pinterest.secor.parser.Iso8601MessageParser
secor.message.transformer.class=com.pinterest.secor.transformer.IdentityMessageTransformer
secor.messages.per.second=100000
secor.offsets.per.partition=10000000
secor.offsets.prefix=offset=
secor.parser.timezone=UTCsecor.qubole.timeout.ms=300000
secor.s3.alter.path.date=
secor.s3.alternative.path=
secor.s3.bucket=...
secor.s3.filesystem=s3a
secor.s3.path=secor-test-9
secor.s3.prefix.md5hash=false
secor.topic_partition.forget.seconds=600
secor.upload.manager.class=com.pinterest.secor.uploader.S3UploadManager
secor.upload.minute_mark=0
secor.zookeeper.path=/secor-test-9
statsd.hostport=
tsdb.hostport=
zookeeper.quorum=...zookeeper.session.timeout.ms=3000zookeeper.sync.time.ms=200
This could be a bug, or it could be a misconfiguration on my behalf, but
I've played around with this config for the better part of a day using the
available documentation and can't improve it beyond this. Hourly
granularity doesn't seem to work either in this setup, but that's another
issue I suppose.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/305, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_xPwabfcs1ZOL2eLYX1oU3FMB_Dmks5ra7fdgaJpZM4L832T
.

While I know the syntax, I'm not familiar enough with the Java ecosystem to tell if it can or not, based on the libraries it's using.

You can write a small java program using the code in Iso860MessageParser to
test your json.
Or add your json with different date formats in Iso8601ParserTest.java to
run the test.
On Thu, Feb 9, 2017 at 6:23 PM, gwilym<EMAIL_ADDRESS>wrote:

While I know the syntax, I'm not familiar enough with the Java ecosystem
to tell if it can or not, based on the libraries it's using.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/305#issuecomment-278839873,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCX_9Sliu59UVzrJZ3k58Dckol1T8hZks5ra8oSgaJpZM4L832T
.

@HenryCaiHaiying I've added the test case in #306 -- the time formats appear to be supported by the parser itself

I guess github interprets "does not fix #n" as "fix #n". @HenryCaiHaiying would you mind re-opening this?

fixed using #307 -- see config sample there


jax-metal: count leading zeros not supported

Description
def f(x):
  return jax.lax.clz(x)

x = jnp.array([0, 1 << 10, 1 << 20])

# Print lowered HLO
print(jax.jit(f).lower(x).as_text())
print(jax.jit(f)(x))

HLO
module @jit_f attributes {mhlo.num_partitions = 1 : i32, mhlo.num_replicas = 1 : i32} {
  func.func public @main(%arg0: tensor<3xi32> {mhlo.layout_mode = "default"}) -> (tensor<3xi32> {jax.result_info = "", mhlo.layout_mode = "default"}) {
    %0 = stablehlo.count_leading_zeros %arg0 : tensor<3xi32>
    return %0 : tensor<3xi32>
  }
}

fails with
Traceback (most recent call last):
  File "/Users/jonatanklosko/tmp/jax_mlir.py", line 71, in <module>
    print(jax.jit(f)(x))
jaxlib.xla_extension.XlaRuntimeError: UNKNOWN: /Users/jonatanklosko/tmp/jax_mlir.py:63:0: error: failed to legalize operation 'mhlo.reduce_window'
/Users/jonatanklosko/tmp/jax_mlir.py:70:0: note: called from
/Users/jonatanklosko/tmp/jax_mlir.py:63:0: note: see current operation:
%2 = "mhlo.reduce_window"(%arg0, %1) ({
^bb0(%arg1: tensor<si32>, %arg2: tensor<si32>):
  %3 = "mhlo.add"(%arg1, %arg2) : (tensor<si32>, tensor<si32>) -> tensor<si32>
  "mhlo.return"(%3) : (tensor<si32>) -> ()
}) {base_dilations = dense<1> : tensor<1xi64>, padding = dense<1> : tensor<1x2xi64>, window_dilations = dense<1> : tensor<1xi64>, window_dimensions = dense<2> : tensor<1xi64>, window_strides = dense<1> : tensor<1xi64>} : (tensor<3xsi32>, tensor<si32>) -> tensor<4xsi32>

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.26
jaxlib: 0.4.26
numpy:  1.26.4
python: 3.10.8 (main, Nov 16 2022, 12:45:33) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
jax.devices (1 total, 1 local): [METAL(id=0)]
process_count: 1
platform: uname_result(system='Darwin', node='chonker', release='23.5.0', version='Darwin Kernel Version 23.5.0: Wed May  1 20:12:58 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6000', machine='arm64')

jax-metal 0.0.7

Same with jax.lax.population_count and jax.lax.cbrt, but all three are not extremely widespread, so I'm just going to mention them on the same issue.

If there is a list of operations that are inherently not supported for jax-metal, it would be very helpful to have the list documented somewhere :)


MIT 6.0001 (1/2) is now MIT 6.100 (A/B) for Spring 2023 onwards and the syllabus has changed a bit

More like an FYI to the curriculum maintainers here (rather than a specific issue).
As the issue title says, MIT course 6 (EECS) does not offer 6.0001 (1) or 6.0001 (2) anymore. The course name has changed to 6.100 A & B. The material has been updated as well to teach somewhat different content.
The course name also reflects the revised focused on data science as opposed to programming. The new course name is Introduction to Computational Thinking and Data Science (vs the old name of Introduction to Computer Science and Programming using Python).
Relevant links
http://catalog.mit.edu/subjects/6/
https://introcomp.mit.edu/spring23
Does this have any implication on OSSU's Intro CS recommendation?

Hi, as a not MIT student how can I view the lessons?

On YouTube
On Mon, Apr 3, 2023 at 6:49 AM codeninka @.***> wrote:

Hi, as a not MIT student how can I view the lessons?
—
Reply to this email directly, view it on GitHub
https://github.com/ossu/computer-science/issues/1148#issuecomment-1494171774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKNLKTHXTPCTY7KW6KHAJB3W7K2LTANCNFSM6AAAAAAWRD4CAE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--

Apologies. It looks like the exact replacement for the former 6.0001 is actually 6.100L  (and not 6.100A/B). https://introcomp.mit.edu/6.100L_sp23

Does this have any implication on OSSU's Intro CS recommendation?

As (nearly) none of us will be taking the courses in-person at MIT, I'm not certain this drastically changes things. We're still going to recommend the free and open versions in MIT OCW, or (for a small upgrade fee) the QoL upgrades available in EdX 6.00.1x/2x.
What this MAY do for us is give us another source of material for a future refresh of the course. Might be worth keeping an eye on, especially if homework gets publicly released and the course is updated to a more modern version of Python.

Apologies. It looks like the exact replacement for the former 6.0001 is actually 6.100L (and not 6.100A/B). https://introcomp.mit.edu/6.100L_sp23.
6.100L retains the focus on python

Looks like 6.100L is just 6.100A dragged out over twice the length of a term. Seems there's some complaint that the course moves too fast, even for the likes of the MIT demigods.

pologies. It looks like the exact replacement for the former 6.0001 is actually 6.100L (and not 6.100A/B).

I think you should update this issue's title as well.

I'd say that @bradleygrant has answered the question posed. As there is no further call to action, I'm marking this as closed.


Breaking changes in Coq 8.5~beta3

The package coq:ext-lib.1.0.0~beta2 does not compile anymore with Coq 8.5~beta3. See bench error. I updated the dependencies in ebb5b6b830a0addb21b9c2ad4f5afd98cf12d9e6.

Thanks.
There isn't an official beta3 tarball yet. Do I just build it against the 8.5 branch to see these errors?

The website of Coq is not updated yet, but there is an OPAM package coq.8.5~beta3 (in the released repository, will soon be moved to core-dev). The tarballs will arrive by a few days, I guess. Else the 8.5 branch should be pretty close to the beta3.

This is fixed now with a new release.


case sensitivity issue on highscores page

good stuff guys.
Found a nit on highscore page:
3	golang/crypto	185	86.49
44	Golang/crypto	185	86.49

This is still an issue. It could be as simple as lower-casing all go get queries?

@hermanschaaf that could work probably, and we'd also want to remove the duplicates from bolt

We're doing a lowercase string comparison when inserting into high scores now, and if an entry already exists we remove it rather than inserting twice.


Cannot find installation of ffmpeg

Cannot find installation of ffmpeg

I am stuck on this issue .. any idea how to resolve this.


Email input is typed as 'mail' instead of 'email'

<input type="mail"> is not a valid HTML input type. Should be <input type="email"> instead

:tada: This issue has been resolved in version 1.4.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


(PUP-11597) Generate types when any module libs are updated

Fixes PUP-11597
Before
Prior to this, the puppet generate types command would only detect resource type changes if the type file itself changed.
For example, if your type was defined at <module>/lib/puppet/type/foo.rb, a change in it would be detected.
But if your type includes libraries from separate Ruby files in lib/puppet/* or even in the PuppetX namespace like lib/puppet_x/*, changes to those other files would not be detected.
The consequence of that behavior is that updates to the signature of a type that come from "other" ruby files would not trigger a metadata re-creation and those types would suffer from the environment-isolation problem (SERVER-94) that puppet generate types was meant to solve.
After
This fixes that problem by updating puppet generate types to scan for changed Ruby files within a module's lib/* directory.
It uses the same Puppet::FileSystem::stat operation to compare mtime of the Ruby files in lib/* relative to the generated metadata file. And if the metadata is older than any of the Ruby files in lib/*, the metadata is regenerated.
The one slight-downside to my solution is that generation of type metadata will possibly happen more often than before and possibly more often than is needed. However, generating types is fast, and it's usually only done when code updates happen (e.g. via r10k or Code Manager), so the total added load is very close to, if not, negligible.

Can one of the admins verify this patch?

I'm not sure why the spec tests are failing in jruby and windows. Anyone have an idea why?

Since CI failed more than a month ago, I have to close and reopen to rekick the GH actions.

@natemccurdy I think Windows tests are failing because of the way the test stubs filesystem access. Perhaps Windows mtime resolution isn't as low as Posix or the time is being truncated?
  1) Puppet::Face[:generate, 0.1.0] when used from an interactive terminal in an environment with two modules containing resource types overwrites if files exists that are not up to date while keeping up to date files
     Failure/Error: expect(stats_before[0] <=> stats_after[0]).to eq(-1)
       expected: -1
            got: 0
       (compared using ==)
```.

I wasn't able to figure out a way to get the tests to pass on Windows by faking a passage of time, so I resorted to a sleep(1) which is what the existing tests did.
I don't that sleep, but it passes the tests.

@joshcooper Is there anything specific here that still needs work?

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

Retriggering tests


Twitter preview of NFT not working (again)

What happened?
This NFt link: https://kodadot.xyz/rmrk/gallery/17475536-62DB09E2F7FAA81C5F-FLOW-THROW_UP-0000000000000007?page=1
This link on my Twitter:
https://twitter.com/Just_Luuuu/status/1648995352786108417

Same in chat:

It should be working on Opengraph: https://www.opengraph.xyz/url/https%3A%2F%2Fkodadot.xyz%2Frmrk%2Fgallery%2F17475536-62DB09E2F7FAA81C5F-FLOW-THROW_UP-0000000000000007
Is this a Twitter issue? Or what am I'm doing wrong?
Please reproduce in steps
Share NFt on twitter
Expected Behavior
show preview
What browsers are you seeing the problem on?
Desktop Chrome
At which address did you encounter bug?
No response
Are you logged in?
None
Which wallet you are using?
No response
At which chain did you encounter bug?
No response
Screenshots

Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

[X] I agree to follow this project's Code of Conduct

hihi, it worked, maybe something wrong/different with production.
@preschian can we deploy recent ssr-opengraph-beta to production one?

#5567

Also I've noticed I can't open that link on canary in incognito, however I can navigate to that URL through collection

https://canary.kodadot.xyz/rmrk/gallery/17475536-62DB09E2F7FAA81C5F-FLOW-THROW_UP-0000000000000007

On beta it works apparently and doesn't work, suspicious
https://beta.kodadot.xyz/rmrk/gallery/17475536-62DB09E2F7FAA81C5F-FLOW-THROW_UP-0000000000000007

Probably that's why OG explorer shoots 500 error? @preschian

oh, CF might had outage..

https://github.com/kodadot/nft-gallery/issues/5739

looks like its working again


Work in progress in readding polyglot support

In order to merge this we need to be able to make polyglot actually detect sentiments. It currently crashes in a division by zero, but the actual reason is that all the words sentiment is getting detected as 0.
I can't download the models either (possible problem) since I get a timeout :(

Looking into this issue a bit more, it seems like it may be an issue on Polyglot's end - I think their method for determining sentiment might just not be particularly sophisticated or robust.
I tried the following script outside of the Objection Engine:
from polyglot.text import Text
comments = [
    "This is a good happy sentence!
]

for i in comments:
    try:
        polarity = Text(i).polarity
    except ZeroDivisionError:
        polarity = 0
    print(f'"{i}": {polarity}')

The results were:
"This is a good happy sentence! Yay haha woohoo!": 1.0
"I hate you so much bad evil angry.": -1.0
"This sentence is neutral": 0
"¡Me encantan los perritos pequeños y te quiero mucho!": -1.0
"Espero que tienes dolor y muertas.": -1.0
"Este oración es neutral.": 0

Overall, it didn't seem to do poorly here, although the Spanish sentence intended to be positive received a negative score. The division-by-zero error is definitely a problem with how the Polyglot code calculates the overall score of a sentence.
I'm unfortunately not sure how it would be best to proceed with this - it seems like it'd require a lot of work on the Polyglot side. Depending on how the bots are using the Hugging Face library, there might be ways to modify them so that they use it more efficiently and don't take up as much RAM.

Using polyglot is a solution rather than a requirement itself. If we can find another way for the bots to use less RAM that would be acceptable as well

Is the current bot code public? I found this from a few months ago: https://github.com/LuisMayo/Objection-Engine-Rabbit-Worker/blob/master/main.py#L11
Each call to render_comment_list() will create a new instance of the Hugging Face model, which is probably where the RAM usage happens. Since the model shouldn't be changing across render requests, we can instead create one analyzer or DialogueBoxBuilder when the bot starts up, and then reuse it (pass it new arguments to render()) each time we want to render a new video.
Here's some pseudocode based off of my fork of this branch:
# Only run this stuff once on bot startup, so that
# these parts are not being recreated and garbage collected
# every time a new thread is rendered
sentiment_analyzer = SentimentAnalyzer()
if len(getenv("oe_bypass_sentiment", "")) <= 0:
    model_setting = getenv("oe_sentiment_model", "hf")
if model_setting == "hf":
    sentiment_analyzer = HuggingFaceAnalyzer()
elif model_setting == "pg":
    sentiment_analyzer = PolyglotAnalyzer()

builder = DialogueBoxBuilder(
    callbacks=callbacks, sentiment_analyzer=sentiment_analyzer
)

# Call this function every time we want to render a thread
# (note that the input arguments are incomplete for brevity's sake)
def render_comment_list(comment_list, output_filename, ...):
    builder.render(
        comment_list,
        output_filename=output_filename,
        music_code=music_code.lower(),
        assigned_characters=assigned_characters,
        adult_mode=adult_mode,
        avoid_spoiler_sprites=avoid_spoiler_sprites,
        resolution_scale=resolution_scale,
    )

The bot code is public.
I like your idea, but we still have the problem than a single instance of the model is already taking a lot of RAM. Your idea would greatly improve renders per time unit though
Maybe when Twitter bot closes this will get alleviated and stops being a problem

How much RAM does the server have available/what would be an acceptable level of RAM usage?
I did a quick test where I rendered the same video 10 times in a row with the same sentiment analyzer, for both Hugging Face and Polyglot.

Polyglot used 0.43 GB of RAM
Hugging Face used 2.45 GB of RAM

So the Hugging Face model definitely uses a lot more than Polyglot. We might be able to find a different model on Hugging Face that is smaller but still works, but it doesn't look like there are many options: https://huggingface.co/models?language=en,fr,es&sort=downloads&search=sentiment
The kinda-good news here is that I got identical results when only rendering the video once, so I'm more confident now that memory usage will not scale with the number of videos rendered, as long as the API is used correctly.


Update cursor styling, typo, startup timing and dependency updates

I’ve decided to close this pull request because it seems that your intentions for the extension were different. For now, I’ll be creating a fork for the changes I wanted to introduce:

Highlight after 2 letters, sometimes I was not able to navigate because the trigger was done after the 2nd letter (Doing this also removes most MAYUS jumps since you get less matches, that also helps you to be more focus in that specific word).
Letter matches now are more focus for the right hand since the initializer letter is s, so is faster to reach.
Decoration background like cursor block shape (no border radius).

If you’re open to implementing a more leap.nvim-like behavior for the extension in the future, I’d be happy to learn more and collaborate on that. Thanks for your work!

@darianmorat I'm sorry to hear that this extension wasn't suiting your needs. I don't see this extension supporting Leap-like behaviour anytime soon as that would require a rewrite of the internals to support which I don't have time for at the moment.
However the suggestionI don't think a full fork is necessary if your goal is for the extension to support your needs:

Matches triggering after a second letter: This seems like a bug, are you able to provide a minimal reproduction of this behaviour that I could look at?
Prioritising left hand matches: This one is tricky because we need to be mind of which keyboard layout someone is using (e.g. qwerty/qwertz/dvorak/colemak etc). We could expose a setting that allows configuration of valid jump characters & prioritize them left to right. Thoughts?
Decoration background: This is easy enough, we can expose new theme settings for this.

Sure!

Matches triggering after a second letter: This happens in specific scenarios, which I don't really encounter that often, but when it happens it's quite stressful... I got a video for you, so u can better understand what I mean: I want to go to await but when I type a one of the jump chars is w so I cannot get there...

https://github.com/user-attachments/assets/ffd138d8-3f11-428f-806b-65267b510418
For that reason I mentioned the Jump chars after 2 letters, so we can immediately discard all the extra words, and get there faster without the fear of triggering that jump, so first check 2 letters, and then jump...

Prioritizing left hand matches: That's a good idea, for example this config in a qwerty gives you most matches in your right hand, and always the j is the first match if just 1 match which I found extremely helpful since the activation for findthenjump is s for me, so pressing s and then j is really good: (also is worth noticing something here, IF YOU GET JUST 1 MATCH the first letter in this array would be the jump char... Making an statement so IF matches === 1, send a as the input, which would jump automatically).

[...'jifokalwersdqmnvhut;cxzgbpy/JIFOKALWERSDQMNVHUTGCXZBPY']

Decoration background: You are right, a setting for changing the border radius would be a good option, since everyone got different taste.


Log swallowed Exception

Prepares #2349

@krusche I found a scond Exception that is not fully logged that would be super important

Oh that got merged as well never mind


Meaning of STUN transaction failed (400 - You cannot use the same channel number with different peer)

Hi!
I'm occasionally getting a bunch of error messages in my aiortc based application that I don't understand.
[webrtc_bridge-1] INFO:aioice.turn:TURN channel bound 16385 ('<IP_ADDRESS>', 33340)
[webrtc_bridge-1] ERROR:asyncio:Task exception was never retrieved
[webrtc_bridge-1] future: <Task finished name='Task-1372' coro=<TurnClientMixin.send_data() done, defined at /home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py:241> exception=TransactionFailed(Message(message_method=Method.CHANNEL_BIND, message_class=Class.ERROR, transaction_id=b'\xd1\x9a>\xc5\xfe\xc2\x15e\xd8\x7f\x0b!'))>
[webrtc_bridge-1] Traceback (most recent call last):
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 251, in send_data
[webrtc_bridge-1]     await self.channel_bind(channel, addr)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 85, in channel_bind
[webrtc_bridge-1]     await self.request_with_retry(request)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 213, in request_with_retry
[webrtc_bridge-1]     response, addr = await self.request(request)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 200, in request
[webrtc_bridge-1]     return await transaction.run()
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/stun.py", line 299, in run
[webrtc_bridge-1]     return await self.__future
[webrtc_bridge-1] aioice.stun.TransactionFailed: STUN transaction failed (400 - You cannot use the same channel number with different peer)
[webrtc_bridge-1] INFO:aioice.ice:Connection(2) Check CandidatePair(('<IP_ADDRESS>', 35871) -> ('<REPLACED_IP_FOR_CUSTOM_TURN_SERVER>, 58892)) State.IN_PROGRESS -> State.SUCCEEDED
[webrtc_bridge-1] INFO:aioice.ice:Connection(2) ICE completed
[webrtc_bridge-1] ERROR:asyncio:Task exception was never retrieved
[webrtc_bridge-1] future: <Task finished name='Task-1376' coro=<TurnClientMixin.send_data() done, defined at /home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py:241> exception=TransactionFailed(Message(message_method=Method.CHANNEL_BIND, message_class=Class.ERROR, transaction_id=b'2 H\xc6\xb60\rm\x10\x13\x06\x83'))>
[webrtc_bridge-1] Traceback (most recent call last):
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 251, in send_data
[webrtc_bridge-1]     await self.channel_bind(channel, addr)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 85, in channel_bind
[webrtc_bridge-1]     await self.request_with_retry(request)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 213, in request_with_retry
[webrtc_bridge-1]     response, addr = await self.request(request)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 200, in request
[webrtc_bridge-1]     return await transaction.run()
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/stun.py", line 299, in run
[webrtc_bridge-1]     return await self.__future
[webrtc_bridge-1] aioice.stun.TransactionFailed: STUN transaction failed (400 - You cannot use the same channel number with different peer)
[webrtc_bridge-1] INFO:aioice.ice:Connection(0) Consent to send expired
[webrtc_bridge-1] INFO:aioice.turn:TURN allocation deleted ('<REPLACED_IP_FOR_CUSTOM_TURN_SERVER>', 59795)
[webrtc_bridge-1] INFO:aioice.turn:TURN allocation deleted ('<REPLACED_IP_FOR_CUSTOM_TURN_SERVER>', 59795)
[webrtc_bridge-1] ERROR:asyncio:Task exception was never retrieved
[webrtc_bridge-1] future: <Task finished name='Task-2961' coro=<TurnClientMixin.delete() done, defined at /home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py:144> exception=InvalidStateError('invalid state')>
[webrtc_bridge-1] Traceback (most recent call last):
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/turn.py", line 164, in delete
[webrtc_bridge-1]     self.receiver.connection_lost(None)
[webrtc_bridge-1]   File "/home/dyno/.local/lib/python3.8/site-packages/aioice/ice.py", line 174, in connection_lost
[webrtc_bridge-1]     self.__c[webrtc_bridge-1]

I'm mostly concerned about the log: STUN transaction failed (400 - You cannot use the same channel number with different peer).
Both sending video and data over WebRTC seem to work for the most part. Video and data transmission does freeze for a couple of seconds from time to time before it starts working again though. Don't know if that is related.
Is this something I should worry about, or can I safely ignore it?

I have the same issue, but in my case it is critical and often completely prevents the new peers from connecting.
I think the issue is that the channel numbers used by client is always the same:
self.channel_number = 0x4000     (line 65 in aioice/src/aioice/turn.py)
A quick fix I use is to substitute that line with
self.channel_number = random.randint(0x4000, 0x4fff)
Maybe this could be implemented in the project as a commit?

I don't understand this issue, unless I'm mistaken I don't see how the channel number is being reused by a given connection, it should be increased for every peer.. unless there is a race condition in send_data, as we only increase the channel number after binding completes?
@Bob10918 : no to randomising the channel number, this solves nothing and leads to non-deterministic behaviour

ERROR:asyncio:Task exception was never retrieved
future: <Task finished coro=<TurnClientMixin.send_data() done, defined at /home/shreehari/.local/lib/python3.6/site-packages/aioice/turn.py:241> exception=TransactionFailed(Message(message_method=Method.CHANNEL_BIND, message_class=Class.ERROR, transaction_id=b'\xe7i\xe5\xb0\xf5\xf9\x9a,\xbf1\x1c\xd9'),)>
Traceback (most recent call last):
  File "/home/shreehari/.local/lib/python3.6/site-packages/aioice/turn.py", line 251, in send_data
    await self.channel_bind(channel, addr)
  File "/home/shreehari/.local/lib/python3.6/site-packages/aioice/turn.py", line 85, in channel_bind
    await self.request_with_retry(request)
  File "/home/shreehari/.local/lib/python3.6/site-packages/aioice/turn.py", line 213, in request_with_retry
    response, addr = await self.request(request)
  File "/home/shreehari/.local/lib/python3.6/site-packages/aioice/turn.py", line 200, in request
    return await transaction.run()
  File "/home/shreehari/.local/lib/python3.6/site-packages/aioice/stun.py", line 299, in run
    return await self.__future
aioice.stun.TransactionFailed: STUN transaction failed (400 - You cannot use the same channel number with different peer)
DEBUG:aioice.ice:Connection(0) protocol(1) > ('<IP_ADDRESS>', 59898) Message(message_method=Method.BINDING, message_class=Class.REQUEST, transaction_id=b'\\\x06\x9a\xb1\x06/\x1bT\x90\xc9m\xa8')

This is bothering me as well. This appears when stun fails to find the candidates and hands the control over to turn. Maybe it has to do something while switching from stun mode to turn mode? would like to hear from you @jlaine

I can confirm that during channel binding there is a race condition which can lead to:

using the same channel number for different peers
sending multiple channel bind requests for a single peer

I've put together PR #52 which should solve the issue. Ideally I'd like to make the code a little cleaner, but it should still fix the situation.


Service Failed to start - InSufficient privileges

When I pass the arguments of iis or os, to execute the wmi_exporter-0.2.3-386.msi it works well, but when I pass the args as "process" it gives insufficient privileges error - Verify that you have sufficient privileges to start system services.
Please advise what is getting missed out (or) what should be the special privilege for the userid.

Are you running the service as a non-default user?

Also, the process collector was added in 0.2.6, so are you using the correct version there?

Am running the service as a Admin user.

0.2.6 worked out .. Thanks a lot, Carl ..


Refactor and Standardize API Endpoints + Use Resource

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/714
Originally created by @derek-knox on 2019-06-14 19:47:05

Standardize python use of Resource and update api endpoints for consistency

View 4 previous comments from the original issue on GitLab


Invalid unicode string from file dialogue on non-Windows system

When using the save file dialogue on non-Windows systems, any non-ASCII characters in the file name result in an invalid string. This is because the string is returned as a wstring: https://github.com/openfl/lime/blob/develop/project/src/ExternalInterface.cpp#L902
While this works fine on Windows, this is problematic on Linux where wstring is not standard. As a result, if the string ends up being passed to hxcpp's system functions, it is considered invalid and we end up with confusing error messages (like Allocating from a GC-free thread which comes from a failed allocation of an exception string to describe this issue).
It's possible to workaround this issue by adding an extra conversion back to a regular string: https://github.com/tobil4sk/lime/tree/fix/unix-unicode-file
However, it would probably be better to refactor the api a bit to remove the need for wstring on non Windows systems altogether, because right now there are redundant conversions between string types just to fit this api:
Conversion to wstring:
https://github.com/openfl/lime/blob/d75b9f9cfaafbca58cafadeaf91475ea36362c8a/project/src/ExternalInterface.cpp#L748-L750
Conversion from wstring to string:
https://github.com/openfl/lime/blob/d75b9f9cfaafbca58cafadeaf91475ea36362c8a/project/src/ui/FileDialog.cpp#L108-L110

It looks like the 8.2.0-Dev branch has updated tinyfiledialogs to a newer version, and tinyfiledialogs has switched the default char encoding on Windows to UTF-8, instead of MBCS. If I'm understanding things correctly, it means that we shouldn't need to use the std::wstring APIs on Windows anymore, and Lime's FileDialog can just use std::string everywhere.

Interestingly, with the following code and Lime 8.0.1, I tried opening a file named åéîøü.txt, and it outputs an incorrect string on Windows!,
var dialog = new FileDialog();
dialog.onSelect.add(value -> {
	trace("select file: " + value);
});
dialog.browse(OPEN);

src/Main.hx:26: open file: C:\Users\josht\Desktop\├Ñ├⌐├«├╕├╝.txt

It seems like, if we were using std::wstring to support Unicode on Windows, it was never actually working, or it broke at some point.

This is just a problem with the trace output on Windows hxcpp, the same thing happens if you manually do trace("åéîøü.txt");:

Good to know, thanks! I guess I need to render these strings inside Lime/OpenFL instead, to verify that they're correct.

I got std::string working on both Windows and macOS for the cpp target, but it was causing weird freezes in HashLink on Windows (but fine on macOS). I don't have any further time to debug that, so I decided to stick with std::wstring for now. I simply fixed the conversion of std::wstring to std::string on non-Windows.

Nice, thanks. Looks like some builds are failing though, because the <cstring> header has not been included.
When you have a spare moment, could you upload the std::string version somewhere? I'm interested in what could be causing the freezes.

Just adding some debugging code that I used, just in case I need to refer back to it in the future:
var debugTF = new TextField();
debugTF.x = stage.stageWidth / 2.0;
debugTF.width = stage.stageWidth / 2.0;
debugTF.height = stage.stageHeight;
debugTF.wordWrap = true;
addChild(debugTF);
function log(...str:String):Void {
	debugTF.appendText(str.toArray().join(" ") + "\n");
	debugTF.scrollV = debugTF.maxScrollV;
}

var openFile = new TextField();
openFile.autoSize = LEFT;
openFile.text = "Open File";
openFile.selectable = false;
openFile.background = true;
openFile.backgroundColor = 0xcccccc;
openFile.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onSelect.add(value -> {
		log("open file: " + value);
		log(sys.io.File.getContent(value));
	});
	dialog.browse(OPEN, null, null, "Open File (åéîøü)");
});
openFile.x = 10;
openFile.y = 10;
addChild(openFile);

var saveFile = new TextField();
saveFile.autoSize = LEFT;
saveFile.text = "Save File";
saveFile.selectable = false;
saveFile.background = true;
saveFile.backgroundColor = 0xcccccc;
saveFile.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onSelect.add(value -> {
		log("save file: " + value);
	});
	dialog.browse(SAVE, null, "/Users/joshtynjala/Desktop/åéîøü.txt", "Save File (åéîøü)");
});
saveFile.x = 10;
saveFile.y = 60;
addChild(saveFile);

var openDir = new TextField();
openDir.autoSize = LEFT;
openDir.text = "Open Directory";
openDir.selectable = false;
openDir.background = true;
openDir.backgroundColor = 0xcccccc;
openDir.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onSelect.add(value -> {
		log("open dir: " + value);
	});
	dialog.browse(OPEN_DIRECTORY, null, null, "Open Dir (åéîøü)");
});
openDir.x = 10;
openDir.y = 110;
addChild(openDir);

var openMult = new TextField();
openMult.autoSize = LEFT;
openMult.text = "Open Multiple";
openMult.selectable = false;
openMult.background = true;
openMult.backgroundColor = 0xcccccc;
openMult.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onSelectMultiple.add(value -> {
		log("open mult: " + value);
	});
	dialog.browse(OPEN_MULTIPLE, null, null, "Open Mult (åéîøü)");
});
openMult.x = 10;
openMult.y = 160;
addChild(openMult);

var openFile2 = new TextField();
openFile2.autoSize = LEFT;
openFile2.text = "Open File 2";
openFile2.selectable = false;
openFile2.background = true;
openFile2.backgroundColor = 0xcccccc;
openFile2.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onOpen.add(value -> {
		log("open 2: " + value);
	});
	dialog.open(null, null, "Open File 2 (åéîøü)");
});
openFile2.x = 10;
openFile2.y = 210;
addChild(openFile2);

var saveFile2 = new TextField();
saveFile2.autoSize = LEFT;
saveFile2.text = "Save File 2";
saveFile2.selectable = false;
saveFile2.background = true;
saveFile2.backgroundColor = 0xcccccc;
saveFile2.addEventListener(MouseEvent.CLICK, event -> {
	var dialog = new FileDialog();
	dialog.onSave.add(value -> {
		log("save 2: " + value);
	});
	dialog.save("åéîøü", null, "/Users/joshtynjala/Desktop/åéîøü.txt", "Save File 2 (åéîøü)");
});
saveFile2.x = 10;
saveFile2.y = 260;
addChild(saveFile2);

I've been looking back at this, it looks like maybe the issue here is actually not on lime's side...
Hxcpp's string encoding should be the same regardless of the OS, so I was struggling to understand why we needed have different behaviour for Linux/MacOS and Windows. alloc_wstring() should in theory result in a valid hxcpp string regardless of the platform. It looks like the real problem is that the String::create(wchar_t*) function in hxcpp doesn't take into account that wchat_t has a different length on platforms other than windows, and so it has been creating invalid strings. Up until the String::create(wchar_t*) call, the strings are all valid.
Using alloc_string() forces it to use String::create(char*), which does not suffer from this issue, so really this might be a workaround rather than a proper fix.
I think this also means that wstring_to_vbytes on hashlink can be the same on all platforms? Since hashlink doesn't suffer from this issue, and from my testing was working fine before the patch.


update deps for netfoundry/ziti-eddge#158

fixes for identity update
migration to fix orphaned enrollments/authenticators

If you are looking at this - I created this because I believe I found the problem w/ the other branch that contains the same changes (use a #158 instead of a -158). I think smoke is failing on the #


chore: add new fact

Closes #4

pls resolve the merge conflicts. Rest is good

merging. Good job :D


Thread safety support and deadlock supression

While building the CI in PR #31 a deadlock error occurred randomly.
It happened either while running cargo test or with cargo tarpaulin that triggers cargo test for creating coverage.
The backtrace is:
---- tests::params::linking::ints stdout ----
Error: FbError { msg: "deadlock\nupdate conflicts with concurrent update\nconcurrent transaction number is 123", code: -913 }
thread 'tests::params::linking::ints' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/d3fb005a39e62501b8b0b356166e515ae24e2e54/src/libstd/macros.rs:16:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    tests::params::linking::ints

I can easily reproduce the problem in my local computer by running:
cargo tarpaulin --ignore-config --out Lcov Xml Html --run-types AllTargets -v --line --count --branch --features 'linking date_time pool'

However, the error goes away when restricting parallel test execution to 1 thread by adding --test-threads 1 to cargo test throught cargo tarpaulin:
cargo tarpaulin --ignore-config --out Lcov Xml Html --run-types AllTargets -v --line --count --branch --features 'linking date_time pool' -- --test-threads 1

Also I saw in the Github Action log that simply using cargo test can trigger the bug:
cargo test --features dynamic_loading date_time pool

I have concluded that is a deadlock bug because:

The error message says deadlock\nupdate conflicts with concurrent update
The fbclient wasn't thread safe sometime ago. I don't know the current status.
The error happened radomnly in anyone of test cases and changed every time command was run.

I don't know what is the resolution that should be taken in this context. Maybe:

this should be fixed because just is a test case bug.
this should be fixed because just is a driver bug.
this should be addressed with some extra checking for handling thread safety.
this should not be handled by the driver, because is a normal database concurrency issue.
the test cases should run with concurrency = 1

This error is occurring when a test tries to create / drop the same table at the same time as another one.
I've added a atomic counter to the tests in the connection module, but it seems i've missed the need to add the same in the tests::params module.
This is only a problem when running the tests of multiple implementations at the same time. The dynamic_linking tests in the ci are actually running the linking tests too because the linking feature is in the default feature set. This can be fixed by adding --no-default-features to the tests while i fix the deadlock to allow testing all features at the same time.

Great!
Just some considerations:

each job instance executes in a separated virtual environment provided by a GitHub-hosted runner in a virtual machine hosted by GitHub.
So each job instance has it's exclusive FirebirdSQL database running in a container inside a virtual machine.
However I don't know if this matters for masking the problem you explained above.
I think --no-default-features may not work because --features is given to cargo test. Anyway ``-- --test-threads 1` should work.

For solving this we can temporarily workaround cargo test as you suggested and revert latter after the test cases are improved.

After trying to fix the tests, it seems the problem is actually a deadlock when trying to create or delete a table at the same time another thread is trying to create or delete any other table, not only the same table, so the atomic fix did not work. With this in mind i think the only way to make it work is with the -- --test-threads 1.
I had the ci set to use this before, but the cargo tarpaulin was not working because of that, but it seems to work now.

After trying to fix the tests, it seems the problem is actually a deadlock when trying to create or delete a table at the same time another thread is trying to create or delete any other table, not only the same table, so the atomic fix did not work. With this in mind i think the only way to make it work is with the -- --test-threads 1.

FirebirdSQL restricts this kind of concurrent table creation?

I had the ci set to use this before, but the cargo tarpaulin was not working because of that, but it seems to work now.

Yes. Last run run test in sequence and it was successful.

@jairinhohw @fernandobatels ,
I need some help to figure out how to build rsfbclient on Windows.
What I need to do for fixing this failure?

@jairinhohw @fernandobatels ,
I need some help to figure out how to build rsfbclient on Windows.
What I need to do for fixing this failure?
note: LINK : fatal error LNK1181: cannot open input file 'fbclient.lib'

I windows you need provide the fbclient.dll. You have installed all the firebird tool kit? I think this is a problem with the PATH env variable.

I have installed FirebirdSQL v3.0.5 in a Github Windows server with the command:
choco install firebird -params '/SuperClassic /ClientAndDevTools'

Any guidance from here?

Not worked copying fbclient.lib to build dir:
Coying C:\Program Files\Firebird\Firebird_3_0\lib\fbclient_ms.lib to D:\a\rsfbclient\rsfbclient\target\debug\deps\fbclient.lib
Coying C:\windows\system32\fbclient.dll to D:\a\rsfbclient\rsfbclient\target\debug\deps\fbclient.dll

= note: LINK : fatal error LNK1181: cannot open input file 'fbclient.lib'

https://github.com/juarezr/rsfbclient/runs/983032932?check_suite_focus=true

Our CI is working now with Windows. I think we can close this issue.


Further improvments on VEP

Adding the possibility to use the VEP GeneSplicer plugin
VCFs from Manta won't over write any more
Add new reduceVCF() function to remove .ann, .gz and .vcf from a vcf file name

PR checklist

[X] PR is made against dev branch
[X] This comment contains a description of changes (with reason)
[X] Ensure the test suite passes (./scripts/test.sh -p docker -t ALL).
[ ] Documentation in docs is updated
[ ] CHANGELOG.md is updated
[ ] README.md is updated

I just need to do some docs, and I'll request approval


Fix for 2483 Author and description null checks & tests

Fix for #2483
I enhanced the PackageValidation for the "null-cases" for the description and author element. The NuGet package author itself is never (or at least currently) null, but maybe filled with an empty element.
No  element set (or an empty description element will trigger the same behavior):

No  or  element set:

Without those checks some other processes might fail (e.g. the lucene job needs the description and might fail if it is null).
Currently you could upload something without the author tag, but when you try to update it you will get a validation error:

This PR should fix the null handling and discard any uploads without a description or an empty author.
It seems that a empty  is also catched, because somewhere in the pipeline the description will be set to null instead of string.empty in this case, so even if you submit an empty description, the code will discard the upload.

Regarding empty authors: Uploading without authors is ok, but changing the package via the edit form would still require a author? Or is this field no longer required?

I removed the author validation. So if you upload a package without any owner or author information it will pass, but if you ever try to update it via the UI the field is marked as "required".

@skofman1 is there anything still open from your review or do you think it is ok?

Looks great! Thanks @robertmuehsig !


Finer Grained Processing Control

It should be possible to choose which nodes process which data by making processor filters target node groups (gh-1623).

Consider using user-defined node attributes to be entered as a processor filter condition. A node to its attributes should be a one-to-many relationship.

Another idea is to allow the user to define a weight for a processor filter. This would add nuance to the existing priority setting and allow users to further prioritise or restrict processor time to streams.

Also see gh-1622, gh-2330


Correct formatting in github alerts in README.md

Correct formatting in github alerts in README.md.
AS-IS

TO-BE

Thanks @jongwooo
/lgtm
/approve


MB-7098 Upgrade to golang 1.15.10

Description
Upgrade to golang release 1.15.10
Reviewer Notes
None
Setup
Install golang 1.15.10 See Manage golang with asdf
asdf install
asdf global golang 1.14.7

What ever you use normally after installing golang 1.15.10:
make server_tests
make server_run
etc

Code Review Verification Steps

[X] Request review from a member of a different team.
[X] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Messages

:book:
:link: MB-7098

  Generated by :no_entry_sign: dangerJS against 46aa82721a79d5a42fa10fe5484a4e5698742df8


default segmentation pattern

When using standard templates (eg 100 slide templates) it is much faster and more accurate to use a template which has already been segmented and cleaned and use the copy function. A default "slide 100" segmentation pattern or similar would streamline this activity.

Good idea - this might best be done by being part of an Inselect template file.

agreed, makes sense as an optional part of a template. Future "slide template" based imaging could then have ingest, segmentation, barcode reading and image cropping out completely automated.

Yes, that would be really good. Perhaps a new command 'Save to new template', which prompts the user for the name of a new .inselect_template file and saves the currently loaded template + current items. Save just the bounding boxes or also include metadata?
Inselect would add the default bounding boxes (and possibly metadata) when a new file is ingested. We would also need to change the ingest command-line tool to take an optional path to a .inselect_template file.

After discussion with the group 2016-03-04, we will have 'Bounding box' / 'Segmentation' / 'Cookie cutter' templates (separate to metadata templates; name to be decided) that specify the locations of bounding boxes. These boxes can be created either when images are ingested or/and as a separate manual step.

A couple of concerns

@benscott feels we should have a more automated method that combines the results of segmentation with the refined boxes previously saved by the user
Drift in precise location of template within images might cause cookie cutter-produced bounding boxes to not align on top of objects - @quicklizard to investigate this


Push to CocoaPods Trunk Repo

https://guides.cocoapods.org/making/getting-setup-with-trunk.html

Done