Wildcarding assets in pubspec.yaml (or allow an entire directory to be included)

I'd like to be able to add an entire folder of assets without having to enumerate each one in my flutter.yaml.

Here is the handling of the assets key, probably pretty easy to add this (and make other improvements to this logic):
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/asset.dart#L343

We're looking to remove .flx files entirely in https://github.com/flutter/flutter/issues/9900 which will likely change how we specify assets.

Still no solution for this? I have a lot of images in my app, and I'm adding more all the time.

Before fixing, we can write a script use your favorite script language (say dart is good choice) to automatically generate the .yaml file.

cc @mit-mit for dev experience

Would be great if we can either add wildcard paths/directories

is there any progress on this... been more than a year now still nothing or has it been put on hold

We've removed FLXes but we haven't yet implemented wildcards. I'll see if we can increase its priority.
We would naturally be happy to review patches if any of you are interested in implementing this before we get to it. :-)

@Hixie has the FLXes removals landed on the dev channels yet?

Really urgent feature, imagine when you have a folder of static web page with more than 3000 files.. :(

@floitschG you can use code generation to add them to the pubspec.yaml file.
Btw: I think they should not be in pubspec.yaml

@zoechi Where you think they whould be put ?

@floriangbh currently they need to be in pubspec.yaml
I just think it wasn't a good idea to require them to be in this file.

Someone please create a simple Dart script that scans a directory, finds all the assets, and adds them to a pubspec.yaml :) You can use this library to parse the YAML file: https://pub.dartlang.org/packages/yaml
The community will really thank you :)

That's not a Dart script, but a simple sh command :
find ./assets -name "*.*" -print | grep '\.' | sed 's/.\/assets/-\ assets/g' >> assets.yaml
In fact , it not resolve my problem, if there are too much file we can't run project : https://github.com/flutter/flutter/issues/15486.. :(

@sethladd I have a POC for such a script, but it's instead scanning pubspec and generating a dart file with static fields, one for each file, to avoid runtime errors. Big static language fan here.
I would love to have it the other way around,
a) no mention at all in pubspec
b) by convention pick folder named assets or whatever other name
c) scan folder for images, json, fonts, etc
d) integrate with whoever used to read that section from pubspec (AssetBundle?)
e) generate dart file with static class / fields for avoiding typing strings in application code
This is probably out of scope for this issue, but I'd like to work on it. I'm mostly interested in the last step, and to make it happen seamlessly I'd need to tap into flutter internal code since there's no way to publish a package and run it as part of a pre-build task.

This really should get done soon

Hope to see this get done soon!

@Hixie , how much work is left for this after #16413 ?

This should be done now.  You can list a folder of assets by having it include a trailing slash.

awesome
On Tue, May 22, 2018 at 10:43 PM Todd Volkert<EMAIL_ADDRESS>wrote:

Closed #4890 https://github.com/flutter/flutter/issues/4890.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/4890#event-1640914177, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA4_ruxWqNsQG1mOsQj7RPGKqnEhZ4dwks5t1PcKgaJpZM4JJ0QF
.

@tvolkert Wilcarding assets is still not possible though? Will that be implemented in a follow-up issue?

No wildcards or subdirectories yet. There are a couple of wrinkles before having them done properly, I plan to be verbose about it on a medium post soon.
Follow ups on that direction are likely on my part but my time is very short, and I'm not part of flutter team so naturally I can't guarantee my ideas will be accepted. Be patient, time frames here were weeks but can get even longer.
Eventually getting the dialog started and showing there's interest will help.

Details on why it does what it does, and why it doesn't what it doesn't (yet). Also there's a shortlist of things related to this work.
https://medium.com/@fmatosqg/contributing-to-flutter-part-3-4b96d924d8ba
One more time, thanks @tvolkert for helping me all along.
cc @MaskyS

According to documentation, now it's possible to add entire assets directory:

To include all assets under a directory, specify the directory name with the / character at the end:

flutter:
  assets:
    - assets/

but it gives analyser error:
warning: The asset assets/ does not exist. (asset_does_not_exist at [my_project] pubspec.yaml:31)

@bunopus seems to be only in master yet. See #16413
You also need flutter/flutter-intellij#2384 which will probably included in the next Flutter IntelliJ plugin update.
Not sure about VSCode @DanTup do you know about that?

@zoechi thanks! Found related bug https://github.com/dart-lang/sdk/issues/33550
Ignore it for now, but it's really confusing, because it's in the docs on https://flutter.io/assets-and-images/

Looks like the fix is in the analysis server, so VS Code doesn't require any changes. You'll get the fix when you get a new Flutter that has a new SDK in it (same as for IntelliJ) 😄

@DanTup hey I was going through https://github.com/dart-lang/sdk/issues/33550 but that's a workaround. Are you aware of a definitive fix for this? if not, I'd like to open new ticket/reopen the above one, and in the mean time add the workaround above in the documentation as a PR.

@fmatosqg The issue is fixed (https://github.com/dart-lang/sdk/commit/d895b4913420036b24110376367946f6195f478b) - the workaround is a temporary fix you can use until you get a version of Flutter that includes the fix.

@DanTup awesome :)
Is there a easy way to know when that change trickles down to flutter master, without checking dart commit hash with flutter --version and cloning dart to track it? I understand the idea but not sure if there's an easy way to know, and occasionally people come to me asking about this issue.
I understand that is not usually needed though.
Bot idea: register your email for when a given dart/flutter commit reaches master / dev / beta

@fmatosqg If you click on the changeset above, you can see the tags shown on GitHub:

The oldest one there is the one you care about (dev64 in this case).
When you run flutter --version, it includes the Dart SDK version:
dantup-macbookpro:Dart-Code dantup$ flutter --version
Flutter 0.5.7-pre.83 • channel master • https://github.com/DanTup/flutter.git
Framework • revision b931640c1d (10 hours ago) • 2018-07-10 06:51:12 +0100
Engine • revision 6fe748490d
Tools • Dart 2.0.0-dev.63.0.flutter-4c9689c1d2 <------

You can see this one says dev.63. So it's not in master yet, but you should be able to tell when it is fairly easily.

Bot idea: register your email for when a given dart/flutter commit reaches master / dev / beta

Yeah, a generic GitHub bot you can mention on an issue that pings you when it hits a release would be awesome; but hard to implement (hard to generically tie issues to releases). A more specific one (eg. just Flutter) would be easier, but less awesome!

Guys, the only issue with this new implementation is that it only gets the repository mentioned, it would be good to be able to do this recursively since I'm trying to package a web application there during the migration to flutter.
Something like assets/**/ or assets/**/*.

Hey,
You can read the comments on the pr, there I explain why the approach you mention is a big risk for breaking backwards compatibility in the future.
Also, web! = flutter. There are assumptions made in the specs that are unique to flutter.

Hi, all:
I'm a member of Fly-Mix Team from NetEase company. My team did develop a flutter development tool: Flr(Flutter-R) to support the feature that you want.
Flr is a flutter resource manager tool, which can help flutter developer to auto specify assets in pubspec.yaml and generate r.g.dart file which likes R.java in Android after he changes the flutter project assets. With r.g.dart, flutter developer can apply the asset in code by referencing it's asset ID function, such as:
import 'package:flutter_r_demo/r.g.dart';

// test_sameName.png
var normalImageWidget = Image(
  width: 200,
  height: 120,
  image: R.image.test_sameName(),
);

// test_sameName.gif
var gifImageWidget = Image(
  image: R.mage.test_sameName_gif(),
);

// test.svg
var svgImageWidget = Image(
  width: 100,
  height: 100,
  image: R.svg.test(width: 100, height: 100),
);

// test.json
var jsonString = await R.text.test_json();

// test.yaml
var yamlString = await R.text.test_yaml();

PS: yes, it is just like the AAPT(Android Asset Packaging Tool) in Android.
Up to now, Flr has supported Android Studio Plugin, CLI, and VSCode Extension:

Flr Android Studio Plugin Version

GitHub:  https://github.com/Fly-Mix/flr-as-plugin

Plugin Homepage: https://plugins.jetbrains.com/plugin/13789-flr

flr-as-plugin Usage Example Gif:

Flr CLI Version

GitHub: https://github.com/Fly-Mix/flr-cli

CLI Homepage: https://rubygems.org/gems/flr

flr-cli Usage Example Gif:

Flr VSCode Extension Version

GitHub: https://github.com/Fly-Mix/flr-vscode-extension

Extension Homepage: https://marketplace.visualstudio.com/items?itemName=LincolnLaw.flr

flr-vscode-extension Usage Example:

Hey guys. Can we expect this to be implemented in the foreseeable future? There are numerous cases I can think of, in which an asset directory along with its sub directories and the files (or short: recursive definition of asset files) is totally necessary. It's very cumbersome to let an external tool do that.


Wiki pages for Photos - Yemen project

Regarding photos that do not say clearly what the building or place is e.g. a photo of an un-named mosque or other building/s - I am using Google images to identify them by looking at similar images - and then using that information to identify the wiki page and/or location. Sometimes, when identified, the specific building/s in Yemen may not have a wikipedia page, but there may be a wikipedia page for the Governorate. The Nasr Palace in Al Hudaydah (identified through Google Images) does not have its own wiki - but I used the Al Hudaydah goverorate page as the wikipedia page. The wikipedia page does not refer to the Nasr Palace but if the wiki page were updated it would be appropriate. In this case, while there is no Wiki page for the Nasr Palace, I have provide the Al Hudaydah wikipedia page as the reference as it is located in that Governorate. Any thoughts?

@tagapic in this case you can use your best judgement, if you think the wiki page is relevant, even if maybe the building is not mentioned you can put it in. Otherwise you can also leave it empty

Thanks, that is fine then.


Question about evaluation

Hi, thanks for awesome repo!
I am a new to keyphrase extraction and would like to know whether I should evaluate my own model on test.author.json, test.readers.json or test.combined.json on NUS/SemEval2010/etc. On the other hand, there are test.contr.json and test.uncontr.json on Inspec. In your MultipartiteRank paper, you used NUS and SemEval2010. Which json file did you used to evaluate your model in the paper?
thanks.

Hi @tagucci,
In the MultipartiteRank paper, and as in most previous works, the test.combined.json (i.e. authors+readers) and the test.uncontr.json (uncontrolled vocabulary) are used for SemEval-2010/NUS and Inspec evaluation.
Best regards,


Updated IMU Constants

Afaik there is no MPU6050_SF constant

Afaik there is no MPU6050_SF constant

It was added recently
https://github.com/SlimeVR/SlimeVR-Tracker-ESP/blob/main/src/consts.h#L64

Afaik there is no MPU6050_SF constant

It was added recently https://github.com/SlimeVR/SlimeVR-Tracker-ESP/blob/main/src/consts.h#L64
This Constant is there to be able in Firmware to select the fusion.

To the Server class ImuID is responsible.
https://github.com/SlimeVR/SlimeVR-Tracker-ESP/blob/main/src/consts.h#L28

please update the branch

please update the branch

Branch updated


Question: Has anyone been able to use azure stream analytic to trigger the device bridge function?

We haven't performed this specific test yet, but device-bridge is in essence an HTTP triggered JavaScript Azure Function. This doc describes how to trigger an Azure Function from Azure Stream Analytics, but the examples are for C# functions, You have to check it the same setup can be used with JavaScript functions.


Text markdown support

hi, can text support markdown on text messages? I want use it build chatbot and should need parse markdown response from ChatGPT

see #87  for adding custom markdowns or styles, either you can use flutter_markdown also


Login and Popup Blockers

$facebook.login calls FB.login using a promise, which means it'll be on nextTick. That means the popup is most likely to be blocked.
It needs to be called directly before the then.

Have you faced a uses cases that the login-popup was blocked? (by far I haven't)

I am actually facing the problem right now with Chrome 40.0.2214.89 on Android 4.2.2 and it is really frustrating.
@elado could you provide some guidance or a pull request to fix the problem? It would help a lot.

I'm facing this all the time unless I allowed pop ups on a certain page.
My current hack is (pardon my coffee):
# HACK! open popup immediately instead of nextTick
if $facebook.promise.$$state.status > 0 && $window.FB
  $facebook.promise.then = (onFulfilled) ->
    onFulfilled($window.FB)

and other than that, usage as regular.
The real solution would be to not use promises as they use nextTick/setTimeout.

thanks @Elad!
Would you honor us with PR? :P

http://www.SocialBelong.com[image: Like us on facebook]
https://www.facebook.com/SocialBelong[image: follow us on linkedin]
https://www.linkedin.com/company/socialbelong*

*Almog Baku, *CEO
Call me:                        +972-50-2288-744Give me a reply:
<EMAIL_ADDRESS><EMAIL_ADDRESS>me to your:
[image: facebook] http://www.facebook.com/AlmogBaku  [image: linkedin]
http://www.linkedin.com/in/almogbaku
Q: Why is this email five sentences or less?
A: http://five.sentenc.es
On Tue, Apr 28, 2015 at 7:05 PM, Elad Ossadon<EMAIL_ADDRESS>wrote:

I'm facing this all the time unless I allowed pop ups on a certain page.
My current hack is (pardon my coffee):
HACK! open popup immediately instead of nextTickif $facebook.promise.$$state.status > 0 && $window.FB
$facebook.promise.then = (onFulfilled) ->
onFulfilled($window.FB)
and other than that, usage as regular.
The real solution would be to not use promises as they use nextTick/
setTimeout.
—
Reply to this email directly or view it on GitHub
https://github.com/GoDisco/ngFacebook/issues/42#issuecomment-97120562.

gg @elado
However this is a bit nasty since you redefine $facebook.promise.then() as being a function that does not return a promise (which is the point, I understand) but that violates the Promises/A+ specification. Still, it works... :smile:
@AlmogBaku It worked for me by putting the code outside of ng-facebook but I guess it would also work if you put it at the end of $facebook.init(). I can provide a PR if needed.

I'm not sure im on track here.. but PRs are always welcome

I too am actually having this as an issue. Being blocked in Chrome 45.0.2454.93, as well as IE 10.

I have similar issue. When I call Facebook login function on Firefox, it just stops at the code below and never execute anything inside the promise.
return $facebook.promise.then(function(FB) { ...
After setting a couple of break points and refreshing the pages, it works once, and then it goes back to not working.
I read the post by elado commented on Apr 29, 2015, but not sure what exactly to do to fix the problem. Do we have a proper solution for this issue?

I don't have time to work on that- PRs are always welcome


DD_HISTOGRAM_AGGREGATES and DD_HISTOGRAM_PERCENTILES take comma-separated values, not space-separated

See descriptions of variables in the readme -- both claim "separated by spaces"
Additional environment details (Operating System, Cloud provider, etc):
datadog/docker-dd-agent:12.7.5321-jmx
Steps to reproduce the issue:

set DD_HISTOGRAM_AGGREGATES=max median avg count sum and start the agent

Describe the results you received:
No histograms pushed via dogstatsd. head /var/log/datadog/collector.log in the container gives:
2019-03-21 15:26:31 UTC | WARNING | dd.collector | config(config.py:306) | Ignored histogram aggregate
max median avg count sum, invalid

Describe the results you expected:
Histograms pushed with the given aggregates.
Additional information you deem important (e.g. issue happens only occasionally):
Workaround is to give a comma-separated list.

FYI This is similarly incorrectly described over here:
https://docs.datadoghq.com/agent/docker/?tab=standard#dogstatsd-custom-metrics


threaded CameraView

perform updating and rendering on worker threads.

There is a more efficient way to use QWindow instead of QOpenglWindow, but I haven't found how to display a overlay QWidget on navtive window.


Workflow is getting mixed up between Alfred 2 in Application Support and Alfred 3

I am unable to change the config to /Users/tiffanywhite/Dropbox/RBT/ or to /Library/Application Support/Alfred 3/Workflow Data/com.stuartryan.alfred.rapidbrowsertabs/ as it automatically defaults to /Library/Application Support/Alfred 2/Workflow Data/com.stuartryan.alfred.rapidbrowsertabs/. I have deleted the Alfred 2 folder, yet, RBT recreates it. I've set up the config the way I needed to but no luck.
Not sure if I can fix it with my limited knowledge. Can you fix it?
Thanks.

Same problem here. Doesn't work at all out the box and the config is saved/recreated all the times in Alfred 2 folder even if it doesn't exist.

I am running Alfred 3, and find that RBT does dump its configuration in the /Alfred 2/ folder, as @twhite96 reports, but this isn't breaking things for me: I think the configuration file location is hard-coded, so that the workflow finds its configuration even though it's officially in the wrong place.

Closing this as their hasn't been any conversation about it plus Alfred is on v4


Customizing shortcuts

I'm coming from SublimeGit and shortcuts on status page is a bit different. I could not get used to it.
For example "d" is "diff" in SublimeGit, but in GitSavvy its "discard" shortcut. Everytime I mispress "d" key.
Is there any way to customize shortcuts?

Just look at the sublime-keymap file. You just need to override the keybinding for diff. However, I do suggest you to re-train your muscle memory.

Ok I created this file on .../Sublime Text 3/Packages/GitSavvy path. Then I modified.
It works well, but it still shows the old shortcut in status page. It's not important anyway.

We can add a command to the command panel like Preferences: gitsavvy Settings

It is a bit more complex because there are more than one file specifying keybinds, Default ($platform).sublime-keymap and Default.sublime-keymap.


ImportError: cannot import name filter

I have installed python-ldap and flask-simpldap. I get this error when trying to run my flask app

File "/Library/Python/2.7/site-packages/flask/cli.py", line 90, in locate_app
import(module)
File "/Volumes/Data/My Projects/flask/myldap.py", line 1, in
import ldap as l
File "/Volumes/Data/My Projects/flask/ldap.py", line 3, in
File "/Library/Python/2.7/site-packages/flask_simpleldap/init.py", line 5, in
from ldap import filter as ldap_filter
ImportError: cannot import name filter

I have tried "import ldap.filter" on the python cli,that doesn't work either
I am using python 2.7.10 .
Has this function been deprecated ?

Uninstalled pyldap and python-ldap.Then reinstalled python-ldap
using these instructions.
https://www.python-ldap.org/faq.html#install_macosx
No import errors now.


Add Support for iOS Targets

Additional targets can likely be added via cc-tools, which should enable cross-compilation of iOS, TVOS, and Apple Catalyst.

This should likely be based off of the code for the cctools-port here, which has an example iostoolchain.


Update theme colors from the design system

Overview
@liliiaorlenko is working on revamping our design system. The first step, colors, is already finished.
We need to cross-check the colors from the new design system, and the ones we use in the app.
Wherever we specify a CSS color, it should come from the theme.
Goals
Make the CSS colors consistent with the design.
Requirements

Go through all components and check that they are using colors from the theme and not hardcoded hexes
Make sure the colors in the theme are the same as in the design system

Screens

Figma: https://www.figma.com/file/uqDJqIGikIB1zVs56XKnIu/Design-System---Web?node-id=347%3A2581

Here is a useful guide for extending themes in MUI that I followed before.
We also need to update any hardcoded colours throughout the project as well. However, that could be another issue. What do you think @katspaugh?

When difficult to migrate to the theme (old MUI classes), we could at least replace the hardcoded colors with constants.


feat(java-binding): generate jni method signature with macro

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
In this PR, we implement a macro gen_jni_sig that generates the jni signature from the java method signature. With this macro, we can avoid manually write the list of jni native and signature list when we register native method.
Besides, since we provide a register native function, in the java binding dynamic library, we don't have to export each jni method one by one. We change to provide a single method JNI_OnLoad, which will be called and register the jni method when the dynamic library is loaded.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

LGTM! These macros are amazing! Maybe we can remove the prefix Java_com_risingwave_java_binding_Binding_ so that we can have the same native method names in Rust.

Find a crate for this -> https://crates.io/crates/jni_fn

Find a crate for this -> https://crates.io/crates/jni_fn

Just had a look at the crate. It aims to convert the method name to a name that is able to automatically recognized by jvm when loading the dynamic library. However, in this PR, we have changed to register the native methods manually instead of via a predefined name mapping rule, and therefore it's unnecessary for us to maintain the method name according to the rule with the macro in the crate.


[BUG] Unable to save timeseries document: lock for timeseries is not allowed

Describe the bug
Document.save() fails when using a timeseries collection, returning pymongo.errors.OperationFailure: Taking test.timeseries_test lock for timeseries is not allowed, full error:
{'ok': 0.0, 'errmsg': 'Taking test.timeseries_test lock for timeseries is not allowed', 'code': 166, 'codeName': 'CommandNotSupportedOnView'}

To Reproduce
with beanie==1.26.0, pymongo==4.8.0, and motor==3.5.1 on python 3.11.5
from beanie import init_beanie, Document, TimeSeriesConfig
from motor.motor_asyncio import AsyncIOMotorClient
from pydantic import Field

from datetime import datetime
from asyncio import run

MONGO_URI = ''
DATABASE = ''

class TimeSeriesTest(Document):
    ts: datetime = Field(default_factory=datetime.now)

    class Settings:
        name = 'timeseries_test'
        timeseries = TimeSeriesConfig(
            time_field='ts',
            expire_after_seconds=60
        )

async def main():
    _client = AsyncIOMotorClient(
        MONGO_URI,
        serverSelectionTimeoutMS=5000
    )[DATABASE]

    await init_beanie(_client, document_models=[TimeSeriesTest])

    ts_test = TimeSeriesTest()

    await ts_test.save()

if __name__ == '__main__':
    run(main())

Expected behavior
Document is successfully inserted
Additional context
The code above does successfully insert when not using a timeseries collection

Am having the same issue.  Is there any recommended work around other than just converting to a non time series collection?

For using a time series collection, I wasn't able to find one, however for my specific use-case, just a regular collection with a TTL index fit my needs, although it's less efficient.


MediaProjection Permission: Popup for Screen Recording Permission is invisible

Steps to reproduce
Android offers the MediaProjection API to record the system screen: https://developer.android.com/guide/topics/large-screens/media-projection
This API can be triggered by calling
startActivityForResult(mediaProjectionManager.createScreenCaptureIntent(), REQUEST_MEDIA_PROJECTION);
When the API is triggered, Android normally shows a popup to obtain the runtime permission for screen recording. In WSA, this popup is loaded, but it is not visible for the user, so there is no way to grant the permission.
Although the popup is not visible, we have found that it is loaded, because the app screen brightness is lowered (this is normaly behavior for when the popup appears). We have also found a workaround to grant the permission: Use the windows keyboard and press the right cursor 2x and then ENTER.
You can reproduce this issue for example with the "Screen Mirroring App" on the Amazon Apps Store: https://www.amazon.com/-/de/dp/B099KKC54B
✔ Expected Behavior
Permission Popup for screen recording should be visible in the app.
❌ Actual Behavior
Permission Popup for screen recording should is not visible in the app.
Other Software
No response
Please specify the version of Windows Subsystem for Android
2301.40000.7.0

We'll investigate and follow up on this. Thank you for bringing this to our attention.

Are there any updates regarding this issue?
Thanks.


Update faq-defender-for-storage.yml

Updated again to provide wording on Malware Scanning

#sign-off


Dynamically generate methods on concrete classes only

Avoid creating public methods on instances of the base class itself, so that derived types don't interfere with each other

@etagwerker @benphelps @mscottford can we get some response to this? Seems like a few PRs have stalled.

I'm waiting for your other requests to be merged before I continue on with anything of mine as changes will be required on my end.

@gap777 hey, can you fix the conflicts in this PR so we can merge it? Thanks!

@gap777 Great, thank you! 👍


Lower required Node.js version to 8

Thanks for taking the time to file a pull request with us.
Please take a moment to provide the following details:
Node.js 8 is still in maintenance and it's used in services like Firebase. This package works with Node.js 8.16.2 without any problem, the engine requirement seems to be related only to build tools (it was introduced in https://github.com/researchgate/react-intersection-observer/commit/fff9c526690e84c85c3f3d956b629e09e3bebde6)
Description
Restore previous Node.js requirements - ^8.10 and >9.10.
Unfortunatelly, Node.js 10.x is required for tests. Maybe the engine requirements might be lowered in package.json right before publishing to npm?
Motivation and context
Node.js 8 is still in maintanence LTS period.
How has this been tested?
I installed the package locally and it works fine.
Type of change

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

Checklist:

[ ] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the CONTRIBUTING document
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

I've found related issue which was already closed: https://github.com/researchgate/react-intersection-observer/issues/63
Since it doesn't solve my problem, I'm gonna use a local copy of this library until we migrate to Node.js 10.
Thank you and sorry for noise.


closing FAQ entries

prevent FAQ entries from closing to easily. Now you need to click on the arrow or headline to close again.

Thanks!
cc @carlopi who originally reported this issue


NIST PubID scheme: “Part” representation sometimes goes with “p” or “pt” (always "pt"?)

"NIST SP 800-57pt1r4” vs "NIST SP 800-57p3"

We propose to always use pt for consistency.

The final PubID scheme (#153 ) confirms that pt is always used in 2.1.4.

@mico is this what we have implement now? If so, please close this, thanks.

We always render part as "pt".


Datetime attribute needs more configuration options

This is a feature request.  A datetime attribute should include the same default option as the form block, where you can have it default to the current date or be blank.  Additionally, there should be an option to allow minutes to only show every 5 minutes, every 10, every 15, every 30, and only even hours.  Right now it shows 00-59.  Allowing the control to show 00, 15, 30, 45 (or 00, 10, 20, etc as needed by the developer) instead improves usability.

That would be a nice improvement! :+1:

Sounds like a nice addition.

What about something like this?

In case we have "1 minute" we'll have the current behavior (we ask only hours and minutes, but no seconds).
In case we have "15 minutes", we'll ask only hours and the minutes "00", "15", "30" and "45".
The same applies for seconds and hours.

That'd be really nice.

#5052 implements this.


Unable to use date in the partitioning key

Elasticsearch version: 6.2.3
Plugins installed: []
JVM version (java -version): openjdk version "1.8.0_181"
OS version (uname -a if on a Unix-like system): 05 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Description of the problem including expected versus actual behavior: Unable to use date as a part of the partitioning key. Reason is stated as: "Unable to make unsigned int (for date) from: '-2147465869'". When date is removed from partitioning key in the example replicated below, the error does not occur.
Steps to reproduce:

In cqlsh:

CREATE KEYSPACE example WITH REPLICATION =

{ 'class':'NetworkTopologyStrategy', 'DC1':2 };

CREATE TABLE example.sessions (

  id timeuuid,

  project_id uuid,

  day date,

  PRIMARY KEY ((project_id, day), id)

) WITH CLUSTERING ORDER BY (id DESC);

INSERT INTO example.sessions (id, project_id, day)
VALUES (
	now(),
	uuid(),
	toDate(now())
);

curl -H "Content-Type: application/json" -XPUT "http://localhost:9200/example?pretty"
curl -XPUT "http://localhost:9200/example/_mapping/sessions" -H 'content-type: application/json' -d '{ "sessions" : { "discover" : ".*" }}'
In browser, seach http://localhost:9200/example/sessions/_search?&q=*:*&error_trace=true

Provide logs (if relevant):

Hello @Piersen,
You're right, thanks for reporting the issue. The mapping of CQL type date is buggy.
As a workaround you can use the CQL type timestamp rather than date, and slightly modify your insert statement with toTimestamp(toDate(now())).

Fixed in <IP_ADDRESS>


k8s example fails on kubectl requirement step (Temporary failure resolving 'apt.kubernetes.io')

Description
limactl version:
$ limactl -v
limactl version 0.8.2

I ran limactl start https://raw.githubusercontent.com/lima-vm/lima/master/examples/k8s.yaml, chose to Proceed with the default configuration, and by the end of the process Kubernetes was running successfully.
After stopping and deleting the lima instance using limactl, I then tried to restart the Kubernetes cluster using the same command above. Logs from limactl were as follows:
$ limactl start https://raw.githubusercontent.com/lima-vm/lima/master/examples/k8s.yaml

? Creating an instance "k8s" Proceed with the default configuration
INFO[0001] Attempting to download the image from "https://cloud-images.ubuntu.com/focal/current/focal-server-cloudimg-arm64.img"  digest=
526.75 MiB / 526.75 MiB [----------------------------------] 100.00% 25.90 MiB/s
INFO[0023] Downloaded image from "https://cloud-images.ubuntu.com/focal/current/focal-server-cloudimg-arm64.img"
INFO[0023] Attempting to download the nerdctl archive from "https://github.com/containerd/nerdctl/releases/download/v0.16.1/nerdctl-full-0.16.1-linux-arm64.tar.gz"  digest="sha256:e47fd1a03545ce539005a7f8a29644cb2ab515012c1c82b8be0bbafbb1670699"
168.20 MiB / 168.20 MiB [----------------------------------] 100.00% 43.42 MiB/s
INFO[0028] Downloaded the nerdctl archive from "https://github.com/containerd/nerdctl/releases/download/v0.16.1/nerdctl-full-0.16.1-linux-arm64.tar.gz"
INFO[0029] [hostagent] Starting QEMU (hint: to watch the boot progress, see "/Users/dhelfand/.lima/k8s/serial.log")
INFO[0029] SSH Local Port: 61175
INFO[0029] [hostagent] Waiting for the essential requirement 1 of 3: "ssh"
INFO[0059] [hostagent] Waiting for the essential requirement 1 of 3: "ssh"
INFO[0059] [hostagent] The essential requirement 1 of 3 is satisfied
INFO[0059] [hostagent] Waiting for the essential requirement 2 of 3: "user session is ready for ssh"
INFO[0099] [hostagent] Waiting for the essential requirement 2 of 3: "user session is ready for ssh"
INFO[0100] [hostagent] The essential requirement 2 of 3 is satisfied
INFO[0100] [hostagent] Waiting for the essential requirement 3 of 3: "the guest agent to be running"
INFO[0100] [hostagent] The essential requirement 3 of 3 is satisfied
INFO[0100] [hostagent] Waiting for the optional requirement 1 of 5: "systemd must be available"
INFO[0100] [hostagent] Forwarding "/run/lima-guestagent.sock" (guest) to "/Users/dhelfand/.lima/k8s/ga.sock" (host)
INFO[0100] [hostagent] The optional requirement 1 of 5 is satisfied
INFO[0100] [hostagent] Not forwarding TCP <IP_ADDRESS>:53
INFO[0100] [hostagent] Waiting for the optional requirement 2 of 5: "containerd binaries to be installed"
INFO[0100] [hostagent] Not forwarding TCP <IP_ADDRESS>:22
INFO[0100] [hostagent] Not forwarding TCP [::]:22
INFO[0112] [hostagent] The optional requirement 2 of 5 is satisfied
INFO[0112] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0115] [hostagent] Forwarding TCP from <IP_ADDRESS>:39313 to <IP_ADDRESS>:39313
INFO[0152] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0193] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0233] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0273] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0313] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0353] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0393] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0433] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0473] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0514] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0554] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
INFO[0594] [hostagent] Waiting for the optional requirement 3 of 5: "kubectl to be installed"
FATA[0628] did not receive an event with the "running" status

From log file:
+ apt-get install -y kubelet kubeadm kubectl
Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
   conntrack ebtables socat
 Suggested packages:
   nftables
 The following NEW packages will be installed:
  conntrack ebtables kubeadm kubectl kubelet socat
 0 upgraded, 6 newly installed, 0 to remove and 9 not upgraded.
 Need to get 32.4 MB of archives.
 After this operation, 215 MB of additional disk space will be used.

 LIMA| WARNING: Failed to execute /mnt/lima-cidata/provision.system/00000000
 LIMA| Executing /mnt/lima-cidata/provision.system/00000001
 + test -e /etc/kubernetes/admin.conf
 + export KUBECONFIG=/etc/kubernetes/admin.conf
 + KUBECONFIG=/etc/kubernetes/admin.conf
 + kubeadm config images list
/mnt/lima-cidata/provision.system/00000001: line 5: kubeadm: command not found
LIMA| WARNING: Failed to execute /mnt/lima-cidata/provision.system/00000001
LIMA| Exiting with code 1

Unfortunately we haven't found a good solution for caching yet, so you would have to fix your internet (and dns)
   Temporary failure resolving 'apt.kubernetes.io'

Is this still a problem, or did get the DNS issues get fixed in later versions of lima (and friends) ?
Not sure what else do to otherwise, except from maybe caching packages and images on the host.

~~https://github.com/lima-vm/lima/issues/397~~

It needs to be able to resolve apt.kubernetes.io and k8s.gcr.io

For the first issuing of limactl start k8s, while cloud-init script is running apt-get update command.
I got the error message that "...connect (101: Network is unreachable) Could not connect to:"

security.ubuntu.com:80
archive.ubuntu.com:80 and
pkgs.k8s.io:443

Lima v0.20.1
QEMU v8.2.1
MacOS 10.15.7 (Catalina)
WORKAROUND : Once limactl returned prompt, just simply stop and start the instance.


Avoid Clone bound on Message

As dmarcuse mentioned in #77 if you have some message variants with uncloneable contents you currently end up having to wrap things in Arc which isn't very ergonomic. Having widgets accept Fn instead of Message avoids this issue.

I am not convinced we should make the overall API more complicated because of the Arc use case.
You can currently have non-cloneable messages just fine by separating your UI interactions from your non-cloneable messages. I detailed how to do this here: https://github.com/hecrj/iced/pull/155#issuecomment-573523405

To me, the API doesn't really seem any more complicated - it's just a closure as opposed to a value. I think making it more consistent with other widgets (slider, text input, etc) which accept functions makes the API more ergonomic as well. As you noted here it is possible to define a second message type that satisfies the Clone bound, but this isn't really obvious (at least not to me) and takes a lot more effort than it would to use closures.
I understand that having a way to introduce non-determinism is a goal of the project, but seeing as other widgets already use closures, I don't think it's worth sacrificing ergonomics for.

Thanks for pointing out that alternative way to structure Messages. I think that will work for my case. However I do agree with dmarcuse that using closures is a much more obvious/discoverable way to allow this. If you're determined not to use closures for this would you be willing to accept a PR adding an example of the pattern discused in https://github.com/hecrj/iced/pull/155#issuecomment-573523405 (assuming I can think of a non-contrived example)?

it's just a closure as opposed to a value

I think this is a big distinction. There is no need for a widget to accept a closure with no arguments just so we can construct many messages. A Clone bound is exactly meant to convey this!

I think making it more consistent with other widgets (slider, text input, etc) which accept functions makes the API more ergonomic

I think it would be inconsistent!  Most of the time, you are not even meant to use the closure syntax:
let text_input = TextInput::new(
    state,
    "Type something to continue...",
    value,
    StepMessage::InputChanged,
)

Forcing our users to type on_press(|| Message::ButtonPressed) just for buttons does not seem right! A different user could ask: why does on_press accept a closure instead of requiring Clone? In the end, this seems very subjective to me.

takes a lot more effort than it would to use closures

Simplicity isn't always convenient. Structuring your Message type properly is key in an application that uses the Elm Architecture.
Moreover, there are also some long term features that will force messages to satisfy Clone, like a time travel debugger. Although we may add that as a requirement in a different trait, I believe it's something we should keep in mind: there are many reasons why a Clone bound may be necessary and giving it up at the root level will have consequences in the long run.

If you're determined not to use closures for this would you be willing to accept a PR adding an example of the pattern discused in #155 (comment) (assuming I can think of a non-contrived example)?

Of course! Feel free to join the Zulip server and share your ideas there!


TimeSignatures-Compound tests broken

@jaredjj3 it seems that #86 has broken the compound timesignatures.
I believe that the problem is that beat should be a string and not a number
https://www.w3.org/2021/06/musicxml40/musicxml-reference/elements/beats/

Thanks for raising this. Sorry about that. Feel free to rollback all the commits from #86 or patch a fix.


keyboard shortcuts does not work

ctrl+c or crtl+v and etc. are not working in the app (at least in dev mode)

May be related to #96 as well

Can you have a look at this? https://github.com/tauri-apps/tauri/issues/2397
For me, windows and linux got shortcuts working.

Thanks Leo, that looks promising, adding menu seems to be the right solutions, since the topbar menu is not working in mac as well. I think it will solve many problems at once for mac


Where is the train.py and the model code?

here i want to reproduce the results and modify the model in the paper, however, i can not get the model file from the repo, also i feel confused that only preprocess python files are provided..

Hi @HC-Guo, thanks for your interest in our work.
The model comes directly from OpenNMT, and you can run the training with the onmt_train command, following the instructions in the README: https://github.com/rxn4chemistry/rxn-standardization#training.

thanks, i will try.


This is my malformed Yaml (this is an ansible playbook):
---
- name: Ensure directories exists
    file: ## this block is indented although it shouldnt
      state: directory
      owner: root
      group: www-data
      mode: 0750
      dest: "{{ item }}"
    with_items:
      - /etc/nginx/scripts/
      - /etc/nginx/common/
      - /etc/nginx/conf.d/

All right I found the issue:
https://github.com/sds/overcommit/blob/master/lib/overcommit/hook/pre_commit/yaml_lint.rb#L14
This should probably look like
return [ :error, result.stdout]

All right I found the issue:
https://github.com/sds/overcommit/blob/master/lib/overcommit/hook/pre_commit/yaml_lint.rb#L14
This should probably look like
return [ :error, result.stdout]


Fix typo

The example usage is not working, substitutions has to be in lowercase

@FedorSherbakov Thank you for your first contribution to webpack documentation.


ENYO-3244: Use strict equality check for content.

Issue
For cases where we are changing the content from one falsy value to another (i.e. from the empty string '' to 0), the content does not update due to the equality check.
Fix
I could not think of any cases where we would want to guard against updating the content when switching from one falsy value to another, so the guard has been changed to a strict equality check.
Enyo-DCO-1.1-Signed-off-by: Aaron Tam<EMAIL_ADDRESS>

Going to close this one as WNF.


azure manual install

Hello,
I try to install flynn on azure using manual install. I am not usinz the cloud installer.
After i follow all the steps in the documentation, open all ports inbound and outbound (for test purposes) in azure, i get this error when i try to bootstrap the cluser :
20:46:56.525491 wait-hosts wait-hosts error: bootstrap: timed out waiting for hosts to come up
The following hosts were unreachable:
the debug information cand be found here :
https://gist.github.com/c7e7f648809b25ac6f6267b47cba28cb
I don;t know what to do next. All the servers are up.
Thx.

Can you try deploying again? It might be a transient failure. We'll take a look at the logs.

I tried using ssh from flynn installer. I have the same error.

@pdaniel-frk You can use the Azure template the installer uses for launching a cluster for reference if you want a custom stack. Though using the installer to launch it for you would be the easiest option.

Hello, Thx for the suggestion. My problem is that my company has multiple azure accounts. I want to be able to start 1 machine in one account, another machine in the second account and a third machine in the third account, and install flynn on this machines, in different accounts. This is a budget limitation, being multiple companies under a group.

Can i see in any way how many machines are online in a cluster? Or can i see if i get network/port errors when my machines are communicating with etcd?

Yes, flynn-host list will show a list of hosts and their Raft status.

I did a telnet
telnet  1113 to see if i can connect. Ports are open. I really don't understand why the cluster nodes cannot communicate with each other.

after i executed the bootsrap command  i got

From the original log, it looks like flannel is not coming up correctly:

Error looking up interface <IP_ADDRESS>: No interface with given IP found

What flags did you provide to flynn-host init?

the same problem i get on google compute engine
19:32:42.676590 wait-hosts wait-hosts error: bootstrap: timed out waiting for hosts to come up
The following hosts were unreachable:
<IP_ADDRESS>:1113
<IP_ADDRESS>:1113
<IP_ADDRESS>:1113

this are all the commands i've executed. The ip's are changed because i recreated the VM's
sudo bash < <(curl -fsSL https://dl.flynn.io/install-flynn)
sudo flynn-host init --init-discovery --external-ip=<IP_ADDRESS>
sudo flynn-host init --external-ip=<IP_ADDRESS> --discovery https://discovery.flynn.io/clusters/aeb0e134-8c68-4f88-99a2-b21c93a4cb62
sudo flynn-host init --external-ip=<IP_ADDRESS> --discovery https://discovery.flynn.io/clusters/aeb0e134-8c68-4f88-99a2-b21c93a4cb62
sudo start flynn-host
sudo
CLUSTER_DOMAIN=mysubdomain
flynn-host bootstrap
--min-hosts 3
--discovery https://discovery.flynn.io/clusters/aeb0e134-8c68-4f88-99a2-b21c93a4cb62

So, i have some results.
After @titanous comment, i had an idea. In azure i configured the virtual machines to have the same public ip and private ip. After i did that, everything worked. So it seems there is a problem with flannel and multiple hosts that are not in the same network. Also the remote hosts do not have the public ip on a network interface attached to the vm. This is a problem i believe with all the iaas providers.

Currently Flynn doesn't make a distinction between advertised addresses and listen addresses in all system services and as such needs the 2 to be equal. This means that nodes should be reachable from other nodes by the addresses they are listening on.
I recommend deploying Flynn on a single private network. If you need geo-redundancy deploy multiple clusters.


updated gatsby 2.25.0

Firebase build error. So I updated Gatsby.
 ERROR #11329

Your plugins must export known APIs from their gatsby-node.js.

See https://www.gatsbyjs.org/docs/node-apis/ for the list of Gatsby node APIs.

- The plugin<EMAIL_ADDRESS>is using the API "pluginOptionsSchema" which was introduced in<EMAIL_ADDRESS>- The plugin<EMAIL_ADDRESS>is using the API "pluginOptionsSchema" which was introduced in<EMAIL_ADDRESS>- The plugin<EMAIL_ADDRESS>is using the API "unstable_shouldOnCreateNode" which was introduced in<EMAIL_ADDRESS>- The plugin<EMAIL_ADDRESS>is using the API "pluginOptionsSchema" which was introduced in<EMAIL_ADDRESS>- The plugin<EMAIL_ADDRESS>is using the API "pluginOptionsSchema" which was introduced in<EMAIL_ADDRESS>
Some of the following may help fix the error(s):

- npm install<EMAIL_ADDRESS>- Rename "unstable_shouldOnCreateNode" -> "onCreateNode"

For a blog, its not as important.

I got Firebase deploy error again.
It sayserror<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=14.15.0". Got "12.22.7", but it's v15.8.0 when I check nvm ls. Should I update node to the latest one? @ssagli

Are you deploying locally or on GitHub?
On Thu, Nov 18, 2021, 17:29 Teru @.***> wrote:

I got Firebase deploy error again.
It sayserror @.***: The engine "node" is
incompatible with this module. Expected version ">=14.15.0". Got "12.22.7",
but it's v15.8.0 when I check nvm ls. Should I update node to the latest
one? @ssagli https://github.com/ssagli
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/1f8/blog/pull/12#issuecomment-972642198, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4SYDPCJAOOBI4CRDFLFVLUMS2OXANCNFSM5IIX6AEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

On GitHub @kwa-z

Should remove the node restriction then.
On Fri, Nov 19, 2021, 11:09 Teru @.***> wrote:

On GitHub @kwa-z https://github.com/kwa-z
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/1f8/blog/pull/12#issuecomment-973661960, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4SYDIP5HJLBSHLCSV2DPTUMWWXJANCNFSM5IIX6AEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Smart Experimental Data Design Config

LHS implementation, as well as other pyDOE implementations do not expect a random seed, thus I just tested the number of samples generated rather than the names since it is random. Here is the underlying code: https://github.com/tisimst/pyDOE/blob/master/pyDOE/doe_lhs.py#L21

@uakyol the code structure makes sense to me and parameters seem reasonable. However, I would still prefer a simple test case based on this PR so I can dig more.
Apologies that this PR has not being reviewed for a while, I tried to modified the simple test in the readme.md but you've made a few changes since then. Can you share a simple test case here, then I can probably update the ready me as well.
Thanks!


Add another solution to distance-to-zero.

Another solution to the distance to zero problem.  This uses a group by.  I think this one is more clear than the other two, but that could certainly just be to me.

Thank you - more alternatives solutions are definitely a good thing. I can see this approach being more intuitive to people, too.
I think that it's possible to simplify the first two lines to a single line using cumcount():
df['Y'] = df.groupby((df['X'] == 0).cumsum()).cumcount()

This is probably more idiomatic than using the lambda/transform combination (but let me know if I've overlooked anything) - are you happy to make this change and then I'll merge?

Ah!  Good tip on the cumcount!  I'll make the change after work and send your way to merge.

Ok, change made.

Thank you very much!


⚠ WSINGRESOS - RECIBOPAGO has degraded performance

In eb60859, WSINGRESOS - RECIBOPAGO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/servicios/wsRecibosPagosWeb.r) experienced degraded performance:

HTTP code: 200
Response time: 2243 ms

Resolved: WSINGRESOS - RECIBOPAGO performance has improved in 36152ca after 15 minutes.


Error when applying with LinkedIn

INSERT SHORT DESCRIPTION EXPLAINING THE HIGH-LEVEL REASON FOR THE NEW ISSUE HERE.
Current behavior
Cannot seem to apply for positions using the LinkedIn apply feature.  Receive this result on Chrome and Firefox:

Expected behavior
Ability to apply.
Steps to replicate behavior

Go to job:

http://www.fastswitch.com/careerportal/#/jobs/31197

Try applying using LinkedIn

(I tried my profile:  https://www.linkedin.com/in/markpukita

Tried with a different email address and it worked.  Does this present a problem for applicants who are already in our BH database being able to apply for a (new) job?

If you have returning applicants whose candidate records are active in Bullhorn, they should be able to apply as often as desired. However, we are aware of (and are currently investigating) an issue that occurs when an applicant uses an email address that is associated to a deleted candidate record in the associated database. Is it possible your email was added to a test candidate that was later deleted?

Yes.
Mark PukitaCEOFast Switch<EMAIL_ADDRESS>-------- Original message --------
From: kbushkoBH<EMAIL_ADDRESS>Date: 04/11/2016  10:55 AM  (GMT-05:00)
To: bullhorn/career-portal<EMAIL_ADDRESS>Cc: mpukita<EMAIL_ADDRESS>Subject: Re: [bullhorn/career-portal] Error when applying with LinkedIn (#54)
If you have returning applicants whose candidate records are active in Bullhorn, they should be able to apply as often as desired. However, we are aware of (and are currently investigating) an issue that occurs when an applicant uses an email address that is associated to a deleted candidate record in the associated database. Is it possible your email was added to a test candidate that was later deleted?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Thanks @kbushkoBH for tackling this.  :+1:
@mpukita if you have any additional questions feel free to reach out. I'm going to mark this issue as closed.

Dear All,
I am facing the exact same issue while applying for jobs at linked In. I will be highly thankful if anyone could help me out.

I saved the application then went back to the job and clicked Continue, completed applying and submitted successfully.


Keymap Button Export

Feature request
I'm submitting a proposal to @CastagnaIT

[*] new feature
[ ] change an existing feature

Describe your request

I was hoping to export a selected video, using a custom keymap request to the route() function, which would add that specific movie/tv show to the local library.
However, when I viewed the code, I noticed that the common.videoId object needs to be passed to that keymap-export function, and AFAIK there is no way to get the common.videoId object to pass it to that function.
Describe the use case

After adding a keymap value in the keyboard.xml (i.e. XBMC.RunPlugin(plugin://plugin.video.netflix/library/keymapexport)
Clicking on F12 while a list of movies\tv shows is open (and selecting one of them), would add that movie/show to the local library.
Describe an alternative

Development tips

Additional details or screenshots (if appropriate)
Specific details

Screenshots

seems interesting
the only thing is that I have some doubts about updating the data on the screen (because it would be done by a different addon instance)
and the reliability of this system to complete the operations without overlapping when commands are repeated too quickly
when i have time i will try

This is the test update, let me know:
plugin.video.netflix_1.9.1_20200927_t1.zip
the endpoind are:
Export to library
RunPlugin(plugin://plugin.video.netflix/keymaps/lib_export)
Remove from library
RunPlugin(plugin://plugin.video.netflix/keymaps/lib_remove)

Now that works like charm, thanks for the help much appreciated.


load-secrets-action doesn't work on windows

Background
OpenDAL adopts load-secrets-action to manage all secrets. We are trying to load secrets to build our artifacts on linux/macos/windows platform. Java maven will use thoes secrets to sign the packages.
Expected
Secrets loaded as on linux platform.
Actual
Meeting errors like the following:
Run 1password/load-secrets-action@v1
"C:\Program Files\Git\bin\sh.exe" -c D:\a\_actions\1password\load-secrets-action\v1/entrypoint.sh
/usr/bin/bash: line 1: D:a_actions1passwordload-secrets-actionv1/entrypoint.sh: No such file or directory
Error: The process 'C:\Program Files\Git\bin\sh.exe' failed with exit code 127

See action at: https://github.com/apache/incubator-opendal/actions/runs/5367357180/jobs/9737444126?pr=2525

Hey there.
Currently this GitHub Action is not currently supported on Windows.
Thank you for sharing that you'd like to see this work on Windows as well. We will keep you updated when there's any progress on this.

+1 - it is poor that this wasn’t designed to be cross-platform from the start.
I was interested in adopting this action, but I validate cross-platform compatibility of my code on Linux, macOS and Windows, which makes this action unusable for my purposes as it isn’t a drop-in replacement for native GitHub secrets.

+1 Are there any plans to fix this? Thanks!

any chance this is going to be mitigated? I'd love to include it in my graal compiling runners for windows builds.

This was an unpleasant surprise :(

Cloud be fix by https://github.com/1Password/load-secrets-action/issues/80


openDrawerOffset increases right margin every open/close

A strange bug popped up after I updated from RN 0.49 to 0.55.2
What's wrong
When setting openDrawerOffset to anything other than 0, the content width becomes smaller than the container. This results in, every time the drawer is opened, the content width slowly shrinks. I added some screenshots as well.
I'm guessing it may be due to the rounding when calculating the offset? On iOS RCTView recently changed and this issue may have something to do with the error.
What's expected
When the drawer is open, content should match the width of the drawer and its openDrawerOffset
Made the background a different color so you can tell easily.
First open, you can see that the content width doesn't match the drawer

8th open, you can see it slowly shrinking

This is with 0 offset, no issues

My versions
"react": "^16.3.1",
"react-native": "^0.55.2",
"react-native-drawer": "^2.5.0",

My team has also updated RN from 0.54.0 to 0.55.2 and is experiencing the same issue. Previously, it was working fine on 0.54.0.
We're using the same versions of the following packages as @cameronmoreau above:
"react": "^16.3.1",
"react-native": "^0.55.2",
"react-native-drawer": "^2.5.0",
This issue is only appearing on iOS, but it seems to work fine on Android devices.
Any extra insight would be super appreciated!

I had the same problem with "react-native": "0.55.2",

same bug, RN 0.55.2 , ios 🤕 🤕 🤕 🤕

Same with me, it happened after upgrade to RN 0.55.2 and for IOS only.

Temporary fix for me was to remove the parallax tween handler

Any solution here, this is making it not useable... :/

I've fixed that issue in this PR https://github.com/root-two/react-native-drawer/pull/352

@numez the fix only works in development mode
I manually put in your two lines of "+	this._length = Math.round(tweenValue*2)/2;"
Worked on simulator.
When exported for distribution, error still occurs.

it works for me, @roycclu can you share Your Environment and How to reproduce?

I had this issue in my app after upgrading RN .54 -> .55. I can confirm the bug fix in your PR #352 fixes the issue for my app, at least in the iOS simulator.
As a temporary fix (modifying package source code is not an option for my development) I have temporarily switched iOS users to use the static drawer type option. I am looking forward to seeing your PR accepted soon though!

Same problem.
I reverted to RN 0.54 to fix it, hope there's a bug fix soon!

@numez Your fix only works on normal iPhone models, the PLUS models or iPhone X still have the problem.

Okay, thanks for clarifying.
The issue I saw earlier with the fix is not due to simulator vs device.
 Please check your 'fix' before you tell others to use it. Still broken on iPhone PLUS models.

Thank you @numez working great on iPhone Plus and iPhone X!

same issues
Thank @numez for fixing but the PR is not worked for me on iPhoneX

Same issue with iPhone Plus models  and iPhone X using the PR #352
"react": "^16.2.0",
"react-native": "^0.55.4",
"react-native-drawer": "^2.5.0",

@hoangvuanhdevelopervn how did you fix this issue?

same bug on "iphone 6s Plus" with<EMAIL_ADDRESS>and that fix. I have seen the same bug when tried to change top/bottom/left/right styleProps through Animated.timing function in Animated.View and has fixed it when replace top/bottom/left/right on transform/translateX/translateY

For what it's worth I did a git bisect between the react-native 0.54.x and 0.55.x and found that this commit in React Native is what causes the problem.  I don't have a solution yet.

use type="static" resolve this issue


Is this step necessary if I use managed Azure API for FHIR instead of the FHIR server?

Sorry this article is a bit confusing, I am trying to follow the steps in tutorial "Access FHIR API with Postman" and there are several attributes are needed for getting access token. Seems I need to go these registration but at the same time one of the closed comments you say this is needed only for FHIR server.

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

ID: 5988a84f-de28-e2e4-f14b-c162bf71a380
Version Independent ID: e4f18f1a-8fdb-a68c-36a4-5030f911aecd
Content: Register a resource application in Azure Active Directory - Azure API for FHIR
Content Source: articles/healthcare-apis/register-resource-azure-ad-client-app.md
Service: healthcare-apis
Sub-service: fhir
GitHub Login: @hansenms
Microsoft Alias: mihansen

It may be confusing because the answer is that it depends. You don't absolutely need to create a resource application registration for your service unless you are using SMART on FHIR, in which case, the SMART on FHIR spec requires that the Audience is the same as the FHIR server URL.
So if you are using the managed service, you can support some use cases without a dedicated resource application registration, but you may want to create one.

#please-close


Saving CSS files while LiveSync running uglifies file

If I save a .css file while I have tns run android running, it minifies the file to one line.

Is this a file that does not have an associated .sass or .scss file?
The expected behavior is that the .css output generated by this plugin is minified and compressed. But I don't think it should be doing that to plain .css files if there is no matching SASS source.

There isn't.
Here are my packages for reference:
  "nativescript": {
    "tns-android": {
      "version": "2.5.0"
    },
    "tns-ios": {
      "version": "2.5.0"
    }
  },
  "dependencies": {
    "lodash": "4.17.4",
    "nativescript-camera": "0.0.8",
    "nativescript-dialog": "0.1.0",
    "nativescript-drop-down": "1.5.3",
    "nativescript-effects": "0.3.0",
    "nativescript-email": "1.4.0",
    "nativescript-i18n": "0.1.6",
    "nativescript-iqkeyboardmanager": "1.0.1",
    "nativescript-loading-indicator": "2.3.2",
    "nativescript-orientation": "1.6.1",
    "nativescript-phone": "1.3.0",
    "nativescript-platform": "1.1.0",
    "nativescript-platform-css": "1.4.0",
    "nativescript-plugin-firebase": "3.11.3",
    "nativescript-ratings": "1.0.1",
    "nativescript-videoplayer": "2.5.0",
    "tns-core-modules": "2.5.2",
    "tns-platform-declarations": "2.5.2"
  },
  "devDependencies": {
    "@types/lodash": "4.14.62",
    "babel-traverse": "6.24.1",
    "babel-types": "6.24.1",
    "babylon": "6.16.1",
    "lazy": "1.0.11",
    "nativescript-custom-entitlements": "0.2.0",
    "nativescript-dev-sass": "1.1.0",
    "nativescript-dev-typescript": "0.4.2",
    "typescript": "2.2.2"
  }

I added this scenario to the demo app for testing (and future regression checks).
I am able to reproduce the behavior you reported: on LiveSync, a plain CSS file is compressed/minified. Let me investigate and deploy a quick patch.

This may be a little trickier to "solve" than I thought.
I believe what we're seeing is the default behavior of the node-sass library (which underpins this plugin). By default, it is processing all files, and there does not seem to be an easy way to make it "ignore" plain CSS files.
Simplest workaround is to just rename all of your existing .css files to .scss. This will have the effect of preserving your original CSS comments/formatting. Probably the best choice.
Will keep investigating to see if there is another option for ignoring CSS that does not have a related SCSS file.

Alright, will do


bug tablename "AS"

hi bro,
thanks for this awesome works.
i used this repo twice, and i remember if this repo have bug.
if we have tablename which contains "as" the tablename will truncated
example:
nama_asli => become: nama_
tbl_xxxxasyyyyy =>become: tbl_xxxx
https://github.com/nacasha/CIgniter-Datatables/blob/0fdcf6f49bd8d81733e0bef7d5023d98fa550976/libraries/Datatables.php#L228

Hi, thanks for open this issue :)
I will fix it as soon as possible

Fixed on 261f83c

@nacasha thanks for this awesome library


Stash only relevant broker config keys on the broker status

Fixes #2914

Proposed Changes

Stash only relevant broker config keys on the broker status

Release Note

Docs

cc @Leo6Leo @lionelvillard

/ok-to-test

Hey @basit9958! Are you being blocked by anything? Any help we could provide you with?

/retest-required

/retest-required


User's API key is re-encrypted on each user update

Each time a user is updated (for example, by changing their editor preference on the User Profile page) the server is re-encryping the user's already-encrypted API key. This causes the user's API key to change, as well as leading it to grow in length. Eventually, after enough updates, the database will reject the key as too long to index.

PR maproulette/maproulette2#454 should address this.


[WebToolsE2E][Aspire] Sign out the vm account and reconnect to vm using the same account, then open the project that has verified the token authentication or create a new aspire project and F5, it still pops up dashboard token authentication again.

REGRESSION INFO: This is a new feature on Aspire 8.0 P6
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install VS 17.10 Preview 4 FB
Install Aspire 8.0 P6 [8.0.0-preview.6.24211.5]
Apply dotnet-tools Feed

REPRO STEPS

Create an Aspire Starter App > give a name 'AspireApp1' > Create
F5 with https > popup dashboard token authentication > Input token and Log in
Sign out vm's account, then connect to vm again using same account
Open the authenticated 'AspireApp1' project, then F5 with https, it will popup dashboard token authentication again

ACTUAL
Sign out the vm account and reconnect to vm using the same account, then open the project that has verified the token authentication or create a new aspire project and F5, it still pops up dashboard token authentication again
Note

Disconnecting server and reconnecting vm works fine.

EXPECTED
There is no need to reconnect vm with token authentication again.

@v-sherryfan after you sign out and sign back in, can you check the status of your localhost cookie as shown here? I gave your scenario a try as well and I can see that the localhost cookie gets deleted when I sign out and sign back in. I believe that's the reason it's asking you for the token again. So, I believe that this is By Design.
Including @JamesNK to confirm

I think this might be a bug. I thought we were persisting the cookie across sesssions, but it turns out sessions is no longer ended when closing the browser. A session ends when logging out of a machine.


Start testing ember-alpha channel.

Functionality works properly, but there are a few issues that were identified while testing:

Cannot use component as a block param name (https://github.com/emberjs/ember.js/issues/14096)
Cannot use component unit tests (still researching)

:umbrella: The latest upstream changes (presumably 7b8be9a) made this pull request unmergeable. Please resolve the merge conflicts.

Is this still something we want to do or can we close this PR?

Actually, this is already happening


Incorrect nsfw detection

Apparently it detects YouTube links as nsfw? And pretty much any imgur or tenor link

Do you have some examples of the URLs you're finding this on? The URLs need to direct to an image file so that may be the issue here.

Well for example the attachment below it sees as hentai
On Sun, Nov 14, 2021, 10:10 AM d00M_L0rDz @.***> wrote:

Do you have some examples of the URLs you're finding this on? The URLs
need to direct to an image file so that may be the issue here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/d00ML0rDz/NsfwSpy/issues/4#issuecomment-968308748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOJF23G6WNTEADOUMMBTTKDUL7GQHANCNFSM5H7ERVHQ
.

Oops it didn't sent the attachment one sec

Unfortunately, NsfwSpy isn't perfect. We usually only see approximately a 3% chance of a neutral image image being identified as NSFW, and this looks to be one of those cases.
As you can see though, from the score below it isn't very confident it's hentai.

I'll add this image to our neutral dataset so that we can train on it in the future.

Cool, thanks!
On Sun, Nov 14, 2021, 10:24 AM d00M_L0rDz @.***> wrote:

Unfortunately, NsfwSpy isn't perfect. We usually only see approximately a
3% chance of a neutral image image being identified as NSFW, and this looks
to be one of those cases.
As you can see though, from the score below it isn't very confident it's
hentai.
[image: image]
https://user-images.githubusercontent.com/66832778/141687315-3e1daa45-2bc1-44bc-b343-268bd8f15743.png
I'll add this image to our neutral dataset so that we can train on it in
the future.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/d00ML0rDz/NsfwSpy/issues/4#issuecomment-968311718,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOJF23HE6EXNM2TM5XBJW4DUL7ICHANCNFSM5H7ERVHQ
.


update-gitops-repository 1.1.0

https://github.com/bitrise-io/bitrise-step-update-gitops-repository/releases/1.1.0
What to do if the build fails?
At the moment contributors do not have access to the CI workflow triggered by StepLib PRs. In case of a failed build, we ask for your patience. Maintainers of Bitrise Steplib will sort it out for you or inform you if any further action is needed.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

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

Hi @ghabor,
thanks for the PR, looks good to me!
First, the bitrise-coresteps-bot comment confused me a bit:
...
step.yml changes compared to previous version:
-  pull_request_body: ""
+  pull_request_body: 'Source commit: https://github.com/bitrise-io/bitrise-steplib/commit/35695f8821bdad5602ec8b797d6849e4b3c57428'
...

The real new default value is:
'Source commit: https://github.com/$BITRISEIO_GIT_REPOSITORY_OWNER/$BITRISEIO_GIT_REPOSITORY_SLUG/commit/$BITRISE_GIT_COMMIT'.
This seems to be a glitch in our diff step.


Integration Test - Release - 2019/02/01 11:50:13

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@adunndevster : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Process running twice

After a great deal of time spent logging out processes and trying to understand the flow of events -- It would appear that the postcss process it being run twice. My hunch is that it's run once per loader that is specified.
The first time through the postcss-hubl syntax is used. The second time through the default syntax is used which overrides and overwrites all of the edits made by the first pass. This is why we are unable to change the output of the comments and hubl expressions in the expected stringifier files.
If edits are made directly to the node files --> It appears that those edits are respected. We initially thought this meant that the postcss-hubl scripts were bing ignored or not extending their respective parent classes properly, but it appears that they are. It is just after they are run they are overwritten.
Confusing. Not sure why they are being run twice. Put a ping in to the maintainer of postcss to hopefully bring some clarity to the situation.
search jazzyclimber
https://gitter.im/postcss/postcss

The issue is not in our code. It is in the postcss-loader/css-loader/postcss interface.
It looks like the issue is coming from the LazyResult class in  lazy-result.js of postcss.
The first time postcss-runs it recognizes css as a string. The second time through it recognizes it as an object. It would appear that only when it's picked up as a string does postcss then honor the actual syntax.

constructor(processor, css, opts) {
    this.stringified = false
    this.processed = false

    let root
    console.log(css)
    if (
      typeof css === 'object' &&
      css !== null &&
      (css.type === 'root' || css.type === 'document')
    ) {
      console.log('First if')
      root = cleanMarks(css)
    } else if (css instanceof LazyResult || css instanceof Result) {
      console.log('second-if')
      root = cleanMarks(css.root)
      if (css.map) {
        if (typeof opts.map === 'undefined') opts.map = {}
        if (!opts.map.inline) opts.map.inline = false
        opts.map.prev = css.map
      }
    } else {
      console.log('else')
      // console.log(css)
      console.log('options', opts.syntax);
      let parser = parse
      if (opts.syntax) parser = opts.syntax.parse
      if (opts.parser) parser = opts.parser
      if (parser.parse) parser = parser.parse
      console.log(parser);

Stringifier Options do not get set correctly. If we force the correct stringifier in lazy-result we get the desired outcome.
stringify() {

    console.log('stringify running from lazy')
    if (this.error) throw this.error
    if (this.stringified) return this.result
    this.stringified = true

    this.sync()

    let opts = this.result.opts
    opts.stringifier = require('@spingroup/postcss-hubl/lib/hubl-stringify')

    console.log('opts from lazy', opts)

    let str = stringify
    if (opts.syntax) str = opts.syntax.stringify
    if (opts.stringifier) str = opts.stringifier
    if (str.stringify) str = str.stringify

    let map = new MapGenerator(str, this.result.root, this.result.opts)
    let data = map.generate()
    this.result.css = data[0]
    this.result.map = data[1]

    return this.result
  }


Incorrect 3D Scan

I'm having a problem with the 3D scan produced by the ROS package.  When I place the lidar a few inches above the floor, in RVIZ it's showing that the floor slopes downward at maybe a 40-degree angle.  It may be hard to tell from just an image but see below.  With the sensor above the floor only ~5 inches, there should not be so much scan below Z=0 and shouldn't be sloping downward (right?).  I tested the unit using the cyglidar viewer and it appears fine there (from as well as I can tell).  So it looks like something with just the ROS implementation.  I used both versions 1.3 and 2.0 and get same result  and also get same result from my ROS2 version I'm working on which was based on v2.0.  I'm using ROS Noetic for v1.3/v2.0 tests and ROS galactic for the ROS2 version.

No, I see it in v1.3.  Am I the only one getting these results?

I did a comparison of v0.1.0 and v0.2.0 of the same scene.   The camera is now placed on the floor, so anything below the camera is invalid.
First is v0.1.0.  Notice, nothing below the level of the camera:

Next is v0.2.0.  Notice, plenty of points below the camera:

The launch file comments says that 0 is for auto and 1 is for manual.  It defaults to 0, so it it correct? (i.e., 0 is auto?)
Which version are you running?  I get different results between main and 2.0.
My floor is a dark brown laminate, doesn't seem overly reflective to me.

This is what it looks like (its a mess).  I've now temporarily mounted the unit onto the robot and its much higher than where I was testing it.  previously, it would be sitting 4-5 inches above the ground.  I guess the floor is reflective to some extent, but I would think that would be less of a problem with the sensor low to the ground since the angle of incidence/reflection is greater.  With where it is now, the floor doesn't even show up in the point cloud.. at least visually on RVIS.

Refer to https://github.com/CygLiDAR-ROS/cyglidar_d1/issues/27#issue-1345133079
Once I implemented @larsvn's bug fix for the duration and I found (what I think) is an error in the documentation (1 is auto, 0 is fixed) then setting duration to 3500 got rid of all that floor garbage.


Bunjs support

Is there any plan for Bunjs support https://bun.sh/

Yes, or most are already done! Because Bun has Web standard API such as Request and Response like a Cloudflare or Deno. I've tried to run the minimal app using Hono on Bun. It works well! You can run the same script:
https://gist.github.com/yusukebe/0822c1629fc05027dcc8302ddaafc932
Maybe some middleware such as jsx are not working now (Currently, JSX Pragma is not supported on Bun). But, almost are working.
You will be able to start using Hono right away if you want. And the day of support for Bun completely is coming soon.

While attempting to use your demo, I found that if you try to return/res json that it breaks. Very important in my opinion, so it should be looked into. Here is my code: https://gist.github.com/cursecodes/ace9e61658b95aa4dc2d99e4fd6bffbd

Haha yes I was missing that. Unfortunately, this did not fix it.

Hi @cursecodes
This code works well in my environment. I can't reproduce your error.
import { Hono } from 'hono'

const app = new Hono()

app.get('/', (c) => {
  return c.json({ message: 'Hello Bun!' })
})

export default {
  port: 3000,
  fetch: (req) => app.fetch(req),
}

// Run the script on bun
// bun run ./hello.ts

$ bun run --hot src/index.ts
error: Cannot find module "hono/dist/hono-base.js" from "C:\Users\PC\Downloads\my-dev\bun-hono-test\my-bun\node_modules\hono\dist\hono.js"
but hono-base.js is available in node-modules

Hi, I got the same problem and I could fix it, following those steps:

Delete file bun.lockb in root folder
Delete node_modules folder if it present
Execute the next comand in the root folder of project to clean the cache bun pm cache rm
Finally install everything with bun install and run the project with bun run dev.
I think it is a problem with lasted and newer dependencies of the project and its cache.


Calling affix after resizing the document

Hello, I love your plugin!
Is there any way to call the affix function?
For example - I resize my document and I recalculate my affix element width and I wish to call the affix to render again. Or I change my data and I wish to change the dimensions of the affix element.
Does it have any callable function to refresh it?

I'm closing this since there was no reply for a long time, feel free to open again in case you still need help :)

@DJanoskova did you find a way to "rerender" affix element?

You can define a ref to your affix element:

Then in your vue methods function, call:
this.$refs['sidebar-menu'].setAffix()
this.$refs['sidebar-menu'].lastState = 'affix'
Refer to the source code https://github.com/eddiemf/vue-affix/blob/master/src/affix.vue for the other available methods...


[#4254] Change some CompilationExceptions to ParsingExceptions in the parser

resolves #4254
Description
We've recently added a ParsingException class. This pull request changes existing CompilationExceptions in the parser files to ParsingExceptions. We will still get CompilationExceptions when parsing from the rendering code, but that's more complicated to change because rendering is done both at parsing and at compilation time. Ideally we will straighten those out in a future effort.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change

@cla-bot check


[OpenAI] Implement AssistantClient

- [ ] Protocol methods
- [ ] Convenience methods
- [ ] Models and model factory methods
- [ ] Architecture review

Complete the tasks above for each of the following features:

Create assistant
Create assistant file
List assistants
List assistant files
Retrieve assistant
Retrieve assistant file
Modify assistant
Delete assistant
Delete assistant file
Create thread
Retrieve thread
Modify thread
Delete thread
Create message
List messages
List message files
Retrieve message
Retrieve message file
Modify message
Create run
Create thread and run
List runs
List run steps
Retrieve run
Retrieve run step
Modify run
Submit tool outputs to run
Cancel run

Given all CRUD operations and object hierarchies here, do you have a sense of whether some should be at protocol only for now? Or do you see us being able to support basic convenience for every one of them?


Consider SSP370

According to this, the standard scenarios agreed upon are SSP126, SSP245, SSP370, and SSP585.  I could add SSP370 or replace SSP585 with SSP370.  SSP585 is pretty whacky extreme, so SSP370 might be a better choice.

Requiring that models have this scenario in addition to others limits the models to the following 10:

ACCESS-CM2
ACCESS-ESM1-5
AWI-CM-1-1-MR
BCC-CSM2-MR
CanESM5
CMCC-ESM2
EC-Earth3-Veg
EC-Earth3-Veg-LR
FGOALS-g3
MPI-ESM1-2-HR

In meeting with @embruna, we decided to try and include SSP370, but keep SSP585.  Maybe doesn't matter that all models are represented in all scenarios.


Allow configuring primary line X/Ys separately

This is an implementation of the feature outlined in this proposal, which at time of writing has yet to be approved.

Bugsquad edit, closes: https://github.com/godotengine/godot-proposals/issues/7588

The implementation looks fine, but there is a concern of compatibility with old values. If someone set primary step values before, they will be lost after this PR.
This code would handle it:
#ifndef DISABLE_DEPRECATED
	if (state.has("primary_grid_steps")) {
		primary_grid_step_x = state["primary_grid_steps"];
		primary_grid_step_y = state["primary_grid_steps"];
	}
#endif

(not sure if DISABLE_DEPRECATED really applies here; maybe a comment would be sufficient. This value won't be needed in the future)

Thanks!


Update travis settings

Travis changes
Ref #14

If you're fine with the patchset, I will rebase-squash the changes :+1:

LGTM, thanks! I can squash from the web UI.


Unable to test service that uses FastEndpoints events

Hi there, I have a service which publishes an event. I am trying to write unit tests for the service. Due to the way that FE events are published: with an extension method instead of an interface, I am unable to mock the dispatcher like I would if I was using MediatR.
Any suggestions on a workaround?

Thanks for the response @KateyBee . In this particular instance, I am not using a full-blown WebApplicationFactory, but rather trying to unit test a service, calling all of its public methods. There are mocked dependencies everything that is injected into the constructor.

Is there a service registered by Fast Endpoints that could be injected into the constructor? The current method of publishing events seems odd.

odd is the price we pay for convenience, and it gets the job done ;-)
FE event bus is not a mediater pattern implementation, so there's no middle man that needs to be injected.

I am not using a full-blown WebApplicationFactory

Factory.RegisterTestServices() is not a WebApplicationFactory. it's just does some wiring up internally so that fake event handlers can be registered during unit testing. i.e. only if you need the handlers to be executed during testing. if not just, do:
Factory.RegisterTestServices(_ => {});

that allows unit tests to run without throwing an exception when a PublishAsync() extension method is encountered.

Thanks for the response. I will give it a try.


RAC-5604 Centos 6.5 maximum payload for regression test

Maximum payload file for CentOS 6.5 OS install in regression testing for Quanta T41.
Paired with @gpaulos

@jimturnquist @hohene


Issues with getting previous command and backspace

Windows Terminal version (or Windows build number)
1.8.1521.0
Other Software
Using bash on WSL on Windows Terminal.
Steps to reproduce
Using bash on WSL on Windows Terminal.

Type some command and run it.
Then try to go back to a previous command (up arrow), and backspace to clear it.

Note that this does not happen every time. Sometimes the command can be cleared as expected, sometimes it cannot.
See below:

The first line ran fine. Then I tried to select a previous command, and was able to clear it fine, and pressed enter. Then again, same thing. But third time I tried to backspace, and it would not go past the li. However, the text actually get's cleared, as if I press enter it gives no output, and I can type additional commands and they work as expected. But visually, the text is still there.
I should add that I have seen similar behaviour before on the online IDE platform repl.it, where commands do not fully get deleted visually.
Apologies if this is a possible duplicate.
Expected Behavior
You should be able to clear the entire command.
Actual Behavior
Only some of the previous command gets cleared, and new text gets appended to it. However the actual text has been deleted, but it doesn't show.

What's your PS1 in bash?

What's your PS1 in bash?

$(horizontal_line)\n\# \t \w $(virtualenv_prompt)\n$
horizontal_line just draws dashes the width of the window.
virtualenv_prompt adds the name of the current python virtual environment.
My PS1 is set as follows (multiple lines are just so it's easier to edit):
PS1=""
PS1="$PS1$(tput setaf 8)\$(horizontal_line)\n"
PS1="$PS1$(tput setaf 1)\# "
PS1="$PS1$(tput setaf 6)\t "
PS1="$PS1$(tput setaf 3)\w "
PS1="$PS1$(tput setaf 5)\$(virtualenv_prompt)\n"
PS1="$PS1$(tput setaf 2)$(tput bold)\$ "
PS1="$PS1$(tput sgr0)"
export PS1

Yep, that's what I suspected. When you're using escape sequences to format your bash prompt, you need to wrap them up so that bash knows that they're not visible characters.
From the docs
              \[     begin a sequence of non-printing characters,
                     which could be used to embed a terminal con
                     trol sequence into the prompt
              \]     end a sequence of non-printing characters

If you slap those on either side of the $(tput ...) commands, that should fix it.


updated crash version to 0.11.1

Coverage decreased (-0.03%) to 81.29% when pulling 099b2128c9d0cf90c9e3077fc6b7d7ff2e14f4a7 on h/crash-0110 into 11dbb93f72b05f4bbe2c8ab7a7addf1a76be9715 on master.

pls fix tests
ddl should not show affected rows

superseded by PR https://github.com/crate/crate/pull/1669


Form submitted by JS provoke full page reload

With the following html
<turbo-frame id="X">
    <form method="POST" action="/some-page" id="myForm">
         <input type="hidden" onchange="myForm.submit()" value="whatever">
    </form>
<tubo-frame>

The backend only need to redirect to the same page.
By contrast if the form us submitted with a button, then, with the same backend, turbo will replace the current turbo-frame with the new content, as expected
<turbo-frame id="X">
    <form method="POST" action="/some-page" id="myForm">
         <input type="hidden" value="whatever">
         <button>Submit</button>
    </form>
<tubo-frame>

And I've notice that the button could be hidden and triggerd by a change on the input field, in this case the behavior is as expected.

Thank you for opening this issue.
Calls to HTMLFormElement.submit do not dispatch submit, whereas calls to HTMLFormElement.requestSubmit will.
The Turbo Handbook: Form Submissions section of the documentation mentions:

To submit a <form> element programmatically, invoke the HTMLFormElement.requestSubmit() method and pass an <input type="submit"> or <button> element as an optional parameter.

Could the Form Submission documentation be improved to make that requirement more clear?

Thank you very much, very helpfull.
I don't understand the relation between the phrase you extracted and the context arround in this doc.
Maybe extract it into it's own paragraph?


MAINT: add threads/jobs primitives

@colinvwood and I spoke offline about this - the reason this is failing is because diversity directly depends on diversity-lib, and the changes implemented in that repo were not included in the latest prepare (since it was just merged along with the rest of the threads/jobs PRs). these failures essentially aren't real - so i'm going to merge this, and we'll use the next prepare (that we need to run anyways to add rescript) to confirm that everything behaves as expected.


The m2k_collect cluster output contains | in the metadata name

Describe the bug
The m2k_collect cluster output has | in the metadata name and the actual name is in the next line, due to which the plan file has incorrect name of custom cluster type in the targetClusters.
To Reproduce
Steps to reproduce the behavior:

Login to the cluster and run move2kube collect.
Check the output cluster yaml.
Run move2kube plan -s /path/to/src/ -c m2k_collect/clusters/.
Check the m2k.plan file.

Example:
cluster yaml
apiVersion: move2kube.konveyor.io/v1alpha1
kind: ClusterMetadata
metadata:
  name: |
<EMAIL_ADDRESS>

m2.plan file
    targetClusters:
      Kubernetes: m2kassets/inbuilt/clusters/kubernetes.yaml
      Openshift: m2kassets/inbuilt/clusters/openshift.yaml
      ? |
<EMAIL_ADDRESS>      : m2kassets/custom/IAM-xyz-1f50a05161df3b57.yaml

Expected behavior
cluster yaml metadata should not contain |
apiVersion: move2kube.konveyor.io/v1alpha1
kind: ClusterMetadata
metadata:
  name<EMAIL_ADDRESS>

m2.plan file
    targetClusters:
      Kubernetes: m2kassets/inbuilt/clusters/kubernetes.yaml
      Openshift: m2kassets/inbuilt/clusters/openshift.yaml
<EMAIL_ADDRESS>m2kassets/custom/IAM-xyz-1f50a05161df3b57.yaml

Desktop (please complete the following information):

OS: macOS
Version: v0.3.0+unreleased
gitCommit: dcf8793a889c0a8f9f4423e9e9ee3a95003c6bcc

Might be related to this https://github.com/konveyor/move2kube/issues/178


[FAQ] Added FAQ for sorting inputs for input errors

The following Pull Request is a part of the Issue: Move FAQs to GTN of the Galaxy_Hub repository.
I have added Sorting your inputs in the GTN.
Kindly have a look at it and suggest any improvements or amendments.

thanks again @garimavs!

Thanks, @beatrizserrano for updating the required changes.


Http issue on iOS with final release of Angular 2

Hi, I'm trying to get some information using a service and when I run the iOS emulator for testing the service, and I get this message: Response with status : 200 for URL: null. Now I'm using the 2.4.2 Nativescript current and latest version.
And also I changed the Info.plist file. Here is my service code:
getRecords(stringUrl:string){`
//Asign Headers
let headers = new Headers();
headers.append("Content-Type", "application/json");
headers.append("Authorization", "Bearer: "+ this.config.loginToken);
// Returns observable with API answer
return this.http.get(
this.config.qqcURL +"/records?from_date=2016-11-22T00:00:00&to_date=2016-12 22T23:59:59&tags=Testing&durop=>=&startduration=00:00:00",
{headers : headers}
)
.map(response => response.json())
.map(data => {
let searchList = [];
data.Result.forEach((item) => {
if(item.session_status==1){
searchList.push(new Record(item.startdate,
item.duration, item.inbound,item.outbound,
item.common_id, item.score_avg, item.my_score,
item.id,item.tags));
}
});
return searchList;
},
err => console.log(err))
}

And my Info.plist file here:

There is something similar on this closed issue https://github.com/NativeScript/nativescript-angular/issues/457 but it doesn't work for me.
Please, If you could help I would be so gratefull.

@genesis06 From your posted code snippet it is not clear from where you are receiving this message.
The HTTP status is 200 (OK) so you have successfully made a request to your server. Then you ar pushing all the needed info in an array called searchList and returning that array.  Can you elaborate where URL: null is coming from!?

Hola @genesis06. I've had that problem before and it's for sure that the problem is the lack of SSL certificate. iOS is very annoying with that. I also tried to add it to the Info.plist and it didn't work for me either...
Anyways, have you tried to make a request from android or (i.e.) postman and it works? Just to make sure that the problem is the insecure request in iOS

Hi @NickIliev and @eleddie, thanks for your reply, and sorry for replying you a little late.
Fortunately I found where was the problem. It wasn't related with the SSL certificate because the API is not running on HTTPS.
That "Response with status : 200 for URL: null" message was because of an encoding problem. There were invalid characters used on the URL:  "<" (less than), and  ">" (greater than). So I used  "%" + the character ascii code, to fix it.
Finally, my URL seems like:
this.config.qqcURL +"/records?from_date=2016-11-22T00:00:00&to_date=2016-12 22T23:59:59&tags=Testing&durop=%3E%3D&startduration=00:00:00"
instead of
this.config.qqcURL +"/records?from_date=2016-11-22T00:00:00&to_date=2016-12 22T23:59:59&tags=Testing&durop=>=&startduration=00:00:00"
Thank you so much.
Kind regards 😃


changed arrangement to reduce height

Tried for more compact arrangement. fix issue 40

failed checking. I'll try again...

Interesting. It actually looks like the errors have nothing to do with the changes you made.
I wonder if something changed with Travis's environment since the last push. I'll check into that.
Thank you for the PR btw. Can I ask the reasoning behind switching from panes to panels? (I am fine with the switch, I just don't know what the difference is. :) )
Anyway, thanks again.

I’m embarrassed to say I can’t remember the reason I chose to use -panel%. I can’t even remember the difference in functionality.


:sparkles: Add rollover_t

Problem:

When dealing with a timer register that rolls over, it is useful to have a type that knows how to handle that.

Solution:

Add rollover_t.
rollover_t behaves like an unsigned integral type, with interesting comparison semantics.

@elbeno: I think I remember what I did...it was with signed integers, but let's pretend that signed overflow is defined behavior for a moment:
bool operator<(auto a, auto b) {
    retutrn (a - b) < 0;
}

That implementation takes care of the wraparound case and there is no case in which a < b and b < a holds true. We should be able to implement the equivalent with unsigned integers.

This is how it should work for uint32_t:
bool operator<(uint32_t a, uint32_t b) {
    retutrn (a - b) >> 31;
}

In 2s complement signed integers, a number is negative if the msb is set. This function does the equivalent check for `uint32_t`.

You're right - and that's exactly equivalent to what I've written. > mid means the sign bit is set. Bear in mind the pesky integer promotion rules...

You're right - and that's exactly equivalent to what I've written. > mid means the sign bit is set. Bear in mind the pesky integer promotion rules...

 Ok....I need some more work to wrap my brain grapes around this one... 🧠🍇


Bring Objects to highest level in output

A complex structure can show the highest level item as 'pruned' forcing one to sift through the json atring to find the one unpruned source. Example: https://jsonblob.com/bf90de6d-caf5-11e7-8bfc-27c198b81400
In the example, the root level "server" is pruned and the original is somewhere in the json with a key NOT labeled "server" making it VERY difficult to find. It would be much better if the root level server had the object.

Am I right in saying that what you want is having, for every set of positions an object is referred to, to keep the one which is at the lowest depth ?
If so this is an interesting request.

Yeah. If by lowest you mean lowest index.
Current
_object: {     server: { ... } }, server: "-pruned-"
Desired:
_object: {     server: "-pruned-" }, server: { ... }

The request is clear.
The problem is that to do this I'd need to handle the object to stringify in two times : a BFS traversal to know where to keep and where to prune, followed by the DFS traversal which is the natural order of JSON writing.
This is feasible but not totally trivial to minimize the impact on performance.

As the changes envisioned here would take some time to code and wouldn't be without impact on the library size and efficiency, I'd like to have more feedback from users before diving into it.
Please comment here or on Miaou.

Could this be opt-in?

For the performance, yes. But not reasonably for the library size.