unknown type name 'hipFunction_attribute' (roc-3.0.0 tag)

When compiling hipSYCL v0.8.0, i got the following error:
/opt/rocm/include/hip/hcc_detail/hip_prof_str.h:595:7: error: unknown type name 'hipFunction_attribute'; did you mean 'hipFuncAttributes'?

I need to patch /opt/rocm/include/hip/hcc_detail/hip_prof_str.h to make the compilation works:
@@ -592,7 +592,7 @@
     } hipMemPtrGetInfo;
     struct {
       int* value;
-      hipFunction_attribute attrib;
+      hipFuncAttributes attrib;
       hipFunction_t hfunc;
     } hipFuncGetAttribute;
     struct {

I want to send Pull Request but I'm not sure why include/hip/hcc_detail/hip_prof_str.h is included in .gitignore: https://github.com/ROCm-Developer-Tools/HIP/blob/36529b168022a644685a194cfd5e402948c98e3c/.gitignore#L16

hip_prof_str.h is a generated file hence it is marked as ignored files in git. There is a py file which generates this file. If you think any fix is needed you need to make change in the python file.
Regarding the change you have proposed I think there are two functions hipFuncGetAttribute and hipFuncGetAttributes ( Notice the extra 's' in second function) when first takes hipFunction_attribute as argument and second takes hipFuncAttributes as argument. None of these functions has got changed recently hence I am not sure why are you seeing compilation error with rocm3.0.

@SarbojitAMD Also I think my patch doesn't make sense but I don't know why it works:

@@ -592,7 +592,7 @@
     } hipMemPtrGetInfo;
     struct {
       int* value;
-      hipFunction_attribute attrib;
+      hipFuncAttributes attrib;
       hipFunction_t hfunc;
     } hipFuncGetAttribute;
     struct {

None of these functions has got changed recently hence I am not sure why are you seeing compilation error with rocm3.0.

I see, probably I did some mistakes then. I'm going to rebuild ROCm and I'll update here.

closing it as 'not a bug' feel free to open a new ticket in case you need any more clarification.

@SarbojitAMD Alright. Btw I can confirm this not a bug. There are conflicts with hipSYCL's HIP submodule so do not pull that submodule. Here what I did to prevent the conflict when building hipSYCL.
Get hipSYCL source and pull only hipCPU submodule:
$ cd ~/projects
$ git clone -b v0.8.0 https://github.com/illuhad/hipSYCL.git
$ git submodule init contrib/hipCPU
$ git submodule update contrib/hipCPU
$ cd hipSYCL

Patch CMakeLists.txt:
@@ -168,7 +168,6 @@ file(WRITE ${SYCLCC_CONFIG_FILE_PATH} ${SYCLCC_CONFIG_FILE})

 install(DIRECTORY include/CL DESTINATION include/ FILES_MATCHING PATTERN "*.hpp")
 install(DIRECTORY contrib/hipCPU/include/hipCPU DESTINATION include/hipSYCL/)
-install(DIRECTORY contrib/HIP/include/ DESTINATION include/hipSYCL/)

 install(PROGRAMS bin/syclcc DESTINATION bin)
 install(PROGRAMS bin/syclcc-clang DESTINATION bin)

Build, compile, and install:
$ mkdir build
$ cd build
$ export HIPSYCL_CUDA_PATH=""
$ cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/opt/rocm/hipsycl -DCLANG_EXECUTABLE_PATH=/opt/rocm/llvm/bin/clang -DWITH_ROCM_BACKEND=ON ..
$ make -j 16
$ sudo make -j 16 install
$ echo "/opt/rocm/hipsycl/lib" | sudo tee /etc/ld.so.conf.d/hipsycl.conf
$ sudo ldconfig

Add the following lines into /etc/profile.d/rocm.sh:
export HIPSYCL_CUDA_PATH=""
export HIPSYCL_GPU_ARCH="gfx900"
export HIPSYCL_CLANG="/opt/rocm/llvm/bin/clang"
export HIPSYCL_ROCM_LINK_LINE="-lstdc++ -rpath $HIPSYCL_ROCM_LIB_PATH -L/opt/rocm/lib -lhip_hcc"

Re-login or execute the export commands above to apply.
To test, try compile add_two_vectors.cpp:
$ mkdir -p ~/examples/sycl-add_two_vectors
$ cd ~/examples/sycl-add_two_vectors
$ /opt/rocm/hipsycl/bin/syclcc-clang add_two_vectors.cpp -o add_two_vectors
$ ./add_two_vectors

The output should be:
Result:
0
4
0
8
0

This is fixed in current hipSYCL master. I also recommend setting the HIPSYCL_GPU_ARCH, HIPSYCL_CLANG, HIPSYCL_ROCM_LINK_LINE etc parameters with cmake (-DDEFAULT_GPU_ARCH, -DROCM_LINK_LINE, -DLLVM_DIR). This will then make hipSYCL use those settings by default without having to modify the environment.

@illuhad

I also recommend setting the HIPSYCL_GPU_ARCH, HIPSYCL_CLANG, HIPSYCL_ROCM_LINK_LINE etc parameters with cmake (-DDEFAULT_GPU_ARCH, -DROCM_LINK_LINE, -DLLVM_DIR)

Noted, I have updated my comment.


Add more resolution scaling filters

Could you add more scaling interpolators?
Bilinear is not a good interpolator. Better would be lanczos, spline or bicubic.
What I mean is the way how it goes to the target resolution. e.g. if the game is on 640x480 and you upscale it to your e.g. 2560x1600 monitor.

What do you mean by spline? And which variant of bicubic, for that matter? I'm not an image processing expert, but even just a little bit of searching leads to many different variations, parameters, etc. I don't want to overcomplicate things. Perhaps I'll just start out with something "simple" like Lanczos, which at least seems to have a well specified single definition (apart from the fact that it also has a customizable parameter), unlike the others you've mentioned.
But, I don't know when I'll have time for such luxury stuff anyway, for now there are a lot of more important issues to be fixed.

http://avisynth.nl/index.php/Resize
Lanczos 3 (=3 taps)
Spline36 you can see in the link too
Bicubic can stay at the default values I think. Same for Lanczos if its then 3 taps.
I dont know if it helps, but dgvoodoo2 has these scalers:
Resampling: When scaling is done by the dgVoodoo for the given scaling mode,
;                         you can choose which filter is to be used for resampling the output image
;                         Available filters are: "pointsampled", "bilinear", "bicubic", "lanczos-2", "lanczos-3"

Once I came across an interesting thread related to a recommendation for downscaling filter for gaming content in OBS - recording software. It is said that bicubic is the default recommendation as lanczos is meant for face-cams and real-life videos.

Lanczos is the same universal usable as bicubic. Same goes for Spline.
Bicubic can be better if you do extreme upscaling like from 480p to 4k or so. Then the sharper lanczos might produce ringing artifacts around fonts.
I mainly speak about upscaling. I doubt there will be much downscaling going on here. Games can have a too low resolution. Never heard of a game where the minimum resolution is too high.
The OBS user probably wants to downscale the resolution for his twitch livestream. For downscaling I would also rather use bicubic.

From what I could understand from that thread (correct me if i'm wrong) is that the described filters work this way for both cases - downscaling & upscaling. I realize that in regards to DDrawCompat we have upscaling in mind.

I cant agree on that. I had always good results with Lanczos. Doesnt hurt to offer it anyway, right?
Even more I like Spline36.

DisplayFilter is used for downscaling when the target resolution is larger than the display resolution. This is often the case when ResolutionScale is used, even with a display(1) setting, since it rounds up to integer scale factors. So a 640x480 -> 1920x1080 resolution scale actually upscales with a 3x factor to 1920x1440, and then downscales to 1440x1080.
The way downscaling currently works is that for scale factors less than or equal to 0.5, it repeatedly downscales the image to half size with simple bilinear filtering, until the scale factor is greater than 0.5. Then the last scaling step is performed with the selected DisplayFilter. This should probably work fine with these new filters too.

I added all requested filters in v0.5.0. I ended up implementing proper downscaling too, using discrete convolution with increased tap count and all that. At least, I hope I got it right, looks good enough to me.

Wouldnt that have to be at "ResolutonScaleFilter" available then? How I read it here, this would still upscale via nearest neighbor and would just do the downscaling to the monitor resolution via e.g. spline.

I don't know where you got all this info from. The requested filters are in the DisplayFilter setting.
ResolutionScaleFilter is a completely different setting that only affects resolution-scaled render targets, and only if the application ever needs to access them through CPU. In this case, ResolutionScaleFilter is used to temporarily downscale the image, so that the application can read/write some pixels directly through the CPU. Then, only the CPU-modified pixels will be scaled back up again, and these always use nearest neighbor. The unmodified pixels preserve their original resolution.
DisplayFilter is universally applicable in all situations when the final display resolution is different from internally rendered resolution. So it will either downscale to the DisplayResolution (e.g. if a large ResolutionScale setting is used), or upscale from the application's internal resolution to the DisplayResolution (e.g. if ResolutionScale=app is used and the display resolution is larger).


[AMBARI-25310] : Replace deprecated methods of FileUtils

What changes were proposed in this pull request?
Replace usages of deprecated methods from FileUtils: readFileToString and writeStringToFile
How was this patch tested?
The patch was tested manually and with unit tests

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5300/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5302/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5303/
Test FAILed.
Test FAILured.

retest this please

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5305/
Test PASSed.

Please review @zeroflag @dlysnichenko

Please review @kasakrisz

@dlysnichenko @hapylestat could you please help me merge this patch?

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5317/
Test PASSed.


./get-wget-lua.sh?

Hello, I'm sorry I'm still pretty new with commands and python. How exactly is the "./get-wget-lua.sh" meant to be entered in the cmd?

The grab scripts are designed for Linux, and won't work on windows.


How To Implement In Flutter

hi,
Someone help me out to integrate bankid login with firebase

Hi @wifirevaithi!
Active Login is a library to get started with BankID for .NET and does not have any support for Flutter/Firebase. If you have experience with .NET and Identity Server, you could use that to make Flutter talk OIDC to Identity Server, but we don't provide any free support on that in addition on what's already available in our docs.
Here are the technical docs from BankID to implement BankID yourself in the platform (Flutter/Firebase) you use:
https://www.bankid.com/assets/bankid/rp/bankid-relying-party-guidelines-v3.2.2.pdf
NOTE: You are not allowed to implement BankID right from the app (Flutter), it needs to be backed by a backend that can host the certificate.
Closing this issue now, as it's not related to Active Login.

Thanks for the response.i need to create my own api and have to use in my flutter app right?

Yes, you need some kind of backend API. That could be Active Login, if you know .NET or implemented in the language of your choice.

no,i don't know .net only php


Migrate some architecture notes from the wiki

Merge https://github.com/rust-lang/cargo/wiki/Cargo's-architecture into
the ARCHITECTURE.md in the repo.
Fixes #5528

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

If this is accepted I think https://github.com/rust-lang/cargo/wiki/Cargo's-architecture should be deleted, or emptied and link to ARCHITECTURE.md.

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing e7f00b34c8a60f8479903e071f5299523bcdc290 to master...


Suggestions for the list

I am investigating the effects of things like square brackets and exclamation marks using statistical methods:
https://github.com/JohannesGaessler/stable-diffusion-tools/tree/master/emphasis
There is also a wiki associated with the Stable Diffusion threads on 4chan's technolo/g/y board:
https://wiki.installgentoo.com/wiki/Stable_Diffusion

This is very interesting, I'll add them when I get the time

Done !

Oh right, I'll rename it to "4chan /g/ wiki"


How to add titles to the code block?

Using Prism with COY theme, I create a code block with  a
 tag:

  public class Test {

    private final static String TAG = PrintUtils.class.getSimpleName();
    public static byte[] NEW_LINE = {0x0A};
    public static byte[] POWER_OFF = {0x1B, 0x3D, 0x02};
}

And my result will like this:

So, my question is: How to make a title for this code block? I would like it actualy like this:

Please help! I don't see this trick in API doc!

You could use the recently added data-language àttribute of the show-language plugin, but that's not what it was designed for.
Anyway what's wrong with this:

Test.java

  public class Test {

    private final static String TAG = PrintUtils.class.getSimpleName();
    public static byte[] NEW_LINE = {0x0A};
    public static byte[] POWER_OFF = {0x1B, 0x3D, 0x02};
}

IMHO, this is better semantically---it's what I use:

Test.java

  public class Test {

    private final static String TAG = PrintUtils.class.getSimpleName();
    public static byte[] NEW_LINE = {0x0A};
    public static byte[] POWER_OFF = {0x1B, 0x3D, 0x02};
}

Yeah not sure why this should be something that Prism addresses tbh.

Thank you, garretwilson and zeitgeist87! both ways are very useful.
Author can close this issue now! :dancer:


Typed out Names

When i type out my dead name it automatically changes it to my name which is generally not a terrible problem but in times when i /need/ to type my dead name like on forms or in the case that some one else has the same name i need to turn off the feature and turn it back on again which is sometimes more dysphoric than just using my deadname.
instead maybe there could be an option where you can toggle it on and off or you can choose whether you want it to autocorrect when you type it out or not. thanks so much for listening !
Gregory Ballard

This should be resolved in the newest version (1.2)


create a contact us form

create a contact us form with following details
name  , email , message

i want to work on this issue please assign me

please assign this to me @iamparas0

i have solved this issue. Please assign it to me and merge the files

Hello sir I would like to resolve this issue, can you kindly assign it to me?

@iamparas0 i want to work on this issue please assign me

GO FOR IT , PUT SS BEFORE PR!

Hi
I would like to solve this issue. I would like to create a contact us page. I'd like to work on this if it is still open.
Thank you!

@iamparas0 I want to contribute for this, please assign me

is this completed yet


Promise.then should use this return type

class Subclass<T> extends Promise<T> { }
new Subclass<void>((resolve, reject) => resolve()).then() instanceof Subclass
// true

Problem: there's no way to rebind the generic parameters of the this type to match the return type of the function passed to then - and, conceptually, a given subclass may not even be generic.

I realised that this is directly related to #6223.  In a theory, not being able to express some sort of generic resolution of polymorphic this will cause challenges.

#5999

see #6223 and #5999


Fix for issue #49

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   53.69%   53.69%
=======================================
  Files           5        5
  Lines         149      149
=======================================
  Hits           80       80
  Misses         69       69

Continue to review full report at Codecov.

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

@tisnik Your image is available in the registry: docker pull quay.io/openshiftio/rhel-fabric8-analytics-f8a-api-gateway:SNAPSHOT-PR-50


Build problem

I'm getting this problem when trying to build the app, you know what it could be?

I am also using phonegap-plugin-push in version 2.2.3

I'm sorry to answer so late, there is a new version without Firebase. Try it and open a new issue if you find new problems


Unable to start

I'm trying to get hotel working on Elementary OS (based on Ubuntu 14.04). When installing without sudo, I get EACCES: permissions denied errors, and when installing with sudo, hotel simply won't start. The following appears in the deamon.log file:
2 Mar 12:09:34 - Watching /home/jaco/.hotel/servers
create /home/jaco/.hotel/daemon.pid 4635
2 Mar 12:09:34 - Server listening on port <IP_ADDRESS>:2000
Exiting
Stop daemon
Remove pid file
events.js:160
throw er; // Unhandled 'error' event
^
Error: watch /home/jaco/.hotel/servers ENOSPC
at exports._errnoException (util.js:1022:11)
at FSWatcher.start (fs.js:1429:19)
at Object.fs.watch (fs.js:1456:11)
at createFsWatchInstance (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:37:15)
at setFsWatchListener (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:80:15)
at FSWatcher.NodeFsHandler._watchWithNodeFs (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:228:14)
at FSWatcher.NodeFsHandler._handleDir (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:407:19)
at FSWatcher. (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:455:19)
at FSWatcher. (/usr/lib/node_modules/hotel/node_modules/chokidar/lib/nodefs-handler.js:460:16)
at FSReqWrap.oncomplete (fs.js:123:15)
Any pointers would be appreciated

Hi @productiveme,
Yes, the fix in https://github.com/typicode/hotel/issues/147 should probably help. Basically, ~/.hotel should belong to user and not root.

I've created an issue for this particular case https://github.com/typicode/hotel/issues/178 and updated FAQ with the fix.


Regenerate Release file for bionic

It seems that https://apache.bintray.com/couchdb-deb/dists/bionic/Release is out of sync with the updated Packages files that now include couchdb (#1314).
$ curl https://apache.bintray.com/couchdb-deb/dists/bionic/Release
Origin: Bintray
Label: Bintray
Suite: bionic
Codename: bionic
Date: Tue, 10 Jul 2018 17:33:05 UTC
Components: main
Architectures: amd64 i386
MD5Sum:
 057a5afbd0c3166bed77efc3e6e9bef7             2057 main/binary-amd64/Packages
 687c07422e11dc076929043f30f97059              947 main/binary-amd64/Packages.bz2
 8bf8227378fb210c3f4f84918659fcb1              797 main/binary-amd64/Packages.gz
 d41d8cd98f00b204e9800998ecf8427e                0 main/binary-i386/Packages
 4059d198768f9f8dc9372dc1c54bc3c3               14 main/binary-i386/Packages.bz2
 3970e82605c7d109bb348fc94e9eecc0               20 main/binary-i386/Packages.gz
SHA1:
 c26ca1be16553191e4f3d9da4ac36d575e253a21             2057 main/binary-amd64/Packages
 f56a37223edd7e633203ecb881a981fb01fa0114              947 main/binary-amd64/Packages.bz2
 aa79723e3ab68736bd2d9bb9721c34d1cab18a5c              797 main/binary-amd64/Packages.gz
 da39a3ee5e6b4b0d3255bfef95601890afd80709                0 main/binary-i386/Packages
 64a543afbb5f4bf728636bdcbbe7a2ed0804adc2               14 main/binary-i386/Packages.bz2
 e03849ea786b9f7b28a35c17949e85a93eb1cff1               20 main/binary-i386/Packages.gz
SHA256:
 228cb32b6a64561ab4577ac2785a3e8177f5b88a2b9ca1b385477bd942b69cad             2057 main/binary-amd64/Packages
 b7a4424efa53e104f311dd5891f92b4cf24138f54e44501f083b3fe1b834bcd9              947 main/binary-amd64/Packages.bz2
 91d10f08d69f36dab28f3194d864eeeb7e0e8cf167b4ea7bb7e280f3c6465ec5              797 main/binary-amd64/Packages.gz
 e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855                0 main/binary-i386/Packages
 d3dda84eb03b9738d118eb2be78e246106900493c0ae07819ad60815134a8058               14 main/binary-i386/Packages.bz2
 f5d031af01f137ae07fa71720fab94d16cc8a2a59868766002918b7c240f3967               20 main/binary-i386/Packages.gz

The file sizes and hashes don't match the files that are now below main/binary-amd64/.
Thanks!

In the mean time a workaround is to
wget -c https://apache.bintray.com/couchdb-deb/pool/C/CouchDB/couchdb_2.1.2~bionic_amd64.deb
sudo dpkg -i couchdb_2.1.2~bionic_amd64.deb

@AndyA  also ran into this. Tried your workaround but the dependency for couch-libmozjs seems to not be installed/installing. Any suggestion?

Ah - of course. You'll also need
https://apache.bintray.com/couchdb-deb/pool/s/spidermonkey/couch-libmozjs185-1.0_1.8.5-1.0.0+couch-2~bionic_amd64.deb
and maybe
https://apache.bintray.com/couchdb-deb/pool/s/spidermonkey/couch-libmozjs185-dev_1.8.5-1.0.0+couch-2~bionic_amd64.deb

This was a bintray issue, for some reason they didn't calculate the metadata correctly.
I've re-uploaded the binaries, which has forced a metadata recalculation.
I tested with a local bionic and it worked fine.
Thanks for the report, and sorry for the difficulty.

Thanks very much @wohali


add more spells

[ ] accio for wget
[ ] add more here

What if wget isn't installed

@yamboy1
exec(command,function(err,stdout,stderr){
		if(err){
			return console.log(chalk.greenBright('FileError: Some error occured while fetching URL'));
		}
		return console.log(chalk.yellowBright('File Download successful'));

});

Here, if err occurs, then you can install wget, simple code will be:
 exec(command,function(err,stdout,stderr){
		if(err){
			console.log(err);
                        exec("sudo apt-get install wget",(error,stdout,stderr){
                                  if(error){console.log(error)}
                                  else{console.log('wget installed successfully');
                        });
		}
                else{
		        return console.log(chalk.yellowBright('File Download successful'));
		}
 });

But since there can be many errors like error with URL parsing and wget not installed error.
so if you have a better code that can differentiate between between these errors then by all means do it and send a pr.
Cheers!

I'll look into this.
BTW: There is more than one package manager. Debian and Ubuntu use apt, but other distros use rpm, pacman (Yes that is actually a package manager), yum and many more so you would have to account for those.

@gabru-md Could you assign this issue to me.

Issues can only be assingned to collaborators i guess.
You can send a Pull Request with the changes done. If you want to collabirate then please verify your emwil address registered with github and let me know.
Cheers!

I'm going to add an npm preinstall hook rather than add the code into accio.js

sure :+1:

Heres my pull request #4. It still needs some testing.

@gabru-md can you suggest any spell which I can work upon and add it into this.

@ms10398 have you seen/read harry potter ?
If yes then you can add any spell with a relevant linux command attached to it.
I donot have any in mind as of now. Do you have any ?

I will see all harry potter spells and check if can get a good spell in mind to add it.

added a spell - nox
PR - https://github.com/yogdaan/LinuxSpells/pull/15


Beautify README into markdown

It was difficult to understand.

Thanks for the patch.  I haven't touched this code in 11 years (!) so it's likely it doesn't work at all anymore.  From that perspective, I think I'd rather leave it alone.  Does that make sense?

I am interested in compilers, this was something unique! I found you on a  HN thread[0]. I was looking into it and thinking about a python implementation somehow.
Yea its old, pretty old , but classic.
That's why, I pulled a merge request, nvm.
[0] https://news.ycombinator.com/item?id=862396


Use extensionPath to resolve resources path for webview

It turned out that if the extension was built with Nix, the webview gets a 401 when trying to access any of its resources.
With these lines:
log.appendLine(`dirname resolved: ${path.resolve(__dirname, '../../resources')}`);
log.appendLine(`extensionPath resolved: ${path.join(extContext.extensionPath, 'resources')}`);

While debugging the extension from vscode, no Nix involved:
dirname resolved: /Users/kubukoz/IdeaProjects/vscode-tlaplus/resources
extensionPath resolved: /Users/kubukoz/IdeaProjects/vscode-tlaplus/resources

Same path.
With nix and home-manager handling vscode extensions:
dirname resolved: /nix/store/i7vk8j8k7iqvmbh9iwjplpry96cdcgp8-vscode-extension-vscode-tlaplus/share/vscode/extensions/alygin.vscode-tlaplus/resources
extensionPath resolved: /Users/kubukoz/.vscode/extensions/alygin.vscode-tlaplus/resources

The contents of the files are essentially the same... but:
Nix put the extension sources in /nix/store/i7vk8j8k7iqvmbh9iwjplpry96cdcgp8-vscode-extension-vscode-tlaplus/share/vscode/extensions/alygin.vscode-tlaplus, and these are the original ones: with the ${resourcesPath} placeholder still there.
The result of building the package with Nix is put into /nix/store/swvbpd1m4cjbdbs50rpqylm8za0ags2x-home-manager-files/.vscode/extensions/alygin.vscode-tlaplus, which is symlinked in my ~/.vscode/extensions by home-manager as alygin.vscode-tlaplus - and this actually contains the right replacements made in:
const resourcesDiskPath = vscode.Uri.file(
    path.join(extContext.extensionPath, 'resources')
);
const resourcesPath = viewPanel.webview.asWebviewUri(resourcesDiskPath);
viewHtml = viewHtmlTemplate
    .replace(/\${cspSource}/g, viewPanel.webview.cspSource)
    .replace(/\${resourcesPath}/g, String(resourcesPath));

so I just went and used the same approach here to get the same path. Both (with links resolved) start with /nix/store, but are completely unrelated, so it makes sense that the browser wouln't recognize them.
As a bonus, I removed the uri format from the Java Home config, as I believe it's incorrect: closes #195.

The change doesn't seem to have an effect on:
viewPanel.iconPath = {
        dark: vscode.Uri.file(path.resolve(__dirname, '../../resources/images/preview-dark.svg')),
        light: vscode.Uri.file(path.resolve(__dirname, '../../resources/images/preview-light.svg')),
    };

but I think it would be best to change these as well, for consistency. What do you think?

The change doesn't seem to have an effect on:
viewPanel.iconPath = {
        dark: vscode.Uri.file(path.resolve(__dirname, '../../resources/images/preview-dark.svg')),
        light: vscode.Uri.file(path.resolve(__dirname, '../../resources/images/preview-light.svg')),
    };

but I think it would be best to change these as well, for consistency. What do you think?

FWIW: I know almost nothing about how the webview stuff works or if this comment applies here.  However, I remember that the webview and resource loading caused problems earlier with gitpod and codespace.

FWIW: I know almost nothing about how the webview stuff works or if this comment applies here.  However, I remember that the webview and resource loading caused problems earlier with gitpod and codespace.

I can try to double-check that these still work :) unless it requires an extension to be in the marketplace already.

I can try to double-check that these still work :) unless it requires an extension to be in the marketplace already.

Codespaces are still invite-only.  The BlockingQueue example can quickly be launched by opening https://gitpod.io/#https://github.com/lemmy/BlockingQueue.  You can manually install a custom build of this extension.

Codespaces are still invite-only.  The BlockingQueue example can quickly be launched by opening https://gitpod.io/#https://github.com/lemmy/BlockingQueue.  You can manually install a custom build of this extension.

Here's a pre-release with this fix: v1.5.3-alpha.1

Here's a pre-release with this fix: v1.5.3-alpha.1

It works 🎉

It works 🎉


the screen seems to be black and white

[Issue](url
) Type: Bug
Visual studio code screen always make black and white dots when i run it
VS Code version: Code 1.47.3 (91899dcef7b8110878ea59626991a18c8a6a1b3e, 2020-07-23T13:12:49.994Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.80GB (7.18GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

python
ms-
2020.3.71659

There is a screenshot
On Wed, Aug 12, 2020 at 10:04 PM vscodebot[bot]<EMAIL_ADDRESS>wrote:

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already
covered by an existing one, like:

visual studio code shows black screen (#91599)
https://www.github.com/microsoft/vscode/issues/91599
Black theme, white block appears in the window (#60430)
https://www.github.com/microsoft/vscode/issues/60430

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/104515#issuecomment-673082924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7F6QILYO5UYBM3K5WQ2HLSALYUPANCNFSM4P5G47AA
.

Looks like the screenshot did not upload. Please upload it again through GitHub

[image: 117649307_422227962058538_8299422017693613576_n.png]
On Wed, Aug 12, 2020 at 11:59 PM Matt Bierner<EMAIL_ADDRESS>wrote:

Looks like the screenshot did not upload. Please upload it again through
GitHub
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/104515#issuecomment-673131626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7F6QNHSLOYBJBA3F4RXPTSAMGF7ANCNFSM4P5G47AA
.

Sry there was an error uploading on github
On Thu, Aug 13, 2020 at 12:07 AM Andrew Samir<EMAIL_ADDRESS>wrote:

[image: 117649307_422227962058538_8299422017693613576_n.png]
On Wed, Aug 12, 2020 at 11:59 PM Matt Bierner<EMAIL_ADDRESS>wrote:

Looks like the screenshot did not upload. Please upload it again through
GitHub
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/104515#issuecomment-673131626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7F6QNHSLOYBJBA3F4RXPTSAMGF7ANCNFSM4P5G47AA
.

Still didn't work. Make sure to use GitHub instead of email

Does this happen for any color theme ?
can you check if the issue happens with exploration build https://az764295.vo.msecnd.net/exploration/45f2f912b8c3bdf45bb92dc613b902136a1b60ed/VSCodeUserSetup-x64-1.49.0-exploration.exe

Looks like #93356 or #94523
Usual fix is to disable hardware acceleration if exploration buid doesn't work.

Thank you disabling hardware acceleration fix it

@Andrew-Samir can you also try the exploration build without disabling hardware acceleration, I would like to confirm if newer runtime fixes the gpu issue. Thanks!

I did try it but unfortunately it doesn't work

thanks for confirming, that helps


ModelBakery Error with Milk Bucket Item Texture (You read that right)

I was building a MC modpack and kept getting crashes whenever the game would load the milk bucket item texture. I went through and re added every mod one by one only until I had them all in but WR CBE. When I added it, the game would crash due to aforementioned texture error. Without it, it would load just fine. I have the most up to date version of CodeChickenCore, CodeChickenLib, Forge Multipart, and Forge itself for version 1.10.2 Attached below is the crashlog.
crash-2018-02-03_20.48.22-client.txt

This is MFR, has been reported hundreds of times to them and is completely load order dependent, adding WR-CBE likely throws the load order so the crash appears. Either remove MFR and use IndustrialForgoing or remove it and wait for MFR to update.


Inno 186: Make card layout to more closely match real cards

Work in progress

If you look closely at the resource icons in this screenshot, do you see how the borders aren't even widths?

For example, changing the background-position-x for .M.icon_4.square_card_icon from -117px to -116px made that icon look better.

I see, I will see what I can tweak

The "button" like highlight of the icon is throwing me off. I think I got them all centered within the color border now. I had to use a screenshot to zoom in.

This does look great!  Nice work @lordmcfuzz !


ManagedDatabaseMoveOperations: Initial commit

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

New API

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.

August 2022

When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.

August 2022

If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@ericshape, @jamestao This PR replaces #20031

@ericshape, @jamestao This PR replaces #20031

@djordje-jeremic , can you close the other PR?

@ericshape, @jamestao This PR replaces #20031

@djordje-jeremic , can you close the other PR?

@raych1 I would've already, if it was owned (created) by me. I took over from my colleague who'll be OOF for some time.

Looks mostly good. .. Can you please explain the issue with moving to the v3 schema ? Which script are you talking about ? Isnt that owned by your team ?

I am signing off , but consider updating your processes to use the latest schema definitions.

Looks mostly good. .. Can you please explain the issue with moving to the v3 schema ? Which script are you talking about ? Isnt that owned by your team ?

@raosuhas We have a script in the the DsMainDev repository which generates Swagger files for APIs we own (Azure SQL), and the schema version is probably hard coded in it. I'm a bit new in this area but if it makes sense to change the script so all future Swaggers from this repo come with v3 instead of v1, I could make a PR for that and add relevant people.

@ericshape, @jamestao Friendly ping for the matching ARM signoff as well: https://msdata.visualstudio.com/Database Systems/_git/DsMainDev/pullrequest/857883


[verifiable-credentials] add expirationDate field to VC schema

As discussed in the design notes it makes sense to put expirationDate in the schema. This makes it easier to check and provides protection against tampering (because it then gets included in the signed didJwtVC field).
This field should be optional (as some VC may not expire).
This issue includes both the code changes to the library to make sure it is set correctly AND the updates to the schema doc itself.
Note that the tamper-proof-clock issue in the design note is not included here.

Hey team! Please add your planning poker estimate with ZenHub @cmcWebCode40 @tahpot

@nick-verida,
@tahpot actually fixed this issue yesterday in the call we had and wrote tests for this too.
The changes are made on this branch feature/142-test-cases-for-vcs


Populate $scope.$$isolateBindings for Angular 1.5 components, the same way as it currently behaves for isolate scopes in directives

Note: for support questions, please use one of these channels: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question. This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
feature
What is the current behavior?
On .component()s created in Angular 1.5.x, the $$isolateBindings defintion is an empty object, and not reflective of the data supplied to the "bindings" property.
What is the expected behavior?
The bindings property should behave as it does for isolate scopes in .directive(), and provide the correct set of definitions
What is the motivation / use case for changing the behavior?
We bind some handler methods automatically - this would help tremendously in that process.
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Angular 1.5+
Other information (e.g. stacktraces, related issues, suggestions how to fix)
For example, this works:
.directive('test', ['$scope', function($scope) {
  templateUrl: 'test.html',
  bindings: {
    onStart: '&?',
    onStop: '&?'
  },
  controller: function() {
    console.log($scope.$$isolateBindings); //Returns {onStart: {..}, onStop: {...}}
  }]
]});

This fails:
.component('test', {
  templateUrl: 'test.html',
  bindings: {
    onStart: '&?',
    onStop: '&?'
  },
  controller: ['$scope', function($scope) {
    console.log($scope.$$isolateBindings); //DOES NOT return {onStart: {..}, onStop: {...}}
  }]
});

This has nothing to do with .directive vs .component (.component uses .directive under the hood anyway). The difference is with bindToController (it is true by default in .component) - you will get the same results with .directive if using bindToController: true.
In any case, $$isolateBindings is a private Angular property (being $$ prefixed) and you shouldn't use t in your code.


[HAC-2169] Details page component (header + horizontal nav)

Note: This PR depends on PR #156
Jira ticket: https://issues.redhat.com/browse/HAC-2169
Test with yarn storybook.
This PR combines the details page header component and the horizontal nav to create a details page component.

Codecov Report
Base: 17.74% // Head: 22.07% // Increases project coverage by +4.32% :tada:

Coverage data is based on head (ae13875) compared to base (7dba4fb).
Patch coverage: 53.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
+ Coverage   17.74%   22.07%   +4.32%
==========================================
  Files          44       56      +12
  Lines        1544     1758     +214
  Branches      413      462      +49
==========================================
+ Hits          274      388     +114
- Misses       1270     1369      +99
- Partials        0        1       +1

Impacted Files
Coverage Δ

.../details-page-header/DetailsPageHeader.stories.tsx
0.00% <0.00%> (ø)

...-utils/src/components/details-page-header/index.ts
0.00% <0.00%> (ø)

...etails-page-header/utils/ActionButtons.stories.tsx
0.00% <0.00%> (ø)

...s/details-page-header/utils/ActionMenu.stories.tsx
0.00% <0.00%> (ø)

.../details-page-header/utils/Breadcrumbs.stories.tsx
0.00% <0.00%> (ø)

...rc/components/details-page/DetailsPage.stories.tsx
0.00% <0.00%> (ø)

...-utils/src/components/details-page/DetailsPage.tsx
0.00% <0.00%> (ø)

...s/lib-utils/src/components/horizontal-nav/index.ts
0.00% <0.00%> (ø)

...ls/src/components/horizontal-nav/HorizontalNav.tsx
93.33% <71.42%> (-6.67%)
:arrow_down:

...mponents/details-page-header/DetailsPageHeader.tsx
80.00% <80.00%> (ø)

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

FWIW this diff is much smaller when merged with upstream now, though Github is still showing the entire diff on this PR.

/approve


Avoid transaction and locking issues in postgresql and sqlite

Changes to

refactored to remove existing nested inTransaction calls (though there remain possibilities of others as long as we have inTransaction being called in DAOs as well as in Resources)
removed DB access from Workers, making them call DAOs instead, so that DB handles are shared (and DB code is appropriately compartmentalized in the database package).

Addresses NCI-Agency/anet#391.

Replaced by NCI-Agency/anet#446.


Library for JSON to CSV

Look up potential library for conversion between JSON and CSV. If there isn't any, figure out the conversion.
Time estimate: 1D
Check story #11 for more details/context.

Shri found a library we can use. Wrote a wrapper for it.


Refactor everything for more idiomatic rust

Trying to get rid of excess Mutexes, Arcs, explicit close functions, clones etc.

FCM:
Refactor everything for more idiomatic Rust (#2 )


Building causes error with AOT

Upon attempting to build this project with AOT with angular-cli I get the following errors, any idea?
zed@gingerninja ~/Development/project.example.com  (master *)$ ng build --target=production --environment=production
Hash: 289f9755f27fde876d85
Time: 11082ms
chunk    {0} polyfills.d270cc2992ad57a6bff0.bundle.js (polyfills) 222 kB {4} [initial] [rendered]
chunk    {1} main.06ee6a03687560961e81.bundle.js (main) 2.1 kB {3} [initial] [rendered]
chunk    {2} styles.40713392d4d9083ea2df.bundle.css (styles) 69 bytes {4} [initial] [rendered]
chunk    {3} vendor.29e2472b36186853adf4.bundle.js (vendor) 1.06 MB [initial] [rendered]
chunk    {4} inline.dee33d806992d75a5a27.bundle.js (inline) 0 bytes [entry] [rendered]

ERROR in Error encountered resolving symbol values statically. Could not resolve vis relative to /Users/zed/Development/project.example.com/node_modules/ng2-vis/components/timeline/index.d.ts., resolving symbol VisTimelineItems in /Users/zed/Development/project.example.com/node_modules/ng2-vis/components/timeline/index.d.ts

ERROR in ./src/main.ts
Module not found: Error: Can't resolve './$$_gendir/app/app.module.ngfactory' in '/Users/zed/Development/project.example.com/src'
 @ ./src/main.ts 5:0-74
 @ multi ./src/main.ts
zed@gingerninja ~/Development/project.example.com  (master *)$

Hm I never tried this lib with an AOT project but this library provides the ng2-vis.metadata.json and this file does not include the "VisTimelineItems". I'm not sure if this is the problem because the error says that it is more likely a problem with the import of the 'vis' library.
I really need to update to the latest angular version and use the @types/vis instead of my own type definitions.
Is you project open source so we can test an updated version of ng2-vis with AOT?

It is not, however I will quickly create a test project :)

This repo has the issue: GingerGeek/vis-aot-failure
It was created with latest angular 2. Get the error with any of:
ng serve --aot
ng build --aot
ng build --target=production

Another issue when try to ng build --prod works well in dev mode
"@angular/core": "~2.4.0"
"@angular/cli": "^1.0.0-rc.0"
"ng2-vis": "0.0.5"
Unexpected value 'VisNetworkDirective in ../node_modules/ng2-vis/components/network/vis-network.directive.d.ts' declared by the module 'VisModule in ../node_modules/ng2-vis/ng2-vis.d.ts'
Any idea?

Currently not working with AOT. I'm currently trying to fix this and also using the new type defs from DT. I hope I can get this running asap

Try building with --prod --aot false
That should fix your issue?

On 2 Mar 2017, at 17:16, joel00<EMAIL_ADDRESS>wrote:
Another issue when try to ng build --prod works well in dev mode
"@angular/core": "~2.4.0"
"@angular/cli": "^1.0.0-rc.0"
"ng2-vis": "0.0.5"
Unexpected value 'VisNetworkDirective in ../node_modules/ng2-vis/components/network/vis-network.directive.d.ts' declared by the module 'VisModule in ../node_modules/ng2-vis/ng2-vis.d.ts'
Any idea?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@GingerGeek Yes it works, although I don't think it's a good idea to disable AOT

That's why we have this issue open 😉

On 2 Mar 2017, at 19:28, joel00<EMAIL_ADDRESS>wrote:
@GingerGeek Yes it works, although I don't think it's a good idea to disable AOT
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

This same issue causes xi18n to crash. It does not have an option to disable AOT this is a complete blocker for me.
This is the stack trace you get when running ng xi18n:
An error occured during the i18n extraction:
Error: Error encountered resolving symbol values statically. Could not resolve vis relative to .../node_modules/ng2-vis/components/timeline/index.d.ts., resolving symbol VisTimelineItems in .../node_modules/ng2-vis/components/timeline/index.d.ts
    at simplifyInContext (...\node_modules\@angular\compiler\bundles\compiler.umd.js:25569:27)
    at StaticReflector.simplify (...\node_modules\@angular\compiler\bundles\compiler.umd.js:25581:17)
    at StaticReflector.annotations (...\node_modules\@angular\compiler\bundles\compiler.umd.js:25073:84)
    at NgModuleResolver.resolve (...\node_modules\@angular\compiler\bundles\compiler.umd.js:17557:86)
    at CompileMetadataResolver.getNgModuleMetadata (...\node_modules\@angular\compiler\bundles\compiler.umd.js:18058:64)
    at addNgModule (...\node_modules\@angular\compiler\bundles\compiler.umd.js:24839:62)
    at ...\node_modules\@angular\compiler\bundles\compiler.umd.js:24850:18
    at Array.forEach (native)
    at _createNgModules (...\node_modules\@angular\compiler\bundles\compiler.umd.js:24849:30)
    at analyzeNgModules (...\node_modules\@angular\compiler\bundles\compiler.umd.js:24724:18)
    at analyzeAndValidateNgModules (...\node_modules\@angular\compiler\bundles\compiler.umd.js:24734:39)
    at Extractor.extract (...\node_modules\@angular\compiler\bundles\compiler.umd.js:27589:22)
    at Extractor.extractBundle (...\node_modules\@angular\compiler-cli\src\extractor.js:40:33)
    at Extractor.extract (...\node_modules\@angular\compiler-cli\src\extractor.js:30:34)
    at Function.NgTools_InternalApi_NG_2.extractI18n (...\node_modules\@angular\compiler-cli\src\ngtools_api.js:92:26)
    at _donePromise.Promise.resolve.then (...\node_modules\@ngtools\webpack\src\extract_i18n_plugin.js:113:59)

I got that previous error to go away by installing the type definitions for vis npm i @types/vis, but now I have a new error. This error occurs when running ng xi18n or ng build -prod
ERROR in Unexpected value 'VisNetworkDirective in .../node_modules/ng2-vis/components/network/vis-network.directive.d.ts' declared by the module 'VisModule in .../node_modules/ng2-vis/ng2-vis.d.ts'

ERROR in ./src/main.ts
Module not found: Error: Can't resolve './$$_gendir/app/main/app.module.ngfactory' in 'C:\Users\leurba\Source\Repos\Dynamics365-HCM-TalentEngagement\src'
 @ ./src/main.ts 9:0-79
 @ multi ./src/main.ts

The issue appears to just be an angular version mismatch. Also, I get an error about vis not being defined which can be resolved by simply running npm i @types/vis.
I just submitted a PR that fixes the issue for me locally.

@itslenny thank you for the PR. I'm glad that this fixes the issue for you. I will fix the lint errors with the next version. I also will merge this PR this weekend and release a new version for you (maybe I have to tag it as pre-release but that should be fine).
Can someone of the others ( @GingerGeek or @joel00 ) confirm that AOT compilation works with these changes?

@seveves Negative. "Same error: ERROR in Unexpected value 'VisNetworkDirective in ...."
using: angular-cli rc2
updating angular 2.4.9 and installing @type/vis

@joel00 ok thank you ... I'm on it right now

Anybody up for testing this: npm install ng2-vis@next

Works for me after removing any reference to __vis and updating a few other types ;) 👍

But these references are in your own code and not related to ng2-vis?

I've tested a bit more with a local ng-cli project and all seems fine. So I published the 0.0.6 release.
Hope it works well for you @itslenny and @joel00 too

Yes - in my code, but they were referencing stuff within ng2-vis, so it introduces some breaking changes

On 18 Mar 2017, at 17:00, Severin<EMAIL_ADDRESS>wrote:
But these references are in your own code and not related to ng2-vis?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm sorry for these breaking changes but as long as it works now with AOT compilation it was worth the change 😸 (next change will be the update to vis 4.19 and angular 4)

Yeah it wasn't anything major, only took 30s to fix - well worth it for AOT compilation
Many thanks for all this 😁

On 18 Mar 2017, at 18:12, Severin<EMAIL_ADDRESS>wrote:
I'm sorry for these breaking changes but as long as it works now with AOT compilation it was worth the change 😸 (next change will be the update to vis 4.19 and angular 4)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@seveves with 0.0.6 AOT is working now. Thanks for your time

Thank you for using it and reporting bugs and being patient 😁 ... if you like this project then feel free to give it a star ✌


Static files handler doesn't route /path/ to /path/index.html

Actual behavior (the bug)
Static files config gets a 404 Not Found for the root path / and other paths ending in / (e.g. /course/), when an index.html exists in the corresponding resource folder.
Video showing behavior change when i switch to my v5 branch: https://www.loom.com/share/bb60c18b18004333a2b5c7402e960a96
Branch being used in the video: https://github.com/A1Liu/schedge/pull/141
Expected behavior
I think I expected the static files config to behave the same as in v4, and respond with the resource at /next/index.html when the user goes to http://localhost:4358/
To Reproduce
I made another branch with much less code that still reproduces the error, without needing to do anything too fancy: https://github.com/A1Liu/schedge/pull/149
It's a Gradle project, so it should be build-able using Gradle, and it runs on port 4358 .
After building, you can run the fat-jar directly using
java -Xmx1G -Djdk.httpclient.allowRestrictedHeaders=host,connection \
  -jar ./build/libs/schedge.jar serve

Additional context

I think I expected the ... resource at /next/index.html when the user goes to http://localhost:4358/

Why do you expect that though, that's very far from the path the user requested?
It's possible this was changed by either Jetty or us, but it does sound like a fix rather than a bug.

Why do you expect that though, that's very far from the path the user requested?

He is serving the /next/ folder at /, and expecting javalin to serve an index.html when a directory is requested, hence serving the /next/index.html file when a user GETs /. This is a pretty common default amongst webservers AFAIK.

It's possible this was changed by either Jetty or us, but it does sound like a fix rather than a bug.

I'm not so sure, taking a peek into jetty shows that index.html is intended to be a Welcome File by default in Jetty 11:
https://github.com/eclipse/jetty.project/blob/c4ef972190f377e17fba73b24e4e98a24fceac83/jetty-server/src/main/java/org/eclipse/jetty/server/handler/ResourceHandler.java#L58
I'll checkout your branch and investigate why this is not happening.

Update: Definitely a bug (at the very least an inconsistency) on our side, when setting staticFiles.precompress = false; javalin serves the index.html file, I'll keep digging.

This has been released as part of 5.1.1

Thank you!


Setup integration tests for third-party channels

Description of Problem: There are currently only few integration tests for the channels (e.g. for REST), but no integration tests for third party channels (e.g. Telegram, Facebook, Slack, etc.).
Overview of the Solution:

As we have to use APIs I could imagine that the tests are a bit flakey by nature. Hence, it might be a good idea to not run them as part of pull requests but e.g. on a daily schedule
I imagine something like a pytest test class which we can extend and

specify how a message is sent to the third party
specify how we test the correct response was received by the third party
allows specifying the credentials for the channel (which end up in credentials.yml)

Definition of Done

[ ] We have a CI job which regularly (either on PR or on schedule) tests our integration with third-party channels
[ ] We have a framework for a test which tests sending a message to the third party and makes sure Rasa Open Source sends back the expected response to the third party (e.g. Facebook)
[ ] the CI job can easily be extended when new channels are added to Rasa Open Source
[ ] We are notified if an integration breaks (e.g. on PR or via Slack notification in case of scheduled runs)
[ ] It's documented how to add a new test for an integration

This issue is not for adding integration tests for all channels. This is handled in further issues (see parent issue)

@RasaHQ/infrastructure I think this is a good task to collaborate on, what do you think? (cc @RasaHQ/enable-squad )

@RasaHQ/infrastructure I think this is a good task to collaborate on, what do you think? (cc @RasaHQ/enable-squad )

Possibly correlated with this issue? (could make sense to tackle both at the same time at the very least)

Possibly correlated with this issue? (could make sense to tackle both at the same time at the very least)

@wochinge @mprazz anyone else :D  Cool, do we wanna have a document or a call where we can discuss what is needed and determine the first steps?

@wochinge @mprazz anyone else :D  Cool, do we wanna have a document or a call where we can discuss what is needed and determine the first steps?

🙋🏻♀

🙋🏻♀

@tczekajlo At least for us the integration tests for tracker store, etc. are more of a priority. If you already want to get started, I'd be happy to have a call / document but in my opinion we can also do this as a next step after doing the other integration tests. What do you think?

@tczekajlo At least for us the integration tests for tracker store, etc. are more of a priority. If you already want to get started, I'd be happy to have a call / document but in my opinion we can also do this as a next step after doing the other integration tests. What do you think?

@wochinge I just wanted to get a sense of what is a priority for the enable squad in order to make planning a bit easier 😄 . I think your replied answers all my questions. First, we'll focus on the integration tests (https://github.com/RasaHQ/rasa/issues/7680) 🚀  Thanks a lot 🙌🏻

@wochinge after some discussion with the infra squad, we got to the conclusion that none of us has a definite answer to how we can implement these tests (nor the time to look into it), and that definitely seems to be the next step here. On the other hand, if the Enable Squad has prioritized these tests and has the time to conduct the investigation required here, feel free to do so. We'd be happy to consult once you have some implementation proposal for these 😊 Feel free to ping me if you need anything!

It's currently not in our OKRs so I don't expect any work on it to happen in this cycle. Let's do a meeting once this becomes a priority again 👍🏻

great product people, what is the status of this? we could really use it with the mattermost bot based on https://github.com/RasaHQ/helpdesk-assistant/issues/59

thanks for your interest @mihai-satmarean ! We haven't currently made plans to address this issue. Is it something you'd like to work on if you were given a bit of guidance? Usually to implement improvements for channels it's better to have people with hands-on experience with said channel.

@m-vdb Thanks for the reply! Yes, I would very much like to help if a bit of guidance is given. Waiting for your proposal on how to get this moved.

Closing as work on this ticket is represented by other issues


Generator rejected this profile with IllegalStateException

Bug Report
Steps to Reproduce:

Run the generator with the attached profile

java -jar libs/generator.jar generate --max-rows=100 --replace --profile-file=pairwiseNumberEqualTo-profile2.json --output-path=test/test.csv

Got an error of IllegalStateException (see below)

Expected Result:
I expect Generator to accept the profile
Actual Result:
Got the following error:
java.lang.IllegalStateException: Failed to merge field specs in related fields
        at com.scottlogic.deg.generator.generation.grouped.FieldSpecGroupValueGenerator.lambda$updateFirstSpecFromRelations$6(FieldSpecGroupValueGenerator.java:88)
        at java.base/java.util.Optional.orElseThrow(Unknown Source)
        at com.scottlogic.deg.generator.generation.grouped.FieldSpecGroupValueGenerator.updateFirstSpecFromRelations(FieldSpecGroupValueGenerator.java:88)
        at com.scottlogic.deg.generator.generation.grouped.FieldSpecGroupValueGenerator.generate(FieldSpecGroupValueGenerator.java:57)
        at com.scottlogic.deg.generator.generation.databags.RowSpecDataBagGenerator.lambda$createDataBags$0(RowSpecDataBagGenerator.java:43)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.HashMap$KeySpliterator.forEachRemaining(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)
        at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.evaluate(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline.collect(Unknown Source)
        at com.scottlogic.deg.generator.generation.combinationstrategies.MinimalCombinationStrategy.permute(MinimalCombinationStrategy.java:32)
        at com.scottlogic.deg.generator.generation.databags.RowSpecDataBagGenerator.createDataBags(RowSpecDataBagGenerator.java:45)
        at com.scottlogic.deg.generator.walker.rowspec.RandomRowSpecDecisionTreeWalker.generateWithoutRestarting(RandomRowSpecDecisionTreeWalker.java:68)
        at com.scottlogic.deg.generator.walker.rowspec.RandomRowSpecDecisionTreeWalker.walk(RandomRowSpecDecisionTreeWalker.java:53)
        at com.scottlogic.deg.generator.generation.DecisionTreeDataGenerator.lambda$generateData$0(DecisionTreeDataGenerator.java:88)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline$3$1.accept(Unknown Source)
        at java.base/java.util.stream.SortedOps$SizedRefSortingSink.end(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)
        at java.base/java.util.stream.StreamSpliterators$WrappingSpliterator.forEachRemaining(Unknown Source)
        at java.base/java.util.stream.Streams$ConcatSpliterator.forEachRemaining(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)
        at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(Unknown Source)
        at java.base/java.util.stream.AbstractPipeline.evaluate(Unknown Source)
        at java.base/java.util.stream.ReferencePipeline.collect(Unknown Source)
        at com.scottlogic.deg.generator.generation.combinationstrategies.MinimalCombinationStrategy.permute(MinimalCombinationStrategy.java:32)
        at com.scottlogic.deg.generator.generation.DecisionTreeDataGenerator.generateData(DecisionTreeDataGenerator.java:90)
        at com.scottlogic.deg.orchestrator.generate.GenerateExecute.execute(GenerateExecute.java:58)
        at com.scottlogic.deg.orchestrator.generate.GenerateCommandLine.call(GenerateCommandLine.java:58)
        at com.scottlogic.deg.orchestrator.generate.GenerateCommandLine.call(GenerateCommandLine.java:43)
        at picocli.CommandLine.executeUserObject(CommandLine.java:1743)
        at picocli.CommandLine.access$900(CommandLine.java:145)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2101)
        at picocli.CommandLine$RunLast.handle(CommandLine.java:2068)
        at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:1935)
        at picocli.CommandLine.execute(CommandLine.java:1864)
        at com.scottlogic.deg.orchestrator.App.main(App.java:35)

Environment:
Build of v1.30.0 (dated Nov 1, SHA5: 633cf64c9a3eb496585e102f832b1efc100e1ff9)
Additional Context:
The generator accept other simplier profiles (I can share those if interested)
The GitHub UI does not allow me directly attached a .json file, so I renamed it to .json.txt
pairwiseNumberEqualTo-profile2.json.txt

I think one possible reason for the Generator to fail is that in the profile, there are more than one pairwise relationships for the same field.
The Generator accepts these other two simpler profiles, which do not have more than one pairwise relationships:
pairwiseNumberEqualTo-profile2a.json.txt
pairwiseDateEqualTo-profile2.json.txt

This is not unexpected behaviour.
This json provided contains contradictory constraints, which are not present in the other two valid jsons, and this is thrown as an exception in FiledSpecGroupValueGenerator.java - 135:40.
If the message is the issue (i.e not particularly helpful to the user), then I would agree as it isn't technically an exception since it is valid domain flow. We should agree an approach to this further within a group discussion.

@cdowding-sl -- Let's have a discussion face to face.  The profile may not actually be contradictory.  Thanks!

This profile is not contradictory and is therefore failing unexpectedly.

Thanks!

Blocked by #1489

Copy of #1489


Provision staging environments

As a VA Notify developer
I want to have a staging environment.
Acceptance criteria:
WHEN I push code
AND the tests and deployment to development succeed
THEN the pipeline provisions the staging environment
Checklist:

[ ] ~using terraform workspaces -> need to have discussion on use of this (if we use this, we need to change S3 permissions and buckets used per environment)~
[x] NEW: have a further discussion about workspaces just to make sure everyone is on the same page about not using them, and invite @punitladtw
[x] parameterize the resources used in terraform by extracting out variables (into tfvars?) (to have identical resources in staging as we have in development)
[ ] create a new workflow file for staging

[ ] to trigger it: create a tag at the end of the development pipeline, and listen for new tags in the staging pipeline
[ ] create staging pipeline

[x] change name for terraform backend bucket (currently terraform-notification-test) and keys (currently things like notification-api-dev-db.tfstate) to remove environment names and make them more consistent
[x] Investigate how to set region in backend by a script (you cannot use variables in backend)
[ ] update cidr block calculation based on environment and vpc cidr

Resources:
https://medium.com/capital-one-tech/deploying-multiple-environments-with-terraform-kubernetes-7b7f389e622
https://codurance.com/2020/04/28/terraform-with-multiple-environments/
https://www.terraform.io/docs/cloud/guides/recommended-practices/part1.html
Assumptions:

Scope for this story only includes provisioning, the deployment was separated and will be part of #79

Questions:

How many days do we want to retain the cloudwatch logs in each environment? (dev, staging, prod)

Resource for options for the number of days: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-logs-loggroup.html

Do we need a new task definition json for each environment? (right now, we are hardcoding "dev" into all the json files) Yes, one yml file per environment
Do we need to parameterize the name of the docker containers? (this is set up in the task definitions) No

I talked with Filip and creating tag for each deployment to staging is the most feasible action.

With git tagging, possible approach:

https://github.com/actions/github-script

Resource:

https://git-scm.com/book/en/v2/Git-Basics-Tagging

Why not just use git tag?

Tagging and CI visual flow

Pair: @sarahproof / @lingtran
We successfully tested staging, all except for actually sending an email due to bug that seems to be related to email-from value (a request requirement).

There seems to be a problem with the queue in staging preventing emails from being sent. We get a lot of errors in the log like
Can only include alphanumeric
characters, hyphens, or underscores. 1 to 80 in
length
and mentioning invalid Sender

Celery creates a queue for each celery node. The queue is named using formula: queue-prefix + hostname, ex: dev-notification-celery_ip-10-0-0-103-us-east-2-compute-internal-celery-pidbox
In staging environment the queue name exceeds the allowed length of 80 characters.
There are 2 options to solve that issue:

Set CELERY_ENABLE_REMOTE_CONTROL to False
This will disable remote management of workers for which the pidbox queue is used.

Change the hostname value for celery when starting using -n flag
celery -A proj worker --concurrency=4 -n worker1@%h
From https://docs.celeryproject.org/en/stable/userguide/workers.html:
The hostname argument can expand the following variables:

%h: Hostname, including domain name<EMAIL_ADDRESS>%n: Hostname only (worker1@george)
%d: Domain name only<EMAIL_ADDRESS>Since currently we are using hostname only we would have to provide a custom variable that would be shorter but unique at a given time - IP address might be a good example.

For now I'm going to do option 1 as we are not performing remote management of workers.


Cannot find module 'firebase' or its corresponding type declarations.ts(2307)

Vscode is not recognizing the firebase.
Code compiles and there is no issue with the project also however, vscodes show it as error.
Also tried changing the file extentions from .ts to .js. It does not show any error but autocomplete does not work for firebase

This issue doesn't appear to be related to the GitHub Pull Requests and Issues extension.

@ts-nocheck
will turn off the error in this case. Firebase provides it's own types.


change affected rows count on refresh table

did you run the itests?

CHANGES entry is missing

otherwise lgtm


Missing "Keyboard Scan Code Specification" document

The document titled "Keyboard Scan Code Specification" and previously available as a file named scancode.doc, is unreachable.
It used to be reachable from the following urls:

http://download.microsoft.com/download/1/6/1/161ba512-40e2-4cc9-843a-923143f3456c/scancode.doc
http://go.microsoft.com/fwlink/p/?linkid=242210

According to archive.org, it was still available as of sept. 2nd, 2019.
As a result, a number of MS Docs pages have a broken link to this document, e.g. this one.

Was just looking for this document, hoping someone finds it soon.

The links appear to be corrected. Thanks.


Added Url class for representing full urls.

Still needs wiring into the hosts and also exposing via the render
context so view engines can generate urls from paths relative to
the nancy root.

For what do you need a custom Url class instead of Uri? With uri it is also possible to create relative urls like.
var root = new Uri("http://google.com");
var mail = new Uri(root,"mail")
will generate http://google.com/mail.

@lanwin that has been discussed before in other issues.

Can please you point me to it?

https://github.com/NancyFx/Nancy/issues/102
I still am more inclined to rely on URI then a custom URL class that might not expose all the bits, sure URI might bork but most of the times i've seen it bork is with non http(s) URI's.
Concidering these RFC's
http://www.ietf.org/rfc/rfc1738.txt
http://www.ietf.org/rfc/rfc2396.txt
http://www.ietf.org/rfc/rfc3986.txt
are just a sampling of URI related RFC's out there and the fact neither OWIN nor this Url class exposes all the bits (I'm missing UserInfo) I think both OWIN and Nancy should expose the full original request Url for us to break down in our own fashion.
I'd like to know your thoughts on this too Steven before i spam the OWIN mailing list demanding a "owin.CompleteUnalteredRequestPath" :)

@Mpdreamz UserInfo isn't in there because it's not really allowed in the HTTP URL scheme, it is allowed in the "generic" URI syntax (although discouraged), but not in HTTP, although depending on how you read the RFCs, and in what order, you may come to a different conclusion (the fun of RFC, and not something I have any interest in debating :-)) From RFC 1738 (the specific HTTP part):
An HTTP URL takes the form:
  http://<host>:<port>/<path>?<searchpart>

where  and  are as described in Section 3.1. If :
is omitted, the port defaults to 80.  No user name or password is
allowed.
As for the URI class, I despise its "exception happy" design and, for something that's purely for HTTP, it brings a lot of irrelevant baggage along with it .. a general purpose URI class != a HTTP URL class.
I'm happy for someone to add To/From URL extension methods, or even implicit casting if they're feeling brave, but don't come crying when it blows up at runtime randomly ;-)

I was a bit baffled by the custom Url class when I first stumbled upon it, especially considering that it didn't override the ToString(), so it couldn't be used as a regular string and it had no implicit nor explicit cast overloads to neither String nor Uri. This makes the Url class a real PITA to use.
Would you accept pull requests to improve on the Url class' interoperability with String and Uri? @grumpydev Exactly what would blow up randomly at runtime?

Hm, I see now that #393 was fixed for the ToString() implementation, but it doesn't seem to be on NuGet yet. When will a NuGet package of Nancy be released with #393 included?


[Bug] Unable to successfully build doris on M1

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
I am currently building the master branch of Doris.

My OS is macOS Ventura 13.1
Clang Version：Homebrew clang version 15.0.6，Target: arm64-apple-darwin22.2.0

What's Wrong?
After I built the entire project, I found that the brpc-1.2.0 package was wrong during linking.

What You Expected?

Successfully build Doris

How to Reproduce?

bash build.sh
or
bash build.sh --be

Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Bugfixed via this solution. Thanks
cd thirdparty
rm -rf installed
curl -L https://github.com/apache/doris-thirdparty/releases/download/automation/doris-thirdparty-prebuilt-darwin-arm64.tar.xz  -o - | tar -xf -


CYF London| Sajad Soornisofla | Form controls | Week2

Learners, PR Template
Self checklist

[x] I have committed my files one by one, on purpose, and for a reason
[x] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[x] I have tested my changes
[x] My changes follow the style guide
[x] My changes meet the requirements of this task

Changelist
This PR adds a simple product pick form with the following features:
Form Fields:
User inputs for name, email, color selection (via radio buttons), size (via dropdown), and delivery date (via date picker).
Validation:
Fields for name, email, and color selection are marked as required, ensuring users fill them out before submitting.
Email input is properly validated with HTML5 type validation.
Date Selection:
Added a JavaScript date picker that restricts the delivery date to a range from today to 28 days in the future.
Questions
Could you please check if the validation and functionality work as expected for the required attribute on the color selection? The only way to satisfy the requirement is to select one of the options by default. Also, let me know if there are any improvements I can make regarding user experience or code structure.
Thank you for your time in reviewing my PR!

well done @sajadsoorni  !!
You can see your work deployed on a PR preview here
https://deploy-preview-205--cyf-ufd.netlify.app/form-controls/

well done @sajadsoorni  !!
You can see your work deployed on a PR preview here
https://deploy-preview-205--cyf-ufd.netlify.app/form-controls/

Thank you, @ShayanMahnam! I appreciate you taking the time to review my project.🙏


fix(android): FlatList with TextInput resize bug

Window SOFT_INPUT_ADJUST_RESIZE doesn't play well with FlatList with a TextInput.

Added keyboardMode prop to mitigate bug when using FlatList with a TextInput
Fixes #41

:rocket: This pull request is included in v0.12.1. See Release 0.12.1 for release notes.


https://docs.microsoft.com/en-us/education/windows/windows-11-se-overview#add-your-own-apps

[Enter feedback here]

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

ID: e2ca5807-f011-cdd1-eb80-c74655aa4738
Version Independent ID: fe016307-69a4-ec63-4979-eac841d13c3c
Content: Use Set up School PCs app
Content Source: education/windows/use-set-up-school-pcs-app.md
Product: windows
Technology: windows
GitHub Login: @paolomatarazzo
Microsoft Alias: paoloma

#10850

addressed as part of 10850


navigation2: 1.0.2-1 in 'galactic/distribution.yaml' [bloom]

Increasing version of package(s) in repository navigation2 to 1.0.2-1:

upstream repository: https://github.com/ros-planning/navigation2.git
release repository: https://github.com/SteveMacenski/navigation2-release.git
distro file: galactic/distribution.yaml
bloom version: 0.10.7
previous version for package: 1.0.0-2

Fixes build issues + cherry picks some patches recently submitted


Feat/predict pdp discount

Objective
What: Add a dynamic discount for FBT if recommendations are over buying power or if current cart total is over buying power
Why: User likely needs an incentive to buy if the above conditions are true
How: Predict zone to show the discount message and predict hook to get buying power to update the price
Usage: Search for "bag", click first result (5VA54H00I), look at FBT to see no discount, check logs to know why. Add something about 75 pounds to the cart, then look for that bag again. This time, show the discount message, check the logs to know why.
Type

[X] New Feature

Tested
Localhost
Translation

[ ] Have you translated all your Paragraphs, ButtonsTitle,... with i18n functionality?

Documented

[ ] Have you documented in changed file using comments
[ ] Have you added instructions to the README if needed?

It's really nice good job @matthew1809 but if there are not Related or not FBT, maybe I don't understand predict correctly but in this case I have the promo code but without any products

Great catch! To test, please try it on the first result for "bag" and let me know what you see if you cart is empty, vs if you have more than 80 in your cart already

https://share.vidyard.com/watch/ykdL4ZJtdPRdUgkMU8sqX9


Replace references to tables to relations. Add include param to union

Started off by adding an include parameter to union. But then when I was updating the docs I decided that having this macro named union_tables is silly so did some work to tidy that up. And then I did the same for get_tables_by_prefix and unpivot. Pretty happy with this pattern!
(This is probably the way we should have done this when we upgraded to 0.14.0, but ah well! This provides us a path to deprecate some macros though!)
I also tidied up some docs along the way.
Deprecating the get_tables_by_prefix and union_tables macros:
Here's what happens when you use the old macros:
$ dbt run -m test_get_tables_by_prefix_and_union
Running with dbt=0.15.0
Found 35 models, 53 tests, 0 snapshots, 0 analyses, 260 macros, 0 operations, 43 seed files, 0 sources

14:22:57 | Concurrency: 1 threads (target='redshift')
14:22:57 |
14:22:57 | 1 of 1 START table model integration_tests_dev.test_get_tables_by_prefix_and_union [RUN]
Warning: the `get_tables_by_prefix` macro is no longer supported and will be deprecated in a future release of dbt-utils. Use the `get_relations_by_prefix` macro instead
Warning: the `union_tables` macro is no longer supported and will be deprecated in a future release of dbt-utils. Use the `union_relations` macro instead
14:22:58 | 1 of 1 OK created table model integration_tests_dev.test_get_tables_by_prefix_and_union [SELECT in 1.20s]
14:22:59 |
14:22:59 | Finished running 1 table model in 2.81s.

Completed successfully

Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

These warnings don't show up when you use the relation versions of these macros:

Warning: the union_tables macro is no longer supported and will be deprecated in a future release of dbt-utils. Use the union_relations macro instead

Replacing the table arg with relation in unpivot:
This one was a little tricker to implement, but it works! LMK if you want me to go through this.

Yeah I saw that PR from Jake --  my mind was 🤯at how few lines of code were involved to implement that! I'll set a reminder to myself to change this later :)


riak-cs: add to nixpkgs

Motivation for this change
Needed to add in specific Riak CS bindings when dealing with the Riak CS package from Basho. Coordinated with this merge request: https://github.com/NixOS/nixpkgs/pull/20128 .
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

cc @cstrahan

I run into some issues with this, I think the build is impure.
fatal: Not a git repository (or any of the parent directories): .git
--2016-11-18 18:40:18--  https://raw.githubusercontent.com/basho/bitcask/develop/priv/scripts/downgrade_bitcask.erl
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... failed: Name or service not known.
wget: unable to resolve host address 'raw.githubusercontent.com'
make: *** [Makefile:29: riak_test/src/downgrade_bitcask.erl] Error 4
builder for ‘/nix/store/5ls6m2p7vv2m3hw8sy5shqysdxabqpiv-riak_cs-2.1.1.drv’ failed with exit code 2
error: build of ‘/nix/store/5ls6m2p7vv2m3hw8sy5shqysdxabqpiv-riak_cs-2.1.1.drv’ failed

@LnL7 it looks like Travis is failing pull requests because of something to do with riak_cs. Here's an example from #20593.
The particular error is Package ‘riak_cs-2.1.1’ in ‘/home/travis/build/NixOS/nixpkgs/pkgs/servers/nosql/riak-cs/2.1.1.nix:62’ is not supported on ‘i686-linux’, refusing to evaluate.
Sorry to make more work for you, but could you try to figure out why this is happening?
I'm going to go ahead and revert this since it's making Travis angry; please push again once you've got it working. If I'm mistaken about this somehow, then my apologies; please revert my revert in that case :-)
I suppose you can reproduce the build error by running the command in the Travis log on an i686 machine: ./maintainers/scripts/travis-nox-review-pr.sh nixpkgs-verify nixpkgs-manual nixpkgs-tarball

@mdaiter


Port Mail as s-c-s sink

As a developer, I'd like to move mail module from XD to s-c-s repo, so I can use it as sink to build streaming pipeline.
Acceptance:

mail as sink module is available in spring-cloud-stream-modules repo
maven repo includes mail module with respective group/artifact/version detail

This issue was moved to spring-cloud/spring-cloud-stream-app-starters#53


Enable us the command-line mode

Please add a feature to enable us to use : to enter the command-line mode of vim. So we can use the command as :s, :w, :x, .etc by vim-mode.
Thanks a lot!

Check out vim-mode-plus and ex-mode.


Named exports

Same a #4 (kinda) and a fix for #9
This code exports both a default and named exports

This is awesome, will try to look at it and get it merged tonight.  If it works well I think we'll apply the same technique to Preact.

updates?

I need to merge just the src changes here, the build stuff will break commonjs exports.

see PR #15

Fixed in 2.0.2.


Issue with Scope Checks for Tenant and Sub-Organization Levels in Frontend

Describe the issue:
We're encountering an issue in the frontend regarding scope checks at both tenant and sub-organization levels. The current implementation in deployment.config.json does not effectively differentiate between scopes that are specific to tenants or sub-organizations.
Eg:
"applications": {
    "disabledFeatures": [],
    "enabled": true,
    "scopes": {
        "feature": ["console:applications"],
        "create": ["internal_application_mgt_create"],
        "read": [
            "internal_cors_origins_view",
            "internal_application_mgt_view",
            "internal_claim_meta_view",
            "internal_role_mgt_view",
            "internal_userstore_view",
            "internal_idp_view"
        ],
        "update": ["internal_application_mgt_update"],
        "delete": ["internal_application_mgt_delete"]
    }
}

In the current setup, for sub-organizations, the frontend appends _org to the internal scopes (e.g., internal_org_application_mgt_view). However, some scopes are either tenant-specific or sub-organization-specific.
For instance, the scope internal_org_cors_origin_view does not exist, leading to issues such as the application view being hidden in sub-organizations due to checks for a non-existing scope.
Expected behavior:
The FE should have a clear distinction and handling mechanism for scopes that are specific to tenants or sub-organizations.
Suggested Solution:
A potential solution could be to establish separate scope definitions for sub-organizations and modify the scope-checking logic to recognize and handle these correctly. This would involve both backend and frontend adjustments to manage these scope distinctions properly.
Environment information (Please complete the following information; remove  any unnecessary fields) :

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

Optional Fields
Related issues:

Suggested labels:

Temporary Solution for Sub-Organization Role Tab Visibility Issue

https://github.com/wso2/identity-apps/pull/5308
https://github.com/wso2/carbon-identity-framework/pull/5410

These removed scopes should be reinstated when a distinct scope checking mechanism is introduced for both tenant and sub-organizations.

The updated scopes under the temporary solution are tracked in the following sheet:
https://docs.google.com/spreadsheets/d/1N1hcwxxlpcAYQr2YPvTVgI_mNXGomKpGGB76EIVGp_M/edit#gid=0

Further, there was a customer requirement where sub-organization users should have the option to invite the parent users but not the ability for direct user creation. To cater to such requirements, we will need sub-feature-wise scope check capabilities. In my opinion, since we have to put considerable effort into separating sub-organization and tenant scopes, it would be better to consider this capability as well.


Remove ipxe-operator image

What would you like to be added:
I guess we can remove the old ipxe-operator image from the artifacts repository.

Pxe-operator OCI images have been removed.


Chapter 9 - Why use type abbreviation for 'Total' and 'DiscountPercentage'?

Hi Ian,
After you mentioning the problems with type abbreviations compared to (private) singe-case discriminated unions, I wonder why you chose type abbreviations for Spend and DiscountPercentage on page 128:
type Total = decimal
type DiscountPercentage = decimal

Was this merely to keep the book code a bit shorter, or is there something else I should keep in mind?
Still enjoying your book!
Cheers,
Stefan

It was mainly for brevity but there wasn't any obvious benefit in these cases (function outputs) from using something more complex. Converting them into single-case DUs is relatively easy should you feel the urge to do so.

Thanks, that makes sense. I converted Total to a single-case DU, but kept DiscountPercentage as a type abbreviation, as in the example code it's only used internally.


For release 0.3.3

I found a typo in Recipe task 2 where the sample rate for the enhanced signal is passed incorrectly.
This originated when we reduced the sample rate of the submission to 32000 Hz

OK. I've approved it.
On Thu, 18 May 2023 at 12:34, Gerardo Roa Dabike @.***>
wrote:

@groadabike https://github.com/groadabike requested your review on: #325
https://github.com/claritychallenge/clarity/pull/325 For release 0.3.3.
—
Reply to this email directly, view it on GitHub
https://github.com/claritychallenge/clarity/pull/325#event-9278812522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACO7SJGKZ3JMCIZZJSFGFFDXGYCNZANCNFSM6AAAAAAYGIKWQ4
.
You are receiving this because your review was requested.Message ID:
@.***>

--
Professor Jon Barker,
Department of Computer Science,
University of Sheffield
+44 (0)<PHONE_NUMBER>


Add support for team repositories endpoint

closes #202

The failing build seems to be network hiccups. Not sure about the failing TLS installation though. Any hints?

I "accidentally" fixed a few more docstrings in this commit now. Does not do any harm, however I decided not to fix them all, just so we don't have all those unrelated changes in this commit & PR.

Thanks!


timezoneId doesn't work

Hi!
I'm using aldeed:moment-timezone and trying to set timezoneId up on my schema, as bellow:
afFieldInput: {
      type: 'bootstrap-datetimepicker',
      timezoneId: 'America/Sao_Paulo',
    },

But my datetime-picker locale doesn't change.

fix it!
I used the dateTimePickerOptions attribute to set up the language, like this:
afFieldInput: {
     type: 'bootstrap-datetimepicker',
     timezoneId: 'America/Sao_Paulo',
     dateTimePickerOptions: {
     language: 'pt-BR',
     },
}


PageCache and ErrorAction

I am experiencing a problem with PageCache when there is an Error on the page.
I have implemented PageCache filter for my actions in behaviors function of controller, it works fine and contents are served from cache for specified duration.
'pageCache' => [
    'class' => 'yii\filters\PageCache',
    'only' => ['display-all'],
    'duration' => 1800,
    'variations' => [Yii::$app->request->url, 'country-code'],
]

````
But when I have some server error on the page and view from `ErrorAction` is rendered, first time I see the error page but after that I get blank page with 500 status code in response until the cache is expired.

So is there any way to avoid `PageCache` from caching the page when there is an server error on page?

500 status code means error. Can you get from logs what's happened?

@samdark  Are you asking about why the error happened or why im getting cached contents?
Regarding error I know why it happened and I fixed it, but since the contents were I cached i was still getting 500 response, even though the error was fixed. I reproduced it by intentionally introducing the error and then enabling the cache, but later i fixed the error but i was still getting cached contents.
So the main point is that I want to avoid caching contents when there is an server error. otherwise in production user can see the error for 30 minutes for example even though the error was fixed after 1 minute. but since the page was cached with error, user dont see the actual contents anymore.

I'm asking for the entry in logs for that 500 page that definitely should be there.

Ah.

@tahirm Could you test from master or apply patch #8613 manually? I think it should fix your problem.

#8614

Thanks for fixing it guys. Sorry for late response. it works now.


jenkins-plugin-cli does not use http_proxy/https_proxy environment variables

Hi,
I am building own jenkins images with preinstalled plugins. In my Dockerfile, I used to execute the install-plugins.sh shellscript like this:
COPY ./plugins.txt /usr/share/jenkins/ref/plugins.txt
RUN /usr/local/bin/install-plugins.sh < /usr/share/jenkins/ref/plugins.txt

This works perfectly, but since some weeks, I warns me because of deprecation:
WARN: install-plugins.sh is deprecated, please switch to jenkins-plugin-cli

This is why I wanted to switch to jenkins-plugin-cli today, so I changed my Dockerfile:
COPY ./plugins.txt /usr/share/jenkins/ref/plugins.txt
RUN jenkins-plugin-cli --verbose -f /usr/share/jenkins/ref/plugins.txt

But it does not work, but throws the following error:
Reading in plugins from /usr/share/jenkins/ref/plugins.txt

No directory to download plugins entered. Will use default of /usr/share/jenkins/ref/plugins
Using update center https://updates.jenkins.io/update-center.json from JENKINS_UC environment variable
Using experimental update center https://updates.jenkins.io/experimental/update-center.json from JENKINS_UC_EXPERIMENTAL environment variable
Using incrementals mirror https://repo.jenkins-ci.org/incrementals from JENKINS_INCREMENTALS_REPO_MIRROR environment variable
No CLI option or environment variable set for plugin info, using default of https://updates.jenkins.io/plugin-versions.json
No war entered. Will use default of /usr/share/jenkins/jenkins.war
Jenkins version: 2.263

Retrieving update center information
Created cache at: /root/.cache/jenkins-plugin-management-cli
Cache miss for: update-center-2.263
io.jenkins.tools.pluginmanager.impl.UpdateCenterInfoRetrievalException: Error getting update center json
        at io.jenkins.tools.pluginmanager.impl.PluginManager.getJson(PluginManager.java:573)
        at io.jenkins.tools.pluginmanager.impl.PluginManager.getUCJson(PluginManager.java:592)
        at io.jenkins.tools.pluginmanager.impl.PluginManager.start(PluginManager.java:141)
        at io.jenkins.tools.pluginmanager.impl.PluginManager.start(PluginManager.java:113)
        at io.jenkins.tools.pluginmanager.cli.Main.main(Main.java:63)
Caused by: java.net.ConnectException: Connection timed out (Connection timed out)
        at java.net.PlainSocketImpl.socketConnect(Native Method)
        at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
        at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
        at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
        at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
        at java.net.Socket.connect(Socket.java:607)
        at sun.security.ssl.SSLSocketImpl.connect(SSLSocketImpl.java:666)
        at sun.security.ssl.BaseSSLSocketImpl.connect(BaseSSLSocketImpl.java:173)
        at sun.net.NetworkClient.doConnect(NetworkClient.java:180)
        at sun.net.www.http.HttpClient.openServer(HttpClient.java:463)
        at sun.net.www.http.HttpClient.openServer(HttpClient.java:558)
        at sun.net.www.protocol.https.HttpsClient.<init>(HttpsClient.java:264)
        at sun.net.www.protocol.https.HttpsClient.New(HttpsClient.java:367)
        at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.getNewHttpClient(AbstractDelegateHttpsURLConnection.java:191)
        at sun.net.www.protocol.http.HttpURLConnection.plainConnect0(HttpURLConnection.java:1162)
        at sun.net.www.protocol.http.HttpURLConnection.plainConnect(HttpURLConnection.java:1056)
        at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(AbstractDelegateHttpsURLConnection.java:177)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1570)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1498)
        at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:268)
        at java.net.URL.openStream(URL.java:1068)
        at org.apache.commons.io.IOUtils.toString(IOUtils.java:2795)
        at io.jenkins.tools.pluginmanager.impl.PluginManager.getJson(PluginManager.java:567)
        ... 4 more
Error getting update center json

I do not see any requests go to my proxy, so I guess that the jenkins-plugin-cli simply does not care about my http_proxy/https_proxy environment variables (which are used by the old, deprecated script).
Do I have to configure the proxy in another way for the jenkins-plugin-cli?
I checked jenkins-plugin-cli --help but there is no option to set a proxy.
Kind regards
Timo

Hi,
I found the following in the documentation from the plugin-installation-manager:
https://github.com/jenkinsci/plugin-installation-manager-tool#proxy-support
But how do I do this when using jenkins-plugin-cli in a docker file?
Kind regards (and sorry for the stupid question)
Timo

You cannot use the script but need to run the java command directly like this:
 java -Dhttps.proxyHost=${HTTP_PROXY_HOST} -Dhttps.proxyPort=${HTTP_PROXY_PORT} -jar /usr/lib/jenkins-plugin-manager.jar -f /usr/share/jenkins/ref/plugins.txt

And yes, I also don't like the current behavior...

Since this is an issue with the plugin-installation-manager-tool, I'm going to close this out.

service docker restart solved the problem.

service docker restart solved the problem.

I ran it with verbose:
jenkins-plugin-cli --war "/usr/share/jenkins/jenkins.war" --plugin-file "/var/jenkins_home/plugins.txt" --verbose

I concluded that i need to set some env vars . i did :
export JENKINS_UC=http://nexus.cicd/repository/updates.jenkins.io/update-center.json
export JENKINS_UC_EXPERIMENTAL=http://nexus.cicd/repository/updates.jenkins.io/experimental/update-center.json
export JENKINS_PLUGIN_INFO=http://nexus.cicd/repository/updates.jenkins.io/plugin-versions.json
export JENKINS_INCREMENTALS_REPO_MIRROR=http://nexus.cicd/repository/repo.jenkins-ci.org/incrementals

Agree or disagree! but it works with me.
In helm release, just add these env vars for the init container in initContainers

I ran it with verbose:
jenkins-plugin-cli --war "/usr/share/jenkins/jenkins.war" --plugin-file "/var/jenkins_home/plugins.txt" --verbose

I concluded that i need to set some env vars . i did :
export JENKINS_UC=http://nexus.cicd/repository/updates.jenkins.io/update-center.json
export JENKINS_UC_EXPERIMENTAL=http://nexus.cicd/repository/updates.jenkins.io/experimental/update-center.json
export JENKINS_PLUGIN_INFO=http://nexus.cicd/repository/updates.jenkins.io/plugin-versions.json
export JENKINS_INCREMENTALS_REPO_MIRROR=http://nexus.cicd/repository/repo.jenkins-ci.org/incrementals

Agree or disagree! but it works with me.
In helm release, just add these env vars for the init container in initContainers

You can set an extra environment variable

name: JAVA_OPTS
value: "-Dhttp.proxyHost=myproxy -Dhttp.proxyPort=3128 -Dhttps.proxyHost=myproxy -Dhttps.proxyPort=3128"

For me worked following:

declare ARG _JAVA_OPTIONS inside of your dockerfile
pass "-Dhttps.proxyHost=myproxy -Dhttps.proxyPort=myport" as --build-arg
I could see that the jenkins-plugin-cli picked up the _JAVA_OPTIONS.  This way you can set your proxy only during the build phase, which might be helpful for others, too.


[Astro] Support astro:env in astro-sst adapter

The build succeeds but throws an error if Astro experimental.env is used:
20:54:36 [ERROR] [config] The feature "astro:env getSecret" is not supported (used by astro-sst).
20:54:36 [ERROR] The adapter astro-sst doesn't support the feature envGetSecret. Your project won't be built. You should not use it.

Despite the error, the build creates some output.
I would assume the astro-sst adapter to use the node adapter internally and expect adding astro:env support to be rather simple. I wonder if additional steps are even required or the error just stems from the key supportedAstroFeatures.envGetSecret = 'experimental' not being present on the adapter.

Closing this and Reopening in Ion

Reopening. Despite the error, env seems to work, except for middleware.

Would appreciate a PR!

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble