Can't read from .env files using AddEnvFile(string)

When using the .AddEnvFile(string) method, specifying a .env file, a FileNotFoundException is always returned.
Example:
.ConfigureAppConfiguration((ctx, cfg) =>
  {
      cfg.AddEnvironmentVariables(prefix: "MyApp_");
      cfg.AddEnvFile(".env");
  })

Stacktrace:
System.IO.FileNotFoundException
The configuration file '.env' was not found and is not optional.
   at Microsoft.Extensions.Configuration.FileConfigurationProvider.HandleException(ExceptionDispatchInfo info)
   at Microsoft.Extensions.Configuration.FileConfigurationProvider.Load(Boolean reload)
   at Microsoft.Extensions.Configuration.FileConfigurationProvider.Load()
   at Microsoft.Extensions.Configuration.ConfigurationRoot..ctor(IList`1 providers)
   at Microsoft.Extensions.Configuration.ConfigurationBuilder.Build()

This is happening because by default, the FileConfigurationSource excludes dot-prefixed files, hidden files, and system files from being read into the configuration.
I've opened an issue on the .NET runtime repo asking why this is the case.
https://github.com/dotnet/runtime/issues/54712

Fixed in #3
The misleading error is being addressed in a PR on the .NET Core runtime PR: https://github.com/dotnet/runtime/pull/54717


Append rows to a file

Every time I need to update a file, I have to read the it and save on memory to append the new rows and then overwrite the content.

This is the expected behaviour according to the documentation so I don't understand the issue ?

is not an issue just a new feature.

What you are requesting is not possible because of the way PHP reads a CSV. The document is treated as a stream so you can write a new one or append an existing one but you can not update selected rows without at least a temporary file.
I would be more than happy to review a PR with your feature request if you can preserve the file cursor position between updates.

sorry if I misspoke, my comment was just to try to improve the library.
Do not think that I did a critique. On the contrary, I used the library many times and it works very well for me.
I just said my opinion to express what could be ideal. I would love to be part of the contributors, i am willing to try and join your list. :) @nyamsprod

@abrahammontas I did not take it as a critique I've only highlighted the issues with your proposed feature.
As for submitting a PR, you don't need to be part of the phpleague to contribute to the library, just submit your work following the rules explained it the contribution guide and I'll be more than happy to review it


Cannot read property 'Polyline

Running the Wrapped.Polyline example leads to UnCaught TypeError: Cannot read property 'Polyline' of undefined in the console.

@BrunerChris are you putting Leaflet within a script tag on index.html of your application? I experienced the same problem since I was using Leaflet through npm?

Yes I am putting Leaflet in a script tag on index. I am also using Leaflet through npm.

Any updates on this? Having the same issue but with .Polygon. I have a Meteor site and am accessing Leaflet through several script tags in my index.html but they are CDN references and that works fine. Since this does not have a CDN I have to reference the file, and Meteor just can't find it regardless of the path. Not sure if something else is wrong. Any chance you could make this a CDN? Might solve the issues above too, if we could just reference it externally.


SCT-1147 More error message cleanup

instance var should be stripped as well

Pull Request Test Coverage Report for Build 68

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 94.237%

Totals

Change from base Build 67:
0.02%

Covered Lines:
278

Relevant Lines:
295

💛  - Coveralls

:+1:


"Clear History" disabled in 1.2.9 beta 12

The "Clear History" command from the menu is disabled in 1.2.9 beta 12. It was fine in 1.2.9 beta 11.

My mistake, it will be fixed soon.

Can you please publish a new release that includes this fix? Or provide instructions on building manually. Thanks.

released.


Known issue: Cbc 2.10.4 incorrectly reports infeasible (--> Update to Cbc 2.10.5?)

What version of OR-Tools and what language are you using?
Version: Google.OrTools 8.2.8710 (from NuGet)
Language: C#
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Cbc
What operating system (Linux, Windows, ...) and version?
Windows x64 19042.870
What did you do?
I solved a MIP using Cbc with a timelimit.
What did you expect to see
Feasible solution or optimum.
What did you see instead?
Cbc reported "Infeasible" for the model.
Anything else we should know about your project / environment
This is a known issue in Cbc 2.10.4 where it incorrectly reports "Infeasible" when using time limits. The following 2.10.5 release from March 2020 fixes the problem. I'd be great if you could update the bundled Cbc version to 2.10.5.

CBC 2.10.5 is know to be buggy.

IIRC https://github.com/coin-or/Clp/issues/130 is blocking us from any upgrade...


Add serenity as a service for building Discord bots

Serenity is a framework for building Discord bots. Implementing Service for it under a flag would make it possible to host discord bots on Shuttle.
use serenity::Client;

#[shuttle_service::main]
async fn init() -> impl shuttle_service::SerenityService {
    let token = env::var("DISCORD_TOKEN").expect("token");
    let intents = GatewayIntents::non_privileged() | GatewayIntents::MESSAGE_CONTENT;

    let mut client = Client::builder(token, intents)
        .event_handler(...)
        .framework(...)
        .await
        .expect("Error creating client");

    Ok(client)
}

This should be a simple addition Serenity's client has a simple start function which selects and connects to a free port via web sockets:
#[async_trait]
impl Service for serenity::client::Client {
    async fn bind(mut self: Box<Self>, _addr: SocketAddr) -> Result<(), error::Error> {
        self
            .start()
            .await
            .map_err(error::CustomError::new)
    }
}

Seconding this, would love to see this added!


Add W3C tracing links

Add variables for W3C tracing

@mariash, if you also want this in TAS 5.0, please add this also to branch TAS-5.0.


want to ADD social icon on left of page

Is your feature request related to a problem? Please describe.
want to add social icons on left of page and it will be fixed at that position as shown in video
Describe the solution you'd like
want to add social icons on left of page and it will be fixed at that position as shown in video
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
want to add here :

demo  :
https://github.com/user-attachments/assets/8d962c41-dfb8-4c40-9c24-3ddcc7b13ce4

@thestarsahil please assign me this issue under gssoc labels


Iterators: Clean up structure, simplify runtime stubs

This PR prepares an up-coming one introducing a first conversion for the Iterators dialect. I am separating the two in order to have CI run on the changed runtime stubs.

can we move mlir/iterators/* to iterators/* or mlir_iterators/* before landing?


FR: provide (partial) query metadata to form endpoint as with execute endpoint for validation

I am trying to build a CRM integration, whereby you sync a list of companies/people from Looker into our CRM (this is a private action hub action).
This requires that the query contains a column corresponding to an id for a company/person. I can validate this at execution time, but it's not a great experience for users as noted in this issue.
So it would be nice if the form endpoint could be used as an opportunity to validate the query in some sense and then return an error (given the form endpoint does support displaying an error message), or if there are multiple possible id columns in the query then provide them in a dropdown select in the form config returned by the endpoint. Currently you don't get so much as a query id on the form endpoint (which might let you hack around this by using other looker api calls of some kind).
Thanks.

The required_fields config should actually cover much of what i had in mind, although it doesn't actually seem to work as far as i can tell with my custom action hub.
I tried it for the Hubspot action, and that seems fine, but when i configure my own action more or less identically to the Hubspot one (based on what i can see in the /integrations endpoint response), it doesn't seem to do anything for me, i.e. the action is shown regardless of which columns are in the query even though the actions list does show "Action can be used with queries that have a field tagged ___" for my action.
Aside from that it would also be handy to still have the original ask here as you could then map columns in the query to fields in the CRM or something

Another use case is if you want to allow users to configure different behaviour for each of the columns in their query, e.g. mapping to a field in a CRM, or handling changes in different ways (e.g. ignore small changes in magnitude, ignore entirely etc.)


Implement slice() for EvaluatedPositionalArgs and concrete classes

Adds slice() to the interface and implements it for the concrete classes. I'm using this for closure components but until now was doing it by ripping positional args apart and creating a new one. This cleans things up nicely.

Is this still relevant?

Nope. Closing.


Inconsistent supported python version with one specified by dependecy modules

Pypi of pylxd ver 2.2.10 build shows at classifier field that it supports upto python version 3.6. However, requests-toolbelt module and ws4py module, what pylxd depends on, appear to constrain python version upto 3.5.
Expected:

classifiers field of pylxd's pypi meta shows supporting upto python ver 3.5 as requests-toolbelt module and ws4py module supports upto python ver 3.5.

Actual:

classifiers field of pylxd's pypi meta shows supporting upto python ver 3.6 (as defined in setup.cfg)

Supplement info:

Pypi meta excerpt of pylxd shows that it supports upto python 3.6
  $ curl -s -L https://pypi.python.org/pypi/pylxd/json | jq '.info | {"name":.name ,"version":.version, "classifiers":.classifiers}'
  {
    "name": "pylxd",
    "version": "2.2.10",
    "classifiers": [
      ...
      "Programming Language :: Python :: 3.6"
    ]
  }

Supported python version by dependency modules appears upto 3.5
pylxd dependency tree in excerpt of pipenv's graph command output for my project using pylxd:

pylxd [required: Any, installed: 2.2.10]

cryptography [required: >=1.0,!=1.3.0, installed: 2.9.2]

cffi [required: >=1.8,!=1.11.3, installed: 1.14.0]

pycparser [required: Any, installed: 2.20]

six [required: >=1.4.1, installed: 1.14.0]

pbr [required: >=1.6, installed: 5.4.5]
python-dateutil [required: >=2.4.2, installed: 2.8.1]

six [required: >=1.5, installed: 1.14.0]

requests [required: >=2.20.0, installed: 2.23.0]
requests-toolbelt [required: >=0.8.0, installed: 0.9.1]

requests [required: >=2.0.1,<3.0.0, installed: 2.23.0]

requests-unixsocket [required: >=0.1.5, installed: 0.2.0]

requests [required: >=1.1, installed: 2.23.0]
urllib3 [required: >=1.8, installed: 1.25.9]

six [required: >=1.9.0, installed: 1.14.0]
ws4py [required: >=0.3.4,!=0.3.5, installed: 0.5.1]

Among them, requests-toolbelt ver 2.8.1 and ws4py ver 0.5.1 support only upto python 3.5:

requests-toolbelt
  $_pymdl="requests-toolbelt"; \
  yolk -M "${_pymdl}" | grep -E '^name:|^version:'; \
  yolk -M "${_pymdl}" -f classifiers | tr ',' '\n' | awk '{gsub(/'"'"'|[ ]+|]/,"",$0); print $0}' | grep -E '[0-9]$' | awk -F':' '{print $(NF-2) "::" $NF}' | sort -r -u -V
  name: requests-toolbelt
  version: 0.9.1
  Python::3.5
  Python::3.4
  Python::3.3
  Python::3
  Python::2.7
  Python::2

ws4py
  $_pymdl="ws4py"; \
  yolk -M "${_pymdl}" | grep -E '^name:|^version:'; \
  yolk -M "${_pymdl}" -f classifiers | tr ',' '\n' | awk '{gsub(/'"'"'|[ ]+|]/,"",$0); print $0}' | grep -E '[0-9]$' | awk -F':' '{print $(NF-2) "::" $NF}' | sort -r -u -V
  name: ws4py
  version: 0.5.1
  Python::3.5
  Python::3.4
  Python::3.3
  Python::3
  Python::2.7
  Python::2

Note that whilst ws4py in pypi shop do not support 3.7 and up, in Debian and Ubuntu ws4py are patched for 3.7+ support.
I don't  know if requests-toolbelt works or not.

Note that trove classifiers are not a great way of determining the version compatibility of a packages.  Many packages will just work with new versions of Python, and developers might not update the classifiers when publishing new releases.
The main concern is the ws4py dependency, right?

Correct ws4py is dead upstream, last known maintainer said they don't have time to do things. Distros have ws4py patched, but the version from pypi is broken.
Maybe you can switch requirements.txt syntax to pull ws4py from Ubuntu or Debian tarball and/or patch it, when building a part for it? As the stock one from pypi is borked.

Or for example please drop ws4py from the requirements.txt it is optional, and code works without it. And if someone has a working copy of ws4py => great, otherwise a broken one is not automatically installed for those that follow requirements.txt from pypi.


Adding Innr BY 285 C lightbulb

The Innr BY 285 C light bulb supports full color and color temperature modes, this adds both measurement files.
This is measured with a Shelly 1PM, using the Shelly API with the new fix. Thanks again for that, it was much easier to use !

Thanks, merged.


[UPDATE] Update Freecad to 1.0.0

FreeCAD released a big update

Should we create also a PR to move FreeCAD to the main bucket?


Limiting acceleration level

As a company, we don't want to allow the users to immediately change the car's direction, so that the motors of the car will stay unharmed.
Acceptance criteria:

The app shall not allow the user to press multiple buttons at the same time.

Since this issue is about keeping the real car safe and we are not going to work with the real car, the group decided that it would not be useful.


problem using number-recognizer

Hi Dear
I  have problem using number-recognizer. Please help me.

You spelled recognizer wrong. Change the function class to NumberRecogniser

thank a lot
It seems that the probes remains.

pip uninstall number_recognition
pip3 install number_recognition

https://github.com/mauro-balades/number-recognizer/releases/tag/1.0.5

Thanks a lot
If the background of the image was not simple the program does not work.

Think about that as much as you train it more, the better the result. In this case, you only trained it 10 times (which it is the default) meaning that it will recognise ease number. But if you trainit 100 times it can recognize a number made by a 3 year old.
if you want to add more trains, add this:
n.init(epochs=100)


License

There is no license specified on this project, which makes it hard to know if it can be incorporated into other projects.

Thanks, i added a license file.


ORC-1721: Upgrade aircompressor to 0.27

This is a clone of

#1943

This is resolved via https://github.com/apache/orc/commit/71fc004ff0b32a001c26fe3f61e469836b046440


Warn: Module not found: Can't resolve 'encoding '

Describe the issue

During runtime, I get a warning message of:
- warn<EMAIL_ADDRESS>Module not found: Can't resolve 'encoding' in<EMAIL_ADDRESS>
Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS>./node_modules/.pnpm/@simplewebauthn+server<EMAIL_ADDRESS>./node_modules/.pnpm/@simplewebauthn+server<EMAIL_ADDRESS>./lib/login.ts

<EMAIL_ADDRESS>Module not found: Can't resolve 'encoding' in<EMAIL_ADDRESS>
Expected behavior
I expect that this warning should not have shown as having @simplewebauthn/server as the project dependency should self-contain all its own dependencies.
Code Samples + WebAuthn Options and Responses

The imports I have in login.ts from @simplewebauthn/server are
import type {
  VerifiedAuthenticationResponse,
  GenerateAuthenticationOptionsOpts,
  VerifyAuthenticationResponseOpts,
} from '@simplewebauthn/server'
import {
  verifyAuthenticationResponse,
  generateAuthenticationOptions,
} from '@simplewebauthn/server'

Dependencies

OS: macOS 13.5.2 (22G91)
Browser: Chrome Version 118.0.5993.88 (Official Build) (arm64)
Authenticator: MacBook Pro Touch ID

SimpleWebAuthn Libraries

@simplewebauthn/browser 7.2.0
@simplewebauthn/server 7.3.1
@simplewebauthn/typescript-types 7.0.0

Additional context
I can fix this by installing 'encode' as a dependency of my project with the command pnpm add -D encoding but my understanding @simplewebauthn/server should already have 'encode' as its own dependency
I am using NextJS App Router and my login.ts file is a 'use server' file.
This warning shows up on page renders before any button is pressed to run any function.

Hmm, I tried to use a barebones project repo using pnpm to install dependencies, but I wasn't able to recreate the issue you're encountering. I'm going to need you to help me out here with a basic reproduction repo that can recreate this, especially since you mentioned that you're experiencing this in your NextJS project.
Once I have something that can pull down locally to reproduce the problem is as basic a NextJS project as you can manage then I'll probably have a shot at figuring out what's going on here.

Three weeks out and there's been no follow-up so I'm closing this out for now. Please feel free to reopen if you can provide a minimal reproduction that I can use to reproduce this locally.


robyn_refresh produces only models different from the original

Project Robyn
Describe issue
Hi team Robyn,
I have a working model, where some channels were set to 0 by the model. Other models from the same run did set 2 or 1 more channels to 0, which did not seem realistic. So I chose the one that had the minimum amount of channels being set to 0, and most realistic ROIs.
After choosing the best model, I reproduced it multiple times to run the budget allocator. The ROIs from the reproduce function were the same as in the original model.
Now, I would like to refresh the model with 30 new weeks of data (original model has 173 weeks). When I do that with the refresh function, two channels are set to 0, which had an effect in the original model. For a test, I tried to refresh it with only two weeks of data, which produces the same effect - two channels are set to 0 in all the model variants. Since that does not seem realistic from the channels perspective, I would want to get more or less similar results to the original model, at least in the sense that the channels that had an effect before still do, even if we update the model with 2 weeks of data.
A couple of things that could be relevant here:

All the hyperparameters seem more or less the same as in the original model, except for penalty, which cannot be specified (it always runs with the range 0-1). It seems like the optimization process is going in the direction of the neighbour models of the original run (with those channels being 0), and cannot settle on anything similar to the original model. Is there any way to also specify the penalty? Maybe that would help nudge the process in the direction of the chosen model.
Several channels being set to 0 can be explained by multicolinearity between channels, however it is lower than 0.5, so i would not expect for it to have such a big effect.

What other reasons can there be for such a result?
Provide reproducible example
I'm attaching the results of the original model (upper), and the example of a refreshed model (lower) (with 2 weeks).

The code I am using for refresh:
 Robyn <- robyn_refresh(    json_file = json_file,    dt_input = data,    dt_holidays = prophet,    refresh_steps = 2,    refresh_mode = "manual",    refresh_iters = 2000,    refresh_trials = 3,    version_prompt=FALSE,   plot_folder = ''  )
I'm also including hyperparameters from the refresh + hyperparameters for the original model.
hyperparameters_refresh_issue.xlsx
Let me know if anything else is needed to debug.
Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Check and share Robyn version: ‘<IP_ADDRESS>09’
R version (Please, check and share: 4.2.2 Patched (2022-11-10 r83330)

@ddempseyr @gufengzhou
Hi there,
I am encountering this issue as well. Is there any update here?
Thank you!

FYI I've shipped an update to refresh and hopefully the results are now more consistent. Please update the package and test

Hi @gufengzhou
I am facing the same type of issues while doing refresh.
This image is after the refresh of the model (have added 2 weeks  in the refreshed file):

This image is for the main model(solid)  :


Code release

Nice works! Hope the code released as soon as possible!

Hi, any progress ?

Sorry to keep you waiting for so long, we have just released the initial version of the code.

Thanks!


Updating channel topic returns 400

Running the following code:
discord_client
    .update_channel(channel_id)
    .topic(text)
    .await?;

Results in the following error (wrapped in my own type):
[mc-server-wrapper/src/main.rs:186] &e = Err(
    DiscordErr(
        Response {
            source: Client {
                response: Response {
                    url: "https://discordapp.com/api/v6/channels/694351655667367957",
                    status: 400,
                    headers: {
                        "date": "Mon, 20 Apr 2020 23:53:47 GMT",
                        "content-type": "application/json",
                        "content-length": "36",
                        "connection": "keep-alive",
                        "set-cookie": "__cfduid=d9f8c6a319ca0c8a0680ba1ef963430001587426827; expires=Wed, 20-May-20 23:53:47 GMT; path=/; domain=.discordapp.com; HttpOnly; SameSite=Lax",
                        "set-cookie": "__cfruid=9b661e569b275d07d16fd2d6f44c2e6a76c40d70-1587426827; path=/; domain=.discordapp.com; HttpOnly; Secure; SameSite=None",
                        "strict-transport-security": "max-age=31536000; includeSubDomains",
                        "x-ratelimit-bucket": "9852e1a53c06ffc5a89d65fef85ca4ce",
                        "x-ratelimit-limit": "5",
                        "x-ratelimit-remaining": "4",
                        "x-ratelimit-reset": "1587426842.243",
                        "x-ratelimit-reset-after": "15.000",
                        "x-envoy-upstream-service-time": "18",
                        "via": "1.1 google",
                        "cf-cache-status": "DYNAMIC",
                        "expect-ct": "max-age=604800, report-uri=\"https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct\"",
                        "server": "cloudflare",
                        "cf-ray": "5872c9660d95c568-ORD",
                        "cf-request-id": "023b9c33c40000c56819390200000001",
                    },
                },
            },
        },
    ),
)

I took the time to match on everything and print out the response body:
[mc-server-wrapper/src/main.rs:190] response.text().await = Ok(
    "{\"name\": [\"This field is required\"]}",
)

Which is very odd. The Discord docs state that all of the parameters are optional.
Unsure where the problem lies here (am I doing something stupid?), so I figured I'd open an issue :).

Could you try editing the channel with the name and see if it'll work? Discord's docs may be wrong.

Sure:
discord_client
    .update_channel(channel_id)
    .topic(text)
    .name("test")
    .await?;

Results in a 400 again with the following body:
[mc-server-wrapper/src/main.rs:190] response.text().await = Ok(
    "{\"type\": [\"Value must be one of (0,).\"]}",
)

If I try to set a type as well:
discord_client
    .update_channel(channel_id)
    .topic(text)
    .name("test")
    .kind(ChannelType::GuildText)
    .await?;

403 with body:
[mc-server-wrapper/src/main.rs:190] response.text().await = Ok(
    "{\"message\": \"Missing Permissions\", \"code\": 50013}",
)

...which is again very confusing. I did a basic sanity check to make sure my bot has the Manage Channels permission enabled, and indeed it does:

At this point I gave my bot the Administrator permission and tried again. It worked! The topic (and the name since I had to add that) of the channel got updated.
 Unfortunately it still didn't work, returning a 400 and again complaining that the name field has to be provided.
It appears Discord's docs are wrong? Fields are required, and on top of that the MANAGE_CHANNELS permission is not enough to edit the channel (although the docs state that it is).
I find it incredibly hard to believe this is broken and hasn't been noticed by anyone else. Is there anything else we could be missing?

Update: I came back to this in an effort to figure out what was going on as I wanted to be able to set channel topics correctly. It turns out that Discord's docs were indeed at least partially wrong; I had been reading the route description that stated that all JSON parameters were optional, but looking below that, the name and type fields are clearly non-optional. I filed https://github.com/discord/discord-api-docs/pull/1632 to fix the docs.
I set up the InMemoryCache to be able to keep channel names and kinds around to pass along with the new topic when updating a channel.
(Oddly enough, the permissions issue I described above disappeared at some point amidst my changes; I'm not sure what caused it and don't feel like figuring it out anyway :smile:.)


Unable to login to amazon.it/amazon.es with 2FA enabled

Describe the bug
Unable to login to amazon.it. The configurator loops at the begin stage.
This appears to only happen for the amazon.it domain.
To Reproduce
Steps to reproduce the behavior:

Set domain to amazon.it
Try to log in using configurator
Configurator loops

Expected behavior
A clear and concise description of what you expected to happen.
Able to log in.
Screenshots
This is the debug page.

System details

Home-assistant (version): 0.98.5
Hassio (Yes/No): (Please note you may have to restart hassio 2-3 times to load the latest version of alexapy after an update. This looks like a HA bug).
alexa_media  (version from const.py or HA startup): 2.1.2
alexapy (version from pip show alexapy or HA startup): 1.1.2

Logs
2019-09-08 22:31:03 DEBUG (MainThread) [alexapy.alexalogin] Processing https://www.amazon.it/ap/signin
2019-09-08 22:31:03 DEBUG (MainThread) [alexapy.alexalogin] Links: {}
2019-09-08 22:31:03 DEBUG (MainThread) [alexapy.alexalogin] Error page detected:
2019-09-08 22:31:03 DEBUG (MainThread) [alexapy.alexalogin] Found post url to get; forcing get to https://www.amazon.it/ap/signin/258-9650445-6341918?ie=UTF8&showRememberMe=true&showRmrMe=1&openid.identity=http%3A%2F%2Fspecs.openid.net%2Fauth%2F2.0%2Fidentifier_select&pageId=amzn_dp_project_dee_it&openid.return_to=https%3A%2F%2Falexa.amazon.it%2F&prevRID=RE5Z1SS4JZW7WR3JJA6Z&openid.assoc_handle=amzn_dp_project_dee_it&openid.mode=checkid_setup&prepopulatedLoginId=&failedSignInCount=0&showPasswordChecked=false&rememberMe=true&openid.claimed_id=http%3A%2F%2Fspecs.openid.net%2Fauth%2F2.0%2Fidentifier_select&email=alandtse%40gmail.com&openid.ns=http%3A%2F%2Fspecs.openid.net%2Fauth%2F2.0&ubid=260-9272972-XXXXXX/YYY-9650445-6341918?ie=UTF8&email=email%40email.com

Additional context
The root cause is the fact something is causing Amazon to invalidate the cookies in the login process. This does not appear to happen with 2FA disabled. Other domains do not have any issues.
Original reports on 2.1.0:
https://community.home-assistant.io/t/echo-devices-alexa-as-media-player-testers-needed/58639/3123
https://community.home-assistant.io/t/echo-devices-alexa-as-media-player-testers-needed/58639/3141

Ok playing around more with this, it feels like only alexa.it has this problem and only if you have 2FA enabled. I have a new build I'll push that resolves the specific error but not the inability to use 2FA for amazon.it.
Workaround when I do the new release will be to disable 2FA to get the cookie and then reenable.
Originally posted by @alandtse in https://github.com/custom-components/alexa_media_player/issues/344#issuecomment-529183568

Hi Alandtse,
I update alexapy to 1.1.2 and also the custom component to 2.1.2.
The 2FA is disabled, but I still have the same error.
This is from log HA:
2019-09-09 20:15:01 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.1361617296] get
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/websocket_api/commands.py", line 130, in handle_call_service
connection.context(msg),
File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1235, in async_call
await asyncio.shield(self._execute_service(handler, service_call))
File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/core.py", line 1260, in _execute_service
await handler.func(service_call)
File "/srv/homeassistant/lib/python3.7/site-packages/homeassistant/components/configurator/init.py", line 235, in async_handle_service_call
await self.hass.async_add_job(callback, call.data.get(ATTR_FIELDS, {}))
File "/home/homeassistant/.homeassistant/custom_components/alexa_media/init.py", line 165, in configuration_callback
login, callback_data)
File "/home/homeassistant/.homeassistant/custom_components/alexa_media/init.py", line 155, in setup_platform_callback
await login.login(data=callback_data)
File "/home/homeassistant/.homeassistant/alexapy/alexalogin.py", line 306, in login
headers=self._headers)
File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/client.py", line 466, in _request
ssl=ssl, proxy_headers=proxy_headers, traces=traces)
File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/client_reqrep.py", line 286, in init
self.update_host(url)
File "/srv/homeassistant/lib/python3.7/site-packages/aiohttp/client_reqrep.py", line 340, in update_host
raise InvalidURL(url)
aiohttp.client_exceptions.InvalidURL: get

Can you please enable debug logging on alexapy? Check the wiki for instructions.

I had the same issue, I had to disable 2FA to make it work again.

I think I've tracked it down as it looks like aiohttp isn't handling expired cookies properly which gets exposed with the 2FA path. Need to think of a clean workaround. Totally possible it's a configuration error on my end but I'll see if the aiohttp can reproduce the issue.

Ended up replacing the cookiejar and submitting a patch request back to aiohttp. Until merged, the PR on gitlab should solve this.  I'll stage it for the next release but anyone who is adventurous can manually replace the files.


add soft fail as an arg

I'll update the BC-action as well after this one

Build will still fail due to problem-matcher. to enable soft fail on checkov-action see the following example: https://github.com/bridgecrewio/bridgecrew-action/blob/master/entrypoint.sh

It would be great if the tiny bit of remaining work for this PR could be completed in order to get this merged.
It would help to get this action running on existing repo.

Will take a look at this!

Requesting further review @schosterbarak :)

Is there any update on this PR? Is it complete and able to be merged to main?


🐛 Fix -examples output, shouldn't require -p or any tokens in environment variables

Fixed in branch update/inurl-and-infile


Split Icons.swift file into 5 parts

Splits the Icons.swift file into 5 equal parts in order to cut down on RAM requirements during build.
Previously Linux would consume around 20GB of ram during a build.
Checking to see if this helps.

I split the .allIcons struct into five parts and that helped reduce the memory footprint during builds by about 10 fold.


Special cases of hexadecimal NCRs are not decoded properly

#8 only escapes HTML codepoints, not other NCRs in hex

Kittens saved! Yay!


mat-calendar: Allow to toggle first day of week

Feature Description
I'm writing an application which should support multiple locales and I would like already rendered calendar objects to reflect the change when the user decides to switch locales.
Use Case
The locale switching is tied to a request for language change. I'm using @ngx-translate for this. Below is a crude example using two locales (English and Norwegian):
this.translate.onLangChange.subscribe(lng => {
  this.dateAdapter.getFirstDayOfWeek = () => (lang === 'en' ? 0 : 1);
  this.dateAdapter.setLocale(lang === 'en' ? 'en-US' : 'nb');
});

The calendar object is rendered inside a mat-menu, and does not destruct when the menu is closed - only hidden.
<mat-menu #dialog="matMenu">
  <mat-calendar cdkTrapFocus [startAt]="value" [minDate]="min" [maxDate]="max" [selected]="value" (selectedChange)="select($event)">
  </mat-calendar>
</mat-menu>

So initially, when the application starts, the locale and first day of week is set to whatever the default language is at that point (set from browsers default language). But after user initiated language change, any change I do to the dateAdapter has no effect on the currently rendered mat-calendar.
I would like a way to trigger a re-render of the calendar using the current settings, and I had hoped that setting the locale would trigger this.
I've worked around this by:
<mat-menu #dialog="matMenu">
  <mat-calendar cdkTrapFocus
      *ngIf="_reload"
      [startAt]="value"
      [minDate]="min"
      [maxDate]="max"
      [selected]="value"
      (selectedChange)="select($event)">
  </mat-calendar>
</mat-menu>

And then toggling the reload in order to destroy the calendar and recreate it:
this.translate.onLangChange.subscribe(lang => {
  setTimeout(() => this._reload = false );
  setTimeout(() => this._reload = true );
});

But this seems to be a hack. Would very much like a better solution.

The first day of the week is controlled by the data adapter's getFirstDayOfWeek property.

Yes, I know @crisbeto . Demonstrated by the first code block entered under use case above. The point is that the mat-calendar is not re-rendered when I alter this setting on-the-fly. Which is what I mean by "toggle" in the issues title.

I see. I must've missed it the first time around. Here's a PR to fix it #18094.


Portal UI

The portal interface displayed the configuration walkthrough is outdated to the portal today. Is there a Portal UI refresh going to be updated for this document? This is significantly different than the UI presented above. Specifically, the Portal UI on adding a JIT policy rule shows that when selecting a CIDR block, there is a (i). This gives the information that you can provide a CIDR block, an IP addresses, or multiple IP addresses separated by a comma. Currently, only single IP addresses or single CIDR blocks are accepted. Can the documentation be updated to correctly reflect the limitations or product updated to correct function as documented?

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

ID: 25f0758b-8274-d2b5-4702-2c7ca6b0b14a
Version Independent ID: d1215107-abf9-fefd-a63b-b20ec3f7f744
Content: Just-in-time virtual machine access in Azure Security Center
Content Source: articles/security-center/security-center-just-in-time.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@cogran
Thanks for the feedback! We are currently investigating and will update you shortly.

@monhaber
Monica, could you kindly look into the document for updation?

Thanks for the valuable feedback!
We're building a plan for a complete refresh of all the screenshots in this doc set and will handle this JIT page along with all the others.

#please-close


Update scala minor versions

Coverage increased (+0.1%) to 96.074% when pulling 941727d352d52b83ce466dce127b4efb7113b473 on nafg:patch-2 into a72df8289201e21de94e3bb9d879031f1fe100ab on mohiva:master.

Coverage increased (+0.1%) to 96.074% when pulling 941727d352d52b83ce466dce127b4efb7113b473 on nafg:patch-2 into a72df8289201e21de94e3bb9d879031f1fe100ab on mohiva:master.

Coverage increased (+0.1%) to 96.074% when pulling 941727d352d52b83ce466dce127b4efb7113b473 on nafg:patch-2 into a72df8289201e21de94e3bb9d879031f1fe100ab on mohiva:master.


Update toggle and click handlers

This PR covers two additions to the angular-ivh-treeview directive:

Add an option to disable expand/collapse toggling on label click
Add the ability to disable the click handler when only toggling expand/collapse

:+1:

:+1:


Fix to stop AddLoadingScreenScript compile errors

Reopening this, because as discussed before, reorganized the project to have a LoadingScript.cs that is already within the play-instant-unity-plugin directory. Instead the changes have been as follows:

A new directory was created in the unity plugin to house the LoadingScreenScript.cs and the LoadingScreenConfig.cs (the Json object).
LoadingScreenGenerator now creates a directory, "PlayInstantLoadingScreenAssets", which houses the new loading screen scene, and a "Resources" file within that directory that houses the newly generated LoadingScreenConfig.json.
LoadingScreenGenerator only generates the LoadingScreenConfig.json with the parameters given from QuickDeployConfig, and then adds the LoadingScreenScript.cs component to the game object in the given play-instant-loading-screen-scene.
LoadingScreenScript.cs, at game runtime, now loads the json asset from the Resources file and then reads from it the assetBundle Url. It then opens the specified assetBundle.

No UI changes done in this PR.


Pie chart LABEL TYPE does not work

The Pie chart has a customization which lets you choose the label to display. The control has no effect.
Expected results
Expecting the labels to display % and/or category based on LABEL TYPE selection.
Actual results
Nothing happens.
Screenshots

Environment

superset version: 1.0.1
python version: 3.7
node.js version: v14.15.5

i believe this has been fixed in master. please double check and lmk~🙏


Fix rnn cell scope bug in _linear

The optional scope argument of _linear is currently ignored, as explained here: https://github.com/tensorflow/tensorflow/issues/6065
This pull request should fix that bug.

Can one of the admins verify this patch?

@rdipietro - Made the same fix in #6009. :)

Oh ok. Closing this pull request. Leaving https://github.com/tensorflow/tensorflow/issues/6065 open.


Not access to featues

Hi Developer,
The site (Windows 11 in React) is not so good then the real windows 11. Requesting to make File Explorer and make a login and sign-up gage so that user can save his work. This is half windows 11 then the real windows 11

No one is saying this is a real windows 11 and of course there will be some limitations as it is being developed as web based application.

Hi Blueedgetechno,
I know that it is not real windows 11 but compared to real windows 11 your team needs to work hard to make it more attractive. And if your web application windows 11 has not much features then no one will enjoy.
Thanks for Comment,
Akshat Anand


compatible with py3

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: gentelyang:x: liyang109liyang109 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

cla problems


Support providing serviceAccountName with create/update/apply commands

Pull request

What this PR does / why we need it
Add support to se serviceAccountName property in a workload yaml file.
It can be added while created or added after creation in yaml file and using update/apply commands for the workload to change
Which issue(s) this PR fixes

Fixes #64
Describe testing done for PR

Unit testing inside source code.
Created a workload with an example yaml file with the property set.
Created a workload with an example yaml file without the property, then changed it to add it as part of the spec, used both, update and apply commands to check if the change was taken.

Additional information or special notes for your reviewer

Signed-off by: Wendy Arango<EMAIL_ADDRESS>

Create a workload with serviceAccountName in the spec section

Delete serviceAccountName from spec section

Change current serviceAccountName using apply

Deleting serviceAccountName by setting it to serviceAccountName=""

Acceptance
Looks good overall however...
if I execute any update that doesn't include spec.serviceAccountName key/value explicitly in the incantation, spec.serviceAccountName the plugin removes it from the workload object.
The plugin should only remove it when spec.serviceAccountName: "" is provided.


docs: update binding example

fix typo and change environ to environment

It should be environ according to this code that handles the tag:
https://github.com/ii64/go-binder/blob/56a8eaa4ce2fcebf2780c014e3af46453492700a/binder/binder.go#L294-L298

LGTM


LibWeb: Account for header size when reading MessagePort message payload

Previously, the fact that this wasn't accounted for could lead to a crash when large messages were received.
This fixes:  https://wpt.live/workers/semantics/interface-objects/002.worker.html  and https://wpt.live/dom/events/AddEventListenerOptions-signal.any.worker.html (and probably others).

macOS CI says:
Timeout: /Users/runner/work/ladybird/ladybird/Tests/LibWeb/Text/input/Worker/Worker-large-message-crash.html


Getting an AUTH_NO_ORIGIN on PM2 instance

Environment

Operating System: Linux
Node Version:     v18.19.0
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
No response
Describe the bug
I am using PM2 to run the Nuxt3 build with sidebase/nuxt-auth in a production-like AWS instance.
After the latest update, the app no longer runs successfully as it throws an AUTH_NO_ORIGIN error in runtime.
My .env file has it:

API_BASE_URL=https://dev.mysite.com/api
APP_URL=https://nuxt3-dev.mysite.com
AUTH_ORIGIN=https://nuxt3-dev.mysite.com

I've also tried setting it separately in nuxt.config.js, but the error seems to remain.
Additional context
No response
Logs
No response

I use set -a && source .env && set +a in the project directory to export env variables and then use pm2 start ecosystem.config.cjs to start.
In case of updates
# Load env variables
set -a && source .env && set +a

# Restart the application
pm2 restart EXAMPLE.com --update-env

@tberk  - your solution helped. thank you!
I'm leaving this as an open issue because the library should look for .env variables defined on the application level.

@matthewairalo nuxt itself doesn't read .env files in production either if that's what you're suggesting

Yes, I believe this issue isn't with the library itself but rather relates to how Nuxt manages environment variables.
From the Nuxt 3 documentation: https://nuxt.com/docs/guide/directory-structure/env

After your server is built, you are responsible for setting environment variables when you run the server.
Your .env file will not be read at this point. How you do this is different for every environment.

I answered here, you can view it here.
https://github.com/sidebase/nuxt-auth/issues/515#issuecomment-2116360307

I set the baseURL instead of origin in auth option on nuxt.config.ts
setup AUTH_ORIGIN=http://localhost:3000 in .env
setup NEXTAUTH_SECRET=<key> in .env
setup process.env.NEXTAUTH_SECRET in NuxtAuthHandler's against secret key
development run is ok, but while running the build, getting the error

nuxt] [request error] [unhandled] [500] AUTH_NO_SECRET: No `secret` - this is an error in production, see https://sidebase.io/nuxt-auth/resources/errors. You can ignore this during development
  at NuxtAuthHandler (./.output/server/chunks/_..._.mjs:40:13)
  at ./.output/server/chunks/_..._.mjs:134:15
  at ModuleJob.run (node:internal/modules/esm/module_job:217:25)
  at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)
  at async Object.handler (./.output/server/chunks/nitro/node-server.mjs:3078:19)
  at async Server.toNodeHandle (./.output/server/chunks/nitro/node-server.mjs:3267:7)


Update README.md

I have added a description to the README.md that is accessible to non-developers, hopefully clarifying jargon. Do you think having this explanation is of value to this project?
(--Assignment for Learn Modern Developer)

Thanks but no thanks. We want to leave the README as-is.
I think what we have on http://underscorejs.org/ is good enough. If you think there's something to improve, please suggest changes to index.html (your metaphor is a bit much though).

Thanks for the feedback regarding the metaphor.


Phantom device left after deleted in CCU

Language: english/DEUTSCH
custom_component version:
1.30.2
Home Assistant:

Version: 2023.2.3
Installation method (HAOS, supervised, core, venv, Docker): HAOS

CCU:

Model: CCU3
OS: original
Version: 3.67.10

Problem-relevant configuration:
Do you use tls? yes
Do you use callback? yes
Do you use username and password? yes
Which interfaces do you use (hmip/bidcos/wired)? hmip/bidcos
Describe the bug
I deleted all HUE devices in the CCU. However, one device remains in HA.
I have double checked whether the device is actually deleted in the CCU (via UI and script) and it is definitely no longer there. How can I delete the phantom device in HA without completely deleting the integration?
Maybe it is safe to edit the HA registry files (core.device_registry / core.entities_registry) or is that a bad idea?
**Screenshots **

Diagnostic Information (available on every device)
{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2023.2.3",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.10.7",
    "docker": true,
    "arch": "x86_64",
    "timezone": "Europe/Berlin",
    "os_name": "Linux",
    "os_version": "5.15.90",
    "supervisor": "2023.01.1",
    "host_os": "Home Assistant OS 9.5",
    "docker_version": "20.10.22",
    "chassis": "embedded",
    "run_as_root": true
  },
  "custom_components": {
    "waste_collection_schedule": {
      "version": "1.34.0",
      "requirements": [
        "icalendar",
        "recurring_ical_events",
        "icalevents",
        "bs4"
      ]
    },
    "frigate": {
      "version": "3.0.0",
      "requirements": []
    },
    "netgear_wax": {
      "version": "0.5.0",
      "requirements": []
    },
    "hacs": {
      "version": "1.30.1",
      "requirements": [
        "aiogithubapi>=22.10.1"
      ]
    },
    "homematicip_local": {
      "version": "1.30.1",
      "requirements": [
        "hahomematic==2023.2.7"
      ]
    }
  },
  "integration_manifest": {
    "domain": "homematicip_local",
    "name": "Homematic(IP) Local",
    "config_flow": true,
    "documentation": "https://github.com/danielperna84/custom_homematic",
    "issue_tracker": "https://github.com/danielperna84/hahomematic/issues",
    "requirements": [
      "hahomematic==2023.2.7"
    ],
    "ssdp": [
      {
        "manufacturer": "EQ3",
        "manufacturerURL": "http://www.homematic.com"
      }
    ],
    "zeroconf": [],
    "homekit": {},
    "dependencies": [],
    "codeowners": [
      "@danielperna84",
      "@SukramJ"
    ],
    "iot_class": "local_push",
    "loggers": [
      "hahomematic"
    ],
    "version": "1.30.1",
    "integration_type": "hub",
    "is_built_in": false
  },
  "data": {
    "config": {
      "entry_id": "83148af4acf1b2f473b65463f25f52e2",
      "version": 1,
      "domain": "homematicip_local",
      "title": "ccu3-webui",
      "data": {
        "instance_name": "ccu3-webui",
        "host": "<IP_ADDRESS>",
        "username": "**REDACTED**",
        "password": "**REDACTED**",
        "tls": true,
        "verify_tls": false,
        "sysvar_scan_enabled": true,
        "sysvar_scan_interval": 30,
        "callback_host": null,
        "callback_port": null,
        "json_port": null,
        "interface": {
          "HmIP-RF": {
            "port": 42010
          },
          "BidCos-RF": {
            "port": 42001
          },
          "VirtualDevices": {
            "port": 49292,
            "path": "/groups"
          }
        }
      },
      "options": {},
      "pref_disable_new_entities": false,
      "pref_disable_polling": false,
      "source": "ssdp",
      "unique_id": "9A49923DC1",
      "disabled_by": null
    },
    "platform_stats": {
      "binary_sensor": 46,
      "button": 4,
      "climate": 6,
      "cover": 3,
      "light": 8,
      "number": 2,
      "select": 2,
      "sensor": 28,
      "switch": 13
    },
    "devices": [
      "HM-LC-Sw2PBU-FM",
      "HM-RCV-50",
      "HM-SCI-3-FM",
      "HM-Sec-MDIR",
      "HMIP-SWDO",
      "HmIP-BDT",
      "HmIP-BROLL",
      "HmIP-BSL",
      "HmIP-CCU3",
      "HmIP-FALMOT-C12",
      "HmIP-FCI6",
      "HmIP-FROLL",
      "HmIP-FSI16",
      "HmIP-FSM",
      "HmIP-PCBS",
      "HmIP-PCBS2",
      "HmIP-RCV-50",
      "HmIP-SLO",
      "HmIP-SMI",
      "HmIP-SMI55",
      "HmIP-SMO-A",
      "HmIP-SPI",
      "HmIP-WRC6",
      "HmIP-WTH-2",
      "HmIP-eTRV-2",
      "VIR-LG-WHITE-DIM"
    ]
  }
}

Delete device dosen't work?
Restart afterwards.
If that doesn't work try delete cache.
Restart afterwards.
If that doesn't work, then the device is not successfully deleted from CCU.

oh shame,
i think i need to read the docu more carefully. I assumed it was a problem because there aren't any buttons in the UI for it.
of course it worked.
Thank you again.


[terra-form-select] Array of numbers for MultiSelect produces proptype warnings

Bug Report
Description

Previously on terra-form-select 5.25.0, given a multiple variant (MultiSelect or TagSelect), we could pass a list of numbers for the value prop:
Select.jsx:100
SelectField.jsx:102
<Select
  variant={'multiple'}
  value={[12, 34, 56, 78]}
>
  {...}
</Select>

When the large refactor happened for terra-form-select 5.26.0, the proptype validation requires value to be an array of strings. MultiSelect.jsx:93
Steps to Reproduce
Create a Select component with a 'multiple' variant
Set the value to an array of numbers
Receive proptype validation warnings
Additional Context / Screenshots

Expected Behavior
This code would not produce a proptype error.
Possible Solution

Update MultiSelect.jsx:93 to accept an array of numbers or strings
Environment

Component Name and Version<EMAIL_ADDRESS>Browser Name and Version: Latest Chrome
Node/npm Version: Node 8, npm 5
Operating System and version (desktop or mobile): Any

@ Mentions
@dev-cprice

Adding a discussion from the Terra support channel with a similar issue.

Hi All, I’m facing issue while consuming terra-form-select 5.27 version. Some validation warnings are coming up in the console, same error screenshot I’ve attached.

I went through the terra documentation where I figured out that Select tag accepts value as an array, string or number and SearchSelect accepts value as string or number only. It looks mismatch in select and SearchSelect value acceptance.

So from our workflow we are sending value as an array to Select and Select sends that array further to SearchSelect which is creating validation error in SearchSelect I think.

The Select previously allowed passing an array for any variant type and had logic to extract the data as necessary for each variant. I do think it is odd / inappropriate to pass an array into a variant type that only allows a single selection, but from a passivity perspective this no longer works.

@StephenEsser

I do think it is odd / inappropriate to pass an array into a variant type that only allows a single selection, but from a passivity perspective this no longer works.

I totally agree. However, does restoring passivity mean that the original bug be reintroduced?
I don't think that it makes sense for single select variants to accept an array for 'selected value', tho. I am not a big fan but one thing that I can think of is to maybe add a custom prop validator in Select to up-front validate the value prop based on the variant and keep the rest of the current code as-is. That is nothing different from the current behavior except for the warning will be seen in Select and not a level below. In addition to that, I'd lean towards saying that the changes that went into v5.25.0 have corrected an existing flaky behavior and is not a breaking change.

This was not a bug prior to the changes made in v5.25.0. The Select had a consistent PropType expectation for all variants.
This worked both ways, the multiple Select variant could also accept a string as a defaultValue.
This avoided a complex prop driven custom PropType validator and allowed consistent behavior and expectations between all variants.
As the Select is being broken out into individual resources I do think the more strict PropTypes are appropriate. I do not think the <SearchSelect /> should accept an array. The issue is being passive with the original <Select /> API.

I do not think the <SearchSelect /> should accept an array. The issue is being passive with the original <Select /> API.

Right. In that case, how do we handle it if the consumers today have been sending down an array for variants that support only single selection in addition to satisfying the condition that those variants should not accept an array? Do we have the liberty to perhaps internally flatten the array or extract the first or the last entry from the entry? I don't think that solves the issue but I am just putting it out there. I am trying to think out loud how the outcome can be passive by satisfying both the conditions.

Well, the original code just did a find if value was a list. That's all we need to do again

@StephenEsser and @dev-cprice, I am sorry the above comments are a bit confusing to me.
Just that we are all on the same page, do we want to keep the current Search API passive (meaning that the <Select /> accepts number/array/string for the value prop) and also make sure that  Single Selection variants don't accept an array and don't throw up when they do? If yes, then I am not sure about how to satisfy both the constraints together.
If not, then does it mean that we will let Single Selection variants to accept an array (like they previously did) to keep the functionality passive with what they are doing today? Maybe it is okay to introduce a custom prop validator for <Select /> when we do MVB in the future?

My preference is let's allow the previous behavior (let Single Selection variants to accept an array) to keep the functionality passive. When we do the next MVB for this package, we can reconsider how we want that to work.

I can get a PR open for this real quick

PR to fix prop-type passivity for all variants: https://github.com/cerner/terra-core/pull/2646


Roll Engine from 7f00e2f6c53c to be8839f59e1c (3 revisions)

https://github.com/flutter/engine/compare/7f00e2f6c53c...be8839f59e1c
2021-01-07<EMAIL_ADDRESS>[macos] Re-land FlutterOpenGLRenderer isolation (flutter/engine#22607)
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 5jKxFxRiQ... to qedaMC5nT... (flutter/engine#23484)
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from de_114mvQ... to cLR0CUb-e... (flutter/engine#23482)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 5jKxFxRiQQBZ to qedaMC5nTQFw
fuchsia/sdk/core/mac-amd64 from de_114mvQcHR to cLR0CUb-eK5h
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.

Commit queue failed; closing this roll.


⚠ Bofrost.nl has degraded performance

In 62f6cc3, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1900 ms

Resolved: Bofrost.nl performance has improved in 2d98f01 after 9 minutes.


Solve training performance

done with batch size = 16 per device with 2 devices
Performance seems to be weak when doing the task 1 and task 2 at the same time. I will try to find out why

Increased training epochs , lr and batch size (jupyter notebook containing that training not yet uploaded, it is in google VM)


Updated with a new way of loading resources from project folders

This work is needed when MRTK is split into separate NuGet packages and may contain assets split across those package folders.

/azp run mrtk_docs


[Issue] Feedback Master Fader Gain does not work properly

On v3.0.0+5764. Once set, the Feedback does not recognize the current value of the Master Fader. For example. When configuring Feedback for level -10 while Master Fader Gain (MFG) is -10 results in green background (works OK). But when the MFG level is adjusted the background remains green (not OK).
When configuring Feedback for level -10 while Master Fader Gain (MFG) is for example -15 it does NOT results in green background (works OK). But when the MFG level is set to -10 the background is NOT set to green (not OK).
Have seen a similar thing in the past for Hyperdeck and then the RCA was "not patched".
Thanks. Regards, Nico Wasmus

Many thanks!!
Op di 23 jan 2024 om 21:32 schreef Julian Waller @.***>

Closed #237
https://github.com/bitfocus/companion-module-bmd-atem/issues/237 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/bitfocus/companion-module-bmd-atem/issues/237#event-11575800414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A25NECFIXEKFVJIVWOVRTHDYQAM7DAVCNFSM6AAAAAAWVQ3XJWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGU3TKOBQGA2DCNA
.
You are receiving this because you authored the thread.Message ID:
<bitfocus/companion-module-bmd-atem/issue/237/issue_event/11575800414@
github.com>


problems working in mobile

Challenge Name
Use a CSS Class to Style an Element
(could happen with others, but only tested this one so far)
Issue Description
kept saying one of the checks was wrong (the text was not showing up red), after several checks it was all correct so i went to the next challenge which was acting odd so i switched to desktop mode, it completed that one then took me to the previous one listed above and was fixed.
Browser Information

Browser Name, Version: google search browser
Operating System: android
Mobile, Desktop, or Tablet: mobile

Hi @vyyres
Thanks for the report, yes this has come up many times especially with mobile browsers. Can you check if the space characters are not the culprit, usually deleting the spaces in the CSS part of your code and adding them again works.
This is weird how mobile browsers are interpreting spaces sometimes.

I think we should have a different strategy for mobile devices completely. The current challenge layout is not very user-friendly.
Note: We recommend using a desktop over mobile because UX of coding on a desktop is much simpler than on a tiny screen. However, we do understand that mobile devices support can be enhanced much more than it currently is.
Suggestions welcome.

i did play with the spacing in the css lines and that does fix it, with this particular problem it was the spaces it puts in before the color: red; line

@vyyres thanks, for confirming.

this issue already fix in beta.  code editor will ignore fake space.  no need to keep this issue open. thanks.
https://github.com/freeCodeCamp/freeCodeCamp/pull/11472


Some way to get, stop, and remove all the reporters.

I'm trying to run some integration tests, and want metrics to be cleanly stopped and then later restarted.  I've not found any way to stop the various reporters other than by keeping track of them when I add them.  Do you have any ideas?

Too old.


Is there a way to make the maps show in offline?

I tried WinUI, WPF samples. Why doesn't it work offline? or How to do it?

You would need to data locally. For an entire world map this is a huge amount of data. For some scenarios it can be sufficient to work with a small subset. In that case mbtiles could be a solution, but you would need to create the mbtiles file yourselves somehow.
For a sample start Mapsui.Samples.Wpf select the 'Data' category and select the 'MbTiles' sample.

There is a way to display OpenAndroMaps (Mapsforge binary) with MapSui?

There is a way to display OpenAndroMaps (Mapsforge binary) with Mapsui?

No, I have not heard about this format before.

https://github.com/bertt/mapsforge-cs looks like at least there're tools to read that format already, so the rest should be easy

I would have a look at this project
https://github.com/charlenni/VectorTileLayers

@Bondarenko1990 The question isn't complete.
I assume, that we always talk about tile maps (don't know if nowadays exist other maps). There are different types of maps: bitmap and vector maps. The first consist of normally 256 x 256 pixel ready rendered bitmaps, which represents a part of a map in a given zoomlevel. The second consist of a set of vector features, which are rendered on the fly with a given style set. This set of vector features could be the same for all zoom levels or different for each zoom level and the style set decide, which feature is shown in which zoom level.
The other thing is the online and offline access to these tiles. Both types of maps (bitmap and vector) could be used online and offline.
Mapsui can handle only bitmap tiles, but this online and offline. To use a map offline, you have to create a database file, which contains all the bitmaps for the tiles, that you want to use in your app.
Mapsforge is a vector format and a map control library written in Java. It is the most common open source map control library for Android. Its offline format is a SQLite database and it uses an own style set format.
@yapus I would say, 10 % of the work for a vector map is needed to read the format, 90 % to bring it on the screen. Each tile contains around 5000 features (lines, polygons and points) that must be drawn in the right style in the right layer. The most difficult part is then to show text and icons on the map. And all this in under 25 ms. It isn't trivial.

@charlenni sorry, my assumptions might be too naive, but i've used AliFlux.VectorTileRenderer combined with Mapbox.VectorTile abstractions to use MapsUI with offline mbtiles maps, and without digging deeper into differences with mapforge i can't say if you're right about 10-to-90% split here...

@charlenni sorry, my assumptions might be too naive, but i've successfully used AliFlux.VectorTileRenderer combined with Mapbox.VectorTile abstractions to use MapsUI with offline mbtiles maps, and without digging deeper into differences with mapforge i can't say if you're right about 10-to-90% split here...

Might be that you could use some of my improvements? I made my version of AliFlux.VectorTileRenderer cross platform. It also doesn't convert the PNG byes till it hands it off to the renderer, so I think it is marginally faster.

The solution is not fast though, it prepares rasterized images in the background as you request them and store them in local cache folder.
And yes, you're right, some texts sometimes look glitchy when they are split over several tiles. I thought it's something I can solve with tweaking some settings, but didn't find which yet

I have a POC Esri ArgGis TPK Compact Cache version 1.01 renderer working too. I will try to get that open sourced if I can. We needed it internally, might be useful to others.

@memsom Yes, thank you, i just checked and appears your fork is what i'm using now

@yapus No reason to say sorry.
The problem with the text is, that text could be bigger than the tile itself. If this is the case, then the text is cut off at the border. There are two possible solutions for this:

You render the text not only for the tile you want to have, but also for the 8 tiles around (or even more)
You render text independent from the tiles
Both isn‘t, if I know it correct, implemented in AliFlux.

Another thing is, that AliFlux creates bitmap tiles. So you get all the disadvantages of them (bad resolution when zooming in).
Beside this it is a great project.

AliFlux was the only option i found at the time of doing this research. If anyone can recommend any other viable solution which lacks the mentioned problems - that would be awesome.

@memson: First of all thank you for your work.
Is there a special reason why you changed the namespace to AliFlex instead of AliFlux?
Second you add GlobalMercator with an own license. Is the license still MIT for the whole project?

@memson: First of all thank you for your work.
Is there a special reason why you changed the namespace to AliFlex instead of AliFlux?

Might have been a typo that propagated when I changed the code. Not sure. Wasn't intentional.
I had to change quite a lot internally to make it work nicely with Xamarin. I have a version we use internally that uses the V4.x API too. It needs more work to make it releasable though.

Second you add GlobalMercator with an own license. Is the license still MIT for the whole project?

I think that comes from mapbox? I can look.
I'm thinking at some point to take the Skia rendering and work out how to render directly to a Mapsui layer. If we do that, we can render text correctly by applying it after all the Lowe layers. Nothing started? But I think it might work.

@memson I do it this way in my project VectorTileLayers. But it isn‘t as fast as drawing a bitmap of the tile. It needs around 35 ms to draw a vector map with text. It has still some main issues: text collision of rotated text and text on paths are missing. And the API is more complicated than that of AliFlux.
One workaround for AliFlux to get real vector maps could be to draw not on a bitmap canvas but to a SKPictureRecorder. It will slow down drawing, but will replace bitmap resizing with vector resizing

I’m sorry folks but you have mis-tagged me! @memson I am not part of this project.
Best regards,
Michelle
From: charlenni @.>
Date: Sunday, December 4, 2022 at 5:47 AM
To: Mapsui/Mapsui @.>
Cc: Michelle Emson @.>, Mention @.>
Subject: Re: [Mapsui/Mapsui] Is there a way to make the maps show in offline? (Issue #1726)
@memsonhttps://github.com/memson I do it this way in my project VectorTileLayershttps://github.com/charlenni/VectorTileLayers. But it isn‘t as fast as drawing a bitmap of the tile. It needs around 35 ms to draw a vector map with text. It has still some main issues: text collision of rotated text and text on paths are missing. And the API is more complicated than that of AliFlux.
One workaround for AliFlux to get real vector maps could be to draw not on a bitmap canvas but to a SKPictureRecorder. It will slow down drawing, but will replace bitmap resizing with vector resizing
—
Reply to this email directly, view it on GitHubhttps://github.com/Mapsui/Mapsui/issues/1726#issuecomment-1336376345, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQUGC37DCRDGUVIXUSNKKLWLRZE5ANCNFSM6AAAAAARASW3W4.
You are receiving this because you were mentioned.Message ID: @.***>

@memson You are right. Sorry for disturbing. Was for @memsom 😉


Submit to Crowdin to help with translations?

https://crowdin.com/page/open-source-project-setup-request
Just like the way SpongeDocs do.
Also, we, CFPAOrg are willing to translate and making progress on advertising CraftTweaker script development.

Or you will just be filling the form and let me maintain registered Crowdin project 🤔

Hello NanamiArihara,
I do not know much about Crowdin but from what I've seen so far do not know if it is suitable for this documentation project:
We are using ReadTheDocs (RTD) to assemble the mkdocs project and display it.
Unfortunately, RTD expects one branch/repository per language, and not one branch containing all translations.
I have not found a way of setting RTD to use one branch (master) as translation source and export the built translations to several branches (master_zh-CN, master_de or translation_de,...).
Do you know if that is possible to do using crowdin?
Otherwise we'd have to change the whole project setup (and maybe even need to host the wiki ourselves).
Also, feel free to ping me up/DM me on discord (@kindlich#9060).

@kindlich Never submit to Crowdin. Fight spying and tracking where it exists. ;)
Weblate is a libre software alternative that has actually gratis hosting for libre software, as opposed to only allowing non-commercial libre software projects apply.

@comradekingu
Thank you for your suggestion.
We are already using Crowdin at the moment.
Truth be told, some of its integrations aren't the way we envisioned them, but it works out more or less.
To be frank, I care less about where the documentation pages are translated than being able to translate them at all and having a way for our CI to create the docs pages in the different languages.
Switching would mean spending time on getting to know a new platform for changing something that is (more or less) working atm. Also, it would mean that the current translators need to switch platforms as well.
In any way, feel free to send a PM on Discord or another medium of your choice and we can discuss this further.
My Discord tag is kindlich#9060

Crowdin has been supported for a long time.


remove fabric8-forker jobs

related issue: https://gitlab.cee.redhat.com/dtsd/housekeeping/issues/56
Forker is not being used and should be decomissioned from preview/prod altogether for now
devtools-fabric8-forker-build-master
devtools-fabric8-forker

Can one of the admins verify this patch?

we will need to remove the jobs from ci.centos.org as well ( which is a manual process.. )

[test]


TestBusTLS test failures

We see, when running under -race, test failures from TestBusTLS.    (Note that StarTest does not use TLS.)

  I think the situation with TLS is strictly related to timing -- fundamentally the test is busted, clearly.

I've improved the debugging, and have confirmed that indeed, the problem is the depth of the queues -- basically the framework was occasionally dropping packets.  The "dup" was a misleading message.  Cranking up the queues to 128 solved the problem.  As a little bonus, the throughput tests are yielding better performance as well.
Btw, IPC is faster with small packets than TCP, but once the packet size increases it tanks ... I this is because the MTU for IPC is tiny.


[do not merge] Dev four

Hi Desoga! What changes do you want to make? :) Your PR is a little weird again, Sinai can’t review it or merge it.

Hi Desoga! What changes do you want to make? :) Your PR is a little weird again, so I can’t review or merge it.

Hi Desoga! What changes do you want to make? :) Your PR is a little weird again, so I can’t review or merge it.

Hi Desoga! What changes do you want to make? :) Your PR is a little weird again, so I can’t review or merge it.

I set the status as draft. Was testing something out, didn't know the PR was still made.


Syntax color problem for a comment inside a complex Flow type

I've just seen a similar problem to #13, but this time with ' inside a comment, and the comment itself inside some complex Flow typing:
export type GeckoSamples = {|
  schema:
    | {|
        stack: 0,
        time: 1,
        responsiveness: 2,
      |}
    | {|
        stack: 0,
        time: 1,
        eventDelay: 2,
      |},
  data: Array<
    [
      null | IndexIntoGeckoStackTable,
      Milliseconds, // since profile.meta.startTime
      // milliseconds since the last event was processed in this
      // thread's event loop at the time that the sample was taken
      Milliseconds
    ]
  >,
|};

Gives

I think the comment itself isn't found as a comment, and that's the real source of the problem here; and this is arguably a complex piece of syntax here :-)

After this error, all the syntax in the rest of the file is broken, because the type generic is never closed.
The full file is at https://github.com/firefox-devtools/profiler/blob/ada7782f25ba67c72d5fdd525d2430ab6118f2df/src/types/gecko-profile.js if you're curious.

@julienw Both #13 and #14 should have been fixed now. Please verify it.


todense() on jax sparse COO returning only zeros

Description
I was testing the jax.experimental.sparse library and found that todense() method on the COO sparse matrix is returning a vector of zeros.
Code to reproduce (https://colab.research.google.com/drive/1_xxQEqEl5c9NMvYYe8zKsqOS6LJ_V_9P?usp=sharing):
coo_shape = (6,1)
python_coo = {1:10, 3:20, 5:20}

# to array
py_indices = list(python_coo.keys())
py_data = list(python_coo.values())

indices = jnp.array([py_indices, [1]*len(py_indices)]).T
data = jnp.array(py_data, dtype=jnp.float32)

vec_coo = sparse.BCOO((data, indices), shape=coo_shape)

vec_coo.todense() # outputs a vector full of zeros

Array([[0.],
       [0.],
       [0.],
       [0.],
       [0.],
       [0.]], dtype=float32)

Currently, I am using a workaround with jax.lax.scatter:
def vec_coo_todense_wscatter(coo):
  indices = coo.indices[:,0]

  dnums = jax.lax.ScatterDimensionNumbers(update_window_dims=(), inserted_window_dims=(0,), scatter_dims_to_operand_dims=(0,))
  return jax.lax.scatter(jnp.zeros((coo.shape[0],)), jnp.expand_dims(indices, axis=-1), coo.data, dimension_numbers=dnums)

# returns a dense vector with the correct data
vec_coo_todense_wscatter(vec_coo)

Array([ 0., 10.,  0., 20.,  0., 20.], dtype=float32)

What jax/jaxlib version are you using?
v0.4.16
Which accelerator(s) are you using?
Tested on CPU and GPU
Additional system info
No response
NVIDIA GPU info
No response

Update:
The "bug" that I was experience was related to my indices and the shape of the matrix i believe:
indices = jnp.array([py_indices, [1]*len(py_indices)]).T
shape = (6,1)

As you can see I am wrongly setting the column indices to "1" instead of "0".
Should this be impossible? I am trying the index column 1, but according to the shape thats impossible. This should thrown an error?

I think you meant to use this:
indices = jnp.array([py_indices, [0]*len(py_indices)]).T

For an axis of size 1, an index value 1 is out-of-bounds (indexing in Python is generally zero-based), and BCOO uses out-of-bound indexing to indicate padded values that don't affect the array.

Exactly, @jakevdp. As mentioned above, I realized that later. However, shouldn't jax have thrown an error by validating out-of-bounds?

As I mentioned, JAX's BCOO matrix uses out-of-bound indices to mark padded values in its representation, so the data you passed is valid, it's just that it represents a different matrix than you intended it to represent.

Ty for the explanation :) Really appreciated the link.
Closing the issue


Upgrade Jackson 2.16.1 -> 2.16.2

This PR contains the following updates:

Package
Type
Update
Change

Jackson
import
patch
2.16.1 -> 2.16.2

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Jackson 2.16.1 -> 2.16.2

See https://github.com/FasterXML/jackson-bom/compare/jackson-bom-2.16.1...jackson-bom-2.16.2


package recipes not use version attribute

Cookbook version
2.1.1
Chef-client version
12.10.24
Platform Details
Ubuntu 14.04
Scenario:
"run_list": [
	"recipe[apt]",
	"recipe[php]"

],
"php": {
	"install_method": "package",
	"version": "5.6.25",
	"fpm_package": "php5.6-fpm",
	"fpm_service": "php5.6-fpm",
	"fpm_socket": "/run/php/php5.6-fpm.sock",
	"fpm_pooldir": "/etc/php/5.6/fpm/pool.d",
	"fpm_default_conf": "/etc/php/5.6/fpm/pool.d/www.conf",
	"conf_dir": "/etc/php/5.6/cli",
	"ext_conf_dir": "/etc/php/5.6/mods-available",
	"enable_mod": "/usr/sbin/phpenmod",
	"disable_mod": "/usr/sbin/phpdismod",
	"curl": {
		"package": "php5.6-curl"
	},
	"gd": {
		"package": "php5.6-gd"
	},
	"ldap": {
		"package": "php5.6-ldap"
	},
	"pgsql": {
		"package": "php5.6-pgsql"
	},
	"sqlite3": {
		"package": "php5.6-sqlite3"
	},
	"packages": [
		"php5.6-common",
		"php5.6-dev",
		"php5.6-fpm",
		"php5.6-cli",
		"php-pear"
	]
},

Steps to Reproduce:
Run chef-solo
Expected Result:
Chef installed php5.6.25
Actual Result:
Chef installed last package in repo
Why not use version attribute for package install ( https://docs.chef.io/resource_package.html ) ?
Actually problem for newer cookbook.

up?

Because each distro has wildly different support so we merely install latest - if you want a custom version installing from source is the best way or writing your own custom recipe to do the installation part.

I'm closing this out as a won't fix. The version attribute is for the source installs. As @cheeseplus mentioned this introduced a level of complexity due to distro mess that we just don't want to touch. If you need something to control particular package installs I'd suggest a simple cookbook that handles exactly what you need on just the distro you need.


Get handlesincorrectly Controller injection and life cycle

I think there is something wrong on how Get handles Controller injection and life cycle
Describe the bug
I think it is more easy to describe this with an example:
We have 3 screens, the home screen, the second screen and the third screen.
When the user navigates from homescreen to second screen, a SecondScreenController is put( Get.put(GenericController(timeStamp)) ).
The generic controller receives a timestamp ( with value 1), and then there is a widget that displays that particular timestamp (aka 1).
From the second screen there is a button to go to third screen, this is just a simple widget screen, and on the third screen there is a button to go to second screen and remove all previous screens with Get.offAllNamed('/second');
Now the Second screen is instantiated, the screen displays correctly the new timestamp (2) but then instead of closing the controller that was generated, the one with timestamp 1, it will close controller with timestamp 2.
**Reproduction code
I have a simple flutter app, that only has get: 4.6.5. I have given an example of the code below
example:
import 'package:flutter/material.dart';
import 'package:get/get.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      getPages: getPagesList,
      initialRoute: '/',
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text("Homepage"),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text('Current page:'),
            Text('Homepage', style: Theme.of(context).textTheme.headline4),
            ElevatedButton(
              child: const Text('Go to Second page'),
              onPressed: () {
                Get.toNamed('/second');
              },
            ),
          ],
        ),
      ),
    );
  }
}

class SecondScreen extends StatelessWidget {
  final timeStamp = DateTime.now().millisecondsSinceEpoch;
  SecondScreen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final GenericController controller = Get.put(GenericController(timeStamp));
    return Scaffold(
      appBar: AppBar(
        title: const Text("SecondScreen"),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text('Current page:'),
            Obx(() => Text('SecondScreen ${controller.timeStamp}')),
            ElevatedButton(
              child: const Text('Go to Third page'),
              onPressed: () {
                Get.toNamed('/third');
              },
            ),
          ],
        ),
      ),
    );
  }
}

class ThirdScreen extends StatelessWidget {
  const ThirdScreen({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text("ThirdScreen"),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text('Current page:'),
            const Text('ThirdScreen'),
            ElevatedButton(
              child: const Text('Go to Second page'),
              onPressed: () {
                Get.offAllNamed('/second');
              },
            ),
          ],
        ),
      ),
    );
  }
}

class GenericController extends GetxController {
  final RxInt timeStamp;

  GenericController(int newTime) : timeStamp = newTime.obs;

  @override
  void onClose() {
    print("Closing Controller $timeStamp");
    super.onClose();
  }
}

final List<GetPage> getPagesList = [
  GetPage(
    name: '/',
    page: () => const MyHomePage(),
  ),
  GetPage(
    name: '/second',
    page: () => SecondScreen(),
  ),
  GetPage(
    name: '/third',
    page: () => ThirdScreen(),
  ),
];

To Reproduce
Steps to reproduce the behavior:

Go to 'Second screen' by clicking the 'Go to second screen' button and notice the time stamp
Click on ''Go to third screen'
Click on ''Go to second screen'
Notice the timestamp displayed on screen
Notice that in the logs, the same timestamp is printed as being closed

Expected behavior
It should close the first SecondScreenController not the one you are currently using
Flutter Version:
3.0.3
Getx Version:
4.6.5

I just checked the pull requests, and I am not 100% sure but I think this pull request is related to my issue: https://github.com/jonataslaw/getx/pull/2421


Get all BikePoint Locations within a given "bounding box"

TFL define a bounding box as a box with it's lat/lon of its north-west and south-east corners as the edges of the box.
i.e. if there were 10 BikePoints within the bounding box area, when making this call I would expect to find 10 BikePoints returned
End Point: /BikePoint?swLat=<sw_latitude>&swLon=<sw_longitude>&neLat=<ne_latitude>&neLon=<ne_longitude>

Added in commit: cf6fe79daf88c71b3d84e103a70c2ee5baa22926


More verbose documentation for KZGSettings struct.

Minor documentation-only PR. This addresses item 7 in https://hackmd.io/o-YnUZPbS2G-cyFSCZ0uiw?view

Going to let @asn-d6 review/merge this one.

Great docs! Thanks Gotti!


Failed to activate the atom-clock package

[Enter steps to reproduce:]

...
...

Atom: 1.14.3 x64
Electron: 1.3.13
OS: Mac OS X 10.12.2
Thrown From: atom-clock package 0.1.5
Stack Trace
Failed to activate the atom-clock package
At Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node.

Error: Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node.
    at Error (native)
    at /app.asar/node_modules/status-bar/lib/status-bar-view.js:93:23)
    at AtomClockView.drawElement (/packages/atom-clock/lib/atom-clock-view.js:54:20)
    at AtomClockView.start (/packages/atom-clock/lib/atom-clock-view.js:13:10)
    at Object.consumeStatusBar (/packages/atom-clock/lib/atom-clock.js:45:24)
    at Provider.module.exports.Provider.provide (/app.asar/node_modules/service-hub/lib/provider.js:29:52)
    at ServiceHub.module.exports.ServiceHub.consume (/app.asar/node_modules/service-hub/lib/service-hub.js:50:18)
    at Package.module.exports.Package.activateServices (/app.asar/src/package.js:404:75)
    at Package.module.exports.Package.activateNow (/app.asar/src/package.js:221:16)
    at /app.asar/src/package.js:190:32
    at Package.module.exports.Package.measure (/app.asar/src/package.js:96:15)
    at /app.asar/src/package.js:183:26
    at Package.module.exports.Package.activate (/app.asar/src/package.js:180:34)
    at PackageManager.module.exports.PackageManager.activatePackage (/app.asar/src/package-manager.js:550:34)
    at /app.asar/node_modules/settings-view/lib/package-manager.js:439:29
    at exit (/app.asar/node_modules/settings-view/lib/package-manager.js:69:16)
    at triggerExitCallback (/app.asar/src/buffered-process.js:303:11)
    at /app.asar/src/buffered-process.js:333:11)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)

Commands
     -5:23 autocomplete-plus:confirm (input.hidden-input)
  7x -5:22.1.0 core:backspace (input.hidden-input)
     -5:20 editor:newline (input.hidden-input)
  2x -5:17.6.0 core:backspace (input.hidden-input)
     -5:13.3.0 core:move-down (input.hidden-input)
     -5:13.1.0 autocomplete-plus:confirm (input.hidden-input)
     -5:13.1.0 snippets:expand (atom-text-editor.editor.is-focused.autocomplete-active)
 11x -5:10.5.0 core:move-right (input.hidden-input)
 38x -5:07.8.0 core:backspace (input.hidden-input)
     -3:29.8.0 core:confirm (input.hidden-input)
 10x -1:47 core:backspace (input.hidden-input)
     -1:41.7.0 core:confirm (input.hidden-input)

Non-Core Packages
ariake-dark-syntax 0.1.3
atom-clock 0.1.5
atom-material-syntax 1.0.2
atom-material-ui 1.3.9
genesis-ui 0.5.0
solarized-dark-ui 0.3.2
solarized-seti-ui 0.8.6

Thanks for notifying me this. I'll look into it as soon as possible!

I am not able to reproduce this bug. Is it still active? According to the stack trace, something seems to be wrong in the status-bar package.
Have you tried starting Atom in safe mode and checking if the package activation still works?

It seems like this issue is becoming fairly common lately in most atom packages including atom itself. Here is an issue where someone had the same sort of issue..
Try disabling your linter and restart atom and see if this fixes the problem!

So, the problem is actually related to the status-bar package itself? I don't use the linter package, maybe that's the reason I was not able to reproduce the issue.

my work space is messed right now  I
'm overwhelmed

my work space is messed right now  I
'm overwhelmed


JBDS-3618 Progress does not reflect actual amount of work done

Not needed after https://github.com/redhat-developer-tooling/developer-platform-install/pull/297 is released.


Changed 'principle' to 'principal' in readme.md

because it's the correct word to use in this situation ;-)

Indeed it is 👍🏾, thanks for the PR!


Can't claim reward 2.1.7

Having a bit of an issue on a server I'm running with a few friends, we've got Spongeforge 2838-7.4.2 I've set up the SQL and have 2 other mods working fine (WT and Bingo) however for some reason, we can't claim the dex completion rewards, /dexr just brings up the interface, even though we're being pestered by the mod that we have a reward level we can claim.
Many thanks

same


Fix pipeless text indentation detection – less intrusive version

This PR has the same effect as #13, but is less intrusive. It allows inconsistent indentation that are disabled in #13, thereby preserving the original behavior.

Nice work! I really like the new error message :smile:
One thing I was wondering: why do test/cases/comments-in-case.jade and test/cases/nesting.jade
no longer fail?

One thing I was wondering: why do test/cases/comments-in-case.jade and test/cases/nesting.jade
no longer fail?

That's the whole point of this PR :laughing:
#13 fixes the original issue by enforcing a strict indentation width for the entire file, and if an inconsistency is encountered in pipeless text the entire file is relexed using a new indentation width. Because the indentation width is fixed and enforced strictly, comments-in-case and nesting fail. Backward compatibility therefore is intentionally broken.
This PR doesn't change the main indentation handling for the entire file, and only errors out when a clear and irreconcilable inconsistency is reached. If inconsistency is encountered in pipeless text, only the pipeless text is relexed with a new indentation width. This means that very sloppy code like:
html
    body
     p.
         Hey
       there!

continue to be supported.
The reason why I did this PR is because of the fact that it is apparent that inconsistent indentations were intentionally supported before, with a test case for it and indentation state being a stack rather than a simple integer. I don't want to break a case that doesn't take too much to maintain.

Cool - I prefer this PR over #13.
Like I said, from where I stand the implementation looks fine, but maybe @ForbesLindesay could take a look as well?


Memory leak due to gcc bug in v2.5.0

Environment
toml++ version and/or commit hash:  2.5.0
Compiler:  gcc 7.3.1 (but I believe all versions including the latest as of writing, 11.2)
C++ standard mode:  17
Target arch:  x64
Library configuration overrides: TOML_EXCEPTIONS is 1
Relevant compilation flags:  None
Describe the bug
There's a gcc bug that causes a memory leak when an exception is thrown by parse_value() here, specifically because the destructor of the moved-to key is not called.
It looks like you've done a significant refactor for toml++ v3, so feel free to withdraw this if it's no longer an issue going forward.
Additional information
We found this using AddressSanitizer. Does it make sense to incorporate this tool into your tests and CI?
Nice library! Looking forward to v3. 😃👍

Thanks for the report!

It looks like you've done a significant refactor for toml++ v3, so feel free to withdraw this if it's no longer an issue going forward.

The parser has seen a significant refactor but I suspect this would still occur. Should be easy to fix though.

We found this using AddressSanitizer. Does it make sense to incorporate this tool into your tests and CI?

Yup. Would also like to incorporate fuzzing at some point too.

Nice library! Looking forward to v3

Thanks!

Also, damn. That GCC bug is pretty nasty. Kinda defeats the purpose of RAII.

Yeah, it's pretty bad!

I suspect this would still occur.

As far as I know (and maybe you already knew this), the bug only affects aggregate initialization (parsed_key_value_pair in v2.5.0 is an example of an aggregate), so it's possible v3 might not have issues?

Oh, in that case it might be sorted in V3 after all. I changed the way strings are stored during parsing to eliminate some unnecessary copies, and in doing so removed a few of those helper aggregates. Next time I do toml++ work I'll have a look to confirm.

@ronalabraham Have just confirmed that it's fixed in v3. Thanks for letting me know about this! I've added ASan to the CI pipeline so hopefully I catch any regressions going forward.


refactor(cmark_base): replace List with Array

Pull Request Test Coverage Report for Build 98
Details

26 of 48 (54.17%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 74.125%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/cmark_base/runs.mbt
7
8
87.5%

src/cmark_base/raw_html.mbt
4
25
16.0%

Files with Coverage Reduction
New Missed Lines
%

src/cmark/block_struct.mbt
1
80.0%

Totals

Change from base Build 94:
-0.07%

Covered Lines:
1398

Relevant Lines:
1886

💛  - Coveralls


changed dependencies to use firestore 1.0.0

I fixed all analyzer errors except this one :
 info • This class (or a class that this class inherits from) is marked as '@immutable', but one or more of its instance fields aren't
          final: Mock._givenName, Mock._givenHashCode, Mock._defaultResponse • lib/src/mock_collection_reference.dart:16:7 •
          must_be_immutable

The issue is about MockCollectionReference class which extends MockQuery from mockito package, with field not final.
And the MockCollectionReference class implements CollectionReference from cloud_firestore package  which is now @immutable.

Thank you for updating the dependencies and cleaning up the warnings @Jouby!
We can't remove implements CollectionReference because the whole point of the class is to behave like a CollectionReference.
The warning is due to Mock's fields, which are not final. There are two ways to do it:

The quick and easy way, which also is the recommended way, is to add an annotation to ignore the error. See https://github.com/dart-lang/mockito/issues/200#issuecomment-511601012.
The more tedious way is to stop depending on Mock. Since we fake everything, we don't actually use Mock's mocking functionality anymore. But this is a larger change. It requires stubbing 6 methods on MockQuery. If we were to do it, it would be better to do it in another PR. See #33.

The 6 methods we would need to stub to do 2.:

So for this PR, would you mind adding the ignore annotation, as well as a comment to remind us to clean it up later, when Mockito fixes the issue? Something like this:
// Required until https://github.com/dart-lang/mockito/issues/200 is fixed.
// ignore: must_be_immutable

Thank for sharing the Mockito issue link.
I have updated my PR by adding ignore annotation and a comment with mockito issue link. 👍

I just published your update to https://pub.dev/packages/cloud_firestore_mocks/changelog#070. Thanks again!


Unsupported option warning when building with linked dramsim2

Unsupported option, but it's necessary to build and link... Probably it's getting passed to runtime when it's needed only at compile time, or vice versa.
Warning-[RT_UO] Unsupported option
Unsupported option '-LDFLAGS' is ignored
Warning-[RT_UO] Unsupported option
Unsupported option
'-L/home/petrisko/bitbucket/pre-alpha-release/external/lib -ldramsim
-Wl,-rpath=/home/petrisko/bitbucket/pre-alpha-release/external/lib' is
ignored

Similarly, when linting in VCS:
cc1: warning: command line option '-std=c++11' is valid for C++/ObjC++ but not for C [enabled by default]

Fixed