[T5A1][W15-B3]Chew Zi Ling

@ChewZL
Some comments added. Please ack & close the PR after reading comments.


⚠ AnimeBytes has degraded performance

In ee9528c, AnimeBytes ($AB) experienced degraded performance:

HTTP code: 200
Response time: 10613 ms

Resolved: AnimeBytes performance has improved in 2079a35 after 5 minutes.


[release-4.15] OCPBUGS-36451: Can't install operator on 4.15 after uninstalling it on a prior version

Manual Cherry pick of commits from https://github.com/openshift/operator-framework-olm/pull/758
Awaiting verification for OCPBUGS-36450.
/hold

/label cherry-pick-approved
/label qe-approved

/jira refresh

/unhold

/lgtm

/label backport-risk-assessed

/approve


can't save metadata after editing

after #175 was merged, i see this error when:

i'm editing 000005 metadata.
remove publications section
add publication
click save

note because of the way @yarikoptic was adding publications, on upload the original publication is just a list. so i first delete it and then add a new one.
@yarikoptic - we should also fix this in the dandi-cli. the new json sections in this archive have now been updated and are closer to our final state.
@mgrauer - we should also discuss a migration path as the schema evolves.

i think this is the unpacked error message:
{
    "message": "No matching route for \"GET 5e70033c0fe0ec33e6b3d732/metadata\"",
    "type": "rest"
}

for this url: https://girder.dandiarchive.org/api/v1/folder/5e70033c0fe0ec33e6b3d732/metadata

but i could do it for dataset 1. going to try 5 again!

nevermind - i wasn't the owner of this (my alter ego was). closing this, but i think it would be really useful to not allow metadata editing if the logged in user is not the owner.

Agreed that this would be useful, it has been tracked at https://github.com/dandi/dandiarchive/issues/100.


Add jQuery dependency to Javascript loading

The Javascript to make the links was loading before jQuery, which was causing issues when a page was first loaded. I have added the dependency such that it loads after jQuery.

The plugin should contain a fallback to work without jQuery. Therefore it may not be forced as a dependency. When the fallback doesn't work, it's a bug.
It's possible to make an option for the plugin to force using the jQuery method.

Hi freelancephp,
Ok -- I'm not an expert on Wordpress (by any stretch of the imagination!), but prior to making this change, the plugin wouldn't work for me the first time the page was loaded.
Checking the page source, the plugin's javascript was loaded before the jQuery plugin and a quick look at the WP docs suggested this solution would fix the issue, and it did.
I think what needs to happen is that if jQuery is available then it should ensure that this plugin is loaded after the jQuery, /then/ fall back onto the non-jQuery option.
Then the second issue will be that the non-jQuery option doesn't seem to work on first page load (I'm not sure if it does thereafter).
Does this make sense? Do you have a suggestion/know a solution?
Regards,

So many sites are using jQuery, so I think this method could be default.
And maybe create an option on the admin page to select the plain/vanilla JavaScript method (without jQuery).


Module 03

Module: 03
Lab/Demo: 301
Task: 6
Step: After step 7
Description of issue
After sync, tge Gaming tile is disabled. However, the other changes in Settings do not appeat  to have applied. The Windows Security changes are effectice however. Suggest you slim lab to remove Gaming tile and make security changes in Antivirus only.
Repro steps:

Hmm. Actually - spoke too soon. They do appea. Disregard.

Right. Done some more testing. Intermittently, the configuration profile doesn't display  as active on the device, so theoretically, it's not being configured. That said, some of the desired settings appear to apply, and syncing doesn't record as having occured. But this all seems quite unreliable. The Gaming tile seems to disappear, but the Start menu items take longer. Thus far, the Privacy settings seem a bit reluctant to apply.
Summary - works, but does seem to take quite a while.

After testing this again, it took around 8 minutes for the profile to apply to SEA-WS1. This falls within the 15-minute recommended wait time.


Rating component

Duplicates ❌

[X] I have searched the existing issues

Which Figma library is concerned?
Not related to one Figma library
Summary 💡
A rating is an element that allows users to select or display a qualitative value.

Issue to take account for this next version.

Dependencies 📦
No response
Examples 🌈
No response
Motivation 🔦
No response
New 🆕

[ ] Validated in a grooming session

Backlog 📋

[ ] Has a good summary
[ ] Has example link(s)

Design in progress 🏗

[ ] Analysis (functional & technical)
[ ] Design refinement
[ ] Dev qualification (properties, breakpoints, tokens etc.)
[ ] Documentation refinement (inside the Figma branch)

Design review 👀

[ ] Figma branch merged
[ ] Documentation review _(and move the documentation into Vitamin Documentation Figma file)

Ready to dev 👍

[ ] Issues are created in all repositories affected (vitamin-<web,android,ios,compose>)

Dev in progress 💻

[ ] All issues are resolved (to replace with issues links)

A new ticket has been created on an inner source repository.


Switching network will not switch network of Safe App RPC

Description
When switching Safes that are on different networks the RPC endpoint provided to Safe Apps will not update and stick to the RPC for the initially loaded network,
Environment

Browser: Chrome
Wallet: MetaMask
Version: 3.18.0

Steps to reproduce

Open Safe on Mainnet
Open Safe App that uses the Safe Apps RPC (https://kwenta.io/ - supports Mainnet/Optimism)
Open Safe on Optimism
Open Safe App that uses the Safe Apps RPC  (https://kwenta.io/ - supports Mainnet/Optimism)

Other example Safe Apps

https://rimeissner.dev/transaction-templates with https://raw.githubusercontent.com/rmeissner/transaction-templates/main/templates/compound.json "just enter any amount, and it will fire 1 request when you press "build". The request will only work on rinkeby (as the template hardcoded a contract address)"
https://rimeissner.dev/sapp-balances/

Expected result

First time Safe App RPC requests are using Mainnet RPC
Second time Safe App RPC requests are using Optimism RPC

Obtained result

First time Safe App RPC requests are using Mainnet RPC
Second time Safe App RPC requests are using Mainnet RPC

Links

Compatible Safe Apps request thread.

This is something that should be checked in https://github.com/gnosis/safe-react/pull/3364 as well.

@rmeissner, the safe cannot change networks. Why would we expect the RPC to change?

Do we want to also fire chainChanged defined by EIP-1193 if the network is switched?


Only enable overlapping pattern semantics on overlap modifier

Addresses the problem raised in #98.
Summary:

The title.
We mean "reduce according to overlap modifier" by: if we have that modifier { try to unfold on each clause anyway }, otherwise { try to unfold, if we see a blocked reduction, fail directly }.
When checking conditions, we also reduce according to the overlap modifier (orderIndependent parameter which is passed everywhere).
When checking confluence, if we don't have an overlap modifier, we still traverse the classification, but we only generate domination warnings in a different way (if any) (see firstMatchDomination).
A red-black tree (no deletion, used to sort a list) showcase.
Conditions are always required to be confluent to avoid a bug in Arend:

\data Adams
  | abel Nat
  | cain Nat Nat \with {
    | zero, a => abel a
    | a, zero => abel (suc a)
  }
\func Adams-id (a : Adams) : Adams
  | abel n => abel n
  | cain n m => cain n m

I forgot to open an issue about it, and I'm not in the mood of opening issues.
bors d=imkiva

This PR is now ready for review. I hereby invite @imkiva for her criticisms.

Question: Can we remove domination warnings for order-independent and overlap-ping patterns? There's no good reason to say some clauses are really unreachable? (the clause from which the result is actually calculated is in the superposition state LOL although the implementation code must be executed sequentially which we don't care)

Sometimes a clause is totally covered that you can safely removed. The confluence checker guarantees that no matter which clause we choose, we get the same results, so it's in general better to remove redundant things. Imagine you have redundant code in Java, surely you want intellij to suggest a removal.
Anyways, thanks for the approval.
bors r+

Build succeeded:

gradle-check

Thank you!

I see. Thanks for kind explanation ❤.


Duplicate previous frame (if available) when we failed to generate a frame

Looks good! Thanks for the contribution :smiley:.


Jmxtrans stop all acitivities after 2 GC collections

Hi there,
We recently updated some configuration (added 300 metrics to an existing agent running a collection of about 300 metrics). The agent run only one server section. The JVM configuration is default. We are running the agent version 270 on centos 7 server with openjdk 8. Since the modification, agents started to stop collecting metrics after a period between a few minutes to one or two hours. Nothing in log, the process does not crash and just stop collecting.
We enabled jmx collection for the agent itself (overide configuration in wrapper configuration) and started collecting metrics with the following result.

What we can see is oldgen is growing realy fast and result in realy long 'stop the world' gc collection. This result in the first gap. Then the same occure again to finaly stop collecting. The only work around we found was to change the heap size to fixed 256Mb (Xms and Xmx). The following screenshot show the jvm metrics after 2 days.

Oldgen grow fast and seem to reach a peak but gc are done correctly. But we can see that it is now slowly growing. So i'm not sure that this will work for a long period of time.
What do you think ?
Regards

Could you grab a heap dump and thread dump of the jmxtrans process ?
https://www.baeldung.com/java-heap-dump-capture
https://www.baeldung.com/java-thread-dump
And analyze them to determine what's consuming memory and what the state of jmxtrans process (or send them to me in private)

Could you grab a heap dump and thread dump of the jmxtrans process ?
https://www.baeldung.com/java-heap-dump-capture
https://www.baeldung.com/java-thread-dump
And analyze them to determine what's consuming memory and what the state of jmxtrans process (or send them to me in private)


List the other unlisted PFPs (-1, 500 & 102)

F Client already does this and it's allowed (atleast from what I can tell since I've used Spinny Meowy for around 5 days now)
By the other pfps that are unlisted, I mean these:

Those should all work on Meower if I remember right.
IDs of these ones are:
-1 (Spinny)
500
102 (Server)
404 (Don't know and haven't even seen on the normal client unfortunately) Does not appear to sync & appear on normal client
-3 Does not appear to sync & appear on normal client

None of these are in the avatars folder. Do they really all work on Meower Svelte?

None of these are in the avatars folder. Do they really all work on Meower Svelte?

Besides questionmark & -3 they appear most of them sync on Meower Svelte

None of these are in the avatars folder. Do they really all work on Meower Svelte?

Did a look again found most:
icon_guest.svg / Guest / Dotted Meowy
icon_err.svg / Error / -3, note that this is also -1 aka spinny meowy however needs a spin affect added
I am going to look for the server one

None of these are in the avatars folder. Do they really all work on Meower Svelte?

Did a look again found most: icon_guest.svg / Guest / Dotted Meowy icon_err.svg / Error / -3, note that this is also -1 aka spinny meowy however needs a spin affect added I am going to look for the server one

Whoops server exists as well:
icon_101.svg / Server


Fixes issue where Parameter and Check operators were marked as "Cancelled" incorrectly

@likawind as lead reviewer. @cw75 tagged to keep in the loop, since we previously discussed writing to the parameter output path in the golang orchestrator instead of the python operator.
@vsreekanti and @agiron123 for any product-related issues/questions about the coloring of nodes.
Describe your changes and why you are making these changes
It used to be that, if an artifact is failed, we'd mark it as cancelled. However, this shouldn't necessarily be the case for checks and parameters. Parameters that fail type checking should be marked failed, as should checks that return false.
The solution proposed here is to decide whether an artifact is cancelled based on whether it has been "computed". Whether an artifact is "computed" simply means whether the artifact's content has been written to storage. This can happen even if the operator fails (once again, see parameters and checks).
This makes it so that failed check and parameter artifacts can be marked as red, instead of grey. This is especially noticeable for parameters - see the following parameterized flow who's parameter has failed a type check.
Before:

After:

Related issue number (if any)
ENG-1715
Loom demo (if any)
Checklist before requesting a review

[x] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[x] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[x] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@kenxu95, this mostly looks good to me. The only thing that I find a little confusing is that there are now two errors that ostensibly say the same thing. "Error creating num" and "Error executing num". Neither really make sense from the user's perspective -- num is a parameter that's definitely not executed and not even really created? At a minimum, I think we should only have one error, and I'd like to maybe rewrite the content to make it clearer that it's not about execution. Thoughts?

I agree with @vsreekanti 's suggestions, would let you decide if we should address in a separate PR.

@vsreekanti  @andre it currently seems like we don't expect artifact's to have status==Failed? I wonder if the best way to fix the duplicated error message is by getting rid of this case: https://github.com/aqueducthq/aqueduct/blob/main/src/ui/common/src/components/workflows/StatusBar.tsx#L386
In such cases, the operator should have errored right? So we don't need this duplication. @agiron123 can you verify?

Fixed the duplication concern by removing the FAILED status case from artifact:

I think there are two cases - 1) artifact canceled due to operator failure and 2) artifact failed as a result of op failure
For 1 I feel the canceled artf should not carry duplication of failure message. It should just appear as 'canceled'. We could probably provide some global context in UI for all canceled op/artf that certain failure result in cancellation.
For 2 I feel we could just assume the failure context is identical to upstream op. In this case I guess we should just remove the Error and Log field for artifact and read it from upstream operator when rendering. If we go this way, the removal can be done in backend and there should be minimum fix required in UI since it sees no duplicated messages.

hmm I'm not completely sure that we can solve this completely on the backend. I don't think this particular fix needs to be blocked on this discussion though, so I'll make follow-up task and we can discuss when @likawind is back in office.

Agree that we don't need to fix the canceled state right now. However, would it be possible to tidy up the error message to change the "Error executing" text?

@vsreekanti does a simple change to Error with num instead of  Error executing, just for failing parameters work? I think the rest of the text makes sense.

Yep, that would be perfect. It's just the headline that's a little confusing. The rest of the message makes sense.


How to loop over key/value pairs?

I am trying to use NamedTuples in my package, but couldn't find out how we are supposed to loop over key/value pairs, my attempt below failed:
nt = @NT(a=1, b=2.0)
for (k,v) in nt
  println(k, " = ", v)
end

I believe a PR with a pairs function would be a nice addition to the package.

Out of curiosity, will this be available in Julia v0.7? Should I close the issue?

Since NamedTuples are gonna be part of the next version of the language, I am closing the issue. Hope the key/value pair loop is addressed there already.


Composer Dependencies not met by local system

I'm not sure if this is more of a vagrant/composer issue than a repo issue, but say I want to add a composer dependency to my magento install.
I run for example
composer require yukoff/magento2-sendgrid-smtp

while in the magento directory (not the project root)
It throws an error about needing x php module installed, but I already know this module is installed on the vagrant box. Is there a way that I should be adding the dependency to the composer.json file, but not install it until it gets into the vagrant box?
Thanks!

Just use --ignore-platform-reqs option while using composer, it will ignore PHP requirements then.


Cannot reproduce WN18RR link prediction results

Hi, thanks for publishing this great contribution!
I ran the link prediction program on WN18RR with exactly the same parameters as recommended except eval batch size 2500 instead of 5000 to fit in my 1080 ti  (took 5 days but finished ok).
python3 run_bert_link_prediction.py
--task_name kg
--do_train
--do_eval
--do_predict
--data_dir ./data/WN18RR
--bert_model bert-base-cased
--max_seq_length 50
--train_batch_size 32
--learning_rate 5e-5
--num_train_epochs 5.0
--output_dir ./output_WN18RR/
--gradient_accumulation_steps 1
--eval_batch_size 2500
My results:
MR               Hits@10
127.38           35.97
Results in your paper:
MR               Hits@10
97                52.4
Do you have any thoughts on what might explain the difference? Did you observe such variation running this program with different random seed?
Thanks for any ideas!
Tim

@timhartill
Hi，thanks for your interests in our work!
I also observed such variation, I tried 3 times on a V100 GPU with different random seeds (generated with args.seed = random.randint(1, 200)) and obtains following 3 results.  I reported the best scores. The scores are sensitive to different seeds, and different machines can produce different results even with the same seed.

08/03/2019 23:26:58 - INFO - main -   Hits @10: 0.4282067645181876
08/03/2019 23:26:58 - INFO - main -   Mean rank:<PHONE_NUMBER>0223356

08/08/2019 21:00:23 - INFO - main -   Hits @10: 0.4728781110402042
08/08/2019 21:00:23 - INFO - main -   Mean rank: 97.40874282067645

08/15/2019 12:37:33 - INFO - main -   Hits @10: 0.5236119974473517
08/15/2019 12:37:33 - INFO - main -   Mean rank: 96.88704530950862

Mystery solved then!
Thanks for your quick response.


Issues that block the pipeline could have some visual cue - discuss :)

Sometimes it is obvious that the program has be stopped (when things stop moving) but often when playing you are messing with fairly static scenes, in those cases it's confusing.
Drinking right now so cant provide anything deeper than this. But will flesh this out at some point
thanks go to stacksmith for this idea.

some conditions are gl in bad state. Don't do outline then.
We could have condition mixin type to signify this

Moving to https://github.com/cbaggers/cepl/issues/120


Reduce specificity of form-element-inline

Removes the requirement of form-element-inline to be inside of horizontal-center.
Example:

Reviewer: @MatApple

Looks good @jfurrow – would you mind posting a screenshot of the integration tests?

@rcorral green!


Anonymous and/or simple authentication bind

I am trying to connect to Openldap deployed using this project from Meteor, but the package I've found uses anonymous bind. Is it possible to setup anonymous bind with docker-openldap?

Hello,
anonymous auth is disabled in : https://github.com/osixia/docker-openldap/blob/stable/image/service/slapd/assets/config/bootstrap/ldif/02-security.ldif
2 options:

define your own 02-security.ldif and mount it to /container/service/slapd/assets/config/bootstrap/ldif/02-security.ldif or extends the image and replace it.

run the container with -e KEEP_EXISTING_CONFIG="true" this will not play boostrap config files and the container will get a default openldap debian config.

docker run -e KEEP_EXISTING_CONFIG="true" osixia/openldap

Thanks!
And what about simple authentication, allowing all authenticated users to bind?

Simple authentication is already possible, but users can do nothing except on their entry.

How to modify 02-security.ldif ?   I tried mount  https://github.com/osixia/docker-openldap/blob/stable/image/service/slapd/assets/config/bootstrap/ldif/02-security.ldif into docker.

I am facing the same issue.  I simply want to turn on anonymous access so that anyone can see the users, but then they would need to know the users actual password to be given access.  I.e.  I want to use LDAP to hold the credentials and tell me if they are a member of a certain 'Group' or not.

I am facing the same issue. I simply want to turn on anonymous access so that anyone can see the users, but then they would need to know the users actual password to be given access. I.e. I want to use LDAP to hold the credentials and tell me if they are a member of a certain 'Group' or not.

This is what I did after setting up openldap, bootstrapping it, etc.
Create this file:
dn: olcDatabase={1}{{ LDAP_BACKEND }},cn=config
changetype: modify
replace: olcAccess
olcAccess: to * by dn.exact=gidNumber=0+uidNumber=0,cn=peercred,cn=external,cn=auth manage by * break
olcAccess: to attrs=userPassword,shadowLastChange by self write by dn="cn=admin,{{ LDAP_BASE_DN }}" write by anonymous auth by * none
olcAccess: to * by {{WHO}} read by dn="cn=admin,{{ LDAP_BASE_DN }}" write by * none

Replace:

{{LDAP_BACKEND}} with your setup, mine was mdb.
Replace {{LDAP_BASE_DN}} with your base DN.
Replace {{WHO}} (default value self) with users if you want all authenticated users to get readonly access, or anonymous if you want to allow anonymous auth. They will not see passwords obviously because of the previous line.

Run:
$ ldapmodify -x -D "cn=admin,cn=config" -H ldap://YOUR.LDAP.URL -f  security.ldif -W
modifying entry "olcDatabase={1}mdb,cn=config"

where security.ldif is the file you just created above. For the password prompt, use your config password (LDAP_CONFIG_PASSWORD)


about sync_bn  when training  the fusion model

hello,  i turn on the sync_bn when training the fusion model，but the Loss unchanged. what is the reason why you did not use SyncBN ? Is it because of the pytorch version not support or when use SyncBN not improve precision so did not use?

Good question. Do you mean that the model will not converge if you turn on SyncBN or the accuracy will not improve if you use SyncBN? According to my experience, SyncBN could be helpful for LiDAR-only detectors, but the improvement is marginal. Since most of the competitor methods cited in our paper did not turn on SyncBN, we did not use it for fair comparisons.

thank you for the  reply. i have solved the question why the model will not converge by updated the pytorch version. after that, i found the SyncBN result when training the fusion model Almost the same with turn off SyncBN.

Closed due to inactivity. Please feel free to reopen if you feel it necessary.


FEATURE: Fit SideTitle inside axis bounding box (prevent title overflow on edge)

Hi All!
I'm adding a feature to add an option for SideTitle to be fitted inside its corresponding axis bounding box.
This will fix #603. Below is the illustration:

Before Using FitInside (SideTitle is overflowed):

After Using FitInside (SideTitle is translated, hence not overflowed):

What is added:

FitInside data class (SideTitleFitInsideData) in lib/src/chart/base/axis_chart/axis_chart_data.dart
FitInside parameter on SideTitleWidget
Test for data class (SideTitleFitInsideData)
Sample usage in example/lib/presentation/samples/line/line_chart_sample3.dart

Thank you for this great feature!
First of all, please check the below bug:
When I hover my mouse, it hides the titles.
https://user-images.githubusercontent.com/7009300/217529221-63d57390-f75c-4799-96e1-7269a8d61c93.mov
It seems it is related to this code:
ValueListenableBuilder(
  valueListenable: _childSize,
  builder: (_, double? size, Widget? child) {
    return Transform.translate(
      offset: _getOffset(),
      child: Opacity(
        opacity: fitInside.enabled && size == null ? 0 : 1,
        child: child,
      ),
    );
  },
)

Also, there are some failing CI workflows. Please check and fix those issues.
Don't forget to update the documentation. Also, you should write a changelog for it.
So it would be great if you read our contributing guidelines.

Thank you for this great feature! First of all, please check the below bug: When I hover my mouse, it hides the titles.
hover-bug.mov
It seems it is related to this code:
ValueListenableBuilder(
  valueListenable: _childSize,
  builder: (_, double? size, Widget? child) {
    return Transform.translate(
      offset: _getOffset(),
      child: Opacity(
        opacity: fitInside.enabled && size == null ? 0 : 1,
        child: child,
      ),
    );
  },
)

Also, there are some failing CI workflows. Please check and fix those issues. Don't forget to update the documentation. Also, you should write a changelog for it. So it would be great if you read our contributing guidelines.

Thanks for the response.
The bug has been fixed. Now the title won't blink when the chart is on hover.
Also, I have added the widget test and increased the coverage of my added changes.
For the docs, I have updated the documentation and added a changelog about this added feature.

Hi, we are ready to merge after fixing the typo.
Thanks!

Thank you very much.

Hi! How can I usee this option?


Opened Accounts Overview Page

From #90:

A cross account Dashboard (I should note that there should still be a way to separate accounts, just that it is also useful to have a summary of multiple ones)

Maybe we could add a separate tab page that just shows data for all accounts:

Total amount
Total income
Total expense

But I wouldn't really know what else to add to this page. @RuboGubo could you give some suggestions?

In GNOME UI, we could add something combined with Adw.TabOverview

Well, I was thinking of something like a merged version of what the individual accounts have, you would just have to make sure that people still make and use individual accounts.

Well, I was thinking of something like a merged version of what the individual accounts have, you would just have to make sure that people still make and use individual accounts.

Do you mean listing all transactions from all accounts?
Maybe the page could look like old versions of Money did:

Without the actions, new, edit, delete, obviously. But just listing all the information.

It could maybe include all the groups, but I don't think that would be useful.

Ability to filter all transactions by groups could be useful in case a user have same groups on different accounts. But this will require filtering by group names, because IDs can mismatch.

WinUI:

GNOME to follow :)

https://github.com/nlogozzo/NickvisionMoney/releases/tag/2023.4.0-rc1


[Agones] Removing/updating token annotation on a GameServer deoesn't update the Endpoint

What happened:
I have a Quilkin Agones xDS server, and connected Quilkin proxies.

I created and allocated GameServer in Agones with the quilkin.dev/tokens: annotation.
I then kubectl edit that GameServer and removed the quilkin.dev/tokens: annotation.
I see two endpoints now in the config for my proxy, not a single one with the token removed.

➜  agones-xonotic-xds git:(examples/xonotic-xds) ✗ curl -s http://localhost:9091/config | jq
{
  "admin": {
    "address": "<IP_ADDRESS>:9091"
  },
  "clusters": {
    "default": {
      "localities": [
        {
          "locality": null,
          "endpoints": []
        },
        {
          "locality": null,
          "endpoints": [
            {
              "address": "<IP_ADDRESS>:7251",
              "metadata": {
                "quilkin.dev": {
                  "tokens": [
                    "NDU2"
                  ]
                }
              }
            }
          ]
        },
        {
          "locality": null,
          "endpoints": [
            {
              "address": "<IP_ADDRESS>:7251",
              "metadata": {
                "quilkin.dev": {
                  "tokens": []
                }
              }
            }
          ]
        }
      ]
    }
  },
  "filters": [
    {
      "name": "quilkin.filters.capture.v1alpha1.Capture",
      "config": {
        "metadataKey": "quilkin.dev/capture",
        "suffix": {
          "size": 3,
          "remove": true
        }
      }
    },
    {
      "name": "quilkin.filters.token_router.v1alpha1.TokenRouter",
      "config": null
    }
  ],
  "management_servers": [
    {
      "address": "http://quilkin-manage-agones:80"
    }
  ],
  "proxy": null,
  "version": "v1alpha1"
}

What you expected to happen:
The token removed from the endpoint, so an end user could remove a playing user based on their connection token in realtime.
How to reproduce it (as minimally and precisely as possible):
See above.
Anything else we need to know?:
Not sure if duplicated to #585 so reporting separately.
Environment:

Quilkin version: dev.
Execution environment (binary, container, etc): Container
Operating system: Linux
Custom filters? (Yes/No - if so, what do they do?): No
Log(s): N/A
Others: N/A

Just did a check on the admin server as well, and see the same config
{
  "admin": {
    "address": "<IP_ADDRESS>:9091"
  },
  "clusters": {
    "default": {
      "localities": [
        {
          "locality": null,
          "endpoints": []
        },
        {
          "locality": null,
          "endpoints": [
            {
              "address": "<IP_ADDRESS>:7251",
              "metadata": {
                "quilkin.dev": {
                  "tokens": [
                    "NDU2"
                  ]
                }
              }
            }
          ]
        },
        {
          "locality": null,
          "endpoints": [
            {
              "address": "<IP_ADDRESS>:7251",
              "metadata": {
                "quilkin.dev": {
                  "tokens": []
                }
              }
            }
          ]
        }
      ]
    }
  },
  "filters": [
    {
      "name": "quilkin.filters.capture.v1alpha1.Capture",
      "config": {
        "metadataKey": "quilkin.dev/capture",
        "suffix": {
          "size": 3,
          "remove": true
        }
      }
    },
    {
      "name": "quilkin.filters.token_router.v1alpha1.TokenRouter",
      "config": null
    }
  ],
  "management_servers": null,
  "proxy": null,
  "version": "v1alpha1"
}

Just did a local manual test - can confirm the token gets dropped.
Will write an extra piece to the Agones integration tests to automate testing of this, and we can close this issue.


Lack of support in network-security-rule for multiple prefixes. gap from the providers' feature/capability.

The network-security-rule doesn't implement the provider's source_address_prefixes property.
If multiple source addresses are needed, the current release of the module fails because it supports only one address.
A simple boolean logic could enable the feature in the module.
### Tasks
- [ ] https://github.com/Azure/azure-data-labs-modules/pull/247

Thanks @aluevano - thanks for the feedback. You okay to create a PR for this?


User pill x fails to update permissions model

After clicking the user tables role pill's x to remove a user role the permissions store is not updated

@richard-cox Let's discuss this one and how much work it is before starting. This is a lower priority P1.

Closing in favour of #2181


[ColorPicker] Unify hex color code to lowercase

Summary of the Pull Request
When using ColorPicker HEX color format in Pick a color and open editor mode, editor shows color code in lowercase. However, when using ColorPicker in Only pick a color mode, HEX format is shown and copied to clipboard in uppercase.
What is this about:
Unify HEX color code to lower case in all ColorPicker modes.
What is included in the PR:
How does someone test / validate:

Open ColorPicker page in Settings, set Activation behavior to Pick a color and open editor. Set Default color format to HEX
Open ColorPicker using hotkey, observe that color code in mouse tooltip is now lowercase (it was uppercase  before this change). Pick a color. Observe that HEX color format is lowercase in editor.
Set Activation behavior to 'Only pick a color`
Open ColorPicker using hotkey, pick a color to copy color code to clipboard, paste color code to some text editor and observe that it is lowercase (it was uppercase before this change)

Quality Checklist

[x] Linked issue: #16488
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Can you please update settings ui too

Can you please update settings ui too

What do you mean?


Add doc and sample for Openshift with custom vnet

What this PR does / why we need it:
Doc and sample for Openshift with custom vnet
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[x] documentation
[x] sample

Release note:

@jim-minter @pweil-

We should have an e2e test that exercises deploying in a custom vnet.

@ritazh many thanks for this PR! :smiley:

@pweil- sorry for the delay! Let me wrap this up today and hopefully get it merged soon.

@pweil- @jim-minter PTAL

@kwoodson will be looking at this next week to help it get merged.  IMO the vnetCidr and IP collision pieces still both need to be resolved before this merges.

@ritazh
@jim-minter has asked me to look at this.  He mentioned in his previous comment that the vnetCidr and IP collision pieces were still pending.
Is there anything I can do to assist?

@kwoodson
Discussed with @jim-minter in person. Latest commit should address outstanding comments. PTAL

@jim-minter,
Anything left on this PR?  I believe @ritazh has addressed your latest concern regarding:

need to remove the documentation wording about putting the IP address at the end

/hold cancel
/lgtm

/assign CecileRobertMichon

@CecileRobertMichon, I think this one is ready for merge.

/approve

@CecileRobertMichon, I believe this to be an etcd flake:
 etcdserver: request timed out


WeightedChoice depreciated - Uses old rand::distributions module

First - amazing work and huge thanks for the tutorial and this module. I know you haven't had time to keep it updated, but wanted to add the issue for other users.
I'm fairly new to Rust and love the flow of the tutorial and am finding it really useful to get familiar with basic gamedev and loops as well.
As for the issue, it seems like WeightedChoice and Weighted in https://tomassedovic.github.io/roguelike-tutorial/part-12-monster-item-progression.html has been depreciated in favour of rand::distributions::WeightedIndex
The pull request and discussion seems to have gone on here: https://github.com/rust-random/rand/pull/518
A working update for the first part using WeightedIndex:
let monster_chances = [
                ("orc", 80),
                ("troll", 20),
            ];

let dist = WeightedIndex::new(monster_chances.iter().map(
    |item| item.1)).unwrap();

let mut rng = rand::thread_rng();

let choice = monster_chances[dist.sample(&mut rng)].0;

let mut monster = match choice {
    "orc" => Object {
        // return object
        }

Cheers!

Yes I have run into the same issue while doing the tutorial last month, but I'm afraid you're on the wrong project.
The project hosting the code for the tutorial already has an issue open: https://github.com/tomassedovic/roguelike-tutorial/issues/40

Sorry about that. Closing.


FAI-7679 - Dedup jobs shared by workflows

Description
Figured it out the root cause finally. Jobs are at the pipeline level and workflows can share jobs if a workflow is executed from the failed step. It will point to the same jobs up to the point of the failure and then create new jobs for the remained.
Type of change

[X] Bug fix
[ ] New feature
[ ] Breaking change

Major kudos.


Email subject line missing and crazy from address label...

Just installed and tested isso. First worked ok with default moderation (off). But want to use email moderation. After hours of messing with it the best I can get is still showing no subject line and replaces the from "label" with "Ich schrei sonst!".
This may be somewhere along the line it's wanting an email from like XXX<EMAIL_ADDRESS>but I tried using a from value like that and it didn't work for me. The raw msg source has a from address but the label has been set as "Ich schrei sonst!". Google translating that comes out as: "I scream otherwise!"
Also, tried debugging the subject string and saw it is making a subject and passing to email Header() function. Seems like some problem there as in the received raw message source there is no subject inserted.

As an aside - I can't figure out why when I use systemctl status I see log entries for each request but they do not show up in the isso.log file I've set. There is core log entries like "connect", but no requests. Where do they go then? systemctl knows where to get them so maybe emitted as stdout but then why not to the log file I have configured?

Bug #139 may be relevant.

That seems likely in some way. I am using this on a Pelican generated static blog site. It probably doesn't account for the strange "from" address. I don't see a fix presented for that bug or maybe I got lost in comments. I quick on a sample blog page shows no H1 being used. The test comment in threads table has a blank last column value. Not sure how to fix though.

Oh. I just saw that "Ich schrei sonst!' is actually the German meaning of Isso (from overview page). I had no idea. So I guess on that aspect of my problem I'd like some way to have the From address on emails show up as something I'd understand like "Isso Admin" or whatever (in English). Is there a way to set his in config file? I tried what I thought may be obvious, eg. from="Isso Admin<EMAIL_ADDRESS>but that did not work.

Ok. After more digging I managed to find you could set the data-title attribute on the embed tag. I tried this on my pelican template and (after some experimenting) it worked out ok. So I now have a subject line. I wonder if maybe it's a good idea to add a default subject when sending the notify email? So in notifications.py when the subject is empty add a value like "Isso Comment Moderation" or something. I'd be willing to make a PR for that is it's going to be accepted. I guess it needs to be translatable text too.
I think my issues here are resolved and are mostly about figuring out the docs on things that aren't very explicit. Speaking of explicit docs - I see there is some kind of "admin" interface now. I tried /isso/admin and got a page. Presumably a password needs to be set somewhere but I have not found that info yet. Any clues how I might use the admin panel? Otherwise can close this issue.

Help to improve the admin panel further would be great. I'm not aware of anybody currently working on it apart from the PR that I have out for review.

Hi @neocogent , did you find any solution to the email subject line. I am currently looking around but can't find anything to make it work.

@mmarif4u , add data attributes data-title (data-title="YOUR-POST-TITLE") in Client Configuration.
If you use the static site generator like hugo, there is an example(My blog.)

@mmarif4u,
Yes. As mentioned by wswr the data-title attribute worked for me. In my case with Pelican I changed my comment section in the comments.html include file to be:

Thanks a lot, @neocogent and @wswr . I will check it out.


clarify the usage of vlanipranges

ISSUE TYPE

Bug Report

COMPONENT NAME

list vlanipranges

CLOUDSTACK VERSION

acutally master

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

i dont know if its a bug but we encounter the following problem.
if you create a physical network and and add 2 vlanipranges aka public addresses.
e.g.
first network (ROOT)
we have one network range "forsystemvms: true" and its assigned to the root domain
   {
      "account": "system",
      "domain": "ROOT",
      "domainid": "23462d06-b5a4-11e8-905f-525400ce8b0c",
      "endip": "<IP_ADDRESS>",
      "forsystemvms": true,
      "forvirtualnetwork": true,
      "gateway": "<IP_ADDRESS>",
      "id": "8aab7a4e-b20c-4f0e-b6a1-e0f6dbe6471c",
      "netmask": "<IP_ADDRESS>",
      "networkid": "0f0d66cb-f8f1-4e33-ad3a-ee2889bb0a09",
      "physicalnetworkid": "6ae13495-fad8-479e-b3f8-b2014fb8de63",
      "startip": "<IP_ADDRESS>",
      "vlan": "vlan://100",
...

a second network (DOMAINX) assigned to other domain,  "forsystemvms: false"
  {
      "domain": "DOMAINX",
      "domainid": "9d77af79-5c87-4b73-b094-a106525803b6",
      "endip": "<IP_ADDRESS>",
      "forsystemvms": false,
      "forvirtualnetwork": true,
      "gateway": "<IP_ADDRESS>",
      "id": "a9db9833-4deb-4591-a860-e9d5f1d85ed5",
      "netmask": "<IP_ADDRESS>",
      "networkid": "0f0d66cb-f8f1-4e33-ad3a-ee2889bb0a09",
      "physicalnetworkid": "6ae13495-fad8-479e-b3f8-b2014fb8de63",
      "startip": "<IP_ADDRESS>",
      "vlan": "vlan://101",
...

the problem which we encounter is that if there no addresses more left in "DOMAINX" network we encounter that the "ROOT" network is used.
is this a normal behaviour?
STEPS TO REPRODUCE

create a ROOT network (set systemvm usage)
create a DOMAINX network
acquire additional addresses if there is no one more left in the DOMAINX domain and you get one from ROOT

EXPECTED RESULTS

normally only the assigned network should be used in the domain

ACTUAL RESULTS

if network from "DOMAINX" has no more addresses it will use the "ROOT" domain

@weizhouapache yes i think so. for my understanding it should not use the systemvm ips in the other domain public ip assignment. i detected if DOMAINX is out of addresses it uses the ROOT ips which are only for systemvm. i think this should not happen.

@svenvogel i see "forvirtualnetwork": true in your range for ROOT domain. It seems to me that you would expevt them to be used. of course this may be a bad choice for flag names. Are you clear yet on how it should have worked?

@DaanHoogland   it should be fixed if system.vm.public.ip.reservation.mode.strictness is set to true.


After update from 0.2.0: AttributeError: module 'jax.core' has no attribute 'eval_context'

After updating from flax 0.2.0 to flax 0.2.2 I get the above error message. Downgrading to 0.2.0 solves this, so the error source is located. I'm working with the now deprecated flax.nn package if backward-compatibility might be the reason for this issue.
The Issue is encountered in a custom RNN, when using the init_by_shape function in conjunction with jax.lax.scan.

Hi @mr128254 -- we are about to officially deprecate flax.nn but if you have a minimal repro we can perhaps take a look. (Also have you upgraded your version of JAX in parallel to upgrading the Flax version?)

I am pretty sure this has something to do with the Jax version. We should raise the minimal version in setup.py


[extensions/rust] add markdown syntax highlighting to doc comments

This is an attempt at implementing #169676. It's currently only doing it for /// comments as I am testing the implementation.
Current state:

Still making /* comment blocks work.

Original problem I faced, keeping it for context.

Markdown is highlighted, though because it's including ///, it does not properly highlight the titles as /// # Title would also not be highlighted in Markdown:

It would be if those weren't there:

So we have to tell it to only apply the markdown on the content of the comment, right? However when changing it to apply for the captured group and not the entire match, it's still recognised as Markdown, but there is no highlighting at all:
				{
					"comment": "documentation comments",
					"name": "comment.line.documentation.rust",
					"match": "^\\s*///(.*)",
					"captures": {
-						"0": {
+						"1": {
							"name": "meta.embedded.block.markdown",
							"patterns": [
								{
									"include": "text.html.markdown"
								}
							]
						}
					}
				},

Results in:

So my thought was that it might be doing something wrong with applying it if it's a specific capture group. What if we change the capture group to include ///?
				{
					"comment": "documentation comments",
					"name": "comment.line.documentation.rust",
-					"match": "^\\s*///(.*)",
+					"match": "^\\s*(///.*)",
					"captures": {
						"1": {
							"name": "meta.embedded.block.markdown",
							"patterns": [
								{
									"include": "text.html.markdown"
								}
							]
						}
					}
				},

That seems to make it give the same Markdown issue as before:

I've tried juggling around with contentName instead of name and some other stuff. But I keep bumping my head against the same rock. Anyone got any idea on what the issue could be? 😅

This will be made into a PR as well on dustypomerleau/rust-syntax, but this keeps it concise for now in 1 PR.

@microsoft-github-policy-service agree

@alexr00 I think this should be contributed by an extension. We do the same for JS/TS: https://marketplace.visualstudio.com/items?itemName=bierner.jsdoc-markdown-highlighting
Including the markdown grammar caused every other grammar to be loaded so we try to avoid doing this in core (the root cause is https://github.com/microsoft/vscode-textmate/issues/106)

@mjbvz good point, that can indeed be an issue. I will make a PR for rust-analyzer that uses injection. Before the holidays I also had a chat with someone else about this and I was going to go the rust-analyzer route most likely for other reasons. Thank you for giving this a look!

Sorta disagree that it should be a different extension.The difference between, jsdoc and rustdoc is that jsdoc is not strictly a part of javascript, but rustdoc is a part of rust, therefore I believe it should be in the rust language extension itself instead of a separate extenion.
This said I've put together an extension that does this, I'm not going to publish this to the marketplace but it is here if anyone wants to load it manually, it is here.
https://github.com/adenine-dev/vscode-rustdoc-markdown-highlighting
This extension is essentially @AndreasBackx's code in extension form. It does have problems but its solved well enough for my purposes. To truly fix this I think rust-analyzer would have to provide this, but I'm not super sure.


feat!: extend the available styles for mode by separating mode from the separator styles

An implementation of the feature request: https://github.com/sxyazi/yazi/pull/732, https://github.com/sxyazi/yazi/issues/1228
Closes https://github.com/sxyazi/yazi/pull/732, https://github.com/sxyazi/yazi/issues/1228

Previously, only a single mode style was supported, which meant the same mode style was used for both the highlight and secondary colors, and the separator style was applied to the secondary color.
This led to the inability to set different styles for the primary color of the mode, and users couldn't control the mode style independently because it depended on the separator style.
This PR separates the mode from the separator style, making it its own distinct style. Users will now be able to set different main and alternate styles for each of the 3 modes individually.
⚠ Breaking changes
mode_normal, mode_select, mode_unset, and separator_style have been separated into a new, independent [mode] section:

normal_main - Main style for the normal mode
normal_alt - Alternate style for the normal mode
select_main - Main style for the select mode
select_alt - Alternate style for the select mode
unset_main - Main style for the unset mode
unset_alt - Alternate style for the unset mode

permissions_t, permissions_r, permissions_w, permissions_x, and permissions_s have been renamed to perm_type, perm_read, perm_write, perm_exec, and perm_sep to improve readability and align with the naming conventions in https://github.com/sxyazi/yazi/pull/1761.

User config file is always theme.toml and unchanged:
https://github.com/sxyazi/yazi/blob/5e48df5126a64c4753ce0034bc1a1667203de88b/yazi-config/preset/theme-base.toml#L1-L3

@sxyazi Thank you, for your effort to implement this! I appreciate it. :+1: :heart:


Readme example doesn't run due to a missing assert import

import assert from 'node:assert'; seems to fix it

Hmm yeah, that import gets added in the tests
https://github.com/bgotink/kdl/blob/367b240207072d26a9900fbd4458a1299ced6563/test/readme.js#L23-L31
It would make more sense to add the import in the readme itself


🧑💻 More lints to improve and ensure better code quality

Please check if the PR fulfills these requirements

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

What issue does this fix?
Closes #12
What kind of change does this PR introduce?
This PR provides more lints for checking and ensuring code quality, which can help improve developer experience.

I don't think a version bump is needed for this

What do cognitive_complexity and rust_2018_idioms check for?

What do cognitive_complexity and rust_2018_idioms check for?

Ok sure, the cognitive_complexity check actually checks for things like how complex the code looks like for example if the function is like 1000 lines long or you have something which has a lot of nesting then this check will suggest to split those code into smaller functions which can help maintaining and reading through and following the code alot easier for anyone working on the project.
For the rust_2018_idioms check it checks how idiomatic your code is like if it is not then it will suggest some suggestions on how to make it. So that your code looks cleaner and looks more rusty. :slightly_smiling_face:

All good! Ty for contributing, appreciate it 🙏


Function Wizard: create Effects

Hi,
I added the possibility to create Effects with the Function Wizard
What types of effects arc creates is determined by the fixture capabilities (Dimmer, RGB, Position).
for each type a normal and a staggered version (start offset spread equally) will be created.

Ooh I like it


Remove tailwinds anti-pattern in PrimaryNav.tsx

Context is on the tracking issue - https://github.com/open-sauced/hot/issues/267
This issue only needs changes to this file.

.take


Connection error on 6379

I manage to install things but it was really painful :)
First should I run fab -R local setup_appand the two other commands if I install using setup_all ? Cause those commands return that no local role has been defined.
If I run the server using ./manage.py runserverit starts but when I try to reach it I got this error :
ERROR:django.request:Internal Server Error: /
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/django/core/handlers/base.py", line 113, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/views/decorators/cache.py", line 89, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)
  File "/srv/newsblur/utils/view_functions.py", line 32, in wrapper
    output = func(request, *args, **kw)
  File "/srv/newsblur/apps/reader/views.py", line 85, in index
    return welcome(request, **kwargs)
  File "/srv/newsblur/apps/reader/views.py", line 127, in welcome
    user              = get_user(request)
  File "/srv/newsblur/utils/user_functions.py", line 83, in get_user
    user = cache.get('user:%s' % settings.HOMEPAGE_USERNAME, None)
  File "/usr/local/lib/python2.7/dist-packages/redis_cache/cache.py", line 179, in get
    value = self._client.get(key)
  File "/usr/local/lib/python2.7/dist-packages/redis/client.py", line 863, in get
    return self.execute_command('GET', name)
  File "/usr/local/lib/python2.7/dist-packages/redis/client.py", line 570, in execute_command
    connection.send_command(*args)
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 556, in send_command
    self.send_packed_command(self.pack_command(*args))
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 532, in send_packed_command
    self.connect()
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 436, in connect
    raise ConnectionError(self._error_message(e))
ConnectionError: Error 111 connecting to <IP_ADDRESS>:6379. Connection refused.

Nothing is listening on <IP_ADDRESS>:6379.
How to process ?

I tried to do a clean install again. This time installation return this error :
[localhost] out: Traceback (most recent call last):
[localhost] out:   File "utils/backups/s3.py", line 56, in <module>
[localhost] out:     get_file_from_s3(sys.argv[2])
[localhost] out:   File "utils/backups/s3.py", line 26, in get_file_from_s3
[localhost] out:     bucket = conn.get_bucket(BUCKET_NAME)
[localhost] out:   File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 409, in get_bucket
[localhost] out:     bucket.get_all_keys(headers, maxkeys=0)
[localhost] out:   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 371, in get_all_keys
[localhost] out:     '', headers, **params)
[localhost] out:   File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 338, in _get_all
[localhost] out:     response.status, response.reason, body)
[localhost] out: boto.exception.S3ResponseError: S3ResponseError: 403 Forbidden
[localhost] out: <?xml version="1.0" encoding="UTF-8"?>
[localhost] out: <Error>InvalidAccessKeyId<Message>The AWS Access Key Id you provided does not exist in our records.</Message><AWSAccessKeyId>XXX</AWSAccessKeyId><RequestId>5AE6E209E1E53B27</RequestId><HostId>5p7rmc+v06l+aezo9pgfYfgFeL6wpc57C6xtum5Kl1z266jpqblbxje9uRD93JhoP4aP4ij5/ac=</HostId></Error>
[localhost] out:

Fatal error: run() received nonzero return code 1 while executing!

Requested: PYTHONPATH=/srv/newsblur python utils/backups/s3.py get static.tgz
Executed: /bin/bash -l -c "cd /srv/newsblur && PYTHONPATH=/srv/newsblur python utils/backups/s3.py get static.tgz"

The first error means you aren't running Redis. The second error means that your AWS key is invalid. You should specify a valid AWS key in local_settings.py. Although why are you using AWS? I use it for static assets, but you should just use DEBUG_ASSETS=True in local_settings.py, since you don't need to optimize for bandwidth like the hosted NewsBlur does.

Well, I didn't want to use it but didn't know how to disable. Cause I get the same error with DEBUG_ASSETS=True

Comment out the problematic lines from the install, like using S3.

I'm not sure which lines I should comment without compromising the install.
And Why didn't I get this problem earlier ? Cause I once got the install work without commenting out any lines..

Well I comment out the transfert assets and also had to comment out 'elif engine.startwith ...' cause None object can't do that.
It works and I can reach it but got some other problems now :)
Posting a new issue


HOMESHICK_DIR defines location of homeshick

The current implementation of homeshick contains hard-coded paths to the (default) installation location. This patch makes homeshick position-independent by allowing the user to specify an alternative installation location using the HOMESHICK_DIR environment variable.

I'd say this PR is ready now - please review it.

This is a great PR. The changes are minimal and still robust. I've added some comments - otherwise I am quite ready to merge this :-)

Glad to hear it. I hope the latest fixes will be enough to nudge it over the merge threshold.

Indeed they were :-)
Merging...

Great!
Another question: How do you feel about making a release of homeshick now? With the current changes on the development branch, I mean.
I can package any arbitrary git revision, of course, but it just looks better if it has a proper version number. And I think the current state of development is more than enough for a release 1.0.0. (Yes, I kind of like semantic versioning, but it might be overkill for a tool like this :-)

How do you feel about making a release of homeshick now? With the current changes on the development branch, I mean.

That's not going to happen right now :-) I have promoted your changes to the testing branch however, meaning quite a few people will get the changes when updating homeshick (I myself run the testing branch on all machines and my colleagues at work do as well).
Normally I let the testing branch simmer for ~1.5 - 3 weeks before merging into master, this improves the chances of a bug free release considerably.

And I think the current state of development is more than enough for a release 1.0.0

Until now I have kept away from versioning, because all releases have been backwards and forwards compatible with respect to to the repo structure. In such a case semantic versioning does not make a lot of sense, it can just be a flow of updates instead, keeping the whole thing a little more simple.
I'd recommend simply taking the short commit hash and tacking a revision number at the end of it, that's what Debian does and it works quite well (i.e. 'homeshick-244aa4-1').

OK, thank you for explaining your process.
Using a commit hash instead of a release version for packaging is fine.


Fatal error: Class 'Faker\Factory' not found

Hi,
I've just installed Faker with composer with the composer.json file you provide.
As a result I get Fatal error: Class 'Faker\Factory' not found error with the php example code you provide.
I've updated the composer.json file with "fzaninotto/faker": "1.5.*@dev" in require section but as a result I get this composer error after composer update command : The requested package fzaninotto/faker 1.0.0 could not be found.
Could you help please ?

Weird. Can you paste the composer.json file? Also, the right way to install Faker to an existing project would be to call:
composer require fzaninotto/Faker "~1.5@dev"

The json file content
{
"name": "fzaninotto/faker",
"type": "library",
"description": "Faker is a PHP library that generates fake data for you.",
"keywords": ["faker", "fixtures", "data"],
"license": "MIT",
"authors": [
{
"name": "François Zaninotto"
}
],
"require": {
"php": ">=5.3.3"
},
"require-dev": {
"phpunit/phpunit": "~4.0",
"squizlabs/php_codesniffer": "~1.5"
},
"suggest": {
"ext-intl": "*"
},
"autoload": {
"psr-4": {
"Faker\": "src/Faker/"
}
},
"autoload-dev": {
"psr-4": {
"Faker\PHPUnit\": "test/Faker/"
}
},
"extra": {
"branch-alias": {
"dev-master": "1.5.x-dev"
}
}
}
And what happens when i run the command composer require fzaninotto/Faker "~1.5@dev"
[root@xxx1 faker]# composer require fzaninotto/Faker "~1.5@dev"
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- The requested package fzaninotto/faker 1.0.0 could not be found.
Potential causes:

A typo in the package name
The package is not available in a stable-enough version according to your minimum-stability setting
see https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion for more details.

Read http://getcomposer.org/doc/articles/troubleshooting.md for further common problems.
Installation failed, reverting ./composer.json to its original content.
[root@xxx01 faker]#

OK, you have copied the Faker package.json, which is not the way it works. This is a not a Faker issue: you probably need to learn more about composer.


Use entity relation for the owner of an entity in the catalog entity page header

Oh there was still a place were we didn't used the relations. I only see this change as an intermediate step to make the header configurable per entity kind / type.
I also changed the behavior a bit:

Both owner and lifecycle are only displayed when they are available. They don't apply to all entities. Here is where customization would be handy.
The context menu is now always displayed, since we now have a guard against removing bootstrap locations, this should be fine.
I removed the workaround that was in place for user entities (why was it there in the first place, but not for groups? 😆 )

I did not touch the old EntityLayout, as I expect it to be deprecated soon.
In the future I could imaging an evolution like this:
const ServiceEntityPage = () => (
  <EntityPageLayout>
    <EntityPageLayout.Header variant="for the color of the header">
       <OwnerHeaderLabel />
       <LifecycleHeaderLabel />
    <EntityPageLayout.Header/>

    <EntityPageLayout.Content
      path="/"
      title="Overview"
      element={<ComponentOverviewContent />}
    />
    ...
  <EntityPageLayout/>
);

OwnerHeaderLabel and LifecycleHeaderLabel could internally use useEntity() to access the current entity.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4652-fmxyagotrysu0k0yuvdqyafxyx37v8hl.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/6034c212f285af4df39580b0

While I think that this PR buys us enough time to postpone this topic, I would be curious what you think about the customization idea above. Is that the way we want to go in the long term?

It does fit the current model. My only worry is that the current model might turn out to be too flexible. Or rather: the installation of individual plugins risk starting to become overly complex, and the update procedures risk becoming burdensome.
The thing that triggered this thought was the Badges thing that's being worked on. The act of installing that particular plugin is to add an entry to this entity kebab menu and then you may need to put the actual popup code somewhere and do the state management for opening/closing it etc. If every plugin starts becoming that involved, it'll become a problem.
So we're having some early thoughts around maybe we should experiment with having a number of fixed "slots" that plugins can put stuff into, so that their "installation" procedure essentially can be concentrated into the plugin register method or similar.

Thanks for the feedback.


Improve Exception backtrace API

Currently an exception's backtrace is an Array(String).  This is usually fine, however there are some use cases where you may want to present the trace in a different way; such as pretty printing an exception.  Currently this is not very easy to do as each portion of the trace needs to be manually parsed either manually or via an external shard.
I propose that the backtrace is also exposed an a like Array(Frame), where each Frame would have getters for the related info of that line.  E.g. filename, method name, line/column etc.  This would provide a much cleaner API to work with.
Given Crystal handles printing each line I would assume it inherently has access to each portion of data.  This could be kept backwards compatible by exposing the frames as another method that the current one could leverage.  E.g. @frames.map &.to_s.

I think it's worth pointing out the two PRs open about this are mutually exclusive.  We should probably figure out which one we want to move forward with and go from there, versus needing to review both all the time...

True. Also probably discuss outside of here what to expose. What's the reason to expose the address and the cryptic function name?

Is there reason to not expose the address?  I guess my thinking was even if most people won't ever need it, still might be helpful to expose; especially if we're going for a 1:1 mapping between the backtrace string and a Frame instance.
Regarding the function name, I combined both names into 1 function ivar.  Mainly because currently there isn't a case where you'd have both (https://github.com/crystal-lang/crystal/blob/master/src/exception/call_stack.cr#L199-L201) in a given backtrace string.  So technically it is still exposed, but the API is the same.

Let's say someone implements an interpreter for Crystal. What would be the address?

Ah, nevermind, in any case it won't happen.

I think we should start with ironing out the data format for a frame.
I've looked at Rust's implementation (https://doc.rust-lang.org/src/std/backtrace.rs.html#152-155) and they actually separate frame and symbols and multiple symbols can be associated with one frame. I don't know when you would have multiple symbols per frame. But there must be a reason for that.
The data structures (converted to the Crystal equivalent) look like this:
record BacktraceFrame,
  frame : RawFrame,
  symbols : Array(BacktraceSymbol)

record BacktraceSymbol,
  name : String?,
  filename : String?,
  line_number : Int32?,
  column_number : Int32?

The process is also split into two parts, first retrieving the raw frames from whatever backend is used to walk the stack. The result is a collection of BacktraceFrame with empty symbols. The second step resolves the frames against the debug information and fills the symbols.

Rust exposes the ip addresses.  No idea who would use them though, but I guess since it's a struct people are free to ignore it.  The reason Rust does multiple symbols per frame is Normally there is only one symbol per frame, but sometimes if a number of functions are inlined into one frame then multiple symbols will be returned. The first symbol listed is the "innermost function", whereas the last symbol is the outermost (last caller). https://docs.rs/backtrace/0.3.2/backtrace/struct.BacktraceFrame.html Also, apparently there are two ways to lookup the function name.  The dwarf has just the name and dlAddr has the name with "parameter types" which I think might be also useful...FWIW.

Not to derail the thread, but I just noticed @asterite's foreshadowing 😂 ☝ https://github.com/crystal-lang/crystal/issues/10681#issuecomment-835854880

It's fine, we can leave the ip address as zero for interpreted mode. In the end backtraces in the interpreter are done in a completely different way, so it makes sense that the values there will also be slightly different.

(I also wrote that comment while I was writing the interpreter, so I knew what was coming :-P)

Getting back to this issue again and got the data model proposed in https://github.com/crystal-lang/crystal/issues/10681#issuecomment-845129773 working, but still have a few questions:

Do we still think its necessary to support having multiple symbols per frame?

Based on https://github.com/crystal-lang/crystal/issues/10681#issuecomment-1016103785, I'm just not sure if this is a rust  specific compiler thing, or if it's also applicable to us given it also uses LLVM. Prob worth investigating/finding a scenario where there would be more than one to test with/how to extract that data.

What to do with CRYSTAL_CALLSTACK_FULL_INFO ENV var?

Currently i'm still doing essentially what it was before by using sname as function and also having it set the ip address on each symbol if its set.

Do we actually want to still expose the address?

I'm assuming yes based on the past comments.

What do we ultimately want the API for this to be like?

Do we want to actually expose the BacktraceFrame type, versus like yielding each of its symbols? If not, then Exception#frames and Exception#each_frame that returns an array of/yields each BacktraceSymbol would comes to mind first. Tho the naming of that is a bit weird given its called "frames" but you get a "symbol"?

My wont would be to have lots of info returned, people can choose not to
use it if they care.  Maybe nodoc the ip address, can't imagine someone
needing it externally? :)
On Mon, Sep 5, 2022 at 8:58 PM George Dietrich @.***>
wrote:

Getting back to this issue again and got the data model proposed in #10681
(comment)
https://github.com/crystal-lang/crystal/issues/10681#issuecomment-845129773
working, but still have a few questions:
Do we still think its necessary to support having multiple symbols per
frame?
Based on #10681 (comment)
https://github.com/crystal-lang/crystal/issues/10681#issuecomment-1016103785,
I'm just not sure if this is a rust specific compiler thing, or if it's
also applicable to us given it also uses LLVM. Prob worth
investigating/finding a scenario where there would be more than one to test
with/how to extract that data.
What to do with CRYSTAL_CALLSTACK_FULL_INFO ENV var?
Currently i'm still doing essentially what it was before by using sname
as function and also having it set the ip address on each symbol if its
set.
Do we actually want to still expose the address?
I'm assuming yes based on the past comments.
What do we ultimately want the API for this to be like?
Do we want to actually expose the BacktraceFrame type, versus like
yielding each of its symbols? If not, then Exception#frames and
Exception#each_frame that returns an array of/yields each BacktraceSymbol
would comes to mind first. Tho the naming of that is a bit weird given its
called "frames" but you get a "symbol"?
—
Reply to this email directly, view it on GitHub
https://github.com/crystal-lang/crystal/issues/10681#issuecomment-1237604525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADBUGDDBKXQ4N2ZH47XTTV42XNHANCNFSM44GCZ55Q
.
You are receiving this because you commented.Message ID:
@.***>


FSBuilder meeting with Sikt

Meeting with Sikt where we will be introduced to FSBuilder. We will be showed how FSBuilder works as well as how it is used. This will give us great insight into what our application needs to focus on.

Most important usages of FSBuilder:
Examples for FSBuilder:
General notes:
Grouping objects, everything should be able to group.
Networks
Hosts
Cluster interface for combining firewalls
Firewalls are registered with their intended target.
Router -> vlans -> individual rules for things across vlans.

Write meeting minute


PI-1081 End-to-end test for pathfinder-and-delius

Test passes locally, but currently failing in CI due to https://github.com/ministryofjustice/hmpps-probation-integration-e2e-tests/pull/378


enable datafactory cli generation

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

@zikalino Do we still need this PR?

@zikalino Since this is a small changes and you included extra files and you're too busy. 😄  I would close it first. Feel free to open a new one~

/azp run automation - sdk

/azp run automation - sdk


Separate APK Google Play Invalid

First of all, thanks for the new separate apks feature on 0.18. It cuts down their respective size by 4 mb!
arm and x86 apk uploads perfectly to play store but the universal apk seems to have problem.
Here's my build.gradle
defaultConfig {
        applicationId "com.foo"
        minSdkVersion 16
        targetSdkVersion 22
        versionCode 7
        versionName "1.6"
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }

It appears that the universal apk is using versionCode as oppose to versionName.

Thanks a lot for trying out the new feature @alvinwoon! It looks like Google Play doesn't like the universal APK because it's unnecessary. What happens if you simply don't upload the universal one?
Maybe we should disable generating the universal APK by default. I thought the universal APK might still be useful in some cases but doesn't look like it is.

Will commit a change that changes this to false soon.

@mkonicek

What happens if you simply don't upload the universal one?

Basically nothing. Play still serves the right apk to different testing devices of mine. So +1 on removing universal. Thanks!


Add LOG_TO_BLOB_STORAGE option

Addresses #85:

Adds azure.log_to_blob_storage to config.json; when running from the command line and it is set, it will store the logs in blob storage as if it was a containerized deployment
Updates documentation
Templates were not updated; the logs will be stored in blob storage anyway, so it defaults LOG_TO_BLOB_STORAGE to true, even though it is never used.

Note: I wasn't sure how to generate automated test cases; if I could be pointed in that direction, please let me know.

Thank you for your PR! 🎆I'm in a workshop tonight and tomorrow, so I will try to review at the weekend 🚀


Show that oneof is broken

If a JSON object has two fields of the same oneof, one field
arbitrarily wins without any error.

Properly, “If the parser encounters multiple members of the same oneof on the wire, only the last member seen is used in the parsed message.” https://developers.google.com/protocol-buffers/docs/proto3#json
If your purposed tests pass, the first one should be misbehavior? As the "title" comes second, and should overwrite the first one. However, the JSON mapping section does not really cover at all the behavior of oneof, so I’m just presuming the same behavior as we would expect from protobuf wire-format.


LogNormalize transform

This PR add LogNormalize transform described in #11

@mbabadi ready for a review!

@ordabayevy oh wait

@ordabayevy now that you're at it, how about also safeguarding dividing by library size (+ eps)? we don't want a NaN if a user gives us an empty cell :D also, I am leading toward adding perform_scaling bool.  It will be a stronger commitment from the user. Always a good thing!

Great! Feel free to merge anytime! :)


Don't include sys/select.h on HP-UX as it doesn't exist

There is no sys/select.h on HP-UX and the man page lists sys/time.h as the correct header for select.

I'm wondering why we haven't hit this before on this platform?

That code is new in 3.x I think - the only place in 1.1 that includes it is include/internal/sockets.h which does this:
#  ifdef OPENSSL_SYS_AIX
#   include <sys/select.h>
#  endif

which is only used on AIX systems.

Is this on master or openssl-3.0? We just built 3.0 last night without errors.

There is specific code for NonStop to avoid this:
# if defined __TANDEM
#  include <unistd.h>
#  include <sys/time.h> /* select */
#  if defined(OPENSSL_TANDEM_FLOSS)
#   include <floss.h(floss_select)>
#  endif
# elif defined _WIN32
#  include <winsock.h> /* for type fd_set */
# else
#  include <unistd.h>
#  if defined __VMS
#   include <sys/socket.h>
#  else
#   include <sys/select.h>
#  endif
# endif

Is this on master or openssl-3.0? We just built 3.0 last night without errors.

There is specific code for NonStop to avoid this:
# if defined __TANDEM
#  include <unistd.h>
#  include <sys/time.h> /* select */
#  if defined(OPENSSL_TANDEM_FLOSS)
#   include <floss.h(floss_select)>
#  endif
# elif defined _WIN32
#  include <winsock.h> /* for type fd_set */
# else
#  include <unistd.h>
#  if defined __VMS
#   include <sys/socket.h>
#  else
#   include <sys/select.h>
#  endif
# endif

Yup. That looks like my handiwork ;)

On the other hand, @mattcaswell, we are getting
make: *** No rule to make target 'include/openssl/configuration.h', needed by 'build_generated'.  Stop.

as of this morning.

That one results from make clean  removing configuration.h

That one results from make clean removing configuration.h

Ah. Can you raise a separate issue for that please?

Opened as #18396

I've merged this to master and 3.0 branches. I did not notice this has CLA:trivial before merging this but I am OK with it. @mattcaswell Can you please confirm, you're OK with CLA: trivial for this?

This builds correctly on both master and openssl-3.0 on NonStop x86 and ia64


sign transaction with an offline wallet

currently it is not possible to sign the transaction with a wallet that does not have access to the network as it needs to know the unspent output (and the associated derivation path).
This is a follow up on issue discussed in #42

can I construct inputs for a transaction and import it to the wallet and try to sign it offline?
As I can export a transaction from a wallet in any stage, I can know the form of a transaction.
I want to do the following step：

with online wallet A, I create a normal transaction and add inputs, outputs after that export the transaction, I got a constructed file
replace the field in the constructed file with my goal real inputs and outputs, and save the file
(the UTXOs and index can get from other source)
with offline wallet B, import the transaction. the UTXOs of the inputs are all wallet B's. and in wallet B, try to finalize the transaction and sign it. and export then send

Is this possible?

did you have any check field in the transfer exported file? like a checksum or hash value that confirms the content not modified?


FIX: Rebuild shared library to fix linux - nvidia error

When I posted in ChloePrime/AAAParticles#22, I did some digging into the issue to further find out what could be the cause. I discovered that the issue wasn't anything to do with OpenGL, Mesa, Nvidia, Etc. I believe it may have been an issue in how you built libEffekseerNativeForJava.so. Noting that the symbol glGenSamplers was undefined in the library and running ldd did not show any GL shared libs being pulled, I figured it was a build issue and it couldn't find OpenGL. I pulled the repo effekseer/EffekseerForMultiLanguages, pulled everything to the newest version, fixed a couple include issues, and rebuild the shared library. I replaced the old shared library in my minecraft folder with this new built one, and minecraft fired right up.
In Summary:
I believe the shared library libEffekseerNativeForJava.so was built incorrectly.
I rebuilt it and replaced it, and minecraft fires right up.
The issue doesn't seem to be related to drivers at all, considering most computers support some form of OpenGL 3.3+.
If I can be of any further assistance, please let me know!
This so was built on:
Arch Linux x86_64 ( Linux-zen 6.10.10 )
nvidia 560.35.03

LGTM

When I posted in #22, I did some digging into the issue to further find out what could be the cause. I discovered that the issue wasn't anything to do with OpenGL, Mesa, Nvidia, Etc. I believe it may have been an issue in how you built libEffekseerNativeForJava.so. Noting that the symbol glGenSamplers was undefined in the library and running ldd did not show any GL shared libs being pulled, I figured it was a build issue and it couldn't find OpenGL. I pulled the repo effekseer/EffekseerForMultiLanguages, pulled everything to the newest version, fixed a couple include issues, and rebuild the shared library. I replaced the old shared library in my minecraft folder with this new built one, and minecraft fired right up.
In Summary: I believe the shared library libEffekseerNativeForJava.so was built incorrectly. I rebuilt it and replaced it, and minecraft fires right up. The issue doesn't seem to be related to drivers at all, considering most computers support some form of OpenGL 3.3+.
If I can be of any further assistance, please let me know!
This so was built on: Arch Linux x86_64 ( Linux-zen 6.10.10 ) nvidia 560.35.03

btw can you help me fix this issule on Windows and fix #31 ?

If I can be of any further assistance, please let me know!

Mind checking if Eldritch_End has the same issue as well?

If I can be of any further assistance, please let me know!

Mind checking if Eldritch_End has the same issue as well?

Eldritch End jarjars this mod. So put new version of AAAParticles in mods folder may fix its issue

If I can be of any further assistance, please let me know!

Mind checking if Eldritch_End has the same issue as well?

Eldritch End jarjars this mod. So put new version of AAAParticles in mods folder may fix its issue

Caused by: java.lang.UnsatisfiedLinkError: /home/sarah/.var/app/org.prismlauncher.PrismLauncher/data/PrismLauncher/instances/2/minecraft/libEffekseerNativeForJava.so: libGLU.so.1: cannot open shared object file: No such file or directory 	at java.base@21.0.4/jdk.internal.loader.NativeLibraries.load(Native Method) 	at java.base@21.0.4/jdk.internal.loader.NativeLibraries$NativeLibraryImpl.open(NativeLibraries.java:331) 	at java.base@21.0.4/jdk.internal.loader.NativeLibraries.loadLibrary(NativeLibraries.java:197) 	at java.base@21.0.4/jdk.internal.loader.NativeLibraries.loadLibrary(NativeLibraries.java:139) 	at java.base@21.0.4/java.lang.ClassLoader.loadLibrary(ClassLoader.java:2418) 	at java.base@21.0.4/java.lang.Runtime.load0(Runtime.java:852) 	at java.base@21.0.4/java.lang.System.load(System.java:2025) 	at mod.chloeprime.aaaparticles.client.AAAParticlesClient.installNativeLibrary(AAAParticlesClient.java:53) 	... 12 more

@HugKitten, that looks like the shared lib is not finding java 21? strange, considering I built the lib using Java 17. To check, do you have java 21?

@HugKitten, that looks like the shared lib is not finding java 21? strange, considering I built the lib using Java 17. To check, do you have java 21?

Yes, both 17 and 21.

@HugKitten Alright, now also double check that you have libGLU installed and updated to the lastest version? if you have java 21 then that's the only missing link. If that doesn't solve it, give me some sysinfo and I'll dig deeper.

If I can be of any further assistance, please let me know!

Mind checking if Eldritch_End has the same issue as well?

I'll jarjar the updated version in the next update. Is it confirmed that works fine on linux & mac now? I can't test personally

If I can be of any further assistance, please let me know!

Mind checking if Eldritch_End has the same issue as well?

I'll jarjar the updated version in the next update. Is it confirmed that works fine on linux & mac now? I can't test personally

Working fine for me on archlinux w/ kernel linux-zen 6.10.10, I'm a couple hours into a playthrough using stock Prominince 2 3.0.2 with only changing this mod version. No idea about mac


Bootstrap Carousel

Description
Add the documentation for bootstrap carousel.
Doc Domain

[ ] Html
[ ] CSS
[ ] JavaScript
[x] Bootstrap

Are you contributing under any open-source program ?

Hackclub Summer Of Code

@purva2906 can I contribute to this issue under HSOCC22.

Congratulations 🎉🎉you can start working on this make sure to add originality to the documentation we'll check for plagiarism


Provisioning profile needs unique bundle identifier

Can anyone help me out with this error? I don't know what to change my bundle identifier to for this to work? @gabrieloc

I changed mine to com.burntscarr.giovanni

Alright, thanks! That got me past that problem. @burntscarr
Did you have this problem too or know how to fix this? Thanks for your help.

Find the Watch.app inside the Giovanni.app (in Xcode of course) and go change it's settings to com.(username).giovanni.watchkit (I'm only partially sure on that it might need to have a different ending like .watchos or .watchapp)

Okay so the fix is in the Info tab.
There's this main tab, where you can edit the watchkitextension bundleidentifier but, it's not the only one.

Over in the info tab (up top-ish) there's another area to edit.

The NSExtension > NSExtensionAttributes > BundleIdentifier setting also needs changed.
Mine is com.burntscarr.giovanni.watchkit although my Main bundleid is com.burntscarr.giovanni.watchkit.watchkitextension for the extension because my giovanni_watchOS bundleid is com.burntscarr.giovanni.watchkit
Hope this helps some people!

I made a few updates to the readme to answer some common questions. https://github.com/gabrieloc/GIOVANNI/blob/master/README.md


Switching path using browserHistory re-creates all the components (not just re-render)

Hello,
I am using react-router version 2.0.1 and have a few routes setup, like this:
    <Router history={history}>
      <Route component={App}>
        <Route path="/" component={Foo}>
          <Route path="bar/:id" component={Bar} />
        </Route>
     </Route>
    </Router>

So the idea is to render the Bar component whenever the user hits /bar/1 for example. This works fine. However, at some point I need to contact my server to do some stuff and in return I get a new id, maybe 2. The users URL in the browser should then switch to /bar/2 which I do using the following:
import { browserHistory } from 'react-router';

browserHistory.push('/bar/' + newId);

Which also works, the URL changes and everything, but the components (Foo and Bar) both completely re-render from scratch - they are recreated and remounted! I checked this by doing a console.log in the componentDidMount. This feels weird, as I would at most expect the components to re-render and not be completely recreated.
Reading the docs regarding lifecycle (https://github.com/reactjs/react-router/blob/master/docs/guides/ComponentLifecycle.md) seem to imply that the component should just re-render.
Am I doing something wrong? Is there another way I should be going about this? Or is this a bug?
Thanks!

That's not the expected behavior, and it's not what we see in the examples.
Do you have a repro case handy?

@taion You're right, I believe this might be an issue with react-router-redux which is what I use to keep the state up 2 date with the router.

I also now experiencing this issue. The component did remount, instead of just rerender.

@firasdib @chaintng i am experiencing this now. can either of you shed some light on a possible cause?


iOS8: Landscape issues.

Hi there,
I'm using iOS8. Portrait works ok, but when I try to rotate the device to landscape, the main view disappears (to the bottom).
Version 1.x, btw...

@gespona I used this solution. Just add the iOS7StyleScreenBounds method. Then, look for instances of "[UIScreen mainScreen].bounds" and replace them with [self iOS7StyleScreenBounds].


Renaming folders in mixs-templates/

Rename checklist folders to "_plus_combinations" and extensions folder to "extensions_only" in mixs-templates/.
Fixes #731

Closing out in favor of #785


How to obtain the current playing time and position when rodio plays music files

How to obtain the current playing time and position when rodio plays music files
At present, I want to try to build a simple music playing software,
Music playing software needs the most basic
"Total duration" of music files
Current play position "Play time position"
Jump to the specified position "Time Position"
I don't know how to get the "play time position" at present
Thank you for your help

Does the functionality in #510 fulfill the "Play Time Position"?


Ly overwrites pam file on update

When upgrading ly on Arch linux, through yay, ly replaces the settings specified in its pam file.
For me, on my ThinkPad T560, it means, that after I enter my password, ly also requires me to use fingerprint to authenticate. There is no additional prompt for me to know I have to swipe my finger, and in docked mode, I have no visual cues for this.
My temporary fix to this problem has been to manually edit the pam file, after each upgrade, however this does not seem like a good solution to this problem.
In actuallity, the pam file should only be created if it does not exist, and should not be modified when upgrading ly.

@Zaph-x What if the default PAM file is subject to updates? This would be the case with #496 for example. Sure, we could add an option, but then you, the end user, wouldn't be able to control that during an update (as ly is most likely going to be installed through a package manager).

For me this sounds more like a packaging issue with Arch Linux. According to the Arch Wiki, this should not happen like @Zaph-x describes it. Either this is the case 'original = X, current = Y, new = X', when the file wasn't changed in the ly package or it should create a .pacnew file in the case of 'original = X, current = Y, new = Z', leaving it to user to incorporate the changes coming in with the update.


Refactored the tutorial notebook "Reproduce: Stabalizing Feedback Controls for quantum systems" for qutip version 4.6.2

I am new to qutip and want to contribute to the project. I have opened this issue to discuss the refactoring of the notebook titled "Reproduce: Stabalizing Feedback Controls for quantum systems"  written by Robert Johansson to work with qutip version 4.6.2.  The same issue is raised here in the google group as well. I have made changes to the code cells of the notebook and it seems to me to be working well. Kindly review it and finally guide me on how to add this tutorial to this repository as do not want to violate any rules here unwittingly.
PS: Below is the .md version of the notebook.
 Reproduce_ Stabilizing Feedback Controls for quantum systems.md

Hey @rupayan2029,
thanks for contributing to this repository. As this repository is relatively new, the contribution guidelines still need some work (I will work on this soon!).
I ran the notebook you attached with the latest release of QuTiP (4.7) and there occurs an error in the cell that calls the function general_stochastic(). For this repository, the notebook should work with the latest version of QuTiP. It would be great, if you could change the notebook to work with QuTiP 4.7 instead of version 4.6.
I can also guide you through the process of adding a notebook to the repository. For that, you can consider creating a (draft) Pull Request, so I can review your code directly. I appreciate your willingness to contribute to this repository and you don't have to be scared of violating any rules.

Sorry for responding late. I have changed the notebook to work with QuTiP 4.7 and resolved the error that was occurring in the cell containing general_stochastic(). Kindly have a look.
Thanks for appreciating contribution from amatuer quantum control enthusiast. I don't know this is the right platform,  I am also looking for a masters/PhD position in Quantum control but don't have anyone who would recommend me as I am mostly self-taught. Any general advice as well as information regarding same would also be helpful.
Reproduce_ Stabilizing Feedback Controls for quantum systems.md


Add asset url support

file:///android_asset/pathsegment1/pathsegment2
android.resource://package_name/id_number
android.resource://package_name/type/name

Done
9f1183b2e04ffd2b5ee37b98173bc7953836f4c3
9fd58c1d233dd015a2651b4ca22d64ac49c53b04

Add


[Feature Request]:   solution for leetcode 953

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to add solution in c++,java,python, javascript for leetcode problem 953. Verifying an Alien Dictionary
Use Case
solution for leetcode 953
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Ajay-Dhangar  can you please assign me.


feat(Ecto.Schema): Add option to skip validations for default values

@josevalim Addresses Issue #3747. Let me know if you need any additional changes or want to add documentation for this feature.

Could you mention the new option around here? https://github.com/elixir-ecto/ecto/blob/0e8954d91d4a4520f96cecc28cc7786037ce163b/lib/ecto/schema.ex#L647
It would also be good if :default documentation specified that it validates the value in compile time.

LGTM, thanks! We should definitely document it. Perhaps list it right after :default, so it addresses @v0idpwn's concerns?

@josevalim I put it at the bottom since it is likely going to be the least used option for a field.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


RSE and RSD in preamble

In the preamble to the survey we need to deal with defining RSE/RSD/other title to be sure that people understand they are all the same no matter how they are called (or can be called) in their respective country

https://github.com/softwaresaved/international-survey/blob/master/survey_creation/2017/de/texts/welcome_message.md

Closed: Obsolete


Add xslt 2.0 requirement in Readme.md

The readme should mention that a XSLT 2.0 capable XSLT processor is needed for doing the conversion.

Readme was changed


Title:  footer

Which pre-existing JavaScript project will you be working on?
i work on footer for best view
Describe your enhancement/improvement
footer is not good
Anything you need to say to the Reviewers
No response
Add any other context or relevant screenshots about the enhancement here.
i am a contributer in gssosc 24

@harsh160311 Thank you for providing your insight but it is not required as of now.


test: increase RAM of httpboot vm to fix RHEL-41024

This is a suggested workaround for https://issues.redhat.com/browse/RHEL-41024

/test-rhel-9-5-ng


Chinese commit messages

Hey,
it would be nice if you guys could do english commit messages instead of chinese, because I cannot understand them.
(English comments in source code would be nice too.)

This may be easier for @qicosmos  and @lucklove , but my English is very poor……
If I use English to write comment, I can only aid google translation.

123marvin123是歪果仁？给你指条明路：学中文


the curve points are wrong from the script

when I use the script ,the result are 3 dimensions .so i don't know how to resove it.

Can you upload an AEP file which reproduces the problem?
On Thu, Sep 26, 2019, 5:24 AM Lanchouchou<EMAIL_ADDRESS>wrote:

when I use the script ,the result are 3 dimensions .so i don't know how to
resove it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SupportClass/ae-ease-to-gsap-customease/issues/9?email_source=notifications&email_token=AAGVENBP3XSSEJMQSLO3UHTQLSSWJA5CNFSM4I2ZHJHKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HN32X3Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVENBDDWFPGZYC3J5MNH3QLSSWJANCNFSM4I2ZHJHA
.


Make the toolbar "DONE" button text configurable

There is no way to change it and i'd like to be able to customize it, the same way it is done to customize the toolbar folder title.
ImagePicker.create(this)
	.toolbarDoneButton("DONE") // done button
	.start();

Merged. Thanks for the PR


Simulator should not log errors when there is no fit

This gets rid of the extraneous logging of the error when a fit is no longer possible.

Second commit removes an extraneous line break:
old:
$ ./cluster-capacity --kubeconfig /home/decarr/.kube/config --podspec=examples/pod.yaml

26

new:
$ ./cluster-capacity --kubeconfig /home/decarr/.kube/config --podspec=examples/pod.yaml
26

Lgtm


Support pod restart on certificate renewal

Issue: Some applications might not watch the secrets for updates. If the certificate expires it will cause a running application to  fail to communicate. Currently, users have to support it in their applications or use solutions like wave.
Let's add support for configurable pod-restart on certificate renewal with annotation.

Sounds like a great idea. Would make the experience for some users less error prone and mysterious: why is authN failing when my certificate should be renewed? Let’s do it.

Proposal:
Add the proposed capability, and include global configuration to automatically enable the behavior for all pods which request Secrets that is enabled at the operator level, as well as configuration that is enabled using pod annotations, so a single workload can request this behavior.
Add a failsafe mechanism that prevents the restart from occurring if the pod started after the Secret was updated, or a similar mechanism which prevents the operator from repeatedly restarting pods due to a bug or unexpected interaction with another operator.
Upon Secret refresh, if:

global setting set by command line flag to always restart (delete) pods upon Secret refresh
pod annotation that requests that the operator restart (delete) the pod
the pod was not already restarted
then delete the pod.

Regarding the proposal, I think we should drop the global setting option. Two main reasons for that:

Explicit is better than implicit - The person configuring the operator is not necessarily the person who is responsible for all the applications or their deployments. If it is going to be a global config, other users might experience unplanned/unexpected restarts.
Keeping it simple - I think that at least at this stage, it is better to keep things simple.

Sounds right - let's do it :)


Add basic EditorConfig file for repo

Added a starter .editorconfig file for the repo. Xcode 16 adds support for a subset of all EditorConfig properties.
#no-changelog

cc: @peterfriese


Prevent Global Duplicate Prevents Undo Last Action

Steps to reproduce:

Add some text https://renji-xd.github.io/texthooker-ui/
Enable Prevent Global Duplicate
Delete a line
Undo deleting that line
The delete was not undone

i think this is similar to your other issues a matter of definition and the way it was intended to work. when changing it i would probably add a new option to allow duplicates on undo or sth. similar

Yeah I think it is similar. I think you probably want to still be able to undo if you have global enabled though.

i missunderstood the request. I thought you wanted to allow inserting duplicates on undo - that you can't in general undo lines with global duplicate check enabled was a bug


Dev wind and solar sensor agents

I have edited the index and property files and the relevant readme files to improve the project and documentation further, as indicated in the three points.