Latest Release breaks compatibility with Python 3.7

The latest changes in v2 are no longer compatible with Python 3.7. Run this example in an environment with Python 3.7 installed to reproduce the error:
>>> from kennard_stone import train_test_split
>>> import numpy as np

>>> train_test_split(np.array([1,2,3]).reshape(-1, 1))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jackson/mambaforge/envs/py37/lib/python3.7/site-packages/kennard_stone/kennard_stone.py", line 142, in train_test_split
    train, test = next(cv.split(X=arrays[0]))
  File "/home/jackson/mambaforge/envs/py37/lib/python3.7/site-packages/sklearn/model_selection/_split.py", line 1600, in split
    for train, test in self._iter_indices(X, y, groups):
  File "/home/jackson/mambaforge/envs/py37/lib/python3.7/site-packages/kennard_stone/kennard_stone.py", line 75, in _iter_indices
    indexes = ks.get_indexes(X)[0]
  File "/home/jackson/mambaforge/envs/py37/lib/python3.7/site-packages/kennard_stone/kennard_stone.py", line 194, in get_indexes
    X=X, lst_idx_selected=lst_idx_selected, idx_remaining=idx_remaining
  File "/home/jackson/mambaforge/envs/py37/lib/python3.7/site-packages/kennard_stone/kennard_stone.py", line 211, in _sort
    sum(lst_idx_selected, start=[])
TypeError: sum() takes no keyword arguments

This change in the call to sum should be reverted, or Python 3.7 should be dropped from the list of supported Python versions.

Thank you very much for your report. I had missed the bug.
I've fixed the bug and performed pytest on each version.
https://github.com/yu9824/kennard_stone/actions/runs/4798295831
I will release the fixed version soon.
Best regards

I have released a modified version (v2.0.1) on both PyPI and Anaconda.


Recursive $ref failed

the sample
{
        "type" : "object",
        "properties" : {
          "child1" : {
                "type" : "string"
            },
          "child2" :{
               "$ref" : "#/properties/child1"
            }
        }
}

it doesn't work
got error   There's a problem: URI is not absolute

I can see that this issue is kind of old but I'm running in the same problem. Is there a plan to fix it ?

So it looks like the original example works correctly (the simple case) - that issue was fixed a long time ago. There is a more complicated example here: https://github.com/joelittlejohn/jsonschema2pojo/issues/250#issuecomment-56911814 and this does not work. This is not a problem with simple recursive refs but a problem with having a schema that is referred to by a fragment ref then using another fragment ref to refer outside the inner schema. This only seems to affect the case where there is no parent file (the schema is supplied as a string), and jsonschema2pojo.org is one such place.


#__blaze-root div overrides css height property?

I was working with the jquery UI Layout plugin and needed to set the height to 100%.  I was attempting to with html, body, and #myContainer, but didnt realize there was an encapsulating div with the #__blaze-root id.  Initially I thought my issue was non-blaze related and had something to do with the load order of the scripts, but eventually realized it was due to #__blaze-root.
Is#__blaze-root supposed to have css properties?  I have since added it to my stylesheet and do not have an issue, but the documentation doesn't mention anything about this div or the id.

Check this. You can change the root as you want.
See: https://github.com/kadirahq/blaze-layout#set-different-root-node

Ah i see. Thank you @arunoda

I see that you can set CSS properties for _blaze-root. But is there any way to set different blaze root properties for each template?
If I set the _blaze-root property to background-color: green, won't it appear green on all templates? I'm looking for a way to change the properties only for one template.
Also, is there a way to specify BlazeLayout.setroute('body'); to different 'body' like elements for different routes or templates?
Also, there seems to be a body tag automatically generated. Is there a way to change the CSS property of this body tag on one template, but not of them all? I can't properly change the CSS background-color fill because I would need to change this body template's properties, but changing this body template's properties would affect all of them.


WIP: Silence expected exceptions

Stack trace that print directly to stderr are suppressed, while exceptions logged at error level are still logged.
Should these logs be suppressed too?
Any advice on the "copyright" for the copied function from cats-effect?
Closes issue  #2049

We usually leave an attribution like this.  You want to copy the copyright of whatever you copied.  And then make sure the project is mentioned in NOTICE, but it already is.
Thanks for this!


Option arity of zero is ignored when value is attached via equals sign

Specifying the arity of a Option like this
@Option(
    names = "-v",,
    arity = "0")
private boolean[] verbose = new boolean[0];

does prevent invoking <program> -v false as intended, but <program> -v=false works.
[picocli DEBUG] [2] Processing argument '-v=false'. Remainder=[-vv]
[picocli DEBUG] Separated '-v' option from 'false' option parameter
[picocli DEBUG] Found option named '-v': field boolean[] test.Test.verbose, arity=1
[picocli DEBUG] 'false' doesn't resemble an option: 0 matching prefix chars out of 11 option names
[picocli INFO] Adding [false] to field boolean[] test.Test.verbose for option -v on Test@5c29bfd

I scimmed through the code and found this line:
            if (lookBehind.isAttached()) {
                arity = arity.min(Math.max(1, arity.min)); // if key=value, minimum arity is at least 1
            }

Hence the question: Is this intended? And if so, how can I circumvent this behaviour?

Yes this is intended.
As you know, for boolean values the default arity is zero (no parameters required), and it is possible to force the end user to specify a parameter by setting arity=1.
The idea was that , even with the default arity of zero (no parameter required), from the parser's point of view there is no harm in accepting a parameter to a boolean option, if it is unambigously clear that the end user intended to specify this value for the boolean option. Hence, only attached values like -v=false are allowed, whereas -v false is not accepted as the false argument could be a subcommand or positional parameter.
If you really want to prevent this (sounds a bit strict, to be honest 😅 ), please take a look at customized parameter processing in the manual. The IParameterPreprocessor section has an example that has some similarities to your use case.
Also look at the Custom Validation section for how to throw a ParameterException that results in the nice end-user facing message.

sounds a bit strict, to be honest :sweat_smile:

Yeah, that's fair.  I just had a hard time determining the level of verbosity in this example by this.verbose.length, since it would still be 1 if the invokation was -v=false, which is obviously not what the user intended.  Yes, I could check wether all items are true or subtract the amount of false items, but those options seem a bit hacky to me since in my opinion there should not be a usecase of someone calling the program similar to -v=false -vv, which is why I'd like to prevent it entirely.
I guess I will resort to implementing custom logic.  Thank you for the quick reply!

Oh, now I see the issue! 😅
Yes, what to do if someone calls the program with -v=false -vv.  Interesting question.
Personally I would not worry about it too much.
Maybe put a description like "verbosity increases with every occurrence on the command line" and if someone complains you can just say the program ignores the parameter and that was intended.
Is throwing an error better? Not sure...
(But that depends on the app, perhaps.)
Thanks for raising this and enjoy using picocli!


Automated cherry pick of #4405: fix panic on fstab no content

Cherry pick of #4405 on release/2.11.
#4405: fix panic on fstab no content

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@d061a79). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #4408   +/-   ##
==============================================
  Coverage                ?    7.1%
==============================================
  Files                   ?     480
  Lines                   ?   78088
  Branches                ?       0
==============================================
  Hits                    ?    5552
  Misses                  ?   72069
  Partials                ?     467

Flag
Coverage Δ

#aFlag
7.1% <0%> (?)

Impacted Files
Coverage Δ

pkg/hostman/guestfs/fsdriver/linux.go
0% <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 d061a79...2bd0b5c. Read the comment docs.


feat(task-cancel): task cancellation endpoint

This PR adds and endpoint for task cancellation. A few things to note:

According to functional specification, we should expect KeyboardInterrupt exception (SIGINT), but this this signal task is not cancelled: it changes status to REVOKED, and then, after process is finished,  becomes successful. To actually kill the job SIGTERM and SIGKILL are suitable.
There are a few comments left, that after implementation of persistence, some code should be added to populate TaskInfo objects.
TaskExecutor and mutation call tests added.

Like to see these async functions in the code :))


verifier loads the proving keys

Results in ~1.5GB of data loaded by verifier in to RAM that isn't needed.

@deepthiskumar to take a look in a bit


Q: Is there a way to disable cookie fallback?

I'd like to disable cookie fallback for certain items. Is there an option for that?

No, but I supposed we could add that. Also feel free to add it yourself ;)


Jacob/docs new

CC @ccurme @baskaryan

@jacoblee93 Hey Jacob. Is it possible to split this PR into smaller PRs, by one PR per one independent change? We agreed to make changes in small PR because it is faster to progress, better in quality, and easier for contributors and reviewers. Thanks!

@jacoblee93 Hey Jacob. Is it possible to split this PR into smaller PRs, by one PR per one independent change? We agreed to make changes in small PR because it is faster to progress, better in quality, and easier for contributors and reviewers. Thanks!

Hey! In this case we've decided to do it all at once in order to reduce the amount of structural changes to developers.
Vercel build is here if you're keen on having a look:
https://langchain-git-jacob-docsnew-langchain.vercel.app/docs/get_started/introduction

@jacoblee93 It seems that this one-go was needed.
Nice!

navbar style is much better!
top-menu is better
I have couple ideas about the Components menu but it should be another PR

@baskaryan ready for review!
Still want to add a style guide but that shouldn't block.

Style guide added @baskaryan - ready to go!


Mistake in documentation

Here https://www.endurojs.com/docs/multilingual-support is one mistake.
Command:

$ enduro addculture en de
doesn't work.

Correct one is:

$enduro culture add en de

;))

Fixed. Thanks for noticing 👍


mergeCollection handle null values

@neil-marcellini @chrispader from  https://github.com/Expensify/react-native-onyx/pull/437
@marc from https://github.com/Expensify/App/issues/34692
Details
On the above linked PR we made the prepareKeyValuePairsForStorage function remove null values. Removing the null values is only needed for SET operations so we don't have redundant keys e.g. {reportID: 123, nonUsedProp: null, anotherNonUserProp: null}. For MERGE however we need to keep those null values (at the top level) so the provider knows that we want to clear them (at a lower level).
Related Issues

https://github.com/Expensify/App/issues/34692
This is coming from https://github.com/Expensify/App/pull/40254#issuecomment-2067778702
Onyx.merge('report_123', {reportID: 123, participantAccountIDs: [10, 20]}); // have reportID and participantAccountIDs

Onyx.mergeCollection('report_', {'report_123': {participantAccountIDs: null}}) // still have reportID and participantAccountIDs <-- BUG

Onyx.merge('report_123', {participantAccountIDs: null}) // have reportID

Automated Tests
Added a test to verify that nulled key-values on mergeCollection are removed correctly.
Manual Tests
Not needed. mergeCollection is hardly used on /App. It's only used for tests, migrations and stories.
Author Checklist

[X] I linked the correct issue in the ### Related Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@s77rt this is already being fixed here: https://github.com/Expensify/react-native-onyx/pull/519/files

Closing in favor of https://github.com/Expensify/react-native-onyx/pull/519


Add thumbnail option to discord embed.

Would love to see the ability to have a thumbnail instead of the poster in discord notifications.

I feel this can be done, seems like a simple addition. This will probably be added in the next update if the team or I get around to doing it (probably me though).

This was included in 0.5.1 I believe


[4pt]  - Display number of likes for each item on the Homepage - Student A

When the page loads, the webapp the Involvement API to show the item likes and combines them with the data from the base API.
Remember that your page should make only 2 requests:

one to the base API
and one to the Involvement API.

This task does not include displaying the likes button (heart icon on the wireframe) for each item.

https://github.com/oovillagran/Capstone-Module2/pull/29


Running out of system memory due to repeated WriteImage

Describe the bug
Calling WriteImage repeatedly uses increasing amounts of system memory until the program crashes.
To Reproduce
Steps to reproduce the behavior:

Operating system, version, and architecture

OS: Ubuntu 20.04
Architecture: Intel Broadwell

Programming language, and version
Python 3.11.3 (main, Jun 29 2023, 13:17:53) [GCC 11.3.0] on linux
Version of SimpleITK [e.g. 1.1.0], output of the SimpleITK Version() function.
SimpleITK Version: 2.4.0 (ITK 5.4)
Compiled: Aug 13 2024 14:08:24
How was SimpleITK installed?
binary distribution via python -m pip install simpleitk
A minimal working example which causes the error.
This script runs out of system memory (64GB) and crashes trying to save the 209th image.

import SimpleITK as sitk

def sitk_write_image():
    """Repeatedly call WriteImage."""
    image = sitk.Image(300, 512, 512, sitk.sitkFloat32)
    for i in range(1000):
        output = f"image-{i}.nrrd"
        print(f"Saving to {output}")
        sitk.WriteImage(image, output)

if __name__ == "__main__":
    sitk_write_image()

Expected behavior
Memory use should stabilize so the program does not crash.
Additional context

The crashes are occurring on a Google Cloud Platform VM.
The same script does not crash in another environment: not on GCP, no VM, running Windows Subsystem for Linux. Instead, the in-use memory reported by Task Manager grows linearly until it stabilizes around 49GB out of the 64GB available.
I am seeing this behavior across various images and file formats.

Hello @beaslera,
Not sure why this would occur. I tried replicating on OSX and memory usage does not change during the execution.
I tried the specified versions via a GitHub actions runner (had to reduce the number of iterations to 40 so that didn't run out of disk space ~300MB for file). From the telemetry on that PR it doesn't seem that the memory usage changes.
Guesstimating here, could this be somehow an issue with running out of disk space on the GCP VM and that leads to some arbitrary error message? For the 1000 files this is ~300GB.
As a side note, the WriteImage has a useCompression flag (sitk.WriteImage(image, output, True)). Turning it on for the existing code reduces the file size from 300MB to ~300K.

Thank you both for looking at this.  I appreciate your responsiveness.
I've been continuing to work on it and I think what's happening is that the environment is using tempfs and I have been working in the tmp folder.  Saving the files somewhere else does not result in the memory usage growing.


fix(server-renderer): Fix call to serverPrefetch in server renderer with an async setup

This fixes a bug where the serverPrefetch option would not be called with an async setup method.
About the fix
A prefetches reference was created while instance.sp was null. However, with an async setup, instance.sp has the serverPrefetch methods only after setup resolves.
The fix is achieved by waiting to for setup to resolve before trying to access the the serverPrefetch options from instance.sp.
I added a unit test that reproduces the problem, by adding an async setup to the component definition. FWIW, I considered and experimented with adding more async setup variations of the existing tests, but backed out after they did not reproduce the problem and decided they weren't critical.
Impact of the bug
The bug can break SSR because the server renderer will not wait long enough to produce serializable state.
Specifically, this bug exists when using:

defineNuxtComponent from Nuxt 3 (because it creates an async setup method)
with the vue-apollo package options API (because it defines a serverPrefetch method that resolves when all apollo promises resolve)

The server will make a request, but the renderer doesn't wait for it to finish. Nuxt dispatches app:rendered, serializes whatever state it has (which excludes the in-flight apollo graphql requests). Then the page loads in a blank state, then the client in the browser will issue the now redundant graphql requests, and finally render the complete page.
This problem affects Nuxt 2 apollo apps that are migrating to Nuxt 3 and is a regression in the ecosystem.

Hi @sodatea, is there anything else you need from me for this PR to merge?


add generic drivers

@cjellick @will-chan

@cjellick updated the generator with service client. It has the ExtractedConfig field now.

LGTM


scf-eirini-v1-0 /var/vcap/jobs/opi/config/opi.yml missing cc_uploader_ip and registry_address

I'm running scf built from v3-develop and my scf-eirini-v1-0 pod is running, but cf push deployments are failing:
$ cf push my-node-app
Pushing app my-node-app to org system / space dev as admin...
Getting app info...
Creating app with these attributes...
+ name:       my-node-app
  path:       /Users/drnic/workspace/cfsummit-hague/bring-buildpacks-to-kubernetes/quick-demo/sample-app-nodejs
  routes:
+   my-node-app.scf.suse.dev

Creating app my-node-app...
Mapping routes...
Comparing local files to remote cache...
Packaging files to upload...
Uploading files...
 667.97 KiB / 667.97 KiB [========================================================================================================================] 100.00% 2s

Waiting for API to complete processing files...

Staging app and tracing logs...
Unexpected Response
Response code: 500
CC code:       0
CC error code:
Request ID:    cb52a32f-87fc-4d45-4b0a-4512919fe483::598dc53b-a4bd-491e-9ddc-fd727036cec3
Description:   {
  "description": "Runner error: job already exists: Job.batch \"d8274f01-1513-4720-8ec8-f68f0c37836c\" is invalid: spec.template.spec.hostAliases.ip: Invalid value: \"\": must be valid IP address",
  "error_code": "CF-RunnerError",
  "code": 170016
}

When I look in scf-eirini-v1-0 pod's /var/vcap/jobs/opi/config/opi.yml file I see has blank/missing values for cc_uploader_ip and registry_address.
These are supposed to be set via https://github.com/SUSE/scf/blob/v3-develop/bosh/releases/pre_render_scripts/eirini/opi/jobs/patch_opi_yml.sh
I can confirm that the pod itself has the correct env vars mentioned in patch_opi_yml.sh:
$ keti -n scf -c opi-opi scf-eirini-v1-0 -- bash -ec 'echo $SCF_CC_UPLOADER_SERVICE_HOST'
<IP_ADDRESS>
$ keti -n scf -c opi-opi scf-eirini-v1-0 -- bash -ec 'echo $SCF_EIRINI_REGISTRY_SERVICE_HOST'
<IP_ADDRESS>

Is the /var/vcap/jobs/opi/config/opi.yml file being rendered before the associated Kube Services are available to the rendering pod?
/cc @viovanov
Originally discussed in https://cloudfoundry.slack.com/archives/C1BQKKNP4/p1570095847136700

Fixed with https://github.com/SUSE/kubecf/pull/66/files.


Android: Keyboard shortcuts for external keyboards

We added this on iOS (thanks to @nriley) and a user just requested it for Android. Let's try to capture as many keyboard shortcuts from the web we can. And here's screenshots of the mappings on ipad:

@samuelclay How can the keyboard shortcut options be viewed in iOS like in the photos above?
While checking the shortcuts in iOS I noticed that some of them are not working. These are Page Down, Page Up, Dashboard, Share This Story, Scroll to Comments(should it scroll to the bottom if there are no comments or just no op?).

You can see the keyboard shortcuts on an iPad by holding the Command button. But hitting '?' should also work, although I think that only works on the web.

I don't know which keyboard meta key should be used, but use your judgement. I'm fine with whatever, so long as it's discoverable.

Added the option key as a way to discover the keyboard shortcuts, alongside the menu button that is available once an external keyboard is connected.


TypeError: super() argument 1 must be type, not None while install Sphnixcontrib-fulltoc

While install requirements its giving me a error
Collecting sphinxcontrib-fulltoc
Using cached sphinxcontrib-fulltoc-1.2.0.tar.gz
Complete output from command python setup.py egg_info:
Installed /private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/.eggs/pbr-3.1.1-py2.7.egg
ERROR:root:Error parsing
Traceback (most recent call last):
  File "/private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/.eggs/pbr-3.1.1-py2.7.egg/pbr/core.py", line 111, in pbr
    attrs = util.cfg_to_args(path, dist.script_args)
  File "/private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/.eggs/pbr-3.1.1-py2.7.egg/pbr/util.py", line 267, in cfg_to_args
    wrap_commands(kwargs)
  File "/private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/.eggs/pbr-3.1.1-py2.7.egg/pbr/util.py", line 569, in wrap_commands
    cmdclass = ep.resolve()
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2347, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/local/lib/python2.7/site-packages/sphinx/setup_command.py", line 23, in <module>
    from sphinx.application import Sphinx
  File "/usr/local/lib/python2.7/site-packages/sphinx/application.py", line 43, in <module>
    from sphinx.util.tags import Tags
  File "/usr/local/lib/python2.7/site-packages/sphinx/util/tags.py", line 11, in <module>
    from jinja2 import nodes
  File "/usr/local/lib/python2.7/site-packages/jinja2/__init__.py", line 33, in <module>
    from jinja2.environment import Environment, Template
  File "/usr/local/lib/python2.7/site-packages/jinja2/environment.py", line 15, in <module>
    from jinja2 import nodes
  File "/usr/local/lib/python2.7/site-packages/jinja2/nodes.py", line 19, in <module>
    from jinja2.utils import Markup
  File "/usr/local/lib/python2.7/site-packages/jinja2/utils.py", line 486, in <module>
    MutableMapping.register(LRUCache)
  File "/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/abc.py", line 109, in register
    if issubclass(subclass, cls):
  File "/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/abc.py", line 180, in __subclasscheck__
    if issubclass(subclass, scls):
  File "/usr/local/lib/python2.7/site-packages/typing.py", line 1265, in __subclasscheck__
    return super(GenericMeta, self).__subclasscheck__(cls)
TypeError: super() argument 1 must be type, not None
error in setup command: Error parsing /private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/setup.cfg: TypeError: super() argument 1 must be type, not None

----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/93/_381xrh12xb7xyxrxb0_hb6c0000gn/T/pip-build-wFM47z/sphinxcontrib-fulltoc/

This is on a mac right?
Is this trying to install sphinx version 1.6.3?
Can you also give us the pip version and your macOS version?

I think its a Python version problem. Looks like you are using Python 2.7.
This docs works only on Python 3.
On Fri, Sep 15, 2017 at 1:18 PM, Karthikeya Viswanath <
<EMAIL_ADDRESS>wrote:

This is on a mac right?
Is this trying to install sphinx version 1.6.3?
Can you also give us the pip version and your macOS version?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hasura/docs/issues/148#issuecomment-329708528, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAlnw-YWlMcOerNPGDqUDO_Cypwde5niks5siivbgaJpZM4PYikf
.

Apologies. Just tested, it seems to work fine on Arch Linux with Python2.7. The installed Sphinx version is 1.6.3.
Can you report the versions of different things you are using?

OS
Python version
pip version
Sphinx version (that might have got installed)
And are you running inside a virtualenv?

Hi ecthiender

Mac OS x Sierra
pip version 9.0.1
Python version 2.7
Sphinx-1.6.3

It looks like it is a Sphinx error:
https://encrypted.google.com/search?hl=en&q=sphinx+TypeError%3A+super()+argument+1+must+be+type%2C+not+None
With versions 1.6.x. We will try to downgrade Sphinx and try and get back
to you.
On Fri, Sep 15, 2017 at 2:47 PM, Ashroyal<EMAIL_ADDRESS>wrote:

Hi ecthiender

Mac OS x Sierra
pip version 9.0.1
Python version 2.7
Sphinx-1.6.3

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hasura/docs/issues/148#issuecomment-329728238, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAlnw0tkan6n8DJGL9yQM6GvdH4VwpO4ks5sikCngaJpZM4PYikf
.

Working fine when i downgraded sphinx to 1.5.6

@ecthiender @vksci123 : Can we specify the Sphinx version in the requirements.txt file?

Closing this.


Please distribute LICENSE file

Thank you for publishing the addon!
The addon is under MIT license so users have to include the license statements in their repos if they include the addon contents in the repos.
For users to skip copying LICENSE.txt to their repos manually, I like you to distribute the file with addon scripts.

Thanks for your advise!
Already add by this commit.


Adding citations to the relevant papers

Can you please add the citations to the relevant papers for UMFPACK, and the ordering methods it relies on (AMD, COLAMD)?  The list of papers is at https://github.com/DrTimothyAldenDavis/SuiteSparse#how-to-cite-the-suitesparse-meta-package-and-its-component-packages .

Sure, I will prepare a PR. Thanks for making UMFPACK available!

@DrTimothyAldenDavis I have prepared a PR (#70) - added the latest reference for each of UMFPACK, AMD and COLAMD to the README.rst, sphinx docs and the umfpack.py module docstring. Let me know it that is OK, or you would like more/other papers referenced.


Multiple namespaces

We have an option like give namespace,
instrument(global.io, {
    auth: false,
    namespaceName: '/',
});

What if we have dynamic namespaces?

This should be fixed by https://github.com/socketio/socket.io-admin-ui/commit/74f1c20f6ad878c3d11c5fc80dd8d12ee02d7bfb, included in version 0.1.2.


SCP-755 integrate term level NEAT with plutus-core and plutus-metatheory

This contains:

General improvements to the NEAT code.
Generation of terms
Current limitations - it does not generate error, wrap, unwrap con or builtin.
Testing of plutus-core and plutus-metatheory typecheckers.
Testing type preservation:

plutus-core CK passes
plutus-core CEK fails (as expected)

Testing evaluation:

plutus-core CK vs plutus-core CEK (fails on int -> int, passes on int)
plutus-core CK vs plutus-metatheory CK
plutus-metatheory L vs CK vs TCK vs TCEKV vs TCEKC

Moving from arity 0 representation of ifix (a.k.a. mu) in metatheory to arity 2.
Fix to the conversion between plutus-core style de Bruijn indices and metatheory style. Next time I have to touch this I think it would be better to use the same approach in both places (i.e., change the plutus-core code).
SCP-1195.

The tests are failing for several of these reasons reasons:

The conversion between metatheory and implementation de Bruijn types is buggy.
Names don't match.
There's a mismatch between the number of args to mu in the spec and implementation so when I convert from intrinsically typed terms (which follows the spec) to  I expand mu A B to (\ A B -> mu A B) A B so something which is in normal form ends up being expanded to a beta redex.

At least 3 is happening and I don't understand the code for 1 anymore (particularly for terms where there is more going on).
Plan: close the gap in 3 and possibly in 1 too so the conversion is not necessary.

The conversion between metatheory and implementation de Bruijn types is buggy.

One option would be to go to implementation + de Bruijn names and then converting from there? We have support for that already (DeBruijn.hs), which hopefully isn't buggy, and maybe going de Bruijn -> de Bruijn will be easier?

Changing the definition of mu has fixed the NEAT tests but now several of the machines are broken.

fixed the typechecker, it passes CI, several things could do still do with improvement.

Marking this ready for review, still plan to do some tidying up.

I wanted to add @wenkokke as a reviewer which I cannot actually do but please comment!

This has diverged a bit from the original plan as I have been extensively refactoring the typechecker to improve error handling/addressing technical debt. I am going to try to finish this off soon.

github says there are conflicts. I think I fixed them. I guess it might catch up eventually.

github says there are conflicts. I think I fixed them. I guess it might catch up eventually.

Maybe those are some new ones?

There is something wrong with this PR. It's out of sync with the branch it refers to somehow. I will close it and open another one.


See logs from all running containers

Is your feature request related to a problem? Please describe.
Much like in the Docker Desktop application, I would love if I was able to see a live stream of logs from all active containers in the project instead of just one container at a time.
Describe the solution you'd like
This, but in lazydocker:

Describe alternatives you've considered
This may already exist, I might just be ignorant and have missed it. Please correct me if this functionality is already in place.

You can see the results of docker-compose logs by pressing m in the Project panel.
https://github.com/jesseduffield/lazydocker/blob/master/docs/keybindings/Keybindings_en.md#project
You can also modfy the command in commandTemplates.viewAlLogs of the configfile .
https://github.com/jesseduffield/lazydocker/blob/master/docs/Config.md#default

Whenever I press m I don't have any logs:

But I have logs in my containers:


Use vm personality first

Blocked by #72
This PR updates the flow to use a VM's metadata first over the image metadata
Our custom flow for creating a Aq Managed VM (with bespoke personality etc) was broken by the rework, so this adds back that functionality

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.30 :tada:

Comparison is base (34a6e5f) 94.01% compared to head (6d24f80) 94.32%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   94.01%   94.32%   +0.30%
==========================================
  Files          15       16       +1
  Lines         935      986      +51
==========================================
+ Hits          879      930      +51
  Misses         56       56

Flag
Coverage Δ

rabbit_consumer
94.32% <100.00%> (+0.30%)
:arrow_up:

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

Impacted Files
Coverage Δ

...penStack-Rabbit-Consumer/rabbit_consumer/aq_api.py
89.78% <100.00%> (ø)

...ack-Rabbit-Consumer/rabbit_consumer/aq_metadata.py
100.00% <100.00%> (ø)

...abbit-Consumer/rabbit_consumer/message_consumer.py
79.41% <100.00%> (+0.78%)
:arrow_up:

...k-Rabbit-Consumer/rabbit_consumer/openstack_api.py
82.60% <100.00%> (ø)

OpenStack-Rabbit-Consumer/test/fixtures.py
100.00% <100.00%> (ø)

OpenStack-Rabbit-Consumer/test/test_aq_metadata.py
100.00% <100.00%> (ø)

...tack-Rabbit-Consumer/test/test_message_consumer.py
100.00% <100.00%> (ø)

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


Create rake tasks to add and remove emergency banner values in Redis

Trello card: https://trello.com/c/zewXbizh
Follows on from PR #990
Motivation
The current emergency banner doesn't allow static to be deployed while the banner is up.
PR #990 retrieved values from Redis to determine:

The banner should be displayed
What the contents of the banner should be, i.e. the heading and banner colour etc.

This PR sets those values.
Expected changes
Two rake tasks have been created, one to deploy the emergency banner, and the other to remove it.
Deploying the Emergency Banner
Run task:
emergency_banner:deploy['{campaign_class}','{heading}','{short_description}','{link}']

Where

campaign_class is the colour of the banner
heading is the <H1> title
short_description is the text that appears under the title
link is the more information link

Removing the Emergency Banner
Run task:
emergency_banner:remove

@bevanloon I've replied to your comments, and fixed up the changes. Cheers!


Return values 'undefined'

First of all thank you for creating something useful.
I'm having a problem, when trying to return a value, it defines as undefined, see:
var list = LolApi.getChampions(true, (err, champs) => champs);
console.log(list); //#undefined

The methods are asynchronous, so you need to continue working with the value you get within the the callback.
var list = LolApi.getChampions(true, (err, champs) => {
    console.log(champs);
}

With the rework in progress now all methods will return Promises, which should then be a bit more obvious.

@Colorfulstan Thanks for replying, I'm working with ExpressJS and would like to pass the variable as a template parameter to be displayed in HTML.
Would?

You need to look it up in the express documentation.
You now know where the variable comes from, that's all I got.
Issue can be closed imo.

This really is more like a ExpressJS or template engine issue.
Like @Colorfulstan said you now know where to get the variable you need but how you get this variable into the template engine you're using is not in our scope.
Closing the issue here.


change consts used for style

Based on
https://wxpython.org/Phoenix/docs/html/wx.richtext.RichTextCtrl.html#wx.richtext.RichTextCtrl
the style constants used are wrong.

I have "This branch has no conflicts with the base branch".

git checkout 1.0.3-release
git pull
git checkout -b eladeyal-intel-change-richtectctrl-style 1.0.3-release
git pull git://github.com/eladeyal-intel/Gooey.git change-richtectctrl-style
git rebase 1.0.3-release

First, rewinding head to replay your work on top of it...
Applying: change consts used for style
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M       gooey/gui/components/widgets/richtextconsole.py
Falling back to patching base and 3-way merge...
Auto-merging gooey/gui/components/widgets/richtextconsole.py
CONFLICT (content): Merge conflict in gooey/gui/components/widgets/richtextconsole.py
Patch failed at 0001 change consts used for style
The copy of the patch that failed is found in: .git/rebase-apply/patch

rebased

👍


improve error messages in handling of Trestle Plan.simulate() exceptions

When Trestle Plan.simulate() encounters an error from a trestle remove operation, the former does not accurately describe the error encountered.
To Reproduce
Steps to reproduce the behavior:

Call trestle remove of a required, valid element from a valid .json file in a valid location that has been previously initialized for trestle.
See the error, which complains about the rollback but not about why the operation failed in the first place.

Expected behavior
A clear and concise error message that describes the reason for the failure.
CLI session
$ trestle remove -f minimal_catalog.json -e catalog.metadata
Remove failed (simulate()): Last stream position is not available to rollback to
$

Desktop (please complete the following information):

OS: MacOS
Version 11.0.1

Additional context
The same nonspecific error message comes up in a few other failed operations, e.g.,
$ trestle remove -f minimal_catalog.json -e catalog.metadata.responsible-parties.organisation
Remove failed (simulate()): Last stream position is not available to rollback to
(trestle)
$ trestle remove -f minimal_catalog.json -e catalog.metadata.responsible-parties.organisation.party-uuids
Warning: trestle remove does not support removing elements of a list: this removes the entire list
Remove failed (simulate()): Last stream position is not available to rollback to
$

This is related to #118.

Closing as removing simulate.


Feature/extract product description logic into a new task

 Extract the product description logic from "Accept a maximum number of orders per day" into a new task


explicitly write core foundation first, only append modules

this fixes the recurring #117, #122 require order problem

Sorry about this guys. Thank you for staying on top of it and contributing!

"Can't find variable: Foundation" again with latest release <IP_ADDRESS> :(

It fixed the issue for me.
@hackeron didn't forget to restart your server?


Implement Async Support

Following on from DbContextScope, Async support and Ambient suppression are required.

Merged to main in PR: #3


Update agents-overview.md

Replaced "on premise" with "on-premises".
Reworded a sentence, it seemed to be remnants of an earlier edit.
A few small cleanups.

Closing/reopening the PR to clear the validation warning.

Learn Build status updates of commit cab4fac:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

articles/azure-monitor/agents/agents-overview.md
:bulb:Suggestion

Details

articles/azure-monitor/agents/agents-overview.md

Line 98, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 128, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@guywi-ms
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Learn Build status updates of commit 4712621:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

articles/azure-monitor/agents/agents-overview.md
:bulb:Suggestion

Details

articles/azure-monitor/agents/agents-overview.md

Line 96, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 126, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Verbose option

I would appreciate a verbose option that let me know what was going on with the pod cache, locking, etc - running it against the raku/doc repository is slow, and it would be nice to have an idea of the progress.

I have included a spinner to show progress.
The compilation used Async, so multiple threads.
More verbose?

Spinner is probably fine, thanks.


Update backwards compatibility info

Update the backward compatibility info for 3.16 symbologies, as discussed in https://github.com/qgis/QGIS/pull/45143

Thanks for the update !
Maybe a cut and paste in source/site/forusers/visualchangelog328/index.rst ?

@Gustry I don't think that change was actually backported, so the 3.28 changelog should be fine. I think if you create a symbology on 3.28.4+ it will still open on 3.16

As I was saying, this commit landed in 3.28.0, so a project made with QGIS 3.28 can't be opened in QGIS 3.16
https://github.com/qgis/QGIS/commit/fc80d24f9a1444629cf2d789dfc4dad1cfd321ec
At the bottom of the screenshot :

@Gustry Good point, I'll try update the changelogs for this and try raise it to try prevent similar issues in the future.

Given the discussion today on the qgis-dev mailing list, I think we can merge this PR ? (I would be in favor applying the suggestion to not blame @3nids ;-) )

Thanks @DelazJ


Upload Image for Nested Components

Describe the bug
Hi Team,
I have Nested Schema with one property as Array of Objects.
I could see  addItem in the admin panel but when I click the addItem I want to have an extra upload Image component for each Item.
Could you please help me How can I do that.
I have tried using modifying the options but no luck
Installed libraries and their versions
"@admin-bro/express": "^3.0.1",
"@admin-bro/mongoose": "^1.0.0",
"admin-bro": "^3.2.5"... mongo, express (whatever you think is relevant)
To Reproduce
Steps to reproduce the behavior:

Create a nested schema with array as type
Add custom component in options of config

Expected behavior
I want upload Image component to be shown in each addItem of admin panel
Screenshots

AdminBroOptions with schema
Paste your AdminBro config with database schema if it could be related to the issue
const adminBro = new AdminBro({
resources: [
{
resource: SuccessStories,
options
},
{
resource: BoardMembers,
options
},
{
resource: Homepage,
properties:{
featuredCauses:{
components:{
edit:AdminBro.bundle('../components/ImageUrl.tsx')
}
}
},
actions:{
new:{
after:uploadImageAfterHook,
before:uploadImageBeforeHook
},
edit:{
after:uploadImageAfterHook,
before:uploadImageBeforeHook
},
delete:{
after:uploadImageHook
}
}
},
{
resource: Events,
options
},);
Desktop (please complete the following information if relevant):

OS: [iOS]
Browser [chrome, safari]
Version [22]

Please do let me know If you any further information for clear understanding
Thanks

Closing because the issue is old. If you still need help, please reopen.


Remove frameworkStartDate and frameworkEndDate

Summary
These two fields can be removed from the frameworkAgreementDetails blob
of data on a framework record as they duplicate information contained at
the top level under frameworkLiveAtUTC and frameworkExpiresAtUTC.
I have already removed these keys from the records across all of our
environments.
Ticket
https://trello.com/c/82u630c9/80

Sorry yes, that's what I meant by "I have already removed these keys from the records across all of our environments."
I've removed them from the records in the DB that had them set (G-Cloud 9/10 and DOS1+2).


Uncaught Error: Cannot read property 'object' of undefinedTypeError: Cannot read property 'objec...

[Enter steps to reproduce:]

...
...

Atom: 1.15.0 x64
Electron: 1.3.13
OS: Microsoft Windows 8.1 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
at ObjectsRegistry.get (C:\APP\DEV-SOFTWARE\atom\app-1.15.0\resources\electron.asar\browser\objects-registry.js:41:28)
at EventEmitter. (C:\APP\DEV-SOFTWARE\atom\app-1.15.0\resources\electron.asar\browser\rpc-server.js:306:31)
at emitMany (events.js:127:13)
at EventEmitter.emit (events.js:201:7)
at WebContents. (C:\APP\DEV-SOFTWARE\atom\app-1.15.0\resources\electron.asar\browser\api\web-contents.js:222:13)
at emitTwo (events.js:106:13)
at WebContents.emit (events.js:191:7)
At C:\Users\Unnamed\AppData\Local\atom\app-1.15.0\resources\electron.asar\renderer\api\remote.js:210

Error: Cannot read property 'object' of undefined
TypeError: Cannot read property 'object' of undefined
    at ObjectsRegistry.get (C:/APP/DEV-SOFTWARE/atom/app-1.15.0/resources/electron.asar/browser/objects-registry.js:41:28)
    at EventEmitter.<anonymous> (C:/APP/DEV-SOFTWARE/atom/app-1.15.0/resources/electron.asar/browser/rpc-server.js:306:31)
    at emitMany (events.js:127:13)
    at EventEmitter.emit (events.js:201:7)
    at WebContents.<anonymous> (C:/APP/DEV-SOFTWARE/atom/app-1.15.0/resources/electron.asar/browser/api/web-contents.js:222:13)
    at emitTwo (events.js:106:13)
    at WebContents.emit (events.js:191:7)
    at metaToValue (~/AppData/Local/atom/app-1.15.0/resources/electron.asar/renderer/api/remote.js:210:13)
    at undefined.remoteMemberFunction (~/AppData/Local/atom/app-1.15.0/resources/electron.asar/renderer/api/remote.js:111:18)
    at ReopenProjectMenuManager.updateWindowsJumpList (/app.asar/src/reopen-project-menu-manager.js:111:16)
    at ReopenProjectMenuManager.update (/app.asar/src/reopen-project-menu-manager.js:79:12)
    at Function.module.exports.Emitter.simpleDispatch (/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/app.asar/node_modules/event-kit/lib/emitter.js:129:28)
    at HistoryManager.didChangeProjects (/app.asar/src/history-manager.js:77:20)
    at HistoryManager.addProject (/app.asar/src/history-manager.js:95:12)
    at /app.asar/src/history-manager.js:36:20
    at Function.module.exports.Emitter.simpleDispatch (/app.asar/node_modules/event-kit/lib/emitter.js:25:14)
    at Emitter.module.exports.Emitter.emit (/app.asar/node_modules/event-kit/lib/emitter.js:129:28)
    at Project.module.exports.Project.addPath (/app.asar/src/project.js:288:29)
    at /app.asar/src/atom-environment.js:1011:40
    at /app.asar/src/application-delegate.js:31:16)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:191:7)

Commands
     -1:06.7.0 application:add-project-folder (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)

Non-Core Packages

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #11692. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


'Yggdrasil Decision Forests' link resolved

link is not valid, so should be corrected.

Thank you for the fix, this was also corrected in a separate PR, so I'm closing this.


Decal maps: Don't throw an error if the decal map plugin can't be instantiated

See https://forum.babylonjs.com/t/sandbox-inspector-not-opening-for-textures/38711/5
Fix #13562

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/13585/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13585/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13585/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13585/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13585/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13585/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13585/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13585/merge#BCU1XR#0


always get error Config: GSLC_SD_EN required despite already enabled

Describe the bug
Config: GSLC_SD_EN required for this program but not enabled. Please update GUIslice/config/
Device hardware

MCU: ??? ATmega2560
*ard-shld-generic1_35_touch (https://www.amazon.de/Anzeigemodul-TFT-LCD-Display-Modul-480x320-Arduino-Vorstand/dp/B07Y6179LG/ref=sr_1_22?__mk_de_DE=ÅMÅŽÕÑ&crid=2G0O6WUT2E2Z0&dchild=1&keywords=arduino%2Btft%2Btouchscreen&qid=1603752162&sprefix=arduino%2Btft%2Caps%2C234&sr=8-22&th=1)
ard-shld-generic1_35_touch

Checklist to try first
Most display/touch issues can be identified by working through the steps in the Configuring GUIslice guide.
Please confirm whether:
Everything works fine
Expected behavior
Arduino should compile
Initialization messages
Config: GSLC_SD_EN required for this program but not enabled. Please update GUIslice/config/
Additional info
I really do not know what is my fault??? Already tried other setups but always get this error despite already enabled in GUIslice_config_ard.h  Is there a further setting which should be done?
Thanks for help
mcarosh

@mcarosh I suggest you start by reading the wiki pages.  In your case look at:
Arduino Mega
It will explain things better than I can.  Basically you are editing the wrong file.
Look down the wiki page for your TFT display driver and touch driver, say it's ILI9341 with stme610 touch (adfruit 2.8 display) in that case it calls for ard-adagfx-ili9341-stmpe610.h  you go into GUIslice/config and find this file and edit it to turn on the SD card support.
Now as the guy who wrote the Builder I would like to be able to output a better warning I'll change it in the next release to point people back to the wiki.
Paul--

@mcarosh -- as Paul noted, the /src/GUIslice_config_ard.h file is not used anymore and has been replaced by the various user configuration files in /config/. You should find a GSLC_SD_EN setting within Section 6 of your config; this will need to be set to 1. I plan on removing GUIslice_config_ard.h in the next version to avoid any confusion -- it was there to enable the library to compile out-of-the-box without any modifications (for a particular display setup), but as users generally need to make some changes to their config files, it probably makes more sense to remove it.
thanks

Thx a lot
I managed Arduino to compile code :)
But display will not initialize???Is there still the problem with sharing pins between LCD-data and touch pins at the mcufriend library??? I can run successfully every example code from the mcufriend library but generated code from GUIslice will not initialize the display???
And again thanks for your help
mcarosh

Ha
Solved it :) Forgot to change the SD_CS-pin to 53. Now everything is working.
And again, really big thx to this great program.
mcarosh


fontbase: Add version 2.10.3

The trusty Fiddler finds another one: https://releases.fontba.se/win/latest.yml


MS Graph for Office 365: Bugfix - Fix for authentication token expire error

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[x] The manual_readme_content.md has been reviewed and added / updated if needed (for bug fixes / features)
[x] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[x] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[x] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[ ] New App
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[x] If you are exposing any endpoints using a REST handler,
please document them in the manual_readme_content.md.
[x] If this is a new connector or you are adding new actions

[x] Please document in the manual_readme_content.md all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[x] If any actions are unable to run on SOAR Cloud, please document this in the manual_readme_content.md.

[x] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[x] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[x] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hello @maciejwalendziuk ,
As Microsoft frequently changes the error messages, instead of doing a direct comparison of the entire error message, we have implemented a solution that searches for specific keywords, namely "expired" and "token," to prevent potential errors or the need for updates in case Microsoft alters the error message in the future. Please let us know in case of any concerns regarding the same.

Changes in this branch have been merged to https://github.com/splunk-soar-connectors/msgraphforoffice365/pull/33, hence closing this PR.


Does it works on latest titanium sdk 12.2.1?

I have tried to configure it but I get error
[ERROR] TypeError: undefined is not a function (near '...WeatherKit.getWeather...')

Hi there! There was an issue in the way the function is called, this is now fixed in the 1.0.0 stable version :-)


Apollo 3.5 appears to have no Lane Tracking?

In Apollo 3.5, I have throughly read through the perception module code.
It appears that there is no Lane Tracking, as in no Lane history, matching detection to previous frames, and no filtering such as a Kalman Filter or Particle filter for processing Lane Lines.
Is this correct?  Or have I missed something? Thanks

We detect lane line type (L0, L1, L2, R0, R1, R2,...) directly from a network. Therefore, we don't need to track lane lines any more. Please check the most recent lane line detection code.

Thank you @techoe
We ended up implementing our own lane tracking...
However, I am confused why you say tracking is not needed....? Isn't there other reasons to have lane tracking, such as motion estimation, prediction, stability, and filtering out bad detections?

When each lane line segmentation results provide lane types (L0, L1, R0, ...). We can simply sample points . To handle bad detection, the RANSAC based line fitting algorithm is added as a post-processing.

Closing this issue as it appears to be resolved. Thanks.


Add our color palette

Currently we use red-hues similar to the fachschafts-colorpalette.
We should design and implement our actual palette

currently figuring out a good way to store system-wide settings. So in terms of implementing the palette, please plan with using the settings query and addSetting mutation respectively. This will look like
mutation addSetting(setting: {
  key: "primary-color"
  value: "#FFFFFF"
})

settings model is fully implemented now and per default serves a setting with key "primary-color" holding our fs red


Revert "Give RunDB an option to find files in storage but not in data…

…base in find_several function (#1244)"
This reverts commit 185dad3256b41d6ea4f7d42612cb932830e5d9c8.
Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coverage: 93.525% (-0.002%) from 93.527% when pulling 6e3e2b3c5dff43b9cd0b8d7667d25346c0210d07 on revert-1244-check_metadata_rundb into a4b2294813183725b32abe42cef1ad95b05d6bb3 on master.


Expose various constants used to create the histogram

When embedding Histogram, it is useful to expose the constants used to create the histogram so that a histogram manager does not have to store duplicate copies of these values.

Thanks!


Add optional sleep_between_fetches parameter

The purpose is to avoid overloading bitBucket server with too many requests per second, if the server is configured with request throttling.

Thanks @ywei2017 this looks good. Would you mind squashing it into one commit?

@mmb Just squashed the commits. Thanks for accepting the PR. By the way, I also did the testing, and it behaves as expected.
Thanks
Yansheng


Why is an &event required for draw_2d function?

How can I draw on the window without an event? Is there some way to invoke draw_2d base on a timer? Thanks.

The event loop handles user input and typical game logic such as when to render and update. It is completely optional and you can write your own logic. One place to start is to study the source code of Events.

Thanks for your reply, I am aware of the use of Events and event loop, but what confused me was that why is an event generally necessary for updating a window? or is there any way to update/render without an event?
 I am using piston_window.

Currently I am just passing a dummy Event::Loop(Loop::Render(RenderArgs{ext_dt,window_size,draw_size})) to draw_2d function, am I in the right direction?

In general, if you want to render you can just do it directly, but you have to swap buffers yourself. However, the events are going to queue up in the window backend since they are enabled. You can manipulate e.g. the underlying SDL2 window etc.
Everything that Piston does in the event loop is a sequence of instructions. Using the Window trait, you can replicate this sequence of instructions through other means. If you wonder whether an event is necessary, the answer is: Of course not. You don't have to do it the way the Piston event loop does it.
You can use the Piston event loop for some more advanced things, like disabling swapping of buffers and then ignore render events. You can just create your own based on a timer. A timer is easy to create: Just use an f64 which is subtracted by delta time from update events and then create your own custom render event when it reaches 0.0. Reset the timer and repeat. Remember to swap the buffer manually at the end of the render event.

The Piston event loop polls events for you, so I think disabling swapping of buffers might solve the problem for you without unexpected side effects.
You can also use https://docs.rs/piston/0.49.0/piston/trait.Window.html#tymethod.wait_event_timeout. It is a lot harder to get it right this way, but it should be possible.
It comes down to whether you are happy with the accuracy that a fixed time step loop gives you. In a fixed time step loop, you trade some accuracy against deterministic behavior. You can do smarter things based on what you need, but I can't help you with that since it requires more complex analysis.

The overhead of e.g. ignored render events with disabled buffer swapping is pretty low, so I would not be concerned about the overhead. What matters is the timing accuracy. However, timing accuracy depends on pretty much everything you do, which requires deeper analysis.
You must also watch out for triple buffering. Some window backends uses more than two drawing buffers, such that you can need to render and swap multiple times before you get the thing you rendered on the screen.

The PistonWindow::draw_2d is just a piece of code that do some instructions, which you can copy and paste into your application to do the same thing and then modify it to your needs. If you don't want to generate a custom render event, I mean.

Alright, thanks a lot for your help! I will try to figure things out as soon as I have time.

Hi, I have dived into the source code for a little bit and decided to prototype with a fixed time loop first. (Since it's easier for now)
Here is a question, what are the purpose/difference between a Loop:: Update and a Loop::Idel respectively? Which is more suitable for a fixed time loop?
Is there any documentation/wiki/tutorial for Piston? I feel like digging into source code is a normal thing when working with Piston lol.

Idle events are emitted before the game loop goes to sleep. Since the game loop can predict how long it is going to sleep, it uses idle event as a way to tell that this is a good time to use the main thread for background tasks.

By a fixed time step I meant the Piston loop. Update events are using fixed time steps.

Thanks, I almost finished prototyping my project. Btw, during these days working whit piston, I find that it will be good to support some sort of multi-threaded event pushing to help better separating game logic and user interface design.
Since this thread is being inactive for some days, and questions already got answers, I am closing this issue now.


Add missing anchor for glossary terms and allow custom formatting for glossary terms and descriptions.

Add missing anchor for glossary terms and allow custom formatting for glossary terms and descriptions.
Signed-off-by: Paul D.Smith<EMAIL_ADDRESS>

What do you mean by 'linking across multiple pages'?  If you mean having a term used one one page but reference a different page then yes and it seems to work fine.  What does not work, and never has in Sphinx, is referencing a glossary term in another glossary description.  So I could not have...
Fred
Fred is the manager of our team and manages (term) Burt
Burt
Burt is (term) Fred's only managee.
The RST markup i.e. :term::fred is just left in place.  This is a limitation of Sphinx and not specific to Confluence output.
Sent from Outlookhttp://aka.ms/weboutlook

From: James Knight<EMAIL_ADDRESS>Sent: 26 April 2018 22:21
To: tonybaloney/sphinxcontrib-confluencebuilder
Cc: papadeltasierra; Mention
Subject: Re: [tonybaloney/sphinxcontrib-confluencebuilder] Add missing anchor for glossary terms and allow custom formatting for glossary terms and descriptions. (#112)
@jdknight requested changes on this pull request.
@papadeltasierrahttps://github.com/papadeltasierra, thanks! I'm liking the PR so far. I have a slew of (hopefully) not too painful requests.
Out of curiosity, have you tried linking a glossary definition across different pages? I'm curious if it works or even if it's suppose to work (haven't played with glossary directive that much; hence why you saw the markup originally marked as completed 😅 ).

In sphinxcontrib/confluencebuilder/translator/wiki.pyhttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184540451:

@@ -55,7 +55,7 @@ def init(self, document, builder):
if SEP in self.docname:
self.docparent = self.docname[0:self.docname.rfind(SEP)+1]

   restricted_macros = builder.config.confluence_adv_restricted_macros

   restricted_macros = self.builder.config.confluence_adv_restricted_macros

This change should not be needed since both builder and self.builder are the same (let alone this change is group in with the glossary change commit).

In setup.cfghttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184540611:

@@ -1,5 +1,5 @@
[bumpversion]
-current_version = 0.9.0.dev0
+current_version = 0.9.0.dev1

I would not change any version related information. Version management should be handled only my @tonybaloneyhttps://github.com/tonybaloney and other maintainers.

In sphinxcontrib/confluencebuilder/init.pyhttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184544194:

@@ -118,3 +118,8 @@ def setup(app):
"""(experimental)"""
"""Support experimental indentation support."""
app.add_config_value('confluence_experimental_indentation', True, True)

While I'm not opposed to adding support for user customization for certain directives, I would recommend at the least to split the proposed glossary fixes and the proposed glossary formatting in individual commits (or even different PRs).
I like your choice on the confluence_fmt_ prefix. I would recommend you categorize the format options into the existing convention for "configuration groups", ex:
"""(advanced-configuration - formatting)"""
""""""
app.add_config_value(...
I would also recommend adding an individual comment per configuration entry (even though they are similar).

In doc/configuration.rsthttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184544708:

@@ -465,10 +465,27 @@ seconds, the following can be used:

 confluence_timeout = 10

+confluence_fmt_glossary_term, confluence_fmt_glossary_desc
Each configuration entry should have it's own dedicated section and own unique example.
I would recommend adding a new section "advanced configuration - formatting" between ("general configuration" and "advanced configuration - processing") and then moving your new configuration entries under the new section.

In test/unit-tests/common/test_custom_formatting.pyhttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184546248:

+class TestConfluenceCustomFormatting(unittest.TestCase):

@classmethod
def setUpClass(self):

   self.config = _.prepareConfiguration()

   self.config['confluence_fmt_glossary_term'] = 'h6'

   self.config['confluence_fmt_glossary_defn'] = ''

   test_dir = os.path.dirname(os.path.realpath(__file__))

   dataset = os.path.join(test_dir, 'dataset-custom-formatting')

   self.expected = os.path.join(test_dir, 'expected')

   doc_dir, doctree_dir = _.prepareDirectories('custom-formatting')

   app = _.prepareSphinx(dataset, doc_dir, doctree_dir, self.config)

   app.build(force_all=True)

   self.doc_dir = doc_dir

   self.app = app

No need to track self.app.

In test/unit-tests/common/test_custom_formatting.pyhttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#discussion_r184546739:

@@ -0,0 +1,34 @@
+# -- coding: utf-8 --
+"""

:copyright: Copyright 2016-2018 by the contributors (see AUTHORS file).
:license: BSD, see LICENSE.txt for details.

(optional) I would suggest the following line instead:
:license: BSD-2-Clause, see LICENSE for details.

(see prospect change: 2a0bc37https://github.com/tonybaloney/sphinxcontrib-confluencebuilder/commit/2a0bc3716a40487211311e641f3186b1a26922ed)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/tonybaloney/sphinxcontrib-confluencebuilder/pull/112#pullrequestreview-115760191, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARXsv-FE66Ol0SQeQe8xhr8yN-GgWserks5tskhogaJpZM4Tm7Mi.

I have been looking at the 'todo' code and thing I can see why you had a concern about cross-page links.  The todo function seems to do links completely differently to the glossary function :-(. Once I've split the function/formatting I'll try and create a simple test for cross-page glossary entries.

I've split out the formatting now.  Once the basic glossary changes are accepted, I'll make a PR for the formatting.  Will split my forthcoming 'todo' updates in the same manner.

@papadeltasierra, thanks for the last changes; much appreciated.


Still have issues sending BCC

I still cannot send BCC, this is the new warning. Can you please give me a tutorial how to fix this. I have no technical background.

Their are different errors every time. Please come up with a solution. I think their are a lot of people with this problem. I use OSX Electron-cash 2.9.2

Change server. That server is hosed. Let us know which one it is.

How do I change server?

Sorry for the questions, I am just not technical, bud i'll get there with some help :-)

I have changed server to abc1.hsmiths.com  60002 also to electroncash.cascharia.com 50002. Still have the same problem. Can you tell me exactly what to do? Because I am not a developer

Try <IP_ADDRESS>

Nope, <IP_ADDRESS> doesn't work :-(

Can you help me kyuupichan. I am really starting to get frustrated. I don't think it is the server. I think it is the 2fa that is the problem. I try to restore my wallet, I try to sweep it. Bud it looks like I am doing it wrong. Please explain to me in many words how I can fix this. Otherwise I don't understand. best Ronald

Sorry I have no idea.  I've only ever see 2FA cause a lot more problems than it solves.
I won't have time to look at this for at least a week.

Can I not turn off 2FA?

Are you the only person who does support for Electron-cash? Because I cannot wait another week. It has already been a week I am spanding every night try to solve this

Ideally the original author of the software you used would support his users. He isn't so we're doing what we can. Trustedcoin, the 2nd signer, also have chosen to not support their users. You should make your complaints known to them.

Much appriciated your support!!! I hope you still can help me when you have the time next week.
I wil complain, this is crazy!  Best Ronald

which 2fa is it?

we dont support 2fa in general because of what kyuupichan said.  there is also #41 thread with many good suggestions.

Solved the issue with the #41 thread :-))))) Thanks a lot! So happy :-)


Including the JS in our own bundle?

I see that the form field declares the media property which loads your react, react-dom and react-modal dependencies + the actual UI code.
We happen to already use react & react-modal in our (admin) application, so it would be cool if we could avoid loading those dependencies multiple times.
Do you have any pointers or documentation on how we could include your UI code in our own bundle? We use webpack for bundling. I can see that https://github.com/bhch/react-json-form is published to npm, but it only lists React v17 in the peer dependencies. Are there known issues with React 18?

I'll test it out with React 18 and update the dependencies.

To avoid loading the dependencies, you can subclass the JSONFormWidget and override the media method to remove those js files which you don't need.

If you go this way, you'll be responsible for rendering the widget yourself on relevant pages. You can find the complete guide in react-json-form docs: https://bhch.github.io/react-json-form/docs/usage/node/.

I've published a new version of react-json-form on npm which supports React 18.


Return basic info about pipeline in GET /projects/:id/pipelines

This is updated v4 API. See https://docs.gitlab.com/ce/api/pipelines.html#list-project-pipelines and https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8875 for more information.

@ivaravko that looks much better (to me at least :wink:) Thanks!


topgrade: 3.6.0 -> 3.7.1

Motivation for this change
https://github.com/r-darwish/topgrade/releases/tag/v3.7.0
https://github.com/r-darwish/topgrade/releases/tag/v3.7.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

/nix/store/0l2v258apn2x8pcskdxfa8b8hjiisvgy-topgrade-3.6.0	  29.0M
/nix/store/ghd8aazlklkgrl8hp3b0yqmn5d6aw182-topgrade-3.7.0	  28.6M

[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This change is

This appears to break MacOS, or at least Catalina. The relevant comment lies in the v3.7.0 release notes, 'Add desktop notifications on failed steps (currently only in macOS)'. This requires a buildInputs = stdenv.lib.optionals stdenv.isDarwin [ darwin.apple_sdk.frameworks.Foundation ];
I am submitting a PR to fix this.


Flet Studio

Adding an app to Flet Studio

[ ] Deep linking in Flutter (1)

[x] iOS
[ ] Android

[x] QR Code generator for Python with output to terminal (1, 2)
[x] Get IP address from NIC (1)

Exit user app

[x] Multi-touch gestures in Flutter (1)
[x] Shake detector (1)

[x] Programmatic update of manifest, icons
[x] FletApp and FletAppStatus controls
[x] flet run CLI
[x] flet create <template>

What exactly are you having in mind with "Flet Studio" ?

https://flet.dev/docs/guides/python/mobile-support#flet-studio-for-ios-and-android

When to release flet-studio-for-ios-and-android？

In a few weeks, hopefully, but it all depends on App Store approval.

Few weeks

😁
I think the flet run is done right?

Well, that's for desktop/web only, not for mobile.

https://flet.dev/blog/flet-for-ios
https://flet.dev/blog/flet-for-android


Did you remove the transformer and run the experiments?

Hi, the transformer is used in your code. Without the transformer, can your method also gain good performance?

Hi, we didn’t conduct ablation without the transformer as it is a vital part of the architecture, but it would be interesting to check the performance in this setup. I hypothesize that the performance would drop significantly as the original dreamer algorithm performs poorly in marl environments

@vladimirrim, Hi Thanks for the clarification. I am also curious that during the trajectory generation procedure, is the transformer also used in "imagination"?

Yes, It is important both for training and execution phases.

Yes, It is important both for training and execution phases.

Dear @vladimirrim, I see. Did some reviewers raise concerns about using the transformer in "imagination"? In MARL, it is challenging to train a world model without using global information since agents have partial observations.

Dear @vladimirrim, I am curious whether you tried another model-based modelling method for MARL. I found without exploiting the global information, the most useful method is to mix all local information to adjust the training of each agent's local world model, like CTDE or the mixing network of QMIX and other Dec-POMDP-based MARL methods. Incorporating the world model into MARL is non-trivial.


When message fetch fails, show error instead of loading animation

Fixes: #4186
There are some subtle things going on here, so I'll want to do some further testing and explaining (in commit messages and JSDocs), so I've marked this as a draft. But having code to look at will likely help along the conversation happening in chat.
So glad to be this much closer to rescuing people from the dreaded infinite loading indicator! 🙂

Mm, from the issue:

Preferably also with a button (low-emphasis, like a text button) to retry.

I haven't done this yet, in this revision.

OK, and I just cleaned this up some more, so it's ready for a first review (I'm unmarking it as a draft). Still not sure how to proceed with fetchOlder and fetchNewer; brainstorming is ongoing here. I'm not sure whether it should block this PR or not. But I think users would be grateful for even an improvement that didn't yet handle fetchOlder and fetchNewer.

(Just addressed some things flagged in CI and pushed again.)

Looks good, merged, thanks!


Issue 994

This can be merged if found ok.
The work that follows this PR is

Op specific Codec integration in EhcacheCodec
tests
ClusteredStore clean up regarding to underlyingStore.(not very sure about it though)

One more thing builders for Chain and Elements, as I think they would be required.

This is looking alright for now, but I wouldn't merge that without the EhcacheCodec changes nor tests.

I have updated the PR just to report/make clear the progress on the issue, & to provide the overview about codec approach as well.
The codecs are added and integrated for all message types.
The code still considerable refactoring and tests are on the way as well.

PR updated. Its done with all codecs and tests.
I have added a passthrough unit test ServerStoreProxyTest to verify client-server interaction as well w.r.t to store interactions only.

Build fails because it times out.

rebased and reviews taken care of


[Task]: Add new relic config to self hosted images that can be enabled for debugging when required

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
We at appsmith can share our new relic license key with the user and should start receiving their monitoring data on our new relic setup.
We need to find a way to specify a unique new relic application name for every self hosted instance. The application name can be specified in the docker env variables.
Sample new relic file is attached:
newrelic.yml.zip

Closing because it is a duplicate of #29892


Update image paths in markdown file

After the recent project restructuring in commit 059d493: Redo Unity project, the image paths in images/PARTS_GALLERY.md were pointing to the wrong location. This commit corrects all image paths, ensuring they now reflect the updated project structure. Images should now display correctly in the document.

This is the current one
and this is the one with my PR

LGTM, thanks for the PR :+1:


Exchange rate is hardcoded, need to query the api to get real time exchange rates

#3 PR fixes the issue


Need timestamp inside log file

Would be helpful to track the progress by adding a simple timestamp on each line of the log file, e.g. 2021-06-07 18:34:59

[duplicated]
asked and discussed in many previous issues, #399 is one example.
There's also already a PR implementing it on a branch, probably will be merged since so many people asked for it.


REFACTOR - Electron 'encryption test' crashes example app on Windows 10

Describe the bug
Running tests that use encrypted sqlite with electron on windows crashes the example app.

To Reproduce

Clone react-sqlite-app-starter
Checkout in the refactor branch
Do all the Installation steps
Run npx cap open @capacitor-community/electron
Change to the tests tab
Click on SQLITE TWO DBS TESTS or SQLITE ENCRYPTION TEST

Expected behavior
Execute encryption tests successfully as informed on Supported methods section
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Pro version 19041
Electron 11.0.1
Plugin version 2.9.0

@gabrielberlanda I will have a look but i do not have a development platform on Windows so it may take some time

@gabrielberlanda I will have a look but i do not have a development platform on Windows so it may take some time

see REFACTOR - All tests failed running electron packaged on Windows 10 #63

see REFACTOR - All tests failed running electron packaged on Windows 10 #63


Added dependencies (and comma) necessary for codegen in 0.2.0-SNAPSHOT

Upgrading from version 0.1.3 to 0.2.0-SNAPSHOT did not work for me, as (presumably) the generated Seq seemed to be missing commas, and there seemed to be types missing. Here is a configuration that works for me locally.

Thanks!


Where is phars for the new releases?

There are no phars for 2.0.4/3 and also 3.x releases, how do i update?

Just build them with the included build script!
Eg,
php ./build
Perhaps you need to enable the write mode for phar-files in php.ini.

I guess I could do just that, but it's much more easy to just download it

You can download in here: http://deployer.org/deployer.phar for 2.x and http://deployer.org/beta/deployer.phar for 3.x.

Thanks


Some emoji are not printed

Hey! Thanks for this gem!
I was just playing around with this with different emoji and fonts and found some behavior which I don't know if it is a bug or I'm missing a configuration option.
Prawn::Document.generate 'sushi.pdf' do
  font 'DejaVuSans.ttf'
  text '👁🤷♀🦄'
end

produces:

The eye is part of Unicode 7.
I tested with unicode-emoji and maybe it's because it has a text representation?
» "👁🤷♀🦄".scan(Unicode::Emoji::REGEX_INCLUDE_TEXT)XT)
=> [
  [0] "👁",
  [1] "🤷♀",
  [2] "🦄"
]

» "👁🤷♀🦄".scan(Unicode::Emoji::REGEX)
=> [
  [0] "🤷♀",
  [1] "🦄"
]

Regarding unicode-emoji, is there a reason why the version constraint even contains the patch version? That way updates from 2.3 to 2.4 depend on a new release of this gem.
https://github.com/hidakatsuya/prawn-emoji/blob/26f83a1c31c70642291d3b2d4a2ddff6379dbe57/prawn-emoji.gemspec#L23

Thank you for your reporting.
:eye: is a Textual Emoji.
https://emojipedia.org/emoji/👁/
Currently prawn-emoji does not render it as an emoji. And Its behavior depends on the following code:
https://github.com/hidakatsuya/prawn-emoji/blob/26f83a1c31c70642291d3b2d4a2ddff6379dbe57/lib/prawn/emoji.rb#L9
Unicode::Emoji::REGEX_VALID means:

Matches (non-textual) singleton Emoji (except for singleton components, like a skin tone modifier without base Emoji) and all kind of valid Emoji sequences
https://github.com/janlelis/unicode-emoji#main-regexes

From the above, the behavior that the emoji is not drawn is the expected but not a bug.
For example, I'm considering providing a feature to change its behavior as follows:
require 'prawn/emoji'

Prawn::Emoji.matcher_regex = ::Unicode::Emoji::REGEX_INCLUDE_TEXT

Regarding unicode-emoji, is there a reason why the version constraint even contains the patch version? That way updates from 2.3 to 2.4 depend on a new release of this gem.

Yes, there is. The reason is that the emoji version of both the emoji image (Twemoji) and the emoji matcher (unicode-emoji) always match.

Thank you for the detailed reply, that's what I suspected. Can't promise I'll find the time for a PR. But if so, I'd guess we should replace REGEX with self.regex? A bit like the groupdate gem does it for certain settings.

@aried3r

so, I'd guess we should replace REGEX with self.regex? A bit like the groupdate gem does it for certain settings.

Sounds good, but the implementation requires activesupport. I implemented in #30 without activesupport. Could you review it?

Fixed by #30 and automatically closed. Please feel free to reopen if you'd like.

Hey! Thank you for #30 and #31, I'm looking forward to trying it out!


Adaptive Cards Roadmap + Feedback

Hey everyone!
Adaptive Cards is constantly changing and the team is working hard to deliver a great experience to all our users. We have the Adaptive Cards Roadmap which can be found on our website and now here on GitHub as well!
What we're planning
Please head over to our Planning Board to check out all the features the team is planning to release in the future. These features do not have a set date for their release, but you could really help the team prioritize by clicking on the feature and describing your need under the "How important is this to you?" section.
Share your feedback
Don't see an important feature listed on our planning board? Head over to our Under Consideration board and take a look there. If your feature is still not there, you can easily submit an idea by clicking the button found on the top right of the site. If you have any questions or feedback on the product please feel free to either leave a comment on this issue or open up a new one and someone on the team will get back to you 😊

Per discussion closing this issue.


No Sign On VirtualHere USB

So Im Using VMWARE 16 To Do This And i dont see The Lego USB On VirtualHere

is the server actually running?

Yeah I got the same issue, I did the troubleshooting thing but still nothing shows up on the VirtualHere

It should be working. Try again, make sure that you're properly installing it and that the script is running.
The script needs to run for virtualhere to work.

Yeah my issue was that I had a VPN running in the background.

Your problem's solved now? Close the issue.

Your problem's solved now? Close the issue.

He didn't open this issue, someone else did.


Error: Only one usage of each socket address is normally permitted

I'm making a python application with eel and ran into an issue when I tried to run multiple instances of application. The error message is:
OSError: [WinError 10048] Only one usage of each socket address (protocol/network address/port) is normally permitted: ('localhost', 8000)
Is there a way to be able to run multiple instances of the same application on the same machine without an issue? I saw this StackOverflow question which seemed to be similar. Can I fix this? Since it's on a localhost server, the port would need to be changed to allow for both to work, right? (I need then to be completely sperate so simply opening new localhost:8000 window wouldn't work).

Having the same thing here

Maybe it has to do with the href's on a file or on the includes, this really seems strange.

Could the localhost:8000 be used by another program that you have running somewhere? (Look (here)[https://stackoverflow.com/questions/48198/how-can-you-find-out-which-process-is-listening-on-a-port-on-windows] to see how to do that)

Hi @FutureLights. If you use eel.start(port=8001), for example, for the second instance of the application you run, then you should no longer have the port conflict.
Without seeing your code I can't provide a concrete code sample to get you going, but I suspect you will want to provide a command-line argument to your script that accepts a port number, and pass this through to eel above, so that you can then run multiple versions of your script and define the port at-will. Eg:
./run_my_eel.py --port 8000
./run_my_eel.py --port 8001

Does this help?

What you could also do is pass {'port': 0} as options to eel at the time of startup. Then the os will figure out an available port and assign it to that particular instance. Then the user can open x number of instances of your app without any issue.

Thanks, I'll definitely be using that.

Hi @FutureLights. If you use eel.start(port=8001), for example, for the second instance of the application you run, then you should no longer have the port conflict.
Without seeing your code I can't provide a concrete code sample to get you going, but I suspect you will want to provide a command-line argument to your script that accepts a port number, and pass this through to eel above, so that you can then run multiple versions of your script and define the port at-will. Eg:
./run_my_eel.py --port 8000
./run_my_eel.py --port 8001

Does this help?

This fixed it, ty


Histogram protocol issue?

I am confused about PlottableAxisContinuous.__getitem__: for uproot4 it seems to give the one-indexed values where I was expecting zero-indexed, since __len__ gives the number of bins without flow. For example:
>>> h = uproot.open('https://raw.githubusercontent.com/CoffeaTeam/coffea/master/tests/samples/testSF2d.histo.root:scalefactors_Tight_Electron')
>>> axis = h.axes[0]
>>> len(axis)
10
>>> axis.edges()
array([-2.5  , -2.   , -1.566, -1.444, -0.8  ,  0.   ,  0.8  ,  1.444,
        1.566,  2.   ,  2.5  ])
>>> axis.edges().shape
(11,)
>>> axis[0]
(-2.0, -1.566)

After much back-and-forth, the histogram protocol settled on not including under/overflow bins, and going beyond the protocol, we are allowed to include a flow=True/False option to turn it on and off. See, for instance, TH1.values and TH1.edges.
But I think what you're running up against is not under/overflow bin handling, but the fencepost principle. If you do __getitem__, you get a section of the fence defined by the two posts that border it. If you do edges(), you see all the posts. It's like the difference between offsets and starts/stops.
If anything, this should be categorized as "policy," rather than a "bug," but I really don't want to revisit the histogram protocol definitions again. I'd like to let those dogs lie.

Uh.. I am fine with fenceposts or whatever but why don't I get the first (not overflow) bin (-2.5, -2) when I ask for bin index 0?

Oh, that is strange, now that I look at the actual numbers. This edges doesn't look consistent with the __getitem__. If you had asked for flow=True, it would add infinity fenceposts to the ends:
https://github.com/scikit-hep/uproot4/blob/ea0456cd4d101eed8c8e20207560e6057bf063c3/uproot/behaviors/TAxis.py#L215-L223
So that means that the __getitem__ should return fXbins[where], fXbins[where + 1] instead of
https://github.com/scikit-hep/uproot4/blob/ea0456cd4d101eed8c8e20207560e6057bf063c3/uproot/behaviors/TAxis.py#L61-L75
which, I guess, only affects irregularly spacaed bins. This is looking more like a bug to me, too (and not a histogram protocol thing).

I've checked and we just don't have any test files with histograms containing irregularly spaced bins. The fact that we have to find files to do proper tests is part of a larger problem: scikit-hep/uproot4#269 and scikit-hep/uproot4#240.
It looks to me like replacing
https://github.com/scikit-hep/uproot4/blob/bfc1380ca304c5b0108417954946b1d793cfeed6/uproot/behaviors/TAxis.py#L74-L75
with
https://github.com/scikit-hep/uproot4/blob/714f86309059d708c180956cd14f1acf6849052d/uproot/behaviors/TAxis.py#L74-L75
is all we have to do. The TAxis.__getitem__ has to be coordinated with TAxis.intervals (same file), but that looks correct.


[dropdown] Wrong default icon for secondary actions

Replace https://github.com/KleeGroup/focus-components/blob/develop/src/common/select-action/index.js#L21 by more_vert
and type should be button

https://github.com/KleeGroup/focus-components/blob/develop/src/list/action-contextual/index.js#L69 It seems the component is not in the props

The this is not bined into the result.

Just add the this as fourth argument

https://github.com/KleeGroup/focus-components/blob/develop/src/list/action-contextual/index.js#L74
there isn't icon for the primary actions

I think the previous pull request #327 also fix this issue


Nvidia-drivers for A10 nodes

Steps to make it work
ssh compute-permanent-node blah
# wget nvidia-drivers from https://www.nvidia.com/Download/index.aspx
sudo yum update
echo "blacklist nouveau" > /etc/modprobe.d/blacklist-nouveau.conf

yum install kernel-devel kernel-headers gcc make
sudo yum install ./nvidia-driver-local-repo-rhel7-460.106.00-1.0-1.x86_64.rpm

wget https://developer.download.nvidia.com/compute/cuda/12.2.0/local_installers/cuda-repo-rhel7-12-2-local-12.2.0_535.54.03-1.x86_64.rpm
sudo rpm -i cuda-repo-rhel7-12-2-local-12.2.0_535.54.03-1.x86_64.rpm
sudo yum clean all
sudo yum remove nvidia-driver-latest*

sudo yum -y install nvidia-driver-latest-dkms
sudo yum -y install cuda
sudo reboot

@steven-basart
I'm out of the loop. What happened here?

When we deploy a cluster with A10s they don't work. These are the steps necessary to make it work.


Make the license machine-parseable

According to http://doc.crates.io/manifest.html, multiple licenses are
to be separated by „/“. More importantly, some tools (eg. lichking)
understand „/“, but not „OR“.

Closed for the same reason as withoutboats/failure#117