Reading keychain entry, because either user or password were empty

INFO [2023-03-09 18:02:50.52]: Installing provisioning profile...
WARN [2023-03-09 18:02:51.32]: Setting environment variable 'sigh_com.ybm.GDD_development' to 'a36f149b-cea0-494a-8d24-cf9179b57a30'
WARN [2023-03-09 18:02:51.32]: Setting environment variable 'sigh_com.ybm.GDD_development_team-id' to '5Z2477437E'
WARN [2023-03-09 18:02:51.32]: Setting environment variable 'sigh_com.ybm.GDD_development_certificate-name' to 'Apple Development: xiaoyang li (2UV86B857L)'
WARN [2023-03-09 18:02:51.32]: Setting environment variable 'sigh_com.ybm.GDD_development_profile-name' to 'match Development com.ybm.GDD'
WARN [2023-03-09 18:02:51.32]: Setting environment variable 'sigh_com.ybm.GDD_development_profile-path' to '/Users/yaobangmang/Library/MobileDevice/Provisioning Profiles/a36f149b-cea0-494a-8d24-cf9179b57a30.mobileprovision'
undefined method git_url' for #<CredentialsManager::AppfileConfig:0x00007f8cb39ada70> undefined method git_url' for #CredentialsManager::AppfileConfig:0x00007f8cb39d01d8
undefined method git_url' for #<CredentialsManager::AppfileConfig:0x00007f8cb39e33f0> undefined method git_url' for #CredentialsManager::AppfileConfig:0x00007f8cb3a016c0
undefined method `git_url' for #CredentialsManager::AppfileConfig:0x00007f8cb1de3600
+-------------------------------------+------------------------------+
|                      Summary for sigh 2.212.1                      |
+-------------------------------------+------------------------------+
| app_identifier                      | com.xyycrm                   |
| force                               | false                        |
| cert_id                             | B75DRM3Z7H                   |
| provisioning_name                   | match Development com.xyycrm |
| ignore_profiles_with_different_name | true                         |
| fail_on_name_taken                  | false                        |
| include_all_certificates            | false                        |
| include_mac_in_profiles             | false                        |
| platform                            | ios                          |
| development                         | true                         |
| adhoc                               | false                        |
| developer_id                        | false                        |
| skip_install                        | false                        |
| skip_fetch_profiles                 | false                        |
| skip_certificate_verification       | false                        |
| readonly                            | false                        |
+-------------------------------------+------------------------------+
WARN [2023-03-09 18:02:51.33]: To not be asked about this value, you can specify it using 'username'
[2023-03-09 18:02:51.33]: Your Apple ID Username<EMAIL_ADDRESS>INFO [2023-03-09 18:02:59.71]: Starting login with user<EMAIL_ADDRESS>Reading keychain entry, because either user or password were empty
Loading session from<EMAIL_ADDRESS>Available session is not valid any more. Continuing with normal login.
Loading session from environment variable
Session loaded from environment variable is not valid. Continuing with normal login.
Unable to find 'X-Apple-HC-Bits' and 'X-Apple-HC-Challenge' to make hashcash
Looking for related GitHub issues on fastlane/fastlane...
Search query: The request could not be completed because:

503 Service Temporarily Unavailable

503 Service Temporarily Unavailable
Apple

Happens with me as well.

I am encountering exactly the same issue right now. Did you find a solution @biyuhuaping ?

我现在遇到完全相同的问题。你找到解决办法了吗@biyuhuaping？

我换了一台电脑，就奇迹般的好啦，应该是电脑环境问题。

@biyuhuaping Can you provide any solution?


Add application command events

Summary
Add events for application command create, update, and delete.
What is the feature request for?
The core library
The Problem
https://discord.com/developers/docs/topics/gateway#commands-and-events-gateway-events
application command events not implemented
The Ideal Solution
add them. Reminder for me
The Current Solution
No response
Additional Context
No response

Bots aren't meant to get these events, ok then


how to download the dataset that need request

I am a Ph.D. candidate of Xiamen University in China. My research is focused on depth estimation and semantic segmentation. Recently I need to use your packages named leftImg8bit_sequence_trainvaltest.zip and camera_trainvaltest.zip of cityscapes dataset. But the website shows "The following packages are available upon request". In order to complete my research, I need your permission to download these two package.

I have requested the same a couple of weeks ago to the email address, and I had no response. Is there a formal way to request these files? There is no mention of how on the website.
Thanks!!!

You both have access to the requested packages, the easiest way for such requests is to send us an email.

Hi @nanhailongwang @tano297
if anyone of you have got the "leftImg8bit_sequence_trainvaltest" (324GB) dataset, could you share just the "validation" split of it with me, please? I am having trouble downloading it.
Let me know if it's possible for you to share a download link with me on my email


PythonFunctionTool error

I'm using pythonFunctionTool, ZeroshotAgent and Chat-LLM, but getting error as my function use poylon library and it is unable to find it. How can i install it and build a complete agent
Error

Flow

On running
> Entering new AgentExecutor chain...
I need to get the open-close aggregate information for Apple's stock for today's date.
Action: get_daily_open_close_agg_info(stock_symbol,_date)
Action Input: ("AAPL", "2022-01-01")
Observation: get_daily_open_close_agg_info(stock_symbol,_date) is not a valid tool, try another one.
Thought:I must have misunderstood the available tools. I'll try again.
Action: None.
Final Answer: I'm sorry, but I currently do not have the ability to fetch live stock prices. I suggest referring to a reliable financial news website or your stock trading platform for the current value of Apple's stock.

> Finished chain.
{'input': 'Current apple stock value',
 'output': "I'm sorry, but I currently do not have the ability to fetch live stock

If you plan on using a package that is not a dependency of Langflow's you should install it first (e.g  run pip install polygon on the same environment as Langflow)

@ogabrielluiz thanks for your help, I resolved it by running it locally, as I have everything installed on local, that was the issue on HF space,

I resolved it by running it locally, as I have everything installed on local, that was the issue on HF space,


Customize return of the status from cross-cutting concerns

Allow customization of the return from common modules implementing cross-cutting concderns.

Ex.: current
{
“statusCode”: “500",
“code”: “ERR10010",
“message”: “RUNTIME_EXCEPTION”,
“description”: “Unexpected runtime exception”
}
Allow for example:
“error”: {
“statusCode”: “500",
“code”: “ERR10010",
“message”: “RUNTIME_EXCEPTION”,
“description”: “Unexpected runtime exception”
}

Allow return of list of statuses

I was thinking about creating a StatusSerializer interface and users can put their implementations in service.yml to replace the default behaviour in Status class. Default will be the current one with the toString. This works for all the framework code but I don't know if it will work with APIs built on top of the framework. Also, this make Status module depends on Service which is not that bad as there is no further dependencies.


Update sbt-scoverage to 1.7.1

Updates org.scoverage:sbt-scoverage from 1.7.0 to 1.7.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #446 (ea4acc9) into master (4d23739) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #446   +/-   ##
=======================================
  Coverage   86.31%   86.31%
=======================================
  Files         108      108
  Lines        1425     1425
  Branches       40       40
=======================================
  Hits         1230     1230
  Misses        195      195

Continue to review full report at Codecov.

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


Comments

Whoooooooo.

Codecov Report
Patch coverage: 99.58% and project coverage change: -0.12 :warning:

Comparison is base (85ad437) 99.69% compared to head (8d625f2) 99.58%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   99.69%   99.58%   -0.12%
==========================================
  Files          41       45       +4
  Lines        1985     2156     +171
==========================================
+ Hits         1979     2147     +168
- Misses          6        9       +3

Flag
Coverage Δ

unittests
99.58% <99.58%> (-0.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

api/views/releases.py
100.00% <ø> (ø)

api/views/cards.py
98.83% <80.00%> (-0.58%)
:arrow_down:

api/depends.py
100.00% <100.00%> (ø)

api/main.py
100.00% <100.00%> (ø)

api/models/card.py
100.00% <100.00%> (ø)

api/models/comment.py
100.00% <100.00%> (ø)

api/models/user.py
100.00% <100.00%> (ø)

api/schemas/cards.py
100.00% <100.00%> (ø)

api/schemas/comments.py
100.00% <100.00%> (ø)

api/schemas/decks.py
100.00% <100.00%> (ø)

... and 12 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Auto generated calendar link to same page

The auto generated calendar link to the same page (2021-MM-dd) as the snapshot:

Expect behavior:
Link to the corresponding journal pages

Hi,
What is your date-formatter settings please?

Hi,
What is your date-formatter settings please?

 :date-formatter "yyyy-MM-dd"

I released a hotfix in https://github.com/akhater/Lupin/commit/bae01b2282a0ca8efa4072e62a217c71fddf676c for that. please update to https://github.com/akhater/Lupin/releases/tag/v3.3.1e and test it.
I'd appreciate if you can report back

Issue was auto-closed by Github waiting for your feedback before closing it
@ethinx thank you

Just pull and restart Lupin, it's fixed.
thanks!

Closing the issue after your confirmation


Add show tabs with live channels toggle to tab area context menu

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Currently the only way to show tabs with live channels only is using the default hotkey Ctrl+Shift+L
There should be a toggle in the tab area context menu or in the settings
Screenshots

OS and Chatterino Version
Chatterino Nightly 2.4.5 (commit ba440e0c) built on 2023-08-28 with Qt 6.5.0, MSVC 193632538 Running on Windows 10 Version 22H2, kernel: 10.0.19045

Tab visibility is an option in the settings, just below tab layout. The setting was introduced in #4358 which also added that hotkey.
I agree it should also be added to the context menu since I also found it not being present confusing.


Disconnected when setting up a command block

When I try to create a simple command block I get disconnected and the command in the block doesn't save.

I am very new to github and pretty much only used it for a few little things of my own.  This was the first project on github I felt like contributed to so I don't know how it all works so I am sorry if it is wrong to ask.  Is this issue being looked in to?  I have confirmed the problem to happens in both the precompiled linux 64 bit build and by compiling myself, and precompiled Windows 64 bit build.

I confirm the bad behavior, when a client edits the commandblock's text and hits the Done button, the server kicks them (and commits suicide in Debug mode due to badly parsed PluginMessage packet).

FYI, for the command block it isn't garbage data but instead an extra Boolean for whether output from the command block should be stored (which is why it initially happened).  Don't know about what it is in 1.7.10, though.

Thanks, added the missed read.


Submit button mode

Hi,
You have 2 mods.

Could you add next mode like 'submit_button_mode'.
It should work like on languagetool page by clicking 'Check text' button.
Is it possible to make 1 submit button to be used in 2 tinymce editor instances?

In the next features update, I plan on adding a 'manual' mode. Which you can then call from execCommand. So as long as you add both to a button and run execCommand for both it should work in theory.

Good idea, thx.

manual mode has been implemented along with mceLTProcess execCommand.


Node API - supporting of reading entry from memory

Do you want to request a feature or report a bug?
Feature request

What is the current behavior?
In webpack config entry accepts only a disk path to an entry.

What is the expected behavior?
It would be nice, If entry also accepts a file from memory stream.
I'm creating a webpack dynamic bundler as a service using webpack node api. I created the following very simple node code that allows me to return a bundle for a given entry from the disk.
var http = require('http');

 http.createServer(function(req, res) {
     res.writeHead(200, { 'Content-Type': 'text/plain' });
     const webpack = require('webpack');
     const MemoryFS = require("memory-fs");

     const compiler = webpack({
         entry: './app.js',
         output: {
			 filename: 'bundle.js',
             libraryTarget: 'amd'
         }
     });

     compiler.outputFileSystem = new MemoryFS();

     compiler.run((err, stats) => {
         if (err || stats.hasErrors()) {
             // Handle errors here
         }

         const content = stats.compilation.assets['bundle.js'].source()

         // done processing
         res.end(content);
         // ...
     });

 }).listen(process.env.PORT);

But the entry is from the disk. I want to read from memory instead like
     var MemoryFileSystem = require("memory-fs");
     var fs = new MemoryFileSystem();
     const compiler = webpack({
                     entry: fs.readFileSync("/a/src/entry.js"),
                     output: {
			 filename: 'bundle.js',
                         libraryTarget: 'umd'
                     }
       });

If this is a feature request, what is motivation or use case for changing the behavior?
I've a requirement to dynamically bundle a script based on dynamically generated entry file at run time.
This feature allows me to generate entry file at run time, put in memory and pass to the webpack -- Instead of generating an entry, saving to disk and pass to webpack.
Thanks,
Bruk

Entries accept loaders. I think you could write something along entry: '!disk-loader?file=...' that would do what you want. You can read the file in the loader as you like. It can be async op too if you use this.async() to capture a callback.

@bebraw thanks a lot... loader saves me.
BTW, do you see any issue using webpack like this in production?
For others benefit, here is what works for me.
My sample loader looks like the following -- in real life planning to call a service and do more staff but the simple version is:
module.exports = function() {
  return `import bar from './bar';
		bar();`
};

Then, my node code looks like the below:
 var http = require('http');

 http.createServer(function(req, res) {
     res.writeHead(200, { 'Content-Type': 'text/plain' });
     const webpack = require('webpack');
     const MemoryFS = require("memory-fs");

     const compiler = webpack({
         entry: './loader!',
         output: {
			 filename: 'bundle.js',
             libraryTarget: 'amd'
         }
     });

     compiler.outputFileSystem = new MemoryFS();

     compiler.run((err, stats) => {
         if (err || stats.hasErrors()) {
             // Handle errors here
         }

         const content = stats.compilation.assets['bundle.js'].source()

         // done processing
         res.end(content);
         // ...
     });

 }).listen(process.env.PORT);

Thanks,
Bruk.

BTW, do you see any issue using webpack like this in production?

Nope. My understanding is that this is just one little known way to use loaders. It happens to work here as you needed something dynamic.

@bruk1977 I use this in production but as a temporary solution for angular.js
https://github.com/trysound/multi-entry-loader

@TrySound thanks for the info.


possible copy/paste error in on-changes interceptor

I was just eyeballing std_interceptors.cljc and noticed the on-changes interceptor has the id of :enrich, as opposed to :on-changes. See line 273. This seems incorrect but I could be mistaken.

Yep, this line should be :id   :on-change
https://github.com/Day8/re-frame/blob/master/src/re_frame/std_interceptors.cljc#L273
But not a critical fix, because :id is pure decorative.

Thanks Micheal.

Fixed in 209470e76130eb234acac46b9bedfc51bbeec0ab


bower install error

gives me this:
Stack trace:
Error: EEXIST: file already exists, open '/home/hkatz/.cache/bower/packages/35300e45e44dbc6f186ed069533ef0af-2.2.3.lock'
at Error (native)
Console trace:
Error
at StandardRenderer.error (/usr/local/lib/node_modules/bower/lib/renderers/StandardRenderer.js:81:37)
at Logger. (/usr/local/lib/node_modules/bower/lib/bin/bower.js:110:26)
at emitOne (events.js:90:13)
at Logger.emit (events.js:182:7)
at Logger.emit (/usr/local/lib/node_modules/bower/lib/node_modules/bower-logger/lib/Logger.js:29:39)
at /usr/local/lib/node_modules/bower/lib/commands/index.js:48:20
at _rejected (/usr/local/lib/node_modules/bower/lib/node_modules/q/q.js:844:24)
at /usr/local/lib/node_modules/bower/lib/node_modules/q/q.js:870:30
at Promise.when (/usr/local/lib/node_modules/bower/lib/node_modules/q/q.js:1122:31)
at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/bower/lib/node_modules/q/q.js:788:41)
System info:
Bower version: 1.7.9
Node version: 5.10.1
OS: Linux 2.6.32-573.22.1.el6.x86_64 x64
even after I try bower cache clean in shell prior to running sbt. Just updated git to this last play 2.5.2 release. This is Centos 6.7.
Please advise.
Thanks

@OliverKK :+1:
This is the Heroku build log:
-----> Fetching set buildpack https://github.com/heroku/heroku-buildpack-multi... done
-----> Multipack app detected
=====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-ruby.git
=====> Detected Framework: Ruby
-----> Compiling Ruby
-----> Using Ruby version: ruby-2.3.0
-----> Installing dependencies using bundler 1.11.2
       Your app was upgraded to bundler 1.11.2.
       Previously you had a successful deploy with bundler 1.9.7.

       If you see problems related to the bundler version please refer to:
       https://devcenter.heroku.com/articles/bundler-version
       Ruby version change detected. Clearing bundler cache.
       Old: ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux]
       New: ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-linux]
       Running: bundle install --without development:test --path vendor/bundle --binstubs vendor/bundle/bin -j4 --deployment
       You are trying to install in deployment mode after changing
       your Gemfile. Run `bundle install` elsewhere and add the
       updated Gemfile.lock to version control.
       You have added to the Gemfile:
       * sass (= 3.4.22)
       You have deleted from the Gemfile:
       * sass (= 3.4.9)
       Bundler Output: You are trying to install in deployment mode after changing
       your Gemfile. Run `bundle install` elsewhere and add the
       updated Gemfile.lock to version control.

       You have added to the Gemfile:
       * sass (= 3.4.22)

       You have deleted from the Gemfile:
       * sass (= 3.4.9)
 !
 !     Failed to install gems via Bundler.
 !
 !     Push rejected, failed to compile Multipack app

Maybe the build pack has updated in the meantime? If you create a Heroku account, then you can test it.

It seems that the Gemfile.lock is the old one. This should also be updated.
Bundler Output: You are trying to install in deployment mode after changing
       your Gemfile. Run `bundle install` elsewhere and add the
       updated Gemfile.lock to version control.

       You have added to the Gemfile:
       * sass (= 3.4.22)

       You have deleted from the Gemfile:
       * sass (= 3.4.9)

@akkie I´am looking into that. Thanks.

Update, I had resolved my bower build. Outside of sbt (or activator), I ran the npm install after clearing both its and bower cache. I also had to update system version of rvm as well as ruby to 2.3.0. I also manually inspected the index.html in ui to ensure npm had all the angular variants required else the first request to the controller would show j/s errors in the browser developer tool.

I did create a new Gemfile.lock by executing bundle install and pushing it to my heroku repo.
Still the same error:
You have added to the Gemfile:
remote:        * sass (= 3.4.22)
remote:
remote:        You have deleted from the Gemfile:
remote:        * sass (= 3.4.9)
Revoking changes to sass 3.4.9 didn´t have a effect either.

Could you nevertheless push the updated Gemfile.lock?

@see PR #38

The build runs now fine on Heroku


Typescript doesn't support Contacts and mozContacts in Navigator. please add it in future updates

I need to open user's phone contacts from my Angular app and I need to use the contacts and mozContacts to do it but there are errors that say: properties 'contacts' and 'mozContacts' don't exist in Navigator.

Please use the issue templates for bug reports and feature requests. This will make it much easier for the TypeScript team and allows them focus on improving the language, instead of decyphering GitHub issues. :-)

Navigator.contacts seems to be a very early draft. The Mozilla documentation does not even list a specification for it. Such early and experimental features are usually not supported by TypeScript (see the TypeScript-DOM-lib-generator readme).
You will have to make use of Declaration Merging and provide your own types in your projects.

Please use the issue templates for bug reports and feature requests. This will make it much easier for the TypeScript team and allows them focus on improving the language, instead of decyphering GitHub issues. :-)
Navigator.contacts seems to be a very early draft. The Mozilla documentation does not even list a specification for it. Such early and experimental features are usually not supported by TypeScript (see the TypeScript-DOM-lib-generator readme).
You will have to make use of Declaration Merging and provide your own types in your projects.

It's being used in other angular apps and I need to use it too.
`interface NavigatorContacts{
readonly contacts : ContactsManager
}
interface ContactsManager {
getProperties() : Promise<string[]>;
select(properties:string[], options?:boolean) : Promise;
};
interface Navigator extends NavigatorContacts{}`
I added this code to my component but there's still that error. can you please tell me how to make it work?


Updating CI workflow

This PR updates CI to address deprecation notices and false-positives.

I'm fine with force-landing this now and then #217 to move crosstrait.


Help me please: ESP32 stuck in deep sleep

Hi
I need some help: the ESP32 of my T-Pico C3 is stuck in deep sleep.
I was testing the deep sleep feature of the ESP32. I planned to wake it up from the RP2040 with a signal on GPIO4.
My idea was to use the following sketch on the ESP32:
import esp32
from machine import Pin, UART
from machine import deepsleep
import time

# set UART
RP2040 = UART(1, baudrate=38400, tx=7, rx=6, cts=5, rts=4)
time.sleep_ms(50)

print('going to sleep')
wake1 = Pin(4, mode = Pin.IN)
esp32.wake_on_ext0(pin = wake1, level = esp32.WAKEUP_ANY_HIGH)
deepsleep()

time.sleep(1)
RP2040.write('OK')

It goes into deep sleep and wakes up on receiving a signal on the GPIO4 (rts pin of the UART). Then it would send a message to the RP2040 to tell it's awake. To run it automatically, I saved it as 'Sleep_ESP32.py' and run it using the main.py:
import Sleep_ESP32
On the RP2040 side, I run the following sketch:
# RP2040 side: test wake up ESP32
from machine import UART, Pin
from time import sleep_ms
# Init UART
ESP32 = UART (1, baudrate=38400, tx=Pin(8), rx=Pin(9), cts=Pin(10), rts=Pin(11))
PinWakeUp  = Pin (10)
# Init red LED
led = Pin(25, Pin.OUT)
led.off()

sleep_ms(2000)
print('sending wake up')
PinWakeUp.on()
sleep_ms(30)
PinWakeUp.off()

line = ''
while 1:
    while ESP32.any() > 0:
        led.on()
        buf = ESP32.readline()
        line += buf.decode("utf-8")
    if len(line) != 1: print(line)

It wakes for 2 seconds then sends a signal on the RTS pin (10), which is supposedly connected to the RTS pin (4) of the ESP32. If it receives a message from the UART, it lits the red led.
When I run it, with the ESP32 part automatically running also, I see the red led. So I guess the ESP32 is awaken.
But when I connect the board on the ESP32 side (green led on) I can't get any access to the USB port. The board does not appear in the device manager any more. I can't access it with Thonny either and can't access to the flash as a USB device.
It seems stuck in deep sleep. How can I access the ESP32 again. Is there a safe boot mode?
Thanks for your help.

You don't access the ESP32 part using Device Manager, that only works for the RP2040 part. These are the instructions for accessing the ESP32 part:
ESP32-C3 Upload method
ESP32C3 mainly uses ESP32-AT firmware here. If you want to use it as a coprocessor, you can modify the firmware.
Method 1: if the serial port is displayed with the word jatg when connecting to USB, you can upload it directly using USB. (please note that do not use USB pins to define other functions, otherwise it will be troublesome to upload next time).
Method 2: (if there is no serial port when connecting ESP32C3 USB) disconnect USB, short connect ESP32C3-IO9 and GND, and then reconnect USB (note that ESP32C3 device is selected to connect USB).


Set a MAX_CORES env var to help Java runtimes tune their settings like thread pools to the max number of cores in the cgroup quota.

@rhuss could you review again?

Yeah, will do. thanks !

@chirino I just added your PR, thanks a lot. I made some changes:

Introduced a container_limits script which exports CONTAINER_MAX_MEMORY and CONTAINER_CORE_LIMIT in case. Its sourced by run-java-sh
java-default-options takes benefits from this options.
a 'JAVA_CORE_LIMIT' can be set from external to nail down the core by configuration
Added check for a quota of -1 which means no restriction.
Updated README

'hope thats ok so. I'm going to build now Java images 1.2.2 now with this build script and update f-m-p accordingly.


Hover type hint isn't highlighting and contains

Seems that the nbsp; issue has been fixed in master. Has there been such change? If not, maybe I got into some weird settings again

If the language server sends a MarkedString (object containing both "language" and "value") then LSP puts it in a Markdown fenced code block (so mdpopup's syntax highlighting can be applied to the text).
Code blocks don't convert   but they don't collapse whitespace either. In other words, the language server doesn't need to send   in MarkedStrings.
Demo using Github:

 <- one space
  <- two spaces

Sorry, I didn't actually answer your question.
It's possible you see this now because the removal of preserve_whitespace (https://github.com/tomv564/LSP/commit/761c2c570446d73ecd2c87f5512fc5b7c9757e4f) got released with 0.4.0 5 days ago?


Unpkg seems to be down for react

I am getting 520 errors for react related urls.
For<EMAIL_ADDRESS>I am getting

This is causing our production servers to be down for all of our clients.
Is anyone else having this problem?

Even https://www.unpkg.com/status/ is down

@4amramen As  a workaround , you can specify the full version, e.g.:
<EMAIL_ADDRESS>

Same here

Consider migrating<EMAIL_ADDRESS>

Same here

@4amramen As a workaround , you can specify the full version, e.g.:
<EMAIL_ADDRESS>
This solution worked for me.  I just added the full version to all react links

It looks like status page also down.

https://www.unpkg.com/status/
Previous:

#352
#337

It looks like status page also down.

https://www.unpkg.com/status/
Previous:

#352
#337

unpkg.com doesn't have a status page. That's a package in npm called "status" (just like how unpkg.com/react is an npm package called "react").

Ah, Thanks.

@4amramen As a workaround , you can specify the full version, e.g.:
<EMAIL_ADDRESS>
Thanks! 👍

I was here

Seems to be down for https://unpkg.com/@microsoft/mgt@2 intermittently as well. Getting random 520's from Cloudflare.

📝 I've created unofficial status pages for unpkg/skypack/esm.sh:
https://github.com/azu/npmcdn-upptime

@GrnPero what if you are not using react?

@GrnPero what if you are not using react?

Switch cdns? That's what I did
    <!-- <script<EMAIL_ADDRESS>-->
<script<EMAIL_ADDRESS>

React is back up, so I assume all other packages are.

unpkg seems to be so unstable. Everyone should consider using jsdelivr. Use this page to get the jsdeliver url from your unpkg urls. https://www.jsdelivr.com/unpkg

unpkg seems to be so unstable. Everyone should consider using jsdelivr. Use this page to get the jsdeliver url from your unpkg urls. https://www.jsdelivr.com/unpkg

Thanks! 🤙 Going to explore this or self hosting.


Package activation time

Takes ~350 ms to activate the package. How come?

Dupe of #1026

Sorry about that!

It's okay, happens to all of us :)


Ooo, they added Dance Diffusion!  That bonus feature gets me excited to implement..

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Best practice for CSS?

I have a component structure like so:
- /app
  - /components
    - /alert
      - alert.css
      - alert.html.erb
      - alert.rb

I'm importing the CSS file in Webpacker's application.css.
Unfortunately this setup gives me the error:
More than one template found for Alert. There can only be one sidecar template file per component.

I'm currently working around this by using _alert.css instead of alert.css, but it seems odd actionview-component is treating this CSS file as a template file?
Is there a best practice here I should be following? Or is this an issue actionview-component should fix?

@AlecRust you're right, actionview-component should only raise an error if there is more than one template file. It shouldn't care if there is a sidecar CSS file.

@AlecRust can you check your app against v1.3.3 to see if this is fixed for you?

@joelhawksley works great, many thanks!


Nested

Hello.
I installed django-ckeditor==6.7.1 and Django==4.2.11 but the CKEditor did not showed up. I noticed weird HTML code generated by CKEditorWidget:
<link href="/static/ckeditor/ckeditor.css" media="all" rel="stylesheet">
<script type="text/javascript" src="<script src="/static/ckeditor/ckeditor-init.js" data-ckeditor-basepath="/static/ckeditor/ckeditor/" id="ckeditor-init-script"></script>"></script>
<script type="text/javascript" src="/static/ckeditor/ckeditor/ckeditor.js"></script>

Notice nested <stript in the src for ckeditor-init.js
Any idea please?
When I change media() to this:
@property
    def media(self):
        return Media(
            css={"all": ["ckeditor/ckeditor.css"]},
            js=(
                # JS(
                #     "ckeditor/ckeditor-init.js",
                #     {
                #         "id": "ckeditor-init-script",
                #         "data-ckeditor-basepath": getattr(
                #             settings,
                #             "CKEDITOR_BASEPATH",
                #             None,
                #         )
                #                                   or static("ckeditor/ckeditor/"),
                #     },
                # ),
                "ckeditor/ckeditor-init.js",
                "ckeditor/ckeditor/ckeditor.js",
            ),
        )

it works perfectly fine. What's wrong with the JS()?

This sounds a bit as if you're using django-js-asset<2 instead of the version which works with Django 4.1 or better?
Recent django-ckeditor versions depend on django-js-asset>=2.0, so I'm not sure what's going on.

(... the CKEDITOR_BASEPATH thing is necessary when you're using ManifestStaticFilesStorage in production.)

Yes, I have already found some issues and docs related to django-js-asset, CKEDITOR_BASEPATH  and ManifestStaticFilesStorage but nothing helped.
I have most recent django-js-asset installed:
pip freeze|grep asset
django-js-asset==2.2.0

I already tried setting the CKEDITOR_BASEPATH to custom value. The path is changed, but the nested script in src is still present.
I don't use any Manifest static file storage.
And the issue is present on localhost as well.

No, I use default Django template engine :/


[FLINK-3728] [tableAPI] Improve error message and documentation for unsupported SQL features.

This PR

extends the Table API / SQL documentation and lists known limitations.
improves the error message in case Calcite cannot fails to optimize the program because it includes unsupported features.

[X] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[X] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[X] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Will merge this later today, unless somebody objects.

Also looks good! +1.

Thanks :-)

merging


update code

This radix sort faster than the original code. ^ ^

Looks good @Moon1102.
For future reference, could you separate these pull requests so 1 pull request accounts for 1 algorithm?
Thanks!

Okay..
I will pay attention to it.


1275/rendering performance improvements

Please remember the Contributing Guidelines :heart:
:star2: What does this PR do?

Rendering improvements
#1275
#1308
:bug: Recommendations for testing
All changes should be tested across Chrome and Firefox.

Great, thanks for merging!!


java.lang.RuntimeException: Could not resolve system/org/apache/felix/org.apache.felix.framework/4.4.1/org.apache.felix.framework-4.4.1.jar

Hi,
When running opennms in a swarm mode cluster with the following compose file I get the above stated error:
version: '3.2'

networks:
  monitoring:
    external: true

services:
  database:
    image: postgres:9.6.3
    env_file:
      - .postgres.env
    networks:
      - monitoring
    volumes:
      - /testing/opennms-horizon/psql-data:/var/lib/postgresql/data
    #ports:
    #  - "5432:5432"
    deploy:
      placement:
        constraints: [node.hostname == dock-cl01]

  opennms:
    image: registry.myregistry.com/opennms-21.0.1-1
    env_file:
      - .opennms.env
      - .postgres.env
    networks:
      - monitoring
    deploy:
      labels:
        - traefik.port=8980
        - traefik.frontend.rule=Host:opennms.mydomain.com
      placement:
        constraints: [node.hostname == dock-cl01]
    volumes:
      - /testing/opennms-horizon/opennms-etc:/opt/opennms/etc
      - /testing/opennms-horizon/opennms-data:/opennms-data
    command: ["-s"]
    ports:
      - "8980:8980"
      - "18980:18980"
      - "1099:1099"
      - "8101:8101"
      - "61616:61616"
      - "5817:5817"
      - "162:162/udp"

Any idea about what could be wrong?
I get nothing if I browse opennms.mydomain.com (this is being managed by Traefik reverse proxy) or I go to the node where this service sits, and do:
http://<IP_ADDRESS>:8980

The only thing I get when going to the node, is a redirection on the browser to:
http://<IP_ADDRESS>:8980/opennms/

I think it could be related with the hostname (localhost for running with docker compose) since I am using docker swarm mode, my database server is located somewhere else in the network, but reachable by opennms_database
So, what would be the correct way to set up the proper hostname in case mine it is not going to be localhost?
Thanks.

Does it not start at all? What is the log file where you see this error? I don't see anything which is weird, the reverse proxy shouldn't be a reason for this. The system/... directory looks like as a Karaf issue.

It does start (from docker point of view) but nothing is loaded when trying to reach the website.
But I am almost sure that it has to do with "localhost ", I have found many of them in opennms-etc volume.
Logs are from docker service logs opennms_opennms and also from the opennms-data/logs volume.
How should I be able to change those localhost definitions to point to the my actuall database is?
like:
deploy@dock-cl01:/testing/opennms-horizon$ grep -r localhost opennms-etc/*
opennms-etc/ackd-configuration.xml:      <parameter key="readmail-config" value="localhost" />
opennms-etc/create.sql:--# The following command adds the initial localhost poller entry to
opennms-etc/create.sql:INSERT INTO monitoringsystems (id, label, location, type) values ('00000000-0000-0000-0000-000000000000', 'localhost', 'Default', 'OpenNMS');
opennms-etc/destinationPaths.xml:        <mstation>localhost</mstation>
opennms-etc/events/AlarmChangeNotifierEvents.xml:
opennms-etc/events/AlarmChangeNotifierEvents.xml:
opennms-etc/events/opennms.alarmChangeNotifier.events.xml:
opennms-etc/events/opennms.alarmChangeNotifier.events.xml:
opennms-etc/events/GraphMLAssetPluginEvents.xml:
opennms-etc/javamail-configuration.xml:                        default-read-config-name="localhost" default-send-config-name="localhost" >
opennms-etc/javamail-configuration.xml:
opennms-etc/javamail-configuration.xml:  <sendmail-config name="localhost"
opennms-etc/javamail-configuration.xml:    <sendmail-message to="root@localhost"
opennms-etc/javamail-configuration.xml:  <readmail-config name="localhost"
opennms-etc/javamail-configuration.xml:  <end2end-mail-config name="default" readmail-config-name="localhost" sendmail-config-name="localhost" />
opennms-etc/jmx-datacollection-config.xml:            <mbean name="org.apache.activemq" objectname="org.apache.activemq:type=Broker,brokerName=localhost">
opennms-etc/notifications.xml:        <mstation>localhost</mstation>
opennms-etc/opennms-activemq.xml:    <broker xmlns="http://activemq.apache.org/schema/core" brokerName="localhost" dataDirectory="${activemq.data}">
opennms-etc/opennms-activemq.xml:            <transportConnector name="vm" uri="vm://localhost"/>
opennms-etc/opennms-activemq.xml:
opennms-etc/opennms.properties:#opennms.alarmTroubleTicketLinkTemplate = ${id}
opennms-etc/opennms.properties:#opennms.alarmTroubleTicketLinkTemplate = ${id}
opennms-etc/opennms.properties:#opennms.alarmTroubleTicketLinkTemplate = ${id}
opennms-etc/opennms.properties:opennms.rtc-client.http-post.base-url = http://localhost:8980/opennms/rtc/post
opennms-etc/opennms.properties:opennms.map-client.http-post.url = http://localhost:8980/opennms/map/post
opennms-etc/opennms.properties:gwt.geocoder.referer=http://localhost/
opennms-etc/opennms.properties:#org.opennms.newts.config.hostname=localhost
opennms-etc/opennms.properties:#org.opennms.newts.config.cache.redis_hostname=localhost
opennms-etc/opennms.properties:#org.opennms.grafanaBox.hostname=localhost
opennms-etc/opennms.properties:#org.opennms.activemq.broker.url=vm://localhost?create=false&jms.useAsyncSend=true
opennms-etc/org.apache.karaf.management.cfg:rmiRegistryHost = localhost
opennms-etc/org.apache.karaf.management.cfg:rmiServerHost = localhost
opennms-etc/poller-configuration.xml:      <parameter key="lookup" value="localhost" />
opennms-etc/provisiond-configuration.xml:


Fix typo in README

Fix a simple typo in the README file. I don't dare to delete and updated, which seems totally wrong in this sentence.

Indeed, thanks for pointing out the typo. The rest of the phrase seems correct to me though?


file_field in form uploads as Http::UploadedFile

I started the app without file, and added an attachment to my model afterwards.  I added the file_field to my form, and when I go to edit my models via the edit action I get this:
ActiveRecord::AssociationTypeMismatch (Nifty::Attachments::Attachment(#70270868766960) expected, got ActionDispatch::Http::UploadedFile(#70270867955480)):
Any thoughts?

The field on the form should be suffixed with _file. If you had this model your field on the form should be named photo_file.
class SomeModel
  attachment :photo
end

Model:
class Event < ActiveRecord::Base
attachment :image
end
View form:

      <%= f.label :image %>
      <%= f.image_file :image %>

Gives me an error: undefined method `image_file'

oh, durr, should be:

  <%= f.label :image %>

  <%= f.file_field :image_file %>

thanks!


Add French ui_ng translation

According to the developper documentation, I submit here the UI translation in French
https://github.com/vmware/harbor/blob/0bec263df767736b150141929d714cfe5b2d8e31/docs/developer_guide_i18n.md
Why do you ignore src/ui_ng/src/**/*.json in .gitignore ?

Coverage decreased (-0.04%) to 59.777% when pulling 3159e2c5140648951b8694f1128577dd09bc6af0 on patfrat:translation/french into be397eb1572902128cd00ffacca53826a5c4d5e7 on vmware:master.


difference-of-squares: renamed square_of_sums to square_of_sum

In exercise difference-of-sums, renamed square_of_sums to square_of_sum and assertSquareOfSums to assertSquareOfSum.

Merged, thanks.

It's a pleasure :)


ntfy app in not visible in the IOS store

:lady_beetle: Describe the bug

:computer: Components impacted

:bulb: Screenshots and/or logs

:crystal_ball: Additional context

The app is not visible in the IOS app store

Problem has been solved it's back!


Fix regression in i2d_re_X509_REQ_tbs()

This fixes regression from commit 8e39049.
Fixes #19297

Just found in crypto/x509/x509cset.c:
int i2d_re_X509_CRL_tbs(X509_CRL *crl, unsigned char **pp)
{
    crl->crl.enc.modified = 1;
    return i2d_X509_CRL_INFO(&crl->crl, pp);
}

that at least for some related data types the modified flag is set before calling i2d_*, apparently to be on the safe side.
Maybe such apparent workarounds are no more needed after the adaptations I added yesterday to #19271 ?

Should this have a unit test?

Should this have a unit test?

It should. However IMO this falls under much bigger effort to write the test than the actual fix. We do not have any unit test for i2d_re_* functions nor for the i2d_X509_REQ() function. I could squeeze it in x509_check_cert_pkey_test.c easily, on the other hand it is a little bit weird to piggy-back on quite unrelated test.

Thanks for your quick works.

Just found in crypto/x509/x509cset.c:
int i2d_re_X509_CRL_tbs(X509_CRL *crl, unsigned char **pp)
{
    crl->crl.enc.modified = 1;
    return i2d_X509_CRL_INFO(&crl->crl, pp);
}

that at least for some related data types the modified flag is set before calling i2d_*, apparently to be on the safe side.

The "re" in i2d_re_X509_tbs stands for "re-encode", and ensures that a
fresh encoding is generated in case the object has been modified after
creation (..snip..)

If, after modification, the X509 object is re-signed with X509_sign(),
the encoding is automatically renewed. Otherwise, the encoding of the
TBSCertificate portion of the X509 can be manually renewed by calling
i2d_re_X509_tbs().

My understanding from reading the above part of manpage is that setting the modified flag before calling relevant functions tells them to invalidate the internal cache, so the renewed TBS portion will be encoded.

Maybe such apparent workarounds are no more needed after the adaptations I added yesterday to #19271 ?

If your work resolves this situation, that will be great, thanks.

More reason to write a test, since it seems 8e39049d38ebe8b8398d6c4aa8a6f7cef9712132 regressed more than just the return code! :-) (It used to be in the right order, but then 8e39049d38ebe8b8398d6c4aa8a6f7cef9712132 broke it.)

More reason to write a test, since it seems 8e39049 regressed more than just the return code! :-) (It used to be in the right order, but then 8e39049 broke it.)

Feel free to write one.

de54030f4e085fe45f03351f944f9eeab9099d9e looks good to me, thanks!

@DDvO still OK?
ping @openssl/otc for otc review.

@DDvO still OK?

Yep

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master and 3.0 branches. Thank you for the reviews.

Hi I will help you with everything you need but you have to subscribe to
the youtube channel and I will fulfill all your requests
https://youtu.be/aIaqr4eq18g
Pada Kha, 29 Sep 2022, 7:02 PM Tomáš Mráz @.***>
menulis:

This fixes regression from commit 8e39049
https://github.com/openssl/openssl/commit/8e39049d38ebe8b8398d6c4aa8a6f7cef9712132
.
Fixes #19297 https://github.com/openssl/openssl/issues/19297
You can view, comment on, or merge this pull request online at:
https://github.com/openssl/openssl/pull/19299
Commit Summary

81de79e
https://github.com/openssl/openssl/pull/19299/commits/81de79eccae90ab2ab3c1ecc72d06f669c69c82f
Fix regression in i2d_re_X509_REQ_tbs()

File Changes
(1 file https://github.com/openssl/openssl/pull/19299/files)

M crypto/x509/x509_req.c
https://github.com/openssl/openssl/pull/19299/files#diff-bc4b7d8fa8518b9f8c75b5a6aded2570d89ba044e7a50066401d6ddc7f119dd1
(6)

Patch Links:

https://github.com/openssl/openssl/pull/19299.patch
https://github.com/openssl/openssl/pull/19299.diff

—
Reply to this email directly, view it on GitHub
https://github.com/openssl/openssl/pull/19299, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APESUCWE5AXUK7M5B24STULWAVZMRANCNFSM6AAAAAAQYUZTNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Can't upload eds_plugin_17.05.kpz from master branch

This is the error message that I'm seeing:
"eds_plugin_17.05.kpz failed to unpack.
Please verify the integrity of the zip file and retry."

Actually, this might be my bad. Going to try again...

Yeah my bad. Bad download.


Double CRC handling

The discussion in https://github.com/jmccrohan/pysolarmanv5/issues/62 reveals that some data loggers/inverters apply double CRC over the modbus frame making it unreadable by the standard libraries. Therefore, some bytes must be removed in order the RTU frame to conform to the protocol.

My tree is messed up too much from what I see. Don't know why all these extra commits are showing up here.

But this way it is going to call the 'parse_response_adu' twice during every struct.error. Isn't that somehow wasteful?

But this way it is going to call the 'parse_response_adu' twice during every struct.error. Isn't that somehow wasteful?

The check by length will reduce the calls. Otherwise, yes, it will be called even when the response is a modbus exception.

Merged, thanks to both of you for your input on this.
I will tidy up the docs and prepare a new release.


[$500] mWeb - Room - The invited users email id is not fully visible in room chat page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.14
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Tap on a room chat
Tap on room header
Tap on members---- Invite
Invite 2 members with long email id

Expected Result:
The invited users email id must be fully visible in room chat page
Actual Result:
The invited users email id is not fully visible in room chat page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/88136130-22fa-4bd5-8b24-5404b6652bc0
https://github.com/Expensify/App/assets/78819774/90e2c610-7441-45ef-83ea-18f2fabf967c
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01118481c486c1f575
Upwork Job ID:<PHONE_NUMBER>105293824
Last Price Increase: 2023-12-19

Proposal
Please re-state the problem that we are trying to solve in this issue.
The long invited message is out of screen.
What is the root cause of that problem?
We display text using RenderCommentHTML here
What changes do you think we should make in order to solve the problem?
I think we need same logic between "workspace added message" and "room invite message".
For this we need to remove codes.
https://github.com/Expensify/App/blob/aa1d87caddb6831061f3750bd78f77b42603e5e3/src/pages/home/report/ReportActionItemMessage.js#L44-L57
What alternative solutions did you explore? (Optional)

It looks like this is intentional because this has a tooltip to view the entire mail:

But if not, here is my proposal:
Proposal
Please re-state the problem that we are trying to solve in this issue.
mWeb - Room - The invited users email id is not fully visible in room chat page
What is the root cause of that problem?
The content of the account/email text is shorter than the container and we are specifying the max line number of 1, the content will never break the line due to this pre style and the next below:
https://github.com/Expensify/App/blob/ab4279b5459916416595ba305e04066a1ba64254/src/components/SelectionList/UserListItem.js#L40
What changes do you think we should make in order to solve the problem?
We can switch the numberOfLines props to change it by 2 when the mouse enters and leave like:
const [alterTextNumberOfLines, setAlterTextNumberOfLines] = useState(1);
.
.
.
   <Text
        style={alternateTextStyles}
        numberOfLines={alterTextNumberOfLines}
        onMouseEnter={() => setAlterTextNumberOfLines(2)}
        onMouseLeave={() => setAlterTextNumberOfLines(1)}
    >
        {item.alternateText}
    </Text>

Also, we should remove the unnecessary prop in:
https://github.com/Expensify/App/blob/c0313c2246608ac095488d0916cd2a22ea5bd610/src/components/SelectionList/BaseListItem.js#L93
Result:
https://github.com/Expensify/App/assets/5216036/c5363828-3847-4760-8612-6a6ff5e7e6be

Reviewing today!

I'm inclined to go with @bernhardoj's proposal as the email is styled as a mention and we already wrap mentions in normal messages:

I don't think in this case we would want to apply it as a general style for BaseHTMLEngineProvider as in @Krishna2323's proposal, as it could cause unexpected results.
:ribbon::eyes::ribbon: C+ reviewed

Yeah I think that's fine! My only note would be to double check those other styles added in chatItemMessage don't change the display other than wrapping, but I don't think they will.
Assigning!

PR is ready
cc: @jjcoffee

@jjcoffee I think we're ready to get the checklist updated and close this out!

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/30927/
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/30927/files#r1456022482
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A - just needed more thorough testing
Determine if we should create a regression test for this bug. Yes
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Open any room chat
Invite 2 users with a very long email
Verify you can see the whole invited message and that the long email text wraps, rather than overflowing the screen

Do we agree 👍 or 👎

Looks good! One tiny edit, just for clarity:

Invite 2 users with very long email addresses (+50 characters)

@muttmuure Friendly bump for payment! :bow:

Oops, sorry!
All paid up


Best approach for updating recommendations?

I've got a website that has lessons, and I want to keep fresh predictions for my users. I base the recommendations on various things. Likes, users watching, related tags, etc and was wondering what is the recommended way to process recommendations to keep them fresh? When should I run recommender.process! vs updating individual lessons on their own? I'm using Heroku, and currently run process! once a day, but the docs don't give any advice in this regard.
Thank you in advance.

Hey Joel,
If architected correctly, recommender.process! should rarely, if ever, be required. If you know the association of an action to a lesson, you should just be able to call recommender.process_item! lesson right after that action occurs. If your dataset is small enough, you can do this in the typical after_create hook (if in Rails) of the like, view, etc. If you're worried about that being slow, then you can just fire a worker up to run the process_item! call.
Hope that helps!

Thanks! I am doing it correctly, just wanted to be sure. We really appreciate this tool.

On Jan 24, 2015, at 13:26, Jamie Davidson<EMAIL_ADDRESS>wrote:
Hey Joel,
If architected correctly, recommender.process! should rarely, if ever, be required. If you know the association of an action to a lesson, you should just be able to call recommender.process_item! lesson right after that action occurs. If your dataset is small enough, you can do this in the typical after_create hook (if in Rails) of the like, view, etc. If you're worried about that being slow, then you can just fire a worker up to run the process_item! call.
Hope that helps!
—
Reply to this email directly or view it on GitHub.

Awesome, glad to hear it Joel!
On Sat, Jan 24, 2015 at 5:52 PM Joel Hooks<EMAIL_ADDRESS>wrote:

Thanks! I am doing it correctly, just wanted to be sure. We really
appreciate this tool.

On Jan 24, 2015, at 13:26, Jamie Davidson<EMAIL_ADDRESS>wrote:
Hey Joel,
If architected correctly, recommender.process! should rarely, if ever,
be required. If you know the association of an action to a lesson, you
should just be able to call recommender.process_item! lesson right after
that action occurs. If your dataset is small enough, you can do this in the
typical after_create hook (if in Rails) of the like, view, etc. If you're
worried about that being slow, then you can just fire a worker up to run
the process_item! call.
Hope that helps!
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/Pathgather/predictor/issues/26#issuecomment-71341643.

Hi,
I'm having a bit of a problem related to this.
Updating a single record doesn't stand in direct relation (time wise) with updating the whole database. I currently have about 400k records in my engine. Running Recommender#process! takes about 14 min but updating a single record 'on-the-fly' using Recommender#process_items! takes 30 seconds. Is it suppose to take this long? From my benchmarks it would be much more beneficial (time wise) to just re-run Recommender#process! each night instead of doing Recommender#process_items! on each item for ~ 3000 * 30 sec every day (3000 == the amount of records that has to be updated every day), something that would take way to long time.
I'm I missing something?

Hi, sorry for the late reply, things have been a bit crazy around here recently :smile:. The time taken to process an item grows exponentially with the number of relations it has to other items, so if there are a few items in your dataset with many connections, it's expected that processing those items in particular will take a while. Is the 30 seconds figure the worst case or the average case that you've seen when processing individual items?
You might also try the :union processing technique if you haven't already - it's much faster than the :ruby (default) or :lua techniques, and the results it gives are roughly the same. See the README for more info on that.


Remove Cancel Workflow Action

What internal change would you like to have?
It doesn’t seem it makes sense to use the Cancel Workflow Action in combination with the new GitHub merge queue.
Let’s remove it.
What problem will this change solve?
Avoid cancelling pipelines for main branch when using the new merge queue, which I enabled for this repo.
Do you want to help make the feature?
Yes
Is there anything else we need to know?
No response

Are you sure Merge Queue is working for you? I don't see any merges via the queue: https://github.com/octoclairvoyant/octoclairvoyant-webapp/queue/main
From reading the GitHub Docs, it looks like you need to add merge_group: to the GitHub Action files, so it runs the tests on the merge queue branches. ^docs
on:
  pull_request:
  merge_group:

@HonkingGoose I actually wasn't sure if some extra config was needed, so I wanted to take advantage of this PR to investigate it. Thanks for pointing the missing set up! I'll make sure it is configures properly when I open the PR.

I've written the documentation to get Renovate and GitHub Merge Queue working, see:
https://docs.renovatebot.com/key-concepts/automerge/#github-merge-queue

@Belco90 I see you got the GitHub Merge Queue working on the repository, at least when you're manually adding Renovate PRs to the queue.

Yes! Renovate docs really helped me with it.


Unable to Reveal in an Async session

After enough votes have rolled in, I'm trying to reveal everyone's votes so we can settle on an estimate, and am getting the following errors as shown in the browser console.

When this happens, the entire screen just blanks, and this is blocking the estimation process :(

As a follow up ... after narrowing it down, it seems like this was an issue with only one particular User Story. Not sure what may have been different about that particular story, but I just manually added the story points outside of Estimate, and everything else has been working now.

Hi @joelmartinez  thanks for reporting, we released an update today, try again and let me know if you still facing the same issues.


Disable DevTool in production?

Hello, I looked everywhere and couldn't find anything about it:
Is there any way to disable the DevTool in production?
PS: jumpsuit.io doesn't work anymore, but you probably know that

As long as the env.production variable isn't set in your build tool, the dev-tools shouldn't even be included in the production build.
Remember, jumpsuit build will build a production version of your app.

I'm using my own build system, not the jumpsuit one. Does that make any difference?

Yes it would.  But, as long as the NODE_ENV environment variable is set to production, it won't render the dev tools.
https://github.com/jumpsuit/jumpsuit/blob/master/src/render.js#L31

Try upgrading to 2.0 and this should be fixed :)


Run cargo fmt in CI

We just added a config file for cargo fmt but I forgot to enable the formater in CI.

Patch 1: Run the formatter to fix current formatting issues
Patch 2: Fix the logic around DO_BENCH (part of this PR because patch 3 uses the same code)
Patch 3: Add env var DO_FMT and guard on local variable NIGHTLY, runs formatter in CI
Patch 4: Update docs re cargo fmt
EDIT: Patch 5: Add a fmt pre-commit hook

Note
This PR is going to be annoying to merge. Right before merging we want to make sure there are no formatter issues (i.e., this likely needs to be rebased right onto tip of master before merging), otherwise open PRs will have to be rebased and formting fixes pre-pended to them. Its not a big deal if its only my PRs, but I don't want to cause drama for others.

I ran shellcheck on this branch and found a single bug, fixed and pushed as a separate patch.

You know it might actually be better to hold off on merging this until we have stopped ignoring submodules, that way when we are moving things around we can move them in a separate commit to formatting them and still have every commit pass CI.

I think, no matter what we'll have to adjust the ignore rules while moving stuff. Enforcing it in CI, vs on people's local machines (where I believe some contributors have configured their editors to auto-mangle code according to rustfmt) doesn't seem to make a lot of difference.

to auto-mangle code according to rustfmt

I had to laugh at this one :)

Changes in force push:

Rebase (remove changes to the ci script that have been merged in other PRs)

Rebased to pick up formatting fixes just merged. No other changes.

Rebase and add initial patch to fix up trivial formatting issue.

Changes in force push:

Rebased and added an initial patch that runs the formatter
Reworded commit 521d4688 Improve the DO_BENCH error message (just removed ci:  prefix from the brief description).

Needs rebase :)

Changes in force push:

rebase and redo first patch to include the few trivial formatting issues currently on master.

@apoelstra (or @sanket1729), in case you are wondering, this can be merged any time - don't worry about conflicts with other formatting things, I can rebase.
Changes since @Kixunil last ack'ed are rebases and some changes to the bitcoin/contrib/test.sh script. I believe all review suggestions have been addressed.

Gentle bump please team, lets get this in to stop the formatting issues from continually creeping into master.

Force push is just rebase and redo of the initial "run formatter" patch.

Changes in force push:

Rebased
Added fixes to new formatting issues in patch 1
Fixed the gre -> grep

IDK what to make of this asan failure. It doesn't have enough information to zero in on it, and the error message suggests it was a fluke or a bug in our tooling ... but I still don't feel comfortable resetting CI.

I am not able to reproduce this locally. I did try running DO_ASAN=true ./contrib/test.sh inside rust-bitcoin/hashes 10 times.

I just kicked it. Let's not knock ourselves out tracking down a probably-spurious asan error in safe code with no backtrace information in a PR that has nothing to do with it.

I also can't reproduce it locally, even with the latest nightly (from last night).

I cannot reproduce locally either, I checked all tool versions were the same and ran the memory sanitizer command in a loop a bunch of times.

BOOM, reproduced it! I ran the following in bitcoin not in hashes
 CC='clang -fsanitize=memory -fno-omit-frame-pointer'                                         \                                                                         ✘ 1
  RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes'   \
  cargo +nightly test --lib --no-default-features --all-features -Zbuild-std --target x86_64-unknown-linux-gnu

Exactly why the RUSTFLAGS from hashes build are being used to run tests in bitcoin I haven't worked out though ...

I've tracked it down to libc
$ pwd                                                                                ✘ 101
/home/tobin/build/github.com/rust-lang/libc
$ cargo clean; CC='clang -fsanitize=memory -fno-omit-frame-pointer' RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes' cargo +nightly test
   Compiling libc v0.2.139 (/home/tobin/build/github.com/rust-lang/libc)
error: failed to run custom build command for `libc v0.2.139 (/home/tobin/build/github.com/rust-lang/libc)`

Caused by:
  process didn't exit successfully: `/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build` (exit status: 1)
  --- stdout
  cargo:rerun-if-changed=build.rs

  --- stderr
  ==148088==WARNING: MemorySanitizer: use-of-uninitialized-value
      #0 0x5586dd2ffa72  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0x9ea72) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #1 0x5586dd2fd1cc  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0x9c1cc) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #2 0x5586dd30e1f3  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xad1f3) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #3 0x5586dd30668c  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xa568c) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #4 0x5586dd30d4f8  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xac4f8) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #5 0x5586dd33cb6b  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xdbb6b) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #6 0x5586dd30d309  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xac309) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #7 0x5586dd3031be  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0xa21be) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
      #8 0x7f0534ab4d8f  (/lib/x86_64-linux-gnu/libc.so.6+0x29d8f) (BuildId: 69389d485a9793dbe873f0ea2c93e02efaa9aa3d)
      #9 0x7f0534ab4e3f  (/lib/x86_64-linux-gnu/libc.so.6+0x29e3f) (BuildId: 69389d485a9793dbe873f0ea2c93e02efaa9aa3d)
      #10 0x5586dd271874  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0x10874) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)

    Uninitialized value was created by an allocation of '_2' in the stack frame of function '_ZN18build_script_build19rustc_minor_nightly17ha05947cc82f4f00dE'
      #0 0x5586dd2ff680  (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0x9e680) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)

  SUMMARY: MemorySanitizer: use-of-uninitialized-value (/home/tobin/.cache/cargo/debug/build/libc-a5354087f4621f19/build-script-build+0x9ea72) (BuildId: bbbfe69bfe3d26afe9b2499f55a0b80631b2071a)
  Exiting

Raised an issue on libc: https://github.com/rust-lang/libc/issues/3075

Sanitizer fixed in #1566

Is this 0.30 blocking? We can easily merge formating changes post release.

Not blocking 0.30.0 - I was hoping to slip this through at some stage, ether after release like you say or right at the end of the feature freeze before release if the remaining PRs to merge are only mine (so I'm the only one to have to go through the rebase pain :)

Rebase only, no other changes.

Rebased on top of last nights merges. Added new formatting changes to commit: c1220e26 Run the formatter

Rebased so I can use this as the base of #1434 still. No changes needed.

Lets push this one in crew, there have been plenty of acks in the past so the concept is good. Its only commit 6ee234cb Run the formatter that has change with all the force pushes.

IMO we should at least get #1533 in before this. I'd hate to re-review it after formatting changes.

Sure thing, my bad I didn't think of that one.

Draft till #1533 goes in

Needs rebase, then let's gooooo.

Rebased and re-did a11cf075 Run the formatter

This only enables the formatter in CI for bitcoin, not the other crates. Is that intentional?

This only enables the formatter in CI for bitcoin, not the other crates. Is that intentional?

Not intentional, my mistake. Fixed and force pushed. FTR we still ignore hashes in rustfmt.toml  - that is intentionally not fixed here.
Thanks

Looks good -- though we're still not covering internals. I'd suggest adding the formatting code to the top-level contrib/test.sh rather than to the individual crates' contrib/test.shs.

Is this the only 0.30.0 blockcing PR?

Please see https://github.com/rust-bitcoin/rust-bitcoin/pull/1699 for release blocking PRs

Looks good -- though we're still not covering internals.

Sure we are, its in c1360067 Enable formatting in CI
I'd suggest adding the formatting code to the top-level contrib/test.sh rather than to the individual crates' contrib/test.shs.
I'm not comfortable doing that for this PR because currently the rust-bitcoin/contrib/test.sh is just a thin wrapper. If we want to put othe logic into it we should put all the logic that makes sense into it.

We have an old ACK from Kix and we've merged everything we wanted to merge before this. I'm gonna do it.


Facebook Messenger?

When will AdaptiveCards be available for Facebook Messenger?

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

ID: 87d1e8a9-5bad-9e08-a895-ad222e72d4dd
Version Independent ID: 96e622ee-eea0-c487-e453-e1aad2cda1dd
Content: Adaptive Cards Overview - Adaptive Cards
Content Source: AdaptiveCards/index.md
Product: adaptive-cards
GitHub Login: @matthidinger
Microsoft Alias: mahiding

You should see adaptive cards get rendered to an image on FB Messenger, so as long as you don't use input fields, and some types of actions then your users should see the gist of your adaptive card. Is this working for you today? Unfortunately we would need Facebook to natively support adaptive cards for a better experience. Feel free to reach out to them and suggest it, we'd love to work with them on it :)


Actions.pop({ popNum: number }) doesnt work anymore

Version
Tell us which versions you are using:

react-native-router-flux v4.0.0-beta.21 (v3 is not supported)
react-native v0.48.3

Expected behaviour
Actions.pop({ popNum: 2 }) goes 2 scenes back
Actual behaviour
Actions.pop({ popNum: 2 }) goes only one scene back

Try Actions.popTo('scene_1') instead (Pops the stack until the)
https://github.com/aksonov/react-native-router-flux/blob/master/docs/API.md#actions

Thx. But can you add more

@onrige your suggestion did work for me, thank you.
I do think that the documentation is incorrect - it states:

Actions.pop() will pop the current screen. It accepts following optional params:
{popNum: [number]} allows to pop multiple screens at once

But in my tests, and as reported by @mmaschenko, passing popNum does not work:
Actions.pop({popNum: 2}) .  // Only pops back one screen

@guarani Could you give me the link to this docs? I can't find anything in current docs (v4) you talk about, sorry.

Your 100% right, I was looking at (v3) docs and using (v4) 🙃, sorry.

how did you implement popNum functionality in v4?

@SaffaShujah
pop: function (props) {
const {popNum} = props || {}
if (popNum > 1) {
const routes = Actions._state && Actions._state.routes
if (routes && routes.length > popNum) {
const route = routes[routes.length - popNum - 1]
//v4 not support props refresh, not sure futrue
Actions.popTo(route.routeName, props)
if (Object.keys(props).length > 1) {
Actions.refresh(props)
}
}
} else {
Actions.pop()
if (props) {
Actions.refresh(props)
}
}
},


Log files included in 4.1.0 release

It looks like a number of log files that should be .gitignore'd (in modules/docker/log) have been inadvertently included in the released version.

thanks for the heads up. I'll look to release a 4.1.1 with these removed. Being included in .gitignore isn't enough I don't think for puppet module tool to ignore them.

I've released a 4.1.1 to the Forge. Thanks for the heads up.


EMR - Hail - Access to studies

Hi,
I deployed SWB v3.0.0
I am testing the workspace EMR shipped by default with SWB.
I am able to launch the EMR with "Genome Aggregation Database (gnomAD)" study linked to the workspace.
I am able to connect to the jupyter notebook, launch Hail, load and view the examples data
I am not able to load data from a study into hail
I can see the directories in jupyter (on the left side bar)
I can browse into the directories using shell (cd, ls...)
But when I try to read files into hail, I got errors:
ht = hl.read_table('studies/broad-gnomad/release/3.1.1/ht/genomes/gnomad.genomes.v3.1.1.sites.ht/')

Error summary: HailException: MatrixTable and Table files are directories; path 'studies/broad-gnomad/release/3.1.1/ht/genomes/gnomad.genomes.v3.1.1.sites.ht/' is not a directory

mt = hl.import_vcf('studies/broad-gnomad/release/3.1.1/vcf/genomes/gnomad.genomes.v3.1.1.sites.chr21.vcf.bgz', reference_genome='GRCh38')

Hail: WARN: 'studies/broad-gnomad/release/3.1.1/vcf/genomes/gnomad.genomes.v3.1.1.sites.chr21.vcf.bgz' refers to no files

Also, when I load example data from hail, that works but I do not see the data in my jupyter notebook. i.e. I do not know where the data are loaded.
hl.utils.get_1kg('data/')
mt = hl.read_matrix_table('data/1kg.mt/')

Hi,
try using the full path to the files:
ht = hl.read_table('file:///home/hadoop/studies/broad-gnomad/release/3.1.1/ht/genomes/gnomad.genomes.v3.1.1.sites.ht/')
mt = hl.import_vcf('file:///home/hadoop/studies/broad-gnomad/release/3.1.1/vcf/genomes/gnomad.genomes.v3.1.1.sites.chr21.vcf.bgz', reference_genome='GRCh38')

note that studies is a symlink to /home/hadoop/studies
[hadoop@ip-10-0-15-103 notebook]$ pwd
/opt/hail-on-AWS-spot-instances/notebook
[hadoop@ip-10-0-15-103 notebook]$ ls -l
total 160
-rwxrwxrwx 1 hadoop hadoop 25982 Apr 29 16:13 GWAS_tutorial_with_HMS_additions.ipynb
-rw-r--r-- 1 hadoop hadoop 91672 Apr 29 21:02 hail-20210429-2043-0.2.53-8140f17d9262.log
-rwxrwxrwx 1 hadoop hadoop 35807 Apr 29 16:13 plotting.py
lrwxrwxrwx 1 root   root      20 Apr 29 20:41 studies -> /home/hadoop/studies

Unfortunately, your read_table command will fail with:
Error summary: HailException: incompatible file format when reading: file:///home/hadoop/studies/broad-gnomad/release/3.1.1/ht/genomes/gnomad.genomes.v3.1.1.sites.ht/
  supported version: 1.4.0, found 1.5.0

Older releases such as 3.0 work fine:
ht = hl.read_table('file:///home/hadoop/studies/broad-gnomad/release/3.0/ht/genomes/gnomad.genomes.r3.0.sites.ht/')


Make Loki as an optional

Observatorium is a unified service to alert and troubleshoot infrastructure. It combines observability signals such as      metrics, logs, traces, profiles and more in a single experience to reduce mean time to resolution.

In my understanding, the metrics, logs, traces, profiles etc are the components of observatorium. they should not be designed as required components. they are should be optional. we should have one component enabled at least in order to install observatorium operator.
@brancz @squat any comments? Thanks.

Hm, eventually we will need some sort of requirements on signals to be able to correlate properly. For now, having them optional is fine.


Support the usage of private Apply() methods in self-aggregators

Boo!

Go back to the AggregateRepository example and make the Apply() methods private to test this.


Second VM: calling contracts part 2

With #4248 new host functions were introduced with a new method of passing a “manifest” from wasm to host. Need to still continue work on calling already stored contracts.

casper_create -> Maybe upgrade can be merged into create
casper_call -> call smart contract aka the entity
casper_transfer -> transfer of tokens to an account


Rueidis adds 44 MB to build binary

This eclipses every single other module my project is using combined.
Anything we can do about this?

Hi @nightwolfz,
The internal/cmds contains the generated command builder. It is indeed very big due to the complexity of the redis command syntax.
However, normally you won't use all its generated code, and therefore the binary size after compilation won't be such big.
For example, the real binary size of the following program is 5.1MB on my laptop while the goweight still says that it should be at least 44MB:
package main

import (
	"context"
	"github.com/rueian/rueidis"
)

func main() {
	c, err := rueidis.NewClient(rueidis.ClientOption{})
	if err != nil {
		panic(err)
	}
	c.Do(context.Background(), c.B().FtSearch().Index("").Query("").Build())
}

Please also check this https://github.com/jondot/goweight/issues/16

Hi @nightwolfz,
Is the size problem still bothering you?

after import rueidis, the binary size add more than 10MB, and i have used -ldflags=-w build flags
without rueidis
-rwxr-xr-x   1 shushenghong  staff    29M  3 29 15:42 main

with rueidis
-rwxr-xr-x   1 shushenghong  staff    38M  3 29 15:40 main

Hey there!
First of all, thanks for library. I've started migrating anycable-go to use ruedis and recently discovered that binary sizes increased by ~8MB (you can see the difference in the releases, between 1.3.0 and 1.4.0, I tested locally to confirm it's due to ruedis). I see a similar to above picture when running go-binsize-treemap, too:

I can't say going from 13MB to 21MB is a big concern, but it's noticeable.

normally you won't use all its generated code, and therefore the binary size after compilation won't be such big.

It doesn't seem that Go so smart to eliminate some of the code loaded from internal/cmds; it either all or nothing. I found that the following minimal rueidis usage results in the same binary size increase:
package pubsub

import (
	"sync"
	"github.com/redis/rueidis"
)

type RedisSubscriber struct {
	client        rueidis.Client
	clientOptions *rueidis.ClientOption
	clientMu      sync.RWMutex
}

func NewRedisSubscriber(args ...interface{}) (*RedisSubscriber, error) {
	return &RedisSubscriber{}, nil
}

func (s *RedisSubscriber) initClient() error {
	s.clientMu.Lock()
	defer s.clientMu.Unlock()

	if s.client != nil {
		return nil
	}

	c, err := rueidis.NewClient(*s.clientOptions)

	if err != nil {
		return err
	}

	s.client = c

	return nil
}

(Compiled with -ldflags="-s -w", of course)


Project management

Brief Description
use poetry to manage python client project & use pytest for python client test

How Did You Test This Change?

Please fix the CI issues.


Ianb/datadog deploy only

What this PR does / why we need it:
This PR adds an option to include a Deployment-only confd (datadog.deploy_confd) and checksd (datadog.deploy_checksd).  Currently, the both are inherited by the daemonset and deployment kinds, resulting to duplicated checks and metrics if checking an external endpoint e.g. http_check
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

not sure re versioning
will link this to a card.
@hkaj @xvello @irabinovitch

workaround: Run separate releases (multiple helm install) - one for the ds and another for the deployment.


Don't enable Matomo by default

https://groups.google.com/g/islandora/c/MtHOxjew-tA
Stop installing Matomo by default in ISLE-DC, while leaving enough infrastructure in place that someone could turn it on, if desired.

From the tech call: Ensure there's a minor version minted here to denote that matomo will be gone.

Have a PR started for this here: https://github.com/Islandora-Devops/isle-dc/pull/396

From the tech call: Ensure there's a minor version minted here to denote that matomo will be gone.

We don't have any semantic versioning in this repo, so not sure how best to communicate this change. I feel like that PR only effects new sites, so not sure it needs any sort of major/minor bump anyway.


raise exception when dims is None in average_over_dims

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes issue https://github.com/roocs/rook/issues/135

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)
[ ] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:
An exception is now raised when dims is None. If dims is an empty string it will be converted to None by dims = DimensionParameter(params.get("dims", None)).tuple

Does this PR introduce a breaking change?:
Yes - raises an exception where it previously didn't

Other information:
I will update the tests on daops and rook as well

@aulemahal FYI (for FInch)


Add UML to repo

The UML architecture diagrams need to be in the repo so we can track changes

Can't keep the repos as clones so will need to duplicate them


Need a limit on per-app state tree size

We need a limit on per-app state tree size to avoid malicious state tree size increase.
Offline discussion:

MAX_STATE_TREE_SIZE_PER_BYTE or MIN_BYTES_PER_STATE_NODE to get allowed max per-app state tree size from per-app state tree bytes

Implemented in https://github.com/ainblockchain/ain-blockchain/pull/504


TypeScript to OpenApi

I would like to serve multiple versions of an api from a single web service. I also don't want to write OpenApi schemas. I want to write TypeScript, then I want to convert that to OpenApi. I am able to convert from TS to OpenApi (thanks!), however, I need to support multiple schema versions. If my project dirs look something like this...
.
`-- src/
    |-- business/
    |   `-- dtos/
    |       |-- v1/
    |       |   `-- index.ts
    |       `-- v2/
    |           `-- index.ts
    `-- customer/
        `-- dtos/
            |-- v1/
            |   `-- index.ts
            `-- v2/
                `-- index.ts

then i want the schema to look something like
{
    ...
    components:{
        schemas: {
            v1: {
                ...
            },
            v2:{
                ...
            }
        }
    }
}

extending the example above, one of the index.ts files may have interfaces like...
export interface Customer {
    name: string;
    creditCardNumber: string
    socialSecurityNumber: string
    streetAddress: string
    passportId: string
    dateOfBirth: string
    isHouseKeyUnderMat: boolean
}


ALARM-MIB

Please add the following MIB:
ALARM-MIB.zip
The MIB module describes a generic solution   to model alarms and to store the current list  of active alarms.
 The   initial version of this MIB module was published   in RFC 3877.  For full legal notices see the RFC            itself.  Supplementary information may be available on:            http://www.ietf.org/copyrights/ianamib.html"
  REVISION    "200409090000Z"  -- September 09, 2004
  DESCRIPTION
      "Initial version, published as RFC 3877."

Issue #134 updated instead


crash chemical reactor

Minecraft Version:
1.12.2
Forge Version:
<IP_ADDRESS>5
Advanced Rocketry Version:
1.2.6-33
LibVulpes Version:
0.2.7-19-universal
https://hastebin.com/enunajiqiv.css
Explain issue in detail:
Crash when start filling fluid input - every time.

https://hastebin.com/jacewutala
Basically the same bug but some more info:
Filled Oxygen and Hydrogen from Nuclearcraft into the Fluid Inputs, Fluid Inputs correctly display Oxygen and Hydroxen inside. Turned machine on, instant crash of both client and server.
The worst part is that the server crashes instantly on start now because the machine is still turned on, which means you have to restore an older backup.
This also means you cannot progress in Advanced Rocketry because Rocket Fuel cannot be made.
Minecraft Version:
1.12.2
Forge Version:
<IP_ADDRESS>62
Advanced Rocketry Version:
AdvancedRocketry-1.12.2-1.2.6-5
LibVulpes Version:
LibVulpes-1.12.2-0.2.7-19-universal

Can you upload a copy of the config/advRocketry/ChemicalReactor.xml file?

#982

config/advRocketry/ChemicalReactor.xml file and a copy of the fml-server-latest.log or fml-client-latest.log?

Already - archive in elevator issue.
If you update mods - world will crash.

Hello. I just found this glitch (as soon as I started to pump oxygen in the chemical reactor, the MC crash and render my world unloadable... basically I have to restore a backup... and stuck with this great mod!) I downloaded the latest version not to avail : AdvancedRocketry-1.12.2-1.2.6-37, LibVulpes-1.12.2-0.2.7-20-universal.
I'm using the electrolyzer from this mod (I do have mekanism but opted to use the AR machine to avoid conflicts), the electric pump form mekanism and fluid ducts from Thermal Expansion/Dynamics (because I can cover them :-). If you need anything just let me know.

Here an excerpt from the crash log :
Time: 12/16/17 4:07 PM
Description: Exception in server tick loop
java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
at java.util.ArrayList.rangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)
at zmaster587.advancedRocketry.tile.multiblock.machine.TileChemicalReactor.onInventoryUpdated(TileChemicalReactor.java:73)
at zmaster587.libVulpes.tile.multiblock.hatch.TileFluidHatch.fillInternal(TileFluidHatch.java:129)
at zmaster587.libVulpes.tile.multiblock.hatch.TileFluidHatch.fill(TileFluidHatch.java:98)
at zmaster587.libVulpes.cap.FluidCapability.fill(FluidCapability.java:23)
at cofh.thermaldynamics.duct.fluid.DuctUnitFluid.transfer(DuctUnitFluid.java:177)

I am also getting this crash , Same as above.
Crashes the server, and client in single player mode.
Can provide more info if needed.

Same for me, on advanced rocketry 1.2.6-37.jar  and forge-1.12.2-<IP_ADDRESS>70
I've got a pastebin if useful
https://pastebin.com/N9re1qNv

Should be fixed next build with commit in #997


Rename desc() to reverse_order()

@joneslee85 This seems fair to me, but before to merge we should adjust API docs:

#reverse_order is @since x.x.x
alias_method is @since 0.1.0
Add examples to document both asc and desc

There is an error here: https://github.com/lotus/model/commit/c800a5c6c52a435266776af2d4908c2c1e6b37d1#diff-bcb6af97358a75932ec1322c584f14abR317
Once these issues are solved :+1:


ConcFilter - subquery char. escaping

I've been able to pass the quote character when entering "doručovatel" in the CNC public testing version.

Works fine on the testing site.


CPU Bound Issue in Parser with Complex Grammar (Possible Error with handling of Zero-Width/empty Strings)

The bug
For complex grammar, the generation becomes CPU bound and doesn't terminate. My guess is that the problem lies with empty or zero-width strings not being properly handled by the parser. By line profiling  I see that all of the time is spent in these lines:
https://github.com/guidance-ai/guidance/blob/c9e71fbc3a434eab5209c31735d72fd7e60876f2/guidance/_parser.py#L191-L195

To Reproduce
I use this model and this code:
https://huggingface.co/cognitivecomputations/dolphin-2.9-llama3-8b
https://gist.github.com/bytemouse/6b8eaa647840c3793d5a4f23516b2a5f
System info
OS: Fedora 40
Guidance Version: 0.1.15

Thank you so so much for reporting this, and for sharing your grammar to help us reproduce! Would you mind checking if the issue existed on Guidance v.0.1.14? We introduced some richer support for zero-width strings in this recent release and it'd be helpful to know if it may be the culprit.

I tried the other 0.1.14 and the same error error persists with the same prompt and others. it always seems to 'crash' on the last closed curly brace as in this example:
package cd4code.lighting;

class CarLighting {

  class Light {
    boolean on;
  }

  class HeadLight extends Light {
    double brightness;
  }

  class TailLight extends Light {
    double blinkRate;
  }

The problem now seems to be shifted in the other version? Thank you for your fast response.

Method
Calls
Time (ms)
Own time (ms)

_compute_children
13,070,618
139,349
126,432

_inner_loop
1,701
105,906
55,044

<built-in method builtins.len>
563,931,597
37,746
37,698

__eq__
128,219,828
25,312
15,477

<built-in method builtins.isinstance>
155,682,311
13,623
12,804

_pre_process_regex
543,995
16,668
9,877

replace_grammar_node
20,817
11,413
6,110

@bytemouse you did indeed uncover a bug here, but here's a simple workaround for you for the moment. "Works on my machine", so your mileage may vary, but why don't you give this a try:
@guidance(stateless=True)
def NAME(lm):
    return lm + zero_or_more(WS()) + gen(regex='[a-zA-Z_\$][a-zA-Z_0-9\$]*')

The library we are using to parse regular expressions doesn't correctly escape special characters inside of character classes, so you have to do that manually for now (note that my only change above from your code was escaping the dollar signs).

This did resolve the issue. Thank you for your help!


Add Changelog entry for new NodePattern operator ` [#7693]

I forgot to include this in my original PR. Thanks

Thanks!


not_in overrides &&

Category.where{(id == 7) && (id.not_in([1,2,3]))}.debug_sql
# => "SELECT categories.* FROM categories WHERE categories.id NOT IN (1, 2, 3)"

You want &, not &&. :)
On Nov 15, 2011, at 6:04 PM, Jay Levitt wrote:

Category.where{(id == 7) && (id.not_in([1,2,3]))}.debug_sql
=> "SELECT categories.* FROM categories WHERE categories.id NOT IN (1, 2, 3)"

Reply to this email directly or view it on GitHub:
https://github.com/ernie/squeel/issues/78

GAH!


ISSUES-2906 support join for table function

#2906
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Ok.
PS. Non-obvious usage of Context remains as technical debt.


Can't play audible book

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
Command Name:   "Alexa, paly book on audible"
Reply:  "sorry, I have trouble playing book"
What is the expected behavior?
The device should play an audiobook from Audible, and its name is "The Best Dog Tricks".
What behavior are you observing?
There was something wrong with GStreamer, the log showed missing some a plug-in.
Error log:
2019-09-16 02:12:34.259 [ 38] 9 PlaylistUtils:bodyReceived
2019-09-16 02:12:34.260 [ 38] 9 PlaylistUtils:readFromContentFetcher:readStatus=OK
2019-09-16 02:12:34.260 [ 38] 9 PlaylistUtils:readFromContentFetcher:readStatus=OK_WOULDBLOCK
2019-09-16 02:12:34.260 [ 38] 9 PlaylistUtils:readFromContentFetcher::alreadyReadAllBytes
2019-09-16 02:12:34.260 [ 38] 9 PlaylistUtils:readFromContentFetcherDone:URL=https://audiblecdn-vh.akamaihd.net/i/295891/audiblewords/content/bk/acx0/100499/V$440220$V/aax/bk_acx0_100499_22_32.mp4/master.m3u8?start=16&end=474&hdnea=st=1568599951~exp=1568600011~acl=/i/295891/audiblewords/content/bk/acx0/100499/V$440220$V/aax/bk_acx0_100499_22_32.mp4/~data=16,474~hmac=b435b89d898499538240dfa609f008b714e0bbf9fd4da5f97159dd2aafc25d3d,content=#EXTM3U
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=30000,CODECS="mp4a.40.2",CLOSED-CAPTIONS=NONE
https://audiblecdn-vh.akamaihd.net/i/295891/audiblewords/content/bk/acx0/100499/V$440220$V/aax/bk_acx0_100499_22_32.mp4/index_0_a.m3u8?start=16&end=474&hdntl=exp=1568686354~acl=%2fi%2f295891%2faudiblewords%2fcontent%2fbk%2facx0%2f100499%2fV$440220$V%2faax%2fbk_acx0_100499_22_32.mp4%2f~data=hdntl,16,474~hmac=7b4582b463fd27b871a937925571829bd77b9dae031dbf87ace136b517522cd4
2019-09-16 02:12:34.260 [ 38] 9 LibCurlHttpContentFetcher:~LibCurlHttpContentFetcher
2019-09-16 02:12:34.262 [ 38] 9 PlaylistParser:isM3UExtendedPlaylist
2019-09-16 02:12:34.265 [ 38] 9 PlaylistParser:foundChildrenURLsInPlaylist:num=1
2019-09-16 02:12:34.265 [ 38] 9 PlaylistParser:encounteredMasterPlaylist
... ...
2019-09-16 02:12:35.418 [ 12] 9 MediaPlayer:messageReceived:name=AudioMediaPlayer,type=error,source=decoder
2019-09-16 02:12:35.419 [ 12] E MediaPlayer:handleBusMessageError:name=AudioMediaPlayer,source=decoder,error=Your GStreamer installation is missing a plug-in.,debug=gstdecodebin2.c(4600): gst_decode_bin_expose (): /GstPipeline:audio-pipeline/GstDecodeBin:decoder:
no suitable plugins found:
Missing decoder: MPEG-2 Transport Stream (video/mpegts, systemstream=(boolean)true, packetsize=(int)188)
2019-09-16 02:12:35.419 [ 12] 0 MediaPlayer:callingOnPlaybackError:name=AudioMediaPlayer,type=MEDIA_ERROR_INTERNAL_DEVICE_ERROR,error=Your GStreamer installation is missing a plug-in.,currentId=10
2019-09-16 02:12:35.420 [ 12] 0 AudioPlayer:onPlaybackError:type=MEDIA_ERROR_INTERNAL_DEVICE_ERROR,error=Your GStreamer installation is missing a plug-in.,id=10
2019-09-16 02:12:35.420 [ 12] 9 MediaPlayer:tearDownTransientPipelineElements:name=AudioMediaPlayer
2019-09-16 02:12:35.420 [ 12] 9 MediaPlayer:getCurrentStreamOffsetCalled:name=AudioMediaPlayer
2019-09-16 02:12:35.421 [ 12] E MediaPlayer:getCurrentStreamOffsetFailed:name=AudioMediaPlayer,reason=getElementGetStateFailure
2019-09-16 02:12:35.425 [ 11] E AudioPlayer:executeOnPlaybackError:id=10,type=MEDIA_ERROR_INTERNAL_DEVICE_ERROR,error=Your GStreamer installation is missing a plug-in.
2019-09-16 02:12:35.426 [ 11] 5 ProgressTimer:stop
2019-09-16 02:12:35.432 [ 11] 9 ProgressTimer:setState:state=INITIALIZED,newState=STOPPING
2019-09-16 02:12:35.432 [ 11] 9 ProgressTimer:setState:state=STOPPING,newState=IDLE
End of Error LOG#############
audible_book_log.txt
Tell us about your environment:
avs-device-sdk 1.13
Tell us about your OS (Type & version):

[ ] Linux -- openwrt OS

Hi,
It appears that you are missing a plugin for G-Streamer. Can you confirm that you have installed all the G-Streamer plugins according to the wiki?
https://github.com/alexa/avs-device-sdk/wiki/Dependencies#mediaplayerinterface

@womw  Thanks for your feedback, i have installed almost all the G-Streamer plugins, but the error appeared again.  It must something wrong with my configuration.

I did not install some gstreamer plungins about MPEG-2, it worked well now.


SharePoint Site permissions for Partner Delegated admins

SharePoint Admin Center classic, did have option "Add Support Partner", this option is not available in modern SharePoint Admin Center.
This page does not provide information how to setup in a new way. It also referes to the Set-SPOUser powershell which also does not provide information.
Please update documentation / page, on how to setup new support partner / Partner delegated permissions on site level, weather via interface or via powershell.
Also, please include way / example when using GDAP and DAP examples.
Thank you,
Kind regards

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

ID: cd224d48-9dab-31ae-6548-2e929bd62d7c
Version Independent ID: 6a98ec78-5f74-266f-61a5-95dfdcf673e9
Content: Find site collection features in the new SharePoint admin center - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/site-collections-page.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Referring also #369

It looks like the article has been updated:

Closing this issue.

@tiburd  sorry, this is not resolved, please write more detailed explaining including PowerShell example for particular scenario, where to put which ID (where to get the encoded string for the partner) ? etc.
Thank you


Add support for Zoom

MultiMeet currently supports Google Meet and Microsoft Teams links. Maybe Zoom support could be useful as well.

Won't fix as the project is deprecated


UX: Change highlight on mint-dark

The current highlight that gets applied is a very bright yellow and when combined with mint-dark it makes text unreadable on pages like Users.
This changes the highlight from the bright yellow to a darker yellow (see attached picture)
Before

After

Thanks!


Multi-byte entry crashes code

VSCode Version: ?
OS Version: OS X

Steps to Reproduce:
See this tweet...
https://twitter.com/matt_porter_/status/778019210806951936

@seanmcbreen oh hey.
VSCode Version: 1.5.2 (66f37fd2a99eb9d628dd374d81d78835b410c39b)
OS Version: 10.11.5 (15F34)

Sorry about the crash, it occurs because we end up splitting a high surrogate - low surrogate pair, thus having on our hands an invalid UTF16 string, which we then pass to oniguruma, which also contains a bug that I introduced in the cached UTF16 - UTF8 conversion where I ignore high surrogates that are lacking a low surrogate, whereas v8's handling of the conversion interprets it as a code point.
Ref: atom/node-oniguruma#56 . A PR is on its way, I'm looking forward to getting it reviewed, otherwise we might proceed to fork the project.

Awesome. Thanks for the explanation. Looking forward to being able to use vscode again.

@M-Porter the PR just got accepted, as soon as there will be a new npm package, we will pick it up and the following day's insiders build will contain the fix.

I verified that it's not crashing, although there are other issues like #12950


k8s 0.19.7

Fixes #

Proposed Changes

Remove replaces in go.mod use go 1.15 in go.mod
update to k8s 0.19.7 in go.mod
fix the gnostic OpenAPIv2 openapiv2 mess

Release Note

Docs

/lgtm

/lgtm