[FFM-8148] - Standardise SDK error codes

FFM-8148 - Standardise SDK error codes
What
Adding standardised SDK error codes + pushing some logs down to debug to reduce noise
Why
Each SDK uses different log messages for its lifecycle. We want to standardise messages across SDKs with a unique code.
Testing
Manual testing + unit testing

Codecov Report
Patch coverage: 80.26% and project coverage change: +0.28% :tada:

Comparison is base (14d3516) 60.98% compared to head (c9a2e25) 61.27%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #158      +/-   ##
============================================
+ Coverage     60.98%   61.27%   +0.28%
- Complexity      364      385      +21
============================================
  Files            31       32       +1
  Lines          1779     1836      +57
  Branches        202      211       +9
============================================
+ Hits           1085     1125      +40
- Misses          541      550       +9
- Partials        153      161       +8

Files Changed
Coverage Δ

...io/harness/cf/client/connector/LocalConnector.java
0.00% <0.00%> (ø)

...va/io/harness/cf/client/connector/EventSource.java
70.83% <66.66%> (+0.20%)
:arrow_up:

...ain/java/io/harness/cf/client/common/SdkCodes.java
73.33% <73.33%> (ø)

...ava/io/harness/cf/client/api/MetricsProcessor.java
79.19% <75.00%> (-1.22%)
:arrow_down:

...ain/java/io/harness/cf/client/api/AuthService.java
77.77% <100.00%> (+0.85%)
:arrow_up:

.../main/java/io/harness/cf/client/api/Evaluator.java
85.60% <100.00%> (+0.59%)
:arrow_up:

...ain/java/io/harness/cf/client/api/InnerClient.java
70.23% <100.00%> (-1.66%)
:arrow_down:

...ava/io/harness/cf/client/api/PollingProcessor.java
81.69% <100.00%> (ø)

.../harness/cf/client/connector/HarnessConnector.java
69.95% <100.00%> (+0.12%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[1.16] Enchantment applicator essense consumption seems off

Industrial Foregoing Version:
I'm actually looking at the source for this:
https://github.com/InnovativeOnlineIndustries/Industrial-Foregoing/blob/1.16/src/main/java/com/buuz135/industrial/block/misc/tile/EnchantmentApplicatorTile.java#L96
The code here is:
    private int getEssenceConsumed(int experienceLevel) {
        int xp = 0;
        if (experienceLevel >= 0 && experienceLevel <= 15)
            xp = (int) (Math.pow(experienceLevel, 2) + 6 * experienceLevel);
        if (experienceLevel >= 16 && experienceLevel <= 30)
            xp = (int) (2.5 * Math.pow(experienceLevel, 2) - 40 * experienceLevel + 360);
        if (experienceLevel >= 32) xp = (int) (5.5 * Math.pow(experienceLevel, 2) - 162.5 * experienceLevel + 2220);
        return xp * 20;
    }

I was looking at this because for higher levels (say 35), the consumption seemed to be overly high.
I notice two things:
The second if ends with experienceLevel <= 30), and the third if starts with if (experienceLevel >= 32).
So level 31 is missing, and would be free?
I made a graph of the 3 expressions, and the switchover points from the if's:

From the first to the second you can see that at level 15 there's a nice smooth transition from the first to second curve, as you'd expect.
From the second to third though there's a big jump at level 30/32, which I think is not what was intended, I'd expect another smooth transition here. And that would explain the unexpected high consumption I was seeing.

Is it true that ranges are wrong but the formula is from the minecraft wiki

Ah, from https://minecraft.gamepedia.com/Experience there's this:
level2 + 6 × level (at levels 0–16)
2.5 × level2 – 40.5 × level + 360 (at levels 17–31)
4.5 × level2 – 162.5 × level + 2220 (at levels 32+)
So I guess the correct code would be:
    private int getEssenceConsumed(int experienceLevel) {
        int xp = 0;
        if (experienceLevel >= 0 && experienceLevel <= 16)
            xp = (int) (Math.pow(experienceLevel, 2) + 6 * experienceLevel);
        if (experienceLevel >= 17 && experienceLevel <= 31)
            xp = (int) (2.5 * Math.pow(experienceLevel, 2) - 40.5 * experienceLevel + 360);
        if (experienceLevel >= 32)
            xp = (int) (4.5 * Math.pow(experienceLevel, 2) - 162.5 * experienceLevel + 2220);
        return xp * 20;
    }

Looks a lot nicer if you graph it :)

It is true that ranges are wrong but the formula is from the minecraft wiki

I just saw! :) But! Some of the numbers are different from the wiki :)

Just to visualize the differences:


fix: adds locale when opening privacy policy link

DCMAW-8571: Adds locale when opening privacy policy link
This is a fix for passing the locale to the browser when privacy policy link is opened in the browser
Checklist
Before raising your pull request:

[x] Commit messages that conform to conventional commit messages
[x] Ran the app locally ensuring it builds
[x] Ran the tests locally ensuring they pass on Build
[x] Pull request has a clear title with a short description about the feature or update
[x] Created a draft pull request if it is not yet ready for review

Before your pull request can be reviewed:

[x] Met all of the acceptance criteria specified in the user story on Jira

[x] Reviewed your own code to ensure you are following the style guidelines

[x] Ran the app and tested the feature on a range of device sizes
Please include iPod Touch, iPhone SE and iPhone 11 as a minimum.

[x] Written Unit and Integration tests if needed

[ ] Met all accessibility requirements?

[ ] Checked dynamic type sizes are applied
[ ] Checked VoiceOver can navigate your new code
[ ] Checked a user can navigate only using a keyboard around your new code

Before merging your pull request:

[ ] Ensure that the code coverage and SonarCloud checks have passed
[ ] Actioned and resolved all comments, reaching out to reviewers for clarifications if necessary.
[ ] Ran the app to ensure that no regressions have been caused by changes during code review.

Think you'll need to update the configuration tests for this 🙂


add a __hash__ method to Vnlv dataclass for Python3.11 compatibility

A breaking change in Python3.11 is to "Change field default mutability check, allowing only defaults which are hashable instead of any object which is not an instance of dict, list or set. (Contributed by Eric V. Smith in bpo-44674.)", see https://docs.python.org/3/whatsnew/3.11.html#dataclasses.
This makes it so Vnlv cannot be used as a default argument for DFXCore.vlnv and import pynqmetadata fails when using Python3.11, as does import pynq
The ways to fix are either to:

add a hash method to Vlnv
use a a field with a default_factory for DFXCore.vlnv (e.g. vlnv: Vlnv = field(default_factory=lambda: Vlnv(vendor="xilinx.com", library="ip", name="dfx", version=(1, 0)))

in order to avoid the error...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/user/PYNQ-Metadata/pynqmetadata/__init__.py", line 5, in <module>
    from .models.bit_field import BitField
  File "/home/user/PYNQ-Metadata/pynqmetadata/models/__init__.py", line 9, in <module>
    from .dfx_core import DFXCore
  File "/home/user/PYNQ-Metadata/pynqmetadata/models/dfx_core.py", line 10, in <module>
    @dataclass(repr=False)
     ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dataclasses.py", line 1210, in wrap
    return _process_class(cls, init, repr, eq, order, unsafe_hash,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dataclasses.py", line 958, in _process_class
    cls_fields.append(_get_field(cls, name, type, kw_only))
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dataclasses.py", line 815, in _get_field
    raise ValueError(f'mutable default {type(f.default)} for field '
ValueError: mutable default <class 'pynqmetadata.models.vlnv.Vlnv'> for field vlnv is not allowed: use default_factory
>>>

After either one of these changes, import pynqmetadata works in Python3.11.

I am not sure what the better approach is: adding the __hash__ method here, or modifying DFXCore.vlnv to use dataclasses.field with a keyword argument of default_factory which takes in a function which creates a Vlnv instance


Arlec Boston Smart Fan

I purchased a Boston Smart Fan and I spent a few hours messing about with the settings with mixed results. I ended up having the most luck from info I gathered from here:
https://github.com/AMoo-Miki/homebridge-tuya-lan/pull/148#issue-548490889
So the fan support 6 speeds, dimmable light and also white/ warm white
These are the settings I landed on:
{                     "type": "Fanlight",                     "name": "Fan Light",                     "id": "xxx",                     "key": "yyy",                     "manufacturer": "Arlec",                     "model": "Boston",                     "dpActive": 1,                     "dpRotationSpeed": 6,                     "dpLightOn": 9,                     "dpBrightness": 10 }
Now the rotation speed works perfectly with all 6 speeds being accounted for.
I can turn the light on and off but the brightness goes up and down as the slider goes up. It seems to support full brightness and slightly dimmed and alternates between the 2 as I move the slider. I've tried using different numbers for dpBrightness but anything that I've tried other than 10 just doesn't work at all.
I can't find a setting that works with the hue but thats not the end of the world really.
Has anyone had any luck with this model or a similar one?
Cheers!

I have the same fans and have the same experience as you.


Browser fallback breaks on phonegap( sencha touch  ).

With the integrated YUI compressor or even without when loading a remote
file the promise breaks.

Unfortunately we cannot accept this patch. For example, catch is a standard (http://www.ecma-international.org/ecma-262/6.0/#sec-promise.prototype.catch), so if this does not work in Phonegap then you should file a bug with them for that. There is no clear need or advantage to change this on the PDF.js side as all changes here should be supported by modern browsers.

Also, please refer to the discussion in PR #7124.


Can I view only one language?

I don't know where to write it cause I haven't use githerb so I'm writing it here.
The past problem has been solved. thanks to you. also thanks for quick response.
This is just a suggestion. I speak Korean. But I'm playing a global server.
Your program is perfect, but there's one more thing I'd like to ask you about.
Being able to view both languages at the same time is very very convenient, but I wish there was an option to view only one language.
I don't know if it can be realized because I don't know anything about programming, but I'd be really happy if you could check it out.
thank you!

Turn off EXDF transformation rules, and pick a language from "Force display language". Keep "Additional game root directories" settings, and you're good to go.


Refactoring the listener container implementations

          This and `DefaultPulsarMessageListenerContainer` are almost identical up to their listener impls https://github.com/spring-projects-experimental/spring-pulsar/blob/64159ff350de734f3bccea48903557a0266685aa/spring-pulsar/src/main/java/org/springframework/pulsar/listener/DefaultPulsarMessageListenerContainer.java#L226.

Same thing for the abstract parents.
Should we consider consolidating the models?
Is this the last one of these containers we will create on the imperative side? If so, then this is probably fine. The next one that gets created though we should stop and refactor for sharing amongst them.
Originally posted by @onobc in https://github.com/spring-projects-experimental/spring-pulsar/pull/342#discussion_r1106588298

Some of this refactoring has been done when the reactive containers were added. As such, there is no giant refactorings we are going to do from this point. We can improve as we go.


how to get ../dataset/scaffold_0 or scaffold_1 or scaffold_2?

I want to finetune in my datasets by the model.
Do you have this code to get scaffold_0 or scaffold_1 or scaffold_2

We used the scaffold_split function introduced in this page: https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py

Sorry，I was confused.I try the code in grover， but I can't directly generate files such as scaffld_split0/1/2.npy. My code ability is very weak,  do you have any code to directly generate scaffld_split0/1/2.npy?
940118471
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "lihan97/KPGT"                                                                                @.>;
发送时间: 2024年1月18日(星期四) 晚上8:44
@.>;
<EMAIL_ADDRESS>主题: Re: [lihan97/KPGT] how to get ../dataset/scaffold_0 or scaffold_1 or scaffold_2? (Issue #5)
We used the scaffold_split function introduced in this page: https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py

hello, I want to know how to save the finetune model?

我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py
The last question, I'd like to ask you how you did your predictions on the molecules in the FDA dataset, you don't seem to have a code for that, do you have a code for that?

我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py

hello, I want to know how to save the finetune model?

you can modify the fit function in KPGT/src/trainer/finetune_trainer.py as follows to save the finetuned model:
def fit(self, model, train_loader, val_loader, test_loader):
best_val_result,best_test_result,best_train_result = self.result_tracker.init(),self.result_tracker.init(),self.result_tracker.init()
best_epoch = 0
for epoch in range(1, self.args.n_epochs+1):
if self.ddp:
train_loader.sampler.set_epoch(epoch)
self.train_epoch(model, train_loader, epoch)
if self.local_rank == 0:
val_result = self.eval(model, val_loader)
test_result = self.eval(model, test_loader)
train_result = self.eval(model, train_loader)
if self.result_tracker.update(np.mean(best_val_result), np.mean(val_result)):
best_val_result = val_result
best_test_result = test_result
best_train_result = train_result
best_epoch = epoch
torch.save(model.state_dict(), "your path")
if epoch - best_epoch >= 20:
break
return best_train_result, best_val_result, best_test_result

我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py
The last question, I'd like to ask you how you did your predictions on the molecules in the FDA dataset, you don't seem to have a code for that, do you have a code for that?

To make predictions for the FDA dataset, you can simply add this predict function in KPGT/src/trainer
/finetune_trainer.py:
def predict(self, model, dataloader):
# model: your finetuned model
# dataloader: FDA dataloader
model.eval()
predictions_all = []
for batched_data in dataloader:
    predictions, _ = self._forward_epoch(model, batched_data)
    predictions_all.append(predictions.detach().cpu())
return torch.cat(predictions_all)

Thank you for your suggestions.But I don't know how to call the code for this prediction, whether it's evaulation.py or finetune.py？Or instantiate an object to make predictions directly?
940118471
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "lihan97/KPGT"                                                                                @.>;
发送时间: 2024年1月20日(星期六) 晚上10:03
@.>;
<EMAIL_ADDRESS>主题: Re: [lihan97/KPGT] how to get ../dataset/scaffold_0 or scaffold_1 or scaffold_2? (Issue #5)
我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py
The last question, I'd like to ask you how you did your predictions on the molecules in the FDA dataset, you don't seem to have a code for that, do you have a code for that?
To make predictions for the FDA dataset, you can simply add this predict function in KPGT/src/trainer
/finetune_trainer.py:
def predict(self, model, dataloader):
model: your finetuned model
dataloader: FDA dataloader
model.eval()
predictions_all = []
for batched_data in dataloader:     predictions, _ = self._forward_epoch(model, batched_data)     predictions_all.append(predictions.detach().cpu()) return torch.cat(predictions_all)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

You can modify finetune.py as follows:
trainer = Trainer(args, optimizer, lr_scheduler, loss_fn, evaluator, result_tracker, summary_writer, device=device,model_name='LiGhT', label_mean=train_dataset.mean.to(device) if train_dataset.mean is not None else None, label_std=train_dataset.std.to(device) if train_dataset.std is not None else None)
best_train, best_val, best_test = trainer.fit(model, train_loader, val_loader, test_loader)
predictions = trainer.predict(model, fda_dataloader)

I've tried a lot of times, maybe my coding skills are too weak, and it still doesn't work.
What I want to do is fine-tune the model on my own data, and then I want to use the fine-tuned model to do the classification task and predict the activity of unknown molecules.
However, my understanding is that fintune.py code can only be implemented to save the fine-tuned model, and whether it is necessary to write code other than the fine-tuned model to do classification tasks.
Have you done the sorting task for your follow-up?
940118471
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "lihan97/KPGT"                                                                                @.>;
发送时间: 2024年1月22日(星期一) 晚上11:57
@.>;
<EMAIL_ADDRESS>主题: Re: [lihan97/KPGT] how to get ../dataset/scaffold_0 or scaffold_1 or scaffold_2? (Issue #5)
You can modify finetune.py as follows:
trainer = Trainer(args, optimizer, lr_scheduler, loss_fn, evaluator, result_tracker, summary_writer, device=device,model_name='LiGhT', label_mean=train_dataset.mean.to(device) if train_dataset.mean is not None else None, label_std=train_dataset.std.to(device) if train_dataset.std is not None else None)
best_train, best_val, best_test = trainer.fit(model, train_loader, val_loader, test_loader)
predictions = trainer.predict(model, fda_dataloader)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Happy New Year！
I would like to use KPGT to do a virtual screening of mPGES-1 targets, just like the discovery of potential inhibitors using KPGT as a target in the article.
I've fine-tuned my data and saved the model, but I have a problem with the next step of the classification task, I used the code you told me before, but it still fails, can you please give me some more advice, or if there is any other contact for further communication.
940118471
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "lihan97/KPGT"                                                                                @.>;
发送时间: 2024年1月20日(星期六) 晚上10:03
@.>;
<EMAIL_ADDRESS>主题: Re: [lihan97/KPGT] how to get ../dataset/scaffold_0 or scaffold_1 or scaffold_2? (Issue #5)
我们使用了本页介绍的_scaffold_split_功能：https://github.com/tencent-ailab/grover/blob/main/grover/util/utils.py
The last question, I'd like to ask you how you did your predictions on the molecules in the FDA dataset, you don't seem to have a code for that, do you have a code for that?
To make predictions for the FDA dataset, you can simply add this predict function in KPGT/src/trainer
/finetune_trainer.py:
def predict(self, model, dataloader):
model: your finetuned model
dataloader: FDA dataloader
model.eval()
predictions_all = []
for batched_data in dataloader:     predictions, _ = self._forward_epoch(model, batched_data)     predictions_all.append(predictions.detach().cpu()) return torch.cat(predictions_all)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


coalesce with falsy

Hi. I'm finding myself using coalesce for achieving clean and idiomatic code. But sometimes my data providers return me objects, where some keys I pick present, but falsy (undefined, null, empty string etc). In the mean time coalesce relies on key existing/non-existing.
Is it possible to implement in this module another variant of coalesce which will check not key presence, but its value falsiness? So, it will work like good old object.key || object.key2 || ... but with benefits of deep paths support.
I've looked through opened & closed issues, but found nothing similar. If it is a dulicate, please, forgive me. I think other users of object-path could face this issue as well, and it's not pleasant to return to ||-stuff when you're already using such powerful utility like this.

in the upcoming version you'll be able to augment object-path to fit your needs. this would require creating a plugin for object-path and adding another parameter to it, would be like this:
objectPath.extend(function(base){
   return {
      coalesce: function(obj, paths, defaultValue, falsy){
        falsy = falsy !== undefined ? falsy : false;
        var value = base.coalesce(obj, paths, defaultValue);
        if (falsy === true){
          if (!value) {
             return defaultValue;
          }
        }
        return value;
      }
   };
});

@pocesar, hm, your snippet does not represent my idea, since it relies on base coalesce. Consider obj.path = null (but present) and obj.path2 = true. Base coalesce will return null since path in in object, and then default will be applied. In my idea it should return true, since there is such a key with non-fasy value in list. The problem is deeper and that is why I've opened the issue.
Your snippet can be effectively reduced to: coalesce(obj, [ paths ]) || defaultValue. That is not what I meant.

indeed, I've rewritten the snippet. if you want to test the 1.0.0 to see if solves your problem, check the 1.0.0 branch

@pocesar, thanks, no testing required, I've looked through your updated version and it's a solution. I think such thing must be in core, since it's a very low-level code and it has complexity comparable to complexity of coalesce itself.

it makes sense to have a fourth parameter to change the behavior of the function (like set has the internal option doNotReplace), it would not break existing code that relies on coalesce

@pocesar, of course it can save compability. This can be done with boolean parameter or predicate for coalese or even distinct function.

@pocesar, I have an idea of such design:
.first(object, paths[], predicate = Boolean);

— for every path in paths checks if path is present and the predicate(value, path, object) is true for it. Returns first key for such conditions.

I'm not sure if defaultValue is required, but if so, it should stay before predicate (because predicate will be Boolean most of the time, and for consistency with coalesce):
.first(object, paths[], defaultValue = undefined, predicate = Boolean);
If user do not pass predicate at all, or predicate is Boolean it can be aggressively optimized to short form without running callback at all (like in your snippet above).

that would definitely be a good plugin to have :)
1.0 is embracing modularity, and we will be able to add awesome things using the base of objectPath. we expect the community to make some nice plugins to use out of the box


Document NIX_REMOTE_SYSTEMS

The documentation for the configuration file and setup for distributed builds is extremely lacking. We need to document the setup and the format of the configuration file. @cleverca22
This would also be a great time to document the process for setting up package signing on multiuser remotes.

another area that isnt documented very much, hydra will accept a : seperated list of files in NIX_REMOTE_SYSTEMS but the nix perl hook does not

I think this is nix feature, not nixos feature. It's documented in http://nixos.org/nix/manual/#chap-distributed-builds
NixOS as far as I could find uses buildMachines and distributedBuilds configuration attributes (they are not very well documented).
I think this ticket should be closed or a reference should be created to Nix manual.

@spinus I think there is a glaring hole in documentation somewhere. It may not be here though. Before closing this let's at least open a ticket in the right place.

But you're right that a link to the docs might be all that's needed.

Let me know if https://github.com/NixOS/nixpkgs/issues/24914 is good enough

@spinus Perfect!


Introduced background and handbook organisation, reworked text a bit

Describe the pull request:

[ ] Bug fix
[ ] Functional change
[ ] New feature
[ ] Code cleanup
[ ] Build system change
[x] Documentation change
[ ] Language translation

Pull request long description:
Reworked the front page according to discussions in the handbook working group.
Changes made:

Added brief introduction on what SDA is
Reworked the existing text around figure and in table to describe three main archive sub-processes: Submission, Ingestion and Data Retrieval, removed the "Status" column in table
Introduced the overall organisation of 4 main section in the text, and briefly their intended content.
Kept original "unused" text towards the bottom indicated with a note to be considered reused in section intro pages or elsewhere more suitable.

Mentions:

@norling

All very good suggestions Martin, accepted and commited now:)

please add the spelling words highlighted in https://github.com/neicnordic/neic-sda/actions/runs/3743236980/jobs/6355162208#step:4:1 i docs/dictionary/wordlist.txt so that next time they are ignored

@blankdots Thanks for the instructions, words added now :)


Send notification again on rerunning single task

Coverage decreased (-0.58%) to 65.05% when pulling efe79003dc5de7dbf047cc318edbb3d6e553bdc7 on guidow:new_notification_on_rerun into e5cf7834fee2f3fe75e40632dab32d4ff5d0808e on pyfarm:master.

Coverage decreased (-0.58%) to 65.05% when pulling efe79003dc5de7dbf047cc318edbb3d6e553bdc7 on guidow:new_notification_on_rerun into e5cf7834fee2f3fe75e40632dab32d4ff5d0808e on pyfarm:master.

Looks good.  I'll go ahead and merge this since there's not any tests here that this could break.


[SourceKit] Fix crasher when retrieving conforming methods

In the added test case, the generic signature of myFlatMap cannot be inferred because the closure’s return type is unresolved (because of the code completion token) and there is no information about the type of SegmentOfResult in the return type, except that it has an Element associated type of String.
Thus myFlatMap returns Foo<_.Element> where _ is an unresolved type. Apparently this type checks fine until we try and coerce it to Foo<String> because of the as Foo<String> expression. By now we have added a deep equality restriction to the constraint system. But since Foo<_.Element> and Foo<String> are not equal, we are hitting the llvm_unreachable.
AFAICT the simple fix is to skip over the check if the from type contains an unresolved type, similar to what we do for toType.
Fixes rdar://76686564

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 2ed06a4d21ee62d862bf072908284d73f55815d5

Build failed
Swift Test OS X Platform
Git Sha - 2ed06a4d21ee62d862bf072908284d73f55815d5

@swift-ci Please smoke test

Yeah, AFAICT conforming method lists hasn’t been ported to typeCheckForCompletion yet(https://github.com/apple/swift/blob/main/lib/IDE/ConformingMethodList.cpp#L71).

@xedin Thanks for your feedback. As discussed offline, removing the AllowUnresolvedTypeVariables flag seems to fix the crash and not cause any other regressions, so it seems like a reasonable fix to me (especially since you would like to eliminate allowing unresolved type variables anyway).

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 74ff6923a1161052debedcad3aca5ddf2a1274ca

Build failed
Swift Test OS X Platform
Git Sha - 74ff6923a1161052debedcad3aca5ddf2a1274ca

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 3d1f9b640289f1ed4f10870f9585fd1aeb7cc61c

@swift-ci Please test macOS

We should remove another usage of TypeCheckExprFlags::AllowUnresolvedTypeVariables in visitReturnStmt.  In fact the reproducer in rdar://76686564 is not fixed without removing it (probably because it's in an implicit ReturnStmt).

@ahoppen @rintaro It looks like there are just a couple of usages of TypeCheckExprFlags::AllowUnresolvedTypeVariables in lib/Sema, should we try to remove all of them and see what in code completion breaks?


Test and benchmark karatsuba

Tests and benchmarks for PR #6
If no one minds, I am planning to merge it in a week.

Cool! Were there any intersting findings from the benchmarking?
I'll be at HacBoston this weekend and I'll give things at least a skim
On Tuesday, August 2, 2016, Bodigrim<EMAIL_ADDRESS>wrote:

Tests and benchmarks for PR #6
https://github.com/cartazio/arithmoi/pull/6
If no one minds, I am planning to merge it in a week.
You can view, comment on, or merge this pull request online at:
https://github.com/cartazio/arithmoi/pull/32
Commit Summary

Expose sqrt on Integer via internal module
Write tests for integerSquareRoot
Write becnhmarks for integerSquareRoot
Fix 7.6 and 7.8 builds

File Changes

M Math/NumberTheory/Powers/Squares.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-0 (122)
A Math/NumberTheory/Powers/Squares/Internal.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-1 (144)
M arithmoi.cabal
https://github.com/cartazio/arithmoi/pull/32/files#diff-2 (4)
M benchmark/Bench.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-3 (27)
A benchmark/Math/NumberTheory/ArithmeticFunctionsBench.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-4 (34)
A benchmark/Math/NumberTheory/PowersBench.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-5 (30)
M test-suite/Math/NumberTheory/Powers/SquaresTests.hs
https://github.com/cartazio/arithmoi/pull/32/files#diff-6 (18)

Patch Links:

https://github.com/cartazio/arithmoi/pull/32.patch
https://github.com/cartazio/arithmoi/pull/32.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cartazio/arithmoi/pull/32, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQwnwjngemg0osW-Lp4snR1KxLngzNks5qb7mbgaJpZM4JbFH1
.

Benchmarks showed that Karatsuba sqrt is faster than Heron starting from ~2300 bits. Previously the threshold was set to 4096 bits.


CSV-report - Timeline

Dear asterictnl-lvdw,
do you have any expectation when the CSV-report will be included?
I see the hint "in the next release" since a few releases back.
(I don't wanna push you, just for my/our information, because we (ps-tron and me) would really appreciate it :-) )
Best
M0nk3y

Good morning,
Yeah, it has been a bit more work than usual since the structure of translating the object and displaying it correctly in a .csv file did not kinda work properly. Also, there were other features that had more priority over it. But I will promise that I will put it in the next release or in a quick bugfix release, depending on what will be changed this version.
I might release it to Alpha so you can test it out there to see if it's what you are expecting at the moment.

Thanks for your fast reply, that would be great! :-)

Hoi,
Could you give a little bit more context about this? It would be great to determine which information you want in the .CSV to see if this information is important for an audit for example or not.
Thanks in advance.
~Leonardo

Hello @M0nk3yOo ,
Thank you for the submission. This would clearly be two enhancements that have some relation to each other.
What I am going to do is the following:

I am going to automate the scripts where possible that are manually checked on the first hand. Some of the things could be automated via scripts. But I did not find everything yet. I need some more research about that.
I am planning once v4.0.0 of the Microsoft 365 benchmark comes out to review it once again and to implement the changes and also to see what manual checks need to be implemented alongside to fully automate things.
The same is for the Azure benchmark as I want to wait until that release is done as well. I want to change some things around and implement it back to the CIS v3.1.0, v3.0.0 and v2.0.0 and maybe even the v1.x.x if I have the time

Currently I haven't put all the information in and only the necessary information. The script is there for to allow checking upon CIS checkpoints to determine if settings are compliant or not and not for actual CIS-reporting at this moment.

I will look into adding the OK status as well with a score of 0 or OK depending on what I find better looking for the report. This will make the report clearer and users can see what they already have compliant and what they need to fix.
The other thing about the .csv I will push something to Alpha once I am done and I will allow you some testing to see if it's something that is upon your liking as well.

~LvdW

Hi @asterictnl-lvdw
many thanks for your detailed explanation! :-)
I really appreciate it.
Maybe it makes sense, even if not all checks can be done automatically, to have them in the report/csv(?)
Best

Can you test the alpha branch for me @M0nk3yOo ? I have submitted the .csv module and I want to check if it has the things that are required and of course if it works.
I will test it myself later as well. But I am busy developing and looking at some things as well.
~Leonardo

Hmm... I'm having an issue while running the script(s).

This is while connecting to Graph (Connect-MgGraph). Do you have a hint for me where the permission change is needed? I remember that there is a registration/permission assignes while running the script the first times, but now I'm having this issue and have no clue (if I'm serious) which app should have the additional permission

Ah, this is related to an implementation in #34 I will remove the Policy.Read.DeviceConfiguration because it seems that it is no such permission there. I will push another alpha release with that fix.
~LvdW

Update: You can try and pull the latest version of Alpha now. It should have been fixed by now so you will get no error by logging in anymore.
~LvdW

The issue itself is solved :-)
(I think - I deleted some references and apps from my profile, don't know if it works because of your change, or what I did. But I think it's more your fix :-) )
Topic CSV: The CSV is created succsessfully, but the result is (today) not in that format (and its content), what we 100% think. But I think the baseline ha a very good stand.
Maybe, if I have some time (trying to reach some MS-certificates such as SC-200), will create some additional CIS-checks.

That's good! Could you tell me what you are missing so I can see if I can add it and if it is valuable to add to the CSV report.
Feedback is always welcome!
~LvdW

Hi, I just want give a "hello" from my side, I'm still here, but still had no time to work on that topic, but I will (but don't know when)!
Best

Dear @M0nk3yOo ,
Sorry for the late reaction.
I will process your feedback in the alpha branch and will give a reply here once you are able to test.
I'll add the CIS-ID in the CSV report. To achieve this, I have to make changes in the inspector modules.
Don't worry, because I have the controls as well so I can easily look them up and the ID is processed in the finding name as well.
For the Level and license, I have to make additional attributes in the PSObject in each inspector as well. This is also handy for the HTML report so I will process this as well in the alpha branch.
I have been looking at the PASS/FAIL schema on how to implement this as this requires code changes as I can add this either in the validation process and pass this as a parameter into each Build function or I can do this a different way.
If I am going to implement the PASS/FAIL structure I am going to create a formula to determine the risk-score as the maximum risk score is determined onto the number of inspectors being ran at that point.
I need more time onto the PASS/FAIL as this depends on multiple factors within the code which requires code change in additional places and not only the inspector modules. I will hopefully bring this feature up to the next major release including with new things.
Donations are very much appreciated <3
I hope I have informed you in proper manner. :)
~LvdW

Hi @asterictnl-lvdw
Is it possible to have an update concerning this request ?
I am also very interested by the possibility of creating reports in CSV format.
Thank you in advance for your reply.
and congrats for all this work

Hello @olivierdumon ,
Thank you for your reply.
The optimalizations regarding the .CSV-format are planned for the next major release.
Since with CSV you want the information to be as simplified as possible, I am looking at besides the feedback which was given earlier to look on how to optimally implement it altogether which benefits not only the .CSV file but also the .HTML file, because there are multiple things are gone well together and could be implemented simultaneously.
I hope to a tiny release at the beginning of Q4 (September) and hopefully somewhere before the end of this year, maybe even sooner the major release including optimalisations in both the CSV and the whole reporting structure.
I hope this gives you a brief overview on what to expect.
This to let you know that it is on the planning, but it takes more time to process and there are still some important bugs to fix first and to release the v3.1 of the CIS first before doing optimalisations on the whole CSV exporting engine.
~LvdW

Hi @asterictnl-lvdw
Thank you for your feedback
I'll be following your project closely :)
Thank you for your work

Hi @asterictnl-lvdw,
regarding the donation: is there a way to get in direct contact to you? Maybe via mail?
Thanks, and don't melt away today :-)

Hi @M0nk3yOo ,
On my profile there should be a LinkedIn of mine. You can send me a message there.
~LvdW

Hi everyone,
Just to give a quick update about this. I am working on version 3.0 which will hopefully include the CSV support including the additional feedback above implemented. It takes more time than planned, because there need to be changes into the way of creating Inspectors as well for the .CSV lay-out.
~LvdW

@M0nk3yOo , @olivierdumon ,
How does this look?

Is this sufficient, because I need to fix all the scripts. This is important for me to know.
I will reset the titles maybe to something different, depending on what the context would be.
~LvdW

Hi Leonard,
I think this looks very good. I think "Status" includes the fail/pass result. This should be available in the CSV, yes! :-)
For debugging/evidence purpose it might also be nice, if the returned and expected vales are in the report, too. But this should be easy, as long as you have the inspectorObject
Best
M0nk3y

@M0nk3yOo ,
Expected and Default values are already in the HTML report where applicable. I will be converting the M365 inspectors first and test them under CUSTOM. A lot of changes have to be implemented regarding this since we are changing the output and validation as well as you want to return all the inspectors now and not only the ones that failed.
For now I have the following schema:

UUID (which consists the Unique identifier of the inspector)
ID (the actual ID which is basically a translation of the UUID with only the numbers e.g. 1.1.1)
Title (The original title in the CIS Benchmark)
ProductFamily (Depending on what the audit script says it is placed under a category)
DefaultValue (Default value if any)
ExpectedValue (Expected value based on the audit schema)
ReturnedValue (The actual value which is returned)
Status (PASS/FAIL/UNKNOWN)
Notes (Which is blank as you can fill this in yourself)
RiskScore (is not included in the .CSV)
RiskRating (is not included in the .CSV)
Description (which only includes the Rationale and some of the description)
Impact (which includes the impact or if there is no impact it will be measured)
Remedation (which is the PowerShell script or the URL if applicable
References (if any)

Reason that I am not going to implement Partial as Partial is a relative definition which can be adjusted after the audit if the case is that it is partially implemented. But looking at it there is only (implemented or not implemented).
If this is implemented this will be available for testing in the alpha branch first. I will post another comment once it is done.
~LvdW

@M0nk3yOo ,
The first alpha version of the .CSV mechanism is pushed to alpha.
You can execute it with the M365SATTester.ps1 script. This is the command:
Get-M365SATReport -OutPath "C:\Out" -Username<EMAIL_ADDRESS>-EnvironmentType CUSTOM -BenchmarkVersion "Latest" -Modules Office365 -LicenseMode "E3" -LicenseLevel "All" -reportType "CSV" -AllowLogging "Warning" -LocalMode -SkipChecks
It is not executing all the inspectors within the paragraph 1 as some are within Exchange as well. Do not change anything in the M365SATTester.ps1 if you want to test the .csv functionality. The full command does not use the experimental mode. The custom inspectors do not work with the normal mode as they have different output formats.
Let me know if you have any feedback regarding the .CSV. I have attached the example_report as an attachment to this comment.
example_report.csv
~LvdW

Dear @asterictnl-lvdw,
that looks amazing and will be very very helpfull for us! <3
Here are some feedbacks from my side

it seems, that there is an issue with the PS-output. CSV says that some checks are successfull, but console says that all failed

From my opinion it is very helpfull to see the expected values and the returned one, perfect! I think we can fill it with some more details. For example in 1.1.2 is expected to have at least 2 global admin accounts, and the result (in my test environment) was 3. So it's not equal. Someone could say "hmm... bad for me", but I think the expected value could describe "more than 2" or "> 2" (in correlation with 1.1.3 for sure).
The CSV file itself (cosmetic): if I open it in Excel, and split it up into columns, the ID (control ID) is broken. Excel interprets it as date.

Maybe (not the "perfect" solution could be to add a "blank" at the beginning of the ID, so Excel doesn't format it as date (tested in 1.1.1 as you can see)

Question: what If, if one control fails from script perspective (sth. in $error). Will the remediation has the status "ERROR" or "SCRIPTERROR"?
Best & have a nice WE

@M0nk3yOo ,
I gave you the wrong script. Please add an extra parameter called: -ExpirimentalMode because it does not work with the normal version.
Get-M365SATReport -OutPath "C:\Out" -Username<EMAIL_ADDRESS>-EnvironmentType CUSTOM -BenchmarkVersion "Latest" -Modules Office365 -LicenseMode "E3" -LicenseLevel "All" -reportType "CSV" -AllowLogging "Warning" -LocalMode -SkipChecks -ExpirimentalMode
I have fixed it in the M365SATTester.ps1

Yes, that looks better regarding the results in console.

The result of 1.2.1 is missing in CSV

@M0nk3yOo ,
Thank you for reporting.
I have fixed the CISMOFF121 the status should now be correctly displayed. There were some issues returning the correct value. I have tested this, so now the Microsoft365 part level 1 should be working as expected.
~LvdW

Hi @M0nk3yOo ,
So far I have implemented from the M365 everything but Teams.
Teams will be implemented next week in the .CSV format for testing. Then I will provide the correct command so you can audit everything and check if it is working and displaying all correctly. Then I can also check if I need to make cosmetic changes and such.
If the format and lay-out is correct, then I will proceed converting the AZURE as that one is more challenging.
~LvdW

Hi @olivierdumon and @M0nk3yOo,
I want to inform that all the M365 modules are now converted to support the .CSV format. I need to verify if you guys get the correct output as well. You can run this script:
Get-M365SATReport -OutPath "C:\Out" -Username<EMAIL_ADDRESS>-EnvironmentType CUSTOM -BenchmarkVersion "Latest" -Modules "All" -LicenseMode "E3" -LicenseLevel "All" -reportType "CSV" -AllowLogging "Warning" -LocalMode -SkipChecks -ExpirimentalMode
Make sure to change the username to the desired option.
Kindly let me know the results.
~LvdW

@M0nk3yOo ,
v4.0.0 is included in the alpha branch. You can use it there. I want to release the .CSV compatibility along with all inspectors fully converted. For the rest I saw there is no update released yet. I am looking at the workbench carefully to determine if updates are released and when I have to check and implement the new versions of the inspectors.
So far, the latest versions are v4.0.0 for M365 and v3.1.0 for Azure.
~LvdW

Hi @asterictnl-lvdw
do you have the same problem as me? I already updated the modules (tested with PS7)

Best

Hello @M0nk3yOo ,
Please remove all modules and install them one-by-one again to see if the issue pre-exists.
I will test for you and come back with results.
~LvdW

@M0nk3yOo ,
This is an issue regarding WAM and the new login experience that has been released in v3.7.0. You can find details about that Changelog here. I am going to file a ticket to Microsoft so they can add a parameter that disables the WAMLoginExperience.
https://blog.icewolf.ch/archive/2024/12/03/exchangeonlinemanagement-3-7-0-released/
A workaround is to downgrade the module to 3.6.0 until further notice as that module does not use the WAM mechanism and works with the flow.
I will keep you updated about this.
~LvdW


Should upgrade the operator_sdk.util version to "0.3.1" for the "k8s event" module

Bug Report

What did you do?
Generated an ansible-operator using operator-sdk: operator-sdk init --plugins=ansible --domain=k8sevent.com
What did you expect to see?
$ cat requirements.yml
---
collections:
  - name: community.kubernetes
    version: "1.2.1"
  - name: operator_sdk.util
    version: "0.2.0"
  - name: kubernetes.core
    version: "2.2.0"

What did you see instead? Under which circumstances?
Expected to see a newer version of the operator-sdk.util
cat requirements.yml
---
collections:
  - name: community.kubernetes
    version: "1.2.1"
  - name: operator_sdk.util
    version: "0.3.1"
  - name: kubernetes.core
    version: "2.2.0"

Environment
Operator type:

/language ansible

Kubernetes cluster type:
OpenShift and Kind
$ operator-sdk version
operator-sdk version: "v1.15.0-8-g1d009d6a", commit: "1d009d6a36661f9beec45f3d9927da905428c800", kubernetes version: "v1.21", go version: "go1.16.8", GOOS: "linux", GOARCH: "amd64"
$ go version (if language is Go)
go version go1.16.8 linux/amd64
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.1", GitCommit:"5e58841cce77d4bc13713ad2b91fa0d961e69192", GitTreeState:"clean", BuildDate:"2021-05-12T14:18:45Z", GoVersion:"go1.16.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.1+6859754", GitCommit:"2ade4d46986c94405aa0542f8d6ff7691bddb153", GitTreeState:"clean", BuildDate:"2021-12-03T16:26:49Z", GoVersion:"go1.17.2", Compiler:"gc", Platform:"linux/amd64"}

Possible Solution

Additional context

https://github.com/operator-framework/operator-sdk/pull/5462

Fixed by PR #5462


Show Organizer & Fix BST

If Organizer is present on an event show it, stripping out mailto: in the
string. Also move location before description as often descriptions are much
longer than the location.
Added check for BST for when timezone reports British Summer Time not DST

Seems straightforward, thanks!

:frowning:  I knew that code was somewhat brittle (relying on __repr__() doesn't sound like the best idea ever), but all timezones I had tested seemed to work. I really should find the time to create timezones from some kind of cache (#146).


Can't pass values to the int[] option

I don't really have idea where to seek for help so I'm writing an issue here.
So a bit of clarification I have parent command cimp which have subcommand affine which have subcommand encrypt. I only have a problem with passing a pair of integers as option in array.
(I also have some parameters but they are not necessary right now)
So I have following code in encrypt class:
@CommandLine.Command(name = "encrypt", aliases = "e")
public class AFEncrypt implements Callable<Integer> {

    @CommandLine.Option(names= {"-k, --key"}, split = ",")
    protected int[] key;

    @Override
    public Integer call() {
        System.out.println(Arrays.toString(key));
        return 0;
    }
}

But when I try to do java -jar kmzi-1.0-SNAPSHOT-all.jar cmip affine encrypt --key 1,2
I get following output:
Unknown options: '--key', '1,2'
Usage: cmip affine encrypt [-k, --key=<key>[,<key>...]]...
      -k, --key=<key>[,<key>...]

I'm really not sure what is going on. Is my input wrong or something?

 Sorry for disturbance and thanks for your help :)

No worries. 👍  We're only human. 😉
How do you like picocli so far? ❓

Don’t forget to star ⭐ picocli on GitHub if you like the project! 😉

Hey @remkop, I hope it's okay to tag you. I do like picocli (I made a little project for my assignment with it before, but now I ran into some problems)
So my goal is to make a program with following command structure:
cmip
│
├── affine
│      │
│      ├── encrypt
│      │
│      └── decrypt
│
└── some other command for next tasks

But I keep getting Unmatched argument at index 0: 'cmip' if I use my subcommand properly. So right now I have the following code:
@CommandLine.Command(name = "encrypt", aliases = "e")
public class AFEncrypt implements Callable<Integer> {

    @CommandLine.Parameters()
    public String file;

    @Override
    public Integer call() {
        System.out.println(file);
        return 0;
    }
}

So executing java -jar kmzi-1.0-SNAPSHOT-all.jar cmip affine e test.txt should just print test.txt in console, but I get Unmatched argument at index 0: 'cmip' instead.
Just for the sake of context I'm running my jar in WSL2 and building it with Gradle and ShadowJar plugin.
Also for context how I handle subcommands (it's pretty much the same everywhere):
@CommandLine.Command(name = "affine", aliases = "af", subcommands = {
        AFEncrypt.class,
        AFDecrypt.class
})
public class AffineCipher {
    // nothing here
}

The help message works just fine by the way (If I try to run command without file paramter) I get following message:
Missing required parameter: '<file>'
Usage: cmip affine encrypt <file>
      <file>

I guess it would be more convenient to ask such kind of question on StackOverflow, but since I made an issue here I thought to give it a try while I'm at it. I hope that's okay.

@realkarmakun First of all, I welcome all questions. It is no problem to use GitHub issues for questions or to tag me in questions, I do prefer to have a new GitHub issue for each question; that way, if the question reveals some problem in picocli, it is easier to track for me if the full history is in the GitHub issue. 😉
About your question (why does this command result in an error):
java -jar kmzi-1.0-SNAPSHOT-all.jar cmip affine e test.txt
Unmatched argument at index 0: 'cmip'

This (without cmip) should work correctly:
java -jar kmzi-1.0-SNAPSHOT-all.jar affine e test.txt

Why: the way I think of it, java -jar kmzi-1.0-SNAPSHOT-all.jar is the cmip command.
This allows you to create a wrapper script called cmip that invokes java -jar kmzi-1.0-SNAPSHOT-all.jar @*, or even use GraalVM to convert kmzi-1.0-SNAPSHOT-all.jar into a native image called cmip: that way, your end users can just invoke the application like this:
cmip affine e test.txt

The picocli manual has a section on packaging your application, in case you are interested in creating such wrapper scripts or using GraalVM.
This stackoverflow question has some more thoughts on this topic.
Don't hesitate to ask more questions!

Oh, that actually makes sense. Thank you very much, now it works. By the way Graal native image looks interesting I'll look into that. And thank you again :smile:


white flash when alt+tab between vscode windows

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
Version: 1.66.2
Commit: dfd34e8260c270da74b5c2d86d61aee4b6d56977
Date: 2022-04-11T07:49:24.808Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 4.9.0-18-amd64
OS Version:
Debian 9
MATE Desktop Environment 1.16.2
Virtual Box 6.1.38 QT(5.6.2)

Steps to Reproduce:

Open 2 Windows of VSCode.
Make them full screen.
Alt+tab between the windows
A white flash(white screen) is shown when the windows are switching place.
It looks like the window that is switched has a blank screen and then loads the actual content
Especially noticeable when using dark mode! I've resorted to light mode.

Are you running on wayland ? Does this issue repro with latest stable 1.74 ?


[gui error report] Error: Initializing daemon...

👉 An error occurs when I open ipfs desktop for the first time
Specifications

OS: darwin
IPFS Desktop Version: 0.20.6
Electron Version: 18.0.3
Chrome Version: 100.0.4896.75

Error
Error: Initializing daemon...
go-ipfs version: 0.12.2
Repo version: 12
System version: amd64/darwin
Golang version: go1.16.15
Found outdated fs-repo, migrations need to be run.
Looking for suitable migration binaries.
Need 2 migrations, downloading.
Downloading migration: fs-repo-10-to-11...
Downloading migration: fs-repo-11-to-12...
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-10-to-11/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-10-to-11/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions"
Fetching with HTTP: "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-10-to-11/versions"
Error fetching: exceeded number of retries. last error was http.DefaultClient.Do error: Get "https://ipfs.io/ipfs/QmdaCHYBDHEhXCMoynH5UcohEay6m1XayZCcxWZzKAHNVN/fs-repo-11-to-12/versions": dial tcp <IP_ADDRESS>:443: i/o timeout
Error fetching: exceeded number of r

For now, I recommend you to retry again and ensure you have Internet connection during migration. Please also check if you have any firewall or antivirus that may be blocking the traffic.
Continued in https://github.com/ipfs/go-ipfs/issues/8911


(OraklNode) aggregator roundID mutex & start round id update

Description
RoundID mutex lock
RoundID is accessed through go thread which makes it thread unsafe
This PR tries to prevent unexpected outcome due to concurrent access to roundID (but not likely to happen 🥹)
Start RoundID update
Previous start roundID was set as latest round from db.
it should be latest round id + 1 to prevent duplicate roundID
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

will be implemented from other pr


Separated dev and prod middlewares

In order to make #1236 a bit skinnier I've extracted the changes I made to the server side scripts.
Please help me with reviewing this PR.

Coverage remained the same at 99.24% when pulling dbb2bf35d7251fb0f5462b1850bfad172a850658 on tomazy:prepare-server-code-for-ssr into 9cfdf6df0b0bdd15c68d4efd81c63a388bc64fb1 on react-boilerplate:dev.

Hi @tomazy I would love to help out where I can, but I have zero experience with SSR. Is there any chance you could explain the reason behind these changes?

Coverage remained the same at 99.242% when pulling 5c22b176814b82312e0da40bb5e6800b8f5a9a8b on tomazy:prepare-server-code-for-ssr into 4a6a8d99d93db2d32950bc39d8d08e1cab093485 on react-boilerplate:dev.

@samit4me The main reason for this PR is to make the future SSR merge easier to digest, but it also makes the current servers side logic clearly separated depending on the environment, which IMO brings a little bit of a cleaner code here.


merging master into release-3-0-0 branch

to: @chunyong-lin / @Ryxias
cc: @airbnb/streamalert-maintainers
Merged Changes

Hotfix/links and spelling (#967)
Bumped slack app timeout (#983)

Coverage remained the same at 96.609% when pulling 0846f2bc56a15a91482aafa7507bd791ac25fc41 on merge-master-release-3-0-0 into c939d1b31ceb181a0d44da75a0e1d61fb3580bcd on release-3-0-0.


Move fh-js-sdk to use upstream sync client

Motivation
Remove sync related source code from fh-js-sdk and use upstream sync client.
Upstream repository: https://github.com/feedhenry/fh-sync-js
Changes map
Removing sync from library
https://github.com/feedhenry/fh-js-sdk/commit/b37954d168906176c47bb6d564fb1208a210f36d
Mounting sync to global namespace
https://github.com/feedhenry/fh-js-sdk/commit/1990255d766e2a4314ecc9b2b5f52b3340e24050
Progress

[x] Cleanup repo from sync
[x] Add sync to build process (browserify)
[x] Add custom handler for sync (use fh.cloud)
[x] Documentation update
[x] Release RC version of fh-sync-js to npm
[ ] Platform testing
[ ] Publishing package (RC)
[ ] Updates in all required dependencies
[ ] Publish production version of packages
[ ] Notify group about this change

Ticket: https://github.com/feedhenry/fh-sync-js/issues/2

@jasonmadigan - Do I have 🍏 green light to decouple client from product point of view?
@vchepeli - do you have time to help me with basic verification for sync?
This is Raincatcher related action - we do not want to use js-sdk anymore in raincatcher code.

@wtrocki I'm :+1:  on this if this means the $fh API is 100% backwards compatible  i.e. a build of this branch will work with existing apps using the previous version of the sdk.

@wtrocki Taking a look now :)

@david-martin only one difference between is that sync will no longer use fh.act - just fh.cloud. Act was deprecated for couple years and after consultation with @wei-lee we decided that we no longer need it. Rest of the code is 100 % the same.

@wtrocki What is the plan for the TypeScript declarations for sync, I see they have been moved to fh-sync-js, but they are duplicated here. That seems like it will cause issues with maintenance of the declaration files?

@aidenkeating Actually previous version I had had attempt to bring and integrate type definition file, but I had problems to do this effectively. We cannot just remove sync stuff from here as users will use fh-js-sdk as whole with sync being internal part of it. Tradeoff is to always maintain definitions in fh-sync-js and then update them on release.

@wtrocki I'm definitely on-board with doing this. Modularising our SDKs like this has been on the radar with a long, long time and I'm glad to see this tackled with Sync as a first step 👍

Verification steps

Use sync acceptance testing project:
https://github.com/feedhenry/sync-acceptance-testing

npm link this version fh-js-sdk

Run tests

RC version published for tests<EMAIL_ADDRESS>

Self verification results :]

[x] Verified using acceptance tests
[x]  Verified locally using example template.
[x] Verified on feedhenry mbaas target.
[x] Verified on OpenShift.

@wtrocki Has the fh-sync-js code been tested as a standalone module? It looks like the fhparams (cuid) etc. wouldn't be sent and sync would not work? I might be missing something though?

@aidenkeating - Going to supply this params in upstream version. This version is working fine as it's using fh.cloud calls, which are supplying this parameters.

The code shows fine, it is just don't respecting the eslint rules. However, I have some doubts about this change. What exactly in the functionalities it can affect? All information about how works the fh.cloud, fh.act and the fh.sync into the Client API docs[1] will still the same after this change?
[1] - https://access.redhat.com/documentation/en-us/red_hat_mobile_application_platform_hosted/3/html-single/client_api/#fh-act
Observations It is really important that all changes be tracked into the JIRA for the product owner has the visibility and for the support team knows that something was changed if came a case for example. It is to hard if we have more than on place to looking for something that can affect the functionality if the things are not all mapped in the same place into the same way.

@camilamacedo86 This change is 100 % compatibile. We just moved code to separate repository. Nothing else happened here.

Rebased on top of the master and retested. Any concerns before merging this?

Change was verified by Camila and Paolo. If no concerns I'm going to merge that tommorow.

Had similar questions to @aidenkeating but you've cleared them all! 👍

Should we make this a major version bump to be safe? For example, perhaps the new upstream client could have breaking changes. It sounds like you tested and it's safe, but just a suggestion.

@evanshortiss - Major version bump should be used to mark release with some breaking changes.
We just moved existing code so there is no breaking changes.

@wtrocki If it has been verified by others then I am happy for it to be merged and published. If you need any extra verification then I'd be happy to.


Fix streaming data parsing in OpenAI::HTTP's to_json_stream

Problem:
The to_json_stream method failed to handle JSON objects split across multiple data packets from the Azure AI service, leading to JSON parsing errors and incomplete/missing data.
Solution:
Implemented a buffering mechanism within to_json_stream to accumulate data until a complete JSON object is formed. This ensures proper parsing and data integrity, regardless of how the data packets are received.

There are a few RuboCop errors present, but they're unrelated to the changes I've introduced with my fix. I've decided to keep these outside the scope.

Hey @mrbokx, simple buffering won't cut it. Even though event streams look trivial to parse with simple regex + string manipulation, a proper state machine needs to be involved to handle all edge cases.
I'm fixing it here: #338

@atesgoral 's fix released in 5.2.0 - thank you for your work on this @mrbokx !


Drag and Drop functionality

Allow the user to drag songs onto playlists and queue etc...

Yeah this would be handy. I don't have a lot of time free atm but I'm open to pull requests!

Closing this on github as it's not a bug.
Feature has been added to the trello board: https://trello.com/c/R07FJX6f/56-drag-and-drop-songs-onto-playlists-queue


release 2.83.0

What are you trying to accomplish?

What approach did you choose and why?

Anything you want to highlight for special attention from reviewers?

Hi @camertron I still get this after upgrading to VC 2.83.0, any clue?
$ bundle exec bundle-audit check --update
Updating ruby-advisory-db ...
Depuis https://github.com/rubysec/ruby-advisory-db
 * branch            master     -> FETCH_HEAD
Déjà à jour.
Updated ruby-advisory-db
ruby-advisory-db:
  advisories:	843 advisories
  last updated:	2024-01-09 11:59:21 -0800
  commit:	02b54706bbb11656e36ce2e6a45d554bd207fce8
Name: view_component
Version: 2.83.0
CVE: CVE-2024-21636
GHSA: GHSA-wf2x-8w6j-qw37
Criticality: Medium
URL: https://github.com/ViewComponent/view_component/security/advisories/GHSA-wf2x-8w6j-qw37
Title: view_component Cross-site Scripting vulnerability
Solution: upgrade to >= 3.9.0

Vulnerabilities found!

@Spone were you able to get to the bottom of this? I saw your comment on the advisory db repo, did that fix things?

Yes! Sorry I forgot to follow up here, the problem has been solved with the security advisory update in bundler-audit.


Ability to search for students and matches

The user should be able to input a student firstname, lastname, firsname and lastname, student number or match number into the search field and find some results.
Requirements

[ ] Students first
[ ] Firstname results first
[ ] All clickable

Note: The listview will show open matches in the current round, search will search all matches and students, closed or not.

Next on the list of stuff to get done. Much easier with changes made in be30fed.


[Snyk] Upgrade html-snapshots from 1.8.1 to 1.9.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 1.8.1 to 1.9.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released a day ago, on 2023-02-21.

Release notes

Package name: html-snapshots

1.9.1 - 2023-02-21puppeteer 19.7.2

1.9.0 - 2023-02-17puppeteer 19.7.0

1.8.1 - 2023-02-06puppeteer 19.6.3

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

904ffd3 Merge pull request #278 from localnerve/rc-1.9.1
dd54519 @ 1.9.1 - puppeteer 19.7.2
8798afa Merge pull request #275 from localnerve/snyk-upgrade-454c551fb839bc71d8ba63c90e70ebf3
2ab94e3 fix: upgrade html-snapshots from 1.8.1 to 1.9.0
89f7c38 Merge pull request #274 from localnerve/snyk-upgrade-dfa0ea791b54264746feab5903b246df
e44faad fix: upgrade html-snapshots from 1.8.1 to 1.9.0
ee592ef Merge pull request #273 from localnerve/rc-1.9.0
eb08915 @ 1.9.0 - puppeteer 19.7.0
19df0c0 Merge pull request #272 from localnerve/snyk-upgrade-4b436284bb2299116cca90ec72940788
dfa8192 fix: upgrade html-snapshots from 1.8.0 to 1.8.1
1418f05 Merge pull request #271 from localnerve/snyk-upgrade-5c878cb1907230d146740fc2229ee045
0dbf89c fix: upgrade html-snapshots from 1.8.0 to 1.8.1
83ef7cd Merge pull request #269 from localnerve/snyk-upgrade-adb1706e472b9ef93e85c820d1eebb45
450e9d4 Merge pull request #261 from localnerve/dependabot/npm_and_yarn/examples/debug-phantomjs/cacheable-request-10.2.7
d2c8ff5 Merge pull request #260 from localnerve/dependabot/npm_and_yarn/examples/process-limit/cacheable-request-10.2.7
324bbef Merge pull request #259 from localnerve/dependabot/npm_and_yarn/examples/simple-promise/cacheable-request-10.2.7
dd77419 fix: upgrade html-snapshots from 1.8.0 to 1.8.1
72eab6f chore(deps): bump cacheable-request in /examples/debug-phantomjs
1790811 chore(deps): bump cacheable-request in /examples/process-limit
759771e chore(deps): bump cacheable-request in /examples/simple-promise
2e96c94 Merge pull request #262 from localnerve/dependabot/npm_and_yarn/examples/sitemap-index/cacheable-request-10.2.7
c616aea Merge pull request #263 from localnerve/dependabot/npm_and_yarn/examples/verbose/cacheable-request-10.2.7
b0ea306 Merge pull request #264 from localnerve/dependabot/npm_and_yarn/examples/html5rocks/cacheable-request-10.2.7
a803df7 Merge pull request #265 from localnerve/dependabot/npm_and_yarn/examples/custom/cacheable-request-10.2.7

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Coverage: 99.242%. Remained the same when pulling 99ab55bc9ee26d7266ba4d1385650721c285ab3c on snyk-upgrade-459f0ca1f116c877d44f90edeaf191d9 into c0c1c9288e2c952ab87f58300e8445d8a4cce458 on master.


[cordova][XWALK-4139] Update cases for cordova cli integration

Impacted tests(approved): new 0, update 12, delete 0
Unit test platform: [Android]
Unit test result summary: pass 12, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-4139

LGTM


Jackson Json newline issue

From<EMAIL_ADDRESS>on November 02, 2013 01:49:02
What steps will reproduce the problem? 1. Using Jackson 2 generate a simple json (which gives back pretty print json as return).
2. Example : Ping health response is : "{\n   "status" : "Success",\n   "pong" : 1\n}" - which is valid JSON but
looks like rest-assured doesn't like it?
3. I get this error : Caused by: groovy.json.JsonException: A JSON payload should start with an openning curly brace '{' or an openning square bracket '['.
Instead, '"{
"status" : "Success",
"pong" : 1
}"' was found on line: 1, column: 1
when I try to assert : assertEquals("Success", from(response).getString("status")); What version of the product are you using? On what operating system? Rest Assured version 1.8.1, Jackson 2.2.3 running on Mac. Please provide any additional information below. using Spring rest template to make http calls - not sure if it matters.
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=261

From<EMAIL_ADDRESS>on November 10, 2013 23:27:37
It seems like your JSON string is not valid JSON since it starts with a ". If " is removed (and \n) then I think it should work.
Status: Invalid


[ai-form-recognizer] Prepare release 4.0.0-beta.4

This is somewhat of a laundry-list PR:

Update the CHANGELOG to include a few missing entries and specify the release date (today).
Updated the README and added several deprecation warnings with linkable anchors.
Updated the migration guide.
Removed table caption/footnotes from the API, as they are not produced by the service.
Fixed a broken sample.
Added a simple test for the presence of paragraphs.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-form-recognizer

/azp run js - ai-form-recognizer - tests

/azp run js - ai-form-recognizer - tests


Virtual keyboard doesn't consistently work in the Nightly build

In the nightly docker image (ls6uniwue/ocr4all:nightly) the virtual keyboard doesn't paste the selected character when "Text View" is active.
The virtual keyboard works as expected when "Page View" is active.

Bug fixed on Nov 14, 2019


[JavaFX] Background Layer with image no longer working (elevator demo)

The JavaFX background layer no longer shows any pictures.
The JS version works as expected.
The resource is loaded successfully. Maybe related to the alignmentPoint?

fixed upstream in b4a875479f480d5efbf83b8fe7396a882317ec5c


Sidebar layout

I use Makeroid layout as sidebar feature its really usefull for proffesional desing.There is too many apps use sidebar on the market and app inventor dont have built in sidebar option for this.Is it posible add using layouts as sidebar? I know its already integrated to Makeroid but its really usefull for all.
This option can work like this
1.We can select layout for sidebar.
We can use layouts in the screen or
2.There is a Special panel for desing layouts as sidebar
I think 2. Option make it easy for users.
And if There is a option set sidebar for all or selected screens its will be good

I think what you mean would be the Android Navigation Drawer which is currently available in Kodular, Thunkable X, Android Studio, Sketchware, etc.

Maybe this could help ?
https://community.thunkable.com/t/free-extension-sidebar-navigation-update-12-04-2018/5376

Nope its not work correctly  and we cant use own desings

You can use Android Navigation Drawer for developing a sidebar. As I understood from above, you can use the Android Navigation Drawer on multiple screens and customize its design on each screen. It is an easy and good option for this one. much professional use this for the sidebar.

Yes but i vote for layout option because it is more  customizable.We can desing every think.Same as layout


Validation Error

2019-03-06T10:21:38.1870541Z @{code=ValidationError; target=representation; message=Parsing error(s): JSON is valid against more than one schema from 'oneOf'. No valid schemas. Path 'paths./user.post.parameters[0]', line 1, position 879.}
2019-03-06T10:21:38.4594175Z ##[error]The remote server returned an error: (400) Bad Request.
Getting this error for the below swagger json
"/user": {   "post": {     "description": "description",     "tags": [       "moveforward"     ],     "produces": [       "application/json"     ],     "parameters": [       {         "in": "body",         "schema": {           "type": "array",           "$ref": "#/definitions/Request"         },         "description": "Process Description"       }     ],     "responses": {       "200": {         "description": "users response",         "schema": {           "type": "object",           "$ref": "#/definitions/Response"         }       }     }   } }
Definitions I have
"Request": { "type": "array", "items": { "type": "string" } }, "Response": { "type": "object" }
After repeated tries sometimes it works without any change.
Any help would be appreciated.

I have the same issue when using the "Create/Update API" task.
Our swagger.json contains the following section:
    "securityDefinitions": {
        "oauth2": {
            "flow": "implicit",
            "authorizationUrl": "...",
            "tokenUrl": "...",
            "scopes": {
                 ...
            },
            "type": "oauth2"
        }
    },
    "security": [
        {
            "oauth2": [
                ""
            ]
        }
    ]

Receiving the following error:
2019-03-29T07:59:21.6744381Z @{code=ValidationError; target=representation; message=Parsing error(s): JSON is valid against more than one schema from 'oneOf'. No valid schemas. Path 'securityDefinitions.oauth2', line 1, position 1371.}

Any idea what could cause this issue?

Hello,
The 400 error message is returned by the Azure Management API itself. The extension is not responsible for this. The extension simply forwards your JSON or YAML declaration to the Azure API, just make sure it is correct. Pay attention not to have weird chars in your document, certainly when working with the artifact and the code way of bringing your specs.


Implement File Upload Feature

relates to #57
Description:
-Develop the functionality to enable users to upload their resume and relevant documents.

file upload for resume has been done and details are saved successfully


Corsair Lightning Protocol - LS100 Repeat

As I have shown in the attached picture, I created an LS100 controller with Arduino Leonardo. With this controller, 135 pixel LEDs are currently on. You said before with this code that a maximum of 400 pixel leds can be lit. In the code prepared for the LS100
CRGB ledsChannel1[135];
CRGB ledsChannel2[54];
values are available. How do I set the code so that I can light a maximum of 400 pixel leds?
Thank you for your help.

Try this:
#include <CorsairLightingProtocol.h>
#include <FastLED.h>

// Hint: The channels of the LS100 are swapped in iCUE, so the first channel in iCUE is here channel 2
#define DATA_PIN_CHANNEL_1 2

#define BUTTON_PIN 4

// Hint: The ATmega32U4 does not have enough memory for 135 leds on both channels
CRGB ledsChannel1[400];

CorsairLightingFirmwareStorageEEPROM firmwareStorage;
CorsairLightingFirmware firmware(CORSAIR_SMART_LIGHTING_CONTROLLER, &firmwareStorage);
FastLEDControllerStorageEEPROM storage;
FastLEDController ledController(&storage);
CorsairLightingProtocolController cLP(&ledController, &firmware);
CorsairLightingProtocolHID cHID(&cLP);

void setup() {
	FastLED.addLeds<WS2812B, DATA_PIN_CHANNEL_1, GRB>(ledsChannel1, 400);
	ledController.addLEDs(0, ledsChannel1, 135);
        ledController.onUpdateHook(0, []() {
		CLP::scale(&ledController, 0, 400);
	});
}

void loop() {
	cHID.update();

	if (ledController.updateLEDs()) {
		FastLED.show();
	}
}


Update message for editor.gotoLocation.multiple

See screen shot. The message is now out dated as we allow to cycle through other results pressing F12

This option is not working

vscode info:
Version: 1.40.0
Commit: 86405ea23e3937316009fc27c9361deee66ffbf5
Date: 2019-11-06T17:09:34.601Z
Electron: 6.1.2
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 18.7.0


SSL Configuration Pending/Stuck

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d30mcnomjmbxgg
Region
us-east-2
Step
SSL Configuration
Status
Pending
Issue or question
It has been over 30 hours, and under "Domain Management" the process is still on the SSL Configuration step. I've followed the steps and the name servers do match up in Route 53 (see attached images). Also, I did add the DNS records before transferring over the domain. I've checked if the DNS is still resolving to the former host (SquareSpace), and there's one location, in Australia, that's still resolving the domain.
Does it need to stop resolving to the previous host in order for the SSL Configuration to continue?
Is the fact that DNS records being added before the domain was transferred into Route 53 the reason it's taking longer than usual?

Error message
No response
Additional information
No response

Issue was resolved by upgrading support plan and having a chat session with a technician.


Robot: Report mesh loading issues

This PR implements https://github.com/ros-visualization/rviz/pull/1597#issuecomment-802670884 and fixes #1589.
@simonschmeisser, please have a look. I decided to summarize all mesh loading issues as part of the URDF field.
As you noticed in #1597, TF updates to the link status will overwrite previous errors and introducing a new status field, I didn't like so much.

I haven't found the time to test it yet unfortunately. Generally I don't mind that much about how we present the errors. My version might be a bit simpler to parse for further use in our URDF editor but this one can also be regexed.
I wonder if that variadic function could be avoided by switching to QString right from the beginning, can open a PR on Monday evening

Which variadic function you are referring to?

Thanks for merging this! I was too busy with other stuff before my holidays unfortunately to open yet another PR


Example doesn't work:  Expected a value of type 'List?', but got one of type 'List<Animal?>'

Get this error when trying to run the example
======== Exception caught by gesture ===============================================================
The following TypeErrorImpl was thrown while handling a gesture:
Expected a value of type 'List?', but got one of type 'List<Animal?>'
What's going wrong?
MultiSelectDialogField(
items: _items,
title: Text("Main Categories"),
selectedColor: Colors.pink,
decoration: BoxDecoration(
color: Colors.pink.withOpacity(0.1),
borderRadius: BorderRadius.all(Radius.circular(40)),
border: Border.all(
color: Colors.pink,
width: 2,
),
),
buttonIcon: Icon(
Icons.category,
color: Colors.pink,
),
buttonText: Text(
"Main Categories - high level categories to list the card in",
style: TextStyle(
//color: Colors.blue[800],
//fontSize: 16,
),
),
onConfirm: (results) {
var selected = results.cast();
}),

Could you please try with
final _items = _animals
.map((animal) => MultiSelectItem<Animal?>(animal, animal.name))
.toList();

That worked!

That worked!!
On Tue, 15 Jun 2021 at 17:29, salamaabid @.***> wrote:

Could you please try with
final _items = _animals
.map((animal) => MultiSelectItem<Animal?>(animal, animal.name))
.toList();
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CHB61/multi_select_flutter/issues/46#issuecomment-861648970,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6N5W232FBJ3BATBMALHY3TS55WBANCNFSM46TAXQ3Q
.


testing: a few nits

Fixes https://github.com/AdamKorcz/go-118-fuzz-build/issues/14 , also panics if the input argument to a fuzz-function is not supported.

Argh, I just pushed a change :)

Sorry


Redirect blog to the new location instead of Medium

Blogs have been migrated to https://blog.kubeflow.org/.

/cc @Bobgy

👍
/lgtm

/assign @Bobgy

/approve
thanks for the fix!


TSV exporter broken

TSV exports are broken on Category Search results page.

Clicking "Download as .TSV" gives following error:


[Cubrid] Procedure/function DDL not available

Description
STR
Create function
CREATE FUNCTION hello() RETURN STRING
AS LANGUAGE JAVA
NAME 'SpCubrid.HelloCubrid() return java.lang.String';

Open function  DDL

DBeaver Version
DBeaver 24.2.0
Operating System
windows 10
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

@longhaseng52 @rathana-pvs Please take a look

Currently, Cubrid doesn’t support Procedure/Function DDL. But this feature will be available in the future.


Sorting

Problem:
If you didnt knew, click on the column name / head to sort different, it's not properly on first load at the moment
TODO:

Fix default sorting
Update UX

Fixed in #13


Mapped Worked and now it doesnt load. after i created a new PTC Account and got a new API, but now the map doesnt load the screen stays white

Mapped Worked after i created a new PTC Account and got a new API, but now the map doesnt load the screen stays white.

Delete the Pokemon folder form you AppData folder and try again. Should work. Did for me.

@mastahpeo Do you happen to know how to locate this folder on windows 10? I cant seem to locate that folder

C:\Users\USERNAME\AppData\Roaming
If you can't see AppData you need to go to Folder Options and turn on Show Hidden Files/Folders.

@mastahpeo thanks, that took care of it

Yeah that fixed worked, thanks a lot

bah, did that fix and apparently I wrote down the wrong password for my burner account. I backed up my previous AppData folder, does anyone know which file in there has the account info stored? is it just the cookie file?


Multiplicity Property for page parts is not explained

Please add some explanation of this mysterious property that I just found :-)

Specify the multiplicity of the part.

Value
Description

ZeroOrOne
Zero or one entity

Many
Any number of entities.

What does this mean? What does either value control? What are some concrete scenarios in which either value is appropriate, and what errors can they result in if an unexpected situation occurs (e.g. more records in filter than specified) - etc.
Thanks!

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

ID: 494f274d-efaa-8bdb-5235-918b4031e74a
Version Independent ID: 7d8f2d48-4724-ee24-8508-570aa2f0e560
Content: Multiplicity Property - Business Central
Content Source: dev-itpro/developer/properties/devenv-multiplicity-property.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

This page
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-develop-custom-api
says:
Tip

Parts are defined as 1-N relationship by default. You can, however, define it to be as 1-0, 1-1 relationship. In order to achieve that add the Multiplicity=ZeroOrOne; property in your part as shown below:

part(carModels; "API Car Model")
  {
      Multiplicity = ZeroOrOne;
      EntityName = 'carModel';
      SubPageLink = "Brand Id" = Field(SystemId);
  }

This will change the NavigationalProperty in the metadata from a Collection to an object as shown below:

<NavigationProperty Name="carModel" Type="Microsoft.NAV.carModel" ContainsTarget="true">
          <ReferentialConstraint Property="id" ReferencedProperty="brandId"/>
      </NavigationProperty>

So perhaps this is only for API pages? But the main property page should explain this either way, at least with a link to another page such as the above.

Thanks for the feedback. Yes, we should be more explicit also about this being for API page parts. We'll update the docs with the next doc refresh.


glTF2 updates: attenuation was changed

Godot version:
https://github.com/godotengine/godot/commit/a390e1e8a2efd1de810e0c1f2e0929ba2918e8a9
OS/device including version:
It's graphics, but this was in result to a render engine change. It applies to all.
Issue description:
The engine formulas don't match with the glTF2 formulas. Attenuation was changed in https://github.com/godotengine/godot/pull/44941.
https://github.com/godotengine/godot/blob/6e23ab4fa20f766ef9e1ec64be88675db6c708af/modules/gltf/gltf_document.cpp#L4936
https://github.com/godotengine/godot/blob/6e23ab4fa20f766ef9e1ec64be88675db6c708af/modules/gltf/gltf_document.cpp#L5027
See original test cases from gltf assets in #39450
Steps to reproduce:
Load the test project in #39450.
Minimal reproduction project:
I don't have a project ready, but see #39450. I think it has a gltf2 asset.

Fixed by https://github.com/godotengine/godot/issues/44945.

Fixed by #44945.

You referenced this issue and not a PR.

https://github.com/godotengine/godot/pull/53109 Mistake.


DocFX unable to "compile" Oqtane client

Trying to compile the Octane 5.0 documents. Octane 5.0 compiles as a project just fine as well as the Oqtane Docs Builder solution once everything is updated to use .Net 8, but running the post build event dotnet docfx fails with the following error:
...\oqtane.framework\Oqtane.Client\Modules\ModuleBase.cs(40,16): error CS0246: The type or namespace name 'ModuleInstance' could not be found (are you missing a using directive or an assembly reference?)
Get same error running dotnet docfx in a command prompt window.

I'll check in 2-3 days.
Could be .net 8 incompatibility, could also be that you are on an incomplete commit...

@iJungleboy Thanks.

@iJungleboy, I'm just following up on this issue to see if you could reproduce. However, I know how hectic the holiday season can be. In the meantime, I've obtained used copies of the Professional DNN7 and Professional DotNetNuke Module Programming books through Amazon to act as "Style Guides". Using them as a reference, I've been using GitBook to start writing custom module documentation in Markdown with the help of ChatGPT4 and Claude2.

@GregStevenson I will take a look into this as well to see if I can help at all with the build issue shortly.  I want to be sure all the different ways I use or am going to use Oqtane are covered well so I can use them as a go to resource.  I am hoping some PRs produced just now help get the ball rolling for me here as a contributor.  I want to see what information from the Oqtane Blog can and should be included here in some form so we have a base for the current Oqtane Version 5 as a new foundation.  It would be nice to be sure every aspect of the framework is covered.  This should help uncover any issues we can attack testing all different scenarios for documenting Oqtane.
@iJungleboy I will throw down what I can when I can and feel I have something worthy for your time to review, I appreciate your help here.  I look forward to a blog/wiki/docfx solutions for all who wish to contribute to which ever documentation projects benefit them the most.  A piece here and another there eventually the puzzle gets filled.
I would like to review ways to keep things from being too redundant like in the installations doc page as it will get pretty long eventually.  However not a big deal I tried at the top to put a generic version then release version and why use a release version.  This all can be improved upon as I was just providing some insight on a potential solution with the PR #14  provided.

@thabaum Thanks for looking into this and your other contributions to Oqtane.

@GregStevenson I was able to make some progress I believe on the compilation issue for me being a first time ever trying and getting to know "docfx".
I updated the project to use "net8.0" in the project files.
That led to that same error still however I resolved some warnings to put in as some additional issues with docs.
So I found out it was relating to the project trying to run the command "dotnet docfx".
I then proceeded to open the terminal and run in manually which then told me a bit more information that I needed to run "dotnet restore tools" or installing missing docfx to get it to compile I believe.
I will create a PR maybe you can test see if it resolves.
Here is the updated project file ensure you have .NET 8 installed.
<Project Sdk="Microsoft.NET.Sdk">
	<PropertyGroup>
		<OutputType>Library</OutputType>
		<TargetFramework>net8.0</TargetFramework>
		<RestoreOnBuild>true</RestoreOnBuild>
	</PropertyGroup>
	<Target Name="PreBuild" BeforeTargets="PreBuildEvent">
		<Exec Command="dotnet tool list -g | findstr docfx || dotnet tool install --global docfx" />
	</Target>
	<Target Name="PostBuild" AfterTargets="PostBuildEvent">
		<Exec Command="dotnet docfx" />
	</Target>
</Project>

I have tested this solution out and has worked with success 2 out of 2 times!

After further investigation of my issue I was able to unblock the file I got blocked somehow along the way.
I used the following command in terminal while navigated to the .config folder
To unblock the file I ran this:
Unblock-File -Path .\dotnet-tools.json
Next I ran the following command to restore the DocFX package:
dotnet tool restore

After running those with the latest .NET 8 updated files I was able to restore docfx:

And ran a successful build.

I hope this helps, cheers!

@thabaum Thanks for looking into this. I've been under the weather for the past few days, but will try out your suggestions today. I tried a lot of the items you mentioned before posting the issue, but not Unblock-File. I'll let you know if it works for me as well.

@GregStevenson I hope you are feeling better and are able to get Oqtane Docs ready to roll!  I was able to host this on IIS as well without an issue.  I am having issue with a Oqtane Framework instance on IIS however I need to iron out in my IDE still.  I put a few moments a day towards each, maybe today I can get that done up as well and start feeling out some fresh ideas.
It was the one thing that brought that error, I am not sure if we need to upload a new file that is not locked down with permissions from whomever uploaded it last?  I am hoping we can just upload a new file, however I copy that repo over after it is unblocked, and it is blocked again.  So I think it is a VS environment issue at least on my end.  Maybe if you open your VS instance up with Run As Administrator it works it out.
My repo also adds installing DocFX if it does not exist which was another issue I had.
Issue #15 should handle all the updates needed to files to run .NET 8 and latest version of DocFX.

@thabaum I tried again without success. As I eluded to earlier, I had already done the things you did except the unlock which is not a problem for me. Oqtane.docs and the framework compile just fine for me. It is the post build event where I have issues and looking back, I did not post my exact errors. I'm getting this doing a dotnet docfx from command line.

It's very weird that ModuleInstance cannot be found since everything compiles just fine.

I am trying to understand why you have oqtane framework building with it?
Are you trying to create a site that runs the DocFX site?  You may need to modify Oqtane to do this?
I just built the Oqtane Docs Builder solution and ran it as deploy I think or release to build the /docs folder outside of the project, used IIS to host that folder.

@GregStevenson I would not mind getting some feedback on how this PR repo works for you.  It updates everything to run the latest versions of .NET  and DocFX along with related doc updates.
My workflow is this:
Download latest source for latest Oqtane release and unzip it to a folder along with this repo placed in the same folder oqtane.docs and oqtane.framework.

If I am understanding things more clearly now as I am green here as well... When you build DocFX it will scan documentation comments created in the code files located in the oqtane.framework folder and  thencompile them into the oqtane.docs/docs folder which it's entire contents is what you are able to host as an IIS website.
Maybe I am doing this wrong, I will need to review I am not missing any api docs...  I am trying to troubleshoot why I cannot run Oqtane itself on IIS using Windows 11 at the same time as helping support issues with docs :).  I can run docs, but not Oqtane which is odd.  I may test out a few alternatives so correct me anywhere I can use it.
I hope this helps, cheers!

AFAIK, I just cloned the Oqtane.docs repository in the same folder that contains Oqtane.framework since that is how the bulk of the Oqtane repos seem to work. My guess is that it is looking at the framework for code comments to build API docs. I did nothing special other than updating target framework to .Net8 and similar minor changes. I'm in the middle of something now, but I'll try to run your repo by tomorrow noon PST.

When you test make sure the file structure looks similar to:

oqtane.docs\Oqtane.Docs
oqtane.framework\Oqtane.Server
Both of these should be found in the root folder used to put the two projects and named/renamed exactly the same for example.
The docs site will compile in oqtane.docs\docs folder
I downloaded the zip files and made sure they did this.
Then I ran the solution inside the oqtane.docs folder that contains the Oqtane.Docs project.
You may have all the solutions/projects in the same folder which would not be correct for the way I was able to compile the docs I do not believe.
I have a blank page for API pretty much on release some reason, not sure why.  All the API files are there but the tree is missing to navigate them:
current build:

docs.oqtane.org:

maybe i am doing something wrong or something is missing or not enabled to allow this to work i will look closer at this later.

@thabaum as I mentioned in the pull request #15 solves all the .net 8 problems, but the menu in the API is missing (just like you noticed as well).
I'll try 2-3 things, but we may have to switch to another theme, like I did on https://docs.2sxc.org

I think I have it figured out...
various breaking docfx changes...

just fixed everything and pushed the latest build.
should be on https://docs.oqtane.org in ca. 3 mins

@iJungleboy I just cloned and compiled the oqtane.docs repository. After a dotnet tool restore,  everything compiled as expected but when opening the output, file:////oqtane.docs/docs/articles/documentation/index.html locally in browser, I still experience, not menu, theming, icons, etc as was described earlier. You last comment mentioned this should fix everything. What am I missing?

@GregStevenson I actually assume that you started with the wrong instructions, resulting in this.
I will make sure there are only one set of instructions and enhance them to explain both Visual Studio and VS Code use.
#35 #36
Should be out in ca. 1h

Just push very updated docs.
My guess is you didn't have the oqtane.framework in the right place, so the compiler couldn't find it - and not generate the API docs. Check out the updated instructions: https://docs.oqtane.org/guides/docs/index.html and give me feedback ;)

No that is not it. oqtane.framework and oqtane.docs are siblings. Everything compiled and the api files are present as shown. In fact, viewing one it seems to find the styles, etc.

Maybe there is some issue launching the index.html page locally rather than using a webserver. AFAICT all the files were generated and in their right relative places in the docs folder, but the index page has no styling and no menus on top and left side.

I did a compare of the view source of what was generated and what is at octane.org. There are a lot of differences. Including the fact that my locally rendered version references the fact that it is 5.01. Indicating it would be current.

@GregStevenson To me it looks as if you're browsing the html files from your local disk.
You need to run it through a web server, otherwise most things don't look right.
It's explained in the docs, incl. how to set it up in VS Code with a live web server, or in IIS.
https://docs.oqtane.org/guides/docs/build.html#build-and-test-locally-using-vs-code
https://docs.oqtane.org/guides/docs/build.html#build-and-test-locally-using-visual-studio-2022
https://docs.oqtane.org/guides/docs/run-in-iis.html

Ok, I'm good now. Used GoLive in VSCode. Somehow I missed this:
The generated docs are static pages and contain various JavaScripts to provide menu and search functionality. These do not work, if you just open the index.html in a browser. So you need to run it in a local IIS to fully test it.


Updating deprecation notice

The agent that will be used is the Azure Monitor Agent. Updating for accuracy.

Learn Build status updates of commit 47330f3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/virtual-desktop/troubleshoot-insights.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@Heidilohr

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#sign-off