Footnotes and Crossreferences are not working in user books with book id greater than 127

Bug Description
I wanted to create a yes bible module that includes a glossar. So I created a yet file and added the additional content of the glossar to a custom book using book id 201 (as recommended in .yet file format specification):

If the book you want to add is not listed there, you can use any other number starting from 201 up to 255.

Then I added footnotes and crossreferences to the verses of this custom book. After converting the yet file to a yes bible module with YetToYes2.jar I added the bible module to the Quick Bibel App v4.9.0. I f I tap on a footnote or cross reference in the custom book, the footnote or cross reference is sometime shown sometimes an error message appears (Something like 'The footnote with arif cannot be retrieved'). It seems like random, if a footnote or crossreference can be retrieved sucessfully.
Current Behavior
Footnotes and crossreferences in a custom book are randomly working or not.
Expected Behavior
Footnotes and crossreferences in a custom book are always working.
Detailed Technical Bug Description
I investigated the behaviour and found the source of the bug. If a yet file is converted to a yes file, the book, chapter, verse, index values in a footnote/xref line are converted to an arif. These arifs are written to the yes file to the 'footnotes' / 'xrefs' section into the arifs[] array: https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/AlkitabYes2/src/main/java/yuku/alkitab/yes2/section/FootnotesSection.java#L13-L20 https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/AlkitabYes2/src/main/java/yuku/alkitab/yes2/section/XrefsSection.java#L13-L20
The arif array is used to find a footnote/crossreference. This is done using binary search:
https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/AlkitabYes2/src/main/java/yuku/alkitab/yes2/section/FootnotesSection.java#L59
https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/AlkitabYes2/src/main/java/yuku/alkitab/yes2/section/XrefsSection.java#L59
And there is the problem: The array is required to be sorted for the binary search (see Android documentation for Arrays.binarySearch(). You mentioned this requirement too). But the array is not sorted, if a book id greater than 127 is used. The order of the arifs in the arifs array of the yes2 file section is in the same order they appear in the yet file (ascending by book, chapter, verse, index; this is enforced by the converter YetToYes2.jar). For book id greater than 127 (128 in the yet file) it leads to an unsorted array because the most significant bit of the arif is 1 so the arif become negative. Example:
xref     40    1    2    1  => arif =   671154689
xref    127    5    2    1  => arif =<PHONE_NUMBER>
xref    128    1    1    1  => arif = -2147417855
xref    128    1    1    2  => arif = -2147417854

Possible Solution
The arif array (and therefore the offset array) should be sorted before written to yes2 file. Most easy solution: Sort the LinkedHashMap or the .entrySet() ascending by key (=arif) before writing the arrays. https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/tools/AlkitabConverter/src/main/java/yuku/alkitabconverter/util/FootnoteDb.java#L102-L122
https://github.com/yukuku/androidbible/blob/eb4462f7e7b81c6415f9704ddf509b74d968fcbb/tools/AlkitabConverter/src/main/java/yuku/alkitabconverter/util/XrefDb.java#L131-L151
At least the recommendation for custom book id/number in the Alkitab / Quick Bible Developer Guide: Bible Version Data (.yet file format specification) should be changed.

Thank you, @restai for the thorough investigation and suggestions.
I did the fix on the binarysearch function of the app itself, because there could be others who have created the YES file with the sorted-as-unsigned-int arrays of arif.

Thank you @yukuku for the fast fix. Indeed it makes more sense to fix it on application side. I hope a new app release is coming soon. Unfortunately the Quick Bible app is still version 4.9.0 and not latest version 4.10.0, the version of the indonesian version Alkitab. In particular the fix for the annoying bug "Footnote dialog links are not clickable" 13a872e6b77b0f4267f589c91c7e789091003243 already in 4.10.0 is not available to Quick Bible app users.

Thank you @yukuku for the fast fix. Indeed it makes more sense to fix it on application side. I hope a new app release is coming soon. Unfortunately the Quick Bible app is still version 4.9.0 and not latest version 4.10.0, the version of the indonesian version Alkitab. In particular the fix for the annoying bug "Footnote dialog links are not clickable" 13a872e already in 4.10.0 is not available to Quick Bible app users.

Hi @restai, I'm sorry it took so long to update to the newest version. Now, both apps have been updated to the latest version with the fix.


Duplicate objects not allowed, regardless of ownership

Step 0:

[x ] I've  read the contribution guidelines!

Expected Behaviour
2 users should be able to make streams with identical object lists
Actual Behaviour
Second user who tries to create a stream and upload objects will find that no new objects are created and the objects shown in their stream are actually owned by the first user.
Reproduction Steps & System Config (win, osx, web, etc.)
create 2 streams full of identical stuff from 2 different users. Here's a basic example of the python code I was using:
def new_stream(name, description=''):
    url = "https://{}/api/streams".format(os.environ["SERVER"])
    payload = {"name": name, "description": description}
    r = requests.post(url, data=json.dumps(payload), headers=headers)
    return r.json()

""" send dummy data to speckle stream """
stream = new_stream(name, description)
stream_id = stream["resource"]["streamId"]
print(stream_id)
url = "https://{}/api/streams/{}".format(os.environ["SERVER"], stream_id)
with open("./dummy_data.json") as f:
    objects = json.load(f)["resources"]
payload = {"objects": objects}
r = requests.put(url, data=json.dumps(payload), headers=headers)

Proposed Solution (if any)
I get that this helps avoid redundancy, but I feel like there's realistic cases where 2 users might wanna work on the same data. 'lowit

Not sure what’s the problem? Object ownership is not enforced, only stream ownership - so the second user can still have his own (private or not) stream with identical objects.
Unless i’m missing something 👀

On 6 Feb 2020, at 14:57, tluth<EMAIL_ADDRESS>wrote:
Step 0:
[x ] I've read the contribution guidelines https://github.com/speckleworks/SpeckleOrg/blob/master/CONTRIBUTING.md!
Expected Behaviour
2 users should be able to make streams with identical object lists
Actual Behaviour
Second user who tries to create a stream and upload objects will find that no new objects are created and the objects shown in their stream are actually owned by the first user.
Reproduction Steps & System Config (win, osx, web, etc.)
do something like this twice with different JWTs:
def new_stream(name, description=''):
url = "https://{}/api/streams".format(os.environ["SERVER"])
payload = {"name": name, "description": description}
r = requests.post(url, data=json.dumps(payload), headers=headers)
return r.json()
""" send dummy data to speckle stream """
stream = new_stream(name, description)
stream_id = stream["resource"]["streamId"]
print(stream_id)
url = "https://{}/api/streams/{}".format(os.environ["SERVER"], stream_id)
with open("./dummy_data.json") as f:
objects = json.load(f)["resources"]
payload = {"objects": objects}
r = requests.put(url, data=json.dumps(payload), headers=headers)
Proposed Solution (if any)
I get that this helps avoid redundancy, but I feel like there's realistic cases where 2 uses might wanna work on the same data
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/speckleworks/SpeckleCore/issues/140?email_source=notifications&email_token=AB2XBA35MQE4UJYMA6L3UPDRBQQM3A5CNFSM4KQ6KSSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILRGKRA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2XBA3O2DSOCH5ZOLS3773RBQQM3ANCNFSM4KQ6KSSA.

So i uploaded some test data to a new account, which had previously been uploaded on another account. But the new account wasn't able to update the objects in the new stream

This whole thing of mongo db referencing old objects when creating new similar objects is still being super annoying.. And if this can happen across accounts then I don't know how you're supposed to deal with this.

closing because it belongs more in speckle server


single yaml where game choice is random creates "error decoding response body" error.

What I expected was for the possible games to be listed, based on the rolling of the game dictionary.
What I get instead, is error decoding response body, even where there is ONLY one possible game that can be rolled for the given yaml.
Both of these failed with "error decoding response body"
CaitSith2-Factorio.yaml
CaitSith2-Factorio-Zillion.yaml
And both of these upload successfully, but not at the same time, since the name is not unique across the both of them.
CaitSith2-valid-Factorio.yaml
CaitSith2-valid-Zillion.yaml

Equally invalid, is the single file multi-yaml format., where you separate your archipelago games with ---

Should be fixed by https://github.com/Eijebong/Archipelago-yaml-checker/commit/fc110a1232dd32de10e635bf073892b69dce1e52 thanks for the report :)


Ankit test

Added machine config parameter action symbols ( edit, copy, export, delete) and last edited column

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: chinchila:white_check_mark: FelipeTrost:white_check_mark: OhKai:white_check_mark: LuisaWittig:x: anshi43You have signed the CLA already but the status is still pending? Let us recheck it.

This seems to contain a lot of commits from the other PR which was already merged and now creates merge conflicts. Can you merge the newest main or rebase your changes after main was synced with upstream so that I can take a look at your changes, please?

Closed as per #335


already-handled errors are also "unhandled"

RxJS version:
5.5.8
Code to reproduce:
window.onerror = function() {
  console.log(`onerror event fired anyway`);
};

Rx.Observable
  .of(`something`)
  .observeOn(Rx.Scheduler.async)
  .flatMap(() =>
    Rx.Observable
      .create(
        observer => {
          throw `trantrum`
        }
      )
  )
  .catch(
    error => {
      console.log(`totally handled the error`);
      return Rx.Observable.empty();
    }
  )
  .subscribe();

Expected behavior:
totally handled the error

Actual behavior:
totally handled the error
onerror event fired anyway

Additional information:
This behavior seems to have been introduced in 5.5.0.  Versions < 5.5.0, such as 5.4.3, exhibit the expected behavior.
If the Observable.create() call is replaced with Observable.defer() (that still synchronously throws an error), we get the expected behavior, even with versions > 5.4.3, including 5.5.8.
If this behavior is expected, it seems completely surprising, and very painful to deal with.  Why create observables with supposedly self-contained error-handling logic, and yet have to ALSO deal with the onerror events?

See also #3560.

This is because you're expected to handle your own errors inside of the init function passed to the Observable constructor (aka Observable.create), then hand those errors to the observer.error yourself.
new Observable(() => { throw new Error('test'); });

// vs

new Observable(o => o.error(new Error('test')));

@benlesh, your comment doesn't seem to explain why the error is delivered BOTH to the .catch() to also to the unhandled error callback.  (See my repro case.)
Are you saying that behavior (one error going to two handlers in contradiction with each other; handled vs. unhandled) is by-design?  If so, can you provide the rationale behind that?


Add coverage for all Asciidoctor plugins

We will only support 3.3.x+ with Gradle 7.0 once the remaining deprecations are fixed.

@bot-gradle test smoke

@bot-gradle test smoke

Sorry I don't understand what you said.
Currently I support the following commands:

@bot-gradle test {BuildStage} to trigger a build

e.g. @bot-gradle test SanityCheck plz
SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/ReadyForMerge/ReadyForNightly/ReadyForRelease are supported
test this means test ReadyForMerge
SanityCheck can be abbreviated as SC, ReadyForMerge as RFM, etc.

@bot-gradle help to display this message
@bot-gradle clean to clear the conversation history

Sorry I don't understand what you said.
Currently I support the following commands:

@bot-gradle test {BuildStage} to trigger a build

e.g. @bot-gradle test SanityCheck plz
SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/ReadyForMerge/ReadyForNightly/ReadyForRelease are supported
test this means test ReadyForMerge
SanityCheck can be abbreviated as SC, ReadyForMerge as RFM, etc.

@bot-gradle help to display this message
@bot-gradle clean to clear the conversation history

@bot-gradle test smokeTest please

@bot-gradle test smokeTest please

Sorry I don't understand what you said.
Currently I support the following commands:

@bot-gradle test {BuildStage} to trigger a build

e.g. @bot-gradle test SanityCheck plz
SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/ReadyForMerge/ReadyForNightly/ReadyForRelease are supported
test this means test ReadyForMerge
SanityCheck can be abbreviated as SC, ReadyForMerge as RFM, etc.

@bot-gradle help to display this message
@bot-gradle clean to clear the conversation history

Sorry I don't understand what you said.
Currently I support the following commands:

@bot-gradle test {BuildStage} to trigger a build

e.g. @bot-gradle test SanityCheck plz
SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/ReadyForMerge/ReadyForNightly/ReadyForRelease are supported
test this means test ReadyForMerge
SanityCheck can be abbreviated as SC, ReadyForMerge as RFM, etc.

@bot-gradle help to display this message
@bot-gradle clean to clear the conversation history


Instance overrides

Hello :octocat:
This PR implements instance overriding in the typechecking phase:

On importResolver, when a module is instanced, we copy it's overridden names setting the reference attribute to the value of the override.

At this point, we also check that the overridden definition is a constant (as we don't support overriding anything else at this point)

Value definitions in our lookup table now include a type annotation, so the copied names from the instanced module preserve the type annotations of the original constant declarations.
On the type system, we add a constraint to ensure the value has the same type as the propagated type annotation in the lookup table
On the effect system, the mode checker ensures the value has pure val mode.

@konnov @shonfeder This is actually my original solution, and not the one we agreed upon last Thursday. While trying to implement that solution, I understood better what Shon said about modifying the IR and that was much harder than I anticipated. But that's not the main issue: there's nothing to modify because the instantiated module is not actually expanded into the IR, only at the lookup table level. So, in total, three factors led me to give up on that:

We'd have to modify the IR a lot, and the consequences of that are unclear for me (source maps would be affected, we'd need to generate new unique ids for the nodes, etc)
Modifying the IR even for a small thing is hard
Error reporting would require additional steps to differ errors generated for user-written expressions and generated expressions (as we discussed in the meeting)

I hope the implemented solution is good enough, and that we can adapt the REPL to find the imports/instances from the lookup table. We can still explain instances as copy of the extended module with the const replaced with a pure val, even if that's not exactly what we are doing internally.
Partially addresses #528

[X] Tests added for any new code
[X] Ran npm run format (or had formatting run automatically on all files edited)
[X] Documentation added for any new functionality
[x] Entries added to the respective CHANGELOG.md for any new functionality
[ ] Feature table on README.md updated for any listed functionality

I think this approach should be automatically compatible with the refactored version of REPL as in #621.


Example to show usage of prometheus trait

Solves issue #3144
cc @squakez
I'm new to Apache camel-k and have taken it up as a prerequisite for understanding the integration and usage necessary for the Outreachy project "Camel-k API and examples". Please let me know if there are any other adjustments I can make; I assure I'm a quick learner.

Yes, Andrea is right. See this doc to learn how to use prometheus trait.
https://camel.apache.org/camel-k/next/traits/prometheus.html

@tadayosi @oscerd the endpoints need not to be configured, right?

@SubhasmitaSw you can find some typical examples here:

https://github.com/apache/camel-k/blame/main/docs/modules/traits/pages/affinity.adoc#L57
https://github.com/apache/camel-k/blame/main/docs/modules/traits/pages/route.adoc#L108

The sections marked as auto-generated can be updated by running make generate-doc, which probably does not currently work on Windows, but that should not prevent you from editing the files.
For the Prometheus trait, the metrics endpoint is configured by the trait, so the idea is more to illustrate the purpose of the trait options, like for the pod-monitor option, mentioning that it can be used for the Integration Pod(s) to expose a metrics endpoint, even if the Prometheus operator API is not installed.

I hope this looks fine now
import org.apache.camel.builder.RouteBuilder;

public class ServiceMonitor extends RouteBuilder {
    @Override
    public void configure() throws Exception {
        from("timer:tick")
                .setHeader("example")
                .constant("Hello Camel K!")
                .to("log:info");
    }
}

and to test it if its running i tried to run using
kamel run -t Hello.java prometheus.enabled=true
However, I'm receiving an issue that I don't seem to understand right now.
Error: cannot read sources: cannot read sources: missing file or unsupported scheme in prometheus.enabled=true

That should be kamel run -t prometheus.enabled=true Hello.java :)

I have one more query, the license comment is auto-generated? or just have to copy-paste it?

I think you have to add it manually in the extra example file. That being said, in the Prometheus trait example, there is nothing specific to the integration, so I wonder if that's really necessary to create an extra integration example for that.

I have one more query, the license comment is auto-generated? or just have to copy-paste it?

I think you have to add it manually in the extra example file. That being said, in the Prometheus trait example, there is nothing specific to the integration (as opposed to the JVM trait example that requires specific integrations), so I wonder if that's really necessary to create an extra integration example for that.

Because some other traits give instances of usage, the description stating Prometheus integration may create worries for first-time users. (In my opinion)

Hello. Let me try to clarify what was my idea of the example. The route definition is the first part. After that, my idea was to figure out some use case and show how you can get some stats. As an example, once the build is over, you can describe how to get certain information from the API server: http://localhost:8080/api/v1/namespaces/default/pods/sample-7c66669766-4j89h/proxy/q/metrics (in a minikube environment, before that you must run kubectl proxy --port=8080):
# TYPE application_camel_context_exchanges_completed_total counter
application_camel_context_exchanges_completed_total{camelContext="camel-1"} 31.0

Here for instance, you can let the people understand how you can get stats on the number of exchanges performed by the Integration. This is some basic example, you can figure out any other one with more complex routes. Ideally you need to play a bit and figure it out some useful things to show. As an example, you can refer to one of the E2E test we have to cover the prometheus trait: https://github.com/apache/camel-k/blob/main/e2e/common/traits/prometheus_test.go#L44
Hope that helps.

@squakez as per your suggestions I've tried to demonstrate a use case for the Prometheus trait. Let me know of any improvements or mistakes.

@SubhasmitaSw @squakez FYI maybe some pieces from the following example could be reused, as it's more tailored to demonstrate the Integration monitoring:
https://github.com/astefanutti/camel-k-example-metrics

Let's merge it. It looks good to me.

Thank you!


Reasons to do strict comparison of pytorch version with 2.0.1?

Hi.
In this line
https://github.com/PanQiWei/AutoGPTQ/blob/main/auto_gptq/nn_modules/fused_llama_attn.py#L82
I see the following code
if compare_pytorch_version("v2.0.0", op="eq"):
            attn_output = F.scaled_dot_product_attention(
                query_states,
                key_states,
                value_states,
                attn_mask=None if is_causal else attention_mask,
                is_causal=is_causal
            )
            attn_weights = None

I see this function were introduced in pytorch 2.0.0, but since we have 2.0.1 already - maybe it's better for it to be something like if compare_pytorch_version("v2.0.0", op="ge")?

Sorry, wrong title. 2.0.0 of course

Sorry about that, this is actually a typo, the op should be "gte"(greater than or equal). Will fix it in a patch release. However fused_llama_attn.py will be removed in 0.5.0, and refactor the fused module injection logics in llama.py without this part of code.

I see it even fixed already. Well, returning to my stuff than.
Thanks, @PanQiWei


Update CONTRIBUTING.rst with translations

POT are generated automatically by the bot, so no need to do it. Also this redaction is not correct. You are allowed to modify translations when migrating a module. The key is that the module doesn't exist yet on the repo, and thus, there's no translation yet on Transifex that can provoke conflict if it's updated meanwhile the PR lifecycle.

You are allowed to modify translations when migrating a module. The key is that the module doesn't exist yet on the repo, and thus, there's no translation yet on Transifex that can provoke conflict if it's updated meanwhile the PR lifecycle.

Yes I noticed that too but I think that's why it says

Pull requests should never directly modify .po files

but it allows PRs that create new .po files that is what happens in migration PRs and PRs that add new modules in general.
Maybe a little clarification might be added stating something like

Pull requests creating new modules can add their own .po* files

Yes, good point.

Extended a bit. Thanks!


Is there a working demonstration or a 'getting started'?

Thank you for this library!
I have attempted to add this into my existing angular-schema-form project, by including it as follows:
<script src="/lib/angular-schema-form-multiselect/dist/asf-bootstrap-multiselect.min.js"></script>
In the console I see the following error stack:
ReferenceError: stdFormObj is not defined
    at Array.selectWithDefault (asf-bootstrap-multiselect.js:160)
    at a (schema-form.min.js:1)
    at schema-form.min.js:1
    at Object.r [as forEach] (angular.js:331)
    at Object.$get.t.defaults (schema-form.min.js:1)
    at Object.$get.t.merge (schema-form.min.js:1)
    at link.h (schema-form.min.js:1)
    at Object.<anonymous> (schema-form.min.js:1)
    at n.$get.n.$digest (angular.js:14300)
    at n.$get.n.$apply (angular.js:14571)

So essentially in asf-bootstrap-multiselect.js, the stdFormObj is undefined in the angular.module('schemaForm').config(... call.
I'd be most grateful for some pointers on what I'm doing wrong in how I'm including this. Then it would be useful to know what to modify in the schema and/or form to produce a multiple select.

Hi,
I had the same issue and fixed it whit two changes in asf-bootstrap-multiselect.js
First, change:
var f = stdFormObj(name, schema, options);
For:
var f = schemaFormProvider.stdFormObj(name, schema, options);
Second, change:
f.titleMap = enumToTitleMap(schema.items['enum']);
For:
f.titleMap = [];
schema.items['enum'].forEach(function(name) {
f.titleMap.push({name: name, value: name});
});
And
f.titleMap = enumToTitleMap(schema['enum']);
For:
f.titleMap = [];
schema['enum'].forEach(function(name) {
f.titleMap.push({name: name, value: name});
});

Thanks for this, I just ran into the same problem. Worth submitting a PR with this fix?

Done :)

Wow, that was quick ;-)


Cannot load a custom prefab component from Globals?

I was trying to load a prefab object with prefab tag in Unity. First, I tried to load from resources with Interop, following docs, but Unity crashes with error End on file. Then, I tried to create a custom prefab component dame as the sample. I added it in Globals like a C# script but not works. If I type the extracted object from Globals, an error type raise.

Update: When I load the component in Globals object. It raises this error:
Unhandled promise rejection: InternalError: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.NullReferenceException: Object reference not set to an instance of an object
  at ReactUnity.UGUI.UGUIComponent.UpdateOrder (System.Int32 prev, System.Int32 current) [0x0019b] in ./Library/PackageCache/com.reactunity.core@0.19.9/Runtime/Frameworks/UGUI/Components/UGUIComponent.cs:651

In that file, if the null verification is carried out, it does not show canvas, only UGUI game objects.

Update: It is possible to set the script during the execution and custom prefab component works. Also, if I change the route (I am using ReactRouter) after the global setting, and back to route that renders prefab component, raises the same error.

I couldn't picture the problem in my head. Can you share part of the code you are having problem with? I will try to check what is wrong during the weekend.

Okey here is my code. This is my C# script:
public class MainStar : MonoBehaviour, IPrefabTarget
{
    PrefabComponent Component { get; set; }
    GameObject instance;
    public float rotationSpeed = 3f;
    public Material customMaterial;
    public GameObject prefab;

    public void Mount(PrefabComponent cmp)
    {

        Component = cmp;
        if (prefab != null)
        {
            instance = Instantiate(prefab, Vector3.zero, Quaternion.identity);
            instance.transform.SetParent(cmp.RectTransform, false);
            instance.transform.localScale = new Vector3(250f, 250f, 250f);

            RectTransform rectTransform = instance.GetComponent<RectTransform>();
            if (rectTransform != null)
            {
                rectTransform.anchorMin = new Vector2(0.5f, 0.5f); // Centro
                rectTransform.anchorMax = new Vector2(0.5f, 0.5f); // Centro
                rectTransform.anchoredPosition = Vector2.zero;
            }

            Renderer renderer = instance.GetComponent<Renderer>();
            if (renderer != null)
            {
                if (customMaterial != null)
                {
                    renderer.material = customMaterial;
                    Debug.Log("Material assigned successfully");
                }
            }
            else
            {
                Debug.LogError("No Renderer found on the prefab");
            }
        }
        else
        {
            Debug.LogError("Failed to load prefab");
        }
    }

    public bool SetProperty(string propertyName, object value)
    {
        return false;
    }

    public Action AddEventListener(string eventName, Callback callback)
    {
        return null;
    }

    public void Unmount(PrefabComponent cmp)
    {
        Debug.Log("Unmounting Main Star component");
        Component = null;
    }

    void Update()
    {
        if (instance != null)
        {
            Vector3 inclinedAxis = new Vector3(0.3f, 1f, 0f).normalized;
            instance.transform.Rotate(inclinedAxis, rotationSpeed * Time.deltaTime);
        }

    }

I set its values in UnityEditor:

This is my component in React. I will only put what is necessary:
export default function MainMenu() {
  const { Star } = useGlobals();
  return (
    <view className="flex flex-col flex-auto gap-10 portrait:gap-20">
        <prefab
          className="basis-52 self-center"
          target={Star}
        />
    </view>
  );
}

Finally, the error is raised and the game object hierarchy looks like this.

I tried with the sample component showed in the page but the result is the same.

You shouldn't assign the React Canvas, but the whole GameObject as the prefab. Can you try that?

Okey, I modified the script:
public class MainStar : MonoBehaviour, IPrefabTarget
{
    PrefabComponent Component { get; set; }

    public float rotationSpeed = 3f;

    public void Mount(PrefabComponent cmp)
    {
        Component = cmp;

    }

    public bool SetProperty(string propertyName, object value)
    {
        return false;
    }

    public Action AddEventListener(string eventName, Callback callback)
    {
        return null;
    }

    public void Unmount(PrefabComponent cmp)
    {
        Component = null;
    }

    void Update()
    {
        Vector3 inclinedAxis = new Vector3(0.3f, 1f, 0f).normalized;
        transform.Rotate(inclinedAxis, rotationSpeed * Time.deltaTime);

    }
}

I instantiated star prefab and I applied the script:

Then I added to Globals:

When I start the game, Unity crashes with End of file error in log.
TcpMessagingSession - receive error
AssetImportWorkerClient::OnTransportError - code=2 error=End of file

I don't know that error, but I see there is a misunderstanding. prefab doesn't render 3d objects in a 2d UI. It cannot. For that, you may need to set-up another canvas and render it with <render> tag. It is kinda an advanced use case but there is an example of it here.

Okey I understand, I will try to use <render> tag. But, as a last doubt, if I set the globals while the game is running using first way, the object rendered correctly and I am a little confused about that.

@gusCreator <prefab> tag expects a game object with a RectTransform. In your first way, you are adding a Transform to RectTransform as a child. Your way may work for you, and that is ok.

Okey, thanks for response!


refactor menu theming to be cleaner and remove css colors

fixes #58496 and https://github.com/Microsoft/monaco-editor/issues/1013

@sbatten please clean up the merge conflicts and bring to latest in master

@sbatten when you do these changes please cross check with the standalone editor as well that things are fine.

@bpasero  don't think that one is supposed to be there anymore, removed it


Using with Ionic / Capacitor not working

Hi guys, i'm trying to use this amazing lib in my Ionic Capacitor Android App with React, everything works very well running in the browser (ionic serve), the oidc data are saved in the localstorage and the hooks/effects does their updates, but when i try to use in a built apk, after i login and redirect to the app, nothing happens with the hooks/effects, the isAuthenticated still false, none of information is saved in the localstorage, my redirect uri have all oidc data as a query (state, nonce, etc...)

I'm using userStore: new WebStorageStateStore({ store: window.localStorage }),  in my oidcConfig.
My redirect URI is right for the browser (http://) and in the native app (my.deeplink.io://)

Its possible to use under the hood or i need to make some specific changes to get this work?

Hi!  I was going to try this library with capacitor now, where you able to circumvent the problem?


feat: Implemented Endpoint to add a new user to the Waitlist

Description

The waitlist endpoint was designed to manage user registrations and store user details for the product until it gets launched. This feature allows users to reserve their spot in line for future updates or access to the boilerplate. Key aspects of the implementation include:

Users can sign up by providing their email address and full name
There is validation to check for duplicate emails or if the name field is null
A confirmation email is sent to the email address provided upon successful sign-up
To protect against abuse, the waitlist endpoint incorporates rate-limiting

Related Issue (Link to issue ticket)
WAITLIST API-BACKEND

Motivation and Context

This ensures that users on the waitlist can have first-hand information about new features and when the product finally launches.

How Has This Been Tested?

I have tested locally using both Postman and unit tests that I wrote

Screenshots (if appropriate - Postman, etc):
Success Response

Error Response

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation as needed.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.

This is an old PR and can be closed. Someone already worked on this feature using the newly appointed spreadsheet method.On 27 Jul 2024, at 12:25, Adegbehingbe, Oluwakorede Joseph @.***> wrote:
@joboy-dev requested changes on this pull request.
Make changes
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Add code example guidance to style guide

Description
Adds code example guidance to STYLE_GUIDE.md.
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vagimeli @kolchfa-aws Can you please review for accuracy and then approve? Thanks!


problem with start_params

in this url for example : https://t.me/mybot?startapp=bbId38341612
when open it with telegram desktop, WebApp.initDataUnsafe.start_param working fine and get bbId38341612
but when I want open that with iphone 13 , always WebApp.initDataUnsafe.start_param  is undefined, my project on react.js
and     "@twa-dev/sdk": "^7.0.0",
help me to fix that

Seems like Telegram bug, not this package.

in this url for example : https://t.me/mybot?startapp=bbId38341612
when open it with telegram desktop, WebApp.initDataUnsafe.start_param working fine and get bbId38341612
but when I want open that with iphone 13 , always WebApp.initDataUnsafe.start_param is undefined, my project on react.js
and "@twa-dev/sdk": "^7.0.0", help me to fix that

Same issue here. did you find any fixes or workarounds?

Had similar issue, got fixed by switching from https://t.me/bot/?startapp=key to https://t.me/bot/referrer?startapp=key


update stack specs with equivalent constraints

ML stack is the only one that has any constraints on CPU or Mem.
Depends on #760

I think CLI might use these for locally deploying a stack

Took a peak and couldn't find anywhere that they are used. However, it is defined in an example here


Bumping tf version in docker

Fixes #163
Just bumping to use 0.13.4

LGTM but what's with the wired commit history?

LGTM but what's with the wired commit history?

Good question 😂  I need to rebase my local fork


Search Broken

Search appears to be broken in Raindrop.io at the moment.
I have tried via an Alfred Workflow, which would typically list all matching results to my search query, but currently just offers to send me to Google - which is the fallback if no matching results are found. I've also tried via logging into the raindrop.io web app (in Firefox) and anything I type into the search field results in an Error page, with the image of a cat inside a TV. Tried the same search in Safari with the same result.

Search context was "All bookmarks" and I have confirmed that all things I searched for do still exist in my bookmarks when I go through all bookmarks manually.
Bookmarks that I have searched for today were definitely working yesterday.
If there's anything else I can provide which might help, please let me know.

Having the same issue on app.raindrop.io and Android app.
I would say the issue might started appearing last 5-7 hours, before then I had no problem like this.
Hopefully be fixed soon.
Thank devs for this great service.

Disaster happen with our search index. We hosted our search engine on @elastic cloud. They just deleted our entire account.
Do not worry, all data is safe. I just need to rebuild search index on a different hosting provider. But this require few days to complete unfortunately

Got it @exentrich. No big deal. Thanks for quick response.

Disaster happen with our search index. We hosted our search engine on @elastic cloud. They just deleted our entire account.
Do not worry, all data is safe. I just need to rebuild search index on a different hosting provider. But this require few days to complete unfortunately

wow, that is douchebag of them. What happened? why do they delete search index account like this??

@exentrich wow! I hope everything is back on track soon

Elasticsearch reserves the right, in its sole discretion, to deny, restrict, suspend, discontinue, or terminate your account, with or without prior notice or explanation, for any or no reason, without any liability to you
https://www.elastic.co/legal/terms-of-use

Warning to anyone planning to use Elastic Cloud: They can terminate your account without prior notice for any reason. This is what happened to our account.
Despite multiple emails, they ignored us. Only one team member on Twitter responded, stating that they blocked my account because they mistakenly thought I was from Russia. However, I am a citizen of Kazakhstan and conduct business here. I hope they will contact me to provide a further explanation.

Yeah, it a lot seems it was just their accident, if that appears to be factual then I believe Elastic must cover whatever costs Raindrop had to spend on this, hopefully they simply admit it.
As long as Electric doesn't bring a vivid logic on whatever their member on X quoted "sanctions" or "regulatory restrictions"--by who though--, they now might be taking their time to make up deal and strategy to this.
Well not that I want things escalate though. My best wishes to Raindrop.io on this.

I am still having problems when searching bookmarks from the browser url bar, is the search index not fully rebuilt yet?


[R2DBC] The state of nested transactions for different databases is wrong.

Describe the bug
When using R2DBC and TransactionAttribute.REQUIRED, the state of nested transactions for different databases is wrong.
The code is as follows.
    fooDb.withTransaction(TransactionAttribute.REQUIRED) {
        // OK
        val newFoo = fooDb.runQuery {
            QueryDsl.insert(Meta.foo).single(Foo(name = "hoge"))
        }
        barDb.withTransaction(TransactionAttribute.REQUIRED) {
            // NG
            val newBar = barDb.runQuery {
                QueryDsl.insert(Meta.bar).single(Bar(name = "fuga"))
            }
        }
    }

Expected behavior
I expect the SQL to be executed against each database without fail.
Actual behavior
It seems that SQL is being issued against an unexpected database.
Steps to reproduce the problem
I have a project to reproduce on my branch.
See here.
https://github.com/yu-ishicawa/komapper-examples/tree/research-transaction
Versions

Komapper: 1.11.0
Kotlin: 1.8.22
JVM: 17.0.2
R2DBC Driver: 1.0.0-RELEASE (?)
Database: h2database 2.1.214
OS: MacOS Ventura 13.4

Thanks for the report. We will investigate.

@nakamura-to
Thank you. I have confirmed that the problem has been fixed.


The Global Admin user in B2C

[Enter feedback here]
Hello,
So after we configured all the steps, and see the logs in the log analytics workspace, can we revoke the Global Administrator access from the user? or even remove it from the Security group? Or should we create a dedicated user and permanently leave it in the security groups?
Clarification is appreciated,
R

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

ID: 24e34c89-9a70-b5b2-3b38-bc81660f83c1
Version Independent ID: 77fa8d3c-56dd-294f-13ae-ac6ba44018a7
Content: Monitor Azure AD B2C with Azure Monitor - Azure AD B2C
Content Source: articles/active-directory-b2c/azure-monitor.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@zaalion
Thanks for your feedback! We will investigate and update as appropriate.

The recommendation in the article is to use the Global Admin for the tenant. But it is only required for the initial configuration and after you've configured the tenant to send logs to Azure Monitor, you'll be able to access those without needing to be a Global Admin. https://docs.microsoft.com/en-us/azure/azure-monitor/roles-permissions-security
There is also a note explaining this in the regular Azure AD configuration article.
CC @kengaderdus


Broken on node 0.10.x (@v1.3.3)

babel-polyfill exists package.json as dependencies.
But, it is never imported from production code.
I think that it can solve by adding require('babel-polyfill') to src/bin/npm-run-all.js and index.js.
What do you think?
Refs:

Job #119.1 - moqada/aws-sdk-config-loader - Travis CI
Polyfill · Babel

Oh, thank you for the issue.
I'm wondering why my Travis CI is green.
I will fix this.

Hmm, I'd like to use babel-runtime and babel-plugin-transform-runtime because I want to keep clean global scope, but I'm encountering with https://phabricator.babeljs.io/T6644

For now, I add import "babel-polyfill"; to bin/npm-run-all.js, and I confirm it works on Node.js v0.10.41.
I published it as v1.3.4.
Please try it.

As you said, it looks more better if it can use babel-runtime...
But, v1.3.4 works fine!
Thanks!!
Refs:

Job #126.1 - moqada/aws-sdk-config-loader - Travis CI


Fix unit tests in Identity-apps to match new ui changes

Describe the issue:
With the new UI improvements, unit tests are failing.
How to reproduce:

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

We have enabled unit tests in the PR builder. Hence closing this issue as completed.


Bad GPX file can cause issues rendering

My suspicion is that the below XML is too long or something like that
https://pastebin.com/dPT7Aziy

I've seen this before when pausing a run on my GPS watch then resuming hours later. If you look at the tool output stats for this file it shows a duration of just over 168 hours (which happens to be exactly 1 week if that means anything to you).
So the tool is FAD (functioning as designed), it's just forced to render everything before the pause in a single frame and everything after in another frame.
I'm going to leave this issue open because I've been thinking about adding a feature to break apart activities with long pauses like this.


Changing CompositorEffect array in the editor while the game is running gives Cannot asign contents of array[Object]

Tested versions

Reproducible in 4.3.beta2
And v4.3.beta.gh-93876 [3ccb371a2] (from github action build)

System information
Windows 10 - Vulkan (forward+) - dedicated
Issue description
Not anything critical, but
if your game is running and you try to change the Compositor array, Godot will spit out an error saying.
E 0:03:29:0436   Array::assign: Cannot assign contents of "Array[Object]" to "Array[Object]".
  <C++ Error>    Method/function failed.
  <C++ Source>   core\variant\array.cpp:245 @ Array::assign()

And won't update the running game.
https://github.com/godotengine/godot/assets/3084189/8f1eb03d-06ba-429c-a72e-1e866e2032c8
Steps to reproduce

Run the game.
Change the environment Compositor array (either add or remove something)

Result: Error in output. No crash, but no change either.
Expected: Running game updated with the new Compositor values like it does for other things.
Minimal reproduction project (MRP)
There is really no need. But here's a project with an empty scene and an environment node.
compositor-error.zip

Is the conclusion that the changes in #93876 caused this bug or did it exist in a previous version? I took a quick look at the bug and couldn't figure out how they would be connected.

No relation at all.
That second 'Tested version' I put there with this merge was just the latest dev build that I tested at the time.
Just to make sure it was not already fixed on dev at the time. No relation with this bug at all.
I think this error always existed since CompositorEffects was implemented.
Syncing scene changes doesn't work with the compositor. This is not a very common use case so I guess it never got tested.
Maybe this is even intended and you might not be able to change the rendering hooks after the scene has started.

Still present in v4.3.stable.official [77dcf97d8]. Happens when I either add a new compositor effect or change the order of compositor effects.


Update maintain-the-website guide to include info about adding links to Live Stream archives

I have added some info about adding the links to livestreams and included an image.

Thank you @kfahn22!


referencing newer EPPlus

Using EPPlus 4.0.5 and EPPlusEnumerable 1.4.0 (retrieved from NuGet) I'm getting the error message:  "Could not load file or assembly 'EPPlus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=ea159fdaa78159a1' or one of its dependencies. The system cannot find the file specified."
I think the fix is to set "Specific Version" to false in your EPPlus reference.

Good catch. I just updated the NuGet package so it should now just work with 4.0 and up.


updated false to boolean in ListItem for it work with value true in typescript

[x] I have followed (at least) the PR section of the contributing guide.

The change breaks some of our tests, could you have a look at it? Do you have a failing test case we should consider that the change aim to fix?

@dilipvamsi Do you think that you will have the time to look at it?

@dilipvamsi I'm closing, for now. Let us know if you find a way forward :)


[stable/sonatype-nexus]: Do not set targetPort for ClusterIP

When deploying stable/sonatype-nexus with a service type of ClusterIP the targetPort must be omitted or the deploy will fail.

/ok-to-test

cc @rjkernick

Looks good to me

/lgtm


[Question]: --remoteOllamaServer Is it a URL or an IP?

What is your question?
The --help says:
--remoteOllamaServer REMOTEOLLAMASERVER
                        The URL of the remote ollamaserver to use. ONLY USE
                        THIS if you are using a local ollama server in an non-
                        default location or port

My Ollama server is under a long security string like this:
https:// ${host} / ${big long string} /ollama/api/generate
What should I use as remoteOllamaServer because I tried almost every combination and it doesn't show the ollama models with --listmodels.
Thank you

Try to just specify the host and port for the Ollama API. This is what works for me
--remoteOllamaServer http://<IP_ADDRESS>:11434

Try to just specify the host and port for the Ollama API. This is what works for me --remoteOllamaServer http://<IP_ADDRESS>:11434
11434 is the default port by Ollama

yes but that assumes your Ollama server's API is under http://<IP_ADDRESS>:11434/api/generate and not under vhost path: http://<IP_ADDRESS>:11434/abcd1234abcd1234abcd1234/ollama/api/generate

@chka Should Fabric support the API path param as well? I'm on the same boat in regards to the API servers have generated paths... For instance, deploying multiple GenAI API Servers similar to Ollama and maintaining the server static, but at a different path...


Containerize your microservice using Docker

Title: Containerize microservice using Docker
As a developer
I need to containerize my microservice using Docker
So that I can deploy it easily with all of its dependencies
Details and Assumptions

Create a Dockerfile for repeatable builds
Use a Python:3.9-slim image as the base
It must install all of the Python requirements
It should not run as root
It should use the gunicorn wsgi server as an entry point

Acceptance Criteria
Given the Docker image named accounts has been created
When I use `docker run accounts`
Then I should see the accounts service running in Docker

merged


[Feature Request] Function Calling in OpenAI API

🚀 Feature
Add ability to pass function to model on supported LLMs
Motivation
With the ChatGPT API you can pass a list of functions to the model, and some models that can be self-hosted require this to function properly, for example: https://huggingface.co/gorilla-llm/gorilla-openfunctions-v1
This would allowe the functions parameter to be used with the models that support them allowing for code calling to work.
Additional context
import openai

def get_gorilla_response(prompt="Call me an Uber ride type \"Plus\" in Berkeley at zipcode 94704 in 10 minutes",
                         model="gorilla-openfunctions-v0", functions=[]):
    openai.api_key = "EMPTY"
    openai.api_base = "http://<IP_ADDRESS>:8000/v1"
    try:
        completion = openai.ChatCompletion.create(
            model="gorilla-openfunctions-v1-q4f32_0",
            temperature=0.0,
            messages=[{"role": "user", "content": prompt}],
            functions=functions,
        )
        return completion.choices[0].message.content
    except Exception as e:
        print(e, model, prompt)

query = "Call me an Uber ride type \"Plus\" in Berkeley at zipcode 94704 in 10 minutes"
functions = [
    {
        "name": "Uber Carpool",
        "api_name": "uber.ride",
        "description": "Find suitable ride for customers given the location, type of ride, and the amount of time the customer is willing to wait as parameters",
        "parameters": [{"name": "loc", "description": "location of the starting place of the uber ride"},
                       {"name": "type", "enum": ["plus", "comfort", "black"],
                        "description": "types of uber ride user is ordering"},
                       {"name": "time", "description": "the amount of time in minutes the customer is willing to wait"}]
    }
]

for i in range(10):
    resp = get_gorilla_response(query, functions=functions)
    print(resp)
print("Done!")

This is example code that would use the feature shown above. Currently the endpoint listed in the code is a MLC chat rest endpoint with the gorilla model, but because the model is not actually getting the functions that it can call, it is producing bad output, effectively making gorilla not work with MLC even though it functions and does try to return a function call: call_me_uber_ride_type("Plus", Berkeley, zipcode=94704, duration=10)

@t-dettling Thank you for bringing this up. We will look into add function calling support to rest API for MLC.

Thanks to @Kartik14, we have the PR #1378 which brings the function call supports. We will upload prebuilt gorilla model weights to HF after the PR getting merged.


Clarify edge direction for GraphMap

Clarify the direction of edges created by the add_edge function.

Hm this does not seem so clear. In an undirected graph, the edges don't have a direction, so I'm not sure what this explanation says. It should say that for an undirected edge, the edge connects a and b.

Duh, of course! It seems i documented edge_key rather than this function. Sorry!

Thanks. Sorry for the delay, indeed.


Path-weighted metrics?

Hi,
Very cool work! I was wondering if you have the path-length-weighted metrics for your model and the baselines in your few-shot table? Thank you!

Hi, here are PLW SR for our methods:
Goal instructions only
LLM-Planner + HLSM
Test Unseen: 3.67
Valid Unseen: 3.08
Step-by-step instructions
LLM-Planner + HLSM
Test Unseen: 3.89
Valid Unseen: 4.11
HLSM:
Test Unseen: 0.09
Valid Unseen: 0.00
FILM:
Test Unseen: 0.04
Valid Unseen: 0.00
Note that PWL SR depends significantly on your choice of the low-level planner, rather than on the high-level planner (e.g. LLM-Planner).

Thank you so much!


Improve Docker Desktop filesharing docs

Proposed changes

I updated the Docker Desktop for Mac filesharing documentation to add the same tips as Windows, now that the implementation is the same on Mac and Windows. I wonder if we should combine Mac and Windows into a "Desktop" section at some point, since more and more of the UX is the same (e.g. the dashboard)
I've also highlighted that  docker volumes are better on Desktop for

raw performance
Linux filesystem semantics (needed for things like databases)

and so should be used in preference to bind mounts for particular use-cases.
I added some basic text about compose to the general storage section too.
Unreleased project version (optional)

Related issues (optional)

Thanks!


Simply add a new Deep Blue theme

Non of the existing themes are really that dark. I figured I would add
one which is darker and has a bit more flair. Takes some inspiration
from Abyss

Fixes
Nothing fixed here as it is an enhancement
Checks

[x] Ran yarn test-build
[x] Updated relevant documentations

Changes proposed in this pull request:

Add a new theme option called deep_blue
Accessible through Settings > Theme > deep_blue

@svenjr Thanks for creating a new theme for Altair. However I wouldn't be merging these changes. You should be able to create a theme plugin for this case which can then be installed by users. That way, we can move these from Altair itself while making it optional. You can read more about writing a plugin here. The important methods in the API for you would be ctx.theme.add(themeName, theme) and ctx.theme.enable(themeName).

Closing this now


Updated CI to use bazelisk and install bazel into /usr/local/bin.

BUG=fixes #1852

0a5c414 (#1853) is incorrect. The issue is that the image on GHCR needs updating.


GH - The image extraction screen is white without image

What
The image extraction screen is white without image, we should instead show for example a alert asking to submit a image

Image

I would like to work on this as a part of Grace Hopper Celebration contributor. Can you please guide me where the file is located as I am new to this project.

Heyy @Puja-Kumari-909 thats great to hear!
The page you are looking for is the EditOcrPage which can be found under ./packages/smooth_app/lib/pages/product/edit_ingredients_page.dart.

Does this seem fine to you?

Looks good @Puja-Kumari-909 but you could change "No image" to something like "Upload a image to automatically extract the provided information."


Fix PKGBUILD for Arch Linux

Some improvements where mentioned by MarsSeed for the AUR PKGBUILD:

Pls. add depends=sh and link to a specific source commit (this should not be a rolling VCS package).

Those should be done, including creating a Release policy(?) for Protonlaunch.

Added a specific commit for 0.10 and sh as depedencies on dev commit 2bcc2e3bcaa9bc45599b45930e934af0637c251f
Now it needs to be uploaded to AUR


USome(null) doesn't pattern match

It looks like, by design, an extractor has no means to match a null value. Since USome(null) is represented as such, we can match it.
Solutions

Wrap null values in USome
Don't support null values in USome

similar to lourkeur/scape#2

Oddly enough, they can match in 2.12...

That's on purpose.


Sketch App wants me to move it to trash

i uploaded it and ran the plug in but when i went to work with it error prompts tell me that MacOS can't verify developer and can only trash the plugin.

Just ran into this issue with a friend today. We got around it buy got to **System Preferences > Security and Privacy > General ** then telling the system to allow the SMFramework. Then rerun your Spec Export and tell the system to Open. That should get you past it.
After getting through that though we still did have an issue with export hanging. We got around that by using this forked and updated version from @qjebbs
Hope that helps.

I can very Ronnin solution worked!


Vickel: Watch illuminatisocietywealth(?:@gmail.com)?

Vickel requests the watch of the watch_keyword illuminatisocietywealth(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

illuminatisocietywealth(?:@gmail\.com)? has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by micsthepick in Charcoal HQ


functionapp: support always ready and prewarmed instances for premium plan

See related issue: https://github.com/Azure/azure-functions-core-tools/issues/2059

route to service team

this command uses & depends on SDK to be updated with new properties in swagger. Once the feature team takes care of updating swagger & new SDK is generated , CLI can use the new SDK. Once this is done, the command has a --set parameter that can be used to update any parameter. Is the ask for these properties to be add as first class parameters? Is this still needed?

CLI has been updated with new SDK a while back & hence --settings can now be used to support the new porperties.


Add introspection client

New feature

Add introspection client for debugging

Command line app
Shows memory usage
Shows connected process
Shows ports

Closed by #22


Example of output

Hi,
Is it possible to see what kind of output your model has produced?
I have implemented some other repos looking at the same issue and it seems that output doesn't seem to output audio files that are easy to understand.
Do you have a couple of inputs and the outputs that you are willing to share?
Thanks

The input noisy audio (1D) is transformed by STFT into a time-frequency map (2D).
The network inputs the TF map and outputs a masking TF map.
We multiply the input noise map with the mask map generated by the network to obtain a clean map (2D).
Finally, clean map is converted to the audio signal (1D) by iSTFT .


Mangas Erro

We can't download purshased Mangas. The script don't recognize them, when they are only available in Volume.

fixed https://github.com/Elastic1/piccoma-downloader/commit/81dff8e2653f01c2ef4a86c06bb15eb80f6ff6d6


Remove unused skip_except

It's unused & untested. Why keep it?

Yes, thanks.  Please merge when buildbot is green.


Email addresses pattern content update

What is the context of this PR?
Content design update to documentation
How to review
Check docs

 Update email addresses documentation

one tiny thing. comma needs to be removed under Explain why you need the email address:
"... to reassure users why you need their email address and what it will be used for."


file I/O-operations

By now in the Lab there are no commands to do file-input or -output Operations.
This would be very usefull to do Data acquisition with the EV3.
I want to use it do send a paramter.rtf-File to the EV3 and it will be interpreted as commands+parameter from the Brick.

won't fix


Can zoom into picture be linked for the complete picture?

Hello SAP Help Team,
I file this issue here as I've faced it when browsing the page SAP Cloud Transport Management Service. On this page there are several screenshots. There is a little magnification glas with a + in the upper right corner of such screenshots. To see the full size one has to click exactly there:

I would hope this can be changed and the whole tumbnail can be made clickable. Also it would be great if one can simply switch between all provided screenshots. Just implement it like it is in the SAP Community Blogs: SAP Cloud ALM Activation.
Looking forward for this usability improvement.
Best Regards
Gregor

Hi @gregorwolf, Thanks a lot for the feedback.
Your request definitely makes sense and would be nice to have... but the current behavior is the default in SAP Help Portal. I checked the options to customize this, there's no way at the moment. Let me check with the experts of the infrastructure. The we can get back to you.
Best regards, Steffi

https://people.sap.com/gregorw


Added GATT Boot Keyboard Output Report Characteristic

Issue
#58.
What does this PR Do?
Added GATT Boot Keyboard Output Report Characteristic
Where should the reviewer start?
GATTBootKeyboardOutputReport.swift

Result of Integration 1
Duration: 21 minutes and 22 seconds
Result: Perfect build! All 170 tests passed. :+1:
Test Coverage: 86%

Result of Integration 1
Duration: 21 minutes and 22 seconds
Result: Perfect build! All 170 tests passed. :+1:
Test Coverage: 86%

Result of Integration 1
Duration: 21 minutes and 22 seconds
Result: Perfect build! All 170 tests passed. :+1:
Test Coverage: 86%

Result of Integration 2
Duration: 9 minutes and 16 seconds
Result: Perfect build! All 172 tests passed. :+1:
Test Coverage: 86%

Result of Integration 3
Duration: 9 minutes and 27 seconds
Result: Perfect build! All 173 tests passed. :+1:
Test Coverage: 86%

Result of Integration 3
Duration: 9 minutes and 27 seconds
Result: Perfect build! All 173 tests passed. :+1:
Test Coverage: 86%

Result of Integration 3
Duration: 9 minutes and 27 seconds
Result: Perfect build! All 173 tests passed. :+1:
Test Coverage: 86%

Result of Integration 4
Duration: 8 minutes and 30 seconds
Result: Perfect build! All 173 tests passed. :+1:
Test Coverage: 86%

Result of Integration 5
Duration: 8 minutes and 39 seconds
Result: Perfect build! All 174 tests passed. :+1:
Test Coverage: 86%

Result of Integration 6
Duration: 8 minutes and 28 seconds
Result: Perfect build! All 174 tests passed. :+1:
Test Coverage: 86%


Automated Package Upgrade

:tada: This PR is included in version 26.0.112 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Translate function parameter formality

Is it possible to add the parameter "formality" to the translate function?

formality	Optional	Sets whether the translated text should lean towards formal or informal language. This feature currently only works for target languages "DE" (German), "FR" (French), "IT" (Italian), "ES" (Spanish), "NL" (Dutch), "PL" (Polish), "PT-PT", "PT-BR" (Portuguese) and "RU" (Russian).Possible options are:

"default" (default)
"more" - for a more formal language
"less" - for a more informal language

Hi Paul,
sorry for the taking so long to answer. I am working on adding the formality parameter. I'll let you know when I am finished.

Hi Paul,
I just merged the formality parameter into the main branch. I will publish an updated NuGet package in a couple of minutes. Please let me know if everything works for you.

Thank you very much David. Everything worked.

I am happy to hear that


changed/fixed Time and TimeSet

I changed Time to fit our new change where times can only be on one day.
updated tests and otherwise along with this change.
Took out get TBA from Time because it wasn't working as planned, instead just all time() (updated in docs)
Added a method called Time.isTBA() to return whether a Time is an empty Time object.

:+1:


Audit new entry problem with foreign key

I just tried the Audit feature and it seems to be exactly what we need. However, I stumbled upon an issue.
Lets say that I have a parent child relationship.
Parent:
int:Id (Identity specification)
Child:
int:ParentId (FK)
When I create new parent and a new child associated with it, the newValue of ParentId (Child) in AuditProperties is set to 0 instead of the actual value of the newly created Parent. If I check the newValue of Id (Parent) it is set to 18. I guess there is an issue related to the id being an identity specification. Any way to solve it?

Hello Compusa,
We successfully reproduced the issue.
It seems our Audit library work with navigation properties but not when the ParentId is directly the value.
We will investigate more the issue this weekend and try to release a fix for Monday.
Best Regards,
Jonathan

Thanks, for the fast response and a fix for Monday will be very appreciated.

Hello @Compusa,
Unfortunately, it will take a few more days before releasing the fix.
We tried two solutions, but both failed (causing side effect).
We took some time to think about a third solution (this time working!). Unfortunately, we will not have the time to implement it tomorrow due to having meeting all the days. So we will try on Wednesday.
Sorry for the delay,
Best Regards,
Jonathan

Ok @zzzprojects. Your feedback is highly appreciated and I'm looking forward to the release of the fix.

Hello @Compusa ,
The v1.4.18 has been released which should fix this issue.
Can you try it and let us know if it's fixed?
Best Regards,
Jonathan

Hello @zzzprojects
Sorry for the late feedback. I have been busy with the release of our product and I didn't have any time to look at this until now. I have verified it now and the bug fix solved the problem. Thank you very much! :)


Inconsistent values for IPA_OUTPUT_PATH and DSYM_OUTPUT_PATH

I'm working on building a custom action and I'm running into a case where it's getting a relative path to the ipa and an absolute path to the dsym.
:IPA_OUTPUT_PATH=>"../APP_NAME.ipa",
:DSYM_OUTPUT_PATH=>"/Users/Shared/Jenkins/Home/jobs/TestFlight/workspace/APP_NAME.app.dSYM.zip"

It's also weird that the ipa path is relative to the fastlane folder, not the root folder of my project. I tried expanding it using File.expand_path(ipa_path) but that doesn't work since it expands relative to the root folder of the project.

I'll update it to store only absolute paths, thanks for letting me know!


test: include all of the necessary CSS in unit tests

Includes the missing theme CSS in the unit tests and removes some of the styles that were used to simulate it.
Fixes #2358.

LGTM


Many-to-many relationship with custom key is not possible

Hi,
just tried to add a many-to-many relationship, where one type has a custom key defined.
The error tells, that the key must be of type ID.
Example from the docs (modified to have a custom key field):
type Post @model @key(fields: ["title"])
 {
  title: String!
  editors: [PostEditor] @connection(keyName: "byPost", fields: ["title"])
}

type PostEditor
  @model(queries: null)
  @key(name: "byPost", fields: ["title", "editorID"])
  @key(name: "byEditor", fields: ["editorID", "title"]) {
  id: ID!
  title: String!
  editorID: ID!
  post: Post! @connection(fields: ["title"])
  editor: User! @connection(fields: ["editorID"])
}

type User @model {
  id: ID!
  username: String!
  posts: [PostEditor] @connection(keyName: "byEditor", fields: ["id"])
}

However, it should be possible to combine a key of String and ID
  @key(name: "byPost", fields: ["title", "editorID"])

The error message in this case would be title field is not of type ID.
Reproduction:

Create the schema
amplify push

This feature may not be implemented yet and would therefor not be a bug. It seems like this should work as a "basic" @connection use case, however. Please make this a feature request, if it isn't a bug.

@tw1t611 in the connection transformer we're not enforcing the ID type, we are just checking if both side of a connection has the same type. Trying out the schema above works as expected, amplify api gql-compile succeeds with the latest version of the CLI.
What version of the CLI were you using? Please try out with the latest version and get back to us.

@attilah it is the latest version.
While building the reproduction, I realized, that I wrote name instead of keyName.
That was absolutely my fault. The error message was a bit misleading though.
Sorry for the noise.

@tw1t611 no worries, and thanks for the feedback on the error message.


Add controller stub and circle CI configuation

New version of #65, checking if CircleCI runs.

Feel free to merge


Replace react-addons-shallow-compare with PureComponent?

According to the docs, react-addons-shallow-compare is a legacy addon. Have you considered replacing it with React.PureComponent?

Doesn't seem appropriate for a lib, considering PureComponent is a recent addition and I'm trying to maintain backwards compatibility. I could make that upgrade in the next major release- but it would be backwards-breaking.

Ah, that's totally understandable. Do you know if there's any public metrics that show each version's usage?  Would it be safe to assume your plan is to follow along with react and drop v14 support when v16 is released?

Hm, no clue. I don't know if even the React team knows that, although I'll ask Ben when I see him.
Yes, that would be reasonable to assume. 😄

Closing for now 😄 We can revisit when next React version drops

@bvaughn Can we revisit this? Dropping shallow-compare would be nice because it frees up that pesky peer dependency.

We can revisit once React 16 lands or with the next major react-virtualized released. 😄

Sounds good! Sorry, I suppose I should have prefaced that last message with "I have no idea what version React was at when you guys first had this conversation".
** subscribes to thread **

No problem 😄  This ticket was from November '16 and React 15 has been out since April '16, so same major version.

FYI thought you might like to know that the upcoming react-virtualized version 9 will drop support for shallowCompare in favor of using React.PureComponent.


Is there any way to install those packages all in one package?

Is there any way to install those components in one package?
For example:
npm i -S @material/react
I found it a little bit messy that I have to install each package!

Try npm i material-web-react.
https://www.npmjs.com/package/material-web-react

That is actually a totally different package. There was a request with creating an all-in-one package similar to MDC Web's material-components-web package. We just haven't had the time. Please feel free to open a PR. I'll leave this open to some day get to it.

@ImSolitude I stand corrected, sorry for misleading!

no worries !


Unknown import:    themes/default/globals/site.variables.import.less

getting this on trying to use the package:
=> Your application has errors. Waiting for file change.
=> Modified -- restarting.
=> Errors prevented startup:

   While processing files with flemay:less-autoprefixer (for target web.browser):
   client/lib/theme.import.less:20: Unknown import:
   themes/default/globals/site.variables.import.less

I don't get the import error, but i am unable to use colors in my less files. It says it's undefined, even though i am reference importing semantic ui and i also tried directly reference importing site.variables.import.less as well.
@import (reference) '../lib/semantic-ui/site/globals/site.variables.import.less';

Same issue for me on a fresh, clean install of meteor, with solely semantic:ui and flemay:less-autoprefixer packages added, and with an empty custom.semantic.json file :
#Errors prevented startup:

While processing files with flemay:less-autoprefixer (for target web.browser):
client/lib/semantic-ui/theme.import.less:20: Unknown import: themes/default/globals/site.variables.import.less

Variables are scoped per component. @blue has a different meaning for each element, see guide on semantic-ui.com

@jlukic Doesn't matter weather they are scoped per component or not, what i want to access is the color Blue that i have mentioned in my site theme, which is something pretty trivial and defeats the purpose of setting colors if you can't access them in your own less files.

@flemay It seems this is a bug in less-autoprefixer
So if you want to reproduce, setup a few colors in your site theme in semantic-ui and then reference importing either semantic ui entirely or the site.variables.import.less file and using one of those color variables. Seems to be undefined.

Does the less package work in this situation?

You can @import any file you like from your own LESS.
Generally though, inheritance in SUI is scoped per-component.
http://learnsemantic.com/themes/overview.html

I found the solution here worked: #74

I'm seeing this issue with meteor 1.2.1 and latest semantic
I tried getting rid of default (marking it false in json file) and leaving material , but it breaks , is that normal or rather expected behaviour ? Can someone confirm.
i can make it work by editing theme.config.import.less and change default to my theme but it would've been nice to just get rid of default theme altogether.
thanks

Same issue, unable to import after just adding the packages and deleting .custom.semantic.json
While processing files with flemay:less-autoprefixer (for target web.browser):
client/lib/semantic/theme.import.less:16: Unknown import:
themes//default/globals/site.variables

+1 Exact same issue

@jeveloper as soon as you set default to false in custom.semantic.json and save, meteor removes the client/lib/themes/default/ folder, so the 'default' pathname values in theme.config.import will present a wrong path down the line.
There is also one file path thats hardcoded in theme.import.less:20
/* Default site.variables */
@import<EMAIL_ADDRESS>
So it might be a good idea to leave default installed.
Most, if not all, themes are just overrides of the default theme. So removing the default theme, might be more problematic and might force every thememaker to include everything.
Why not just put your theme into the /site folder and leave it default? The /themes folder gets regenerated and shouldn't be edited anyway.
@aesmon did https://github.com/Semantic-Org/Semantic-UI-Meteor/issues/74 work for you?
As in ...

stop meteor
delete .custom.semantic.json
start meteor

I just did a fresh install of semantic and i also had to stop meteor before it generated the inital files and folders, maybe we should add that step to the docs too.

I'm still having this issue on a fresh installation of Meteor and following the steps in the docs.

same here. deleting .custom.semantic.json makes no difference.
client/semantic-ui/theme.import.less:20: Unknown import: themes/default/globals/site.variables.import.less

thanks @KristerV, that fixed it for me as well

Closing this

You need ONLY site: true, as soon as I removed unused definitions I got rid of all the errors which came after I just added site: true...

this also helps:
https://github.com/Semantic-Org/Semantic-UI-Meteor/issues/78#issuecomment-187677357


added better styling to reder jupyter notebooks nicely.

especially fixed the spacing issues.
also deleted an unused block in one of the doc pages.

great job @zodwick!
just one more thing regarding the code blocks
jupyter

md

can we make it the same font and size? I think the code block itself is a bit too large (the copy botton is larger too)

I observed that mkdocs content tabs used to set different LLMs  does not get rendered in notebook https://github.com/explodinggradients/ragas/pull/1398

those are not supported in mkdocs-jupyter actually, we will have to render it as html danielfrg/mkdocs-jupyter: Use Jupyter Notebook in mkdocs

i made the clipboard icon smaller, but mkdocs-jupyter uses a different svg compared to normal md so there is going to be slight differences
jupyter vs md as of now. close but not exact

@jjmachan @shahules786


Problem with Expression index

I am triing to ensure that my context is synced with external schema. I am triing to scaffold context, but ef failed with error about expression index.
Expression index 'ux_bla_bla' on table private.awesome_table cannot be scaffolded, expression indices aren't supported and must be added via raw SQL in migrations.
System.NotImplementedException: The method or operation is not implemented.

I can't drop that index because of ther reasons and i can't skip it (or i don't know how)
Could you pls provide some flag or method to complete scaffold with warning about skipped objects or allow to skip em manually?
i am usin "dotnet ef dbcontext scaffold" command

@WhoStoleMyNick - Are you using EF Core?  And which version?

Npgsql.EntityFrameworkCore.PostgreSQL 2.2.4

Wrong project?(

cc: @roji
@WhoStoleMyNick - Are you using EF Core 2.2 or EF Core 3.0?

2.2. I guess if i use 3.0 version of provider would be 3.0+

Was just making sure that we are not mixing package versions. Given the stack trace, it seems to me that the method is not just implemented (so again some issue in mixed package version), expression indices is just a warning and it is being skipped but something else is causing error. NpqSql provider repo would be correct place for this issue.

Oops, i change target framework of csproj from 2.1 to 2.2 and get
Expression index 'ux_sync_prices_info_price_unique_key' on table private.sync_prices_info cannot be scaffolded, expression indices aren't supported and must be added via raw SQL in migrations.
and context generats successfully, sorry my fault, thanks for fast responce.

Context was generated with warn as expected.

@WhoStoleMyNick yeah, as the exception message says, the Npgsql EF Core provider doesn't support expression indices in any way. This is tracked by https://github.com/npgsql/efcore.pg/issues/119 and hopefully will happen one day. But you can still set them up via raw SQL in your migrations, so this shouldn't be blocking.


feat(deps): dynamic renaming of dependency elements with version #127

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #127
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Moved back to Work in Progress - import Auro-Library once it's a package and call scripts from there.
https://github.com/AlaskaAirlines/auro-library/issues/13

All code changes look good, but the javascript examples are not working on the demo

Not sure if related but I'm seeing this error in the console when I run the demo:

All code changes look good, but the javascript examples are not working on the demo

Not sure if related but I'm seeing this error in the console when I run the dynamic menu api demo:
Otherwise changes look good

@Izook This was definitely related. Great catch! doc example updated to work with dynamic tags.

:tada: This PR is included in version 1.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


iOS Navigation slide direction bug

blocked by: https://github.com/matteopuc/swiftui-navigation-stack/issues/80


Special methods don't work with Python 3.5 type hints

This works fine in Python 3.5:
class AwesomeView(FlaskView):

    def patch(self, id):
        return { "something" }

However if I want to use Python Type Hints, I have to manually add a @route decorator:
class AwesomeView(FlaskView):

    @route('/<id>', methods=['PATCH'])
    def patch(self, id: str) -> dict:
        return { "something" }

If I omit the decorator, then Flask-Classful triggers a ValueError with the following stack trace:
  File "/(...)/api/resources/posts.py", line 121, in <module>
    AwesomeView.register(app)
  File "/(...)/virtualenv/lib/python3.5/site-packages/flask_classful.py", line 134, in register
    rule = cls.build_rule("/", value)
  File "/(...)/virtualenv/lib/python3.5/site-packages/flask_classful.py", line 290, in build_rule
    argspec = get_true_argspec(method)
  File "/(...)/virtualenv/lib/python3.5/site-packages/flask_classful.py", line 360, in get_true_argspec
    argspec = inspect.getargspec(method)
  File "/usr/local/Cellar/python3/3.5.1/Frameworks/Python.framework/Versions/3.5/lib/python3.5/inspect.py", line 1045, in getargspec
    raise ValueError("Function has keyword-only arguments or annotations"
ValueError: Function has keyword-only arguments or annotations, use getfullargspec() API which can support them

@JivanRoquet thank you for your report. It's best if you could create a PR with test and code fixes for this. Otherwise, I'll look into this soon.

@hoatle Thanks for your feedback. I can't set a local testing environment at this time, but I guess something like:
# line 360 in flask_classful.py
try:
    argspec = inspect.getargspec(method)
except ValueError:
    argspec = inspect.getfullargspec(method)

would be a good step towards a solution?

@JivanRoquet thank you for your code fix, it works with my test, I'm finishing the tests and will push it soon.

it should work now (on develop branch), I'm going to release v0.12.0 to include this fix.

@JivanRoquet v0.12.0 released, enjoy! https://pypi.python.org/pypi/Flask-Classful

This is awesome!! Thank you


Sort translation keys

When I extract all translation keys with this plugin, I want sort keys.
This:
{
   "ADD": "Add",
   "CANCEL": "Cancel",
   "DELETE": "Delete"
}

Instead of this:
{
   "CANCEL": "Cancel",
   "ADD": "Add",
   "DELETE": "Delete"
}

I hadn't seen this option: stringifyOptions


Do not save state when a block fails checkpoint validation

Currently the state is saved after processing a block before any consensus hashing or checkpoint validation takes place.
This results in the following scenario:

Client processes block n
State for block n is written out
Checkpoint validation for block n is performed and fails
Client is shutdown
Client is restarted
Client loads latest state from disk (which is now block n)
Client continues parsing from block n+1

This PR changes the block handler to only save the state if there is no checkpoint failure.  On client restarts after a checkpoint failure we thus load state from block n-1 and run into the checkpoint failure again parsing the next block.

After you removed the else, or if you think it's better to leave it there: ACK. Will merge, after the update or comment from you. :)


Revit Connector: Parameter Name set to GUID

Hey,
Objects.Converter.Revit.COnverterRevit.GetParamInternalName sets the applicationInterName for shared parameters to the GUID.
https://github.com/specklesystems/speckle-sharp/blob/feb6dae98f2f27602aea02ff9d91610ddd45cb9f/Objects/Converters/ConverterRevit/ConverterRevitShared/ConversionUtils.cs#L421-L424
It causes trouble as the objects' name contains dashes (-) and GraphQL does not support that. Should the dash characters be removed? Or can we add the parameter with name to the Parameter Array (instead applicationInternalName?
thanks!

Hey @ks-cph! Dashes shouldn't be a problem - can you share with us the query you're trying to make and from where? It might be just a matter of escaping things properly (on your end or on our end!).

thanks for the feedback.
@didimitrie I am new to GraphQL and I read that it does not work with a dash. Eventually I want to connect from PowerBI. For now I am testing with the playground. Here is the basic query which does not succeed:
query Objects ($myQuery: [JSONObject!]) {
  stream(id: "e4505675be") {
    name
    object(id: "7037d7cc47695869490ba548e377df0f") {
      totalChildrenCount
      children(
        query: $myQuery
        select:["category" "family" "parameters.0511d859-633b-4326-a8ea-5ab75dc311c0"]
      ) {
        totalCount
        cursor
        objects
        {
          data
          speckleType
          applicationId
        }
      }
    }
  }
}

@didimitrie thanks again for the quick response. in Revit we have different kinds of parameters.
Your query does work for Revits' builtin-parameters but not for shared parameters. Shared parameters have the guid as internal name.
The shared parameter:
"8d5d6d46-5941-4a67-a6ab-68458526bfb0": {
                    "id": "15f770f16f74ad7ea42c3b391d7418cb",
                    "name": "BIM7AATypeCode",
                    "units": null,
                    "value": "253",
                    "isShared": true,
                    "isReadOnly": false,
                    "speckle_type": "Objects.BuiltElements.Revit.Parameter",
                    "applicationId": null,
                    "applicationUnit": null,
                    "isTypeParameter": false,
                    "totalChildrenCount": 0,
                    "applicationUnitType": "autodesk.spec:spec.string-2.0.0",
                    "applicationInternalName": "8d5d6d46-5941-4a67-a6ab-68458526bfb0"
                  }

My query:
query{
  stream(id:"d4af7b0846"){
    object(id:"bbdd498c9f673e918736b33786c161c0" ){
      children(
        query:[{field:"speckle_type", value:"Objects.BuiltElements.Wall:Objects.BuiltElements.Revit.RevitWall", operator:"="}]
        select:["category", "family", "parameters.8d5d6d46-5941-4a67-a6ab-68458526bfb0"]){
        totalCount
        objects{
          data
        }
      	cursor
      }
    }
  }
}

Error result:
{
  "errors": [
    {
      "message": "with \"objs\" as (select \"id\", \"createdAt\", \"speckleType\", \"totalChildrenCount\", jsonb_path_query(data, $1) as \"0\", jsonb_path_query(data, $2) as \"1\", jsonb_path_query(data, $3) as \"2\", jsonb_path_query(data, $4) as \"3\", jsonb_path_query(data, $5) as \"4\", jsonb_path_query(data, $6) as \"5\" from \"object_children_closure\" inner join \"objects\" on \"objects\".\"streamId\" = \"object_children_closure\".\"streamId\" and \"objects\".\"id\" = \"object_children_closure\".\"child\" where \"object_children_closure\".\"streamId\" = $7 and \"parent\" = $8 and \"minDepth\" < $9 and (jsonb_path_query_first( data, $10 )::text = $11 ) order by \"id\" asc) select * from \"objs\" RIGHT JOIN ( SELECT count(*) FROM \"objs\" ) c(total_count) ON TRUE limit $12 - syntax error, unexpected IDENT_P at end of jsonpath input",
      "locations": [
        {
          "line": 4,
          "column": 7
        }
      ],
      "path": [
        "stream",
        "object",
        "children"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR",
        "exception": {
          "length": 125,
          "name": "error",
          "severity": "ERROR",
          "code": "42601",
          "file": "jsonpath_scan.l",
          "line": "283",
          "routine": "jsonpath_yyerror"
        }
      }
    }
  ],
  "data": {
    "stream": {
      "object": null
    }
  }
}

Just an additional note that I forgot to mention earlier, we use the GUID for shared parameters also because their name might be different based on the current localization.

Related: https://speckle.community/t/not-able-to-query-shared-parameters-from-revit/3163/3

Closing this as it's something to be fixed on the server-side, and already tracked there ☝🏼


how to use InstantID for face swapping tasks?

in the paper, it showed a comparison between Instant ID with InsightFace Swapper (Inswapper). the results are shockingly amazing when it comes to eastern asians
Are you guys planning on releasing the code for that specific task? and for this specific task, did you guys increase the number of reference images to see if a better quality was achieved?

More reference images do not significantly improve the results

More reference images do not significantly improve the results

To me that seems like a great improvement. The ones using only one ref seem sometimes a bit off whereas the last ones are spot on.


[WIP] feat: impl basic test suite loader and runner

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

feat: impl basic test suite loader and runner

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.82%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2300

Relevant Lines:
2619

💛  - Coveralls


Issue about personachat_data_dir

In ProfilePromptGenerator, it seems that the argument "personachat_data_dir" is missed. When I run the profile_main.py, the error: TypeError: init() missing 1 required positional argument: 'personachat_data_dir' appeared.
Thanks again for your open code.

Thanks for the feedback :)
We will correct and update it ASAP :)


update function to get previous route to include logic for 404

I'll take a look. Give me an hour or so to get next to a computer. If all is well, I'll add it and up the package version for you all to install. Thanks

Apologies! This was the default repo and opened the pull too soon. In case you run into a similar situation or decide on a different fix, this was a fix for if no path is returned for the previous page, so we need to provide 404 as a fallback or else .hide() on the 404 page will never be called. I'll update the forked repo, thank you though!

That makes sense. No worries.
On Tue, Jun 9, 2015 at 3:27 PM Sally Traynham<EMAIL_ADDRESS>wrote:

Apologies! This was the default repo and opened the pull too soon. In case
you run into a similar situation or decide on a different fix, this was a
fix for if no path is returned for the previous page, so we need to provide
404 as a fallback or else .hide() on the 404 page will never be called.
I'll update the forked repo, thank you though!
—
Reply to this email directly or view it on GitHub
https://github.com/mkay581/route-manager/pull/4#issuecomment-110475604.


test to ensure R is 1d

this would probably best be done in the inMpc decorator

the code can now handle arbitrary R dimensionality


added governance doc

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos
[ ] Infra

Description
The governance model

LGTM :-)


Make uninstalling unnecessary ports easy

port reclaim is quite nice for cleaning up waste, but it doesn't do anything about ports you no longer need, such as build dependancies — like apt-get and its autoremove. To work around this, I'd occasionally do repeated port uninstall leaves until no such leaves were left — which is a bit silly…
This pull request attempts to address this in two different ways. The first commit adds an rleaves alias which identifies all unrequested ports. The seconds commit achieves the same within port reclaim.
I considered extending leaves to list all proper leaves — but opted not to, in the interest of backwards compatibility.

Looks good to me at a first glance.
I assume you tested this?

Yeah, I did my best to optimise reclaim and ensure that it works as expected. rleaves isn't quite as fast, due to lacking access to sort_portlist_by_dependendents. From my perspective, the main risk is that it'll expose bugs like this:

User installs some port A depending on port B.
User installs port C, from source, which has no declared dependency on port B.
During its configure stage, port C picks up port B and links against it during build.
User uninstalls port A.
User later runs port reclaim which uninstalls port B, breaking port C.
port rev-upgrade notices the breakage and reinstalls port C.

I didn't actually test this, or run into it, since I tend to install all ports using trace mode. (Shouldn't that be the default by now, anyway?) Still, this is likely to expose bugs like that — and speaking of bugs, I should probably add a run of rev-upgrade within reclaim to catch them.

That's fine. If it exposes dependency bugs in ports, then those bugs can be fixed.
Trace mode didn't really work until the release of MacPorts 2.4.0 a few weeks ago. Assuming we get no bug reports about it now, it could be come default, but last I heard it still had a 50% performance penalty, so maybe that's not so great. In any case, not relevant to this pull request and can be discussed elsehwere.


.Outlook

Check out Samsung Members to get help with any issues you might have with your Samsung Galaxy phone. Tap the link to download Samsung Members.
http://bit.ly/1rjtDOM

Samsung


Add sanity checks for registering large amounts

https://github.com/VolumeFi/paloma/issues/36
Add sanity checks against registering large amounts.
Testing completed

[x] Compiles
[ ] Tested for real

This is not what we want. Closing.

looks good, please please open the PR from VolumeFi:chase-deduct into Paloma:main


Installation using helm in website

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Added kueue installation guide using helm in website. The content was used from README
Which issue(s) this PR fixes:

Fixes #753
Special notes for your reviewer:
Does this PR introduce a user-facing change?

None

cc @kerthcet

/ok-to-test
/cc @moficodes

The new pull request is here


Support offset frame type

What problem does this PR solve?
Issue Number: ref #7376
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @windtalker @SeaRise

/cc @SeaRise

/cc @SeaRise

/cc @windtalker

/merge