Contacts page for website

This issue tracks the drafting, review and approval of the Contacts page for the Catalyst website.
This issue is complete when: the copy of this page is approved and can be moved to GitHub ready for publishing on the website (at that point we need to move the copy to the "code" section of this repo on GitHub).
Google doc for drafting copy:
https://docs.google.com/document/d/1Q0w-Sff2i_kC_Q6d0C238oUKHmbGc26Xq9Jy8Gp9lgE/edit?usp=sharing
Roles
Drafting content: @katiepratt and @Tajuddeen1
Initial review: @LouWoodley
Coordination / project management: @Tajuddeen1
Review: corresponding project sub-teams
Handoff - @katiepratt hands off to @jnywong to prep for website publishing

[x] Draft initial page content (Katie)
[x]  Add missing information (Taj)
[x]  Initial review (Lou)
[x]  Send to sub-teams for review (Taj) - N/A
[x]  Address edits / suggestions (Katie)
[x]  Jenny to check re 2i2c approval (Jenny)
[x]  Final signoff (Katie)
[x]  Handoff to Jenny (Katie)

@katiepratt I don't have anything to add to the contacts page. However, I would like us to agree on the contact channel. Through the documents I have seen at least 2 contact options
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I've generated a website preview at #22.
Please let me know your comments and suggestions 🙂

This one looks good to me!

This issue depends on #12 and #13.


[2.8][BUG] Node Driver Whitelist Domains

Rancher Server Setup

Rancher version: v2.8.0
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): docker

Information about the Cluster
The issue resides within the Rancher node driver's logic. Despite configuring an effective whitelist, I encounter difficulties removing it. Rancher's backend aims to implement deletion logic, but evidently, there's a flaw in this logic.
https://github.com/rancher/rancher/blob/07b114dd21372f69efdec2d5c5222b696af9eaa0/pkg/controllers/managementapi/whitelistproxy/nodedriver/nodedriver.go#L21
Here's the relevant code section that is intended for deletion. However, its execution triggers the removal of a whitelist not present in Spec.WhitelistDomains, rendering this code segment redundant.

workaround
A restart of the Rancher pod resolves the issue by automatically deleting the whitelist.

Thank you for your response. The whitelist can be deleted but remains ineffective. We've developed a node driver for Rancher that utilizes an external JavaScript file. Here are the repro steps:

Add a node driver and link an external JS file: https://wangsiye-dev.oss-cn-beijing.aliyuncs.com/cce/dist/component.js
Add wangsiye-dev.oss-cn-beijing.aliyuncs.com to the whitelist.
Access the UI, successfully fetching component.js.
Remove wangsiye-dev.oss-cn-beijing.aliyuncs.com from the whitelist.
Despite removing the entry, the UI can still access component.js.

For a simpler validation:

Set up an Nginx server serving a static resource, placing a JS file.
Add the whitelist entry for 'xxxx'.
Verify access via http://rancher/meta/proxy/http:/xxx.xxx.xx/xxx.js.
Remove the whitelist entry for 'xxxx'.
Verify access again via http://rancher/meta/proxy/http:/xxx.xxx.xx/xxx.js.

Normally, removing an entry from the whitelist should prevent access to the JavaScript file. After reviewing the backend code, it seems the whitelist removal logic isn't functioning as expected. The 'remove' action does not take effect as anticipated. @Josh-Diamond


Add TickTick.app v1.0.1

Thank you for the contribution. It needed some fixes, so they were made in a9c843b299e3d6ce26fc52d477323c4356245a3d. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


NPE when trying to change folder settings of outbox folder

Describe the bug
I tried to change my folder settings for outbox folder but app crashes.
To Reproduce
Steps to reproduce the behavior:

Tab on sandwich icon to open side menu
Tab on 'Manage folders'
Tab on 'Outbox' (2nd list item)
App crashes

Expected behavior
App should show folder settings
Environment (please complete the following information):

K-9 Mail version: actual main (1a68adc8)
Android version: 10

Logs
FATAL EXCEPTION: main
Process: com.fsck.k9.debug, PID: 9972
java.lang.NullPointerException: cursor.getString(7) must not be null
at com.fsck.k9.storage.messages.CursorFolderAccessor.getSyncClass(RetrieveFolderOperations.kt:151)
at com.fsck.k9.mailstore.FolderRepository$getFolderDetails$1.map(FolderRepository.kt:62)
at com.fsck.k9.mailstore.FolderRepository$getFolderDetails$1.map(FolderRepository.kt:8)
at com.fsck.k9.storage.messages.RetrieveFolderOperations$getFolder$1.doDbWork(RetrieveFolderOperations.kt:44)
at com.fsck.k9.mailstore.LockableDatabase.execute(LockableDatabase.java:273)
at com.fsck.k9.storage.messages.RetrieveFolderOperations.getFolder(RetrieveFolderOperations.kt:32)
at com.fsck.k9.storage.messages.RetrieveFolderOperations.getFolder(RetrieveFolderOperations.kt:16)
at com.fsck.k9.storage.messages.K9MessageStore.getFolder(K9MessageStore.kt:94)
at com.fsck.k9.mailstore.FolderRepository.getFolderDetails(FolderRepository.kt:52)
at com.fsck.k9.ui.managefolders.FolderSettingsViewModel$loadFolderDetails$2.invokeSuspend(FolderSettingsViewModel.kt:74)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:738)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)

I'll create pull request with a fix

The Outbox is a folder that is only available locally. We should probably hide all "folder class" settings except for "folder display class" for such folders.


encryption: support encryption on top of custom reader and writer

Signed-off-by: tabokie<EMAIL_ADDRESS>What problem does this PR solve?
Problem Summary:
Unable to use existing encryption facility in raft engine.
What is changed and how it works?
What's Changed:

Allow creating encrypted file on top of arbitrary writable/readable objects
Support random read for encryption reader
Reduce buffer allocation of encryption reader/writer
Fix EncrypterWriter bugs when using Plaintext encryption method or underlying writer reports EOF

Related changes
Check List
Tests

Unit test

Release note
None

@yiwu-arbug I'm not familiar with encryption algorithm to know the proper placement for things like Iv::seek, AES_BLOCK_SIZE, and where to reject settings that don't support random read. PTAL~

/merge


Java duplicate declaration

I'm getting a Duplicate declaration error.
 ==> default: Debug: hiera(): Looking up jenkins::libdir in YAML backend
==> default: Debug: hiera(): Looking for data source local
==> default: Debug: hiera(): Looking for data source wyndow.dev
==> default: Debug: hiera(): Looking for data source common
==> default: Error: Duplicate declaration: Class[Java] is already declared; cannot redeclare at /tmp/librarian-puppet/modules/jenkins/manifests/init.pp:136 on node wyndow.dev
==> default: Error: Duplicate declaration: Class[Java] is already declared; cannot redeclare at /tmp/librarian-puppet/modules/jenkins/manifests/init.pp:136 on node wyndow.dev

I see that issue 224 addresses this:
https://github.com/jenkinsci/puppet-jenkins/issues/224


app py fix

:star: PR built and available in a preview environment atlas-cicd-python-http-pr-1 here

:star: PR built and available in a preview environment atlas-cicd-python-http-pr-1 here

:star: PR built and available in a preview environment atlas-cicd-python-http-pr-1 here


Review all is:issue is:open label:SignalR -label:blazor

Review all is:issue is:open label:SignalR -label:blazor

closing as this looks like an automated "thing" done in May. :)


Unexpected behaviour of template()

Hi guys, thank you first of all for your awsome theme framework!
In my single.php controller I am doing this:
use function AppTheme\template;
template( [ 'single', get_query_var( 'post_type' ) ] );

Expected behaviour
When I request /books/ for a custom post type books I expect it to first look for single-books.tpl.php, then for single.tpl.php, then throw an error.
Actual behaviour
It looks for single-books.tpl.php, but then it looks for single-.tpl.php. The dash shouldn't be in there.
See here:
Fatal error: Uncaught Tonik\Gin\Foundation\Exception\FileNotFoundException: Template file [resources/templates/single-.tpl.php] cannot be located. in /app/themefiles/vendor/tonik/gin/src/Gin/Template/Template.php on line 54```

This happens when you pass an empty string as a template name. We have to additionally check for an empty string.
template(['single', '']);

For now, I would recommend you to create a post type specific controllers or check for an empty string at get_query_var.
Thanks for the greatly exemplary bug report :)

Fixed with tonik/gin:v2.0.3 release

Thank you for this quick response. Applied the update and can confirm that it is fixed!


Allow urlPath override for httpAdapter

This PR allows you to specify a urlPath option when running an async method.
Example:
store.filter('customer', myQueryParams, {urlPath: '/report/specialPath'}).then(function (result) {
  console.log(result)
})

@internalfx Can you please remove the changes to dist/* from your commit?

@jmdobry sure thing

arrggg. I think I've now totally messed up my repo. Git is great until you need to change history. And I'm not that good at git.

How about I just throw your changes into a commit so I can cut a release?

@jmdobry got it.

K, how about rebasing those commits into 1?
1. git rebase -i HEAD~7
2. set the bottom 6 to "s" instead of "pick"
3. save the commit message
4. force push

I only see 1 commit. let me look

nope, refreshed

ok, cool

I also made your changes to the array

Needed to remove the .split too. I took care of it.

@jmdobry I'm also simultaneously migrating a chat server. Perhaps my attention was a little divided.


Determine desired format for Kubernetes provider secrets

What problem are you facing?

Kubernetes cluster connection secrets were previously changed from using the full kubeconfig to only necessary fields for connection in #273. However, #841 expands the use of these secrets to include BYO cluster scenarios, meaning that the secret would be "manually" created. It is not clear whether we want to use the current format given this increased functionality.
How could Crossplane help solve your problem?

Converge on a format that is consistent for all Kubernetes cluster connection secrets. This may not involve any change, but it should be revisited if the BYO cluster scenario proves cumbersome.

I think our connection string should be raw kubeconfig, which is accepted by all k8s clients I can think of, including kubectl, go-client, controller-runtime, helm etc.
This is especially important when users just want to use Crossplane for infrastructure and deploy the apps on their own, either via helm or some other tool.

I am facing an issue where I am able to connect to a GKE provisioned cluster by building rest.Config from connection secrets but I am not able to create a kube config that I can use to install my application using helm. This is a big problem for us because we will have build a kube config in our code to do helm install.

@hasheddan @muvaf It will be helpful if this issue is fixed. We want to deploy apps only using helm. Do you think it can be prioritized for next release?

@vadasambar @aeli0 This is implemented across 3 cloud providers and will be included in the next release!

Thank you so much @muvaf !


[BUG] Annotation schema attribute definitions not filtering when defined on classes

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04.3 LTS
FiftyOne installed from (pip or source): pip
FiftyOne version (run fiftyone --version): v0.15.1
Python version:  Python 3.9.10

Commands to reproduce
When submitting a schema, attributes are not filtered if attributes are only attached to classes.  For example, if a sample of object1 had fields of condition1, condition2, and other_field, the following schema would allow other_field into CVAT annotations:
        "type": "detections",
        "classes": [
            "object0",
            {
                "classes": ["object1"],
                "attributes": {
                    "condition1": {
                        "type": "checkbox",
                        "default": False,
                    },
                    "condition2": {
                        "type": "checkbox",
                        "default": False,
                    },
                },
            }
     ]
}

This schema would block the other_field field from going to annotations:
        "type": "detections",
        "classes": [
            "object0",
            {
                "classes": ["object1"],
                "attributes": {
                    "condition1": {
                        "type": "checkbox",
                        "default": False,
                    },
                    "condition2": {
                        "type": "checkbox",
                        "default": False,
                    },
                },
            }
     ],
     "attributes": {"test": {"type": "checkbox", "default": "False"}}
}

Describe the problem
Schemas need not define global attributes, they can also be defined class by class.
Code to reproduce issue
import fiftyone as fo
import fiftyone.zoo as foz
import tempfile
import numpy as np
from PIL import Image

USERNAME = ...
PASSWORD = ...

# Attributes are only defined by class, `bad` will go through despite not being listed
schema = {
    "test_field": {
        "type": "classification",
        "classes": [
            {
                "classes": ["test"],
                "attributes": {
                    "good": {
                        "type": "checkbox",
                        "default": True,
                    }
                },
            }
        ],
    }
}

# Include an attribute dict to force filtering in _get_attributes
hacky_schema =  {
    "test_field": {
        "type": "classification",
        "classes": [
            {
                "classes": ["test"],
                "attributes": {
                    "good": {
                        "type": "checkbox",
                        "default": True,
                    }
                },
            }
        ],
        'attributes': {
            "another_one": {
                        "type": "checkbox",
                        "default": True,
                    }
        }
    }
}

with tempfile.TemporaryDirectory() as tmpdir:
    # Generate images
    for i in range(3):
        imarray = np.random.rand(100, 100, 3) * 255
        im = Image.fromarray(imarray.astype("uint8")).convert("RGBA")
        im.save(f"{tmpdir}/temp{i}.png")
    # Create a dataset
    dataset = fo.Dataset.from_images_dir(tmpdir)

    # Assign some bogus classification tags with attributes
    for sample in dataset:
        sample["test_field"] = fo.Classification(label="test")
        # Assign a desirable and undesirable attribute to the classification
        sample["test_field"]["good"] = True
        sample["test_field"]["bad"] = ":("
        sample.save()
    dataset.save()

    # Annotate and go to CVAT to see result

    dataset.annotate(
        "test_annotation",
        # Swap schemas here
        label_schema=hacky_schema,
        backend="cvat",
        url="https://cvat.org/",
        username=USERNAME,
        password=PASSWORD,
    )

What areas of FiftyOne does this bug affect?

[ ] App: FiftyOne application issue
[x ] Core: Core fiftyone Python library issue
[ ] Server: Fiftyone server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another
member of your organization be willing to contribute a fix for this bug to the
FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently.
[x ] Yes. I would be willing to contribute a fix for this bug with guidance
from the FiftyOne community.
[ ] No. I cannot contribute a bug fix at this time.

Thanks for bringing up this issue and the reproduction code @geoffrp!
To recap there are three different ways that you can define the attributes being annotated in order or priority:

For a single class in one field in your schema, you can provide explicit attributes that only apply to that one class.

schema = {
    "test_field": {
        "type": "classification",
        "classes": [
            {
                "classes": ["test"],
                "attributes": {
                    "good": {
                        "type": "checkbox",
                        "default": True,
                    }
                },
            }
        ],
    }
}

For all classes of a specific field in your schema, you can provide the attributes directly under the label field to apply to all classes in that field, like your hacky schema.

hacky_schema =  {
    "test_field": {
        "type": "classification",
        "classes": [
            {
                "classes": ["test"],
                "attributes": {
                    "good": {
                        "type": "checkbox",
                        "default": True,
                    }
                },
            }
        ],
        'attributes': {
            "another_one": {
                        "type": "checkbox",
                        "default": True,
                    }
        }
    }
}

For all label fields, you can provide a value to the attributes keyword argument when calling dataset.annotate(). For existing fields, this can either be a boolean to load all or no attributes, or a dictionary/list defining the specific attributes to load. This will then be applied to all label fields annotated in that run.

results = dataset.annotate(anno_key, label_schema=label_schema, attributes=True)

So the easiest way to currently do what you want is to define the class-specific attributes in the way that you have, and then add the attributes=False argument when calling dataset.annotate() to prevent all other attributes from being loaded. This is necessary since by default, attributes=True so all existing attributes will always be loaded no matter what you define within the label schema.
There are a few options going forward:

Set attributes=False by default so that no attributes are loaded unless specified. I vote against this since a common workflow is to just provide a label field directly to dataset.annotate(), and you would expect everything to be loaded in that case.

Update the code so that attributes=True still, but if you provide class- or field-level attributes, then it will internally set attributes=False. There is also a problem with this approach since a supported workflow should be to allow defining NEW attributes class- or field-wise and then still letting attributes=True to automatically load the remaining existing attributes.

schema = {
    "test_field": {
        "type": "classification",
        "classes": [
            {
                "classes": ["test"],
                "attributes": {
                    "new_attr": {
                        "type": "checkbox",
                        "default": True,
                    }
                },
            }
        ],
    }
}

results = dataset.annotate(anno_key, label_schema=schema, attributes=True)
# Annotate all existing attrs and `new_attr`

Default attributes=None which has the properties of 2) where it will load all attributes unless any specific class- or field-level attributes are provided, but still allows the option to explicitly set attributes to True/False. The main downside of this is that it is more complex and likely harder to understand what is happening for new users, but I think this would probably be the best approach.

I'd be interested in hearing your thoughts @geoffrp and @brimoor

Since it is possible to get the behavior that @geoffrp wants by simply providing an additional parameter, I see an argument for option 4:

Don't change any code, just update/clarify the docs to explain the implications of top-level attributes=True being the default behavior.

Side comment: +1 on @ehofesmann's comment that attributes should be included by default in simple syntaxes like:
dataset.annotate(anno_key, label_field="existing_field_with_attributes")

This is consistent with other I/O in the codebase, such as how there's an extra_attrs parameter that defaults to True on many exporters to automatically include everything known about an object unless otherwise requested.

@brimoor @ehofesmann I think that option 4 could work here, and re-reading the documentation, this line slipped by me:
label_schema (None): a dictionary defining the label schema to use. If this argument is provided, it takes precedence over label_field and label_type
The presentation in the documentation pushes label schema as a catch-all method of specifying the whole annotation task, but there are other parameters that gate certain functions.  Maybe for some users, it is useful to have all attributes show up to contextualize the annotation, so having the open attributes=True could be default behavior.  However, in my case I'm strictly prescribing a specific amount of information that will go to CVAT.
I'd be happy to submit a documentation update that might make this clearer for users in my use case.


Is it possible to lazy load dependencies?

For example, if it's used in Vite or Rollup, these deps should not be loaded.
https://github.com/unjs/unplugin/blob/85a7ce6b1c89972ebf047055cbd0798eadd39656/package.json#L31-L35

If the API should be kept as is, it seems that the only way is to use the old-style require() to lazy load modules here (synchronously):
https://github.com/unjs/unplugin/blob/85a7ce6b1c89972ebf047055cbd0798eadd39656/src/define.ts#L11-L22

You can use buildStart hook, or basically every hook you want to ensure the async import been resolved.

Maybe I didn't make it clear, I mean the dependencies of unplugin itself.
For example, how about the refactor as follows? The benefit is to avoid Webpack related modules load if it is not called.
export function createUnplugin (factory) {
  return {
    // ...
    get webpack () {
      const { getWebpackPlugin } = require('./webpack')
      return getWebpackPlugin(factory)
    },
    // ...
  }
}


:sparkles: Tag/TagCategory forms: fix placeholder text

Resolves https://issues.redhat.com/browse/MTA-909

LGTM!


Adding mergeCell() functionality removes all formatting from resulting .xlsx file

I'm sorry to keep bringing up issues but I'm using the most recent version of OpenXLSX from the development-aral branch in MS Visual Studio 2022 on Windows 11 x64 and writing a program that produces four .xlsx files; three of those files work just fine when opening in my default .xlsx file reader, Office 365 Excel. The fourth one produces an error on opening and is the only one which has the "mergeCell" functionality. The error is:
Removed Part: /xl/styles.xml part with XML error. (Styles) Load error. Line 6, column 0. Repaired Records: Cell information from /xl/worksheets/sheet1.xml

I am able to build and open the Demo10.xlsx file without issue, so I compared the underlying <xl/worksheets/sheet1.xml> files and found the following difference. The file my program creates contains the following (lines 4-8 post-formatting):
<sheetViews>
       <sheetView tabSelected="1" workbookViewId="0"/>
           <selection activeCell="B1" sqref="B1:K2" />
       </sheetView>
   </sheetViews>

while Demo1.xlsx : sheet1.xml contains (lines 4-6 post-formatting):
<sheetViews>
       <sheetView tabSelected="1" workbookViewId="0"/>
   </sheetViews>

After removing
           <selection activeCell="B1" sqref="B1:K2" />

from my file's sheet1.xml, modifying that section’s header and footer accordingly, rezipping the xml files into a .zip file, and changing the extension to .xlsx, I am able to open it in Excel without issue. Does anyone know how the problematic line is being added to the sheet1.xml file and what I might be able to do differently to avoid adding it in the first place? Thank you!
Edit: fwiw I’ve tried to emulate the format of Demo10.cpp to the best of my ability, and can attach the code that pertains to writing the excel file if it would help. Thank you!

No need to apologize, but I'll have to ask you to have some patience, my vacation ends on 8 November, but until then I don't want to spend too much time on coding :)
From what I can see, it appears as though you are opening an existing Excel file that has this <selection> tag - because OpenXLSX neither uses that tag, nor does the template for an empty XLSX workbook contain this tag for worksheets.
I will have to read up on what it does and - if it turns out to be redundant information, the easiest fix I can think of is to auto-remove the <selection> tag when encountered (upon modifying a worksheet).
Regarding the use of styles, what I wanted to add to yesterday's talk:
The styles functionality was so much that I provided a relatively "raw" interface to the underlying styles XML, without being able to implement validation functions for every setting. So it is possible to "break" a workbook by setting invalid combinations of styles (the disabled test section of Demo10 creates some invalid settings, thus I disabled it).
A prime example would be to set a style reference for an index for which no such style exists.
So please, when looking at Demo10, disregard the code that I disabled with the testBasics boolean in line 231, I only used that to manually check how the XML looks, without opening the file in Office.

No worries, please enjoy your vacation! I’m in no rush to get this project finished at all

Just a progress update, but I’m trying to use a slightly modified version of the OpenXLSX library to access the “sheet1.xml” file and remove the offending child node as a temporary fix. I’ll let you know how it goes, thank you!

orr... you could unzip the resulting file, remove it from the xml with an editor and re-zip it, I understand you have a working zip solution for that since the previous investigation :)

I'm not sure how, but the node I thought was causing the issue to begin with has disappeared from the sheet1.xml file in the unzipped directory. However, I still get an error on opening the .xlsx file in MS Excel saying that there's an issue with /xl/styles.xml: line 6 column 0 and with cell info from /xl/worksheets/sheet1.xml; I looked through both files and could not find any obvious issues that would cause these errors. Worksheet.xlsx is the original, unmodified spreadsheet my program currently creates.

On a first look, I don't see anything wrong with sheet1.xml (so yea, eat **** Microsoft ;) - I'll investigate when I get home.
For the styles neither, especially not in column 6. I wonder if MS Office has a problem with the empty number formats node.
There are only two things I can suggest you try in the meantime:

(preferably this step first) reduce your document down to the absolute minimum that produces an error - that makes comparing the before/after LibreOffice save easier
create at least one single number format entry to have a non-empty node for that

Beyond that, I can only suspect a problem arising from a mix of the empty workbook template that OpenXLSX has hardcoded in binary and the styles node changes that my XLStyles implementation performs.
It appears I did not sufficiently test this - I was mostly testing with an existing workbook where I removed / modified the styles.xml to have / not have certain nodes and where I verified that my code would be able to deal with it and produce valid XML.
Nevertheless, since you said you can open Demo10.xlsx in Office without errors (and that uses the template) I would not expect an issue there.

Okay, I am "back in action" - did you have a chance to test anything, or are you busy looking for ways to flee the country? :(

bump
I have had another report in #304 and #305 - and I am wondering if maybe the styles.xml error message you get at the beginning is part of the problem - and/or if also in the sheet1.xml, the sequence of XML tags is what makes MS Office complain. Either way, I'll look into the sequence correction and then would ask you if you can test again with a patched version.


close 815: upgrade to go 1.18

This PR upgrades go to version 1.18

We have been running stargaze with go 1.18 without issues so far for a month probalby, we have not introduced any of the new features (generics) to the codebase tho.

Given (whatever this is)
https://app.circleci.com/pipelines/github/CosmWasm/wasmd/2111/workflows/7a47baf1-f484-4803-891d-eac1de26f0a5/jobs/7124
could we consider moving to plain github actions?

The new circleCI image is unfortunately not a drop in replacement. I have opened #872 and will 👀  when I get a chance

Kinda my point about migrating to actions.
Also, if we use actions, the whole repo becomes more easily forkable.

got it :)

@alpe what is the status here? I was happy not to merge before v0.27.0 (there was plenty in there already), but what about merging this now, to include in v0.28.0.
Cosmos-sdk v0.46.0-rc1 us on Go 1.18 and Stargaze's experience is also positive (as they use CosmWasm and the whole stack)


contribute

Hello Alireza
Can I contribute to this project and add content to the book?

in fact, I made this project for other developers can write and share their knowledge.
Please make a new branch for your own and push your new things on there.


Double encoded callback url causes issues

Description:
I've been using the framework successfully for awhile, but recently it stoped working with Imgur with The redirect URI provided is missing or does not match error. The issue doesn't seem to be related to any particular changes in the framework, but rather to Imgur backend updates. However, oauth works with Postman and Paw, which leads to certain conclusions.
Digging deeper the issue seems to be related to double encoded callback url in OAuth2Swift::authorize method. Removing double encoding solves the issue. I believe services take this into account and un-encode the url, which is good and bad, as it makes it easier to work with, but introduces uncertainty.
queryString composed in OAuth2Swift::authorize combines every parameter as is with exception for callback url, which gets encoded. The composed query then gets encoded, which causes url to be double encoded. If query would be composed with every parameter encoded immediately and without re-encoding everything, it would produce a valid query and eliminate the issue.
P. S. I see that logic has changed recently, however, the problem occurs in the following urlQueryEncoded invocation.
OAuth Provider (Twitter, Github, ..):

[x] Imgur

OAuth Version:

[x] Version 2

OS (Please fill the version) :

[x] OSX :

Installation method:

[x] Carthage

Library version:

[x] v1.1.2

Xcode version:

[x] 8.0 (Swift 3.0)

someask ask for encoding PR #325 e56fafadc8a0db781ab51e45eb0522d70058026c
I make it optional using encodeCallbackURL boolean, issue #339
by default there is no encoding of callback url

I know. I've seen those. This is great, but not the case. If it's set to true url gets encoded twice, first here, then here. Frankly, if url is not encoded, it doesn't get encoded the second time. Like #339 describes – slashes ( / ) are being replaced with %252F – when encoded, they should become %2F, but % gets encoded twice, producing %252F.

Happy to PR.

If you do not encode twice, how the server could decode and know to what url query parameters belong to
If we take this query
x=5&callbackurl=http://toto?y=8&z=4
the z is for callbackurl or main url
If we encode the first time using encodeCallbackURL =true
x=5&callbackurl=http%3A%2F%2Ftoto%3Fy%3D8%26z%3D4
then reencode (callback is encoded two times)
and then the server decode - I think server could know that z is for callbackurl
some server expect callbackurl as last query parameters...
It's difficult to make compatible with a lot of providers because of weird choice...
If you are able to provide a code that work for your provider, please PR.
But If we can keep the current two workflow...encodeCallbackURL=true and encodeCallbackURL=false
Maybe have an another dirty boolean, or make an enum with 3 cases (keep the boolean as compatiblity with deprecated and remapped to the enum)
Maybe I miss something
I must code, make unit test with URL to fully understand, try with some providers to validate that all are compatibles

No, you are right. I thought of it in the first place, but lost it out of sight with other stuff in mind. But like you say, if we don't encode it at all, the same thing happens – query parameters might get mixed. I'm mostly bummed out by the fact that it used to work… I'll look into this.

Also tested with my project, everything seems to work! 👌


Timeformat is too fragile

I tried changing the timeformat in ~/.jrnl_config to "" (because I don't care about the dates or times and don't want them cluttering up the output) but now jrnl shows no entries at all. I then tried changing it to simply ">" and then "%d", but each time, no records were displayed. It seems that jrnl expects all the datetime elements to be present in the format string, which severely limits its flexibility.

This might be addressed in the 2.0 branch, but I don't have time to check this morning.  I seem to recall a similar issue being reported before.

This is solved in 2.0 :) Closing this for 1.x


http connection manager: Add dependency manager

WIP

@htuch

@auni53 looks good, I think maybe if you can add support for dealing with dependency type matching that would complete this for the decode path. Thanks.
/wait

/retest


Test task from @levozavr

close #2

[x] Test Github flow
[x] Allow conflicts

Is it a good pull request?
Have I solved the conflicts correctly?

@levozavr
It seems to be OK!


Suggest cleanup

[x] Cleanup CompletionItem[][] mess
[ ] Create a view model
[ ] Use tree.refresh

Related to #1006


save audit logs (gh-301)

[ ] test in two VMs
[ ] check in react client and swagger UI as well
[ ] also check in postman

retest it as issue gh-370 was merged in dev branch.


Change button styling to match Uiuapad's

This replaces the current default-styled buttons to match the ones shown on uiua.org/pad.
Some additional stuff you should know:

buttons now have the glyph itself wrapped in a div so that glyphs such as ⍉ display properly on a background.
The logic for adding buttons has changed slightly. Rather than attaching a mouseup event to #button-holder, buttons are given a click event listener as they are creating that achieves the same purpose (minus the header that previously displayed the glyph's name; it isn't needed anymore).
Current issue is that hovering over a button on the far right of the keypad cuts off the tooltip text, which will likely be a problem. A bodge would be to just give the button holder a right margin of 7em, but this wastes space. I don't recall a CSS feature that allows absolutely-positioned elements like the tooltip to change their position to remain within a container, but it can probably be done somehow, and I'm going to try to figure out how to go about that (hence the draft status).

Hi @thekifake :) Thankyou for working on this.
I am really sorry I didn't see your PR until after I made a heap of changes today.
I'll take a look, I may have already made a few changes that improve the button style.
Regarding the tooltip, I am more inclined to stick with the current hover-status-bar at the top approach which eliminates the issues with tooltips overlapping things. Is there a reason you wanted to change the tooltip style?
I was worried that the existing status bar text might be hidden by some themes? In today's changes I manually set the font color and body background color to help ensure that themes don't make the text invisible.
I think you are right that wrapping the button contents in a div may help to fix the formatting, especially for the trans glyph.

Regarding the tooltip, I am more inclined to stick with the current hover-status-bar at the top approach which eliminates the issues with tooltips overlapping things. Is there a reason you wanted to change the tooltip style?

I changed it primarily to match what appeared on the Uiua website (for consistency, I suppose). In retrospect the original method of using a header might be the better option. Though, it might be worth trying to add the description of the glyph beneath it, sort of similiar to Uiuapad displaying a small info box when you hover over a glyph:

The one thing preventing me from implementing this easily, though, is the fact that these one line summaries of the functions aren't contained in a database, or anything. They're contained as comments in the defs.rs file of the language itself. This works in the context of the website because I assume it's a feature of Rust similar to JSDoc, except you can extract the documentation as a string (I'm not familiar with Rust so I could be wrong). However trying to get this information from the file outside of Rust, without making changes to the defs.rs file itself, would involve something like a regular expression which, while it can be written, will probably be relatively slow and I'm not sure it would mesh well with the pre-existing method you have for getting glyph icons and names. It might be worth a shot regardless.
Despite the fact that you've already made the thematic changes to the buttons, I'm still going to keep this open as a draft in the hopes that I can maybe implement what I've just described. I will say that, in the current version, the keypad does seem a bit zoomed in, and justifying the individual buttons to the center would likely look a lot better. The grid misalignment it will likely introduce could probably be fixed by just making the backgrounds of the buttons transparent until hover, like in my implementation (if that makes any sense).

for consistency

Ah thats fair enough. I guess it is best to make it an easy switch from the web version.
I am not sure if it is implemented yet, but I thought of this article from google about tooltip anchoring; https://developer.chrome.com/blog/whats-new-css-ui-2023/#anchor-positioning
Even so, in VS code the keypad will generally have quite a lot of extra space, especially in tall-skinny aspect ratios. Plenty of room to put the tooltip as a status bar. Maybe in short-wide aspect ratios it could switch to a tooltip strategy?

the defs.rs file

Ah well done I made a few attempts to find it but gave up :)
I certainly could parse the source... but I am reluctant to try since it is unstable. I tried once on the discord to ask if there is a JSON definition that could be queried but no response from the language author yet.
If it is in rust as a macro, perhaps more rust macro wizardry could be use to make an endpoint that can be queried...

does seem a bit zoomed in

Agreed. I struggled with the media queries a bit to fix it... but without knowing the aspect ratio (skinny tall, or wide short, or square) it is hard to optimize the styles for all situations and sizes.
I would welcome any improvements to the current media query mess. Possibly the border-radius and grid/flex gap needs to be decoupled from the font-size to get a more consistent appearance... dunno.

justifying the individual buttons to the center would likely look a lot better. The grid misalignment it will likely introduce could probably be fixed by just making the backgrounds of the buttons transparent until hover, like in my implementation (if that makes any sense).

I think I did end up with everything centered? Anyway I am sure it could be improved further.
I will try just stick to just updating glyphs for a while.
Thanks again and hopefully you get a chance to update your PR :)

Closing this since it's been obsoleted and I don't plan on making future changes to it.


Garbled Message published from OTMql4Zmg

The first 16 bytes of a message published from OTMql4Zmg are sometimes (not always)
garbled, including nulls. The rest of the message is fine. The workaround it to pad the
message with 16 bytes.
I checked with wireshark and it is garbled in the transmission.
I checked in Mt4 and it is going to the dll OK - StringToCharArray is not the culprit.
Pity I can't run metatrader under a debugger...

After modifying OTZmqCmdEA.mq4 as shown in the gist and OTZmqSubscribe.py, it looks like the garbage text comes out of the debug messages
https://gist.github.com/a3bhea/b865edd21ec7055eb81d

Very Interesting. This kind of thing is very possible, and is why in general with
OTMql4Py I have to be rigourous about using sys.exc_clear()
I'll look carefully at the gist information, and even if I can't find the cause,
I may check in your code to help with the diagnostics. I really would like
to get this working, but the problem is hard to diagnose.
I appreciate your help with tracking this down; you've given me something to look for.
PS: if you "correct" my capitalization, it makes it much harder for me to read the diffs...

I put some more diagnostic messages in where you were working, but the results are
not good: see the discussion I've written up at
https://github.com/OpenTrading/OTMql4Zmq/wiki/CompiledDllOTMql4Zmq
The corruption is coming from a Mt4 print statement to stdout after the ZmqSend.
We'll have to wait until someone who knows C can look at it.
The Python version is running fine: I'll start adding a test harness for it:
https://github.com/OpenTrading/OTMql4Zmq/wiki/PythonOTMql4Zmq
Thanks very much for your help on this - it really identified the problem.
If you're adding logs from tthis, and if you are able to, upload the logs
in a zip or rar file: they contain binary data that may get munged by the gist.
And tell ne what OS and version, because the messages are probably being
generated in Unicode, which also might not survive the gisting.

I may have found the problem. Even with the debugging code in there, everything
seems working for Mt4 sending - havent tried publishing to Mt4 yet.
I simply replaced MQL4/Libraries/OTMql4/libzmq.dll with libzmq-v100-mt-4_0_4.dll  from http://miru.hk/archive/ZeroMQ-4.0.4~miru1.0-x86.exe
Do a git pull and see if this helps, and followup here.

I spoke too soon: looks like nothing is working either way anymore.
I'll try recompiling with debugging enabled and see if I can make some progress.

I recompiled the mql4zmq.dll against the libzmq-v100-mt-4_0_4.dll from http://miru.hk/archive/ZeroMQ-4.0.4~miru1.0-x86.exe with MSVC 2010 and put them into src/mql4zmq/pre-compiled.
Make sure you find the old dlls and renamed them to something like .old, then
copy  mql4zmq.dll into you Metatrader MQL4/Libraries/OTMql4 directory and
libzmq.dll into your windows system32 or equivalent directory.
Sending from Mt4 works fine now, but receiving by Mt4 doesn't yet: nothing is received.
But wireshark says that the message is sent by bin/OTZmqPublish.py and
that there is an ACK back, so we are making progress.
Test it out and post back your results here.

Looks like I've been able to solve the problem by dropping back to v2.2.0.
Give me a few days to write it up and test it out, but both directions appear to be working
with it. I couldnt't get the libzmq-v100-mt-4_0_4.dll  solution above to receive.

Try the most recently checked in code. It's only been lightly tested, but seems to be
working both ways. It's based on ZeroMQ v2.2.0.
I'll be testing it more over then next 2 weeks - post your results here when you try it.

This issue I'm closing because the root of the problem was the MSVC compilation of the DLL.
We feel that the recompiled DLL has no issues (other than it's an old ZeroMQ rather than 4.x).
The message has zero length is an unrelated issue, and may be just an overzealous debugging warning from when we were tracking down the DLL issues - I'll look into it.


code and tests for decoding the flag in the blosc header

Coverage increased (+0.01%) to 95.307% when pulling 6a9064d03dfb9f199fa2c763cc453d474974c58e on esc:esc/feature/decode_flags into 9e732807dc6d077158e8eff7f5d2a6af14bb3470 on Blosc:master.

Coverage increased (+0.01%) to 95.307% when pulling 6a9064d03dfb9f199fa2c763cc453d474974c58e on esc:esc/feature/decode_flags into 9e732807dc6d077158e8eff7f5d2a6af14bb3470 on Blosc:master.


FileNotFoundError: [Errno 2] No such file or directory: 'config.yaml'

Hi,
First, I got error of ValueError: [x] Unknown Model: DiffusionNew. After reading your solution i left the model address on the left side of gui, empty and i read and saved config file in exp/diffusion-test directory in gui. when i press start conversion, i see this error: FileNotFoundError: [Errno 2] No such file or directory: 'config.yaml'
I checked, config.yaml is in exp/diffusion-test directory.
Please let me know what to do. Thanks.

Have you selected the pt model correctly? It will automatically load config.yaml in the same directory as the pt file.

Thanks. It resolved the problem but I got another error:
(<class 'sounddevice.PortAudioError'>, Error opening Stream: Illegal combination of I/O devices [PaErrorCode -9993], <traceback object at 0x0000024246CAAA40>)
I tried different output but I got same error. Please help me with this.

Try to use the same MME type device


Plugin not working in phonegap app

Hello,
I want to use this plugin into my phonegap application to scan for nearby bluetooth devices but I have a problem.
I have added the BC.js and bluetoothapi.js to my project but I am still not able to access all the methods and classes of the plugin.
Why is this happening?
Which is the right way to be able to use the plugin?
Thanks

Could you post your log here?

The curious thing is that when adding the line 	<gap:plugin name="org.bcsphere.bluetooth" /> to the config.xml, all the other plugins I use stop working as well.
How do I have to add it?
Log:
Build Date: 2015-06-09 02:22:13 +0000
Buildfile: /project/build.xml
-set-mode-check:
-set-debug-files:
-check-env:
[checkenv] Android SDK Tools Revision 24.3.0
-setup:
[echo] Project Name: MySirenum
[gettype] Project Type: Application
-set-debug-mode:
-debug-obfuscation-check:
-pre-build:
-build-setup:
[getbuildtools] Using latest Build Tools: 20.0.0
[echo] Resolving Build Target for MySirenum...
[gettarget] Project Target:   Android 5.0.1
[gettarget] API level:        21
[echo] ----------
[echo] Creating output directories if needed...
[mkdir] Created dir: /project/bin
[mkdir] Created dir: /project/bin/res
[mkdir] Created dir: /project/bin/rsObj
[mkdir] Created dir: /project/bin/rsLibs
[mkdir] Created dir: /project/gen
[mkdir] Created dir: /project/bin/classes
[mkdir] Created dir: /project/bin/dexedLibs
[echo] ----------
[echo] Resolving Dependencies for MySirenum...
[dependency] Library dependencies:
[dependency]
[dependency] ------------------
[dependency] Ordered libraries:
[dependency]
[dependency] ------------------
[echo] ----------
[echo] Building Libraries with 'debug'...
nodeps:
-set-mode-check:
-set-debug-files:
-check-env:
[checkenv] Android SDK Tools Revision 24.3.0
-setup:
[echo] Project Name: CordovaLib
[gettype] Project Type: Android Library
-set-debug-mode:
-debug-obfuscation-check:
-pre-build:
-build-setup:
[getbuildtools] Using latest Build Tools: 20.0.0
[echo] Resolving Build Target for CordovaLib...
[gettarget] Project Target:   Android 5.0.1
[gettarget] API level:        21
[echo] ----------
[echo] Creating output directories if needed...
[mkdir] Created dir: /project/CordovaLib/res
[mkdir] Created dir: /project/CordovaLib/libs
[mkdir] Created dir: /project/CordovaLib/bin
[mkdir] Created dir: /project/CordovaLib/bin/res
[mkdir] Created dir: /project/CordovaLib/bin/rsObj
[mkdir] Created dir: /project/CordovaLib/bin/rsLibs
[mkdir] Created dir: /project/CordovaLib/gen
[mkdir] Created dir: /project/CordovaLib/bin/classes
[mkdir] Created dir: /project/CordovaLib/bin/dexedLibs
[echo] ----------
[echo] Resolving Dependencies for CordovaLib...
[dependency] Library dependencies:
[dependency] No Libraries
[dependency]
[dependency] ------------------
-code-gen:
[mergemanifest] Merging AndroidManifest files into one.
[mergemanifest] Manifest merger disabled. Using project manifest only.
[echo] Handling aidl files...
[aidl] No AIDL files to compile.
[echo] ----------
[echo] Handling RenderScript files...
[echo] ----------
[echo] Handling Resources...
[aapt] Generating resource IDs...
[echo] ----------
[echo] Handling BuildConfig class...
[buildconfig] Generating BuildConfig class.
-pre-compile:
-compile:
[javac] Compiling 98 source files to /project/CordovaLib/bin/classes
[javac] Note: Some input files use or override a deprecated API.
[javac] Note: Recompile with -Xlint:deprecation for details.
[echo] Creating library output jar file...
[jar] Building jar: /project/CordovaLib/bin/classes.jar
-post-compile:
-obfuscate:
-dex:
[echo] Library project: do not convert bytecode...
-crunch:
[crunch] Crunching PNG Files in source dir: /project/CordovaLib/res
[crunch] To destination dir: /project/CordovaLib/bin/res
[crunch] Crunched 0 PNG files to update cache
-package-resources:
[echo] Library project: do not package resources...
-package:
[echo] Library project: do not package apk...
-post-package:
-do-debug:
[echo] Library project: do not create apk...
[propertyfile] Creating new property file: /project/CordovaLib/bin/build.prop
[propertyfile] Updating property file: /project/CordovaLib/bin/build.prop
[propertyfile] Updating property file: /project/CordovaLib/bin/build.prop
[propertyfile] Updating property file: /project/CordovaLib/bin/build.prop
-post-build:
debug:
-code-gen:
[mergemanifest] Merging AndroidManifest files into one.
[mergemanifest] Manifest merger disabled. Using project manifest only.
[echo] Handling aidl files...
[aidl] No AIDL files to compile.
[echo] ----------
[echo] Handling RenderScript files...
[echo] ----------
[echo] Handling Resources...
[aapt] Generating resource IDs...
[echo] ----------
[echo] Handling BuildConfig class...
[buildconfig] Generating BuildConfig class.
-pre-compile:
-compile:
[javac] Compiling 26 source files to /project/bin/classes
[javac] Note: /project/src/com/chariotsolutions/nfc/plugin/NfcPlugin.java uses or overrides a deprecated API.
[javac] Note: Recompile with -Xlint:deprecation for details.
-post-compile:
-obfuscate:
-dex:
[dex] input: /project/bin/classes
[dex] input: /project/CordovaLib/bin/classes.jar
[dex] input: /project/libs/android-support-v4.jar
[dex] Pre-Dexing /project/CordovaLib/bin/classes.jar -> classes-f51f41294609d3a486a8c861c358f8e8.jar
[dex] Pre-Dexing /project/libs/android-support-v4.jar -> android-support-v4-2f8c19972ef2d53c6eb7faa4b6d719ed.jar
[dex] Converting compiled files and external libraries into /project/bin/classes.dex...
[dx] Merged dex A (44 defs/63.9KiB) with dex B (223 defs/326.3KiB). Result is 267 defs/474.3KiB. Took 0.8s
[dx] Merged dex A (267 defs/474.3KiB) with dex B (15 defs/14.2KiB). Result is 282 defs/623.5KiB. Took 0.2s
-crunch:
[crunch] Crunching PNG Files in source dir: /project/res
[crunch] To destination dir: /project/bin/res
[crunch] Processing image to cache: /project/res/drawable-hdpi/icon.png => /project/bin/res/drawable-hdpi/icon.png
[crunch]   (processed image to cache entry /project/bin/res/drawable-hdpi/icon.png: 0% size of source)
[crunch] Processing image to cache: /project/res/drawable-ldpi/icon.png => /project/bin/res/drawable-ldpi/icon.png
[crunch]   (processed image to cache entry /project/bin/res/drawable-ldpi/icon.png: 0% size of source)
[crunch] Processing image to cache: /project/res/drawable-mdpi/icon.png => /project/bin/res/drawable-mdpi/icon.png
[crunch]   (processed image to cache entry /project/bin/res/drawable-mdpi/icon.png: 0% size of source)
[crunch] Processing image to cache: /project/res/drawable-xhdpi/icon.png => /project/bin/res/drawable-xhdpi/icon.png
[crunch]   (processed image to cache entry /project/bin/res/drawable-xhdpi/icon.png: 0% size of source)
[crunch] Processing image to cache: /project/res/drawable/icon.png => /project/bin/res/drawable/icon.png
[crunch]   (processed image to cache entry /project/bin/res/drawable/icon.png: 0% size of source)
[crunch] Crunched 5 PNG files to update cache
-package-resources:
[aapt] Creating full resource package...
[aapt] Warning: AndroidManifest.xml already defines debuggable (in http://schemas.android.com/apk/res/android); using existing value in manifest.
-package:
[apkbuilder] Current build type is different than previous build: forced apkbuilder run.
[apkbuilder] Creating MySirenum-debug-unaligned.apk and signing it with a debug key...
-post-package:
-do-debug:
[zipalign] Running zip align on final apk...
[echo] Debug Package: /project/bin/MySirenum-debug.apk
[propertyfile] Creating new property file: /project/bin/build.prop
[propertyfile] Updating property file: /project/bin/build.prop
[propertyfile] Updating property file: /project/bin/build.prop
[propertyfile] Updating property file: /project/bin/build.prop
-post-build:
debug:
BUILD SUCCESSFUL
Total time: 17 seconds

The problem seems to be that the phonegap version is not updated.
It will be of huge help if you could update it


[WIP] Random weight and favored

This PR is basically a combination of random favored idea https://github.com/Practical-Formal-Methods/AFL-public/pull/4 and weight random selection https://github.com/Practical-Formal-Methods/AFL-public/pull/5.

@wuestholz, I am setting up the fuzzbench experiment with the following fuzzers.

vanilla AFL
the latest setting of the random favored PR (increase_boost_inputs && base_weight_fac == 4.0 && max_weight_fac_decr == 3.75)
combination of random favored and weight random selection (this PR with the customzed values same as fuzzer no.2  — increase_boost_inputs && base_weight_fac == 4.0 && max_weight_fac_decr == 3.75)
combination of random favored and weight random selection (this PR with default parameters)

@wuestholz, could you please help to confirm that I have correct fuzzers for the evaluation?

@jiradeto Thanks! This will be a local experiment, right?
Did you already run an experiment to compare increase_boost_inputs with new_increase_boost_fast_seqs (from https://github.com/Practical-Formal-Methods/AFL-public/pull/4#issuecomment-879207228)? If not we should also include just the increase_boost_inputs configuration.

@jiradeto Thanks! This will be a local experiment, right?

@wuestholz, yes I mean the local experiment.
So, based on your your comment in another PR (https://github.com/Practical-Formal-Methods/AFL-public/pull/4#issuecomment-882443156), I assume the current setting means the default value of increase_boost_inputs which is:
default && max_weight_fac_incr == 15.0

If this is the case, I will run fuzzbench with the following fuzzers:

vanilla AFL
the increase_boost_inputs setting of the random favored PR (max_weight_fac_incr == 15.0)
combination of random favored and weight random selection (this PR with max_weight_fac_incr == 15.0)
combination of random favored and weight random selection (this PR with default parameters)

Could you please tell me if I am missing something?

@jiradeto Great! Thanks!
I think we don't need the fourth configuration.
Could you please also update this PR to make 15.0 the default value?

Let's add the following flags:

DISABLE_WRS: disables weighted random selection
DISABLE_RF: disables random favorites
ENABLE_UF: enables uniformly random favorites (i.e., weight always 1)

DISABLE_RF: disables random favorites

@wuestholz, does this mean that we disable both random weighted selection (PR https://github.com/Practical-Formal-Methods/AFL-public/pull/4) and uniformly random favorites (PR https://github.com/Practical-Formal-Methods/AFL-public/pull/5)?

DISABLE_RF: disables random favorites

@wuestholz, does this mean that we disable both random weighted selection (PR #5) and uniformly random favorites (PR #4)?

@jiradeto No, just  uniformly random favorites (PR https://github.com/Practical-Formal-Methods/AFL-public/pull/4).

DISABLE_RF: disables random favorites

@wuestholz, does this mean that we disable both random weighted selection (PR #5) and uniformly random favorites (PR #4)?

@jiradeto No, just uniformly random favorites (PR #4).

@wuestholz OK, that makes sense.
But I am unsure when to use ENABLE_UF since the default weight is already set to 1.0 if all additional parameters enable_throttle_inputs, enable_boost_fast_seqs, enable_boost_inputs are not enabled.

DISABLE_RF: disables random favorites

@wuestholz, does this mean that we disable both random weighted selection (PR #5) and uniformly random favorites (PR #4)?

@jiradeto No, just uniformly random favorites (PR #4).

@wuestholz OK, that makes sense.
But I am unsure when to use ENABLE_UF since the default weight is already set to 1.0 and untouched if additional parameters enable_throttle_inputs, enable_boost_fast_seqs, enable_boost_inputs are not enabled.

@jiradeto I see. I suggest we replace enable_throttle_inputs, enable_boost_fast_seqs, and enable_boost_inputs with a single flag ENABLE_UF. When it is not set we will use enable_boost_fast_seqs and enable_boost_inputs.
We can probably remove the enable_throttle_inputs flag and the corresponding code.

@jiradeto Great! Thanks for the latest changes. There's only a small typo I noticed. I think we could try to set up an fuzzbench experiment with the following fuzzers:

afl (vanilla AFL)
afl_no_favs: DISABLE_WRS && DISABLE_RF && !ENABLE_UF && DISABLE_FAVS
afl_wrs_rf: !DISABLE_WRS && !DISABLE_RF && !ENABLE_UF && !DISABLE_FAVS
afl_rf:  DISABLE_WRS && !DISABLE_RF && !ENABLE_UF && !DISABLE_FAVS
afl_rf_u:  DISABLE_WRS && !DISABLE_RF && ENABLE_UF && !DISABLE_FAVS
afl_wrs: !DISABLE_WRS && DISABLE_RF && !ENABLE_UF && !DISABLE_FAVS

Can you think of another setup that would be interesting?

afl_rf_u: !DISABLE_WRS && !DISABLE_RF && ENABLE_UF && !DISABLE_FAVS

@jiradeto Sure, why not. :)
I was thinking a public one.

@wuestholz, I feel that I'd rather at least have a local experiment result before I can go with the public one. Maybe we should start with the local experiment to see whether the above configurations behave properly?

@jiradeto Sure, that's also fine with me. 👍

@jiradeto As discussed, I would first do the following:

close #8
merge #9
merge #7
close #3, #4, and #5

Afterwards, it would be great if you could run a local experiment that compares the following:

afl (vanilla AFL)
afl_wrs_rf: !DISABLE_WRS && !DISABLE_RF && !ENABLE_UF && !DISABLE_FAVS && DISABLE_RP
afl_wrs_rf_u: !DISABLE_WRS && !DISABLE_RF && ENABLE_UF && !DISABLE_FAVS && DISABLE_RP
afl_wrs_rf_t: !DISABLE_WRS && !DISABLE_RF && !ENABLE_UF && !DISABLE_FAVS && DISABLE_RP && changes below

The changes for the last configuration are:
      if (!enable_uniformly_random_favorites) {
        // enable_boost_inputs
        double base_weight_fac_boost_inputs = 1.0;
        double max_weight_fac_incr = 7.0;
        double scale_fac_boost_inputs = 0.001;
        double num_selections = (double)q->num_fuzzed;
        weight *= base_weight_fac_boost_inputs + max_weight_fac_incr / (scale_fac_boost_inputs * num_selections + 1.0);

        // enable_boost_fast_seqs
        double base_weight_fac_boost_fast = 8.0;
        double max_weight_fac_decr = 7.0;
        double scale_fac_boost_fast = 0.005;
        double execs_per_sec = 1000000.0 / (double) q->exec_us;
        weight *= base_weight_fac_boost_fast - max_weight_fac_decr / (scale_fac_boost_fast*execs_per_sec + 1.0);
      }

@wuestholz thank you for your detailed guidance. The fuzzbench evaluation for the changes you requested is available and can access using the following links:

bloaty
sqlite
libjpeg
libxml
freetype

@jiradeto Thanks a lot! Looks like it's slightly better. If you agree I suggest we update the code.


Expose App.parse_commands; provide cookbook example of using pyproject.toml

Expose App.parse_commands to public API.
Fix crash when displaying help where the only parameter has show=False.
Added a cookbook example of how to use pyproject.toml to the documentation.
Fixed out-of-date meta-app documentation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a264371) 95.79% compared to head (d702da6) 95.68%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   95.79%   95.68%   -0.12%
==========================================
  Files          16       16
  Lines        1664     1667       +3
  Branches      414      415       +1
==========================================
+ Hits         1594     1595       +1
- Misses         34       35       +1
- Partials       36       37       +1

Flag
Coverage Δ

unittests
95.68% <100.00%> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Setting AMGX device id

Hi, I really enjoy this project!
I would like to use my Julia code on multiple devices (no MPI) and need to set the device id. Is there any way to do that, besides AMGX_resources_create?
If not, is there any chance the functionality is going  to be implemented?
Although, not familiar with wrappers, I might also help out whereever I can.
Thanks!

How would you do it from the C library?

const int deviceID AMGX_config_handle cfg; AMGX_resources_handle rsrc; AMGX_resources_create(&rsrc,cfg,NULL, 1,&deviceID);
A workaround is to start Julia with:
CUDA_VISIBLE_DEVICES=deviceID, that is not my preferred solution.
Thanks!

Okay, only resources_create_simple is currently wrapped (https://github.com/JuliaGPU/AMGX.jl#not-implemented).
There could be another create! function as in https://github.com/JuliaGPU/AMGX.jl/blob/f812a39efcbb3c82e13936521a6b46497cea7bb4/src/Resources.jl#L19-L26 that takes a device id as well.
Maybe just create!(resources::Resources, cfg::Config; device_id) and if device_id is given it calls  AMGX_resources_create otherwise AMGX_resources_create_simple.


chore(main): release bspwm 1.16.0

:robot: I have created a release beep boop
1.16.0 (2024-08-11)
Features

compositor: update name from compton to picom and add global opacity (51e75b4)

Bug Fixes

compositor: inactive window opacity need mark-ovredir-focused = false (0a5a581)

Miscellaneous

arch: add arch settings (730a639)
picom: add comment about opacity excludes (3ebb8e5)
zsh: run ssh keygen directly (db14087)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

bspwm-v1.16.0
:sunflower:


TabView not work in rtl

-FlowDirection="RightToLeft" not work in tabview
-when use a rtl entry in tabview,by focus on entry the tab switch to last tab

Please use the issue template to provide more info on the issue you encounter, thanks!


Does not compile against recaptcha@#head

The recaptcha library added a second overload for initRecaptcha.  Consequently, nimwcpkg/webs/captchas.nim:setupReCaptcha must provide a third argument (either a bool or a Provider) for Nim to unambiguously resolve the call.  This will matter more once recaptcha stamps a new release.  You may want a when compiles guard to handle both old & new recaptcha libs.

Hi @c-blake
Good catch and thank you!! Actually, I should have noticed this, since I made the PR :dango:  https://github.com/euantorano/recaptcha.nim/commit/0c471d24fee645fc50ffa35849a88d1a3ed00e82

I found it compiling a big chunk of The Nimbleverse to test https://github.com/c-blake/nimp.  Nimble uses latest tagged releases while nimp uses #head all the time, really..(or whenever the installer last did nimp up for all their packages).
I started to "patch/fix" said Nimbleverse before deciding to compare how much regular nimble failed - which was "also a lot". :-)  The main Nim does important packages CI, but 30-40% of "wider world" packages just break from neglect.  { I do not think nimble devs test against the 1350 packages they could.  The onus is instead upon package authors to test nimble. }


Add Token Failure

When ever you try to add tokens nothing happens.

#463 read the thing that says [README], maybe?

These hacks work perfectly fine. https://github.com/rxzyx/Blooket-Hacks

It says unable to get username someone help

These hacks work perfectly fine. https://github.com/rxzyx/Blooket-Hacks

rzx is unreliable and mostly stolen from this repo, if you want cheats that are actually made by someone who knows what they're doing go here https://github.com/Minesraft2/Blooket-Cheats

doesn't work it says error getting username help please :(


New: Object detection tutorial

Just waiting to merge https://github.com/dotnet/machinelearning-samples/pull/882 to get code snippets.
Can replace #21192

@Youssef1313 anything else needed here?
sample has been merged

The sample isn't yet live at https://docs.microsoft.com/samples/dotnet/machinelearning-samples/mlnet-object-detection-model-builder, but this doesn't seem to be a blocker from merging. We can update the link later when it's live.

Do you know why there's a build error?

@briacht
There are intermittent outage on the build system. I don't have an ETA at the moment.

Ok no prob! Wanted to make sure there's nothing to fix on my end.

closing and reopening for a fresh build.

Looks like all checks passed. @Youssef1313 thanks for all your help!! @BillWagner ready to merge?


Exec tachikoma update<PHONE_NUMBER>2411

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io

Coverage remained the same at 77.5% when pulling ab968e0e0d21b88c903de1a25c5d38ff121e398e on tachikomapocket:tachikoma/update-20150226102411 into 5dc354127eb256891397176231fae0a4a9c364b1 on app2641:master.


[SPA] Update to latest oidc-version and perform appropriate cleanups

Updates to the latest version of oidc-client and removes a bunch of template code as a result.

I plan to merge this as part of preview9, but getting it done early in the milestone.

tactics approved - wait to merge for @wtgodbe go ahead.

@javiercn let me know when you're happy and i can do the formal release of oidc-client.

You don't need to wait on me for this PR, that's only for CoreFx/Core-Setup/CoreClr


Different codellama models - confused

What is the difference between

ollama run codellama:7b
ollama run codellama:7b-instruct
ollama run codellama:7b-code

for general purpose (not python specialised) programming?

@abulka sorry that it wasn't clear.

codellama:7b is the same as 7b-instruct, we alias it so that it'll be simple for people just wishing to run codellama.

instruct is for general purpose - this is where you can just 'chat' or have a dialog with it.
code - is made for code completion. So if you pass in partial unfinished code, it'll try to complete it. This is usually made for apps, code editors etc.
Hope this helps! Let me know if I've answered your question, and how we can make it clear on the page. Did you find it in the Ollama model library?

Thanks for the explanation!  Would be useful if this info was added to https://ollama.ai/blog/run-code-llama-locally at least.
I'm subscribed to the Ollama newsletter and probably heard about it there - or possibly via the TLDR newsletter.

Hey there, the overview for the model on ollama.ai (https://ollama.ai/library/codellama) is the best place to go to find out, but I didn't make it easy to translate the name of the model variant to the actual tags. Now I have an example for each one.


Inject logger when create new instance of mailjet service

It would be great to include this on next release! Those logs are making my eyes bleed.


Make Dispatch work on Docker for Mac

To get around the local image registry availability problem, use
host.docker.internal:5000 as the imageRegistry in dispatch install
config.yaml:
  imageRegistry:
    name: host.docker.internal:5000
    insecure: true

Run a standalone image registry:
docker run -d -p 5000:5000 --restart=always --name=registry registry

In Docker for Mac preferences:

add host.docker.internal:5000 to the list of insecure registries
add http://localhost:5000 to the list of registry mirrors

Great! Can you also update the docs with your findings? E.g. add a "Docker on Mac" section


Create documentation for SSO integration through SAML

Purpose of PR?:
Fixes #
Implementation of documentation for SSO via SAML Integration for MSColab Server
Does this PR introduce a breaking change?
No
If the changes in this PR are manually verified, list down the scenarios covered::
Yes
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Created after the suggested improvements of Draft document (https://docs.google.com/document/d/1fX7F4KYQ-iIibvMjSqF0oInse-ew2naJ_ataqylNU8Q/edit?usp=sharing)
Does this PR results in some Documentation changes?
If yes, include the list of Documentation changes
Added new documentation for SSO
Checklist:

[ ] Bug fix. Fixes #
[ ] New feature (Non-API breaking changes that adds functionality)
[ ] PR Title follows the convention of  <type>: <subject>
[ ] Commit has unit tests

From where are the png files? Have you done these yourself?

Yes.. screen shots taken from my virtual machine ; )

please use a different dir than gallery. It might look suitable, but this is a placeholder for the gallery stuff created on building the documentation, see https://mss.readthedocs.io/en/stable/gallery/index.html
e.g. images

Got it.. Thank you..  Updated :)


Dynamic content not getting displayed.

Hi I have this code which uses dynamic content on iron swipeable pages
<iron-swipeable-pages selected="0" id='swipePages'>
    <template is="dom-repeat" items=[[data]]>
        <div id="[[item.number]]">
          Some info here...
        </div>
    </template>
</iron-swipeable-pages>

It works fine when page loads initially but after that as the data gets changed it is not showing the items in view. Any idea how can I bind dynamic content in here?

It should "just work"; it does on my site. Most likely the changes you are making to the data array are not observable.
Check the contents of your data array in the debugger after you set it to sanity check that it's actually getting set the way you think it is. I was pulling my hair out over a similar issue recently and it was something that simple.
Inspect the properties on the div and template tags to ensure data is actually propagating.
Set an observer on data.0.number and put console.log("data.0.number changed") in it (and maybe set a breakpoint) to verify that the changes are observed when you mutate data. I bet it does not. I've found making observable changes to arrays is pretty tricky in Polymer. Be sure you are using the proper array manipulation technique.
See also https://www.polymer-project.org/1.0/docs/devguide/data-system#observable-changes

let's close this!


Allow datastore clusters to be nested in folders

Currently datastore clusters must be top level for the cpi to find them, this change will add functionality to search for them within storage folders in vcenter.
Slashes are not valid in folder or datastore cluster names, they must be escaped in vCenter, so we use slash as our expected folder delimiter.
For the "pattern" matching case, after splitting on slashes, we have to convert each segment into a regular expression in case the user is relying on that for wildcard matching. As regexes don't use forward slashes, and the entity names they are searching for also can't have slashes, this should be safe.
The unused StoragePod.find method was also removed

We realized that this behavior will not work if you specify a datastore pattern that has multiple regex options already. i.e. /folder1/datastore1|/folder2/datastore2|datastore3

We've updated search_in_subfolders to now build a full index of datastore clusters (StoragePods) and the folders they are in, then use the provided pattern to match the appropriate ones.

@lnguyen could you take an other look at the changes?


Query interface for Transactions

Because statistics! /cc @fabian-rump

We should probably use https://rubygems.org/gems/ransack for this.

Ransack looks good, Will probably land in the next few days.

Before you move forward with the implementation, let me provide some input from the client side. Specifically, Angular JS supports filtering and aggregation directly in the client, given that the API can provide a rather "raw" data output at a suitable granularity. I have some ideas how to build a nice admin statistics interface in mind, but need to read through some pieces of the documentation to formulate the exact requirements which I would need to implement this. Maybe I can do this tomorrow on my way back from Berlin to Bonn. Just poke me if I forget to get back on this ;-)

Bäm, there is now the possibility to search and filter for transactions. See the email I sent you for some examples. Once you have something working off of that for stats, we can also have a quick chat to see how to define the queries you'd need.

Oh, and there's no docs for that now, ping me if I forget.

Last but not least: as this allows some fine-grained stuff, I'd keep it behind a specific app token only for this purpose that we should keep a good eye on.


branched-chain polymer building

Hello. Thank you for creating such a wonderful toolkit. Using this feature, is it possible to produce a branched-chain polymer with more than three reactive sites within the monomer?

We appreciate your feedback. Currently, PSP doesn't support creating models
for branched-chain polymers with more than three reactive sites within a
monomer. However, we have taken note of your request, and we will certainly
consider adding this capability in a future update. Thank you so much.
Thank You!Harikrishna Sahu
Postdoctoral fellow | Georgia Institute of Technology, USA
Webpage: http://harikrishnasahu.com
ORCID https://orcid.org/0000-0001-5458-9488 | Web of Science
https://publons.com/researcher/4003680/harikrishna-sahu/ | Google
Scholar https://scholar.google.co.in/citations?user=kiTk5rAAAAAJ&hl=en |
LinkedIn https://www.linkedin.com/in/harikrishna-chem/ | ResearchGate
https://www.researchgate.net/profile/Harikrishna-Sahu-2
On Sun, 20 Aug 2023 at 20:44, hirose125 @.***> wrote:

Hello. Thank you for creating such a wonderful toolkit. Using this
feature, is it possible to produce a branched-chain polymer with more than
three reactive sites within the monomer?
[image: image]
https://user-images.githubusercontent.com/113399340/261885742-3fec5acd-b509-43f3-905a-31e7681af17d.png
—
Reply to this email directly, view it on GitHub
https://github.com/Ramprasad-Group/PSP/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFBG736RK3B565GEDTLUOUDXWKVNXANCNFSM6AAAAAA3XUEOE4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for your prompt reply.
It might be challenging, especially when there are multiple reaction points, due to the order of rare numbers or randomness. However, I believe it would be extremely beneficial for actual material development, particularly in the formation of network polymers. I understand it's just a suggestion, but I'm looking forward to the addition of this feature.


Process service test fails to exit correctly on Windows

When running this test on Windows, I keep getting an error killing the process (I believe treekill is to blame).
This logic resolves the problem.

Codecov Report

Merging #8514 into master will decrease coverage by 0.37%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8514      +/-   ##
==========================================
- Coverage   59.11%   58.74%   -0.38%
==========================================
  Files         510      510
  Lines       23564    23564
  Branches     3813     3808       -5
==========================================
- Hits        13931    13842      -89
- Misses       8725     8805      +80
- Partials      908      917       +9

Impacted Files
Coverage Δ

src/client/testing/serviceRegistry.ts
48.64% <0%> (-46.85%)
:arrow_down:

src/client/testing/codeLenses/main.ts
44.44% <0%> (-33.34%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
66.66% <0%> (-33.34%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
75% <0%> (-25%)
:arrow_down:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
75% <0%> (-25%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
60% <0%> (-20%)
:arrow_down:

src/client/common/utils/icons.ts
83.33% <0%> (-16.67%)
:arrow_down:

src/client/api.ts
78.57% <0%> (-14.29%)
:arrow_down:

src/client/datascience/cellFactory.ts
57.69% <0%> (-12.83%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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


Wrong placed data.json prevents starting

data.json file was in root directory of SOFTEdit instead of data\data.json. See log file.
logs-2023-03-26.txt

This is strange. I just downloaded the zip file from github and it does indeed have the data.json at the right place:

Is it possible that your path is just too long? Windows has a certain limit. Could you try to download it again and decompress it to a different location?

Its more like.. im using the "r2modman-3.1.39" where i did download your mod. So, maybe its wrong on their side?

Hm it's possible. So does the version from github work for you? Because I've basically uploaded the same ZIP file everywhere. My best guess is that the pathname is just too long

I think I found something:
https://github.com/ebkr/r2modmanPlus/issues/944
So it is indeed their way of handling zip files. I'll have a look tomorrow at that I can do.

Yes, im 100% certain its because of the flat file hierachy.

BTW. This is not a BeoinEx Plugin but rather a standalone application. Does that make any difference in regards to your report? Is it even the right way to install it like the way you did?

Hey, i think its up to you if installing the tool via modmanPlus is the right way anyway. I installed it via the program and saw its a .net exe in appdata which cant be started as a real "mod" via game. So i executed your exe after install just to see the file hierarchy for the json is wrong and i corrected it manually anyway. I think this issue can be solved if you make it either flat or dismiss it from this mod tool. I just think about other people who could potentionally have the same problem and cant read .net exceptions.

To be honest, extracting zip archives to a flat structure because of legacy reasons doesn't justify that I change the whole application structure. In my opinion it's a bug in r2modman
I'll try to find another way.

Understandable. For now ill use the "standalone" tool directly from your github.

I raised a feature request on their github:
https://github.com/ebkr/r2modmanPlus/issues/997
Let's see what they respond


Refactor to pypi

[ ] Make the package pip installable and organize with a setup.py.
[ ] Pass linting
[ ] Unit tests
[ ] Coverage > 90%
[ ] .github actions workflow
[ ] descriptive README.md
[ ] submit to pypi

Functions should also match the scikit learn API


enable OSP pruner again for resolver memory issues and results prune performance issues  seen on prod-rh01

@enarha @gbenhaim @ralphbean PTAL
rh-pre-commit.version: 2.3.0
rh-pre-commit.check-secrets: ENABLED

/hold

keep-since added @enarha
that said, I'm not sure if the e2e's ran last time ... will try to launch manually
/test appstudio-e2e-tests

is is running now

/test appstudio-e2e-tests

/test appstudio-e2e-tests

/lgtm

tests ran successfully .... details in https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/redhat-appstudio_infra-deployments/4171/pull-ci-redhat-appstudio-infra-deployments-main-appstudio-e2e-tests/1816154072160407552/artifacts/appstudio-e2e-tests/redhat-appstudio-e2e/build-log.txt
reminder pod logs are not collected on passing tests
/hold cancel


Upgrade gRPC to 1.25 which has gRPC STS feature

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Upgrade gRPC to d8f4928fa779f6005a7fe55a176bdb373b0f910f (1.25.x).
Follow https://github.com/envoyproxy/envoy/pull/9041 that updates gRPC from 1.22 to 1.25.
Follow https://github.com/envoyproxy/envoy/issues/6584 that fixes compile issues complained by clang-8.
Description:
Risk Level: Low
Testing: Existing Tests
Docs Changes: N/A
Release Notes: N/A
https://github.com/istio/istio/issues/20133

Got a compile issue
WARNING: The following rc files are no longer being read, please transfer their contents or import their path into one of the standard rc files:
/usr/local/go/src/istio.io/istio-envoy/envoy/tools/bazel.rc
INFO: Writing tracer profile to '/usr/local/.cache/bazel/_bazel/692781258b79ccbcfe64aa8d9ce81e87/command.profile.gz'
INFO: Analyzed target //source/exe:envoy-static (4 packages loaded, 1489 targets configured).
INFO: Found 1 target...
ERROR: /usr/local//go/src/istio.io/istio-envoy/envoy/source/common/http/BUILD:11:1: C++ compilation of rule '//source/common/http:async_client_lib' failed (Exit 1) clang failed: error executing command /bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -fcolor-diagnostics -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections -fdata-sections ... (remaining 347 argument(s) skipped)
Use --sandbox_debug to see verbose messages from the sandbox
source/common/http/async_client_impl.cc:20:73: error: call to implicitly-deleted default constructor of 'const AsyncStreamImpl::NullRetryPolicy'
const AsyncStreamImpl::NullRetryPolicy AsyncStreamImpl::RouteEntryImpl::retry_policy_;
^
bazel-out/k8-opt/bin/source/common/http/virtual_includes/async_client_lib/common/http/async_client_impl.h:147:33: note: default constructor of 'NullRetryPolicy' is implicitly deleted because field 'retriable_status_codes' of const-qualified type 'const std::vector<uint32_t>' (aka 'const vector') would not be initialized
const std::vector<uint32_t> retriable_status_codes_;
^
1 error generated.
Target //source/exe:envoy-static failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 143.211s, Critical Path: 29.15s
INFO: 62 processes: 62 linux-sandbox.
 This applies even more so to master and 1.5 branch but I think is relevant here as well

I will update envoy upstream in a separate PR today or tomorrow, and upgrade gRPC SHA to the same SHA as this PR.
Envoy upstream and istio/envoy 1.5 branch both use gRPC 1.25.0, only 1.4 requires upgrading gRPC from 1.22 to 1.25. All the changes to other files are to fix build issues caused by upgrading from 1.22 to 1.25. envoy upstream master and istio/envoy 1.5 branch do not need those changes. I think the risk of diverge is low.

Ok I think this makes sense, but I would like approval from a proxy owner before merging

Ok I think this makes sense, but I would like approval from a proxy owner before merging

Thanks a lot!

Two comments:

Why do we need this in 1.4? We've made a decision long time ago that old branches are only for serious bug fixes and CVEs. This looks like a new feature?
In theory, we already switched to Stable releases of Envoy and Istio Proxy switch to unmodified Envoy (this is not strictly true, since we still maintain this repo, but we should move away from it), which means that this change should be suggested against Envoy v1.12 stable branch and we should get it from there.

Merging, because but I still think it's a bad idea...


Crash Report

When I open some skill books, the game crashes, when i reload the world, all entities become invisible.
Mods: Epic Fight, Iron's spell, Weapons of minecraft, Paragliders, Paragliders Stamina Compats, Paragliders with Double Stamina

Crash should attach crash report

I think it's should be fixed. But you need attach crash report file to this issue


Unhandled Exception: NoSuchMethodError: The method 'forEach' was called on null. for Function rtcEngine.getUserInfoByUid

Describe the bug
A clear and concise description of what the bug is.
when using the getUserInfoByUid on the event of the user joining a room. You get "The method 'forEach' was called on null." from package:agora_rtc_engine/src/rtc_engine.dart:266:32
[VERBOSE-2:ui_dart_state.cc(186)] Unhandled Exception: NoSuchMethodError: The method 'forEach' was called on null. Tried calling: forEach(Closure: (dynamic, dynamic) => void) #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:54:5) #1      new LinkedHashMap.from (dart:collection/linked_hash_map.dart:87:11) #2      RtcEngine.getUserInfoByUid.<anonymous closure> (package:agora_rtc_engine/src/rtc_engine.dart:266:32) #3      _rootRunUnary (dart:async/zone.dart:1362:47) #4      _CustomZone.runUnary (dart:async/zone.dart:1265:19) #5      _FutureListener.handleValue (dart:async/future_impl.dart:152:18) #6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:704:45) #7      Future._propagateToListeners (dart:async/future_impl.dart:733:32) #8      Future._completeWithValue (dart:async/future_impl.dart:539:5) #9      _completeOnAsyncReturn (dart:async-patch/async_patch.dart:254:13) #10     MethodChannel._invokeMethod (package:flutter/src/services/platfor<…>
To Reproduce
Steps to reproduce the behavior:
`
userJoined(uid, elapsed) async {
print("user: $uid joined!");
print("Getting User Info");
UserInfo value = await RtcEngine.getUserInfoByUid(uid);

}
`
use this code, now try to join a channel with people already on it and it will throw that error.
Expected behavior
A clear and concise description of what you expected to happen.
For it to return the correct UserInfo object about the users already in the room.
Smartphone (please complete the following information):

Device: [e.g. iPhone11]
OS: [e.g. iOS14.1]
Version [3.3.3]
flutter

Fixed by waiting 200 ms with a Future
await Future.delayed(Duration(milliseconds: 200));

pls refer to https://github.com/AgoraIO-Community/Agora-Flutter-Quickstart/issues/217


If TargetPlatformVersion is not set, adding this package will cause the project to not load

If I create a new netcore project and reference the Microsoft.UI.xaml nupkg, NuGet will throw an error about not being able to parse something.  This is because the TPV/TPMV are not set and you are trying to use them in .targets/props files.

Looks like this was fixed in newer package versions.


utilities.helpers.make_valid_url breaks some urls

Code
def make_valid_url(url):
    """Make sure url doesn't contain unsupported characters

    Args:
        * `url`: a url string
    """
    if '/+/http' in url:
        glue = '/+/'
        quoted = [make_valid_url(unquoted) for unquoted in url.split(glue)]
        return (glue).join(quoted)

    scheme, netloc, path, query, fragment = compat.urlsplit(url)
    quoted_path = compat.quote(path.encode('utf-8'))
    quoted_query = compat.quote_plus(query.encode('utf-8'))
    new_url_tuple = (scheme, netloc, quoted_path, quoted_query, fragment)
    quoted_url = compat.urlunsplit(new_url_tuple)
    return quoted_url

Example
Before this function (OK-200):
http://next.openspending.org/fdp-adapter/convert?url=https%3A%2F%2Fraw.githubusercontent.com%2Fkravets-levko%2Fdata%2Fmaster%2Ftest.xlsx.csv

After this function (ISE-500):
http://next.openspending.org/fdp-adapter/convert?url%3Dhttps%253A%252F%252Fraw.githubusercontent.com%252Fkravets-levko%252Fdata%252Fmaster%252Ftest.xlsx.csv

cc @akariv @kravets-levko

OpenSpending server can't accept this e.g: http://next.openspending.org/fdp-adapter/convert?url%3Dhttps%3A%2F%2Fraw.githubusercontent.com%2Fkravets-levko%2Fdata%2Fmaster%2Ftest.xlsx.csv

This function makes url invalid after encoding query part: ...convert?url%3Dhttps... should be ...convert?url=https.... I'm not a Python guy, but it looks that compat.urlsplit returns query as string which is then entirely url-encoded, instead of url-encoding only query values.

сс @georgiana-b


Fix event handler on objects

If you have multiple objects trigger handler, you have to bind this into the handler function first with different function name. And do not forget to make off on the handler. For example:
class MyClass
...
this.onMyEvent__ = this.onMyEvent.bind(this);
events.on("myevent", this.onMyEvent__, this);
...
events.off("myevent", this.onMyEvent__);

If you have multiple objects trigger handler, you have to bind this into the handler function first with different function name. And do not forget to make off on the handler. For example:
class MyClass
...
this.onMyEvent__ = this.onMyEvent.bind(this);
events.on("myevent", this.onMyEvent__, this);
...
events.off("myevent", this.onMyEvent__);


Initial functions to implement the Fusion 360 Post Processor API.

Thanks! I'm not sure it makes sense to merge this in as-is though as it effectively turns the executable into a single-purpose thing - for instance someone runs this just expecting to try out the interpreter (or tries to embed it in some other project), and it will have already created an out.nc file (but there was no mention of the fact it would do that).
You could make it a command-line option (--fusion) to add the relevant fusion functions and create the out.nc file I guess, or you could just keep your work as a fork of this repo as you have done so far, which might actually be the best option.

Sorry, I had the wrong repository to merge this with, as you suggest. This should be in my fork. Thanks for writing TinyJS. Clearly I think it's handy.


support tcp syslog

The only problem is that using the stdlib syslog package prevents you from changing the syslog tag across writes. That's why right now I Dial once for every message sent over syslog (but it's ok because it's udp). However, TCP would require a persistent connection.
Perhaps the solution is to vendor and fork the syslog package as it is not that complicated. A patch upstream seems unlikely to be merged because this is a much more uncommon use case (logging on somebody else's behalf).

Master supports TCP syslog and can easily support TLS if somebody submits a transport.

@progrium is there any docs on how to use TCP vs UDP syslog? It seems to UDP by default?

This one is allegedly working.

@gtmtech With master, better try syslog+tcp:// as described in #107.

Thanks I found it in the end. Real shame it doesn't support TLS, I cant use it.

It's probably a 20-50 line addition for TLS. Surprised nobody has added it
yet.
On Thu, Aug 6, 2015 at 4:48 AM, Geoff Meakin<EMAIL_ADDRESS>wrote:

Thanks I found it in the end. Real shame it doesn't support TLS, I cant
use it.
—
Reply to this email directly or view it on GitHub
https://github.com/gliderlabs/logspout/issues/1#issuecomment-128311080.

--
Jeff Lindsay
http://progrium.com

This is handled in #116, right?


[Microsoft Generator CSharp] Refactor ApiTypes Properties into Expression / Statements Overrides

Currently, the ApiTypes class contains abstract properties for certain types, ie. ResponseType. This design should be refactored to be backed into expression and statement overrides instead, which extending plugins can override. In addition, ApiTypes should be combined with the ExensibleSnippets class, as there are several overlaps.
Original discussion: https://github.com/jorgerangel-msft/Microsoft.Generator.CSharp/pull/12#discussion_r1491485984 and https://github.com/jorgerangel-msft/Microsoft.Generator.CSharp/pull/13#discussion_r1496720672

Semantically speaking, the responsibility of "getting what type we should have for a client response" should belong to the TypeFactory as its name suggests.
And for those giving a complicated snippets, we could put them into the Snippets extensions.

I think its also fair to have methods on the TypeFactory which return an Expression.
For instance lets imagine we have a generic ChangeTrackingListExpression and instead of ApiTypes with generic snippets we have a method on TypeFactory
public virtual ChangeTrackingListExpression GetChangeTrackingList();

Now people can get and use that expression by its interface without needing to know the implementation or call into a separate place where we add extensions such as snippets.
I think haven't multiple places people need to go for these extensions is probably a bad thing.

I think its also fair to have methods on the TypeFactory which return an Expression.
For instance lets imagine we have a generic ChangeTrackingListExpression and instead of ApiTypes with generic snippets we have a method on TypeFactory
public virtual ChangeTrackingListExpression GetChangeTrackingList();

Now people can get and use that expression by its interface without needing to know the implementation or call into a separate place where we add extensions such as snippets.
I think haven't multiple places people need to go for these extensions is probably a bad thing.

I agree but this could lead to that we are putting everything into TypeFactory therefore we should add some organization for it so that it could have hierarchies.

#4720


Generate Multi-level Feature Maps

Hello, may I ask what is the difference between using ViT Encoder and Convolutional Decoder to generate Multi-level Feature Maps, and using PVT Encoder to generate Multi-level Feature Maps directly?

Hi
I didn't specifically test PVT encoder, but the idea to use a convolutional decoder is to preserve the color information from the source images.
We found that only using the ViT encoder could cause blurriness in some areas.


Check for presence of 3.x openssl runtime

if the newly loaded engine contains the symbol
EVP_PKEY_get_base_id, we know it is linked to 3.x openssl.
Abort loading this engine, as it will definitely crash.
This is meant in combination with the first commit in #17112 which cherry-picks cleanly to 1.1.1

Nothing new with load of engines. 20 years old story . We all now do not load incompatible plugins - not only in openssl but for all projects.
So silly defect must not be fixed.
Plus functional issues - hidden disable "at exit " . What if application relies on automatic cleanup? And the check for symbpol presence is not reliable (pointed in prevoius PR).

OTC hold added for the same reason as given in https://github.com/openssl/openssl/pull/17112#issuecomment-1015340909

re-based to latest 1.1.1 branch, with now change, was necessary to work around the recent force-push to 1.1.1

what is the OTC decision about this PR, shall it be merged to 1.1.1 ?

Although OTC did not explicitly discuss 1.1.1 they did discuss whether it was suitable for backport to a stable branch (i.e. 3.0). The decision was "yes", so I see no reason why the same doesn't also apply to 1.1.1. Lifting the hold.

Thanks for clarification.  @levitte may I assume your approval still valid?

Merged to 1.1.1 branch.  Thanks!