#399 Singlefile support

Summary
Add https://github.com/gildas-lormeau/SingleFile as an extractor
**Related issues: #399
Changes these areas

[ ] Bugfixes
[X] Feature behavior
[ ] Command line interface
[ ] Configuration options
[ ] Internal architecture
[ ] Archived data layout on disk

@pirate This first version is functional but it is lacking a lot of flexibility. It uses the CHROME_BINARY. I need to add new options to be able to use other browsers.
Also, the installation is not possible via package managers, like it is for wget and chrome. I had to clone it, install npm dependencies and create a symlink to my bin folder. Should we leave this method enabled by default?
It currently works, if you have chrome installed. It creates a new file in the archive named single-file.html. Currently, it is created directly by the single-file process.

A good start, things left:

add an iframe to the link_details.html template for singlefile
add a files column icon for SingleFile in the core.admin snapshot list
add it to archviebox.index.latest_outputs
add it to archviebox.index.canonoical_outputs
anywhere else you can fine (search for dom_path and add it next to it everywhere)

[ ] add an iframe to the link_details.html template for singlefile
[ ] add a files column icon for SingleFile in the core.admin snapshot list
[ ] add it to archviebox.index.latest_outputs
[ ] add it to archviebox.index.canonoical_outputs
[ ] anywhere else you can fine (search for dom_path and add it next to it everywhere)

@pirate please review. I think we can close this and add the new config options to the cli in another PR. Let me know if you prefer adding that in here.

The last thing remaining is to add it to the docker image so it works out-of-the-box.

@pirate Added a fixture to the tests to avoid using methods we were not actually testing. By default it will bypass all the extractors. The python tests should be running faster now because of this change.
I also added the singlefile installation as part of the process (it was that or remove the singlefile test).
Additionally, I added a config variable that was missing (SAVE_SINGLEFILE) and that was causing erratic behaviour.


[Angular] Is this compatible?

Is this plugin compatible with the latest angular versions?
Thanks!

Yes it is compatible. Because this plugin doesn't require you to register any UI elements.


Show download progress bars

With brew install and language-specific dependency tools, we get a nice progress bar for downloads.
With brew bundle we only see output after something is installed, so it is unclear to the user what is being currently installed and what its progress is. The Command should relay the download progress that brew install provides for a better user experience.

brew bundle --verbose provides a progress bar.


[flash] avoid more than two writes to the same 32-bit word

Hi all, this PR updates the Flash driver and associated unit tests to avoid the issues explained in #4706.
I think it would be best to review this as separate commits. The changes are:

9e7e920: Update the flash driver to avoid marking with kFlagFirst the record with index == 1. This is the most "critical" commit, because it allows the use of the mReserved half-word for the kFlagFirst and kFlagDelete flags. Please review it carefully; it slightly changes the underlying logic for calculating index 0.
cdf7a2a: Update the unit test to check the maximum number of 32-bit words written between erase cycles. This is fixed to 2. See #4706.
d6a1a4e: Implements the new flash format, optionally keeping backward compatibility. The new format uses the mReserved half-word to write the kFlagFirst and kFlagDeleted, but (optionally) understands the old format. The backward compatibility is enabled by default.
fa6ebef: Updates the unit test to check compatibility: writing and reading with the old driver, writing and reading with the new driver, and writing with the old driver and reading with the new one.

Comments are welcome...
Thanks.

@dismirlian , thank you for this contribution! Were you able to get an answer to https://github.com/openthread/openthread/issues/4706#issuecomment-612017804 from Silicon Labs to ensure that considering 32-bit words is sufficient?

Hi @jwhui, no, I haven't had any response. What do you suggest?

Hi @jwhui, no, I haven't had any response. What do you suggest?

@dismirlian, let me try to get an answer from Silicon Labs.


Ads playing?

Is there a state for when ads are playing?

No. Spotify's applescript API does't provide that data.


add more settings

hey cool python module, i am currently building an api and ui for it.
unfortunately i still have a few settings missing, for example reading the actual voltage limit and current limit, as well as set voltage/current values.
also, according to the manual, my power supply offers some functions to set voltage curves. my power supply is the "KIPRIM,DC310S".
thanks for your help

Thanks! Feel free to open a PR with extra functionality, shouldn't be hard to add!

ok too bad, the documentation does not give more.
the support also does not answer.

this repo made it easy to add the missing comamands
https://github.com/maximweb/kiprim-dc310s
have sent you a pull request


Update protoc-docs-plugin

@hzyi-google

I'm not sure if the failing test is related to the change:
You should consider upgrading via the 'pip install --upgrade pip' command.
Command "/usr/bin/python3 -u -c "import setuptools, tokenize;__file__='/tmp/pip-install-xgxe2dz6/grpcio/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/pip-record-w7d3ru_m/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-xgxe2dz6/grpcio/
You are using pip version 10.0.1, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
The command '/bin/sh -c bash install_protoc.sh' returned a non-zero code: 1```

Let's try rerunning the test first.

Bumping the pip version in install_protoc.sh seems to have fixed it. PTAL

Will release artman shortly - I want to update gapic-generator as well :)


Clarification on "angular/flex-layout" support

Documentation Request
Can someone please comment on the use of "angular/flex-layout" with Covalent?
We have an existing Angular app and we already use "angular/flex-layout". We could benefit from some of the Covalent components, but it is not clear what dependencies we are taking on and if Covalent's Media Queries (as described at https://teradata.github.io/covalent/#/components/media) and Covalent's Flexbox Layout ("grid") (as described at https://teradata.github.io/covalent/#/layouts) are such dependencies, are required, and are conflicting with "angular/flex-layout".

You can choose not to use flex layout by not importing the covalent-layout mixin in your sass file. But even if you do import it and use it, it will not conflict.. same with the media service.

closing issue, question answered


Unable to delete the pods in cluster

Not able to delete these pods from the cluster even after applying the force

 Not able to delete these pods from the cluster even after applying the force

you can use command kubevpn reset to delete resources if you don't need it anymore, you can not delete pod if it has a controller, like rs or sts


Support for linking to/embedding large videos, audio and PDFs etc. stored on local file system

It would be useful to be able to embed videos and audio in pages or in a "attachments" view associated with pages (in other apps I find embedded audio/video is often ugly and counterintuitive anyway). A well done gallery or attachments property for pages might even be better than embedding anyway. Perhaps it would be easier to implement as well. As far as in line references to big media files, 99% of the time just being able to embed a link to a local FS file which has already been "aliased" to the page as an attachment to open it in the system default media player like VLC would be good enough.
Regardless, if the developers feel embedding content would be easier or more intuitive for the user they should by all means do this.

A layout like this would solve my problem in 97% of cases. The 3% would be cases where I might have an index page of many (15+) media files. In this case maybe a hideable vertical split panel of files would be good.
As regards linking to the attached/aliased files, I would like to be able to rename individual links so I dont have to read "This-is-the-clip-you-need-to-watch-march-5th-2021.mp4" and instead put "please watch this great clip".
Maybe it should be called aliases instead of attachments because the file stays in the local FS. Anyway, hope that's clear.

Would love to see this


Automatically add the contributers images in readme file and make it more beautifull

Hi can i have this issue i would like to work on this.

Thanks for the recommendation. While that would be awesome, we have considered and decided against that in the past and are not going to do so for foreseeable future.


Incorrect behavior with TypeScript interface

With the configuration:
'prefer-arrow/prefer-arrow-functions': ['warn', {
    allowStandaloneDeclarations: true,
    classPropertiesAllowed: true,
    disallowPrototype: false,
    singleReturnOnly: false,
}],

input:
export interface Example {
    toString(): string;
}

is transformed into:
export interface Example {
    toString: () => string;
}

Sorry, nevermind. It was a change from the @typescript-eslint/method-signature-style.


Small simulation (backward workplaces)

安藤さん
「pDESyについて、バックワードのときはWorkPlaceだけフォワードと違って反転にしないといけないという話があったと思いますがそれを再現する小さいシミュレーションを作ってもらうことはできますか?」

Can't upload ipynb files here.

TEST MODEL
project = BaseProject(init_datetime = datetime.datetime(2021,4,1,8,0,0), unit_timedelta=datetime.timedelta(minutes=1))

# Components in Product
a = BaseComponent('a')
b = BaseComponent('b')

# Register Product including Components in Project
project.product = BaseProduct([a,b])

# Tasks in Workflow
# define work_amount and whether or not to need facility for each task
task_a1 = BaseTask('task_a1', need_facility=True, default_work_amount=4)
task_a2 = BaseTask('task_a2', need_facility=True, default_work_amount=6)
task_a3 = BaseTask('task_a3', need_facility=True, default_work_amount=8)
task_b1 = BaseTask('task_b1', need_facility=True, default_work_amount=4)
task_b2 = BaseTask('task_b2', need_facility=True, default_work_amount=6)
task_b3= BaseTask('task_b3', need_facility=True, default_work_amount=8)

# dependencies between tasks
task_a2.append_input_task(task_a1)
task_a3.append_input_task(task_a2)
task_b2.append_input_task(task_b1)
task_b3.append_input_task(task_b2)
task_b1.append_input_task(task_a1)

# Register Workflow including Tasks in Project
project.workflow = BaseWorkflow([task_a1,task_a2,task_a3,task_b1,task_b2,task_b3])
# workplace in workplace model
# define max_space_size which decide how many components can be placed
workplace1 = BaseWorkplace('workplace1', max_space_size=1.0)
workplace2 = BaseWorkplace('workplace2', max_space_size=1.0)
workplace3 = BaseWorkplace('workplace3', max_space_size=1.0)

# define input_workplace_list (conveyor model)
workplace2.append_input_workplace(workplace1)
workplace3.append_input_workplace(workplace2)
# workplace2.append_input_workplace(workplace3)
# workplace1.append_input_workplace(workplace2)

# facility in workplace model
# define workplace_id (each facility is placed which workplace) and cost_per_time
machine1 = BaseFacility('machine1', workplace_id=workplace1.ID,cost_per_time=10)
machine2 = BaseFacility('machine2', workplace_id=workplace2.ID,cost_per_time=10)
machine3 = BaseFacility('machine3', workplace_id=workplace3.ID,cost_per_time=10)

# define each facility task skill value
machine1.workamount_skill_mean_map = {task_a1.name:1.0, task_b1.name:1.0}
machine2.workamount_skill_mean_map = {task_a2.name:1.0, task_b2.name:1.0}
machine3.workamount_skill_mean_map = {task_a3.name:1.0, task_b3.name:1.0}

# define facilities belonging to wach workplace
workplace1.add_facility(machine1)
workplace2.add_facility(machine2)
workplace3.add_facility(machine3)

#Team in team mode
team = BaseTeam('factory_A')

# worker in team model
# define cost_per_time and add each worker to the relevant team
w1 = BaseWorker('w1', cost_per_time=10.0)
team.add_worker(w1)
w2 = BaseWorker('w2', cost_per_time=10.0)
team.add_worker(w2)
w3 = BaseWorker('w3', cost_per_time=10.0)
team.add_worker(w3)

# define each worker task skill value
# (Set the skill value of an average worker as 1.0)
w1.workamount_skill_mean_map = {task_a1.name:1.0, task_b1.name:1.0}
w2.workamount_skill_mean_map = {task_a2.name:1.0, task_b2.name:1.0}
w3.workamount_skill_mean_map = {task_a3.name:1.0, task_b3.name:1.0}

# define each worker facility skill value
w1.facility_skill_map = {machine1.name:1.0}
w2.facility_skill_map = {machine2.name:1.0}
w3.facility_skill_map = {machine3.name:1.0}

# Register Organization including Team in Project
team_list = [team]
workplace_list = [workplace1,workplace2,workplace3]
project.organization = BaseOrganization(team_list, workplace_list)

# Component <-> Task
a.extend_targeted_task_list([task_a1,task_a2,task_a3])
b.extend_targeted_task_list([task_b1,task_b2,task_b3])

# Team <-> Task
team.extend_targeted_task_list([task_a1,task_a2,task_a3,task_b1,task_b2,task_b3])

# Workplace <-> Task
workplace1.extend_targeted_task_list([task_a1,task_b1])
workplace2.extend_targeted_task_list([task_a2,task_b2])
workplace3.extend_targeted_task_list([task_a3,task_b3])

FS simulation
project.simulate(max_time = 100)

For getting the same result by using BS simulation, the setting of TEST MODEL should be changed as following:
# define input_workplace_list (conveyor model)
# workplace2.append_input_workplace(workplace1)
# workplace3.append_input_workplace(workplace2)
workplace2.append_input_workplace(workplace3)
workplace1.append_input_workplace(workplace2)


Box and whisker bug

Fixes #101
Changes

Fix boxplot crash
Enforced all vega marks for the vegaMarkEncodingMap
Fixed associated type errors

Testing
See #101

Other than this, it looks good!


runnning out of vram

A1111 v1.6
latest inpaint anything version: [4f0d8eba]
Model: sam_vit_h_4b8939.pth
nvidia driver version: 537.34
Windows 11
When just clicking on "Run Segment Anything" after adding a picture, my vram is fully eaten and it is using shared ram

As you can see, i got 12GB VRAM and 32GB shared RAM (i got 64GB Ram) and i don't understand why does it require 46GB VRAM (that i don't have) to do the segmentation.
After 2mn (very long), i got finally the mask

VRAM usage decreased at normal:

Any guidance would be appreciated.
Thanks

I downgraded to nvidia 531.79 version and it stops using my ram.
now got OOM error.
2023-09-27 10:58:11,061 - Inpaint Anything - ERROR - CUDA out of memory. Tried to allocate 8.72 GiB (GPU 0; 12.00 GiB total capacity; 3.31 GiB already allocated; 6.51 GiB free; 3.40 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

I notice it works now on small resolution image up to 1641x2048 but now it takes 1mn.

On a 1920x1080 image, the sam_vit_h_4b8939 model used about 16GB of memory total on my computer. This model is the biggest of the three sam_vit models if I remember correctly so it will use more memory than the others. On top of that, it sounds like you're running segmentation on much larger images.
Keep in mind that increasing the resolution of the image will exponentially increase the required memory. For me, an image over 2500x2500 will eat every last bit of my 10GB of VRAM and 64GB of RAM. Then I just have to wait while my computer slowly crashes.
I'm not really sure if there is a nice solution to this. Do you really need to use the full size images for segmentation? If all you're doing is creating masks, then it may be worth while to scale the images down (while maintaining aspect ratio), run segmentation and create your mask, then scale the mask back up before running inpainting or whatever. But even then I imagine you're going to hit even harder limits to your image size when running the inpainting models.


Add subscription update for Wemo switches, fix bug in Insight switches, fix wemo motion bug, fix wemo discovery

Description:
This PR mainly bumps the pywemo version - which adds subscription_updates for Switch devices (so that the library doesn't call for state info when a switch status is changed). However it also fixes a bug with the Insight Switch subscription update which didn't correctly update when wemo insight switches were turned off.
When doing this I found a problem that the base wemo component which looks not to have been updated with a discovery change. So fixed that too.
Also adds a fix for a problem with Wemo Motion introduced in the last update.
Related issue (if applicable): fixes #7056
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Thanks for this fix! When will it make it to a public release... 42.5? I'd like to get my Wemo motion sensors working again.

@lindsaymarkward I expect it'll be part of 43 - which should be out at the weekend.


Properly end call, update to cct 0.15 rc6, refactoring

Fixes #35, updates to latest cct, and improves some internal stuff.

LGTM!


looping video

Feature request  / 功能建议
Does anyone know if it is possible to force the video to loop seamlessly?
Motivation / 动机
Would be a great feature for posting short videos
Your contribution / 您的贡献
I might be able to implement this with some guidance would gladly contribute this back to the project.

Could you describe it in more detail? Are you trying to extend the video by continuously generating 6-second segments and stitching them together into a longer video? :)

Well - I meant something like this 😉

If interested, I can try to clean this up a bit and submit it as a pull request as a pipeline option or maybe just a standalone demo using callback_on_step_end.

That sounds interesting! Are you connecting the first and last frames of a video to create a 6-second looping animation? If so, I’d be happy to contribute a pull request (PR) as a tool! Looking forward to collaborating.

Well - I meant something like this 😉

If interested, I can try to clean this up a bit and submit it as a pull request as a pipeline option or maybe just a standalone demo using callback_on_step_end.

looks great! came looking for the same

looks great! came looking for the same

Thanks - my version is a quick hack, but I cleaned it up a bit and pushed it to a with_looping branch in my fork in case anyone wants to use it as a starting point. There's a utility called cli_demo_loop.py that will create videos that more or less match at the beginning and end. Here's a few examples:
python cli_demo_loop.py --prompt "A couple are having dinner together."

python cli_demo_loop.py --prompt "A woman is running on the beach in the late afternoon."

python cli_demo_loop.py --prompt "A lion is drinking water."

There's still some glitchiness at the seam and depending on the prompt you might get only an approximate match. For example:
python cli_demo_loop.py --prompt "A man is skiing."

or it might not match at all and just insert a cut frame 🤷♂
python cli_demo_loop.py --prompt "Nightfall in a metropolis."

but when it does work it's pretty neat. I might come back to it and refine it or happy to discuss & improve it with anyone.

https://github.com/feizc/CogvideX-Interpolation/tree/main?tab=readme-ov-file


When forwarding, apply default params from initial and final routes

defaultParams were applied before forwardTo, causing issues when matching paths on start.

Awesome, thanks!

v5.8.2 released

Just wondering: if this fix related to #248? :)

No, just tested it. It does not fix the issue in #248

@ibc no it doesn't, the issue is a separate one

Thanks, then it's just that I found a corner case (a combination of options) that makes it work (but it's too constrained).


Create default security group for AWS clusters

What this PR does / why we need it:
Enables the control plane operator to create a default security group for AWS HostedClusters. If a NodePool does not specify a security group, the default security group is used for its AWSMachineTemplate.
NodePool reconciliation will wait for the security group to be present before proceeding with template creation. For HostedClusters with a control plane operator that does not implement the default security group creation, NodePool reconciliation will proceed as before.
Security group creation status is reported as a condition on the HostedCluster. The security group ID is reported in a platform-specific status of the HostedCluster.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes # HOSTEDCP-789
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes docs.
[x] This change includes unit tests.

I did a first pass, dropped some minor comments. Looks great.

Thanks @enxebre @mjlshen for your reviews! Comments addressed.

@enxebre addressed the last set of comments, ptal

Thanks!
/lgtm
/test e2e-aws

/cherry-pick release-4.12


libdoc error message can't handle path containing non-ascii characters

Hi,
I find libdoc will throw UnicodeDecodeError exception if generate xml file failed and the python file path contain Chinese character . The error message is not accurate enough.
C:\Users\tony\Downloads>python -m robot.libdoc test测试\random.py test.xml
Building library 'test测试\random.py' failed: UnicodeDecodeError: 'ascii' codec
can't decode byte 0xb2 in position 71: ordinal not in range(128)

Try --help for usage information.

I expected this ressult:
C:\Users\tony\Downloads>python -m robot.libdoc test\random.py test.xml
Importing test library 'C:\Users\tony\Downloads\test\random.py' failed: ImportEr
ror: cannot import name is_native_int
Traceback (most recent call last):
  File "C:\Users\tony\Downloads\test\random.py", line 29, in <module>
    from Cryptodome.Util.py3compat import is_native_int
PYTHONPATH:
  C:\Windows\system32\python27.zip
  C:\Python27\DLLs
  C:\Python27\lib
  C:\Python27\lib\plat-win
  C:\Python27\lib\lib-tk
  C:\Python27
  C:\Python27\lib\site-packages
  C:\Python27\lib\site-packages\win32
  C:\Python27\lib\site-packages\win32\lib
  C:\Python27\lib\site-packages\Pythonwin
  C:\Python27\lib\site-packages\wx-2.8-msw-unicode

Try --help for usage information.

I upload the simple demo project
test测试.zip
Python 2.7.16
pycryptodome 3.6.4
Robot Framework 3.1.1 (Python 2.7.16 on win32)
System: Windows 7 SP1 X86_64

I was able to reproduce this problem on my Windows virtual machine with Python 2 but everything worked fine with Python 3. Also on Linux both Python 2 and 3 worked fine.
Interestingly on my Windows machine I was able to use non-ASCII characters like ä in directory names just fine. That may be due to my system using cp1252 and my console using cp850 which both support ä but don't support Chinese.
Could you @kenifanying test does this work with Python 3 on your environment? If it does, I'd highly recommend migrating there because it supports non-ASCII characters in general so much better than Python 2. Python 2 EOL is also very close, so everyone should be migrating to Python 3 also for that reason.
I can still try debugging this a bit more, but because this may be environment config specific it can be hard. Could you @kenifanying tell what encoding your system uses?

I was able to reproduce this problem on my Windows virtual machine with Python 2 but everything worked fine with Python 3. Also on Linux both Python 2 and 3 worked fine.
Interestingly on my Windows machine I was able to use non-ASCII characters like ä in directory names just fine. That may be due to my system using cp1252 and my console using cp850 which both support ä but don't support Chinese.
Could you @kenifanying test does this work with Python 3 on your environment? If it does, I'd highly recommend migrating there because it supports non-ASCII characters in general so much better than Python 2. Python 2 EOL is also very close, so everyone should be migrating to Python 3 also for that reason.

Thank you for your reply. It works with python 3 on my current environment.  I know python 2 EOL is so close, but I can't decide the migration on my company's project.

I can still try debugging this a bit more, but because this may be environment config specific it can be hard. Could you @kenifanying tell what encoding your system uses?

I think the encoding is gbk(or called cp936) in Simplified Chinese Windows environment.

Seems I cannot set cp936:
E:\robotframework>chcp 936
Invalid code page

When you import something like my/path/mylib.py with Robot, what Robot does is adding that my/path to Python's sys.path before importing module mylib. Based on my investigation that happens also in this case, but for some reason that doesn't work with Python 2 on Windows when the path contains Chinese characters (and possibly other non-ASCII characters). My guess is that this is due to the path that's added to sys.path being Unicode while Python 2 in general apparently expects sys.path to be bytes (i.e. str). It's somewhat strange there are no problems when I use a path like hyvä/polku, but that might be due to my Windows system having Western encoding supporting ä in general.
One thing to try is encoding the Unicode path to bytes before adding it to sys.path. The problem is that we needed to know what's the correct encoding is and, annoyingly, the correct encoding depends on the operating system. Windows interestingly also has a different system encoding and console encoding, and both of them depend on the system configuration. Linux (and OSX) nowadays using UTF-8 everywhere is probably the reason this particular problem doesn't occur on Linux even with Python 2.
We have some pretty advanced utilities that try to sniff what the correct encoding is and they could possibly be used to encode the Unicode path to bytes. My understanding is that on Windows sys.path is encoded using the system encoding, so we could use our system_encode utility. It should be used in robot/utils.importer.py as shown by the diff below:
diff --git a/src/robot/utils/importer.py b/src/robot/utils/importer.py
index 1b0a47f..c76dd82 100644
--- a/src/robot/utils/importer.py
+++ b/src/robot/utils/importer.py
@@ -19,7 +19,7 @@ import inspect

 from robot.errors import DataError

-from .encoding import system_decode
+from .encoding import system_decode, system_encode
 from .error import get_error_details
 from .platform import JYTHON, IRONPYTHON, PY3
 from .robotpath import abspath, normpath
@@ -239,6 +239,7 @@ class ByPathImporter(_Importer):

     def _import_by_path(self, path):
         module_dir, module_name = self._split_path_to_module(path)
+        module_dir = system_encode(module_dir)
         sys.path.insert(0, module_dir)
         try:
             return self._import(module_name)

I tested the above changes but they had no effect on my machine. That's not really a surprise, though, because system_encode uses cp1250 on my environment and it cannot encode test测试 correct. The fix could work on environments where the system encoding actually supports that string, so it would be great if you @kenifanying could test this.

A possible workaround for this problem is importing the library by name, not by path. In practice that means not importing like my/path/mylib.py, but instead adding my/path to PYTHONPATH before execution and then importing just mylib.

When you import something like my/path/mylib.py with Robot, what Robot does is adding that my/path to Python's sys.path before importing module mylib. Based on my investigation that happens also in this case, but for some reason that doesn't work with Python 2 on Windows when the path contains Chinese characters (and possibly other non-ASCII characters). My guess is that this is due to the path that's added to sys.path being Unicode while Python 2 in general apparently expects sys.path to be bytes (i.e. str). It's somewhat strange there are no problems when I use a path like hyvä/polku, but that might be due to my Windows system having Western encoding supporting ä in general.
One thing to try is encoding the Unicode path to bytes before adding it to sys.path. The problem is that we needed to know what's the correct encoding is and, annoyingly, the correct encoding depends on the operating system. Windows interestingly also has a different system encoding and console encoding, and both of them depend on the system configuration. Linux (and OSX) nowadays using UTF-8 everywhere is probably the reason this particular problem doesn't occur on Linux even with Python 2.
We have some pretty advanced utilities that try to sniff what the correct encoding is and they could possibly be used to encode the Unicode path to bytes. My understanding is that on Windows sys.path is encoded using the system encoding, so we could use our system_encode utility. It should be used in robot/utils.importer.py as shown by the diff below:
diff --git a/src/robot/utils/importer.py b/src/robot/utils/importer.py
index 1b0a47f..c76dd82 100644
--- a/src/robot/utils/importer.py
+++ b/src/robot/utils/importer.py
@@ -19,7 +19,7 @@ import inspect

 from robot.errors import DataError

-from .encoding import system_decode
+from .encoding import system_decode, system_encode
 from .error import get_error_details
 from .platform import JYTHON, IRONPYTHON, PY3
 from .robotpath import abspath, normpath
@@ -239,6 +239,7 @@ class ByPathImporter(_Importer):

     def _import_by_path(self, path):
         module_dir, module_name = self._split_path_to_module(path)
+        module_dir = system_encode(module_dir)
         sys.path.insert(0, module_dir)
         try:
             return self._import(module_name)

I tested the above changes but they had no effect on my machine. That's not really a surprise, though, because system_encode uses cp1250 on my environment and it cannot encode test测试 correct. The fix could work on environments where the system encoding actually supports that string, so it would be great if you @kenifanying could test this.

I test the robot/utils.importer.py code changes, but it still throw same UnicodeDecodeError exception.

This problem affected also library imports during execution, and in that case the whole execution crashed making the problem a lot more severe than I initially thought. Luckily it only occurs if there's an error importing the library, the path contains non-ASCII characters, and you are on Python 2.
The same problem also caused failure traceback (logged on DEBUG level when a keyword fails) to have non-ASCII paths printed wrong. In that case non-ASCII characters like ä were shown like \xc3\xa4 so this wasn't too severe.
Fixing all this was also a bit more complicated than I though. With Python 2 traceback entries are bytes, and as I had noticed earlier, paths were encoded using the system encoding. Entries also contain the line where the error occurs, and its encoding depends on the source encoding of the file where that line is located. In Python 2 the source encoding is ASCII by default, but you can set it to something else with a magic comment like # coding=UTF-8. As a result entries can contain items with different encoding and we cannot even know what encoding the text actually uses. I decided to handle this so that paths are decoded using the system encoding, but text is decoded using UTF-8 (an ASCII compatible guess) so that error handler is set to 'replace'. That ought to avoid all UnicodeDecodeErrors and tracebacks ought to be shown correctly in over 99,9% of cases.


Add a help page & logging when gateway starts

Help Page and Logs

This PR introduces a webpage that opens up when the gateway is started through node.js
The webpage contains help information on using the gateway
The logs tab provides realtime running logs for requests made through the gateway

How to try it out

Pull this branch and run the gateway normally through the npm run dev:node command.

Motivation:

Besides helping new users to consume gateway features fast, it also helps identify bugs faster through the logging section.

Tested docker, npx deployments

@roh26it - Should we also put a env or some flag check to enable the help page and logging? For example, production env might not need the UI.

Aah, yes we could do that. Let me add a production check to env as well.

@VisargD made the requested changes


fix: smaller snapshots

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fix, to avoid Core running out of memory in some situations.

What is the current behavior? (You can also link to an open issue here)
The RundownPlaylist-snapshots include an unlimited amount of ExpectedPackageWorkStatuses, PackageContainerPackageStatuses and PackageInfos.

What is the new behavior (if this is a feature change)?
ExpectedPackageWorkStatuses, PackageContainerPackageStatuses and PackageInfos are limited to 500 items each.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Base: 68.35% // Head: 68.35% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (6a1861f) compared to base (cc28bcb).
Patch coverage: 70.58% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##           release44     #836      +/-   ##
=============================================
- Coverage      68.35%   68.35%   -0.01%
=============================================
  Files            279      279
  Lines          33742    33748       +6
  Branches        4511     4513       +2
=============================================
+ Hits           23065    23069       +4
- Misses         10223    10225       +2
  Partials         454      454

Impacted Files
Coverage Δ

meteor/server/api/snapshot.ts
33.14% <0.00%> (-0.10%)
:arrow_down:

meteor/lib/api/methods.ts
76.92% <90.00%> (-0.22%)
:arrow_down:

meteor/lib/lib.ts
67.04% <100.00%> (+0.18%)
:arrow_up:

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

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


Error on Build with gradle-7.2  - :amplitude_react-native

Failure build using gradle 7.2

Packages:

"@amplitude/react-native": "2.4.0",
"react": "17.0.2",
"react-native": "0.65.1",
classpath("com.android.tools.build:gradle:7.0.2")
distributionUrl=https://services.gradle.org/distributions/gradle-7.2-all.zip

Step Reproduce:

npx react-native run-android

Error:
FAILURE: Build failed with an exception.

* Where:
Build file<EMAIL_ADDRESS>line: 14

* What went wrong:
A problem occurred evaluating project ':amplitude_react-native'.
> Failed to apply plugin 'com.android.internal.library'.
   > Android Gradle plugin requires Java 11 to run. You are currently using Java 1.8.
     You can try some of the following options:
       - changing the IDE settings.
       - changing the JAVA_HOME environment variable.
       - changing `org.gradle.java.home` in `gradle.properties`.

Hi @mayconline,
I don't think is our @amplitude/react-native issue. That's related to the android studio setting.
https://stackoverflow.com/questions/68728918/error-android-gradle-plugin-requires-java-11-to-run-you-are-currently-using-ja. Hope this link can help you solve the issue.
Thanks.

thanks, sorry for the issue,
i thought it was a amplitude issue, but this link helped me ^^, i was still using java 1.8.


[poem-openapi] Path parameters in prefix_path do not generate the correct openapi specification

Hello !
I have an api specification of the form :
#[OpenApi(prefix_path="/hello/:name")]
impl Api {
    #[oai(path = "/:surname", method = "get")]
    async fn get(...) -> ... { ... }
}

Note that I have path parameters in both prefix_path and path.
Expected Behavior
I expect the parameter name to be included in the api spec, so the generated spec should be something like
"paths": {
    "/hello/{name}/{surname}": {
      "get": {
        "parameters": [
          {
            "name": "name",
            ...
          },
          {
            "name": "surname",
            ...
          }
        ],
}}}

Actual Behavior
Instead, what is being generated is
"paths": {
    "/hello/:name/{surname}": {
      "get": {
        "parameters": [
          {
            "name": "name",
            ...
          },
          {
            "name": "surname",
            ...
          }
        ],
}}}

This causes problems with api viewers (tested with redoc and swagger-ui), as they do not generate the correct request.
Expected:
curl -X GET http://localhost/api/hello/myname/mysurname

Actual
curl -X GET http://localhost/api/hello/:name/mysurname

Additional testing
For the record, I tested the following api construction in Rust:
#[OpenApi(prefix_path="/hello/{name}")]
impl Api {
    #[oai(path = "/:surname", method = "get")]
    async fn get(...) -> ... { ... }
}

This generates the correct specification, but poem is then unable to correctly use the route and I get a 404 error when trying to access it.
Specifications

Version: poem 3.0.1, poem-openapi 5.0.2
Platform: tested on MacOS and Linux (probably platform independant)
Subsystem: poem-openapi

 Full code example
use poem::{listener::TcpListener, Route, Server};
use poem_openapi::{param::{Path, Query}, payload::PlainText, OpenApi, OpenApiService};

struct Api;

#[OpenApi(prefix_path="/hello/:name")]
impl Api {
    #[oai(path = "/:surname", method = "get")]
    async fn index(&self, name: Path<String>, surname: Path<String>) -> PlainText<String> {
        let name = name.0;
        let surname = surname.0;
        PlainText(format!("hello, {name} {surname}!"))
    }
}

#[tokio::main]
async fn main() -> Result<(), std::io::Error> {
    if std::env::var_os("RUST_LOG").is_none() {
        std::env::set_var("RUST_LOG", "poem=debug");
    }
    tracing_subscriber::fmt::init();

    let api_service =
        OpenApiService::new(Api, "Hello World", "1.0").server("http://localhost:3000/api");

    let ui = api_service.swagger_ui();
    let spec = api_service.spec();

    println!("{}", spec);

    Server::new(TcpListener::bind("<IP_ADDRESS>:3000"))
        .run(Route::new().nest("/api", api_service).nest("/", ui))
        .await
}

So I found the problem at this line.
When the full path is generated below, no conversion is applied to the prefix_path, only to the oai_path. This means the path is half-standardized in the final output.

So I searched a bit for this problem. There is indeed no genereted oai path for the prefix_path attribute. However, due to this line:
https://github.com/poem-web/poem/blob/f81424dedeec44cbdf4dbe2e828b0d3bbf579486/poem-openapi-derive/src/api.rs#L26-L27
The prefix_path can actually be a variable, and not just a string literal. This makes it impossible to validate that path unless a breaking change is introduced requirering it to be a string literal just like the regular path.
There is already a test for this case:
https://github.com/poem-web/poem/blob/f81424dedeec44cbdf4dbe2e828b0d3bbf579486/poem-openapi/tests/api.rs#L133-L144


File disappeared

Excuse me, is the source code fully open source? There are a lot of problems in the process of reproduction, and some files will be missing

Yes, I encountered the same issue. A lot of problems...

Thanks for your interest and for pointing this out. There was a missing file and a few minor bugs, which have now been fixed. I’ve checked the repository, and it runs as expected. Please pull the latest changes and try again.
Best,
Avik


ENH: Reconstruction?

Would people be interested in looking at the extent to which e.g. Evoked data can be reconstructed following projections (un-doing the induced signal bias)? Here is the idea:
import os.path as op

import mne
from mnefun import plot_reconstruction

data_path = mne.datasets.sample.data_path()
raw = mne.io.read_raw_fif(op.join(data_path, 'MEG', 'sample',
                          'sample_audvis_raw.fif'), preload=True)
raw.pick_types(meg=True, eeg=False, eog=True, stim=True, exclude=())
events = mne.find_events(raw, 'STI 014')
raw.add_proj(mne.preprocessing.compute_proj_eog(raw)[0])
raw.add_proj(mne.preprocessing.compute_proj_ecg(raw)[0])
epochs = mne.Epochs(raw, events, {'Left auditory': 1}, -0.2, 0.5,
                    baseline=(None, 0), proj='delayed',
                    reject=dict(grad=4000e-13, mag=4e-12, eog=150e-6))
evoked = epochs.average()
evoked.plot_reconstruction()

Would give you something like:

The "reconstructed" data is obtained by projecting the (SSP-projected) data to a sphere using MNE and coming back out. The critical step is that the projection operator is applied to the lead fields as well as the data, thereby undoing the bias to the data.
In Xfit, this is known as reconstruction "on" vs "off", and helps visualize whether or not your projectors have eliminated a true signal of interest. Techniques like dipole fitting and minimum-norm estimates both do this type of compensation during estimation, so it gives you a better idea in sensor space of how much of your signal of interest is still there.

nice.
this should be made public. We'll see what's the best way to expose this.

Cool, will do.
It's pretty straightforward from an implementation standpoint, except that right now we only have _map_meg_channels using MNE -- _map_eeg_channels uses spherical splines which will not work (not sure why we didn't use MNE for that?). So i'll need to add a method arg to _map_eeg_channels, then it should only be ~30 lines or so for the viz function.


Update metadata files

Move tautology metadata files into the .meta directory, and
delete unnecessary metadata.yml files for problems from the common pool.
The .meta thing is new as of today. Instead of putting description.md and metadata.yml in the root of the exercise directory, it should now go in .meta.
I want to clarify (this is not documented—apologies for the confusion) one thing: if an exercise comes from the common pool of exercises then it doesn't need description.md and metadata.yml. In general, if you're basing an exercise off of an existing one in the OCaml track or any other track, then it's probably in the common pool, defined in the x-common repository: https://github.com/exercism/x-common/tree/master/exercises)
If the exercise you're basing it on is custom, then it will have the description.md and metadata.yml files. In that case, the right move would be to move the metadata/description into the common pool, and then delete the files from .meta/

I'm going to go ahead and merge this, since it will fix a downstream build.


New version: GeoStats v0.9.0

Registering package: GeoStats
Repository: https://github.com/juliohm/GeoStats.jl
Created by: @juliohm
Version: v0.9.0
Commit: 64fe9bae215321add70fc6acee17bc006e922bf7
Reviewed by: @juliohm
Reference: https://github.com/juliohm/GeoStats.jl/commit/64fe9bae215321add70fc6acee17bc006e922bf7#commitcomment-33862117

Depends on #1277 and #1276

Just curious; why do you have these as three separate packages if you always release them all together anyway? They don't seem to be very independent.

I think it is because the codebase is still under heavy development. But I am actually considering merging GeoStatsBase.jl and GeoStatsDevTools.jl into one.


Match font in Doctor

to what we have in docs.minio.io. Exact match to size and type.

Completed.


Multiple Language Support

Currently English is the only option (hard-coded). This needs to be dynamic and support the current locale from Dnn.

I have tested the module today. Not all labels will be shown correctly. I think there are fields, which are in the RESX like "Edit page" on Tab Setting, but it will display only the english text.


ChimeraOS does not start after installation

Hello,
It appears these messages when turning on the PC but does nothing else.

The PC has 2 SSD disks, it has been tried to install on both disks and the result is the same.
Can help us?

What GPU do you have?

What GPU do you have?

PCI port: ATI Radeon HD 4670
My motherboard has integrated Intel GMA 3100 graphics.

Oh my, that is way too old and not supported.
Supported GPUs are listed on the download page:
Nvidia GTX 900 / AMD RX 400 / Intel HD Graphics 500 series or newer

Oh, my hardware is very old, thanks for the information!


dep(*): update tidb to include DM SchemaTracker fix

What problem does this PR solve?

Issue Number: close #11408
What is changed and how it works?
to include https://github.com/pingcap/tidb/pull/54761
Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Fix DM fails to replicate DDL of LIST partitioning tables.

/retest


Install script

curl -fsSL https://raw.githubusercontent.com/Wiredcraft/pipelines/install-script/install.sh | sh -s dev/stable

./install.sh dev/stable

@zbal / @JuhaS  got a minute to review?

as soon as the "help" is added @sunsongxp - please go ahead and merge.

@sunsongxp done

Let's get moving - not optimal - can fix later...


Download from site to local repository

I can rate everything related to uploading to FTP at 5. But, there is no reverse upload from the site.
I really hope this option is available.
And there will be a full-fledged deployer, as it exists in other editors.

Agreed - I have to go to other systems to do this.  Awkward.

Hi, I have released a new version 2.3.1. Now you can download or compare files.


Use IDeliveryClient wherever possible

Places like internal DeliveryItemResponse(JToken response, DeliveryClient client) should use IDeliveryClient.
The concrete implementation - DeliveryClient - shouldn't be used in the SDK at all.

fixed in the related PR


error when loading data json

i loaded data from json. but the chart not show and many error.
how to fix it, please help me.
thanks very much

There are many tutorials, snippets and resources showing how to use JSON with D3.js.
It is not matter of techan.js.
(Anyway it seems your data contains NaN values: maybe you wrongly mapped JSON data and D3.js structure).


Uncaught TypeError: Expected Buffer

on api example, it says :

input must be a Buffer or an Array

but gives error when running :
bs58.encode(new Uint8Array([1, 2])); or
bs58.encode([1, 2]);
bundled using browserify and running on chrome (latest version)

This was introduced in<EMAIL_ADDRESS>and was a breaking change from 3.0.4, so anyone that re-installed or re-packed would get errors if they didn't use Buffers.
https://github.com/cryptocoinjs/base-x/issues

Closing this issue for the other:
https://github.com/cryptocoinjs/base-x/issues/63


support for promise

first of all, thank you @typicode for this project, this is one of my favourite tools which makes me more productive.
I always use json-server with jsf. But with new version of jsf it is async now. So my index.js looks like below.
var jsf = require('json-schema-faker')
const schema = require('./schema')

module.exports = () => {
  return jsf.resolve(schema).then(data => {
    return { users: data }
  })
}

now if I run json-server index.js it throws error Data must be an object. Found object.See https://github.com/typicode/json-server for example
To support this can we add promise support in json-server?
I would be happy to send a PR for this.
Feel free to close this issue if you think its out of scope for json-server

Any progress on this issue?

Just stumbled across this issue while also trying to use the latest version of JSON Fake Schema. Would really like this to be supported!

async/await is still not supported


fix: remove timely formatting

The type of this PR is: Fix
This PR solves GRO-1354
Description
Force should not be doing its own formatting for sale. displayTimelyAt, as Metaphysics already does it for clients.
Related Metaphysics deploy.

I'm going to self-merge so I can test on staging.


Adjust the time zone inside the signalserver

Dear Ladies and Gentlemen.
In the meantime, I have installed and set up a signal server based on a normal Intel PC (64 bit). He works very well too.
After loading the first Qctools analytics data to the signal server, I should commit that the upload date is not the right time. There is always a deviation of 6 hours.
As it stands, the time zone in the signal server does not match the host.
So far I have tried to extend the yml file by the following lines. This did not fix the problem.
environment:
- SET_CONTAINER_TIMEZONE=true
- CONTAINER_TIMEZONE=Europe/Berlin
volumes:
- "/etc/timezone:/etc/timezone:ro"
- "/etc/localtime:/etc/localtime:ro"
Here are some dates and version numbers of my installation:
qcli 0.9Apr 10 2017 21:00:02
Docker version 18.02.0-CE
Everything is installed on a Debian Buster Alpha2 with the latest kernel, patches and updates.
How can I adjust the time zone in the Docker containers to be in sync with the host?
Many thanks in advance.
Marco

Good afternoon,
As far as I can see: containers, strictly speaking, are properly set to UTC (which is really fine).
Nevertheless, django applications (applications, not containers) have their own timezone set.
As far as I tested, you can change used TZ in signalserver/settings.py, changing the TIME_ZONE variable, which is currently set to US/Eastern.
I hope this help.


Update aws-lambda-java-log4j2

See https://aws.amazon.com/security/security-bulletins/AWS-2021-005/
Customers using the aws-lambda-java-log4j2 (https://repo1.maven.org/maven2/com/amazonaws/aws-lambda-java-log4j2/) library in their functions will need to update to version 1.3.0 and redeploy.

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

Thank you for your submission, @OllieJC ! We require both signed commits and signing the CLA for this project.
I should mention that this update is only in an example, and log4j is not a dependency of this plugin.

Thanks @mrickard, I'm not strictly the originator of the contribution so won't be signing the CLA. I'm happy for this to be closed and recreated

@OllieJC Great--thank you again for contributing! I'll close this and open a signed one.


Document usage

There's currently no documentation on how to run the POET server. Add at least a basic "# Running the server" section to the README. It should contain instructions on how to connect go-spacemesh to it.
Happy to work on this.
@moshababo, does anything special need to be done? When I run poet locally, I noticed that it's looking for a poet.conf. Can we add a sample of this to the repo? Do I need to supply any commandline args? Or should go-spacemesh connect to it (on the default port) with no special params/config required?

@lrettig sorry for the delay. This was addressed by https://github.com/spacemeshos/poet/pull/95. Please verify.

Looks good! I was able to build and run a poet server locally using the updated README. Thanks!


feat: implement google gemini chatbot

Description
Implement the google Gemini chatbot using google.generativeai and gradio.
Acceptance Criteria

[] implement multimodal chatbot using gemini and wrap it with commune.
[] Implement simple UI

How Has This Been Tested?

[] Locally
[] Unit Tests
[] Staging
[] Other

https://github.com/commune-ai/commune/assets/19830274/c93e2b83-4c1c-4e0c-86a4-9f1d61fa0ba4

Add gemini module.


Fix e2e tests - Check if this works on CI

Description

Last scenario in viz.feature fails, which I was able to reproduce manually in a clean environment. It is actually expected behaviour, we should update our tests.
Development notes

We don't run kedro install after creating a Kedro project in e2e tests, which is an incomplete workflow.
QA notes

Somehow this failed on my machine with a weird error (https://stackoverflow.com/questions/54093020/no-module-named-pandas-libs-tslib/54539768) complaining about pandas already being installed, so checking if this is reproduced on CI. If yes, it's sth weird we're doing in behave setup.
Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

Legal notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Great find! Have it been passing purely because of CircleCI caching?

Potentially. I'm not sure what's happening, it's failing locally for me and @DmitriiDeriabinQB .


Documentation for markerClick

The documentation says the markerClick function has the following signature:
    markerClick: function(chartContext, { seriesIndex, dataPointIndex, config}) {
      // ...
    }

But it looks like it's actually:
    markerClick: function(mouseEvent, chartContext, { seriesIndex, dataPointIndex, config}) {
      // ...
    }

Thanks for the catch!
Fixed it.


Post tech-review tasks

[x] Migrate to github.com/ansys
[x] Change the URLs from github.com/ansys-internal to github.com/ansys
[x] Set up branch and tag protection
[x] Generate PyPI token, right before going public
[x] Uncomment public PyPI release action
[x] Transfer visibility to public
[x] Do a first release to PyPI
[x] After going public, allowing codecov.io GH app  to access this repository and store codecov token as secret
[x] Add this project to https://github.com/ansys-internal/pyansys-dev
[x] Add project to metapackage
[x] Remove code-style action in favor of pre-commit.ci

Closing since all tasks have been performed


FEAT Helper to check if a model is a PEFT model

After internal discussion, we decided to add this convenience function.
>>> from peft.helpers import check_is_peft_model
>>> check_is_peft_model("gpt2")
False
>>> check_is_peft_model("peft-internal-testing/gpt2-lora-random")
True
>>> check_is_peft_model("peft-internal-testing/non-existing-model")
False
>>> check_is_peft_model("path/to/local/model/also/works")

Ping @stas00

Excellent, thank you @BenjaminBossan.
The only possible suggestion is for a different naming as check and is are both verbs, so perhaps either is_peft_model or check_if_peft_model?
I'm not sure if you have a precedent for other similar check-if utils to follow the same convention.

The only possible suggestion is for a different naming as check and is are both verbs, so perhaps either is_peft_model or check_if_peft_model?

Good point, I renamed it to check_if_peft_model. My reasoning for not using is_peft_model is that I often want to call the variable that name in similar situations, e.g. is_peft_model = check_if_peft_model(...); if is_peft_model: ...

yes, I totally agree that:
is_peft_model = check_if_peft_model()

sounds perfect to me.


Provide info about annotated functions

This is a loose idea but I often add a service to dependencies of another one and then remove its uses but forget to remove the injection. Now, linters often have an option to report unused parameters of functions not only at the end of them, e.g. see:
http://eslint.org/docs/rules/no-unused-vars.html
with {"args": "all"} but then I'd have lots of false positives since in case of event handlers, replace functions on strings etc. we're sometimes interested in a second parameter but the first still has to be there.
Since ng-annotate already figures out which functions need to be annotated, I wonder if it was possible to pass this info along in some ways so that it'd be possible to check for unused injections in outside projects.
Or maybe ng-annotate could do this checking on its own? It rarely makes sense to inject components that aren't used in the code after all...

https://github.com/olov/ng-annotate/issues/245


Requests to /sync endpoint resulting in 403

Describe the bug
Hi there, first off, thanks for the software!  It's entirely possible this is an error on my end, but I've gone over the documentation and worked on this issue for a while.
I am writing a script that's purpose is to call the /sync endpoint on every single bucket.  I have authentication working for every endpoint I need besides /sync.  Calling for example /bucket to list buckets works great, as does /status.  However, when I call the /sync endpoint I get either two types of results.
Result 1
Sync response for bucket 98b739a2-ca0e-4f02-b215-cc4d55e3b064:
{"type":"https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/403","title":"Forbidden","status":403,"detail":{"name":"SignatureDoesNotMatch","$fault":"client","$metadata":{"httpStatusCode":403,"requestId":"8e22ee0a:19021c74657:b5c27:319a","extendedRequestId":"","attempts":1,"totalRetryDelay":0},"Code":"SignatureDoesNotMatch","RequestId":"8e22ee0a:19021c74657:b5c27:319a","message":"The request signature we calculated does not match the signature you provided. Check your Secret Access Key and signing method. For more information, see REST Authentication and SOAP Authentication for details."}}

This result type is 90% of my responses.
Result 2
This result is far more rare, and just says:
0

I'm assuming a response of 0 is following the Linux exit code convention where 0 = success.
(If useful, my full logs are this, but the first two lines are my scripts logging lines)
GOING TO SYNC 19a94bd1-bd39-4d31-9b4e-2fce4d9314a4
Sync response for bucket 19a94bd1-bd39-4d31-9b4e-2fce4d9314a4:
0

Version Number

To Reproduce
Steps to reproduce the behavior:
For simplicity I have copied my entire script here.  It refers to an .env file which I will not include:
if [[ -f .env ]]; then
    export $(cat .env | grep -v '^#' | xargs)
else
    echo "Error: .env file not found"
    exit 1
fi

# Ensure CLIENT_ID and CLIENT_SECRET are set from .env
if [[ -z "$CLIENT_ID" || -z "$CLIENT_SECRET" ]]; then
    echo "Error: CLIENT_ID or CLIENT_SECRET not set in .env file"
    exit 1
fi

# Fetch the access token using client credentials grant
TOKEN=$(
    curl --location --request POST '[KEYCLOAK-URL-HERE]/auth/realms/[KEYCLOAK-REALM-HERE]/protocol/openid-connect/token' \
    -s \
    --header 'Content-Type: application/x-www-form-urlencoded' \
    --data-urlencode "grant_type=client_credentials" \
    --data-urlencode "client_id=$CLIENT_ID" \
    --data-urlencode "client_secret=$CLIENT_SECRET" |
    jq -r .access_token
)

# List of all buckets - note each application has it's own 'bucket'. This is not the same bucket as OCIO provided us, rather
# this is a "COMS Bucket" that is created every time we create a COMS mounting.
BUCKET=$(curl -X GET "[COMS-URL-HERE]api/v1/bucket/" \
-H "Authorization: Bearer $TOKEN" )
echo "\nBUCKET: $BUCKET"
bucketIds=$(echo "$BUCKET" | jq -r '.[].bucketId')

echo "\nBucketIDs: $bucketIds"

for BUCKET_TO_SYNC in $bucketIds; do
    echo "GOING TO SYNC $BUCKET_TO_SYNC"
    # Make the API request to sync each bucket
    SYNC_RESPONSE=$(curl -s -X GET "[COMS-URL-HERE]api/v1/bucket/$BUCKET_TO_SYNC/sync" \
    -H "Authorization: Bearer $TOKEN")

    # Process the SYNC_RESPONSE as needed
    echo "Sync response for bucket $BUCKET_TO_SYNC:"
    echo "$SYNC_RESPONSE"
    echo  # Add a blank line for separation
done

In the .env file, there are two variables:
CLIENT_ID=c
CLIENT_SECRET=

Expected behavior
Since my authentication works for other endpoints (like /bucket), I expect it to work for /sync.
My question: does /sync have different auth requirements from /bucket? For example do I need to configure the Keycloak clients differently or setup any additional permissions to call /sync on a bucket?
Screenshots
N/A
Desktop (please complete the following information):

OS: Using Mac, but running a bash script
Browser N/A, CLI
Version N/A

Smartphone (please complete the following information):
N/A
Additional context

Hi @acoard-aot ! Auth should be the same across all endpoints as far as I'm aware. I presume you're self-hosting COMS?
Based on the error you're seeing though, from a quick search it appears to be related to S3's authentication and not so much COMS. Best guess is that it's some weird edge case, perhaps with characters in the S3 credentials that require escaping (e.g. /, +) but it'll probably require more investigation.

Still investigating this bug (I've managed to reproduce it locally when running in basic auth mode, so almost certainly something to do with S3 rather than JWT/OIDC/Keycloak etc), but in the meantime:

This result is far more rare, and just says:
0

I'm assuming a response of 0 is following the Linux exit code convention where 0 = success.

The /sync endpoint returns the number of objects that have been added to the queue. In this particular case, nothing was added (e.g. the bucket is empty).

Out of the box, the /bucket/<bucket id>/sync endpoint requires either basic auth or an ID token of a user.
We don't currently support client credentials authentication.
Did you modify your own instance of COMS?
what kind of token would you be using?
if youre using basic auth, you wuld just need this username/password: https://github.com/bcgov/common-object-management-service/blob/43693b6d932f9c67417d592773b2751a26b73d05/app/config/custom-environment-variables.json#L4

Looks like the only way I was able to replicate the issue was with incorrect S3 credentials.
COMS will verify that they work when you initially create the COMS bucket, but if the S3 credentials change after that it's not going to know until it tries to do something with the underlying S3 bucket (e.g. reading directly from it).
Updating the S3 credentials should fix it - you can do that with PATCH /bucket/:bucketID.
Just out of curiosity though, what's your setup here? Like Tim said we don't support client credentials at the moment so I'm interested in how you implemented an automated script without requiring manual user login.

Thank you for the answers you've given here, I really appreciate it.  I've been incredibly busy, but will be diving into this issue in more detail today.  Perhaps it is a cred issue, and I appreciate the tip.  I'm a bit surprised that the creds work for some "buckets" (in COMS) and not all when I don't believe that's how it's configured (we have one shared S3 bucket with same auth for this POC), but it's entirely possible I messed something up.  Thanks again.
I will make sure to report back and close this ticket with my findings in the next couple days.


Add new cri log support

Support https://github.com/kubernetes/kubernetes/pull/55922.

I'll send another PR to update cri-tools.


MissingDeclSyntax Can Have Attributes and Modifiers

Strange as it sounds, we parse
@foo public
in item position as a decl that has a hanging attribute and access
control modifier. We need to be able to stick... something in the
tree here so we don't just drop these tokens on the floor.

apple/swift-syntax#578
@swift-ci smoke test


allow alignment of ordinal.rangeRoundBands

ordinal.rangeRoundBands is very useful to create nice, sharp bar charts. To accomplish this it calculates error pixels and distributes them 50/50 to the start and end of the range.
However when creating a bar chart often an author wishes to left align it. Error pixels are also sometimes incorrectly identified as padding (probably in pursuit of alignment) - see https://github.com/mbostock/d3/issues/1592.
As an api consumer it would be nice to decide where those pixels go.
Code:
https://github.com/mbostock/d3/blob/master/src/scale/ordinal.js#L72
Maybe an additional parameter? align: center, default = Math.round(error / 2), left = 0, right = error.

Nice idea. Filed d3/d3-scale#16 for 4.0.

Fixed in d3/d3-scale#28 for D3 4.0.


Add project/collection as a material type of external activities.

PT story: https://www.pivotaltracker.com/story/show/172153300
Related portal-pages PR: https://github.com/concord-consortium/portal-pages/pull/121

@emcelroy This looks good to me.  Is there anything else you want to do on it, or should I merge it in?

No, that's it. Ready to merge. Thanks.


StringList typemaps in Python3 don't work

See https://gist.github.com/karlnapf/b9122a880c9954cf795f20471b12363c
This was happening on cloud.shogun.ml which runs python3.
This task is to

reproduce the bug locally
investigate whether it only happens in Python or also in other languages
investigate what is happening
send a PR that fixes it

Nontrivial entrance task for anyone interested in how Shogun typemaps work. Check out shogun/src/interfaces/python/swig_typemaps.i to see how the StringList is copied.

Can I work in this issue if it is still open?

@DhruvThakker I'm still working on it and send the PR soon


loadModule crashes on syntax error in slang file, slangc doesn't

I have a slang file where I accidentally used a variable mvp without declaring it, like so
  out.position = mul(mvp, float4(vertexIn.position, 1.0));

Compiling this slang file via loadModule results in an AbortCompilationException with no useful message as far as I can tell, while using slangc gives me a useful error:
.\debug.slang(18): error 30015: undefined identifier 'mvp'.
  out.position = mul(mvp, float4(vertexIn.position, 1.0));

Is there a way to get similar helpful information from loadModule?

If you ignore the AbortCompilationException and continue execute the code until after the loadModule call, were you able to get any diagnostic message in the outDiagnostics blob?

I'll try and check again later, but I don't think AbortCompilationException is something that gets thrown across the COM boundary? It gets picked up as an uncaught except in the slang dll file.
I also am not sure where I would get it's definition, since it's not included via slang.h, and I don't thing I'm meant to include core/slang-exceptions.h

Hmm, if you see AbortCompilationException leaking through COM boundary then that is definitely a bug. We'll take a look at this.

I am able to confirm this issue and has fixed it in the linked PR. The fix will be available in the next release. Thanks for reporting the issue!


Create a name mapping infrastructure

There needs to be a way to save existing application structures so that it's possible to avoid boilerplate code that drills down into an object hierarchy.
For example, instead of doing this:
def saveFile(file):
    app = pywinauto.Application(backend='uia')
    appconnect = app.connect(path='sldworks.exe')
    window = appconnect.window()
    window.set_focus()
    button = window.child_window("Save")
    #This won't work but you get the idea
    wrapper = input.wrapper_object()
    wrapper.sendkeys(file)
    button.click()

We could instead do something more along the lines of this:
def saveFile(file):
    TestedApps().SolidWorks.dlgSaveAs.Edit.SetText(file)
    TestedApps().SolidWorks.dlgSaveAs.btnSave.Click()

This would enable auto completion in IDEs like PyCharm so that it would be possible to write entire scripts without opening the tested application and feeling out the correct properties and windows to search for.
Several test runners already do something like this with XML mapping to build a pre-defined object model. The biggest ones that I am familiar with are TestComplete and the GUI automation already built into Visual Studio Enterprise. In both instances, they call it a name map.

Hi @shalexxo , this might be a good idea. But personally I can't consider it as a high priority. Our primary efforts are all about making pywinauto really cross-platform. So next steps after 0.6.x series are AT-SPI accessibility technology support (probably 0.7.x) and Apple Accessibility API support (0.8.x?).
Backend "win32" has some capabilities to save current structure of a dialog into XML file. It was implemented for testing localization having only 1 script for English and reusing it for any other language. We didn't try the same approach for UIA yet. We even don't have multi-language test application. So any help would be appreciated.


Update check_collection.md

Overwrite with new ones_zh

Hi, @arthuo , we have change the default branch to v2.0.x. Would you mind modifying your PR by changing the branch that the PR is going into?


Rule proposal: Prefer lookahead and lookbehind

Avoid capture unneeded groups.
Fail

'I love unicorn!   I hate unicorn?'.replace(/(?<=love )unicorn(?=!)/, '🦄');

Original posted in https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1113, can we implement it here?

Interesting rule, but we have to be careful when implementing this.
The problem is that capturing groups consume parts of the string while lookarounds do not. This can lead to regexes behaving differently. Examples:
"aaaaaa".replace(/(a)a(a)/g, "$1b$2")
// 'abaaba'
"aaaaaa".replace(/(?<=a)a(?=a)/g, "b")
// 'abbbba'

"aaaaaa".replace(/(^a+)a(a)/, "$1b$2")
// 'aaaaba'
"aaaaaa".replace(/(?<=^a+)a(?=a)/, "b")
// 'abaaaa'

That being said, the rule is still a good idea. We just have to find a way to do it safely.

I also want to say that the lookbehind replacement might prevent certain optimizations. IIRC, then v8's Boyer–Moore can't be used when a regex contains lookbehinds like this. I think, I read a v8 issue related to this some time ago but don't quote me on that.
We should definitely do some benchmarks to see whether this transformation negatively impacts performance.


Add script to clear out user data.

A simple script that clears out user data from the database. Leaving the 'dummy' account.
Fixes Issue
Fixes the issue of having personally recognizable data on developers machines.
Changes proposed

[x] - Script added

@whitingjr what problem does this solve?

@johnaohara I provided details elsewhere. Not available in the open.
The issue is to do with handling PII information.

Handling PII is an operational issue, i.e. how do users handle production data, which is a concern that exists outside the project.
The intended purpose of this PR is to delete data from a production database, but the workflow of retrieving a prod backup, boot-strapping it with horreum to be able to run the script is not the correct process to handle PII. All PII  should have been cleaned before a user can even take a copy of the production DB
I do not think we should add scripts to the project that deletes all users from the database, which is production data.

-1 from me as well. If users need to call this script to remove PII from a running postgres container, probably is too late for that :wink:!!


Allow violations to downgrade themselves to a warning

It may be desirable for a policy rule to dynamically downgrade itself from a violation to a warning. One of the use cases is to allow certain schema violations, e.g. additionalProperties, to emit a warning instead of a violation.
This behavior is somewhat similar to how effective_on works today. In fact, we could achieve the same result by using a date that is always in the future.
Acceptance Criteria

Come up with an annotation that can be emitted by policy rules to trigger the behavior described in this issue, e.g. downgrade: true.
Implement functionality to downgrade such violations to warnings.

Perhaps we could introduce generic rules in addition to deny and warn rules, they will need to provide all the necessary data in their result, including for example a severity field. The balance would need to be struck on what fields must remain static, e.g. short_name
Something like:
rule contains result if {
    result := {
        code: ...,
        msg: ...,
        effective_on: ...,
        severity: ...
    }
}

That will likely require some invasive changes in the evaluator code. Maybe something we can eventually do once EC-533 is done?

Added a comment to EC-533 for us to consider introducing that idea.


Kanban Board

Link to the project: Kanban board
No of people in the project - 3
Team member 1: hassanShakur
Team member 2: Innocent259
Team member 3: Anand8317

Hello @Anand8317 @Innocent259 and @hassanShakur          👋
CHANGES REQUIRED ♻
Impressive work on the project so far👏
You've done extremely well so far in this project,  However, there are some adjustments you need to make to move to the next project. 💯

Highlights

Good use of GitHub features ✔
Nice work with the Kanban features ✔
Good work with card descriptions ✔

Required Changes  🌵

[ ] I noticed from the card that the allocated points are not divided evenly. @Anand8317 has 28 points, @Innocent259 has 19.5 and @hassanShakur has 28. You can see the difference with @Innocent259 is almost 9 points. Kindly make sure the points are <=5 between the team members 👍

_Check the comments for a detailed review
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing really improves your application.
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you. Points are even now.

Hello Team @Anand8317 @Innocent259 @hassanShakur .
I mistakenly approved your project on the Dashboard instead of asking for changes. You'll get this corrected shortly. Please look forward to your dashboard, so you can submit the changes and get approved.
My sincere apologies

@Anand8317 @Innocent259 @hassanShakur Your dashboard has been updated. You can now request another code review request. Thank you

Got it. Thanks @enniopo .


Add topic tags

I suggest adding the topics utf-8, utf8, strings, fortran in the About section, as explained at https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/classifying-your-repository-with-topics

Thank you for the suggestion, I have added the tags to the About section.


Managed Rails 5 'ActionController::Parameters' as a Hash

Also fixes failing specs on master for Rails 5, which were failing because ActionController::Parameters does not inherit from Hash anymore.

Just bumped into the same issue on my new Rails 5 project!
+1 on this PR 👍
@nicolasblanco @stantoncbradley

thanks !

That's great! Thanks @nicolasblanco, and thanks @silviusimeria.
Can I ask what are the plans for the next release of the gem on rubygems.org 😄?

Hi @nicolasblanco, any planned release on rubygems.org to give everyone compatibility with Rails 5?


Test Failed, wrong mass

Looks like we're getting different results for the mass endpoint as of last week. Is the new result more accurate, or is there a bug here? @jessfraz @hanbollar
thread 'tests::test_create_file_volume' panicked at 'assertion failed: (left == right)
Diff < left / right > :
Some(
<    53.601147,

48.800293,
)

new result is more accurate

looks like spec got updated but tests werent bumped to match

can you make a fix for the new test values? thx 🙏

FWIW youll want to run EXPECTORATE=overwrite when running make so it regenerates the generated tests expects for openapitor

Ok cool sounds good 👍

can you also check if any other endpoints where affected by this for the tests (phys specific ones) - a few other baselines also depend on the internal thing that was changed to be more accurate


include 'about' into section heading

Hi,
first of all many thanks for this very elegant theme!
I am struggling to adapt the theme so that the 'heading row' reads as 'Hugo Theme Basic / About ' when clicking on the 'about' link.  Currently, the heading remains unaltered and 'only' reads as 'Hugo Theme Basic'.  However, when I click on 'post series' the heading reads as 'Hugo Theme Basic / Series'. So I wanted some consistency here and use the heading row as something like a breadcrumbtrail.
I included 'about' into the taxomonies in the config.toml what results in the heading 'Hugo Theme Basic / About".  However, I am struggling for days now to make the text of the about.md file (in the content folder) appear under the modified heading. Any help? Many thanks.
fyi - here my clumsy attempts link.

Hi @zoowalk !  I'm happy that you are finding the theme useful!
I made a few changes to display the <site title> / About breadcrumbs show on your /about page.
First, I changed the about.md type to page in the front matter config. This will give it the the about.md page the properties of the base page archetype (https://gohugo.io/templates/single-page-templates/).
Then, I added a theme override for the way that single pages are displayed.  The theme overide allows you to make your own custom changes on top of the base theme(s) that you are using.  And, if for some reason the base theme changes and your project git pulls then your custom changes will remain since they live in your project and not the theme submodule.
Hopefully this helps!
https://github.com/zoowalk/blog/pull/1

That's more than kind! Many thanks. Just merged the pull request and it's all good!


Enables report testimony + request to rescind own testimony

Resolves #1155.
Summary
Enables the report modal and creates a specialized one for requesting to rescind your own testimony.
Changes

Enables the reporting mechanism, since we have #1144 merged
Users own testimony replaces the "Report" action with "Request to rescind"
Adds/fixes translations for most of the reporting functionality

Design
Example of "Report" and "Request to rescind"

When clicking "Request to rescind"

When clicking "Report"

Reason is required for the "Request to riscind"

On success

@mvictor55 @nesanders can we defer the work to create a separate page with details or should that be part of this change as well?
This MR doesn't include a separate page with more info from this comment. I was thinking we could create a new ticket for it as a follow up.

🚀


Error Windows 7

When you click BOOP the following error occurs:
Getting error "Could not load type 'System.Runtime.CompilerServices.IAsyncStateMachine' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'"

I solved my problem by installing Net Framework 4.8, follow the link from where I downloaded it.
https://dotnet.microsoft.com/download/dotnet-framework/net48

thx for reporte and solved


ancient flag ignored for input functions

Snakemake version: 5.29.0
Describe the bug
If the input files are retrieved with a function, snakemake will ignore the ancient flag and always execute a rule if the input is newer than the output.
Minimal example
Snakefile:
rule all:
    input:
         ancient(lambda w: "new.in")
    output: "old.out"
    shell: "touch {output[0]}"

Create output and input, make input newer than outpu (execute in shell):
touch old.out; touch new.in
Invoke snakemake with snakemake --dryrun. The rule all is scheduled for execution, although the ancient flag should prevent it.
Additional context
For now, I check the presence of the ouput files in the input function and return empty strings/lists/dicts if the output exists.

I think I had this kind of thing work by using lambda w: ancient("new.in") instead.
Well in fact in one of my workflow I have a defined input function that use the ancient flag inside the function, but this should be the same.
If that does not work, you could also try:
def get_new(w):
    return ancient("new.in")

rule all:
    input:
         get_new
    output: "old.out"
    shell: "touch {output[0]}"

Disclaimer: Only a user here, just stating what seems to work for me.


feat(#920): jeo.string -> org.eolang.string

Use org.eolang.string instead of jeo.string.
Related to #920

@yegor256 Could you review these changes, please?

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 10min).


fix(convert2hf.py): fix the rotary_emb.inv_freq KeyError

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Fix KeyError that occurred in convert2hf.py.
I also meet the problem that issue reports.
Modification
Add a default value None to the states.pop function.
BC-breaking (Optional)
Does the modification introduce changes that break the backward compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here and update the documentation.
Checklist
Before PR:

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[x] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[x] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[x] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[x] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects.
[x] CLA has been signed and all committers have signed the CLA in this PR.

Thanks for your contribution~


Blog app: add devise

this Pr includes:
Remove current_user method in ApplicationController, devise will provide us with one.
Install and setup devise.
Can register a new user.
User logs in with a combination of email and password.
Hashed password should be stored in the database.
Ask for confirmation of email.
Can reset password.
Modify the views in devise for registration and login to match the wireframes from the Sneak Peek and your styling.

I dont know why log_out not working  please suggest me the way
reset passoword mailer did not send mail please suggest me solution


Scheduling time out for .... jId: 2, in 18 s

2019-04-04 16:10:46.673 1126-5987/? D/JobServiceContext: Scheduling time out for '.core.service.NotifyJobService' jId: 2, in 18 s
2019-04-04 16:10:46.675 1126-1126/? D/JobServiceContext: handleServiceBound for fb9a063 #u0a938/2 com.icbc.im/.core.service.NotifyJobService
2019-04-04 16:10:46.675 1126-1126/? D/JobServiceContext: Scheduling time out for '.core.service.NotifyJobService' jId: 2, in 8 s

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


Timezone feature change request

Suggestion request, instead of using roles for storing/reading timezone could it perhaps be stored in a local config for plugin.

The bot has a database, yes, once again something I could make togglable.


Add debug instructions.

@HowardWolosky did you know about this?  I just found this today.
via https://discuss.atom.io/t/how-to-make-developer-tools-appear/16232/3

I did not, and that's amazing.  I was trying to figure that out all week.  I had given up on it.  This is great.  Thanks for the heads-up!


Limit on POST request size

Hi @cnoon,
I'm having the same exact problem described in Stackoverflow.
Basically I'm adding base64 equivalent of an image to the request POST parameters. The request is failing always with a 400 status code whenever the image size is bigger than 80x80 with a 0.7 compression ratio.
I'd really appreciate your help. Thanks!

Replied @marcmatta. You'll need to provide more info.
Cheers. 🍻

This is the response:
{ status code: 400, headers {
"Cache-Control" = "no-cache";
Connection = close;
"Content-Length" = 727;
"Content-Type" = "text/html; charset=utf-8";
Pragma = "no-cache";
} })
The error is nil.
@cnoon Is there any limitation to your knowledge to POST param size?

You'll have to troubleshoot with the server team. Unfortunately, there's nothing we can really do to help here @marcmatta.


Improve Environment Variable Handling for API Key

Refactor the code to handle the absence of the OPENAI_API_KEY environment variable more gracefully in the Advisor  class, maybe by adding a proper error message or by asking the user to input it manually if it's not set.

 I've assigned it to you. Thanks for taking this on! 😊

Hey I can work on other issues as well but unfortunately I don't have OPENAI API KEY to test on. Is there anything I can do? @davidalami

Hey @mananchawla2005 , in order to obtain the API key, one should login to the openai website  and create a new secret key.


Change name of Death test name.

The name of the test cae for deth test should be terminated as "DeatTest". See GoogleTest Advanced Topics.

Merged to develop. Ready to release.


[explorer] time not display correctly

Steps to Reproduce Issue
install a fullnode with docker
Expected Result

Actual Result

time is not displayed correctly, the problem seems to be resolved if I restart the node with
"docker compose restart"
you can test my fullnode with https://sui-rpc.nelrann.org in explorer
System Information

OS: Ubuntu 20.04

Sui is not supported on Ubuntu 20.04 as stated in documentation. Is the problem replicable in supported OS-es?

A lot of the code around timestamps for the explorer has changed, this should work now


Loading save doesn't load Prayer Beads bonuses

As the title states, am on an eclipse run where I printed 7 prayer beads over and over again (feeding it into the same printer) on stage 3, for about 10 minutes, and my HP was at 3k.
Just after getting into stage 9 the next day to resume the run, my HP was lowered to 1.6k, is this a Prayer Beads bug or a Mod's bug?
The save file in question:
2c003e7d-ac94-487d-bfcc-247fdd87af40.json

same issue happened to me on a 2 player engi run also i think this happens for infusion and other (if there are any other) stat boosting items that have requirements

Infusion always worked, and there aren't any other (vanilla) items like that. Now should be fixed.


fatal error: concurrent map writes in multicluster istiod tests

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/25363/integ-multicluster-k8s-tests_istio/6921#1:build-log.txt%3A3141
This was probably fixed but then reverted by https://github.com/istio/istio/commit/7915c7326a2f9659915d0f233fa5efa27d8babbd? Looks like the cobra stuff cannot be called in parallel

This makes all multicluster test fail all the time, can a fix be deployed ASAP? It's blocking PRs for 1.7 release

https://testgrid.k8s.io/istio_istio#integ-multicluster-k8s-tests_istio

this just means add a sync.Mutex?

is the issue https://github.com/istio/istio/blob/master/pkg/test/framework/scope.go#L66?
It seems like it already has a mutex. Is there a way to get better visibility into where the faliure is happening

Should be fixed by https://github.com/istio/istio/pull/25633 - seems like a PR made it in without finishing MC tests. viper. SetDefault is the culprit.

Is anyone still seeing this? MC tests are passing on new PRs.


has no attr named '_XlaCompile'

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with the
documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go
here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script
provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue
happens on a mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@reckzhou
Request you to fill issue template,
Please, share simple standalone code to reproduce the issue.It helps us in localizing the issue faster.
Please, see the comments of similar error here and see if it helps you.Thanks!

thanks ravikyram ,i solved.  docker used different cpu. one of cpu is new.


npm run build.prod.exp fails

IMPORTANT: This repository's issues are reserved for feature requests and bug reports. Do not submit support requests here.
Steps to reproduce and a minimal demo of the problem
run 'npm run build.prod.exp' with commit 7042bea3d54c673fa9c21df6bf90b1e64593032a
Generates the following exception at the gulp 'compile.ahead.prod' task:
Error: Error encountered resolving symbol values statically. Function calls are not supported. Consider replacing the function or lambda with a reference to an exported function (position 28:19 in the original .ts file), resolving symbol MultilingualModule in /Users/lukefarran/ProjectsGithub/angular2-seed-advanced/dist/tmp/app/frameworks/i18n/multilingual.module.ts
at simplifyInContext (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/static_reflector.js:473:23)
at StaticReflector.simplify (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/static_reflector.js:476:22)
at StaticReflector.annotations (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/static_reflector.js:61:36)
at _loop_1 (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/codegen.js:53:54)
at CodeGenerator.readFileMetadata (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/codegen.js:66:13)
at /Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/codegen.js:100:74
at Array.map (native)
at CodeGenerator.codegen (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/compiler-cli/src/codegen.js:100:35)
at codegen (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/tools/tasks/seed/compile.ahead.prod.js:10:86)
at Object.main (/Users/lukefarran/ProjectsGithub/angular2-seed-advanced/node_modules/@angular/tsc-wrapped/src/main.js:30:16)
Compilation failed
What steps should we try in your demo to see the problem?
  This works in the parent angular2-seed
Other information

exp == experimental ;)
If anyone can help look, would be appreciated. Will be 2 weeks before I get a chance.

@NathanWalker I'll try to help out as I've been using AoT.  It's picky and tricky, hopefully it'll be improved in the next RC or official release.

There is a pull request on ng2-translate to fix this: https://github.com/ocombe/ng2-translate/pull/229. Best to wait for it.

And the pull request was done. Now trying with ng2-translate 2.4.4

Thanks @BrainsoftLtd ! Lemme know. 👍

I still see errors with build.prod.exp.
Error: Error encountered resolving symbol values statically. Function calls are not supported. Consider replacing the function or lambda with a reference to an exported function (position 28:19 in the original .ts file), resolving symbol MultilingualModule in /klant/connectender/website/ng2-connectender/dist/tmp/app/frameworks/i18n/multilingual.module.ts
    at simplifyInContext (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:528:15)
    at StaticReflector.simplify (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:532:18)
    at StaticReflector.annotations (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/static_reflector.ts:92:28)
    at _loop_1 (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/codegen.ts:63:48)
    at CodeGenerator.readFileMetadata (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/codegen.ts:57:22)
    at /Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/codegen.ts:105:56
    at Array.map (native)
    at CodeGenerator.codegen (/Users/iminar/Dev/angular/modules/@angular/compiler-cli/src/codegen.ts:105:33)
    at codegen (/klant/connectender/website/ng2-connectender/tools/tasks/seed/compile.ahead.prod.ts:14:69)
    at Object.main (/klant/connectender/website/tools/@angular/tsc-wrapped/src/main.ts:44:12)
Compilation failed

This is fixed by exporting the function like so
export function newTranslateLoader(http: Http) {
  return new TranslateStaticLoader(http, 'assets/i18n', '.json');
}

@NgModule({
  imports: [
    CommonModule,
    RouterModule,
    FormsModule,
    HttpModule,
    TranslateModule.forRoot({
      provide: TranslateLoader,
      deps: [Http],
      useFactory: newTranslateLoader
    })
  ],
  declarations: [

However, this raises another issue, namely Angular #11307

I removed all the components/frameworks code and created just one component to see if AOT works or not.
But, after doing that 'compile.ahead.prod' task is throwing below error:
Error: Unexpected value 'AppComponent' declared by the module 'WebModule'
I tried to found the root cause of the issue and found this comment https://github.com/angular/angular/issues/7968#issuecomment-223779992. So I think our custom decorator BaseComponent will not work with AOT as per that comment.
When I replaced BaseComponent to Component as shown below, I was able to build the project successfully.
@Component({
  moduleId: module.id,
  selector: 'sd-app',
  templateUrl: 'app.component.html',
  changeDetection: ChangeDetectionStrategy.Default // Everything else uses OnPush
})
export class AppComponent {
...
}

But this will remove the purpose of using same component across platforms. I think to make it work with AOT, we need to have different components for platforms which requires different metadata (i.e. templateUrl) and share code using services. Let me know if anyone has any other simple solution for this.
Thanks.