mutate ws object?

Hey Alex, thanks for publishing this, glad to see v15 available and appreciated as always.
Quick question, with other WS schemes we generally would attached handlers when a request is opened something like
WS.on('connection', startWebSocketConnection);
then we had to do something along these lines with your previous uws package:
function registerCallbacks(ws, client) {
  ws.on('message', message => client.handleMessage(message));
  ws.on('close', client.handleClose);
  ws.on('error', client.handleError);
  ws.on('pong', client.handlePong);
}

In the new API we don't really have that, so is the idea that we would instead mutate the WebSocket object so we can store client-specific data as-needed?  I noticed in the C++ documentation you discuss using getUserData() for such things.
I did notice the ws object does appear to be unique per connection, and I can see how this would be a far more efficient way to handle things!  Just making sure my logic is sound!
Also want to make sure that object isn't used in some way and I should restrict to just storing simple data like a clientID or something or if I can/should just store a full-on client state there.

On another note, I noticed you can't really attach listeners for ping, pong, and error.  These may not be much of an issue.  Also noticed we can't send ping from the server itself.  Are these planned for the future?
In the past I'd use this to monitor clients, i do see we now have the idle timeout which could be an ideal solution to this anyway, but just wondering!
I do see that you are handling the ping from clients.
Thanks again!!  Appreciate the efforts!  Excited to see what benefits using the http portions of this will yield as well!

Just to outline my thinking in a simple way
let i = 0;
UWS.App()
  .ws('/*', {
    open: (ws, req) => {
      ws.id = i++
    },
    message: (ws, message, isBinary) => {
      console.log('Message from client id: ', ws.id);
    },
    close: (ws, code, message) => {
      console.log('WebSocket closed ', ws.id, code, message);
    },
  })

If that is for any reason not recommended the other method would be something more like the below, which would clearly be far less performant:
const clients = new WeakMap();
let i = 0;
UWS.App()
  .ws('/*', {
    open: (ws, req) => {
      clients.set(ws, {
        id: i++,
      });
    },
    message: (ws, message, isBinary) => {
      console.log('Message from client id: ', clients.get(ws));
    }
})

Lastly, how would we go about getting some information on the underlying socket?  For example, I used to do something along these lines to capture the connecting clients remote address:
    const { headers = {} } = ws.upgradeReq;
    let ip;

    if (headers['true-client-ip']) {
      ip = headers['true-client-ip'];
    } else {
      const { remoteAddress } = ws._socket;
      ip = remoteAddress ? trimLeft(remoteAddress, '::ffff:') : 'unknown';
    }

I see how to get the header so that one is simple enough!

Hello,
Many good questions here.
Yes, you can and should mutate the ws object (and res object) in any way you want. Attach anything to it you want. It will be valid from open event to close event. Res object is valid from request event to res.end or res.onAborted or res.tryEnd did succeed.
Getting the IP is WIP and will be added.
Sending ping is also WIP.
Errors, we don't have errors. You don't need to care more than to handle open/close events.

Thanks!


Cleanup: Use either async-file or fs-extra in the libraries and tests

Versions
What package version of the SDK are you using.  4.2.x-4.3.x
Describe the bug
async-file and fs-extra are both used in the libraries, and they both do the same thing, so we should reduce down to one package.
In botbuilder we use async-file for the library and for tests.
In botframework-config, the tools/ folder,  we use fs-extra for the library and helper code. fs-extra is also used in the tests for botbuilder-azure and botbuilder-ai, but isn't listed as a devDependency for these two projects. (The tests rely on the hoisted dependencies, otherwise they will fail with import errors)
@justinwilaby FYI
Edit: added fs-extra as a devDependency to botbuilder-ai in #766
[bug]

@benbrown, this seems straight forward provided it can be fixed w/o breaking compat. If this is just internal implemenation we're good. If we've exposed these on interfaces, then we should live with what we've got rather than break compat.

These aren't really the same, as they provide non-overlapping sets of helper methods... and we're using those methods!
That said, this is all internal implementation details so we should be able to change them without introducing any breaking changes.

I went with fs-extra, as it required the fewest changes to our codebase.
https://github.com/Microsoft/botbuilder-js/pull/805


Wallets Page: Including Open/Closed Source

I think it is important to include in the Wallets page also whether a wallet is closed or open source, exactly as they do on the Bitcoin's website.
I know MyDoge had plans to open up their source code a while ago, but since their app now includes a lot of additional functionality unrelated to basic Dogecoin wallet functionality, this might be unlikely.
I think this is important.

I Think this should cover :P https://dogecoin.com/wallets/
I Will close this issue Misha :)


Added TAGGING_AUTOCOMPLETE_MAX_RESULTS setting.

Hello Ludwik,
I added TAGGING_AUTOCOMPLETE_MAX_RESULTS setting which only overrides MAX_NUMBER_OF_RESULTS in order to keep backward compatibility.
Although this setting should reduce the probability of collisions with external settings, the old one still may affect the default value stated in readme.
Best Regards,
Ernest

Thank you :)

I released this and other recent changes on PyPi as 0.5.0


Move contributing guidelines to correct path (closes #4)

I looked more closely and realized there were two CONTRIBUTING.md files
already in the repo, so this commit moves the detailed one into the
right path.
Closes #4.
Signed-off-by: Justin W. Flory<EMAIL_ADDRESS>

@vrothberg I updated my commit to address your feedback. The URL is now fixed.


Testing and Validation for the Weather System

Test the weather system for functionality and realism.

[x] Conduct testing to ensure proper functionality.
[x] Optimize the system based on test results.

#66


mountains swamp

I think that the swamp should not have mountains

Agree

remove the Biome as well from the list. if its not being used, remove it from the environment as well

ill approve when you remove the de-listed file from the workspace


cannot install:

I've downloaded the file but when I try "python setup.py install" it says:
Installed /root/.pythonbrew/pythons/Python-2.7.7/lib/python2.7/site-packages/SimpleGUITk-1.1.3-py2.7.egg
Processing dependencies for SimpleGUITk==1.1.3
Searching for pygame>=1.9.0
Reading https://pypi.python.org/simple/pygame/
Reading http://www.pygame.org/
Reading http://www.pygame.org/download.shtml
No local packages or download links found for pygame>=1.9.0
error: Could not find suitable distribution for Requirement.parse('pygame>=1.9.0')
What to do?

You need to first install the package pygame.


Add info about OHHTTPStub/Swift to readme

@ianks Thanks a lot for this PR. That's a very good idea to add this information.
However, the added information is not totally true (that proves that adding that information is indeed needed 😉 ), because the OHHTTPStubs/Swift subspec only depends on OHHTTPStubs/Core (so that you can opt-in only for what you need), and does not automatically brings everything in the default OHHTTPStubs/Default subspec.
So in practice, if you only put pod 'OHHTTPStubs/Swift in your Podfile, you'll only have the Core + Swift features, but you won't have support for NSURLSession, JSON and OHPathHelper that is provided by the default subspec. That's intended, so that people only wanting to opt-in for some subspecs but not all can configure to their liking.
So I think the best suggestion to give in the README to cover most of what people uses is to list both the default and the Swift subspecs:
pod 'OHHTTPStubs'
pod 'OHHTTPStubs/Swift'

I've rearranged some stuff in the README, about Swift 2.2/2.3/3.0 and about subspecs.
I've directly integrated your recommendations in that rework, explaining about the Swift subspec too. Feel free to review the last changes in the README on master to confirm it looks ok and more clear to you!
I'm thus closing this PR now. Thanks a lot for that suggestion!


Pictures made with HoloLens show very dark background when using OpenXR

Describe the bug
When you take a picture with HoloLens, the background (reality) is very dark:

It used to be like this:

The weird thing is, this happened before, and was fixed. I blogged about this here: https://localjoost.github.io/Fix-Dark-background-when-taking-pictures-or-making-movies-with-HoloLens-2-and-OpenXR/
Now this problem seems to be back again.
Expected behavior

Your setup (please complete the following information)

Unity Version 2020.3.27f1
MRTK Version 2.7.3
Mixed Reality OpenXR plugin 1.2.1 / 1.4.3 (upgrading did not fix it this time)

Target platform (please complete the following information)

HoloLens 2

I have the same issue when using post processing on the camera. And it also affects video capturing (mixed reality capture).
When disabling post processing on the camera (Camera->Rendering->Post Processing), it doesn't add the dark background. By the way, I am using the URP render pipeline - it might be related to that as well.

Hey @LocalJoost, this is definitely unexpected behavior! Would it be possible to try to repro with the sample project at https://github.com/microsoft/OpenXR-Unity-MixedReality-Samples/#welcome? Preferably by taking a screenshot with the "debug data board" in view, so we can try to investigate with the exact versions of the runtime and whatnot. Thanks!

I did. And with this app, the problem does NOT occur. Very annoying

Hrm yeah...I'm also unable to repro in a couple of local projects. Any chance you might be able to put together a min repro? Or any specific project settings you might have changed recently?

@LocalJoost Do you have postprocessing active in your project? And what render pipeline are you using (built in, URP, ...) ? In our project, the dark background only appears when we have postprocessing active. When we disable it, the dark background disappears.

I have no postprocessing

@LocalJoost , we cannot observe this issue using a few projects in our hands.  Could you please find a Unity project with this issue and send to us to diagnose?

Hi, I have demolished the project we observe this in to the point it does nearly nothing but show a floating window from the MRTK. You can find it here. The effect can be clearly seen

@LocalJoost Hi, I've been encountering the same problem. What was surprising, was that every now and then the problem was disappearing and appearing again. I've managed to track down the issue to Unity Splash Screen setting.

If you untick this setting and disable splash screen - photos will get the dark background. Keeping it in the app fixed the issue on my side.

Hi,
I'm also running into this issue since changing the render pipeline to URP. I don't know if it is directly connected to that, but suddenly I have these dark backgrounds in the mixed reality capture.
I already made sure that the splash screen is enabled and post-processing disabled. They were already correctly set. Still, the issue remains. Is there anything else I could try?

I'm also experiencing this with OpenXR and URP in Unity 2022 (MRTK3).
Maybe related: prior to switching to Unity 2022 and URP, transparent splash image was transparent - now it has a solid color background.

For those using URP: can you try turning HDR off on your main camera (under Output) and see if the issue goes away? I narrowed down that setting to a similar, but not identical, rendering issue and I'm curious if that affects anything here.

Thanks for the tip @keveleigh  - turning off HDR in the rendering pipeline settings has resolved the issue for me.  👍

Thanks for the tip @keveleigh - turning off HDR in the rendering pipeline settings has resolved the issue for me. 👍

Apologies, @keveleigh - I was either mistaken or it intermittently doesn't occur - with HDR disabled this is still happening, at least in recorded video.

I have the same issue, I have an empty app with just a single cube. In HoloLens it looks fine, Mixed Reality Capture has dark background. I tried enabling/disabling splash screen (bought Unity Pro trial just for this), disabled HDR, still the same. Any other ideas what could help ?
Unity : 2021.3.30f1
MRTK : 3.0.0
HoloLens 2 Windows version : 22621.1133
HoloLens 2 OpenXR Runtime Version : 112.2211.2002.0

I also have the same error occurring - I am incorporating QR code tracking from @LocalJoost and running into the same black background overlay. Has anyone found a solution yet?

This recently came up again for us during a Unity/dependency update. - I think from going from MRTK 3 pre-16 to 3.0. It seems the MRTK rig sets a solid clear color for the main camera, whereas it should have alpha=0.

The splash screen still doesn't support transparent backgrounds, but that's much less of an issue.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Update dependencies

Coverage remained the same at 91.667% when pulling b433b6ed225046e02dc824f60af6c35793a552d2 on update-dependencies into 632aa9175a3a4661dc63f6d32c29e0f1765c7261 on master.


TcpServer is not a constructor

Hi, I am attempting to replicate the sample code from your README, but it will not execute on my machine.
Machine specs:

macOS 10.15.5 (Catalina)
Node.js v12.18.1
npm 6.14.5

Package.json:
{
  "name": "telnet-test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@svrooij/tcp-server": "^1.0.1"
  }
}

index.js:
const TcpServer = require('@svrooij/tcp-server')

const server = new TcpServer({ port: 3000, host: '<IP_ADDRESS>' })
server.start()

server.on('textReceived', (text, remoteAddress) => {
  console.log('%s -> %s', remoteAddress, text)
})

server.publish('Publish message').then(success => {
  console.log('Message send ', success ? ' successfully' : 'failed')
})

process.on('SIGINT', () => {
  console.log('Stopping server')
  server.stop()
})

Error message:
Mon Jun 22 19:50:24
(imac) ~/Repositories/personal/telnet-test $ node index.js
/Users/dpassarelli/Repositories/personal/telnet-test/index.js:3
const server = new TcpServer({ port: 3000, host: '<IP_ADDRESS>' })
               ^

TypeError: TcpServer is not a constructor
    at Object.<anonymous> (/Users/dpassarelli/Repositories/personal/telnet-test/index.js:3:16)
    at Module._compile (internal/modules/cjs/loader.js:1138:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
    at Module.load (internal/modules/cjs/loader.js:986:32)
    at Function.Module._load (internal/modules/cjs/loader.js:879:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47

Mon Jun 22 19:50:29 the last command exited with code 1

I've scanned the source code, but nothing seems obvious to me at first glance.
Would you be willing to check (and possibly update) your sample code, and/or ensure that your tests are actually exercising the transpiled code? Something doesn't seem to add up here.
Thanks!
David

I think this commit changed the exports from being at the base of the library to having multiple exports.
So the code needed changing from
const TcpServer = require('@svrooij/tcp-server')
// to
const TcpServer = require('@svrooij/tcp-server').TcpServer


return filename when saving jld2 files

Addresses #80 to return the filename when saving to .jld2 file.

Codecov Report

Merging #90 (d005931) into master (af9af7b) will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
- Coverage   82.65%   82.45%   -0.21%
==========================================
  Files          18       17       -1
  Lines        1384     1385       +1
==========================================
- Hits         1144     1142       -2
- Misses        240      243       +3

Impacted Files
Coverage Δ

src/io.jl
0.00% <0.00%> (ø)

src/distance.jl
88.59% <0.00%> (-0.68%)
:arrow_down:

src/SeisNoise.jl

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 af9af7b...d005931. Read the comment docs.


rebuild for hash change

latest build seems to have trouble with the mumps update, despite not directly linking to it.

@conda-forge-admin please rerender

actually, I don't think dolfinx needs this, I think slepc is the problem.

apparently the 0.7.3 hash changed between PR and build, so this is needed


lock screen for audio payers

Hi, i hope you will well.
Kindly can you let me know how can i get proper code for showing lock screen in flutter using audioplayres plugin, thanks.

We already have this feature for ios, just not for android.

Closing in favour of https://github.com/luanpotter/audioplayers/issues/561

We already have this feature for ios, just not for android.

Where is this? They do not seem to work for me


New version: DocumenterVitepress v0.0.13

Registering package: DocumenterVitepress
Repository: https://github.com/LuxDL/DocumenterVitepress.jl
Created by: @asinghvi17
Version: v0.0.13
Commit: fbae4032cebbcbf8e33fb0e30ce6a077ec2a7224
Reviewed by: @asinghvi17
Reference: https://github.com/LuxDL/DocumenterVitepress.jl/commit/fbae4032cebbcbf8e33fb0e30ce6a077ec2a7224#commitcomment-140207672
Description: Documentation with Documenter.jl and VitePress

cc @avik-pal

Everything is up an running now https://lux.csail.mit.edu/dev/


ue5 support

Do you have a script that support unreal engine 5? or make one?

"Night crows" is ue5, and the script seems to be working well.
Which UE5 game are you trying to dump? I'll take a look at it.

i am actually trying to mod a vr game called pavlov shack it uses unreal engine 5.1


Move protobuf files to common module

CI seems to have some problems. @jovany-wang

close since this is already done in another PR #188 .


JOINS

LJ Panas
Instruct them to look at the different columns in a little detail.
Example: if you do str, the dates are all plausible sans any other information to the researcher.
str(biscoe$date)
str(dream$date)
Other approaches may provider a little more information to parse out what the dates are doing.
df %>% select(date) %>% print(n = 50)
df %>% glimpse %>% select() %>% print(n=50)
head(df$date, 50)
You point out the dates issue as part of the exercise but maybe make it hidden in a hint or as a question when moving on to the actual recode (with the code also in the hint) to encourage working through data exploration in R and then look at the explanation.
Torg is already a date, so it does not require a conversion.
The current draft also is recoding island to be the date that is already a date.
torg <- torg %>%
mutate(island = ymd(date))
Given that this is working within tidy, should rbind() be bind_rows()?
Rbind still returns a tibble, so either is fine. Just asking for consistency purposes.
I assume you’ll have a longer discussion of joins before this, so an explainer will not be necessary as to why and when various approaches are of use.
Thoughts and findings. Take anything of use and feel free to ignore the rest. 😊

Complete


How to run script on opening new page?

driver.execute_cdp_cmd("Page.addScriptToEvaluateOnNewDocument", {
  "source": """
    Object.defineProperty(navigator, 'webdriver', {
      get: () => undefined
    })
  """
})

Like above, I want to run some script on new document. How can I do this?
Above ruby script is for selenium.

How about this method?
https://playwright-ruby-client.vercel.app/docs/api/browser_context#add_init_script

Got it, thanks
Do we have wait element method?

Probably you can use this
https://playwright-ruby-client.vercel.app/docs/api/page#wait_for_selector

Thanks
Playwright.create(playwright_cli_executable_path: 'npx playwright') do |playwright|
  playwright.chromium.launch(headless: false) do |browser|
    page = browser.new_page
    page.goto('https://github.com/YusukeIwaki')
    # I can not write much script scraping certain page
  end
end

How can I organize bunch of scraping script here outside the Playwright.create scope?

We can use Playwright.create to create PlaywrightExecution and PlaywrightExecution#stop, instead of Playwright.create do ... end
https://playwright-ruby-client.vercel.app/docs/article/guides/launch_browser#define-startend-of-the-playwright-session-separately-without-block

Playwright Official documentation introduces many useful features. Please take a look at the documentation before using this Gem.
https://playwright.dev/docs/next/intro
This Gem is just a Ruby port of Playwright. Most features are common with TypeScript or Python's library.
Feel free to post a new issue when you find playwright-ruby-client specific problems or questions. Thank you.


Several Problems on TV platform

Hi Flutter Tizen Team,
1) Cursor not showing
I was creating a web app for Samsung M8, however, there is no cursor showing on the app even if the mouse is connected.
Here is my very simple architecture:
Main UI --> Display only one button, and this button is linked to url_launcher_tizen (Cursor not shown here)
url_launcher_tizen --> launches a browser with an url. (Cursor is shown here)
May I know if this feature is not implemented or is it a bug.
2) Enter Key (from keyboard) not responding to the textfield onTap() or onFieldSubmitted() method
I was trying to create a textfield to ask for user input, but the Enter key from the keyboard seems not responding to the submit feature.
(I could only use the "Go" button on the virtual keyboard to submit the request), do you  know if there is any other way for Flutter tizen to acquire user input other than TextFormfield?
Much appreciate your help.

cc @bbrto21 @JSUYA

Hi @raynoryip

Enter Key (from keyboard) not responding to the textfield onTap() or onFieldSubmitted() method

When I tested using TextField and onSubmitted(), I checked that it works normally when I press Enter after inputting text with a physical keyboard.
Try using TextField and onSubmitted() ?
...
String _test = "TEST EnterKey";
...
children: <Widget>[
            Text(
              _test,
            ),
            TextField(
                onSubmitted: (text) {
                  setState(() {
                    _test = text;
                  });
                },
                decoration: InputDecoration(
                  border: OutlineInputBorder(),
                  labelText: 'Test',
                )),
          ],

Hi @raynoryip

Enter Key (from keyboard) not responding to the textfield onTap() or onFieldSubmitted() method

When I tested using TextField and onSubmitted(), I checked that it works normally when I press Enter after inputting text with a physical keyboard. Try using TextField and onSubmitted() ?
...
String _test = "TEST EnterKey";
...
children: <Widget>[
            Text(
              _test,
            ),
            TextField(
                onSubmitted: (text) {
                  setState(() {
                    _test = text;
                  });
                },
                decoration: InputDecoration(
                  border: OutlineInputBorder(),
                  labelText: 'Test',
                )),
          ],

Hi JSUYA,
Thanks for the code, I think it works now!
But do you guys know about the cursor too? I still can't have my mouse cursor on any app I deploy on tizen...

Sorry for not being able to help you quickly.
When I tested on my TV device, the cursor was displayed normally.

Does not the cursor display in the newly created default app in the same way?

ex) flutter-tizen create test_app
      cd test_app
      flutter-tizen run

Can you check the log message?
command : sdb dlog | grep Cursor
if it's normal

I/VD_WIN_UTIL(23716): cursor_module.c: Cursor_Set_Config(1046) > config_type as 1
I/CURSORMGR(  268): e_mod_cursormgr.c: _e_cursormgr_cb_cursor_set_config(344) > Cursor_Set_Config request from client with process:23716 config[1] !

You could find these logs.
If you have any other error messages we can refer to, please share to us.

Sorry for not being able to help you quickly. When I tested on my TV device, the cursor was displayed normally.

Does not the cursor display in the newly created default app in the same way?

ex) flutter-tizen create test_app
      cd test_app
      flutter-tizen run

Can you check the log message?
command : sdb dlog | grep Cursor
if it's normal

I/VD_WIN_UTIL(23716): cursor_module.c: Cursor_Set_Config(1046) > config_type as 1
I/CURSORMGR(  268): e_mod_cursormgr.c: _e_cursormgr_cb_cursor_set_config(344) > Cursor_Set_Config request from client with process:23716 config[1] !

You could find these logs. If you have any other error messages we can refer to, please share to us.

Hi JSUYA, Sorry for not replying for a month, I had an urgent trip and just came back.

I created an app from your command, and no mouse cursor appears on screen:

(I had both mouse and keyboard connected, and they works fine on the default browser app on TV)

I tried to use sdb dlog | grep Cursor too, but nothing happened.
So I tried to use sdb shell, and i got:

closed
closed

Hi. Sorry for late reply
I've searched for a way around this but still haven't found the problem.
According to this code, Cursor_Set_Config() should be called when TV_Profile.
https://github.com/flutter-tizen/embedder/blob/c71260bb8b4cce9d63d0dc62ebe75f3b618c86df/flutter/shell/platform/tizen/tizen_window_ecore_wl2.cc#L125
If there is a problem, you can see an error message.
We currently do not have any M8 devices available for testing.
Hi @yman-son  Do you know how to enable Mousr Cursor on M8 device? Do we need any other special logic?
@raynoryip
On the screenshot, the app doesn't seem to be debug mode.
Could you please share all logs after flutter-tizen run?
@swift-kim Do you know any issues with the Mouse cursor?

Hi. Sorry for late reply
I've searched for a way around this but still haven't found the problem. According to this code, Cursor_Set_Config() should be called when TV_Profile. https://github.com/flutter-tizen/embedder/blob/c71260bb8b4cce9d63d0dc62ebe75f3b618c86df/flutter/shell/platform/tizen/tizen_window_ecore_wl2.cc#L125 If there is a problem, you can see an error message.
We currently do not have any M8 devices available for testing. Hi @yman-son Do you know how to enable Mouse Cursor on M8 device? Do we need any other special logic?
@raynoryip On the screenshot, the app doesn't seem to be debug mode. Could you please share all logs after flutter-tizen run?
@swift-kim Do you know any issues with the Mouse cursor?

Hi JSUYA,
Somehow a rounded cursor showed up when I changed to a partner license....
Which is quite weird, so using other licenses blocked the display of the mouse cursor?
And the cursor seems a bit different than the original one too:
（Sorry, inside screenshot doesn't show this cursor as well :()

Original one:
(Screenshot not supported for Tizen L3xxxxxxx)

And Do i have to download other packages to enable the sdb command?

Sdb commands (such as shell and dlog) are not supported on TV devices due to VD's security policies.
secure_protocol:enabled

Sdb commands (such as shell and dlog) are not supported on TV devices due to VD's security policies.

As in this comment, shell access is not supported in release binaries for security reasons. (intershell_support:disabled)

Somehow a rounded cursor showed up when I changed to a partner license....

It is correct that a rounded cursor is showed up. (currently...)

Which is quite weird, so using other licenses blocked the display of the mouse cursor?

I'm glad your problem has resolved(?). I've been looking to see if the license(partner) has anything to do with this problem, but I haven't been able to find it yet.

So... is using the partner-level certificate the final solution?


feat: dal10 DC per support

Description
Enable DAL10 DC PER support
Fixes:

NFS server on sharefs instance for /sapmnt folder
Renamed sap_instance_init module to ansible_sap_instance_init

Notes:
IBM provider Version to 1.56.1
Ansible Power Linux SAP version: 1.1.3
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
v1.1.0

Your semantic-release bot :package::rocket:


chore: set user agent header

Fix #336

:tada: This PR is included in version 6.15.0 :tada:
The release is available on:

GitHub release
v6.15.0

Your semantic-release bot :package::rocket:


How does this application work?

I'm trying to understand. Does this app take the Spotify link you input and then search YouTube for the song(s) and download them from there?
I apologize if this isn't the place for this question. I didn't know where else to ask.

Hi of course.
The application uses a Spotify API to search for playlists either liked or saved, public or private.
The application then searches YouTube for the best matching song, then adds metadata, and song covers.
A folder is then created with output.


Increase font size

Is your feature request related to a problem? If so, Please describe.
Sometime we need to increase font size in web view, but the textZoom is android only. Is there any other way to increase font size

still searching for solutions

still searching for solutions too.

Any updates on this?


Adding commands to devenv

It would be interesting if there was a way to extend devenv commands. This would allow for some kind of concept of plugins.
{ ... }:
{
  subcommands.mysubcommand.exec = ''
    echo hello
  '';
}

And running it:
$ devenv mysubcommand
hello

Could be used with #103 for cli parsing.

I think this feature would completely eliminate the need for a Makefile.

What is the benefit of the subcommand over the already existing script support?
It seems the only difference is whether the script is on the PATH vs. the subcommand ia built-in the devenv script.

I think I'd prefer it to be a global config in configuration.nix or home.nix.
I'd use it like devenv someplugin someargument.
E.g.
devenv bettersearch mypackage

Inspiration: gh.

I've been using this approach successfully for years, using infranix. infranix is using the same module-based approach as climod, but it is generating a Makefile behind the scenes while climod is just generating bash code.
Anyway, I just wanted to say that the "plugin experience" you get from this (especially when you're using the module system directly, I don't know if end users will do that in devenv) is really nice. I also gave a talk about this at NixCon London 2018.
If you haven't seen just, I recommend you look into it. It is a very nice alternative to Makefiles, and could either be useful behind the scenes in devenv, or even exposed directly to users.
Also, if you ever find yourself implementing auto completion, compleat is a very useful DSL for it.

See https://github.com/cachix/devenv/issues/1362

We need to explore different ways how we can make plugins hook as a subcommand, Cargo itself does this.


[TwigComponent] Add 'hierarchy' proxy to context when rendering embedded components

Q
A

Bug fix?
no

New feature?
yes

Tickets

License
MIT

This PR's is an alternative approach to the problem stated in #863.
Due to the changes proposed in #920, which deals with passing blocks down to nested embedded components, I realized that just being able to access the host (or parent) component isn't sufficient if blocks can be passed down multiple levels deep (as already mentioned in https://github.com/symfony/ux/pull/863#issuecomment-1575123112).
This PR adds a proxy object Hierarchy which allows this syntax:
Component.php
final class Component
{
    public function scream(int $screamHowHard = 1): string
    {
        return 'sf-ux rules' . str_repeat('!', $screamHowHard);
    }
}

{# component.html.twig #}

<twig:Foo :someProp="this.something(1)">
    {{ someProp }}
    {{ this.parent.something(3) }}
</twig:Foo>

{# Foo.html.twig #}

{% block content %}{% endblock %}

Resulting in

    sf-ux rules!
    sf-ux rules!!!

Adding another layer below Foo, would ask for something like
{# component.html.twig #}

<twig:Foo>
    {{ this.parent.parent.something(3) }}
</twig:Foo>

to still point to Foo's function.

Questions
There's also a ComputedPropertiesProxy object. Should we have a cached version of the Hierarchy proxy?

NOTE
This PR continues on the changes of #920, because having a hierarchy only makes sense if blocks can be passed down multiple times. So #920 may have to be merged first, but if needed, and the this.parent syntax is preferred over the syntax of #863, I could push my branch that starts of the main branch instead.

IMO a child component should not have access to its parent.

IMO a child component should not have access to its parent.

Well... the tricky thing is that you ARE still kind of "inside" the parent template - i.e. you're inside component.html.twig and want to access the Component object... but you currently can't if you're coding inside the <twig:Foo> section of that template.
I'm not on one side or the other yet.

Since #920 is merged now, I rebased this branch on the current HEAD.

Thanks Bart!


Add a nice way to switch between open and closed issues/prs

Currently you have to edit the actual search field and swap open with closed. That takes too much time.
Ideally the app provides a nice toggle between open and closed views of issues and prs. Perhaps by tapping on issues/prs field again would switch between two views. So if in issues, user taps issues tab on top, it would switch to closed issues view. Tap again, go to open.

I would love to implement this. Any where I can look to dig into solving this?

Hey this is already a PR #2687
Sent with GitHawk

Dup of #374


Running MeiliSearch on cluster

Hi,
Is it possible to run MeiliSearch in cluster. Eg. If I have 2 separate server where I'm running the MeiliSearch lib and both are pointing to same data volume.

Hello @shashank-sachan,  this is not something that has been done before, and it is probably not a good idea to open the same meilisearch DB with two processes. Maybe it will work, maybe it will not, maybe you will encounter weird race conditions when writing... We can't really help you on this one unfortunately. We will soon start working on high availability, this will maybe fit your usecase once it's shipped.

Thanks @MarinPostma. Is there any load test report or benchmark where we can see how many request can single node handle for given memory and cpu ?
Also just a thought, if we write into ms db from single server but read it from multiple server will that help in scaling? I'm not sure if anyone explorered it.

checked with @gmourier, we are starting to explore the HA, and once this feature is out you should be able to use MeiliSearch in a cluster

@curquiza Hi! any update about HA?

Hello @samatzhusup!
You will find more information here: https://github.com/meilisearch/product/issues/49
We plan to work on this next year.

any update?

Hello @vl-kp,
This is currently in the works, and we have released the first iteration prototype, which is provided here. Feel free to test it and do not hesitate to share your feedback with us. You can find the dedicated discussion for this feature in this product discussion, and you're welcome to have a chat about it also on this discord channel.
Thanks.


Access EmpOnTriggerComponent

About the PR

a
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

super


[DRAFT / RFC] Add PR checks for build/test/install

(Draft change -- requesting feedback/comments)
This is an initial attempt at adding PR checks that detect changed specs, build + test them, and try to install/uninstall the built RPMs. For an example run, see this private run.
Note that the install/uninstall script is an as-is copy from existing ADO pipeline scripts.
The changes to the toolkit golang code needs the most feedback (e.g., the emitting of a tests results JSON). Specifically note the proposal to allow a .spec to use a specialized comment in this line:
#!AZL expected-test-failure

to denote a baselined ptest failure.

I think this PR can be split into multiple chunks.

100% agreed. I'd still like to get feedback on the approach before doing that, though.


Tensorboard

Is there a reason that there is no option to enable tensorboard logging?
I have seen that the logger has variables to save the summary writer and directory. But the there is no parameter for the runner classes to use it.

Using TensorBoard as the visualization tool? rlpyt uses viskit as its visualization tool.

Hi!   I haven't used tensorboard with rlpyt before, always viskit.  But does it seem like it would be helpful and easy to incorporate tensorboard into the runner+logger?  Anyone else used it?

Yes, it's not difficult. I have created a pull request, so that you can see the changes I made. I had to add some code to set the current iteration in the logger. Maybe this should replace the logger.prefix call. And if you want to enable tensorboard logging you just have to set the tf_summary_writer in the logger. This could maybe be added to the logger_context.

OK I think this has all been incorporated!  If it's not working please re-open :)


Update README.md

Change docs to use same name db consistently for the redux-orm slice.

Same here, maybe we can close this PR merging :S @plandem


Open Unread / open-all-notifications no longer works on Enterprise GitHub

This is sort of a bug, sort of a feature request.
Unfortunately, GitHub has moved to a more email-like notification structure.
More unfortunately, my company's on-premise GitHub instance hasn't been updated to the new UI, so I've lost the "Open Unread" functionality I loved from Refined GitHub.
Is it possible to add a switch in open-all-notifications for the old way of doing things, for those companies that are constantly behind the times?
URL: https://github.mycompany.com/notifications
Screenshot:

We try not to break GitHub Enterprise and we accept Enterprise-only bug fixes, but we’re not going to hold back doing non-breaking changes like this. If you need more control, you can clone the repo and use the extension as a local extension. That way you can control when to update.
Originally posted by @sindresorhus in https://github.com/sindresorhus/refined-github/issues/1530#issuecomment-421553029

Note: I'm not being snippy.
If by "non-breaking" you mean "removing a previously working feature", then sure.
/me Reads linked issue comment.
Makes sense. Thanks for the reference!

For the record, if you have a PR that restores the support to GHE and it's not a drastically different implementation, we're open to that. Notifications is not something I use on GHE and have basically ignored all of those PRs to ensure they were still compatible.


Add XenaPhenoset

Closes #57.

Once the code part is finalized, I will add the docstrings.

The main purpose of this PR is to get the phenotype data via the API only (no downloading of XML files). If we know that if API data from a project (TCGA, TARGET) is not sufficient, we will not allow it here.


[JS] Crash when reading IPC file with Union types

Describe the bug, including details regarding any error messages, version, and platform.
When loading an IPC file in Arrow JS using arrow.tableFromIPC, I get a crash when the file contains either a Sparse Union or a Dense Union column.
TypeError: Cannot destructure property 'length' of '(intermediate value)(intermediate value)(intermediate value)' as it is undefined.
 ❯ VectorLoader.readData node_modules/apache-arrow/src/visitor/vectorloader.ts:125:56
 ❯ VectorLoader.visitBool node_modules/apache-arrow/src/visitor/vectorloader.ts:61:113
 ❯ VectorLoader.visit node_modules/apache-arrow/src/visitor.ts:26:48
 ❯ VectorLoader.visit node_modules/apache-arrow/src/visitor/vectorloader.ts:54:22
 ❯ node_modules/apache-arrow/src/visitor.ts:23:44
 ❯ VectorLoader.visitMany node_modules/apache-arrow/src/visitor.ts:23:22
 ❯ VectorLoader.visitSparseUnion node_modules/apache-arrow/src/visitor/vectorloader.ts:103:150
 ❯ VectorLoader.visit node_modules/apache-arrow/src/visitor.ts:26:48
 ❯ VectorLoader.visit node_modules/apache-arrow/src/visitor/vectorloader.ts:54:22

Repro:
git clone https://github.com/kylebarron/arrow-js-ffi
git checkout 50dd5d1
yarn install
yarn test

In particular, this array is generated with this snippet from Python:
def sparse_union_array() -> pa.Array:
    """Generate a sparse union array

    This is derived from the example here https://arrow.apache.org/docs/python/data#union-arrays
    """
    # First child array
    xs = pa.array([5, 6, 7])

    # Second child array
    ys = pa.array([False, False, True])

    # Type mapping
    types = pa.array([0, 1, 1], type=pa.int8())

    # Union array
    union_arr = pa.UnionArray.from_sparse(types, [xs, ys])

    assert isinstance(union_arr, pa.UnionArray)
    assert union_arr[0].as_py() == 5
    assert union_arr[1].as_py() is False
    assert union_arr[2].as_py() is True

    return union_arr

which is then dumped into an uncompressed IPC file, which can't be read in JS.
Component(s)
JavaScript

Fixed by https://github.com/apache/arrow/pull/37122


add router test (mb client) to plow scripts

similar to uperf we will compare two runs using touchstone https://github.com/cloud-bulldozer/benchmark-comparison/pull/27.
run_router_test.sh will be a wrapper which does e2e (initiating, comparing, exporting to gsheet with Pass/Fail) benchmarking.
@jtaleric @chaitanyaenr

+1 to the venv idea. LGTM as well


ConvKB training

Describe the bug
Thanks for the ConvKB implementation but I have a confusion since ConvE and ConvKB training functions look similar.
To Reproduce
Expected behavior
In ConvKB (h,r,t) should be stacked and passed to convolution layers instead of h,r at a time. Can you please clarify?

should be fixed in #114 .


Convert get_public_url to use Python Pathlib library

See title - for additional details one can refer to #13

There's no longer a need for this since we use _request() which should resolve any confusion around paths


Re-fetch Season seems  to do nothing

it actually works like this:
if you got a season of a currently-running show, and themoviedb only has half of the season when you fetch it the first time, refetching it will fetch all the new episodes that were added since last time. And this behaviour works for me. Doesnt it for you?

Hey,
It does seem to work that way. I thought it would also re-fetch episodes that have been deleted from a season. Would that be a good addition? It might be a good idea to give some kind of a notification if the re-fetch is successful or not. Also it would be nice if it could automatically notify if there are new episodes available before the user clicks anything.

I agree that the refetch should update episodes, too. I was a bit iffy about that at first because that might override custom episode data, but I think I can find a way around that somehow. But exactly how do you mean refetch deleted episodes? You have manually deleted some episode from a season and refetch won't add it again? I would expect that to work, so if it doesn't I'll look into it.
I like your ideas for notifications and will implement them :)

Yeah, that's what I meant. The only ways to add a deleted episode back into a season seems to be to delete the whole season and add it again or to add the episode manually. At first I thought this was what re-fetch was meant for.

I guess in the method where I check which episodes need to be fetched and which ones already are there I don't check for the deleted flag. Should be an easy fix. I'll look into it soon


[Node.js] Expose all native errors to Javascript world

Native errors would be exposed no matter recoverable or unrecoverable.
This is to be consistent with how the C++ layer deal with errors
especially to make sure users conscious of native errors even for APIs
without return values. Thus make it easy to target the cause of errors.
Cases are also changed to adpat this.

@halton PTAL, thanks.


Laravel 10.48.17 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.48.17 branch.
Before merging, you need to:

Checkout the shift-ci-v10.48.17 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:x: Shift was unable to run composer update. Please be sure to run composer update before merging to ensure your dependency changes are fully compatible and reflected in your composer.lock file.


Improve the HybridEngine

The HowFuzzilliWorks document describes how I currently believe the HybridEngine should eventually work (feedback on that very welcome!). The current implementation already supports some of these features, but not everything (amongst others, the Type generation and instantiation needs to be improved, as does the type system and the CodeGenerators). There also still seem to be crashes related to it, e.g. probably https://github.com/googleprojectzero/fuzzilli/issues/198.
I don't currently have enough time to work on this, so creating a new issue to keep track of it.

Related: https://github.com/googleprojectzero/fuzzilli/issues/209, https://github.com/googleprojectzero/fuzzilli/issues/210, and https://github.com/googleprojectzero/fuzzilli/issues/211


demo/rollkit: fix the namespaces

Set the namespace used for the gm demo that is compatible with the current
version. Specifically, the current version requires to be the version byte
(0th index) be zero and the reserved bytes (1..5th indicies) be zero as well.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #186

PR #187

PR #188   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 12, 5:25 PM: @rphmeier started a stack merge that includes this pull request via Graphite.


ctrl - enter is going to the next line.

Hi All,
While pressing the ctrl - enter to compile the python code, it cursor is going to the next line.
Any quick help to resolve. I'm in the middle of some important code debugging.
Appreciate your help.
Thanks,
Vishal

Can you try opening up your keybinding resolver w/ atom ctrl-.? See what command is actually running when you press ctrl-enter.
ctrl-enter (or cmd-enter on mac) is mapped to hydrogen:run, which should not be moving the cursor at all with our default keybindings. If you are pressing shift-enter, this is mapped to hydrogen:run-and-move-down and the cursor moving to the next line is expected.

Currently it is set to newline.

Can you please tell how to fix this? It'll be really helpful.

I dont see hydrogen anywhere in your screenshot. Are you sure you have it installed?
Try running hydrogen:run from the atom command palette, does that work as expected?

Yes Ben. I have it installed.
I just sent the screenshot of Keybindings.

pfa-

yes Ben. I have installed it.
I just sent the screenshot of keybindings.
pfa-

Can you try the key bind resolver with ctrl-. Either hydrogen:run is not running at all, or another command is running on top of it causing the cursor position to change.

I am having similar issues. hydrogen:move-and-run-down runs the entire document instead of moving to the next line. It seems to only run till the end of document when the cursor is on a def/class line. I used ctrl-. to see what key binding is getting called, here's a screenshot.

@vishal786btc It looks like Hydrogen isn't correctly installed. Could you try and reinstall it?

I have had the same issue occur a few times recently. It seems to be related to the hydrogen installation getting corrupted somehow. A reinstall fixes it.

Closing as stale


Bug: Pin to start crashes pinned application

Description
When starting the pinned application it will crash some programs. I encountered it with games (Kingdoms of Amalur & Dust: An Elysian Tail).
Dust showed a DirectoryNotFound error:

Steps To Reproduce

Pin a game exe to the start menu using files
Click pinned icon
Crashes

Requirements
Application should not crash upon starting
Files Version
<IP_ADDRESS>
Windows Version
21H2 19044.3693
Log File
debug.log

Thanks for the report

A fix will be included in the next release


Move example path overrides to CI

Following on https://github.com/cachix/devenv/pull/347#issuecomment-1414286036, this makes the simple example again  test against the checkout, instead of what's in the lockfile. This benefits the direnv test, which can now pick up on any breakages.
I thought this also affected other examples, but noticed they all have a devenv.yaml path override. At the same time, there's a todo to expand devenv init to other examples, so these would also be affected down the line.
This PR removes all those path overrides, and instead adds them in CI right before running the test. This modification makes the lockfiles a little useless, so I've removed them. That also saves a few kb in releases, and helps us catch any breakages via nixpkgs or other inputs.
I also noticed the comment in examples/simple/devenv.yaml was leaking in devenv init. This can be seen in the 0.5.1 release. With this change, we can get rid of it.

Hmm, removing the lockfiles apparently means we hit GitHub API rate limits.
I'm always mixed on if it's better to test against latest (nixpkgs et al) or if CI stability is preferred. Think I'm coming back on this, and stability is better.
I've kept the change moving path overrides to the CI workflow, and updated the lockfile devenv entries.
(https://github.com/cachix/install-nix-action/pull/157 probably helps with rate-limits too, but I think we should be good now.)

Huh, CI fails at the nix profile install . step. That seems like a mishap?

Seems like my conflict resolution broke something. Could you revert that and rebase?

Definitely was my fault, simple typo. 😅 I discovered CI works decently well on the main branch of my own fork, so was able to test more there. Simply appending the input to devenv.yaml also doesn't work for simple-remote, because it has an imports section, so I changed it to to insert after inputs: via awk, and that works.


[feature] Create event section

This feature is based on the issue https://github.com/svelte-society/sveltesociety.dev/issues/103. It is a page like the component section, but about the svelte society events. This is only a MVP version so feel free to customize and criticize.
Commits:

add events page to navbar
create event layout and add it to rollup config
create index file for event page
create first event (svelte summit 2020)
new event directory in routify config

Preview

I love what you have done, also I would like to have a larger youtube embed.

.fullscreen-embed {
	position: relative;
	padding-bottom: 56.25%; /* 16:9 */
	padding-top: 25px;
	height: 0;
}
.fullscreen-embed iframe {
	position: absolute;
	top: 0;
	left: 0;
	width: 100%;
	height: 100%;
}

I love what you have done, also I would like to have a larger youtube embed.

.fullscreen-embed {
	position: relative;
	padding-bottom: 56.25%; /* 16:9 */
	padding-top: 25px;
	height: 0;
}
.fullscreen-embed iframe {
	position: absolute;
	top: 0;
	left: 0;
	width: 100%;
	height: 100%;
}

I will leave this to @kevmodrome to merge :)
Thank you @HAUDRAUFHAUN you have begun a starting point for all of us 👍 🥇

Thank you for your positive feedback 😃

I will leave this to @kevmodrome to merge :)
Thank you @HAUDRAUFHAUN you have begun a starting point for all of us 👍 🥇

Thank you for your positive feedback 😃

I love what you have done, also I would like to have a larger youtube embed.

.fullscreen-embed {
	position: relative;
	padding-bottom: 56.25%; /* 16:9 */
	padding-top: 25px;
	height: 0;
}
.fullscreen-embed iframe {
	position: absolute;
	top: 0;
	left: 0;
	width: 100%;
	height: 100%;
}

Commit https://github.com/svelte-society/sveltesociety.dev/pull/105/commits/408267001c27147353b1d79643aec8ba4e34f06a above should do this. Thank you @peopledrivemecrazy!

I love what you have done, also I would like to have a larger youtube embed.

.fullscreen-embed {
	position: relative;
	padding-bottom: 56.25%; /* 16:9 */
	padding-top: 25px;
	height: 0;
}
.fullscreen-embed iframe {
	position: absolute;
	top: 0;
	left: 0;
	width: 100%;
	height: 100%;
}

Commit https://github.com/svelte-society/sveltesociety.dev/pull/105/commits/408267001c27147353b1d79643aec8ba4e34f06a above should do this. Thank you @peopledrivemecrazy!

IMPORTANT
If we are using the youtube iframe, we have to manage, that google is sets cookies via the iframe (privacy policy). In your eyes, how should this be done?

IMPORTANT
If we are using the youtube iframe, we have to manage, that google is sets cookies via the iframe (privacy policy). In your eyes, how should this be done?

IMPORTANT
If we are using the youtube iframe, we have to manage, that google is sets cookies via the iframe (privacy policy). In your eyes, how should this be handeled?

Youtube allows embedding and this has always been the case this is not something we should worry about :)

IMPORTANT
If we are using the youtube iframe, we have to manage, that google is sets cookies via the iframe (privacy policy). In your eyes, how should this be handeled?

Youtube allows embedding and this has always been the case this is not something we should worry about :)

Nice work!

Nice work!

Nice work!

Nice work!


main: add flags for digitalocean-ignition

The purpose of this OEM is to configure the network interfaces without
applying any user config. It's intended to be used by Ignition to
configure the network on boot.

I don't like this approach.


Audiences, Attributes and Events implementation

Summary

Audiences Implementation
Audience Support added in OptimizelyConfigService
Modification of Experiments to support serialize functionality from Conditions
Attributes and Events added to OptimizelyConfig
Cleanup of parsing functions to reuse code
Additional test cases added

Test plan

FSC

Issues

"OASIS-7813"

if ( and if( seem to intermix.

Might be something that can be automatically checked. Don't let it gate a merge.

if ( and if( seem to intermix.

good catch! thanks

Pull Request Test Coverage Report for Build 1740

186 of 214 (86.92%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 90.324%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-api/src/main/java/com/optimizely/ab/optimizelyconfig/OptimizelyExperiment.java
2
3
66.67%

core-api/src/main/java/com/optimizely/ab/optimizelyconfig/OptimizelyFeature.java
4
6
66.67%

core-api/src/main/java/com/optimizely/ab/optimizelyconfig/OptimizelyConfig.java
13
16
81.25%

core-api/src/main/java/com/optimizely/ab/config/Experiment.java
47
58
81.03%

core-api/src/main/java/com/optimizely/ab/optimizelyconfig/OptimizelyAudience.java
7
18
38.89%

Totals

Change from base Build 1719:
-0.08%

Covered Lines:
4686

Relevant Lines:
5188

💛  - Coveralls

Testing the serialize portion of the audienceConditions as part of Experiments is a bit different in this one since each condition is actually based off an interface and many of the checks pertaining to edge cases (ie. empty list, or a list of just audienceId's) are covered during the parsing of the datafile and creates either an EmptyCondition or and AudienceIdCondition, both which implement Condition, so when its time to actually serialize we can check if an empty condition or an AudienceIdCondition. If a list of AudienceId's is present in the datafile parsing then it already creates an OrCondition with a list of AudienceIdConditions, meaning we will never have a case in the Experiment that we only have a list of AudienceId's as it will always be an OrCondition object with a list of those ID's.

FSC is still failing on the new config fields branch, for the same reason as the master branch. Need to look more into why it doesnt seem to find the new fields when doing the custom build. https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233250468

@jaeopt almost ready with this PR if you want to take a peak to get a head start. some small things and some comments to add still but coming along.

FSC is still failing on the new config fields branch, for the same reason as the master branch. Need to look more into why it doesnt seem to find the new fields when doing the custom build. https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233250468

What was your custom yaml?

FSC is still failing on the new config fields branch, for the same reason as the master branch. Need to look more into why it doesnt seem to find the new fields when doing the custom build. https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233250468

What was your custom yaml?

based on the mnoman/optlyConfigNewFields branch
env:
global:
RESULTS_DIR: ./test_results
RUN_ALL: false
RUN_SDK: java
SDK_REPO: git
SDK_BRANCH: "jbrown/oasis-7813"
TESTAPP_BRANCH: "master"
TESTAPP_PORT_BINDING: 3000

FSC is still failing on the new config fields branch, for the same reason as the master branch. Need to look more into why it doesnt seem to find the new fields when doing the custom build. https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233250468

It looks like these failures are because we didn't turn on OPTIMIZELY_CONFIG_V2 flag for java-sdk.

@jaeopt  seems to still fail for same things (Looking for Audiences, attributes, events), plus some double quote issues in the audience -> conditions.
https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233349190
example:
"experiments_map": {
"ab_paused_exp_targeted": {
-      "audiences": ""experiment_audience"" <-- This is showing as red but shouldn't
"id": "10420810913"
"key": "ab_paused_exp_targeted"
"variations_map": {}
}

Most updated FSC but same issues: https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233443821

There are still scenarios when using the custom build not accounting for V2 config.
https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/233519998

Why did that last one pass? It's connected to the old config (for the FSC).

Why did that last one pass? It's connected to the old config (for the FSC).

Yes, I turned off v1 config in app.optimizely, so its looking for v2 now, but that is not merged so I expected it would pass on Master till it is merged. I ran the custom build separate that has the true data. Not looking to merge till the custom build passes fine.
https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/builds/234091446

3 failing scenarios pass with custom build on FSC fix branch: https://travis-ci.com/github/optimizely/fullstack-sdk-compatibility-suite/jobs/530428847


Update specs2-core, specs2-mock to 4.12.5

Updates

org.specs2:specs2-core
org.specs2:specs2-mock

from 4.6.0 to 4.12.5.
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.specs2" } ]

labels: test-library-update, semver-minor

Superseded by #416.


[hw,rv_dm,dv] Skip updates to monitor state machine if tck_en is low

Skip updates to internal state machine in jtag_monitor.sv, if tck_en is set to low

This PR resolves https://github.com/lowRISC/opentitan/issues/18138

What's the impact of this on the overall JTAG issue?

What's the impact of this on the overall JTAG issue?

Not sure about other issues. I only looked at https://github.com/lowRISC/opentitan/issues/18138


The examples run more slowly on GPU than on CPU

Is this the expected behavior?
Btw, I  tested w/ a gtx-1070 gpu versus an i7-6700 CPU.

I only had the opportunity to test it on GPU with an AWS g2.2xlarge/g2.8xlarge instances, and indeed it was as fast as running it on CPU. Unfortunately I don't have access to any dedicated machine with a GPU at the moment, so I can't run any further tests on that.


Display whether a repo is archived on SoftwareHeritage.org

They are an industry-sponsored academic infrastructure project to "collect and preserve software in source code form" and are doing the former from several major code hosting platforms already. See their archive of sindresorhus/refined-github for example.
In order to protect the scientific record against programmers removing their code from one place, scientists can for example cite the SwH-copy of a software, rather than the original repo/project URL.
refined-github could help this, by adding a status icon like for "build/CI status" to indicate whether SwH already archived a repo. This could possibly be done through their API's visit datapoint.
cc @SoftwareHeritage.

Feel free to archive this repo to any service you want, including SoftwareHeritage, but it's not something we're interested in taking part in.

Yeah, it's probably something they need to build into a browser extension of their own. The feedback was worth the post ;-)


Widget column selection general configuration

Is your feature request related to a problem? Please describe.
All widgets have to check for their columns, we could define, in a func or in conf, rules about columns unique values limit and type. The widget will only the compatible functions to be selected.
Maybe a parameter of the columns component modal ?

Maybe ask a designer what would be the best way to give a hint on what column to use
TLDR : Widgets can set warning rules, commanded columns types or category, and unsupported columns.

Done with unfolding update


URL hash update based on scroll is not working.

I am trying to use this plugin. I see the headerHash appear but it does not update as I scroll. I have attached pictures of what my code looks like and there are no console errors so I know the syntax is not the issue. Please help! Also, I have the three script files included below all my code right before the closing body tag.

UPDATE: So now the anchor is working but it is not reading the first data-hash="welcome" it starts at dog but is anchoring dog to the beginning of the website which is hashed welcome.

Your updated issue is related to #30, which has just now been fixed.
Also, make sure you put your
 divs inside the
 div.


Add client-side manual time addition

The client needs a way to add time manually, it's a little hard to do this server-side

Assigning you on this @gtaEPIC if you don't mind because I can't do HTML/CSS :D

Looks like this will take a bit longer than I expected. A few other issues occurred while I was testing, so I'll fix those tomorrow

That's never what you want to hear 😅
If anything's out-of-scope on this PR, PR the fixes in their own one just to keep things organized


Ignite 12088

Cache or template name should be validated before attempt to start,  max valid name length is 235 characters (20 is reserved for internal prefixes, suffixes or extensions).

TeamCity is green https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_BuildApacheIgnite&tab=buildTypeBranches

fixed the review comments.


[Bug]: Windows validation fails for 2.17.0

Describe the bug
See the distribution validation for windows platform
https://build.ci.opensearch.org/job/distribution-validation/1173/
https://build.ci.opensearch.org/job/distribution-validation/1179/
Even though the artifact is valid, it fails to pass. Tried deploying both OpenSearch and OpenSearch-Dashboards on windows machine, it works well. Also integration test without validation goes through.
To reproduce
Run the above workflows or ./validation.sh with windows parameters.
Expected behavior
Validation for windows should go through
Screenshots
If applicable, add screenshots to help explain your problem.
Host / Environment
No response
Additional context
No response
Relevant log output
No response

The root cause of this was https://github.com/opensearch-project/opensearch-system-templates/pull/26
We always validate both OS and OSD in the validation workflow. Since OSD test CI image defaults to java 11, system-template was causing cluster to crash indirectly affecting OSD to not find backend connection.
@Divyaasm Maybe we don't need to validate OSD for opensearch integration tests. Also looks like we might need to run integration test against compatible java matrix that we say we support here: https://opensearch.org/docs/latest/install-and-configure/install-opensearch/index/#java-compatibility
cc: @getsaurabh02 @peterzhuamazon
Closing this issue. Maybe create a new one for new requirements.


Add support for FTB modpacks

Currently you can successfully install modpacks from curseforge.
Some modpacks however are dropping support for curse and moving to the ftb launcher only instead.
(likely cause curse is not very nice to the devs)
Eg. see the note at the top of https://www.curseforge.com/minecraft/modpacks/ftb-interactions
Moved to FTB https://www.feed-the-beast.com/modpack/ftb_interactions
This site has also some nice stats https://ftb.neptunepowered.org/pack/ftb-interactions/
I did not really do the research whether some secret url can be mined out from the ftb site, which would have the same format as curse (or whether they require authentication).
If it is a pain to do, could you please leave at least some leads for me? I might consider opening a PR for it in the future..

The API used for FTB modpacks (api.modpacks.ch) is a beauty compared to curseforge. The functionality has been implemented in de32159.
Usage: picomc mod pack ftb install [OPTIONS] PACK_ID [VERSION]

  Install an FTB modpack.

  An instance is created with the correct version of forge selected and all
  the mods from the pack installed.

  PACK_ID can be the numeric id of the FTB modpack or the slug from the URL
  to its website.

  VERSION is the version name, for example 2.1.3, not its ID. If VERSION is
  not specified, the latest is automatically chosen. If --beta is used, the
  chosen version can be a beta version. Otherwise, only stable versions are
  considered.

Options:
  -n, --name TEXT  Name of the resulting instance
  -b, --beta       Consider beta modpack versions
  --help           Show this message and exit.

Please test it and close this issue if everything is in order. Version 0.4.1 has been released with this included.

Works, like a charm, thanks for the quick response!


add a test folder for downloading files off of github

Codecov Report

Merging #75 (3e24ae9) into main (0d76951) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   80.95%   80.74%   -0.21%
==========================================
  Files          24       24
  Lines         966      966
  Branches      166      166
==========================================
- Hits          782      780       -2
  Misses        133      133
- Partials       51       53       +2

Flag
Coverage Δ

unittests
80.74% <ø> (-0.21%)
:arrow_down:

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

Impacted Files
Coverage Δ

belay/packagemanager.py
81.73% <0.00%> (-0.97%)
:arrow_down:

belay/device.py
77.68% <0.00%> (-0.29%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Subtitles as tracks in transcription services

This PR aims at contributing to the implementation of #4407, but touches on just one of the aspects to realize #4407's vision. It should therefore likely not be merged alone, but alongside other PRs.
This changes the "Attach" WOHs for our Speech-to-Text integrations. The subtype of the target flavor can now be set freely, default output format is vtt, default output type is track.
Due to lack of access to the various external Speech-To-Text services this PR is not particularly well tested. It would be great if someone with access to one ore more services could test those!

@karendolan @rute-santos Can you test Watson?
@lkiesow Can you test Amberscript?  Or know someone who is using Amberscript?
@wsmirnow Are you guys using Azure?
Now we just need someone who uses the Google side of things.

The Azure side was completly rewritten and will bei pushed upstreem soon. I would also update the code to be compatible with #4407.

Unfortunately I can't review:

Amberscript
Azure
Watson
Google

But I can review #e82dc6f and #f4d9acd. Would be possible to split this PR?

This PR has been opened since April and there are no remarks regarding the transcription services. I will suppose that the transcription services work properly.

This PR has been opened since April and there are no remarks regarding the transcription services. I will suppose that the transcription services work properly.

Unfortunately, it does not. This has been a valid configuration before and is still valid according to the documentation:
  - id: amberscript-attach-transcription
    fail-on-error: true
    exception-handler-workflow: partial-error
    description: Attach captions/transcription
    configurations:
      - transcription-job-id: ${transcriptionJobId}
      - target-caption-format: vtt
      - target-tag: engage-download

The target-flavor should be derived from target-caption-format.
But this now fails with:
2023-09-01 12:50:30,122 | ERROR | (WorkflowOperationWorker:140) - Workflow operation 'operation:'amberscript-attach-transcription, state:'FAILED'' failed
org.opencastproject.workflow.api.WorkflowOperationException: Configuration key 'target-flavor' must be set
        at org.opencastproject.workflow.api.AbstractWorkflowOperationHandler.getTagsAndFlavors(AbstractWorkflowOperationHandler.java:489) ~[?:?]
        at org.opencastproject.transcription.workflowoperation.AmberscriptAttachTranscriptionOperationHandler.start(AmberscriptAttachTranscriptionOperationHandler.java:95) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowOperationWorker.start(WorkflowOperationWorker.java:212) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowOperationWorker.execute(WorkflowOperationWorker.java:117) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowServiceImpl.runWorkflowOperation(WorkflowServiceImpl.java:723) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowServiceImpl.process(WorkflowServiceImpl.java:1741) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowServiceImpl$JobRunner.call(WorkflowServiceImpl.java:2102) ~[?:?]
        at org.opencastproject.workflow.impl.WorkflowServiceImpl$JobRunner.call(WorkflowServiceImpl.java:2068) ~[?:?]
        at java.util.concurrent.FutureTask.run(FutureTask.java:264) ~[?:?]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128) ~[?:?]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628) ~[?:?]
        at java.lang.Thread.run(Thread.java:829) ~[?:?]


Support changing the Rust version

Is your feature request related to a problem? Please describe.
This action internally uses a container with a fixed stable Rust release. However, as the maintainer of a Rust project that uses nightly features in its Cargo.toml manifest, I'd appreciate the action to be also fully compatible with nightly Rust projects.
Moreover, although this project looks well-maintained enough to bump the Rust version, in the future someone could be using a feature that is not supported in the hardcoded Rust version at the time, and the inability to readily change it may be a blocker or a turn-off.
Describe the solution you'd like
An action input parameter to select the Rust version that is used in the container to run commands such as cargo metadata, which cargo deny needs to function.
Describe alternatives you've considered

Not using nightly Rust, or only using the features supported by the hardcoded Rust version. However, that is not always practical.
Forking the project to tweak the Dockerfile as needed.
Changing how cargo deny works so that it doesn't rely on Cargo being installed at all. I've no idea of how complicated this would be, but it definitely doesn't look like a piece of cake.

Additional context
I don't have any additional context to describe.

I've recently discovered that the rust-toolchain file can be used to override the Rust toolchain used for a repository, and Cargo takes care of selecting and installing it transparently. Using such a file made Dependabot work again on my repository with nightly features, and it might make this action work too.
I'll update this comment after testing whether the override file makes a difference.


const admin = require('firebase-admin') crashes with TypeError: Cannot read properties of undefined (reading 'env')

Earlier this week I, like others, started receiving web pack errors such as
Module not found: Error: Can't resolve 'util' in '/Users/david/Documents/NativeScriptProjects/rrr/node_modules/@fastify/busboy/lib/utils'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
	- add a fallback 'resolve.fallback: { "util": require.resolve("util/") }'
	- install 'util'
If you don't want to include a polyfill, you can use an empty module like this:
	resolve.fallback: { "util": false }

I had over 200 such errors.  After investigating for several hours, I edited my webpack.config.js file thusly:
const webpack = require("@nativescript/webpack");

module.exports = (env) => {

  // See https://stackoverflow.com/questions/70428159/nativescript-8-1-migration-adding-external-to-webpack-config-js
  env.appComponents = (env.appComponents || []).concat(['./foreground-service.android'])

  webpack.init(env);

	// Learn how to customize:
	// https://docs.nativescript.org/webpack

  webpack.Utils.addCopyRule('data/**')
  webpack.Utils.addCopyRule('files/**')
  webpack.Utils.addCopyRule('**/*.db')

  webpack.mergeWebpack({
    resolve: {
      fallback: {
        assert: require.resolve("assert/"),
        browserify: false,
        buffer: require.resolve("buffer/"),
        child_process: false,
        crypto: require.resolve("crypto-browserify"),
        dns: false,
        fs: false,
        http: require.resolve("stream-http"),
        https: require.resolve("https-browserify"),
        http2: false,
        net: false,
        os: require.resolve("os-browserify/browser"),
        path: require.resolve("path-browserify"),
        querystring: require.resolve("querystring-es3"),
        request: false,
        stream: require.resolve("stream-browserify"),
        tls: false,
        tty: require.resolve("tty-browserify"),
        url: require.resolve("url/"),
        util: require.resolve("util/"),
        zlib: require.resolve("browserify-zlib"),
      },
    },
  });

	return webpack.resolveConfig();
};

In short, I installed the packages suggested by the log file errors, such as util, and added the associated require.resolve, and set to false those items that did not have a suggested package.  This resolved the build errors, but now I get a crash when I navigate to a page that requires firebase-admin (so as to send firebase messages):
  System.err: An uncaught Exception occurred on "main" thread.
  System.err: Calling js method onClick failed
  System.err: TypeError: Cannot read properties of undefined (reading 'env')
  System.err:
  System.err: StackTrace:
  System.err: ./node_modules/util/util.js(file: app/webpack:/rrr/node_modules/util/util.js:109:11)
  System.err:   at __webpack_require__(file: app/webpack:/rrr/webpack/bootstrap:24:0)

I have to think these two are related, but I’m at a loss as to how to proceed.  Can anyone confirm whether this is the correct approach to handling the webpack 5 polyfill errors?

Most likely you need to polyfill global.process as it's undefined in NativeScript, and these libraries check for process.env.
That said, I'd strongly recommend against using firebase-admin in your app. From the package documentation:

Please also note that the Admin SDK should only be used in server-side/back-end environments controlled by the app developer. This includes most server and serverless platforms (both on-premise and in the cloud). It is not recommended to use the Admin SDK in client-side environments.

Using it in your app is most likely result in your firebase credentials leaking and being abused, as opening up APKs and IPAs is trivial.

@edusperoni The only reason I was using firebase-admin is because that's the only example I saw in the @firebase-messaging doc.  Should I be posting hhtp requests like I did with the older firebase plugin, but using the newer firebase API?

@dlcole the examples there are for the backend to send messages. In which case you need to have a backend with your credentials and expose an API that the user will call which in turn calls the admin SDK to post a message.
You'd usually need firebase messaging if you need to send messages to specific users (chat app, user send message to another user, you trigger a firebase message with the data), send messages independent of user action (ad campaign, etc), and similar things. If you need to display local notifications instantly or after a while you can use the local-notifications plugin


Support or document existing mechanism to --append-hash to configmaps and secrets

I know the current recommended practice for rolling deployments is to use https://helm.sh/docs/howto/charts_tips_and_tricks/#automatically-roll-deployments
However the kubernetes team seems to recommend a workflow that new configmaps are created and then old deleted a la
https://github.com/kubernetes/kubernetes/issues/22368
https://github.com/kubernetes/kubernetes/pull/49961#issuecomment-319440152
I know of course it all depends on the application and whatnot but is there an easy way to add the hash of the configmap data to the name? I know you could hash the relevant values and perhaps that's enough though it is inconsistent with --append-hash behavior.
Perhaps also this whole workflow is more of a can of worms as helm may delete the previous configmap before the new pod is ready under current behavior, I've not tested that as yet.
Regardless the mismatch here seems worth looking it whether there is a feature missing, documentation of the values hash pattern or simply noting that helm doesn't recommend the same thing the kubernetes team does here. Especially with immutable configmaps coming down the pipe.
Thoughts?

There are cases to update the configmap or secret in place and there are times to create a second one and then remove the first. It kind of depends on your use case. This is important to note as the original intent of configmaps and secrets was for a more limited set of use cases than people use them for today.
In any case, being able to change the name when the contents change is possible but tricky.
First, you need to generate a hash of something. There's an example of generating a hash for a configmap at https://helm.sh/docs/howto/charts_tips_and_tricks/#automatically-roll-deployments. It's possible to generate the data section or some other information in a template function and then generate a hash of this.
Second, this hash would need to be used any place the reference to the configmap or secrets name is used.
This can all be done within template functions. It is the kind of thing you want as part of the chart instead of a flag because charts are packages and the business logic for that chart should be encapsulated in it.

It's possible to generate the data section or some other information in a template function and then generate a hash of this.

Yeah I think this is mostly my point in terms of "current" solutions that duplicate the behavior being pretty complex... Linking to the name using a template is pretty easy, hashing the data is pretty hard since as you note it's got to generate the content to hash before generating the hash.
In case it's not clear here the first request is not "add the flag to the CLI". It is rather to support or document how to achieve (ideally easily) the behavior of the kubectl flag in building charts . I'm with you 100% that adding a flag to the helm CLI doesn't make a lot of sense here.
Second of course I'd love a sense of how the different recommended best practices from kubernetes "use new names" vs helm "update existing and roll with an annotation" might ultimately be reconciled before they drift too much.

@mattfarina any further comment? I'm not looking for "support" here so that tag isn't really correct. More a design discussion on how we might align the recommended approaches from Helm to update the configmap and roll the deployment vs the kubernetes team saying absolutely don't do that, instead make a new configmap hashed this way and reconfigure the deployment.


A question about document store

We know Documents inside a DocumentArray can live in a document store instead of in memory，and DocArray supports multiple storage backends with different search features. We can use match() or find() to perform approximate nearest neighbour search.
so, is the implementation of those vector search algorithm done by DocumentArray or by those document store?

Hello @melowYX ,
In those cases the actual implementation is done by those backends and DocArray offers a very convenient Pythonic Interface.

Hello @melowYX ,
In those cases the actual implementation is done by those backends and DocArray offers a very convenient Pythonic Interface.

Thanks for your response


Problems with the validation when loading the pictures

Hi.
In the configuration is the line

`            messages: {
acceptFileTypesError: 'This file is not in a supported format: ',
maxFileSizeError: 'This file is too big: '
}

,`
acceptFileTypesError - it's work
maxFileSizeError it's don't work
в UploadHandler.php there are lines

     'max_file_size' => null,
        'min_file_size' => 1,
        'max_number_of_files' => null,
        'image_file_types' => '/\.(gif|jpe?g|png)$/i',
        'correct_image_extensions' => false,
        'max_width' => null,
        'max_height' => null,
        'min_width' => 1,
        'min_height' => 1,

For example, if I set 'max_file_size' => 10, and download a large file, it returns this response: {"files":[{"name":"sh-logo.png","size":8204,"type":"image/png","error":"File is too big"}]}
But how to get this error message in alert(), or another way?
There might be a possibility to check the parameters (max_file_size, max_width, and others) of the image to load before downloading?
with jquery.fileupload-validate.js - there are problems ...
P.S. Sorry for my English.

Uunderstand

  			    always: function (e, data) {
  			    	console.log(data.jqXHR.responseText);
  			    }


Create dev-zfi4 branch request + Minor changes to get to build

I am working on creating some visuals for EDO office at CDC.
Request 1:

Please create a branch dev-zfi4 or grant user jcbowyer permission to create branch dev-zfi4 and publish

Request 2:
Review notes below.
When doing a pull request and following these steps on Ubuntu, I ran into a couple of issues:

Start by cloning this repo and running yarn install at the root.
Run lerna bootstrap and Lerna will initialize all the packages for you.
Run lerna run build to build all of the packages in the correct order.
To begin working on an individual package, run lerna run --scope @cdc/package_name start, replacing package_name with the package's name (ex: @cdc/map).

Changes required in package.json in devDependencies

changed "react": "17.0.1" to "react": "17.0.2"
changed "react-dom": "17.0.1" to "17.0.2

Changes required in package.json in dependencies

changed  "@cdc/core": "^1.1.2" to "@cdc/core": "^4.23.1"

@jcbowyer - discussed internally. If there's any additional input needed please re-open.


Enable global classes in subfolders of res://addons/

This PR lets GlobalClass classes get added to the "Create New Node" dialog no matter where they are under addons/ as long as it's not in a disabled plugin (or any subfolder of such as those are assumed to belong to the disabled plugin).
This also then lets you keep non-plugin files under addons/ and have their GlobalClasses work, as seems encouraged by https://docs.godotengine.org/en/stable/tutorials/best_practices/project_organization.html saying "In general, keep third-party resources in a top-level addons/ folder, even if they aren't editor plugins. This makes it easier to track which files are third-party."
Fixes https://github.com/godotengine/godot/issues/78400.
The code could presumably be optimised some e.g. by not using String::split, but I'm guessing code clarity and brevity takes preference over squeezing out performance here?

Superseded by #91337. Thanks for the contribution!


Workaround for discrete optimization?

Imagine that I build objective function such that internally it discretizes the real input numbers to integer ones (e.g. 1.6 -> 2, 0.3 -> 0 etc) and computes the objective on these discrete values. This way while GPyOpt sees variables as real numbers effectively it does discrete optimization, as I can always discretize found solution while preserving objective value. On the one hand side, it is guaranteed (at least in theory) that in the limit the result of optimization is globally optimal, thus I would expect that result of implicit discrete optimization will also be global. On the other hand side, it is usually assumed that the objective should be Lipschitz continuous (in theory), which it is not for the outlined case, but it might work just fine in practice. Is this the way people adapt Bayesian Optimization to the discrete case or is there something more to it?

In the devel branch you can use discrete variables. Have a look to the
notebook:
https://github.com/SheffieldML/GPyOpt/blob/devel/manual/GPyOpt_mixed_domain.ipynb
It works by treating all inputs as continuous variables in the model but it
restricts the search in the acquisition function to feasible values of the
domain. This may be a bit inefficient sometimes but worked well in the
examples that we tried.
On 30 March 2016 at 09:48, iaroslav-ai<EMAIL_ADDRESS>wrote:

Imagine that I build objective function such that internally it
discretizes the real input numbers to integer ones (e.g. 1.6 -> 2, 0.3 -> 0
etc) and computes the objective on these discrete values. This way while
GPyOpt sees variables as real numbers effectively it does discrete
optimization, as I can always discretize found solution while preserving
objective value. On the one hand side, it is guaranteed (at least in
theory) that in the limit the result of optimization is globally optimal,
thus I would expect that result of implicit discrete optimization will also
be global. On the other hand side, it is usually assumed that the objective
should be Lipschitz continuous (in theory), which it is not for the
outlined case, but it might work just fine in practice. Is this the way
people adapt Bayesian Optimization to the discrete case or is there
something more to it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/SheffieldML/GPyOpt/issues/10

I see. Thanks for the quick reply!


Add --stage argument to chalice commands

This pr builds on https://github.com/awslabs/chalice/pull/264
to provide a consistent CLI experience when working with chalice stages.
There's several changes in this PR.
First, the code now clearly indicates whether we mean "chalice stage", which is
a completely separate collection of AWS resources, or an "API Gateway stage"
which is a specific configuration within a chalice stage (it's essentially the
URL prefix to use for your REST API for now).  chalice_stage_name is
generally used when we mean a chalice stage and api_gateway_stage is used
when we mean the latter, and with the exception of the actual CLI option
(--stage) every existing reference to a "stage" has been renamed to one
of those two values.
Next, I've deprecated the form "chalice deploy ".  You now get an error
message if you use this form which instructs you use to use
--api-gateway-stage:
$ chalice deploy prod
You've specified a deploy command of the form 'chalice deploy <stage>'
This form is deprecated and will be removed in a future version of chalice.
You can use the --api-gateway-stage to achieve the same functionality, or the newer '--stage' argument if you want an entirely set of separate resources.

To get the new stage functionality introduced in #264 you need to use
chalice deploy --stage prod.
And finally, I've added a --stage option to the remaining set of chalice
commands that should be stage specific:

url
logs
generate-sdk
package

All of these commands have also been updated to pull the deployed values
from the .chalice/deployed.json file instead of trying to query it using
the SDK as it was doing previously.

Codecov Report

Merging #270 into package will increase coverage by 1.48%.
The diff coverage is 80.19%.

@@             Coverage Diff             @@
##           package     #270      +/-   ##
===========================================
+ Coverage    83.98%   85.47%   +1.48%
===========================================
  Files           17       17
  Lines         1592     1645      +53
  Branches       196      200       +4
===========================================
+ Hits          1337     1406      +69
+ Misses         202      186      -16
  Partials        53       53

Impacted Files
Coverage Δ

chalice/package.py
97.1% <100%> (ø)
:arrow_up:

chalice/awsclient.py
82.01% <100%> (ø)
:arrow_up:

chalice/deploy/deployer.py
73.43% <100%> (ø)
:arrow_up:

chalice/cli/__init__.py
75% <68.23%> (+10.35%)
:arrow_up:

chalice/cli/factory.py
84.21% <84.21%> (ø)

chalice/config.py
97.26% <95.83%> (+6.35%)
:arrow_up:

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 7607b22...95ca88e. Read the comment docs.

@kyleknap I believe all the feedback's been incorporated.

Cool, also @stealthycoin I improved the error messages on import errors:
 $ chalice deploy
Unable to import your app.py file:

File "/private/tmp/error/errors/chalicelib/james.py", line 1
  this is a syntax error in chalicelib/james.py

SyntaxError: invalid syntax

Rebased and merged in https://github.com/awslabs/chalice/commit/76bd8e161c024dedf8a2fae869124de599f19654


Request: add “sleep” state

Hey!
Loving the add-on! Thanks for developing this :)
any chance you can add the “sleep” state at all? I see on the app it tracks light/deep sleep state - is this an attribute  that can be pulled over somehow?

Unfortunately, Owlet doesn't expose this information via the API. I think that must be done on the backend using some sort of post-processing ML model.
If you have the Owlet Smart Sock v3, you can probably hack together a similar metric by combining something like % of minutes in the last 10 minutes in which there was motion detected + heart rate within a specific range. I know for my son, I'm able to quickly tell at a glance if he's sleeping or not by viewing the heart rate for a few measurements.


Moving qint4 models takes a large amount of time

Quantizing the new Flux.1 models and saving them to a file, and then loading them back in using (re)quantize takes a huge amount of time. (900+s, specs here)
I've looked into the requantize function and thru basic debugging I have found that the .to(device) call takes the most time. (~90%)
Just in case, this was a local problem, I went ahead and got a fresh install of linux with the minimal dependencies, to see if it was just a case of faulty drivers, but it wasn't.
The following is the script I'd like to run:
import json
import torch
from diffusers.models.transformers.transformer_flux import FluxTransformer2DModel
from optimum.quanto import freeze, requantize, qint4, quantization_map, quantize
from safetensors.torch import load_file, save_file

dtype = torch.bfloat16
repo = "black-forest-labs/FLUX.1-schnell"
transformer = FluxTransformer2DModel.from_pretrained(repo, subfolder="transformer", torch_dtype=dtype)
quantize(transformer, weights=qint4)
freeze(transformer)
save_file(transformer, "model.safetensors")
with open("quantization_map.json", "w") as f:
    json.dump(quantization_map(transformer), f)
del transformer

with torch.device("meta"):
    with open("config.json", "r") as f:
        transformer = FluxTransformer2DModel.from_config(json.load(f)).to(dtype)

tdict = load_file("model.safetensors")
with open("quantization_map.json", "r") as f:
    tmap = json.load(f)

requantize(transformer, tdict, tmap, device="cuda")

One more thing to add, this slowness also happens every time when you move the models between cpu and gpu on my end.

@Disty0 is correct: when moving models between devices, the weights might be reformatted as expected by the optimized kernel used on that device. This happens in QBitsTensor.create().
Here, the weights are first loaded on CPU, and formatted for the TinyGemm pytorch kernels. The takes time, and is pretty useless because eventually they are put back in their original format when moving to a CUDA device.
I could add a context to disable kernel optimizations: something like with quanto.disable_optimizations():, and wrap the weights loading with that context in requantize, so that the formatting only happens when moving to the target device.

That is (imho) probably the best course of action, that's exactly what I ended up doing with torchao in the meanwhile

The following locally stored quantized objects load 100% faster than quantizing them on the fly:
#1 Do this once: Quantize the transformer and TF encoder in Flux:
from diffusers import FluxTransformer2DModel
from transformers import T5EncoderModel
from optimum.quanto import freeze, qint8, quantize, quantization_map
from pathlib import Path
import json

base_model = "black-forest-labs/FLUX.1-dev"
dtype = torch.bfloat16

transformer = FluxTransformer2DModel.from_pretrained(
    base_model,
    subfolder= "transformer" ,
    torch_dtype=dtype
)
quantize(transformer, weights=qint8)
freeze(transformer)

save_directory = "./flux-dev/fluxtransformer2dmodel_qint8"
transformer.save_pretrained(save_directory)
qmap_name = Path(save_directory, "quanto_qmap.json" )
qmap = quantization_map(transformer)
with  open (qmap_name, "w" , encoding= "utf8" ) as f:
    json.dump(qmap, f, indent= 4 )
print('Transformer done')
text_encoder_2 = T5EncoderModel.from_pretrained(
    base_model,
    subfolder= "text_encoder_2" ,
    torch_dtype=dtype
)
quantize(text_encoder_2, weights=qint8)
freeze(text_encoder_2)

save_directory = "./flux-dev/t5encodermodel_qint8"
text_encoder_2.save_pretrained(save_directory)
qmap_name = Path(save_directory, "quanto_qmap.json" )
qmap = quantization_map(text_encoder_2)
with  open (qmap_name, "w" , encoding= "utf8" ) as f:
    json.dump(qmap, f, indent= 4 )
print('T5encoder done')

Now you can refer back to this doing your inference:
    from transformers import T5EncoderModel
    from optimum.quanto import QuantizedTransformersModel, QuantizedDiffusersModel
    start = time.perf_counter()
    dtype = torch.bfloat16

    class  QuantizedFluxTransformer2DModel (QuantizedDiffusersModel):
        base_class = FluxTransformer2DModel

    transformer = QuantizedFluxTransformer2DModel.from_pretrained(
        "./flux-dev/fluxtransformer2dmodel_qint8"
    ).to(dtype=dtype)

    class  QuantizedT5EncoderModelForCausalLM (QuantizedTransformersModel):
        auto_class = T5EncoderModel
        auto_class.from_config = auto_class._from_config

    text_encoder_2 = QuantizedT5EncoderModelForCausalLM.from_pretrained(
        "./flux-dev/t5encodermodel_qint8"
    ).to(dtype=dtype)

    pipe = FluxPipeline.from_pretrained(repo_id, transformer=transformer, text_encoder_2=text_encoder_2, torch_dtype=dtype)

    if device == 'cpu':
        pipe.enable_model_cpu_offload()
    else:
        pipe = pipe.to(device) #'cuda'