Problem with styles when not using CSS Modules

I believe It's better to use plain css / inline in order to not cause problems with project not offering css modules.

Use this css file. I gathered it from the one in the repository. don't forget to fix the extension to ".css"
evan-calender-style.css.txt

If you're not using css-modules, as stated in the README you can pass a css-module-ish object:
const styles = {
    referenceClassName: 'classNameInYourCSS',
    ...
};

You can find all the reference CSS classes here

Hey guys, amazing job, thanks.
I think styles object should pass over "calendarStyles" prop not styles according to mentioned code line below.
https://github.com/evandhq/react-persian-datepicker/blob/master/src/components/DatePicker.js#L144


Ubuntu WSL2 is broken when opening from Run

Windows Terminal version
1.18.3181.0
Windows build number
10.0.19044.3930
Other Software
No response
Steps to reproduce
If I open Ubuntu from Terminal:

But if I open Ubuntu from Run command:

Expected Behavior
I expected no difference in opening Ubuntu WSL2 from a tab in Terminal or from a Run command.
Actual Behavior
Terminal behaves differently based on how you start the Ubuntu WSL2 shell.

Thanks for filing! Looks like this is caused by the existence of two ubuntu profiles (one provided by us and one by Ubuntu). We're actually tracking that over in #12582. So we'll mark it as a \dup of #12582.
In the meantime, we recommend you move your font settings to the "defaults" section. That should be a nice workaround.


[MIST-1028] Allocate MIST tasks after MIST master allocation is finished

This PR resolves #MIST-1028 via

Allocating MIST task evaluator request after MIST master evaluator allocation is done in MistDriver

I've tested it on Yarn cluster and it works well! I will merge this PR.


Cannot find pretrained model

Did you upload the pretrained model of "vggish_pca_params-970ea276.pth"?

Hi, you could ignore this pretrained model since we do not employ it during audio feature extraction.


Add Moonbeam mainnet network support

Endpoints for verification:
Moonbeam Chain ID (1284) : https://rpc.api.moonbeam.network
Block Explorers:
https://docs.moonbeam.network/builders/tools/explorers/
CC: @rmeissner

Hi @rmeissner , how can I add you as a reviewer? Thanks.

Hi @rmeissner , do I need to do something else? thanks

This PR is still making more changes than expected. It should only add the addresses for the moonbeam chain id, no other files should be touched and no other networks removed. Until this is resolved this cannot be merged.

Perfect, I'm going to close this PR and I'll create a new one cleaner.

I'll create a more cleaner PR


preprocessing

Hi, sorry for disturb you again.
I am just wondering what is the purpose of enhance_marker and marker.py. I did preprocess.sh and should I do this enhance_marker afterwards?

func of enhanced marker is for better representation of entity by adding entity type instead of only putting start and end around the entity word. You should put it before model training if u decide to use this. More enquiry please feel free to email me<EMAIL_ADDRESS>


Passing undefined to value prop turns component into controlled mode and breaks

Hi, I've found the issue with the way of handling value in react-quill. Passing undefined breaks component. This situation happens if you create wrapper around <ReactQuill /> and you want to pass value explicitly without assigning value prop in wrapper itself (eg to limit customisation).
example (click button to see the error):
https://codepen.io/coderitual/pen/MZXGNe
I think this line is causing this:
https://github.com/zenoamaro/react-quill/blob/master/src/component.js#L142
Would be good idea to mimic standard behaviour as in react docs:
https://reactjs.org/docs/forms.html#controlled-input-null-value
If you pass null or undefined your component stays uncontrolled.
Is there a bug in Quill?
No
React-Quill version

[x] 1.3.3

I see this in version 1.3.5 too.

As a workaround, you can avoid the problem by making your wrapper component control the editor if no value prop is provided from the parent.
const ReactQuillWrapper = (props) => {
  const [uncontrolledValue, setUncontrolledValue] = useState("");
  const value = props.value ?? uncontrolledValue;

  const onChange = (newValue) => {
    if (typeof props.value === "undefined" || props.value === null) {
      setUncontrolledValue(content);
    }
    if (props.onChange) {
      props.onChange(newValue);
    }
  }

  return <ReactQuill value={value} onChange={onChange} />
};


Run terminal view via docker?

Is it possible to run the terminal view through docker?
I'm trying this:
docker run --rm -it -e TERM -v $MW_PATH/logs/:/logs goaccess --log-format caddy -c /logs/caddy_access.log

But I see nothing.  What am I doing wrong?
I would mention I'm also trying this on iterm2 on Mac but over SSH to AWS workstation...

Ok.  Just the -c is not working.  Once I removed that everything is working!!


When loading cached Node.js project display name of the project

Would be interested to known which project is loading now, so I could see the progress. I have relatively slow PC sometimes and it is interesting to know how much time left until VS started.

Doesn't this already display in the status bar?

Actually not. VS just display other statistics, like amount of functions loaded from analysis. But all that displays without indication of project name and after the project loaded. But when working with slow loading projects (which is any project withinn NTVS for me) would be good display project name before it start loading, then when I wait for it to load I will know which one project is slow.
Not sure if this would be realiable due to async nature of VS but would like give it a try. Please reopen

Gotcha. OK. Thanks.

This is a feature for Visual Studio, and not limited to Node Tools. Unfortunately they weren't interested in taking this at this time.
You're best bet at this point would be to open an issue on the Visual Studio developer community


Fix Windows importer issue with new file detection

fixes https://github.com/godotengine/godot/issues/96397
The importer was not detecting new files in the root directory due to handling of trailing slashes.
Related to changes from https://github.com/godotengine/godot/pull/91902
Tested with the mrp from https://github.com/godotengine/godot/issues/91413

I confirm that this PR fixes the issue. Thanks!

Thanks!


Typescript 3.0 Support

This has just released, any plans to implement it in SerenityJS?
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-0.html

Hey @marktigno! Yes, it's already on the 2.0 branch :-)

Closing this one. It seems the current version is working with TS v3.0+. Thanks! :)


remove xx dependencies

can't find poetry or pyenv after running ubuntu_setup now, I'll check it out.


fix(select): clickable caret icon to toggle the dropdown

Fix and Enhancement: Toggle Dropdown and Custom Caret Icon
This PR addresses an issue and introduces an enhancement to the Select Dropdown component.
Fix:

Resolved a bug where the dropdown did not toggle when clicking the caret icon.

Enhancement:

Added the ability to customize the caret icon using the new "caret" slot scope. Developers can now easily replace the default caret icon with their own custom icon.

Changes:

Modified the behavior of the caret icon to toggle the dropdown when clicked.
Added a new "caret" slot to the Select Dropdown component to provide customization options for the caret icon.

This enhancement improves the user experience by allowing users to toggle the dropdown not only by clicking on the select element but also by clicking on the caret icon. Furthermore, it empowers developers to customize the appearance of the caret icon to better match their application's design.

@adenvt The caret icon should function as a toggle, just like on https://vue-multiselect.js.org/. If we only use "pointer-events: none" in the CSS, when we click the caret icon, the toggle function won't work. It will only open without the ability to close again.
And the "slot-scope" attribute can be used to customize the caret icon. It allows you to define a custom icon for the caret and gives you more control over its appearance and behavior.

@adenvt The caret icon should function as a toggle, just like on vue-multiselect.js.org. If we only use "pointer-events: none" in the CSS, when we click the caret icon, the toggle function won't work. It will only open without the ability to close again.
And the slot-scope attribute can be used to customize the caret icon. It allows you to define a custom icon for the caret and gives you more control over its appearance and behavior.

@radyakaze Yeah, that make sense.
But i'm very satisfied with prop caret. Maybe you should change it to no-caret. Similar to Dropdown component, it's use for hide the caret icon. On click behaviour keep it as is.

Mas @adenvt , for the caret, I've added a slot-scope instead of using props, and this is different from no-caret. The slot-scope caret is used for a custom caret icon.
And now, I've also added no-caret prop. 😀


AU's 'free electronic resources' should have full text online button, no lock

Holding in the location AU Electronic Free Resources like this item are open to anyone 7753323. Instead of just hyperlink and AU Only with lock icon, show a full text online button, no lock or restriction note.
This example also seems to break the columns, because the Request link is shifted over to the right.

addressed show available to all in m27_001


Logo proposal

Greetings, @Et0h
I am a designer in development and an open source enthusiast who would love to contribute to your project by designing a logo for it. I found your project exploring GitHub and found it interesting. About the process, we would be working together to create the logo that fits best with your project. The process is, of course, totally free and you can quit whenever you want. In case you agree, it would be useful if you had any ideas (shapes, colours, etc) to have something to start with.
Best regards!

Thanks for the offer @michaelizergit
Catlinman has designed us a logo icon available from https://twitter.com/catplease/status/966485574218592258 which was approved a while back - what do you think?
(I guess we should really get around to implementing it...)


postcss config validation

Hi all,
Is there any issues with my config for the postcss ??
{
  test: /\.css$/,
  use: [
    {
      loader: 'style-loader',
    },
    {
      loader: 'css-loader',
      options: { sourceMap: true, importLoaders: 1 }
    },
    {
      loader: 'postcss-loader',
      options: {
        sourceMap: true,
        // https://github.com/postcss/postcss-loader/issues/92
        // https://github.com/postcss/postcss-loader/issues/8
        plugins: () => [
          precss(),
          postcssImport({ addDependencyTo: webpack }),
          postcssNested(),
          postcssCssnext({
            browsers: ['last 2 versions', 'ie >= 9'],
            compress: true,
          }),
        ],
      },
    },
  ],
}

The hotloading feature doesn't work, I know I have to put that to the first line of the plugin, but if I put before the precss, I got an compilation error

You can reproduce the issue in this repo
https://github.com/iroy2000/react-redux-boilerplate .

{
      loader: 'postcss-loader',
      options: {
+       ident: 'postcss', // <= This line (fix)
        sourceMap: true,
        // https://github.com/postcss/postcss-loader/issues/92
        // https://github.com/postcss/postcss-loader/issues/8
        plugins: () => [
-          precss(), // deprecated, unmaintained (additionally)
-          postcssImport({ addDependencyTo: webpack }), // @import is handled by `css-loader` (additionally)
          postcssCssnext({
            browsers: ['last 2 versions', 'ie >= 9'],
            compress: true,
          }),
        ],
      }

Actually it doesn't seem fixing my problem, I still see the same thing
{
  test: /\.css$/,
  use: [
    {
      loader: 'style-loader',
    },
    {
      loader: 'css-loader',
      options: { sourceMap: true, importLoaders: 1 }
    },
    {
      loader: 'postcss-loader',
      options: {
        ident: 'postcss',
        sourceMap: true,
        // https://github.com/postcss/postcss-loader/issues/92
        // https://github.com/postcss/postcss-loader/issues/8
        plugins: () => [
          postcssCssnext({
            browsers: ['last 2 versions', 'ie >= 9'],
            compress: true,
          }),
        ],
      },
    },
  ],
}

And @michael-ciniawsky , did you get this working with this ---> https://github.com/iroy2000/react-redux-boilerplate ?

The warning you get is bc 'postcss-import' is unable to find/resolve configs/color, the path seems to be simply incorrect. In your boilerplate you use precss (Includes a nesting plugin), postcss-nested (dito) && cssnext (Includes a nesting plugin aswell), so you definitely should choose one nesting solution. PreCSS && CSSNext are just wrappers around PostCSS Plugins. The same goes for cssnext, check their package.json for plugin duplicates. Also if you really want to use postcss-import (it will cause duplicates in your bundle) then use it as the first plugin and not after precss && postcss-nested

Thanks, that helps a lot.

postcss-partial-import supports _partial.css (SASS) import style to my knowledge, but...
as long as you don't 💯  need to migrate a SASS code base to PostCSS free yourself from SASS conventions whenever possible as you won't really need them anymore 😛


Reveal the automated opened file in the "Changes in PR" view

Testing #2136
When entering focus mode, a file automatically opens (the first diff it seems). This file should be selected in the changes tree also:

This has since been fixed.


*** Critical Security Bug with Electrum BTCP Wallet and Ledger Nano S ***

Today I set up the Bitcoin Private Electrum Wallet v1.1.1 with my Ledger Nano S.
When opening up the wallet without my Ledger Nano S device connected, I get the following dialog box:
Please insert your Ledger.  Verify the cable is connected and that no other application is using it.
Try to connect again?
Yes  /  No
When I click "No", I am able to go into my wallet without my Ledger device!  So anyone can get into my wallet without the device and send my BTCP away without my authorization!
Another (less critical) issue is that you can see my available balance without the Ledger being connected.  This should be only visible when the Ledger is connected.  See the attached picture.

Can you actually sign a txn and send funds out of electrum without your ledger plugged in?

I was NOT able to sign the transaction without my Ledger.
But I agree that the wallet reveals too much info without being signed in.
You can close the ticket.  Sorry if I caused panic!


Fix #3124 - Move enums to top level schema definitions

This fixes #3124 by moving enums to top level schema properties, original inspiration from #2996.
@frantuma I know the tests are likely going to break because this changes how enums work.  I was hoping to get your feedback on how to make this configurable, like #2996 originally proposed.  Should it be an annotation on the class or a configuration property of the OpenAPI?

@mtraynham thanks for looking into this! I share here some thoughts.

I guess the changed code is not sufficient (I also don't understand the change to Components) to address the issue; we would need to convert the resolved enum into a ref and add that to the components sections, as done for non primitive types (depending on scenario); also specific tests would need to be added

About the way to control behaviour, I can see 2 ways, independent from each other:

A. implement a new annotation e.g @EnumAsRef(name = "")` and check it in code for point 1 above.
B. add it as a Configuration member; this is more complicated, as atm configuration is not passed down to ModelConverter instances (e.g. ModelResolver) and related classes; we would probably need to slightly refactor converter related code to gain access to config property; the alternative based on shared static variables sounds more limited and not really consistent, also not applicable e.g. in maven plugin or in scenarios with no java code related to swagger-core config.

@frantuma heh, I also do not entirely understand why the change in #2996 to Components is necessary, but it does seem to be.  I removed it and Swagger instead produces an empty block for enums with no properties.
I'm not against the @EnumAsRef or maybe represented as @Schema(enumAsRef=true), my only concern there is consistency throughout a user's API.  But, I also am unfamiliar with how this code works, so it's likely the safer route for me than configuration changes.

@mtraynham What is the current state of this feature? Thanks.

@xiaoxiangmoe This works, but I haven't really heard back from @frantuma.

Alright, I think the initial implementation was a bit strange, since it had a Components change which nulled the $ref.  But after following some of this recursion in ModelResolver, I think my latest push is a better implementation.  It simply adds the enum model as a top level schema to the context and returns a simple Schema with a ref.  I reverted the Components change and added a test for this.

By the way, I have found that if you have an enum that uses a different method of serializing and deserializing with say Jackson, such as an integer enum, you can accomplish top level enums in a similar fashion with:
    @Schema(enumAsRef = true, type = "integer", allowableValues = {"1", "2"})
    public enum NumberedEnum {
        NUMBER_1(1),
        NUMBER_2(2);

        private final Integer value;
        NumberedEnum(Integer value) {
            this.value = value;
        }

        @JsonCreator(mode = JsonCreator.Mode.DELEGATING)
        public static NumberedEnum fromValue(Integer value) {
            for (NumberedEnum other : NumberedEnum.values()) {
                if (other.value.equals(value)) {
                    return other;
                }
            }
            throw new IllegalArgumentException("Unsupported value " + value);
        }
    }

Is there any feedback on this pr? @frantuma

@frantuma Please have a look. Thanks.

@frantuma any updates on this?

It's in our radar, we plan to have it in for next release

@frantuma When will the next edition be released?

No exact ETA, but within 2 weeks

@frantuma any updates on this?

Thanks a lot! this is included and replaced by #3352 which adds also global support via sys prop / static var


changed the documentation endpoint to index.html

IBM Cloud toolchain: Delivery Pipeline deployed wishlists to dev, including commit 96cdd8ac03ff92f46682da0e49f1a38b48c07c9c


Lighthouse Performance issue

Bug report
Describe the bug
I ran the 'npm run build' command, and then put my site live through 'now' command. When I run a Lighthouse audit on my website, I'm currently getting a performance hit, because of the following:
"Consider using <link rel=preload> to prioritize fetching resources that are currently requested later in page load".
I can see the CSS is being loaded in the  tag as follows: .
Is there a way to "resolve" this performance issue?
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Go to https://website-thomas.thomasledoux1.now.sh/
Run a Lighthouse audit
Check the suggestions for Performance

Expected behavior
With the default 'build' command and setup, I would expect the CSS to be loaded as efficiently as possible.
Screenshots
https://snipboard.io/HCwVSX.jpg
System information

OS: Windows
Browser (if applies) Chrome
Version of Next.js: 9.2.2

Looks like nothing on the page is pre-rendered 🤔 Can you provide a repository.


kubectl apply -f azure-vote.yaml results in following error

error: error validating "azure-vote.yaml": error validating data: ValidationError(Deployment.spec): missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec; if you choose to ignore these errors, turn validation off with --validate=false

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

ID: 6115d555-45a1-6539-4af6-fbc20a99f008
Version Independent ID: 21f80c83-ba10-9b7e-c54f-51a9c140517c
Content: Quickstart - Azure Kubernetes cluster for Linux
Content Source: articles/aks/kubernetes-walkthrough.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

adding
selector:
matchLabels:
app: azure-vote-front
to the deployment sections corrects this error.

Thanks for the feedback! We are currently investigating and will update you shortly.

@redshiftz thanks for this.
@iainfoulds I confirmed I get the same error running through the doc as and as @redshiftz mentioned adding
selector:
matchLabels:
app: azure-vote-front
seems to fix the errors. Can you take a look?

Yep, sorry about that @redshiftz. I was already working on an update for this and a couple of other docs. I updated the manifests to use apps/v1, but didn't get the selector in place. This doc update should publish this afternoon.
#in-progress

@redshiftz This update has merged and should publish in the next couple of hours.
@MicahMcKittrick-MSFT #please-close

Thanks @iainfoulds!

Thank you both!
On Wed, Oct 17, 2018 at 3:19 PM Micah<EMAIL_ADDRESS>wrote:

Closed #17075 https://github.com/MicrosoftDocs/azure-docs/issues/17075.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/17075#event-1910599946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATr_5QUsR0CN3Rghvt3zwFwgG04rz8lWks5ul5DKgaJpZM4Xkwfw
.


fix markdown rendering of headings with IDs

Move the newline after the heading text to after the ID.

Cleaned up (gofmt'ed) and pushed.


Can't centre images in "Image Mode"

Test 1

Create a container with both width and height set at 100%
Append an img containing a small image
Apply ImageViewer to the image (Image Mode)

The image sits top left instead of being centred
Test 2

Create the same setup as above
Set both width and height of iv-container at 100% in hope it will keep all the sub-divs centred

Images are now centred, but they're stretched to fill the screen

Same problem in container mode

For Test 1,
ImageViewer in image mode does not care about its container, instead, it wraps image inside an image-viewer container. That must be disturbing your centered aligned image. Instead of positioning image you can position .iv-container, something like
  .iv-container{
    position : absolute;
    transform : translate(-50%,-50%);
    top : 50%;
    left : 50%;
  }

For Test 2 it shouldn't happen, I will try to reproduce it though. It will be helpful if you can share any jsfiddle or codepen

Thanks. I've just tried what you recommended for test 1, but it didn't achieve what I wanted, because the container is then the size of the image, so that's not the right approach for what I'm trying to do which is to start with the image at normal size and then zoom in so that it fills the container.
I probably won't be able to easily create a code sample for test 2, but I'll try to describe it.

So here is the setup for test2

and the code used
var container = $('#slideshow')
var imageContainer = container.find('.imageContainer')
var viewer = ImageViewer(imageContainer );
viewer.load(image.src);

The slideshow CSS
#slideshow {
	position: fixed;
	left: 0;
	top: 0;
	width: 100%;
	height: 100%;
	z-index: 100000;
	display: none;
	background-color: black;
	background-position: center center;
}

The large image has the following style applied to it: width: 918px; height: 918px, but the original image is 50x50.

The problem comes from this line which stretches the image to fill the container:
https://github.com/s-yadav/ImageViewer/blob/master/imageviewer.js#L608

I'm closing this as the answer given works for centring an image.
I've opened an new issue for the problem when using a container: https://github.com/s-yadav/ImageViewer/issues/7


Use only one master node in experiments

Since the experiments use the master/worker paradigm it makes sense to use only one master and N worker, so having the ability to choose the number of master nodes when running a test is pointless.

Closed by #14


Synology camera verify SSL problems

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
.32.1
Python release (python3 --version):
3.5.2
Component/platform:
camera/synology
Description of problem:
verify_ssl is passed into requests, where I believe it should be verify. I am getting this exception
Expected:
Use the correct kwarg name
Traceback (if applicable):
Traceback (most recent call last):
  File "/srv/hass/lib/python3.5/site-packages/homeassistant/helpers/entity_component.py", line 143, in _async_setup_platform
    entity_platform.async_add_entities, discovery_info
  File "/srv/hass/lib/python3.5/site-packages/homeassistant/components/camera/synology.py", line 75, in async_setup_platform
    verify_ssl=config.get(CONF_VERIFY_SSL)
  File "/srv/hass/lib/python3.5/site-packages/aiohttp/client.py", line 403, in get
    **kwargs))
TypeError: _request() got an unexpected keyword argument 'verify_ssl'

No, this is not a problem with requests as this platform is not using requests. It is using aiohttp.
It looks like that if we want to disable SSL verification, we need to create a new session with it disabled.
verify_ssl = config[CONF_VERIFY_SSL]

if verify_ssl:
    websession = hass.websession
else:
    websession = aiohhtp.ClientSession(connector=aiohttp.TCPConnector(verify_ssl=False))

# Now use websession instead of hass.websession for requests

A PR for this is welcome.


Wrong Docs URL on Data Exporter - Python Goocle Cloud Storage

Describe the bug
Wrong URL for the python data exporter regarding google cloud storage.
To Reproduce
Steps to reproduce the behavior:

Load a new data exporter selecting python and google cloud storage
Click on the Docs: https://github.com/mage-ai/mage-ai/blob/master/docs/blocks/data_loading.md#googlecloudstorage
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

👀

Thank you for reporting, will fix asap.

Hi @ReynaldoMR3, the links should have been fixed in https://github.com/mage-ai/mage-ai/pull/1696. Can you update to the latest version of Mage (v0.7.60+) and see if new blocks have working docs links? Existing block files in your project may still have the old link since those retain the same content.
For reference, you can check the docs here for info on the GCS data exporters.

Thanks! Everything looking fine 👍


int32_t is a long int

In static Value vm_intToStr(VM* vm, int32_t i):
size = MVM_SNPRINTF(buf, sizeof buf, "%d", i);
must be:
size = MVM_SNPRINTF(buf, sizeof buf, "%ld", i);

Ah yeah, thanks. Should be fixed now!

The correct way of doing this is with the PRId32 macro. On LP64 platforms, coercing to long is inefficient.

Ah yeah, good call. Thanks @davidchisnall!


feat: update functions like transfer sell and burn in NFT Details

Please pull changes from staging @louisdevzz


Add AMactorId to the C API

@orionz, I've exposed the automerge::ActorId struct to C so that the logic to convert from a hexadecimal string to bytes and vice versa wouldn't be repeated for multiple structs and so that a client application/library can synthesize a conforming actor identifier without guesswork.
@lightsprint09, this PR removes AMgetActorHex(), AMsetActorHex() and updates AMgetActor(), AMsetActor() and AMchangeActorId().

What's the reasoning behind carrying around a cached string representation of the actor ID? Is that idiomatic in C? It feels strange to me because it forces a bunch of memory use on the user when they may not care at all about hex encodings of the actor.

What's the reasoning behind carrying around a cached string representation of the actor ID?

@alexjg, it's there so that AMactorIdStr() can return a const char* (the hexadecimal string) instead of taking char*(a buffer into which the hexadecimal string is to be copied) and size_t (the length of the buffer) arguments and returning a 'size _t` (the length of the hexadecimal string so that one can detect when the buffer is too small).
This simplifies API usage in the following ways:

The alternate approach requires two AMactorId() calls--first with a NULL char* argument to retrieve the length of the buffer to allocate and second to copy it--because the length of the hexadecimal string varies between ActorIds.
The alternate approach requires a subsequent free() call (for the buffer) in addition to the subsequent AMfree() call (for the AMresult containing the ActorId) or it will leak the copy of the string.

Is that idiomatic in C?

No.

It feels strange to me because it forces a bunch of memory use on the user when they may not care at all about hex encodings of the actor.

Only if the user actually calls AMactorIdStr() on the given AMactorId, otherwise it only forces a RefCell<Option<ActorId>> of memory use on the user.


Edits to unicon docs based on Amr's feedback

@marcomonaco The edits here should address https://mesosphere.atlassian.net/browse/DCOS-9731
@somikbehera At Amr's request I've changed the term "experimental" to "technical preview". Do you approve?

LGTM


vscode version

please make ths plugin available on Vscode please.

yea, i got simple-ftp and its better then this remote-ftp

yea, i got simple-ftp and its better then this remote-ftp

Which one are you using? I couldn't find anything that 'just works' in the way remote-ftp does. I really liked how remote-ftp would save a config file locally in the directory you're working in, and then just automatically use that to connect to the right host with the right settings.
All the FTP extensions for VSCode seem to require some kind of global config file and custom paths and whatnot, which is a huge pain when you're managing 20 different websites.


update README with correct instructions

Currently, the instructions in the README are broken.
This PR fixes the instructions so they work.

Thanks for the improvements!


Change "Kensho Technologies, Inc." to "Kensho Technologies, LLC."

Coverage remained the same at 90.22% when pulling ba30a71bbbe131fd71fe68fd275f6470be2f107f on change_company_name into b21a6193e2c0f8aae13a7748726a01600e2cd9a7 on dev.


[SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

What changes were proposed in this pull request?
Add CreateViewStatement and make CREARE VIEW  go through the same catalog/table resolution framework of v2 commands.
Why are the changes needed?
It's important to make all the commands have the same table resolution behavior, to avoid confusing end-users. e.g.
USE my_catalog
DESC v // success and describe the view v from my_catalog
CREATE VIEW v AS SELECT 1 // report view not found as there is no view v in the session catalog

Does this PR introduce any user-facing change?
Yes. When running CREATE VIEW ...  Spark fails the command if the current catalog is set to a v2 catalog, or the view name specified a v2 catalog.
How was this patch tested?
unit tests

Test build #114334 has started for PR 26649 at commit 45dcf5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/19173/
Test PASSed.

Test build #114334 has finished for PR 26649 at commit 45dcf5a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114334/
Test FAILed.

Retest this please.

Test build #114356 has started for PR 26649 at commit 45dcf5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/19195/
Test PASSed.

Test build #114356 has finished for PR 26649 at commit 45dcf5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114356/
Test PASSed.

Test build #114418 has started for PR 26649 at commit 65b09be.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/19255/
Test PASSed.

Test build #114427 has started for PR 26649 at commit e878207.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/19263/
Test PASSed.

Test build #114418 has finished for PR 26649 at commit 65b09be.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
sealed trait ViewType

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114418/
Test PASSed.

Test build #114427 has finished for PR 26649 at commit e878207.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114427/
Test PASSed.

thanks, merging to master!

Thanks! @viirya @cloud-fan


Test playbooks on Ubuntu 21.04

Ubuntu 21.04 is due out this week. There is a beta available already. It would be good for us to verify whether the playbooks work properly on this new version of the OS, and also include it in the list of OSs deployed by the DockerStatic role (possibly replacing 20.10)

[ ] Check if the existing playbooks run on 21.04
[ ] Make any changes required for running on 21.04
[ ] Add to VagrantPlaybookCheck
[ ] Add to DockerStatic role

Details:

First hurdle

It also seems to have a problem with the NTP time role

iirc, ubuntu 20.04 had this same problem

ref https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1226

GCC role has a guard to stop it executing on Ubuntu 20 - needs to be modified to say >=20
https://github.com/AdoptOpenJDK/openjdk-infrastructure/blob/a4d8543253f2cf3edbe9b606b809a3ad5fd6beab/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Common/tasks/Ubuntu.yml#L89
NTP may be similar but to clarify did we get a solution that didn't just involve disabling NTP on 20.04?

Can't you use timesyncd for Ubuntu?

NTP may be similar but to clarify did we get a solution that didn't just involve disabling NTP on 20.04?

https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1340
https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1226#issuecomment-634068286 is the reason why the Set timedatectl set-ntp no task of the NTP role fails

Yeah looks like another case where we've done == 20 instead of >=20

Yeah looks like another case where we've done == 20 instead of >=20

Same with https://github.com/AdoptOpenJDK/openjdk-infrastructure/blob/a4d8543253f2cf3edbe9b606b809a3ad5fd6beab/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Common/tasks/Ubuntu.yml#L84, as there isnt gcc-4.8 or g++-4.8 package available for ubuntu 2104

Besides the gcc and ntp tasks, the rest of the playbook runs fine. Im running builds and tests on the machine to test if everything looks good

@sxa Is there a reason the OpenSSL102 role doesnt run on ubuntu? The OpenSSL role does but it isnt included in the top level main.yml, and doesnt install the latest openssl

@sxa Is there a reason the OpenSSL102 role doesnt run on ubuntu? The OpenSSL role does but it isnt included in the top level main.yml, and doesnt install the latest openssl

The OpenJ9 minimum version is 1.0.2. That role is only needed on distributions which have an earlier version included by default.

You're so kind :joy:

Interesting issue with the Vagrantfile:
11:14:36 + ssh -p 2202 -i /home/vagrant2/workspace/VagrantPlaybookCheck/OS/Ubuntu2104/label/vagrant/adoptopenjdkPBTests/willsparker-2143/ansible/id_rsa<EMAIL_ADDRESS>'cd /vagrant/pbTestScripts && ./buildJDK.sh --branch master --fork AdoptOpenJDK --version jdk8 '
11:14:36 + tee /home/vagrant2/workspace/VagrantPlaybookCheck/OS/Ubuntu2104/label/vagrant/adoptopenjdkPBTests/logFiles/willsparker.2143.Ubuntu2104.build_log
11:14:38 bash: ./buildJDK.sh: /bin/bash: bad interpreter: Invalid argument

Not sure what the issue is, but it works when I try it on the VM manually, and it works when I run ssh ...etc 'bash /vagrant/pbTestScripts/buildJDK.sh ...etc' - I've got a commit in that does that instead, and tests all the other platforms to ensure it doesn't break anything - I doubt it will, but better safe than sorry: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1158/

With #2169 merged, the only thing left is adding it to the DockerStatic role, going off the list in the issue description.

Ill work on adding ubuntu 21.04 to the DockerStatic role right now

I think this can be closed now, with #2189 in, right? :-)

I think this can be closed now, with #2189 in, right? :-)

@sxa ^^

Can be closed as https://github.com/adoptium/infrastructure/pull/2189 has been merged


Module replacements not able to resolve nested modules

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.92.2+extended linux/amd64 BuildDate=unknown

Does this issue reproduce with the latest release?
Yes
The issue
I have the following in my config.yaml.
module:
  replacements:
    - github.com/UtkarshVerma/hugo-yoru -> ../../hugo-yoru
  imports:
    - path: github.com/UtkarshVerma/hugo-yoru

Now, the hugo-yoru module depends on FontAwesome. When I run Hugo server, I get the following error:
Error: module "github.com/FortAwesome/Font-Awesome" not found; either add it as a Hugo Module or store it in "/home/subaru/GitHub/bitbanged.com/themes".: module does not exist

Thanks in advance.

This is a limitation of replacements, I'm afraid, and not something I think that's easily solvable. The replacements just replaces the path value, nothing else.
I would suggest that you use the replace directive in go.mod. Note that all of this will (hopefully) be much improved in a month or so with Go 1.18's workspaces.


[FR] possibility to use .java files directly In apps or modules

I would be great to be able to do this. Right now you have to create a jar or car which mean maintaining another lib in another project/environment.
I don't think it would be that hard as it would just mean copying the java file the correct build dir where you put the generated java files.
Or maybe use in gradle:
sourceSets {
    main {
        java.srcDirs = ['src/main/java']
    }
}

Hey @farfromrefug, I've been away from the project for a while, but can't you do this in App_Resources/Android/src/main/java ? So long as the file is in the correct namespace it should be alright?

@petekanev tried that but it did not seem to work:
I created a java file like this:

I also added this in my gradle
sourceSets {
    main {
        java.srcDirs = ['src/main/java']
    }
}

I see that gradle builds it because if the java file contains an error, it crashes
However trying to use that java class in JS did not work, i tried:
declare namespace org {
    namespace nativescript {
        namespace demo {
            class TestClass {
                hello(): string;
                static helloStatic(): string;
            }
        }
    }
}
if (app.android) {
    console.log('testing inner java files');
    console.log(org.nativescript.demo.TestClass.helloStatic());
    console.log(new org.nativescript.demo.TestClass().hello());
}

But JS complains that TestClass does not exist.

Hi @farfromrefug, the problem is that your hello and helloStatic methods are not public, by default methods in Java are private, so you need to add public and it should work.

@vtrifonov thank you! indeed now it works! Will test this within a module but should be good too. This  should be documented.
Thanks!

@vtrifonov thank you! indeed now it works! Will test this within a module but should be good too. This  should be documented.
Thanks!


Provide option to set modified icon

Currently the modified icon is hardcoded in tabline.lua on line 291.
The icon is not part of a standard font (I am guessing it is some NerdFonts symbol), so it does not display correctly with an unpatched font.

This should be resolved now.


Allow enumerating Project.Targets.Keys

The lack of this API causes CPS to fail when it calls project.Targets.Keys in some DDRIT scenarios here:  https://mseng.visualstudio.com/VSIDEProj/_git/VSIDEProj.CPS?path=%2Fsrc%2FMicrosoft.VisualStudio.ProjectSystem.Implementation%2FBuild%2FOutputGroupProviderBase.cs&line=75&lineStyle=plain&lineEnd=76&lineStartColumn=1&lineEndColumn=1
Removing this functionality was observable from outside MSBuild because
fields backed by a RetrievableEntryHashSet are exposed in the public
API. Specifically, 6881b07 caused a regression in TypeScript by throwing
when CPS tried to enumerate target names using Project.Targets.Keys.

I think I like this approach (implement the dropped .Keys without bringing back ReadOnlyConvertingCollection better. Anyone disagree?

@dotnet-bot test Windows_NT Build for CoreCLR please


[main]feat: add .snyk file

feat: add .snyk file
This file is used for excluding vendor folders from snyk code scans
Release note:
NONE

No need to run CI, because there is no functionality change.
/override ci/prow/e2e-functests


[formrecognizer] add missing simple tests

for samples added after we made change to start testing the sample files
Live tests pass: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=451275&view=results

/azp run python - formrecognizer - tests

/check-enforcer override


[Bug] plaintext codeblock backtick missing

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected packages and versions
latest version
Link to runnable example
No response
Steps to reproduce
Can reproduce in the playground: https://milkdown.dev/playground
[1] Begin with the following raw text:
```
```
[2] Start typing in the middle of the backtick
[3] Note that the backticks disappear
Expected behavior
The backticks should still be there as one types.
Actual behavior
The backticks are missing.
Runtime
Chrome
OS
macOS
Build and bundle tools
No response

Actually this is a false alarm since codeblocks can be made by indentation (https://www.markdownguide.org/basic-syntax/#code-blocks)


Polyfill enter vr bug

There is an issue using polyfill the way we use the vrdisplaypresentchange event. We assume the event to by of type VRDisplayEvent, but in polyfill its not, and the display property is not set.
https://github.com/googlevr/webvr-ui/blob/master/src/webvr-manager.js#L210
@borismus Is this something that could be added to polyfill, or should we make a workaround for polyfill?

Oh looking closer, display is supplied, but under detail: https://github.com/googlevr/webvr-polyfill/blob/93253c78bcffc22ae263cb9aba361c3c172d16e6/src/base.js#L335  Is this a old spec syntax?

Yes, IIRC the only field a custom event can provide is 'detail'... So we
can't change the polyfill in this regard. I guess this should be more
clearly documented.
On Thu, Jan 26, 2017 at 6:40 AM Jonas Jongejan<EMAIL_ADDRESS>wrote:

Oh looking closer, display is supplied, but under detail:
https://github.com/googlevr/webvr-polyfill/blob/93253c78bcffc22ae263cb9aba361c3c172d16e6/src/base.js#L335
Is this a old spec syntax?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlevr/webvr-ui/issues/1#issuecomment-275403876,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ8gK53Fc0v8CXYcNbNp_vcE0YWWOEBks5rWLBqgaJpZM4Lus2m
.

Aha, okay thanks. Then we'll make this workaround.

Thanks! Let me know when I can update from NPM.
On Thu, Jan 26, 2017 at 10:27 AM Jonas Jongejan<EMAIL_ADDRESS>wrote:

Aha, okay thanks. Then we'll make this workaround.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlevr/webvr-ui/issues/1#issuecomment-275469574,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ8gE8UOHOULzYpeM1z2Ko-ZhHfYsyrks5rWOWagaJpZM4Lus2m
.

Updated, and released on npm


Does it support using in React to parse encrypted Excel (office, wps)

@alone88888 Yes, you can try.

Is there an example to show how to use this in a browser environment

@alone88888是的，你可以尝试。

有没有一个例子来展示如何在浏览器环境中使用它

import exceljs from "@zurmokeeper/exceljs";

const uploadFilesChange = (file) => {
  const fileReader = new FileReader();

  fileReader.readAsArrayBuffer(file.file);

  fileReader.onload = (event) => {

    const { result } = event.target;
    const workbook = new exceljs.Workbook();

    workbook.xlsx.load(result, {
          password: "123",
     }).then(res => {

    })
  }
}

It can parse unencrypted files. When the password parameter is carried, the code will report an error（Uncaught (in promise) Error: The input must be a buffer）.

@zurmokeeper Is there an example to show how to use this in a browser environment

@alone88888 Can you provide me with your excel file so I can see what the problem is first?

@alone88888 What is your version @zurmokeeper/exceljs?

"@zurmokeeper/exceljs": "^4.4.3",

@zurmokeeper
"@zurmokeeper/exceljs": "^4.4.3"

@alone88888 What is the version of the dependency library officecrypto-tool? Go to node_modules and look at it specifically.

@zurmokeeper
"officecrypto-tool": "0.0.3"

@alone88888
const data = new Uint8Array(e.target.result);   // Try adding this line to your code
<!DOCTYPE html>
<html>
<head>
  <title>读取Excel文件示例</title>
  <script src="exceljs.min.js"></script>
</head>
<body>
  <input type="file" id="excelFileInput">
  <button onclick="handleFile()">读取Excel文件</button>

  <script>
    function handleFile() {
      const fileInput = document.getElementById('excelFileInput');
      const file = fileInput.files[0];

      if (file) {
        const reader = new FileReader();

        reader.onload = function(e) {
          const data = new Uint8Array(e.target.result);   // Try adding this line to your code
          const workbook = new ExcelJS.Workbook();

          workbook.xlsx.load(data, {password: '123'}).then(function() {
            // 在这里处理Excel文件的内容
            const worksheet = workbook.getWorksheet(1);
            worksheet.eachRow(function(row, rowNumber) {
              console.log(`第${rowNumber}行数据: ${row.values}`);
            });
          });
        };

        reader.readAsArrayBuffer(file);
      }
    }
  </script>
</body>
</html>

@zurmokeeper  thank you very very very much，it works

@zurmokeeper  I'm interested, but I have no experience submitting PRs, and I feel like I'm a rookie compared to you, and you can do better than me.


KAFKA-14821 Implement the listOffsets API with AdminApiDriver

Aside from consistency with the implementation of newer APIs this also provides more resilient retries (e.g. in case of TimeoutExceptions and other RetriableExceptions in general) and simplifies the code to some extent.
One notable detail is the addition of unsupported version handling in the failure callback in the AdminApiDriver and as a new callback in the AdminApiHandler and the AdminApiLookupStrategy. The relevant callback is invoked in case an UnsupportedVersionException is caught by the driver and ~returns a flag indicating whether the exception could be handled (like the similar callback in Call)~ it is used to process and fail just the keys for which the exception can't be handled.
Testing
The change has been tested manually and with a new KafkaAdminClientTest, as well as with added unit tests for the new handler and the changed driver behavior. The retry aspect also unexpectedly stalled a Connect AdminClientUnitTestEnv test due to retrying after a timeout where the test expected an immediate failure.
Also a JMH benchmark for the deleted equivalent of the fulfillment stage of the API has been deleted and not replaced as I wasn't too sure whether I can recreate it after the AdminApiDriver refactoring.
Notable changes after review comments

Made the unsupported version handling callback return a mapping of keys to be failed instead of a simple boolean flag;
Added an unsupported version handling callback for the lookup strategy as well;
Limited lookup retries to happen only after specifically NotLeaderOrFollowerException or LeaderNotAvailableException instead of after any InvalidMetadataException;

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

After a rerun, the CI this time failed on CoordinatorTest.testTaskRequestWithOldStartMsGetsUpdated and MirrorConnectorsWithCustomForwardingAdminIntegrationTest.testCreatePartitionsUseProvidedForwardingAdmin.
Like with the previous failing test, I didn't see the listOffsets(Map<TopicPartition, OffsetSpec>, ListOffsetsOptions) API being triggered when debugging either of these, but unlike the previous failing test I was unable to locally reproduce a failure over 500+ runs for the former and over 50+ runs for the latter (which is much slower, hence only 50 runs):
> Configure project :
Starting build with version 3.5.0-SNAPSHOT (commit id 3f77d983) using Gradle 8.0.2, Java 14 and Scala 2.13.10
Build properties: maxParallelForks=16, maxScalacThreads=8, maxTestRetries=0

> Task :trogdor:test

Gradle Test Run :trogdor:test > Gradle Test Executor 527 > CoordinatorTest > testTaskRequestWithOldStartMsGetsUpdated() PASSED

BUILD SUCCESSFUL in 4s
19 actionable tasks: 1 executed, 18 up-to-date
Completed run: 505

...

> Configure project :
Starting build with version 3.5.0-SNAPSHOT (commit id 3f77d983) using Gradle 8.0.2, Java 14 and Scala 2.13.10
Build properties: maxParallelForks=16, maxScalacThreads=8, maxTestRetries=0

> Task :connect:mirror:test

Gradle Test Run :connect:mirror:test > Gradle Test Executor 533 > MirrorConnectorsWithCustomForwardingAdminIntegrationTest > testCreatePartitionsUseProvidedForwardingAdmin() PASSED

BUILD SUCCESSFUL in 1m 3s
80 actionable tasks: 1 executed, 79 up-to-date
Completed run: 53

please also change the description to include the modifications you made

@dengziming I have updated the PR description a bit and added a "review modifications" section. I assume I shouldn't squash the commits now to preserve the commit history in the PR.
Let me know if there's anything else I need to do regarding that.
I'm now looking at the latest PR CI results.
There seems to be a significant number of test failures, a big increase over the results from previous commits. I suspect they are related to the recent mailing list thread about test failures but am still verifying that.

Notable changes better than old logic:

Retry lookup stage on receiving NOT_LEADER_OR_FOLLOWER and LEADER_NOT_AVAILABLE, whereas in the past we failed the partition directly without retry.
Removing class field supportsMaxTimestamp and calculating it on the fly to avoid the mutable state, this won't change any behavior of  the client.
Retry fulfillment stage on RetriableException, whereas in the past we just retry fulfillment stage on InvalidMetadataException, this means we will retry on TimeoutException and other RetriableException.


remove path dependency from assertions

Fixes Android couldn't use this version due to issue with not finding java.nio.file.Path

On Android you can use the https://github.com/joel-costigliola/assertj-core/blob/master/src/main/java/org/assertj/core/api/Java6Assertions.java entry point, which has no Path dependency.


Error after inserting: RangeError: Maximum call stack size exceeded

I'm getting this error
Exception in delivering result of invoking '/cfs.images.filerecord/insert': RangeError: Maximum call stack size exceeded
    at looksLikeArray (http://localhost:3000/packages/underscore.js?0a80a8623e1b40b5df5a05582f288ddd586eaa18:144:33)
    at Function._.each._.forEach (http://localhost:3000/packages/underscore.js?0a80a8623e1b40b5df5a05582f288ddd586eaa18:157:16)
    at Object.EJSON.clone (http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:424:5)
    at http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:425:22
    at Function._.each._.forEach (http://localhost:3000/packages/underscore.js?0a80a8623e1b40b5df5a05582f288ddd586eaa18:164:22)
    at Object.EJSON.clone (http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:424:5)
    at http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:425:22
    at Function._.each._.forEach (http://localhost:3000/packages/underscore.js?0a80a8623e1b40b5df5a05582f288ddd586eaa18:164:22)
    at Object.EJSON.clone (http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:424:5)
    at http://localhost:3000/packages/ejson.js?8659e45802513ffad53befd6dbee7e6e80df9da4:425:22 debug.js:41
HTTPUploadTransferQueue RELEASED

I'm trying to integrate collection-fs into my package but I've been hitting roadblocks. The first was CollectionFS wouldn't allow me to use it in a package if it was also being used in the project itself.
So I tried to create a helper to register it with the project:
SirAkshat.registerImageUploader(function(file, metadata, callback) {
	var fsFile = new FS.File(file),
		self = this;

	fsFile.metadata = metadata;

	console.log(fsFile);

    Images.insert(fsFile, function (err, id) {
        console.log(id);
        console.log(Images.findOne({_id: id}));
        console.log("===");
        //callback(['/cfs/files/images/', id, '/image.jpg'].join(''));
    });
});

But with the above I get the Error described above.
I'm not too sure what is causing it, what can I do to make it work.
Additional info might be useful: the Images.insert 'callback' never fires.

I had same problem and installing cfs:ejson-file solved it.

Had the same problem, installed cfs:ejson-file , no more problem 🍾


AND does not work

Description du problème
According to FHIR documentation a conjunction of values on a certain field can be expressed as /=&=. But Annuaire Santé seems to translates it into an OR.
Fichier•s concerné•s
I noticed the issue on PractitionerRole::name - I did not check other fields.
Sample:
GET https://gateway.api.esante.gouv.fr/fhir/PractitionerRole?name%3Aexact=Amélie&name%3Aexact=AGIER&_include=PractitionerRole%3Apractitioner
This requests tries to implement a fuzzy search on the whole name by requesting the PractitionerRoles with matching names containing the exact words "Amélie" and "Agier".
The results are:

Amélie GIANNOTTA-MAGNIER
Amélie ALBRECHT
Amélie BASEYA
Amélie BASEYA
Amélie POURCHET
Amélie DEMONT
...

Placing the query as an explicit OR gives the very same results:
GET https://gateway.api.esante.gouv.fr/fhir/PractitionerRole?name%3Aexact=Amélie%2CAGIER&_include=PractitionerRole%3Apractitioner
Solution proposée
Please fix.

Thank you for reporting this.
I've confirmed this is a bug in our API and have created a task in the project task tracker for fixing it.
In the meantime, you may be able to work around this bug with a search that includes  a family name and a given name, for example :
GET [base]//fhir/v1/PractitionerRole?name:exact=Am%C3%A9lie&family:exact=AGIE


Intercept routes with query parameters

So I have this in my route:
model() {
  return this.store.query('foo', {
    bar: 'baz'
  });
}

I need mirage to intercept the API request only if there's a bar query param that equals baz, and passthrough all the other cases. I expect something like the following in the mirage config:
this.get('https://api.example.com/foo?bar=baz', (db, request) => {
  debugger;
});
this.passthrough('https://api.example.com/foo');

Unfortunately it doesn't work that way.

Hmm, that doesn't seem to fit the way pretender works under the hood. Might be worth opening an issue there to see what they think of changing things? I would imagine they might not want to do so though as it could be a breaking change ...
I'll close this issue for now, feel free to comment if you think this still needs addressing ...


Application fails to publish on 3.0 sdk docker image with "rzc discover exited with code -2147450730"

Default application from a template fails to publish on latest 3.0 sdk container with an error:
C:\Users\ContainerAdministrator\.nuget\packages\microsoft.aspnetcore.razor.design\3.0.0-alpha1-10062\build\netstandard2.0\Microsoft.AspNetCore.Razor.Design.CodeGeneration.targets(69,5): error : rzc discover exited with code -2147450730. [c:\test\test.csproj]
microsoft.aspnetcore.razor.design package for 3.0.0 indicates 2.x net core tool however 3.0 sdk docker image doesn't have 2.x installed
rzs.runtimeconfig.json:
{
  "runtimeOptions": {
    "tfm": "netcoreapp2.0",
    "framework": {
      "name": "Microsoft.NETCore.App",
      "version": "2.0.9"
    }
  }
}

Steps to reproduce:

docker run --rm -ti microsoft/dotnet-nightly:3.0-sdk-nanoserver-sac2016
next command execute inside a container
md test && cd test
dotnet new mvc
dotnet publish --source "https://dotnet.myget.org/F/aspnetcore-dev/api/v3/index.json;https://api.nuget.org/v3/index.json" (use nightly sources for asp.net core 3.0)

Expected behavior
Publish should succeed
Diag logs:
1.log
Docker Image info:
microsoft/dotnet-nightly:3.0-sdk-nanoserver-sac2016, sha256:0465ba2b0f11f490a834906da13bea895b26abda231f95fe90407785b32b4a55
dotnet --info:
.NET Core SDK (reflecting any global.json):
Version:   3.0.100-alpha1-009642
Commit:    8cd38e2a49
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.14393
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.0.100-alpha1-009642\
Host (useful for support):
Version: 3.0.0-preview1-27004-04
Commit:  35a0b3f880
.NET Core SDKs installed:
3.0.100-alpha1-009642 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 3.0.0-alpha1-10062 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 3.0.0-alpha1-10062 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.DesktopUI.App 3.0.0-alpha-27006-1 [C:\Program Files\dotnet\shared\Microsoft.DesktopUI.App]
Microsoft.NETCore.App 3.0.0-preview1-27004-04 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

@rynowak / @pranavkm / @NTaylorMullen - any idea on this?
BTW -2147450730 = 0x80008096 in case that helps.

from what I see here https://dotnet.myget.org/feed/aspnetcore-dev/package/nuget/Microsoft.AspNetCore.Razor.Design the latest version 3.0.0-alpha1-10655 has still a reference "tfm": "netcoreapp2.0" just like in the example I pasted in the description above.
So the answer is no, it has not been updated

Has rzc been updated to target netcoreapp3.0

Not as yet.


Save a ParMesh as a serial mesh

The method PrintAsSerial added  here allows us to save a ParMesh as a serial mesh that unlike ParMesh::PrintAsOne does not have duplication of vertices and nodes at processor boundaries. This functionality is needed for a project where we will optimize a mesh on N ranks and then use it in LiDO on M ranks.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#3035
@kmittal2
@v-dobrev
@atallah727 + @bslazarov
5/25/22
⌛due 6/8/22
⌛due 6/15/22

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] Update CHANGELOG:

[ ] Is this a new feature users need to be aware of? New or updated example or miniapp?
[ ] Does it make sense to create a new section in the CHANGELOG to group with other related features?

[ ] Update INSTALL:

[ ] Had a new optional library been added? If so, what range of versions of this library are required? (Make sure the external library is compatible with our BSD license, e.g. it is not licensed under GPL!)
[ ] Have the version ranges for any required or optional libraries changed?
[ ] Does make or cmake have a new target?
[ ] Did the requirements or the installation process change? (rare)

[ ] Update continuous integration server configurations if necessary (e.g. with new version requirements for each of MFEM's dependencies)

[ ] .github
[ ] .appveyor.yml

[ ] Update .gitignore:

[ ] Check if make distclean; git status shows any files that were generated from the source by the project (not an IDE) but we don't want to track in the repository.
[ ] Add new patterns (just for the new files above) and re-run the above test.

[ ] New examples:

[ ] All sample runs at the top of the example source file work.
[ ] Update examples/makefile:

[ ] Add the example code to the appropriate SEQ_EXAMPLES and PAR_EXAMPLES variables.
[ ] Add any files generated by it to the clean target.
[ ] Add the example binary and any files generated by it to the top-level .gitignore file.

[ ] Update examples/CMakeLists.txt:

[ ] Add the example code to the ALL_EXE_SRCS variable.
[ ] Make sure THIS_TEST_OPTIONS is set correctly for the new example.

[ ] List the new example in doc/CodeDocumentation.dox.
[ ] If new examples directory (e.g.examples/pumi), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add example-specific documentation, see e.g. the src/examples.md.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] In examples.md, list the example under the appropriate categories, add new categories if necessary.
[ ] Add a short description of the example in the "Extensive Examples" section of features.md.

[ ] New miniapps:

[ ] All sample runs at the top of the miniapp source file work.
[ ] Update top-level makefile and makefile in corresponding miniapp directory.
[ ] Add the miniapp binary and any files generated by it to the top-level .gitignore file.
[ ] Update CMake build system:

[ ] Update the CMakeLists.txt file in the miniapps directory, if the new miniapp is in a new directory.
[ ] Add/update the CMakeLists.txt file in the new miniapp directory.
[ ] Consider adding a new test for the new miniapp.

[ ] List the new miniapp in doc/CodeDocumentation.dox
[ ] If new miniapps directory (e.g.miniapps/nurbs), add it to MINIAPP_SUBDIRS in the makefile.
[ ] If new miniapps directory (e.g.miniapps/nurbs), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add miniapp-specific documentation, see e.g. the src/meshing.md and src/electromagnetics.md files.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] The miniapps go at the end of the page, and are usually listed only under a specific "Application (PDE)" category.
[ ] Add a short description of the miniapp in the "Extensive Examples" section of features.md.

[ ] New capability:

[ ] All new public, protected, and private classes, methods, data members, and functions have full Doxygen-style documentation in source comments. Documentation should include descriptions of member data, function arguments and return values, template parameters, and prerequisites for calling new functions.
[ ] Pointer arguments and return values must specify whether ownership is being transferred or lent with the call.
[ ] Any new functions should include descriptions of their intended use e.g. for internal use only, user-facing, etc., along with references to example code whenever possible/appropriate.
[ ] Consider adding new sample runs in existing examples to highlight the new capability.
[ ] Consider saving cool simulation pictures with the new capability in the Confluence gallery (LLNL only) or submitting them, via pull request, to the gallery section of the mfem/web repo.
[ ] If this is a major new feature, consider mentioning it in the short summary inside README (rare).
[ ] List major new classes in doc/CodeDocumentation.dox (rare).

[ ] Update this checklist, if the new pull request affects it.
[ ] Run make unittest to make sure all unit tests pass.
[ ] Run the tests in tests/scripts.
[ ] (LLNL only) After merging:

[ ] Update internal tests to include the new features.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

I think this addition will be more useful and general if it's like void ParMesh.SaveAsSerial(Mesh* mesh).

@kmittal2, @atallah727, I made some changes and added some tweaks in my last commit (f2f489a3) -- take a look at the commit message and the changes and let me know if you have any comments.
@kmittal2, if you have some tests you can run to make sure I did not break your code -- it'll be great if you can run them.

Merged in next for testing...

@atallah727, I merged the PR in next for testing, however, if you have any feedback on the latest changes, we can still make changes before we merge the PR in master -- just let me know.


add another solution

Here is error message for user code.
It seems like there are strict rules for javascript code, but I think you can easily fix it by following error messages above!


Config file for application settings

Create a configuration file so system administrators can configure the application for their environments

The quick-and-dirty way to get this done is to have a config.php-style file

Done in b6921957cbcfaa7be6201055d52a64c671ecba4c


[SPARK-10210] [STREAMING] Filter out non-existent blocks before creating BlockRDD

When write ahead log is not enabled, a recovered streaming driver still tries to run jobs using pre-failure block ids, and fails as the block do not exists in-memory any more (and cannot be recovered as receiver WAL is not enabled).
This occurs because the driver-side WAL of ReceivedBlockTracker is recovers that past block information, and ReceiveInputDStream creates BlockRDDs even if those blocks do not exist.
The solution in this PR is to filter out block ids that do not exist before creating the BlockRDD. In addition, it adds unit tests to verify other logic in ReceiverInputDStream.

Merged build triggered.

Merged build started.

Test build #41495 has started for   PR 8405 at commit 89aaf9b.

Test build #41495 has finished for   PR 8405 at commit 89aaf9b.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41495/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #41497 has started for   PR 8405 at commit 9a81b93.

Test build #41497 has finished for   PR 8405 at commit 9a81b93.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41497/
Test PASSed.

LGTM except one minor comment

Merged build triggered.

Merged build started.

Test build #41507 has started for   PR 8405 at commit 2b6f76e.

Test build #41507 has finished for   PR 8405 at commit 2b6f76e.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41507/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

LGTM

Test build #41521 has started for   PR 8405 at commit 7590937.

Test build #41521 has finished for   PR 8405 at commit 7590937.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41521/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #41523 has started for   PR 8405 at commit c62e37d.

Test build #41523 has finished for   PR 8405 at commit c62e37d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41523/
Test PASSed.

Merged build finished. Test PASSed.

Thanks @zsxwing for reviewing. Merging this to master and 1.5


Stats GUI is blocking Train Menu Control Hints

Description

Is it possible to change this?
Reproduction

Factorio 2.0
Select Train Menu for planning/sheduling.
 My bad.
I fixed it.

This is actually a bug :)


Allow for pulling kube-system images from somewhere other than gcr.io

For scenarios where egress is limited - or when bandwidth is restricted, it'd be pertinent to support pulling images for the kube-system namespaces from somewhere other than gcr.io. I imagine this would work as follows:
Configuration option on kubernetes-master charm
On kubernetes-master there should be a configuration option: addon-repository.
options:
  addon-repository:
    description: The docker registry for which to pull kubernetes addons (kube-dns, dashboard, heapster, etc)
    type: string
    default: gcr.io/google_containers

During AddonTactic, the template would be re-written to include that as a variable and the charm would parse/update the template prior to creating the pod. Here's an example of that:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: kubernetes-dashboard
  namespace: kube-system
  ...
spec:
  selector:
    matchLabels:
      k8s-app: kubernetes-dashboard
  template:
    ...
    spec:
      containers:
      - name: kubernetes-dashboard
        image: {{ addon_repository }}/kubernetes-dashboard-{{ arch }}:v1.5.1
    ...

In doing so, the default value will be in line with upstream. However, the operator could provide alternative urls for fetching containers such as:

marcoceppi: pull images from marcoceppi user on docker hub
<IP_ADDRESS>/marcoceppi: pull images from marcoceppi user on private, on prem, registry
gcr.io/marcoceppi: pull images from marcoceppi user on public, alternative, registry

The charm should do a header lookup at each address to make sure the image exists and raise a blocked message when an error is occurred (but before deployment of the pods). If the configuration value is empty or NULL, raise blocked message.

Another note to this request is that a possibility of using auth credentials should be supported.
https://docs.docker.com/engine/reference/commandline/login/#privileged-user-requirement


[REQ] docs: add search

What kind of request is this?
Improvement of existing experience
What is your request or suggestion?
add algolia docsearch to docs website
Are you willing to submit PRs to contribute to this feature request?

[ ] Yes, I am willing to implement it.

This was completed in #389


Expose notify_update to apt::conf

Created ticket: https://tickets.puppetlabs.com/browse/MODULES-2269
This pull request simply exposes notify_update parameter of underlying apt::setting defined type to the apt::conf defined type as it isn't always appropriate to trigger an update for a specific configuration file.
I don't believe apt::pin shouldn't trigger an update either (set notify_update to false) - however willing to rework this if people disagree and simply expose notify_update as for apt::conf.
Happy to discuss further.
Thanks

Pins indeed don't need to trigger an apt-get update, they take effect at the next apt call that tries to determine which version of a package should be installed.

Hi @daenney, is there anything else I need to do to get this merged?
Thanks!

Oh, yes, sorry. It's currently lacking tests for the behaviour you've changed so though the original examples still work we're not validating the new possible uses.

Hi @daenney, I've added documentation and rspec test for apt::conf, I hope its sufficient - this is my first attempt at rspec-puppet tests.
With respect to the apt::pin change I'm not sure if any form of test is worthwhile here as before it unnecessarily triggered an apt-get update, now it will not. However I have documented this behaviour. Please let me know if there is anything else necessary.
Thanks

Hi @daenney is there anything else necessary to get this merged?
Thanks

Yes, I would really appreciate a squash here. It's a single feature and as such I'd like to merge it as a single commit into history.

Hi @daenney squashed as requested, rebased to current master as well.

@mhaskel Good to go?

@bdellegrazie thanks for the contribution!


[FR] Diff View to open & focus new tab if current tab pinned

When I open "Diff View" via command palette or shortcut, the behavior is a bit inconsistent.
On a regular tab, the diff view opens in the current tab.
If the tab is pinned, then the diff view opens in the first non-pinned tab it can find (and doesn't focus the view).
It would be nice if when the current tab is pinned, the "Diff View" opened in a brand new tab and changed focus to it.

I'm using the default Obsidian behavior with reusing an existing tab, when the current tab is pinned. So I won't change that for now. But I fixed the focus when using the command instead of the source control view.

Cool, thanks.
I will point out that the default Obsidian behavior is not good either (they do make a lot of UX mistakes). I will file a FR with core Obsidian for that.

You can use ctrl click in the source control view. At the moment, I don't plan to add an extra command for this.


Hokify Job Bot

Create an Hokify Job Bot:

[x] Fetch jobdata
[x] Create and publish job needs
[x] React to user requests
[x] Get location data
[x] Set fetch/update intervall
[ ] Filter out contact info
[ ] Parse job type field

requires #2520
because the structure of the need creation has been changed, to reduce the pain of merging/rebasing the required PR should be merged before hand

moving from in-progress to ready since assignment was empty

So far the issue seemed to be solved and the changes and adjustments of the job bot could be an own issue for future work.


Can it be modified to use decorators?

Can it be modified to use decorators,  like tsoa or nestjs style?
thanks!

Yes, it is possible! However, I prefer it this way, so I have no plan of implementing decorators. But if you want to create your own version, feel free to fork it.


Pass an empty array to entry point

Fixes https://github.com/aspnet/AspNetCore/issues/17419

@aspnet/build do you know what the package error is about:
.dotnet/sdk/3.1.100-preview1-014400/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(198,5): error NU5104: (NETCORE_ENGINEERING_TELEMETRY=Build) A stable release of a package should not have a prerelease dependency. Either modify the version spec of dependency "Microsoft.Extensions.DependencyInjection.Abstractions [3.1.0-rtm.19572.8, )" or update the version field in the nuspec.


[BUG] Double quoted string literals in SQLite called from Python are being deprecated

Description
Double quoted string literals in SQLite called from Python are being deprecated.
See https://sqlite.org/quirks.html#dblquote

Steps to reproduce
python -m manage version
5.1.1
python -m manage migrate
...
File "/usr/home/geoff/Projects/django/.venv/lib/python3.11/site-packages/django/db/backends/sqlite3/base.py", line 352, in execute
return super().execute(query)
^^^^^^^^^^^^^^^^^^^^^^
django.db.utils.OperationalError: no such column: "cms_page" - should this be a string literal in single-quotes?
Debug print(query) yields the SQL statement:
SQLite version 3.46.0 2024-05-23 13:25:27
sqlite> SELECT sql FROM sqlite_master WHERE type='table' and name="cms_page";
Parse error: no such column: "cms_page" - should this be a string literal in single-quotes?

Expected behaviour
sqlite> SELECT sql FROM sqlite_master WHERE type='table' and name='cms_page';
CREATE TABLE "cms_page" ("id" integer NOT NULL PRIMARY KEY AUTOINCREMENT, "created_by" varchar(70) NOT NULL, "changed_by" varchar(70) NOT NULL, "creation_date" datetime NOT NULL, "changed_date" datetime NOT NULL, "publication_date" datetime NULL, "publication_end_date" datetime NULL, "in_navigation" bool NOT NULL, "soft_root" bool NOT NULL, "reverse_id" varchar(40) NULL, "navigation_extenders" varchar(80) NULL, "template" varchar(100) NOT NULL, "login_required" bool NOT NULL, "limit_visibility_in_menu" smallint NULL, "is_home" bool NOT NULL, "application_urls" varchar(200) NULL, "application_namespace" varchar(200) NULL, "publisher_is_draft" bool NOT NULL, "languages" varchar(255) NULL, "revision_id" integer unsigned NOT NULL CHECK ("revision_id" >= 0), "xframe_options" integer NOT NULL, "site_id" integer NOT NULL REFERENCES "django_site" ("id") DEFERRABLE INITIALLY DEFERRED, "depth" integer unsigned NOT NULL CHECK ("depth" >= 0), "numchild" integer unsigned NOT NULL CHECK ("numchild" >= 0), "path" varchar(255) NOT NULL UNIQUE, "parent_id" integer NULL REFERENCES "cms_page" ("id") DEFERRABLE INITIALLY DEFERRED, "publisher_public_id" integer NULL UNIQUE REFERENCES "cms_page" ("id") DEFERRABLE INITIALLY DEFERRED)

Discussion
'table' and 'cms_page' are string literals in this context but could be names which can be double quoted in other contexts e.g., SELECT * from "cms_page";

Hm. I cannot reproduce this error message. It appears that you got this message from django CMS' initial migration, which does not use any raw sql but only Django migration commands. The underlying SQL is generated by Django.
I've tested this with Django 5.1.1 and is just worked fine. Any idea how to reproduce this?

I'll switch this to Django/Django as that is where I think the problem lies.


Access Denied issue

Hi Randy,
Thanks for the medium post, It is so useful. I started working on the serverless CI/CD, but stuck in running cim stack-up. I started getting the following error.
cim stack-up
{ AccessDeniedException: The ciphertext refers to a customer master key that does not exist, does not exist in this region, or you are not allowed to access.
at Request.extractError (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/protocol/json.js:48:27)
at Request.callListeners (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/sequential_executor.js:105:20)
at Request.emit (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/sequential_executor.js:77:10)
at Request.emit (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/request.js:683:14)
at Request.transition (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/request.js:22:10)
at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/state_machine.js:14:12)
at /usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/state_machine.js:26:10
at Request. (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/request.js:38:9)
at Request. (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/request.js:685:12)
at Request.callListeners (/usr/local/lib/node_modules/cim/node_modules/aws-sdk/lib/sequential_executor.js:115:18)
message: 'The ciphertext refers to a customer master key that does not exist, does not exist in this region, or you are not allowed to access.',
code: 'AccessDeniedException',
time: 2018-05-24T11:07:32.807Z,
requestId: 'a80f1197-5f42-11e8-a069-114a65b0fdc8',
statusCode: 400,
retryable: false,
retryDelay: 65.09868186058327 }
Any idea, why this issue is popping up
Thanks
Lakshman

It sounds like something is wrong with the KMS key.  Did you create your own KMS key and encrypt your github token?  https://github.com/thestackshack/serverless-stack-cicd#encrypt-secrets


[main] Upgrade to latest dependencies

Update with knative/serving and knative/eventing 1.6.0 release -evankanderson
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 2d2e049...0eb566a:
  > 0eb566a upgrade to latest dependencies (# 6448)

/retest


Node 8 compatibility

lock ember-lodash 4.17.5 to remove strong ember-cli-babel 6.1.0 dependency. This makes mirage node 8 compatible

I've bumped these yarn.lock as this PR does in https://github.com/samselikoff/ember-cli-mirage/pull/1191. However I did not add Node 8 to the testing matrix.

Only test in oldest supported Node
https://github.com/samselikoff/ember-cli-mirage/pull/1198

This PR can be closed.


Adding dependency libraries to zip package

This PR request primarily focuses on changes to adding ONNXRUNTIME libraries to the zip packages. Depends on the Target EP, it could also contain cuda_provider library or DML libraries.
Changes to cmake/package.cmake:

cmake/package.cmake: Modified the set_target_properties to provide more specific destination paths for the library, runtime, and public header. Also, updated the CPACK_PACKAGE_FILE_NAME to be more specific based on the system and processor type. This helps in creating more specific and identifiable package names.

Changes to libonnxruntime.so handling: To reduce linux package size.
The major changes include replacing copy commands with symbolic links for libonnxruntime.so and modifying the set_target_properties in cmake/package.cmake to provide more specific destination paths and package file names based on the system and processor type.

.github/workflows/linux-cpu-x64-build.yml: Replaced the copy command with a symbolic link for libonnxruntime.so. This creates a symbolic link to the original file instead of creating a copy, saving disk space and ensuring any updates to the original file are reflected in the linked file.
.github/workflows/linux-gpu-x64-build.yml: Similar to the above change, replaced the copy command with a symbolic link for libonnxruntime.so.
.pipelines/stages/jobs/steps/utils/download-ort.yml: Again, replaced the copy command with a symbolic link for libonnxruntime.so.

What dependency libraries are you adding? Would you mind explaining that?


Error about FEMBase

when I use
test JuliaFEM
there is an error , and I am looking forward for your help. thanks a lot!!!! :
The Julia Lang version is v 1.1.1
the error code is below:
┌ Warning: Package FEMBase does not have Test in its dependencies:
│ - If you have FEMBase checked out for development and have
│   added Test as a dependency but haven't updated your primary
│   environment's manifest file, try Pkg.resolve().
│ - Otherwise you may need to report an issue with FEMBase
└ Loading Test into FEMBase from project dependency, future warnings for FEMBase are suppressed.
WARNING: using FEMBasis.interpolate in module FEMBase conflicts with an existing identifier.
WARNING: could not import Base.start into JuliaFEM
WARNING: could not import Base.next into JuliaFEM
WARNING: could not import Base.done into JuliaFEM
WARNING: could not import Base.endof into JuliaFEM
ERROR: LoadError: LoadError: TypeError: in AbstractElement, in M, expected M<:FEMBase.AbstractFieldSet, got Type{FEMBasis.Seg2}
Stacktrace:
[1] top-level scope at none:0
[2] include at .\boot.jl:326 [inlined]
[3] include_relative(::Module, ::String) at .\loading.jl:1038
[4] include at .\sysimg.jl:29 [inlined]
[5] include(::String) at C:\Users\Administrator.julia\packages\FEMBeam\oEQQ6\src\FEMBeam.jl:17
[6] top-level scope at none:0
[7] include at .\boot.jl:326 [inlined]
[8] include_relative(::Module, ::String) at .\loading.jl:1038
[9] include(::Module, ::String) at .\sysimg.jl:29
[10] top-level scope at none:2
[11] eval at .\boot.jl:328 [inlined]
[12] eval(::Expr) at .\client.jl:404
[13] top-level scope at .\none:3
in expression starting at C:\Users\Administrator.julia\packages\FEMBeam\oEQQ6\src\beam3d.jl:15
in expression starting at C:\Users\Administrator.julia\packages\FEMBeam\oEQQ6\src\FEMBeam.jl:27
ERROR: LoadError: Failed to precompile FEMBeam [962f5c4a-ca9e-56d3-a659-14f0d9baaf78] to C:\Users\Administrator.julia\compiled\v1.1\FEMBeam\uHLOY.ji.
Stacktrace:
[1] error(::String) at .\error.jl:33
[2] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197
[3] _require(::Base.PkgId) at .\loading.jl:960
[4] require(::Base.PkgId) at .\loading.jl:858
[5] require(::Module, ::Symbol) at .\loading.jl:853
[6] include at .\boot.jl:326 [inlined]
[7] include_relative(::Module, ::String) at .\loading.jl:1038
[8] include(::Module, ::String) at .\sysimg.jl:29
[9] top-level scope at none:2
[10] eval at .\boot.jl:328 [inlined]
[11] eval(::Expr) at .\client.jl:404
[12] top-level scope at .\none:3
in expression starting at C:\Users\Administrator.julia\packages\JuliaFEM\DbhTT\src\JuliaFEM.jl:138
ERROR: LoadError: Failed to precompile JuliaFEM [f80590ac-b429-510a-8a99-e7c46989f22d] to C:\Users\Administrator.julia\compiled\v1.1\JuliaFEM\JKouA.ji.
Stacktrace:
[1] error(::String) at .\error.jl:33
[2] compilecache(::Base.PkgId, ::String) at .\loading.jl:1197
[3] _require(::Base.PkgId) at .\loading.jl:960
[4] require(::Base.PkgId) at .\loading.jl:858
[5] require(::Module, ::Symbol) at .\loading.jl:853
[6] include at .\boot.jl:326 [inlined]
[7] include_relative(::Module, ::String) at .\loading.jl:1038
[8] include(::Module, ::String) at .\sysimg.jl:29
[9] include(::String) at .\client.jl:403
[10] top-level scope at none:0
in expression starting at C:\Users\Administrator.julia\packages\JuliaFEM\DbhTT\test\runtests.jl:4
ERROR: Package JuliaFEM errored during testing

What FEMBase.jl version are you using? In FEMBase v0.3.1, by looking (https://github.com/JuliaFEM/FEMBase.jl/blob/v0.3.1/Project.toml) file, there should be Test dependency defined.
name = "FEMBase"
uuid = "fbcbbc08-f1bf-5204-9233-b69f5d396135"
authors = ["Jukka Aho<EMAIL_ADDRESS>version = "0.3.1"

[deps]
FEMBasis = "353fb843-c566-51e6-ba49-78b3e3d5ebb5"
FEMQuad = "be8e8821-3f6f-54c2-987c-d2773c3a52cb"
LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"
SparseArrays = "2f01184e-e22b-5df5-ae63-d93ebab69eaf"
Statistics = "10745b16-79ce-11e8-11f9-7d13ad32a3b2"
TimerOutputs = "a759f4b9-e2f1-59dc-863e-4aeb61b1ea8f"

[compat]
julia = "1"
FEMBasis = "0.2"
FEMQuad = "0.3"
TimerOutputs = "0.5"

[extras]
LinearAlgebra = "37e2e46d-f89d-539d-b4ee-838fcccc9c8e"
SparseArrays = "2f01184e-e22b-5df5-ae63-d93ebab69eaf"
Statistics = "10745b16-79ce-11e8-11f9-7d13ad32a3b2"
Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

[targets]
test = ["Test"]


[wip] Scroll the individual conflict marker list.

Use a scrollbar when you have more conflicted files than can be comfortably shown in the bottom panel.
Fixes #138.

This is a little more reasonable:


Interactive fiction

You might consider adding inform 7 and inkl to the list.

hey @Jerimee feel free to open a Pull Request with the links. I would highly appreciate it!

any updates @Jerimee ?

Soul is willing, flesh is weak.
here is link for ink: http://www.inklestudios.com/ink/
and here is home page for inform: http://inform7.com/
sorry for failing to open pull request...

No problem i will add it then. Thanks for the suggestions!


SAK-49599 Portal: Fix lost focus on eid and pw fields

The added if clause clause will prevent loosing the focus from the login fields to the focus on the portal wrapper.
Removing the "focus password field" code will prevent the username field from becoming focus while typing the password. This piece of code was already inactive due to the code focusing on the portal wrapper.
https://sakaiproject.atlassian.net/browse/SAK-49599

Thank YOU @kunaljaykam  :)


MobileElement cant resolve on java-client 8.3

The main issue is that java-client 8.3 does not have MobileElement. this is a conflict.
this is my dependencies:

this is my drivers:

appium_flutterfinder_java doest not have MobileElement.
this is the erros in the SDK:

appium : 2.0.0-beta.52
Flutter: 3.3.9
what should i do?

FYI: https://github.com/appium/java-client/blob/master/docs/v7-to-v8-migration-guide.md#mobileelement

Hey @amiravisar89, please update your Maven latest Java client updated.

@KazuCocoa please close this issue.


Use ConsumerArgs where possible instead of passing 4 parameters

It looks like we're passing 4 parameters around in multiple places, but we could simply use ConsumerArgs.

Use ConsumerArgs instead of passing 4 parameters
Move default method parameters to ConsumerArgs.default

This is a non-breaking change as I've kept the existing methods, marked them as deprecated, and they call the new methods I've defined.

Have checked this with the MiMa tool and it failed, so will fix those errors or mark them as unimportant

@mergifyio rebase


Hashing functions (ref #105)

A simple Hash class to make one-way password hash and to check that a string matches a hash (#105)
The class can be augmented with more functionalities if needed.
It uses standard php functions password_hash and password_verify
PS: @RogerCreasy you might want to add to your composer.json and in your README that PHP 7.1 is required in order to install/run this framework.

Thanks for your note regarding PHP 7.1. I'll take care of it.


New Sensor HmIP-STHO-A

https://github.com/home-assistant/home-assistant/blob/d8a7c547dfeae3dcccd01285bedcedbb668b2ffc/homeassistant/components/homematic/__init__.py#L85
This line needs to be removed in order to get the temperature from HmIP-STHO-A which was added here https://github.com/danielperna84/pyhomematic/blob/devel/pyhomematic/devicetypes/sensors.py#L590
Please also add  'ACTUAL_TEMPERATURE': '°C', in this section https://github.com/home-assistant/home-assistant/blob/8792fd22b91731141185bacdbc03a6e7a9856592/homeassistant/components/sensor/homematic.py#L35
and  'ACTUAL_TEMPERATURE': 'mdi:thermometer', in this section
https://github.com/home-assistant/home-assistant/blob/8792fd22b91731141185bacdbc03a6e7a9856592/homeassistant/components/sensor/homematic.py#L56

Closing this issue because it will be supported with the new pyhomematic version, and required changes are made with another PR.


Allow multiple MIE deployments per region

Allow multiple MIE deployments per region. Currently stack deployments will fail if MIE has already been deployed in the region.

duplicate of #61


Demo still uses Recurly.js v3

In the README.md file, the source code provided for the "Minimal billing information" example shows how to use Recurly.js v4, but the Demo is still using the older Recurly.js v3.

This demo is now totally offline


UnhandledPromiseRejectionWarning: Error:

I'm getting this error when trying to configure my Hoobs. Anyone having similar issues or know how to fix it?
UnhandledPromiseRejectionWarning: Error: write EPROTO<PHONE_NUMBER>:error:1425F102:SSL routines:ssl_choose_client_version:unsupported protocol:../deps/openssl/openssl/ssl/statem/statem_lib.c:1942:
at WriteWrap.onWriteComplete [as oncomplete] (internal/stream_base_commons.js:92:16)

(node:13634) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:13634) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

same issue


Added readRateLimit and writeRateLimit as config params to CliConfigs

Adding support for writeRateLimit and readRateLimit properties for ndbench CLI.

Thanks for the contribution. As far as I can tell, these configuration values will not be applied to the actual test because they aren't passed to the driver in NdbenchCli.
If we add new configuration parameters, we should make sure the driver respects them; also we should add documentation for the new configuration parameters. I'm happy to make suggestions on how to do that.


Changing entry status removes link

I'm not sure if this has been asked yet, or if perhaps I'm missing something.
I've noticed that if an entry status is changed to inactive, this will remove the entry from any associated LinkIt fields.
If I later on change that entry back to active, to restore the link, I have to return to any associated LinkIt fields, and relink to that entry.
It would be ideal if status changes did not remove the link association.

This issue has been resolved in the new version, still fixing the last few issues before getting a release out.

Thanks. Looking forward to the new release.


Can't remove/add events and clock scrambles for Weekly Competition 18 were not generated

scrambling service crashes on competition edit removing events

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

resolved


Fix issue #1064: Resource leak when register_signal=False

Memory leak fix for issue #1064
Fix the raven.contrib.flask.Sentry client so that the after_request handler is called to release all resources in all cases.

i wonder if the reason we did this was blinker becomes optional if you do register_signal=False?
either way lgtm


search enhancements: more clever commands

[x] "clear document" = delete all children in body except tool pallete
[x] "translate to ${lang}" = run google translate on the page #162
[ ] "accessibility semantics" = remove all styles
[ ] "$0" = currently selected element #65 & #106
[x] "do a barrel roll" = easter egg throwback to chrome search trick
[ ] "upside down" = flip body on it's head, so web designers can do what print designers do to evaluate harmony
[x] "wireframe mode" = remove all color
[x] "trashy" or "screen reader check" "simulate screenreader" = which apply Trashy CSS to the page. user the freebie code from the bookmarklet as a starting point, or just use it straight up
[x] "outline" = for debugging, put outlines on all elements, just like https://github.com/mrmrs/pesticide
[ ] "a11y focus"= focus spoofing and focus state verification #96
[x] "construct" = https://t7.github.io/construct.css
[ ] "Pally" reports https://github.com/pa11y/pa11y-webservice

my "debug layout" suggestion seems pretty much like the outline done by pesticide.
I don't know if i'd do both.

👍🏻
On Wed, Nov 28, 2018 at 19:27 Adam Argyle<EMAIL_ADDRESS>wrote:

want me (or you can) to close #233
https://github.com/GoogleChromeLabs/ProjectVisBug/issues/233 since it's
a dupe?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChromeLabs/ProjectVisBug/issues/98#issuecomment-442533194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFl9RHssOW_lYR2oKodb0CYFPEjwskqyks5uzseUgaJpZM4WxGU7
.

--
Ido Schachter


Encourage/enable edits to OSM

This is a meta issue split out from conversation in #177.
Need: OSM data is imperfect. The more users know about it and know how to edit it, and the more they actually edit it to improve it, the better ATIP data, and all evidence derived from OSM data, will be.
There are many options from simple to complex including:

[ ] General message along the lines of "ATIP depends on good OSM data. Contribute at osm.org" could be in about page, would be great for it to be read by many.
[ ] Links to OSM ways, e.g. as a tooltip when hovering over layers, as per https://github.com/acteng/atip/issues/177#issuecomment-1554334683
[ ] OSM editing mode in ATIP as per https://github.com/acteng/atip/issues/177#issuecomment-1554334683 also

Tagging as v3 issue for now, with reference to the first two options. The third, if we decide to do it, is probably a bigger issue out of scope for v2. I think we could usefully mention OSM data with a link to https://osm.org in v2 though, maybe in the about page.

Ah no, closing as inactionable. We link to OSM from the About page in the sketch tool and on all relevant layers in the browse tool. There are much larger open questions about how to collect feedback from users about things they see that look wrong and validate that and funnel it back into OSM.

Aha. Just tested for one of the OSM layers in York. Great that it jumps to the relevant OSM way on that layer, it's a bit sudden and does not give warning. UX experience that I guess would be less surprising would be to click on an entity, see some info, including maybe 'View in OSM' and possibly streetview, and then explicitly have one more click.

Overall: agree re wider questions that need some thought.


Partial Ownership NFT

What is the intended/suggested way to create partial ownership asset?

https://womplay.io/id/unclezawkyi1/nfts/edition/dinoparknfts/1


Connect to SQL Server via SQL Authentication

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): <IP_ADDRESS>6603
Visual Studio Version VS 2019 ~ 16.11.2

Describe the bug

On which step of the process did you run into an issue: Data/Add Data
Clear description of the problem: Connecting to SQL Server via SQL Authentication does not work. Testing the connection is successful...

...but when you click OK, there is a login failure error:

Thanks, @beccamc! 👍


Is there any way to run this on windows?

We're working on adding better support for running LangGraph Cloud apps on Windows/Linux in the coming weeks! I'll post an update in this repo & update the Readme once that's ready

@lanesky
Do you know why the LangSmith API key is needed if LangGraph is open source and can be run locally?
I'm working on a project where privacy is crucial, so we need to run everything locally, including the LLM (hopefully this is possible with open-canvas). Any insights would be greatly appreciated!

@Julsgaard I also trying to find a way to run this locally to save privacy.
If you find a way, please share.
Thanks!

@lanesky could you please help me to set open canvas locally

Look better Mistral AI for free

@inweb-pro It's an open source?