podman not opening port

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
The port is supposed to be routed to the host, but is not. See the output of podman ps. Port 10080(it is in a different docker-compose/pod) works but 10000 and 18081(both in the same docker-compose/pod) does not, there's no error message or similiar.
Sometimes it works though, I'm not sure what triggers it that it works sometimes.
If I use a bash in the container the port 3000 is open and it returns stuff as its supposed to, so seems to be a mapping bug? In my container I'm listening on *:3000.
lsof -i -P -n | grep LISTEN:
exe     2204  jc2    9u  IPv4  30349      0t0  TCP <IP_ADDRESS>:10080 (LISTEN)
podman ps:
[jc2@v220190910461598152 backend]$ podman ps
CONTAINER ID  IMAGE                                   COMMAND               CREATED         STATUS             PORTS
                                 NAMES
0be12725dfb4  docker.io/library/mongo-express:latest  mongo-express         5 minutes ago   Up 5 minutes ago   <IP_ADDRESS>:10000->3000/tcp, <IP_ADDRESS>:18081->8081/tcp  backend_mongo-express_1
6596ecacfe98  docker.io/library/mongo:latest          mongod                5 minutes ago   Up 5 minutes ago   <IP_ADDRESS>:10000->3000/tcp, <IP_ADDRESS>:18081->8081/tcp  mongoJC2
c282bfe1789a  docker.io/library/node:latest           bash -c cp /usr/s...  5 minutes ago   Up 5 minutes ago   <IP_ADDRESS>:10000->3000/tcp, <IP_ADDRESS>:18081->8081/tcp  backend_node_1
94a666479072  docker.io/library/nginx:alpine          nginx -g daemon o...  40 minutes ago  Up 40 minutes ago  <IP_ADDRESS>:10080->80/tcp                             nginx_web_1

Steps to reproduce the issue:

For me start the corresponding docker file

Describe the results you received:
Port is not available/open
Describe the results you expected:
Port is opened and listening
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Version:            1.8.2
RemoteAPI Version:  1
Go Version:         go1.13.6
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.13.6
  podman version: 1.8.2
host:
  BuildahVersion: 1.14.3
  CgroupVersion: v2
  Conmon:
    package: conmon-2.0.14-1.fc31.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.14, commit: 083a0be12178013d44ff51ceda3090ea741b6516'
  Distribution:
    distribution: fedora
    version: "31"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1006
      size: 1
    - container_id: 1
      host_id: 231072
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1006
      size: 1
    - container_id: 1
      host_id: 231072
      size: 65536
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    name: crun
    package: crun-0.13-1.fc31.x86_64
    path: /usr/bin/crun
    version: |-
      crun version 0.13
      commit: e79e4de4ac16da0ce48777afb72c6241de870525
      spec: 1.0.0
      +SYSTEMD +SELINUX +APPARMOR +CAP +SECCOMP +EBPF +YAJL
  SwapFree: 0
  SwapTotal: 0
  arch: amd64
  cpus: 2
  eventlogger: journald
  hostname: v220190910461598152
  kernel: 5.5.15-200.fc31.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.0-20.1.dev.gitbbd6f25.fc31.x86_64
    Version: |-
      slirp4netns version 0.4.0-beta.3+dev
      commit: bbd6f25c70d5db2a1cd3bfb0416a8db99a75ed7e
  uptime: 47m 12.41s
registries:
  search:
  - docker.io
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - quay.io
store:
  ConfigFile: /home/jc2/.config/containers/storage.conf
  ContainerStore:
    number: 6
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.7.8-1.fc31.x86_64
      Version: |-
        fusermount3 version: 3.6.2
        fuse-overlayfs: version 0.7.8
        FUSE library version 3.6.2
        using FUSE kernel interface version 7.29
  GraphRoot: /home/jc2/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 7
  RunRoot: /run/user/1006/containers
  VolumePath: /home/jc2/.local/share/containers/storage/volumes

Addiotional information
I'm not sure if this is related, but when exiting the bash in the container I get this message:
Error: non zero exit code: 127: OCI runtime error
ERRO[0203] unable to close namespace: "close /proc/39186/ns/user: bad file descriptor"

I noticed something else, my system reports the ports being opened for a few seconds, then they close again.

This is related/the same issue: https://github.com/containers/podman-compose/issues/107

There were different fixes with rootlessport after 1.8.2, would it be possible to build podman from git and try with it?

Is there a container which can do the build? I'm not really keen to install all the dependencies on my server/desktop
And is there a how to build file? I did not find any

Dockerfile.fedora has the dependencies you need.
You could use it to create an image with all the build tools, and then create a container with a volume and do the go build within the container.

Same problem here with fedora 32 (podman 1.8.2). Port is not accessible on host system in rootless mode. (It works in root mode.)

@giuseppe:
I have compiled podman (libpod) on my own on fedora 32. The issue is solved on master d6b3bc18f85236aa.
However, I'm unable to access mounted files in the container (like --mount type=bind,source=/mnt/home/aanno/scm/scm/docker-compose-files/config/solr/libs/ojdbc8-<IP_ADDRESS>.jar,destination=/opt/solr/server/lib/ext/ojdbc8-<IP_ADDRESS>.jar,ro). Is this a known issue (e.g. still #3683 or similiar) or should I report that?

However, I'm unable to access mounted files in the container (like --mount type=bind,source=/mnt/home/aanno/scm/scm/docker-compose-files/config/solr/libs/ojdbc8-<IP_ADDRESS>.jar,destination=/opt/solr/server/lib/ext/ojdbc8-<IP_ADDRESS>.jar,ro). Is this a known issue (e.g. still #3683 or similiar) or should I report that?

what error are you seeing?  Also, who owns that file?

I was able to fix the issue with
$ chcon -Rt svirt_sandbox_file_t /mnt/home/aanno/scm/scm/docker-compose-files/config/solr/*

like mentioned in #3683. Hence it is SELinux related. However, I think that should be at least be in the troubleshooting guide...

$ ls -Zl  /mnt/home/tpasch/scm/scm/docker-compose-files/config/solr
insgesamt 12
drwxrwxr-x. 3 aanno aanno unconfined_u:object_r:container_file_t:s0    19 15. Apr 17:28 collections
-rw-rw-r--. 1 aanno aanno unconfined_u:object_r:container_file_t:s0 11402 31. Mär 17:32 jetty.xml
drwxrwxr-x. 2 aanno aanno unconfined_u:object_r:container_file_t:s0   176 15. Apr 17:28 libs

I think that should be at least be in the troubleshooting guide...

would you like to open a PR to describe the issue and the solution you've found?

Well, I commented on PR #5839. If still needed afterwards, I will consider the PR.

@aanno Any movement on this?

I am going to close this, since it seems to be dead now.  Reopen if I am mistaken.


Cannot Open Realsense Viewer

librealsense
2.43.0 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Program terminated due to an unrecoverable SEH exception:
Illegal CPU instruction!
Possibly newer CPU architecture is required

Hi @Albthomas  This message indicates that the age of the processor in your computer makes it incompatible with the RealSense Viewer tool that is bundled in the RealSense SDK.  This Viewer seeks an Intel Core 6th Generation processor or newer.  Your E5-2620 processor is 2nd Generation.
You can download a standalone version of the Viewer from the Assets file list at the link below though that does not have this compatibility check.
https://github.com/IntelRealSense/librealsense/releases/

Please right-click on the Intel.RealSense.Viewer.exe link in the list and select the file download option from the menu (it may have a name such as Save Link As or Save Target As).
When the download has completed, go to the folder location that you downloaded the file to and double-leftclick on the file to launch the Viewer.  Do not use the launch shortcut on your Windows desktop, as this leads to the version with the compatibility check.
If you need a shortcut to the downloaded Viewer to place on your desktop, delete the original shortcut and make a new one by right-clicking on the Intel.RealSense.Viewer.exe file and selecting the Create shortcut menu option.

Case closed due to correct answer provided and no further comments received.


Imagenet example failed to evaluate the model

I couldn't run imagenet example. It failed to evaluate the model.
Traceback (most recent call last):
  File "examples/imagenet/train_imagenet.py", line 166, in <module>
    main()
  File "examples/imagenet/train_imagenet.py", line 162, in main
    trainer.run()
  File "/home/unno/git/chainer/chainer/training/trainer.py", line 269, in run
    entry.extension(self)
  File "/home/unno/git/chainer/chainer/training/extensions/evaluator.py", line 134, in __call__
    result = self.evaluate()
  File "/home/unno/git/chainer/chainer/training/extensions/evaluator.py", line 169, in evaluate
    eval_func(*in_vars)
  File "/home/unno/git/chainer/examples/imagenet/nin.py", line 29, in __call__
    h = F.max_pooling_2d(F.relu(self.mlpconv1(x)), 3, stride=2)
  File "/home/unno/git/chainer/chainer/links/connection/mlp_convolution_2d.py", line 73, in __call__
    x = f(l(x))
  File "/home/unno/git/chainer/chainer/links/connection/convolution_2d.py", line 101, in __call__
    x, self.W, self.b, self.stride, self.pad, self.use_cudnn)
AttributeError: 'Convolution2D' object has no attribute 'W'

I think it is caused by Link.copy not handling the uninitialized parameters. The image net example currently copies the model before running the training to run evaluations in test mode. In order to fix it, we have to make Link.copy also copy the information of uninitialized parameters (_uninitialized_params) and catch up with the actual initialization of these variables. One solution is to add a reference that points to the original link as an attribute of the copied link, and use this link to check in add_params whether the original parameter is already initialized, and if so use a shallow copy of the initialized parameter instead of initializing a new one.

This problem seems to be essentially same as #1615.

How about to abstract parameters to have the states of initialized and uninitialized?
original_link - W attribute -----> parameter (initialized or uninitialized)
                              |
copied_link - W attribute ----+

Alghtough this change may have effect to the internal implementation of links that already support uninitialized parameters, we should be able to keep external APIs of Link and such links.
To hide the extra abstraction layer when accessing link attributes, e.g. link.W and link.W = some_array, we may need to implement __getattr__ and __setattr__ methods to Link.

Ah-, I find that shared is only Variable's data and parameter Variables are independent, it is more complicated than what I wrote.

I note a possible design of Variable's uninitialized state:
Link -> Variable ---> VariableData -> xp.ndarray or None
                  |
Link -> Variable -+

Here I add VariableData abstraction, through which Variable indirectly has parameter array. VariableData actually has initialized/uninitialized state. If it is initialized, it has xp.ndarray. If it is uninitialized, it has, for example, None as its uninitialized representation.
VariableData make Variable possible to have uninitialized state as well as to share parameter array with other Variables. VariableData would be hidden from users and Variable would be able to transparently access parameter array.
Link and Variable would have shared_copy method to copy them with sharing VariableData, copy which is deeper than shallow copy and shallower than deep copy.


Colon emojis

Hi, I would like to know if there is already a feature to use emojis with :colon: or if there is good advice as how to integrate this.
thanks

This isn't supported in Emoji Button, unfortunately. The emoji data from Unicode that it parses does not include the colon short codes. There is a package, node-emoji (https://www.npmjs.com/package/node-emoji), which maps these short codes to emoji characters and back again, but I'm not sure if it works in the browser.
I probably won't add support for this because it would involve a lot of data about the short code mappings, and this library is already pretty large with the basic emoji data 😄
Hope this helps.


Unify logging + Add tag and time

https://github.com/orgs/vizzuhq/projects/10/views/1?pane=issue&itemId=32331162

I created a new commit but it didn't showed here:
https://github.com/vizzuhq/vizzu-lib/commit/efcad31b722ba1c36ffa299911d962ed237662fd
I create a new PR


React + Server side render + Server respond before api response updated to reducer when calling API from redux-saga

I have implemented server side rendering for my application with expressJS, react and redux. My server respond before API result are updated to the reducer. I'm making API call from redux-saga. I added the console log after API success and able to see the API results. After that, triggered the dispatch action to update the value into reducer. But express server respond before value get updated into reducer. It is working fine If I add some timeout like 2000ms to wait on server side.
/**

Server.js
*/
let preLoadedState = {
context: {
cookie: req.cookies,
},
};
  const store = storeFactory(preLoadedState, rootSaga);

  ReactDOMServer.renderToStaticMarkup(<Provider store={store} ><Router /></Provider>);
  const tasks = store.tasks;
  store.dispatch({type: 'CHANGE_STATE_CONTEXT_SHOULD_RENDER', payload: true});
  store.dispatch(END);

  Promise.all(tasks.map(task => task.done)).then(() => {
      const content = ReactDOMServer.renderToString(<Provider store={store} ><Router /></Provider>);
      const finalState = store.getState();
      console.log('************ final store', finalState);
      res.send(data.replace('

',
          `<div id="root">`+
          `${content}`+
          `</div>`+
          `<script> window.__PRELOADED_STATE__ = ${JSON.stringify(finalState).replace(/</g, '\\\u003c')}</script>`
      ));
  });

/**

storeFactory.js
*/
const storeFactory = (initialState, rootSaga) => {
const sagaMiddleware = createSagaMiddleware();
const store = createStore(rootReducer, initialState, composeWithDevTools(applyMiddleware(sagaMiddleware)));
store.tasks = rootSaga ? [sagaMiddleware.run(rootSaga)] : [];
return store;
};

/**

Saga.js
/
export function getExpenseList(action = {}) {
try {
const response = yield call(FactorySaga, ExpenseApi.getExpense, action.param);
const { expenseList, storeList, categoryList } = response.result.data;
console.log('********* getExpenseList saga******** expenseList');
yield putResolve(initExpenseList(expenseList, storeList, categoryList));
} catch (e) { console.log(e)}
}

I am able to getExpenseList saga logs in server console before server response. But reducer is got triggered after server respond.

Closing due to inactivity.  Please feel free to reply if you are still having this issue and I’d be happy to help debug.


error when projecting null features

when selecting features that contain null values, there is an error in the console, but no message (for the user it seems like it is still initializing the process)


Do you have to package your app a certain way for this to work?

I can't seem to get this to work. Do you have to package your app a certain way for this to work?
Is anything else besides this code needed?
Thank you.

Not especially; can you post your code?
Does your init function/callback receive any data?

I can't even drag anything into the Dock icon. With other apps, the app icon dims, and a tooltip appears. With my app, there's no roll over state, and it won't let me drop the file on there.
var electron = require("electron");
var Menu = electron.Menu;
var app = electron.app;
var menu = require("./menu");
var windows = require("./windows");

// Attempt to bind file opening #1
var fileOpener = require('./electron-file-opener.js');
fileOpener.init(function(filePaths) {
  console.log(filePaths);
  windows.create(undefined, filePaths);
});

// Attempt to bind file opening #2
app.on('will-finish-launching', () => {
  // Event fired When someone drags files onto the icon while your app is running
  app.on("open-file", (event, file) => {
    windows.create(undefined, file);
    event.preventDefault()
  });
});

Do you actually have to build the app, or will electron .  work?

Ahh gotcha. It's been a minute since I was messing around with Electron stuff, but I do believe you need to tell your Electron app which kinds of files can be opened by it. From what I recall, that's how it knows to have the Dock icon dim and make those files selectable.
I'm using Electron Forge to build my app, and within the config file for my app, I define an extendInfo plist file which tells Electron that any images can be opened using my app.
My Info.plist file looks as follows:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
  <key>CFBundleDocumentTypes</key>
  <array>
    <dict>
      <key>CFBundleTypeExtensions</key>
      <array>
        <string>png</string>
      </array>
      <key>CFBundleTypeIconFile</key>
      <string>electron.icns</string>
      <key>CFBundleTypeMIMETypes</key>
      <array>
        <string>image/png</string>
      </array>
      <key>CFBundleTypeName</key>
      <string>PNG image</string>
      <key>CFBundleTypeOSTypes</key>
      <array>
        <string>PNGf</string>
      </array>
      <key>CFBundleTypeRole</key>
      <string>Editor</string>
    </dict>
    <dict>
      <key>CFBundleTypeExtensions</key>
      <array>
        <string>jpg</string>
        <string>jpeg</string>
      </array>
      <key>CFBundleTypeIconFile</key>
      <string>electron.icns</string>
      <key>CFBundleTypeMIMETypes</key>
      <array>
        <string>image/jpeg</string>
      </array>
      <key>CFBundleTypeName</key>
      <string>JPEG image</string>
      <key>CFBundleTypeOSTypes</key>
      <array>
        <string>JPEG</string>
      </array>
      <key>CFBundleTypeRole</key>
      <string>Editor</string>
    </dict>
    <dict>
      <key>CFBundleTypeExtensions</key>
      <array>
        <string>gif</string>
      </array>
      <key>CFBundleTypeIconFile</key>
      <string>electron.icns</string>
      <key>CFBundleTypeMIMETypes</key>
      <array>
        <string>image/gif</string>
      </array>
      <key>CFBundleTypeName</key>
      <string>GIF image</string>
      <key>CFBundleTypeOSTypes</key>
      <array>
        <string>GIFf</string>
      </array>
      <key>CFBundleTypeRole</key>
      <string>Editor</string>
    </dict>
  </array>
</dict>
</plist>

Here are a couple links for reference:
https://electronforge.io/config/electronPackagerConfig
https://github.com/electron-userland/electron-forge/issues/492
Hope that helps you get started!

Thank you!!! This really helps. I was completely lost.
I would suggest adding a point about this in the package README to save future developers.
I ended up using electron-builder (not sure if it's the best one), because it has a simple API:
electron-builder --config electron-builder-config.json

electron-builder-config.json
{
   "appId": "com.example.app",
   "files": ["**"],
   "fileAssociations": [
    {
        "ext": ["txt"],
        "name": "Plain Text",
        "role": "Editor"
    }
   ]
}

I created an example project for future reference:
https://github.com/rchrd2/example-electron-file-association
Note, it does not use this library (electron-file-opener) for simplicity's sake,

Nice that's a pretty clean config file. I need(ed) forge a bunch of different things, and while having to create a plist file isn't ideal, it definitely works within my workflow. I'll keep electron-builder in mind for the future thought :)


Key Vault - Certificates Readme Issues

Section link:

Reason: Link error.
Suggestion:
Add link for managed identity: "https://learn.microsoft.com/zh-cn/entra/identity/managed-identities-azure-resources/overview".
@sandeep-sen , @joshfree , @heaths and @chenrujun for notification.

@vcolin7 please take a look at this broken link issue for the keyvault readme


GCC 4.7.2-5 doesn't like implicit conversion of unsigned to signed types

To make rapidjson compile cleanly with our warning flags, I had to make this
trivial change:
Index: include/rapidjson/writer.h
===================================================================
--- include/rapidjson/writer.h  (revision 131)
+++ include/rapidjson/writer.h  (working copy)
@@ -129,7 +129,7 @@ class Writer {
                char buffer[10];
                char *p = buffer;
                do {
-                       *p++ = (u % 10) + '0';
+                       *p++ = char((u % 10) + '0');
                        u /= 10;
                } while (u > 0);

@@ -151,7 +151,7 @@ class Writer {
                char buffer[20];
                char *p = buffer;
                do {
-                       *p++ = char(u64 % 10) + '0';
+                       *p++ = char(char(u64 % 10) + '0');
                        u64 /= 10;
                } while (u64 > 0);

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 28 Oct 2013 at 5:00

May I know what is your warning flags settings?
After some efforts, the master branch at https://github.com/miloyip/rapidjson
can be compiled with -Wall -Wextra -Weffc++ -Wswitch-default
Can you verify it in your environment?
Thank you.

Original comment by<EMAIL_ADDRESS>on 13 Jul 2014 at 4:42

Changed state: Accepted


is it possible to get an api key?

Hi, I'm in a project and I'm using this openai-kotlin client. Is it possible to send a request with user and password and get the user api key or token with that?
So the user would introduce user and password to use my app, instead of finding the api key manually.
I can do this sending a request with okhttp for example but it would be more comfortable to do it with the client.

Hi @Javi3rV, is there any official API for authentication? I couldn’t find any!
If yes, can you please share the documentation link? Thanks!

I'm new to coding, and am also wondering if API proxy like Apigee can solve the problem. Any thoughts?
I would also like to learn how to streamline the user experience.

Hmm no, it's not really, I thought yes. I'm sorry!
I asked because I used a chatbot from the play store that asks you to login
with your openai account to work. The name is Aico - GPT AI Chat
They may be using a web view
Thanks anyways
El jue, 30 mar 2023, 22:50, Mouaad Aallam @.***>
escribió:

Hi @Javi3rV https://github.com/Javi3rV, is there any official API for
authentication? I couldn’t find any!
If yes, can you please share the documentation link? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/aallam/openai-kotlin/issues/132#issuecomment-1490936577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR3EFP4S7DEEI4KG6UMAB5DW6XWZVANCNFSM6AAAAAAWNAPSJA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Rewrite 0036-BRIE-event_queue (Ethereum Events) spec

During the test coverage review it was identified that we need to rewrite BRIE (Ethereum Events) spec
We need to also consider:
Expand known filters regarding this AC
Using configured external blockchain nodes, the Event Queue filters for specific events provided in GetSubscribedEventSourcesResponse
What is the param which controls this? - Event Queue retries sending events that have gone too long without being accepted ([0036-BRIE-006]
Can we include a diagram from core on how events are triggered/sent etc including all internal statuses
The spec need to be:

[ ] rewritten in order to describe the protocol as is.
[ ] have the ACs reviewed/rewritten in order to be more functional
[ ] review with @vegaprotocol/qa so that they can:
[ ] tag tests that cover these ACs
[ ] plan for the remaining test to be written

The expectation is that the ACs are likely to be already covered in other tests (needs checking once above done).

@C0deMunk33 has updated this from the ethereum side, however, this spec now needs to be updated in terms of functionality now EEF is in the core.

Many things are tested in other specs.
In this one we should specifically have ACs about how many out of whatever total (and with what weight) saw the events on Ethereum before it gets accepted as "vega truth".
So we still need the spec updated and the AC removed / added to reflect this.

@jeremyletang
The final updates need for this spec change are:

covering the process around the 50 confirmations needed before the tx/event is accepted
details on how many validators need to confirm these conformations in the network
ACs to cover the testing on the above

A new issue will be raised to review the 50 confirmations now that Ethereum has gone from PoW to PoS.


Bundles dissapear on startup

Description
I have set up some folders as bundles (amazing feature btw) but everytime I start files the bundles are not there and I have to import them. It's a minor inconvinience but I thought I'd report it.
Steps To Reproduce
No response
Expected behavior
Not having to import bundles everytime you open the app
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19043.1526
Windows Version
10
Relevant Assets/Logs
debug_fulltrust.txt
debug.txt

Merging with #8530


Policy Plus not downloading latest ADMX files? (and a bug?)

So I was making sure I had all the latest policies. I downloaded this from Microsoft's website to make sure I did.
It would appear policies from 21H1 and 21H2 (if you go to the "Administrative Templates" sheet on that that spreadsheet) aren't getting downloaded (or at least aren't appearing?) from Policy Plus, even when I go to "Help" > "Acquire ADMX Files"?
So I tried getting the files manually from Microsoft and (incorrectly?) importing them, but then I got this error that pops up whenever I open Policy Plus:

It can't be closed and I can't get to the regular Policy Plus window. I force closed the program and deleted the directory I was trying to load the files from which fixed it.
Can some help be given as to how I get the latest policies? I assumed pressing "Acquire ADMX Files" would but it seems it isn't up to date? Thanks.

Thank you for the report. Acquire ADMX Files was indeed downloading the 20H2 package. I have updated it to download the Windows 10 21H2 package—please try the latest build.
For mysterious reasons, Microsoft is also distributing Windows 11 ADMX files that are not a superset of any Windows 10 ADMX package. I will need to think about what to do about this, possibly provide a choice in the download dialog.
As for why manually obtaining the policy definitions didn't work: I notice that staging_ has been prepended to the ADMX names in your screenshot. Unless the ADML files in the language-specific subdirectories were similarly renamed, that would break the association between ADMX and ADML. Unfortunately there were so many files that the Yes and No buttons went off the screen. Pressing Escape might choose "No", though I'm not sure if that always works.

although it's nice that the latest build of Policy Plus can download the latest ADMX Files from MS, it should be improved to detect the Windows version being run first and then download the appropriate ADMX files from MS.
I had a minor problem when running Policy Plus on a Win8.1 home system and choosing the Acquire ADMX Files option actually downloaded the ADMX/ADML package for latest Win10 version, which is not what I want and a few of those admx/adml files would not load under Win8.1 home (because those policy files are only compatible with Win10 and not any pre-Win10 version)

Thanks for the report. I agree that obtaining inapplicable policy definitions can be undesirable and it would be better to offer version-specific definitions. I've opened a new issue #55 to track this feature request.
Could you please clarify how exactly the Windows 10 definitions failed to load on Windows 8.1? Policy Plus has its own ADMX loader that should work the same regardless of host OS, so if it couldn't handle the newest definitions, that's a separate bug. But if the Local Group Policy Editor was installed on Home (an unsupported configuration) and choked on the new definitions, that's not a Policy Plus bug (but does emphasize the usefulness of getting the right definitions for the current Windows version).


FCM branch how to use the google-services.json

Expected Behaviour
Uploading stats to Firebase console
Actual Behaviour
Not working
Reproduce Scenario (including but not limited to)
google-services.json put on the project folder
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android
Plugin version
2.0.0 (compiled with custom code added)
The Question
How can we add the google-services.json to the plugin to get the stats on the console.
If the plugin is compiled and then added to the project, do we have to add the json at the end before the cordova build or on the plugin directory?

Right now the google-services.json is not used, we should create a hook that copies it to the Android project

In case some is looking for the Sender ID to configure the plugin then it is in the google-services.json:
{
  "project_info": {
    "project_number": "[SENDER ID]"
...

Now explained at: https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/INSTALLATION.md#installation-requirements


Not rendering files with .md extension

When I use an .md extension on a file, I get the following:
WARNING:root:No parser found for tutorial.md. Using default renderer.
.markdown and .mkd work fine.

Thanks for the initial issue report! Please file a new issue if the hooks issue is still present in the latest version of wok.


Share Room Button

Create a share room button that copies the room link to a user's clipboard

I'd like to work on this issue

I'd like to work on this issue

Assigned it to you


reversed RTS1 barrier polarity

A cell phenotype in which the RTS1 replication termination site prevents replication fork movement in the opposite of normal direction. Normally, RTS1 acts as a barrier to replication forks moving in the cenII-distal direction.
PMID:18723894

reversed RTS1 barrier polarity FYPO:0005360

edit file: 653f4b9a27d3fcabf1d06845eeb5caab64d1d108
release: 21dbe123051bc992f88e2e2f9cc898264ea4e28d


Update BlueprintEntity for Factorio 2.0

Blueprints have changed in Factorio v2.0. Currently there's only the old manully-defined type for BlueprintEntity.
This might take some reverse-engineering work, and/or begging Factorio devs for better documentation.

Done in v3.5.1


Reading screen scrolls to the top after term save

Bug seems introduced in 3.0.9.
If the text doesn't fit on the screen and user scrolls down (eg narrow screen, large font, whatever), on save the screen scrolls up.
https://github.com/jzohrab/lute-v3/assets/1637133/0e309593-8cc0-43bd-9ae9-dbd01852a337

Confirmed bug not present in 3.0.8, is present in 3.0.9.
(.venv) MacBook-Pro:lute-v3 jeff$ git log 3.0.8..3.0.9 --merges --oneline
302421e Merge branch 'windows' into develop
b3969b0 Merge branch 'fix_excess_image_records' into develop
b4f7aff Merge branch 'handle_utf8_error' into develop
b251532 Merge branch 'add_sanskrit' into develop
a6f90a7 Merge branch 'issue_97_fix_ci' into develop
 Merge pull request #88 from webofpies/read-slider   <<< bug added here
e17aeb6 Merge branch 'delete-button' into develop    <<< bug not present here

commit aa2e66eddb76d35e7e25012feedcbdafcd3ebe15 (HEAD)
Merge: e17aeb6 d19847f

$ git log e17aeb6..aa2e66e --oneline
aa2e66e (HEAD) Merge pull request #88 from webofpies/read-slider
d19847f (webofpies/read-slider) redesign/relayout   << PRESENT
bb4865e hide bookmark button. remove unused css  << NOT PRESENT
84bf6b0 booktable padding   << NOT PRESENT
90f6bc3 first pass   << NOT PRESENT

Pretty sure bug was introduced with commit d19847f

It appears that the problematic line is
// for the reading page return the main container to full width
document.querySelector(".container").style.cssText = `width: 100%; margin: 0;`;

introduced in commit d19847f.  Checking that line in develop

Ah no, it's something else I'm not following.  If I reload the page, sometimes the problem occurs, sometimes it doesn't, depending on if I resize the frames.  Not sure what's up.

Double checking, was wrong during checks before -- bug sometimes shows up, sometimes not ... hard to track down

90f6bc3: bug present
3.0.8: bug present
3.0.7: present
3.0.6: not present

Brutal.  Now checking between .6 and .7:
$ git log 3.0.6..3.0.7 --merges --oneline
a6f4e90 Merge branch 'feat_text_options_revised' into develop    <<< present
4770f2b Merge branch 'save_book_filter_state' into develop   <<< NOT present  (???)
3dd6f20 Merge remote-tracking branch 'origin/develop' into develop
53b860d Merge branch 'read_hamburger' into develop
e772dff Merge pull request #62 from Anand-S23/readme_discord_link
8c80b72 Merge branch 'feat_54_arrow_key_scrolling' into develop
12496b3 Merge branch 'feat_58_done_final_page' into develop

double-checking 4770f2b - not present, I don't think
a6f4e90 - present ???

Checking a6f4e90 -- here are commits to check:
$ git log 4770f2b..a6f4e90 --oneline
a6f4e90 (HEAD) Merge branch 'feat_text_options_revised' into develop
cfdf372 Remove unused styles.
ee3a8a3 Click outside of slide-in menu to close.
5266111 Prevent title flash.
a2614e7 Fix error.
e6a9fc7 Use column svgs.
3598484 One or two columns only.
1b37486 Remove comments.
9ab1607 Keep observing so style changes are reapplied on div reload.
c2f065d Remove unused svgs.
7e7fb3e Move text options to slide-in menu.
a08334c Fix js error when no audio file present.
642c17b Manual port of changes.
a46bf67 Player style and js.
02bd911 Text options js.
9b256ba New svgs for font options.
8b7b368 Styling changes only.
31eba00 Small js fixes.
e88450b Fix slide-in menu on top of player.
9812d34 Fix audio not loading reliably in firefox.


KNOX-2020 - AWS federation support added to hadoop-jwt cookie

This contribution enhances the Hadoop JWT cookie to contain AWS credentials to interact with the AWS ecosystem. This is done via an optional configuration when using SAML for authentication. The account admin can set up AWS federation with SAML (https://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_providers_enable-console-saml.html). The SAML Response in such a case will contain the Identity Provider (IDP) injected Role. Simple Token Service (STS) is invoked for AssumeRoleWithSaml using this SAML Response for credentials, and on  success a cookie is added to the browser. The value for this cookie is the Base 64 encoded credentials. The TTL for this cookie matches the TTL for AWS credentials. From there on the usual Knox flow continues.
Knox will add the hadoop-jwt cookie that is understood by the Hadoop ecosystem.  A JWT provides standard info about the subject, TTL and claims. We enhance the claims to contain the AWS credentials by using the info in the previously stashed aws cookie. This way the Hadoop ecosystem services can now interact with AWS services too. The lifetime for the cookies matches the life of AWS credentials.
What changes were proposed in this pull request?
The Pac4j filter has been modified to optionally do AWS federation. This change has no effect if the configuration is not enabled.
A new security module gateway-provider-security-aws handles all the logic to do SAML federation and adding an AWS cookie.
This module receives the SAML Response and federates into AWS, and assume the first role in the assertion. This could be enhanced in future to determine which Role to pick. AWS credentials are stashed in a cookie for further usage (after the redirect happens in the filter). At the end it returns back to the pac4j filter with a cookie added to the response.
Methods to issue JWT token have been enhanced to take custom claims as parameters.
In KnoxSSO we enhance the hadoop-jwt cookie with the contents from the AWS cookie.
This is done by using custom claims. The procured AWS credentials are added to the JWT claims. Please look at the testing section for how the cookie looks like after the change. Note the cookie is unaffected if the option to do AWS SAML federation  is not turned on.
Knox token has also been modified to reflect the new token format.
How was this patch tested?
Ran all unit tests
Tested on an EC2 deployment (AWS EMR). The knoxtoken (/knoxtoken/api/v1/token) looks like
{
  "sub": "[sharad]",
  "aud": "tokenbased",
  "iss": "KNOXSSO",
  "aws-session": "eyJ1c2VybmFtZSI6InNoYXJrYWxhQGdtYWlsLmNvbSIsIkFXU0FjY2Vzc0tleUlkIjoiQVNJQVNTRkFaVFBBV0tGWTdVNFMiLCJBV1NTZWNyZXRLZXkiOiJYT01yS0FHdFBNakdaWi91SlJibzM2eFhzRFlwT0RzU1IvWlA1N2RQIiwic2Vzc2lvblRva2VuIjoiRlFvR1pYSXZZWGR6RUZFYURDNE8vS0NVZ2pNNWFYY0luaUtsQWh1dTNvK2NkTEoyNmwxMzdKNVI1dFNROUZKUHIzQ2Y4SGJhdHMzc1pWSzJZeUlGTFZGcjJUTXBCZjBxVjZTQ2ZYSzZwUzZ2L3hHeFZvZGl2L1VOVzZLclBEeUhkbEtPT3lKdVBncGdPRkdoakdSZUw3YTBqSmNpVUhZalVhRmo4UVd5R2lkM05xRVFzeTVQRlEwalM0SmFZUkU0K04yczlycWhxSWF4K1hreDBEb0gzWGl1Y2RDVUtBQVcwK0NNaHB3eGV1d2paZDIwZGJNOTFOTTVsR2dNbmxZdzBiNjdPdXlwUDI3T0NoeU9TYkdGVVJtbC9DY3YrbjNwMFA2RVY0Q3pJbnZlUjBRN28zL0tDQmRsdGF2Mi9zWXUzSDZ0N0pXVE9ZYUJvUHhralZzTENlZmN2OXRJc2Q3bGx4cmxsUEhmRUZBMmJIOXRaa0xOQlY2alE4NmRzL1NlS09ZTlJBTnNaTmJOckxZOTlJekR2NmdPQklaVWlTNXZ4aUY4ajZMVEs2VjBLSnV2NitzRiIsImV4cGlyYXRpb24iOjE1NjgzMzUyNzUwMDB9",
  "exp":<PHONE_NUMBER>
}
The AWS credentials are in Base 64 encoded format, and the exp matches the AWS credentials exp.

@sharad-oss looks like the jdk11 Travis failure might be real as well:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project gateway-provider-security-aws: Fatal error compiling: java.lang.ExceptionInInitializerError: com.sun.tools.javac.code.TypeTags -> [Help 1]

@risdenk I wasn't aware Knox was being built on JDK-11 too. My current tests were all done using JDK-8. I will invest some time to build with JDK-11.

From my comment on JIRA KNOX-2020:
@sharad-oss - this looks interesting!
I'd actually like to see a one-pager type doc that describes the usecases, the design and security considerations. Please attach it to the JIRA itself.
One thing that I am concerned about is the inclusion of sensitive credentials in the JWT based cookie.
The cookie from KnoxSSO is intended for browsers and generally represents the authenticated user but doesn't include credentials. It is essentially in clear text since the JWT is merely base64 encoded. This is not sufficient protection for credentials that can be used outside of the scope of Knox itself.
In terms of usecases, I'd like to understand the full flow including how/where the credentials are actually used and what consumer will be provided in Knox for the credentials.

@lmccay sure will work on the one pager doc soon, and update the JIRA.

@risdenk Addressed your feedback. I am seeing some test failures but they don't seem related to my changes.
I am also working on a one pager as requested by @lmccay.

This PR has not been touched for over a year ago; closing it.


Automatic foreign key assignment

Good Afternoon! Today is my second day using the ERD and I'm loving it so far. Nice work man!
Now there is one little issue I'm running into and I'm not sure if it's just me or not.
When I go to assign a foreign key the ERD keeps automatically mapping it to a random column in another table that I choose. And I'm not allowed to adjust the column it maps to. Is this normal or could be there some option that I have to enable?

It's a normal function.
The order of designation is as follows.

😅 ahh ok now I see lol. It's weird because I thought I was doing that yesterday but after all it was a long day haha. Thanks for the timely response man really appreciate it 👍!


pss tasks [technical debt]

remaining pss tasks, to submit independent of #974

implemented in #960 :

remove context from pss.unwrap - https://github.com/ethersphere/bee/pull/942#discussion_r522792838
introduce cancel for the handler routines conditioned on pss.Close (closed quit channel)

implement proximity check in pss.TryUnwrap in alignment with the target length used by send

if po(baseAddr, chunkAddr) < targetMaxLength*8` { return }

tests that use shorter target and use TryUnwrap will fail to deliver

@zelig , given that you mentioned in today's sprint planning that you want to tackle #974, I suppose that this one should be tackled as well? Based on what you write in #974

Adding this to the active sprint, based on that. Please decommission if my analysis is incorrect.


Update DateTime.h

fix const-correctness

Sure, when working with GameLift, I wanted to use the FleetAttributes::GetCreationTime::WasParseSuccessful as it sounds like a way to check if my fleet is allocated (I simply move its result into my object), but couldn't compile due to WasParseSuccessful being non-const. Given the getter nature of this method, that appeared to be strange to me.
Though after more testing, turns out this pretty much always returns true so i'll use now give a try to FleetAttributes::GetStatus.

Thank you, will merge is ASAP


Fix raw_connect param order in verify_credentials

I didn't notice this was wrong because https://github.com/ManageIQ/manageiq-ui-classic/issues/7647 caused us to not even get far enough to test raw_connect.
After reverting the PR that introduced the issue I noticed this issue

Backported to morphy in commit fc12e7b55956bee46681fcfd022b1d36b1d337b8.
commit fc12e7b55956bee46681fcfd022b1d36b1d337b8
Author: Oleg Barenboim<EMAIL_ADDRESS>Date:   Tue Mar 2 17:01:53 2021 -0500

    Merge pull request #9 from agrare/fix_bug_verify_credentials

    Fix raw_connect param order in verify_credentials

    (cherry picked from commit 010e8b58cec987757013fec3e4fa44b744d69483)


Tabnine is not working.

please complete the following information:

OS version: macOS Monterey 12.6.1, Darwin arm64 21.6.0
Editor version:
[VS Code]:
Version: 1.74.3
Commit: 97dec172d3256f8ca4bfb2143f3f76b503ca0534
Date: 2023-01-09T17:05:47.808Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.6.0
Sandboxed: Yes
Programming language: ts, js, ...
TabNine extension version: TabNine.tabnine-vscode 3.6.36

If the issue is related to TabNine failure, please provide log files:
Type TabNine::config in your editor, the logs are located at the bottom of the page
Editor console logs:
Help -> Toggle Developer Tools
Issue Details:
I can open the page throw TabNine::config and it works as usual:

But when I open the tabnine settings it's blank:

And the dev tools in VS Code show errors :

gz#20798

Hi @raykle please restart your IDE, should be fine now. Thanks for letting us know.

@TzufTabnine Yes. It works now. 👍


Fix Linux tuner details detection

Those changes come from various tests at user systems. I can't provide an explanation for each individual change.
But what I can say is this: On a very simple and typical setup: Ubuntu with a Hauppauge USB tuner, this implementation didn't work. After I had re-applied my earlier changes (= this PR), it was working again.

I think that your code is not correct here:
    UString infodir(devname + u"/device");
    ...
    if (!LoadText(str_vendor, infodir, u"idVendor", report)) {
        infodir = infodir + u"/..";
    }

If idVendor does not exist in /sys/devices/.../dvb/dvb*.frontend*/device, this is only because it is not available. Trying the idVendor in /.. is incorrect, it points to the controller device (USB or PCI), not the DVB device.
See an example with a MyGica S270 USB stick:
$ tslsdvb
0: /dev/dvb/adapter0 ("Siano Mobile Digital MDTV Receiver, MDTV Receiver 2.00", ISDB-T)
$ find /sys/devices -name dvb0.frontend0
/sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0
$ ll /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0
total 0
-r--r--r-- 1 root root 4096 déc.   7 12:59 dev
lrwxrwxrwx 1 root root    0 déc.   7 12:57 device -> ../../../1-2
drwxr-xr-x 2 root root    0 déc.   7 12:57 power
lrwxrwxrwx 1 root root    0 déc.   7 12:59 subsystem -> ../../../../../../../class/dvb
-rw-r--r-- 1 root root 4096 déc.   7 12:56 uevent

So, adding /.. to the device (infodir) points to /sys/devices/pci0000:00/0000:00:0c.0/usb1, the USB controller device, not the tuner USB device.
If you are still not convinced, just have a look at the idVendor files in the two directories (using USB vendor id list from http://www.linux-usb.org/usb.ids):
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/idVendor
187f
$ grep -i ^187f usb.ids
187f  Siano Mobile Silicon
$
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/../idVendor
1d6b
$ grep -i ^1d6b usb.ids
1d6b  Linux Foundation

You can see that device/idVendor is the correct vendor id for the tuner while device/../idVendor is the generic Linux id for the USB node.
Another example with an old Hauppauge USB stick:
$ tslsdvb
0: /dev/dvb/adapter0 ("DiBcom 7000MA/MB/PA/PB/MC, Hauppauge Nova-T Stick 2.00 SN:4027216814", DVB-T)
$ find /sys/devices -name dvb0.frontend0
/sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/idVendor
2040
$ grep -i ^2040 usb.ids
2040  Hauppauge
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/../idVendor
1d6b

Another one with an AverMedia USB stick:
$ tslsdvb
0: /dev/dvb/adapter0 ("Silicon Labs Si2168, AVerMedia TD310 Device 2.00 SN:202866000009", DVB-T, DVB-T2, DVB-C)
$ find /sys/devices -name dvb0.frontend0
/sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/idVendor
07ca
$ grep -i ^07ca usb.ids
07ca  AVerMedia Technologies, Inc.
$ cat /sys/devices/pci0000:00/0000:00:0c.0/usb1/1-2/dvb/dvb0.frontend0/device/../idVendor
1d6b

You can see that using ../idVendor leads to reading the vendor id of the wrong device.
I think we should not expect all files to be there in the device directory. I haven't seen any missing idVendor file. But the MyGica device has no serial file for instance. And I do not trust the version file because it returns "2.00" for all devices.

I think we should not expect all files to be there in the device directory.

I agree.

I haven't seen any missing idVendor file.

I have. IIRC it was about the following device:
/dev/dvb/adapter0 (Vuplus FE, DVB-C)

tslsdvb: ioctl FE_READ_BER on /dev/dvb/adapter0/frontend0: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SNR on /dev/dvb/adapter0/frontend0: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SIGNAL_STRENGTH on /dev/dvb/adapter0/frontend0: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_UNCORRECTED_BLOCKS on /dev/dvb/adapter0/frontend0: Resource temporarily unavailable
  Status: has signal, has carrier, has viterbi, has sync, has lock

  Device Path:      /dev/dvb/adapter0
  Device Serial:
  Bit error rate ................................. 0 (0%)
  Signal/noise ratio ............................. 0 (0%)
  Signal strength ................................ 0 (0%)
  Uncorrected blocks ............................. 0
  Frequencies:
    Current ............................ 482,000,000 Hz
    Min ................................. 47,000,000 Hz
    Max ................................ 862,000,000 Hz
    Step .................................... 62,500 Hz
    Tolerance .................................... 0 Hz
  Symbol rates:
    Current .............................. 6,900,000 sym/s
    Min .................................... 870,000 sym/s
    Max ................................. 11,700,000 sym/s
    Tolerance .................................... 0 sym/s
  Spectral inversion ........................... off
  FEC(inner) .................................. auto
  Modulation ............................... 256-QAM

  Capabilities: inversion auto, FEC auto, 16-QAM, 32-QAM, 64-QAM, 128-QAM,
    256-QAM

/dev/dvb/adapter0:1:1:1 (Vuplus FE, DVB-C)

tslsdvb: ioctl FE_READ_BER on /dev/dvb/adapter0/frontend1: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SNR on /dev/dvb/adapter0/frontend1: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SIGNAL_STRENGTH on /dev/dvb/adapter0/frontend1: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_UNCORRECTED_BLOCKS on /dev/dvb/adapter0/frontend1: Resource temporarily unavailable
  Status: has signal, has carrier, has viterbi, has sync, has lock

  Device Path:      /dev/dvb/adapter0:1:1:1
  Device Serial:
  Bit error rate ................................. 0 (0%)
  Signal/noise ratio ............................. 0 (0%)
  Signal strength ................................ 0 (0%)
  Uncorrected blocks ............................. 0
  Frequencies:
    Current ............................ 562,000,000 Hz
    Min ................................. 47,000,000 Hz
    Max ................................ 862,000,000 Hz
    Step .................................... 62,500 Hz
    Tolerance .................................... 0 Hz
  Symbol rates:
    Current .............................. 6,900,000 sym/s
    Min .................................... 870,000 sym/s
    Max ................................. 11,700,000 sym/s
    Tolerance .................................... 0 sym/s
  Spectral inversion ........................... off
  FEC(inner) .................................. auto
  Modulation ............................... 256-QAM

  Capabilities: inversion auto, FEC auto, 16-QAM, 32-QAM, 64-QAM, 128-QAM,
    256-QAM

/dev/dvb/adapter0:10:10:10 (Vuplus FE, DVB-S)

tslsdvb: ioctl FE_READ_BER on /dev/dvb/adapter0/frontend10: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SNR on /dev/dvb/adapter0/frontend10: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_SIGNAL_STRENGTH on /dev/dvb/adapter0/frontend10: Resource temporarily unavailable
tslsdvb: ioctl FE_READ_UNCORRECTED_BLOCKS on /dev/dvb/adapter0/frontend10: Resource temporarily unavailable
  Status: has signal, has carrier, has viterbi

  Device Path:      /dev/dvb/adapter0:10:10:10
  Device Serial:
  Bit error rate ................................. 0 (0%)
  Signal/noise ratio ............................. 0 (0%)
  Signal strength ................................ 0 (0%)
  Uncorrected blocks ............................. 0
  Frequencies:
    Current ...................................... 0 Hz
    Min ................................ 950,000,000 Hz
    Max .............................. 2,150,000,000 Hz
    Step ................................. 1,011,000 Hz
    Tolerance ........................... 29,500,000 Hz
  Symbol rates:
    Current ...................................... 0 sym/s
    Min .................................. 1,000,000 sym/s
    Max ................................. 45,000,000 sym/s
    Tolerance .................................... 0 sym/s
  Spectral inversion ........................... off
  FEC(inner) .................................. auto

  Capabilities: inversion auto, FEC 1/2, FEC 2/3, FEC 3/4, FEC 5/6, FEC 7/8,
    FEC auto, QPSK, recover

I've shortened the output. It has 16 tuners.

Hi @SS-FF,
Shall we close this one? Do you agree that collecting device info in /.. when not found in /device accesses a different device (typically the USB hub in the case of a USB tuner)?

Hi @SS-FF,
Shall we close this one? Do you agree that collecting device info in /.. when not found in /device accesses a different device (typically the USB hub in the case of a USB tuner)?

Do you agree that collecting device info in /.. when not found in /device accesses a different device (typically the USB hub in the case of a USB tuner)?

No, I don't agree because certain devices like in the example I have given can be modeled in a way that the desired info is one level above. I don't have an example for USB, but for USB there's a concept of "compound" or "composite" devices that have several sub-devices internally. In such cases, the desired information might also exist at the parent level, because a device manufacturer might want to present the device as a single one.

Shall we close this one?

Yes. It's about rare cases only ;-)

Do you agree that collecting device info in /.. when not found in /device accesses a different device (typically the USB hub in the case of a USB tuner)?

No, I don't agree because certain devices like in the example I have given can be modeled in a way that the desired info is one level above. I don't have an example for USB, but for USB there's a concept of "compound" or "composite" devices that have several sub-devices internally. In such cases, the desired information might also exist at the parent level, because a device manufacturer might want to present the device as a single one.

Shall we close this one?

Yes. It's about rare cases only ;-)


Fix for incorrect prop type on children, fixes #1360

This PR closes #1360
Description
Fix for incorrect prop type on children

We are moved to LocalizationProvider. I don’t know why MuiUtilaProvider is still here

@dmtrKovalenko This is a v3.2 bug that has been sitting around since October. There was a comment about fixing it but it was never implemented.
Or have v3.2 bugs been kicked to the curb…?

I would encourage declining all pull request to the v3 branch unless there is a major issue or a security concern. We have limited resources, I think that it's better invest them for the future.


Remove index_summary_upgrade_test.py

See the rationale in CASSANDRA-11127.

+1, based on persuasive rationale in that ticket. Can you please include the ticket number in the commit message though?

Good call.  I've force pushed a new commit message.


Cyclic FIN Layer to Linear FIN Layer

Hi !
First of all thank you for your work, I was waiting your code since I read your paper !
I was wondering if you can give some advice to modify your code from a cyclic function of the FIN layer to a Linear one ?
I actually try to only replace every single cos_phi / sin_phi to a simple phi, but I'm not sure that will be enough.
Maybe I will miss some major points by only changing these.
Thank you again !

Hi!
Thank you for your interest. First of all, this is a simple implementation of linear FIN that you can use as reference.
class LinearFIN(nn.Module):
    def __init__(self, dim):
        super().__init__()
        self.instance_norm = nn.InstanceNorm2d(dim, affine=False)
        self.a_gamma = nn.Parameter(torch.zeros(dim))
        self.b_gamma = nn.Parameter(torch.ones(dim))
        self.a_beta = nn.Parameter(torch.zeros(dim))
        self.b_beta = nn.Parameter(torch.zeros(dim))

    def forward(self, x, phi):
        gamma = self.a_gamma.unsqueeze(0) * phi.unsqueeze(-1) + self.b_gamma.unsqueeze(0)
        beta = self.a_beta.unsqueeze(0) * phi.unsqueeze(-1) + self.b_beta.unsqueeze(0)
        return self.instance_norm(x) * gamma.unsqueeze(-1).unsqueeze(-1) + beta.unsqueeze(-1).unsqueeze(-1)

As regards the modifications to the rest of the code, you should be mostly fine if you just replace the cos_phi and sin_phi with a single phi. The only point where you should be more cautious is in the phiNet_A angle estimation, which should be replaced by a point estimation on a linear manifold. Hence, you should avoid extracting sine and cosine of the estimated angle and you should use directly the predicted value.


[SPARK-49934][SQL] Add implicit cast for accessing collated map with literal

What changes were proposed in this pull request?

Add a new case in collation type casts for retrieving values from a map where the keys are collated strings. This will allow the literal key to be implicitly cast to the appropriate collation type used in the map.
Why are the changes needed?

So explicit collate is no longer needed when accessing map elements.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

New unit test.
Was this patch authored or co-authored using generative AI tooling?

No.

+1, LGTM. Merging to master.
Thank you, @stefankandic and @zhipengmao-db for review.


pcfdev fails to start all services with AWS provider

Started pcfdev using start-osx script after modifying the start command to specify --provider=aws and confirming required environment variables were populated.
VM is created successfully and provisioning script is started.
46 of 48 services are started successfully, but after hours, the remaining two have not started.
Command cf api <public DNS name> --skip-ssl-validation fails with a 404 error
Gist:  https://gist.github.com/mdcarlson/ff9fb0cadc665271c6cb703a2491dfab#file-pcfdev-console-log-txt
Monit summary on the AWS pcfdev instance shows that cf-mysql-broker and quota-enforcer were in a not monitored state.  MySQL appears to be failing to start due to an error attempting to connect to the mysql-_broker at mysql.local.pcfdev.io in the mysql-broker-dbcreate and mysql-broker-dbmigrate scripts.

Addressed in #51.


How to get encoded sentence vector

Hello, Sir
Thank you for implementation Keras version BERT.
Is there any way to get an encoded sentence vector?
I have a sentence like:
The Top 25 Songs That Matter Right Now
And I want to get encoded sentence vector after putting this sentence to BERT.
Thx again.

See #7 and #19.

Hi,
I put some English words into a sentence encoding, But get KerError.
Example : [ '[CLS]', 'I', 'can't', '[SEP]' ] etc.
Is there any solution for this situation?


Are there plans to integrate a registry server into ied?

Thank you for the excellent ied.  I'm wondering if there are plans to have ied work like a private registry?

No, but I'm working on a separate registry server. This will take anywhere between 3 - 5 months to be released. ied itself should purely be a client side package manager.
I appreciate the feedback and will keep this issue open to keep post a link once I have the first version in a stable state.
Thanks!

Really?! That is really interesting. I'm currently running a fork of local-npm and have been keeping an eye on npm-ipfs. What in particular are you planning to do better/different than the existing repository servers?

Closing for now. I started working on a private registry server. It's here: https://github.com/alexanderGugel/nerva
Let's continue the discussion there if you're interested.


Looks to fix issue #10 EVALSYS-1578

If someone has a better fix I'm all for it! The fix to StandardDynamicListInputEvolver.java I'm pretty confident in, but the one to GeneralLeafParser is when saving the form back. It seems to work fine and be able to parse this with that class. I'm not 100% sure how it was able to parse String[] before since the explicit parse for that in GeneralLeafParser was commented out years ago but maybe there's something else I'm missing here.
     //registerParser(ArrayUtil.stringArrayClass, StringArrayParser.instance);
We could create a custom Parser for List but it really doesn't seem to be necessary.

@ern?

Do we need to release a new RSF after this merge?

I don't really remember much of this change now. I feel like it might not be correct and I guess I didn't run this test?
There were these 4 "historical" parsers in here that were disabled, int/double/string arrays and matrix.
I think I tried to do a "quick hack" to handle the List.class  but the render method for a ComponentList isn't correct here.
I'm inclined to believe the code is wrong at
    registerParser(List.class, new StringParser());
rather than the test. I feel like I tried to use the historical parsers that were there but those looked like they had a lot of issues with them.
I also don't know why these historical parsers were removed back in 2008 in 832c763d732e948089c7951e8adf23c01d055c2e
This is going to need some more work.

I only really tested this on Evaluation and this was never QA'd in regular Sakai so I probably wouldn't release in 20 without switching it and doing more testing especially with this test failure. I created a new issue for that.

This issue was solved in https://github.com/sakaicontrib/evaluation/pull/147, Matt you okay with reverting this change?

@ern #12

Gosh, guys - I really had no idea this stuff was going on so long after. You are all superheroes for threading your way through this maze.
I ask for forgiveness in plaguing future generations with this code - I thought I was doing good at the time, is my only defence - but what kind of defence is that : P
Hats off to all of you for keeping Sakai distributions running and viable 12 years later.

Hi @amb26 and nice to hear from you! This is a maze and it is still actively in use! On the roadmap we've got plans to replace the tools that are still using RSF but it's a multi-year effort. Many of the core tools still are using RSF.
Luckily this library hasn't really needed many major changes (yet) and we've been able to work around most of them without changing the code. The concern is that it might eventually need large changes, any changes have regression potential, and it things could be done to increase stability.
Like this change solved one problem but caused another so it was incomplete and was reverted and fixed another way. :/


added a CLI install instruction

that is a process I tend to prefer over the one in the UI of pyRevit
if you don't know the pyrevit CLI, it is installed separately or with pyRevit and allows you to perform a bunch of stuffs, tyr 'pyrevit --help' in the command line

Thanks, I like this alternative for people more capable of this method. On average I expect this is for more technical users or in preparation of a smarter install procedure.

Hrm that doesn't seem ideal then as the user would need to make their own
.extension folder if they install manually after download from git should
they choose not to use CLI. A lot of users probably wont understand how to
run cmd I think so I might have to stick with download>pyrevit settings.
I'll think about it...
On Tue, 15 Mar 2022, 00:19 jmcouffin, @.***> wrote:

@aussieBIMguru https://github.com/aussieBIMguru for the CLI to work you
will have to change the organization of your github repo as follow
[image: image]
https://user-images.githubusercontent.com/7872003/158179943-da975dd5-3709-4636-ab54-c53b8d1bf193.png
—
Reply to this email directly, view it on GitHub
https://github.com/aussieBIMguru/pyRoovit/pull/1#issuecomment-1066776284,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBEVYZUYDK6WW46W254LWLU744GRANCNFSM5QVBL2DA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Hrm that doesn't seem ideal then as the user would need to make their own .extension folder if they install manually after download from git should they choose not to use CLI. A lot of users probably wont understand how to run cmd I think so I might have to stick with download>pyrevit settings. I'll think about it...
…
On Tue, 15 Mar 2022, 00:19 jmcouffin, @.> wrote: @aussieBIMguru https://github.com/aussieBIMguru for the CLI to work you will have to change the organization of your github repo as follow [image: image] https://user-images.githubusercontent.com/7872003/158179943-da975dd5-3709-4636-ab54-c53b8d1bf193.png — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOBEVYZUYDK6WW46W254LWLU744GRANCNFSM5QVBL2DA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.>

Check out pychilizer or ef-tools extension. we all went this way. Much more reliable and streamlined.
That also leads to a way of getting your extension in sync with github with a startup script.

Ah yes I appreciate the CLI workflow is the most streamlined, but knowing
my channel audience being a bit on the beginner side I'll be anticipating a
bit of confusion.
Used to use Orchid for dynamo in tutorial videos on my channel and about
50% of queries were just 'how do i download and install Orchid', super time
wastey for me.
Might maintain two repos but need to think about it. Having to put
everything related to the repo in the extension limits design space
somewhat...
On Tue, 15 Mar 2022, 01:56 jmcouffin, @.***> wrote:

see: https://discourse.pyrevitlabs.io/t/pyrevit-cli-explaination/734
—
Reply to this email directly, view it on GitHub
https://github.com/aussieBIMguru/pyRoovit/pull/1#issuecomment-1066902894,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBEVYZGIE3VAVXGB3IS2QDU75HP3ANCNFSM5QVBL2DA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


zipkin: Fix wrongly rendered timestamp value

This patch fixes the wrongly rendered timestamp value of serialized Zipkin span by introducing replacers. This path is chosen since the way protobuf struct holds numeric data is by using double as its data type and when it is serialized (through MessageToJsonString; internally by snprintf-ing the value to a string) sometimes lead to a value with scientific notation involved (e.g.<PHONE_NUMBER>476870 will be printed as 1.58432429547687e+15, as reported in #9341).
A better solution probably by going down to fix protobuf MessageToJsonString.
Risk Level: Low
Testing: Unit-tested
Docs Changes: N/A
Release Notes: N/A
Fixes #9341
Signed-off-by: Dhi Aurrahman<EMAIL_ADDRESS>

is your image base on the same as Envoy.  I have re-pushed it. Could you help to re-pull it again?

yep :) doing it now!

ok image looks ok, running app with some tracing to see if http 400 is happening... So far so good but let me runs this for 1 hour...
Let me get back to you in about 1 hour with the results!

@marcantoine-bibeau awesome. Thank you!

So after a bit more than 30min, no http 400 when sending zipkin v2 traces, ix seems perfect :) Thank you very very much. Will you be able to merge that back to 13.x?

All Good :) Thanks again!

@dio you mention A better solution probably by going down to fix protobuf MessageToJsonString.. After reading the change I wonder if it is safer to go ahead and do the work necessary to ensure that we get the correct/desired value in the first place. It seems precarious to me to start holding state about replacements, and edit strings.

@junr03 yes, the possible changes will be introducing control how we render double value* inside the protobuf's strutil https://github.com/protocolbuffers/protobuf/blob/422053f3bcb39cac483d2769e936c473e7c8bcdb/src/google/protobuf/stubs/strutil.cc#L1248-L1290, since rendering uint64_t will be string: https://github.com/protocolbuffers/protobuf/issues/2679 (well a flag probably can be introduced here as well). However, I think the "replacements" are pretty well-scoped and especially for the v2 serializer.
*double is chosen since we use struct as "container" before rendering it to (JSON) string using protobuf lib. We also do the same thing with the "typed" access log.

So what I am hearing is that while you'd like to do that long term, you want to proceed with this PR as a short term fix to solve #9341, right?
If that is the case, do you mind creating issues and linking inline so we can keep track of that? After that I'll go ahead and review this approach more closely. Does that sound good to you?

I have logged here. https://github.com/envoyproxy/envoy/issues/10411. I also think that probably keeping it as string probably the safest way, but it needs an effort on documenting/enforcing it (e.g. by helping the collectors out there to embrace it).

tl;dr I think there's actual merit to merging this in for the short term, and chasing down a more permanent fix in protobuf (as mentioned in the comments of these PRs)
I totally understand the nervousness at doing the conversion at this layer (It's totally not the most ideal)!  However, at the same time it'll probably take awhile to get the fix upstream'd in protobuf (and a release cut, even when it is merged in). Since the code in protobuf is used for both actual doubles, and uint64_t's it has to be done with an extra branch, and some math trickery. Not to mention getting golang (or other languages with this problem) to parse scientific notation. Probably has more edge cases in the end, than us doing this here.

/azp-run envoy-linux

/azp run envoy-linux

I can leave #9341 open (and add a specific issue for that). @htuch, I think TODO is added.

Will add test as per @adriancole.

/retest

@junr03 do you have more comments on this? Thanks!

Can you merge master and we can get this merged?
/wait

@dio @mattklein123 Seems like this fix is having a side effect - the string replacements are causing other (tag) fields with the same numeric value as the duration, to have their quotes stripped. See https://github.com/istio/istio/issues/24177.

@objectiser could you help to review @douglas-reid's PR as well? I've made couple of comments too. Thank you!

Thanks for fixing this @dio. Do you know if this has made into any releases yet?

Doesn't look like the merge commit is in a release yet. You would have to user 1.15.0-dev / master for this fix.
git tag --contains 01fe4b620f643bea06bdf5cea3443f9b0767fc40


feat: Add Javascript File Export Options

This is a fix for Issue #189

@andrii-bodnar You're welcome


MNT: use openradar as DOCKER_USERNAME

Need to merge this. The problem is obviously with the creation of the package in ghcr.io.


Added rake tasks to cleanup workflows/requests

Add two rake tasks:
rake approval:requests:cleanup[days]    # Cleanup those requests older than certain days
rake approval:workflows:cleanup         # Cleanup those workflows with empty associated group_refs

to cleanup requests/workflows
https://projects.engineering.redhat.com/browse/SSP-625

@bzwei Please review/merge when ready


Fix constraint upper bound including prereleases

Resolves: python-poetry/poetry#2271

[x] Added tests for changed code.
[ ] Updated documentation for changed code.

Thanks!


Guide doesn't provide a clear python version to use

Same line, different text, I guess is 3.12 as both lines suggest some version or higher:

Python 3.12 https://github.com/ConsenSysDiligence/napalm/blob/main/docs/running-with-napalm.md#install-napalm
Python 3.10 https://github.com/ConsenSysDiligence/napalm/blob/main/docs/module-hacker.md#install-napalm

good catch!
it should've been python 3.12, I just fixed it 🚀  thanks for the issue


YETUS-896 Add an emoji column to the github vote table

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/1/console in case of problems.

:+1: +1 overall

Emoji
Vote
Subsystem
Runtime
Comment

:+1:
0
reexec
1205
Docker mode activated.

_ Prechecks _

:+1:
+1
dupname
0
No case conflicting files found.

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

_ master Compile Tests _

_ Patch Compile Tests _

:+1:
+1
shellcheck
0
There were no new shellcheck issues.

:+1:
+1
shelldocs
0
There were no new shelldocs issues.

:+1:
+1
whitespace
0
The patch has no whitespace issues.

_ Other Tests _

:+1:
+1
asflicense
10
The patch does not generate ASF License warnings.

:+1:

1231

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/67

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 91aca168d630 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 6b94d9d

Max. process+thread count
46 (vs. ulimit of 2000)

modules
C: precommit U: precommit

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.6.0

Powered by
Apache Yetus 0.11.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

The vote column seems redundant with the emoji maybe?
Is it worth a cli option to pick between emoji + text vote or just text vote or just emoji for votes?
Also should 0 votes have no emoji or perhaps a different one?

The vote column seems redundant with the emoji maybe?
Is it worth a cli option to pick between emoji + text vote or just text vote or just emoji for votes?
Also should 0 votes have no emoji or perhaps a different one?

I prefer to only use emoji. It is enough to show the vote result.

Okay so how about an option that determines if the vote column is emoji or text?

The complete list of github markdown emoji markup can be found in https://gist.github.com/rxaviers/7360908

Okay so how about an option that determines if the vote column is emoji or text?

Add a with-emoji option, if with-emoji, then use emoji and vote text both. If not, just use the vote text.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/2/console in case of problems.

:+1: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
2016
Docker mode activated.

_ Prechecks _

+1
dupname
1
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ master Compile Tests _

_ Patch Compile Tests _

+1
shellcheck
1
There were no new shellcheck issues.

+1
shelldocs
1
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

_ Other Tests _

+1
asflicense
10
The patch does not generate ASF License warnings.

2045

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/67

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 3a3de9025db3 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 6b94d9d

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

modules
C: precommit U: precommit

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/2/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.6.0

Powered by
Apache Yetus 0.11.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@busbey  @aw-was-here  Any more concerns?

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/3/console in case of problems.

:+1: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
1870
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

_ master Compile Tests _

_ Patch Compile Tests _

+1
shellcheck
0
There were no new shellcheck issues.

+1
shelldocs
1
There were no new shelldocs issues.

+1
whitespace
0
The patch has no whitespace issues.

_ Other Tests _

+1
asflicense
9
The patch does not generate ASF License warnings.

1897

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/yetus/pull/67

Optional Tests
dupname asflicense shellcheck shelldocs

uname
Linux 950bccfc6c13 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
precommit/src/main/shell/personality/yetus.sh

git revision
master / 6b94d9d

Max. process+thread count
46 (vs. ulimit of 2000)

modules
C: precommit U: precommit

Console output
https://builds.apache.org/job/yetus-github-multibranch/job/PR-67/3/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.6.0

Powered by
Apache Yetus 0.11.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Ping @busbey

I agree that the emoji is duplicated with the vote. I suggested that, we can introduce an option called --github-use-emoji, then we will just use the emoji instead of the old +1, -1 0, -0 and so on.
This is my implelmentation.
https://github.com/apache/hbase/commit/8e99b85718a2cd76103c535e5ba71afabf75b482
I tried to use it in hbase first but it does not work...

Closed in favor of #68


Fix urls in the example

The urls in the docs/03.API-EXAMPLE.md was pointing to
the old samsung.github.io pages.
JerryScript-DCO-1.0-Signed-off-by: Kristof Kosztyo<EMAIL_ADDRESS>

lgtm


Update vbs-ad-health-report.vbs

Added Domain Controller report source on line 192.

Thanks for your work!


Feature: Streamline Projectile & Trajectory Line Logic

Description
Currently the trajectory line and projectile use different ways to calculate the graph and gravity affect trajectory line.
Functional Requirements
None
Acceptance Criteria
None
Dependencies
None
Definition of Done

[x] Follows the naming conventions
[x] Follows the coding conventions
[x] Meets the acceptance criteria
[x] Pull request has been opened

@kianniek & @dpattipeilohy are working together on this

Works yay


Fix startup of docs container

Closes #1185
Code Changes

[x] Updated the docs Dockerfile to build in stages. The first stage builds everything and the second stage just copies in what is needed for the docs.
[x] Updated the nox commands to build the doc container from the root directory.

Steps to Confirm

[ ] Run nox -s docs_build and see no errors
[ ] Run nox -s docs_serve and the docs container should start and docs should be viewable at http:localhost:8000
[ ] Run docker-compose up docs and the docs container should start and docs should be viewable at http:localhost:8000

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[x] Issue Requirements are Met
[x] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

Closing in favor of #1271

Closing in favor of #1271


add unit-test for getAppArmorSecurityOpts & getSeccompSecurityOpts me…

…thod which locate on cri/v1alpha1/cri_utils.go
Signed-off-by: Coordinate36<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Add unit-test for getAppArmorSecurityOpts & getSeccompSecurityOpts method which locate on cri/v1alpha1/cri_utils.go.
Ⅱ. Does this pull request fix one issue?

fixes #1931
Ⅲ. Describe how you did it
Build unit test by condition combination coverage(ccc).
First, build program structure graph. According to the graph and the principle of ccc, we derive the test cases.
Ⅳ. Describe how to verify it
cd cri/v1alpha1; go test.
Ⅴ. Special notes for reviews
no

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #2004 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2004      +/-   ##
==========================================
+ Coverage   63.31%   63.34%   +0.02%
==========================================
  Files         200      200
  Lines       15522    15522
==========================================
+ Hits         9828     9832       +4
+ Misses       4455     4450       -5
- Partials     1239     1240       +1

Flag
Coverage Δ

#criv1alpha1test
33.85% <ø> (ø)
:arrow_up:

#criv1alpha2test
34.29% <ø> (+0.01%)
:arrow_up:

#integrationtest
38.42% <ø> (-0.02%)
:arrow_down:

#unittest
21.2% <ø> (+0.34%)
:arrow_up:

Impacted Files
Coverage Δ

daemon/mgr/container.go
53.85% <0%> (-0.16%)
:arrow_down:

ctrd/watch.go
75.75% <0%> (ø)
:arrow_up:

cri/v1alpha1/cri_utils.go
75.35% <0%> (+1.07%)
:arrow_up:

We have merged  a PR which solve the same problem
Thanks for your work
Close this now :)


Modular/Plugin based graphql schema

The ability to use a modular or plugin based graphql schema would help greatly with using GraphQL in extensible applications using the GRAND stack.

Thanks for the suggestion @aoighost
What exactly did you have in mind? Something like graphql-import?

+1 for this.
If you have multiple schemas/type definitions in different files, each with their own seperate Query and Mutation types, using makeAugmentedSchema doesn't merge the Query and Mutation from the different files (as they are strings), so what happens is that the last Query and Mutation are exposed and none of the others. In order to use the @cipher and @relation directives, you need to use makeAugmentedSchema or augmentSchema. I may be missing something, but have tried various ways to make this work. Any help with this would be great as keeping the entire schema in one file isn't maintainable for large projects.


API documentation [not ready]

Done

add comments to C++ code
create an API.md file
create a basic html site for the API documenting examples, methods, and return types
add a dev-dependency for documentationjs
add an npm script for generating API.md and index.html by running npm run doc

Remaining

[x] double check param types to make sure I understood them all correctly
[x] double check return object types against real output
[ ] group items within the OSRM class together; move typedefs for result objects to the bottom
[ ] root page with install instructions?
[ ] document generating documentation :smile:

You might have already seen this, but https://github.com/Project-OSRM/osrm-backend/wiki/Server-api contains a reference to most of the parameters and objects returned. Let me know if there are some subtle differences - the APIs should be as similar as possible.

Hey @morganherlocker what is the status of this? Any way I can help?

Really close on this. Only blocker I have at the moment is a potential issue upstream in documentationjs, but it's nothing big. More soon.

@morganherlocker is the documentationjs issue resolved?

@morganherlocker can this be pushed over the edge? would be great to release with osrm release next week :rocket:

hitting this today

@morganherlocker can I lend a hand? Would like to do a new node-osrm release tomorrow at the latest.

I committed a "hand-formatted" API.md to this branch. The remaining documentationjs issues are blocking the release of the full site for now, and I have not had a chance to tackle them, but linking to the API.md will be a good 80% solution for now.


nokogiri java does not properly handle xml files with byte order markers

We're updating our app from CRuby to JRuby, so we're swapping to the Java version of the nokogiri gem. I'm seeing some test failures, and the root cause seems to be that nokogiri was happy to work with XML files/schemas contaiing byte-order-markers before, but the java version does not like them. An error is occurring in the java extensions. The issue can be reproduced with the following script - which works fine on the non-java version of nokogiri, but produces errors or otherwise misbehaves on the java version. Tested with 1.8.3.
# create a string with only the UTF-8 BOM
bom = [239, 187, 191].pack('c*').force_encoding('UTF-8')

schema = bom + %Q(<?xml version = "1.0" encoding = "UTF-8"?>
<xs:schema xmlns:xs = "http://www.w3.org/2001/XMLSchema">
   <xs:element name = "contact">
      <xs:complexType>
         <xs:sequence>
            <xs:element name = "name" type = "xs:string" />
         </xs:sequence>
      </xs:complexType>
   </xs:element>
</xs:schema>
)

Nokogiri::XML.Schema(schema)

Which results in an error like the following:
Nokogiri::XML::SyntaxError: Content is not allowed in prolog.
	from nokogiri/XmlSchema.java:134:in `from_document'

When trying to parse XML with a BOM, it does not barf, but the document appears empty:
# create a string with only the UTF-8 BOM
bom = [239, 187, 191].pack('c*').force_encoding('UTF-8')

xml = bom + %Q(<?xml version = "1.0" encoding = "UTF-8"?>
<book>
  <title>Learning XML</title>
  <price>39.95</price>
</book>
)

Nokogiri::XML(xml)

Which results in:
 => #<Nokogiri::XML::Document:0x7d2 name="document">

I understand that we can strip the BOMs to work around this issue - but as the CRuby version of the gem seems to handle them just fine, I'm curious if the JRuby version can be likewise updated?

Thanks for reporting this. I'm tagging some of the people who know the Java implementation better than I do, @jvshahid and @kares, to see if they have some time to look into it.

@corycrowe how are you getting strings with BOM in them ? are those strings that you read from disk ? If so, can you use File.read(path, "bom|utf-8") to let JRuby/MRI strip the BOM before parsing the content.
For context, Java doesn't handle BOM as part of encoding (there are a couple of issues that were marked as wontfix with backward compatibility cited as the reason). I'm not sure which part on MRI is stripping the BOM, but libxml2 support BOM stripping as part of the internal encoding code. I think we could use something like BOMInputStream but I would like to justify the change given that Ruby have tools to strip those BOM for us.

we try to keep the two implementation consistent as much as we can. obviously this isn't always possible, since doing so would require non trivial changes sometimes. That said, I think this situation can be fixed in Java by wrapping the input streams that gets passed to the parser here. I think if we decide to go down this path, we will have to be careful to not throw the BOM away and instead try to figure out the encoding and pass that through to the parser.

@jvshahid @flavorjones do you guys happen to know if BOM removal happens at C-Ruby level or libxml?
... there's some support for BOM skips in JRuby so it might be a question of using more of the native APIs

I haven't looked deeply at the C-Ruby implementation but I saw in the libxml2 docs that BOM is supported and used to set the encoding of the input stream. I can look into this later.

Heyo, libxml2 natively handles BOM via the method xmlDetectCharEncoding which is invoked by both the xml and html parsers. Nokogiri's C-Ruby implementation doesn't know anything about BOM (and neither do I, really!).

This was fixed by ba16682aae2cbd42c196bd8afdfcfe8a5d82fbdb in v1.11.0 (2021-01-03).


MMCA-4890: Ability to request & download the recent cash account transactions when more than 1000 records

You have conflicts, worth merging with main


view: add upsert behavior to record_block to respect key uniqueness constraints for duplicate commitments

Describe your changes
Closes #3945
Closes #4136
Checklist before requesting a review

[ ] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

This only changes the view server behavior in record_block, so only non-consensus storage.

This makes sense to me, but would like to defer to @redshiftzero to make sure this address the ping/reject


fix: serialization of newline characters

Fix related to #157546
To test simply retry the steps referenced in the issue to reproduce the bug and you can easily see its gone.
As far as I could see, semicolons were already serialized, only newlines were having this issue.

@microsoft-github-policy-service agree


Update README.md

Add Dutch translation support in readme as well

@IhorSyerkov how can I let this pass? It's saying something about the patch structure, while I just try to submit a pull request from my own fork (don't know how it can be done another way?)


Can't send dashboard emails

When I setup dashboard emails I can't send any email with this error:
Apr 15 21:29:36 node6 celery[1195780]: [2021-04-15 21:29:36,607: INFO/ForkPoolWorker-1] Starting report for slack: None and recipients<EMAIL_ADDRESS>Apr 15 21:29:36 node6 celery[1195780]: No row was found for one()
Apr 15 21:29:36 node6 celery[1195780]: Traceback (most recent call last):
Apr 15 21:29:36 node6 celery[1195780]:   File "/usr/local/lib/python3.8/site-packages/superset/utils/celery.py", line 51, in session_scope
Apr 15 21:29:36 node6 celery[1195780]:     yield session
Apr 15 21:29:36 node6 celery[1195780]:   File "/usr/local/lib/python3.8/site-packages/superset/tasks/schedules.py", line 264, in deliver_dashboard
Apr 15 21:29:36 node6 celery[1195780]:     driver = create_webdriver(session)
Apr 15 21:29:36 node6 celery[1195780]:   File "/usr/local/lib/python3.8/site-packages/superset/tasks/schedules.py", line 208, in create_webdriver
Apr 15 21:29:36 node6 celery[1195780]:     get_reports_user(session)
Apr 15 21:29:36 node6 celery[1195780]:   File "/usr/local/lib/python3.8/site-packages/superset/tasks/schedules.py", line 214, in get_reports_user
Apr 15 21:29:36 node6 celery[1195780]:     session.query(security_manager.user_model)
Apr 15 21:29:36 node6 celery[1195780]:   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/query.py", line 3500, in one
Apr 15 21:29:36 node6 celery[1195780]:     raise orm_exc.NoResultFound("No row was found for one()")
Apr 15 21:29:36 node6 celery[1195780]: sqlalchemy.orm.exc.NoResultFound: No row was found for one()

Expected results
Emails are sent without any problem
Actual results
There's no emails sent and when I setup Dashboard emails I can see error written above.
Screenshots
N/A
How to reproduce the bug

Setup dashboard emails
See error in log

Environment
(please complete the following information):

superset version: 1.1.0
python version: 3.8.7
node.js version: v12.21.0

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

@willbarrett @lilykuang 🙏

It appears that you're using the older, deprecated alerts & reports functionality. Please see the docs for the new Alerts and Reports functionality: https://superset.apache.org/docs/installation/alerts-reports


SSL does not work with proxy

When using SSL and proxy together, Tomcat fails on startup with

SEVERE: Parse Fatal Error at line 75 column 35: Attribute "scheme" was already specified for element "Connector".

Example declaration:
  class { '::jira':
    javahome           => $profile::java::java_home,
    tomcatNativeSsl    => true,
    proxy              => {
      scheme    => 'https',
      proxyPort => '443',
      proxyName => 'jira.example.org',
    }
  }

The server.xml.erb template should not add scheme twice in this case.

Thanks for the patch.


Arabic rendered as Left to Right

Windows Terminal version
1.22.240823002-preview
Windows build number
10.0.19045.4529
Other Software
No response
Steps to reproduce
write Arabic text
Expected Behavior

from ver  1.19.11213.0 disabling "AtlasEngine"
Actual Behavior

I did this intentionally so that the Arabic mode in (neo)vim would work (vim -A). For instance, using the sample text from https://r12a.github.io/scripts/arab/arb.html:

I was hoping to add a setting in the future which enables RTL overrides and/or detection, but I haven't gotten there yet myself.

Probably should just bucket that with #538, yea?

Yeah, I'll close this as a /duplicate of #538. Thank you for the report though!

Yeah, I'll close this as a /duplicate of #538. Thank you for the report though!

@lhecker these are tow different  bugs this is new bug in AtlasEngine, #538 is old bug before AtlasEngine.

#538 tracks us actually supporting RTL text properly. It is the right issue for RTL issues because we have not officially declared support for RTL, on the old or the new renderer 😄

It's also worth mentioning again IMO, that I did this intentionally, so it's definitely not a bug. I did that because vim -A among others actively expects a terminal to not support RTL. I suspect it'll be easier to add RTL support once the rendering architecture has been refactored, because then AtlasEngine will be a fraction of its current size.


No longer available

May I hear the reason why this app was removed from google play? It was there a few months back.

Hi Aiosa,
Thanks for the heads up. Google removed it because I hadn't completed their family / children / whatever suitability declaration. I've now completed it, and apparently that means they'll put it back up after a review. I would assume that will take a few days, but they don't actually give a timeline.

It seems to be back now.


Migration to AiiDA 1.0

@kjappelbaum and @danieleongari I modified your PR to clean it up a little bit. Before it has a lot of creations-deletions, now it should be more clear.


add user management

The login component now calls the API to ensure that a copy of the user ID exists.
The profile component makes a request to the API for the user profile.
Auth0 registration now requires choosing a username. If social sign on is used username is  received from social profile

@jonrkarr  could you confirm that the current changes do not break the profile demo?

@jonrkarr could you confirm that the current changes do not break the profile demo?

This is confirmed to work. When the api is running, a one line change will pull non-demo data

Build is failing since other services depend on removed getuserbyauthid() method.
Potential solutions:

Other services get username from authentication service, then call user service on that username

A little bit more work per service, but maintains good separation of concerns
Username might not be most secure method of checking access

User service returns current user if no username is provided

User service will have dependency on auth service
User profile can be cached as observable (check how the auth service does this, I think it provides caching by sending the same observable object every time)

Auth service includes method for current user profile

objectively bad idea.  Auth service will then depend on user service, user model and create circular dependency

@jonrkarr  It seems that the method is used for providing a "default" route parameter wherever a username is expected. This is essentially the same issue that is solved in the refactored profile component.
Do you think it would make sense to move that logic into the user service to have it be reusable? The issue with this setup is that the root level urls will now depend on the user being logged in. This makes sense for the "/user" route, but not for the eg. "/models" route.
I believe the current set up is  essentially routing "/models" to "your models" when it should be routing it to browse models

I'll look into this.
getByAuth0Id is used in the user browse projects/models/simulations/visualizations components. We don't need the user profile. We just need to get the list of projects/models/simulations/visualizations owned by the user. There's a todo for this. I've ignored implementing this.

The tests should be passing now. I implemented out the management of resource ownership slightly further. The retrieval of projects/models/simulations/visualizations owned by a user is now moved to the services. My intension was to put that functionality in the services (more specifically the API behind the services). I largely haven't gotten that far.

All set to merge with master