Looking for maintainers!

Hi everyone,
Given that I am not using Wordpress these days and I haven't really been using WooPoly for a while. I am looking for maintainers to take over this project.
If you're interested, please reply to this issue or get in touch with me via email: hyyanaf [at] gmail [dot] com
I will give access to the project so you can contribute and move the project forward.
I can still handle uploading new releases to the WordPress repo.
Thanks!

@hyyan We would love to continue this project and take it forward.

hi, i php development i want to take your plugin to the best level !!! talk to me please :
<EMAIL_ADDRESS>+972535245050

@ahmedkaludi @wsdevcoil if you would like to contribute, please start by submitting a pull request or otherwise contribute to open issues.

Hi, did you found soemeone to maintain the plugin ?
I'm not interested but I like your plugin. I hope someone brilliant will continue your project.

@Niavl Unfortunately not ye

Interested.

valid ways to indicate interest are by contributing to closing issues and raising pull requests

Thank you.

Examples which might be good for those interested in taking over the plugin:

#208 import functionality
test WooCommerce blocks and implement any missing functionality
test the new WooCommerce 4.x provided features for Facebook and Google shopping etc and implement any missing functionality

For those interested in helping to get new functionality, test the features and raise detailed issue stating exactly what you expect to happen / what additional support is needed from the compatibility layer.

Hello @hyyan !
I am at a point where i want to translate a WooCommerce shop wit Polilang an i would get this to work with actual Versions of WooCommerce and Polylang...
Habe u got some advicec for me?

Hi @johannesgruber do have any updates?

Is this plugin working or is there any other plugin with this functionality?  I would like to use it in my website.

the answer to both questions is yes

@hyyan what will happen if nobody wants to help maintain this plugin? will there be no more updates in the future? hopefully whoever continuing to maintain this plugin can keep it free for the wordpress users. I see that some people forked this to create their own paid version...

@ewallz comments apply to any/all open source code.
There has been a release this year and there’s another one here: https://github.com/hyyan/woo-poly-integration/releases/tag/1.5.1  So at the moment some people are still helping and there are still updates


added support for docPub to also be a function

EXAMPLE:
new Tabular.Table({
  pub: function(sub, tableName, ids, fields) {
       var options = {};
       options.fields = fields;
       options.selector = {_.id: {$in: ids}};

       sub.subscribe('anySubscriptionName', options); //i have a pubSub system with an arguments definition like this
  }
});

any word on including this?

Hey brother no word on this one? It's the easiest pull request to merge in the book.

Thanks! It would be good to support this, but this is probably not the best way. It doesn't make sense to set pub to a function that actually "subs". I'm thinking possibly this instead:
new Tabular.Table({
  pub: false,
  sub: function(tableName, ids, fields) {
       var options = {};
       options.fields = fields;
       options.selector = {_id: {$in: ids}};

       return Meteor.subscribe('anySubscriptionName', options);
  }
});

What do you think?


Failed reflection resolving for .NET Core runtimes

Thanks for this!
For string encryption I get this error

Hi Holly, i've the very same problem on a .NET 3.5 dll
Could u suggest me a fix to try your library?
Thx!
A.

Your assembly was probably protected with an older version of Eazfuscator. Try de4dot.

Please, help me with an error. thank
https://drive.google.com/open?id=1pcAxZ1T7TmB2WsCwtyVPQycWADrlq0OT

This is an issue related to reflection resolving, not a support thread.

Closing since EazFixer now targets both .NET Framework 4.7.2 and .NET Core 3.1, and Harmony has been updated to v2.x.


[Sponsor] Specify what is Consent Number/ID

Add a tiny question mark as a superscript and provide definition for consent number/ID. Additionally, above the drop down for mandatory/optional, add a heading that says, “Consent Type”

"provide definition for consent number/ID" so what is the definition i need to add?


Properly use the 401 and 403 HTTP status codes

According to the HTTP standard, a lack of authentication credentials or
incorrect authentication credentials should send back a 401
(Unauthorized) with a WWW-Authenticate header with a challenge that
can be used to authenticate. This is because a 401 status should be sent
when an authentication attempt can be retried by the browser.
The 403 (Forbidden) status code should be sent when authentication
succeeded, but the user does not have the necessary authorization.
Previously, the server would always send a 401 status code.
Fixes #6959.

minor nit, otherwise +1

Great work. Thanks for the fast pickup of this issue!


Add the kwarg timeout to Connect to Browser

Is your feature request related to a problem? Please describe.
When calling New Browser the kwarg "timeout" determines the time that Playwright waits for a selector to be resolved. The standard value is 30s. When connecting to a running browser with Connect to Browser there is no way to change this time interval.
Describe the solution you'd like
Connect to Browser has a kwarg timeout.
Describe alternatives you've considered
I tried setting the timeout to 1s with Set Browser Timeout but Playwright still waits 30s to resolve a selector.
Additional context
Add any other context or screenshots about the feature request here.

Sounds reasonable
@allcontributors please add @mardukbp for ideas.

@aaltat This enhancement is not urgent. Feel free to lower its priority.

fixed in #3930


Release v0.30.0

Other changes:

Added RequiredPositionalArguments method for codegen (#773).
Support locking in integration tests (#776).
Update OpenAPI spec and fix incompatible changes (#778).

API Changes:

Added Exists method for w.Tables workspace-level service.
Added w.LakehouseMonitors workspace-level service.
Added catalog.CreateMonitor, catalog.DeleteLakehouseMonitorRequest, catalog.ExistsRequest, catalog.GetLakehouseMonitorRequest, catalog.MonitorCronSchedule, catalog.MonitorCronSchedulePauseStatus, catalog.MonitorCustomMetric, catalog.MonitorCustomMetricType, catalog.MonitorDataClassificationConfig, catalog.MonitorDestinations, catalog.MonitorInferenceLogProfileType, catalog.MonitorInferenceLogProfileTypeProblemType, catalog.MonitorInfo, catalog.MonitorInfoStatus, catalog.MonitorNotificationsConfig, catalog.MonitorTimeSeriesProfileType, catalog.TableExistsResponse and catalog.UpdateMonitor.
Added InitScripts field for pipelines.PipelineCluster.
Added ValidateOnly field for pipelines.StartUpdate.
Added ValidateOnly field for pipelines.UpdateInfo.
Changed CreateOboToken method for w.TokenManagement workspace-level service with new required argument order.
Changed Get method for w.TokenManagement workspace-level service to return settings.GetTokenResponse.
Changed LifetimeSeconds field for settings.CreateOboTokenRequest to no longer be required.
Added settings.GetTokenResponse.
Changed Create method for w.Dashboards workspace-level service . New request type is sql.DashboardPostContent.
Added Update method for w.Dashboards workspace-level service.
Removed sql.CreateDashboardRequest.
Added HttpHeaders field for sql.ExternalLink.
Added RunAsRole field for sql.QueryEditContent.
Added sql.DashboardEditContent.
Added sql.DashboardPostContent.

OpenAPI SHA: e05401ed5dd4974c5333d737ec308a7d451f749f, Date: 2024-01-23

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

Comparison is base (a28e2bb) 18.69% compared to head (be109d6) 18.69%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #780   +/-   ##
=======================================
  Coverage   18.69%   18.69%
=======================================
  Files         115      115
  Lines       15135    15135
=======================================
  Hits         2829     2829
  Misses      12071    12071
  Partials      235      235

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


Accept non-stringified request and response in webv test cases

Description

A clear and concise description of what the bug is
Currently,  people need to convert request and response to a stringified format by escaping double quotes etc to make them work in webv

Expected Behavior
json request response should be accepted as it is in webv test cases without the need for them to be stringified.
Reproduce
Enter request body and response without stringifying them. webv throws error.

Closed by: https://github.com/retaildevcrews/webvalidate/pull/8


Add release cli command

seems to already be available via grunt. Don't forget documentation.

This functionality is available with the version command.


Mendeleev does not work

Describe the bug
I installed Mendeleev with pip and my code does not work. I got this error:
Traceback (most recent call last):
File "c:\Users\Alan\Documents\VSCodeWorkspace\testing\Mendeleev-testing.py", line 1, in
import mendeleev
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\mendeleev_init_.py", line 27, in
from .mendeleev import *
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\mendeleev\mendeleev.py", line 32, in
from sqlalchemy import create_engine
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy_init_.py", line 12, in
from sqlalchemy.sql import (
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy\sql_init_.py", line 7, in
from sqlalchemy.sql.expression import (
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy\sql\expression.py", line 32, in
from sqlalchemy import util, exc
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy\util_init_.py", line 7, in
from .compat import callable, cmp, reduce, defaultdict, py25_dict,
File "C:\Users\Alan\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy\util\compat.py", line 196, in
time_func = time.clock
AttributeError: module 'time' has no attribute 'clock'
To Reproduce
Steps to reproduce the behavior:

Install Mendeleev with pip
Test it out

Expected behavior
I used the tutorial code to test Mendeleev out. Specifically, this chunk of code:
import mendeleev
from mendeleev import Si, Fe, O
print("Si's name: ", Si.name)
print("Fe's atomic number:", Fe.atomic_number)
print("O's atomic weight: ", O.atomic_weight)

I expect this result:
Si's name:  Silicon
Fe's atomic number: 26
O's atomic weight:  15.999
Screenshot
If applicable, add screenshots to help explain your problem.
Specification

mendleev version
python version
platform

Additional context
Add any other context about the problem here.

Update: I tried to install the conda version instead and it worked

Thanks for submitting the issue. Could you let me know what version of python are you using (python --version)?

Yes, it is Python 3.8.3

Thanks, since it's a problem related to sqlalchemy, could you check the version of that package you have installed? You should get it by typing python -c "import sqlalchemy; print(sqlalchemy.__version__)"  into your shell.

I checked my version of sqlalchemy and the version I have is 0.7.7.

Unfortunately I cannot reproduce your issue, but I'm on a different OS so it's hard to say definitively. I found this issue on stack overflow https://stackoverflow.com/questions/62436786/attributeerror-module-time-has-no-attribute-clock-in-sqlalchemy-python-3-8 which seems very similar. It might be worth trying to update your sqlalchemy version since 0.7.7 is at least 8 years old.
I'll update the setup.py and pin the version of sqlalchemy which should patch it.


[New Project]: BMI Calculator

I wish to add a project on BMI Calculator. This project will take height and weight as input from the user and will calculate BMI. It will also show the user, the category they falls in:

Normal Range
Underweight
Overweight

I'll be making this project using HTML, CSS and JavaScript.
Kindly assign me this issue under GSSOC'23.
@Dharmesh177

Sure assign to u
Complete it in 1 week and follow contributing.md file guide while making PR
Happy Coding !!


Responsive navbar

Hi,
Sorry if this is already covered somehow, but is there an easy way to make a responsive navbar?
Maybe this could be included as an example?

Hello @willfinnigan (sorry for the delay to answer your question).

Sorry if this is already covered somehow, but is there an easy way to make a responsive navbar?

The responsive navbar is something that we have in mind to improve for the v4.0.0. At this moment, You can use the sizes of the siimple-navbar element to make the navbar responsive (https://docs.siimple.xyz/layout/navbar.html and https://docs.siimple.xyz/layout/overview.html).

Maybe this could be included as an example?

Yes, or maybe as a tutorial.

v4.0.0 will support responsive navbar using only CSS (no Javascript or extra HTML elements are required)  :tada:


update config-template to reflect the placeholder names provided in staged-config

In stage-config the --include-placeholder flag provided params with names, using _ as the separator (not the .).
This means config-template would have params also in the format.
For example,
product-properties:
  .properties.some-name:
    value:
      identity: ((.properties.some-name.identity))
      password: ((.properties.some-name.password))

would look like
product-properties:
  .properties.some-name:
    value:
      identity: ((properties_some-name.identity))
      password: ((properties_some-name.password))

We're going to tackle this issue and submit a PR when it is completed.

dm $ om config-template --product ~/Downloads/p-isolation-segment-2.3.0-build.182.pivotal --include-placeholder | grep \(
      secret: ((properties_nfs_volume_driver_enable_ldap_service_account_password.secret))

made a pr related to this issue, reopening


Trigger tests for P/Z v4.3.8

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
TODO(replace-me)
Use this space to explain how you validated that your change functions exactly how you expect it.
Feel free to attach JSON snippets, curl commands, screenshots, etc. Apply a simple benchmark: would the information you
provided convince any reviewer or any external reader that you did enough to validate your change.
It is acceptable to assume trust and keep this section light, e.g. as a bullet-point list.
It is acceptable to skip testing in cases when CI is sufficient, or it's a markdown or code comment change only.
It is also acceptable to skip testing for changes that are too taxing to test before merging. In such case you are
responsible for the change after it gets merged which includes reverting, fixing, etc. Make sure you validate the change
ASAP after it gets merged or explain in PR when the validation will be performed.
Explain here why you skipped testing in case you did so.
Have you created automated tests for your change? Explain here which validation activities you did manually and why so.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/test help

/test powervs-4-12-qa-ebpf-e2e-tests
/test powervs-4-13-qa-corebpf-e2e-tests
/test powervs-4-13-qa-ebpf-e2e-tests
/test powervs-4-14-qa-corebpf-e2e-tests
/test powervs-4-14-qa-ebpf-e2e-tests
/test powervs-4-16-qa-corebpf-e2e-tests
/test ibmcloudz-4-14-qa-e2e-tests
/test ibmcloudz-4-15-qa-e2e-tests

/test powervs-qa-ebpf-e2e-tests

/test ibmcloudz-4-15-qa-e2e-tests

/test ibmcloudz-4-14-qa-e2e-tests
/test ibmcloudz-4-15-qa-e2e-tests

/test ibmcloudz-4-14-qa-e2e-tests
/test ibmcloudz-4-15-qa-e2e-tests
/test powervs-qa-ebpf-e2e-tests

/test powervs-qa-corebpf-e2e-tests
/test powervs-4-14-qa-corebpf-e2e-tests
/test powervs-4-13-qa-corebpf-e2e-tests

/test ibmcloudz-4-15-qa-e2e-tests

/test ibmcloudz-4-15-qa-e2e-tests


Add more docstrings.

enhance the codebase with informative doscstrings

@rajveer43 is there a section of the codebase you're specifically referring to, or is this just a general request?

Got it. Yes, please feel free to submit PRs for this. Thanks!


Write blog post "More Flexibility for Workflows in the OBS and SCM Integration"

The blog post now includes a screenshot for the form changes of both workflow configuration file options.


feat: Adds layers prop to SSR Functions

Not added for Next sites as necessary Node prop is also absent.

Very useful PR, follow to see when added to codebase

Thanks @bayssmekanique, made a quick update.
Moved layers into cdk.layers. The thinking was that any prop that takes an CDK object goes into cdk.*. And if that prop becomes commonly used, SST will figure out a way for ppl to pass that in without needing to import CDK.
Can u give it a quick test?

@fwang, it appears there are permissions missing for the IAM role to get a LayerVersion from an ARN.
        cdk: {
          server: {
            layers: [lambda.LayerVersion.fromLayerVersionArn(stack, 'ChromeLayer', layerArn)]
          }
        }

|  Site astro-site/ServerFunction/ServerFunction/ServiceRole/DefaultPolicy AWS::IAM::Policy CREATE_COMPLETE
|  Site astro-site/ServerFunction/ServerFunction AWS::Lambda::Function CREATE_FAILED Resource handler returned message: "User: arn:aws:sts::856863919685:assumed-role/cdk-hnb659fds-cfn-exec-role-856863919685-us-west-2/AWSCloudFormation is not authorized to perform: lambda:GetLayerVersion on resource: arn:aws:lambda:us-east-1:764866452798:layer:chrome-aws-lambda:22 because no resource-based policy allows the lambda:GetLayerVersion action (Service: Lambda, Status Code: 403, Request ID: 304ccc37-ed2e-4418-86fd-158451888f08)" (RequestToken: e6fa1c44-68cf-b507-9c25-bc93fc9b7e2a, HandlerErrorCode: AccessDenied)
|  Site astro-site/Certificate/CertificateRequestorResource AWS::CloudFormation::CustomResource CREATE_FAILED Resource creation cancelled
|  Site astro-site/S3Handler/ServiceRole/DefaultPolicy AWS::IAM::Policy CREATE_FAILED Resource creation cancelled

Lambda is deployed to us-west-2 and the layer is in us-east-1. Could that be the issue? Also, does that layer allow cross account usage?
^ I havne't used layers much.. I might not know what I'm talking about

Good call @fwang, that was the issue. This appears to work as expected using the CDK configuration.

@fwang anything else needed on this?

Going to create a new PR that's for SSR, not just Astro.


java.io.IOException: Error getting access token for service account

Hi guys!
I am using the 1.4.0 version of com.google.auth:google-auth-library-oauth2-http library and sometimes I get this error:
java.io.IOException: Error getting access token for service account: Remote host closed connection during handshake, iss<EMAIL_ADDRESS>at com.google.auth.oauth2.ServiceAccountCredentials.refreshAccessToken(ServiceAccountCredentials.java:605)
I don't know really how to solve this, this is affecting production and I need to solve this as soon as I can.
this is the code I wrote to get the AccessToken:

        GoogleCredentials googleCredentials = GoogleCredentials
                .fromStream(classPathJsonfile.getInputStream())
                .createScoped(Arrays.asList(SCOPES));
        AccessToken token = googleCredentials.refreshAccessToken();

        this.token = token.getTokenValue();
        this.expireOn = token.getExpirationTime().getTime();

        return this.token;
    }

Thanks!

Hi @jcdesignweb , sorry for a delay, did you find the root cause?

Please reopen if you're still experiencing this issue.


Kube template

Resurrect the Kubernetes template. The template will be consumed by catasb.
Changes proposed in this pull request

Jinja2 k8s template
Python script to render the template using variables in k8s-variables.yaml

 (20) Agnostic Runtime phase 1

This works for me now. Could use some help testing from folks with different environments.


Need for cropImage(image,path) canvas flutter.

There is a need for a function like crop images that crop the image and resizes it according to actual image width and height. the function similar to the clip image.

Hi @abhishekove
You need to be clear while posting a proposal for Flutter.
Where this cropImage should be added?
How it should exactly work?
Should it return the cropped image?
Please elaborate more.

According to me cropImage must be in canvas and must return cropped and transformed image.

I'm still not sure what you are asking exactly so will leave this opened for the right person to check.
Cc @dnfield

It would be helpful to know what exactly you're trying to do, and what you've tried so far, and how it's failing.
You may be interested in:
https://api.flutter.dev/flutter/dart-ui/Canvas/drawImageRect.html
Which you could use to draw a cropped portion of an image and then use something like https://api.flutter.dev/flutter/dart-ui/Picture/toImage.html to rasterize it again.
There are no plans to add any additional API to Flutter for scaling/cropping images - but there should be enough base API to write a package to do this if you want.

@abhishekove Does the information in https://github.com/flutter/flutter/issues/68984#issuecomment-717065470 helps you solve your use-case?

I want something like this.
https://github.com/flutter-clutter/flutter-simple-edge-detection

That repo uses dart ffi to do all the processing in C++.  You could either crop the image in C++, or crop it in flutter using the API I mentioned (or perhaps drawAtlas), but it will be more efficient to just crop it on the native side so there's less to decode when drawing it in flutter.

Closing this as resolved. We have api to do this.

@dnfield drawImageRect does not crop an image. For example, in the following:
canvas.drawImageRect(
 image,
 Rect.fromLTWH(0, 0, 2400, 3000),
 Rect.fromLTWH(500, 500, 1000, 1000), Paint())

Just draws the entire 2400x3000 image at offset (500, 500) but scaled to fit a 1000x1000 area.

@dnfield drawImageRect does not crop an image. For example, in the following:
canvas.drawImageRect(
 image,
 Rect.fromLTWH(0, 0, 2400, 3000),
 Rect.fromLTWH(500, 500, 1000, 1000), Paint())

Just draws the entire 2400x3000 image at offset (500, 500) but scaled to fit a 1000x1000 area.

You are right. I think I meant to link drawAtlas, which should work.  Otherwise using a cliprect with drawImage should work

You are right. I think I meant to link drawAtlas, which should work.  Otherwise using a cliprect with drawImage should work


Extra List of Privescalation checks

Might find a few extra possibilities to check off this tweet. https://twitter.com/JaneScott_/status/1144134954353217536

Further binaries added by https://github.com/Anon-Exploiter in https://github.com/rebootuser/LinEnum/pull/43
Further additions welcome, but closing open issue at this time.


Breakpoints don't work when not using Theme

Description
Breakpoints inside createStylesheet() don't work when the theme is not used / it's not returned as a function
🔴 can't work:
const stylesheet = createStyleSheet({
    container: {
        flex: 1,
        justifyContent: 'center',
        alignItems: 'center',
        backgroundColor: {
            sm: 'cyan',
            md: 'magenta'
        },
    },
};

🟢 working correctly:
const stylesheet = createStyleSheet(() => ({
  container: {
        flex: 1,
        justifyContent: 'center',
        alignItems: 'center',
        backgroundColor: {
            sm: 'cyan',
            md: 'magenta'
        },
    },
}));

Steps to reproduce

Try modif this example
Check switch to potrait & landscape mode on real device or emulator

Snack or a link to a repository (optional)
https://github.com/jpudysz/react-native-unistyles/tree/main/examples/expo
Unistyles version
2.0.0
React Native version
0.73.0
Platforms
Android
Engine
Hermes
Architecture
Paper (old)

Hey 👋
I have found the issue. There was acutally one less dependency in useMemo.
Thanks!
I will release 2.0.1 shortly


Phrase search / Exact match

Is your feature request related to a problem? Please describe.
I have a lot of documents with special characters like . (dots) and , (commas). THe most common use case is to search for an exact match:
"1. Title Chap. 2,3"
Describe the solution you'd like
Currently the tokenizer strips all . (dots) and , (commas). Using phrases with quotes is not supported as far as I see. If there is a way or a possibility to find all documents with the query above, please instruct me how to achieve that.
Describe alternatives you've considered
Algolia supports a phrase (query language) and also Lucene supports a query parser language.
Not having the possibility so search for an exact phrase can be a tough show stopper for a lot of scientific searches. I know, that meilisearch targets dummy users that just type a word and want a fast result.
But meilisearch could also be a successful search engine for scientific documents.
Even more if I could define a language specific tokenizer. But this is an other topic.

related to #902

Hello,
The phrase/exact search feature is something we would like to support, however the complexities of being able to search in documents with words that are abreviated and chapter numbers is a little more complex but we will look into maybe a setting that will allow the user to specify settings by document attributes and therefore be able to force the engine to ignore punctuation.
The "2,3" will be supported by the new incoming tokenizer.
Thank you for this well described issue :)

Great news! Thanks a lot for this Open Source effort! You are doing a great job and many (anonymous) user appreciate your hard work!
I really appreciate your help and looking forward to a possibility in meilisearch for phrase/exact search. If there is a new version or something to test, please mention it in this thread so I can test it.

Hello, Thanks for this feature proposal!
I will close this issue in favor of our public roadmap.
I invite everyone interested in this feature to upvote it on the roadmap.

is there any update on this?

Hello @2567910 it's now possible
you can quote your query in the q parameter


Visibility into Jenkins-readiness of instructor assignment repos

In the assignments index page for instructors, we would like to be able to know the "build status" of each assignment. This allows instructors to know whether they have properly set up their assignment repos for auto-grading.
We would like to be able to see an icon to the right of each assignment list item in the index page of the assignments view:
https://github.com/project-anacapa/course-github-org-tool/blob/feature/assignment-view/app/views/assignments/_index_instructor.html.erb#L5
The icons for each status are as follows:
PENDING == yellow hourglass
SUCCESS == green checkmark
FAILURE == red X
ERROR == red ?

You may want to add methods to the Git Strategy for accessing & updating the commit/build status for a given repository. You can add those to the files in this folder:
https://github.com/project-anacapa/course-github-org-tool/tree/feature/assignment-view/lib/strategies

Commit Status API for GitHub: https://developer.github.com/v3/repos/statuses/
Build Status API for GitLab: https://docs.gitlab.com/ce/api/commits.html#post-the-build-status-to-a-commit

Operations needed:
Set Commit Status
Get Commit Status

You probably want to add this method to the base class git_strategy.rb:
create_status(repo, sha, state)

You'll pass through an empty hash to the underlying octokit method documented o this page
http://octokit.github.io/octokit.rb/Octokit/Client/Statuses.html#create_status-instance_method
create_status(repo, sha, state, options = {})

For gitlab, we can just leave it alone for now, since we aren't using it yet.  The "not implemented yet" error will get thrown and that's ok for now.
That method comes directly from

We also need a get_most_recent_status(repo, sha) which wraps the
(Array<Sawyer::Resource>) statuses(repo, sha, options = {})

method but only returns the most recent status.


Implement #501 #509 #521 #522

Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Implement #501 #509 #521 #522

Coverage increased (+0.08%) to 73.92% when pulling f21b73a6f5829370028793f2435aa55e94581341 on lionel-bijaoui:lb_implement_501_509_521_522 into 21918ba05e438330aa26902c9df4f38fd1e14a8b on vue-generators:v3.


Find a way to futher optimize GROUP BY * queries

At the moment, we optimize queries like this well because we process the various iterators in parallel and perform what amounts to an iterative map/reduce:
SELECT count(value) FROM cpu

Unfortunately, we don't handle this type of query with the same grace. This type of query is bottlenecked at using one core.
SELECT count(value) FROM cpu GROUP BY *

The issue is with how internal processing of the query is done and how the final query is read. For the original query, we can process each series separately and perform an initial aggregate on a single series in parallel. Then we can combine the results of each series together. Since all of these points are processed in parallel with each other and the final result is read from only one location, it's much easier to merge them.
But, when we have GROUP BY * or a grouping by something with a high cardinality, the emitter will have to read all of the points from the first series before it can continue to reading the points from the second series. This means that only the first point in the second series is processed in parallel and then it blocks until it's time for the series to be read. Since series are processed in parallel, this makes GROUP BY * especially bad since it will inevitably only process series serially.
The emitter emits the series serially because that's what's required out of the emitter. It returns each series in order so it reads all of the series in order. While reading series 1, we cannot continue to process series 2 because the entirety of series 1 has to be emitted before we continue. The only way to continue processing series 2 would be to buffer the results of series 2 until they could eventually be read. With queries that emit a lot of points, this could result in running out of memory. If we limit the number of buffered points to prevent a query from running out of memory, we still don't solve the underlying problem which is that queries will always inevitably lead to using only a single core at some point within the query lifecycle. We just delay the issue.
Another solution might be to try and process a single series in parallel rather than serially like we currently do. This may or may not work. I'll expand on ideas, but we can't have a naive solution to doing this. The reason why we read a series in order is because seeking within the TSM tree generally doesn't improve performance very much. The slowest part of the query is finding the initial cursor and iteration is generally pretty fast after that. Note: Double check this assumption as it changes the math behind this if it's not true.
Either way, we will likely need a hybrid approach. Sometimes, processing the series in parallel is faster such as when we're merging series together. Sometimes, processing the series itself in parallel might be faster if we can think of a way to do that efficiently.
Another potential factor to consider is that this query will perform much slower than it likely should:
SELECT max(min) FROM (SELECT min(value) FROM cpu GROUP BY *)

This is because the parallel optimization only happens within the TSM1 engine and is not done to combine the results of subquery. That's likely an easier optimization we can introduce too.

@rbetts @timhallinflux hopefully this helps explain some of the potential problems with parallel query processing we have. There are at least two issues. One related to subqueries and one related to a fundamental difficulty within the system.

Continuing along with this of potential ideas for optimizing aggregates more, we would need to determine which part of the iterator is slow. How quickly can we process and discard an entire series of data and how much of that do we spend performing the aggregation? If processing the actual series is quick, we can work to quickly fill a buffer with the points for an interval and send those points to the proper aggregate. Then we let the aggregate do its more consuming work in a separate goroutine and keep the series processor moving.
If the series processing is not as fast as it should be, we should explore the idea of having a separate goroutine operate on finding the starting node for each time series so that we can perform more efficient seeking between nodes and hopefully create a zipper formation. So while one goroutine processes [0s, 10s), another would process [10s, 20s). Currently, we don't do this because seeking is far more expensive than just reading through.
We should also consider using sync.Pool to do buffer management for memory.  We can see if sync.Pool helps with that. It seems to help with memory sensitive libraries like zap.

Just ran a cpuprofile on the BenchmarkEngine_CreateIterator_Count_1M function (which should be one of the fastest to finish). The cpuprofile showed that 35.65% of the time was spent in the reduce() function (where the primary work is done for aggregates). 23.61% was spent finding the next point in the window. There's further down areas that might be optimizable too, but the difference in percentage between those might indicate something is there. No certainty though as if these are fast functions, the percentages may be meaningless.

Looked a bit at a flame graph related to this today (so 2 months since I last looked at it) and found that about 1/3 of the time spent within the call iterators is spent finding the subset of tags when there's a dimension specified. The other benchmark mentioned in the previous comment doesn't use a dimension inside of it so it doesn't represent this problem very well. Also, that function wouldn't have shown up on the flame graph because the influxql.(*Tags).Subset function returns immediately with an empty Tags struct if the number of dimensions is zero, which is not the case for something using GROUP BY *.
I'm also unclear how much parallelization may help although I'm willing to mess around with it a bit. It will be necessary to perform additional memory allocations (which we currently try to reduce) if we add parallelization, but the parallelization may offset any loses due to using a bit more memory. I'm going to try and experiment a bit. Just writing down my current thoughts at the moment.

Another note, it seems like a lot of time is spent in influxql.keysMatch. It checks if the keys of a map match so it can reduce doing a memory allocation, but this seems a bit silly since we should know before the reduction if the keys match or not. If you have SELECT mean(value) FROM cpu GROUP BY host, the values are sent with the correct subset to begin with.
Part of this problem seems to stem from subqueries (which I knew about and that problem of too many Subset calls should be fixed), but some seems to be present before subqueries existed. It may be time to try making this a bit smarter so it only retrieves the subset of tags when it needs to.
This is still just initial guessing. There may be something I forgot about for why things are this way.

this is still relevant.
It seems to me that the sorted-merge-iterator uses a heap to sort all points but the Less function is REALLY slow. As @jsternberg stated, the keysMatch is not efficient.
The performance gap caused by sorted merge iterator is significant. Just compare the time of two aggregations: SELECT mean(*) FROM m GROUP BY * which is not sorted, and
SELECT median(*) FROM m GROUP BY * which is. The difference is ~30x on my sample data.
Seems the heap doesn't have to do a point-to-point comparison if the inputs are from different series. I'd send a patch as RFC.


manually entered long URI does not wrap in Chrome

When a long URI is manually entered in the Sinopia editor (in Chrome) it does not wrap, therefore you cannot see the x symbol to delete it.  Putting a cursor on the URI to move to the end of the link serves to activate the link instead and takes you to the source vocab, so that is not an option.
My specific example is from the un-nested Serial Work template, Form of Work and the the URI http://id.loc.gov/authorities/genreForms/gf2019026015 .  It also didn't work in the rt2:Friday test RT that Michelle created (again, in Chrome).
MIchelle said that this does not happen in Firefox.

I replicated this issue using Chrome on my Mac laptop. In the stage environment opening https://stage.sinopia.io/editor/ld4p:RT:bf2:Serial:Work:Un-nested template and pasting the URL in the Form of Work field results in:

Opening the same resource template in Firefox show the delete button on the field:

Found the following two workarounds using Chrome.
Expand Chrome to full-screen shows the correct x delete button:

Or use the responsive feature of Sinopia and contract the view to a single column shows the correct  correct x delete button:

So, with Windows 10 and opening Chrome to full size, I still can't see the delete functionality.  But the responsive thing did work. Thanks!

I'm going to close this because it works in Firefox and because there is a Chrome workaround. If there is some reason we need to make this work in Chrome, please reopen.


feat: add support for Mistral Large 2 (24.07)

Issue #38 :
Description of changes:
added support for Mistral Large 2 (24.07)

@daixba  please review this PR.

Thanks for the PR


Merge Devel into Master

The tag related to the master branch is: https://github.com/robotology/walking-controllers/releases/tag/v0.2.1

Maybe it is worth running it once on the robot.

Probably we can now merge

Agree!


v3: remove obsolete QualifiedName

In the new symbol resolution rules, we always search the tabsyms
if there's no match in colsyms. This make the QualifiedName field
of the colsyms redundant because qualified references will be
matched when searching the tabsyms. In fact, that's what makes
the colsyms legitimate.
It turns out that QualifiedName was already not being used after
the change. So, we can just drop it.

@bbeaudreault @acharis I mentioned during the v3 presentation that QualifiedName may be obsolete. I just confirmed that it is the case.

don't we want to still set a '*' Alias in pushStar?

It's not needed. A colsym is useful only if it causes a new alias to be created. For example, col.id will produce id as alias. Otherwise, the matching is done against the tabsym underneath.

cool, then LGTM

LGTM


Added a code action for auto-fixing deprecated qubit binding syntax

This PR introduces code action to fix the deprecated qubit binding syntax.
I believe this would be a helpful change for the users in the short term - as it allows them to easily deal with the recent changes to the binding syntax, and opens up a better path to fully remove the old syntax instead of just deprecating it.
Since Q# compiler doesn't have "round-trippable" ("lossless") syntax trees at the moment, since QsFmt is not yet integrated in the main compiler code and since the change here is a rather simple one (it is a glorified trim start/trim end) I implemented it using a simple regex (and even that is not really needed, the same could really be achieved with string operations).
In case the regex fails, the fallback is to simply replace the keyword using -> use, borrowing -> borrow, instead of trying to handle the entire fragment.
Example:

The rule for the following curly brace are:

if it was on the same line as the binding keyword, it stays on the same line
if it was on its own line, it stays on its own line
if it was on a different line than the binding keyword, but shared that line with the closing ) bracket, it gets pulled up

Note that the code action is only offered on the WarningCode.DeprecatedQubitBindingKeyword (so really on the using/borrowing keyword) - it could be offered on the WarningCode.DeprecatedTupleBrackets but I wanted to keep it simple.

/azp run

/azp run

cc @tcNickolas who asked about something like this in another issue

@filipw Thank you for doing this! Too bad we already went through the Katas to update this syntax by hand :-) I'm glad, though, that after the next release this will be a more automated process!


improve typing annotations

improves type annotations throughout the codebase
bumps versions in pre-commit config
de-globs and tightens mypy whitelist

Finding it a little challenging to make much further progress on this... might need the author to fill in some of the blanks!
(@danwos ;) )


Running npm run demo fails on windows.

I am getting below error on windows. I tried on git bash which is closure to linux bash but same result.
C:\GitHub\ng2-formly>npm run demo

<EMAIL_ADDRESS>demo C:\GitHub\ng2-formly
NODE_ENV=dev node node_modules/.bin/webpack-dashboard --title ng2-formly-demo -- node_modules/.bin/webpack-dev-server --quiet --port 9000 --content-base demo --config .config/webpac
k.demo.js --open

'NODE_ENV' is not recognized as an internal or external command,
operable program or batch file.

can you try without NODE_ENV=dev node  ?

it does run without NODE_ENV=dev node


Unauthorized Error When Using Get-EXORecipientPermission

After activating my Exchange Administrator Azure AD role in Privileged Identity Management, I cannot use Get-EXORecipientPermission. I get an error that I am not authorized. If I use the non-EXO version of the cmdlet, it functions correctly.
I'm using the most recent version of the Exchange Online Management module (version 2.0.5). The screenshot below is for Windows PowerShell 5.1, but the issue also occurs when using PowerShell 7.1.3. Issue appears to be with the REST service.
It's unclear how best to let the product team know about these issues now that User Voice has been retired for use by Microsoft.

Error text:
Get-EXORecipientPermission : Error while querying REST service. HttpStatusCode=401
ErrorMessage={"error":{"code":"Unauthorized","message":"User is not allowed to call
Get-RecipientPermission","innererror":{"message":"User is not allowed to call
Get-RecipientPermission","type":"Microsoft.Exchange.Admin.OData.Core.ODataServiceException"}}}
At line:1 char:1
+ Get-EXORecipientPermission -Identity $identity
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ProtocolError: (:) [Get-EXORecipientPermission], RestClientException
    + FullyQualifiedErrorId : An error occurred while processing this request.,Microsoft.Exchange.Manag
   ement.RestApiClient.GetExoRecipientPermission

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

ID: 201c8f92-cffb-8d52-ab07-ccd1cc4f680d
Version Independent ID: abc1c16d-93b5-1ee2-6e59-422b56e62983
Content: Get-EXORecipientPermission (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/Get-EXORecipientPermission.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@bmkaiser, send email to exocmdletpreview[at]service[dot]microsoft[dot]com.
Be sure to include the log files in your email message. To generate the log files, replace  with the output folder you want, and run the following command:
Connect-ExchangeOnline -EnableErrorReporting -LogDirectoryPath <Path to store log file> -LogLevel All

Or, you can open a support case (v2.0.5 of the module is in full mainstream support).

Thank you, @chrisda. I've just tried to replicate the issue and found it's no longer occurring. If it occurs again, I'll be sure to follow your advice for log capture and reporting.


Firestore blows up on GET {source: "cache"} if document exists in cache

Issue
When the data gets in a certain state (I don't know how), when I request a document with {source: 'cache'} that doesn't exist in the cache, the library blows up with the following error. While I am using React Native, the RN Firebase folks have triaged the issue and are relatively confident it's not on their end.
Error follows:
'NSInvalidArgumentException', reason: '-[FSTDeletedDocument data]: unrecognized selector sent to instance 0x600000244230'
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000110da91e6 __exceptionPreprocess + 294
	1   libobjc.A.dylib                     0x000000010f695031 objc_exception_throw + 48
	2   CoreFoundation                      0x0000000110e2a784 -[NSObject(NSObject) doesNotRecognizeSelector:] + 132
	3   CoreFoundation                      0x0000000110d2b898 ___forwarding___ + 1432
	4   CoreFoundation                      0x0000000110d2b278 _CF_forwarding_prep_0 + 120
	5   OllieApp                            0x000000010c2783de -[FIRDocumentSnapshot dataWithServerTimestampBehavior:] + 238
	6   OllieApp                            0x000000010c2782df -[FIRDocumentSnapshot data] + 47
	7   OllieApp                            0x000000010ca82911 +[RNFirebaseFirestoreDocumentReference snapshotToDictionary:] + 305
	8   OllieApp                            0x000000010ca816b7 __62-[RNFirebaseFirestoreDocumentReference get:resolver:rejecter:]_block_invoke + 151
	9   OllieApp                            0x000000010c2d3883 __59-[FSTFirestoreClient getDocumentFromLocalCache:completion:]_block_invoke + 499
	10  OllieApp                            0x000000010c2b3504 _ZZ34-[FSTDispatchQueue dispatchAsync:]ENK3$_1clEv + 36
	11  OllieApp                            0x000000010c2b34cd _ZNSt3__128__invoke_void_return_wrapperIvE6__callIJRZ34-[FSTDispatchQueue dispatchAsync:]E3$_1EEEvDpOT_ + 45
	12  OllieApp                            0x000000010c2b3299 _ZNSt3__110__function6__funcIZ34-[FSTDispatchQueue dispatchAsync:]E3$_1NS_9allocatorIS2_EEFvvEEclEv + 41
	13  OllieApp                            0x000000010c2b0c8b _ZNKSt3__18functionIFvvEEclEv + 123
	14  OllieApp                            0x000000010c24d27c _ZN8firebase9firestore4util10AsyncQueue15ExecuteBlockingERKNSt3__18functionIFvvEEE + 476
	15  OllieApp                            0x000000010c24fc07 _ZZN8firebase9firestore4util10AsyncQueue4WrapERKNSt3__18functionIFvvEEEENK3$_0clEv + 39
	16  OllieApp                            0x000000010c24fbcd _ZNSt3__128__invoke_void_return_wrapperIvE6__callIJRZN8firebase9firestore4util10AsyncQueue4WrapERKNS_8functionIFvvEEEE3$_0EEEvDpOT_ + 45
	17  OllieApp                            0x000000010c24f989 _ZNSt3__110__function6__funcIZN8firebase9firestore4util10AsyncQueue4WrapERKNS_8functionIFvvEEEE3$_0NS_9allocatorISB_EES7_EclEv + 41
	18  OllieApp                            0x000000010c2b0c8b _ZNKSt3__18functionIFvvEEclEv + 123
	19  OllieApp                            0x000000010c25db01 _ZZN8firebase9firestore4util8internal13DispatchAsyncEPU28objcproto17OS_dispatch_queue8NSObjectONSt3__18functionIFvvEEEENK3$_0clEPv + 33
	20  OllieApp                            0x000000010c25dad8 _ZZN8firebase9firestore4util8internal13DispatchAsyncEPU28objcproto17OS_dispatch_queue8NSObjectONSt3__18functionIFvvEEEEN3$_08__invokeEPv + 24
	21  libdispatch.dylib                   0x0000000114bf87ec _dispatch_client_callout + 8
	22  libdispatch.dylib                   0x0000000114c00be5 _dispatch_queue_serial_drain + 1305
	23  libdispatch.dylib                   0x0000000114c014fa _dispatch_queue_invoke + 328
	24  libdispatch.dylib                   0x0000000114bfd344 _dispatch_queue_override_invoke + 726
	25  libdispatch.dylib                   0x0000000114c0436c _dispatch_root_queue_drain + 664
	26  libdispatch.dylib                   0x0000000114c04076 _dispatch_worker_thread3 + 132
	27  libsystem_pthread.dylib             0x000000011511d1ca _pthread_wqthread + 1387
	28  libsystem_pthread.dylib             0x000000011511cc4d start_wqthread + 13
)

The firestore debug logs are not helpful:
Committing transaction: <LevelDbTransaction Start MutationQueue: 0 changes (0 bytes):>
Committing transaction: <LevelDbTransaction NextMutationBatchAfterBatchID: 0 changes (0 bytes):>
Committing transaction: <LevelDbTransaction ReadDocument: 0 changes (0 bytes):>
-[FSTDeletedDocument data]: unrecognized selector sent to instance 0x60c00005fd40
...(The above error follows)

Steps to reproduce:

Get your data in a corrupted state. Not sure how.
Run the following code:

await firebase.firestore()
    .collection('my-collection')
    .doc('some-doc-id')
    .get({ source: 'server' });

await firebase.firestore()
    .collection('my-collection')
    .doc('some-doc-id')
    .get({ source: 'cache' })

//Native exception thrown...

Note: After nuking local data, the error goes away and the request happens just fine.
Environment
IOS
macOS Sierra
RN 54
RN FIrebase 4.3.5
Firestore 5.4.1 in
Not using Typescript.

Additional info: I can clean up the corrupted data by simply doing another request with source: 'server'. Then once I request source: "cache" again, it works fine.
So it's likely some discrepancy between the local database and the server.

Given step 1 of "Get your data in a corrupted state. Not sure how" this isn't going to be a terribly actionable bug.
Please try to narrow this down somehow. Does this happen with a certain kind of data? Duration?
Worst case, could you get a dump of the leveldb database if you find a document that has this behavior? We'll need the dump and a full name of the document (or the query you're running that's hitting it).
If you're running in the simulator on a recent Xcode you can find the database in a location matching this shell glob:
~/Library/Developer/CoreSimulator/Devices/*/data/Containers/Data/Application/*/Documents/firestore/__FIRAPP_DEFAULT/*/main

I have a dump of the database and the query. What's the best way to get that to you?

Thanks for the report! I've reproduced the issue and believe I have a fix (see referenced PR above).

Cool! Does this bug affect Android too? I'm not certain but I'm relatively sure I've experienced it on Android as well.

@scottmas It shouldn't crash, but it will probably give you a failed Task rejected with an error saying "Failed to get document from cache." rather than giving you a snapshot with .exists() false.  I'm working on porting the fix to web / Android right now.

Okay great, what error code does it raise? I'm currently handling the code firestore/unavailable which is the code I'm currently handling when the document doesn't exist in the cache (in a non-blow-up scenario)/

Yes, that is the correct code although I'm not sure if/when you'd encounter it as a string firestore/unavailable.  On Android you'd be checking .getCode() against FirebaseFirestoreException.Code.UNAVAILABLE I believe.

I'm using react-native-firebase, so they must do some translation layer like that.

Closing this as https://github.com/firebase/firebase-ios-sdk/commit/da1c5346dd9dbf502f5b3eb8d3fc39077e69aa3e should resolve this.  It'll be included in our next release.

@mikelehen I'm having a similar problem but with the add method.
If you load the page "Online" but switch to "Offline" and run this code, it won't ever throw an error:
try {
    var db = firebase.firestore();
    db.collection("emails")
    .add({
        email: emailVal
    })
    .then(function(docRef) {
        console.log("Document written with ID: ", docRef.id);
    })
    .catch(handleError);
} catch (e) {
    handleError(e);
}

@NessDan That sounds like expected behavior.  The Firestore SDK is designed to work seamlessly offline and online, so when you perform write operations offline, they are queued until the client comes back online and then they're sent.
If you want your writes to fail while offline, one option is to do them via the transaction API, since it is not designed to work offline.

@mikelehen Thanks for the heads up! I'm assuming that, if someone tried to run the .add() method and left the site before the request went through, it'd never send out correct?

@NessDan That's correct.  If this is a concern (the data is critical), you might want a UI indicator to let the user know that the data hasn't been written yet, e.g.:
collection.add(...).then(() => {
  clearUIIndicator();
});
showUIIndicator("Saving...  [make sure you are online]");

Though you might want to add a small delay (200ms or something) before you show the UI indicator so that it doesn't flash in/out when you are online.  All depends on the UX you are going for.


This should say southwest

https://github.com/Zoinkwiz/quest-helper/blob/5dcb1ff108a28a5da5174dc33e8bb17d9efea86f/src/main/java/com/questhelper/quests/legendsquest/LegendsQuest.java#L675

proof:https://oldschool.runescape.wiki/w/Vanilla_pod

@Zoinkwiz This needs to be manually closed since you merged the duplicate PR that didn't have a linked issue.


UpdateBuilder doesn't clear after calling Build()

Not even sure how this code is compiling ATM, b/c Build is implemented as:
Updates UpdateBuilder::Build() const
{
	return Updates(std::move(this->updates));
}

std::move(this->updates) should not be a const operation.

Fix in PR #267


Merge pull request #275 from Realytics/beta

Release 1.3.0

I merged this so beta and master are up to date. Also, I've disabled merge commit option for merging pull requests - now we can squash or rebase only :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


No files in folder list

When i open a folder in debian 64bit, there is no files in folder list
build: sprint 30 experimental build 0.30.0-9116 (On branch master,master 60dc6b72f)

same issue still no fix ;(

For anyone interested, i've had this issues since 1.11 and now on 1.13.
The fix for me was to go to Debug -> Switch Language -> English (US) as outlined in the post here
https://github.com/adobe/brackets/issues/9883
For the sake of helping someone in the future who runs into this issue. Simply go to Debug-->Switch Language. change from system default to the language of interest and the issue should be resolved.

I had same issue but solution provided by @i0x915 did the magic for me

@i0x915 thank you

@i0x915 works thanks!

Glad that it helped, hopefully one day they will fix this.


noip logo

noip logo

URL of the site:
https://www.noip.com/

Logo / icon info:

Background color info:

Illustrator files (if you have any):

[ ] I have carefully read and followed CONTRIBUTING document.

noip logo

Don't skip filling of the the PR template.


Singular version of TargetSpec data type

Use Case
TargetSpec accepts a single Variant[String, Target] OR an Array of the same, and is understood to best be treated as if it could refer to multiple endpoints.
Sometimes a plan (or task) parameter should accept a single target only. At present, there is no built-in data type to support that.
Describe the Solution You Would Like
Create and provide a data type defining a single Target(Spec).
Boltlib::SingleTargetSpec?
Alternatives Considered
Considered parameterizing TargetSpec, e.g. with size constraints such as TargetSpec[1]. Seems infeasible in the type system implementation.
Considered changing Bolt CLI or other interfaces to pre-convert target information, so that type definitions can just worry about Target and Array objects. Seems to put more burden on the user though.
See https://tickets.puppetlabs.com/browse/BOLT-1363.

Fundamentally I don't this level of validation with the puppet type system.

The bolt inventory is fundamentally dynamic.
Strings may resolve to a single target or a group of targets.

We can make a weak validator as in #1363 which is useful to fail fast in some situations but plan authors can not rely on a SingleTargetSpec to actually return a single target get_target.  I don't think weak validator should ship with bolt since plan authors cannot rely on it. They will still have to handle cases where the type did not correctly validate.
We've done some magic with how run_plan handles TargetSpec and Optional[TargetSpec] but I don't think we can rely on that magic for validation since it means this type would not work well in other compound types like Variant[Pattern, TargetSpec], Any, or Hash[String, TargetSpec].

Making Target objects easily and intuitively passable from the CLI could be a reasonable solution. The ideal user experience is a transparent one, where the user doesn't have to do anything special and just passes their server name, as per usual.
For now I find enough utility in rough type checking that I'll implement it in the module I need it in. Looking forward to any solution that is robust enough to be built into Bolt directly.


Polish "compute" package documentation

For example sometimes the "Returns" is missing in the documentation:
Here:

Or here:

The Pull Request for the Fit-module ist merged into the Master.


Opening in new tab

Hello, thanks for the great extension. What about opening package in new tab on middle mouse click?

Should work since v3.8.3

As I found in github-linker/chrome-extension#47 there is also ctrl+click which works for me on v3.8.3, but not middle mouse click.

What is your OS?

Ubuntu 14.04

I don't have neither Ubuntu or a three button mouse to test it. So could you please got to this page and press the button with the middle mouse and reply the result here. Thank you

Event value: 2

Mhhh it should work, because it's already implemented like this. Can you please test if the middle mouse click works for this demo http://output.jsbin.com/yowofoxisu/ ?

Yep, it works in demo

It's nearly the same code. I`ll looking for an ubuntu image and a test mouse.

It would also be nice if the github-link span element was an anchor element with an href that could be opened in a new context. Simply like this;

This could probably be achieved through a HTML/JS data-URI or simply resolve every dependency in a batch request and insert the links in the DOM when it resolves (didn't it work like this before?).

Yes, in the previous version it worked like that, because the extension had a huge list of all repo urls. unfortunately this cause a big performance issue #38. So I decided to replace that static url lists with an api service https://github.com/github-linker/live-resolver/. That's the reason why the behaviour is now a little bit different. Solving all links by default would be overkill and to heavy for the API / my heroku plan.
I will consider / think about a solution for the next major release 4.0.

With my limited knowledge of how the extension works, wouldn't it be possible to simply href all dependencies to e.g. https://githublinker.herokuapp.com/npm/foo-bar?redirect=1, with a ?redirect=1 flag which tells the server to respond with a 301 (perhaps 302)?
However, when the user clicks the link it'll prevent the default action and resolve it in-browser, keeping the same functionality as of now.

Yeah that could work, but I'm worried about the UX if a package can't be resolved by the API. Right now the linker shows a nice inline message.

It would still retain current functionality. It would be possible to resolve the URL both in-browser and through HTTP redirects.

:+1: I often want to open a dependency in a new tab, but can't... A redirect ?redirect=1 flag sounds like a good idea.

Use cmd + click to open a link in a new tab. Context menu and middle mouse click are not supported yet.

Middle-click would make this extension a lot better.

+1 to middle-click

Fixed in https://github.com/OctoLinker/browser-extension/commit/2a3302e2f965f8053ab788531be257187a742c35
(@stefanbuck, it wasn't automatically closed because the commit message format was slightly off. See https://help.github.com/articles/closing-issues-via-commit-messages/#closing-multiple-issues)


Primary key constraint forces not null (#851)

Feedback appreciated, also thanks for the fantastic library!

Fixes #851

Fixes #851

Thanks for the first-time code! Really appreciate you adding a test as well

Thanks for the first-time code! Really appreciate you adding a test as well


Run npm.info in getUnpublishedPackages in parallel

This was running serially, this should speed it up quite a bit
@lukebatchelor

Oh sweet. Nice


Fix specimen body parsing

Before, any Specimen body which contained valid YAML was parsed. That meant also something like const foo:Bar = 3; or
foo
 was parsed (because of the :)!
This PR introduces two changes to this logic:

Specimens which only need YAML (e.g. ColorSpecimen), now use a function which only does that
Specimens which contain primarily non-YAML content (e.g. CodeSpecimen) now won't attempt to parse any YAML unless they contain a frontmatter-like section separated with ---. This is similar to before but it's a) opt-in for specific specimens and b) only content before --- (if present) will be attempted to be parsed.

Fixes #95. Pinging @wereHamster who reported this.

@wereHamster I improved the splitting and added a test for the case where a line ends but doesn't start with ---\n


Stop pretending that macOS 11 is macOS 10.16

This reverts the functional portions of commit 5566ace6e027a5609b3a16ae696b4f5b25264c72 and adjusts the tests.
Fixes rdar://64137561. Equivalent to #33225.

@swift-ci please test


added 300ms curl timeout for the push_notifications. otherwise the findblock cron will run default 300s timeout if a service goes down. (notifymyandroid is down.)

Cronjobs will run +300s because the running into an curl timeout because of notifymyandroid is down. running in production at https://vrm.n3rd3d.com and https://vrm.mining-pool.ovh

Can you re-target against development? Looks like a good change to me that we should merge.

Sure


Upgrade to Monaco 0.14.x

Apparently, it was released 2 days ago: https://www.npmjs.com/package/monaco-editor/v/0.14.2

sure, no hurry :)

Ah missed this ticket, there is a pull request for the version 0.14.2

fixed with https://github.com/TypeFox/monaco-languageclient/pull/107


new Openai API keys causing 429 error

Confirmed this way due to change in how OpenAI does billing. You must now pay for credits upfront. Need to update instructions in README.

Resolved in README


Dashboard docs refactor

Refactor dashboard docs. Currently all docs are on one page. This splits the docs out into their own section.
The content is written for preview 6. It must be merged after preview 6 ships.
Changes:

Create dashboard section in TOC
Add dashboard overview
Add dashbaord configuration
Add dashboard security considerations

Internal previews
Toggle expand/collapse

📄 File
🔗 Preview link

docs/deployment/azure/local-provisioning.md
Local Azure provisioning

docs/fundamentals/dashboard/configuration.md
.NET Aspire dashboard configuration

docs/fundamentals/dashboard/explore.md
Explore the .NET Aspire dashboard

docs/fundamentals/dashboard/overview.md
docs/fundamentals/dashboard/overview

docs/fundamentals/dashboard/security-considerations.md
Security considerations for running the .NET Aspire dashboard

docs/fundamentals/setup-tooling.md
[Visual Studio](https://review.learn.microsoft.com/en-us/dotnet/aspire/fundamentals/setup-tooling?branch=pr-en-us-674)

docs/get-started/build-your-first-aspire-app.md
Build your first .NET Aspire app

docs/storage/azure-storage-components.md
Tutorial: Connect an ASP.NET Core app to .NET Aspire storage components

docs/whats-new/dotnet-docs-aspire-mod0.md
docs/whats-new/dotnet-docs-aspire-mod0

docs/whats-new/dotnet-docs-aspire-mod2.md
.NET Aspire docs: What's new for February 2024

docs/whats-new/preview-3.md
.NET Aspire preview 3

docs/whats-new/preview-4.md
.NET Aspire preview 4

@IEvangelist we should start the p6 branch. This is one of the p6 docs.

@IEvangelist I think this is good for a starting point. Can you review/improve language?


Partially implemented and tested SchedulignService

Goal

Testing

Release Notes

WHAT:
WHY:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1426  👈
#1425
#1384
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite

Merge activity

Sep 27, 2:52 PM EDT: @bidetofevil started a stack merge that includes this pull request via Graphite.


words: re-add words service

Hello @jazzdan, @nicks,
Please review the following commits I made in branch maia/words-again:
be35b3a557781829505b1c3c3ecaaafb822c1882 (2018-11-07 12:55:38 -0500)
words: re-add words service
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

I think the thing going wrong last time was this bug -- this shoulllld work, though you may have to first kubectl delete deployment [owner]-words.


Add travis CI

Implementation

Adds a travis.yml
Adds the README badge showing the status of the master branch
Run formatter in the codebase so we are able to check for it on CI (can be a separated PR if needed)

Additional things :warning:
I couldn't test against elixir 1.4 because the ex_doc dependency was erroring out at compile time so I went with the nearest version that was 1.5.

 error

We will probably have to update our version requirement to at least 1.5 in mix.exs soon since right now it does not work in 1.4.
Not forgetting that we will have to enable the repository in https://travis-ci.org for it to build and test our branches/PRs.

If I give you access to the Elixir lexer repo, could you add support for travis CI there please?

Yep, np 😄


[Debian] Add release 9.2

See https://www.debian.org/News/2017/20171007

https://app.vagrantup.com/nlamirault/boxes/debian-9/versions/9.2


enable bash completion for commands

it'd be really nice to enable bash completion like we can do with kubectl when using the commands. particularly for things like the names of functions / flows / connectors but also for other commands like kinds etc

instructions here: https://github.com/spf13/cobra/blob/master/bash_completions.md

I added a command, for generates bash and zsh completion.
I think that we just need to update the docs....

Docs updated


Add support for Google Ads API

Are you a current Fivetran customer?

Dylan Baker, Fivetran
What change(s) does this PR introduce?

This adds support for the upcoming Google Ads connector.
Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[X] No  (please provide explanation how the change is non breaking below.)
This will continue to work as normal for existing users of the package. Adds new functionality.

Is this PR in response to a previously created Issue

[ ] Yes, Issue [link issue number here]
[X] No

How did you test the PR changes?

[X] CircleCi
[ ] Other (please provide additional testing details below)

@fivetran-joemarkiewicz any reason why we are delaying the merge of this PR and the one in fivetran/dbt_google_ads? Saw you are waiting for confirmation from Google?

Hi @milo157 yes we were able to get confirmation from Google actually to inform us on how the aggregations have changed with a few of the reports with the new Google Ads API. The report landing_page_view we are using in this PR (which takes the place of the adwords API final_url_performance report) does not generate the entirety of the ad click and spend totals. It only generates those of the landing page, and none outside. Because of this, the totals will not match the UI.
Our team is currently working on an alternate route where we will most likely attempt to build up the reports from the base tables instead of leveraging the provided reports from Google so we can provide accurate metrics. I apologize for the delay, but will be sure to post back here once we have a succesful solution for you to use prior to our team merging any changes.


Template bug - action object url incorrect

Bug causes AttributeError failure when action has no target

Coverage remained the same at 92.31% when pulling 6359dee52604832f8c920f56a4b9689df08e9bb7 on ryangallen:patch-2 into a267e6aee4a49d6f26fc60c9f2f8e97d719376ed on justquick:master.


codeunit 7204 "CDS Setup Defaults" - Event Publisher OnBeforeResetPaymentTermsMapping

in procedure ResetPaymentTermsMapping
START Code on Top of procedure
OnBeforeResetPaymentTermsMapping(IntegrationTableMappingName, IsHandled);
if IsHandled then
exit;
END Code
similar to procedure ResetItemProductMapping in codeunit 5334 "CRM Setup Defaults" (no EnqueueJobQueEntry Bool)
to OnBeforeResetItemProductMapping

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Converted SQL mail into multiple files.

Converted SQL mail into multiple files.
This pull request is aimed at improving the overall quality of our SQL codebase by converting the main SQL (age-x.x.x.sql) file into multiple smaller files in the sql directory. The primary goal is to enhance code readability, maintainability, and organization for the benefit of the development team.
(cherry picked from commit f6e6af7c2d286135cfafea330aaa61839c149cce)

We currently don't create PRs for PG15 as the master reflects version 15


Error report when using apex to accelerate training

When I use apex to speed up the training, I will raise the error that the cudnn algorithm cannot be found. When apex is manually closed, it runs normally. Use apex to accelerate normal in the ultralytics version

Cudnn version may be wrong


fix: add lxml with html_clean extra to pyproject.toml

Currently, cannot start project if installed with poetry install

Thank you @ldorigo


Codespace loading error

My codespace is unable to fully load and sometimes shows an error saying the codespace has stopped, is there a way i could fix this error.
Version: 1.85.2
Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Evacuation controller, test: Tidy the imports

What this PR does
Before this PR:
After this PR:
Import aliases have more meaningful names.
Imports are better organized.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[X] PR: The PR description is expressive enough and will help future contributors
[X] Code: Write code that humans can understand and Keep it simple
[X] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

/sig code-quality


Update gnss fix types, actually send them

Sends actually fix information for GNSS. The firmware now has its own enum type for fix type, because the ROS and UBLOX fix types don't fit our needs well.

Mavlink and Airbourne PRs are pending.

This is now ready for review.


Almost all Vulkan shaders are broken on NVIDIA graphics cards (except Turing GPUs)

I make this bugreport just to make you know, about status of Vulkan drivers in Nvidia
this original bugreport to Khronos forum link
Bug confirmed on Nvidia 7XX, 9XX, 1XXX cards
Godot version:
Godot 4.0
OS/device including version:
Linux and Windows, Nvidia Only
Issue description:
the bug works on Linux and Windows, all drivers start from 2019(I did not test older)
imgur video link, how bug look like in Godot 4.0
Correct shader result this shader shadertoy link (fixed, link work now)
Bug is - just launch this shader, or any shader that uses fract(or mod) and floor
Godot 4.0 result:

Steps to reproduce:
shader code:
shader_type spatial;
render_mode blend_mix,depth_draw_opaque,cull_back,diffuse_burley,specular_schlick_ggx,unshaded;

void vertex() {

}

vec4 mainImage( in vec2 fragCoord , vec2 iResolution)
{
    vec2 st = fragCoord.xy/iResolution.xy;
    float ratio = iResolution.x/iResolution.y;
    st.x *= ratio;
    float time = 1.+max(0.,min(0.,st.x));
    vec3 color = vec3(0.0);
    float rows = 30.0;
    vec2 ipos = floor(st*rows);
    vec2 fpos = fract(st*rows);

    float pct = 1.0;
    vec2 v=mod(fpos,vec2(9.));
    pct *=v.x*v.y;

    //BUG

    if ((ipos.y > 0.0)||(ipos.x < fract(time))) {
        color = vec3(pct);
    }

    //FIX
/*
    if (ipos.y > 0.0) {
        color = vec3(pct);
    }
    if (ipos.x < fract(time)) {
        color = vec3(pct);
    }
*/
    return vec4(1.-color , 1.0);
}

void fragment() {

    vec4 c=vec4(0.);
    c=mainImage(UV*512., vec2(512.));
    ALBEDO = c.rgb;
}

This shader works fine in OpenGL on the same hardware. (in Godot3 or webbrowser)
Minimal reproduction project:
Godot 4.0 project with this shader
shader_test.zip

Does the fix solve the issue?

Does the fix solve the issue?
fix sole only this shader

If yes, then the issue is with logical operator ||?
I have found this bug when sorting lots(hundreds) my shaders, and like "lots" of them got noise or other similar sort of bugs
all "snow-like shaders" that generate noise layers(from texture or shader random) broken, and lots of other shaders broken
I can list some of bugged shaders if you want

I did not dig too much to see "source of bug"(I do not want to spend weeks on that doing free job), even if its only "or" operator bug... this is just impossible rewrite all shaders removing all or operator when even simple shaders use 10-20 if operators with or
also, this bug triggered only when all variables "local", if you make any variable as varying(even final color) or change structure of this logic - bug gone... so it can be variables or floor/mod/fract operator bug(removing those operators or rewriting them also fix bug)
so for me this bug is global and ruin lots of my own shaders, randomly, fixing this bug "by hands" I do not see realistic
I have post bugreport to Nvidia forum and Khronos forum, I do not know if they gonna fix it

fixed in 456.71 Windows driver version


Router.cr version

Hi @tbrand,
We use route.cr 0.2.2.
However, on shard.yml, I can found that version is 0.1.13.
In which version is router.cr ?
Regards,

I've pushed the v0.2.3 which is tagged and shard.yml is udpated.
Could you use it?

sure
I find it easier / more understandable if the version is the same number of the tag


Reset fields on AuthenticateContext

This is needed for cases where IAuthenticationHandler instances delegate
or modify the output of each-other.

/cc @Tratcher

@HaoK do you think this is going to have any interesting side-effects?

I think if we had any code that called multiple different of these methods already we would have found this :laughing:
Claims transformer calls Authenticated again only if the context is already authenticated, which works in the existing code.

Yeah this change seems fine, before we were not expecting these to be modified more than once, basically expectation was one of these methods is called and that's it...

:shipit:

Ok cool, does someone want to stamp a :shipit: on here then?

beat me to it :laughing:


FIX FormField::Link works when no form is currently set

Most places where $this->form is used in FormField are checked for truthy first, this part of the code fails when requesting a form schema for a readonly form (e.g. history viewer) when an UploadField (that uses $this->Link('upload')) is cast as readonly with no form.
Simple fix to check for truthy before prepending the current form's action. It's not the best fix but at least stops fatal errors.
Fixes https://github.com/silverstripe/cwp/issues/115

Seems good in that it stops fatal errors, and that's ideal. But I feel there should be a follow up issue opened as to what the effect is (if any) of having an invalid link (or is it somehow valid?) rendered out when no form is set.

I’m a little hesitant about this change, given it’ll return an obviously incorrect URL. This is usually solved by $field->setForm($form); or $fieldList->setForm($form);, is that not possible in this case?

I agree with @kinglozzer. under what circumstances would a working URL be given if there was no form?

Yeah, fair enough. You can still do TextField:create('foo')->Link() to cause this error, which I don't like, but I agree that it'd be better to fix it in a different place

My instinct at this stage would be lean towards yes. So I don't think we should throw an exception. However we should be upfront about the fact that the returned link is meaningless by returning null.

Yeah, that was the intention of this PR - although returning invalid data is worse than returning null. I'll update this to return null if no form is available. I've fixed the original issue in the implementation rather than here :)

I'll give a chance to @kinglozzer and @dhensby to disagree before merging.

Yeah great, thank you

Hmm, null is better, but I’m still not sold 😛. If someone uses $field->Link(), or {$Link} in a template, it’ll silently return nothing and they’ll have to dig through to try to find out why.
My preference would definitely be an exception. By calling Link() without a form set, you’re doing something that’s...

Sounds like a LogicException candidate to me. Given that’d currently result in a fatal error, I don’t see any BC issues by introducing it in a patch release :)

Yeah, I agree with @kinglozzer again.
At the moment this is an API change (return type is changing) and what would have been a fatal error is no longer an error at all. Even though we're becoming more permissive it's still a change in behaviour and one that I'm not sure is warranted.
In all honesty, if you've fixed the problem upstream and agree it was an implementation problem, then I don't see that this patch is warranted

Good feedback here, I'll switch it to an exception instead :)

Looking at the rest of FormField, securityTokenEnabled() and Required() both need a form set to perform their task. They both check to make sure $this->form is set and return a non-value if it isn't.
I can't imagine anyone out there would rely on catching a Call to a member function on null fatal error to determine if a FormField is orphan or not. So I would consider it a bug fix as opposed to behavior change.
If we can't get a consensus on this specific PR, it might be best to have a separate RFC and take holistic view of how orphan FormField should behave.

State of play with this is that it's throwing an exception if the form isn't set yet. Are we happy to merge that, or shall I close this and leave it as it is?

OK - I'm concerned we are moving from a non-fatal error to a fatal error in a patch release here. This is something I really don't think we should be doing...
I'd prefer to leave it as is or replace it with a warning and skip the call to $this->form->FormAction(). It seems that the updateLink hook would still be called even if there was no form assigned at the moment(?)

It’s already fatal if you have no form, this is just more structured than allowing a php error.
Either way there’s no strong desire for this change and plenty of friction against it, so not worth it :-)

Ah, ok, if it's fatal, then this is ok - sorry, just looking at the error report I saw "alert" instead of "fatal" so made an assumption!
I'm happy with throwing an exception, then... let's get this over the line

Thanks all!


GH-38936: [JS] initialize overrides for DOM and Node in IIFE

Closes: #38936

@nrabinowitz can you test this change? If we get it in my Monday, it will be in the next release.

@domoritz I think the IIFE here works because it's initializing a static property on the class, so bundlers can't throw away the side effects. I think we'd have to do something similar here.

I see. The issue with my code is that the method might still be thrown away?

@domoritz yes, I believe so. I'm not sure if there's a clean way to do this...

What impact would setting sideEffects: true have?

My understanding is that it makes tree shaking much less effective but we should check to see the impact. Or is there some way to remove the side effect for example by extending the builder and batch reader base classes with the node and dom specific methods and then exporting an extended class?

@nrabinowitz can you test this change? If we get it in my Monday, it will be in the next release.

Sorry I missed this until now. Based on the discussion here, do you still want me to test this fix as it stands or should I wait for another approach?
FWIW, my current workaround is to use a Webpack override to specify sideEffects: true. This works, but I agree with the assessment that tree-shaking will not be as effective (and may not be used at all).

Based on the discussion here, do you still want me to test this fix as it stands or should I wait for another approach?

I'm curious whether it works or not.

The internal factory functions that create builders will be creating instances of the non-extended class, but we could definitely use this strategy as a way to patch the prototypes w/ the static-property-initialized-by-an-IIFE style that defeats tree-shaking.

How could that work? If someone imports only makeBuilder or builderThroughIterable for example, they might not import Builder and so we wouldn't execute the


Introduction

Wrote introduction/introduction.md, gathering all the Introduction topics into a single file (I think separating them into individual) files became unnecessarily fragmented. Updated README.md accordingly.
Added some dummy Implementation pages to avoid 404s.

Looks good to me!

+1


I think we need a new manager

I think it is time that we, the people, take charge and fork the repository and popularize the forked edition as we can work on it as a team rather than having this repo, which appears to be dead going forward. I have created a fork of this repo and am willing to add anyone willing to work on the project, I just ask for responsibility and consideration. Anyone wanting to collaborate on this can leave a comment saying they do. Feedback on my idea is welcomed with open arms, I'm not afraid of criticism.
The forked repo
Related issues:
#4 #98 #150 #149 #146 #136 #95 #78 #71 and many more, I assume. - Shows that the microcontroller this firmware works on is outdated.
#137 - Shows it is possible to get it working on other microcontrollers
#79 - Possibly useful for development

I don't forsee any changes to this project or a fork because honestly there are not many people who understand how the microcontrollers work. These are not Atmel AVRs, or ARM7, this requires years of research before even writing a single line of code. Good Luck, I don't see you getting anywhere.

@audiohacked Thank you for your support, your optimism is appreciated! Good luck to you to in all your adventures!

Trying to do in SMI chips!


Execute connect method in onResume won't bring the connection back after disconnect execution in onPause

I'm building an app with chat functionalities using Amazon IVS Chat Client Messaging SDK and I stucked with handling lifecycle of the chat room and web socket connection.
According to the documentation Disconnect from a Chat Room:

Because the WebSocket connection will stop working after a short time when the application is in a background state, we recommend that you manually connect/disconnect when transitioning from/to a background state. To do so, match the room.connect() call in the onResume() lifecycle method, on Android Activity or Fragment, with a room.disconnect() call in the onPause() lifecycle method.

So I implemented that way, but I never get connected state back again (I need to close activity and open it again, then all works fine):
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Changed state: DISCONNECTED -> CONNECTING
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Calling onConnecting
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Requesting token
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Did receive token: ChatToken(token=CHAT_TOKEN_HERE#1, sessionExpirationTime=Fri Sep 08 13:20:24 GMT+02:00 2023, tokenExpirationTime=Fri Sep 08 13:20:24 GMT+02:00 2023)
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Connecting to web socket wss://edge.ivschat.us-east-1.amazonaws.com
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTED) Changed state: CONNECTING -> CONNECTED
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTED) Calling onConnected
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTED) Connected
// App moves to ON_PAUSE state
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (DISCONNECTED) Changed state: CONNECTED -> DISCONNECTED
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (DISCONNECTED) Calling onDisconnected
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (DISCONNECTED) Did receive socket onClosing event: disconnect reason CLIENT_DISCONNECT, socket code 1000, socket reason:
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (DISCONNECTED) Reconnect not needed: disconnect reason CLIENT_DISCONNECT
// APP moves to ON_RESUME state
🟢 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Changed state: DISCONNECTED -> CONNECTING
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Calling onConnecting
🟤 Room 58a2b001-7c9c-4c2b-935b-ef07e74cc2c3 (CONNECTING) Requesting token

I keep seeing Requesting token. For the demo app I found here, I noticed that you didn't execute disconnect method explicitly when app lifecycle changes state to Pause/Stop. Was it implemented on purpose?
Please take a look on implementation of wrapper for ChatRoom class. Did I miss something?.:
internal class ChatRemoteApiImpl @Inject constructor(
    private val webinarRemoteApi: WebinarRemoteApi,
    private val chatClientStateMapper: ChatClientStateMapper,
    private val chatMessageRemoteMapper: ChatMessageRemoteMapper,
) : ChatRemoteApi {

    private var _chatRoom: ChatRoom? = null
        set(value) {
            field = value
            if (value != null) {
                mutableChatRoomFlow.tryEmit(value)
            }
        }

    private val isConnected: Boolean
        get() {
            return chatRoom.state == ChatRoom.State.CONNECTED
        }

    private val mutableChatRoomFlow = bufferedFlow<ChatRoom>(replay = 1)

    private val chatRoom: ChatRoom
        get() {
            return checkNotNull(_chatRoom) { "Chat Room need to be initialized" }
        }

    override val stateChange: Flow<State>
        get() = mutableChatRoomFlow.take(1)
            .flatMapLatest { chatRoom ->
                chatRoom.stateChanges()
                    .map(chatClientStateMapper::map)
            }

    override val messages: Flow<ChatMessageResponse>
        get() = mutableChatRoomFlow.take(1)
            .flatMapLatest { chatRoom ->
                chatRoom.receivedMessages()
                    .map(chatMessageRemoteMapper::map)
            }

    override suspend fun createClient(params: ChatClientParams) {
        val chatTokenProvider: suspend () -> ChatToken = suspend {
            val response = webinarRemoteApi.createChatToken(params.webinarId, params.userId)
            with(response) {
                ChatToken(
                    token,
                    tokenExpirationTime.toDate(),
                    sessionExpirationTime.toDate(),
                )
            }
        }

        _chatRoom = ChatRoom(
            regionOrUrl = params.region,
            tokenProvider = chatTokenProvider,
        )
    }

    override fun connect() {       // call onResume
        if (!isConnected) {
            chatRoom.connect()
        }
    }

    override fun disconnect() {   // call onPause
        chatRoom.disconnect()
    }
}

Hi @rusmichal, sorry to hear you are having this issue. Our team will investigate this issue, but we may not be able to respond within the next few weeks.
In the meantime, if you have not already, please post this question to AWS re:Post where our developer advocates and the AWS community may also provide additional insights.

Hey @slee-aws . Thanks for your message. I'll try to find topic related to my issue in AWS re:post. if not then I'll post the new one.

I think we can close it. I moved the question here: https://repost.aws/questions/QUs4dkvutPSt2j_edsc0yQ7A/amazon-ivs-chat-client-messaging-android-execute-connect-method-in-onresume-won-t-bring-the-connection-back-after-disconnect-execution-in-onpause


README refers to older version of Android Studio

The ReadMe references "the latest version of Android Studio 3.3" whereas the latest is 3.4.1. Should "3.3" just be dropped so it says "the latest version of Android Studio"?

Thank you!


Add link attribute to JsonExample for JsonWriter

Some viewers use the link attribute for viewing specific
examples. Apitome requires
the attribute to be present for the specific example.

Closing due to age.


Added option to Makefile to build w/o Docker

Refers to the discussion in #23.
Signed-off-by: Clovis Durand<EMAIL_ADDRESS>

Please squash your commits into one commit.

Done.

PR remarks taken into account.


Improve SEO: new browser detection mode: onlyOnCookie

Is your feature request related to a problem? Please describe.
There were a number of SEO related changes in recent releases. onlyOnRoot has improved things greatly, but sometimes even more restrictive mode is needed, especially if no_prefix default language is not english. So for this reason we want to disable detectBrowserLanguage  if there is no cookie set (to make sure crawlers are not redirected.)
Describe the solution you'd like
add a new mode to detectBrowserLanguage - onlyOnCookie. This will behave in a following way:

if cookie set it will use that cookie as default language
if no cookie set it will use prefix ( or no prefix as a default) to detect language (set this as a cookie for next visit)

Hello,
onlyOnNoPrefix option probably does everything you've asked?
Thanks!

Hello,
onlyOnNoPrefix option probably does everything you've asked?
Thanks!

Hi, onlyOnNoPrefix is even less restrictive than onlyOnRoot. I am looking for opposite.
İ don't want any browser detection if there is no cookie set. Basically that will make sure that crawlers are not affected.

Hi, onlyOnNoPrefix is even less restrictive than onlyOnRoot. I am looking for opposite.
İ don't want any browser detection if there is no cookie set. Basically that will make sure that crawlers are not affected.

What you want is to basically never detect browser language but only use the cookie to store the last-used one. So that wouldn't really belong in detectBrowserLanguage object, I feel.
And in that case you can probably just implement that logic yourself by storing the cookie on page load if not set and restoring locale from cookie if set.

What you want is to basically never detect browser language but only use the cookie to store the last-used one. So that wouldn't really belong in detectBrowserLanguage object, I feel.
And in that case you can probably just implement that logic yourself by storing the cookie on page load if not set and restoring locale from cookie if set.


Accelerate the assemble of ParLinearform

Hello my friends,
I try to implement mass lumping for the time-dependent wave equation, similar to ex23.
The ode_solver is ForwardEuler, and it seems the time consumption of solving mass matrix decreases more than 10x (from 23350ms to 1818ms), which is great. Here's the run time of
Gauss-Legendre Integration
LF assemble time = 3767.8939ms,
M_solver solve time = 23350.589ms,
Other time in Mult() = 1897.5179ms,
and Gauss-Lobatto Integration (mass lumping)
LF assemble time = 3415.9194ms,
M_solver solve time = 1818.6688ms,
Other time in Mult() = 1480.581ms.
The problem is, the incident wave is a gaussian impulse, so the RHS must be assembled in every time step. The runtime of linear solver is so low for mass lumping that the LinearForm's assemble time exceeds it. Is there a way to reduce the assemble time of ParLinearform? The Mult() of TimeDependentOperator is shown bellow:
void WaveFirstOrderOperator::Mult(const Vector &ep, Vector &dep_dt) const
{
    int sc = height/2;
    Vector e(ep.GetData() +  0, sc);
    Vector p(ep.GetData() + sc, sc);
    Vector de_dt(dep_dt.GetData() +  0, sc);
    Vector dp_dt(dep_dt.GetData() + sc, sc);

    { // f^{n}
        auto t1 = std::chrono::system_clock::now();
        F1Coef->SetTime(t);
        F2Coef->SetTime(t);
        F1->Assemble();
        F1->ParallelAssemble(f1_vec);
        F2->Assemble();
        F2->ParallelAssemble(f2_vec);
        f1_vec *= c_const*c_const;
        f2_vec *= c_const;
        auto t2 = std::chrono::system_clock::now();
        t_lf += std::chrono::duration_cast<std::chrono::nanoseconds>(t2-t1).count();
    }

    auto to1 = std::chrono::system_clock::now();
    de_dt = p;
    z = 0.;
    K_mat->AddMult(e, z);
    B_mat->AddMult(p, z);
    z.Neg();
    z.Add(1.0, f1_vec);
    z.Add(1.0, f2_vec);
    auto to2 = std::chrono::system_clock::now();
    t_other += std::chrono::duration_cast<std::chrono::nanoseconds>(to2-to1).count();

    auto tm1 = std::chrono::system_clock::now();
    M_solver->Mult(z, dp_dt);
    auto tm2 = std::chrono::system_clock::now();
    t_msolver += std::chrono::duration_cast<std::chrono::nanoseconds>(tm2-tm1).count();
    ;
}

One obvious thing is to combine F1 and F2 into one assembly, avoiding looping over every element / quad point and computing things like detJ twice. Then you can think about partial assembly and other optimizations.

Thank for your reply.
I have adapted the problem to a 3-D H(curl) version now. F1 and F2 involves boundary integrator VectorFEBoundaryTangentLFIntegrator and VectorFEDomainLFIntegrator respectively. I added them to one ParLinearForm and there's no acceleration.
I guess I can benefit from a customized LinearFormIntegrator that combines both F1 and F2?

Yes, I meant one integrator (one Assemble...() function) to avoid the overhead of iterating twice.

If you are either using GPUs or using high-order elements, linear form integrators can be sped up quite a bit by implementing "fast assembly" kernels. These aren't currently implemented for the cases you need (we have VectorFEDomainLFIntegrator, but for H(div) instead of H(curl), and don't have VectorFEBoundaryTangentLFIntegrator currently).


swb plugin checker: check "Field(...) *"

The checker missed incorrect SWB annotation Field(...) * (likely should
be Field(Field(...)*)).

@leirocks Note the CI detected the annotation error now.

nice! :shipit:

@dotnet-bot test Windows x64_test please


Upgrade js-tokens

This allows stuff like console.log(`the current env is ${process.env.NODE_ENV}`). Also, it makes the tokenization more correct in general (but a little bit slower).
js-tokens@6 uses Unicode Property Escapes, which requires Node.js 10 or later.

Why not merge?


move users api endpoint from internal to identity

move users api endpoint from internal to identity
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[x] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!