google-cloud-node idioms, for consideration

Originally brought up: https://github.com/GoogleCloudPlatform/google-cloud-node/issues/1801#issuecomment-261092666
google-cloud-node (GCN) has some conventions which ideally could be used here. It would be great to achieve a common UX that can be re-used by a developer when hopping between APIs.
Resource hierarchy
The GCN library layers its classes so that if you need a GCS "Object", for example, you go through 2 parent layers, the GCS class, and then a bucket:
var gcs = require('@google-cloud/storage')({authInfo});
var myBucket = gcs.bucket('my-bucket');
var myFile = myBucket.file('my-file');

myFile.getMetadata(function(err, metadata) {});

I can't speak for certain how this would look with the generated layer, but a simple interpretation would be something like:
var gcs = require('generated-storage-client')({authInfo});

gcs.getObject({
  bucketName: 'my-bucket',
  objectName: 'my-file'
}, function(err, resp) {});

The hierarchy used by GCN is nice because it lets you cache one resource that you intend to make multiple calls with; myFile.delete(), myFile.createReadStream(), etc.
Accessor methods
GCN distinguishes between two types of objects: a "Service" (Google Cloud Storage), and a "ServiceObject" (a Bucket). A consistent set of methods are exposed on a ServiceObject:

ServiceObject#create({ config options }, function(err, serviceObjectInstance, apiResponse) {})
ServiceObject#delete(function(err, apiResponse) {})
ServiceObject#exists(function(err, exists, apiResponse) {})
ServiceObject#get(function(err, serviceObjectInstance, apiResponse) {})
ServiceObject#getMetadata(function(err, metadata, apiResponse) {})
ServiceObject#setMetadata({ new metadata }, function(err, apiResponse) {})

*Methods that don't apply for a specific ServiceObject are removed, e.g. you can't delete a Compute Engine Region, but you can get its metadata.
Streaming methods / naming conventions
The upstream API has its own implementation of logical naming patterns, and in the generated layer, those probably shouldn't be tampered with. However, it might be appreciated by Node.js developers to recognize some names they know from other libraries, for example, createReadStream() and createWriteStream() where there are readable and writable streams.
In GCN's Bigtable API, we expose a createReadStream() to access the proto service's "ReadRows" method.
We've also seen naming conflicts between JavaScript/Node.js definitions and the language-agnostic API terminology. Having a small handwritten map of convenience/conventional names to upstream names might be a big help.
// cc: @bjwatson @jgeewax @callmehiphop @jmdobry

👍

In addition to everything @stephenplusplus mentioned, there are a few differences between the gapic layer and GCN.

LRO - AFAIK the gapic implementation requires the user to manually poll an endpoint to see if the Operation has completed. In GCN we expose an EventEmitter and perform the polling for the user. This allows them to simply tap into either complete/error events or a Promise resolution.
Promises - In GCN we allow the user to provide a Promise module (es6 compatible) if they were prefer to use something like Bluebird instead of native Promises. I do not believe this is configurable in gapic.
Retry Logic - It looks like gapic will only retry requests for codes matching either timeouts and unavailable (4, 14). GCN also retries requests for resource exhausted and internal error (8, 13) errors as well.
Emulator support - In GCN we accept a customEndpoint parameter for users who wish to use an emulator with the client (which gapic supports via servicePath & port). AFAIK emulators require you to send payloads via plain text which can be done via grpc.credentials.createInsecure(). In GCN if we see that the user is using a custom endpoint, we do this for them. In gapic I believe the user must also generate the creds in addition to specifying the custom endpoint information.

LRO - AFAIK the gapic implementation requires the user to manually poll an endpoint to see if the Operation has completed. In GCN we expose an EventEmitter and perform the polling for the user. This allows them to simply tap into either complete/error events or a Promise resolution.

I think this is being addressed in #64

@callmehiphop does this work for the Emulator support point: https://github.com/googleapis/gax-nodejs/issues/46?

Basically, we made the design decision that the gapic layers to be close to gRPC definitions (i.e. the same method names + protobuf messages directly). @omaray and @anthmgoogle for the confirmation. cc @bjwatson too.
Because of this, naming changes or restructuring the data structure might conflict with the design policy unfortunately -- and that would require lots of changes on our code generators and codegen configurations (for mapping).

Promises

You're right, it's not possible to use other implementations of promises. Filed #72.

Retry Logic

I don't remember why that's only them. @geigerj might remember why those error codes are chosen (I vaguely remember some documentations for which codes should be retryable, but I've lost it).

Emulator support

It is also discussed internally, allow customizing the emulator endpoints as some environment variable IIRC. I haven't heard any updates for that recently, but that will be addressed later.

@callmehiphop does this work for the Emulator support point: #46?

I think it could! Here's how we currently do emulator support
var bigtable = require('@google-cloud/bigtable')({
  customEndpoint: 'a/b/c:8080'
});

I believe the gapic equivalent would be something like
var grpc = require('grpc');

var bigtable = require('@google-cloud/bigtable')({
  servicePath: 'a/b/c',
  port: 8080,
  sslCreds: grpc.credentials.createInsecure()
});

(note: we should review the discussion again and create issues for individual improvements and fixes)

2.5 years in - I think most of the APIs we have exposed here in gax are pretty settled in sadly.  @stephenplusplus @callmehiphop - if we want to make any of this stuff actionable, I would really love it if y'all would create individually actionable bugs that can be picked at one at a time :)


(chore) Update docs to suggest install brew from --HEAD

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
The formulae has been deprecated from hombrew's side in https://github.com/Homebrew/homebrew-core/pull/155315 due to lack of new release and failing test.
Installing without --HEAD will get the very old version but with --HEAD can get all the latest improvements even though a release hasn't been rolled. Given the release blockers this seems like the best option for those looking to install fresh.

https://github.com/ytdl-org/youtube-dl/issues/31585
https://github.com/ytdl-org/youtube-dl/issues/31067

You could also use the POSIX-y one-file nightly build if your environment has a python executable (eg, symlink to $(which python3)).


'DEBUG is not defined' on Chromium and 'NS_ERROR_NOT_IMPLEMENTED' on FF 50.1.0

The last time everything worked was a few months back.  I think I was on skate 4.0.2 and skate-web-components 2.x.  Those versions stopped working for Firefox 50.1, where I'm getting NS_ERROR_NOT_IMPLEMENTED when the polyfill tries to call customElements.
I tried upgrading to skate.js 4.5.0 and skate-web-components 5.0.0.
I still get the same error on Firefox.  On chromium I get a whole bunch of errors like this:
Uncaught ReferenceError: DEBUG is not defined ...

Here's a branch with my examples and source: https://github.com/micahscopes/tangled-web-components/tree/buggy_skate_4_5_upgrade
Check examples/index.html, etc.  Building is done with npm run transpile.

For reference, we can view the problems on https://micahscopes.github.io/tangled-web-components/examples/.

As for the 'NS_ERROR_NOT_IMPLEMENTED' on FF 50.1.0... It is completely unrelated.  And I have a minimal working example, it's the counter example from the documentation, updated to run the latest versions of skatejs and skatejs-web-components https://jsfiddle.net/eukhtw7z/1/: https://jsfiddle.net/eukhtw7z/1/

Hmm, I loaded that up in Firefox and it is working.

Firefox 50.1.0?
On Jan 5, 2017 9:06 PM, "Trey Shugart"<EMAIL_ADDRESS>wrote:

Hmm, I loaded that up in Firefox and it is working.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/skatejs/skatejs/issues/998#issuecomment-270820642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXylqe77EfF9rd368PIdYdK2L4B3aHaks5rPa-sgaJpZM4LbR63
.

P.s. I'm using Arch Linux Firefox 50.1.0 from the arch repo. But even on
the development version I had trouble. I think this could be a bug in how
the web-components polyfill detects customElements support, cause the error
seems to have to do with the polyfill calling customElements.define()...
But Firefox doesn't do that!
On Jan 5, 2017 10:41 PM, "Micah Fitch"<EMAIL_ADDRESS>wrote:

Firefox 50.1.0?
On Jan 5, 2017 9:06 PM, "Trey Shugart"<EMAIL_ADDRESS>wrote:

Hmm, I loaded that up in Firefox and it is working.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/skatejs/skatejs/issues/998#issuecomment-270820642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXylqe77EfF9rd368PIdYdK2L4B3aHaks5rPa-sgaJpZM4LbR63
.

Interesting. I'm going to update the web components stuff soon (ish), but maybe it's worth trying the latest polyfills (from master) to see if that resolves it for you?

@treshugart I could try that.  Here's a related ticket I opened: https://github.com/WebReflection/document-register-element/issues/94

No, skatejs-web-components uses the canonical fill at webcomponents/custom-elements.

FWIW I couldn't reproduce @micahscopes problem using same configuration (Firefox on Arch Linux)
https://github.com/WebReflection/document-register-element/issues/94#issuecomment-271160528
All green for me (or at least my test page)

Actually, I've tried the skate test page and indeed it has errors (although still unrelated with my poly)

Ooh... a clue!  I had webcomponents and webcomponents.customElements both enabled in Firefox 50.1...  When I disabled them, things started working again!

Hey @micahscopes any progress on this? In a recent update, we've started using process.env.NODE_ENV instead of DEBUG, just FYI. This is internal, though, and isn't present in the dist builds.

I'm not experiencing either of these errors anymore, although now I'm having new problems!  I'll make new tickets for new problems :)


[cocoon] Use NNBD

This requires removing the mocks in the test code. Then we can follow up with removing the null checks throughout the codebase.

There are some details about using build_runner for codegen of mocks in https://github.com/dart-lang/mockito/blob/master/NULL_SAFETY_README.md cc @CaseyHillers

All dependencies now have null safety versions. Due to our use of auto-generated code, I need to regenerate these, which requires the codebase to be migrated to null safety.


Parallelize tokenization

Currently tokenization is not actually parallel, because all threads use a single tokenizer and the underlying tokenizer is thread-hostile i.e. can only be used in one thread a time - see #1421 and the upstream issue in tokenizers.
When setting --num-threads=N, we should actually spawn N fast tokenizers, one for each of N threads, so that we actually get N-fold parallelism.

Currently tokenizers are singletons, and it would be a fairly big architectural change to make them not singletons, so I'm closing this as won't fix.


APIC Upload with Paperclip

Like a previous issue, I am trying to read and extract a picture frame and prepare it to be uploaded using paperclip. I have read the documentation and even tried what was recommended in this issue, https://github.com/robinst/taglib-ruby/issues/29 , but I can't figure out how I should set it up for paperclip.

I'll need to see the code you're using to help. It would be best if you could provide a standalone script that shows the problem without paperclip.

In my tracks model I have
require 'taglib'

before_save :extract_coverart

def extract_coverart
	TagLib::MPEG::File.open(audio.queued_for_write[:original].path) do |file|
	   tag = file.id3v2_tag
	   covers = tag.frame_list('APIC')
          unless covers.empty?
	     cover = covers.first
           File.open(audio.queued_for_write[:original].path, "w+") do |f|
              self.coverart  = f.write(cover.picture)
	       end
          end
        end
	end

I continue receiving the "\xFF" from ASCII-8BIT to UTF-8 error.  I have tried other code where I do manage to save to a binary column and the file looks like \xFF\X.... but I'm not sure how I'm supposed to convert that into something paperclip can use.

As already mentioned, a standalone script and a example audio file would be useful.
Are you using Windows? Try changing "w+" to "wb+".


[v2.0034] adjust building CPS for sugar lumps

caveats:

can't guarantee this multiplier is applied in the right order with respect to other multipliers, but it looked like it from code inspection and it doesn't seem to be wildly wrong;
this only accounts for sugar lumps.  in particular, it appears that v2.0034 also introduces the concept of gods which provide CPS multipliers for buildings, which are not accounted for.

fixes #119.

Thanks for doing this!  It gives me something to check my logic against, lol.  We did similar things, so I don't think I'll merge this however.  Sorry =(.


Add tests for calculation of totalPrice

Built to solve "Total Price totals up all prices of all products properly" in #4
This PR extracts the logic of totalPrice to util.js allowing it to be tested separately from the rest of the hook logic.
Changes made:

refactored repeated setup for tests using beforeEach
moved totaling and conversion logic to util.js
handled when currency is undefined
add tests for calculateTotalPrice:
adds all prices together
handles cents
handles different currencies
handles missing currencies

Other concerns
I do, however, have some concerns around the calculation of totalPrice. As I was looking at the code, I saw that you're inferring the currency based on the items inside the cart. This is fine unless it's possible that the items in the cart have different currency properties. That would cause the item in the cart to be improperly converted (1 to 1 instead of say 1 to 1.25 for GBP to USD). I don't have much experience with Stripe so this may just not be possible and therefore not an issue. If it is we can open up a new issue for that.
Let me know if I need to make any changes 👍

@ChrisBrownie55 whoa! Thank you so much!
Honestly, the way things are added to the cart may need to be reflected on. The expectation is to never have different products with different currency, it's one of those things where I'm sure I'll never come across this issue and set it up this way rather than thinking of a way to avoid that possibliity altogether.
This makes me think that there should be an issue to talk about the some default values and ways to let devs pass configs to this hook.
Perhaps in the context provider we can pass a "currency" value to refer to that instead of referring to what actually is getting passed?

So instead of hardcoding the default value to 'USD' would the new version look akin to:
  <CartProvider
    billingAddressCollection={...}
    successUrl={...}
    cancelUrl={...}
    stripe={...}
    currency="GBP"
  >

And then inside the useStripeCart hook:
  const [cart, dispatch] = useContext(CartContext);

  const {
    /* other stuff */
    currency
  } = cart;

  const totalPrice = () => calculateTotalPrice(currency, cartItems);

And then for the logic of calculateTotalPrice should it be that only the passed in currency is used or should it still only use that as a fallback when cartItem.currency is undefined?

To be safe, I think that we should make sure that it's only the passed in currency. The developer needs to determine what it is from the start. The intention is that these items will be coming from Stripe's API, so the currency needs to be determined before hand anyway

So would it be safe to assume that in all calls to toCurrency, the value passed as the currency should be replaced with the new one that's being passed down through CartContext?

Yes, exactly. Thank you for the suggestion

Update
I have added in the changes talked about in the aforementioned comments. To summarize, these changes are:

Added currency prop to CartProvider component
Now using that currency with formatDetailedCart and calculateTotalPrice
calculateTotalPrice now uses only the provided currency prop from CartProvider
Tests have been updated:

One of the tests for detailed SKUs was failing because it was adding an item with the currency of GBP and one with the currency of USD when the provided currency through CartProvider was USD. Therefore, the result is that the GBP was automatically converted to USD and thus failed the test.
Tests for calculateTotalPrice now reflect the change in how currency is used.

Other changes
While I was modifying formatDetailedCart, I was looking at the reducer that was created and noticed a lot of redundancy and refactored it with the optional chaining and nullish coalescing operators.

@ChrisBrownie55 reviewing now 👁

This is great! Thank you!


Hook Model

Moved Hook Model
Moved hook model over to new repo with the addition of a new query addSetToHook which assigns a panel set to a hook. Models, queries, and endpoints are all implemented.
New Query: addSetToHook
Sets the next_panel_set_id of a hook to a new panel set id (will fill null value stored in next_panel_set_id)

Parameter
Type
Description

hook_id
number
ID of the hook that is being updated

panel_set_id
number
ID of the panel set that is being linked to the hook

Endpoints
createHook (POST)

Values
Type
Description

position
number[]
Position of the hook on a panel

current_panel_id
number
ID of panel the hook is on

next_panel_set_id
number
ID of the panel set the hook links to (can be null)

getHook (GET)

Values
Type
Description

id
number
ID of the hook to get

getPanelHooks (GET)

Values
Type
Description

id
number
ID of panel to get associated hooks from

addSetToHook (PATCH)

Values
Type
Description

hook_id
number
ID of the hook to update

panel_set_id
number
ID of panel set the hook links to

Would you mind merging main into your branch so you have the updated version of the user queries

Everything else looks good. Check out the reviews and I'll accept the PR


Open link at the top of the entry

With the old UI, the title of the entry was clickable and opened the link.
The title is no longer clickable and thus the only way to open the link is to scroll to the bottom of the entry and click on the "Open Link" button.
Reported here https://github.com/Athou/commafeed/discussions/1015#discussioncomment-4107894

Do you already have some conception on how to solve it?

What about a context menu when right clicking on the entry header (long press on mobile) ?
I feel like it would elegantly solve this issue as well as #1027, #1025 and #1019.

I think it could be a solution

So far I like it.
Could you also add opening in new background tab to the menu?

Sure, why not, although that's what middle-clicking does already.

Done :)


Traefik collects usage data without permission

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I have this in the global section of the static configuration:
[global]
    checkNewVersion = false
    sendAnonymousUsage = false

What did you see instead?
I see this in the logs:
time="2023-07-03T18:35:08Z" level=info msg="Stats collection is enabled."
time="2023-07-03T18:35:08Z" level=info msg="Many thanks for contributing to Traefik's improvement by allowing us to receive anonymous information from your configuration."

I have also seen #9598, and although it is similar, this is a different problem.
This is NOT a duplicate.
What version of Traefik are you using?

The TOML static confiuration (at /etc/traefik/traefik.toml) contains the following:
[global]
    checkNewVersion = false
    sendAnonymousUsage = false

[pilot]
    dashboard = false

[log]
    #level = "warn"
    level = "debug"

[api]
    insecure = false
    dashboard = true

# Note: [endpoints] intentionally removed from the snippet I posted here

[providers]
    [providers.docker]
        exposedByDefault = false
        network = "t2-proxy"

    [providers.file]
        directory = "/etc/traefik/dynamicconf/"
        watch = true

Platform is Linux on ARM, as can be seen above.
If applicable, please paste the log output in DEBUG level
2023-07-02T23:30:52.020891801+02:00 time="2023-07-02T21:30:52Z" level=info msg="Configuration loaded from file: /etc/traefik/traefik.toml"
2023-07-02T23:30:52.028534497+02:00 time="2023-07-02T21:30:52Z" level=warning msg="Traefik Pilot has been removed."
2023-07-03T20:35:08.799145662+02:00 time="2023-07-03T18:35:08Z" level=info msg="Configuration loaded from file: /etc/traefik/traefik.toml"
2023-07-03T20:35:08.800339402+02:00 time="2023-07-03T18:35:08Z" level=info msg="Traefik version 2.10.3 built on 2023-06-19T16:18:54Z"
2023-07-03T20:35:08.805571801+02:00 time="2023-07-03T18:35:08Z" level=debug msg="Static configuration loaded {\"global\":{\"sendAnonymousUsage\":true},\"serversTransport\":{\"maxIdleConnsPerHost\":200},\"entryPoints\":{...},\"providers\":{\"providersThrottleDuration\":\"2s\",\"docker\":{\"watch\":true,\"endpoint\":\"unix:///var/run/docker.sock\",\"defaultRule\":\"Host(`{{ normalize .Name }}`)\",\"network\":\"t2-proxy\",\"swarmModeRefreshSeconds\":\"15s\"},\"file\":{\"directory\":\"/etc/traefik/dynamicconf/\",\"watch\":true}},\"api\":{\"dashboard\":true},\"log\":{\"level\":\"debug\",\"format\":\"common\"},\"pilot\":{}}"
2023-07-03T20:35:08.805669596+02:00 time="2023-07-03T18:35:08Z" level=info msg="Stats collection is enabled."
2023-07-03T20:35:08.805692077+02:00 time="2023-07-03T18:35:08Z" level=info msg="Many thanks for contributing to Traefik's improvement by allowing us to receive anonymous information from your configuration."
2023-07-03T20:35:08.805712873+02:00 time="2023-07-03T18:35:08Z" level=info msg="Help us improve Traefik by leaving this feature on :)"
2023-07-03T20:35:08.805736021+02:00 time="2023-07-03T18:35:08Z" level=info msg="More details on: https://doc.traefik.io/traefik/contributing/data-collection/"
2023-07-03T20:35:08.809294964+02:00 time="2023-07-03T18:35:08Z" level=warning msg="Traefik Pilot has been removed."

Hello,
it's because you are using pilot.
You have to remove the pilot section.
duplicate of #9895, #9771, #9508

I am NOT using pilot.
Pilot has been removed, or isn't it? And even the configuration tells to not use it.
There is zero reason why disabling a nonexistent component should somehow force-enable your data mining.
Please reopen the issue, as it has not been solved.

Sorry but you are using pilot:
in your configuration:
[pilot]
    dashboard = false

in your logs:

2023-07-03T20:35:08.805571801+02:00 time="2023-07-03T18:35:08Z" level=debug msg="Static configuration loaded {"global":{"sendAnonymousUsage":true},"serversTransport":{"maxIdleConnsPerHost":200},"entryPoints":{...},"providers":{"providersThrottleDuration":"2s","docker":{"watch":true,"endpoint":"unix:///var/run/docker.sock","defaultRule":"Host({{ normalize .Name }})","network":"t2-proxy","swarmModeRefreshSeconds":"15s"},"file":{"directory":"/etc/traefik/dynamicconf/","watch":true}},"api":{"dashboard":true},"log":{"level":"debug","format":"common"},"pilot":{}}"
...
2023-07-03T20:35:08.809294964+02:00 time="2023-07-03T18:35:08Z" level=warning msg="Traefik Pilot has been removed."

You have to remove the pilot section of your configuration.


Wont Connect to LocalHost

Issue Type: Bug
Error message where my query won't connect to local host.
"An unexpected error occurred while connecting..."
Azure Data Studio version: azuredatastudio 1.35.0 (e9c8eb8564f537726b36b5f15c7f35e62f229284, 2022-02-23T17:36:02.883Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-10210U CPU @ 1.60GHz (8 x 2112)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.77GB (3.18GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

import
Mic
1.5.0

@LuisRaulRivera24 are there any error messages or callstacks in the "Help->Developer Tools" Console?  does the connection work when you add to Servers viewlet and only fail when running queries?  Do connections to other servers work fine?  Have you tried to connect to localhost with other tools, such as SSMS, successfully?  Thanks!

Closing per previous comment and inability to repro.  Please let us know if you are still impacted by this issue and can provided additional diagnostic info.  Thanks!


 Please review.

Comparison is base (91a956b) 81.13% compared to head (50907db) 81.08%.

Files
Patch %
Lines

...smi/src/engine/regmach/translator/instr_encoder.rs
61.76%
13 Missing :warning:

...wasmi/src/engine/regmach/executor/instrs/branch.rs
60.00%
8 Missing :warning:

.../wasmi/src/engine/regmach/translator/result_mut.rs
0.00%
4 Missing :warning:

...mi/src/engine/regmach/translator/visit_register.rs
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #797      +/-   ##
==========================================
- Coverage   81.13%   81.08%   -0.05%
==========================================
  Files         274      274
  Lines       25271    25323      +52
==========================================
+ Hits        20503    20533      +30
- Misses       4768     4790      +22

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


work with v2 of pouchdb

Is this the PouchDB server version or the PouchDB version?  I think the client-side version is up to 5.1.

this related to updating it to work with persistent indexes in pouchdb library 2.0, honestly don't know if it still works with 5.1 but I don't think any of the breaking changes apply.

The test suite shows all passing for both PouchDB 3.6.0 and 5.1.0.

that should be expected as that aspect of pouch hasn't changed and this
issue was only left open by accident sorry about that
On Tue, Dec 29, 2015 at 11:01 AM superlou<EMAIL_ADDRESS>wrote:

The test suite shows all passing for both PouchDB 3.6.0 and 5.1.0.
—
Reply to this email directly or view it on GitHub
https://github.com/pouchdb/geopouch/issues/4#issuecomment-167819971.

No problem, was just doing a double-check for sanity.  Thanks for the plugin!


When creating a temporary data source, the user does not know when the data source has been created.

Is your feature request related to a problem? Please describe.
When creating a temporary data source, the user does not know when the data source has been created.
So when the user pops up a chart preview, no data is displayed.
After some time, the user has to redraw the chart to see the data.

Describe the solution you'd like
must tell the user when the data source creation is complete.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@eltriny 아래 이슈도 함께 확인 해주시면 좋을것 같네요.


Zip instead of cloning and removed reference to vanished repository

The API Guru repository is very big. Using zip instead of cloning the repository makes it smaller to download and hopefully solves the CI issue in #15.
The Nexmo repository is removed and couldn't find any clone so I just removed the reference.

@jongotlin I can confirm this works on my machine 🎉.

@jongotlin I can confirm this works on my machine 🎉.

Thanks for confirming.


setup.py does not use build_scripts, therefore lacking shebang replacement

This issue was originally created at: 2013-02-06 03:55:07.
This issue was reported by: mgorny.
mgorny said at 2013-02-06 03:55:07

The build_scripts command of distutils replaces shebangs of installed scripts with the Python
version used to build the package. SCons does not use it but installs the scripts directly,
therefore having their shebangs left unfixed.
This results in a failure to 'plainly' run SCons on a system that has Python3 used by default:
$ scons
scons: *** SCons version 2.2.0 does not run under Python version 3.2.3.
Python 3.0 and later are not yet supported.
While proper shebang substitution would result in SCons being run with Python 2.7 (as the user
expects it to be).
I'd like to request either using build_scripts to fix shebangs unconditionally or providing an
option to do so.

alastairt said at 2013-04-29 09:22:21

Fix for scons.setup.py.
This bug occurs as self.scripts returns ['script/scons', 'script/sconsign',
'script/scons-time', 'script/scons.bat']
But this is not where build_scripts has put them.
If we look at
To mirror
/opt/lib/python2.7/distutils/command/build_scripts.py:copy_scripts
   for script in self.scripts:
       adjust = 0
       script = convert_path(script)
       outfile = os.path.join(self.build_dir, os.path.basename(script))

Thus the setup.py install_scripts should be using the paths as above.
N.B. self.scripts == self.distribution.scripts == self.get_inputs()
Patch to fix this issue.
+from distutils.util import convert_path
setup.py: line 299: install_scripts.run
-inputs =  self.get_inputs()
+inputs = [os.path.join(self.build_dir, os.path.basename(convert_path(script)))
for script in self.get_inputs()]

closing in favor of #3451


URL-ify strings in the views

Some of the data we're displaying has urls as strings. We need to be able to wrap any url string on the website with link tags, and maybe have a dialog window that warns the user they're leaving the site.

Autolinking conflicts with markdown links

@andymeneely What was the decision for implementation here? Were we going to manually parse the linking on the controllers after Markdown is processed?

use cramdown, then parse for autolink after that.

look into allowing markdown without html

closing this issue since we've decided this should be done in the loaders


[PulsarAdmin] Brokers to async

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

This PR is blocked by https://github.com/apache/pulsar/pull/6547


Error [ERR_MODULE_NOT_FOUND] on 1.15.x with Nextjs

Discussed in https://github.com/typedorm/typedorm/discussions/281

Originally posted by matus123 September  7, 2022
hi,
i am trying to use https://github.com/typedorm library (https://github.com/typedorm/typedorm) with nextjs... but there is problem when i use this library in nextjs apis... i am getting runtime error
error - Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/home/matus/repos/typedorm-nextjs-example/node_modules/@typedorm/core/esm/public-api' imported from<EMAIL_ADDRESS>    at new NodeError (node:internal/errors:371:5)
    at finalizeResolution (node:internal/modules/esm/resolve:416:11)
    at moduleResolve (node:internal/modules/esm/resolve:932:10)
    at defaultResolve (node:internal/modules/esm/resolve:1044:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:422:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:222:40)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

i tried tinkering with tsconfig or webpack configs, but without any success
when i tried to use next-transpile-modules it started to work but getting these kind of warnings
warn  -<EMAIL_ADDRESS>Critical dependency: the request of a dependency is an expression

<EMAIL_ADDRESS>Critical dependency: the request of a dependency is an expression

minimal repo: https://github.com/matus123/nextjs-typedorm-example
i opened discussion here aswell (https://github.com/vercel/next.js/discussions/40313)
i dont know if it is problem on nextjs or typedorm side
thanks

I'm hitting this with SvelteKit and Vite as well.  The issue seems to be that the build output for ESM modules isn't correct.  If you inspect the published packages you'll see the syntax in<EMAIL_ADDRESS>is:
export * from './public-api';
export * from './private-api';

Whereas it should be:
export * from './public-api.js';
export * from './private-api.js';

relative import paths need full extensions (e.g we have to write import "./foo.js" instead of import "./foo")

Source - ECMAScript Modules in Node.js
If I manually add the file extensions to the import/exports in my node_modules then it works.

PSA to Next.js users: It seems that you can circumvent this issue for now by including the installed @typedorm/* packages in the transpilePackages setting under next.config.js (Next.js 13), or by using the transpile-packages plugin if using a previous version of Next.

I've encountered this issue in Nuxt.js also, and can't get it to work despite downgrading to 1.14.x, or trying to import and re-export in a .cts file. Does anyone have a simple repo demonstrating it working? Thanks!
 ERROR  [worker reload] [worker init] Cannot find module 'C:\projects\personal\test-app\node_modules\@typedorm\document-client\esm\public-api' imported from<EMAIL_ADDRESS>
  at new NodeError (node:internal/errors:387:5)
  at finalizeResolution (node:internal/modules/esm/resolve:330:11)
  at moduleResolve (node:internal/modules/esm/resolve:907:10)                                                         21:09:11
  at defaultResolve (node:internal/modules/esm/resolve:1115:11)
  at nextResolve (node:internal/modules/esm/loader:163:28)
  at ESMLoader.resolve (node:internal/modules/esm/loader:841:30)
  at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
  at link (node:internal/modules/esm/module_job:75:36)

Relevant for people using Vite: https://github.com/vitest-dev/vitest/issues/2313

I'm running into the same issue when testing with vitest. My project is on TypeScript + ESM.


Default Output Colorizer

The Output Colorizer extension adds syntax highlighting to the output panel.  However, other extensions (e.g. Code runner) can cause conflicts, removing the syntax highlighting.
Apparently this is a limitation with VSCode itself. I'm not exactly sure what needs to be done here to get extensions to play nicely. I suppose we would need to be able to select a default colorizer?

Yeah we currently do have support for users to choose a colorizer.
Thus leaving this open as a feature reuqest.
fyi @jrieken and @sandy081


Vulnerability package dependency "defaults-deep"

Version
v2.4.3
Reproduction link
https://www.npmjs.com/advisories/778
Steps to reproduce
Please fix this vulnerability in your dependency tree:
nuxt > @nuxt/core > @nuxt/server > serve-placeholder >  defaults-deep
What is expected ?
0 vulnerabilities found when audit packages.
As no patch is currently available for this vulnerability it is our recommendation to select another module that can provide this functionality.
What is actually happening?
All versions of defaults-deep are vulnerable to prototype pollution. Provided certain input defaults-deep can add or modify properties of the Object prototype. These properties will be present on all objects.
Additional comments?
https://hackerone.com/reports/380878

This bug report is available on Nuxt community (#c8618)

Hi. Thanks for the report. This is being addressed fast.

jonschlinkert/defaults-deep#5
https://github.com/jsless/defu

@pi0 Can you please explain shortly how you fixed that? (I followed the 2 links but ...)

For clarification as also described in CVE-2018-16486 the impact is possibly more depending on the application. and no Nuxt.js users are affected because options to the middleware are not from user input but only from nuxt.config.
For general:
I submitted the fix to prevent accepting contructor.prototype key on defaults-deep but as it is unlikely to fast merge and publish, I created an alternative package that only does what we want.


[SPARK-22103][FOLLOWUP] Rename addExtraCode to addInnerClass

What changes were proposed in this pull request?
Address PR comments that appeared post-merge, to rename addExtraCode to addInnerClass,
and not count the size of the inner class to the size of the outer class.
How was this patch tested?
YOLO.

@cloud-fan @viirya I addressed the post-merge comments you had in https://github.com/apache/spark/pull/19324

Test build #82197 has started for PR 19353 at commit 55104df.

LGTM

LGTM

Test build #82199 has started for PR 19353 at commit c0488fb.

Test build #82197 has finished for PR 19353 at commit 55104df.

This patch fails Spark unit 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/82197/
Test FAILed.

jenkins retest this please

Test build #82204 has started for PR 19353 at commit c0488fb.

Test build #82199 has finished for PR 19353 at commit c0488fb.

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/82199/
Test PASSed.

Thanks! Merged to master.

Test build #82204 has finished for PR 19353 at commit c0488fb.

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/82204/
Test PASSed.

Merged build finished. Test PASSed.


org.springframework.cloud.consul.discovery.ConsulCatalogWatch makes Consul high CPU

Based on the branch 2.0.x, I found that ConsulCatalogWatch would make Consul high CPU, it calls "/v1/catalog/services" frequently, nearly caused an accident. When reading codes, I dont't know its usage

Hello,
I'm having the same issue. I have tried to disable the check with spring.cloud.consul.discovery.catalog-services-watch.enabled=false but it didn't work.


NBcategorical.category_ratio - marks in red celles that have less than 90% of the data

Changed the function to return all in categories in black except the ones that have more than 90% of the same category.
This makes more sense because we want the imbalanced features to 'pop'
The change:

The result:


docs(pcf-exchange-kit):icon format changed to SVG

Description
In this pull request, I have changed the format of the icon's image in the adoption view, operation view and changelog of the PCF Exchange kit from PNG to SVG.

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[X] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[X] Copyright and license header are present on all affected files

LGTM, please update icon for specification page

okay, I am doing the changes.

LGTM, please update icon for specification page

@SujitMBRDI Changes have been added successfully.


All drones doing same thing when running multivehicles.py

I've followed the installations instructions for the multivehicles capability.  When I run multivehicles.py and MAVProxy, I see multiple drones.  However, on further inspection and experimentation, they are all following the same command.  For example, I cannot get them to go to different waypoints.  Is there something wrong with my pymavlink installation?

actually, I found the same problem. Do you have any progress? @AndreasMegatranus

@peterbarker


Net plugin optimizations

Change Description
Optimizations:

Minimize packed_transaction copies
Remove calls to get/find_plugin<chain_plugin> as lookup is surprisingly expensive
Provide more efficient sha256_less for id lookups

net_plugin:

Fixed issue during sync where head block num checked instead of lib
Remove large_msg_notify and associated code as not used

Consensus Changes
None
API Changes

No changes to net_plugin wire protocol
signal accepted_transaction
-- transaction_metadata packed_trx changed from packed_transaction to packed_transaction_ptr

Documentation Additions
None

The test failures appear to be real. Investigating.

Reverting "fix sync check for lib" fixed the smoke test. Created issue #6466 for investigation.


Align runtime requirements with EF packages

An issue we found using this package is that as soon as you upgrade to EF 7.0 packages, but still running on .Net 6, it fails since it's referencing the wrong EF assemblies.
Specifically, we're seeing issues like this:
System.MissingMethodException: Method not found: 'System.Collections.Generic.IList`1<Microsoft.EntityFrameworkCore.Metadata.Conventions.IModelFinalizingConvention> Microsoft.EntityFrameworkCore.Metadata.Conventions.ConventionSet.get_ModelFinalizingConventions()'.
   at XO.EntityFrameworkCore.NpgsqlJsonSerializerOptions.JsonSerializerOptionsConventionSetPlugin.ModifyConventions(ConventionSet conventionSet)

To align this package with how EF packages are published (all compatible with .Net 6, even 7.0 packages), this PR removes the runtime conditional switch for dependencies.
In this move, support for EF 6 is also dropped, but I'd argue that anyone needing EF 6 compatibility could use a earlier version of this package, just like how they consume EF itself.
Would love to hear your thoughts on this, because we've found this package quite helpful while we await EF 8.0!

Sounds reasonable to me. I wanted to make the library compatible with both versions 6 and 7 of EF Core, but I suppose publishing different major versions would have been a better way of doing that. Thanks for contributing!

Yeah, having matching versions, maintaining different packages for 6.0 and 7.0 is probably "the most correct" approach, but also takes effort.
Anyways, with the latest 2.0.4 version you published with these changes, our app is happy again, able to do nice ser/de!
Thanks for the quick response here!


2.0.0rc3

Pull Request Test Coverage Report for Build 644

19 of 19 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 637:
0.0%

Covered Lines:
1288

Relevant Lines:
1288

💛  - Coveralls


Error on Importing a list of xml-files

Good morning,
I have the task to import a long list of around 9000 xml files, all in the size range of 1 to 150MB.
I have encountered some errors while importing, and I can't find a solution for them.
The most common error is the following:
[10:13:07 ERROR] Aborting import due to errors.
[10:13:07 ERROR] Caused by: java.lang.NoClassDefFoundError: org/citydb/core/operation/common/cache/BranchCacheTable
[10:13:07 ERROR] Caused by: java.lang.ClassNotFoundException: org.citydb.core.operation.common.cache.BranchCacheTable
[10:13:07 WARN] Database import aborted.
Exception in thread "main" java.lang.NoClassDefFoundError: picocli/CommandLine$IExitCodeGenerator         at picocli.CommandLine$RunAll.recursivelyExtractExitCodeGenerators(CommandLine.java:2441)         at picocli.CommandLine$RunAll.extractExitCodeGenerators(CommandLine.java:2438)         at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2180)         at picocli.CommandLine$RunAll.execute(CommandLine.java:2391)         at org.citydb.cli.ImpExpCli.process(ImpExpCli.java:305)         at org.citydb.cli.ImpExpCli.start(ImpExpCli.java:157)         at org.citydb.cli.ImpExpCli.main(ImpExpCli.java:149) Caused by: java.lang.ClassNotFoundException: picocli.CommandLine$IExitCodeGenerator         at java.net.URLClassLoader.findClass(Unknown Source)         at java.lang.ClassLoader.loadClass(Unknown Source)         at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)         at java.lang.ClassLoader.loadClass(Unknown Source)         ... 7 more
The error occurs mostly on the bigger files of the list (110MB+), but if I import them as a single file, everything runs fine.
Apparently this error comes up when the Importer/Exporter is already running some time on other files, and then finally gets somehow overwhelmed from those bigger files.
It is hard to investigate in this, as the errors only occur when the Importer is running for some time and on a bigger list, but the BranchCacheTable Keyword, has always been in the description of the error.
The error also happend while using the GUI, where it looked like this:
[01:27:05 ERROR] Aborting import due to errors.
[01:27:05 ERROR] Caused by: java.lang.NoClassDefFoundError: org/citydb/core/operation/common/cache/BranchCacheTable
[01:27:05 ERROR] Caused by: java.lang.ClassNotFoundException: org.citydb.core.operation.common.cache.BranchCacheTable
[01:27:05 WARN] Database import aborted.
Exception in thread "AWT-EventQueue-0" java.lang.NoClassDefFoundError: com/formdev/flatlaf/MnemonicHandler$1 	at com.formdev.flatlaf.MnemonicHandler.showMnemonics(MnemonicHandler.java:205) 	at com.formdev.flatlaf.MnemonicHandler.processKeyEventOnWindows(MnemonicHandler.java:125) 	at com.formdev.flatlaf.MnemonicHandler.postProcessKeyEvent(MnemonicHandler.java:82) 	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(Unknown Source) 	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source) 	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source) 	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source) 	at java.awt.Component.dispatchEventImpl(Unknown Source) 	at java.awt.Container.dispatchEventImpl(Unknown Source) 	at java.awt.Window.dispatchEventImpl(Unknown Source) 	at java.awt.Component.dispatchEvent(Unknown Source) 	at java.awt.EventQueue.dispatchEventImpl(Unknown Source) 	at java.awt.EventQueue.access$500(Unknown Source) 	at java.awt.EventQueue$3.run(Unknown Source) 	at java.awt.EventQueue$3.run(Unknown Source) 	at java.security.AccessController.doPrivileged(Native Method) 	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source) 	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source) 	at java.awt.EventQueue$4.run(Unknown Source) 	at java.awt.EventQueue$4.run(Unknown Source) 	at java.security.AccessController.doPrivileged(Native Method) 	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source) 	at java.awt.EventQueue.dispatchEvent(Unknown Source) 	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source) 	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source) 	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source) 	at java.awt.EventDispatchThread.pumpEvents(Unknown Source) 	at java.awt.EventDispatchThread.pumpEvents(Unknown Source) 	at java.awt.EventDispatchThread.run(Unknown Source) Caused by: java.lang.ClassNotFoundException: com.formdev.flatlaf.MnemonicHandler$1 	at java.net.URLClassLoader.findClass(Unknown Source) 	at java.lang.ClassLoader.loadClass(Unknown Source) 	at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source) 	at java.lang.ClassLoader.loadClass(Unknown Source) 	... 29 more
Both error logs look somehow different on the paths, but happened on the same files and around the "expected" time (in the later part of a list, where bigger files were located.
Unfortunately I cannot provide the files, that are producing this error due to privacy reasons.
But as the error is not occurring when I import the files one by one (and also the validation function returns that the files are valid), I hope to find a solution to this without the files.
If you need further information, I'll respond as soon as possible.
Thanks! I'm curious about your answer!
Little remark:
It is somehow a little bit annoying, that the Import/Exporter does not follow the order of the files in the folder, that is selected. For me it seems like the order in which the Importer works through the list is somehow random, which makes it impossible to continue an import, that was interrupted by an error. If the Import had an option to import in an size-order or alphabetical-order, it would be easy to find the list of files, that are already imported or otherwise that are still remaining.
My setup:
Importer/Exporter 5.1.0 used in CLI mode
OS: Windows Server

Hi, which java version is running on the Windows Server?

Java Version 8.0.1620.12 (build 1.8.0_162-b12)

Could you install Java 11 and test the import again?

It will take some time to test, as the error occurs normally after some running time, but I'll check that out, and come back as soon as I have news!

Your issue looks strange because your JVM cannot find classes like BranchCacheTable which are contained in the JARs of the Importer/Exporter. These JARs are located in the lib folder inside the installation folder of the Importer/Exporter and the filenames start with impexp-*.jar.
Cache tables like BranchCacheTable store temporary information when importing/exporting datasets. The Importer/Exporter first holds this temporary information in main memory but writes it to the database (or a local cache) as soon as a predefined limit is reached to avoid out-of-memory situations. That's why your issue only occurs after some time because only then these limits are reached. And obviously the limits are not reached when you only import single files.
You can set these limits under "Preferences -> Import -> Resources". Check the "Cache settings" paragraph in this chapter of the online user manual. So, by reducing the numbers for the geometry and feature cache (e.g., to 20 or 50), you can force the Importer/Exporter to write the temporary information much earlier. And thus, your issue will occur much earlier. Just an idea to speed up your tests.
To resume an import operation, read this.

Thanks for your advice so far!
So it seems like the newer Java Version (11) changed a lot. I am not getting the BranchCacheTable Error anymore.
Thanks for the advice with the log file. This works as well, as I got an "Out of Shared Memory" Error. Is there also a specific property so get this under control?
One idea:
For me it seems like the "Skip"-Filter, where I put in the Log-File of the previous, aborted run does only filter for gml-ids. This is useful, when I want to continue to import a large file. For my scenario it would be more helpful if I could filter for file-names, as they are also printed in the logfile, but I cant find an option in the GUI to tell the Imp/Exp to skip entire files. At the moment I am importing a large zip-archive which contains a lot files. As the Imp/Exp under Java 11 is now working on the files in alphabetical order, I was able to just unzip it, and delete the files until the last entry in the logfile. But now on the unzipped files, the Imp/Exp is again working in a "random" order, so I won't be able to just delete the "already imported" files. As the most time consuming operation seems to be the reading of the xml-files, the gml-id filter doesnt help much to speed up the import, as all files have to be read again, and also the "Shared Memory" Error will come again..

I guess your Out of Shared Memory error is thrown by your PostgreSQL database? You might need to increase the max_locks_per_transaction paramemter in your PostgreSQL configuration (see, for example, here). Alternatively, you can reduce the number of parallel workers and connections that are used by the Importer/Exporter. See the online manual and a similar discussion here.

Thanks for the advice! I think the reduction of the parallel workers might be a useful way to go for my setup.
Thanks for your help on my problems, I think this issue can be closed with my little Feature-idea of the skip-file-filter.


Publish manifest file to App Service as well

How can I deploy the manifest file to the App Service so it's accessible by URL?
I want to use this URL for Centralized Deployment to my Office 365 organization so neither a File Share nor SharePoint are required.

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

ID: 9a6812c3-fbf6-1a89-6305-8da71f83b5a3
Version Independent ID: 614c2c5a-f3e7-fa28-6e4a-8ad68634c0d9
Content: Host an Office Add-in on Microsoft Azure - Office Add-ins
Content Source: docs/publish/host-an-office-add-in-on-microsoft-azure.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @aaronjones1 thanks for asking this question! I'm not sure if you are just looking for articles about hosting the add-in/manifest, but these two cover this information.
Host an Office Add-in on Microsoft Azure (although you don't have to use Azure)
Publish Office Add-ins using Centralized Deployment via the Office 365 admin center
Or are you looking for more specific documentation? If this is a how to question, you can also ask on Stack Overflow and tag it with the "office-js" tag (http://stackoverflow.com/questions/tagged/office-js).
Thanks!


Can't select organization

I've been unable to select the organization I am using. In fact, IntelliJ just stops working once I get to the organization screen. I can see the orgs, but cannot select any of them or close the window. When I click on the window, I get the classic Windows "ding" notification that indicates that I can't do something (I don't know how else to describe that noise).

Actually I think I found the solution. It may have been caused by the multiple instances of IntelliJ that opened once I started the app. I close all the instances except for one and it worked.


improve grammar

imo this whole section could be excluded from the admin training topic, since it does not make much sense

Looks like there is a Docker image for the admin topic. If you think it is not useful, you can remove the Docker image name in the metadata.yaml (docker_image: ) of the admin topic. Then the possible Docker image will not appear on the topic page :smile:

A Docker image still makes sense, but a public Galaxy server does not.

@bebatut but the whole section will still render no matter if the image is there I think: https://github.com/galaxyproject/training-material/blob/master/_layouts/topic.html#L155

@martenson True: I was checking something else and missed what you meant 😟

@nsoranzo even the docker training does not use this docker image, I do not think it is of much use here.

@nsoranzo that sounds awesome, but if there is no image yet, I would refrain from advertising it on the page (similar as linking to incomplete tutorials, and providing placeholder files in repo)

Right now we have a Docker image per topic, in fact the template uses topic.docker_image, so we would need to extend that to tutorial-specific images.


Analytics kinesis

Issue #, if available:
Add Anlytics Kinesis Plugin
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

👍

Codecov Report

Merging #965 into master will increase coverage by 0.03%.
The diff coverage is 88.7%.

@@            Coverage Diff             @@
##           master     #965      +/-   ##
==========================================
+ Coverage    87.3%   87.34%   +0.03%
==========================================
  Files          78       79       +1
  Lines        3718     3831     +113
  Branches      709      728      +19
==========================================
+ Hits         3246     3346     +100
- Misses        449      462      +13
  Partials       23       23

Impacted Files
Coverage Δ

...ify/src/Analytics/Providers/AWSPinpointProvider.ts
89% <100%> (ø)

packages/aws-amplify/src/Common/Facet.ts
100% <100%> (ø)
:arrow_up:

packages/aws-amplify/src/Analytics/Analytics.ts
87.95% <83.33%> (+0.14%)
:arrow_up:

...lify/src/Analytics/Providers/AWSKinesisProvider.ts
88.28% <88.28%> (ø)

Continue to review full report at Codecov.

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


Having two unnamed arguments and making the first optional

Forgive me if Issues are not meant for questions, neither the contributing guidelines nor the ReadMe say so, so please allow me to ask a question here. I'd like to make clap parse the following:

todo TEST "Test things" means CODE is TEST, TEXT is Test things.
todo "Test things" means CODE is not set, TEXT is Test things.
todo TEST "Test things" -m means CODE is TEST, TEXT is Test things, marked is true.

Issues are fine for questions :smile:
Currently you can't exactly do that, but depending on your actual use case there may be a way to make it work. The reason I say you can't exactly do that, is that unnamed arguments are purely index based, clap doesn't know which value is which, simply first come first served.
But in order to prevent an XY problem, could you tell me a little about the actual use case and what you're trying to model? There may be a better way, or something that clap does support in a slightly different manner.
Thanks for taking the time to do the detailed write up! :+1:

Oh I can definitely make this work simply by making TEXT the first unnamed argument and CODE the second, optional one. This is what the second code snippet does. I'm fine with that, too, it's not a big deal to have the arguments reverted, but the reason I wanted them to be the way I initially described is that it feels more ergonomic for my use case.
I'm new to Rust and I'm trying to create an app for managing todo files (I use them for projects I work alone at because I dislike having issues be out-of-the-band, unversioned in relation to the source code) and I'd like to be able to create a todo just by invoking todo "Have a nice day" - this would make a todo titled "Have a nice day". Since I also intend to offer capabilities for attaching files to todoes I figured it would be nice to be able to set explicit code for todos. If not provided, it would be either sequential number rising or first letters of the title words combined or something like that. Being able to do todo FixNav "Fix the second menu link style - icon and add menu links for ToS and privacy policy" feels nicer than todo"Fix the second menu link style - icon and add menu links for ToS and privacy policy" FixNav. There's not much more to it. :)
I'm closing the issue for now. I believe code-wise this could be changed when no multiple arguments are present. You could probably be able to detect the case I'm describing. But it's super niche, so it's not really necessary. Thank you!

I'd like this feature as well, and in my case I can't just reorder the arguments.  Could you please reopen this?
My use case: I'm adding subcommands to git-series similar to "cp" and "mv", which take "source" and "dest" positional arguments in that order; they need to come in that order to follow well-established convention (cp source dest).  I also need to make "source" optional, because it has a default (the current series).  So, I'd like to make source optional and dest mandatory.  In this case, if clap sees one positional argument, it should go in dest, but two positional arguments should go in source and dest.
Does that seem reasonable?

Reopening for @joshtriplett.

@joshtriplett with the new 2.17.0 allowing things like mv <files>... <target> does that help this issue, or is it something different?

@joshtriplett I could special case this if there are only two positional arguments, but my worry is what if there are 3, or 4? It's also hard to determine at that point if the user input the the first arg correctly, but accidently left off the second, but then clap interprets it differnetly.
$ prog current_series

Gets interpretted as src=current_series(default), dest=current_series
Which may not be a huge deal in your case, but imagine a hypothetical mv
$ mv src

Gets interpretted as src=(somedefault file/folder), dest=src which could be catostrophic.

@kbknapp The version from 2.17 almost works, but I don't want to allow multiple arguments (or display it as ...).  Could you also allow an optional second-to-last positional argument, which would have the same effect (0-or-1 and 0-or-more seem equivalent here)?

Since there's now a form of look-ahead for the <src>... <dest>  type positionals, I'd imagine it'd also work for [src] <dest>, but I'm unsure how to inform clap of such.
@joshtriplett and @TomasHubelbauer in both cases is the second positional always required? If that's the case I could implement this, because saying, "the final positional is required, but the one prior is not" is the information clap needs to special case. Otherwise I'd have to add something like an Arg::secondary_index which would be more difficult to explain/document without a decent round of bike-shedding.

@kbknapp Yes, the case I have always requires the second positional argument.

I have this implemented and it's working. I have one final issue to implement, then I'll put out 2.20 on crates.io


Introduce a friendly web interface

That's a little bit better!
Don't forget to update the front_view function before merging!

Closed in favor of #407


Overriding WordPress globals is prohibited.

phpcs: 3.3.1
wpcs: 3.3.1
php: 7.1.16
I'm creating a plugin but the same thing happens if I'm creating a theme.
Steps:
I'm creating a new page:
add_menu_page(
      esc_html__( 'Test', 'test' ),
      esc_html__( 'Test', 'test' ),
      'edit_posts'
      'test'
      array( $this, 'get_welcome_page' ),
      'dashicons-laptop'
);

providing template callback:
  public function get_welcome_page() {
    require_once plugin_dir_path( dirname( __FILE__ ) ) . 'admin/settings/welcome.php';
  }

in template I'm assing some value to $title, $id, $menu, $paged, $post_id variable:
$title = 'some value';

and i get linting error:
Overriding WordPress globals is prohibited. Found assignment to $title
    |       | (WordPress.WP.GlobalVariablesOverride.OverrideProhibited)

I think that this is a false positive because I'm not using any global variable inside this tempalte and I would like to use $title variable.

On its own, the $title in the view file appears to be in the global scope, which is why it is flagged. It's only at run-time do we see that the view file is required inside of a function.
PHPCS doesn't execute the PHP, so it can't detect this.
You've got a few main options:

Don't set the $title in the view file, but set it inside get_welcome_page(), and then just escape and echo it in the view file (keep the logic out of the view file).
Echo 'some value' directly where you need it and don't use the variable at all.
Use // phpcs:ignore WordPress.WP.GlobalVariablesOverride.OverrideProhibited in the line before the assignment to $title in the view file.
Choose a different but equally as useful variable name.

What @GaryJones said + it looks like you're not using the PrefixAllGlobals sniff as that would have complained as well.
The problem with your code is that, while in your example case, you are including the view file from within a function call, making the variable a local variable, there is no guarantee that the file will always be included from within a function call - think other plugins or themes interacting with yours, but also hackers -.
So, I would like to strongly suggest to enable the PrefixAllGlobals sniff and to prefix the $title variable with your project prefix.
See this page in the wiki about how to enable the PrefixAllGlobals sniff: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Customizable-sniff-properties#naming-conventions-prefix-everything-in-the-global-namespace

I understand and get all your points but:
@GaryJones your options are valid but let's say that I'm using namespacing in a plugin so everything that I define inside that namespace will live only there. Shiff should be able to se if I'm using namespacing or not. Also asking developer not to use for example $title variable because WP uses it not so friendly towards the developers. Also, I can't just echo the variable that I got from somewhere else because I can't know that that variable will exist or not.
@GaryJones This is a completely valid point if I'm writing on a theme that lives inside global namespace. But I'm using namespacing inside my plugin and I think that I'm safe with globals overriding my code.
Please correct me if I'm wrong.

let's say that I'm using namespacing in a plugin so everything that I define inside that namespace will live only there. Shiff should be able to se if I'm using namespacing or not.

The sniff does detect and take namespaces into account.
$variables are not namespaced, so what you say is incorrect and your code is still overriding the global variable.

ok tnx for your answers. I will close this issue.


Icons for the macros are pointing to the wrong directory

Macro icons are pointing to modules/com-hud/ui/
But the module is now named mist-hud. Thus the icons aren't found.

Solved, thanks!

The Otherscape icons are fine. The macros for City of Mist still point to the wrong directory.


Design Files

As I was browsing the stunning website, I became curious about how such beauty could be crafted. I speculated that it must have been designed using software before being brought to life, so I began searching for the design. However, despite my efforts, I was unable to locate it. (I am learning from the project.)

Check this out https://ui.shadcn.com/figma

Thank you


SharePoint Graph new Pages API: no webPart data returned for some sites

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
SharePoint REST API
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Describe the bug / error
For some sites, the Graph endpoint that should return webpart data does return an empty array.
https://graph.microsoft.com/beta/sites/{site-id}/pages/{page-id}?expand=webParts
or
https://graph.microsoft.com/beta/sites/{site-id}/pages/{page-id}/webParts
returns:
{
<EMAIL_ADDRESS>"https://graph.microsoft.com/beta/$metadata#sites({site})/pages({page})/webParts",
"value": []
}
For other sites, the endpoint does return data.
Steps to reproduce

This cannot be reproduced because it depends on the my site/page Id's.

Expected behavior
All our sites return webpart data. This is probably due to code rollout but we are actively testing the API and it would be very appreciated (if you can control it) that our sites in our tenant do return webpart data.
Question: if you can enable this for all of our sites, please contact me on<EMAIL_ADDRESS>so I can give tenant & site Id's.

@redlab-chris : Can you try this again and let me know if you are still seeing this error?

Yes still having this error for some sites. For those sites, we also get an error while expanding the webParts for all pages in the site using:
https://graph.microsoft.com/beta/sites/redlabapp.sharepoint.com,b2c6a78f-771a-4c4d-b681-47171dbc90e8,2a2f8caa-8baa-4c99-8133-feda60e2426c/pages?$expand=canvasLayout
or:
https://graph.microsoft.com/beta/sites/redlabapp.sharepoint.com,b2c6a78f-771a-4c4d-b681-47171dbc90e8,2a2f8caa-8baa-4c99-8133-feda60e2426c/pages?$expand=webParts

Another but probably related thing is that we sometimes don't get an error but no webParts. For a single page:
https://graph.microsoft.com/beta/sites/365redlab.sharepoint.com,c006d14d-1e79-4dc1-9048-9ec48991f220,46067dc3-f26c-414e-a048-994708d6bfd1/pages/7f0c7ae3-7c86-46bd-bb66-e0f6e2e9f85a?expand=webParts
or for all pages in the site:
https://graph.microsoft.com/beta/sites/365redlab.sharepoint.com,c006d14d-1e79-4dc1-9048-9ec48991f220,46067dc3-f26c-414e-a048-994708d6bfd1/pages?expand=webParts
result fragment for webParts:
"webParts": []
while the page has 1 text web part:

Issue resolved. Closing thread


Feat/119

Proposed changes
This PR doesn't close task #119, but start to introduce a set of components that will be part of the @morfeo/preset-default package.
Components are temporarily placed under the web-sandbox application. It introduces also a prototype for the generic morfeo component.
Types of changes
What types of changes does your code introduce to morfeo?

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #120 (7b37d32) into main (a0b0e6d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #120   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           72        72
  Lines          843       843
  Branches       119       119
=========================================
  Hits           843       843

Continue to review full report at Codecov.

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


Refactor/fetch definition fluent interface refactoring

breaking change, so i won't merge before preparing PRs in the dependent services
(formatting changes were done automatically by go fmt)

Coverage increased (+0.3%) to 93.788% when pulling e88fd8fb2ce2b68295e8714e4ae829efa3f2c8b3 on refactor/fetch-definition-fluent-interface-refactoring into da867f285b220de6b3e777c1aed3c53946c8b0af on master.


The Paid page doesn't update after a successful payment

If a Consumer is paying and initiates a Metamask transaction, when the tx confirms the Paid page doesn't update. The "Datastream Endpoint" field doesn't appear. If you reload then it does appear. We need to trigger a reload.

This is solved.


"daemon" deserves an entry in the main help

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-2" git@github.com:crquan/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@crquan why did you close? If it's because of the sign-off, please reopen and update this PR, using the instructions @GordonTheTurtle gave above :+1:

However, I noticed this comment in the source, so maybe that's related here; https://github.com/docker/docker/pull/15733/files#diff-51976df2ecfaa7e213c848b2ec76f6fbR21


*: Update dependencies

Dependency updates in preparation for the v2.22 release series.
@roidelapluie @brian-brazil @beorn7 @juliusv

ah, go lint is unhappy:
storage/remote/intern_test.go:36:48: printf: Sprintf format %d has arg interned.refs of wrong type go.uber.org/atomic.Int64 (govet)
	testutil.Assert(t, interned.refs.Load() == 1, fmt.Sprintf("expected refs to be 1 but it was %d", interned.refs))

Yeah working on fixing the linting errors.


Question: recommended settings for curl

FACT-Finder SDK version (see package.xml): 4.1.9
Magento edition and version: 1.14.2
PHP version: 5.5
Expected behavior: Seamless connections to FactFinder
Actual behavior: Every so often we get a "Resolving timed out after 2000 milliseconds" but then after refresh of page the problem goes away.

I am just wondering if there are any "recommended" settings for curl or other php settings that come with this module? I know the problem could also be with FactFinder or the server but I just wanted to make sure we are working with the recommended settings here.

@dmanners Sorry for the looong delay.
There are no recommended curl settings for something like this. 2 seconds is in any case too much. This does not normally depend on the Magento module, how fast the FF server responds. So you should configure both servers in accordance to each other.
I will later review the whole functionality and the documentation and if there appears anything that we would consider as "recommended" in regards of PHP settings, I will let you know and place it also into the docu.


version upgrades and trpc v10 syntax

This PR is a combined PR from two that I made; all changes are reflected here.
What's Changed
Updated tRPC syntax to v10

Better Typescript: the compiler is faster in v10, so it's doing a better job.

Different naming conventions: uses the current best practives from create-t3-app, specifically changing the trpc wrapper to be named api. Query/mutation hooks are now called as api.post.like.useMutation() etc.

Easier maintainability: along with faster TS, it puts the application in line with the current version of tRPC.

Upgraded versions of multiple packages to bring beam up-to-date

The package @instantish/mack, which was being used here, has been deprected, which would seem to be a part of the problem highlighted in issue #72. The new package that's recommended, @tryfabric/mack has been rewritten, moving away from rehype for parsing to marked, and has been updated here.

Updated next, next-auth, react, react-dom, and @radix-ui/react-tooltip to fix peer dependency errors that were happening on build. This should also do the work of #75, updating next-auth implementation in the code.

Updated prisma and @prisma/client and changed the schema.prisma to reflect new features. This should serve to fix #81.

I have not had the ability to test all of these updates in a live environment, so please hit me back with feedback/changes.

@thejessewinton Thanks so much for the work you've done here. I apologize for the lack of response here. We're currently working to free up some resources to help oversee the maintenance of this repo. My expertise is more on the design side and I'm not the best person to review this work. Hold tight, we'll get to this soon :bow:

@jasonlong you got it, glad I could help out!

I know the build process isn't ideal currently since the Vercel account we're using is part of the PlanetScale org. In the meantime, I can help surface deployment errors.
[19:51:00.715] uncaughtException Error: Next.js requires react >= 18.2.0 to be installed.
[19:51:00.715]     at Object.<anonymous> (/vercel/path0/node_modules/.pnpm/next@13.2.0_sfoxds7t5ydpegc3knd667wn6m/node_modules/next/dist/build/webpack-config.js:1797:11)
[19:51:00.715]     at Module._compile (internal/modules/cjs/loader.js:1085:14)
[19:51:00.715]     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
[19:51:00.715]     at Module.load (internal/modules/cjs/loader.js:950:32)
[19:51:00.716]     at Function.Module._load (internal/modules/cjs/loader.js:790:12)
[19:51:00.716]     at Module.require (internal/modules/cjs/loader.js:974:19)
[19:51:00.716]     at require (internal/modules/cjs/helpers.js:101:18)
[19:51:00.716]     at Object.<anonymous> (/vercel/path0/node_modules/.pnpm/next@13.2.0_sfoxds7t5ydpegc3knd667wn6m/node_modules/next/dist/build/webpack-build.js:12:46)
[19:51:00.716]     at Module._compile (internal/modules/cjs/loader.js:1085:14)
[19:51:00.716]     at Object.Module._extensions..js (internal/modules/cjs/loader.js:1114:10)
[19:51:00.720]  ELIFECYCLE  Command failed with exit code 1.
[19:51:00.735] Error: Command "pnpm run build" exited with 1
[19:51:00.823] An error was thrown and it will end the deployment but the thrown error had no message body. Perhaps other parts of the log offer a clue.

@jasonlong very weird error, React version is fully current, and it’s not something I can replicate locally. Will keep doing some research, but was the error message any more verbose than that?

The only other thing I can think is possibly an outdated package-lock.json. I'm going to use pnpm, create a new lockfile and push.


Pandoc export failed: TypeError: Cannot read properties of null (reading 'data')

Pandoc export failed: TypeError: Cannot read properties of null (reading 'data')

I have the same issue - happens every time I try to use the pandoc plugin

Same here. Additionally, there is no "Export to PDF" option in the Command Palate.

Any update on this? I have just installed the plugin, and get this error.

I also get this error intermittently (cannot read null, and missing export to PDF), but it vanishes when I reload the plugin / restart obsidian. Classic heisenbug

I'm seeing this also, is there a work around?

My PR (#191) is a hacky workaround that rephrases the latex syntax to work normally.

I had this error when trying to export a note containing other notes embedded as
[[other-note]]
and the error disappeared when I switched to
![[other-note]]

Turning Internal link processing to Leave unchanged fixed this problem for me.


How to build kernel with 9p (v9fs) enabled for CentOS 6?

This is slightly off-topic but I think we need to document how to build Linux kernel 2.6.32 with 9p (v9fs) support for CentOS 6. It seems v9fs is disabled by default.
CONFIG_9P_FS is not set
Reference: http://wiki.qemu.org/Documentation/9psetup
/home/tatsuya/tmp/centos63% vagrant ssh
Welcome to your Vagrant-built virtual machine.
[vagrant@centos6 ~]$ uname -a
Linux centos6 2.6.32-431.el6.x86_64 #1 SMP Fri Nov 22 03:15:09 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux
[vagrant@centos6 ~]$ grep CONFIG_9P_FS /boot/config-$(uname -r)
# CONFIG_9P_FS is not set

Log
/home/tatsuya/tmp/centos63% vagrant up
Bringing machine 'default' up with 'kvm' provider...
==> default: Importing base box 'centos-6.5'...
...
==> default: mounting p9 share in guest
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

mount -t 9p -o trans=virtio,version=9p2000.L '/vagrant' /vagrant

Stdout from the command:

Stderr from the command:

mount: unknown filesystem type '9p'

/home/tatsuya/tmp/centos63% vagrant ssh
Welcome to your Vagrant-built virtual machine.
[vagrant@centos6 ~]$ sudo modprobe 9p2000
FATAL: Module 9p2000 not found.
[vagrant@centos6 ~]$ sudo modprobe -l | grep 9p
kernel/net/9p/9pnet.ko
kernel/net/9p/9pnet_virtio.ko
kernel/net/9p/9pnet_rdma.ko

Just in case if it is still actual, I have kmod setup for 9p for centos6/rhel6.
https://github.com/antst/centos-9p


WIP: Get product by mission and/or instrument

Fixes #19

Add mission tags
Fix instrument name to title case rather than all upper case

TODO:

[ ] Add commands to list all missions and/or instruments
[ ] Add commands get products with matching mission and/or instrument

Coverage remained the same at 100.0% when pulling 37f6f49d32ca37910e977909d49717b8eef3b47c on pbvarga1:issue19_get_by_instrument into 0f73cd4221b29f298580c652de147cac488f1de3 on planetarypy:release-0.4.0.

Coverage remained the same at 100.0% when pulling 37f6f49d32ca37910e977909d49717b8eef3b47c on pbvarga1:issue19_get_by_instrument into 0f73cd4221b29f298580c652de147cac488f1de3 on planetarypy:release-0.4.0.

Coverage remained the same at 100.0% when pulling 6dc4f42e01245aa4bf46683600a50e5ef967ae25 on pbvarga1:issue19_get_by_instrument into 0f73cd4221b29f298580c652de147cac488f1de3 on planetarypy:release-0.4.0.


Remove email+pw signup from marketing website

Places to remove email+pw signup on public properties:

https://sourcegraph.com/cody
https://about.sourcegraph.com/
Modal on about.sourcegraph.com
https://about.sourcegraph.com/cody
https://sourcegraph.com/get-cody

Update on github.com/sourcegraph/about

CC: @dominiccooney @toolmantim @danielmarquespt @unknwon @bobheadxi

This should cover the about required about site changes:
https://github.com/sourcegraph/about/pull/6478

Reassigning back to me for prioritization.

This is already done


ToRGB() Transform

🚀 The feature
Some datasets (like ImageNet) contain both RGB images and grayscale images; to make the images batchable, they all need to have the same channel dimension. The following transform solves this problem by converting all grayscale images to RGB without branching.
class ThreeChannel(torch.nn.Module):
    def __init__(self):
        super().__init__()
    def forward(self, input: Tensor):
        return input.expand(3, -1, -1)

Unfortunately, it is only compatible with 3D Tensors in [C, X, X] format. However, since transforms are applied per image in most torchvision datasets, this shouldn't be a problem.
Motivation, pitch
Some datasets (like ImageNet) contain both RGB images and grayscale images; to make the images batchable, they all need to have the same channel dimension.
Alternatives
No response
Additional context
No response

Thanks for the feature request @kevinMEH . That sounds reasonable but before we commit to adding a ToRGB() transform, have you considered decoding the images directly into RGB format? E.g. either using img.convert("RGB") if img is a PIL image, or
if you're decoding tensors directly you could use read_image(mode=ImageReadMode.RGB).
LMK if this doesn't address your use-case

Thanks for the suggestion! I did not even realize that image.convert("RGB") was an option (I am not familiar with PIL images). I am now using the following transform before ToImage() and it seems to work. Perhaps it can be added to the default transforms library?
class ToRGB(torch.nn.Module):
    def __init__(self):
        super().__init__()
    def forward(self, image):
        return image.convert("RGB")

Glad it works. Calling .convert("RGB") is actually what happens by default in the datasets, but it's true that it's a bit hidden.
We could add a new ToRGB() transform to extract that logic out. One minor caveat here is that converting to RGB while decoding


Unexpected error when restoring a database

I created a simple call to restore a database but even when the restore is done without any error in the MSSQL server, I'm getting this error from the driver:
mssql: RESTORE DATABASE is terminating abnormally
This is how I implemented the function:
query := `RESTORE DATABASE mydb FROM DISK = 'mybackup.bak' WITH RECOVERY`
_, err := d.conn.Exec(query)
return err

Connect to master DB in connection string, not restoring DB?

I tried that approach but again it only works fine when the restore is forced with replace

I have no idea. Looks like SQL Server is aborting the connection for some reason. The only thing I can think of is if SET ENV TDS command is not being processed correctly.  Please re-open if have a suggestion or if we can document a different client behaving differently.


Accept HPA memory target as %

When the autoscaling metric is set to "memory", Knative expects the target to be supplied in the unit of Mi (ref). It is common for users to configure the HPA using resource utilization % instead and this is the unit used for the "cpu" metric by Knative as well. Moreover, if Mi is used as the unit, the users may need to update the autoscaling target value every time the memory requested is altered.
Thus, this PR changes the handling of the target value set for the memory metric so that the saved config will be treated as % and subsequently applied as Mi. This will make the configuration consistent for the end users.
Main changes

api/turing/cluster/knative_service.go - Add method getAutoscalingTarget for manipulating the target, if "memory" is used.
api/turing/cluster/knative_service_test.go - Add unit tests for the getAutoscalingTarget method, to test the conversions
Update the documentation in the UI, for the memory unit

CC: @ariefrahmansyah

Thanks for the review! Will merge it shortly.


Website Security Warning: NET::ERR_CERT_COMMON_NAME_INVALID

Description
When visiting https://reactnative.dev/ you are greeted with a security warning:

I assume this error originates from this certificate:

It is not possible to visit the docs as HSTS is enabled.
Documentation version
N/A

Closing as this is fully resolved now


Enhancement: Add support for OpenAPI objects like x-codeSamples

Summary
Redocly has support for x-codeSamples as an OpenAPI object (or the x- vendor extensions entirely)
https://redocly.com/docs/api-reference-docs/specification-extensions/x-code-samples/#code-sample-object
This results in the docs in something like this:

This would greatly help in (quickly) testing the API with examples to copy and paste.
Basic Example
One way to do it would be to implement it similar to FastAPI:
@get(
  path="/example",
  openapi_extras={
    "x-codeSamples": [
      {"lang": "Python", "source": "import requests; requests.get('localhost/example')", "label": "Python"},
      {"lang": "cURL", "source": "curl -XGET localhost/example", "label": "curl"},
      {...}
    ]}

Drawbacks and Impact
No response
Unresolved questions
No response

Certainly would be nice to be able to support that.
Is there anything that would go in these that couldn't be generated from the route handler info? Maybe we could build a plugin to generate them..

Would be great to have them auto generated (perhaps with an override) based on the route/parameters.

Is there anything that would go in these that couldn't be generated from the route handler info?

What do you mean by 'these'? For the redocly vendor extensions the only ones which seemed relevant and are part of the operations section of the openapi spec are

x-codeSamples: list
x-hideTryItPanel: bool

By "these" I mean these:
[{"lang": "Python", "source": "import requests; requests.get('localhost/example')", "label": "Python"},
      {"lang": "cURL", "source": "curl -XGET localhost/example", "label": "curl"}]

I feel like it should be trivial to generate them  based on the info we have attached to the handler.

how would you actually set values once you have the class?

from litestar.openapi.spec import Operation

@dataclass
class CustomOperation(Operation):
    x_scopes: Optional[List[str]] = field(default=None, metadata={"alias": "x-scopes"})
    x_requires_auth: Optional[bool] = field(default=False, metadata={"alias": "x-requires-authentication"})

    def __post_init__(self) -> None:
        self.x_scopes = []

this works fine and has the default values in it.. but for every route that includes the operation_class (handle.operation_class = CustomOperation)
but then how do you actually set the value on those for a route? right up until the openapi schema is generated the operation_class is just a pointer to an uninitialized class. it only gets initialized in _openapi/path_item.py
operation = route_handler.operation_class(
                operation_id=operation_id,
                tags=tags,
                summary=route_handler.summary or SEPARATORS_CLEANUP_PATTERN.sub("", route_handler.handler_name.title()),
                description=get_description_for_handler(route_handler, use_handler_docstrings),
                deprecated=route_handler.deprecated,
                responses=create_responses(
                    route_handler=route_handler,
                    raises_validation_error=raises_validation_error,
                    schema_creator=response_schema_creator,
                ),
                request_body=request_body,
                parameters=parameters,  # type: ignore[arg-type]
                security=security,
            )

and as far as i can tell isn't available to add values to the fields after that.


About me and Contact sections are not clearly delineated on Home page

On home navigation, about me and contacts do not have distinctive sections.

Created headers to separate these items and display them with very clear sections.


Details Tag works in Chrome and Not in IE 11

Problem with 'GOV.UK Frontend' (https://frontend.design-system.service.gov.uk/)
I am using the following code, to get the details control.
I have tried with the  1) Nunjucks and 2)HTML.

Nunjucks

HTML

Problem
It renders okay in

Chrome
Firefox
but in IE 11, I get the following issue.

In Chrome

why it works in chrome and not in IE 11

Thanks for raising this issue @hari-gillala. We haven't been able to replicate it in IE11 so I'm going to close this issue. However if you have a version we could take a look at please let us know and we can investigate. If you prefer, you can contact us via Slack or email.


[disk-io] Filter Disks which are really mounted

Describe the solution you'd like
Additional to #708, I would be perfect that the plugin would find "important" disks by itself and do not waste influx disk space with unnecessary disk information. (I have 15GB of data only for this measurement for 14 Hosts with some months of data in it)
I have some servers with 122 disks/partitions are found from this plugin, but only 10 are really mounted. This could happen to many Linux features for example multipath or Linux raid.
multipath has many sd* disks with nothing needed from them, because they will be merged in dm-* virtual disk and only this disk will be mounted and is interesting for disk I/O
Same thing for Linux raid you have 12 physical disks with sda, sda1, sdb, sdb1, .... but they are useless in disk I/O info’s because that is only raid magic which happens there, interesting is only md0 which is the virtual disk for the Linux array and that is very my data flows.
I think the best way to figure out if a disk is important is, when it is mounted to a folder.
Maybe that is only 99% and may someone will find a disk which is not mounted but it is important. So, you can implement a switch for filter_disks_mounted=yes|no or you make force_include_disk=xyz parameter, for special use.
Additional context
No response

Very interesting, thank you.

Fixed by https://github.com/Linuxfabrik/monitoring-plugins/commit/173edda33cfe511c9a7dd232434664bf71bf59c1


Avatar component prop has a typo: placehoder should be placeholder

Thanks for taking the time to fill out this bug report!
Describe the bug
Avatar component prop has a typo: placehoder should be placeholder
To Reproduce
https://flowbite-svelte.com/avatar
Code samples
Simple code that reproduces the bug.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please update to v0.21.5 and let us know.
https://flowbite-svelte.com/avatar

I close the issue for now.
Thank you for your help.


Update CheckoutAction and CheckoutActions to have status-specific labels

Type: minor
Describe the bug
A checkout action's label changes as the status changes:
- "Enter shipping address” when you’re on that step, and then shortened to “Shipping address” for the collapsed step
- "Choose a shipping method” -> “2. Shipping method"

To Reproduce
Currently, checkout action labels are static and do not change with status change.
Expected behavior
A user should be able to input 2 labels, a instruction label and a post-success label.
The component should know when to show which label.

Here are some more specific implementation guidelines:

In CheckoutAction component, remove the label prop and replace with three props: activeLabel, completeLabel, incompleteLabel
Also in CheckoutAction component, in each of the render*Action functions, replace props.label use with the matching specific label prop
In CheckoutActions component, update actions shape to have the three separate labels instead of just label.  In renderAction, pass all of those to CheckoutAction rather than label.
Also in CheckoutActions component, we currently key by label. Instead, add a new id prop on the actions shape and key by that. This means updating getCurrentActionByLabel and currentActions.find by label, etc.
Update the CheckoutActions markdown to have separate labels to show that it works

:tada: This issue has been resolved in version 0.41.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


TS Server fatal error:  Debug Failure.

TypeScript Version: 3.9.5
Steps to reproduce crash

index.html

<html>

<body>
    <script<EMAIL_ADDRESS>    <script src="./sketch.ts"></script>
</body>

</html>

sketch.ts

// https://github.com/processing/p5.js/wiki/Global-and-instance-mode
const s = (sketch) => {
  let x = 100;
  let y = 100;

  sketch.setup = () => {
    sketch.createCanvas(400, 200);
  };

  sketch.draw = () => {
    sketch.background(0);
    sketch.fill(255);
    // sketch.rect(x, y, 50, 50);
  };
};

let myp5 = new p5(s);

Open ts file
TS Server Error Stack

Error: Debug Failure.
    at Object.releaseDocumentWithKey (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:114902:35)
    at Object.onReleaseOldSourceFile (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:138758:34)
    at Object.createProgram (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:100035:26)
    at synchronizeHostData (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:138726:26)
    at Proxy.getProgram (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:138818:13)
    at InferredProject.Project.updateGraphWorker (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:143310:53)
    at InferredProject.Project.updateGraph (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:143253:42)
    at ProjectService.assignOrphanScriptInfoToInferredProject (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:145149:25)
    at ProjectService.assignProjectToOpenedScriptInfo (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:146643:26)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:146900:97
    at Object.flatMap (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:465:25)
    at ProjectService.applyChangesInOpenFiles (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:146900:41)
    at Session.handlers.ts.createMapFromTemplate._a.<computed> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:147763:46)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:149526:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:149517:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:149526:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:149550:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:150866:27)
    at Interface.emit (events.js:210:5)
    at Interface._onLine (readline.js:316:10)
    at Interface._normalWrite (readline.js:461:12)
    at Socket.ondata (readline.js:172:10)
    at Socket.emit (events.js:210:5)
    at addChunk (_stream_readable.js:308:12)
    at readableAddChunk (_stream_readable.js:289:11)
    at Socket.Readable.push (_stream_readable.js:223:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:182:23)

Similar error stack to #38505

I dont see the exception if i create a folder with these two files and open ts file or open vscode in empty directory and then create these two files.
Can you please provide more details on what steps you took. tsserver log would be helpful too.
Thanks.

I tried same setting for new version of vscode and typescript , but i cant reproduce this error. I forgot, I also use deno extension.


clarified the process of CLA agreement checking

Hello everyone! When I first tried to submit a PR to the facebook/react, I thoroughly explored the docs for contributors and after reading If you are submitting a pull request for the first time, just let us know that you have completed the CLA and we can cross-check with your GitHub username, I thought that I need to tell about my CLA in the PR description, but in reality facebook-github-bot automatically verifies it and there's no need to additionally tell about your CLA. So I decided to clarify this little part of the documentation. I hope this is going to be helpful to the future contributors.

Hello @bwv847!
Yes, the Bot automatically checks if the CLA is signed and suggests steps if it's not. For that reason there's no mention of CLA process anymore in the contributing guidelines, as the current solution let's community to smoothly go through if needed.
I will close this PR as the sections you update are no longer there in the docs, but thank you for taking the time to contribute!