Cannot use imported modules with joblib

Description
What steps will reproduce the problem?

Run joblib with an imported local function. This will generate an error "ModuleNotFoundError: No module named 'MyOwnModule' ending up with 'BrokenProcessPool: A task has failed to un-serialize. Please ensure that the arguments of the function are all picklable.'
For example, save:
import time

def slow_mean(data, sl):
    """Simulate a time consuming processing."""
    time.sleep(0.01)
    return data[sl].mean()

to slowmean.py and then try to use it with joblib:
# from https://joblib.readthedocs.io/en/latest/auto_examples/parallel_memmap.html
import numpy as np
import time
from joblib import Parallel, delayed
import slowmean as sm

data = np.random.random((int(1e7),))
window_size = int(5e5)
slices = [slice(start, start + window_size) for start in range(0, data.size - window_size, int(1e5))]

tic = time.time()
results = Parallel(n_jobs=2)(delayed(sm.slow_mean)(data, sl) for sl in slices)
toc = time.time()
print('\nElapsed time computing the average of couple of slices {:.2f} s'.format(toc - tic))

The error does not appear if the above is run from terminal or if the function (in this example 'slow_mean()') is defined directly instead of being imported. The problem does not occur with Spyder v3.3.6 but does occur in v4.0.0 and v4.1.4 (I haven't tested other versions in between though).
However, the above example will run without error if you first manually restart the kernel from the IPython console before doing anything else. So I therefore think this is probably a Spyder problem rather than a joblib problem.
I am running Linux and have seen this on various Python versions from 3.6 to 3.8.5 too.
Traceback
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'
ERROR:traitlets:Exception in comm_msg for c64971a0e78c11ea814f83083ff53143
Traceback (most recent call last):
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'
ERROR:traitlets:Exception in comm_msg for c64971a0e78c11ea814f83083ff53143
Traceback (most recent call last):
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 177, in comm_msg
    comm.handle_msg(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/qtconsole/comms.py", line 265, in handle_msg
    return self._msg_callback(msg)
  File "/home/dave/envCAML_py385/lib/python3.8/site-packages/spyder_kernels/comms/commbase.py", line 330, in _comm_message
    self._message_handlers[spyder_msg_type](
TypeError: _handle_remote_call_reply() missing 1 required positional argument: 'load_exception'

Versions

Spyder version: 4.0.0
Python version: 3.8.5
Qt version: 5.12.9
PyQt5 version: 5.12.3
Operating System: Linux 5.4.0-42-generic

Dependencies
cloudpickle >=0.5.0          :  1.5.0 (OK)
pygments >=2.0               :  2.6.1 (OK)
qtconsole >=4.6.0            :  4.7.6 (OK)
nbconvert >=4.0              :  5.6.1 (OK)
sphinx >=0.6.6               :  3.2.1 (OK)
pylint >=0.25                :  2.6.0 (OK)
psutil >=0.3                 :  5.7.2 (OK)
qtawesome >=0.5.7            :  0.7.2 (OK)
qtpy >=1.5.0                 :  1.9.0 (OK)
pickleshare >=0.4            :  0.7.5 (OK)
zmq >=17                     :  19.0.2 (OK)
chardet >=2.0.0              :  3.0.4 (OK)
numpydoc >=0.6.0             :  1.1.0 (OK)
spyder_kernels >=1.8.1;<2.0.0:  1.9.3 (OK)
qdarkstyle >=2.7             :  2.8.1 (OK)
atomicwrites >=1.2.0         :  1.4.0 (OK)
diff_match_patch >=20181111  :  20200713 (OK)
intervaltree                 :  None (OK)
watchdog                     :  None (OK)
keyring                      :  None (OK)
pexpect >=4.4.0              :  4.8.0 (OK)
pympler                      :  None (OK)
sympy >=0.7.3                :  1.6.2 (OK)
cython >=0.21                :  0.29.21 (OK)
IPython >=4.0                :  7.17.0 (OK)
matplotlib >=2.0.0           :  3.3.1 (OK)
pandas >=0.13.1              :  1.1.1 (OK)
numpy >=1.7                  :  1.18.5 (OK)
scipy >=0.17.0               :  1.4.1 (OK)
pyls >=0.31.2;<0.32.0        :  0.31.10 (OK)
xdg >=0.26                   :  0.26 (OK)

As I said, I still get this error in v4.1.4, which is the latest release. It also occurs in 5.0.0a1. I have installed these using pip; I am not using Anaconda.

@quokka79, thanks for reporting and the reproducible example as well. I managed to run your main script by going to the menu Run > Configuration per file and selecting the option called Run in console's namespace instead of an empty one:

That was added in Spyder 4 to prevent picking up already defined variables in the console when running your files.
Please try that and let us know if it works for you.

@ccordoba12 thanks for looking into this!
I have just tried the 'Configuration per file' option using the console's namespace (in v4.1.4) but it still gives the error as before.
If I restart 'Console 1/A' kernel or if I choose 'Execute in a dedicated console' then it runs without any problems.

Yes, it's very strange. Things don't work for the first console created by Spyder, but they do for any other console or after restarting the kernel.
@impact27, any ideas about this?

I have the same issue/observation. Is it possible that the current working directory (initially set to the user's home folder) or some search path is not properly updated to the script/project directory when the script is first run on the first console?

This issue is not easily resolved. I think multiprocessing doesn't conserve the current working directory.
It looks like the cwd of the process is actually the initial cwd where python was launched.
You can recreate the issure by:

launch spyder, the console cwd is home
test joblib -> doesn't work, but change the CWD to the current folder
restart the console. The console CWD is now the folder you expect
test joblib -> works
change CWD
restart the console. The console CWD is now the another folder
test joblib -> doesn't work

To fix it, if you are willing to modify standard libraries, in multiprocessing/spawn.py, remove:
    try:
        i = sys_path.index('')
    except ValueError:
        pass
    else:
        sys_path[i] = process.ORIGINAL_DIR

To fix it, add this to your code:
import multiprocessing.process
multiprocessing.process.ORIGINAL_DIR = os.path.abspath(os.getcwd())

Thanks for your help @impact27! Any chance to make this fix work automatically in spyder-kernels?

https://github.com/spyder-ide/spyder-kernels/pull/244

Thanks @impact27 for your help!

This was fixed by #13818.


Issue 163 fix for arete report typo

Find/replace fix for #163
The test case here is to generate a test using the Loadtest option, and ensure that running the resulting test doesn't break on
         expect(report.averageResponseTimeInternal).            to.be.lessThan('TIME DATA HERE');

I have no interest in signing a CLA at this time.  Closing this one for now.


paginator not work

It's same https://github.com/swooletw/laravel-swoole/issues/35

This repository is archived. Welcome to use the preferable replacement, sluemn.

Thanks for your reply


Low Success Rate of Lookup Action in HotpotQA Reproduction with ReactReflectAgent

Hi,
I am currently reproducing the results of HotpotQA. However, I have a question regarding the behavior of the Lookup action on specific keywords.
Most of the time, the Lookup action returns "No Result." I have attached two examples to illustrate this behavior: the first shows a "No Result" for a short sentence, and the second for a single word.

"No Result" for a short sentence

"No Result" for a single word

Experiments
Experiment 1

react_llm: GPT-3.5-Turbo-0301
reflect_llm: GPT-3.5-Turbo-0301
Data: 100 questions from hotpot-qa-distractor-sample.joblib
Total trials: 5
Reflexion strategy: REFLEXION
Agent Type: ReactReflectAgent

During this experiment, there were 9 Lookup actions performed in total, but only 2 returned successful results: one for "Abheri" and another for "cryptocurrency."
Experiment 2

react_llm: GPT-3.5-Turbo-0301
reflect_llm: Llama-3.1-8B-instruct
Data: 100 questions from hotpot-qa-distractor-sample.joblib
Total trials: 5
Reflexion strategy: REFLEXION
Agent Type: ReactReflectAgent

In this experiment, no successful Lookup cases were observed.

Hello,
The "Lookup" tool is imported from LangChain (see hotpotqa_runs/environment.py):
from langchain import Wikipedia
from langchain.agents.react.base import DocstoreExplorer
explorer = DocstoreExplorer(Wikipedia())
explorer.search(...)
explorer.lookup(...)

From my interpretation, explorer.lookup looks for instances of the query string in the last document searched by exporer.search.
In the examples you provide, the "Lookup" tool seems to be functioning correctly:

https://en.wikipedia.org/wiki/Greatest_Hits_Radio_South_Yorkshire has no instance of the string "broadcasting company based in".
https://en.wikipedia.org/wiki/Baba_Yaga_(film) has no instance of the string "sisters".

Further, see #43.


fix(helm): fix bug when helm update can't find release 1.

If a release's v1 config map was removed, 'helm upgrade' would fail. It
was calling the wrong function to get its history. This has been fixed.
Closes #1829

If you want to manually test, you need to:

Create a release
2 Upgrade the release
Use kubectl to delete the config map for the release's v1
Upgrade the release

hey @technosophos this worked manually but I couldn't actually reproduce the problem on master. I'll lgtm though.


Bump @volar/vue-language-server from 1.0.16 to 1.0.24

I'm not sure if those are specific to this version because those trigger randomly but I'm seeing some state de-sync issues with this version.

The "desync" issues were probably caused by issues fixed in https://github.com/sublimelsp/LSP/pull/2182.


pageLength is off by one for /v1/tags with forTag=

If you include forTag in a POST /v1/tags call, along with a pageLength, the endpoint is returning pageLength + 1 results.  Easy to fix, just wanted to have a record of this bug.
This is a bug in my current tags-support PR.  Depending on where @laurelnaiad is in merging, I'll add to the branch or put in the next chunk of commits.

I don't think I see this bug even prior to fix....

:)   this is only in the forTag case ... it wasn't just pageLength that needed fixing, but start as well.
without forTag, we're retrieving just what the MarkLogic REST service provides, and start is a server-side parameter.
with forTag, we want start to be the nth tag matching the pattern, and pageLength to be the number of tags returned.  So we have to use start=1 for the ML REST call, and then compute the start and results in the middle tier.  As we've discussed, moving this to a transform would be an interesting performance exercise.

Tried filtering via tags and pagelength are seen correctly . @laurelnaiad  can you please what more need to be tested here ?

This particular bug was one that I never noticed. Charles explained that it was a "sometimes" bug and that's about all I know. See followup issues from Mike re: what is and isn't ready for testing w/respect to tags searches.
Re: this one, I would leave it to @grechaw to help you understand how this was reproducible.

BTW, why is this back in verify? Because you're not sure how to test it?

Hard to test without looking at middle tier services. I could demonstrate but @gghai hasnt done much testing of those specifically

Not sure if this is actionable for me

I propose we call it a technical bug found/fixed by dev.

Good morning!

I'll ship this bug then as something covered by unit tests and opaque to QA.


Upgrade required conan version

A recent upstream change to the googleapis conan recipe broke the conan build for some folks in some situation with an error message saying that protobuf was not found.
I analysed the problem and it turns out that the update of googleapis triggers a known and fixed bug in Conan and an upgrade of Conan fixes the problem.

Thanks Henning! :)


[BUG] -  GE MRS and nifti anatomical LR flip - coreg_ge_nifti - Simmonite & DeMayo

Describe the bug
Voxel orientation for GE MRS in combination with nifti anatomical images have a LR flip.
To Reproduce
Steps to reproduce the behavior:
Use any GE MRS sequence in combination with nifit anatomical images.
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Bug reports can be found here (https://forum.mrshub.org/t/voxel-flipped-l-r-when-using-nifti-t1-for-ge-data/255/14)

Closed in #470


warningCountIncreaseSpecifiedScoreAndHigher incorrect evaluation

It appears that the warningCountIncreaseSpecifiedScoreAndHigher is intended to compare the warnings from the current build to the warnings in the previous build, but the code doesn't actually look at the previous build.  Instead it appears to calculate the percentage of the warnings in the current analysis that have a score exceeding the threshold.
Current help documentation says:

Set the build result to result if the number of warnings with rank minrank or higher issued by the CodeSonar analysis exceeds the number issued for the previous analysis by more than limit%.

The code computes the warning "increase" percentage as follows:
calculatedWarningPercentage = ((float) severeWarnings / totalNumberOfWarnings) * 100;

Where severeWarnings is the number of warnings from the current analysis that exceed the input rankOfWarnings threshold (which is actually a score).
The fact that the code compares warning scores and not ranks deserves a different issue report.

The fact that the code compares warning scores and not ranks deserves a different issue report.

See Issue #18

Addressed in PR #16


Get actual image format instead of extension

Allow to load any image file by guessing its type from file metadata instead of extension, using the builtin utility imghdr.what('filename').  Using this, the specific loader is always selected, not depending on the file extension, which can be a problem when loading from servers where extension type is not controlled.

This would only work reliably if imghdr would be extended to recognize all file types supported by the different image providers. The list of formats recognized by default also differs depending on the python version.
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_sdl2.py#L22
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_tex.py#L17
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_pygame.py#L29
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_pil.py#L45
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_ffpyplayer.py#L47
https://github.com/ismailof/kivy/blob/f0f45e29c7e6ac1ceb54fe2391169d5aad509e54/kivy/core/image/img_dds.py#L16

I made a simple change to default back to extension if the image format cannot be guessed by imghdr. This way all of the formats can still be supported, but the most common ones (those on imghdr) does not rely on the file extension.
Do you think that this would be enough by the moment? imghdr can still be extended for extra cases if necessary.
My personal motivation for this PDR is that I have to retrieve images (usually jpg or png) from a server where the extension does not match the format, so I was getting Unknown <obj> type, no loader found. exceptions.

I have not tested, but I believe this should work now. 👍

@ismailof, we could also do the same here for the bytestream: https://github.com/ismailof/kivy/blob/77c6cdac069a4ecd772a7040ea72e40a18525e91/kivy/core/image/init.py#L332.

Ok, I'll take a look on it. Adding the imghrd part doesn't seem complicated.
In addition, the ImageLoader code looks a little bit messy to me, as it plays with atlases, zips and loaders in a different fashion but kind of alltogether, so if you don't mind, I would like to reorder it on a different PR.


Adding respective icons to the navbar

The current navbar lacks icons, which impacts its visual appeal and usability. Users might find it less engaging and intuitive to navigate through the website without visual cues provided by icons.
I propose adding respective icons to the navbar to enhance its visual appeal and improve user navigation experience. These icons will align with the theme of the website and provide users with clear visual cues for each navigation option

please assign me this issue.

please assign me this issue


issue #266 fix

Preview is added while taking a picture in Record activity .

Don't work on my side: no preview displayed in Chrome 72 and do an error on console in Firefox 65.
What's your testing environment?

@llaske added support for Firefox 65 and Chrome 72 .

It's now work on Chrome but not on Firefox: the photo is displayed but it never recorded. There is an error on console:
  Recorder state changed: recording recordrtc.js:705:17
  Initialized recorderType: MediaStreamRecorder for output-type: video recordrtc.js:99:13
  Stopped recording video stream. recordrtc.js:125:13
  Recorder state changed: stopped recordrtc.js:705:17
video/webm -> 217 Bytes recordrtc.js:166:17
  La ressource média data:video/webm;base64,GkXfowEAAAAAAAAfQoaBAUL3gQFC8oEEQvOBCEKChHdlYm1Ch4ECQoWBAhhTgGcB/////////xFNm3QBAAAAAAAAABVJqWYBAAAAAAAASSrXsYMPQkBEiYgAAAAAAAAAAE2AmFFUbXV4aW5nQXBwTGliV2ViTS0wLjAuMVdBmVFUd3JpdGluZ0FwcExpYldlYk0tMC4wLjEWVK5rAQAAAAAAADWuAQAAAAAAACzXgQFzxYQ+kgeYJYaIg1ZQOIOBAYaFVl9WUDjgAQAAAAAAAAiwggKAuoIB4A== n’a pu être décodée. index.html
  La ressource multimédia data:video/webm;base64,GkXfowEAAAAAAAAfQoaBAUL3gQFC8oEEQvOBCEKChHdlYm1Ch4ECQoWBAhhTgGcB/////////xFNm3QBAAAAAAAAABVJqWYBAAAAAAAASSrXsYMPQkBEiYgAAAAAAAAAAE2AmFFUbXV4aW5nQXBwTGliV2ViTS0wLjAuMVdBmVFUd3JpdGluZ0FwcExpYldlYk0tMC4wLjEWVK5rAQAAAAAAADWuAQAAAAAAACzXgQFzxYQ+kgeYJYaIg1ZQOIOBAYaFVl9WUDjgAQAAAAAAAAiwggKAuoIB4A== n’a pas pu être décodée, erreur : Error Code: NS_ERROR_DOM_MEDIA_METADATA_ERR (0x806e0006)

Plus, could you remove package.json and package-lock.json from your PR.

@llaske now it shall work on all platforms.

Unfortunately I've got the same issue on Firefox.

Works perfectly on my Firefox without any error on console . can you please try again 2-3 times as the other you were getting will be solved by just restarting your browser . ref -  https://stackoverflow.com/questions/44780856/firefox-html5-audio-media-resource-could-not-be-decoded-onmediasinkaudioerror

Hmmm. Seems to works now. Strange.
BTW there is another issue: the security popup is shown two times (like in your video). Could you fix that?

@llaske we are getting pop up twice because we are making call to the camera twice(one for the preview and other for capture) and since we are testing the project on localhost , browser does not remember that we have granted the access and asks again . However on HTTPS this issue would not occur and browser will remember our choice .

@llaske everything works perfectly now .

Nice. It works now.
Please remove commented code.

done

Good work. Thanks.


Picture location is sometimes lost despite being present in EXIF

EXIF metadata sometimes gets stripped from pictures, making the uploaded result less useful.
The custom picker had this issue until Siva Subramaniam kindly fixed it.
There are still features in the app that strip pictures of their EXIF location. The goal of this issue is to:

Imagine all (or most) scenarios
Fix the scenarios where EXIF location gets stripped.

For (1), non-exhaustive list of hints:

Uploading from the "Nearby" tab loses EXIF location
Sharing from Google Photos. Both pictures taken by the phone, and pictures from elsewhere shown by Google Photos. In Google Photos swiping up a photo seems to have some kind of effect on metatata, so better check with and without. Adding/modifying location within Google Photos may also have an effect.
Sharing from third-party gallery such as Samsung's or https://play.google.com/store/apps/details?id=com.simplemobiletools.gallery
Uploading from the custom picker, normal picker, built-in camera, using an old version of Android and a recent version of Android (can use emulator).
Other ideas very welcome.

For (2), this might help: https://github.com/commons-app/apps-android-commons/pull/5190. It might not be enough though.

Hey @nicolas-raoul
Can you please clear me more as I am beginner to GSOC as well as open source contributions, I have been programming in java and have keen interest in  android development, If I can be any helpful, I am ready , please help me in understanding the issue . Thanks.

@sagarojha772 There is not much time left, but if interested please read the instructions at https://github.com/commons-app/commons-app-documentation/blob/master/android/Students.md thanks and good luck! :-)

Hey @nicolas-raoul
After exploring lot , I found that we can use EXIF libraries such as Metadata-Extractor and save it in a separate variable.
When displaying the picture, use this saved location data instead of relying on the EXIF data in the image file.
1-
We can use this concept to all other features google sharing photos:
Retrieve the EXIF data using Metadata-Extractor and save it in a separate variable or database field before sharing.
Check the effects of swiping up on metadata and modify the saved data accordingly.
When displaying the picture, use this saved location data instead of relying on the EXIF data in the image file.
2-
Third party galleries
3-
Uploading from custom picker, normal picker, or built-in camera:

Sharing from third-party gallery such as Samsung's or https://play.google.com/store/apps/details?id=com.simplemobiletools.gallery
Uploading from the custom picker, normal picker, built-in camera, using an old version of Android and a recent version of Android (can use emulator).

I happen to be observing an issue w.r.t both of the above cases. See https://github.com/commons-app/apps-android-commons/pull/5190/files#r1167466682
Notes:

I'm using a OnePlus Nord running Android 12.
I used Simpletools Gallery to share to the app

Lets us leave this issue open for now. We could close it once we've done one last exhaustive check and ensure all known location loss issues are gone.
If we find any new onesduring the check , we could open new issues for specific cases and address them.


ActionsSdkApp Response Validation Errors

Hi, I got this error message ("expected_inputs[0].possible_intents[0]: intent 'actions.intent.OPTION' is only supported for version 2 and above.") when I used askWithList() in ActionsSdkApp. What should I do?

Hi @poerwiyanto. To help us figure out the issue, could you tell us if you created this project from scratch using the Actions console or if you used this we a pre-existing project?

Hi, @smishra2. I created this project from scratch using the Actions console. I used the Node.js client library (ActionsSdkApp) and some functions, like ask() and buildInputPrompt(), work well, but whenever I used askWithCarousel() and askWithList(), I got that error message. I tried to run the sample code in https://developers.google.com/actions/reference/nodejs/ActionsSdkApp#askWithList and still got the same error in simulator. Thanks.

Hi, @smishra2. I have updated to version 1.1.1, but the problem remains the same. Is there anything that I missed (like header)? I have also tried response-builder.js to create carousel or list, but nothing worked. Thanks in advance.

Hi @poerwiyanto, sorry for the confusion. Could you try creating a new Actions project, with an identical Action package. But use add "fulfillmentApiVersion": 2 to your conversation object, so your Action Package is something like this:
{
	"actions": [
      {
        "description": "Default Welcome Intent",
        "name": "MAIN",
        "fulfillment": {
          "conversationName": "welcome"
        },
        "intent": {
          "name": "actions.intent.MAIN"
        }
      }
    ],
    "conversations": {
      "welcome": {
        "name": "welcome",
        "url": "https://example.com",
        "fulfillmentApiVersion": 2
      }
    }

}

Actually you should be able to update your existing Action Package and Actions project, and then use gactions test to push it

Thank you very much. It works!


Check link function

Please check the link function for the help page: https://unikube.io/help/
Affected links are:

Get started
https://unikube.io/docs/guides/getting-started/

Developing with Unikube
https://unikube.io/docs/guides/getting-started/developing-with-unikube.html

Installing the Unikube CLI
https://unikube.io/docs/guides/getting-started/installation.html

@marty-2015 Fixed, the links on the help page work as expected now.

Yes, it works now. Thanks @tobifroe


Update Carbon Ad Code

@NicolasCARPi looks like I need to update the gh-pages branch because it's the mirror of site for GitHub pages. Do you mind reviewing this PR again? Thanks!

np


bin/fix-vf-meta.py: NameError: global name 'os' is not defined

I tried to use this script on two VF.ttf running it like
python gftools-fix-vf-meta.py WorkSans-VF.ttf WorkSans-Italic-VF.ttf
Traceback (most recent call last):
  File "/gftools-fix-vf-meta.py", line 330, in <module>
    main()
  File "gftools-fix-vf-meta.py", line 318, in main
    ', '.join(map(os.path.basename, ttfonts))
NameError: global name 'os' is not defined

I've just pushed a commit to #83 which will solve this.
Could you send me the vf please?
The script thinks the families don't share the same name.

Original files:
WS-VF.zip
Hey Marc, great to hear, here are the files. I realised actually as part my shell script I had been processing them both individually already with the script, now that i did it again without processing using the script I just had, these are the files:
Archive2.zip
and the new error:
    from fontTools.ttLib.tables import otTablesos
ImportError: cannot import name otTablesos

Thanks
That error isn't possible sir ;-) https://github.com/googlefonts/gftools/pull/83/files. I take it you've done some edits by hand?

Today I'll update the script so it will work.

Yep, I've update my repo right now too


integrate install doesn't work

Visual Studio 2019 (16.9.4)
git clone https://github.com/microsoft/vcpkg.git
cd vcpkg
bootstrap-vcpkg.bat
vcpkg.exe integrate install

Accept the changes to the system.
The console states:
All MSBuild C++ projects can now #include any installed libraries.
Linking will be handled automatically.
Installing new libraries will make them instantly available.
A  vcpkg.system.props file is created in C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\v140\ImportBefore\Default
vcpkg.exe install openssl:x64-windows-static

The library builds as expected.
Create a new empty Windows Console C++ project in Visual Studio.
Add a hi.cpp file
#include <iostream>
#include <openssl/ssl.h>

int main()
{
	std::cout << "hi" << std::endl;
	return 0;
}

Switch solution platform to x64.
Try building.
Result:
hiosl.cpp(2,10): fatal error C1083: Cannot open include file: 'openssl/ssl.h': No such file or directory
Most probably related to issue #10534 and #13503, possibly a regression.
Also related to https://stackoverflow.com/questions/59865669/how-to-use-vcpkg-installed-packages-in-visual-studio-in-reality
Also the vcpkg screen in Configuration Properties does not appear in old projects (might need a separate bug report)

Can you please also run ./vcpkg integrate project?

@JackBoosY
./vcpkg integrate project
Created nupkg: C:\.........\vcpkg\scripts\buildsystems\vcpkg.C.thunderpants.libs.vcpkg.1.0.0.nupkg

With a project open, go to Tools->NuGet Package Manager->Package Manager Console and paste:
    Install-Package vcpkg.C.thunderpants.libs.vcpkg -Source "C:\.........\vcpkg\scripts\buildsystems"

Followed the instructions
...
Successfully installed 'vcpkg.C.thunderpants.libs.vcpkg 1.0.0' to vcpkgNewProjTest

Project builds fine afterwards.

@JackBoosY
another workaround I used before is adding a vcpkg.props to the projects property sheets containing:
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <ImportGroup Label="PropertySheets" />
  <PropertyGroup Label="UserMacros" />
  <PropertyGroup />
  <PropertyGroup Label="Globals">
    <VcpkgTriplet Condition="'$(VcpkgOSTarget)|$(Platform)'=='linux|x64'">x64-linux</VcpkgTriplet>
    <VcpkgTriplet Condition="'$(VcpkgOSTarget)|$(Platform)'=='windows|x64'">x64-windows-static</VcpkgTriplet>
  </PropertyGroup>
  <ItemDefinitionGroup />
  <ItemGroup />
</Project>

still it would be lovely if it could just work as documented

Also, we can set it here:


Can't open videos

Hello, when I'm trying to render a scene, it renders the video, but I can't open it. I'm not getting any errors (except when I try to preview a video), and the file is saved. But, I can't open it, I think it's empty.
I don't think it comes from ffmpeg. I'm running everything with python 2.7, on OS X El Capitan.
This might help:
python2.7 extract_scene.py -w example_scenes.py
1: CountingScene
2: PiCreatureScene
3: SquareToCircle
4: TeacherStudentsScene
5: WarpSquare
6: WriteStuff

Choose number corresponding to desired scene/arguments.
(Use comma separated list for multiple entries)

Choice(s): 3
Writing to ~/DEV/others/manim/files/movies/example_scenes/SquareToCircle.mp4
  0%|                                                              | 0/60 [00:00<?, ?it/s]

It's working for me (### is my hidden path to manim's folder):
python2.7 extract_scene.py -w example_scenes.py
1: CountingScene
2: PiCreatureScene
3: SquareToCircle
4: TeacherStudentsScene
5: WarpSquare
6: WriteStuff

Choose number corresponding to desired scene/arguments.
(Use comma separated list for multiple entries)

Choice(s): 3
Writing to ###/manim/files/movies/example_scenes/SquareToCircle.mp4
Animation 1: ShowCreationSquare: 100%|████████████████████████████████████████████████████████████████████████████████████████| 60/60 [00:11<00:00,  5.34it/s]
Animation 2: TransformSquareToCircle:  75%|██████████████████████████████████████████████████████████████▎                    | 45/60 [00:08<00:02,  5.24it/s]^C

Traceback (most recent call last):
  File "extract_scene.py", line 187, in main
    handle_scene(SceneClass(**scene_kwargs), **config)
  File "###/manim/scene/scene.py", line 48, in __init__
    self.construct(*self.construct_args)
  File "###/manim/./example_scenes.py", line 35, in construct
    self.play(Transform(square, circle))
  File "###/manim/scene/scene.py", line 323, in play
    self.update_frame(moving_mobjects, static_image)
  File "###/manim/scene/scene.py", line 113, in update_frame
    self.capture_mobjects_in_camera(mobjects, **kwargs)
  File "###/manim/scene/scene.py", line 102, in capture_mobjects_in_camera
    self.camera.capture_mobjects(mobjects, **kwargs)
  File "###/manim/camera.py", line 90, in capture_mobjects
    self.display_multiple_vectorized_mobjects(vmobjects)
  File "###/manim/camera.py", line 100, in display_multiple_vectorized_mobjects
    canvas.flush()
  File "/usr/local/lib/python2.7/dist-packages/PIL/Image.py", line 741, in frombytes
    s = d.decode(data)
KeyboardInterrupt

Using a Linux Mint 18.1 system.
PS: the 00:00<?, ?it/s shown in your terminal's copy affected me somewhere else (I don't remember where), and my ChoicesInNecklaceCutting.mp4 file seems to be even empty (262 on the size column when doing ls -l on it).

Closing, as this issue is old and inactive.


Cognitive Services updates

Update LUIS Authoring auto-gen'ed SDK files using latest specs.
Update manager classes for CustomVision Prediction and Training.

Can one of the admins verify this patch?


ExampleLength Exception for Hashes

In quite a few of my tests (especially when verifying a response in a controller test), the majority of the spec is simply well-formatted hashes.  ExampleLength in this case is of dubious value.  It would technically be possible to just put the hash all on one line and fix this issue, but that makes it an unreadable mess.
I'd love to see an option to exclude hashes from the lines counted in specs.
As an added bonus, removing the lines which only contain brackets/braces/parens would also be great.
Example Spec
  it 'does stuff' do
    thing = create(:thing)

    put :update,
        params: {
          id:   thing.id,
          data: {
            attributes: {
              address:     '444 Franklin Road',
              city:        'Scottsvegas',
              description: 'Updated Description',
              ended_at:    Time.utc(2017, 1, 2, 0, 0, 0),
              name:        'Updated Thing Name',
              permissions: %w{can_do},
              started_at:  Time.utc(2017, 1, 1, 0, 0, 0),
              state:       'AA',
              tags:        %w{party},
              zip:         '12345',
            },
          },
        }

    expect(json_response['data']).to eql(
      'id'         => thing.id,
      'type'       => 'things',
      'attributes' => {
        'user-id'     => current_user.id,
        'address'     => '444 Franklin Road',
        'city'        => 'Scottsvegas',
        'description' => 'Updated Description',
        'ended-at'    => '2017-01-02T00:00:00Z',
        'latitude'    => 90.0,
        'longitude'   => 180.0,
        'name'        => 'Updated Thing Name',
        'permissions' => %w{can_show_by_direct_link},
        'started-at'  => '2017-01-01T00:00:00Z',
        'state'       => 'AA',
        'tags'        => %w{party},
        'zip'         => '12345',
      },
    )
  end

Related: #249
As a workaround, you can move the definitions of the hashes to lets and the examples will be short

#249 would be an acceptable workaround to this.  Thanks @Darhazer!


feat(linter): add suggestion for no-console

Part of #4179

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

#4312  👈
#4223
#4222
main

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

Join @DonIsaac and the rest of your teammates on  Graphite

Merge activity

Nov 21, 2:38 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


User Profile Pages to show indavidual user activity

User Dashboard
Found at /user/{id}. (Move current /users/ to /settings/users/)

User activity
Created content (Recent 5 with link to all at /user/{id}/content)
Avatar with name and user stats

Joined time (User for x years)
Pages, Books & Chapters created
Pages, Books & Chapters updated (Count page revisions)

Maybe have a nice graph to spice things up and show user activity over time like GitHub's contribution chart?

Added as of 4d9726dbdd4a3eef054a78d378a942eec5a1a388


bug: ion-slide-box doesn't load the relative slide inside a view

Type: bug
Platform: all
Hi,
I'm not expert of Ionic...
I have a problem with the ion-slide-box (maybe it is easy).
I have a ion-tabs and in the first tab (that is hidden) I want to show a ion-slide-box.
I added the relative code for the slide inside a template. The slider is generated but each single slide disappear...
If I move the code of the ion-slide-box inside the tabs, it works...
Do you know why?

Greetings @lucapantano!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Codepen/Plunkr example?

Here it works:
http://lucapantano.com/altro/ionic/example-that-works/index.html
The "welcome tab code" is in the "http://lucapantano.com/altro/ionic/example-that-works/templates/tabs-welcome.html"
Here the example that doesn't work:
http://lucapantano.com/altro/ionic/example-not-working/
The "welcome tab code" is in a separate view named "http://lucapantano.com/altro/ionic/example-that-works/templates/welcome.html"

Please try to isolate the issue in a codepen/plunkr/jsbin.
I do not need your whole project.

Greetings @lucapantano!
My sensors indicate a reply was requested, but as we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.


feat: FromToJson for recursive inductives

Also adds a VSCode setting to trim trailing whitespace.

These derive handlers sure can get complicated :sweat_smile:

Yeah, it's on my backlog to factor out some of the patterns appearing in almost all of them like creating a match (with enough flexibility) to Deriving.Util or even Meta.Basic.


Integrate the other opennars-* respositories into the bot on riot to post notifications

Integrate the other opennars-* respositories into the bot on riot to post notifications

@TonyLo1 my bad


feat: arkconnect delegate voting

Summary
https://app.clickup.com/t/86dt0w98r
Todo

[x] test resigned delegate unvote

Checklist

[ ] I checked my UI changes against the design and there are no notable differences
[ ] I checked my UI changes for any responsiveness issues
[ ] I checked my UI changes in light AND dark mode
[ ] I checked my (code) changes for obvious issues, debug statements and commented code
[ ] I opened a corresponding card on Clickup for any remaining TODOs in my code
[ ] I added a short description on how to test this PR (if necessary)
[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

@ItsANameToo I made quite a few changes to this PR which is based off of https://github.com/ArdentHQ/arkscan/pull/848, such as some tweaks/fixes. It's potentially worth just merging this PR and closing the other but I'll leave it for you to decide


Don't choke when controller is not present in the request

Many thanks @gobijan for reporting and troubleshooting 🙏. Added as coauthor.
Close https://github.com/hotwired/spark/pull/54
Fix https://github.com/hotwired/spark/issues/53

Thx for jumping in so quickly Jorge. I've been to a Christmas dinner and couldn't respond so till now. Your check if interceptable method in the middleware makes a lot of sense and keeps the code simpler. It's a good practice to check at the boundaries 👍
Thx for releasing a new version too.


integrations: Add webhook code, API endpoint, documentation and tests for Delighted.

integrations: Add webhook code, API endpoint, documentation and tests for Delighted.

Automated message from Dropbox CLA bot
@540KJ, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott Opinion please?

Should be fine now, except fixture+test?

Yeah, looks good modulo that issue and one question (posted above)

Added fixture + test for non-promoter.

Merged, thanks @540KJ!!


Can't add register or add new customer.

When register
After I press a registration button, I can't progress or there is no other error message provided.
When to add via admin
The system said there is an error but I can't know what the error is or how to fix them.

Seems to be working fine for me. Check your error logs

Ok, this is an error. In the error log.
ERROR - 2016-03-08 11:40:51 --> Query error: Field 'address_id' doesn't have a default value - Invalid query: INSERT INTO `ti_customers` (`first_name`, `last_name`, `email`, `salt`, `password`, `telephone`, `security_question_id`, `security_answer`, `newsletter`, `customer_group_id`, `status`, `date_added`) VALUES ('test1', 'test2'<EMAIL_ADDRESS>'51a407c92', '6db832eb143ea16dab045f0394df0d8eaeaa1a03', '123456', '11', 'cat', '0', '11', '1', '2016-03-08')
And also, I tried to add other data with no luck. A similar problem is found. Some fields don't have a default value. So I looked up in the database. those fields have a "not null" setting. Normally, MySQL won't allow inserting "not null" without data. I'm not sure if there are any option to bypass such behavior.
Btw, this some environment info.
Server version: 5.6.22-log - MySQL Community Server (GPL)
Protocol version: 10
Apache/2.2.25 (Win32) PHP/5.4.33
Database client version: libmysql - mysqlnd 5.0.10 - 20111026 - $Id: c85105d7c6f7d70d609bb4c000257868a40840ab $

Sorry the response is late. You should post on the community forum next time.
You can fix this error by Turning off Strict Mode in your SQL with this command
SET @@global.sql_mode= '';

Alright, next time. I'll ask in the forum.
However, that would change the whole system. I can't do it because I have other databases too. Is it possible to adjust the structure of the table? So the field that should be null is set properly. What do you think?

You can't change the mode per database - just global or session. You can use session instead but i wouldn't advice this as it hacking into the core files and will loose changes when you upgrade TI.
Open system/tastyigniter/core/Base_Controller.php and after line 54, add
            $this->db->query("SET SESSION sql_mode='';");

But.. it's the reason I opened this issue. To update the core files or SQL scripts. If you don't have a self-host, you'll have to do the hacking as you said. Anyway, thanks for the answer, I'll have to use that for now.


[1.15.2] Crash on world load

Issue description:
Game crashes when loading a world intermittently.
Restarting the game and trying again will eventually let me in.
I'm guessing it's due to Optifine because it seems to be fine without it.
Version (make sure you are on the latest version before reporting):

Minecraft: 1.15.2
Forge: 31.1.77
Refined Storage: 1.8.3
Optifine: HD_U_G1_pre16

Does this issue occur on a server? [untested]
If a (crash)log is relevant for this issue, link it here:
https://gist.github.com/DuelMonster/91ca0f4be221762c7ac3c8e3ece8eb16

I would wager this is caused by optifine. There haven't been any reports of this before.

Please report this to Optifine


fix(front): 打开不同业务的任务记录报错问题

Codecov Report

:exclamation: No coverage uploaded for pull request base (v1.0.15-rc@c600c96). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             v1.0.15-rc     #147   +/-   ##
=============================================
  Coverage              ?   53.62%
=============================================
  Files                 ?      182
  Lines                 ?     8853
  Branches              ?        0
=============================================
  Hits                  ?     4747
  Misses                ?     4106
  Partials              ?        0

Continue to review full report at Codecov.

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


Browse all

i developed ui and added links to Browse all category button
Description

i added icons for each categoy
i have fetched the list of categories from "All Category" button besides search bar
Motivation and Context

it was not working

https://github.com/MAVRICK-1/e-commerce_website/issues/151
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@Tanishasi great work


Android Presentation API and iOS AirPlay

It would be really great if Sky could support the use case of having two independent display/view instances that can communicate together. This would make Sky very appealing to game developers who want to make multi-screen games (phone/tablet as controller and large TV/monitor as second screen).
Both Android and iOS have support for this use case today using the standard frameworks.
There are two issues that would need to be addressed to make this work:

A simple way to connect the two Sky views/apps so that they can quickly pass messages (mostly it will be one way messages from the controlling view to the dumb screen view). A critical feature would be to have a way for the controlling view to know when the other view is available to receive messages (can't start the game until all views are initialized).

Exposing the necessary API in Dart to query for external displays and to be able to create a view to send to the external display. For Android this would be exposing the Presentation API and for iOS there is a Multiple Display Programming Guide for iOS.

For inspiration, here is a cordova Presentation API plugin that implements all of the necessary plumbing for both Android and iOS:
https://github.com/fraunhoferfokus/cordova-plugin-presentation

FWIW, I designed the Android presentation API.  Let me know if you have any
questions.
On Mon, Jul 20, 2015 at 7:19 AM Lex Berezhny<EMAIL_ADDRESS>wrote:

It would be really great if Sky could support the use case of having two
independent display/view instances that can communicate together. This
would make Sky very appealing to game developers who want to make
multi-screen games (phone/tablet as controller and large TV/monitor as
second screen).
Both Android and iOS have support for this use case today using the
standard frameworks.
There are two issues that would need to be addressed to make this work:

A simple way to connect the two Sky views/apps so that they can
quickly pass messages (mostly it will be one way messages from the
controlling view to the dumb screen view). A critical feature would be to
have a way for the controlling view to know when the other view is
available to receive messages (can't start the game until all views are
initialized).
2.
Exposing the necessary API in Dart to query for external displays and
to be able to create a view to send to the external display. For Android
this would be exposing the Presentation API
http://developer.android.com/reference/android/app/Presentation.html
and for iOS there is a Multiple Display Programming Guide for iOS
https://developer.apple.com/library/ios/documentation/WindowsViews/Conceptual/WindowAndScreenGuide/UsingExternalDisplay/UsingExternalDisplay.html
.
For inspiration, here is a cordova Presentation API plugin that implements
all of the necessary plumbing for both Android and iOS:
https://github.com/fraunhoferfokus/cordova-plugin-presentation
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130.

@vlidholt could be something for you to look at as part of gaming?

Fwiw, I am putting some thought into how the Sky Shell should evolve to
support Android features such as this.  It will be nontrivial given the way
things are plumbed right now.
On Wed, Jul 22, 2015, 5:57 PM Ian Hickson<EMAIL_ADDRESS>wrote:

@vlidholt https://github.com/vlidholt could be something for you to
look at as part of gaming?
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130#issuecomment-123919466.

My initial ticket implies that the two views are running in different isolates but I don't know if this is necessarily the case. I noticed some code recently where Android Activities (I think that's what they were) were getting instantiated and started from within Sky app. Seems like it shouldn't be much of a stretch to implement the Presentation API patterns then. But maybe I'm missing something, Sky has accelerated in development so much lately that it's been difficult keeping track... that's a good thing! :+1:

The Sky code may run in isolates but the Android code shares the state of
one process.  Currently native viewport is designed to attach to the
current Activity.  This will need to be enhanced to allow viewport backed
by Dialogs.
I've started sketching out some thoughts around how the Android embedding
of Sky / Mojo may be improved going forward (partly for performance as well
as creating a clearer path for adding enhanced capabilities).  So if you
can sit tight until I get back from Seattle we can chat about it next
week.  :)
Now if you're thinking about the Cast story in general (not just
Presentations but also MediaRouter, MediaSession, etc.) then we have a long
way to go!  The lifecycle considerations get quite interesting (because we
need to go beyond Activities...)
On Wed, Jul 22, 2015, 6:33 PM Lex Berezhny<EMAIL_ADDRESS>wrote:

My initial ticket implies that the two views are running in different
isolates but I don't know if this is necessarily the case. I noticed some
code recently where Android Activities (I think that's what they were) were
getting instantiated and started from within Sky app. Seems like it
shouldn't be much of a stretch to implement the Presentation API patterns
then. But maybe I'm missing something, Sky has accelerated in development
so much lately that it's been difficult keeping track... that's a good
thing! [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130#issuecomment-123926318.

I've got an idea for an app that I want to build, a few Android devices, a Chromecast and I know Dart pretty well so I'm very interested in any workable solution that would let me use the Chromecast as a second screen from a Sky app.
Several months ago I attempted to get this all working on my own by using network sockets to communicate between two Dart isolates that I was starting from the Java/Dalvik side. But it was quite a bit of a hack and just wasn't really workable. I also hard coded the C++ Sky engine to start two Sky Views... like i said it was a monster :-)
I think it's best if I wait for someone who knows what they are doing to implement this part. I'll be happy to test and experiment once there is something working though.
My app idea is for an eye exercise program that helps with focusing and convergence insufficiency issues. Currently behavioral eye therapists use the "Hart Chart" for this but it's a little too low tech and gets insanely boring after the first couple of times and practically impossible to get a child to do it. I have eye issues and my daughter who is 9 also suffers from this, a digital version of the Hart Chart would be a life saver (literally). I have a prototype that I implemented using WebViews but communicating between WebViews is more than a bit slow on Android (because they run on different threads and you have to synchronize the messages between them, it's a mess)...
Anyways, I'm really looking forward to this feature being supported in Sky!

Cool.  Well I can certainly help with this stuff (given I designed most of
the Android Framework parts).  :)
Now that said, you might not even need to rely on Presentation for this use
case.  You'll need to activate Chromecast mirroring for that to work anyhow
at which point perhaps you could just draw the content you want on the
device as usual and let it be mirrored over.
(Presentation is mostly interesting for showing different content on
different displays.)
Another option would be to develop a native Chromecast app and connect to
it from the phone.  Note that the APIs involved in doing that are somewhat
different (still involves MediaRouter but not Presentation; instead you
talk to the Cast API in Google Play Services).
So lots of options depending on the desired user experience.
On Wed, Jul 22, 2015, 7:07 PM Lex Berezhny<EMAIL_ADDRESS>wrote:

I've got an idea for an app that I want to build, a few Android devices, a
Chromecast and I know Dart pretty well so I'm very interested in any
workable solution that would let me use the Chromecast as a second screen
from a Sky app.
Several months ago I attempted to get this all working on my own by using
network sockets to communicate between two Dart isolates that I was
starting from the Java/Dalvik side. But it was quite a bit of a hack and
just wasn't really workable. I also hard coded the C++ Sky engine to start
two Sky Views... like i said it was a monster :-)
I think it's best if I wait for someone who knows what they are doing to
implement this part. I'll be happy to test and experiment once there is
something working though.
My app idea is for an eye exercise program that helps with focusing and
convergence insufficiency issues. Currently behavioral eye therapists use
the "Hart Chart" for this but it's a little too low tech and gets insanely
boring after the first couple of times and practically impossible to get a
child to do it. I have eye issues and my daughter who is 9 also suffers
from this, a digital version of the Hart Chart would be a life saver
(literally). I have a prototype that I implemented using WebViews but
communicating between WebViews is more than a bit slow on Android (because
they run on different threads and you have to synchronize the messages
between them, it's a mess)...
Anyways, I'm really looking forward to this feature being supported in Sky!
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130#issuecomment-123930689.

Could you create a "Hello World" Sky app where it would list available displays, then after picking a display it would show something on that display?

Ideally the user shouldn't have to choose the display itself but rather the
intended device to project onto.  That's handled by MediaRouter.
Now if you want to see a native Android app that does enumerate displays
and shows content on them look at the Presentation samples in
development/samples/ApiDemos in the Android tree.
Jeff.
On Wed, Jul 22, 2015, 7:33 PM Lex Berezhny<EMAIL_ADDRESS>wrote:

Could you create a "Hello World" Sky app where it would list available
displays, then after picking a display it would show something on that
display?
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130#issuecomment-123937284.

I have a working Java/Android implementation but how do I do this purely within a Sky app?
MediaRouter is not exposed in Sky, nor is it clear how one would create the Presentation/Dialog to render to and associate with the remote display, all within Sky. That's the part that's still missing and would need to be implemented.
Basically the Sky/Dart version of the Cordova plugin I link to in my original ticket description.

Yup.  Like I said, there is a long way to go.  :)
We don't have the infrastructure for this yet and I am tempted to clean up
some of what we do have first before tacking on these features.
On Wed, Jul 22, 2015, 7:42 PM Lex Berezhny<EMAIL_ADDRESS>wrote:

I have a working Java/Android implementation but how do I do this purely
within a Sky app?
MediaRouter is not exposed in Sky, nor is it clear how one would create
the Presentation/Dialog to render to and associate with the remote display,
all within Sky. That's the part that's still missing and would need to be
implemented.
Basically the Sky/Dart version of the Cordova plugin I link to in my
original ticket description.
—
Reply to this email directly or view it on GitHub
https://github.com/domokit/sky_engine/issues/130#issuecomment-123940154.

Okay, great! I'm really looking forward to this :-)

@j9brown Is this something you are still working on/thinking about? Sky/Flutter has changed a lot over last couple of months. Do you think adding some way to use Presentation API in Flutter is still a long way off?


Installation fails under El Capitan

I can't install battleschool under Mac OS X 10.11 because of System Integrity Protection (SIP) see: https://support.apple.com/en-us/HT204899. You are no longer allowed to install into /usr.
running install_data
    creating /usr/share/battleschool
    error: could not create '/usr/share/battleschool': Operation not permitted

Could you please change the install directory to /usr/local.
Thanks.

closed via ae22fb613d5465973e1a0437bfc484f24cb90e01


Modify Transaction Category API

implemented the modify transaction category api and tested it , everything works as intended.


fix(ext): restore container hidden state

Types

[x] 🐛 Bug Fixes

Background or solution
Changelog
Fix extension register container hidden state not restore

/publish

只是一个暂时的修复，这里是可以重构以获得更优雅的实现方式的。

Codecov Report

Merging #610 (dc7ce41) into main (183284a) will increase coverage by 0.00%.
The diff coverage is 55.00%.

@@           Coverage Diff           @@
##             main     #610   +/-   ##
=======================================
  Coverage   58.98%   58.98%
=======================================
  Files        1208     1208
  Lines       74636    74640    +4
  Branches    15463    15464    +1
=======================================
+ Hits        44021    44027    +6
  Misses      27911    27911
+ Partials     2704     2702    -2

Impacted Files
Coverage Δ

...on/src/browser/sumi-browser/contribution/editor.ts
28.57% <ø> (ø)

...rc/browser/sumi-browser/contribution/editorSide.ts
63.63% <ø> (ø)

...on/src/browser/sumi-browser/contribution/tabbar.ts
63.26% <ø> (ø)

...n/src/browser/sumi-browser/contribution/toolbar.ts
68.00% <ø> (ø)

...es/file-tree-next/src/browser/file-tree.service.ts
61.90% <ø> (+0.45%)
:arrow_up:

...es/extension/src/browser/extension-node.service.ts
22.68% <12.50%> (-1.24%)
:arrow_down:

...s/main-layout/src/browser/tabbar/tabbar.service.ts
63.01% <33.33%> (+0.17%)
:arrow_up:

packages/addons/src/common/index.ts
100.00% <100.00%> (ø)

...xt/src/browser/services/file-tree-model.service.ts
54.43% <100.00%> (+0.43%)
:arrow_up:

...ages/file-service/src/node/file-service-watcher.ts
71.18% <0.00%> (-1.70%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


Add view background color matcher

Hi Guys, I've added a view background color matcher for check the android view background based on color resource or color code. If there is something wrong just let me know and I am happy to discuss.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: anton46:x: anton.tuhadiansyahanton.tuhadiansyah seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@anton46 thanks for the contribution :)
Two things

There is one of your users that needs to sign the CLA
Can you squash the commits ?

@Alviere Thank you for the prompt answer. ThehasBackgroundColor() should be pass a function or the color information(res/string)?

@cdsap Yep, I will update the commit.

@anton46 it should be overloaded function. hasBackgroundColor(@ColorRes resId: Int) and hasBackgroundColor(color: String)


training dataset

Hello, could you share x8 training dataset? default DIV2k has no x8 images

It is available here
HR:
http://data.vision.ee.ethz.ch/cvl/DIV2K/DIV2K_train_HR.zip
Val x8:
https://competitions.codalab.org/my/datasets/download/4ccbd8c6-8af2-4310-ae9a-5f25b2a6d65
2LR x8: https://competitions.codalab.org/my/datasets/download/70ccff08-fda4-4e09-ad9f-cb28b170ece9

@alterzero Hi, the link address of  HR and  Val x8. Could you share them again?

the link address of HR and Val x8.    Could you share them again?

the link address of HR and Val x8. Could you share them again?

If you, like me, came here and tried to download the datasets, found that the valid part link is broken. The good news is that was only missing one char, and I started trying one by one. Thankfully, it did not take me too long to find the right one. Here is the correct link: https://competitions.codalab.org/my/datasets/download/4ccbd8c6-8af2-4310-ae9a-5f25b2a6d652


redirect is annotated to return a werkzeug type

My app doesn't need werkzeug directly except for type annotations of views that contain a redirect call, even when I explicitly provide Response=flask.Response.
https://github.com/pallets/flask/blob/5cdfeae2e82541577e17f5179d91775550e34ebd/src/flask/helpers.py#L233-L235
I think this can be achieved in a similar way to classmethods:
_BaseResponse = t.TypeVar("_BaseResponse", bound="BaseResponse")

def redirect(
    location: str, code: int = 302, Response: t.Optional[t.Type[_BaseResponse]] = None
) -> _BaseResponse:

Duplicate of https://github.com/pallets/flask/issues/4976


feat(STONEINTG-931): remove sbom-json-check

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
As part of STONEINTG-931 we marked sbom-json-check as deprecated in build-defnitions, removing it from templates, leaving just information to customer about how to remove that task from their pipeline.
We no longer need to test for this task since it will be removed.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest

/retest

/retest

/retest


Fix invalid foreign key constraint name with long schema qualified tables

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?
This PR generates syntactically valid names for foreign key constraints when the table name is greater then 64 characters and the table name is schema-qualified, e.g. my_schema.a_very_very_very_very_very_very_very_very_long_table_name.
Fixes https://github.com/go-gorm/gorm/issues/5029.
Previously the name formatter would leave . characters in the long schema-qualified table name intact. However, this generates an invalid foreign key constraint in Postgres and migrations fail with an error like this:
2022/01/26 17:14:17 Failed to auto migrate, but got error ERROR: syntax error at or near "." (SQLSTATE 42601)

The new behavior is for the . in the table name to be replaced with _.
The . to _ translation was already occurring when the table name was less than 65 characters due to this code.  However, when the table name length hit the 65 character limit the table name was used without any invalid character replacement. One question I have is whether all names that pass through NamingStrategy.formatName should have this fix applied. Right now I've only fixed FK names to minimize any unexpected behavior changes.
User Case Description
This change is necessary for me because I'm working in a database with many schemas and foreign key constraints that span schemas. When a table name is long in this setup it currently causes migrations to fail due to invalid foreign key constraint naming.

Hi @mnussbaum
I think we should fix the issue in https://github.com/go-gorm/gorm/blob/5edc78116fe46a7d001db52d80a78f97756ac1ad/schema/naming.go#L98
Change it to formattedName = formattedName[0:56] + hex.EncodeToString(bs)[:8]?

@jinzhu I agree what you said this suggestion.
------------------ Original ------------------
From: Jinzhu @.>
Date: Sun,Feb 20,2022 8:30 AM
To: go-gorm/gorm @.>
Cc: heige @.>, Comment @.>
Subject: Re: [go-gorm/gorm] Fix invalid foreign key constraint name with longschema qualified tables (PR #5045)
Hi @mnussbaum
I think we should fix the issue in https://github.com/go-gorm/gorm/blob/5edc78116fe46a7d001db52d80a78f97756ac1ad/schema/naming.go#L98
Change it to formattedName = formattedName[0:56] + hex.EncodeToString(bs)[:8]?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

@jinzhu I've pushed an update using your suggestion, looks like that should work to me

Thanks!


Android Basics: Introduction to Room and Flow

"FullScheduleFragmentDirections" showing error as Unresolved reference
and " .navigate(action)" as Overload resolution ambiguity
But the project builds successfully.

@grvknight
Please provide the required details (codelab URL, step, visual proof of the error message... )while reporting an issue so that we can help you with troubleshooting.

The issue is closed as no further clarification of it was provided.


centos-7 bootstrap not working on latest CentOS release

This is the (sanitised) bootstrap I get
#!/bin/bash -x
# Bootstrap for CentOS 7 and maybe other EL-derived platforms.
#
# Note: Amusingly doesn't actually work on RHEL itself, since ruby-devel
#       does not seem to exist on it :-/ If you actually care about RHEL
#       itself, I'll happily accept a pull request that does whatever is
#       needed to fix ruby-devel on RHEL.
#
(
exec 1> >(logger -s -t user-data) 2>&1

rpm -ivh https://yum.puppetlabs.com/puppetlabs-release-el-7.noarch.rpm

yum update --assumeyes
yum install --assumeyes puppet ruby-devel rubygems gcc zlib-devel libxml2-devel patch gnupg2

gem install pupistry --no-ri --no-rdoc
mkdir -p /etc/pupistry
mkdir -p /etc/puppetlabs/code/environments
cat > /etc/pupistry/settings.yaml << "EOF"
general:
  app_cache: ~/.pupistry/cache
  s3_bucket: pupistry-puppet-designerwardrobe
  s3_prefix:
  gpg_disable: true
  gpg_signing_key: XXXXXX
agent:
  puppetcode: /etc/puppetlabs/code/environments
  access_key_id: XXXXXXXXXX
  secret_access_key: XXXXXXXXXXX
  region: ap-southeast-2
  proxy_uri:
  daemon_frequency: 60
  daemon_minimal: true
  environment: master
EOF
chmod 700 /etc/pupistry/settings.yaml
chmod 700 /etc/puppetlabs/code/environments
pupistry apply --verbose
)

Error is as follows:
user-data: ERROR:  Error installing pupistry:
user-data: 	nokogiri requires Ruby version >= 2.1.0.

Seems that ruby in the repos is too old (ruby.x86_64 0:<IP_ADDRESS>8-29.el7). I end up having to install a new version with RVM. Also it seems puppet doesn't get installed.
yum -y install gcc-c++ patch readline readline-devel zlib zlib-devel
yum -y install libyaml-devel libffi-devel openssl-devel make
yum -y install bzip2 autoconf automake libtool bison iconv-devel sqlite-devel
curl -sSL https://rvm.io/mpapis.asc | gpg --import -
curl -L get.rvm.io | bash -s stable
source /etc/profile.d/rvm.sh
rvm reload
rvm install 2.4.0
rvm use 2.4.0 --default
gem install puppet

Hmm so the problem here will be that we pull the latest Nokogiri when we install Pupistry and it has had it's minimum requirements raised beyond the version of Ruby that CentOS ships. Welcome to the wonderful world of Ruby!

Puppet should have installed regardless however. What errors did you get around yum install puppet ?

Sorry about the delay, I haven't built a new server until now.
This is what happens with RPM puppet and ruby2.4.0 from RVM with a pupistry apply --verbose at the end of the bootstrap:
user-data: D, [2017-04-02T10:27:48.836705 #16806] DEBUG -- : Looking for configuration file in common locations user-data: D, [2017-04-02T10:27:48.838816 #16806] DEBUG -- : Loading configuration file /etc/pupistry/settings.yaml user-data: /usr/local/rvm/gems/ruby-2.4.0/gems/json-1.8.6/lib/json/ext/generator.so: [BUG] Segmentation fault user-data: ruby 2.0.0p648 (2015-12-16) [x86_64-linux] user-data:
After a gem install puppet the error goes away


[DO NOT MERGE]: PoC Fail Log Collect

PoC to demonstrate loss of logs in e2e tests that trigger pod restarts

/test ocp-4-16-nongroovy-e2e-tests

/test ocp-4-16-nongroovy-e2e-tests

/test gke-nongroovy-e2e-tests

Closing - PR was a proof of error for future consideration.


Simple rate limiter

I'm probably simply not getting something, but I use an API that accepts at most 10 requests per minute, so I need to throttle my client. From Bucket4j's documentation I get that this should be simple.
https://bucket4j.com/8.1.1/toc.html#using-bucket-as-throttler
Bandwidth bandwidth = Bandwidth.simple(10, Duration.ofMinutes(1));
Bucket bucket = Bucket.builder().addLimit(bandwidth).build();

However, if I run an endless loop with:
bucket.asBlocking().consumeUninterruptibly(1);

The first 10 tokens being distributed immediately, then for the 11th token there is a short 2 second pause, but he 11th token  is handed out only 6 seconds after the 1st was. This alone would make for 11 requests within the first minute and would immediately break the throttle restriction. It should wait for at least a minute before handing out the 11th token. (The 12th and further tokens are also handed out too soon.)
I can hack around this by using intervallyAligned, but this behavior seems so totally wrong.
How do I setup a bucket that hands out available tokens immediately, but then correctly waits the full minute for each token before refilling?

Hello @tbee

This alone would make for 11 requests within the first minute and would immediately break the throttle restriction. It should wait for at least a minute before handing out the 11th token.

Definitelly no. Bucket4j is implementation of token-bucket algorithm. Token-bucket by itself is not addressed to solve restrictions like N permits/ M time_window.  Its math model is more complicated, I higly recommend to read something about it, especially abourt burst.

I can hack around this by using intervallyAligned, but this behavior seems so totally wrong.

Both intervally and intervallyAligned refills are not helpfull to prevent the burst, because the burst is inherent part of token bucket.

Then the documentation which states "100 times per 1 minute" is confusing. To me at least, because that is exactly the behavior I am looking for. And in my example I get more that 100 times in the first minute.
Anyhow, I've implemented a RateLimiter backed by a DelayQueue, and that works as I need it to. So thanks for responding!

nyhow, I've implemented a RateLimiter backed by a DelayQueue, and that works as I need it to. So thanks for responding!

The sorted hash map where timestamp is the key, is the place where  typical programmer starts usually. This honey-moon can prolong as long as capacity does not become too huge(like terabytes per second) as well as amount of buckets does not increase like millions buckets per application. The typical way in which programmers give up to satisfy hard limit N/M is described there https://www.blogger.com/blog/post/edit/3564973286727861628/5966188154863706900
Most discussions have stopped 30 years ago, when Token-Bucket was accepted as preferred algorithm for network devices, as a algorithm that can operated on fixed(and small) amount of memory.

The advantage of using DelayQueue is that the whole waiting is handled for you. One has to insert Delayed implementations with the correct timestamp when they may execute.
And what can I say? The service I'm addressing limits the request in the way I described; max 10 per 1 minute, in its most simple and unsmoothed form.


Also Run tests on Centos

If you're new, read the guidelines.
Good code is tested, designed for change, easy to understand
and able to cope when things go wrong.
Good pull requests are descriptive, small and short-lived.
For the Author

[ ] Description explains what you have done and why.
[ ] Tests have been written (or you have explained why you don't need
tests).
[ ] Documentation/comments have been added/updated.

For the Reviewer

[ ] The code is well-designed and able to cope with change.
[ ] The code is easy to understand.
[ ] Tests cover happy and unhappy paths in the code.
[ ] Tests are green on CI (or pulled/tested on your machine).

I don't see any comments on your Pull Request.
Are you too good for code reviews now?
No. No you aren't.
Maybe you forgot to say who reviewed it. That's fine, but let me know.


Cannot find main.* for any supported scripting backend

I think this script needs updating for the newer version of mpv ?

file-browser.lua cannot be inside a folder in the ~~/scripts directory, if must be in the top level.

Where is this documented?

https://mpv.io/manual/master/#script-location

Ok, I didn't want to pollute the top-level folder so I just renamed the script main.lua and kept it in its own directory and that seems to work.


Allow genqlient types to be marshaled safely

Summary:
When genqlient generates output types, it generates whatever code is
necessary to unmarshal them.  Conversely, when it generates input types,
it generates whatever code is necessary to marshal.  This is all that's
needed for genqlient itself: it never needs to marshal output types or
unmarshal input types.
But maybe you do!  (For example, to put the responses in a cache, which
is the use case that @csilvers hit at Khan, although there are others
one can imagine.)  While we can't support every serialization format you
might want (at least not without adding plugins or some such), it's not
unreasonable to expect that since genqlient can read JSON, it can write
it too.  Sadly, in the past this was not true for types requiring custom
unmarshaling logic, for several reasons.
In this commit I implement logic to always write both marshalers and
unmarshalers whenever they're needed to be able to correctly round-trip
the types, even though genqlient doesn't do so.  I wasn't starting from
scratch, since of course we already write both marshalers and
unmarshalers in some cases.  But this ended up requiring surprisingly
large changes on the marshaling side, mostly to correctly support
embedding (which we use for named fragments).
Specifically, as the comments in types.go discuss, the most difficult
issue is spreads with duplicate fields, which translate to Go embedded
fields which end up hidden from the json-marshaler.  Ultimately, I had
to do things quite differently from unmarshaling, and essentially
flatten the type when we write marshaler.  But in the end it's not so
ugly -- indeed arguably it's cleaner!  Mainly it's just different.
One thing to note is that we do marshal __typename based on
what we know about the types; users need not fill it in (and if they
do we'll ignore it).  This seemed to me to be a better UX, and
didn't add much complexity.
In general, I begin to wonder whether using encoding/json at all is
really right for genqlient: we're doing a lot of work to appease it,
despite knowing what our types look like.  I think it would still be a
significant increase in lines of code to roll our own, but that code
would perhaps be simpler, and would surely be faster (although if we
just want the speed gains we could use another JSON-generator library,
see also #47).  Anyway, something to think about in the future.
Test plan:
make tesc

I'm probably not the right person to review this, but it all seems plausible to me!  It is indeed tricky stuff, but it's commented really well.
I do feel like the code would be easier to understand in some ways, actually, if you codegen-ed the json marshal/unmarshal code; you're kinda doing that already for the hard cases.  But I agree it would make the size of these generated files explode -- even for the common (?) use-cases where people don't have these fancy embeds and interfaces -- which is a consideration in itself.

I do feel like the code would be easier to understand in some ways, actually, if you codegen-ed the json marshal/unmarshal code; you're kinda doing that already for the hard cases.

Yeah, that's my theory.  I think there's a lot of stuff to write to handle things like omitempty as well as the basics of encoding strings, bools, numbers, etc., all of which we're avoiding now, so I think even just for the genqlient code it would be a big increase in the quantity of code.  But I think it would all be much simpler.  I may prototype it if I'm bored sometime, and see how it looks.

But I agree it would make the size of these generated files explode -- even for the common (?) use-cases where people don't have these fancy embeds and interfaces -- which is a consideration in itself.

Yeah, I don't know how much of an issue that will be for folks.  It's probably a nonzero perf/binary-size impact, too, in that I don't think the Go compiler/linker can be very smart about pruning methods since you might (and indeed encoding/json does) call them via reflection.  This PR already adds quite a bit; I think if it becomes an issue for anyone we could add an option to go back to the old behavior of generating marshalers only for input types and unmarshalers only for output types.  I think generating both is the better "batteries-included" default; people who care a lot about binary size should expect to need to tune some things.  But with generating the json-encoding code, we wouldn't want to have to support both ways.

I did wonder if we could offload some of your heavy lifting to a library... but other than possibly the one I slacked about, I didn't casually find one that was an obvious candidate.

Yeah, thanks for raising that option.  I was thinking about how I feel I am learning more and more places where the stdlib API leaves a lot to be desired, but I didn't think to actually look for what others have done.  Sadly, as I wrote on #120, as far as I can tell none of the ones we've found will help a lot with the marshaling side, where embedding is the real challenge, rather than interfaces and custom marshalers.  (I mean, both require nontrivial code, but the latter is just copied from unmarshaling, so if you went to the pains to wrap your head around what we do there, it's not so bad.)  My feeling is that on the whole it doesn't make sense (at least not just for the code-complexity gains; if we used go-codec the perf benefits might be worth it).  But I think in any case since the biggest benefits are on the unmarshaling side, I'd want to do it as a separate PR.


npm warning; bower install

when I run a npm install I get the following warning:
npm WARN cannot run in wd<EMAIL_ADDRESS>bower install (wd=/mnt/c/users/.../Repos/xyz)

Thanks for reporting the issue, can you please provide the exact npm install command that produced the warning?  On a recent internal build I was able to install npm and bower without error:
root@STEHUF-TEST:/mnt/c/Users/Test# npm install bower npm http GET https://registry.npmjs.org/bower npm http 200 https://registry.npmjs.org/bower npm http GET https://registry.npmjs.org/bower/-/bower-1.7.9.tgz npm http 200 https://registry.npmjs.org/bower/-/bower-1.7.9.tgz<EMAIL_ADDRESS>node_modules/bower root@STEHUF-TEST:/mnt/c/Users/Test#

What build are you running?  We are tracking some npm errors in #3.  The status of npm on build 14342 is:

npm install bower should run with no errors on both / and /mnt/x directories
sudo -g npm install foo should run with no errors
There is still a bug on downloading larger files tracked by #313

From build 14342:
russ@DESKTOP-K1JHN54:/mnt/c/tmp$ npm install bower
/mnt/c/tmp
└──<EMAIL_ADDRESS>
npm WARN enoent ENOENT: no such file or directory, open '/mnt/c/tmp/package.json'
npm WARN tmp No description
npm WARN tmp No repository field.
npm WARN tmp No README data
npm WARN tmp No license field.

Please let me know if this is not your experience or if you're trying to do something else.

I initially received this error when running npm install for the angular-seed app. When I run the npm install bower command directly I receive the following error:
root@CHARLES-PC:/mnt/d/users/charles/Repos/MyAngularSample# npm install bower
npm ERR! Linux 3.4.0+
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "bower"
npm ERR! node v4.4.3
npm ERR! npm  v2.15.1

npm ERR! shasum check failed for /tmp/npm-481-faf74f57/registry.npmjs.org/bower/-/bower-1.7.9.tgz
npm ERR! Expected: b7296c2393e0d75edaa6ca39648132dd255812b0
npm ERR! Actual:   1b0a51719006147ffe3263e090e9c3101571e816
npm ERR! From:     https://registry.npmjs.org/bower/-/bower-1.7.9.tgz
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /mnt/d/users/charles/Repos/MyAngularSample/npm-debug.log

this is the same error from #364

Closing this out as this seems to be fixed in a more recent insider build.  Please reopen with your exact sequence of commands you're using if you continue to see this issue.


added macOS support for model name

This PR adds support for macOS in the new model function

You can use sysctl -n and then get rid of the 2nd line iirc.

Thanks :+1:


Hardcoded absolute paths in code

Absolute path /home/igor/ is hard-coded in some places (files listed below) in the code base. I think it might be better if this was configurable through some mechanism like env vars etc. (Not sure if it's already possible. Did not thoroughly look into the code)

globals.py
get_answer.py
panela/panela_colors.py
adapter_learnxiny.py

You are 100% right. I will fix it. Thank you very much for pointing me at it.

I made a pull request to hopefully help resolve this

So, can this be closed?

I close this one


CA.pl thorough structure change

Structure change for ease of subroutine reuse and comprehensibility. Plus some bug fix; perl emerges error when
(1) some tunable shell variable is missing,
(2) called whith no arguments.

Cleaning my git. I will send the same pull-request later.


PDFtk cannot be installed using Homebrew as mentioned in the Readme

PDFtk cask has been removed from Homebrew as per this gist. The reason mentioned in the gist is quoted below.
This workaround install is necessary because PDFtk was pulled from homebrew-cask due to issues with it aggressively overwriting file permissions that could impact other installed libraries. See this homebrew-cask issue.
Please update the readme if you find this helpful.
Thank you

brew install https://raw.githubusercontent.com/turforlag/homebrew-cervezas/master/pdftk.rb
Or we can install using this command.

Some time has passed since that. And I finally have to use a Mac at work so I can try this out


PHP 8.1 Support

Q
A

Bug fix?
no

New feature?
yes

Doc update?
no

BC breaks?
no

Deprecations?
no

Fixed tickets

License
MIT

This PR updates SwiftMailer to be compatible with PHP 8.1. Additionally, I've migrated the Travis build to GitHub Actions. More tests are being run now thanks to this.
The reason why I sent this in is because we'll need this to offer PHP 8.1 support on the mail component in Laravel 8. For the upcoming Laravel 9 release in January 2022 we're switching to Symfony Mailer. I've also manually tested this PR through using the Laravel 8 mail component.
I've tried to go over all the pieces of code that I changed in comments below. Thought that was a bit more useful to review.
Would love to get feedback on this soon as PHP 8.1 is now fast approaching! 🙂

@driesvints Thank you so much for wokring on this. Actions are enabled, so I'm not sure why that does not work. Trying to figure it out.

@fabpot is there an "enable" button here? https://github.com/swiftmailer/swiftmailer/actions


Document consistency flag in mounts

Deploy preview for docsdocker ready!
Built with commit 6dd9fa00e99949505f7e29236de6b6c5b0b39f2e
https://deploy-preview-7836--docsdocker.netlify.com


Add table to model cards

Solves #87
Description
This feature adds a new method to the Card class called add_table. It
adds a generic table to the end of the model card by creating a new
section (similar to plots).
Any generic table of type dict[str, list[Any]] is supported, as well as
DataFrames. However, the main use case is for adding CV results, as e.g.
created by GridSearchCV.
Implementation
I hope you can stomach the type-y implementation Adrin ;)
I wanted to make the extra sections more generic in PR #89 as a
preparation for this PR (thus avoiding a big PR with refactor + new
feature). However, the changes in #89 were not sufficient (I guess
that's the danger of splitting the work).
The problem was that the formatting of adding the plots in the save
method was very specific to plots, as a markdown link would be added.
This, of course, does not work for tables.
My solution is that _extra_sections now takes ExtraSection objects as
values instead of strings. These objects only need to implement the
format method (so technically, str would still be okay). So for a
plot, the PlotSection class has a format method that returns a markdown
link. And for tables, the TableSection class has a format method that
uses tabulate to return a table as string.
This approach should hopefully be flexible enough for us to later add
new section types if needed.
At first glance, the implementation might look overly complex. However,
I think it's better than having a bunch of if...else inside of save that
formats each possible type differently. But please LMK if you have a
better idea, I'm open to suggestions.
Tangential changes
So far, the plots were just added without an extra section to the bottom.
Now, there is an "# Additioinal Content" section and each plot and table
gets their own subsection.
Examples
Here is the hyperparameter table generated by plot_model_card.py:

Instead of an extra type, I'd just store the tables, and have a little function which takes the table and return what should be rendered in the md file. We don't need a protocol for adding tables to the model card.

Instead of an extra type, I'd just store the tables, and have a little function which takes the table and return what should be rendered in the md file. We don't need a protocol for adding tables to the model card.

I assume you mean inside the save method. How would you know at that point what function to use to render the value? Sure, we could say str -> plot and dict -> table but I don't like that too much and it would probably not work with more types being added. IMO it also doesn't add much complexity -- you can directly see how different types of section are formatted and don't need to figure out which function is being called on them.
As to the protocol, it is not strictly needed, it's just to add a bit of extra type safety.

I'll work on that and let you know once it's ready for review.

@adrinjalali This is now ready for review.
In the changelog, I have not yet added subsection for new features, bugfixes, etc. LMK if now is a good time.


Clean up legacy branches

In GitLab by @cmastalli on Jan 31, 2019, 11:31
Could we delete the following branches: (topic/refact, topic/nmsd, topic/debugkkt, topic/cpp)?

In GitLab by @nmansard on Feb 2, 2019, 09:25
Anything that as no commit inside: you can kill it.

In GitLab by @nmansard on Feb 2, 2019, 09:25
closed


Failed to load resource: the server responded with a status of 403

I am trying to deploy a basic application to amplify console. When i work with localhost everything is perfect but when i try to build it with console i dont see any errors in console but when i click the domain i dont see anything just blank page i these errors in chrome console.

Here is some additional information. If you need something else i can attach.

Here is my scripts in package.json file

"scripts": {
"start": "react-scripts start",
"build": "react-scripts build",
"test": "react-scripts test",
"eject": "react-scripts eject",
"compile-sass": "node-sass src/assets/scss/now-ui-dashboard.scss src/assets/css/now-ui-dashboard.css",
"minify-sass": "node-sass src/assets/scss/now-ui-dashboard.scss src/assets/css/now-ui-dashboard.min.css --output-style compressed",
"map-sass": "node-sass src/assets/scss/now-ui-dashboard.scss src/assets/css/now-ui-dashboard.css --source-map true",
"install:clean": "rm -rf node_modules/ && rm -rf package-lock.json && npm install && npm start"
},

Thanks for everyone.

hi Can you explain for about this issue

Hi @cnbrkkaya   Can you explain for about this issue


Joint consensus Issue

if originally , we have
{s1,s2,s3}
then the joining server config like {s1,s2,s3,s4,s5} , this config will send to leader , leader will have two collections,one of which keeps the old config ,and one of which keeps  the new config , the new config will be  {s1,s2,s3,s4,s5} ,then it mark the local state  as Cold_new. after that leader sent appendrequest(including the union collect of old and new ) to followers , follower will append entry ,take effect the new config immediately without confirming  the comittment of the entry. leader received the response ,then  commit entry ,change the status to new , reset old peers to new peers ,empty the new peers collection , But i dont know why you do it like this ,could you tell me why ? For vote part ,keeping old peers can handle votes from old peers ,if it is new peers ,use new rule (new quorum size ) ,why? let'say s1 is the leader ,and 2 new servers s4,s5 join the cluster ,then s1 partition for a while, so that s1 ,s4,s5 is still a cluster ,becuase use the new rule ,its quorum is 3, For s2,s3, they restart the election ,to have a new leader(s2 or s3 eventually) ,because they dont know s4,s5 . it seems this way can not prevent issue happening .

This repo is unmaintained.


Proposal: Provide a Card control or CardStyle in WinUI 3

Proposal: Provide a Card Control or a CardStyle

Summary

Currently there are no card control or a card style that developers can readily use as defined here. We only have few styles like card stroke, card fill colour etc but not a complete style that also incorporate elevation of 8.

Rationale

Allows developer to easily implement cards matching fluent design

Open Questions

Should there be a Card Control or a Card Style on a Border?

I think this a very good idea, I think a style could be enough for this.

I think this a very good idea, I think a style could be enough for this.

A Style alone would not handle the Solid and Layered backgrounds, along with the elevation (which requires manually adding the shadow and translations, or using helpers from the toolkit)

A custom Card control is really great implemented in the PowerToys app, but it needed some further improvements (Setting, SettingExpander, e.g.):
https://github.com/microsoft/PowerToys/tree/main/src/settings-ui/Settings.UI/Controls

It would be great if Card control is implemented in WinUI, because there are already a few things to adjust for the content.

@chigy FYI

You can modify and use ControlExample User Control with its styles in WinUI gallery as a workaround for now..

@harvinders @chingucoding Interested in this idea - besides having a simple style (that @ozanyasindogan mentioned), what kind of value would be in a control? Adding some additional properties like e.g. a Title, interactive/hover states? Would love to hear more!
For those looking at the referenced settingscontrols: we just released our latest version of our SettingsCard and SettingsExpander for Windows Community Toolkit Labs: https://github.com/CommunityToolkit/Labs-Windows/tree/main/labs/SettingsControls/src
This allows you to set create a Settings page very easily with just a few lines of XAML and does all of the styling etc. for you!

Making it a XAML element will add some complexity to the visual tree.
I think ideally, it would be nice to be able to attach it to existing panels like StackPanel, Grid, RelativePanel etc as a  which could have properties like Elevation="4" which controls the shadow rendering, and VisualType="Layer".
VisualTypes could include Acrylic, Mica, Layer, Surface - to match the Design Kit guidance.

Inbox styles could be done, but it will not handle the elevation aspect from within XAML.  AFAIU Z value translation can only be done by code behind right now

WinUI 2 needs also.

WinUI 2 needs also.

Why add new stuff for a deprecated solution ?


Add Lucid to list of users

Coverage increased (+0.01%) to 67.573% when pulling c7848545b32937681275371dc1ae82c5305f36a9 on jbrownlucid:patch-3 into 0b78001f8419a2197fcf0cf8293d5b419e6e701e on airbnb:master.

Thank you!


Evaluate Gungnir to ouput compatible comments with Odin pkg

Odin core documentation tools is pkg https://github.com/odin-lang/pkg.odin-lang.org
The main issue with this tools is:

You must build it yourself (no already available binaries or asdf-vm recipes).
Requires too much configuration before hand.
It has a heavy history (500mb). So it must be downloaded using -depth 1 with git or a tarball with curl. This is easy to solve but requires that knowledge since the readme does not specify how to download the code.
Is not as nice as the Gungnir Approach.

But for Gungnir to become more accepted in Odin community it needs to be compatible, or at least output something that can be compatible with style pkg comments.
Example
/*
Whether the given string is "example"

**Does not allocate**

Inputs:
- bar: The string to check

Returns:
- ok: A boolean indicating whether bar is "example"

Example:
	foo("example")
	foo("bar")

Output:
	true
	false
*/
foo :: proc(bar: string) -> (ok: bool) {}

One approach is using json output of the procedure and then convert that json to a pkg compatible comment. In a new file without markdown comments but with pkg comments.

Read the original file with markdown comments.
Process with Gungnir algorithm and save the result in <procedure>.json.
Take the generated json and read the original file. Replacing all the Markdown coments with pkg comments. This will generate a file that can be processed with pkg tool.

Gungnir takes the mdbook backend as the default renderer. But maybe can also be compatible with the pkg renderer.

priv/documentation/mdbook -> mdBook compatible files for documentation rendering.
priv/documentation/pkg -> pkg. compatible files for documentation rendering.

Upon further investigation this can be solved with documentation extensions.
An extensions directory can be created so files can be replace it with the contents from those files.
This way the Core can continue using "Odin Doc" comments
and a Gungnir Documentation can be created alonside. By replacing documentation comments and declarations
to be more extensivly documented.
Maybe this endevour can be migrated to another project focused only on Documentation in Gungnir for the Odin core libraries.


Set nostackexec and stack-protector options

CoreRT port of https://github.com/dotnet/coreclr/pull/4257. Ported stack-protector compiler settings as well while I was on it.

This page says it's preferred to add a line to the .S files instead of passing --noexecstack to the compiler for portability reasons. Just want to make sure that option was considered. LGTM otherwise.

Adding ifdefed code to every .s file as they are suggesting does not seem great for portability. It means that you have to fix a ton of places instead of just one when you need to change it for new target. And you have to be careful to add the cruft to every assembly file in your project. It is easy to miss one.
Searching for -Wa,--noexecstack on github shows that it is used by many projects. It should be portable enough.
cc @janvorli

LGTM