Saved pokedex not loading

I started a pokedex and got about 80 pokemon selected. When I went to reopen it, it would not load. I even left it overnight just in case it was taking a while on the server, but nope. I don't want to have to refill in my pokemon each time I log in; that's most the reason I chose to try this app. Please help with the solution. Thank you!

what's the link of your dex?
without that I cannot help

https://supereffective.gg/apps/livingdex/dQllXWUDQFouodn8ioUh
Thank you
On Sun, Feb 12, 2023 at 11:00 AM Javi Aguilar @.***>
wrote:

what's the link of your dex?
without that I cannot help
—
Reply to this email directly, view it on GitHub
https://github.com/itsjavi/supereffective.gg/issues/18#issuecomment-1427067559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53HP7O4Z37H3VWXJY2NH3LWXECJLANCNFSM6AAAAAAUZL4AY4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I can see it, maybe is your browser or your internet connection.
What browser are you using? (browser name, version and operative system)

Chrome, Version 96.0.4664.110, Windows 11
On Sun, Feb 12, 2023 at 11:37 AM Javi Aguilar @.***>
wrote:

I can see it, maybe is your browser or your internet connection.
What browser are you using? (browser name, version and operative system)
[image: image]
https://user-images.githubusercontent.com/122741/218324112-b1c5d071-da99-4fb4-a808-1a437d657d19.png
—
Reply to this email directly, view it on GitHub
https://github.com/itsjavi/supereffective.gg/issues/18#issuecomment-1427075400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53HP7MTBA6IH4QDLNVEJT3WXEGWJANCNFSM6AAAAAAUZL4AY4
.
You are receiving this because you authored the thread.Message ID:
@.***>

I can see that the file is there on the screen that shows your different
dexes, it just doesn't load for me.
On Sun, Feb 12, 2023 at 11:33 AM Kenneth Harris @.***>
wrote:

Chrome, Version 96.0.4664.110, Windows 11
On Sun, Feb 12, 2023 at 11:37 AM Javi Aguilar @.***>
wrote:

I can see it, maybe is your browser or your internet connection.
What browser are you using? (browser name, version and operative system)
[image: image]
https://user-images.githubusercontent.com/122741/218324112-b1c5d071-da99-4fb4-a808-1a437d657d19.png
—
Reply to this email directly, view it on GitHub
https://github.com/itsjavi/supereffective.gg/issues/18#issuecomment-1427075400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53HP7MTBA6IH4QDLNVEJT3WXEGWJANCNFSM6AAAAAAUZL4AY4
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's also working for me with Windows 11 and Chrome 96 (you are using a very old version anyway)
It might be your internet connection, or some extension that you have installed.
Please update your browser or use another one like Firefox or Edge.

I got it to work on Edge. I should have tried that first. Thanks for the
help
On Sun, Feb 12, 2023 at 11:57 AM Javi Aguilar @.***>
wrote:

It's also working for me with Windows 11 and Chrome 96 (you are using a
very old version anyway)
It might be your internet connection, or some extension that you have
installed.
Please update your browser or use another one like Firefox or Edge.
[image: image]
https://user-images.githubusercontent.com/122741/218325144-5bb3bd74-077b-4ca5-b838-1a817fb1ceff.png
—
Reply to this email directly, view it on GitHub
https://github.com/itsjavi/supereffective.gg/issues/18#issuecomment-1427080497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53HP7JCHRLMNUXWA7NVE2DWXEJAZANCNFSM6AAAAAAUZL4AY4
.
You are receiving this because you authored the thread.Message ID:
@.***>


(PDB-1286) backport resource timestamp normalization to stable

In master, we've changed our report hashing so that timestamps are converted to
a consistent representation before being hashed.
This backports that change to stable so that users will receive the update in 2.3.1,
and by the time they make it to 3.0 their reports should already be right.
Additional handling will be required in master for various edge cases.

This is a kind of hard patch to test. To see that it works, what you need to do is

Clear your database and check out stable
Do a puppet run and stop puppetdb / master
Checkout master and start PDB so your report gets migrated
Curl reports and write down the hash
Export your data, clear your database, and reimport the data. Curl reports again and note that the hash does not match.

Repeat the above with this patch instead of stable, and the hashes should match. If anyone has thoughts on a good way to test for this I'd be interested.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/886/


NPE in GelfAppender.append kills the host app

Using these dependencies:
ch.qos.logback.logback-core 1.1.2
ch.qos.logback.logback-classic 1.1.2
me.moocar.logback-gelf 0.12
com.google.code.gson.gson 2.2.4
I somehow managed to product this stacktrace:
java.lang.NullPointerException
at me.moocar.logbackgelf.GelfAppender.append(GelfAppender.java:60)
at me.moocar.logbackgelf.GelfAppender.append(GelfAppender.java:21)
at ch.qos.logback.core.AppenderBase.doAppend(AppenderBase.java:85)
at ch.qos.logback.core.spi.AppenderAttachableImpl.appendLoopOnAppenders(AppenderAttachableImpl.java:48)
at ch.qos.logback.classic.Logger.appendLoopOnAppenders(Logger.java:273)
at ch.qos.logback.classic.Logger.callAppenders(Logger.java:260)
at ch.qos.logback.classic.Logger.buildLoggingEventAndAppend(Logger.java:442)
at ch.qos.logback.classic.Logger.filterAndLog0Or3Plus(Logger.java:396)
at ch.qos.logback.classic.Logger.debug(Logger.java:503)
And it killed my app.
I can see that the try/catch in GelfAppender.append re-throws the exception.  Should it now swallow it?

I figured out how I managed to produce this NPE.  I was using an environment variable for the graylog server param, like this
    <graylog2ServerHost>${GRAYLOG_SERVER_IP}</graylog2ServerHost>

and it was not set, so the value GRAYLOG_SERVER_IP_IS_UNDEFINED comes in, which causes this exception
Caused by: java.lang.IllegalStateException: Unknown host: GRAYLOG_SERVER_IP_IS_UNDEFINED: unknown error. Make sure you have specified the 'graylog2ServerHost' property correctly in your logback.xml'
at 	at me.moocar.logbackgelf.InternetUtils.getInetAddress(InternetUtils.java:53)
at 	at me.moocar.logbackgelf.GelfAppender.initExecutor(GelfAppender.java:89)
Then subsequent appends throw NPE

Perhaps a good middle ground here is to add a regex for graylog2ServerHost to ensure that it matches an IP. Then, if it doesn't, print a warning. PRs welcome :) otherwise I'll eventually get to it.

I see.  What if somebody wanted to use a dns name?  As it happens my plan was to move my system to do that, so apps can keep logging if I cycle my graylog server and it gets a new ip address.

yeah, good point. I guess this would he hard to protect against. Out of interest, what part of the stack is setting the value to "GRAYLOG_SERVER_IP_IS_UNDEFINED"? I've never seen that before.

I think its logback.

Closing as error message is better after merging #49


Add ability to run user mod commands with sudo

These commands typically need to be run with sudo, so having the option to invoke them with sudo would be useful.

Thanks for review! Will merge this in.


Add GN Build

This CL adds the necessary infrastructure to create a GN build. Only the libraries are generated at this point. Fuzzer targets have been added to show that building and linking of the static_libraries works correctly.

Rebased, squashed, and pushed into master as 3b9f197be1

Yep, I rebased and squashed on my machine but failed to push ...
I've rebased again and pushed as a114e1f30c34

Thank you!


Update test.sh script and use ~ instead of ^ semver range

Removed build.sh and updated test.sh to follow the standard CI project test flow:

install dependencies
post-install checks, e.g. ensure that the committed yarn.lock file is not outdated
analyze code for potential problems
run unit tests
build all packages
run end-to-end tests

In package.json files, use ~ instead of ^ semver range by default for better stability, see here for comparison.

@vojtechszocs looks good to me, thank you :slightly_smiling_face:

/assign

Is the concern that minor versions will break the semver contract and thus break us? Minor versions are supposed to be backwards compatible

In theory, yes :smiley: in practice, minor version bumps might cause breakages because people do make mistakes.
Also, dependencies are not isolated units of vendor code - they can, and often do depend on each other. It might happen, for example, that react version will get out of sync with @types/react. Always bumping to latest stable version (especially when done on per-single-dependency basis, instead of per-logical-dependency-group basis) can lead to problems.
(Sadly, package.json has no concept of logical dependency groups that would guard consistency across multiple related vendor packages. This responsibility therefore falls on the developer.)
So my main concern was to be very conservative when it comes to dependencies, given that we're developing a library to be used by multiple applications and lots of plugins. If you think I'm over-cautious, we can keep using the ^ ranges, it's just that I personally prefer the ~ ranges for better stability.

I'm kinda surprised ./build.sh didn't cause this to break (cc @florkbr)... We are using it theoretically

Right, but the build.sh script is not called by CI when testing our PRs, so I've expected this would pass :smile:
Anyway, I think it's fine to have a single script like test.sh to verify all packages within the monorepo, including unit and e2e tests. We can update the CI job config and remove the build.sh part, but it seems it's not in effect anyway.

RE Number 1

I think this is a weird stance to take, personally. Nothing at all, ever, stops them from breaking API. Patches can break API. I think NPM stops zero version update of code, but if they can do that... THAT can break API.
I think we either agree to the semver, or not. If we don't... not sure what our alternative is... maybe just go very specific with the version 1.1.1 instead of ~1.1.1 or ^1.1.1.
Also, we have a lock file. Isn't the purpose behind this is to never update unless we touch the dependencies in some way? Like update another dependency that overlaps with ours or like delete the lock and rebuild it? Because I don't ever recall it auto updating as I have ran yarn install on its own.

RE Number 2

Quite bizarre if you ask me -- but if it works it works. We should 100% definitely delete the reference in the release repo though. Since there will not be anything called build.sh anymore.


Can't do az login: RuntimeError: 0. The ID token is not yet valid.

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
The command failed with an unexpected error. Here is the traceback:
0. The ID token is not yet valid. Current epoch =<PHONE_NUMBER>.  The id_token was: {
  "aud": "04b07795-8ddb-461a-bbee-02f9e1bf7b46",
  "iss": "https://login.microsoftonline.com/e1b4ceea-1305-421c-9ea8-f724a6a835d7/v2.0",
  "iat":<PHONE_NUMBER>,
  "nbf":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "aio": "AVQAq/8TAAAAaKquAsSjlP+y4x0LEAR4fCQ5S1UqNp/BIYiYBn8EQ+gcjDGt8TVhQYyQbJZg6wbszWN+ha06Su0WS8jTwaqsCilJDyvI69ZOy2xJysxOYas=",
  "name": "Carlos Garc\u00eda Aparicio",
  "oid": "43fa9432-6f2c-4298-8219-1305c911c7f0",
  "preferred_username"<EMAIL_ADDRESS>  "puid": "10033FFFACF6551C",
  "rh": "0.ASEA6s604QUTHEKeqPckpqg115V3sATbjRpGu-4C-eG_e0YhAD4.",
  "sub": "hOHnYP1huZEUv2McMHm8f177Q_hG_UgXY8tLy5dAqLE",
  "tid": "e1b4ceea-1305-421c-9ea8-f724a6a835d7",
  "uti": "etstUW59LkaBvLDfDRwgAA",
  "ver": "2.0"
}
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/command_modules/profile/custom.py", line 154, in login
    use_cert_sn_issuer=use_cert_sn_issuer)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/_profile.py", line 153, in login
    user_identity = identity.login_with_device_code(scopes=scopes, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/auth/identity.py", line 132, in login_with_device_code
    result = self.msal_app.acquire_token_by_device_flow(flow, **kwargs)  # By default it will block
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/application.py", line 1561, in acquire_token_by_device_flow
    **kwargs))
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/oauth2cli/oauth2.py", line 384, in obtain_token_by_device_flow
    result = self._obtain_token_by_device_flow(flow, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/oauth2cli/oauth2.py", line 347, in _obtain_token_by_device_flow
    self.DEVICE_FLOW["GRANT_TYPE"], data=data, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/oauth2cli/oidc.py", line 115, in _obtain_token
    ret = super(Client, self)._obtain_token(grant_type, *args, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/oauth2cli/oauth2.py", line 777, in _obtain_token
    "response": _resp, "params": params, "data": _data,
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/application.py", line 524, in <lambda>
    event, environment=authority.instance)),
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/token_cache.py", line 307, in add
    super(SerializableTokenCache, self).add(event, **kwargs)
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/token_cache.py", line 113, in add
    return self.__add(event, now=now)
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/token_cache.py", line 153, in __add
    if id_token else {})
  File "/usr/lib64/az/lib/python3.6/site-packages/msal/oauth2cli/oidc.py", line 77, in decode_id_token
    err, _now, json.dumps(decoded, indent=2)))
RuntimeError: 0. The ID token is not yet valid. Current epoch =<PHONE_NUMBER>.  The id_token was: {
  "aud": "04b07795-8ddb-461a-bbee-02f9e1bf7b46",
  "iss": "https://login.microsoftonline.com/e1b4ceea-1305-421c-9ea8-f724a6a835d7/v2.0",
  "iat":<PHONE_NUMBER>,
  "nbf":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "aio": "AVQAq/8TAAAAaKquAsSjlP+y4x0LEAR4fCQ5S1UqNp/BIYiYBn8EQ+gcjDGt8TVhQYyQbJZg6wbszWN+ha06Su0WS8jTwaqsCilJDyvI69ZOy2xJysxOYas=",
  "name": "Carlos Garc\u00eda Aparicio",
  "oid": "43fa9432-6f2c-4298-8219-1305c911c7f0",
  "preferred_username"<EMAIL_ADDRESS>  "puid": "10033FFFACF6551C",
  "rh": "0.ASEA6s604QUTHEKeqPckpqg115V3sATbjRpGu-4C-eG_e0YhAD4.",
  "sub": "hOHnYP1huZEUv2McMHm8f177Q_hG_UgXY8tLy5dAqLE",
  "tid": "e1b4ceea-1305-421c-9ea8-f724a6a835d7",
  "uti": "etstUW59LkaBvLDfDRwgAA",
  "ver": "2.0"
}

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Linux-5.5.1-1.el7.elrepo.x86_64-x86_64-with-centos-7.9.2009-Core, CentOS Linux 7 (Core)
Python 3.6.8
Installer: RPM

azure-cli 2.30.0

Extensions:
ssh 0.1.8

Additional Context

@jiasli for awareness

Could you check https://github.com/Azure/azure-cli/issues/20158#issuecomment-958485251?

Symptom
The ID token is not yet valid. Current epoch =<PHONE_NUMBER>.

Root cause
This issue is because the time on the computer is not correct. MSAL checks nbf and thinks the ID token is not yet valid.
Solution
Please make sure your computer's time is configured correctly.

For the record, I was also facing this issue, even after synchronizing my machine's time. Only after an additional reboot the problem went away.

@ant0nsc, if your device is managed by Microsoft, you shouldn't be able to change the time:

If you can, please do make sure the time zone is also correct.

@jiasli , yes, I am on a MSFT device, but still can set my time manually. Even when I had that authentication problem, the time seemed off by less than a second at most, and the timezone was set correctly. On the same machine, I was able to authenticate successfully 4 hours earlier.
The only thing I could think of: My authentication request came from within WSL, and I seem to recall that there had been issues with WSL and it's timekeeping in particular after putting the laptop to sleep.

@jiasli , yes, I am on a MSFT device, but still can set my time manually. Even when I had that authentication problem, the time seemed off by less than a second at most, and the timezone was set correctly. On the same machine, I was able to authenticate successfully 4 hours earlier. The only thing I could think of: My authentication request came from within WSL, and I seem to recall that there had been issues with WSL and how its clock can be off after sleep. This morning, I had the same issue, so I ran sudo hwclock -s in WSL, and the problem disappeared.

This fixed my issue! Thank you!

Just as an FYI for anyone else who hits this on WSL
There is an open WSL bug (https://github.com/microsoft/WSL/issues/8204) where WSL2 date is set incorrectly after waking from sleep which seems to be causing this alot at the moment.

@jiasli , yes, I am on a MSFT device, but still can set my time manually. Even when I had that authentication problem, the time seemed off by less than a second at most, and the timezone was set correctly. On the same machine, I was able to authenticate successfully 4 hours earlier.
The only thing I could think of: My authentication request came from within WSL, and I seem to recall that there had been issues with WSL and how its clock can be off after sleep.
This morning, I had the same issue, so I ran sudo hwclock -s in WSL, and the problem disappeared.

This worked this morning. Now the error appears even after running sudo hwclock -s.

@jiasli , yes, I am on a MSFT device, but still can set my time manually. Even when I had that authentication problem, the time seemed off by less than a second at most, and the timezone was set correctly. On the same machine, I was able to authenticate successfully 4 hours earlier. The only thing I could think of: My authentication request came from within WSL, and I seem to recall that there had been issues with WSL and how its clock can be off after sleep. This morning, I had the same issue, so I ran sudo hwclock -s in WSL, and the problem disappeared.

This worked perfectly! Thank you :)

This is the solving workaround https://github.com/microsoft/WSL/issues/8204#issuecomment-1338334154


Add workflows to ease maintenance

Code changes

Add workflows to ease maintenance
Add Jupyter Development Team as copyright holder

Thanks!


Change in matplotlib causes colomap of imageviewer to give an error

Getting this error:
2023-03-17 11:11:37,674 - ERROR - AttributeError: module 'matplotlib' has no attribute 'colormaps'
Using matplolib version 3.4.3 with gdesk 0.5.0.
Probably caused by some change in matplolib...

Was working against an older than supported version.


Reformatting everything(including library)

Did you apply black to the whole library? That breaks some stuff...

really? It still works well here locally


use ipython for nameko shell

if available
see e.g. https://github.com/django/django/blob/9996158db467f9bef8243fcc36dc3602570e3613/django/core/management/commands/shell.py

I'd strongly vote for bpython.

i guess we could look for various shells. the django code linked to above looks for ipython and bpython.

Merged with https://github.com/onefinestay/nameko/pull/252

Hi!
ipython is pretty good, but how about  this https://github.com/jonathanslenders/ptpython ?

rather than trying to keep up with a growing list of alternative repl's, maybe we can refactor things a little to make it easy for people to roll their own


Increase operationTimeout to 30m in huge-service test

In some environments, upgrade process takes even 17m. Let's increase timeout to have some slack.
/assign @jupblb

/lgtm


Query to map in DynamicMessage generated with JsonFormat.Parser cause ClassCastException

Hi, thanks for cel-java.
First of all, sorry if this is not the correct place to ask questions.
I tried to run a query to access to map against DynamicMessage which built with JsonFormat.Parser, then I get following exception.
Exception in thread "main" java.lang.ClassCastException: class com.google.protobuf.DynamicMessage cannot be cast to class com.google.protobuf.MapEntry (com.google.protobuf.DynamicMessage and com.google.protobuf.MapEntry are in unnamed module of loader 'app')

According to the stacktrace, it looks like that exception is happening here. I run same code with debugger and stop program at that line, then Message built using JsonFormat.Parser seems to contain map entry as list of DynamicMessage and it can not cast to MapEntry.
I run same query against a Message which build with specific message's builder and put MapEntry using builder's putAll function.  In this case, the query ended without exception.
I'm not sure whether this behavior is bug or not.
Are there any way to run a query to access a map against DynamicMessage generated with JsonFormat.Parser?
If currently this operation it not supported, are there any plan to support queries on objects that hold map as a List of DynamicMessage in the future?
To Reproduce
First, define following message and generate java source code with protoc command.
message Test {
  map<string, string> a = 1;
  Contents contents = 2;
  message Contents{
    map<string, string> b = 1;
  }
}

Then build a CelRuntime.Program instance with these code.
        CelBuilder celBuilder = CelFactory.standardCelBuilder()
                .addMessageTypes(Test.getDescriptor())
                .setOptions(CelOptions.newBuilder()
                        .enableCelValue(true)
                        .enableOptionalSyntax(true)
                        .build());
        Cel cel = celBuilder
                .addVar("a", MapType.create(SimpleType.STRING, SimpleType.STRING))
                .addVar("contents", StructTypeReference.create(Test.Contents.getDescriptor().getFullName()))
                .build();
        CelValidationResult validationResult = cel.compile("contents.b.size() > 0");
        CelRuntime.Program program = cel.createProgram(validationResult.getAst());

Finally, build a DynamicMessage value and query to it, then exception is thrown.
        String stringJson = """
                {
                    "a" : {
                        "test_key_a" : "test_value_a"
                    },
                    "contents" : {
                        "b" : {
                            "test_key_b_1" : "test_value_b_1",
                            "test_key_b_2" : "test_value_b_2"
                        }
                    }
                }
                """;
        DynamicMessage.Builder messageBuilder = DynamicMessage.newBuilder(Test.getDescriptor());
        JsonFormat.parser().ignoringUnknownFields().merge(stringJson, messageBuilder);
        // Following cause Exception : Exception in thread "main" java.lang.ClassCastException: class com.google.protobuf.DynamicMessage cannot be cast to class com.google.protobuf.MapEntry (com.google.protobuf.DynamicMessage and com.google.protobuf.MapEntry are in unnamed module of loader 'app')
        Object queryResult2 = program.eval(messageBuilder.build());

My code is located here, and all of stacktrace is following.
Exception in thread "main" java.lang.ClassCastException: class com.google.protobuf.DynamicMessage cannot be cast to class com.google.protobuf.MapEntry (com.google.protobuf.DynamicMessage and com.google.protobuf.MapEntry are in unnamed module of loader 'app')
	at dev.cel.common.internal.ProtoAdapter.adaptFieldToValue(ProtoAdapter.java:216)
	at dev.cel.runtime.Activation.fromProto(Activation.java:180)
	at dev.cel.runtime.CelRuntime$Program.eval(CelRuntime.java:59)
	at org.example.Main.main(Main.java:69)

I'm truly sorry.
I close this Issue because it is possible that my test code was wrong.
I will try again by myself and confirm the problem.
I apologize once again.


support storage.tsdb.retention.time for prometheus >=2.7.0

Relates to #2533

lgtm on green

Hmm seeing the same AM flake:
FAIL: TestAllNS/x/AMZeroDowntimeRollingDeployment (47.35s)
            alertmanager_test.go:600: One notification expected, but 2 received.

I’ll re-kick the tests

still flaking the same way but this time with 3 alerts received :/
        --- FAIL: TestAllNS/x/AMZeroDowntimeRollingDeployment (47.02s)
            alertmanager_test.go:600: One notification expected, but 3 received.

                Alertmanager Notification Payload Received
                Alertmanager Notification Payload Received
                Alertmanager Notification Payload Received

kicking again

flaked again :/

as I mentioned, this is flaking super hard lately, we need to fix that

Glancing over what is happening I saw that in majority of CI jobs flakiness comes from a test responsible for scaling alertmanager to 3 instances, sending alert to each of those, waiting arbitrary number of seconds, and expecting to receive 1 notification. However majority of flakes get 3 notification instead of 1. So without modifying most of the test we can either increase time needed for alert propagation or relax a test and expect "1 or more" notifications.
Time increase:
diff --git a/test/e2e/alertmanager_test.go b/test/e2e/alertmanager_test.go
index cb9b9bac..90609833 100644
--- a/test/e2e/alertmanager_test.go
+++ b/test/e2e/alertmanager_test.go
@@ -573,7 +573,7 @@ inhibit_rules:
        }

        // Wait for alert to propagate
-       time.Sleep(30 * time.Second)
+       time.Sleep(60 * time.Second)

        opts := metav1.ListOptions{
                LabelSelector: fields.SelectorFromSet(fields.Set(map[string]string{

Test relaxation:
diff --git a/test/e2e/alertmanager_test.go b/test/e2e/alertmanager_test.go
index cb9b9bac..7679d07e 100644
--- a/test/e2e/alertmanager_test.go
+++ b/test/e2e/alertmanager_test.go
@@ -596,8 +596,8 @@ inhibit_rules:
        }

        c := strings.Count(logs, "Alertmanager Notification Payload Received")
-       if c != 1 {
-               t.Fatalf("One notification expected, but %d received.\n\n%s", c, logs)
+       if (c >= 1) && (c <= 3) {
+               t.Fatalf("One to three notifications expected, but %d received.\n\n%s", c, logs)
        }

I am not in favor of any of those, it's just what came to my mind when looking at those tests.

Relaxing to expect 1 or more is not suitable IMO. In any case we should
allow the Alertmanager cluster to settle again after the rolling deployment
for more time before triggering and checking the alert.
On Thu, Apr 4, 2019 at 11:02 PM Paweł Krupa<EMAIL_ADDRESS>wrote:

Glancing over what is happening I saw that in majority of CI jobs
flakiness comes from a test responsible for scaling alertmanager to 3
instances, sending alert to each of those, waiting arbitrary number of
seconds, and expecting to receive 1 notification. However majority of
flakes get 3 notification instead of 1. So without modifying most of the
test we can either increase time needed for alert propagation or relax a
test and expect "1 or more" notifications.
Time increase:
diff --git a/test/e2e/alertmanager_test.go b/test/e2e/alertmanager_test.go
index cb9b9bac..90609833 100644--- a/test/e2e/alertmanager_test.go+++ b/test/e2e/alertmanager_test.go@@ -573,7 +573,7 @@ inhibit_rules:
}
    // Wait for alert to propagate-       time.Sleep(30 * time.Second)+       time.Sleep(60 * time.Second)

    opts := metav1.ListOptions{
            LabelSelector: fields.SelectorFromSet(fields.Set(map[string]string{

Test relaxation:
diff --git a/test/e2e/alertmanager_test.go b/test/e2e/alertmanager_test.go
index cb9b9bac..7679d07e 100644--- a/test/e2e/alertmanager_test.go+++ b/test/e2e/alertmanager_test.go@@ -596,8 +596,8 @@ inhibit_rules:
}
    c := strings.Count(logs, "Alertmanager Notification Payload Received")-       if c != 1 {-               t.Fatalf("One notification expected, but %d received.\n\n%s", c, logs)+       if (c >= 1) && (c <= 3) {+               t.Fatalf("One to three notifications expected, but %d received.\n\n%s", c, logs)
    }

I am not in favor of any of those, it's just what came to my mind when
looking at those tests.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coreos/prometheus-operator/pull/2534#issuecomment-480062553,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATiQP0JWlaP0idNVR1EbPCfkUi5SCzE2ks5vdmhsgaJpZM4ccPrq
.

kube-prometheus just moved to https://github.com/coreos/kube-prometheus.
See contrib/kube-prometheus/README.md for details.
It seems you made a change to contrib/kube-prometheus/examples/prometheus-pvc.jsonnet and that now needs to be done against the new repository.
Thank you for your contribution and sorry for the inconvenience!

Rebased to resolve merge conflict and include tests from #2544.

lgtm on green

@mxinden seems like AM mesh initialization test is failing: https://travis-ci.org/coreos/prometheus-operator/jobs/520829906#L633
I have restarted that job and it is failing constantly in on the same test.

I have restarted the tests as our test-framework improvements merged. In case they still fail, you can wait for https://github.com/coreos/prometheus-operator/pull/2568 which should make them more stable.

rebased to include #2568

lgtm on green


GET uploads response including fileSize would be great

Why are you submitting this feature request
Currently, for the Temporal Database Api, the Get Upload endpoint returns an array of files. However, the fileSize is not included in the response.
Describe the solution you'd like
It would be great to have the fileSize included in the response.  Thanks!

I'll do this tomorrow


Added hamburger menu to moblie view and checks for login to view menu

!bounty 20

!bounty 20

!bounty 75

Documentation for the whole process. 20 points for that

!bounty 115

!bounty 135


Won't start; sed: unmatched '/'

Hi frenck, I have the following error while starting up homebridge 2.0. Could you help me with the issue please?
Raspberry pi 3
hostOS 1.1
Supervisor 0.72
HA 0.56.0
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] 10-homebridge: applying...
[fix-attrs.d] 10-homebridge: exited 0.
[fix-attrs.d] 20-dbus: applying...
[fix-attrs.d] 20-dbus: exited 0.
[fix-attrs.d] 30-avahi: applying...
[fix-attrs.d] 30-avahi: exited 0.
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
-----------------------------------------------------------
 Hass.io Add-on: Homebridge v2.0.0
 HomeKit support for your Home Assistant instance using Homebridge
 From: Community Hass.io Add-ons
 By: Franck Nijhof<EMAIL_ADDRESS>-----------------------------------------------------------
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
Log level is set to DEBUG
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] 02-updates.sh: executing...
DEBUG: Requested API resource: http://hassio/supervisor/ping
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {}}
DEBUG: Requested API resource: http://hassio/addons/a0d7b954_homebridge/info
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {"name": "Homebridge", "description": "HomeKit support for your Home Assistant instance using Homebridge", "version": "2.0.0", "auto_update": false, "repository": "a0d7b954", "last_version": "2.0.0", "state": "started", "boot": "auto", "options": {"log_level": "debug", "avahi_interfaces": "", "avahi_hostname": "", "avahi_domainname": "local", "enable_ipv6": true, "packages": [], "init_commands": [], "plugins": []}, "url": "https://github.com/hassio-addons/addon-homebridge", "detached": false, "build": false, "network": null, "host_network": true, "privileged": [], "devices": null, "logo": false, "webui": null, "stdin": false, "hassio_api": true, "homeassistant_api": true, "gpio": false, "audio": false, "audio_input": null, "audio_output": null}}
DEBUG: Filtering response using: .version
DEBUG: Requested API resource: http://hassio/addons/a0d7b954_homebridge/info
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {"name": "Homebridge", "description": "HomeKit support for your Home Assistant instance using Homebridge", "version": "2.0.0", "auto_update": false, "repository": "a0d7b954", "last_version": "2.0.0", "state": "started", "boot": "auto", "options": {"log_level": "debug", "avahi_interfaces": "", "avahi_hostname": "", "avahi_domainname": "local", "enable_ipv6": true, "packages": [], "init_commands": [], "plugins": []}, "url": "https://github.com/hassio-addons/addon-homebridge", "detached": false, "build": false, "network": null, "host_network": true, "privileged": [], "devices": null, "logo": false, "webui": null, "stdin": false, "hassio_api": true, "homeassistant_api": true, "gpio": false, "audio": false, "audio_input": null, "audio_output": null}}
DEBUG: Filtering response using: .last_version
INFO: You are running the latest version of this add-on
[cont-init.d] 02-updates.sh: exited 0.
[cont-init.d] 20-dbus.sh: executing...
[cont-init.d] 20-dbus.sh: exited 0.
[cont-init.d] 30-avahi.sh: executing...

There is a temporary workaround. See this thread for full details. https://github.com/hassio-addons/addon-homebridge/issues/40
What you have to do is enter a hostname in the avahi-hostname field. Save it, then restart homebridge. I had the same issue as you and everything is working great now after doing that.

Hi drock, thanks for your advice, I've already tried that solution but it doesn't work for me in this case.

No problem siewyc.
One other question though. I see you use a Rpi3; are you using the wifi by any chance?

Nope, it's connected with an ethernet cable

Whats the full output of your options? This is mine (no addons)
{   "log_level": "info",   "avahi_interfaces": "",   "avahi_hostname": "hassio",   "avahi_domainname": "local",   "enable_ipv6": true,   "packages": [],   "init_commands": [],   "plugins": [] }

Here's mine:
{
"log_level": "debug",
"avahi_interfaces": "",
"avahi_hostname": "hassio",
"avahi_domainname": "local",
"enable_ipv6": true,
"packages": [],
"init_commands": [],
"plugins": [
"homebridge-aqara"
]
}

Have you tried running homebridge without the addon to see if it boots?

Yes, I've tried that already :(

Hi @siewyc,
Thank you for this bug report. You wrote that you had connected your PI using ethernet, but it seems like it is connected to WiFi as well (simultaneously). Which breaks stuff, so that needs fixing.
Looking at your configuration, it seems like you have set the avahi_hostname, which is not needed in your case since the add-on is able to connect to the API of Hass.io.
So my suggestion to workaround this issue would be:
{
  "log_level": "debug",
  "avahi_interfaces": "",
  "avahi_hostname": "hassio",
  "avahi_domainname": "local",
  "enable_ipv6": true,
  "packages": [],
  "init_commands": [],
  "plugins": [
    "homebridge-aqara"
  ]
}

PS, there is another issue open concerning the use of plugins. You'd probably need to add git to the packages (see #44).

Hi @frenck,
Thank you for your reply. You were right that my pi 3 was connected to both wlan and ethernet. I have since changed the setting to connect via ethernet only. However, this is the new error that I have:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] 10-homebridge: applying...
[fix-attrs.d] 10-homebridge: exited 0.
[fix-attrs.d] 20-dbus: applying...
[fix-attrs.d] 20-dbus: exited 0.
[fix-attrs.d] 30-avahi: applying...
[fix-attrs.d] 30-avahi: exited 0.
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
Hass.io Add-on: Homebridge v2.0.0
HomeKit support for your Home Assistant instance using Homebridge
From: Community Hass.io Add-ons
By: Franck Nijhof<EMAIL_ADDRESS>[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
Log level is set to DEBUG
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] 02-updates.sh: executing...
DEBUG: Requested API resource: http://hassio/supervisor/ping
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {}}
DEBUG: Requested API resource: http://hassio/addons/a0d7b954_homebridge/info
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {"name": "Homebridge", "description": "HomeKit support for your Home Assistant instance using Homebridge", "version": "2.0.0", "auto_update": false, "repository": "a0d7b954", "last_version": "2.0.0", "state": "started", "boot": "auto", "options": {"log_level": "debug", "avahi_interfaces": "", "avahi_hostname": "hassio", "avahi_domainname": "local", "enable_ipv6": true, "packages": [], "init_commands": [], "plugins": ["homebridge-mi-aqara"]}, "url": "https://github.com/hassio-addons/addon-homebridge", "detached": false, "build": false, "network": null, "host_network": true, "privileged": [], "devices": null, "logo": false, "webui": null, "stdin": false, "hassio_api": true, "homeassistant_api": true, "gpio": false, "audio": false, "audio_input": null, "audio_output": null}}
DEBUG: Filtering response using: .version
DEBUG: Requested API resource: http://hassio/addons/a0d7b954_homebridge/info
DEBUG: API HTTP Response code: 200
DEBUG: API Response: {"result": "ok", "data": {"name": "Homebridge", "description": "HomeKit support for your Home Assistant instance using Homebridge", "version": "2.0.0", "auto_update": false, "repository": "a0d7b954", "last_version": "2.0.0", "state": "started", "boot": "auto", "options": {"log_level": "debug", "avahi_interfaces": "", "avahi_hostname": "hassio", "avahi_domainname": "local", "enable_ipv6": true, "packages": [], "init_commands": [], "plugins": ["homebridge-mi-aqara"]}, "url": "https://github.com/hassio-addons/addon-homebridge", "detached": false, "build": false, "network": null, "host_network": true, "privileged": [], "devices": null, "logo": false, "webui": null, "stdin": false, "hassio_api": true, "homeassistant_api": true, "gpio": false, "audio": false, "audio_input": null, "audio_output": null}}
DEBUG: Filtering response using: .last_version
INFO: You are running the latest version of this add-on
[cont-init.d] 02-updates.sh: exited 0.
[cont-init.d] 20-dbus.sh: executing...
[cont-init.d] 20-dbus.sh: exited 0.
[cont-init.d] 30-avahi.sh: executing...
DEBUG: Detected Avahi interfaces: wlan0
[cont-init.d] 30-avahi.sh: exited 0.
[cont-init.d] 50-config.sh: executing...
[cont-init.d] 50-config.sh: exited 0.
[cont-init.d] 60-ipv6-hotfix.sh: executing...
[cont-init.d] 60-ipv6-hotfix.sh: exited 0.
[cont-init.d] 80-user-packages.sh: executing...
[cont-init.d] 80-user-packages.sh: exited 0.
[cont-init.d] 81-user-scripts.sh: executing...
[cont-init.d] 81-user-scripts.sh: exited 0.
[cont-init.d] 82-user-plugins.sh: executing...

<EMAIL_ADDRESS>added 1 package in 3.073s
[cont-init.d] 82-user-plugins.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
starting version 3.2.2
[services.d] done.
Found user 'avahi' (UID 86) and group 'avahi' (GID 86).
Successfully dropped root privileges.
avahi-daemon 0.6.32 starting up.
chroot.c: chroot() helper started
WARNING: No NSS support for mDNS detected, consider installing nss-mdns!
Successfully called chroot().
Successfully dropped remaining capabilities.
chroot.c: chroot() helper got command 02
No service file found in /etc/avahi/services.
*** WARNING: Detected another IPv4 mDNS stack running on this host. This makes mDNS unreliable and is thus not recommended. ***
*** WARNING: Detected another IPv6 mDNS stack running on this host. This makes mDNS unreliable and is thus not recommended. ***
Joining mDNS multicast group on interface hassio.IPv6 with address fe80::42:f2ff:fe71:1d4f.
New relevant interface hassio.IPv6 for mDNS.
Joining mDNS multicast group on interface hassio.IPv4 with address <IP_ADDRESS>.
New relevant interface hassio.IPv4 for mDNS.
Joining mDNS multicast group on interface wlan0.IPv6 with address fe80::7896:64da:55ca:f74e.
New relevant interface wlan0.IPv6 for mDNS.
Joining mDNS multicast group on interface wlan0.IPv4 with address <IP_ADDRESS>.
New relevant interface wlan0.IPv4 for mDNS.
Network interface enumeration completed.
Registering new address record for fe80::42:f2ff:fe71:1d4f on hassio..
Registering new address record for <IP_ADDRESS> on hassio..
Registering new address record for fe80::7896:64da:55ca:f74e on wlan0..
Registering new address record for <IP_ADDRESS> on wlan0..
Received packet from invalid interface.
Received packet from invalid interface.
Received packet from invalid interface.
Received conflicting record [hassio.local	IN	AAAA fe80::2953:5739:259a:85dd ; ttl=120] with local record to be. Withdrawing.
Withdrawing address record for fe80::42:f2ff:fe71:1d4f on hassio.
Withdrawing address record for <IP_ADDRESS> on hassio.
Withdrawing address record for <IP_ADDRESS> on wlan0.
Host name conflict, retrying with hassio-2
Registering new address record for fe80::42:f2ff:fe71:1d4f on hassio..
Registering new address record for <IP_ADDRESS> on hassio..
Registering new address record for fe80::7896:64da:55ca:f74e on wlan0..
Registering new address record for <IP_ADDRESS> on wlan0..
..
..

Basically, it goes on and endless loop with host name conflict, retrying with hassio-3 then hassio-4 all the way to 20... 30..

Try setting avahi_interfaces  to eth0.

I have done that but it's still the same problem.
the error message is the same but wlan0 is replaced with eth0 instead.

It is a puzzle, right now... you could try both at the same time; like this: eth0,wlan0.
Also disable IPv6, there are known problems with the stack you are using.

I've tried rebooting the host and it seems that there is some success.
Scan this code with your HomeKit App on your iOS device to pair with Homebridge:
┌────────────┐
│ 314-89-098 │
└────────────┘

[10/22/2017, 11:59:12 PM] Homebridge is running on port 51826.
[10/22/2017, 11:59:12 PM] [HomeAssistant] Received event: {"event_type": "state_changed", "data": {"entity_id": "sensor.worldclock_sensor", "old_state": {"entity_id": "sensor.worldclock_sensor", "state": "06:58", "attributes": {"friendly_name": "Singapore Time", "icon": "mdi:clock"}, "last_changed": "2017-10-22T22:58:41.903234+00:00", "last_updated": "2017-10-22T22:58:41.903234+00:00"}, "new_state": {"entity_id": "sensor.worldclock_sensor", "state": "06:59", "attributes": {"friendly_name": "Singapore Time", "icon": "mdi:clock"}, "last_changed": "2017-10-22T22:59:12.406404+00:00", "last_updated": "2017-10-22T22:59:12.406404+00:00"}}, "origin": "LOCAL", "time_fired": "2017-10-22T22:59:12.406526+00:00"}
Withdrawing address record for <IP_ADDRESS> on hassio.
Withdrawing address record for fe80::aed3:dbda:67b8:853a on eth0.
Withdrawing address record for <IP_ADDRESS> on eth0.
Host name conflict, retrying with hassio-17
Registering new address record for fe80::42:53ff:feda:d296 on hassio..
Registering new address record for <IP_ADDRESS> on hassio..
Registering new address record for fe80::aed3:dbda:67b8:853a on eth0..

Got SIGHUP, reloading.
No service file found in /etc/avahi/services.
recvmsg() failed: Connection reset by peer
Failed to open /etc/resolv.conf: Connection reset by peer
Got SIGTERM, quitting.
dbus[681]: [system] Reloaded configuration
Leaving mDNS multicast group on interface hassio.IPv6 with address fe80::42:53ff:feda:d296.
Leaving mDNS multicast group on interface hassio.IPv4 with address <IP_ADDRESS>.
Leaving mDNS multicast group on interface eth0.IPv6 with address fe80::aed3:dbda:67b8:853a.
Leaving mDNS multicast group on interface eth0.IPv4 with address <IP_ADDRESS>.
avahi-daemon 0.6.32 exiting.
[s6-finish] sending all processes the KILL signal and exiting.

Upgraded to 0.56.1 and everything seems to be fixed. Homebridge is running right now.
Thanks @frenck and @drock1985!!

The issue with multiple default routes has been fixed and merged into the master branch and will be in the next release.


High CPU

We converted from Structure Map and are experiencing high CPU issues when we went to production. Nothing except the IoC framework changed. CPU  comsumption was not a problem with Structureap. We strictly implemented the resolve release pattern that's meantioned in the docs. Our lifestyles is transient for all components and services.  Has anyone seen this sort of thing?

I've been a using it for a number of years and have not seen CPU issues yet. Can you provide a repro project?

Not at the moment but I will put one together and put it up. Our Web app is a mix of webforms and MVC 5. We do Resolve components in our webforms side of things but we control that resolution and immediately release the component from the Kernel. Our controllers are fairly heavy in they take 6-7 parameters but Structure Map handled that without issue. I'm really at a loss in what I'm doing wrong with Windsor.

@colby02184 as @jnm2 has already said you'll need to provide a full running repro for someone else to have any chance of working out the problem, or better yet use a profiling tool like dotTrace which should help you find your problem quickly.
I'm going to close this issue for now until we've got something to action (please reopen when you've got more info) because there are no known issues that cause high CPU usage.

You did say all your components are transient, so that means every dependency is getting reconstructed all the time, was that your setup with StructureMap as well?
Jonorossi has the best idea of using a profiler which aside from a repro will give maintainers some actionable insight.

Just the default which is transient. But it seems like structure maps implementation of transient it's like Windsor's perwebrequet.
http://structuremap.github.io/object-lifecycle/supported-lifecycles

@colby02184 actually, the default in Windsor is singleton... And per web request means if you're resolving some component twice within the same request, it will appear as singleton, while you will get back different instances in different requests.

@mario-d-s thank you for the clarification on the perwebrequet lifestyle and yes I knew Windsor's defay is Singleton and  due to the fact I control component registration in a common project that's shared across all projects -- by control I mean I set registrations to transient.
Btw which lifestyle is recommended with an MVC mixed with a Webforms application?


compare to groundhog

Is it as accurate as groundhog in machine translation?
Thanks in advance

It is indeed, many recent research using this codebase. Besides the implementation is cleaner and straightforward.

Thanks. one more question?
what are the file "setup_cluster_env.sh" and "setup_local_env.sh" for? If I wanna run the jobs on the cluster, what should I do? As it works on my machine, but it will be hanged while running on the cluster nodes. Your answer will be appreciated.
Thanks

@papar22 those scripts are prepared for setting up the environments for dl4mt winter school. Submitting a job is a bit of cluster specific, so you may need to check this just as a reference to submit a job.


New directory structure

(I accidentally opened this against the main vs my own fork — please ignore!)


custom_options event.time changed from rails 5.2.2

In the readme there is a section to add the time to the logs:
 config.lograge.custom_options = lambda do |event|
    { time: event.time }
 end

This isn't the correct time anymore since this rails commit: https://github.com/rails/rails/commit/7c4542146f0dde962205e5a90839349631ae60fb (included from Rails 5.2.2), because now Process.clock_gettime(Process::CLOCK_MONOTONIC) is used as start time for events, which is the monotonic time since boot.
Can this be replaced in the readme with  { time: Time.now }?

@jlemaes Sounds good, could you please propose a PR?

@pxlpnk See #267

Super thanks! 👍

Hi @jlemaes, just wondering--does that change how precise the timing of said event is? I worry by doing that, if the request took a while to load that we lose the actual time the event came through. But perhaps that is wrong?

+1 to @FeminismIsAwesome's question. I think instead you want:
now = Process.clock_gettime(Process::CLOCK_MONOTONIC)

start_time = (now - event.time).seconds.ago
end_time = (now - event.end).seconds.ago

config.lograge.custom_options = lambda do |event|
   { start_time: start_time, end_time: end_time }
end

This happened in my Rails app, using lograge and the Datadog Heroku buildpack.
I found the fix by following what GitLab did: https://gitlab.com/gitlab-org/gitlab-foss/-/commit/aeb67dd489b1ccc7f0ab1d702725729ab9cc3e27#7fbfd491b0620e7fa9f3cd8818582f52ce3e8f31_22_21
Thanks for updating the README!


Publication Tumor Type command in schematic

The manifests have a “Publication TumorType” column and schematic expects a “Publication Tumor Type”, I think this is probably happening when the manifests are being generated (?)

Addressed with #27


TypeError: 'type' object is not subscriptable

Hello,
I am trying to run this application, but getting the error
TypeError: 'type' object is not subscriptable
File "python3.8/dist-packages/streamlit/runtime/scriptrunner/script_runner.py", line 552, in _run_script
exec(code, module.dict)
File "chatbot/aws_well_arch_chatbot/aws_well_arch_st.py", line 13, in
import utils
File "/chatbot/aws_well_arch_chatbot/utils.py", line 18, in
def get_embedding(text: str, model: str = EMBEDDING_MODEL) -> list[float]:
TypeError: 'type' object is not subscriptable
Can you please help to resolve this issue.
Thanks

try updating to python 3.9+

Try updating to 3.9+


Refactor system configuration parsing for improved Destination AE support

This commit adds support for starting an external storescp process based on the values of the QueryAet and DestAet fields in the system.json configuration file. If the QueryAet and DestAet are different, it is assumed that the external storescp process is already started and the code skips starting another process.
linked to issue #133 .

Oh ok got it, thanks for the feedback :)


Unable to load site locally

Hi there,
Thank you for building such a great tool! I'm trying to get a Notion page up and running on it, but after installing dependencies and running npm run dev I'm having an issue where my page is never loading. On localhost:3000, the page never finishes loading and no content shows up. I've tried with the default notion page ID as well as my own page ID with no luck, and if I refresh the page then I see NextJS recompiling successfully once again in the logs. After starting the server my log looks like the following:
><EMAIL_ADDRESS>dev /Users/aidan/.../nextjs-notion-starter-kit
> next dev

ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info  - Using webpack 5. Reason: Enabled by default https://nextjs.org/docs/messages/webpack5
event - compiled successfully
event - build page: /
wait  - compiling...
event - compiled successfully
{
  domain: 'somedomain.com',
  name: 'Aidan Site',
  rootNotionPageId: '78fc5a4b88d74b0e824e29407e9f1ec1',
  rootNotionSpaceId: '',
  description: 'Example site description'
}
wait  - compiling...
event - compiled successfully
event - build page: /
wait  - compiling...
event - compiled successfully
wait  - compiling...
event - compiled successfully

Any help to resolve this would be greatly appreciated! Thanks.

@aidanlw17 make sure your project is updated with the latest changes from the base template and run it with yarn & yarn dev

This should be resolved with the latest release.


Import 55bd695e46: Del'd enable_material_proc_shaders check (by Caitlyn)

This is an automated proposal to look at a commit made by Caitlyn and import it into Overte

Commit:
55bd695e46792f8128503af5790ce10875dce66c

Author:
Caitlyn

Date:
Tue, 25 Feb 2020 20:51

Del'd enable_material_proc_shaders check

Stats:

Filename
Stats
Lines
Added
Removed
Lines in blame

libraries/render-utils/src/MeshPartPayload.cpp

2
1
1
⚠ 0

1 files
-
2
1
1
0

To work on this, please assign the issue to yourself, then look at the commit and decide whether this would be a good addition to Overte.

If the commit is useful, tag it with "Tivoli: Keep", and keep it open until it's merged.
If the commit is not useful, tag it with "Tivoli: Discard", and close it.
If the commit is not useful right now, but might be later, tag it with "Tivoli: Maybe later", and close it.
If it's hard to decide, tag it with "Tivoli: Discuss", and keep it open.

Useful commits should be submitted as a PR against Overte. Tag this issue in PR, so that it's automatically closed once the PR is merged.
You can cherry-pick this issue with this command:
git cherry-pick 55bd695e46792f8128503af5790ce10875dce66c

Removes a check we don't have


[HW] Jack Day

Name: Jack Day
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/jack-day-05413168/
Attach the homework screenshots below for both step II and step III:

https://api.badgr.io/public/assertions/RiTHxFBAQvCLkY7ZrmvPfA?identity__email=dayjack%40live.co.uk


parse from string using regex

This PR add functionality to parse from string, using regex. (Kindly re-check from the Test scene)
example :
IdleCash cash = IdleCash.Parse("135a");   // similar to new IdleCash(135,"a");
IdleCash cashFail = IdleCash.Parse("random123"); // will fail, return IdleCash.Zero instead
// or using TryParse
IdleCash.TryParse("314a", out IdleCash cash2); // return true
IdleCash.TryParse("1a1a1", out IdleCash cash2); // return false

I have fixed a small issue when regex is matched but the type is actually not valid.
For example "34a" is not valid if we don't have type "a" in our settings.
So, I am parse it as "34" and ignore its type.
Here is the commit for the fix: 496fcf31c0e6270e221bde75b1088cb371f86774

Great! I forgot to check that


Strange Rectangles drawn in weird places

Versions

Python: 3.7
OS: Windows 10
Kivy:  1.10.1
Kivy installation method: https://kivy.org/doc/stable/installation/installation-windows.html

Description
The titel of this issues isn't really helpful, but I will do my best to describe it in detail. As you can see on the pictures I attached, there is a white rectangle drawn at some random spots.
Normally there should be buttons, and the buttons are created on demand. They are created in the "on_pre_leave" function from the first screen. The "list_resource_locations" is called for that.
# Python Code:
class ResourceLocation(GridLayout):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        # Do some stuff, not relevant

    def list_resource_locations(self):
        for resource_location in Something:
            button = ResourceLocationButton(text=resource_location, id="R" + resource_location)
            self.add_widget(button)

# Kivy Code:
    <ResourceLocationButton>
    background_color: [0.671, 0.596, 0.596, 1]
    background_normal: ''

    BoxLayout:
        orientation: "vertical"
        size_hint_y: 0.5
        Label:
            size_hint_y: None
            height: 35
            text: "Resource Location"
            font_size: '20sp'
            canvas.before:
                Color:
                    rgba: .48, .48, .48, 1
                Rectangle:
                    size: self.size
                    pos: self.pos

        ScrollView:
            size: self.size
            size_hint_y: 1
            size_hint_x: 1
            canvas.before:
                Color:
                    rgba: .52, .52, .52, 1
                Rectangle:
                    size: self.size
                    pos: self.pos
            ResourceLocation:
                id: resource_locations
                cols: 1
                size_hint_y: None
                height: self.minimum_height
                spacing: 1
                padding: [1, 0, 1, 1]

The intressting thing is that this bug nearly never occurs, but more often when I run the code in the debug mode. I have the feeling it has something to do with the speed everything gets executed or so.
I find it really hard to track down the problem, has someone a simillar issues or some idea what might be the cause of it?
I would like to attach a small example on how to reproduce the error, but in the my short version the error didn't occure.
This is how that part should look like:

The Error sometimes occurs on the left side as well. I used the same structure (Boxlayout, ScrollView, GridLayout) and similar functions to create the buttons here. The intressting thing is that 3 Buttons are missing and they are "replaced" by that white rectangle?!

I used breakpoints to stop the execution at some points to inspect the kivy data, but I can't find the rectangles there.

Are there threads involved in your version that produces the issue? If so, make sure to execute anything that create/update/delete widgets from the main thread.

Hi, thanks for your help, yes another thread was the problem.
I fixed it by using events to add the buttons.


[BUG] menu not showing ents properly

Just loading in the acf server and opening the menu guives me this error every time, its been a couple days.
Just broken menu, specificaly entities like weapons , missiles, engine, ect..
server: GIT-dev-3b2bb2a
cient: ZIP-master-f61425d
Load in the server and open menu, acf go brrrrr.

Try updating the server to the latest version, there's some changes on 871f8e0 that should help deal with this. Also duplicate of #168.


initGooglePay doesn't resolve upon first load

Describe the bug
Trying to implement Google Pay, I'm noticing that even using the exact code in https://github.com/stripe/stripe-react-native/blob/master/docs/GooglePay.md doesn't work properly
	useEffect(() => {
		const initialize = async () => {
			const { error } = await initGooglePay({
				testEnv: true,
				merchantName: intl.formatMessage(messages.nativePayRequestCompanyLabel),
				countryCode: countryIsoCode,
				billingAddressConfig: {
					format: "FULL",
					isPhoneNumberRequired: true,
					isRequired: false,
				},
				existingPaymentMethodRequired: false,
				isEmailRequired: true,
			});

			console.log("initialize");

			if (error) {
				console.tronIfAvailable.log(error);
				return;
			}
			setGooglePayInitialized(true);
		};
		initialize();
	}, [countryIsoCode, initGooglePay, intl]);

If I leave and re-enter the screen that includes that code, then it resolves properly
Expected behavior
It should resolve every time
Smartphone (please complete the following information):

Device: Xiaomi Mi 9T with Google Pay set up
OS: Android 11

I was able to resolve it on my end. It seemed to be a problem with my setup.

hi @bill-pairaktaridis , I tested this using the example from the docs:
React.useEffect(() => {
    async function initialize() {
      const { error } = await initGooglePay({
        testEnv: true,
        merchantName: 'Test',
        countryCode: 'US',
        billingAddressConfig: {
          format: 'FULL',
          isPhoneNumberRequired: true,
          isRequired: false,
        },
        existingPaymentMethodRequired: false,
        isEmailRequired: true,
      });

      if (error) {
        Alert.alert(error.code, error.message);
        return;
      }
      console.log('this is logged :)');
      setInitialized(true);
    }
    initialize();
  }, [initGooglePay]);

and it seems to be working for me on first load. Could you provide a link to a minimal project that reproduces the issue you're running into?

Closing for now since this seems to be working on the most recent version. if you're still running into this issue, please let me know and share a link to a project I can use to reproduce the same behavior


narwhalify BaseMappingTransformer

make BaseMappingTransformer polars compatible while maintaining backwards compatibility with pandas.
Why?
Making the project polars compatible for speed benefits. Narwhals allows us to do this piece by piece and maintain compatibility between components as it makes it possible to pass pandas dataframes to polars logic. This involves converting between pandas/polars dataframes.
How?
convert logic in BaseMappingTransformer to function with polars dataframes and add narwhalify decorators to methods to ensure compatibility with pandas dataframes.
Update testing where required.

PR https://github.com/lvgig/tubular/pull/368


Batch of fixes

In hope this PR will help you keep this awesome theme working with newer versions of Redmine :crossed_fingers: :rocket:
Fixes issues #222, #233, #242, #245, #246, #253, and few others not already reported :bug: :adhesive_bandage:
Tested on Redmine 4.2.10 and 5.0.5 with the following plugins enabed :

Additionals
Additional Tags
Impersonate
Dashboard
Hearts

Awesome. Thanks!

Some of these changes touch basic Redmine layout - were these changes tested in:

Different configurations? E.g. fixed layout true/false.
Earlier Redmine versions? Although I guess it shouldn't matter as much, they're old.

Yes I tried to test theses changes in different configurations : fixed layout + css-grid-layout/$flexbox-layout + sidebar-position.
Regarding earlier Redmine versions, I only tested on the 2 latest releases.


Commands not being recognized as expected when Atm_command is attached to a SoftwareSerial object

I'm not sure if this is an Automaton issue by itself, but I've noticed that commands that otherwise work OK when in conjunction with the Serial stream are only recognized if an extra ' ' (blank space) is added to the end of the command when attached instead to a SoftwareSerial object instance. Again, not sure whose library is at fault here but I think it should warrant some investigation on either part.

There were some bugs in the command matching of Atm_command. They were fixed in the deveolpment branch, could you give this version a try and let me know if perhaps it fixes your problem?
https://github.com/tinkerspy/Automaton/tree/53f9078ec75846be7bd564c74373f57e5c06dd47
Changes were:

commands are now matched fully
Both CR & NL are now recognized as an end of line character

Hi, thank you for the super speedy reply! For good measure (I should've done this in the first place), before updating the library I went on and tested this same functionality with some other application/terminal connected at the other end of this "pipe". When I reported the problem I was using a C# script inside a Unity test app/game. It must have been some problem on my side then, because now that I've retested this with AGG Software's Serial Port Monitor app the commands are all being correctly recognized. I'll have to further investigate this C# script inside the Unity game to try to find what could be the problem. In perhaps you have any ideas whatsoever, I'd more than appreciate. BTW, this library of yours is absolutely fantastic. Congratulations on this amazing piece of software!

You're welcome. There can be timing issues with SoftwareSerial, especially with ATtiny microcontrollers that use their built in oscillator. Perhaps changing the baudrate could help?
Thanks for the praise, I've started a testimonials page on the wiki and added yours ;-)

Thanks for the tip. For now I'm doing my tests on the Arduino Uno (SparkFun's RedBoard version, that is). I'll certainly take a deeper look into all this and share my findings, if any! (LOL).
Sure, I'm glad to help and take part of your testimonials page!

You probably won't have timing issues on an Uno because it uses an external chrystal.


nprogress multi instance

Hi,
Is it possible to instantiate nprogress multiple time, I wanted to use it in different DOM elements not globally?
Here's what I've done:
$.ajax({
....
beforeSend: function(data) {
   NgProgress.configure({ parent: '#'+element});
   NgProgress.start();
},
success: function(response) {
   NgProgress.done();
}

If I call Ajax in different DOM to get some data, the nprogress will be used one time in one place only  for the first ajax call.

This would be great and I'm also looking for a feedback of this being possibile.
Something like this:
var bodyElement = document.body;
var bodyNProgress = new NProgress(bodyElement);

var otherElement = document.getElementById("first");
var otherNProgress = new NProgress(otherElement);

bodyNProgress.start();

Would be great to have such feature

There should be such feature, for now, you can run the NgProgress.configure every time you want to run NgProgress.start for each instance, not a great solution but works in my react app.

This PR will enable you to have multi instance
https://github.com/rstacruz/nprogress/pull/179

Any feedback on this?

Any updates on this? Thanks!!

And is it real now?

want

if you are looking just for this feature, i have it in a forked package https://www.npmjs.com/package/multi-nprogress

if you are looking just for this feature, i have it in a forked package https://www.npmjs.com/package/multi-nprogress

goodjob!


Uncaught TypeError: Cannot read property '1' of null (Windows Error?)

Chartist.js does not work for me on ANY browser on a windows machine, but it DOES work on both Safari and Chrome on a Mac. Are there any reasons why that would be so?? The error is pointing to this point of the chartist.js code:
c.quantity = function(a) {
if ("string" == typeof a) {
var b = /^(\d+)\s*(.*)$/g.exec(a);
return {
value: +b[1], **ERROR
unit: b[2] || void 0
}
}
return {
value: a
}
}

Seen from Chrome on Mac

Seen from Chrome on Windows.
My code hasn't changed and I cannot figure out why it works no matter which browser on Mac but not on any browser on Windows.

I also noticed that I can re-initiate the statement and it will show up.
This is my code for the top chart:

If I copy the var damageChart = new Chartist.Bar() and paste it in the console, it works. I thought maybe it was trying to initialize the charts before the data was coming in, but it does not make sense why it loads on a Mac and why it would not load at the same pace on a windows machine; especially when both OS are using Chrome.

I also see it is coming from the animations (I didn't include it in the above picture, didn't think that was the root cause)
startAnimationForBarChart: function(chart) {
                    chart.on('draw', function(data) {
                        if (data.type === 'bar') {
                            seq2++;
                            **data.element.animate({**
                                opacity: {
                                    begin: seq2 * delays2,
                                    dur: durations2,
                                    from: 0,
                                    to: 1,
                                    easing: 'ease'
                                }
                            });
                        }

The asterisk'd part is what is throwing the error in my code.
If I comment out the animations it now show's fine on both windows and Mac. What would be causing the issue with animations on windows?

Hello @EricTurner3 👋
Chartist has recently been updated to version 1.0 with some new exciting changes like ESM and TypeScript support. The migration guide can be found right here.
Please let me know if the issue is still relevant to you so we can further investigate it!


do not hard code the tx fee

Change: https://github.com/ava-labs/avalanche-explorer/blob/1bdb1614549f831baa816eded1f1fe96377d3017/src/components/TransactionDetailCard.vue#L57
To a calculation:
burned = sum(ins) - sum(outs)

I am not sure how well this will work..
ex: https://explorer.avax.network/tx/2EzXrN89escxvpin5TBQzzhJfuqAMQfW2YLAY6jFh9xstAkDZs


'End' doesn't go to line end if there's whitespace

When I've got a line like this, the dots denoting spaces:
This is some line...

pressing End takes me to the end of the "line" word, not to the real end of the line. It's a small issue and I know some IDEs have the option to customize the End behavior to behave this "smartly" but in this basic text editor, I would expect End to go to the end of the line, not to the last word.

This is documented bug in Avalon Edit.
https://github.com/icsharpcode/AvalonEdit/issues/12
I've asked to have it looked at again.


Duplicated link without destination in the left blade check the screenshot

Duplicated link without destination in the left blade check the screenshot
Assess physical servers for migration to Azure

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

ID: ec2de1ed-faeb-a5c6-05aa-183b8818bab8
Version Independent ID: 91118096-7111-033a-a0af-3e6665621ffb
Content: Assess VMware servers for migration to Azure VMs in Azure Migrate - Azure Migrate
Content Source: articles/migrate/tutorial-assess-vmware-azure-vm.md
Service: azure-migrate
GitHub Login: @rashi-ms
Microsoft Alias: rajosh

@kikofarad Thanks for your comment! We will review and provide an update as appropriate.

@kikofarad We have verified and updated the document as required. The update will go live in less than 24 hours. Thanks again!


Add links to message spans in settlement spans

Contributes to https://github.com/Azure/azure-sdk-for-net/issues/30656

/azp run net - servicebus - tests

API change check
API changes are not detected in this pull request.

This seems like a straightforward change for the settlement case of this feature request.

Yep, I just snuck in a bunch of clean up 😆


Support Bound and Default together in TypeVar

Change Summary
Add support for Bound and Default together in TypeVar:
from pydantic import BaseModel

class MyBaseModel(BaseModel):
    field: str

class ExtendedBaseModel(MyBaseModel):
    another_field: str

T = TypeVar('T', bound=MyBaseModel, default=ExtendedBaseModel)

Related issue number
fixes #9418

Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @sydney-runkle

please review

@FyZzyss,
Thanks for your contribution! We'll review this soon - working on pushing v2.10 out now, but I'm anticipating we can get this into v2.11 :)

I think it's safe to support all combinations with bounds, constraints and defaults. In any case, we prioritize the default type over constraints or bound:
T1 = TypeVar('T1', bound=int, default=SubInt) --> generates a schema for `SubInt`
T2 = TypeVar('T2', int, str, default=int) --> generates a schema for `int`

Having a different bound and default shouldn't be an issue. The only requirement -- from a static type checking perspective -- is that default should be a subtype of bound. But such checks are out of scope for Pydantic (see https://github.com/pydantic/pydantic/issues/10462#issuecomment-2367301169).
This should work:
    def _unsubstituted_typevar_schema(self, typevar: typing.TypeVar) -> core_schema.CoreSchema:
        try:
            has_default = typevar.has_default()
        except AttributeError:
            # Happens if using `typing.TypeVar` on Python < 3.13
            pass
        else:
            if has_default:
                return self.generate_schema(typevar.__default__)  # pyright: ignore[reportAttributeAccessIssue]

        if typevar.__constraints__:
            return self._union_schema(typing.Union[typevar.__constraints__])

        if typevar.__bound__:
            schema = self.generate_schema(typevar.__bound__)
            schema['serialization'] = core_schema.wrap_serializer_function_ser_schema(
                lambda x, h: h(x), schema=core_schema.any_schema(),
            )
            return schema

        return core_schema.any_schema()

Totally agree that it's safe. This code only works for unsubstituted models, so we simply prioritize the default type (expected behavior for me).
I will take Viicos suggestion


Create TypeScript typings

This project isn't written in TypeScript, but many TypeScript users would like to use published typings. For this issue, create TypeScript typings that cover the exported functions, and contribute it back to DefinitelyTyped!
This issue is great for someone who wants to learn more about TypeScript!

I'd like to work on this.

All yours! Thanks @claeusdev!

Hi @claeusdev, still want to take this up? If not, let me know! Just wanted to give others a chance if you're busy.

Hey, @mstuart I would like to take it up but I'm a little bit busy atm if there's someone available to take over sure otherwise I'll be able to work on it over the weekend.

@mstuart I would like to work on this!

I have a question about this issue, are we using:
https://github.com/typings/typings
because it appears to be deprecated.

Is this issue still open @mstuart ?

Sorry for the late reply @imrishabh18. I think so! I don't see any zoid typings in the DefinitelyTyped repo, so it seems to be open.

Any updates on this effort? I believe there are several tools which exist to create .d.ts files from flow types, though I don't know if any are up to date/well maintained.

Hello, guys! Any update about it?
This issue is very old and we don't have the types yet.
Can anyone help? @mstuart @pramodrhegde @mike-robertson-fast


feat: Added Hindi Translations

Fixes #1275
Changes: Added Hindi Translations to the app.

@iamareebjamal @nikit19 please review

@iamareebjamal please review

Many translations are wrong. It'll be worse for a person who is using hindi as a language option than seeing the strings in English. I think the same thing is true for other language translations. But I could verify only this because I understand Hindi. Internationalization is not our priority right now

@iamareebjamal I'm not proficient in hindi, you can suggest changes

That's the point. Why add changes which are not required and wrong. Obviously I'm going to spend time reviewing and suggesting on other PRs solving actual issues

Well, there is a cool hack to translate the strings using google sheets.


March 07 2018

Basic demo of Docker workflow

How to build the image, or download the image
How to run the image in a container, with optional volume mounting of external directory
How to make changes to the image's environment via the Dockerfile

Discuss Demo Release

goals
tasks
timeline

Modelling:
Fixed error with crash lags
Almost finished with predictions by week


Cards are empty (properties not rendered ?)

Thanks for this exemple.
I found it through Youtube and cloned this repo as a first contact with Kivy/KivyMD (after doing the Pong exemple :)).
However, when executing the code from commit (03f660194b282195d2d2a4a775ff4689e6393cdb), i get the following screen :
It looks like Cards do not get their properties...

I could make it work with those changes:
In main.py, I declared the properties of ElementCard in its own class:
class ElementCard(MDCard):
    text = StringProperty()
    image = StringProperty()
    items_count = StringProperty()
    subtext = StringProperty()

in main.kv, as it's already done just above, I removed the inheritance from MDCard and  also the properties declaration :
<ElementCard>:
    md_bg_color:69/255,55/255,86/255,1
    padding:dp(15)
    spacing:dp(15)
    radius:dp(25)
    ripple_behavior: True

    orientation:'vertical'
    Image:
        source:root.image
    MDBoxLayout:
.........
.........

use kivymd==0.104.2


Need of "actions" node in inventory, missing part in Escoria's documentation

Hello once again!
Today, I was going through inventory system, I followed everything as it is written here in Inventory chapter, but it seems, that it is missing a word about node of Control type called "actions".
Before adding this node into inventory scene, I was experiencing unsuccesful attempt to call function get_child_count in inventory.gd, then I checked basic inventory setup, which comes with Escoria, and there it was. After adding, no problem. Please, for another new users, update documentation. There is written, you have to include those arrows for going through items, which I don't need, but it is there, so I added them and it works. This should be included too.
By the way, what is that "actions" node used for? Is it used while using items from your inventory? Thank you for your response, sir!

Yeah, there's some discussion about improving the test scene in #22. The framework is more confusing than it needs to be, both because of such issues and because there several nice features in Escoria that are not even used in the test scene.
Which action node are you referring to? I couldn't find that, but there's the action_menu scene which can be used to create verb coin interfaces, like the one in Full Throttle, and the action variable, which is exported by the item script, which is used for one-click interfaces, like in many point-and-clicks from recent years.

I am not refering to  action_menu scene, I am refering to inventory scene. When I am building my own custom inventory scene, it has to have empty node called "actions" of Control type, else it won't run. There is nothing written about it in documentation, but that "actions" node is actually included in sample project, that comes with Escoria.

Oh, I see. Looking at the code in inventory.gd, I don't think that node serves any purpose at all, but the script makes the assumption that it will be there, hence it fails if it isn't.
Declaration of actions array:
var actions = []

Signal handler:
func action_pressed(n):
    current_action = n
    for but in actions:
        but.set_pressed(but.get_name() == n)

Connecting signal in _ready():
var acts = get_node("actions")
actions = []
for i in range(acts.get_child_count()):
    var c = acts.get_child(i)
    actions.puah_back(c)
    c.connect("pressed", self, "action_pressed", [c.get_name()])

Presumably acts.get_child_count() fails, since acts is not defined. Also, puah_back() is misspelled, and not an actual function of the Array object, but execution never gets that far.
I think this is code that was refactored into the separate action_menu.tscn, as action_menu.gd has a similar implementation, but one that actually works.

Oh, okay. I was wondering, what type of action could an inventory system have. Thank you.


Use Status instead of DisconnectReason in event

According to the spec, the reason field of the event may have any valid status code. For example, if host initiates a disconnect, the controller will generate a disconnect event with a reason outside the DisconnectReason enum. Changing it to use the Status type will ensure it can handle any value the controller is allowed to send.
The earlier approach was to extend the DisconnectReason with values returned by the controller, but this allows the user to break the spec wrt. which reasons can be send in the Disconnect command. Therefore, the DisconnectReason enum now only allow the values as defined by the spec.

Relevant part in spec for reason field in DisconnectionComplete event:
https://www.bluetooth.com/wp-content/uploads/Files/Specification/HTML/Core-54/out/en/host-controller-interface/host-controller-interface-functional-specification.html#UUID-332adb1f-b5ac-5289-82a2-c51a59d533e7
Relevant in spec for reason field in Disconnect command:
https://www.bluetooth.com/wp-content/uploads/Files/Specification/HTML/Core-54/out/en/host-controller-interface/host-controller-interface-functional-specification.html#UUID-6bb8119e-aa67-d517-db2a-7470c35fbf4a


few tick timeout comparison and other issues/improvements:

Hi @RobMeades
I had some time to screen the https://github.com/u-blox/ubxlib/tree/preview_fix_tick_wrap_rmea branch a bit and found some issues many of them timeout related but not only. Towards the end you also find a suggestion how to avoid these mistakes in the future with an improved timeout API that hides the MS handling from the user.

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/cell/test/u_cell_geofence_test.c#L404
change: (uPortGetTickTimeMs() < gStopTimeMs)
to: (((uPortGetTickTimeMs - gStopTimeMs) < 0)
suggest to use uPortTickTimeBeyondStopMs

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/cell/test/u_cell_loc_test.c#L779
change: (uPortGetTickTimeMs() < gStopTimeMs)
to: (((uPortGetTickTimeMs - gStopTimeMs) < 0)
or use uPortTickTimeBeyondStopMs

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/common/geofence/src/u_geofence.c#L1906
review: dynamicsMinDistance.lastStatus.distanceMillimetres = uPortGetTickTimeMs();
possibly change to dynamicsMinDistance.lastStatus.timeMs = uPortGetTickTimeMs();

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/common/mqtt_client/test/u_mqtt_client_test.c#L471
change: (uPortGetTickTimeMs() < startTimeMs +
(U_MQTT_CLIENT_RESPONSE_WAIT_SECONDS * 1000))
to: (uPortGetTickTimeMs() - startTimeMs < (U_MQTT_CLIENT_RESPONSE_WAIT_SECONDS * 1000))
or use uPortTickTimeExpiredMs

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/src/lib_mga/u_lib_mga.c#L1191
change: if (now > pMsgInfo->timeOut)
to: if (now - pMsgInfo->timeOut > 0)
suggest to rename now to startTimeMs and timeOut  StopTimeMs
sugguest to use uPortTickTimeBeyondStopMs

many more occasions throughout this file ....
https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/src/lib_mga/u_lib_mga.c#L1237
https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/src/lib_mga/u_lib_mga.c#L2246
https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/src/lib_mga/u_lib_mga.c#L2265
https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/src/lib_mga/u_lib_mga.c#L2291
... potentially more ..

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/test/u_gnss_geofence_test.c#L901
https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/test/u_gnss_geofence_test.c#L920
change: (uPortGetTickTimeMs() < gStopTimeMs)
to: (uPortGetTickTimeMs() - gStopTimeMs < 0)
suggest use of use uPortTickTimeBeyondStopMs

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/gnss/test/u_gnss_pos_test.c#L777
chnage: (uPortGetTickTimeMs() < gStopTimeMs)
to: (uPortGetTickTimeMs() - gStopTimeMs < 0)
suggest use of use uPortTickTimeBeyondStopMs

maybe there are even more of these things ...
To avoid all such mistakes my suggestion is to have tick uPortGetTickTimeMs return a "handle" and move all the comparison functions into the function where this is allways properly handled. properly, this way the develpper can't make any mistakes as he does not know what using this handle has and how to convert it
TIMEOUTHANDLE_t uPortGetTimeoutHandle() {
    return (TIMEOUTHANDLE_t) uPortGetTickTimeMs();
}

You could extend the tick API with a elapsed millisecond and use this throughout the code in all the printfs
uint32_t uPortGetElapsedMs(TIMEOUTHANDLE_t timeoutHandle)
{
   return (uint32_t)(uPortGetTickTimeMs() - (int32-t)timeoutHandle);
}

uint32_t uPortGetElapsedSeconds(TIMEOUTHANDLE_t timeoutHandle)
{
   return uPortGetElapsedMs() / 1000;
}

of course the current API would also take only these handles

Very useful indeed, thanks Michael, give me time to absorb and I will comment here, tagging issue #209 so that warasilapm sees this.

Some of the issues are just in test code, so will not affect a real application.

Here is some more:

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/cell/src/u_cell_net.c#L751
likely a bad idea, just remove: (pInstance->startTimeMs > 0) &&
probably you would need an extra bool to indicate that startTimeMs was never set.

https://github.com/u-blox/ubxlib/blob/3cf5348e22fddd185f878eea49e3bb7526cccdf7/common/location/test/u_location_test.c#L332
probably better to init with uPortGetTickTimeMs or maybe this = 0 is not needed at all, but seting 0 here is not better than having a random value.

Yeah I've been noticing this repeatedly as well - though we've not been reviewing the test code applications. Zephyr does something similar as suggested by wrapping all "times" in an "opaque" (but not really) struct that is supposed to be passed into macros/static inlines to handle the encapsulated values. You can see the encapsulation in k_timeout_t here. I imagine you'd end up with a uTick_t struct similar to this.
Note Zephyr does some things differently as it generally leverages int64_t to avoid any wraparound issues. We've found a couple more edge cases to share on the other issue later today after we do some more investigation.

The fix for this issue, as proposed (and reviewed) by @mazgch, can be found in commit 79e808eb8be3095b4f94efcffba97515ba1870bd.


fix usage with nix 2.15

warning: The interpretation of store paths arguments ending in.drv recently changed. If this command is now failing try again with '/nix/store/m5raazy9x0zm8p732y001aqypds0hb2m-gebaar-libinput-+6ce14eb.drv^*'
this will break nix 2.15< but i don't know enough golang to make it conditional so help needed
fixes https://github.com/Mic92/nix-build-uncached/issues/52
There's most likely dozens of silently broken CI right now so it's important to get this in and to nixpkgs

bors merge

bors merge


All fields are market as required

I have setup instantFields on 3 models. Each CRUD Model have optional and required fields but when the modal shows up i have red asterisc to all of the fields labels.
On the standard CRUD update form everything is ok.
I've also tried to comment backpack's setRequiredFields  and add setAjaxStoreRequest but with no luck.

This is not easy to answer, we don't have an issue like this in our projects. Maybe you can provide some code?


fix: remove deprecated bling COPY for files and rpms

https://github.com/ublue-os/bling/blob/main/Containerfile the containerfile copies an empty folder in these locations for backwards compatability. /tmp/files and /tmp/rpms has been deprecated from bling, all of the RPMs have mostly moved to COPR or ublue-os/config

Looks like there is a problem running builds for fork PRs. I'll have to look into this later. I'll test locally and merge if it's good


Support for special characters

It would be great if special characters like á, é, í, ó, ú was supported.
Thank's for the good job!

My fault. Ignore it.


Size badge shows higher size than the truth

Are you experiencing an issue with...

[X] shields.io
[ ] My own instance
[ ] gh-badges NPM package

:beetle: Description
npm minified bundle size (min) badge shows 6.44kb, when the file size is 5.44.
:link: Link to the badge
https://img.shields.io/bundlephobia/min/htmel
The minified file I'm referring to:
https://github.com/lefetmeofefet/htmel/blob/master/dist/htmel.min.js
Thanks for this awesome service by the way, truly cool project :)

Hi.
We don't calculate this value The bundlephobia badge reports the size from the bundlephobia API: https://bundlephobia.com/api/size?package=htmel If you think that is wrong, you could raise an issue on the bundlephobia repo: https://github.com/pastelsky/bundlephobia That said, bundlephobia is reporting the size of your entire npm package, not just one file, so we would expect the total reported bundle size to be a bit larger than just the size of that one file.
If you wanted to report just the size of one file in your GitHub repo, we do have a badge for that too:
https://img.shields.io/github/size/lefetmeofefet/htmel/dist/htmel.min.js
Thanks

@chris48s beat me to it 😄
https://bundlephobia.com/api/size?package=htmel
{
  "assets": [
    {
      "gzip": 2515,
      "name": "main",
      "size": 6439,
      "type": "js"
    }
  ],
  "dependencyCount": 0,
  "dependencySizes": [
    {
      "approximateSize": 10812,
      "name": "htmel"
    }
  ],
  "description": "",
  "gzip": 2515,
  "hasJSModule": "./src/htmel.js",
  "hasJSNext": false,
  "hasSideEffects": true,
  "name": "htmel",
  "repository": "",
  "scoped": false,
  "size": 6439,
  "version": "1.0.3"
}

Thanks for explaining :)
I'll be using the tag you linked
On Tue, 10 Mar 2020 at 21:03, Caleb Cartwright<EMAIL_ADDRESS>wrote:

@chris48s https://github.com/chris48s beat me to it 😄
https://bundlephobia.com/api/size?package=htmel
{
"assets": [
{

  "gzip": 2515,

  "name": "main",

  "size": 6439,

  "type": "js"

}

],
"dependencyCount": 0,
"dependencySizes": [
{

  "approximateSize": 10812,

  "name": "htmel"

}

],
"description": "",
"gzip": 2515,
"hasJSModule": "./src/htmel.js",
"hasJSNext": false,
"hasSideEffects": true,
"name": "htmel",
"repository": "",
"scoped": false,
"size": 6439,
"version": "1.0.3"
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/badges/shields/issues/4757?email_source=notifications&email_token=ADEOEHITC5KLBHU5IACHACLRG2FJDA5CNFSM4LFFMB42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOMXJLA#issuecomment-597259436,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADEOEHK73KVMQ75G7QJ55OLRG2FJDANCNFSM4LFFMB4Q
.


release: 1.0.1

oh, I just tried it on iOS using the preview link and tapping Valora doesn't work:
https://github.com/celo-org/rainbowkit-celo/assets/57791/cb08d3c7-3949-4be0-bba0-4d509acd63f0

This can be reproduced just with RainbowKit.
Here's the issue: https://github.com/rainbow-me/rainbowkit/issues/1336

We can merge this in since the issue isn't related to the rainbowkit-celo but the rainbowkit/wagmi walletconnect connector


fix: update builder mergemock sim with builder block assertions

Previously builder sim was running without any validations.
Modified the mergemock builder interaction to enable differentiating by fee recipient and use that to add engine and builder block assertions

:tada: This PR is included in v1.9.0 :tada:


Add support for custom User Assigned Identities against Aspire Projects

See https://github.com/dotnet/aspire/pull/3339  for more information.
This change adds support to azd to consume a new userAssignedIdentities property in the Aspire manifest.
An example being:
"userAssignedIdentities": [
  {
    "clientId": "00000000-0000-0000-0000-000000000000",
    "resourceId": "/subscriptions/\u003Csubscription_id\u003E/resourcegroups/my-rg/providers/Microsoft.ManagedIdentity/userAssignedIdentities/my-user",
    "env": "TEST"
  },
  {
    "clientId": "{identities.outputs.clientId}",
    "resourceId": "{identities.outputs.resourceId}",
    "env": "TEST2"
  }
]

The change adds support for:

Detection of template strings within this new property
The generation of environment variables to support the Client IDs
The addition of the userAssignedIdentities in the container's yaml

I'm going to close this for now - based on the discussion in https://github.com/dotnet/aspire/pull/3339 we do want to support this scenario long term, but we don't think this is the right way to do it (and it likely is something we'll want to do post GA once we move the logic for created the IAC for Container Apps into the Aspire deployment package and outside of azd.


Faster, more consistent tests

Zoom, zoom, zoom.
https://www.youtube.com/watch?v=O0ikW7pVa_I

Coverage decreased (-1.3%) to 93.088% when pulling 37a2bc2b8024139ed8ca28f7fe334ae0b0bc6163 on test-improvements into df49c207fffa15a8d61bac59f5942db4d25c5e96 on master.

Coverage remained the same at 94.412% when pulling 21d31878366ce8a2341d0049a3e1d1f4d9298163 on test-improvements into df49c207fffa15a8d61bac59f5942db4d25c5e96 on master.

@jwngr Ptal - removed a couple uneeded tests, fixed the rest and sped everything up.

Looks good to me although I think @katowulf has a better grasp on the digest loop stuff than I do. Tying it into our enableLogging() call seems fairly hacky but also fairly ingenious! The tests certainly do run significantly faster and seem to be much more stable. Great job!
https://www.youtube.com/watch?v=gTIcIzKjGjg

Actually, Abe, I'm thinking your commit song should be this one.

lgtm! Jacob and I both made minor nits. Decide if you want to address those and send it back my way to merge.

Coverage remained the same at 94.412% when pulling d6e759e96094433578db0ebf31df8e40167ad059 on test-improvements into df49c207fffa15a8d61bac59f5942db4d25c5e96 on master.

Back to you Mr. W


Exploratory Sprint: Meltano supports multiple workspaces/projects to help users divide up their work

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/596
Originally created by @dmor on 2019-04-17 14:46:30

We have decided to revert the multiple projects feature, per the discussion here. https://gitlab.com/meltano/meltano/issues/591
However, there may be a need to revisit this functionality in the future when we have users with a lot of different data sources and dashboards they are looking to organize and collaborate around. We will revisit this when it becomes a user demand, rather than a pre-emptive feature from us.

[ ]  Write opportunity assessment (started here: https://docs.google.com/document/d/1Xbnma-a90RyKz1W1NBDcuINOyJj3kjmX417_8cDFunk/edit#heading=h.wtfp7g1r16nq)
[ ]  @mbergeron @jschatz1 technical feasibility and architectural discussion
[ ]  create an issue detailing the work to be done to implement MVC feature
[ ]  @dmor go/no-go decision and milestone assignment

View 4 previous comments from the original issue on GitLab