* should match empty paths

Ref: https://github.com/http-rs/tide/issues/783
curl http://localhost:8080/test
ok
curl http://localhost:8080/
404
curl http://localhost:8080
404

happy to take this on, although i'm not sure it's a bug in the semver sense (i think this is semver-minor in that it's a change in expected behavior

This is still an issue. The wildcard is commonly understood to mean zero or more repetitions of an item, in this case path segments.

@yoshuawuyts Do you mind if I take a crack at this sometime in the next few weeks?

Is this an issue on head? The router has changed between the last release and head

@jbr Yes, still an issue on head. The NFA is not configured to allow an empty string for the colon wildcard as a valid acceptance state. I added some quick tests in lib.rs to verify this as well:
    #[test]
    fn empty_wildcard_colon() {
        let mut router = Router::new();

        router.add("/a/:b", "ab".to_string());

        let m = router.recognize("/a").unwrap();
        assert_eq!(*m.handler, "ab".to_string());
    }

    #[test]
    fn empty_wildcard_colon2() {
        let mut router = Router::new();

        router.add("/a/:b", "ab".to_string());

        let m = router.recognize("/a/").unwrap();
        assert_eq!(*m.handler, "ab".to_string());
    }

Ahh my mistake, I hadn't noticed that we were in the route recognizer crate, not tide. However, this original issue was about star routes, not colon routes. Wanting /a/:b to match /a would be a different issue from this one.
However, tide moved to a different routing crate ([`routefinder](https://github.com/jbr/routefinder)) because route-recognizer is not particularly easy to change. It might be possible to add this behavior to routefinder. If that's something you'd want, feel free to open an issue over there, but it might be a week or two before I get to it, and it might need to be an opt-in feature (probably new syntax for the parser, like `/a/:b?` or similar).

No worries.
I'm currently working on a project using Yew, which uses route-recognizer for routing. It might be possible to convince them to switch to routefinder since the route specification syntax is similar (identical?). I would love the ability to specify a /:b? in my routes, as this mirrors the route matching in Rocket, which I'm also using: https://rocket.rs/v0.5-rc/guide/requests/#multiple-segments. Their multi-segment wildcard allows matching an empty path, which simplifies routing in an application that deals with file paths where you'd also want to accept the root path as valid.

@willhodges I'd be very excited to hear if yew was interested in switching to routefinder! Although the syntax is identical, there are several small differences: Routefinder only supports a single wildcard (*) per route, unnamed parameters are not supported, and the prioritization between routes may not be identical (I am not aware of any differences, but retaining route priority wasn't as important as making the priority behave as I thought was desirable). * does match the empty path in routefinder, which is another difference from route-recognizer.


Add swift build command

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.72%. Comparing base (996b3cb) to head (77146c7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   93.72%   93.72%
=======================================
  Files          13       13
  Lines         239      239
=======================================
  Hits          224      224
  Misses         15       15

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


Select A CI/CD Provider

Travis CI and Circle CI look like good choices to handle our continuous integration and continuous delivery needs.  We need to pick one.

My vote goes to TravisCI. I was initially leaning towards CircleCI but it seems to support a very limited selection of OSes, and Travis seems to be widely adopted so it can't be all that bad.

@fromcj I don't suppose you could whip up a short comparison for us?  This could either be written or in verbal form.  Whatever we chose should go into the Decision Log of the guide book so we don't forget why we selected one over the other.

I'm writing up a comment for Serverless framework but if you read this page it recommend using some of the capability it provides as your CI/CD provider: https://serverless.com/framework/docs/providers/aws/guide/deploying/

I agree with @fromcj to pick Travis CI. According to https://hackernoon.com/continuous-integration-circleci-vs-travis-ci-vs-jenkins-41a1c2bd95f5

Travis CI and CircleCI are almost the same
Both of them:
Have YAML file as a config
Are cloud-based
Have support of Docker to run tests
What does TravisCI offer that CircleCI doesn’t?
Option to run tests on Linux and Mac OS X at same time
Supports more languages out of the box:
Android, C, C#, C++, Clojure, Crystal, D, Dart, Erlang, Elixir, F#, Go, Groovy, Haskell, Haxe, Java, JavaScript (with Node.js), Julia, Objective-C, Perl, Perl6, PHP, Python, R, Ruby, Rust, Scala, Smalltalk, Visual Basic
Support of build matrix
Build matrix is a tool that gives an opportunity to run tests with different versions of language and packages. You may customize it in different ways. For example, fails of some environments can trigger notifications but don’t fail all the build ( that’s helpful for development versions of packages)

Travis CI also appears to be more popular and offers tutorials for different languages. https://stackshare.io/stackups/circleci-vs-codeship-vs-travis-ci#more

@fromcj I dedicated some of my "nerd time" this weekend to reading the Travis docs and I see that they support publishing to Lambda, which could be super useful to us.

@fromcj It also appears to have HipChat integration, so we can all know when a build breaks!

The Atom and desktop notifications are also very interesting.

@kurron Do we want to settle on Travis CI for now for our testing? It seems both easy to get started with, and has so many things we can add.

Linting integration for sure! Here is a sublime and atom one :)
https://github.com/sabhiram/sublime-travis-yml-lint Sublime
https://github.com/mschuchard/linter-travis-lint

@sarahmau I think that decision needs to made as a team so we'll probably want to wait for @fromcj  notes but I can say I liked what I've seen in the documentation enough to rig up one of my personal Groovy/Docker projects to see how it fairs.  I still don't know how secrets are kept out of the Travis descriptor, though, which is super important.

So here's a quick and dirty rundown to cover the most important points (to me at least)
Travis and CIrcle are, at the most base level, more or less identical. That means with Travis we get a cloud based service with a documented REST API and Web UI that supports Docker and is easy to set up and get running. That covers our most basic needs for this project.
The biggest advantage Travis provides us actually isn't the build matrix (though that is cool) but simply it supports more operating systems that Circle.
As to @kurron's question of how to handle secrets in Travis, it seems that they have a fairly intuitive system in place to handle them documented here: https://docs.travis-ci.com/user/encryption-keys/
To me, I think Travis will cover every use case we have for this project. That plus the fact that it is widely recommended and thus heavily documented not just in its own docs and forums but other places on the internet like StackOverflow make it at least, if not a clear choice, a fully serviceable one for us.

If anyone has questions about this, feel free to leave them here and we can meet and go over anything yet to be covered or answered


Releases + Tags

Hey there.
Is it possible to do releases or tags here?
Love, someone having to maintain a Gopkg.toml

Hi @doertedev,
Certainly. I will address it shortly.
Regards,
Stan

@doertedev, see https://github.com/kazarena/json-gold/releases/tag/v0.1.0.
I'll be tagging releases going forward.
Stan

Very much appreciated! Thanks for the rapid tag.


Native Code required?

Hi,
I have been facing an issue when adding Camera_filters, do I need to implement and add native code? since adding any text and or filters results in an error of unable to find writeVideofile Method.
Please let me know if so

can you specify the error, I mean any screenshot?

can you specify the error, I mean any screenshot?

If you try to apply any color filter and or add text, an error will appear, the loading will be infinite, the error is :
E/System  ( 8392): Uncaught exception thrown by finalizer
E/System  ( 8392): java.util.concurrent.RejectedExecutionException: Task android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy$$ExternalSyntheticLambda0@6cff93b rejected from java.util.concurrent.ThreadPoolExecutor@31f4858[Terminated, pool size = 0, active threads = 0, queued tasks = 0, completed tasks = 3]
E/System  ( 8392):     at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2082)
E/System  ( 8392):     at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:842)
E/System  ( 8392):     at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1374)
E/System  ( 8392):     at java.util.concurrent.Executors$DelegatedExecutorService.execute(Executors.java:678)
E/System  ( 8392):     at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.onClosed(CallbackProxies.java:104)
E/System  ( 8392):     at android.hardware.camera2.impl.CameraCaptureSessionImpl.close(CameraCaptureSessionImpl.java:588)
E/System  ( 8392):     at android.hardware.camera2.impl.CameraCaptureSessionImpl.finalize(CameraCaptureSessionImpl.java:894)
E/System  ( 8392):     at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:339)
E/System  ( 8392):     at java.lang.Daemons$FinalizerDaemon.processReference(Daemons.java:324)
E/System  ( 8392):     at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:300)
E/System  ( 8392):     at java.lang.Daemons$Daemon.run(Daemons.java:145)
E/System  ( 8392):     at java.lang.Thread.run(Thread.java:1012)

which is related to the native code, as when i did not have the native code of the editor which may be using something extremely outdated and or there is a loop within it., it was giving an error of missing channel which was also not identified in the documentation.

This bug has been fixed in new update, if you have further queries please reopen the issue.
Thanks for using this package


Dynamic groups

What changes are proposed in this pull request?

Dynamic groups app support (backend tracked by #2475)
Group code modularization and some cleanup

Notes to the reviewer:

Pay special attention to files in the app/packages/core/src/components/Modal/Group/DynamicGroup directory

How is this patch tested? If it is not, please explain why.
Dataset for testing was generated using the following snippet. Every th samples are folded into a group (here, 25).
import fiftyone as fo
import fiftyone.zoo as foz
from fiftyone import ViewField as F

dynamic_groups = foz.load_zoo_dataset("quickstart-groups").clone()

every_th_sample = 25
# for each slice, for every th samples, assign an arbitrary "scene_id"
for slice in dynamic_groups.group_slices:
    dynamic_groups.group_slice = slice
    scene_id_counter = 0
    order_by_counter = 0
    for sample in dynamic_groups:
        sample.set_field("scene_id", scene_id_counter // every_th_sample)
        sample.set_field("timestamp", order_by_counter % every_th_sample)
        sample.save()
        scene_id_counter = scene_id_counter + 1
        order_by_counter = order_by_counter + 1
dynamic_groups.save()

Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[x] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[x] Core: Core fiftyone Python library changes
[x] Documentation: FiftyOne documentation changes
[ ] Other

LGTM!

Thanks for leaving the comments. Will address comments in another PR.


Declare strides when ACL objects init and activation in ACL depthwise

The following fixes are in PR:
(1) Since when we are configuring ACL objects we know exactly what is expected formats for weights so we should set strides at that point and not leave it to ACL to estimate based on weights dimensions,
(2) If depthwise convolution comes with post op that converts to ACL’s activation layer operation we were not executing that activation function because the convolution was not configured with it.

cc: @penpornk


FIX: Query name collision with multiple filters

https://meta.discourse.org/t/error-when-adding-3-or-4-filters-to-knowledge-explorer-plugin/167123/
If using more than two tags to filter, we'd run into an SQL alias
collision on a few joins. Resolved by ensuring unique names are used across the query.

This was a duh moment for me - I thought I had added specs last night but apparently I didn't! 🤦


The Ingress and the Services should be inside the same Namespace

[Enter feedback here]
Please add this IMPORTANT NOTE: The Ingress and the Services should be inside the same Namespace. Otherwise, the Ingress won't find the Service even with its full name: ..svc.local

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

ID: fd0e7d5a-37e9-07dc-f395-9cb5bb580e45
Version Independent ID: 7e4faf70-9724-7e7b-832b-1cd99a974920
Content: Create an ingress controller - Azure Kubernetes Service
Content Source: articles/aks/ingress-basic.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the issue, @HoussemDellai. I've assigned this to the doc author to review.

Thank you @HoussemDellai  for the feedback.
We've added a task to our backlog and will prioritize accordingly.
#please-close


Fixed broken anchor link

Fixed anchor link to routing template section

Deploy preview for tryghost ready!
Built with commit 5a9adc135d52548994cd07a4b3ff661c3850f6ea
https://deploy-preview-22--tryghost.netlify.com

oh nice! :heart: this way of deploying


Add openfunctionv2 model inference script and fix minor bug

Summary
This PR introduces a new model handler openfunctions_handler.py to run inference on OS model gorilla-llm/gorilla-openfunctions-v2 and reproduce the results on leaderboard
Issue: https://github.com/ShishirPatil/gorilla/issues/352
Changes

Merge the input data into a single file since that's what the evaluation script is expecting, data is under /gorilla/berkeley-function-call-leaderboard/data/BFCL/questions.json
Fixed a minor bug in utils.py, the returned object function is mistakenly put inside the for loop, instead it should be put outside loop
Added a new OpenfunctionsHandler in handler map
openfunctions_handler.py with prompt template and decoding step compatible with openfunctions_v2 model
A simple handler_runner.py to run the inference and save the result for evaluation
Add instructions to readme.md

Test

Generates the inference result with vLLM

python model_handler/handler_runner.py --data-path /home/jobuser/gorilla/berkeley-function-call-leaderboard/data/gorilla_openfunctions_v1_test_all.json --model-name gorilla-llm/gorilla-openfunctions-v2 --model-path {PATH_TO_MODEL}/gorilla-openfunctions-v2/

Result: https://www.toptal.com/developers/paste-gd/GLTJTe4l

Generate eval result with eval_runner

generate evaluation for AST based metrics
python {PATH_TO_REPO}/gorilla/berkeley-function-call-leaderboard/eval_checker/eval_runner.py --model gorilla-llm/gorilla-openfunctions-v2 --skip-api-sanity-check --test-category simple sql relevance parallel_multiple_function parallel_function multiple_function

🦍 Model: gorilla-llm_gorilla-openfunctions-v2
2024-04-15 18:37:42,909 INFO worker.py:1616 -- Started a local Ray instance. View the dashboard at <IP_ADDRESS>:8265
🔍 Running test: parallel_multiple_function
✅ Test completed: parallel_multiple_function. 🎯 Accuracy: 0.695
2024-04-15 18:37:43,761 INFO worker.py:1454 -- Calling ray.init() again after it has already been called.
2024-04-15 18:37:43,774 INFO worker.py:1454 -- Calling ray.init() again after it has already been called.
🔍 Running test: parallel_function
✅ Test completed: parallel_function. 🎯 Accuracy: 0.845
2024-04-15 18:37:43,813 INFO worker.py:1454 -- Calling ray.init() again after it has already been called.
🔍 Running test: simple
(raylet) /home/jobuser/build/openconnect-lib-core-image/environments/satellites/python/lib/python3.10/site-packages/ray/dashboard/modules/reporter/reporter_agent.py:56: UserWarning: gpustat package is not installed. GPU monitoring is not available. To have full functionality of the dashboard please install pip install ray[default].)
(raylet)   warnings.warn(
✅ Test completed: simple. 🎯 Accuracy: 0.845
2024-04-15 18:37:43,854 INFO worker.py:1454 -- Calling ray.init() again after it has already been called.
🔍 Running test: relevance
✅ Test completed: relevance. 🎯 Accuracy: 0.6875
2024-04-15 18:37:43,876 INFO worker.py:1454 -- Calling ray.init() again after it has already been called.
🔍 Running test: multiple_function
✅ Test completed: multiple_function. 🎯 Accuracy: 0.935

Leaderboard scores

Rank,Overall Acc,Model,Model Link,Organization,License,AST Summary,Exec Summary,Simple Function AST,Python Simple Function AST,Java Simple Function AST,JavaScript Simple Function AST,Multiple Functions AST,Parallel Functions AST,Parallel Multiple AST,Simple Function Exec,Python Simple Function Exec,REST Simple Function Exec,Multiple Functions Exec,Parallel Functions Exec,Parallel Multiple Exec,Relevance Detection,Cost ($ Per 1k Function Calls),Latency Mean (s),Latency Standard Deviation (s),Latency 95th Percentile (s)
1,80.48%,Gorilla-OpenFunctions-v2 (FC) from HuggingFace,https://huggingface.co/gorilla-llm/gorilla-openfunctions-v2,Gorilla LLM,Apache 2.0,83.00%,0.00%,84.50%,84.50%,0.00%,0.00%,93.50%,84.50%,69.50%,0.00%,0.00%,0.00%,0.00%,0.00%,0.00%,68.75%,N/A,N/A,N/A,N/A

Hi Jason, on the onset, thank you so much for taking the time to review and modify our codebases. We appreciate your feedback and are actively working on verifying the result and merging the code.
Here is a list of actionable items we are going to take in the next few days:

We have a script eval_data_compilation.py that compiles all the data pulled from HuggingFace into a single file for vLLM batch inference. We are open to this change as it allows users to set up the inference pipeline easily using the HuggingFace Models. Here is what we plan to do:

Remove all data from the ./data folder and put the 2000-line data file under HuggingFace.
Modify apply_function_credential_config.py such that it will also apply credentials to the new data file.

We welcome the idea of handler_runner.py. This is an easier and simplified interface for locally deploying models to run on our evaluation. We would like to have a single reference point for model result generation. In other words, users can call openfunctions_evaluation.py only to accomplish data generation. Here is what we can do:

We will merge the handler_runner.py content into openfunctions_evaluations.py . python model_handler/handler_runner.py --data-path /home/jobuser/gorilla/berkeley-function-call-leaderboard/data/gorilla_openfunctions_v1_test_all.json --model-name gorilla-llm/gorilla-openfunctions-v2  --model-path  {PATH_TO_MODEL}/gorilla-openfunctions-v2/  → python model_handler/openfunctions_evaluation.py --model gorilla-llm/gorilla-openfunctions-v2

In eval_runner_helper.py, model_name → model_name_escaped change will introduce inconsistencies. The MODEL_METADATA_MAPPING  will substitute ‘_’ with ‘/’ in the model name and will dedicate a raw model name mapped to the one displayed on the website. We are going to revert that change before merging in.
Regarding your inference result, we will run a local evaluation based on your modifications and check if our result matches. We will also check if the current handler matches what we have in our backend. We will respond to that soon with more information.

Again, thank you very much for improving our code quality and spot-check issue. Looking forward to collaborating on this issue.
BFCL Team


WFCORE-6984 Allow ResourceDefinition.builder(ResourceRegistration, ResourceDescriptionResolver, SubsystemModel) to accept a null deprecated model

https://issues.redhat.com/browse/WFCORE-6984

Thanks @pferraro


Bug Fix

@aritra-tech Do review code and merge as soon as possible :)

Please fix the ktlint checks

checked !!! successful at my side

Are you sure that the code that you have done is working?
When I tested it I couldn't save empty notes on giving spaces also the save button doesn't appear.
When I tested your branch there were some changes that are not in your branch.
Please check it out once.

I don't get it it the logic I have implemented so far will not allow not to be get saved until there will be no characters and buttons to save will remain disappeared and will only appear once user will type some characters rather spaces and my branch should do that I will double check ✅ IF MY ABOVE logic is sufficient or  I am wrong at some point ?

This issue that you have talked about was already handled previously.
I wanted the user can also save empty notes or let's say he/she doesn't give the title and writes only the notes then it will be saved. Just like Google Keep Notes

I don't get it it the logic I have implemented so far will not allow not to be get saved until there will be no characters and buttons to save will remain disappeared and will only appear once user will type some characters rather spaces and my branch should do that I will double check ✅ IF MY ABOVE logic is sufficient or  I am wrong at some point ?

You mean if title is even not written then note with only description should be saved and if I am wrong provide a demo VIDEO for change implementation

Yeah right.

IS THIS GOOD TO GO  @aritra-tech

IS THIS GOOD TO GO @aritra-tech

Yeah right.

@aritra-tech now what :) review & merge it

Working fine but when I run your branch I couldn't find many changes which are already present on develop.
Do one thing.

Right Click on develop and click on Checkout.
After checkout to develop update the develop branch.
Once develop branch is updated then checkout to your branch after checkout click on the develop branch once and you will see an option named Merge develop into bugFix on doing that your bugFix branch will be updated and you can see the changes then only.
Please do that.

@aritra-tech actually i have to do these on github or at android studio as on github on right click web options are displayed not any checkout any video you can provide for this or should i open a new PR ?

I would suggest opening a new pr.
Before creating a branch from develop first pull the develop branch and then create a branch from develop

Any update @Ismail-AD

Involved in preparation of exams so you can assign it to someone else

Involved in preparation of exams so you can assign it to someone else

It's okay @Ismail-AD
All the best for your exams


Refactor ts-force-gen

I'm thinking about this too.
It would be nice to have the steps broken out and configurable.

Pull all relevant data from Salesforce.
Probably with filters applied so we don't over-fetch data.
Tweak & add data needed for types.
a. Exclude fields
b. Convert API names to clean property names
c. Renaming duplicate fields
c. Anything else. The more logic we can move to this step the simpler the type generation will be.
Based on output from above step generate types

The smaller we can get the functions the more customizable it will be.


Add CSV support

Hey! Thanks for the amazing library.
I needed to be able to perform redactions on CSV representations of serialized objects, so added an assert_csv_snapshot macro.
The PR follows the rest of the repo wrt to the csv dependency and associated macro being optional.

Thanks a bunch. This is good stuff!


[Table] Loading storybook mdx file is slow

What package is this for?

[x] React
[ ] Angular

Describe the bug
Loading the table.mdx is really slow.
I think this is due to the new mdx imports (where smaller mdx-files are imported to table.mdx) that happens at runtime. Not sure if this is only a problem in DEV.
To Reproduce
Steps to reproduce the behavior:
This is noticeable when it is loaded from the Playground story by clicking the "Docs" tab or from the sub mdx files and when clicking links leading back to the main table.mdx.
Expected behavior
Faster loading
Environment/versions:

OS: [e.g. MacOS, Windows]
Browser: [e.g. chrome, safari]
carbon-addons-iot-react version: [e.g. v2.60.0]

Additional context
If applicable, add screenshots to help explain your problem.
Add any other context about the problem here.
Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
NB: The core contributors will try to work with your timeline, but it's not
guaranteed. The earlier you make a request in advance of a desired delivery
date, the better!

@bjornalm If we are seeing it on the deploy preview then it is also slow in production.

True, it is loading faster in the deploy preview... but it is not blazing fast there either and probably will ger slower the more docs we add and link. I think we need to keep an eye on this.


ActivityStarter gives error "java.lang.String.isEmpty"

Seen by users on the forums, e.g. here and here.
Create an app with a button, an activity starter, and a notifier. Have the button trigger a StartActivity event, and then have ActivityError show the message. In the emulator it will show java.lang.String.isEmpty. This will also occur on older Android deployments because isEmpty was introduced in JDK 1.6 whereas older versions of Android are compatible with JDK 1.5 only.
Proposed fix: Locate any calls to the .isEmpty() method in components/ and replace them with .length() == 0 check instead for backwards compatibility.

Fixed (in ucr pending release)


Boost module doesn't seem to be working.

Hi, have you tried it with the boost module ? I have added the boost module and it doesn't render it using the canvas/image as it should. Any pointing directions ? Thanks

I have never used the Boost module. Maybe add a jsfiddle showing the issue?

same issue

Wondering why this issue is closed, has it been resolved? Anyway, I am also having problems with the boost module. Need to check if it's a highchart or a highcharts-ng issue.
http://jsfiddle.net/j950np50/25/
You can see in my script, I enabled the "boost" in the chart configuration.
    boost: {
      useGPUTranslations: true,
      seriesThreshold: 0,
      boostThreshold: 0
    },

and the default dashStyle of a series graph should be dash (not solid):
      series: {
        animation: false,
        dashStyle: 'dash',

because I have the above in my chart configuration. The dashstyle is solid though when boost is enabled, it is only dashed when I remove the boost.
A second issue: in my script, I am changing the series dashstyle and markers after 1 second.
    series.update({
      marker: {
        symbol: 'triangle',
      },
      dashStyle: 'dot'
    }, true);

The dashstyle should be "dot" and the markers should be triangles. It works fine without boost, but with boost enabled, the dashStyle does not change, and the marker triangles only show when the user hovers over a point, but in the normal state, the point markers are still shown as circles. This should not be the case.


[Feature Request] Mute System button on Top Menu

This feature is useful when remoting into a system and wanting to make it silent. Could be for many reasons, including for table authors so when working/developing/testing a table, it does not bother people who may be in the area.
Currently I must kill PinVol manually and place OS on Mute
Ideally Studio's single 'Mute' button would do this...

Pause Pinvol / Temporary disable it
Mute the operating system volume

... and reverse when taken off mute.

Fixed with 3.6.0

thank you!


Ninja teleport will not Teleport over certain objects that you can normally see over

Description
Ninja cannot teleport over certain objects like computers and machines that you can normally see over these objects act as if they obscured line of sight and can lead to unfortunate deaths. From what I see if bullets cannot go through an object you cannot teleport through it either except for glass.
Reproduction
to reproduce this you just have to attempt to teleport over any computer or machine like the communications computer.
Screenshots
N/A
Additional context
N/A

Coulda sworn that #27793 was suppose to fix that.

issue was resolved


Unable to Handle model dialog in two different it().

Hi,
I've a scenario where clicking on a button will cause alert pop-up for a non-angular js app

. I need to read the text inside the alert (alert.getText() is working) and then accept it(alert.accpet() working also). However, when i split alert.getText() and alert.accept() to two different its() the script is failing with Model dialog present
i Need to alert.getText() and alert.accept() in two different its(). Could some one please let me know if that is possible?
ENV:
Protractor 3.0.0.
IE 9.0
jasmine 2.x
Regards,
Jagadeesh

Hi there!
Your question is better suited for StackOverflow or Gitter. Please ask a question there with the 'protractor' tag or post in the Gitter Channel to get help.
From the the getting help section of the README:

Please ask usage and debugging questions on StackOverflow (use the "protractor" tag) or in the Angular discussion group. (Please do not ask support questions here on Github.)

Thanks!


In the dataset，what does（x.tensor,y.tensor,valid_c_idx.tensor) specifically represent?

Hi, In the dataset，what does（x.tensor,y.tensor,valid_c_idx.tensor) specifically represent?
Thanks

Hi, In the dataset，what does（x.tensor,y.tensor,valid_c_idx.tensor) specifically represent? Thanks

x, y represent the projection points of points on the feature, and valid represents whether it is valid on this feature,,you can see dump-data.py for detailed instructions

Thanks,


feature: notify camper by email on completion

LGTM - thank you for building this so quickly!

Refers #8389


fix module path to make it go install-able

Currently it is not go install-able
$ go install github.com/kachick/never18/cmd/never18@latest
go: github.com/kachick/never18/cmd/never18@latest: <EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: never18
                but was required as: github.com/kachick/never18

module path should be github.com/kachick/never18 to make it public.

@pankona Merged your commits via #1, Thank you!


Ignore missing / broken link warning does not seem to work in newer sbt versions

We've recently switched from sbt 1.6.2 to 1.9.8. Our build workflows now fail due to missing / broken link errors when doing something like [[SomeClass]] in scala doc.
In sbt 1.6.2. this could be fixed / ignored by setting "-no-link-warnings" and scoping to doc / scalaOptions. However this setting does not seem to take effect anymore on sbt 1.9.8.
Is this an intended change? If so, there is no documentation available on this in the scaladoc section.

After some further testing, we found that newer versions still respect this setting, but required scoping to be done a bit different:
sbt 1.6.2. (accepts doc / scalacOptions := Seq( .... "-no-link-warnings"), sbt 1.9.8: requires scoping to Compile / doc / scalacOptions. In any case it would be nice to add a hint regarding this option and how to use it to the Scaladoc section of the docs


frontend: give a warning for error-prone features

@st1page: there are some features not fully supported in our system such as index and some conterintuitued features such as create mv order by. We can give a warning and return it with pgwire in these situation https://github.com/singularity-data/risingwave/blob/06b3c11faecf3773bcbcbe9247a4b79e7765ed04/src/utils/pgwire/src/pg_response.rs#L117

Is this issue solved? Pgwire will return the error message if unimplemented features return an error.

Is this issue solved? Pgwire will return the error message if unimplemented features return an error.

No. And for the totally unimplemented features, I think we need to return an error. This issue is for those features half state which usually is undocumented and not exact as human intuition or other DB's behavior.


Fix using "languages" pull config from .tolgeerc

Pull config item "languages" is not accessed correctly and is thus not respected.
Currently default value (all) is always used regardles of .tolgeerc content.

@krisgerhard thanks for the catch 🙂

Thanks for the quick release :) Works as expected now :) Great tool! Thank you!


add app-api i18n

What's changed?

https://github.com/dromara/hertzbeat/issues/1212
Checklist

[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.
[x]  I have added the necessary unit tests and all cases have passed.

@all-contributors please add @novohit  for code


How to forward in time?

During my Anchor tests, I need to simulate certain time change for my Solana program to react differently, such that the time_now below in my Solana program should change:
use solana_program::sysvar::clock::Clock;
let time_now = Clock::get().unwrap().unix_timestamp;

can I do that?

Hi could you explain a little ? I looked it up and found nothing / did not know where it is.

It's possible with https://crates.io/crates/solana-program-test
Unfortunately no, warp_to_slot forwards the slot, but I am pretty sure it does not update the Clock sysvar

Clock sysvar is changed with warp_to_slot

Clock sysvar is changed with warp_to_slot

Oh weird, it actually does
        &context
            .banks_client
            .get_account(sysvar::clock::ID)
            .await
            .unwrap()
            .unwrap()
            .data,
    )
    .unwrap();
    println!("slot: {}, unix_timestamp: {}", clock.slot, clock.unix_timestamp);
    context.warp_to_slot(10_000)
        .unwrap();
        let clock = deserialize::<Clock>(
            &context
                .banks_client
                .get_account(sysvar::clock::ID)
                .await
                .unwrap()
                .unwrap()
                .data,
        )
        .unwrap();
    println!("slot: {}, unix_timestamp: {}", clock.slot, clock.unix_timestamp);```

The output of this code is
slot: 1, unix_timestamp:<PHONE_NUMBER>
slot: 10000, unix_timestamp:<PHONE_NUMBER>

I said no because this chat said the clock syvar wouldn't move:
https://discordapp.com/channels/428295358100013066/758929221061050378/854394968084840509


Addition of a copy code button on the website

Is your feature request related to a problem? Please describe.
It will be really helpful if we can copy the Python script code and use them on our system for this we can add a feature of displaying the scripts and letting users copy their code so they can enjoy it
Describe the solution you'd like
Additon of a copy code button
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Go Ahead @Shaurya10101

The script.html file isn't available on this repo or is it?

switch the gh-pages branch for that @Shaurya10101

This issue has been inactive for a long time
I would like to work on this issue
Please assign

I am working on this issue so please have patience

It's been 3 weeks @Shaurya10101 , Still No updates, I'm assiging it to @sagnik-p


Fixed debug hook issue #152

Fixes issue https://github.com/feathersjs/feathers-hooks-common/issues/152 will be released with v3.2.1.


Exclude benchmark/, CHANGELOG and dotfiles from sdist

Codecov Report

Merging #59 (1f9c6ab) into master (170de46) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #59   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          511       511
  Branches        98        98
=========================================
  Hits           511       511

Continue to review full report at Codecov.

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


[8.x] Fix flushDb (cache:clear) for redis clusters

When running php artisan cache:clear with a redis cluster as the cache driver, the flushDb() function is currently unable to run on each master if you are using a password or TLS scheme - https://github.com/laravel/framework/issues/35180
For redis clusters an empty $config array is passed through to the constructor of vendor\laravel\framework\src\Illuminate\Redis\Connections\PhpRedisConnection.php causing authentication to fail.
However, even if you update vendor\laravel\framework\src\Illuminate\Redis\Connectors\PhpRedisConnector.php to pass through the cluster password it still fails because the current $redis = tap(new Redis)->connect($host, $port); does not connect to each master node using the correct TLS scheme or any stream context such as local_cert, local_pk, cafile or verify_peer_name.
This pull request fixes the issue by using directed node commands as described here - https://github.com/phpredis/phpredis/blob/develop/cluster.markdown#directed-node-commands
It uses the already authenticated RedisCluster connection with correct scheme and context to direct the flushdb command to each master node to succesfully clear the entire cache.

I will probably need someone actually using Redis Cluster to confirm this.

Are you using any particular Docker setup to test Redis Cluster?

Got a cluster up and running with Docker and this does indeed seem to not break anything at least.

I've tested on Homestead running 6 instances 3 masters, each with 1 replica (slave).
I've also tested on an Ubuntu staging server running 20.04, using self signed certs and TLSv1.2.
Thanks for the merge!

Sorry to comment on an old MR, but I have found this after we have come across the same issue when using PHPRedis, however we are on laravel/framework v6.20.19.
I have verified that the issue is the exact same, and changing the PhpRedisConnection@flushdb to the above submitted code, does indeed fix the issue.
@taylor or @willbrowningme ... Can this fix be back-ported into the V6, given it is still LTS and receiving support fixes?
If not, is there an alternative that can be suggested?
Thank you.

@J5Dev maybe try a PR to see if Taylor would accept it?

@J5Dev maybe try a PR to see if Taylor would accept it?

No problems, will put one together now.

@J5Dev maybe try a PR to see if Taylor would accept it?

No problems, will put one together now.

Could you make PR to laravel:7.x , we also facing this isssue.
It's wold be helpful a lot, thanks in advance.

Laravel 7 isn't maintained anymore


Converted Split Pdf Log to table

Any logs with QR scan errors will be highlighted.
Also changed the composition of the title because it makes more sense in the Romantic languages.

@DecathectZero thanks for doing this - I made a few other comments.


Implement skeleton

Can you provide a basic readme so I can understand the application easier?

See GoDoc reference: https://godoc.org/github.com/nogoegst/wrh


open folder option not available in MAC ???

This page does not talk about how to handle the same in MAC

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

ID: 970f0d59-b2b4-7082-1c2c-a468c93571ef
Version Independent ID: a3135966-c343-5a6b-cd97-772a0c34fd93
Content: Develop code without projects or solutions - Visual Studio
Content Source: docs/ide/develop-code-in-visual-studio-without-projects-or-solutions.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @gewarren
Microsoft Alias: gewarren

yes that "banality" is crucial to development omg - link is closed because not enough info wtf??? - make it happened ms - I cannot use your product from 2010 because many failings

would be great show some workaround like - open it through command line


test: remove .only()'s

Removes two remaining .only()'s
Issue number
Relevant issue number
N/A
Please check the following

[ ] Do the tests still pass? (see Run the Tests)
[ ] Is the code formatted properly? (see Linting (Formatting))
For New Features:

[ ] Have tests been added to cover any new features or fixes?
[ ] Has the documentation been updated accordingly?

Please describe additional details for testing this change

lol something is wrong with this


Add DG components to the build system

Add DG components to the build system
Fixes #595

Change Description
This PR moves the definition of the number of DG components and the CG degree from hard coded values spread between ModelArrayDetails.cpp and the dynamics module and kernel classes. These values are now defined as part of the CMake configuration step, with the CMake command line argument -DDynamicsType= with valid parameter values DG1 and DG2. A value of DG1 sets the advection DG degree to 1 and the number of advection DG components to 3. A value of DG2 sets the advection DG degree to 2 and the number of advection DG components to 6. In both cases the CG degree is set to 2 and the number of DG stress advection components to 8.
These values are implemented as values for preprocessor macros that are applied to all of the source files compiled into the nextsimlib compilation target. This is then used by the model binary and most of the unit tests.
The ModelArrayDetails.cpp file defines the number of components of the DG-related ModelArrays based on these values, but also defines values equivalent to the DG1 option as defaults.
In the dynamics, the number of advection DG components is used as the template parameter in instantiating the relevant dynamics kernel. The CG degree is used in the CGDynamicsKernel class to as the CGdegree template parameter and to calculate the number of DG components for stress. This latter value is calculated using the CG2DGCOMPONENTS function. These two numbers of DG components are used as the template parameters for the DynamicsKernel class. Within these various dynamics kernel classes the template parameters for the underlying dynamics classes such as DGVector and CGVector are set.

Test Description
Tests still compile and run.

Documentation Impact
Information on the options added to installation.rst.

Subsumed into #606

Subsumed into #606


Updated readme from docs pillar review

Fixes #2403 (part of SDK tracking issue https://github.com/microsoft/botframework-sdk/issues/5898)
Updates the readme in line with previous updates to .NET / Python readmes, following the docs pillar review.  Brings standardization to readmes, adds missing packages and also brings in the content regarding building the SDK in from the wiki (which will be removed following the merge of this PR).

Pull Request Test Coverage Report for Build 140697

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.961%

Totals

Change from base Build 140212:
0.0%

Covered Lines:
12915

Relevant Lines:
15236

💛  - Coveralls


Meaning of valid cookie

I don't understand what a valid cookie is and specifically I don't get this part:

Validating the cookie on every request mitigates the risk of revoked users accessing the app.
What I am trying to do is authenticating a user using his credentials, then create a cookie with a session ID and use that session ID to validate further requests by the user. So to my understanding the cookie/session ID has to always be validated.

Can you help me understand this?

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

ID: d2229fc6-f8c8-952e-e567-b1d5a463055a
Version Independent ID: b18a015b-914d-f014-f711-120d208904d9
Content: Use cookie authentication without ASP.NET Core Identity
Content Source: aspnetcore/security/authentication/cookie.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.


Revert "Reflect origin in CORS"

Reverts mjackson/unpkg#227 which caused errors due to origin being cached
Fixes #229

Thanks for the quick fix, @kjarnet 👍

Thanks for deploying it so quickly! And for the awesome cdn, btw 🙂


Fix script urls in the entrypoint file that point to other packages

 Issue by sigmundch
Originally opened as dart-lang/sdk#20092

Since in the output build folder we have a single packages/ directory in the web/ folder, we make sure that URLs containing 'packages/' are normalized to point there. We've fixed several issues regarding this recently, but here is one corner case we still don't handle.
I've noticed this error in polymer 0.12.0-dev.
These two examples should produce the same output, but today they do not not:
[example a]:
A file web/a/b/index.html with these contents:

      <link rel="import" href="packages/polymer/polymer.html">

    <polymer-element name="x-foo" noscript><template></template></polymer-element>
    <x-foo></x-foo>

[example b]:
A file web/a/b/index.html with these contents:

      <link rel="import" href="b.html">

    <polymer-element name="x-foo" noscript><template></template></polymer-element>
    <x-foo></x-foo>

and file web/a/b/b.html with:

    <link rel="import" href="packages/polymer/polymer.html">

After all the inlining is done, in [example a] we produce:

In [example b] we produce:

We should be producing the output of [example b]. For now, users can workaround this limitation very if they write '../../packages' in web/a/b/index.html in [example a], but that might not work correctly if they also use Dartium without a special development server.

 Comment by jakemac53

fixed in dart-lang/sdk@5c6f5d10450a1e669d86f1c6eb4aed6b3afd2625

Added Fixed label.

 Comment by jakemac53

Set owner to @jakemac53.
Added Accepted label.

 Comment by jakemac53

Added Started label.


Improve precursor mz determination for raw files

Improve precursor mz determination and improve readability. Now even with REFINE_PRECURSOR_MZ_AND_GET_INTENSITY_FROM_MS1 = false still have very accurate precursor mz read from the raw file.
Fix indentation.

Hi Stefan,
I merged your changes but then realized that they result in fewer identifications (~10% for my test dataset). I believe this is because your code retrieves the most abundant isotope / isolation center m/z rather than the monisotopic m/z. Do you have any suggestions or should I revert?
-Craig

Craig,
I agree, there is a problem.
I believe the code I submitted retrieves the precise mz for a peak (possibly the most abundant isotope?), which is used by the spectrometer to calculate the Isolation center (probably by rounding).
Your code identified a rounded value for monoisotopic m/z, and then used that to find the precise value by scanning backwards.
I'm submitting a pull request to fix this. Sorry about the confusion!


Weird performance issues

I'm getting weird performance issues when trying to connect to the host. In case if it's unsuccessful, i'm reconnecting and here is the screenshot of profiler log.
https://www.dropbox.com/s/efz3r1g8d0ezjq3/Screen Shot 2015-06-11 at 18.28.29.png?dl=0
Shouldn't worker thread stop when attempt is unsuccessful?

It will be taken off the runloop once the NSStream delegate reports back the connection is unsuccessful. You are attempting to reconnect every time it fails to connect to the host? Are you getting the webSocketDidDisconnect: delegate calls?

Hello!
I do get webSocketDidDisconnect: calls. Here is the screenshot of case with empty webSocketDidDisconnect delegate method implementation. https://www.dropbox.com/s/kw7z14ptaideuue/Screenshot 2015-06-15 13.06.21.png?dl=0

Interesting. I would add some logging and make sure the isRunLoop is getting set to NO.
https://github.com/acmacalister/jetfire/blob/master/JFRWebSocket.m#L307
https://github.com/acmacalister/jetfire/blob/master/JFRWebSocket.m#L258
If the isRunLoop gets set to NO, then the thread should stop shortly afterwards.

Hey, getting same issue.
[JFRWebSocket stream:handleEvent:] is being called, there isRunLoop = NO  there, but in
while (self.isRunLoop)
[[NSRunLoop currentRunLoop] runMode:NSDefaultRunLoopMode beforeDate:[NSDate distantFuture]];
isRunLoop is still 'YES' and it executes forever

Fixed it by switching strings :
self.isRunLoop = YES;
[self.outputStream write:[data bytes] maxLength:[data length]];

Otherwise disconnectStream: is being called before self.isRunLoop = YES, so it has no chance to be setted back to NO.

@maxydey nice find, I could have swore I already change that, must have done it in Starscream. I will change that shortly, thanks!

It would be nice if you could make a new tag / release for CocoaPods. Right now you get 0.1.2 from Jan 8, which does not include this fix.

@BarryHaanstra, I will update it today.

I think 0.1.4 update should take care of this.


Nginx response header error when Drupal's new Cache Tags value is large

Issue Type

Support Request

Summary
Nginx throws response header too big error with Drupal 8 pages that produce a large number of Cache Tags in the header.
*2 upstream sent too big header while reading response header from upstream

The new Cache Tags headers (X-Drupal-Cache-Tags) are quite extensive for a site I'm working with. These appear to be overflowing the default buffer size for Nginx and causing a 502/Bad Gateway error.
The problem can be solved by increasing the buffer size in the nginx conf. I've found the following lines work together to do the trick,
fastcgi_buffers 16 16k;
fastcgi_buffer_size 32k;

I'm wondering if you have come across this issue before and how you addressed it. It looks like these cache tags are causing a certain amount of debate right now.
It's likely I need to automate this additional config to sit on top of drupal-vm so other developers can just start without having to do any manual configuration. I see two possible routes I could use to add these settings to the nginx conf,

Use post_provision_task to run an additional Ansible script to do the task, or

Run a different playbook via Vagrantfile.local.

Any feedback you might have could be useful and appreciated. Thanks.

You can also add them to extra_parameters for the nginx_hosts https://github.com/geerlingguy/drupal-vm/pull/1156#issuecomment-278053767

@oxyc Thank you for the quick response and also apologies for not searching the issues hard enough myself and seeing the previous post and solution. I'd been looking through the Ansible role but not spotted the extra_parameters option.


Swap teleport to house menu items

Is your feature request related to a problem? Please describe.
I need to teleport outside my house so I can use my friends house more efficiently
Describe the solution you'd like
Swap "Cast Teleport to house" with "Outside Teleport to house" on the normal spell book

It's already possible

Not for the spell tab, only for inventory items.


Licensing compliance of using user licenses on devices

Product licensing terms mention the E licenses as user licences https://www.microsoft.com/licensing/terms/productoffering/Office365Suites/EAEAS.
In such case, these licenses should not be used to license device accounts. If so, can we get clarity in this document to inform that using E1/E3/E5 etc (user) licenses is not compliant from licensing perspective and should be changed to TR license as soon as possible?
Something like:
If you are currently using E1, E3, E4, E5 SKUs with Skype for Business Plan 2 with Audio Conferencing or with Office 365 Phone System and a Calling Plan, these will continue to work. However, using user licenses (for example E1/E3 or E5 license) is not compliant from licensing perspective, therefore you should consider moving to Team Room license as soon as possible.

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

ID: a2070d20-21d1-c8bb-bbb4-697083efcd20
Version Independent ID: f4998e3c-74ed-4a2b-fe7c-c55044b9a6b4
Content: Microsoft Teams Rooms licenses - Microsoft Teams
Content Source: Teams/rooms/rooms-licensing.md
Service: skype-for-business-online
GitHub Login: @dstrome
Microsoft Alias: dstrome

Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi! My customers do not read it as "you are non-compliant from licensing perspective and NEED to change once your licenses expire". They understand the message as "you can continue using and you can consider to TR license but you do not have to. It will be just simpler for you, but if it's cheaper to use E1, you can continue". In other words, current wording is soft and does not clearly state that this is not compliant to use the E user licenses with MTR devices.

If needed I can share answer from LicenseQ regarding licensing requirements

@Larryngolog, I cannot say or confirm that we can use the word non-compliant here. I've forwarded this to another team and I'm waiting for confirmation.

Hello, this issue was closed but was not resolved. What was the answer of the other team?


extend type t to contain tag and message (both type string), fixes #18

use rresult and result types (instead of option)

this also removes two failwith (Invalid_severity / Invalid_facility) by adjusting F|S_of_int to return an F|S option.
I'm curious about the decode function checking for < 1025 -- is this a limit only for UDP or also for other transports?

There's a comment in the code:
(* FIXME Provide default Ptime.t? Version bellow doesn't work. Option type
let parse ?(ctx={timestamp=(Ptime.of_date_time ((1970, 1, 1), ((0, 0,0), 0))); hostname="-"; set_hostname=false}) data =
*)

You can use Ptime.epoch for this -- but I still wonder what decode should accept!? a syslog packet without timestamp?  without hostname? (I suspect the way forward is to move to 5424 ;)
in case this is accepted, I'm happy to adjust my logs-syslog with the revised types
EDIT: the type annotations in syslog_message.ml (: ('a, [> Rresult.R.msg ]) result) are necessary since otherwise the locally defined Error constructor (from type severity) is used.

this PR splits the message part into tag and message part -- it uses the first N alphanumeric characters as bytes, if N>32, then it fails.  on output, there's no separator inserted between tag and message -- should there be a space?  or some other character?  maybe only if the message does not start with a non-alphanum character?

Thanks for the changes and refactoring the code \o/

I'm curious about the decode function checking for < 1025 -- is this a limit only for UDP or also for other transports?

Maybe it would be better to implement message truncation in the individual transport code and remove it from syslog-message. I've added it because section 4.1 in the RFC says the message must not exceed 1024 bytes. In case of the old RFC it is only UDP.

There's a comment in the code:
(* FIXME Provide default Ptime.t? Version bellow doesn't work. Option type
let parse ?(ctx={timestamp=(Ptime.of_date_time ((1970, 1, 1), ((0, 0,0), 0))); hostname="-"; set_hostname=false}) data =
*)
You can use Ptime.epoch for this -- but I still wonder what decode should accept!? a syslog packet without timestamp? without hostname?

This is indeed not straight forward. The whole ctx record just exists to accommodate for broken messages and to fix them according to section 4.3.2. If the timestamp of the incoming message is invalid, the info provided in ctx is used instead. Since RFC 3164 doesn't have a year in the timestamp, the user provided Ptime.t is also used to validate the incoming timestamp. Hence a good default would be to use Ptime_clock.now, which doesn't support Mirage afaik. I think we should remove the comment, because it might not be possible to have a sane default for timestamp without Ptime_clock.now.

(I suspect the way forward is to move to 5424 ;)
That is absolutely right. I'm channeling some motivation to continue the work on that branch :)

Since there is a tag field now, maybe we should also rename the message field in type t to content. According to the 4.1.3 the message field consists of a tag and a content field.

makes sense, I pushed a commit which renames message to content :)

also removed the length checks in decode, and tweaked encode to not trim the message by default. -- maybe we need to add documentation now that the caller has to check length invariants

on output, there's no separator inserted between tag and message -- should there be a space? or some other character? maybe only if the message does not start with a non-alphanum character?

According to the RFC, there has to be a separator between tag and content. Maybe defaulting to a space is a good idea, if the first character in the content field isn't non-alphanumeric.
4.1.3 - MSG Part of a syslog Packet

Any non-alphanumeric character will terminate the TAG field and will be assumed to be the starting character of the CONTENT field.

Example: https://tools.ietf.org/html/rfc3164#section-5.3

@verbosemode done in 9f23be3

ping


nbdev_new pins on major+minor version of nbdev-template

Since every version of nbdev uses latest nbdev-template, we can't make any breaking changes in nbdev-template. I propose that nbdev_new pins the major+minor version of nbdev-template and that nbdev-template bumps minor version on breaking changes.
Example
Implementing https://github.com/fastai/nbdev/issues/1083 would be simpler with this change.
Currently, we can implement it by updating the default in _apply_defaults and having nbdev_new move a known list of files (*.ipynb, *.css) to cfg.nbs_path.
We wouldn't have to keep that list in our code if we could move those files into a new sub-dir of nbdev-template, but that would break nbdev_new for prior versions of nbdev.
cc @hamelsmu @jph00

Message ID: @.***>Sounds good to me!

Thanks, fixing this with #1096

Closing. https://github.com/fastai/nbdev/issues/1103 is a better solution to the underlying problem


Slot component causes a warning in React 19 about accessing .ref directly

Bug report
Current Behavior
The Slot component accesses .ref on the element, which seems to be a no-no in React 19. The problematic line seems to be this one:
https://github.com/radix-ui/primitives/blob/main/packages/react/slot/src/Slot.tsx#L66
This results in the following warning:

Warning: Accessing element.ref was removed in React 19. ref is now a regular prop. It will be removed from the JSX Element type in a future release.

Expected behavior
No warnings
Reproducible example
CodeSandbox Template
Suggested solution
I haven't tried it, but maybe something like
ref: forwardedRef ? composeRefs(forwardedRef, (children as any).props.ref ?? (children as any).ref) : (children as any).props.ref ?? (children as any).ref,

Your environment

Software
Name(s)
Version

Radix Package(s)
Any using Slot
latest

React
n/a
19.0.0-beta-6946ebe620-20240508

A PR is open to address this. https://github.com/radix-ui/primitives/pull/2811

Got the same issue in a
React 18.3.1 ??!!!
NextJs 15.0.3.
App ..
Console Error

Accessing element.ref was removed in React 19. ref is now a regular prop. It will be removed from the JSX Element type in a future release.

Call Stack
Next.js
SlotClone
node_modules/<EMAIL_ADDRESS>(40:47)
Next.js
react-stack-bottom-frame
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
renderWithHooks
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
updateForwardRef
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
beginWork
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
runWithFiberInDEV
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
performUnitOfWork
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
workLoopConcurrent
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
renderRootConcurrent
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
performWorkOnRoot
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
performWorkOnRootViaSchedulerTask
./node_modules/next/dist/compiled/react-dom/cjs/react-dom-client.development.js
MessagePort.performWorkUntilDeadline
./node_modules/next/dist/compiled/scheduler/cjs/scheduler.development.js


Update Get-PublicFolderItemStatistics.md

'LastAccessedTime' is only available for on premises versions of Exchange and is not available for Exchange Online.

Docs Build status updates of commit 5930d92:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 5930d92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d15796c:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit d15796c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 663d04d:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 663d04d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 0bf4573:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0bf4573:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0bf4573:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0bf4573:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit c228b06:
:clock10: Full build: the average full build time is 11 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit c228b06:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


🔧 (eleventy-plugin-local-respimg) Fixes failure on permalink: false

Re: liamfiddler/eleventy-plugin-lazyimages#13

@googlebot I signed it!


Migration to swift 4 + some checks

I added all swift4 required changes + some checks I required with some kml files that have extra spaces between tags
I hope it helps! :)

Is there any reason this can’t be merged? I notice it includes a fix removing the assert for <2 points on a line, which was the cause of a crash I recently experienced using this library. If this entire PR isn’t ready for merging, I’m happy to write one up for just the line assertion fix.

Thank you!


Set transparency background color when saving to jpeg?

Hi nfnt,
thank you for creating this great library. Saves me from using another non-go dependency like imagemagick :)
I am resizing PNG images to JPEG. These images can include transparency. What is the best way to change the black background color in JPEG to for example white?
I know that JPEG doesn't support transparency and so it would be nice if one could just change the black to another color.
Best regards
Bijan

Hi Bijan,
part of the appeal of this library is, that's it's very simple and easy to use. It should provide a solution to common image scaling tasks. It is not meant to be the best solution for every imaginable use case -- that would make it much harder to use, as the API would have to support all of these use cases. Your suggested use case would result in a more complicated API to support the old as well as the new behavior. So, let's keep it simple, thus keep things as they are right now.

Sounds feasible.


first.py in vendor isn't used anywhere and can be removed

I tested with:
oz123@:~/Software/pipenv$ grep -r "first("
pipenv/patched/pip/_vendor/pyparsing.py:            def remove_first(tokens):
pipenv/patched/pip/_vendor/requests/packages/urllib3/util/url.py:def split_first(s, delims):
pipenv/patched/pip/_vendor/requests/packages/urllib3/util/url.py:        >>> split_first('foo/bar?baz', '?/=')
pipenv/patched/pip/_vendor/requests/packages/urllib3/util/url.py:        >>> split_first('foo/bar?baz', '123')
pipenv/patched/pip/_vendor/requests/packages/urllib3/util/url.py:    url, path_, delim = split_first(url, ['/', '?', '#'])
pipenv/vendor/jinja2/asyncfilters.py:async def do_first(environment, seq):
pipenv/vendor/jinja2/filters.py:def do_first(environment, seq):
oz123@:~/Software/pipenv$ grep -r "import first"

Sorry, grepped on early version (not master).
$ grep -r "import first"
pipenv/vendor/first.py:>>> from first import first
pipenv/vendor/first.py:>>> from first import first
pipenv/vendor/first.py:>>> from first import first
pipenv/core.py:    from pipenv.vendor.first import first


Command mappings () do not work if MacVim was called from Vim

To Reproduce
Run the TUI Vim. Launch MacVim with:!mvim or by typing mvim in :term. Mappings with the Command key do not work (like, pressing ⌘-v in Insert mode will result in <D-v> being inserted, pressing ⌘-a in normal mode will result in entering insert mode with :help a command)
Expected behavior
⌘ mappings should work.
Environment:

Vim version:
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Oct 11 2020 06:56:08)
macOS version
Included patches: 1-1830

MacVim version:
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Sep 21 2020 04:03:24)
macOS version
Included patches: 1-1719

Hmm, this is an odd issue. Don't think I would have thought people use Vim that way. But I think the issue is the regular Vim would set the VIMRUNTIME environmental variable, and then when you use regular Vim to launch mvim, MacVim inherits the VIMRUNTIME from it, which is why it lost the MacVim-specific functionality.
I'm not sure if I want MacVim to lock the runtime dir because launching Vim with a customizable VIMRUNTIME is allowed in regular Vim. I think that does mean launching different types of Vim this way is probably not advised. I think you could just make sure to use TUI MacVim to keep it consistent. Otherwise if you really want to use the regular TUI Vim to launch MacVim, you could call unset VIMRUNTIME first.

Thanks, @ychin!


Enabled near hover, pick and grab for WebXRControllerPointerSelection

Introduced the following to the webXRControllerPointerSelection code:
nearPick
nearHover
pick PickingInfo (it is now common for both near and far interaction)
hover PickingInfo

Do you have any plans to open up onXRFrame to support both hands in near interaction? Right now it looks like the check at the start of the function limits usage to the dominant device/hand, since Babylon.JS traditionally supports just one pointer.

For instance all dist folder should not be pushed

You are right, my latest push should not have contained all that. i think there is something wrong with it, lemme fix it!


Add missing comma to Auth0 SAML field mappings

What changed, and why it matters
Fixes the content for Auth0 SAML docs field mappings so that it's a valid configuration.

Edited the title :) Brain not functioning today it seems.


Revert "Add deprecation notive for Hub V1 API"

Reverts docker/docker.github.io#15461

DO NOT MERGE. Waiting for Silvin to confirm the updates from his team.


why use 24 and not 18 for default digits?

18 decimals is very common in ERC20 tokens and is a pretty useful standard.  https://github.com/paulrberg/prb-math/ has a library called PRBMathUD60x18 that is a standardized 18 decimal number.  Anyways, am going to refactor just wanted to get some background on this design decision.

There are definitely better libraries out there at this point for fixed pointed integers. Especially for basic math, this project was mainly an exploration of implementing higher order functions in a fixed integer form. I would suggest using a 18 decimal version. At the time I was building on top of fixidity library extending it to hyperbolic functions, I liked there approach to the fixed point math however 18 digits or eve 32 is more sensible for most computations.  In the case of hyperbolic functions I would caution to think carefully about approximations and to what degree of precision you need.


Add grouped convolution support

Issue: https://github.com/tenstorrent/tt-metal/issues/8340

@tapspatel I think we should wait a couple days -- to get the new c++ convs in and then update this PR to use the new version.

pipeline: https://github.com/tenstorrent/tt-metal/actions/workflows/all-post-commit-workflows.yaml

@tapspatel could you please use the new test_new_conv2d.py for the latest version?

pipeline: https://github.com/tenstorrent/tt-metal/actions/runs/9356946489


Syndicate pAI for uplink [broken]

About the PR

Part 2 for #18625, changed the description to retain the HHGTTG reference and added it to the uplink. You can only buy one pAI per uplink to prevent ghost role spam.
Media

https://github.com/space-wizards/space-station-14/assets/44417085/da6671d6-340c-4c47-bdf5-193ff56865a5

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: crazybrain

add: Uplinks can now purchase one Syndicate pAI.

sprite definitely should be looking like normal

I think I broke something.

you committed engine somehow
git checkout --theirs RobustToolbox and commit

Okay so now it's worse

I'm just going to make a new branch for this.


[FEATURE] Enable posting without images

Description
Currently, users must upload an image when posting. Which is not a good UX.
It would be great to enable users post without uploading an image
Screenshots
No response
Additional information
No response

I have added this functionality ,kindly review my pr

I have added this functionality ,kindly review my pr

Kindly follow a pattern or you'll get tagged as spam

@partik03 bro, @mathiasayivor is the author of this issue, if he doesn't want to work on that , then you can. Or you can find bugs and raise a issue by yourself as well.

@narayan954 you can assign this to @partik03 since he's already created a pr.

okay sure , @partik03 you have a green light!

@narayan954 @mathiasayivor I really apologize as this issue was opened and no one was assigned so I thought I could work on it.
@narayan954 I have already added my pr kindly review it.

@partik03 its ok , even though no one was assigned , you could've dropped a message and then the pr.
I commented on that PR , please check it.


chore(merge-back): 1.20.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: b943ab504af44b9b8237f3f422feb3c58e21d386
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Not Getting Array Paths in Database of Laravel Project

Hey I have Problem with multiple files array images while submitting images.
Description:
I have form in which form I have three files input, First one is for single image selection , and two are array in multiple way,
Probelm is: When i submit form i'm getting everything from form except last array file images in database but getting images in public->uploads folder after submission,
What can be possible issue for it ? how i can solve it ?

Laravel Version: 5.6
PHP Version: 7+
Database Driver & Version: MySql 7+

Thanks
Moix

@Moix1  Can you show your form code?

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
Can I suggest closing your issue ticket here? Instead try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

Thanks in advance.


[Bug Report] Dialog contents not destroyed even with :destroy-on-close="true"

Element UI version
2.13.0
OS/Browsers version
Chrome Version 79.0.3945.79
Vue version
2.6.11
Reproduction Link
https://codepen.io/hollowcast/pen/MWYbQBw
Steps to reproduce
Create a dialog with option :destroy-on-close="true"
What is Expected?
Contents of the dialog should be destroyed after it closes. (Similar to the behavior of drawer destroy on close)
What is actually happening?
Contents are not destroyed.

Since your Form value is entered and determined, the value of the Form still exists when the dialog element is destroyed, and will be re-assigned when you reopen it

Is there a way to solve this now？


[HUDI-2383] Clean the marker files after compaction

What is the purpose of the pull request
Clean the marker files after compaction
Brief change log

Clean the marker files after compaction
Add tests

Verify this pull request
This change added tests and can be verified as follows:

TestInlineCompaction
TestAsyncCompaction
TestHoodieMergeOnReadTable.testClustering

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run travis

@minihippo : Can you rebase and let me know once CI succeeds. I can land it once ready.

@minihippo : Can you rebase and let me know once CI succeeds. I can land it once ready.

Done.


PERF: Avoid creating multiple instances of map entry when using a stream iterator

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Here is one example of performance improvement (Top N Benchmark from Druid incremental index).
Notice that we see an improvement in the first test.
Oak master
Benchmark                                  (indexType)  (numSegments)  (rowsPerSegment)        (schemaAndQuery)  (threshold)  Mode  Cnt        Score       Error  Units
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000                 basic.A           10  avgt<PHONE_NUMBER>.345 ? 42883.469  us/op
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000       basic.numericSort           10  avgt<PHONE_NUMBER>.476 ? 38590.743  us/op
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000  basic.alphanumericSort           10  avgt<PHONE_NUMBER>.948 ?  9310.287  us/op

Oak Stream Iter
Benchmark                                  (indexType)  (numSegments)  (rowsPerSegment)        (schemaAndQuery)  (threshold)  Mode  Cnt        Score       Error  Units
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000                 basic.A           10  avgt<PHONE_NUMBER>.596 ? 38920.114  us/op
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000       basic.numericSort           10  avgt<PHONE_NUMBER>.398 ? 62738.640  us/op
TopNBenchmark.querySingleIncrementalIndex          oak              1           2000000  basic.alphanumericSort           10  avgt<PHONE_NUMBER>.378 ? 12070.627  us/op

Thank you! I see the Druid improvement. I believe we should see also improvement in Oak's benchmark related to entry stream scan. Do we have the results of Oak's benchmarks? I cannot find them... If there is nothing surprising in the Oak's benchmarks and we do see improvement in the entry stream scan, I will merge the pull request.


trigger rebuild

Let's also turn on the Windows build: https://github.com/AnacondaRecipes/pugixml-feedstock/pull/1/files


go-import-add is very slow

go-import-add when triggered is very slow as it scans through a lots of directories. Is it possible to cache the previous results for faster response?

I'm using a custom go-packages-function function: go-find-external
It uses the external GNU find command to index the packages. On  my system it's 4 times faster than go-packages-native

Thanks @juergenhoetzel

go-import-add when triggered is very slow as it scans through a lots of directories. Is it possible to cache the previous results for faster response?

In the common case, your operating system will cache these file system accesses and speed up consecutive runs. Unless you're severely memory constrained, these cached entries will likely persist for your entire development session.
In a future, process-backed rewrite of go-mode, we will look into forms of caching. For the current version of go-mode, caching isn't viable. We would need a way to invalidate the cache, and I am not inclined to look into file system monitoring in pure elisp.

In #299 we deprecated (go-packages-native) and switched go-import-add to use (go-packages-go-list) by default. This is much faster for go modules than it was for GOPATH projects.
As people switch to modules they will get this speedup.


When 'TERM=alacritty-direct' micro takes a long time to start. Wrong background colour

When TERM is set to 'alacritty-direct', trying to run micro will cause it to hang for a long time (30 seconds or so). Afterwards micro does run, but the background becomes completely blue. This doesn't happen when TERM is set to 'alacritty'.
To reproduce, run the following in either alacritty or xterm.
export TERM=alacritty-direct
micro

Version: 2.0.8
Debian GNU/Linux bullseye/sid x86_64
alacritty, xterm.

Do you have a terminfo entry for alacritty-direct?

Do you have a terminfo entry for alacritty-direct?

I do. Here's the output:
infocmp.log
This happens with xterm-direct too apparently (#1874). Noticed this issue only recently. It's exactly the same issue as mine, albeit different TERM value.

I do. Here's the output:
infocmp.log
This happens with xterm-direct too apparently (#1874). Noticed this issue only recently. It's exactly the same issue as mine, albeit different TERM value.


java11 or recent java is required to run.

Issue Type: Bug
i have downloaded the recent version still my vs code is not working
VS Code version: Code 1.47.3 (91899dcef7b8110878ea59626991a18c8a6a1b3e, 2020-07-23T13:12:49.994Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.91GB (1.92GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

vscode-html-css
ecm
0.2.3

cpptools
ms-
0.29.0

java
red
0.65.0

python
tht
0.2.3

vscodeintellicode
Vis
1.2.9

vscode-java-debug
vsc
0.27.1

vscode-java-dependency
vsc
0.10.2

vscode-java-pack
vsc
0.9.1

vscode-java-test
vsc
0.24.0

vscode-maven
vsc
0.23.0

few days back it was working fine. But al of sudden I started facing this problem

java11 or recent java is required to run

@tripadeep where do you get this error?

java11 or recent java is required to run

@tripadeep where do you get this error?

@rebornix whenever i try to run my program i face this problem along with it it shows there is problem activating inteli  for language java

java11 or recent java is required to run

@tripadeep where do you get this error?

@rebornix whenever i try to run my program i face this problem along with it it shows there is problem activating inteli for language java

@tripadeep - Are you using Java extensions, specifically the popular Language Support for Java by Red Hat?
If so, you'll be getting that error if your extension auto-updated, as the latest version 0.65.0, released about a week ago, requires Java 11.
You can roll back to the prior extension version and it'll work, or you can move to Java 11...which may not be possible depending on your project.

@chucknelson ok.. I'll check it once...

java11 or recent java is required to run

@tripadeep where do you get this error?

@rebornix whenever i try to run my program i face this problem along with it it shows there is problem activating inteli for language java

@tripadeep - Are you using Java extensions, specifically the popular Language Support for Java by Red Hat?
If so, you'll be getting that error if your extension auto-updated, as the latest version 0.65.0, released about a week ago, requires Java 11.
You can roll back to the prior extension version and it'll work, or you can move to Java 11...which may not be possible depending on your project.

@chucknelson ok I'll check it once


Linux: Use a DispatchSourceRead instead of a FileHandle.readabilityHandler

swift-corelibs-foundation does not currently support FileHandle handlers
so use DispatchSource.makeReadSource instead of
FileHandle.readabilityHandlerForReading if not on macOS.

This is only a temporary fix and there may be other Linux issues, but it was enough to get the example in the readme working (although I had to rebase this fix on top of tag swift-DEVELOPMENT-SNAPSHOT-2018-08-25-a for testing).

@spevans, @akyrtzi, @ahoppen, @aciidb0mb3r

This is only a temporary fix and there may be other Linux issues

I have been using a SwiftSyntax on Linux for a while now (for documentation generation and proofreading), though I have been doing it with the source from the 4.1.2 release. There were only three things I needed to adjust:
(I was lazy and hoisted SwiftSyntax up above several other modules to make it easier, so you won’t want to copy what I did, but the links will still help you find the issues.)

The compiler invocation. This you already found and fixed.

The search for the compiler. This is more critical on Linux because Swift tends to get installed all over the place instead of just as part of Xcode. It might be a good idea to provide an API to allow clients a means of specifying the location—and thereby the right Swift version (just like they can when linking against the Package Manager).

Looking at the current source, changing this...
private static let _swiftcURL: URL? = SwiftcRunner.locateSwiftc()

...to...
public static var swiftcURL: URL? = SwiftcRunner.locateSwiftc()

...would probably suffice. (private → public, let → var, drop underscore?)

An annoying fatal error. Not really Linux specific, but SwiftSyntax was pretty much useless out here in the wild as long as it chose to terminate instead of just report an unknown node. The “get” and “set” keywords were its most notorious triggers, but maybe all that is already fixed in the 4.2 branch. My own test files are here, if someone wants to check against them with 4.2 (or add pieces of them to this package’s test suite).

For the compiler search, it looks like the full path can be passed to the SwiftcRunner(sourceFile: URL, swiftcURL: URL? = nil) constructor otherwise it will search the PATH to find it.
see https://github.com/apple/swift-syntax/blob/master/Sources/SwiftSyntax/SwiftcInvocation.swift#L175

@spevans,

For the compiler search, it looks like the full path can be passed to the SwiftcRunner(sourceFile: URL, swiftcURL: URL? = nil) constructor otherwise it will search the PATH to find it.

Hmm... You’re right. But the structure itself is internal. Does it actually surface in the API somewhere?...
...never mind. I found it. It does. I guess that’s been dealt with since 4.1.2.

Thanks for the great work here.
The ability to specify the swiftc executable to SyntaxTreeParser was added here: https://github.com/apple/swift-syntax/commit/fbd16280a6f782defaa05e1c5e5b1011976d5f73. We should maybe cherry-pick it over to swift-4.2-branch so that we can get Linux support for that branch as well.
The fatalError has also been resolved on master (https://github.com/apple/swift-syntax/commit/b3171b7b00c0c5fc3d8b07a1d53d048218fb010b). If useful for Linux development we can also cherry-pick that over to swift-4.2-branch.

Looks good! Thanks a lot for tackling this.
I would like for PR testing to be set up before merging this but that shouldn't take much longer.
Also, as I understand it, SwiftSyntax will completely work on Linux with this PR, right? If so, you can also close SR-8670 once this is merged.

@ahoppen Ive just updated the tests so the swift test will work correctly on Linux. It needs the tests to be explicitly listed on Linux and a LinuxMain.swift added as well. All of the tests pass on Linux.

That's amazing! Thanks for all your work on this.

https://ci.swift.org/view/Pull Request/job/swift-PR-osx/7361/
https://ci.swift.org/view/Pull Request/job/swift-PR-Linux/7402/
CI for linux is not activated yet, but Mac is passing so let's 🛳 it.


Add log_follow_unfollow, log_filename

Firstly, fixes the fact that though log_filename is defined in api init, it is not defined in bot init, and so is always None,
Also adds an option log_follow_unfollow (default True) that also writes follow/unfollow messages to the log (currently, only like messages are written, follow/unfollow is only written to console)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a6847d6). Click here to learn what that means.
The diff coverage is 25%.

@@            Coverage Diff            @@
##             master    #1091   +/-   ##
=========================================
  Coverage          ?   41.29%
=========================================
  Files             ?       29
  Lines             ?     3453
  Branches          ?      576
=========================================
  Hits              ?     1426
  Misses            ?     1879
  Partials          ?      148

Impacted Files
Coverage Δ

instabot/bot/bot_unfollow.py
7.31% <0%> (ø)

instabot/bot/bot.py
74.25% <100%> (ø)

instabot/bot/bot_follow.py
29.19% <40%> (ø)

Continue to review full report at Codecov.

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


chore(wandb-workspaces): Update README

Updated the readme

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.88%. Comparing base (77a946d) to head (1fe6883).

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   70.88%   70.88%
=======================================
  Files          16       16
  Lines        2356     2356
=======================================
  Hits         1670     1670
  Misses        686      686


Fails with Zip64 files (large files)

When adding files larger than 4GB. Trying to unzip gives the following error:
error: invalid zip file with overlapped components (possible zip bomb)

warning [text.zip]:<PHONE_NUMBER> extra bytes at beginning or within zipfile
(attempting to process anyway)
error [text.zip]:  start of central directory not found;
zipfile corrupt.
(please check that you have transferred or created the zipfile in the
appropriate BINARY mode and that you have compiled UnZip properly)
when running unzip -t text.zip on zip larger than 4GB. Has about 10 1Gb files inside.

Ok I will take a look

Just its hard to test large file


Change URLs to work with Gwiddle

We need to change URLs from x10.mx to gwiddle.co.uk. Note that the developer sandbox is at dev.opensprites.gwiddle.co.uk now.

Maybe using / at the start is a good idea? Then we can use htaccess documentroot.

Yeah, I suppose we'd have to re-write the URLs again once we move to the main site.

Hey, I've migrated the site-user-api to work with the new forums- http://dev.opensprites.gwiddle.co.uk/site-api/user.php?userid=1 = http://opensprites.gwiddle.co.uk/forums/?p=member/1-cheeseeater

Forums aren't working any better for me...

I'm able to login


Jco transpile: implementation of wasi-logging for Web

Tasks

[ ] logging.log

WASI logging is being removed so we can delete these.


We need a Search Grid

Hello Jean,
I realized that to get all coordinates of all plants, a search grid will be nessesary at some point.
And before some arbitrary grid gets implemented I took it upon me to research the (your) map in depth and combine it with all the data I accumulated about in-game measurements.
And I finally cracked the code.
I will attach the gist of it as maps.
There are three versions:
A. Dodecal grid MILES. My preference, the MILE grid (dodeca = 12 in Greek). Every 12 degrees are exactly 1 in-game mile. That makes it not only a fine search grid but it also serves as a Mile grid. (Duh)
B. Octal grid. The octal grid. This is how the (digital) logic of the map works. All binary numbers. This is the only grid that has nice even center coordinates.
C. Decimal grid. That might seem be the most obvious, but it's not. It has nothing to do with the digital numbers of the map, which are all powers of two.
ATTENTION: The decimal grid does NOT represent KILOMETRES! It represents 10 degrees of map coordinates! The map distance measurements are entirely in miles.

I took it one step further and named the resulting sectors of the mile-grid.
There are 84 square miles that actually need to be searched. That's manageable I think.
The idea is that individuals can get (a range of) sectors assigned in which they can collect all plant coordinates.
I myself will start collecting coordinates in sector H7.

Thanks, I'm testing some plugins to create a grid.

Getting my logical senses hyped about this. Jean. Great work so far mate. Im impressed with the herb map as im a loyal user of the herbs. Should add a link on the main map to the herb map so others can link to it freely. Great work.

Grid L9 done. Also I'm couting how many plants are in the area.
E.g: Burdock Root #10, 1 marker for 5 plants

My current progress is here:  https://jeanropke.github.io/RDOMap/assets/images/RDO Full Map.jpg


Building still fails with Python 3.4?

Sorry for bothering you again with the python version 3.4. But I still can't build it.
Here is the current error what I see when running build.py:
I wouldn't like to run testing distribution yet, so no version 3.5. :-(

It's working! I love this.
The building itself took over 1 hour even though I have pretty decent PC.  By the way, is there some reason why there aren't pre-built packages available for the new releases to download (e.g. for Debian/Ubuntu)?

Glad to hear it's working now.

By the way, is there some reason why there aren't pre-built packages available for the new releases to download (e.g. for Debian/Ubuntu)?

That's because I haven't released a new version yet. I use tags to denote snapshots of the master branch that I deem stable or "good enough" for general use.
As to why I haven't tagged a new version yet: there are a few things I wanted to fix related to buildlib, but I guess they can wait. I might tag a new version soon.

Yeah no hurry man, better safe than sorry, it's all good. Keep up the good work.


flow incorrectly reporting errors only when using create-react-app

Is this a bug report?
Yes

Can you also reproduce the problem with npm 4.x?

Yes
Environment

node -v: v8.0.0
npm -v: 5.0.1
yarn --version (if you use Yarn): 0.24.6
npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>
Then, specify:

Operating system: MacOS Sierra
Browser and version (if relevant):

Steps to Reproduce

(Write your steps here:)
// App.js
// @flow
import LoggedProfile from "./Profile";

const goodProfile = (
  <LoggedProfile
    username={"pcardune"}
    onEdit={() => console.log("not implemented yet")}
  />
);

// logOnRender.js
// @flow

export type FunctionalComponent<Props> = (props:Props) => React$Element<*>|null;

function logOnRender<FirstOrderProps>(FirstOrderComponent:FunctionalComponent<FirstOrderProps>):FunctionalComponent<FirstOrderProps> {
  return function NewComponent(firstOrderProps:FirstOrderProps) {
    console.log("Rendering with props:", firstOrderProps);
    return <FirstOrderComponent {...firstOrderProps}/>;
  }
}
export default logOnRender;

// Project.js
// @flow
import logOnRender from "./logOnRender";
import type { FunctionalComponent } from "./logOnRender";

type ProfileProps = {username:string, onEdit:Function}
function Profile(props:{username:string, onEdit:Function}) {
  return (
    <div>
      Your Username: {props.username}
      <button onClick={props.onEdit}>Edit Profile</button>
    </div>
  );
}
const LoggedProfile:FunctionalComponent<ProfileProps> = logOnRender(Profile);
export default LoggedProfile;

Expected Behavior

yarn flow v0.24.6
$ flow
No errors!
✨  Done in 0.17s.

Actual Behavior

yarn flow v0.24.6
$ flow
src/test/logOnRender.js:8
  8:     return <FirstOrderComponent {...firstOrderProps}/>;
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ props of R
eact element `FirstOrderComponent`. Expected object instead of
  8:     return <FirstOrderComponent {...firstOrderProps}/>;
                                         ^^^^^^^^^^^^^^^ FirstOrderPro
ps

Found 1 error
error Command failed with exit code 2.

Reproducible Demo

Code taken from:
https://github.com/saltycrane/flowtype-hoc-exploration/tree/3-files-withfix
which type-checks correctly both on the flow version used and the most recent version of flow
create-react-app version can be found here:
https://github.com/tmonte/flow-hoc.git

which type-checks correctly both on the flow version used and the most recent version of flow

Can you clarify "typechecks correctly"?
 This type is incompatible with
  6: type ProfileProps = {username:string, onEdit:Function}
                                   ^^^^^^ string. See: Profile.js:6

App.js:6
  6:   <LoggedProfile
       ^ React element `LoggedProfile`
  8:     onEdit={"not implemented yet"}
                 ^^^^^^^^^^^^^^^^^^^^^ string. This type is incompatible with
  6: type ProfileProps = {username:string, onEdit:Function}
                                                  ^^^^^^^^ function type. See: Profile.js:6

Found 2 errors

The README also says this is expected: https://github.com/saltycrane/flowtype-hoc-exploration/tree/3-files-withfix#results
However your "Expected Behavior" says you expect no errors.
So what is expected behavior?

I'm actually looking at goodProfile, so I've commented out the one that should fail. The issue is that flow is seeing an error in goodProfile in the create-react-app version, but not on th other one

I also think this could be just an issue with flow and nothing to do with CRA, but I only saw that behavior here

I ran yarn add<EMAIL_ADDRESS>in your CRA project, and it typechecks. So this is an issue caused by different versions of Flow. I don't know whether it's valid or not, but it's unrelated to CRA.

Note: to verify I had to apply this fix (https://github.com/evcohen/eslint-plugin-jsx-a11y/pull/274) because Flow currently mistakingly reports an error in eslint-plugin-jsx-a11y. You can put it in [ignore] in the config in the meantime.


Update Kotlin to v2.1.0

Note-to-self, potential fix: https://youtrack.jetbrains.com/issue/KT-70543/Gradle-create-migration-guide-for-those-who-are-using-Kotlin-compiler-classes-indirectly-available-in-buildscripts#focus=Comments-27-11111943.0-0

Superseded by #817.