[Question] When run finetune.sh, train.py shows an error "in preprocess_plain     assert len(source) == 2"

Question
I print the source in sources, it  shows as below:
[{'from': 'human', 'value': '\nWhat types of signs are in the image?'}, {'from': 'gpt', 'value': 'The image contains several wooden street signs at an intersection, with some of them giving directions to various destinations. There is also a street sign next to a parking lot with a FILA store, and a group of signs posted on a post.'}, {'from': 'human', 'value': 'How are the signs directing people in the picture?'}, {'from': 'gpt', 'value': 'The signs in the picture direct people by providing information about street names, distances, destinations or points of interest, and by guiding them towards specific locations or facilities like a parking lot or a FILA store at that location.'}, {'from': 'human', 'value': 'What is the purpose of these signs?'}, {'from': 'gpt', 'value': 'The purpose of these signs is to provide guidance and necessary information to pedestrians, drivers, and other people navigating the area. They help individuals find their way to specific places, understand road rules and regulations, and ensure a smooth and orderly flow of traffic. They also contribute to the accessibility and readability of a city or town, making it easier for residents and tourists alike to explore and navigate the surroundings.'}, {'from': 'human', 'value': 'What kind of store is the FILA store?'}, {'from': 'gpt', 'value': "The FILA store is a retail outlet for the FILA brand, which is a sportswear company known for its footwear, clothing, and accessories. Established in Italy in 1911, FILA has become a popular brand worldwide for its sports and lifestyle products. The store typically sells athletic shoes, sports clothing, and accessories for various activities such as running, tennis, basketball, and training, as well as casual wear like t-shirts, hoodies, and sweatpants that showcase the brand's distinct style and logo."}]
as the code in train.py:
def preprocess_plain(
sources: Sequence[str],
tokenizer: transformers.PreTrainedTokenizer,
) -> Dict:
# add end signal and concatenate together
conversations = []
for source in sources:
# sft
print(source)
print(len(sources))
print(len(source))
print("-------")
# source=source[0]
# print(source)
assert len(source) == 2
assert DEFAULT_IMAGE_TOKEN in source[0]['value']
source[0]['value'] = DEFAULT_IMAGE_TOKEN
conversation = source[0]['value'] + source[1]['value'] + conversation_lib.default_conversation.sep
conversations.append(conversation)
# tokenize conversations
input_ids = [tokenizer_image_token(prompt, tokenizer, return_tensors='pt') for prompt in conversations]
targets = copy.deepcopy(input_ids)
for target, source in zip(targets, sources):
tokenized_len = len(tokenizer_image_token(source[0]['value'], tokenizer))
target[:tokenized_len] = IGNORE_INDEX
return dict(input_ids=input_ids, labels=targets)
the source's length is always not  as 2,  is it a wrong of my json file? I use llava_v1_5_mix665k.json as input.

@Jiushanhuadao Can you let me know how you resolved this ? I am getting this error assert DEFAULT_IMAGE_TOKEN in source[0]['value']

Now I am getting the same error as you assert len(source) == 2 AssertionError

Now I am getting the same error as you assert len(source) == 2 AssertionError

I got the error because I use the old version finetune.sh and use a error json which is not suitable for the llava1.5-finetune.


trying to run "sendtoaddress" RPC in Bitcoind

Could you please provide an example of how you would make the call to Bitcoin Core's "sendtoaddress" method.  I will also need to preset the "walletpassphrase" password phrase prior to calling sendtoaddress.  Thank you in advance for your help.

Hey @pepper28! No problem, the library has a very 'literal' approach to Bitcoin RPC parameters, so for example, with the latest RPC docs here.
A simple unlock + transfer would look like:
const RPCClient = require('@jskitty/bitcoin-rpc');

// Create a new RPC class for a Bitcoind RPC daemon
//                        user    pass    host         port
const btc = new RPCClient('user', 'pass', '<IP_ADDRESS>', 8332);

// Unlock the wallet for 15 seconds (afterwards, it will auto-lock)
btc.call("walletpassphrase", "password123", 15).then(() => {

    // Create a transfer of 0.1 BTC to an address
    btc.call("sendtoaddress", "1DGNa15ZQX0xUgFoqJ2i7Z2DPU2J6hw62i", 0.1).then(res => {
        console.log("TX Hash: " + res); // This should be a TX hash!
    });

});

(this is untested code, but should work, just do some obvious adjustments such as: RPC information, password, bitcoin address and amount)

Thank you so MUCH!!!!

Hope all went well! @pepper28 closing this issue for now, I assume you've gotten what you need. :pray:

Yes thank you very much. Your solution works like a charm.
Sent from my iPhone
On Nov 3, 2022, at 5:23 PM, JSKitty @.***> wrote:

Closed #1https://github.com/JSKitty/bitcoin-rpc/issues/1 as completed.
—
Reply to this email directly, view it on GitHubhttps://github.com/JSKitty/bitcoin-rpc/issues/1#event-7735997746, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABU5GMHSWQ6P44OMUDDSYSLWGQUNFANCNFSM6AAAAAAROECULA.
You are receiving this because you were mentioned.Message ID: @.***>


Authentication fails with "TypeError: Incorrect padding"

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

General information
Updated my RaspberryPi3 running latest stable LibreElec to the latest Leia build (build no. 125), installed widevine using the standard script, installed Netflix plugin by downloading the latest master from this git repo as a ZIP.
Prerequisites

[x] Are you running the latest version?

Description
Plugin installs, on first run, it asks for credentials. When these are provided, the entry in the folder listing shows the user, selecting the user results in the message that the Netflix addon encountered an error.
When Kodi now starts, Netflix already throws an error.
Steps to Reproduce

Boot LibreElec
Done

Expected behavior: No errors, Netflix at least browsable.
Actual behavior: Error when Kodi is started, Netflix not browsable (or playable)
Context (Environment)
Installation
Tried both:

[x] I installed the plugin via zip from the Releases page - 0.12.7
[ ] I´m using the Netflix Repo
[x] I´m using a different source (Please tell which) - current master as of 26.01.2018

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM) - RPI3
[ ] Android

Debug log
https://pastebin.com/VvcgQe8E
Other information
It looks like this is the issue:
17:18:34.354 T:1631515536   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.TypeError'>
                                            Error Contents: Incorrect padding
                                            Traceback (most recent call last):
                                              File "/storage/.kodi/addons/plugin.video.netflix/service.py", line 16, in <module>
                                                from resources.lib.NetflixHttpRequestHandler import NetflixHttpRequestHandler
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/NetflixHttpRequestHandler.py", line 30, in <module>
                                                netflix_session=NETFLIX_SESSION)
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/NetflixHttpSubRessourceHandler.py", line 28, in __init__
                                                self.credentials = self.kodi_helper.get_credentials()
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/KodiHelper.py", line 173, in get_credentials
                                                'password': self.decode(enc=password)
                                              File "/storage/.kodi/addons/plugin.video.netflix/resources/lib/KodiHelper.py", line 203, in decode
                                                enc = base64.b64decode(enc)
                                              File "/usr/lib/python2.7/base64.py", line 78, in b64decode
                                            TypeError: Incorrect padding
                                            -->End of Python script error report<--

I get this error after trying out release 0.14.0 beta 18 and going back to 0.13.9. Is there any chance bringing 0.13.9 back to work again?
13:21:23.482 T:139696378189568   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.ValueError'>
                                            Error Contents: Padding is incorrect.
                                            Traceback (most recent call last):
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/service.py", line 243, in <module>
                                                NetflixService().run()
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/service.py", line 101, in __init__
                                                self.nx_common)
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/resources/lib/NetflixHttpRequestHandler.py", line 81, in __init__
                                                netflix_session=netflix_session)
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/resources/lib/NetflixHttpSubRessourceHandler.py", line 28, in __init__
                                                self.credentials = self.nx_common.get_credentials()
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/resources/lib/NetflixCommon.py", line 75, in get_credentials
                                                return NetflixCredentials().decode_credentials(email, password)
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/resources/lib/NetflixCredentials.py", line 47, in decode_credentials
                                                'email': self.decode(enc=email),
                                              File "/var/lib/kodi/.kodi/addons/plugin.video.netflix/resources/lib/NetflixCredentials.py", line 83, in decode
                                                block_size=self.bs).decode('utf-8')
                                              File "/usr/lib/python2.7/dist-packages/Cryptodome/Util/Padding.py", line 90, in unpad
                                                raise ValueError("Padding is incorrect.")
                                            ValueError: Padding is incorrect.
                                            -->End of Python script error report<--

My fix for this on Kodi 18.2-RC1 OSMC was ‘pip install pycryptodomex’


outputContexts

Hi,
Is possible to persist outputContexts through conversations ?
To have followup intents working ?
Tks

Hello, I'm facing the same issue, did you manage to find a solution ?
Thanks you in advance !


Error trying https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/edit

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/create
Operating System: linux
Program.cs(10,56): error CS0103: The name 'DataTime' does not exist in the current context [/home/xxx/dotnet/myApp/myApp.csproj]
The build failed. Please fix the build errors and run again.

This looks like you made a typo: it's DateTime, not DataTime.

Closing since @svick's suggestion should fix the problem. Let us know if that's not the case.


[PE-D][Tester C] Delete Meeting Bug

When I input a command to delete meeting with a date 31 Feb,  the meeting that has a date 28 Feb gets deleted.

Labels: type.FunctionalityBug severity.Low
original: dexter-sim/ped#5

Similar Issue: #152


After version 12.15, it cannot be used on the Chinese server

There was a problem before 12.15. After entering the game, you need to wait for one minute before changing the skin. After the game is updated, it can no longer be used

我也是这样，你解决了吗

没，等更新吧


 Example comes from here

Just fixed it, thanks for pointing it out.
https://github.com/TimH96/frog-champions/commit/70c2200b6f596da6f75028d264f255fd716781b2


make sure we can use this addon with Ember versions below 2.5.x

Ember.assign was first introduced in 2.5.x. If we try to use this add on with 2.4.x versions it might break if clients browser don't implement Object.assign. This PR makes sure we would try to use Ember.merge as the last resort when neither of the above are present.

Thanks for the PR 👍


BUG: Fix to ITKSetPython3Vars ignoring Python virtual environments (#…

…2068).
New behavior for CMake > 3.15 only.  For CMake > 3.15, ITK should now find Python as documented in FindPython3 in CMake.

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Are Windows Python checks disabled on pull requests (I don't see it)?  I'll go ahead and run it on my Azure account.

I've gone ahead and modified the WindowsPython YAML as mentioned in #2056.  When I tried running it on my Azure account it failed saying it couldn't find Numpy.  I'm running it now and will report the results.

It would be good if you explicitly removed mentions of Python 3.5. We recently decided to only support python 3.6+. That could be a new commit in this PR, or a new PR. (Suggesting this since you are knee deep in Python finding at the moment).

Not sure if this was the commit that made the ITK Python build on Azure pass (seems related in contents and in time), but that is great news. Simply awesome ! Thanks for doing that.

What is the best way now to make CMake pick the desired python installation? I used the "secret" option Python3_EXECUTABLE. If this is the preferred way, should we expose it in the CMake interface?

It's complicated.  Different behavior if CMake < 3.15.  For CMake > 3.15,
I'd recommend following the CMAKE FindPython documentation.  Setting
Python3_EXECUTABLE is a ITK-specific way of doing it.  For FindPython,
Python_ROOT_DIR is the variable to guide the find package.  Virtual
environments are also reliable (just have them active when you call CMake),
except for an unfortunate Windows-specific "feature".  Unless you
set Python_FIND_REGISTRY=LAST, the default will return the highest Python
version found in the Windows registry before evaluating virtual
environments (though Python_ROOT_DIR might still have priority).
On Fri, Nov 6, 2020 at 4:13 PM Dženan Zukić<EMAIL_ADDRESS>wrote:

What is the best way now to make CMake pick the desired python
installation? I used the "secret" option Python3_EXECUTABLE. If this is the
preferred way, should we expose it in the CMake interface?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/pull/2072#issuecomment-723302217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGPH7A7PLY32XZ7G26RWKDSORRIHANCNFSM4TA44VIQ
.

--
Brad T. Moore, Ph.D.
Senior R&D Engineer, Medical Computing
Kitware, Inc.
http://www.kitware.com

For posterity: Python_DIR and Python_ROOT_DIR are ignored. Python3_ROOT_DIR has the desired effect.

Ack, you're right, I made the mistake in my reply.  Python_FIND_REGISTRY
should be Python3_FIND_REGISTRY as well.  I wish they hadn't split out the
API names like that, it's confusing (and easy hard-to-miss bugs).
-Brad
On Mon, Nov 30, 2020, 5:19 PM Dženan Zukić<EMAIL_ADDRESS>wrote:

For posterity: Python_DIR and Python_ROOT_DIR are ignored.
Python3_ROOT_DIR has the desired effect.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/InsightSoftwareConsortium/ITK/pull/2072#issuecomment-736090576,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGPH7EWS5G2LCXKK3HYGTTSSQLAJANCNFSM4TA44VIQ
.


hazure option undocumented

From personal communication, I know that one can get stores allowing key names containing '/' also for azure ('hazure://'-urls)
This feature is currently undocumented, though.

I think this is resolved with 41df98c?


build_torcx_store: Upgrade to Docker 17.10 on disk

Part of coreos/coreos-overlay#2851

Updated to build both 17.10 and 17.09 for automated testing.


⚠ Bofrost.be has degraded performance

In c92108a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1129 ms

Resolved: Bofrost.be performance has improved in fa7a4c7 after 31 minutes.


Add GoSec Security Scan

Motivation
Follow up to issue open-telemetry/oteps#144
GoSec is a static analysis engine which scans go source code for security vulnerabilities. As the project grows and we near GA it might be useful to have a workflow which checks for security vulnerabilities so we can ensure every incremental change is following best development practices. Also passing basic security checks will also make sure that there aren't any glaring issues for our users.
Changes
This PR adds GoSec security checks to the repo

After every run the workflow uploads the results to GitHub. Details on the run and security alerts will show up in the security tab of this repo.

Workflow Triggers

daily cron job at 1:30am
workflow_dispatch (in case maintainers want to trigger a security check manually)

cc @alolita

Shouldn't this check be also perform for all PR builds?

Can we do on PR and recurring against main branch?
Detecting known issues before they are merged is ideal, however, I think it could be possible for issues to be added to the detection engine after merge to main.

Added. How should the GoSec scan errors be addressed?

@xukaren, I'll make a PR to your branch.

@xukaren I decided to just extend your PR, and opened #157, which will supersede this one.

I'm merging this, including the -no-fail flag, which I will remove in #157


Fix/refactor ret opcode

This wasn't breaking any test, but when doing reverts and ok, we were not clearing the registers and the context as specified in the spec. Since ret.panic, ret.revert and ret.ok have very similar behavior, I though it was proper to unify them in ret to make sure we ran the same common behaviour in all of them.

Depends on #144

when doing reverts and ok, we were not clearing the registers and the context as specified in the spec

Be sure to add a link to the specific part of the spec next time, like this.


I Got UnicodeDecodeError because of Turkish characters

PS : i realised that when i change folder name with non turkish characters it works
"İ" , "Ç" , "ı" characters causes error
how can i fix this ?
i got this error when using playsound
Traceback (most recent call last):
File "C:\folder1\folder2\DERS İÇİN\eklenmeye hazır olanlar\blabla v2.0\blablav2.py", line 34, in
uEfekt(a)
File "C:\folder1\folder2\DERS İÇİN\eklenmeye hazır olanlar\blabla v2.0\blablav2.py", line 31, in uEfekt
playsound(musicFile)
File "C:\Users\username\AppData\Local\Programs\Python\Python39\lib\site-packages\playsound.py", line 35, in _playsoundWin
winCommand('open "' + sound + '" alias', alias)
File "C:\Users\username\AppData\Local\Programs\Python\Python39\lib\site-packages\playsound.py", line 30, in winCommand
'\n    ' + errorBuffer.value.decode())
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xfd in position 24: invalid start byte

This is now fixed on master. I have added in a test scenario, testBlockingCYRIL_WAV, which should ensure that it doesn't regress.


[breaking] [jvm-packages] Remove scala-implemented tracker.

close https://github.com/dmlc/xgboost/issues/9044 .

cc @wbo4958 .

@trivialfis  may I suggest removing the whole ml.dmlc.xgboost4j.scala.rabit package as it becomes dead code.

I indeed see someone is using the scala tracker, so I don't know if it's appropriate to delete it.

@wbo4958 well,  it says "  * Note that this implementation is currently experimental, and should be used at your own risk." in documentation. I canupdate it instead of removing, if you prefer. However, I know nothing about it, so I'd have to ask for help. In case, this is preferred way of tackling this issue, please, rephrase the ticket and I can lend a hand

I think we will remove it instead. It's not ideal for us to keep maintenance on two different implementations of the tracker. At some point, we should move the implementation into a shared core.

LGTM if the CI/CD can pass

@trivialfis @wbo4958 FYI this change is causing hard to debug errors if your codebase was using the Scala tracker, as TrackerConf(0L, "scala") means that the Python tracker will try to bind to the scala address, which fails without a clear error message. It would be nice to mention that somewhere in the release note, or better yet, avoid using stringly typed configuration.

The Python tracker is next to remove. We will rework the networking module


More refactoring to take advantage of collective aggregators

Also added ShouldHaveLabels() to encode when we expect to have labels.

@trivialfis @hcho3 Any ideas about the R test errors? Doesn't seem to be related to this PR.

@rongou Please ignore it for now. We are aware of the issue (https://github.com/dmlc/xgboost/issues/9091). Not sure when I'd get time to fix it.


Inputs do not occupy the same physical space error

Slicer now switched to ITKv5 so I gave another go of using this filter in Slicer. Build was successful but when I wanted to run the filter it failed with "Inputs do not occupy the same physical space" error.
The test data set is available here: https://1drv.ms/u/s!Arm_AFxB9yqHnM1-c0YPKUFyxx_sJg
Command-line parameters and process output:

Found CommandLine Module, target is  C:/D/SlicerHeartExtension-win64dbg/inner-build/lib/Slicer-4.11/cli-modules/Debug/PhaseSymmetryFilter.exe
ModuleType: CommandLineModule
Phase Symmetry Filter command line:

C:/D/SlicerHeartExtension-win64dbg/inner-build/lib/Slicer-4.11/cli-modules/Debug/PhaseSymmetryFilter.exe --wavelengths 3,3,3,6,6,6,12,12,12 --orientations 1,0,0,0,1,0,0,0,1 --sigma 0.55 --angularBandwidth 3.14 --polarity 0 --noiseThreshold 10 C:/Users/msliv/AppData/Local/Temp/Slicer/DGHEE_vtkMRMLScalarVolumeNodeB.nrrd C:/Users/msliv/AppData/Local/Temp/Slicer/DGHEE_vtkMRMLScalarVolumeNodeC.nrrd

Inputs do not occupy the same physical space!
InputImage Spacing: [3.8367384e-01, 3.9261347e-01, 2.5173077e-01], InputImage_1 Spacing: [1.0000000e+00, 1.0000000e+00, 1.0000000e+00]
	Tolerance: 3.8367384e-07
InputImage Direction: -1.0000000e+00 0.0000000e+00 0.0000000e+00
0.0000000e+00 -1.0000000e+00 0.0000000e+00
0.0000000e+00 0.0000000e+00 1.0000000e+00
, InputImage_1 Direction: 1.0000000e+00 0.0000000e+00 0.0000000e+00
0.0000000e+00 1.0000000e+00 0.0000000e+00
0.0000000e+00 0.0000000e+00 1.0000000e+00

	Tolerance: 1.0000000e-06

Phase Symmetry Filter completed with errors

Source code (cloned CLI module in the repository and added padding filter as shown in Python example):
https://github.com/SlicerHeart/SlicerHeart/blob/ad7ffff1e6285b7bb0c4296f31c54355ab3d7329/PhaseSymmetryFilter/PhaseSymmetryFilter.cxx

This sounds familiar, was this with ITKPhaseSymmetry master HEAD?

Yes, it is with latest master version.

Hi there,
are there any plans of getting these issues fixed? I would like to use the filter for some preprocessing task. Thanks a lot.

@zahilshanis and I are looking into it. Thanks


Feature request: "global" Connect button

I have a idea for a feature to add to Limor's suggestion to include a bar at the top that shows your pairing status, and is red when no device is shared, and green when your device is paired.
My suggestion is to include a "Connect" button in that red bar when no device is paired. So basically the red bar would say "No device paired." and have next to the text a "Connect" button, exactly like the one inside a project when no board is paired (or a different color if it clashes with red).
The button would not be present on the green bar when a device is paired.
At the moment, the expectation seems to be that a user should click on a desired project, and then click Connect once viewing a project. This feels like it might be a bit obtuse for new users.
I know you're in the middle of making a bunch of UI changes, and some resolution to this may already be in progress. But, basically, overall, being taken immediately to the project list page leaves a lot of questions unanswered. I think with a few minor changes, we can make it a much clearer experience.
Please let me know if you need or want any more info!

Update: I included this suggestion on the mockup design in Zeplin.

"global" Connect button added.
Will be added in TestFlight 1.1(6)


Evaluation Scripts & Results Reproduction

Thanks for the impressive work of SimAlign.
Currently I am working on reproducing the results in your paper, but I don't know how to compute the redefined precision and recall measures presented by Och & Ney, 2003, i.e., which labels should be S labels or P labels?
Could you please provide the eavluation scripts on the test sets used in paper?
Thanks.

Hi @qianliu0708 we plan to add the evaluation scripts soon.

Hi @pdufter, thanks a lot to you and your collaborators on this very interesting work. When your time permits, it would indeed be great if evaluation scripts are provided to reproduce the scores. -- Thanks.

Hi @spookyQubit,
We have added the evaluation script as well as some sample sentences with gold alignments.
If there are any issues with the code or the samples, please reach out to us.

@masoudjs  thanks a lot. Really appreciate it.


Take delegations into account when looking for large bids

This should fix a bug encountered by AJ during testing where a validator with a small bid, but large delegated stake didn't get removed properly in the upgrade and caused the network to stall.

bors merge


Text-Align for texts of different lengths

So I was working on a project which will embed names on the white box and those names differs in terms of length. So I was wondering a method to make the text to appear on the center of the field no matter how long the names are.
P/S : thanks for making this great library been a great help for me

Okay I solved the issue it it was based on issue #10  just follow the method 4 and adjust the height according to your requirements. In my case I did this
firstPage.drawText(name , {
        x: (width - textWidth) / 2,
        y: (height-368) - (textSize * 3),
        size: textSize,
        font: contName,
        color: rgb(0, 0, 0),
        rotate: degrees(0),
    });

Okay I solved the issue it it was based on issue #10 just follow the method 4 and adjust the height according to your requirements. In my case I did this
firstPage.drawText(name , {
x: (width - textWidth) / 2,
y: (height-368) - (textSize * 3),
size: textSize,
font: contName,
color: rgb(0, 0, 0),
rotate: degrees(0),
});


[firebase_admob] feat: support adaptive banner for admob

Description
Support admob adaptive banner.
https://developers.google.com/admob/ios/banner/adaptive
https://developers.google.com/admob/android/banner/adaptive
Related Issues
https://github.com/FirebaseExtended/flutterfire/issues/2003
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

It would be very nice to have adaptive banners. Any idea when it will be available?

better have an example on how to apply the "quickstart"
https://developers.google.com/admob/android/banner/adaptive#quickstart

better have an example on how to apply the "quickstart"
https://developers.google.com/admob/android/banner/adaptive#quickstart

@amirh @bparrishMines please have a look at this pr by @uny .

@amirh @bparrishMines please have a look at this pr by @uny .

@uny WIll you tell, how to use it. I have copied your code but unable to figure out how to use it or start with it. Thanks in advance.

@uny WIll you tell, how to use it. I have copied your code but unable to figure out how to use it or start with it. Thanks in advance.

@uny @ycv005 Our team is unable to look at this as we are currently focused on adding widget support for Ads and have started a closed beta: https://medium.com/flutter/new-ads-beta-inline-banner-and-native-support-for-the-flutter-mobile-ads-plugin-e48a7e9a0e64.
Feel free to follow https://github.com/flutter/flutter/issues/12114. The work on the new Ads plugin will most likely lead to deprecating this one, so we are focusing on getting it to open beta where everyone can use the new plugin.

The firebase_admob plugin will be deprecated in April 2021 in favor of Google Mobile Ads SDK for Flutter.
Google Mobile Ads SDK for Flutter is a new Flutter plugin that supports more Ads formats than firebase_admob does. Google Mobile Ads SDK for Flutter currently supports loading and displaying banner, interstitial (full-screen), native ads, and rewarded video ads across AdMob and AdManager. It also supports displaying banner and native ads as Widgets as opposed to being overlayed over all app content.
We're now recommending projects currently using firebase_admob migrate to Google Mobile Ads SDK for Flutter following the instructions outlined here.
Issues with migrating or with using the new package should now be raised over on the Google Mobile Ads SDK for Flutter repository.


Implement shield spawning

I suspect that this will need to be done separately for each game mode. For each one there's a few things that need to be determined:

The general distribution and frequency of shield spawns across the whole map.
The location of any specific shields on the map (e.g. base shields in CTF)

It might be possible to generalize 1. across all game modes, but 2. will definitely be different for each one.
The easiest way to go about this is to create a system that sends events into a ShieldSpawn channel (I'll create this in the next few days and update this issue once it's done).
When determining the general positions of all the shields, the STATSBOT logs might be helpful for raw data. I'm not sure if they have these events, but they might be helpful if they do

There are too many shields in FFA right now.
Could you make the respawn period of the shield in FFA to be approx 2min? This reduces the number of shields to 1

I've pushed a commit that will halve the shield spawning rate. It'll take a bit for the update to reach the server (it has to get to @derps-airmash 's repos and then up to the server). Hopefully this will bring the number of shields in FFA to more reasonable levels.
If this isn't enough open a new issue and we can continue to tweak it.


feat(ct): add mount support for template strings

This PR adds support for mounting Angular components via a template string. This enables a number of additional test cases:

Testing content projection via ng-content
Testing directives
Testing pipes
Testing component boundaries

Mount has been extended with the following additional options:
imports - If a string template is passed to mount, imports are passed to WrapperComponent imports. If a component is passed to mount, imports are passed to TestBed.configureModule. It is done this way to prevent users from adding new declarables to the component's scope.

The intention behind adding the following options is to enable testing all of the component's boundaries.  They are not needed for mounting template strings.
environmentProviders - Passed to TestBed.configureModule for top level providers that should exist outside of the component injector, such as provideStore from ngrx.
providers/viewProviders - Passed to TestBed.overrideComponent intended for stubbing component level providers.

This PR also contains a number of fixes for the mount function so that it reuses the root element given by Playwright and properly maps aliased component outputs.

See the following test files for example usages:
tests/components/ct-angular/tests/providers.spec.ts
tests/components/ct-angular/tests/slots.spec.ts
tests/components/ct-angular/tests/template.spec.ts

For template string compilation to work, the code must be compiled in JIT mode.

Hi @chronospatian, thanks for your effort. However, we previously decided to hold off on supporting template/imports/providers until the library is merged. Otherwise, we risk continually adding features and delaying the merge indefinitely. So, I'm closing this for now.

All good, hope it gets merged soon :+1:


Trying to run chessgraph script on windows x86 (6.1.7601) 64-bit platform got error message.

C:...\chessgraph>python chessgraph.py --alpha -135 --beta -110 --depth 9 --boardstyle none --san "1. g4 d5 2. e3"
Traceback (most recent call last):
File "chessgraph.py", line 14, in
import cairosvg
File "C:\Users\Raivis\AppData\Local\Programs\Python\Python38-32\lib\site-packa
ges\cairosvg__init.py", line 26, in
from . import surface  # noqa isort:skip
File "C:\Users\Raivis\AppData\Local\Programs\Python\Python38-32\lib\site-packa
ges\cairosvg\surface.py", line 9, in
import cairocffi as cairo
File "C:\Users\Raivis\AppData\Local\Programs\Python\Python38-32\lib\site-packa
ges\cairocffi__init.py", line 47, in
cairo = dlopen(
File "C:\Users\Raivis\AppData\Local\Programs\Python\Python38-32\lib\site-packa
ges\cairocffi__init__.py", line 44, in dlopen
raise OSError(error_message)  # pragma: no cover
OSError: no library called "cairo-2" was found
no library called "libcairo-2" was found
cannot load library 'C:\Program Files\Graphviz\bin\cairo.dll': error 0xc1
cannot load library 'libcairo.so.2': error 0x7e
cannot load library 'libcairo.2.dylib': error 0x7e
cannot load library 'libcairo-2.dll': error 0x7e

Yes, all requirements satisfied.
C:\Users\Raivis\Documents\GitHub\chessgraph>pip install -r requirements.txt
Requirement already satisfied: graphviz in c:\users\raivis\appdata\local\program
s\python\python38-32\lib\site-packages (from -r requirements.txt (line 1)) (0.20
.1)
Requirement already satisfied: cairosvg in c:\users\raivis\appdata\local\program
s\python\python38-32\lib\site-packages (from -r requirements.txt (line 2)) (2.7.
0)
Requirement already satisfied: requests in c:\users\raivis\appdata\local\program
s\python\python38-32\lib\site-packages (from -r requirements.txt (line 3)) (2.28
.2)
Requirement already satisfied: chess in c:\users\raivis\appdata\local\programs\p
ython\python38-32\lib\site-packages (from -r requirements.txt (line 4)) (1.9.4)
Requirement already satisfied: cairocffi in c:\users\raivis\appdata\local\progra
ms\python\python38-32\lib\site-packages (from cairosvg->-r requirements.txt (lin
e 2)) (1.5.1)
Requirement already satisfied: cssselect2 in c:\users\raivis\appdata\local\progr
ams\python\python38-32\lib\site-packages (from cairosvg->-r requirements.txt (li
ne 2)) (0.7.0)
Requirement already satisfied: defusedxml in c:\users\raivis\appdata\local\progr
ams\python\python38-32\lib\site-packages (from cairosvg->-r requirements.txt (li
ne 2)) (0.7.1)
Requirement already satisfied: pillow in c:\users\raivis\appdata\local\programs
python\python38-32\lib\site-packages (from cairosvg->-r requirements.txt (line 2
)) (9.5.0)
Requirement already satisfied: tinycss2 in c:\users\raivis\appdata\local\program
s\python\python38-32\lib\site-packages (from cairosvg->-r requirements.txt (line
2)) (1.2.1)
Requirement already satisfied: charset-normalizer<4,>=2 in c:\users\raivis\appda
ta\local\programs\python\python38-32\lib\site-packages (from requests->-r requir
ements.txt (line 3)) (3.1.0)
Requirement already satisfied: idna<4,>=2.5 in c:\users\raivis\appdata\local\pro
grams\python\python38-32\lib\site-packages (from requests->-r requirements.txt (
line 3)) (3.4)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\users\raivis\appdata
local\programs\python\python38-32\lib\site-packages (from requests->-r requireme
nts.txt (line 3)) (1.26.14)
Requirement already satisfied: certifi>=2017.4.17 in c:\users\raivis\appdata\loc
al\programs\python\python38-32\lib\site-packages (from requests->-r requirements
.txt (line 3)) (2022.12.7)
Requirement already satisfied: cffi>=1.1.0 in c:\users\raivis\appdata\local\prog
rams\python\python38-32\lib\site-packages (from cairocffi->cairosvg->-r requirem
ents.txt (line 2)) (1.15.1)
Requirement already satisfied: webencodings in c:\users\raivis\appdata\local\pro
grams\python\python38-32\lib\site-packages (from cssselect2->cairosvg->-r requir
ements.txt (line 2)) (0.5.1)
Requirement already satisfied: pycparser in c:\users\raivis\appdata\local\progra
ms\python\python38-32\lib\site-packages (from cffi>=1.1.0->cairocffi->cairosvg->
-r requirements.txt (line 2)) (2.21)

Maybe @LazyFrankB can first try to follow the steps here to install WeasyPrint? Let us know...

Just by chance, does chessgraph work now?

Maybe this helps: stackoverflow.


Add DateTimeProgression for java.time

It seems that there is no progression for DataTimeProgression. Is there any particular for this to be the case? There is one for joda time. If not, allow me to request this feature.

@PedroAlvarado
Welcome to PR for that feature.


اشتباه تایپی

با سلام.
گزارش یه مورد کوچیک:
تو لیست ماه ها ، ماه پنجم به جای "مرداد" نوشته شده: "امرداد"
(یعنی یه الف اولش اضافه داره)
با تشکر

https://fa.wikipedia.org/wiki/مرداد
https://fa.wikipedia.org/wiki/امرتات

@behrang Is there option to provide the get the common name instead?

Yes, you can override it yourself. Also I think you can use the localization provided by the moment itself.

Yeah but there should be some kind of option other than forking the repo since so many people want this.
On Thu, Aug 25, 2016 at 3:44 PM +0430, "Behrang Noruzi Niya"<EMAIL_ADDRESS>wrote:
Yes, you can override it yourself. Also I think you can use the localization provided by the moment itself.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/jalaali/moment-jalaali/issues/39#issuecomment-242352159

None of these two need forking the repository. Adding moment localization is just adding a dependency, and overridgin provided localization is a few lines of code in your project.

you mean override original files in node_modules I don't think that would be a good solution.

No. Read moment docs about localizing to understand the API. Read http://momentjs.com/docs/#/customization/ and http://momentjs.com/docs/#/i18n/.


Hardware Detection Tool boot fails

The Hardware Detection Tool fails booting with error "No such file or directory".
The sha256 hash of the netboot.xyz iso: 31fe9dffa917ccb4b1ebfd5f3ac80d661d8d89d1f6c6688861e9b1b281f0f7f6

Looks like http://www.hdt-project.org/ is no longer hosting the project so I'll remove it unless I get an updated/trusted source.
https://github.com/antonym/netboot.xyz/commit/eef9a191c59528bfe8b8fb4cd478e8666039b9bc
Thanks!


I have one question about training method.

Have you applied the RePr training technique of the VGG sota model before RepVGG mentioned in this paper?

No. We have never tried RePr. We cited it here just because it was the best performance of VGG, to the best of our knowledge.

Thank you. Your ideas are easy and effective. Personal guess, but if you apply RePr, you might get a little higher. safe regardly.

@imbadded Do you have a plan to try the RePr on RepVGG?

I think it would be a good experience to try. I'll link to this issue once it's tested.


More rb-sys interop things

Looks like we stepped on eachother's toes a bit, but I made some adjustments to your initial 3.2 upgrade branch. The main goals being:

Hopefully static linking (our production build system does not have --enable-shared for ruby-head)
Expose some rb-sys features for easy usage (global-allocator, ruby-static)

I'm going to test these changes on a prod build tomorrow and will report back! In the meantime, let me know your thoughts! ❤

In general I was imagining people could set rb-sys features like so:

[dependencies]
magnus = { git = "https://github.com/matsadler/magnus", features = ["rb-sys-interop"] }
rb-sys = { version = "0.9", features = ["ruby-abi-version"] }

Personally, I think this could prove to be a bit of a PITA for gem authors. rb-sys is more of an implementation detail to them. For example, I would never want someone to run into the ruby_abi_version symbol missing error... The way I see it is rb-sys should handle the plumbing transparently, and let magnus shine at making the ergonomic API. Gem authors should really only know about magnus until they run against something low-level enough to justify diving into the raw bindings.
I added a feature called gem for rb-sys (taking inspiration from the embed feature). Seemed like a nice way for users communicate that "this Rust code is for a gem", rather than saying "define ruby_abi_version and forego linking Ruby". It's more declarative, and really users shouldn't need to care about that junk.

Ok, so I went ahead and removed all Magnus generated bindings in this PR, and addressed the other comments. Pinging for CI run and re-review ❤

Got CI passing, phew
✅ https://github.com/ianks/magnus/actions/runs/2579001284

Rust's features are a bit awkward to use, so I don't really want to but something required for the common use case (ie writing an extension) behind a feature. It could be a default feature, but that adds even more friction to the embed use case, as then you have to turn off the default features too.
The two thoughts I had were either:

define a ruby_abi_version function with #[cfg(not(feature = "embed"))]
as the init macro will only ever be used in extensions, define ruby_abi_version as part of that macro

The downside of 1 is, features are designed to be additive, so that kind of going against how they should be used. The downside of 2 is, if for some reason you want to write the Init function out by hand without using the macro, you need to also define ruby_abi_version by hand too.
What do you think?

Overall I think 1 is the best option because:

It seems a bit less harder to get wrong
It makes transitioning from other Rust/Ruby frameworks a bit easier

If it's ok with you, I'll leave the changelog update for you. That should give you a last chance to tell me if something needs to be fixed before publishing.
❤

This looks good, thank you so much!


[not ready for merge] data23

This is not ready for merge because it depends on https://github.com/usegalaxy-au/infrastructure/pull/2211 going in first and /mnt/user-data-volD exisiting on galaxy, galaxy-handlers and workers

volA still has 8TB so it can be filled up some more


'ebase-merge' is not recognized...' when doing interactive rebase with atom

Description
When I try to do an interactive rebase with atom, atom screen doesn't show up and error message is shown (see actual behavior). This is the only link I could find relevant to the issue. If I unset atom as core editor, I can successfully rebase with vim. @lee-dohm
Steps to Reproduce (approximately)

[Install git & atom, clone a repo, make atom core editor]
exact command I used was git rebase -i origin/master~3 master

Expected behavior: atom to open
Actual behavior: error message is shown:
'ebase-merge' is not recognized as an internal or external command,
operable program or batch file.

Versions
atom 1.8.0
apm  1.9.2
npm  2.13.3
node 0.10.40
python 2.7.11
git 2.9.0.windows.1
visual studio 2013

Judging from the error message, this is not a bug in Atom. The error you are seeing is coming directly from the Windows shell.
Since we are trying to minimize the deluge of open issues, I'm going to close this issue. However, I will try to assist you in getting the problem resolved via this issue if you want. But I would highly recommend that you join us on Slack instead.
If you prefer to keep going here, please share your Git config, or at least the output of
git config --global core.editor. Thanks! 🙇

Git config (pasted from Slack):
core.symlinks=false
core.autocrlf=true
core.fscache=true
color.diff=auto
color.status=auto
color.branch=auto
color.interactive=true
help.format=html
http.sslcainfo=C:/Program Files/Git/mingw64/ssl/certs/ca-bundle.crt
diff.astextplain.textconv=astextplain
rebase.autosquash=true
credential.helper=manager
filter.lfs.clean=git-lfs clean -- %f
filter.lfs.smudge=git-lfs smudge -- %f
filter.lfs.required=true
user.name=Alican Bozkurt
<EMAIL_ADDRESS>merge.tool=kdiff3
diff.guitool=kdiff3
core.autocrlf=true
core.excludesfile=C:/Users/Alican/.gitignore_global
core.editor=atom --wait
push.default=simple
alias.lol=log --oneline --graph --decorate --all
alias.co=commit -m
alias.pu=push -u origin
gui.recentrepo=D:/Documents/MATLAB/JSTSP
gui.recentrepo=D:/Documents/MATLAB/SC Paper
core.repositoryformatversion=0
core.filemode=false
core.bare=false
core.logallrefupdates=true
core.symlinks=false
core.ignorecase=true
remote.origin.url=https://github.com/Microsoft/monaco-editor.git
remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
branch.master.remote=origin
branch.master.merge=refs/heads/master

Sorry I wasn't able to follow up on this today ... some family stuff went down. I'll be on it tomorrow.

Solved it using the solution provided in #12027

For future Googlers, on Atom 1.8.0 my atom.cmd in %localappdata%\atom\app-1.8.0\resources\cli was already the same as the one at https://github.com/atom/atom/blob/master/resources/win.
The fix for me was changing my git config to this:
git config --global core.editor "'%localappdata%\atom\bin\atom.cmd' --wait"

The above command didn't work for me on Windows 8.1, but this one did:
git config --global core.editor "atom.cmd --wait"


Application Customizer suddenly stopped working on SitePages

Application Customizer Suddenly stopped working on SitePages Page.
It was working on rest of modern page.
I tried deploying it to a new tenantt , it is working there as well
Error screen shot

What are you trying to get the domElement of?

It is a react mega menu code, trying to top placeholder.
This same extension is getting loaded on Allitems.aspx,settings.aspx and other OOB pages.But not loading on SitePages Library Page.
Also if i refresh many times, sometimes the extension is loaded and sometimes it is not

Just an update, I have not made any changes to my code today and now its working.
is something going on with SitePages in background ?

There was a regression, which was then fixed. Thanks for following up on it to work now properly. Closing the issue.


Brian/rebalance set issue updates

Moved settleLeftoverTokens to ExchangeWrapperLibrary to be used as a standard in all exchangeWrappers
Validate params inputs for both issue and redeem
Require explicit definition of rebalancingSetQuantity for issue
Clear any tokens traded for that exceed receiveTokenAmounts from rebalancingSetIssuanceOrderModule to caller

Pull Request Test Coverage Report for Build 5479

11 of 11 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 5461:
0.0%

Covered Lines:
997

Relevant Lines:
997

💛  - Coveralls


Z -algorithm

I'm new to open source,I wanted to implement z-algorithm for strings

Done.
Thanks @hariv for the PR!
https://github.com/trekhleb/javascript-algorithms/tree/master/src/algorithms/string/z-algorithm


Crash-recovery

Issue Type: Feature Request
Recover data from crash of the system.
My experience: i had opened a new file and made some edits on the same (but not saved).  However, my laptop crashed. After the reboot, VS code did not have the file open, nor did any edits.
P.S.:  Something like Chrome's "Restore Tabs" after a Chrome/Windows crash
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.16299

@r-math hot exit is meant to do this already, it saves all edited files against a folder and restores them when you next open VS Code. Are you sure you opened the same folder up again? Can you share your settings.json file and also check the contents of the %APPDATA%\Code\Backups directory which is where the backups are stored.


Add Mirroring Guide for e2e.test / Conformance tests

I think this will be a common use-case: Not needing to mirror core Kubernetes but needing to mirror the necessary components to use e2e.test
We should have a new docs/mirroring/e2e-tests.md with sections for "Identifying Images To Mirror" and "Using Mirrored Images". For the format see https://github.com/kubernetes/registry.k8s.io/pull/213 as an example. We should additionally list this guide as "e2e tests / conformance tests: [...]" in the docs so people searching for conformance tests can find it.
References:

https://github.com/kubernetes/kubernetes/blob/fc786dcd1d2efcc241e0e2392086934f2806555d/test/e2e/framework/test_context.go#L358
https://github.com/kubernetes/kubernetes/pull/93510

/kind documentation
/sig k8s-infra testing
/help

/assign


High performance/availability cache setup - feature request

Hello!
I'm currently adding a new feature to node-cache-manager. However I am not sure of the way to go yet, because I can't find a consistent API for the available stores.
The idea of the feature request is to add an always available, low latency multi store setup. This way even if our frontend server loses connection to backend for whatever reason, it will keep working at maximum speed for our front users.
Idea for the high perf/high availability setup using node-cache-manager:

Using a multi store, and wrap method

First store have a TTL of 0 (always resolve, keep data forever)
Secondary stores have a TTL > 0 (fetched from it if not in primary store, expires after TTL)
When data expires from secondary store, data is still returned immediately from first store, while secondary store is being updated, once done primary store will be updated too.

However, since API of stores is not defined, I can't have access to TTL, expired date/time of keys, etc. and of course I don't want to call .get on each store every time wrap is called, to check if it contains the key in order to update it.
Any idea on how to implement this in an efficient manner?
Feature specs available here: https://github.com/dial-once/node-cache-manager/commit/9948303260b1e89967629f9489bd2759a3c0ffba

Are you saying you need methods like cacheStore.getTtl()?

Something like that, for a given key. It will be needed for this feature
I can add it to memory cache, as well as redis cache, but for other caches it may cause the feature to not work because this method wont be available. Any idea for this?

I suppose this could just be added like it is in caching.js:
    /**
     * Binds to the underlying store's `ttl` function if it exists.
     * @function
     * @name ttl
     */
    if (typeof self.store.ttl === 'function') {
        self.ttl = self.store.ttl.bind(self.store);
    }

Your new feature should probably check if the ttl function exists before trying to use it.. or maybe throw an error so it fails hard.

Thanks for your answer, I will assume that .ttl function is required for this feature then. I would like to keep backward compatibility so I don't think I will throw any error, but rather disabling the feature if ttl function is undefined on one of the stores.

I think this sounds good.  This should probably be something a user enables via config.  Something like autoUpdateLowerPriorityCaches: true (hopefully you can think of a better name though).

Ok, will go for a config var then, thanks for your inputs!

@PuKoren I was looking to do something almost exactly the same (in fact your solution sounds better than what I was thinking). Did you end up creating something for this?


[#457] FIX: Upgrade webdriver-manager version for Apple M1 chips

Google decided to rename the chromedriver file for Apple M1 based chips recently which breaks the URL in webdriver-manager 3.8.3. This issue is fixed in webdriver-manager 3.8.4 and so this dependency should be upgraded.

3.8.4 have a new bug in 3.8.4.

@aleksandr-kotlyar is this related to the packaging module? Or what issue specifically?

@MatuMikey yes. Fixed in 3.8.5 wdm. Please update the PR. :)

This fix will go to master branch and will be used in upcoming release-candidate versions.
I have already released support for wdm of 3.8.5 version in 2.0.0b16 from separate branch. Yet the master branch is waiting this PR, so looking forward!

Already in selene 2.0.0b17 release.
Close.


model.lua no zeiler.t7

Hello, I run: th main.lua, get the following error. So where is the the zeiler.t7 file? Thanks
/home/silva/torch/install/bin/luajit: cannot open <data/models/zeiler.t7> in mode r  at /home/silva/torch/pkg/torch/lib/TH/THDiskFile.c:484
stack traceback:
[C]: at 0x7fc61fa65470
[C]: in function 'DiskFile'
/home/silva/torch/install/share/lua/5.1/torch/File.lua:292: in function 'load'
/home/silva/src/object-detection.torch/model.lua:7: in main chunk
[C]: in function 'dofile'
main.lua:32: in main chunk
[C]: in function 'dofile'
...ilva/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
[C]: at 0x00406670

Is this https://github.com/ShaoqingRen/SPP_net/blob/master/model-defs/Zeiler_conv5.prototxt

I indeed took the pretrained models from there, that contain the conv layers and the linear layers. I used a variant of loadcaffe to load it in torch format. I'll put this model online and update the README this weekend.

Thank you, you are so kind.

@northeastsquare this should be addressed in https://github.com/fmassa/object-detection.torch/commit/1ac9fa097666d3d12aa462018c9dd006ffccfe17
Let me know if you have any troubles using the code.


[proof] Make LeafCount type alias

Motivation
Add a type alias for LeafCount as u64 so that 32-bit builds will
continue to work.  This is required for external HSM code that
needs to include types crate.  We lose some type checking but
gain readability and consistency on what is a Version versus
what is a LeafCount.
Have you read the [Contributing Guidelines on pull requests]
Ja, ek het dit gelees.
Test Plan
CI
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Toolchain update left me without rustfmt :(  Fixed.

@bors-libra r+

:pushpin: Commit 126ca6d has been approved by zamsden

:hourglass: Testing commit 126ca6d87cda66f0acf00927b75ec98852e93b8f with merge 91310dc4a07c1bda7b79ad595bb870a58ead799d...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: zamsden
Pushing 91310dc4a07c1bda7b79ad595bb870a58ead799d to master...


⚠ Bofrost.lu has degraded performance

In 3ae14db, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2619 ms

Resolved: Bofrost.lu performance has improved in 50dcb45 after 15 minutes.


Proposing Ben Luddy as reviewer/approver

Signed-off-by: Ricardo Noriega<EMAIL_ADDRESS>Based on his great understanding of the Kubernetes and OpenShift control plane and his great contributions to MicroShift, I think Ben should be part of the MicroShift's core team.
Thanks @dhellmann for suggesting this.

LGTM

/lgtm


[DOC] fix loc/scale parameter names in TDistribution docstring

Fixes location-scale parameter names in the docstring of TDistribution, the parameter names in the docstring were inconsistent with the actual parameter names.
FYI @Alex-JG3, before we release we can easily switch that either way - do you have a preference?
My default action is always to fix the docstring if the logic and the docstring disagree.

Codecov Report
Patch coverage has no change and project coverage change: -0.07% :warning:

Comparison is base (4f46ee9) 68.81% compared to head (1edf272) 68.75%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   68.81%   68.75%   -0.07%
==========================================
  Files          93       93
  Lines        4881     4881
  Branches      886      886
==========================================
- Hits         3359     3356       -3
- Misses       1294     1298       +4
+ Partials      228      227       -1

Files Changed
Coverage Δ

skpro/distributions/t.py
98.59% <ø> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

FYI @Alex-JG3, before we release we can easily switch that either way - do you have a preference?

I have no preference, I am happy with fixing the docstring. Thanks for making this change.


Optimize mergeBuffers

Optimized mergeBuffers to avoid repeated copies, drastically improving performance when there are lots of buffers to merge.

Thanks @kring, looks good at quick glance!
@lilleyse can you please review?

Looks good.


simplify rlimit handling in tests

What does this PR do?
Automatically handles rlimit memlock limit removal in more test cases
Motivation
Strange errors during the first test of a suite
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Extend Select2 drop down with a tree view

Allowing for a tree structure with multi-select would be tremendously valuable for many use cases. Numerous Select2 users would be grateful as evident in the Google Group.
In many usage scenario's Select2's reliance on search for larger sets of values does not cut it. For one the term to search for might be unclear to the user or worse exist multiple times under different parent items, furthermore it does not work in multi-select scenarios for structured data, i.e., user wants to filter a product list by product (sub)categories.

+1

+!

+1

+1

+1

+1

+1

+1

Anyone wanting this right now might look into using this: https://github.com/SE7ENSKY/jquery-select7

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

try react version TreeSelect. https://github.com/react-component/tree-select

+1

+1

:+1:

+1

Guys, do you plan to implement this?
@kevin-brown

+1

Guys, do you plan to implement this?

Eventually? Yes. It would be as a separate results adapter and definitely wouldn't be in the light build of Select2.
It's not on the roadmap though, but if anyone is interested in creating the adapter we will gladly link to it from the documentation.

+1

Best example to follow is this one made with angularjs, http://codepen.io/chrishanson/pen/nqcGA

+1

+1

+1

+1 👀

+1

+1

+1

Guys, Santa just brought us a little something: https://github.com/maliming/select2-treeview

Hi guys, This is my solution, Hope it's useful.
https://github.com/lonlie/select2tree

@lonlie Your plugin looks promising. Kindly improve documentation and translate it in English please so everyone can enjoy it. Thanks

@dustapplications

DariaPlotnikova help me do that. She added instructions in English and Russian for select2tree.

+1

+1

+1

+1

+1

+1

+1

+1

Please refrain from submitting "+1" comments, everyone.  To show your interest, simply click the 👍 button on the top-level comment.

I also faced this issue in my projects.
And I implemented one, your can check if it's suitable for you.
It's here: https://github.com/clivezhg/select2-to-tree

I also faced this issue in my projects.
And I implemented an extension, your can check if my approach is suitable for you.
It's here: https://github.com/clivezhg/select2-to-tree

I also faced this issue in my projects.
And I implemented an extension, you can check if my approach is suitable for you.
It's here: https://github.com/clivezhg/select2-to-tree

I created a tree view based on google drives tree navigation, when you move a document into a folder. Hope it can be of use to someone else;
http://charl.faceclues.co.za/select2gtree/

We don't have immediate plans to provide this. We are focused to fix some major UI bugs (that are majority of issues and PR's). But if you open a PR with unit tests, I will be glad to review and approve if everything is ok :+1:

I was in need of this feature and made my custom implementation of @lonlie version, maybe its useful for someone:
https://jsfiddle.net/kikoDev/6vmzqbn9/

I also faced this issue in my projects.
And I implemented an extension, you can check if my approach is suitable for you.
It's here: https://github.com/clivezhg/select2-to-tree

@clivezhg this is a great solution! thanks!
The only thing that I couldn't sove with your extension is that the parent items remains visible when using the multiple+search options. Image example using your documentation array demo:

Could you please tell me if it's possible to force that?


Iron out text wrapping

Ref: https://github.com/gmsgowtham/react-native-marked/issues/522

https://github.com/gmsgowtham/react-native-marked/pull/527


Provide more secure example of using and storing API credentials

This should be replaced since it is an anti-pattern that we do not want to recommend in our workshops:
https://github.com/dlab-berkeley/Python-Web-APIs/blob/3923c1576085af5baded7722947c5d15784cfaae/lessons/02_api.ipynb#L67
While super convenient it almost always leads to someone accidentally checking in their credentials into github.
Instead, let's replace this with a re-useable pattern of getpass and configparser where the code will:

first try to obtain previously saved credentials by loading with configparser
then if not found, using getpass to request the credentials from the user (which works in notebooks as an input prompt)
then saves those user-inputted credentials using configparser to a path such as ~/.notebook-api-keys which would be presumably outside of the .git controlled directory so it doesn't accidentally get added and checked in.

Note it will also be useful to mention the The JupyterLab Credential Store, though since not everyone is using JupyterLab, we should have a more general solution available, as well, as noted above.
Here are some useful references:

Securely storing configuration credentials in a Jupyter Notebook by Vicki Boykis
Keeping credentials safe in Jupyter Notebooks | by Alexandra Souly | Towards Data Science
How do I properly protect my data access passwords (not jupyter tokens/passwords) on 3rd party jupyter hub services? - Special Topics / Security - Jupyter Community Forum
The JupyterLab Credential Store

@aculich Great point, I just added this to the notebook and pushed to
secure_api
Can you have a look and check if this works on your end? If so, we can push to main.


Ace group viewer

By submitting this pull request, you agree to the contribution guidelines

If you aren't familiar with how to contribute to open-source repositories using GitHub, or if you find the instructions on this page confusing, sign up for one of our Sharing is Caring events. It's completely free, and we'll guide you through the process.

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
Adaptive Card Extension that displays the group members for the current SharePoint Group site.

Thank you @zachroberts8668 for the sample. We will be processing this in the next couple of days. 💜🧡

Closed due to inactivity. If you would like to finish this sample please create a new PR.


Fix suggest contracts windows regression

I guess it worked on Windows purely by chance and got broken when I did the URI parsing refactoring and fixes.

I'm also thinking we should create SourceFile.abs_path_from_uri since we're calling Path.absname on the output so often.

Good idea. Besides that we'd need some test coverage on suggest specs. I'll add those in this PR later.
It would also help if we had Windows CI pipeline.


unable to get<EMAIL_ADDRESS>...

I am trying to run a gorgonia app that import gorgonia/tensor. in trying to get the tensor v0.9.10, i am getting  the following error...
go: gorgonia.org/tensor<EMAIL_ADDRESS>parsing go.mod: go.mod:6: require github.com/apache/arrow/go/arrow: version "latest" invalid: must be of the form v1.2.3
I am running go version 1.13.6 on macOS Mojave
Any help with resolving this would be greatly appreciated. Thanks in advance - jay

My bad... it looks like I did a Go mod tidy but then accidentally reverted the Go mod file

@jay-jayasimha fixed. You can now get @v0.9.11

Thanks @chewxy ! i have gone past the previous issue and am getting the following error:
../../pkg/mod/gorgonia.org/gorgonia@v0.9.13/blas.go:46:12: cannot use b (type BLAS) as type "gorgonia.org/tensor".BLAS in argument to "gorgonia.org/tensor".Use:
BLAS does not implement "gorgonia.org/tensor".BLAS (missing Caxpy method)
Thanks!!

There is no version v0.9.13.... I don't know where the issue might arise

It is referring to gorgonia v0.9.13 and not tensor. I did get the latest tensor 0.9,11.
Thanks, Jay

Will look into it

Thanks !! Appreciate your help.
On Wed, Sep 9, 2020 at 3:03 PM Chewxy<EMAIL_ADDRESS>wrote:

Will look into it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gorgonia/tensor/issues/78#issuecomment-689845766, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AISYN7KQKQONAX6LZ5YORBDSE73TXANCNFSM4RAY6JXQ
.
--
Thanks and with best wishes,
Jay

Hello,
I think I have a related issue:

gorgonia master(eff23e4ec2af4613340fa438b6f3b95ce430ee57) or v0.9.13
tensor master( v0.9.11)

# gorgonia.org/gorgonia
../../gorgonia.org/gorgonia/blas.go:46:12: cannot use b (type BLAS) as type "gorgonia.org/tensor".BLAS in argument to "gorgonia.org/tensor".Use:
BLAS does not implement "gorgonia.org/tensor".BLAS (missing Caxpy method)
FYI:
NO ISSUE with tensor v0.9.9 (but same probelm with tensor v0.9.10)

Pushed a change to Gorgonia that fixes it. Will put out a new release now

Thanks! all good!


[Bug Report] Don't remove child's DOM element in element-ui table with children

Element UI version
2.7.2
OS/Browsers version
win10/chrome
Vue version
2.6.10
Reproduction Link
https://codepen.io/Danik_codepenIO/pen/gyMQry
Steps to reproduce
I have a table with a hierarchical structure.I remove child with array.splice() from table but dom do not re-rendering reactively.
It seems to be an issue with the way element-ui handles data internally. When modifying nested data directly, the table is not getting re-rendered. (But the data was indeed mutated correctly, as could be seen when logging this.tableData after the splice().)
Click button to delete.
What is Expected?
Remove dom child from table.
What is actually happening?
It seems to be an issue with the way element-ui handles data internally. When modifying nested data directly, the table is not getting re-rendered. (But the data was indeed mutated correctly, as could be seen when logging this.tableData after the splice().)

这个bug啥时候能解决呢。会做解决的吧

Here is a hack solution, works fine for me.

define el-table element, and add ref

<el-table
 ref='treeTable'>
</el-table>

update lazyTreeNodeMap data

let table = this.$refs.treeTable;
table.store.states.lazyTreeNodeMap[rowId] = newData;

If there is doubt, print table.store.states.lazyTreeNodeMap  and take a look.


tests: ClusterDetails for AI

First unit test for the AI features
Depends on:

[x] https://github.com/openshift-assisted/assisted-ui-lib/pull/892

/retest


Validation of dynamic inputs

Versions:

VueJs: 2.5.13
Vee-Validate: 2.0.4

Description:
I can't seem to get validation properly working on dynamically added inputs. My issue is when I start filling out the dynamic fields, it triggers the validation and even when they are filled (it's required) it still shows the The XX field is required. below each field and stops the validation method from proceeding regardless if it's filled or not.
Steps To Reproduce:
<template>

		First Name
		<input type="text"
			class="form-control"
			id="first_name"
			v-model="contact.first_name"
			v-validate="{required:true}"
			data-vv-as="first name"
			:name="'contacts[' + index + '][first_name]'">

		{{ errors.first('contacts[' + index + '][first_name]') }}

</template>

It could be related to the inputs not having a key attribute which makes vue re-use them. please try o produce this issue in codesandbox if possible and I would be happy to check things out.

@logaretm I tried adding a key attribute with no luck, I'm sure there's something I'm missing but I'm not sure exactly what it is. https://codesandbox.io/s/n1jz6jl9m

You had some issues with the name binding, using : or v-bind tells Vue to evaluate the attribute value as an expression. so your string firstName_${index} always get interpreted the same for all fields since you are adding single quotes to your bound expressions so they will evaluate to the same value every time which is: 'firstName_${index}' and not 'firstName_0'.
The second issue is you have a model named contact which confuses vee-validate on which model to watch, the one in the loop context or the one in the component data, so renaming it/ removing it would fix the value detection issue.
Here is a simplified example of yours, that seems to work fine. the list ordering is out of the issue scope:
https://codesandbox.io/s/zr0rqk4x2l

Thank you @logaretm that fixed it, cheers. The ordering of the list is as intended 😄


Bump dependencies so fsevents is above v1.2.7

What
Bump the following packages:

nunjucks from ^3.2.0 to ^3.2.2
jest from ^24.7.1 to ^24.9.0

Why
Both nunjucks and jest have fsevents as a (nested) dependency. Version 1.2.7 of fsevents has been deprecated and doesn't work with version 12 of Node. This can be validated by running nvm use 12.13.1 and then running npm install<EMAIL_ADDRESS>the install fails due to a missing binary. Version 1.2.7 of fsevents has also been deprecated.

You can preview this change here:
Built with commit e575a663b55878b1f76d588119ddcbceba3ab5a3
https://deploy-preview-1318--govuk-design-system-preview.netlify.app


Loading associated temporal records only ever works once

... per table, I guess.
In a one-to-many association of two temporal tables, loading the parent (history) record of the child (history) record works exactly once (per runtime) for me, and never again.
Example:
task demo: :environment do
  ActiveRecord::Base.logger = Logger.new STDOUT

  puts 'POST 1:', post_of_comment.inspect # => Outputs the correct PostHistory
  puts 'POST 2:', post_of_comment.inspect # => Outputs nil
end

def post_of_comment
  post = Post.create
  comm = Comment.create(post: post)

  comm.history.at(Time.current).first.post
end

Expectation:
Each call of post_of_comment should find a PostHistory.
Current behavior:
The first attempt succeeds, all other attempts return nil.
This reproduction case is on Github at Zyndoras/temporal_tables_demo (see lib/tasks/demo.rake)
It seems to me that the timestamp used to load the PostHistory is somewhat arbitrary and does not necessarily match anything related to the records created or the timestamp passed to history.at.
What I do not understand though, is why it consistently works the first time the function is called, but then never again.

I've run into a likely related issue. There's something weird going on with the activerecord statement cache where it can't tell the difference between history queries of the same model. I haven't had a chance to dig in deep yet, but my solution so far has been a monkeypatch to disable it for History records
module TemporalPatch
    class << self
      def apply_patch
        puts 'asdfasdf'
        const = find_const
        mtd = find_method(const)

        # make sure the class we want to patch exists;
        # make sure the #filter method exists and accepts exactly one argument
        unless const && mtd&.arity == 1
          raise 'Could not find class or method when patching ' \
                "ActiveRecord::Associations:Association's skip_statement_cache method. Please investigate."
        end
        const.prepend(InstanceMethods)
      end

      private

      def find_const
        Kernel.const_get('ActiveRecord::Associations::Association')
      rescue NameError
        # return nil if the constant doesn't exist
      end

      def find_method(const)
        return unless const

        const.instance_method(:skip_statement_cache?)
      rescue NameError
        # return nil if the constant doesn't exist
      end
    end

    module InstanceMethods
      private

      def skip_statement_cache?(scope)
        klass.name.end_with?('History') ||
          reflection.has_scope? ||
          scope.eager_loading? ||
          klass.scope_attributes? ||
          reflection.source_reflection.active_record.default_scopes.any?
      end
    end
  end

  TemporalPatch.apply_patch

Definitely related. Your patch makes my tests green.

monkey-patch implemented in v3.0.0


Add unit tests for the Registrar.

What's this PR do?
Just wanted to knock this out while waiting on PR review... this adds some simple unit tests for the Registrar's resolve, resolveOrFail, and validateCredentials methods.
How should this be reviewed?
Check that the tests seem to cover all expected behavior and edge cases for those functions!
Checklist

[ ] Documentation added for changed endpoints.
[x] Tests added for new features/bug fixes.

For review: @angaither @weerd

¡OLÉ! :sheep: it!


fix: wrong region passed

Issue
Some 'aws' commands have region passed via the AWS_DEFAULT_REGION flag. This flag is overridable via the AWS_REGION flag which has a higher priority. If this variable is set in your environment variables the commands use this environment variable which may not be equal to AWS_DEFAULT_REGION, commands will fail in such a case.
Fix
We can instead pass the region as explicit flags which will block it from being overridden.
Notes
A new release for github actions will be needed. Create v4.1.1 as this is a backwards compatible patch fix.

Is this something that can be contributed upstream?

Is this something that can be contributed upstream?

Yes

That would be the right thing to do so we don't need to keep a divergent fork.


arc.write - add records/row to existing feature layer in GDB

Is it possible to add records/rows to existing feature layers in a GDB? If not, could this feature be added to the arc.write() function?
Being able to add records would be incredibly useful for updating with R processed spatial data (e.g. ML predictions) with new location data, without having to couple with other software that allows this (e.g. the FME - Esri Geodatabase (File Geodb) Writer, which can do this).
As far as I know, you can only create or overwrite features.
Thanks!

Is your use case strictly for appending rows to a feature layer? Or are you also updating rows in a feature layer?


Overlapped navigation links wrapper by other UI elements

Wrapper for navigation links should not be overlapping by other UI elements.

works fine now


Move ranger code to new repo

Review CLM reports for ranger-gaian plugin -- multiple issues reported
see https://nexus-iq.wl.linuxfoundation.org/assets/index.html#/reports/odpi-egeria/5c16abc7c09d40c8a28358f959c68ef7

The ranger gaian plugin has the following issues - which do not affect any other modules in egeria
apache-beanutils:common-beanutils:1.7.0 - Used in Apache Ranger Gaian plugin - CVE-2014-0114 (7)
common-beanutils:common-beanutils-core:1.8.0

Used in Apache Ranger Gaian plugin - CVE-2014-0114 (7)

io.netty:netty:3.7.0.Final - CVE-2015-2156 (7)

CVE-2014-0193 (5)
org.apache.hadoop:hadoop-common:2.9.2
CVE-2018-8009 (8)

org.apache.kafka:kafka_2.11:1.0.0

CVE-2018-17196 (8) - CVE-2018-1288 (5)org.apache.zookeeper:zookeeper:3.4.6 - CVE-2017-5637 (7) - CVE-2018-8012 (7) - CVE-2019-0201 (5)org.codehaus.jackson:jackson-mapper-asl:1.9.13 - CVE-2017-7525 (9)

The ranger (and to a lesser extent gaian) code is raising many security scan issues and
liable to quite a number of CVEs
This is brought in through the current ranger code we use, and in particular the client libraries.
We either need to be able to get the client very thin, or more likely move the ranger code to it's own repo

We had also discussed the potential to move this into a Hadoop-oriented connector repository (and do the same with the Apache Atlas connector) -- that could be a reasonable compromise between proliferating too many repositories while still keeping logical units together?
(Apache Atlas relies on many of these same underlying libraries, so there should hopefully also be some synergies in addressing the CVEs in a common way across them.)

See also https://github.com/odpi/egeria/issues/2739 for a broader discussion (my notes) in terms of how we might manage repos.
Broadly in agreement though that for the hadoop case moving both items mentioned there would make sense

Also to add to the hadoop list -- the docker build for atlas/ranger

Starting work on this today ... in conjunction with #odpi/egeria-connector-hadoop-ecosystem#147

We should move

Atlas docker builds
Ranger docker builds
other ranger/atlas / very specifically hadoop related code

to our hadoop ecosystem repo
and setup appropriate build pipelines
Anything related to the helm charts can go to the samples repo
cc: @cmgrote

I believe you are looking at this first. Please assign back/raise issue with any remaining tasks you need me to look at as part of the migration

Yes - I can do this ... :)

I propose to now remove our ranger/gaian related code from the egeria repo. This includes the ranger security connector, gaian, and docker containers.
They will be recreated in an alternative repo when we are ready to continue working for them, as we can retrieve from history.
Currently they are not functional or used, and are contributing to the cost of build performance, security issues etc


chore(GamutProvider, setupRtl): Updating children prop safety for React 18 usage

Overview
In React 18+ React.FC and similar typings no longer implicitly provide the children prop. Per Dan Abramov, the right thing to do is to "define them manually".
This PR adjusts two widely imported gamut "wrappers", the GamutProvider and setupRtl to be 18+ compliant by changing their types to be explicit about including the children prop. No functional code has been touched.

Update the typing on GamutProvider and setupRtl to explicitly allow the children prop, which is a requirement of React 18+.

PR Checklist

[x] Related to designs: N/A
[x] Related to JIRA ticket: CONPLAT-2194, Update Author to React 18
[ ] I have run this code to verify it works
[x] This PR includes unit tests for the code change (N/A)
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-icons@7.8.1-alpha.56459b.0@codecademy/gamut-illustrations@0.32.1-alpha.56459b.0@codecademy/gamut-kit@0.6.205-alpha.56459b.0@codecademy/gamut-labs@33.5.1-alpha.56459b.0@codecademy/gamut-patterns@0.8.14-alpha.56459b.0@codecademy/gamut-styles@14.0.5-alpha.56459b.0@codecademy/gamut-tests@4.0.13-alpha.56459b.0@codecademy/gamut@50.6.6-alpha.56459b.0@codecademy/markdown-overrides@0.9.11-alpha.56459b.0@codecademy/monaco@0.6.88-alpha.56459b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6328cc8aa2eb0e14c6784661--gamut-preview.netlify.appDeploy Logs


chore: Prep for release 3.10.1

Coverage remained the same at 99.454% when pulling 52475c24d0d38be32efbfa4955f6b40d6604f076 on zeeshan/prep-3.10.1 into 2178ca684f523d76aad30b4e68d41813ff0170a8 on master.

Coverage remained the same at 99.454% when pulling 52475c24d0d38be32efbfa4955f6b40d6604f076 on zeeshan/prep-3.10.1 into 2178ca684f523d76aad30b4e68d41813ff0170a8 on master.


Change default violation level to warning

Problem Statement
Currently it's error which makes builds fail when running via Xcode build script. And there's not even an option like --lenient to downgrade errors to warnings.
Suggested Solution
Just default to the warning level instead of error like now.

Oohps, wrong repo, intedned to be reported on AnyLint. 😅


Laravel Mix Issue

Version 2 seems to break with Laravel Mix. Webpack says its building but fails to launch the browser, and browsersync. Webpack says its watching files and then almost immediately says its done and drops back to terminal prompt.
This breaks:
"devDependencies": {
    "browser-sync": "^2.23.6",
    "browser-sync-webpack-plugin": "^2.0.1",
    "cross-env": "^5.0.5",
    "laravel-mix": "^2.1.7"
  },

This works
  "devDependencies": {
    "browser-sync": "^2.23.6",
    "browser-sync-webpack-plugin": "^1.2.0",
    "cross-env": "^5.0.5",
    "laravel-mix": "^2.1.7"
  },

Any idea what the problem could be? Trouble is, Mix auto installs the most recent version of browser-sync-webpack-plugin so i'm sure I won't be the only person to run into it.
I'm using it standalone, not in a Laravel project.

I can confirm the issue. Just did a npm update today and it stopped working.

@HashandSalt @alesvaupotic what versions of node are you using? I'll do some tests with laravel-mix when I get the chance

Phew! glad its not just us. Running on Node 8.9.1 on a Mac (High Sierra).

Ubuntu 16.04.4 LTS
ales@r35:/var/www/$ node -v
v8.10.0
ales@r35:/var/www/$ npm -v
5.7.1
Not sure about which version was last used, I just did an update as part of my weekly routine.

if it helps with testing theres a simple repo over here that the issue is present in.
It's also affecting Slate, our open source framework, so hoping the issue is resolved soon.

Went back and found out the last working version for me is 2.0.1.

@HashandSalt Alright, I figured node version wasn't the issue but had to check. The repo link you provided is a link to this thread, can you provide the right link? Thanks

Oh whoops, sorry, ive fixed the link.

Thanks, my fix exactly. I hardcoded it also to 2.0.1 and it's been fine since.

@HashandSalt @alesvaupotic I fixed it here https://github.com/Va1/browser-sync-webpack-plugin/pull/67
Just needs to merged.

fix is shipped with version 2.2.2. thanks guyz

Works like a charm. Thanks!


Unable to get the css to style the notifications.

For some reason the notification boxes are not being styled by ember-notify.css. All of the notification boxes are grey with black text, regardless of using .success, .warning, etc.
Any ideas?

It's based on Foundation or Bootstrap styling, so you'll want one of those or roll you own.

Okay, that makes sense, thank you for the quick reply. Could you point me in the right direction to style according to whether it is .success or .warning, etc. I'm new to this stuff so thank you for your patience.

The messages will have CSS classes info, success, warning and alert. You
can use the browser Dev Tools and Inspect element to see what it’s
generating.


Add L5.5 support

Please review

All checks passed! Can you update the repository? So that we can pull the new drivers and the repo? @jenssegers

@sagargulati @jenssegers All checks pass because this just removes lot of assertions removed from tests, see EmbeddedRelationsTest.php. I'm not sure if this really is the PR we're all hoping for.

@ciruvan I only removed events from dispatcher because there is a new "retrieved" event which cause problems with tests. I didn't work with Mockery, but I saw in @RemiCollin PR how it can be handled. I updated code, thanks to RemiCollin

So this is good to go now @MCMatters
Yes, I saw that but a minor big may help ascertain for a final PR @ciruvan

Awesome work dude, thank you.

@MCMatters nice work, man! if that PR is accepted by @jenssegers, I owe you one!

Good job! I've just upgraded to 5.5 and all I'm waiting on is this. Looking forward to it!

Thankyou so much @MCMatters this is the solution for all of us, thankyouu... now we're waiting PR accepted by @jenssegers :)

@jenssegers , when should we expect you will accept the PR by @MCMatters ? :)

Shouldn't someone update README.md as well? :)

@sagargulati where exactly do you have an error?

It's was on belongsToMany. I am using Alpha now and seems it working fine.


Fix path through #define. Remove static from polysoup functions

Amend CMakeLists.txt (largely a cleanup, add highest warning level)
Remove statics from polysoup.cpp. It used to be a self-contained tool but now it is needed in other compilation units -> MSVC complains about this, understandably.
Windows will build using MSVC, not clang.
There are resource files compiled into the executable. It is the image from an old project. Artists are welcome to change this to match our game :)

@NicoB-Coding can you please try it out?

Works on Windows, small bug still there, see appendix

Thank you for checking. I ran this only on my Nvidia RTX2070-Windows10 Box. I don't have the graphical glitches there. @benekoehler had the same issue on his integrated Intel GPU. On a different branch this didn't happen. I am going to check this on my other Windows10 Box with AMD GPU later.


xdg-open doesn't work on GNOME 3.30

(node:30702) UnhandledPromiseRejectionWarning: Error: Exited with code 4
    at ChildProcess.<anonymous> (/home/jente/Documents/git/sfinxdatabase/SfinxDatabaseFrontend/node_modules/opn
/index.js:83:13)
    at Object.onceWrapper (events.js:273:13)
    at ChildProcess.emit (events.js:182:13)
    at maybeClose (internal/child_process.js:962:16)
    at Socket.stream.socket.on (internal/child_process.js:381:11)
    at Socket.emit (events.js:182:13)
    at Pipe._handle.close (net.js:606:12)
(node:30702) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by thr
owing inside of an async function without a catch block, or by rejecting a promise which was not handled with .
catch(). (rejection id: 1)

This appears to be a problem with the xdg-open that is shipped.
On line 537 and below:
    if gvfs-open --help 2>/dev/null 1>&2; then
        gvfs-open "$1"
    else
        gnome-open "$1"
    fi

The problem is that the current version uses neither gvfs-open or gnome-open. Instead it uses gio open.
When replacing line 540 with gio open "$1" everything works fine.

The problem is that gnome-default-applications-properties is not installed on my distro (4.18.8-arch1-1-ARCH) so the shipped xdg-open doesn't recognize gnome3 on this line: which gnome-default-applications-properties > /dev/null 2>&1  || DE="gnome3"

The problem is on this line:
which gnome-default-applications-properties > /dev/null 2>&1  || DE="gnome3"
DE is not properly being set to gnome3 and instead defaults to just 'gnome'

This is a problem with the xdg-open and should be reported to https://cgit.freedesktop.org/xdg/xdg-utils/

@sindresorhus I did figure so much by the end, in any case this repo is still shipping with a non functioning version of xdg-open so it should also be changed at some point when the problem is fixed

Yes, I'll update from https://cgit.freedesktop.org/xdg/xdg-utils/ when it's fixed there.

@sindresorhus Actually it looks like the problem is already fixed.
With this file the problem doesn't happen
https://github.com/freedesktop/xdg-utils/blob/master/scripts/xdg-open.in

Made PR here for updated version of xdg-open
https://github.com/sindresorhus/opn/pull/105


pkb: add scimark2 + its license to README.md's benchmarks

Also fix some formatting inconsistencies for surrounding benchmarks in the list.
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/271 added scimark2 support but
didn't update the README.

Nice catch... thanks for fixing it.


Copy ZucksAdapter from internal project

Copy internal classes to zucks adapter project from internal project.

@h-minamiohtsu Please review this 🙇


QueryDSL fetching set collection in SELECT projection

I have a nested JPA entity structure which looks something like below
@Entity
public class Applicant {

  @OneToMany()
  private Set<Address> addresses;

  // other applicant fields

}

@Entity
public class Address {

  private String houseName;

  private String houseNumber;

  private String postcode;

}

And I have trying to get applicant details with set of address by following query but it returns 'Type Mismatch'
private static final ConstructorExpression<AddressEntity> qAddresses = QAddressEntity.create(
        qAddressEntity.houseName, qAddressEntity.houseNumber, qAddressEntity.postcode);

@Test
public void shouldTestAddressWithProjection() {
    JPAQuery<ApplicationEntity> jpaQuery = new JPAQuery<>(entityManager);
    QueryResults<ApplicantEntity> queryResults = jpaQuery.select(
            Projections.bean(ApplicantEntity.class, qApplicantEntity.forename, qApplicantEntity.surname,
                    GroupBy.set(qAddresses).as("addresses")))
            .from(qApplicantEntity)
            .leftJoin(qApplicantEntity.addresses, qAddressEntity)
            .where(qApplicantEntity.forename.startsWithIgnoreCase("Nikki"))
            .fetchResults();

}

I know of the fact that I can use .transform() and it works with is but I loose the pagination that comes with fetchResults(). Please let me know the solution for this issue.

Hi,
Any updated on this please.

did you find any solution on this ?

GroupBy.set groups in memory, not in SQL, so the count query would return the wrong count anyway. GroupBy expressions can not be used with fetchResults and I would be surprised if they work properly with select() instead of transform() at all.
Also, fetchResults is deprecated for JPAQuery because the count query can't be reliably generated in JPA.

Hi @jwgmeligmeyling , this ticket was closed but I didn't see any feedback on the OP's actual issue of using Projection with set, there is Projection.list but no Projection.set. Is there a way to accomplish this?
Thank you.

The OP tries to use Projections.bean with a nested GroupBy.set in a selection using fetchResults(). GroupBy expressions are not supported as selection, only as result transformation. And by definition cannot be used together with fetchResults().
The OP doesn't mention Projections.list at all nor does he mention Projections.set - which indeed does not exists.

Hi @jwgmeligmeyling, thank you for your response. I may be wrong but the OP might have tried the Groupby.set because he's trying to do a projection for the OneToMany field "addresses" which is a set and not a list. It's also the name of this issue "fetching set collection in SELECT projection". So the groupby/transform is not the intention but just something tried because the way to achieve this is not clear. Well at least for me, I have a similar setup and facing a similar issue because there is no Projections.set, I tried Projections.list but as expected failed.
How would one do a bean or field projection to add the set of addresses?
Thanks in advance

@garfieldcoked were you able to find solution?

Having the same problem , are there any solution for this??


Testing in linux varients

Need to test and report issues on each and every Linux distros

@kUSHAL0601  Can you update the status here ?

SecureTea has been tested on the following linux variants :

Ubuntu 16.04
Ubuntu 20.04
CentOS 7
Fedora 32
Kali linux 2016
ParrotSec 4.2

The testing includes installation and unit testing

Awesome. Can we put a table in the Readme stating the status and tested OSs @kUSHAL0601

Sure will do! Although could you clarify on the status part. Should I put error/no error and put some other OSs which we should probably test on ?

Passed, Partial, Need  improvement as status also you can put any specific comments too

Hi, is this issue still open? I have a Debian Buster VM?
Any ideas on how to go about it?

Hi, is this issue still open? I have a Debian Buster VM?
Any ideas on how to go about it?

Hi, is this issue still open? I have a Debian Buster VM?
Any ideas on how to go about it?

Still up. You can update the test results

Hi, is this issue still open? I have a Debian Buster VM?
Any ideas on how to go about it?

Still up. You can update the test results

Hi! I tried installing SecureTea on Debian as well as Kali and am facing some errors. I guess it's because of one of the dependencies NetFilterQueue to be exact. I tried manually building it from source.

Hi! I tried installing SecureTea on Debian as well as Kali and am facing some errors. I guess it's because of one of the dependencies NetFilterQueue to be exact. I tried manually building it from source.

Hi. Did you try using the bash script? It should help you with installation issues in general!

Hi. Did you try using the bash script? It should help you with installation issues in general!

Yes I did run it, but I encountered this netfilterqueue at that time only.
That is why I tried building from source

Yes I did run it, but I encountered this netfilterqueue at that time only.
That is why I tried building from source

NetfilterQueue issue can be fixed in this way https://github.com/OWASP/SecureTea-Project/pull/242. This can be closed.


D2GAME_PACKETS_SendPacket0xA0_A1_A2_6FC3D610 Uses Incorrect value for 32bit Sends

https://github.com/ThePhrozenKeep/D2MOO/blob/e4306096435ec2af96ac36df07c1001f77698a88/source/D2Game/src/GAME/SCmd.cpp#L801
As per the assembly, the packet should be storing the full (new) value and not the difference when taking the 32bit path:
6FC3D70C    MOV EDX,DWORD PTR DS:[EDX+C]
6FC3D70F    MOV DWORD PTR SS:[ESP+E],EDX
6FC3D713    PUSH 0A
6FC3D715    LEA EDX,DWORD PTR SS:[ESP+10]
6FC3D719    MOV ECX,EDI
6FC3D71B    MOV DWORD PTR SS:[ESP+16],ESI                                ;  nNewValue
6FC3D71F    CALL <D2Game.D2QueueSendPacket>

Waiting for @szumigajowy's packet PR before fixing this


Getting Article Banners to show up in meta tags

I want to be able to make the article banner image become the image for og:image and twitter:image meta tags. Unfortunately, it seems like the original source location of the image gets lost when trying to modify the Article index.tsx, even if I wanted to use Helmet directly.
Any ideas on how to support this?

Hey @Surg-Dev! Modifying the Article/index.tsx seems like the right way to do it. Did you try modifying the SEO component too and passing the image url as props? It may be that the SEO component overrides your og:image you set inside the Article template.
Anyways, I like the idea and will add this to the core theme as soon as I have time for it. Thanks for opening this up!

I'm not super versed in "best practice" when it comes to webdev, (very systems guy). Passing it into SEO probably is the more modular answer, especially if I make other pages that want that meta tag made. Right now, this is the solution I'm using in Article/index.tsx...
    const article = props.pageContext.article;
    const image = article.banner?.src?.childImageSharp?.gatsbyImageData?.images?.fallback?.src;
    return (
        <>
            <Seo title={article.title} description={article.description || undefined} useTitleTemplate={true} />
            <Helmet>
                <meta property="og:image" content={image} />
                <meta property="twitter:image" content={image} />
                <script src="https://polyfill.io/v3/polyfill.min.js?features=es6"></script>
                <script id="MathJax-script" async<EMAIL_ADDRESS>            </Helmet>
            <Page>

I'm not entirely "comfortable" with how nested the actual image is in the Gatsby Image Wrapper, but it seems to work?

It's correct that the image src is such deeply nested. The actual image gets processed by gatsby-image during the build and it spits out the gatsbyImageData object.
Great to hear that you found a working solution. I leave this issue open until I added proper support for this in the core theme 👍


Underline text bugged

Type: Bug
Underline text is bugged since last update. The line appears on the text instead of under.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.88.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Still happening on the latest release. Looks similar to https://github.com/microsoft/vscode/issues/209614

Still happening on the latest release. Looks similar to #209614

Yes, looks very similar. What theme are you using? Does the problem remain if you switch to a default theme?

Closing this as a duplicate of #209614 in favor of keeping the relevant information central to that issue.
/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!