Fluentd envVars to pass environment vars to pod

Is your feature request related to a problem? Please describe.
Setting environment variables to the fluentd pod.
Describe the solution you'd like
A straightforward way to pass environment variables to the fluentd pod, similar to the way we can do with fluentbit pods using envVars
Additional context
No response

https://github.com/fluent/fluent-operator/blob/master/pkg/operator/sts.go#L144-L146 looks like it should work with fluentd as well.

https://github.com/fluent/fluent-operator/blob/master/pkg/operator/sts.go#L144-L146 looks like it should work with fluentd as well.

When I asked this question it was still 2.1.0:
https://github.com/fluent/fluent-operator/blob/v2.1.0/pkg/operator/sts.go


Adding Yukino application in data.json

Adding the Yukino application for Windows, Android and macOS platforms. It currently has MAL and AniList support but does not have Kitsu/SimKL support. Please visit the app link - https://zyrouge.github.io/yukino-app/ - for more details

Yukino has already been added to the index. data.json is just a historical thing now, which we use to experiment with the new versions of the index

@Yasamato May I know what is the new approach of populating various apps and streaming sites?

Please report incorrect or missing data at our Discord server in #index-wiki, not here.

Taken from Readme.md


ImportError: cannot import name downsample

Hello,
Could you tell me what version of theano and lasagne you used?
I tried to run python train_network.py 0, and it gave me an error: ImportError: cannot import name downsample
My version theano is 1.0.1, and lasagne is 0.1.
Thank you,
Po-Yu

I upgraded my lasagne to 0.2.dev1 and it solved my problem.
pip install --upgrade https://github.com/Lasagne/Lasagne/archive/master.zip


Add 2 stars visualisation for 2 outstandings

It's always sad to see only one star on the profile page when we actually got 2 outstanding evaluations. This change fixes that by displaying 2 stars for 2 outstanding evaluations, while keeping 1 star for 1 outstanding and 3 stars for 3 outstanding.

If showing only one star for 2 outstanding is intentional, feel free to reject this pull request.

This has been something I've wanted to add for a long time, thank you for your contribution!
I'll test and release this in an update asap.


vdk: add new stage (pre_release_image) and fix trigger rules

Control Service CICD should not be triggered if top level gitlab-ci is
changed. As top level CI contains only orchestration (stages and top
level CICDs); No other project CI is triggered if top-level is changed
so it is consistent
Testing Done: this PR
Signed-off-by: Antoni Ivanov<EMAIL_ADDRESS>

Judging by the Gitlab CI/CD, it looks like the Control Service pipelines are still run. Is this expected?


[democracy] wrong interpretation of approval threshold

proposal 3 here is wrongly displayed as currently passing with 10% aye.

Proposal id: 3 (reputation commitment purpose id: 2)
🛠 action: ProposalAction::UpdateNominalIncome(ejv5h7TBKTE, 1.1000000000000000888)
▶ started at: 2024-06-22 14:01:54 UTC
🏁 ends after: 2024-06-22 14:21:54 UTC
🔄 start cindex: 3
👥 electorate: 10
🗳 turnout: 10 votes = 100.000% of electorate (turnout threshold 0.01 votes = 0.100%)
🗳 approval: 1 votes = 10.000% Aye (AQB approval threshold: 50.000%)
state: ProposalState::Ongoing

fixed in 066a1f9


Many project function links are dead

Go to the papermario project page, load a couple pages of functions, and then click one of the names. Many 404
example

I figured this out. it's rate-limiting


[Subtask] initialization Gravitino Python client

Describe the subtask
Design according to the proposed document: https://docs.google.com/document/d/1Eq9qd94967cYpSagTps34vHbHNhO4VMPs4R-6tNK8Eo/edit,
we decided initial Gravitino Python client.

Create Gravitino Python client module in the gravitino/clients/client-python directory.
Use Python REST library requests to connector Gravitino Server RESTful interface.
Implementation getVersion() Python function in the gravitino/clients/client-python/metadate directory.
Implementation getVersionTest() UnitTest in the gravitino/clients/client-python/tests directory.

We can refer to this article to create a Python library step by step. https://towardsdatascience.com/deep-dive-create-and-publish-your-first-python-library-f7f618719e14
Parent issue
#2113

hi @Lanznx
Are you interested in implementing this task?

yes, i'm willing to do it


SA not working

Describe the bug
I did the SA account JSON key on a paid account as written. It doesn't work.
To Reproduce
Steps to reproduce the behavior:
Run
gwbackupy --service-account-key-filepath sa.json gmail backup --email<EMAIL_ADDRESS>
INFO 2023-08-08 13:33:10,814 - Starting backup for<EMAIL_ADDRESS>INFO 2023-08-08 13:33:10,814 - Scanning backup storage...
INFO 2023-08-08 13:33:10,814 - Stored items: 0
INFO 2023-08-08 13:33:10,814 - Backing up labels...
INFO 2023-08-08 13:33:10,814 - Getting labels from server<EMAIL_ADDRESS>INFO 2023-08-08 13:33:10,816 - file_cache is only supported with oauth2client<4.0.0
INFO 2023-08-08 13:33:10,818 - Attempting refresh to obtain initial access_token
INFO 2023-08-08 13:33:10,820 - Refreshing access_token
INFO 2023-08-08 13:33:10,981 - Failed to retrieve access token: {
  "error": "unauthorized_client",
  "error_description": "Client is unauthorized to retrieve access tokens using this method, or client not authorized for any of the scopes requested."
}

Desktop (please complete the following information):
Ubuntu Linux CLI

I've found the following writeup about delegating domain-wide authority, but it still doesn't work.
https://developers.google.com/identity/protocols/oauth2/service-account#delegatingauthority
What is the scope I might need to add there?

Found the correct scope on the GAM wiki
https://github.com/GAM-team/got-your-back/wiki#google-workspace-admins

@hyperknot Did you generate SA access based on this guide? Service Account Setup

Yes, but the last part is missing. Steps 12-16 in the linked GYT wiki.

Scope https://mail.google.com/ in domain-wide authority is working

The documentation is incomplete and incorrect. Domain-wide authorization is required for SA operation.
In editing the SA on the cloud console. Domain-wide delegation https://mail.google.com/ scope is enough.


[BUG] - Mutli-APT DP/STARs Commands

Describe the bug
When completing a DP/STAR command (chart recall/fixes/etc...) for a procedure that serve multiple airports, it only provides the data for one of the charts.
To Reproduce
Steps to reproduce the behavior:

.KAYLNC
.KAYLNF

1-Brings up the VLL airports version of this procedure when DTWs version may be wanted.
2-Seems to draw the fixes for DTWs version or maybe it draws all versions together.
Expected behavior
We may have to put the airport code at the beginning of the syntax:

.VLLKAYLNC .DTWKAYLNC
.VLLKAYLNF .DTWKAYLNF

Fixed Version 0.7.1

Fixed Version 0.7.1


[esbuild] background-urls processed with errors while process .css files from node_modules without files

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When I use .css from node_modules (from my library) with background urls that is could not be resolved. When I use webpack, I have no errors about it (also not have warnings - I think it's not good also) but that not stop me with errors in node_modules css background urls which could not be resolved.
Minimal Reproduction
Just add some.css file to any node_module library, and import it into project.json for processing. That .css should contain
.something {
    background-image:url(assets/somefolder/my-header.jpg);
}

but we have no image in library. That's no problem for webpack. Webpack ignore it, but esbuild angular css-resource-plugin can't ignore it.
I'm trying to fix that by my own via to check css-resource-plugin.js and add some code to build.onResolve
if (result.errors.length && kind === 'url-token' && importer.includes('node_modules') && result.errors[0].text.includes('Could not resolve') ) {
   result.warnings.push(result.errors[0])

   result.errors[0] = undefined;
   return {
       warnings: result.warnings,
       path: args.path,
       external: true,
   };
}

and that solve my problem but when tailwindcss process my in app background-image urls that importer is node_modules too via node_modules/tailwindcss. That leads to all errors processed by tailwindcss also been marked as warnings. That's not a good idea.
I'm add tailwindcss explicitly. Not the best way, but works)
if (result.errors.length && kind === 'url-token' && (importer.includes('node_modules') && !importer.includes('tailwindcss')) && result.errors[0].text.includes('Could not resolve') ) {
   result.warnings.push(result.errors[0])

   result.errors[0] = undefined;
   return {
       warnings: result.warnings,
       path: args.path,
       external: true,
   };
}

That fixes problems and mark all node_modules css imports as warnings, and only process properly tailwindcss
Exception or Error
Could not resolve "assets/somefolder/my-header.jpg" [plugin angular-css-resource]

    node_modules/@org/ui-lib/some.css:1:63459:
      1 │ ...ound-image:url(assets/somefolder/my-header....
        ╵               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  You can mark the path "assets/somefolder/my-header.jpg" as external to exclude it from the bundle, which will remove this error and leave the unresolved path in the bundle.

Your Environment
Angular CLI: 17.0.0-rc.0
Node: 18.16.1
Package Manager: yarn 3.6.3
OS: darwin arm64

Angular: 17.0.0-rc.0
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, language-service, platform-browser
... platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.0-rc.0
@angular-devkit/build-angular   17.0.0-rc.0
@angular-devkit/core            17.0.0-rc.0
@angular-devkit/schematics      17.0.0-rc.0
@schematics/angular             16.2.7
ng-packagr                      16.2.3
rxjs                            7.8.1
typescript                      5.1.6
zone.js                         0.13.1

Anything else relevant?
I can prepare PR if this solution is accepted by angular team.

If somehow I can mark it as external via builder config and mute errors - it's also an option!

The new build system provides the externalDependencies option which can be used in this case to notify the build system that these files should not be processed at build time.
Example:
"externalDependencies": ["assets/*"]

@clydin it's good, but I really want to process files from assets, but somehow ignore from node_modules library

@pumano, assets are not intended to be processed by the build system. They are copied as are. Otherwise, you'll get double in the output if they are.


[ID] Visually distinguish grants that have been Unread (unreviewed) vs. Read (reviewed)

Related to issue #74 but creating a new issue to start with a clean slate since the proposed solution no longer involves creating a filter.
Taking a cue from existing patterns in email clients, we've settled on using a bolded/unbolded text styling to visually distinguish unread/read grants (respectively). We believe there are some advantages to a solution like this:

It's a visual language pattern that users should already be used to
It's low effort implementation for engineering
It reserves other potential visual cues to be used for other solutions (for example one iteration I toyed with involved using badges to mark the grants)

Below is a screenshot of my mockups for what this would look like:

Another stylistic change to note: for sake of simplifying the table a bit, the background color for the Opportunity Number column has been removed, as well as the alternating stripe background for the rows. The aim is to reduce the number of colors in the table so that we can use them a little more purposefully in the future.

The work around removing columns is tracked separately in #889.

The work around removing columns is tracked separately in #889.

Noted. Thanks Mindy. As soon as I hit publish on this I immediately changed my mind and edited my original post to take out the stuff about columns. Should be more focused now.


Do not do initial commands in pyats

resolves: #12

code-wise this lgtm. I wonder what will break if we do not send the initial commands down to the device. are we ensuring that e.g. paging and logging is turned off? but maybe I misunderstand what the goal here is.
edit: read up on #12 -- who / what ensures that for devices where the initial commands are working / do useful things those commands are actually sent?

I believe that PyATS module itself handles that.
The code here doesn't change the default behavior.  It just allows a user to override it.
I cannot say whether #12 was caused by improper SD-WAN setup/config or there's actually missing/broken support in PyATS.   I just created a PR with the proposed workaround.  We can mark it as won't do instead.


store Drafts -> Hp

functionality to store Drafts of the HpRechnungInterface (same as in KgRechnungInterface)

Finished -> rewrite store Draft

Finishes -> look rewrite store_draft


Unable to escape output for HTML attributes

I'm trying to escape output for use inside HTML attributes, where the value may include a quote, ampersand, etc. like so:
<meta property="og:title" content="<%= htmlWebpackPlugin.options.title %>" />

According to the EJS documentation (and an old issue here), the <%= '...' %> delimiter should escape the output, but it does not do that here. According to the lodash template documentation, I can escape the output by using a <%- delimiter, but if I do that in my template, I get the following error:
ERROR in Template execution failed: ReferenceError: _ is not defined
ERROR in   ReferenceError: _ is not defined

Is there a way to get this to work here? Or is this a bug that needs fixing? Thanks.

Hey the template is generated here:
https://github.com/ampedandwired/html-webpack-plugin/blob/master/lib/loader.js#L35-L50
Maybe we could extend it with var _ = require("lodash"); - would that fix your problem?

Why is this closed? I feel like if use of a documented feature results in a crash with a cryptic error message, that's a bug worth investigating.
My code does not make explicit reference to _. I'm guessing that the missing reference is a result of some code generated by this plugin or one of its dependencies. I'd offer to dig deeper, but I'm not familiar enough with the code in question.
I have managed a workaround to explicitly require lodash.escape and run my variables through that function. But it kind of defeats the purpose of a template. And it won't prevent others from running into the same problem. So I'd hope this could be solved, or at least have the documentation updated to reflect that this plugin does not support that syntax.
Thanks

It was closed because of inactivity and no responds to my question..

I'm sorry I wasn't able to get to it right away. Can we re-open now please?

Sure - could you please debug a little bit why this is happening?

Maybe you could log the result here to see what's happening behind the scenes:
https://github.com/ampedandwired/html-webpack-plugin/blob/master/lib/compiler.js#L98
and as mentioned before here (where the templating happens):
https://github.com/ampedandwired/html-webpack-plugin/blob/master/lib/loader.js#L35-L50

Closed because of inactivity


Moon not appearing on Lovelace Card

Describe the bug
Unfortunately I can't see the Moon on the Custom Lovelace Card.
After updating the Sun-Card to the latest version (also version 4.0) is simply isn't present.
Browser has been cleaned an also different browsers has been used for testing.
To Reproduce
Steps to reproduce the behavior:

Open Chrome / Firefox
Go to Home Assistant Lovelace View
Locate the Sun-Card --> No Moon

Expected behavior
The Sun-Card should show the Moon Position, but isn't.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Chrome (Version 79.0.3945.130 (Officiel version) (64-bit))
Firefox / Safari (Latest release)

Smartphone (please complete the following information):

Ariela Android App, with webview (Latest)

Additional

No errors shown in the logs comlaining about the syntax / Moon integration.

Same here :/
My card:
      - type: "custom:sun-card"
        name: Orto-Ocaso
        entities:
          elevation: sun.sun
          max_elevation: sensor.max_elevation
          moon: sensor.moon
          noon: sensor.solar_noon
          sunrise: sensor.sunrise
          sunset: sensor.sunset
        style: |
          ha-card {
          --sc-sun-night-color: #606060;
          --sc-sunbeam-color: orange
          }

In the newest version I've added some guard checks and validation of moon sensor state values. Could you check it out, please?

Hi @mishaaq , i'm with the new version but still no moon :/
(BTW is day now, i should wait till night to see it?)

(BTW is day now, i should wait till night to see it?)
No, it should be visible throughout the day. Can you check the console for entries prefixed with "Sun Card"?

Nothing in the home-assistant.log with "sun" :/
(there are other place to check?)

Thanks for testing.
I meant browser's console - https://kb.mailster.co/how-can-i-open-the-browsers-console/

Ok! :D

hassiopastor.duckdns.org-1583842739165.log

I see the exact same on my end. Running latest (Hass + Sun-Card):

Same for me. Probably because sensor.moon has a state value of waning_gibbous which is not present in sun-card.js.
moonIcon(e) {
        const t = {
            new_moon: "new",
            waxing_crescent: "waxing-crescent",
            first_quarter: "first-quarter",
            waxing_gibbous: "waxing-gibbous",
            full_moon: "full",
            last_quarter: "last-quarter",
            waning_crescent: "waning-crescent"
        }[e];
        return t || console.error(`Sun Card: Unexpected state value '${e}' for moon sensor.`),
        `mdi:moon-${t}`

I just added the missing phase and now it's working


Javadoc for TaskOutputs.files() shouldn't talk about disabling cache anymore

Since 5.0(?) we allow this method to be called with anything and still have caching enabled.

/cc @gradle/build-cache

We have lifted the requirement in 5.0 indeed; https://github.com/gradle/gradle/commit/12a918c210da61ec91d2be4a5e9baa0bccd12216


Add a Java port information in README.

This is a port of llama2.c in Java https://github.com/neoremind/llama2.java. The implementation is able to inference both the stories series baby llama2 models and Meta’s llama2 7B model.
This piece of work is done with educational purpose, and enriches llama2.c forks.
In terms of performance, llama2.java offers the similar level of token/s compared with pure C version.
I will keep updating and enhancing the forked version with the base C implementation.

@karpathy would you please take a look of the PR and see if it is ok to merge?

Sorry, yes I can merge


Openwhisk Pywren Credentials and Setup

Hi,
I'm running vanilla openwhisk locally and want to run pywren. What steps did you all use to get this to work? I saw post #62 but it did not given any details about how to do this.
For one, when I'm setting up credentials I'm not sure what to put down for region/endpoint/etc.:

When running the hello-world openwhisk, this is pretty much the only information avaiable to me:

Any help getting this setup would be enormously helpful for us. Thank you for all your work on this project.

Hi @shelbyt
In OpenWhisk , a fully qualified name has three parts: /namespaceName/packageName/entityName. So based on the image, the endpoint is http://<IP_ADDRESS>:9090 and the namespace is 23bc46b1-71.... If you can access OpenWhisk without credentials, there is no need to put anything in api_key.
In any case, could you deploy the hello_world action again and verify you can invoke it by sending a curl request? curl "http://<IP_ADDRESS>:9090/api/23bc....". Can you also verify you always get the same namespace when deploying actions? For example, can you share what is the output of the wsk list command?

In OpenWhisk , a fully qualified name has three parts: /namespaceName/packageName/entityName. So based on the image, the endpoint is http://<IP_ADDRESS>:9090 and the namespace is 23bc46b1-71.... If you can access OpenWhisk without credentials, there is no need to put anything in api_key.

Thank you, that was helpful.
I can invoke by sending curl and I verify that I always get the same namespace when deploying.
See output below:

When running the testsuite python -m pywren_ibm_cloud.tests I seem to now be getting this error on all the tests:

I assume this is because I have to change the backend from ibm_cf to either docker or localhost?

@shelbyt Thanks for the information. Seems the communication with the backend is slightly different between IBM CF and OpenWhisk.  Need to check other things to go ahead with this ;)

Can you try if you can invoke the action by using "guest" namespace?
for example: curl http://<IP_ADDRESS>:9090/api/guest/hello
Can you try if you can invoke the action by specifying api version?
for example: curl http://<IP_ADDRESS>:9090/api/v1/guest/hello
If 1 or 2 works, can you try also:
curl http://<IP_ADDRESS>:9090/api/namespaces/guest/hello
curl http://<IP_ADDRESS>:9090/api/v1/namespaces/guest/hello

Sure, to give you a full picture this is how I'm creating the API:

Unfortunately, none of thsoe commands you mentioned works:

@shelbyt many thanks, In PyWren actions are not web, so can you try creating the action without --web True?

There is an error when doing this, I can't create an API without it being web it seems:

I think there is no need to create the api, you can invoke the action directly

I can invoke the action, but it doesn't seem like I can curl without setting up the API parameters, is there a way to do this?

Can you share the output from wsk list?

See following:

Interesting, can you try with:
curl http://<IP_ADDRESS>:9090/api/guest/hello
curl http://<IP_ADDRESS>:9090/api/v1/guest/hello

curl http://<IP_ADDRESS>:9090/api/_/hello
curl http://<IP_ADDRESS>:9090/api/v1/_/hello

curl http://<IP_ADDRESS>:9090/api/guest/_/hello
curl http://<IP_ADDRESS>:9090/api/v1/guest/_/hello

None of those seem to work:

Ok, no problem, can you share the output of: wsk action invoke /guest/hello?

Ok, no problem, can you share the output of: wsk --debug action invoke /guest/hello?

nice, so now you can see the full url at the beginning
REQUEST:
[POST] https://172...

So you should be able to curl  that url with -X POST and get the result

This was the URL I wanted to see :)
Now, in pywren config file you have to write:
ibm_cf:
    endpoint: http://<IP_ADDRESS>  # https if it works with https
    namespace: guest
    api_key: should_not_work

In addition to the config, PyWren still requires a little change in the communication lib, I will submit a PR soon with the changes

On curl this is what happen:

When I'm running this using the testing infrastructure on pywren, these are the errors:

I think  there is something else SSL related that is causing an error but we're really close I think.

Does curl work with http? Also, you have to delete the port 9090 from the endpoint

It doesn't work with http:

Take a look to this: https://github.com/apache/openwhisk/blob/master/docs/rest_api.md
Seems basic auth is in ~/.wskprops file. you can also get it by wsk property get --auth.
You need to write this auth in config file too. However, I have to see how this auth looks like to determine if it is the expected one by PyWren, so can you paste the output from  wsk property get --auth?

Yep, this one is the expected one, so copy all the key 23bc46.....wP into api_key in config

Same SSL Error as before, but I"m glad we did figure out what the API key should be:

Error: (Note it defaults to port 443 now)

See all properties below:

By the way I was just able to get the hello  program working with CURL but only after I gave it both the username/password AND passed the -k option to curl i.e.

Result:

This is expected. I'm modifying pywren communication lib to allow insecure OW backends

Added fix in #245. When merged add 'insecure: True' in config file:
ibm_cf:
    ...
    insecure: True

I couldn't test it since I couldn't access to an OpenWhisk deployment, but I think the changes are enough to make it working.

Using the new commits it seems to fix the auth problem. However it was then giving me this problem:

I was able to solve it by changing this in config.py  :

Then basically all the tests run! Except for one of them:

I believe the "timeout problem" is an OpenWhisk compatibility "issue" because the vanilla configuration in OW is to have a hard limit of 5min and pywren has 10 minutes:
https://github.com/apache/openwhisk/blob/master/docs/reference.md

The timeout limit N is in the range [100ms..300000ms] and is set per action in milliseconds.

The same issue comes up when I try to increase the memory in pywren to 1024:
if I use that  same config.py to increase the memory I get this error:

Once more..https://github.com/apache/openwhisk/blob/master/docs/reference.md

The memory limit M is in the range from [128MB..512MB] and is set per action in MB.

--
And just for future people to see how awesome this is, I'm running the compute locally on OpenWhisk, but I'm using IBM COS as the storage backend:
LOCAL

IBM COS

Really great stuff!!!! Thanks a lot for the help! Let me know if you have any ideas about that last test that's failing, if not that's fine. You're awesome, really saved me at least a week+

Also for future reference, I was able to get 1024 mb RAM functions of openwhisk running with the openwhisk-devtools:

Just add this
CONFIG_whisk_memory_max:<PHONE_NUMBER>
After these lines in the openwhisk invoker and controller:
https://github.com/apache/openwhisk-devtools/blob/master/docker-compose/docker-compose.yml#L83
https://github.com/apache/openwhisk-devtools/blob/master/docker-compose/docker-compose.yml#L126

@shelbyt Great to know all is working fine :) Thank you alot too for your feedback. The test that fails is not an important one, so you can run function without problems.
I added a new openwhisk_executor() in #245 , and also a new section in configuration to easily differentiate between vanilla openwhisk clusters and IBM Cloud Functions. So you can now move your config to:
pywren:
    storage_bucket: ....
    compute_backend: openwhisk

openwhisk:
    endpoint: ...
    namespace: ...
    api_key: ...
    insecure: ...

Then, you can use pywren by instantiating a new openwhisk_executor(), for example:
import pywren_ibm_cloud as pywren

def my_function(x):
    return x + 7

if __name__ == '__main__':
    pw = pywren.openwhisk_executor()
    pw.call_async(my_function, 3)
    print(pw.get_result())

Got it, I'll check it out. Closing this issue!


Letting user know which extension modified the line recently

Issue Type: Feature Request
When I work on Typescript some extension automatically modifies my import statements, while it is a good thing since I am anyways going to do it. But since I have a repritore of extensions installed, I sometimes have no clue which extension modfied the line. Because of which I have to disable bunch of extensions to see which one is the culprit.
It would be good to give user insight into which extensions modified a line, so that they can disable some setting if they want to. Also they can uninstall or disable the extension if it is interfering with their editing.
I would love to implement this feature if accepted. 😄
VS Code version: Code 1.39.2 (6ab598523be7a800d7f3eb4d92d7ab9a66069390, 2019-10-15T15:33:40.634Z)
OS version: Linux x64 5.0.0-31-generic

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Why not nginx?

I was wondering, what' the benefit of using this over nginx?

We actually wanted to implement our rate-limiting concept as an nginx plugin written in lua at first, because we already use that combination for another purpose in our infrastructure. We eventually decided against it, because it comes with a much more complicated development environment compared to node.
In addition, we haven't used node before in production and this seemed like a great opportunity to gather some experience :)


msvc output parser cant parse compiler messge

Hello,
I ve got compiler output
c:\prog\myvcprojects\projects\log-viewer\src\main.cpp(81) : warning C4715: 'format_msgv': not all control paths return a value

and it not parsed by msvc parser
my config
"cmake.enabledOutputParsers": [
                "cmake",
                "msvc"
        ]

I think that regular expression in MSVCDiagnosticParser :: msvc_re need to be changed to
^\s*(?!\d+>)\s*([^\s>].*)\((\d+|\d+,\d+|\d+,\d+,\d+,\d+)\)\s*:\s+((?:fatal )?error|warning|info)\s+(\w{1,2}\d+)\s*:\s*(.*)$

(added \s* after ((\d+|\d+,\d+|\d+,\d+,\d+,\d+)) )

after using this i found anouther problem,
if compiler print messages in different regiter like
c:\prog\myvcprojects\projects\log-viewer\src\main.cpp(81) : warning C4715: 'format_msgv': not all control paths return a value```
vs code threat it like two different files,
if you make lines lowercase before parsing, it will not macth real path of file, so vs open new editor.

Thanks for the report and information! I'll see about fixing these in the next release.

Closed.
If this issues still persists in the current 0.11.0-beta4, feel free to reopen this one or create a new one.


feat(Utils): Split string use non capturing group

PULL REQUEST
Overview
Changes the regex to use a non capturing group instead of a capturing group for better performance and prevents possible arbitrary EOL capturing
Status

[X] Typings have been updated or don't need to be.
[X] This PR have been tested and is ready to be merged.

Semantic versioning classification

[ ] MAJOR: This PR introduces BREAKING changes (direct API change).
[ ] MINOR: This PR adds new features, improve the code and/or implies minimal API changes.
[ ] PATCH: This PR fixes a bug, if needed it also references the relevant issue or documentation.
[X] PATCH: This PR improve performance or code refactor without API changes.
[ ] PATCH: This PR only includes non-code changes (documentation, style, CI, tools...).

Can you show the results for this regex and the old one so I can compare?

Can you show the results for this regex and the old one so I can compare?

With capturing group:

With non-capturing group:

Bear in mind that sometimes they might take the same amount of time but on average, non capturing seems to perform better. I can get a benchmark as well if it's good
(no i didn't forget about this)


upgrade to<EMAIL_ADDRESS>

Issue: https://github.com/Brakebein/node-three-gltf/issues/10

Issue: https://github.com/Brakebein/node-three-gltf/issues/10

Thanks!


Update webui react version

update react version
update react-router from v3 to v5
use context to pass value

@ultmaster @liuzhe-lz
This PR will change to draft status for continuing to work for fluent ui.


Postgres auto-generated ID schema not working

I'll try to get a PR together soon, but just getting this recorded as a bug before I forget:
https://github.com/micronaut-projects/micronaut-predator/blob/233bbb94d3e8fe23fe089f1bdcbca7600d1f6659/predator-model/src/main/java/io/micronaut/data/model/query/builder/sql/SqlQueryBuilder.java#L245
The code clearly things the keyword "GENERATED" is the right way to declare an auto incrementing ID in Postgres. But that isn't true :) Instead, they recommend the data type be SERIAL or BIGSERIAL as per these docs:
https://www.postgresql.org/docs/8.1/datatype.html#DATATYPE-SERIAL
Unfortunately, the way the code is currently structured it makes it a bit hard to quickly patch this so I'll have to spent more time before I can send back a test + PR.

It is actually supported. See http://www.postgresqltutorial.com/postgresql-identity-column/
However only since Postgres 10, older versions are out of luck.

Support for sequences added in https://github.com/micronaut-projects/micronaut-data/commit/ac7560eb95c9ba8bee571259ada7bf5067827a7c


Add helpful toString() for Response Objects

7. Consider implementing `toString()` for `CacheGetResponse`
    1. Right now I’m seeing `momento.sdk.messages.CacheGetResponse@5c6000cd`

Provide toString() for our POJOs (I’m so lazy that I’m always sad seeing sth like momento.sdk.messages.CacheInfo@126945f9)

Our 1.0 response objects have toString implementation coverage.


Implementation of std::hash::Hasher trait

Exports implementations of std::hash::Hasher for crc32::Digest and
crc64::Digest. This allows any types which #[derive(Hash)] to easily
hash using one of the CRC implementations.

@mrhooray

good stuff. thanks @andreweduffy
will release new version soon


refactor: introduce decoupled platform layer and convert models to state objects

this PR focusses on shaping the model and platform concept.

models are only state objects now and the service character got killed in favor of Platform extension points ResponseFactory and ResponseConverter => decoupling => way better testability
opens up the platform requests to be quite generic => leeway to supporting large platforms like ollama, huggingface or  replicate in a generic manner
bridge namespace heavily inspired by Symfony

Only thing missing right now is fixing embedding by bringing support for parallel calls on Platform back
best to review in IDE

Sorry for being not so active, I am a little bit busy right now.
I didn't find time to check this PR in detail

@OskarStark no worries at all Oskar - i'm currently finishing up those 4 MRs and will try them in my demo app with the bundle to get them merged :+1:

Thanks, if my response format is working for tools, I could also switch entirely to our bundle/lib


feat: add option to use PAT to register runners on enterprise level

Currently it`s only possible to register runners on repository level using the PAT option.
And via github app it`s only possible to do this on repo or org level.
I would like to be able to register runners on enterprise level using GHES.
You can set in the pat permissions the option to

manage_runners:enterprise | Manage enterprise runners and runner groups
This would give us new opportunities to use the runners at scale and simplify the management.

I'm open to it, but don't have GHES to test.

How do you register runners on the enterprise level with API? I can only see API for organization registration that can take a token with manage_runners:enterprise.

Havent registered a runner yet on enterprise level. As far as Ive read this should work<EMAIL_ADDRESS>

Seems like this is the API we were looking for.


multiple fclose

\Volume_11\Number_4\Bernadon2006\Source\lut.cpp line 74
remove second fclose(fd);

I just noticed this - I guess I missed the email way back when. Fixed, finally, and thanks.


No known conditions for "." specifier in "@abc/theming" package

Describe the bug
I'm using a theming(@abc/theming) library which only has *.scss files and this library I use in Angular  and Lit  based project.
I'm using vite to build Lit based project and while building this project I'm getting "No known conditions for "." specifier in "@abc/theming" package" error.
@abc/theming package has an export entry in its package.json file.
package.json of @abc/theming -
{   "name": "@abc/theming",   "description": "Theme utility functions and mixins for Project",   "keywords": [     "style",     "theme",     "typography"   ],   "exports": {     **".": {       "sass": "./index.scss"     },**     "./styles/*": {       "style": "./styles/*.scss"     }   }, }
If I remove    ".": { "sass": "./index.scss" }, from package.json of @abc/theming then build works fine.
Reproduction
NA
Steps to reproduce
npm run build
or
vite
System Info
transforming (57<EMAIL_ADDRESS>file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:22005
    throw new Error(`Failed to resolve entry for package "${id}". ` +
          ^

Error: Failed to resolve entry for package "@abc/theming". The package may have incorrect main/module/exports specified in its package.json: No known conditions for "." specifier in "@abc/theming" package
    at packageEntryFailure (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:22005:11)
    at resolvePackageEntry (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:22000:9)
    at tryNodeResolve (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:21736:20)
    at Context.resolveId (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:21487:28)
    at Object.resolveId (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:41588:46)
    at async Object.<anonymous> (file:///C:/workspace/RUF-Next/ruf-lit-monorepo-1/ruf-lit-monorepo/node_modules/vite/dist/node/chunks/dep-3007b26d.js:61954:21)

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

If exports field exists, @abc/theming/index.scss won't be resolved because exports fields work like that.
Closing as supporting sass condition is tracked at #7809.


Unable to connect to the terminal after closing and restarting the app

As the title suggests, our app is unable to connect to the terminal after closing it by swiping it out of the recents list. Steps to reproduce:

Create a transaction
Complete it or cancel it
Close the app (by swiping it out of the recents list)
Open the app again
Create a transaction

Result: the terminal no longer connects
Workaround: force close the app and start it again.
Additional info:
Our app has a foreground service that keeps the app running all the time. This service comes with android:stopWithTask="true" in the AndroidManifest.xml, so when the app is closed manually by swiping it out of the recents list, the foreground service stops and the app closes. However, the process does not seem to be killed by Android at this point and after relaunching the app, SumUp is no longer working. The app needs to be force closed to restore connectivity.

The same issue I do have now. After upgrading to 4.0.3 but a little different:

Create a transaction
Complete it or cancel it
Create another transaction

Result: the cardreader is no longer connected
I have to kill my app and start it over again. After that reapeats Steps 1-3.
PLEASE HELP our customers wait for a solution, because the have to use new Cardreaders with serial numbers greater then 108

I have the same problem,
Testing the Sumup Solo and after payment, i close and open the app, but app not connected on card reader.


docs(tutorial): updates to tutorial

A rebased version of #147

included simulation in the tutorial
used a more meaningful predicate to illustrate sampling

Merge together with: Modelator#288
Remaining issues:

[x] Make sure that there are no fees in the execution (since fees are not modelled) : Issue #148

Blocking:

[ ] Next release of modelator
[ ] Bump modelator version

@rnbguy , mdx tests should be timing out because of the issue we discussed on the call, that atomkraft init always uses a template from dev and not from the current branch. Other than that, I believe this is ready to be merged so I'm asking for your final review

Thanks :slightly_smiling_face: but this should be fixed before it gets merged. PRs are coming :rocket:

The timed out CI jobs should be fine when #181 is merged.


fix(helm): ignore empty objects

Apparently, helm does sometimes return elements in the YAML stream that
consist only of comments – meaning they are empty.
Because they don't pass our validation code, and also because they are
useless we need to ignore them.
Also fixes the name layout, putting metadata.name first and kind
last, as it was in the original helmraiser PR

Why turn name and kind around?

It was this way before. This is merely restoring old behavior.
I'm in favor of doing kind_name, because this goes from less specific to more specific. Anyways, I'll add another PR soon to make this configurable from Jsonnet (https://github.com/grafana/tanka/issues/361)


Added floating kind to values to readme

Just thought this would be nice to see in the readme for people that didn't see this option in a Youtube video.

...youtube video?

Yea, saw it on this YouTube video by DevOps Toolbox

cooool :)


File path resolved incorrectly when using mono repo structure

Seems like certain file paths are resolved incorrectly when using the following mono repo structure:
root
\--- packages
     \--- stand-alone-pkg-1
     \--- stand-alone-pkg-2
     \--- stand-alone-pkg-3

Here's a screenshot from a simple example project:

If I open the package directly:

Same issue applies to .eslintignore:

I can provide the files if necessary.
btw. if I open the project in Atom, it will lint the files correctly.

@scholtzm can you provide a GitHub repository I could clone. That will ease tracking this down.

@dbaeumer Here's the code from screenshots: https://github.com/scholtzm/vscode-eslint-issue-310

@scholtzm thanks. You need to let ESLint know which working directories you have. The reason is that ESLint resolves the ignore file locally to the current working directory which in your case is the folder which contains the packages folder. The documentation is here: https://github.com/Microsoft/vscode-eslint/blob/master/README.md
In the example you provide and entry like this is necessary:
	"eslint.workingDirectories": [
		"./packages/foobar"
	]

@dbaeumer Thank you for your response. I did not know about that setting.
I tried placing it inside workspace config and it seems to solve the issue but only partially:

@scholtzm a little bit more tricky since the path in .eslintrc is relative. You can either make it absolute or ask ESLint to actively perform cwd operations using the following settings:
{
	"eslint.workingDirectories": [
		{
			"directory": "./packages/foobar",
			"changeProcessCWD": true
		}
	]
}

And it works 👍 Thanks


Now with even less abstraction

I was parsing the antlr model into my own (less descriptive) model which wasn't a great idea. Less pain points! There might be a better way to do Column.kt but this seemed like the lowest friction moving from the old one.

looks great


docs: add backlog

Document on what features are envisioned, planned and focused on.

Backlog is empty, so closing this for now.


Set GOPROXY

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Was there a problem where Prow couldn't just set and export GO_PROXY for the job? All this wouldn't be necessary if that was done.

Was there a problem where Prow couldn't just set and export GO_PROXY for the job? All this wouldn't be necessary if that was done.

Was there a problem where Prow couldn't just set and export GO_PROXY for the job? All this wouldn't be necessary if that was done.

We just need to set GOPROXY to direct for 2 of the csi targets. Prow preset will set the GOPROXY value to the athens service dns. For these 2 targets, we need to use direct, hence this change.

Was there a problem where Prow couldn't just set and export GO_PROXY for the job? All this wouldn't be necessary if that was done.

We just need to set GOPROXY to direct for 2 of the csi targets. Prow preset will set the GOPROXY value to the athens service dns. For these 2 targets, we need to use direct, hence this change.


Refinements to address accessibility issues

The changes are focused on improvements in the following browser / screen reader combinations:

Internet Explorer and JAWS
Firefox and NVDA
Safari and VoiceOver
iOS Safari and VoiceOver

..although we've also considered the NVDA and JAWS readers with each combination of Firefox, IE, and Chrome.
The changes are in part refinements to the markup of the component (type and placement of ARIA attributes), and in part changes to the detail and timing of ARIA live region updates ('status' updates in the terminology of the component).
Summary of refinements:

Ensure immediate announcement of hint/status text when field receives focus.
- Previously 'Select your country edit blank"
- Now 'Select your country combobox collapsed editable blank. Type in 2 or more characters for results."

On typing in a field with a 2 character minimum configuration (optional configuration)
- Previously typing  'a' .. the typing echo reads 'a', and nothing else
- Now reads 'a' .. then reads 'type in 2 or more characters for results'

On finding no search results..
- Previously, reads 'No search results dot' .. multiple times (up to 7 in some cases)
- Now, reads 'No search results' a single time, and nothing else.

On finding search results
 - Previously typing a few characters e.g. 'as' .. reads '8 results are available' multiple times.
    Selecting one of the search results then reads "8 results are available, UK, 1 of 8 is selected dot" - multiple times
 - Now, reads '8 results are available' a single time
    Selecting one of the search results reads '8 results are available, UK, 1 of 8 is highlighted' once. (period removed)

Generally, no longer reads 'comma' or 'dot' on aria announce. Have refined the previous 'comma' approach (see here for original discussion). Now using a pair of aria live regions and flip-flopping status updates between. This change addresses the problem of announcing duplicate status messages (e.g. where number of results has not changed with new user input), and also the issue of repeated duplicate announcements that are caused by React virtual DOM behaviour.  See here for background inspiration.

Form fields are now visible in form overview modes (e.g. NVDA Insert + F7 , rotor in VoiceOver)

(Related to the above) Form fields can now be selected directly using relevant form navigation key (e.g. key 'F' in NVDA)

Reduced overly verbose selection announce in JAWS on IE, by removing role=alert

Simplified aria live region 'div' elements (removed inner span element). Improves situation in JAWS on IE where aria live content would not otherwise be announced without first disabling 'virtual PC cursor' mode (which is on by default).

Introduced a general event debounce on status update (aria live region update). This delays and limits frequency of aria live updates, and has the following benefits:
 - Improves situation on Mac Safari / VoiceOver where typing echo can otherwise interrupt announcement of the aria live content.

 - Improves the rapid typing situation in IE + JAWS, where fast user input can sometimes fail to trigger announcement.

 - Improves a situation in NVDA where stale results are sometimes announced.
    e.g. when arriving at an initial result list and then filtering further by typing more characters, a situation can arise where the screen reader announces stale count information, e.g.
    '8 results are available, UK, 1 of 8 is selected' ..then ..
    'UK, 1 of 23'
    ..where the '23' relates to the size of initial result list.

Introduced additional aria-describedby hint when input is empty (inspired by haltersweb autocomplete, and recommended by DAC). The aria-describedby attribute is removed once user input is present to cut down on overly verbose screen reader announce.

Silenced the aria live region when a valid choice has been made. (Where valid choice is detected as direct input of a valid value, active selection of a matched value from the result list, or the auto-select of currently highlighted value on focus blur when autoselect is enabled)

Re-display menu on re-focus of input, whenever a matched query is present and no valid choice has yet been made. Aligns behaviour with the 'No search results' scenario, and also ensures navigation possibilities are aligned with the resulting aria live announcement.

Introduced aria-label attribute on each list item element within the matched results (iOS only). The label includes the index of each item (e.g. 'France 2 of 7') to ensure this info is announced when exploring by touch/swipe. (By default iOS only announces 'list start' and 'list end' on first & last elements respectively.)

Thanks so much for this Mark! The team is focussed on releasing the next version of GOV.UK Frontend right now, but we'll be picking up this pull request for review after that - in about 2 weeks time.
Thanks again.

No prob, it's been an interesting piece of work for us. Will check back for tech / functional feedback. I can make further refinements where required, or you can always fork my fork - all good :)


New Contributors

Hey all,
So for a little bit of context I created this neotest extension off the back of @rcarriga great plugin as I was keen to use neotest with go. Unfortunately I don't currently work very much with go ATM (which could change) but in the meantime it would be great if anyone who is actively using this plugin and wants to help improve it could step forward and @rcarriga can hopefully add them to project so they can merge changes etc. and aren't dependent on me. I'm not completely gone but definitely not worth poking me or waiting on me as I'm also not actively monitoring this repo.
@sergii4 I've noticed you've been active on some issues. Would this interest you?

Thanks for flagging @akinsho, and thanks @sergii4 for putting yourself forward! I've added you as a collaborator to neoteset-go :smile:
I'm not very familiar with Go so probably won't be much help on Go specifics, but feel free to ping me as well if you need help with Neotest internals.

Hello guys!
This would be my first time contributing to a project. I would like to open a PR fixing an issue when executing tests that contain complex build tags but it seems that I don't have permissions to push a new branch to the repository.
Do I need to be a contributor for that?
Thanks in advance!
cc: @akinsho

Hi @Micloroc, you can open PR right now forking the repo and opening a PR from there
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

Great, I've already opened it #46, thank you!

I've actually had a hard time fixing some of the issues I've had with neotest-go due to how the tests are managed. I've taken a stab at learning the neotest interface a bit better and taken a different approach which actually mitigates a lot of issues, which has resulted in another neotest adapter for go: https://github.com/fredrikaverpil/neotest-golang
I'm using this adapter of mine daily, at work, dogfooding myself and I've also found a few upstream issues. Feel free to take any inspiration from this project, or join me in trying to ironing out the final kinks.


Add mock plist to all

Updates in scripts/prereq_core.sh will not add a mock plist.
Regular and legacy Quickstarts share the same plist under a project directory.


Adding google auth functionality

Sigin functionality is not working
this error can fix by adding google auth functionality and  email authorization
assign this issue to me(Gssoc' 23 contributor)

@Prashant-2505 I want to contribute.Please assign me

Thanks @Prashant-2505 for raising the issue..you can work on this!

Thanks for assigning . I start working on it from today

Thanks @Prashant-2505 for raising the issue..you can work on this!

i done chnaging and i also genrate pull request can u please check it out or merge if you find i do good work or resolve the issue

In last pull request there is some issue was occured during commit so i again genrate pull request so please check it out and if you find i do good work or resolve the issue  then merge otherwise its not then give me some suggestion to improve it


Fix: Revised Plunker Template for new AF2 4.0

The current Plunker Template is now broken because of AF2 4.0's breaking changes. Here are steps to fix.
My updated plunker is http://plnkr.co/edit/1lW4B4?p=preview
Required changes to work without breaking
In app/app.component.ts
change import { AngularFire } from 'angularfire2';
to  import { AngularFireDatabase } from 'angularfire2/database';
change constructor(af: AngularFire) {
to constructor(db: AngularFireDatabase) {
change af.database.list(`test`, {
to db.list(`test`, {
change af.database.object(`test/i0`).set(random);
to db.object(`test/i0`).set(random);
In app/app.module.ts
add import { AngularFireDatabaseModule } from 'angularfire2/database';
add BrowserModule AngularFireDatabaseModule. Revised NgModule looks like this:
@NgModule({
  imports:      [
    BrowserModule,
    AngularFireModule.initializeApp({
      apiKey: "AIzaSyDFbuKX0UeXje-PRAvwIymYo2jk-uGqMa4",
      authDomain: "test-bc800.firebaseapp.com",
      databaseURL: "https://test-bc800.firebaseio.com",
      storageBucket: ""
    }),
    AngularFireDatabaseModule
  ],
  declarations: [ AppComponent ],
  bootstrap:    [ AppComponent ]
})

Optional updates
In systemjs.config.js
upgrade typescript version to 2.2.2 (or newer<EMAIL_ADDRESS>upgrade firebase version to 3.9.0<EMAIL_ADDRESS>

Hey @WebAddict I'm pretty new to Plunker, but when I open yours, it just says Loading... on the right side. Shouldn't it show the
Angularfire2 example app
?

I just checked and it works for me at http://plnkr.co/edit/1lW4B4?p=preview
Check your browser's console log if it doesn't work, let me know what message you see

Haha! It works now. I've experienced this before where it'll just not work
and then work. Why is plunker the best we have?!? :)
On Tue, May 9, 2017 at 5:30 PM Rich Johnson<EMAIL_ADDRESS>wrote:

I just checked and it works for me at
http://plnkr.co/edit/1lW4B4?p=preview
Check your browser's console log if it doesn't work, let me know what
message you see
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/angularfire2/issues/947#issuecomment-300307477,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJTs0oUIAHQivExJcLB7dG8TQmWDtZvgks5r4Nr0gaJpZM4NSSfD
.

I just keep getting '57VM1726 database.js:159 [Violation] Avoid using document.write().' in console.

Switching to Stackblitz as it's a much better tool for Angular apps.


Improve Fraction Class

[ ] implement method isProper.
[ ] Implement method  isImproper.
[ ] implement method isEquivalent.
[ ] implement method add.
[ ] implement method multiply.
[ ] implement method divide.


Fix issue 27 - Screen elements jump up when activating tagging

This is a fix to Issue 27

This fix doesn't appear to work properly on some browser. A better fix is forthcoming.


Allow stateful and stateless header formatters to both be configured

Title: Allow stateful and stateless header formatters to both be configured
Description:
Currently the header formatter extension point allows the definition of using a stateful header key formatter, or a stateless formatter, but not both at the same time. An example of a stateful formatter is the preserve case formatter, for the stateless there is only the ProperCase formatter currently.
During the review of #15619 it was mentioned that it would be useful to allow stacked formatters, or at least the Proper Case formatter in addition to a stateful formatter. The use case mentioned during that PR was for Envoy generated headers.
Another use case is if the downstream connection is HTTP/2, but upstream clusters are doing HTTP/1.1. In this use case the stateful formatter will not be created and will not be passed to the upstream request. In this case, it could be useful to allow the Stateless formatter.
The requested feature is to allow exactly one stateful formatter, but additionally allow configuration of the Proper Case formatter. It does not cover stacked stateful formatters.
Originally I was thinking this could be limited to the case where there is no stateful formatter to use the stateless formatter attached to the connection. However, now I'm wondering if it would be clearer if the change would be to have a configured stateless formatter only act on unmodified headers in the event that a stateful formatter did not modify the header. Open to suggestions.

I am not sure. But can we just unified stateful and stateless header formatters and then created a composed formatter (just like filters chain)? :thinking:

@wbpcode Unfortunately, if the downstream connection is HTTP/2 there will not even be a stateful formatter that gets created, so for that to work we would still need to make it so the stateful formatter would be created when going upstream. I suppose that could be one approach. 🤔

Not sure when I'll have the bandwidth to work as we have a work around we're using at the moment, and will be on vacation next week, so still open to another grabbing this. Can try to find some time once I'm back otherwise.


Disable polling in Playgroud by default

The "polling" feature of new Playground makes it really slow on a local machine, so I suggest that we disable it by default and users will re-enable it manually if they want to use it.

@korycins I think there is no need, similarly as we don't say that you can switch between dark/light modes in Playground ;)


No DbContext named 'SomeDbContext' was found.

My scenario: I created a separate asp.net class library for my DAL using EF7, and the code for my dbcontext was very simple like this.
public class MediaDbContext : DbContext
    {
        public string connectionString { get; set; }
        public DbSet<AlbumCategory> AlbumCategories { get; set; }

        public MediaDbContext(DbContextOptions<MediaDbContext> options) : base(options)
        {
            connectionString = options.FindExtension<SqlServerOptionsExtension>().ConnectionString;
        }
        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            optionsBuilder.UseSqlServer(connectionString);
        }

    }
}

I had a separate asp.net 5 web application with a call to use EF service in the Startup.cs file like this.
services.AddEntityFramework()
                .AddSqlServer()
                .AddDbContext<MediaDbContext>(options =>
                {
                    options.UseSqlServer(configuration["Data:DefaultConnection:ConnectionString"]);
                });

The ConnectionString was defined in the appsettings.json file. I wanted to add a migration to my dbcontext using this command for the class library project above
dnx ef migrations add Initial -c MediaDbContext

And I got this error message

So, did I miss to configure something, or did I do wrong?

#4084 should improve this experience

As @bricelam mentioned, we've made some improvements to help get you on the right path in RC2. In the meantime, invoke the command on your startup project and specify your data access project with the -targetProject parameter.

I used -targetProject as you mentioned, and got these message.

Is it only available in RC2 version, and I need to upgrade EF?


feat: extend bluetooth module

Resolves #688
Goal is to show selected bluetooth adapter and its connections. Still a work in progress.
TODO:

[x] adapter information

[x] use config field with name to select an adapter or select one randomly if config field not defined (adapter-alias)
[x] add format replacements for label and tooltip
[x] add new styling classes

[x] connected devices

[x] add format replacement for label for 1 connected device

[x] select either the last connected device for this or from a predefined config option to rank which connected device to display (format-device-preference)

[x] make possible to show information of all connected devices in the tooltip
[x] experimental bluez battery percentage

[x] add format replacement for connected device battery percentage (if available)
[x] show as experimental feature in waybar-bluetooth man page

[x] update the module only when signaled from the bluez D-Bus bus or when the rfkill state has changed (if enabled).
[x] update doc when all features finished
[x] fix style so it matches the project style

I also noticed that there is no entry for the bluetooth module in the wiki. So that should also maybe be added.
Possible future improvements not part of this merge request:

show icon field of the connected bluetooth device when available using a config defined icon theme/size
have format-icons for the connected device battery percentage
change the selected connected device that appears in the main label changeable via scrolling or clicking?? Don't know how that would interact with the current format-device-preferences or show the latest connection model.

Oh cool! I was just about to do this! :)

I think its ready to be reviewed.
This is my first time using meson, so hopefully I didn't make any beginner mistakes there.
Also for the battery percentage feature, I just put it in a seperate experimental section in the waybar-bluetooth man page, since wrapping the code in an experimental build flag would make the code more complicated in my opinion. The code should work in all cases (whether bleuz experimental features are turned on or off and whether a device provides its battery percentage or not). But if it should be wrapped with an experimental build flag, let me know.

Forgot to hide the module when its empty from the linked feature request, so I quickly added it.

LGTM, thx!


Unhandled Exception: MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/package_info)

@Lukaalishi
can you provide your flutter doctor -v, your flutter run  -v and a minimal reproducible code sample

@Lukaalishi
can you provide your flutter doctor -v, your flutter run  -v and a minimal reproducible code sample

Hi @Lukaalishi
I see there's an open issue (https://github.com/flutter/flutter/issues/72970) addressing the case you described.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you

Hi @Lukaalishi
I see there's an open issue (https://github.com/flutter/flutter/issues/72970) addressing the case you described.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Implement drag and drop objects

PR implementing requested object-name drag and drop functionality requested in #1135 & #10.
Dragging resource groups does nothing (as they are just logical containers and mean nothing to the underlying SQL engine).

Thank you for your contribution!
Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You have made the needed changes to the docs (N/A)
[x] You have written a description of what is the purpose of this pull request above

I meant to mention in the PR, but forgot - The new feature will not work until version 1.78 is released (or using the insider's build).
That being said, I did test on 1.77 and there are no breaking changing due to this feature. The functionality will simply be latent until VSCode's underlying bugs are fixed in 1.78.

Confirmed today that this enhancement works with the updated production 1.78 release of VSCode.

Can we merge this one?

Hello @mtxr , can this be reviewed and merged if ok?


MICROS and MICROS_MICROS

I can't find an explanation between the two in your explanations.
if I understood the code correctly, by default "MICROS"
write "Ticker timer (printCounter, 1000, 0);"
& "Ticker timer (printCounter, 1000, 0, MICROS);"
it's identical, and 1000 is x1000 internally, so 1 second
Ticker timer (printCounter, 1000, 0, MICROS_MICROS);
it is 1000 microsecond (1ms) in this case, correct?
(with 4µs resolution for ATMega328 @ 16Mhs)
therefore MICROS, the duration is multiplied by 1000 internally,
if we are not careful, and logically we say to ourselves, since the resolution is microsecond, we make the mistake of thinking that 1000 is 1 millisecond
the most logical would have been to be in MILLIS by default (exactly like delay), and not to ask any questions.
and specify only MICROS for microsecond times but not multiply by 1000 internally.
your chosen solution does not make sense to me, but I am not criticizing.
it should just be explained correctly in the documentation
thank you again for your work

If you want to discuss things, because you don't read the docs, use the discussion forum.
The explanation you can find under 'New in v2.0' says micros() is used when the interval time is under 70min.

when I look for the syntax of a function, I expect to have the options in the same place, and not to read the set of revisions which cancel each other to clear up the use.
so yes I'm boring, but at least I accept constructive criticism

Sorry to steal this closed issue but it relates :)
Great library, super helpfull :)
But

i dont understand the difference between MICROS and MICROS_MICROS in example says 1000us is 1 s .... 1000us is 1ms => but i guess only default internal unit is us timer is still in ms? => if so then in examples it should be written 1000 ms is 1s :)

So if i get this correctly MICROS => internally uses microseconds but timer is in miliseconds and MICROS_MICROS => internally same microseconds and also timer uses microseconds?
BR
Ales


When do elite troops spawn at villages

When do elite troops spawn at villages

Power of 300+ of indidual notables required to start spawning


Unicode 16.0

Validation tests pass.
This test needs to be updated (we need to find a character that is not assigned to a script, 771 is now Hebrew):
    # unknown script for character
    c = chr(771)
    e: CurableSequence = ens_process(f'bitcoin.bitcin.bi̇tcin.bitсin{c}').error
    assert e.general_info == 'Contains visually confusing characters from multiple scripts (Latin plus other scripts)'
    assert (
        e.sequence_info
        == 'This character is disallowed because it is visually confusing with another character from the Latin script'
    )

There is no pyunormalize for unicode 16.0.
UnicodeWarning: Unicode version mismatch: pyunormalize is using 15.1.0, but the ENS Normalization spec is for 16.0.0 (2024-09-10T20:47:54.200Z)

@Carbon225 There is pyunormalize for unicode 16.0

I wrote this code:
for cp in tqdm(range(0, 0x10FFFF)):
    c = chr(cp)
    e = ens_process(f'bitcoin.bitcin.bi̇tcin.bitсin{c}').error
    try:
        assert e.general_info == 'Contains visually confusing characters from multiple scripts (Latin plus other scripts)'
        assert (
            e.sequence_info
            == 'This character is disallowed because it is visually confusing with another character from the Latin script'
        )
        print(f'{cp} - OK')
        break
    except AssertionError:
        pass

It should find a new example for this test.
But it did not find any characters that cause this message.
I will make this test xfail, most likely all codepoints are now included in ens-normalize-js.

I changed this in the readme:

Based on JavaScript implementation version 1.9.0

Original implementation based on JavaScript implementation version 1.9.0

I did not want people thinking we are using some old reference implementation.


kubernetes service proxy bypass

Is your feature request related to a problem? Please describe.
Currently ingress controller defines a single upsteam for a service, in form of http://service.namespace.svc.cluster.local.
The requests therefore are being proxied by the Kubernetes and distributed between the pods, which introduces an extra unnecessary network proxy. Also load balancing and health check options are largely meaningless.
Describe the solution you'd like
Monitor k8s Endpoints resource for a service and update upstreams accordingly with each individual endpoint.

Interestingly, we do need a way to disable this when implemented.  In a service mesh deployment, we should actually use the kubernetes service so the mesh can do its thing.

Is there a standard way to determine if you're running inside a service mesh?

I'm not aware of one.  Nginx implements this as a per-ingress annotation:  https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#service-upstream


Add clear, focus and unfocus methods and few more style properties to ConfirmationInput widget

In this PR, I added a few properties and methods for the ConfirmationInput widget
Style Properties:

height
width
padding
borderRadius
borderWidth
Added custom type for field

NOTE: 4th and 5th properties will work only if you've fieldType as custom
Methods:

clear
focus
unfocus

Small changes and you should be good to go


swift-async Testcontainers cloud

OK, finally, a swift-async solution that is reliable on GitHub Actions with Testcontainers cloud.
Some notes:

Introduced what I'm calling ScalableURLSession that is meant to be a drop in replacement for Apple's URLSession, but is able to handle 10k connections reliably on GitHub Actions.
Removed code related to URLSession stability from EasyRacer.swift, making the racing logic easier to follow.

Wow Jack. I'm sure that was quite the journey!  Nice work getting it working :)


Fake XP drop on zero hitsplash

Close #10658
When enabled, hitting a 'zero hitsplash' will drop a fake XP drop with value of 2 XP. Enabling this disables fake XP drops on tick value set by user.
Passing value < 2 to XPDROP_DISABLED script second argument doesn't allow the script to fire, hence the value 2.

As mentioned on Discord and in my comment on the linked issue thread this is not a solution we're interested in adding for this problem.

I will be working on moving this to a hub plugin


Form validation is broken

HTML Forms' validation is based on the form's post meta. When a form is saved in the WP backend, the required & e-mail fields are serialized & saved to _hf_settings. The required fields are extracted with JS from the #hf-form-editor field, which is populated by the post content.
A possible solution would be to ask the HTML forms plugin creator to add a filter to functions.php:100 return $form;, so we can inject our form code (everything inside the  tag). A small tweak to our sage-html-forms/resources/views/components/html-forms.blade.php is needed, so it is similar the original output:
<form {!! $attributes->merge($form->attributes) !!}>
  <input type="hidden" name="_hf_form_id" value="{{ $form->ID }}" />

    <input type="text" name="_hf_h{{ $form->ID }}" value="" />
    {!! $hidden !!}

    {!! $slot !!}
    Please enable JavaScript for this form to work.

</form>

I don't have a solution for this inside of the package but take a look at https://gist.github.com/Log1x/35f3f1f20e430d6a36a0c2e8e8f13ffc if you want to manually add form validation. It also includes an example for implementing Google reCAPTCHA.


switch to matchmedia module for client/server abstraction

I was able to keep this PR much smaller than anticipated due to the matchmedia modules abstraction for client/server.  It just takes an optional 2nd parameter, values, that it uses if window.matchMedia can't be accessed.

lgtm just need more info on the new prop then :+1:

I think I got everything.  Diff is large because I rebuilt the samples to add the static sample.

I removed the rebuilding of dists

Updated

Thanks! Owe you a :beers:


Error in delta_time of containers

There is an error in the output with some of container's times.
I opened the output in QGIS and I saw this error in its attribute table(on delta_time attribute).
The problem is that the input service time is 1.6 and some of the output times are less than 1.6 and they have to be more than service time plus the distance between the container and the last point of sequence.

The query used is the same as C02 test case:
--Empty Tables
delete from containers;
delete from vehicles;
INSERT INTO containers(id,x,y,demand) (
SELECT
cg.id,
ST_X(ST_TRANSFORM(cg.geom,4326)),
ST_Y(ST_TRANSFORM(cg.geom,4326)),
cast(cg.capacidad as float)
FROM contenedores_geom cg, sig_municipios s
WHERE s.municipio='C' AND cg.cod_recorr='C_DU_RM_CL_02' AND ST_Intersects(s.geom,cg.geom) AND cg.turno_hora='DOMINGOS, MARTES Y JUEVES CON FERIADOS LABORABLES: Nocturno (22 a 06 hrs.)'
-- Insert demand, open and close time. Containers without demand(equals to zero) are set with a demand of 2400.
update containers set open=360, close=840, service=1.6, demand=demand0.150.45;
update containers set demand=24000.150.45 where demand=0;
-- Insert Vehicles
INSERT INTO vehicles VALUES (1,1000001,1000002,1000001,20.45,10000,360,840);
-- Execute routing function from osrmint with an appropiate name and description to obtain the sequence using VRPTOOLS,
-- Save the output(route id) in order to use it later.
SELECT new_route('vrptools_C02_SunTueThuMorn2206_s32', 'mC_DU_RM_CL_02-p150-c50-v3-vt20.45-vc10000')
-- Execute the following function with the same name as the last function in order to obtain the each of the vehicle's route line
SELECT update_routes('vrptools_C02_SunTueThuMorn2206_s32');

@cvvergara I'm looking at this issue and it is not clear what is happening here. Here is my analysis so far:
delta_time is coming from  vehicle_path_t *Solution::getSolutionForPg( UINT &count ) const
http://imaptools.com:8080/docs/vrp-doc/trash-prob_2baseTrash_2solution_8cpp_source.html#l00151
line: 184 is fleet[i][j].deltaTime();
http://imaptools.com:8080/docs/vrp-doc/classTweval.html#a72acbab56fc006aa312d8291cd525066
http://imaptools.com:8080/docs/vrp-doc/tweval_8h_source.html#l00055
which is returning deltaTime_ that gets updated when when void Tweval::evaluate(const Tweval &pred, double cargoLimit) is called
http://imaptools.com:8080/docs/vrp-doc/tweval_8cpp_source.html#l00051
In the results we are seeing lots of values that are less than the service time of 1.6 for the container.
Q: when the code got refactored to deal with trips, are we no longer calling void Tweval::evaluate(const Tweval &pred, double cargoLimit) to maintain the values on nodes? or should I be retrieving these values another way?

[Reply aid=683379 to=20568786-740291401] Stephen Woodbridge
Try:
http://imaptools.com:8080/docs/vrp-doc/trash-prob_2baseTrash_2solution_8cpp_source.html#l00184
results[seq].deltatime = fleet[i][j].deltaTime();
to
fleet[i][j].departureTime() -  fleet[i][j-1].departureTime();
about the Q:
getCostOSRM
http://imaptools.com:8080/docs/vrp-doc/costvehicle_8h_source.html#l00137
calls evaluateOsrm
http://imaptools.com:8080/docs/vrp-doc/basevehicle_8cpp_source.html#l00496
which calls
path.evaluate(0, getmaxcapacity());
and evaluates the whole path.

Probably to make sure its the latest evaluation, call
getCostOSRM
before getting the results

In case were useful, I think we also saw this problem in previous versions of vrptools. Many thanks.

getCostOsrm() is already getting called before we call getSolutionForPg(), but just to be sure I added an explicit call just prior to calling getSolutionForPg() and I'm getting the same type of results.
I'll try changing:
results[seq].deltatime = fleet[i][j].deltaTime();
to
results[seq].deltatime = fleet[i][j].departureTime() - fleet[i][j-1].departureTime();
And see if that helps, if it does not then we need to look at why departureTime().

ok, for the record, I can reproduce the same type of bad results, and when I run the same problem from the command line we are getting the correct results. We are looking into why this is happening.

@etorrano
Debido a la complejdad de la informción que tienen, yo no me he dedicado a decifar lo que dicen o hacen sus queries para preparar los datos. Asi mismo, seguramente no estoy usando el mismo OSRM ni que ustedes, pero para asegurar que un issue es corregido debidamente, los datos no entrada al menos no deben cambiar. Lo mas probable es que los datos de salida cambien debido a que usamos osrm diferente.
Para evitar que los datos de entrada sean diferentes, les pido por favor crear los archivos:
issue-27.containers.txt
issue-27.dmatrix-time.txt
issue-27.otherlocs.txt
issue-27.vehicles.txt
siguiendo la estructura de los archivos en:
https://github.com/woodbri/vehicle-routing-problems/tree/montevideo-release-v3/src/trash-prob/trashWithMovingDumps/tests
Una vez que los tenga, yo localmente creo una base de datos con esa información solamente, de esa forma me puedo enfocar en la ejecución de ese problema.
Una vez resuelto, ustedes pueden hacer lo mismo, y enfocarse a los datos que enviaron, y comparar  y comentar.
La otra cosa, los screen-shots de resultados me son inútiles. Pueden mandar el shape file, agregarle una columna para comentarios y poner comentarios en unos renglones representativos del problema, o un archivo de excel con comentarios pertinenetes en los renglones pertinentes.
Eso sería de mayor utilidad para mi.
Todo eso se lo pueden mandar por correo a Steve, y el se encarga de transmitirme su información con la que yo puedo trabajar.
@woodbri here is a general translation of what I just transmitted in Spanish
I am sure we are not using the same OSRM version and/or data. So, I will need the following from your part, probably the data you are working with might have changed, so, I want to test at least for a same input. (osrm differences will throw different output)
So for that I would like you to prepare the following files:
issue-27.containers.txt
issue-27.dmatrix-time.txt
issue-27.otherlocs.txt
issue-27.vehicles.txt
from the files I will recreate a small database  and I will have fixed values.
please follow the structure of the files found here:
https://github.com/woodbri/vehicle-routing-problems/tree/montevideo-release-v3/src/trash-prob/trashWithMovingDumps/tests
and instead of giving the screen-shot of the results please create a shapefile that I can use.

Something that interested me when I first saw the error is that only happens when the service time for the container is a real number. I remember that if the service time is an integer everything works fine.

This issue is fixed in branch montevideo-release-v4, closing.


KeyError: mylist

I'm submitting a ...

[X] bug report
[ ] feature request
[ ] support request

General information
Can't navigate through the Netflix navigation. For example when I go to "Recently added" a popup appears with the an addon error message KeyError: mylist
Addon version used
0.14.0~beta22
Description
[Can't navigate through the Netflix navigation. For example when I go to "Recently added" a popup appears with the an addon error message KeyError: mylist
This only happens in my profile. If i switch to my wifes profile, I can browse the Netflix navigation and watch movies.]
Steps to Reproduce

Launch Netflix addon
Select my profile (we have four profiles)
Select any of the sub navigation items, for example "Recently added"

Expected behavior: Content of the sub navigation should be displayed.
Actual behavior: Popup appears with Addon Error. KeyError: mylist. Content of the sub navigation is not displayed. --> Can't select a movie to watch...
Context (Environment)
Raspberry Pi 3
LibreElec 9.01
Kodi 18.1
Installation

[X] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[ ] I'm using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[X] Linux (ARM)
[ ] Android

Additional informatin on the environment
LibreElec 9.01
Kodi 18.1
Debug log
https://pastebin.com/2TLwGzcP

It seems like this error occurs when "my list" is empty...
I logged in to Netflix in my web browser, added a movie to "my list", switched to Kodi and now navigating through the Netflix addon navigation works...

Fixed in the latest DailyBuild


[5.9] [Macros] Fix lazy expansion of freestanding macros within types/extensions

Explanation: An incorrectly-specified starting context for name lookup meant that macro lookup itself was failing, and therefore did not trigger expansion of the freestanding macro or introduction of the macro-expanded declarations into the context.
Scope: Narrow; affects code using freestanding declaration macros within types or extensions.
Risk: Low, no effect in code that doesn't use macros.
Issue: rdar://108824784
Original pull request: https://github.com/apple/swift/pull/65670

@swift-ci please test


structure repo as a software bundle and plugin to be added to an openmct deployment

Model after
https://github.com/hudsonfoo/kerbal-openmct
but accounting for updates to the mct plugin and telemetry structure

upon further inspection it appears that a software bundle is associated with the legacy plugin installation process.
Instead provide plugin installation as show in github.com/rosmod/openmct-tutorial


execution spec bails out on the first failure

Currently execution_spec runs each test one by one and fails on the first error. The expectation is to run the tests in parallel and show all the failures at once.

/bounty 100$

@tusharmath I would like to attempt it.

This issue is better articulated here https://github.com/tailcallhq/tailcall/issues/1500


xSQLServerSetup: PowerShell DSC resource MSFT_xSQLServerSetup failed to execute Set-TargetResource functionality

Details of the scenario you tried and the problem that is occurring:
Using <IP_ADDRESS>, same on SQL server 2012, 2016
VM has reported a failure when processing extension 'Microsoft.Powershell.DSC'. Error message: \"DSC Configuration 'Main' completed with error(s). Following are the first few: PowerShell DSC resource MSFT_xSQLServerSetup failed to execute Set-TargetResource functionality with error message: System.Exception: Test-TargetResource returned false after calling Set-TargetResource. The SendConfigurationApply function did not succeed.\"
The DSC configuration that is using the resource (as detailed as possible):
xSQLServerSetup 'InstallDefaultInstance'
{
InstanceName = 'elasta'
#Features = 'SQLENGINE'
SQLCollation = 'SQL_Latin1_General_CP1_CI_AS'
#[PSCredential] SQLSvcAccount (Write): Service account for the SQL service.
SQLSvcAccount = $SqlInstallCredential
#[PSCredential] AgtSvcAccount (Write): Service account for the SQL Agent service.
AgtSvcAccount = $SqlInstallCredential
#[String[]] SQLSysAdminAccounts (Write): Array of accounts to be made SQL administrators.
SQLSysAdminAccounts = $SqlInstallCredential.UserName
ASSvcAccount = $SqlInstallCredential
ASSysAdminAccounts = $SqlInstallCredential.UserName
InstallSharedDir = 'C:\Program Files\Microsoft SQL Server'
InstallSharedWOWDir = 'C:\Program Files (x86)\Microsoft SQL Server'
InstanceDir = 'C:\Program Files\Microsoft SQL Server'
InstallSQLDataDir = 'M:\SQLData'
SQLUserDBDir = 'M:\SQLData'
SQLUserDBLogDir = 'L:\SQLLogs'
SQLTempDBDir = 'D:'
SQLTempDBLogDir = 'D:'
SQLBackupDir = 'U:\Utilities'
SourcePath = 'C:\SQLServerFull'
UpdateEnabled = 'False'
ForceReboot = $false
PsDscRunAsCredential = $SqlInstallCredential
DependsOn = '[WindowsFeature]NetFramework35', '[WindowsFeature]NetFramework45', '[File]FolderG', '[File]FolderL', '[File]FolderM', '[File]FolderT', '[File]FolderU'
}

Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Win Server 2012 + SQL Server 2012,
Win server 2016 + SQL Server 2016
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
xSQLServerSetup
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
<IP_ADDRESS>

I looks like Features is commented.
#Features = 'SQLENGINE'

Please change to
Features = 'SQLENGINE'

If that does not resolves it, then I'm uncertain of below.
SQLTempDBDir = 'D:'
SQLTempDBLogDir = 'D:'

I would have written it as
SQLTempDBDir = 'D:\'
SQLTempDBLogDir = 'D:\'

But both should work (unless there is a bug somewhere there). :)
The rest of the config looks okay to me. Let me know if any of this solves your issue.

Have tried with Features enabled, exactly the same.
Changed the paths to D:\ to double check, the same

Can you please send the verbose output from Start-DscConfiguration?
Also, out of our offline discussion you are trying to use xSQLServerSetup to change values in an already present instance ('elasta'?). And you are seeing some values change, and some values does not change. I understood that you were trying to change the service accounts and paths in the already present instance 'elasta'? I interested in what did change, with the above configuration, and what did not change.
And what errors were you seeing in the different scenarios?

For others reading this I forgot to add that xSQLServerSetup is not meant to be able to change those values. But since @MagicMarty, in our offline discussion, said he are seeing values actually change, I'm interested to see what values do change.


hello! build Error!!

You see the picture, the terminal print error message( error: array must be initialized with a brace-enclosed initializer )
I want to solve this problem... plz...

Thanks for your feedback. The sick_scan_xd driver requires C++14. C++11 introduced the initialisation of a std::array with braces like std::array<int, 3> a = {1, 2, 3}.
What gcc version and OS are you using? It looks like gcc does not yet support C++14, similar to #401 or #251. We recommend upgrading to Ubuntu >= 20.04 with gcc >= 5.

Unknown CMake command "target_link_options".
gcc version:  5.4.0
ubuntu version: 16.04 LTS

Hello! I have a question!!...
I use TIM7xxS object. I'm trying to port separately without using api ( without ROS and Ubuntu ). How can I access it? We need to use 18.04 version.

Thank you for more information. Ubuntu 16.04 is no longer supported. If you are using the Linux subsystem on Windows, we recommend to upgrade to Ubuntu 20 or newer. The cmake command target_link_options requires cmake 3.13 or later; we will update the cmake requirements.
To use a TiM781S without the sick_scan_xd API or ROS, you need to establish a TCP connection and communicate with the lidar using the appropriate SOPAS commands. See the TiM781S user manual and the TiM781S telegram listing for details. We recommend using the sick_scan_xd driver on Windows or Linux with Ubuntu 20 or newer.

Oh, I'm seokwan! Thank you for your response.
I have a one more question. If i use api, where i find Lidar
measurement data?? I need to your guide. I read document. But, I don't find
measurement data...
Thank you...
2024년 11월 6일 (수) 오후 5:36, rostest @.***>님이 작성:

Thank you for more information. Ubuntu 16.04 is no longer supported. If
you are using the Linux subsystem on Windows, we recommend to upgrade to
Ubuntu 20 or newer. The cmake command target_link_options requires cmake
3.13 or later; we will update the cmake requirements.
To use a TiM781S without the sick_scan_xd API or ROS, you need to
establish a TCP connection and communicate with the lidar using the
appropriate SOPAS commands. See the TiM781S user manual
https://cdn.sick.com/media/docs/4/54/254/operating_instructions_tim7xxs_en_im0083254.pdf
and the TiM781S telegram listing
https://cdn.sick.com/media/docs/7/27/927/telegram_listing_telegram_listing_ranging_sensors_lms1xx_lms5xx_tim2xx_tim5xx_tim7xx_lms1000_mrs1000_mrs6000_nav310_ld_oem15xx_ld_lrs36xx_lms4000_lrs4000_multiscan100_picoscan100_en_im0045927.pdf
for details. We recommend using the sick_scan_xd driver on Windows or Linux
with Ubuntu 20 or newer.
—
Reply to this email directly, view it on GitHub
https://github.com/SICKAG/sick_scan_xd/issues/414#issuecomment-2459003376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AL3WVZR57WFE7AYLXUX7AY3Z7HIKFAVCNFSM6AAAAABRF32AJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJZGAYDGMZXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kim Seok Wan
Handong Global University
Heunghae-eup, Pohang
[image: phone-icon]  010-5636-3680
[image: email-icon]  @.***
Github https://github.com/wanilly
*Commit your way to the *
LORD, trust in him and he will do this:Psalm 37:5
[image: facebook icon] https://m.facebook.com/profile.php [image:
linkedin icon] https://www.linkedin.com/in/seok-wan-kim-9b1956206/

The scan data is encoded in the LMDscandata telegram, see table 158 on page 97-109 in the TiM781S telegram listing.
Note that receiving the LMDscandata telegrams requires proper initialization with the appropriate SOPAS commands.  Ubuntu 16 and 18 have reached the end of their life cycle a few years ago and are not supported by sick_scan_xd.


Added note to Docker caching section

Just to clarify that the filename should be replaced with your own YAML filename, as this tripped me up recently

@markgaze : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@markgaze Thank you for your contribution! Approved ✔

#sign-off


SpawnGroup fields are inconsistent

As seen on the picture below, SpawnGroup fields are not correctly named, especially the "animal" one which should obv not apply to misc entities

The peaceful field is used only for this:
if ((spawnAnimals || !spawnGroup.isPeaceful()) && (spawnMonsters || spawnGroup.isPeaceful()) && (shouldSpawnAnimals || !spawnGroup.isAnimal()) && info.isBelowCap(spawnGroup)) {
	spawnEntitiesInChunk(spawnGroup, world, chunk, (entityType, blockPos, chunkx) -> {
		return info.test(entityType, blockPos, chunkx);
	}, (mobEntity, chunkx) -> {
		info.run(mobEntity, chunkx);
	});
}

The shouldSpawnAnimals used in this context is determined by worldProperties.getTime() % 400l == 0l. Perhaps it should be shouldSpawnRarely then?

I think animals is something from server properties file. Need to check again.

I think animals is something from server properties file. Need to check again.

Not a problem in latest yarn 07a.

Why? "Misc" spawn group still passes true for the "animal" field, which doesn't make sense

Hmm, so it's looking for usage of isAnimal

Afai see, that isAnimal is never called on MISC instances. In fact, it is only called in SpawnHelper.spawn(ServerWorld, WorldChunk, SpawnHelper.Info, boolean spawnAnimals, boolean spawnMonsters, boolean shouldSpawnAnimals) (the boolean fields have inadequate names) in a loop over SPAWNABLE_GROUPS, which has everything except MISC.

Now I see. that is not isAnimal, more like isRare. pr coming soon


Contribution ?

Cool stuff, but it's only contain open api spec ?
Is there any todos or plan for development ?
Thanks

@bahirul there are 3 working implementations that I know of, the only open source one so far is the FullCtl one at https://github.com/fullctl/peerctl
Thanks for asking tho, I'll add that to the README


Add support for marker clustering

Is your feature request related to a problem? Please describe.
I'm always frustrated when there are multiple pins in the same location and I can't use marker clustering :)
Does this feature exist in the Google Maps SDK for android and iOS? Please link the documentation for this feature.
iOS: https://developers.google.com/maps/documentation/ios-sdk/utility/marker-clustering
Android: https://developers.google.com/maps/documentation/android-sdk/utility/marker-clustering (video inside)
JS: https://developers.google.com/maps/documentation/javascript/marker-clustering
Describe the solution you'd like
Support native marker clustering for iOS and Android. I'm currently stuck with the mediocre performance of the JS implementation.
I'd love to help with the implementation, but I don't have much experience writing plugins or even native code, I'm a web developer. Can y'all start by checking how easy or difficult this seems?

+1
I did some research and found out that we need to implement the "Maps SDK for Android/iOS Utility Library". Looking at the implementation examples I don't even think it should be to hard to implement it using the default styling. Maybe that allowing for custom styling makes it somewhat more complicated. But we could even split this issue in two: "Add support for marker clustering" and "Add support for custom styling on marker clustering" for example.
About the Utils Library: of course it adds some weight to this package. So ideally we should publish it as a separate package. something like "@capacitor-community/capacitor-googlemaps-utils".

Hello, is this feature already on the way for an implementation ?
I ask you because I have a new projet which need this, and would avoid to use the JS implementation in favor of this plugin.
Unfortunately I'm also a web developper and helping for this kind of implementation seems difficult for me.

With the v2 release, this package is also getting interesting for us, so we can replace the javascript implementation. But we also need markerclustering before we could switch.

It's definitely on the roadmap and coming soon. But it's taking a little more time than expected.

So the documentation provides the enableClustering method, however no matter how I try I can't seem to get this to work. I note that this issue is still open - is this a work in progress or is there another issue I should post separately going on?

For all the companies who want this: how about you support the maintainers with your time & expertise or a budget?


dark mode support added to docs

hi any progress on this @roman-right ?

dark mode support added to docs.


Generator usage in place of paginators that loop over lists

Is your feature request related to a problem? Please describe.
Paginators are notoriously known for storing large amounts of data. If a paginators stores pages that could (potentially) contain a list millions of rows of data, this causes a big memory usage spike that will only become a bigger problem as our bot grows.
Describe the solution you'd like
Every generator from now on must store data or fetch data with the use of generators. Look into how generators work, how it's used and most importantly how to handle it.
Describe alternatives you've considered
None
Additional context
Lists in python are very dynamic and cover the vast majority of use cases like looping over specific indexes etc. Generators on the other hand, create the objects in memory only when they are looped over by calling next(gen).

2 attempts were made, both to no avail, I feel because of the limited understanding I have about how generators work. For now, I'll have to look into documentation before digging into this issue later on.

I unlikely don't have any more time left to look into this. Since the code is often paginating over connection query results, I may need to look into the row_factory attr for SQLite python wrapper docs.
I do believe it is possible to allow fetches to return generators but as of right now I've not looked into this much yet.

Might be a great idea to use the itertools library for this.


Users/scottluskcis/wire up events

this PR is setting up a lot of foundational stuff. Creating a PR now to go over with @GeneralPallas and start picking up the pace moving forward. this is still a work in progress but a lot of good stuff added

this addresses parts of #25 but not 100% @GeneralPallas

@GeneralPallas the test project that appveyor was failing on no longer exists. I'm in process of adding test project(s). For now I updated appveyor file to not run the test proj that no longer exists