Feature request: fetch guests (i.e. attendees)

It would be great if we can fetch the guests (i.e. attendees) of an event, perhaps in the form of:
[
    {
        "name": "John Doe",
        "email"<EMAIL_ADDRESS>    },
    {
        "name": "Jane Doe",
        "email"<EMAIL_ADDRESS>    }
]

Context: this library is being used in the Home Assistant Google Calendar integration. If I know I'm in my kitchen (using presence detection) and I have a meeting, I'd like Google Assistant to notify me about this. I tend to create events for myself without guests (e.g. focus time) and I don't need to trigger automations on those.

Really cool suggestion.
I feel like the example you gave was very compelling.  I think of two possible ways to consider this.

Dump the data from the calendar API into the home assistant data (entity or calendar event in the trigger). It might not be standard across other calendars so we'd have to work through a generic way to add this to the data model

Actually associating the guests with People in home assistant which might be more powerful and potentially useful by other integrations. This might get more interest from the dev community about other things we can connect and power.

Going to keep thinking about this...

My other thought is that location may also be an interesting similarly shaped attribute.
(kow who is invited and where they should be)

My other thought is that location may also be an interesting similarly shaped attribute.
(know who is invited and where they should be)

Thanks for giving this some thought! Are you also the maintainer for the HA integration?

Dump the data from the calendar API into the home assistant data (entity or calendar event in the trigger). It might not be standard across other calendars so we'd have to work through a generic way to add this to the data model

Yes, I was thinking about something like this. I'm not too familiar with how HA works, but I was assuming I could attach something like a data dictionary to the trigger events that I can then handle in HA. I was about to write this myself, but noticed guests aren't exposed from this library, which brought me to this feature request.

Actually associating the guests with People in home assistant which might be more powerful and potentially useful by other integrations. This might get more interest from the dev community about other things we can connect and power.

This sounds very cool. Most of the people I meet with aren't a Person entity in my HA instance, though I can see its use. I think the gcal-sync library still would need to be extended to include guests in its data model though, am I right?


Null satellite ?????

Version of Advanced Rocketry

<IP_ADDRESS>.7.0-238

Have you verified this is an issue in the latest unstable build

[x] N

Version of LibVulpes

LibVulpes-1.12.2-0.4.2-75

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] N/A
We use the Modpack Enigmatica 2 - E2

Crash report or log or visualVM (if applicable)

https://pastebin.com/Sc5DVjFc

Description of the problem

Upon assembling a space station module in the overworld, the server crashed. Since then, the server keeps crashing on every startup with the aforementioned crash log. The Microwave Reciever in question currently has 44 satellites connected.

Reopen if reproduceable on the newest 1.12 version.

The mod does throw null around a lot with regard to satellites. A potential fix could be the addition of null checks before saving data to disk and after loading it, followed by dropping the offending satellite from the appropriate places.

Please unstale, Mr. Bot!

oi, stalebot. actually bugger off would you.

unstale pls, stalebot

Stalebot is off atm while I do rewrites. Or else everything would go stale. :p


Google Closure Compiler silently fails if installed from npm via --no-optional flag

STR:
echo { "dependencies": { "google-closure-compiler": "20220301.0.0" } } > package.json
npm install --no-optional

Then try invoking any Closure Compiler command, e.g.
node_modules\.bin\google-closure-compiler
node_modules\.bin\google-closure-compiler --version
node_modules\.bin\google-closure-compiler --help
node_modules\.bin\google-closure-compiler --platform=java --help
node_modules\.bin\google-closure-compiler --platform=java --version
npx google-closure-compiler

but they will all fail and directly return to command prompt, with Windows %ERRORLEVEL% status of -4058.
Tested on Windows without Java in PATH.
Expected: Presumably omitting --no-optional will only install the Java version, so execution should fail with an error message "Java not installed." instead of silently failing.
I don't 100% know if the issue is caused by missing Java, but I am assuming so in the lack of better information.
npm list gives:
E:\test>npm list
E:\test
`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | | `--<EMAIL_ADDRESS>  | |   `--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  |   `--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +-- UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>  +-- UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>  +-- UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | | +--<EMAIL_ADDRESS>  | | +--<EMAIL_ADDRESS>  | | `--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>deduped
  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>deduped
  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>  | |   | `--<EMAIL_ADDRESS>deduped
  | |   `--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>

Running a regular npm install without the --no-optional flag will avoid the issue. In that case, npm list returns
E:\test>npm list
E:\test
`--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | | `--<EMAIL_ADDRESS>  | |   `--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  |   `--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +-- UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>  +-- UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | | +--<EMAIL_ADDRESS>  | | +--<EMAIL_ADDRESS>  | | `--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>deduped
  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>deduped
  | |   +--<EMAIL_ADDRESS>  | |   +--<EMAIL_ADDRESS>  | |   | `--<EMAIL_ADDRESS>deduped
  | |   `--<EMAIL_ADDRESS>  | +--<EMAIL_ADDRESS>  | `--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>

Can you move this issue over to the https://github.com/google/closure-compiler-npm repo? That's where all the npm related code resides. Also with what you list out, this may be working as designed. There isn't a good way to detect whether Java is available or not without simply trying to invoke it. The error received from invoking an executable that doesn't exist is pretty platform dependent.

Moved to https://github.com/google/closure-compiler-npm/issues/227


Feature/typescriptize random

Codecov Report

Merging #175 into develop will decrease coverage by 0.32%.
The diff coverage is 86.36%.

@@             Coverage Diff             @@
##           develop     #175      +/-   ##
===========================================
- Coverage    86.77%   86.45%   -0.33%
===========================================
  Files           37       38       +1
  Lines         1618     1624       +6
  Branches         9       14       +5
===========================================
  Hits          1404     1404
- Misses         210      213       +3
- Partials         4        7       +3

Impacted Files
Coverage Δ

packages/js-crypto-random/src/index.ts
100% <100%> (ø)

packages/js-crypto-env/src/index.ts
52.38% <100%> (ø)
:arrow_up:

packages/js-crypto-random/src/random.ts
81.08% <81.25%> (ø)

Continue to review full report at Codecov.

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


Refactor & detach network metrics.

This PR is supposed to address https://github.com/paritytech/substrate/issues/6093 alongside some refactoring. In particular

sc-network metrics are aggregated into the service/metrics submodule, introducing two more sourced metrics to avoid duplicate atomics (for "num connected peers" and "major syncing").

The periodic sc-service metric/telemetry updates have their own update interval, independent of the intervals at which network status and state information is requested, so that network-independent metrics are updated on every tick and network metrics whenever available.

Some disk I/O-related metrics that are no longer used by the telemetry have been removed (cf. https://github.com/paritytech/substrate-telemetry/issues/277).

total_bytes_inbound/total_bytes_outbound have been removed from NetworkState, as they're already on NetworkStatus.

To me the difference between a network state and a network status is not intuitive, but I guess that is out of the scope for this pull request.

I agree, and for a moment I thought about a renaming of the form NetworkStatus -> NetworkInfo, NetworkState -> NetworkDebugInfo. Not sure if that would've been a great improvement, but the distinction seems to be simply that one is just high-level, cheap network information (counters etc). whereas the other is a (partial) snapshot of the network state (connected peers etc) that can be more expensive to compute. However, as you already guessed I didn't want to derail this PR too much with technically unnecessary renaming.

To me the difference between a network state and a network status is not intuitive, but I guess that is out of the scope for this pull request.

I agree, and for a moment I thought about a renaming of the form NetworkStatus -> NetworkInfo, NetworkState -> NetworkDebugInfo. Not sure if that would've been a great improvement, but the distinction seems to be simply that one is just high-level, cheap network information (counters etc). whereas the other is a (partial) snapshot of the network state (connected peers etc) that can be more expensive to compute. However, as you already guessed I didn't want to derail this PR too much with technically unnecessary renaming.

I am in favor of the rename. I also agree that it would add unnecessary noise to this pull request.

NetworkStatus was added by someone not in the networking team as part of a big PR, and I didn't spot the change. It should just go away entirely.


Couple of post-release Fixes

Fixes #390 to add building link to README
Fixes #391 for registry top page
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

ping @dmp42 @stevvooe This should fix the registry issue Olivier.  I'll have a test server up shortly.

@moxiegirl In the future, try to only do one fix per PR. It will make reviewing and merging much faster.

LGTM!

@stevvooe Yep, I'm trying to get more rigorous in that way...I learned from working with y'all. It is interesting what each team finds important in PR and the like.  Pretty soon I'll be like the perfect PR person cause I'm picking the best "DNA" of practice from each team.


How to test using our image?

How to test using letter image? or how to use it as OCR?

hi @ilham-bintang you can use this repo for creating your own dataset https://github.com/miendinh/vi.word2img


v2: Feature Request - expose a method for add lifecycle hook

right now life cycle hook can only be set when build a container as below
`injector := do.NewWithOpts(&do.InjectorOpts{
HookAfterRegistration: func(injector *do.Injector, serviceName string) {
fmt.Printf("Service registered: %s\n", serviceName)
},
HookAfterShutdown: func(injector *do.Injector, serviceName string) {
fmt.Printf("Service stopped: %s\n", serviceName)
},
Logf: func(format string, args ...any) {
    log.Printf(format, args...)
},

})`
my case is that I am using do as DI container build some basic infra in the group for other projects.  it's obvious that other teams(projects) also need hooks for project specific design. I export the DI but they can not register hook.
proposal:
if HookAfterRegistration and HookAfterShutdown are slcie of functions and di expose a new method to add a new hook method then it will work in my case
of course there is gotcha:  this new added method which for adding hook will throw error when the DI is not emtpy
thank you!

I agree.
Let's change that before v2 release. A slice of hooks might be a breaking change in container options.

Closing your ticket in favor of #70 + #71 + #72.


SAK-29696 NPE when grading assignment with 0 score using two decimals

+1, looks good. I'll merge this in when the CI build is happy

Tested locally; fix works, although it solves a StringIndexOutOfBounds exception rather than NPE :)


Help me, blank page

I see blank page, but code of page
localhost:8000/admin/users

@vad11 we need more information with issues like these.  looks as if your setup is wrong. Get me more info and open another ticket please.


Export internal utility

[ ] src/formatter
[ ] src/rules

In fact, I'm not sure if anyone would want to use these utilities after exporting them. If there is such a person, it would be great if you could tell me the use case.


Update README to include secure best practices in handling secrets/creds in git repos

This sections about storing API keys/passwords in the conf.py file is a very bad idea when it comes to best practices in code repositories:

https://github.com/sphinx-contrib/confluencebuilder#configuration
https://sphinxcontrib-confluencebuilder.readthedocs.io/en/stable/configuration.html

Source: https://github.com/sphinx-contrib/confluencebuilder/blob/master/doc/configuration.rst

You should never store this information within a git repository, especially in plain-text. Your CI pipeline tooling should be handling secure content, so that any contributor isn't simply pulling down plain-text API keys to their workstation in order to just contribute to docs.
If using something like GitHub, and GitHub Actions, for example:

https://docs.github.com/en/actions/reference/encrypted-secrets

GitLab CI docs can also be referenced for managed env var secrets, or external providers of secrets, if using their pipeline tooling.
For more information on why password storing in git is a bad idea, there is a good discussion and breakdown of things on Stack Exchange here:

https://security.stackexchange.com/questions/191590/why-is-storing-passwords-in-version-control-a-bad-idea

There is no intent to promote the idea that a user should store a plaintext password/secrets inside a shared version control system. The quick README/minimalistic configuration examples are there solely to help a new user of this extension to quickly setup a "first" environment. If users are actually storing the contents of confluence_server_pass in, for example, a shared Git repository, I would suspect there would be additional security concerns to consider for said users.
That being said, this is not the first time the best practices for secrets has been brought up. While a notice was added inside confluence_server_pass's explicit section and the tip's document was updated on a way to deal with CI environments, maybe there can be more done here. For example:

For the project README, tutorial example and the mini-configuration in the configuration document, we can replace confluence_server_pass with confluence_ask_password. When users then want to prepare to, for example, set a CI setup, they can find more details on confluence_server_pass when searching the documentation and be given the relevant warnings then. A concern to consider is trying to handle the use case where MinGW users can have trouble using the confluence_ask_password in their sessions; or,
Leave the confluence_server_pass entries as is, but add a trailing comment such as the following when referenced:

confluence_server_pass = '<value>' # do not share secret

Dedicate a section insider the "Tips" document which points a general description on secrets management, and have the confluence_server_pass configuration section point to this new section.
Add a warning note and reference to new "Tips" section in the confluence_client_cert_pass option as well.

With changes to the confluence_ask_password option to better supprt MSYS/MinGW environments (see #456), it was opted to adjust a series of documentation examples (see #462) to use the ask-password option over an explicit password configuration. From this, users who wish to investigation the ability to explicitly provide passwords for a publish run will lead the to the configuration-list document which includes an explicit caution about secrets management.
Note that changes to #462 do not yet include changes for the README document, which also includes an example configuration set. Changes to README are planned (with the following patch); however, they will be held until near a release event, to help reduce the chance of user's using the current stable release without the fixes in #456:
diff --git a/README.rst b/README.rst
index 940f8e8..3d58a0b 100644
--- a/README.rst
+++ b/README.rst
@@ -86,8 +86,8 @@ and publishing:
    confluence_space_name = 'TEST'
    confluence_parent_page = 'Documentation'
    confluence_server_url = 'https://intranet-wiki.example.com/'
-   confluence_server_user = 'username'
-   confluence_server_pass = 'api-key-or-password'
+   confluence_ask_user = True
+   confluence_ask_password = True

 For a complete list of configuration options, see also:

The above patch has been applied to the main branch. This should finalize the changes desired to help reduce the risk of new users persisting secrets in various repositories. With v1.5 is now available on PyPI and the stable documentation now promoting ask options by default, marking as closed.


Potential XSS in utils.js escapeExpression() regarding equal sign handling?

I'm using a library that has handlebars as a dependency. My application has been audited (at work) and apparently the audit report shows that there is a potential vulnerability in handlebars (4.0.11). I've attached a screenshot of the issue below.

Does anyone know if this is this valid or a false positive? I don't have access to VulnDB but will try to get more details about it if needed. It seems like it is complaining about the equal sign not being escaped? I'm new to the handlebars code but at first glance it seems fine to me? (The escaping is done at the end of the function.)

The code does escape the "equals"-sign (https://github.com/wycats/handlebars.js/blob/4.x/lib/handlebars/utils.js#L8)
There is also a test case (https://github.com/wycats/handlebars.js/blob/4.x/spec/utils.js#L21)

The change was made in 2015 and should be in all recent release (especially in 4.0.11).
The entry does not seem to be up-to-date. There was a rather long period where the fix was in the git-repo but no release was made. I have made the release in April 2017, so it may just be that the person who wrote this entry has not checked the newer versions.

Is this related to #1363?

Thanks for the quick response. That's what I thought when I quickly glanced over the code. Since I don't have access to VulnDB (and there may be others using it), I'm going to see if I can ask my contact to see if they can report that issue as resolved.


KDE Plasma 5.19

Plasma 5.19 was released on June 9 but nixos-unstable still packages 5.17 (released on October 15, 2019!). What is the status on this?

https://github.com/NixOS/nixpkgs/pull/84542
https://github.com/NixOS/nixpkgs/pull/84232

@jwinnie From the looks of it the qt update is going to be merge very very soon, and I believe that likely means the plasma update as well.

Is this targeting the 20.09 release @petabyteboy? :)

It would be good to have that in 20.09, but I can not say for sure that I can completely fix up the Plasma 5.18.x PR until the freeze.

@petabyteboy I believe last release we made a bit of an exception to try to get plasma 5.19 into the release. Considering Plasma 5.17 was released on October 15, 2019 I think this should (perhaps) be a priority.
Are you still waiting on re-review of https://github.com/NixOS/nixpkgs/pull/84542 from @ttuegel ?

The new GTK settings menu is not working. It was rewritten and new patches need to be written from scratch. That's the only real thing, but I don't know how and where to start with this, so the chances of getting it done in the next month or so are small if noone else takes up the task of fixing the GTK settings menu.

We're on 5.20, with #110536 updating to 5.21 now, so can this be closed?


Magritte vs Mediapipe

I have used Mediapipe for face detection using their tutorial code here.
I read that Magritte uses Mediapipe library under the hood, but I couldn't figure out if Magritte does a better job than Mediapipe in identifying faces. Have you compared their results?

We use the MediaPipe face detection models under the hood, but we provide a few extra graphs that combine different rotations, or the short-range and full-range models. Please refer to the detection graphs documentation for more information.
We have not compared the results yet.


Bug- False "UserWarning: Explicitly requested dtype <class 'jax.numpy.int64'> requested in astype is not available, and will be truncated to dtype int32

Description
Python 3.9.19 (main, May  6 2024, 19:43:03)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import jax
>>> import tensorflow_probability.substrates.jax as tfp
>>> tfd = tfp.distributions
>>> rng = jax.random.PRNGKey(0)
>>> tfd.OneHotCategorical(logits=jax.random.normal(key=rng,shape=(3,4)), dtype=jax.numpy.float32).sample(seed=rng)
/home/user/anaconda3/envs/ml_exp/lib/python3.9/site-packages/jax/_src/numpy/array_methods.py:68: UserWarning: Explicitly requested dtype <class 'jax.numpy.int64'> requested in astype is not available, and will be truncated to dtype int32. To enable more dtypes, set the jax_enable_x64 configuration option or the JAX_ENABLE_X64 shell environment variable. See https://github.com/google/jax#current-gotchas for more.
  return lax_numpy.astype(arr, dtype, copy=copy, device=device)
Array([[1., 0., 0., 0.],
       [0., 1., 0., 0.],
       [0., 0., 0., 1.]], dtype=float32)

I explicitly set jnp.float32 and did not ask for int anywhere, but received the warning
System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.30
jaxlib: 0.4.30
numpy:  1.26.4
python: 3.9.19 (main, May  6 2024, 19:43:03)  [GCC 11.2.0]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1
platform: uname_result(system='Linux', node='user', release='6.8.0-45-generic', version='#45-Ubuntu SMP PREEMPT_DYNAMIC Fri Aug 30 12:02:04 UTC 2024', machine='x86_64')

Thanks for the report! This warning is arising becuase tensorflow_probability passes an int64 to JAX. You can see this by re-executing your code after running the following:
import warnings
warnings.simplefilter('error')

This will turn the warning into an error with a full traceback, which points to the place where this problematic dtype is passed, which is here: https://github.com/tensorflow/probability/blob/c6c86e788516a9b16958460528867735a721ef40/tensorflow_probability/python/internal/backend/numpy/random_generators.py#L106
If this warning is a problem to you, you could think about reporting it to the tensorflow_probability maintainers, but there's nothing JAX can do to fix this short of removing the warning entirely.


BOSH fails to create a vm with default CPI when cpi-config is in use

Describe the bug
bosh recreate failed with error:  if the target az is not specified with cpi
Error 100: CPI '' not found in cpi-config
To Reproduce
Steps to reproduce the behavior:

You already have bosh director and deployment running.
You use external cpi-config,  for multiple cpis.
Bosh recreate a vm,  target az of which is to use default cpi,  the cpi is not explicitly defined in cloud_properties of the az.

Expected behavior
vm can be successfully recreated.
Logs
Started updating instance networking_db > networking_db/ebd43510-37e1-4ac0-86d4-79266a1dddc0 (0) (canary). Failed: CPI '' not found in cpi-config (00:00:40)
Versions (please complete the following information):

Infrastructure: [vsphere]
BOSH version [268.1.0]
BOSH CLI version [4.0.1]
Stemcell version [bosh-vsphere-esxi-ubuntu-trusty-go_agent/3468.47]

Additional context
This can be fixed by modifying here:
https://github.com/cloudfoundry/bosh/blob/v268.1.0/src/bosh-director/lib/bosh/director/deployment_plan/stemcell.rb#L100
with the following:
return model_for_default_cpi if cpi_name.nil? || cpi_name.empty?

This is the thread on slack:
https://cloudfoundry.slack.com/archives/C02HPPYQ2/p1540190522000100
https://cloudfoundry.slack.com/archives/C02HPPYQ2/p1539845673000100
peri [1 day ago]
@h4xnoodle, from the code, it seems that default cpi should be used, cpi_name is nil or empty is checked almost everywhere, except the stemcell.rb,  after adding the check,  it worked! So I assume this is something that is missing?  You can find the code fix at the bottom of the issue.
h4xnoodle [14 hours ago]
if we do end up checking elsewhere, we'll make sure this still makes sense. Do you have insight on how your CPI config had an empty CPI defined? How come the CPI was not defined in cloud_properties of the az?
emm,   do you have insight?  how come?

Thank you for your patience. After careful consideration, we decided not to implement a fix for this. However, we will update our documentation to mention that operators using cpi-config must explicitly define a default CPI for each of their availability zones.


VAAL implementation is incomplete?

Hi is VAAL implementation incorrect in this toolkit. I see that it is very different from the author's original implementation and it also looks incomplete, unusable. Can you please let me know if it is?

I agree with your point. In current version, the parameter of VAE is not being updated.

Hi is VAAL implementation incorrect in this toolkit. I see that it is very different from the author's original implementation and it also looks incomplete, unusable. Can you please let me know if it is?

In https://github.com/SineZHAN/deepALplus/blob/master/query_strategies/vaal.py
vaal-vae is trained in line 31.

Hi is VAAL implementation incorrect in this toolkit. I see that it is very different from the author's original implementation and it also looks incomplete, unusable. Can you please let me know if it is?

Thank you for point out this issue, I found that I wrongly merged the vaal version (the old version lost discriminator part). I have updated it.


Metrics code gets reused, turn it into a core library

The metrics code gets duplicated across methods, to avoid this we need to turn it into a core library under pavedroad.io/core/go.

Duplicates issue#60 in roadctl, https://github.com/pavedroad-io/roadctl/issues/60


The product's image should be visible properly in mobile view.

Bagisto Version: Master
Steps To reproduce--
1. Visit the shop on mobile view.
2. VIew the products on the home page.
Actual result: THe product's image is not showing properly in mobile view. Please find the screenshot Here.
Expected result: The product's image should be visible properly in mobile view. please take a reference from here.

This is because it's carousel. If you fit images to grid on mobile, as I suggest, you should also change carousel to hard coded grid on homepage. 6 on one row on large screen, 4 on one row on tablet, 4 on two rows on mobile. Same for NEW and FEATURED products. Carousels are loading slow on mobile and only if you begin scrolling down a little.

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Bump version

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

I'm not sure if this should be updated to the new compilers in this PR or after the fact...

@dpryan79 you can merge this PR and then add the bot-rerun label to the other PR. Then the auto-tick bot will do an updated PR on the new version.

@xhochy Makes sense, thanks.


Adds a strong tag to the main components

Issue https://github.com/basemate/matestack-ui-core/issues/80: Strong tags
Changes

Adds strong tags to the set of main components

Notes

No notes

Re opening due to weirdness with git


Display current value of a property in settings.json

When typing a property in a json settings file, it would be helpful if the suggestion list displayed the current and default value at the top of the list.
Example:
The first object in Default Keybindings is
{
  "key": "escape escape",
  "command": "workbench.action.exitZenMode",
  "when": "inZenMode"
}

If I edit keybindings.json and start typing { "key": "escape escape", "command": ""
The suggestion list should display the current and default value for the key "command" which in this case are the same "workbench.action.exitZenMode". The list then continues with other possible values.
Another example in settings.json:
  "workbench.colorCustomizations": {
    "activityBar.background": ""

In this case the suggestion list should display first the default value from the default theme and second, the current value if applied from another theme.

At the moment the suggestion list displays #ff0000 for all color settings.

If a property value exists in Default User Settings, the correct default value is displayed in the suggestion list e.g "editor.tabsize: " shows the value "4"

I think the keybindings case is better suited for the UI which already essentially does this.
The theme case specifically feels useful. But it's tough to generalize, some settings can have multiple "current values", and the current value for complex settings is hard to display as a suggestion.

@roblourens
Yes the keybindings case wasn't the best example.
I'm trying to figure out how something could have multiple current values, can you point to such a value?
I just thought this could be handy in general, but the themes example is however where I was missing this feature from the beginning, here to be more precise:
"workbench.colorCustomizations": {}
"editor.tokenColorCustomizations": {}
I was also thinking about what default value to show if your inside a theme
"editor.tokenColorCustomizations": {
    "comments": "",      // Show VSCode default
    "[Default Dark+]": {
        "comments": ""  // Show theme default
    }
}

I'm trying to figure out how something could have multiple current values, can you point to such a value?

If something is configured different in multiple workspace folders, or in different languages. There is not one value, it's contextual.
I think our investment for new UI features like will mainly be in the settings GUI but I'll leave this open for now.

Default value is included, "current" value is hard to define.


How to use Unreal-Linux-From-Windows-Toolchain.cmake to cross compile for Linux from Windows?

I am tring to cross compile cesium-native for linux from windows. And I created this bat:
cmake -B build-linux -S . -DCMAKE_TOOLCHAIN_FILE="unreal-linux-from-windows-toolchain.cmake" -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DCMAKE_BUILD_TYPE=Release
cmake --build build-linux --target install

According to the content in cmake, I added these to Enviroments.

But it still run into this error:

This is the cross compile tool chain:

Hi @lincoln310, we haven't tried cross-compiling in awhile, because these days we just compile for Linux on Linux. You might find it easier to do the same by using WSL2.
That said, you may be able to fix your immediate problem just by adding -G Ninja to your cmake configure command-line. You'll need to install Ninja, for course.


Added fix for scroll view insets #tiny

👍

ah yeah - this didn't include them for the gene view

@orta yeah, we should probably just move this to the base component VC.


broadcast

how to broadcast to all players

Hi, store the active sessions in a hashmap and iterate ?

Thank you. I'm a rust newer. I tried to use HashMap to save sessions but build with error. Haha, I'll learn rust doc again

this is my test code, how can i use hashmap
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=f56d983a9de6a315c7161a5b382bcb77

It should works : https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=c34fd866631d2e139c58e3c2cc7cd757
However the framework is a prototype for my game under Unity, it's not very stable.

thank you very much. aha.., I'm very interested in rust, so try learn some about asynchronous network programming on rust


Update README to remove references to sample yml

The sample configs aren't present in the repository anymore.

Coverage remained the same at 54.082% when pulling d6074d2022f335d78be8446edf04918f2b4886f7 on readme/sample-ymls into 1222a615569fa9b8a1acc0017352935f85aad25a on develop.

@little9 Sorry!  They are present, but they got moved to config sample:  https://github.com/psu-stewardship/scholarsphere/tree/develop/config/sample


Add HOSTESS_OS env to override OS

For testing and debugging, allow the OS detection to be overridden via HOSTESS_OS. This makes it easier to test different OS hostsfile defaults.

This is now called HOSTESS_FMT and is not actually implemented yet.

The docs and CLI help mention this but it doesn't exist. Oops. Better fix that.

This is implemented here: https://github.com/cbednarski/hostess/blob/master/hostess/hostlist.go#L429-L449


Infinite loop when parsing certain tweets.

The issue

Sometimes when parsing tweets, in parse_output.c, it gets stuck in an infinite loop
Here is an example output.json in which the issue is reproducible when parsing 3 or more lines.

Possible solutions

Attaching a debugger shows the culprit to be the deleteWord function.
Specifically, the processTweet function seems to be calling deleteWord infinitely.
Possible first steps could be checking the deleteWord function to see whether it works for all strings.

I can create a fix to this issue. Please assign it to me.

Thank you!


net: DNS timeout not implemented per spec

Per man resolv.conf -
"timeout:n
sets the amount of time the resolver will wait for a response from a remote name server before retrying the query via a different name server. Measured in seconds, the default is RES_TIMEOUT (currently 5, see <resolv.h>). The value for this option is silently capped to 30."
This indicates that after timeout seconds, the querier will move to the next server.
However, in Go 1.7, this seems to be a hard max timeout.  The config structure is even noted as such -
"timeout    time.Duration // wait before giving up on a query, including retries"
This in itself doesn't make much sense, since server iteration happens inside the retry for loop (which is correct).
Then in dnsclient_unix.go starting at 183, a hard deadline seems to add cfg.timeout to current time before even starting retries or iterations.  Ideally instead this should be calculated and reset in the server for loop.
This is problematic because many set a low timeout in resolv.conf to keep a bad server from hanging for so long.  Doing so now will cause queries to fail rather than move to next server as is supposed to happen.
Thanks
Alex A Skinner
Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
1.7

What operating system and processor architecture are you using (go env)?
Linux amd64

I'm pretty sure this is a dup of the discussion we're having in #16865. Do you agree?

Yep, missed that sorry.  We're getting production errors now and I'm sure others will too with the move to 1.7.  This can be closed as dupe, but this behavior described in both tickets is unexpected and really should be remedied.
Thank you

Thanks.


1Password issuer

Add all the stuffz here
Hello I like your browser but the idea that I have to type my 1password (master) every time is not good. Please let me know if you need anyone to test it.
Mac M3. Thanks

Ok thanks but does not do that in chrome froop safari but it’s a real pita with this. I like it but wanted to share my experience. Thanks
--
Sent from iPhone

On giovedì, feb 22, 2024 at 06:40, Mike423439 @.*** @.***)> wrote:
Dear Mac M3,
This is not a behavior of the browser. This is a behavior of 1Password
1Password browser extension will always lock when the browser is exited regardless of lockout settings.
—
Reply to this email directly, view it on GitHub (https://github.com/Alex313031/thorium/issues/555#issuecomment-1959592416), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AA27YG6FWOH3EF3BLHKLOE3YU5KFDAVCNFSM6AAAAABDHRMZNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJZGU4TENBRGY).
You are receiving this because you authored the thread.Message ID: @.***>

I also have this issue. It occurs because the extension can't "find" the desktop application in Thorium.

This does not occur in Chromium, so this is a Thorium-specific issue.

See https://github.com/Alex313031/thorium/issues/363


PROVISIONING_PROFILE_SPECIFIER and PROVISIONING_PROFILE changes in Xcode 8.3 causing white labelled app to fail to sign

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Since the migration from XCode 8.2.1 -> 8.3, our white labelled app (which we parameterize app identifier and app name among other things) now fails to sign.
The relevant piece of our Fastfile is this:
        # ensure there is a private key and signing cert available for the app.
        match(
            app_identifier: newAppId,
            git_url: 'ssh://some/path/to/git/repo.git',
            type: 'appstore',
            username: '[redacted]'
        )

        # ensure there is a provisioning profile available to build and sign the app with
        sigh(
            app_identifier: newAppId,
            provisioning_name: "match AppStore #{newAppId}"
        )

        # update app identifier so can sign with newly obtained provisioning profile
        update_app_identifier(
            xcodeproj: Paths::PROJECT_PATH,
            plist_path: Paths::PLIST_FILE_PATH,
            app_identifier: newAppId
        )

        # set the project provisoning profile UUID to the mobile provisioning profile obtained from sigh
        update_project_provisioning(
            xcodeproj: Paths::PROJECT_PATH,
            target_filter: '.*',
            profile: "./AppStore_#{newAppId}.mobileprovision"
        )

        # finally, build app
        gym(
            scheme: Schemes::APP_SCHEME,
            workspace: Paths::WORKSPACE_PATH
        )

after getting to the end of gym, and starting to sign, there are a bunch of these spat out:

❌  Code signing is required for product type 'Application' in SDK 'iOS 10.3'

XCode 8.3 and Apple's related build tooling now seems to use BOTH the PROVISIONING_PROFILE_SPECIFIER and PROVISIONING_PROFILE TOGETHER to determine if a particular provisioning profile can be used.
We have confirmed this by after running the script up until gym, we have manually changed the PROVISIONING_PROFILE_SPECIFIER as well. After this change, code signing works as it did.
Because not everyone white labels their apps, may I suggest the way to 'solve' this problem is to add a profile_specifier flag to the update_project_code_signing action? it doesn't seem that difficult, and I am happy to perform the change if this is the agreed approach.
For now, to unblock our builds, we are just regexing our 'white-labelled' specifier with an injected replacement.
Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12

Ruby
2.2.4

Bundler?
false

Git
git version 2.11.0 (Apple Git-81)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.12 (16A323)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
true

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.28.3
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

CFPropertyList
2.3.5

activesupport
<IP_ADDRESS>

addressable
2.5.0

babosa
1.0.2

bigdecimal
1.2.6

bundler
1.14.3

claide
1.0.1

colored
1.2

colored2
3.1.2

commander
4.4.3

commander-fastlane
4.4.4

domain_name
0.5.20161129

dotenv
2.2.0

excon
0.54.0

faraday
0.11.0

faraday-cookie_jar
0.0.6

faraday_middleware
<IP_ADDRESS>

fastimage
2.0.1

gh_inspector
1.0.3

google-api-client
0.9.26

googleauth
0.5.1

highline
1.7.8

http-cookie
1.0.3

httpclient
2.8.3

hurley
0.2

i18n
0.7.0

io-console
0.4.3

json
1.8.1

jwt
1.5.6

libxml-ruby
2.9.0

little-plugger
1.1.4

logging
2.1.0

memoist
0.15.0

mime-types
3.1

mime-types-data
3.2016.0521

mini_magick
4.5.1

minitest
5.4.3

multi_json
1.12.1

multi_xml
0.6.0

multipart-post
2.0.0

nanaimo
0.2.3

nokogiri
1.5.6

os
0.9.6

plist
3.2.0

power_assert
0.2.2

psych
2.0.8

public_suffix
2.0.5

rake
10.4.2

rdoc
4.2.0

representable
2.3.0

retriable
2.1.0

rouge
1.11.1

rubygems-update
2.5.2

rubyzip
1.2.0

security
0.1.3

signet
0.7.3

slack-notifier
1.5.1

sqlite3
1.3.13

terminal-notifier
1.7.1

terminal-table
1.7.3

test-unit
3.0.8

thread_safe
0.3.5

tty-screen
0.5.0

tzinfo
1.2.2

uber
0.0.15

unf
0.1.4

unf_ext
<IP_ADDRESS>

unicode-display_width
1.1.3

word_wrap
1.0.0

xcodeproj
1.4.4

xcpretty
0.2.4

xcpretty-travis-formatter
0.0.4

generated on: 2017-04-24

Thanks!

Been having the same issue https://github.com/fastlane/fastlane/issues/8821#issuecomment-292876401

@glesage, thanks for the ref! I did have a look at your ticket before creating one, and it didn't seem that @SiberianCMS was discussing about the ProvisioningStyle = Manual; stuff was related.
Only if you update both the PROVISIONING_PROFILE and PROVISIONING_PROFILE_SPECIFIER to the profile of the newly updated PRODUCT_BUNDLE_IDENTIFIER which is altered by running update_app_identifier as per the above snippet of a Fastfile

I see, I think what you posted is actually a step further than I was at the time (:
I'll have to do the regex approach as well for now

@jdouglas-nz @glesage I'm a bit of an amateur using Fastlane within a white label context as well. Any chance either of you would be willing to share your final work around?
I'm not a regex pro and would love to unabashedly stand on the shoulders of giants to get around this issue.

@jdouglas-nz @glesage nix that request. I was able to get everything working with:
match(app_identifier: @bundle_id, type: "appstore", username: @user)

    # ensure there is a provisioning profile available to build and sign the app with
    sigh(
      app_identifier: @bundle_id,
      provisioning_name: "match AppStore #{@bundle_id}",
      username: @user
    )

    # set the project provisoning profile UUID to the mobile provisioning profile obtained from sigh
    update_project_provisioning(
        xcodeproj: "EventCast.xcodeproj",
        target_filter: '.*',
        profile<EMAIL_ADDRESS>    )

(variable names my own of course)

I'm having the same issue as this and, at least in my case, it cannot be resolved by using sigh as I do not have direct access to the ADC account.
Are there any other ways to set the 'PROVISIONING_PROFILE_SPECIFIER' in the .xcodeproj correctly?

I'm pretty sure the update_project_provisioning action needs to be updated to set the PROVISIONING_PROFILE_SPECIFIER as well as PROVISIONING_PROFILE.
See: update_project_provisioning.rb Line 66
Can someone else validate that this is the right approach? I can put a PR together but what to confirm this is the right approach.

Ok, I've found a workaround but to be honest I'm pretty surprised how difficult it is to build an app with a different set of signing details. Here's what I have now that "appears" to be working. (The real test will be tomorrow when I actually upload to the AppStore).
desc "Create IPA for AppStore submission"
lane :prod_release do

  update_app_identifier(
    plist_path: "TestApp/Info.plist",
    app_identifier: "com.example.appname"
  )

  update_project_team(
    path: "TestApp.xcodeproj",
    teamid: "ABCDE12345"
  )

  update_project_provisioning(
    xcodeproj: "TestApp.xcodeproj",
    profile: "~/Library/MobileDevice/Provisioning\ Profiles/007e69e4-1234-5678-9012-3ch008c95da1.mobileprovision",
    target_filter: "TestApp",
    build_configuration: "Release"
  )

  gym(
    codesigning_identity: "iPhone Distribution: Example (ACBDE12345)",
    export_method: "app-store",
    export_options: {
      provisioningProfiles: {
        "com.example.appname" => "AppStore com.example.appname"
      }
    },
    xcargs: "PROVISIONING_PROFILE_SPECIFIER='AppStore com.example.appname'"
  )
end

I definitely think there is room here for a specific fastlane action that handles all of this mess. Something like:
desc "Create IPA for AppStore submission"
lane :prod_release do

  update_code_signing_options(
    path: "TestApp.xcodeproj",
    app_identifier: "com.example.appname",
    team_id: "ABCDE12345",
    provisioning_profile: "AppStore com.example.appname",
  )

  gym(
    codesigning_identity: "iPhone Distribution: Example (ACBDE12345)",
    export_method: "app-store",
  )
end

Or am I just over simplifying this?


API Reference missing?

Hello. It seems all links to the API reference are missing. For example: https://fsprojects.github.io/Cricket/reference/cricket-actorconfiguration-1.html
And they would be very helpful to try to learn Cricket faster. Do the pages exist somewhere?

I must of broken the build at some point, I'll look at re-instating this, this weekend.

Any movement on this issue?

I'm just having a look at this now, It seems like there is an issue in FSharp.Formatting. The is preventing the docs being created.

Should be available now @ http://fsprojects.github.io/Cricket/reference/index.html. Reopen if not

Thanks bud


Bug fixes for cloud=>self hosted data export.

This PR fixes a couple bugs reported when importing a Cloud export onto a self-hosted system.

@mateuszmandera FYI


sold: init at 0-unstable-2024-02-22

Description of changes
Adding a package for sold, a fork of mold that supports linking for iOS & MacOS platforms.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I think you forgot to add this package to all-packages.nix like mold, and sold-wrapper like mold-wrapped
And referring to mold this package should be located at pkgs/development/tools/sold

That isn't necessary as it's added in the new by-name tree.

That isn't necessary as it's added in the new by-name tree.

Seems that https://github.com/bluewhalesystems/sold had been archived & deprecated a few days ago in favor of Apple's new linker (included in Xcode 15 and newer)


feat(topology): add affinity key to core

PR https://github.com/openebs/mayastor-control-plane/pull/846 raised for the same.


How to verify problem 8?

I see the HTML outputted is the right answer but by running learnyoureact verify I get the following error:

I'm getting this same issue too

Same issue here to

+1

same issue!

Same problem ! can anyone help ?

+1

Same issue. Even copy/pasted the solution files and got failed verification

is still still happening? what are the solutions that failed verification? I've just completed the tutorial and have been able to verify properly.

I copy pasted the same things as the solution repo and still had verification errors. I don't know why :(

this is my package.json:
{
"name": "learnyoureact",
"version": "1.0.0",
"description": "",
"main": "index.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1"
},
"keywords": [],
"author": "",
"license": "ISC",
"dependencies": {
"babel": "^5.8.23",
"body-parser": "^1.15.2",
"express": "^4.14.0",
"express-react-views": "^0.9.0",
"react": "^0.14.8",
"react-dom": "^0.14.8"
}
}
are the dependencies the same?

Mine's this:
{
"name": "learnyoureact",
"version": "1.0.0",
"description": "",
"main": "index.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1"
},
"keywords": [],
"author": "",
"license": "ISC",
"dependencies": {
"babel": "^5.8.23",
"babel-preset-es2015": "^6.9.0",
"babel-preset-react": "^6.11.1",
"babelify": "^7.3.0",
"body-parser": "^1.15.2",
"browserify": "^13.0.1",
"express": "^4.14.0",
"express-react-views": "^0.9.0",
"react": "^15.2.1",
"react-dom": "^15.2.1"
}
}
It would seem the only difference is react and react-dom. Do you think that's the cause?

+1. I'm getting the same error during verification as in the screenshot above. Knowing little about React, it does look like something caused by latest React version producing different HTML than the workshop expects.

yes I would try to install that version that i have posted considering how react has been going through some major changes

Just a quick note to say I resolved this problem by deleting my node_modules folder, replacing the dependencies list in my local package.json with the one from the repo itself then running npm install. The problem verifies OK after doing that.

Funciona, borrando la carpeta de node_modules luego npm install como dice @rossb

setting your package json to 0.14.0 for react and react-dom packages (effectively, removing the ~), also works.  Maybe there should be a notice put on the read me about this?

@rossb 's suggestion solves it. It was literally right in front of us lol! Thanks for the reality check, mate.

I have a problem with this one, I managed to verify solution, code is being copied from solutions dir. But I see proper html file only after first page load and then when I reload or I try to navigate to /bundle.js I see this error in console:
events.js:160
      throw er; // Unhandled 'error' event
      ^

TypeError: Cannot read property 'error' of undefined (While processing preset: "/Users/jankowalski/projects/nodeschool/learnyoureact/node_modules/babel-preset-es2015/index.js") while parsing file: /Users/jankowalski/projects/nodeschool/learnyoureact/app.js

@hanskoff
I got the same pb as your's ... but following rossb's solution I got it right ...
just verify after the first time and it should be ok (or if it's still out, go to next CSS2 and verify back)

im getting the same error as @hanskoff
when I verify it works but when I run  node program.js
I get the error and can not load the page at localhost:3000

@krischamp @nharrisanalyst
Maybe #111 will be of some help.

Changing the following worked for me.
require('babel/register')({
    ignore: false
});

replace with:
require('babel/register');

Hi,
I'm having an issue with learnyoureact Isomorphic exercise. I continue to get an error:
var body = DOM.body;
^
Cannot read property 'body' of undefined
Can you assist? Thanks

@mjohnson518 Have you tried this: https://github.com/workshopper/learnyoureact/issues/103#issuecomment-239604351?


junit formatter problem

Hello,
When I tried to launch my behat test suite uniquely with the junit formatter I got this error :
[Symfony\Component\Debug\Exception\ContextErrorException]
Warning: mkdir(): Invalid path
behat version : dev-master

What is the command you used exactly ?

bin/behat --format junit

You need to add the option --out=build/logs

I works Thanks.
I close the issue

I had the same issue. Wouldn't be better to catch the error?

@jbenoit2011 indeed, fancy sending a PR?


Benchmarks and performance improvements

I've added benchmarks project and improved device detection hot path performance.
Core idea:

do not lowercase user agent string for every check (major perf boost)
do not allocate delegates on array comparison (minor perf boost)

Hardware and Software
BenchmarkDotNet=v0.12.1, OS=Windows 10.0.17763.1158 (1809/October2018Update/Redstone5)
Intel Core i5-3570K CPU 3.40GHz (Ivy Bridge), 1 CPU, 4 logical and 4 physical cores
.NET Core SDK=3.1.102
  [Host]  : .NET Core 3.1.2 (CoreCLR 4.700.20.6602, CoreFX 4.700.20.6702), X64 RyuJIT
  LongRun : .NET Core 3.1.2 (CoreCLR 4.700.20.6602, CoreFX 4.700.20.6702), X64 RyuJIT

Before:

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Detect
92.13 us
0.397 us
2.053 us
44.9219
-
-
137.66 KB

After: only ToLower()

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Detect
30.94 us
0.195 us
0.173 us
7.6294
-
-
23.51 KB

After: do not allocate delegates +  ToLower()

Method
Mean
Error
StdDev
Median
Gen 0
Gen 1
Gen 2
Allocated

Detect
29.03 us
0.094 us
0.486 us
28.95 us
6.2256
-
-
19.13 KB

Removed another large chunk of allocations. Tablet and Mobile collections were created every time they were accessed.
Current result is:

Method
Mean
Error
StdDev
Gen 0
Gen 1
Gen 2
Allocated

Detect
25.86 us
0.497 us
0.629 us
4.4250
-
-
13.63 KB

Very impressive improvement you have contribute to this library. Also open another world of knowledge that I never explored before.
Thank you for your help and very much appreciated your contribution.

Very impressive improvement you have contribute to this library. Also open another world of knowledge that I never explored before.
Thank you for your help and very much appreciated your contribution.

Thank you for the swift response!


feat(FluentNumberField): type based validation of input values in FluentNumberField Component with tests

Pull Request
📖 Description
FluentNumberField extends FluentInputBase where the input value type can be int, long, etc. As of now it is possible to set up lower Max value than Min.
This PR brings validation for these types.
When the provided value is outside of the range the type has the Parse method throws Format exception.
The fact that Max should be bigger than Min is checked by a validation method, in this case it throws Argument exception.
The validation is added to FluentInputBase class hooked up to the OnParametersSet method.
🎫 Issues
Closes #350
👩💻 Reviewer Notes

📑 Test Plan

tests are provided for the validation and other functionalities of the component

✅ Checklist
General

[x] I have added tests for my changes.
[x] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[ ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[x] I have modified an existing component

⏭ Next Steps
To be considered:

during validation we throw two types of exceptions (1, FormatException due to that the string to be parsed is outside of the range of the type, 2, ArgumentException when Min is bigger than Max). In my opinion worth to throw only one and make the situation clearer for the customer. WDYT?

I say let's make it only throw 1 exception.
The validation for this type is only applicable for the number field. Maybe better to not 'pollute' FluentInputBase with these validations and just put them in FluentNumberField?


Data Prep from tabular time series data

Hi team,
Great work; it looks very promising!
I am currently working on a time series project with data in tabular format. I would like to know if this package includes few shot,if so does this package includes functionality for converting tabular data to images. If not, how can I create images from tabular text data?  I have thousands of stores and hundreds of products, so saving line plots for each store-product combination is one option. Will this approach help my VisionTS model learn patterns more effectively and achieve better results than tabular-based models?

Hi @Manjubn777
Thank you for your inquiry!
VisionTS is a numerical forecasting model. You need to convert your tabular data into numerical sequences. Specifically, the input to VisionTS consists of a series of numerical values representing a length-$L$ historical window, and VisionTS will output their future forecast (length-$H$). You can refer to demo.ipynb for guidance on setting $H$ and $L$. You don't need to convert the numerical sequences into images, as we have already implemented it.
If you require few-shot fine-tuning, the forward method in the VisionTS module is differentiable, allowing for standard PyTorch backpropagation for tuning. By default, only the LayerNorm parameters will be updated. Notably, fine-tuning may not necessarily yield better results than zero-shot learning, as it depends on your scenario.

That makes sense and I will try to replicate on my data.
We have been trying few foundational models for our few QSR use cases however we have not got good results with zero shot and after fine tuning we did see a little improvement but again not better results than our current models.
Thanks for your immediate response @Keytoyze I appreciate it.


PMM-7 fixed rbac default argument. and removed 2.11.0 from matrix.

https://github.com/percona/pmm-ui-tests/actions/runs/5681322363


Bug: Every time you bozos update the home bridge plugin you break things!!

Device and Model
your mother
Node.js Version
12.21.21
NPM Version
7.22.2
Homebridge Version
1.3.4
Homebridge Switchbot Plugin Version
4.1.6
Homebridge Config UI X Plugin Version
No response
Operating System
MacOS

Well not seeing any logs provided so not much we can do to help you.

Homebridge logs say that the plugin can't connect to several AWS servers. The usual BS with Switchbot. Homebridge logs also show that this plugin and the Switchbot ecosystem are the most GARBAGE ecosystem any person can install in their homes. That's what the logs say.

Those kind of logs were never coded into the plugin, so you must be using the wrong plugin.

Been using the same Homebridge Plugin for almost a year now. It's attempting to connect to several AWS servers in Virginia and timing out.
                Crypto Knight Crypto & NFT Enthusiast(800) 555-0199(800<EMAIL_ADDRESS>King Am I Creek, Somewhere, NY 07001

On Dec 22, 2024, at 7:59 AM, Donavan Becker @.***> wrote:
Those kind of logs were never coded into the plugin, so you must be using the wrong plugin.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Those kind of logs were never coded into the plugin, so you must be using the wrong plugin.

[SwitchBot] Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/21/2024, 9:50:34 PM] [SwitchBot] Error making request: Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/21/2024, 9:50:47 PM] [SwitchBot] Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)
[12/21/2024, 9:50:47 PM] [SwitchBot] Error making request: Failed to get device status: Connect Timeout Error (attempted addresses: <IP_ADDRESS>:443, <IP_ADDRESS>:443, <IP_ADDRESS>:443, timeout: 10000ms)

Been using this same plugin for a year unfortunately, and the logs point here via a URL to report bugs. So yes, this is YOUR plugin. Wish I could flash the bot with custom firmware. The switchbot app and plugin have been the buggiest out the bunch in my smartphone.

Also generates this error: "[SwitchBot] Bot: Buzzer Unknown statusCode: undefined, Submit Bugs Here: https://tinyurl.com/SwitchBotBug, statusCode: undefined"
Plugin at least worked half the time before, now it refuses to work on Homekit at all. Been using this same plugin for a year unfortunately, and the logs point here via a URL to report bugs. So yes, this is YOUR plugin. Wish I could flash the bot with custom firmware for direct integration into HomeKit, but unfortunately that's not possible. The switchbot app and plugin have been the buggiest out the bunch in my smartphone.

this is resolved in the next release.

I checked the Beta 4 and the same issue persists. So I have to wait till the next release for this to be resolved? Is this a normal thing? A device's TOTAL functionality breaks with an update and people have to wait till the next release for a fix?!? Some people use these devices for pretty important stuff like opening doors, electronic gates, etc etc, not just to turn on their coffee pots. This is a total joke.

I have you have a Merry Christmas.


[Task]: POC for pixel based resizing in Auto Layout

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
POC details: https://www.notion.so/appsmith/POC-Resizing-widgets-without-grid-1563f0d5c95c4d09b1660d83943e7203

Ability to resize widgets pixel based
Update widget width and height properties
Auto Layout resize module autonomy

The feasibility for pixel based resizing has been proved. so closing this POC.


fix: disabled custom LSP requests

First thanks for the implement for Volar!
getDocumentSelectionRequest and showReferencesNotification is custom LSP request that specially handle in language client:

https://github.com/johnsoncodehk/volar/blob/12c346571719dea4921267247beaf0f93fc402e2/extensions/vscode-vue-language-features/src/features/showReferences.ts#L7-L20
https://github.com/johnsoncodehk/volar/blob/12c346571719dea4921267247beaf0f93fc402e2/extensions/vscode-vue-language-features/src/features/activeSelection.ts#L7-L17

There should be disabled if do not have support these custom request.

Hi there! Thank you for your work on Volar and here with this PR!
I'll see later if we can provide support for these custom request. For the moment I'll accept your PR and disable them.
Thanks 😆


vendor: add a vendor directory

This does a couple of things:

Keeps us inline with other "build" repos - buildx, buildkit, etc.
Makes dependabot for new versions of syft a lot easier to review, since we can easily analyze exactly what code that we are using is changing.

https://github.com/docker/buildkit-syft-scanner/blob/3dc2dd050a199829cd46c32f323a46a8fb94fbb5/Dockerfile#L43-L46 needs to be removed
Also needs https://github.com/docker/buildx/blob/daba16f4be78f4bb6bb6ac73ac5c5615023a2bd7/Dockerfile#L20

Woop, thanks @crazy-max :heart:


An APP

I have written an app for sending and receiving files , now i want to use custom file browser or file chooser in it but not able to do it . I can show my source code if anyone wants it so one check the what am i doing wrong and what's the main error and how can i make it better ?

This place is for bug reports, not support questions. Please ask on IRC or the mailing list. See https://kivy.org/#support.

@Avernch7, your account has been limited from interacting with Kivy repositories for a week.
Previous requests for not using the issue tracker as a support forum:
https://github.com/kivy/kivy/issues/4215#issuecomment-214010942
https://github.com/kivy/kivy/issues/4205#issuecomment-212953119


poweranks breaks towny and other plugins that use permsissions

the way how powerranks set permission makes it so that any other plugin that adds permissions just doesnt add permissions, this breaks towny, luckperms,permissionsex and MANY other plugins, developer fix this
for as long as this is an issue, and until the developer makes a patch that fixes this and you see that in the release notes: DO NOT USE THIS PLUGIN

Closed as duplicate of #60


Beaufort Cipher Implementation

Implemented the Beaufort Cipher as my first project. +A readme document with explanations and examples regarding the encryption method.

I've checked your project and that was fabulous and marvellous! Congrats on your new role as FCA intern !
Good Luck, Have Fun, Code a Lot!


release updated mysql dependency to avoid bignumber.js bug?

is it possible for to do a hotfix release with the updated mysql version dependency so that the bignumber.js "critical bug" is avoided?
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>critical bug fixed in v2.0.4
<EMAIL_ADDRESS>node_modules/db-migrate
├── ...
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>the current ~2.7.0 version would be great.   https://github.com/db-migrate/node-db-migrate/blob/master/package.json#L34
thanks!

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/22929626-release-updated-mysql-dependency-to-avoid-bignumber-js-bug?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

Hey,
sry for beeing so late. I had a few tough weeks and needed to manage some kind of things and am back working effectively now :)
I will check if the upgrade has any negative effect and if not then releasing an upgrade.

updated mysql dependency released with v0.9.18


Add data-toggle in squirrel state (flipper-ui) 💅

Motivation
At the first time I saw the features in the Flipper-ui, I was a little confused about the colors of the squirrel, and when my partner saw it, he tried to click on the squirrel. So I had a simple ideia to put a data-toggle with the feature state name.
Solution
It looks like this:

@jnunemaker just on hover :)


wasn't working

ive tried reinstalling and everything but nothing works, i checked the task manager to ensure something is running in the background, but there are nothing happening at al, even with the test pack where the config menu said no errors were detected

this is a delayed response but if you are still having issues feel free to contact me elsewhere or open another issue- unfortunately due to the massive amount of variances from setup to setup there isn't a clear path on what is going wrong on your install


Fix Laplace and EP GLMM when K is available but G is not set

Current version has this bug for both Laplace and EP versions of GLMM:
from numpy import dot, zeros
from numpy.random import RandomState
random = RandomState(0)
nsamples = 10
X = random.randn(nsamples, 2)
G = random.randn(nsamples, 100)
K = dot(G, G.T)
y = zeros(nsamples)
y[1:5] = 1
from fastlmm.inference.ep import EPGLMM_N3K1 as GLMM_Method
model = GLMM_Method('logistic')
model.setX(X)
model.setK(K)
model.sety(y)
model.optimize()

output:
    self._K += self._sig02*(dot(self._G0, self._G0.T))
AttributeError: 'NoneType' object has no attribute 'T'

This patch fixes them.

I've accepted the push. Thanks! If you can, I'd love a new test in fastlmm\inference\tests\test.py
that failed before and that you fixed. Thanks, again.


add options for verbose, prefix and path

@KesterTong with that yo should be able to install in the right location :-P

Thanks, that fixes installation on my desktop.


Invalid password was supplied to the broker

I keep getting this error whenever I try to pass a large object from one server to another. The weird part is one of the servers does this all the time with no problems and the other one seems to blow up our staging server every time it sends the same packet.. They are both running socketcluster:5.0.16

The one on the left is working great and the one on the right, which by the way is a copy paste from the working one, keeps breaking my staging server lol.

well I rebuilt the server and now it works great.. no idea why.. thanks!

It's back again. This is a different case though. Now it seems to do it after the server has been running for maybe 3-4 days. I just reset it and it works great for 3-4 days and starts doing it again. It's not really that big of a deal because it's a staging server but it's still kind of annoying. @jondubois do you know why this may be happening? THanks

@happilymarrieddad Can you give be the version numbers of the following dependencies for your staging env and also your production env (to make sure that they're the same):

socketcluster-server
sc-broker

Are you setting a custom secretKey or changing it at any point during runtime? (you might want to search through your code for references to secretKey - This shouldn't change at runtime - To change it, you should restart your entire SocketCluster instances (all processes).
Basically it means that your worker process has a different secretKey from your broker process. The secret key is passed from the master process to all children processes (both workers and brokers).
I sounds like it's caused by:

The options.secretKey (in either master, worker or broker) gets changed in your code somewhere during runtime - And this causes the mismatch. Or;
For some reason the worker and broker processes have different parent master processes... Maybe a child broker process was left over from a previous deployment and somehow SC thinks it belongs to the current instance.

I will try to reproduce on my end.

@happilymarrieddad A good test to do would be to add a console.log('SC SECRET KEYS:', command.secretKey, scBroker.secretKey) on line 187 of sc-broker/server.js just above the line result.error = 'Invalid password was supplied to the broker'; - You should log it when everything is working fine, then wait a few days for the error to show up and check if any of the secretKey values have changed since last time (they should never change unless you restart the SC master process).
If one of them did change, it would tell us whether it's the broker or the worker which has he wrong key.

Ok, I'm pretty sure you are right about the leftover deployment. We're good now. Thanks!

@happilymarrieddad how did you terminate SC though? Maybe there is an issue there... It shouldn't leave behind old processes.


Pass AWS environment variables to docker terraform

@mat285 we are moving away from docker for running this. It is not well documented, but you can jus set "docker": false in your fogg.json.

Codecov Report

Merging #211 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #211   +/-   ##
=======================================
  Coverage   57.99%   57.99%
=======================================
  Files          11       11
  Lines         807      807
=======================================
  Hits          468      468
  Misses        266      266
  Partials       73       73

Continue to review full report at Codecov.

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

Awesome, thanks for the heads up! Is there any other docs I can take a look at?

@mat285 not really. I have a task do to a docs pass in our next sprint, so hopefully it will get better soon.


Feature request: possibility to change background color

It would be great if I could change background of the outline window (not preview window) so I can have it visually distinct from my code window.
For example in Nvim Tree provides NvimTreeNormal highlight group. Can we add something similar here please?

same here, it would be nice to have this feature


Practitioner Reference not picked up in Care Team

https://inferno.healthit.gov/suites/test_sessions/efcbf834-1ae7-4914-a0eb-a5f27864cbef
In our test, our encounter reference test fails; however, condition returns a reference to encounter that the encounter service can return form.  Are we formatting something incorrectly?

https://github.com/onc-healthit/onc-certification-g10-test-kit/wiki/FAQ#q-how-does-inferno-test-us-core-encounter-resource

@Jammjammjamm  that was perfect, thank you


Add generic docker-based building for multiple platforms

This forks the yelppack directory into a build directory and makes it a much more generic Docker-based build process that will output binaries for linux64, darwin64, and windows64.
This will run well on Linux, OS X, and very likely Windows.

@bobtfish I created this PR to create a non-YELP specific way to build this for multiple platform utilizing containers. It is based on the yelp pack work, but more generic (yet a bit more robust then the simple Makefile.

Thanks!


I need help

Describe your bug here.
i am in a rush so i need help on fixing this, i was trying to compile psych engine 0.6.2 but this shows up. I don't know what i did to mess with something or idk but i need help fixing this, i tried looking up some issues about it and i couldn't find anything about it
Command Prompt/Terminal logs (if existing)
C:/HaxeToolkit/haxe/lib/linc_luajit/git/llua/Convert.hx:92: characters 11-12 : Missing type declaration

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no, i didn't edit anything

https://github.com/ShadowMario/FNF-PsychEngine/issues/12876 u checked this already?

yeah i already did that but it didn’t work but, i recompiled the build by delete all the haxe files i have and reinstalling them and compile them again so uh, we’re all good


HHH-14044 : Extract code from Loader into protected methods

https://hibernate.atlassian.net/browse/HHH-14044
This is for Hibernate Reactive:
https://github.com/hibernate/hibernate-reactive/issues/120.
The changes:

extract code from Loader#prepareQueryStatement that binds a PreparedStatement into protected method named #bindPreparedStatement;
change private processResultSet( ResultSet, RowSelection, LimitHandler, boolean, SharedSessionContractImplementor) to protected preprocessResultSet(...), since it really does preprocessing. (`#processResultSet(ResultSet , QueryParameters, SharedSessionContractImplementor, boolean, ResultTransformer, int, List) actually does the processing).

Great! This looks excellent, @gbadner. I have a couple of comments and questions but I want to actually try it out first. Hold on.

Applied on master and 5.4
Thanks @gbadner


Proto Cluster with Consul shows connection errors

For local development, I've configured Proto Cluster with Consul support using random ports.
The cluster is working and I have two nodes:

An API Server which acts as a bridge (WebSocket to Actor cluster bridge);
A Lobby room server which is responsible for the game;

All nodes are communicating with each other normally, but the Lobby server is showing some RPC exceptions like if it wasn't able to communicate with the API server, but both are green on Consul as you can see in the second image.

This looks suspicious.
Why does it say localhost and not <IP_ADDRESS> which is what is registered in Consul?
Can you show the remote config for both nodes? are there any advertised host set?

Could it be that you are manually constructing a PID that points to localhost and try to communicate with that?

This looks suspicious. Why does it say localhost and not <IP_ADDRESS> which is what is registered in Consul? Can you show the remote config for both nodes? are there any advertised host set?

So, for local development, which is this case, I'm using the GrpcNetRemoteConfig.BindToLocalhost()
And consul is being configured using the default settings:

Note, this also happens with the K8 provider but usually there it becomes more stable with time:

Is there any chance you are sending messages that are not supported by the serializers?
I don´t quite see how there can be RpcExceptions for nodes that are otherwise healthy.
Are the logs above the only logs that appear? nothing else?

Yes, I'm using custom serializers but, in theory, I should have received a warning if an unknown message was attempt to be serialized. I've put a fallback serializer to raise a warning if I'm serializing an unknown message type, and I can't see any warnings in the logs...
public sealed class AnyMemoryPackableSerializer(ILogger logger) : ISerializer
{
    public ByteString Serialize(object obj)
    {
        var bytes = MemoryPackSerializer.Serialize(obj.GetType(), obj);

        logger.LogWarning(
            "Serializing {Type} by AnyMemoryPackableSerializer resulting in {Bytes} bytes",
            obj.GetType().Name,
            bytes.Length
        );

        return bytes.ToByteStringDangerously();
    }

    public object Deserialize(ByteString bytes, string typeName)
    {
        logger.LogWarning(
            "Deserializing {Type} by AnyMemoryPackableSerializer loading {Bytes} bytes",
            typeName,
            bytes.Length
        );

        var type = Type.GetType(typeName)!;

        return MemoryPackSerializer.Deserialize(type, bytes.Span)
            ?? throw new InvalidOperationException($"Failed to deserialize message of type {typeName}");
    }

    public string GetTypeName(object message)
    {
        return message.GetType().AssemblyQualifiedName!;
    }

    public bool CanSerialize(object obj)
    {
        return obj is IMemoryPackFormatterRegister;
    }
}

   private static GrpcNetRemoteConfig AddSerializers(GrpcNetRemoteConfig remoteConfig, ILoggerFactory loggerFactory)
    {
        var serializerId = 2;
        var priority = 100;

        return remoteConfig
            .WithProtoMessages()
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<GameEvent>(typeName: nameof(GameEvent))
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<GameCommand>(typeName: nameof(GameCommand))
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<ILobbyRoomActivatorMessage>(
                    typeName: nameof(ILobbyRoomActivatorMessage)
                )
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<ILobbyRoomMessage>(typeName: nameof(ILobbyRoomMessage))
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<ILobbyParticipantMessage>(
                    typeName: nameof(ILobbyParticipantMessage)
                )
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority--,
                serializer: new MemoryPackableSerializer<RemotePid>(typeName: nameof(RemotePid))
            )
            .WithSerializer(
                serializerId: serializerId++,
                priority: priority,
                serializer: new AnyMemoryPackableSerializer(
                    logger: loggerFactory.CreateLogger<AnyMemoryPackableSerializer>()
                )
            )
            .WithSerializer(
                serializerId: serializerId,
                priority: -1, // NOTE -1 means after all other serializers but before the JsonSerializer
                serializer: new NonPackableErrorSerializer(
                    logger: loggerFactory.CreateLogger<NonPackableErrorSerializer>()
                )
            );
    }


Fix: AWS Well Known Labels now use - instead of . to align with upstream

Fixes #
Description
See discussion thread: https://kubernetes.slack.com/archives/C02SFFZSA2K/p1655266249940299
How was this change tested?

Unit, Manual

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

AWS Well Known Labels now use - instead of . to align with upstream

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Need to change examples/provisioner/general-purpose.yaml as well. I merged in a fix that made it match the code this morning.

I thought I had ctrl+f to find references but my filter was set to *.go. Nice catch.


Set mutator is being called multiple time in create

Laravel Version: 9.6.0
PHP Version: 8.1.4

Description:
I have a Service Controller with store method:

And inside Service model:

And inside File Controller:

When I am trying to create new service icon file is creating many times:

Why Laravel call this set method multiple times ?
On the other side I tried to use setIconAttribute method:

The file created once but icon column in database is null !
So where is the problem and what should I use for scenarios like this?

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Thanks @korkoshko ,
Using below works for me
Attribute::make(...)->withoutObjectCaching()

Do you think upload file on saving with additional "isDirty" checking is better than upload file in mutator ?


Update study submission process

Summary Change Description
Simplify the study submission process by having the researchers add things to Google Drive themselves and simply point us to the Google Drive folder instead of attempting to have them submit the files via the form.
Affected Issue Numbers

Resolves #318

Code Review Notes
Checklist

[x] All issue requirements satisfied (or no linked issues)
[x] Linting passes.
[x] Migrations created & committed (or no model changes)
[x] Tests implemented (or no code changes)
[x] All tests pass

For anyone else wanting to look at the page comparison to get a sense...

That's a good point @hepcat72. The text is more verbose, and perhaps that makes it more intimidating. I'd like to separate the comments on the specifics of the text from the comments on the process as a whole. Both are important, but I want to make sure we agree on the process.
One thing to note, the headers are much larger (not really intentional, just realized now the originals started at h4). Shrinking those would probably help.
Overall, I feel the text in TraceBase is too big, especially in the tables. In fact, I display it by default at 80% just to make it readable, but that's a separate issue.

Forgot to add one comment, I didn't intend to leave anything off about validation of the AccuCor files, I'll push an update to address that shortly. Also, I'm not aware of a "file limit issue" with the validation, but please let me know if there is one. I'm sure there is a practical limit at least, uploading hundreds of files will probably cause issues.

Also, I'm not aware of a "file limit issue" with the validation, but please let me know if there is one. I'm sure there is a practical limit at least, uploading hundreds of files will probably cause issues.

Well, I'm effectively referring to the practical difficulties.  I have some vague recollection of some set limit in Django, but I'm probably mistaken about that.  Suffice it to say, I was thinking of the difficulties in submitting a bunch of files.  Have we even tested a bunch of files to make sure it doesn't time out or something?

Thanks to feedback from @hepcat72, I used some collapsible elements (and modified some text) to make the page a bit more approachable. Could still be improved I think, but this seems improved to me.

Updated comparison


http2: disable priority forwarding

Too complex? It's two if statements...
The logging is set to debug, not very verbose IMO...


[BACKEND] Fix heap-use-after-free in createAsyncCopy

There are two tests that failed under AddressSanitizer:

test/TritonGPU/loop-pipeline.mlir
python/test/regression/test_functional_regressions.py

with an error:
==8475==ERROR: AddressSanitizer: heap-use-after-free on address 0x50c000bd0be0 at pc 0x557b03278847 bp 0x7ffd69b2c4a0 sp 0x7ffd69b2c498
READ of size 8 at 0x50c000bd0be0 thread T0
    #0 0x557b03278846 in getNextOperandUsingThisValue [third_party/llvm/llvm-project/mlir/include/mlir/IR/UseDefLists.h:43](https://cs.corp.google.com/piper///depot/google3/third_party/llvm/llvm-project/mlir/include/mlir/IR/UseDefLists.h?l=43&ws=aliia/3018&snapshot=215):58
    #1 0x557b03278846 in operator++ [third_party/llvm/llvm-project/mlir/include/mlir/IR/UseDefLists.h:322](https://cs.corp.google.com/piper///depot/google3/third_party/llvm/llvm-project/mlir/include/mlir/IR/UseDefLists.h?l=322&ws=aliia/3018&snapshot=215):39
    #2 0x557b03278846 in mlir::ResultRange::UseIterator::operator++() [third_party/llvm/llvm-project/mlir/lib/IR/OperationSupport.cpp:614](https://cs.corp.google.com/piper///depot/google3/third_party/llvm/llvm-project/mlir/lib/IR/OperationSupport.cpp?l=614&ws=aliia/3018&snapshot=215):5
    #3 0x557affde38c4 in operator++ [third_party/llvm/llvm-project/llvm/include/llvm/ADT/iterator.h:281](https://cs.corp.google.com/piper///depot/google3/third_party/llvm/llvm-project/llvm/include/llvm/ADT/iterator.h?l=281&ws=aliia/3018&snapshot=215):5
    #4 0x557affde38c4 in createAsyncCopy [third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp:117](https://cs.corp.google.com/piper///depot/google3/third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp?l=117&ws=aliia/3018&snapshot=215):26
    #5 0x557affde38c4 in createAsyncLoad [third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp:135](https://cs.corp.google.com/piper///depot/google3/third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp?l=135&ws=aliia/3018&snapshot=215):3
    #6 0x557affde38c4 in createAsynOps [third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp:501](https://cs.corp.google.com/piper///depot/google3/third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp?l=501&ws=aliia/3018&snapshot=215):5
    #7 0x557affde38c4 in mlir::triton::preProcessLoopAndGetSchedule(mlir::scf::ForOp&, int, mlir::triton::PipeliningOption&) [third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp:740](https://cs.corp.google.com/piper///depot/google3/third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/MatmulLoopPipeline.cpp?l=740&ws=aliia/3018&snapshot=215):7
    #8 0x557affe01c0c in pipelineLoop [third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/SoftwarePipeliner.cpp:76](https://cs.corp.google.com/piper///depot/google3/third_party/triton/lib/Dialect/TritonGPU/Transforms/Pipeliner/SoftwarePipeliner.cpp?l=76&ws=aliia/3018&snapshot=215):19
...

This is likely happening due to iterator being invalidated after alloc.erase().
This PR moves erases of allocations outside of a loop and fixes heap-use-after-free issue.
Do you know if there is an easy way to run the tests under sanitizers upstream? It would be handy if we can automate it, so we catch this kind of errors early on.

Any reason this is a draft? Can we merge it?

Any reason this is a draft? Can we merge it?

yes we can merge, I was only waiting until the checks finished running

Do you know if there is an easy way to run the tests under sanitizers upstream?

About this, I don't think we have enough CI resources to do that unfortunately.


The Flutter recipes yapf style requires yapf 0.30, and depot_tools has yapf 0.27

The .style.yapf in flutter.googlesource.com/recipes says
based_on_style = yapf
which only works in versions of yapf after 0.30.
This may have been needed to make tricium work.
depot_tools is pinning version 0.27 in its vpython wheel, but there is an issue filed to update this to 0.31, so this bug should be fixed by that: https://bugs.chromium.org/p/chromium/issues/detail?id=1306598
A workaround is to manually change the version number to 0.31.0 in depot_tools/yapf locally.
No further action needs to be taken, I think.
@godofredoc

Depot_tools has updated its yapf to 0.31.0.  The issue is fixed.

Reopening after incorrectly closing, still no action necessary on our part.  The cause of this breakage is https://github.com/google/yapf/pull/789 , where there was discussion that not having a transition plan would break users.


Bifacial snow model correction

-Fixes (https://github.com/NREL/SAM/issues/1739)
-Add rear POA to heat transfer balance equation for determining snow sliding
-scale snow cover losses by poa_front / total poa, as snow is not shading rear of module
-Add test coverage for snow/bifacial interactions
-Confirm that snow model calculates no snow coverage for vertical arrays.

Code looks good for irradiance and power production calculations. I wonder about the "DC power loss due to snow" output. In the released version, this was a useful indiciator of what % of the panel was covered:

now it's a bit noisier with the bifacial calculations:

Is it worth adding an additional output (maybe just to detailed PV) that matches the old output such that it's easier to see what the snow model is doing?

I think in pvsamv1 Subarray X Snow cover (0-1) is sufficient for that visual checking. Agreed on PVWatts, I can look at adding a similar output.


Drop quotes from input commands in guardian-admin.yml

This change removes quotation marks wrapped around arguments of the command provided from workflow_dispatch inputs.
From the Terraform documentation, it appears you can run commands with quotations around arguments like -replace="someresource.name". However, in the GitHub Actions environment, the inputs.command is treated as a literal string and is not expanded by bash, leaving the quotations as is. Guardian uses os/exec to run the Terraform commands, which

does not invoke the system shell and does not expand any glob patterns or handle other expansions, pipelines, or redirections typically done by shells

so we encounter errors if quotations are present around the Terraform arguments.
Fixes #321.

Open to alternatives 👋


⚠ Bofrost.lu has degraded performance

In 23b05d9, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1123 ms

Resolved: Bofrost.lu performance has improved in a2d9dc6 after 20 minutes.


Porting C++ Test Agent to 1.5.8

This PR update c++ test agent to work with 1.5.8 api with addition on L2 apis

Updating to 1.5.8
Refactoring
Feature file updates

Closing this PR since we moved to RC1. This PR work moved to PR: https://github.com/eclipse-uprotocol/up-tck/pull/90 with addition of rc1 compatibility


Storing json in wiki ?

Storing the JSON for shields.io in Gist is really smart, but this requires a few shenanigans.
Do you think it might be possible to store the json in a github wiki file?
Explanations
You can clone a repo's wiki (git clone git@github.com:owner/repo.wiki.git) and push files to it. Files pushed to a wiki are not visible in the repo's UI, but you can fetch them through the URL https://raw.githubusercontent.com/wiki/:owner/:repo/:file-path. This seems to work even if wiki is disabled for the project. (usually takes a few minutes for the URL to be updated but for the badges, I'd say it's usually not a problem, shields also does caching anyway)
Pros

You don't have to configure a gist. This makes the action much easier to configure
Also, you don't end up with multiple gists linked to repos around your gist account

Cons

You mess with the repo's wiki. Not that much that it might be problematic, I think, but still.
There's no web UI letting you inspect the json. I don't think it's an important part of the action
GitHub might change their way of doing. In the future, wiki URLs might become unavailable for repos that have disabled wikis. But it seems to work as of today.

Unknowns
Will the token provided in GitHub actions be enough to push on the wiki ? We'll have to test that.
It's possible that it still requires a PAT, but no gist. But the scope of the PAT might need to be bigger. Need to test this more.
Conclusion
If you think this is out of scope for this action, I'll probably try on my own (might be a fork of this action)

After a test, we'll need a PAT with repo scope for this to work.
Most of what would be done is similar to https://github.com/Decathlon/wiki-page-creator-action

That's a pretty neat idea. I didn't know that the wiki is accessible even if it's disabled! I think we could include this if we make it optional. So user could fall back to the gist option if GitHub changes this behaviour in the future.
Btw, one issue of this action is that it fails if the gist is updated concurrently by a matrix run. I wonder whether this could be any better if we pushed to the wiki instead? We will have to try it.
Would you volunteer to try this approach and create a pull request if it works? That would be awesome!

I can try but I don't promise anything. If you don't hear from me by 2 weeks, assume I gave up and it's up for grabs

I guess I can close this for now :)


Support bluetooth keyboard input in flutter app

Use case
my app is based on  flutter version 2.5.1.when using bluetooth keyboard  input did not  have response

Proposal

Hi @nocsg, Thanks for filing the issue. Is this issue on a specific platform or all target platforms? Can you please try upgrading to the latest stable 2.5.3 by running flutter upgrade and see if that works?

Thanks to replay! My app platform is iOS 14.5.0.
this is my flutter doctor info:
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.5.2 20G95 darwin-x64, locale
zh-Hans-CN)
• Flutter version 2.5.1 at /usr/local/Caskroom/flutter/2.2.3/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ffb2ecea52 (9 weeks ago), 2021-09-17 15:26:33 -0400
• Engine revision b3af521a05
• Dart version 2.14.2
[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at /Users/songgaocao/Library/Android/sdk
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for
more details.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 13.0, Build version 13A233
• CocoaPods version 1.10.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
[✓] Connected device (2 available)
• M2006J10C (mobile) • beytp78dt8twam8t • android-arm64  • Android 10 (API
29)
• Chrome (web)       • chrome           • web-javascript • Google Chrome
96.0.4664.55
! Doctor found issues in 1 category.

Thanks for the info @nocsg,
if I understand correctly you are connecting a physical keyboard to android or ios device and its not working in a flutter app is that correct? if so can you please confirm, if it works in other apps (non-flutter apps)?

@maheshmnj
oooops... It don't work in other apps as well, so maybe the Bluetooth keyboard don't support the mobile phone device. Thanks  for you  r help!


Install Oracle Java via general azavea.java role

Use azavea/ansible-java/pull/2

Straightforward, merging.


Wrong result is returned when same name events occurs at once

How to reproduce
Part of browser process code:
ipcPromise.on('delay_echo', function(params, event) {
  return new Promise(function (resolve) {
    setTimeout(function () {
      resolve(params);
    }, 10);
  });
});

Part of renderer process code:
Promise.all([
  ipcPromise.send('delay_echo', 1),
  ipcPromise.send('delay_echo', 2)
]).then(function(values) {
  console.log(values);
});

Expected behaviour:
Console output is [1, 2] .
Actual behaviour:
Console output is [1, 1].

fixed 855ae01