[Accelerator] Cambricon MLU support

Description
This PR includes Cambricon MLU accelerator support.
With this PR, DeepSpeed supports MLU as backend for training and inference tasks.

Please see here to fix formatting issues.
Also, consider updating the following accelerator docs:

Tutorial.
HW list.

@Andy666G -  curious if you would be able to run the precommit formatter to resolve the formatting errors?

@Andy666G - curious if you would be able to run the precommit formatter to resolve the formatting errors?

Hi, the formatting errors have been resolved.

@Andy666G, do you plan to address my feedback regarding documentation in a separate PR?
https://github.com/microsoft/DeepSpeed/pull/6472#issuecomment-2326446658


Updated MultipleGiftsForSpouse to work with the up-coming SMAPI 3.0 release.

Fixed the events to match the new events that will be implemented with the up-coming SMAPI 3.0 release. Also fixed the updatekey for nexus.

@Mizzion Your commit is included in #2. Can you close this one to avoid confusion?


Incorrect stack sizes not yet fixed in 0.5

From my experimentation, these are the incorrect stack sizes I've found.
Items that stack too high on server

Snowballs should max at 16
Banners should max at 16
Signs should max at 16
Armor stands should max at 16
Eggs should max at 16

Unstackable items that are stackable on server

Beds
Shulker boxes
Turtle shells
Fishing rods
Enchanted books
Potions and water bottles (which turn into uncraftable potions on drop)
Filled buckets, including fish and liquid
Carrot on a stick
Elytra
Shield
Trident
Music discs
Totem of undying
Shears
Boat
Mushroom stew
Beetroot soup
Rabbit stew
Cake
All types of minecarts
Horse armor

There may be more not listed.

Thanks for the thorough list! I'll add these to the function and get back to you.

I've added the items in your list—feel free to let me know if there's more missing.


Fix: No str.split or f-string for subprocess args

Never use str.split to split arguments to subprocess or f-strings to create arguments for subprocess. This breaks as soon as there a non-ASCII characters in any path, e.g. file names with a white space.
You either need shlex.split and shlex.quote (which does not work on non-Unix shells) or use list of string all the way.
Also fix a typing bug with port. It's now an int all the way.
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

How are users expected to call vllm-args from the CLI now?

It's now a list of strings instead of a single string. Let me add a unit test as example.
serve:
  vllm:
    vllm_args:
      - "--egg"
      - "--spam"

How are users expected to call vllm-args from the CLI now?

It's now a list of strings instead of a single string. Let me add a unit test as example.
serve:
  vllm:
    vllm_args:
      - "--egg"
      - "--spam"

Yes I understand the structure for the config but how about the CLI --vllm-args though? Thanks

Yes I understand the structure for the config but how about the CLI --vllm-args though? Thanks

I didn't know that there was a --vllm-args CLI option because there was no unit test or integration test. I now fixed the option to be multi-arg, so you can do --vllm-args='--dtype auto' --vllm-args='--enable-lora'.

Removing the "hold" label as Sebastians question about the structure of --vllm-args for ilab serve has been addressed by Christian.
Nevertheless future design discussion about --backend-args instead of --vllm-args need to take place in a future PR.


Add handling of descriptor protos into mixer

Right now, we have placeholders for log_entry_descriptors and the like in our codebase. We need to properly plumb them throughout the codebase, making them available to managers, and perhaps adapters (metrics backend programming, etc.).

@douglas-reid you think this is needed for NEXT?

Now that all aspect validation has been implemented all that's left is to validate Descriptors themselves.

@ZackButcher any update in status?

Did we decide to punt descriptor validation to post alpha? Otherwise I can implement pretty easily, just haven't been treating it as a priority.

Can this be closed?

Yup, with https://github.com/istio/mixer/pull/613 this is done.


Corrected Spanish translation of challenges radio buttons section

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @anamariaaya thanks for the PR. I recommend you work on your fork next time please?


"Deploying a self-contained app" no localhost cert for ssl, no dotnet installed

i think this page is gd, but... what i can not figure out... is for "Deploying a self-contained app"
This means that dotnet is not installed, which mean you can not run such command "dotnet dev-certs https -v"
which mean you can not get ssl working locally
now... what would be nice if they explain how you would get this working step by step.
I have some gd links but it a combination of some of them, but still not 100%... so there does not seem to be a gd guide on this.
"Deploying a self-contained app"
sudo  ASPNETCORE_URLS=https://*:5000 ./dotnetWebApp
This will not work as there is no localhost cert. it suggests "dotnet dev-certs https -v" but this is not installed.
if you install what was the point of "Deploying a self-contained app" if you installing dotnet.
Please help, with how you get local ssl when you have not installed dotnet as you have gone done the "Deploying a self-contained app" path
looking for how this is done on Raspberry OS
[Enter feedback here]

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

ID: 853c5e7f-a0b8-c6ba-8744-293407a12e86
Version Independent ID: 3f79892b-de26-fb7a-77b1-ec02b8a7e566
Content: Deploy .NET apps to Raspberry Pi
Content Source: docs/iot/deployment.md
Product: dotnet
GitHub Login: @CamSoper
Microsoft Alias: casoper

sum how two were created so just killing this one


Bump min tls version to 1.2 for digital twins live test event hub

Either the tests or deployment script are triggering violations on minimum tls version usage. Enforcing a minimum at the event hub level.

/azp run net - digitaltwins - tests

/azp run net - digitaltwins - tests

/azp run net - digitaltwins - tests


[10.x] Adds a new doesntHave method to the session store

What?
This PR adds a doesntHave method to the session store, which allows to check if a given key is missing from the session of if it's value is null.
Why?
Before this PR, you have to use ! session()->has('key'). If this PR gets merged, you can use session()->doesntHave('key'), which IMO is much more readable.
//Before
if (! session()->has('key')) {
    // Do something...
}

// After
if (session()->doesntHave('key')) {
    // Do something...
}

PS..: I'm sending this to master because i think adding a new method to a non internal interface is a major breaking change accordingly to semver.

@driesvints I'm closing this, please check #41738.


[Python] Add a couple quality-of-life improvemenets to testing.util.assert_that

These changes just extend assert_that to accommodate a couple of common footguns I've seen.

assert_that will now check to see if you're calling it with a pipeline that has already been run. This catches the following circumstance:

with beam.Pipeline() as p:
    my_pcoll = p | # ...
assert_that(my_pcoll, equal_to([1,2,3]) # should've been indented

assert_that automatically creates a unique label if the label it's given is already taken. Usually when writing unit tests, we don't really need very specific labels for assertions. When this comes up, we usually have to just manually increment every assertion which is quite tedious.

assign set of reviewers

I ran the linting instructions here:
https://cwiki.apache.org/confluence/display/BEAM/Spotless+pre-commit
But they had no effect on any files

Turns out I jumped the gun and there are more than just lint failures. There were a few true tests failures that came up from actual tests that weren't actually using assert_that correct. I fixed a few but all the groupby example tests weren't trivial to fix. I confirmed that if I changed the assertion in those tests, they would've always passed before the changes in this PR

Turns out I jumped the gun and there are more than just lint failures. There were a few true tests failures that came up from actual tests that weren't actually using assert_that correct. I fixed a few but all the groupby example tests weren't trivial to fix. I confirmed that if I changed the assertion in those tests, they would've always passed before the changes in this PR

So what is the plan here? close this PR for now or you want to keep working on this?

I pushed test fixes and skips so we should be all set
On Wed, Mar 27, 2024, 6:57 PM liferoad @.***> wrote:

Turns out I jumped the gun and there are more than just lint failures.
There were a few true tests failures that came up from actual tests that
weren't actually using assert_that correct. I fixed a few but all the
groupby example tests weren't trivial to fix. I confirmed that if I
changed the assertion in those tests, they would've always passed before
the changes in this PR
So what is the plan here? close this PR for now or you want to keep
working on this?
—
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/30771#issuecomment-2024119891, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQJSFHDQR6P6VV2NLMJNM3Y2NFGJAVCNFSM6AAAAABFLM7DIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRUGEYTSOBZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I created a spinoff issue since I think fixing the tests (which were already broken before these changes) is probably out of scope

Bump @liferoad whenever you have a moment

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

Project coverage is 71.44%. Comparing base (827e96d) to head (ef4829d).
 Consider uploading reports for the commit e718a8f to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #30771       +/-   ##
===========================================
+ Coverage   38.52%   71.44%   +32.92%
===========================================
  Files         698      710       +12
  Lines      102371   104823     +2452
===========================================
+ Hits        39440    74894    +35454
+ Misses      61301    28299    -33002
  Partials     1630     1630

Flag
Coverage Δ

python
81.23% <100.00%> (+52.08%)
:arrow_up:

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

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

The last commit failed with some weird pydantic/hypothesis errors that seem unrelated to these changes

Gah, I have no idea where this failure couldve come from:
    run()
  File "/runner/_work/beam/beam/sdks/python/test-suites/tox/py38/build/srcs/sdks/python/target/.tox-py38-tft-113/py38-tft-113/lib/python3.8/site-packages/hypothesis/entry_points.py", line 35, in run
    hook = entry.load()
  File "/runner/_work/beam/beam/sdks/python/test-suites/tox/py38/build/srcs/sdks/python/target/.tox-py38-tft-113/py38-tft-113/lib/python3.8/site-packages/setuptools/_vendor/importlib_metadata/__init__.py", line 208, in load
    module = import_module(match.group('module'))
  File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'pydantic._hypothesis_plugin'

I think that error is unrelated to this change, I see it on other PRs as well. Filed: https://github.com/apache/beam/issues/30852

I think this might be ready to be merged then if it's safe to ignore the pydantic/hypothesis errors

pydantic errors should be resolved now. ptal at linter errors.

Looks like this is very close to crossing the finish line - @hjtran any chance you can take another look at this?

Yes, I'll try to take another round of looks at all the PRs I have up this week. I haven't been able to get a reliable dev environment going so every round of iteration has been a bit tough

I haven't been able to get a reliable dev environment going so every round of iteration has been a bit tough

Oh that's not good.  We have https://s.apache.org/beam-python-dev-wiki that might help, or if not we can definitely edit the instructions there.
Some people previously wrote and used https://github.com/apache/beam/blob/master/local-env-setup.sh specifically for this purpose (getting  a working environment fast), but I am not sure if the script has enough usage, possibly it hasn't been maintained and I haven't used it recently.

@damccorm finally got a consistent dev environment up. I updated this branch according to @robertwb 's last comment. Hopefully it loos good now


Internal server error on CSV upload

[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>] 'got exception "\\'ascii\\' codec can\\'t decode byte 0xe2 in position 127: ordinal not in range(128)"'
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>] ['Traceback (most recent call last):\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  '  File "/usr/lib/python2.6/site-packages/ermrest/ermrest_apis.py", line 250, in METHOD\\n    first = result.next()\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  '  File "/usr/lib/python2.6/site-packages/ermrest/sanepg2.py", line 146, in pooled_perform\\n    result = bodyfunc(conn, cur)\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  '  File "/usr/lib/python2.6/site-packages/ermrest/url/ast/data/path.py", line 197, in wrapbody\\n    raise rest.BadRequest(e.message)\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  '  File "/usr/lib/python2.6/site-packages/ermrest/exception/rest.py", line 39, in __init__\\n    WebException.__init__(self, status, headers=headers, data=data, desc=desc)\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  '  File "/usr/lib/python2.6/site-packages/ermrest/exception/rest.py", line 29, in __init__\\n    self.detail = urlquote(desc % data)\\n',
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]  "UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 127: ordinal not in range(128)\\n"]
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>] Traceback (most recent call last):
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/web/application.py", line 239, in process
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     return self.handle()
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/web/application.py", line 230, in handle
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     return self._delegate(fn, self.fvars, args)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/web/application.py", line 405, in _delegate
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     return handle_class(f)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/web/application.py", line 396, in handle_class
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     return tocall(*args)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/ermrest_apis.py", line 286, in PUT
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     return self.METHOD('PUT')
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/ermrest_apis.py", line 250, in METHOD
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     first = result.next()
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/sanepg2.py", line 146, in pooled_perform
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     result = bodyfunc(conn, cur)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/url/ast/data/path.py", line 197, in wrapbody
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     raise rest.BadRequest(e.message)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/exception/rest.py", line 39, in __init__
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     WebException.__init__(self, status, headers=headers, data=data, desc=desc)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]   File "/usr/lib/python2.6/site-packages/ermrest/exception/rest.py", line 29, in __init__
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]     self.detail = urlquote(desc % data)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>] UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 127: ordinal not in range(128)
[Thu Jul 23 15:06:01 2015] [error] [client <IP_ADDRESS>]

We did a bit of triage and it looks like the data file was causing an integrity error in Postgres which is coming back with an error detail string that is UTF-8 encoded with unicode characters coming from the input data and choking while trying to embed this error string into the ERMrest rest exception response.
This line in exception/rest.py seems to be the place we need to deal with already encoded UTF8 data values to protect ourselves.

I think this is fixed once #15 is merged..


Sync analysis_options.yaml & cleanups

Syncs the file with the latest list from https://github.com/dart-lang/linter/blob/master/example/all.yaml and enables the following new lints:

collection_methods_unrelated_type (found a real bug: https://github.com/flutter/flutter/issues/117356)
enable_null_safety
implicit_call_tearoffs
unnecessary_library_directive
unreachable_from_main (found a fair amount of dead code)
use_string_in_part_of_directives

To be enabled in a follow-up PR:

dangling_library_doc_comments
library_annotations

Should be fixed now :)

Never mind, looks like it is still having issues.

This cleanup makes me so happy! 😃


typo fix from retreived to retrieved

- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Thanks @liyongxin  - this repository is only used to cut off releases, and to do back ports from upstream changes into release branches. Could you open this PR in the upstream
https://github.com/moby/moby repository instead?


[FOR TRACKING]: Update data for Dash View

Dash View work from: https://github.com/chef/automate/issues/3639
dash view
filtering when selecting "time since last checked in" item
The "start" and "end" parameters filter on the checked-in date time.
-API: https://a2-dev.test/api/v0/cfgmgmt/nodes?pagination.page=1&pagination.size=100&start=2020-05-03T21:48:57Z&end=2020-05-06T21:48:57Z

Too Soon to work on this - closing for now


Can it support solid？

Please respect the issue template, and for this kind of Q&A start a Discussion or use the Discord chat server


Sending event inside handle_notifications()

Hi, I'm using nostr-sdk with a project but I'm stuck sending the event, I'm starting from the Nostr SDK example on the readme and I want to send an event after receiving a specific notification but I get and error, here is a simplified code:
// code ...

    client.connect().await?;

    let subscription = SubscriptionFilter::new()
        .pubkey(my_keys.public_key())
        .since(timestamp());

    client.subscribe(vec![subscription]).await?;

    client
        .handle_notifications(async |notification| {
            println!("{:?}", notification);

            let event = EventBuilder::new(Kind::Custom(11000), "test", &[])
                .to_event(&my_keys)
                .unwrap();

            client.send_event(event).await;

            Ok(())
        })
        .await
}

And I get this error
error[E0708]: `async` non-`move` closures with parameters are not currently supported
  --> src/main.rs:54:31
   |
54 |         .handle_notifications(async |notification| {
   |                               ^^^^^^^^^^^^^^^^^^^^
   |
   = help: consider using `let` statements to manually capture variables by reference before entering an `async move` closure

error[E0308]: mismatched types
  --> src/main.rs:54:31
   |
54 |           .handle_notifications(async |notification| {
   |  _______________________________^^^^^^^^^^^^^^^^^^^^_-
   | |                               |
   | |                               expected enum `Result`, found opaque type
55 | |             println!("{:?}", notification);
56 | |
57 | |             let event = EventBuilder::new(Kind::Custom(11000), "test", &[])
...  |
63 | |             Ok(())
64 | |         })
   | |_________- the found `async` closure body

Any ideas? thanks in advance

Hey, for now handle_notifications method can't handle async/await functions.
You can try in this way:
client.connect().await?;

let subscription = SubscriptionFilter::new()
    .pubkey(my_keys.public_key())
    .since(timestamp());

client.subscribe(vec![subscription]).await?;

let mut notifications = client.notifications();

while let Ok(notification) = notifications.recv().await {
    println!("{:?}", notification);

    let event = EventBuilder::new(Kind::Custom(11000), "test", &[])
        .to_event(&my_keys)
        .unwrap();

    client.send_event(event).await;
}

Ok(())

It works great! thanks

Is there a way to tell from which relay a notification or event was received?
This is relevant for cases lile
    let client = Client::new(&my_keys);
    client.add_relay("wss://relay.nostr.info", proxy).await?;
    client.add_relay("wss://rsslay.fiatjaf.com", None).await?;
    client.add_relay("wss://relay.damus.io", None).await?;

    client.connect().await?;

Is there a way to tell from which relay a notification or event was received?
This is relevant for cases lile
    let client = Client::new(&my_keys);
    client.add_relay("wss://relay.nostr.info", proxy).await?;
    client.add_relay("wss://rsslay.fiatjaf.com", None).await?;
    client.add_relay("wss://relay.damus.io", None).await?;

    client.connect().await?;

Currently no, but can be simply added in RelayPoolNotifications enum.
What do you mean with "cases lile"?

Sorry typo, meant "cases like".
Thanks for the pointer, I'll look into that.

Sorry typo, meant "cases like".
Thanks for the pointer, I'll look into that.

Something like this:
#[derive(Debug, Clone)]
pub enum RelayPoolNotifications {
    ReceivedEvent(Url, Event),
    ReceivedMessage(Url, RelayMessage),
}


Fix package description for MassTransit.AmazonSQS

Just a simple fix to the package description for the Amazon SQS transport.

Merged.


The contributing host profiles section in the documentation doesn't mention updating visit-install.

The "Contributing Host Profiles" section of the VisIt User's Manual doesn't mention that the visit-install script needs to be updated when adding a new site. Here is a link to the section.
https://visit-sphinx-github-user-manual.readthedocs.io/en/develop/dev_manual/ContributingHostProfiles.html

Could visit-install be modified to populate the list of host configs from src/resources/hosts/networks.dat ?
The windows installer does this.  Might be more work than it's worth to avoid the maintenance headache of having yet another file to modify.

Yes, I had thought about that as well. The code update is pretty kludgy and involves manually renumbering things when adding a new site in the middle of the list. The networks.dat file is pretty simple, so parsing it shouldn't be too hard, but nothing is easy in Bourne shell. :(

visit-install would have to be a generated file, from some template, adding the parsing of networks.dat. Otherwise it wouldn't be useable as a stand-alone program as it would need access to our source tree in some fashion.  Another hurdle.

Adding it to the documentation is the simpler thing to do, so let's go with that.


Backprop sometimes needs additional memory

Revisit https://github.com/chainer/chainer/pull/4301#issuecomment-392438444
Status
I measured how much memory is used in back-propagation.  The peak consumptions of gpu memory (max of cuda.memory_pool.used_bytes) are

model
initialize
forward
backward
fwd/bwd

ResNet50
0127614464
3626270720
4164088320
0.8708

U-Net
0081935360
0785561088
1011115520
0.7769

ResNet50

code: chainer.links.model.vision.resnet.ResNet50Layers
input: (32, 3, 256, 256), float32

U-Net

code: UNET in https://github.com/pfnet/PaintsChainer/blob/master/cgi-bin/paint_x2_unet/unet.py
input: (16, 4, 128, 128), float32

Feature request
Such memory would be reduced by a feature that allows del loss before executing loss.backward().  I'd like to implement it.

Could you elaborate what this means?

del loss before executing loss.backward()

Is is really related to #4301? #4301 does not reduce maximum memory consumption.

Reopen the issue as a PR is ongoing.


adispatch_custom_event not emitted in time when run via langgraph up (works on LangGraph cloud)

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
# This snippet is from the CopilotKit <> LangGraph integration
# We are using copilotkit_emit_state to send state updates to CopilotKit while
# a node is running.
# copilotkit_emit_state is just a thin wrapper around adispatch_custom_event:
# https://github.com/CopilotKit/CopilotKit/blob/3b362e5858f193c56e0e3baf184451eb1d6170fe/sdk-python/copilotkit/langchain.py#L120

async def search_node(state: AgentState, config: RunnableConfig):
    """
    The search node is responsible for searching the internet for resources.
    """
    ai_message = cast(AIMessage, state["messages"][-1])

    state["resources"] = state.get("resources", [])
    state["logs"] = state.get("logs", [])
    queries = ai_message.tool_calls[0]["args"]["queries"]

    for query in queries:
        state["logs"].append({
            "message": f"Search for {query}",
            "done": False
        })

    await copilotkit_emit_state(config, state)

    search_results = []

    for i, query in enumerate(queries):
        response = tavily_client.search(query)
        search_results.append(response)
        state["logs"][i]["done"] = True
        await copilotkit_emit_state(config, state)

    config = copilotkit_customize_config(
        config,
        emit_intermediate_state=[{
            "state_key": "resources",
            "tool": "ExtractResources",
            "tool_argument": "resources",
        }],
    )

    model = get_model(state)
    ainvoke_kwargs = {}
    if model.__class__.__name__ in ["ChatOpenAI"]:
        ainvoke_kwargs["parallel_tool_calls"] = False

    # figure out which resources to use
    response = await model.bind_tools(
        [ExtractResources],
        tool_choice="ExtractResources",
        **ainvoke_kwargs
    ).ainvoke([
        SystemMessage(
            content="""
            You need to extract the 3-5 most relevant resources from the following search results.
            """
        ),
        *state["messages"],
        ToolMessage(
        tool_call_id=ai_message.tool_calls[0]["id"],
        content=f"Performed search: {search_results}"
    )
    ], config)

    state["logs"] = []
    await copilotkit_emit_state(config, state)

Error Message and Stack Trace (if applicable)
No response
Description
On cloud, the custom event is immediately sent. With the local setup via langgraph up, the events are sent with a delay and arrive in the wrong order (so the user can see the current status of the searches being performed)
System Info
CLI

$ langgraph --version
LangGraph CLI, version 0.1.60

System Information

OS:  Darwin
OS Version:  Darwin Kernel Version 23.0.0: Fri Sep 15 14:41:43 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T6000
Python Version:  3.11.9 (main, Apr  2 2024, 08:25:04) [Clang 15.0.0 (clang-15<IP_ADDRESS>)]

Package Information

langchain_core: 0.3.19
langchain: 0.3.7
langsmith: 0.1.143
langchain_openai: 0.2.9
langchain_text_splitters: 0.3.2
langgraph: 0.2.52
langserve: 0.0.41

Other Dependencies

aiohttp: 3.10.5
async-timeout: 4.0.2
fastapi: 0.115.0
httpx: 0.27.2
httpx-sse: 0.4.0
jsonpatch: 1.33
langgraph-checkpoint: 2.0.5
langgraph-sdk: 0.1.36
numpy: 1.26.4
openai: 1.54.5
orjson: 3.10.7
packaging: 24.1
pydantic: 2.9.2
PyYAML: 6.0.2
requests: 2.32.3
requests-toolbelt: 1.0.0
SQLAlchemy: 2.0.35
sse-starlette: 2.1.3
tenacity: 8.5.0
tiktoken: 0.7.0
typing-extensions: 4.12.2

Closing. The issue was that response = tavily_client.search(query) is a blocking call. Switching to an async call fixed the issue.


[STAPLER-121] Initial proposed consolidated annotations for Stapler

See this thread on the jenkins-dev mailing list for context.
@reviewbybees @jenkinsci/code-reviewers
Here is the initial proposed annotations.
ToDo:

[x] Wire up the @StaplerPath, @StaplerRMI and @StaplerObject, annotations so that they actually work (the @StaplerGET / @StaplerContent / etc annotations work out of the box because they build on top of the @InterceptorAnnotation meta-annotation. The @StaplerFacet and @StaplerFragment annotations are documentation only - retained with runtime retention as some consistency checks possible with these may need to be implemented via a unit test generator.
[ ] Implement some annotation processor based consistency checks
[ ] Implement some automated unit test generator for the additional checks that cannot be performed by an annotation processor

[ ] Investigate whether the unit test generator could be implemented as an annotation processor

[x] Tests for the new annotations
[ ] Documentation

@jglick so functionally the annotations are complete.
I still need to:

[ ] Write some tests for annotated fields in complex class hierarchy
[ ] Write some tests for facets in complex class hierarchy with different @StaplerFacet annotations present
[ ] Write some more unit tests for the helper classes

That would close out the testing story... after that there is implementing some annotation processors and fleshing out the documentation.


Adding a logo to the masthead

I'd like to add a logo in the top left corner of the screen, left from the website's title.
I foolishly tried to add an  tag in masthead.html, where I also defined my static website title. It somehow worked, but the image was above the title and I guess this would also lead to problems with the "responsiveness".
What's the best way to add a logo to the masthead?
Thanks!

By editing _includes/masthead.html
Just by replacing the {{ site.title }} text with an image it works pretty well.

       ...

Which looks something like this:


Brittle test for MVT metadata (test or feature bug?)

I noticed this test fails in local:
$ ./run_tests.sh test/acceptance/layergroup-metadata.js
[...]
Running tests

  layergroup metadata
    1) tiles base urls with api key: 1234
    2) "after each" hook for "tiles base urls with api key: 1234"

  0 passing (648ms)
  2 failing

  1) layergroup metadata tiles base urls with api key: 1234:
     Uncaught AssertionError: null == 'https://localhost.localhost.lan:8888/api/v1/map/23f0a323b3b5207f277c68a763403945:0/layer0/{z}/{x}/{y}.mvt?api_key=1234'
      at testClient.getLayergroup (test/acceptance/layergroup-metadata.js:38:24)
      at test/support/test-client.js:907:20
      at Server.<anonymous> (test/support/assert.js:120:24)
      at emitCloseNT (net.js:1555:8)
      at _combinedTickCallback (internal/process/next_tick.js:71:11)
      at process._tickCallback (internal/process/next_tick.js:98:9)
[...]

But I also noticed its behavior depends on the config of the renderer.mvt.usePostGIS and it could even depend on the postgis version.
So, I'm wondering if the problem is the test itself, the postgis implementation or both. Is the metadata required to be exactly the same for mvt's rendered by mapnik and by postgis?
cc/ @Algunenano and @oleurud for awareness, and @dgaubert for brown dispatching (maybe RT?).

https://github.com/CartoDB/Windshaft-cartodb/blob/master/test/acceptance/layergroup-metadata.js

So, I'm wondering if the problem is the test itself, the postgis implementation or both. Is the metadata required to be exactly the same for mvt's rendered by mapnik and by postgis?

Yes, the metadata should be the same.

In a fresh installation in my environment, I get all green, both running yarn test and yarn run docker-test which they have also both Postgres/PostGIS's configurations 9.5/2.3 and 10/2.4.
What configuration do you have?. Could you paste here the metadata content for that test? Because it's not failing the instantiation and the response has body.metadata.layers[0].tilejson at least.

Unfortunately I deleted and got re-created my config/environments/test.js. I tried different things and I cannot reproduce it anymore. So I guess we can close this ticket. Sorry about the noise.


Why not compare the instruction directly when Paul and Vicky find the disputed one?

For example, when Paul and Vicky find the disputed instruction is :
Add. add_A, add_B, add_C, val_A, val_B, val_C
Both Paul and Vicky could reveal this instruction and push these data on Bitcoin to execute Punish Equivocation.
Why do there need additional challenges for each element?

yes, you're right! we updated the graph and changed the code accordingly
https://github.com/BitVM/BitVM/blob/main/docs/bitVM_graph_v2.svg


Errors when running Geometry optimizations

Hi there! I have been trying to perform geometry optimizations with the semiempirical approach and I have discovered that there is a misspelling in the following lines:
Class RMINDO3   line 175  from pyscf import semiemprical   -> from pyscf import semiempirical
Class UMINDO3   line 253  from pyscf import semiemprical   -> from pyscf import semiempirical
Once these two misspellings are corrected the geometry optimization is performed. I hope this error is corrected in the repo since
for the moment I need to clone the code, correct it myself in order to be able to use it. Thanks in advance.

Thanks for noticing this: can you please submit a pull request? Then you
are part of the developer community!
On Thu, Mar 10, 2022 at 8:33 AM alejandrosantanabonilla <
@.***> wrote:

Hi there! I have been trying to perform geometry optimizations with the
semiempirical approach and I have discovered that there is a misspelling in
the following lines:
Class RMINDO3 line 175 from pyscf import semiemprical -> from pyscf import
semiempirical
Class UMINDO3 line 253 from pyscf import semiemprical -> from pyscf import
semiempirical
Once these two misspellings are corrected the geometry optimization is
performed. I hope this error is corrected in the repo since
for the moment I need to clone the code, correct it myself in order to be
able to use it. Thanks in advance.
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/semiempirical/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN5NRDRCP7S4DCGEB44NDTU7IP5HANCNFSM5QNE75AA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Fixed in PR #5


Provide a BigQuery Public Dataset of OSV data

An interesting conversation today spawned the idea of providing OSV data via a BigQuery Public Dataset.
Apparently there's a path from Cloud Data Store to BigQuery. Something to explore further in the future.

I initially thought https://cloud.google.com/bigquery/docs/loading-data-cloud-datastore may be an easy way to achieve this, but our underlying Datastore entity format does not match the OSV schema exactly, so it may not be easily usable.

Having it as a time series day partitioned as well as just latest would be brilliant.

Interesting. So if I'm reading https://cloud.google.com/bigquery/docs/loading-data-cloud-storage-json and http://jsonlines.org/ correctly, this is a matter of emitting a single (or perhaps, per-ecosystem) file with all of the records concatenated and optionally compressed? This doesn't seem like it would be particularly difficult to modify the exporter to do.

So I am doing something like this in my use case and have been for several years already and have had to deal with 2 areas over this time period. Big query you cannot easily change a column type so in schema 1.58 the credits[].type field changed to an array so to credits[].type[] so I had to force the single entries pre 1.58 to an array. So bought the array forward to the newer schema.
The other area has been the "database_specific" and "ecosystem_specific" fields which I serialise to json and load as a string as unclear if this would remain valid for big queruy. Big query constrains characters allowed in column names https://cloud.google.com/bigquery/docs/schemas#column_names and the osv schema does match but I would assume governing names out of scope of osv is a possible challenge. This approach has worked for me otherwise with no issues. Just wante to make sure the constraints on field names was clear.


FP improvements proposal in security refactor

Checklist

[ ] Tests have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?
Bug fix, feature, or docs update
What is the current behavior? What is the new behavior?
You can also link to an open issue here for all the info.
Does this PR introduce a breaking change?
What changes might users need to make in their application due to this PR?

Commend addressed in the original PR


great code , but from where to get data ?

great code , but from where to get data ?

I'm sorry, I can't provide support on this project anymore.


Licensing: MIT?

Hi,
is it fair to assume the same license tas NSQ itself, ie. MIT? Could this be made more explicit?
Thanks
kind regards
Thilo

Yes, I will add one!

done

Thanks!


Get different traceroute results on different segments...

Just launched a 6.2 cluster (not master). Installed fine. Had some weirdness with a VM getting 2 IPs, and no tap interfaces. Killed the instances and tried again. Second time it all seemed to work.
Then did a traceroute and the intermediate hop showed up in the path! Did not expect that. Tried from a different instance on the other segment and got the * * * as I expected.
Will leave the cluster running at http://<IP_ADDRESS>/
Not going to fuss with this right now. I got the output Ineeded for the page I'm updating

I think this has been addressed. Traceroute inconsistency was sorted a while ago, and bug fixes to the installer have been applied for avoiding virbr0 creation.


Correct area code for Montenegro

Pattern for extracting 2-digit Montenegro area code (based on http://www.howtocallabroad.com/montenegro/).

Thanks for the update @bsboris. Would you be able to add some tests for ME in test/countries?

Done.

Woo! :tada:. Thanks @bsboris.


build check manual publish

#42  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mAmineChniti and the rest of your teammates on  Graphite


State of inMemoryCache is not always persisted/restored correctly

Describe the bug
When the cache grows, writing bigger cache to a file cant finish before the app was teared down. As a result persisted cache is incomplete and can result in unexpected behavior when this incomplete cache was restored (references not being found).
To Reproduce
Steps to reproduce the behavior:

Setup a Query resolving a rather large result set (>1000 records of complex Types referencing other Types that will be normalized)
Close the app after the result set was loaded (don't just move to background, but suspend the app) (cache will only partially been written to file)
Reopen the app (partially written cache will be restored)
Check your cached result set to w newly fetch Query result. It will not be of the same length

Expected behavior
Writing inmemorycache to persistence layer and restoring it to memory from persistence layer should always be idempotent.
Additional context
CacheProvider can't await cache writes when cache is only being written to file on app lifecycle methods. https://github.com/zino-app/graphql-flutter/blob/128b937d2ef5fbc6225d46de3da1a5cf1832dfca/packages/graphql_flutter/lib/src/widgets/cache_provider.dart#L49. didChangeAppLifecycleState can be written as an async method, but in our experience, when awaiting the client.cache.save method will not stop the app from closing.
It would seem less fragile to us, writing to cache on several occasions when the app is not about to be suspended.
For instance, debounce a client.cache.save method call everytime the inmemory cache was being written to.

We managed to setup an InMemoryCache implementation with a file based persistence backend not relying on app lifecycles, but rather persist cache when new cache writes have completed (debounced).
However, when writing big caches to file, the application's UI is jittered, since Dart is single threaded. Correct me if I'm wrong, but since file writing (IO in Dart) is happening in another thread, it must be the JSON encoding of the current inmemorycache's data to string to be able to write them to file, that is hogging all the resources.
We need to offload this encoding to another thread, when we want to persist cache during the app's lifetime.
Compute (https://api.flutter.dev/flutter/foundation/compute.html) and IsolateRunner (https://pub.dev/documentation/isolate/latest/isolate.isolate_runner/IsolateRunner-class.html) are 2 possible abstraction layers on top of Darts Isolates hat seems fit.
However, you can only pass primitives (or lists of them) as argument to the function that was offloaded, since they need to be serialized I guess. So the cache, consisting of Typed objects, is not fit to be passed as argument to the Isolate to perform some background processing on it.
Since our Dart knowledge is not sufficient, we dont know how to JSON encode the cache state in a background process (if we don't want to jitter the rendering in the main thread or if we dont want to wait to persist cache until app is about to be destroyed).
With the right advice, we're happy to contribute an alternative InMemoryCache with a different approach on persistence (still using file as a backend) avoiding incomplete cache restores.
Probably, another persistence backend like SQLite or Moor (cfr #402 ) is probably more suited, with that will be a much bigger effort. This alternative could be a better temporary solution...

Update on this issue.
We weren't able to offload the ache write to another thread. We then reverted all of our efforts and decided to keep things a they originally were setup, except for:

we don't write to cache.txt directly, because the file will be truncated as soon as the first line is being written. When the process is killed before the cache.save method can finish, you end up with an incomplete cache. We first write the cache to a temp file and when all has been written rename the temp file to cache.txt to overwrite existing cache.
we await cache restorization before building the app (during startup screen), to make sure all of our queries hit cache

This way, cache is still being persisted when the app moves to background or is being suspended, but it can never corrupt cache and we always end up with cache results if we have ever cached before.
Worst case now, is that cache was not persisted in edge cases, but this can not break user experience.

I wonder if we should could just depend on https://pub.dev/packages/sembast. It's api is async, which might be an issue.

@micimize there's also https://github.com/hivedb/hive

we await cache restoration before building the app (during startup screen), to make sure all of our queries hit cache

@Sewd Specifically how do you await cache restoration? cache.restore() appears to be synchronous, but does not restore the cache before returning.

Closed by the addition of HiveStore in v4 (4.0.0-beta.1) #648


Update shopify.extension.toml

@hheyhhay I copy pasted too much from auto complete, I'll check to see if I messed that up anywhere else


declare var in with same value statement make compiler error

Flags: -O ADVANCED
I wrote:
let a = b = c = 1

And the compiler throw:
statement.js:1:8: ERROR - [JSC_UNDEFINED_VARIABLE] variable b is undeclared
  1| let a = b = c = 1
             ^
statement.js:1:12: ERROR - [JSC_UNDEFINED_VARIABLE] variable c is undeclared
  1| let a = b = c = 1
                 ^

Though the code on plain js works correctly, the compiler still throw error
Also same with compare statement:
statement.js:3:0: WARNING - [JSC_USELESS_CODE] Suspicious code. The result of the 'gt' operator is not being used.
  3| a > b > c
     ^^^^^^^^^

I you want to do multiple declarations on one line, you still have to assign the values to each variable individually:
let a = 1, b = 1, c = 1;

The error code in the compare statement is different, it says that the result isn't used, which is the case in the given example.


error TS2345: Argument of type '{ nullable: false; onDelete: "RESTRICT"; onUpdate: string; }' is not assignable to parameter of type 'RelationOptions'.

Hi there,
My generated entities cannot compile:
C:/Applications/Node/AWS/api/src/entity/Article.ts(21,129): error TS2345: Argument of type '{ nullable: false; onDelete: "RESTRICT"; onUpdate: string; }' is not assignable to parameter of type 'RelationOptions'.

Sorry for long delay. I didn't have a lot of time for OSS lately. Can you provide some reproduction data(DDL statements)? I tried to reproduce the issue, but didn't succeed.

Closing as no response from the author.


[Python] Support creating Binary/StringView arrays from python objects

@jorisvandenbossche do you plan to add concrete classes pyarrow.StringViewArray and pyarrow.StringViewScalar like we have for other data types?


tlp: 1.3.1 -> 1.4.0

Motivation for this change
Re-do of #141782, which was reverted in #141840.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

I've verified this one by deploying to my laptop and doing a cycle of rebooting, suspend/wakes, etc. Seems to be working fine.


chore: add Node.js 10

This change is

LGTM!

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Constrain Airbyte worker memory to fraction of host total

Description/Context
The production Airbyte server is getting crushed by memory pressure due to the connector images being free to use as much memory as they want. We want to set the JOB_MAIN_CONTAINER_MEMORY_LIMIT environment variable to be approximately 1/6th of the total host memory.
This can probably be done most easily by writing a small script that gets run by cloud-init at launch to write the desired value into an /etc/default/ file path to be consumed by consul-template.
Acceptance Criteria

[ ] Airbyte doesn't get crushed by memory pressure any more

This is more complicated than it appears on the surface.
You can't set env vars equal to the output of a shell expression in the .env or in docker-compose.yml, not allowed.
You also can't do the same with /etc/default/consul-template as suggested in the description. What you can do, is create a fully static /etc/default/consul-template file using an ExecStartPre="/bin/sh echo "$(shell code to generate env var + value)" > /etc/default/consul-template"
And then in the consul template for .env. you should be able to do something like `JOB_MAIN_CONTAINER_MEMORY_LIMIT={{ env "NAME_OF_THE_VAR_YOU_CREATED_IN /etc/default/consul-template" }}

This was a red herring and is no longer blocking our use of Airbyte.


Implement redesigned filter tabs

Should clear filters display all the time?

@ibrechin i think this is good now. shall we check with flo re the clear button?


IE11 support

It looks like when trying to use this component in IE11 it throws errors for "SCRIPT438: Object doesn't support property or method 'assign' "  which tracking down the issue means that it doesn't like Object.assign()
I found a workaround if you want to try it (if you really want to support IE11) to use the object-assign package here: https://github.com/sindresorhus/object-assign instead of the native Object.assign()

I had a similar problem and fixed it by adding polyfills to add the missing methods.

It was interesting because I had polyfills instead but it must have been caching because it's working now.


Top Users Structure is Incomplete

The initial structure of the top users module is created but the codes are not finalized

it was done


Improved 'revert snapshot'

The 'host' is optional and in case is not provided will be auto-detected.
Was tested on vsphere 6.

Thanks for this! Will check out in a day or so.

On Jun 8, 2016, at 9:15 AM, Sorin Ciorceri<EMAIL_ADDRESS>wrote:
The 'host' is optional and in case is not provided will be auto-detected.
Was tested on vsphere 6.
You can view, comment on, or merge this pull request online at:
https://github.com/imsweb/ezmomi/pull/64
Commit Summary
Improved 'revert snapshot'
File Changes
M ezmomi/ezmomi.py (10)
M ezmomi/params.py (1)
Patch Links:
https://github.com/imsweb/ezmomi/pull/64.patch
https://github.com/imsweb/ezmomi/pull/64.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Looks good! Thanks again...merging...

More to come in the future.
Thanks for your work, it helped me a lot to skip 'reinventing the wheel'
Sorin
On Jun 10, 2016 8:11 PM, "Jason Ashby"<EMAIL_ADDRESS>wrote:

Looks good! Thanks again...merging...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/imsweb/ezmomi/pull/64#issuecomment-225240854, or mute
the thread
https://github.com/notifications/unsubscribe/AQmXEQADH0HUWiqnSYzP7BJd1FVa4BkVks5qKZrFgaJpZM4Iw8it
.


[SPARK-43988][INFRA] Add maven testing GitHub Action task for connect client module

What changes were proposed in this pull request?
This pr aims to added Maven testing job on GitHub Actions for the connect-client-jvm module.
Why are the changes needed?
Improve the guarantee of Maven testing success rate for connect-client-jvm module when submitting code.
Does this PR introduce any user-facing change?
No, just for test.
How was this patch tested?

Pass GitHub Action

wait https://github.com/apache/spark/pull/41487

7c0409b rebase to test

Do you think this is necessary?
For the connect server module, testing can be added after resolving SPARK-43646

friendly ping @HyukjinKwon @dongjoon-hyun

also cc @hvanhovell , I would like to add a maven test GA task for the connect modules because I think sbt testing always masks some issues(due to the visibility of the classpath)

Don't feel strongly but I think we should better set a regular job ... with a broader scope .. doing this for every PR is I think a bit of overhead.

For example, a daily job and  tests all modules using maven?

ProductAggSuite in catalyst module test aborted when using maven:
ProductAggSuite:
*** RUN ABORTED ***
  java.lang.NoClassDefFoundError: Could not initialize class org.apache.spark.sql.catalyst.expressions.codegen.CodeGenerator$
  at org.apache.spark.sql.catalyst.expressions.codegen.JavaCode$.variable(javaCode.scala:64)
  at org.apache.spark.sql.catalyst.expressions.codegen.JavaCode$.isNullVariable(javaCode.scala:77)
  at org.apache.spark.sql.catalyst.expressions.Expression.$anonfun$genCode$3(Expression.scala:200)
  at scala.Option.getOrElse(Option.scala:189)
  at org.apache.spark.sql.catalyst.expressions.Expression.genCode(Expression.scala:196)
  at org.apache.spark.sql.catalyst.expressions.codegen.GenerateSafeProjection$.$anonfun$create$1(GenerateSafeProjection.scala:156)
  at scala.collection.immutable.List.map(List.scala:293)
  at org.apache.spark.sql.catalyst.expressions.codegen.GenerateSafeProjection$.create(GenerateSafeProjection.scala:153)
  at org.apache.spark.sql.catalyst.expressions.codegen.GenerateSafeProjection$.create(GenerateSafeProjection.scala:39)
  at org.apache.spark.sql.catalyst.expressions.codegen.CodeGenerator.generate(CodeGenerator.scala:1369)


Can aqueduct server be run on a flutter app?

I need to serve clients over wireless lan where in a flutter app can act as a lightweight webserver. Can Aqueduct run in flutter?

No, there are some libraries/packages that Aqueduct imports and Flutter doesn't allow. I think Aqueduct would be overkill for this use case, and the low-level HttpServer from dart:io is a better candidate.

Would be great if you specified this on the website.


Suggestion to clarify when to use JavaSE vs Tomcat/Wildfly

[Enter feedback here]
https://azure.microsoft.com/en-us/blog/general-availability-of-tomcat-and-java-se-on-app-service-on-linux/
Somewhere on this page we should clarify when to use Java SE vs Tomcat/Wildfly .
If you look at the above blog page it says Java SE option should be used to deploy Java web frameworks like Spring, Play, Micronaut etc.. I think the heading should be re-worded or clarified to use JavaSE option to deploy .jar based applications. This was clarified by the Azure PM in an internal email thread. see below
If the Java app is a JAR file, then choose Java SE. Frameworks that product JAR files are:
Spring Book
SparkJava
Micronaut
Play Framework
Vertx
Quarkus
…
-- Asir
-- Java on Azure

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

ID: 80721d46-351f-8643-7439-e7e903d56c9a
Version Independent ID: 7dc8d5f1-8a5e-9301-98de-cedd07c61f4d
Content: Configure Linux Java apps - Azure App Service
Content Source: articles/app-service/containers/configure-language-java.md
Service: app-service
GitHub Login: @bmitchell287
Microsoft Alias: brendm

@kumarallamraju , Thanks for raising the issue. We will check the document.

@bmitchell287 , Can you please take a look at the document and update accordingly.

Thanks for the feedback @kumarallamraju! We have assigned the issue to the content author to investigate further and update the document as appropriate.

@KarlErickson - Would you please take a look into this issue please?

@JasonFreeberg - request you to help

Topic has been updated. #please-close


Question mark params not being replaced in runSql

I have some code like this:
db.runSql('INSERT INTO authGroups (id, isActive, isLocked, name, description, created) VALUES(?, TRUE, TRUE, ?, ?, NOW())', [id, name, description])

I expect that the question marks in sql will be replaced with the values from params (as the documentation describes). But the sql is being run with the question marks still there. I'm getting this error:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '?, TRUE, TRUE, ?, ?, NOW())'

These are the values of id, name, description:
id: '4dfad450-7c17-11e9-8808-0242ac170002'
name: 'Users'
description: 'Test.'

Btw, I'm using promises, not the callback function.
What's strange is if I add a callback function as the third parameter, the placeholders are replaced. This seems like a bug.
If I use this function instead of db.runSql, it works fine:
const runSql = (db, query, params) => new Promise((resolve, reject) => {
    db.runSql(query, params, (err, data) => {
        if(err) reject(err)
        else resolve(data)
    })
})

For example:
runSql(db, 'INSERT INTO authGroups (id, isActive, isLocked, name, description, created) VALUES(?, TRUE, TRUE, ?, ?, NOW())', [id, name, description])

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

https://github.com/db-migrate/mysql/blob/master/index.js#L375 That function's called from runSql and it's not quite right.  It shift the first argument (sql), then pops the last. It blindly expects the last argument to be callback.  It should probably check if the last argument is an array and, if so, treat it as params.  If I get some time I'll try to fix it and file a PR.

I am running into this bug as well. This prevents runSql from being used with Promises and async/await, which is a huge pain.


Bundle "zkctl" binary as well in the docker image of zetcd

I'm trying to use zetcd for the first time using Docker+Kubernetes.
I wanted to debug/try and see whether the zetcd proxy routes the requests to etcd servers.
However, the utility "zkctl" listed in the README.md of the project does not seem to be bundled in the docker image. It would be helpful if it is bundled in the docker image so that we can quickly verify whether the zetcd proxy is routing the requests to the server or not.

@h2002044
This is a good idea. Would you like to help to make it happen? Thanks.


CacheFrom Flag splits the value on comma

First of all thank you so much for making this plugin.
When I use cache_from as part of my plugin settings,
- name: build
    image: thegeeklab/drone-docker-buildx
    privileged: true
    pull: if-not-exists
    settings:
      daemon_off: true
      dockerfile: docker/Dockerfile.pnpm
      repo: kameshsampath/drone-ci-extension-ui-base
      cache_from:
       - "type=registry,ref=kameshsampath/docker-extension-ui-cache"
      cache_to: "type=registry,ref=kameshsampath/docker-extension-ui-cache"
      tag:
        - latest
      platforms:
        - linux/amd64
        - linux/arm64
      context: ui

I see the CLI splits the value on , and hence the resulting docker command is like,
/usr/local/bin/docker buildx build --rm=true -f docker/Dockerfile.pnpm ui --push --pull=true --cache-from type=registry,ref=kameshsampath/docker-extension-ui-cache --cache-to type=registry,ref=kameshsampath/docker-extension-ui-cache --build-arg DOCKER_IMAGE_CREATED=2022-10-29T12:57:10Z --platform linux/amd64,linux/arm64 -t kameshsampath/drone-ci-extension-ui-base:latest

When debugging the issue further I found that https://github.com/urfave/cli splits the value using , and that resulting in this erroneous command flags.  https://github.com/urfave/cli/blob/main/flag.go#L382

Thanks for reporting!

If the PR doesn't fix the issue, please ping me, then I can re-open it.


jpm sign failed with "Incorrect authentication credentials"

The command
jpm -v sign --api-key 'user:xxxxxxxx:xxx' --api-secret xxxxxxxxxxxxxxxxxxxxxxxxx

failed with the error message

Error: Received bad response from the server while requesting https://addons.mozilla.org/api/v3/addons/%40addonname/versions/0.x.y/

Version is 1.0.4
The full verbose output: (addon and file names modified)

JPM [info] binary set to /usr/bin/firefox
JPM [info] verbose set
JPM [info] Checking compatability bootstrap.js and install.rdf for xpi
Validating the manifest
JPM [info] Creating fallbacks if they are necessary..
Creating XPI
JPM [info] Creating XPI...
JPM [info] .jpmignore found
JPM [info] Adding: /path/to/file
[...]
JPM [info] Adding: /path/to/another_file
JPM [info] XPI created at<EMAIL_ADDRESS>(121ms)
JPM [info] Removing fallbacks if they were necessary..
JPM [info] Creating updateRDF...
Created XPI at<EMAIL_ADDRESS>
JPM [info] Created XPI for signing<EMAIL_ADDRESS>
JPM [debug] [API] -{ url: 'https://addons.mozilla.org/api/v3/addons/%40addonname/versions/0.x.y/',
  formData:
   { upload:
      { _readableState: [Object],
        readable: true,
        domain: null,
        _events: [Object],
        _eventsCount: 1,
        _maxListeners: undefined,
        path<EMAIL_ADDRESS>
        fd: null,
        flags: 'r',
        mode: 438,
        start: undefined,
        end: undefined,
        autoClose: true,
        pos: undefined } },
  headers: { Authorization: '<REDACTED>', Accept: 'application/json' } }
JPM [debug] [API] <- { headers:
   { allow: 'GET, PUT, HEAD, OPTIONS',
     'content-type': 'application/json',
     date: 'Thu, 17 Dec 2015 10:17:23 GMT',
     server: 'nginx',
     'set-cookie': '<REDACTED>',
     'strict-transport-security': 'max-age=31536000',
     vary: 'Accept, X-Mobile, User-Agent',
     'www-authenticate': 'JWT realm="api"',
     'x-frame-options': 'DENY',
     'content-length': '51',
     connection: 'Close' },
  response: { detail: 'Incorrect authentication credentials.' } }
JPM [error] FAIL
Error: Received bad response from the server while requesting https://addons.mozilla.org/api/v3/addons/%40addonname/versions/0.x.y/

status: 401
response: {"detail":"Incorrect authentication credentials."}
headers: {"allow":"GET, PUT, HEAD, OPTIONS","content-type":"application/json","date":"Thu, 17 Dec 2015 10:17:23 GMT","server":"nginx","set-cookie":["multidb_pin_writes=y; expires=Thu, 17-Dec-2015 10:17:38 GMT; Max-Age=15; Path=/"],"strict-transport-security":"max-age=31536000","vary":"Accept, X-Mobile, User-Agent","www-authenticate":"JWT realm=\"api\"","x-frame-options":"DENY","content-length":"51","connection":"Close"}

    at /usr/local/lib/node_modules/jpm/lib/amo-client.js:85:13
    at tryCatchReject (/usr/local/lib/node_modules/jpm/node_modules/when/lib/makePromise.js:845:30)
    at runContinuation1 (/usr/local/lib/node_modules/jpm/node_modules/when/lib/makePromise.js:804:4)
    at Fulfilled.when (/usr/local/lib/node_modules/jpm/node_modules/when/lib/makePromise.js:592:4)
    at Pending.run (/usr/local/lib/node_modules/jpm/node_modules/when/lib/makePromise.js:483:13)
    at Scheduler._drain (/usr/local/lib/node_modules/jpm/node_modules/when/lib/Scheduler.js:62:19)
    at Scheduler.drain (/usr/local/lib/node_modules/jpm/node_modules/when/lib/Scheduler.js:27:9)
    at doNTCallback0 (node.js:417:9)
    at process._tickCallback (node.js:346:13)

See also http://stackoverflow.com/questions/34332450/generating-signed-xpi-via-jpm-failed.
The params were copy-pasted. The online form signed the addon correctly.

Maybe this was related to issues #1071  and #1072. The clock was off, though by less than a minute. After the fix, it worked.


[class^=""], [class*=" "] not compatible with other webfonts

I have a project in which I am using fontawesome and simple line icons, the older code which lists out all the classes worked fine, but when I updated my fontawesome icons vanished.
Problem was this selector:
[class^=""], [class*=" "]

I wound up reverting to this commit:
https://github.com/thesabbir/simple-line-icons/blob/a8dcf0c5bd4cd0c36604e054478c021dda6aef8e/scss/simple-line-icons.scss
Could you explain why the change was made?

I agree that it's not the most elegant solution and definitely will collide when mixing font icons.

@notacouch
With latest version can you try with [class^="icon-"] and confirm if it's working.

@thesabbir I still wouldn't do that, because that would conflict with fontawesome 3.x.

One way would be changing prefix and using [class*=#{$simple-line-icon-prefix}]
Other way is back to listing all classes.

I chose the latter as other projects' markup is already using the icon- prefix.

Well, I see nothing wrong with listing all the classes.
What fontawesome does for current and older versions is use the base of the prefix as well as the actual icon class. e.g.  and . This serves a number of purposes, e.g. right now I am constructing a menu and want to change the color of icons in that menu, instead of targeting a tag, if every icon has a common class like icon, I can just select them via #menu .icon as opposed to #menu i.
In order to avoid conflicting with other libraries, instead of icon[-] you'd namespace it e.g. sli[-] (where sli is short for "simple line icon").
TL;DR
scss:
// $simple-line-icon-prefix ... becomes
$simple-line-icon-prefix-base: "sli" !default;
$simple-line-icon-prefix: "#{$simple-line-icon-prefix-base}-" !default;

// ...

//[class^=""], [class*=" "] becomes
.#{$simple-line-icon-prefix-base} {

markup:

Problem is... moves away from the simplicity that you have as well as changes the class name.

Please check out https://github.com/thesabbir/simple-line-icons/tree/notacouch

What @notacouch suggested looks like a good approach.
Although it could be simpler by simply appending a font-icon prefix (sli) to the existing classes

Looking at commit https://github.com/thesabbir/simple-line-icons/commit/ba17e6f8c42692f05ee538a96b06488cbfadae94 collision can still occur when other font icons are using something like icon-user, right?

@MichielDeMey that's true. But as @notacouch said:

Problem is... moves away from the simplicity that you have as well as changes the class name.

I'm fine with the commit, but it might be a good idea to let the developers know about the classnames (and how you can set them with the sass/less variable) in the readme?

@MichielDeMey
Please have a look https://github.com/thesabbir/simple-line-icons/tree/notacouch#customizing-lesssass-variables

So #23 it is

That's ok, anyone who knows either SASS or LESS will most likely know how those variables work.

That's better.
Thanks!
I would consider having the simplicity as the default and a namespaced version (where they have to add an additional class to their tags) as an option. This way backward compatibility can be maintained while being able to play it safe if needed.

@notacouch @MichielDeMey
Thanks!


fix(di): correct dependency ids

The misplaced dependency ids were causing route authorization to fail.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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 93.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
135

Relevant Lines:
144

💛  - Coveralls


modify the react demo in README.md

hey,  bro, recently i try to use this lib
but when i copy the react demo in the README.md file,
i find it doesn't work like what i want
i debug and find it is due to the react strict mode,
the useEffect hook would execute twice,
and it leads to the repeated Timeline instance creation:
https://github.com/ievgennaida/animation-timeline-control/assets/112551283/065dbc67-d38b-485f-98ed-57c761797bbe
so it shows two timeline in the development environment:
https://github.com/ievgennaida/animation-timeline-control/assets/112551283/e17d906e-64cc-4146-bd12-06323c1796af
when i modify the demo which is the same as this pr, it works great
so i hope that you can check my pr,
or if have any better advice, please tell me
thanks!

Hi, thank for the PR, before using useRef i would just try to dispose of a newly created timeline because it should be more than sufficient. Try to dispose on unmount new timeline instead of timeline.

Hi, thank for the PR, before using useRef i would just try to dispose of a newly created timeline because it should be more than sufficient. Try to dispose on unmount new timeline instead of timeline.

i did try to dipose timeline but it doesn't work on it,
it just unsbscribe events and clear data, but doesn't remove the canvas element

Could you please remove it from the canvas instead in the dispose? This will be a proper solution.

Could you please remove it from the canvas instead in the dispose? This will be a proper solution.

oops, my bad
now i dispose the newTimeline instead of the useState timeline, it works nice
but i think the READE.md file react demo shold be modified too,
please check my newest pr
thanks!


Add complete Protherm data and update README accordingly

Added the complete ProTherm data in its original format. Readme-file file was updated to take this into account.


Update .gitignore / final bot test

Final test

@flinkbot is @rmetzger a PMC member?

Let me ask again :)
@flinkbot is @rmetzger a PMC member?

@flinkbot what do you think about a closed PR?

@flinkbot approve all


Send your own email

Instead of triggering a verification email to go out, use the ticket system to get a new verification link and send our own email.
This is to account for a change where we only sometimes want to send this verification email message, but Auth0 currently doesn't provide means for us to do this.
See this discussion on Auth0 Community: https://community.auth0.com/t/do-not-send-email-verification-in-select-cases/92189

[ ] Get the verification link from the Auth0 API
[ ] Send that link by email
[ ] Update documentation

This expects that the verification email template is disabled.

We're moving away from Auth0 always sending email on each account creation, so this system now needs to take care of sending the initial email too. How do we keep from sending too many of these verification emails?
Should we track the sent-at time in user metadata?
The first verification email should be sent as soon as possible.
Yet we should allow users to request another email to be sent (e.g. in case they lost the first one).


KeyVault updates

Fixes https://github.com/Azure/azure-sdk-for-net/issues/42349

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Provisioning


instruction needed

this need some instruction or cmakelist file to build,
please provide some.

+1

here is the cmakelist
`#-
==========================================================================
Copyright (c) 2018 Autodesk, Inc.
All rights reserved.

These coded instructions, statements, and computer programs contain
unpublished proprietary information written by Autodesk, Inc., and are
protected by Federal copyright law. They may not be disclosed to third
parties or copied or duplicated in any form, in whole or in part, without
the prior written consent of Autodesk, Inc.
==========================================================================
#+
cmake_minimum_required(VERSION 2.8)
include the project setting file
include($ENV{DEVKIT_LOCATION}/cmake/pluginEntry.cmake)
include_directories(
${PROJECT_SOURCE_DIR}/src
${PROJECT_SOURCE_DIR}/src/eigen
)
specify project name
set(PROJECT_NAME Cmd_DefTransf)
set SOURCE_FILES
set(SOURCE_FILES
${PROJECT_SOURCE_DIR}/src/Cmd_DefTransf.cpp
${PROJECT_SOURCE_DIR}/src/DeformationTransferManager.cpp
${PROJECT_SOURCE_DIR}/src/DeformationTransferManager.h
)
set linking libraries
set(LIBRARIES
OpenMaya
OpenMayaAnim
Foundation
)
Build plugin
build_plugin()
`

here is the cmakelist
cmake_minimum_required(VERSION 2.8)
include($ENV{DEVKIT_LOCATION}/cmake/pluginEntry.cmake)
include_directories(
${PROJECT_SOURCE_DIR}/src
${PROJECT_SOURCE_DIR}/src/eigen
)
set(PROJECT_NAME Cmd_DefTransf)
set(SOURCE_FILES
${PROJECT_SOURCE_DIR}/src/Cmd_DefTransf.cpp
${PROJECT_SOURCE_DIR}/src/DeformationTransferManager.cpp
${PROJECT_SOURCE_DIR}/src/DeformationTransferManager.h
)
set(LIBRARIES
OpenMaya
OpenMayaAnim
Foundation
)
build_plugin()

here is the maya scrip with UI
import maya.cmds as cmds
import maya.mel as mel
class TransferUI:
def init(self):
self.sources = []
self.targets = []
self.ui()
def ui(self):
    self.window_name = "transferui"
    if cmds.window(self.window_name, ex=1):
        cmds.deleteUI(self.window_name)
    self.window = cmds.window(self.window_name)
    cmds.columnLayout(adj=1)
    #row = cmds.rowLayout(nc=2)
    cmds.separator(h=10)
    cmds.button(l='Add Sources', c=self.add_sources)
    self.list_src = cmds.textScrollList()
    #cmds.setParent('..')
    cmds.separator(h=10)
    cmds.button(l='Add Targets', c=self.add_targets)
    self.list_tgt = cmds.textScrollList()

    cmds.separator(h=10)
    cmds.button(l='Transfer', c=self.transfer)

    cmds.showWindow(self.window)
def add_sources(self, *args):
    sl = cmds.ls(sl=1)
    cmds.textScrollList(self.list_src, e=1, a=sl)
    self.sources += sl

def add_targets(self, *args):
    sl = cmds.ls(sl=1)
    cmds.textScrollList(self.list_tgt, e=1, a=sl)
    self.targets += sl

def transfer(self, *args):
    src = ','.join(self.sources)
    tgt = ','.join(self.targets)
    mel.eval('defTransfer -src "{}" -tgt "{}";'.format(src ,tgt))

TransferUI()

thanks a lot. A question, how is the workflow with different vertex count or not vertex wise correspondence? the page is addressing it with a non-rigid-registration....is that already included?

It is not working as expected, for example I couldn’t find the proper implementation of markers in the source code.

@kenculate, did you used mesh with vertex wise correspondence?

@kenculate, thank you i found it out.....

he method requirespoint-wise correspondences between source and target shapes. Localshape deformation is then represented using deformation gradients,which are transferred from the source to the target shapes.

@kenculate do you have the ".mll" file

yap, I will upload somewhere.

@kenculate thank you

https://srv-file2.gofile.io/download/sRCE8k/Cmd_DefTransf.mll

https://drive.google.com/open?id=1fOT8OU2ErJ9jO7hCB0Ncg2jBYSZ025RX

@kenculate agha merci

moteasefane natunestam contacti az shoma peida konam, ye dusti dare talash mikone ye hamchin kari ro anjam bede va hamchenan movafagh nabude, age momkene contactetun ro baraye man beferestin ta baham connectetun konam shayad be rah hal moshtarak beresin!


Will not deploy file if it has space in the name

Hi @charlesbedrosian,
Thanks for reporting, we'll definitely find a solution for this!

Hey @charlesbedrosian! 👋
Just checked the step multiple ways, and looks like it is working as intended. Please see the results below. Could you please provide more info about the build you had this issue?
Thanks!
+------------------------------------------------------------------------------+

| (1<EMAIL_ADDRESS>                                             |
+------------------------------------------------------------------------------+
| id: deploy-to-bitrise-io                                                     |
| version: 1.3.7                                                               |
| collection: https://github.com/bitrise-io/bitrise-steplib.git                |
| toolkit: go                                                                  |
| time: 2017-09-25T10:25:53-07:00                                              |
+------------------------------------------------------------------------------+
|                                                                              |
Configs:
- BuildURL: https://www.bitrise.io/build/643ae0386df50b2a
- APIToken: _nwxh5Vd92Yk0HXTYdyPkg
- IsCompress: false
- DeployPath: /Users/vagrant/deploy
- NotifyUserGroups: everyone
- NotifyEmailList:
- IsPublicPageEnabled: true
Deploying the content of the Deploy directory separately
List of files to deploy
- /Users/vagrant/deploy/filewith spaceinthename.txt
Deploying files
Uploading file: /Users/vagrant/deploy/filewith spaceinthename.txt
creating artifact
  file size: 8B
uploading artifact
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100     8    0     0  100     8      0     49 --:--:-- --:--:-- --:--:--    49
finishing artifact
is_enable_public_page is set, but public download isn't allowed for this type of file
Success
You can find the Artifact on Bitrise, on the Build's page: https://www.bitrise.io/build/643ae0386df50b2a
|                                                                              |
+---+---------------------------------------------------------------+----------+
| ✓ |<EMAIL_ADDRESS>                                  | 7.77 sec |
+---+---------------------------------------------------------------+----------+

I am closing the issue now, as it still seems to be working fine.
Please feel free to re-open if you encounter any issues!