Is there way to get history data in initialize function ?

Hello.
I'm trying to build trading algo with machine learning and face to problem to get history data before simulation for build ML model.
Could be point me on some example or function to get data history from initialize() function or out simulation ?

@alifanov I would use before_trading_start(context, data)
def before_trading_start(context, data):
	# this is being called daily at 0:00 UTC
        ml_model.run()
	return

if __name__ == '__main__':
          run_algorithm(
                                  before_trading_start = before_trading_start,
				  live = True if cmx_config['global_mode'] in ['live', 'paper'] else False,
				  live_graph = False,
				  analyze_live = None, # pass a function
				  simulate_orders = False if cmx_config['global_mode'] == 'live' else True,
				  capital_base = cmx_config['risk_max_notional'],
				  data_frequency = 'minute' if cmx_config['signal_candle_size'] == '1T' else 'daily',
				  data = None,
				  bundle = None,
				  bundle_timestamp = None,
				  default_extension = True,
				  extensions = (),
				  strict_extensions = True,
				  environ = os.environ,
				  # initialize = (initialize, cmx_config),
				  initialize = initialize,

				  handle_data = handle_data,
				  analyze = analyze,
				  exchange_name = cmx_config['exchange'],
				  algo_namespace = cmx_config['global_namespace'],
				  quote_currency = cmx_config['risk_quote_currency'],
				  start = cmx_config['time_start'],
				  end = cmx_config['time_end'],
				  output = cmx_config['record_pickle'],
				  auth_aliases = None, # For example: 'binance,auth2,bittrex,auth2'
				 )

another way is to train your model inside handle_date() while keep your trading / execution disabled till training is done.

@FWangTrading before_trading_start() call every day. I want the only one time call of function.
Right now I prepare the model by other script with using ccxt to get data but it is not convenient

@alifanov how about adding a one time flag in  your initialize(), then
def before_trading_start(context, data):
	# this is being called daily at 0:00 UTC
        hist_data = data.history(context.asset,
                                 'price',
                                 bar_count = int(window_info['barcount']),
                                 frequency = window_info['barsize'],
                                )
        if self.already_trained:
                return
        ml_model.run(hist_data)
	return

@FWangTrading, thanks for the suggested solution.
I am closing this, @alifanov, feel free to reopen if you feel otherwise.


Feature/register quay no auto builds

This fixes issues #19 and #120 , also adds tests for them.
For the case of #19, you can manually add quay containers with no autobuilds and they will work fine, but if they are quickregistered/auto you will need to use the command line updateContainer to manually set the git url.  Currently this can not be done through the UI (#117)

:+1:  after the explanation


fix(common): adding support for binary files in configMaps from files

Description of the change

configMapsFromFolderBasePath now supports binary files
Removed

Fixed

Added

configMapsFromFolderBasePath can now add binary files (like images) to your generated ConfigMaps by adding .binary to the file name

Changed

Modified the configMap class to support data, binaryData or both

Benefits

Possible drawbacks

Since now the data key in the configMap class is optional there's a scenario where a generated ConfigMap might not have this key. I can add a check under "common.lib.configMap.validate" to force that at least one of data or binaryData need to be defined

Applicable issues

fixes #

Additional information

Checklist

[X] Title of the PR conforms to the Conventional Commits standard.
[X] Scope of the of the PR title contains the chart name.
[X] Chart version in Chart.yaml has been bumped according to Semantic Versioning.
[X] Chart artifacthub.io/changes changelog annotation has been updated in Chart.yaml. See Artifact Hub documentation for more info.
[X] Variables have been documented in the values.yaml file.

Thanks again for adding this! I'll go ahead and merge it, and will address the "issue" of field optionality in the json schema


Update readme to reflect reduced size

Update readme.md to reflect changes made by @Frago9876543210 -  #2
Verify binary size

Addressed in 59a8d04efdea8b65aae8c7eb01f5961b54052a11, ba802770defc3655a6b3feafb53846d1dbba9b99, 07b44bf31f776f47aad8ad4336f455af1b3622b7.  Nice work @Frago9876543210 😁

Addressed in 59a8d04efdea8b65aae8c7eb01f5961b54052a11, ba802770defc3655a6b3feafb53846d1dbba9b99, 07b44bf31f776f47aad8ad4336f455af1b3622b7.  Nice work @Frago9876543210 😁


Publish live test results

Live tests should generate and publish results

This work was part of the matrix discussion but this issue separates this work from the matrix work so we can keep moving forward.


feat: Implement o200k_base encoding and support gpt-4o

Implement o200k_base encoding and support gpt-4o

Hello @tox-p  thanks for all the support on maintaining this repo. Is there anything we can do to help you with this PR? we're relying on this library in our project and righ now we're looking to start using gpt4o but it seems the library doesn't have support yet for gpt4 o. please let us know if there is something we can do to help with this.

Checking in on this approval

@tox-p Sorry to annoy you directly but any plan to review this PR?

Hey, @tox-p Can you provide update on when are you planning to review the PR? I need gpt-4o encoding ASAP. Thanks!

meanwhile :zipper_mouth_face:
<dependency>
    <groupId>net.optionfactory</groupId>
    <artifactId>jtokkit</artifactId>
    <version>1.1.1</version>
</dependency>

net.optionfactory

I can't find this on maven central? Can you help me?

meanwhile 🤐
<dependency>
    <groupId>net.optionfactory</groupId>
    <artifactId>jtokkit</artifactId>
    <version>1.1.1</version>
</dependency>

meanwhile 🤐
<dependency>
    <groupId>net.optionfactory</groupId>
    <artifactId>jtokkit</artifactId>
    <version>1.1.1</version>
</dependency>

I can't find this on maven central. Can you help me?

meanwhile 🤐
<dependency>
    <groupId>net.optionfactory</groupId>
    <artifactId>jtokkit</artifactId>
    <version>1.1.1</version>
</dependency>

I can't find this on maven central. Can you help me?

Strange, I've released this morning and I see in central portal: https://central.sonatype.com/artifact/net.optionfactory/jtokkit

meanwhile 🤐
<dependency>
    <groupId>net.optionfactory</groupId>
    <artifactId>jtokkit</artifactId>
    <version>1.1.1</version>
</dependency>

I can't find this on maven central. Can you help me?

Strange, I've released this morning and I see in central portal: https://central.sonatype.com/artifact/net.optionfactory/jtokkit

Not sure what dependency that is but not the one from this Github repo, this is the correct maven repo: https://mvnrepository.com/artifact/com.knuddels/jtokkit

@mkemmerz of course that dependency has different groupId. It came from a fork with merged support for GPT-4o and GPT-4-turbo models. Nothing "strange", I've need of this functionality for some customers, so I've published myself. Here the fork https://github.com/optionfactory/jtokkit and thanks for the peole who made the PR.

Any progress made to merge it to the main repo?

Hey, @tox-p there is a plan to review this PR soon? we really need this encoding.
Thanks

Thanks 👍


term of Crash consistency recovery point

When we access "Create replication policy" in Azure Portal, there is information below.

Crash consistency recovery point will be generated every 5 minute and pruned after an hour. Learn more

But this page says,

Site Recovery creates a crash-consistent recovery point every 5 minutes. You can't change this frequency. For the most recent 2 hours, you can choose from 24 crash-consistent points and 2 app-consistent points.

Which is correct 1 hour or 2 hours?

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

ID: 7a8f2ca3-19b1-330d-8dd3-6dcdce38c8d4
Version Independent ID: 700335aa-c802-0314-1579-3493918f5000
Content: General questions about the Azure Site Recovery service
Content Source: articles/site-recovery/site-recovery-faq.yml
Service: site-recovery
GitHub Login: @atikmapari

@NakayamaKento
Thanks for your feedback! We will investigate and update as appropriate.

Both the statements are correct.
Pruning of crash consistent recovery points happens every hour. In this process, we will remove older recovery points. This is basically a process which is run at a frequency of 1 hour and the intent is to remove the recovery points which are past their retention point. In this cycle, we will also remove recovery points which are beyond the last 2 hours.
Customers will have 24 recovery points for last 2 hours and then 1 recovery point per hour beyond the last 2 hours.

#please-close


Fix DRF permissions and nested elements

Domyśle DjangoModelPermissions mapuej tak permissiony
class DjangoModelPermissions(BasePermission):
    """
    The request is authenticated using `django.contrib.auth` permissions.
    See: https://docs.djangoproject.com/en/dev/topics/auth/#permissions

    It ensures that the user is authenticated, and has the appropriate
    `add`/`change`/`delete` permissions on the model.

    This permission can only be applied against view classes that
    provide a `.queryset` attribute.
    """

    # Map methods into required permission codes.
    # Override this if you need to also provide 'view' permissions,
    # or if you want to provide custom permission codes.
    perms_map = {
        'GET': [],
        'OPTIONS': [],
        'HEAD': [],
        'POST': ['%(app_label)s.add_%(model_name)s'],
        'PUT': ['%(app_label)s.change_%(model_name)s'],
        'PATCH': ['%(app_label)s.change_%(model_name)s'],
        'DELETE': ['%(app_label)s.delete_%(model_name)s'],
    }
    ```

    Trzeba nadpisać dla GET aby korzystało z read permission


Fix typos in README.md

This pull request fixes some simple typos that I found while reading README.md.

Hi @daschult, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Dan Schulte). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Thanks!


shinyapps.io

Hey @sckott,
In the blog post I did for ropensci there is an iframe pointing to shinyapps.io. I don't plan on paying for an account, but I have my own server, so you could use this link --
http://<IP_ADDRESS>:3838/LDAelife/
Just FYI, the source for this app is located here

Good catch. We recently changed to https from http on ropensci.org, so anything thats served over http doesn't work, see devtools in chrome/firefox on https://ropensci.org/blog/2014/04/16/topic-modeling-in-R/ for the error message.
Would work if you could put https on your server, but I imagine that's too much to ask.  One solution we could do is a screenshot with a link out to the interactive shiny app. Sound okay?

Sounds good, that'll work for now, I'll have a look at switching to https (although, to be honest, I have no idea how complicated that would be)

we just did the https thing, you gotta pay for a ssl certificate, we can help if you want to
Okay, made the change on your blog post, does it look okay? https://ropensci.org/blog/2014/04/16/topic-modeling-in-R/

Looks great, thanks!

Of course, as soon as I try to work around the new restrictions, rstudio offers an advocate program...
Since this app is listed in the shiny user showcase, they are willing to host it for free (without restrictions), so feel free to go back to an iframe pointing to -- https://gallery.shinyapps.io/LDAelife/
Sorry for the inconvenience @sckott

@cpsievert okay, will change it back, no problem


Update metadata forms layout

Closes #938
Streamlines some of the layout issues in the metadata forms.

[x] Requires an updated schemaservice Docker image

[x] Issue a Github release of modified schema service
[x] Update pass-docker and pass-ember docker compose configs to use newly released schema service image

PR is up for pass-docker update. The change is already present in this PR for pass-ember


Fix failing build by removing bin/release

Builds fail, because bin/release doesn't emit valid yaml.
bin/release is useless, since addons and default_process_types are not needed for this buildpack.

💥  Merging untested. 💥

That was fast! The fix works for me...

I can merge pull requests without testing all day, yo.

We all can, but we usually regret it later :D


List view doesn't rendering anything

I am using SharePoint Framework React controls's list view to render the items from document library. I am able to get the items from library by selecting the list from dropdown. But While rendering the items, it is not showing any items in the view. But, when i click on item I am able to write data in that row in console. It is rendering like below:

I am using ListView like below
<ListView
            items={this.state.listData}
            iconFieldName="ServerRelativeUrl"
            compact={true}
            selectionMode={SelectionMode.multiple}
            selection={this._getSelection}
             />

[X] Question

Hello,
I have similar issue where I defined the ViewFields:
export const  viewFields : IViewField []= [{
name: "Car",
displayName: "Car",
//linkPropertyName: "c",
isResizable: true,
sorting: true,
minWidth: 0,
maxWidth: 150,
render: (item:any) => {item.Car['Car']}
},{
name: "DistanceKm",
displayName: "Distance",
isResizable: true,
sorting: true,
minWidth: 0,
maxWidth: 100
},];
But the Car field is not rendered and I get error. Each other field is rendered as it should. I tried every combinations like item.Car.Car, item['Car'].Car etc. I receive the items from Sharepoint as following:

The error that i got is:

Please help.
Thanks.


zblacklist does not handle lines longer than 999 characters correctly

See https://github.com/zmap/zmap/blob/master/src/zblacklist.c#L181
If zblacklist encounters a line longer than 999 characters it will:

Process the first 999 characters as a single input line
If that line is to be outputted, will not output a newline character for this partial line (relies on the input new line, which has not yet been consumed)
Process all subsequent characters as a new input line (up to 999 characters again)

Depending on circumstances and https://github.com/zmap/zmap/issues/381 this may result in erroneous output. Example (character Y appears at position 1001):

$ cat test_input
<IP_ADDRESS>
<IP_ADDRESS>#XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXYYYY
<IP_ADDRESS>
$ cat test_input | zblacklist -b /etc/zmap/blacklist.conf --no-duplicate-checking
<IP_ADDRESS>
<IP_ADDRESS>#XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXYYYY
<IP_ADDRESS>
$ cat test_input | zblacklist -b /etc/zmap/blacklist.conf
<IP_ADDRESS>
<IP_ADDRESS>#XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX<IP_ADDRESS>

With verbose flags, you see:

Oct 20 19:27:19.926 [DEBUG] zblacklist: input value XYYYY
Oct 20 19:27:19.926 [WARN] zblacklist: invalid input address: XYYYY

Personally I'd like to see lines with long code comments supported.  Thoughts?

If we don't set a limit, we would have to do some thing like infinitely read in characters and doubling our array size in memory forever. This doesn't sound great. Otherwise, fgets  or something custom is going to to want a defined buffer of some length. As such, I'd just say let's up the length to be something much higher. I'm not really sure what that was. I didn't think people would be passing in 1kb lines. My gut sense would be to up this to 100KB and move on.

I'm also cool with FATAL'ing when we hit something > 100kB.

Why can't we just read 1000 characters, output 1000 characters, read 1000 characters, output 1000 characters, and so on? We only need the first INETADDR_MAX characters to do duplicate checking based on IPs.

@dadrian We can, and yeah, it would work, but at the same time..... just upping and constant and moving on seems about how much work this issue truly deserves.

Does this solve all of the immediate problems? https://github.com/zmap/zmap/pull/384

@zakird Yes that PR solves the intermediate problems.
@dadrian that causes issues as the follow-on reads are invalid entries. How do you handle those? Right now it's buggy. Depending on the answer to #381 it may not work right. This is also slightly awkward because of how you rely on \n at the moment. I think Zakir's fix is the right one.

This has been solved by #384.


There is no command to "Import an existing project"

[Enter feedback here]
As you can see in the image posted below, there are only two commands to choose from.

If I select the Open Microsoft Teams Toolkit command, the "Existing Bots" section just spins.

Can you help with these issues?
Thanks in advance!

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

ID: c5e07135-25cc-d865-2cad-b3dc6f22f750
Version Independent ID: b1c26476-0b6a-7f6c-f64a-abd1d9ae8d21
Content: Build apps with the Microsoft Teams Toolkit and Visual Studio Code - Teams
Content Source: msteams-platform/toolkit/visual-studio-code-overview.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Hi @faceoffers28,
We are unable to reproduce the issue from our end. We can able to see "Import app package" and able to select BOT.

Could you please take latest version of "Microsoft Teams Toolkit" and try again.

Hi @RamaMohanaChary. Thanks for the quick response. As you can see from my screen shot, I'm using version 1.1.8 and the command is simply not there. Do you have to logon to see the Import app package command? Speaking of login, I assume we are logging into the tenant that contain the App Service and Bot Channel Registration. Is this correct? What happens if that tenant doesn't have an Office 365 Teams license?

Did you enable the developer preview mode? Could you please check Prerequisites to set up Teams toolkit? try if that helps


crystal failed to compile on OSX(10.11 El Capitan) (version: 0.9.0)

Recently got crystal upgraded via homebrew from 0.8.x to 0.9.0. Earlier everything was working fine, but suddenly with this upgrade, crystal started to fail to compile with error
crystal eval 'puts "Hello world!"'
ld: library not found for -levent
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Error: execution of command failed with code: 1: `cc -o "/Users/konflikt/.crystal/crystal-run-eval.tmp" "${@}"  -rdynamic  -levent -lpcl -lpcre -lgc -lpthread -ldl`

All the required crystal dependencies are install
brew install \
  bdw-gc \
  gmp \
  libevent \
  libpcl \
  libxml2 \
  libyaml \
  llvm
Warning: bdw-gc-7.4.2 already installed
Warning: gmp-6.0.0a already installed
Warning: libevent-2.0.22 already installed
Warning: libpcl-1.12 already installed
Warning: libxml2-2.9.2 already installed
Warning: libyaml-0.1.6_1 already installed
Warning: llvm-3.6.2 already installed

Seems like an issue with current upgrade.
System: Mac OSX (10.11 El Capitan), Crystal: 0.9.0, LLVM: 3.6.2

export LIBRARY_PATH="$LIBRARY_PATH:/usr/local/lib"
this fixed for me, but if there are problems check http://stackoverflow.com/questions/3146274/is-it-ok-to-use-dyld-library-path-on-mac-os-x-and-whats-the-dynamic-library-s

@TBD yes, thanks.
This solve the issue for me too. But that should be handled internally. no?

Not really, this is a homebrew configuration. It could be installed in /opt instead of /usr/local/lib for example.

I also had this problem after upgrading to 10.11 but I solved it reinstalling the command line tools: xcode-select --install and selecting as the default active toolchain: xcode-select --switch /Library/Developer/CommandLineTools
Seems like XCode doesn't look by default for libraries at /usr/local/lib anymore. My understanding is that Homebrew should at least warn about this when running brew doctor.

@conflikt I just upgraded to El Capitan, had the same issue, did what @waj suggested and I'm having everything working again. Can you confirm this fixed it for you too?

Same problem (levent not found despite having deps installed).  Same situation (upgraded to 10.11).  @waj fix worked for me.  Thanks!  :cake:

@asterite, I already mentioned the same thing worked for me too in my last comment conflikt commented on Oct 22, 2015. I guess you missed that :smiley:

@asterite Maybe this should be mentioned in the Mac OS install docs. I was having the same problem and was lucky enough to find this Github issue..

Yeah, it's just so frequent that I think it's worth it. Just did so: http://crystal-lang.org/docs/installation/on_mac_osx_using_homebrew.html

:+1:


use deprecated available slots

CHANGES:
Swap endpoints from /api/slots to /api/available_slots

@StupidCodeFactory before deploying this we need to merge and deploy the pvb2 side


how can i build for qnx

Hello, thanks for opening the obs sdk code, we want to run the sdk on the qnx 710 platform, do you have any experience in adaptation?

在Cmakelist中设置交叉编译工具


Add new var: app_config_entries

This allows use of the new related functionality in
terraform-azurerm-caf, which allows for adding entries to an azure
appconfig instance in a more flexible way.
See, https://github.com/aztfmod/terraform-azurerm-caf/pull/1221

Upstream PR: https://github.com/Azure/caf-terraform-landingzones/pull/385


定时关闭增加播放完当前歌曲后生效按钮

仅仅提交一个建议。Submit just one proposal.
在你提交此建议前请注意：Please note before you submit this proposal:

详细阅读椒盐音乐帮助内容（应用设置-帮助）。Read more about the SaltPlayer help content (app Settings - Help).

反馈时候请注意：When giving feedback, please note:

尽量详细的描述。Be as detailed as possible.
不接收任何 UI 样式调整相关的反馈，出现此类 issue 会被关闭（如 “我不喜欢此设计” 、“我觉得这样设计更好” 、“还没原来好看” 等等类似反馈）。Do not receive any feedback on UI style changes, and the issue will be closed (" I don't like this design ", "I think this design is better", "it's not as good as the original", etc.).

建议内容 The proposals
<在此输入你的建议 Enter your suggestions here>

8.4.0-beta01


Creating a Mastodon account

Given the recent news and the philosophy of Mastodon. We discussed the possibility of creating a Mastodon account for scikit-learn with @ogrisel. What do you think about it?

If we do, we will have to choose which server to use. Each server might have different moderation rules and hosting stability guarantees.

The Fosstodon server could be a suitable one. It is one recommended by Mastodon as "consistently committed to moderation against racism, sexism, and transphobia" and is dedicated to open source software.

Yes the fosstodon server sounds good.
The details about the server admin:

https://fosstodon.org/about/more

Note that it's an English-only server but this is fine for the scikit-learn account I think.

A bit of background of the load increase on fosstodon:

https://hub.fosstodon.org/elon-twitter-post-mortem/

After the upgrade, it seems to be operating well again. If we locate the scikit-learn account there we should try to see if we could do a small regular donation to help with the hosting fees (e.g. from the numfocus account?).

I created an account, waiting to be reviewed by Fosstodon maintainers.

It is accessible on here: https://fosstodon.org/@sklearn

It is accessible on here: @.***

Awesome!! Great job!


fix: replace kube-rbac-proxy image tag v0.8.0 with its digest equivalent (#510)

What type of PR is this?
/kind bug
What does this PR do / why we need it:
This PR fixes installation in environments that require images to be mirrored by digest rather than tag, such as openshift in a "restricted" network install.
Note that this PR includes modifying older CSV versions.  I'm not sure that's kosher.  If not, please free to leave out that part, or kick it back to me for fixing.
Have you updated the necessary documentation?
N/A
Which issue(s) this PR fixes:
Fixes #510
How to test changes / Special notes to the reviewer:
Issue #510 includes a description of how to test that the actual problem is fixed.
That said, it's a lot of work. It should also be possible to upgrade an existing installation, and just look at the output of
ns=argocd; # or whatever
kubectl -n $ns get po -l control-plane=controller-manager;
kubectl -n $ns get po -l control-plane=controller-manager -o json | jq '.items[].spec.containers[0].image' -r

If the pod is running and has a kube-rbac-proxy image specified by digest instead of tag, you know it's working.

Hi @mortya thanks for your PR!
I think it would be best to leave this change out of the released version bundles
would you mind restricting this to master and the 0.7.0 bundle?

Hi, @jaideepr97 !  I do not at all mind modifying the PR to only apply to new cluster service versions.  Done.  Not sure what the fashion is here for correcting PRs; did it as an amend and a force push rather than a new commit.  Hope that's OK!


feat: add sns producer and consumer annotation

Description
Adds the VisualizeSnsProducer and VisualizsSqsViaSnsConsumer annotation to mark methods sending messages to SNS. SNS messages are always consumed via SQS and we want to be able to style this relationship in a different way as an ordinary SQS consumer.
Fixes missing annotations on the other annotations, e.g. @Retention(RetentionPolicy.RUNTIME).
Checklist

[x] My code follows the style guidelines of the project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation

:tada: This PR is included in version 2.8.0 :tada:
The release is available on:

2.8.0
GitHub release

Your semantic-release bot :package::rocket:


Allocate AVPacket with av_packet_alloc()

PSA: The Great Deprecated API Removal landed in git HEAD a little while back, after 4.4 was tagged. Among the big changes are removal of the old decoding API and removal of AVPacket from the ABI (needs to be on the heap, allocated with av_packet_alloc() now). Test your codebases!

https://twitter.com/FFmpeg/status/1389618769262174209

    From FFmpeg/doc/APIchanges:

        2021-03-17 - f7db77bd87 - lavc 58.133.100 - codec.h
          Deprecated av_init_packet(). Once removed, sizeof(AVPacket) will
          no longer be a part of the public ABI.

Fixed by:

e8b053ad2f1da25f2b0e06d2874cabb12ff44e01
318b6a572e0e866554c5aa5a2ed8c4d0d774a5dc
4af317d40da70d63dee44bf645694ed5c4f05d65
cd2894570d224d854d6e27abb401434fa6ec2840


Version Check

Hello Developer,
Your code is work perfect but play store recently change in code like "div[itemprop" & "softwareversion" tag is remove or change so now code stop working.
Thanks

@solankiranvir absolutely right, did you know any alternative library for this..?

@solankiranvir absolutely right, did you know any alternative library for this..?

Have you found any?


future dxvk setup_dxvk.sh script removal

Issue description
In future releases of dxvk the setup script won't be included anymore so this issue is just to notify so something can be figured out for how the nix package should handle it beforehand. (Hope such an issue is okay)
See https://github.com/doitsujin/dxvk/pull/3097

CC @reckenrode

Thanks for the heads up. The DLLs are already available via a bin output, but this will break anyone’s set up that relies on setup_dxvk.sh. A short term solution would be to vendor the script. Longer term, it would be ideal if there were a way to declaratively create and manage a Wine prefix.

2.1 has been released with the script removed 🙂

I saw that. It can also build natively on Linux, which is interesting. I need to think about how I want to do things and maybe see what other distributions that package it are doing.

God speed 🙏

There is a nix-gaming repository that contains such derivation: https://github.com/fufexan/nix-gaming/blob/master/pkgs/wineprefix-preparer/default.nix
It also handles vkd3d-proton: https://github.com/fufexan/nix-gaming/blob/master/pkgs/vkd3d-proton/default.nix

One approach I’m considering is splitting dxvk into dxvk_1_10 (for Darwin and hardware no longer supported by 2.0) and dxvk_2_x (or just dxvk_2, not sure). The plain DXVK derivation could continue to provide the script (most likely copied from before upstream removed it) while the versioned ones would be built for the native platform only (Linux and Windows).
Well, I guess if it builds on Darwin, I can set the platform to unix then mark it broken on Darwin. I doubt DXVK 2.0 will work on Darwin any time soon. There are a number of extensions DXVK needs that aren’t implemented yet in MoltenVK, and since DXVK is always adding and using new ones, it’s a moving target.

I created #213026 to add the script and bump DXVK. It also exposes native builds on Linux.


support inject volume provider value during install controller

What this PR does / why we need it:
Fix regression after alicloud webhook is introduced.
For private deployment on Alicloud, Soild cluster uses k8s cluster provided by AliCloud. This cluster used FlexVolume instead of CSI. We place a value in annotation with name persistentvolume.garden.sapcloud.io/provider to differentiate.
The intree code was in https://github.com/gardener/gardener/blob/0.25.0/pkg/operation/cloudbotanist/alicloudbotanist/controlplane.go#L138
The `ControllerInstallation` controller does now inject the volume provider name into the Helm charts for the respective seed if it was annotated with `persistentvolume.garden.sapcloud.io/provider=<name>`. The path of the value is `.gardener.seed.volumeProvider`.

/cc @stoyanr, we missed this during the migration


Subject Object Extraction

HI
Hope all is well
So lately I have been playing around with the spacy model and python code written by https://nicschrading.com/project/Intro-to-NLP-with-spaCy/
In the section for subject verb object extraction I keep getting an error no module named subj_obj_extraction found
Is there an alternative way to implementing Subject-Verb-Object extraction within Spacy?

The module subj_obj_extraction isn't a part of the spaCy package, it's written by Schrading himself, and can be found here: https://github.com/NSchrading/intro-spacy-nlp/blob/master/subject_object_extraction.py


left_sidebar: Add Alerts View to the left sidebar.

This PR adds the Alerts View to the left sidebar.
Commits:

c9956362014525329a3449db55aa089a954fb224: Preparatory commit.
2e283f552973aa430457cff70a198a04cda82aaf: Adds the Alerts view to the left sidebar both in condensed and non condensed view (without unread counts).
e8be6137c45d0266e4011ff405e400b95b95de29: Adds the narrow banner, and message_view_header to the Alerts View.
b307a8b15aa5ab61c0c1b7b4f50cd02c6d155b04: Adds the unread count to the Alerts View.

Fixes: #28778

Screenshots and screen captures:

When alert words and corresponding messages exist.

When alert words exist but corresponding messages don't

Views when alert words don't exist.

Opening of settings from 3 dot menu.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I guess the api changes are so minimal and obvious (similar to mentions)... does this still require an api design thread?

Yes, please start one.

API Design Thread

@alya please do let me know when the icon for this View is confirmed. I'll then rebase and update the pr.

Sure, but I don't think the icon is a blocker for proceeding here. Will the PR be ready for review once you've resolved conflicts, or is there more you need to do?

yup, the icon was the only thing left...I guess I'll rebase the pr then.


Add a __pre_init function to be called at the start of the reset handler

I needed this for a project so I went ahead and rebased and tweaked @jcsoo 's changes from #71. I will admit, I got a little impatient waiting and that also played into why I did this. If either @jcsoo or @japaric have an issue with this PR, please let me know and I will remove it. I apologize for toes that I have stepped on.
Now onto the PR. This is possibly the simplest implementation that is possible. No nightly features were used in this implementation. Also, there is no hand-holding for the end user; if they want to mess with uninitialized statics, that is on them.
If you would like me to add more documentation, please let me know what you would like to see.
Fixes #17

Hi @japaric, could you give this a look? Cf. #17

@japaric Pinging to make sure this didn't get lost in the shuffle.

@yodaldevoid thanks for changes. I was looking at the disassembly and it seems that the (LLVM?) optimization passes assume that all function pointers can't be null so __pre_init is always called. We'll have to choose a different sentinel value.
Could you also add an example for this to he examples directory?

it would be good if someone can test that this doesn't break existing code when pre_init! isn't used.

The examples are compiled as part of the test suite so you can be sure that no accepted change will break linking those examples.

@yodaldevoid:
I did some testing, setting the sentinel value to 1 instead of 0 fixes the problem and the comparing instructions are added.
Before (0):
08000424 <Reset>:
 8000424:	f000 f840 	bl	80004a8 <____pre_init_veneer>

New (1):
08000424 <Reset>:
 8000424:	f240 0001 	movw	r0, #1
 8000428:	f2c0 0000 	movt	r0, #0
 800042c:	2801      	cmp	r0, #1
 800042e:	bf18      	it	ne
 8000430:	f000 f83e 	blne	80004b0 <____pre_init_veneer>

With pre_init! (looks strange due to some #[inline(never)] sprinkling):
0800042e <Reset>:
 800042e:	f240 4011 	movw	r0, #1041	; 0x411
 8000432:	f6c0 0000 	movt	r0, #2048	; 0x800
 8000436:	2801      	cmp	r0, #1
 8000438:	d135      	bne.n	80004a6 <Reset+0x78>

...

 80004a6:	f7ff ffb3 	bl	8000410 <__pre_init>

...

08000400 <_ZN20test7my_init17hd14b71b0b4782276E>:
 8000400:	be00      	bkpt	0x0000

...

08000410 <__pre_init>:
 8000410:	f7ff fff6 	bl	8000400 <_ZN20test7my_init17hd14b71b0b4782276E>

Could you fix this and add the example @japaric asked about? I'd like to get this PR moving. :)
@japaric:

A more general approach would be implement .init_array then any crate would able to inject code that will be run before main but I just checked and that requires the unstable #[used] attribute on the caller site or the thing doesn't work when optimizations are applied. (OTOH, letting any crate run code before main sounds hard to reason about.

Indeed, I would not do this. When running code before the init of RAM it is commonly of paramount importance to know the order of operations, so merging it into .init_array can cause unexpected behavior, together with .init_array being performed after RAM init. I would keep it as a single pre_init function as it is here.

@japaric

The examples are compiled as part of the test suite so you can be sure that no accepted change will break linking those examples.

In terms of people checking if this was broken or not, I was asking for someone to do the exact check that you did. Checking the disassembled binary completely slipped my mind.
I will be sure to add an example.
@korken89

I did some testing, setting the sentinel value to 1 instead of 0 fixes the problem and the comparing instructions are added.

I figured this would work, but some problems came up this week that kept me from working on anything this week. I will try to get everything taken care of either tonight or tomorrow.

@japaric I have added an example and changed the sentinel value to 1. Looking at the disassembly it seems to be working. The branch is optimized out when a function is provided, though not when none is provided.

Please could you add the pre_init example to ci/script.sh?

The branch is optimized out when a function is provided, though not when none is provided.

I'm not seeing this optimisation kick in. If I build the pre_init and alignment examples in --release I still get the check and branch right at the start:
pre_init example, showing branch to _stext which contains optimised __pre_init:

alignment example, as per master, showing no comparison or branch:

alignment example, with this PR, showing we still load and compare:

Is this the expected behaviour? Can we optimise out this check when the branch will never be taken?

I don't think this can be optimised as the required address is added in the linking stage, I have seen this behavior during all testing of these additions with the sentinel value.
Though I think @japaric can abet this better.

In that case maybe we should actually provide a DefaultPreInit which is an empty function, PROVIDE that in the linker script, remove the check and always call __pre_init in Reset? It emits fewer instructions in both cases (with and without a user function), as far as I can tell.

With the empty DefaultPreInit suggested above, in an example with no pre_init! call, built with release and fat LTO:
08000438 <Reset>:
 8000438:	f000 f933 	bl	80006a2 <DefaultPreInit>
 800043c:	f240 0008 	movw	r0, #8
 8000440:	f240 0204 	movw	r2, #4
 8000444:	f2c2 0000 	movt	r0, #8192	; 0x2000
 8000448:	f2c2 0200 	movt	r2, #8192	; 0x2000
 800044c:	4282      	cmp	r2, r0
 800044e:	d25a      	bcs.n	8000506 <Reset+0xce>
...
080006a2 <DefaultPreInit>:
 80006a2:	4770      	bx	lr

Note the branch to DefaultPreInit which immediately returns. Two instructions/six bytes/two branch overhead.
Example with pre_init!() pointing to an empty function (the current pre_init example):
08000400 <Reset>:
 8000400:	f240 0000 	movw	r0, #0
 8000404:	f240 0200 	movw	r2, #0
 8000408:	f2c2 0000 	movt	r0, #8192	; 0x2000
 800040c:	f2c2 0200 	movt	r2, #8192	; 0x2000
 8000410:	4282      	cmp	r2, r0
 8000412:	d25a      	bcs.n	80004ca <Reset+0xca>

DefaultPreInit is omitted entirely, no overhead, same as we currently have on master.
Example with pre_init! doing a ptr::write_volatile(0xcafe as *mut u32, 0xbeef):
08000400 <Reset>:
 8000400:	f64c 20fe 	movw	r0, #51966	; 0xcafe
 8000404:	f64b 61ef 	movw	r1, #48879	; 0xbeef
 8000408:	f240 0200 	movw	r2, #0
 800040c:	6001      	str	r1, [r0, #0]
 800040e:	f240 0000 	movw	r0, #0
 8000412:	f2c2 0200 	movt	r2, #8192	; 0x2000
 8000416:	f2c2 0000 	movt	r0, #8192	; 0x2000
 800041a:	4282      	cmp	r2, r0
 800041c:	d25a      	bcs.n	80004d4 <Reset+0xd4>

The new (and inlined) function is the very first thing to run. No other overhead.

@adamgreig I tried making a simple empty default pre-init function and built with fat LTO and didn't see it inlined. That said, I like that implementation better if only to get rid of the conditional branch.

Right, if you have a empty default pre-init it doesn't get inlined (my first example) but is at least a lot less overhead than the conditional approach. If you then use pre_init!(some_empty_function) it's entirely optimised out (my second example), so for people who were concerned about the small startup overhead, there's a very simple way to get rid of it entirely.

I've changed over to using the empty default function instead of a sentinel as suggested by @adamgreig. Please feel free to rip it to shreds.

Thanks! LGTM. I'll let @korken89 approve his review and then I think we're good to merge, maybe we can sneak it in to v0.5.2 #88 too.

Looks like one of the travis builds stalled.

This does indeed generate better assembly, good catch and good work!
bors r+


Add .jinja2 file extension to supported web files.

Seeing that the web lang module already supports .jinja files, I don't think it's a far cry to add the .jinja2 extension to the list.
It's not super fancy but better than no major mode support.
Working with limited lisp experience to expand the highlighting functionality. :shrug:

Thanks!


use __call

Hi,
I'm trying to call the create a form with a categorie translate, like in doc. Everything works good, i can create and translate entity. But, when my form is submit, i have this error:
Neither the property "name" nor one of the methods "getName()", "name()", "isName()", "hasName()", "__get()" exist and have public access in class "AM\AppBundle\Entity\Category".

I have try to do this in my class Category but it's not beeter.
    /**
     * This function makes it possible to call translated properties without having to know if the enitity is translated.
     *
     * @param type $method
     * @param type $arguments
     * @return type
     */
    public function __call($method, $args)
    {
        if (!method_exists(self::getTranslationEntityClass(), $method)) {
            $method = 'get'. ucfirst($method);
        }

        return $this->proxyCurrentLocaleTranslation($method, $args);
    }

If someone can help me it really apprecied!
Thanks

Can you show your FormType class for your form?

class ClassifiedAdsType extends AbstractType
{

    protected $group;
    protected $user;

    public function __construct($group = null, $user)
    {
        $this->group = $group;
        $this->user = $user;
    }

    /**
     * @param FormBuilderInterface $builder
     * @param array $options
     */
    public function buildForm(FormBuilderInterface $builder, array $options)
    {
        $builder
            ->add('title', 'text', array(
                'label' => 'form.label.title',
                'attr' => [
                    'placeholder' => 'form.placeholder.title'
                ]
            ))

            ->add('description', 'textarea', array(
                'label' => 'form.label.description',
                'attr' => array(
                    'rows' => 10,
                    'placeholder' => 'form.placeholder.description'
                )
            ))
            ->add('category', 'entity', array(
                'empty_value' => 'form.label.category',
                'class' => 'AM\AppBundle\Entity\Category',
                'label' => 'form.label.category',
                'property' => 'name',
            ))

        ;
    }

    /**
     * @param OptionsResolver $resolver
     */
    public function configureOptions(OptionsResolver $resolver)
    {
        $resolver->setDefaults(array(
            'data_class' => 'AM\ClassifiedAdsBundle\Entity\ClassifiedAds',
            'validation_groups' => [ 'classifiedsAds']
        ));

    }

}

The Form component is checking for the property name with getters/setters like getName(). This method seems not to exist in the class.
It's an issue from PHP itself, and your code.
And as said on php.net, method_exists() does not care about the existence of __call()

Oh i was thinking the __call doing the job, but it's work, i make this in CategoryEntity :
    public function getName()
    {
        return $this->findTranslationByLocale($this->getCurrentLocale())->getName();
    }

Thanks a lot for your time :-)


added transitions and effects to the footer

#153 done you can check once......

these erros coming because i have not updated the branch but footer.jsx code is working fine transitions and effects working fine

@devxMani please resolve the merge conflicts.

@devxMani ?


Frame on Windows For Native Menu

Hi,
First - thank you for producing this package. I've been waiting for years for this effect to arrive on Windows, and thanks to your work it is now. Saw your PR to get this officially into Electron too!
I had a question regarding the usage of frame on the Electron BrowserWindow. Whilst your documentation states to switch it off, I wondered why as it is actually functional (albeit with some styling issues which I explain below - hopefully something fixable):
Okay. So:
1. When frame: true
The menu does appear, and the window shows the thick frame (thickFrame: true is default on BrowserWindow)

2. When frame: true, thickFrame: false
This removes the thick frame, which then causes a gap around the menu toolbar

2.1. Toggling Windows Light/Dark Appearance
This seems to cause the menu to "fit" the window again.

Moving Forward
Clearly when the appearance mode is changed, the window undergoes some kind of "refresh" that makes the native menu fit the window. Not too sure how to trace what this process is, but it'd be great if the BrowserWindow could somehow replicate this on startup so your library can offer a BrowserWindow with a menu in line with non-acrylic windows.
I thought I would check your thoughts on this matter as I assume you are probably already aware of this during your development efforts.

I'm sorry, but this is the first time I've noticed this. That's a great idea.
However, when I look at this article and test it, there are a few problems: the first is a minor problem: there is still a slight difference if you change the theme and resize it. The second is a serious problem: the window that has been processed like this cannot be resized.
Of course, if I look at the internal structure of the electron, I'll be able to solve this problem, but I don't have the skills and time to do it :(
Instead, try custom-electron-titlebar. I haven't tested it yet, but I think it will work well with this module. I'm also considering adding this module to dependencies after testing later.

Hi @Seo-Rii, thanks for your detailed response!

the first is a minor problem: there is still a slight difference if you change the theme and resize it.
I notice the title bar always remains white unlike the menu bar.

The second is a serious problem: the window that has been processed like this cannot be resized.
Actually, this is just because of the thickFrame property and I can confirm that this happens on the standard BrowserWindow. When thickFrame: true you are able to resize the window. When thickFrame: false you cannot.

Instead, try custom-electron-titlebar. I haven't tested it yet, but I think it will work well with this module. I'm also considering adding this module to dependencies after testing later.
I did have a look at this and have tried it before, but it's a little concerning that they are archiving the project shortly.

To summarise, these are the things I noticed that might be fixable. I will research further and see if I can help:

Title bar does not change colour when the appearance is changed to light/dark
When thickFrame: true it seems like the vibrancy effect is being added to this layer where I think the window shadow should be (no shadow can be seen)

Hi @Seo-Rii, thanks for your detailed response!

the first is a minor problem: there is still a slight difference if you change the theme and resize it.

I notice the title bar always remains white unlike the menu bar. Are you seeing anything else that I have missed?

The second is a serious problem: the window that has been processed like this cannot be resized.

This appears to be due to the thickFrame property and I can confirm that this happens on the standard BrowserWindow. When thickFrame: true you are able to resize the window. When thickFrame: false you cannot.

Instead, try custom-electron-titlebar. I haven't tested it yet, but I think it will work well with this module. I'm also considering adding this module to dependencies after testing later.

I did have a look at this and have tried it before, but I am hesitant given that they are archiving the project shortly.

Whilst I have used NAPI, I am completely new to the Windows accent policy stuff, but I will research and see if I can offer any help.
The only "real" issue I have noticed is:

the title bar not changing colour when the appearance changes
when thickFrame: false the menu does not fit to the screen until the window repaints

I'm so glad you can help!
First of all, the color of the title bar does not change only when this module is applied, but it is a problem that appears in all electronic programs. Perhaps to solve the problem, it will be necessary to use the above-mentioned modules, or to create new modules to play that role.
I'll look for a problem that I can't adjust the size when the thickFrame is false.

Windows 10 control buttons redesigned in electron
x16 windows 10 control buttons:
I got the icons by redesigning them in paint.net. They are precisely identical to the one used in both FileExplorer and Notepad. It can be used on any color or surface due to the transparency in the icons.
There a 165ms transition on the background of the icon. I calculated this with the frames of a 60fps video and a 30fps video of the transition. I checked on both Notepad and FileExplorer.
The color of the close button is #E81123.  Checked on both Notepad and FileExplorer.
The color added when hovering is rgba(0, 0, 0, 26). I got the opacity added to the other control buttons on hover by calculating the difference between the white background (255) and hovered (229). I checked on both Notepad and FileExplorer.
The buttons should be 45x29px but can be designed differently on different applications. In paint.net, the controls are 25x21px.


[Bug]: Not able to install vernemq aws linux 2023

Environment

VerneMQ Version: 1.13.0
OS: Linux

Current Behavior
When i run this command
yum install vernemq-1.13.0.amzn2.x86_64.rpm
Iam getting below error
Last metadata expiration check: 2:06:19 ago on Fri Jun 16 05:06:54 2023.
Error:
Problem: conflicting requests

nothing provides libcrypto.so.10()(64bit) needed by vernemq-1686126158:1.13.               0-1.x86_64
nothing provides libcrypto.so.10(OPENSSL_1.0.1_EC)(64bit) needed by vernemq-<PHONE_NUMBER>:1.13.0-1.x86_64
nothing provides libcrypto.so.10(OPENSSL_1.0.2)(64bit) needed by vernemq-168               6126158:1.13.0-1.x86_64
nothing provides libcrypto.so.10(libcrypto.so.10)(64bit) needed by vernemq-1               686126158:1.13.0-1.x86_64
(try to add '--skip-broken' to skip uninstallable packages)
[root@ip-172-31-33-213 ec2-user]

Expected behaviour
It should install vernemq
Configuration, logs, error output, etc.
No response
Code of Conduct

[X] I agree to follow the VerneMQ's Code of Conduct

@syedzahidulla Thanks, ugh, an issue with the linked libcrypto. Let me investigate this.

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

@syedzahidulla in the amz2 builder for the package, we yum install a couple packages, namely openssl openssl-devel snappy-devel. Please check whether OpenSSL is installed in your Linux 2 instance.

:point_right: Thank you for supporting VerneMQ: https://github.com/sponsors/vernemq
:point_right: Using the binary VerneMQ packages commercially (.deb/.rpm/Docker) requires a paid subscription.

Seems like it was missing openssl openssl-devel and snappy-devel in this new version.
So, thanks


Azure-identity importing for Python3 Runbook throws OpenSSL issue

Package Name: azure-identity
Package Version: 1.7.1
Operating System: Azure Automation Account
Python Version: 3.8.0

Describe the bug
There seems to be a version conflict with OpenSSL package from Python3 vs. OpenSSL from package "cryptography", which is a dependency package from azure-identity or azure-storage-blob.
When using the Import_py3package_from_PyPi script in an Azure Automation Account, the following error occurs on the second run:
Traceback (most recent call last):  File "C:\Temp\etqormcy.jez\be1925d7-d9b7-4f8c-80e7-2a98f1a60b25", line 127, in <module>
    token = get_automation_runas_token()  File "C:\Temp\etqormcy.jez\be1925d7-d9b7-4f8c-80e7-2a98f1a60b25", line 35, in get_automation_runas_token
    from OpenSSL import crypto  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\crypto.py", line 15, in <module>
    from OpenSSL._util import (  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\_util.py", line 6, in <module>
    from cryptography.hazmat.bindings.openssl.binding import Binding  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\cryptography\hazmat\bindings\openssl\binding.py", line 203, in <module>
    _verify_package_version(cryptography.__version__)  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\cryptography\hazmat\bindings\openssl\binding.py", line 192, in _verify_package_version
    raise ImportError(ImportError: The version of cryptography does not match the loaded shared object.
    This can happen if you have multiple copies of cryptography installed in your Python path.
    Please try creating a new virtual environment to resolve this issue. Loaded python version: 35.0.0, shared object version: b'2.8'

To Reproduce
Steps to reproduce the behavior:

Use Import Runbook to import "pandas" (works as expected)
Use Import Runbook to import "azure-identity" (works as expected)
Use Import Runbook to import "azure-storage-blob" (Error is thrown)

The Error is thrown in the Import_py3package_from_pypi runbook in method:
def get_automation_runas_token():
      from OpenSSL import crypto

On the first run the error is not thrown. But Azure-Identity also installs cryptography package. Importing the second azure-storage-blob package then throws the error as shown above. You can also reverse the order of installation with the same error (azure-storage-blob first, then error when importing azure-identity)
Expected behavior
The same procedure worked when using Python 2.7 Runbooks for importing both azure-identity and azure-storage-blob package.
Also within logic runbooks there were no error when using "from OpenSSL from crypto" for Authentication.
Python 3 Environment seems to deliver a separate OpenSSL version that is not compatible with the referenced OpenSSL in package "cryptography", which itself is used in azure-identity
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@CedricOettel Msal (one of the identity dependencies) recently made a change of the dependent cryptography version. (https://github.com/AzureAD/microsoft-authentication-library-for-python/commit/16dcb22690c1c66e96a7c96b4dda4cf7ceb58e00#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7)
It bumped cryptography from <4 to 35 which may be the cause.
Could you try install cryptography<4?
Does it solve the problem?

How can i do that in an Azure Automation Account?
Do i need to reference a specific version of azure-identity which does not have the reference to 35.0.0 ? If so, which version of azure-identity has a lower version of MSAL than 1.15.0?

I am not an expert of Azure Automation. Is it possible that you can specify something like requirements and install cryptography < 4 before installing identity library?

Hello,
we have tried to install cryptography 3.2.2 (the version, that is installed with azure-identity/-storage-blob for Python 2.7 in Azure) before. Then the same error occures on the first install of azure-identity or -storage-blob. So it seems not to be a problem of the "wrong" version of cryptography.

@irm-phoudek when you say same error, do you mean the error with message
"Please try creating a new virtual environment to resolve this issue. Loaded python version: 35.0.0, shared object version: b'2.8'"
Isn't 35.0.0 the version of cryptography library?

@xiangyan99 of cause the versionnumber is changed as also the id of the runbook-VM and a few line numbers, but stacktrace and error is the same:
Traceback (most recent call last):  File "C:\Temp\a1315bzt.kjf\6906cfc5-c203-4dbc-a68a-fae3c64e56a0", line 127, in <module>
    token = get_automation_runas_token()  File "C:\Temp\a1315bzt.kjf\6906cfc5-c203-4dbc-a68a-fae3c64e56a0", line 35, in get_automation_runas_token
    from OpenSSL import crypto  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\crypto.py", line 15, in <module>
    from OpenSSL._util import (  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\OpenSSL\_util.py", line 6, in <module>
    from cryptography.hazmat.bindings.openssl.binding import Binding  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\cryptography\hazmat\bindings\openssl\binding.py", line 170, in <module>
    _verify_package_version(cryptography.__version__)  File "C:\WPy64-3800\python-3.8.0.amd64\lib\site-packages\cryptography\hazmat\bindings\openssl\binding.py", line 159, in _verify_package_version
    raise ImportError(ImportError: The version of cryptography does not match the loaded shared object.
    This can happen if you have multiple copies of cryptography installed in your Python path.
    Please try creating a new virtual environment to resolve this issue. Loaded python version: 3.3.2, shared object version: b'2.8'```

Looks like cryptography 2.8 is installed in Azure Automation Account. @automation team, can anyone help to confirm it?

Yes, cryptography 2.8 is installed in Automation Sandbox to support assets in Python.

@mahakjain314 Thanks for the information. Just to confirm, after Sandbox installs 2.8, it is ok to upgrade cryptography to a higher version, right?

Ideally, one should be able to upgrade to a higher package version but we have identified an issue because of which installing the new version of crypto package is not overriding the previous one.

@mahakjain314, do you have any reference to this issue so I can track it? If not, what's the status on this?
This currently prevents us from using the latest azure.mgmt libraries on Azure Automation in Python3 runtime, and as the libraries that are already installed are apparently ~4 years old, they have very limited functionality for the more recent Azure services.
If this issue is not expected to be resolved anytime soon, we will have to try and temporarily switch to Python2 runtime I guess

Closing because this is not an SDK issue.
Please open service support ticket to track it.

If anyone is looking for a workaround, creating a fresh automation account (without any packages installed) and running to importpypackage runbook from it targeting my actual automation account made it work


Outlook Mail - Search results do not work for dates specified in MM/DD/YYYY format.

Electron build with commit id https://github.com/gptscript-ai/desktop/commit/56cadbd5779acd6ae9ef9bd4a192b44b8f4d599c
Steps to reproduce the problem:

Create an assistant with outlook mail and Sys.Time.Now tool.
Start a chat with this assistant and ask it to retrieve the email for a specific date MM/DD/YYYY format like 8/13/2024
Notice that no matches are returned.

When we ask the same question for the same date in words like August 13th 2024 , matched emails are returned.

The tool calls made in both cases seem to be the same , but one returns results and other does not.

@sangee2004 You probably didn't receive any email messages in 2004. Can you double-check that wasn't the problem?

This is my bad. I am able to get mails to be fetched as expected when date is specified in mm/dd/yyyy format.


Spring cloud config server git and vault

SpringBoot: 2.0.0.M3
cloud: Finchley.M1
I need to use Config server with Git and vault . (application.properties)
And another configuration on Configserver that will use Git(But will point to local file path) and Vault (application-local.properties).
POM
<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-config-server</artifactId>
		</dependency
	<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-vault-config</artifactId>
		</dependency>

When I provide the properties for vault as below.
spring.profiles.active=vault, git
spring.cloud.config.server.vault.order=1
spring.cloud.config.server.git.order=2
spring.cloud.config.server.vault.host=<IP_ADDRESS>
spring.cloud.config.server.vault.port=8200
spring.cloud.config.server.vault.scheme=http
spring.cloud.config.server.vault.token=419113c3-d058-faca-4965-b6e03a918a28
#spring.cloud.config.token=419113c3-d058-faca-4965-b6e03a918a28
spring.cloud.config.server.vault.uri=http://<IP_ADDRESS>:8200
spring.cloud.config.server.vault.connection-timeout=5000
spring.cloud.config.server.vault.read-timeout=15000

It doesn't connect to vault server and i get an exception -
.lang.IllegalArgumentException: Token (spring.cloud.vault.token) must not be empty
springframework.util.Assert.hasText(Assert.java:276) ~[spring-core-5.0.0.RC3.jar!/:5.0.0.RC3]
springframework.cloud.vault.config.VaultBootstrapConfiguration.clientAuthentication(VaultBootstrapConf
springframework.cloud.vault.config.VaultBootstrapConfiguration$$EnhancerBySpringCGLIB$$937026be.CGLIB$

When i provide the values in bootstrap as below -
spring.cloud.vault.host=<IP_ADDRESS>
spring.cloud.vault.port=8200
spring.cloud.vault.scheme=http
spring.cloud.vault.token=419113c3-d058-faca-4965-b6e03a918a28
spring.cloud.vault.uri=http://<IP_ADDRESS>:8200
spring.cloud.vault.connection-timeout=5000
spring.cloud.vault.read-timeout=15000

It is then able to give me the required details when i hit the url - http://localhost:1010/ttt-microservice/default/develop/
However, if i do not give these values in bootstrap but provide the same in application.properties I again run in to the same error. Should these configurations not be handled automatically when I am providing them for spring.cloud.config.server...
Also, if i omit the spring.cloud.config.server.xxx configurations and just provide the spring.cloud.xx properties it keeps looking for the localhost for vault server
"timestamp":<PHONE_NUMBER>084,
    "status": 500,
    "error": "Internal Server Error",
    "message": "I/O error on GET request for \"http://<IP_ADDRESS>:8200/v1/secret/ttt-microservice\": Connection refused: connect; nested exception is java.net.ConnectException: Connection refused: connect",
    "path": "/ttt-microservice/default/develop/"

Please clarify. Because this is bit different then the  example for git and svn. thanks aakash

Related to https://github.com/spring-cloud/spring-cloud-vault/issues/164

@mp911de thanks for your prompt reply is there any provision/ticket so cloud config server supporting vault can handle the connections from clients dynamically instead of hardcoded token through xconfigtoken

Yes, you can pass X-Vault-Token with each HTTP request. I'm not sure whether that's also possible using Spring Cloud Config Client. Maybe @spencergibb or @ryanjbaxter can help.

Please learn how to properly format code and logs.
Config server doesn't recognize spring.cloud.vault.* properties.
The config client needs to set the vault token in spring.cloud.config.token in bootstrap.yml

@spencergibb thanks can you update is there any provision in future to move away from x-config-token to dynamic authentication at client side. As token may not be secured way and will have ttl and other issues to handle on restarts

hi @spencergibb can you please update so i will close the issue. thanks

I am also facing this issue when using Spring cloud config server with vault. Can you please provide some URL reference for complete configurations? I have referred https://spring.io/blog/2016/06/24/managing-secrets-with-vault .

Hi All,
Followed below URL and created a sample example of spring-cloud-vault locally.
https://github.com/gmarziou/demo-spring-cloud-vault
Showing below connection refused error.
 main] LeaseEventPublisher$LoggingErrorListener : [RequestedSecret [path='secret/bootstrap/vault', mode=RENEW]] Lease [leaseId='null', leaseDuration=0, renewable=false] I/O error on GET request for "http://localhost:8200/v1/secret/bootstrap/vault": Connect to localhost:8200 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused: connect; nested exception is org.apache.http.conn.HttpHostConnectException: Connect to localhost:8200 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused: connect

org.springframework.web.client.ResourceAccessException: I/O error on GET request for "http://localhost:8200/v1/secret/bootstrap/vault": Connect to localhost:8200 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused: connect; nested exception is org.apache.http.conn.HttpHostConnectException: Connect to localhost:8200 [localhost/<IP_ADDRESS>, localhost/0:0:0:0:0:0:0:1] failed: Connection refused: connect
Tried with changing the above mentioned configuration still showing errors.
It would be great help.
Thanks

Is the problem still there in Finchley.RELEASE?

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


mattdrayer/catch-token-exception: Also handle InvalidTokenError

@clintonb @douglashall @e0d -- added InvalidTokenError to the list of exceptions we watch for during JWT decoding.

👍

[Current coverage][cc-pull] is 100%

Merging [#15][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #15   diff @@
===================================
  Files          10    10
  Lines         361   361
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits          361   361
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last updated by [7ee6832...a51ff5c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/edx/edx-drf-extensions/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/edx/edx-drf-extensions/compare/7ee6832e664d3ed8187e3381d1ec123667b42f91...a51ff5c203ac1736869f520125621ceb1d0314a2
[cc-pull]: https://codecov.io/gh/edx/edx-drf-extensions/pull/15?src=pr

Tests?

Tests are passing with coverage @ 100% without adding new tests?

I've added a new test which passes in an "invalid.token" to the decoding workflow

👍

Adding new release tag now


Shutdown should return in memory data

Yes please, I would like to safe this data to a file before exiting my application in order to avoid losing data.


D3FC unit test

Added support for Jest unit tests w/babel to the perspective-viewer-d3fc plugin, with a stolen test implementation from the D3FC PoC repo

@matt-hooper There were a few issues:

babel-jest did not add support for Babel 7's babel.config.js until version 24, which caused our settings to not get picked up.
The top-level jest config had transform: {}, and used roots instead of projects which caused the package local configs to not get picked up.
There is a bug in jest which causes it to always ready the babel.config.js from the <rootdir> rather than the local package when using the multi-project runner, which required a custom transformer workaround.

I've also moved the test suite into integration/ and unit/ sub-directories.  Let me know if this is OK to merge.

Thanks for the PR. I will take a look and let you know how I get on.

@texodus, I have no issue with this being merged. It will be very useful. Thanks

Groovy, thanks!


Build issue for AT through UART

Environment

Development Kit:      ESP32-Wrover-Kit
Kit version (for WroverKit/PicoKit/DevKitC): v4.1
Module or chip used:  ESP32-WROVER-B
ESP-AT version: v<IP_ADDRESS>_esp32s2-122-g0c15ea3
Build System:         CMake
Compiler version : 8.4.0
Operating System:     Linux
Power Supply:        USB

Problem Description
Hello
If I build the standard configuration (AT through UART) I run into compile errors in the file "main/interface/hspi/at_hspi_task_esp32.c":
In file included from ../esp-idf/components/soc/esp32/include/soc/io_mux_reg.h:17,
from ../esp-idf/components/soc/include/soc/gpio_periph.h:18,
from ../esp-idf/components/driver/include/driver/gpio.h:23,
from ../main/interface/hspi/at_hspi_task_esp32.c:37:
../main/interface/hspi/at_hspi_task_esp32.c: In function 'pull_high_cb':
../main/interface/hspi/at_hspi_task_esp32.c:44:38: error: 'CONFIG_AT_HSPI_HANDSHAKE_PIN' undeclared (first use in this function); did you mean 'CONFIG_PTHREAD_STACK_MIN'?
#define SPI_SLAVE_HANDSHARK_GPIO     CONFIG_AT_HSPI_HANDSHAKE_PIN
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
and so on.
After looking in the code of "at_hspi_task_esp32.c", I could work around the issue by adding
"#ifdef CONFIG_AT_BASE_ON_HSPI" in line 37 and a "#endif" at the end of the file.

@kkrischker Thanks. This is indeed a problem introduced by our latest commit, and we will fix this problem as soon as possible.

@kkrischker Sorry for the late reply. esp-at had fixed it one month ago, please try with the latest version.

@kkrischker Sorry for the late reply. esp-at had fixed it one month ago, please try with the latest version.


WTFcsv

Using this file in WTF -- https://www.dropbox.com/s/jpdnbdl5dni7rs2/HowAmericaInjuresItself_FromNEISS.xlsx?dl=0
Got the below error which seems to be new. This could also just be the same encoding issue we have seen previously.

Just checked and changing encoding makes the error go away - going to paste this on the other bug


Spelling of the word information

@sam-m888 thank you for your contribution!  Can I get you to sign to CLA associated with this repository, and I can get it pulled into the main branch.  Thanks!

@gep13 Your welcome, thanks for chocolatey :+1:

Perfect, thank you very much!


Saving voice information

Hello, I would like to ask if there is any relevant way to find or save the voice information entered by the user when I only found the text of the question and answer during the execution of your code. Any reply would be appreciated

Hello,
What do you call "voice information"?
The transcription of what they said?

Hello! When we ask pepper a question, pepper's built-in voice-to-text module converts what I say into text. My question is: Is there any way to find the audio file of my question, like.wav or.mp3?

@victorpaleologue i'm also in the same situation. recording audio source from the tablet is not the best :/ i need it to submit to whisper and/or gpt ai


checksum support , improve control with new cmd line switches

This PR leverages the MD5 checksum available through local, blob, or Data Lake storage metadata fields, based on the discussion in https://github.com/Azure-Samples/azure-search-openai-demo/pull/2113.
By default, the PR introduces a check during ingestion (for both local and Azure Data Lake Storage sources) to see if each file already exists in the target storage. The MD5 checksum is saved in the index as a verification method, enabling a lookup that ensures at least one record with the same category/filename is found. This also permits the same file to be copied to multiple categories if needed.
Additionally, a new command-line parameter, --folder <name>, allows specifying a dedicated subfolder within the data folder or Data Lake container for each script run. This enables isolated runs with explicitly controlled parameters and a specific subset of files.
The new --enablevision true/false option allows per-folder and per-category control over whether vision embeddings should be generated. This option is beneficial when dealing with folders containing predominantly heavy image content, providing control over resource use.
Example usage:
./.venv/bin/python ./app/backend/prepdocs.py './data/createindex_dummy/*' --category newtest --folder test --enablevision false --verbose

The PR also improves progress reporting by displaying the number of files processed and providing updates every 10 files. This enhances visibility into the processing speed, which is particularly useful when working with partially processed data. Additionally, the log level of some messages has been adjusted from INFO to DEBUG to reduce console noise and retain progress updates longer.
Finally, MD5 checksum comparisons are introduced at various stages: a top-level check (to avoid redundant processing), and checks for blobs, text, and image embeddings. A flag to bypass the top-level check for known MD5 values in the index may be introduced later (e.g., --ignore_checksum).

I've gone in and made the changes I recommended, but I have some higher-level questions about the approach, which I've left as comments.


add requirement plz

pip install python-bash-utils==0.1.5
this package is only working on this project for bashutils

Try pip install python-bash-utils==0.3.2.  I'll update the requirements.


Extend lifetime of string containing namespace specific portion of document id.

@baldersheim : please review
@vekterli : FYI

I think it is better to change String2ResultNode to use vespalib::stringref instead instead of using a const vespalib::string & reference instead of adapting to the bad code.

Obsoleted by upcoming better fix from @baldersheim.


[Spark-20775][SQL] Added scala support from_json

What changes were proposed in this pull request?
from_json function required to take in a java.util.Hashmap. For other functions, a java wrapper is provided which casts a java hashmap to a scala map. Only a java function is provided in this case, forcing scala users to pass in a java.util.Hashmap.
Added the missing wrapper.
How was this patch tested?
Added a unit test for passing in a scala map

Can one of the admins verify this patch?

ok to test

Test build #77379 has started for PR 18094 at commit a2f99ec.

Test build #77379 has finished for PR 18094 at commit a2f99ec.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77379/
Test FAILed.

Test build #77381 has started for PR 18094 at commit 27a8c26.

Test build #77381 has finished for PR 18094 at commit 27a8c26.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77381/
Test PASSed.

thanks, merging to master!


Ensure no java.lang.UnsupportedClassVersionError are thrown if runnin…

…g on Java7 and try to check if conscrypt is available.
Motivation:
We need to ensure we not try to load any conscrypt classes directly (which means without using reflection) in the same class that is used to check if conscrypt is available. This is needed as otherwise we will have the following problem when try to use netty on java7:
java.lang.UnsupportedClassVersionError: org/conscrypt/BufferAllocator : Unsupported major.minor version 52.0
at io.netty.handler.ssl.ConscryptJdkSslEngineInteropTest.checkConscrypt(ConscryptJdkSslEngineInteropTest.java:49)
This regression was introduced by 4448b8f42f599e79db1744cd8f5fdfee702c195e and detected on the CI when using:
mvn clean package -DtestJavaHome=$JAVA7_HOME
Modifications:
Move the detection code in an extra class and use it.
Result:
Works correctly also when using Java7.

@nmittler PTAL as it was your commit that introduced the regression. Beers on you =P

let me just merge this one... as @Scottmitch approved.

Cherry-picked into 4.1 (32f497760f4fc18b797d53ed26b690f268f2ee2b) and 4.0 (04df3e21ed33bb7e05103bfdbc413480a3963391)


Calling format with y mutates the state

Describe the bug
Calling format mutates the state.
To Reproduce

Go to this fiddle: https://jsfiddle.net/zborwn0u/8/
Fiddle logs current time as formatted moment h:mma y h:mma.
Notice that the time after y is always 12:00am
Then, it logs h:mma again, and toISOString() to display 12:00am, proving mutated state.

Expected behavior
Calling format should not change the state of moment object.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu 18.04
Browser: Chrome
Version [e.g. 22]

Please run the following code in your environment and include the output:
console.log((new Date()).toString())
console.log((new Date()).toLocaleString())
console.log((new Date()).getTimezoneOffset())
console.log(navigator.userAgent)
console.log(moment.version)

Sun Jun 21 2020 00:19:01 GMT+0100 (British Summer Time)
21/06/2020, 00:19:01
-60
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.106 Safari/537.36
2.25.3

From my research on this issue. It looks like it has to do with the y or year format options.
let m = moment()
let x = moment()

console.log(m.format('y h:mma h:mma y')) // "2020 12:00am 12:00am 2020"
console.log(x.format('h:mma'))           // "8:39pm"

but.....
let m = moment()

console.log(m.format('h:mma'))            //  "8:40pm"
console.log(m.format('y h:mma h:mma y'))  // "2020 12:00am 12:00am 2020"
console.log(m.format('h:mma'))            // "12:00am"

Hope this helps 🦄
My environment was the same JSFiddle
"Sat Jun 20 2020 20:43:56 GMT-0400 (Eastern Daylight Time)"
"6/20/2020, 8:43:56 PM"
240
"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.0.5 Safari/605.1.15"
"2.27.0"

There are a couple of instances on the era.js file where they mutate the moment object.
https://github.com/moment/moment/blob/4b615b9d4b00808c16c06c69cfcaa4ecdaedd7b4/src/lib/units/era.js#L167
https://github.com/moment/moment/blob/4b615b9d4b00808c16c06c69cfcaa4ecdaedd7b4/src/lib/units/era.js#L187
https://github.com/moment/moment/blob/4b615b9d4b00808c16c06c69cfcaa4ecdaedd7b4/src/lib/units/era.js#L210
All can be possibly fixed with:
val = moment(this).startOf('day').valueOf()

I think this is fixed in 2.28.0


Some adaption for conform/exrm

@c-rack thanks for quantum first of all :)
I've got some troubles using quantum with exrm.
First I need to generate some configs like this:
$ mix conform.new
$ mix conform.configure

and after
$ mix conform.effective

it fails. Looks like it can't convert quantum's config into his own of release. I've tried many variants and decided to use simple form of configuration, just like in cron:
* * * * * MyApp.MyModule.my_method

Configuration looks like:
config :quantum, cron: [
    "* * * * * MyApp.MyModule.my_method"
]

Hope this will be useful for others.

I think I ran into the same issue with edeliver which uses exrm under the hood.

@alPacino Thanks! I will release a new version and publish to hex.pm shortly.

So version 1.3.0 has been released to hex.pm.
However, the PR has decreased the code coverage by 1.56%.
Fortunately, it is only one line not being covered.
Any volunteers to bring code coverage back to 100%? :smile:

Sorry, it's my bad :) I'm on it. Will send another PR.
Also will add configuration example into README:
config :quantum, cron: [
    "* * * * *": {"MyApp.MyModule", :my_method}
]


Doesn't work

You can see the error on screenshot.

I'm afraid Cumcord is dead, and is no longer maintained or expected to work.
For more info see here.
I suggest you use either Vencord, the current largest client mod, or shelter, which is, I guess, Cumcord's spiritual successor.


adds option to have comma separation (french and other languages)

This does seem the regress performance a bit, note that running benchmark twice gave significantly different results on my computer... (but each time with some regressions)
Benchmarking on my computer for this PR:
     Running bench.rs (target/release/deps/numfmt_benchmarks-c34caa3adfdc86b5)
numfmt default zero     time:   [33.368 ns 33.385 ns 33.403 ns]
                        change: [+0.8768% +1.1387% +1.3964%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 11 outliers among 100 measurements (11.00%)
  1 (1.00%) high mild
  10 (10.00%) high severe

numfmt cached zero      time:   [2.2635 ns 2.2685 ns 2.2736 ns]
                        change: [+0.4760% +0.7272% +0.9739%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 17 outliers among 100 measurements (17.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  12 (12.00%) high mild
  3 (3.00%) high severe

std zero                time:   [41.733 ns 41.819 ns 41.918 ns]
                        change: [+1.3226% +1.7623% +2.2284%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

numfmt default _1234    time:   [129.10 ns 129.34 ns 129.62 ns]
                        change: [-0.8351% -0.5640% -0.2967%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  1 (1.00%) high mild
  1 (1.00%) high severe

numfmt cached _1234     time:   [81.148 ns 81.278 ns 81.396 ns]
                        change: [+1.9613% +2.2701% +2.5776%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 4 outliers among 100 measurements (4.00%)
  3 (3.00%) low mild
  1 (1.00%) high mild

std _1234               time:   [79.693 ns 80.016 ns 80.329 ns]
                        change: [+6.4334% +6.8675% +7.2924%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

numfmt default lrgdec   time:   [139.88 ns 140.33 ns 140.75 ns]
                        change: [-2.8866% -2.3800% -1.9372%] (p = 0.00 < 0.05)
                        Performance has improved.

numfmt cached lrgdec    time:   [93.500 ns 93.631 ns 93.792 ns]
                        change: [-1.3219% -1.1511% -0.9652%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

std lrgdec              time:   [112.80 ns 112.88 ns 112.97 ns]
                        change: [-3.3537% -3.0926% -2.8167%] (p = 0.00 < 0.05)
                        Performance has improved.

numfmt default sn       time:   [181.31 ns 181.59 ns 181.93 ns]
                        change: [-1.8346% -1.6280% -1.3481%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  2 (2.00%) high mild
  6 (6.00%) high severe

numfmt cached sn        time:   [131.92 ns 132.09 ns 132.34 ns]
                        change: [+0.9991% +1.1885% +1.3928%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe

std sn                  time:   [260.52 ns 260.73 ns 260.93 ns]
                        change: [-0.5715% -0.3167% -0.1062%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 7 outliers among 100 measurements (7.00%)
  6 (6.00%) high mild
  1 (1.00%) high severe

Bench for main :
numfmt default zero     time:   [32.967 ns 33.059 ns 33.158 ns]
                        change: [-6.0488% -5.1831% -4.3027%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
  7 (7.00%) high mild
  1 (1.00%) high severe

numfmt cached zero      time:   [2.2459 ns 2.2503 ns 2.2545 ns]
                        change: [+0.3476% +0.5324% +0.7034%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 16 outliers among 100 measurements (16.00%)
  8 (8.00%) low severe
  5 (5.00%) low mild
  2 (2.00%) high mild
  1 (1.00%) high severe

std zero                time:   [40.684 ns 40.873 ns 41.080 ns]
                        change: [-1.7038% -1.2958% -0.8568%] (p = 0.00 < 0.05)
                        Change within noise threshold.

numfmt default _1234    time:   [130.21 ns 130.62 ns 131.06 ns]
                        change: [+0.8531% +1.0624% +1.3380%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

numfmt cached _1234     time:   [78.864 ns 79.089 ns 79.326 ns]
                        change: [-0.7862% -0.4996% -0.1658%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high mild

std _1234               time:   [73.907 ns 74.080 ns 74.291 ns]
                        change: [-7.1703% -6.8100% -6.4341%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild

numfmt default lrgdec   time:   [143.88 ns 145.01 ns 146.32 ns]
                        change: [+2.9165% +3.3563% +3.8937%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 10 outliers among 100 measurements (10.00%)
  2 (2.00%) high mild
  8 (8.00%) high severe

numfmt cached lrgdec    time:   [94.915 ns 95.027 ns 95.134 ns]
                        change: [+0.3181% +0.7374% +1.1036%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  1 (1.00%) low mild
  1 (1.00%) high mild

std lrgdec              time:   [115.00 ns 115.53 ns 116.03 ns]
                        change: [-0.4650% +0.2277% +0.8501%] (p = 0.51 > 0.05)
                        No change in performance detected.
Found 5 outliers among 100 measurements (5.00%)
  5 (5.00%) low mild

numfmt default sn       time:   [184.24 ns 184.36 ns 184.50 ns]
                        change: [-1.3577% -1.1571% -0.9767%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

numfmt cached sn        time:   [130.04 ns 130.29 ns 130.60 ns]
                        change: [-1.0067% -0.8446% -0.6675%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe

std sn                  time:   [260.57 ns 260.99 ns 261.36 ns]
                        change: [-8.9069% -8.6678% -8.4128%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  4 (4.00%) low mild
  3 (3.00%) high mild
  2 (2.00%) high severe

Thanks for the PR!
It looks decent enough, but I'll probably require a few tweaks to the format string to support this.
Likely an autodetection on the separator (if the separator . is used, switch to using , for decimal) could be used, which would save having to add another field.
I am a bit busy at the moment, so I haven't had the chance to fully review, I'll get to it in the next few weeks 😞

I've gone and extended your changes in https://github.com/astrale-sharp/numfmt/pull/1

Thanks and sorry for the delay! It's ready


Added Body Detection

Description
Here I have added added Body Detection feature which detect your body using full-body cascade xml file.
Fixes #533
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@kaustubhgupta  its working when I am running like this

This is the structure

@Avishake007 I still not able to run your script. Can you add a GIF to your README showing the demo of your script?

@antrikshmisri @santushtisharma10 Can you confirm this script working?

@antrikshmisri @santushtisharma10 Can you confirm this script working?

Sure , will get back to you in a min.

@kaustubhgupta The script is working fine.

Okay then👍💯


[SpiceProxyRedirect] How to get infos

Hello Josh,
thank you for this Software!
However i dont understand what to put in here:
# The Spice Proxy provided by the Proxmox API may need to have its host/port rewritten
# These rewrite rules are written `IP:port = IP:port`
# 1. Use the inidebug and read the current proxy=pve1.example.com:3128
# 2. Add your proxmox ip to the right side e.g. <IP_ADDRESS>:6000
pve1.example.com:3128 = <IP_ADDRESS>:6000

Left is FQDN from proxy and right side my ip of the proxmox host?

[virt-viewer]
type=spice
password=14a60aa38fab20d4e945590fa1c4b195a8fc6cd4
toggle-fullscreen=Shift+F11
tls-port=61000
proxy=http://my2ndFQDN:3128
delete-this-file=1
title=VM 100 - v100-u20-foxy
host=pvespiceproxy:64198b7e:100:myHostName::3dbce15a99118cc74b36b6ac1d813d8325fa3f4c
release-cursor=Ctrl+Alt+R
secure-attention=Ctrl+Alt+Ins
ca=-----BEGIN CERTIFICATE-----\n
....
-----END CERTIFICATE-----\n
host-subject=OU=PVE Cluster Node,O=Proxmox Virtual Environment,CN=my1stFQDN

Ok got it :D

Cant get Graphics Display - seems like something still missing :(

I've added the ca certificate under
[AdditionalParameters]
ca=--...
no luck..

Hosts are for connecting and authenticating against proxmox. You set the IP or FQDN of one or more of your proxmox hosts, followed by = and then the port number
What you need to implement is SpiceProxyRedirect to redirect http://my2ndFQDN:3128 to an actual host on your network
This is done by using the SpiceProxyRedirect section as follows:
[SpiceProxyRedirect] my2ndFQDN:3128=<IP_ADDRESS>:3128
where <IP_ADDRESS> is the IP address of the host in question.


DIC produces invalid CUE file for a certain disc.

DIC Version: 20220909
Bug Description:
DIC produced the follow CUE:
FILE "ZERODIVIDEX.bin" BINARY
  TRACK 01 MODE61/2352
    INDEX 01 00:00:00

Redumper, on the hand, produced an valid one:
FILE "dump_230106_102503_J.bin" BINARY
  TRACK 01 MODE1/2352
    INDEX 01 00:00:00

Is this a bug with DIC or I'm looking at a really wacky disc?
Logs: https://cdn.discordapp.com/attachments/705884111448834100/1060840873820180560/Zero_Divide_TW_Logs.7z

Test please.
DiscImageCreator_test.zip

Uploaded.
DiscImageCreator_test.zip

Uploaded. DiscImageCreator_test.zip

Thanks Sarami. This build works like a charm!


Unexpected ENDFORM (structure)

Playground:

source:
MV45AFOI_TSW_UPDATE_WORKLIST_N

what happens if you delete the first 19 lines in the example above?

The same. It seems he has problems with the tabs though...

Interestingly, when i remove all the tabs, the issue is gone... left one tab

Removed last tab...

Note that this source comes from a SAP program so it can not be changed.

form moo using value(bar)	type c.
endform.

try again, 2.41.2 published

Issue resolved. Thanks


Update Node.js version used in CI systems

After the upgrade to Electron v3 Atom's codebase runs on Node v10.2.0, so upgrading the CI systems to also use the same Node.js version brings consistency (in terms of language features) and can also impact on build speed improvements (although these may be minor).
This PR updates the Node.js version to v10.2.1 (we cannot update to v10.2.0 since there are build failures caused by https://github.com/nodejs/node/issues/20921).

Some warnings appear both when installing the npm dependencies on the script/ and on the root folder.
Some of these warnings already appear on master, it's just that upgrading Node.js has added a few more. We should fix them at some point.
Root folder
ld: warning: text-based stub file /System/Library/Frameworks//Foundation.framework/Foundation.tbd and library file /System/Library/Frameworks//Foundation.framework/Foundation are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreFoundation.framework/Versions/A/CoreFoundation.tbd and library file /System/Library/Frameworks//CoreFoundation.framework/Versions/A/CoreFoundation are out of sync. Falling back to library file for linking.
../src/main.cc:26:15: warning: 'Call' is deprecated [-Wdeprecated-declarations]
    callback->Call(1, argv);
              ^
../../nan/nan.h:1617:3: note: 'Call' has been explicitly marked deprecated here
  NAN_DEPRECATED inline v8::Local<v8::Value>
  ^
../../nan/nan.h:98:40: note: expanded from macro 'NAN_DEPRECATED'
# define NAN_DEPRECATED __attribute__((deprecated))
                                       ^
../src/main.cc:46:24: warning: 'Utf8Value' is deprecated [-Wdeprecated-declarations]
  std::string command(*String::Utf8Value(info[0]));
                       ^
/Users/vsts/.node-gyp/10.2.1/include/node/v8.h:2821:5: note: 'Utf8Value' has been explicitly marked deprecated here
    V8_DEPRECATED("Use Isolate version",
    ^
/Users/vsts/.node-gyp/10.2.1/include/node/v8config.h:321:29: note: expanded from macro 'V8_DEPRECATED'
  declarator __attribute__((deprecated))
                            ^
../src/main.cc:63:33: warning: 'Utf8Value' is deprecated [-Wdeprecated-declarations]
    args.push_back(std::string(*String::Utf8Value(js_arg)));
                                ^
/Users/vsts/.node-gyp/10.2.1/include/node/v8.h:2821:5: note: 'Utf8Value' has been explicitly marked deprecated here
    V8_DEPRECATED("Use Isolate version",
    ^
/Users/vsts/.node-gyp/10.2.1/include/node/v8config.h:321:29: note: expanded from macro 'V8_DEPRECATED'
  declarator __attribute__((deprecated))
                            ^
3 warnings generated.
ld: warning: text-based stub file /System/Library/Frameworks//Security.framework/Security.tbd and library file /System/Library/Frameworks//Security.framework/Security are out of sync.


Transform failed in Webpack 5 production mode with vue-loader and babel-loader

When working with vue-loader and babel-loader, the transforms fail. And only in production mode.
Related issue: https://github.com/sxzz/unplugin-vue-define-options/issues/23
this._compiler? $unpluginContext is undefined, it seems there is no inject to compiler?

Since I am in unfamiliar with webpack plugin development, I guess it might be a problem with the inject here.

Here is reproduction: https://github.com/sxzz/unplugin-issue
Please run nr build && nr preview
https://github.com/sxzz/unplugin-issue/blob/3ab41951cb179cd11bef2473a77f3686c4ca4744/vue.config.js#L8
Removes babel-loader and then unplugin works

https://github.com/sxzz/unplugin-vue-define-options/issues/23#issuecomment-1170283222

I met this isuue too, and for me there is nothing to do with thread-loader, I turned it off and the error still.

sxzz/unplugin-vue-macros#23 (comment)

I met this isuue too, and for me there is nothing to do with thread-loader, I turned it off and the error still.

ok, I found the constant file is called by a worker, maybe that's the point.

It turns out to be worker-loader, replace it with new Worker() upon Webpack5 solve this issue.