runtime error: invalid memory address or nil pointer dereference

Describe the bug
I keep getting the following error:
iBazel [10:37AM]: Error getting Bazel info exit status 2
iBazel [10:37AM]: Querying for files to watch...
iBazel [10:37AM]: Error getting Bazel info exit status 2
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x7a2a00]

goroutine 1 [running]:
main.(*IBazel).realLocalRepositoryPaths(0xc000150000, 0x921e00, 0xc0000a8120, 0x0)
        ibazel/ibazel_unix.go:31 +0x50
main.(*IBazel).queryForSourceFiles(0xc000150000, 0xc00030a020, 0x1d, 0x43d01f, 0xc00003d800, 0x4d7c1c, 0xc00030c014, 0x7ffe232b1091)
        ibazel/ibazel.go:454 +0x6e
main.(*IBazel).watchFiles(0xc000150000, 0xc00030a020, 0x1d, 0x91f800, 0xc00000e030)
        ibazel/ibazel.go:502 +0x5d

I saw an issue recently (#386) where this was supposedly fixed, but I think I am on the latest version of ibazel and I'm still seeing this problem. Also I think the issue is different - the earlier issue mentioned a broken WORKSPACE file, whereas the WORKSPACE file I'm working with is fine.
Reproduction instructions
I haven't yet found a consistent repro but I likely will in the next few days. The Git repo I'm working in is https://github.com/buildbuddy-io/buildbuddy (the issue happens when running ibazel run server). If I run rm -rf node_modules then the problem goes away temporarily. Not sure yet what causes it to pop back up.
Sometimes I'm using USE_BAZEL_VERSION=3.4.0 so maybe that is messing with things. If I can isolate it to a particular pattern then I'll follow up.
Version (please complete the following information):

OS: Ubuntu 20.04
ibazel 0.13.1
Bazel 3.1.0

Bazelisk version: v1.6.1
Build label: 3.1.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Wed Apr 22 10:32:27 2020<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Hey Brandon, please say "hi" to Siggi and Tyler for me. Thanks for the bug report! I would love to look into this more once we get something reproducible. NPEs are the absolute worst and very maddening for both developers and consumers. node_modules is a tricky beast so I'm not super surprised that there is something going on in there. Any chance your ibazel is coming from node_modules instead of from the one on your system? ibazel, as it is distributed by npm, is actually a node script that tries to find a viable ibazel on your path. Could that be part of the issue?

Will do!
When I run which ibazel I get ~/.yarn/bin/ibazel which is a symlink to ~/.config/yarn/global/node_modules/.bin/ibazel, so I guess this means it's the one I installed with yarn global add? Let me know if there's anything else I can do to help

I did notice a pattern where I can only run ibazel without this exception if I first run rm -rf node_modules. So basically I have to run every ibazel command with rm -rf node_modules && ibazel ... otherwise I get the exception

Could I have you build and run from master. It looks like I forgot to do a released since then. Lemme kick that off.

I ran the "release this" program but it was stuck waiting for me to type in an OTP. I just found that tab in the background and now it is actually released. Sorry for the delay! I'm going to leave this open in case you run into it again, please comment on the bug. There is a bug autocloser that will close this after 60 days of inactivity or something like that, so NBD if you forget about it.

I can confirm this happens, when using the npm package @bazel/ibazel to run the program. If the node_modules/ dir is present, it segfaults. rm -rf, restart, works fine, until restart again with node_modules/ back.
Removing @bazel/ibazel from my machine, just building ibazel tool out of git, and moving it into $PATH works-for-me.

I did a release this morning. Is this error report for v0.14?


Ground Station not updating

[ERROR] [1712906202.752508397]: Client [/ground_station] wants topic /prometheus/log/control to have datatype/md5sum [prometheus_msgs/LogMessageControl/767e4a3010d3d61d9f8e8972600381a9], but our version has [prometheus_msgs/LogMessageControl/05aaccac2f3815ac451740cffc55ed71]. Dropping connection.

删除install、build、devel中有关ground_station的文件夹，重新编译安装。


New version: HenriWahl.Nagstamon version 3.14.0

Pull request has been created with komac v2.1.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree


Remove doubled border for list-groups beeing the last item inside cards.

Remove doubled border for list-groups beeing the last item inside cards.

I think a22d2fb5a5a702997509f29d86dc37ba917a103a might've already fixed this.

Ah this is already fixed in a22d2fb. Sorry was not running the latest code.


[5.5][Diagnostics] Adjust Self conformance check to find non-decl overload choices

Cherry-pick of https://github.com/apple/swift/pull/38045

Explanation:

Use findSelectedOverloadFor instead of findResolvedMemberRef
to cover cases where member is found via base type unwrap, otherwise
conformance constraint would be re-inserted but never re-attempted
which results in a compiler crash.

Scope: Expressions with leading-dot syntax where one of the conformance requirements is not satisfied and member itself is found via unwrap of the optional base type.

Main Branch PR: https://github.com/apple/swift/pull/38045

Resolves: rdar://79268378

Risk: Low

Reviewed By: @hborla

Testing: Regression tests added to the suite

Resolves: rdar://79268378
(cherry picked from commit be6e2fad8c28265068dd4299e72c90c796e771a2)

@swift-ci please test

@swift-ci please test Windows platform


Improvements to README

The README file is sparse.
Some particular omissions:

To get the python scripts to run, I needed to install the following packages in the vagrant VM via pip:

base58, asn1, enum34

For latest scripts, (which depend on pybitshares), Python3 is needed.  Follow install directions for pybitshares. Also, these are needed:

Use pip3 install to install:

(if setup tools is old, update first with pip3 install --upgrade setuptools.)
asn1, ledgerblue

Readme has been updated for 0.0.0 Release and addresses these issues.


Support type "Boolean"

Hi :)
And thanks for this very useful plugin.
Do you plan on adding the type "Boolean"?
https://prismic.io/blog/prismic-boolean-fields
Right now when using it I'm getting:
Error: Cannot get field 'show_as_button' from type 'PrismicGlobalElementsBodyNavigationItemPrimaryType'. Field does not exist.

This is how the schema for that type looks like:
"show_as_button" : {
  "type" : "Boolean",
  "config" : {
    "placeholder_false" : "Yes",
    "placeholder_true" : "No",
    "default_value" : true,
    "label" : "Show as button"
  }
}

Hi @apdrsn, v3.0.0 was just published and includes support for Boolean.
It looks like you might not be on latest version so here's a migration guide: https://github.com/angeloashmore/gatsby-source-prismic/blob/95b85688a58efc5cb5cbf78628b6f43eb6cdaa96/docs/migrating-from-v2-to-v3.md

Hi @angeloashmore
I was using 3.0.0-beta.25 – updating now :) Thanks a lot 👍


Change many incorrect uses of ValueError to TypeError

These are TypeError instead of ValueError. See https://docs.python.org/3/library/exceptions.html#TypeError :
Passing arguments of the wrong type (e.g. passing a list when an int is expected) should result in a TypeError, but passing arguments with the wrong value (e.g. a number outside expected boundaries) should result in a ValueError.

Sadly the exceptions raised by TensorFlow are part of our public API, and as such we cannot change them, as changing would break existing code which uses try...except to catch exceptions.

Sadly the exceptions raised by TensorFlow are part of our public API, and as such we cannot change them, as changing would break existing code which uses try...except to catch exceptions.

Thanks @alextp. I also found many occurrences of TypeError. For consistency, do you think we also need to modify them?

@I-Hong sadly API stability trumps consistency :-/
We could have reasonably made this change for 2.x only before the 2.x alpha, but that ship has sailed.


converting issue

I tried using the fnirs on a head model to check the event marker. Later, I converted the result into a snirf file but found that I cant open the converted snirf file by Homer3.  This is the error returned by matlib:
test5.snirf:  Invalid File Name
Dot indexing is not supported for variables of this type.

Error in MainGUI>listboxFilesErr_Callback (line 648)
if ishandle(maingui.handles.msgbox)

Error in gui_mainfcn (line 95)
feval(varargin{:});

Error in MainGUI (line 21)
gui_mainfcn(gui_State, varargin{:});

Error in matlab.graphics.internal.figfile.FigFile/read>@(hObject,eventdata)MainGUI('listboxFilesErr_Callback',hObject,eventdata,guidata(hObject))

Error while evaluating UIControl Callback.

DataFilesClass.ErrorCheck - ERROR: In file "snirf_test3 (1).SNIRF" "data"  field is invalid.. File will not be added to data set
DataFilesClass.ErrorCheck - ERROR: In file "test5.snirf" "data"  field is invalid.. File will not be added to data set
Waiting for user responce, t = 30 ticks
Do you have any idea what was going on in the data?

Unfortunately Homer3 doesn't fully support the SNIRF specification (for example see https://github.com/BUNPC/Homer3/issues/180), and I expect this might be the same problem.
Are you able to upload a small example file to this issue so that we can check?

Thank you @labteng, we will look into this and come back to you early next week.
@nidadu are you able to confirm that this file is indeed triggering https://github.com/BUNPC/Homer3/issues/180 ?

@samuelpowell I can confirm that this data triggers the same part of the code and produces the same error as in https://github.com/BUNPC/Homer3/issues/180.

Thanks @nidadu, in the absence of a fix in Homer the quickest thing we can do is to add an option to export a full time series format for the data. This could be guarded by a switch. If you have time to do this today would you open a PR, else we’ll revisit next week.

Thank you for checking the bug. I just found that when I convert the data into .nirs and use Homer 3 to make it a .snirf, somehow, I can avoid the bug and read the data successfully. However, accelerometer data is still invisible. I was supposed that accelerometer data had been enclosed during converting as I can clearly see that occurring during the recording. Is accelerometer missing due to the format conversion?

Indeed, Homer3 will not convert any of our additional metadata, hence it will be lost during format conversion. In any case, you should not have to go through this process. We will prepare an update to this package for you to test by close of business today.

@labteng the linked PR #48 provides an additional options for SNIRF output which writes the time vector in the way that Homer3 expects. For example
snirf = data.write_SNIRF(snirf_filename, 'time', 'explicit');

We will perform testing and release the new version when validated, in the mean time you are welcome to download the branch and test yourself.

@labteng the explicit time vector option is now available in lumomat version 1.7. Please try exporting your data using:
snirf = data.write_SNIRF(snirf_filename, 'time', 'explicit');
This should load cleanly in Homer3. If you have any problems, please reopen this issue.

@samuelpowell. Would you please help me check again for the conversion function? I tried your method, but the Homer3 could not successfully load the event marker, and the timeline will only go up to 6 seconds. While the aux data show up indeed. Thank you for that.
See if the log can help:
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\

In defaulterrorcallback (line 12)
In matlab.ui.internal.legacyMoveGUI (line 15)
In movegui (line 125)
In matlab.hg.internal.openfigLegacy (line 102)
In gui_mainfcn>local_openfig (line 286)
In gui_mainfcn (line 233)
In MainGUI (line 19)
In Homer3 (line 58)

ans =
 []

Displaying   time: [7745x1],    data: [7745x3456],   channels []
Warning: Rank deficient, rank = 0, tol =  0.000000e+00.

In MainGUI>DisplayAux (line 1350)
In MainGUI>DisplayData (line 1179)
In MainGUI>Display (line 1038)
In MainGUI>checkboxPlotAux_Callback (line 1643)
In gui_mainfcn (line 95)
In MainGUI (line 21)
In matlab.graphics.internal.figfile.FigFile/read>@(hObject,eventdata)MainGUI('checkboxPlotAux_Callback',hObject,eventdata,guidata(hObject))

Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels []
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [2305  2306  2307  2308  2309  2310  2311  2312  2313  2314  2315  2316  2317  2318  2319  2320  2321  2322  2323  2324  2325  2326  2327  2328  2329  2330  2331  2332  2333  2334  2335  2336  2337  2338  2339  2340  2341  2342  2343  2344  2345  2346  2347  2348  2349  2350  2351  2352]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [2353  2354  2355  2356  2357  2358  2359  2360  2361  2362  2363  2364  2365  2366  2367  2368  2369  2370  2371  2372  2373  2374  2375  2376  2377  2378  2379  2380  2381  2382  2383  2384  2385  2386  2387  2388  2389  2390  2391  2392  2393  2394  2395  2396  2397  2398  2399  2400]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [2353  2354  2355  2356  2357  2358  2359  2360  2361  2362  2363  2364  2365  2366  2367  2368  2369  2370  2371  2372  2373  2374  2375  2376  2377  2378  2379  2380  2381  2382  2383  2384  2385  2386  2387  2388  2389  2390  2391  2392  2393  2394  2395  2396  2397  2398  2399  2400]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [2353  2354  2355  2356  2357  2358  2359  2360  2361  2362  2363  2364  2365  2366  2367  2368  2369  2370  2371  2372  2373  2374  2375  2376  2377  2378  2379  2380  2381  2382  2383  2384  2385  2386  2387  2388  2389  2390  2391  2392  2393  2394  2395  2396  2397  2398  2399  2400]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
ProcStreamOptionsGUI  size: [125.00, 29.00]
Error using signal.internal.spectral.welchparse>parse_inputs
Window size must be larger than 1 sample.
Error in signal.internal.spectral.welchparse (line 31)
parse_inputs(x1,esttype,varargin{:});
Error in welch (line 55)
signal.internal.spectral.welchparse(x,esttype,args{:});
Error in pwelch (line 170)
[welchOut{1:nargout}] = welch(x,funcName,inputArgs{:});
Error in MainGUI>menuItemPowerSpectrum_Callback (line 1941)
[pxx,f] = pwelch(d(:,iCh(i)), window, overlap, bins, fs);
Error in gui_mainfcn (line 95)
feval(varargin{:});
Error in MainGUI (line 21)
gui_mainfcn(gui_State, varargin{:});
Error in matlab.graphics.internal.figfile.FigFile/read>@(hObject,eventdata)MainGUI('menuItemPowerSpectrum_Callback',hObject,eventdata,guidata(hObject))
Error while evaluating Menu Callback.
Displaying   time: [7745x1],    data: [7745x3456],   channels [2353  2354  2355  2356  2357  2358  2359  2360  2361  2362  2363  2364  2365  2366  2367  2368  2369  2370  2371  2372  2373  2374  2375  2376  2377  2378  2379  2380  2381  2382  2383  2384  2385  2386  2387  2388  2389  2390  2391  2392  2393  2394  2395  2396  2397  2398  2399  2400]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [2641  2642  2643  2644  2645  2646  2647  2648  2649  2650  2651  2652  2653  2654  2655  2656  2657  2658  2659  2660  2661  2662  2663  2664  2665  2666  2667  2668  2669  2670  2671  2672  2673  2674  2675  2676  2677  2678  2679  2680  2681  2682  2683  2684  2685  2686  2687  2688]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\
Displaying   time: [7745x1],    data: [7745x3456],   channels [961   962   963   964   965   966   967   968   969   970   971   972   973   974   975   976   977   978   979   980   981   982   983   984   985   986   987   988   989   990   991   992   993   994   995   996   997   998   999  1000  1001  1002  1003  1004  1005  1006  1007  1008]
Warning: Error updating Legend.
String scalar or character vector must have valid interpreter syntax:
\

Hi,
I got some warnings while converting .lufr to .nirs for some of my data points:

Warning: The specified LUFR file contains incomplete frames. Consult the frame error
counter to determine those frames for which the data is incomplete. Note that missing
data will present as NaN in the associated data structure

Warning: Layout ID (0x0000012b) does not match group ID (0x0000012a), enumeration may be inconsistent

I'm wondering what each warning means. does that mean the data point has some problems and it is not usable? and Can I do anything to fix it?
Any help would be appreciated!


add accept headers

Added accept header to get request made in test 5.2.05 to address discrepancy noted here. This PR fixes GitHub Issue #47

Please add the ticket to the title before merging.


IMPORTANT: draggable DOM attribute blocks text selection

Describe the bug
If you pass down the drag property to the ref attribute of your DOM element, then a draggable native html attribute will be set to true. It doesn't matter if canDrag is set to false. The draggable HTML attribute is always true and because of this, the user cannot select a text in this component.
Reproduction
https://codesandbox.io/s/quirky-chaplygin-89xqe
Steps to reproduce the behavior:
Just try to select the text "My Draggable component". You will see that this doesn't work.
Afterwards you can take a look into the MyDraggableComponent.tsx. As you can see, the canDrag option is set to false.
Expected behavior
If canDrag is set to false, I expected that the HTML attribute draggable is set to false.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.89 Safari/537.36
Version react-dnd: latest

Edit:
I saw that the draggable attribute comes from HTML5BackendImpl class.

As you can see, the 'true' parameter is hardcoded and not variable. Are there any workarounds for this?

Edit 2:
If anyone have to same problem:
I have found a workaround for this. It does fix the issue, but isn't a recommended way.
   useLayoutEffect(() => {         if (rootRef.current) {             if (disabled) {                 rootRef.current.setAttribute('draggable', 'false');             } else {                 rootRef.current.setAttribute('draggable', 'true');             }         }     }, [disabled, rootRef.current]);

Related https://github.com/react-dnd/react-dnd/issues/2565 I also tested that this prevents text selection in both chrome and safari as well.
Another workaround that's slightly more declarative is to set draggable={false} on a container element lower down then the drag element.

+1

still seeing this with most recent release, I wonder if it would just be possible to make react dnd conditionally set draggable based on the canDrag option it's passed

我用了相似的一种解决方法，思路是相似的。在鼠标onMouseEnter进入input时，解绑drag元素上的ref，将标签中的ref赋值为null。在鼠标onMouseLeave时再恢复。但不推荐这样做，而且需要写注释，否则后来者无法看懂。需要注意的是，chrome上用此方法无问题，但 safari and FF 上也需要测试一下，如果无缺陷即可行，如果有缺陷则需另寻他法。

编辑2： 如果有人遇到同样的问题： 我已经找到了解决方法。它确实解决了问题，但不是推荐的方法。
useLayoutEffect(() => {
    if (rootRef.current) {
        if (disabled) {
            rootRef.current.setAttribute('draggable', 'false');
        } else {
            rootRef.current.setAttribute('draggable', 'true');
        }
    }
}, [disabled, rootRef.current]);


Add support for storing Google Cloud key as ENV var directly

This is a little verbose, but was in a hurry and wanted the fewest diffs.
Being able to use the key directly is much simpler for multi env production apps so we don't have to deal with files, especially when we only use Google Cloud for Vertex.

@weilandia, thanks for opening the pull request!
Yeah, I got it. So, I'm thinking about the name of the key. json_key works but can be a bit confusing. I was thinking about keeping the semantics of file and using file_contents so it can match the already existing file_path:
client = Gemini.new(
  credentials: {
    service: 'vertex-ai-api',
    file_path: 'google-credentials.json',
    region: 'us-east4'
  },
  options: { model: 'gemini-pro', server_sent_events: true }
)

So, it would be:
client = Gemini.new(
  credentials: {
    service: 'vertex-ai-api',
    file_contents: File.read('google-credentials.json'),
    region: 'us-east4'
  },
  options: { model: 'gemini-pro', server_sent_events: true }
)

client = Gemini.new(
  credentials: {
    service: 'vertex-ai-api',
    file_contents: ENV['GOOGLE_CREDENTIALS_FILE_CONTENTS'],
    region: 'us-east4'
  },
  options: { model: 'gemini-pro', server_sent_events: true }
)

How do you feel about it?

Sounds good, I updated the PR.

@weilandia Awesome, thanks for your contribution!


Added kist college domain

Established in 1995, KIST is now a top +2 college in Nepal that offers high standard NEB + 2 Science & Management courses, a range of top quality Bachelor’s degrees – BBA, BIM, BIT, BBS & BSc Microbiology – followed by excellent Master’s degrees – MBS & MSc Microbiology.
Domain: https://kist.edu.np/introduction

@sapkotamadhusudan First stage of verification passed, official domain is kist.edu.np as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto Thanks for the updates 😊

Hi @philipto any updates on this?

@sapkotamadhusudan Thank you for your patience while waiting for a review and processing your request. Pull request merged.


CASMPET-5181 : Adding USER:GROUP as 65534:65534 to have container run as non-root user

Summary and Scope
Adding USER:GROUP as 65534:65534, to have container run as non-root user.
Issues and Related PRs

Resolves https://connect.us.cray.com/jira/browse/CASMPET-5181

Testing
Tested on:

Virtual Shasta
mug

Were the install/upgrade based validation checks/tests run?(goss tests/install-validation doc) Yes
Was a fresh Install tested? Y/N   If not, Why? - Chart was removed and re-deployed with the fresh image reference
Risks and Mitigations
NA

Closing this PR as different approach has been used to address this issue.


Avoid printing comments on static fields twice.

In TS4.4, when a transformer adds a static field with a synthetic
comment to a decorated class, TS prints the synthetic comment twice:
@Decorator
class MyClass {
  /* comment */ staticField = 'x';  // field and comment added by transformer
}

Becomes:
var MyClass = class MyClass {}
/*comment*/
/*comment*/
MyClass.newField = "x";
__decorate(MyClass, Decorator, ...)

This is because the classFields transformer calls setOriginalNode(n, propertyDeclaration) on both the expression statement , and the
assignment expression contained in it, leading to the synthetic comment
appearing twice.
This change avoids the problem by only calling setOriginalNode on the
outer expression statement. All callers of the inner method take care to
setOriginalNode on the resulting container.
Fixes #45696.

@rbuckton this might be related to recent changes made by you here?
Can you advise on the source map change? I'm not sure if that's OK.

I believe the test suite failure is because I'm on a Mac, and the golden file update process produces \r\n in some locations here. Oddly enough the source map file in question is a mix of \r\n line endings and \n line endings, so it also isn't easy to fix using dos2unix or similar.
If we agree on the path, I can fix the file up manually or see if I can find a windows box to run this on.

Friendly ping.

Thanks for the feedback.
I've added the test you asked for, and made the change to explicitly delete synthetic comments (instead of dropping the setOriginalNode).


Move to jupyterlab-contrib organization?

@axelfahy what do you think about moving this repo to the jupyerlab contrib organization: https://github.com/jupyterlab-contrib website: https://jupyterlab-contrib.github.io/?
I think the positive would be that there would be even more robustness to maintainer dropoff (currently just you and me) as I believe this would allow a community sustained org to give access to more people to push and publish changes  (which currently only you have the power to grant).
Also cc @fcollonval (who I believe started that org) in case I got any of this description wrong.

Yes, I think it makes sense to have in under jupyterlab-contrib if it's okay with them.
Do you know if there is still work in progress to have the extension directly in jupyter-lab?

Hey thanks for pinging me @ianhi
@axelfahy yes anyone can request to transfer a project in the organization. The process is described when opening an issue with the following template: https://github.com/jupyterlab-contrib/jupyterlab-contrib.github.io/issues/new?assignees=&labels=transfer&template=transfer-repository-to-this-organization.md&title=

Transfer to jupyterlab-contrib is done, closing. Thank you @ianhi for proposing this.


AddOrUpdateContactEmailRequest no longer takes FabId?

The example uses FabId as a part of the constructor for AddOrUpdateContactEmailRequest.  However it does not appear that this parameter is a part of the constructor any longer?

Closing - no doc referenced - feature requests / suggestions are better placed in the forums


gardener-node-agent should be able to detect/resolve inconsistencies between last-applied-osc and actual files written to filesystem

How to categorize this issue?

/area robustness
/kind enhancement
What would you like to be added:
We have seen cases in which a VM reboot caused a partially corrupt filesystem and the restore from the journal brought back an outdated /var/lib/gardener-node-agent/config.yaml, which didn't match the content of the last-applied-osc.yaml.
When at a later point in time, the OSC changed, but the content of the config.yaml compared to the last-applied-osc.yaml stayed the same, this caused outdated k8s version labels to be added to the node, resulting in kube-proxy no longer being installed on that node, as the DaemonSet didn't match the node label anymore.
Our request is that gardener-node-agent should be able to detect these situations and resolve them, for example by comparing the content of last-applied-osc.yaml to the content/hash of the files actually written to disk.

/assign @ScheererJ
/triage accepted


Who uses TOAST UI Calendar?

Do you use TOAST UI Calendar in production?
Please let us know so we can add you to the Used By section of the README!

If you want, feel free to edit the README.md file directly and send a PR.
Otherwise, Please provide

The official name of your company/brand/service
The link to your official website
(optional) The logo image

shopby
payco-shopping

NHN Edu - iamTeacher

HIDDEN TRACK - Calendar Subscription Service by PR

CleanCodeStudio for Client Scheduler Service

I am a develoer


storcon: AZ awareness when filling nodes

(Plan on bundling this into https://github.com/neondatabase/neon/pull/9916)
Problem
When we fill a node after draining it, we should move back the shards whose home AZ matches the node being filled, rather than arbitrary shards.
Part of https://github.com/neondatabase/neon/issues/8264
Summary of changes

While planning fills, exclude any shards that have an AZ preference that doesn't match the node being filled.

There are edge cases where this might not be ideal, such as if a system is extremely overloaded to the point that many shards are running in non-preferred AZs.  However, filling nodes is an optimisation, not something that has to happen, and if a significant number of shards can't be scheduled into their preferred AZ then we have bigger problems.

This will land as part of https://github.com/neondatabase/neon/pull/9916


Key Vault Secret references are not loaded

I want to load configuration settings and key vault references into Kubernetes ConfigMaps and Secrets. The configuration settings are successfully loaded into the Kubernetes ConfigMap, but the Secrets are not being populated with data. Although the Secret is created, it remains empty.
used version: tried with 1.3.1 and 2.0.0-preview
I use the following manifest:
apiVersion: azconfig.io/v1
kind: AzureAppConfigurationProvider
metadata:
  name: myapp-backend
  namespace: myapp
spec:
  auth:
    workloadIdentity:
      managedIdentityClientId: XXXXXXXXXXXXXXXXXXXX
  endpoint: https://myapp.azconfig.io
  secret:
    target:
      secretName: myapp-backend-secret
    auth:
      workloadIdentity:
        managedIdentityClientId: XXXXXXXXXXXXXXXXXXXXXXXX
  target:
    configMapName: myapp-backend-appconfig

There are no error logs in the az-appconfig-k8s-provider pod

@tobiasholzner-whiteduck Have you checked this FAQ? Although it refers to the configMap, the scene for secrets is actually the same.

Sounds like a bug according to the doc:
https://learn.microsoft.com/en-us/azure/azure-app-configuration/reference-kubernetes-provider
If the spec.configuration.selectors property isn't set, all key-values with no label are downloaded.

@zhenlan Right, it's a bug scenario, in other providers, labelFilter=string.empty and labelFilter=null are equivalent.

@tobiasholzner-whiteduck  I would not suggest you to use this keyFilter/labelFilter pair to select the keyvault references.
configuration:
    selectors:
      - keyFilter: '*'
        labelFilter: ''

Although it now works for you, it should not behave in this way. Please specify the desired label of your keyvault references you want to select.

The doc is right.

If the spec.configuration.selectors property isn't set, all key-values with no label are downloaded.
This statement in doc means all keyvalues with non-label will be selected, it does not get everything loaded in despite of the labels.

The doc is right; the code is not.


fix(core, web): fix Recaptcha instantiation error

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
closes #12264
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Hi @adifyr, with a flutter pub get you should be upgraded to the latest version. We are aiming at a release today with a couple of fixes for web.


[stable/ambassador] Bump ambassador version 0.74.1

Signed-off-by: Noah Krause<EMAIL_ADDRESS>
What this PR does / why we need it:
Bump ambassador version 0.74.1
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm


Bump nginx and alpine

nginx: 1.21.0 -> 1.21.3
alpine: 3.13 -> 3.14.2

Thanks @rmens. I'll check the build tonight and merge if all is good.

Thanks!


HBASE-23943 Updated Rubocop configuration to current version

Signed-off-by: Nick Dimiduk<EMAIL_ADDRESS>Signed-off-by: Viraj Jasani<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 41s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
yamllint
0m  0s
yamllint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-1 Compile Tests _

_ Patch Compile Tests _

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+1 :green_heart:
asflicense
0m 52s
The patch does not generate ASF License warnings.

3m 28s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1308/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1308

Optional Tests
dupname asflicense yamllint

uname
Linux 972a23fa0b14 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1308/out/precommit/personality/provided.sh

git revision
branch-1 / 200bf3d

Max. process+thread count
41 (vs. ulimit of 10000)

modules
C: . U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1308/1/console

versions
git=1.9.1 maven=3.0.5

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Allow a literal to have no language

Default should be English.

And in the RDF, there should be no language: <> <http://id.loc.gov/ontologies/bibframe/mainTitle> "foo";


Fix Hugo warnings

Fixes WARN 2019/08/11 14:56:21 make non-relative ref/relref page reference(s) in page %q absolute, e.g. {{< ref "/blog/my-post.md" >}}

Travis failure:
Getting links from: http://localhost:1313/docs/console/accounts/saml/
1728-├─BROKEN─ https://www.samltool.com/prettyprint.php (BLC_UNKNOWN)
1729├─BROKEN─ https://www.samltool.com/prettyprint.php (HTTP_undefined)

The certificate for this site expired 2 days ago. I thought about adding an exclusion to the checker for this URL, but if the cert isn't updated, we shouldn't be linking to this site.

Is there a flag or something you can find to get a listing of the specific pages that introduced the warning?

I contributed https://github.com/gohugoio/hugo/pull/6061 back in June and used a private build to see the pages. But looks like version v0.56.0 and later includes my fix.

I don't quite understand why this warning is happening.

The warning happens when the path passed to rel or relref doesn't start with / (i.e. isn't absolute) and the page can't be found relative to the current page, but is found somewhere else on the site. For example, if I had {{< relref "stack.md" >}} on a blog post I was writing, the warning will be emitted because stack.md isn't an absolute path and can't be found relative to my blog post page, but was found at /docs/reference/stack.md. Hugo is warning because the page should be specified as {{< relref "/docs/reference/stack.md" >}} (or I'm missing a stack.md page next to my blog post page).

Is it that relref will only search for the file within the current directory and its children? And you are encouraged to use ref for something that exists outside of that? Similarly, what is the point of using relref if you fully-qualify the reference. e.g. why use relref "/docs/... instead of just ref "/docs/...?

The only difference between ref and relref is the resulting URL generated by Hugo -- it has nothing to do with the path you pass to these (the path you pass is treated the same for either). ref generates a fully qualified URL that includes the baseURL, relref generates a URL without the baseURL. It's similar to Permalink vs. RelPermalink.
For example:

{{< relref "/docs/reference/stack.md" >}} generates /docs/reference/stack/
{{< ref "/docs/reference/stack.md" >}} generates https://www.pulumi.com/docs/reference/stack/

The vast majority of our links do not need to have the baseURL in front of them, hence my preference is to use relref across the board unless there's a situation that necessitates the baseURL being there (e.g. if a sitemap or rss feed or something requires the baseURL to be there).

The cert for that site has been fixed.

Getting links from: http://localhost:1313/docs/console/accounts/saml/
1728-├─BROKEN─ https://www.samltool.com/prettyprint.php (BLC_UNKNOWN)
1729├─BROKEN─ https://www.samltool.com/prettyprint.php (HTTP_undefined)

These links must be coming from the SAML doc I added couple of months ago. It seems that the site is once again valid.

The cert for that site has been fixed.

Ha! I added my comment at the same time.


Port pagebar component from lui into lui2

this: http://devel.less-ui.net/demo/lui_pagebar
into lui2 repository

implemented in https://github.com/Foreveryone-cz/LUI-2/commit/dca17a681f38fd1ecd1792077a7afa08f7230119


Ability to use Peripheral resource in a sequence

Why?

We have a read_pin, but manually entering pin numbers can become tedious.

What's New?

Ability to click use peripheral on a Read Pin block and select a predefined peripheral from the list.

Coverage increased (+0.007%) to 87.428% when pulling c6b3e5d800a90149b8a0720374fe071206820a49 on RickCarlino:read_pin_from_peripheral into fdd41c5820286f60955d2067bcd457e7067cadfa on FarmBot:staging.


Can't mine to v6 Identities or Addresses

If you send me a private message I'll share the identity and screen shots of the wallet in question to you for troubleshooting.
The user:

Created a new address & identity in v6 wallet
Tried mining to the address, it is not recognized and fails hard
Tried mining to the Identity, it is accepted by LeafPool
Payouts from the mining pool do not make it into the wallet, they get rejected as "Waiting for Miner's Wallet" as though the wallet is offline
Tried a rescan
Failed back to a v5 address

Leafpool updated their wallet last night, no change.

Tried again post-hard fork.  Still can't get the beam.
Here's the identity in the wallet:

and here's the mining pool:

Hello!
Please kindly note, that in order to be able to receive coins you need to use an Address, which is not equal to the mentioned Identity. Please make sure to click on three dots (under the page with your Active Addresses book) from the right and choose to Receive, it will redirect you to the page with detailed information on the address you are about to use (the QR code and the address itself under it).
Please let me know if you will have any further questions.
Best regards,
Beam Support Team

Nope, it's not working, leafpool isn't recognizing the address:

I should add- the address has not expired.  It expires on 7/26/2021.

A new address has been created, there are miners running against it now, we'll see if it works, but this is really frustrating.  I feel like I should contribute to the documentation when this is resolved so others don't have these troubles.

Ok, the SBBS Address of the same address (that is not expired) has miners on it, it will take an hour to carve enough Beam from the pool to test payment. We really appreciate your help.

The miners have over a beam unconfirmed, waiting for confirmation.

Yup, that did it.  SBBS Address is what people need to use for the mining address.  I'd be happy to write the documentation, if it helps?

Using the long address does work as well, but it changes anytime you look at it, so SBBS address is definitely the way to go.  For the record.  We're done here, again, thanks for all the help.


EAGLE-467: Job list apis for querying jobs regardless of the status

https://issues.apache.org/jira/browse/EAGLE-467

The serialization/deserialization of org.apache.eagle.jpm.mr.runningentity.JobConfig is a little complicated. Since it is a java HashMap,  should we consider to use java build-in  serialization/deserialization?

If you declare a field as Map, then eagle will automatically adopt java build-in serialization/deserialization.
Although JobConfig extends from Map, but eagle still regards it as a user-define type object.

LGTM


updated device support table

Docs Build status updates of commit 0769c3f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/spatial-mapping-design.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


osu! private message ban list

On twitch or osu!, provide a mod-level command !ban <osu-name> such that the specified user cannot:

get !voted (twitch error message: <osu-name> has been banned from osucelebrity)
interact with osucelebrity on osu! (osu! error message: You have been banned from using osucelebrity. [https://osu.ppy.sh/u/osucelebrity Contact] our developers and moderators for more info. is sent with a cooldown of 2 minutes)

There already is !timeout. What's the point of 2)?

Will osucelebrity ignore all Bancho IRC messages from people that are the target of effective !timeout ?


Support guild hubs

https://github.com/Discord-Datamining/Discord-Datamining/commit/c3fc7a596349a33176d123e9a2940d0ab1eda13c#commitcomment-52049019

Good to keep a track but I guess it's too early to implement it.
It's a closed experiment

Guild Feature: HUB

Guild feature: LINKED_TO_HUB
https://github.com/Discord-Datamining/Discord-Datamining/commit/f06792f1460042831e528dc51502ea7476cf0891#commitcomment-57969940


Allow direct time input from widget

Default behavior is the same, only difference is that disabled <input> element is used to display time. When {timeInput: true} option is provided, input is activated and allows direct input of time string. In case of invalid input value is reverted.
In my application original input is used to passively display date, since more fancy canvas graphics is used to render event occurrences. Still, I need way pinpoint arbitrary point in time (outside of canvas scope), and using direct input might be more natural than sliders (which I primarily use).

Thanks for these changes.  I made several revisions to the css, example so be sure to grab the updated code from dev.


[decompiler] let form maker re-uses function argument names if that register is re-used

appears in time-to-ground (dynamics-h)

The loop/let:
   (while (< (- arg2) f0-0)
    (let ((arg0 (- arg0 (* 0.0033333334 arg1)))) ;; should be a set!
     (+! f0-0 (* 0.0033333334 arg0))
     )

is actually an incorrect decompilation - the value of the outer arg0 should update on each iteration of the loop and this should be a single variable.  The variable insertion pass has done this correctly (there is only a single arg0 variable according to the decompiler).  But the let pass put a let in a place that doesn't make sense, so there end up being two variables named arg0 in the final output.
This is actually a fundamental issue with the insert_lets pass. The scope of the let is decided based on the outer most scope where the variable is referenced, not the scope of where the variable is live.  Getting this information after expressions have been built isn't really possible. So I think this is an equivalent condition:

If register is dead before the initial set!, allow inserting a let.
If register is live before the initial set!, only allow a let if the variable in the register is different.

I will try implementing this and see if it can solve this issue.


Support updater function in setState

Fixes #17
Spec: https://reactjs.org/docs/react-component.html#setstate
Code based on react apollo implementation: https://github.com/apollographql/react-apollo/blob/a42f7da5bd4163223a87ab545dceb88e778046fe/src/getDataFromTree.ts#L105

Awesome thanks! I am on holiday at the moment but will take a look when back. Appreciated. 🤘

@ctrlplusb this also looks good, include in next release?


how to close activated sessions and force user to re-enter his credentials?

One user can log in through multiple systems with various IP addresses, so is there any way to deactivate one of his sessions? (or all other sessions except the current logged in one) if yes, how?
The thing I want to do is exactly like Telegram which you are able to close any of your activated sessions.

any solution?

StackOverflow is a more appropriate place for your question.
If you or your company need commercial support, please see here for more information.
Thanks!

You are saying that you, as the writer of this library, have no answer for this question??? And maybe someone in a stackoverflow knows??

I'm saying that the issue tracker is mainly for bug reports and feature requests. There are other places to get help with with designing your application.

so the question label is useless then


Focus lost on linear view after Y/N on structure arrays first item

Version and Platform (required):

Binary Ninja Version: 3.6.4783-dev, 6fc13294
OS: macos
OS Version: 14.2
CPU Architecture: arm64

Bug Description:
See the title.
Steps To Reproduce:

Define a structure array in linear view
Do Y / N on any member of first item of the structure array
See focus lost

Expected Behavior:
I expect focus to stay where I was before Y / N.
Screenshots:

I am not sure about this one. It could be an intended behavior. Will check with the team


Removing water

Is there an easy way to remove the water surface from a video taken from a moving boat?

Hi @kaosmaja   If you are referring to RGB video recorded by a RealSense camera, there may be video editing techniques that you can use to remove such artifacts from the video.  I suggest googling for remove water splash from video for more information.  An example reference found from such a search is the one in the link below:
https://www.researchgate.net/publication/285317526_Video-based_Water_Drop_Detection_and_Removal_Method_for_a_Moving_Vehicle

Hi @kaosmaja   If you are referring to RGB video recorded by a RealSense camera, there may be video editing techniques that you can use to remove such artifacts from the video.  I suggest googling for remove water splash from video for more information.  An example reference found from such a search is the one in the link below:
https://www.researchgate.net/publication/285317526_Video-based_Water_Drop_Detection_and_Removal_Method_for_a_Moving_Vehicle

Yes, that is exactly it.

Yes, that is exactly it.

You could set a Threshold Filter to exclude from the depth data measurements outside a minimum and maximum depth sensing range (for example, excluding depth readings less than 1 meter from the camera).  This should also take care of excluding unwanted depth readings from water spray directly in front of the camera as the boat moves.
Another approach would be to use a technique called RANSAC to remove the floor (in this case, the ocean) from the depth image.
https://github.com/IntelRealSense/librealsense/issues/3330
Another navigation technique is to read image tags attached to objects instead of sensing depth.  This approach was used by an MIT project called roboats to navigate robotic boats to docks via RealSense 400 Series cameras on the boat's bottom cushion (navigation was by Velodyne lidar camera atop the boat).  This would not be practical for negotiating objects that have not been pre-tagged though.
https://support.intelrealsense.com/hc/en-us/community/posts/360034814013-Update-on-RealSense-equipped-robot-roboat-autonomous-boats

You could set a Threshold Filter to exclude from the depth data measurements outside a minimum and maximum depth sensing range (for example, excluding depth readings less than 1 meter from the camera).  This should also take care of excluding unwanted depth readings from water spray directly in front of the camera as the boat moves.
Another approach would be to use a technique called RANSAC to remove the floor (in this case, the ocean) from the depth image.
https://github.com/IntelRealSense/librealsense/issues/3330
Another navigation technique is to read image tags attached to objects instead of sensing depth.  This approach was used by an MIT project called roboats to navigate robotic boats to docks via RealSense 400 Series cameras on the boat's bottom cushion (navigation was by Velodyne lidar camera atop the boat).  This would not be practical for negotiating objects that have not been pre-tagged though.
https://support.intelrealsense.com/hc/en-us/community/posts/360034814013-Update-on-RealSense-equipped-robot-roboat-autonomous-boats

Hi @kaosmaja   Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


Trouble repeating the setup of the first two slides

Hi,
Ive tried to setup multiple slides to reveal in the way that slide 2 appears from under slide 1. The problems Im seeing is that following your demo format, by the third slide when the users cursor passes over textual content in slide 3, scrolling behaviour seems to break down (the action of scrolling stops) I dont know if that makes sense. I will post the exact code tomorrow in case that helps to make any sense, I dont have it here.
I was wondering from the stand point of creating a stack of slides which all appear from under the previous (much like slide 1 to slide 2) what is the best format of commands to take?
I appreciate your hardwork extending skrollr to encompass the additional functionality offered by skrollr deck.
KF

I can't get skrollr-decks to work in jsFiddle, so I'll just have to post code and hope it works for you.
Essentially, you copy what he did for Slide 2 (#second);
A) Duplicate the .float-deck section above data-anchor-target=#second (I've named it #third):

	3

note: if you don't duplicate the section above `#second' then 2 will hide under 3 and slide up out-of-view as you scroll
B) Duplicate the .float-void section inside the .skrollr-body element, below #second:

Hi Dave, very many thanks for your assistance, will update the pages Ive been working on and see how it sits within the rest of the framework Im working within. We're building a site in a cms called Kentico which clashes / causes its own issues. Very much appreciate your help with the parts of the puzzle linked to skrollr-decks. Thank you :)

No problem at all. Sorry the author didn't get back to you. Overall it would be nice if the docs mentioned this; I wanted to do the same thing so I had to just play around with it (I've never used skrollr itself before, but I'd imagine this is documented there and that's why the author didn't bother). Once it's working, it's a very cool effect.

Sorry, guys! I'm here. Forget about this issue. Thanks @daverogers ! You can make PR to update the documentation.


ticdc: fix get source id error

What problem does this PR solve?

Issue Number: close #8110 #8115
What is changed and how it works?
Upgrade tiflow's package dependencies on tidb, pd, and tikv to properly call the pdClient interface.
TiCDC behavior after PR merge:

TiCDC version > 6.5.0:
Compatible with both low and high version PDs.
Can get the correct sourceID.
TiCDC version = 6.5.0:
Cannot get the correct sourceID from higher version PDs (v>6.5.0).
No error, but no effect on data correctness.
TiCDC version < 6.5.0:
The interface to get the sourceID is not called.
No compatibility issues.

Check List
Tests

Manual test (add detailed scripts or steps below)
Unit test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Fix the problem causing changefeed creation to fail due to inability to retrieve the sourceID from PD.

/run-integration-tests

I see that The job was not started because recent account payments have failed or your spending limit needs to be increased.....

/run-verify

/hold

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-kafka-integration-test

/run-dm-integration-test

/merge

/run-dm-integration-test


Replace DB seed with all unsigned ids

This is an update of the db seed with all unsigned ids. Please merge directly @pylipp if it works for u.

 check if primary keys are unsigned integers or signed integers #small

Can you rebase this onto current master such that comparison is easier? Also then I can rebase the WIP branch peewee-unsigned-integer-pks onto this one.
It appears you also changed a couple of FK properties (on-delete handlers e.g.). I'll have to update the model definitions accordingly. I wouldn't merge before.

List of changed FK properties by tables:

borrow_items

CONSTRAINT `borrow_items_ibfk_5` FOREIGN KEY (`created_by`) REFERENCES `cms_users` (`id`) ON DELETE SET NULL ON UPDATE CASCADE
CONSTRAINT `borrow_items_ibfk_6` FOREIGN KEY (`modified_by`) REFERENCES `cms_users` (`id`) ON DELETE SET NULL ON UPDATE CASCADE

borrow_transactions: several changes
Why is the id of the box_state table now unsigned NOT NULL (no AUTO_INCREMENT) ? Same for cms_functions table
camps: new fields beneficiaryisregistered, and more. Should they be added to the peewee model?. Handlers for created_by and modified_by fields

Why is the id of the box_state table now unsigned NOT NULL (no AUTO_INCREMENT) ? Same for cms_functions, cms_users, genders, locations, people, products, sizes (@jamescrowley @aerinsol is this related to the failures you've seen on master recently?)

This is my bad. I did something wrong here when writing the migrations.

camps: new fields beneficiaryisregistered, and more. Should they be added to the peewee model?

only beneficiaryisregistered and beneficiaryisvolunteer

borrow_, cms_functions, cms_functions_camps, cms_settings, cms_usergroups, history, itemsout, laundry_, library_*, people, transactions, translate: changes in the FK fields (but tables not available as peewee models)

don't worry about it. not important for warehousing.

locations: for box_state_id field, unsigned NOT NULL instead of DEFAULT '1', is that on purpose? same in stock table

my bad. I have to change this.

new tables people_tags and tags: add as peewee model?

don't worry about it. not important for warehousing.

@HaGuesto why would you ask @pylipp not to import this into the Peewee model? Yes, it's not important for warehousing, but it is overall important for the back-end.

new tables people_tags and tags: add as peewee model?

don't worry about it. not important for warehousing.

@aerinsol We have not implemented peewee models for anything not related to warehousing except user access tables.
I would wait to integrate peewee models for tables until they will be used in the new app. Otherwise we just replicate the "horrible db design" of before and have to edit the models as well when we change the db in the old app.

I don't actually think this is correct. Due to the pwiz export by @mc18g13 , I believe all of the tables - including Beneficiary, Transactions, etc. have been exported as peewee models. I don't think tests have been created around them, but I think the models are there. @mc18g13 or @pylipp please comment if you know off the top of your head.

These are the models that are implemented

OK good to know @HaGuesto. That means this issue log: https://trello.com/c/Ew2XkaDt is incorrect and will be put back again on the backlog. @pylipp for future reference, this is essentially this is just the ability to assign tags to people - I'll tag you in Slack on the video that demos this feature. In the future, we will likely want to align this with the tag boxes feature https://trello.com/c/XjNwO3sL

 Import Beneficiary/Shop and Log Tables into Peewee models

@pylipp can you take a look at this?

I'll have a look on Saturday. From what I remember, a correct database dump is needed to base the peewee model definitions upon. I can achieve that by editing the dump I suppose :)

 Create local Peewee support of MySQL default for signed integers

That's a PR that would be nice to have merged soon =)

@pylipp done


Re-add indigo cask

Reverts caskroom/homebrew-cask#9019
This download is now available without registration.

This will need to be updated to the new cask template, see here.

The homepage is also new (it redirects).
Instead of adding the removed cask, please try making it from scratch, to catch all those little differences.


Archive the repo?

Should we archive this repo? It seems like we're not actively developing this and doing these sorts of things in other places.

Yeah, makes sense to me. Will do.


Wash/Self trading

Currently trades with self (wash trading) are allowed, this should really be disabled.
Is this the relevant file to change?
https://github.com/rubykube/peatio/blob/master/app/trading/matching/executor.rb#L40

@TheHolyRoger yes

Why was this closed @ysv ?


standardize full date time hours min seconds format

Need standard date time format and where to use it.
Note for @Chwy5
portfolio: date in open orders is different as well.

Closing old ticket, we have newer better defined issues.


get rid of snapshot references

this is not possible right now with cloudmanager, we need those releases in our processes

We need to decide how we want to do this. Options:

Request Cloud Manager team to support Github packages
Get our packages onto repo.adobe.com
Embed the packages in the places we need it (docs)
Something else

as discussed, @auniverseaway 4. could be bringing our released packages to central repository (may be at release time or something)

https://issues.sonatype.org/browse/OSSRH-59561 can @auniverseaway and @jenssingler you create an account in sonatype JIRA so that you can have deploy rights? ? they should be named cmillar and jsingler
if you manage to create those accounts only with different names, i'll update the JIRA ticket

re-opening for demo

not sure when we'll start versionning & releasing docs, but for the rest, we are good with docs deps:
[INFO] Building Adobe DX - Documentation - App package 0.0.1-SNAPSHOT     [3/3]
[INFO] --------------------------[ content-package ]---------------------------
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/admin/0.1.0/admin-0.1.0.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin/0.1.0/admin-0.1.0.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin/0.1.0/admin-0.1.0.pom (7.6 kB at 38 kB/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/admin.reactor/0.1.0/admin.reactor-0.1.0.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin.reactor/0.1.0/admin.reactor-0.1.0.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin.reactor/0.1.0/admin.reactor-0.1.0.pom (1.9 kB at 6.7 kB/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/content/0.0.10/content-0.0.10.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content/0.0.10/content-0.0.10.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content/0.0.10/content-0.0.10.pom (5.7 kB at 16 kB/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/content.reactor/0.0.10/content.reactor-0.0.10.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.reactor/0.0.10/content.reactor-0.0.10.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.reactor/0.0.10/content.reactor-0.0.10.pom (0 B at 0 B/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.pom (0 B at 0 B/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.pom (0 B at 0 B/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/cq/core.wcm.components.parent/2.11.0/core.wcm.components.parent-2.11.0.pom
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.parent/2.11.0/core.wcm.components.parent-2.11.0.pom
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.parent/2.11.0/core.wcm.components.parent-2.11.0.pom (41 kB at 168 kB/s)
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/admin/0.1.0/admin-0.1.0.zip
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/testing/0.0.10/testing-0.0.10.jar
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/admin.core/0.1.0/admin.core-0.1.0.jar
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/structure/0.0.6/structure-0.0.6.zip
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/structure.core/0.0.6/structure.core-0.0.6.jar
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/content/0.0.10/content-0.0.10.zip
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.jar
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/dx/core/0.0.12/core-0.0.12.jar
Downloading from adobe-public-releases: https://repo.adobe.com/nexus/content/groups/public/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.zip
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin/0.1.0/admin-0.1.0.zip
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin.core/0.1.0/admin.core-0.1.0.jar
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/testing/0.0.10/testing-0.0.10.jar
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/structure/0.0.6/structure-0.0.6.zip
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/structure.core/0.0.6/structure.core-0.0.6.jar
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/testing/0.0.10/testing-0.0.10.jar (14 kB at 45 kB/s)
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content/0.0.10/content-0.0.10.zip
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin.core/0.1.0/admin.core-0.1.0.jar (35 kB at 45 kB/s)
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.jar
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/structure/0.0.6/structure-0.0.6.zip (97 kB at 119 kB/s)
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/dx/core/0.0.12/core-0.0.12.jar
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/structure.core/0.0.6/structure.core-0.0.6.jar (13 kB at 15 kB/s)
Downloading from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.zip
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/admin/0.1.0/admin-0.1.0.zip (331 kB at 345 kB/s)
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content.core/0.0.10/content.core-0.0.10.jar (36 kB at 15 kB/s)
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/content/0.0.10/content-0.0.10.zip (61 kB at 25 kB/s)
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/cq/core.wcm.components.all/2.11.0/core.wcm.components.all-2.11.0.zip (957 kB at 379 kB/s)
Downloaded from central: https://repo.maven.apache.org/maven2/com/adobe/dx/core/0.0.12/core-0.0.12.jar (26 kB at 10 kB/s)
[INFO]```


Completed Issue#411

Short description of what this resolves:
Improve video feature according to Issue#411
Changes proposed in this pull request:
Now if the request includes the word trending and a correct region code it will get the most popular videos from that country
Fixes: #
None

Thank you for taking the time to contribute! This is sincerely appreciated. 😇
(Powered by GitMate.io)


Can't find octopus_intelligent integration

I've downloaded via hacs and the logs state
(SyncWorker_0) [homeassistant.loader] We found a custom integration octopus_intelligent which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
However I do not see it in the add integration option

This can be closed - integration was there on desktop browser but not phone app


Error when trying to add midi files

@Believe82 Unfortunately, this appears to be coming from msupcm++, so there's not much I can really do here. It actually appears that libsox, the library used by msupcm++, doesn't support midis either. I'm guessing the reasoning is that midis can sound very different based on the sound font on the OS, so they can sound different for each person.
While not ideal, I think the best option here would be to find a different audio file that isn't in midi format, find a converter to change the midi file into a wav file, or record the midi yourself using software into a wav file.

Alright I will look for a convertor, thanks for the quick reply!


Add pointer to azure-credentials in the credentials section

+1 Thank goodness for the new cred addition


Long form blog content

make blog posts
To index us better we need lots of text on a page
so we need to generate ai articles on "wow goldmaking using the auctionhouse" and "ffxiv gil making using the maketboard" and make sure it mentions "Saddlebag Exchange"
we can use this to help, the articles will be in markdown
https://app.simplified.com/ai

convert markdown to html
We can take all of the wiki guides and convert them using this: https://markdowntohtml.com/
take html and make blog files
then we make blog pages like this, just take out the highlighted section and replace it with html:
https://github.com/ff14-advanced-market-search/saddlebag-with-pockets/blob/master/app/routes/blog.wow.tldr.tsx#L5-L108
raw page:
const WhateverRandomPageName = () => {
  return (
    <div className={`m-12`}>
      <main className="flex-1">
        ... random html goes here...
      </main>
    </div>
  )
}

export default WhateverRandomPageName

then just copy them over and make it blog.$GAME.$NAME.tsx and add them into https://github.com/ff14-advanced-market-search/saddlebag-with-pockets/blob/master/app/routes/blog._index.tsx#L5-L33

hey @cohenaj194 , i am working on it.


:memo: Update dependency list

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Update to latest game version

Thanks! Appreciate the fix

No problem, didn't have a time to take a look at chams. Might give that a go tomorrow


Pack always builds a project with reference assemblies

Add a reference assembly to a project, like described in the blog (e.g. https://github.com/AzureAD/microsoft-authentication-library-for-dotnet)

Build the sln, which builds both project and ref project

Pack via "msbuild /t:pack /p:nobuild=true"

Actual:
The pack does not obey the "nobuild" setting. It needs to rebuild because the ref assemblies are not actually produced in step 2
Expected:
Pack should not rebuild
Impact:
Because pack rebuilds, some DLLs are overwritten. This makes DLL signing difficult. Ideally all DLLs should be on disk and /t:pack just packs
Workaround:
I can workaround by packing twice. So my release pipeline looks like this:

build
pack
sign lib assemblies
sign ref assemblies
pack /nobuild

I'll look into this when I can, but in general, I would not suggest making sign as part of the build step. I'd have build/pack do just that, then send the nuget to the signing mechanism once built (you can always extract/repack the nuget with zip before the nuget itself is signed).

Having similar problem here.
Can't run a Target\Task from a .targets inherited from a PackageReference before GenerateNuspec  like this
<Target Name="ForceCall" BeforeTargets="GenerateNuspec">
    <CallTarget Targets="TheTask" />
</Target>

I guess the .targets isn't available at that point. (It's working fine if I hook it during build, like AfterTargets="CoreCompile" + working if the .targets is part of the project)

This is why I used dotnet pack instead (it respects the nobuild). I did test this last week, and the problem seems solved?
The scripts I use are here:
https://github.com/GeertvanHorrik/RepositoryTemplate/blob/master/replace/deployment/cake/components-tasks.cake#L218
Will investigate to make sure that references are not being rebuild.

I am using the preview versions of VS, but msbuild with nobuild & pack no longer seems to build reference assemblies (and thus overriding the signed assemblies).


Omnisharp, autocomplete, etc. not working for one project

Environment data
.NET Command Line Tools (2.1.4)

Product Information:
 Version:            2.1.4
 Commit SHA-1 hash:  5e8add2190

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.4/

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

VS Code version: 1.21.0
C# Extension version: 1.14.0
Steps to reproduce
Open a specific project of mine in the Unity editor. Double-click a C# file to open in VS Code.

Actual behavior
Get no project-specific autocompletion, and Go To Definition yields "No definition found for [X]" for anything. No error messages are apparent.
The strange thing is that it's only this Unity project; I have no problems with others. I've tried various restarts to clear things up, including deleting my local Github repo and cloning it anew.
I think it has something to do with a bad solution file; the flame in the lower right says "[Project].sln" for the projects that work, but simply "Switch projects" for the bad one. Clicking that yields only the current solution and no alternatives. I've tried quitting VS Code, deleting the .sln file, and opening it up again; no change.
Thanks for any assistance you can provide.

Hi! Could you provide your OmniSharp Log after opening the project in VS Code? You can retrieve this be selecting View->Output from the VS Code menu and then choosing "OmniSharp Log" from the top-right corner of the Output pane.

`Starting OmniSharp server at 2018-3-12 11:35:04
Target: /Users/myname/Dev/UnityGames/MyGame/MyGame.sln
OmniSharp server started wth Mono
Path: /Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/OmniSharp.exe
PID: 6985
[info]: OmniSharp.Stdio.Host
Starting OmniSharp on MacOS 10.12.6 (x64)
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
Located 2 MSBuild instance(s)
1: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.8.0/lib/mono/msbuild/15.0/bin"
2: StandAlone 15.0 - "/Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/msbuild/15.0/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
Registered MSBuild instance: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.8.0/lib/mono/msbuild/15.0/bin"
CscToolPath = /Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/msbuild/15.0/Bin/Roslyn
CscToolExe = csc.exe
[info]: OmniSharp.Cake.CakeProjectSystem
Detecting Cake files in '/Users/myname/Dev/UnityGames/MyGame'.
[info]: OmniSharp.Cake.CakeProjectSystem
Could not find any Cake files
[info]: OmniSharp.DotNet.DotNetProjectSystem
Initializing in /Users/myname/Dev/UnityGames/MyGame
[info]: OmniSharp.DotNet.DotNetProjectSystem
Auto package restore: False
[info]: OmniSharp.DotNet.DotNetProjectSystem
Update workspace context
[info]: OmniSharp.DotNet.DotNetProjectSystem
Resolving projects references
[info]: OmniSharp.MSBuild.ProjectSystem
Detecting projects in '/Users/myname/Dev/UnityGames/MyGame/MyGame.sln'.
[info]: OmniSharp.MSBuild.ProjectManager
Queue project update for '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
Queue project update for '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
Loading project: /Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj
[info]: OmniSharp.Script.ScriptProjectSystem
Detecting CSX files in '/Users/myname/Dev/UnityGames/MyGame'.
[info]: OmniSharp.Script.ScriptProjectSystem
Could not find any CSX files
[info]: OmniSharp.Stdio.Host
Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider
[info]: OmniSharp.Stdio.Host
Configuration finished.
[info]: OmniSharp.Stdio.Host
Omnisharp server running using Stdio at location '/Users/myname/Dev/UnityGames/MyGame' on host 6970.
[info]: OmniSharp.MSBuild.ProjectManager
Adding project '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
Loading project: /Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj
[info]: OmniSharp.MSBuild.ProjectManager
Adding project '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
Update project: Assembly-CSharp
[info]: OmniSharp.MSBuild.ProjectManager
Update project: Assembly-CSharp-Editor
[warn]: OmniSharp.MSBuild.ProjectManager
Unable to resolve assembly '/Users/myname/Dev/UnityGames/MyGame/Temp/bin/Debug/Assembly-CSharp.dll'`

Starting OmniSharp server at 2018-3-12 11:35:04
    Target: /Users/myname/Dev/UnityGames/MyGame/MyGame.sln

OmniSharp server started wth Mono
    Path: /Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/OmniSharp.exe
    PID: 6985

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on MacOS 10.12.6 (x64)
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 2 MSBuild instance(s)
            1: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.8.0/lib/mono/msbuild/15.0/bin"
            2: StandAlone 15.0 - "/Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/msbuild/15.0/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.8.0/lib/mono/msbuild/15.0/bin"
            CscToolPath = /Users/myname/.vscode/extensions/ms-vscode.csharp-1.14.0/.omnisharp/omnisharp/msbuild/15.0/Bin/Roslyn
            CscToolExe = csc.exe
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in '/Users/myname/Dev/UnityGames/MyGame'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Initializing in /Users/myname/Dev/UnityGames/MyGame
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Auto package restore: False
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Update workspace context
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Resolving projects references
[info]: OmniSharp.MSBuild.ProjectSystem
        Detecting projects in '/Users/myname/Dev/UnityGames/MyGame/MyGame.sln'.
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: /Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in '/Users/myname/Dev/UnityGames/MyGame'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.Stdio.Host
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider
[info]: OmniSharp.Stdio.Host
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location '/Users/myname/Dev/UnityGames/MyGame' on host 6970.
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: /Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project '/Users/myname/Dev/UnityGames/MyGame/Assembly-CSharp-Editor.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: Assembly-CSharp
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: Assembly-CSharp-Editor
[warn]: OmniSharp.MSBuild.ProjectManager
        Unable to resolve assembly '/Users/myname/Dev/UnityGames/MyGame/Temp/bin/Debug/Assembly-CSharp.dll'

Update: This problem... fixed itself? I don't know what happened, but the flame now says "MyGame.sln" and things are working as expected. Advice still appreciated in the event this crops up again, but for now I think I'm good.

Did you happen to build your project before it fixed itself?

I think I only used the Play button in the Unity editor. Not sure if that counts.

I would expect that to count, yes.

Righto, then. I'm happy to close this for now.

No i have this problem:(
In OmniSharp Log are ony [info].
Any CodeLeans not work.


npm audit fix

All the vulnerabilities this fixed were also addressed by the recent @dependabot reviews. See #194, #196 and #197. This only updates the lockfile format now, which we can also do on another occasion. :shrug:


Add missing declarations to slice.pxd

PySlice_Unpack() and PySlice_AdjustIndices() were missing from
slice.pxd.
The comments are taken from the C API documentation
(https://docs.python.org/3.8/c-api/slice.html).

Thanks


CASMINST-5817: add error checking for template input

Description

Checklist Before Merging

[x] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[x] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport main


Expose a type for PathSegment

Here's an addition that exposes a standalone type for path segments.  I think this improves the ability to work with paths and partial paths.  There may be some missing interface functionality, but I'll let you decide how to handle the tradeoffs there with respect to backwards compatibility and API design preferences.

I'm going to marge this in but there are some issues with it i will have to fix before making a new cut for hackage:

The segString record has been exported, i'm going to remove this and replace it with a function PathSegment -> String as usage like this defeats the purpose of the mkPathSegment function and ruin the safety of the type:

*Data.FilePath Data.Functor> (\c -> c {segString = "Lots'O'SlashesOhNoBroken!!//////////"}) <$> mkPathSegment "noSlashesOrControlCharsIsSafe"
Just (PathSegment {segString = "Lots'O'SlashesOhNoBroken!!//////////"})

Going to add a function to create FilePaths of the appropriate types from PathSegment, allowing you to get around a layer of Maybe, which to me seems like the point of the change.

I'll probably add a function String -> PathSegment function that filters the undesirable chars out of the input String and gets you a safe PathSegment allowing you to create safe constants where quasiquoters dont help. (The function would be the identity on valid strings)

for ex:
import Date.Time.Format ( FormatTime, formatTime )
import System.Locale ( defaultTimeLocale )

-- Can't do this at the moment, but should be able to with this new PathSegment Type:
-- The output shouldn't have to be of type Maybe (FilePath ...), we know formatTime is going to produce
-- a String without invalid chars given the literal format string its been given.  Cant see how to do this with
-- QuasiQuoter though...
--
yearPath :: (FormatTime t) => t -> FilePath a Directory
yearPath = mkDirPath . formatTime defaultTimeLocale "year=%Y"

basenameSeg and segQ were also missing from the export list, i'll fix that too

@mightybyte
For 2. : It doesn't look like a PathSegment -> FilePath Relative a type function is currently possible as the PathSegment type does not certify that the null check on the string has taken place....
Please see #8. I have made mkPathSegment also run the null check, hence PathSegment certifies that all the checks have taken place, meaning we can safely have functions of type PathSegment -> FilePath Relative a.
Please let me know if the exclusion of empty strings from PathSegment causes you any problems, I'll probably merge that change in sometime tomorrow if i dont hear any objections
(the empty string problem makes point 3. I made in the last comment irrelevant)

Yeah, all this makes sense to me.  I definitely agree that excluding empty strings from PathSegment seems like a much better thing to do now.  I don't think I had a good reason for going with Monoid except that Monoid is much more widely used.  My main use case was being able to prepend and append things to PathSegments for disambiguation, so you wouldn't want to use empty strings there anyway.


[Windows unit-tests] Investigate why the rasa executable isn't in the search path on windows

For some reason, the unit tests for Windows fail due to the rasa executable isn't in the search path.
https://github.com/RasaHQ/rasa/pull/7480/checks?check_run_id=1522792530
The https://github.com/RasaHQ/rasa/pull/7500 fixed it but still is something that should be investigated to have a clear picture of what has changed.

Poetry, since >= 1.1.0b switch to virtualenv  that's why there was an issue with the discovery of the rasa executable.


Intended way to expand the controller?

So not sure if this is more "Godot" related or addonrelated. Might need to RTFM :P.
How would one go about adding their own inputs to this (For example, left click to shoot)
So in a previous issue it was found you can import and use the Player scene, which handles various input mappings.
I can see that it is possible to extend a particular script within Godot.
Should you be extending the Player scene/script or instead, writing your own Player and then extending the FPSController?
I would imagine it's the latter as if you attempt to write
func _physics_process(delta):
    #Your code here
    pass

Then this will just overwrite everything in the existing player.gd provided by this addon.

The idea of player.gd is to be an example to be followed, that is, it must be edited/created by the user


feat: audit logs support added

Feature: GCP Audit Logs support has been added to the module.

Docs to be generated

Thanks for contributing! Could you add the changes to the CHANGELOG.md also?
Many thanks

Added !


Port invalid_device_id pysys test to rust

Port tests/Pysys/misc_features/invalid_device_id pysys test to rust.
The test creates the certificates using tedge and the device-id that contains the invalid/unsupported characters.
Assert on the error messages.
Describe the solution you'd like
Once the test is ported successfully, it should be able to run using cargo test
Describe alternatives you've considered

Additional context

I think we can just remove the test, for me, it is duplicating integration test.

We already have unit tests to check characters used in device ID. https://github.com/thin-edge/thin-edge.io/blob/main/crates/common/certificate/src/device_id.rs
While we work on moving from pysys to the new system test framework, we will decide if we should keep this test as system test or not. Therefore, I would close the issue.


Add custom annotations into GatewayClass and NginxGateway manifests

Proposed changes
Add custom annotations for GatewayClass and NginxGateway objects.
It can be useful in different cases
Problem: For deploying of our service we use ArgoCD. For this we have a Helm Chart that contains multiple resources that are using CRDs, that are not managed by this helm chart. When trying to sync this application, it fails if at least one CRD is not installed in the cluster. This is expected.
Solution: To work around this problem, we want to use SkipDryRunOnMissingResource option as annotation for  GatewayClass and NginxGateway objects from the nginx-gateway-fabric chart
Testing: Helm template works fine
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING doc
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that all unit tests pass after adding my changes
[ ] I have updated necessary documentation
[ ] I have rebased my branch onto main
[ ] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Actually, do we really need GatewayClass and NginxGateway objects during the installation of operator? Can we mark them as optional and create only when we need it?

@sgavrylenko Thanks for the submission. Right now we do require the NginxGateway resource because we bootstrap NGF based on its contents (right now just one field, but could expand in the future).
GatewayClass though may not be necessary. Worth a discussion amongst the team at least.


Application Name support in connection string

These changes add support for a new optional parameter, ApplicationName. If specified, this parameter's value will be added to the ConnectionString's Application Name parameter. If not supplied, an attempt to determine the parent script's filename is made.
I've also changed the creation of the ConnectionString from basic string formatting to using the SqlConnectionStringBuilder class to reduce redundant code and make adding parameters into the ConnectionString on a conditional basis cleaner.

Hi @alevyinroc!
Thanks for this, awesome idea (both adding the application name, and building the connection string without silly string manipulation).
I think it might make sense to submit this PR to the function in dbatools - still deciding what to do with this repo (e.g. push to it if invoke-sqlcmd2.ps1 changes in dbatools, leave it alone, etc.)
cc @ctrlbold
Cheers!

I didn't even realize that it was in dbatools! Is there an easy way to migrate the PR there, or should I fork that repo, redo my changes and then submit?

Forked & recommitted my changes, will have to review the contribution guidelines before I go for the PR.

 Though that's a moot point now. Hmmmm. But I just keep an eye on this and when changes are pushed, I copy it, nbd. @RamblingCookieMonster - mind if I accept PRs for this repo?

Works for me, awesome!

@alevyinroc - make sure you sync this one before adding back your other changes!


ERROR:  Array is not a function (evaluating 'e.docChanges()')

Here is my code:
 const db = firebase.firestore();
    const userCollection = db.collection('interviews');
    const geoobj = new GeoFirestore(userCollection)
    let query  = geoobj.query({
      center: coordinates,
      radius: 100,
    //  query: (ref) => ref.where('d.count', '==', 1)

    });
    query.on("ready", () => this.setState({ loading: false }));
    query.on("key_entered", (key, coords, distance) => {
      console.log("asdada",key);
    });

Probably the same reason as in #19, try to update firebase.

i am using react-native-firebase package so i have to update it...

This package isn't designed for react-native-firebase, it's designed for the pure JavaScript firebase library. Unfortunately it will not work. (I am open to PRs for support though)

@MichaelSolati thanks when i use firebase its work.

@MichaelSolati  Will the package work for firebase-admin sdk???

@kendozz it should with versions 5.x.x.

Hi, @MichaelSolati Just to let you know if you don't already. I am using react-native-firebase and geofirestore(2.2.1) seems to be working fine with it, I can read and write.
Thank you very much for this library it's very helpful and easy to use! keep it up, please ! :)

Hi Michael,
Great job on the library. I am using  firestore "firebase": "^4.13.1"  along with geofirestore "^2.2.2".
I am trying to accomplish the following query using array-contains. I have the following  array example below.
array=['cat','dog'];
now i want to loop via firestore and do something like a
.where('d.name','array-contains',array). See my query below for reference.
const collectionRef = firebase.firestore().collection('Consents')
const geoFirestore = new GeoFirestore(collectionRef);
          var arr =['cat','dog']
          const geoQuery = geoFirestore.query({
          center: new firebase.firestore.GeoPoint(lat, long),
          radius: val,
          query: (ref) => ref.where('d.name', 'array-contains',arr)
   });

how can i achieve this using geofirestore, firebase and firestore.

should i update my firebase to react native firebase to be able to use the array contains
method along with geofirestore?

@graig12 In Firebase docs, they usually do array-contains using a string parameter. Your code is using an array, I'm not sure if that's the problem.
Try:
query: (ref) => ref.where('d.name', 'array-contains',arr.join(' '))

that is correct @joscmw95 i want to achieve an OR. Since this is not possible can how would
u suggest i accomplish this ?

As written at the bottom of the docs:

Logical OR queries. In this case, you should create a separate query for each OR condition and merge the query results in your app.

You'll have to query for every element in the array and merge the results:
var arr =['cat','dog']
var results = []
arr.forEach(el => {
    const geoQuery = geoFirestore.query({
      center: new firebase.firestore.GeoPoint(lat, long),
      radius: val,
      query: (ref) => ref.where('d.name', '==', el)
    })
    geoQuery.on('key_entered', function(key, document, distance) {
      results.append(document)
    });
})

This is assuming that you want the results in random order and it is probably not performant because geo query will need to run n times. I would suggest you to just get data from a single geo query and filter the data client side.

HI @joscmw95  thanks for the input i will try and share the results.

Hi @joscmw95 are you able to share any insight here on this post of stack over flow."https://stackoverflow.com/questions/53094022/firestore-multiple-where-query?noredirect=1#comment93100280_53094022" I am attempting
to do a filter on a geofirestore/firestore database that conatians a collection of books. I am trying to filter on booktype and book age however i am getting the error "uncaught error in onsnapshot firebaseError: cursor position is outside the range of the original query " the code that i am using is below.
const collectionRef = firebase.firestore().collection('Books')
       collectionRef.where('d.details.BookType',"==",BookType)
       collectionRef = collectionRef.where('d.details.bookage',"<=",age)
       collectionRef = collectionRef.orderBy('d.details.bookage')

         const geoFirestore = new GeoFirestore(collectionRef)

          const geoQuery = geoFirestore.query({
          center: new firebase.firestore.GeoPoint(lat, long),
          radius: val,

          });

        geoQuery.on("key_entered",function(key, coords, distance) {

storeCoordinate(key,coords.coordinates._lat,coords.coordinates._long,newdata)
});

Internally geoFirestore gets its results by using startAt and endAt. The way you declare your collectionRef clashes with how it works hence the cursor error.
Detailed explanation in stackoverflow


Symbols disallowed as object keys

TypeScript Version:  4.0.0-dev.20200725
Search Terms:
Symbols as object keys
Code
type symbolKey = {
    [key: symbol]: any
};

Expected behavior:
The code compiles without error.
Actual behavior:
TSC throws an error disallowing symbols to be used as object keys.
Playground Link: https://www.typescriptlang.org/play?ts=4.0.0-dev.20200725#code/C4TwDgpgBAziC2AjA9gGwNIRFAvFA3gFBQlQDaA1lgFywIqoC6tAhgHYiEC+A3EA
Related Issues: Couldn't find anything related
Symbols were introduces with the primary purpose of being used as object keys, and are still primarily, if not only, used with that purpose. To not be usable as keys makes no sense whatsoever. They can be used on any object type too, array, generic object, map, Element, etc.

#26797

Duplicate of #1863

Yeah okay, I'm a bit late.


MessageGUID: class replaced by UUID

Motivation:
MessageGUID don't add any advantages to the project.
Instead the are some disadvantages:
They add more object's (higher gc load)
To store the UUID as Strings needs more memory

This Update will break old MapDB files!

Hi @AndreasMager MessageGUID instead carry a big advantage to the project, it's a value object in the domain model, it say what it needs; I mean a UUID is generic , could be a uuid for subscription, for message or for anythiing, MessageGUID say that's a uuid for message. So The class has a big reason to stay.
We can drop the String of UUID and keep just the byte[] representing that UUID, this eliminates the 34 bytes for string chars + the reference to it, keeping just a byte[] of 16 bytes; but I wan to keep the MessageGUID

I still see no benefit only trouble.  Do you wan't create a class for each Datatype you use like class MessageCounter { int counter} or class Topic { String topic}?
With the Streams Api's from java 1.8 you will hate this stuff.

Sorry but don't agree with you, having some Map<UUID, Something> for me it's like having Map<String, something> so I prefer to use some more meaningful concept as key. In this case I think that using a domain object carry great value to readability.


multiple_upload field doesn't allow delete in update view.

Bug report
What I did
CRUD::addField(
    [
        'label' => 'Extra Photos',
        'name' => 'extra_photos',
        'type' => 'upload_multiple',
        'upload' => true,
        'wrapperAttributes' => [
            'class' => 'form-group col-md-6',
        ],
    ]
);

What I expected to happen
The CRUD field appears and works across CRUD.
What happened
In update view, when you click the cross X beside the file name to mark it as delete, it doesn't work properly -- it doesn't trigger the relevant JS functions.
What I've already tried to fix it
The problem happens to be in the view file:
@php
    if (!isset($field['wrapperAttributes']) || !isset($field['wrapperAttributes']['class']))
    {
        $field['wrapperAttributes']['data-init-function'] = 'bpFieldInitUploadMultipleElement';
        $field['wrapperAttributes']['data-field-name'] = $field['name'];
    }
@endphp

Instead of merging, it skips loading of upload field init function when you have wrapperAttributes. So I manually add this two line into my code, and that avoid the bug -- yet I still think this is a bug.
        // in my EntityCrudController
        'wrapperAttributes' => [
            'class' => 'form-group col-md-6',
            'data-init-function' => 'bpFieldInitUploadMultipleElement',
            'data-field-name' => 'extra_photos',
        ],

Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
### LARAVEL VERSION:
laravel/framework                     v6.4.0   The Laravel Framework.

### BACKPACK VERSION:
backpack/crud                         4.0.12   Quickly build an admin int...
backpack/generators                   2.0.4    Generate files for laravel...
backpack/logmanager                   3.0.0    An interface to preview, d...

Looks like this issue have already reported in #2188 . My report will stay here as it includes sample solution code for those who have no clue fixing this, but i'll close this for good.


output video is muted

is this supposed to work like that? :shipit:

Hii @webgtx.
Thanks for opening an issue!

is this supposed to work like that? :shipit:

No, It isnt supposed to work like that lmao.
i just tried downloading a video(as it's been a while i downloaded a reddit video) and the audio wasnt there. that's actually surprising because the audio's were always there!! :/
i'll look into this issue soon. hopefully there'll be something i can do to resolve this.

If you use yt-dlp and the audio is in m4a format, you can look at their code for audio and video extraction details. To merge them, you could use ffmpeg.
ffmpeg -i video.mp4 -i audio.m4a -c:v copy -c:a aac -strict experimental output.mp4

Hiii @webgtx, I'm sorry for stalling this long to resolve this. :(

Usually, Reddit serves the video and audio tracks separately and there used to be a way i could retrieve the audio by modifying the video url

for example, i could get the audio of this video (https://v.redd.it/2nboabex3d3c1/DASH_720.mp4) by simply replacing DASH_720 with DASH_audio i.e. https://v.redd.it/2nboabex3d3c1/DASH_audio.mp4  and then merge both the audio and video with ffmpeg.

but for some reason idk, that isnt possible now.
I've implemented a solution using DASH playlists. You can now use the --dash flag to download videos along with their audio. For example:
reddit-dl --url <reddit_video_url> --dash

So in the end I'm getting audio file with video file separately?

nopeee, now you get them together.


Expose disableExecuteApiEndpoint for Rest APIs

Resolves #117.
I'm planning to add a test, but it worked for me in manual testing (observing that "Default endpoint" in the console API Settings toggled between an execute-api URL and "inactive".

Exercising the parameter in the simple-go example (plus your manual verification of the impact in the console) seems like enough testing to me

Added one anyway before I saw your response. More testing is good.