[24.05] nixos/amdvlk: init module

Description of changes
Backport of #318175 sans release note changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

It would be acceptable to add release notes, if you want.


Adstock functionality

Hi Edwin,
Just want thank you so much again for sharing your knowledge and your remarkable work here. I want to ask a quick question about adstock functionality and I wonder whether I am making the right step here. I want to test with an excel file dataset and I am using the function make_adstock_metric to create adstock but I kept running into issues. Do you have any issues create adstock with a different dataset (an actual database in csv)? Which function do you suggest me to create for adstock process if you don't mind me asking.
Apologies for the inconvenience. I appreciate your great insights and valuable guidance as I am learning through this great package.
Greatly appreciate your time and patience,
Sincerely,
Lan Nguyen

hi @nguye2tl  you can take a look on the source code how I parametrize adstock.


⚠ Bofrost.fr has degraded performance

In 83e4eb8, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1084 ms

Resolved: Bofrost.fr performance has improved in 6fe2df5.


model-graded-factuality and model-graded-closedqa asserts using AzureOpenAI

I am trying to add this two assests using the AzureOpenAI provider with gpt4 model.
but I am getting error, is this asset type supported with the AzureOpenAI provider? why the error is refering to the OpenAI API key?

Thanks!

Have a look at https://promptfoo.dev/docs/configuration/expected-outputs/model-graded/#overriding-the-llm-grader
Looks like you need to override the LLM grader to use the AzureOpenAI provider.  For example:
defaultTest:
  options:
    provider: azureopenai:gpt-4
tests:
    assert:
      - type: model-graded-closedqa
        value: ...

Thanks for your response...
I have tried to  override the LLM grader but i am still getting errors:
Here you are my yaml
prompts: [prompt1.txt, prompt2.txt]
providers:
  - id: azureopenai:chat:gpt4
    config:
      apiHost: testinstance.openai.azure.com
      temperature: 0
      max_tokens: 50
      apiKey: XXXXXXXXXXX
evaluateOptions: { cache: false }
defaultTest:
  options:
    provider:
      id: azureopenai:chat:gpt4
      config:
        apiHost: testinstance.openai.azure.com
        apiKey: XXXXXXXXXXXXXXXXXX
  assert:
    - type: javascript
      value: Math.max(0, Math.min(1, 1 - (output.length - 100) / 900))
tests:
  - vars:
      query: What is the max purchase that doesn't require approval?
      context: file://docs/reimbursement.md
    assert:
      - type: contains
        value: '$500'
      - type: factuality
        value: the employee's manager is responsible for approvals
      - type: answer-relevance
        threshold: 0.8
      - type: context-recall
        threshold: 0.9
        value: Employees must submit their expense reports by the first business day of the month
      - type: context-relevance
        threshold: 0.9
      - type: context-faithfulness
        threshold: 0.9
  - vars:
      query: How many weeks is maternity leave?
      context: file://docs/maternity.md
    assert:
      - type: factuality
        value: maternity leave is 4 months
      - type: answer-relevance
        threshold: 0.8
      - type: similar
        value: eligible employees can take up to 4 months of leave

ERROR:
Error: Provider azureopenai:chat:gpt4 does not implement callEmbeddingApi for answer relevancy check, falling back to default
Error: Provider azureopenai:chat:gpt4 does not implement callEmbeddingApi for answer relevancy check, falling back to default
at getAndCheckProvider (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\matchers.js:79:15)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async matchesAnswerRelevance (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\matchers.js:297:29)
at async runAssertion (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\assertions.js:665:17)
at async runAssertions (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\assertions.js:73:24)
at async Evaluator.runEval (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\evaluator.js:210:37)
at async C:\Users\francisco.j.lopez\AppData\Roaming\npm\node_modules\promptfoo\dist\src\evaluator.js:475:25


Remove YAML dependency

It should develop myself YAML parser version

First read, validate format and count connector.
Second read,


Swagger UI not rendering in .net6 with "Microsoft.EntityFrameworkCore.Cosmos" version >= 7.0.0

Describe the issue
We are migrated from .net core 3.1 to .net6 in azure functions projects.
After migrating we are facing problem with swagger ui rendering.
We need to use Microsoft.EntityFrameworkCore.Cosmos version >= 7.0.0 because of dependency on RBAC which is supported in v7.0.0 onwards.
To Reproduce

Create a azure function app with .net6 in-process target.
Add package reference

Add <_FunctionsSkipCleanOutput>true</_FunctionsSkipCleanOutput> to surpresss dll not found issue.
Run the project and try to render Swagger UI.
Gives the error message "System.Private.CoreLib: Exception while executing function: Functions.RenderSwaggerUI. Microsoft.Azure.WebJobs.Host: Exception binding parameter 'log'. Microsoft.Azure.WebJobs.Host: No value was provided for parameter 'log'."

Expected
The swagger UI should render.

Bump 😉

I found that downgrading the package Microsoft.Extensions.Logging from 7.0.0 to 6.0.0 fixed my issue.


Subscribe problem

Hi, trying to use your library.
But can't get the subscribed data.
Here is the sample code:
public void SubscribeService()
{
if(!_isSubscribed)
{
_subscribeToCharacteristic = new SubscribeToCharacteristic(_deviceUuid, serviceId, readCharacteristicId, OnDataReceive, true);
BleManager.Instance.QueueCommand(_subscribeToCharacteristic);
_isSubscribed = true;
}
}
Before that, I scan the devices, connect to the one I need. After that, I sign up for a characteristic.
At the same time, in Android Logcat I can see that data is coming in, but the callback function is never called (OnDataReceive)

Hi @asadkov-hts
Interesting, the fact that data is coming in is good, but interesting that it's not passing it to your command. Could be an issue with the CustomGatt check in the SubscribeToCharacteristic.CommandReceived method. Can you show me one of the debug messages that the Android Logcat puts out? And the UUID's of the Service and Characteristic if you want to share them, helps with the comparison to see what's exactly going wrong.
What I'm interested to see is how the BleObject data that the Android Logcat shows compares to the data that you put into the constructor.

Hello @Velorexe.
I had to switch to a paid plugin due to project deadlines.
But I am interested to help solve this problem, I can test it within a week and provide you with logs.
What else might be interesting, I was able to get data via "BleMessageReceived UnityOnMessageReceived". By identifying my data through the "CharacteristicValueChanged" command. But there's a problem here too, the data is always 20 bytes in size. In my case it should be 43 bytes. Probably we need a mechanism for MTU size matching.

Unfortunate to hear that you had to switch, but I understand that you're pressed with deadlines too. Glad you gave this plugin a chance though!
I'm looking forward to hearing about those logs. I'll run some tests as well with custom UUID's and larger byte arrays to see if I can find the same issue.

Hi @lucapauri ,
Interesting, I haven't taken a deep dive into the MTU size matching at all and since the default of 20 bytes always worked for me, I didn't bother checking it out. I'll take a look at adding an option to change the MTU size or just match the size to the packages being send.
Sorry about the hassle that it may have caused!

Hi,
This should be fixed in the new v0.0.3-alpha.0 release. Inside the BleDevice class there's a method where a new MTU size can be requested with a callback if changing it was done successfully or not.


python3Packages.pdfminer: 20200726 -> 20201018

Motivation for this change
https://github.com/pdfminer/pdfminer.six/releases/tag/20201018
Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.pdfminer


ansible dynamic inventory for linode

Any ansible dynamic inventory available for linode ?

@prasathsekar - Not in the book, no—but there is a generic Linode plugin for dynamic inventory that can be used like any other.

Do you have any example playbook by using generic Linode plugin for dynamic inventory

No, you'll need to reference the Linode plugin's documentation and you can ask on Ansible's libera.chat IRC channel or elsewhere. I use DigitalOcean for most of my production infrastructure, and don't use Linode's dynamic inventory for the Linodes I run.


Setup Project and Add the necessary dependencies

I need to add:

For state management

mobx
mobx-react

For form handling:

mobx-react-form
validatorjs - form validation software

For routing:

mobx-react-form


Is there any good documentation on how to use the JST Filter?

I want to use the JST filter to generate templates that I can access via JST['template'] in my javascript but I couldn't find documentation on how to actually use the filter, only that it exists.

Yeah, I saw that link in the documentation but its for a ruby implementation and didn't really have anything to do with webassets themselves other than its based on the same ideas.   I'll read through their docs and see if I can figure it out but it seems like they would be different

I don't even see how I would use the jst filter with the standard bundle setup, since it should be outputting a js file with all the templates in it

Like this:
Bundle(
    Bundle('**.jst', filter='jst'),
    ''my_javascript.js',
    filter='jsmin', output='out.js')

This would preprend your own Javascript code with the generated JS code that holds the templates. In your Javascript code, you could then do something along the lines of:
 JST.license({name : "Moe", birthplace : "Brooklyn", bio : "Moe was always..."});

Which would render the "license.jst" template with the given data.

I think I found my issue.  It seems it can't handle multiple directories.  So if my structure is:
templates/foo.jst
templates/layouts/bar.js
it chokes on layouts directory because it tries to treat it like a file.

Hm, it shouldn't. I'd need a stacktrace and possibly your bundle definition to tell what is going on.

I'm not sure how to get a call stack,  I tried but for some reason it was losing all context (probably someone catching an exception and then re-raising it), but this is the code I used to find the error:
jst = Bundle('templates/*',
        filters='jst',
        output='jst.js')

assets_env.register('jst', jst)

try:
    jst.urls()
except IOError as exc:
    import pdb; pdb.set_trace()

(Pdb) jst.urls()
*** IOError: [Errno 21] Is a directory: '/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/layouts'
that succeeds if there are no other directories in the folder

So the last one would find templates/foo.jst and templates/layouts/foo.jst?

Correct.

Hmm, its still not quite right.  If I include it in my site like:
    {% assets filters="jsmin", output="jst.js",
        "jst",
    %}
        <script type="text/javascript" src="{{ ASSET_URL }}"></script>
    {% endassets %}

and have debug=False, it actually just tries to include the .jst files inside the template rather than generating the jst.js file.  This should generate unless I have updater=False (no matter if debug is on), since I need the file all the time

It is also only picking up 1 file even though there are plenty of files in the directory.

The way this currently has to be done is described here: http://elsdoerfer.name/docs/webassets/css_compilers.html
Bundle(
    Bundle('**/*.jst', filter='jst', debug=False),
    'my_javascript.js',
    filter='jsmin', output='out.js')

Note the debug=False.

That solves the generation issue but not the only picking up one template out of 10

(Pdb) get_all_bundle_files(jst)
['/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/adduser.html', '/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/userlist.html', '/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/user.html', '/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/userdetail.html', '/home/sontek/.virtualenvs/eventq/src/eventq.net/eventq/static/templates/userlist_layout.html']
but jst.js only has 2:
window.JST['userlist']
window.JST['userdetail']

I tried debugging it but it seems like its running subprocesses for the filters or something, my pdb never gets hit inside jst.py

Strange. There is no subprocess. Try putting print statements inside the jst filter.
You are using the latest version, right? I.e. this wouldn't be the caching bug that the most recent commit has fixed?

I'm using the latest on pypi, not from github.  I'll try pulling master and see what happens

So weird, I just started running from git and I'm still not getting it to hit output or input within jst.py but it does generate a jst.js file.
Still missing majority of the templates though

Its generating it when I hit the site for the first time, Its being hit by a template tag which tells it to generate, I wonder if that has something to do with it.

It sounds like you might have multiple versions installed. Stepping through the code with pdb, or playing with the imports in a shell should tell you which specific file paths are behind a module. E.g.
 >>> import webassets
 >>> webassets

Should give you a path.

Its running from the correct path:

import webassets
webassets
<module 'webassets' from '/home/sontek/.virtualenvs/eventq/src/webassets/src/webassets/init.pyc'>

sontek@beast$ (git::master) pwd
/home/sontek/.virtualenvs/eventq/src/webassets
(eventq)~/.virtualenvs/eventq/src/webassets
sontek@beast$ (git::master) git diff
diff --git a/src/webassets/filter/jst.py b/src/webassets/filter/jst.py
index ba55c17..41c3e2a 100644
--- a/src/webassets/filter/jst.py
+++ b/src/webassets/filter/jst.py
@@ -34,6 +34,8 @@ class JSTFilter(Filter):
self._templates = []
 def input(self, _in, out, source_path, output_path):

   print 'omgz wtf bbq'

   import pdb; pdb.set_trace()
   data = _in.read()
   self._templates.append(
       (source_path, data.replace('\n', '\\n').replace("'", r"\'")))

@@ -42,6 +44,8 @@ class JSTFilter(Filter):
out.write(data)
 def output(self, _in, out, **kwargs):

   print '11omgz wtf bbq'

   import pdb; pdb.set_trace()
   base_path = self._find_base_path() + os.path.sep

   if not self._bare:

@@ -62,6 +66,8 @@ class JSTFilter(Filter):
 def _find_base_path(self):
     """Hmmm.  There should aways be some common base path."""

   print 'hi'

   import pdb; pdb.set_trace()
   paths = [path for path, content in self._templates]
   if len(paths) == 1:
       return os.path.dirname(paths[0])

sontek@beast$ (git::master) pip search webassets
webassets                 - Media asset management for Python, with glue code for various web frameworks
INSTALLED: 0.7.dev
LATEST:    0.6

Have you tried to disabling the cache (Environment.cache / ASSETS_CACHE), or deleting the cache directory (.webassets-cache in Environment.directory)? The filter will not execute if the cache triggers. Of course, simply changing any of the source files should also cause the filter to re-run...

I was modifying files but it wasn't killing the cache. But that was definitely the issue =)

That really shouldn't happen, especially since there is actually a test that should validate the behavior. If you can pinpoint the problem, that would definitely be helpful.


merge utf8 to master

utf8 changes to be merged with master

I've fixed the left & right operation bugs.
For the result printer, I tried to fix it but can't find the solution for an issue, even if we count A and 大 as 1 character, when printing them, the later takes up more space than A. We can use the utf8proc library to count chars properly, but it provides no facility find how much space a utf8 character takes up.
I've raised an issue here #1047

I ran some queries with duckdb for long utf8 characters and it seems they don't print the characters.
They just cut it short with ellipsis ...

Both list_extract and array_extract didn't have UTF8 char support, that's why they were not working.
I've fixed both of them:

array_extract('都是大叔大的撒的', 4)

叔

(1 tuple)
Time: 19.24ms (compiling), 1.39ms (executing)

list_extract('都是大叔大的撒的', 4)

叔

(1 tuple)
Time: 8.62ms (compiling), 0.60ms (executing)
For the find function, not sure which function are you referring to ? There is no such function in the catalog currently.
Also I checked DuckDB's find function code (the needle haystack algorithm) and they have no UTF8 specific code there.


[epic] add model class

spatio-temporal?
what type of model: interpolation / forecasting?
black box vs grey box

closing this to be integrated into #210


fix: Fix ECDSA certificate creation

This is a regression with the rewrite of they new way the keys are handled. For some reason you cannot init subproperties in PS like you do in .NET - so the X and Y values of Q never change, thus issuing ECDSA certificates fails.

ECPoint is a struct, so Q gets copied, when it's accessed as the property.
It would need to be stored into a variable and then reassigned to Q after making the changes...
Missed that during the rewrite. Thanks for fixing the bug.
I'll merge the code and take another look at #130 before releasing the next patch version.


chore(ci): auto-approve dependabot pull requests

Add auto approval for dependabot pull requests to allow auto merging of dev dependencies

We can just use automerging of dependabot


Support for the experimental syntax 'decorators-legacy' isn't currently enabled,How to configure？

Using mobx and craco in projects,How to configure？

https://github.com/gsoft-inc/craco/blob/master/recipes/use-mobx/craco.config.js

@renleiabc hi! Have you solved the problem?


[FALSE-POSITIVE]

Domains or links
hufgard.pl
More Information
How did you discover your web site or domain was listed here?

Website was hacked

Have you requested removal from other sources?
Please include all relevant links to your existing removals / whitelistings.
https://www.barracudacentral.org/lookups/lookup-reputation
https://sitelookup.mcafee.com/en/feedback/url
https://talosintelligence.com/reputation_center/lookup?search=hufgard.pl#ip-addresses
Additional context
Add any other context about the problem here.
We had problem with plugin on our site , site now is clean and patched.
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR on the whitelist.me file

https://github.com/mitchellkrogza/Phishing.Database/blob/master/whitelist.me/whitelist.me
Please include the same above information to help speed up the whitelisting process.

Can't find any record. Closing.


Unarchive the archived Submissions

Fixes #3047
Add description here.

@frjo We can put it on test again with the latest changes, as I updated the code for small issues I noticed on test site.

I might have done something to break this PR @sandeepsajan0. Can you take a look at the failing test please.
I have merged in the archive PR to main.

I have just merged in https://github.com/HyphaApp/hypha/pull/3075 so the migrations will need some updates.

@frjo There was a problem with the dispatch method, at the time of getting the object. I am not sure why these tests were not failing before or it might have happened because of rebasing. Actually, I also got many conflicts on rebasing it with the main branch.
I don't know why it is picking up some of the Archive PR #3003 commits(looks weird) but file changes are the same as before.
I think we should put it on the test first?


Add support for inputOption.activeBackground

Describe the bug
VS Code will be introducing a new token color, inputOption.activeBackground, in the July release (1.37). This color is for showing when a button toggle is active (search, find, terminal find, etc.). It also re-uses focusBorder but in this theme the focus color is set to transparent, which means the new token color is also transparent. See below where you can't tell which item is "active", please consider adding support for this. You can test this by using the Insiders version, which already adds this.
To Reproduce
Steps to reproduce the behavior:

Go to Search
Enable one of the search options (match case, whole word, regex)
:bug: You can't tell if the option is on/off

Expected behavior
To visibly tell the the difference when a toggle is on/off
Screenshots

+1


Make RoleBinding subjects case insensitive

What did you do?
Currently resources name are case insensitive. But not the RBAC cache.
If you have an user "Toto" and you have a role binding with subject: "toto".
What did you expect to see?
It should have binded role to "Toto"
What did you see instead? Under which circumstances?
"Toto" do not have the role binded
System information
No response
Perses version
0.43.0
Perses configuration file
No response
Logs
No response

Correction: the system is actually case sensitive today. There is nothing in the code that lowercase the IDs


Point Kafka to eventing-contrib, not the vendored dir for eventing.

Release Note

Now using eventing-config/ for source.

/retest

/lgtm
/approve


[Scatter2] Rewrite scatter plot tool in React

status: waiting for date and collection api

While we wait for the new APIs, could you start reviewing? I don't anticipate large changes. Thanks!
I made some changes to timeline_search.tsx and a webdriver test kept failing for me and I could not fix it. Could you take a look?
@beets @shifucun @tjann

Reviewable status: 1 of 19 files reviewed, 25 unresolved discussions (waiting on @intrepiditee and @shifucun)

a discussion (no related file):
Very clean and structured code. My comments are mostly for the use of React hook and make the wrapper/abstraction easy to understand.

Thanks for reviewing.
For the context, the setXXX functions are totally unnecessary. I just wanted to place all the logic that manipulates the store in one file. If we want, we can remove and inline all those functions. Another way that I think is nicer is just like what you said - attaching the functions to the store so we can then do context.setXXX instead. Let me know which way you prefer. Thanks.

@beets @shifucun
Thanks for reviewing. Most of the comments have been addressed. An exception is moving populations and perCapita out of Axis, mainly because I am not yet sure if that's better and whether we should also move data and log out. We can discuss more on Monday, but I think this PR is ready for another round of reviews. Thanks.

Thanks for the update!

Hi Bo, I am thinking about pulling populations and data out of Axis but keeping log and perCapita. What do you think and is it a good idea? Thanks

Thanks for the update!

Hi Bo, I am thinking about pulling populations and data out of Axis but keeping log and perCapita. What do you think and is it a good idea? Thanks

perCapita is more of a data processing step.. Imagine the case 1) a special denominator other than total Population 2) other operations than "perCapita" are cases that can blow off this data manipulation.  Have this step separate out will be simple to maintain in general. The the "Axis" is really just about plot.

Thanks for the update!

Hi Bo, I am thinking about pulling populations and data out of Axis but keeping log and perCapita. What do you think and is it a good idea? Thanks

perCapita is more of a data processing step.. Imagine the case 1) a special denominator other than total Population 2) other operations than "perCapita" are cases that can blow off this data manipulation. Have this step separate out will be simple to maintain in general. The the "Axis" is really just about plot.

Could you explain a bit more what's the issue with other denominators? perCapita is a boolean just like log

Thanks for the update!

Hi Bo, I am thinking about pulling populations and data out of Axis but keeping log and perCapita. What do you think and is it a good idea? Thanks

perCapita is more of a data processing step.. Imagine the case 1) a special denominator other than total Population 2) other operations than "perCapita" are cases that can blow off this data manipulation. Have this step separate out will be simple to maintain in general. The the "Axis" is really just about plot.

Could you explain a bit more what's the issue with other denominators? perCapita is a boolean just like log

I see, so for the Context struct, perCapita is fine. I was thinking it for plot but it's actually a global state. That said, maybe the perCapita field in chart.tsx can be taken out. The plot params is derived from it.

That makes sense. Will do that. Thanks.
On Mon, Nov 30, 2020, 6:16 PM Bo Xu<EMAIL_ADDRESS>wrote:

Thanks for the update!
Hi Bo, I am thinking about pulling populations and data out of Axis but
keeping log and perCapita. What do you think and is it a good idea? Thanks
perCapita is more of a data processing step.. Imagine the case 1) a
special denominator other than total Population 2) other operations than
"perCapita" are cases that can blow off this data manipulation. Have this
step separate out will be simple to maintain in general. The the "Axis" is
really just about plot.
Could you explain a bit more what's the issue with other denominators?
perCapita is a boolean just like log
I see, so for the Context struct, perCapita is fine. I was thinking it for
plot but it's actually a global state. That said, maybe the perCapita field
in chart.tsx can be taken out. The plot params is derived from it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datacommonsorg/website/pull/668#issuecomment-736132552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHTEYS7J3HZMAKBPLYMWCNLSSQYWTANCNFSM4TOC4PJQ
.

server/routes/api/place.py, line 546 at r6 (raw file):
Previously, beets wrote…
This won't work for get arguments since it won't be in the cache key -- we've worked around this in the past by adding the args to the path, or calling a helper that is cached.

I thought the query_string option was for that?

static/js/tools/scatter2/chart.tsx, line 74 at r8 (raw file):
Previously, tjann (TJann) wrote…
Nit: I'm not sure what we need a utility to do? More than 3 decimals?

to determine the number of decimals places based on value

static/js/tools/scatter2/statvar.tsx, line 204 at r8 (raw file):
Previously, tjann (TJann) wrote…
nit: filtered

fixed

@shifucun @beets @tjann I am happy with the pull request. Could you merge? I don't have write access.

thanks @intrepiditee! merged, and we should see it on staging soon!


Docs: update requirements information

This PR updates the "requirements" document to include requirements for ruby-ray, node-ray, vue-ray, and alpinejs-ray.  I feel that it's important to have these requirements as visible as possible is because at a glance, Ray can appear to be a PHP-only debugging app.
Ensuring that it's clear at a glance that multiple languages are supported will add value to the Ray app and help drive license sales.

Thanks!


Update python version to 3.10+

This pull request is an attempt to update babelizer to use Python 3.10+.

@mdpiper The failing tests are because numpy removed numpy.distutils with Python 3.12. Are you ok if we merge this as-is and then fix things in a separate pr that adds the meson build system?

Yup, go for it!


Is it possible to add an attribute to disable Calligraphy on a TextView?

I'm required to have custom fonts app-wide while using https://github.com/JoanZapata/android-iconify.
However, android-iconify does not work with Calligraphy.
Is there a workaround? Can I do something like this?
<IconTextView
    android:text="{fa-heart}"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    app:disable_calligraphy="true"
    />

Thats a good idea. What would be prefered a attribute or a ignoreClass(Class<?> clazz) in the builder?

Right now an attribute will be great because it would fit my needs perfectly. Let me see if I can contribute by looking at the code.

Well in the interim I've found that using Iconify.addIcons(yourTextView) works for me. Leaving this here for future reference.
Meanwhile I'll submit a pull request soon for the new attribute.

How would this play out with menuitems, which don't give access to the underlying TextViews? Or are you somehow building out menus using IconTextViews?

I don't think the menu items get any font applied as they are pragmatically
created by Android.
On Mon, 22 Jun 2015 21:14 rothschild86<EMAIL_ADDRESS>wrote:

How would this play out with menuitems, which don't give access to the
underlying TextViews? Or are you somehow building out menus using
IconTextViews?
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/152#issuecomment-114246934
.

@chrisjenx do you have caligraphy and android-iconify running together in you project?

No, never used iconify.
On Mon, 22 Jun 2015 21:18 rothschild86<EMAIL_ADDRESS>wrote:

@chrisjenx https://github.com/chrisjenx do you have caligraphy and
android-iconify running together in you project?
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/152#issuecomment-114249165
.

@chrisjenx thanks for a great library
Just faced the same issue and in my opinion it should be an ignoreClass since you don't want to write attribute in every single place where you use iconify in a layout. Created #189

Why not both options? An attribute and an ignoreClass in the builder? Either can be helpful.

OK, seen as more than one person wants this I'll add it as a feature
request.
On Mon, 13 Jul 2015 13:38 kashban<EMAIL_ADDRESS>wrote:

Why not both options? An attribute and an ignoreClass in the builder?
Either can be helpful.
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/152#issuecomment-120913772
.

@kashban on the other hand maybe there is some scenario where this could be of use. Initially I only fixed this for Iconify. Pushed additional changes to pull request with calligraphyIgnore attribute

@rothschild86 I think until this gets implemented, another way to get both library to work well together would be to update your iconify library to use the v2 that has been released recently. Its author says it now works with Calligraphy. You can have a look at the new features here: http://blog.joanzapata.com/iconify-just-got-a-lot-better/

Cool nice work. Just remember, there are spans used in certain places
(Action Bar) still. Apart from that looks like a nice overhaul.
On Fri, 11 Sep 2015 09:32 k3v1n4ud3<EMAIL_ADDRESS>wrote:

@rothschild86 https://github.com/rothschild86 I think until this gets
implemented, another way to get both library to work well together would be
to update your iconify library to use the v2 that has been released
recently. Its author says it now works with Calligraphy. You can have a
look at the new features here:
http://blog.joanzapata.com/iconify-just-got-a-lot-better/
—
Reply to this email directly or view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/152#issuecomment-139485268
.

@k3v1n4ud3 right on! switched to iconics, which also plays nicely with caligraphy.

I'd still wish to see a disable option. I'm currently using a default font on the default textAppearance. But my font doesn't look good with numbers, so I'd like to disable it for a specific price TextView.
Is this still on the roadmap?

Pinging on this one since it's been 6 months... - Is this still a consideration

No. If it doesn't do it anyway, I'll change it so that fontPath="@null" would set the font to system default.

@chrisjenx  Good idea; I just tried that and it didn't work, so if you end up doing that please let me know. (PS sorry for duplicate issues)...
I'll do that though, I'll just ship the fonts myself for now...
<style name="MyNotoSerifHeader">
    <item name="android:textSize">30sp</item>
    <item name="android:textColor">@color/text_black</item>
    <item name="android:fontFamily">serif</item> <!-- Noto serif, introduced in lollipop -->
    <item name="fontPath">@null</item>
</style>

Thx for letting me know.
On Wed, 21 Dec 2016, 00:45 Neil Jay Warner<EMAIL_ADDRESS>wrote:

@chrisjenx https://github.com/chrisjenx Good idea; I just tried that
and it didn't work, so if you end up doing that please let me know. (PS
sorry for duplicate issues)...
I'll do that though, I'll just ship the fonts myself for now...

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chrisjenx/Calligraphy/issues/152#issuecomment-268392264,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHRsRJKxLO1ZA5kIM8sd46xm5PvR1kDks5rKGiMgaJpZM4D2tRB
.

Hi! Great and usefull library, thanks!
I'm trying -as others as I can see in this thread- to disable default callygraphy font on specific Textfields to show the system serif default font.
My Activity has a lot of Textfield & fragments using the default callygraphy font, so it's not a good idea to remove that.
I've tried with this fontPath="@null" (Without @ as well) parameter on the Textfield but it doesn't work for me. There is another workaround or way to achieve that?
Here is the code:
`
                    <TextView
                        android:layout_width="wrap_content"
                        android:layout_height="wrap_content"
                        android:layout_gravity="center_vertical|left"
                        android:paddingLeft="20dp"
                        android:text="@string/filter_title_vip"
                        fontPath="@null"
                        android:fontFamily="sans-serif"
                        android:textColor="@color/text_color_dark_blue"
                        android:textSize="15sp"
                        android:letterSpacing="0.06"
                        android:textAllCaps="true"
                        />`

Thanks in advance


Errors when editing solution which contains both C# .csproj and F# .fsproj projects

Environment data
dotnet --info output:
.NET Command Line Tools (2.1.4)

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

Runtime Environment:
 OS Name:     Windows
 OS Version:  6.1.7601
 OS Platform: Windows
 RID:         win7-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.1.4\

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

VS Code version: 1.20.1
C# Extension version: 1.14.0
Ionide-fsharp: 3.17.2 - 22.02.2018
Steps to reproduce
mkdir test-vs
cd test-vs
dotnet new library -lang F# -o .\ClassLibrary
dotnet new console -lang C# -o .\MainProgram
dotnet new sln
dotnet sln add .\ClassLibrary\ClassLibrary.fsproj
dotnet sln add .\MainProgram\MainProgram.csproj
dotnet add .\MainProgram\MainProgram.csproj reference .\ClassLibrary\ClassLibrary.fsproj

Add this line code to program.cs: ClassLibrary.Say.hello("world.");
dotnet run -p .\MainProgram\MainProgram.csproj

All the above succeeds without error.
In Visual Studio Code, first open the ClassLibrary folder. Building the project (Ctrl-Shift-B) succeeds.
(This step is based on guidance from #371.)
In Visual Studio Code, now open the solution folder (test-vs). Notice the following:

Building the solution (Ctrl-Shift-B) succeeds, as displayed in the Terminal window.
However, the Problems window shows one problem. "The name ClassLibrary does not exist in the current context."
Additionally, if you open the Program.cs file in the editor, the ClassLibrary class is underlined with the same error.
Finally, Output window and the OmniSharp Log displays the following error:

    fail: OmniSharp.MSBuild.ProjectLoader
    The target "GetTargetPath" does not exist in the project.
    c:\Temp\test-vs\MainProgram\MainProgram.csproj

Expected  behavior

The Problem window should display no problems for the solution.
The Editor window should display no problems for Program.cs.

Actual behavior

The Problem window says, "The name ClassLibrary does not exist in the current context."
The Editor window underlines the ClassLibrary class in Program.cs, with the same error.
An error appears in the Omnisharp Log.

It's pretty clear that one of your projects isn't loading properly. Could you provide your OmniSharp Log? Select View->Output in VS Code and then choose "OmniSharp Log" from the dropdown at the top-right corner of the Output pane.

Starting OmniSharp server at 2/28/2018, 9:18:50 AM
Target: c:\temp\testvs\temp.sln
OmniSharp server started
Path: C:\Users\NB61805.vscode\extensions\ms-vscode.csharp-1.14.0.omnisharp\OmniSharp.exe
PID: 13192
    Starting OmniSharp on Windows 6.1.7601.65536 (x64)

    Located 3 MSBuild instance(s)
        1: Visual Studio Build Tools 2017 15.3.26730.15 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\MSBuild\15.0\Bin"
        2: Visual Studio Enterprise 2017 15.5.27130.2020 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin"
        3: StandAlone 15.0 - "C:\Users\NB61805\.vscode\extensions\ms-vscode.csharp-1.14.0\.omnisharp\msbuild\15.0\Bin"

    Registered MSBuild instance: Visual Studio Build Tools 2017 15.3.26730.15 - "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\MSBuild\15.0\Bin"

    Detecting Cake files in 'c:\temp\testvs'.

    Could not find any Cake files

    Initializing in c:\temp\testvs

    Auto package restore: False

    Update workspace context

    Resolving projects references

    Detecting projects in 'c:\temp\testvs\temp.sln'.

    Queue project update for 'c:\temp\testvs\MainProgram\MainProgram.csproj'

    Detecting CSX files in 'c:\temp\testvs'.

    Could not find any CSX files

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider

    Loading project: c:\temp\testvs\MainProgram\MainProgram.csproj

    Configuration finished.

    Omnisharp server running using Stdio at location 'c:\temp\testvs' on host 1124.

    The target "GetTargetPath" does not exist in the project.

c:\temp\testvs\MainProgram\MainProgram.csproj
    Adding project 'c:\temp\testvs\MainProgram\MainProgram.csproj'

    Update project: MainProgram

Thanks! It looks lile the VS 2017 build tools installed on your machine are causing the problem. When OmniSharp is selecting an instance of MSBuild to use, it's picking the Build Tools. You can address this in a few ways:

Uninstall the build tools if you don't need them.
Update the build tools to the latest version. It appears that you have an older version installed (15.3).
Ensure that the build tools have both C# and F# workloads installed.
Start VS Code from a VS 2017 Developer Command Prompt, which will have environment variables set that cause OmniSharp to select the VS 2017 MSBuild.

I used the Visual Studio Installer to do this...

Update the build tools to the latest version.
Ensure that the build tools have both C# and F# workloads installed.

...I am no longer receiving errors.
Thank you @DustinCampbell !

Not a problem. I'm glad it's working for you now.


Prefix bound names with # path segment

In order to easily differentiate bound paths (ie paths that are to be processed by a namer rather than by delegation) we should apply the convention where bound paths start with a # path segment.
e.g. /#/io.l5d.fs/foobar
That is, /# should be prepended to the prefix of all namers.
This makes delegations easier to understand because it will be more immediately obvious which paths are "terminal".

One thing I realized with this change is that it will be difficult for folks to know that they have to update their configs.  I had a namerd config with:
    namers:
    - kind: io.l5d.experimental.k8s
      host: localhost
      port: 8001
    storage:
      kind: io.buoyant.namerd.storage.inMemory
      namespaces:
        default: |
          /ns         => /io.l5d.k8s;
          /iface      => /ns/prod;
          /srv        => /iface/incoming;
          /host       => /srv;
          /http/1.1/* => /host;

After picking up this change, namerd still happily loaded that config, but routing was broken because the /ns => /io.l5d.k8s entry was no longer doing anything useful.  It's unfortunate that /ns => /io.l5d.k8s is still a valid entry, even though its effect has changed entirely.  It makes me think we might want to do something drastic like abort linkerd/namerd startup if we recognize that baseDtab has an old namer string that's not prefixed with a "#".

Totally valid concern.  We'd have to hardcode a list of "old namer strings" and warn and/or abort if we see them.  We'd probably also only want to keep that check around for 1 or 2 versions: enough to give users a chance to transition but not keep it around forever.
Ideally this type of error would be trivial to debug by looking at the delegator UI, but I suspect most of our users wouldn't know to do that.

I encountered another subtle upgrade issue with this change and the tls bound path matcher.  I'm not sure there's anything we can do about it, but I wanted to post it here in case other folks run into similar.
Suppose I have a linkerd config as follows:
namers:
- kind: io.l5d.experimental.k8s
  host: localhost
  port: 8001
routers:
- protocol: http
  label: demo
  identifier:
    kind: io.l5d.path
  baseDtab: |
    /ns    => /io.l5d.k8s;
    /iface => /ns/kltest;
    /srv   => /iface/incoming;
    /http  => /srv;
  client:
    tls:
      kind: io.l5d.clientTls.boundPath
      caCertPath: /io.buoyant/linkerd/certs/cacertificate.pem
      names:
      - prefix: "/*/*/*/{service}"
        commonNamePattern: "{service}"
  servers:
  - port: 8080
    ip: <IP_ADDRESS>

When I change the k8s namer dtab entry to /ns => /#/io.l5d.k8s, then I also have to change the tls boundPath prefix to /*/*/*/*/{service}, since the bound path for my service has changed from /io.l5d.k8s/kltest/incoming/svcname to /#/io.l5d.k8s/kltest/incoming/svcname.


DatastaxShardStateIO & DatastaxMetadataIO prepared statements

Updating DatastaxShardStateIO & DatastaxMetadataIO to use prepared statements & query for metrics async() (no batches or in clauses).

Coverage decreased (-0.01%) to 75.728% when pulling 7bcb7517218bec817ff49a6c4c13629c1cf12dd2 on usnavi:metadata_prepare into 108298853edcfbb9142cfa77256fa84594fa475d on rackerlabs:master.

lgtm :+1: Thanks for doing this!


question

Hello, is there any difference between train_fcos and train_net in this file?

no significant difference except train_fcos trains FCOS detector and train_net trains Faster RCNN detector.

ok， thanks for your answer


labhub.py: Change invite alias

labhub.py: Changed invite alias
Changed invite alias from 'invite' to 'welcome'.
Closes https://github.com/coala/corobo/issues/166
Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

@LifetimeAlone , this bug is to create an alias.  invite and welcome should both work, identically.  Thanks.
Also please rebase your patch. (See dev guides, or ask a mentor for help.)  Do not press the silly GitHub green buttons which cause an enormous mess.

Though "Add "invite" alias" would've been better - But it's fine.
ack 1f5d19f53f512fe76f745b6f9377de2059ad9d99

@rultor merge

@rultor merge

@jayvdb OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@jayvdb Done! FYI, the full log is here (took me 2min)


Lost nomad_use_consul functionality

With this commit: https://github.com/brianshumate/ansible-nomad/commit/8828144807c6a09c21ac420e82bd47f0433bb562
We lost the ability to auto-configure nomad based on consul.  Let's bring that back, unless there was a good reason to drop it.
If you agree, I'll submit a merge request.

:+1:  Can't remember if this was an accident or that I did it on purpose with the idea of bringing it back later. But I'd like to see the functionality back either way :)

Thanks for accepting the merge, closing.


fish loading does not seem to be working

Ref. lib/reel.rb:63
# FIXME: fish loading does not seem to be working

Fixed


Certain views are unusually slow to load

Slowness varies between user (ex. 40 s for @MatthewKhouzam, slightly longer for @arfio). @tahini suspects this is unusual and worth more exploration. Views to look into:

Irq Analysis – latency vs time
Futex contention analysis- latency vs time
System call latency – latency vs time
Flame chart

This is not urgent right now. @thefinaljob is working on #284 which will make this less confusing.
More context in #296 (comment)

It depends a lot on the trace being analyzed. In my case, it was pretty slow but the trace was >1GB. There should be some way of having a progress bar at least.

It depends a lot on the trace being analyzed. In my case, it was pretty slow but the trace was >1GB. There should be some way of having a progress bar at least.

Good point! I added that this was not urgent and that @thefinaljob is working on adding a progress bar #284.
I created the issue to have a record, but we can close it if eventually we realize it was just the file size.

All of the latency vs. time views are currently filtered out client-side (#508) since these views contain discrete data and scatter plots aren't supported yet #45


YYKit-Text Edit  crash

Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '<UILabel: 0x7fd733625800; frame = (10 0; 54.9102 40); text = 'Vertical:'; userInteractionEnabled = NO; layer = <_UILabelLayer: 0x6000022fbed0>> has been added as a subview to <UIVisualEffectView: 0x7fd733514c10; frame = (0 64; 414 40); layer = <CALayer: 0x600000194e60>> effect=<UIBlurEffect: 0x6000003c0b10> style=UIBlurEffectStyleExtraLight. Do not add subviews directly to the visual effect view itself, instead add them to the -contentView.'
"has been added as a subview to "

Fixed, not merged to master
commit :1787ae4693f1700ebc0bd7d75067e3e7a14eefd4


Make the ToFuture instance for ZIO polymorphic over the error type to…

… make the usage of Output operations with ZIO more seamless

What's up with the build though?

Build is flaky :/


Doubt_regarding_model_structure

Please correct me if I am wrong: Stacked Denoising Autoencoder needs to be trained layerwise upto my knowledge. I cannot see the code written in this fashion. Please correct me if I am missing on something.

At first, layer-wise training was useful for denoising autoencoder in small dataset. However, we can improve our performance by fitting a lot of data or RNN to predict our denoising spectrogram. That's why I didn't train in layer-wise way.
references:
https://hal.inria.fr/hal-01163493/document
https://www.citi.sinica.edu.tw/papers/yu.tsao/6258-F.pdf

At first, layer-wise training was useful for denoising autoencoder in small dataset. However, we can improve our performance by fitting a lot of data or RNN to predict our denoising spectrogram. That's why I didn't train in layer-wise way.
references:
https://hal.inria.fr/hal-01163493/document
https://www.citi.sinica.edu.tw/papers/yu.tsao/6258-F.pdf

Thank-you for clarification. I went through the code and could not find this method of pre-processing included. Do you have any other repository where you have implemented this?


segmentation fault (or different bad behavior) with too many files

I have too many gzipped files and in some situation there is no results where it have to be results omitting stats etc.
I was able to simulate similar (maybe related) problem with version 2.2.0:
$ cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.6 (Maipo)

$ mkdir /tmp/ag
$ cd /tmp/ag

$ python -c "import gzip;[gzip.open('tst{i}.txt'.format(i=i), 'w').write(b'some line') for i in range(22000)]"

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst11111.*' line  # this is good
tst11111.txt
1 matches
1 files contained matches
1 files searched
9 bytes searched
0.151214 seconds

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst1111.*' line  # stats is missing too!
tst1111.txt
tst11110.txt
tst11112.txt
tst11111.txt
tst11113.txt
tst11114.txt
tst11115.txt
tst11117.txt
ag: error read core: ag: truncated file: Bad file descriptor
tst11119.txt
Bad file descriptor

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst1111.*\.txt' line # segfault :/
tst1111.txt
tst11110.txt
tst11111.txt
tst11112.txt
tst11113.txt
tst11114.txt
tst11116.txt
tst11117.txt
tst11115.txt
tst11118.txt
Segmentation fault

version 0.31.0 doesn't seems to have problem in this simulation. Unfortunately I have still problem in real work where it don't find what has to be found.
I did another test with 2.1.0 version:
# don't forget clean /tmp/ag directory!

python -c "import gzip;[gzip.open('tst{i}.txt.gz'.format(i=i), 'w').write('some line {i}'.format(i=i).encode()) for i in range(22000)]"

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst9111.*\.txt\.gz' 'line 1+9'
0 matches
0 files contained matches
1 files searched
14 bytes searched
0.166568 seconds

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst1111.*\.txt\.gz' 'line 1+9'
ag: tst11119.txt.gz
truncated file: ag: truncated file: Bad file descriptor
Bad file descriptor

$ ~/work/the_silver_searcher/ag -z -l --stats -G 'tst1111.*\.txt\.gz' 'line 1111+9'
ERR: File ./tst11112.txt.gz failed to load: Bad file descriptor.
ERR: File ./tst11118.txt.gz failed to load: Bad file descriptor.
Segmentation fault

What do you see in /var/log/messages ?


package version create stops after 1 second - static resource with mp3

Summary
When trying to create a package version with an mp3 in a zipped static resource the package command stops without error right after the start.
Issue started right after update to the latest version  7.199.
Steps To Reproduce:
sfdx package version create -p "PACKAGE" --wait 60 -v DEVHUB -c
static resource:

Private
application/zip
sounds

The directory sounds includes a small mp3 of 2 seconds.
Expected result
Start package creation process
Actual result
sfdx package version create -p "PACKAGE" --wait 60 -v DEVHUB -c
Version create.... done
right after the start.
System Information
Mac OSX ZSH terminal
sfdx version 7.199

Same problem here (without any MP3) - Windows 11 / VSCode 1.78.0 / sfdx version 7.199.7:
sfdx package version create --package "xxxxx" --installation-key yyyyyyy --definition-file config/project-scratch-def.json --wait 20
Display is just:
Version create... -
Version create...
Version create... |
Version create... done

I'll try it with the --json param and post the result here. I was able to build other packages with the same version. When i removed the mp3 file from the static resources the package was built.
The version was not created in the org.

Nothing is displayed if I add the "--json" parameter, and I'm back to the prompt after 2 seconds.
I work with only 1 package (source files), so can't say about other packages.
I can create a new package "sfdx package create ........" without problems (same source files).
But when I use "sfdx package version create .....", same problem. Exit after 2 seconds.

@Stassy you can downgrade to version 7.188, that works for me atm.

As far as i know you can only select your version when you use the npm install

I'm not using a CI for this project, only on local.

@Dietermartens - I cannot repro what you're seeing.  I am using an npm installation of 7.199.7 on a macbook, zsh.  I created a public github repo for this issue and issue 2092 here: https://github.com/shetzel/pvc-no-output.  When I create a package and package version from the source in that repo, everything is created as expected.  If you notice anything missing from the repo I created let me know.
If you run the command with --dev-debug is there anything you notice in the debug output?  What version of nodejs are you using?

@shetzel I tried it with your repo and that worked fine indeed. So i started testing a few oher things too, and i think i found it. We are using a custom directory structure, where we split the metadata per app in a folder. Like the Falcon Template (https://github.com/sfdx-isv/sfdx-falcon-template) (https://www.slideshare.net/VivekMChawla/salesforce-dx-201-advanced-implementation-for-isvs#34)
Now this worked fine before (and still does in older versions) but I noticed when you add another app and also put a folder 'staticresources' in there, you can replicate the behavior. I submitted an example to your test repo.
This is a log file for what happens during the packaging when i add --dev-debug
sfdx-debug.log

@Dietermartens - what are the exact steps you are following with your fork that cause the bad behavior?  I've tried a bunch of combinations and it works every time for me.  What does sfdx version --verbose output?

There is no CI, only local and Node is 18.16.0
Here is my debug log
sfdx-debug-log.txt

@shetzel That is weird, so when i added the second staticresources folder that also included a zip upload i could reproduce the issue 100% of the tries. When i removed the zip in one of the folders it worked again.
Steps:

Create package version -> See TEST in sfdx-project
Create package version with basic options -> sfdx package version create -p "TEST" --wait 60 -v DEBHUB -c -k KEY

If you like I can create a short screen capture to show it
(node:91466) Warning: Deprecated environment variable: SFDX_AUTOUPDATE_DISABLE. Please use SF_AUTOUPDATE_DISABLE instead.
(Use node --trace-warnings ... to show where the warning was created)
(node:91466) Warning: Deprecated environment variable: SFDX_DISABLE_AUTOUPDATE. Please use SF_DISABLE_AUTOUPDATE instead.
CLI Version:
sfdx-cli/7.199.7
Architecture:
darwin-x64
Node Version:
node-v19.9.0
Plugin Version:
@oclif/plugin-autocomplete 2.1.9 (core)
@oclif/plugin-commands 2.2.14 (core)
@oclif/plugin-help 5.2.9 (core)
@oclif/plugin-not-found 2.3.23 (core)
@oclif/plugin-plugins 2.4.7 (core)
@oclif/plugin-search 0.0.16 (core)
@oclif/plugin-update 3.1.13 (core)
@oclif/plugin-version 1.3.3 (core)
@oclif/plugin-warn-if-update-available 2.0.36 (core)
@oclif/plugin-which 2.2.19 (core)
apex 2.2.14 (core)
auth 2.7.13 (core)
community 2.2.11 (core)
custom-metadata 2.1.17 (core)
data 2.3.11 (core)
deploy-retrieve 1.8.18 (core)
info 2.6.8 (core)
limits 2.3.13 (core)
org 2.7.2 (core)
packaging 1.16.8 (core)
schema 2.3.10 (core)
settings 1.4.5 (core)
signups 1.4.13 (core)
source 2.10.6 (core)
telemetry 2.1.6 (core)
templates 55.4.11 (core)
trust 2.4.12 (core)
user 2.3.9 (core)
sfdx-cli 7.199.7 (core)
OS and Version:
Darwin 21.6.0
Shell:
zsh
Root Path:
/Users/dietermartens/.config/yarn/global/node_modules/sfdx-cli

Are you able to find a solution for this one? I am stuck on the same issue; the debug logs are producing the same results. I have tried on the previous versions as well but no luck

Are you all able to successfully source convert the same package?  E.g., sfdx project convert source -p force-app -d converted or does that also silently fail to actually convert the source?
2 other teammates and I have been unsuccessful reproducing any of the package version create issues.  I've been working in a docker container for the past hour with no success.  All of the debug logs I've seen from people who can repro the problem have similar output.  A ComponentSet of source is created and should be converted and zipped, but there is no debug output that anything was zipped.  For example, my debug output contains:
sf:core TRACE Setup child 'srcDevUtils#zipDir' logger instance +56ms
sf:srcDevUtils#zipDir DEBUG 205603 bytes written to /var/folders/_x/0dvp4tv91zbfy4d31lff9fd40000gp/T/0Ho4p000000bnxMCAQ-b6b5552740577a27790602805a18f254/package-version-info/package.zip in 56.456ms +0ms
sf:core TRACE Setup child 'srcDevUtils#zipDir' logger instance +57ms
sf:srcDevUtils#zipDir DEBUG 205213 bytes written to /var/folders/_x/0dvp4tv91zbfy4d31lff9fd40000gp/T/0Ho4p000000bnxMCAQ-b6b5552740577a27790602805a18f254/package-version-info.zip in 12.500ms +13ms

This suggests that the problem is in the source-deploy-retrieve library, most likely during zip operations.

The current theory with this issue is that it's nodejs version related.  Looks like nodejs v18.16.0 and higher have this problem but downgrading nodejs to below 18.16.0 should workaround the issue.  We're continuing to investigate why.  I'll post updates as we find them.

We've got some workarounds and a fixed RC available.  I'd love to know that this is working for you.
https://github.com/forcedotcom/cli/issues/2125

On Node 19.9.0 -> Same issue with version 7.199.7 or 7.200.7
On Node 18.15.0 -> Both versions work.
So I guess this ticket is a duplicate/ variant of #2125

It’s happening because of static resource of zip type. I was able to make
the package by removing the zip file.
On Thu, May 11, 2023 at 9:51 AM Dietermartens @.***>
wrote:

On Node 19.9.0 -> Same issue with version 7.199.7 or 7.200.7
On Node 18.15.0 -> Both versions work.
So I guess this ticket is a duplicate/ variant of #2125
https://github.com/forcedotcom/cli/issues/2125
—
Reply to this email directly, view it on GitHub
https://github.com/forcedotcom/cli/issues/2116#issuecomment-1544030675,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOISTDIUKWHPJWEVIIQZO3XFTVG5ANCNFSM6AAAAAAXYK4KR4
.
You are receiving this because you commented.Message ID:
@.***>


web UI: add text aspect for pure text files

https://perkeep-review.googlesource.com/c/perkeep/+/18466

@timatnuf The above CL pretty much only works for files with a .txt extension for now, since something is wrong with our mime type detection (I'm looking into it), but you can already try it and see for yourself if you want.

Yes, I'll take a look over the weekend.
My JS skills are not the best but I'll see about that, too
Tim
On Thu, Sep 6, 2018 at 4:22 PM mpl<EMAIL_ADDRESS>wrote:

@timatnuf https://github.com/timatnuf I still have to double-check some
things, but the text file detection should now be working for most files
containing text. Could you try
https://perkeep-review.googlesource.com/c/perkeep/+/18466/2 out and let
me know what you think?
If yes, would you feel confident reviewing it as well?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/perkeep/perkeep/issues/1214#issuecomment-419228394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZ9V1LH4fGaYCsz7KW-Xn8ZZJtykQm7rks5uYYP7gaJpZM4Wb3SE
.


feat: Add allowlist script and manual github action

Description
Allows additions to allowlists via manual GitHub action.
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[ ] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] If package-lock.json has changes, it was intentional.
[ ] The base of this PR is master if hotfix, develop if not

I've pushed this to master of my personal branch and it's working correctly. Can try it out here: https://github.com/timjrobinson/frontend-v2/actions/workflows/allowlist.yml

Replaced by #3446


Npm Install Errors?

I can't seem to npm install the dependencies not sure whats going on here? I deleted everything and tried cloning, removing the origin remote then running npm install and same errors. Not, sure whats going on here. Just wanted to let you know this is happening.
Here are my versions;
Git: version 2.31.1.windows.1
Node: v14.16.1
Npm: 7.11.1
The errors when running npm install

UPDATE:
Okay so managed to find out the issue. I installed npm, node as global packages through the command line but, also used the node installed on node website. So think this was conflicting with each other.
I removed the global packages via npm un package name -g and then went through the process again cloning the repo cd into the simplefolio removed the remote added my own remote and then npm installed and seem to work this time.


NestedScrollView physics not changing as expected

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Wrap a SliverAppBar and a CustomScrollView in a NestedScrollView
Create a ScrollPhysics class that slows down the useroffset by a factor
Apply this phyics to the NestedScrollView and the CustomScrollView

Expected results
The scroll motion gets slowed down
Actual results
The scroll motion only gets slowed down when the SliverAppBar is fully collapsed AND we are scrolling down. When scrolling up and the SliverAppBar is collapsed, it is the same speed/the applyPhysicsToUserOffset method does not get called (as when we are scrolling down and the SliverAppbar is slowly collapsing)
Code sample
Code sample
import 'package:flutter/material.dart';

class SlowScrollPage extends StatelessWidget {
  const SlowScrollPage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Slow Scroll Example')),
      body: NestedScrollView(
        headerSliverBuilder: (BuildContext context, bool innerBoxIsScrolled) {
          return <Widget>[
            SliverAppBar(
              title: const Text('SliverAppBar'),
              pinned: true,
              expandedHeight: 200.0,
              flexibleSpace: FlexibleSpaceBar(
                background: Image.network(
                  'https://placekitten.com/400/200',
                  fit: BoxFit.cover,
                ),
              ),
            ),
          ];
        },
        body: CustomScrollView(
          // Apply the SlowScrollPhysics here
          physics: const SlowScrollPhysics(),
          slivers: <Widget>[
            SliverList(
              delegate: SliverChildBuilderDelegate(
                (BuildContext context, int index) {
                  return ListTile(title: Text('Item #$index'));
                },
                childCount: 30, // Number of list items
              ),
            ),
          ],
        ),
      ),
    );
  }
}

class SlowScrollPhysics extends ScrollPhysics {
  final double scrollSpeedRatio;

  const SlowScrollPhysics({ScrollPhysics? parent, this.scrollSpeedRatio = 0.25})
      : super(parent: parent);

  @override
  SlowScrollPhysics applyTo(ScrollPhysics? ancestor) {
    return SlowScrollPhysics(
        parent: buildParent(ancestor), scrollSpeedRatio: scrollSpeedRatio);
  }

  @override
  double applyPhysicsToUserOffset(ScrollMetrics position, double offset) {
    return super.applyPhysicsToUserOffset(position, offset * scrollSpeedRatio);
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.13.9, on Microsoft Windows [Version 10.0.22621.2428], locale de-DE)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.6)
[√] Android Studio (version 2022.3)
[√] Connected device (4 available)
    ! Device R58M56F1JEH is not authorized.
      You might need to check your device for an authorization dialog.
[√] Network resources

• No issues found!

Thanks for the report. Reproduced this issue on master channel as well.

Demo video
https://github.com/flutter/flutter/assets/104349824/68d34975-7ecb-4e6a-90e7-2c802427fb1c

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.13.9 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (11 days ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Users/huynq/Downloads/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.84.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.76.0

[✓] Connected device (4 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D                         • android-arm64  • Android 11 (API 30)
    • iPhone (mobile)  • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.7.9 19H365
    • macOS (desktop)  • macos                                    • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome                                   • web-javascript • Google Chrome 118.0.5993.117

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.17.0-4.0.pre.20, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.17.0-4.0.pre.20 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 89692884a1 (70 minutes ago), 2023-11-07 17:53:22 -0800
    • Engine revision 3e3be5e33b
    • Dart version 3.3.0 (build 3.3.0-102.0.dev)
    • DevTools version 2.30.0-dev.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.84.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.76.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 119.0.6045.105

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

May be related: https://github.com/flutter/flutter/issues/34316


Failure with latest hapi, typescript etc.

I get a fatal error if I combine hapi-async-handler (1.0.3) with hapi 16.0.1, asyncfile 2.0.2, node 7.0 and typescript 2.1. The process outputs the following and the request stalls.

(node:5494) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: text.replace is not a function
(node:5494) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Example handler:
handler: async function (request: hapi.Request, reply: hapi.IReply) {
        let newsFile = Path.join(assets_folder, 'news.md');
        let input = await fs.readFile(newsFile);

        let converter = new showdown.Converter();
        let html = converter.makeHtml(input);
        reply(html);
      },

Correction: The first bug warning is due to readFile that should be readTextFile.
There is still a major problem though, as promise failures are left unhandled in the current implementation.


[RFC] Alternatives to Awkward Variant Modifiers

Linked from https://github.com/stardust-ui/react-old/issues/2.
Feature Request
Problem description
In Semantic UI V1 we supported modifiers/variants such as:
<Button basic />        // this is cool
<Button basic="very" /> // this is not

The problem with this is that it's extremely unintuitive, reads awkwardly, and overloads what should be a boolean or otherwise mutually exclusive property (basic).
Proposed solution
This issue is designed to encourage conversation around alternatives to this problem. One simple idea would be to support something like:
<Button basic />
<Button veryBasic />

Better ideas are encouraged.

We are dealing with this on a case by case basis.  The agreement is to not have any property-value pairs that make for awkward english reading.


Log: show the next test from the beginning, not from the same place

When switching from a test to the next one, show the test from the beginning, and not from the same place where it stopped when viewing the previous test.

Can't reproduce. Are you on the latest version?

Should be fixed


vertical position missing

Hey @jeatheak,
I have a Mitsubishi Heavy Industries - SRK50ZSX climate.
After testing all possible settings I have noticed one vertical position is missing.
SWING_3D_AUTO = "3D Auto"
SWING_VERTICAL_POSITION_1 = "Highest"
SWING_VERTICAL_POSITION_2 = "Middle"
SWING_VERTICAL_POSITION_3 = "Normal"
SWING_VERTICAL_POSITION_4 = "Lowest"
SWING_VERTICAL_AUTO = "Up/Down Auto"
There is a vertical position between "Middle" and "Normal".
If I change vertical position via remote from "Middle" one step lower the integration still shows "Middle".

First I have thought its because a general remote control but I can see the climate wing have these vertical position.


HIP-RA-X technical review; Migrate redundant GEOPHIRES water property functions to utils IAPWS implementations

https://github.com/softwareengineerprogrammer/GEOPHIRES-X/pull/7: https://github.com/NREL/GEOPHIRES-X/issues/109

https://github.com/NREL/GEOPHIRES-X/issues/101

Moved redundant water property calculations in Reservoir and WellBores to call IAPWS implementations
In some cases, existing examples generate temperatures beyond IAPWS bounds, added previous calculations as fallbacks for a workaround, marked with FIXME TODO to address as part of wider SHR work
Most example results were impacted by a small percentage by the updated water property calculations, hence the large diff.

https://github.com/NREL/GEOPHIRES-X/issues/95
https://github.com/NREL/GEOPHIRES-X/issues/105

Changes covered by updated unit tests, also updated & tested web interface.

Thanks for the approval @malcolm-dsider!

It looks like you are switching to a new code model for printing error message:
msg = 'Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....' model.logger.fatal(msg) print(msg)
Instead of: model.logger.fatal('Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....') print('Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....')
Is it your guidance to do this going forward?

My guidance would be to not both print and log - ideally log messages should go to the logger only. However in this case I preserved the existing behavior in case there is some functional reason why both printing and logging were configured - I'm guessing it has to do with incomplete/misunderstood logger configuration that caused someone to add print as a fallback when they weren't seeing messages they expected from the logger. The change I made here is just to define the logged/printed message in a single variable to cut down on duplication and improve readability (which I needed to chop away at to understand what was going on with HIP-RA code).

The idea behind both printing and the logging the messages is that these different approaches serve two different classes of users – users who are running the Python app on their desktop via the command line need to see the messages printed to the screen; users who run the web app need to see error messages provided by the server, most likely generated in the structured log file. And server admins are used to looking into the server log files to track down errors.
If there was a way to tell the logging software to do both things, perhaps via a switch, then I’d be happy to drop the print statements. Or perhaps we build our own logging module that handles print, logging (using the same logging library we are already using), and/or both with just a one-line call in our code?
From: Jonathan Pezzino @.>
Sent: Monday, February 5, 2024 9:19 PM
To: NREL/GEOPHIRES-X @.>
Cc: Malcolm Ross @.>; Mention @.>
Subject: Re: [NREL/GEOPHIRES-X] HIP-RA-X technical review; Migrate redundant GEOPHIRES water property functions to utils IAPWS implementations (PR #108)
Thanks for the approval @malcolm-dsider https://github.com/malcolm-dsider !
It looks like you are switching to a new code model for printing error message:
msg = 'Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....' model.logger.fatal(msg) print(msg)
Instead of: model.logger.fatal('Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....') print('Error: GEOPHIRES failed to Failed to validate CLGS input value. Exiting....')
Is it your guidance to do this going forward?
My guidance would be to not both print and log - ideally log messages should go to the logger only. However in this case I preserved the existing behavior in case there is some functional reason why both printing and logging were configured - I'm guessing it has to do with incomplete/misunderstood logger configuration that caused someone to add print as a fallback when they weren't seeing messages they expected from the logger. The change I made here is just to define the logged/printed message in a single variable to cut down on duplication and improve readability (which I needed to chop away at to understand what was going on with HIP-RA code).
—
Reply to this email directly, view it on GitHub https://github.com/NREL/GEOPHIRES-X/pull/108#issuecomment-1928711977 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AWGVYTZ324MX2OK5PDHBYGTYSGOK7AVCNFSM6AAAAABCVQXCKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYG4YTCOJXG4 .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AWGVYT5KSIFZMT6ZODDXWZTYSGOK7A5CNFSM6AAAAABCVQXCKGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTS6XHSS.gif Message ID: @.*** @.***> >

If there was a way to tell the logging software to do both things, perhaps via a switch, then I’d be happy to drop the print statements.

Yes this is possible by configuring multiple handlers for the logger. In fact, this seems to already be present in logging.conf - the consoleHandler is configured as a StreamHandler with sys.stdout: https://github.com/NREL/GEOPHIRES-X/blob/bcaaf2120b591821a23649d68776ec84938a2212/src/geophires_x/logging.conf#L14-L18
I have looked at the logging configuration with some amount of scrutiny previously but deferred any major changes as it's always been comparatively low priority. I don't recall the exact conclusions, if any, I came to about what was working and not working and what should be changed, but here are a few key points that occur to me with a fresh look just now:
Having just looked now, I think the first thing I'd do is change line 16 from WARN to INFO as setting WARN level means INFO messages will be suppressed. I suspect the duplicitous use of print arose because someone tried logger.info, didn't see anything printed to the console because INFO is suppressed by the current settings, and thus they added print to brute-force output to the console instead of changing the handler level.
Another issue is the passing of the model object to its composite children classes for the sole purpose of providing access to the logger; example: https://github.com/NREL/GEOPHIRES-X/blob/8533d39ea7d1ec537f900435abc4a1c37445aeee/src/geophires_x/LHSReservoir.py#L44-L55
This creates a web of unnecessary dependency which causes a number of issues with typing and class inheritance (IIRC one of these issues was I tried adding type annotations to methods that take the model but was unable to do so because it created a circular dependency error). I think the correct solution to this is to simply add a private logger field to each class instead of passing the model, as I did in this PR in GeoPHIRESUtils:
https://github.com/NREL/GEOPHIRES-X/blob/8cbcc999b6946c9871ecb9d2fe4b00714334f7dd/src/geophires_x/GeoPHIRESUtils.py#L17

So to summarize:

I'll just re-iterate that the specific change I made here was not a change to behavior in logging nor did it have to do with logging behavior specifically; it was just basic programming best practice of declaring a variable instead of repeated hardcoded constants where the hardcoded constants in question just happened to be messages to be logged.
I would continue to treat proactive refactoring/overhaul of logging as low-priority for now - if it ain't broke, don't fix it
I created tracking issues capturing the enhancements that could be made when bandwidth/priority permit:

https://github.com/NREL/GEOPHIRES-X/issues/111
https://github.com/NREL/GEOPHIRES-X/issues/112

Follow the private logger pattern as in https://github.com/NREL/GEOPHIRES-X/blob/8cbcc999b6946c9871ecb9d2fe4b00714334f7dd/src/geophires_x/GeoPHIRESUtils.py#L17 for any required new instances of loggers


Mouse Capturing not working on Godot

Tested versions

Used Godot 4.3 Web Editor**
Downloaded Project Source for others to try on native version of Godot 4.3 to see if it works

System information
Godot 4.3 Web Editor
Issue description
The code written for mouse capturing isn't working for the Godot Web Editor. Here is my code:
extends CharacterBody3D

var gravity = ProjectSettings.get_setting("physics/3d/default_gravity")
var speed = 5
var jump_speed = 5
var mouse_sensitivity = 0.002

func _physics_process(delta):
	velocity.y += -gravity * delta
	var input = Input.get_vector("left", "right", "forward", "backward")
	var movement_dir = transform.basis * Vector3(input.x, 0, input.y)
	velocity.x = movement_dir.x * speed
	velocity.z = movement_dir.z * speed

	move_and_slide()
	if is_on_floor() and Input.is_action_just_pressed("jump"):
		velocity.y = jump_speed

func _input(event):
	if event is InputEventMouseMotion and Input.mouse_mode == Input.MOUSE_MODE_CAPTURED:
		rotate_y(-event.relative.x * mouse_sensitivity)
		$Camera3D.rotate_x(-event.relative.x * mouse_sensitivity)
		$Camera3D.rotation.x = clamp($Camera3D.rotation_x, -deg_to_rad(70), deg_to_rad(70))

I don't think I did anything wrong. Do I have to download Godot to be able to use mouse capturing?
Steps to reproduce
My code here:
extends CharacterBody3D

var gravity = ProjectSettings.get_setting("physics/3d/default_gravity")
var speed = 5
var jump_speed = 5
var mouse_sensitivity = 0.002

func _physics_process(delta):
	velocity.y += -gravity * delta
	var input = Input.get_vector("left", "right", "forward", "backward")
	var movement_dir = transform.basis * Vector3(input.x, 0, input.y)
	velocity.x = movement_dir.x * speed
	velocity.z = movement_dir.z * speed

	move_and_slide()
	if is_on_floor() and Input.is_action_just_pressed("jump"):
		velocity.y = jump_speed

func _input(event):
	if event is InputEventMouseMotion and Input.mouse_mode == Input.MOUSE_MODE_CAPTURED:
		rotate_y(-event.relative.x * mouse_sensitivity)
		$Camera3D.rotate_x(-event.relative.x * mouse_sensitivity)
		$Camera3D.rotation.x = clamp($Camera3D.rotation_x, -deg_to_rad(70), deg_to_rad(70))

Minimal reproduction project (MRP)
fps_game_2024-10-28_22-29-44.zip
That is my project source. Could anyone please open it up on the native / downloaded version of Godot 4.3 and see if mouse capturing works? If it does, please notify me on Github so I know that I have to download Godot 4.3 to be able to use mouse capturing.

-		$Camera3D.rotation.x = clamp($Camera3D.rotation_x, -deg_to_rad(70), deg_to_rad(70))
+		$Camera3D.rotation.x = clamp($Camera3D.rotation.x, -deg_to_rad(70), deg_to_rad(70))
+
+func _ready() -> void:
+	# Or somewhere else where you initialize the game.
+	Input.mouse_mode = Input.MOUSE_MODE_CAPTURED

You don't have Mouse Capturing enabled.

It is only working for up and down. Could anyone please give me any code for looking left and right using mouse?

The mouse is getting captured but not able to look around with it.

You need to correctly map the event‘s axes to the camera’s axes. This requires writing appropriate logic.


Remove the Linear-Nonlinear model example

This was confusing error when I presented the notebook to him.

That was very straightforward.


Equal comparisons fail on models containing a list of itself.

Goals
I have a category model which has a list of subcategories. A subcategory is just a category itself.
class Category: Object {
  @objc dynamic var name = ""

  let subcategories = List<Category>()
}

Comparing the category with another query result fails the comparison even though the categories are the same.
This issue was introduced with Realm 3.0.0.
Expected Results
Making an equal comparison between two categories that are the same results in true.
Actual Results
The comparison fails because the subcategory list is different every time.
Steps to Reproduce
let realm = try! Realm()
let category = Category()
category.name = "category"

try! realm.write { realm.add(category) }

let search = realm.objects(Category.self).filter("name == %@", category.name)
let search2 = realm.objects(Category.self).filter("name == %@", category.name)

print(category)
print(search.first!)
print(search2.first!)

category       == search.first! ? print("We are equal!") : print("Category NOT EQUAL to SEARCH")
search2.first! == search.first! ? print("We are equal!") : print("SEARCH NOT EQUAL to SEARCH2")

Results in the following output:
Category {
	name = category;
	subcategories = RLMArray<Category> <0x600000306ff0> (

	);
}
Category {
	name = category;
	subcategories = RLMArray<Category> <0x600000307350> (

	);
}
Category {
	name = category;
	subcategories = RLMArray<Category> <0x600000307500> (

	);
}

Category NOT EQUAL to SEARCH
SEARCH NOT EQUAL to SEARCH2

Version of Realm and Tooling

Realm framework version: 3.0.0
Realm Object Server version: None
Xcode version: 9.0 (9A235)
iOS/OSX version: iOS 11
Dependency manager + version: None

@tuacker The == comparison uses the isEqual: method. This works as you would expect if the primary key is set. Otherwise, is just considers the equivalence of the object identity. You can use isSameObject(as:) to get what you expected instead.
Please see also: https://github.com/realm/realm-cocoa/blob/master/RealmSwift/Object.swift#L275-L292

Thank you for the explanation, I was not aware of this. I've actually only used == comparisons in a couple unit tests so this didn't affect any actual code. isSameObject(as:) is gonna be just fine for what I need.


Refactors recipe form components

Refactors the FormComponents, all common code has been moved to a parent BaseComponent. File has been moved to common/form.
Refactored the Ingredient and Direction lists. Lists reside in DataList.js and list items in DataListItem.js.

Fixes #78.

Code Looks Great! Though I'm having an issue with the state not being able to update when I add any text to the directions or ingredients. If you can't reproduce I'll dig into it more tomorrow.
Uncaught TypeError: this.setState is not a function
    at Object.update [as change] (http://localhost:8080/bundle.js:66726:13)
    at Input.handleChange (http://localhost:8080/bundle.js:66845:21)
    at Object.executeOnChange (http://localhost:8080/bundle.js:21529:35)
    at ReactDOMComponent._handleChange (http://localhost:8080/bundle.js:21350:39)
    at Object.ReactErrorUtils.invokeGuardedCallback (http://localhost:8080/bundle.js:13983:17)
    at executeDispatch (http://localhost:8080/bundle.js:13768:22)
    at Object.executeDispatchesInOrder (http://localhost:8080/bundle.js:13791:6)
    at executeDispatchesAndRelease (http://localhost:8080/bundle.js:13185:23)
    at executeDispatchesAndReleaseTopLevel (http://localhost:8080/bundle.js:13196:11)
    at Array.forEach (native)

Good catch, not sure how that slipped by, should be fixed now.


site.options.dev not set on _config.js when using --dev

Is supporting --dev specific config a thing?
It seems that site.options.dev is always false inside _config.js and it's only set afterwards.

dev mode is used to render pages in draft. And you can set it in _config.js by const site = lume({ dev: true }) or override with the --dev option in CLI.

yeah, I get that. But wouldn't it make some sense for it to be set on the lume() call, so it can be used inside _config?
For the a case like this:
const site = lume();

site.options.location = new URL(site.options.dev ? "http://localhost" : "https://mysite.com");

that can be controlled by the command line flag.
I'm not 100% sure if that's a reasonable use case.

For this specific use case, you can set a different location from CLI (--location=url). But I'm agree that dev configuration could be useful in _config.js.
Maybe a environment variable, like DENO_ENV (like NODE_ENV) or LUME_DEV (specific for lume) could help.

@fserb FYI, this is fixed in 0.22.2 (released 2021-06-07).


Define and implement a health check endpoint

The API should provide a health check endpoint for the live environment to use and establish if a container is healthy enough to service requests, to establish this:

[x] Check if there's a connection to the database
[x] Check if there's a connection to the queue processor
[ ] Check if the application can log

Do not attempt to read or write from the database, it's should be a very quick endpoint to respond

docker compose provides a way of performing healthcheck even at a compose  configuration:
healthcheck:
  test: ["CMD", "curl", "-f", "http://localhost"]
  interval: 1m30s
  timeout: 10s
  retries: 3
  start_period: 40s

once this is completed we should move compose and k8s to use the endpoints. This is now pending #7


Sealed classes should be treated as effectively final when it comes to pattern matching

Disclaimer
I am aware, that the issue I am about to report may be a non-issue after all, because I am just missing an important mechanic with sealed classes, that makes the change I am suggesting impossible to implement. Or maybe there is already an analyzer/linter rule that covers this case, and I am just not aware of its existence.
I did look through the existing issues, but did not find one about exactly this question/suggestion, so I decided to create a new one.

Dart version: 3.5.1
Minimal reproduction: https://github.com/Yegair/pattern_never_matches_value_type_reproduction
What is the problem?
Assuming there are two sealed classes sealed class Foo and sealed class Bar, where no implementation of Foo implements or extends Bar and vice versa.
sealed class Foo {}

class FooImpl extends Foo {}

sealed class Bar {}

class BarImpl extends Bar {}

When pattern matching a value that is known to be of type Foo at compile time against a pattern of Bar(), it should be clear, that this pattern match will never succeed.
switch (FooImpl()) {
  // ❌ does not cause an error or a warning like: pattern_never_matches_value_type
  case Bar():
    print('matched Bar()');
    break;

  case Foo():
    print('matched Foo()');
    break;
}

The problem is, that no warning is shown, neither by the compiler nor the analyzer, explaining that the pattern can never match the type. I.e. running dart analyze on this code causes the analyzer to report: No issues found!
It is possible to get the desired warning by adding a final modifier to all implementations of Foo.
sealed class Foo {}

- class FooImpl extends Foo {}
+ final class FooImpl extends Foo {}

sealed class Bar {}

class BarImpl extends Bar {}

Now trying to pattern match on Bar() correctly yields an analyzer warning pattern_never_matches_value_type.
switch (FooImpl()) {
  // ✅ correctly causes warning: pattern_never_matches_value_type
  case Bar():
    print('matched Bar()');
    break;

  case Foo():
    print('matched Foo()');
    break;
}

However, assuming that sealed classes are effectively final, it should be possible for the compiler/analyzer to figure out, that a value of type Foo can never match the pattern Bar() even if the implementations of Foo are not explicitly marked as final.
Why does it matter?
I did a rather large change on a big project, replacing a sealed class FooFromApi in many (but not all) cases with another sealed class FooFromDatabase.
Unfortunately there was no support by the analyzer/compiler that helped finding all the cases where patterns were still trying to match FooFromApi(...), that should now instead match FooFromDatabase(...). In the end, I overlooked one such pattern that was not covered by tests, which then introduced a bug into the application logic.
Since the project uses Freezed to generate most of the data classes, it is not possible to make the implementations of the sealed classes final. Thus it would be really helpful if the analyzer/compiler could take care of this, since the usage of Freezed is very common in Flutter/Dart projects these days.
Even without the usage of Freezed or a similar tool, one might just forget to put final modifiers on all implementations of sealed classes, in which case support by the analyzer/compiler would also help a lot.

The compiler is correct.
Your example allows someone to write
class Baz implements FopImpl, BarImpl {}

Adding final prevents that.

That is true, and I fully understand why the analyzer/compiler can not know what subclasses might exist in the general case, but isn't the whole point of sealed classes to give the compiler exactly this information? I.e. after compilation, there should be no way to add another subclass to any class that is extending/implementing a sealed class, or do I get something wrong about how sealed classes work?

Sealed classes only tell you direct subclasses, so you know that any implementations of Foo implement FooImpl.
But you are allowed to implement FooImpl.
But you can add the final modifier to FooImpl, in this case the analyzer will know that only subtypes of FooImpl are in this library.

Thank you for the clarification, I think I understand now why it is the way it is 😊.


[Feature Request]: Support SetState in SparkRunner

What would you like to happen?
Currently, SparkRunner does not support SetState.
We need to support SetState to close the gap with other runners.
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Infrastructure
[X] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Transform numpy vars to tensors

Reordering the if statements allows torch.ToTensor to be applied without breaking things. mask_array is cast to np.int64 as its default type, np.uint64, isn't supported by some pytorch functions.
This method still results in errors in some cases. See followup comment for my thoughts on implementation issues.

I believe there may be an error when dealing with z_size>1

Ok, I think this is my final alteration. Of note:
Images are returned in channel first mode (C, X, Y) for 2D images and (C, Z, X, Y) for 3D images. The 3D shape is consistent with the result of the old ver with the torch.ToTensor transform. The 2D shape was originally (C, Y, X) after torch.ToTensor which would not have impacted training, but I swapped the last two axes for consistency
I moved the transform if statements into the 2D block as pytorch does not support transforms for 3D images, and the ToTensor scaling ([0, 255] --> [0, 1]) is integrated into the dataset code.

I'm getting outputs where image_arrays of format (z, y, x) map to mask_arrays of (x, y). Can someone verify?

Some minor notes! For task 2, we need to change the defaults to --task MTNeuro/taskconfig/task2_2D_3class.json --network MTNeuro/networkconfig/UNet_2D_3class.json
Installing on python3.8 I found cloud-volume and pyrsistent were missing dependencies. will continue verifying new functionality with torch tensors before merging.

Looks like task 1 is also reliant on the toTensor transform being applied to the mask array which stores a 256x256 "image" of roi labels for each example. Is there a reason for this pixel-level region labeling? The BossDBDataset class takes the first index from the flattened mask_array and using that as the region label for the entire image.

ToTensor addressed in broader PR #27 closing